labofsahil/github-event-dataset-2012 huggingface dataset:
Теоретически варианты выбора из 4х, 10, и xxxxx количества должны быть равны, возможно это предрассудки, но более правильное распределение получается именно при выборе из некоторого количества значений, то есть на отрезках. У меня не большие эти отрезки, в общем я не заморачивался выбрал всего-лишь по 25.
Вариант с выбором из 4х вполне себе тоже не плох, вообще можно посмотреть примерное распределение значений при большом количестве попыток, ради интереса.


Sun, 04 Nov 2012 11:21:45 -0800 от s1dney <notifications@github.com>:
>	
>
>
	
	
>
		
		
			
>
>In DNAProject/DNAProject/DNAProject/Cell.m:
>> +//
> +
> +#import "Cell.h"
> +#include <stdlib.h>
> +
> +const int ARRAY_COUNT = 100;
> +
> +@implementation Cell
> +
> +-(id) init {
> +    self = [super init];
> +    if (self) {
> +        _array = [[NSMutableArray alloc] init];
> +        for (int i=0; i<ARRAY_COUNT; i++) {
> +            int r = arc4random() % 100;
> +            if (r>=0 && r<25) {
>
>вероятность чего?
буду благодарен, если дашь пояснение, я не вижу разницы между выборкой из 4-х элементов или из 100 по 25…
>
>On 04.11.2012, at 23:09, VGPETROV <notifications@github.com> wrote:
 In DNAProject/DNAProject/DNAProject/Cell.m:

 > +//
 > +
 > +#import "Cell.h"
 > +#include <stdlib.h>
 > +
 > +const int ARRAY_COUNT = 100;
 > +
 > +@implementation Cell
 > +
 > +-(id) init {
 > +    self = [super init];
 > +    if (self) {
 > +        _array = [[NSMutableArray alloc] init];
 > +        for (int i=0; i<ARRAY_COUNT; i++) {
 > +            int r = arc4random() % 100;
 > +            if (r>=0 && r<25) {
 Для увиеличенной вероятности, в данном случае 100 Sun, 04 Nov 2012 11:00:56 -0800 от s1dney <notifications@github.com>:
 In DNAProject/DNAProject/DNAProject/Cell.m: > +// + +#import "Cell.h" +#include <stdlib.h> + +const int ARRAY_COUNT = 100; + +@implementation Cell + +-(id) init { + self = [super init]; + if (self) { + _array = [[NSMutableArray alloc] init]; + for (int i=0; i<ARRAY_COUNT; i++) { + int r = arc4random() % 100; + if (r>=0 && r<25) { забавно, а почему именно arc4random() %100 ? почему бы не сделать просто %4 и выбирать символ из 0 1 2 3?
 if (r == 0) {
 _array addObject:@"A"];
 }....
 —
 Reply to this email directly or view it on GitHub.
 —
 Reply to this email directly or view it on GitHub.
>
              —
>
              Reply to this email directly or view it on GitHub.
С Кнутом не поспоришь ) Спасибо за ссылку.
зачем вам синоним? =)
Тут проверяем длину набора уникальных индексов
Тут проблемка есть.
getRandomElement может вернуть элемент равный текущему. Тогда замена по сути не произойдет и итоговая мутация будет неверной.
Sorry for the delay.

After looking a bit more into it, with the current implementation, if we didn't render the columns in the first place, we would need to manually reinsert the columns in Javascript when the user changes which columns are displayed (or reload the entire table). 

Since a column can be hidden on the initial load of the table and re-shown after at any time, it is easier to render and hide it when the table loads. This way, the column's code is still there and all we have to do is show it (if the user chooses to).

What do you think?
If we have proper routes, the request should always be a post and we shouldn't need this check. Thoughts?
messages não deveria estar no plural?
これ ```%pre~``` じゃなくて大丈夫でしたっけ。複数行+インデントが絡むと駄目になるような気が。
Can you please write this [without the space in the ternary operator](https://github.com/github/objective-c-conventions#expressions):

```objc
[values[i] addObject:x ?: RACTupleNil.tupleNil];
```
I've been thinking about that, but it seems antithetical to the idea of a subscribable pushing values to the subscribers, vs. subscribers pulling values from a list. By definition it can't be lazy in resolving future items since no one's requesting them.

Note that it *is* lazy in a certain sense, though. The generator only starts generating values once someone subscribes. Then they're off to the races.

It's definitely less than ideal compared to truly lazy generators, but even with truly lazy, infinite generators you still have to limit them somehow, once you want to get values out. So this generator might end up generating more values than is necessary, but it won't (usually) generate the whole sequence.

Try, for example, inserting a test that calls `-toArray` on an infinite list. Spins for a while and then crashes. But we can still use an infinite list without resolving it completing.
Don't use TODOs, use Github issues
On Aug 26, 2012 9:27 PM, "Jarvis Johnson" <notifications@github.com> wrote:

> In app.js:
>
> > @@ -19,13 +32,17 @@ everyauth.password
> >      .authenticate(auth.authenticate)
> >      .loginLocals(function(req, res) {
>
> add //TODO(thedekel): Figure out the difference between
> loginLocals/registerLocals
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gt-webdev/attendance/pull/153/files#r1461615>.
>
>
I’d find it more natural to have this function (and the one above for owners) as methods in `metashare.repository.editor.resource_editor.ResourceModelAdmin` (see [third possibility here](https://docs.djangoproject.com/en/1.3/ref/contrib/admin/#django.contrib.admin.ModelAdmin.list_display)). This wouldn’t require any generator changes. And the purely admin-related functionality is only in the admin class, instead of in the model.

But perhaps that’s just my personal taste …
It's okay, yesterday I forgot that I had already written in some functionality I was requesting :(

I'll make that change.
let's keep this in mind for 0.6 then. I agree with what you say.
I would put n_pca_components=64 here rather than when call of pick_sources_epochs
Crap! Sorry about that -- I'll be more careful about it. Before merging those offending changes, I ran nosetests -x, but I didn't run make test-doc. I'll do that from now on. I thought it was weird that I had to change those files and I was going to investigate it this morning. Thanks for finding it.
The AUTHORS file was changed on master when I added you as a collaborator to the repo.

Has master been merged into couchpotato recently?
`Given I have checked out a new branch named "barney-and-wilma"`

would seem to be a little clearer. The way this is worded doesn't make it clear that you're creating a new branch in this step
...addListener(dispatch);

if (typeof(exports) != "undefined") {
  exports.dispatch = dispatch;
}
It should say "You have now"
fixed, all tests consolidated to AmqpIntegrationTest.
Did you intend to delete this?
Оу, и чо, оно правда работает? Вот забавно!
Ньюлайн же после.
¿En qué parte esto resuelve nombres?
Import only classes that are actually used.
Excellent detail in these answers!
Please translate it ! :) {% trans %} ...  {% endtrans %}
Please do not use hard coded strings.
this print doesn't goes here... i just forgot to delete it ^^
Sent by email
This is important enough to get its own Python module.
This log here seems redundant as the rpc layer already logs what's being unpacked.
I should have been clearer, you got the points from this one. Sorry about that. You are only missing points from the last two exercises.
in the case we have an exception at line 199, the tx should be rolledback here, not forcefully committed.
Ah. It seems not... ok, that bit was a little confusing, perhaps an overall comment describing the strategy of this method would be helpful here.
I wonder if it should be the other way around: By default a backend cannot blit, but if you implement the functionality, and toggle the switch, you get it...
suggestion -- small tweak to the comment -- <code># select a random featured version, except the 1st in the list since it may not have data</code>
Does this cover the whole bug? I.e.: is there more to do in terms of validating the structure/content of the new dictionary format?
the test failed for me at this line. 

E       NoSuchElementException: Message: u'Unable to locate element: {"method":"id","selector":"text-filter"}\nCommand duration or timeout: 14.25 seconds\nFor documentation on this error, please visit: http://seleniumhq.org/exceptions/no_such_element.html\nBuild info: version: \'2.24.1\', revision: \'17205\', time: \'2012-06-19 17:28:14\'\nSystem info: os.name: \'Windows XP\', os.arch: \'x86\', os.version: \'5.1\', java.version: \'1.7.0_04\'\nDriver info: driver.version: EventFiringWebDriver\nSession ID: 67cda6fe-e8bc-424b-846d-f696610bc347' ; Screenshot: available via screen ; Stacktrace: Method newInstance0 threw an error in None
Согласен!
I cannot speak to most of this
+1 better writing
Thanks for making the unit tests more rigorous!
One thing I don't like about the `$clear` argument is having to call false on every subsequent call to make it append instead of clearing the existing predicates. I'd prefer the `addFrom()` and `setFrom()` pairs which are clearer, and there's already some use of this pattern with `addFilterToJoin()` and `setFilterToJoin()` anyway.
Isn't that supposed to be `Member::get()`? `Member::create()` will give you a new instance of Member I thought.
@adrexia isn't this accomplished already by the TabSet.js::onremove? Looks like this could be safely removed.
Yes, we have different views, and it looks that one doesn't convince other:)

I like thinking about future. When I buy a car, I don't choose one that carry me only from city A to B, because this is my current need. I now that in future I will want to drive to city C, D, ...
I think that taking a wihile to wonder if class is well deisigned is not so expensive to not doing this. I often have to rewrite some classes from basis, because developer who designed it didn't think about future. It causes false believe that I can not worry about future, because this class is sealed.
Let someone else worry about it in the future.
I agree that for example FormCommit class should be sealed, and most of its method should be private, because chance that someone inherit from it is very low. But I do not see any reason, but using ReSharper, why GitComandName can not be public.

> At first, they allow you to avoid inheriting

If I have full access to source code, I just add a method to that class instead of adding extension method for that class. This aproach is better because when I look for some method then I find it regardless to namespace for extension method is in using block.
It is used in the Pro editions for editing field level ACLs. But we can tag this out in the CE edition.
Are you sure ``'allow_self_signed'`` has the correct type as ``string`` is not a sub-type of ``string``?
Are you sure ``function ($test) {     r...ib\ResponseInterface; }`` has the correct type as ``object<Closure>`` is not a sub-type of ``array``?
I will move this function and resubmit pull request
Зачем здесь это?
you should keep this line to keep B/C
Please add a constant for cropping (like there are for the scale methods) and change the default values to the corresponding constant.
You got an extra space before the second comma.
Please check `skip_blanks` first, it's an improvement for this option when true.
what about other options (e.g. timeout?)
No need for this?
But it would really look ugly here. :-) You can leave it as it is. ;-)
Just a question. Why this method takes `post_date` as an argument and not just read it from `self`?

Probably we will use this also for comments and other? But then this should go out of the Post and be a stand-alone function.
imports should be at the top
Having the semantic stuff makes it much easier to filter. :)
Alright, didn't know that the standard library has an equivalent print function.
you can pull down his changes and try it out yourself.
-('unstuck', 0, 'Syntax: .cheat status \n\nShows the cheats you currently have enabled.');
+('cheat status', 0, 'Syntax: .cheat status \n\nShows the cheats you currently have enabled.');
You can replace the next 4 lines with just one:

VisitNearbyGridObject(radius, searcher);
drin
What is chenv directory for?
Finally, I will stick with Zend convention (that is, the adapter have a name of "where does the values come from", so ObjectManager).
The election information would also need to be accounted in the Expense table, right?
`viewHeater.getLocations(percent, offset)` makes more sense to me as a signature.  That way I know that I'm asking for `x` % of the locations that the heater knows about, starting with chunk `n` (again, probably 0-based).
Why do you sometimes open the brace on a separate line, and sometimes not (see line 170)? I suggest you never put the brace on a separate line (but that's a personal choice, just be consistent :) )
nit: start, stop or restart a selenium grid hub
should raise 405 method not allowd
Okay, that's what's left for later? Seems okay then.
`delete lock_manager()`?
no, if the Release output diectory is there, it should be OK.
Minor nit, but let's move the opening " over to the right in the comments. Let's have the closing " on the same line as the comment (so not on a new line).
docstring standard

est_vect: ndarray, (n,k)

``ndarray`` if you explicitly require numpy arrays and not other iterables.

I prefer numpy shape notation, (n,k) but I don't know if this is numpy doc standard
did Greene use 'R' and 'AK' ?  just to understand this.
is this used by the route generator even if the route does not have such a parameter?
Wouldn't it be better to do smth like `code_point = '0' * (4 - code_point.size) + code_point if code_point.size < 4` instead of looping and maybe extract that `4` into some constant with a meaningful name?
Protected members should no longer be prefixed with an underscore.
pushed an edit that addresses this
Usually in these kinds of files, we put a comment in about which commit version was used to make the existing data file.  Probably worth doing here too.
I like this idea.  I think it's a little more complicated than your above examples because we'll be passing a python Shear so we have to get its ```_shear```, i.e. the code would be ```self.SBProfile.applyShear(args[0]._shear)```.  But still, it's a good plan, so I'll go ahead and make that change shortly.
From the previous, call this CustomTracking <something>
缩进丢了
That's a fine idea, will do
I know, I was just saying in a long-term project it would be silly to have two foreign keys, only one of which would be used at any time (either user or campaign_friend), but I'm not suggesting any change
Hi Aled, these latest changes are already in the master of Brooklyn-central?

If so, I'll try it without my fix.
There's probably stuff needed around closing the ssh connections if we're serializing/moving the location object, but presumably that code wouldn't be going here anyway.
have removed this -- my original thinking was to query isComplete to know whether the policy had values from all sensors, so if any member hadn't published we would know the average was a "partial".  but of course in the downstream policy (e.g. scale out) we only look at the _output_ of this enricher policy and can't query whether it was complete or not.  we could expose a "onlyPublishWhenComplete" sensor to suppress publishing of averages when some sensors haven't published but that started to seem too complicated.  instead we just rely on the resizeStabilitzationDelays in the auto-scaler which are good enough.

at some point we might have an option to emit another sensor if averager doesn't get a value from some sensor in some period of time, and then it might be okay not to publish if some things are busted ... or that might be somewhat brittle
Thanks again David. Well played, but, I don't know why, it slowed down a little bit ... It performs around ~30ms with the same benchmark.
Could use Option(defaultValue) orElse ... getOrElse ...flatMap(...)?
We probably should set the exception code here too - just always 0 isn't very useful.
Note: do not merge until after 1.3.2 is released.
"#footer-links a:nth-of-type(1)'
wouldn't it be readable this way?
Is this reasonable? Maybe not...
I tested with ffmpeg 0.10, and `-itsoffset` has no effect when choosing input offset in the invocation you're using here - it's always the same frame no matter what number's used. I believe this is because `-itsoffset` is not a seek option, it's a timestamp offset option.

The time taken while using `-ss` depends on whether you place it before or after the input video. It's very fast when placed before the input, though that doesn't work with every format.
This is not entirely clear to me---what is this doing?
as discussed previously connected app might not be appropriate place as this called when new connection is made from browser
Very nice!! Removed from create and update ;)
I think $request->get('top_groups') should actually work as well for retrieving data from a $_POST variable
`!user.is_a?` rather than `! user.is_a?`, in this case doing `not` instead of `!` is also good, you might also consider the guard style here (following the "Flat is better than nested" mantra from Python)

````ruby
def self.approved?(user, role)
  user = user.name if user.respond_to?(:name)
  raise "<message>" unless user.is_a? String
  @@roles[role].include? user
end
````

which is a little more idiomatic
Set mysql username
Ok, thanks. Could you please edit the commit message to mention the issue number you fixed? E.g. "fixes #607"
Табы + пробелы?
Не ругайся в коде.
А это что такое?
\u0005 is java's way of encoding special character, which cannot be accept in web service. In Web Service world, it is widely adopted to use Numeric Character Reference (http://en.wikipedia.org/wiki/Numeric_character_reference ). Unfortunately, decode numeric character reference is not provided in standard Java library. We can consider doing that for the customer in the convenience layer. 
A new issue filed https://github.com/WindowsAzure/azure-sdk-for-java/issues/142
`failed because of the following reason:`
missing whitespace after operator
this is a slightly lazy way out, since you know what you made and where, you could technically just add it yourself, but ok
yeah definitely, i think that would be wise.

but it brings up a good point - that even videos on your disk should allow
for Asynchronous loading (this is what I meant by synchronous in the first
comment)

I think for this to be correct isLoaded() should always be true after a
call to load() unless "SynchronousLoad" is OFF, and it should be ON by
default.

"SynchronousLoad" should apply to both URLs and file assets.

On Mon, Aug 27, 2012 at 10:40 PM, Elie Zananiri <notifications@github.com>wrote:

> In examples/video/osxHighPerformanceVideoPlayerExample/src/testApp.cpp:
>
> > -		ofSetHexColor(0xCCCCCC);
> > -	}
> > -	else{ofSetHexColor(0x000000);
> > -	}
> > -	ofDrawBitmapString("keys <- -> frame by frame ", 190, 340);
> > -	ofSetHexColor(0x000000);
> > -
> > -	ofDrawBitmapString("frame: " + ofToString(fingerMovie.getCurrentFrame()) + "/" + ofToString(fingerMovie.getTotalNumFrames()), 20, 380);
> > -	ofDrawBitmapString("duration: " + ofToString(fingerMovie.getPosition() * fingerMovie.getDuration(), 2) + "/" + ofToString(fingerMovie.getDuration(), 2), 20, 400);
> > -	ofDrawBitmapString("speed: " + ofToString(fingerMovie.getSpeed(), 2), 20, 420);
> > -
> > -	if(fingerMovie.getIsMovieDone()){
> > -		ofSetHexColor(0xFF0000);
> > -		ofDrawBitmapString("end of movie", 20, 440);
> > -	}
> > +    if (fingerMovie.isLoaded()) {
>
> One thing to note is that I had to wrap the movie drawing code in an if
> (movie.isLoaded()) { ... } statement, to give the file time to download
> during the first few cycles.
>
> Do you think it would be better to just create a separate example for URL
> loading?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/obviousjim/openFrameworks/pull/1/files#r1464150>.
>
>



-- 
- James
monkeypatch does the same thing as mock.patch.
does both what? it's not clear to me what this test actually does :)
Можнно еще явно передавать переменную `undefined` в параметрах замыкания, не задавая ей значение. См. https://github.com/bem/bem-bl/blob/master/blocks-common/i-bem/i-bem.js#L6
I see now that `groupBy` uses the default map, so it doesn't keep insertion order. But I guess you could still do something like

````
lines map { line => line match {
    case Offset(off) => (off, line)
    case _ => ???
  }
}
````
and not re-implement looping logic.
A docstring should start with a single line describing the class, so something like:

    """
    Kalman filter/smoother for linear Gaussian models.

    [Long description]
    """
On Sun, Oct 14, 2012 at 11:24:34PM -0700, Peter Prettenhofer wrote:
> do we need '\r' for windows new lines?

I don't think so.
+1

This would be very helpful IMO.
On Aug 9, 2012 12:43 PM, "Satrajit Ghosh" <notifications@github.com> wrote:

> In sklearn/ensemble/gradient_boosting.py:
>
> > @@ -525,13 +525,11 @@ def fit(self, X, y):
> >          self : object
> >              Returns self.
> >          """
> > +        X, y = check_arrays(X, y, sparse_format='dense')
>
> could we use sparse_format as a property of the class? so that it can
> also be used to generate the docs to indicate which estimators/transformers
> supports sparse and which don't?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1003/files#r1341514>.
>
>
I don't think it's right as the second term is not flipped. It should be the log of:

http://upload.wikimedia.org/math/a/7/b/a7b0994a598b99d7599780e61feda718.png
ups, I meant the other code. the one with final at the end. Next time I will only push the one file I am working with. Sorry
hmmm, also, it'd really be `Room.names` :)
Переименовал $type в $method, $path теперь со слешами.
I completely remove the caching feature.
Please check with Appcelerator about this. I think you should be able to provide an offset and no length to fill the rest of the buffer.
These should be in the initialization list
Vérifier les :doc:`prérequis <Prerequis>` et procéder à l':doc:`installation <Installation>`.
what is `'?' not in referer`?
"Pros:" rather than "Pros :"
Just noticed - It's not generating hash dedups, because no ,HASH ...
>I think the attribute 'num' is because XML does not guarantee order, so external programs could mess up the list and it'd still be correct.

Ah yes, that will be why
you can't call this function (queryKeySize) if the selectInterface fails.

If selectInterface fails then you can't extract the key.  (But nothing to worry about since only on old workunits)

I should have moved them into a separate interface to make it clear/safer
(You may need to at least make sure there is a NULL entry following the last one you removed)
no reason.
I tend to add spaces before the function name.  Probably did it by accident.
These look like they should have more indentation.
The "standard" is three lines. 

1. the regex
2. the view function
3. the name
Cluster name doesn't change. So, there is no reason to pass it in as a parameter.
Actually, `windowUtils.xulWindow` *contains* more sense! `windowUtils.xul(window)` is far from being explicit.
`windowUtils.getNsBaseWindowFor` is the 100% explicit version of it.
Again, probably just me... but I hate the magic dollar-sign-followed-by-punctuation variables.  It doesn't look like "require 'english'" creates an alias for $".  I almost wonder if we should just create our own alias for that somewhere.
re format stuff, i think it is so rendering the partial works.... it hink you can specify 
```
render_to_string :partial => 'people/person.html.haml', :locals => hash
```
@shacker can we remove the {{scheduled}} and the other ones beneath?
`(not (:death-year author))` ajaa saman asian.
Maybe change to "två separata strängar: "herp" och "derp"".
typo => mgirate => migrate
I think the code is more readable in this. If it is not a requirement I would leave the code this way. Opinion?
occurrence; full-stop at end of sentence.
We [cover locale redirection really well](https://github.com/mozilla/kitsune/blob/master/apps/sumo/tests/test_locale_middleware.py), I don't think you need to duplicate that here.
Well, upon further reflection, promises only resolve once, where this thing resolves multiple times.  I'll think about it.
Leave it if that's what the formatter does.
since u are using the target, why not make the target the argument of the function rather than passing the whole payload. This way u are making sure that the intent is clear, which is checking if this is the target of the event.
then when calling this method you can call it this way: handler.proceed(payload.target)
Also rule 15, baseline 3.1.1, etc.  (Annoying, I know, but we gotta do it.)
Related to my other comment can we consolidate this generated class into a single method?
We can probably just return contentWidth here instead of using max since you already check for it in the if statement.
а какой прогамой проверить валидность? или посмотреть как это будет герерится?
`this`
After form save we should redirect somewhere: possibly back to the list view of runs? Also it would be good if we could add a message to the effect that the comment was saved.
The `events` parameter is not `const` in the C prototype so please don't make it `const` here.
just extracted duplicated code
This worked fine to show me the method was deprecated when using it. It warned me in the code using the library. It could be added to if you think it's better, sure.
You shouldn't set Velvet as build dependency if you just need access to it's sources.

Just include the Velvet tarball in `sources` like you did in https://github.com/hpcugent/easybuild-easyconfigs/pull/52, and then use something like this (untested, but should do the trick I think):

```python
velvet_src_dir = [d for d in os.listdir(self.builddir) if d.startswith('velvet')][0]
```

And then adjust the below to

```python
self.cfg.update('makeopts', 'VELVET_DIR=%s' % os.path.join(self.builddir, velvet_src_dir))
super(EB_Oases, self).build_step()
```

Calling the `build_step` of the parent is nicer, e.g. it will also enabled building in parallel and maybe other cool stuff done by the `ConfigureMake` easyblock, so why not use it.
This should be WFK, and this change needs submitting to Aerogear
I though we said to use constructor injection?
Won't this essentially do:

````
SELECT DISTINCT 'sourced_id'
  FROM v_curriculum_learner
  ....
````

Rather than the desired:

````
SELECT DISTINCT sourced_id
  FROM v_curriculum_learner
  ....
````
I wasn't sure what to do with this file, this was created before we switched to klein routing, I was going to use just the PSU Controller. This entire file might not be neccessary
`$key`/`$val` pairs need to be sanitized.
If you **want** to do this, it probably belongs in a class file so the code isn't duplicated across a butt load of apps.
Why not camel case?
I'm working on the release config again. Will this format work better? 

I think I've deployed so many tries to day I might have hit a ceiling b/c I'm getting a strange error. Anyway, I've run out of time today, and will pick this up again tomorrow.
This allows, potentially, for File["${apt::params::root}/sources.list.d"] to be defined outside of the apt module. See further comments on line 61
We are not authoritative on this. It is linux's kernel specific code, and the documentation doesn't belong to us in any ways.

The purpose of this code is to reflect what is specified in linux's doc.
SUGGESTION: Use opeartor/ provide by boost::filesystem::path to ensure native directory separator support.
You only want to print the group header once per group.
Well, we always had method that did multiple operations. In fact almost all of them are and always have been. 
I guess it assumes that these operations are going to be executed as part of a transaction scope already defined by the user. So I am not sure how to answer that since adding it certainly changes the original design (which may not be such a bad idea after all)
Artem, can you please be more specific. I don't want to guess what you mean.
Looking at this code it pretty much says that it can only ever be a Map or a List with no possibilities of transforming it to a different structures. Perhaps enumeration of types to which we can add in the future.
Needs to be moved up.
really, anyway it's not used anymore.
Please can you remove debug.
I removed most of the text from the second example because is exactly duplicates the previous example.
The idea of the second example is to show usage of a different syntax, but the parts that I removed stay the same.
I believe the sentence in the beginning of the example makes it clear enough: "It can substantively be the same as
 the array configuration from the previous example."
Thanks for the tip, I knew this was possible in the back of my mind, I just forgot to go back and fix it.
Poor Rory. His Github profile link hijacked.
Александр, так нормально?
Если в приложении в другом месте подключается jQuery через стандартный механизм, то Yii поймёт, что jQuery уже зарегистрирован и подключить его только один раз.
that's
try to unit test with mocking
Empty line too much.
head is not needed (gets set automatically by Django's View class)
What does nodefault mean from a user's perspective? When is it needed?
You could remove the `\` in the line you did not edit. For PEP-8 compliance, I'd prefer `run_in=Scheduler()...` and `loops=-1` without blanks, but others might disagree in this case.
Can we not name it e.g. SpecializedPickup and leave out the "allowed:" part. We can then just have something like this:
```yaml
- SpecializedPickup:
  - BUILDINGS.MAIN_SQUARE
  - BUILDINGS.PAVILION
  - BUILDINGS.VILLAGE_SCHOOL
  - BUILDINGS.TAVERN
```

Since there is no "forbidden" list, I think this makes it simpler.
Use bool(displayed) here
Missing javadoc, this is important because it is the api exposed to our users
In the case of the client closing the stream:
[SchedulerProtocolet|INFO|4:04:23]: Shutting down...
[SchedulerProtocolet|INFO|4:04:23]: Terminated
[SchedulerProtocolet|INFO|4:04:23]: Protocol successfully ended
opacity bede
opacity : 0.5 bedi doros mishe
Thanks for taking the time to add some tests.  I think the only additional tests I'd want to see is around some of the edge cases.  Maybe add a test that, if you pass in nil data, you get back a nil response.  Another interesting one could be to verify that if you send in a limit that's greater than your prescribed ceiling, that the ceiling limit is substituted instead.

Overall, it's good though!  I like that you've mixed in a combination of values and non-values in your existing tests, to prove out that edge case functionality.
The current behavior is to center on (0,20) with z=1, so this fallback mimics that. The only other place this function is called is when the user clicks 'center map on my location' - but when he does that, the implication is that he would allow the browser to detect his location.
Nooooooooooo. This is forces to fall more then 90 tests in oracle and firebird :(
I know we discussed the possibility of having etag tag a decorative/assertion-checking value.  In this case, the stag's aren't as far away as some of the later cases, but I think it would be really useful.  Or if not the extra argument, then a comment.  (All instances.)
Is the link supposed to be #setResult(Object) ?
missing standard copyright header
copy past error :-)
Moved line from no.26 , Git being strange
move it too
This should be an hop extension
Ok. Just seems weird to write code for something already provided for us with wrench and since we are already using wrench in this file, it makes sense to use it to "collect" files as well. Not a big deal though :).
wrong license notice
I think there is one other fix that is needed here. If you look at the remove function, it does a webworks.exec("remove") ONLY if there is 0 callbacks in the array. That is done because we should only stop listening to the event when there is absolutely no clients listening to it. So similarly, I think in this function, we should do webworks.exec("on") ONLY if we are registering the first callback function. If it's adding extra callback, we don't need to start listening again, because we'd already be listening. So we should add an if-statement to wrap around the exec call.
On the former, sure. On the latter, [PEP 257](http://www.python.org/dev/peps/pep-0257/) says otherwise.
Edellisen tehtävän `divides?`-funktiota olisi voinut käyttää apuna
Tässä voisi käyttää äskeistä tehtävää `divides?` avuksi.