labofsahil/github-event-dataset-2012 huggingface dataset:
One option is, instead of using the record, we could simply change the first element of the tuple returned by :dict.new to be `Dict` (or `Dict.Record`). This way we avoid creating a new unnecessary tuple (although creating a new tuple may be even faster than changing an existing one). Another option is to simply set a global reference, like this:

    refer Erlang.dict, as: HashedDict

This way we wouldn't need to do any conversion. :)
Win what ? ;-) Ok this issue might be valid (I don't have any Windows PHP setup to test).
Since, this combo loader is done only for YUI files, "hello.." is not acceptable. It is done for YUI files only because we currently miss a system to properly declare custom YUI module so that it can be asynchronously loaded and so that it plays well with our override system. The combo as it is done now, is only a first step to improve page loading time since we more and more use YUI.
well, it should take container in constructor actually, it is a hard dependency. but then I don't need to implement ContainerAwareInterface I guess.
Fixed in https://github.com/ezsystems/ezp-next/commit/0bad0be4f43e35378241be3c500b28e65e30f36b
More like the error message used bellow maybe?
Fuck github. Sorry for the dumb comment :p
fix `sanityCheckPaths`, don't leave commented out stuff
It is the default for run_cmd. the output is logged in case of failure, I'll just remove it and use the defaults?
This could be obtained in `oauth_utils` too, same as line above.
It's more overhead and it do you really need to filter on whether, for example
Hey @readevalprint does this look right?
no newline here.
This test doesn't need to use the `plus()` mechanism, it's 100% synchronous.
this isn't truly an optional attribute, as it defaults to down in the code.
@sarken - thank you, the zero was there, but it was on the wrong side of the decimal. I thought it was fixed before I pushed, but apparently I didn't write the file out when I changed it.
b/c if it was that important you would have changed two lines of code instead of writing a comment.

The switch is clearer of intention.
I guess this is "create_date", which is our standard name for that kind of functionlaity..

Also use column type date_time.
Doesn't this cause duplicate events as all children have the child event bind "once" call set to fire?
Which node references?
This is a minor nit, but keep in mind that ```/SeriesListing.asp/i``` is a regular expression. It will match ```SeriesListing.asp```, which I presume is what you want, but it will also match ```SeriesListingAasp``` or ```SeriesListingXasp``` since ```.``` matches any character. In this code, you're unlikely to get unintended behavior, but generally it's a good idea to escape the dot (```/SeriesListing\.asp/i```) or to use something like:
```javascript
var lowercaseURL = url.toLowerCase();
if (lowercaseURL.indexOf("serieslisting.asp") !== -1  || lowercaseURL.indexOf("itemslisting.asp") !== -1 || lowercaseURL.indexOf("photosearchsearchresults.asp") !== -1) ...
```
Why sleep a bit? Also Thread.sleep(A_BIT) perhaps
Spaces???? why????
I'd like to see `prefix = '#ctl00_ctl00_ctl00_MainContent_SubContent_SubContent'` to reduce line length.
лучше не называть одинаково с функцией
что это вдруг он стал виртуальным?
If that is the case you can leave it alone.
Shouldn't this be `if (code > 0)`?
I meant 050f634
`path.existsSync` causes deprecation warnings now. It should be `fs.existsSync`.
A better way to check for the type will be to use isinstance()
This seems racy; why don't you iterate directly over the directory content rather than checking it twice?
One enter too much.
Using `IO.getFileListForFolder` now.
On second thought, don't bother with this.
You may want to move this inside the ``try`` block, or the resource may never be closed.
Yes, that was intentional. The current timestamp is set automatically in Subscription::add() now.

The previous implementation would allowed users to create a subscription with arbitrary data. For example one could create a subscription with a date way back in time, causing the wiki to create a huge change mail. Not sure if that's a real problem.

Was there any rationale behind having the data supplied from the frontend?
Yes, we could check that when we fetch the image.
Currently we only check the getRowDistanceByte, but we could check the buffer pointer too:
https://github.com/shahriman/TuttleOFX/blob/dev_swscale/libraries/tuttle/src/tuttle/plugin/ImageFilterProcessor.hpp#L38
handig om misschien boven deze functies een comment te schrijven die uitlegt wat dit precies doet.
Save library names in a property, leave PHP to handle file includes.
@slebedev и, если уж на то пошло, то заменить != сделать отличным от текущего.
Если "А" стало "А" - то ячейка не изменилась.
на самом деле мы как бы и создаем свой конструктор
у меня компилятор ругается на эту строчку. 
Ничего толкового больше сказать не могу... у меня сортировка вручную была сделана.
or "test_support_files", I don't remember the decision in the meeting.
Just to be clear:
```
// Collect all receiver's `next`s into a single NSArray.
//
// Note that this signal will not send any `next` if the receiver does not
// complete.
//
// Returns a signal which sends a single NSArray when the receiver completes.
```
?
This will actually not work. My bad... Please disregard.
If you make these just `var` it will work for both cases.
I included the metashare_test.db in the init_data directory.
The problem of generating metashare.db, as well as schema.xml and updating other files in the init_data, is known.
I will try to add this in future commits.
Instead of taking just the first episode found, please loop through all the episodes returned, so they all get set back to wanted.

For example, this NZB failed for me: Transformers.Prime.S01E01-S01E08.720p.WEB-DL.DD5.1.AAC2.0.H264-BluZilla.nzb, which left episodes 2-8 still in snatched status instead of changed back to wanted.
if not self._preloaded:
It's only busted when that method doesn't exist in `core/lib/refinery/application_controller.rb`, and the logic isn't there to stop the redirect from happening by calling this method. That was the whole point of this PR really... To make the thing the guide tells you to do actually work, and to make showing the welcome page optional via the core configuration.
I'm not sure this should be defined here. It'll get defined every time `_send_request` gets called and it doesn't seem to be using anything in this closure.
This is wrong. It makes this listener impossible to use with others authentication listeners as the exception will prevent them from running. If you cannot authenticate the user for this request, you should simply do nothing.
不过这些头的处理过程可以使用<link id="proxy_ignore_headers"/>指令忽略。
thus没翻译，上下文断开了。
本来这里是有“因此”的
I don't think discard is a particularly common use case, I don't really think we need this. If we get demand for it we can add it later.
` = false`
Validation should take place in the model instead
the data structure is supposed to be like this:
array(
    array('col1'=>value, 'col2'=>value .... ),
    array('col1'=>value, 'col2'=>value .... ),
    array('col1'=>value, 'col2'=>value .... ),
    ....
);

so array_keys($this->data[0]) will give you the name of the columns
Another one. ;)
The timer is winded when you first load the SMART connect lib (just before launching the channel's constructor). You can call `SMART.fail()` at any time independently of the timer. Nothing is tied to `onload`. So, overall I think that 2-5s is about the right range for this timeout.
Nit: Can you move this down one line? :grinning:
Do we want an IntMatrix base interface?  Otherwise the return types are always double and unnecessarily cast to that type. :(
Now uses nanos
@mmarkus, this seems like a new feature, do we really need it?


On Tue, Dec 11, 2012 at 8:37 PM, Mircea Markus <notifications@github.com>wrote:

> In
> core/src/main/java/org/infinispan/configuration/cache/LockingConfiguration.java:
>
> > @@ -53,6 +55,15 @@ public int concurrencyLevel() {
> >     }
> >
> >     /**
> > +    * For non-transactional caches only: if set to true(default value) the cache keeps data consistent in the case of
> > +    * concurrent updates. For clustered caches this comes at the cost of an additional RPC, so if you don't expect your
>
> @danberidei yes it does make a difference for local caches. when this is
> explicitly disabled (it is enabled by default) we don't add a locking
> interceptor to the interceptor chain. So concurrent updates are very
> un-predictable. I think (this would require more though though) that this
> would only affect consistency, but I really want to discourage people doing
> concurrent updates on a cache that doesn't have the locking interceptor in
> the stack as this would open a can of worms.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1520/files#r2382476>.
>
>
Author should be you ;)

And since version should be 5.2
Why are you removing this? It's a valid way of doing things.
This break is a little untidy, perhaps replicate the breaking on the new lines 2088--2089?
Note a subtle difference between the macosx backend and most others for the show.  macosx backend uses self.show() instead of self.window.show().  I am also concerned about the note saying that this is needed to make ginput() work correctly.
Note, this may be why you need to rebase, as we are deprecating the usage of nxutils.
I can't help but see one word with this variable name, I think the variable name ```ypadding``` would probably make things clearer (I would personally go for ```y_padding```, but that is probably a bit inconsistent with other variable names in matplotlib).
Incidentally a dev for Socorro recently updated this method in socorro-tests to check for jQuery's existence and that .active == 0 to work around problems where on pages where jQuery might not exist.
I was wondering why to remove this one. I guess this is because `setImgSrc` also calls `stopObservingElement` in that case.
Can you move the catch to this line?
Yep, in 1.4 it'll just be `(map int xs)`. @robinkraft, once we get forma-clj riding on 1.4, I think this little change will do it and we can merge this thing.
Hey, do it like I'd said and make a separate function, then have

```clojure
(def text-map (resource-seq "admin-map.csv"))
```
@bp323 Extra space after `)`
that's coming from libocas, but i can change it in shogun
should be rather section
Yay. ;)
Changing it to map:: instead of Geocoder:: shouldn't be a problem :)
Added to #218.
If a variable product is hidden and one of its variations with enough stock ($_product->is_visible==true) & enabled (status==published) has been added to the cart, the ( ! $_product->is_visible ) will return false.

So - we enter "else if":

To disable the hyperlink, we must also run is_visible() for an instance of the variable product, not just the variation.

That's what happens below.

I hope this all makes sense. I'm pretty certain I'm not out of my mind... well... who knows!
Bundled or combo'ed items can use this to add special styling info. Same below.
How portable is this?
Forgot a space ; )
I think this section is unnecessary, as developers will already have the dependencies if they can run Freeseer.

I'd probably make this an "Introduction" section and very briefly describe what is used for testing (i.e. `unittest` and `QtTest`) and why they were chosen.
Are you sure ``$validator->isValid('b')`` has the correct type as "Boolean" is not a sub-type of "Boolean"?
Are you sure ``null`` has the correct type as ``null`` is not a sub-type of ``array``?
Are you sure the class "Zend\ModuleManager\Feature\AutoloaderProviderInterface" actually exists, or did you maybe forget to add a USE statement?
Is this fall-through intended? If so, this would be worth a comment, no?
Please remove first `/`, as with it API will generate wrong url.
please remove :)
Yes. Derp. I seem to keep leaning towards browser compatibility even when it doesn't apply.
Man I hate that word. We really can't think of _anything_ better?
you can leave it under "A little more security"
Another Boring List
-------------------

* Okay I guess I want a new list.
* Yep this is a [list.][16]

  ### This is an interesting case, because it could be highlighted as markdown (since it is)

  * Naw, I want a real code segment, showing markdown syntax.
  * But this *is* a code segment!
  * Well this is an interesting case.
Agreed. Spent some time trying to find a decent solution. That's the best I could come up with given the constraints (and the WebKit ugliness). Open to suggestions.
not sure why you added this here .. actually this parameter should be removed from all the other methods since it doesnt appear to be used
I don't think `<a>` tags have a `name` attribute in HTML5. I would just use `id`, unless this is one of those "Yeah, but then all the browsers in the world explode" scenarios.
This is following conventions defined in REP 105 (http://www.ros.org/reps/rep-0105.html).
No need to declare these variables.
We use `'` for constant strings, not `"`.

We write space before `()` in anonymous functions.

List all arguments of the function, even if you do not use them. Because in this way somebody after you will maybe get some idea what is possible to do more. In this case, `event` is passed.
Ehm, focus event.
p is not needed
Probably should be `mem[1:]` for all three.
Makes sense. I did not know it was an enforced rule in zmq, my bad for not having asked before.
No need to write a test suite since the top level script does this for you. Check ts_python.sh in the top level appscale directory, or run nosetests in the current directory to run all the unit test in a given folder.
Is it possible (license-wise) to include this file in the bio2rdf-scripts repository?
What about using zipgroupfileset and removing merge external?
s/optionnaly/optionally/
Oh, so this means that we really need serious testing on the performance side: hope it's not taking too much time to watch this all the time.
The rest looks good to me.
Save => Go
Would like a newline after this.
`Grid::ClearCell` was added to support the `nil` stuff. Since the other `Clear` methods were exposed to Lua, I added this one too.
partway

write a _nbinom_logpmf function with your parameterization (this method as function with exog instead of self in signature).
Call this standalone function then directly from your _ll_nb2, _ll_nb1, and _ll_geometric . You don't need a _ll_nbin then. AFAICS
I don't like this code repetition much. One possibility is to use the change in the if structure as in another method, or use a default cache (more separately)

also this will be calculated repeatedly if both bounds are None
OK, I can see you model this according to the Java API right?
Alias as `ZendDb` (this is what we do elsewhere in the framework as well).
Yes, I think you're right.  Sigh!  :)  I have to confess that the difference between the two wasn't really fully clear in my mind before I started this task last month... I'll fix.
should use ${RUBY_VERSION} macro here as well
Is there a reason for 1/0 here rather than True or False?  Or even, just passing the request.user.is_authenticated() value directly as 'is_logged_in' on line 77?
I think keeping one photo is fine. This photo is just like an avatar photo of user for identity. I dont think we want a user to have an album of photos.
this was overridden just to prevent it from crash-and-burning if there are errors instantiating any of the classes.  have added comment to that effect.
Why not add the line if there is no recipe-config yet defined?
Can't we use the following?

 HttpTestUtils.assertContentEventuallyContainsText("http://" + drupal.getAttribute(Drupal.HOSTNAME) + "/index.php", "Welcome");
Let's keep it without *datalock* term, I don't think it would help understanding this post.

However, I'm interested in the citation reference. I understand the problem, but never heard of this term.
This one also misses the "void"
"we customize"
Other parts of rspec share error messages by redirecting to another method. Something like `raise message_must_be_string unless String === message`.
Requires ID ;)
On 28/02/2012 17:47, Ben Francis wrote:
>>
>> -<div id="filter-controls">
>> -<span class="button" onclick="Camera.toggleFilter(this)" data-filter="fGB">blur</span>
>> -<span class="button" onclick="Camera.toggleFilter(this)" data-filter="fCMS">saturate</span>
>> +<div id="controls">
>> +<a id="switch_button" name="Switch source"></a>
>> +<a id="capture_button" name="Capture"></a>
>> +<a id="gallery_button" name="Go to gallery" href="../gallery/gallery.html"></a>
> I agree that a relative URL is wrong, but this is required because of the temporary way we're hosting all the apps on localhost and I think that directly calling the homescreen app's WindowManager object would be even worse. Eventually this should either be a hyperlink with an absolute URL to another origin which the window manager automatically opens as the Gallery app, or some new kind of intent for launching apps. Alternatively the camera and gallery could be different entry points to the same app as previously discussed.

As I said on IRC, WindowManager.launch is bad too but this is the Gaia's 
way of launching app for now. So let's use that and open a followup and 
a discussion on the mailing list for any behavioral changes that you 
don't like. Or even better, implement a proto-intents API that we can 
use! (but in a followup! ;))

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/andreasgal/gaia/pull/672/files#r497061
If we put the timezone in the session, then this would just be:

`self.context['request'].session = {'timezone': '<the timezone>'}`
ajust -> adjust
changed all the lines to have the same indentation.
Delete the subtypes, there is an easier way to do it.
You will need to add the following at the top of the file:

```ruby
class ::Chef::Recipe
  include ::Openstack
end
```

so that calls to endpoint() are found.
@yuhunglin & I agreed that it feels wrong to support one & only one logged-in user who is injected outside the code that checks the ID. So he's going to tinker to try to get more of a traditional "login_as" function into spec_helper.
Hmmm, I'm sure we've talked about this kind of behaviour, Grant.
Good point. I'm nervous about just changing this since the DB is parametrized by context, and it would be a bad thing to write to the wrong DB if the code was ever refactored. Do you have a better idea here?
@zacc @bebef1987  will do so, was thinking of doing this in the prev commit but waited to see the comments on the string return.
[pep8](http://www.python.org/dev/peps/pep-0008/) says it should stay: `dense_output=True` (in kwargs).
I can do that but I feel then the example becomes very contrived as then all values have to have their maximum value, i.e.
``[1, 2, 3]``.
Maybe I'll just add another data point...
merci. I always forget that builtin.
It is for default values, in `pxd  files. Don't ask why it is like that though ;)

http://docs.cython.org/src/reference/language_basics.html#optional-arguments
+1, this would be consistent with `SGDClassifier`.
If you name an attribute with a trailing underscore, you have to put it in back-ticks (double looks better). Otherwise sphinx will complain.
`more_opts` can be refactored into a method - we try to avoid setting variables inside `if` statements.
I'm glad you remembered this! :+1:
how about DDRC |=  _BV(PORTC2)?
As it stands after merging this and https://github.com/D-Programming-Language/d-programming-language.org/pull/229, we'll have an isPOD trait that lies, and documentation that lies. It seems perfectly reasonable to me.
Still awaiting confirmation
This will break functionality within String formatting methods.
To reduce repetitive code, this could just be `else if (strcmp(longopts[opt_index].name, "ignore") == 0 || strcmp(longopts[opt_index].name, "ignore-dir") == 0)`.
There _are_ two call_S_ to make:
So, on the examples below, the rows with activity were removing more fields than they should (which?) and this will prevent it?
I totally agree. 'lookupSuperFile' also has some cache logic that will need changing (and it's much clearer now), but that's not for today.
It fixes a crash in standalone mode
Is that needed? I would assume that django does exactly this by default.
missing space between Module and for
Is it really needed?
In other places I wrote a better comment:

https://github.com/mozilla/addon-sdk/pull/312/files#L5R408

would similar be sufficient ?
根据文档的规则，当有中文和英文夹杂的时候，英文的左右两边都要空出一格，比如『各个Common Lisp实现』应该写成『各个 Common Lisp 实现』，如果有括号包围则除外。
That's because it's used as a predicate in the connection string settings verification. Imagine something like

    List<Predicate<bool, string>> predicates = new List<Predicate<bool, string>>();
    predicates.Add(Util.getIsBase64Encoded());
    predicates.Add(Util.getIsValidUri());

Then somewhere in the code this list is used to verify some elements.
It just seems way too verbose

This code should do the same in less lines of code

````
                var args = (cb.apply === true && cb.args && Object.prototype.toString.apply(cb.args) === "[object Array]") ? cb.args : [cb.args];
                var resIndex = (cb.resIndex === undefined) ? 0 : cb.resIndex;

                if (resIndex > -1) {
                    args.splice(resIndex, 0, res);
                }

                try {
                    return callback.apply(scope, args);
````
We cannot really merge this file in to the main repo; sorry about that!
Can we avoid creating this which duplicates the method on line 32?

FD is slightly different in the way it uses regions.
Why, though?  There's no reason someone couldn't have $Id$ on a line that doesn't fall between lines of ##.
Yeah I don't think this is supposed to be a debug line or anything.
Ah, didn't see the exclude for gemcache.  Still holds for other third-party libs, though.
Yep you are absolutely right - I intended to this but forgot about it will change that :older_man:
So, the source file will remain in the patched state on the server until it is reloaded form disk, or is patched again. Ensime automatically reloads from disk before any command that requires up-to-date state (unless explicitly told not to, which is now the case with swank:completions!), so server commands will always be executed against the state that you expect.

Just as before, if you undo, or make any buffer change that is not saved, you will be out-of-sync with the server. The difference is that now it is possible for the server to be out-of-sync with the disk.

Does that make sense?
Hah, seriously? Does that just call the parent block? Cool.
I don't think we need process rules for reviewing comments, but I'll keep in mind your personal preference.  I was slightly mistaken as it's just the @see tag and not seealso.  I updated the code to reflect this.  I also removed the try/catch since as @shunter pointed out they are not needed.
Isn't this JulianDate UTC not TAI?
you should create a variable and re-use it instead of calling drawables[i] twice.
YES == YES ?
Ugh! This is vital!  Thank you for spotting it, Vishal. Fixed.
Separate vars should go on a separate line
Should fill in (at least purpose, maybe not description)
Why did you delete these?
I've only set up Travis once but I imagine this should reference the office repo:

    (https://secure.travis-ci.org/copycopter/copycopter-server.png)](http://travis-ci.org/copycopter/copycopter-server)
[type=text] is now redundant, isn't it?
lets try git://github.com/railsware/js-routes.git
Yes, it's a matter of taste and the code can be re-written with if else very easily but the logic won't change anyhow. The if else version would be something like this:

	if(!empty($section) && !empty($page)){
		$markdown_file = $section . DS . $page;
	else {
		if(!empty($section)){
			$markdown_file = $section;
		else if(!empty($page)){
			$markdown_file = $page;
		}
		else {
			$markdown_file = "home"
		}
	}

I write this, this way as I feel comfortable with the ternary operator when assigning to single variable based on multiple condition. If you compare the if-else and ternary operator version, you'll find that the if-else version is two much verbose and repeating "$markdown_file =" in multiple places.

Still, this is not the core part of the pull request and I can further commit with if-else version and send another pull request :-)
fine (this actually works great with the suggestion to make this a class)
Place braces in its own line, it is Phobos style.
Probably just an intermediate change that I didn't put right. I'll check and fix.
What's the reason for this change? Just to clarify?
I would even prefer

<% if (app_mon || comp_mon || stor_mon) -%> 

to creating this extra variable
Do you mean the filename(path) of the file that will be uploaded?
(To be clear: this will depend on whatever the user running the model loader has decided to stick under data/content/.. as those files are not comitted to the repo.)
These are getting long and `true` doesn't mean anything unless you know the format. Maybe we want to switch to an object?

```
"events": [
  {
    "name" : "contact/foursquare",
    "path" : "/events",
    "stream" : true
  },
  // ...
]
```
The reason why we had this is because the earlier version required imagemagick for PDF conversions. As it looks now, this feature would require ghostscript. Thinking a bit more about this, I think it would be a good idea to create a separate group for rghost in the Gemfile as well. As there won't be much use having a default of false here and still requiring all users to install rghost, which supposedly depends on ghostscript. Thoughts?
Again, instead of using regex match, do something like assign class/ID to the relevant able elements and search the content for the patient name
what if the serialized user was new ?
Believe this is a copy/paste from another controllers rescue block, can remove.
This script is loaded after the elements it applies to, so we don't need a document-ready binding.
This line is required, if it's not set the getter is "virtualValue" and not "getVirtualValue".

This is because VirtualPropertyMetadata checks if the first 3 characters of the method name are "get" and if so, it changes that, I guess that for a prettier serialized output by default.
in the DynamicRouter we check ``if (isset($defaults['_locale']) && $locale == $defaults['_locale']) {`` .. why do we check the defaults and not the requirements? its certainly easier ..
'were' should be 'are'
True. Should we just used the shared managed object context on the app delegate?
Тебя не учили, что за C-style cast надо отрывать яйца?
I see. 
It wasn't clear to me that the map can be reached under http://www.geocaching.com/map/ as well.
I will change that.
Great suggestion, started this pattern in the latest changeset
For some (historical?) reason the GC->Thread interface is done through extern(C) linkage.
I think this is where the thread prefix originates and I also think we should get rid of it.
There is no reason why gc.gcx can't directly import core.thread the important part is that core.thread
does not depend on gc.gcx.
Having a typesafe, mangled and overloadable interface is something valuable and if someone would want to write a GC in pure C he could interface core.thread through a tiny D wrapper module instead.
Is X technically needed, or just for organizational purposes? In the latter case I think the function is small enough to just delete it.
... and so on
PipelineNam**ed**ReplyQueuesJmsTests ?
this is a class, not a method :)
I'm not sure why this is in the diff - it is already in the module?

Appart from this oddity - I think it is ready for merge
Is osapi_compute used on controller only?
Placed this at the correct class.
Great FB, good to see you motivated with this.
Just a comment, it is seems some config about whitespaces or something changed and now the commit is unreadable. 
https://github.com/titoBouzout/SideBarEnhancements/pull/11/files
Can you please  clean the commit to keep only relevant changes? Thanks!
xAxis?
Why are you removing the above? This should stay in.
T ← この型パラメータ使ってないのでは?
あと中括弧いらないです。
エラー時のステータスコードも書いて欲しいです。
Put each resource on a single line to make it easier to read.  Also make sure to do the only thing so taht you can't hit actions that aren't there, I kept trying to go to applicants/show which is a problem.
should be form ?
Seems like these would be better in a wrapper class around the listener rather than in here.
NIT: Fix ident
replace these getters/setters after changing the "property" property.  Also add methods for addProperty(String, Object) and getProperty(String) to automate adding and getting individual properties.
Does this belong somewhere more general as well?
mock callback instead (to verify).  mockito.verify
Can drop the comment about the namespacing, as above.
This will iterate all settlement buildings for all building_to_check, as you probably know.

If it's only the regular range check, it's defined by "building1.position.distance( building2.position ) <= building1.radius", so you could use that here. If you know the latter one to be a building, you can also use distance_to_rect, which is a bit faster.
This file shouldn't be checked in. Actually the whole folder should probably be masked out in the .gitinclude
You don't have to manually maintain indexes in a forEach - use the second parameter to the callback, which is the current index:

    endpointsAsList.forEach(function(endpointInfoStr, j) { ... }

Although watch out for off-by-one-errors.
can you make it so that it doesn't match it to any string instead matches to any of three possible strings?

if the strings change our tests should fail so that we don't break apps.
changed in 4ba2bc0c2bd38855ee7662a35885cbd76ba64a52.
Interface methods don't need the "public" modifier.
once you update the JSdoc of getRoute, you will be able to remove this inline type hinting
Translation.where(:name => :en)..
remove logs from this class
make the tag a constant
Marker can be moved also by simple click in map somewhere else.
All comments with word "resistance" were written for selection strings making usual segment tree persistant)))
Consider a more useful message so devs aren't left scratching they're heads if they see this in the log.
I'm not really sure what's going on. But as far as I can tell there are two problems with your code:

1. My ids are something like `#id_content_type`. So the split does nothing and the for loop doesn't even run.
2. Line 43 `contentTypeSelect = $('#' + outstring + 'content_type').first();` fails to work if a `related_name` is set on the foreign key. Using a model like:
    class Blah(models.Model):
        content_type = models.ForeignKey(ContentType, related_name="cool_name")
        ...
can never work, because line 43 expects the foreign key to be named `content_type`. This generates problems if more then one generic relationship is used.
Error handling in asynchronous methods is an interesting story. You'll get an AggregateException when calling Task.Wait or when obtaining Task.Result property. That exception provides you access to all exceptions that happened while executing the call. 

Note that you cannot define your own public exceptions.
Also, while you're at it, perhaps change .split(" ") to .split(/\s/) :D
I agree with the original intent to have some common attribute for all events of who caused it. Writing build logic for a job that is triggering on many types of events would be easier to do if that common named env var was available, my reaction was that uploader had changed name to account, and this was the first place where I saw it was in use.

In short; adding account as a common attribute is very ok by me, but removing uploader from PatchsetCreated is bad.

If you look at a serialized build by the current released version of the plugin you can see that the field uploader is saved in the xml. If I where to start Jenkins with this code in our production environment I would have more than 20K builds that fails to load completely. That was my concern with the backwards compatibility, but I should probably have commented on that at the place in the code where that is changed in PatchsetCreated.java (or am I mixing up commits?)

The reason for the findUploader method was because the code is very lenient about what data that is received from Gerrit (with intent) up until here when it is time to put it into the parameters for the build. There is a nasty NPE being thrown if any string parameter has a null value and it is put into the EnvVar map. So I added that method just in case data was missing, I think it actually was in the snapshot version of Gerrit that I was using at that time. So it is just there for extra precaution, but I agree it looks a bit ugly and unnecessary.

So your suggestion sounds reasonable, but perhaps with a slight change given the account discussion.

1) Remove findUploader() (if you dare ;) )
2) In setOrCreateParameters, set PATCHSET_UPLOADER if the event has a
patchset and it has an uploader
3) Set GERRIT_ACCOUNT as a common parameter for all events with the mapping that you suggested.
4) Set UPLOADER, AUTHOR, SUBMITTER, ABANDONER etc. (along with _NAME and
_EMAIL) build parameters as appropriate if they exist. 
5) set any other usable event specific parameters if there are any.

And put back the uploader field in PatchsetCreated so that all old builds can be deserialized.
I went ahead and moved the automation tests into the automated SpecRunner. Once we're comfortable that this is working well, we could merge the spec files.
Still have some commented lines. Please remove.
Typo in "funq1ction"
else should be placed on this line
Also: non of these sites have > 10 events listed. That's not very useful to most people.

I rather send people to generic event listing sites where they can make sure they don't clash with any kind of event that might steal away participants. These are often not just hackdays, but also barcamps and other events.
Docstring typo? "Add the item" should be "Remove the item".