labofsahil/github-event-dataset-2012 huggingface dataset:
done, I ran THEANO_FLAGS=mode=DebugMode,DebugMode.check_preallocated_output=ALL python test_basic.py and there are no warning related to CSMGradC, however, there are some for the other ops in basic.py.
It could, but would be less performant. Changing the format allow to return a view on the input data.

Can you check the transpose op? It don't tell that it return. Can you check that we it do is to not return a view? The scipy doc tell is return a view by default... http://docs.scipy.org/doc/scipy/reference/generated/scipy.sparse.csr_matrix.transpose.html

So there could be a bug in our code here.
There's another connect(timeout) method that takes a timeout value, does the session timeout work here? If not, we might want to put a timeout here as well.
I will look for other sources to confirm the EXCEPT ALL behaviour tomorrow
1. put Meta directly under field definitions
2. missing ordering attribute
not kwargs[field].strip() just to be safe?
what is mgr used for ?
`treeish` -> `revision`
Without the clone that method was definitely polluting the cache. I think the problem was that when the new model was created, it was resolving some of the ForeignKeys and doing the resolution/modification we discussed above - perhaps that would go away when the don't-edit-other-models flag came in but, as we said further up, that might not be in this patch...
That, or check that the error is about not having the table in the DB in the first place. The database error could be caused by multitude of other error conditions, too, and swallowing those errors could lead to silent table leak...
I don't like assigning to a new variable, (~qobject).matches should work too. And the issue is IMO not that a new Q object is created but that attribute look up comes before bitwise not in the operator precedence list.
you should remove these extra lines
Avoid one-letter variable names, especially when they are going to be used as keys.
No big deal. It was worth a shot! Perhaps we can move the parsefloat stuff to curCSS... although it would be dfined twice, it may still be a win.
If one of the parameters is present, the other two parameters should always be present. Otherwise, we need throw a error.

