labofsahil/github-event-dataset-2012 huggingface dataset:
Added more comments in 580007dd928798f99547510c2d482ed92e460fb4
I think that would break setting $S as the extracted folder has the architecture in it.  I'm not sure what would be the best way to handle this.
in addition, if no user is found, we should error out.
We should just walk the tree recursively instead of doing this in a bunch of for loops. I suppose this can stay for now, though it's not pretty :-D
Too many spaces here ;-D
The browser is not used. You removed it from the sensio connect PR yesterday.
Could these be called `setItem` and `getItem`? That's from the localStorage API, and makes more sense given what they're doing.
@retornam pretty sure that bebe might not know about "Bedrock" -- can you give a little more-thorough explanation about why this is important?
Rather than document `assertRaises`, can this comment explain what the assertion is: e.g. "First, there shouldn't be an API key."
Oh darn. Right, Thanks!
Thanks for the details (I should have read the code before writing) my
comment is irrelevant, you're already doing it the right way.
今回のところじゃないんですが， belongs_to :user なのがよくわからなかったです．
口頭で聞いたら owner を表す関係だということなので，それならば
```ruby
belongs_to :owner, :class_name => 'User'
```
などとしてあげると明確だと思いました．
Ah great.
Won't these override the default behavior of the fragment?
@ohadlevy , if the host_id is wrong, the return message is the same as what the UI returns - "Couldn't find report"
I think we can simply show a console.log saying "Your current storage
account is located at XXX, while we're creating the Service on YYY. For
better results, performance, stability and as a best practice we recommend
you to use YYY for both"

That's what I'd change before merging

On Wed, Jun 27, 2012 at 6:15 PM, Tomasz Janczuk <
reply@reply.github.com
> wrote:

