labofsahil/github-event-dataset-2012 huggingface dataset:
I see you've used the annotation for a custom config builder, but shouldn't this be used in core configuration classes? 

I guess in core/ things are tighter and we know which builder goes with which build object.

If that's the case, we should explicitly indicate that this annotation is purely for non-core configuration extensions.

Either way, we need to be explicit.
Maybe i´am missing smth. here, but the path module doesn´t seem to have a method to do such things (e.g. converting a path to an array)
OS independent.
As above, we should use the `BOOL *stop` pattern instead.
I'm not sure I think making hist2d return that big tuple is the best thing to do... I'll try to think of something neater either tomorrow or over the weekend.
@NelleV No that's ok.
Nit - add a blank line at the end of the file
Ok, I've had more time to think about this. I think this line needs to read something like

```php
if ($this->getTarget() != 'Any' && strtoupper($info['source']) != strtoupper($this->getTarget())
    return false;

if ($this->getTarget() == 'Email' && $this->getEmailId()
        && $this->getEmailId() != $info['emailId'])
    return false;
```
Can we rework this to a `switch` statement?
Note to self - check whether this still works with/without the branding widget
It is saying import nothing as opposed to import everything.
Should there be more or less on this comment? It's such a cliffhanger!
Can we add `:focused` here so it works properly with keyboard navigation?
https://www.evernote.com/shard/s5/sh/3c528d04-54ec-4e6c-ac93-49aa248c8496/f33964a7761ab3146309eb024db95da4 
has the third item highlighted via keyboard, but you can't see it :)
`$this->value` should be NULL if it can't parse the date.
https://github.com/hash3g/webfontloader/commit/c4c4c1ffe54ac584968d8e2d763c4f82e78a75e5

2012/12/29 Bram Stein <notifications@github.com>

> encodeURIComponent
>
Good question: this is out in the wild already, so I'll fix this.
Keys beginning with "@" are to be used as attributes.
Are you sure that the method "getMetadataBag()" exists? I also found this on lines 88, and 89.
players.Where(player => player.team==balance && (player.object==null || player.object.hp<=0))
    .Min(player => player.stats[POINTS]);

GML y u no LINQ? :P
Anyway, this can actually teamswitch the leading player on a team if he's the only one who is currently dead. Is that on purpose?
Maybe you can set the thread pool to non-deamon and use jettyServer.join() here?
sry, can't be static but a private inner class will do.
yeah. any good ideas where? perhaps it would be good enough just to see it in the docs for rel.create and rel.read?
Should we just make default_value return the correct value for array, hash and set? Or maybe introduce a new method on key which encapsulates all of this?
theres a space that shouldnt be there.
`is_valid()` -> `full_clean()` as above.
add resource from plugin(jar) that is marked Vaadin-Resources
Very useful for distributed css
Done and assigned to Neeraj.
Probably SDK internal. Recommend package access.
Better, but could be done dynamically. So dynamically define those panels classes. :-)
Why move to settings? This is a program constant. Not an user setting.
Then you can also define those validity method on that embedded document! Which is much better place!
done in next commit.
@gibson042 thanks for compiling that information, super helpful. I can't get to this today, but I will surely try to work with this data ASAP... (please do the same if/when possible :D )
We also haven't thrown Leopard completely under the bus yet---so if there is no 32 bit option, there has to be some logic that detects Leopard and fails with an explanation.
this is the fix to the warning problem
One more question, you don't need to specify the redirect URL with eway? Isn't this a security hole letting people redirect to arbitrary sites?
if input is string, then run original "import_from_csv .py"
I'm not sure about the various inline filter functions here, update get's called a lot, right? It probably won't make much difference but we normally avoid defining them inline then. You can use helpers.eq in some cases, maybe add a helper for this simple "has a truthy prop by name x" case here.
Sorry about these!
Yeah, you'd need to select an object in the result list before its icon would get loaded and check if the object view gets updated when the icon changes in the result view. Better warm up that mouse button ;-). Or maybe by slowing down the icon loader ? Anyway, maybe there's something else that something else causes the object view to redraw, and since at that time the object icon is ready it just updates in place.
`pacman` used to be called `archlinux`, but I renamed it precisely because it is used by more than Arch Linux, and hopefully it will increase in popularity since it is the best package manager ever created.

