labofsahil/github-event-dataset-2012 huggingface dataset:
AnnouncedJobIsNotDeletedWhenConfirmationIsFalse
Please remove the copyright and license tags here. They don't apply to this code.
``$diff->removedColumns['parent'] =  $table->getColumn('parent');``
The first parameter ìd`is missing.
https://github.com/fabpot/Pimple/blob/master/lib/Pimple.php#L174
If LOGGER uses the standard Python logging mechanism,  you can avoid a couple of parenthesis and just write

LOGGER.debug("Loss ratio curve is %s, write to key %s", loss_ratio_curve, key)
I would write it simpler:

```python
for i, task in enumerate(task_gen):
    if i == concurrent_tasks:
        break
    hazard_curves.apply_async(task)
```
The function naming should match the existing code style.

`function page_name() {`
Even `\A\s*`.
I don't see why lmc_fval here is ok with 1.  From the delete line above, it seems 2, no?
You've got two except blocks here and the second one unindexes... That seems wrong.
might benefit from a little comment about why this duration is important.
Should this have an `extract_name` helper like below?
Good catch! I moved returning state outside the case block so it indeed doesn't need to be here. Fixed.
This is what I usually end up typing myself, but I think the examples read easier (and it's an established style) if we replace the first "=" with a space.
Would it be possible to reorganize this function a bit so that the more "global" checks happen first?
Hm, I don't think a test should cause us to add new instance vars, but I can't think of a different way in this case. @fabiokr @tfwright any ideas?
This is related to what I was discussing above.
This will produce a notice. Should use `isset()`.
ah yeah definitely need a callback here
Our convention seems to be to prophesy disaster & then let it happen naturally, rather than to kill the program straight away. But that's a convention I'd be fine doing away with.
We do this in a few places to explicitly do module init work like ProjectManager and SidebarView. I can take it out if we prefer.
When does perfData[name] get initialized with an array value?
This can be merged into the if () statement above (they are both checking if (commandData) )
This was intended to be the reference (or base) that all other context menu z-indexes are based off of. Currently, this is the only one. :)

Would be nice to unify this with the main menus, but currently these index values are hard-coded in bootstrap (even though they are using LESS).
Change to sprint 15
I don't see this promise used anywhere. Am I missing something?
Nit picking. need a space before the colon like other strings.
* Memory leak, you didn't declare `server_options`
* We don't use underscore in variable names. Use camelized names. `serverOptions`

Otherwise, lgtm
I would say no. No point in writing an entire new module for few lines of code, unless there is reason to believe that the module will host more functions in the future.
Out of curiosity have you tried mixing in `Ext.util.Observable` as opposed to extending it? `Ext.util.Observable` is typically used as a mixin in Ext JS.
This should either be logged and/or rethrown instead of just printed as a stack trace.
The following condition would be sufficient :
```
if(!lt.isSetDataURL()) {
```
limits<float>::max()?
Restored to default.
Can we do without this? If we plan on shipping for IE/Opera at all, depending on this property might be a problem: https://developer.mozilla.org/en/CSS/pointer-events/
Use the XHR module in butter here.
Line to long, break
spacing `function( e ) {`
Same comments about return and undefined callback.
Doing this introduces a second break statement. I'd like to avoid this duplication.
Sorry for that. I will remove it.

2012/12/30 Saúl Ibarra Corretgé <notifications@github.com>

> In Makefile:
>
> > +	python setup.py nosetests
> > +
> > +venv:
> > +	@echo "=> Creating virtualenv files..."
> > +	@mkdir -p venv
> > +	@for version in "3.3" "3.2" "3.1" "2.7" "2.6"; do \
> > +		echo "   * Python $$version"; \
> > +		which python$$version 2>&1 >/dev/null && \
> > +		virtualenv -p python$$version venv/$$version || \
> > +		echo "[SKIP]"; \
> > +	done
> > +
> > +clean:
> > +	rm -rf ./build ./*.egg-info
> > +	find . -name "*.pyc" -type f | xargs rm -f
> > +	find . -name "__pycache__" -type d | xargs rm -rf
>
> Why this makefile? Also, there are many Python versions and interpreters,
> so I'd rather have users install them manually. Please remove this change.
> If you feel strongly about it, though, feel free to submit another pull
> request just with this change, and we can discuss there :-)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/saghul/pythonz/pull/17/files#r2520991>.
>
>



-- 
张江阁<tonyseek@gmail.com>
---------------------------------------------
Skype: tonyseek.z
Home Page: http://tonyseek.com
Do you need to guard the dict lookup here? It should default to root when the configuration is built. If a user explicitly unsets user, we should probably raise that in config validation.
anonymous -> anonymous_
Tässä vaiheessa kannattaa jo korjata nämä, sillä mitä tapahtuu, jos avain on longin mittanen ja koitat lukea siitä intin ja sitten muuttaa longiksi?

Tutustu tähän: http://stackoverflow.com/questions/7693324/converting-string-to-long-in-java
let's pull the base64 string out into a var and simplify the replace expression a bit

```js
var urlData = 'url(data:' + mimetype + ';base64,' + new Buffer(fs.readFileSync(filePath+seperator+fileName)).toString('base64') + ')')
props.replace(/url\([^\)]*\), urlData)
```
Should we care about the corner case of `null evs` and not bother yielding a `Flush`?
Je continue à trouver que c'est bizarre de l'appeler add. Je suis d'accord avec toi qu'on "ajoute" un tracking, mais initialmeent on le créé. Par cohéreence avec les autres mapper, je pense qu'il vaut mieux l'appeler "create" (histoire d'avoir le parallèle avec CRUD partout).
:trollface: khily jalebe ke pas chera male man fargh mikone male man intoriye
```
#hidebtn img{
	display: block;
}
```
in class male tag a 
```
#hidebtn{
	display: block;
	width: 48px;
	height: 48px;
	margin: 12px auto 0;
}
```
Same thing here as with the other specs :)

You can just update the spec to use 'v0.2', no need for a new tag.
Please set the gemspec to unicode :)
I noticed that although we include the :title on 23 and 26, when the user hovers on the favorites_icon, they title is not displayed.  (This was observed on FF 11.0)
Would it be possible to wrap these calls in a resources block?
Please spaces instead of tab.
No need to call new here, recommended to just do a `var noticeArray = [];`
Favors are the absolute number of favors for this memplex. While selffavors are the flag wether the current user has favored it or not.
add private to these.
@fabi1cazenave ?
asking the test runner for metadata.
There's definitely some value to that.  Though developers are often turned off by more marketing-oriented content, and this page is designed to, *ahem*, "get them pregnant" :)   I vote to leave this out or maybe put somewhere else.  Don't feel terribly strongly about this either though.
I'm not sure what I'm looking at re the URL above. Why 5 rather than 4 or 6?
You don't need the keyword ".self" in this method. Ruby automatically assumes that you're calling an instance method on self unless there is a local variable with that name. The only time you need self is to disambiguate if there really is a name clash with a local variable, or on the left hand side of an assignment statement (because otherwise ruby thinks you're creating an instance variable). While certainly just always using .self will work, the general convention amongst ruby devs is to not use it if it isn't needed.
Is there a reason why this can't just all be in a dict literal?
I had a wrapper solution in mind: have a HttpCookie class that wraps a Cookie instance.
More simple, but still don't like to instanciate additional long life objects if  I can find a way to avoid it...

Also, why java and not scala?
> I have taken it from the template: /tests/templates/default/issueXXXX.html

Right. That reminds me of needing to create a separate template for FBTests. This one is actually primarily meant for manual test cases but we also use it for automated test cases.

> What do you think?

I think you didn't have a look at the [examples](https://getfirebug.com/wiki/index.php/Creating_a_Test_Case#Examples_2). :-) They all have already relative paths. I.e. we do know were the CSS file is located relative to the test case files and we don't have to use any JavaScript hacks to get that. In your case the relative path is `../../../_common/testcase.css`.

Btw. the reason why we use relative paths for the test cases is that when you run the tests locally they don't have to fetch the CSS file from getfirebug.com, which makes it much faster.

Sebastian
wow, this looks pretty random. @adou600 symfony components extend core exceptions so you can specifically only catch exceptions from that component. we don't do this in createphp, please make sure to always ``use RuntimeException`` (use statement is still the correct and better than \RuntimeException in the code.
Oh, OK, I was mixing up the example-running code with the documentation-generating code, where it really does need to be removed (and not just commented).
You could write this as

```
env_vars <- c(
   if (cran) cran(),
   if (check_version) "_R_CHECK_CRAN_INCOMING_" = "TRUE"
)
``` 

etc
When "x <- 10; x" is parsed using parse_all, it results in a list.
Previously, the code was taking only the first element of that list.

On Wed, Oct 24, 2012 at 8:27 AM, hadley wickham <notifications@github.com>wrote:

> In R/eval.r:
>
> > @@ -44,10 +48,14 @@ evaluate <- function(input, envir = parent.frame(), enclos = NULL, debug = FALSE
> >
> >    out <- vector("list", nrow(parsed))
> >    for (i in seq_along(out)) {
> > +    expr <- parsed$expr[[i]]
> > +    if (!is.null(expr))
> > +      expr <- as.expression(expr)
>
> Could you give me an example of where this might happen? It doesn't match
> my mental model of parsing + expressions.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/hadley/evaluate/pull/22/files#r1929714>.
>
>
default tag should definitely be restored to v2.0, +1 to and/or
I agree with Carlos' approach. We can assume that the reference names will be properly prefixed.
Ahhh, the noob spotted tests using $idx and flocked the birds of a feather together. I guess there weren’t $idx tests in the other file that I noticed?  I’ll move these and can add others there at the same time. OK?

Rich

From: Veena Basavaraj [mailto:notifications@github.com]
Sent: Wednesday, August 15, 2012 1:07 PM
To: linkedin/dustjs
Cc: Ragan, Richard
Subject: Re: [dustjs] Make $idx/$len work for all array objects and not leave old values in contexts (#113)


In test/jasmine-test/spec/grammarTests.js:

> @@ -70,6 +70,20 @@ var grammarTests = [

>        message: "should test an array"

>    },

>    {

> +     name:     "array",

> +     source:   "{#names}({$idx}).{title} {.}{~n}{/names}",

> +     context:  { title: "Sir", names: [ "Moe", "Larry", "Curly" ] },

please move this tests out of this file to the other one.

Not sure when this file was split up ( since I did not see a PR, @jairodemorais<https://github.com/jairodemorais> )

There is no grammar changes relates to $idx, hence

—
Reply to this email directly or view it on GitHub<https://github.com/linkedin/dustjs/pull/113/files#r1386643>.
Right, and presumably it needs to be served with the right mimetype which you'll need to do in the view.

Speaking of which, how can I don't see a url added in urls.py for this?
Since this behavior depends on a value in settings.py, should we either use that value, or explicitly set a value for that setting for this test? That way the test is more robust if the setting changes in the future.
Now that we are using expect.* we do not need the try/catch clause anymore.
I'm a bit confused. `IOError` isn't a subclass of `OSError`. Why `OSError`?

The two errors I would expect are `(IOError, MemoryError)`. Both are likely'ish.
this is set and exposed to the view from the controller, line 232 in controllers/report.php
Yeah, sounds good to me.
tut, tut

`pip install pep8`  

or maybe

pep8nazi                  - PEP8 Nazi: because really, you have no excuse.

:-/
(or even just ASCII. The log statement assumes there won't be any byte tearing, which is a lie. You're better off making one giant string and then logging LOG_CHUNK_SIZE characters at a time)
I know, is just for CS.  :)
Yeah, I was speaking of relations.
this should be ``Merges the singular and plurals back into 1 item`` (the phpdoc describes what the method does)
you should use the short class name in the phpdoc
no need for this code at all. CSS already support the needed behavior: ``scroll`` will always add the scrollbar even when not needed. But if you use ``auto`` it will add it only when needed
We can check for mbstring availability first and do unicode or fallback to ASCII (without #u), right?
Shouldn't this be "03"?
AFAIK, we don't use ``+`` anywhere in the Symfony codebase to merge arrays. Could you use ``array_merge`` or ``array_replace`` ?
how about adding an accessor `$this->getLastEvent()` and raising the exception there. so you don't need to repeat this twice
Yes, that would be helpful as well, I haven't noticed that, thanks :)
Yeah, let's do that.

Keep the constructor, but inline the check into it.
it should be ``Boolean``
I would rather use 
```ruby
arguments = params[:registration] ? params[:registration][:arguments] : {}
```
It is a bit more verbose but more clear what we expect here.
And I think that the assignement in line 416 is redundant as we initialize an empty hash here.
I might implement this method using `@invitation`, constructed in a `def construct_invitation` method, then extract the error constructor method.
Syntax is correct. `find_by_email` is stubbed on `User`, although it returns nil by default.

I'm noticing we're just asserting this method is called, not doing anything with the result. This if fine - please disregard this comment.
Can set this ivar at line 19 and use in 25/26
Easier thing here is to just require 'lib/failsafe/backends/exceptional'
Is there a precondition that the sequence + K is still convex ?
The formatting of github has not displayed what I wrote:
pow\< dimension -m \>( 2 )
Could use a collections.namedtuple here.
> They're always the same.

Nope. They aren't. Try looking at this example:

```
import cartopy.crs as ccrs
import cartopy.io.img_tiles as cimg_tiles

