labofsahil/github-event-dataset-2012 huggingface dataset:
Before we go down the Regkey route, we should walk $PATH looking for `git.exe` or `git.cmd`- this strategy also isn't foolproof, but it's a bit more resilient
I will send another PR for this!
Should you use `php_url` here? Not sure if there's a rule about where to and where not to use it.
This outputs an extra dash on the Home page of the style guide:
    <title> — Mozilla Style Guide — mozilla.org</title>
Unnecessary, the for loop takes care of this case.
It looks like the celery task is keeping track of the # of retries and will bail out once you hit the max. You just need to tell it how long to wait each time before retrying.

This all makes sense!
ok cool this makes sense to me, as something we want to override outside of the checkout
exports.top = function (req, res, next) {

虽然目前的代码规范比较差，但是新增加的代码需要严格遵守。
What part? I only added the lines to double check the writability of some blog subdirectories ..
Good catch here foobarHD.

Yeah Ryan, I just went through and made sure everything was switched for v2.2. So while 2.1 used lowercase we need to use all lowercase now.

Just a thought: `return (bool)  preg_match("/^([-a-z0-9_\-\.])+$/i", $str);` would be a little more readable and concise and should do the same thing.
This is correct and should not be changed to -1.
Fixed by 8c1a330d18f044417c2230f7b73f130b16aeff5b .
This looks at your route definition and validates that every defined `path_param` is actually a `:blah` param in the route string.  Maybe it's not needed but I figured it was good to verify that the `path_params` you have defined are in fact part of the routing path.
this one should not have a default value IMO
this must be an array, otherwise you get an error when using it in ``array_merge``
something seems wrong there, bad rebase?
``get`` ? creating the service ? This is likely to be broken
This proposed change is very elegant, no BC and ``setValue($foo, true)`` works with (right behaviour) or without (bad behavior in some cases) this change. But I agree that it's not a nice solution. So should I leave ``setValue`` as alias for ``upload`` and create ``setFilePath``method?
why doing a count before the loop ?
the default value should be ``null`` to mean the default connection IMO. Nothing enforces to have a connection named ``default`` (and this is a regression as it was handled properly before)
use a newer version of Twig. It provides Twig_Function_Node (and uses it)
I only do this type of  ``` switch(true) ``` when it adds more readability to the code instead of having the whole bunch of ``` elseif () ``` lines followed by the long line again and so on as it allows for better separation when going quickly thru the code.
Do you think it's necessary?
@jalliot Do you mean this one? The first parameter isn't optional, so that wouldn't be a problem.
add_index追加しておきます！referencesは今まであまり使っていませんでした。。。

以下のように一括で生成した方が良いのか。。。
rails g model topic user:references:index
I like the idea of letting users just put a boolean value to get a default or a string to get their desired format.
No, i'm not sure. I thought the score will be some probability measure that a user likes an article, so implemented this way
你使用的是win系统，所以才有微软的那一套字体，对于我而言其实一样因为没有微软的那一套字体会带来很多问题，不过正如你所说，还是留给用户选择好了，确实不应该包含在cls中，我会在下一版的sample-bachelor.tex里添加的，可能是我之前想的有点偏激了……囧……原谅作为一个程序员的洁癖……
users_controllers ii apartine controlerului Sessions de la devise

```rails
class Users::SessionsController < Devise::SessionsController
end
```
sa mai fac un controller numit users nu o sa mearga, sau fac unu user sau profile.. 

users_contrller de la devise nu e disponibil in aplicatie, cel putin daca scriu   url/users in browser  primesc eroare uninitialized constant Users.
I think a more informative name is good here. _create_status event or something.
That should be modified in an other commit, it is not directly link to that one
Why include 4 channel cases if we don't want to process files with alpha?
Something has gone wrong here.
Docstrings...
why not turn this into an actual js boolean? (Using an if statement)
Not sure what is the best practice. Please suggest what's considered best practices.
I chose "disable" in order to keep current behaviour on default false value. But as noted, it can be changed, considering also other search options like @search_contains.
That's better - i'll do that!
There are problems with expression handling here. Both ARCHIVE and MODULE support expressions, but this OSH isn't resolving them and RaOperationUtil also isn't.

I'll comment again in a bit with a link to a possible fix.
To fetch each locale dictionnary, t would be easier to somehow expose this anonymous method, call it from build file:
  https://github.com/fabi1cazenave/gaia/blob/9b06329a9c3973a24b1a2f3038767bc3ca5bdba1/shared/js/l10n.js#L242-L267
It would allow to not expose gL10nData in l10n lib and simplify a lot this code.
I think that everything can be synchronous and be done in the for loop used to initialize `l10nDictionary`:
  https://github.com/mozilla-b2g/gaia/pull/7166/files#L1R213
This is for console.log in workers and is stolen from nodejs and modified to work under use strict...
but we would be breaking the semantics of the first level child directories within Comms which correspond to entry points, thus I would prefer to keep the js file within the app that makes more use of it, in this case Contacts
It might be a performance drawback if I add the pending message query, merge and sort two different message sources, then callback the final result... It would be easier to implement this way but user might get crazy especially we need to query the DB frequently without cache. I will try to append the pending messages after the list show up, and only show the pending message in message thread view(not in list view, do you think it's reasonable to display if the latest message is pending message? ) But even we reduce the effort in thread list view,  the UI logic will still much more complicate than before(and we have to remove it if sms DB could also handle pending message someday).  I will try to find out the reasonable solution for displaying the messages. If you have better solution for this, please inform me ASAP, thanks.
My opinion is that using a bogus href is still a shame but I won’t bikeshed about that. Looks good enough for an unstable BB imho.

@pivanov r?
I would recommend to translate all the strings using the l10n. 
You can add a commit in this pr or I can do it when this one lands, up to you :)
Remove this `if` entirely. we not support the Back key anymore.
I belive this is not needed since you don't use it in this application.
What do you mean??? Sorry, maybe it's a bit late... ;-999
Why `Shared code II` ?
give function a name please.
`!==` :-/
s/rencontrée/rencontré/
Why is there always this little arrow after a 'a'? It does not make sense.
yes, we need to do that
You seem to be using an old building block here, which has just been landed as `shared/style/confirm.css`. Could you please use it instead of the current HTML structure?
nit: add a |return;| and remove the else
Hi Tim,
Event handling part is refined, thanks.
Yes, because we need the FB data ready if a pick Contact Activity is invoked.
Why does it `exists()` if you deleted it?
You can get the precision with:

    expr.get_precision()
As we discussed, the perf loss here is too minor to be a concern for this class
The repo named makes sense when we want a modifiable repository so that they are put in unique locations. They're probably pretty useless since the repository always gets deleted. so we might as well just make up random names for it. That said, I don't want to change the API since it might break stress APIs
why doesn't the current version work with your modifications?
Yes, it's printing help now as well:

<pre>
$ razor -nc
invalid option: -c
Usage: razor [global options] [slice] [command] [command options] ...

Global Options:
    -v, --verbose                    Enable object verbose output.
    -d, --debug                      Enable Ruby stack trace output.
    -w, --webcommand                 Accept web commands.
    -n, --no-color                   Disable console color.
    -h, --help                       This help message.
</pre>
这里边的 body 部分移到 mkbucket 里边的 body 里边去，最后再执行 rs->drop 清场
Becouse it's look like x: xxx.xxxxxxxxxxxxxxxxxxx, y: xxx.xxxxxxxxxxxxxxxxx, z: xxx.xxxxxxxxxxxxxxxxxxxxx :) too long
without fix it's look like java object Location in text... not coordinates
c&p alert! ;) please modify to `should check for crash end`.
bites -> bits
Note that having this upper bound also makes it reasonable to use something like SortedTakeMonoid which has a fixed size (or, a fixed-size max-heap representation).
good idea...done.
Referring my repository was necessary since it's not merged yet :)

