labofsahil/github-event-dataset-2012 huggingface dataset:
Yes.
In explicit mode, for a continuous feature, null will be encoded as the average value of this feature; for a discrete attribute, null will be encoded as another kind of distinct value. For example,  the 1st feature (fid=1) is a continuous feature, its average value is 25.66; the 2nd feature (fid=2) is a discrete one, it has three distinct null value, then null value will be encoded to 4. Therefore, they will occupies two rows in the KV table for attributes. Please see the following table. At this point,  **If it's not continuous value but its value is NULL**, then it should be m.code.

The KV table for attributes.
```sql
fid fval  code
1  null  25.66
2  null   4
```
I got an error here.

```
psql:/tmp/madlib.VprIUZ/regress/test/multilogistic.sql_in.tmp:89: NOTICE:  CREATE TABLE / PRIMARY KEY will create implicit index "pk_patient" for table "programs"
psql:/tmp/madlib.VprIUZ/regress/test/multilogistic.sql_in.tmp:89: ERROR:  relation "pk_patient" already exists
```
Unless we hit an error from a malformed query, in which case checking nrows() is the wrong way of detecting the issue.
`while (`
Haven't forgotten.  Really swamped over here.

On Fri, May 4, 2012 at 8:36 AM, Azat Khuzhin
<reply@reply.github.com>
wrote:
>>              size_t amt = fread(buf, 1, 4, file);
>> +            // end of fifo/file
>> +            if ( feof(file) ) {
>> +                break;
>
> @andy10gen ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mongodb/mongo/pull/205/files#r773788
Is there a reason you put slaveOk in the middle rather than the end so we can preserve backwards compatibility?
flake8 didn't complain. I'll update it to match your second version.
This was my spell checker getting a bit happy...
`homepage` goes before `url`
Maybe we should just to a HEAD request.
bad doc formatting

http://www.python.org/dev/peps/pep-0257/
e.message is the right way
Still don't love this error message. I can't think of anything good, but I think ```"You can't go through {}."``` is less bad -- it looks silly in some cases, but "traverse" looks stilted and programmerese *all* the time.
Same here. Always polite to have sane error messages.
If the case is we set timeout for 6000 millis
so the total connectAndBind can be more than 6000 millis.. why? because conn.setSoTimeout was set to enquireLinkTimer (of course the value is non 0)
You get what I mean?

I think the conn.setSoTimeout(...) should be set to the value of the remaining timeout taken by the createConnection... and when the state is bounded then the conn.setSoTimeout(...) should be set to enquireLinkTimer (set the timeout to normal)

So the total connectAndBind would be no longer than 6000 millis

What do you think?
Ok, I will change this. WPML use context so maybe it slipped into my mind this way :)
* コメントには//を使うようにしましょう。一般的に、/* */ はドキュメント用のコメントに使用します。
* XcodeはCommand + / でコメントアウト/インの切り替えができて便利
* 行末の;重複
* [ ] の内側のスペースを除去
  * ソースコードは複数人が手を入れることになります。フォーマットの統一はとても大事
Why would you ever need this? Why not just configure the input field to be whatever the actual field name is?
Why is this line formatted differently than the one above?
Just so you don't miss it, formatting here as well
Ahh the Obj-C naming conventions...
Seems weird to have this adMob teardown code repeated twice (Line 112 and here)
Method name is a little confusing since the term "range" is so generic -- seems easy to misread as "set the range of selected indices" (which will actually be possible if/when we do the full XD design for inline editors).  Not sure if it's worth changing at this time, though.

Would be good to put some minimal docs on this though (or remove the empty comment block, if not).
Yeah, or my fault... but just a merge thing.
two things fail. First, the language.

Second, and maybe, more to the point, I can POST your Servlet with curl without ever even logging in.
Should these parameters have `checkNotNull`?
This should actually be modified to use Util.Translation.archPackageName
Can you move this somewhere to the top of the file? (To load everything as the first thing at one place.)
Thats is weird, i see them OK, both are translated.
This should be inside of `li`.
Can you give me more code?
Ok, I will update pull request.
actually i copied it and it seams that spaces were transformed to tabs or something.
nevermind, I see whats you did here
I don't think this is entirely necessary, is it?
What happens if it's not really a quotaExceeded error, and just some other 403, 404, or 500 error? Do we care?
To avoid `close` being called twice, set `_currentEditor` to null here.
videos-cdn.mozilla.net, https, no butteruid
This isn't enough of a check to ensure that the textbox's contents have actually changed. How about greying out (disabling) the button until the url has actually changed?
```
humph
  just indicate we're using under MIT
humph
  don't mention gpl
```
At least I can give the details of the report I've filed, if you wish so. ;) The comment could make it easier to tell from Opera's changelogs when the bug is fixed.
No need for the extra `{...}` here. JavaScript puts all variables in the same scope.
服务端和客户端都应该是 uploadToken 的上传方式，旧版的 putAuth / putFile 在代码中可以保留，但是在新版文档中就不必体现了。
http://docs.qiniutek.com/v3/api/io/#upload
I just switched to using bundler's method to do this, it's got an ensure in it so does that work?
`#synchronize` yields the client instance variable, a variable we wouldn't use in this case. That is, unless you'd like to see us directly operate on the client (`client.call`) instead of calling `#watch` and `#unwatch`.

