labofsahil/github-event-dataset-2012 huggingface dataset:
No, we don't. That's why I am saying to remove it. ;-)

Originally, that `with` statement was used only for login. So it was OK to try, one after another:
 * `next` from current template context
 * `next` from request's GET
 * `LOGIN_REDIRECT_URL` from `settings.py`

Of course the last one should not be in a general form template we are having here. But it could be used (just as an example) in `login.html` template or somewhere, before you include the form template, like this:

```
{% with next|default:request_get_next|default:LOGIN_REDIRECT_URL as next %}
    {% include "form/form.html %}
{% endwith %}
```

Because this sets `next` template context variable, in the included form template, `with` gets the value in `next` and uses that for `next_url`.
This looks good, but can you actually make a new NET_45 variable and use that instead?
We should get rid of these `__author__` etc tags - they date back from a time when revision control systems were not able to keep track of these things (the `__docformat__` tag, however, _is_ useful, as it helps the documentation generation tool to parse the doc strings.
Or you can just switch #ifdef DECLARE_UDF in the other headers if it's not from main.cpp.  I assume DECLARE_UDF is void if DECLARE_LIBRARY_EXPORTS is not defined.
I don't know about any other program. But specifically because you are using Ex Falso, this makes TagLib basically destroy the tags for you if you save the file, no?
probably __package_opkg in the header? ;-)
Should be `depends_on 'xz' => :build`, since this is only needed at buildtime.
Never mind... saw you moved this into upstream attrs.
Log fehlt.
my eclipse is pretty old. will merge this now.. will see if this issue comes up again.
Sorry, I've missed the "store *in redis*" point before.
I've to think about it, but my first thought is that's too complex. "Complex" for me mean that it adds some strong constraints in the code. 
Here are some practical points I can think to:
- a store action at each first collection call is not costless
- this also means that each call of collection will almost check in redis if the temporary key exists, which another time is not costless
- if you ask for the first element of a very huge set, you'll add this set again in RAM, and the store operation another time will not be costless
- there is also some implicit/explicit issue for me
- we also have to think about thread safety, and in the other hand a minimum of centralization: the fact of a huge set stored just to get one element could be tricky if several threads do the same in the same time...
- saying "cache", we have to say "invalidation" also: what if some data stored in a temporary key changes?

I have to think about the "store" option of the "collection" method you've mentioned. It's more explicit, and thus I smell it better. But, also, it will really add complexity: we will have both scenarios (storing and not storing) to code and maintain.

At this time, I vote for not coding this feature, but keeping it in mind. Starting to play with the CollectionManager in real life, we'll have a better opinion of the need of this kind of enhancement.

To be continued... ;)
Please leave a space between the cast operator and the variable.
personal preference...fill works for all field types and type only makes sense for text fields.
Also you completely ignore some other arguments I made: 

- Both in JS and DOM order in which handlers are invoked is deterministic. Making API that invokes handlers in undeterministic order is pretty risky design choice that I expect to be harmful. 
- I already described a use case where undetermined order will force users into handling queuing manually. 

I still don't see strong arguments against making order deterministic. All I hear is:

- users / contributors won't be aware

  I hardly see it as a problem, not all drivers know how their car engine works, but they still can use it to drive around.

- That is a shaky bad idea.

  Sorry, but I'm not convinced, for reasons outlined before.
Should have a space before the `?`. Might format it with the `?` on the next line too.
I would add here a comment that LINEBREAK_CHARS are added by purpose to NAME tokens.
background-color would be sufficient here.
Suggest also words like 'Wing' and 'Arm' - represent some fly-boys. A lot of fiction uses a naval analogy for military forces in space, whereas the United States, for example, uses its air force to fly to space, rather than its navy.
If we are adding a mock library only for this case I think is better do

```ruby
def Rails.application
  nil
end
```
`aslias`->`alias`?
Should probably be underlined to make it a title. Doesn't feel like it belongs to the minimal example section

