labofsahil/github-event-dataset-2012 huggingface dataset:
Actually this bit is a performance optimization when using mysql which does a case insensitive match by default. ilike is very slow so when using mysql you set a gp to do eq. See https://tickets.openmrs.org/browse/TRUNK-2767
It is not a pretty name thing.  A build version (ie +) semantically denotes something when it comes to sorting/precedence as documented in #12 of Semantic Versioning 2.0.0-rc.1.  Basically build/nightly versions are sorted before release versions.

Once we have a real CLI option we could expose an option that allows you to say a build is in fact a release build thus leave the the build denotation (ie `+` and everything after) bit off.
totally. will update.
Not my idea in the first place, but you are right!
Sure! :) I guess we already need PSR-4!
Can you refactor this to only look for orientation when we need it? As it is now, it'll be parsing out orientation always.
use `currentMoneyMode()` here instead of duplicating the functionality
intentional. we always send back async, irrespective of doneSync in camel processor.
`assert.notEqual(e.toString().indexOf(dlerror_msg), -1)`?
`Paths`, блин. Пройдись sed'ом по исходникам и позаменяй все Pathes на Paths.
Кстати, я бы меню один раз заполнял, в хуке. Здесь этот вызов вряд ли нужен, иными словами.
>> +* (1) Como usuario, me gustaria poder modificar los datos extraidos en
las entrevistas a los postulante para corregir errores o agregar
información omitida.
Me parece que no

> ¿Este lo podemos considerar hecho? Si es así, lo quitaría del backlog.

Me parece que aún no tenemos nada de entrevistas, solo las notas...
Doesn't (navigate :new-organization-page-via-org-switcher) do the same thing as (manage-orgs)?
Please fix indent
alors, regarde sur ta machine, ca ne passe pas
What exactly do you mean by "object equality"?  Obviously, here arrayToFilter[0] !== arrayToFilter[2] but some kind of deep equality check might return true.
for @Portree-Kid this  Fixes #1600.
Shall we postpone this until we have a better strategy for the above issue  ?
Yeah, you are right. Let me correct that in javadoc
Style: dictionary literal
for keyword arguments, don't put spaces around the equal sign (see PEP8).
In fact this is tolerated by pep8 and it is much nicer. i've updated the PR
Thinking about it, it would be sensible to use the python3 compatible StringIO here...
I'll yield to whatever you guys want me to do there...as I had mentioned before, I am not sure exactly what role the second return value from contains() plays.  I looked at all of the patches, and it looks to me like they all return an empty dict for that second arg anyway.

If you want me to return dict({}.items() + patch_cattr.items()) I can add that.  I wasn't sure what the worse thing was to do...just return an empty dict straightaway (possibly has a problem in the future if patches start to return useful data there), or do the dict update (possibly has a problem in the future if the same key shows up in the two dictionaries).
Not sure I get the setTimeout with the extra Date param?
We do in order to overwrite the s3d-actions-share CSS.
@christianv @apex13 Is there a reason for not using the `debug.error` here?
SG_PRINT instead of printf please
I looked to see, and there was no actual difference. I wondered if it had recorded them on different lines because they had been pushed down by the stuff above. Doesn't usually happen though.
needs a space after foreach
Yeah I used to think that too (go to the try/except route) but it makes it more difficult to debug later  when there could be multiple issues that arise. But yes, second option would be great!
Are you sure that the method "canBeUnset()" exists?
Are you sure ``version_compare($install...Version, '5.4.0', '!=')`` has the correct type as ``*`` is not a sub-type of ``boolean``?
Sorry i didn't notice earlier, but the `{` should be at the end of the if line to match the surrounding code
reverted Back - this is actually not affecting
this option will move /usr/bin/firefox to /usr/bin/firefox.main and have /usr/bin/firefox run /usr/bin/firefox.main with LD_PRELOAD
Image resource
I think that it would be enough to do  <code>if (is_string($key) && $id)</code>. If the condition fails for $key it will fail for is_string($key) too.
Would you make this "let's" not "lets" please while you're updating tis file.
The class of the value (NamedBlobImage or NamedImage) is available as the _type attribute of the field.  So if you set field up above (and you're sure that it's either NamedImage or NamedBlobImage) you could just do:

    setattr(obj, pf, field._type(request['uploadfile'].read(), filename=unicode(id)))

instead of the current lines 198-201. Then you don't have to track use_blob either.
Why not move this to defaults? Why do two queries?
You can keep it defined, but don't use it in the code, if you can store it into a variable.
Why do you need this variable? You are using it only once? And you can use jQuery syntax.
`dict()` => `{}`
Indentation. We use 4 spaces.
But why would I use `with` if it works the way I did it now?