I'd prefer using `#mon_synchronize` in this case since it does the minimum amount to achieve the goal, to synchronize everything within the block, while keeping the logic for how `#watch` and `#unwatch` are performed in exactly one location.
This spec could use more `describe` or `context` blocks with `before` blocks that specify setup that's more specific.  That would really help to clarify why all the setup code is necessary, and it would also help to indicate which contexts are actually *not* tested.
I don't feel that it does belong to the public API, not any moreso than _put_conn, _get_conn, _make_request, etc. They're not the methods you call when you have an instance of the class; they're the method you override when you're extending it to change an implementation detail.
when does this happen? Should we do something here?
The next 12 methods need formatting/indentation updates.
Icons in bootstrap have a `margin-right: 2px` already, so I don't think this is necessary. They leave a space between the icon and the text in their examples: `<i class="icon-foo"></i> Bar`
The tag in the repo is called 'v1.0', not '1.0'.
On 10.10.2012 15:39, Lukáš Zapletal wrote:
> If there is not an easy way of determining if foreman support is installed,

This is "just" system tests, where we somehow expect to be everything on 
one machine. So I suppose we can do:

if rpm -q foreman; then
   test_success "architecture create" architecture create 
--name="$ARCH_NAME"
   ...
fi
Before incoming calls and initial connections right? How do I know what context this is being called in? Like what method am I trying to authorize etc.
Does screen.mozEnabled does not work here?
This line was important, it sets the title corresponding to the URL in the global history. Removing it breaks the history listing, no titles are displayed.
I have no idea why this line has been removed — on my repo, this line has been removed in all apps I’ve worked on.

That’s part of the current makefile magic, I guess.
Per IRC I was wrong :)
Just a last typo: conv**e**nience
Also missing description
I would argue that the user should use a different type of pixel mask for image `b`, since it is indeed not strictly boolean.

If it did use a separate type of mask - and that mask knew how to communicate against boolean masks - then you would not encounter any weirdness. 


On Monday, 30 April 2012 at 8:29 PM, Erik Tollerud wrote:

> > + 
> > + def _arithmetic_operation(self, operand):
> > + self._check_operand(operand)
> > + return np.logical_and(self, operand)
> > + 
> > + def mask_add(self, operand):
> > + return self._arithmetic_operation(operand)
> > + 
> > + def mask_sub(self, operand):
> > + return self._arithmetic_operation(operand)
> > + 
> > + def mask_mul(self, operand):
> > + return self._arithmetic_operation(operand)
> > + 
> > + def mask_div(self, operand):
> > + return self._arithmetic_operation(operand)
> > 
> 
> 
> Here's an example of the sort of thing that worries me. Lets suppose we have two `NDData` images `a` and `b` we want to combine: `a` uses a boolean mask like the one here where False is "good pixel", and True is "bad pixel". `b`, on the other hand, is a (slightly) more complicated integer masking scheme where 0 is "good pixel", 1 is "cosmetic defect", and 2 is "saturated pixel". If I then want to add these two images, I naturally expect False and 0 to combine to give 0, but all other combinations should give whatever the integer value is (e.g. False w/ 1 -> 1 and True w/ 1-> 1, False w/ 2-> 2, etc.). But that's not what happens - instead I get all kinds of weirdness like False w/ 2 -> False. 
> 
> The key point is that you can't count on boolean masks combining with boolean masks all the time, and given that, it seems dangerous/confusing to override the standard numpy behavior.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/astropy/astropy/pull/202/files#r750081
> 
>
I believe it is to use multiline imports


On Sat, Dec 22, 2012 at 12:51 AM, jao <notifications@github.com> wrote:

> In bigml/api.py:
>
> > @@ -56,7 +56,8 @@
> >  except ImportError:
> >      import json
> >
> > -from bigml.util import invert_dictionary, localize, is_url
> > +from bigml.util import invert_dictionary, localize, is_url, \
> > +    clear_progress_bar, reset_progress_bar
>
> is the \ needed?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bigmlcom/python/pull/28/files#r2493876>.
>
>
Nit: our coding style is:
  4 space indenting, no tabs

(see doc/coding.txt)
This introduces a bug. Uninitialized char arrays are not guaranteed to be null.
You sure are right. Looks fine in Netbeans though, but I'll take a look in another editor.
Need a space after `if`
Me gustaría que esté en castellano haha... but because of consistency with every other ```$indexes```name :P
Then it requires to:

* test wether the result is of length > 0 instead of just using any() in the test
* index using names instead of a boolean vector

