labofsahil/github-event-dataset-2012 huggingface dataset:
У тебя тут произойдет следющее: функция  `initialise()` будет вызвана, а результат ее выполнения будет передан в функцию `.on()` те фактически:

```javascript
var result = initialise();
$(document.body).on('load', result); // смысла нет
```

Скорее всего это просто невнимательность :)

Вобще лучше использовать событие DOMContentLoaded вместо onload

```javascript
$(document).ready(initialise);
```

или сокращенно

```javascript
$(initialise);
```

Разница между DOMContentLoaded и onload в следующем: load срабатывает когда *все ресурсы страницы загрузились* DOMContentLoaded - когда загрузился только документ. На страницце может быть много картинок и в случае с onload она "оживет" только когда они все загрузяться.

http://stackoverflow.com/questions/3698200/window-onload-vs-document-ready
Where are these functions used?
It feels very weird to pass a field from 'self' to a method on 'self'. What's the reasoning behind doing this instead of having self._log_request use self.connection directly?
II would need to subclass after looking into this, I'd do something like the example here:

http://docs.python.org/library/collections.html#collections.namedtuple
You should make them enter a 0 for exiting.. this would make much more sense and stick with only having your set of possible entries be a subset of the integer numbers.
useless blank line here, below there are still declarations
same for the Yaml component
Also, if you're using a non-integer sales tax like 8.1%, your final answer might have to be rounded. 

$45 * 8.1% = $3.645

So I'd make a note to the user that they need to round to the nearest cent, or you can make the sales tax an integer.
Actually, I think having it perform arbitrary logic will confuse some users, and may break some code when compiled on a platform where real == double.

Surely D guarantees that real is the biggest float type, and therefore >= double.
There should be no case ever where real is < double?
If real == double, then there is no harm in is(Promote!double == real) ...?

Even if float == double == real, I still think this is the correct behaviour for portability reasons.
The only possible error case is if double > float && real < double, and that should surely be impossible?
Throws a spec error, the name isn't matching.  It looks like because path is an array, it's putting the square brackets in - this is quite possibly a ruby 1.8/1.9 difference, so you might want to explicitly use #{path.inspect} to force the same style.

       -:name => "Auth rule matching ^/catalog/([^/]+)$",
       +:name => "Auth rule matching [\"^/catalog/([^/]+)$\"]",
I will change this, as well as remove notions of Wicket (result of copy-paste)
You should define equals and hascode based on the entities natural id.
nit: early return here, de-indent later.
What is the purpose of pulling call_status out into it's own template assignment?
Na verdade, você deveria ter colocado o número do período no construtor.
t'as fucké le "bon" formattage jnsi ;)
this syntax works however?
it is the default for the type. but this is not what the RestActionReader receives. It receives a resource (this does not affect your example as the default value is not really used anyway: the RestControllerReader always pass it explicitly as an empty string for the new behavior).
add try ... catch block to all runners.
Удалить весь файл
I believe this should be an ``InvalidArgumentException``.
На то и анстейбл)
I prefer to leave personal preferences out of this. It's a solid standard not only on PHP, but other languages as well.
I've completelly changed the BannerInfo* classes, so just take my changes
I'm not sure I follow. The way I understand it,

    auto foo = cast(ubyte*)0x10;
    *foo = 1;

would result in a `NullPointerError` with your current code, no?
Will do

https://jira.springsource.org/browse/INT-2494
Look at the setReleaseStrategy method. It has the same code. Is that what you are asking?
please change the arg order as well if you agree with that
this design enforce the customer to pass in the root Uri so that it reduces their chance of connecting to a Azure server they do not understand.
First bullet should be

Discover endpoint address and port for other role instances in your service (or something like this)
Don't use hard-coded URLs.
You can also do

```
(topic for topic in tree.children if topic.id == "math").next()
```

which doesn't filter everything, only as many as necessary. Not sure which is nicer.
Sorry not sure how that slipped in!
+10.

Alternately, util.message("code") works

-Darius

On Thu, Sep 20, 2012 at 1:49 PM, Michael Seaton <notifications@github.com>wrote:

> In
> api/src/main/java/org/openmrs/module/reporting/report/renderer/TextTemplateRenderer.java:
>
> > @@ -108,6 +109,7 @@ public void render(ReportData reportData, String argument, OutputStream out) thr
> >  				bindings.put("reportDesign", reportDesign);
> >  				bindings.put("data", replacements);
> >  				bindings.put("util", new ObjectUtil());
> > +				bindings.put("i18n", new MessageUtil());
>
> I think i18n will be tedious to type. Maybe bind this to "msg"?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openmrs/openmrs-module-reporting/pull/4/files#r1655498>.
>
>
{{ product.description }}
We do this in other projects too. If omitting the slash works too then I'm fine either way.
You can get away without saving this encounter, and therefore you will have less restrictions on what has to be set.  Just set the values that are checked in the snapshot and you will be good.
Nevermind, I see how it's actually trying the other byte order, now.  Still, you can save yourself some trouble just by asking BitConverter what the platform endianness is.
You're right. I figured out the correct way to do this and have committed.  :)
I think the control-flow could be improved so that's easier to grasp what's going on here, my solution:

