labofsahil/github-event-dataset-2012 huggingface dataset:
While they are loaded, as they first register a handler to be called when the page load event is being triggered, and resolve the promise returned by their load function when said handler is called.

Switching the document first could result in the agents registering their page load handlers AFTER the document has been loaded (never to be called then).
Switching the document afterwards is not possible, as the agents (three of them, anyway) do not finish loading (i.e. do not resolve their promises) until the page load event is triggered (no document switch -> no page load event).
Yes, the intention of this is to indicate whether the attribute already has an equal sign or not so that the caller can decide whether to append the ="" or not.
Nice! This also fixes issue #1873.
Done - the replace code was reverted to simply populate the search field, and this function was refactored into `doSearch()`
Ideally we would leverage KeyMap.translateKeyboardEvent() here instead of having our own mapping/constants.  If that's tricky to do at this point, could you add a TODO?
I think we do this same check in a couple of places (`(window.localStorage.getItem("locale") || brackets.app.language)`). Should we factor this into a convenience method, e.g. `brackets.getLocale()`?
TODO comment format: ``TODO (issue #2319): ...``
I reviewed myself & I was wrong
true - no need for the `bind` - I will remove it now
If "Fill configuration" is a label (shown to the user) a call to I18N.tr should be node
We've been working with Core Git and MSysGit to add some consistency to Unicode handling in Git. The result is as follows: UTF-8 *everywhere*. **All the time**. We do not consider any output encodings anymore because this would be a disaster under Win32. Anybody using something that is not UTF-8 is wrong and stupid and ugly and should feel bad about it.

The only exception to this are Commit Messages, which have support for an `encoding` line to define a custom encoding. We already handle those properly in 1.9.x
Why not class based views?
Just leave it `auth`.
Yeah, we don't store name or username like Popcorn Gallery does, so I'm just using email for all 3 attributes instead
nvm I see this is in an interal butter folder or something
no space after `setTimeout(`
Do we need to normalize the range of the numbers we accept for width/top/left?  What if they are negative?  0?  huge?
I do need jQuery to be loaded, but I guess the code that actually calls `editorHelper` is wrapped in `DOMContentLoaded` anyway, so this is unnecessary
Use __NAME_PREFIX here.
Not more recent versions of IE?
is it just me or is `il` not declared anywhere?
`ideas in motion.You can submit`

If you could add a space, I'll get this merged and deployed.
No, no they are not. And by the time this is merged they no longer will be!
I think this is better as:
```
Then I should see the validation error "Database selection only accepts one option"
```
No way should the string "trustedads" occur in Aji.
You should remove commented out lines of live code. Ditto for the others below.
Stash the fully qualified function name here so that it's not necessary to do string gymnastics later on.
just in case siesta is now staged 1.1 and snap bumped to 1.2-SNAPSHOT
echo "copying $NGX_OBJS${ngx_dirsep}${dso_binout}${ngx_soext} to $NGX_DSO_PREFIX"
用现在进行时？
Let's add a comment here describing what it does
Apparently not. Handed by Rails black magic.
The date is defined as the first month you controlled your domain name. (It SHOULD be the "first full month" to provide a better guarantee. Sun violated this SHOULD.) This is to ensure uniqueness, as domain names can change ownership. See http://en.wikipedia.org/wiki/ISCSI#Addressing as well as the RFCs that references.

But remember, the uniqueness of the date & domain is about establishing a unique naming *system*. So as long as we're using the same algorithm, it's fine to use the sun.com one from ZFS. From what I've seen online, Sun has two: iqn1986-03.com.sun:01:* and 1986-03.com.sun:02:*. The latter seems to be what they use for targets. I haven't yet figured out where they're getting the GUIDs that they use after the :02:.

Alternatively, we could use some other, fixed, domain like "iqn.2010-10.org.zfsonlinux:01:*" and define our own algorithm. Datasets seem to have a unique ID (as seen in zdb). So if we combine the pool GUID and the dataset ID, we'd have a unique identifier. For example: iqn.2001-04.org.zfsonlinux:01:13104831232857787788:43214
A pointer to a delegate should work fine, what went wrong?
Create() has "name for the architecture (required)", Update() has "architecture name". Can you be consistent? Ideally put these strings to constants.py please and use constants here.
It appears that view name is not unique (in the org or for a given view definition).  As a result, should the request be using label?
Can we get rid of this completly and use katello-repos.rpm?
why 800px? How did you derive this number?
Good catch...I think it's kinda lame that inbound doesn't include some information.  I'll look into making this better.
Actually the easiest.  You dont need to bother informing the user, as they did it.  The apply and ok buttons dont need to change, they just need to set the settings as they do now?
Extra space after `(` and before `)`. If block should always have braces `{}` even for single lines.
I guess not, I can remove them also.
Please don't add trailing whitespaces and don't break the indentation of the phpdoc
Tried to make some of these stubs... but the stub_model command wasn't available inside this spec for some reason...
Can do - could automatically translate RolesAtBase to roles_at_base?

As far as ordering goes, I'm not sure? I don't think they do need ordering, might be able to make them into a dict of role name?
true that. hm, but this /is/ a serious chicken-and-egg problem. basically i think we need to change our metadata structure to put the ``rev`` info into the (child) types rather than the collections. otherwise it is impossible to know which collection to look at. this however needs https://github.com/bergie/create/issues/122 to be fixed. not sure how to proceed here. @bergie, any E.T.A on this bugfix?

of course we could also do #24 before finishing this PR, but as bergie said he wants to fix the collection template thing, would be nice to not have to do all that work right now.
```
foreach (var remote in remotes)
    remote.EnsureTfsAuthenticated();

```
Instead.
I'm *pretty* sure this is right, tried to match the range that was copied to.  Could invalidate per each line I guess as well...

-[Unknown]
In fact, since the user can trivially add a directory to the front of the path, I'd leave this parameter out altogether.
We should be able to fix this issue.

One way to go about it would be to set `posix=False` in `parse_options` and then modify `globlist`, to be something like:
```
for a in args:
    out.extend(glob.glob(a) or [unquote_ends(a)])
```
changed to `(start | end)_pos` squashed and force pushed.
running self.shell.safe_execfile_ipy in a try block might not be good... but i wanted to match how the file var was cleaned up in the older code
I think you can reuse some code here.. ex. GroupNamingConvention
2 points here:
1- defaultProperties() is supposed to set default properties and not to check-and-set
2- do not get properties from system directly e.g. this property could be overriden without setting it in the sysprops
We should probably move this to global CSS file. I'm not sure if PlayerSkinMvcpf.css always loaded.
There hasn't been much in the way of porting the object query system over yet so for now rewrites will rely on the simpler traverse. We can look at expanding this in the future if it really becomes worth while. All the RegEx work was likely slowing things down for no good reason. It is worth noting that at present all the rewrites will do a complete tree traversal, this could be optimized.
Yes, that seems like a good idea. I'll fix :)
sorry, I should have said newline :-)
you mean I need to drop the product_list bit right?
The code is actually very simple so I'm all for it.
I like this so much better.
I initially had that but decided against it to reduce a little duplication, and also because this code is basically stop-gap and not going to be permanent.
I believe you're incorrect here:

~~~ 
$ bash -c '(( 0 )) && echo true'
$ bash -c '(( 1 )) && echo true'
true
~~~

1 for true 0 for false, since you're doing numeric tests and not using it as return values.

We've also got to think about backwards compatibility with existing configs...
Align `=`
`$count = substr_count($command, '"');` does the same, but faster
Yep, an new line is missing =)
I cannot typehint here, because when running the code in prod, the $col will be an instance of Propel\Runtime\Map\ColumnMap

but in test, the $col will be an instance of Symfony\Bridge\Propel1\Tests\Fixtures\ColumnMap
extra space after ``list``
This should be moved **after** the skip check. Otherwise, it will fail when the intl extension is not loaded (the stub implementation does not allow changing the locale).
either ``that`` should be removed, or the end of the sentence is missing
@manoelcampos could you please take a look first at http://symfony.com/doc/current/contributing/index.html
@stof he used online editor in github.
Because I don't remember this was even possible in 2.0 ... [the documentation](http://symfony.com/doc/2.0/reference/forms/types/collection.html#prototype) says the prototype option is supposed to be a boolean, so the `'prototype' => 'proto'`, part looks very dubious to me. And the `$$name$$` default does not look it is configurable at all in [the code in 2.0](https://github.com/symfony/symfony/blob/2.0/src/Symfony/Component/Form/Extension/Core/Type/CollectionType.php#L28)... what do you think?
@Tobion This code was copied from the previous test in this class (the test for the pattern), only that there the pattern was passed to the `pattern` rule, and here I use it for the hostname_pattern.  And I think is right as it is, because that part won't be escaped.
We can use a `UserProvider`, be we will have to add another `User` class which will support `setPassword()`. I think it's useless to add an overhead here.

`UserProvider::loadUserByUsername` is not possible due to ldap limitation. Thats why, IMHO it's easier to no use this method.
This seems like it will only add complexity. We know the database gets cleaned between every run. Why make one line of code three?
We've used `include` a lot for string-matching, but maybe RegEx is more appropriate.
OK, I have delated it in the latest commit.
为嘛是3？
Too many non-optional arguments!  I hope we can make some of these optional (an example action which has optional args is --capture).  As argument list, I suggest:

--histogram:cumulative=0 size channel

So:
  * cumulative is optional, and defaults to off.
  * size is binsXheight, eg 256x100 will give you 256 bins with a height of 100 pixels.
  * channel is as before
  * Not sure what to do about output format and stuff yet.
This change has been required by the new jsonified responses
@twinturbo it would be nice to add support for passing the key into sortFunction.  The sortFunction is somewhat useless if you have multiple sortProperties that are the same data type.

```javascript
sortFunction(get(item1, propertyName), get(item2, propertyName),propertyName);
```
You forgot to import ``array`` from numpy. Additionally, you forgot ``import warnings``.
Any chance of div by zero here?
The client is never closed, which is one of the things the @ServerSetup approach is meant to prevent.
IMO, work->hash should be expected to hold the byte-for-byte SHA256 result.
@TomCoder what happened to my suggestion of re-writing this as?

    import os.path.splitext
    fileName, fileExtension = os.path.splitext('/path/to/somefile.ext')
    return fileName + suffix + '.' +  fileExtension
`f` stands for 'formatter' here. I’ve used a single-char variable to be consistent with the line above…
it's preferable to push { name: bubbles[i].dataset.name, address: bubbles[i].dataset.address } here rather than just the address as a string.  I was just being very lazy about this function previously.
If there is no condition and we use `homescreen.launch()` in some part of our code we are adding a new frame for homescreen, so it will not be working properly. Not it's explained in code and the condition has been improved.
This in `init()` might not be needed.
Yes, should be next PR of @albertopq
Correct, it is a little slower, and this is actually a bad place to use forEach because of the amount of data processed here.
why not just <code>font-size: 2.26rem</code>?
Right, makes sense, ok :)
aria-level is not allowed on header, according to the w3c validator.