I don't know what is best in general (I'd guess that performance is not an issue here).
I noticed just as you commented. It's been a long (and drunken) night lol.
When is this called?  It seems like this typesets without the needed latex replacement logic.
We are trying to use the `var foo = function ()` style of function defs, please convert to this.
Nope.  2to3 does this transform, so there is no problem (this import is all over IPython).  Of course, it's probably a good idea to start getting used to these renames.
I would change bellow/above for left/right, what do you think ?
this seems really awkward, can we at least use `kind_of?`?
checkNotNull(name, "name"), checkNotNull(value, "value of %s", name)
I'm fast  haha
Is this intended? This smells like it should be an `assert`
I think `REBASE_INTERACTIVE` makes more sense here, alongside `REBASE` and `REBASE_MERGE`.
You have a bug here - txPacketSequence will not increase.
I rewrote it to:
outBytes[2] = unchecked(txPacketSequence++);
I think 0 is default, fwiw.  But if not, ignore this.
It's not required in HTML5 but nor is it forbidden. I just copied it from the previous version of the page, but I'll remove the slashes for consistency's sake, since we generally don't use XHTML syntax.
replace this with ``url(...)``, now that this will not be hosted on PHP any longer.
So this is another GenericMailer, right? The original Generic Mailer took in one url and label, whereas this takes in a list of urls and labels. We could refactor the existing code to use this GenericMailer instead, and then not have two mailers that do the same thing. (I'm not saying you should do this, but it seems like the DRY solution to me.)
I believe the definition for that namespace is still there, I just moved it to another location.
В мастере FOSUserBundle пока что нестабильный код. Из-за него падают тесты. Я взял последний стабильный тег. С ним все ок
typo: whether
Ok, but what I don't like is the word "straightforward" here, I think "powerful" is way better
The SizeLength guess should also take the "max" value into account.
what about a static `create` & fluid setters ?
this has nothing to do in the EventDispatcher component. the HttpKernel provides the profiler, and data collectors for a bunch of components (including the EventDispatcher which is an hard dependency to HttpKernel anyway). This class is totally unusable without the HttpKernel and so it is not logical to move it here
no, again :)
It seems like overkill to call a method downcase that just downcases the string. I assume you meant strip, which would do the same thing. What is the readability win?
`standardize` over `normalize`? Or `strip_whitespace_and_downcase_email`.

Or better yet keep as `normalize_email` and have that private method call two other separate private methods, `normalize` and `downcase`.
There are some variants to OFF (NOFF, 3OFF, COFF, ...). I guess for now OFF is good enough.
DGtal class name starts with capital letter "Cone" ;)
I don't think this will work as expected.  Try feof(file)?  Hitting end of file doesn't return an EOF character, it should fail the fread and never get to this line.
Biome IDs are only for the vanilla protocol. BiomeSelectors do not know which biome they are actually selecting and cannot without a bit of refactoring.
i changed the implementation, what about now?
When using Guidance Automation Toolkit, it creates a file <Solution Name>.gpState
All of the undefined vars should move to the first line of the var.
Fixed in 868d668.
Thanks. Removed.
Until @gregorwagner tells us something about this, you are gonna love the way we now group the calls. Since the calls are now grouped also by "phone type" and the phone type is a value of a contact, the grouping of the calls cannot be made until all the contact details are returned.
This friends list is missing the 'cool head effect' created by @albertopq (maybe another reason to have a single point), using the library:

https://github.com/mozilla-b2g/gaia/blob/master/apps/contacts/js/fixed_header.js
nit: should be camelCase
I'm trying to understand this block of code where key is not defined.  When do you expect that to happen?  Is this just for a particular type of event? If so, maybe include that in the condition.  Is this for when the user slides their finger off the original key and on to something that is not a key?

A comment would help here.

Also, how does this interact with the code at line 509?  It seems like the logic here and at 509 could be combined somehow.
The discussion from our meeting with @caseyyee was that we would show the e-mail address when the user clicks on it in the menu thing that comes up.   This regrettably does not appear to have shown up in the wire-frames.  Casey, if you could formalize that in the wire-frames, that would be great.

I think we should get that change in the pull request before we land a change to only show the name.
Use |this| everywhere or use |bind| everywhere.
why do we need `content` here?
Please add the comment explaining why this arbitrary value is not an arbitrary value :)
I am not sure if we are going to let the user toggle this option now.(Should be always enabled?)
If PM/UX confirms then it's fine.
You should add 'box-sizing'. There are scroll issues but padding + width 100% = overflow for sure ! ;)
Is is on purpose? Otherwise, let's kill the Zombie!
See the comment above. I would live to avoid this 'app' type for now. As I said before a web site could use that to install a bookmark on the homescreen of the user without it's validation.
I'm not comfortable having commented code and TODOs for a feature that was working previously...
typo: should be networkName instead of neworkName.
Add |var node = this.numberNode| and use that later.
Thanks for the modification, but why not use this.view for smaller search scope?
[Nice!](http://translate.google.com/#auto/en/%E9%85%B7%E5%BA%A6%E9%85%B7%E5%BA%A6)
], function(err) {
   test.finish();
});
My fault, everyone. I didn't mean to cause any trouble. I just saw the thread in the issues and thought I would give it a shot. I can close this request if necessary. Please let me know.
actualy it's 3.2.8 these days. But I liked your earlier technique of adding "or higher"... maybe we can change the "result" method to have a parameter that says "output '(or higher)' with different typography"
you can use a mongoose virtual method to provide the `isCurrent` property
There's no actual tests here. What are you trying to test? That no exceptions are thrown? If so, you should be doing 

```
assert_not_raised(Exception) { 
  ea_class = ErrorApp.new("testapp", "ERROR")
  ea_class.generate()
}
```

Although you shouldn't be checking for the generic class `Exception` and should be checking for something more specific.
if something is sent which *isn't* a `FormData` object, `form` would be a misleading name
Well, so I realized, after I wrote my original comment, that it is likely a remote URL, rather than a path on the slave. (I guess I should have been clearer).
One other thing that I see that could be problematic is the other attributes of the block are discarded here.  For example a "rainbow" class is added to the block once it has been highlighted so that rainbow knows not to highlight it again if you call ``Rainbow.color`` again.  

I think this could cause rainbow to try to highlight blocks that are already highlighted which could cause issues.

