labofsahil/github-event-dataset-2012 huggingface dataset:
Good point. In commit f4b7993 I made `preRestart` more robust. It now catches `AkkaException`, but avoids to swallow other exceptions like `InterruptedException`.
I'd really like to do it differently, but I haven't found a way, yet.
or why not use ".distinct" since you're not going to use the set anyway
Это для обновление вид чекбокса. Все должно быть именно так.
Офигительное название функции!

Мог бы и лямбдой ее сделать прям в std::sort.
Лучше в ядре (или где-то еще) сделать метод, который бы регистрировал вкладку — соединял все нужные сигналы и все такое. И его дергать и отсюда, и из `TabOpenRequested`. А то завтра ты захочешь добавить еще один коннекшон, например, для смены имени вкладки, и куда-нибудь добавить забудешь.
should return 1 if n.zero?
you should use brackets even for a single line
The menu option for theme is hard coded in the base map class makes the items available for all map types. I suggest that map specific options is handled by the classes that implements the specific maps.
And getting rid of image_update to boot.  :(
Oikea idea, toteutus on... milenkiintoinen:
~~~clojure
(defn permutations [a-set]
  (if (empty? a-set)
    '(())
    (mapcat (fn [p] (cons (first a-set) p)) (permutations (rest a-set)))))
~~~
Eclipse compiler complained about something I cannot remember and this shut it up :-). For duplication, see my comment about RemoteExecutable
To pass in the SKIP_REMOTE_LOOKUP and SKIP_CACHE_LOAD flags efficiently when doing puts within an AtomicHashMap.  Same as before, except this works with the CacheAdapter as well.  The previous solution caused certain errors when a user also attempts to pass in a flag.
Same thing again, duplicate code, except for the extends. Can't you use some delegation instead of inheritance to avoid code duplication?
When will the tx be cleaned up from the tx table if the originator has crashed?
use "is".  Never use '==' for testing for None.
Please add <code>_locator</code> to all the locator's names.
I thought this was from `selenium.webdriver.support.wait` but this seems to work anyway.
I agree with @probins. I'd use `document.getElementById in the examples. `OpenLayers.Util.getElement` isn't part of the API, and I don't think it should be. OpenLayers isn't a DOM manipulation library, so users should rely on other lib like jQuery for DOM manipulation.
This template probably shouldn't have made the rc cut. Most of that is prototyped stuff, rather than the proper working version. This has been fixed in the implementation (which uses chosen).
Haha, no! :)
Commented out `console` calls should not be submitted for review.
@willdurand either a key in the extension's semantic configuration, like:

    knp_oauth:
        doctrine: false # true by default

or a compiler pass ?
Let's make this a bit less verbose.

starting

Same with stop below.
Is there a particular reason you want this in the base API?

I would argue that this should be implemented on the application side when they want to use this feature.
(They can implement it exactly as you did here very easily).
Maybe put this outside the autoCookie method so that it is not re-created on every call but instead shared amongst calls. Also, what's the dangling `_` doing? That has no purpose in javascript.
With this many params, I think it's easier to read if you do one per line.
Denne service-definisjonen dupliserer (mer eller mindre) den i service.pp. Vet ikke om den hører hjemme her eller der, men den bør vel hvertfall ikke være begge steder :)
Mongo::MongoClient.from_uri will correctly return either MongoReplicaSetClient or ReplSetConnection depending upon the format of the URI.  If it has multiple hosts, it returns the former.  MongoMapper parses the uri though and the parsing will fail if first not extracting comma separated hosts.  
The workaround is to set MongoMapper.connection oneself (the MongoMapper.database also has to be set).  Perhaps consider using Mongo::MongoClient.from_uri instead.
Added system dependency for compatibility with Boost.Thread 3
Remove direct loading of core.js, let ::core() method do it
Can you remove the BC constants? Once that's fixed I press the button.
Vi sugestão de colocar no `config/application.rb` e depois usar com `Cec2012ptServer::Application::CURRENCY` mas não me agrada muito. Acho que por agora fica assim.
Empty file. Should be removed no?
Why would you need this? Console.log should not be in prod code.
I guess it should be
```
                            ->arrayNode('i18n')
                                ->children()
                                    ->scalarNode('fallback')->isRequired()->end()
                                ->end()
                            ->end()
```
I've an issue on sf 2.1 with that
You always have been a genius. :-)
Why additional panel? You could just in original panel file extend another panel class and register it. To reduce duplication. So, use inheritance.
I will also add `PATCH` for updating fields.
Yes, this is one other way to see that code is too much coupled together. That parts which should be separate are not anymore.

