labofsahil/github-event-dataset-2012 huggingface dataset:
@@class_variables are bad in Ruby. You don't want to [ever use them](http://www.oreillynet.com/ruby/blog/2007/01/nubygems_dont_use_class_variab_1.html). Essentially, they don't just apply to themselves and subclasses, the apply back up their parents and into other siblings. So, we use class instance variables instead which don't suffer from this problem.

CONSTANTS are meant to be constant. Once they're populated, they should not be changed. Ruby doesn't inforce this, but it does throw warnings. So instead of using a CONSTANT I just went with traditional class methods that use class instance variables to memoize results when appropriate. In the end it's really about doing the least surprising thing. Manipulating a CONSTANT is surprising, so we should avoid doing it.

Not that it really matters, but I don't see how a Ruby class is a constant. If you feel like explaining that idea more, I'd be interested, but if you don't have time, don't worry about it. I'll look into it myself.
ugh, yes.
ok, i've reverted it.
main thing is that the slot arg isn't used anymore except checkArgument.. should it be a parameter?
a scalar with a default value being an array ? really ?
we can set it to a calendar object once created but it can't be instantiated as a calendar object. the calendar class is abstract.
Incepand cu <a href="https://github.com/MPS-UPB/Unknown-Revengers/pull/32/files#L3R63">linia 63</a>.
Ultimately I'm compelled by Vinz' argument for doing this (to keep the non-ISR safe and slow stuff out of an ISR).  I would actually think this same should be done for the HMC5883L driver.  However suddenly we are eating up a lot of ram on small tasks.

So we should ask ourselves for future (not blocker): is there a way to generalize to register on a single high priority drivers thread that is watching various sensor queues and reads them as appropriate (so the registration method would pass a semaphore and callback basically)

And how does this interact with the higher level sensors thread.  Can we maybe even make the drivers push into a hardware agnostic data queue?
Copyright date is important! Copyrights expire after a while - a long time
though, so a date makes sense.


On Fri, Dec 7, 2012 at 6:02 AM, Kenn Sebesta <notifications@github.com>wrote:

> In flight/targets/Bootloaders/Quanton/Makefile:
>
> > @@ -1,6 +1,7 @@
> >  ###############################################################################
> >  # @file       Makefile
> > -# @author     The PhoenixPilot Team, http://github.com/PhoenixPilot
> > +# @author     The OpenPilot Team, http://www.openpilot.org, Copyright (C) 2009.
> > +#             The PhoenixPilot Team, http://github.com/PhoenixPilot
>
> In case it's ever important (and I doubt it would be) we should put 2012
> (2013?) whenever we add to the existing header.
>
> The PhoenixPilot Team, http://github.com/PhoenixPilot<https://github.com/PhoenixPilot>,
> Copyright (C) 2012.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PhoenixPilot/PhoenixPilot/pull/54/files#r2348509>.
>
>
link_to on redmine1.4 produced an anchor with onclick handler. redmine2 does not anymore. It produces
```javascript
<a class="close_sprint" rel="nofollow" data-method="put" href="/redmine/rb/sprints/infrepma/close_completed">Close completed Sprints</a>
```
That is handled by redmine2 prototype javascript code. (binding on anchors containing data-method attribute)

I learned:
1. we do not include prototype code.
2. we must not rely on redmine javascript framework.

as of 2. redmine master/trunk will drop prototype in favor of jquery. They then will handle active anchors for put/post/delete using jquery-ujs by requiring the gem "jquery-rails".

This file is the javascript part of jquery-rails active link handling taken from jquery-ujs.
It enables backlogs to use link_to statements without further changes in backlogs code for redmine1.4,2 and trunk.
Ещё будет исключение при пустой строке.
You don't need to reparse this. We already do this when the request comes in:
https://github.com/divoxx/em-http-request/blob/ca893b84a526262fbdb8dce6ddd856e07cfd4777/lib/em-http/client.rb#L258

