labofsahil/github-event-dataset-2012 huggingface dataset:
If we can look for the header locally, one way to get it but avoid a memcached dependency (if that's undesirable) would be to use a sub-formula to download memcached and copy the headers from there.
если инстанс интерфеса то getName
We usually put a space between the `version` fields and the `depends_on` fields.
as @Vincent-Michael stated, 101 is wrong, blame SpellWork for always adding +1 to the basepoints
Can we not use one of the meme gifs? Those are huge! Like sure use an animated thing to demo that (although you should also have a static one to demonstrate that if it's just an image you can just reference the asset directly; no need for sprites).

Also: not clear this is the best place for this. This isn't fundamentally part of Karkat. I would create dialogs/karkatDialog.xml and move all the bg stuff into there. That's the better pattern in my opinion (because you can purge it by just deleting all the dialogs).
Лучше сделать socket_handler обычным методом, в потоке запускать self.socket_handler, self в args не передавать.
К слову, args принято передавать как tuple а не list.
Aha!  Cool.
why do we set this attribute here ?
That is how **Q** behaves and in fact it's pretty important let me give you an example why:

lets say we you have two promises `a` and `b`. Two handlers `h1` and `h2` are registered on a promise `a`.
Handler `h1` has side effect of registering `h3` handler on the promise `a`, that attempts to resolve promise `b` with `foo`
Also `h2` handler attempts to resolve `b` but with `bar` instead.

One would expect that `b` is resolved with `bar` as `h2` was registered before `h3`. But if you do not guarantee FIFO order
`b` will be resolved with `foo` instead.

In real life cases are obviously way more complicated than that and reasoning about them becomes way harder if such order is not
guaranteed.

Not to even mention that promise resolution by itself is designed around "first come wins" bases so sticking to that design is highly
important.
`#index` was the fastest way to figure it out when I measured different approaches. I agree it's cleaner with a Regexp though.
Thanks Virgile,  
I looked briefly through it and don't see anything to "complain".  It might take a few days until I have time to look at it more carefully, and to play with the tuning parameters for the large sample case.
The added variables aren't really params...so I called them x_added.  The params are now called x_params.  Together they make x_full.
public methods with a name starting with a underscore ? what is this convention ?
Since all ``exit_action``'s are same instance, their visibility will be linked together.
I like `next_page` more than `redirect_to`. ;-) But it is discussable.
Personal preference - this is exactly the use case for Optional in google collections.
````java
private Optional<Attachment> getAttachment();

Optional<Attachment> attachment = getAttachment();
if (attachment.isPresent()) { ... }
````
If we start doing this by default everywhere the presence of null checks should go way down.
Correct me if I am wrong but these surrounding whitespaces seems to be not in compliance with Symfony Coding Standards: http://symfony.com/doc/current/contributing/code/standards.html.
I don't think you meant to delete this file.  There is a simlink from devel/credo.txt to the now non-existant docs/credo.txt.  Probably you meant to move credo.txt to either doc or devel.
I'm not sure how useful safe really is.  I was thinking that if we know that the parameters aren't changing, we can just use the previous "current_val" rather than reparse the config.  The code for this is in BuildGSObject:
```
    # If we have previously saved an object and marked it as safe, then use it.
    if 'current_val' in ck and ck['safe']:
        #print 'current is safe:  ',ck['current'], True
        return ck['current_val'], True
```
But this is almost certainly a case of premature optimization.  Parsing the config stuff is extremely fast compared to most of the rest of the things we do.  So I should probably just get rid of all the safe stuff to make the code a bit cleaner.
I like that a lot!  I think that covers every case of practical importance and it will be quite straightforward to document.  If it has a side-effect of making users actually ponder a very simple physical model of what is happening at the detector, all the better...!
Something else I've been thinking we should change.  I think we shouldn't be clearing the image before drawing -- either here or in the regular draw command.  For two reasons:
1) It is inefficient.  We already set all values to 0 by default when we create a new Image.  So clearing it again is usually a waste.
2) Sometimes we will want to be adding an object to an existing image that already has stuff on it.  e.g. when we are testing blended objects.  To do this now, we need to create a temporary image, draw the second thing on that, and then add that to the original image.  It would be simpler to just apply both draw commands to the same image and have that mean add the photons to what is already there (if anything).
well there's already a GetImportedVoucherRequest and it only has an `int ImportedVoucherId` property. i used the GetVoucherRequest since it takes a code and secret.
Much nicer to do $.deferred here than ad-hoc loading.
Yes, we can do apt-get install python-numpy to install.
agreed.  now call it "obtain" here, consistent with brooklyn provisioning location terminology.
Should we enforce the "only settable once, before catalog is loaded"?
it's a way to show up that you're not using that variable
I'd rather expose this more explicitly in RSpec::Core - something like RSpec::Core::BacktraceFormatters. WDYT?
Why use the ``escape`` filter?  Django will escape by default.
This feels like the wrong folder.  What is this js file for?  It looks like it's a 3rd party script and so shouldn't be in the ``oscar/js/oscar`` folder.
What do you mean by this?