```
Basic Documentation
-------------------
```
in general I prefer to keep the loglike definition in statsmodels, mainly to know what's used and so I don't have to pay attention to scipy versions
This function is a generalization of `allWorkspacesExceptWith` so it should be merged into.
哦，这样我就明白些了~ 那么credentials这个词就是授权/权限的意思么，不过按照原文来讲，似乎只是讲设置名称。是不是应直接译成“设置工作进程使用的非特权账户名称”，或者“设置授权工作进程使用的非特权账户名称”？
Are these HIEM changes related to Direct?
These comments in the individual protected methods seem unnecessary.
more system.out
switch to spring managed injection
absolute file path, must be add inside the bundle + use asset()
Yes, I was a complete dork and didn't notice that one of the "copies" of that file was actually a symbolic link.  Now I have to figure out how to undo the removal of one file from a commit that included the removal of several, sigh.
I would still like to see the ellipsis here replaced by something more informative.  See me previous comment about this, which GitHub now thinks is "outdated", so I just want to make sure it doesn't get forgotten.
I think the link here should be changed to the top level wiki, rather than Config-Documentation per se.  And the sentence could then be slightly more expansive: "A guide to using the configuration files, a FAQ for installation issues, and other useful references can be found..."
What is `args`?
No, never ever noticed such a behavior since working with AE + jQuery...

I thought (jQuery.browser) this will always work -- the only evidence that there could be a problem I found on the jQuery docs:
"Contains flags for the useragent, read from navigator.userAgent. We recommend against using this property;" http://api.jquery.com/jQuery.browser/ -- I only can image that navigator.userAgent is manually set incorrect or so...

Somehow I think that it should stay as it was... but there was this bug report (which I also somehow trust)... so I'm not really sure & hope you'll know the best solution :-)
since I am specifying just the date they are all at midnight.
3/14 is exactly 24 hours although it could be seen as 24 hours and 1 second.
I could be convinced that it's 24 hours and 1 seconds and thus there is no refund
「欲しいのは UserGlossary の id だから」というところの意図がまだよく理解できていないです... ><
UserConfigを構成する要素(glossary/source_language/target_language)はPOST/PUTのパラメータとして受け取れば困ることはないと思っているのですが、何かうまく伝えられていないですかねぃ。。
"Shard ID: Default" would be acceptable.
Any reason for this change?  'basket' should be part of every template's context.
I don't think you need this scope. This is a standalone web app.

But you should leave it if you have your own reason on this.
nit: correct title
Expand a bit on this docstring to indicate that its "like a PushButton which stays down when pressed" or something to that effect.
Okay let me work on them as my next project, so that way we can take these further and improve them, we still don't have OpenJDK building on other OSes - I don't have one atm other than my pet Ubuntu!. Is that something for our to-do list?
Careful with `!ea instanceof lively.morphic.HandMorph`.  Operator `!` has precedence!

What about:

```js
if (!action || !this.loggingEnabled || !action.morph.isLoggable) return false;
var parentDoesNotWantUndo = action.morph.ownerChain().detect(function (ea) { return !ea.isLoggable && !ea.isHand })
if (parentDoesNotWantUndo) return false;
action.time = Date.now();
if (this.stack.length > this.counter) {
    this.stack.splice(this.counter);
}
// keep changes at a bundle if they happen at the same time
var otherAction = this.stack.last() && this.stack.last().last();
if (otherAction && (action.time - otherAction.time) <= 100) {
    this.stack.last().push(action);
} else {
    this.stack.push([action]);
    this.counter ++;
}
return true;
```
This file seems like the next target for consolidating.  We basically have one file to launch an F3/4 and one for F1, I think.
Yes. It's similar to the [BSD KNF C style](http://en.wikipedia.org/wiki/Indent_style#BSD_KNF_style)

