labofsahil/github-event-dataset-2012 huggingface dataset:
We should only have X in init not X and the active set. As we call the _solver functions with G[:, active_set], and modify the actual active set based on the results of the _solver function in the main function, we don't need it. As a consequence of transferring it, you need the hack in _mixed_norm_solver_prox where you set active set again to a full vector of ones. So why should we transfer data, that is not actually needed.

In _mixed_norm_solver_prox you (lines 176 - 193) also define active_set as zeros first and then do the hack:

    if init is None:
        X = 0.0
        active_set = np.zeros(n_sources, dtype=np.bool)
        R = M.copy()
        if gram is not None:
            R = np.dot(G.T, R)
    else:
        X, active_set = init
        if gram is None:
            R = M - np.dot(G[:, active_set], X)
        else:
            R = GTM - np.dot(gram[:, active_set], X)

    t = 1.0
    Y = np.zeros((n_sources, n_times))  # FISTA aux variable
    E = []  # track cost function

    active_set = np.ones(n_sources, dtype=np.bool)  # HACK


And by the way, G in _solver_mixed_norm_prox is set to G[:, active_set] in the main function, so if we'd take the active_set from the init (which is the size of the number of sources in the full leadfield) the line (R = M - np.dot(G[:, active_set], X)) could break.
if you don't plot anything don't call it plot
The idea here was to keep it as close to the C function as possible (which allows you to not input a filename and have it cleared). I'd vote for keeping it in and adding a note, but if you don't mind moving away from the C code some, I'll remove it and add a comment saying that's how they can get this same functionality.
Turns out we weren't testing to enough decimal places. Note that if the decimal places are taken more precise than 12 (note that our data have magnitudes around 1e-11...), it throws an error. I think this might have something to do with the transition bandwidth (see below).
With Kitsune we wanted a stable ordered indexing. I was on the fence about whether to do that here, too, but I think do because then things are always indexed in a reproducible order.

However, the autofield/pk thing is a good point.

Part of me thinks this is a reasonable default especially where we explicitly state how it works in the docstring. If someone needs something different, they can easily override it. But if we can address the autofield/pk thing, that'd be good.

If you have a good way of doing that, I'm game. Otherwise, I'm going to leave it like this for now.
I'm thinking the BLOCKED response should probably return a `403 Forbidden` instead of a `200 OK` since it was forbidden and it's not ok.
How are you setting the preference for picking names e.g if i have two concept name hits for 2 different concepts, how do i rank the concepts in the result list?
add @should pass if there is at least one valid description
Would the command prompt "$" be included in the output from the command?
"What if the body is application/json? text/xml? This is saying that for an application to have CSRF enabled, it cannot accept anything but multipart/form-data or x-application-urlencoded submissions, otherwise it will fail to parse them. "
The user must choose when to apply this Filter (in it's global).

"I would suggest checking for the token in the query string," 
 That's the default behaviour.
what does the ... mean?
If you look at the [yield line](https://github.com/rails/rails/blob/master/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb#L152), you'll see it calls `connection` which will check out a new connection.  The `connection` word on on that line is a method call, not a local variable.

Yes, I ensured my test failed before implementing the fix. :-)
I think this sentence should be reformulated to better fit the style of a tutorial.  For example: "If it cannot compute the sum, it prints the corresponding summation formula."
Отбивай функции одной пустой строкой лучше. И лучше
```c++
Event ()
: EntryDBId_ (-1)
{
}
```
Потому что не стоит пихать рефы везде воо6ще. Здесь семантика другая.
más =P
The shorthand for value = value + number is value += number
You can simply do

    head :ok
Why are you using setTimeout rather than $timeout here?
$installArguments is a global script variable, that's why this works. Your changes make this more visible.
NP, I'll remove it.
you're including some whitespace changes. I like the changes as I hate those trailing whitespaces too, but next time please keep whitespace changes isolated from the functional commits.
Could be improved to

``` php
<?php

return new Response('Unauthorized', 401, array(
    'WWW-Authenticate' => 'Basic realm="GitList"',
));
```
14は何を指してますか？
意味のある数であれば定数にしてください。
Это первая итерация, по хорошему все проверки должны проходить в примитивах ну собсно он должен выстовлять себе ошибки если таковы были.

Но я тут начал эту тему делать и прифигел, тама все примитивы надо будет затронуть практически, по этому оставляем на потом.
jQuery dependency ?!
@sumpygump Ah, now I got the idea behind that constructor. Thanks for clarification.
so please update rest lines too :)
The Instance startup_tags were being populated at some point; I remember
this from when I was testing.  But they never got updated once the instance
launched.