If `yaourt` can **only** connect to the Arch AUR, then it should be split into its own module.
is this still for binary endog only ?
space after not
``is None`` , not equal
sounds weird to me. is create.js creating a chaos? or do we have inconsistencies?
аналогично тут. название upload весьма misleading
Would you mind?  That would be great, thanks!
Hi Mike, I've pushed a pretty big revamp of the draw and drawShoot methods, covering most of your suggestions here.  Would you be happy now to edit as you see fit?  I think I've got most of the important things you mentioned (I did actually leave off normalization though...)

I did have one question: what happens if `max_extra_noise` is 0?
Ok, I will add this heuristic for the generation.

Thu, 4 Oct 2012 16:41:29 -0700, Thomas Gazagnaire :
>
> In packages/FrontC.3.4/opam:
>
> > @@ -1,11 +1,14 @@
> >  opam-version: "1"
> >  maintainer: "contact@ocamlpro.com"
> > +substs: ["opam.patch"]
>
> there is nothing to replace in this patch, why adding it to |substs| ?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/OCamlPro/opam-repository/pull/105/files#r1768279>.
>
Okay will Objects.xxx it.
Oh wait, git might actually be using a very bright yellowy green - let me know if you would prefer a different color code.
This code change is ok.
Well, stated the way you did above, I completely agree.   I don't mind the idea the bandit.evaluate having (at least part of) the trials interface, and ALL result injection is expected to be done via the bandit in an asynchronous way.  Presumably the bandit would be required to interact with the Trials object through some interface that would be pretty identical to how experiment interacts with Trials, ast least in terms of result injection (though probably not "run").   That style seems totally reasonable to me (although not OBVIOUSLY superior), in which case this whole extended interface idea is irrelevant. 

What I was objecting to was that I thought you were proposing a mixture, e.g. that the bandit would return one job  (the "original one") one way, and then as a side effect possibly return others, and I think that's really strange and unpleasant to have to think about.
Well, because actually the result corresponding to the original spec needs to be treated differently: it has top be updated as opposed to added.   So I figured, IF things are going to have to be different anyway, might as well make the datastructure make that easier and clearer.  There has to be a convention for where the original result is anyway ... 

Btw does it make sense to separate the jobs DB from the results DB?   (Or really, the Trials object from a Results object?)  Trials are what control the job scheduling and submission, and Results are what's interacted with to for hyperopt purposes and results analysis.   One Trial can correspond to several entries in Results.  The multiple things that the andit would inject asynchronously should be in Results, not Trials.
oh ok, your example had a '1' behind both '?'. i'll change the code. thanks
a bit weird spacing here...
I haven't read it all but this doesn't seem right.  We shouldn't rely on VID/PID to identify a board type - that should come from FirmwareIAP BoardType / Revision.
Ja noch wird se nie verwendet ... das liegt nun an euch was ihr draus machts ... wenn ich die alleine in meiner eAlarm Klasse setze dann habts ihr die nirgendwo
Word of warning:  2.10 will split this into more than one JAR. i.e. you'll have to account for a `scala-reflect.jar`
Так, конечно, тоже можно. Но этот метод полностью потоконебезопасен.

Я имел в виду, что надо кешировать полностью всю структуру объекта (поля, сериализаторы для полей и тому подобное). Но нельзя хранить изменяемое состояние в классе XmlBinder - как минимум от этого он становится непотокобезопасным. Как максимум - alreadSerialised не даст сериализовать объект повторно.
args может быть null, может быть и пустым.
What will `name` hold?
Why not just always do:

```
account = where(email: email).first if email.present?
```

This will work in Mongoid 2 and 3, no need for a conditional.
Forgot that one, it is now fixed as well!
I think if it worked for the dense case, that would already be great.
But how do you get a function pointer from a callable ;)
@glouppe Seems like the MemoryError is raised here
typo ``theta``
The convention is to first import standard-lib modules, then external modules, then import internal stuff.
shouldn't this be s/RBM/RestrictedBoltzmannMachine here and 2 lines below?

as for the module, `neural_network` is still shorter than `cross_validation`, why not?
This does not display right in the docs. I believe that there is a missing white space between '..' and 'math'.
я закоментировал кусок кода, потом убрал. но оно все-равно пометило его как отредактированный
в мене по-іншому не валідується, але я ще спробую
yeah true your example up there is not bad, though I guess there's not much difference in maintaining an array at the socket level or the "encoder" level
OK, so let's do it in future, right?
I *prefer* the terser output, but that's 100% only the color of the bike shed that appeals most to me.  :-)

