labofsahil/github-event-dataset-2012 huggingface dataset:
Updated to reflect that CC is only available in SMTP API
could you add a branch alias in your bundle so that we don't need an unbound constraint here ?
is the "auth-token" rest specific? IMO establishing a connection to a protected WebSocket endpoint would require an "auth token" as well
Do we also need to escape values here, like for columns that can't have arrays?
Second parameter is for specified whether the model has already been persisted.  Though I think passing a true should be good enough.  Probably no need to use 'persisted ='.
Такой код очень тяжело поддерживать. Желательно выделить атомарные действия (которые не делают ничего побочного). И дальше сделать getMessage(), который будет последовательно выполнять эти атомарные действия, закрывая/удаляя, когда это нужно и так далее. Ну, т.е. надо думать над структурой программы, чтобы не было таких неявных зависимостей.
Этот код вообще работает?
Почти что 4 Мб на каждый запуск программы? Надо динамически память выделять.
System.in закрывать не надо
Надо возвращать ненулевой код возврата.
Нет. Для многопоточных: Vector, Stack, Hashtable и коллекции из java.util.concurrent.
Again, we're just wrapping an API call. Unless we retrieve the ID from the mock application, I don't see why this method belongs here.
Per https://wiki.mozilla.org/QA/Execution/Web_Testing/Docs/Automation/StyleGuide#Locators, please space these out.
`http` (and therefore `https` too) can deal with a `socketPath` or `host` and `port` argument. In each case the opposing argument(s) are undefined and therefore not passed to the `agent`.
Have a look at proxy-table.js from line [125](https://github.com/nodejitsu/node-http-proxy/pull/254/files#L2R125) to [136](https://github.com/nodejitsu/node-http-proxy/pull/254/files#L2R136) and [153](https://github.com/nodejitsu/node-http-proxy/pull/254/files#L2R153) to [136](https://github.com/nodejitsu/node-http-proxy/pull/254/files#L2R164).
I'm not sure what Visual Studio is doing there. I think it's just written the same thing with slightly different formatting. I think its because I'm using the full version of Visual Studio, rather than Visual Studio Express. You can probably just ignore my version of this file, and just add my 'pvr.nextpvr' project to your solution file.
"Bio-Formats's" is considered more correct in modern writing. See [this short article](http://www.englishrules.com/writing/2005/possessive-form-of-singular-nouns-ending-with-s/), and [this long one](http://www.worldwidewords.org/articles/apostrophe.htm) for details.
It's currently not needed here but ``last_alpha`` needed to calculate the sequential strong rule.
Thanks for the comment. I wanted to ask you about that but thought I'd reread your paper first ;)
My intention was to use the same hyper parameters for all folds. Will check again.
same comma here
Really? Pretty sure that not, as you can not guarantee mixing of the chain.
The theory says that in order to achieve prediction consistency, the penalty parameter should be kept constant
as the number of samples grows.
Does this need to be sync? The modules should be implemented in an independent manner so concurrent loading shouldn't be a concern from the javascript standpoint. Reduces the code amount and also simplifies the race condition between preload and actual load.
That's fine. The server module already has a package org.jboss.as.server.parsing where this can fit.
Will do, new version coming up soon.
You actually don't need to worry about doing this; VCR already does something similar:

https://github.com/myronmarston/vcr/blob/master/lib/vcr/cassette.rb#L148

https://www.relishapp.com/myronmarston/vcr/v/2-0-0/docs/cassettes/naming

Yours is a bit different (it downcases, and VCR's doesn't), but I think I prefer to stick with the consistency of what's already in place.
It would be nice if request() were a bit more verbose of a method name
As a more general behaviour note, please verify how badly formatted fonts behave on other platforms.  are errors/exceptions raised or are they silently ignored?  This goes for both the key string and the value string
Ok.

I'm fine with this article being tagged "us-east-1 only" at the top until all AMIs etc are rolled out. Same goes for the AZ discussion for non-us-east-1 regions.
You can't follow the key extract route if not available because the key/element comparison functions aren't available, and I doubt you want to always create a key.

You could probably improve the logic e.g., for fixed length records don't store the key if the row and key will be the same allocated size.
Should this be if(rollup)?
Replace rowIf->queryRowMetaData() with iRecordSize
Agreed, was being lazy and copied name from hthor entry.

And yes abortSoon is unrelated, it's a true abort condition, external to activity, indicating job failure.
I slipped abortSoon with this commit, but not related to issue really.

I'll rename abortEarly, to ... shortcircuit or something
I have open issues that will require me to use this class in additional areas.  Shall I look at moving and refactoring a bit then?
The call is still there.  Does that mean it is correct, or was the move to prepare() missed?
Good catch, whoops. I accidentally committed the fix tot he typo on the wrong branch. Will push that back up.
Ask upstream Wikipedia mobile ;)
"сайн" биш зөв гэвэл илүү оновчтой
We need a space between `solid` and the color:
`document.body.style.border = "5px solid " + color;`
Indeed but for the most part it's actually a convenience thing in terms of wrapping parameters etc. 

For this case in particular, however, i agere the validate methods should do the validation instead of returning a fucntion. I'll update.
add function destruct
Uncomment it !
I'm changing it to 'thread'.  It makes more sense that way as I write the blog post :)
Also, do you prefer parameters or endpoints like `/notifications/all` and `/notifications/participating`? I guess they are technically separate collections.
As a style note, I think we should start writing these like `60 * 60 * 24` so they're easier to read.
Why does `parse` not returning the OrderedSet instead of taking one in parameter?
Whoops. Reverting...
why not just return fingerprint? fingerpring is already a string...
Though i concur i'm a bit hazy on how to do this. I tried adding payload options like normal exploits have at the init hash, but they're not taken into account. Could you provide a quick mock-up for me to work from?
Fails on 1.8;  add something like ```if dot_net_code.respond_to? :force_encoding```
Unfortunately, I don't think this clause is compatible with the license for scikits-image ([modified BSD](http://en.wikipedia.org/wiki/BSD_licenses)).
##### `numpy` casting change

This line wouldn't work for me unless I manually cast *both* inputs of `np.multiply` to the same `dtype` as `b`. I'm guessing this introduces a copy, so there may be a better way.
What's the utility about `doc.sess.cookie.expires` and `_expiresDoc` variables ? You never use there...
I love `#tap` as much as the next person (probably more), but it reads oddly here.

With `#tap`, I assume that the thing I'm returning is unchanged by the block, but that's not true here. This code is essentially saying "Return message - oh, and change message first, I forgot to mention that".

What about this?

```ruby
def add_clarifications_to(message)
if @delegated_arguments.present?
  message << " with arguments: #{@delegated_arguments}"
end

if @method_on_target.present?
  message << " as :#{@method_on_target}"
end

message
end
The existing Paginator code that uses the TreeWalkers already throws a notice when you feed it a query it doesn't understand. 

Would it be acceptable if I fixed that bug in the TreeWalker so it throws a \RuntimeException instead of a Notice, then use that exception to detect if the correct walker has been chosen in this unittest?

A bigger problem might be how to make the TreeWalker reliable detect when it is fed a query it cannot handle. It already detects queries that have multiple FROM clauses, and I could add the code to detect mixed results. But you could probably come up with many queries that would still cause trouble form the TreeWalker and have it generate notices.
Haha, weird, I didn't explicitly do it.  Not sure how it happened.
I'll change back.

On Fri, Jun 29, 2012 at 10:13 AM, Joaquin Rivera Padron
<reply@reply.github.com>
wrote:
>> @@ -3,7 +3,7 @@
>>  /**
>>   * Defines ThreeScaleClient class.
>>   *
>> - * @copyright 2010 3scale networks S.L.
>> + * @copyright 2010 threescale networks S.L.
>
> company name is 3scale ;-)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/3scale/3scale_ws_api_for_php/pull/2/files#r1076110



-- 
Ilya
Agreed.  Start with an example using `Material.fromId`.  Also, show explicit assignment to a primitive's material, like `polygon.material` so they see things in context.  Dig through the [wiki](https://github.com/AnalyticalGraphicsInc/cesium/wiki/Fabric) for examples.  See how I build from simple to complex?
You should add the clear_params="true" line to your launch file, otherwise removing parameters will have no effect as the values still exist in the ros parameter server.
Windows paths and '\' ?
This is my preference, and is optional.  I won't reject the PR since we don't NEED it here.
ok, I removed the `sed` and `cp` commands and renamed `boostrap.php` to `bootstrap.php.dist`. ([9196b6b](https://github.com/ericclemmons/ECVagrantBundle/commit/9196b6b6348cd846e5dfae88ceffe91e9a5ed23f))
Agreed, but I didn't want UserProvider tied to using Basic Auth (and therefore base64encoded user:pass).
I could have pushed it down to the page implementation, but since that's implementation independent for the Request and Response objects, it would have meant changing that interface, and then adjust play and internal versions of HttpRequest and HttpResponse.  I can't decide if it was laziness or the right thing to not change all of that, I don't think it would have been any cleaner.
Short of creating an AuthenticationService class that take that management responsibility this was the least worst I could come up with.
I prefer to use the reset function to get the first item in an array as you often cannot guarantee that an array is associative or indexed. As with the previous items; I do not consider this blocking for the merge
What about the next guy who comes along and is messing with tests and has to rediscover it? I think it's worth having the decisions about the code and its tests and building written in place.
надо вписать куда-нибудь Function.prototype.bind
What happens if the user uses the `random` `pathMethod`? Also, I think it's `pathMethod`, not `methodPath`.
I'm with @fwenzel on which style to use.
!Important test!
Please use 4-spaces instead of tab.
If you change /* to /** on line 142 you would fix Issue 5358
> TBH, I don't know what a inout(int)[] really is.

`inout(T)[]` should be treated as same as `const(T)[]` in most trait templates such as this.

A variable which typed `inout` cannot be declared in normal function. It is only allowed in inout functions.

A hack to avoid the compiler check is using `is(typeof((inout int _dummy = 0){ ... }))` instead of `is(typeof({ ... }))`. Adding one inout parameter makes the lambda inout funciton, then inout variable declaration (e.g. `inout(int)[] r;`) will be allowed. (Note that `(inout int = 0)` is an *unnamed inout parameter*.)

In #954, @denis-sh uses `T lvalueOf(T) inout @property`. 
It it a hack to avoid another compiler check: If a function has an inout return type, it should have one or more inout parameters.
...But, I think this is not correct. `lvalueOf` is a module level function, not a member function. And qualified module level function is illegal in D. I'm not sure why this is accepted ... it is almost certainly a bug in the compiler IMO.

However, back to here, `R r() inout @property;` might be legal, because the r() is a nested function, and *qualified nested function* might be legal _in the future_. But I think there is no benefit using `R r() inout @property;` instead of `R r = void;`.

Finally, there is no merit by changing how to declare `r`.
I wanted to add a static assert here, so the user would have a good message about what was happeneing (as opposed to "error conv.d l 3454").

The assert is the same thing as the line bellow, works on my computer, buts fail the D2 tester, so I commented it out:
http://d.puremagic.com/test-results/pull.ghtml?runid=314624

Is my syntax wrong? Any idea? I think it is better with it in there.
personally I don't like type annotating locals -- I expect that the editor/dart_analyzer can infer reasonable types
You can do some logging, but not actually mark the exception as handled in
your handler.


On Wed, Nov 28, 2012 at 7:42 AM, Rafael Benevides
<notifications@github.com>wrote:

> In
> deltaspike-exception-handling/src/main/java/org/jboss/as/quickstarts/deltaspike/exceptionhandling/Controller.java:
>
> > +
> > +    // Inject CDI Event
> > +    @Inject
> > +    private Event<ExceptionToCatchEvent> catchEvent;
> > +
> > +    // The Service implementation always throws Exceptions
> > +    @Inject
> > +    private MyService myService;
> > +
> > +    public void testMyException() {
> > +        try {
> > +            // This Operation will throw an Exception
> > +            myService.doSomeOperationWithAnException();
> > +        } catch (MyException e) {
> > +            // Fires the Event with the Exception to be handled
> > +            catchEvent.fire(new ExceptionToCatchEvent(e));
>
> One doubt: During my tests I had no @exceptionhandler<https://github.com/exceptionhandler>and I got an @ObserverExcepton: no handlers found to ....
>
> Isn't that a deltaspike way to check if the exception can be handled ?
>
> I made another test and removed the handledAndContinue() method from all
> handlers and the isHandled() continues to return true.
>
> So I'm not sure when the isHandled can be false. Can you clarify me ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jboss-jdf/jboss-as-quickstart/pull/361/files#r2250855>.
>
>



-- 
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp

Software Engineer
Open Source Advocate

PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu
Global tag should be lifted out at some point.
should I just add a catch(Exception e) block?
A complete list of instructions for this topic
This looks eerily familiar... :)
unless config[:user].nil? || config[:password].nil?
Sure, I deleted.
A lot of this stuff looks the same; can we put it somewhere common?
Third parameter should be an integer - remove the quotes.
spaces in formatting in this statement. :)
Yes, im aware of that 'weird requirement' but I didnt have any idea how to pass FOSRest route generator limitation.
Anyway: multiple inheritance won't be neccessery as only the parent class must inherit DTO.
(notice line: https://github.com/imagin/FOSRestBundle/blob/17df9f878fae7c756b48999332dc52c6b2c4b9ba/Routing/Loader/Reader/RestActionReader.php#L272 
```php
$argumentClass->isSubclassOf($class)
```

(btw is there better way on github to add link to a code fragment?
shouldn't it be a booleanNode ? And you could use ``defaultFalse()``
``@static`` should be removed
I added naming only because the constructor has two string arguments, and it is very easy to mistake one for another.
assertExceptionWasCaught
on the rendered page view that I looked at, these two lines run together
This will only allow you to make pulls against the same user. This works well for our purposes, but may not be flexible enough for the gem.

Is it possible to only specify where you would like to merge the code? It could use the existing checked out branch as the source and the params you specify as the destination.
According to style guidelines, public keyword must be explicitly set
Two things I wasn't working on: settings and YapiService fasade.
It's also temporary solution.
The reason I reverted to versioned statements was that it was very unclear to the reader what these types are supposed to be. For instance, it is mentioned nowhere on dlang.org (at least from what I could find) that pointer - pointer = signed word (which, for the record, seems absurd to me since the address space is unsigned, but I guess it's carried over from C).

I think it's also semantically wrong to alias `size_t` to `AnyT.sizeof` because that in turn is returned as a word-sized integer, while `size_t` itself represents size of a reference to an object in memory (these two are different things).

Due to the above, I think it's generally easier (and saner) to manage these aliases by hand. Either way, I'd have to add a version forest just for `ssize_t` because I don't know how else I could define it.
the excludes should be in the parent pom (similar to other projects) and the child pom only references the dependency (see https://github.com/NessComputing/components-ness-event/blob/master/jms/pom.xml) for a example
Ah yup. You're right on the money.

I'll refactor and commit.
The problem is that any errors in settings.py file is hard to debug. So this is why it is advisable to import as little as possible there.
Please add a _description too
Picked up that config in https://github.com/zendframework/ZendService_Amazon/pull/32
replace this Javadoc with something useful, describing the purpose for the class.  We do not need audit info (like IntelliJ, who created it, etc).
The property was changed so it could be updated after the polygon's inner and outer rings were joined. Once that code is refactored and decoupled from the shapefile project, this will go back to being immutable.
To cut back on line length, how about
```python
bg_image = horizons.main.fife.imagemanager.load("content/gui/images/background/sq.png")
res_icon = horizons.main.fife.imagemanager.load(get_res_icon_path(res, 32, False))
self.renderer.addImage(group, node, bg_image)
self.renderer.resizeImage(group, node, res_icon, 24, 24)
```
making corpuses into a colleciton of corpusmask as they are smaller and all that is necesary for a link view
Yes, I knows this is probably coming up later on. I think we can do it as clickstop 2: at this point I did not want to work on 2 separate projects (SDK for Mobile and CLI for mobile).
misspelling of polymorphic (here and a few other places)
@mandrews what exactly do you mean by that?
khob man k hamin karo kardam .... bara imagehaye gallery ye foolder va baraye slide ye folder e dg va har dosh too ye folder be esme soodmand-img ..... manzooretoon daghighan az folder mojaza chie????? mage hamin nist???
Feel free to leave anything non-essential alone for now.

On Mon, Oct 1, 2012 at 9:22 PM, Jeff Hemphill <notifications@github.com>wrote:

> In DjangoProject/HvZ/models.py:
>
> > @@ -124,7 +124,6 @@ class Registration(models.Model):
> >               ("Z","Zombies")
> >      )
> >      player = models.ForeignKey(Player)
> > -    hardcore = models.BooleanField(default=False)
>
> Will changing the model screw things up? I'm of a mind to leave things as
> is at the moment.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Claremont-HvZ-Web-Team/claremontHvZ/pull/50/files#r1735589>.
>
>
Again, a good time for ```if not actee.owner.organizations.exists():```
the nagative class does not provide any meaning even it is misleading. Can we think of another interesting name?
this kind of simplification looks really bad. `<section>` elements aren’t necessarily direct `<body>` children, there can be a bunch of `<header>` elements, etc. I’d have kept at least the `[role=region]` to avoid applying rules to any `<section>` element.
This needs to be `image-url('theme/openstreetmap/img/map_sprite.png')` in order to work via the assets pipeline.
We probably should call the callback here.
Hit the refresh button. :)

It's `{ SaveInSent: 'T' }` below (although I think Gmail always saves in sent anyway).
It was acutally outputting <pre> and </pre> in the link text when i tried to do it inside the l function's first argument.  I'll see if i can find out how to make it work properly.
No, thanks, the about page doesn't have any special link_url whereas these did
That should maybe be changed to be more remote-debug friendly.
As far as the template is a copy of p.p.collection, this should be reported upstream shouldn't it?
still not good enough. Think about fitting in 4-5 lines
can you reverse the test to return null if $id is not set. and remove the else.
I think the wmode can be removed from the embed tag. Studio complains about it as a proprietary attribute, and removing it seems to make no difference.