labofsahil/github-event-dataset-2012 huggingface dataset:
When deprecating stuff it's good to tell which version the deprecation occurs in (0.12 in this case) and in which version backward compat won't be maintained any more (0.12 + 2 == 0.14 in this case).
please fix the docstrings
single feature representation?
постійно були проблеми з іменами класів, помилки вилазили в несподіваних місцях, вирішив проблему таким чином
Possibly we could generate free ports here.
That's fine. We're forcing a "check and reevaluate who's master". It has been the "technique" since the inception of HA.
what about: measureDistance.setChecked(measurementActivity.getMeasureDistanceMode())
run preproc -> datsink to some location
run l1 takes input from the location output by preproc and saves to a new output location

l1 should not rely at all on the preproc's working directory.
Maybe you missed my Skype message, but I said we should probably have Ti.Buffer exist so it's seen in javascript.
This method is not necessary if you're not doing anything. The default method will be called anyway.
That's project official name
self.log.exception('Caught exception: posting notification to object (%s) with method (%s)' % (_observer[0], _observer[1]))
Is this the best place to add this variable? It seems like it could be defined at the top.
I don't know about this. I've only benchmarked ag on dual-core CPUs and single-core CPUs with hyperthreading. More cores might yield better performance. On Linux, the performance impact of extra threads is minimal. On OS X, performance doesn't degrade significantly until there are 8 workers on a dual core machine.
So, some uses rely on it failing and throwing, others on it returning false. I won't change that behaviour now and adding an error message to the specific case when the callee returns false on the caller is not a good approach.

I'd open an issue on Jira to fix that
Will check all constructor calls for null like is being done for id parameter.
@peterbe In this function I am assuming versions is None hence lines 1152 - 1156 is executed without a condition. That should ensure that version is never None. Thing is, I suppose a product can be loaded without a featured Nightly but then I wonder, what would be the fallback as this report is only for Nightly and Aurora builds.
This should probably be in utils
This silences any exception thrown by get_download_url
Hi Patrick,

That's correct, we're no longer using a third party library.

Formerly, we used Kevin Schaul's Share phonegap plugin. Hence the licensing
blurb.

Now we use a custom plugin.

Best,
Shea


On Mon, Mar 26, 2012 at 9:33 AM, Patrick Hayes <
reply@reply.github.com
> wrote:

> > -of this software and associated documentation files (the "Software"),
> to deal
> > -in the Software without restriction, including without limitation the
> rights
> > -to use, copy, modify, merge, publish, distribute, sublicense, and/or
> sell
> > -copies of the Software, and to permit persons to whom the Software is
> > -furnished to do so, subject to the following conditions:
> > -
> > -The above copyright notice and this permission notice shall be included
> in
> > -all copies or substantial portions of the Software.
> > -
> > -THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> OR
> > -IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> > -FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> THE
> > -AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > -LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> FROM,
> > -OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> IN
> > -THE SOFTWARE.
>
> Actually, maybe I'm confused. Are you removing an existing third party
> library? Then this is fine.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/wikimedia/WiktionaryMobile/pull/19/files#r603989
>
No Prob. I put it there because everything else i2c related was also there.
It is far from being easy to catch, but I'm throwing an exception from `getRulesForLocal` (which is auto-generated).
Should I instead print a warning message and return a fallback method?
I may either return `en` method and consider that it is always defined, or,
return an harcoded method like this:
`function (n) { return "other"; }` 
It just says that whatever number is, use the generic `other` form.
> I could simply store in the clipboard module and don't expose it, if there is no better options.