As I described, define a new signal (post_published, for example), and then connect to it in frontend app.

This should not be in settings because it is not a configuration option, but a constant string.
Or, if we will have dictionary, you can easier remove it.
You have to make them in the same branch and push to the repository you are doing pull request from.
We have those? Where?
here and above...why were there 5 dots?
Yep, I missed that while renaming "Moq" to "NSubstitute". Actually, I was not creating something new here, but simply taken AutoFixtgure.AutoMoq as base and just adapt code to NSubstitute.
Aquí tuve algunos conflictos con dependencias detalles de mi OS la verdad lo corri sobre trisquel y tuve conflictos con la librería `event.h` de momento son detalles que estoy revisando ademas de otras cosas que ando configurando en mi OS(hace poco formatie).

Lo que hice fue crear el archivo source/allow con un simple touch y cancelar la ejecución de la función`make_daemon` al comentar esta linea para que el guión me pudiera seguir ejecutando el testing.
Maybe, "CreditCard#type has been renamed to #brand. The alias for #type will be removed in a future release of ActiveMerchant."
No, but it can change the context within which the tiddlers is rendered and thus how links within the tiddlers resolve.

Also, we'd prefer to publish tiddler URI is a canonical location so that bookmarks etc all make some kind of sense in terms of ownership etc. So we at least want the tiddler to end up "in" its space. It remains an open question whether that should be the friendly (from / ) or bags location. Originally the bags location was deemed best because that's the "works with any tiddlyweb" location, whereas the friendly thing is a tiddlyspace add on.
Ohhh. Sry, needs to sleep:)
Not totally sure, but I think the final consensus was that we shouldn't use :locals for passing from controllers to views when possible, and should use :locals for passing params to partials when possible [1]. It's a minor issue though.

https://lists.fedorahosted.org/pipermail/aeolus-devel/2012-August/thread.html#12054
I don't know what the best practice is.  I thought the best practice was to reference attributes directly no matter where.
The only time I feel it is necessary to create variables is if some type of calculations need to be performed in the recipe (model).
Zelfde als boven
use .fetch
Also, the contents of the array are not very clear.
Correct, of course I mean to always bind the unbound function. A bit surprised I didn't see this failing, I thought I saw a case where we have to play back a  "response" message because sendRequest took quite long to reply. But it would have thrown, so I guess that wasn't the case.
I don't. Fixed.
@jeffkaufman : I found a good explanation of this at http://wiki.nginx.org/HeadersManagement, which explains some other nice stuff as well regarding header manipulation.
Should this be a well-commented py file or a pdf with code? I uploaded 
examples on the emplike_reg branch.  I will add an example to this 
branch as well in a sec.

