labofsahil/github-event-dataset-2012 huggingface dataset:
Fixed. Boost uses shape. Matlab uses scale. R uses nothing.
You've used a format operation again... ;-)
Please fix the headers, for consistency.
Does this mean we cannot have more than 9 pending req?
yes and no. That can be just ```el.node```.  I think at one point in time I thought I had to grab the parent if it was VML
I'm reusing some of this code in my other `g` attempt - is this a typo?  this conditional sets the same object for both cases.
Can you check if it is POST request we are in? And otherwise throw a 403 error?
Added constraints for future dates.
Need some options to go with this select.
I don't think these tests are really needed here. If they don't exist, we're in trouble in many other ways :P.
I'm pretty sure we don't need these, right?
Hmm. I guess it makes sense that you would want to wait until the dragging stops here.
multi line the object and spacing `} );` -> `});`
doesn't this need to set ``application =...``?
`/`, `:`, `\`, `|`, `<`, `>`, `"`, `?` and `*` are already always stripped.
it may be undefined if one of the extjs extensions or utility method call Nexus.log *on setup*, not when they are used on runtime of the Nexus UI.
Btw I saw many manifests temp files the other day. We are obviously not deleting them, is this a bug? Is this the place to put remove in?
validate_hex is just a function i've added above. had a look for a wordpress colour validator but couldn't see anything suitable :(
Return the continuation Task and execute the continuation synchronously.
You need to add a guard for settings here too.

Storage -> this
How would you mock this? Actually I don't have a clue how to mock a destructor.
Ack. Sorry this should be in the construct with a check for core. I'll update the PR. Thanks Mark!
Should the length of an enum field be the number of elements in it?  If you're generating inputs, length is used a maxlength, which could result in incorrectly generated inputs.
end of file new line!
constants should be declared before properties
I think "Then I should be alerted that the user is already in the group" would be better. It's more specific to the user's experience.
Can there ever be more than one entry here for a character?
That was a typo indeed.

It's pretty simple to do this, the DI component takes care of this by itself if you implement it correctly.

Example: https://github.com/siphoc/Slic/blob/master/src/Slic/Application.php#L90-115
This loads the config file you provide. If you take a deeper look into the YamlFileLoader, you'll see it autoloads the parameters (as well as the services, but that's for later I guess):
https://github.com/symfony/DependencyInjection/blob/master/Loader/YamlFileLoader.php#L54-59
the syntax is copy and past from same file : 
```
kernel.js:                $([IPython.events]).trigger({type: 'status_busy.Kernel', kernel: this});
kernel.js:                $([IPython.events]).trigger({type: 'status_idle.Kernel', kernel: this});
kernel.js:                $([IPython.events]).trigger({type: 'status_dead.Kernel', kernel: this});
```
about the order, I didn't saw is use to be the other order... What do you think is the best way ?
please remove the lines you comment out, as we can always look at source history for what it was.
You case is good, but we cant do it, due to weighted edges. I see what you meant. The code should be something like this (in the loop):

```java
 if (!g.containsEdge(e)) return false;
 if (!g.getEdgeSource(e).equals(source)) return false;
 if (!g.getEdgeTarget(e).equals(target)) return false;
 if (Math.abs(getEdgeWeight(e) - g.getEdgeWeight(e)) > 10e-7) return false;
```
So, the line 

```java
 if (Math.abs(getEdgeWeight(e) - g.getEdgeWeight(e)) > 10e-7) return false;
```
Checks weight of the same edge (of the same object) and it always be true. Than we need to find equals edge in the second graphs ("ee" in our case) and perform checking for different objects.

PS. We can't use just g.getEdge(source, target) due to graphs with multiple edges.
```python
fqdn = os.environ.get('COLLECTD_HOSTNAME', socket.getfqdn()
```
Ehrm... what? You may want to change the body of the loop

~~~~~ c
do {
		ptr = buf->data;
		/* Whilst we're searching for the pack */
		while (1) {
			if (buf->offset == 0) {
				break;
			}

			error = git_pkt_parse_line(&pkt, ptr, &line_end, buf->offset);
			if (error == GIT_ESHORTBUFFER)
				break;

			if (error < GIT_SUCCESS)
				return error;

			if (pkt->type == GIT_PKT_PACK) {
				git__free(pkt);
				return git_fetch__download_pack(out, buf->data, buf->offset, t->socket, repo);
			}

			/* For now we don't care about anything */
			git__free(pkt);
			gitno_consume(buf, line_end);
		}

		error = gitno_recv(buf);
		if (error < 0)
			return -1;
		if (error == 0)/* Orderly shutdown */
			return 0;
	} while (read_bytes > 0);

	return read_bytes;
~~~~~~~
use https://developer.mozilla.org/docs/Project:How_to_Help
Seeing that there is most likely going to be a contact form of some sort, there may end up being tests for this. Emptied the file but keeping it for now.
You don't have to pass the "password" argument to the function.

See https://github.com/mozilla/kitsune/blob/master/apps/users/tests/__init__.py#L49
I might just re-order this so that matchedAll is first, that way if there are additional declarations with no assignments they can be grouped together at the start.
`s/sysog/syslog`
good point.  I wish the hbase client library had the exception groups created at declaration time rather than instantiation time.  That's something to change in the future.
We don't print anything ilke "Safe return" or "aborting" other places that we do this, so I'd do the consistent thing and either change the others or leave it out of this one. Not a big deal.
it's not new.  it's heavily refactored code from 2007 destined to be replaced in Q1 2013 by in intern - unless the tedium of this back end code kills him...
Same here, what if a user enters ``'report'`` as a value? Why don't you just keep the value? Or do something like

