labofsahil/github-event-dataset-2012 huggingface dataset:
remove ModelManagerInterface::getModelIdentifier

This method is required by the interface, but not used at all by the bundle; 
may it be removed, or am I missing something here?

```
 AdminBundle git:2.0 ✗ ≠ git grep 'getModelIdentifier' ~/Ormigo/repository/ormigo/vendor/bundles/Sonata/AdminBundle
Model/ModelManagerInterface.php:    function getModelIdentifier($class);
```
Removed default template passing to setTemplates in Mongo and PHPRC

Removed default template passing to setTemplates in Mongo and PHPRC config files.
https://github.com/sonata-project/SonataMediaBundle/pull/239
Added typehint for Model\Product::addOptions

Seems like this function should only accept an array of options (maybe an Optionset?)
Export the markdown form fields

I currently use these on two projects, would be nice to not have to copy the source out of the haddocks.  Figured I'd save you the trouble and just ask for the export via pull request.

Thanks,
Pat
v0.4

It now does everything as advertised... login & display thru DNApi, shout from local.
menu lavet

Her er min menu!
Projectiles tweaks and cleanup

Added a slight increase to visible size proportional to distance from the camera. Allows for smaller projectiles without losing as much visibility.

Added width to Equip::lasers for maximum bikeshedablity.

Added a specific glow texture rather than sharing with thrusters. Matched it better with the side texture. Cleaned up both a little. Added a fourth glow quad to make the view angle transition smoother.

Organized code better. Added and clarified comments.
Made installation tabs clickable

Renamed language strings to tab id instead of number
Cleaned up some code syntax
Optimize select and lock in destroy_descendants

...at extend past the deletion point.  Same as issue reported in #119
UserModel Admin Cleanup and UnicodeSlugModel -> SlugModel

- created_by and updated_by no longer show up in the admin. 
- UnicodeSlugModel is now SlugModel. Old slug model has been deleted.
- All base models now inherit from cachemodel.
- Moved UserModel admin logic into UserModelAdmin and removed unnecessary setting of created_by and updated_by.
Gimme!

bladibloo
changed the URL of API to ver.1.1

old version of the URL is not available.
returns errors.

 {"errors":[{"message":"Sorry, that page does not exist","code":34}]}
Repository tidy

Fixes #3.
Tutorial Fix

Replaced all instances of 'factory' with 'forge'.
Azure 11744 right amqp
listing page: before/after feature+cut string title
Added initial Loreal Paris config tempate file
fix gio makefile

fix gio makefile
DUDU
Updates

Changed $OPENSHIFT_RUNTIME_DIR to $OPENSHIFT_DATA_DIR and added minor settings.py improvements.
Webdriver phpdoc

Here is an updated/added PHPDoc @method annotations for most classes that you normally access while working with webdriver. Sorry, I don't use Selenium2TestCase thus properly documented Session is required.

One important fix - 7ac78cd68. Without it my Netbeans hangs on autocompletion.

Have a great weekend.
Update the specs and add failures for some use-cases

I've updated the specs a bit to try and work out what the expectations are for this gem. 

@tomtaylor - I've included your specs

@eshear - I've included your specs


I've also added a few things: 

1. Should we be raising a warning when we try to use symbol-keys? HStore doesn't support these, so either we warn that we are converting them to strings, or we store them as ":symbol" or something so they can be converted back. 

2. What should we be doing with literals? Should {a: true} come out as {a: "true"} ? 


I guess this all boils down to: Should we be making sure that any hash going into hstore is always identical to the one that will come out of it? 


Please comment on the new spec file, and once we've come to some agreement I'll try and implement everything needed.
Updated LiveDocx 2.1 component to ZF2 RC3

..and some more very small fixes.
Update grails-app/views/twitter4j/verifyPin.gsp

Just a detail in the text.
Added support for patternProperties. The way it does this is to collect ...

...all checked properties (from both properties and patternProperties) in a $checkedProperties array. Then it does the array_diff, as you did before, but instead it checks the original list of properties against the list of checked properties to see if there's any left over. Then if additionalProperties is false, it'll fail, and if additionalProperties is a schema, it'll check each additional property against it. NOTE: I did not do any unit testing; I didn't even check to see if there's any syntax errors, but I just thought I'd rough it out for ya. Enjoy!
모바일에서의 인덱싱

버그 fixed!
sidebar-close-via-X-image-74

jkakar:**approve**

This branch turned out to be misnamed. It doesn't seem to be possible to add event handlers to the iframe that's created on the pages we add the iframe to. It looks like we'll need to deliver JS to do that sort of thing from the extension endpoint on fluidinfo.com so that it's loaded by the iframe.

Fixes #74
iOS 6 instructions

Updating the readme to explain iOS 6 details to users.
Cgit fixes

2 Fixes:
* author_email instead of author
* split master_branch variable for validation on commit
[Proposition] Using NSOperationQueue instead of straight GCD to give the option to the clients of canceling the drawing operations
handle /(de)?objectify_text/ for <script> extraction

Many scrapers handle <script> data extraction via similar construct:

```perl
    $self->tree->elementify();
    $self->tree->objectify_text();
    if (my $script = $self->tree->look_down(
            q(_tag) => q(~text),
            sub { $_[0]->attr(q(text)) =~ m/var\saddress/sx }
        )->attr(q(text))
        )
    {
```

This patch makes HTML::TreeBuilder::LibXML compatible with it.
Next ablake: Fixing errors and warnings when building for QNX.
Sending a pull request

