labofsahil/github-event-dataset-2012 huggingface dataset:
[Default](https://docs.djangoproject.com/en/dev/ref/settings/#middleware-classes) is with CsrfViewMiddleware before `AuthenticationMiddleware`. In any way: do not do such "hidden" changes among all other changes. This could break things (no forms working) and nobody would imagine that this was caused by a commit connected with language chooser. If you see something like this (so that something else from some other code) is wrong and you fix it, please do it in a separate commit with clear commit message (*Reordering middleware classes.*, for example).
Agreed, but it's been like this for a while, and I think the basket_client looks for it. This should be part of the API cleanup though.
Can we break this onto multiple lines?

<pre>
if ( !options.pluginPath ) { 
  options.pluginPath = "js/plugins/speak/"; 
}
</pre>
What if someone enables the UI, but doesn't want the UnloadDialog? (IE, like me for UI testing)

In this case, we need both checks. The onLeaveDialog only determines if we actually do setup for it, where as the UI enabled check will still handle adding in the header/tray elements.
I don't see this listed on the jshint options page, what does it do? http://www.jshint.com/options/
It is, but I hate being blocked on such a small fix.
`|| 0` right?
It can't be, because Jade will complain about indentation that doesn't represent tag nesting.
Empty actions aren't valid HTML. Either omit it or, better yet, add in an explicit action. But don't leave the attribute empty; it's not valid and it's just noise anyway.
i think it's better to move that into separate css file, e.g. viewer-b2g.css, without ".B2G" part and remove classList modification below
This probably should be left unchanged.
Should we be checking if change_syms includes k, or key?
i.e.
a.__send__(ons, key, v) if change_syms.include?(key)
Wow, I never heard of that, but it's awesome! Thank you
Why was `'>= 0.0.12'` removed? It won't work with `< 0.0.12` correctly.
Sorry the above was implementation of the kt_environment.promoting_to method.
Not related with the API at all. This is a permission management thing and this will change with the new backend (https://bugzilla.mozilla.org/show_bug.cgi?id=707625)
This seems slightly dangerous; is 4s generous enough to avoid sporadic failures?  I guess if we see any problems on the CI server, we can increase the sleep or create a more deterministic method of waiting.
We now reference webapi.js like this

```    
<script>
      var host = document.location.host.replace(/^[a-z]+\./i,'');
      var src = 'http://' + host + '/webapi.js';
      document.write('<script src="' + src + '"><\/script>');
</script>
```
good catch! fixed
missing `E`
I'd say we try it like this, and we can correct it manually and then fix the script after, if necessary.
Is it strictly necessary to make this an in-line import.  There's no reason I can think of not to have this at module-level.
Oh yes, indeed. Fixed.
For blowfish the password is not unset as the password from the db has the
unique salt for that password hash. The salt is needed to hash the incoming
password for comparison. You could add db columns to store the cost and
salt but it would be more complicated really since crypt() handles all the
formatting for you.

Good catch on the hard coded field, will fix once i get home.
On Aug 18, 2012 5:08 PM, "ADmad" <notifications@github.com> wrote:

> In lib/Cake/Controller/Component/Auth/BlowfishAuthenticate.php:
>
> > +		$fields = $this->settings['fields'];
> > +		if (!$this->_checkFields($request, $model, $fields)) {
> > +			return false;
> > +		}
> > +		$user = $this->_findUser(
> > +			array(
> > +				$model . '.' . $fields['username'] => $request->data[$model][$fields['username']],
> > +			)
> > +		);
> > +		if (!$user) {
> > +			return false;
> > +		}
> > +		$password = Security::hash(
> > +			$request->data[$model][$fields['password']],
> > +			'blowfish',
> > +			$user['password']
>
> You are hard coding column name here which is bad. Plus if your column
> name is password wouldn't it already get unset in _findUser() ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cakephp/cakephp/pull/775/files#r1409290>.
>
>
I'm not sure about the name of this interface. It looks more like a ReflectionProviderInterface
@group should be group.  This is causing the issue.  If you look at add_subgroup.html.haml where the template is called @subgroup is passed into the local 'group'.
Really don't do this. It's going to bite you when a group has thousands of discussions. You're loading all discussion objects. Do it with a query.
pep-8: double blank line before function defs at the top level.
How would I get the original output back?  Would that require an additional request to the server?  Does it still have it?

I'm calling fixConsole before fixCR here so that fixConsole isn't called again on the existing text.  It probably wouldn't hurt anything, but it's additional unnecessary computation.

This is turning out to be more problematic than I suspected...
If you have a real rename handle, then i would use rename inside.
Eventually I would make a 'def save_a_copy(name=None)' (and it counterpart in js) that save a copy with the given name, and if no name is given then do it by appending  a timestamp.

Some user want to auto create  backup files, but after long debate we came to the conclusion that this should not be enable by default, but should be easily configurable. With this it is then only one line in `custom.js` in your profile.
You can just do `% locals()` here since you're using all the same names and don't need to do sub-attribute access.
foreing -> foreign
connexion -> connection?
Undo this and above change removing whitespace
In the core, `store_history` defaults to False, and the code path that executes user input sets it to True. I don't know if it makes sense to do the same here, I just thought it worth mentioning.
I'd rather not, I think forcing lower-case and no spaces on what are essentially GUI documents is terribly ugly.
rename to ReturnAbsent...
I'm taking it this didn't work out in some cases?
sure, use a data attribute.
implementation hint: the bulk (or all?) of the actual data can come from product_details' region data.
This is part of the new Unverified Email protocol. see https://bugzilla.mozilla.org/show_bug.cgi?id=794634#c9

I believe that the client persona library uses this to identify the issuer to request the cert from. :ozten could provide more info for how it's used, however the domain should match the root for the new verifier (called by the backend)
We don't want it to be conditional. I did this to make the next 4 lines be the same for both batch and normal calls (accomplished by making the non-batch args look like batched args with just one set of args passed in).

For example: `HardWorker.perform_async(1, 2)` gets here with `base_message['args']` looking like `[1, 2]`. If we then did the `args.collect do ...` bit below without wrapping it in another array, it would create a message with two jobs (iterating for 1 and then for 2). Instead, we wrap it so it's `[[1, 2]]` so that the parameters are grouped as one set of args.

However, looking back at it, I realize I've clouded the clarity of the code. Not sure what I was thinking yesterday, but I've made a small change that should make it much simpler.
Should be removed too, there is no wiki anymore.
Perhaps `/^\w{0,2}$|^\w[\w|\-|\_]*\w$/`

Also: tests! :)
you should maybe put `menuItems[elem].children[0]` in a variable
This cannot happen (unless the root of the collection has a non empty prefix, which is invalid)

What should I do ?
I would pass a Memcached instance and then make the Cachebundle use the DIC to configure it.
This will certainly have unit tests done, but I first need to get the component into a better shape and have the time for writing the tests. If anyone is willing to help...
Again, this should be "03".
It still looks scary in the context of this method. Should we come up with something else or am I overreacting?
This form should also hide the select form, so that the user can not do anything but cancel or valid
If `^` isn't clear for you, then `!=` would make a better alternative.
yes, will do thanks
Тоже в принципе будет работать, если оставить просто
```ruby
def signed_in?
  current_user
end
```
Правда в этом и смысла вроде бы нет, кроме, разве что, логического выделения.. не?
``#!/usr/bin/env python`` or ``#!/usr/bin/env python2``? (not sure the latter is usable everywhere)
leave it like it is. It will be translated again.
I'd like to see the additional data in a separate parameter so that users can continue to cleanly pass position to `.css()` or `.animate()`.
Oh, I’ve just understood your message. I’m very surprised that there isn’t any solution to keep the sub-header within the first `<header>` element.

Which applications are using this sub-header at the moment?
Argh. I have already merged it. I will backout the bakground color changes. If UX is not happy send them to me.
I don't understand the reason to split `AlarmsDB` and `AlarmsTestDB`.

I understand that you had move things out of this object because of the comments from the previous pull. Don't you think after all that this object kind of becoming a meanless middleman?

Please merge this with `AlarmsTestDB` and make it the new `AlarmsDB`.
Sorry for my lame english. Open a new github issue 'Implement a UI when saving bookmarks to the homescreen'. And copy-paste the issue number in the code with the title of the issue. And that's it.

If you want to assign yourself to the other issue, that's even better ;)
Sounds great and will reduce the code redundancies for sure ^^
I will change this in a following up, thanks.
You could add (as in `settings.css`) a common `font-family`  like `font-family: "Open Sans", sans-serif;` in order to avoid to repeat the same through the code.
Why not use a click event here and below?  That would still allow the user to change their mind by sliding their finger off the button after touching it.
A 1000 of sorries, you are right, I'm blind!
() not needed
'use' --> 'user'
Some people prefer to use onload instead of onreadystatechange so you don't have to check the readyState.
This part is the same as the first few lines in _onTopUpSuccess(), could you define a function to get the notification?
(Besides, need localization for title/message)
We should avoid inline script in HTML?
It would, will add
This should be using standard for loops, not for...in
It´s an anonymous function that calls `Appendable.apply {}`. We already used that in some other tests. It´s necessary do recognize errors that come from Ember itself. @DominikGuzei can presumably teach you the deeper logic behind this. I can´t :(
Include "raj@appscale.com"
I think this wording could be a little better, maybe "Installing AppScale with $1 as the only supported database." Grammar you wrote doesn't sound so smooth.
Include the name of the app you're trying to stop.
Might want to specify which connection change state and from which state. 
Also use logging instead of print.
wanna break this up so it's under 79 cols? :D same for the ones below
A single sample JSON item.  I have the rest of the Errata items done, I just need to alter the build system to take them into account, then I'll add them into git.
In my experience, code comments are an indication that your code is not appropriately self-documenting.  I took a stab at a quick refactoring of lines 165 - 173 that I think provides better documentation and removes unnecessary duplication:


    function last_index_of($segments) {
      return count($segments) - 1;
    }

    function last_segment_of($segments) {
      return $segments[last_index_of($segments)];
    }

    function has_pagination_flag_as_last_segment($segments) {
      return (substr(last_segment_of($segments), 0, 1) === 'P') ? true : false;
    }

    ...

    if(has_pagination_flag_as_last_segment($pagination_segments)) {
      if(has_pagination_flag_as_last_segment($freebie_segments)) {
        $freebie_segments[last_index_of($freebie_segments)] = last_segment_of($pagination_segments);
      } else {
        $freebie_segments[] = last_segment_of($pagination_segments);
      }
    }

I'm not familiar with the functionality of this plugin or the problem that this code is trying to solve, so take this with a grain of salt ;).
Ah, that factory was added to replace the lonely one in feature/geometry.scala so that one could be removed, but the code wasn't changed to replace it.  I'll do that.
My comment was more directed to using a private argument to share an implementation.
@Seldaek Good to hear.

Just one note though. How will you handle packages that don't have any tags/branches other than `master`? (like some official symfony/sensio bundles at the moment)
原文: Backbone's 0.9 series should be considered as a release candidate for an upcoming 1.0. Some APIs have changed, and while there is a change log available, and many new features to take advantage of, there are a few specific changes where you'll need to take care:

監訳案:  Backbone.js ver0.9系は、ver1.0のリリース候補版として位置付けています。いくつかのAPIが変更され、多くの活用すべき新機能があります、それらの変更については[更新履歴](#changelog)に記載されていますが、特に以下のいくつかの変更点については注意が必要です:
> dont use variable names like p, use project instead

fixed https://github.com/zzet/gitlabhq/commit/3313627d7424abc72ab0a9117d7c94d73ad50609
this wont work. Did you tested?
agree with Patrick. Might look messy but prefixing all these classes with `.football-stats-` is actually nicer than wrapping them all inside a second-level selector. SMACSS principles!
It is zero copy an the pointer returned by the NT_NET_GET_PKT_L2_PTR macro is a memmapped address where the packet was originally dma'ed into system memory. I am not sure if the Napatech API supports processing on future packets before releasing the previous ones. I will take some time today to play around with it and see if I can get the data release mechanism working.

Matt Keeler

On Nov 2, 2012, at 5:19 AM, Eric Leblond <notifications@github.com> wrote:

> In src/source-napatech-3gd.c:
> 
> > +
> > +        ntv3->pkts++;
> > +        ntv3->bytes += NT_NET_GET_PKT_WIRE_LENGTH(packet_buffer);
> > +        
> > +        // Update drop counter
> > +        if (unlikely((status = NT_NetRxRead(ntv3->rx_stream, &stat_cmd)) != NT_SUCCESS))
> > +        {
> > +            NT_ExplainError(status, errbuf, sizeof(errbuf));
> > +            SCLogWarning(SC_ERR_NAPATECH_3GD_STAT_DROPS_FAILED, "Couldn't retrieve drop statistics from the RX stream: %lu - %s", ntv3->stream_id, errbuf);
> > +        }
> > +        else
> > +        {
> > +            ntv3->drops += stat_cmd.u.streamDrop.pktsDropped;
> > +        }
> > +
> > +        if (unlikely(PacketCopyData(p, (uint8_t *)NT_NET_GET_PKT_L2_PTR(packet_buffer), NT_NET_GET_PKT_WIRE_LENGTH(packet_buffer)))) {
> We are doing a copy here. Is there a zero copy support available in Napatech ? In this case, you can use PacketSetData (see https://github.com/regit/suricata/blob/master/src/source-af-packet.c#L749). You could also use the data release mechanism: regit/suricata@8fecd95
> 
> PS: the code line is really long here
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 


--------------------------------------------------------------------
The information contained herein is for the exclusive use of the original recipient.  This information is granted for limited distribution within the recipient's organization for planning purposes only.  Further dissemination, whether private or public, is prohibited and may be covered under a non-disclosure agreement.
whats this 42 here?
I've updated the pull request with another commit changing them both to errors instead of warnings.
The function only advances cursors if there's no text selected. I'll see what I can do about putting it under line 39, though. I previously thought that you had to `clear()` all the regions and then add them all back, but I was wrong about that.
"str" is a built-in function. Please don't override it.
Would be better to move this just before `module` is set.
So that we won't expect it to be define before here and line 258.
Nit: We tend to just use `Error` without new.
> I don't see why base would contain .. nor ..

There is absolutely no guarantees what `base` looks like:

```
./foo/bar.js
./foo/./bar.js
./foo/bar/../bla.js
```
I've commited {} as the default config (in lib/configuration), so that there is a default, disabled bigtent.
oh crap.  good catch, thanks.  Yeah, just a bug.  Using your suggestion...
Please use the notifications module here:
file:///data/code/addon-sdk/doc/packages/addon-kit/docs/notifications.html
This is something useful for a sysadmin and not necessarily an error that should be investigated by a programmer.
The reason for putting them together is that they all get set at the same place. Even though they are different facts, we are only testing one single method here.
This is the only test that checks the ordering of files in the plugin mount. That should be preserved somehow, using master/agent or the plugin face.
These lines are also style changes. Please split them from the actual change.
you've forgot to delete this line
in case where in if is whole function I prefer quit exit, as it is easier to read it then complete indented function. so:
```
if (!Arch::S390())
  return true;

...
```
You'll need rebase this PR and move this entry to the ServiceManager equivalent
Afaik this is just to prevent unwanted assignments if you accidentally omit one of the "=" in comparisons with "=="'. 
I personally don't think this is necessary in heavily tested environments as well as in php in general due to the "===" 
This should be general decision because there is no rule for that yet. Some components do it this way, some the other way.
Update this docblock with the new params
The new component Zend\Math\BigInteger coud help here
Probably need to validate the $element value to ensure it's valid for a tag name.
arrondi*s*
I like the fact that you have a file-level docstring, most people forget to do high level commenting.

This should probably have some more info in it. See [here](http://www.python.org/dev/peps/pep-0257/) for more info and convention on docstrings, specifically where it talks about "The docstring of a script."

Also, it's the convention to use triple double quotes (""") around docstrings.
Oops! Sorry, done
Added this new check to stay in bounds.
I don't follow why you changed the test here. Wouldn't it be better to have both?
cascades could be handy for this case.
E.g.
editElement.on
  ..keyPress.add(....)
  ..blur.add(...)
should remove the rest of it
The blank is a bit of a long story! At the moment there's no range checking implemented. So if you have say the years from 2002 to 2022 in the year dropdown and then select 2022 and then use the next button to go forward to a month in 2023 you are in a year that is not in the dropdown. In this case the year dropdown defaults to showing the first entry - without the blank this is 2002 - so although everything is working and you are in the right month in 2023 the dropdown shows 2002 which is confusing. It's probably a bit of an edge case but by adding the blank it means if you go beyond the dates in the dropdown, the dropdown displays blank. Hope that makes sense!
this is wrong. We don't want to prevent people to see a package if they cannot delete it. We simply want to prevent displaying the delete form. See the code using this method: https://github.com/composer/packagist/blob/master/src/Packagist/WebBundle/Controller/WebController.php#L304
@vesak $laskurow["valkoodi"] on aina eri ku tyhjä, joten ehkä tässä voisi aina palauttaa $laskurow["valkoodi"]. Eiks se siis ollut niin, että tuotteen_myyntihinta-funkkari palautta hinnan aina laskun valuttassa?
I think it would make sense to place the ReflectionException as previous exception
Missing line break.
What I previously have referred of normal flow being your current level's branch is also valid here.
yeah this would be good to have then as well .. but it will be quite painful to do.
would be good to have tests for the current behavior as a start .. and of course some docs
Please use `Code.load_file` instead of `Code.eval File.read`.
> We could introduce ?g option and then remove it in translate_options.

This is what I had in mind. But if we are going to have a different result because of the flag ?g, it is a no-go imho. :S Maybe we should have a different API then just for group checks.

> * :global

Great.

> * :return
>   * :list
>   * :binary

Do we really need it? If you want binary as response, you can simply convert the input to binary before calling it.
This should probably not be in the api until it is finalized, and we have some docs and wx implementation.
This is not needed any more here since with those changes you have all needed dependencies, including ezpublish-legacy as it's installed by Composer.

So now there is no reason to run those tests in *ezpublish5 context* because you already have all you need in ezp-next.
maybe you should do an `escape( 'url' )` for the mailto ?
I should do that for the jts quickstart also.
I actually have a change to rename lastMotion to a boolean stickyHPos, since there isn't really a use case for the last motion otherwise. What do you think?
s/breadcrum/breadcrumb
Instead of doing `'\''` you could just do `"'"`
s/doesn't/didn't/ ?
http://www.w3.org/TR/html-markup/table.html#table.attrs.border
`Indicates that its table element is not being used for layout purposes.`
という意図で付けたんですか？

例えそうであっても、
http://www.w3.org/TR/2011/WD-html5-20110525/tabular-data.html#table-descriptions-techniques
`th`があれば non-layout table である意図は表明できるらしいので、いらないんじゃないかなあ。
це не має відношення до блогу. ти ж бачиш, що код лежить в src/Application/Bundle/DefaultBundle/Controller/UploadController.php
Sure we do, in [EnableAndStartServices](https://github.com/yast/yast-runlevel/blob/51e8391edd1de497a3f4fec42850e6c5d61b9909/src/services_proposal.ycp#L275).
Zotero.done iand Zotero.wait aren't needed anymore in Zotero 3.0 - you can just delete them.
Haha, I did exactly that first then someone on IRC complained I should do it the other way because it is more clear (to him I guess). Anyways, I agree with you.
So how does this log line work? The Object[] is probably not inserted into the string, as it is already using java variables to comlete the String argument.
`<var>plural(NUM_1, "is")</var><code><var>NUM_1</var></code>`
It is 1 at the moment, because I was restricting everything to R^2 to get
something working.  I'm going back over the kmeansp and kmeansbi to extend
them for R^M, which is what we have already done with the gomatrix library.

You are correct about the variance.  The distance measured has to be
between x_i and its nearest centroid mu(i).  I'm correcting it now.  It
hasn't been called by anything yet since I wanted a review of the BIC
components before doing anything else.

I think that mu(i) and mu_n are the same thing.   mu_i denotes the
coordinates of the centroid closest to the i-th data point and n indexes
the clusters, so mu_n = mu(i) for all x_i in D_n.  Anthony, can you clarify
this on the github stream?


On Thu, Aug 9, 2012 at 12:28 AM, Dan Frank <notifications@github.com> wrote:

> In km.go:
>
> >  //
> > -// The points matrix contains the coordinates of the data points.
> > -// The centroids matrix is 1Xn that contains the centroid cooordinates.
> > -// variance = 	// 1 / (numpoints - numcentroids) * sum for all points  (x_i - mean_(i))^2
> > -func variance(points, centroid *matrix.DenseMatrix,  measurer matutil.VectorMeasurer) (float64, error) {
> > -	crows, _ := centroid.GetSize()
> > -	if crows > 1 {
> > -		return float64(0), errors.New(fmt.Sprintf("variance: expected centroid matrix with 1 row, received matrix with %d rows.", crows))
> > -	}
> > -	prows, _ := points.GetSize()
> > -	
> > -	// Term 1
> > -	t1 := float64(1 / float64((prows -1)))
> > +// variance = 	(1 / (R - K) * \sigma for all i  (x_i - mu_(i))^2
> > +// where i indexes the individual points.
> > +// N.B. mu_i denotes the coordinates of the centroid closest to the i-th data point.  Not
>
> I'm not sure that this line is entirely accurate - I mean, at least as far
> as this function is concerned, you are only passing in one centroid to
> compute variance against, not dynamically picking the closest centroid to
> each point. I think that there should actually be centroids for all the
> clusters passed into this function, but I can't really be sure since it's
> not being called anywhere (at least as far as I can tell).
>
> Now, looking over this led me to go back to @afoglia<https://github.com/afoglia>'s
> notes, and it seems as though he is assuming that mu(i) and mu_n are always
> equal for x_i in D_n (this is in step 3, from the email sent on july 14th).
> I'm not saying this is wrong (I don't feel like I have a good enough grip
> on the context to say that), I just wanted to bring it to everyone's
> attention and make sure we're ok with it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bobhancock/goxmeans/pull/11/files#r1339198>.
>
>



-- 
Bob Hancock

Blog - www.bobhancock.org
Twitter - bob_hancock
https://profiles.google.com/hancock.robert/about
not a big deal but this seems to break an abstraction barrier (what you actually care about is self.startdate_utc being set, not startdate). similar comments could apply in other places.
This all looks good otherwise.
From what I can tell looking around a bit this would fall over on IE8. Maybe underscore should be used, would also make it consistent with the rest of the code.
True, I'll move it up the hierarchy.
Looks like we could use `for (currentParamType <- currentParamTypes) { ... }` instead of going through an index. It would likely be faster, since we're iterating over a list, and indexing is slow for large lists.
Why not a synchronous call? You immediately block on `future.receive` one line below.
.git doesn't affect bundling, and I only added this to the PR as I saw it and cleaned it up at the same time.

On 10/04/2012, at 8:22 PM, Henrik Hodne wrote:

>> @@ -43,7 +42,7 @@ group :assets do
>>   gem 'uglifier',           '~> 1.2.0'
>>   gem 'compass',            '0.12.alpha.4'
>>   gem "i18n-js",            '~> 2.1.2'
>> -  gem "localeapp-i18n-js",  :git => 'git://github.com/randym/localeapp-i18n-js.git'
>> +  gem "localeapp-i18n-js",  :git => 'git://github.com/randym/localeapp-i18n-js'
> 
> Same thing applies here and to the rest of the changes (except the `sinatra` one, of course).
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/travis-ci/travis-ci/pull/499/files#r666996
This bit is very messy. You should be able to get what you want from lineups/tags using the:
mm I think we should use classes here rather than a non-allowed attributes
e.g. removeClass('error')
Yes you're right, my initial idea was to complete information from provider to provider.

but I'm fine with this implementation
You can pass variables into partials so it could be {error: @cohabitabt.error?} etc.
Thats because my base was Clip v0.9.2 and not the current trunk. :-)
Ahh, I'm sorry. I should have mentioned that we have a simple way to build a streamable fixture for unit testing, see https://github.com/BrightcoveOS/Diamond/blob/master/src/collectors/bind/test/testbind.py#L21 for an example. It should make it easy to wrap around any call that expected a IO class (File, network, etc).
Space after `(array)`. Do we need to cast here at all?
В .h-файле напишите нужно задать константу в таком виде:

extern type *const name;
Это метод для получения другой случайной ноды.
Только тут есть подвог, т.к. генерация здесь не совсем случайная.
Берем случайное число 0,1,2,3. Если оно равно нашему значению тогда мы его либо увеличиваем либо уменьшаем.
Эта конструкция разделяет вероятность следующим образом
Если у нас число 0 - тогда мы получим такие результаты с такой вероятностью ( 0(50%), 1(25%), 2(25%) )
поясните пожалуйста. как вы пришли к этой формуле?
> кстати элемент с индексом (index + 1 + arc4random()%(kMaxDNAItems - 1))%kMaxDNAItems может хранить тот же символ и в итоге мы 
> заменим "А" на "А" ;)
Для данного примера в принципе верно, но в задании в mutate передаются процент мутаций от днк, а не число их число.
brackets aren't needed.
Wouldn't it be much easier to use `actual = Math.min(Math.max(opt.min, actual), opt.max)`
Actually, it turns out that they're doubles.
I think we'll need to tear down old bindings for this too, or at least replace it for any old `RACCommand`.
This won't work if the signal immediately sends nexts upon subscription. We should add a unit test which exposes this case.

Can you please also change the style here: `x ?: RACTuple.tupleNil`
Does github recognize .markdown?  I thought it was .md?
to me you can avoid this if branching since if n_jobs =1, parallel is list and my_resample is resample
this should also be null=True
Isn't it already Config Management? All the Salt stuff is being added to the 201 section. Which speaks to needing to split stuff up into sub-sections, but that's a whole other battle...
This happens because of the callback which overrides this value. 

As we want to enforce the date to be 3 days ago, I don't see any problem in what you did. Maybe we should use instead `update_attribute` to clearly show that we're enforcing it.
@sadache Should I remove it ?
you should use the ``:class:`` syntax to link to the api doc (for all appropriate places)
The space between Read and the rest of the sentence is missing. This seems to be a problem throughout the json versions of the trails.
is that true? What if the transport just doesn't exist?
Only use `mvim` on a Mac if `vim` doesn't have `+clientserver`: this covers both people who have symlinked `mvim` to `vim` and those who haven't.
Why not make this "_isIgnoreable" and account for all 3 cases where parent === -1, internal, and magic methods?
You are right, apologies
What is ``sublime_files``? Is it a built-in command?
By the way, code from line 43 to 52 could be removed and replaced by a Firewall check
comment is duplicated
Actually, let's just drop the project dir. When we do implement it, we'll do it as an option, not an argument.
If size > endOfMemory, this will probably lead to a crash
i'm the one who submitted the merge-sort function and it's a port from C. I can't find the original source right now but i'll keep searching.
I thought about that. I figure you won't see these unless there is a failure. And the beginning/end of the list would indicate which hook it is. 

I also was concerned that `B` and `A` would not be understood to be Before and After, since we use simple `.` Passing and `F` Failing for step definitions, not specifically calling out a Given When Then letter.

All that being said, it's easy to change to B/A, and I'm not really opposed to it.
Eclipse should be configurable, to order things in the way the Java style guide recommends. Something like:
java
scala
very general like Apache projects
lift
application
This used to throw the following exception if the DNS name did not exist, and it was uncatchable in this form.

events.js:66
throw arguments[1]; // Unhandled 'error' event
^
Error: getaddrinfo ENOENT
at errnoException (dns.js:31:11)
at Object.onanswer as oncomplete
ya im not going to change the mycnf pkg.
In this case, I say lose the trailing commas.
In multiline listings, I vote keep trailing commas just to make it easier to maintain
Isn't that for components only ?
"The created consistent hash can have more or less owners" -> at this stage i.e. first creation of the CH shouldn't the numOwners attribute be mandatory (of course if possible, i.e. numOwners <= members.size() ) ? It's only when members leave when you have (temporarily) fewer owners and only in the CH returned by the e.g. updateMembers method.
Are you sure new GenericJBossMarshaller() works with values from different modules?
doesn't seem that  "cancelledSegments" is what this field holds and that confused me. It seems to hold "segmentsForThisInboundTask" :-)
I'm an idiot? :p
So, the result of rotate_axes is that the data `jX` gets plotted on the x-axis. The data `jY` on the y-axis and the data `jZ` on the z-axis. I think the x-axis data limits should be determined by the data that is plotted on them, i.e. `jX`, not `X`.

I think the auto_scaling on `X`, `Y` and `Z` will look wrong for data stretching longer in one direction than another.
I don't it should be an API property here. There are controls with no handler or with multiple handlers, and `handlerOptions` makes no sense for those.
where else should I put this note.  this was changed (wrongly) by three different people, three times.  I just want to make sure that it doesn't happen again.
Whoa, so `(integer 4)` isn't working for me because:

```clojure
(clojure-version)
;  "1.3.0"
```

Strange since our [project.clj](https://github.com/reddmetrics/forma-clj/blob/develop/project.clj#L13) rides on 1.4:

```clojure
[org.clojure/clojure "1.4.0"]
```

 Maybe mah Swank needs an upgrade.
apagar comentário
@bp323 Thanks for updating the tests!
Can we use single quotes for the changed lines?
You should be able to do an ``instanceof AdvancedMethodInvocation`` check there, no?
I tried that first, it didn't work at all until I added the function wrapper =x
why does this line-height not match the one used for div/pre? Same Q for the other ones
Did you mean to push this change?
I fixed dumpfile.

I took a look at restructuring -- it's messy, I'm going to leave this alone unless you feel strongly.
Another solution is to declare $woocommerce_errors as global in the beginning, so that it can be used from within the hooked function to add errors to the variable itself.
No, disappearing now.
@unknownbliss bump. the meta_refresh() and message stuff below is no longer needed since we're skipping it with a redirect.
I guess the change in map_with_request_methods  is not relevant anymore.
Why would you obfuscate the logic here? IMO a ternary is a bit more clear than using short-circuit logic...
Fjern utkommentert `notice()`
This doesn't fit the method signature of AddAttachment():

    public function AddAttachment($path, $name = '', $encoding = 'base64', $type = 'application/octet-stream') {
From the way I've been taught it's always better to have separate tests that way when one fails the later ones still run plus it's much easier  to trace failures.
is that the only one we care about? I guess if you do ``.startswith``, we might just do ``text/`` here and XHTML will also match, if anyone uses it.
Oh, supposed to refer to the line below where it gets interpolated.
I guess that should probably be a single regex. Problem was the original regex doesn't catch anything shorter than 4 characters, so if you had `/\w+/` as your validator, it would pass (not warn).
`if (self._parse) parse = self._parse` would be fine here, or `|| self._parse` above
I see. It is a bug. I will create a fork and fix it. You then add my fork to `requirements.txt`.
*Nepravilen*
```
from lxml import html
tree = html.parse('http://slovenskenovice.si/lifestyle/astro/oven')
print tree.findtext('.//div[@id="horoscope-sign-right"]//div[@class="view-content"]//span')
print tree.findtext('.//div[@id="horoscope-sign-right"]//div[@class="view-content"]//strong')
```

You should really learn XPath. So please explain XPath above.

And I will leave regex for parsing date out of the string to you.
Aren't all fields required?
Model namespace
Shouldn't it be renamed to AppHarborClient?
Doh!  Copy and paste fail.
Same comments as those for the controller plugin
¿Puedes eliminar los paréntesis de esta línea? En este caso no añaden información y por lo tanto son redundantes. Si tuvieses:

if (self.role == 'Boss' && foo = bar) || self.role == 'Underboss'

Los paréntesis serían perfectamente apropiados porque ayudan a entender en qué orden se evalúan las diferentes condiciones.
Hibernate Search which can be used for advanced query capabilities (see section "reference to how data is queried") ...
I see. I got confused by especially `hibernate.dialect` and `hibernate.ogm.datastore.grid_dialect`. I just wanted to make sure that the dialect which is detected by Hibernate OGM is `hibernate.ogm.datastore_grid_dialect`. Is that correct ?
should probably be a stub
You can use the survival function instead of 1 - chi2.cdf. So chi2.sf(llr, 1). Sometimes it will be more accurate.

It's also a bit of an unwritten rule to return test statistic, p-value rather than the other way around.
Can you rename the directory to tests/ for consistency? You'll need to use git mv and commit the move so as not to lose history.
The coding standards should be updated to follow PSR-2 as Swiftmailer has been converted yesterday
what if stage is nil?

On 16/12/2012, at 12:09 PM, Henrik Hodne <notifications@github.com> wrote:

> In lib/travis/worker/shell/helpers.rb:
> 
> > @@ -132,7 +132,7 @@ def timeout(stage)
> >            if stage.is_a?(Numeric)
> >              stage
> >            else
> > -            config.timeouts[stage || :default]
> > +            config.timeouts[stage || :default] || config.timeouts[:default]
> Right, I guess the stage || :default is a bit superflous. How about this?
> 
> config.timeouts[stage] || config.timeouts[:default]
> —
> Reply to this email directly or view it on GitHub.
> 
>
the other point to not get the next message is that maybe you want to stop the wait loop after you got the return.
This can be done in the callback.
Thanks, now using this.warn
Again, see above.
This else clause doesn't seem to be a direct alternative to handleMessage.
makes sense - that should be do-able and clearer


On Thu, Dec 13, 2012 at 12:21 PM, Brian Humphrey
<notifications@github.com>wrote:

> In Product/Production/Common/Properties/pom.xml:
>
> > @@ -11,6 +11,9 @@
> >      <name>${project.artifactId}</name>
> >      <properties>
> >          <log4jDefaultLevel>INFO</log4jDefaultLevel>
> > +        <directAnchorStore>${env.NHINC_PROPERTIES_DIR}/../gateway.jks</directAnchorStore>
> > +        <directPublicCertStore>${env.NHINC_PROPERTIES_DIR}/../gateway.jks</directPublicCertStore>
> > +        <directPrivateCertStore>${env.NHINC_PROPERTIES_DIR}/../gateway.jks</directPrivateCertStore>
> >      </properties>
>
> I think the direct stores should be named like the following:
> direct-public.jks, direct-private.jks and and direct-ca.jks. In otherwords
> don't use gateway
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/CONNECT-Solution/CONNECT/pull/250/files#r2410144>.
>
>



-- 
Michael Landis
Software Engineer
www.5amsolutions.com | Software for Life(TM)
phone: 866.526.6042 x774
email: mlandis@5amsolutions.com
skype: mlandis5am
I think we decided elsewhere that it would be a little more readable to (a) not call it l5 (looks like 15) but something like lan5 and (b) put the keywords i.e. ```galsim.Lanczos(5, conserve_flux=True, tol=1.e-4)```  I now realize this has been inconsistently implemented, e.g. in some places the code was fixed but not the documentation, but I think it does look cleaner so it would be good to fix here (and if you're feeling generous, in other places in base.py etc.).
Regarding your latter point: I originally wanted DoubleGaussian because it's a decent approximation to ground-based PSFs (at least, better than Gaussian :), and we didn't have Kolmogorov.  So it was useful despite the limitations you point out. Now that we have Kolmogorov, I see less of a need for DoubleGaussian, and would be okay with removing it and seeing if anyone complains.  This will involve modifications to some demo scripts, but not a big deal.  More to the point, as a matter of strategy, I think it would be a good idea to move forward with these new base classes and update config, rather than spend a lot of time and energy on how to handle this one class.  It doesn't seem worth doing anything fancy or time-consuming here, so I actually like your last idea best.
obtain() and release() machine methods aren't being used here, so neither is the finally block in the tearDown doing anything.
Should we just delete that code from here?
there's an extra forward slash here
Hmmmmm? Why is that removed? And the windows ifdefs above?
I would introduce the username as a parameter to the localisation so that it can read properly in other languages that might put the username somewhere else in the sentence e.g.:

```ruby
<% content_for :body_content_title do %>
  <%= t('.posts_written_by', :name => @author.username) %>
<% end %>
```
можна для "SonataAdminBundle routings" одразу посилання додати https://github.com/sonata-project/SonataAdminBundle/blob/master/Resources/doc/reference/getting_started.rst#step-1-define-sonataadminbundle-routes
auth_sync() => auto_sync()
This isn’t working for me… the checkbox is always checked when I reload the page!
Aha! This is what I was trying to figure out last night but I couldn't get it to work.
same as above for the 'g'.
Need to fix the spacing. You should uses spaces rather than tabs.
I'll clean up the docs to be more coherent and add a note in the tests to look at the docs.
ditto hier
Uhm.. any reason this can't be in hotspot_expose?
can you give a reason like the error message that we would see otherwise?
Then, further down the road, we can check and perhaps remove this if a new clang version or a new version of kent tools is available.
Seems better, though I'm not sure why they had it the other way
Лишние приведения типа.
Чуть более дружественное сообщение надо сделать.
В задании всё есть
Теперь ты будешь считать циклической ссылкой:
```
Object o = new Object();
Object[] array = new Object[] { o, o };
```
Is there a benefit to clearing them in this way instead of overriding what needs to be overridden below?
if it is invalid URI, the request will fail at the server.
I agree about the attraction of piquant text, but do you see that "job" is misleading in this context, as if it referred to any particular job position?  It will make people think "If only I was a man, I would get 30% more for doing this same job", which is not implied in the data.
Should proxy be disable/not support for this type of connection?
Why do you think we should default to 0?
Line 48 is looking for the precompiled, digested version OR returns the asset as it was. That line is also only run if asset digests is enabled.
If we are just accessing a variable that is stored in the renderer (simple type) we dont' need an auto release pool. If we are calling any Cocoa methods that we don't know the internals of, there is a chance that memory will be allocated by them that is expecting an autorelease pool to clean up.
No, not unnecessary. You could either check for `> 0` or `> threshold`, because some of the peaks in coords may already be set to 0.
@ndawe Can you comment on this? I don't any strong opinion on this.
Maybe explicitly state that this is useful when the whole kernel matrix would not fit in memory  e.g. when the number of samples is too large.
For consistency with trees and forests, I think we shouldn't multiply by 100.
" It would avoid a kkt check for every point on the path although there is not
new variable to add or remove."
I don't understand this, a step on the path means a different alpha?
If this is true, I can't know beforehand if there is a new variable to add or remove.
Otherwise I could know the right features from the beginning.
cache $(img).attr('src') to src instead of using multiple times
But I don't see a point in this loss of information. You're creating a TiUIOptionDialog, so why not return a TiUIOptionDialog? It's usually better to need to cast from specific to general (TiUIOptionDialog to TiUIBase), not the other way around.
C'est possible, mais en tout cas pour les documents comme la facture et le BV, on peut pas donc ds tous les cas il va falloir faire tous les cas.
Après pour factoriser ceux qui sont pareil, @doc.getNomDocument() c'est clairement faisable, mais pour la route du doc à générer, je vois pas trop comment puisque les arguments ne sont pas tous les mêmes.
A méditer
Could you do a quick check and see if there is actually a requirement for (explicitly) an `EntityManager`? I'd love to move this logic to have it consume a generic `Doctrine\Common\Persistence\ObjectManager` and the `Doctrine\Common\Persistence\Mapping\ClassMetadataFactory` interface.
And we can refactor the transaction mechanism later, to allow for delayed global actions.
Also note here that if you wanted a specific branch you'd just do:

``` io
setDependencies(list(
  "krainboltgreene/ostruct.io -b derpbranch"
))
```

Neat! Of course any flag can be passed here.
So how about to leave the prefixed name for now and only fix the bug with this PR. I'm willing to help with another PR to change this CS everywhere if @l3pp4rd agrees.
Remove this. If you're calling `Bugs().get([])` then you're doing it wrong. Check that you're not sending an empty list in in the code in views.py
That means there is a strong connection between the github and travis ci account? In such a case I would prefer to move the project to the mozilla account. Please let me know about it and I will do it.
"另一种情况" 写成了 "另一中情况"  
some spell mistake here needs to be corrected.
Can't the need for "removeDuplicateSites" be eliminated by focusing only on the "category": "All sites" array entry and ignoring the others?
I think there is exception when rule is used to a case that this rule doesn't apply to.
GitModule is kind of utility class. GetUntrackedFiles, GetTreeFiles and others don't return some internal members of GitModule which should be protected from modification. They return list of objects and they don't matter who and what will be doing with this list. Hence the higher type you return, the more flexibility caller will have. [Here](https://github.com/gitextensions/gitextensions/pull/1408/files#L3R2048) you have an example.
 The opposite case is GitItem.SubItems. Here subitems should be declared as IEnumerable if SubItems collection should not be modified from outside of GitItem class. [Here](https://github.com/gitextensions/gitextensions/pull/1408/files#L7R1100) LoadInTree shouldn't change the order of items elements. Changing type to IList showed it up. But changing type to IEnumerable would even showed it up more.
So maybe I'm reading this wrong, but are you trying to pop "/binA/sh" here?
I think here Exploit::Failure::Unreachable suits better :)
I did this a few other places; can I infer your recommendation is to eliminate `.lower().strip()` in every case?
`rows` should be sufficient. We get overlapping borders for buffer_size > rows / 2 here but that does not matter. This both results in the same mask.
Should one of these be `tform` instead of `tform.inverse`. Otherwise, I'm not sure what you mean by "alternatively".
Does pep8 also apply to cython? Hm ok then I'll have to enable it there. Have never tried that.
I'm trying to be consistent with ActiveModel https://github.com/rails/rails/blob/master/activemodel/lib/active_model/errors.rb#L431  which treats :attribute and :model as the humanized versions.  Perhaps there is a better way to do this - any suggestions?  We do ultimately need to pass :attribute and :model values containing humanized strings into the translate call.
Ah, I see the confusion. I was referring to this code, not the spec for it. I mean that you should do something like this:

```ruby
check_value_in_array(@attribute)
```

instead of 

```ruby
check_value_in_array(subject.attr)
```

because `subject` is not guaranteed to have an attribute named `attr`. Does that make sense? I was pointing out that if you change the test to use an attribute named something other than `attr` (like `foobar`, since "attribute" is a disallowed attribute name), then the test will fail.
Yes and yes.  We can potentially still use some other techniques to further reduce the number of vertex attributes.  We'll need to do so if and when we have a need for another attribute.
It was only like 8 files (half of which were specs) So I just made the change locally, haven't submitted yet, but will do that soon with the rest of the review changes.
Instead of "the debug call", perhaps you should just say, "the controller should define a "doClick" method, which is called when the menu item is clicked."
Why are you setting "num d", and then returning d? Just return the value of the expression. Same goes for the code in pitch().