On Fri, Apr 20, 2012 at 2:53 PM, Tony Spataro <
reply@reply.github.com
> wrote:

> > @@ -108,7 +108,10 @@ def run
> >            @context.audit.append_error('Could not discover tags due to
> an error or timeout.')
> >          else
> >            # or, it could give us anything else -- generally an array)
> -- which indicates success
> > -          CookState.update(InstanceState, { :startup_tags => tags })
> > +          #This tag query is duplicated in InstanceSetup#initialize,
> but doing
> > +          #it here ensures we have fresh tags on every run.
> > +          InstanceState.startup_tags = tags
>
> Oh -- another issue -- if the startup tags never get populated at all
> (which they were not, previously), then they are not startup_tags, they're
> useless_tags. ;-) It's a good thing they're not used anywhere!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rightscale/right_link/pull/80/files#r716231
>
`lisr` -> `list`
Can you split this line up into 2 lines for readability?
Re: reverse logic

  I think you've miscounted parens ;-). Here is the same code with whitespace added:

```
(
  !(
    User.class.isAssignableFrom(authorizable.getClass()) && 
    ((User)authorizable).isAdmin()
   )
)```
This was never meant to be included in the pull request: I sent a pull request for three commits (ef427ed, ea911b5 and c15ded7) but github included also the following commits on my branch. I'll check if I can pin the pull request to the original commits.
What about pushing fragments from the backend? Say, user agent has requested "EditForm" fragment, but as an exceptional outcome of our backend calculation, we decide we also need to push "FooBar" fragment to the user. 

Is it always a responsibility of the user agent to always know what fragments to request?
instance variable dependencies over a subclass hierarchy should be prevented whenever possible. This makes the code brittle and hard to follow.
I also removed the text-shadows because they were confusing people.
note sure what this changes...any idea what the changes to slowparse were?
yeah, basically, all VM support each language, but they have multiple versions only for their own one.
I think `.float-top` is probably cleaner for the content. `.column-top/bottom` make no sense when in a `.span-all` and `.page-top/bottom` makes no sense when _not_ in a `.span-all`.

    .float-top {
      float: column-top;
      .next-page { float: column-top next; }
    }

    .span-all {
      .float-top {
        float: top;
        .next-page { float: top next; }
      }
    }

You could also probably reduce the code by using parametric mixins...
removed line 274 targeting div.colophon.end-of-book-solutions | redundant code
Are you sure ``'setOptions expects an array or Traversable'`` has the correct type as "string" is not a sub-type of "string"?
How about using ``$packages['package1-1.0.0.0-beta']->name`` instead of ``$packages['package1-1.0.0.0-beta']->getName()``. There are some issues with ``getName()`` and APC-enabled PHP versions (see https://bugs.php.net/bug.php?id=61384).
Their code has 1 tab and 4 white-spaces (in front of view.updateLayout({width:150 * scaleX,height:150 * scaleY});)
code added by me has 2 tabs.
So I changed their code to have 2 tabs instead of white-spaces (which is in my opinion better than make my line of code to have 1 tab and 4 white-spaces)
mach zuerst das in der db, sonst gibt es inkonsistenzen wenn das speichern in der db fehlschlägt
Just `(contains? (:awards book) award)` would do.
nil and blank are the same thing. How much of a gain do we get from short circuiting the nil check?
I understand what this code is doing (using `<>` instead of whatever the default is) but why is this needed?
No, just link to my fork. We will probably also do other changes. You are also welcome to fix things if you find problems.
You are not using this?
Yes. you should define a global JavaScript variable in template part, which you then reference here. Ugly, but better than this.

(And you define this variable with use of `url` template tag.)
Now I noticed. This is not really a param. You use this when commenting functions and which parameters they take. In this case this is a class. and even class constructor does not take this parameter.

You might simply move this line above `layout` definition bellow and just leave it as a in-code-only comment.
Static string.
Fixed logic back to what it was before.
Why? Why not simply display horoscope, if user has birthdate, or error if not. Why is important if it is authenticated or not? Code checks already if birthdate is set and this is all we are interested in.
Why those changes?
Do we really need this variable? If, then it should be called `users`. :-)
Need tests for the other distribution functions.
We could just take out the assigned as well. It's easier to add fields than it is to delete them
Copy/paste error