I attached my sample project. Please review this.
Tips for Guard and Testing External Services
Updates to your awesome Reset Wordpress Theme

... File

Some pretty modest updates but I love the blank theme you made and these few things were bugging me

1. Removed Modernizr 1.7 and replaced it with version 2.5.3. 
2. Added custom comment layout to the functions.php. 
3. Updated the direct link to JQuery on GCode pointed at the latest 1.7 release. 
4. Moved the respond form to the inc folder to keep the loop clean
Framework2.1

yummy url service based mp4 videos
Added as_view() method to SearchView, removed search_form_factory. Fixes #348.

I have added an as_view() method to the SearchView class, replacing search_form_factory.  The result is a slightly friendlier interface and some similarity with Django's class-based views.
Fixed super-verizon-pre2-2.2.4 script missing / is youtube copy...
Twerk twerk

Werk werk
fix Django 1.4 compatibility issue

```
Django Version:	1.4
Exception Type:	 TypeError
Exception Value: __init__() takes at least 3 arguments (3 given)
Exception Location: /image_cropping/widgets.py in __init__, line 60
    super(CropForeignKeyWidget, self).__init__(*args, **kwargs)
```

The signature for `ForeignKeyRawIdWidget.__init__()` in Django 1.4 is:

```python
class ForeignKeyRawIdWidget(forms.TextInput):
    def __init__(self, rel, admin_site, attrs=None, using=None):
```

The required argument `admin_site` is new to Django 1.4. The `formfield` method of `CropForeignKey` in fields.py doesn't include the `admin_site` argument:

```python
def formfield(self, *args, **kwargs):
    kwargs['widget'] = CropForeignKeyWidget(self.rel, field_name=self.field_name,
        using=kwargs.get('using'))
    return super(CropForeignKey, self).formfield(*args, **kwargs)
```

This patch inspects the arguments of `ForeignKeyRawIdWidget.__init__()` to see if `admin_site` is a required argument and patches it into the `__init__` kwargs for `CropForeignKeyWidget`. If admin_site is required it uses django.contrib.admin.sites.site as the value.

```python
class CropForeignKeyWidget(ForeignKeyRawIdWidget, CropWidget):
    def __init__(self, *args, **kwargs):
        if 'admin_site' in inspect.getargspec(ForeignKeyRawIdWidget.__init__)[0]:  # Django 1.4
            kwargs['admin_site'] = site
        self.field_name = kwargs.pop('field_name')
        super(CropForeignKeyWidget, self).__init__(*args, **kwargs)
```

Could be argued whether this belongs in the Widget init or the CropForeignKey formfield.
Some tweaks and updates for #341

Crossed a few things off my list. Small improvements.

Still needs heavy documentation.
Update: Localisation-Support (new text strings) + german translation

there you go :) now its up2date again ;)
subtle bugs introduced in last pull request fixed
Part12 fixes for clarity

Fixed two things :

  * the first example didn't work, the duplicate name '`HttpKernel`' makes the import impossible. I understand that this example is not supposed to be followed but it's here and most people will make the modifications to `src/Simplex/Framework.php` and `web/front.php`. For the sake of don-t-panic-the-newbie, all the code presented should work ::)
  * Linked to the first point, expose the final code of `src/Simplex/Framework.php` to make sure everybody gets it.
Fix array attributes and Added printers_spa
Proyecto de Joan Navarrete Sanchez

Se me han colado el .settings y las prefs.
Bootstrapping Melbourne post

Here's the first draft, with edits from Aleks. Hoping to get feedback in the next few days from Melbourne before merging and publishing for next week.

Of course, feedback welcome and encourage. Feel free to push to the branch.
/Emote change...

So yeah I'm extremely noob with GitHub and couldn't find out how to change the file name but... :D
Updating to lastest wu version
Runtime-detection tests updated to use RedDeer master + edited .gitignore file

Runtime-detection tests updated to use RedDeer master
Edited .gitignore file (Portal/Gatein tests)
explicit cast of webview index to integer
Ratings and song metadata fixes

The first commit allows ratings 0-5, which Google Music happily accepts and will display if you enable it in Music labs.

The second and third commits update the api to reflect the change from metajamId to matchedId (second code, third docs).  See https://github.com/simon-weber/Unofficial-Google-Music-API/issues/36 for more information.
Added sorting to Pagination. 

Fixes #108
Fix for http://jira.grails.org/browse/GPRABBITMQ-25

Cannot bind queues to fanout exchanges through Config DSL. This is a one line change.
some more fixes and improvements to bgs004
Hotfix/label submit

Addresses Issue #9: Button label when not using display group

https://github.com/Emagister/zend-form-decorators-bootstrap/issues/9

I, too, am seeing labels rendered for submit buttons.

This fix is a quck hack that creates a Label decorator extending Zend_Form_Decorator_Label that simply bails if the element is a submit button; otherwise, it passes along to the parent.
Fix Twitter_Bootstrap_Form if getView is NULL

hi,

in Twitter_Bootstrap_Form::_initializePrefixes adds an HelperPath to the View. Unfortunately if you are using Unittests on the Form there is no View. Therefore i added an is_null check.

best regards, dready
see bellow




+- Let plone viewable even without the diazo theme unapplied:

+  public and columns css are not overidden but then, not cookable anymore.

+  Thus we can remove them from being fetched inside the diazo theme. [kiorky]

+- Do not override the globalnav viewlet to be a full diazo template [kiorky]

+- Add 'last' class on globalnav last element to be 1140 compliant {kiorky]

+
Возможность удалять EXIF из JPEG-ов