On 07/25/2012 09:40 AM, Skipper Seabold wrote:
>> +
>> +        Difference between log-likelihood of corr0 and a pre-specified
>> +        value.
>> +
>> +        Not intended for end user.
>> +
>> +        """
>> +        return self.hy_test_corr(corr0, nuis0=None, mu1_min=self.mu1_lb,
>> +                       mu1_max=self.mu1_ub, mu2_min=self.mu2_lb,
>> +                       mu2_max=self.mu2_ub,
>> +                       var1_min=self.var1_lb, var1_max=self.var1_ub,
>> +                       var2_min=self.var2_lb, var2_max=self.var2_ub,
>> +                       print_weights=0)[1] - self.r0
>> +
>> +
>> +class DescStat(OptFuncts):
> A usage / brief tutorial style example would be greatly helpful. Ie., a standalone file. I don't have Owen's book because I can't afford it right now ;), and I'm not really sure what I'm supposed to do with this. Is the intention of this class just to be a univariate / multivariate location and scale estimator based on optimizing the empirical likelihood? If that's the case, I might suggest some changes. Otherwise I'm going to keep posting some comments on just the code.
>
> Are there any papers that cover this outside of the book? I don't need all the details.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/statsmodels/statsmodels/pull/326/files#r1234792
do the ``u = x[K:] have a statistical interpretation?

we could return them in the retval dictionary if they give any useful information
Oh ok. I thought you had gone over this stuff. I've worked through a text book and should have some fairly significant changes over the break. Not a major rework of anything that's done but built out a bit more.
Much nicer than zero  :)
``return min($count, $this->getMaxResult())`` ?
默认值 -> 默认名称
Isn't the preferred "Pythonic" way to view docs:
```
help(galsim.Sersic)
```
I think it's okay for magic numbers to not be in all caps, since they are explicitly in the sbp namespace.

On Jun 4, 2012, at 9:46 PM, Mike Jarvis wrote:

>> @@ -13,6 +13,15 @@
>> 
>> namespace galsim {
>> 
>> +    namespace sbp {
>> +
>> +        // Magic numbers:
>> +        
>> +        /// FT must be at least this much larger than input
>> +        const double oversample_x = 4.;
>> +
> 
> We can go back to all caps for the magic numbers if people prefer that convention.  I prefer reserving all caps for preprocessor macros and constants (to highlight their even more special status).  
> 
> The way I did it, all the magic numbers in the code are preceded by `sbp::`, so that tells you right away where it is set.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/176/files#r926767


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
And this final.
rsync --daemon reloads it's configuration file automatically when it changes. So this isn't really necessary.
vire le `in fact`
All fixed, and much more
I disagree - as in some cases we don't have a test for things, and therefore having random things fail because an originally expected output suddenly change is a good thing. It's perhaps a bit annoying, but I rather have tests fail than not seeing something break. The main issue here is that the code that did change this output shouldn't have been committed. If 1 was indeed %d, we would never even have noticed it.
This is confusing---you can either get one listening room w/ no params, or pass in an id and get a listening room---what's the difference? :)
rename it to subjects.json , else it appears as if  it is some editor back up file
Let's remove this profile case for now.
What I have in mind for that is something else, i want to add some preferences to firefox to let him search a specified path for the webapps/  folder and the offline cache folder - similar to the way it search Gaia on /data/local/ first.
Are these redundant with the above?
Why did you replace get_security_level() with security_level?
why enabling some of them by default but not others ? This seems weird to me.
I honestly don't like the static nature of this plugin, we already have several files where the boards are defined, this will be one more. I would prefer this information would be defined on a common 'ground' 'flight' location such as 'shared/include' and picked up at compile time or even better at runtime (would not require recompilation after new board added)
Actually from discussing this on IRC I think you should be following the model of Freedom.  This code should be moved to
flight/Modules/Sensors/sensors.c
and then for the fusion algorithm run
flight/Attitude/revo
> ?? looking at your pullreq. That would mean that the results of the query above it do not fill all rows.

What do you mean? update wouldn't fill rows?
Do you expect these two lines to throw an exception?
True, they shouldn't but stranger things have happened;-) @meineerde pulled in the updated jQuery libraries too from another PR.
Это будет работать медленнее. Но если у тебя и так будет быстрее - не вопрос.
Страшно представить, что будет, если добавить ещё пару-тройку переключателей.
со строчной
If we had used a hash, this would look like:

```ruby
@points[point.label] << point
```
please wrap this line and line 320, probably before the message
and me. Boost can be a bitch to build on some platforms. Header-only is one thing, but libs are a no-go.
Exactly: it has an effect only if we allow the user to set arbitrary priors. What's probably a bit confusing is that I haven't implemented that option yet (I was actually waiting for your input), but I was thinking of processing `class_prior` in the same way that the `weights` are processed right now: one could either set `class_prior` to

1. `'default'` or `None`; in both cases, the classifier then reverts to the "regular" priors (e.g. `K_k / K` for the kNN classifier).
2. `'flat'`; in which case each class is considered equiprobable.
3. A list or `np.array`, of the same length as the number of classes, and whose sum is 1.0.

Otherwise we raise a `ValueError`.

Does this make sense?
And maybe even be named `has_converged` instead.
@GaelVaroquaux This line causes some exactly 0/1 case to fail, make nmi gives values a little bit larger than 1 or very smal negative values. I think ```log(a) - log(b)```  may be more accurate than ```log(a/b)``` as division may lose precision and a quick test by others is here: 
ref: http://www.jonkagstrom.com/floating-point-precision-and-sum-of-logs

However, I think I need more time to have a deeper look at those issues :)
Done in 40745ce.
Interesting stuff. The only suggestion I would have would be to prefix DEF_VALUE and CONST_DEF with something like TIEXCEPTIONSTRINGS_ to avoid a conflict.
Consistency is king. I'll match it up with UrlBasedViewResolver's.
exact merci
Amazon has a different endpoint for each region. If no endpoint is specified the code discovers the right one using the region name.
Or just add the old error message to the new error message, that should do the trick.
Looks a bit odd, isn't there a isAlpha() or something?
Seems like an awfully long timeout for a command-line tool to connect to dali. I would have thought if you run daliadmin and dali is not up you'd like to be told about it immediately.
please dont remove crap IDE comments in the files
Here, same point as above regarding the report_list_no_data.html
This is still wrong. Try to get it fixed
Could you change this to a different value for this section? This test is supposed to be verifying that an overridden value is used correctly.

Also your `=>` is indented too far.
I'm fine with that !
As above regarding title case.
> It might be relevant to decouple this from this pull request?

Also, I'm totally fine to have this unification as a followup.
I guess the default value isn't correct
Btw, this value is detected and set at run-time with the pci_find_gfx function.
I would prefer to completly remove "scala-ldap-version" property, as it is no more used now.
remove the commented out code.
Typo. IDE format and git conversion issue.
very long line.
I don't think the .spec should be here, it'd quickly get out of sync with OBS and become confusing.
okay I'm most definitely using it wrong
Not sure where DPORT is defined at? Could be missing it though...
What is the purpose of the extension? Why is one better than another? This should be addressed in the description, otherwise I can't figure out how to use this module.
A local variable can be used here for peer, no prob, I'll update it by myself when merging
I'm clearly half asleep... dividing a float array by anything is obviously float division :)
What do you think about writing down the most important conventions of skimage in a `CONVENTIONS.txt`?

This makes it easier to contribute and it takes you less time to check PR.
I'm glad that this method isn't used :)

Let's remove all the methods you added to the header and let's just use the session delegate as a property.
Just a small style issue that doesn't need fixing, but using a backslash for a line continuation isn't required within brackets, and Python's style guide discourages the use of back slashes for line continuation.
Not sure that we need this example given it is almost the implementation of `_eliminateHole` below.
In order to be robust with uniforms name conflicts between a primitive like a polygon and a material, shouldn't we do more than this `combine`?  Shouldn't we rename uniforms (from the GLSL perspective, not the JavaScript perspective)?
I wrote it like that originally, but jshint reported:
> Do not use 'new' for side effects
Then when I assigned the result of the new to something, now that something was unused.  So what you see here is the simplest thing that didn't give warnings.  Maybe I should just ignore the warnings?
Same comment above. This doesn't have to do with your changes, but this tests that `this._width` and `this._height` are even not a power of two.
これ、必要かどうかよくわからなかったです。
Simulator version should be a configurable value.
This might be an XSS vulnerability.
Node's querystring module: http://nodejs.org/docs/latest/api/querystring.html
Returning a pointer to a local static variable seems a bit dirty to me. I know it shouldn't cause any problems, but it still looks somewhat odd. Other than that, well done token method, would have done it the same way.
This seems wrong to me; it should be `int.alignof` on 32-bit systems.
Let me make sure I understand - are we discussing the entire algorithm, or just this particular function? In the latter case, no problem because I thought it could be improved significantly.
ah, I see. That makes sense.
nothing wrong with using reified generics, but since the value type is Dynamic anyway, I personally would've just used "Map"
Deferred APIの$.whenや.pipeは、同期や非同期のサブクエリを連鎖させたい場合に、大変な威力を発揮します。
I don't think you need to decrypt, just ask the db to check for `Passwords.ENCRYPTED_USERNAME + " = ? AND "` + with record.encryptedUsername as an argument.  But you've been working on this, and that's the obvious solution, so please document why it doesn't work.
Split this line to align the conditionals vertically.
А якщо так
’’’Then the response meta "description" should contain "Description your site.Sony MicroVault Mach USB 3.0 flash drive.iPad 3 rumored to come this March with quad-core chip and 4G LTE.Toshiba unveils 13.3-inch AT330 Android ICS 4.0 tablet.’’’

Подивись ще на використання масивів. думаю можна взагалі писати щось назразок
```
Then the response meta should contain:
    description: blabla
    keywords: parara
```
Need to handle dates and boolean too.
I tend to use `foo.must_be :blank?` for stuff like this. MiniTest::Spec will take any symbol with a `?` at the end and call that method on it and use the symbol for pretty error messages. So good things like `:valid?`, `:present?`, etc are all possible.
Maybe I misunderstand, but if this is supposed to be a port of bootstrap for use in a SASS environment, then shouldn't it simply follow what the framework already does in LESS? I'm a huge fan of following specifications and normalizing mixins and whatnot to match them, but this seems really counter intuitive. I've already had at least one instance of a designer running afoul by following other Twitter Bootstrap resources.

I understand the conundrum given that this may conflict with Compass, but not everyone using Twitter Bootstrap uses Compass (in fact the company I work for has at least one project that uses your gem as an alternative to Compass).

+1 for following what Bootstrap does, given that this is, well, supposed to be "bootstrap-sass is bootstrap for Sass, ready to roll" packaged nicely in a gem.
**/!\  BAD TEST** : tout peut être `false` sauf une seule valeur `true` (i.e. un seul méronyme rencontré), le test passera !!
No, it just occurs to me that it could happen elsewhere. If this is literally the only case, then that's awesome. Just trying to be thorough
Started thread [Let's schedule WinAPI ASCII functions for deprecation!](http://forum.dlang.org/thread/jpgks6$267t$1@digitalmars.com) about it.
Hello, Gary
how do you feel about such options?
````java
RootBeanDefinition expressionDef = new RootBeanDefinition(hasUrlAttribute ? LiteralExpression.class : ExpressionFactoryBean.class);
expressionDef.getConstructorArgumentValues().addGenericArgumentValue(hasUrlAttribute ? urlAttribute : urlExpressionAttribute);
````
Or it will be not readable and not so clear or does not match some conventions?
Good point - we could document that "put"ing an arbitrary item is allowed, but it becomes one of the pool and may be expunged later if the pool size is reduced, or the "put" itself causes the pool size to be exceeded.
Just pushed most of the changes form the latest round, There are still some thing i need to do (eliminate recursion and add a few tests), but overall "KABOOM" comment is addressed and test is modified to demonstrated that exact scenario.
I also need to add documentation why I need to maintain a local session cache (that is yet another improvement we would need to do in SPR Jms Template code)
The same about "@ since" and refactor code, please: whitespace after comma
small nit-pick 

     messageConsumer.receive(timeout)

or this.receiveTimeout (but since you just captured this.receiveTimeout int timeout ...)



Also, couldn't you restructure the while below to include this receive?

````
while (notTheOneIWant) {

    receive(timeLeft)

    notTheOneIWant = !isTheOneIWant || null

}
````


Also, you always complain when I occasionally put a continue or break in a while statement; aren't returns in a while() {} just as "bad" as break?
Nope, it should have been removed.
inferior => fewer

Inferior is used when you specify a significance over something else. Fewer/Less are used when amounts are compared. Fewer when the things you count are countable, Less when they're uncountable.
if you use single `use` key word notation, at least be consistent
add types to the param tags, even if it just says "Object" or "Varies", e.g.

```
@param {Type} type A type...
```
Change this to say "node": ">=0.8"

Our code won't exactly run on node 0.1 :).
不必要な import 文が無いか今一度確認していただけますでしょうか。(他のソースファイルも)
use pep008 styling on code
I guess this can just be @tempquueprefix instead of quoting it in a string? small optmization and readability
Where ``block_name`` will be used ?
kml says the href  location can either be on a local file system or a remote web server. I'll remove the check for http. Not sure the implications with security. Does this open the door for a kml file to have an href that points to something malicious? How can we check on that?
No limit you say? :suspect: Okay...for now.
I think `settings.STATICFILES_DIRS[0]` should be replaced with `root`
`you would do away with the defoverridable line` should probably be `you would go away with the defoveridable line`.

In any case, the nice thing about `defoverridable` is that it allows you to choose when to override and when to keep the definition. For instance, you could add inside `InheritMod` the following after your :z clause:

      def test(_) do
        super
      end

And it will call the parent definitions. Not sure if this is worthy adding or not. :)
I don't think this will get lofted automatgically, so it should probably get lofted to a top-level constant.
Further description is needed here , i.e. "A Blob can represent any opaque data or input stream"
Can your configuration be refactored like this ? I don't get the need of the id name is the yaml section.

```
sonata_notification: 
    backend: sonata.notification.backend.rabbitmq

    default_queue: mail
    queues: 
       - { queue: transcoder, routing_key: test.notification.transcoding_start }
       - { queue: transcoder, routing_key: test.notification.transcoding_send }
       - { queue: mail, routing_key: test.notification.mail }

    backends: 
      rabbitmq: 
          exchange:     router
          connection:
              host:     %rabbitmq_host%
              user:     %rabbitmq_user%
              pass:     %rabbitmq_pass%
              port:     %rabbitmq_port%
              vhost:    %rabbitmq_vhost%

```
(question) I see that the LineOfBestFit sequence calls getNewColor, but I don't see how other dataSets get their colors. Is it required to be set in the input json?
this should be handler403
Do we even need to do this slice?
typo in callack
VERY OFF TOPIC - Is there a way to get cppLint to catch things like this?
type ?