labofsahil/github-event-dataset-2012 huggingface dataset:
I think this is sign of premature optimization at cost of code maintainability. According to that benchmark reduce took  0.00323918114 ms. In addition these benchmarks does not really apply to this case as since they compare simple array
iterations which is different from our case where we wrap xpcom simple enumerator with JS iterator wrapped in yet another JS iterator + access to different getters. Cost of reduce is much lover than these operations and completely irrelevant. In addition it's known that built-in map / filter / reduce is much slower than hand written one [bug 743634](https://bugzilla.mozilla.org/show_bug.cgi?id=743634) also visible here http://jsperf.com/eval-join-vs-reduce/4 and finally JS team doing some awsome work on function inlineing which will be able to compile such filter / map / reduce to a same thing as plain `for loop` so we better let them do optimizations. We can start doing targeted optimizations once that will be a bottleneck.

That all being said I'm ok with keeping it as an internal utility function for the time being.
"which makes it possible"
`_times[id] = undefined;` would work too right? and iirc Eddie's presentation it is faster.
So if I do:
```
class_list_converter(['Alpha', 'Beta'])
```
I get a TypeError
I think we should explicitly exclude any relative paths from `$PATH` / `%PATH%`. We're doing this for security reasons with custom facts, it makes sense to be consistent and also do it with our executable search path.
It is entirely opaque what this is used for, but it looks like it is used to stand in for a real parser scope.

That suggests that, at the very least, this needs to change API when any consumer of the parser scope changes, along with the parser scope itself.

That feels both awkward and risky - I think it would be better to use a real scope object.  If there is some reason that isn't possible, better to modify the scope class to make it possible than to create this stub object.
Replaced previous regex with a better one, and used captures to return both sides of the # (or more than one #) in 5f607bb.
Is this the real behavior of this? I assume that by throwing this error you are trying to avoid something else from happening.
I forgot to clean up these comments and a couple of associated loose ends. This one comment I don't plan to act on now so i've just removed it, the rest I have acted on.
Unless this is used in more than one terminus implementation, we can squash it down into the actual hiera endpoint.  We might as well do that - the "code reuse through inheritance" approach is not awesome.
I'm seeing a funky character now. You might want to double check that the encoding is still UTF-8
Hi @ralphschindler 
PHP handles objects just fine with its instanceof operator. However is_subclass_of accepts also objects and the is_subclass_of method is also broken for objects.

see:

php5 -v

PHP 5.3.6-13ubuntu3.8 with Suhosin-Patch (cli) (built: Jun 13 2012 18:02:19) 
Copyright (c) 1997-2011 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2011 Zend Technologies
    with Xdebug v2.1.0, Copyright (c) 2002-2010, by Derick Rethans

php5 -a

interface A {}
class B implements A {}
class C extends B {}
var_dump(is_subclass_of('B', 'A'));
bool(false)
var_dump(is_subclass_of('C', 'A'));
bool(true)
var_dump(is_subclass_of(new B, 'A'));
bool(false)
var_dump(is_subclass_of(new C, 'A'));
bool(true)

About performance: The method is basically yours (taken from Zend\Di\Di), i just added the case-insensitive behaviour. I will look into this and see how this can be improved. Thanks for your review.
Hi cgmartin

I don't want to be looking at a complete refactor here. It goes beyond the purpose of the changes i needed/wanted to complete.
Would be nice to align these.
Replaced - and was me. Sublime Text 2 defaults kicked in on me.
@BinaryKitten graciously corrected me on IRC that passing the $labelPosition doesn't help - that FormLabel doesn't render the element, and lines 96-104 would still be required. Carry on! ;)
I don't know what happen here but this Component is using the new exceptions. This should be, at least, `Crypt\Exception\ExceptionInterface`
the authorization will happen for each method, also for index method
@EdGruberman Indeed, I stand corrected. I've always been under the impression that line breaks are frowned upon (just from the glances I've had of the import lists of Bukkit source files). Thanks for the knowledge. :)

Based on the above, I +1 this pull. ;)
why injecting the container instead of the real dependencies of the class ?
Session has access to the datatable so storing is possible. If you can make the change to that (I think it would be better) then i can pull.

Either way it can't be auto merged for some reason.
I'm only talking when it's alone. It doesn't feel clear to me, that's all.
Just for completeness and safety, can you add a `2` limit to the explode?
because i didnt know where else to put it
That's seriously weird. that shouldn't be true, I don't recall having to do that with the bootstrap gem I've used.
Between RDEPEND and DEPEND.
I'm not sure why a style has a PHP requirement, but this was in michael's original commits so I left it.
I think this sentence might be clearer without the word "each" in it
Shouldn't this be _get_involved_header_link?
Any reason for the `u` over `user`? `user` is a better variable name, though it's not a big deal in this context.
Comment should end in a '.'.
4 spaces of indentation.
Existe um problema de sanidade neste método. Quando o xml não é válido, é retornado ``None``, e quando é válido é retornada uma estrutura de dados Python (dict, listas).  Outra coisa, como a passagem da XSD é opcional na instanciação do objeto, a validação não deveria ser obrigatória no método ``deserialize``.
this is the only part i still prefer, i dont care much for the linters
Dude again! Careful. Don't use tabs.
why did you remove the admin checks from here?
That breaks Linux hosts:


    facter -p hardwareisa
      unknown
    facter -p architecture
      amd64