Generally speaking, I've tried to stay away from URI normalization because broken servers depend on broken URI's (sigh). Having said that, missing path... I hope is safe.
Formatting ;)
I'm unfamiliar with this methods usage - what are the types that can be sorted on? Maybe add a docstring.
I'd rather see a method called "type" and some properties for each of these locators.  This would allow you to liberate many of these methods.  

As it is right now, the tests that use these have to pass in a parameter explaining what's being typed anyway so you get blah.type_support_url(app[support_url])  or similar.  Stereo is nice but I think mono would be better here.
We should remain to single quotes here too.
Please add a comment that shows what the dict will look like with the values.  Someone will need to be able to look in here and quickly see what you're generating.  Write now I have to dig through the default parameters and find where they fit in the dict with all the string stuff.
I thought implicit waits were just that: implicit.  Does it needs to be specified, even?
"Less then ... " should instead be "Less than..."
Any better idea to get the number of lines then using `.split('\n')`?
Can we use `utils.extmath.fast_svd`? That would allow us to support sparse matrices.
I prefer to be explicit when using the axis argument of a ufunc, and write '.sum(axis=1)'.
It is probably not problematic in this specific situation. In general, @GaelVaroquaux argued that using inheritance instead of ducktyping prevents people from being sklearn compatibile without building a dependency on it.

As I said, I would to that in the more specific classes an not here. That would get rid of any tests and I don't see a downside. Maybe I'm overlooking something.
set  **the** amount
You have a missing indentation here, thus the 'topic' does not display right.
Could be.. we follow that in SA, but it might just be my bad spelling .. haha
Yes it would be better to follow PEP 257:

```
"""One iteration of Kalman Smoothing.

Calculates posterior distribution of the hidden state at
time t given the observations from times [1...t].
```

One-liner short description, blank line, then details.
Hopefully, yes ;)
More meaningful error message would be nice here.. plus probably shouldn't be blasphemous ;-)
Suite should have an e both in filename and in class name.
throw errors instead of warn/broken for all of them
Retourne la liste des paniers qui contiennent un document spécifié par son identifiant d'enregistrement.
Why add all files in directory? 
It looks to me, (after reading the man pages quickly) as if inotify_add_watch can monitor the directory directly.
You're not returning the events anyway, but if you monitor the directory, you can still find out which file the event corresponds to:

>>"When monitoring a directory, the events marked with an asterisk (*) above can occur for files in the directory, in which case the name field in the returned inotify_event structure identifies the name of the file within the directory."
And fix getPathName if necessary... its also only called here.
Is this directory name right? Would we have to mount the files volume here? If so, maybe we can make it /opt/files like the app servers.
Why is this file need? I don't see any changes?
well, the assertion are not supposed to fail. But a failure in them does not provide error reporting cleanly to the end user (and AFAIK, the is no way to change the message they get to be just a bit more friendly)
Shouldn't this method throw an exception such as an AssertException here, instead of failing silently and returning null?
Isn't treating the key/value pair as a full connection string a little overly general? It also means that if somebody types in "this=that;something=other" that'll parse too, won't it? Is this the desired behavior?
```exploit``` is already aliased by ```Auxiliary```
Please avoid comments in the JavaScript, since it's going to be delivered to clients when using the module. Please inclue this documentation (it's very interesting) as ruby comment above.
Thanks, this was the missing part of the docstring that confused me last time--all good now.
Capitalize sentences.
Thank you for removing these post-if conditionals.
bindigs -> bindings (nusnus)
I noticed that there is a little ajax when you click vote. Selenium clicks through so quickly that I think it is causing an error message to say 'there was an error processing your vote' You should wait for that ajax/POST action to complete before moving to the next page.
You can leave off the "string", e.g. "@type {Object.<twig.ExtensionInterface>}".