`evt.data` is an object, not a string
Скобки вокруг new Exception лишние.
@stupchiy  I added some screen shots 
https://marketplace-dev.allizom.org/app/hypno/ 

If you want to use a separate app in this test you could add it but not as a global app for all tests
@phase:type@の記法により
owner and author seem like different concepts to me; a owner can be reassigned. Sure you don't want to have a separate `IsAuthorOrReadOnly` permission?
right, better update those headers then
I accept that.  The reason I wrote this is because I saw all the nanny code surrounding `np.argsort` and hoped that there would be something faster than that.  I can certainly improve the documentation.
`hashfn` = > `hash_function`
Is there a reason that this is done for a list comprehesion, rather than numpy?
This is a terribly slow way to find out if a player is banned or whitelisted. What if there are 10000 players on the whitelist? The API needs to be changed to use Set, backed by a HashSet. Then you can do a contains call and have it be O(1) vs the current O(n) lookup time.
Good point. I'll adopt your idea.
need to decode like the other args
just noticed that the return value of this function is not being used. I am also wondering why for all the setEventHandler functions, it returns NATIVE_ERROR_NOTSUPPORTED. Need to discuss with Davic C.
This looks like it will break a lot of ignores. For example, if I'm searching directory "blah" and "./blah/src/.agignore" exists, this code will cause ignore_filename to be "/blah/src/.agignore". The `fopen()` will fail and ignore patterns won't be loaded. Or worse, the wrong ignore file could get loaded.

I should probably rename `ignore_filename` to `ignore_path`.
it should be `level 0` or `0th level`
Is this a hard limitation somewhere else? Physically, this only makes sense if each row is 2^32 bytes long... ;)
Error checking added for row case too.
scheduleadmin is in /opt/HPCCSystems/bin/
I think having capacity in the name is confusing - it makes it sound like it is a size rather than a count.   maxRows() would probably be better.  @jakesmith
I couldn't find a way where it'd be NULL, either, but half of the uses all around the code do use it, and I decided to be in the safe side.

It does look like too much copy&paste, and when the API changed mid-way and only the new uses got right...
Not sure I follow what you mean
Variables have been removed.
I'm not clear on whether you are agreeing it is looking up in the transaction here too early or not from that reply..
I expected it to lookup in prepare() I think.
There are three states: paused, stopped and running/active. Thus, I assumed "active" is when the queue is neither stopped nor paused.

Since paused and stopped states only occur when *all jobs* are on those states, I assumed "running" is when *any* job is in that state.

This is not necessarily the right assumption (and seem to break the logic).
The if(stopping) test below means it has no ill effect.
ups. was correct before you told me to simplify, now indeed wrong :-) thanks, fixed
I'm struggling to see what's actually changed in this diff.
La consistance est en effet de mise.
Si ça ne dérange personne, on va avec les conseils de l'académie (cf. http://fr.wikipedia.org/wiki/Usage_des_majuscules_en_fran%C3%A7ais#Accentuation_des_majuscules_et_des_capitales).

OK ?
Reverted to distinct available_* attributes in 786bcfc743
@ninfaj Where is this fixed?
This could be module level, would save redoing it for every table.
Change to `Android/1.0`
In python, ````not ''```` evaluates to False, so the second case in this if is redundant.
I agree this fixes the sort order, I wonder if generally we should sort in templates or controllers. Lots more flexibility with new Django.
86672 : QNX QCONN Unauthenticated Remote Command Execution
Should be ```icmp_listener``` for consistent style.
if res and res.code == 302
I just looked up the correct meaning of the word "warp" in a dictionary. This already implies a distortion which is not covered by a similarity or affine transform which is your point here, isn't it?

I'll then move it back to geometric.
Username has a `u/` in it? I don't get it.
Good catch - instead why don't we do this:

```python
for i in xrange(10):
    code = get_random_string(32)
    if not UserProfile.objects.filter(confirmation_code=code).count():
        break
```
Assets/Imagery/earthspec1k.jpg only makes sense as a default if water is only being used for a full ocean. If you just wan't a water texture on some primitive, you may not wan't to have to specify a specular map
@pjcozzi and I see things the same way.  Just because we can supply a default value, doesn't mean we should.  In this case, I if someone meant to pass in a time and forgot to, having `getPoint()` default to 0 would mask a bug.
I think this one is just the diff looking odd maybe?  The .pdb is still in the list, and Azure.psd1 is still there (which is the cmpD1930B...) it just moved to the top of the file for some reason (and of course has new GUIDs).