If we don't have any other use case and can't fine a good place for it it's absolutely fine to keep it in `clipboard` module until we discover another use case or a good place to put it to.
Changed with $reader.
Please use `escapeshellarg()` on arguments - and same in the test to build your expected strings, for cross platform support.
Missing bracket from this code example.
This belong to Javadoc
Should this link to # rather than to / ?
@christianv Good to know, thanks.
We limited the truncation to 60 because of the url length, now twitter is shortening automatically the urls, so 120 "should" be fine.
Global variables are bad
Yes sure, but it's being done in this way in other wbem modules, so just testing the FileDropper mixin. About this modules, I'm more worried about lock issues I'm not being able to full understand / avoid :( Anyway, I'm going to fix :)
Because if I find this, I get to bail early?
88103 : SSH Tectia Pre-authentication SSH USERAUTH CHANGE REQUEST Account Password Change Request Remote Authentication Bypass
Actually, this needs to get normalized before it gets passed to report_service(). It looks ugly on the console output
Again, this breaks style with Django's conventions (i.e. https://docs.djangoproject.com/en/1.4/intro/overview/#design-your-urls).
newlines added
note to self, learn to use PEP8
I don't think that's the right target, this should be failing. Hmm.
I believe the doc bug still exists, unfortunately.  Poor Kristian getting CC'd on this pull request. :)
Thanks for adding the extra tests.
Its checking for an orthographic frustum. Yes, it is needed right after the morph. I ran into a problem when testing where the mode was 2D but the camera still had a perspective frustum.
I forgot about this hack, I'll revisit and spend some time on it.
It's pretty awesome to actually do a `TODO`.
这一堆mouse的应该和Key的合在一起。
The mutating hash in various methods seems wrong. Is this a pattern established by `ActiveMerchant` or Litle?
This will work.  FWIW, UIDevice has responded to userInterfaceIdiom since iOS 3.2.  But I support completeness in the pattern, if for no other reason than to educate neophyte iOS developers to come. :)
On Android, there is one more method - getActiveTab()
Looking at the PR, a lot of complexity appears to be injected for a temporary fix.  Unless there is a good reason, I think the stack should be associated with the activity itself and not a window.  If we look at the relationships, I don't think the individual windows should even need to be aware of the stack details beyond calling the equivalent of pop/push for the stack (IE: activity) they currently are on.

Maybe there are some architectural details that are getting lost in translation.  @hieupham007 perhaps we need a chat tomorrow between the 3 of us to clarify the architectural approach taken
Should read "Number of bytes to read" ?
its a trick, do you have a better way to force text cursor to end of input line?
interesting - so do these no longer compile, or are they just potentially going to leak (over reference count) ?
Could be rewritten to be `if(value)` since null is falsy.
Мы *негласно* договорились, что будем два подчёркивания ставить для приватных штук.
попробуем без копирования?
> I think TDPL is quite clear that there are no such things as "unmoveable" in D.

If it states so, this is good. But isn't it about GC heap? (I really never read)

> I never heard of anything broken with algorithm.move.

My comment above? )
Again, no comma.
Oh, wait, actually, line 2833:
`if (hasSlicing!R && hasLength!(typeof(range[0 .. n])))`

However, this is fixd by @jmdavis' slicing fix, so LGTM either way.
I see your point, but I still don't like the fact that if I do the following:

    SomeRange foo, bar;
    auto a = refRange(&foo);
    auto b = refRange(&bar);
    RefRange!SomeRange x;

then the two lines

    x = a;   // Only affects x
    x = b;   // What, now b is assigned to a?

do two completely different things. I'm inclined to say that you should just drop support for slicing in RefRange, if this is the only reason for this behaviour. (Anyway, isn't it a bug if an algorithm does stuff like `r = r[n .. r.length]`? At least from the `std.range` documentation, there doesn't seem to be any requirement that `opSlice` return a range of the same type – just that it returns an input range.)

At any rate, there needs to be a null pointer check before `rhs._range` is dereferenced.
I can not see any benefit in doing this, because I can not imagine any overloads allowing different lengths that would make sense.
This way, you get at least a better error message. However it's not needed to fix #8284 and should be in an pull request on it's own.
Sorry this appears to have been attached to the wrong line, block I'm questioning is at line 1102 with comment "needed for do_POST_Document_bulk_docs"
s/a R extension/an R extension/
I don't know, but isn't the class also of interest?
better to wrap bundle == null into a isLoadedFromBootstrapClassloader(bundle) method?
Also, `return` in the middle of the two `for` loops is suspicious. I imagined you wanted to perform that action for all the `bindingGroups`?
Separate with a tab.
No worries - 3.5 users can just stick with the Jurassic based compiler.
pass che taghiri to code ha dai !
rasti to chera bidari?
I know it's a ton of code, but given that especially, it might be worth throwing all of these onto their own dedicated lines... if only for readability and especially diffs over time (adding/removing additional system types).
with up2date, I had a similar bit  of code. One small problem was when something broke elsewhere, this field was never populated, even
in released rpm versions, and no one really noticed.
URL, welche auf eine Firma zeigt, bauen lassen
No, in fact that's the problem. It just indicates that it expects a value in the range of 0-100, which it then converts to a decimal 0-1 for the standard CSS rule.