Can you also fix the indentation?
I agree with Scott that a general purpose rectangle object would probably be useful here (and probably other places in the future).  I assume the properties would be left right top and bottom.
I suppose I have to see the rest of the code first, but why return `undefined`?  Wouldn't we rather queue it up?
existing, not exiting
Good point!! :)
this should be `latest/#!` -- make sure the link resolves in the browser.
highlightedColor also available on android
For information, apt-get and aptitude permits the use of the flag -y to use the yes.
Why is there a space after $msg?
I would suggest passing in a dictionary, say ``p``, so that you can get the needed parameters, i.e. ``p['lam']``, to get rid of the numerical values and reduce the number of arguments you pass in.
A formal definition of a Type is shown here: https://github.com/phpDocumentor/phpDocumentor2/blob/develop/docs/PSR.md#appendix-a-types

A function, property, method or constant is not a type and is thus identified by a FQSEN. Basically; what this means is that the signature should contain only FQSEN since I cannot think of a situation where you'd want to indicate that you use 'an integer'
I think this might leave us open to an SQL injection, depending on how user_geo_edit_download.island_ids is populated?
Second readme?
This will fail with  `@disable this`. I'm thinking of replacing with this:

```D

            if(auto p = typeid(T).init().ptr)
                deinitialize(source, *cast(T*)p);
            else
            {
                static if (hasContextPointer!T)
                {
                    static ubyte[T.sizeof] empty;
                    deinitialize(source, *cast(T*)empty.ptr);
                }
                else
                    memset(&source, 0, T.sizeof);
            }
```

But I'd rather do this in another pull, and concentrate on static arrays/structs deinitialization correctness for now.
The `///` lines do appear in the generated docs. The old doc-comment is completely moved to the template member function.

BTW I had second thoughts on this pull request, as it may be somewhat disruptive due to the long diff. So I made another request manually adding the signatures:
https://github.com/D-Programming-Language/phobos/pull/773

Of course this first request has the nice property that the signatures in the documentation can't get out of sync with the actual code.
Never create a function named `init`. It probably shouldn't even be legal. It will cause problems with the type's `init` property.

Also, thish should just stay as a constructor. One constructor can call another. The new constructor would then be something like

    this(R input, size_t howMany, size_t total, Random gen)
    {
        this.gen = gen;
        this(input, throwMany, total);
    }
I wish I could, but opSlice isn't a template: That was the reason there was a problem to begin with: compile error on definition of SortedRange, regardless of opSlice's use.
Can we use a rabl template for response formatting?
Maybe use a better variable name?
tab this down to be within the li
why removing the method ?
This one currently fails.. i believe inject_foreign_key_for_new_association() doesn't work for a "has many through" relationships. Needs fixing.
Adding an extra comment that hopefully makes it a bit clearer:

// Annotations can have a reply button when they are merged with comments (and thus stored as comments). Custom annotations will not have this button displayed.
var replyButton = container.down('a.commentreply');
...
Using id fields make it somewhat difficult to implement step definitions. If we simply use names, the tests may become easier to read and the step definitions can be implemented using Model.find_by_name(...)

ie. in the next table replace patient_id and registrar_id with patient_name and registrar_name
Wait, why is this back here? This stuff should be in the `_drug_edit.html.haml` partial.

Did you pull from master before working & pull requesting?
I thought I killed this.  It's not used and will be removed.
Sure ! But maybe we could add a symfony_cmf_routing_extra.multilang.locales entry in the configuration and use it ?
`CaseStep.objects.create(caseversion=case_version, number=step_num+1, instruction=new_step["instruction"], expected=new_step["expected"])`
I guess this test still needs to be written?
I think that `version(Posix)` makes perfect sense within D code which is using POSIX functions which are declared elsewhere or which needs to differentiate between how a larger platform functions (e.g. std.datetime has PosixTimeZone and WindowsTimeZone which are tied to their respective platforms, so POSIX and Windows systems need to act fundamentally differently there). As long as the stuff which differs across POSIX systems is properly abstracted, I see no problem with using `version(Posix)`. The problem is with C declarations (like here). Those need to be platform-specific so that we're sure that each declaration is correct for its respective OS.
Find human readable names for date facets and merge those in the array for normal facets.
There's been some confusion on this style's internal name.  Right now, both are supported, but the actual name will be `valentines_land_of_rising_hearts`.
Awesome; thanks!  This is the first time I've contributed to another open-source project, so I'm looking forward to seeing how this turns out.
That should go one line further down, I guess.
Uh, this is a strict version requirement to make. Why?