Also for platforms that don't support or strip the ``code`` tag such as tumblr the ``data-language`` attribute is placed on the ``pre`` tag.
I think it is better to rename m_nValue to m_dValue.
Because its type is double.
Definitely, and TBH I think we should simply rely on linking to eg Celery and Haystack's "what we support" pages, as that's what matters. But examples of (maybe) known working configs, might be nice. "We've tested with this combination."
So far I only found `1`, but this is addressed when I update this pull request.
There is a better way to handle the exception in a more JUnit-y way. See for example: http://www.mkyong.com/unittest/junit-4-tutorial-2-expected-exception-test/

If you update with this, I'll merge straightaway: the code itself looks great.

Thanks ;)
didn't know that..
Why would anyone do that? Seems very strange to have uppercase attribute names.
this did a major f...up to all the profiles. only after modifying the script to use bash instead of sh and removing the "unset LANG", the script worked as proposed.
what's about "tooltip.h"? Use "*[^_][a-z].h
150111015 Mücahid Ömer Bozkurt
Where is this `db` variable coming from? If it's a global, declare it as a global.
this are too much positions for the default command_gui. Can we reduce it to have maximum 5-6 positions? which transitions are collision free? is there a group of "side" and "front" positions where all transitions are collision free?
Looks fragile. What if `targets` is e.g. `"-s something.xml compile"`?
Actually in content for example edit.own only applies at the category adn compoent level, you can't further specify it at the item level for performance and other reasons.  You give edit own in a category and the user can edit everything belonging to her in the category.
I'll merge this but replace 'has_key?' with 'include?'
Shouldn't you also be performing the value-updating logic in the else clause? An early return here would prevent that.
Please add a paragraph here about the injector rules for the method parameters.
use update_attributes instead of many lines
setting this to `null` is better for performance iirc from Eddie's talk from the work week correctly.
this line might be superfluous, since the page redirects to 123done's homepage without help from the test.
These three lines could just be `for key in sorted(batches.keys()):`
How is `pushlog[index]['date']` any different here than `node['date']`? If it isn't (and I don't think it is), then you don't need the `index` variable or the `enumerate` on the loop.
ok, I think my new comments address this.
apparently it's a good design to name callbacks
this was in one of the links about code style you posted
Ok. That is good. I just wanted to check that I wasn't missing something.
Didn't you delete that just above?
Another one here.
We have to keep the master branch in the stackrc. We can use localrc in order to specify any branch.
Should be noted that the English counterpart of this page also contains the same commented out JS block.
You only need put SearchOptions here (both files are under the same namespace)
Will options be flat, or deep? If they'll be flat, we can simply use iterator_to_array()
Also pass in the $labelPosition:  $labelHelper($element, null, $this->labelPosition);

Couldn't we use this $labelHelper __invoke call for both cases, when 'id' attribute exists or not? Seems like there is some duplicated logic here in FormRow that the FormLabel already takes care of. This could remove lines 96-104 (but still need to append the $elementErrors afterwards).

Also, seems like there might be an existing bug when $type === 'multi_checkbox' || $type === 'multicheckbox' || $type === 'radio', where the $elementErrors aren't being added....since you're in here :)
I don't know if it's the right usage: slug is available only for a latin website at this time. How does the developer of a Greek blog?

You can use PR 2105 maybe (like DASPRiD).
Combine these into a single conditional using '&&'.
If they're not equivalent, what happens on wakeup? Will the filters be repopulated? if not, is serializing a filter chain a good idea?
app_dev.php @stloyd ?
Parameters are generally preferable over inlining like that. Unless there's a good reason to change it?
Because it also works without it. But I agree, we should put it back in.