I will add more check and throw appropriate error messages here.
Replaced by almost-equal testing.
Cool, yeah I'll let you make some more progress and we can have another round of comments. Let's make this happen! :)
The possible return values are (taken from libspotify's API):
  SP_TRACK_AVAILABILITY_UNAVAILABLE = 0, ///< Track is not available
  SP_TRACK_AVAILABILITY_AVAILABLE   = 1, ///< Track is available and can be played
  SP_TRACK_AVAILABILITY_NOT_STREAMABLE = 2, ///< Track can not be streamed using this account
  SP_TRACK_AVAILABILITY_BANNED_BY_ARTIST = 3, ///< Track not available on artist's request
When we update the locked field on admin page it complains about these values being blank (if we don't set blank=True)
You mean we can avoid cmake? Two birds, one stone!
I think you mean 421, and also a `cause` should be specified (e.g. the error message that is produced).
No reason for these to be keg-only.
Highline list can help a bit here, but the general answer is helpers.  Lots of helpers.  And preferably generic ones we can reuse from an existing library
J'avais une erreur c'est pour ca que je l'ai mis en commentaire
Good point.  Pushed a new commit for this.
I'm -1 on on defaulting to `0` if not set, can see this causing confusing in production. I'd rather have it crash?
Is there a way to avoid calling a protected method of lxml.html?. As this may raise some compatibility issues on future/past lxml versions.
I did consider that approach. Bit of a faff for the parent application to require uglify and pass it in on a per file bases. Not sure what is best, I'm inclined to  say leave it as is and we'll see how it goes.
As I understand it, processingOver can only be true on the first pass if no code generating annotations are found for any processor. If that were the case, this processor would never be called.
I firmly disagree... maybe you think they shouldn't be hand-rolled (I agree), and that we, as an effort, shouldn't "support" it (which we won't), but the class loading will entirely work if someone hand-rolled their own that was to-spec.

After all, hand-rolled is just generated code that was generated by a human.

But I'll rephrase to avoid accidentally "approving" of hand-rolled.
I think we should leave them there for now - the this will make it clear what terms are taken from 24751 and which ones we've made up ourself... and I'm not only saying this cause I'm lazy and afraid to break stuff if I change the -provisional-s
Is it a good idea to call code outside our control (the listener) while holding the lock? In particular, it says this in the Javadoc for `ReentrantReadWriteLock`:

> Additionally, a writer can acquire the read lock, but not vice-versa. 

So if the listener has a side affect of adding a listener, we will block.

I think it would be better to make a copy of the listener collection in the constructor of SafeNotifier
I will try to move the cycle to a method.
`itemid` should be `data-item-id` to make the HTML5 gods happy.
I've been thinking we should show the load screen in mobile web as well. Currently, it just shows a white screen with a spinner that you can't see. I don't think it needs to be handled in this pull request though, so this is ok for now.
How often should the number of lines be updated? It doesn't seem to update while I am editing a file, even after saving.
Actually, a broader thought on this. It seems kind of bad that we're reloading this rule data on every change event. (I know we're already *sending* the new style sheet on every change event, but every little bit of work we can avoid on typing seems worthwhile.)

If the browser already knows all the textual offsets for each rule, could we just send the offsets to the browser and have *it* calculate the location in the stylesheet? Or is the stylesheet info accessible via `Inspector.CSS.getStyleSheet()` richer than what you get from the CSSOM itself within the browser?
This export is unused. Is it safe to assume register() is the only creation point for Commands?
Might be good to clarify (document) when this case is expected to happen.  Is it just on first launch?  (Or I guess, any contiguous string of launches from the first one up until the first time we successfully fetch & cache update info).  Or could it happen later, too?
Add docs: Extracts jasmine.log() and jasmine.expect() messages from spec results
We have a variable now actually, `window.cloud9config.hosted`, that when set to true means you're on c9.io or our different staging environments. This way you don't have to use a listener at all~
oops this comment actually applies to the previous loop (well both loops)
Same here, no space between id and <XMLRPCEncoder>.
Please make the level headers generic.
I hate this form. What is wrong with from risklib import models and then using models.input instead of input_models?
jajajaja... security... security my amor
This is not correct. Wrong import. And import only modules!
No, this is only inside butter. We will eventually have to deal with popcorn code, but for the purposes of this editing code, popcorn is always within the closure.
Leaving `true` here might not have been such a bad thing. Why not persist the plugin-list too? Does it not work?
I've been separating different kinds of variables because (a) there is a comment on top of the first one, explaining its purpose, and (b) it's a different kind of variable altogether -- namely, an instance variable, right (hence the `_`)?

Also, add some vertical whitespace here. Friendly reading :).
This looks wrong. Is it?
`( function` -> `(function`
Nanoseconds can overflow here. You should process `get_elapsed_by_childs()` result to fix this.
It'll work, and it'll properly log the exception, but in Python 2.x exceptions don't have stacktrace information associated with them, so passing the exception has the side effect of twisted not creating a failure from the current exception and capturing the stack. 

The existing code works, but doesn't log the whole stack.
Capitalize S and add a anding dot. (Yes, I'm a pedant perfectionist ^^)
Will check - but I think I remember there was a reason for this. If so I will add a comment explaining it.
junit-dep should be picked up transitively from litmus-testsupport.  Please do not duplicate these core test deps in projects, as it makes version management a pain.
这里的函数顺序搞成自顶向下的吧，这样便于代码的阅读，也符合nginx的风格。
"version of nginx (required %ui.%ui, but found %ui.%ui)."
why use 7?
Tohle je v překladu dvakrát, pokaždé přeložené jinak, i když jde o jednu a tu samou věc.
Made changes. Didn't see a NotImplementedError type exception, so I'm just raising `error("not yet implemented")`.
Foreman is creating its admin user by itself. With orchestration enabled it would fail on admin name already taken in Foreman. I'll investigate for other possible solutions.
Ok looking at the controller a bit further .. Looks like this is not possible since parent_repo is the library repo and those rows always are based of library repo,
Yeah tots, go for it. Was just a quick hack to fold in the ST_GeomFromText
stuff.


On Mon, Nov 26, 2012 at 9:49 AM, Robin Kraft <notifications@github.com>wrote:

> In src/clj/fossa/utils.clj:
>
> > @@ -53,7 +53,7 @@
> >  (defn latlons->wkt-multi-point
> >    "Returns WKT MULTIPOINT string from supplied lats and lons."
> >    [lats lons]
> > -  (let [out-str "MULTIPOINT (%s)"
> > +  (let [out-str "ST_GeomFromText('MULTIPOINT (%s)', 4326)"
>
> Could we revert this to return only the MULTIPOINT (...) string? The
> function is only supposed to return a valid wkt string (see name and docs).
> Including ST_GeomFromText breaks tests. We could fix that, but then the
> function goes beyond that orginal, simple, specific scope. A simple wrapper
> could add ST_GeomFromText, and be called in this function<https://github.com/MapofLife/fossa/blob/d0474cbd7d2955c342d20e0a1ec5ed6063ebee81/src/clj/fossa/utils.clj#L302-311>
> .
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/MapofLife/fossa/pull/33/files#r2225015>.
>
>
Use TaskAsyncHelper.FromError
I don't see how this could be correct or useful. The analogous code on DistinctNode is only (or not actually) called specifically on a DistinctNode by a method of a PushdownReceiveDominators class which is never instantiated. It takes the distinct node and either copies or moves it to below the receive node regardless of any nodes in between -- which are possibly very significant -- there could be an Agg in there!
It assumes that an exact clone of the distinct can be patched in below the send -- something I generally doubt.
What does any of this have to do with UNION? Are you planning on enabling this code and expanding it so that a UNION is going to be patched in below a send node? How is the send's requirement for one child node going to be reconciled with the UNION's requirement for multiple child nodes? Your code gives a nod to this by aggressively cloning child instances -- though as far as I can tell the clones are unattached to anything and instantly unreachable and ready for garbage collection. But even if you tried to retain them as children of the copied UNION (?), where/how does the SEND node's original child fit in to make a plan that retains its correctness? I'm not seeing a win, here, just lots of open questions and issues.
This can *not* wait for a follow up, fixing it right now :)
you mean indent! Too much web intent in the morning is the cause of this symptom!
Really? I thought premake-devel is a place where premake 5 is developed, and this 'master' branch is a direct mirror of it.
I still need to review this PR, but I think that we should try and follow the convention we've used for other packages - in the docs we should have a 'getting started' section, a more detailed user guide, and then the API docs at the very end. I do agree that in this case it ends up looking similar, but for consistency, I would argue for including the bulk of the docs in the rst files, and keeping the module docstring fairly brief. Here's a template we've been trying to follow for other sub-packages: https://github.com/astropy/astropy/blob/master/docs/_pkgtemplate.rst
As best I can tell, it looks like you're sending a list, it takes a Dict, we could fix it to handle a list though, it'll just take another few hours on my end.
Talk to me on this, we should standardize on something...
k? better variable names please.
Should we throw an exception if not? Something seems to be wrong in that case in the calling code, no?
I don't think this needs to be a list; it can be a dict instead.
but lets not block the merge by this question, before we just had a 500 in that case, now at least we handle it. maybe open a new issue?
Just a quick question: is there any particular reason we don't allow creating objects without parent? I mean, with PHPCR this doesn't work, but others might have a more liberal data structure. Also, I'm not sure if TypeNotFound is a good name for the exception. The type we wish to create has been passed as parameter to the function, what isn't found is a parent reference, as the message states. But that's just a minor nitpick, AFAICT, it shouldn't even be necessary to throw an exception here. For phpcr, the mapper implementation itself should throw the error
Waar in de code gebruikt, mag je rechtstreeks de container aanspreken.
Please add parens because bitwise OR's precedence is always confusing
It would be better to integrate this with `set_env`
shouldnt you check that these below are null first?
Does this need to be a `TraitError`? I'm generally a bit wary of catching exceptions and reraising without the original details.
Oops, that's a bug in py3compat. It should be `basestring` rather than `str` on this line: https://github.com/ipython/ipython/blob/master/IPython/utils/py3compat.py#L35 .  You can just make the change in this branch.
Shouldn't the partitions be specified as Set rather than List?
Why the change in the Sort type?
Nit: I'd indent these two lines in.
I'd add a comment above this line mentioning what you're doing here.
Bah, I grepped for `for-os` and `for-version` and it doesn't look like we use that anymore. I'll file a bug to remove and save bits.
Is this since the last approved translation or since the last translation in any state including rejected?
yes.  rats.  I moved them all to individual files to try to emulate test patterns elsewhere, and forgot to remove from the original file.  same with ProductVersionResourceTest.  fixed.
ahh, ok.  I think it WOULD have been necessary if we'd used their apikey auth model.  I seem to remember something about that.  but, ok.  we can take it out.  done.
Not true. If ``params.uuid == ''` this exception won't be raised.
This SQL might fail if `len(dead_processors)==0`. Should we defensively add an if statement?
Don't mean to be an ass but could you put that comment into the doc string or replace the `_11` with that in short form. 
Nitpicking, I know.
`aswell` should be `as well`
Also perhaps something like `Array(@plugins).each` instead of `@plugins.each` to guard against things like `nil` or single-plugin declarations…?
Is it the HTTP client's fault that the service invokes a piece of code and an exception is thrown? Is it always the HTTP client's fault that the service thinks it can parse an URI, but it can't?

Is Hyprlinkr an HTTP framework or a helper library?
how about `user != 'admin'`, which is slightly clearer.
This doesn't seem to serve any prupose.
It's initially confusing what this does.
Could you write a sentence about how this is different from executeSynchronously?
why ``false`` ?
This is not needed as synchronized can only be false when setData would raise an AlreadyBoundException anyway.
True! Is not `unescaped`, is `unreplaced` or `escaped`. Then, should I change it?
DigestData class is in DigestAuthenticationListener file (see #2876)
does ``self`` really work inside the closure ?
Tnx ... done ... all the above too.
yes. dont see it in the messages thread.
Please use camelCased names for the arguments

and please add the ``@param`` annotation
is the first `?` useful here ?
putting a mandatory argument after optional arguments is weird (as they are not optional anymore)
%s/variable/constant/g

A *constant* feels like the right home for something like a filename path, when I imagine I'll need to try and find it sometime in the future.
what about putting in a comment here to let others know that slowlog requires redis >= 2.2.12 ?
QString provides a static version for formatting numbers:  QString::number() -- that's probably cleaner than constructing an empty string and then formatting it.
Why do you want this behaviour? Aren't missing keys an indication that something is wrong?
Are you trying to compare the category lists for equality here?  For example, if note 1 and note 2 have all the same categories, return 0?  

I may be wrong, but what will happen here if rightNote has 5 categories (let's say A, B, C, D, E), and leftNote has 3 categories (say A, B, D).  Should that be returning 0?  If the notes are passed in reverse order, the results will be different, because leftNote will not contain all of the categories held by rightNote.
The setters do some validation conversion prior to setting the value.  Should I not be relying on these in the init?
You don't need `controller: users` here.
What I mean't was that last time I checked JBoss Modules had a hard coded exclude for everything in META-INF for system dependencies, so any META-INF path takes no effect. I am not sure if this is still there, but I do not think it has been changed.
@mikesherov It's cute that you think we've gotten around to adding a contributing.md :-P I sense one coming from you very soon...
Looks like maybe you were trying to normalize to the max? But you've got 100 hard-coded.

As false is the default, false is no longer in an invalid value, so when asking for the value, it should not be normalized to a number/the max. Or if it is, let's normalize it to NaN. No better indeterminate number after all.
You are creating 6 identical function here.

move the `on` function outside of the `forEach` and have the events bind to the same function.
Hmm, I'm pretty sure that design spec is wrong, the "Clear History" and "Clear Private Data" buttons shouldn't be at the bottom of the screen should they? @lco, @patrykdesign ?

Semantically I don't think it's a footer either. Is "menu" what the Settings app uses for these kinds of lists of options?
Why does the `#statusbar` needs to grow height? I thought it will be covered?
Kudos on adding the new strings!
The comment says something but the code does nothing ;)
`return` early here instead.
@vingtetun I was wondering if we could limit the hack to the `homescreen` instead of using the `ApplicationMock`, any idea?
You mean `//XXX: we need a 'initializing' state` right?
Let's change it to `data-zindex-level="lower-case"`. `data-type` is too ambiguous, upper case just don't feel right.
I would default to an empty array here if it doesn't exist
This needs to be a for loop not a for each loop
For each of those test describing 'when current user is present' from my point of view it would be enough to assert the decisive `user_id`, e.g. in the last test: `(expect highscores.displayedRuns[1].user_id).toBe 2`. But it´s a matter of taste and preciseness.
Wait why do this?
Put a `:` after `AppScaleDBConnectionError`.
Use `hasOwnProperty` instead of `in` please.
I deleted the unused CSS files and created a base template that includes bootstrap.
Maybe.  The default project is '' too.
I'd be curious to know what the error was.
See PR description
is this the only hook proposed? I say we should have more or give at least ideas of the tasks that could integrate composer with git in a much tighter way
Why putting this method here ? It is only useful for the GitArchiverTest and so should go in this class
I'll make the changes
Rules Engine"
"(2004)"  is not part of the title :)
That is certainly an improvement. It would be better if the code did a CRS.decode("EPSG:4326") instead, to get an axis order conforming to the current referencing system setup... but is the CRS class available there?
Hum. It's probably coming from the rebase. I did not want to add this.
The best solution to this is to isolate this diff -- revert this commit from your master branch, create a different branch that contains just this fix, ask cpp-netlib/cpp-netlib to pull just this commit in that branch.
necessary comments, or remove?
I think we should have a scenario called 'Navigate to US desktop site within the US edition' but I'm not sure how to force the test in to USA mode.
I would prefer a name like htp_flags or similar.
I see that is public domain code, cool.

The HAVE_NSS wrapped code is probably not from libtomcrypt, is it?
Yes that is correct. But an additional right for an item should also be available in the component actions, otherwise you can't set an action item wide.
Joomla! standards validation found:
 179 | ERROR | Whitespace found at end of line
 182 | ERROR | Concat operator must be followed by one space
 182 | ERROR | Concat operator must be preceeded by one space
 183 | ERROR | Concat operator must be followed by one space
 183 | ERROR | Concat operator must be preceeded by one space
Wait what? both of these branches are the same.
ha, nice..
got it dc0140b
I hesitated between both, looks like the simple choice is better :)
this code was designed to work if json_locale_data was not defined, but threw an exception when it was removed.  In development we were always downloading a really old client.json file, and for strings it did not know about the code would work ok.  now it works when json_locale_data is not present.
NetworkX requires Python >= 2.6 so I don't think we need this import
As currently written, `/sbin/sysctl -n kern.disks` will only be executed once, the first time Facter loads this file.  Before we merge this patch the fact needs to be updated to refresh the data from the system every time Facts are evaluated through the block passed to setcode.
Yeah but the *module* isn't driving execution... so it shouldn't be calling "exit".  It should raise an exception and let whatever executable is driving the execution of the program decide whether to exit or not.
It looks like all the calls to this new error method were in place before the method was added. How did this work previously?
This felt a little icky, and was only to prevent logging multiple times on a single run. It doesn't need to be the first run, as long as it's requested once (or often), however I'm going to try to pull this out (see next comment).
ah, always a great place to be in. we're doing it wrong, but at least we're doing it _consistently_ wrong. 

fair enough, put it is.
Separate examples with blank lines.
I'll handle that. Good call.
Since we don't need to use this event again (such as checking `isDefaultPrevented()`), there's no need to create a a `jQuery.Event` for it.

You could simply write:

```js
this.$element.trigger('shown')
```

Same applies for hidden.
You're completely right. I actually made a big mistake there :/... Thanks for correcting it !
I agree with @Ocramius here -- I remember running across this when trying some experiments for the MVC integration, and really, all you want to see is if the _service_ requested (the name, after resolving any aliasing) can be created. That said, @Ocramius -- why would we pass the $serviceLocator? Is that to ensure we have any dependencies we may need to create the service?
What will the established convention be for naming containers? I worry that you might get another object entirely... which brings me to the next point: line 79 should verify you have an instance of AbstractContainer before setting the container, and raise an exception otherwise.
- $location can be undefined
- If the given argument of $map is invalid, than your exception message is useless, too.
  e.g: "Map file provided (integer) does not return a map"
using the ``@Bundle`` notation with the cssrewrite filter is broken. You should tell Assetic to use the file copied in ``web/bundles`` by ``assets:install`` instead: ``'bundles/knpbundles/css/bootstrap.min.css'``.
Otherwise, paths to images will be broken (as it will try to access the files in the bundle instead of the copied ones, and they are not accessible from the web)
All in all good, looks like the $wpdb function handle most to the DB protection, its a bummer that wordpress plugins don't have a template layer (built in).

I'd probably make some of the DB varchar lengths shorter like name, contactName, contactPosition, emailAddress, city, state.
As with above, these (up to FileUpgradeFlags) are the only real changes.
std::replace( s.begin(),s.end(), ' ', '_') might be better than a FOR_EACH
There seems to be an over-abundance of `isset`/`empty` checks in this patch, whereas you should simply be setting defaults in the appropriate places.
10 minutes might be a little too much, maybe 3?
and i noticed you have a case for format === "Thumbnail" but the function is never called with argument of that value. Do we really need to have it then ?
`$request->isMethod()`
@anti yhtiö-joini olisi muodon vuoksi nätti!
@anti Oho, HTML-koodia t-funkkarissa.
Can you change to `inspecting`? Almost all doctrine developers are not native English speakers and I only knew you were right when i went to translator and places the sentence.
@lsmith77 it's just about understanding if `associationsMappings`, `referrersMappings`, `childrenMappings`, `parentMapping`, `node` and `nodename` are fields or associations.

To be more clear, I'd personally consider a field something like an embeddable or a scalar, while an association is probably something pointing to documents that can live on their own.
ran into a problem here where we never managed to save back to private -- the problem was that wal would be a string, so "0" in javascript (unlike perl) was a true value -- meaning that the condition was always true. Fixed before commit, but please take note of the trickiness for the future.
`kernel.root_dir` is always set (relative to the Kernel)
why do we need to depend on lxml explicitely?
Also fixed (-v is for the easyconfig version. --toolkit-version is for the toolkit version)
Should at least be warning?
REST URLs should follow the HATEAOS principal.
The URL scheme is the responsibility of the application designer.
I  chose "tasks" for GET requests that return multiple elements (aka tasks in the application context) and I chose "task" for GET requests that return a single element which is the natural URL scheme (notwithstanding my comment about HATEAOS).
Jingo docs say to use the normal django render functions. I'm sure that's not required, but just thought I'd mention it since for django devs it might be clearer code. Not something I'd r- for though.
There is a PEP8 issue here: base_test.py:28:1: W391 blank line at end of file
I still don't like the error here, but I guess it is not really important right now.
Good question.  The functionality as you say mimics "tail -f" / "tail --follow" -- my initial instinct was also to call the option "tail", upon reflection I but I think "follow" is the operative part of the command we're emulating here.
I think parsing the config should be moved to the Bot's  `Configuration`.
Let's make a `randKey` or something that does `randKey(PROBLEM_DATA)` here.
Here's another. Might want to find them all by searching for "{}" on this page.
use constant for 1 since it's used in several parts of the code
$replace_existing_with = 0?
I think you might be right, let me look into this a little more.
Thanks again for the checkup, I've just made it work in my project and frankly, I tend to forget other operating systems other than Linux.

So basically, it should be something like this (probably pseudo-code):
    
    var settingsStr = "";
    var flag = true;
    process.stdin.resume();
    process.stdin.setEncoding('utf8'); // TODO: Get current system encoding

    process.stdin.on('data', function (chunk) {
      settingsStr += chunk;
    });

    process.stdin.on('end', function () {
        flag = false;
    });

    while (flag) {};


Right? (I'm no Node-expert, far from it)
I'd rather prefer to keep the minimum and add methods when we need them, but fair enough: done :) .
why do you no longer throw the exception?
Small typo at the end. Could you add one line of documentation per API call so it reads faster?
Should the above ``runtimeClasspathEntry`` bundles be included in the MANIFEST?
I would prefer to keep `ScalaReferenceType` as it is. It is wrapping a specific JDI type, and cannot do all that `ScalaClassType` can do. But we could create a trait (or maybe two) to contain the common logic (finding method or field), and then delegating to a specific method to do the real invocation.
Should we throw an exception if a driver is not implementing the AdvancedDriverInterface? After all, you won't get all class names anymore as soon as there is a driver which does not implement it.
for(
I see. Good to know.
Two pointless blank lines here.
Dit moet allemaal nogsteeds in een move-function
@samxli Thanks for pointing that out. I seem to recall that I intended this to be a drop-in fix with no DB changes. I'm willing to bet that I was pushing that extra column into the DB for my own debugging purposes. I'll double check and fix.
Зачем везде `@autoreleasepool`?
в XCode можно добавить категорию через меню файл. Так же как новый класс. Тогда хкод всю грязную работу делает сам - создаёт отдельные файлы .h и .m где хранит описание новой категории.
Добавлять категорию в файл main.m неправильно.
Спасибо. Не нашел этой функции, поэтому сделал такой костыль.
1 подключи ARC
2 прочитай http://ru.wikipedia.org/wiki/Objective-C#.D0.A3.D0.BF.D1.80.D0.B0.D0.B2.D0.BB.D0.B5.D0.BD.D0.B8.D0.B5_.D0.BF.D0.B0.D0.BC.D1.8F.D1.82.D1.8C.D1.8E
Он человеку и не нужен больше, он забыл его удалить из описания класса.
Nevermind. I see that a path separator can be embedded in GROUP_NAME.
You might take a look at http://api.rubyonrails.org/classes/ActiveModel/Errors.html#method-i-full_messages
This `Given` step is an assertion.

`Given`are not assertion but context setters. Instead of checking that the environment variable is indeed set, this step should actually define the variable instead. 

That being said, I still don't totally understand the reason for that variable :)
Missing line breaks
Align `=` signs
This should be clearer:

> in the current scheduler, after any blocks already scheduled have completed.
The problem with this is that this plugin can manage multiple accounts at once, not just a single user (which would indeed be better done using environment variables). Apart from that, hubot understands private messages, so the credentials don't show up in the public chat if you don't want that.

Marvin
I think we might be better served by just deleting this code. Leaving the leftover p4ruby library code just makes me sad because it just didn't work at all.
why not just add these lines to the existing raw ICA example and avoid creating a new example?
this example as some duplication with the other example including the main docstring that is used
to name an example on the web page.
minor formatting change -- just remove space between ) and ; for last three methods.
I can't help but wonder if this typo is intentional?
"... meant for testing and performs..." (Drop comma.)
The time information is available per browser.
session seems useless here ?
That doesn't make any sense. preRestart is called on the old instance. and the old instance will be orphaned.
Translated to Python code, `Piecewise((x, x>1), (0, True), S.NaN)` means

```python
if x > 1:
    return x
elif True:
    return 0
else:
    return S.NaN
```
which is clearly equivalent to 

```python
if x > 1:
    return x
else:
    return 0
```
but not to simply `return 0`.
Даже для не-енум-класса ты можешь писать EnunName::Value в C++11.
I won't add a setter to the interface, I clearly understand the contract is meant for reading.
I tried to say that a could use the complete constructor instead of the small constructor plus the setter.
Once again, as Misagh and you don't like this at all, I will simply use the complete constructor and remove the setter.
I'll update the javadoc regarding format + UTC.
For ParseException, it's somehow obvious that this exception will arise when the parsing of the string date fails. Nonetheless, I'll update the javadoc as well.
You used a #select here ... I think an #each would work better... A select is used to reduce the array down by returning true/false to use again later.
Yes, empty labels are rendered as non-break-space to force rendering of the label in the browser
@ignore is unnecessary for methods whose names begin with "_".
Sure.
I'll take mine, as "Paths" can be easily confused with "Path".
as opposed to a half-blown instance? jk ;)
Right!. Done.
+1, with a simple lock() you don't need to catch InterruptedException either :)
this will probably break on windows
Why even accept it as a pointer, then? Is there a portability reason I'm not really aware of?
We should assert that `patch` isn't `NULL`.
`profile.include` doesn't currently accept a `time` argument (these arguments are passed on to the filter functions available in a Profile). can you remove the `self.time` parameter here, and maybe we can figure out a more robust way to provide data to filters?
Yes, thanks for the reminder!

The methods `plot_surface` and `plot_wireframe` don't currently support masked domains, so this is a big win in my opinion.
Thanks for the correction, I'll use string equality.
Good point. Also a version-specific conditional would potentially be more future-proof, as well as "stretching the legs" of the new functionality.
can you explain why some of the Google examples still have google.css included, but it's removed from this one?
According to: http://stackoverflow.com/questions/5819912/webkit-transition-end-in-mozilla-and-opera
"It seems that in Opera, as of version 12, "oTransitionEnd" event was renamed to "otransitionend"; you should register an event handler for both events"
Potrzebna jest mapa templatów?
Your point about the data structures is well taken.  The confusion, here, is that -- before our code -- timeseries had a very specific meaning, which should be well known to the people actually using this function.  http://en.wikipedia.org/wiki/Time_series.  Same with idiosyncratic disturbance -- very specific meaning, one which has got to be known before using this function beyond its intended use.  We made a lot of concessions in writing the deseasonalization this way to make it faster, which restrict its generalizability.  I will make a note about this.
Doesn't this mean that infinite scroll will no longer work in normal view mode ?
Can this be changed to `.hover(hoverIn, hoverOut)`?
As with the other class, I believe we can replace the slightly divergent code for resource-type and QString with shared code.
Can we make this assumption safely? What about the helper-menu which is outside these two fragments - specific scenario is when removing a page by clicking a button the tree needs to be reloaded as well.
I was thinking snapshot count <= 10, or something like that -- the setup & function requirements of bsearch have to be more than 4-5 memory references, don't they?    (IIRC, qsort didn't pay off until you have roughly 20 items in the array, for example).   We're calling bsearch even if there are just a very few entries.
This is a bug fix, I think.   We haven't been checkpointing indices when a target is specified to WT_SESSION.checkpoint.

This change seems to work, but I don't have much confidence in my understanding of indices, please review this one carefully.
This code is intrinsically fine - I would just explicitly materialize the logic of inferring the target version into a variable so that it is self-documenting, e.g.

```
val nextVersion match {
  case None =>	
    log.info("Detected empty schema, using version 0 as the starting version")
    0
  case Some(vsn) =>
    log.info("Current schema version is %d".format(vsn))
    vsn + 1
 }
```

Also, note the suggestion of a more informative log statement . Doesn't have to be this, but something more descriptive than 

```
log.info("Empty schema")
```
i < md_size, just a coding style thing
Should the type for parameter ``$formatter`` not be ``FormatterInterface``?
It permits more flexibility. We have no way to mock the response in actual code. It's used in tests currently :-s
Another possibilit would be to use a ResponseFactory. (The Buzz one for example).
HashMap is not thread safe. Jetty will call this from up to 250 separate threads. Make class thread safe, and add a comments at calling locations so it is clear that the calls are safe.
Sesame-2.7.0 will have a new artifact named sesame-rio-nquads, that is based on the sesametools nquads parser. However, in the mean time, there are many examples of URIs that I have given people that rely on /nquads/ which will not be available if you remove this dependency. Try going back to sesametools version 1.6 if you have issues using this module.
Thanks - fixed in pull req #22.
Во-вторых, это не является классом, соответсвенно имя должно быть 
autoloader  и храниться файл должен где-то в /src
this was handled elsewhere in the code, and is legacy behavior
Mulig jeg bare er puppet-noob, men hva er hensikten med å wrappe `exec { "download": ... ` i en `define` hvis den blir likevel blir umiddelbart kalt her? Hva er forskjellen fra å bare ha `exec`'en rett i klassen?
I'd probably prefer to see this variable in camelCase ``$loadLanguage``. We should only use underscore separators in things like column names.
No entiendo por qué tenemos dia y horario por separado? Por qué no usamos un único Date para guardar ambos?
Não estás contente com o quê em concreto? A informação dos seatType's misturada com a currency?
Particular reason why not just: 

``` js
if (Element.ShortStyles[property]){
```
This isn't quite the same code; you've removed all the fallbacks where we check the feed/search to see if they have a thumbnail before using the default.
D'oh :shipit:
Frame maybe? :)
`twitter_id`? We don't have that anymore.
Indentation is a bit off, you don't keep the style (`:` has space sometimes and sometimes not).
But OK. Because it is a periodic task. We could add it explicitly.
Or is this what you have problems with?
This comment is not closed at the end with `*/`.
Normally I'd say to use regular `const char*` here, it would break parallel calls to TagLib, but now with atomic refcounts I guess it should be ok.
So you can change BASE_DIR to something else and have the script still put the output in the current directory.
Add space after if.
Pep8 issue on this line. I wouldn't ask what matches this regex.
if you decide to reduce this log information to debug or trace, make sure to avoid string concatenation, apply the formatting only when needed, and don't format twice (reuse the value for the return value)
Remove the new in front of the Net::Appliance::Session call.  It will not work since it will create a reference.
Not if you pushed it to the same branch.
Can we reuse a `BaseCache` object instead of creating a new one for each key?
You should include key in the output message here and below.
What is this ".." syntax?
For the fourth time: _an observer_ is singular, not plural!
Same remar as `resolution`.
Seems the faked message argument won't really be looked at, maybe skip it?
Yeah, its event dispatch order. If we did it here (and changed the dispatcher to only fire this event at a single widget instead of cascading), it would distribute the events bottom-up instead of top-down. For this particular event (which cannot be hooked or stopped by external clients) its probably not so bad, but I've tried really hard to keep things consistent so its easier for widget writers and UI clients to understand, even if it means the core code is more complex. And that means "all events dispatched top-down".

So yeah, comment might be the go here :)
I found that if you don't specify the language, google uses current language that you use, so one problem fixed.