> -----Original Message-----
> From: André Rodrigues [mailto:reply+p-1191222-
> e399ed7b6e6bad773fe0200e1abb2756d62f2119-320265@reply.github.com]
> Sent: Wednesday, July 18, 2012 12:52 PM
> To: Dino Viehland
> Subject: Re: [azure-sdk-tools] Dev (#283)
> 
> >                  <File Id="fil41AB56A0AA56DEC0BB1F1CDC8C66DCDA"
> KeyPath="yes"
> Source="$(var.sourceDir)\Microsoft.WindowsAzure.Management.pdb" />
> >              </Component>
> > -            <Component Id="cmpD1930BE03EC593478EB6B8DB607DE198"
> Guid="23B92C5B-E65B-491C-9DB5-69FC56CF946E">
> 
> For some reason this is being removed and it shouldnt.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/WindowsAzure/azure-sdk-tools/pull/283/files#r1191222
> 
>
remove extra line (should clean up this function in fact)
*eventing. :)
They extend different classes. So no.
if ([self viewAttached]  && [TiUtils isIOS5OrGreater] ){
    TiThreadPerformOnMainThread(^{ [self view].accessibilityElementsHidden = [TiUtils boolValue:accessibilityHidden def:NO];},NO);
}
[self setValue:accessibilityHidden forUndefinedKey:@"accessibilityHidden"];
Instead of calling fire change, we should just see if the new value differs from the existing value.  If so, set the value and fire the event.  In the current form, the map is updated regardless of whether the value is identical to the current value or not which we should not be doing.

A note on checking to see if the value exists will put on the PR itself and not a specific line.
For clarification purposes, I am saying remove whitespace being printed to the html output (i.e. the '\t's in the strings), not the whitespace in the PHP file itself.
OK; misread the code at first. Thought it was KindOfClass instead of MemberOfClass. That clears it up.
I mean even within the control/theme. For example, were 'yellow' always a default, a label would be useless because by this statement, you'd have yellow text on a yellow background, making it illegible.
> what does it do something that starter-kit / emacs-live don't do ? 

Regarding starter kit, this was not done by it.
Regarding emacs-live, i do not know because this pull request is prior of our use of emacs-live.

> does the +1 add something more ?

From http://emacswiki.org/emacs/ParEdit, this is the way to activate paredit.

``` quote
To use ParEdit with Emacs’ Lisp modes, add the following to .emacs:

    (autoload 'paredit-mode "paredit"
      "Minor mode for pseudo-structurally editing Lisp code." t)
    (add-hook 'emacs-lisp-mode-hook       (lambda () (paredit-mode +1)))
    (add-hook 'lisp-mode-hook             (lambda () (paredit-mode +1)))
    (add-hook 'lisp-interaction-mode-hook (lambda () (paredit-mode +1)))
    (add-hook 'scheme-mode-hook           (lambda () (paredit-mode +1)))

```
Оно в `text`.
Please remove the comma.
The rationale is that when you re-use a variable as such, by declaring it, and then assigning to it, you don't "special case" the syntax of the first assignment compared to that of the others.

That said, I also agree with your second argument, making mine moot. I'll change it to the way you suggest.
Given that the signature for `replaceInto` has 4 arguments, whereas `replace` has 3, we _could_ use the same name without any ambiguity whatsoever.

The real question is about the general case though. If we use `replaceInto`, then that would imply that functions like this in general should have `Into` in their name. Doing that does increase clarity, but it also creates a bunch of new function names where they could almost always (if not always) just be overloads of the functions like them which don't write to an output range. There may be some cases where it would be required though (particularly if the non-"Into" version has default arguments), so I don't know. I'm divided on the matter.
this especially excites me
Add an empty line before `def patch_step`, to separate class docstring from the rest.
Well, a `get_variable` function would hide the `flatten` part, so I'd go for that. The splitting business is a lot more sensitive to breaking.
create #500 to track that
hmm, I thought previous error was about the placement of the function brackets. Good to fix!
should we have this configurable like the standard symfony router has it?
`.change` -> `.on('change'`, also missing a space after the function()
Yeah, giving the fact that BF is set in one method and accessed in another NPE is possible if invoked outside of Spring's lifecycle
If we're initializing the field, (because we want tests to pass without afterPropertiesSet(), this code is not necessary.
there is no any parsing of this sub-element in the **JpaInboundChannelAdapterParser **...
make it not greedy, especially since we're doing a multi-line regex match
are these subject to overflow without the length specified? my current clone of v8 doesn't even seem to have this method
I think this should be phrased differently.
better to say