```python

# Display error when no quick saves were found
if not save_files:
  if session:
    session.gui.show_popup(_("No quicksaves found"), _("You need to quicksave before you can quickload."))
  else:
    print "Error: No quicksave found."
  return False

# close widgets that might be open
if session:
  session.ingame_gui.on_escape()
```

I wrote that down without thinking too much about it (just a feeling that it's nicer), but I'l try to argue for it. This function is primarily responsible for loading a game, and therefore needs to handle the case when there are no save games. Handling this case should be obvious, it has been implemented by an early return and should stay this way. Once the error is detected, it is decided how to display it - using either the session or a simple print. So that's why those `ifs` are swapped.

If you look at my code, you'll see both blocks are commented at the beginning - clearly saying what is done. I think you'll have problems doing the same in your version, because the first branch is doing two different things.

**tl;dr; I think my change makes the code easier to read.**
Use the utility function written in the LoginUtils class instead of rewriting the logic here
Great, I'll make those changes you suggested to the latest version.  I appreciate the feedback. 
Last night I removed all those relative imports and brushed up a few more things. I'll make a new pull request soon. 

Sent from my iPhone

On Nov 15, 2012, at 6:34 AM, Mark Rickerby <notifications@github.com> wrote:

> In bigcommerce/api/__init__.py:
> 
> > @@ -0,0 +1,32 @@
> > +import os
> > +import sys
> > +import base64
> > +import logging
> > +
> > +from bigcommerce.api.lib.connection import Connection
> > +from resources import ResourceAccessor
> > +
> > +log = logging.getLogger("bc_aaapi")
> > +
> > +
> > +class bigCommerce(object):
> As noted above, ApiClient (or something similar) would be a more appropriate name here.
> 
> —
> Reply to this email directly or view it on GitHub.
>
as long as you have UTILS=yes in your common.mk you can just use #include <webworks_utils.hpp> and it should work
Do you think adding the following would help?

```
>>> X_2[0, 7]
0
>>> X_2[3, 7]
1
```
ternary operator ?

locale.primary_locale ? "primary" : "secondary"
Bouger la derniere ligne dans un after create.
Is it safe to not set a version? Doesn't the build become un reproducible?
I'm not too sure how to give an overall comment on the commit, so I will list it here. I believe in comparison to v2.0, v2.1 has a larger number of inconsistencies and undesired behavior. Tag matching would be a nice feature, but at its current state, it does not seem to be worth it. It has yielded a lot of broken aspects of the plugin. I mentioned a lot of the gripes I had with the changes on individual lines, but here is a recap and possibly some new issues.

Interpolating argument attr in IC tags broke
Special if tag's broken [if-item-field]
Unrecognized end tags don't highlight
Perl/Calc/Comment totally needs a better colors. I appreciated the less intrusive nature of them.
tmp/seti don't interpolate
tmpn/set don't mark content as strings
You left in proprietary (foreach/fnet_button/template)...Missed 'loop', 'and', 'or', 'either' 
Why do you have different scopes for tmp,set tags?

I have a page with a bunch of use cases that I test with if you would like that so you can see what I have been anal about making sure works, vs what I have marked as less significant.
check case: ProtocolMetaData
I get a lot of interesting looking typos because sometimes my emacs has keyboard focus when I hit control-T to get a new tab in firefox :)
Makes sense. That would be a larger refactor. For now, let's just make the text of the error messages more informative.
It appears that this does not currently work on iOS (the setter is invalid). Because this is so trivial and it seems like we have some upcoming scrollview issues I will leave a comment on whatever ticket is created for that.
I am not sure if this code is needed. WebViews are never used to contain other views (children).
Can we just rip this out?
whose (I know, I'm getting ridiculous with the grammar stuff :) )
we should make this
Should not be a class. Instead this should be a protocol with `@required` accessor for toolbar. This will also likely be of limited use to module developers.
Make idempotent
By the last comment I'm guessing this was https://groups.google.com/forum/#!topic/scala-user/7B6fPJ03Vww/discussion ;-) Could you add this to the "source", cf. [puzzler #21](http://scalapuzzlers.com/#pzzlr-021)? Thanks!
the names are confusing. Why do I need to supply an "XLinkURL" to return an "XLinkUrl", but then get a string?
Please overthink the names a bit.
I think they should give hints on whether something is generated (generateXLinkUrl) or just read from some source (getXLinkUrl) or something else.
This needs to be properly fixed in socinfo.h in mach-msm, but i am lazy and sleepy now, i will push one more commit to fix this, sorry :(
add an index to the audio array,

(and make the audio arrays for the tdfm and tdfp by doing ls and find and replace in eclipse)
I am not a fan if storing constants in Interfaces. We should use an Enum instead. Also, it looks like the constants are using only by the XML Parser classes - should it be better to move the constants to the respective package - eg: 

* org.springframework.integration.aws.config.xml or
* org.springframework.integration.aws.config
Oh, that's what you meant. :)
we don't seem to be using bps, so we don't need to link against it
Removed this add_subdirectory line and changed it in the main CMakeLists.txt instead.
I just mean that if  ```` args.options.sensor === "" ```` then by definition ````!args.options.sensor```` must be true since ```` !"" === true````