Perhaps the solution is just to swap the two facts around, so hardwareisa is checked first?
Can we do this without passing in the url? ie did you have any luck using `url not about:blank` ?

I would rather not have to consider the url particularly if it's an unknown app being opened by another app or by a click. I'd like to make the method more generic, is what I mean.
You can do some additional testing here using `ZU.cleanISBN()`
Committed fix with $object removed from the call.
I was thinking since there were six points, we could use the six primary and secondary colors.
But it might also be stylistic to have only three colors, one for each line.
"interpreted relative to remote_user homedir"
Thanks, I'll take a look on that. I just used the same one that there was already used in the old version.
Oh! Got it. I'll change it accordingly. And I'll add some documentation about it in the README.
I think we want to make sure *something* can access the original API. Otherwise each extension that manipulates pages would have to account for all the ways previous extensions changed the path (for example, the "is this a blog page" regex would have to take into account the possibility that directory indexes might be enabled). Handling everything from an unchanged, predictable "source path" perspective and then allowing extensions to manipulate the destination however they want is a lot easier.
Call it a nit or pedant but I don't really like lumping everything into one variable and then checking it.

For reasons of it being more like a manual test case and more readable I'd rather do:
`for review in review_page.reviews:
    Assert.false(body in review.text)`
Why didn't you normalize from <code>'most_users'</code> to <code>'Most Users'</code>?

Using the link text in its natural form - what appears on the web page - would improve test readability. The <code>sort_by(self, type)</code> method already replaces the spaces with underscores and changes the text to lowercase.

Feel free to disagree with me .. but let's choose a pattern and replicate it across tests.
Yeah, something as simple as that would probably work.  My main problem with using a module is that people might try to mix it in and it wouldn't work how they expect.  You could have it raise an exception if it's mixed in.  If a class were used I would probably do something to warn against inheritance since @@ variables have surprising characteristics IMO.

You could summarize my opinion on configuration libraries is that I've seen so many of them get out of hand and become overly complex that I almost always prefer something stupid simple.
Can you take out this comment, though?
Where is this function defined? I don't see it in this patchset, though I could be on crack.
Please move this to a util.js file.
multiget fixed.
Also, callback should probably be an optional function.
Facebook needs additional configuration
I see why there is no return... but how do you signify failure? (i.e. either this func should return a bool so it can indicate success/failure, or it should have some OOP-y way to indicate it)
Hm, the `zendframework/zend-http` have a require section. 
```json
            "2.0.0": {
                "type": "library",
                "homepage": "http://packages.zendframework.com/",
                "repositories": {
                    "type": "composer",
                    "url": "http://packages.zendframework.com/"
                },
                "name": "zendframework/zend-http",
                "description": "provides an easy interface for preforming Hyper-Text Transfer Protocol (HTTP) requests",
                "license": "BSD-3-Clause",
                "keywords": [
                    "zf2",
                    "http"
                ],
                "autoload": {
                    "psr-0": {
                        "Zend\\Http": ""
                    }
                },
                "target-dir": "Zend/Http",
                "require": {
                    "php": ">=5.3.3",
                    "zendframework/zend-loader": "self.version",
                    "zendframework/zend-stdlib": "self.version",
                    "zendframework/zend-uri": "self.version"
                },
                "version": "2.0.0",
                "dist": {
                    "url": "http://packages.zendframework.com/composer/Zend_Http-2.0.0.zip",
                    "type": "zip"
                }
            },
```

You can see it here http://packages.zendframework.com/packages.json.
That's really weird.
Take out the log statements?
Why not? The reason they are there is to achieve uniformity. Some methods were returning integers, and others were returning strings. When data types are not uniform, strict comparisons (`===`) fail.
This outlines exactly why this function does it the way it does.
Очего бы он должен был вывалится в той строке?
Если malloc воспринимает параметр как unsigned it, то значение отличное от 0 (а percent гарантированно не 0 ибо есть проверки) выдаст массив минимум в 1 элемент величиной. Иначе наверно вывалится сам malloc.
Ох и классов же вы нагородили :-)

Не могли бы вы прояснить, как работает функция mutate ?
Проблема в том, что она, на первый взгляд, вовсе не меняет ДНК клетки (а меняет какую-то его копию, причем удаляет из копии X генов).

Из 10 запусков программы с параметрами
        [cell01 mutate:1];
        [cell02 mutate:0];
Все десять дали результат:
        77
        77.

То, что 
        [cell01 mutate:50];
        [cell02 mutate:50];
дает 77 85 - загадка, которую пока не разгадывал.
Спасибо, давно на сихе не писал. и забыл что тут есть до вайл
это лишнее, так как #import и так проверяет инклуды. 
Раз уж переходим с С/С++ на ObjectiveC то надо брать оттуда все лучшее ;)
The test above already checks the function's return value, this test was to ensure NotImplementedError is not raised if a secure random source is unavailable.. but I suppose the above test will fail implicitly if anything is raised so I'll just remove this test altogether.
You might want to check for these functions before checking sector types, since what is watery or open air has been abstracted.
How about adding a new method, CanCloseBrowser(), and then call it from here? This way we don't have to use #if defined(OS_WIN).
`expect`s unfortunately (sometimes?) don't fail the unit test when they trigger from a background thread. If possible, these assertions should be rewritten using the main thread.
What state is the folder of that download in after it fails in sab?  Could you post a directory listing?
I'm trying to find a similar download that fails on my providers.
Is it possible for progress to be > 1.0 or 0.99999999999 due to FP rounding issues?
not I guess you meant
don't put too much logic in the find_sources.

