labofsahil/github-event-dataset-2012 huggingface dataset:
@fcharlier why does this not default to 35357?
Are you sure about that, as far as I understand here you're creating a new service, Am I wrong? Really not familiar with the psexec technique, feel free to correct me if I'm wrong!
I think it shouldn't be here!
already implemented in Post::File
you need to save ``$this->directories`` too
I think I found the main cause of this - for equivalent backwards-compatible results I shouldn't be taking the square root of the gradients.  After removing that my tests are passing.
Added test case for non-circular polygons.
Missing some white-space?
As I said above - I would just make `context` required since it is for most materials, and one day it is likely to be for all.
There's no need for the undefined check, since undefined will turn into `0` for the comparison.
The convention is generally to use 4 spaces for indentation.

Also, please adjust this code wrt to `MODEL` as per the discussion on your DMD pull request.
In my testing, it was necessary on rare occasions. Since not casting can result in an exception being thrown, I figured better safe than sorry.
Move var i out of the for loop and use i++
Note that doing this does not replace any existing autogenerated values in the property list, or values from the template, such as `CFBundleDevelopmentRegion`, `CFBundleURLName`, or `CFBundleURLSchemes` (neither of which should be replaced). The following values should be considered "static" (immutable) by the `<plist>` extension, because they are directly controlled by universal tiapp.xml settings:

* CFBundleURLName
* CFBundleIdentifier
* CFBundleInfoDictionaryVersion
* CFBundlePackageType
* CFBundleVersion
* LSRequiresIPhoneOS

The following should be considered "appended to:"

* CFBundleURLSchemes

The following should be considered "overwrite" (takes precedence over existing deprecated properties):

* Any value not listed as "static" or "append" above
It's paredit.
It's some code i dug from internet and work as expected.
I think this is acceptable for a demo, but in real life, you shouldn't look for the presence of a trailing "/". Instead, you should do a stat on the dest to see if it's a directory.
Let's not mix tags and ids. Just use the id.
You can also just use show() and connect it directly.
When I moved the text, I didn't bullet it, but did so afterward because the presence of a `then` function was independently testable. For example, Crock's vows.js would fail that test, though the implementation of its `when()` matches the spec for `then()`.

I'll remove the bullets. I liked the code snippet preamble followed by behavioral specifics as well.
@jtopjian I actually *want* the ssl default parameter to be `false` by default, and I want the parameters you've added and the template changes you've made. But I think I'd like to leave the merging to @bodepd and just have him take the blame`^W`credit ;).

Just thinking out loud... this causes `apache::ssl` to not be included, which means the `package` and `a2mod` resources are not in the catalog. This would impact anyone who depends on `mod_ssl` but doesn't use the `SSLEngine` setting in a puppet-managed vhost, or is using the `vhost-proxy.erb` template without specifying `ssl => true` in the `apache::vhost` definition.
I will try to rewrite it to something simple here (and hopefully correct!), and then a further document can later be added explaining in more detail.
почему не в конструкторе? скорее всего конструктор будет вызван внутри обработчика, а аплод может быть чутка на потом оставлен