```
function somethingWithArgs(one, two, three) {
  if (foo) {

  } else {

  }
}
```
The overflow-y should probably be added to application.css as overflow-y: 'auto'.
А каким образом searchString может оказаться пробелом?
okay, I'll update
This is a browser tab actor, so you could just use 'title'.
I think side loading is a way to support related objects in one response. We're using embedding for related objects so we shouldn't need the sideload line. You should double check that what I'm say is correct and then remove this if we don't need it.
I don't think that this error message is particularly clear
Это тоже проблема прототипа. В будущем будет _иначе_.
Well ``_random_sample_mask`` is slower than the above function but it sets exactly ``n_total_in_bag`` items to True. Gilles proposed something along the lines of ``np.random.permute(n_total_samples)[:n_total_in_bag]`` but this slower than ``_random_sample_mask``.
Here is the code for the 20 newsgroups data fetcher:

https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/datasets/twenty_newsgroups.py

And the documentation for the mldata loader:

http://scikit-learn.org/stable/datasets/index.html#downloading-datasets-from-the-mldata-org-repository
> you should capture the warnings to test that it is indeed raised.

do you have an example?
``"X[0] <= 0.0"`` sould be ``"X[0] <= 0.0000"`` due to ``%.4f``
in this case i think we are ok either way. although similarity has higher word frequency than affinity!

for citations, here are a few:

Ng et al. used affinity matrix
bach and jordan: similarity or affinity matrix
zelnik-manor and perona: affinity matrix
von luxburg used similarity graph

wikipedia uses similarity matrix: http://en.wikipedia.org/wiki/Spectral_clustering
A note should definitely be added somewhere around here. Maybe even more prominent. (also the formulation is a bit awkward at the moment).
sorry @ogrisel, @GaelVaroquaux i don't think i have represented the diffusion map algorithm well. even though i initially thought it's merely a variant i think we are talking of several differences. my reading of the literature suggests that the spectral embedding implementation currently in sklearn is what the paper 1 below refers to as a laplacian eigenmap.

in general these algorithms are of the form: data -> similarity matrix -> some transformation -> eigendecomposition -> some projection (hence also the notion that this can be done via a pipeline).

i hope to use sklearn to reproduce results such as these for my data/domain (and i think we should support the algorithms included in the comparison):

1. http://www.iai.uni-bonn.de/~jz/dimensionality_reduction_a_comparative_review.pdf (a particularly nice example of sets of algorithms that we should have in sklearn and also a consideration of computational and memory complexity - table 1)
2. http://dx.doi.org/10.1109/MCSE.2011.55 (access limited 

further, it's currently unclear on what basis algorithms are chosen for inclusion. even by crude metrics such as citations, "diffusion maps" has many more citations compared to ltsa. if anything, hopefully this discussion can lend some clarity to what one can contribute to sklearn before somebody spends time trying to develop something for the package.

so here are some questions for you:
- what is the objective standard by which you measure practicality and usefulness? 
- what is a variant on an algorithm?
The `None` case should be documented as well.
$count => $wallpapersCount || $userWallpapersCount
As it stands it makes sense for it to go under the listener as it is an option of the listener although I do understand @r1pp3rj4ck's concern that is out of keeping with the other options for the listener and I'm not too concerned if it does not go under the listener in the config.