anybody with a bit of numpy experience can do : source_idx = source_idx[score > 0.05]
I saw it today, but didnt explore. Good to know

On 27.09.2012, at 23:21, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/artifacts/ica.py:
> 
> > +    if noise_cov != None:
> > +        whitener, _ = compute_whitener(noise_cov, info, picks, exclude)
> > +        del _
> > +        data = np.dot(whitener, data[picks])
> > +
> > +    elif noise_cov == None:
> > +        data = _zscore_channels(data, info, picks, exclude)
> > +        # data = data[picks]  # bypass
> > +
> > +    data = data.T  # sklearn expects column vectors / matrixes
> > +
> > +    print ('\nCalculating signal decomposition.'
> > +           '\n    Please be patient. This may take some time')
> > +
> > +    ica = FastICA(n_components=n_components, whiten=True, *args, **kwargs)
> > +    S = ica.fit(data).transform(data)
> you can use: S = ica.fit_transform(data)
> 
> —
> Reply to this email directly or view it on GitHub.
>
Actually my plan was to remove this RawFromMergs as soon as it has proven superfluous. The fact that I dod not visit this part of the code for long time clearly argues in favor of dropping it ;-)
Yes, I know it's not exactly what we want, but it's the smallest change needed to make it work. I think we should be simply returning concepts here.
According to the spec, the way to terminate an SSE session is for the client to close the connection, which is technically not incompatible with keep-alive, since the client can close the connection at any time with keep-alive too.  I think in the case of SSE, keep-alive is irrelevant.

If the connection is closed for any reason other than the client closing it, then the client will just reconnect.  So, it would actually be wrong to finish the enumerator stream on the server if using SSE, the server cannot initiate an SSE session end.  The way to do it would be to send an event to the client, which would tell the client to close the connection.  However, if the enumerator does finish, obviously the wrong thing has been done by the application code, but we still need to clean it up.  Since there is no way for the application code to interact with this channel again, the only thing we can do is close it.

And of course, if using this method without SSE, ie to stream a non chunked response with no content length (there are valid use cases for doing this, eg some clouds out there have old nginx versions doing reverse proxying which don't support HTTP/1.1), we must close the connection.
Find another way to detect repositories. These are slow, which, in turn, will make the prompt display slow.
You should break the line after the first word that crosses the 72th character
Should we split these two lines across multiple lines like in the previous code sample?
I believe Appraisal only depends on `shoulda-matchers`, not `shoulda` itself - could you change it to `shoulda-matchers` instead?
Can we narrow this down to specific chapters? It's a pretty big book.
We usually increase the version when we plan to release a maintenance update. Here it probably makes sense...
I think this line does not belong to the tutorial, at least not this early.  Also, it doesn't help a person who wants to get started, IMHO.
How will this work on Windows?
И вот это все счастье тоже. И сделать его константным, ибо едва ли тебе потребуется менять размер эвента или длину буфера.
Ня, `const` — это хорошо.
Потому что через него определяется какой пост ты используешь: draft или из remote entries
Es cierto, a mí se me complicó porque lo empecé en Javascript y ya estaban los nombres en camelCase, pero no es difícil arreglarlo, mañana lo hago.
El mismo espacio de sobra acá.
Why are you calling RevertUpgradeConditionsToRelevance ?
Wouldn't make more sense to just import it in that case?
$fp 是作为数组用的，是不是应该初始化为数组？
I'd like to make this function atomic somehow (I should have put a todo here) so that only the fields sent to this function are updated and not everything else. So if two of these come in from different threads changing different values, in the end both changes get saved instead of having the last one to hit the update statement win all as is the case now. 

I'm not sure how to do this in SqlAlchemy, but saw Nova was doing something similar to this to update instance info in the compute manager and figured if it was good enough for Nova we could use it in the short term.
Before this you need to checks whether a-seq b-seq is empty, and return `false` if so.
Btw, I still think what I said in an email:

> It looks to me that JBoss Libra is only covering the most basic/simple case.

> We had the discussion already about EhCache that had enabled memory based eviction, and we got feedback that it was a PITA to maintain (I heard it from someone...) and that it had tons of exception rules, i.e. 32 bit, 64 bit, JVM vendor (JRockit, IBM…etc)

> However, you just need to look at EhCache's code to get an idea of the the potential number of cases that you need to deal with, have a look at:
http://svn.terracotta.org/svn/ehcache/trunk/ehcache/ehcache-core/src/main/java/net/sf/ehcache/pool/sizeof/JvmInformation.java
There's no need for the catch section here. Any exceptions, if they happen, they'd be thrown directly from the put and would be propagated to the top of the test and be logged by TestNG. 

You might also wanna check what the effect on the contents of the cache if an exception is thrown by an async listener. IOW, after the put, does a get() return what it's just been put?
If you've gone the length to measure the difference between the two mechanisms, might as well you have added a test to VersionAwareMarshallerTest comparing them (as well as verifying that it can be marshalled...)? 

I agree it might extra code than what you currently write, but I still think it's worth doing it.

Have you also compared the speed of both methods? With Serializable you might have to do classloading to load the class (unless JBMAR optimises that somehow). With our externalizers, apart from smaller payloads, there's no classloading (unless it's in AS7 modular env) ...
This should be `const char *` unless we have a need to modify it.
I mean to move the block initialization into the `beforeEach` block, instead of doing it out here.