On 27.12.2012, at 18:42, Nathaniel Talbott <notifications@github.com> wrote:

> In lib/active_merchant/billing/integrations/axcess.rb:
> 
> > @@ -0,0 +1,96 @@
> > +require File.dirname(__FILE__) + '/axcess/helper.rb'
> > +require File.dirname(__FILE__) + '/axcess/notification.rb'
> > +
> > +module ActiveMerchant #:nodoc:
> > +  module Billing #:nodoc:
> > +    module Integrations #:nodoc:
> > +      # To start with Axcess, follow the instructions for installing 
> > +      # ActiveMerchant as a plugin, as described on 
> > +      # http://www.activemerchant.org/.
> > +      # 
> > +      # The plugin will automatically add the ActionView helper for 
> > +      # ActiveMerchant, which will allow you to make the Nochex payments.  
> What is the relationship of Axcess to Nochex? Is this just a copy/paste error?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Change this to STATUS_NEVER
ADD:

	    void HandleEarlyFrostScript(SpellEffIndex /*effIndex*/)
            {

            Unit* caster = GetCaster();
 
	        if (caster->HasAura(SPELL_MAGE_EARLY_FROST_R1_T)) // Check Talent
			   if (!caster->HasAura(SPELL_MAGE_EARLY_FROST_R1_CD)) // Check Trigger 
		           caster->CastSpell(caster, SPELL_MAGE_EARLY_FROST_R1_CD, true); // Cast Trigger - 15 Sec Cooldown
	   
	        if (caster->HasAura(SPELL_MAGE_EARLY_FROST_R2_T)) // Check Talent
			    if (!caster->HasAura(SPELL_MAGE_EARLY_FROST_R2_CD)) // Check Trigger 
		            caster->CastSpell(caster, SPELL_MAGE_EARLY_FROST_R2_CD, true); // Cast Trigger - 15 Sec Cooldown
             }
pls use:

if (Unit* target = SelectTarget(SELECT_TARGET_RANDOM, 0, 0.0f, true))
    DoCast(SelectTarget(target, SPELL_TRUE_FULFILLMENT, true);
Why creating a new loader ? There is already one.

And why creating a new XML file instead of putting it in the existing one ?
I wonder if grouping measure first by distance and then by time is more efficient.
I like this. It shows the different ways to do this? Can you throw some comments in here to explain what you've got going on in here?
DNAT or SNAT?
I had an issue with 'arguments', btw 'arguments' is a reserved word in the coffeescript (https://github.com/jashkenas/coffee-script/issues/1547), therefore I think it is better to avoid usage of that. If you need to pass to other function all arguments passed to the current one, it is better to use splats (http://coffeescript.org/#splats). It is only my thoughts, you are owner of this project, therefore I will follow other rules if you have :)
а что такое "/dev/null"? а для чего оно?  а где оно в тесте используется?
Can this be a private/protected method? We don't want extra, implementation details confusing users when using PHP code completion.
The new unit test fully stubs out any external data the module needs and action the module may take.
Could this be done with ellipsis in css instead? Assuming this is to shorten the string for the UI.
Linear scans are almost always bad but I assume that memcache_server_.size() is small?
Yes please, do so. Thanks.
@johnf When I try to do a librarian-puppet install it fails at this method.

```
/home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/lib/librarian/puppet/source/git.rb:122:in `to_s': undefined method `[]' for nil:NilClass (NoMethodError)
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/dependency.rb:69:in `to_s'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:34:in `block (2 levels) in recursive_resolve'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:109:in `block in debug'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/logger.rb:31:in `block in debug'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/ui.rb:32:in `debug'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/logger.rb:31:in `debug'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:109:in `debug'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:34:in `block in recursive_resolve'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:33:in `each'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:33:in `recursive_resolve'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver/implementation.rb:20:in `resolve'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/resolver.rb:18:in `resolve'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/action/resolve.rb:25:in `run'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/cli.rb:161:in `resolve!'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/lib/librarian/puppet/cli.rb:69:in `install'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/thor-0.16.0/lib/thor/task.rb:27:in `run'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/thor-0.16.0/lib/thor/invocation.rb:120:in `invoke_task'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/thor-0.16.0/lib/thor.rb:275:in `dispatch'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/thor-0.16.0/lib/thor/base.rb:425:in `start'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/vendor/librarian/lib/librarian/cli.rb:29:in `bin!'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/gems/librarian-puppet-0.9.4/bin/librarian-puppet:9:in `<top (required)>'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/bin/librarian-puppet:19:in `load'
	from /home/lunix/.rvm/gems/ruby-1.9.2-p290/bin/librarian-puppet:19:in `<main>'
```
is this the update to latest versions of all those? i assume that our branch had very outdated versions?
not crude at all! thanks very much for helping. in fact, it may be my bad wrt failing... not sure why only the REE build is failing either. I'll look in to that.
Can we do something like:

    DateTime->now->add( %{ params->{expiration} || config->{expiration} }

So that end users can set it via a dropdown on the page? Like pastedance does.
When a hash is the last argument in the method's call Ruby allows to omit curly braces. I think it's a good thing to do.
You could simply pass all those values as attributes to the constructor and not assign them one by one later on.
IMHO, an @Ignore'd test is as good as no test.

We need to come with a Test that does test the user scenario (even if that mean an IT).  Otherwise there is no way of telling if the problem got fixed or not and it will likely rise again.
w3scools site is a troublesome and faulty resource and is deprecated (at some point)
more info on http://w3fools.com/

WebPlatform.org is an open community built learning resource approved by MDN, Facebook, Google,  W3C, Opera  etc
Oops, i see your later comments on this for corrfuncs - I see you are working on this with Jim.
version bump, how irritating.  will do this on 0.4.0 branch also.

that's it for significant changes in the pull request...
sorry was being daft, you're quite right

Tasks.uncheckedGet(Task)  ?

and/or  Entities.callXxxx  which does invoke and get, with no checked exceptions ?
Shame about the order of these parameters (very different from `super.newLocatoinFromString(Map locationFlags, String spec)`). Should we try to fix that (e.g. deprecate one of the methods and overload, relying on no users having implemented the interface)? But not in this pull request.
Does this really want to be in a finally block (e.g. will log "unable to retrieve..." and then immediately "retrieved catalog from...").
flag is restricted in scope to the entity so fine for it to be shorter if unambiguous.
(for web we have  httpPort  and   httpsPort)

that is different to the config key ID which should be entity specific as it might be set on the app, either by reference to the static final constant, or in a properties file using the ID
boom, mergin
Why is `finish` needed?  Although not obvious at first, I'm convinced that it will work as long as `getBlock` works, but I still don't see why it would make any difference.
In fact, I suppose a better solution would be to remove Chebflag alltogether, and to replace `if Chebflag` by `if chebcoeff is not None`.

Later we should also move some of those initialisation bits in separate function, both for better testing, and to make that long function `__init__` easier to read.
Instead of handling the case of "no accept header" in ```accept-exists?``` is would be more consistent if ```best-allowed-content-type``` handled this case also. I have to thing about this. Nevertheless the proposed patch seems to implement it correctly. 

Note: in which case can```(best-allowed-content-type "*/*" ...)``` return ```false```?
Minor style issue: ```(if a true b)``` can be replaced be replaced by ```(or a b)```
need to remove this line
Check for `null` here and emit a polite message suggesting that the user might be accidentally running on a JRE rather than a JDK.
це круто. але функціонал створення тумби в тому чи іншому вигляді все одно потрібен
The reason I have implemented this in ``ProductNotification`` and not in ``AbstractNotification`` is that the two methods ``get_confirm_url`` and ``get_unsubscribe_url`` are specific to the product they are attached to. I am not sure what a generic implementation for ``AbstractNotification`` should include.

With that said, I agree that my ``models.py`` is not as bare as the other ones. It might make sense to actually move the implementation to ``AbstractProductNotification`` but then we'll loose access to all instances derived from ``Notification`` using the inheritance manager. I am not sure what the best way is to handle this. What do you suggest?
We should probably cache the rounded image in the Query (pass cover() an image type like Normal, Rounded, etc) like we do for Source::avatar()
This makes use of `Buffer::__toString()`.
missing words here
+1 on this however, we would, as suggested, have to have a whole other bug to track this as things are a bit all over the place at the moment. In this case I am going to move the method to the model
Maybe call it job-bad-count for consistency. That way, all the job-* will be clustered together.
use a slugifier trait, so that you can do `$this->slugify`, instead of using hell static methods.
you should be testing ``2.0.*``, ``2.1.*`` and ``2.2.*`` with Travis instead of ``2.0.5`` (totally outdated) and ``origin/master`` (invalid version)
Good idea, I opened a Jira
(AEROGEAR-623<https://issues.jboss.org/browse/AEROGEAR-623>)
for it

Summers Pittman
>>Phone:404 941 4698
>>Java is my crack.




On Wed, Nov 14, 2012 at 11:58 AM, Bruno Oliveira
<notifications@github.com>wrote:

> In src/org/aerogear/android/impl/pipeline/RestAdapter.java:
>
> > @@ -100,9 +103,18 @@ protected AsyncTaskResult doInBackground(Void... voids) {
> >                      applyAuthToken();
> >                      byte[] responseBody = httpProvider.get().getBody();
> >                      String responseAsString = new String(responseBody, "utf-8");
> > -                    T[] resultArray = gson.fromJson(responseAsString, arrayKlass);
> > -
> > -                    return new AsyncTaskResult(Arrays.asList(resultArray));
>
> What do you think about extract the method below to another utility. The
> usage could be something like JsonBlah.parse(responseAsString);
>
> Might be useful to another classes.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aerogear/aerogear-android/pull/37/files#r2126660>.
>
>
Not a big deal, but SIMPLE_MODULE_NAME can be final.
Please avoid changing the return value of existing methods. Furthermore, errors should be thrown and caught instead of passed as return values. Also see https://www.chiliproject.org/issues/953 for more info.
Add [eAccelerator](https://eaccelerator.net/)?
I forgot to add the replacements according to the `Storage`, will be fixed.
This test used to create adapted deferreds. Different libararies needed different methods of adapting which I felt made this test inaccurate. Now all adapter provide a function which creates deferreds in a library specific way. Therefore, it reflects real world usage.
```has_one``` should be the same as ```has_many``` except using ```:first``` instead of ```:all```  

I think you probably should name this ```belongs_to``` as that follows ActiveRecord's conventions.  If this class has the key, then it's ```belongs_to```, if the other class has the key, then it's ```has_*```
Здесь ты закрываешь System.in. А это нехорошо. Вообще, общее правило - если стрим приходит как параметр (не в конструктор), то закрывать его должны снаружи.
can you be more specific about "approach in PHP"?
Can we just inline getCarrier,  getModel, getVersion?
can you just do `return self.current_step == self._current_step`
Let's just delete this for now.
haha! :-P
давай HTTP до конца домучаем :)

1. `type` это на самом деле `method`
2. `path` не должен быть без слешей (то есть следущий пример лучше `$api->request('GET', '/')`)
Please add spaces between operands and operators.
This should be `string[]`.
Stop messing up my indentation :P
The default is "blackberryndk.py [options]", which probably not what we want.
David, can you set the hostname of a device? Otherwise i'm not sure what the usefulness would be to support hostname
if one of the tab_, navigationPane_ or backAction_ pointers is NULL, it means that create function couldn't allocate enough memory for at least one of these objects, so we can return a NATIVE_ERROR_OUTOFMEMORY error
update accordingly to scope of :x_on_behalf_of

headers should be added only if there is value ( empty string is still a value )
Use `find` instead. This is premature optimization
Wow, thanks for writing asprintf() so fast. [One of my friends](https://github.com/pquerna) mentioned that Solaris didn't have it, but since Ag doesn't support Solaris *yet*, I figured it could wait a little while.
"it shall be used" is very strong, you are saying it's mandatory? I don't agree, one could also use getters annotated with

    @Latitude
    @Transient
    double getY() {
           return _some complex computation_;
    }
When would an associated file be on a different ip? And would it not/could it not also apply to the dll if so..?
Yes, will change
The xpath is coming back is a result of user selection.  queryPropTree will never return NULL.
You've added the same comment twice.
Let's add this:

```
First running `collectstatic` and `compress_jingo` is more realistic compared to how the production server is run. Also, it's faster.
If you want to disable static file compression you can add `COMPRESS_OFFLINE = False` to your `settings/local.py`.
```
oh,again it is moving the js to the bottom
"plug and play" should be hyphenated as "plug-and-play".
Instead of checking that each time we call updateTitle, setting the new handler should be done in init(). Same as what we do in browser.js.
This setting seems to actually override the `-isolateDevice` option (I tested it by setting a bogus value for it), but is it really necessary? I think it just confuses users, better let users report a bug and suggest to use BusID as a temporary workaround until the code get fixed (in case detection proved to be wrong).

To list all VGA and 3D controller devices, use the command `lspci -nn | grep '\[030[02]\]'`
This is the part of the code that will take exercise names from database
This will add N queries. Maybe get a user's groups once and then do `if perm.group in user_groups`?
And we should have a performance test on the number of queries generated in the case of multiple groups with multiple users.
here it is again
this should be configurable
right, this was more of an fyi thing than something that would need to be fixed. I will leave it up to egypt or todb as to what their preference as to implicit/explicit returning is
if res and res.code ! 500 and res.code != 200
Agreed, have done changes accordingly.
I'm not sure if Cython can speed up these kinds of loops.
I renamed so that we could push to clojars. Apologies :)
Why do we need these try/catches?
Looks like a file mode change 100755 → 100644

This is likely my fault from when I added these files originally from Cygwin before I discovered I should probably disable file mode tracking there (I think msysGit does this)
Why not?
```
function ImageryProviderError(imageryProvider, message, x, y, level, timesRetried) {
```
Functions are first-class objects, right?  The later `return ImageryProviderError` statement would be OK, right?  At one point, there might have been a doc bug where reference doc didn't show up depending on how we declared a constructor function, which is why most of the code uses `var foo = ...`.  CC @kristiancalhoun
tomato tomahto
为什么public前有空格？
'{' should be on a separate line for methods
missing space between `if` and `(` here, and many more below
Yes, it can cause a new paragraph.
Thank you - fixed in next update.
This is kind of a nit, but if we enter "Void" explicitly, it shows up as "Returns: Void", whereas if the return type is left off, it says, "Returns: void".

Personally, I favor the lower-case here. Perhaps just because it's what I'm used to.
You don't have to use a global variable to point to this function. Just create a normal, local function that you can get at via closure.
Is there any way you can avoid writing "if (fs == null)" in three different places? For instance, perhaps you can setup your callbacks only after you successfully initialize the filesystem.
Does it make sense to make ownMouse a getter that returns "canvas == document.webkitPointerLockElement"?
I could make this it's own method, easier to read and know what's going on.  Same as the embedded document check below.
Dado que a gente falou sobre polimorfismo, o código abaixo não tá legal :(

Se vocês tem triângulos diferentes, o que implica comportamentos distintos, o correto seriam 3 classes de triângulo distintas.
I didn't see a range defined in redis' incr. I'm down for removing unless you can find any mention of an increment range.
The rocket ships got misaligned here
Good catch, fixed in 1baaff379a8848d636f3c650e571ad6a68628ec1.
remember to free funny_data and what's contained (everything you allocated when you created it)
ew again
The docs are outdated and need to be fixed. And as it stands, that signature doesn't even work with bool, because you actually need a second overload of `opEquals` which takes `const S` (either that, or you need to use `auto ref` instead of `const ref`).
Actaually I'm a bit divided as well but the motivation is as follows. 
The idea was to add a family of functions with Into in the name. 
While I thought that we might overload them allright the problem is that 
overloading doesn't look right in this case.
Imagine reading source code where Into/non-into versions are intermixed in code.
Another case for Into  is if we _might_ have collision overload somewhere later on.
Thanks for the explanation. And yes, it will be really good to have this in docs, if you will be kind enough and will manage to find enough time.
Please add:

    extern (C):
    @system:
    nothrow:

Here at the top.
You don't need to do it, but many of us consider it good practice to directly initialize values rather than relying on their default initializer like you're doing here. The code will work either way though. It just makes it clear that you really meant `0` rather than that you forgot to initialize it.
Erm, if I may.

I try to follow the coding style to the best of my abilities, but I've also been taught that when you "split a line", it ceases to be "indentation", and becomes "alignment". That's my rational for those lines anyways, and it hadn't even occurred to me they even broke any convention.

Good thing for me; `if (` is exactly 4 chars long, so we have perfect alignment there ;)

I'll re-align tonight then, I don't mind.
```
"committers_to_ignore" => %w(committer@example.com)
```
Use the commit object here instead of hardcoding the value. That way we aren't duplicating it.
```
"committers_to_ignore" => [commit.committer.email]
```
would you mind pep8-ing it ?

e.g.

s/1<<1/1 << 1

thanks
Shouldn't we be doing this then?
The getCallerData function grabs only the pertinent information for the caller, the problem is that originally the username that was shown was being pulled from the call itself, and in some cases (~500 calls in july)  the username was saved as the wp_id, but this will override that information with the proper information from PSU person.
oh, 'is' is a DataInputStream and JavaDoc says it throws EOFException for readByte() used by readSignedVarInt(). So depending on EOFException detection of EOF seems fine.
Best to not suppress warnings on preg_match
Амперсанд в значении ссылки прилипает к типу.
Maybe the div class could be "gallery" so that someone can change the style for the gallery later. The image gallery is different than most pages (mostly images instead of text) and may require different styling (for example, underline the heading, extra spacing between sections, etc.)
And your thoroughness is well appreciated! If/when I can reproduce this bug, I like the idea of falling back to lang_LANG.
I'd rather not have this assertion be dependent on the original ``new_tags`` list being lexicographically sorted, that's a non-obvious dependency. Just make this ``sorted(new_tags)`` in the assertion.
These three lines should be replaced by `case = Case.objects.create(product=product)
This guards a large portion of the code. Better yet mention

    if (!aa.a) 
    {
        return 0;
    }

at the very beginning of the function, after which you get to handle the real case.
k, no problem then. :)
which i especially needed to loop through the date facets
we're not splitting filter strings with separators anymore
Function to collect all active date filters and calculate a new gap for the active date range. Perform a new solr call if necessary to return new range facet.
Well i thought about it and here is what I think

Remember its a gateway, so if destroy is called you won't be receiving any messages anyway, and realizing that these maps are going to be updated at runtime and trying to get the most juice out of it i figured its OK for now and we could address the destroy issue as part of your 'graceful shutdown initiative", but i could be convinced otherwise since new entries are only going to be added to the maps per thread and unless you have gazilion threads it won't be much of an overhead.
I am always unsure about declaring a utility class abstract. It gives the wrong impression to users that the class is intended for sub-classing.

I usually prefer:

public final  class ExpressionEvaluatingParameterSourceUtils {
   	private ExpressionEvaluatingParameterSourceUtils() {
		throw new AssertionError();
	}
}
typo genrally but in any case, just remove it 

When retrieving ...
No we do not
Another option that would be cleaner would be to inline this function in the constructor, i.e.:

```JavaScript
var titaniumSDKPath = ...
...
Messaging.on("projectProcessingBeing", function() {
    var jscaString = ...
```
JsonUtils.prettyPrint() wasn't used anywhere in the glu, linkedin-utils or linkedin-zookeeper code, so I thought renaming was OK. But I guess it could be used elsewhere, I'll restore the name.

In my mind, actions are for (often stateful) procedures, while nouns are for (stateless) functions. I always find it odd when I see object.doSomething() return a value... :) That probably comes from the fact that I spent more than a decade working in Eiffel (where stateful functions are like a sin [stateful functions break Eiffel's 'command-query separation' principle...], and should never be verb-like or convey the idea of an action).

I find it more natural that object.toCompactRepresentation() would perform some stateful action on the object, while object.compactRepresentation() would return something (while NOT modifying any state). 'prettyPrinted' was an attempt at noun-ifying the name... Perhaps I should have used 'prettyRepresentation()' instead, but that is apparently at odds with Java conventions :)

Also, I prefer the clear intent of 'prettyPrint(value)' always return a pretty-printed representation (no ambiguity), rather than having:
- prettyPrint(value, 2) return a pretty-printed representation
- prettyPrint(value, 0) return a not-pretty-printed one (the name is then misleading here)
- you get the benefit of being able to tell quickly what was meant by the code you're reading, and you can find callers more easily (you can search for one then the other, without having to keep track of what int was passed as 2nd argument, the int in question being dynamic at times...)
C'est quoi ce délire avec les fonction write alors qu'elle n'ont pas été modifiées...
J'espère que c'est pas encore un coup de "je me touche les fins de ligne"@git-teube...
needs esc handle.. should be somehow in superclass anyways
personally liked the blue for a bolder .navbar 

...if we go with #333, then the .hover color on "a" tag should be darker
Oh... I misread the comment, never mind, but I see tons of tests failing for io in enterprise perimeter
According to this one
I have done that due to @davehunt suggestion to have tests using the accept-language header to determine the locale and this test (test_request_ends_with_slash) is running on background using urllib
For grabbing from footer select pulldown I have created this one - (test_locale) https://github.com/stupchiy/marketing-project-template/blob/fixes/tests/test_file.py#L29 , which is using WebDriver
So let me know if you want to remove some test or you want to do them both different things or any other suggestions... Will change it when needed
You could (should?) use the Filesystem component.
default values for options can be set in the configuration of the options, which is better as they will appear in the help of the command IIRC
Please take out the trailing comma, it's good style on multiline, but inline it looks strange :)
const T&
думаю, что эту сложную конструкцию нужно унести в роутер.
rightmostChildValue()
нужно немного расширить интерфейс роутера, чтобы стало более читаемо
This seems to have an integer mode, and Object[] parts as paremeters, but it is not used...
OK, I didn't understand that from the existing doc, so I need to clarify this section a bit. 

If I call:

    myWebView.stopLoading(true);

Then myWebView no longer exists? Or the iFrame displaying it no longer exists?

In other words, is there any way to revive the web view after calling stopLoading(true), short of creating a new web view?
https://github.com/phpBB-Social-Network/phpBB-Social-Network/pull/108/files#r1928893
thats strange because i have pulled and my repo was up to date before creating a new branch and editing it
Tracked in issue: blackberry/BB10-WebWorks-Framework#126
No valid reason.  Browser code just has a different copyright.  All files set to our copyright now
We don't have jsDocs anywhere else in the framework. I think we should remove these