labofsahil/github-event-dataset-2012 huggingface dataset:
I think it would be nice to standardize on using `Object.toStringHelper()`. We have a a range of toString formats with square brackets, currly brackets, etc. But don't feel strongly about changing it here.
vire le 'them' a la fin
package should be on separate line?
This is the position within the parameter set - instances is in the instances parameter set and Runtime is in the Runtime parameter set (along with version).  RoleName is in both parameter sets at position 0.

So, for example, you can use:

Set-AzureServiceProjectRole WebRole1 2  -or -
Set-AzureServiceProjectRole WebRole1 node 0.6.20
Why is this needed? I can't see it being used anywhere.
Heck, wonder how this did not break when I tested, hmmmmm. Fixing
I think someone else is refactoring it out of Application and making another controller for it.
Cred ca trebuie stearsa.
trying with that.. you make me think about it and just did a quick debug with irb and it seems to work :-/ mmm
I always figured setting up the repository was done by an admin person... There is scm status in the admin menu, but a 404 error is the wrong message when scm is not configured correctly.
Thinking about it: the closed and done together should never amount to more than 100%, so the overflow might even not be necessary. If updating jQuery UI moots this concern anyway though, we should update them.
Looks like a lot of corners were taken, when I wrote this code...
И как может в сортировке FormatterException возникнуть?
Здесь ты перезаворачиваешь FormatterException из try
i used `and` instead of `&&` so i don't have to add parentheses around method parameters (it's cleaner without them)
The issue is when when an asset is not precompiled. I should have added the comment to the digest_for_stylesheet (since images should all be compiled by default). Using the example from my other comment, say there is a file xproject/stylesheets/style.css. And this file has not been precompiled. If the assets settings are: 

config.assets.compile = true
config.assets.digest = true

Now an asset reference is made like this:
theme_stylesheet_link_tag 'style'

The code above will fail to find xproject/stylesheets/style.css because it isn't precompiled. It will then fall back to style.css even though xproject/stylesheets/style.css actually exists.

With the settings above, I'd expect the code to figure out the asset exists, then compile it and return the digest. By using asset_path or asset_paths.digest_for instead of Rails.application.config.assets.digests, this is what will happen.
Thats actually true. I just focused on improving the sort sort function.
I will fix it
Typo (summary/sumary)
I think the test would be more worthy if these 5 properties (below this one) used `is_element_visible` rather than `present`. It's quite important that they're actually visible to the user so that they can click, read and interact with them.
面倒なWARファイル配置などの実施もなく

は

    WARファイルの面倒なデプロイをする必要もなく

ではどうでしょう
Here instead of saying __RADIO OFF, which was when we were trying to detect our network state with the Airplane Mode Hack, maybe we should use use __RESET .. ie:
   Log.d(TAG, "__RESET - tickling airplane mode")
   Log.d(TAG, "__RESET - done tickling airplane mode, toggling APN")
   Log.d(TAG, "__RESET - done toggling APN")
`@@logger || setup!` will fail if @@logger is not set already. Please don't use class variables. `@logger` would have the same effect, making the logger effectively global.

Try in IRB:

```
[4] pry(main)> @@logger
NameError: uninitialized class variable @@logger in Object
from (pry):4:in `<main>'
```
I'm not sure what you mean here?  The () is to order the multiplication by norm_ones after the normalization.
I wouldn't worry to much about the overhead of either calling ``_decision_function`` or ``atleast2d_or_csr``.
I would add that `loo_values_` contains the residuals by default or values of score/loss function if provided.
@dwf cool, I had somehow overlooked of that proof :).

On Wed, Oct 3, 2012 at 5:54 PM, David Warde-Farley <notifications@github.com
> wrote:

> In sklearn/rbm.py:
>
> > +        v_pos: array-like, shape (n_samples, n_visibles)
> > +
> > +        Returns
> > +        -------
> > +        pseudo_likelihood: array-like, shape (n_samples,), optional
> > +            Pseudo Likelihood estimate for this batch.
> > +
> > +        References
> > +        ----------
> > +        [1] Tieleman, T. Training Restricted Boltzmann Machines using
> > +            Approximations to the Likelihood Gradient. International Conference
> > +            on Machine Learning (ICML) 2008
> > +        """
> > +        h_pos = self.mean_h(v_pos)
> > +        v_neg = self.sample_v(self.h_samples)
> > +        h_neg = self.mean_h(v_neg)
>
> @amueller <https://github.com/amueller> Yoshua's book doesn't mention
> this AFAIK but this technique (replacing a sample with its expectation in a
> formula) is known as "Rao-Blackwellization", and it's trivial to
> demonstrate that it's still an unbiased estimator but with lower variance
> than the original sample-based estimate. See, for example, this tutorial<http://www.cs.ubc.ca/%7Enando/papers/ita2010.pdf>
> .
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1200/files#r1756078>.
>
>
`RuntimeError` will be deprecated see http://docs.python.org/2/library/exceptions.html#exceptions.RuntimeError
On 10/09/2012 07:23, serge-sans-paille wrote:
>
> In pythran/tables.py:
>
> >                  "pop": MethodIntr(),
> > +            	"remove" :MethodIntr(),
> > +		"update" : MethodIntr([ lambda self, node: [ self.combine(node.args[0], node_args_k, unary_op=lambda f: cxxtypes.SetType(cxxtypes.ContentType(f)), register=True) for node_args_k in node.args[    1:] ] ]),
>
> This is not the combiner we talked about. Use the simpler
>
> [ self.combine(node.args[0], node_args_k, register=True) for
> node_args_k in node.args[ 1:] ] ]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serge-sans-paille/pythran/pull/12/files#r1563255>.
>
Bet it won't work...
Could you be consistent with the spaces around the .-operator? I'm not sure what the Symfony CS rules are (spaces or no spaces) ...
I can add isRequired as well.
probably worth documenting this if statement - not clear to me glancing at it for first time why we do this
könnts raushauen ;-) wird eh nichts drin gemacht
Alternatively, you could also do `from django_ses import settings`.
`inp` and `outp` are marked `pure` (the Na part)
Type::tvoidptr
There's more work to do in transform.  When you toString (or JSON.stringify) a regexp, it adds the '/' and any arguments.  Example:

1) var re = new RegExp('.*[a-z].*$', 'gi')
2) var reString  = re.toString();  (converts re into '/.*[a-z].*$/gi' - notice the slashes and extension)
3) var re1 = new RegExp(reString) - This creates a new regexp that looks like this: //.*[a-z].*$/gi/ 

Our transform will need to parse the RE string into it's components and pass them as arg1 and arg2 to new RegExp
Disabled copy constructor and assignment operator.
we use index in the NativeListViewObject::scrollToIndex function to initialize QVariant with string (not integer). don't see the reason to add convert it to int here
Votre serveur ** effectue une requête** à l'adresse suivante avec le code
How can that even happen?
Hmm, why's that?
yes I know they removed it. I'm very glad you didn't use that specifically :)  but was missing the rest of your reasons. Nice post, thanks!
can you remove the TODO and the javadoc?
Right. Reflection is probably not needed here.
Yes by the time writer is released it should not be able to write.
I commented to that effect somewhere, but my base class seemed sufficiently different from the one in roxierowbuff to warrant separation.
Where CThorExpandginRowArray has no spilling/firstRow/commit functionality and CThorSpillableRowArray derives from it.
A similar hierarchy could exist in roxierowbuff, which would make things more common.
Can revisit later I think.
This is not strictly necessary
Missing 1.3.1, 1.3.2 and 1.4.1 here (if we're planning on including all "full" releases).
I'm not using any MIDDLEWARE_FILTERS variable in MyTardis 3.0 and I believe it worked fine, at least with the previous version of the atom app.

I have a feeling MIDDLEWARE_FILTERS was implemented in RMIT's version of *core* MyTardis pre-2.5 to get filters working with their ingestion method. I also believe it's Joanna who implemented it.

Should be redundant now.
Was just going off what Rohit posted in Chat, but I can add that.
So finally, what should i do ? Revert to the previous way (the "1") ? Even after my last comment ?
IMO  there is no place for _ in URL. As this page has "zf2-tutorial" in URL - not "zf2_tutorial" ;) It is also the case for many webpages:

https://addons.mozilla.org/en-US/firefox/addon/greasemonkey/
http://msdn.microsoft.com/en-us/library/ms533052(v=vs.85).aspx
http://www.skype.com/intl/en-us/get-skype/
http://www.lego.com/en-us/default.aspx
Sure, if it's something to export the name should be different, maybe something like `readLocalURI`? But I didn't understand in which module we should add this function.
Also, our module seems totally different from the node one. In addition I also put there the `DataURL` object that I created for the clipboard API images implementation: if you think we shouldn't modify that module, please let me know if and where I have to move this object.
Why are you changing the order here? Precedence is important -- $mergeChildren and $captureTo can both evaluate true, but in the case of $captureTo, we do _not_ want to merge the children into the parent array.
How users are supposed to pass the certificate in node.js? Is it by path?
Use id = "detail" instead
Sounds reasonable -- let's merge!
> I got used to "var" because it makes code shorter and easier to refactor (when renaming classes or introducing interfaces).

But harder to find all class usages in this code.
Same here as with `has-a-living-author?`.
`object_setInstanceVariable` doesn't support ARC
Need to deal with idiots -- there's no else case where both Expect and Python are skipped.

Probably want to check this at the top.
Yes, the exploit can be modified to pop a shell, but most of the metasploit functions fail due to limited write permissions on the target directories (php has low privs).

This exploit abuses the DB user's file write priviledges to drop the php file on the web directory, via SQL Injection.

After extracting the password, the user can login to the Plesk control panel using it and have easy access to the target host.

Let me know if you still require it to pop a shell ?
Oka will be checking and reviewing also ASAP :) Hope to update soon!
Maybe we should test for the actual number of constant values here (rows*tx + cols*ty - tx*ty).
Actually, we should probably go over all the `PolylineCollection` examples.
More allocations...but it's so clean.  I know.
Thankfully that's just a CHANGES typo and not in the code :)  I'll fix it.
These properties must be top, left, right, bottom. Maybe also specify if they are optional.
No point in the declaration until the initial error check passes since we just return immediately.
This would probably be better:

    readOnlyProps.split(",").forEach(generateGetter);
Do we need the () at the end of this line?
`TiC.PROPERTY_FORWARD` should be used here
We could use a more detailed comment regarding why we are not setting the model listener here.  Since this is a complex bug, we should probably put some comments in relevant places like getView() to describe the situation.
This var statement (and the one in the neighboring for loop) should be grouped with retval a little ways up
Same as above. This way I return a clean error message instead of a confusing `TypeError: Cannot read property 'version' of undefined`
These two iterators are the exact same, so we should probably just create a local function outside of the `toggleMode` function that does this work
Style: whitespace here as well.
Good catch, you are right.
Because if you don't do that, then `drop` and `popFrontN` don't work. They do nothing. It becomes like if an array assignment assigned only the pointer and not the length.
So, you mean do

    return 1 + ((u >= 0xD800) & (u <= 0xDBFF));

instead of

    return 1 + (u >= 0xD800 && u <= 0xDBFF);

It appears to be considerably worse to use `&` (at least with the strings that I tested). Without optimizations, it takes around 30% longer, and with them it's 40 - 50% longer. It's probably because `&&` shortcuts when the first half is `false`. It's possible that it would be faster if most of the characters were in that range, but by far the most common case is that they're not.
I think `is(S == class)` is better than `is(S : Object) && !is(S == struct)`.
I feel environment variables are easier, no API for people to get familiar with, they just need to know which environment variable to use. It also allows us to make changes to toolchain, if needed.

The biggest downside is that something else may be setting the environment variables as well, outside of EasyBuild, I guess.

Is there currently an easy way to access stuff like `CC`, `MPICC` and things like `CFLAGS` and `MPI_LIB_DIR` via `toolchain`?
Maybe I don't remember correctly, but isn't this restriction (need to be named raiseEvent) already removed?
you're kidding, aren't you?
OoooooOOOOOOoooooooo
Please use the github web interface, the email gateway does a horrible job of formatting your emails :)
yep, should definitely be on upsert! I'll change that.
Done in 9dcac2c7ec39625d3e9ac869e27960d44e3b1de9
Each data row contains a name, a value, and either a
It's not used anywhere else in the project, so I don't think it makes sense to do it in this particular place. I don't think it affects readability much, so it doesn't really matter.
I think you can refactor these lines in a function called `recreateContainer` because it's job is very commonly used
It's in the explicit_delegator, not needed in this file.
or if you want to use it, put it in a shared section, not in the Composer-specific one
Can you see what happens if you remove the line immediately before this?
```$j( this ).dialog( "close" );```

I didn't want to use confirm() because I wanted to be able to be able to put class-specific information in the confirmation. You'll notice that the class code is in the title of the confirmation box.  In theory, we could also put other information in the message itself (like the full class title and the full set of times it is offered). If we can easily get the jQuery UI widget to work, I'd prefer to do that. Otherwise, yes, we can fall back to using confirm() and we'll still get the same functionality.
also a syntax error.
This does not respect CS, please use 4 spaces, can you also update the related unit test
Лол. А те, у кого лаура уже была запущена, у тех все по пизде пойдет.
А, оук.
Values contains the set of data it could be a subset of the entire dataset, it could be the entire dataset... depends on what you specified in delivery_format
Checking for nil is a good idea prior to doing the string split. A nil username will cause an uncaught exception.  I will update this

I like the idea of providing more help about the format of the user name, However i think that we are better off doing so later in the process (say [Line 113](https://github.com/opscode/knife-windows/pull/11/files#L0R113)) . A nil or empty username are valid if you are using kerberos ( or at some point soon Windows SSPI) for authentication, so we really should only provide additional sanity checks if you are not using kerberos authentication.

Thoughts?
Is there a unique reason why we need this here and in no other tests or js files?
I don't think this template exists yet, right?  Or at least not by this name.  

Shouldn't this take in a "patient" ?
private isMember() is no longer used; remove.
I am just making it identical to OALHS
Well that actually should work since it is exactly how it was when I was caching temp queues locally in JOG. So I'll give it a shot
This particular effort is to advise __just__ the handle request part of an AbstractReplyProducingMessageHandler, and __not__ the downstream flow. For example retry, circuit breaker, post processing (e.g. deleting a file after an ftp send). The last one will advise the whole handler (because it's not an ARPMH). It's not really intended for transactions but, I suppose, you could add a tx advice to a service method this way.

We are not certain that we want to do INT-2481. But, I agree, we have to be careful about the configuration. I haven't really thought about that yet; there are a number of possibilities, including:

&lt;int:local-advice-chain/&gt; subelement - local implying it only advises the immediate object, not the entire flow from this point
&lt;int:advice-chain advise-downstream-flow="false"/&gt;

etc...
Gary, np, I'll do it if you say.
I enjoy working with you!
But then we should make cross-commits again.
Here I'm I a little unsure, that's why I tell it as a question.

And further I'm not sure in the description of 'request-payload-type'...
It configures `JsonAwareInboundMessageConverter` independently of 'json' in the content-type...
Is ```adminKeys``` the name of the permission to create other keys?
Align != wrap.
> /v1/metric is defined by our package

No it is not. Test project, showcase project has it defined liked that. There is completely no reason why some other user would not define it in some other manner.

So this should really be a configuration switch with URI templates I wrote about. I could accept that there is some default. Or even that you construct default like you are doing it here, but it cannot be just concatenation you do here.
Please use the same code format used throughout the code (and at LinkedIn...) which is curly brace on the next line
Should be `<inheritdoc />`.
Could you do get_object_or_404 here please
drupal_get_form http://api.drupal.org/api/drupal/includes!form.inc/function/drupal_get_form/6 doesn't take form state as a parameter.
VERY AWESOME
Good point, maybe we could add it via a require "active_model/serializers/set" ?
Yeah, that's probably not a bad idea: just move your ``create_html`` function to the new file and import from there. That file should probably be called ``format.py``.
That's in the docstring but I an also add it here.
*sigh* :)
Зачётный короткий селектор!
Is it possible that the first file is in a subfolder ?
wizLog to NSLog
Вместо swap?
Good point. Was struggling with that sentence. Fixing.
Should not be necessary.
I know that ;), but still I'm unblamable for that :p
default Parameter macht bei einem setter keinen SInn
and remove the ``routing_key`` too for the latter.  you also need to add a note in the CHANGELOG file about a BC break in the configuration.
> Essentially pushing Article.unread_count_by(current_user) and Activity.unread_count_by(current_user) down to the view layer. Like I said I am not crazy about that.

I used to think like that as well, I created those as helper methods in ApplicationHelper and calls them in view template, and I feel even we put these in helper methods, but it still query DB in view layer. Is it good enough to break MVC at this point?
Those are repeated a lot, maybe make one test which checks those for all types of resources at once? So the fonts test can focus on specifics.
@jeffheifetz i vote we don't use a status flag at all. See my comment about editing the existing keepSource flag.
Please revert this file