About that local search, since Google Web Search API is deprecated I used Google custom search engine instead so local search option isn't available here. One solution is that I rewrite not using google custom search engine, or we specify search options. This search engine can be fully customized and it is also already prepared  for AdSense. So if you would like later to use different search engine specifications or AdSense or something, you just change the Search engine unique ID.

Please check if this search engine is customizable enough for our needs. more info on: http://www.google.com/cse/
You shouldn't pass a string as an argument to ```setTimeout```. Just pass ```scrollwindow``` as a function.
define "configuration dict" earlier?
D'oh! Neglectful copy and paste on my part. Good catch. I'll fix for next PR.
Ah.  it's just a test.  looks like a vestige.  I will get rid of.

On Dec 13, 2012, at 12:44 PM, Joe Blaylock <notifications@github.com> wrote:

> In main/courses/course_materials.py:
> 
> >                      key = 'exam:' + str(exam.id)
> >                      if exam.section_id == section.id and not level2_items.has_key(key):
> >                          children = get_children(key, level1_items, level2_items)
> >                          
> > -                        item = {'type':'exam', 'exam':exam, 'index':exam.index, 'children': children}
> > +                        item = {'type':'exam', 'exam':exam, 'index':exam.index, 'children': children, 'records':exam_user_records, 'foo':'foo'}
> What's with this 'foo' thing?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
No clue. Tried with puppet master only. You could find out ;)
This is still missing the third parameter. Does this work this way???
Undefined method include.
I don't think that works...
Would be good to use `private final StringBuilder lineSoFar`, instead of the synchronized StringBuffer
template parameter is not used?
And this one adds "B" w/o adding "A".  Is this building on the results of the previous entry in the Examples table?  If so, this is breaking the rule of test independence.