<pre>
$date_range_type != 'build' ? 'report' : 'build'
</pre>

Also, please use simple quotes over double quotes as much as possible in PHP.
It's good to add new keys to translation file.
Should the InstallerException be located inside the Installer_lib.php or move it away into `"<i don't know where>` ?
I'm presuming the lsat digit should be 7 (seven)?
... though the cast could throw a ClassCastException if for whatever reason your last parameter isn't a Class<?>. For example, it'll probably crash if the last parameter is a GenericArrayType like `List<String>[]`. Defend against that by doing
   hasCallback = typeToCheck instanceof Class && Callback.class.isAssignableFrom(...);
Github doesn't know how to escape less than signs. What I typed was "I think you want a List{String}" or something. Silly github.
It's a problem ? in this case, the ldap bind can be possible, so a `UsernameNotFoundException` will be throw.
I'm not against this change but then, `RequestHelper`, `SessionHelper` and `RouterHelper` should also be moved. What others think?
Does it really need to be protected?
ali too barname bezar behem yadavari kon free shodam inaro dorost konam
kheyli zayast !
hatta nemishe yeki dad
yani masalan khorooji ino bebin :D `fixed:top`
There seems to be some inconsistency with spacing after keywords in here -- the PlasmaShop coding style prefers a space:  ```if (condition)```  ```while (condition)``` etc.
Why do you think these are separate questions? Both are about whether or not to replace existing files with new files.
The `subject`, `tags` and `comment_snippets` methods are parts of the interface that need to be fulfilled by any subclass of `ZendeskTicket`.

Part of the test is that those methods are invoked, so they are kind of important to have there. Or have I not understood you correctly?
For my own edification: sometimes (what look to me like) statements end in a semicolon, sometimes the don't: is there any rationale?
none ... just for devs haven't to care about the order of property in outlet helper ...
Please prefer "Sentence casing" over "Word Casing".  Also, we avoid saying the word "item" in the UI because it's too generic.
So this should be "Only all new comments" and "Only comments on photos, movies and albums".

Also - wouldn't it make more sense to have these be checkboxes as opposed to a dropdown?
I added a member variable to Import to hold the header we extracted from the user's file so that they can keep their data fields. this is particuarly useful if they have a ton of predefined fields like in a toolbox verb lexicon.
Why not Collections.sychronizedList()?
Don't register this attribute on resource, as it is not resource attribute.

Instead create new operation maybe called "add-modcluster"..
by implementing OperationStepHandler & defining it via operation definition
```java
static final OperationDefnition DEFINITION = new SimpleOperationDefinitionBuilder("add-modcluster", ModClusterExtension.getResolver())
.addParameters(ModClusterConfigResourceDefinition.ATTRIBUTES)
.build()
```

this way you wouldn't need to change any resources, add end up with simple operation call from CLI
it should be something like this:
```
/subsystem=modcluster:add-modcluster(atribute1=value,attribute2=value,...)
```
sorry, This line below is correct. I have tested and passed compile, but I forgot to commit about fixed code error.

double ver = Double.parseDouble(testMCBans.getDescription().getVersion().trim());
You can add background-repeat, backgound-position to #ssl-indicator and only change the background images here.
same comment as above, I suggest to do this through the JSON declaration format
If we remove the date update we shouldn't leave dead code behind.
I think this is not the place for this. Can we have a generic resource with the dialogs or even generate dynamically the markup?
nope. import is reserved by the language for future use, see https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Reserved_Words
Yes, just setting it by default closer to where it is defined would probably make things more clear if someone is debugging.  Especially if it is ignored by implementations that don't understand.
Didn't find any other usages except one that was already worked around and waits for bug 793955.
Ugh, the W3C validator complains on that:
> The label element may contain at most one input, button, select, textarea, or keygen descendant.

As this structure has been applied everywhere in this BB, I’m afraid we have an issue. :-/
Update comment here
Remove this white space.
As there are some callbacks with long body content, when the readability is compromised I prefer to just intent one place and avoid starting on the first parameter column.

I hope you agree.
Yep, it isn't used. The reason it doesn't send any data is that it isn't needed and that for some reason putting any data in the variant hash made the other operator crash. For some reason this gets called even when receiving data from the bus - I don't know why. My guess (!) about what happens internally is that this operator is first used to figure out the signature/data structure and then the other one to actually populate the date. I can spend more time investigating this but it isn't really needed.
Ah yeah, that's true. In the past I've ended up with weird mixes of import foo + from bar import blah, etc. and I didn't like how it looked, so I often change them so they look nicer. Always remember the first rule:

http://www.python.org/dev/peps/pep-0008/#a-foolish-consistency-is-the-hobgoblin-of-little-minds