So the user profile should work? Because I get: `Generic detail view UserView must be called with either an object pk or a slug.` And I have no idea why.
Write TODO not Todo. :-)
But comment is already displayed automatically? I do not understand what has moving post to the top with displaying comment automatically?
You should leave `:` as part of the string to translate. Different languages display it differently. (French have space before `:`.)
This should not be in settings.
I checked tweepy code, i just had to put a parameter when demanding for authorization url, so twitter_auth.get_authorization_url(signin_with_twitter=True) :-)
Put into one string, not line. :-) Like you did. Is OK now.
t -= time_step / accelRate maybe?
target->CountPctFromMaxHealth(50)
don't change this, because DoScriptText() in CreatureEventAI uses `creature_ai_texts`
On Thu, Aug 30, 2012 at 8:53 AM, bethard <notifications@github.com> wrote:

> In src/main/java/org/apache/ctakes/temporal/eval/Evaluation_ImplBase.java:
>
> > +              "",
> > +              "UseMemoryIndex",
> > +              true,
> > +              "IndexDirectory",
> > +              getUMLSFile("/lookup/rxnorm_index")),
> > +          "OrangeBookIndexReader",
> > +          ExternalResourceFactory.createExternalResourceDescription(
> > +              LuceneIndexReaderResourceImpl.class,
> > +              "",
> > +              "UseMemoryIndex",
> > +              true,
> > +              "IndexDirectory",
> > +              getUMLSFile("/lookup/OrangeBook"))));
> > +    }
> > +
> > +    //add lvg annotator
>
> You should add new values to the AnnotatorType enum, and bracket the
> adding of these annotators to the pipeline with an appropriate check like
> the if (this.annotatorFlags.contains(AnnotatorType.PART_OF_SPEECH_TAGS))check above. That way, only the annotators that actually need these things
> in the pipeline will have to run them.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bethard/ctakes-temporal/pull/1/files#r1496215>.
>
>
Very good point! Do you happen to know which AnnotatorType constants I
should add for LVG, Dependency Parser, and SRL?
I'm wondering if we should check for is_superuser or view_payroll_summary ?

In our case, I think everyone that approves timesheets are admins, but it seems that we need a role for this.
Same for all headers
To disable the `X-Frame-Options` header you need to do something more like:

    configure do
      set :protection, :except => :frame_options
    end
This actually returns undefined, myNum never got a value
I've pulled in his change. Will push soonish.
Not entirely sure what you mean or intend here but I tried to pass a nonexistent windowID and the CreateRequest() command failed.
Who is Scott Chisamore? Did you mean Seth Chisamore?
avoid import *,   almost always, you only have two, AFAICS

```
from statsmodels.sysreg.syssem import Sys2SLS, Sys3SLS
```
It's copied before this is called.
comentario de prueba sobre la linea
In the new pull request, we don't have this anymore. We get it directly in ```currency_formatter```.
删除英文的文档
Typo fixed. Also updated description little bit, hope it is more understandable now.
> the commit one after this commit

Probably useful to document which one that is, so people can quickly redo this script if necessary.
Am I right that only `seed` is actually used below?   I spent a little time wondering what ix, iy, and n were supposed to be if they were inputs to the draw_stamp function, and only after a short time realised that they are being propagated for later bookkeeping purposes only.

I wonder if it might make the structure of the demo clearer for the extra `ix, iy, n` args not to be passed.  This would only require a change to the worker func of `result = draw_stamp(args[3])` on line 872.  Oh, and you'd need to change line 891 to  `im, t = draw_stamp(seeds[k])`.
When SBProfile does FFTs, it requires a size that is a power of 2 or 3 x power of 2.
This is as good a time as any.  I don't think there are any difficult implications to this one, so probably doesn't need its own issue.
OK, will push.
Unusually uninformative TypeError?  (Admittedly I have a hard time seeing how one could reasonably end up in this situation...)
Typo, pushed fix
Does this happen more than a couple places? Perhaps I should add a method to c2g/util.py. 