Please put your HTML file in the validator, there are other errors (mostly in the `<head>`) that should be corrected.
If you remove the left/right padding of the entire keyboard then it would fit the 11 key-per-row layouts (not only Chinese Zhuyin, but also Russian)
No, because we didn't any type conversion in `FakeRedis`, `None` remained `None` when we stored it in `FakeRedis` -- real Redis stores it as `"None"` when we try and store `None` -- which is what it does now.
I thought about it, but one thing to another, we'd really want to wrap every single I/O calls. Let's keep it targeted to observed conditions for now. Hopefully, at some point we'll have a way to get the site offline during updates.
What about:

```ruby
def each_sexp(&block)
  grep(Sexp).each(&block)
end
```
It would be nice to know why the classloader is needed. Between the JDT model and the scala AST, a lot of information are already available.
Since this is a fix dealing with mutable init value, do you want to add a test that it is fixed now?
a) Doesn't adding an array of types break the ```transformForRead```signature defined in ```Source```  ?

b) I do get your point that those conversions have unforeseeable side effects, but:
b.1) Why not use the mechanism offered by 
```def map[A,T](fs : (Fields,Fields))(fn : A => T)
                (implicit conv : TupleConverter[A], setter : TupleSetter[T]) : Pipe = .... ```
e.g. ```map[Array[Byte], Int]``` and such and friends.
b.2) We can always hide the implicits and use them only in context of HBaseSource

c) If we are going on something in the spirit of (a) then my only idea is to use an ```Each``` with ```Identity``` and tie it with some sort of mapping of fields and their target type, although it's a full step.  Tell me if you've got something concrete or have an example for such an override. I am really new to the inners (btw, beautiful code) of Scalding and my head is still a bit dizzy going through the code and the very high level of Scala (far above my current expertise). Another btw, any idea on how I could debug the thing (there is no 'local mode' HBase)
Replace ii with a more descriptive name - rows?
It was being started in the next round of when the apps are being loaded. Removing "next" as suggested.
Should this be "2011-2012"
Recount function really should be in the helper as it isn't done against $this.

PS: PHP raises strict warning if non-static function is used as static..
To be consistent with the rest of this app, please use a JSON-encoded response, e.g., { status: "OK", archive_name: "gs_archive_name" }
It would be nice to keep changes in this commit focussed one one particular task, as opposed to grouping arbitrary changes into it.
onAttached
1) was implemented for now, test passes.
We also need some sort of versioning for our plugins. So we can have a proper/better migration guide or change log.
Maybe we just keep it as is because this code-base is still pre 1.0.0 and we can lock down the API after.
Fixed microtime & var_dump leftovers
@davispuh its all ok with installation instructions for master :) 
We use gitolite v3 as default for master. And pu branch is gitolite 2.
Ok, thanks fot the precision.
Capturing error and doing nothing with it is no good, either don't catch or do something with it. If this error is really useless then use `finally { throw e }` instead.
It looks like we no longer need this workaround could you please remove in favor of `module.exports = tabs`
I would suggest to remove the `if/else` branch, and just follow the styles we have in other unsupported tests, so start to define the all tests, and at the end, check the condition and if it's true, overwrite the `exports` with `testUnsupported`. That will remove for all the code below one level of indentation.