But in this case, it looks fine as is, so let's go with that.
Das in der Klammer ist ein Satz. Fehlt da dann ein Punkt am Ende?
_path.Substring(0, _path.LastIndexOf(@"\")) try to get from c:\folder\file.txt to c:\folder and ensure that folder exists before writing to a file under that folder.   trailing slash?
Row, column. Row, column. Row. Column.
this is incorrect style for this project
Just use `SecureRandom.hex`.
I think you can replace heavyhitterorder with Ordering.by { h: HeavyHitter =>   (h.count, h.item) }
We need a task for this I think. Alongside it is the ability to dictate from the client?
ohhh they inherit, I didn't notice that, ok cool
Ok I have updated it with 2 levels of keys for the ones that require no renaming or processing.
you should remove the ``else`` as the ``if`` returns
.any? should already only return true or false.
what can be
not `sqllite` but `sqlite`
that's the only variable left, don't need to align with old attributes
I'd obviously rather not do this, but it seems capybara won't always wait long enough for the browser to reflect the content change in the assertion. (Heisenbug) any suggestions?
"str" and "set" are both built-in functions. Please don't override them.
You can get rid of parenthesis around `requirer.id`.
I'd say helper addon used by an add-on sdk.
Isn't there a missing comma after `In general`?
What is this global value `prototype`?
No, It doesn't. I assume it has to parse before executing, since it must do function hoisting.

A for-sure time would be a new `script` tag, above the main one, that only has `JS_START_TIME = new Date()` in it, and never gets combined into the rest.
can we have this `===` so that it is type checked
do we have a method to retrieve the current capacity of the buffer? A test for that woud be nice here.
Shall I rename the folder to 'xpi'?
Fine.
Fixed
I'm not sure we should even mention the alternate methods in here. It just muddies the issue.
wouldn't mind moving this up near the top of the file so it's out of the function
call to `isValidHost` should be enough here if not maybe we are having a bug.
Simplify the above to:
```php
$pattern = null;
if ($namespace !== '') {
    /* set $prefix and $pattern */
}
```
This line must go inline with the above line. https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#41-extends-and-implements
is the remove not suppose to have a space here?
Dekorator wnętrz
I also wanted to be able to easily "Deploy" to the Emulator, so I've added support for that to the deployment scripts.
ternary: `tempTodo.done = !tempTodo.done`
maybe saying that this theme is intended for writing other themes on top of it would be a good idea?
Meu nome é Breno Pitty, assim como meu primo Brad.
you should typehint the UrlGeneratorInterface as it is the only part of RouterInterface you really need :)
If you're also accepting Criteria objects here, we should consider dropping all other arguments and refactor code that uses this method everywhere
this should go into your global git configuration as its specific to your dev environment, not to the project.

(like the `*~` above. ok, so it was not clean before either)
I am also thinking about adding a feature of "stealing" specs from remote modules compile time (when those remote modules are in fact available)
Skor by som sem dal vsetky potrebne "from infoboard.nieco import Trieda" - podobne ako v Jave alebo PHP nepises vsade nazvy tried so vsetkymi namespacami ale hore si definujes co pouzivas.
Also, co za verzia pythonu to vlastne je, 2 ci 3? Matie ma nepritomnost infoboard/__init__.py (normalne sa proste necha prazdny) - to tusim pribudlo az v pythone 3.3... (Alebo si ho zabudol commitnut?)
Could you also add sourceURLs here, and to the other .eb files?
All links from official docmentation http://vk.com/developers.php?oid=-1&p=%D0%90%D0%B2%D1%82%D0%BE%D1%80%D0%B8%D0%B7%D0%B0%D1%86%D0%B8%D1%8F_%D1%81%D0%B0%D0%B9%D1%82%D0%BE%D0%B2
Looks good.

On 1 April 2012 16:47, sgilda
<reply@reply.github.com>
wrote:
>> +
>> +All you need to build this project is Java 6.0 (Java SDK 1.6) or better, Maven 3.0 or better.
>> +
>> +The application this project produces is designed to be run on JBoss Enterprise Application Platform 6 or JBoss AS 7.
>> +
>> +
>> +Configure Maven
>> +---------------
>> +
>> +If you have not yet done so, you must [Configure Maven](../README.html/#mavenconfiguration) before testing the quickstarts.
>> +
>> +
>> +Add an Application User
>> +---------------
>> +
>> +This quickstart uses a secured management interface and requires that you create an application user to access the running application. Instructions to set up an Application user can be found here:  [Add an Application User](../README.html/#addapplicationuser)
>
> I agree. I think the overview needs to be removed or made clear that it's an overview. How about this:
>
> This quickstart takes the following steps to implement EJB security:
> 1. Define a security domain in the `standalone.xml` configuration file.
> 2. Add an application user with access rights to the application.
> 3. Add a security domain reference to `WEB-INF/jboss-web.xml`.
> 4. Add a security constraint to the `WEB-INF/web.xml` .
> 5. Add a security annotation to the EJB declaration.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/quickstart/pull/167/files#r630827
Wouldn’t this by a Pyrrhic victory? To make Object.create(null) work but also make it prohibitively expensive such that it would be unwise to use?
Imports should be grouped in the following order:

    standard library imports
    related third party imports
    local application/library specific imports
here, also: is_developer_opportunities_header_visible
Was this `width` not being respected anyway because it's part of a `td`?
Well, if I hadn't tested it, I wouldn't have been surprised ;)
I know this seems nitpicky but any particular reason you went with a dash here instead of an underscore?
What I was trying to say is that the pipe operator `|` splits the expression into two, so that regex is __not__ matching `http://bofip.impots.gouv.fr/bofip/ext/.+` or `http://bofip.impots.gouv.fr/bofip/simple/.+` as you're expecting, but `http://bofip.impots.gouv.fr/bofip/ext` or `simple\/.+` You want to use parentheses (preferably non-capturing) to group `ext` and `simple`. I.e. `http://bofip.impots.gouv.fr/bofip/(?:ext|simple)/.+`

(removed some escaping for clarity)
stray space in here
nit: `strokeWidth is better.
icecast_mount has a prepended slash; icecast_mountpoint does not.

Why? Who knows.
`strange_case.extensions.uuid.uuid` ?

and, below, `strange_case.extensions.uuid.urn`
spurious whitespace or intenional
Hehe, so @evert has a new todo :-)
And btw, the resize should also be done when the treemenu is unfold otherwise you can get something like:
![Treemenu issue](http://vrac.pwet.fr/admin3_treemenu_unfold.png)
оно меняется?
Does this action have 4-character indentation, or is it just me?
i did that because attribute name is dayOfMonth, so i think could be
strange to do anyDate.withDate( 10 ) and look for it in anyDate.dayOfMonth
...


2012/11/25 Roland Tepp <notifications@github.com>

> In source/ceylon/time/base/PeriodBehavior.ceylon:
>
> > @@ -0,0 +1,78 @@
> > +shared interface PeriodBehavior<Element> given Element satisfies PeriodBehavior<Element>{
> > +
> > +	doc "Returns a copy of this period with the specified amount of years."
> > +	shared formal Element withYears(Integer year);
> > +	
> > +	doc "Returns a copy of this period with the specified amount of months."
> > +	shared formal Element withMonths(Integer month);
> > +	
> > +	doc "Returns a copy of this period with the specified amount of days."
> > +	shared formal Element withDaysOfMonth(Integer daysOfMonth);
>
> Yeah, I am currently not all that sure about how to call this field in
> Date/DateTime. It seems to me a bit verbose. How about calling dayOfMonth
> simply a date? Roland
> … <#13b37895aa5164b2_>
> On 25.11.2012, at 11:40, Diego Coronel <notifications@github.com> wrote:
> In source/ceylon/time/base/PeriodBehavior.ceylon: > @@ -0,0 +1,78 @@ >
> +shared interface PeriodBehavior<Element> given Element satisfies
> PeriodBehavior<Element>{ > + > + doc "Returns a copy of this period with
> the specified amount of years." > + shared formal Element withYears(Integer
> year); > + > + doc "Returns a copy of this period with the specified amount
> of months." > + shared formal Element withMonths(Integer month); > + > +
> doc "Returns a copy of this period with the specified amount of days." > +
> shared formal Element withDaysOfMonth(Integer daysOfMonth); Ooops.. Sry it
> was because copy and paste from date behavior Em 25/11/2012 05:10, "Roland
> Tepp" <notifications@github.com> escreveu: … — Reply to this email
> directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/luolong/ceylon.time/pull/4/files#r2216633>.
>
>
we should have a region for this methods...
We will use them in some other tests and then we will have to duplicate this methods let's avoid this from start
At this point, I think the actual code is cleaner than what rburchell suggested.
Style:

catch statement on new line
The default behavior of split is to split on whitespace. Also, splitting on whitespace obviates the need to strip.
It would be nice to have a more complicated query in the example...something that, say, asked for Class[foo] or something. That's the most common use case for this stuff.
```erb
<% if @image.errors.empty? %>
```
I see, in that case should the start/stop tenant methods below bypass the `setTimeout` if there is an error?
Maybe replacing it by something like that:
```scala
private val managedThreads
private def threads= managedThreads
```
With an scarier name than `managedThreads`.
And I think this can be generalized even further...
any idea what i do wrong here? maybe i don't understand phpunit well enough? or is there actually a bug i am too blind to see?
this works, but not sure if there would be a more elegant way to determine the version of symfony we handle. also not sure if there is an ugly performance penalty for method_exists?
Actually it's a fair point, possibly breaking EPG and EPG GRAB code under 2 defines would be a good idea. I can deal with that though so leave as is.
All providers used to call it: ENDPOINT_URL
ranges need to consume a value instead because calls aren't selectors.
В задании написано "переменную DNA типа NSMutableArray" поэтому не property, я понял буквально.
Кажется, что-то вы забыли убрать из "примерного" варианта лекции 1. =)
Поле Description используется исключительно для логирования. Нельзя (ну как, никто не запретит, но это моветон) его использовать для других целей
https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSObject_Class/Reference/Reference.html#//apple_ref/occ/clm/NSObject/description

Проперти создать нужно, почти что по заданию - такова была тема лекций. А делать ли его readonly или нет - исключительно ваш выбор :)
Same thing here :)
file is used for different projects
Jess wanted only vertical.
It occurs to me, one way of resolving this would be that, 
Given the part of the uri which changes is the path to the sandbox where the features are executed
When we compare the actual JSON with the expected JSON
Then we could only compare the part of the uri within the sandbox e.g. <code>cucumber-js-sandbox/features/a.feature</code>
And ignore everything before that.

WDYT?
One tiny nit-pick: please check in the test that the username is appended to the `?userid=` query. Then it's ready for merging.
There's a repeated write here
Tutaj brakuje przecinka między "powtarzał" a "poinformuj".
to me we should not replace the standard way of parametrizing an ICA which is by the number of components. But add an extra option to set this value in a data driven way using explained variance. I would keep n_components and detect that if its between 0 and 1 n_components is provided in terms of explained variance.
I mean above, not below.
This will need to check to make sure it's an integer, right?
Now that after the discussion about copying Raw in #229 I think having this method may be "dangerous". A user could e.g., find events, create Epochs, and then crop the Raw object.. which would totally mess up the timing. Maybe this function shouldn't modify the object in-place but instead return a cropped copy, WDYT?
Check for error before trying to use result.
I really hate that this filename has spaces in it. It'll still work, but I want to go on record with that.
Change this to `// ... add node definitions to the root of the tree`
You should mention that the first matching item of the map will be used. So the order in the map is important
Yea, the constants "scared" me at first to - I wondered - even though I know this would never get into the core - if the component were relying on these contants for some reason!

What if we just use variables instead - I think then it's probably obvious that since we're just setting these variables locally, that you could be storing them however you want.
dont think it makes sense to include the ``<container>`` tag here so i left it out .. also are the underscores not dashes in xml?
系统环境变量PATH中。
记录
状态
This string should be internationalized.
case Some(Left(l)) => sender ! l.createChannel
case Some(Right(_)) => what do you do when it's a Cancellable?
case None => ...
What happens if there's a failure here?
Got it. See my previous comment.
Если ты класс определяешь в .cpp, то и эти функции можно определять инлайн, не отдельно, прямо в определении класса.
Содержимое свитча не отбивается же, ну.
Ну што ты как первокурсник какой-то, «оно как-то плохо работало, строчки
местами поменял, стало хорошо». Верни в initializer list, навесь const и
убедись, что у тебя правильный порядок инициализации (а он соответствует
порядку объявления членов в классе).
I think line 80,81 can be compressed to  
 (filter #(= (.id %) id) rows)
please revert extraline
Maybe it would be better to add a private constructor to make sure this utility class is never instantiated.
I've come to really dislike any code that operates on Sass lists. It's really quite unpleasant to write or read. So I opened a new issue for sass to add anonymous functions: https://github.com/nex3/sass/issues/472
I really think it’s either:

* `Phone.dataset.helpContent` if we consider *properties* are translated
* or `Phone.data-help-content` if we consider *attributes* are translated

As a developer, I like the former because that’s the way `data-*` attributes are mapped to properties in JavaScript (same thing for style properties, by the way).

I understand the latter is more straight-forward for human beings. I’ll have to think of an *ad hoc* syntax for things like `.innerHTML` that cannot be expressed as attributes any more.
Please remove this comment. :)
ContextTe[s]t (sp)
Shouldn't we just update the existing one instead of opening a new instance.
Do we *really* need a new maven project for this?

Why not take the query module and add a test dependency on lucene directory (or viceversa), and add a package for integration tests?
Oh yes, you're right. I thought you were speaking about the impl, not
the interface.

Sent from my mobile phone

On 20 Mar 2012, at 18:43, "Galder Zamarreño"
<reply@reply.github.com>
wrote:

>> @@ -48,7 +49,8 @@
>>    /**
>>     * Flushes a cache (using unicast or multicast) for a set of keys
>>     */
>> -    public NotifyingNotifiableFuture<Object> flushCache(Collection<Object> keys,
>> -          Object retval, Address origin);
>> +    public NotifyingNotifiableFuture<Object> flushCache(Collection<Object> keys, Object retval, Address origin);
>
> From the JLS: "It is permitted, but strongly discouraged as a matter of style, to redundantly specify the public modifier for interface methods." :)
>
> Interface methods are already public, without the need of the modifier.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/infinispan/infinispan/pull/1003/files#r581281
The transaction may have been moved to new nodes via state transfer. I don't think it's safe to invoke the rollback command only on the nodes where prepare was executed.
This is indeed not a proper fix, as it is breaking lots of stuff in the testsuite. But I'm still waiting for a reproducable testcase for the issue.
Style: no parentheses necessary for this cast
There is a python2.5 still here. Not sure what should replace it though...
This test is failing here for me now - need a switch_to_frame() in the `uninstall` method too I suppose.
<3 the inserting from a dict
No need to require these, because they are already required by Format/GeoRSS.js.
If you are complaining about the fact that "1" is shorter than
"warning", that does not matter to the logs. It's just the interface of
log() that is affected by this. The log library can write whatever it
wants to disk.
We are screening out *bad* values -- right?  A useful function, generally, would be a wrapper that checked for `nil` or `-9999` and its type variants (floats, ints, etc.).  Maybe something like `valid-data?` which is easier to follow and more generally useful.
efficient by what metric?  it's very fast ...
Removed medium
This doesn't make sense. The data-widget property tells the widget loader which widget to load. The widget declaration will look like this:

```
<div data-widget='footer' />
```
You're right, my hand slipped on the reformat key. I'll push a revert of the whitespace changes.
K mému zděšení mi teď timer hází kernel panic - unhandled exception. Tak to zrovna zkoumám.
any reason this isn't just "categories" ?
Hmm, ok, well we'll have to make it false to begin with as per Matt's specification anyways so lets explicitly set it.
There's a python file in src/txn, src/txn/interleave.py.   Is it still useful?
That's what I mean. It's up to you to decide which one to use (preferably some that are close enough to the current code).
Are you sure ``$domain`` has the correct type as "integer|string" is not a sub-type of "string"?
Is it possible to define the element type for the array of parameter ``$config`` (using ``array<SomeType>``, or ``SomeType[]``)?
Are you sure that the method "children()" exists?
see above comment about same change
Ist jetzt Threadsafe
`#define`
Ok, I will add that section now
This pretty much defeats the point of having these in memcache
@topolik When the LDAP password policy is enforced/enabled, don't we also want to make sure auth.pipeline.pre and auth.pipeline.post property values, if specified, are set to LDAP compatible authenticators (e.g. auth.pipeline.pre=com.liferay.portal.security.auth.LDAPAuth)?
Here's where I set the module.exports or window namespace
JCryptPasswordSimple shouldn't be instantiated here.  You need to instantiate it in the constructor or at least create a way to inject the dependency.
pickedup => picked up?
it does, but I still have to change it anyway.
we need to change that a little bit.
Because right now we have tree very similar things.:

    service - url for "website" from where user came
    destination - url for "website" where user should have possibility to logs in 
    continue_url - url for "website" where user should have possibility to be redirected.

From which:
- continue_url is not used anywhere because even if you will render login page instead of missing logout, there will be no any information with it.
- destination in form which you provide is not used anywhere because there is no any handle for that in the form. 

So I doubt that we need all of them. I think that we should use only service url which can be used as it is right now, plus possibility to do not redirect user after logout but just render login page with this service_url

For example:

    /logout?service=http://service1&r=f <- this one will render login page (f= false, t= true, r=redirect)
    /logout?service=http://service2&r=t  <- this one will redirect to service

and we could add settings for it, like:

    redirect_to_service_after_logout: false

so everybody will have possibility to set default behavior and in case if it will be needed, we can force different behavior by passing param into url `&r=t` or `&r=f` 

What do you think about that?
pls drop the empty line
I am looking in section: Querying.
OK. But this we will have to REALLY improve. Now you read ALL users from database into Python just to know which users not to process. ;-)