Specta might invoke this after the outer block returns because I'm pretty sure the `it` blocks just register things to test (they don't actually occur immediately).
I don't see a ```from __future__ import print_function```.  We need to target py2.x when we code, and allow for 2to3 to take care of any py3k issues for us.
Попапу-то все равно, нам не все равно.
Смысл в том, что попап должен оставаться в пределах своего view, чтобы, когда view перерисуется, попап бы тоже перерисовался.
знакомая тема, натыкались
I am on my way out, and can't deal with it right now, unfortunately.  It should be a pretty simple fix.  The `singular?` function is in `forma.matrix.utils` with tests and examples.  

```clojure
(if (singular? (i/mult foc-mat (i/nrow ts-mat)))
   [nil nil]
   ...)
```
It's not primitive, it's an array. Try destructuring it like you did
before. It just has a different toString method.

On Thursday, July 19, 2012, Robin Kraft wrote:

> >
> >  (defn parse-line
> >    "Accepts a vectorized string, as outputted by `text-map`, and
> >    returns a single hash-map with the GADM ID (integer) as the key and
> >    the ISO3 code (string) as the value."
> >    [x]
> > -  (let [[iso gadm-str] x]
> > +  (let [[iso gadm-str] (string/split x #",")]
>
> That returns a primitive type - how do we work with those? `(prn (.split
> "asdf,asdf" ","))` gives `#<String[] [Ljava.lang.String;@70751932>`, not
> the actual string.
>
>
> On Thu, Jul 19, 2012, at 09:38 PM, Sam Ritchie wrote:
> > >
> > >  (defn parse-line
> > >    "Accepts a vectorized string, as outputted by `text-map`, and
> > >    returns a single hash-map with the GADM ID (integer) as the key and
> > >    the ISO3 code (string) as the value."
> > >    [x]
> > > -  (let [[iso gadm-str] x]
> > > +  (let [[iso gadm-str] (string/split x #",")]
> >
> > Better to do this since we don't need regex:
> >
> > ```clojure
> > (.split x ",")
> > ```
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/reddmetrics/forma-clj/pull/150/files#r1203066
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reddmetrics/forma-clj/pull/150/files#r1203162
>


-- 
Sam Ritchie, Twitter Inc
703.662.1337
@sritchie09

(Too brief? Here's why! http://emailcharter.org)
@gsilver Can we leave the `px` of here? Afaik jQuery adds that in for us.
Hi, I updated, and please review again.

Concerning this and related problem, there is actually a minor issue. In order for the strategy to know num_machines, it should know num_classes. We have three options:

1. tell the strategy num_classes in constructors: this is inconvenient because we do not know the number of classes when constructing classifiers, until we get the labels
2. tell the strategy num_classes when training: this has the problem that several *true* multiclass machines do not use the strategy object to train, so the strategy object get no chance of knowing num_classes
3. tell the strategy num_classes as a parameter: this is what I'm currently using, and I'm not very satisfied with this either. But it seems this is the current best solution.
such callback is not necessary here - please just use the enum switch you do above (and drop the function above)
Theoretically, any SVM will work, or any binary kernel machine will work. But since I'm creating the machines myself (instead of using some machines from the user), I know I will not create any other kind of machine other than libsvm. If I want the flexibility here as to allow any kernel machine, I think I will face the same dilemma here as KernelFactory.
i got some erros after upgrade so will require some code changes so i would wait for RC version
Because it hasn't inheritors.
I usually mark all such classes as sealed (except framework or public api) until they have inheritors. Using this keyword gives some optimization and static analysis opportunity. When class marked as sealed, compiler and code analysis tools have additional knowledge to detect redundancies and errors at compile time.
Isn't the point of log levels that you don't have to check this in code?
In this case the logline comes after and is only doing a string concatenation. I believe this is safe.
Write, I'm getting into it...
Hmm, both good questions.

The same issue does apply to prev, in the sense that we may return some zero records at the beginning of a reverse scan that would not be seen if an aborted append had never happened.  It's probably slightly easier to fix prev by skipping invisible records at the beginning of a reverse scan, though we don't do that now for deleted records.  I've added a comment at the same place in the prev code.

The only documentation I am aware of for the existing behavior of fixed-length column stores in the docs for cursor->remove: "Removing a record in a fixed-length column-store is identical to setting the record's value to 0".  That doesn't say anything about where scans begin or end.  If we say that aborting an append is equivalent to removing the appended record, there is no real problem with the existing behavior.

To be clear, the underlying issue here is not caused by transactions, it is that we are already somewhat vague about where scans will begin and end in fixed-length column stores.  If we changed next and prev so that applications never see zeroed records at the beginning or end of a full scan, that would avoid visible side effects from aborted transactions.
I don't understand why you need "WSF_FILTER_HANDLER" here ...
WSF_HANDLER is enough

and if you set_next with a WSF_FILTER_HANDLER .. then the chain will continue.

with next: detachable WSF_HANDLER
you can even end the chain with a real WSF_HANDLER  (not being a filter) ...
Are you sure that the method "getPaginationBean()" exists?
Are you sure you want to override the parameter ``$filename``? This often makes code less readable.
@_@ .... I think I meant the 1st form at the beginning. Then I started to doubt if I understand what I meant means. Maybe what I thought it means means the 3rd form. Then I started to feel that the 3rd form means the same thing as the 1st one does. By comparison, I started to feel that the 3rd one express a milder meaning than the 1st one. The fact is that BackboneMVC tries to solve some other issues, so yes, no matter what I meant the first time, the 3rd way is the best now. 

I guess the Italics means a slighter attitude?
Добавтьб к методам контроллера префиксыы "Action" и убрать суфиксы "post"
Do you need to test 'traceToFile' (wont traceWriter be set every time traceToFile is set)
``liip`` does not seem an appropriate keyword to describe the bundle. But other keywords could be found (for instance ``monitoring``)
rad test--can you also test that !object.hasOwnProperty('setThing')?
You are using web.form.Form and form.Textbox. Can you just use form.Form and add `from web import form` to the top of the example?
Tell [them yourself](http://youtrack.jetbrains.com/dashboard/PY#tab=0). ;-)
Renamed back to tastypie_fields.ForeignKey.
Be verbose, write out fully.
Please update from main repository. I have updated form template there to allow extending it. You can now instead of manually copy-pasting the original template and changing things, rather extend that one and just inject necessary data where you want.
Use `'` around keys.
> Well the code would be like this if it was placed in tastypie-mongoengine. In MongoEngine I could not use this import so I had to copy/paste these query_terms:

But you should not be changing MongoEngine! Or, you should be changing it only in a way to externalize those local variables I wrote about!

And then use them in django-tastypie-mongoengine.

I hope I am clear enough?
Hm, shouldn't `.hide()` be before `.prependTo($('.posts'))`? So that we do not attach to DOM, and then hide, but first make it hidden, and then attach.
Import with relative import as you can see bellow.
This cannot be tuple?
I forgot to delete this method.
Agreed. The source started off nasty, and I really pounded on it further.
master is a moving target; would need a stable release to consider this.
This looks sensible.
i think this can be removed.
This is still used...
I've seen a pattern like this a lot in some other projects, but how would you feel about using .update() as follows?:

context.update({
    'form': form,
    ...
})
return context
Good call. Oops. Will remove and update.
On May 15, 2012 5:04 PM, "Lee" <
reply@reply.github.com>
wrote:

> > @@ -119,3 +122,13 @@
> >                                      (SentenceSampleStream.))
> >                                 true
> >                                 nil)))
> > +
> > +(defn ^DoccatModel train-document-categorization
> > +  "Returns a classification model based on a given training file"
> > +  ([in] (train-document-categorization "en" in))
> > +  ([lang in]
> > +     (println (str "train-document-categorization lang: " lang))
>
> This println should probably be taken out.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dakrone/clojure-opennlp/pull/12/files#r827211
>
I'd suggest removing IDE dependent comments.
Also, please align all `=` signs
You've got some methods, the static ones that should be attached to the measures (in GroupMeasuresByHierarchicalClusterin get_time, while in GroupMeasuresBySequentialClustering the following: time_distance, magnitude_distance, space_distance)
I googlized a little and most of the time, people says:
- if you update a critical data use the safe mode
- if "loosing" 1 write among many others if acceptable so don't use the safe mode 

https://groups.google.com/forum/?fromgroups#!topic/mongodb-user/znJjUPmag08

By default the driver doesn't use the safe mode and we should follow this no ?
but it **is** limiting
by "inherit" here you mean that options defined on _PropertyMapping_ override options set at entity level, which overrides options at global level? (when such an option is defined multiple times)

Or would it never happen for the same option to be defined in multiple places (among the three interfaces) ?
I think just use normal punctuation here and it will get made pretty by the markdown converter as appropriate.
Hrm, can we just call this cookbook "statsd" ? I don't envision a scenario where someone's going to be using this cookbook alongside another statsd cookbook in the same kitchen namespace.
yeah that's only for the web stuff
Do you need this comment? I'm not sure it adds much value.
I can see how zoom <1 might feel nicer. I think I would like if it bounced back to 1 when released, personally.

You decide on the maximum. I have no strong opinion.
?> at end?
Shouldn't flip to first after last, other views don't either.
This is where the up direction vector is calculated. f is camera frames parent. What does camera->GetOrientation give you the transformation matrix to take a vector from camera body space to camera space or the coordinate system transformation:)

What is the correct expression?
What do you think about removing all the links that don't have related content? It will be confusing to users since they will just think it's a dead link.
Note that `ndim=1` is the default if omitted
are these leading ``..`` for commenting out the code?

fine with me, yule walker will be moved and needs separate docs
remove 'user wants'. Same with other comments for externalProperties.
I've pushed such tests for the first type of test you mention, will get to the second type ASAP and let you know. Caught a bug in the Airy HLR scaling factor, so very, very good suggestion...
4.2.1

On May 1, 2012, at 9:47 PM, Mike Jarvis wrote:

>> @@ -5,6 +5,7 @@
>> #if defined(__GNUC__) && __GNUC__ >= 4 && (__GNUC__ >= 5 || __GNUC_MINOR__ >= 2)
>> // I think pragma GCC was introduced in gcc 4.2
>> #pragma GCC diagnostic ignored "-Wunused-variable"
>> +#pragma GCC diagnostic ignored "-Warray-bounds"
> 
> OK, I needed it for gcc 4.4.6.  What version are you using?  (Want to make sure I set a guard with the right value.)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/128/files#r759968


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
I don't believe Controllers will have a variable number of buttons or axes so you should replace this with TWO arrays. One for buttons and one for Axes so that it will have faster access time and you won't have to type cast later on.
curious -- why do you prefer single to double quotes here?  I tend to use single quotes for token-type things, like hash values, and double quotes for arbitrary strings.  This seems like a case for the latter.

Not quibbling as much as looking to learn.
.. and this one.
dont know... probably stay like this..
This seems to assume the input is always an array. It may be an array, a string, or another SourceNode instance.
Would `klass.ancestors.detect {|ancestor| registry[ancestor] && registry[ancestor][operator] }` work?
More commented code. Please delete or document criteria for reenablement.
I think we need to share this code w/ PlaylistModel. It's too delicate to duplicate. And in general code duplication should be avoided imho.
Doesn't this ReadFromDisk() fail for memory pool transactions?  I could've sworn memory pool txns weren't written to disk until they were included in a block...
I thought pep8 said to space methods by 1, not 2
I think it should be a list
Checking the demo site - http://jqueryui.com/demos/datepicker/localization.html - they include each JS file by itself instead of the combined version.

Each i18n file does try to set itself as the default language for the 

```javascript
$.datepicker.setDefaults($.datepicker.regional['language-code']);
```

That would mean that after including the combined version the default calendar would be in 'zh-TW' (Taiwanese Mandarin) and would need to be reset to the users default language (not hard be would need a note for future historians).
Nice move here, buddy. Well sharper than messing about with isProd in application code.
space between `<` and `Formula`
I would like a stable release. Sorry, no head downloads.
We're already in the context of an app, so we can simply call this 'submit'
why do we change this?
fixed, thanks, didn't remember that was part of the spec
This line should be removed, since it duplicates the default case.
Are you sure you want to waste all that memory just for an array of ones? I was trying to avoid doing this.
I think I did a similar change in the past and @ogrisel (or someone else?) didn't like it because all estimators are supposed to define y=None anyway.
Thanks for the hint! I changed that. Too bad all these functions are buried (and hidden) into our codebase :(
Dans l'absolu ce serait bien de mettre ces chemins en conf ou en constante, histoire d'éviter les duplications entre les différentes commandes et de pouvoir maintenir le truc en cas d'évolution du DoctrineFixturesBundle.
Fix tab vs space while you're there..
Those are records from [`ofs_userspace.hrl`](https://github.com/FlowForwarding/LINC-Switch/blob/master/apps/of_switch/include/ofs_userspace.hrl#L87) and [`of_protocol.hrl`](https://github.com/FlowForwarding/LINC-OF-Protocol/blob/master/include/of_protocol.hrl#L84). Please do not use plain tuples instead of records as this fact should be hidden and the implementation may change.
Use the same indents as the original
Doesn't look like it, no. I'll try and contact them to see if they add it.
Never since if added the magic '/', i will remove that.
On dev environnement and when executing androidannotations module tests (in
memory compilation)
Le 25 nov. 2012 12:15, "Pierre-Yves Ricau" <notifications@github.com> a
écrit :

> In
> AndroidAnnotations/androidannotations/src/main/java/org/androidannotations/generation/ApiCodeGenerator.java:
>
> > +		this.messager = messager;
> > +	}
> > +
> > +	public void writeApiClasses(Set<Class<?>> apiClassesToGenerate, Map<String, List<Element>> originatingElementsByGeneratedClassQualifiedName) {
> > +
> > +		for (Class<?> apiClassToGenerate : apiClassesToGenerate) {
> > +
> > +			String cannonicalApiClassName = apiClassToGenerate.getCanonicalName();
> > +
> > +			String apiClassFileName = cannonicalApiClassName.replace(".", "/") + ".java";
> > +
> > +			InputStream apiClassStream = getClass().getClassLoader().getResourceAsStream(apiClassFileName);
> > +			try {
> > +
> > +				if (apiClassStream == null) {
> > +					// It seems we are not executed into a jar. In this case, We
>
> When does that happen exactly?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/excilys/androidannotations/pull/410/files#r2216569>.
>
>
Also useful if you need to compare several runs... like master -> dataset, master -> dataset2, dataset1 -> dataset2, etc... 

You just need to run three times and compare as many as you like.
Ok, so it's equivalent to saying transaction->lookupSuperFile ?
It so, would be clearer I think if used that form.
would be prettier with additional space, &&<space>!precompileHeader
I did try that, but I am not sure how to not get microsecond and timezone info in the output.. will give it another try, though.
Does only integer matcher actually need a description?
other than that looks good thanks!
I disagree. It is tens lines and is not used anywhere else. Extracting it to a helper method would mostly just add a layer of indirection/obfuscation.
Honestly I just copied whatever the code in request was doing, just changed the names.
The issue isn't really with assert() but rather that there was an error in the code before.
Yep. good point
You should composer install before running the tests. This is intended.
OK,I review the code, HP does exist the lsdev ,but it total diff with ibm's lsdev ,IBM AIX Lsdev return the device list ,But HP don't
I try the Let the IBM do the Lsdev and HP do ioscan module
Shouldn't these be curly braces? { }
Removed passing of driver. Now there is only an empty constructor.
Please use an instance variable that correctly reflects the method. So this would be better:

    @bitmasks ||= {}
I just verified with egypt that this is indeed a bug (key being nil).  Please fix that too whenever you can.  Thanks.
What would you add then? Can't think of a much more detailed description here.
It's actually handled by Matlab's [regionprops](http://www.mathworks.com/help/toolbox/images/ref/regionprops.html) as a variation on the constructor (4th example under "Syntax"); you just pass in a labeled image and an intensity image.
It's called "warps", which was meant to be a library of warps (originally called "zoo", I think :-)
This description is insufficient.
Not in love with this. The spacing makes the comprehension harder to read, as it's less clear what the return value is.
Same as the above comment, we should test this 6 times, with only 1 url missing each time.
Yeah after writing that I thought about that too.  You're probably right that the branch may be more expensive than just redundant assignments.
Why not a switch?  Easier to add other cases that way.
Putting specular in the name is really misleading; it has nothing to do with specular highlights.  The specular map is an implementation detail to determine where the coastline is.
Hint: deselect it and select it again.
I think this and related code should be moved into `debug { ... }` sections (or maybe just removed if it isn't very useful anymore).
all source code uses 2011 maybe that's another issue to fix
Considering there are 22 other instances where I do an open() and I don't check file permissions, I'm leaning towards not caring. We'll be addressing this sort of thing when we rewrite the tooling.
This should be TYPE_TOP
I think window.setTimeout has a smaller surface area because it doesn't
involve pulling in isolates. Furthermore, as to which API is more
beautiful, I will remind you that you scolded me in another thread for a
useless use of a class ;)

Happy Hacking!
-jj
On Sep 26, 2012 7:06 PM, "Seth Ladd" <notifications@github.com> wrote:

> In notifications/quick/notifications_sample.dart:
>
> > +      scheduleNotification();
> > +    } else {
> > +      window.webkitNotifications.requestPermission(scheduleNotification);
> > +    }
> > +  }
> > +
> > +  /**
> > +   * Pause for a second, and then show a notification.
> > +   *
> > +   * The reason I'm pausing is because I'm pretending an event is happening
> > +   * later. Chrome won't let you show a notification that isn't the result of
> > +   * a user event unless you've requested permission ahead of time. Hence,
> > +   * I'm providing that I requested permission ahead of time.
> > +   */
> > +  bool scheduleNotification() {
> > +    window.setTimeout(showNotification, 1000);
>
> In this case, Timer is compiling down to window.setTimeout in the browser.
>
> The benefit is a simpler "surface area" to teach developers. Another
> benefit might be "new Timer()" looks better than "window.setTimeout". A
> developer that is new to web programming (aka our audience) doesn't know
> what window is, but could infer what "new Timer()" is.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dart-lang/dart-html5-samples/pull/20/files#r1702615>.
>
>
A little bit of me is worried that there are this many objects involved in this. There's a page, a user provider, the management filter. That and the `extractCredentials` method below has me wondering if this responsibility would fit better on one of the other participants.
"validate input with no type attribute, defaulting to text"?
Sorry, i do not want copyrighted code in this repository.
But you can add an `@author` tag if you really want.
recommend meta search over using send
А у тебя не будет обобщения над виджетом и стендалон-диалогом? Типа как `main` для отдельного `@uploadcare-uploader` Я бы вот туда положил.
I don't think the ranges use value semantics.
For function, function pointer, and delegate types, I've discovered that following factors should also be formatted...

- Function attributes - `@safe` `@truested` `@property` `pure` `nothrow` `ref`
- Variadic style of function parameter - `f(...)`, `f(T arg, ...)`
- Function linkage - e.g. `extern(C) void function()`

These are correctly treated in `SetFunctionAttributes`. Its implementation might be helpful for you.
I don't know why I didn't think to.  It's a shame you can't leave the parentheses off with the new lambda syntax.  It looks weird without the parentheses anyway.
there seems to be a missing indentation space on lines 423-425
Can you add few more lines?
`ParameterListMetaData` it shall be :)
You can't make this change without first implementing a prefs versioning and migration scheme, or verifying 100% that you're not going to break existing accounts.

And are you totally confident that this branch doesn't add value — such as the ability to wipe all global prefs, or all prefs for all accounts, or whatnot, independent of others?

We're eventually (and I mean "when I write the resetting code") going to move to a versioned `account.product.profile` structure, so having a known place to put stuff that doesn't get blown away seems like a good idea to me.
why not simply taking a model instead of checking for one?
no comma here
I think you should at least define constants, it would help readability a lot. I think it's ok to waste a few keystrokes, we know how to type :). (You can come up with shorter names, of course!).

```
const MOD_NEED_ACTIVE = 0;
const MOD_NEED_NOT_ACTIVE = 1;
const MOD_IRRELEVANT = 2;
```
yea, think rob mentioned this, but this guy needs earmuffs
The assets need to be available on the internet, the easiest way is to just push this thing up to heroku with this real simple Rack app for serving the assets.
Node
>
test1
test2
test3
>
Its true that the worse that can happen is just that the conversion is not performed, which maybe isn't a big deal.

Another way to handle it might be for OSL to `alloca` storage for the "worst case conversion" (which I think is float -> triple, 3x more memory). But I agree that it probably ends up doing more data-shuffling that necessary in those cases.

As for derivatives, it only happens when interpolating vertex attributes, but we definitely want the derivatives propagated in these cases (the user-data could represent new texture coordinates, or Pref, etc ...).
`checkboxes.each()`
trailing commas make me sad :(
Yeah, we won't be getting rid of new_axes() anytime soon because it is a public-facing API and would have to go through the deprecation process.  I am not exactly sure of the use-case for it, but there really isn't much of a rational for removing it, either, I think.
If this isn't testable, we should probably remove or add a comment here so others don't fight with this
Cool - so can we remove the conditional then and just use `$('html').scrollTop()` for all browsers?
For these code samples, it's better to use Github flavoured markdown's fenced code blocks, so you get highlighting:

    ```ruby
    CompaniesHouse.cache_args = { expires_in: 10.minutes }
    ```

results in:

```ruby
CompaniesHouse.cache_args = { expires_in: 10.minutes }
```
обе исправил
Does this need to be checked twice?
nit: remove "the".  ie. "auto update from khanacademy.org live datastore"
Please don't add comments like this, version control is enough to know you've added these changes. git blame works well ;-)

Could you please remove this?
There is a test for select boxes that when :name=>nil is passed it still puts the correct name in the html output.  In Rails 3.x this worked fine since it basically does   options[:name] ||= <code to calculate name>  so passing a name of nil gets ignored.  In rails edge it uses options.fetch(:name) { <code to calcualte name> } so the nil doesnt get ignored.  To make the test pass this code removes the :name option from the html_options if its nil
Can't this be done with a simple String.Split on whitespace?
Can hexColor include alpha?  Does FromHtml allow that?
Why not store this as an actual mysql datetime object?  I think it would be more useful there, especially if you need to add an index for range queries on this column later.
Basically, yes. Before these tests were "passing" because of except: pass, even though they should have been failing, or even exploding would have been better. Any exception that's tripped in the client that isn't a known one with http_status (which I'm expecting) won't be anything to handle here, other than failing the test and dumping it out.
(Just curious, you do not need to write it into the documentation.)
One empty line before call to  `super`.
That's probably better indeed.
I did not advertise this change in the commit summary but it should still be safe.
Vu du, un peu chelou... 'En creusant les' ?
Isn't this import broken since you removed netfields.fields?
this is completely insane. i am 100% opposed to this. why wouldn't you allow someone to use bootstrap, for example? or async? or... anything? using javascript in 2012 means being able to pick and choose decent, fast libraries.
try this test case:
1. specify a .md file path using src= argument, manually.  
2. immediatly, (without clearing the src= argument manually) upload a .md file.

you will notice that the uploaded file is not showing because src= argument is present. 

a simple solution is, when file upload is specified, then clear the src forced.
This is a lie right now; the IMAP implementation uses _accessFolderForMutation which gives it a mutex and ensures it gets cleaned up when the operation is done.  You are bypassing that by calling getFolderStorageForFolderId directly and then using it.  Since my move/delete patch landed, activesync has _accessFolderForMutation too, and you can just use that the same way.
(er, and the rationale for the comment is just that it's not immediately clear to me if this is to avoid the roundtrip or if this is something that we really should be doing elsewhere.)
Does it make sense to have difference exit codes for different errors?
I think "regionchanged" will be a better name for the event.
open does not take any parameters in android
Change this to

    if (x === require("Ti/UI").SIZE) {
What is the blob's `id`? Did you mean `type`?
`1 to width - sb.length`?
*encrypted.
For more information about <emphasis>URI template variables</emphasis>...
I agree with @kwallis 
we will eventually solve this problem with a config.xml editor, this is not the place for checking typos
Added. See https://github.com/blackberry-webworks/BB10-WebWorks-Framework/commit/423bd84ef3c582ab0def8514a26e6beb3492e53b
It needs to be done as the same commit, because previously we were using browser-require from the web platform-framework. But the web platform is now using almond.
There's a lot of places where utils.loadExtensionModule gets called to pull in events but they are all part of the index.js rather than client.js. I was just getting confused around loading this extension in the client webview vs the controller.
Why is this inside the method declaration and not above?
Can you drop this line ? The Mongo Admin must not contains SQL statements.