labofsahil/github-event-dataset-2012 huggingface dataset:
No need to strdup option strings. Just use the pointer, the data it points to isn't going away.
Здесь должно быть System.exit(1)
Так писать плохо, т.к. ты теряешь исключение, из-за которого всё это произошло.
Для этого есть метод Character.isWhitespace
can we have the tests at the beginning of the file ?
and the helper methods at the end
Sorry, I pressed the 'comment' button too fast on the above comment. If you are reading this through your mail, please do look at the webpage, to have the full comment.

To add to the above comment, a interesting aspect of introducing the 'iter_set' variable, is that it opens the door to choosing coordinates to iterate upon in a more clever way than a vanilla cyclic coordinate descent. The recent literature strongly suggest that things like random choice of order, or grabbing the coordinates that violates most the KKT conditions, gives much faster convergence. @mblondel might have more to add here, as I think that he looked a bit at similar problems.
The default affinity doesn't match the docstring.
What does this do? IPython is not a dependency...
We don't use spaces in between parentheses like that it should be, <code>@Command(aliases = "kick" ...)</code>
Same goes for the rest of the pull
Agree about the size, but other than putting the code for saving the data into a gpx file, I do not see any obvious restructuring.
Awwww hell yeah
Deserialzier -> Deserializer
odd formatting, tab?
Avoid gratuitous whitespace changes.
I probably did this because it was the quickest fix. The line is deleted in my next ticket anyway.
I'd recommend a bare `.split()` here, which splits on any whitespace, instead of specifically making it a single space. If someone accidentally introduces an extra space into one of those `locales=` attributes, `split(" ")` would give you an empty locale string in the returned list, whereas `split()` would not.
Tämän muuttujan ideana pitää tallessa MAIN_MemXXXXX funktiolle annetun muistipalan osoite, se kun saattaa muuttua kun muistipalan kokoa muutetaan.
serviceOptions, or storageAccountOptions?
Just to make sure - is it ok to call `rufus_scheduler.send` without asserting `is_master?` first?
Ohh right. We actually do require it when you're specifying it as a radius - but when distance comes back it defaults to km, which won't be an issue. I retract my statement, you should make them specify it
boolean exists = false; 
for(int j=1;j <= 12;j++){
    if(data.equals(map.get(j))){
          exists = true; 
          break; 
    }
} 
if(exists){
    return data; 
}
Was missing a closing div
Done in c6fb54d.
Indent this and the following variable declarations
Iterator.remove does not return anything, so we can remove the else {...} block
Maybe it wont be executed even if there is an exception on some of the methods above.
What do you mean by "this"? The squaring operation? Could you explicitly state it in the comment.
@greater_than.nil? || disallows_value_of(@greater_than, @greater_than_message)
Add occlusion culling.
This doesn't have to be in the scope of this pull request, but I submitted #265.
Put back `Context.prototype.createTextureAtlas` and document that.
Shouldn't this be
```JavaScript
var data = D() || {};
```
?
It's better not merging them because there is "else" block after that.
We should mention the constant to use. IE, Ti.UI. Titanium.UI.AUTOLINK_MAP_ADDRESSES in this case.
these test are always bundled
Some Comment.
Can you just delete this? At the very least, get rid of the curlies and just write MouseKeyboardCameraController();
Shouldn't we bail at this point?
instead of grepping the Gemfile.lock you can use `rake_task_defined?` method. Then it doesn't depend on the source of the rake task.
Would exactly do you mean by this? Is that some kind of special D magic :)
I think that it's ugly (if not outright evil) that this accepts `char`. It's _not_ a string. But the original did, so I don't know... I'd still be tempted to disallow it though. People can cast if they really want to pass in a `char[]`.

Regardless, I would point out that if you're going to rewrite it like this, why not just make it take a range?
If it’s not obvious from the context *why* you’re testing it, yes. In this case, I would probably suggest it’s obvious enough from the test name.
This is where that separation comes in.

If we ask Android to not call us for 24 hours, and then we pair a new device, we won't have the opportunity to sync again when we should.

So there needs to be a `Math.min` in here with some "try again" interval (still padded, of course).

5 minutes seems about right for now — that is, the smallest interval at which we do want to sync.

The exception to this is a backoff/retry-after request from the server. We absolutely do want to honor that. This might require us to split `getEarliestNextSync` and `getEarliestNextPerformSyncCall` — the former is `seconds` here, and the latter is the smallest of 5 minutes and the requested backoff, unless the backoff is server-requested.

I'm open to how you want to actually effect this, but this `delayUntil` phrase is the tip of the spear.
I am not particularly keen on this many retries.

If we hit the same exception on the second retry, there's no point hammering on our error logs. I'd be inclined to try exactly once more, unless we see real evidence that suggests that we need more.
i realise this was here before but it's making me squint really hard. 

can dataPathRoot() return a pointer rather than a reference, then we dereference it? still weird but not quite as weird.
More of a RFE -- I wonder if a method like ``boolean gateway.deleteIfPossible(ctx, obj)`` would be sensible. or ``deleteQuietly``.
So `phpBB-Blog/phpBB-Blog-for-3.1`? I couldn't find any documentation on this file so kinda guessed it.
final Logger logger
This method already returns a list--there's no need to copy its contents into a new arraylist
assertions here and next test
Ah, it has a testsuite folder, so it should need a easyblock that can do this?
According to Kenneth someone was working on this?
Yeah, but I'm inclined not to bother.  There's no reason to sort if you are querying the entire cloud, and that's the case that we are able to fold.  So I think it's probably fine as it is.
Здесь не хватает теста: если не происходит смены родителя, то колбэки не вызываются.
Should probably add a test to exercise the toggling nature of the  map on line 35 - i.e. when mykey is ["foo", "!bar"] then !@mykey applies ["!foo", "bar"]
I agree =)
I think that the switching is needed, because I want that your interceptors should be loaded instead of mine