Referring had was the easiest solution -- feel free to refer a specific revision.
This ZipArchive argument is totally useless as you are deleting it just after at the beginning of ``reinitZipArchive``
Throws an error:
Commented a line in jqGrid because it throws an error: 
Uncaught TypeError: Object <(subclass of DS.Model):ember402>,<(subclass of DS.Model):ember407>,<(subclass of DS.Model):ember409>,<(subclass of DS.Model):ember411>,<(subclass of DS.Model):ember413>,<(subclass of DS.Model):ember415>,<(subclass of DS.Model):ember417>,<(subclass of DS.Model):ember419>,<(subclass of DS.Model):ember421>,<(subclass of DS.Model):ember423>,<(subclass of DS.Model):ember425>,<(subclass of DS.Model):ember427> has no method 'getURL'
Depending on what we're using, templates might belong higher up- in a node.js project, for example, we're using them for rendering server-side, or in a rails app, we might render server-side as well. This probably belongs i app/assets/templates rather than app/assets/app/templates. Might be worth checking with @spikebrehm on where he wants his templates for node projects.
Sure - actually, I'll merge it and fix it up.
I agree, better test for ``self.codebases is not None``
In base.py (line 231) an extra change_filter is added on codebases when self.codebases is not None (to prevent changes to come in with a unknown codebase). Thus if codebases is set to {} then no change will ever be processed.
I wonder if we want to grab the json incantation from `buildbot.util` for this.
I think it’s even slower than standard for loops which do not the passed array at all.
It doesn't compile in Xcode if I leave them in.  It can't find curl.h.
would you then even put config into a private component or expose it as a global? see #5
To be perfectly safe I think this should be getNames and not getName, that way if a package is installed because it provides something you require, it will still match .
> > And, do you have commit privs or are you an outside commenter like me?
> 
> I think that I have, but have never exercised it. We can let this pull
> request happen, and
> then I might clean up.

You show up as Repo Collaborator, so you should be able to merge the
2 commits in my pull into droolsjbpm master, then do any further clean-up 
you want.