If you like hungarian notation something like is_storage_local_p(), and if not, is_storage_local(). I personally prefer the latter though it makes the test less Englishy to read.
wanted to assert it stops cleanly, for good measure
Ah I see it's being added elsewhere.
You don't need to support Chrome extensions in this patch, but we should plan support for them. If you could make the help test simply refer to 'browser extensions' that would be great.
controller側で
user_config = UserConfig.find_by_user_id(params[:user_id]) || UserConfig.new
としてるからnilになることは無いかなあと思ってたんですけど、怒られてますか？
This should include the :migrate_data hash on the end.
"A challenge was for example:" --> "For example, one of the challenges was:"
OK - as long as we test the dictionary version :D
Based on the comment above, I'm assuming this test doesn't pass as it is currently written?
Are you sure you are looking at the latest version? Sorry I'm not on the computer. 

El 22/09/2012, a las 7:29 PM, eav <notifications@github.com> escribió:

> In lib/ddp-client.js:
> 
> > @@ -121,7 +121,7 @@ DDPClient.prototype._updateCollection = function(data) {
> >    }
> >    
> >    // clean up
> > -  if (self.collections[name][id] === {})
> always returns false
> 
> —
> Reply to this email directly or view it on GitHub.
>
It seems manualProgression is a subset of readOnly? Only seems checked in autoPlayResolveFinished(), but m_readyOnly is checked there too, so I don't really see the need.

It seems to me both m_readOnly and m_manualProgression aren't needed if you just don't allow double-click-to-play w/ startPlayingFromStart?
On Thu, Feb 23, 2012 at 2:22 PM, Dan Yamins
<reply@reply.github.com>
wrote:
>> +                    orig_result = result[0]
>> +                    assert hasattr(orig_result, 'keys')
>> +                    assert all([isinstance(x, tuple) and len(x) == 2 for x in result[1:]])
>> +                    assert ([hasattr(x[0], 'keys') and hasattr(x[1], 'keys') for x in result[1:]])
>> +
>> +                    return_val = []
>> +                    orig_result_return = handle.update(self.current_job,
>> +                                           dict(result=orig_result))
>> +                    return_val.append(orig_result_return)
>> +
>> +                    num_new = len(result) - 1
>> +                    new_ids = self.trials.new_trial_ids(num_new)
>> +
>> +                    for new_tid, r in zip(new_ids, result[1:]):
>> +                        spec, res = r
>> +                        new_rec = copy.deepcopy(self.current_job)
>
> Agreed.   So the things that SHOULD be copied are:
>    exp_key
>    owner
>    version (?)
>    book_time
>    refresh_time
yes

>    state (must == JOB_STATE_DONE anyhow)
For now :P

> And then what about misc:
>    everything except tid, idx, vals?
How about copying nothing? and just setting these three to the new
values they need?

Also, I like the idea of storing a job_from in miscs.

These are related -- if the banditalgo needs to infer other miscs
attributes by following job_from links, then it should do that.
Much of your configuration is copied from one pipeline to the next.  Consider creating a dictionary variable outside the function and passing it into mainWF() and initAvgWF() as an input.  

This will make your code smaller, easier to understand, and easier to modify.
These lines too.  Stop copy pasting without checking what is changed.
Might also want to return 1 there, as it does at the end of the proc.
It does not need a deep cascading. It is simply failing because the markdown parser puts a newline at the end where there is no newline in the expectation string (the end of the Heredoc string is ``\nEOF``, not ``EOF`` so there is no newline at the end in ``$text``)
Ok, thanks. Now merged!
Without the following line, I get an 'unacceptible request' from conductor, because there is no json return type (the default for ARes)

```
self.format =
```
Looks like amp will be never used.
Same question here as below.
Код 1 предназначен для пользовательских ошибок (http://stackoverflow.com/questions/1101957/are-there-any-standard-exit-status-codes-in-linux)
I would expect that this could be triggered within the XML library itself. For example, that one could say to encode when writing out the XML string.
i also think there aught to be a method in the contact_information class that takes a MockApp object and fills in the fields appropriately.
I don't like that we're rewriting a test and now are xfailing it; we have support for Nightly on Grid, let's just please switch Marketplace tests over to use that browser environment (it'll force us to do a better job to keep it updated, too).  Thanks!
you can use `hasDefinition()` iirc
> sklearn.neuralnet?

neural_net dude! :}
Shall we actually take care here of negative sample weights? I would rather check that no weights are negative in tree.py and trigger an exception if one of them is.
Probably should have ticks here, too.
Have you checked (by looking at the generated C code) that it was not useful to type the input of xrange. I don't know know if Cython is clever-enough to guess that X.shape[1] is an int.
I'd say "Cannot" or "Can't", and "anyway" below. Also I'm not sure about the clarity of this message.
Well, yes, but this also works (despite the warning) and gives me the value I need. The nil value produced by non-existent key is directly checked.
This line breaks the build. I don't want any QT stuff within the Ti objects. The design is to separate the Ti objects from the native layer.
32 for sure
Yes, well spotted!
Shouldn't this switch have a default? Even if the operator shouldn't be any different than all cases above...

