labofsahil/github-event-dataset-2012 huggingface dataset:
I believe this needs to be iisnode.+, Ogail, can you quickly verify?
This method is a little large -- do you think it would be better to break each property it's own method?
No, when the user is at the dashboard (root activity), hitting back will exit the application. When the user is at the tabgroup (which is opened over the dashboard), hitting back will close tabgroup, and we're now at the dashboard (this is correct behavior). However, when the user is at tabgroup, tabgroup.close() will close both the tabgroup and dashboard. That, I don't think is correct. Closing the tabgroup should NOT close the dashboard.
this needs re-indenting
you are right. gradientLayer should be fixed too
Thanks
Probably "Adds one or more users" would be better.
The problem is that it causes confusion since it isn't technically accurate. As previously mentioned, there are ways of checking if a fd is valid. They may be a bit hackish, but they do exist. You need to either remove the comment it, or update it to reflect that you felt fstatSync wasn't what you wanted because it felt hackish (I'm fine with either).
Might want to switch to a plus.
these are tests, but be careful about using explciit 3.7.100 references when 3.7.0 is sufficient.
hrm - the `require`s are first here, it was originally two `var` blocks
if key is only one word then add to "General" Category
On the downside, the whole naming scheme was built around the idea that `isSafe` would be `true` for `@trusted`: `isSafe`, `isUnsafe`, `areAllSafe`, and for the most part, you only care whether something is `@system` or not... So, I don't know. Having `isX` and `areAllX`  for all of them seems a bit much. Maybe I can up with a good templatization scheme using enums or something. I'll have to think about this.
I think this can be expressed without asFlatStaticArray by simply recursing for multidimensional static arrays. That would obviate the need for asFlatStaticArray and probably would make this marginally simpler.
Add a string. Now is the time.
Don't do this. Call `CommandProcessor.getProcessor()` directly — otherwise you could violate singletonness.
Check for null cursor.
I tried something like that, but couldn't get the code to accept the variant (clearly incorrect) that I came up with...  That's going away now anyway.
FYI these methods were auto-generated by Eclispe, so they should be pretty sound logic.
Should we change the name of this function then to get_token_or_user()?
We could also have 3 functions: get_token(), a get_user() and get_token_or_user().
The indentation format of the line you added is inconsistent with the rest of the file. Please use two tabs to indent the line.
nit: I was under the impression the standard for this kind of comments would be something like "The request's HTTP method."
Well, we don't know what the renderer will do in its callback.
Э, ну, мы так не кастуем же :(
Feels slightly too strict and I don't see a benefit: If someone puts spaces there, it looks ugly, but it doesn't hurt the readability.
Are we always going to have to put this in templates? Can `butter` just include it automagically somehow? Perhaps we can `require` it?
there is already a ``NODE_BIN`` path in the config. No need for a new one
It gets a little mentally confusing picturing the final output here, with the lines joined by newlines, and then also some newlines included in the actual text. I'd replace the latter newlines with empty strings in the `output` list, if the goal is a blank line in the output; gives a clearer picture of the output directly in the `output` list then.
created a separate admin page for the RSS display profile, to fill out some basic values to populate the RSS feed. It doesn't include all possible RSS values, but just a couple that are required (title) or others that are optional. For some RSS values not included in the form, a default is provided. For example the link element, will always be the object url (fedora/repository/pid:123) , etc..
date filter block styling
There is probably a test missing
I was waiting for this comment, so obviously I did it on purpose. We can go with mutually-exclusive attribute, but I think  (as we talked about it several times before) If we had SpEL available to us from the beginning we would be limiting our dependency on mutually-exclusive attributes especially for the cases like this. So personally I would like to keep it as is and see how it goes and possibly introducing it in 3.0 for other elements that currently have the distinction between SpEL vs literal values. Makes XML simpler
I made PR doctrine/DoctrineORMModule#150 for doctrine/DoctrineORMModule#106 before implementing it here and doctrine/DoctrineORMModule#151 for doctrine/DoctrineORMModule#112.
You couldn't help yourself could you?! ;)

No problem, I need to start learning to use that lib anyways :D
Please remove default value from
https://github.com/anandu/rightscale_cookbooks/blob/pgfixes_rebased/cookbooks/app_tomcat/metadata.rb#L90
so, why :
$this->getResponse()->getHeaders()->addHeaderLine('ContentType', "image/png") is Invalid header line ? 
correct me if i'm wrong :)

thanks in advance.
im not sure about this whole class
this is very nitpicky, but you're mixing single quotes with double quotes
Why not just always store strings in the ImageMap?  Then you don't have to re-encode the image every time you see it, and the image map can store type string instead of object.
never mind. i see whats going on.
此行和122行合并为一个
Not sure if check_permissions is a good name.
Now that this method is public we should consider a more meaningful name for it.
verify_permissions_for_user maybe?
If the MojamComponent has focus , set paused2 to false , thus disable the activation of the pause menu .
"for cascading validation" would be better IMO.
no medley relies very heavily on includes. and ocaisionally on partial rendering you have ruled both of those out.
Hardy, NOOP in this context means to skip any indexing work, which might be different than what the engine usually does.
_UNCHANGED_ means "do whatever the indexing engine would normally do".
I don't particularly like complex ? : expressions.  I would probably have added a member item.hasTag() or split the condition onto a separate line.
The coordinates are in UIWindow coordinate system.
On iOS it's always only one UIWindow in view hierarchy. It is different from Titanium.UI.Window which is just a top-level UIView.
Briefly, the primary screen and every dialog has a separate UIWindow. The primary UIWindow size is set to screen bounds.
So you could just say the coordinates are screen coordinates.
Not in pixels but in system units. This is pixels in Android and DPI on IOS. Also on IOS this is dependent on `screen orientation`.  So on an iPhone application in portrait mode this is 320 but in landscape mode is 480. 
Screen orientation is not the same thing as phone orientation. So a window defined as 

var win = Ti.UI.createWindow({
	backgroundColor:"white",
	orientationModes:[Ti.UI.PORTRAIT]
});

will always return 320 on iPhone even if the device itself is held in landscape.
Should be ..
                  CFBundleShortVersionString = app_version_[0]+'.'+app_version_[1]+'.'+app_version_[2]
meanie!
I c/p from :newissue, wasn't sure why this was needed there either, but have no way of testing this currently.
You made 2 indentations here... it should be only one in array indexes definitions.
>if(post.isOp) post = getThread(post);
> forAll(function(post) { if(post.isOp) post = getThread(post);

Зойчем джва раза?
spaces around `=` sign: http://www.python.org/dev/peps/pep-0008/#whitespace-in-expressions-and-statements
Pole powinno mieć nazwę w liczbie mnogiej.