labofsahil/github-event-dataset-2012 huggingface dataset:
This test doesn't seem reasonable. The `name` and `callback` for the event are mandatory for `on`, and I don't see  where this would ever be an expected case.
Not really a own implementation. No points from this one.
maybe use `getOpacity(element)` directly here.
I don't think so.  PopulatorJob is part of the scheduled job, whereas PopulatorInit I think is part of the job that loads data into COD at startup. I believe this will change so that it loads data from several input cources.
OK. Then let us do "confirmation" everywhere? OK?
Try to explain this in the comment in `form.html`. ;-)
And please wrap things properly.
What is this? This is some recursive stuff?
This has been already tested (the Likeness functionality inside the `Equals` method). The existing tests (should) cover that the generated Resemblance is equal to the original value..
I don't see why, capture is a valid standalone function of the gateway.  If you want it private anyway, let me know
is there no way to just reference the base gemfile and override, we need to specify everything?
add in new sql file
What is OrmAdaptor? Should this be replaced with ClassyEnum::Adaptor::Base ?
"Has" is for singular subjects, and you're talking about "all coupons"... :P
I don't see why you need a different variable
I'm not totally understanding why you need to store the _entityName_ . It seems you're needing it in the context of _convertFromJsonOn_ invocations; could you elaborate on the need?

It would be great if you could figure out the entity from the used table and field names, as Hibernate always does from other databases.

If you really need it, I'd prefer to see some consistency rather than have it occasionally set to "" as in the current code, or we should at least commend this carefully.
Kool. If this passes test, I love it.
This is not correct.  There is a distinct difference between `git@github.com:jenkinsci/jenkins.git` and `ssh://git@github.com/jenkinsci/jenkins.git`.  It might look like they are interchangeable to you, but that is incorrect.  In fact, by default, the second one would be identical to: `ssh://git@github.com:22/jenkinsci/jenkins.git`.  Note that the SSH port number must be specified via a colon.  With your change, you are now assuming that the path to the git repository is "22/jenkinsci/jenkins.git", and that is wrong.

Take Gerrit code review, as an example:  `ssh://jenkins@gerrit.company.com:29418/jenkinsci/jenkins.git`.  This is the proper format of an `ssh://` URL.  Your change makes that incompatible.
What happens when conf.get(key) is null? Does the method return null? Should add that case to javadocs and tests if it's not there already.
The default behavior of <code>click_sign_in(self, action="new")</code> is already <code>new</code>. Did you specify <code>action="new"</code> for readability?
Yes, should have been with the TextEntry/selection stuff in #1634.
``` haml
- a = capture do
  World
Hello #{a.call.strip}!
```

?
Should not this be --contains --tags?
store history of all betas and attach to result instance
(need check for consistency across models, currently incomplete)
这里加入translator属性，标示你的名字。
If we're keeping the viewClass around, I'd say the `|| Batman.View` should be kept as part of that, so viewClass always has a value.
Why is this string duplicated with the one in line 24? Wouldn't it be better to store these strings in constants?
Good. Just about the naming:

Can you call this on_xmpp_loop_tick ?

Thanks!
I see the logic in this, it compensates slightly for the excess flux, but it's a little confusing.  I think it might be clearer to simply add Poisson noise to the image as before, and then the excess variance is very clearly just `sky_level_pixel/100` at worst in the image rather than `sky_level_pixel/100 - (phot_image - sky_level_pixel)`...
OK, I've no problem with that, will let you know when it's pushed...
Yeah, go ahead.  Should only involve the above function.
Sure.  The above text was pulled verbatim from MultiObjectDemo.py, and this line was meant to explain how this is implemented in json files.  But it is redundant, so I'll rephrase this.
+1 

Big fan of separating things like the `RiakBucket` or `RiakObject` from the transports. These stuff really need to be done across the board. (Works rather well, as tested in my riak-python2 experiment) 