It did occur to me though that maybe using the method should also be available for populating the index anyway. it would be less efficient than a repository method that only returns the relevant entities but for someone not indexing a large amount allowing it to be called when populating the index would make for a more consistent approach to deciding what gets indexed. I'm particularly thinking of cases where it is not just checking a property or two which could be easily translated into a query but the use cases like checking acl where a service may be used to do the checking. @jmikloa what are your thoughts?
you should revert it. It cannot be merged as is
seems i did it everywhere in this patch, where exactly you mean?
This is essentially copy/paste from the existing Left- and RightDrawer.java files which have the same comment (at the same location).
Corrected this and other spelling errors in af375e6
Do you think we should advocate explicit AZ selection?
It is truly # of eclcc processes right? not threads, so less confusing if var was max..Processes ?
erm... wait. My brain is backwards. I'll try not to comment so early in the morning again >_<
Тут скачазати нічого не можу  Сергій їх там  для чогось зробив , його зараз нема, нема в кого спитати
Done in 1809868
It's to slow to call QString::mid, try to use QString::midRef
Also wrap your c-string by QLatin1String
We do validate input option types and we should do it here as well.
phpdoc was missed
Is this part of the PR ?
Unfortunately Handlebars comments can only be used for leaving notes - they can't wrap around code to prevent execution. So using JSP comments here.
This should be broken up into smaller test cases.
This is going to catch all of the third-party stuff including gemcache.
A little more descriptive name pls
value :P
weird, ok
what is the meaning of the "#:" ? is that pep8?
Could you remove these two comments?
Is there a reason why `getDefaultTexture` and `getDefaultCubeMap` aren't just user settable properties?  What if I have a Cesium based application and I want to replace the default texture with something else?  Seems like a useful and 'free' feature to have if we just expose them directly.
I'm fine with this.  My thought is that since unit tests are white-box tests, we're allowed to omit a test for a method if another method simply wraps it.  Otherwise we bloat the tests for no reason with lots of redundant code.
Do we still need to clone here since it is copied above?
Things created in 2012 have a copyright date of 2012.

