labofsahil/github-event-dataset-2012 huggingface dataset:
This is not supported and we already know about it. Pretend this branch doesn't exist and don't try to use it.
Right.  Just doesn't need to be committed to repo, unless you want to lock developers into certain versions of development or test gems.
setup better messages
Have you managed to find a way to compile libOmxVenc from Cyanogen's repo?
functional approach sucks. do you know why? do you need my explanation?
try to use classes. Flanigan Javascript book 9th chapter may help you.
Trying black out instead of ICS grey...
Yeah, the existing code has this problem equally - and you're right that it's not likely to occur in reality, though it'd be a nasty one to track down if it did. If you don't feel like fixing it, that's fine, since it's not really related to this pull request, but we should still extract the `path_contains_path` function rather than duplicating that same bug two more times.
Yes, I missed that one.
I thought it perhaps served some documentation purpose to be public.  Those constants are public in the OpenSAML libs, so I felt justified doing same here.
Is console available in all browsers?
No worries. I was think that local vars would be final, but this works too.
Is it better to return a negative box or exit with a warning?
I decided to change it to a warning because a negative box would probably result in an error in a subsequent call that uses the negative value (eg. MouseBox), which would cause an error in an unrelated function (eg. in math.simba), so i thought it would be better to just warn and return a (0, 0, 0, 0) box which would reduce the chance of a run time error.
intentionally not calling super.preRestart?
Why the extra spacing?
Evernote was unable to submit your note for the following reason:
The storage limit for this account has been reached.

Original message information:
From: drewhk <notifications@github.com>
Delivered to: thanhbv.fa84f@m.evernote.com
All recipients: akka/akka <akka@noreply.github.com>; giabao <giabao@sandinh.net>; 
Subject: Re: [akka] Minor cleanups in new remoting (#981)
Sent: Fri Dec 21 08:41:37 PST 2012

To prevent excessive emails, you may not receive another error reply
for the next 360 minutes.
yes, thats right
the problem was that Helena was adding another cluster actor, which use subscribe and sometimes that call was done (from another thread) before this val was initialized.
The ugly part is that we expose this (as ClusterEnvironment) to the cluster actors (in constructor of ClusterDaemon) before the constructor of Cluster has finished. Since it's only for the internal cluster actors I think it's alright, or do you have a better suggestion of how to do it?
single ticks?
U sure you want to close over the outer here and not just do context.system here?
this means that there is no dedicated supervision for this manager and nobody watches it either; also, it would be nicer to not create top-level names (with some semi-complicated scheme); there should probably be a TransportSupervisor which reports to the HeadActor (or similar)
I appreciate your comments, and interpret them as the code is not clear enough, so I removed ScatterGatherFirstCompletedRouter from JoinSeedNodeProcess. Hope that is less confusing, more straightforward.
Nice catch, what if payload is "null" tho?
yes, that was my initial idea, but when looking at it I got scared of its specialization for logging (LoggingBus), but if you say that is ok then I'll change
Registering a custom doctest parser is typically something that I would try very hard to avoid. I'd try very hard to solve that problem simply based on ellipsis.
Thanks, good spot :-)

Fixed in bb9b78b
I like how this looks and reads with the call out to bind_itinerary. Just feels so neat and packaged.
Kinda awkward to have capitals in an instance variable. Not the end of the world and I see where you're going with it, but it'd stand out less as self.ok or self.succeeded.

Interesting question on i18n here. Should we run OK/FAILED through i18n? It'd screw up (or at least complicate) the whitespace and alignment. I wonder what Fedora does. Not something I'd change now, but something to keep an eye out for in case you can find the answer.
y, that looks right.  thanks!
above seems unnecessary
Yes, this makes sense. I will change it. Thanks.
What's the condition you're thinking of?
Not sure I like the fact that context is mutable. If I understand, we set the context.rate to something else because another part of the program uses context.rate...shouldn't we hide the throttle, unthrottle logic inside the context then and only call context.throttle() and context.unthrottle() to change the task rate...this would allow some cleanup in this class.
I’ll revert this one.
"any tables currently locked"
->
"any currently locked tables"
or
"any tables which are locked"
or
"any tables which are currently locked"
Above two lines are incorrectly aligned.  It would be instead:

```python
7 	

        """Set compression quality for the image.

        :param quality: new compression quality setting
        :type quality: :class:`numbers.Integral`

        """
```
I decided to merge in a ToggleIconButton kind that had been in review.  Thanks,
probably not true
I think I assumed for this that "word" meant "keywords" (ie. `htmlWordListDesc[2]` in LexHTML.cxx) like it does for [other lexers](https://github.com/geany/geany/blob/master/data/filetypes.c#L10). If "word" means "identifiers" this change will be a big improvement judging from quick test just now. Probably `jscript_keyword` should be set to `keyword_1` then, to be consistent with other filetypes?
no reason. i was running a test, its a typo.
Should probably be in `<var>` tags rather than `<samp>`
You could use `cipher.getOutputSize(readable)` to determine the best capacity at the beginning for even more savings.
Cache is optional and the engine will be supplied this. Plan to provide Documentation on how to setup a cache-enabled App.
поля именются с префиксом m
I was talking with @carlosantoniodasilva and we think is better to store the casted model in a local variable and pass it to the thread local and to the instance variable. This would avoid cast the name twice. Something like this:

```ruby
backend = name && cast_backend_name_to_module(name)
self.current_thread_backend = backend if current_thread_backend
@backend = backend
```
My logic was that I think `options` will be a `Hash`, rather than a `HashWithIndifferentAccess`. I may be wrong here. Does Rails do something clever that I've missed to convert it into a `HashWithIndifferentAccess`?

If not, the reason I did it was that (as you no doubt know), back in early Rails days, documentation often suggested hashes with strings rather than symbols, and so symbolize_keys was used throughout to make up for this.

So I guess we should say one of:
1) Rails 4 just doesn't support options hashes with strings as keys. We go through and remove the calls to symbolize_keys 
2) Rails 4 supports options hashes with both strings and symbols. We go through and make sure there is a symbolize_keys everywhere.
3) I've missed something and this is being treated as a `HashWithIndifferentAccess`.
Ditto html_safe in both branches.
why not simply use `safe_join` back again?
No need for braces.
The `def desugaredApply = tree match {` will do.
Yep, needs a rebase.
no version numbers?
fuzzy_and isn't used
Done. Do we want even more?
Wouldn't it have to be hashable, ruling out a dict?
My bad. I did now know about removeO and getO. Will use them in the next PR.
I put this in for a while because I wanted to remove scalars from MatAdds. It turns out that special-casing zero is very handy. This is documented in the MatAdd constructor.
Sorry I forgot it again. I will try do it in the future.
They have been now - the pull request looks updated.
This is a bad idea IMO: links generated in the subrequest will not have the path prefix, making them broken. The right way is to prepend the path prefix instead.
The current behavior is that everything defined on the `ControllerCollection` is *defaults* that are *always* overridden by the controller values (it does not depend on when you call the methods).

There is another possibility: apply the values *when* they are defined. So, if you define them *before* some controllers, they are applied as defaults (you can override them when defining the controllers), and if you define some others *after* some controllers, they *override* the current values. Perhaps this is indeed more flexible and the behavior is probably easier to understand from an end user point of view.

What do you think?
not quite sure we want to refer to variable assignment as 'named j and t'.
Let's try this first and if db performance becomes an issue we'll iterate.
In that case avatar -> avatar driver.
these functions should have @return null
You may want to use the v1_xml parser thing in utils here.
What does this supposed to do?
I'm not sure what you mean exactly; can you gist some code samples of how it would look?
No, it didn't. Just some spacing issues. This query and the same in mol.map.boot.js will be getting replaced by a postgresql function.
You will have to add this back in your next ticket.
if I make a method inside Application definition and call it as 'this.updatePatientswaitingTitle' it gives error about method is not defined..
this error is probably because I am calling method when tap event occurs in ""action"" , so its unable for him to recognize this as Application.
I tried super or this.super also but they also didnt worked, So I used this way and it works fine..
Your loading mask should come before you get user privileges, in case our GET calls take a while
Added this as a "coding standard" (https://raxaemr.atlassian.net/wiki/display/RAXAJSS/Coding+Standards). See #4 under "Recommended". Please update with your feedback if you disagree or see a way to do it better.
It looks weird to be outside of define...is it possible to add it in the view as a listener?
avoid use of HTML formatting in Sencha, since it's difficult to maintain.  (On related note, we'll be doing a later pass to create consistent UI throughout all our modules, so don't want devs to invest more than minimal time in fine tuning padding, offsets, bold, etc if it might all be changed in the later design phase.)
Agreed. There's exactly one or zero per data dimension.
But it's there, on line 84.
This looks like an unused code path.
Why `http_method_not_allowed()`? Shouldn't that rather be `handler()` or `get() / post() / put() / ...`?
Its true of the line below that existed before this, I asked about it before and you simply said there was a reason but you couldn't remember.

I can make another patch if you accept this on that tries to remove the ++'s in merge_headers and see how that goes.
Data is already being split into packets by TCP, I'm not sure what your option would do better. This test just specifically forces it because that's what we test here.
Should this be a `TODO`?
This only inserts padding on the inside of parenthesis, eg func(a,b) becomes func( a,b )  (actually, I'm not sure if it will insert spaces after commas or not, I don't see anything in the documentation that does this, but it would be nice)

I like the pad-paren-in setting as opposed to pad-paren-out or pad-paren, however it would be good to clarify the comment a bit maybe (probably doesn't matter a whole lot)
Please update to match the latest style guide, thank you!
It's good intention, it's just very bad from a maintenance point of view.
Really?  I think I did like this in viterbi modules, too.
Unclear error message.
I tend to define things that are used before they're used. It makes it easier to read the file top to bottom.
Can this throw an exception if `search_date_start` isn't in the right format?
Might be worth looking into the `Pathname` or `File` helpers in Ruby itself to manage this path inspection. c.f. http://www.ruby-doc.org/stdlib-1.9.3/libdoc/pathname/rdoc/Pathname.html
oh ja, it's not really why I changed it.. t'was more the fact that the old link was broken, so I searched the RFE in the scikit learn docs and
replaced it - just happen to have the `dev` on open on my browser.. should I change it to `stable` quick ?
Same remark here (and everywhere with resume): is there a reason not to turn on resume by default?
Yeah you're right. I update the pull request in a moment.
Certe mais ça fait un plus jolie message d'erreur pour un joueur. Et avec ce genre de traitement, on pourra même faire un jolie pop up sur l'écran du joueur plutot que de tout casser violement
I'm not a Maven expert, but does this indicate that the `jargs` package depends on `junit`? Or is it just for local use?
@croman you're right, I missed the fact that it was in a loop context
err, whops
should this be `editted = "editted"`?
Can you make this take a hash instead?
if (version >= 0x1070) {
JUnit style is to import individual methods.  Sorry.
PS: VoxelBorder DOES allow TeleportCause.ENDER_PEARL  and internally disables the use of EnderPearl for teleportation
There should, but none of us have access to it; therefore, we are stuck until A) we do get the source or B) a new border module is integrated into Guest. Part B is my personal favourite, as it is a form of administration.
Likewise, this'd be in the `li { a {` above.
Nit pick. Suppressing _savePreferences seems like the exception rather than the rule. Can we change the naming here to ``_isProjectChanging`` to reflect that we're ignoring a bunch of event noise during that time?
ID selectors should not be nested
Extensions' main.js files and any modules defined within an extensions should always be relative to the baseURL. I don't believe "../" should ever appear within the extension's context.
I would suggest to rename this function to make more clear that the document title is updated like updateDocTitleNotifications, I think updateBarNotifications is too generic and might be confusing.
or, the two should be refactored into one:
`bloom_score(struct stats *stats, int should_positive, int result)`
called like this:
`bloom_score(&bloom_stats, 1, counting_bloom_check(bloom, key, strlen(key)))`

or maybe
`bloom_check_and_score(counting_bloom_t *bloom, struct stats *stats, int (check_func*)(void *, const char *, size_t), int should_positive, const char *key)`
if you can get that to work ;)
Indeed! That was a shitty VIM s/this/that/g :S Sorry about that!