и я вообще не уверен что это нужно здесь(
should we have an interface for this? or maybe we should add this method to the base transport?
I prefer `$var === null` to the method calls.
A robust idiom to ensure that Scoped_store is aligned is to place the store in a union with a type of the highest alignment for a given size.
You need `Unqual!(FloatingPointTypeOf!T)`. It looks like `FloatingPointTypeOf!T` does not strip modifiers, so as it stands, `isFloatOrDouble` would be true for stuff like `immutable real`.
I think here we need to return `const(T)[]`.
Nit: spacing.
i would write the classname File, uppercase
Why are all the tests deleted?
We'll probably want to bomb out if it isn't an expected value.

````
if( 'spraddr' != $table ) {
  $response->abort( 400, 'UNEXPECTED_VALUE' );
}//end if
````
We should follow best practices in existing projects such as Boost and LLVM/Clang.

Once we start implementing some of these functions (and namespaces), we'll have something more concrete to examine and to discuss.

For now, I feel that it's simply important to have the "right" amount of granularity (levels of namespaces) to make the API:
* intuitive
* logical

From my discussion with Tristan, we came to the conclusion that:
1. `SageBuilder` namespacing should remain the same, i.e. `Builder::buildXXX` all of the `buildXXX` are in the `Builder` interface
2. `SageInterface` should be logically namespaced. For example, `SgArray` functions should be contained in `Interface::Array`. As an example: `Interface::Array::get_dimension_information(SgArray ...)`. Again, this is simply to follow good API design principles to make the API intuitive, but at the same time, not overly and unnecessarily verbose.
if retry_with_cleaned_cache:

would be the more pythonic version
Please add a space after the colon (`:`) in this line for consistency.
It's good to have code commented, but ... well, we know what *return TRUE/FALSE* does in a 10-line method. :)
https://github.com/FriendsOfSymfony/FOSUserBundle/blob/master/DependencyInjection/Configuration.php#L46
no @author tags allowed anymore :)
You're absolutely right -- I'll push an update to the tests with the (hopefully) last fix for this scenario.
here the rule set directories are hard coded, I am wondering whether it will work on any machine or not
it is required by checkstyle to have exactly one space between ) and {. The same apply to the following six cases.
Would be great if you could do it!
Did you add the Traversable check here?
не могу. Я пробовал на одном проекте. Я на код не могу смотреть из-за нее.  :-(
Oops, it was called "caseversion_set", but I prefer it to be called
"caseversions". Just pushed the fix to master.

Carl
Make Unqual a module-level private template? Then you get to share it.
Not sure why; was busy at the time and tbh <code>::message(std::string)</code> isn't called on the buffered entries at all. Its more a, in case someone else does, making sure we have a consistent expected behaviour.
I know you schedule and not process, but if the delay is <= 0 they will be sent immediately.

You can't "hide" the TS because on some platforms (e.g. WAS) you are not allowed to create your own threads and so you have to use a TimerManagerTaskScheduler (and a WorkerManagerTaskExecutor when you need a TE).

I don't think discard-channel is right (that is normally for internal problems). Maybe error-channel?
Became obvious when I got down to the tests.
I agree we need a test for the 2613 case.

The other issue is tricky; I don't think we want to parse the expression to figure out if the Map argument was consumed in full in the payload-expression.

If we want to suppress adding it to the headers as well, maybe we need another attribute "suppress-automatic-mapping"  ???

What happens if the gateway is send(Map map1, Map map2, Map map3)  ???
Yeah - this was originally caled last (see the JIRA comments).
These get automatically added as the result of the Additional conditions in Save actions in eclipse. This was enabled for removal for trailing white spaces but they by default contain some more actions like add Override annotation, remove unnecessary casts etc. I thought keeping these defaults it won't hurt.
And how about to avoid mentioning 'version of changes' throughout the manual?
Just change paragraph with actual state of code and add a note into "What's new" and migration appendix.
It will be more flexible and comfortably for guys who already know e.g. Spring Integration 2.1 and want to pay attention only about 2.2 changes.
So doctrine/DoctrineModule#21 supersedes this?
Im pretty sure that refinery have some methods or helpers for this..
Aren't there a few more parameters? Please check.
"its" instead of "it's" (possessive does not use an apostrophe)
Ah and `Licence` header is missing =)
Clojuresta löytyy myös valmiina predikaatit `odd?` ja `even?`.
@willyaranda This is a temporary hack. We should study how the UDP client should register WA without re-registering again the UA (since the connection will be closed again ...) - A new topic to be discussed in the office ;)
hmm... I swar I fixed that...  (it was the result of a stupid mass replace :/)
That should be mentioned in a comment so as to avoid my questions in the
first place ;)

On Fri, Mar 9, 2012 at 9:47 AM, Zoran Simic <
reply@reply.github.com
> wrote:

> > +  }
> > +
> > +
> > +  @Override
> > +  public void serialize(Map<?,?> value, JsonGenerator jgen,
> SerializerProvider provider)
> > +      throws IOException, JsonGenerationException
> > +  {
> > +    if (SortedMap.class.isInstance(value)) {
> > +      super.serialize(value, jgen, provider);
> > +    } else {
> > +      super.serialize(new TreeMap<Object, Object>(value), jgen,
> provider);
> > +    }
> > +  }
> > +
> > +  private static HashSet<String> toSortedSet(String[] ignoredEntries) {
> > +    if (ignoredEntries == null || ignoredEntries.length == 0) {
>
> Everything in these 2 Java classes is a copy-paste from the ancestor class
> to make things work. The only useful bit is the redefinition of the
> function serialize(). Everything else is fluff :) taken as-is and strictly
> minimally adapted to make things work.
>
> The usual way of providing a custom serializer does not work unfortunately
> when one is trying to provide a custom serializer for Map (it works and was
> designed for non "basic" types, such as BankAccount or Employee and such).
> Tatu took a look and confirmed, this was the way to go to customize
> serialization of descendants of Map unfortunately. I agree with you: weird
> :)
>
> So this is a bit of a hack as is, but it will be taken out with Jackson 2.0
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/linkedin/linkedin-utils/pull/6/files#r538141
>
I think this is a real thing, though.
Probably you mean *EmbeddedSortedListField*, not *EmbeddedSortedList*?
Good. Use reST for documentation.
This is fragile given how we rely on a "thread.sleep". I wonder if this test needs to care if the keys were actually changed by the ZRP. After all it tests the CLI, not the ZRP. Perhaps validation the command executed property (i.e. some valid response was returned) is sufficient?
Можно в начале кода добавить `/*jslint plusplus: true */`, чтобы jslint не ругался
The original had the wrong phpdoc for an abstract base class that doesn't extend/implement anything...
Isn't that simple the square norm 2 of Q[i, :]? If so, I would think that it can be computed without the for loop.
you don't need all of this. ``$extractor->getExposedRoutes()`` returns you an array of Route objects
Shall I update all the new files ?

Thx
--
Issa

On 11/09/12 14:41, Patrick Roumanoff wrote:
> In
> src/main/java/org/sonatype/nexus/plugins/crowd/client/rest/jaxb/SessionPostResponse.java:
>
>> @@ -0,0 +1,34 @@
>> +/*
>> + * Copyright (c) 2010 Sonatype, Inc. All rights reserved.
>
> if it's a new file, I don't think we need to keep this copyright statement
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PatrickRoumanoff/nexus-crowd-plugin/pull/1/files#r1575889>.
>
>
do I need to say it ? :)
This is one of those deep ActiveSync questions I was wondering how you were going to address, especially since there are blanks in my mind about the exact semantics of ActiveSync synchronization, especially in regards to its filters.

Answering your question about MODSEQ down below, it's like ActiveSync's synckey.  CONDSTORE uses it like a changeset number and is actually tracked per-message.  See http://tools.ietf.org/html/rfc4551

A naive activesync implementation presumably does a full folder-sync that is similar in nature to an IMAP full-folder sync when using QRESYNC.  It tells you what changed in the entirety of the folder since the last time you did a full folder-sync.  However, IMAP with CONDSTORE (implied by QRESYNC) makes it possible to use its MODSEQ values in many places, including FETCH and SEARCH commands.  I can perform a SEARCH over a time-range (which is the IMAP idiom) and use MODSEQ to only hear about the UIDs that have actually changed.

So I guess the first question to answer is how you are going to approximate an "iterative deepening" strategy for ActiveSync, and do that in terms of refreshing?  If the user only cares about the most recent 2 weeks of data on their phone and deals with other stuff on their desktop, we really don't need to hear about what happens 3 months in the past.

If you are allowed to reuse synckeys like we can reuse MODSEQ/HIGHESTMODSEQ values, maybe you could use searches, and store the sync-key for the given search time period so that next time you synchronize you can just ask for the changes during that search span.

If you aren't, it seems trickier.  I'll let you respond before I write down any more random thoughts...
invoke
exits
Add a test whereby arg[0] is an array of strings.
http://developer.appcelerator.com/apidoc/mobile/latest/Font-object.html
Bump down to every 10 minutes.
As you state `top` attribute in line 184, you can delete this one.
ahh I think I get it now, ignore the last comment
Could be reduced. 2 seconds is not horrible, but I'll change it to a half second.
elephant — WTF?
Double space before "module"
Vastaluvun -x saa helpommin myös näin: `(- x)`