> > @@ -857,13 +866,20 @@ exports.action = function (cmd) {
> >  <CreateStorageServiceInput xmlns="
> http://schemas.microsoft.com/windowsazure">\
>
> If you are fine with a potential mismatch, let's just leave the code as
> is. It is probably not worth the extra complexity.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/tjanczuk/git-azure/pull/54/files#r1063820
>
This should use the standard event format even if internal.
Many ResolveProblems classes.
Location: is not so much used for OK responses, just redirects and 201 created.
Schedule refresh or refetch?
Hmm. Maybe it doesn't work when you call .attr?
I havent even added title. Probably should.
You don't need that feature anymore since now supports real armors.
I can't see any need for this image as you have already linked to it.
This breaks the max login feature.
please check indent
I placed `lein-swank` here -- this is compatible w/ lein 1 and 2, better than including plugins as a dev dependency.
Need to fix this: <caml/mlvalues.h> (not sure how to make the path include properly in os x?)
priority -100 is really ugly to read. We could change it so we have 1, 2, etc. Then we could have modules that don't define it, which will be loaded once all of the ones with priority have loaded.
We probably want to check if the current user can update this comment. (isAdmin, is it his comment, perhaps even if the current user is manager of the content object on which this comment is posted?)
We'd be best on having a re-usable "signing" API as profiles pictures are also signing things. Given the authentication requirements we'll need to be able to "sign" a message on behalf of a given tenant, and we'll also have to "verify" a message given a particular tenant.

Out-scoped this to #277 as it will be a lot of additional code and tests, should probably be reviewed separately.
Much cleaner. Fixed.
True. Merged this for now - I guess we'll need for search results and nearby.
На данный момент лишняя проверка.
Да и вообще в цикле можно (и даже нужно) уменьшить количество итераций до процента от общего размера DNA.
Good catch. I think we should change the readme to Brackets\cefCache. I don't see "cef_data" mentioned anywhere but the readme.  @gruehle added this text in commit 2aae2d74312c43a382cfb3fec0bda8263866042d.
Ok, looks good -- just remove line 153
As above: `signals = [signals copy];`
you should look for it in the event block to avoid problems.
why not having read_source_spaces_from_tree return the SourceSpaces

I realized this doing my first attempt with the new object

```
fwd = mne.read_forward...
print fwd['src']
```
i think subjec_id is pysurfer ...

On 16.12.2012, at 19:48, Eric89GXL <notifications@github.com> wrote:

> In mne/viz.py:
> 
> > @@ -866,6 +867,79 @@ def plot_cov(cov, info, exclude=[], colorbar=True, proj=False, show_svd=True,
> >          pl.show()
> >  
> >  
> > +def plot_source_estimates(stc, subject_id, surface='inflated', hemi='lh',
> > +                          colormap='hot', time_label='time=%0.2f ms',
> > +                          smoothing_steps=10, fmin=5., fmid=10., fmax=15.,
> > +                          transparent=True, time_viewer=False,
> > +                          subjects_dir=None):
> > +    """Plot SourceEstimates with PySurfer
> > +
> > +    Parameters
> > +    ----------
> > +    stc : SourceEstimates
> > +        The source estimates to plot
> > +    subject_id : str
> And I think we've had this argument as 'subject' elsewhere for consistency.
> 
> —
> Reply to this email directly or view it on GitHub.
>
change this to test on another branch
this closure compiler option controlled weather we needed to create a .min version of a js file and do commonJS processing or not https://github.com/playframework/Play20/blob/master/framework/src/sbt-plugin/src/main/scala/jscompile/JavascriptCompiler.scala#L26 So I am wondering if removing this flag was not going to cause trouble
nvm, just saw another PR with the fix for it
On Dec 8, 2012 7:02 PM, "Philip Arndt" <notifications@github.com> wrote:
>
> In pages/lib/refinery/pages/menu_presenter.rb:
>
> > +        @options[:css]
> > +      end
> > +
> > +      def dom_id
> > +        @options[:dom_id]
> > +      end
> > +
> > +      def roots
> > +        @options[:roots].presence || @collection.roots
> > +      end
> > +
> > +      # Determines whether any page underneath the supplied page is
the current page according to rails.
> > +      # Just calls selected_page? for each descendant of the supplied
page
> > +      # unless it first quickly determines that there are no
descendants.
> > +      def descendant_page_selected?(page)
> > +        (page.rgt != page.lft + 1) &&
page.descendants.any?(&method(:selected_page?))
>
> What's wrong with calling &method?
>
> —
> Reply to this email directly or view it on GitHub.

Isn't `&:selected_page?` equivalent? Why is method required here instead of
using a regular `Symbol#to_proc`?
this part is documenting a method which exists since 2.0, so you should send it as a separate PR to the 2.0 branch (and btw, we may want to organize things a bit, as mentionned in #1811)
<img src="http://i.qkme.me/3s1eb2.jpg">
can this matrix have negative values?
Can't use 'distinct' directly since does not exist on Iterable. 
Would have to do: values.toSeq.distinct
So instead I do: values.toSet
Так как теперь можно енам указывать явно, то префиксы не нужны. Ну, кроме None, пожалуй.
This method brokes WebSocket large message support. I'm investigating why, but may rollback for 1.0.0.beta3.
ret til
```python
from Utilities import AppRequestHandler

class pubmed(AppRequestHandler):
...```
How's your schedule looking for Tuesday night?  I'll be available.
You have already import \Sorien\DataGridBundle\Grid\Rows and \Sorien\DataGridBundle\Grid\Row.

```
...
$rows = new Rows();
foreach ($data as $item) {
    $row = new Row();

```
Here is a good example of the importance of indentation. If something is to be evaluated in an 'if' block, the statements should be aligned. Here, you get the impression that only the first statement ('status') is evaluated, but not the second.
Add baketElement to deleted records
Add **s**
Delete **s**
I think you are referring to a message on exception. Fair enough, will add.
You're a champ.
Naw, i also commented on that dep loop in the PR, i have known about it for a while. Remember when we brought the params into the get_client method in the models? same issue.
No need to shout, BEFORE can be be written as before. We all know how to read :)
You can add getHashFunction() to the list. They are mostly for convenience now. getNumOwners and getHashFunction had more usages before but now only a handful remained and in those places we actually know it's a DefaultConsistentHash. So we could remove this from ConsistentHash interface and keep them only in DefaultConsistentHash.
I do have another idea.  Why not just maintain the activation counter in the interceptor and if the remove succeeds, update the counter.  I don't see why they are so tightly coupled.
Could you avoid formatting, newline and whitespace changes in a functional patch?

If you think formatting needs to be fixed, you're welcome to change it but please keep the changes in a separate commit. Otherwise it's going to create massive conflicts when porting patches to/from other branches.
No semicolon please
ИМХО это надо в Dialect вынести.
готово, но вот не уверен насчет тестов. я обещался, но переключился на ветку Георгия с миграцией.
мы сможем позже влить ветку с тестами или ждем меня?
pt[1] -= style.labelYOffset || 0;

Showld I use that behavior to backward compatibility?
ReferenceError: msg is not defined
Please can you drop the wording "strongly encouraged" and change it to recommend, or "where possible try to fold code at 80".

There also needs to be a note that code sniffers should only issue notices (and not errors) for code exceeding 120.
nitpicky, but you can use `(not-empty seq)`
This looks funky.
Let's do a predicate macro pull request
Done. Renamed "timestamp" to "freeze-date".

I'm actually setting "build-date" to the actual day that the image is built because it can differ from the freezedate.
I'm pretty sure this was intended to _replace_ the line before it...
why do you need j outside the loops? I would prefer for (int32_t j=0; ...) statements that make clear that j is only used within this loop
Can you change the rule to $('.cms form')? That way we can use it in search forms etc as well.
@willrossi You could probably get away with modifying getTemplate() to return the class name if $template isn't set.
Maybe the input.hide should go in forms? The reason it is needed at all is because the forms declarations are overriding the generic .hide helper rule (input has 'display: block' on it). 

I think the idea with some of these objects is that they will be picked up and placed in .cms-content-actions.south when the preview disappears. I suppose we could wrap them in both/either?
you can have just + instead of  + '' +
since this generation is handled by cornice, are we sure we want to add it here?
This is wrong. The base path is `app/javascripts`.
I am wondering if we should throw `\InvalidArgumentException` when either is not set. What do you think?
I don't think redis has a boolean type; aside from collections, every value is a string. The call to `@experiments.set` in `set_experiment_enabled` is a bit misleading since there's an implicit conversion from enabled as a boolean into its string representation.
you dont need to
If it is going to uncompress another world file, it is necessary to erase the /home folder that results, as this doesn't differ with each world and will be overwritten. With tar, this is not necessary, but it doesn't hurt anything, so I left it in there for both. You never know if someone forgot to remove this directory when they changed their backup format.
Are you sure the method ``debug`` can be called on ``$this->logger`` (of type "null|object<Symfony\Component\HttpKernel\Log\LoggerInterface>")?
擦。记得改掉copy代码的习惯。没用的代码要删掉。
or maybe `typestef`
Coloquei ele lá em cima, onde faz mais sentido!
`static_cast<int>`ified the value - will that address this? It's strange I don't seem to get any of these warnings on OSX (with llvm g++ 4.2.1)
This is the title of a table view section that contains some figures about the event (like the total of expenses in the event, the average cost per friend, ...)
this stylesheet is unknown
Indenting is messed up. Makes it really hard to understand what this block is doing.

Also, this `msg` object with `retry_count` seems like a leaky abstraction. Is this a Sidekiq interface? If not, could we encapsulate it better, or at least colocate the code that handles the retries with the updating of the retry count.
i think this is still a bit ambigous. what should i do if getIsPublished is true but getPublishDate is false?

i think implementations should either use publish start and end date or an explicit publish flag. but all can be queried with getIsPublished or isPublished.

i think the interface should only have the getIsPublished. writing should be a separate interface extending this one. or rather 2: one that just adds setIsPublished (or setPublished) and the other that only has setPublishDate and setUnpublishDate. documents of the later type would calculate isPublished as `$publishDate <= now() <= $unpublishDate`
Maybe use Tempfile? http://www.ruby-doc.org/stdlib-1.9.3/libdoc/tempfile/rdoc/Tempfile.html

That should be cross platform.
Style TODO same as others (space before TODO, start with an upper case letter).
Where should I put it ? Settings ? Backend ? Maybe somewhere else ?
And why is this model so much better than mine? What are benefits?
Remove the comment here
Ok, I am confused.  So the original solution I had does exactly that.  In this case I don't see the need to use gettext_lazy() since we use the text property and we can just use _() as normal, that should be evaluated as required.
The first knockback must be retarded 1 cycle, because the colision of the waterfall takes about 2 seconds to take effect.

If we cast the nockback here, ingame you see the knockback first, and about 2 sencds later the colission take effect, and this is not blizzlike.
Is there a reason these need to be atomic? This has performance ramifications, and honestly most properties don't need to be atomic. Only if race conditions are uncovered or there's an extremely obvious one that can happen should properties be made atomic.
Let's not assume that the existence of permissions in the meta field implies that the object was shared by a different user. Can you change this conditional to be `if (owner && permissions)`? Of course, if we change `owner` to actually be the current `CMUser` instead of `nil` that'll have to be taken into account.
and this service could be private as it will be used through an alias anyway
I agree but I have not had time to sit and rethink. I have created <https://hibernate.onjira.com/browse/OGM-190>
Should be filed in a ticket so that we can track upstream for a resolution. It's not normal to require manual dpkg steps for debian users but it could be acceptable while an upstream fix is being considered.

A reference to the ticket in the comment should be added.
You are correct, the API changed with version 3, my mistake. Fixing it.
`.*$` could be replaced by nothing and have the same effect
this code is duplicated grrr
No, I'd move the whole conditional down.
I would suggest `parent_view.handle.call(parent_view, action_id, event, target);` here. `arguments` is basically deprecated.
@bnoordhuis 
I'm not sure what you're asking.  Which `Init()` are you talking about?
Seems like DIRECTIVE_IS(args, x) or DIRECTIVE_IS(args[0], x) would be better here. I'm not a fan of this sort of magic :/
That sounds like a good plan @rbirnie.
Thanks. I'll do that when I get more time on my hand. Though if compctl does not get removed, I don't see any problem in using it. It seems more robust (no init, less code) and easier to understand than compsys (at least to me).

What's your opinion on pulling this module? I'll do some more testing on the fix and merge it into master. But maybe you want to wait until I tag a new version.
I provided examples in some of the individual functions.  Should I 
provide examples here or should the "full description" be mostly a 
general discussion?

On 06/21/2012 01:45 AM, Ralf Gommers wrote:
>> +
>> +        Difference between log-likelihood of corr0 and a pre-specified
>> +        value.
>> +
>> +        Not intended for end user.
>> +
>> +        """
>> +        return self.hy_test_corr(corr0, nuis0=None, mu1_min=self.mu1_lb,
>> +                       mu1_max=self.mu1_ub, mu2_min=self.mu2_lb,
>> +                       mu2_max=self.mu2_ub,
>> +                       var1_min=self.var1_lb, var1_max=self.var1_ub,
>> +                       var2_min=self.var2_lb, var2_max=self.var2_ub,
>> +                       print_weights=0)[1] - self.r0
>> +
>> +
>> +class DescStat(OptFuncts):
> This is the main class right? At least it's the only one that's not noted as "not intended for the end user". Then here I'd expect a full description of what empirical likelihood estimation does and how to use it.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/statsmodels/statsmodels/pull/326/files#r1024006
I started using asarray in places where it can do no damage (e.g. I don't use it for alpha since alpha is passed in from a calling script).
Do you have an example of word not being converted by iconv ?
Actually, Twig is using iconv first : https://github.com/fabpot/Twig/blob/master/lib/Twig/Extension/Core.php#L819
Ha!  Nice.
These should actually all get implemented. The reason they weren't was because the original submitter was lazy - not because of design.
This is fine for now, just push this as is, and we will solve things as we progress with journal.
consider using while(true). While they both do the same thing, while(true) is more readily understood. No change really necessary though.
[@mabster](http://twitter.com/mabster) linkified :P
can you please add semicolons to the end of these lines + @douglascrockford
None is fine with me.
A comment is missing here. New coders cannot understand what this is.
this code would be much easier to read if you just avoided doing string processing just in time and just used some intermediate variables:

prefix = course.handle.split('--')[0]
suffix = course.handle.split('--')[1]
makes sense -- caller can setJmxTimeout if they want to
Is this right for all other versions of AMQP except 0.10? Surprising.
Sorry, forgot to check the original Java code.
Welp, I just should forget the silly code I was trying to use, and just extend _.isNumber:

`toString.call(obj) == '[object Number]' && isFinite(obj)`
It's a convincing argument and submitting to it will hurt me deeply
Why break the chain?
tirar ```@Test```
Here do `return self.selenium.find_element(*self._name_locator).text.strip()`
What's SNAFU? --> Make "SNAFU" it into a link for people who don't know ;)
callback is not required here which has been included in update function
nit: i think this line is useful to distinguish between the headers for the editor and the javascript 'strict' instruction
I don't think I'm familiar with what MPI is.  The Julia build process uses `gfortran` to compile arpack-ng, (they skip the build process and build only a subset of what they need, but I figured I'd make things as general as possible) so that's what I did as well.  What's the difference between `mpif77` and `gfortran`, and why does the compile succeed with `gfortran`?
Why do you add $cron_url, it's done on line 198 ?
This should be document, not docment.
I made a quick attempt at this, but I couldn't figure out how to easily access the default custom post status. What's the easiest way to get that information?
Конкатенация должна выполняться через StringBuilder
@raydavis thanks!
I don't understant. This is the README-tomcat.md
No need for this import
Please include a comment that gives more context to value, and also consider a variable name that gives the reader more information about what's in here.
I'm finding this difficult to read. Could you comment on what it's doing?
Using `objects.exists()` is a better idea here because len() does an actual full count on the db whereas exists just checks if a single row exists.
h_students_peoplesoft
Yes, thanks! Will remove that...
For ```contingency[nnz]```, it is a good idea to pre-calculate it. For the second part, spliting it will make them to be more clear. I will change it after we have made decision on the numeric and coo_matrix issues :)
could be just `@state`
Where did loadComplete go?
Cau, bolo by mozne spomenut tu, ze funguje osmand.cz ? Dik.
Brackets again.
+1 for parens, good catch, thx
Good point. I'll change the names to `PROPmem[gs]et`, and `PROPufc[gs]et`.
This condition only needs to check for IsString.  The ToString() methods should be applied to StringObjects also.  This is how JS behaves in Chrome.
don't add spaces in template uses.

`Handle<Object>` instead of `Handle < Object >`
Actually, checking `$this->assertSame($em, $locator->get('doctrine_em'));` works. I am unsure this is what we want...
one parenthesis pair is not needed (if it was needed you would need a space around it)
> Actually, it's just a few very useful Query implementations that deal with regex patterns. See the JavaDocs for an idea of what's included.

ah right, looks good. What are you saying about the version numbers, are these not released using the same numbers as the rest of Lucene?

> As for the slot name, we do use the default slot on all of ModeShape's modules, but we used a non-default slot name for any module that wasn't "official" (e.g., not really ModeShape libraries). For example, we expected/hoped that HS would create your own module, and if installed we didn't want it to stomp or overwrite our definition of it. However, once you have yours, we'd prefer to just reuse yours. Installing is still a bit hairy, but that's will eventually be easier (especially if there's a central repository for modules).

So whe're good using the default for HS ? One of my goals is of course to maximize reuse and make it more practical for your project.
@richardkchapman is the program currently included in the distribution?  It doesn't look like it is to me.
I changed this name more for the sake of semantics, as the logic is the same. I had to subtract one to make the semantics correct, but I still think it's not a bad change.
This has already been fixed.

http://www.php-fig.org/faq/

Thanks.

On Thu, Jul 5, 2012 at 11:28 AM, xanido
<reply@reply.github.com>
wrote:
>> +## What standards have been passed so far?
>> +
>> +<dl>
>> +     <dt><a href="https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md">PSR-0</a></dt>
>> +     <dd>Aims to provide a standard file, class and namespace convention to allow plug-and-play code.</dd>
>> +     <dt><a href="https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md">PSR-1</a></dt>
>> +     <dd>Aims to ensure a high level of technical interoperability between shared PHP code.</dd>
>> +     <dt><a href="https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md">PSR-2</a></dt>
>> +     <dd>Provides a Coding Style Guide for projects looking to standardise their code.</dd>
>> +</dl>
>> +
>> +
>> +## Who appointed you to make these decisions?
>> +
>> +The group was bootstrapped by a number of framework developers at php|tek in 2009.
>> +Since then various other memebers have applied and been voted in, increasing the size
>
> Small typo, "memebers" should be "members"
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/php-fig/php-fig.github.com/pull/23/files#r1103901
This should be put into the `finally` clause. If something fails we will not have a valid report in that case.
What is the benefit of moving this to a views block preprocessor instead of using the preprocess_block?
Seems that these should be:

```ruby
  gem.add_dependency "activesupport", [">= 3.0.0", "< 3.3.0"]
```

The assumption should probably be that this will work with any 3.2 version of ActiveSupport/ActiveModel.
`0 === strpos($version, 'dev-') || '-dev' === substr($version, -4)`
this is a bit funny.. if the URL cannot be fetched, why pass the URL into the XML parser? I think it'd be better to not catch the exception and just let it kill the program.
This whole file should be deleted. Bootstrap source shouldn't ever have been modified directly. After deleting this file, you can just `@import 'bootstrap';` at the top of a LESS file, and then modify the bootstrap variables as needed. Look at what I did in the Nangala CSS - changing bootstrap variables this way cuts back on the CSS. You can include only the relevant bootstrap modules to cut back on the total CSS file size
@shacker I've given this a bit more thought and didn't see the point of adding it here.
Sitä on lähdetty hifistelemään Java interopin kanssa :). Merkkijonoja voi kyllä vertailla myös `=`:lla.
You can pass `alive?` straight to `filter` like this `(filter alive? authors)`.
Hah - true! And I thought it would be a nice optimization... oh well.
not 2.6 compat
code example please
Worse, this whole file looks copy-pasted. It would be much better to abstract all the common stuff out and inherit. Same with the other handlers.
A bit more description would be helpful. Something like: "The `build_gauassian_pyramid` function takes an image and yields successive images shrunk by a constant scale factor. Image pyramids are often used, e.g., to implement algorithms for denoising, texture discrimination, and scale-invariant detection." Feel free to tweak that however you wish.
"Number of source and destination coordinates must match."
Could you remove those git submodule changes from the pull request?
Hmm, I was unaware of `always`.  Pretty awesome, thanks.
Needs an `@exception`.
Important to mention that the data in the new interval takes precedence over any existing intervals in the collection.
I meant drop line 40 and change 32, 34, and 36 to s/enum defaultCompiler/string compiler/.  I fully understand the value of helping rdmd default to the compiler that it's shipped with.
What happens when the user sets only one of shadow color or shadow offset?  Should we have some default values for these? If both are needed, we should only call setShadowLayer() when both are present.
We absolutely should but at the time I considered this a "next phase" sort of deal that would come along with the ability for users to submit their own test results (maybe a certification system who knows).
you can do !~i to save a few bytes.
To be clearer here, I think we might say, "To set all of the sections in a table view at one time, use the table view section property."
Помню я кому-то говорил про то, что нужно избавляться от генерации функций и выносить максимум в статические функции. Если это был ты, то забей ;)
should mention that embeddedDoc should be an array.  Maybe even type check it.
this seems to be a nonrelevant change ?
haha isso é fácil de mudar
Ну в `$.ajax` тебе возвращают, например, `error`, `timeout`, etc. Коды нужно документировать, и они здесь не экономят ничего. Сомневаюсь насчёт локализованной строки, но насчёт бесполезности кода точно не сомневаюсь.
I think `/// ditto` is better.
No need of "WindowManager.getTopWindow()", because this call was used to know if there was at least one window, and you make this check with the second condition (WindowManager.getWindowCount() > 1).
What does this do if the account is disabled for this app?
(mentioned above) I was going to make this a separate pull request, if we decided we want GA out.