(I first tried use `require` and therefore removed the whole readFile stuff, when I noticed it wouldn't work, I wrote it again by hand and left out `. toString()`)
Don't throw and error if localStorage is not supported. Better to fail gracefully.
If Gemfile.lock is checked in, then that already locks the version down;
what does locking it down in Gemfile too gain us?
On Aug 14, 2012 4:30 PM, "James Stewart" <notifications@github.com> wrote:

> In Gemfile:
>
> >  gem 'newrelic_rpm'
> > +gem 'statsd-ruby', '~> 1.0.0'
>
> We should be locking to specific versions of gems now, so this should be
>
> gem 'statsd-ruby', '1.0.0'
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/alphagov/publisher/pull/16/files#r1373385>.
>
>
There is probably a way to do this better - Build numbers are a very common thing, so VStudio probably has built-in support for incrementing them or at least putting the build-date into the text.

Looking around, there is a __DATE__ macro which expands to a string describing the date at which it was compiled.  You could stuff that into your string (see _swprintf for how) and then MessageBox with the date-ed string.
Thanks, fixed that!
Oh, i missed the RC circle. And as i understand such features can't be added in 2.3.x, am i right? It is sad. Because this feature is really needed for a query abstraction.
this change looks good to me .. especially since once the collection has been initialized, there is no reason to use this optimization anyway.
There is a bug in this code, I should delete the `defimpl Enum.OrdIterator, for: HashDict.Record`.

The idea is as follows. When the user passes a dict to one of the functions that require an ordered iterator, they'll get the error `** (UndefinedFunctionError) undefined function: Enum.OrdIterator.Any.ordered_iterator/1`. If it makes sense for their application, they could define an implementation of `Enum.OrdIterator` for the dict. And besides, it is only used in a few functions, so it should not be obligatory for users to implement ordered_iterator for their types if they don't want to. This is why I keep this function in a protocol of its own.
vscuser -> ship schema
Hi Kenneth,

On Tue, Sep 11, 2012 at 12:18 PM, Kenneth Hoste
<notifications@github.com> wrote:
> > +# File::      $File$
> > +# Date::      $Date$
>
> Are $File$ and $Date$ still supposed to be here?

At some later point in time we will attempt to try keyword
substitution with these;
I'd say ignore them for now and consider them as placeholders in the comments.

Also fixed sanitychecks in this one, so please review the pull request.

Fotis
log.debug or self.log.debug?
why is this present in this pull request?
It works, and without someone who can spent few hours debuging wtf is wrong with Flickr/Yahoo here, it's impossible to have working code and not using such "hack".
Again, what is the curl default? Imo the peer should be verified by default, because the `oauth2` protocol relies on the fact that communication happens over SSL.
This seems to need abstract="true" in order to fix #71
For consistency's sake, shouldn't this be get_note_message_text?
I like semicolons in my JS.
What do you think about something like:

```js

while ( clonedEvents.length ) {
  clonedEvents.shift().call( this, args );
}
```
I'm ok with this as is, but this could be wrong, given that an instance could have more than 1 ip associated with it. It would be great if we can correlate this someway else, but the only way I can think of is checking the database.
Hmm, or I could have misread the diff. Still, thanks for the patch! :)
It's ok to do that. I can't see any problem.
Falta a herança de ``object``
What do you think of this? Did this so we can just override it as necessary so I didn't have to pull in anything like active_support/inflector. (https://github.com/sparkapi/spark_api/pull/20/files#L2R21)
Feel free to add a link to the folder on github here.
This was being used to be able to draw the ingest link in the manage tab. How are we doing that now?
This method probably should be `synchronized` so that multiple concurrent calls will not stomp on each other.
Notice how we can just use a `ConcurrentHashMap` here in the unit tests for WorkspaceCache. That's because `RepositoryCache` does all the work of setting up the Infinispan caches, while `WorkspaceCache` is only dependent upon the caches being instances of `ConcurrentMap<NodeKey,CachedNode>`.
Will do. Added to the tech debt: https://enyojs.atlassian.net/browse/ENYO-1000?focusedCommentId=12647&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-12647
I have kept this code because I have not decided yet if fields like format and version will be displayed by the popup.
But I agree I will clean this code.
cpus ?
If we use this pattern For Realz, then we need to ensure that the size of hash-cache is bounded.
Why? Each extension should have its own dummy app../
Don't think so, I'll add in a check.
FWIW, _stopTenant uses explicit booleans..
Good point, I didn't realize they have a different result type. However, both Java and Scala allow overriding to be covariant in the return type, so we should be able to make `getDebugTarget` return `ScalaDebugTarget`.
What do you think of renaming this to ``scalaLibLocations``? Does it better express what the method is for?
OK, think we can probably work around that. I'll take a look later.

Adam
Yes, I can see what you mean about the field ordering (I think I missed that point). However some of your format strings are wrong (possibly because they were previously wrong, I think what appears to have happened is a couple of the arguments have been muddled).

%d makes more sense for year and %02d makes sense for month. Also to be honest we should possibly consider using strftime() to format the time rather than doing it like this. But we can sort that out later.
Carlos, can you rebase your master on the latest and have a branch with these two changes alone? I will pull it in and commit.
i'd prefer msg on a separate line in case we need to change it but no big deal (and likewise for other 2 buttons)
Why not?

```php
if (null === $content || '' === $content) {
    throw new NoResultException(sprintf('Could not execute query %s', $query));
}
```
This line blocks on the main thread - for a while - when opening a file for the first time.

Not sure of a solution (brain is fried) but concerning nonetheless.
исправил, проверьте пожалуйста
тут второе создание массива, первое тогда надо убрать
@fljot простите, а зачем вам так компилировать? Суть класса в том, чтобы все его внутренние методы определялись и менялись внутри класса, а не снаружи.
@Senior-Student или не то, что зависеть, а состоять из одной рекурсивной функции. Правда, если честно, что-то алгоритм в голову не напрашивается. =)
please remove this,  $ADDON_VIDEO_DIR is already set  (sourced 2 lines above via vdr.conf and vdr.conf.default)
you also have to change:
> VDR_ARG="$VDR_ARG --video=$ADDON_VIDEO_DIR"
to
> VDR_ARG="$VDR_ARG --video=\"$ADDON_VIDEO_DIR\""

to ensure clean start if user selects a directory with spaces in its name
So does the form know to submit modelType with the upload?
There is no need to closure this value if all you are doing is providing a get / set for it.  Just add it as a field on the object literal.
@stof this file can now be reduced to an empty class extending `Doctrine\Common\Persistence\Mappingr\Driver\PHPDriver`, no? :)
CS: lower-case `AS`.
Any possibility to send these patches upstream?
Thanks @arrbee for the explanation. @ben  I think callbacks would be a definite plus. Another option is to simply have an object represent the clone operation and it'd have events for the various callbacks.
videoFiles = sorted(filter(helpers.isMediaFile, fileList), key=lambda x: os.path.getsize(ek.ek(os.path.join, dirName, x))))