This is why databases support queries. So that you can limit what is transferred between database and Python to only what you are interested in at the end.

So please create a MongoEngine query which will return only users which have not content and which are was more than the timeout inactive. Then run over them `is_anonymous` and delete them.
I tried but something didn't work so i decided to write class based views
Did not suspect that this would work :)

Fixed.
Hm, OK. Grr, why they use lists ...
No dot at the end for simple comments.
Also please reset save_sources to false
Sweet to get some feedback from outside the paid dev team!  Thanks for the eyes.

Ben

On Thu, Feb 23, 2012 at 2:09 PM, Will Kahn-Greene
<reply@reply.github.com>
wrote:
>> +        cursor.execute("SELECT path FROM metadata_status")
>> +        self.paths_in_metadata_table = set(row[0] for row in cursor)
>> +
>> +        # get info about each item on the device
>> +        self.device_items = []
>> +        for file_type in (u'audio', u'video', u'other'):
>> +            if file_type not in json_db:
>> +                continue
>> +            for path, data in json_db[file_type].iteritems():
>> +                if path not in self.paths_in_metadata_table:
>> +                    self.device_items.append((file_type, path, data))
>> +
>> +        # track the next id that we should create in the database
>> +        self.id_counter = itertools.count(databaseupgrade.get_next_id(cursor))
>> +
>> +    def invoke(self):
>
> You could put the invoke() code in __call__ and thus make the class instance callable.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pculture/miro/pull/215/files#r481780
Endlich ist mal dieser blöde Test weg :P
That is my virtualenv name



