labofsahil/github-event-dataset-2012 huggingface dataset:
Правильно. Но на exit(1) всё будет закрыто операционной системой. Т.е. это как бы лишний код. Плюс, то, что там делается System.exit(1) - это моё послабление. В реальности таких решений код внутри библиотеки принимать не может.
It is our design, when the token has passed 50% of the expiration time, we will renew it. The same design is used in service bus.
Hum, don't really like to add this clone function, since it is and will not yet be used elsewhere in the code.

I'd prefer you implement that:  `        , fieldOptions = $.extend( {}, options, $this.data() );`

By adding `{}` as fist param, jQuery will create inside this singleton the result of the `options` and `$this.data()` mergin. It will free 9 code lines ;)
I know it's not part of your patch, but can you remove this `throw` and replace it with a `return;`, I think it's better if we can use `closeToolbox()` without worrying if the toolbox is already open or not.
Spacing around the assignment (PEP-8)?
We should have a method applied to this locator instead of just using it, like is_visible or find_element.
Could you add some comments to this method - it's not clear to me what it's doing
Ok. Thx for the information. I will take a look at that this weekend and maybe add a new pull request with newer code that matches the current AndEngine master branch
次回コミットタイミングで修正したいと思います。
"If you are using FIJI, there is a separate updater" could be "FIJI includes an updater"
You can also setup Mongoid 3 using a configure block in database.rb directly, not sure we need a database.yml. Check out http://rdoc.info/github/mongoid/mongoid/Mongoid/Config#sessions%3D-instance_method basically you can just do:

```
Mongoid.sessions = {
  :default => { :database => "!NAME!_development" }, :hosts => ['localhost:27017'] }
}

To follow the convention of avoiding database.yml in our generator for mongoid
```
s'il y a possibilité autre chose que du xml sous forme de chaine. Sinon tant pis
After some more fiddling with the data I've got the filesize down to 69KB.  I hope this is sufficient enough for inclusion? 

@ogrisel This option could work, except for the significant amount of preprocessing I had to do before getting it into a ready state.   This is partially renaming variables, but also involves running a reference version of the Kalman Filter / Smoother / EM algorithm to check my results against.  I hope the dataset is now small enough for direct inclusion in `sklearn`.
Looks good to me:

    __pyx_t_6 = __pyx_v_n_outputs;
    for (__pyx_t_7 = 0; __pyx_t_7 < __pyx_t_6; __pyx_t_7+=1) {
        __pyx_v_k = __pyx_t_7;
навіщо це?
zomg :) nano seconds! Will do
"One stop shopping" means that it's a central place, not that you can't have it as a member. We'll also need it to be a member to connect to the levelChanged signal.
backgroundColor above likely also isn't supported on BB yet.
Done, though caught in Ti*
Need to update the documentation. For the others too.
I asked for it in a pull request : https://github.com/balupton/watchr/pull/7
Why do you remove this?
Is this in FreeBSD?
no, it is a bit weird - I copied it from the eclrtl.

It is using the parameter passed to setValue to initialise the associated value (the RBCLocale).  That creates the collator.

I could recode it, but it is reasonably clean.
It was moved incorrectly in gh-3391 (to before flags was set), causing :
>Also fixes a regression in all join, that was causing all rhs to match
Probably but I'm not convinced it would make things clearer
Do you really want to use ```get``` here? Either way, use a generator rather than list comprehension or personally, I'd use:
```python
for result in results:
    item = result.get('subject')
    itemlist.append(...
```
line too long (pep8)
This is now wrong and you broke the tests.  Please run the tests before sending a pull request.

The self.port == other.port is needed for self and other both being nil.
Yeah adjusting the policy, sounds the like the better solution. Alternatively you can insert an alias repository after each repository, that would also work, but that doesn't sound as clean as using the policy.
I don't really like the fact that this returns output. I think it should stick to the original way of taking a $output by reference and return the status code, that way you can properly check for failure. At the moment in getTags for example you just use the output without checking if it worked or not (granted that should have been done before as well).
Ah, i see, my bad. Thanks for pointing that out!
Can we do keyword arguments for the last two arguments on this method call? Not obvious from reading it what's going on.
Not sure what's going on here - the function ensureRepositoryUri takes context and continuation, not just the continuation. Am I looking at the wrong function?