Or am I completely misunderstanding this?
Wow, the step-definition code must be really smart to figure out that it needs to read lines 32 and 33 as "one line" in order for the test to work.

Was this considered?

    | desired state         | action                                 |
    |                       | cancel A @ $30 now, cancel B @ $20 now |

Or do I not understand here?
This should be `self.approved = false`, without quotes
I'm actually leaving the `base` as a pure string now to avoid unnecessary complexity. Can you change these back?
Yes, but this was never supported in the connection string, and the connection string specs don't allow for just "true" -- it really does mean the name of the replicaset there.
If the username for the test is not an empty string, perhaps we should use `%s` instead of `%S` here?
revision and maybe repository should be attributes, so the user can specify a tag or commit rather than HEAD.
Where did you have in mind?
nit: add spaces between : and the size
The allowEmpty check could be a separate method, or since its a simple condition, it could be inlined.  I think this parsing of validation rules, has cemented the idea for me that validation rules really need to be managed by some sort of object that contains all this logic.  Having it in both FormHelper and Model expressed in slightly different ways has been problematic over time.
you should pass the Main's toolkit into make_icon instead of relying on the default_toolkit() to be correct:

Main(MainWindow):
    attr my_icon = make_icon(toolkit) # 'toolkit' is an attribute on BaseComponent