import matplotlib.pyplot as plt


ax = plt.axes(projection=ccrs.PlateCarree())

ax.add_image(cimg_tiles.MapQuestOpenAerial(), 1)
ax.coastlines()

plt.show()        
```
In that case it would be `if not roles and vms`, because we want to add the host always when a role hasn't been given or when a role is given but it has a virtual machine.
I think that the "Limiting Server Thread Usage" discussion (http://lists.jboss.org/pipermail/jboss-as7-dev/2011-September/003764.html) seems related to the "general pool" discussion that we want.

Also discussion in "Two thread pools sections in management API" thread http://lists.jboss.org/pipermail/jboss-as7-dev/2011-October/004200.html
Do not import *, use FQN imports.
Is there a reason this was copied from accordion?
Move error constants to errors.py or a constants.py?
Reuising it ;-) Thanks!
good idea! split this code in different files
nit: indentation is broken
Would you mind moving this below Line 30/31 so it's easier to know where `self` is assigned?
please delete the comment now that the issue it describes is fixed.
well done! this was old code
@aymanmaat works for me, will update the PR to satisfy all the needs.
Is the important rule needed? Can you try to remove those, that's usually something nice for debugging but not otherwise.
This fix the second error explained here https://github.com/mozilla-b2g/gaia/issues/5405
All this setHeight mechanism is spread across all modules. I wonder if we can make something smarter with css only ?
Ok, but then they should be localized, don't you think? I'll fix the rest of the errors ASAP.
It's a good idea to place the external libraries to a sub directory. So I put id3-minimized.js and ogg.js to apps/music/js/ext/ without modifying them. It's clear to maintain/update indeed.
If we only reload the list here, will result on and outdated list when going back from details/form
When you are already on the landing page you likely want to go back to the main page?
Can we localise this?

Don't like the strings being concatenated, looks to me better if we put it on a string and pass parameters to the l10n engine, and for other countries we can check if we have a different format
Isn't there Locale.ENGLISH constant?
You'd think EnsureDirectory should be able to deal with the trailing slash.
As noted in IRC, this approach is consistent with the path shown by the yum distributor. I'd rather keep those two consistent and perhaps come up with a more robust solution in the future. We should also consider that pulp doesn't have to be deployed at "/pulp" necessarily, and knowing one's own base url is generally a difficult problem for web apps to face.
I do not know if there is a standard or not. I can do the research.

I am not convinced about your suggestion, because in that way, it does not produce exact result, but only approximate.

In fact, we can use something similar to: http://api.rubyonrails.org/classes/ActionView/Helpers/DateHelper.html#method-i-distance_of_time_in_words
To verify that we handle the click event last at ALL times we remove the handleAfterClick event, and re-add it
I'd like to leave these verify steps in, maybe merged into the section at the end of the page. (asking git, ruby, and rails what version they are)
Don't know if it's per spec but if it's not a RefusePuts policy, you'll go past maxItems anyway.
The "move data variables" is the *new* refactor that you wanted of moving the variables from data.py back into the appropriate views.
given that you're now doing versioning, this should now just be a pure insertion e.g. 
    new_record = ... thing with version = 0 and user set ... 
    self.col.insert(new_record, safe=True)

safe=True is set to make sure that you aren't violating any uniqueness requirement on the index.
array_key_exists() => isset()
Should we have a condition to check just for JDEBUG or KunenaFactory DEBUG.  Only because I think regardless of the state debug Kunena is in I think JDEBUG should take priority over Kunen here.  Saves time then having to enable the option in more than one place.
I'm not sure what part doesn't make sense, so I'll just explain my thought process for this segment.

The except is supposed to catch any exceptions raised by either the `request.get()` function that attempts to access the GitHub or Zendesk API or the `raise_for_status()` function that raises an exception if the status code of the aforementioned request is not 200. If an exception is raised from one of these two functions, the `args` attribute of the exception will contain a tuple with its only value being the error message of the exception (i.e. "Invalid URL", "Status code 502", etc.). I redefine this `args` attribute to contain both the original error message and a string that records if the RequestException occurred while trying to connected to either Zendesk or GitHub. Then, this exception is raised so that it can be caught by the except in the `home` function where an error message string is constructed that is rendered to the home page to tell the user both what error prevented the application from loading and whether the error came from Zendesk or GitHub.
Spikebrehm thanks! https://github.com/thisconnect/javascript/commit/ee91591b047b4c4a8b87fc898a06ab9eb64e7bf2
This CSS layout doesn't feel consistent with the formatting of CSS in other projects. Examples at: https://github.com/alphagov/styleguides/blob/master/css.md

Would be good to keep that consistent.
This could be a lot clearer by using Rails' select_tag helper, and options_for_select.  Something like:

```ruby
select_tag "section", options_for_select(@departments, @old[:section]), :prompt => "Select department"
```
This assumes that `@departments` is a hash of label => value.

Note: with this version you won't need to add the `EMPTY_DEPARTMENT` option in the controller.

See: http://apidock.com/rails/ActionView/Helpers/FormTagHelper/select_tag and http://apidock.com/rails/ActionView/Helpers/FormOptionsHelper/options_for_select for reference
Changed methods to Jareds proposal.
At this moment these methods are only called from a loop that iterates through self.codebases. The codebase parameter origins from self.codebases.
See how it's used everywhere else then use it the same way here.
Don't remove the `403` check. Now you're letting anyone add a sponsor.
Would be great to have a docstring here explaining what the difference is with the 4 kinds of alt records (or point to the spec)
It would be nice to have specs for these changes, and not just in the RSpec matchers!
OK, so you have local variable, that you set to false. In this case you set it to true and then you don't need it.
It is just relict of old different code-flow. So correct fix is to remove error variable at all from this method.
É uma dependencia do debugtoolbar...
sem ele nao instala de jeito nenhum
This doesn't appear to be standard syntax. For Github Markdown, use:

```html
<html><body>Hello World!</body></html>
```

(View source to see.) Reference here: http://github.github.com/github-flavored-markdown/
default zero
secure this line by retreived using hash
Because `extends` and `implements` are not valid identifiers, but are valid property names. They are reserved for ES.next as keywords that would do the same thing as this. In other words `var extends = ...` would trow exception.
Thanks for making this reusable.
We don't know how to handle `Try` yet, for now it needs to be skipped.
I added handling for any json ValidationError to skip the branch and do a println if settings.DEBUG is True.  Otherwise, it silently skips it.
If we fail in posting the content to pastebin we will not re-enable the menuitem.
__whimboo__ commented on commit
> I hope that we can have a patch for the MPL2 license soon. Would you be interested in doing that?

Sure!
I'd like to provide a file list with three element in the MPL2 issue (#39) before creating the commit:
 * files with new boilerplate
 * files with changed boilerplate
 * files with no boilerplate
Of course all files would came from `extension` directory.
Can you please make sure that your PR doesn't revert the ordering/matching behavior described in [this ticket](https://github.com/bitprophet/ssh/issues/33) (especially the later comments)? I remember this sorting-by-size line from that issue, and it was removed/replaced for a specific reason -- adding it back verbatim makes me worry.

EDIT: Yea, I checked out this branch & ran `python ./test.py` and the tests added/affected in that ticket now fail, when they pass on master. Hopefully easy to address.
Can you sort the use statements?
This untaint seems too aggressive, it's untainting paths that were previously tainted, why?
Add a check to see if the element has a value. Also, it should be the charCode, keyCode just tells you which physical key you pressed (its the difference between 'a' and 'A' if shift is held down.
But you are changing the API by returning an order different than the prior API.
lucky you -- we only recently abandoned python 2.5 ;)
@denixport Add a comment describing the regex.

@weierophinney was a stub
`la directory genitore per come` -> `la directory dandole il nome del tuo progetto`
Something seems to be wrong with indentation here?
The only reason it was a static variable before was to keep it out of the DI API, since we've created an API for this function, it should be fine to make it a static member of this class.
Rename to `assertNotResponseHeaderRegex`
This comment is not required, the following code line is already explicit enough.
oh, sorry. I had thought Travis could be specified each minimal version.
This looks suspicious. The application name does not end with a '/' and when using it as part of a path boost should take care of it. If that does not work, we probably have a problem elsewhere.
FWIW I like the simplicity of the current approach, and I agree with your assessment about modern stack sizes (for instance RLIMIT_STACK on my box is 8MB - depending on the Apache threading model maybe that value is significantly less per thread but I doubt it will be an issue in practice due to the nature of the function call in question).

In any case I think far too many man-hours have been expended debating this by now so let's just leave it at 64k :-)
Why ordinal and not invariant culture?
Link aus Mail. Hier die Route: (aus config/routing/password.xml):

    <route id="rtxlabs_user_password_reset" pattern="/reset/{token}">
        <default key="_controller">RtxLabsUserBundle:Password:reset</default>
    </route>
Remove console.log from the code.
Nvm, it's grouped models.
I think it would be better not to iterate over all settings. Something like:
```
if 'FEED_DOMAIN' not in settings and any([k in settings and settings[k] for k in feed_keys]):
```
Maybe do something similar here? Determine the guesses for all keys and then print them all out in a single go.
use this instead:of getcfg/setcfg

self.updatecfg('makeopts', "TBB_DIR=$SOFTROOTTBB/tbb MPI_CXX=$MPICXX OPM_FLAG=%s" % self.tk.get_openmp_flag())
self.updatecfg('makeopts', "MPI_DIR='' MPI_INC='' MPI_LIB='' MY_CXX=$CXX MPICH_IGNORE_CXX_SEEK=1")
@stloyd True, on the other hand I think it would mix up responsibilities

@Fran6co Not needed
You are correct, filepath is only used to find the resources folder. And `xbmcswift2 run` requires the cwd to be the root of the addon so this could definitely be changed. I'll created a separate issue for now.
Won't maven generate application.xml for us based on the ear plugin?
might be useful to wrap this in a `new Error()` so we get a stack trace
please switch back to using """ instead of '''.
Docstrings have to use double triple " :)
Double quotes and padding please :)

`(msb === 'MSBFIRST')` => `( msb === "MSBFIRST" )`
I'm in favor of first finishing this PR, then doing a big cleanup, so that separate things are done in separate commits.
Alphabetize?
I'm missing the class import api
what does the JS do? its not part of the commit?
mozApps.install doesn't directly return an app object - you have to get the result in a success callback from the DOMRequest object returned.
Strange. It shouldn't be zero. that is what is used for the auto refreshrate switching.Ie: g_settings.m_ResInfo[GetResolution()].fRefreshRate
I wish it made sense to me. I don't think gabe ever actually do a "thor new" on a brand-new machine.
Would be nice if the size varied appropriately so we don't get so much space:

![](https://s3.amazonaws.com/uploads.hipchat.com/6574/26709/x70h7079josgt9t/upload.png)
You <del>can</del> should leave off the `KhanUtil` since it's in scope when vars are evaluated.
However, it is kind of curious why we would use two different testing tools to test different behavior. Is this just convention?
These lines will wrap to one. Either add an extra new line, full stops, or bullets.
shouldn't you pass the output instead of echoing directly from the fixer ?
удалять кто будет
for (Iterator it = vasya->petya->kolya[idx]->masha->lyubit->sashu[x].begin(); it != vasya->petya->kolya[idx]->masha->lyubit->sashu[x].end(); ++it) {
}
I would have put 0.1 but was it too small?
There is probably more than one link with the same rel value.
This code should also determine the position (x-th occurance of link with rel="value") and use that to build the query.
What does the "a" do?
The main reason I stuck everything under one file is that there are a couple of methods that call down to private methods.
Now that we're settling on an internal folder where private stuff can go under, this can be changed I think.
RabbitMQ has been implemented, and the app nodes can be configured to bind to the task queue as a worker. If disabled, they don't handle indexing, if enabled, they do. By default, indexing is enabled so work better in dev. environments.

Special care has been taken to make sure the app server firing the task does as little work as possible, and offloads data-access, membership access etc... to the indexing workers.
Отличное решение, так действительно будет совершаться меньше телодвижений.

P.S. этот код я уже видел в более позних пул реквестах, а это говорит о том что не одному мне код понравился)
Выбран не очень оптимальный алгоритм мутации. Скажем при 100% мутации, если число элементов в DNA 100 и последний элемент который осталось изменить находиться по индексу 42, то рандом может подбирать его неоправданно долго.
вот как раз мы сначала ищем место для авто, потом паркуем.
что бы было, если мы выделили память под объект NSObject, а места под переменные Cell не хватило бы.
self = YES , все вроде ок, но нет.
как правильно добавил автор "Работает, но для красоты ещё исключения бы добавить :)"  и я не могу не согласиться. 
Как только мы хотим выделить память, нужно обрабатывать исключение!
Sorry. I think I have some conflicts in my github repository, let me check it again.
I tried to eliminate the redraw function and animation didn't worked. I'll just make it simpler as I commented before.
extra comments, incorrect tabbing
couldnt this be `return (boolean) $this->queryFields` ?
since this is called twice... maybe a inner function in the close method would be useful for DRY.  thoughts?
Any method that is not an action should be private. That goes for both the above methods and the ```edit_*``` methods.
Hmm, where did this come from?
Yep, you 're right.
the casting warning is certainly a bug

the division by zero I am not sure. It's weird to have a true 0 on real data.
We should add some checks to make sure data has the same number of time points and channels as raw
but also those will be converted on init. look at the related part in read_epochs

On 07.12.2012, at 19:00, Eric89GXL <notifications@github.com> wrote:

> In mne/proj.py:
> 
> > @@ -121,14 +121,11 @@ def compute_proj_epochs(epochs, n_grad=2, n_mag=2, n_eeg=2, n_jobs=1,
> >      else:
> >          data = sum(np.dot(e, e.T) for e in epochs)  # compute data covariance
> >      event_id = epochs.event_id
> > -    if event_id is None:
> > -        event_id = 0
> > +    if event_id is None or len(event_id.keys()) == 0:
> Oh wait, right, in case someone already has some data saved with an event_id as integer or something I should keep it in. Nevermind...
> 
> —
> Reply to this email directly or view it on GitHub.
>
can you call it resample? does it sound right to you?
fix commited
I'm curious what `mem` stands for?
So it is, sorry, I misunderstood what you were saying, I've fixed it now
if you want to show the same stuff than in YAML, you need to add the root node (loading the config to be passed to the extension) which is named ``config`` by convention (the name does not matter for the loader as it will use the namespace to associate it with an extension, but it will matter when providing an XSD and all core bundles are using ``<config>`` as root node):

```xml
<twig:config>
    <twig:extension id="twig.extension.foo" />
    <twig:extension id="twig.extension.bar" />
