labofsahil/github-event-dataset-2012 huggingface dataset:
Так же стараемся не делать вызовы JavaScript в HTML коде.

```javascript
// Можно эту функцию вызвать внутри IIFE тогда и initialise не нужно будет прокидывать в глобалы.
document.body.addeventListener('load', initialise, false);

// или просто
setTimout(initialise, 0);
```
Why is this interface empty?
Maybe this file should just be tba_variables? We can have the Bootstrap vars and the TBA vars?
Tässä tarvitse funktioita `filter` ja `first`.
`(not (empty? ...))` would be a better test.
Check OS?
Yes, I used slice thickness interchangeably with out-of-plane voxel dimension.  Apologies for the confusion.  I can expound upon this in the Notes.  Perhaps it should be the last optional argument?  The people that need this know what it is; pretty much everyone else should leave it alone!
Should this be wrapped in an `else`? It will execute regardless of the above condition `!Popcorn.youtube`
Why provide the optional `width` and `height` arguments?  No one is using them.

If we keep them, we should use `defaultValue` below.

I suspect this code is pretty old.
Picky mode, once again.  This tests only makes sure `negativeZ` throws an error and doesn't confirm that any of the other properties cause an error.  In reality, this same test should be repeated for each side of the cube.
Throughout Cesium, do we really need this check?  Seems like a useless optimization.  @emackey is there something about setting the canvas' width and height?
`Core/FAR` is a hack because I don't know how to define infinity in GLSL.  Even if the value is the same here, the semantics are different, and perhaps `FAR` is a bad name.
This should be two different functions, one that takes a source document and one that just takes czml packets.  I'm not sure we really need the later one yet, so I'm fine with also just removing the czml parameter.  I also like the name loadCzml better than add, but that could be just me.  It's all going to change eventually anyway.
Extract out all node specific artifacts
Hmmm, it's definitely pointing at the appcmd.exe in the iis express directory.  errorlevel is just undefined for successful install, not sure why the discrepancy.

BTW - do you think it's useful to add an uninstall step as well?
default:INPUT_BUTTONMODE_NEVER
Don't  we  need the value in the JSON (I'm probably missing something here due to just looking at diffs)?
Ah ok my bad.
What is this meant for? Is this meant for getting the results of the dependency resolution?

There is missing a function to specify which packages should be installed by the backend.
IMHO it is a bad idea to inherit from ruby's basic types like this. You expose a large interface that doesn't always make sense for the instatiator to use. Besides that, it makes more sense in my mind to think of things like this as "a wrapper around a collection of items" rather than "a collection of items".
Yes, this is just incorrect and unittester passes only because there is no unittests for `unsigned`.  @wfunction's mistake here illustrates that this change just messes things. Shouldn't be merged!
You should probably add tests that verify that `canSwapEndianness` and `isFloatOrDouble` give the correct results.
Sorry for the confusion. Actual code is following:

```d
//haystack.popFront();
haystack = haystack.dropOne();
if (startsWith!pred(haystack.save, needle.save.dropOne()))
    return result;
```

Yes, I use countUntil unittests and std.datetime.benchmark.
But, with hindsight popFrond looks better...not need assignment.
Here, this comment seems redundant (to me), since apart from 1 (already documented) overload, the input *does* have to be a forward range. So I don't see what this note is trying to highlight to the reader.

Nitpick: `acepts` => `accepts`
Pete, just to be clear, are you talking about an injectable ResourceBundle and creating a ResourceBundleProducer class?
Changed this to an elseif at a lower point
Why do you need these two lines?

"sync.prefs.global" indicates a *file*. This appears to be fetching the prefs file that's named "sync.prefs" and removing from it the entry "global", which is entirely different to the entries in the "sync.prefs.global" file.
Please explain life-cycle of this activity and how it interacts with regular syncs.  Writes a pref that sync reads?  Sync then clears pref?
While we're here, could we add a log line to remark that we're returning nothing?
If there is an error in my logic, it is possible that we enqueue records that never get inserted (written to the db).  This will at least warn.
While we're touching this, can we make `utf8` `final`?
y not wrapping this into a method with a useful name?
Yes, I agree -- it should be OF_SCALEMODE_FIT, etc.  I'll fix it.
I'm not opposed to the direction here, but this breaks the tests. I'd prefer if getUserMedia remained a function (no-op), but had an additional expando property `isNop` (or similar) that indicated the function would do nothing.
was a tird, flushed!
The changes in the -moveTick: method should be innocuous as they seem to be just formatting changes, so nothing impacting code or changing any functionality.
I'm not so sure about that. 

How would you differentiate the case of forgetting to add a ``@return`` comment vs. not returning anything? Note that the system only adds these on interfaces as it has no code to analyze there. You won't see it being added on methods that have an implementation.
What will happen if `string` or `doc.FirstChild.NodeName` is is null? Does the code will just work fine?
Yup, was just being lazy. Fixed now, in this and the Queue and Table versions too.
drop `.`
I see now that you clarify that detail in the docs. :+1: 

LGTM
Someone please confirm that clicking "Remove class" will follow the hyperlink redirect and remove the class from the schedule.
$grade muss nullable sein... für AUTO-Bewertungen