I pushed the code.  You should be able to see it when you click the Diff tab.
@ProCynic 
Why does this start with # ?
reading throw a notice. Writing does not
`mpif77` is a wrapper around the Fortran compiler that was used to build Open-MPI. As far as I know it doesn't have any fundamental changes compared to the program that it wraps except for adding a few pre-processor definitions and linker/include paths by default.
True. Api is not final. But it is important to have one place where all
board hw info is located and can be queried...
 On Dec 11, 2012 10:32 AM, "peabody124" <notifications@github.com> wrote:

> In ground/openpilotgcs/src/plugins/coreplugin/boardmanager.cpp:
>
> > +        if (!list.contains(vid))
> > +        list.append(vid);
> > +    }
> > +
> > +    return list;
> > +}
> > +
> > +/**
> > + * @brief BoardManager::getKnownBoard
> > + * Returns a known board for a given vendorID and productID
> > + *
> > + * @param vendorID USB vendor ID
> > + * @param productID USB product ID
> > + * @return A board type if the board is known, NULL otherwise
> > + */
> > +IBoardType* BoardManager::getKnownBoard(int vendorID, int productID)
>
> I would argue this is invalid. A serial connection doesn't have a board
> type. Neither does PipX (it has two boards really, the remove and local
> one). Because there isn't a unique map from VID/PID -> board type I can't
> think of a reason to use this method except to make a bug ;-). Plus it
> isn't used.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PhoenixPilot/PhoenixPilot/pull/79/files#r2382397>.
>
>
copy-paste -- надо вынести в отдельный метод
Слишком много copy-paste. Надо сделать Comparer<String> для case sensitive сравнений, а для case insensitive использовать String.CASE_INSENSITIVE_ORDER.
Нужно информативнее что-то выводить.
Check before adding the `https://` scheme that `mediaServiceBaseUri` doesn't has scheme.
Oh, that’s cool. Sorry about that
Loops using iterators are not accepted! =)

