labofsahil/github-event-dataset-2012 huggingface dataset:
I would like to move away from new/delete eventually. Placement new coupled with ngx_pool_cleanup_add() is okay but plain new() is not - a std::bad_alloc will kill the server.
Superfluous?
And isn’t this a bunch of code from the old crash detector? We don’t want to re-add that.
You are right, I mistakenly assumed that it needs to be executable to be `source`d anyway (which is obviously wrong).  But in this specific case, it shouldn't be a problem:  The [install script](https://bitbucket.org/dhellmann/virtualenvwrapper/src/tip/setup.py) of virtualenvwrapper installs `virtualenvwrapper.sh` as "script", so it can expected to be executable.
np.asarray(x)  unless you need a copy, which I don't think we do.
wrong indentation in this file too
Didn't know either, just googled it and was very pleased when found this solution.
By the way, have you seen that using misc/local.html is a great way to debug / analyze your tests? Just making sure you don't spin your wheels now, or in the future.
Do you mean that one icon is inside another? It's just an effect of my intentional selections since they're wire images. We could add TRIANGLE_FILLED, et cetera.
Curious why these methods are necessary. Is Rails implementing it's own connection pooling and retry mechanism? Memcached has that already at the lowest level.
This is to be implemented on the next pull request.
style preference: I like to use String.format() for these types of things - easier to read.
The war between the Zionists and the Machines occurred on Zion, not Palestine.
What is this for?  Could it be moved to SWF?
does not work.  needs -o option.
Can we make it profilepath? The help text is fine as it is.
And this one too please:
Test for PHP-394: Crashes & mem leaks
We don't know the username, it is configurable - plus the test explicitly tests an empty username too :)
i think this logic is wrong - you should only be checking for the presence of convert options.
I was wrong. :-/

Keep the code but fix the indent then.
@samueljohn wrote...
> Let's hold this off a bit until the MPIRequirement is fixed. mxcl/homebrew#16678
Looks like the constructor below is never used. Delete?
Is escaping all parameters across API adapters outside the scope of this story?
И здесь.
Не вижу поддержки нескольких файлов.
А кто будет close() вызывать?
@JoinScribbly, if you don't destroy the session, you'll end up with two, which can lead to all sorts of problems.
Typo: s/yoursef/yourself
Use ' instead of " and typo in requires.
According to the style guide, imports should be grouped in the following order:
    standard library imports
    related third party imports
    local application/library specific imports
So we should move this import.
it's required to build the `class_option` descriptions for engine / framework =(
`iris.data` should already be a numpy array, no?
I just notice that in the SpectralClustering, there is a ```from ..utils.arpack import eigsh```. So if scipy uses ```arpack``` by default, does import ```eigsh``` from ```..utils.arpack``` has some special intention? @GaelVaroquaux
changed - thx!
Could you add here the figure generated by your example.
You might wanna fix indentation here. No tabs, and 4 spaces per tab, please :-)
Yes, too much logging. Thank you.
I have cleaned it.
Okay, you are probably right, I didn't check in how many situations it would _not_ be applicable for comparison…
Done with == NULL.
This gem is not necessary.
It crossed my mind also, another time.
Personally I think rowCapacity is better than any of the alternatives so far suggested.
Not sure it would work if it was
Needs to check if it isAConstant() i.e. expr->getOperator() == no_constant (or expr->queryValue() != NULL).  

isConstant() is also true for 1 + 2;
does the yaml mapping work properly in the tests ?
just call it number to avoid the name clash :)
С тем же успехом, думаю, можно было передать просто ```nil```, типа
```ruby
cookies.signed[:remember_token] || nil
```
why not use path's dirname / join etc?
Elsewhere you seem to be standardising on PREF_ prefixes for such constants. Did you want to do the same here?
As example check the ´Profile´ template entry, where it is handled exactly the same.
I'd be consistent, and keep the "return early", to be honest. (I.e., don't change this)

The _work_ of the method is on line 39. Line 36-38 is saying, "I'm not interested if this is not implementing LocatorRegistered."
No need for `break` as I use `return` statement.
`done( false );`
jquery.formatCurrency is in spree_flexy_variants.js manifest, store/exclusions was included twice, now its just included dynamical. maybe we can find a better solution.
you can get length of header from curl:
        $length = curl_getinfo($connection, CURLINFO_HEADER_SIZE);

        $header = substr($response, 0, $length);
        $body   = substr($response, $length);
Ok, I've added the escaping - this time using the correct branch.. ;-)
Hi @bodepd 