See for instance [test-selection](https://github.com/mozilla/addon-sdk/blob/master/test/test-selection.js#L702) (but all unsupported test follows the same pattern).
good to be generic, because at the beginning we'll probably generate warnings / log errors only at runtime (`cfx run`), and generate them at building time (both `cfx run` and `cfx xpi`) only later.
proxyidp should no longer be an assertion type, this should be chained certs instead.
This isn't directly related to the changes in your branch, but now that datasource has the transaction capabilities, I think everything that `load_test_data` does should be inside a transaction, so it isn't possible to end up with a partially-loaded json blob. We could make this a separate story/branch, I guess.
Given that all the log file handling code should be in the logger and that those constants will never change it's fine to leave them at the current spot.
should be `a Schema`
As long as Hiera also follows semver I don't see a reason  '~> 1.0' wouldn't work here. As long as Hiera doesn't break api or break backward compatibility all of 1.x should work with 3.0.0.
If you switch the first two conditions you can have only one if/else

```
if [ -e ".ruby-version" ]; then
  RBENV_VERSION_FILE=".ruby-version"
else
  RBENV_VERSION_FILE=".rbenv-version"
fi
This is fixed up in a later story/commit (schema is factored out)
Tab characters on lines 95, 96, 97.
$type is not used in the loop Why save the key in a var?
Hmm, I've been following this style guide:
http://framework.zend.com/wiki/display/ZFDEV2/Coding+Standards#CodingStandards-If%2FElse%2FElseif

Is this not correct?
Previously I wrote this, is more universal and efficent:

```php
if (DIRECTORY_SEPARATOR !== '/' {
    return str_replace(DIRECTORY_SEPARATOR, '/', dirname($baseUrl));
} else {
    return dirname($baseUrl);
}
```

But finally I used the other solution for consistency with other similar cases in the code.

@Thinkscape If you want, open a PR to fix this cases along the repository.
well, even if the data are not migrated, the schema still need to be migrated.
true, fixed with : 

if ( isset ( $_GET["errmsg"] ) && $_GET["errmsg"] != "" ) {
```js
return soma.Event.call( this, type, {
    todoTitle: todoTitle,
    todoId:todoId,
    toggleAll:toggleAll
});
```

Spacing, linebreak
i dont understand why you have both a model set and an inline schema. It should be either the one or the other.
Some of this should probably be put into a method on `User`.
You should avoid one letter variables whenever you can. They aren't descriptive and therefore they decrease readability and comprehension by other devs. They also make it harder to refactor later.
VersionInfo contains ContentInfo, so might make sense to remove passing of ContentInfo, but depends on use.
Can you try building Trilinos with an ictce toolkit with this branch (from scratch, without having ictce+deps modules already there)?

That's a very good testcase, because it touches a lot of easyblocks that were changed by this.

I don't expect it to fail, just want to make sure.
Otherwise, you need to break, I think.
I thought it's good for illustration, especially with the  session context comment
merging, we would need to add the 'show-old-login' switch as well
This stuff doesn't need `gettext` then?
Why not just get_or_create()?

apikey, created = ApiKey.objects.get_or_create(user=self.user)
return apikey.key
have a look at the user method :P
But should we really remove it? Popcorn extends these off the media object so users may expect them to be there. Shouldn't we do everything that the media element does? I say leave it in.
can we use element or web_element here?
If I'm reading it right, and the following is the condition block that you need:

<pre><code>&lt;% if commentable.is_a?(Comment) %>
&lt;% else %>
  &lt;% if current_user.is_author_of?(commentable) && commentable.anonymous? %>
    &lt;p class="notice">While this work is anonymous, comments you post will also be listed anonymously.&lt;/p>
  &lt;% end %>
&lt;% end %></code></pre>

this can be expressed more succinctly as:

<pre><code>&lt;% if !commentable.is_a?(Comment) && current_user.is_author_of?(commentable) && commentable.anonymous? %>
  &lt;p class="notice">While this work is anonymous, comments you post will also be listed anonymously.&lt;/p>
&lt;% end %></code></pre>

Rails / ruby quits the evaluation of a logical expression as soon as it can determine its value, so if commentable is a Comment, it won't cause any errors on subsequent tests, because the tests won't be evaluated. Yay ruby :D

Are we sure commentable can't be a Chapter or something other than a Work or a Comment, at that point? Would being a Chapter cause any trouble?
I have to admit that I have no idea as I literally just pasted in these actions from the admin/skins_controller file and fiddled with the action names.
>「正しいけどネットワークには取りに行きたくない状態」
完全にこの意味でした・・・
valid_twitter_nameだったらtry_to_get_valid_responseする感じです。
validateを外して、try_to_get_valid_responseの中で呼んだほうが良いでしょうか
```
def try_to_get_valid_response
  return nil unless valid_twitter_name?
  self.hogehoge
rescue
...
end
```
I copied the code from the :my_hosts lambda function, which has:

app/models/host.rb: line 73
    domain_conditions    = sanitize_sql_for_conditions([" (hosts.domain_id in (?))",dms = (user.domains).map(&:id)])

There I assumed dms was some kind of temporary variable, maybe meaning "domains"? So I did something similar. Only now do I realise it's not re-used like dms is, so I guess I can update that.
mocha 0.12.7 looks old, perhaps try an update to 0.13.x?
know your typography: only a hair space around em-dash. And while I cannot type that, even Firefox displays it about right automatically.
I'm not too sure. I can't actually remember what I used as a base for this problem, but I definitely copied that part from somewhere in the codebase or wiki...

I'm fine with switching it to whatever's most consistent with the other exercises.
Since this method call reads the stream, the next 'getContentLength(InputStream)' also has to read the stream (from the database?). Seems like it'd be more useful to measure the size of the stream by using the [SizeMeasuringInputStream](https://github.com/ModeShape/modeshape/blob/master/modeshape-common/src/main/java/org/modeshape/common/util/SizeMeasuringInputStream.java) utility class. This calculates the size as the stream is being used.
You should be able to replace this with `inject([]) do |artifacts, piece| ... end` and strip it inside the block. I would try to find a better variable name than `piece` though.
```
[http://www.phpbb.com/kb/article/how-to-create-a-style-basics/](http://www.phpbb.com/kb/article/how-to-create-a-style-basics/)
```

Can be replaced with:

```
<http://www.phpbb.com/kb/article/how-to-create-a-style-basics/>
```
Misplaced! I'll fix it
When do cookies not produce errors?
sorry juste figured, this have nothing to do with your commit
Can you remove this change, as it's not relevant, and as commented elsewhere, we'd rather do this with a CSS class?
Should name be part of cachekeys to avoid crash with similar cache blocks? (also for the one bellow)
dual не очень удачное слово
IMO, having a `foo.1.png` and a `foo.png` in the same directory is a edge case as it implies the developer hasn't understood how cache busting works here... This is the same problem in the `.htaccess` version too.
I think, for such edge cases, that the documentation should warn the user about this and the code should be as simple as possible.

But that's my thoughts :)
And I can then simply use TagHelper from _tags_cloud.html.erb? do I have to
include or require the file somewhere?

On Mon, Jun 11, 2012 at 10:06 PM, Jean-Philippe Boily <
reply@reply.github.com
> wrote:

> > +      content_tag(:section, :class => 'widget') do
> > +        content_tag(:header,content_tag(:h3,title)) +
> > +        capture(&block)
> > +      end
> > +    end
> > +
> > +    def tag_url(tag)
> > +      "#{Monologue::Engine.routes.url_helpers.root_path}tags/#{tag.name
> }"
> > +    end
> > +
> > +    def size_for_tag(tag,min,max)
> > +      #logarithmic scaling based on the number of occurrences of each
> tag
> > +      #we have defined 5 sizes in the CSS hence 5
> > +      min<max ? 1 + ((5 -1)*(log_distance_to_min(tag.frequency,
> min))/log_distance_to_min(max, min)).round : 1
> > +    end
> > +
>
> you could do that in tags_helper.rb (you would have to create it)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jipiboily/monologue/pull/83/files#r964564
>
Can we just use the new `override_settings` decorator in Django 1.4?
curl rocks

curl_setopt($ch, CURLOPT_HEADER, TRUE); 
curl_setopt($s,CURLOPT_TIMEOUT,$timeout);
that's exactly why i used my_e...i didn't wanna risk conflicting namespaces
`pr-str` is actually quite expensive, in terms of garbage created, dynamic dispatch overhead, and plain number of ops executed.  You would almost certainly get better performance with `(if (string? x) x (pr-str x))` here, and this seems like a location that would actually benefit from that testing.
what about the ``configuration`` and the ``port``? Missing TODOs?
Maybe you could extract a method ``EventSet => List[Future[Boolean]]``
``threads`` can potentially be accessed by different threads. Since it is declared as a ``var``, it can suffer visibility issues.
Same comment as in https://github.com/scala-ide/scala-ide/pull/220/files#r2084572
Are you sure that your commit fixes this memory leak?

I've made various improvements at https://github.com/john-tornblom/tvheadend/commits/http_stream_testing

Do you think you could give that one a go? It seems to do the trick for me. Oh, and be sure to test it on a stream with mpeg2 video packets, I don't think h264 is affected but the leakage.
We could remove this condition actually, we drop all IP addresses below
this will throw a notice if there is no cookie with this name, which is wong
Зачем после каждой строки идёт пустая строка?
Теоретически бесконечный цикл.
а ведь должно while (mutated_elements_count < elements_to_mutate) {
в итоге на один больше
опечатался наверно..
Brace should be moved to the end of previous line.
Do we need this feature?
Some of these notes could go into their own paragraph, below the summary. (It just makes the "Returns" section a bit long.)

Also, `+interval:` isn't much more precise – `NSTimer`s can be delayed if the run loop stalls.
OK, Jim.

On Wed, Apr 18, 2012 at 11:19 AM, James Ross Hinkey <
reply@reply.github.com
> wrote:

> > @@ -1,6 +1,5 @@
> >  # Installation
> > -
> > -Welcome to the Liferay Quickstart Guide! This document is intended to
> > +This document is intended to [Here is a "change" on this test branch.]
>
> @markhealy This really isn't a good idea. Rephrase it in such and such way
> ...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jhinkey/liferay-docs/pull/5/files#r701678
>
There's a fair amount of discussion about this over on #213 (and code on #221). I'll see if I can't convert the progress reporting to callback delegates, which is *way* more idiomatic.
Possible Typo: Should be

`if 'accepts non-latin chars' do`
robots.txt is always expected to be found in the root of a web site. Thus, it makes sense to create it only if DJANGO_BASE is empty.
Hibernate supports a special "id" parameter which just means the PK. Perhaps we should call the method getByPrimaryKey(Object pk), and verify that this works with both integer PKs (most things) and String PKs (privilege, role).
Is it worth making a citation to the http://en.wikipedia.org/wiki/Project_management_triangle ? I think this is a core concept that we might need to explain further somewhere.
maybe $output? ;)
wrong. It is a 403, not a 401
sure, but there is no need to include ``<?php`` here as you are only showing some PHP code. ``html+php`` is only used for PHP templates in the doc (which indeed mix HTML and PHP)
通过<command>configure</command>命令配置nginx编译环境。
This doesn't seem to pass the same logic tests as before. With BrightCove, the test looked like `mimeType.equals("application/x-media-brightcove")` but now `application/x-media-brightcove` is not checked. This is also the case with the Matterhorn implementation. How does `application/x-media-matterhorn` and `application/x-media-brightcove` get handled with this new setup?
You need to also mark the underlying channel as broken here, as no further frames can be read
I see no test for concurrent registrations.
Don't use the `message` keyword here.  That string is added to the existing docstring, so this results in duplication:
```
In [7]: radon?
Type:       function
Base Class: <type 'function'>
String Form:<function radon at 0x18ecb2f0>
Namespace:  Interactive
File:       /Library/Frameworks/Python.framework/Versions/7.2/lib/python2.7/site-packages/numpy/lib/utils.py
Definition: radon(*args, **kwds)
Docstring:
`radon` is deprecated!
`radon` is deprecated in scipy 0.11, and will be removed in 0.12

    For this functionality, please use the "radon" function in scikits-image.

    

`radon` is deprecated in scipy 0.11, and will be removed in 0.12

    For this functionality, please use the "radon" function in scikits-image.

    
```
И зачем ты на этом уровне объявляешь? В месте присваивания же логичнее.
Hay un espacio de sobra al comienzo acá.
Bad file name.
Bad group
I think I know why you add 100 to the sample number, but this seems like a very risky operation to me..
some functions are not documented
Among the sub-classes to be considered is a PSF that varies over an image.
I'd also like to handle the case where one is fitting the PSFs to individual frames that contain some distortion (e.g. WISE).
You shouldn't call the get... method twice.
Please add data type
In the real code, next_marker is set to None if the number returned is <= limit. 

Either the real mode code should change to return a marker even if its useless, or, if that would be a bug, we should change fake mode and look at adding additional asserts to RDLI since the presence of an unnecessary marker should be caught there.
key change #1
Oh, I forgot that. Thank you very much :)
rename to "activeTopologyUpdates"?
@danberindei I would have imagined each cache instance would have its own TM lookup instance, which would get around the issue of needing it to be concurrently accessed. That sounds a more sensible approach, doesn't it? Might consume more memory, but in general these instances should be pretty lightweight.
I mean, like I bitched about in Campfire, it's just awkward either way. If you think returning `BOOL` is better, I'm fine with that. It just sucks either way.
If you check out the commit history I initially had this exposed as a simple enumeration but then 180d assuming that patch files would be "lightweight". As that is not the case it would make sense to go back to simply exposing enumerating the deltas with a given block.
I have looked through all the params in the source and matplotlibrc file and 'sans-serif' seems to be the only one.
I think it looked a little funny since the conditional was wrapped around the second line. python parsed it fine, but i've lined everything up to look nicer. will see with my next commit/push
Should we be putting the outData on the dataChan even if PrepOutData returns an error?
The only place that had same code was in `Show-AzurePortal` and it's refactored, do you know other places that should be refactored as well?
And does this mean that WMS 1.1.0 + SLD 1.0.0 does not support DescribeLayer?
Addressed in d9170ba
why is setting the cloud file only for centos?
please don't add doc / data here. use git submodule update --init to get in sync
please use (or at least utilize) MulticlassAccuracy::get_confusion_matrix()
Is it merged with the PHP statics? If it is, which one has priority? Can you specifically unset either way through an empty value?
Same as above, no need to use escape_str() here
You could shorten this to record && !!record['enabled'], but this is fine too.
vind m nog niet helemaal goed
Is it possible to define a more specific type for parameter ``$value``; maybe using a union like ``null|Object``, or ``string|array``? Please also check line 255.
remove spaces directly inside parentheses
`{` for function call should be on a line by itself, right after the function declaration
The second and third arguments were there for a reason (better minification for instance). Do we have to drop that for AMD support?
I've used this pattern of coding from XMLEditor :)
thanks it is fixed !
he still should clean up the memory leak (IE). best practice.
Unfinished documentation?
Yeah, sorry. It will convert symbol keys, but not values.
i opened another pull request with working reused code :)
Rewrap:

```
context.update({
    'panels_with_dependencies': {panel.get_name(): panel.dependencies for panel in self.request.user.get_all_panels()},
})
```
I don't understand what are you doing here, why you first manually do queries to the database to get IDs and then again?

Doing queries in a loop is very bad. Queries to the database cost! And the idea of query is that you do one query, exactly the one which gives you data you want, not that you do that in your code, but that you leave to database to do this.

So what is wrong with simply:

```
models.BicikeljStation.objects(location__near=(lat, lng), location__within_box=((lat-BICIKELJ_BOUNDS, lng-BICIKELJ_BOUNDS),(lat+BICIKELJ_BOUNDS,lng+BICIKELJ_BOUNDS)))
```
Format this a bit better. Newlines and such.
Indentation. Don't put end space into translatable string.
There could be some race condition here. For example, user double-clicking the notification. You could happen to decrement twice. It would be better if you would count all notifications. Slower, but more precise.
Bring my synthesizes back. :-)
Давай сделаем отдельные setParams(Params $params) и getParams. В getData проверяем что $this->data не null и тогда формируем и выводим масив из парамсов.
Убрать абстракты, так как ресурс может не импелментировать какие-то из методов
why only angle and remove angleplus / angleminus?
you forgot zerooo
Does the removal case need to be accounted for in the else clause of the outer `if` statement?
Because sometimes, as in test-file.js:11, the File() object is created without any properties given. While your for loop simply ignores this case, testing for the hash key within an undefined object would throw an error.
Added a guard to prevent improper usage

2012/9/3 Giuseppe Vallarelli <notifications@github.com>

> In eqcatalogue/harmoniser.py:
>
> > +        self.domain = domain
> > +        self.target_scale = target_scale
> > +
> > +    def is_applicable_for(self, measure):
> > +        """Returns true if the measure given in input can be converted"""
> > +        return measure in self.domain
> > +
> > +    @classmethod
> > +    def make_from_model(cls, model):
> > +        """
> > +        Build a conversion model by a regression model
> > +        """
> > +        return cls(formula=model.func, domain=model.native_measures,
> > +                   target_scale=model.target_scale)
> > +
> > +    def apply(self, measure):
>
> This api can be misused, i.e. an user of this class can use apply without
> using is_applicable_for. Perhaps a check inside apply is needed.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gem/oq-eqcatalogue-tool/pull/16/files#r1515516>.
>
>
If anyone knows a better way to enforce `'expanded' => true` than this please let me know.
RE: lines 80 - 81

Shouldn't it be dis**crim**inator instead of dis**cim**inator?
Why is this a bad idea? It's not just about cluttering the global scope; there's also the issue that other parts of a program can change the value of a global variable, which is generally undesirable because it can lead to unforeseen side effects.
По PEP-8 в начале файла пишут импорты: первая секция - для стандартной библиотеки, второя - для третесторонних библиотек, третяя - импорты из своего проекта. Каждая секция отделяется одной пустой строкой.
Same as the domain empty test. Is this needed?
Apologies, I should have explained this better.  With anything other than SOCK_STREAM or SOCK_SEQPACKET, you can't call listen() period, it throws an error:
```
$ circusd test.ini 
2012-07-13 04:23:35 [6328] [INFO] Starting master on pid 6328
2012-07-13 04:23:35 [6328] [INFO] Arbiter exiting
2012-07-13 04:23:35 [6328] [INFO] foo stopped
Traceback (most recent call last):
  File "/usr/local/bin/circusd", line 9, in <module>
    load_entry_point('circus==0.5.1', 'console_scripts', 'circusd')()
  File "/usr/local/lib/python2.7/dist-packages/circus/circusd.py", line 110, in main
    arbiter.start()
  File "/usr/local/lib/python2.7/dist-packages/circus/util.py", line 283, in _log
    return func(self, *args, **kw)
  File "/usr/local/lib/python2.7/dist-packages/circus/arbiter.py", line 205, in start
    self.initialize()
  File "/usr/local/lib/python2.7/dist-packages/circus/util.py", line 283, in _log
    return func(self, *args, **kw)
  File "/usr/local/lib/python2.7/dist-packages/circus/arbiter.py", line 187, in initialize
    self.sockets.bind_and_listen_all()
  File "/usr/local/lib/python2.7/dist-packages/circus/sockets.py", line 98, in bind_and_listen_all
    sock.bind_and_listen()
  File "/usr/local/lib/python2.7/dist-packages/circus/sockets.py", line 52, in bind_and_listen
    self.listen(self.backlog)
  File "/usr/lib/python2.7/socket.py", line 224, in meth
    return getattr(self._sock,name)(*args)
socket.error: [Errno 95] Operation not supported
```
where test.ini is:
```
[circus]
check_delay = 5

[watcher:foo]
cmd = foo.py $(circus.sockets.foo)
use_sockets = True
numprocesses = 5

[socket:foo]
host = 0.0.0.0
port = 5400
type = SOCK_DGRAM
```
Therefore I think we need to not call listen() except for on valid socket types.
The toString() method for the class is missing.
wer'e => we are
Patching Net::HTTP is bad - is there another way to do this?
Should be `0`, in other places too.
Is passing in the function you're binding as the second argument intentional here? Unless I'm missing some context, it will always be `undefined` at that point.
Request-specific?
Is it necessary to check for sys._MEIPASS existence and have fallback to sys.argv[0]?

sys._MEIPASS should always point to exec_path - in onedir and in onefile mode.

Or do you experience any situation when sys._MEIPASS does not work as expected?
If we're not at an "official" release yet, let's brand this as a WIP maybe?
Looks like we're handling the same list of features as in `ALL_JS_FEATURES` here. Will it be nicer if we turn this constant from array into a hash (with classes as values) and replace this whole method with `ALL_JS_FEATURES[feature]`?
Now that you mention it, it does look silly, will fix.
Actually, never mind.  I was over-reading the phrase "can be run", but that can be just "python blah.py" rather than implying that the file itself is an executable.  So I think it's fine now.
inner and outer works for me.
Great to see all this speeded up by trading in squared quantities.
Like master-lincoln said, "If this code will not be used, please delete it."
Same as here.  My mistake.
a) __all__ is an indicator of what should get imported when you do a import *
b) by default, I don't believe anything that starts with an '_' gets imported with an import *

So I think we should be all good here.
add the "~ " for now, so that user can pass just regex in, not nginx config (and a TODO for other path options, e.g. literal, case-insensitive)
Is this file supposed to be referenced? Or is it purely example of what someone else might want to include?
I don't mind, but it never worked up until now. I didn't want to change behaviour when moving it over. I don't mind fixing it though... important enough?
@michelemorello  @paolovergori  
I got rid of the seq on this file. 
It was mandatory in order to do the backoff and re-request of the image file.
Can we replace its functionality on other files too, so that we can stop having this package requirement? It's so confusing by the way...
pzp.config.name should get your pzp name
huh? heh
This is dangerous because the reset could be done before the executeAsync is done. Please add the handlers (error and completion) to the executeAsync
I think that beagle is probably better installed via the pkg for this reason. Does homebrew support this? The CUDA drivers aren't necessarily needed for beagle (since beagle will still use multiple processor cores) but it is a "nice to have."
Ar fi mai bine sa facem o metoda statica in clasa Config cu tot ce e pana la linia 51 si doar sa apelam metoda aia in main.
I don't know. The only thing I know is: without it, no stun at all.
This should probably be a constant defined in MediaConfiguration.
Can we also do the same for MEDIA_URL
We should give this method a more suggestive name.
this property verifies that the locator contains the expected bodyclass
This is a good point. Let me explain the logic for what I've done.

First off, in the lastest push I have renamed is_the_current_page to no_forms_are_open (I'm not sure which you are looking at as GitHub shows me the old code above your line comment). 

I observed that the page we want to assert that we are on has a number of sub-forms, and as each one becomes active a Save Changes button appears, and then when the sub-form is submitted the Save Changes button goes away. I was trying to assert that none of the sub-forms was open. Perhaps the logic to ensure that none of the forms is open is ok, but we also need to add another check that looks for something else that we expect to be present on the page?  Or perhaps I should be checking that an element of the Basic Info form is no longer present on the page?

When the basic info is submitted there doesn't seem to be a success message that we can use - the form just disappears from the page.
I would copy the content, are only 3 engines and 2 template frameworks
On Sun, Sep 30, 2012 at 08:02:42AM -0700, Wei LI wrote:
> @amueller @GaelVaroquaux I have question here: I see that pipeline code and
> checks whether each of the transformer has "transform", and if I delete this,
> this cannot be put into the pipeline.

Indeed, the current code cannot be put in a pipeline, because it does not
know how to do out of sample projection. Adding a stub raising a
NotImplementError will not fix it.
again, it is not clear to me whether this actually frees memory if you don't explicitly call the gc.
I'd change this to *After training, the fitted model no longer expects the ``y``*
domain.local-host-name=The name of the locally running host controller.
Just only TypeStruct and TypeClass requre this check, and there is no common super class of the two.
In this case, I'd like to collect codes in one function.
Really need to separate function to TypeStruct::checkAliasThisRec and TypeClass::checkAliasThisRec with adding virtual function call overhead?
You still need -m$(MODEL)
I did not know that existed, thanks! The only problem I have is that the Json format is incompatible with the one stcToCBuffer is printing. Although it would be nice to share the implementation, I don't know how feasible it really is. It might be better to put a comment in both places (or maybe only in json.c) pointing to where it was copied from.

Alternatively, the SCstring table could be moved to an externally visible location so they could at least share that table. That makes the most sense to me.
Where do the TEXT_ALIGNMENT_* values come from?

I don't see them defined anywhere, they are part of Ti.UI in the API.

Please add them and update the apidoc for them as well.  The actual defined values should then be used here in the switch instead of bare integers.
this allows not to spend some minutes next time
I think fgetln is BSD-only, but I don't know of any linux distros that lack getline, so I think this is OK.
You are correct it does not add a duplicate
So, is this allowing for , GROUPED attribute to be specified in the future? (which isn't currently in the grammer for hashdedup?)
I'm not sure I understand this..

Shouldn't this be looking for the super within the transaction? 
Or should it be looking up it up at all at this stage... shouldn't it be looking for it in the transaction action only?

e.g. what if I do something like:


FileServices.StartSuperFileTransaction(),

FileServices.CreateSuperFile('regress::superfile1'),

OUTPUT(FileServices.SuperFileExists('regress::superfile1')), // true
FileServices.DeleteSuperFile('regress::superfile1'),

FileServices.FinishSuperFileTransaction()
Is the ~ really correct here?
If you were avoiding it, it wouldn't have to detract from the improved clarify or robustness IMO. i.e. it can be implemented cleanly, within the transaction.

But I agree, It's not enough time involved to be worried about, even if 1000's of subs., the time will be dwarfed by rest of action.
Not sure it's the same thing... In Gavin's code, if neither are null, it still checks the types.

But I believe Richard's code is more correct, since if they're not null, and their bodies aren't equal, they won't match.

@ghalliday if it can be used by non-constants, I think it should have a more generic name...
The function could be expanded (it isn't currently used).  However the whole inline/not decision needs revisiting anyway (there are some issues already open for related topics)
Shouldn't you be reading these from environment.conf ?
typo: in?
The comma notation is only useful when you have a really long list of variables. 
By writing

```
var newHashString = something;
var currentKeyValue = something;
```
then you can delete on or all a third without the diff needing to affect the line above or below too.
Apologies for nitpicking, but you could use a space after the `<trackerName/>` tag here. Everything else looks great though!
Tha's true, verified. On top of that, we keep a list of these observers that we unregister on unload.
No I think we do / were doing something similar either in loader or bootstrap. Also I remember there were some edge cases fixed later on.
This sentence is missing a word: "Event may of specific `type` ..."
I think I've given up on being SQL and mongo.  It's really hard, and I don't think it's that helpful.  If they want SQL they can use vanilla whiskerboard.
Cool.  That's a good idea.
You might consider adding "-Dmaven.test.skip=true" to speed the build.
Based on our discussion yesterday, how a "Thread is a set of messages", maybe we should call it thread?  Though changing the terminology internally won't be a lot of fun :)
This is just `(or (apply <= a-seq) (apply >= a-seq))`.
There might be other web servers that interpreter extension names the same way as an Apache server, I'm not sure.  I just leave that as a note and let the user figure out.
Trailing comma in array?
Hmm, I just realized that point 2 in your PR summary suggests that float values outside of [-1, 1] are clipped, which explains why you've removed this check.

I'm uncomfortable with this change, and would definitely prefer an error message. It's not difficult to accidentally convert an integer image to a float image, and this change would hide the mistake. Clipping is appropriate in some situations but not all, and it's probably best not to guess what is or is not appropriate here. I think this was discussed on the mailing list, but I can't seem to find it.
`NOT_A_NUMBER = "abcd"`
How is this ever set?
In JS?  Since you're not using Date.parse anymore, it's just our parsing code that has this limitation, right?  It would be easy enough to change the regular expressions to support either delimiter, and maybe a tiny bit faster because you could avoid allocating a new string to hold the replace result.
The touch handler is creating the types of movements needed to submit to the camera controller.  It makes a `distance` movement that is used for zooming, and the `angleAndHeight` movement is used for rotate/tilt.  This allows it to re-use the existing update3D, updateCV, update2D functions.