labofsahil/github-event-dataset-2012 huggingface dataset:
is that search term always going to have 11 pages of results?
i suggest looking at https://github.com/mozilla/Addon-Tests/blob/master/tests/desktop/test_extensions.py#L28 for an example of testing pagination.
why not supporting array of strings properly ?
There's already a deprecation warning for ``p`` (below in the code)
Might be simpler to merge the ``hasattr`` check in:

```python
if getattr(X, 'dtype', None) != DTYPE or X.ndim != 2
```
But it's not only scaling the variance, it's also centering. `ZvalueScaler` or `StandardScaler` maybe?

http://en.wikipedia.org/wiki/Standard_score
Can you check the times when there are, say, 50 classes?  I think that would lead to a bigger difference.
See 19a73cb
My tests were broken without that fix. But I'm suprised that they do run for you on windows.
I'm not awake enough to find the place were something is using realpath or else (is it stream_resolve_include_path in "toAbsolutePath" in Configuration?) so that the forward slashes get converted to backslashes, and thats why these tests fail.

But on the other hand, whats wrong with DS?
this is useless. Please revert it
you can remove this, and add unicorn in
chrome.showSpinner() shows a spinner saying we are loading, and it is outside the if :)
I found that this put all the regions on the same line-- you have to add the newline character explicitly.
Tu peux mettre TODO ou FIXME comme ça, Eclipse le note et on peut y accéder plus facilement
typo attribute
dont you need a require login before filter here?
`future-fact` ?
I'd set ```nil``` to ```@#{remote_rel}``` in case, if somebody will use this variable.
The module required stored-configs. Seemed way overly complex for what I needed to accomplish.
Oh, it gets a +1 here... ok.
This leaks.  
Typically I would name this variable left.
Now I remember... This was not a lock problem in 3.8, well, before THIS patch, because previously I was calling ```addEntry``` which does not create the connections (thus, doesn't lock), and now I'm using ```attach```, which does. And that's how I found the lock, while testing this patch.

So, in 3.8.x, it's just a memory leak. With this patch, it was a lock leak.
For caches we do not pull specific branches; every cache is intended to have all available changesets.
Did you forget to add the text?
what about `xulify(window)`, and `basify(window)` instead ?
As 'log' is a unique ID, we should use `(By.ID, 'log')`
Deux helpers bien utiles. Je n'aime pas quand il y a des tas d'algorithmes dans les vues.
C'est beaucoup plus clean de chaque côté :)
Please look into this char to see if this is a localization issue or if this is something related to the IDE settings.
This needs to start out commented to be uncommented by the user
@shacker @yuhunglin just fyi I moved the widgets section up (didn't change anything to it) so we can override it later.
can you add the commands to create cache directory?
mkdir -p data/cache && chmod 0755 data/cache
I'd suggest calling the stop() method here, then the logic is in one place.
What does this do? Fill @issuesum with content? That doesn't sound much Railsy.
I think the indentation is wrong..
Again I lacked context due to the site being down.
I found 35 is short but 42 fits in (even with 42 letters "m") so I'll take another shot.
I added a comment for the order of the parameters to each A matrix.
Actually, if my above comment is correct, maybe the renaming should go: `resized` -> `image` and `layer_image` -> `smoothed_image`
Update the dbaas.xsd to include flavorRef in the complex type "Resize". The sequence should be an either or of "volume" or "flavorRef"
Scary.  Should we do this for all primitive's `modelMatrix`?
Given the number of arguments, would using an object literal instead be better so client code is self-documenting?  The more I used JavaScript the more I lean towards object literals for function arguments even beyond implementing overloads.  Perhaps my JavaScript mentor, @shunter, needs to shed some light.
Asynchronously?  huh?  No need to go into detail in CHANGES.md.  Keep it concise so people read it..  Just say something like:

Added `addImage` and `getGuid` to `TextureAtlas` so images can be added to a texture atlas after it is constructed.
Use FirstOrDefault instead of try/catch
licensen header missing
It may be overly picky, but I feel like these constants should be listed in a more natural way.  IE:  small, normal, large, xlarge, undefined

It can be easy for a user to look at values until they hit an error or undefined value and then stop since they assume they have hit the end of the list of "good" values.
Given that now you do direct eval(code), seems like you don't need 'javascript:' prefix ?
I'm assuming most of our (serious) analytics failures will occur in the wild, and getting information like this from user logs could be important. Good point about the logs though.
Maybe extend it to multiple ranges at once?
This is inside takeExactly
Which functions?
false => isFalse
yeah, that's my guess. I had the same issue where if I put in a "print" and had the debugger open, it worked. Maybe the debugger affects message pumping? not sure but it's spooky
Can the installation proceed if this fails? Or will error also bail?
Generate a serialVersionUID
fluentのversionいつから入ったんだろう
Oh, hmm.. interesting. I guess in the router code we had something similar, where we automatically walked up the tree and injected parents middleware. I think it's a reasonable thing to do and support in Goliath in the subclass use case.. we should pull this out into a separate story/bug and tackle it there.
The URL you have provided for your repo is incorrect (404 error). Please also ensure you insert it in alphabetical order.
`run_start` and `ran_recipe?` assume that `node` and `@run_status` exist, but if the runner is not the chef runner they will fail. If we add this then the test suite cannot be ran without the chef runner.

Besides, `run_start` is only used in the examples, we should not add to the library anything that's not actually used internally. If someone wants to define that method as a helper is welcome to do it.
That seems wrong here? Didn't we add WorkspaceManagement lately? Merge Issue?
hmm.  again, I guess you're right.  I think in earlier comments we'd talked about returning a dict, rather than formatted text.  But then I carried this idea forward even when I now return a class object and a dict isn't needed.  So, good point.  removing...  :)
There *is* a global QUnit async timeout. I think it's `QUnit.config.timeout` or something.
This might be a personal preference, but please, don't.
@croby I also discussed this in the jira ticket for someone to have a look at security implication. In this specific case we're setting the document title.

When we run the safeOutput over it, the title will become "Test &amp; test" instead of "Test & test".
Do you have suggestions on how to fix this in another way?
You might also want to make this a `naked` function to avoid that DMD emits the EBP push/pop code.
adding date facets to parameters
Gary, why `org.springframework.integration.ip.tcp.SyslogdTests` has  `ctx.destroy();`, but this doesn't?
I probably did and you agreed ;). 
I guess after ending up with a repetitive task i figured we need it after all. So I'll take the blame.
Re-phrased the section with commit ad78b7b
Wrap TemporaryQueue in <interfacename/>. Maybe I am just going overboard here, but I generally like the idea of providing additional links, e.g. linking TemporaryQueue to the respective JavaDoc (http://docs.oracle.com/javaee/6/api/javax/jms/TemporaryQueue.html)
I was really going after <task/> and our internal TaskScheduler. I think there are issues going after everything. I'll look again, but we might have to defer to the follow up JIRA.
This can just be the service name for the logger, which would allow us to reuse the logger in ZDT
negative eleven?  just a typo?
I would really suggest that the first thing you do is to store `this` to `self`. Because I have many times had problems when jQuery sets `this` to something else in the context. By having current object always accessible through `self` (and you use only that everywhere where you want to access current object) you will make your life much easier.
I agree, deleting `assert` will fix my problem too ;-)
You're right, 404 or 406 would be better. Also Exceptions in batchAction() should be changed - they throws RuntimeExceptions, too. This cleanup of this CRUDController can be done in another PR as the main target of this PR was to fix security issue.

Is there any prefered Symfony Exception, that returns error 406?
Sweet - if you can tweak these to use ```blank?``` (make sure to squash afterwards), I'll merge ASAP. Thanks!
Methods should be named as verbs, not nouns.  GetPartType maybe?
Probably not, but the code thinks it does now. I plan on taking some of this duplicate test.conf stuff out of the code soon, but not on this PR.
Hm, it is based on [this MongoEngine field](http://readthedocs.org/docs/mongoengine-odm/en/latest/apireference.html#mongoengine.SortedListField). I have never used myself, though. So I am unsure how exactly it behaves. But our name should match it, no?
should be logger.error
I think is better to make the `amount` param as required in every wrapper, and only make it `optional` in the external API `TimeWindowDropCollector.drop`.

This way is easy to test and maintenance.
I needed to add this tweak here because the sign out feature wasn't passing when trying to delete the session. Devise uses delete method for deleting the session, and cucumber wants to test get, so it wasn't working. I did a little research and found the following explanation, which made sense at the time: https://github.com/plataformatec/devise/wiki/How-To:-Test-with-Cucumber

Even in their examples they show we need this trick: https://github.com/RailsApps/rails3-mongoid-devise/blob/master/config/initializers/devise.rb
If the exception isn't working properly, is that because the exception function has a bug or because your code has a bug?  

Either way, it's broken. :-)
you may want to add 

```
else { DrawNpcOverlay(npc); }
```
Where is size defined?
Does this tie one-jar into the package phase? AKA can I build a one-jar artifact with just mvn package?
I would remove these if we're not going to use them in the example.
const Vertex&
Just show examples of passing a low threshold, and a long string - we don't really need examples of bad usage of the function.
All properties for `TiBuffer` should be considered read-only for the purposes of module development and as a result their signatures may need to be changed (however, we still need them internally).
Any idea to easily allow both propertyPath and constants ?

ATM the parameters hash looks like this:
```
{
    "parameterName"  = "propertyPath",
    "parameterName2" = "=constant"
}
```
Oops, yes, I initially had it in the route, but I think eventually it'd be nice to have null handling for providers also, so I moved it to the relation.
ok the name must be ``monitoring.rst`` with a ``t`` and the syntax is not valid, currently it is mardown
I guess we should be consistent. In this current implementation its console for xhr but alert otherwise. The good thing about alert is you don't need the console. Maybe we should just do both.
The implementation for the tablet looks like it uses "Ok" but on the simulator, I see "OK" (all caps).  The same thing happens for me on the BB10 simulator (Ok shows up as OK).
Please just merge and then fix your own code.
Especially considering that it's a sentence and should have a full stop :)