django-datastream is a library, it should not require so strict dependencies.
I would move this into the parent file. The plugin is not large enough to really warrant two files IMO.
Okay, removed `attr_accessible` and `update_attributes` calls.
Need to use Ruby standard underscores for variables. Might also be good to use something more descriptive like ```extra_options``` instead of ```x_opts```.
here, you should add some assertions.  I would set a variable to the result of saveUserLocation, then assert that an ID and a UUID are assigned.
Spelling error!! I don't really care
... if **constructor** parameters ...
这里定义的所有函数，如果是全局型的，就应该命名为hotquestion_xxxxxx()，以避免和其它模块的函数名冲突。

最理想情况是做成面向对象的。想挑战吗？
Nice removal.
Ok. 

Also should we change the test node names to a convention similar to phpcr? As we add more tests here that might help avoid confusion.
They use lists in tastypie.


On 9. apr. 2012, at 17:29, Mitar wrote:

>> +        bundle = self.full_hydrate(bundle)
>> +        new_index = int(bundle.data['id'])
>> +        lst = getattr(self.instance, self.attribute)
>> +        lst.pop(index)
>> +        lst.insert(new_index, bundle.obj)
>> +        self.instance.save()
>> +        return bundle
>> +
>> +    def obj_delete(self, request=None, **kwargs):
>> +        index = int(kwargs['index'])
>> +        self.obj_get(request, **kwargs)
>> +        getattr(self.instance, self.attribute).pop(index)
>> +        self.instance.save()
>> +
>> +    def obj_delete_list(self, request=None, **kwargs):
>> +        setattr(self.instance, self.attribute, [])
> 
> Do they use lists in tastypie?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mitar/django-tastypie-mongoengine/pull/1/files#r660492
> Hm, it is based on [this MongoEngine field](http://readthedocs.org/docs/mongoengine-odm/en/latest/apireference.html#mongoengine.SortedListField). I have never used myself, though. So I am unsure how exactly it behaves. But our name should match it, no?

Umm...you are extending tastypie, not mongoengine. tastypie has no SortedListField.
> If it is possible to have a list of related resources from another collection

This is something else then. A list of references probably.
le temps **de** hacker **sur**
If this were a long running process, yes that would be a good idea.  In this case, it would add another layer of nesting (would require a try/catch inside the existing try/catch.  Further, the call to writer.close() requires it's own try/catch.  

It would just make a mess of a utility method IMO.
PS: "transaction is deactivated" == (commit | rollback | give up, a.k.a. destructor)
I don't think it's true that the folder must have a "standard name", since it's locale-dependent.
\# Request가 들어온 커밋에 대해 위와 같은 방식으로 리뷰를 달아줍니다.
마크다운(markdown) 서식을 쓸 수 있어서 링크나 이미지 등도 쉽게 넣을 수 있습니다.
it is ``doctrine/mongodb-odm-bundle``
Docs updated to reflect radial gradient limitations. Fortunately, there aren't any limitations on start/end points for linear gradients.
hide any soft input area when normally appropriate (when the user is navigating forward to your window).
What's that magic string?
Does it have to be the classes simple name or should it be some unique ID actually identifying an OO-element?
wops, the maps part, should be getApiKey, will change.

not sure what you mean by "agnostic" ? that the method should'nt be fixed ?
This should be a unique id
Can you clarify what the point of these 2 variables are?
pass it where and how?
love to change it if you know how I can pass this
A simpler way to get -x is `(- x)`
A few commits ago I moved translations to `Admin.xx.yml`. After carefully reconsidering I see that was a mistake.

In this commit I revert that change and move those back to `Admingenerator.xx.yml`.

See [updated i18n docs](https://github.com/loostro/symfony2admingenerator.github.com/blob/feat-twitter-theme/documentation/i18n.markdown#general-messages) where I carefully explain which messages should be translated with `Admingenerator` and which with `Admin` (or `i18n_catalog` if set).