If you beat me to testing it feel free to verify that that works ;-P Other than that I'll pull this soon, thanks.
Here we go. Just updated.
Let me think about making it class based. It may not be that much work and would make it more future proof :).

Regarding all-to-all connectivity with labels. What we should have is a function like `label_aggregator(stcs, labels, mode)` where labels is a list of labels and mode specifies the way the time series inside a label are aggregated (e.g. 'mean'). The function returns a a generator which can be passed to `spectral_connectivity`.
regarding

> user may want to compute the connectivity between a seed in the lh and all sources in rh and vice versa (but not the connections lh-lh and rh-rh)

I think I could leave with 2 function calls and stick to seeds and targets.

an alternative it to pass a sparse connectivity matrix where the non-zeros are the edges to compute.

I am not sure. Just throwing ideas...
would put is in logger.debug
it's common not to mark bad channels in the noise cov and to rely on the data bads to remove the proper lines and columns in the noise cov but based on channel names. I think it would be much more robust. Basically I would take from the noise cov the channels present in the data and raise an error if one data channel in not present in the noise cov.
We should mention that the same e-mail address as the one used for the github account should be used
is this code block covered by a test?
? I think you should say what this means. Does that mean there's no temporal regularization? Is this what it should typically be? etc...
Had to convert Character to Integer because of Constructors. Hope this is acceptable.
We don't want this method here. MessageSender should be always setup when returned from MessageService.
Or maybe deprecate this default and remove it on next version ?
Any reason why this attempt shouldn't be in it's own file?
Bad i18n. This is almost impossible to localize for languages other than English.
Thanks for moving this method into this "class". It's been needed for a while.
I think this should be "createBuilder()". Names of methods in the text of the documentation also always include the parentheses.
Please move this to the correct position in the class definition. Now it is between `toLong` and `toString`. Maybe it could be after `byteAt`.
Users will only deal with the MailboxType, not the Mailbox itself, so you can rename the Mailboxes and keep the MailboxTypes intact.
А инкрементировать API level в `interfaces/iinfo.h` кто будет?
Why is this method ```protected``` and not ```public``` ?
It seems it's already fix on master, please rebase
revert copyright year
same for urlencode http://twig.sensiolabs.org/doc/filters/url_encode.html
Should use ```isLiveMode()```
can we just remove these?
no change  here. read the first comment
Using `defn` inside another `defn` is not a good practices, as functions defined with `defn` always end up in the top-level.
Invalid method ID?  Please be consistent with your error message - this will confuse the hell out of a sys admin or support engineer parsing logs!  :)  Look at what we do for other commands in this regard.
I'm not quite sure what you mean here. I appreciate that some of the if
statements are repeated in _uniform_y and _proportional_y, but the logic
applied within is different. Could you clarify please?

On 14 March 2012 21:32, Benjamin Root <
reply@reply.github.com
> wrote:

> > @@ -613,16 +630,26 @@ def _uniform_y(self, N):
> >          if self.extend == 'neither':
> >              y = np.linspace(0, 1, N)
> >          else:
> > +            extendlength = 0.05
>
> Maybe this stuff should be refactored into a internal function (i.e., one
> that has a name that starts with an underscore) so that it is not repeated
> twice?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/matplotlib/matplotlib/pull/766/files#r560139
>
I forgot that preferences could be a dict or a list of 2-tuples :(  So this will need to be more complicated

preferences = profile_args.setdefault('preferences', { })
if isinstance(preferences, dict):
    preferences['extensions.jsbridge.port'] =  jsbridge_port # note that this overrides any value that we have here :( ideally we would read it
else:
    # preferences is a list of 2-tuples
    preferences.append({'extensions.jsbridge.port', jsbridge_port)) # putting it at the end of the list will override and prior instances
Can we split this out to avoid duplication with the connect method?
cool? :-)
I made some changes here to select the search container in the widget, then the children, rather then doing a find.
I moved the common code between the search widgets to search_util.js
I think both the principalId and canManage are passed into the widget through widgetData. canManage can be used to determine whether or not to show the button.
It's better to put logout in the `finally` block. Otherwise, the logout might not occur.

We've got a nice convenience method for this now, so that you don't have to put yet another try-catch in the `finally`:

    SparseUtils.logoutQuietly(session);
Spaces were here instead of tabs.
Add a class to the templates?
isn't this a duplicate of what's above?
At this point, if I am understanding correctly, the session is still referenced from the connection list.  So we either need to move this free until after the connection no longer references the session, or have the hazard checking code cope with the case where session->hazard == NULL.
Are you sure that ``$mixed`` (of type "object<Zend\Navigation\AbstractContainer>") is traversable?
Are you sure that the method "quoteIdentifier()" exists? Please also check line 311.
it doesn't seem like we support this file, revert these changes and put an alert instead
All of these string literals should be escaped for Javascript…

```jinja
'{{ google_analytics.apiKey|e('js') }}'
```
Reminder: add php format
Existe aqui uma inconsistênciazita: 'payment_id', vou tratar disso.
This was resolved in 704ffcc.
I guess reconfigure can be a private method?
@visionmedia : yeah, this is much more friendly to the eye. Good call.
Do you have to put tuple here? Just for optimization?
What is this file???
Change to:

```
../manage.py makemessages -l <language code> -d djangojs
```
This could be a fragment:

```
<a><img src="/static/piplmesh/panel/openhours/external_website.png" /></a>
```

