labofsahil/github-event-dataset-2012 huggingface dataset:
I'm on Mountain Lion 10.8.2 and using Homebrew, and a `gem install pg` with no additional flag sufficed. Gem version is 0.14.1.
It definitely doesn't make sense to raise `ArgumentError` in `build_order`. Otherwise you'll have a situation like this:

``` ruby
posts = Post.order(id: :DESC) # ArgumentError should be raised now, as the argument to `order` is invalid
# ... some other code ...
posts.to_a # but instead it is raised now
```

We could just default to `:asc` but this seems more surprising from a user's perspective if they make a mistake.
I know it isn't really this commit's problem, but this should certainly be nme.CONSTRUCTOR.
Changed in 65352cbda05c0d64d86830e95b44015486845d9e
Since I haven't done anything to the LYRALightcurve's init function
compared to the original LightCurve, I can just delete those two lines,
right?

The EVE and GOES lightcurve objects also have the same lines of code.  I
copied the the GOES lightcurve to start the LYRA one.  Those should be
changed too.

Thanks,

Jack



On Oct 6, 2012 10:57 AM, "Florian Mayer" <notifications@github.com> wrote:

> In sunpy/lightcurve/sources/lyra.py:
>
> >      """
> > +    def __init__(self, *args, **kwargs):
> > +        LightCurve.__init__(self, *args, **kwargs)
> > +
>
> Why do this?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sunpy/sunpy/pull/323/files#r1779484>.
>
>
How's this `isinstance(X.shape[1], (int, Integer))`
Should be `return NotImplemented` (and elsewhere).
I think that's because deep (for expand) entered Add, Mul and Pow?
Yes it could/should be `Basic`.
Concerning the `pl` comment you have made a few lines above: If I understand correctly my old code you have the right to burn me on the stake for the abomination that I have written. I **think** that `pl` should be `the_args` and that the only reason that this does not crash is that by chance in the current lexical scope `pl==the_args`. I am sorry to have made you look at this.
```

>>> factor(4*x**2 - 21*x*y + 20*y**2, modulus=3)
x**2 - y**2
```
It might be called `finite_diff` instead.
Did you try just `sympify(string[:-1])`? I opened an issue pointing out that this is perhaps a redundant function. If you have found that it doesn't work then I should close the issue.
> It's not a big deal, but you could replace the second if block with the third argument to reduce().
>

Sure, done.
Whole point was support of recursion. so it is needed.
Zero, even if theres no point in a zero-width bar :>
Is there a reason you're ```#include```ing an internal header rather than the top-level interface ```<boost/filesystem.hpp>```.  Specifically, ```<boost/filesystem.hpp>``` transitively includes ```<boost/filesystem/convenience.hpp>``` so we usually do not need to include the latter manually.
It's too bad we have to do this filtering after we generate the hashes of all of the values, but I can't think of a reasonable way to do it otherwise.
I wasn't aware that the `Symfony\Component\Serializer` also supports YAML. I'll have a look at it later today, when I'll finalize this PR.
This should be the only method in a "Clip view swapping" group.
There has to be a better way to do this, right?
Yes. I have been changing them as I encounter them.  I'll continue to do so.
Maybe make it more general?

    missing_keys = set(default_config).difference(config)
    if missing_keys:
        default_config.update(config)
        with open(CONFIG_PATH, 'w') as f:
                json.dump(default_config, f, indent=2)
Rename button to be `Center Map` and without `<strong>` tag
I'm not sure of this line.... I just copied it from above.
is it :program:`mongod` ?
documents is singular in the glossary. so use 

     :term:`documents <document>`.

build the manual to see the error.
should we use angle brackets. around `<array>` just to make things clear? I'm leaning towards yes.
s/you can //
I am trying to only use '/'.join([]) method to replace all concat method. If you can, please use the join method. Trying to sunset concat utility method.
That is defined because there is a block named `available`. If you create a block the template engine will create that variable is the block isn't empty. So the "header" is only outputted when there are actually available extensions.
That's being really nitpicky now, but shouldn In be in?
It's actually not the exact same code.
https://github.com/phpbb/phpbb3/pull/1096 fixes this separately.
includejs etc?
All 3.0 styles become obsolete until converted. They are not backwards compatible, therefore it has to be reset and admin then can manually install updated style.
@cyberalien not fixed yet
See the line below your comment.
How about something like: "Gravatar is a service that allows you to maintain the same avatar across websites. Visit www.gravatar.com for more information." I don't like "globally recognized avatar".
Don't we want to do this regardless of a callback being defined?
Was a mistake I previously used table.Copy for something but didn't need it later then forgot to remove the brackets. my bad,
You're right, it doesn't add any ID into the log. I'm getting this:

mojito-composite-addon:execute timeline=634 (slot-1,slot-2,slot-3,slot-4,slot-5,slot-6,slot-7,slot-8,slot-9,slot-10,slot-11,slot-12,slot-13,slot-14,slot-15,slot-16,slot-17,slot-18,slot-19,slot-20,slot-21,slot-22,slot-23,slot-24,slot-25,slot-26,slot-27,slot-28,slot-29,slot-30,slot-31,slot-32,slot-33,slot-34,slot-35,slot-36,slot-37,slot-38,slot-39,slot-40,slot-41,slot-42,slot-43,slot-44,slot-45,slot-46,slot-47,slot-48,slot-49,slot-50)

and I should be getting: mojito-composite-addon:execute[ID-HERE]. What will be the right way to collect the instanceId in this case?
Yeah, deleting `binder.children` is quite dangerous, since I think it might break some of the advanced features of client-side mojito.
This is to check whether height & weight combination legal or not. So, it should not be removed
is all of this CSS necessary for your patient list styling? Suggest that you limit investment in CSS as much as possible, at this point, and focus on functionality.

If this CSS Is needed, please label specifically to clarify that it's for the patient list
Might be useful to have a few sanity checks here, height_cm <=0 or height_cm >= 300 and similar ones for weight.
Why is this an abort condition for INIConfigManager but not for XMLConfigManager. Is this intentional?
.all() is not needed here
Finally, unless I'm missing something, neither negotiate nor renegotiate will warn about an invalid value for `use`/`prefer`.
If we don't actually need exec for django why don't we simplify this implementation and give it some name so it won't be public (e.g. `_exec`).
"is by calling the URL definition"
maybe
"is by looking up the URL definition"
Thanks,
I am having a hard time following: a command is issued in a room, but when I handle it, i can't seem to find the room either in the context or the connection.
Thanks
Marco

 
Marco Falcioni
415 886 7363


________________________________
 From: Eliot <reply@reply.github.com>
To: marcofalcioni <marcofalcioni@yahoo.com> 
Sent: Friday, May 4, 2012 12:30 PM
Subject: Re: [mongo-irc] UserHost command implementation (untested) (#1)
 
> +package xgen.irc.commands;
> +
> +import xgen.irc.*;
> +
> +public class UserHostCommand extends Command {
> +    
> +    public UserHostCommand() {
> +        super( "USERHOST" );
> +    }
> +
> +    public void handle( Context context , Connection conn , String line ) {
> +        String[] nicknames = line.trim().split( "\\s+" );
> +        // for each nickname, if found in room, return ident
> +        StringBuilder sb = new StringBuilder();
> +        for(String nickname : nicknames) {
> +            if(conn.inRoom(nickname)) {

conn.inRoom( x ) is if the user associated with conn is in room x, not if nickname is in room

--- 
Reply to this email directly or view it on GitHub:
https://github.com/erh/mongo-irc/pull/1/files#r776402
/kill fragcache \o/
But you used svec_to_string() anyway.  Text type shouldn't have bug, no?
is this intending to join the selectors into one long smashed string with no separation at all? Because that's what it looks like it does
Add a line break here
At least some of these can be handled like this:

```rb
system "make", "DESTDIR=#{prefix}", "SYSCONFDIR=#{etc}", "BINDIR=#{bin}"
```

etc.
I'd like for sqerl to track a tag of emysql. I think there is already a tag cut off of current master, but please verify it is what you want.
I think this can now go?
Again, copied from the HTTP API.
Probably want to remove the `timeout`
ComponentLibrary should be injectable without need for using holder
Why I am getting Emails from you guys, I don't even know you.. maybe it's the "@adapter" in your Code.
Argh. Sorry.  I'll get used to this.  I swear. lol
This has nothing to do with the proposed fix, so it shouldn't be in here.
We should start at the outset breaking up "base.json" into multiple files relating to the different components (solutions/preferences/etc) since it is already unmanageably mixed
Remove this comment?
yeah, we don't have any custom changes in this file.
Whoops, yes. Changes pushed.
Good catch! I patterned that after the other dialog and forgot to go back and clean it up.
seems like the original variable name was clearer?  I usually expect something named context to be a dict, not a list of dicts.
Is this parsed in examples?
Well in my branch I was only cycling inside the ops (I believe, I have been changing all of this a lot, hence the messiness).
I think this comment is useless: constants should be named so that from the name itself it is clear what they do.

Furthermore, we put space after `#`.
Why additional space?
No, each base font size is set in pixels, so that all the plugins set font sizes in `em`s relative to that base size.

For example, let's say a plugin sets the font size to `1.5`em.

At the smallest size:
* width: `560px`
* base font size is `14px`
* plugin's font size set by user: `1.5em`
* plugin's computed size: `14px`/1.5  =  `21px`
* plugin's computed size / width = 560/21 = *0.0375*

At medium size:
* width: `640px`
* base font size is `16px` ( medium width / small width  * small base font size )
* plugin's font size set by user: `1.5em`
* plugin's computed size: `16px*1.5`  =  `24px`
* plugin's computed size / width = 640/24 = *0.0375*

This way, the ratio of the computed font size to width is always the same.
Needs prefix versions.
This works, but I'll update with process.exit(1).
Ah, this is just git being confused. Will delete this file.
`PATH.resolve(__dirname, 'config_load')` можно сохранить в переменную и использовать 4 раза
Chybný styl. Ale i já tu mám chybu. Mělo by být "Probíhá aktualizace katalogu", protože to není použito ani v záhlaví ani ve stavovém řádku, takže "probíhá" nemá být vynecháno.
I split the commit in two.
On Monday, April 16, 2012 05:20:19 AM you wrote:
> isodd() and iseven() are defined in intfuncs.jl, although aren't documented
> yet...

Good to know, I just pushed a changed version.

--Tim
I am not sure I like this idea. This feature of katello supporting utf - 8 username is being tracked in https://bugzilla.redhat.com/show_bug.cgi?id=871537

So to me this is unnecessary.. Not a blocker though..
This can be written as:
```
repo_file.write("""#
# Repository for the locally mounted katello repo
#
...
enabled=1
gpgcheck=1""")
```
I thought that I sent this during first review, but I do not see it there. I probably forgot to hit submit button.
Is the time tag valid html5? I don't think I've seen this one
up to you, but i would have a function for each of the ways to compute the locale, and then write this code as:

```python
locale = from_data(data_locale) or from_alias(data_locale) or default_locale() or windows_default()
```

with better names for the functions :)  I'm fine if you prefer your version though.
I doubt caching it helps a lot, as the values need to be copied into the output Object anyway. Doing that directly is probably just as slow as copying it from a cached value. That said, I doubt it's the only place where we use allocated objects in static variables, so I don't really care.
How about this?
 if (GetBoolArg("-shrinkdebugfile", !fDebug))

Don't forget docs:
 "  -shrinkdebugfile       " + _("Keep debug.log file small (default: opposite of -debug)") + "\n" +
Any reason to refer to vin[0] specifically?
Compiles and works just fine, I'll update with this change!
typo -> attribute + I guess it should `property`.
Mind removing this chunk from the patch?
@stof are you sure these replacements are required? He's using `realpath` anyway...
`Then` statements tend to be nicer when you use the word `should`. So in this case: `Then "fake@email.com" should be a member of the group`
Can we change this wording to "Proposal titles should be clear and specific, so people know exactly what it means if they click ‘agree’ or ‘disagree’."

Sorry, I should have mentioned this on the Trello card.
If I'm not mistaken, ``self.connection.ops.spatial_version`` (and other version infos) are already computed when ``ops`` is instantiated, so we are just accessing the value here.
Right. I kept the variables that I knew they would be needed in the future,
but since I thought the JIT was completed, I didn't think they'd be needed.
Please re-add them then. :)

Le 4 nov. 2012 23:34, "Henrik Rydgård" <notifications@github.com> a écrit :

In Common/x64Analyzer.cpp:

> @@ -31,12 +31,9 @@ bool DisassembleMov(const unsigned char *codePtr, I...

I'd rather you commented them out, at least the ones that look like they
could be semi-useful in the future :)



—
Reply to this email directly or view it on GitHub.

[image]
Accept a pull request ;)
There's no way `__name__` could be missing too, I hope?
NameError: global name 'IPython' is not defined
Did you want to make this change?
Did that in the docs block for `NISegmentedControlFormElement` as this is where you would normally set this array. Should this be repeated here?
This is to have a little padding to the right, it wouldn't look good if the segmented control sits directly on the right edge. frame.origin.y should no get that big. Maybe it's better to rename `kSegmentedControlVerticalMargin` to `kSegmentedControlMargin` margin and use that instead of origin.y?
I know docurium doesn't handle `@deprecated` but I thought we needed some way to mark this.
Officially: ``?clear=1``
Are we not using Django's TestCase? `self.client` is setup for you already if you are.
The search box should not show on the search results page so we would still need the if-statement. Or we could put it in a block that we blank out on the results page.
If you look in `questions.js`, there are a few selectors that require a `#question` wrapping the top question area. If we add this to the new template, a bunch of the ajaxy stuff will just work hopefully. Probably will need some style tweaks.
If the js rewrites the field, I think your idea to put a "dummy" date is ok! :)
Depends on when ready is set. If it happens also for a failure case then another check would be necessary. But that happens with `expect.ok(!failure,` already, which should then be an assert too. If the file isn't existent it would be a hard-stop for us.
I updated this part. What do you think of using ``else:`` here? Is it a good thing or overkill? As this part of the code is common to most PostgreSQL-related functions, I'd like to find the very best way to write this.
Why are these imports here? Did pyflake not catch that?
Should emit error event instead if there are error listeners.
Github put this in the wrong line again :(
Anyway, this refers to engines/pegasus/neighborhood/caldoria/caldoria.cpp:1931
the comment is:
"// * Doesn't work yet, need global movies."
Nit: Use `!= -1` rather than `> -1`. Doing `> -1` is less precise, and it implies a bunch of impossible scenarios that can't happen, like read() returning 0 or -2.
wrong change?
Well, why would the angle be different on one case and not in the other ?
Yes saw it just now
I think it's `public static function` now following PSR-2
It wont delete the ones after, that is not possible in PHP. Any locked buffer will prevent access to any buffers stacked under it (there exists ugly hack to access them, but ugly I said).

The $level !== $previous check will just prevent an infinite loop. If a locked buffer is met, then we cant delete it, then we exit the loop.
Nobody should use locked buffers, but PHP uses one : the gzip one. We got Notices without this patch with zlib.output_compression=1 here :)
shoudln't this interface be in the Mapping subnamespace ?
Thread.sleep?
to hight => too high
Extract interface EntityManager and call this StringEntityManager
The boost docs suggest that extension() can return the empty string - in that case we'd better check s.empty() before accessing s[0].
The tolerance just became five orders of magnitude larger... does that matter?
Didn't do this (its just a comment and my personal preference is to break the parentheses mid way (so

that you know the comment has wrapped and meant to be read together).
bergerkiller loves his hoes
There goes the workaround for the buggy USB-UART chip!
@dominictarr I have experienced bugs with browserify not behaving when doing `require("../..")`. Consider me paranoid
same issue with the cast instead of assert here
we still need to store the ID.
nit: indent is wrong
Can we put this line at the top of `findCallback` in order to avoir tu duplication with line 318?
another unwanted change.
I believe we should make an helper for those. But that can be a different pull request.
I didn't know 5- and 6-byte sequences was removed from the spec until I checked Wikipedia!
You could remove contact[0].name != '' because empty string would also be false state.
Cool, cleaner than before!
You are right. I have removed all of them ( I hope so). If we only had a good debugging tool ...
D'oh. It should! Fixed.
`CONST` is a keyword and should be lowercased by convention :)
Not really thinking of an attack, only the fact that whether the settings are stored as json or anything else shouldn't matter to the user,
But I guess just bubbling the exception isn't so bad.
Not sure what you have replace here is the exact same results.
Hrm.  Good question.  My thought was to output to the Razor log.

I guess one warning per API call isn't too terrible. :)
Can you stick an extra set of parentheses around the second ternary operation (the "ip6.valid" one) to be a little more explicit about the intended precedence here.
no magic numbers -> move 0.94 to its own constant at the top of file
could you please add the space back - this fails without it
Make an issue for this on GitHub, so at least we can be aware of this as an engineering team.
Once again, we need `threadID = 'new'` to do the error check later.
Also nulling `threadID` later will strip it from the POST data, which is always a good thing.
Is there a reason we need to cache this?
That seems more complex to me, and less explicit about what's in the resulting data dict.
I'm not sure myself at the moment. For those wondering why this is being done here, PHP doesn't check for both the CN and SAN entries, it only checks the CN. As a result, PHP's internal handling can fail a connection even on a valid certificate for the current Host which is a serious reliability issue since SANs have been accepted practice for years now.
if we use a find here, it should wait a while to find the nsfw posts
I still think `-e` it's necessary, otherwise it won't stop at the first failure.
`preview_project_notes_path(@project)` maybe?
I'm starting to wonder if this API is better than the one I suggested above for user-level components.   I'm leaning toward the one above....what do you think?  *SORRY!!!*
this link doesn't go anywhere
no worries, just makes reading the diffs/blame/history later confusing  ;)
Thanks for making that unit testable.
making this change...
Yes, it works. I already push the changes.
Oh yes... No problem then, I thought it was a change in the `composer.json` file, not the `.lock`.

Thanks Jordi.
I just meant to the "validate_options" call itself, like in the previous method.  :)
这三行处理方式不一致，确认是否某一种是错的。
You're right, forgot to clean it up.
As a rule of thumb, always handle timeouts whenever they can happen.
could use change this to single quotes - thanks!
`è prendere` -> `consiste nello scaricare`
```php
$validator = new NoRecordExistsValidator('users', 'field1', array('field' => 'id', 'value' => 1), $this->_adapterNoResult);
```
Confusing; keep all elements of an array on the same line.
dedent "return $logger" 4 space
No, because if normalize is on the result of RstConvert::xmlFileNameToRst contains the extensions .rst.
I was looking at the Di component for modern documentation conventions, and noticed that none of the constructors report return values. I find it superfluous.

Is this covered in the coding standards at all? I didn't find a reference to it.
Yes, of course there are no traits. It's just what my error log said, and I didn't give it a second thought. I guess it's the Closures that cannot be serialized.

"If they're not equivalent, what happens on wakeup? Will the filters be repopulated? if not, is serializing a filter chain a good idea?"

Actually, I think that filters are already populated at this point, so this would only cause problems if filters were changed after unserialization. I need to check this when I get back to my computer.

Could be that there is no perfect way to handle serialization here...

--
Juha Suni

On 31.8.2012, at 21.42, Matthew Weier O'Phinney <notifications@github.com> wrote:

> In library/Zend/Filter/FilterChain.php:
> 
> > @@ -224,4 +224,18 @@ public function filter($value)
> >  
> >          return $valueFiltered;
> >      }
> > +    
> > +    /**
> > +     * Prepare filter chain for serialization
> > +     *
> > +     * FilterPluginManager cannot be serialized, as it contains Traits. For
> > +     * this reason property 'plugins' is excluded and thus serialized
> > +     * filter chain is not necessarily equal to current chain.
> If they're not equivalent, what happens on wakeup? Will the filters be repopulated? if not, is serializing a filter chain a good idea?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Please do it and I will commit it
why not updating to 1.9 as it has been released ?
This is a bit too verbose :) I think only the first two sentences are really necessary.
Only if key isn't exists.
The problem is when CookieToken and BodyToken are both undefined, the first clause matches.
The *verbose* name of the language. (potentially translated). not necessarily English. (Depends on user installaiton)
you should remove the extra ``"``
Import before from ... import
if not self.getcfg('test_only') and not self.getcfg('sanityCheckPaths'):
So basically what you are doing is relying on a system property that openshift always sets up?

Do we use this servlet now on localhost as well?
I don't see why you are activating CDI, you aren't using it.
nice comment, but i would actually prefer we don't put it here - these quickstarts target JBoss AS 7/ EAP 6, which is an EE 6 container :-)
(which isn't to say I agree with the issue as it was raised - it just seems to be another instance where the confusing dichotomy of Exception.new and Kernel.raise cause problems).
Change attribute name to "toolbar-location"
This Request is actually the one from .api (I know it's a bit confusing, but my Request class basically wraps the one from requests).
Is the < intentional? What does it do here?
If `setupFn` might not be a function, lets just call it... `setup` ;)
@CristinaRO - just added this Release Notes button in since you reviewed this. (mumble broke out the kitten eyes. I had no choice.)
There's a stray space between the opening quotation mark and the S in Status:
Reviews don't have to not touch pre-existing code. And moreover it's a 5-minute errand to run.
I was curious about how that was expected to work; I just duplicated the old connect-timeout stuff.
@jcredding - Yeah, I don't love something you have to do in your view.  Views should be distinct from the invocation of them.

* customize output before tests run
* show output while tests are being run
* customize output after tests run

In thinking a bit though, I guess you could make the view developer define 3 things (instead of the one `template`).  They could define `on_start`, `on_result`, and `on_finish` templates separately and then the runner would just call those templates at the appropriate time.  More of an evented approach to rendering output.  How does that strick you?  I'm not sure what I think about that.
This comment goes for the templates as well.
After merging, I think this needs to be refactored a bit. Maybe a dedicated class? You can just define that inline.
Same comment as above, I think throughout the hints, we should refer to the "diagram" or "figure" or something rather than "pile" since that term could be confusing.
OK. I didn't want to make it too big, because when combined with a large number of whole units it's a lot of counting to do. (43/8ths!)

Maybe I could make the total of these two numbers follow a bell curve, so you *either* have a large denominator or a large number of units, but not both.
If only you had learned with Khan Academy! Where we make horrible math expressions fun!
The name cannot be blank. If it is blank an exception will be thrown in the VdbModel constructor. This way the logic of which constructor parameters cannot be blank remains in the VdbModel.
We generally don't include JavaDoc on methods that are overridden, as the inherited JavaDoc is often sufficient. Same for the `equals` method.
As currently written, the logic *always* must construct a new string before calling the internal `debug` method. This string creation is done whether or not DEBUG is actually enabled.

So, the correct way to do this is to change this code to directly call the Logger (see earlier comment) as follows:

    debug(">>>>done sequencing model {0}\n\n", entryName);

The message string and (in this case) the one `entryName` parameter already exist, and will simply be passed to the `Logger.debug(...)` method as-is. When debug is not enabled, the debug method simply returns without doing anything (i.e., no string manipulation); only when debug is enabled is the string manipulation performed.
For both these methods, does it make sense to name them `from_hash` and `to_hash`? Their current names seem to indicate that they'd be returning a Hash or transforming a Hash into something else. If that is the case, I think they should be renamed.
Why commenting those lines ?
In the head tag!
In the interest of less code, and given the comments below, it might be worth trying to get rid of this entirely. Saving an owner with the upstream consumer set, it should get persisted automatically, we might not need an entire dedicated curator for it.
Good call. I had been looking at the logEndpoints above, which didn't have the prefix.
Use label id not a hardcoded label
We should check that the requested file doesn't exist, before changing its URL.
ok, I understood the part of the compile, but why use findall?
Shouldn't the api-version be set to 2 after the transform?
Using `readFile` makes more sense then require, I followed up on that in my branch.
My vote would be to continue using Content, etc.
What is the problem with this test? Why did you remove it? If there's another way to test the same thing, please update the test. It tests against a bug we fixed recently: the Scala library wasn't passed on the bootclasspath, and therefore it was ignored by scalac.
This looks much better than before!
I don't think this belongs here. Maybe ``ScalaProject`` object?
No "ip" command on the mac.
If we choose to stay mac compatible, we need to change this to something else, potentially a conditional change of some sort.
hm. we could follow the model of core symfony Router here and inject a class name to instantiate and composition the routing. not sure if that is really a cool idea. 

the problem with injecting a service is that you can't change the route collection after the matcher is created, so i think this can't work. inject class name and do as in core symfony Router? see https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Routing/Router.php#L223 (minus the whole cache handling i guess, as we have route collections on the fly)
I didn't know about that one. Useful!
@dmitrykh вы меня простите, просто для меня "такой метод есть и класса инстанса" означает, что метод с таким же названием есть и у объекта класса, чего быть не может.
16 лет на Дельфи. Компилятор бы там такое не пропустил)
Если вы знаете будущее количество элементов массива, то лучше инициализировать его через arrayWithCapacity.
Тут лучше использовать вещественные числа и потом округление
Получается, что нет проверки, а вдруг новый нуклеотид совпадет со старым и тогда не выполняется условие задачи: заменить СТРОГО X процентов.
кажется в NSMutalbeArray есть функция проверки содержит ли массив указанный объект или нет, можно было использовать её (вот только не знаю как она сверяет объекты - по указателям или сравнивает содержимое)
Времени исправлять уже нет. По какому принципу ставил оценку? "Метод mutate: создан в отдельной категории, он ЗАМЕНЯЕТ СТРОГО X процентов" 0/2
1)создан в отдельной категории (создан в отдельно)
2)ЗАМЕНЯЕТ СТРОГО X процентов (тут ошибка)
should there be something here?
either that, or the import above might not be needed...
This should have a `getter = isDisposed`.
Sorry for the back-and-forth on this code, but the shared testing here can be done solely through the use of `describe`, `beforeEach`, etc. Shared examples are mostly useful for offering tests that are available _across_ specs (e.g., for different classes).
Method here too.
It's probably better to type this as `IEnumerable<PushStatusUpdate>`.
Actually, I think @JaviSoto's recommendation is good. I'm going to implement that and try it out.
I would get the picks that meg or eeg as then do a np.setdiff1d with np.arange(n_channels)
you can also have misc channels, Continuous HPI ...
I think function name should start with a verb (start with actions).
It's picked up magically (the attribute name is the key to the spring context file).
I don't like that we loose history here and it looks as a new file. If you first change the name to DataExportIT and then commit without creating a new DataExportTest, git should now it's the same file and we preserve the history.
You will have to copy/pasta it to use it, but I think the approach is maybe a better one.
Gehört das nicht unter §4.3 anstelle der alten Zeile 145?
Switched to Q which has Q.resolve()
please use ``:doc:`/cookbook/email/spool` `` for consistency
当<link id="proxy_pass"/>指令使用变量时，<literal>default</literal>参数不允许使用 。
ngx_http_access_module模块
for some reason, that didn't work out so well.
symbols tuples -> limits
this could be simplified to:

```
return ' '.join([os.path.split(name)[1] for name in val])
```
Why do you use repr?
No porque por cuestiones de formato tiene que estar dentro del Form
Is the `type(list[1])` necessary?
I originally did that, but cut it out when I was hoping to do non-exhaustive reads to determine metadata. I'm really hoping to avoid doing exhaustive reads, since I'm interested in applications where a single read is about as costly as the entire model fitting process.

But for now you're probably right.
If you're using the constructor why do you need this?  Please *DO NOT* add a setter to the interface.
Probably can make this static.
This accounting for the first row could also be done by adding the line

```
runinfo_csv.next()
```

between line 115 and 116 (before the loop).
это убью, конечно. просто попробовал, что все работает )
fixing the problem below and sending another pull request.  attatched sql
for state table

On Sat, Apr 7, 2012 at 8:47 AM, Andres Castro <
reply@reply.github.com
> wrote:

> > @@ -50,7 +50,7 @@
> >
> >  $db['default']['hostname'] = 'localhost';
> >  $db['default']['username'] = 'root';
> > -$db['default']['password'] = 'root';
>
> Need you to change this back to root or add it to your .gitignore file so
> that it doesn't mess with everyone's database settings.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/acastr7/ProfessorVote/pull/9/files#r657754
>
@nlegoff tu me confirmes que ce changement est bon ?
Oops sorry, I get errors so I added it temporary and forgot to delete it... :-/
I think I know what you're talking about with the login issues, but I think I pinned it down to network connectivity issues. Pandora seems to be super sensitive with the syncTime key when it comes to logins, and I think most of these bugs could be resolved with some automatic reconnect attempts. I want to build an error handler for Pandora's error codes to better resolve what issues we're coming across, so before you do anything let me get something up first.
yes, I know that's invalid base64 data too. Apparently at some stage before outputing the data uri, the code is encoding line breaks as &lt;br&gt;. As you have generated the sample dictionary, is it possible that the base64 string is including line breaks in the dictionary? Line breaks, as &lt;br&gt;, are not valid in data uris (source Wikipedia):

<pre>
In Mozilla Firefox 5, Google Chrome 17, and IE 9 (released June, 2011), encoded data must not contain newlines. 
</pre>

So this might be the case for QtWebkit too.
pls remove superflous newline
see my previous answer, I'd have to write more code to keep the list of futures and then iterate over them etc...
Regardless of whether we should create temp caches or not for map/reduce, the default for a generic CreateCacheCommand should be to use the default cache's configuration.
You really should be defining a new executor in the global configuration to control parameters of this executor, in order to be nice with AS7 :).

For those cache level executors, things are a bit more fuzzy as shown in https://issues.jboss.org/browse/ISPN-1396
Yeah but I like them :-)
ok. but the variable naming statement still holds (```fname``` could be a file handle or a filename).
Ew.

Why can't you use logic like you did on y to overlap by the size of the toolbar? All of this platform{Width,Height} insanity looks bad.
Are we scheduling this for removal in the future, if so it might make sense to add a TODO? Or will we always keep it?
`flatten` is usually never, never right. Try using mapcat here.
Hmmm, I tend to use ++x , no particular motivation. I'll fix that.
Every time someone uses !important, a kitten dies.
Ingo told me it needed more comments, so I added more comments. ;)
The reason is just cleanup, as this is the only place i found in the whole sugar CE code where this variable is used at all.
is this comment documenting the format, or just old code?
Fixord.
It would help to have a docstring describing what is this object represent.
Very intelligent comment, Fran!
- Are you sure the variable ``$return`` exists, or did you maybe forget to declare it?
- Are you sure the variable ``$stream`` exists, or did you maybe forget to declare it?
Is it possible to define the element type for the array of parameter ``$listeners`` (using ``array<SomeType>``, or ``SomeType[]``)? Please also check line 215.
What should we do with invalid javascript variable names that contain dots or slashes, should they be sanitized by our ViewHelper or just throw an error on the Javascript side. 
I vote for a simple preg_replace which filters nonalpha characters which should be sufficient.
In the html Api docs it was showing to be supported for Blackberry that's why I allowed opening it. 
Looked Jira task there is no task approving this so modified code by adding 
//TODO remove this code when Ti.UI.createTableViewSection() will be implemented for Blackberry
	if (isBlackberry) {
		alert('Ti.UI.createTableViewSection() is not supported for Blackberry yet');
		return;
	}
in the top of the tv_api_sections() function
right. no wonder all the meda disappears for all channels!
why not allowing expressions for the name ?
Две таблицы для того чтобы создать связь один ко многим. Чтобы пользователь мог заказать несколько блюд в одном заказе. Если сделать один order detail то будут дублироваться данные
Sure. We've got a good reputation to defend :)
are you doing some stof works ?
I think I need to move this on outer scope to avoid memory leaks too.
I proposed that because I think that a value class is a good tool to enhance existing classes with implicit views. If I got it right, no new instance haas to be created if the new interpolator is invoked. If that is not the case, then one could pass on using value types.
The constraint on the Children annotation is "*item", I'm just appending '-item' to make it look nicer ...
Pourquoi ce changement dans le gitignore?
YES!!!

Rename to `USER_CHANNEL_ID`. It will not be used only for notifications.
You could make this class-method of Node object.
But good idea to do it with a tag.
`get_supported_languages()`
I don't like the idea that you assume here how ids are generated. You should retrieve this in some other manner.  Maybe store field if into data attribute, too?
Well the code would be like this if it would be placed in tastypie-mongoengine.  In MongoEngine I could not use this import so I had to copy/paste those constants.
Fails at run-time.
Of course, choices should be tuple of tuples, not dict. You have to transform it.
The contents of the `wrap` array is a number (depth) and strings that get concatenated around `elem` during construction and set as innerHTML of the `div` that gets created and appended to the DocumentFragment. The while loop reassigns the same `div` to the correct node at the given depth. This is the same `div` that ultimately gets removed. 

Here is some data to help illustrate what I've done:


<table>
  <tr>
    <th colspan="3">
      Subject
    </th>
  </tr>

  <tr>   
    <th>Observing</th>
    <th>Before</th>     
    <th>After</th>   
  </tr>
  <tr>
    <td colspan="3">
      http://localhost/test/?filter=manipulation
    </td>
  </tr>
  <tr>  
    <td>safeFragment.childNodes.length</td>  
    <td>168</td>  
    <td>27</td>          
  </tr>  
  <tr>
    <td colspan="3">
      http://localhost/test/
    </td>
  </tr>
  <tr>  
    <td>safeFragment.childNodes.length</td>  
    <td>309</td>  
    <td>36</td>          
  </tr>  

</table>
The naming of the include file appears to be unlucky.
Do you have resharper set to change this? I'd really, really, really prefer having usings inside the namespaces. Really.
Ugh. Betting it's not blizzlike.
Yes but because of the number and size of files I use rsync. Curlftpfs
let's me mount a volume and then use rsync to check which files were
changed and only download those files.

On 2012-09-08, at 1:10, micheldumontier <notifications@github.com> wrote:

In pubchem/pubchem.php:

> +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.
> +*/
> +
> +
> +/**
> +* @author :: Dana Klasen
> +* @version :: 0.1
> +* @description :: The Pubchem database parser
> +**/
> +
> +/**
> +* REQUIREMENTS
> +*	1. curlftpfs

PHP has curl functionality
http://php.ca/manual/en/book.curl.php

—
Reply to this email directly or view it on
GitHub<https://github.com/bio2rdf/bio2rdf-scripts/pull/147/files#r1559955>.
Nope I'll move that
I believe you need this data ONLY when making a new payment url. i.e. in ```get_gateway_url()``` method.

Leaving it in __init__ will cause that every instantiation of PaymentProcessor class will require fetching this data all over again.
I'd like to introduce you to ```str.consume``` :)
Did you consider rack middleware? It'd be nice to do this without monkey patching any rails code (e.g. is that hook point stable across all versions of rails)

The tricky part there would probably be surfacing the user_object to the middleware.

Alternatively, can you modify the response.body in an after_filter? (as a way to avoid monkey patching)

I guess a caveat with this will be for pages that are cached for logged in users, that's kinda unavoidable tho? right?
smb-check-vulns is said to be dangerous: http://nmap.org/nsedoc/scripts/smb-check-vulns.html. Although there's a default set of 'safe' checks, we might want to validate with vulnerable targets in our lab.
Actually, attempt mktemp, on fail, detect os and use the above?
What's the reason for having an RCFile-specific loader, rather than hooking into the MultiInputFormat-based loader? Having a single loader would make things much easier for users.
Why conf needs to be passed is not obvious at first glance. Seems like passing FileSystem makes more sense here.
Renaming Swagger Controller to DocsController allows this route to work under default webApiRouting of api/{controller}/{id}
I insist that the new `onResolve`/`onReject` are _not_ normalized forms of arguments `onResolve`/`onReject`. Using the same name for two functions that have different signatures encourages misreadings.
Adding `var messages = window.messages;` a bit further up.
Addendum: maybe spin on EINTR, EAGAIN and EWOULDBLOCK but consider everything else an error. Still, spinning is rather anti-social.
Is this if necessary? it seems to be doing nothing.
For the declaration, this should just be `void SetLights` rather than `void State::SetLights`. I would also move the implementation into the .cpp file, where you will need to add an include for `Light.h`.
You could use *content_tag* to avoid *html_safe*, like this:

```ruby
content_tag(:iframe,"",width: '640', height: '360',src: url, frameborder: '0', allowfullscreen: 'allowfullscreen')
```
Done. And yeah, it's a neat trick.

``sorted`` works on any iterable where as only lists have the ``sort`` method.  
So you could get a tuple and run it through ``sorted``. The only problem is 
that it returns a list and not an instance of what it was passed so for tuples 
(assuming you only want tuples) you could do: ``tuple(sorted(iterable))``.

Also as you can surmise, it keeps you from running sort in place.
Write space after `:`.
Hmmm, but then a noun is missing. I'm pretty sure about that. By the default "what" ? :)
I think we can remove this commented out domain.
1911990 is not a good password. Your birthday?
Surely a role is the join between a user_id and a module_id(+version) - if so then changes to an existing role will be PUTs
(Note Rachel... In the pysrc wrapper the keyword for conserve flux is `fluxConserve`, which goes against our lower case + underscores keyword convention.  I'm going to change that to `conserve_flux`, so line 370 will become

`l5 = galsim.Lanczos(5, conserve_flux=True, tol=1.e-4)`

Sound OK?
This seems to be the same solution to the non-linear equation that you mentioned above, and which is in the pysrc wrapper.  However, in both of those places, the value is 1.6783469900166605.  More digits of course, but the last digit seems to be in error here.
since the new downloader has no downloadbar... and isn't fully stable i guess.. this should be false?
Do these files new need to have names of a particular structure? Or could it use File.createTempFile?
Guava's Strings.isNullOrEmpty does exactly this.
You should use this:
http://api.drupal.org/api/drupal/includes!common.inc/function/url/7

It is safer then concatenation, especially when dealing with translated sites.
separate commit + PR
Name to() is a bit confusing since this method not only delegates but also calls the closure?
s/type/send_keys
Did you push the code? Because I don't see that changed?!
Eek, that string should be a 404 and an exception... /me confused
One sec, making a pull request with my changes.
Because it's used in more places than that.
this shouldn't be there I guess
Should this be `data_all.forms[i].parcels`? I.e. `i` instead of `0`?
UMD does this (set site with api/v2) because serve does this

```ruby
  45   galaxy_client {
  46     version   "v2"
  47     path "api/#{version}"
  48     site "http://serve.offerengine.com:3001/#{path}"
  49     user      "serve"
  50     password  "serve"
  51     format :json
  52     timeout_length  5
  53   }
```

chase the same thing
Пробелы вокруг =
You have to configure Oracle before starting the server. This section should move up to #8 above.
what is dis hek ??
If we're using `compressor` we should probably drop this?
文書が硬いようなきがしました。以下ぐらいのやわらかさでも良いかも。

自動化テストはGrailsにおいて重要な機能です。Grailsでは単体テストから機能テストまで、簡単にテストを作成できる様々な方法を提供しています。この章ではGrailsがテストのために提供している機能について詳しく説明します。
cause it should be libnextpvr_addon_la_SOURCES
Is this the correct Bio-Formats link?
You're right, I overlooked the changes in the loop.
can you remove these two lines ?
ти перевіряв чи тест працює?
Changed back to `info`
Oh, that's an idea, netbeans lets me do that easily! :D
Ok, will do in the evening :)
I have a TODO for that above
Use statements instead of this. Also, we should probably rely on `Doctrine\Common\Persistence\Mapping\ClassMetadata#getTypeOfField($fieldName)`
no reason other than I just changed the indentation of this block.
Copy&paste typo on (&&), should be (or !=). Late night patching... glad we have code reviews.
I know and I don't like it either, but this CS is used throughout the extensions repository. I can easily remove the prefix only from the method I've added but this will be inconsistent with other method names of the same class ( _getAttribute(), _isAttributeSet() ). Let me know If it's ok and I'll fix this and phpdoc issue in the other note.
line too long (pep8)

Also, I'd like to avoid hardcoding channel names like this, but I see we do it elsewhere so won't ding you for it.

The DB has a "release_channels" channel which has the channel names, and Socorro has an assumption that there are 4 primary channels:
Nightly, Aurora, Beta, Release

Some other systems might call them something like:
Unstable, Alpha, Beta, Release

Or whatever. I'll file a bug to work this out since this isn't the only place, just something to keep in mind.

</rant>
You need to add this ResultOperator as pass-through because it may break some rewriting scenarios
Have no idea, it's not the new code, I just moved it from request.js to here & did not wanted to change anything.
Please write the layer with the lowest priority (1) at the bottom, and the one with the highest priority (8) at the top.  Like this, the interpretation of the _drawOrder section becomes easier.
@Flukey ?
@shacker .img still seems to be in here.
Yhdessä `let`issä voi olla monta nimi-arvo-paria joten molemmat `letit` voisi tässä yhdistää yhdeksi `letiksi`.

~~~clojure
(let [[ranksort ranksort2]
        [(sort (map rank hand)) (sort (replace {14,1} (map rank hand)))]
      [pienin pienin2] [(first ranksort) (first ranksort2)]]
~~~
Just `(> (author-count book) 1)` is sufficient.
These print statements are a syntax error in python 3.
return array[++index]
Wrap at 80 cols, please
In the past we've had different answers from msf devs.  Some favor report_auth_info, some prefer store_loot.  According to Todb's documentation about report_auth_info(), he implies that when that function is used, there must be a username/password and that it must be sure.  If only "likely", and I translate that as "not sure", then store_loot() should be used:

"report_auth_info(): Used to store working credentials that are immediately reusable by another module. For example, a module dumping the local SMB hashes would use this, as would a module which reads username:password combinations for a specific host and service. Specifically, merely "likely" usernames and passwords should use store_loot() instead." -- https://github.com/rapid7/metasploit-framework/wiki/Acceptance-Guidelines

So as far as I can tell, zeknox's decision is probably based on that statement.

egypt's opinion: "a username is useful on its own, but obviously much less useful without a pass."  So what I'm gonna do is keep report_auth_info() for now, and then at one point I'll let Todb clarify on that again.

However, store_loot() has its own problems -- when there's too many files, they're difficult to manage. So ideally it's best not to use it in a loop.
I did s/inverse_map/inverse, but

1. Why should this docstring refer to ProjectiveTransform instead of base class GeometricTransform?
2. (Why does warp code use the variable name inverse_map, instead of inverse?)