On 10/26/2012 08:47 AM, emperorcezar wrote:
>
> In .gitignore:
>
> > \ No newline at end of file
> > +.env
> > +chenv/
>
> What is chenv directory for?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/chicagopython/chipy.org/pull/6/files#r1952939>.
>
s/cloudmineb/cloudmine/
If this is never used outside the ``Dendrogram`` class, should we make it private? (``_add_footprint``)
Of the whole commit, this is what I would take... the rest seems really an overwhelming effort in the long term
It knows from the directory $(CURDIR)/debian/packetfence-remote-snort-sensor
even better, will get rid of it.
Move to `WorldView`.
Earth surface density..done:)
The :A doesn't really matter actually, it was more for the consistency.
If you're in the root folder of your git repo, ``git rev-parse --git-dir`` return ``.git``, if you're in another folder you'll get the absolute path. Even if the relative path is used, it should still work.
Actually, line 300 already took care of it (line 304).
Using url call brings no advantage over tuples when you include urls. Only advantage of url call is, that you can name urls.
But, I can change it, if you want.
Ack.  Thanks for double checking.
Actually not.  You don't need to specify pix even if the draw method were fft.  If it's not there, it automatically uses a square pixel with the same pixel_scale as the image.  I added a comment to this effect to remind people of this.
Done.  I did this in terms of a configuration optional parameter (not sure if this is what you had in mind, but it's kind of the only way to pass in an optional kwarg here).  So the documentation will go on the Config Documentation page.
Ditto comment above really...
Agreed. If we change it later, would we have an additional config key for the quarantine group (and create a new one if not set) - so additive. Or would we use just a single config key for QUARANTINE_GROUP: if set then quarantine, if not then don't quarantine - i.e. not backwards compatible?

I'll deal with that in a subsequent pull request so we can get this available for the customer sooner...
Why the switch back to hash rocket?  To run on 1.8?  I see you removed a Hash#assoc() which is also 1.9 only.  
Why is month float and semimonth an integer?
"actual prefix value" is odd, how about "a prefix, such as a domain name"
I'm not sure about this `:hover` pseudo-class. Have you tested in old IE?
the ``@author`` annotation should be put in the phpdoc of the class. and you should add the license header
Didn't we decide elsewhere to avoid using models and use DB Active Record and DBForge only?
There is no need to support ruby 1.8 for any new codebases, it is nearly eol.  Our current target should be ~1.9.3-p194, so you can count on useful things like hash behavior in newer ruby.  If someone is determined to run this on a crufty ruby, they can add one of the gems which backports much of tne 1.9.x behavior to 1.8.x
Mmh, just saw the datepickers need an extra class, so we definitely need a way to add the calendar to fields that don't have it through `calender_for` until we can remove it.
Не очень корректно использовать это исключение. Правильнее сделать своё.
Вместо этого можно использовать System.console().readLine()
Что это такое? И не по code conventions.
Exactly the same as the code for 'date'. Except for variable names.
Why do we have to remove this? What's the impact of having this here?
Let's not tie to a single app. Drop mock_app as an argument here and include it as an argument to the various methods.
Suggestion: provide a docstring for this method
This needs to be inside a case statement for the platform_family.
Maybe make this if statement a little more readable (eg extra vars)
this function should be moved in linear_model.base
why is 4, 33 and 35 the good answer?
и пример на без getUrl, но с методами (вроде есть только который без методов)
Instead of `file_put_contents()` you can also use `io.FileUtil::setContent(new File($reportfile), $this->processor->output())`.

Btw. you're initializing the variable `$reportfile` two times and the variable `$now` seems to be unused in this function.
dpMM? What are d and p? I can guess dots per mm, but the value passed seems more like pixelsPerMM
There's a lot of unnecessary line wraps introduced. Make sure your IDE is configured to use the proper line width allowance.
Pour obtenir directement un jeton d'accés, utiliser les informations
Source provides easy and clean read/write access to the underlying `localStorage` string. If you want I could split this into `setSource`/`getSource` or another name that you prefer?
I think these should be standard to most if not all extensions - I find it annoying if I need to associate some data with an object and it doesn't have any alterable values.
could you introduce the term `boCellIds` somewhere ?
>This may requires some more thought. hthor became unresponsive when allocators were added for each time round an iteration loop - I suspect thor already reuses them in this situation.

Yes, it will at most create them once per activity.
So it's probably not too bad, there shouldn't be many/any repeated calls within a query, creating an allocator for the same activityId.

Allocators will be destroyed at end of query (graph sent to Thor) (not end of subgraph), so there could be quite a lot of them.

This is as it was btw, pre-roxiemm. 

>You also want to consider using packed allocators in thor - though whether to use it may depend on the number and likely lifetime of the rows.

Something I need to look into.
Good question. Don't have a good answer to that. It seemed arbitrarily "large enough" to me. But apparently not.
That's wrong. It's supposed to be `/static/`. It's corrected in https://github.com/mozilla/socorro-crashstats/pull/82 just to confuse matters.
How about something like:

`raise http.HttpResponseServerError("Middleware responded with a %s error" % e)`
Again, "recommendations" is preferred over "standards".
If we use the Django token generator code correctly, we should not need this model any longer. It is overhead we don't need to continue to accrue.
You are right. It slipped in because you just recently added it and I forked before that point in time.
Where is the struct declared in the header file. If we go the typedef route, we can declare a typedef of a forward declared struct, like:

typedef struct tempodb_config * tempodb_config;

Put this line near the bulk struct definition. This way it makes it clear that there is an external type being defined.
ok, may you add a comment? you have to perfectly know how `off` works in order to understand why removeListener is special.
Isn't it `escape` instead of `fixedEncodeURIComponent` ?
Btw, is this code from Node, like unit test? If yes, we should put licence header too.
could you please linebreak after as so it's easier to read in editor.
please include this only in d8 tests and exclude form d3 tests
Not sure where cleaned_data comes from.
read-your-writes consistency.
You should use ```Rex::Socket.source_address``` with no argument in case 50.50.50.50 ever gets allocated and we need to change to a different reserved block.
It would be nicer if this was randomized by using rand_text_alpha()
Oh, yeah, I just meant that we could show how to derive them in a docstring.
"consists of" not "consists from"
OK, having looked at load_prompt_from_id, where @challenge_signup is set to @prompt.challenge_signup, because this before_filter now comes after that one, this will never get invoked except in a bad situation where the prompt has gotten detached from its challenge_signup to begin with. I think you want to remove this before_filter entirely and just stick with the signup_owner_only check.
We want to remobe this since "templates/" is already on the appropriate directory.
Srsly "T"?
I agree and see this function going away in the future if we have a good promise system in place.  I originally had it as a private function only used in DynamicBillboardVisualizer, but Cozzi thought it might be useful stand-alone.  Just to be clear, was there anything you wanted me to do here, or were you just waxing philosophically on promises?
Cleaner.  Thanks.
This code isn't on the critical path for performance.  Readability is more important here, and the word "toggle" isn't the correct term for what's happening.  Also I've [seen](http://scientificninja.com/blog/boolean-function-parameters) several [articles](http://blogs.msdn.com/b/oldnewthing/archive/2006/08/28/728349.aspx) discouraging the [use](http://silkandspinach.net/2004/07/15/avoid-boolean-parameters/) of boolean function parameters in this form.  Perhaps I should follow their advice and make two separate functions for highlighting and un-highlighting.
This comes from the environment ($TEMP_DIR, $TEMP or $TMP), which I presume is the best location for it as it works globally for all temporary files created by the tempfile module. 

I totally forgot about cleaning up, though. I'll add a context manager and a cleanup method in a minute.
default color of text is black, but default value of switch.color is undefined on Android
If fontValue is nil, we should revert back to the default font. In native terms, that is [UIFont systemFontOfSize:0]
Missing whitespace in front of last paren
Undefined var on top.
Use asyncTest and remove the stop() line
I owe you content for this section.
The `poll_event` parameter is not `const` in the C prototype so please don't make it `const` here.
Yes, I am. Version 5.1.1 for what's it worth. Did you move on to something else?
они ниже, в теле функции) Пользователь вконтактовский идентифицируется либо только по username, либо только по vk_id из профайла, в зависимости от настройки в settings.py. При этом оба поля в базе есть и заполнены как-то независимо от настройки.
Can you explain this SQL to me? I have no doubt it works, but have never see anything like it before.
Have you added these lines?  I ask because I think they were removed recently.
the = is not aligned with the rest of the code. My recommendation is to shift the rest equal signs right wards to align with this one.
Have you done the performance analysis? If not, we have no idea which one is faster, or even if it make an appreciable difference.

By not using JSON, we impose a *semantic* overhead on code maintainers and people browsing the code.
if we're requiring it here, what happens if the app doesn't have the gem?
Should this be optional behavior?
This has been regressed despite you trying to add another commit.
Should be `https` instead of `http` perhaps?
I think it would be enough to use a hardcoded name as long as it is unlikely to be used in a project by the developers. Other bundles define their routes as well with fixed names (e.g. FOSUser ...).
Not sure if it's the most correct way, but at least is how we do it everywhere else
Видимо, нет. Потому что не работает нихера.
shouldn't the write skew take place during prepare time? so shouldn't this be renamed writeSkewInPrepareTime?
This should be changepassword.aspx.
I think this will cause problems when using PHP as a template language. In this case you would have to write

```
<div>
<?php if (foo)
{
  ?>
  hello world!
  <?php
}
?>
</div>
```

which I think looks ridiculous. Having the braces in the same line makes it much more readable

```
<div>
<?php if (foo) { ?>
  hello world!
<?php } ?>
</div>
```

Plus, this syncs up with the additional syntax PHP provides:

```
<div>
<?php if (foo): ?>
  hello world!
<?php endif ?>
</div>
```
I know, which is why it confused the hell out of me! ;) In any case, if this works, then there's no problem.
I think this should be 'entry_category' instead.
Gary, the XOR note was in the closed PR #624 ;-)
Here I said we did the 'channel' checks in the specific parsers before in my last PR #592.
And also I pointed to issue about 'reply-channel' and race condition in the `AbstractConsumerEndpointParser`
Am I clear?
My comment is hidden in this PR above
Not sure why it has to be class loader aware
**waitForTasksToCompleteOnShutdown ** is a property of TS, and if we reuqire a dedicated one there is no any reason to allow manipulate with his properties in the end components. Or in other words: it will be very bad, as we saw that, when I was showing Delayer's SmartLifecycle.
About documentation: don't worry - I'll try to explain all my changes in the Manual & Migration Guide
IMHO: it should be just **WARN** as it done in many other Spring places, e.g. in MVC...
"uri variables will be ignored" and that's all
I don't feel comfortable removing content that was already reviewed.
Any reason for this syntax rather than Post::get()->filter() which is a) shorter and b) a little bit clearer as to the intent.
Yeah, it's unused now - I'll remove it. One of the benefits of dropping testing means we can mirror how the yum repo code works (2 repos, stable or nightly/testing)
`It's in fact better to see it a member of` -> `It's in fact better to see it as a member of`
`encoding` pragma is for encodings other than `UTF-*`. For `UTF-8` use [`utf8`](http://perldoc.perl.org/utf8.html) pragma.
committed something, but we have to check if relative paths in the css files work out allright.
could you remove all these 'auto reorder imports' patches?
What are these parameters and how are they used?  Are they just query parameters?  I was under the impression we agreed on a `find and replace` style strategy so that tokens in the URI would be replaced client-side for flexibility.  @shunter ?
use new isDeleted flag
you got it

Dictated but not read,
Ed Cranford


Craig wrote:

> @@ -0,0 +1,15 @@
> +#!/bin/bash
> +
> +ROOT_DIRECTORY=/opt/stack/
> +
> +for project in glance horizon keystone nova noVNC python-keystoneclient python-novaclient python-proboscis python-quantumclient python-reddwarfclient

oh where projects-list is a file with all the projects you wanted to update?

---
Reply to this email directly or view it on GitHub:
https://github.com/hub-cap/reddwarf_lite-integration/pull/40/files#r653299
This checks nics in the actual DB, so it could just use:
<pre><code>
if WHITEBOX:
</code></pre>
4 пробела
2 пробела
выбери что-нибудь одно :)
Matchlighter getPercentageCapacity() will not work in SMP server...i use the Container to transfer  getPercentageCapacity() Server -> getCapacity() on Client