For the rest, I am not sure what we are talking about. Maybe I do not understand the pull request. But I understood, that we are not yet moving map around and the only thing is that we based on node location fetch after page loads in user browser Odpiralni Casi data from their server and display it. And the question is how we render that data we fetch from them. And I am suggesting a simple fragment you have in template prepared and is thus in DOM and you clone it, fill in data, and inject into DOM.

So where you do 2.), which parts are 3.)?

So I am not sure what are you talking about when you are talking about switching nodes? We do not yet support that? Or do we? (We currently support only mocking nodes, but this is just for staff users.)

Anyway, see how we are doing posts or comments to see how you "enumerate/uniquely label/ these loaded HTML blocks".
Indentation is still wrong in this file. Please check tabs and spaces.
Define a setting for number of days and use this here. And return only `True` of `False`.
No, this is not correct way to of building URL to contact. Please read `url` template tag documentation. So don't stick manually home and contact together. This defeats the purpose of using the tag.
We write *e-mail*.
You could put the invoke() code in __call__ and thus make the class instance callable.
Rather than use a branch (which could theoretically get new commits), let's use `:tag => '0.11.0'`
I need it ! so Yes, it is needed :D
Nested exceptions tend to avoid stack trace confusion when re-throwing.  Also, what's up with the asymmetry between the read and write path here?  Is there a reason the write path IOExceptions are ignored and these are thrown?
Github doesn't seem to show it, but there is trailing whitespace on this line, and typo 'Calidation'
Good catch, Lee. Thanks. Resolved, updated pull request.

On Sat, May 19, 2012 at 3:43 PM, Lee <
reply@reply.github.com
> wrote:

> > @@ -192,5 +199,8 @@
> >      {:pre [(string? text)]}
> >      (let [categorizer (DocumentCategorizerME. model)
> >            outcomes (.categorize categorizer text)]
> > -      (.getBestCategory categorizer outcomes))))
> > -
> > +      (with-meta
> > +        {:best_category (.getBestCategory categorizer outcomes)}
>
> idiomatic clojure is hyphens instead of underscores, how about
> `:best-category`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dakrone/clojure-opennlp/pull/14/files#r849801
>



-- 
David Park
(m) +1.317.270.4287
daviddpark at gmail dot com
Here, I think git as been a little abused.
gui_reset is still here with this lines. The corresponding diff modif is just 15 lines above. Don't know why git add and remove this in the same diff thought... Maybe a error from my one ?
Why a separate file ? Put this in dbal.xml (where the DBAL data collector is also registered)
Same here... Even if I love the idea, I think it is not DoctrineModule's scope. ZfcUser, which aims at providing auth utilities, [has it](https://github.com/ZF-Commons/ZfcUser/blob/master/src/ZfcUser/Controller/Plugin/ZfcUserAuthentication.php).

This may instead be suggested for ZF2 itself...
does it contain valid user identity? in the `if` statement you check whether it equals to `id_res`.  so I think the identity is another field. would you check it or  should I?
but I'm trying to express the "IF you use a Maven like tool"
Is this class thread safe?
This isn't always true. Even well-behaved proxies can end up with something bad here if  the client sends their own X-Forwarded-For. See my other comment about including this section.
Thoughts on making the Mahout stuff its own subproject? A lot of people running pig nots are not using Mahout.
Guess I didn't finish the thought completely - the idea is that  

    (settings.tokenValue == "id") ? item.name = item.id : item.name = item[settings.tokenValue];
and
    item.name = item[settings.tokenValue]

Are the same, since when settings.tokenValue is "id" then it's item.name = item.id.  Less code, fewer tests.
If commented lets remove.
Shouldn't CamelCase.
make it configurable? It should check the resolver if it's not NULL.
I can see why you've added the stop here, assuming the events come back down from the container again, but its not obvious to me why we're doing this in the first place, especially for some containers and not others . 

I'm sure the code is right.  I just feel the need for comment explaining things  :)
Of course you could just follow what the [_ls] [1] completion code that ships with zsh

```
local is_gnu

_pick_variant -r is_gnu gnu=gnu unix --help

if [[ "$OSTYPE" = (netbsd*|freebsd*|openbsd*|darwin*) && "$is_gnu" != gnu ]]; then
  if [[ "$OSTYPE" = (freebsd*|darwin*) ]]; then
    -G
  else
    <no color>
  fi
else
   --color=auto
fi
```

This should be pretty fast and work across multiple distros

  [1]: https://github.com/zsh-users/zsh/blob/master/Completion/Unix/Command/_ls
sounds sloppy maybe but IMO this doesn't really matter, for something you run once per process a bytes is meh
Since we will only use hsb in case the match for rgb fails, what about calculating the regex in case rgb doesnt match? Here you are calculating it always and most of the time is not necessary.
Ditto comment above about `argparse`
This stuff, which is currently commented out, was used to generate the following images.  Here's the correlation function (CF) drawn by the `CorrFunc` using the `GSObject` draw method:
https://dl.dropbox.com/u/495780/GalSim/10Dec2012/cf_drawn.pdf

And here's the CF calculated from the input noise field using standard FFT methods, which should be exactly the same as the drawn image above:
https://dl.dropbox.com/u/495780/GalSim/10Dec2012/cf_original.pdf

Here are the residuals:
https://dl.dropbox.com/u/495780/GalSim/10Dec2012/cf_drawn_residual.pdf

These are good almost everywhere, except on one edge.  I'm going to investigate whether this is a generic issue with `draw()` for `SBInterpolatedImage`-type objects: we would not have spotted it at the edges before since *normall* these edges are zero.  But does anyone have any ideas?

The level of the difference is not huge: it seems that the drawn image is slightly smoothed/suppressed, perhaps due to an interpolation effect with neighbouring zeros.  But it's enough to make this test fail if the full images are compared.  Below, therefore, I compare the images within a single pixel border region.
My own typo: "the the" -> "that the"
in the SDK, I think we will want two separate request and response objects - one for create and one for update
Ensure your VS environment is set for tabs.
Didn't we already have a mechanism to load native libs?
This method is bad! It should mark the thread as interrupted again, before propagating the exception (but not this pull request's fault). We also have a RuntimeInterruptedException that Peter added: creating one of those will automatically set the the current thread as interrupted.
should be 'Camptocamp Kibana Module'
I understand where you are coming from, but for `_.functions` are we not looking for callable properties (functions, methods or what ever you want to call them) on an object?  If that is so, then looking at `typeof` will garner the same set of results that `toString.call` would produce, or yet in the special case you are trying to point out, `typeof` would return even MORE methods if a method is callable but NOT a function as you are pointing out.

If `_.functions` is not for returning all callable methods on an object, then the two should NOT be aliases for each other.
underclared dependency
Why not index.css? . and i suppose it should be the only one: i.e. no addtional prettify.css include required
The `:locals` parameter should be removed.
There are so many warnings like that already that I missed it!
I thought it made sense to tell the caller if the function did anything or not, which is why I included a return value.
Just for future compatibility really, doesn't matter in this case.
req is a stubbed HTTP Server Request object. It's completely fake, but it looks enough like a real request so that the `request-timeout` module doesn't know the difference. No actual request is made.
10 * window.devicePixelRatio ? :) or something like that. This seems a pretty important UX issue
Are you saying:  don't use checkpoint to insert results at all anymore (even the "first one")?
Routes has been updated.
I think that `r += lib_t2 * len(self.lib)` would make sense to replace the if statement and python for-loop here (lines 410-412).

the same could be done with the css and lib for-loops above by converting them to a tuple, like ` r += css_t * len(self.css) % tuple(self.css)` and slightly cleaner if css was already a tuple, but it's probably not worth it in this case to conver those.
/bin/sh please
Pune te rog un doc block pentru clasa. Inainte de '{' trebuie sa fie spatiu.
deci nu sunt ghilimele?
https://github.com/MPS-UPB/Unknown-Revengers/pull/18#preview_bucket_180
httpProviderProvider is correct.

This can't be final because I inject a new value into it in my tests.
AuthToken is specific to the RestAuthenticationModule.  There is no WebSocketModule yet in any capacity.

You actually bring up a good point.  I think the whole AuthenticationModule thing may be a bit underdeveloped.

A Module has three parts.  A FetchStrategy (how do logins/renewing tokens/etc work?), an ExpireStrategy (how does logging out  work), and a ApplyStrategy (How is a login maintained/re-authenticated).  Right now we have One class which does all three which I think may not be the best of ideas.  On the other hand, making it bigger will make it much more complicated to develop for.
1. not sure for all.
2. apache+passenger yes:

        RailsEnv production
        <Directory /srv/redmine/pub>
                RailsBaseURI /redmine
                ...

        </Directory>

, and no settings in any environment.rb or such. That is my setup. 

I manually tested the sidebar on
ruby-1.8.7-p358@redmine14 passenger-3.0.13
ruby-1.8.7-p358@redmine20 passenger-3.0.13
ruby-1.9.3-p194@redmine20 passenger-3.0.13

I'm currently under high workload, so i will not be able to test other setups the next 4 weeks.

I consider this patch a bad hack, since i did not yet understand what is happening on the actionpack layer. There must be a more clean solution.
Убери это из коммита.
minusI и minusU - непонятные идентификаторы.
Здесь достаточно объявить output как Writer. Для файла использовать FileWriter, а для System.out -- OutputStreamWriter.
Test if `checked=true` here.
this is no longer needed because the bug was resolved WONTFIX.
The following section looks like it needs sub/subsubsection names adding.
How about checking for files within templates/config/initializers on run? This way adding additional frameworks is as simple as dropping in a framework.txt and a simple_form_framework.rb (minus seeing if the framework works as intended).
In order to make sklearn.linear_model.LogisticRegression link to it's reference page, and same for those below,
you can write them as  ``` :class:`sklearn.linear_model.LogisticRegression` ``` instead.
use camel case for variable names
It was done to keep an alphabetical order
a Android Virtual -> an Android Virtual

Clock -> Click

Button -> button
There were a few of these semi-colons floating around. We don't need them at the end of functions; just at the end declarations.
The CBroadcaster would be useful outside of lookup join, but won't move as part of this commit
I'm not sure if I'm understanding the issue here. 

If the concern is in ensuring that subdirectories are added to the watch, I don't see an issue here.  In the current implementation <code>IDirectoryIterator \*dirIter</code> will be of type <code>CLinuxDirectoryIterator\*</code> which will return all subdirectories; other derived class may implement this behavior differently.  In either case, the subdirectories that are added to the watch are based on how the derived class implements the iterator.  With <code>CLinuxDirectoryIterator</code> all subdirectories would be added.  

Am I understanding your concerns with this code?