Экономит траффик + `куклоскрипт заботится о приватности пользователей`.
filter directories away from public header listing

fixes builds followed by build action 'clean', I only tested the previous patch with empty build dirs. oops
CFLoader's autoloader method prevents other autoloaders from working

By design, spl_autoload_register should only return true when it successfully loads the required class otherwise it should return false to let other autoloaders give it a try. Specifically, if SwiftMailer (swiftmailer.org) is invoked after AWS (meaning SwiftMailer's autoloader happens after AWS), an uncatchable error occus because AWS' autoloader always implies that it worked.

This change is released under the Apache 2.0 license.
division by zero error resolved

Tweaked the code so that division by zero error doesnt occur if there is no negative sentiment found.
Added "-lang-c++" flag to connection unitTests cmake file
Updated to signals 0.9 BETA

Brings into line with the latest versions of Signals and the Robotlegs SignalCommandMap utility.
get vibration working

this enables vibration. it is tested and verified working.
maybe needs to be revisited, when (if) we have the sourcecode from lenovo.
vibration strength can also be tweaked later via (int)timer.

+ 

mkfile now  uses proper product_locales (mdpi).
timob-10099: Android: Facebook - Login failures

https://jira.appcelerator.org/browse/TIMOB-10099
Test case in Jira. Please also run the test case on iOS and MW to make sure nothing is broken.
added daemon text for de+es and some fixes and missing texts (de+es)
Вызывает scriptCSS только после завершения работы с DOM
Corrections faites

Il reste la question du "Bouge ! Bordel !" (il s'énerve sur la manivelle)
For consistency -- elsewhere we use get/setUrl for this.
'Cause Firefox doesn't render application/.., and no web library complains about text/plain.
This is not a good idea, this function can have other GC refs in its stack frame, or more can be added later. Also, mini/ should be free of GC dependencies, i.e. boehm-gc.c and null-gc.c should contain a dummy mono_gc_set_stack_end ().
+1 for the 0.7.17 release. Finding it popping up on my app.
The plan was to store DVDPlayer scaling method on the first 16 bit, and DSPlayer one on the other 16 bits, in order to avoid a db change. If the DS_SCALINGMETHOD_* have not been changed, I'm afraid you'll override DVDPlayer scaling method.
Wait, you guys still think @homakov a troll right?

https://github.com/rails/rails/commit/2b74968f03fa4fbdf8afaa497b8446cb00fbddcd
Need to chacnge {L_LINK8} to {L_LINK10}
我有 review 過，有修掉原來的一些不良轉碼，整體品質應該比原來的好不少。

反正應該沒任何人在用 s-tsin.src 。
Agree, they both do sound fragile. Also limit being used doesn't stop the behaviour, the following will apply the edit on every state run.

    /etc/nsswitch.conf:
      file:
        - sed
        - limit: "^passwd:"
        - before: "passwd:.*"
        - after: "passwd:\tfiles"

While `file.substitute` sounds good I still think adding a new option to `sed` is the easiest and most straight forward. In my mind the option should indicate that `after` should be matched before `before`, and it should default to False.
Ruby rocks :-)
Of course, you're always free to do `Class.extend Cls::ClassMethods` in your app. I wish you could do `Class.include Cls`, but `include` is private.
good idea, though you'll want to guard so that you don't join the same thread twice (start, sessionchange, stop).
Please send that to the mailing list in the form of a patch.

On Fri, Mar 9, 2012 at 1:42 PM, rofl0r
<reply@reply.github.com>
wrote:
> here's how to install it on SUSE
> sudo zypper in python-setuptools
> sudo zypper in fuse-devel
> sudo zypper in libacl-devel
> sudo zypper in python-devel
> sudo easy_install fuse-python
> sudo easy_install pylibacl
> sudo easy_install xattr
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/apenwarr/bup/commit/081ca808cd950ff6d1249b5fe27d958b4bf9d559#commitcomment-1066950
Un logo dans le footer c'est peut-être beaucoup. C'est proposé par http://humanstxt.org/, et moi j'aime bien cette démarche.
A voir si on le garde ou pas...
This one seems a bit silly. I'll approve the pull request, but revert this one after.
Looks interesting, but maybe should also have an assertion at the end to validate that the stacks did not get mixed up or mangled along the way?  As far as I can tell, this just verifies to make sure we did not crash, which is good, but there is probably yet more validation that could be done.
I love this patch. :-p.
Very helpful. I'm tired of restart the app like a monkey.
Good stuff! Let's get build/pip-delete-this-directory.txt out of the repo.
This is kind of an ugly way to do this. I'd suggest writing the size of the hsRAMStream separately from writing the RAMStream itself. Not only can you avoid this crap that way, but on reading you don't have to compensate for the fact that the size of the size marker is included in the size (say that 10 times fast)
Don't increment the iterator here, since you're (potentially) erasing later on
```coffee
randSelect = (array) ->
  random = (n) -> Math.floor Math.random() * n
  array[random array.length]
```
i have already did it on arcscripts2, these changes temporally are in private, it will pushed, when all of them will finished to move to database, due i'm going to move all gossip options to database - this will give full support of multilanguage.

there is one thing, that ALL of wow server development cores didn't have - poi names also should be localized...
amazing! I was looking forward to this commit :)
Probably worth opening this up for discussion on the forum.
try fixing some additional typos
Ooops sorry, my bad! :)
Yup, this was used just for stencil shadows. 