@thomas-mcdonald is saying he prefers the mixin to take the same value as the CSS rule, requiring no conversion.
Your warnings are backwards.  The string comes first, then the deprecation.
Yes, that's short and simple..
Previously the fonts of these elements (other than body) haven't been changed. In particular, this affects the comment form in the idea page.
La longitud de estas líneas debe coincidir con la longitud del título, es decir te sobran 6 #
`make -j build`, in case I change the default to do build-and-test in the future.
this does not respect the CS
I think this is an issue with using ruby 1.9.  It looks like Base64 no longer has the b64encode method and now has an encode64 method.  I'll add some detection code in there to that it works in both.
I'm going to have to check this to be sure, but I think blank subject would return from IMAP as "(No Subject)" .... If it doesn't I can easily set it to default to that
Can we call the setting REGISTRATION_ENABLED
Okay guess that makes sense actually, just surprised to see .META['QUERY_STRING'] is actually a thing
Ah - this pull request also contains all the changes from the previous pull request.  Hence, I can't really merge it.

This is why you need to use clean branches for separate pull requests, not build them on top of each other.
d = dict(foo='bar')
    d['hi'] # exception
    d.get('hi') # None
How did I not see that? Weird.

At any rate, I would say let's simplify this further. We could make the default `event: 'click.easytabs`, and then the user has the power to change the event namespacing if they want as well.
since as and asb get closed over and used in the closures, it's probably worth using const here.  (When there's no closure, I presume it's easier for the JS engine's analysis to see the variable never gets assigned to, but it might not do the legwork with closures.  Although maybe it does.)
Is there any sort of documented standard or best practice regarding this that we could follow? I'd like to set a good example in the skeleton.
the EventEmitter that you're talking about is a js file, I dont think that can be linked..
Remove dead addition from source
".... create it in *the* KrollRuntime thread".   (I point it out only because you say "the KrollObject" earlier, so it sounds weird to not have the "the" for KrollRuntime)
Done! Can't believe I missed that :-)
the more i think about, the more i tend to say - not really, but i need somehow to define what users can do with it - if it's a "normal" workflow i know (from WF - only one user can access the object at one time)

on the other hand we could also have just a list of who approved it already and once it's aproved by som person or by some number of people it is finished.
Add comment like `// convenient way of returning the builder`?
Don't we need "handledirsep=true" here, for Windows? Or does mapper work differently from globmapper?
typo here, it should be 'vertical';
The simplest check could be
```php
if ($role->id < $role_id) {
    // check if no role with given 'id' already exists
    // ...
}
```
That should keep the PKs clean, but would reduce the use case of this functionality to Moodle instances that have deleted roles (resulting in PK holes).

I don't see a normal use case where this code will work then.
Consider simplifying this block:

```php
$server = isset($config['options'])
    ? array_replace_recursive($defaultServer, $config['options'])
    : $defaultServer;
```
Could you investigate this and make sure this is the case?
А у вас не наблюдается бодыль с меню кнопки обновления, що находится на панели скрипта? Иногда не хочет пропадать.
Forgot the anchors:  (^local:\/\/)(.+)(\.js$)
We might not always have a German locale for a project, but en-US is almost always guaranteed; can we just leave this dynamic, for now?

e.g. can we grab the locales from the footer select pulldown and check that they show up in the URL field?
Please don't use a underscores for variables. In C# this means that the variable is available over a proprety. (getter)