labofsahil/github-event-dataset-2012 huggingface dataset:
On Tue, Mar 27, 2012 at 9:23 PM, Joshua Peek <
reply@reply.github.com
> wrote:

> > @@ -627,6 +627,14 @@ Lua:
> >    - .lua
> >    - .nse
> >
> > +M:
>
> Seems like the primary name ought to be `MUMPS` rather than `M`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/github/linguist/pull/148/files#r613173
>

Dear Joshua,

Except that for various legal reasons, the name MUMPS was officially
changed to M.

Something about vendors not like the old name, AND Massachusetts General
Hospital
wanting to keep the ownership of the name MUMPS.

Cheers,
  --ldl

-- 
---
NOTE: If it is important CALL ME - I may miss email,
which I do NOT normally check on weekends nor on
a regular basis during any other day.
---
LD Landis - N0YRQ - de la tierra del encanto
3960 Schooner Loop, Las Cruces, NM 88012
651-340-4007  N32 21'48.28" W106 46'5.80"
can we split this into one or two boolean variables so as to make reading the if-condition easier?
(Unnecessary)
fixed, it looks like an `IndexError`.  Note that this, and the other things you noted above, are remnants of code written ~ 10 years ago, when I was learning python as I developed IPython.  So it should be no surprise to see some pretty awful stuff here :)  But thanks a ton for the review!
I would reverse this alias, because `render_traceback` is a real public API (not deprecated), and `_render_traceback_` is the custom IPython-specific semi-private API.

I know there's no real difference, but I think this way gives the wrong idea.
Since the `_repr_*` mechanism assumes singletons, we cannot do this.  What is the `_repr_png_` of a dozen PNGs?
... or, we install it with a wrapper script.
no space after >

Valid tag...
<html >
a
</html>

Invalid Tag

< html >
a
</html>


Please fix all places, we do not allow spaces or even s space betwee >partial 

Same is tre for