Then @zz85 also used edges in `SubdivisionModifier` but these seem to be separate implementation, just reusing some pieces from the old code.
这里多个空格
You're right. I'm going to sleep ;-)
Actually, I think we might scratch that for now. We'll need to bring in modernizer. for that shim for older browsers, right?
Even C doesn't handle this syntax case well. If we make "-" part of the literal syntax, then `-18446744073709551615` probably has to be an error, but I guess that's better since the number isn't representable at all. Anyway, it's not such a huge issue since this is what `typemin(Int64)` is for :)
顺便删了bootstrap里的多余文件
Hey, thanks I will take a deeper look at this.

Originally I had some problems with Python 3.x and that's why I decided to use a gzip module.
Hast echt nen krassen Scheiß hier zusammengebastelt :) Respekt
you should catch it by reference
still bug with priest dispell magic

Only have spanish screen, I will look for english

http://img18.imageshack.us/img18/5569/screenshotnav.png
use concat() to copy array
Is there any reason why we don't just allow all symbols in the files including windows.h by adding a single define before the include? Those should usually be system specific source files, thus it seems safe to allow all symbols there anyway. It seems a bit nicer than allowing setjmp and longjmp globally for mingw-w64.
do you want to delete [`NewsletterForm`](https://github.com/mozilla/zamboni/blob/master/apps/devhub/forms.py#L749)?
@sjl Yeah, I think it's probably worth it, just for the sake of being clearer
What you actually want to check is the lexsort depth
Normalizing the user agent checks to lowercase below makes sense, but why is it needed for the cookie check? The value comes straight from Mezzanine itself - if someone had an uppercase device name in their settings then wouldn't this code break?
no jruby-1.6.6 available on http://jruby.org.s3.amazonaws.com/

on mac, `rvm install jruby` is still refer to jruby 1.6.5.1 (ruby-1.8.7-p330).

on debian, `rvm install jruby` is correct.

why?
怎么？..
Fabrice, sorry for asking, but did you checked that there is no side effects on other tags ? I suppose that would be better to add a really complex example into tests ( with a different tags, nested, ... ). WDYT?
This is a heavily modified version of flipview, if you want the clean one get it from the original repo:
https://github.com/Reefaq/FlipView

Thanks
added possibility to embed javascript
my bad.  added the files in the latest commit.
So, I've changed this to a hook.
@silviapfeiffer can you look at how I fixed this for collection and do the same
What is this for? What is man/cat?
This will break on JRuby 1.6.6 and earlier on 64-bit Windows, because intptr_t was added for x86_64-windows after jruby 1.6.6 was released (see https://github.com/jruby/jruby/commits/master/lib/ruby/shared/ffi/platform/x86_64-windows/types.conf)
these might make the test easier to read if these are named IndexedAttributes_zero, IndexedAttributes_one, IndexedAttributes_two, etc
@khobbits I was not aware of this problem.
So if I manually send a packet of :
server.getHandle().sendAll(new Packet201PlayerInfo(oldName, false, 9999));
server.getHandle().sendAll(new Packet201PlayerInfo(name, true, getHandle().ping));

I can crash clients? Sounds like a minecraft bug :(
@CodeForFame

We need to move away from XStream? :(
What do you suggest I do/work on?
this includes only work on unix systems, because unistd.h only exists for unix. simlilar iptions are in io.h and dosio.h. so currnet version is only usable on unis system
You can use [Govspeak::Document#to_text](https://github.com/alphagov/govspeak/blob/master/lib/govspeak.rb#L26) to do this more simply; it'll also handle encoded entities properly.
just ignore "/test/mail"
@Sbte modified in relation with what you said. Added a log.debug to check If this is used.
Rather than embedding the URL construction into Trigger, I separated this out as a new helper function. Otherwise, I'd have to create a dummy BuildStepStatus class and then call getURLForThing on it, which seemed like a worse approach than just creating a new function here.
(To be clear, I'm complaining about youtube, not about vidscraper. IIRC even their descriptions are different between feed and api.)
These should all use "install".
I think we can update the build script to pass in --disable-optimizations, which will do the same thing. Otherwise Jenkins can't handle things.
what about ``OPTIONS`` ? This could be needed if you are testing a CORS webservice for instance
Mental note: must add a handler for anyone trying to access the site via .php calls of any kind. 
Maybe a recursive thing, if nothing can be found, just throw a redirect to /404 path? hmm decision...
Ah, I see it now, thanks.  Merging!
Ah, sorry about that.  It's pushing out now.

Alex

On 2012-01-12, at 12:26 PM, Lee Hambley wrote:

> You never pushed this version to Rubygems? http://rubygems.org/gems/css_parser
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/alexdunae/css_parser/commit/0970ca41e7c2869a2c0a4f253d8c298d4eb6d1b7#commitcomment-862265

--

Dialect // 778.992.2229 // dialect.ca
Hmmm, what happened here? Looks like the deleted and added code is the same - did github get confused about the diff? I am not going to review these files that appear to be identical.
Yes, it was tested, SQLite, Postgres and MySQL after an issue was found.  https://github.com/gregbell/active_admin/issues/34

https://github.com/gregbell/active_admin/pull/1063 is the pull request for this issue.
It was only 1 file right?  I just replaced the file since I didn't make any changes to it previously

This is what I get when I click restore stock

Fatal error: Call to a member function clear_product_transients() on a non-object in /wp-content/plugins/woocommerce/classes/class-wc-product.php on line 225
Ja, gissningsvis.
Interesting technique! One thing to be aware with this approach is that it in practise disables the option to have margins in ```em``` on the object where the ```.ir```-class is applied. Not a deal breaker, but sensible to keep in mind. :) In such cases one could of course resort to ```rem``` where supported.

What about prefixes for the text-shadow?
Here and a slew of indentation changes that follow actually set things to the wrong indentation. Please check to make sure that your editor is using 4 spaces for tabs, and wherever you modify lines replace all tabs with spaces.
Nothing wrong with this, per se, but I think I might have preferred to see it generalized in some way so you could define something like

```ruby
data_required = {:overall_stats => [:members_count, :communities_count, etc...],
:content_partners => [:content_partners, :published_resources, etc...]}
```

...somewhere useful and then generalize this block with something like:

```ruby
row << 'Created'
row += data_required[params[:report]].map {|which| I18n.t("activerecord.attributes.eol_statistic.#{which}")}
```

...or the like.  [shrug]  No big deal, just a thought.
That's true.  The default init was created since I thought there maybe some usefulness in instantiating the class without a need for a sprite but it's rarely used so it'll be better to remove that init and move all those initializers into initWithSprite.
Ja aber wem bringt das dann was?
Am ende hockst du alleine auf deinem Core und niemand kann sich an ihm erfreuen.
We should probably mention that this sound path MUST be relative to resources AND in the resources folder.
Does anyone know what this property is for?
Make these 5.2 obviously :)
since these maps only contain one entry, you could use the much more efficient SingletonMap returned by Collections.singletonMap(), which would also make thse one-liners
I thought you were going to say that, I just like the old code better since it is more explanatory. Either way works. Thanks.
hehe, this might not be the most informative... maybe use streetname?
AuthSocket.obj

rebuild solution
Oh yes. In our area their is small chance of collision, therefore I missed this one.
@drusk I was wondering why this cannot be done inside BoxedTextSvgElement, e.g. in the constructor.
It might be better to have specific groupings for background, arcs, nodes to eliminate this special case.
creature?
Should this strip the '.ecl' ?
No. Send another default message e.g. "person has been gagged by person"
django 1.2 syntax
oh yes. I did that in a slightly earlier commit, I was fumbling a bit with the merge.
cuando yo las vi morir de un golpe, fue con el escudo del pala tanke, ese que se lanza a varios enemigos
You're missing documentation of allowable `rel` attribute values for your links. These are needed as metadata to describe to clients how they may navigate from one representation to another.
Rather than having separate attribute values for `SearchSongGenre`, `SearchSongKey`, and `SearchSongTempo`, you might consider having a single `class` attribute value `SearchSong`, which could be applied to any of your templated query links for finding songs. This would more clearly communicate to a client that these are links are all alternative ways of getting to the same kind of resource: a filtered list of songs.
If you find a solution for TinyTDS, I would be more than happy to take a patch!
On what platform?  POSIX defines free(NULL) behavior, so you were perhaps looking in the wrong place.

   See [the specification](http://pubs.opengroup.org/onlinepubs/009695399/functions/free.html) or just [try it](http://codepad.org/79CxJDvD) (also [valid C++](http://codepad.org/M262JxsH))
Hey Eric,

There's a weird bug with Ruby's IO that causes it to hang when 1) it's
reading from the proc fs, AND 2) it's running in a separate thread. (the
agent uses threading to enforce timeouts). The bug only affects certain
versions of the Linux kernel. Centos 5.7 is affected, which was hitting a
number of our customers.

   - https://projects.puppetlabs.com/issues/4466
   - http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-talk/155744

Sadly, the only sane fix we could find was shelling out to cat.

Andre

On Thu, Jan 12, 2012 at 12:10 PM, Eric Lindvall <
reply@reply.github.com
> wrote:

> What was the reason for changing this?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/highgroove/scout-plugins/commit/d45be2f5fe9ba889c4bed7b8d7cd571523810e00#commitcomment-862229
>



-- 
Scout Web Monitoring and Reporting ~ http://scoutapp.com
blog: http://blog.scoutapp.com
I’ll fix it everywhere later.
@treyhunner 
tag has been made here.
It's hackariffic
As far as I can tell, 'path' is referring to an eclipse project name, not a filesystem path. The project name is specified as TodoTxtTouch in [the .project file](https://github.com/ginatrapani/todo.txt-touch/blob/master/.project#L3). We could change it there instead.
well that is a strange way to name a method
Fracking awesome yo!
@jbridson Wrong coding convention, please only use dashes, not underscores.
Two function calls are complicated? This is not a DSL, this is a minor convenience.
I like to have it in document, it makes document more readable and understandable. I don't like generate such things in js
changed title in gitHub to see how it handled code changes
You accidentally the verb
No, I did not. I just went ahead and copied the if out of the sbv1.ctp-file and forgot the closing-bracket. Thanks for pointing that out!
@jeffmccune Since we install to `/etc/puppetlabs/facter/facts.d/puppet_enterprise_installer.txt` on Unix, shouldn't we install to `File.join(Dir::CommonAppData, PuppetLabs, facter, facts.d)` on Windows?
With this step displayed RAM jumped up 1Mb (currently 343Mb, JW1) - not much but did not note any side effects yet.
Nice change!
I was working on more comments in PmsConfiguration and already wondered about thumbnail generation vs. thumbnail usage.
Ugh. That's horrible... Is there really not a better way of doing this? I don't understand why this would change in 1.9 anyway, surely this is a matter of what directories rails puts on the search path?
nit: NullCursor vs NullPointer. (Repeated a few times in the file.)  Also, it's better to throw this in the testMethod than to catch it and fail -- if the error does happen, you lose the exception traceback :(
I think no.
Because you should ideally always able to do a small commit apply for each certain task.
You know that node-websocket-server doesn't support the latest versions of the protocol, right?

Also, node-websocket-server packages with it a linkedlist storage of connections, which tended to perform slightly better at times.
Well it is easier to just at the p's now then to go back plus I do look at the formatted javadocs through eclipse (when I hover over the class name)
/src/server/game/Entities/Unit/StatSystem.cpp: In member function âvirtual void Player::UpdateAttackPowerAndDamage(bool)â:
/src/server/game/Entities/Unit/StatSystem.cpp:374: error: âGetShapeShiftFormâ was not declared in this scope
/src/server/game/Entities/Unit/StatSystem.cpp:377: error: jump to case label
/src/server/game/Entities/Unit/StatSystem.cpp:374: error:   crosses initialization of âfloat bearplusâ
do in your repo root folder :

rm -r vendor/cm
repo sync vendor/cm
I've briefly spoke with Andy and what he told me is that he hasn't touched the gridfield.
PFM is supposed to be a 'portable float' image container, in fact it's pretty much raw
DX can dump pfm's straight from the API so it's an easy one to implement.
yes, it's in both pypi (so you could `pip install -U cytoplasm`, if you installed with pip)  and the cytoplasm github now.

It's more than just a name change, though; `interpret_filelike` interprets a source filelike object to a given destination filelike with a given suffix. `interpet_to_filelike` interprets a path name for the source into a destination filelike. The latter uses the former.
did you push the name change (interpret_filelike) in cytoplasm interpreters to the main cytoplasm repo?
Haha, sorry, didn't mean that, wasn't sure if for some reason on Android it was named encodeURL.
Non mi piace molto questa API. Vedi https://gist.github.com/1555897 per una proposta embrionale di alternativa
90 te va Ludo ?
This worries me as you're running this on any AJAX call. Would be better to use explicit callbacks if possible.
Yeah, but it was bugging me, so it had to go :)
yeah and also setting smoothzoomsupported to true causes a graphical glitch with the slider, and it really doesntadd much smoothness, thats why I reverted them both.
Yeap! because the connection isnt closed instantly even using unbind!
If there is no such node, return the default one.
Hi! what's wrong with the formatting?
Yep, just working on this as/when I get time. Hope that's not a problem! I'll pull this module out soon.
Will mark.
does this need to be public?  If not make it private.
mkay I'll change it soooon
You are checking if this node is a singleton AND is also part of the other node's membership table? I don't get it.
It the companion to the public ByteIterator (which has to be public, of course). I guess it could be ```private[akka]```, in principle - would that increase binary compatibility? Or should I make some of the things inside private (this was not done for the ```ByteString``` object in the past)?
Is this really something we want to publish as samples?
Why lazy val and why are we caching the configurator?
Patrik saves the day. I'll fix
aux ctor
`if (Kohana::$environment === Kohana::DEVELOPMENT)`
This is for a later version and needs some thought, but we need to do the whole process more customizable and allow extensions to hook their own previews. It should be easier to add more formats than modifying the code, and extensions providing their own previews should be able to register them somehow (e.g wms preview on the spatial, some rdf viz, etc.)
branch names now like ``1224-some-feature``
I don't think PEP-8 mentions this specifically, but I think the same principle applies that testing for identity here works in CPython, but isn't guaranteed by the language, and isn't explicit what you really want to compare. You really want to know that the length equals 0 in the mathematical sense.
Crud, I meant to mention that nearly that entire repo.py file needs to be reworked and to not look too closely. In particular, there's not an ounce of i18n in there. That'll be another commit.
It seems odd this is being added.  I think this is the external representation we want for numeric (card_id) access, right?
I'd use iterators for both.  Not sure what the cast is for on the first one.
Yea. But method cookieExists was added in this commit https://github.com/Behat/Mink/commit/d4f07c77dc2e374fe775c491a33967e9830ea24c. This pull request adds cookieEquals method. Which code styled correctly. You can see it here: https://github.com/mekras/Mink/blame/issue3/src/Behat/Mink/WebAssert.php#LID100
THIS IS SO MUCH BETTER THAN "logs = simulator.run()" 

YUSSSSSSSssss
Think I should change it, eh?
Yea changed it to a more general example with the output also mentioned
sizeNoAppend() -> schemaSize() ?
Predefined value should be used (Locale.ENGLISH perhaps?).
This needs to be configurable. But defaulting it to environ['HOME'] is a good idea
Oh, lock_mode has to be a string. So forget about the theoretical value stuff.

Should we be enforcing an empty string being no lock?
Here, dnephin and eskil commented (on #43, please see https://github.com/Yelp/zygote/pull/43#discussion_r1788017)

The reason zygote_collection can return None is that zygote may have died and we receive a sigchld before we read the message. In that case reap_child removes the zygote. Can zygote be None in any means other than master getting sigchld? It looks like it can't and even so, this code doesn't the previous behavior it just guards.

I'll add and info log for this situation, but I don't think it should be an error.
I would probably name TestFile objects something like foo_test or just test.  So naming this "run" would be more clean to me:

```
test.run()
```
Shouldn't this be `traversalRules` here?
Cool story bro! ☺
Yes that would be best. I wasn't aware of this issue before this point so a test case illustrating the problem and a separate fix would be nice.
shancat/node@2772218781
Style: needs to align. Maybe use a temp var.
listen for the 'pause' event
I didn't think it necessary.  But I'm open to a community vote. An array of callbacks would all initiate from the same point in the event loop, so could cause confusion with the sequential operation of `promise.then(fnA).then(fnB)` vs the ungauranteed sequence of `promise.then([fnA, fnB])` - if fnA is synchronous, they are equivalent, if async, then fnB will complete before fnA (assuming fnB is synchronous or resolves faster than fnA).

I admit, this does complicate the use case of _wanting_ multiple callbacks on the same promise, as it requires breaking the chain. `var and = promise.then(fnA); and.then(fnB); and.then(fnC)` to get fnB and fnC initiated after fnA.

Maybe this could be resolved with promise.and(callback, errback)? I've seen that in other  implementations. Maybe supporting an array of callbacks would be easier.  But in either case, I wouldn't consider it a core requirement, since it can be emulated as shown in the broken chain snippet above.
Sure about not having tryFailure here?
The %a modifier is only for the total number of days between two _dates_.
Since there is only 1 date we don't have a difference.

On Sun, Aug 5, 2012 at 11:37 PM, Stanislav Malyshev <
reply@reply.github.com
> wrote:

> > +  int(0)
> > +  ["d"]=>
> > +  int(2)
> > +  ["h"]=>
> > +  int(0)
> > +  ["i"]=>
> > +  int(0)
> > +  ["s"]=>
> > +  int(0)
> > +  ["invert"]=>
> > +  int(0)
> > +  ["days"]=>
> > +  bool(false)
> > +}
> > +2
> > +(unknown)
>
> I'm not sure I understand - why it's unknown if the string says "2 days"?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/php/php-src/pull/148/files#r1309145
>



-- 
Lonny Kapelushnik
http://lonnylot.com
(732) 807-5509
I followed the pattern couple lines below
Thank you and best regards
Hongcheng Chang





On 5/17/12 5:12 PM, "Subbu Allamaraju"
<reply+p-842954-9c0a039e31604174dd37b356707b774616c48b2f-1302440@reply.gith
ub.com> wrote:

>> @@ -299,7 +299,9 @@ var Console = module.exports = function(opts, cb) {
>>  
>>  
>>                  // collect default query params if needed
>> -                _.defaults(holder.params, route.routeInfo.defaults);
>> +                _.each(route.routeInfo.defaults,
>>function(defaultValue, queryParam){
>> +                    holder.routeParams[queryParam] =
>>defaultValue.toString();
>
>Why toString()?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/ql-io/ql.io/pull/465/files#r842954
а это какой-то особенный коммент. пока я его не поставил, показывало ошибку при сборке. потом Саша сказала закомментить и оставить так.
I'd rephrase it to "you need to have lm-sensors installed". Looks more like English :).
Why do this? Also, inline if is probably better here anyway.
Make sure you add spaces after `(` and before `}`.
But since I've already pushed to master, I will not be able to squash them. Wouldn't you prefer it if I gave u just one single commit? I was thinking of just creating another branch with all the changes from your feedback and issuing another pull request. I have the branch ready. All I have to do is push it and issue another pull request. Whichever works better for you.
I think it's fine to leave them here, in case we want to change the default some time in the future, the example config to run AR tests doesn't need to change.
3-2-stable upgraded to Sprockets 2.2+ and switched to `env.each_logical_path(paths)` instead of handling `compile_path?` itself.
If we're going to check the api in other views, we might want to just turn it into a decorator.
I'd start with this check
IMPLEMENTATION_ARTIFACT is definitely too much, but the flag doesn't have to match the name exactly.  We should be discouraging direct flag manipulations anyway, internally as well as externally.

But if we are going to eat a really long name, I feel sure we can do better than isImplementationArtifact.
I think Josh just volunteered for that change, when he’ll need it ;-)
> I don't know whether heuristic comparison will work once we have adaptive sampling because it is randomized.
It will, because of a "resemblance" factor that was set very low.
However, it is not that useful as test.
>> +    [(4*g*sin(q2) + 10*r*u2*u3 - 5*r*u3**2*tan(q2))/(5*r)]
>> +    [                                          -2*u1*u3/3]
>> +    [                             (-2*u2 + u3*tan(q2))*u1]
>
> Please shift to the left those outputs (as they was).
> It seems that I was restore them incorrectly.
>
>  Or shift to the right the examples code lines.
>

They are shifted to the left in keeping with all other tests in that file.
What kind of exceptions are we trying to catch here? What are the cases when this would fail? When cmath doesn't have the function? Let's just catch this exception explicitly, so that all other exceptions are not silently suppressed.
@smichr interesting. I hadn't though of cases where a == b but I might not want them to be substituted. For now I'd like to leave it as is. I suspect I'll want `2.0` to substitute against `2` more often then not. This is something to think about though.
I'm not familiar with javascript. Is this valid syntax?
Yes! I did not know about it (I should have expected it to be implemented already).
just checking: you *don't* want to rewrite in this case?
Replaced with a span.
due to the way you call `.ani.plot.hook.html()`, `animate` is always `TRUE` here, so do you have to check `if (animate)` at all?
documentation does not say anything about that. It simply does nothing if it is not able to clean output.
`header_remove` and setting empty header is equal in case you do not use `mod_deflate` since apache will remove empty header.
but there is a difference you can see when you debug result with [headers_list()](http://php.net/manual/en/function.headers-list.php).
`header_remove` will really remove it form the list, setting empty header will leave empty header in the list.
When using mod_deflate it tries to extend the existing header even if it is empty by adding ` , gzip` to the end, which might result in browsers complaining about that.
ignored?
they get extracted on the line 57
cs - blank line required before final return.
a10dd13 has it
I think this should go up into the first section.
Can you explain this change in more detail? It looks like you've just changed the 2 existing callers to pass in the same value for the `states` and I'm not sure  why.
Move sync and ensureSameTransaction to `init`.
Not sure I follow. The default.xml isn't part of the Janky gem itself. If a user is upgrading, they'll need to manually modify the default.xml file they use.
/you can /d
we don't have celery running on prod
This is a very wrong definition of "common words" but if that's what the code does leave it be.
This really doesn't sound right.
I think that this should actually specify which requirements were not met, not just that it does not meet all requirements.
Is this safe from differences in phpbb_root_path?
unapproved -> pending? Not at all a fan of negatives especially in column names.
Ugh. realpath?
I'd like to see more variables being added here:
submit, preview, cancel, save, load, delete, mode (requires to be moved down some lines) and error (requires to be moved down some lines)

So one can overwrite these values, modify the mode and add errors.
This variable is not set in the constructor, therefor only listeners from styles/all/template are executed
Yeah.  I think there was a reason why it was that way before... changed..
cum am zis pe irc, nu e nevoie de vecini la superuser sau la Others, deci ramura asta trebuie scoasă
What's the reason for round-tripping this through the JSON parser? I think I recall seeing an issue about this but I don't recall the reasoning for it.

Which JSON parser is being used? Since we are making two additional calls to the JSON parser for every job we should make sure we're using the fastest parser we can.
This documentation is in the wrong place. Most of it isn't necessary, since it's already documented in the `mojito help start`. The fact that `options` can have a `perf` should be in the API docs for addMojitoToExpressApp().  Hmm... none of the other options.* are documented today, so you are forgiven :smile: .
на локалке проблема с кодировкой чтоли была, забыл удалить перед отправкой
The indenting in this module is a bit.. psychedelic :smile_cat:
Are these asserts necessary? Won't the function just fail later if they don't hold? Seems a performance hit that's undesirable in a library function.
Should be ```Ext.Msg.alert```
Use ``you`` / ``your`` to be consistent with rest of sentence
I'm not sure. They all write the same code and they all compile it the same way. So I don't think we can get failues due to that. What do you think?
It's a good thing to keep in mind.

I don't think conventions can avoid the need to clean up memory mid-way through a computation if it fails. As far as I know, there's not a very clean way around that using C. In C++, the way you get around that is the compiler calls the destructor for everything on the stack when it goes out of scope. In C, the closest you can get to that is calling free on everything that isn't NULL at the end of the function, and making sure that you always exit the function via the same exit point, possibly using goto statements.

The existing convention seems to be:
-If the function should return a pointer, NULL indicates failure and everything else indicates success.
-Otherwise, nonzero indicates failure and 0 indicates success.

The second case is needed for things like device_free, which don't have access to any valid pointers to return.
The first is for the policy enforcement which is a NOP. Got rid of the dead code.
Should be: invoice_date = models.DateField(default=datetime.date.today)
Unusual, but confirmed with Matthew that it's NaB. ("./sst_stats -opt1 file -opt2" results in opt2 getting silently ignored. However, the intent is to support "./sst_stats -opt1 file -opt2 file2".)
Shouldn't it be cfPS(fail)?
To parse very last close caption parsed.
You should use `raw_string_ostream` there too.
I'd like to call a friend on this one... my eyes tricked me into thinking [this](https://github.com/gibson042/jquery/blob/f7067566d1bda266e8bc9bc7c51d8755d068fbd4/src/data.js#L232-240) was deeper then it is.

Day 1 of TC39 + trying to look at PRs. Whoops!
I know, but client-side unit tests shouldn't have to be dependant on a flaw in the PHP implementation of json_encode.
It's so funny, every time I move code from one place to another in the test suite, it's got style issues. Fixing in a bit. Thanks for the eagle eye as usual.
I see. I am kind of convinced. Sticking to a specified anatomy for no special reasons is not necessary. So I just leave one more point here in case in the future we'd like to make a more serious decision on this. Some DBMS's (e.g., Oracle) allows users to provide an Initialize() function for a UDA. In such case, a peek at data tuple may not be as natural as in GPDB/Postgres. This is a point for portability.
"Parent", "Base", "Derived"... This seems to have everything necessary for confusing people in C++ class inheritance... I need some more comment or a conversation with Florian before I can try to understand this...
Could you please use the following style? (No spaces before semicolons in objects). This is also described under the "Spaces between tokens" section of https://github.com/meteor/meteor/wiki/Meteor-Style-Guide

{
  property: value
}
And all these TranslateFacing() calls probably shouldn't be here, facing should be done by the skill system. The client does it automatically for the player skills so I haven't yet added it in for monsters.
My only remaining nitpick since I didn't figure out any thing more about GEnums is that I would prefer on/off instead of On/Off, that or just do a ``lower()`` or ``upper()`` on everything so we ignore the capitalization of the user input.
Small refactor following chat with Chris and Luca.
a prendre en compte ;-)
I'd prefer not splitting this mid-word.
Like


````python	
def compose_message(self, subject, message):
    """A deprecated alias for send_message. Do not use."""
    raise DeprecationWarning('Renamed to send_message. Use that.')
    return self.reddit_session.compose_message(self, subject, message)

def send_message(self, subject, message):
    """Send message to subject."""
    return self.reddit_session.compose_message(self, subject, message)

````

I'll rename compose_message in LoggedInExtension as well and point send / compose message to the new function. This is an internally used function, so shouldn't need any warnings.
This whole block should receive a comment explaining what id does since it's not really the most descriptive piece of code.
Never mind, that's right after all.
Hoist these out to around line 37 to avoid evaluating them each time