However there will be more setup code in `RiakBucket` or `RiakObject`
why do we want to allow configure() with no args ?  there's no need for this
Agreed, but wish we had overloading for effectors. Passing the entity is so much nicer everywhere except in the web-console.

Returning the new member id: yes, more useful.

---
For throw if not a member, thinking out loud...

If it's not a member, then (at least in some situations) we're "done", compared to if we failed to replace it. We could use different exception types (e.g. `NoSuchElementException`), but that's possibly harder to program to (for certain use cases anyway). But returning null for "false" is unpleasant.

The first use-case for this (to replace members running on a particular LXC Host when it's about to be shutdown, or has failed), then we do treat `false` as an exception situation. We do that for belt-and-braces, to be absolutely sure that we've replaced everything on the host before we shut it down. The caller can call again if it fails (e.g. because an auto-resize had concurrently removed the entity). But really NoSuchElement should be good enough to know that we're done (because an entity doesn't move between groups).
Probably shouldn't have a default value for `url` any more.
This is confusing. What is a handler then? what's the difference with a behavior?
OK we'll leave it :)
Any real need for this inline script? Maybe just stick it in `game.js` or something?
That's a good idea. I've done that, and something similar for job.cancel().
Incidentally, it's not clear to me what these special cases are doing. Perhaps a comment would be nice.
Remote it. No dead code!
I think we should not worry about this issue right now.    I think the idx/vals discussion fromthe other note thing is more important, and maybe reviewing the break_signal PR?
¿Es necesario que exista un config-sample.php? Si no... fuera. :P
Пробел между if и (
Не только для -a надо отображать слова со счётчиком, но и для -u и -U.
/s 
http://images.mylot.com/userImages/images/postphotos/2554246.jpg
/ 
http://cdn.theguardian.tv/brightcove/poster/2012/8/23/120823HoverCraft_6577861.jpg

:)
Any advantage doing something like?

```
X_a = X_i[X_argsorted_i[a]]
X_b = X_i[X_argsorted_i[b]]
t = X_a + ((X_b - X_a) / 2.0)
if t == X_b:
    t = X_a
```
article is missing. Maybe also "of an eps-embedding" or "a good embedding" to be more specific.
The label encoder must be kept to transform the labels back in predict
thought.
On Aug 17, 2012 8:51 PM, "Alexandre Gramfort" <notifications@github.com>
wrote:

> In sklearn/lda.py:
>
> >          return self
> >
> > +    @property
> > +    def classes(self):
> > +        warnings.warn("LDA.classes is deprecated. Use "
> > +                "LDA.label_encoder_.classes_ instead.", DeprecationWarning)
> > +        return self.label_encoder_.classes_
>
> I thought every classifier should have as attribute classes_
> I am -1 in exposing label*encoder* to the user.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1000/files#r1402034>.
>
>
Could you tell me what those lines are?  If I knew a better way I would gladly use it.
The check is now using substr().
Protection to chars is already duplicated in JSON and HdrGen,
you might want to merge them, see dawgfoto@f3513e6.
Probably it will correct to initialize other debug members here.
Apart from the comment, this doesn't actually check if `user_id` IS NOT NULL.
Although technically true, our types are a bit messed up right now. Callers need to be able to use this as a `<RCLSignal>`, and since `RACSignal` (the class) conforms to both, we kind of hack around the fact that it's the wrong protocol.
With the new syntax DATASET(<n>, transform) it is conceivable that the number of rows will be > 2^32 - e.g., for generating sort test data.  
Unfortunately the current helper function returns an unsigned, (and the transform probably takes an unsigned) so we will need a new helper function to fix that.
However the logic in thor should assume there might be very large numbers of rows.
Ah I missed this, ignore my comment about the order of the foreach in the template :)
Indeed. Merge artifact...
"ihttp"? ;)
"Эсвэл 'or' нөхцөлийг ашиглаарай" гэвэл илүү оновчтой. Удирдлагын урсгал гэвэл хүмүүс ойлгохгүй
maybe chromeSetTimeout etc will be easier to read.
probably, i just hate patch so much fo this type of use case.
i am a bit concerned about this method as well as ``setExtraHeaders`` .. the logic in them isn't easily obvious and seems fairly redundant. isn't there a better way to achieve this?
security-wise there isn't really an issue .. since anyone that can change the config, can likely do more harm otherwise. however you might still want to use http://ch.php.net/manual/en/function.escapeshellarg.php
Nevermind, my brain was just zooming past the <type> after the :m for some reason.  Yeah, it shouldn't be `:modification_time` there.
So, testing now, but I feel you should check for the existence of the files before trying to open them. Here and below on the hive.
not only 422 is possible, right? what about 401, maybe some more
I'm not quite sure why the previous version was working for me after
looking at the gem source for oauth.  I also couldn't tell for sure
from your list of gems where a mismatch would have occurred.  I
thought it could be the rack gem, I was using 1.3.5, you were using
1.4.1, this is required by oauth.  I upgraded, but I still couldn't
repro this bug.  So, I'm handling both cases, and since other people
must be having this problem, I guess, I pushed another version.  It's
0.0.98b and committed the changes to the repo.