```
if @winner.nil?
```
Очень хитрая сортировка  ;)
should be `(!this.watoken || !this.pbk)`.
@gvangool `[[ "$OSTYPE" == darwin* ]]` is the proper way to check  for OS X in Zsh, but this is not Zsh; it's Python.
Because when we're calling save(), we need to know if the comment exists in persistence to check if it is edit or create that is the required permission.

The comment could already be scheduled for insertion, but not be persisted. We dont want that. (If save is called after the comment is already scheduled for insertion, we still want a create permission, not an edit one)
self. !!! nice!!
Without the shade plugin, the META-INF/services files aren't copied and merged properly
缩进有问题。所有缩进都是4格，而且是使用空格填充
yeah true...
Should this be the string, or should we be referencing the GuestStatus voodoo?
I don't want to use a function starts with '_' from another file. Also I don't want to change general behavior of RELEASE_VERSION variable.

But if you insist on using like that I can make changes (I can rename _get_git_version to get_git_version too).
The latitude component when searching by coordinates.
To test that the default config is what I expect it to be.


On Mon, Nov 26, 2012 at 4:40 PM, ovaussy-virtuoz
<notifications@github.com>wrote:

> In test/dml_config_test.js:
>
> > @@ -0,0 +1,7 @@
>
> What's the goal of this file ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/VirtuOz/node-dynamic-module-loader/pull/1/files#r2229694>.
>
>



-- 
Kevan Dunsmore | *Director of Engineering*
*
*6001 Shellmound Street | Suite 500
Emeryville, CA 94608
Office: 510 379 7714 | Cell: 512 422 9351

Email: kdunsmore@virtuoz.com

www.virtuoz.com

Follow Us: Facebook <http://www.facebook.com/VirtuOzVirtualAgents> |
Twitter<https://twitter.com/#!/virtuoz>
 | LinkedIn <http://www.linkedin.com/company/virtuoz>
Can you please break these out into multiple test methods?  If this `assertRaises` failed, we wouldn't know that the next `assertRaises` also fails (or passes) until we fixed the first method.  It's generally a good rule to keep one assertion per test method -- or at least one concern.  Here, `all()` and `exclude()` are two different completely different methods that just happen to raise the same error.
**Error:** Expected "if (...) {\n"; found "if (...){\n"
Indeed, I have broken that functionality. Thanks for pointing it out, I'll fix it.
You can use either &#169; or &copy; for copyright sign in HTML representation.
"Instantiates the private SharedPreferences collection with the given name and context."
This works on CentOS but needs testing on Debian at some point.
Is there a reason that these are processed in a specific order? If not, could we do a fast enumeration? (For x in...)
All of these should be `verbose`.
Since you call get on the next line, it would be better to use entrySet() here.
`raytrace`?  Is this to avoid confusion with the `trace()` call ?
Woops. I was debugging the other problem and left this in.
This was more because we made on (id, type) we wanted something like an onAll(type)
It tests how we deal with entries with multiple requests, in this case how the body is shown while content-tracking is off. It could be tested without https, but I don't really know of other scenarios leading to several requests on one entry.

The test makes sure that we retrieve the body via GetResource, but that we only show it for the correct (last) response.
https://github.com/phpBB-Social-Network/phpBB-Social-Network/commit/b691fdf2dd6d221c55278d72219e6184a6a05723 was merged already. This is the reason why this PR cannot be merged automatically. Try simply `git pull upstream develop` on your track_fix branch. Then push changes and we will see.
@nukulb @kwallis Do we want to use the same error message from Snarf? (see below).

    [ERROR] Invalid config.xml - failed to parse the <content> element(Invalid source or the source is not specified.)
Not really.  Can just leave it as is.  More of an FYI if anything
actually now that you mention it, I think I see why it's a pointer now since it's being used in a thread it has it's own heap, and it won't be accessible that way, false alarm :)
I just get it , you are creating a new folder called automation.

I was getting mixed up with this.
https://github.com/blackberry-webworks/BB10-WebWorks-Framework/blob/next/test/functional/automatic/blackberry.app.js

How do you feel about providing a checkbox for ui-agent in the automated test, instead of entire new SpecRunner?
Also needs unit tests
I made a change. 
Before I customized the test callsPerMethod was equal 2 then it was appropriate. Thanks.
can you use ``form.children[field_description.name]`` ?
Well, please fix it in the PR so that it can be merged
the same for propel I haven't seen any option, it will be passed by the options form in the generator yml file, same in ODM
Thanks, this is kind of new to me.
Ah, so now I see how templating works. Does jQuery have something built in?
(== (mod n divisor) 0) palauttaa jo sellaisenaan booleanin
Note that it's not "all" as in the zii widgets there are breadcrumbs, menu and portlet