I used port and bind_address as an example which used the params namespace. Should I change them to not use the params namespace?
"pyxcomp" -> "ccomp"
My question isn't about the logic of the method, but just the way it's written: `_coeffs` is always called with the `+ [8]`. Couldn't `_coeffs` just be initialized with the trailing `8`?
This is the main line that I added.
@jakeisonline just as a reminder, if you do the change to start using the [new API V2 endpoint for Salesforce ticket data](https://github.com/zendesk/zendesk/pull/3106) to get the crm data, you'll have to access the records and status with: ```data.crm_data.records``` and ```data.crm_data.status```.
yeh its because i have 2 repo's on this pc and think i've merged them because i had a mental breakdown. I'll do a re-pull of the original repo shortly.
One flaw with `defaultValue` it is loses [short-circuit evaluation](http://en.wikipedia.org/wiki/Short-circuit_evaluation). Couldn't this be a problem for several `update` functions?
I always see @shunter combine lines like this to `this._sunDirectionECDirty = this._sunDirectionWCDirty = true;` I'm not sure if it's just personal preference of if there is a performance difference.  Scott?
This is not optional on IOS.
Can we put something inside this array, so users can know what values are accepted
Since you said "See console for details", maybe you should use window.console.log.
I was wondering if this block is necessary since:

1. Composer will install to `/vendor`, so that directory shouldn't require being explicitly set
2. [Other libraries][1] simply reference the `bootstrap.php.dist`, in case any customization is needed
3. Likewise, PHPUnit will look for [phpunit.xml or phpunit.xml.dist (in that order)][2]

[1]: https://github.com/KnpLabs/Gaufrette/blob/master/phpunit.xml.dist#L12
[2]: http://www.phpunit.de/manual/current/en/textui.html#textui.clioptions
Fix is part of https://github.com/hpcugent/easybuild-framework/pull/358 pull request.
return on error, avoid additional indention with the `else`.
Could you move this to the left one char? Sorry for the nit-pickiness, on my first pass I usually only look for obvious typo mistakes and the like.
Why have the array in this case?
`:is_title` and `:is_hidden` is not very idiomatic ruby. I'd do something like this:

```
attr_reader :title, hidden
alias_method :title?, :title
alias_method :hidden?, :hidden
```
`static immutable T i;`
The constructor asserts.
Yeah, I gave that example, but they are actually really bad tests, since the values of the chars don't correspond to the numeric value on the other side. At best, it tests compilation, but it makes no sense at runtime.

Here are some tests that should give some better runtime coverage, as well as make more sense for someone reading the code:

```D
assert(commonPrefix("12345" , [49, 50, 51, 60, 60]) == "123" );
assert(commonPrefix("12345"d, [49, 50, 51, 60, 60]) == "123"d);
assert(commonPrefix([49, 50, 51, 60, 60], "12345" ) == [49, 50, 51]);
assert(commonPrefix([49, 50, 51, 60, 60], "12345"d) == [49, 50, 51]);

assert(commonPrefix!"a == (b - '0')"("12345" , [1, 2, 3, 9, 9]) == "123" );
assert(commonPrefix!"a == (b - '0')"("12345"d, [1, 2, 3, 9, 9]) == "123"d);
assert(commonPrefix!"(a - '0') == b"([1, 2, 3, 9, 9], "12345" ) == [1, 2, 3]);
assert(commonPrefix!"(a - '0') == b"([1, 2, 3, 9, 9], "12345"d) == [1, 2, 3]);
```

If you don't mind putting it in, or some variant thereof.
A little loop to do that?
ここは'post'になるでしょうか？
These use a full node? Also put """ on the next line?
Support for ACML was completely lacking, now it supports both.

I don't see the issue?
condition is just duplicated.  What should it say?
Um horário não tem muitas turma, é o contrário.
Alias, é bom consertar esse belongsTo para
belongsTo = [turma:Turma]
It's a test class; no validation is needed.
According to .NET framework design guideline, each summary should be a single sentence.
It is OK to change hosts if there is a secure hash known already, e.g. from user input or a Digest header
Indent it as it was previously + we don't need `,` at the end.
Currently not, I've just made the simplest implementation. This could have been easily solved by adding two new attributes to the link xml-namespace and xml-namespace-uri
i also noticed its output on errors.

should we give it a hardcoded name or use the provided name and document it is relevant?
Can these tests be run in-place or do they need to be compiled into an EXE?
Docstring is wrong here.
Probably don't need this, right?
I'd rather stay with 'warn'. We have cases like this in email for example. Its not exactly about "you can't specify X if Y = Z" because you CAN, but it will be ignored. That is exactly the case in the outbound adapter etc.
So I am staying with 'warn'
this should not be in the "core" package - only essential building blocks of SI belong in that package
Shouldn't this be called RedisStoreMessageSource ???
Yea, just fix it and re-send.  Make sure to test with a clean pull!  

-Ben


On Monday, October 8, 2012 at 1:12 PM, Matthew Urtnowski wrote:

> In models/ion_auth_model.php:
> > + public function is_time_locked_out($identity) { > + > + return $this->is_max_login_attempts_exceeded($identity) && $this->get_last_attempt_time($identity) > time() - $this->config->item('lockout_time', 'ion_auth'); > + } > + > + /** > + * Get the time of the last time a login attempt occured from given IP-address or identity > + * > + * @param string $identity > + * @return int > + */ > + public function get_last_attempt_time($identity) { > + if ($this->config->item('track_login_attempts', 'ion_auth')) { > + $ip_address = $this->_prepare_ip($this->input->ip_address()); > + > + $this->iondb->select_max('time');  
> Opps, that's a bug from related to some other feature I wrote for myself. It should just be DB instead of iondb. Should I do a different pull request?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/benedmunds/CodeIgniter-Ion-Auth/pull/321/files#r1787368).  
>  
>  
>
Indent for both HTML and Django template tags.
Watch the case - once again it sounds like this Hack idea is already done
Я это поменял, чтобы было одинаково по всему тексту рецепта. Вообще, может тогда просто переформатировать все рецепты в соответствии с CS? :)
Same here, please use ```assert```.
Perhaps a better way to do this is to get rid of the settings key entirely, and just use message.respond_to?(:api_key)  instead???
That's fine with me.  Is there a way to comment the schema?  This way we can group and comment which property belong to which transport type?
i thought the syntax would not allow it hmm I guess it does hmm, where is the final location url of this documentation do you know?
Can we keep the named param passed into this method even though you set the default to 2.

`(num=2)`
you can pass in the conf test file though right?
Hi all,

Here is what is used in the Beta version of the Cloud Identity Client
Developer Guide:

<?xml version="1.0" encoding="UTF-8"?>
<auth>
  <apiKeyCredentials
    xmlns="http://docs.rackspace.com/identity/api/ext/RAX-KSKEY/v1.0"
    username="jsmith"
    apiKey="aaaaa-bbbbb-ccccc-12345678"/>
  </auth>

I would be happy to change to this for the next version of the doc if you
all prefer.

Please advise,

Mike




On 7/5/12 11:23 AM, "Nirmal Ranganathan"
<reply+p-1106263-90f89f6d31b89094c3c0826baeae7fbf1114bcc6-1015169@reply.git
hub.com> wrote:

>> @@ -0,0 +1,14 @@
>> +POST /v2.0/tokens HTTP/1.1
>> +User-Agent: curl/7.21.0 (x86_64-pc-linux-gnu) libcurl/7.21.0
>>OpenSSL/0.9.8o zlib/1.2.3.4 libidn/1.15 libssh2/1.2.6
>> +Host: identity.api.rackspacecloud.com
>> +Accept: application/xml
>> +Content-Type: application/xml
>> +Content-Length: 88
>> +
>> +<?xml version="1.0" encoding="UTF-8"?>
>> +<auth>   
>> +   <apiKeyCredentials
>> +       
>>xmlns="http://docs.rackspace.com/identity/api/ext/RAX-KSKEY/v1.0"
>> +       username="hub_cap"
>
>yeah that's just a dummy api key. But you are right we should probably
>change that .
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/hub-cap/reddwarf_lite-integration/pull/103/files#r11062
>63
Im scared of all this.
Rename, please
Работает в Яндексе
The pixels/dip information has been added to the next update. However, I will need to speak to you about the rest. Please ping me on irc, when you have time.

Thanks
IOS does not fire this event
It's not only for testing, it's to not have pass through methods in all the stages that might want to callback.  Seems the pain of synchronizing is more than the pain of the pass through methods, so I'll revert to private and add the little methods.
I tend to prefer `public synchronized` over `synchronized public`, FWIW.
We should use the AS7 one. No real good reason, just because.. :)
Not really, you need to actually step over the function to get the return value data. It does not work if you simply stop after a function.
Id null all these out by default, so we can make the "happy path" be without any code.
I think making it a function would make unit test less painful. I don't know how to mock properties.
Sure! Done! :)
Why are you using references to specific courses in the variable name? This looks like there should be an array of hashes used here, we shouldn't be referring course names in the variable name itself.