Tests probably pass because there is not much coverage of these parts :-/
might need hours eventually? looks good for now
blockinflect?
Or Google how you can freeze that.
There is unnecessary space at the end, you could remove it?
I've refactored the code as you wished.
Can you make `tooltip-no-transition-on` a const? Kate did a nice job of organizing this file that way.
Nit picks, but you could just write something like....
<pre>
inputs[ i ].classList.toggle( "error" );
</pre>

And forego the if statement checking state
I'm thinking we should be fine placing this code somewhere inside the butter app LESS rather than what's specific for our template. This way any other templates that are written in the future will have this applied to them as well.
This isn't new though. It's just how this little helper guy works. It's been here for a while. I'm just saving the interval now so it can be stopped :P.
If you don't need these elements, might as well remove them.
Change this to http://popcornjs.org/ ; mozlla.org sends https://developer.mozilla.org/en/The_X-FRAME-OPTIONS_response_header which means that it can't be embedded
Ughhhhh this is a hack and we should fix this properly in v0.4
You can remove the `frameborder` attribute. `iframe-template` should take care of that.
Yeah, this works fine, I think.
shouldn't this be 'neither ... nor'?
这里不用，下面已有
you can just do: ```remote.setId(match.get(Task.ID));```
Pretty much all methods need to change formatting. Close mustache should have same level of indentation as the line with the open mustache.
Perhaps make a note here that we don't actually fully rely on zipkin-finatra yet but that it is the future?
I'm not wildly excited about having a fixed navbar. This has the effect of dropping 60 pixels from the viewable area that you have for the actual content on the page. If it was a toolbar and something that operated on the page, that's one thing. But this is just navigation.

Is there any other way to do this? Have you looked through the Twitter Bootstrap docs and mailing list (or whatever it is they have for support)?
Consider using MAX_LIFETIME in the log message, also?
IMO we need at least one IME keyboard to test out IME UIs, like candidate panels, etc., but your concern is reasonable and I will remove it.
Can you move this to `setup_helper` so it can be used more easily by affiliated packages (if they want to)? (e.g., make a function that takes the package name as the argument and then in *this* file give it ``'astropy'`` as the package name.)
Also, an important gotcha here regarding isscalar: if you do isscalar in a 0-d numpy array, it gives False (stupidly, in my opinion...), but it's still not iterable.  So you might change all the `isscalar` calls to something like ``np.isscalar(z) or (hasattr(z,'shape') and len(z.shape)==0)`` .  Or take a look at @240, where I'm proposing a function to fix `astropy.utils.isiterable` to do (I think) what you would want here.
The should be `Mod` not `riak_search_vnode`, no? Also, why doesn't this break things -- ie. your provided riak_test passes for me.
It is used in ``connect(mapper, SIGNAL(viewModified()), this, SLOT(enableApplyButton()));``, where I can't use ``ui->applyButton->setEnabled(true);`` directly, right?
Use ``true`` (what Stéphane suggested I think).
"widgets/GoogleEarthPanel" => "widgets/GoogleEarthPanel.js"
Please put newlines after `def`s and refers and such. All of this code is shoved together. The same thing is happens in compile. It makes it hard to read.
Yep @jonlemmon, feel free to change this as you please. I just had a go with your way but didn't manage to have everything working as expected (popover wouldn't show up after revert). Actually it might just be that I'm tired, so you can either do the changes yourself or wait that I get a good night of sleep and give it another go tomorrow.
I haven't really checked yet, but was this function moved here or copied? Having the same funtion twice in the lib is of course very ugly, so a separate helper class would be a way out of this.

