labofsahil/github-event-dataset-2012 huggingface dataset:
Would it be a good idea to have a check in clone that returns without doing anything if this === result?  That way if you wanted to set a column in a mutable way quickly, then m.setColumn(0, newColumn, m) would do the least amount of copying.
"reverse" should also be a constant
Unrequired imports
Can't access this--when are these docs going to be online?
On Android, this is a valid unit for sizing a view. See:

http://developer.android.com/guide/topics/resources/more-resources.html#Dimension

I will clarify in the doc.
could use [NSArray arrayWithObjects:]
Can you add a link to the example you are porting (is this a port?)? See the other samples in the project.
Oops -- got it.
@rs2487 please do not delete theme path output. is needed in some projects
@robert-chiniquy It doesn't like making functions inside a loop.

I'm fine with using this approach which lint wouldn't complain about although in cases like this I prefer to create a function.
я про аргументы к функции
why do we need mercurial at all?
smoking gun: the introduced symbol is a value (etc)
Well, do we want to remove the check from the code, or update the documentation...?
Besides, the doc says "up to" N calls to pred. Not sure about this one.
What I'm saying here is that the proposed design is arcane enough to warrant exploring alternatives.
This was "redundant control flow" (diff is unfortunately outdated for the first occurence). What exactly  do you mean by this? The else can't be eliminated because we get statement unreachable warning otherwise. Unfortunately the two checks which are mostly copy-paste can't be merged into one private function because we'll produce different return types then.
I cannot do that atm, because the xsd in the api jar is not updated yet
Source formatting note: Please don't use this pattern. We have ATTRS for this type of stuff (for instance, you are using a getter to access certain properties, but that's the point of .get()
Could you explain what you were doing when this was a problem?
need to return a result for every member of the input cohort
For some reason I get scared when I see this, but it's just the same to use while (true) ... keep it it's a test ;-)
does this mean that if the test is not named as one of the recipes in the list the test doesn't run? I don't really like this kind of restriction.
pep8 wait for two lines here
return @_num_pages || 1
connect (subtitleMenuAction,
					SIGNAL (triggered (bool)),
					wrapper,
					SLOT (setSubtitle ()));
Setting `$match` should be doable using functions instead of loops. But it's not -- we need better list and string functions. >_<
I would add a bit more detail to the docstring here about what exactly this method expects in its arguments, and what it returns.
We can add it, but the main reason why I didn't want to add it in the first place is to preserve existing single-threaded semantics
But i guess exposing it is ok as well. In fact we can consider it to be an improvement since current IMAP-IDLE had no way of producing SI Messages concurrently (if let's say you wanted to)
The default __mask__ is 0xFF
Well, true and that's how it works so perhaps I need to add to an IF (correlationKey exists) statement to eliminate the confusion. Although again if consumer caching is not enabled then this Queue will be destroyed right away, but yes no point of creating it if we already know that its not going to be used, so I'll polish this.
I like to be explicit if I may so including 'message' in the names if that's what I am receiving is OK with me, but I do agree on 'receiveCorrelatedReplyMessage' so will change.
s/~~over~~/to/
you cannot suppose that the action label will always be at the end of the sentence. This must be done with a placeholder in the xliff file
This is the mailchimp gem, so we should avoid including debug code for another project in here.  There is not currently a pattern in the gem for debugging output. If you want to add one, it would need documentation and should follow the Logger pattern instead of using puts.
can you fix this and the next line to "average_..."
Should I add a function, WriteCartographicDegreesValue, to PositionCesiumWriter or is there currently a better option?
again, avoid using future tense
Oh man... if NASA ever sues us how will I say I came up with this in a clean room?
Linked to:
https://github.com/mitar/django-tastypie-mongoengine/blob/master/tests/test_project/test_app/tests/test_basic.py#L725-733

Allows creation of non-existing `Person` objects.
I had added it before adding creation of a `Person` on lines 725-729.

Since it is not necessary and does not cover the test purpose, it could be removed if non-existing `Person` are removed from the test.
Don't include max_length - custom user models may not have this restriction
If a Label text is filled later, just use `text=""` so that the string (that will probably never show up in-game) does not end up in our translation templates
This import must be aliased OR replace the uses of `Rest\` in the code to match with `ZendRest`
Please add comments in javadoc-style explaining what are the arguments to this function.
Так же табуляция и UTF-8
Don't we want extant users to be able to register, though?

I think this prevents that from happening...
If they have a cell number, then their cell number field won't be blank.
add // startSession is a required method (even if nothing happens here)
Is this the standart we're using for documenting functions which take in an options object?
How is this different from ERROR_LOCATION_UNKNOWN?
Should be removed from public docs
With it undefined, we can preserve the existing behavior and not set it. If a developer knows what they are doing and want to set withCredentials to something, then we provide that mechanism. That is why I choose the implementation in this PR.
So what is the decision here - I don't think root_home should represent anything but the original intention - aka. the root users home directory.
Gotta love `TypeTuple` and `foreach`! Assuming that it makes sense, it would probably be a good idea to test with varying levels of constness and immutability rather than just on character type.
Eventually, yes, but not in this checkin. I cannot expose .Net classes in WinMD libraries, so I need to come out with the right abstraction for the pipeline and filters.
Если у родительского узла нет потомков, то наш узел становится первым потомком.
I wouldn't say I have all time trust in the check() method after the last fiasco. If it's a good enough method for JQuery.isEmpty, it's good enough for me.
I am thinking if it's better to have the features definition at the top of the file (like how you had it before for enable-flash), so that it's easy to find out what features we support.
Seems a bit much for this use case, since we shouldn't ever inherit properties. but other files may use this function so it makes sense to do it this way.
@kwallis - do we have decision on the exact string?
Few things I suggest we change are-
1. it should say webworks-xxx.js - basically something to suggest the version number.
2. Then also tell them where to get the webworks-xxx.js file i.e Framework/clientFiles/webworks-xxx.js
Is it only for "type collection"? What does it do exactly? Cause fields with type "collection" works fine. And I checked "prototype" value and there is string with field's html. So I think it's correct, isn't it?
<pedantic>
some whitespace would be nice, e.g.

    if (row.title ...) {

IT'S WHAT CROCKFORD WOULD WANT! :)
</pedantic>
(<= 13 age 19) works too :)
Probably worth mentioning that, in general, anything that performs mutation should not be a "getter property" with the exception being lazy loading.

I hate .reverse and .sort.
I really don't like this syntax.  Is there a technical reason for doing this versus `obj.related.lead_art`?