One thing that's throwing me though, as I read it is that we put the `AWS_SES_SECRET_ACCESS_KEY` in the value `AWS_SECRET_ACCESS_KEY`.  Any thoughts about dropping the `AWS_` prefix on the internal variable so we only have `SECRET_ACCESS_KEY` and `ACCESS_KEY_ID`?  Or maybe even shorter, `SECRET_KEY` and `ACCESS_KEY`?
I think you should call `targ->resolve` only when `tok2 == TOKtemplate`. Otherwise, targ resolution will run `twice` in general cases (first is `targ->resolve`, second is `targ->semantic`).
My image is:
```
if (tok == TOKtemplate)
{
    Type *ty; Expression *e; Dsymbol *s;
    targ->resolve(loc, sc, &e, &t, &s);
    s = s ? s : t ? getDsymbol(t) : getDsymbol(e); 
    if (!s) goto Lno;
    tded = targ;
    goto Lyes;
}

Type *t = targ->trySemantic(loc, sc);
...
```
All the comments on formatDate apply to formatTime also
I was using dpMM to be consistent with the terminology DPI. Since dots are pixels, dpMM is the same as pixels per millimeter.
Changed name to be more explicit.
The current state of the switch doc doesn't seem to match the implementation of this patch.  Unless i'm missing something it doesn't seem like the enabled property is implemented, and as such should list all platforms but blackberry in this yml file.

Are there no other properties that can currently be supported by cascades?
It's a good suggestion, but it's not available in the bbndk libs.
Will do.

Speaking of these user info helpers, I've found these useful in other projects, outside the context of RZFileManager. I actually modified them into a category on RZWebserviceManager, which might be a handy thing to include in the repo.
Utiliser FileSystem.getSeparator() pour les séparateurs.
cf: http://stackoverflow.com/questions/8075373/file-separator-vs-filesystem-getseparator-vs-system-getpropertiesfile-separa
Service_name is an unused variable.
whi git-core and vim as default packages?
Looking at how GitHub indent things, there seems to be an indentation problem (space vs tabs) with the lines you added.
This could return boolean or even void. See below.
Not sure I understand code above

Shouldn't it be:

> $name = $xml_cl->{$cluster}->{name}; // i.e. name of <Cluster>  [ that's prob. wrong syntax! ]
> if ($name eq 'thor' and not defined $xml_cl->{$cluster}->{$process}->{Legacy}) {
>     $type = "thorlcr";
If we were to allow, then there would have to be other changes elsewhere I think.
The keyLen is used to size the key manager buffer, so simply allowing it to exceed,  would cause corruption.
Also need casting to a memsize_t
libgit2 was just an example, I haven't looked too deep (thought it seemed to do what we needed - read files).

Scripts doing Git plumbing is ok, but plugging a core part (IFile) of HPCC to it seems a bit too much. Then again, it might just be my paranoia...
Everything is working fine, but could you look into the focus styling on these header  links? It's not looking right at all.
> Are you expecting that the end user supplies their own version of phantom so they can choose?

Yes, that is what I was thinking. The phantomjs npm does not seem to follow the same semver as PhantomJS does, so keeping it loosely coupled would help. Thanks! I'll round back to merging this later tonight too.
Scrooge 3.0.1 depends on the following versions of util and finagle
finagle-core: 5.0.0
finagle-thirft: 5.0.0
finagle-ostrich4: 5.0.0
util-core: 5.0.3
util-codec: 5.0.3

Does it work if you put newer versions here? (it might, but I just want to make sure)
Do not use short tags.
But it's possible to edit it by user? What if I want to catch situation like this: "%nick% is %nick%" where first one should be replaced by nick and second one not
Wouldn't it be better to use a single implementation for both traits and use `return this._public; || this` instead?
Sorry, I'm not sure I understand this.
Välttämättähän ei joka pelille tarvitse luoda uutta tyhjää funktiota, mutta ehkäpä helpompi silti niin että luodaan. Välttyy tarkistuksilta että onko se nyt kuitenkin luotu vaikka ei ole laitettu MAIN_Stop funktioon
@christianv This is a different approach to organizing this logic - you're duplicating the logo img and not using ng-class for the button state. But in the end, it looks like slightly less code, so that's a good thing.
Tässä voisi myös tarkistaa `(not (empty? ...))`.
There should also be a test that we're respecting existing CSS, even if it's no longer allowed.
Fixed. I lack context but maybe "[JOB] #{job.to_s.truncate(120)}" would be better on Twitter.
I am actually quite ok with curly braces in this case.
This is a pointless comparison as far as I can see.  Also, you're using the String ```'false'``` which is a true value.  Same with the other rescues below.
Also, which version of oauth gem.  That would probably explain it.

On Sat, Mar 3, 2012 at 2:49 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
>
> 1.9.3.
>
> I'll test against a some different versions in a couple of hours and see if I can make sense of this.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spudtrooper/rdiorb/pull/7/files#r514953



-- 
Jeffrey Palm --> http://jeffpalm.com
So it looks like [getActiveUniform](http://www.khronos.org/opengles/sdk/docs/man/xhtml/glGetActiveUniform.xml) gives us no way to tell the difference between a single uniform an an array with one element.

I suppose this check can be just:
```
if (activeUniform.name.indexOf('[') < 0) {
```
Maybe have some tests that use the same instance as left or right directly as the result parameter?
Although it is not as readable, it would perform better to pack latitude into a `vec2` and Mercator fields into `vec3`.  It would also free up some uniform slots.
It doesn't really set the transform, right?  It's more like it appends an additional transform to the existing one.
If you doesn't need to modify it, you'd better to use a tuple.

If it's a constant you should use UPPERCASE (style convention).

Would it make sense to get these values out of the class to make them reusable in other part of the project?
? the activity checks are there before dialog.show() is called in line 263
No, it's the stripping was being done already, it's the predefined keys that were messed up. Pull #2033 has fix.
Could be a better check. Start < 0 could default to start=0. End > textLength could default to end=textLength
Probably also good to check start<end
cache document.getElementById
Смотри, у тебя и кодогенерация и шаблоны - используй что-нибудь одно ;-)
Thank you. Will fix in a minute.
I thought Maven style dictates *not* wrapping this line?
Should the for-loop and `cancel(true)` lines all be inside the try block? IOW, move the 'try' (line 234) before the for-loop. That change would not really change how the individual requests are cancelled, but it would guarantee that `this.composite.freeze()` is always called.

As we discussed yesterday, this really is just attempting to prevent a non-returning wait in the pathological case of an OOME or something relatively catastrophic. After all, the current `request.cancel()` and `request.freeze()` calls are pretty low-risk, and the `request` references should never be null. But even though there's very little chance that something could go wrong within this method, it doesn't hurt to make sure.
Hah!
(...not sure if I should be offended)

I'll admit, there are some reasonable arguments in that link... but that's not how language works!
Language isn't based on reasonable arguments, or we wouldn't talk ANYTHING like we do, and we certainly wouldn't speak English!
If I go and say 'optimize' is spelled with a 'z' because "that's more like how it sounds", that doesn't make it right!
That's just how language is! ;)
There was no way to put a comment in the JSON file, unfortunately, but the thread discussing this test is at https://groups.google.com/forum/#!msg/json-schema/Q5F30CZtMb4/yk_niLnsdQ0J - but anyway yes, I typoed that because it was one of the first ones I did.  Tests.py, from which I was porting, has this one listed as invalid, so I'm changing it here.
I'd be temped to give regenerateList an optional arg and remove this extra call frame
If possible don't import everything
This is a perfectly valid docstring.
This doesn't seem right. You should still be able to use @resource here?

Also, I think we should demonstrate the resource producer pattern here.
If you wanted, you could lift what's in forms for doing this multiple cursor fetch with Callables, but your sequencing looks good.
Can we make it not too early? :P
haven't we said to use the constructor?
5.5.0.CR1 is in central afaics, so the repo shouldn't be needed
it would be faster to do === null
why getting the *definition* to add it ? You should be using a *reference* instead.
well, if you want the default to be an empty string in the RequestParam, don't set it to null. but currently, you make it impossible to pass ``null`` as a default as it replaces it. And for a RequestParam coming from a json input, null can be passed (with a form encoded input, it would always be a string, but not in json)
This references the heroku app that serves the assets, see the comment on config.ru.
Should we be mixing in this from ActiveSupport instead?
Again, I think this probably needs to default to `true`.
I think this should be `e.args[0]` - you don't want to pass the `ValueError` object as the first argument to `.warn()`, you want to pass the skip constant that was the first argument to the `ValueError`.
Why the static assert? Shouldn't OSX execution just continue here?
Cool, I was just wondering since triple equals tests the type and value. Thanks
I hate to say it, but there is a race condition here (in a cluster); can result in duplicate messages - two or more instances return true for contains, but actually only removed by one.

I think you need to use MessageStore.removeMessage() which will return null of someone else released it already. Means a cast to MessageStore so we should assert (during init) that it is an instance of MS.
This IS a MessagSource, so yes RedisStoreMessageSource is the right name. The SPCA is the adapter, delegating to a source.

The AmqpInboundChannelAdapter is different. It's an active component (not polled) that extends MessageProducerSupport so it has the channel itself, thus it is a channel-adapter.
We need this (and the insert code below) to be synchronized in case 2 threads are trying to create the same group concurrently. Otherwise we'll get a DuplicateKeyException

Maybe another use for the global lock registry (on groupId).
You can fix that
I will define the base.html when I add another template.
это слабая форма неопределенного артикля :)
else の位置とか分かり辛いので、