Thanks
Ian
you missed $("#project_path") I think
There's no need for this kind of clutter. Just the include will do.
Good to know, I thought we use actually all of them. :)
Maybe we could implement `setTabTitle(tab, title)` too ?
URI_SAFE_FOR_UNTRUSTED_CONTENT means that regular web-pages can link to the new about: page. You should not use this flag if you are giving your page system privileges as I think it opens a potential security hole (certainly a spoofing risk).
It's little strange that you use both nsIFile and this module wrapping it together. I'd suggest to stick either one.
this is a nit.  whitespace changes inside feature branches makes merges harder.  especially for something as big as this, we should avoid them.
That reminds me that we should move away from a guid and use a name like 'nightlytestertools@mozilla.org' for the id. Would you mind filing an issue for it?
not needed here, just use sklearn
yes you can put it below the footer.
Given params and options are literally identical, is there any reason to create the new map?
"the 2.8 series" is Telly. :)
Does this mean that the run_mode that a user sets on the command line will be overridden by the application? I can understand why that might be good from the application's standpoint, but doesn't that just cause aggravation for a user because their command is not being followed?
What I mean is I think this code is just `Application.load`. It doesn't really need to be *extracted* anywhere.
The platform is darwin. Why would `#windows?` return true? (the test itself actually asserts that it should be false).
This function has no effect, is it correct?
`.replace('hidden', '')` is fine if we're going that route, I would just globber the classname in this case since we control it all
So we delete `portname` and assign `PORTNAME` to `chan_mode` instead? That's wrong.

Now I know that other related code is messed up already, but we must not add to the mess. Either fix it right away or at least explain the situation and have a plan.
My bad... I fixed it.
No. This class is not intended to be instantiated -- and we can enforce that by making it abstract.
`@link` is an inline tag that must be surrounded by brackets. Maybe you want to use `@see` here
CS, spacing

if ($this->getWriters() === array()) {
Should we be removing the Zend\Memory docs? we haven't removed that component (yet)...
It is already inside existing CraftBukkit comments.
Options -> AbstractOptions
@samiluukkonen Tääki on vähän turha ku kaikkialla mistä tätä funkkaria kutsutaan on jo laskurow haettu. Eiks sen vois päässätä parametrina?
The README and the UPGRADE files shouold be distributed. They are meant for users
which action? s_login is used if someone tries to submit an article without being logged in
Indeed it must. But it was there before I came ;-)

To do...  at least the PR makes it possible. Not a blocker.
I don't like it split either, but it's too difficult to understand in its current form.

Is it acceptable to move all that into a separate Bash function, with a meaningful name, ideally receiving the file name as an argument?
HPC LDAP, and ship project schema
Fixed with a ' '.join(cmd, makeopts)
Instead of changing an existing test, please add a new test that specifically tests the bug you are fixing.
Mainly just added a comment
Please change this parameter to be `haaaaaaaaaaaaay sexxxay lady!` Thank you.
I don't see any rhombi?
How similar are the following methods (`setTypeFields` and `convertValueToType`) to the existing private methods on `Component`? If they're similar, then we should have just one impl of each that is called in all the places. For example, it seems like the list of field names to exclude might need to be passed into the method.
Organize it better - keep public/private, static/instance fields and properties separate.
Were these %p tags there? Do we need them?
Казалось бы, что он и такой chooser может написать, нет?
лучше завести структуру. с информацией о каждой вершине
This should be (By.ID, "upandcoming")
I do agree though that readAll() makes sense
Martin, es mejor si aca dejas get_actives_plugins y lo dejas mas o menos asi:
def get_actives_plugins(self):
    warnings.warn("deprecated in behalf of a typo free method name", DeprecationWarning)
    return self.get_active_plugins()

Para no romper plugins existentes, creo que lo razonable para cambios de API es dejarlo un tiempito.
Think we should reference "TerraFrost" as the name?
Unsure, I just refactored it to use the new interface. It runs equivalently.

Removing it is out of scope for what I was doing but feel free if it's obsolete!
Will this line be parsed properly if there's not `@return`?
It doesn't make complete sense as to why checkPassword would be returning the user.
Also spacing is a little messed here
Please add JSDocs for these.
These tests I think can also move to oae-config-aggregator along with the init stuff when that PR is in.
The way we're using it, it can only be the first segment of a path, so it must be a String. We could make it more general and allow full paths for the prefix, but it's not necessary for what we're using. I'll implement it anyway. But I'll keep using a default parameter instead of `Option`, it would be too clumsy to wrap it every time in a `Some`.
That is what I came up with, but I'm not convinced I like it :-)

```scala
private def scanBackward(pos: Int): Option[scalariform.lexer.Token] = scan(pos, _ <= 0, _ - 1)
private def scanForward(pos: Int): Option[scalariform.lexer.Token] = scan(pos, _ > tokens.length, _ + 1)

@tailrec
private def scan(pos: Int, shouldStop: Int => Boolean, shiftPos: Int => Int): Option[scalariform.lexer.Token] = {
  if (shouldStop(pos)) None
  else {
    val token = tokens(pos)
    token.tokenType match {
      case WS | LINE_COMMENT | MULTILINE_COMMENT => scanBackward(shiftPos(pos))
      case _ => Some(token)
    }
  }
}
```
야머에서 슥 봤는데 `type` 이 없어서 그랬던게 아닐까여?
`POST` or `GET` 이 없는데..
Do we need to drop and create? Does the DB dump from Heroku handle that for us?
the phpbb_dispatcher global should be on a new line.
Good point. I see Peter has removed this, so I could wait for it to get removed completely. :) I wasn't thinking about it :)
zorgt voor de juiste grote van de colission box van de maincharacter
Я лишь хотел спросить, что за число мы получим, округлённое или нет. Вы просто должны понимать, что делает ваша программа, вот и всё.
Эта структура пропустит первую ячейку массива (индексы начинаются с 0).
Вы создаете переменные _dnkArrayLength и _nukleotides в каждом объекте класса  Cell, но храните в них одни и те же значения. Почему бы не сделать их константами или статическими переменными?
I think that we don't need to copy `Am` to `Amt`. I think that all we need is to copy `Am` to `c` and then pass `c` instead of `Amt` into `dsyevd`.
This looks wrong to me, what is this doing here?
I don't know about this... we shouldn't wrap the line so quickly.
これ結構いいですよ: https://github.com/stouset/twitter_bootstrap_form_for/tree/bootstrap-2.0
Can you please use dot syntax here: `NSError.class`
Remember this syntax is only supported since Xcode 4.5 :)
On 19.11.2012, at 21:39, Eric89GXL <notifications@github.com> wrote:

> In mne/connectivity/spectral.py:
> 
> > +    for m in method:
> > +        if m == 'coh':
> > +            accumulator_fun.append(_coh_acc)
> > +            normalization_fun.append(_coh_norm)
> > +        elif m == 'cohy':
> > +            accumulator_fun.append(_coh_acc)
> > +            normalization_fun.append(_cohy_norm)
> > +        elif m == 'imcoh':
> > +            accumulator_fun.append(_coh_acc)
> > +            normalization_fun.append(_imcoh_norm)
> > +        elif m == 'pli':
> > +            accumulator_fun.append(_pli_acc)
> > +            normalization_fun.append(_pli_norm)
> > +        elif m == 'wpli':
> > +            accumulator_fun.append(_wpli_acc)
> > +            normalization_fun.append(_wpli_norm)
> Sounds reasonable to me -- I was hoping @mluessi would say it would be easy to make it class-based an just implement it, but I too doubt that that's the case :)
> 
> @dengemann , are you going to be doing an all-to-all connectivity using labels (e.g., the aparc 2009 labels)? If so, that's something I had planned on implementing and using, but I'll hold off if you're going to do it.
> 
Keep going, sounds (damn) cool! I'll start with a  seed-based approach aiming at psts / tpj which is driven by previous fmri / meta-analytic work from our lab (and partners). But i'd be happy to get into what you're pointing at (esp. with regard to ASD populations we're dealing with).
> It would be nice to be able to add that type of functionality, basically taking in a set of label names (or instances) and compute all-to-all connectivity for them in different frequency ranges would be really cool -- and it's the sort of thing we can add pretty easily once this gets merged, if people thought it woulrd be useful.
> 



> Perhaps we should open an issue to discuss these types of future extensions / functions
> 
yes, great idea. this stuff certainly is leading / cutting edge. we should keep up with this!

D

> (and let this one be merged if @mluessi and @agramfort are also happy with it)?
> 
> —
> Reply to this email directly or view it on GitHub.
>
We want messageService to have mailSession injected by Spring (see my comment below).
It should call getDeathdateEstimated.
empty return tag will still produce a javadoc warning
Probably should also mark w/ @Deprecated annotation
why changing it ? It is done on prupose to move the file only once the entity has been persisted. This avoid moving the file if the transaction is rollbacked because the query fails.
缓冲写入
写缓冲
You also need to do this:

override def get(system: ActorSystem): CamelExtension = super.get(system) to get the Java API (return type will be barfed for Java otherwise)
Ну давай тогда уже на выход из канала такое же повесим. А что? Отметит как
не прочитанные и по дате отсортирует. На маленький. Мы же не удаляем их, а
отмечаем как прочитанные
А иначе ты его не убиваешь?
That could be done while working on the constructor :)
"service" is correct. "service instance" is not correct and should not be
used: use "container" instead. It's less ambiguous, less overloaded, and
closer to technical reality.

On Wed, Aug 29, 2012 at 12:42 PM, Louis Opter <notifications@github.com>wrote:

