labofsahil/github-event-dataset-2012 huggingface dataset:
Is there a reason to use this `sqrt` function instead of `math.sqrt`?
Maybe rename `threshold_adaptive` to match `threshold_otsu`?
I really don't think these should be failing, but the expected values are wrong, they should go to `/en-US/mobile`
`primitiveCommandList` isn't a command list.  It is an array of command lists - a list of lists.  Come up with a better name.

I also have ideas for reorganizing and flattening this list, but I want to hold off since the suggestions could be irrelevant when we introduce command trees.
As for spaces. I'll rewrite that commit from home (8-9 hours later) after work.
Shouldn't you also just `break` here? (since the flag only needs to be set once)
We have a `TiC. API_LEVEL_ICE_CREAM_SANDWICH` to use instead of `14`.  Also, I don't know all the rules of java operator precedence, but i would think you would need to put the 'or' conditions (||) together in parens for this to work as you expect.  Even if it does work and is legal, please put parens around for readability.
Personalamente I'd prefer an optional indent arg on util.log rather than these big space gaps.
change to use remove(i)
nitpick - capital DOM
Space after "for".
I think Jasmine already handles this with the `toBeCloseTo` matcher. Check out: 

* http://pivotal.github.com/jasmine/jsdoc/symbols/jasmine.Matchers.html#toBeCloseTo
* http://pivotal.github.com/jasmine/  (Do a find for toBeCloseto)
These are different if `newData` is `nil`. That probably can't happen, but have you thought through that code path?
can we name this `customTrackParameters` ?
Maybe say "without optional arguments" rather than abbreviated"
Может перенести файл и поменять `JST['uploadcare/crop/template']()` на новую функцию после мержа? Я не очень дружу с rebase'ми и смутно понимаю как мне забрать те изменения чтобы этот PR не сломать (хоть те изменения уже и в мастере)
Попробую
No spaces before the `&$Model` variable.
Also, while the `$(D DollarToken` here can just be `DollarToken`, it currently lacks the `$`, so it's wrong regardless.
oddly, all labels in the switch should be aligned the same... they're essentially goto labels.
prolly we can delete this
Need to do this all over the place with `$entry()`.
Another Note
The reason from the JavaScript style guide is obsolete in dart given that
with """ it doesn't matter at all whether you used single or double quotes.
I believe multiline strings are superior to cascades for the import case.
 The code is more readable this way as it doesn't include \n or extra
quotes.

On Tue, Aug 28, 2012 at 10:50 AM, John Messerly <notifications@github.com>wrote:

> In components_polyfill_boostrap.dart:
>
> > +   */
> > +  final Element html;
> > +
> > +  CustomElementDeclaration(this.url, this.html);
> > +}
> > +
> > +/// Build up a dart file that when executed laads all components on the page.
> > +void runComponents(List<CustomElementDeclaration> declarations) {
> > +  var sbHeader = new StringBuffer();
> > +  var sb = new StringBuffer()
> > +    ..add("""
> > +#import("dart:html");
> > +#import("$POLYFILL_LIBRARY_PACKAGE", prefix: "polyfill");
> > +""");
> > +
> > +  var sbMain = new StringBuffer()..add("void main() {");
>
> I thought we inherited that guideline (prefer single to double quote) from
> the JavaScript style guide? It makes it easier to build HTML strings.
>
> http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml?showone=Strings#Strings
>
> I disagree on the triple quotes though. Multiline strings are the perfect
> use case for triple quotes. That's why we have that feature :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dart-lang/dart-web-components/pull/16/files#r1477425>.
>
>
<b>データアクセス：</b>データアクセスAPIを$.Deferredで実装するのは、大抵の場合正しい選択と言えます。
I'm not convinced that serializing should preserve the null entries
Correcting this in my next commit...
same issue for the web root
You've removed `libopenssl-ruby` here. Is there a reason for that?
I think hide! is not available for 3.2, is it? Wouldn't it break the generator in case it is?
And do this instead?

```
module WoopleTheme
  class ContentItemPresenter
  end
end
```
I have been making this mistake for soo long!!
This should be 

```
.. note ::

    When a key ...
```

to be formatted correctly on the website.
TIME PARADOX
The git links come from the gitwash project, which provides a template for users of many projects to have a consistent doc for how to use git.  While we should include these link fixes in this PR in our own tree, you may consider submitting a PR against gitwash too so it will be fixed for other users


https://github.com/matthew-brett/gitwash

As an aside, I am not too happy with some of the suggestions in that document (and hence in ours) in terms of how to handle upstream version, and so would like to make some changes to the suggested git workflow, either by patching gitwash or decoupling from it.
two more blank lines here
Why would you need to mark something as explicitly weak `pure`? So that it can be called from `pure` functions but calls to it can't be elided?
Should there be a NotNull Assertion?
Opps, that's a bug from related to some other feature I wrote for myself.  It should  just be DB instead of iondb.  Should I do a different pull request?
For this part, yes, that should work (or even `new Date(date)` -- `new Date(+date)` if we're paranoid -- should work).

I'll still want to flesh out the test suite before pulling this in, though (honestly, should've done that months ago!).
You should remove this file now that you put the function in the database.py.
The idea is to check right here, whether the given name is absolute or relative, i.e. starts with '/' or not. In the latter case you need to fetch the current contentDocument to get its id and append the given name. Maybe this should then be normalized to allow for relative paths to siblings (e.g. '../../block-1').
This was added for ACE, which does not seem to be a good solution for what we want to do. I'll look into better alternatives.
I was expecting a ForeignKey.
does this work properly on all PHP versions or does it try to call the method in some of them ?
commited your change request
Yeah, okay. The problem is, VERSION.RELEASE_VERSION can be anything, for example 2012.1. In setup.py what you want is the git version, nothing else. The filename itself says that.

However there probably is a problem when checking out a release, where RELEASE_VERSION is hardcoded. This probably breaks if gitversion always include a commit sha (and other players with the same release have '2012.1' as RELEASE_VERSION), unfortunately. Oh well, then it needs to stay this way.
With Gemfile like this:

```ruby
source 'http://rubygems.org'

gem "rubysdl"

group :test do
  gem "rspec"
  gem "simplecov"
end
```

I tried to convert all "require sdl" to bundler:

```ruby
require 'bundler'
Bundler.require(:default)
```

And changed spec_helper:

```ruby
require 'bundler'
Bundler.require(:test)
```
But it didn't work, I got the follow error:

_NameError:  uninitialized constant Screen::SDL_

Is "rubysdl" the correct gem I have to use?

thx
Non t'as raison.
double `the`
Need to rebase and move this to test-app/automation
Well basked on what was implemented you would get back a hash.
I assumed I was following what you intended.

So if your data bag follows the convention of:

    {
      "id": 'index",
      "index": "value"
    }

Then it all works fine.  This is what I thought was intended, the previous impl just returned a hash.
Regarding the structure, what about making this a sub section of @Indexed or add it as an advanced topic? In the latter case a link in section '4.1.1.1. @Indexed' would be good
(potential-bug) Potential array out of bounds if there are more than @colorIndex dataSets.

→ Please do boundary checking here, and wrap the index from 0 → array_len -1
each phrase has its own list of regexes that come from the phrases respective n-grams, so have to do that in the loop
I'll look into it. This is also the logic used in the server. Another option is modifying custom require to have some like a require.exists to see if a file exists.
I'd really rather not bother with methods like this which are such thin conveniences for what's already in Foundation