According to [Wikipedia](http://en.wikipedia.org/wiki/Copyright)
> In the United States, the term for most existing works is a fixed number of years after the date of creation or publication. 

If you post it as 2011, I suppose that Microsoft loses a year of copyright protection.
Or you could have just done:

    applyProperties: function(props) {
        require.mix(this, props);
    }

There's a bit of a debate if var i should go in the for loop or not. JSLint says var i should NOT go in the for loop. However, JSLint says you should do a hasOwnProperty() check on i. mix() does the hasOwnProperty() check. I would say move the var into the for loop or use require.mix().
pinch and touchmove are supported on mobile web (all touch events are supported on all controls).
Do we need this console.debug() call left in here?
infSource
That means like `Object.keys(options).forEach(function(key) { var value = options[key]; ....` ?
This should have been a `git mv`, instead of deleting and recreating.
We actually only need timeit and apache_log, not incr_count.
It's showing `0x` in this final hint when you have a problem like `(x + 3)(x - 3)`.

Maybe `<span data-if="B + D !== 0">signOp( B + D ) + ( D + B ) + "x" +</span>`?
This comment should read 'return 0 if it was an all-pass'
Anyhow, `static T i;` should be equivalent to `static T i = T.init;`. Better yet, let's make this `static immutable T i;` so as to avoid thread-local storage.
Also ditto:

```D
auto commonPrefix(alias pred, R1, R2)(R1 r1, R2 r2)
    if (isForwardRange!R1 && !isNarrowString!R1 &&
        isNarrowString!R2 &&
        is(typeof(r1.front == r2.front)) : bool))
{
    commonPrefix(r2, r1); //Swap and forward to the other implementation
}

auto commonPrefix(alias pred, R1, R2)(R1 r1, R2 r2)
    if (isNarrowString!R1 &&
        isForwardRange!R2 &&
        is(typeof(r1.front == r2.front)) : bool))
{
    //...
}
```

Of course this time, the implementaion will also have to `static if (isRandomAccessRange!R2)`
The idea is to unittest that as well.
> You have also probably forgot to mention function parameter storage classes

Yes, that's right. It is also required.

> My understanding is both are needed as one refers to transitive constancy of delegate type and second qualifies hidden class context. Is it correct?

It is correct. For function, function pointer, and delegate types, don't apply transitivity.
Shouldn't this assert too? It seems inconsistent to use exceptions here, but asserts everywhere else...
somewhere the release branch must have gotten merged in. can you change this back to: 6.x-dev.
I believe `#validate` throws an exception if a requirement env var is missing. Should it be printing a usage to STDOUT instead?
When you use it to qualify/enforce our Validator implementation it seems right. From that perspective it is probably the best name.
Fix multiline docstring:

```python
"""
Configuration step, we set FC, F77 is already set by EasyBuild to the right compiler,
FC is used for Fortran90.
"""
```
I've been writing this instead of Level.isLevel = true as this seems to be the general coding practice here
Javadoc about what exactly this does and whether it can be called on the main thread.  Lots of this stuff will strict mode warn!
Nope, my bad, can't see why that came to be... Will fix now.
i18n research project due
This is not the typical way to return data - why not use the i18n library?
But we want it to be a false value for older iptables, remember?

irb(main):007:0> iptables_version='1.3.1'
=> "1.3.1"
irb(main):008:0> mask_required = iptables_version and Puppet::Util::Package.versioncmp(iptables_version, '1.4.1') >= 0
=> false
irb(main):009:0> mask_required
=> "1.3.1"
Zu testende URL erstellen "/firm/__FIRMID__"
Думай головой все ж при копипиздинге.
Oops. I had the `ignore` calls implemented as a decorator, originally. Thanks!
The code still won't fire unless the system detected a segfault though, right?
Change to use class name. Will take care of it during merge
In that case it would be best to change this for ```_do_mb_search``` with an extra commit after pulling in the changeset.

It does get a bit tricky having different indentation in the same file. But that would be fine with me.
Depending if you will be using any feature which is not in older versions. We can limit to > 1.4 (there is no difference in API between 1.4 and 1.4.1). At least I personally don't bother with older versions.
Globals are used inside the bundles.
I was going to add "Applies to [class|property]" for each annotation, but kept it simple
updated.. dont you see it now?
This should not overflow, you are calculating the fibs one too far. No points from this one.
yep, /me has forgotten the right syntex
Any way to not hard code this? For during staging, etc. this needs to be modifiable.
Did you look if the key is used for the batch delete action ?
No MVC application here? Is testing modules directly the intended usage?
The ";"  alignement is a convention ?
revert this entire file ... it will be in REPORTS-29
move this change to REPORTS-29
a metaphysical statement?
I know these are tests, but do we want prints here?
please use proper whitespace :)
I actually like having both, since you may want to perform a system upgrade, but not a dist upgrade, or maybe indeed a full dist upgrade. Not necessarily as relevant to vagrant (since you could just change your VagrantFile to reference a newer dist image), but relevant within the scope of general server administration if this script were used while managing non-vagrant boxes.
You're right, we should retain BC for the moment... at least in a minor release.
alan inja man age span nazanam be khde div nemitunam background color bedam.
nemidunam chera?
You can use a single console.log statemente here:

`console.log('sidebar toggle. element is', sidebar)
de-indentation needed.
Would be nice with an example for this as well. Do we have a test?
Hum, I guess an exception could be thrown here if the uri is already registered.
Existing behavior isn't important for us since we are a new platform and nothing is really set in stone. It's better to have a sensible default.
That method does not have a @Kroll.method annotation on top of it, so it is not exposed to the user.
Shouldn't it be `Localizable`?
scroll up/down :-)
Oh,  that's just lame but your right it fixes the Traversable problem.

The new problem is a Call to undefined method Mock_NodeMock_ec0e77c2::getPath() Error.
The method is defined in the \PHPCR\ItemInterface and should have been mocked, adding the function like in the original Test makes no difference.

As you can tell I'm not an experienced tester but this all behaves a bit weird in my opinion.
You'll notice I added the "e" argument here, this is a standard event arg that is passed to all jQuery event handlers. I've also avoided requerying here just by checking the native .checked property.
Would be awesome - thanks! Just keep the text and break it into multiple lines e.g. as emacs would.