val itemsPerPage = if (...)
    ...
  else
    ...

とか
Doing something semi interesting with the different activities
I was using a very old version of growlnotify script :-(
Thanks !
should this be `if (!this.collection || this.collection.length === 0)` ? that way if there is no collection, it will show the empty view. if there is one, and the length is 0 it will show the empty view.

will need to put some specs around this, to make sure it works the way i think it does.
Too much is going on in this `it` block.  I would do this instead:

```ruby
context 'underlying controller' do
  let(:controller) { MockController.new }
  let(:adapter)    { GoogleAuthenticatorRails::ActionController::RailsAdapter.new(controller) }
  after { adapter.cookies }
  subject { controller }
  it { should_receive :cookies }
end
```
@bp323 Can we write the variables with lowercases and spaces?
Similar to what we have in the other controllers?
well, the _id is basically the apptoken ...
Probably will be better to respond the URL and not only the apptoken.
so far nothing changed affects the pull request with Cesium. With that request, it can visualize the satellites using Cesium Viewer. I also have it working in a "real time" mode where the top level document is just one object using the external property pointing to the webservice returning eventsource data. Now if we modify the schema like I suggested below, then we'll need to revisit the Cesium pull. Is everyone ok with the below suggestion?
Why not publish the gem? http://railscasts.com/episodes/303-publishing-a-gem
Surely we can fold these into argparse/optparse? Not now (NGHUTPR), but soon?
i prefer "instanceof" but it is up to you.
Man, if this was Haskell... grumble grumble :P
Added, thanks for pointing that!
I hate you!
Whilst you are at it you might as well allow questionnaire objects to be passed in for consistency?!?
this line isn't needed; you've already specified outputDirectory above.
Hmm, okay. Mainly I'm worried about messages without long spans of text, that end up being uber-tiny. Would full-width be better?
The reason there is no code there (or the other instances) is because I wanted the person who maintains that engine to write whatever code they would expect.
"null" is unnecessary. Params are option in JS.
OK, Update to 82b94bcdf7b494cce8549802f9d97cdee82ac54a and keep the compatibility. but not test yet.
You can try.
тут кстати тоже нужно какой-то namespace типа persistent сделать.
No idea. You'll have to ask TPTB about that.
Consider using "workQueue.put()"  here instead of "add()". add() will throw an IllegalStateException if the queue is full. put() will block waiting for space to become available.
Recently, DPS gateway just 'secret' changed the check string form "PXHOST" to "PXL1", that make this pull request to be merged as a 'must-do'
not sure about that code, i got this straight from the php code which seems to work. Though I don't have an IEMobile device to test it with :/
here are the technical details interesting again; or links further down into the documentation if it's explained later
Can you put the test code into a different file? I liked that you wrote test cases, but keep the tests separate from the production code
Actually LoginOptions doesn't have a toString() method so all you see is:
com.salesforce.androidsdk.rest.ClientManager$LoginOptions@41030048

It's not very useful, I should take it out.
Need the copyright header in here as well.
the braces are useless here
distraits, pas distraient