Originally, I had a JsonLd class in the repo that wrapped this functionality, but at some point, this became impractical. The big headache for any solution is imo to keep track of all possible namespaces. I would say that this screams for a global registry, but unfortunately, global state was declared evil at some point...

Expanding the names during type construction sounds like a good idea, though. If we can make sure that all rendering functions have access to the namespace map for their type, this might work. And I think it could also help us to avoid rendering unused namespaces, if we manage to track which namespaces have been "rever resolved" during a given render call.
Seems to be a different tab spacing in use in your changes
Why not

```
final int sleepMillis = timeout / 1000000; // truncates
```

?
Is this in any way expensive?  I.e., should this become:

```python
if self._anchor:
    self._anchor = None
    QtGui.QToolTip.hideText()
```
I think i have done something similar, the back up is only created when the user clicks "Rename Existing" otherwise he has the same normal rename options "Overwrite"  and to enter a new name (after clicking cancel ) please look at the code and the implementation when you have time
Note that this affects how the options for help are printed - I think it's awkward to have the options displayed in a random order each time you do something like `ipython --help-all`.

Given that this method is only called to display help information (`class_get_help()`), I think the cost of Python's built in sort is going to be negligible compared to the speed with which output can be written. This was probably the place I was most confident that sorting was the right way to go.
this if statement shouldn't be necessary as there's already a checkArgument on floatingIpClient
I assume you meant a function that would take a predicate and return another predicate. this would require exposing the assignments map from BasePoolingComputeService, which is probably not a good idea right? did I miss the point?
The below looks like the way to retrieve a certificate, right?  What if the
credential were the certificate in PEM form instead? I mean using keystore
looks like extra work, unless I'm missing something.  Ex. This is how
jclouds-chef works...
On Oct 31, 2012 9:07 AM, "David Ribeiro Alves" <notifications@github.com>
wrote:

> In
> labs/oauth/src/main/java/org/jclouds/oauth/v2/functions/OAuthCredentialsFromPKCS12File.java:
>
> > +   private final String identity;
> > +
> > +   @Inject
> > +   public OAuthCredentialsFromPKCS12File(@Identity String identity, @Credential String keystorePath,
> > +                                         @Named(PKCS_CERTIFICATE_KEY_NAME)
> > +                                         String keyName, @Named(PKCS_CERITIFICATE_KEY_PASSWORD) String keyPassword) {
> > +      this.identity = identity;
> > +      this.keystorePath = keystorePath;
> > +      this.keyName = keyName;
> > +      this.keyPassword = keyPassword;
> > +   }
> > +
> > +
> > +   @Override
> > +   public OAuthCredentials get() {
> > +      File file = new File(keystorePath);
>
> there is an identity. In google's case its the account number email.
> my question was related to the "oauth.credential" property and the use of
> file credentials. Specifically, I was wondering if, in the case of using
> file-based credentials, such as pk's the *.credential property would store
> the path to the file.
> The current ctor args are:
>
> @Identity <https://github.com/Identity> String identity,
> @Credential String keystorePath,
> @Named <https://github.com/Named>(PKCS_CERTIFICATE_KEY_NAME) String
> keyName,
> @Named <https://github.com/Named>(PKCS_CERITIFICATE_KEY_PASSWORD) String
> keyPassword
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jclouds/jclouds/pull/920/files#r1994176>.
>
>
it isn't bad, just saying punt for now, since this pull has a lot going on already :)

ImageExtension can be made from any node... so you need to filter it.  I suppose you could make a delegate type called FilteredImageExtension extends ForwardingObject implements ImageExtension
omniauthの依存で入ってくるんですねー
OK, done. I've made it return the size of the string instead of the buffer because we're not MS after all and try to keep it POSIXy, but tell me if it's too cumbersome that way.

This does make testing a bit annoying, as `cl_git_pass` thinks it's a failure even if the return value is positive.
Is there an anchor with ``id="top"`` somewhere?
The same 80px padding here lines up the heading with the text under it.

This does mean the icon is floating off in space a bit, but giving the image a left margin fixes that. 10px isn't enough, 20px seems too close, and 15px looks about right to me. Or we could just use slightly bigger images if we have them, since there's a little more space to fill.
that looks like something I would write
Perhaps I'm late to the party but shouldn't that be "CrashStorageOOIDSource"? OOID is not a word.
Before you can run this you need:  `sudo apt-get install python-software-properties` and before that you need to first run the first `sudo apt-get update`
Why is this a `property`? It looks more normal to do `if mystorageerrors.has_exceptions(): ...`
Shouldn't this be either 'Setting' or 'GraphDatabaseSetting<String>'?
Unused `Loop` argument.
"검은색 글/흰색 배경 "
should be
"흑백"
Fixed, https://github.com/DarkArc/worldedit/commit/f7ea67f471c05ebbbc7d82d4566db1c69a4ba016 needs reopened though.
IOException ?
better would have been `elseif`
In the case where the constructor simply returns (line 80), the internal state should not be changed.
This does not make sense. You should use the container to configure the manager and pass a CacheManager to the constructor.
@fabpot added it on all interfaces last week as it was the only points where the Symfony2 CS were not matching PSR-2. update your sniff :)
@kufi if the propel-bundle is not updated, this type will not be registered either in it, so it will not break PropelBundle (it will simply forbid you to use the new unsupported feature, except by adding the support yourself, which is always possible).
I don't know what standards are you talking about, but it follows the examples you can find on phpdoc site
`assertTrue()`
It does: Endianess is Netty (and in Java generally) is handled by the underlying `ChannelBuffer`. So to do this “properly” you could either demand a little endian `ChannelBuffer` factory, or override the `LengthFieldBasedFrameDecoder` and wrap the incoming buffer with one that is little endian.