That'd also progress towards cleaning the switch warnings in Clang (and future versions of GCC) sooner.
This part of the test is removed, since we can't compare it. Should be enough in the compiler tests, though.
You need to mask cacheId with ALLOCATORID_MASK.  (See equivalent code in ccdserver.cpp(28154)).  Same for all these functions.
On the regression suite? Don't think so.
If this is still `else:` it will never reach the `raise NotImplementedError` in the next block.
There are 2 spaces too many here.
I'm actually ok with putting this in as a dev dependency in another pr.
These are outdated comments, the code is gone. It now installs dependencies by default, and there is currently no option to prevent that, but if someone has a good reason I guess we can add a --no-install
I am not sure what you are afraid of. highline was already there in the ask method, used by vimgolf setup so ran once at least by every golfer. Anyway, i don't know highline status, i can only report that it's used in my favourite tool for building command line ruby tools, that is visionmedia/commander, and that commander right now is all green on travis-ci. Looks good to me.
http://travis-ci.org/#!/ggilder/commander
Please use the following rather than `strtod()`:

```c
parse_value (token, &beancounter.values[HELD], DS_TYPE_GAUGE);
```
Why are we using this? We are automatically pulling in FB profile image URLs
spaces should be outside the braces, not inside them: ``if (....) {``
@elicochran Can we fix the spaces here?
I'm also wondering why we need the `!important`
Two reasons: one, the image doesn't exist on most miro community sites, only on the main site; and two, in the grand scheme of things, this didn't seem like a blocking issue, since this is the same url the old code used.
I don't think this is really an error condition.  ```vprint_status``` would probably be better
Should be ```OptPath```
We should tell the user what to do to fix this or at least provide a link to explain what to do to fix this.
Is the intention to do integer division?  Preferably use ``from __future__ import division`` at the top of the file, and then ``/`` for float divide or ``//`` for integer divide.
another one to add
````
// Wrong
- (void)finishedProcessingWithString:(NSString *)string andImage:(UIImage *)image;

// Correct
- (void)object:(NSObject *)object didFinishedProcessingWithString:(NSString *)string image:(UIImage *)image;
We usually put variables like this right before where they get set. It easier to see all in one spot if the code looks like:

```python
limit = forms.PAGINATION_LIMIT
if form.is_valid():
    limit = form.cleaned_data['limit']
```

as apposed to everything being separated. Defining all your variables at the beginning is nice but in this case it makes it less clear.
I think this just makes it read-only. The MDN says it can be used to modify an existing property: https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Object/defineProperty
Why not `positions.push(position.x, position.y, position.z);` instead?  Is that better or worse JavaScript-wise?  CC @shunter
It's probably not a big deal but the assignment wasn't being associated here. I'm not sure why.
Mobile Web follows the Android behavior.
Apologies, you're right. I was confusing myself with setPropertyAndFire... will fix this
Is there a better way to disable hooking up the module listener without changing the signature of these methods?

We can't change methods that have the '@module.api' tag on them since it will break external modules.
I believe there is an ````@apinotice```` tag, but I could be wrong.
don;t edit bootstrap css, this file gets updated wholesale and your changes will be overwritten. any changes you want to make have to go in application.css
Why are you writing 2*45 instead of 90?
It seems like the use of blank lines is inconsistent in this block of code. I can't figure out what rule you're applying.
See [da87dfa](https://github.com/seiffert/ECVagrantBundle/commit/da87dfa2c77f30cae8bb377c8b508e578e5e3c4c)
I believe you'll want `t.datetime` instead, no?
use `$(D pathname)` instead of pathname
I actually would have argued that the intent was clearer with `T.min == 0`. It's also less error-prone, since the whole point of the assertion is to make sure that `unsigned` is working properly, and `T.min == 0` is far more likely to fail than `T.min == T.init` if there's a problem (e.g. the template constraint gets changed for some reason).
I can leave it, but it's a pointless alias, and we generally try to _not_ have those. The _only_ reason that I see to leave it is that it will break some code to remove it, and we're trying to avoid that, which may make it worth keeping, but ideally, it wouldn't be left here.
I think [BVAL-198](https://hibernate.onjira.com/browse/BVAL-198) would help here. I'll try to create a pull request for this.
i don't understand the question. str(self.toolchain.variables['CC']) ? you could write a tiny funrciotn (get_variable) to do just that.
If it's safe to do so (i.e. nothing else in `chef-client` uses this), this should probably be `nil`. launchd doesn't use pidfiles.
:features does not exist
Why not just have the unpacking inside the `.each`?
m.up/right/left/down
why this ternary operator ? it is not needed IMO.
We should use hbs instead of hb I think because hbs is a possible extension for handlebars template files.
another blank line here; two blank lines between methods in a class
These lines exceed 75 characters wide.  Please follow PEP 8.
yes, camelCase and append underscores if necessary please
Really ?? getAttribute(MessageHeaders.CONTENT_TYPE)

We don't couple the name of a header to the name of a schema attribute any place else.
Yes, but as I said in my previous comment let's look at it as an additional feature. In fact I say raise a JIRA and resolve it when this does so we can have a record of such improvement.
I will say I kinda liked the original implementation - rolls off the tongue nicely and the schema is very simple.

I do find the XML transformers extremely confusing with all the options (if the in is x, the out is y, unless some factory is defined, and it's not a leap year, etc, etc). Gunnar re-wrote the docs to try to make it clearer but it's still very confusing.

So, we can either leave it the way it is, or use
    <syslog-to-object-transformer result-type="map" />

Mark, if you want me to incorporate the Tuple instead of List, let me know.
An ammended commit (or a rebase) would be appreciated, to keep the history clean.  Thanks for these doc improvements!
Oops, trailing whitespace. will deal with it before I merge
, aparte de la página de inicio de este tutorial,
Accidentally carried over || arg === '-k', this should be deleted.
btw., cant see the difference here, what did you change?
Sorry, editor formatting.
NIT:
Why random time?
If you want to assure that they were executed separated I think is better use st. like 10*i
Haha nice. Illl add that fix to my pr. 

Sent from my digital shackles. 

On Mar 20, 2012, at 12:21 AM, Craig<reply@reply.github.com> wrote:

>> @@ -31,6 +31,10 @@ PATH_TESTS_CONF="$PATH_INTEGRATION_CONF/tests.conf"
>> 
>> PATH_ENV_CONF="$PATH_INTEGRATION_CONF/env.rc"
>> 
>> +# Package repo directory
>> +PACKAGES_DIR=/var/lib/packages/debian/
>> +BUILD_DIR=/tmp/build/
>> +
>> # Passwors used by devstack.
> 
> +d
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite-integration/pull/17/files#r578381
It is not the goal of this test.
As you see here, 
* we need the self.opts
* we need the super().__init__ being done later
 * again this is needed because the *fields = self.get_fields()* in super().__init__ needs the url field in the base_fields

This still requires this metaclass stuff..

Another way of achieving this would be something like
**__init__**
```
self.opts = getattr(self, 'opts', self._options_class(self.Meta))
```
then the self.opts *could* being initialized before the super()__init__ would do... but -- think thats no nice way to solve this.
Perhaps this can work well in the special case where n_samples >> n_features >> rank. In this case the low rank approximation obtained via fast_svd could be exact, by requesting n_components = rank. If the approximation obtained by fast_svd is not exact then we miss the smallest singular values which make the largest contributions when we apply the inverse matrix.
Why not:

```scala
customBucketMap.values.collectFirst { case files if files.contains(file.getName) => true } getOrElse false
```

Or:

```scala
customBucketMap.values.exists { files => files.contains(file.getName) }.isDefined
```

You could even skip the .values call altogether (which can be expensive depending on the backing data structure):

```scala
customBucketMap.exists { (id, files) => files.contains(file.getName) }.isDefined
```
zaim did dont know why??
If the menuWidth is set to a number and is bigger than 0 then overrides the original value.
we can’t afford embedding fonts in CSS for otoro.
Please give us a list of the fonts you want and we’ll include them in the platform.
keepFocus is set to true by click() method when someones clicks inside the menu, but not inside any menu item (so we need to keep the menu opened)
final as well
`django.core.cache.cache` is an instance of a cache backend. I suspect Fred is right and this will fail. Wrap in `try...except ImportError: cache = None`?
Лучше избавляться от генерации функций (Event каждый раз будет создавать одну и ту же функцию с тем же абсолютно телом) аналогично с leaveMark.

Можно использовать чисто функциональный подход `setLocation(context, gps, name)`
"Turn the 'Highlight selected element' switch on."
This is not actually an Array, so forEach is not available.
Could be done with collection, but I simply wanted to interact with the files - so I ensure that file is sitting in the proper location.
No, it's fine the way you have it, IMO.