labofsahil/github-event-dataset-2012 huggingface dataset:
`hibernate.ogm.datastore_grid_dialect` is the correct property, but
I'd prefer you to not check on this, for two reasons:
1) this module is going to be activated _from_ that property, so it's
responsibility of the core to make sure the property is correctly
interpreted - not this module.
2) this one might be extended and be registered as a different
datastore_grid_dialect
3) We might be able to auto-detect the correct grid dialect in future,
so that the `hibernate.ogm.datastore_grid_dialect` might become an
_optional_ override from the user part: not necessarily set but still
correct to use your Redis backend.

On 28 June 2012 21:20, Seiya Kawashima
<reply@reply.github.com>
wrote:
>> +     private Pattern setterPattern;
>> +     private JedisPool pool;
>> +     private Jedis jedis;
>> +     private static final String PROPERTY_PREFIX = "hibernate.datastore.provider.redis_config.";
>> +     private static final String ENTITY_HSET = "OGM-Entity";
>> +     private static final String ASSOCIATION_HSET = "OGM-Association";
>> +     private static final String SEQUENCE_HSET = "OGM-Sequence";
>> +     // entityKeys is meant to execute assertions in tests only same as getEntityMap() and getAssociationsMap().
>> +     private final ConcurrentMap<String, EntityKey> entityKeys = new ConcurrentHashMap<String, EntityKey>();
>> +     // associationKeys is meant to execute assertions in tests only same as getEntityMap() and getAssociationsMap().
>> +     private final ConcurrentMap<String, AssociationKey> associationKeys = new ConcurrentHashMap<String, AssociationKey>();
>> +     private static final String SEQUENCE_LABEL = "nextSequence";
>> +
>> +     private static enum RequiredProp {
>> +             // Reused from VoldemortDatastoreProvider
>> +             PROVIDER("provider", "hibernate.ogm.datastore.provider"), DIALECT("dialect", "hibernate.dialect"), PROVIDER_URL(
>
> I see. I got confused by especially `hibernate.dialect` and `hibernate.ogm.datastore.grid_dialect`. I just wanted to make sure that the dialect which is detected by Hibernate OGM is `hibernate.ogm.datastore_grid_dialect`. Is that correct ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hibernate/hibernate-ogm/pull/98/files#r1071587
the indentation of the = ?
Probably worth a comment what this does.
Indented for consistency' sake within this file.
これだめ
это да

On Thu, Aug 30, 2012 at 12:27 PM, Valentin Golev
<notifications@github.com>wrote:

> In pyuploadcare/file.py:
>
> > @@ -25,7 +25,14 @@ def __getattr__(self, name):
> >          return super(File, self).__getattr__(name)
> >
> >      def keep(self, wait=False, timeout=5):
>
> а, ну это конечно. но саму функцию стоит по другому назвать. store
> наверное лучше чем save
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/uploadcare/pyuploadcare/pull/12/files#r1494341>.
>
>



-- 
dm.
odd header
Done.  I used pk2 and pk1 for E and B respectively.
> "and/or" isn't quite right. (The or part.) I would switch to "and", and then add a sentence: "The latter one or two may be optionally omitted if you do not want to specify a shift or a dilation."

My mistake.  "and/or" is correct.  I forgot you added that functionality.
Oh, cool, I didn't know about that. Thanks.
Definitely. I also found it annoying when I was reviewing my own changes.



----- Sef Kloninger <notifications@github.com> wrote:
>In main/courses/reports/generation/quiz_data.py:
>> @@ -137,14 +163,17 @@ def gen_quiz_data_report(ready_course, ready_quiz, save_to_s3=False):
> >     
> >     for ex_id in ex_ids:
> >       if ex_id in r: ex_res = r[ex_id]
> > -      else: ex_res = {'completed': '', 'attempts': '', 'score': '', 'median_attempt_time': ''}
> > +      else: ex_res = {'completed': '', 'attempts': '', 'median_attempt_time': '', 'score': '', 'score_after_late_penalty': ''}
>
>in the future, can you code this way?
> 
>if a:

>   then do this

> else:

>   something else
> 
>I find it easier to read and is generally python style (http://www.python.org/dev/peps/pep-0008/)
> 
>
>        —

>        Reply to this email directly or view it on GitHub.
>
>
Photos can be used for farmers and users and any other model
please remove empty lines (here an below), because they're looking ugly. It's also nice to have a comment on each method which is written by us, because comments makes code more readable :)
This is not a groovy 'variable', but a shell script variable.
MockAppServer ?
Better to use guava's PreConditions.checkNotNull.
I'm not a big fan of using `assert` for things that the user is responsible for passing in or setting. Most real users will not run with assertions enabled, so users don't get nice error messages.

But I know that this is a straight groovy->java conversion with the use of `assert` being in the original groovy...
`"Resources/CCB" ` does not receive a 4th parameter in JSB.
So, it is super confusing and very hackish to use it.

Instead, if you need it for HTML5, what you should do is something like:

```js
// in the boot-html5.js file add something like:
cc.BuilderReader.setResourcePath("Resources/CCB/");

```
Put a blank line after this.
There is some weird whitespace issue going on here.
I don't know what happened to my previous comment.. but I believe this is related to https://jira.mongodb.org/browse/PHP-502
Unless I create a build scrip to take them all out, then yes.
They are INCREDIBLY necessary for development purposes.
The user will NEVER see the logs, though, unless they plug their phone into a tracer in development mode.

Either way, both OS's are constantly outputting information like this anyway.
Roger that and already fixed.
Because the Configuration endpoints are weird and don't work like anything else in the system. It's opaque, but not going to become clear without a larger change to architecture.
restartSplunk was there in the Java SDK when I got it. It's in the test suite, though, so renaming it shouldn't be a problem.
`Symfony\Filesystem` already do that work. Why did you do that ?
opps that wasnt meant to be committed..
This line looks a no-op but it is used in the main b2g/ package. If you want to remove it you have to do a patch in b2g/chrome/content/webapi.js, b2g/chrome/content/shell.js
this interface is a stub for the PHP 5.4 interface
Sure you meant to change this?
Remove pop call so our token stays valid for duration of session
Is this the only dependency?

They provide a Portfile which states:

```
depends_build       port:rml-mmc \
                    port:omniorb \
                    port:sqlite3 \
                    port:gtime \
                    port:cmake

depends_lib         port:lp_solve \
                    port:gettext
```

OS X provides sqlite and gettext, I think.
Not sure about the others. Could you double check the deps?

Also their portfile has `--without-paradiseo` in the flags because it doesn't build  with clang.
`Math.floor(stars)`
Sure. The question was around accessing the configured values for upstream modules.

As an example: module B depends on module A. When I do test-compile for module B I want the analysis from module B compile, and module A compile. These are cached in memory after compiles, and fall back to reading from the cache file. So the settings needed are module A compile cacheFile (default A/target/cache/classes), module B compile cacheFile (default B/target/cache/classes), module B test-compile cacheFile (default B/target/cache/test-classes).
that is, ```load_remote_error``` method
Конструктор EolnNormalizedReader не умеет кидать исключения.
Левый вывод в stdout недопустим.
Fixed 2020cbb
I feel this belongs in an info block, rather than inline text
We should use spaces in the locator here.
this is wrong. ``$attributes`` does not contain the attributes of the tags here, but the array of tag attributes (a service could be tagged several times with the same tag).. You need to loop over it (and you should rename the variable to avoid the confusion)
Hibernate、そしてデータベース
↓
Hibernateとデータベース

のほうが自然な気がします。
You should just be able to pass the `group` hash directly in as the second argument to `@connection.post()` without needing to convert it to a string. Were you having problems with that?
**bias of** the `L1` norm.
My mistake, I removed that in error.  Its back now, only min_density has been removed from `forest.py`
Go for it! Feel free to just get this merged as is in the mean time. It's nice to do pull-requests/merges in small incremental chunks.
How about returning a tuple, so this can be used with multiple symbols?
I just ran a test, it's still required for some reason:

```ruby
grep_stdout(/plan/) { PlanOption.all.map(&:expense_ratio).average }
# => 0.00555181439866831
grep_stdout(/plan/i) { PlanOption.all.map(&:expense_ratio).average }
# => ActiveRecord::StatementInvalid: Mysql::Error: MySQL server has gone away: SELECT * FROM `plan_options` 
```
why is pickfirst here?
Comment renamed.
Les options d'utilisations de l'indexeur doivent passer dans une rubrique dediee, ici on ne traite que de windows.
-> supprimer la portion qui suit (généralite) et la placer dans une zone dédié indexeur, et coller un lien pour y acceder
-> mettre en evidence l'installation windows :

```
  .. code-block:: none
    Windows specific options :
    [--install]                         : install as service
    [--remove]                          : remove installed service
    [--run]                             : run into console
```
This smells wrong. I think the group should be renamed to just :test. @JordanHatch can you confirm that it was loading webmock in production?
This fixes e.g. http://ole.michelsen.dk/viewsource/?uri=%3C/title%3ELOLWAT.
The closest bug I can find that might be related is here:
https://cloudfoundry.atlassian.net/browse/CF-13
That code is still present in gerrit at the moment.
The reason it needed a try/catch was to catch its own throwing via ```lookupFile```.
Necessary to compile under Windows
These lines can be combined with

    $$.setExpr(record.getClear(), $1);

I am slowly changing existing code when it is modified.
No, because that happens later on line 163. 

All the other if statements above first turn it into a `str`.
over-indented (pep8)
Doesn't it? It does in the URL but not the link text, or am I missing the point?
Проміжний метод додали через те що, ти сказав  підключати  фічі з тестів не через require_once, як ми робили раныше  а через $this->useContext , якщо робити так то ми не можемо достукатися до методів Мінка, тому використовуемо проміжний метод
Agreed, this sucks, and that comment is inaccurate. There's a ticket to fix it I think. This refactor should actually make it easier, since it's now in an object that's a singleton in puppet, instead of in a bunch of autoloader instances.
you can actually just unset the top property with jquery's css function. this will remove the property from the style attribute without obliterating anything else that may for some reason be applied
JSDoc has to be updated: the first parameter is now the whole configuration and not only 'On' listeners.
attach cc-left class and remove from this definition
should use a different param other than redirect_url to avoid potential conflicts in the oauth dance.
I think these follow on ifs should also be sub methods, rather than all inlined.
`(not= (count (set a-seq)) (count a-seq))` would be a cleaner choice.
Why aren't you using datetime's built in isoformat()?
Why not same as above?
Is there an easy way to see how up-to-date a slave is? Seems like that would be much better than just picking one at random...
Why were the changes in this file necessary?
I also don't know about the `c` suffix to the function name, but I couldn't come up with anything better at the time (warning: this is some of the first JavaScript code I wrote).
If we had any places using real mocking, it would be confusing, but until then I don't think it matters.
TerrainProvider is private (until we merge the terrain branch) so I'm not going to worry about documenting it properly for now.
This was updated in the pr - it's now deleting the section if it exists and always adding the new one.
this is true for Mobile Web all the time, no need to set style property (it isn't actually supported by Mobile Web).
I prefer the constant check as I mentioned before.
Actually it seems that mobile web also supports the INPUT_BORDERSTYLE constants.
Fixed in next update.
Could be.  I was thinking they should be set outside the class and then passed to it, as this is more flexible for someone wanted to experiment with different values but not wanting to mess with the class.  But either way is fine with me.
and I can add a line comment right here.
Think filter_attributes could be simplified to:

source_attributes.reject { |r| r.keys.first =~ /^deployment/ }
I like that a lot. Breaks it down for easy testability too. Just make sure the predicate "if `promise` is fulfilled" is consistent with the rest of the re-worded document's phrasing for that situation.
I think this is supposed to be `if ($this.data('originalTitle')) {`. I'm guess that the not operator is a leftover from the first version, and `originalTitle` is saved via `.data()` below, not `.attr()`.
Upload behavior save on database table "{$model}.{$field}_dir" (ex: users.avatar_dir) , so this helper should work the same (but i don't test it yet).
Is the check for `isDynamicArray!S` actually necessary? Doesn't `isSomeString` already guarantee that it's an array - at least now that you've changed it so that `alias this` doesn't pass `isSomeString`?
We should reach an agreement with @WalterBright and the other compiler devs which attributes of functions must be specified explicitly and which are to be deduced. Otherwise it seems we'll be adorning symbols redundantly.
not worth distinguishing between `null` and `""`
I think "has length" is actually better here
Wait-wait. This clear should empty AA, I think it's a method of AA.
 If destroy does exactly this - fine. If not it's better to revert this change.
I'd say generally we're phasing string predicates out anyway, so it's unlikely new code would even use string predicates.
Please don't use parens with template arguments when there's only one template argument. It's just clutter really, and it's generally agreed upon that it's better to elide the parens.
shouldn't it be enough adding this into the root pom? would reduce the places where to look for additional repositories...
I think this can be combined with the previous

```ruby
config.vm.customize ["modifyvm", :id, "--cpus", 2, "--memory", 1024]
```
This probably shouldn't be in here.  If anything, add this to `PSU\Template` if it will be included everywhere.
So you're abusing the prototype here (you shouldn't call functions on an object's prototype directly). Are we never creating an instance of KeyListener? Or is it not available here? Anyway, if you want static functions, a separate object would probably be cleaner.

```
let KeyListenerManager = {
  saveKeybindings: function () {
    ...
  },
};
```
Appears to work for me. What error are you getting?
On Jun 29, 2012 10:13 AM, "Eric Hayes" <eric@deployfx.com> wrote:

> This should be a pretty straightforward fix. I'll see if I can push out an
> update to this around lunchtime PST today :-)
> On Jun 29, 2012 10:11 AM, "Ben Hughes" <
> reply@reply.github.com>
> wrote:
>
>> > @@ -291,6 +293,8 @@ def git_with_identity(*args)
>> >
>> >          return ret
>> >        end
>> > +    elsif @resource.value(:user)
>> > +      su(@resource.value(:user), '-c', "git #{args.join(' ')}" )
>>
>> FreeBSD:
>>
>> <pre>
>> [hal:~]% su -h
>> su: illegal option -- h
>> usage: su [-] [-flms] [-c class] [login [args]]
>> </pre>
>>
>> yeah, that will break horribly....
>>
>> Yes, it needs a patch for sure.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/puppetlabs/puppetlabs-vcsrepo/pull/25/files#r1077212
>>
>
I'd make it use pthreads and the person who cares can fix it if it is broken ... the odds are decent that it works.
missing ``public`` (see also later methods)
I like that change
I agree with this last comment. We don't generate rdoc from theme, yet, using the `::` syntax makes it so the documentation parsing doesn't work correctly
We Devs at CC set a bad precedent by not decomposing our SproutCore views.  

In the future, lets try and break-out complex view items like this into their own SC views.  

It would just make them easier to maintain and test.
si, pero lo saqué afuera por un tema de expresividad. Meh.
Тогда уж и @brief для счастья.
Can we use toggleClass here?
Since this plug-in exposes a global variable, can we use one that doesn't do that?
https://github.com/luminosity-group/jquery-equalheights/blob/develop/jquery.equalheights.js
`parseInt` is missing a radix param (should be 10)
Problem right here... in add_axes(), the key is made in the beginning without the extra_kwargs, but here, the key could be made with the extra kwargs, possibly preventing the ability to find an existing axes.  This key making needs to be moved up to an earlier spot in the function.
consider a very large import with hundreds or thousands of cases, all of them created by two or three users. Might want to maintain a cached map of emails to User objects, rather than looking them up anew each time.
MINOR: There no a sneaky 't' trying to get in on the act!
Also, this is a temporary code that will be removed once the SPR issue is addressed.
You created a private attr_reader for @puzzle which is great, so you should probably use it in this method :smile:
If you only want to show text1 then in the filter defined in config.py you set text=text1 rather than text=text1+"\n"+text2+"\n"+text3+"\n"+text4
This then means that other entities can be supported by setting entity.text with line breaks where appropriate in the genPOI rather than setting 4 separate text variables
We could remove the dev subdomain access.... but it doesn't really matter, as the only people that would be able to use anything from that server would have to be on the VPN with their mobile device... and have access to the dev server.

Leaving it in only helps save time when pushing code to production stage. :)
前に処理したPullリクエストとコンフリクトしているので、pull して再トライしてください。
Isn't there a risk that the system falsy believes that the paswword is always brand new if we let the "_enforce" field ?
From what I understand, _enforce would mean from the cfengineers that the user cannot change it's password, but what's actually happening is that each day the /etc/shadow file will be updated, with a newer "day_since_epoch", isn't it ?
Good call. Want me to make that change quick?
I agree that is better!
try
```python
return body or ''
```
right, i will create a new email service method to send an email for this flag
> How about class Foo extends Record { val x = ... }? I'm not sure if this is actually supported right now, but we may want to support case classes in the future.

So `new Foo` would have type `Rep[Record { val x: … }]` (as `new Record { val x = … }`)? Ok, I can try to change that to accept all manifests (it requires to adapt the `emitDataStructures` function).

> Would m: Manifest[_ <: Record] work for the constraint?

Yes but the cast below would still be required since `sym.tp` has type `Manifest[Any]`.
Should this be **Copyright 2012 Research In Motion Limited.**?
I didn't actually change this entire file - I just converted all the tabs to spaces.
> Btw, IndexingActionType.UNCHANGED is a adjective describing a state whereas the other values are verbs. I think this should be consistent.

True but by definition not changing something is not an action :) Do you have a proposal for the name? We used to have `PASS_THROUGH` but it does not solves your problem and was not really descriptive.

On the subject of the interceptor name, here it what it does in detail:

- if the hierarchy does not define a specific interceptor, then that's semantically (and in practice) as if there was no interceptor
- if the hierarchy defines a specific interceptor, then we inherit this interceptor and reuse it

Any name proposal? I am not too happy with the current name either.
surely < not >=
changed on "ASSIGNMENT"
A blurb about each supported dimension type would probably be useful here
The parameter is not of type view but rather a dictionary with a parameter view
so the call would be of the form `optionDialog.show({view:myView})`
Android-specific
Fixed.  Please review.
Instead of adding this explicit config to settings we could use Django's conventions.

The [documentation](https://docs.djangoproject.com/en/1.4/topics/i18n/translation/#how-django-discovers-translations) says:

> 1. The directories listed in LOCALE_PATHS have the highest precedence, with the ones appearing first having higher precedence than the ones appearing later.
> 2. Then, it looks for and uses if it exists a locale directory in each of the installed apps listed in INSTALLED_APPS. The ones appearing first have higher precedence than the ones appearing later.
> 3. Then, it looks for a **locale directory in the project directory**, or more accurately, in *the directory containing your settings file*.
> 4. Finally, the Django-provided base translation in django/conf/locale is used as a fallback.

I understand that today the project layout is a bit messy, but we're working to simplify it.
are we using this css or is it from the seed app?
Это как я понял делается с помощью волшебного прототипа?
function Event (start, end, location, id) {
.............
}
Event.prototype.leaveMark = new function(star) {...};
Cool function
Still not seeing it
shouldn't this be existsSync?
What is this isDelete flag?
The browser couldn't use a submodule because of their build system, but for webworks we should be able to
rebase from next, this method has change on the webplatform.framework
run jake clean before all other jake task to avoid the jake lint error.
On 02/23/2012 01:47 PM, Christophe Coevoet wrote:
>> @@ -32,6 +32,7 @@ public function __construct(\BaseFacebook $facebook)
>>
>>       public function logout(Request $request, Response $response, TokenInterface $token)
>>       {
>> -        $response->headers->clearCookie('fbsr_'.$this->facebook->getAppId());
>> +        $this->facebook->destroySession();
>> +        //$response->headers->clearCookie('fbsr_'.$this->facebook->getAppId());
> please don't use comments to keep old code. Git is about versionning code, comments are not
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/FriendsOfSymfony/FOSFacebookBundle/pull/122/files#r480084
>
Don't be mad please, I wont do it again ;D

-- 

*Behçet Mutlu*
/Senior PHP Developer, Enuygun.com <http://enuygun.com> A.Ş./

Merdivenköy Yolu Cad. No:6
D Plaza B Blok Kat 4-5
34750 Ataşehir İstanbul

Tel: 0216 575 0140
Faks: 0216 575 0142
If it's in the __construct why call a setContainer ?