However, this is so trivial, that it’s both simpler and clearer to do it “manually” as you have done.

But remove the comment I think :-)
is indention right here?
Should we limit ourselves to the verbs people actually use for now, so we
don't crowd the dog with less useful / always 0 metrics?

On Wed, Oct 10, 2012 at 11:04 AM, Carlo Cabanilla
<notifications@github.com>wrote:

> In checks.d/iis.py:
>
> > +        ('iis.httpd_request_method.head', 'gauge', 'PostRequestsPerSec'),
> > +        ('iis.httpd_request_method.put', 'gauge', 'PostRequestsPerSec'),
> > +        ('iis.httpd_request_method.delete', 'gauge', 'DeleteRequestsPerSec'),
> > +        ('iis.httpd_request_method.options', 'gauge', 'OptionsRequestsPerSec'),
> > +        ('iis.httpd_request_method.unlock', 'gauge', 'UnlockRequestsPerSec'),
> > +        ('iis.httpd_request_method.lock', 'gauge', 'LockRequestsPerSec'),
> > +        ('iis.httpd_request_method.mkcol', 'gauge', 'MkcolRequestsPerSec'),
> > +        ('iis.httpd_request_method.move', 'gauge', 'MoveRequestsPerSec'),
> > +        ('iis.httpd_request_method.propfind', 'gauge', 'PropfindRequestsPerSec'),
> > +        ('iis.httpd_request_method.proppatch', 'gauge', 'ProppatchRequestsPerSec'),
> > +        ('iis.httpd_request_method.search', 'gauge', 'SearchRequestsPerSec'),
> > +        ('iis.httpd_request_method.trace', 'gauge', 'TraceRequestsPerSec'),
> > +        ('iis.httpd_request_method.other', 'gauge', 'OtherRequestMethodsPerSec'),
> > +
> > +        # Errors
> > +        ('iis.errors.net_found', 'gauge', 'NotFoundErrorsPerSec'),
>
> They don't do http status codes? Lame
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/DataDog/dd-agent/pull/221/files#r1807556>.
>
>
I am not sure about styling it here.
Mixins don't have to be Ember.Mixins, they could just be plain objects.
This is an interesting idea, but for the moment it does not really allow one extend the empty widget... 

I would suggest to either remove this, and rely on `create_empty_widget` for all customisations, or to have the default empty widget create buttons dynamically based on this dictionary.
Missing quote here (I can fix this)
i made the seracheditview work like the corpuseditview, in that it now has a seperate function to refresh renew the connection between its updating collection and the model taht it is supposed to refer to, in this case the current corpus' datum fields.
这个地方是不是要判断空，防止nullpointexception
Let's wait for @basiclines comments, sure he find more than just the rems ;)
Can we use the same duration for the popups and this? (0.3s)
This css file is meant to be shared. Please modify other versions of the files, or better but it into the `shared` :)
Missing git ignore?
what's the preferred alternative?

we use it in other places in our code, we'll create a new branch to remove "ffos" from the code.
Oh, I didn't think about this and tried to put all the HTML in templates like in rest views. But this is way smarter, thanks.
Also since you want to return an array I suggest that you use Array.map instead of Array.forEach
This is just testing the same behavior as the current razor bin:
https://github.com/puppetlabs/Razor/blob/master/bin/razor#L44