Any why a "naked" ensureRepositoryUri instead of scm.ensureRepositoryUri?
Delete plz!
Override the navigateTo Method from DFBasePage
> Do you think why ToList returns List instead of IList?

I would say ToList() is a method whose purpose is to convert any IEnumerable (regardless which underlying implementation) to a special implementation: the List class. This fact is not hidden from the user. This can be seen as contrast to GetUntrackedFiles() where the focus lies in returning a collection of something (untracked files) where the concrete type of the collection is not relevant (except when performances issues or other things rule against it).

> GitModule is kind of utility class

I thought otherwise: GitModule is heavily used all around GE (244 occurances) and it can be seen as an abstraction to access git functions. Therefore general rules (like programm against interfaces) could be applied. There is also the interface IGitModule but with much less methods and less usages (32 occurances). So maybe IGitModule could be used on many places where currently GitModule is used.

> Hence the higher type you return, the more flexibility caller will have.

True. With the drawback of being forced to program against a concrete implementation. I just noticed that instead of AddRange the linq extension method Union could be used, could't it?
I see what you mean after off-github clarification.

@niqt, you need to split the search pattern as well.
Same response as earlier.
If you're not registering anything, then you don't need register_options at all.
I think you can save a byte here by just pushing edi directly here instead of putting it in ecx first.

```
metasm > push [edi]
"\xff\x37"
metasm > mov ecx, [edi]; push ecx
"\x8b\x0f\x51"
```
Not sure if this is an issue, but you are not sure client is not nil right here. Could be a bit more defensive.
Is there any reason to not check the response in this case to be sure this step was fine before going for upload?
This won't be right on systems where %SystemDrive% isn't  C:
Is there any reason to not use store_loot to store the information? Seems a good idea for me.
The Graphics Gems license definitely reads as if it's BSD-compatible. And apparently at least [one other person came to the same conclusion](http://wiki.call-cc.org/eggref/4/graphics-gems). Let's work under the assumption that this is fine, until someone suggests otherwise.

Unfortunately, we're a bit behind on our pull requests (lack of time/manpower), so I can't  promise that I'll review this soon, but it's not because of lack of interest. Hopefully, someone else will step up in the mean time.
We have already had migration 7 and migration 8. How about 0009_auto__ ... as the file name?
Although it may actually have been fixed by that patch I made to processDocuments() today, which seems to have boosted the number of translators passing their tests. It looks like it failed every day until today.
Does this throw exceptions?  If so, they should be documented?  Do we need to call this though?  Can't the extent be validated when it is created?  Or perhaps it is mutable...
Given `get2DMercatorYPositionFraction` in `CentralBodyVS.glsl`, couldn't we create a general `czm_` GLSL function?
If you view an extent on a map, should you be looking at the middle latitude or at the actual center of the extent on the map?  With a geographic projection those two are the same, but in mercator they're not.  Looks like you're doing the former here and in CV.
This can be cached from frame-to-frame as mentioned above.
Don't worry. This was added in the last commit. ;)
This statement will not catch outOfMemory errors.  Either include an OOM catcher or catch Throwable
How about we just test if tmpFiles is null OR empty. If this is true then just return.
Otherwise iterate through the files and delete each one. Once this loop is done
just null out tmpFiles (we don't need to clear since it will get GC'ed).
We should be logging these exceptions with Log, not printing the stack trace out.
Oops. That's a pretty big typo. Will fix.
@mstepanov it is guaranteed it will have atleast one object , thats why i am looking for lastobject and objectAtIndex:0 which could possibly be the same coordinates.
Could this be null if this view has no parent?
This stuff isn't required.
We should be sure to throw if the amount is not a number.
Please don't use static variables here. If you need to store state do it as a member variable.

The name also is misleading. Why should show once be a counter?