> In dotcloud/ui/parser.py:
>
> >              parents=[common_parser])
> >      open_.add_argument('service', nargs='?', help='Specify the service')
> >
> >      # dotcloud run service ...
> >      run = subcmd.add_parser('run',
> > -            help='Open a shell or run a command inside a service instance',
> > +            help='Open a shell or run a command inside a container',
>
> Why using container here (and below)? Service / Service instance are what
> we use everywhere else for user facing stuff, do we want to change that?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dotcloud/dotcloud-cli/pull/17/files#r1489774>.
>
>
This line is epic
+1 im NOT a fan of variable reuse like that. I think only @ed- does it. :P
I didn't even know I had one of those.
I don't think it's an option. It's contract breaking to _not_ show on create. :(

Sent from my digital shackles. 

On Jun 19, 2012, at 8:13 PM, Tim Simpson<reply@reply.github.com> wrote:

>> @@ -366,8 +365,6 @@ def create(cls, context, name, flavor_id, image_id,
>>         service_status = InstanceServiceStatus.create(instance_id=db_info.id,
>>             status=ServiceStatuses.NEW)
>> 
>> -        dns_client = create_dns_client(context)
>> -        dns_client.update_hostname(db_info)
> 
> After create you won't see it, no. It will appear once it's ready or not appear at all should something go wrong.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/140/files#r1014918
Do you really need to move the `des` declaration outside of the `try` block? It doesn't seem that you use it for anything (here or in the other tests where you made the change).
Remove @since tag?
Don't you want to stop waiting at some point?

I actually consider allowing interruptions a plus, so that it works with Future.cancel(), but that's your call.
Again, `new(SimpleT)` is the Go-ish way to do this. But don't worry, I know you stole this code from me... ;)
The old code results in a number with 11 digits, whereas the new code results in a number with 10 digits (on my system anyway). Just wanted to point this out in case it is an issue.
same here https://github.com/zacc/moztrap-tests/blob/68bec9950e23e5d41d98ec084cf1ed4de885512f/tests/test_homepage.py#L58
rather than putting the same code in 6 time we could maybe just check this in navigator or in the motion system.
isolate -> isolated
i and l are defined already. So it is enough to do

    for (; i<l; ++i) {}
It is necessary to catch remote errors here and in other event handlers. E.g:

```js
if (data.success === true) {
    // do something
} else {
    PMA_ajaxShowMessage(data.error, false);
}
```
Opted for the change in the require function.
@bp323 please fix the white spaces & double quotes in this file.
changed to Newscoop\EventDispatcher\Events\GenericEvent
Ah, yeah, I've still got my tab set to 2 space.  Are we sticking with 4 space everywhere then?
There is `Object.isUndefined()`
Are you sure ``'Zend\\Mvc\\ApplicationInterface'`` has the correct type as ``string`` is not a sub-type of ``string``?
Are you sure ``$request`` has the correct type as ``object<Symfony\Component\HttpFoundation\Request>`` is not a sub-type of ``string|array``? I found this a couple of more times on lines: 154, 218, 282, 342.
This alert is just added in order not to be surprised when not everything will work. Just to inform that it is not still implemented.
It's due to memory usage in the node process. We're also using haproxy for request queueing
Is this hard-coded token right?
Tässä käy ohrasesti jos esim. `(pred nil) ;=> true`. Helppo korjaus:
~~~clojure
(pred (first (filter pred a-seq)))
~~~
Is the three-part format required for a version number or we can put "6.2" or "7.0" here?
Do you mean

    $asset = JFactory::getApplication()->input->getCmd('option');
Double period?
I'd prefer `({map: 1, …})[name]`.
oh, yeah. fixing now
Hm, but you know that `user.connections` is `[]`? Why setting it again?
It should be:

```
msgstr[0] "pred %s dnevom"
msgstr[1] "pred %s dnevoma"
msgstr[2] "pred %s dnevi"
msgstr[3] "pred %s dnevi"
```

Probably?
AA, i dont know what you think about with ALL_CAPS :) - caps lock
Those are not views. :-) Just leave `http` as it is. ;-)
If you are fetching data here, then also name this method/function like this. You have `get_horoscope` but in fact you do `fetch_data`.

Name it then `fetch_horoscope_data`.
might be more clear to return null here
Kopieren von Backend?
Referring to [ruby-doc](http://www.ruby-doc.org/core-1.9.3/Regexp.html#label-Character+Classes) `\s` matches any of following white space characters `/[\t\r\n\f]/`, which should be just fine.
Viel der Logik zum Formatieren muss in allen Outputs (Formatters) dupliziert werden. Wie können wir das verhindern?
what are you getting this for? Looks like the line could be deleted
CAPS ? by convention, caps are used for constants if i'm not mistaken...
what's the bind version shipped with debian 6?
Why is this no longer a HudsonTestCase derivative?
Agreed. My first attempt on writing this article was more on focussing on the differences for programmers who didn't get in contact with JS yet, but then I thought I might miss the point
file should be something that uses a reserved token from lucene, or something you declare as reserved. If a user enters 'file', 'storage' you don't know if they're queries or a path.
- Description
  Link to comment/ issue
  Link to file
I don’t think this is valid Objective-C. ;-)
Maybe leave having some kind of config for another PR?
I put not now in the wrong class
"not now" refers to replacing aic and bic in MNLogit with the generic MultinomialResult version. I think the df_xx should be fixed.
Maybe then aic in MNLogit will also be correct, but I didn't go through the roundabout calculation (first subtract constants than add them again)

I can try df_model also tomorrow:
The main df_model is an attribute of the model instance. Initially I thought you have nnz_params in the model not in the result.
what should work right now, from my reading of the source, is to add after ``self.nnz_params = (self.trimmed == False).sum()`` in the ``__init__``
``self.model.df_model = self.nnz_params - 1``
and 
``self.model.df_resid = self.model.nobs - self.model.df_model``  #I'm not sure where nobs is attached.
so the two df are correctly updated

the more appropriate place *might* be to calculate nnz_params in fit() of the model

did you try to attach df_model as a property (decorated ``def df_model``)? I guess that doesn't work because we assign to df_model.
Did you follow an algorithm or a spec for this def?  If so, please give the URL here :)
Better written without the loop as:

opts[:prefix_key] = opts[:namespace] if opts[:namespace]
opts[:prefix_delimiter] = opts[:namespace_separator] if opts[:namespace_separator]
When does `loadsettings` happen?
why add the module ?
Ah, good spot, will change thanks.
We might be coming at this differently based on different experience with cosmology calculators?  The two that I typically use always produce power spectra assuming that radians are the natural units, and the one could I have dealt with more for generating shears from a power spectrum took the input galaxy positions and converted to radians.  So I tend to see radians as being fairly natural units that a user might be working in if they are using outputs from a standard cosmology code.
Yes, I agree about a more systematic treatment to unexpected parameters.  And now that you're mention it, you're right that it should be an exception.
does it work without the "sh" ?
As discussed, for consistency, let's make this a link as is done for App commenting.
you can also do `Closeables.closeQuietly(testLocation);`. These days, an SshMachineLocation keeps its ssh connection objects around so they can be reused. Closing the location will close all of them. If you don't, they should be gc'ed and closed anyway so no big deal.
Change sysout to log.info
Should this be cobertura.user.java.nio, instead of obertura?
Yeah, you're probably right. Ideally I would like to have different approach than what I'm using here. Having to create a new parser and to call parse seems a bit of an overkill. WIll have a better look at git-like-cli.

I'm open to suggestions in the meanwhile.
Done. I guess this is the order defined in the AbstractSoftwareProcessSshDriver.
becausing ? :)
Another Lore engineer here. I just wanted to chime in that this is the first I've heard of using syntax to make the function/procedure distinction, and I think it's pretty darn awesome.
Some files in dwt.internal.mozilla package does not seem to become so. (Them contains windows line endings)
Because I converted line endings of them, them should be correct now.
nope: siehe bsp unter base rules.

a:hover ist keine state rule im sinne einer eigenen class die wir einführen, sondern ein standard pseudo-element.
braucht auch kein javascript
needs to be here : ```path = require.resolve(filename)```
Whoops! Okay, restored, and the tests still all pass.
Can you find a better name for this? I'm not sure if it's the best one...
Doesn't this handle that already `if ( options && (options['yes'] || options['no']) )` ?
should not this file be in webinos/test?
this method is not called from anywhere. so I assume it's oxid's convention of autoloading classes. please confirm :)
something readable.
Add those messages to |this| as the target and redirect them in the handleEvent method to not have to use bind.
I think eventually we'll want to move this validation client side.
Will it work with all installation methods?
http://www.redmine.org/projects/redmine/wiki/HowTo_Install_Redmine_in_a_sub-URI
specifically, Passenger's RackBaseURI?
```coffeescript
allCss = (changedFiles.length isnt 0) and (changedFiles.every (file) -> file.type is 'stylesheet')
```
Ok ... sonst immer für alles regions da ned ... :smiley: mach ma.
Let's stick to single-quotes for consistency.
According to http://gembundler.com/, the lockfile should be included, because it's used as a snapshot of all the required gems and their dependencies. This way bundler doesn't have to recalculate all the gem dependencies each time you deploy, etc.
Same comment for all the OAUTH_* constants
Should the test comment be moved to here?
<code># Now we start to test the marketplace pages</code>
hopefully のニュアンスを入れるか入れないかですよね。わかりにくいから省いたならOKです。

それらは業務プロセスについての状態やうまく行けば振る舞いを保持します。
in-memory　は「メモリ上の」でよいかと。
参考：http://eow.alc.co.jp/search?q=in-memory
Need info: Why not make one filter object with objects for all properties?
filter: { 'regions' : [], 'countries' [] }
Maybe not for now, but is we add more search filters. Right?
On 08/10/2012 16:58, Helen Flynn wrote:
> In docs/sphinx/loci/bio-formats-fiji.txt:
>
>> @@ -0,0 +1,21 @@
>> +Fiji
>
> Good question! How do I delete pages??

In git?

Do "git rm <page>", where page is the name of the page.  And then "git
commit" like normal.  Remember to delete it from the toctree at the same
time!


Roger

The University of Dundee is a registered Scottish Charity, No: SC015096
What does `null` mean in a ManyToMany field? I suspect it's unnecessary?
Okay, I am an idiot... I set "parts" to null anyways right after... splice is a waste lol
This should be `@param`.
Faster version maybe?

```C++
if ((f->mod & (MODconst | MODimmutable | MODshared | MODwild)) && !isThis())
```
There is no need in explicit for copy ctor
apidoc/Titanium/UI/Font.yml
Why not CFLAGS += ...?
Actually, `Doctrine\Common\Util\Debug::dump(array($key => $value))` should already produce what we want
still has an empty line ;)
Ah, yes! Glad you spot that. I was meaning to ask you where was the writing place, but I couldn't find it (well, actually, forgot to look for it more thoroughly is the right term).
Same as before change I think, but I guess if helper->numRows() is 64bit  (inconceivable that a temp table really will be!)..
then currentRow/lastRow should also be 64bit. Even though there's an even small chance of it being significant.

You probably get 'possible loss of data' warnings at the moment
These lines will possibly break with LN Builds due to the "."
The ptrs is the input of  the dfuhelper->addSuper() and the dfuhelper->removeSuper() (implemented in dfuutil.cpp). Both of them request char**.
It's supposed to return an array of IFileIO objects for all the file parts. With this patch that array is going to be empty (well, will just have the dummy NULL value in position 0 - I forget why that is there...) which is NOT the correct behaviour even if there are no dali file descriptors.