{#a

{?a

{+a
I know there is not much documentation in the atoum code, 
Yet I think the link of the resource "http://www.floating-point-gui.de/errors/comparison/" must appear in the docblock (just a @link).

What do you think @jubianchi @geraldcroes @mageekguy  ?

I didn't tested the feature itself, but I could handle this PR this week.
I suspect this might lead to issues with database adapters that do not automatically cast values (i.e. mysql opposed to mysql2). It's probably right not to cast, but this requires adapting other methods (like `include?`) to be aware of that.
I thought dotlang is going away.
What isn't it re-raised? Like it is in `test_doing_work_with_one_worker` later.
so, what's the point of sending a waring to stderr?  The person that incorrectly used 'get' instead of 'post' will never see the warning.  During a web server operation, where does stderr actually go?  I think it would be better to actually use our logging infrastructure to tell us that someone is using 'get' instead of 'post'.  At least that way, we'll find out about in the logs.
can't we fix the issue directly?
That's the general idea, yes.  Backing out of the song selection can take some time, which can be inconvenient in some situations with lots of people present (sometimes as few as three).  It's too easy to forget to manually save the profile (if such an option was added to e.g. the sort menu), so the most logical choice was to save it after every song.  I tried to make it as non-intrusive as possible by doing it in a background thread and starting as soon as the song finishes.  On my home machine it finishes saving before I can get out of the results screen, but I don't know how much slower an arcade machine would be.  It's also entirely possible that I've left a race condition somewhere; please let me know if you find any.
FWIW, I think you don't have to implement this callback if everything is valid.
Please add this (the English one) string into every the language file for every language append `#translate` after it so that the translators now that it must be translated. If it exists only in the English lang it will be impossible for others to know that there is something to translate, plus it will produce an error if the language key is not in the loaded file.
Not a big deal. Thanks for clarifying.
Fixed in eb35945
Very minor: No spaces between params? :-/
Missing newline before return cs.sensiolabs.org
you have extra braces here
I prefer `Dit type creditcard is niet ondersteund of heeft een ongeldig nummer.`.
The CS does not say anything about line length afaik. I checked http://symfony.com/doc/master/contributing/code/standards.html
(Fixed in next push as all the others)
... and the getters use magic values
you should use ``&&`` instead of nesting the conditions IMO
Eh... Seems like it's friday and I'm bothering ppl. with stupid questions =)
Thanks.

Not really. This is something for the docs.
Just curious :)

```
$session = $request->getSession() ?: null;
I think you might mean `input._isUTC` instead of `moment._isUTC`.  The `moment` global does not have an `_isUTC` flag on it, only instances have the flag.  There is a function available if you want to typecheck moment instances, `moment.isMoment`.  However, you said that a moment's `_isUTC` flag is ignored when you call the `moment` function on it.  Maybe the true bug lies in the moment function.
Yeah, that would be more informative.
"Indent continued lines two spaces" [link](https://github.com/thoughtbot/guides/tree/master/style#formatting)

So should look like:
```ruby
  <p>To see the results, <strong>
  <%= link_to 'click here',
    event_url(@event),
    html_options = { style: 'color: #EC6E4D' }
```
In-line with `locals` on line 2
lol is all I really have to say about this
Non c'ets un simple enum non annoté donc un int va bien (j'ai vérifié en base)
yap, right you are.
In that case we will have 2 loops, one in `littlechef` and another here (for evaluating that it doesn't start with the excluded role prefix).

If we need performance, we should copy the needed code from littlechef and put the evaluation inside the loop.
...same with this and the following values.
I don't mind dividing the 12000 range up with this at all really. Just didn't know if we wanted id's split like this. The main issue would be if 12010 is used here, no compile time errors will occur if it's also used in process/. Though really we face that issue anyway, so maybe it's not really that big of a deal.
When the first start container with XTSServiceTestName, the XTSService class start() will invoke the test class. if we undeploy xtstest.war, this will fail with exception. In fact, we need to undeploy xtstest.war when the recovery has complete, but the container is shutdown by byteman "killJVM()".
We should filter out elements which contains panels. I think it's better to avoid bugs where it's possible ;-)
This indentation looks pretty wrong. Do the open tags correspond to the close tags properly here?
1) and 2) In k-fold crossvalidation, we just create one monitor. This monitor monitors each fold distinctly and if we don't reset the monitor, we get wrong total number of batches (we want to see only the number of batches for one fold)

3) It will more computational and memory expensive to create a monitor for each fold in the case of k-fold cross-validation. If you think that the computational complexity will not affect the performance of the k-fold cross-validation, we can create a new monitor for each fold.

4) We will modifiy the comment if the method is needed.
On 03/07/2012 18:58, albertopq wrote:
>>         var onMouseMove = function onMouseMove(event) {
>>           currentPosition = event.clientY;
>> -        if (startPosition < currentPosition) {
>> -          contactDetails.classList.add('down');
>> +        var newMargin = currentPosition - startPosition;
>> +        if (newMargin > 0 && newMargin < 200) {
>> +          contactDetails.classList.remove('up');
>> +          var calc = '-moz-calc(' + initMargin + ' + ' + newMargin + 'px)';
>> +          contactDetails.style.marginTop = calc;
> Done #2150
<3
@VictoriaGerchinhoren is currently preparing the sprites and I will include them all around the Communications app... ;-) I'll keep you posted! ;-) Thank you very much.
the `storage-space` class is undefined (and not required), please remove it everywhere.
response.response?
`load()` should be `init()`.
Heh I just spend 20 minutes debugging this, should have paid more attention to the TODO

how problematic is this to fix? I would prefer it be done in this pull request as its very confusing
Use one-row menu and insert the original character to middle instead.

Don't implement two row for now.
please avoid anonymous functions only for debug purpose although it is not compulsory
If you don't want a warning when manifestURL does not exist you have to :

      if (manifestURL in this.installedApps) {
          return this.installedApps[manifestURL];
      }
      
      return null;
I wonder if you want to abstract that at some point and add a method that returns you a dom element with the value set.
So we do it for all system messages then?

If that's the plan, this PR is ready to merge!
Seems like the `check_connection` and the logging can fall outside this block, since they are not dependent on anything but the configured option.
アドホックコマンドや テンポラリジョブの実行は管理者の日常業務の一部です。

→

    アドホックコマンドや Temporary ジョブの実行は管理者の日常業務の一部です。
Yes you do need that. We might as well add all the properties of scalar multiplication in a vector space. Maybe it makes sense to rename this class at some point? This is starting to look more and more like a Group.

Any thoughts Oscar?
maybe use 2
Ha... a curse is preventing me from using dict comprehensions....
Change ER-prepopulate to redirect.
Always pass $node.
Should be `integer|boolean` according to note
That's just going to DDOS sys.4chan, remove that.
I believe this wasn't necessary in Brackets because the editor was always bumped up against the right edge, but it's necessary in the browser.
Note that we no longer scroll before the updateDisplay(), so we only have to check our `preserveScrollPosOnSelChange` flag below, after the update.
Maybe all the above printing/logging stuff could be combined into a class to reduce some of the code duplication?

i.e.
logger = logging.getLogger()
you did. ``./composer.phar update`` does **not** installs the composer deps specified currently. It **updates** them and writes the new lock file. If you want to make sure your local setup has the right deps, you should run ``install``
should we keep the documentation for it (adding a deprecation warning) for now, so that people can still understand it ?
indentions of comments are jittering
Doh, yeah, I'll switch that over
OK, it makes sense. I was not trusting the user received string. But control may have been made previously.
external libs should be put in the 'register' method down below.
ahh you are right, forget what I said.
Did that. https://github.com/mozilla/browserid/pull/2830
@seanmonstar - this is one thing I was wondering here too, would completeTransition ever call the callback with a non-successful status or would it call the error callback directly?
I like this more generic approach.
suggest URI-encoding the parameter, just to be safe. Would be better to use a URI encoding library in general.
I think that we can even check for the nsILocalFile instance here.
Thanks.  I have this merged locally with all conflicts resolved.  I just need to clean up a few items.
Yes I will open the issue right now.
Won't this fail trying to call `exported = true` on `nil` when the resource isn't in the catalog?

That should probably be a hard error.
I added --no-daemonize as separate from PUPPET_OPTS as it is used elsewhere in genconfig. Even though daemon works due to a quirk of bash, which disables SIGHUP, I felt that it was not safe to rely on it [see so.q](http://stackoverflow.com/questions/11443802/does-bash-c-work-like-nohup). I also felt that directly capturing the pid file after execution was less brittle than relying on pgrep (since pgrep relies on a pattern to match the pid).
Don't worry about changing this since you've already written it, but I think it'd be a good idea for us to use the new `expect` syntax on all new specs going forward.
echo "- Fixed overwriting of default gw after closing NIC edit dialog using Back button." | wc -m
83

Hmm, so maybe the maximum it's actually less than 80
I think this actually does need to be an Iterator and not just Countable?
Yep, there is a todo in there for this, before beta4
Same here… and probably other places?
Actually, the original version of this line was fine; it was simply using the ternary shortcut. If $tableGatewayEvent evaluated to null, it would create a new instance; otherwise, it wouldn't.
Can you document the params?

@param string $query (optional)
@param  array|Traversable|SearchOptions $options (optional)
firephp/firephp-core
This is wrong. If you want line use `<hr>` and style it.
Spacing parens is a personal preference thing... which I don't care for.
Ditto for spacing inside brackets.
@harjis $tilaus_on_jo ei itse asiassa kerrokaan riittävän tarkasti pitääkö tähän haaraan mennä vai ei. Tänne siis kuuluisi mennä VAIN silloin ku JT-rivejä poimitaan "automaattisesti" joko tilaus_myynti.php failissa olemassa olevalle tilaukselle tai editilaus_in.inc failissa sähköiselle myyntitilaukselle.
If delegate or owning class changes the selectedDate, move to that month and show the selected date highlighted
Please change the function declarations in this Pull Request to have a space between the function name and the opening parenthesis.  This is more of my mistake, as I didn't document this style rule in the style guide.  I just [updated the styleguide](https://github.com/jeremyckahn/rekapi/commit/ec40a895387b57f7de1301c27b0e5cfa4200e8d0) to reflect this.
I actually meant to ask you about this in my original comment.  You are correct, keyword args will just be ignored for normal javascript functions, and I agree that some sort of error would be nice here.  I'll do my best to come up with something.
Nope, `action` is not required. http://www.w3.org/TR/2011/WD-html5-20110525/association-of-controls-and-forms.html#attr-fs-action
This docstring looks funky. Rewording it like this makes it fit.
```
"""
Staff group exclusive for user email addresses in AUTO_VOUCH_DOMAINS.
""" 
```
Yeah, when i wrote this there were 3 levels but I think we're supposed to add the 3rd one back in with locations as far as I understand. Doesn't your implementation use the so called `magic strings`?
same comment for line 173-174
This line doesn't do anything?!1
indeed you're right. I just added ``m`` as an attribute.
fixed (in new branch)
I tend to err towards brevity, so I could be wrong. Go ahead and keep it.
I like the mathjax'd `i`. I would suggest also adding `font-style: italic` to the style as well.
Well, the key is to make sure this passes even if the user doesn't have Pro installed. If that's the case, then ignore my note. If not, redundancy is pretty OK in features.
So, I started to do this but then ran into a timing issue.  The problem, as I see it, is that the Service is initialized with the Environment before the server instance is created.  Thus, the problem becomes how does the Service "know" when it can access the Environment's Server to get the required information?   The nice thing about the mechanism here is that the Service gets this callback when the Server is already initialized and started.  Thus, all the critical information, such as the dynamically allocated ports, has been determined and is available.

Is there some hook, or some lifecycle event I can use to accomplish the same things?
$startTime, if possible, please. Same below, of course.
странный стиль именования. не то ни се
`manipulate_page_list` surely shouldn't be a bang method since it no longer manipulates `pages` in-place.
Lets depend on a specific version of mozinfo so that our package isn't broken when API changes in mozinfo happen.
Kinda bothers me that these class names are not uppercase
These are really sizes, not positions, so please use W and H rather than X and Y.
I would just do a Cassandra.rowToHash here
I haven't really given it any thought. I'm just making sure that there is some space between the type, parameter name and description. I don't think it matters other than for consistency, but it feels like it's probably a non-issue.
@sjyluc Should this be ``true`` since it is a nightly?
About something like that (I'm not in front of my computer)

```scala
name.take(prefix.size) == prefix
```

to check the case.
Looks good! One more thing that you might want to test is

    package a.b
    package c
``dispose()``
я не повинен бачити тут закоментований код. якщо код непотрібен, то його видаляють
This shouldn't be necessary. If you declare a Pig step in `steps()`, then mrjob already knows you support Pig steps.
Why is this needed?  Can't @phenotype_attempt.mi_attempt be used?  Why not just @mi_attempt = @phenotype_attempt.mi_attempt ?
I believe the name came from "Post Ingres", so Postgres makes sense.
`mutatePercentage` содержит проценты, а используется как количество элементов. Если ДНК будет содержать отличное от 100 количество элементов, то этот код перестанет работать правильно.
А еще лучше: arc4random_uniform([DNA count])
Опять же, у нас есть возможность сравнивать сразу ячейки одной ДНК с другой. Так давайте ею и воспользуемся, зачем плодить переменные?
Я думаю, в рамках нашего курса лучшим решием было бы
```obj-c
static const NSArray *nucleotides = @[ @"A", ... ];
```
+ extern или соответствующий метод `- (NSArray*) possibleNucleotides`
Я понял, на будущее буду иметь ввиду. Опять таки в свое оправдание :) могу сказать, что когда писал Cell то не подозревал, что похожий функционал будет в категории. Спасибо, за разъяснение!
Насчет того, что теоритически никгода не завершится, я бы поспорил (вероятность этого = 0), но то что решение дурацкое согласен.
okay making a comment on this line
It might've been before that convenience method existed. Not really sure, but it should be the same semantically.
This needs to be in `Dependencies` too.
All these documentation sections needs to be indented 2 spaces for parsing.

Also, please add `hubot` to the beginning of the Commands. Less confusion for people using `hubot help` :) Don't worry, hubot gets replaced with the actual name at runtime.
@codyhoag the second part of this sentence seems a bit vague and maybe even frightening (from a security standpoint) "offers users unlimited freedom to your site" :-). I imagine you wanted to express the freedom that OAuth gives the user to add valuable data from third-party sites they use.
This should probably be "Hello" instead of "Hell".
注释好像跟实现不符? 注释说`otherwise returns nil`, 但是实现的是没有`upper`的话会抛异常
maybe epochs.average should accept picks ...
My bad, removed it from doc and code but not from parameters. We were on the same page that the arg is ugly and therefore added picks to epochs.average
make

should be enough.

it does make clean, make in, make test

we should add to it "make test-doc"
@gugahoi @mrkipling this is the line that is giving be trouble. If there are enough modules on the page to exceed browser height when pressing back the whole body of the page will be scrolled to the library item. The effect I'm looking for is only the library module scrolls to the library item. If one you you jquery gurus can figure it out that would be great, otherwise as a temporary fix this line could just be commented out until we can find a fix.
Probably get rid of this local variable and just call: return getConcept(......);
Actually, you don't need to extend from AbstractType, only to implement the FormTypeInterface. Extending from AbstractType is only a way to avoid redefining empty methods each time when you don't need to customize all of them
@WouterJ No, the SE always used the ClassLoader component for the autoloading in 2.0
This is incomplete. A working example should be

```php
->arrayNode('extensions')
    ->prototype('scalar')->end()
->end()
```

Btw, ``fixXmlConfig`` is only meaningful for prototyped nodes as its goal is to take care of the special case for XML (node vs array of node depending of the count)
That is actually a multi-line var declaration, hence the indentation level. It would be equivalent to
```javascripnt
var direction = (event.keyCode == 38) ? -1 : 1;
var input_scrollback = activeChat.get('input_scrollback'),
var new_position = input_scrollback.index + direction;
```
Yup, I've inserted standard HUB4 dictionary here. I don't know how (and if) using general dictionary instead of a more tailored one affects model's performance though.
I don't think the prior implementation was by any means 'clever': it was downright useless for debugging.

That said, what exactly is gained by decomposing failure_message? It seems to make it less clear.
This method is strange in this case since it does not verify that the connection actually belongs to the internal system (which should be removed anyway). I'd suggest dropping this method altogether and just tell people to send a PoisonPill
doesn’t work: there is no system available while constructing MailboxType :-(
What's appropriately? I would expect either 4 spaces or matching the bracket on the line above (so in the example below ``"Oh`` would like up with ``PyExc``).
done (final)
renderItem would be consistent with existing namings. 
renderItemRow means we are introducing a new name "row" which doesn't appear in any of the existing APIs for CDetailedView.
Note that while default CDetailView is using a table row to display an attribute, users can customize it to be floating divs or columns. That's why I think the row is inappropriate.
Плагин зависит от внешних либ, поэтому лучше по дефолту оставить его в OFF.
Пробел после скобки лишний.
Another case where you can use recur.  And this time i'm more sure it's in the tail position.
Sorry for missing this before. Lets make this a list.
might want to revert all of these line deletions to keep the git history clean
why not refactor this sleep pause is the abstract parent ? 

function pause($time)
{
    for ($t = 60 * 10; $this->running && $t; $t -- ) {
        sleep(1);
    }
}
Yes, absolutely. This would be much better.

________________________________________
From: Elad Ben-Israel [reply@reply.github.com]
Sent: Wednesday, March 14, 2012 10:40 PM
To: Yosef Dinerstein
Subject: Re: [node-wintools] Parse XML output of WMIC. (#1)

> -            if (!e.cmd) delete e.cmd;
> -            if (!e.prog) delete e.prog;
> -
> -            if (e.pid) {
> -                output[e.pid] = e;
> -            }
> -        });
> -
> -        callback(null, output);
> -    });
> +* Returns all the system processes
> +* @param filter {function(p)->Boolean} A filter function executed on each process before pushing it to the list.
> +* @param callback {function(err, list)} Called with the list of all processes.
> +* @remarks Runs only on Windows (uses WMI)
> +*/
> +module.exports = function (filter, callback) {

May I suggest that instead of `filter` we will receive `options.filter` so we can have it optional (the options object is distinguishable from the callback function so it is easy to make it optional):

```js
function(options, callback) {
  if (typeof options === 'function') {
    callback = options;
    options = {};
  }

  options = options || {};
  options.filter = options.filter || function() { return true; };
```

---
Reply to this email directly or view it on GitHub:
https://github.com/anodejs/node-wintools/pull/1/files#r559415
The code convention for JavaScript is camelcase, could change `server_express` to `serverExpress`?
Why is this not a static?
apparently this is valid...
Yes, how would we identify a task's data otherwise. Yes to name the cache!
About AssertJUnit, it's not a problem, I just found it odd. It might be Galder's preferred method, but I didn't know about it :)
Step 2 in line 878, step 3 in line 880, step 4 should be somewhere there... i'll add
logic has changed a little bit. 

before we used to check as fallback if primary language is allowed, and then return the primary locale (regardless of anything).

Now with the validator, a list will be compiled (dependend on the strict parameter), and the first / highest priority q-value in accept-language will be used
The `IndexError` comes from `args[1]` being out of bounds. The `ValueError` comes from the casting to `np.float` being invalid. This is the case in your example of `stem(x, y, 'r--')`.
Docstring glitch.
Again, `pyflakes` flagged `b` as unused?
These won't show up as links.
Could you add a space after the first comma? Like so: `plt.plot(x, y, label='mylabel', lw=10)`.
I didn't know about those sub-folders. Will definitely use one in the future.
I'm not sure I understand @jds2501 here regarding GaiaTestCase. It seems fine to me to have this in setUp.
Use the platform filesystem paths, never hardcode paths that can't work on Windows
I make a mistake when I did the commit : only the modifications in s23_site.js have to be considered. I need to do these modification in order to use the hybrid.
link to http://dev.sourcefabric.org/browse/CS
Máš pravdu, má patřit před něj. Díky a omlouvám se. Už jsem commitoval opravu.
Shouldn't this be the minimum amount of cache a database can have, not a connection?
Are you sure ``$channel`` has the correct type as "object<Zend\Wildfire\Channel>" is not a sub-type of "object|null"?
Are you sure you want to override the parameter ``$options``? This often makes code less readable.
-  if(_.isString(match) ) {
    +  if (_.isString(match)) {
Por alguna razón. no funcionaba con esas líneas, tira error de for, prueba en local.
latürnich.
Das lässt sich ev. mittels Generics oder Class Parameter in eine Methode in die Abstrakte Model Klasse auslagern. Ist ja bei jedem Model dasselbe.
styling: space before {
It's not clear to me how "a regex" would be of type String[].
I'm a fan of two-space indent as well, but we use four spaces for indentation everywhere else in the project. Please convert to four-space indentation here.
Here [you go](https://github.com/mitar/django-mongogeneric/tree/bugfixes). Try it out.
No, you don't understand my question. How we are getting settings variable to be available in the template in the first place? Can you find one such variable and grep our source code?
So use `self` everywhere please, where you want to access current object.
Oh, you are already passing `data.user` here. :-)
Rename to `weather_tasks`.
Use `blocktrans` and translation interpolation. Do not concatenate translations, what if in some language e-mail cannot be at the end?
Ups. Use `data` here. And pass into constructor `data.user`. It is much better (now that we have a separate dict only for user fields).
yes.

here is whole error log :
http://www.rubik.si/klub/TypeError.htm
`url` tag knows how to put results into the context itself. :-) [See](https://github.com/wlanslovenija/nodewatcher/blob/master/nodewatcher/frontend/templates/registration/registration_form.html#L16).
Right, but I'm not interested in that - I want to know why you changed a method that is accessible publicly, despite my warnings that user _do_ _use_ these.
note: the opera spec say default to 1, obviously it's safer for us to default to 9 - but perhaps a note that this is our default, not opera's
I understand it could be clearer, but what is it you think is best? Should I move the null test into the `set` method? Or is it the missing `x` from `a ? x : y` that isn't good style?
What about removing the two temp variables here as they make the code less readable?
Bind is ES5+ and we're already using `[].slice.call` without `Function::call.bind` etc.
Shouldn't need curly braces here
Here too you can do:
```javascript
var crafter_request_id = this.allocated_res_ids[res_id];
if (crafter_request_id)
  logger_entry.crafter_request_id = crafter_request_id;
```
(Assuming the id can't be 0 here.)
Why it can't return value?
Please move useless comment.
ftr: the impair minor versions are testing releases of jackrabbit, so the current stable is 2.4.X, currently 2.4.3.
а [2] уберешь заодно?)
Nope, still uses Math.sqrt()
i think the only idea was that the model would only be updated when the value was valid rather than updating all through the drag even when the limits have been hit. but you're right, the value is never committed when it hits that upper/lower bound so that other code is just sitting there doing nothing but pretending.

pretending.
Anyone knows a better word for fade?
Hmmm, very sorry I didn't check that!  Will look and fix...
Fixed and fixed.
Minimum donations may have a problem, where for a $500 project, someone donates $499.99, making it impossible to finish the project. Need a way around that, but since Stripe imposes minimums, might as well add a $1 minimum.
I'm basically redeclaring some things that are in RavenClient.m. I'm not sure if theres a better way to structure these kinds of things.
I could not find these classes: Email, Required, Model.  Does any one know how to fix this? 

import play.data.validation.Email;
import play.data.validation.Required;
import play.db.jpa.Model;

jars are present in classpath, 
I'm using play 2.0.4
Me parece que não temos nenhum ganho real com a criação deste método, haja vista que não existem atributos de relacionamento na tabela de ligação entre ``sponsor`` e ``collection``.
Requires `"minimum-stability": "dev"`
This file may not be needed.
I don't think `vector_pop` qualifies for a return value. Popping from an empty vector should be a programming error (and hence `assert`ed.
Definitely not stupid, I totally get it. As far as spaces and not tabs, the software I use is set to use 4 spaces in place of a tab.
You don't directly set member fields in the html.  In the html you have a form that posts the data to a controller, which then creates/alters the object, and then calls .save() on the object to put it in the database.
This can be dangerous if you don't make it explicit the the t1 and t2 have to be the same length, if you look below, we take the min size of the inputs and output.
I made it an enum so debuggers can help us figure out what's going on, but you're right, it makes more sense to put it in the existing bitfield.
Readability is subjective:
Your way
![yourway](http://content.screencast.com/users/talkitivewizard/folders/Jing/media/fbc18a6c-06ce-4245-9c45-02404dcffe7a/2012-12-06_1043.png)
My way
![myway](http://content.screencast.com/users/talkitivewizard/folders/Jing/media/08542fd6-954a-4959-89e5-eca15b6ea690/2012-12-06_1045.png)
Just remove this one proc, and the rest is fine.
no @stof, have you actually run the test in your local?

it does something different
@ywen you want me to revert since you merged your fix?
Надо сделать метод, который будет закрывать один стрим.
Этот код полезно закешировать.
I don't particularly like this variable name, especially since it's going to appear in the XML. @ge0ffrey - feel free to change it to something better, if you feel like it.
We should encapsulate the Time.now into a randomness method as was done in the POC (and the OAE before that). That's one of our most frequent test fixture needs. Not sure where those utils should live... spec_helper maybe?
s/meeter/meter
Are you sure ? It seems that my implementation match the current one : 
* `filters` are optional GET parameters
* `requirements` are route placeholders (mandatory) GET parameters
* `parameters` are FormTypes/JMS Serializer Metadata (POST/PUT parameters)

But maybe i missed something ?
where? (do you mean $str .= "\n" instead of $str.= "\n". i can change that, but i thought it was custom to not use a space then.)
Can we convert this to a index based for loop?
了解。
Duplicated section.
plugins, etc can add non tag producing components to wrappers, so if you wanted to be a good simpleform citizen, you'd add your component namespace this this list.  I mean, I know you can modify the constant in place, since it's Ruby, but this would allow you to do something like this:

```
module SimpleForm
  module Components
    module Whatever
       SimpleForm::Wrapper::Builder.non_tag_components << :whatever

       def whatever
         input_html_options[:'data-whatever'] ||= whatever_value
         nil
       end
      
       def whatever_value
         options[:whatever]
       end
   end
end
```

That said, I could abstract this so you're not directly addressing the builder class.  It seems that theoretically the responsibility to know whether or not it produces a tag should really lie with the component definition - so maybe we move all of this logic there?  That sounds like a separate patch though.
If you put it within the ``<artifactItem>``, then it will only apply to that specific dependency. In this case it doesn't make any difference, but I still think it's better to have it outside as that folder will be good for any additional dependency we may add in the future...
This is not pep8.
> 2 lines before functions

You are right. I fixed that
On Fri, Sep 21, 2012 at 10:54:16AM -0700, Mathieu Blondel wrote:
>    Not that I know of but I hope that users have enough common sense to not
>    enable n_jobs at two different levels :)

I dream of a world in which joblib had a 'grand central dispatch'
pattern, and thus multiple levels of parallelism would be possible. That
world might happen at some point...
ok, never mind. thanks for checking :)
Or maybe no exception is raised and the data is converted / copied to floating point values silently? In that case it would be interested to check the resulting dtype of the kernel in an new assertion.
I would rather say "any cluster has at least" instead of "must have", as it is not a requirement but a consequence of the algorithm iirc.
I have no idea about dbscan, though ;)
Sorry about his silly mistake... Will fix it :)
Yes. Old dmd code used the old style, but in recent expected code style has been changing. (It is not yet properly documented/announced, but Andrei likes the new style.)

Therefore I'd like to recommend it.
Andrei"s opinion is irrelevant, he never contributes to DMD. Unless Walter changes his style (or at least makes some comment about it), nobody else should either.
refactor the non-redundant components to before the `if`
Question: as getNativeHandle will now return the pointer to the container, how are we going to get access to the contained control?  Or is this not going to be needed (though naively i doubt that will be the case)
Weird. The '-' makes the difference. Good thing I have a TODO.
the signal being `valueChanged`, the slot should be named something like `setValue` or `onValueChanged`
To clarify: the problem is not to actually lock down the class, but to make sure a reader (such as myself now) understands what your design intentions are.
Right, I would base the choice on what client side tools were available not on what seems more light weight.  The difference if any is usually very slight.
Better to make the diff not touch any more lines than it absolutely has to, so it can be more easily reviewed and is less likely to conflict with other changes. I _think_ you are only really changing a couple things here but it is hard to see that. (OK for indentation to not quite match brace structure.)
yep, you're right. if annotation has option it is key, otherwise is value. covered.
I just tested this out in my console and it appears to work:

```js
var env = new twig.Environment();

var t = env.createTemplate(form_div_layout.html);
t instanceof twig.Template; // outputs true
```
hmm it was missing when i pulled last night to merge
I think that it should be default off for backwards compatibility. Plus I am not certain how many users will want it. I would much rather have those users turn it on and request the default be on.
please reinvent before committing.
Yes, this is stupid. But maybe there's something I don't grock; is there some reference between the proxy model and the person so that initially, when we create the proxy model, I can connect the signal to all persons? 

One option would be to add method for the proxy model to get a list of all contacts from the source model, and iterate that. But not sure how nice that is. 

Or what am I missing?
you figure that this thing shouts "put me in a function", no? You always have an Enumerable and you always put a specific attribute into a specific value.
I had actually implemented this exact things we keystone::client::authtoken (b/c its the same for all things that need to interact with keystone)

Can you see if this makes sense for you?

https://github.com/puppetlabs/puppetlabs-keystone/blob/master/manifests/client/authtoken.pp
left this as legacy if one want to "quickly" add a new target with only "system"
Looks like a regression here. target_uri.path vs "/#{target_uri.to_s}"
OptInt
Could you precompute this slicing operation? For example, add the following outside of the loop:

```python
subsample = np.index_exp[cy / 2:cy * n_cellsy:cy, cx / 2:cx * n_cellsx:cx]
```

Also, it couldn't hurt to break up this assignment:

```python
temp_filt = uniform_filter(temp_mag, size=(cy, cx))
orientation_histogram[:, :, i] = temp_filt[subsample]
```
Yes. This is how we denote that a user hasn't set their username.
This code will change it to `1.0`.  We have to go with the ugly `undefined` check, right?
I'm not sure the logic/flow is really any different in your version, but that could just be personal preference on my part.  I already re-wrote this function once because @shunter didn't like the original "nested if" flow ;)

From a technical viewpoint, is if(!this.isSampled(rgba)) going to be the same performance as if(this.isSampled(rgba)), why 'not' the statement if I have code that executes for both.
These two lines would be less confusing if cameraController were called cameraMouseController.
no they're not. I'll admit it was personal preference, since I thought white was a better default than blue, but I'll change it back if you're concerned about backward compatibility.
Should exit with status 0 when `--help` is passed.
Is not a big deal, but lets follow the pattern testNoExceptions for example. Wdyt?
Use TiC. PROPERTY_PASSWORD
@pingwang2011 this is **excludes** section
Why these versions of Ruby?

* 1.9.2 makes sense to me since it's the default on Heroku Cedar.
* 1.9.3 makes sense since it's the current stable release of Ruby.
* I haven't used JRuby in a long time. Do you want to support it?
* I've never used Rubinius. Do you want to support it?
* A lot of people still use 1.8.7. Do you want to support it?

I think I would recommend supporting as few options as possible.
nit pick: can all these URLs be structured the same way with the same host?
Ah! I knew it should exist. :) thank you!!
So this is going to start working once I've set up Travis for my own repo/account?
I have changed the implementation, and used a QHash of the details labels
It shouldn't be. The scala compiler uses laziness only in types, not in symbols... Every symbol has an owner, since they are always created through factory methods, like `owner.newValue` or `owner.newClass`. That being said, to my great surprise `TypeTree.symbol` is delegating to its type to retrieve the symbol. However, that looks like a bug or an exception and I'm willing to take a risk here for the `owner`. As soon as we get the API (yeah, I know, it sounds like empty promisses), this can be switched to the 'safe' API.
the raw makefile part is not needed, please remove it
No, it wouldn't make any difference. CTFE is not used here, and never has been. It's C++ style template metaprogramming which is slow.
The std.metastrings module predates CTFE, it should be taken out the back and shot.
> Here, this comment seems redundant (to me), since apart from 1 (already documented) overload, the input does have to be a forward range. So I don't see what this note is trying to highlight to the reader.

First, I'd like to have standard comment text on every member that works on non-ranges.
Second, there is no *words* like "does not have to be a range" in this ddoc comment so reader can just skip word "static" in sentence about static arrays as its contains "mostly for performance reasons" words which can be incorrectly interpreted as it is *only* for performance reasons (this is mostly for non-native English speakers).
Third, it can be obviousness for a newbie that static arrays aren't ranges (for my shame, it wasn't obvious for me today).

> Nitpick: acepts => accepts

It's not a nitpick. It's a typo fix. And typo is often a major bug. So it's a major bug fix. )
Nifty trick. :) `_testSave` should probably be `private`.
Can you explain the purpose of this block?  It causes an NPE running the one of the tests (when performing a delete bodyDict is null).  This is easily worked around by adding bodyDict != null to the condition, but I'd like to better understand the purpose, as the iOS version doesn't have this logic.
Well, it shouldn't have been, because it's not there.

See https://github.com/hpcugent/easybuild-framework/tree/master/easybuild/easyconfigs/b

Will fix v0.8 release notes.
Do something with these…
Logging is good, and a message when saving fails would be nice...
I don't believe this will leave logs anywhere unless you use `StandardOutPath` or `StandardErrorPath`, or the `-L` option to `chef-client`.
why the uppercase? i prefer lowercase, since that's how it is elsewhere and it's easier than having to hold shift.
Sure, no prob.
Don't mind it being a separate one. No rush.
you didn't like my wink?
This needs to be commented, is  a hack from the annotations pov, since it gets objects directly and uses annotation id as an object number, which is not necessarrily a hard requirement. Other storage can use the ann id differently.
Maybe we could put a test for the annotation class around it?
sure. JMSBot/php-code-analysis#17
ignore previous comment. Do all files camelCase styled.
`Build Pintab for pinning important tabs`. Ну и ON по дефолту можно вполне.
No need to changes the quotes in the license block here or anywhere throughout the codebase
what happens if such an uri is present but the machinery was not run / is disabled ?
I just created a test case which is a copy of the same test for OrderedAwareLinkedHashSet but for this class and it has pretty good concurrency coverage and its all good, but if you can write a failing test than we can see what needs to be done.
Remember this is a specialized wrapper, that is why its protected. . .
Anyway, I am writing some performance tests as well as others to see where is the value.
We may need to make AD configurable since this class is definitely slower on 'add' part
I would change all aliases to be mongo_default or odm_mongo_default because there are other ODM's (CouchDB, etc) supported by Doctrine.
Not sure why you want to include extra double quotes here?
I doubt you meant to define this method twice...
I agree that my regex might be a bit too rigid, but I worry that these might be a bit too permissive?  For example, wouldn't your setting regex allow there to be an equals sign in the actual setting name?  e.g.:

    foo=bar = baz

I would be happy to make these regexes more permissive but my instinct would be to do it by adding additional characters to the "\w\d" rather than switching over to \S.  Thoughts?
Try to avoid crossing 80 chars
Please reference the sparkbox repo.
`Zikula.Config.baseURL+'index.php?...'` could be used here?
@merk done.
It's an out-of-project default. The `core` is referencing an `app` implementation. It's not terrible but it might be a bad precedent (or perhaps we do this in other places?).
Test also successful if no Exception thrown!
Are we a global company or what?  :)

Should this be in the sdk or as an api endpoint?
$fbdata['verified'] is true if facebook has verified the account.

Not sure if this means the email address they've given is valid or just if one of the emails on the account has been verified / user has verified a mobile number.

I think redirecting to another login form is probably the way to go.
"for more information" -> "to see all the details firsthand"
in general it seems like this could be abstracted as a mapping between strings and positions
reset.css ro ham ezafe kon albate az root proje
think  zaim cut and pasted
ah found it get_or_create() function, will play with it
This should be a sensible default, like false and not the string "false".
This appears to be unused now (?!?) and may need to be marked deprecated or have further investigation.
Can you add more about what `arg` actually represents here? It is general user data for a Message
NM, I overlooked "type: Font" line
Why is this assigned to an undeclared object property?
Ah, true.  I forgot to fix that.
Can you fix this line, too? :)
is it still needed?
`Date.now` - статический метод, `new Date` - создаём объект - потребляется память. `getTime` не обязателен.
`Date.now` не поддерживает только опера версии < 10.5
add new "receivedLocalNotification" event if the developer want to get all local notifications.
Try/Catch should already be done by Jasmine. If post setup is crucial probably the best thing is to wrap in a describe and use afterEach.
Those tools are listed as hack submission tools, not event listing tools. Not sure I want to be rude but I simply just never heard of any of them and simply don't want to send people to sites that don't get many useful entries for hackdays anyway...
2 useless new lines