What is this variable used for at all?
incorporated the mentioned changes
Does it work for you? `CGI.escape` seems to escape too much here. I propose

    URI.encode(url_str, /[^-_.!~*'()a-zA-Z\d;\/?:@&=+$,\[\]+]/n)

We probably should not rely on how Ruby escapes it but make it so that Django decodes as intended. Therefore I think putting the constant string in there is a better idea.
Explain in log why it was skipped, and when will kick in.
Leave a note about the bug if necessary, but remove this comment otherwise. The comment below, "Tim Sort implementation", is adequate.
Fix my formatting :)
I would make these top level classes.
Kill the `@author`.
Maybe instead say "New {$note->class} note added to user {$username}."
All selectors within entwine should be scoped to its logical "root element", so in this case `this.closest('form')`
Is there a way to pull this stuff from geoserver so if we choose a different region it could snap to that instead, not really necessary, but could be important if we generalize some of this functionality out?
this is dangerous because if you use matchers like 'allOf', `describeTo` may be called without `matchesSafely` being called before (when the a matcher before this one already failed) -> NPE
The raxa time code does too much for a sparkline. I agree that there's potential for some DRYness, though that may be better suited for a future sparkline feature diff, not a bugfix diff.
http://mxr.mozilla.org/mozilla-central/source/netwerk/base/public/nsIURI.idl#105
Ok, we should fix this, and make the tarball easyblock more flexible for embedded tar's
https://github.com/hpcugent/easybuild-easyblocks/issues/51
That should be fixed in the compiler, not the library.
Not sure I get you. You don't catch any exception here. Is it maybe thrown from the call to `getEntitiesForRevisions(…)` call following? If so, I'd vote for a simple `return revisionNumbers.isEmpty() ? new Revisions(Collections.emptyList()) : // getEntitiesForRevisions…`.
I will add an assertion while merging
Don't ask what... 

On 29. 09. 12, at 12:43, Mitar <notifications@github.com> wrote:

> In tests/test_project/test_app/static/test_app/js/datastream.js:
> 
> > +
> > +    $.fn.datastream = function (options) {
> > +        return this.each(function () {
> > +            (new Datastream($(this), options));
> > +        });
> > +    };
> > +
> > +    $.datastream.metricList = function (callback) {
> > +        $.getJSON($.datastream.defaults.url, function (data) {
> > +                callback(data.objects);
> > +            }
> > +        );
> > +    };
> > +
> > +    $.datastream.metricName = function (metric) {
> > +        var name_tag = $(metric).attr("tags").filter(function (o) { return o.name; });
> filter method is available only in JavaScript 1.6+. Use jQuery .filter.
> 
> Hm. I am not sure if this code works. What exactly are you doing here? attr("tags") returns a value. It is not a list/array.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
IDE specific files should not be ignored by the project (otherwise it should support any existing IDE which would clutter the file). You should ignore it locally as described in http://help.github.com/ignore-files/ (and you should probably use the gobal method to avoid doing it for each repo)
I understand that, but for these types of  cases, just use the eventSource property. you don't need to poll it.
We can also do `@".\..\..\SampleShapefiles\ne_110m_admin_0_countries.shp"`.  I think that is the .NET method of choice.
NGHUTPR

```
@test(groups=[tests.INSTANCES, GROUP], 
           depends_on_groups=["dbaas.listing"])
```
Same issue as the other pull request, we will need an isset here!
This change should be unnecessary - rendering the `changset` partial from a changeset view should default to the one in the same view.
That's still an open question. That's one of the reasons I marked the feature as experimental.
Well, I moved it to shared since it's being used by two different controllers. I wasn't sure if it was idiomatic to have the SearchController reach into the Articles views and use them, but I'm happy to move it back to its original location.
Are you sure () are legal here?
Fail.
Если я захочу скрыть пост через меню кнопки скрытия "Схожие изображения", "Скрыть выделенное", то ничего не произойдет - spellsList не инициализирован.
You can set it up to return a Collection just as much pr Test method ?
Maybe worth opening an issue over. I do not believe this is the only place we mock XHR requests like this.
Else clause put in.
Why were the changes to this file made?
and this is a bit confusing as you are overriding key
*is* found