Code elsewhere needs to make sure that this function has the information it needs to be able to return a valid IFileIOArray. That most likely should be done by creating an IFileDescriptor to describe the locally resolved file...
Now I get what you meant and yes, you're right :) 

I personally would do a massive refactor everywhere that pattern is used in a different branch.
And those two lines raise two notices as well: 

<pre>
Notice: Undefined index: include_fields in /var/www/mediawiki/extensions/Bugzilla/BugzillaQuery.class.php on line 215 
Notice: Undefined index: include_fields in /var/www/mediawiki/extensions/Bugzilla/BugzillaQuery.class.php on line 216
</pre>
Why on earth does it have all these HTML comments?? :)
More pressingly, the `&` should be `&amp;`. Eg.

```
...:{{version}}&amp;query_search=signature&amp;query_type=c...
```
@seancribbs If we do that, we'll end up sending all lowercase headers in our responses (since `Webmachine::Headers` automatically does `key.to_s.downcase` for keys).

Although not strictly a problem, since the spec for headers indicates they are case-insensitive, there certainly may still be poorly written agents/clients that don't handle the lower-case variants properly.

Thoughts?
Fair point, can you add it to other Vcs downloaders though?
Fantastic! I was completely unaware of the `setup.py` behaviour.

I probably need to do a clean-up of `buildout.cfg` at some point.
I think the string "ABN: " should be put before the abn value
Looks like this is cleaner syntax, we also don't have to use .erb files anymore for CSS
Are the above three selectors being redeclared by this plugin?
Just doing `(has-author? book author)` is enough.
please add a space after 'if'
Respecting it how?
According to description the msi will be placed in "data/posts", not "data/exploits"
fixed, sorry for that. not sure how that happened, I have all my editors set up with spaces instead of tabs (I thought)
I must extend `Inject` or type checking trows error here:

```php
final class InjectParams
{
    /** @var array<JMS\DiExtraBundle\Annotation\Inject> */
    public $params = array();
```
Yup, fixed this.
I think it would make more sense if it was a PUT with update=1 as parameter. Then there is no confusion about what it does? Because we may want to add an API for creating packages, and would make more sense as a POST /packages/foo/bar/ with the package details as request body.
"its silly friends".
These are bad names for arrays since in every other primitive, `_va` is an actual `VertexArray`, not an array.
What is `valid`?  Can we return `undefined` or `occludeePoint` directly instead so we can avoid allocating the object literal?
Extra tabs?
Removing it will certainly decrease functionality but adding a StaticTextureAtlas doesn't really give all that functionality back.  What if I have a mix of single textures and pre-made texture atlases?  Why wouldn't I be able to combine them easily in a single atlas to use in BillboardCollection?
Is throwing an exception better?
It looks to me that the Geometry.getVertices and Geoemtry.equal implementation should be moved to Points.js.
unbumped in 3b906c89775f4939e850bd84ca1a5d03b9f4e97b
OK, it was just slightly misleading because it uses the "catch-all" pattern. I prefer handling only those exceptions that can actually happen. But no big deal.
Was trying to centralize the exception handling, but then only ended up calling it once. If it remains as it, agreed, it can be removed.
Multiple assignment, plus looks like a typo.
Parent should be set to null event if the current parent property does not point to correct proxy.
typeof will always return a string. So it is fine to use "==" here.
We support four of the INPUT_BORDERSTYLE constants: NONE, LINE, BEZEL, and ROUNDED. I think there may be other constants too though, and if so, then we don't support them.
Should technically be 2012 only =)
No strong feelings here.  I think it would be nice to centralize the logic in one place but since it's only one line of code, it's not a huge deal.  The IDE compatibility is probably not an issue for very many people around here :)

I never noticed the missing method names in the output though... I would be interested in fixing that at some point.
Could you delete this? And all the other `@Transactional` annotations in this class.

PR regarding `@Transactional` will handle this.
This should also probably either restrict the schema search to the ones found in the search path, or a custom defined list in the yaml
There's a lot of logic that directly keys off of the `default` value that is spread around. I would recommend retiring the "default" constant and instead replacing it with whatever the actual assumed version is (like: 4.3).

This would a significant enough refactoring to warrant separate commit IMHO.
I meant the else could (and IMHO should) be removed:

    if (t)
        break;
    return;
Well, someone else picked the words (presumably someone who can actually read them), so it could be complete gibberish for all I know - though google translate claims that they're "beer" and "pony." I just needed a test case for the bug.
Fixed in https://github.com/ElasticPortalSuite/BungeeCord/pull/51
Change the comment to: Deltaspike Impl. We use compile scope as we need its Impl
This seems in the wrong place.  There are repository sessions with no CP anywhere behind them.
@chillu Woops, that's a good point. I'll update the commit.

On Tue, Mar 27, 2012 at 12:14 PM, Ingo Schommer <
reply@reply.github.com
> wrote:

> > @@ -73,9 +73,13 @@ public function getEditForm($id = null, $fields =
> null) {
> >               }
> >
> >               $form = new Form(
> > -                     $this,
> > -                     'getEditForm',
> > -                     new FieldList($uploadField, new HiddenField('ID')),
> > +                     $this,
> > +                     'getEditForm',
> > +                     new FieldList(
> > +                             $uploadField,
> > +                             new LiteralField('AllowedExtensions',
> sprintf('<p>Allowed extensions: %s</p>', implode('</em>, <em>',
> $uploadField->getValidator()->getAllowedExtensions()))),
>
> Can we move the HTML out of the _t() call? I know we tend to do that in
> core, but its not a great habit as it leaves too much room for error for
> translators.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/silverstripe/silverstripe-cms/pull/78/files#r606365
>
Modify the commit to do as  you just suggested.
I don't understand why you need this hack and your comments don't help. Is it "comments panel" or "comments taaaaab"? Couldn't you call directly the listener of the "right" button or make it listen to a logic event (e.g. xwiki:comment:reply) instead of click?
You're welcomed. :-)

Hrm but I do have a Chef server in my dev environment, and AFAIK the intention is to support both scenarios?
If `GHC-6.12.3` is binary and thus doesn't actually require a compiler, this should become ```('GHC', '6.12.3', '', True)```.
This should go on top, above `name`, to make it more explicit.
Не raise callbacks, а fire callbacks )
This should also consider if there are no conflicts. For now, that could at least be a "TODO" comment.
This seems to be working like expected.
Please dedent one level (and you probably don't need those braces either).
Yeah, the name should be corrected. It was actually based on the previous state of schema where each update was actually a create since for each message in the group there was a record in the table.
This warn looks wrong - it looks like it was copied from 568 below ??

Should it be "Using a custom correlation key witout ...   " ??
Sorry, was looking at it from the phone.
This should've just been `timeout` (which is what I did). Alternatively it could've been `(int) timeout.TotalMilliseconds` but that seems pointless since there's an overload that takes a `TimeSpan`.
Pourquoi mettre ces lignes vides ?
getBanco()

e faltou pular uma linha.
Dev?  Wat?
`$appDir` should probably be escaped.
This should say "you can define jump as false"
why a blank comment
static variables are all uppercase with underscores to differentiate them from members => ``JACKSON_MAPPER``
are you just saying remove the parameters property?

cause you can surely do what you want using the eventsource property like so.

eventsource: 'http://myapp/{Year}/{Month}/{Day}/?f=czml'

and add client side code to handle the replace. You could probably still use the parameters tag to give hints to the client as to what each parameter to be replaced should be.
Mind letting this Ruby newb know what the above accomplishes? Is this a namespacing thing?
Yeah thats a good idea.
I wasn't sure about, what you were targeting, so I've not changed it (for now :-) )
I'd prefer a more meaningful name and a syntax like that:

```

var RestartlessWindowHelpers = {
  startup: function () { 
  },

  shutdown: function () {
  },
};
```

(you can also rename the file).
Will this cause any ref cycles? Is the connection reference on the table operations short-lived enough that this won't be a problem? We need to run the tools for that before we go on.
There are currently no examples/ dirs AFAICT, I guess you meant pyplots/? 

I copied what was already used in the scipy-intro section in several places. Propose to fix this all at once in a separate PR, is that OK?

Also, I meant to not show the end result (filtered image), just the original one. With the pyplots/ folder, I don't see how that's possible, except by not plotting the end result at all.
Ok I do that and keep a deprecated alias with the old name.
thanx  noted ..i kinda like single liner it more Pythonic
Is it necessary to invoke the callback when there's no place to put
the sidebar `IFRAME`?
Effectively duplicate return description here as well, please remove the top one
- pull out $user variable (= global)
Okej. Mitenköhän se sitten oli tarkoitus tehdä? 

Oma ratkaisuni yhdistää annetut listat martiisiksi, eli listaksi jonka ensimmäisenä alkiona on lista kaikista parametrilistojen ensimmäisista alkioista, toisena lista kaikista parametrilistojen toisista alkioista j.n.e.
Tämän jälkeen parametrina annettua funktiota sovelletaan jokaiseen kottuun listaan...

En ainakaan suoralta kädeltä keksi mitään tapausta jossa tämä ei toimisi. Ei yksittäisillä pisteillä niin väliä, mutta jos kokonaispisteet jäävät juuri alle 120 niin se kyllä vähän harmittaa.

Jeremias

On 24.11.2012, at 20.36, Jani Rahkola <notifications@github.com> wrote:

> In src/one_function_to_rule_them_all.clj:
> 
> >  
> > -(defn count-params [x]
> > -  :-)
> > +(defn my-map 
> Tässä tehdään nyt jotain niin monimutkaista, että en voi antaa pisteitä, vaikka testi menevätkin läpi.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Better to use. Array.filter
wouldnt it be required in the notice file then?

On Tue, Feb 28, 2012 at 16:07, pgrm
<reply@reply.github.com>
wrote:
>> @@ -0,0 +1,5 @@
>> +﻿<?xml version="1.0" encoding="utf-8"?>
>> +<packages>
>> +  <package id="DotNetZip" version="1.9.1.8" />
>> +  <package id="HtmlAgilityPack" version="1.4.3" />
>> +</packages>
>
> we don't distribute their software in source code form, only in object code form (there are only DLLs), so the licence wouldn't be necessary, but i still can add it to their folders if you want
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/loom-csharp-visualstudio/pull/9/files#r496417
W List<T> jest metoda AddRange();
Wordt deze ergens gebruikt?