On Sat, Mar 3, 2012 at 5:42 PM, Scott Hyndman
<reply@reply.github.com>
wrote:
>> @@ -299,8 +299,7 @@ def call(method,args,requires_auth=false)
>>        if Rdio::log_posts
>>          Rdio::log "Post to url=#{url} method=#{method} args=#{args}"
>>        end
>> -      resp,data = access_token(requires_auth).post url,new_args
>
> Running against the gem in your dropbox works perfectly.
>
> Weird, huh?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spudtrooper/rdiorb/pull/7/files#r515089



-- 
Jeffrey Palm --> http://jeffpalm.com
`fromPointsAsFlatArray` has no tests that provide their own `center`, `stride`, or `result` parameters.
You can use spinning in place of spin.getMovement(), or remove && spin.getMovement() from the assignment to spinning above.
If this throws, won't `toString` crash later?  Should we initialize `this.error` to an empty string (assuming strings have `toString`)?
Would a `switch` or hash lookup be faster here?
Let's use `defaultValue` throughout for code like this that we updated.
These should be require_once() to make sure the classes always exist
Parameters are positional, so we can change the name in the docs.

I recommend changing this one to "animation."
wrap individual conditions with ()
Why would we bother replacing the correct full path with '~'? It only get expanded later.
That should work. Unit tests to the rescue!

Feel free to update this PR. Can just amend the existing commit and force-push. Or add new commits and I'll squash later.
if this is getting minified anyway, can you put this all on one line?
don't mix up declaration and assignment :)
Should note we enforce() that the init range is empty.
All the code I ever wrote in Phobos uses an integral number of indents. Variable-length indentation is awful to maintain.
Declare the function definition outside is a really good idea.
I should have thought before.

I don't have any preference between function or variable, so I will do it your way ;)
would it be possible to use width=\textwidth instead of width=120mm?
Ah, OK, so your previous comment is wrong — `AndroidBrowserBookmarksInsertionManager` is not static, so it indirectly modifies the GUID maps via `updateBookkeeping`.
They don't need to be there :-) you can extend and keep private versions
for your self :-)
On Oct 8, 2012 10:04 AM, "Felix Mayerhuber" <notifications@github.com>
wrote:

> In
> components/ekb/common/src/main/java/org/openengsb/core/ekb/common/EDBConverterUtils.java:
>
> > @@ -30,6 +39,7 @@
> >   */
> >  public final class EDBConverterUtils {
> >      public static final String FILEWRAPPER_FILENAME_SUFFIX = ".filename";
> > +    public static final String REFERENCE_PREFIX = "refersTo_";
> >
> >      private EDBConverterUtils() {
>
> But EDBObjects are the main information units of the EDB (located in the
> edb-api) so they are known in publicity
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/openengsb-framework/pull/776/files#r1782529>.
>
>
I did not want the BundleTracker to pick up the .transformation-file in the pax-exam-probe
You need to put at least `2.0.*` to allow all Symfony versions in the 2.0 branch.
Not sure if the bundle has been modified to be compatible with 2.1 yet though.
`:locals => ..... `
"4 days"
Would it be better to use 2.1.*,<2.3-dev ?
So why `0.3.01`?
This is still needed to dismiss the alert.
What if you navigated to this page directly?
this should even be done in the Configuration class IMO as it is about normalizing the config
Why is this line commented? There seems something to be something wrong here. Could you add a test which covers this?
Could you call this ``delegate`` instead?
Cool, works for me
В угловатых скобках.
пусть будет. Авось пригадится
:-1:

For omitting the `()` when a constructor doesn't take any parameters.

It might be a personal preference, but the `()` has some significance (at least to me) in terms of readability.
Please single quote the changed lines in this file
Same as aboce
RedisTemplate doesn't explain much, no more then what's there now. I would if it did. Having said that I would expect people who use this adapter to know few things about red is collections and why the serializers are necessary.
Can you guess here why it is OK and why I added 'isNested()'? ;-)
And also look at AbstractChannelAdapterParser#createDirectChannel.
And let me know if the joke is not acceptable
Why isn't it enough to have here just default `Enum.valueOf()`?
We have not done this before ...
and another magic constant 5.
I have a personal preference for minimizing the size of controller methods, and I found that when I inlined all of that code, the show method became difficult to quickly visually parse. Granted, this was before I extracted the puzzle_presenter method to clean up instantiation. I still think it cleans up the class and improves readability enough to justify extraction though, even though it's not being re-used.

Thoughts? @jordanbyron
I don't think api_key makes sense as a configuration option. It is only needed for the JsonIodocsWriter, so I think you should handle it there. I would recommend just computing it based on the api_name, because the exact value is unimportant except to iodocs.
ここも private メソッドに抽出
@bp323 We shouldn't be using in SQL - can we replace this and the second line beneath with ActiveRecord syntax?
damn .. i seem to have screwed up adding the file .. and i have since then composer has removed the bundle again.
I think the rest of the info is a server internal data. Other implementation can have different way of storage so I see strange to send all raw data :s

What about store only payloads on nodes collection?
This is nil when this function gets ran
Do you intend for this to explode if your Exception doesn't have an http_status?
Out of interest - What is it that means this block ends up needing to get moved around?
I'd recommend adding a getter in this case. The string id and the handle must agree, allowing arbitrary manipulation of the id could lead to bad things happening.
Added a couple - using 'IN' with arrays, and using %K for substituting keys. I already had other substitution variables for dates / strings as well.
good one, i think it was a left over while i was implementing, first was using something else. then cleaned it up and eneded up being the same. will remove it.
Please remove the exception
Yes, I thought about that, but I choosed a class annotation only because a "SoftDeleteable" class would be easy recognizable with just seeing class annotations. In any case, we can change that.
dont need caps. startSession would be fine.
change this to require_once
Y U dporssi?
This should be deprecated and removed in favor of existing mechanisms (GCD, operation queues, etc.) and should not be exposed publicly as a result.
The method signatures of these delegate methods need to be changed to take `TiAnimation*` and should have their parameter renamed to `animation`. Accordingly their documentation should be upgraded.
One design pattern I do want to start heading towards in the future is usage of TiThreadPerformOnMainThread. That is, deprecate ENSURE_UI_THREAD, do all the variable grabbing in the JS side of things (allowing for range checking so that if we do throw exceptions, the JS actually gets it instead of being dropped on the floor), and then use a block to do the actual command.
the master has spoken.. ;)
Why don't you simply use self from above? What it this declaration for?
moved the method to the html-options a line above.
Однако.
Их там несколько?
Ежели нет, то лучше $del($x(
There isn't a particular reason.
I think the original dialog had OK in call caps? Minor thing but maybe someone can check?
refactor out into method