</twig:config>
```
 
Your current XML is only equivalent to

```yaml
extensions: ['twig.extension.foo', 'twig.extension.bar']
```
如果通过<header>X-Accel-Redirect</header>头或者<link doc="ngx_http_core_module.xml" id="error_page"/>产生了跳转到另一个服务器组的内部跳转，
Well, there's a bit of logic down below that could be refactored out so it'd be easier to add more Cython modules in the future.
I was with this patch up until here. Is there a way we can keep this functionality for now and keep the speedup? Thanks for your work on this, @phene.
Merely having the expectation causes a method to exist, and the interpolator to try it.
И тут же.
I'm not sure that `:badpass`, `:baduser`, `:noemail`, `:nopass` are factories you want to have. You can easily create these off of your `user` factory by passing in the attributes you want. Ex: `FactoryGirl(:user, :password => 'bad')`
Readability nit: no spaces around params

     window.extend(WebDialogExtensions)
I work pretty hard to keep PyLint happy so that every file is green. Many other changes are for the same purpose such as removing extra parens or switching to +=. Pretty much every change I made that appears to be purely stylistic was actually made to keep PyLint green. A clean bill of health from PyLint has saved my bacon many times.
Nice use of ternary!
Can we rename `@out_of_band` to `@early` or some such? We don't need to duplicate terminology.
I'm probably going to change this to inherit from Exception baseclass; raise Exception("Not in addItemTag transaction")
Shouldn't this also come with a patch that removes that DELETE hooks we are currently using? I love it.
I think this package organization is perfect
So you found a way to access non-cache components from a distributed task :)
Since when does core have 2000 msgs assigned?
I think someone complained on IRC about the use of blacklisted, it doesn't have any context so it's not clear what kind of blacklist this is. Maybe we should log something like "Requests to retrieve state from these owners failed" instead?
btw, what is the exact behavior you are trying to achieve ? Serving the source asset directly instead of dumping another file containing it ?
Acronyms should always be capitalized in Objective-C code. This should be `removeDirectoryAtURL:`.
As above, this should be using `+stringWithUTF8String:`.
This bug came out in the pcolor test (the pcolor was not in the correct place). I don't understand how it wasn't failing before, but the fact that the bug has been highlighted by this PR is a good sign.
Does just `mplDeprecation` seem ok? The reason is that it's either being used in code like `warnings.warn("message", mplDeprecationWarning)` and we then have the word `warn` written three times on one line, but there are other places where this class is being raised as an exception, so it's not just being used to warn, but actually to throw an error.

I was using `mDeprecation` because I thought we have that as a convention for prepending an `m` for matplotlib internal imports (`axes.py`, `axis.py`, `collections.py` and `contour.py` utilize this for matplotlib module imports)
Do we not want this default output?
I think we usually use ```pre``` instead of ```before``` for hooks. Doesn't matter, just wanted to mention it.
Вряд ли там переменные подставлялись. Думаю просто взяли и сделали именно так когда-то. Но раз уж обратили внимание, то я все таки уберу это и сделали подгрузку через require, там все равно уже есть BaseException.
The TouchEvent also have a clientX/clientY property, so it's the same logic. Thought it could be useful other places, but it might also just be confusing. Should I remove Event from the comment?
js to js file.  Use `{PSU_JS}`
So far this is just a kludge.  We aren't cleaning the timeseries, i.e., removing and interpolating unreliable values.  We need to to more research about this before implementing.  We can remove this right now.  It sort of serves as a reminder.  Probably doesn't belong in the code.  I will remove and open an issue.
loopback_filename(false)}.gz should work for ec2 (non partitioned), but will be wrong for other clouds I think.  so this should be 
loopback_filename(partitioned?)}.gz I think
This CSS is overwriting the button CSS and needs the !important to work. However, we can still overwrite this in the widget CSS.
Also include the properties so we also update the widget when the translation changes
This is way cleaner than before, we actually relied on the URL for the default bundle being called default.
Now we just use the bundle default property from the config.json for a widget.
Was missing localization, added in a later commit.
Also, do you need to change these conditionals at all? I might be missing something, but won't it suffice to do `var overflow = element.style.overflow || ''` like you did in the line below?
Gotten rid of the escaping code https://github.com/aoduor/Ushahidi_Web/commit/b55d2b3dd4b65d54b72081e82f45d579c3a03b63 . Still don't see the need of splitting this into two functions though :)
Ah I see. In that case, perhaps a `.` separator? This way, the last part of a `view_pre_render` hook will always be the view name.
If we're not interested in internal pages for this test, we don't need to test the root page, it's always an internal page; but, more interesting to me, why did you add the test for internal pages, what was the error you're saw?
Wrong return hint.
I must've removed this line by accident, it must be added back.
Are you sure that the method "getFlashBag()" exists?
This will fail if cache was not found and response was not modified (i.e. some GC cleaned temp folder)
```
if (!ini_set(
    'memory_limit',
    self::nbBytesStringToInt(self::$memoryAllowedOnShutdown)
        + self::nbBytesStringToInt(ini_get('memory_limit'))))
{
```
Das ist bisschen unschön, kurze Erklärung dazu: Eigentlich haben wir keine JavaDoc Pflicht für Getter und Setter, Checkstyle erkennt diese Methoden aber nicht als Setter (weil sie keine Zuweisung an ein Datenfeld machen, glaube ich). Ich habe bis jetzt keine Lösung darüf gefunden, hier meine StackOverflow Frage dazu: http://stackoverflow.com/questions/9955255/checkstyle-no-javadoc-for-getter-and-setter-only-works-for-getters
Ok. Well, then it's bad code. Always be critic about code you copy ;-)
WAT?

In ruby, `Module#include` includes all the methods in the module to the active class definition. So, in this case, `Tumblargh::Renderer::Blocks`, (because the way this file is required), but typically Kernel. This is definitely not necessary nor desired. [Reference](http://www.ruby-doc.org/docs/ProgrammingRuby/html/tut_modules.html) if you're interested.
I really don't think there is a need to have `def()` for this object.
There's still a return false in load, and the DocBlock would need to be changed.  Is there a risk the chaining here will cause a false positive in terms of backward compatibility?
Shouldn't this be

    if ($input->getOption('mit-bread')) {
my fault, that checks were to let me test the other Array methods, they can safely be removed now
Well... that is part of the examples cleanup task :) The Death Star Epic. The command-line examples worked as documented when I wrote it up (I verified them all), and they were VERY inconsistent in how they accepted things at the command line, in which order, when you needed a param (in this case you don't need a --search= ). This is what needs to be fixed... 

...

Just now I am trying to run these examples, and I actually can't run any of the java ones anymore. !!! I don't know what is going on... something is broken. I'll follow up separately. This is a bigger issue than this one particular example.
i wasnt sure here .. i went with "get" to be consistent. i figured it might make some use cases easier when introspecting model instances.
Move one line up.
It is simple. In our django-tastypie-mongoengine we have a dummy QuerySet object. And which does not have a `query` property or even `query_terms`. You should check what exactly it is/should be returned by that. And extend our QuerySet object in django-tastypie-mongoengine to return something meaningful when you do `.query.query_terms` on it.
Don't complicate. Use fieldsets.
Be verbose, `dep` -> `dependency`.
The name is not really true. Because you also set zoom. Probably `configureExtendedMap` and `configureBasicMap` would be a better choice.
Could be, though I would expect LLVM to complain under any circumstance an intrinsic is improperly codegenned not contingent on `-g`.
Yes, I think that makes sense as a convenience, will add.
I agree with @fabiokr
house ForeignKey constructor should be passed `null=True, on_delete=models.SET_NULL`, to allow the field to be nullable, as our workflow allows users to register without joining a house.
We prefer not to pull directly from the head commit for a formula. Is there a stable release of this game?
Any reason these are called `test1` and so on? Can we give them more detailed test names maybe?
I can easily find a School subscription.
this is important to avoid, as it looks like you might have changed code, and I'll need to play "find the differences" to understand I don't need to worry.

Problem is, I'll have to play the game for every other commit on top of this, conflicting with this change. please keep in mind that for me it's not a one-off problem, this is a cost paid by every person working on this code and not having incorporated your commit yet, and if he needs to rebase he'll need to deal with it for _each_ commit in his branch.
We could pass it directly to the ControllerApp.
We should just have an "error" for invalid ID and "unknown" for no ID.
Yeah, I no longer use the one in meters.
Maybe add a link to that wiki page above the formula.
Rewrite like this: "Do not include libpython in the generated package; to be run, it will require an installed Python interpreter in the target system".
gif.gif
Oh, crap!  I was looking for Zone-J.
I think it's ok to leave it there in the YAML/JSON scripts, provided that a comment is given that the Python script is not multi-processed, and that demo9.py is referred to for an example for running MP with Python scripts.
Good idea, pushed.
> In galsim/shear.py:
> 
> >  class Shear(object):
> > +    """
> > +    @brief A class to represent shears in a variety of ways.
> ooops while I was forced to work offline part of today I went and edited these! Hopefully it won't cause too many conflicts if other branches merge from this as soon as it's on master.
> 

I'm sure we'll sort it out, no worries.  Did you do all of them or just the ones in shear.py?
DON"T ADD STYLES TO APPLICATION.CSS.SCSS

ADD IT TO ANOTHER FILE

Application.css.scss is for GLOBAL styles, this is where all of the css gets compiled into, it shouldn't have this specific of a styl
That's better. I've updated the code.
${result} and ${summary} need converted to Java
meme en-dessous, avant que l'IPO ait lieu
I noticed it on the way. Next time I will do so.
What is %2?
I mentioned earlier best to use `is_element_visible` here because it better matches the intention of this property.
Multiple `rescue`s are fine and you could merge this in, but the `rescue Chef::Exceptions::ShellCommandFailed` is of no value at all, not even back-compat.
This can be done using the `@author`:

```ruby
@posts = @author.posts.live.includes(:comments, :categories).
                                             page(params[:page])
```
Well the benefit of the assert would be that developers might get notified about doing things, which aren't really nice, like pushing values from unknown charsets to isAlpha etc. and hoping it behaves in the same was as the original or similar things.

That said our code seems to use isDigit on filenames, outch, so this really isn't that nice... Returning false might be a somewhat sane option to work around more serious problems in our client code, which either doesn't care much about the specifications or it just hopes it will work never the less.

At any rate we need to clearly state that we only support ASCII values in these functions, everything else is just rejected.

The "crash" you talk about only happened in the MSVC *debug* runtime, to show that someone is doing a bad thing IMHO. Of course other libc implementations might yield similar results in normal mode too. So I think the benefit of this pull request is either to give us better ways to detect problematic client code or at least to work around bad client code ;-).

Personally I would've suggested we use the assert in debug builds and just a return false in release builds, that should help us and avoid annoying "crashes" for the users. (On a related matter: we should really improve our overall error handling to avoid asserts for any checks on external data in release builds).
Can we use a Calendar object or something for this?
I think you want to have :start, :enable declared on the jenkins service block.
Could you explain whats the purpose of this "cast stealth after vanish 3 seconds expiration"? i'm kinda confused, how it is working right now on the core and how it should word?
Param is duplicated here it should be something like:

     /**
       *
       * @param headerName name
       * @param headerValue value
       */
А если там будет табуляция?
Вместо \n правильнее использовать System.lineSeparator().
Не является потокобезопасным.
According to https://code.djangoproject.com/browser/django/tags/releases/1.1/django/db/models/manager.py the attribute does exist in 1.1, so I'd have to assume yes.
This change makes the mystery index.html and dashboard.html pages not show up in /public anymore.
The idea is that the base provider class provides generic use out of the box, but if a provider needs to do something different then it should implement the code itself. Movingt the whole logic over the the Google Provider would mean no hacking is required, just shove in a doc-block.

Maybe we chance `function access` to be `function access_token` and `function refresh_token` so that we have the two available much easier, and they can one or both be overridden.
It can be nil or "":

```ruby
Addressable::URI.parse("http://www.google.com/").query == nil
Addressable::URI.parse("http://www.google.com/?").query == ""
```
is it me or weights is not used?
Please add a test for `K = chi2_kernel(X, X)` and check that the diagonal elements are equal to `1.0` and the non diagonal elements are all strictly lower than one yet positive.
можно сгенерить урл по роуту "fos_user_security_login" и проверять точное соответствие урла
I was stuck in my Ruby mindset and wanting it to be called `include?`. Since I couldn't do that, I just struck the question mark.
Why does this work? For me, there is no obvious connection between '@buildScreenViewCreateStub' and '@buildScreen' ...
blackberry-deploy supports only ip address, so probably if we decide to do that, needs to be resolve the ip for given hostname. Probably no need to do it right now.
I'd suggest checking qcolor.isValid() first and returning an error if not valid.
This is definitely a needed feature but it's redhat specific; would it make sense to be able to specify the most common parameters for bonding, and then let the debian and redhat bonding interfaces sort out how to apply that?
True, symptom of copy&paste from prior daliadmin method.
This isn't going to be correct.  Probably best to give an internal error, and revisit if/when it is implemented
can 'logicalname' ever != 'newName' here? 

If state reset, it should always be same I think, it looks odd that different to run()
querySelectsUsed(cur) gives a list of all fields that are used by a given expression.

That is then passed to processSelects to indicate that each of those fields is used.  This is what was happening already for no_skip, but it should have been done for no_assert, and any other expressions that aren't assignments within a transform.
Yes, I have, and it removes completely the inline tables. I've added a test to show this.
can we change to [ ... ] syntax please
I want to remove it after it is encoded, mainly because I don't really know how that char is represented in a string. I know it can be ``\n``, but what about other representations? ``\r`` for example? I thought it was simpler to remove it once encoded because there is only one encoded representation.
А сам метод, думаю, тоже можно укоротить примерно так:
```ruby
post = current_user.posts.find(params[:id])
redirect_to root_path unless post
```
или даже (но мне это чуть меньше нравится):
```ruby
redirect_to root_path unless current_user.posts.find(params[:id])
```
Не принимай на веру (не проверялось), и это не очень конструктивное замечание , просто знай что почти всё можно написать чуточку короче и обычно мы это делаем, если это возможно ~__O
Yes, there are something which prevents us from that!
Thats why the comment block below is so long and descriptive.
Well I guess then you should test it. I doubt that there need to be so many divs. Just imagine how the data of the page is structured and add the right html elements in the right pattern for the data. Add as few extra nothing elements as you can get away with to let the bootstrap stuff work.