If XML is appropriate we can determine whether our response should be .to_json or .to_xml and we can change the test according.
Can we preload these? I know it's asynchonus, but since we know each creature needs one, it seems like preloading it would be cool.
Agreed, that's why I added the lengthy comment along with the code. I could
rewrite the code but it would be longer; is that acceptable?
4 spaces (I'm useless)
Ah, that's fine then.
unless i'm missing something (nyquil) i think we could move `instructions.push(token) out of the condition and to the bottom of the loop, as it's done on line 209, 222, and 224 (and 219 returns)
But due to sampling, we can't trust the innerjoin is a true zero, vs a sampled zero where there is actually a heavy key on the other side, right?

Alternatively, Edwin and I were discussing use CountMinSketch, which (we thought) could conclusively distinguish a true zero from a rare item.  If we did the count-min-sketch instead of sampling (and Edwin already implemented that in algebird) we could get the advantage of a bloomfilter at the same time.

Perhaps we should punt on that for a v2 and just get this version out.

In truth, the use case here is to replicate Bieber, Obama, Gaga, etc... and not replicate everyone else. So the savings from getting a few keys to replication 0 are probably not even measurable since those keys are by assumption very low mass to begin with.
Yes, as agreed with curators - if dose exists and compound doesn't, we fail the experiment load. They have committed to load into Atlas only experiments with both compound and dose, or just with compound - if dose is not available. If there indeed exist experiments with dose and radiation - then your question is for the curators. But the code is implementing the contract agreed with the curators as of now.
Not sure if this is an issue but echo has extra spacing
https://github.com/AppScale/appscale/issues/106
That issue is vague and doesn't indicate what the problem is, how you intend to solve it asides from "use JSON" (which does not magically solve your problem), or even what services need this help. Set a good example here of how to solve this problem and fix this the right way.
IMO it should not throws FileNotFound exception. Please see that method for example https://github.com/KnpLabs/Gaufrette/blob/master/src/Gaufrette/Filesystem.php#L67 we should be able to create file too, not only working with existing ones. It can looks similar to:

```php
public function createFile($key, Filesystem $filesystem)
{
    $file = parent::createFile($key, $filesystem);	

    $filename = $this->computePath($key);
    $file->setName(basename($filename));

    if ($file->exists()) {
        $file->setSize(filesize($filename));
        $created = new \DateTime();
        $created->setTimestamp($this->mtime($key));
        $file->setCreated($created);
    }

    return $file;
}
```

Thanks for PR and info about it. I will try to implement it base on your solution.
Can we use error codes like JError does? Then default value would be either null/0 (no header) or 404.
Whether or not to include a condition makes very little difference.  To be clear, the "condition" in this case is that the master is not already (in)active.  So the two are identical ("set active=true" vs. "set active=true where active=false").  I'll remove the comment.
prop:
Why can't this be fixed in toCSS? this looks like a hack.
Why is this useful? The "type" usually carries a lot of information, e.g. what this metric is actually about. Without it, metric names may no longer be unique, so removing it doesn't make much sense to me.
See Qt qml coding conventions. 
http://doc.qt.digia.com/4.7-snapshot/qml-coding-conventions.html
For qbs files name as simular as id in qml files.
some really basic testing is available -- run 'fail2ban-testcases'
I simply copied it from lines 25/26:

```
                  - if @project
                    | #{@project.name}
```
@phamann and I talked about this and it does actually work this way in swipe/non-swipe cases, just more aggressively (eg more preloading) for swipe because it has to be seamless (eg the next image should already be visible when starting to swipe).
I think you mean `'\\.{}$'`. This will match any character where the '.' is.

Personally I'd prefer 

    if re.search('\\.{ext}$'.format(ext=self.extension), self.outfile_root):

As it makes the code just a tiny bit more readable. But the regex is definitely wrong.
What happens if this returns nil?
Yes, currently it only works with the `:index` locator.  With these first commits (for `Table#Rows` and `Row#Cells`) I just wanted to get the minimal in there to make it conform to the `ElementCollection`.  I planned on adding support for other locators after this was pulled in.
Oops! I think something weird happened when I re-merged after the squashed merges.
Let me dump this branch and redo it...
"array promise" or "promised array" telling just an "array" is confusing.
Don't you need to remove `_emit('deactivate', ...` it from wherever it was called before ?
Let's use `NetUtils.asyncFetch`, it handles some extra edge cases, uses a fancy nsIPipe, and I imagine it has some maintenance & tests.
When is this data saved to localStorage?
OTOH, if the labels in the DOM are never shown, why not just embed the contents here and surround them with gettext?
it should be... they may be doing the "wrong" thing, but everyone should correct it.

@digitarald any thoughts?
This function needs a docstring to explain what it's doing and why.
I think that this was only for debugging purposes?
Yep, that worked. We might want to update it in the future if we want better Windows support though.
Maybe it's better to write:

```
$array = array();

if (null !== $columnName) {
    $this->cleanupParameters();
    foreach ($this->getParameters() as $key => $parameter) {
        if ($parameter['column'] === $columnName) {
            $array[$key] = $parameter;
        }
    }
}

return $array;
Doing this now.
it "uses the compatible encoding if they are compatible" would describe it more directly i think. The concept is already named "compatible encoding", so no need to introduce a new one called "better fitting encoding".
Try again: ``file:`{$SERVER}/sagecell/```
just move this to the test then?
Do this as simply:
```php
$sharedEventManager = new SharedEventManager();
```
Given that the database is a mandatory argument for the constructor, I'd say that this check is jitter?
Committed the Uri validator, but some expected test results are failing (Commented out and marked as TODO):
https://github.com/zendframework/zf2/pull/1521/files#L4R36

The expected results were tested manually against Chrome's Url validation, with this page: http://wufoo.com/html5/types/3-url.html

Could I get assistance from someone who has worked on Zend\Uri\Uri to take a look? ( @shevron perhaps? )
Just nitpicking -> http://www.phpunit.de/manual/3.5/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.exceptions
It's only a quick way to obtain the same result here.
What do you mean by magic ?
you don't need this one as you don't even enable it
I see an issue here. Currently, KnpBundles expects to have a github username as username in the database. This will work fine for people logging in through Github (as done currently) but it will break when using Sensio Connect if your sensio connect login is not the same than your github login (I can give you an example: @lsmith77 is [lsmith](https://connect.sensiolabs.com/profile/lsmith))
Is this related to the main purpose of this commit?
It looks like the layer is created and stored as well (textBoundsLayer)
I didn't really have any use for adding the payload to the hook's arguments, but that's a good idea.

I don't want to handle only non-matched message types in this, though--I want to allow custom handling of *any* message type.
It breaks specs on TravisCI, We should use `postgres` role there.
Not sure about the addition of the space after the `=`. Where does that come from? Coding standards?
Not a fan of setters with fluid interface...
what is wrong with using `$request` as the variable name? wouldnt know what it was if i saw a 5 line snippet without the method definition.
yes, its just a hack for now. Not finished
Missing spaces around !.
also only load it in production
CS: spaces before and after parenthesis
"For array settings"
that's because ``samples/content/another_super_article-fr.rst`` does not contains any date so with the pr this file won't be generated.
i don't get this one..
Could you add a test for another locale? Would be better to make sure that it works outside of the special case that en-US is.
What should we change the admin media path to on non-local instances? Currently they're being served from `/admin-media/` -- it that still cool?
you're right; I can't think of anything we wouldn't want to use OpenSans for
make this options._type instead.
We don't need the created date or author details.
I'm just not clear what's going on in this line? Holding off until have a chance to talk, otherwise looks good
That would work I guess. I'll see if can make that work
Add comments here describing what this test is doing here.
Not to nit-pick, but I don't see any reason for this variable to be so terse. In the interest of readability as an example, we'd be better off using `project_name`.
Another nit, but you don't need to check for `null` here. All fields that are `null` will be dropped by Zotero anyway.
I don't understand the replyTo pattern
Added back the global delay to the displayed audio delay.
final static?
We do:

    var moveFourPoints = function(x, y)
Thanks!

On Mon, Jun 18, 2012 at 4:15 PM, Jason Normore <
reply@reply.github.com
> wrote:

> > @@ -65,6 +65,8 @@ Make this change to request write access to products
> and read access to orders:
> >
> >  (Note that you can change your API permission scopes on the fly, but
> the merchant will have to approve each change and your computed API
> password will change.)
> >
> > +You may need to create this file if it doesn't exist, and add a
> reference to the omniauth-shopify-oauth2 gem to you Gemfile.
>
> It looks like there is a file for omniauth in the generator to be copied
> but isn't creating it in a new app for some reason. I will investigate this
> and create a new issue for this problem.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Shopify/shopify_app/pull/21/files#r1004940
>
`(ex.:account activation)` -> `(e.g. account activation)`
@Mnkras is completely right about that, you want `if(!is_object($fak) || !$fak instanceof CollectionAttributeKey){` not `if(!is_object($fak) && !$fak instanceof CollectionAttributeKey){`
Would it not be more compactly and efficiently if you write `return isset( $selected[0] ) && $selected[0] != ''`?
I just noticed that the paginator property is in the base class which is incorrect. Not your fault but something we need to address.
Even though `test.authenticate.js` is in the `test` directory, `nconf` uses a path relative to the root of the repository; that is, `local-test.json` would refer to `local-test.json` and not `test/local-test.json`. Thus, I do need to specify the test directory explicitly.
Also, this doesn't return SQL, so the name should probably be changed to something more descriptive.
That seems like it would work, assuming that we included event timestamps in the data that we calculated the hash over.

I'm not sure what happens there once we want to support other types of event submissions besides reports (e.g., streaming) because, if you're assigning the ID based on a hash of all of the events, then you can't compute the ID until all of the events have arrived.  But I suppose that that is a relic of my having tried to tackle this problem in a more general way, and there's no reason why reports can't do something more specific.

I'll take a swing at it a little later.
@zaphod42 says it should be `are-queries-different?`, or then just `queries-different?` or `queries-differ?`. I agree!
I don't think we need both a commentArr and a comments array. One `comments` array should be enough.
For this and the `getAdminRestContext` method. There is a big difference between a context that may have existed before and a context that is being created new. For example, if it may have existed before, you need to worry about whether or not it is still authenticated.

That said, maybe these methods should be prefixed with `create` instead of `get` so it eliminates doubt.
I think getPagingParameters will take care of this as well, so no need to check explicitly.
Can you rephrase `end files` ?
Если очередь не очищать, то при загрузке более чем одного файла с каждым файлом в очередь будут попадать предыдущие.
The timeout shouldn't be necessary, but pulsar requires the server channels to have already been initialized before the client tries to connect.  So without the timeout, the client tries to set up its channels before the server has re-initialized them on its side.

We could avoid the setTimeout, and a number of other issues by queuing channel requests on the server side.  If the channel doesn't exist yet, you can queue the listener, then once the channel is created add any queued listeners.  This would make the timing issues around channel creation much less touchy, and as a result would give greater reliability to the library.
waar is de indentation>
Another test you could do here would be to word wrap without the second parameter and then count the string (so we know how long the default wrap is). Could you add that too? :)
Придумать другой алгоритм ;) Их много уже придуманных, очень.
В данном частном случае размер массива DNA равен 100, для более общего случая отсутствует проверка size (это ведь не размер, а процент мутаций) на допустимый диапазон [0;100].
После того как получили self от родителя, необходимо проверить не вернули ли нам nil и после этого можно уже инициализировать.
self = [super init];
if(self){
      //my init
}
return self;
Считаются совпадения, а не несовпадения
Needed to trigger the building of addons with external dependancies.  Currently that is cmyth which depends on mysql.
MaterialData uses that class for the "vanilla" items, it's supposed to be filled with good info, but okay, i will probably refactor this part soon anyway...
In the init case, the matrix I pass, is still the old one, which is a matrix n_new_as x n_times, with non_zeros in n_old_as rows and zeros otherwise
shouldn't this be memoized?
I did have the creation of the gherkin formatter being done by the JsonFormatter constructor but then moved it out when I was writing the jasmine-node tests so as to be able to create it with a dummy gherkin formatter for test purposes.  Happy to revert to having the JsonFormatter do the instantiation.
Fairs.
If it's only used in a single action, it doesn't really deserve a before_filter - just inline or call it from the action directly.
I don't know that it hurts to provide `Content`, since it should be clear to the consumer that to make a byte array of content we would need to allocate a byte array for the content. It may be worth adding a comment in the API that for cases where that memory pressure is not acceptable we encourage the use of `ContentStream` instead.
@ioannagian:  It seems you have not worked on the latest verison of clazzbase.py. ou removed the bug fix for the import/export problem.
Why would you like to remove this short URL?
let's discuss this based on a draft.
should make everything more (num)pythonic.

On 29.10.2012, at 19:12, "Denis A. Engemann" <denis.engemann@gmail.com> wrote:

> i think the clue is to add x-arg funcs to find sources and displace the order-bookkeeping.
> but wait. the solution is almost obvious.
> just dont sort anything and do everything with the scores:
> 
> return unsorted sources
> 
> get scores for those
> 
> reorder on plotting using scores (and assume sufficiently aware / informed / users, also tell them)
> 
> 'the plot visualizes the score sort, component index relative to unsorted sources'
> 
> get args based on exploration, relative to the still unsorted sources...
> ... and all should be unmessy.
> 
> makes sense??
> 
> On 29.10.2012, at 18:58, Alexandre Gramfort <notifications@github.com> wrote:
> 
>> In mne/artifacts/ica.py:
>> 
>> > +        Returns
>> > +        -------
>> > +        fig : instance of pyplot.Figure
>> > +        """
>> > +
>> > +        sources = self.get_sources_epochs(epochs)
>> > +        fig = plot_ica_panel(sources[epoch_idx], start=start, stop=stop,
>> > +                             n_components=n_components, source_idx=source_idx,
>> > +                             ncol=ncol, nrow=nrow)
>> > +        if show:
>> > +            fig.show()
>> > +
>> > +        return fig
>> > +
>> > +    def find_sources_raw(self, raw, target, sources=None, score_func=None,
>> > +                         start=None, stop=None, sort_func=stats.skew):
>> Yes, so I think making _find_sources a method, making it smarter so that it handles x-arg and x-y arg score funcs and making it update the .sorted_by attribute would do the trick. Shouldn't be too much of an effort. As a result we would drop sort_sources and we would get around ica.argsort_sources_epochs_very_long_method_name(epochs) ^^ Wdyt?
>> sounds good. I would however see while coding how much lines it adds or better removes :) it is a bit hard to tell without a close inspection of the code
>> —
>> Reply to this email directly or view it on GitHub.
>>
that's buggy as if verbose is False then :

>>> data = np.abs(data)

is not done
That's fine its local :)
I would set the charset part only for `text/-something-`
So that I don't have to pass :page => params[:page]
sale here: ``--no-ansi``
why ? Button works too
you have an extra ``web`` folder
用逗号
Hey @hhoang, do you know what this line does exactly? Does it provide a test oauth callback or something so you don't have to set up a proper app with something?
Note that this one is a bit misleading since the template is not part of what is called the wiki descriptor on wiki side and is just used when creating the wiki.
I wanted to simplify things by getting rid of the distinction between data and metatdata, but OK, I see your point.
Я бы в этом случае в корень вставлял.
Strange, I checked it with Nov5 katello build and it's "provider_submit", may be the katello nightly has the "upload_form_button"
derp? :)
Typo, whereever
Why do these have defaults anyway?
That actually should have been a TODO to use poll_until if we bring it back.
same w/ this, lets call it dns instead of rsdns
Instead of this, you could just return `items`.
This comment is about style.

This is very java1.4-esque ;)

In Scala you have both enums (not highly recommended since they're a bit odd to use), or sealed case classes, see http://stackoverflow.com/questions/1898932/case-classes-vs-enumerations-in-scala

Also, making classes extend this trait is very java-esque as well. Importing the case classes and matching over them is more sensible.
Again, the word "percentage" would be helpful in this explanation.
Aha. I see the next line does ```buffer.reshape((self.renderer.height, self.renderer.width))``` **not** ```buffer.reshape((self.renderer.height, self.renderer.width, 4))```.  I see your motivation now :smile: .
Double back quotes needed to render as a code block.
Instead of a trailing backslash, the RHS of this statement needs parentheses; it's a tuple.
I don't see the point in rewriting the initial code, which does the same thing(release_channels is the list of ReleaseChannels objects) and [-1] take the last element of the list.
There is no improvement in execution time, plus there is an extra locator.
This should probably be a warning. These seem like breaking changes, so they should raise a warning outside of the compatibility bump.
I wrote the Python code. The algorithm is derived from the C++ module in mozilla-central that handles version comparisons. What would you rather it be a submodule of?
а что Coding style уже поменялся ?
Should only `$.parseJSON` once here and set it to a variable
why do you need <vector> and <map> ?