Personally, I like to keep pull requests fairly atomic, regarding functionality changes.  It's not that I don't want to, it's just that there's no reason to make that part of this pull request.
Agreed. As part of this PR or in a new PR?
compile_select is used by other things - so we need to make sure that we are counting otherwise we'd disturb legacy stuff and non-counting active record query's. I definitely agree the backticks are not universal, I think postgre has some problems with them. I can't verify the COUNT(field) but, at the end of the day, if you're using a database that you know doesn't support COUNT(field) then you shouldn't be specifying a single column - I do realise that might be a little harsh, so, the only solution that I can see is to "whitelist" the DBMS's that can do it.
Falling back on old code for FX35 because I can't figure out (yet) how to get the 4.0 stuff above working.
verbosity detracts from purpose:

    context.actorOf(Props(new IndexStorageActor(config.mongoURL)), "index-storage")
Le seul bobo... t'as pas mis le formatter a off / on  faque it looks like shit ;)
Really? ;)
ok, it is not a boolean. But you should probably limit the possible values
You should have named these things like DAPSONE_CONCEPT_UUID and included them under the Concepts heading, to be consistent with other things.
How about using jQuery's `trigger` function to simulate the button being clicked?
it does:) i will update
I believe I explored that option but could not get the precise results I wanted, let's discuss.
_Institute_ ??
Рендер адреса должен быть в отдельной ячейке
Please add back in the ` && packet_id >= 0`.
It is an essential bugfix and I'm not sure why it's being reverted in your code.
Shouldn't we be using single quotes for:

`sakai.api.Content.getMimeTypeData("x-sakai/collection").URL`
That's weird, but good to know for sure.
this logic would fit better in the TotaltOrderInterceptor, as it doesn't have to do with replication at all.
The !Solaris build failures should be fixed now.
I disagree that we should *only* support key-expression or that we would have done that if we had always had expression support and the reason is as I stated.... consider the most common case. I would bet 90%+ of the time people will just simply provide a literal key value. Forcing them to always use quotes around that doesn't feel right.
Why default to "production"?
AGreed.
That's because the pipe will fail online documentation - see http://zf2.readthedocs.org/en/latest/modules/zend.cache.storage.adapter.html#zend-cache-storage-adapter-methods-get-metadata
I agree on the case issue, but then I checked the Symfony2 codebase to see what is most used (I also see `inheritDoc` a lot):

``` bash
..symfony $ git grep inheritDoc | wc -l
302
..symfony $ git grep inheritdoc | wc -l
447
```
If you rather have it case sensitive I'll update it.

On the other issue, I guess that as soon as `@inheritdoc` is used in a comment the `@param` `@return` statement etc from the parent should hold? If you have another idea I'd be glad to hear. (We only use `{@inheritdoc}` to inherit the full docblock from the parent currently.)
Yeah, I know. I looked at a bunch of cases and that didn't seem to ever happen, but it's something that I'll add when I get a chance.
This should be renamed to `sakaiproject` to keep in line with all our dev docs
ok, I will dedicate some time later today.
Probably better to use assert_raise here.
aș pune decoratorul ăsta în auth.py
Y U NO use js2coffee.org?
It's a bug in the model.  The field isn't pulled from the database.
no need for "LABEL_"
this looks hand done? should be spaced over like above?
Wouldn't child.getAbsolutePath().startWith(parent.getAbsolutePath()) be enough?
Or doesn't commons.io provide methods for this...?
Actually this also applies to a number of methods here, if you're already at it, could you fix those as well?
\GeometriaLab\Api\Stdlib\Hydrator\Api move to \GeometriaLab\Api\Stdlib\Hydrator
Totally a preference call there, but I agree, alignment vertically with the `||` would be rad.
formatting appears to be off.
i might suggest using a more general scgitx.yml file with a buddies key here.  the reason is that we have on the roadmap to make the aggregate braches configurable, and this would be a very logical place to configure what aggregate branches are supported.

ex:
```ruby
# config/scgitx.yml
buddies:
  emilyjames:
    buddy: charlesdavis
    socialcast_username: EmilyJames

# example of adding additional config options here
aggregate_branches:
  - staging
  - prototype
  - v6-redesign
```
Again please do not use "till"
I'm undecided if it should be public. It'd be useful for creating Windows batch files on all platforms, but I don't know if that's a use case common enough to add a public, documented function to the standard library.
This was causing issues with the code when testing, I think the value undefined is beinf set as a string elsewhere, I agree its misleading. Ideally it should be (typeof dictionary[value] !== undefined) - but that should remain consistent and never be set as a string. I will review this and see if I can trace why it was causing problems in my live tests.
This is th only needed change
For requiring connectionJNEXT, you can just use require("./connectionJNEXT"), since they are located in the same folder.

The requireLocal function is only needed when we need to require stuff from lib or other extensions.
Why are you adding browser require as a part of this change? It's a little weird as a part of this change, at the least we should separate it as a new commit
Please change this, our agreed style is new line for every variable
should participate in the round trip unit test for translation.