Preferred way of looping is from "size - 1" down to 0.
didn't really mean to get rid of this line...whoops
Wouldn't it be better to compute the output exactly and test whether `nom == 0` explicitly and set `res = 0.0` in that case instead of using the `1e-10` offset? I wonder if it's possible to pass branch prediction hint to the compiler in cython. Maybe @larsmans knows?
Fixed (see new commits).
`fit_transform` would help if there were input validation, though, as it might prevent a copy.
you mean the classes property? where?
Yuk. Obviously a typo. Will check if pep8 complains ;)
As this is a FAQ on the mailing list, it will be really good to have a section on the docs for it. Thanks a lot.
Actually, `$_SERVER` seems to be all we need, `$_ENV` is completely included in `$_SERVER` on cgi-fcgi (see http://thekid.de/server-env.php) and empty on apache2handler (see http://xp-framework.net/downloads/server-env.php)
Is distance used somewhere ?
`res[ topic ]`
Usually in dmd reference variables are not used, and I'm against using features that are not present in D.
This is ready to merge either way, but what do you think about removing it and loading then storing instead?
Could you put a TODO here since we'll want an automatic way to determine this instead of hardcoding
This sentence `The boxes are rectangles in projected space but the related area on earth is not !` is hard to follow I believe as there is a switch in plurality: not sure were to relate the `not` too, plus the extra space makes me think a word is missing :D
Maybe better to stress the target?
`The boxes are rectangles in projected space but the related area on earth is not a rectangle!`
Here we're using the check + assert technique, which logs all problems and only fail at the end.
Is it ok if you join when it wasn't started?  (dali was null)
Not sure you really need a roxie-specific version of the ecl (the #option etc may not be needed except on Roxie, but it should be harmless).

If you DO have a roxie-specific version of the ecl, you should be adding a //noroxie on the generic one, I think?
I don't set the fileCrc attribute if compressed, but I still calculate it.
``$c`` and ``$u`` are not really good variable names. Could you please use more explicit names?
It doesn't feel right to reference `settings` in the templates. In fact, I don't even know where it's coming from.