btw, it is better to change the configuration or throw a ConfigurationException?
If suites should be optional, then this should be wrapped in `if "suites" in case_data:`. Does that answer your question from IRC?
It's explained in the reference - this is a user-hooks class so they can modify the sockets after creation - e.g. to apply non-standard cypher suites. The default implementation does nothing, but the strategy provides an extension point.
So, if we'll allow to use `request-handler-advice-chain` for gateways within the `<chain>` I'm going to simplify this parser error message and I'll prohibit to use `request-handler-advice-chain` for outbound-adapters within the  `<chain>` at all.
We've been limiting the line length of the READMEs to 80 characters.
you need to check against the interface
As I commented in https://logstash.jira.com/browse/LOGSTASH-436 , I'm only interested in the "invalid byte sequence in UTF-8" exception right now. And my ruby-foo is quite weak, I don't know how to filter different types of exceptions yet. I'll give it a try tomorrow.

But while we are at it, should exceptions in Regexp crash grok, and consequently logstash? Should't all exceptions be rescued and let grok fail tagging the event with _grokparsefailed?
I'm not sure about for the background gradient, but in my app I have just a standard CATextLayer with a semi-transparent black background. It works for that; I've NSLog'd the layer sizes to be sure. As I recall, leaving this line in would basically mean that the text layer would have a height of 1, or 0.
Minor note: I don't love the use of a false here to indicate that nothing was found. This could be referred to as "primitive obsession". It gets the job done, but a different class like NilMovie might be "better"
@apex13 Can we use ActiveRecord here, we shouldn't be writing SQL at all.

The page I usually use for this is http://guides.rubyonrails.org/active_record_querying.html
object or array? I'd prefer an array (easier to parse later on response)
In this release its using a localhost wakeup server listening on the port 8090. I leave this unconfigurable since this will depend on the OB detection system we shall define and implement ASAP.
Does Id need to be settable?
Yeah, I gotcha. I think for my specific use-case, i'll end up making my fork have a `API::V1::UsersController#create` method at some point that i won't send a pull request for. However, in this case, since it is just on the provider, having it in the `UsersController` would be more along the lines of the rest of the project
Doesn't ```user, host``` require to be a tuple:

```% (user, host)```
Right, I get what you're saying, I just think it's a confusing convention. You don't know what you're looking at when you see `TestCase` in the code unless you verify the import. No such confusion occurs with unique names. I'm still unsure what this convention that has "already been decided" gains us.

I don't understand why we would want to make it harder to know what's going on, which this decidedly does.
I know it's very hard to get into pychan, so this is really something that happens to many people.
The thing is, pychan provides properties for most things where somethign like _setItems() or _getItems() exists, it's usually called "items" then. It's listed as class variable[1] and should also show up on dir() or when you read the code of it of course (those 3 methods are usually used for pychan).

So usually, it's not necessary to call methods starting with "_" in pychan, although there are real exceptions.

[1]: http://fifedocs.boxbox.org/epydoc/
Can't `session_check` be `None` here? You explicitly check for this in the `if` above.
Same here - it should really be the user's choice how to store this data.
In `removeAll` I wanted to reuse the `remove` function. Therefore `remove` shouldn't always pop the required menuitem from the Array,
because `removeAll`'s loop would mess up.
Therefore I don't care about `Array.slice()`'s new value. Of  course the `Array.filter()` was overthinked. :)
Added a new class for corpus link view that doesnt have any of its children in it. it is much lighter, and is only used to provide a link to the users dashboard whic hthen loads the corpus. no other info is needed in the link view...
This feels a little backward. I would not have that file in the repo. Then I would make a set request, verify that it's in the filesystem and then remove it at the end of the test. So you would not need to keep 1234.json in the file system.
Maybe you should also test the score?
you should preallocate coef and residues and avoid using a list here
By just checking the `best_score_` attribute of `cv` is defined, by checking its positive or something else?
What is this condition supposed to be testing? It shouldn't be possible to have a user with no ID so I don't understand what it is trying to do?

Equally I don't see why you would want to display a different message if the user is logged in - did you actually mean to test if the user is looking at their own changeset list? That would need a test like `@user and @user.display_name == params[:display_name]` or something.
Shouldn't happen, since we are transforming them one by one. So, if we are transforming lib/bootstrap.less, then dotless imports all files it needs (typically in the same folder or in subfolders, but not above I guess), then we handle the next file, maybe above, but we don't care, since we are done with the first one. Since we transform one by one, the second argument for the Transform method has only one item. Does that answer your question?
ditto re distinction between the callback itself and the param
What about MODE_WRITE?

Also, this is definitely used on Android. See:

titanium_mobile/android/titanium/src/java/org/appcelerator/titanium/io/TiFile.java:L270
Duplicates the `@return` below
@tomgross what is the difference in your patch with the first one ?
You are right. I check at runtime because the Test doesn't use the DependencyInjection.
The code deals with ppsUtils of two following methods can be refactored and call some getPPSField(ppsPath) that returns field's value.
@jkeshavarzi Is there an existing test case I can look at which does this?  backgroundColor would have been a good one to go off of because it's similar, but I couldn't find a unit test for it in test/unit/lib/webview.js.
http://lovelysystems.github.com/robotframework-androidlibrary/AndroidLibrary.html
- remove old default tab checkbox
C&P :)
is -> are