Zudem bedeutet $grade == null gerade, dass es eine Auto-Bewertung ist....
Since it's got ASP tags in it, it's better to just delete than to comment out.
Expect for names, this is what is done. NullPointerError inherit from SignalError . InvalidPointerError is better, I'll rename that.
How is this Generic OBR-ENC related to the PID for this ticket?  Where is the liquibase changeset for the "Generic PID" template?
What's the intention behind this  change? I have quite a few objections here:

# What's the problem with returning an empty `Revisions` object it. It actually nicely expresses the outcome of the invoked method: no revisions found. # If it makes sense, shouldn't the envers-specific exception only be thrown from Envers itself?
# We must not throw a Envers specific exception out of our implementation as this would result in the clients need to know about the actual revisioning infrastructure. If we really decide to go the exception way we either need to find a suitable `DataAccessException` or introduce a new one.
# The exception text should contain use case specific assumptions. The suggestion will be wrong most of the times so there's no real point in having it.
If caching is **off** (`jdbcCallOperationsCacheSize = 0`) then a **JdbcCallOperations** instance will be created for each Stored Procedure execution (which will as far as I have seen it cost you 100+ms). 

However, if you have just a single Stored Procedure to execute, the `jdbcCallOperationsCache` will by default only have a size of `1`. I think with the current implementation, I am able to re-use the same code for both *dynamic Stored Procedure use-cases* and for the use-case of *just having one Stored Procedure*.
Commit 06f782b provides better JavaDoc.
Right, that was late refactoring as part of merging the 3 methods into one. Fixed.
Discussed with Oleg by 'phone - Wiki will be enhanced. Merging.
`  - Also provides an Auth Adapter.`
Why bump the version number? Perhaps you need to pull from master?
You can check [here](https://bitbucket.org/mitar/jquery-visage) how to define jQuery plugin.
@christianv Can you add the word "page" after the development setup link?
I thought AsyncErrorHandler read better than AsyncErrorHandling
AsyncErrorHandling sound like a tool for handing any errors in an async manner
AsyncErrorHandler sounds more like something to handle async errors.

A small distinction I know.
Happy to be dissuaded.
Make sure you have considered every possible scenario here.  Is it possible for a person not to have a name, and what is the expected output of this method if that is the case?  Is there a possibility that someone has only one or two names?  Is the output verified in a test?  Use an @should for each scenario.
For the last test you could just pass nothing into the `testTheme()` function and then have something like the below in your `testTheme` function. That way all the tests can use the new function.

`if (theme){
    data['feature'] = { '@': { id: 'blackberry.app' },
 	 param: { '@': { name: 'theme', value: theme } } };
}`
Originally I struggled with this a bit. My first desire was to name the module assert, but since its a reserved keyword that wouldn't fly. 

After that comes the next problem which is that every other name for the module isn't quite good enough. I personally don't like "asserts.is_none"- it reminds me a bit of "assert_equals" with the "s", which was deprecated in the unittest module.

The problem is if I rename all the "assert_star" functions to remove the "assert_" prefix, then if people want to use the function by itself you'd just see "is_none" "is_equal" etc in the middle of everything which seems kind of ugly too. Maybe I could dynamically make a new module that would behave this way if anyone wants to see it. If you really want that, you could raise an issue on Proboscis. It wouldn't be hard at all, I'd just need to have a good name for it (I wouldn't want to put the other methods in the asserts module).

Finally I did this to emulate the nose test functions, since the goal of having asserts was to be able to run Proboscis in Jython or IronPython which lack Nose. 

In other Python cultures it is not a huge sin to do star imports. I'm reading a book on Tkinter where they do it all the time. I really dislike * imports in most cases but I think import all the asserts would make a lot of sense. Of course, then we'd be going against the Nova coding guidelines we're trying to emulate. For now I'm content to just import everything.
I wanted to make the style-guide changes first, as it is touching all manifests and later submit some more pull requests.
Do we need to do this?
Josh pointed me that the instance variable is the one which is receiving the copy; but since we're dealing with a copy property, we can use the property accessor for that: [self seCompletion:completion];
This should be specific to the actionbar. Meaning the selector should be more specific and should be moved to the action bar css file.
same as get)questionnaire_obj (line 97)
Yay, much better :D
это не привычно, обычно pop возвращает void
The issue is that we're getting the text size of JUST the text. TextArea adds a margin around it. The 10 is that margin.
I'm not sure that this is entirely accurate. AsyncResult is a semaphore that blocks the current thread when getResult() is called until another thread calls setResult(). TiMessenger uses it, but so do other places in our code (and it could be used by any multi-threaded code that wants to block on an asynchronous result)
With the exact same code in dealloc and lockTogether, this should be made into a single method called from both places (e.g. releaseScrollViews or something). Use RELEASE_TO_NIL instead of '[scrollViews release]' when made into a helper method.
Early return for `!shouldUpload()`.
don't call it a "great" limitation.
Also link to the issue
I think this needs the full header?
I was under the impression all local paths used three slashes, is this not the case? This is the same function that exists in the webplatform utils, I'm expecting the behaviour to be the same.