labofsahil/github-event-dataset-2012 huggingface dataset:
well, I just needed a token to send around, and protobuf does not really have this feature, so I thought a string is extensible at the same time (i.e. containing what it was that is done now, should that become necessary)
top-level is initial slash.
I think this section could benefit from a small detour of at-most-once, once-and-only-once and -atleast-once guarantees
This is still racy: if TerminationHook message is being processed and finds a child, but before finishing `switchOn` that child terminates (i.e. removeChild is called), then there will never be a confirmation of the shutdown. Terminated processing needs to be done under the switch lock.
will add docs
It looks like this doc section is duplicated in two places now. Keep it DRY.
Do we ship the html docs in the download?
In that case it doesn't work to take the url.
See my answer to Patrik below. If unacceptable, we have to discuss it and come up with a better solution.
If the actual percentage value has no practical impact on the gossiping then leave it (but I assume this is not true). 
If it has impact then I think it should be configurable, different clusters have different needs, better to be overly tunable.
1. emails are poorly auto tested.  any alteration here must be tested manually if we don't add tests.
2. this seems like it was just done for taste reasons.  no reason to have  this in this branch?  if so, please try not to bundle in future.
Google thinks otherwise "at the next level" 422M,  "to the next level"  70M. Somehow, to seems nicer. This is what I initially wrote and then changed it to at.  No worries, will ask Ian or Matthew who are native english speakers.
hehe, thanks

I've been on a wonderful/awful "reduce" kick lately
Isn't some of this redundant to things in app/models/card/reference.rb?
`--pool-wait-minutes` should not imply `--pool-emr-job-flows`.
Я тут подумал, а не вынести ли нам все эти волшебные строки в константы?
Yes they could, however they are not really meaningful in this context. However, I'll add a ShellMessages.error in here
I'm not a fan of using `Timeout::timeout()` when there are other better ways to solve it. Can `block_until_no_more_write_locks()` and `block_until_max_locks_satisfied()` take a timeout argument?
comment seems outdated now?
I'd prefer if we could avoid a bit of the duplicated code, but I see how it may not be desirable to sent the complete `QGestureEvent` into the zoomable. But since `pinchGesture` goes in, I'd prefer that the `changeFlags` check be moved to `Zoomable::handlePinchGesture`. It should be no problem to `return true` also when the pinch gesture did not change scale, anyway.
The message is not really self-explanatory :smile:
`var plug = data.lookup('plug')` ?
@jspahrsummers Will have to explain that to you.  It was originally a `TUIPopoverBackgroundView*` cast, but he wanted id https://github.com/github/twui/commit/fbbcee5ea029830e10fe8abea56e0c7c157c2f8c
Can you please fix it?
nechcelo by to potom zmeniť aj `/bin/sh` na `/bin/bash` resp. nejaky iny?
code which is commented out should be removed
Does it need `_tuple_of_integers` in there also? I was thinking of a single function for the whole bit. Also, `inline` isn't (or wasn't) portable, use `NPY_INLINE` instead. The name looks good.
Mostly I was pointing out the `.to_yaml.gsub(/\n|---/, "\n# ")`.

At the very least, it'd probably be worth making that its own method. It'd be even nicer to use an existing library if possible.

(Also, I think the semantics of this implementation are surprising `foo # bar` and `    # baz` won't do what you'd expect.)
This isn't the required arguments, it's just describing the arguments that the function takes.

The logic that determines what is required and optional is in the function itself.

If only 1 parameter is passed, it checks it is a SessionHandlerInterface, then the second param is optional. Otherwise the first 6 are required and the 7th is optional.
One more place where declaration should go upper )
Under normal circumstances I'd mock you for having a Facebook account in
the first place. But o think you've already seen the error of your ways ;)
On Aug 11, 2012 2:51 PM, "Andrew Faulds" <notifications@github.com> wrote:

> In ext/standard/info.c:
>
> >  		php_info_print_hr();
> > -		php_info_print("<h1><a href=\"");
> > -		php_info_print_request_uri(TSRMLS_C);
> > -		php_info_print("?=PHPB8B5F2A0-3C92-11d3-A3A9-4C7B08C10000\">");
> > +		php_info_print("<script>(function () {\n");
> > +		php_info_print("    'use strict';\n");
> > +		php_info_print("    window.onload = function () {\n");
>
> Yes it did. Last time I trust them with my contact list. This is very
> irritating, I apologise.
> -- Sent from Samsung Mobile Andrew Faulds http://ajf.me/ nikic <
> notifications@github.com> hat geschrieben: In ext/standard/info.c:
>  php_info_print_hr(); - php_info_print("<h1><a href=\""); -
> php_info_print_request_uri(TSRMLS_C); -
> php_info_print("?=PHPB8B5F2A0-3C92-11d3-A3A9-4C7B08C10000\">"); +
> php_info_print("<script>(function () {\n"); + php_info_print(" 'use
> strict';\n"); + php_info_print(" window.onload = function () {\n");
>  @TazeTSchnitzel <https://github.com/TazeTSchnitzel> I think your
> Facebook just went wild ;) — Reply to this email directly or view it on
> GitHub.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/php/php-src/pull/132/files#r1357361>.
>
>
Jen pro jistotu: takhle jsi přepsal odkaz na knihovnu graph!
[deprecated_method_warning(method_name, message)] will be called with the *deprecated* method name and the message it was declared with. Return a warning message.
The pipeline with sprockets being adopted brought people problems,
some people didn't even like :cache => true

I get asked and asked how to disable the pipeline.
I stopped talking to my friends about Rails "because" of the pipeline.

If there is :remote=>true, I think this section should also be in the rails guides,
it's an organized technique and people know that _js.erb_ exist,
to reduce the bloat on js files.
We kinda follow that one-line changelogs have the name at the end, multiline have the name in a new line separated by a blank line.

About the indentation, if you look the file through github you'll notice that it contains only two spaces, whereas the others have 4. Are you using tabs by any chance? If so, please change to spaces.
doesn't this effect how the QueuedMessage class gets inherited into the parent class?
On a second thought, that would require changing the `Hashing` interface.

I would have a separate class then for the `Byteswap`, to avoid indirections if only the regular `Byteswap` is to be used.
Sure, hence the minus one
I did a quick PR to remove these from the numpy code as well: https://github.com/numpy/numpy/pull/308
Okay, I agree on this one. We can remove them and think about a way to
warn about complex values some other time (research task for
GoogleCodeIn?). However I would still prefer if there is some way to
test this function. A small change in the errors/warnings raised by
numpy or sympy can brick it as it is written at the moment.
The length of the args is not what you should be testing for at any point, this is easily broken by doing ``ask(Q.imaginary(x**y), Q.imaginary(x) & Q.imaginary(y))`` (should be None, but gives False).
>>
>> I don't see what q.v. stands for here.
>>
>
> http://en.wikipedia.org/wiki/Q.v.#quod_vides
>
> Should I change it to "see there"?
>

That's new to me. In my own writing I've used vida infra...but since
the docs are not for a math journal, perhaps we can error on the
non-latin forms for such things.
I'm not a fan of this API.  I would rather just make it explicit.  If the user wants just symbols, have him pass `{'Symbol': Symbol}`.
Actually, this isn't the case. Only if x>0. Otherwise, you can get cases like ``(-1)**(1/2)``.
OK I like this way of handling it in `qapply`.
@smichr @asmeurer I'm sure there must be a standard way to wrap a function not implementing 'deep' to make it deep, i.e. to call it recursively on all functional parts of the expression. Could you guide me to the relevant part of the core (I suppose) -- I couldn't find it.
Yeah, changing codebase isn't right since we'll break BC.
Need extra comment as for `getSort`.
would it make sense to also add YAML?
Add @since tags to new api
will change prev; updated ruby so good on that count.
I think some degree of docs on a key API would be a good idea, even though a lot of the actual calls are self-explanatory.
i think those chat locks are unnecessary for messages, since the AddMessageToChat function is locking the chat.MessageBuffer (and the chat itself is never locked for adding messages as far as I can see)
this seems absurd as written becasue "synced" doesn't have an object so its more of a features without the corresponding usecase, which is weird.) 

The idea/approach should be more like: when you want to store files on systems in multiple data centers, you can use a gridfs store in combination with a replica set to ensure that each facility has a local (read) copy of all files that is always (roughly?) current.
Should these white space changes be in this PR?
`ext` -> `ext/`?
IMO yes.
Space after // please.
I added some thoughts in IRC. I like the version payload for commanded upgrades, but it is something we can add later.
People get kicked out of the game when they get banned nowadays, so it holds true :)
This api change wasn't mentioned in the backward-compatibility notes.
https://raxaemr.atlassian.net/browse/TODO-67
Why do you take the first cube in the list rather than the last (as before)? If the merge can't find anything to merge and does nothing, you are changing the behaviour.
I prefer:
```python
with warnings.catch_warnings():
    warnings.simplefilter("error")
    with self.assertRaises(DeprecationWarning):
        for subcube in cube:
            pass
```
as it is standalone (it does not create ```cube_iterator``` for use by later test code)  and it's a little clearer (to me at least) than the generator expression. It's still not thread, but I can live with that.
Perhaps better renamed as "is_brewer"?
It is safer to use ``from theano.tensor.sort import sort`` to be more robust to various configurations (where ``sort`` might happen to point to another module than the one intended).
Actually there would be a bug in that.

>>> x = T.matrix()
>>> y = x + x
>>> y.owner.inputs.index(x)
0

node.inputs.index will only return one index, even if the variable appears in the inputs list multiple times. So this would discard some paths that the gradient can flow through. Is there a clean way to get v to map to a dictionary mapping nodes to index lists?
I also don't want v to map to a list of (node,index) pairs because I would need to postprocess that later to detect repeated nodes. I only want to process each node once per variable in grad_dict.
вообще-то правильный хостнейм лежит в Administration | Settings | Host name and path

я уверен, что к нему есть доступ )
please define search_fields_ranges and iterate over them
Yea, you make good points that this should be less of an issue given the more automatic nature of capabilities.  +1 for leaving things as they are.
remove comments from the code
This method does merit more detailed documentation since it is not necessarily intuitive.  I'll lobby again for an expedient resolution.
The user object here is actually the one created from the fixtures that we're modifying.
well, it verifies that the thing we're trying to make work actually works  :)
This is going to have implications with respect to managing embedded `<script>`s, so test it *extra* thoroughly. Also, we use `===` instead of `==` except for the special case of `window`.
I don't think you're going to get a perf boost here. It's just replacing one function call (getting a cached tokenize) with another. Also, it's so much prettier without it. :)
A function like LinearRegressionAccumulator::operator=() is exactly what I like to abstract out; otherwise, we are having the exactly same code for most (prob. all) derived classes of DynamicStruct
I think we should do this in check_expectation instead. I think this will break otherwise if you change the deadline of an already instantiated Expectation.
I just implemented slugifying of playlist names using the `slugify()` function from Django. I've added a test that verifies that it removes e.g. `../../` from the playlist names.
If you did the above, you wouldn't have this problem.
https://docs.djangoproject.com/en/dev/topics/generic-views-migration/
So remove this entire line? When it was Net/IMAPProtocol it was impossible to run tests against it.
Can we also check here that the annotated fields have indices that include every integer from 0 to some n?
Formatting, did you use spaces?
I'm surprised you added "get." :P
I believe this will work with any property that has a getter and a setter. I think a property such as this would be serialized:

private string MyProperty { get; set; }

I wanted to discuss this one a bit further to figure out how we want this to work. We could always suggest people use InternalsVisibleTo as a way to work around this particular issue.

 Thoughts?
if loadBanner is preferred, and both createBanner and loadBanner are being called, do we need createBanner at all?
needs a space between `if` and `(`
Do both of these lines need to be console.error()? If an exception is thrown, it shows up as 2 errors in the console.
``i`` is already declared in the outer "CSS Context Info" ``describe()``
I fixed the syntax and also removed optional since commands are now required. I also added "position" and "relativeID" to the parameter list with the associated documentation.

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Thursday, May 24, 2012 4:49 PM
To: Ty Voliter
Subject: Re: [brackets] Menu and Command and Shortcut Extensibility implementation (#936)

> +     * TODO Question: for convenience should API provide a way to remove related
> +     * keybindings and Command object?
> +     */
> +    function removeMenuItem(id) {
> +        // TODO
> +    }
> +
> +    /**
> +     * Alternative JSON based API to addMenuItem()
> +     * 
> +     * All properties are required unless noted as optional.
> +     *
> +     * @param { Array.<Object.<
> +                    name:       string,
> +     *              id:         string,
> +     *              command:    string | Command (optional),

Instead of ```Object.<...>``` I think you want ```{...}``` -- the former syntax is supposed to be for Objects that are used as a map.  Also, use "?" instead of "(optional)".  Also I think there's an extra ")" and "}" near the end of the line for "bindings."

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/936/files#r879036
FYI, these shortcuts work on Win7, but not Mac 10.6, for me.
I'd `dd_loc` to be `False`by default.
No problem, I will remove this commit.
On a related note, someone should really put this in dimagi-utils, and if not go through the entire codebase and replace @property @memoized uses, at least start using it and replace uses as we see them:

    def cached_property(fn):
        return property(memoized(fn))
maybe in settings to be overwritten in localsettings if someone wanted to? this can be done later i suppose
Cool, I hadn't seen Array.wrap before. That's handy. I was specifically trying to avoid the [nil] result, which it handles beautifully (by returning an empty array). I'll switch to that.
Even though we're using VCR I think this would be better written with a stub to be more explicit. That way we don't need the extra cassette for a special route.
“Transforms”* “into its”*
What I found strange was that although I had this...
This is a test file, I'm assuming. Does it need to be in the tree?
Actually, I'm not sure this matters. I checked some documentation, and nowhere does it say `start()` must be in a specific place.

Also (from start function in qunit.js):

```javascript
		// A slight delay, to avoid any current callbacks
		if ( defined.setTimeout ) {
			window.setTimeout(function() {
				if ( config.timeout ) {
					clearTimeout(config.timeout);
				}

				config.blocking = false;
				process();
			}, 13);
		} else {
			config.blocking = false;
			process();
		}
```
I guess not, since the DOM will definitely be loaded in Butter's ready function, right?
Wait, why do you want start and end in the advanced options? Or did I just accidentally do that and never noticed?
This feels too fragile; I think it'd make more sense for Cornfield to create this URL. Maybe you can just copy the Referrer header?
You can move this back to where it was.
Do these need `url`? Is that wrong?
Actually means 'long' - but either way I couldn't face using such a horrid class name :(
Shame there can't be a single `save()` method to avoid the POST/PUT code duplication. They'll line-for-line aside from that.
botio lints merged with master files, so line numbers can be different
-1 
String.format is evil sauce in logging - if you are trying to make it look like what slf4j does, suggest you write a wrapper string formatter that never could throw an exception if you mistyped a token.
@adreghiciu that code was changed by formatter, not by me. I did not want to touch anything more than needed in here (but reformat slipped in).
This is great, was actually going to do some similar refactoring. Would be good to see a description of it in the pull request :)
Same as above. We should not add the results to TreeSet
I saw the comment on the pull request about the inclusion of the copyright on the haml.  I'm Ok with including it; however, when I raised the question about including them in the past, I was told that it was not necessary for the haml.  

That said, I'm not sure that we need to include info on File Type/Author/Date/Time, that information always becomes quickly out of date and can be easily obtained through git history.
Apipie can give you live view on your documenation in development mode, I thing it's worth leaving it there
Volatile.Read(ref _subscriptionState)
This README is incomplete.

1. source file (`data.txt`) should be removed from Gaia
2. this URL doesn't give me the `data.txt` source file.
3. please verify the how-to below.
4. `cook.js` generates `db.json`. Not `words.json` and `phrases.json` in JSZhuyin.
Should a writer function be returning `None`?  Probably doesn't matter for these tests but looks odd.
looks good.

```
SUPPORTED_PROTOCOLS = (PROTOCOLS + SECURE_PROTOCOLS).freeze
```
is nicer since Array#+ generates mutable object.
Does it matter that `#handoff_status.type` is never set for inbound transfers and defaults to `undefined`?
I'm a sucker for erlang:min(Length, MaxStart)
The point is if you already have to import eg qtipcserver.h in any files which are going to be using MAX_URI_LENGTH, there is no reason to have it separate in a guiconstants.h, it could just reside in qtipcserver.h, and I'd think the same is true of all the other constants here.
Assignment inside a conditional is against the coding standards, and I'm pretty sure the code sniffer looks for this.
An & on the end of a query string is not valid.

if you look at the spec you see the query string should be terminated by a # or the end of the url.

http://tools.ietf.org/html/rfc3986#section-3.4
Let's actually leave this as destroy and just make sure that the dependent events get destroyed too.
I think we should rename this step to `... "fake@email.com" is an existing Loomio user`. Again, fake@email.com should be a regular-expression argument (`"(.+)"` should probably work) that gets sent to a step which creates the user based on the email provided in the step. That way we can use the same step to add multiple users with different email addresses.
Agreed. =)
Don't do as I do :wink2: I know I did this in `CourseMembershipsController`, but that was mainly because I was writing throw away code. You might be planning on adding error handling in a later iteration, but let's make sure we do typical:

```ruby
if task.save
  # Yay 
else
  # Oh no!
 end
```

You should probably pull some code from Community to help showing these errors:

<https://github.com/mendicant-university/community/blob/master/app/helpers/application_helper.rb#L3>
<https://github.com/mendicant-university/community/blob/master/config/application.rb#L59>
This looks very confusing. We already have a ```newInstanceFromClassLoader```, adding a second method with identical name appending 2 in the end is a recipe for confusion. I mean, every single person encountering this class will have to read all the code to attempt to figure out the difference between one and another.

Howard, could you add a Javadoc to this method that makes clear how it differs from the original method, and rename the method to some name that makes more sense than ```sameMethodAsAboveNr2```?
yes, it has some additions
For CloudStack architecture parameter takes different set of values
For this then I will add the method
RegisterImageOptions asArchitecture(String architecture) to RegisterImageOptions class
I guess that not on purpose?
Oh yeah. D'oh. Not much speed up like this...
We can get the layer we stored with each combobox item using itemData(index).toPyObject().  toPyObject will return QgsVectorLayer
You might considering making an sms-base template that has these og tags and other repeated content between the two pages.
`v` is the module here, right? Would you mind to change it to aModule. Similar in endModule?
Not sure why the getter is removed. Initialize it here in `_contentLoaderInfo` and keep the simple getter to "signify" read-only nature of the property
I.e. the word Crontabber
I'm not following your argument here.  The argument is an optional kwarg but I'm specifying it positionally - is that discontinuity that is problematic?
Git blame is going to show that you used double quotes! Imagine the shame.
Two newlines here
Commented code? Why do we need it?
This is ok, but I wonder if it would be slightly cleaner to push the logic a bit further into chef_db and expose a function like `chef_db:is_only_admin_user(Ctx, Name)`. This could allow us to implement it in a more efficient fashion I suspect. Instead of count(*), we could select where username/id is not equal to what we have and limit 1.
if ($value instanceof Elastica_Query_Abstract)

This should work ;-)
tl;dr the current commands make a lot of assumptions about being called by a local player and being able to grab session and world info from the player
Yep, it used to cut off.  Made figuring out things like payments/list a real nightmare.
The main "idea" was indeed taken from the GsonResponseHandler (and its father - CallbackResponseHandler). The thing is, there are lots of differences between those solutions, the json parsing is the only one really in common here. The error handling is different - calling the Callback's method vs. throwing an exception. I thought about putting this code in a common place, but getting it to work with both method could doing lots of changed to the GsonResponseHandler/CallbackResponseHandler, and it's not really worth it. 
If you have a good idea how to make it easy - do tell!
exceptions have messages AND message keys. No need for the translator to get the message in plain english.
casting is necessary because of the check in ``Form::add``: an integer would trigger the exception line 862
Why don't you move the assignment `$propertyValue =&` out of the conditional statement? I.e. `$propertyValue =& $this->isIndex[$i] ? ...`
It should be quite easy to implement something more sophisticated using ``preg_split``.
what about ``return isset($this->unresolvedChildren[$name]) || isset($this->children[$name]);`` ?
broken indentation (using tabs ?)
It isn't generally expected for your pull requests to include version bumps. I'm fine with this here, but just FYI, a convention exists.
However, for `property_expressions`, I'd replace this:

    member_expression WS? '[' WS? expression WS ']' | ID '.' ID;

with this:

    member_expression WS? '[' WS? expression WS ']' | identifier_expression '.' identifier;

or, if we want to support the following syntax: 

    foo.bar.baz
    foo['bar'].baz
    foo.['bar'].baz
    foo(bar).baz
    (expr).baz

with this:

        member_expression WS? '[' WS? expression WS ']' | ( member_expression | identifier_expression) '.' identifier;
inam gozashti :D
It's not creating a new metric per parameter. It's extracting the url to
match against user-defined regexes for the purpose of tagging URLs. I guess
I can provide better comments to explain.

HAPROXY_RE extracts a bunch of metrics from each a log line + the URL.
self.tags is a mapping between a URL regex and a tag, to match against the
URL itself (an analog is URL rewrite rules in a webserver).

One is inherently limited by the number of url-regex->tag entries to
provide in self._tags.


On Wednesday, September 19, 2012, Matt Perpick wrote:

> In dogstream/haproxy.py:
>
> > +
> > +HAPROXY_RE = re.compile("^.*haproxy\[(?P<pid>\d+)\]: (?P<client_ip>[\d.]+):(?P<client_port>\d+) \[(?P<accept_date>[^]]+)\] (?P<frontend_name>\S+) (?P<backend_name>\S+)/(?P<server_name>\S+) (?P<tq>-?\d+)/(?P<tw>-?\d+)/(?P<tc>-?\d+)/(?P<tr>-?\d+)/(?P<tt>-?\d+) (?P<status_code>\d{3}) (?P<bytes_read>\d+) . . .... (?P<actconn>\d+)/(?P<feconn>\d+)/(?P<beconn>\d+)/(?P<srvconn>\d+)/(?P<retries>\d+) (?P<srv_queue>\d+)/(?P<backend_queue>\d+).*\"(?P<cmd>\w+) (?P<url>\S+) HTTP/.\..\"$")
> > +
> > +class HAProxyLogParser(object):
> > +    def __init__(self, config):
> > +        self._logger = logging.getLogger('haproxy-logparser')
> > +        self._state = {}
> > +
> > +        # Maps url regex to one or more tags
> > +        # { url_regex: ("tag1", "tag2"), ...}
> > +        patterns = config.get('url_tags', {})
> > +        self._tags = {}
> > +        try:
> > +            for p, tags in patterns.items():
> > +                # compile the regex and turn the tag or tags into a list of tags
> > +                self._tags[re.compile(p)] = ["url:%s" % t.strip() for t in tags.split(",")]
>
> I think we're going to have to be careful with this. Will every single new
> query parameter create a new metric? Do people want this? We need something
> like this, but we need to figure out the right way to opt-in to
> things/patterns that users care about.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/DataDog/dd-agent/pull/193/files#r1640294>.
>
>


-- 
*Alexis Lê-Quôc *+1-917-512-6452

*Try Datadog for free (http://goo.gl/swO1h) <http://goo.gl/swO1h> no cc
req'd*
And follow @datadoghq <https://twitter.com/#!/datadoghq> on twitter
Put the binary operator on the end of the previous line.
"the function" maybe? 

use it as a marker ? - how?
Meh, that returns a number, most players are way too dumb for that :(
I'd suggest manually wrapping this, so they don't have to scroll
no longer need `end_key_len`
nitpick, but we're core devs, so we should set an example. PEP8 style please. ``lang + "/"``.
The content editor is always alone on a page - 

please take care of the other placeholder stuff, and ignore that thing if you like.
These should be moved to constants.
Looks like this needs fixing identation
@mecathcart the models ahve to be specified as sisters of initialize, not daughters. for similar example see Sessions
I formatted the file the way we did by converting it to a .js file and doing shift command f and then changing it back to .json.  I then reloaded the extension to make sure there were no problems.
C'est exactement le même code que tatami-commun.js non ?
I verified (with jstack) that the "PersistenceUnitDeployer-threads -NNN" threads are destroyed after about a minute of idling (after each deploy/undeploy).
actually, seems like it's "up to"
I will update mdn thanks!
I see. Maybe you could put `custom_dialog.css` in the `style/bb/` directory to be consistent with `edit_mode.css`, that’d be a good reminder. </my2¢>
Thanks :) I can pay you a beer for that ;)
if (!this.shown) {
Just as commentary, I had used `data-widget` instead of  `data-type` because type is already a property and it completes the meaning of the `<input>` tag. `widget` only adds some kind of representation.
Which input height would you need to update in the ThreadList page?
/!\ @alivedise
If you extract the variable line 213 (previous comment) you can reuse it here.
Is "Cancel" the word that UX and visual design wanted you to use for this?  It seem to me that "Clear" or an icon would be better.  But that's a UI issue, so not really relevant here.
Remove this extra call; you are already doing this in L22
When we mean direct children we should always use the explicit child operator in CSS, e.g. ```#apps-total-space > *```. But really, there's no need for ```querySelector``` here. I would prefer if you left the old form, just gating on the existence of ```apps-total-space```.
As mentioned above, you can return null here, and mediadb will not even report the file.  But now that I think about it, the problem then is that it will keep re-detecting it each time the app starts.  Maybe you could just return something like {playable:false} so you can skip the other canPlayType() check.
I mean, message is not an `sms`, is a `system-message` right?
Can it lives in the <head>...</head> ?
@dominickuo : I sync the toolbar z-index to header z-index to make the toolbar click-able while no message.
I also disable the edit/search button and hide the search input while no message.
Not very intuitive that returning null means we got the lock :)
perhaps these should be "tip" or "warning" callouts instead of being in the mainline instructions
new_sample is not defined.
Can you schwack this guy?
Like above, this comment doesn't explain much. From being pedantic, I would assume that this means that command failed to execute at all, which is likely not what you intend it to mean.
`$('select[name="filetag"]').cloneNode true`
The whole thing with thread isn't really sexy, but whatever for now.
Logging isn't a channel, and can you point out anywhere else where you chat in several channels simultaniously? It doesn't happen in any games I can think of, nor IRC etc... You might be "in" several channels, but you only actually type into one.
Yeah, I've been looking at other REST API implementations recently, and realized that many of them strive to pass data straight through from the database to the HTTP response.  The alternative is a lot of fairly repetitive code.  I'm not entirely convinced this is a bad thing, though.  The DB API accomplishes a few things nicely:

 * provides a higher-level-than-tables view of the database
 * isolates database-specific code (dealing with concurrency, dialect-specific bugs, and the details of the schema)
 * implements a well-defined interface that can, in principle and practice, be replaced by an entirely different implementation

I think that a pragmatic approach to the API makes sense, as we've been using all along.  Where we need a function, we'll add it.  It may come to pass that the data API ends up with its own getSingular and getPlural methods for each DB API connector.  That's OK!  The only downside is a rather monstrous amount of code, when you include fakes, unit tests, docs, and the actual implementation.
uses a file-based?
I still don't get it. Where exactly we win over `toJSON()`? It's recursively serialized every time model is rendered, right?
Why should we make libExtensions's include directory depend on 'ProjectDir' since all projects in cocos2dx solution rely on 'SolutionDir'?
You want to give this a go?

I have reviewed your code: if you are going to add this, I'll wait to merge so that I can also review that and do one big merge :)

Otherwise, tomorrow I shall resume my work on GhostDriver (I was away from this code for the last 4/5 days) and I'll do it.
As you prefer.

I'm based in Europe (to give a time reference).
Should'nt these be configurable by menuconfig?
Should be wrapped around an `else` like it was before, but without checking for `tasks.length`.
No, `INOUT` is correct (cf. [docs](http://www.postgresql.org/docs/9.1/static/sql-createcast.html)):

> **Parameters**  
> WITH INOUT  
> Indicates that the cast is an I/O conversion cast, performed by invoking the output function of the source data type, and passing the resulting string to the input function of the target data type.
why have a *MINLEN that needs -1? seems to contradict
drop the brace to the next line - or just get rid of it (nitpick)
The method name sounds a bit clunky to me. I like `self.requires(*keys)`, or if that's too similar to ruby's require, then `self.requires_context(*keys)`. I'd also `to_sym` all the arguments to normalize any passed in strings.
onClick?
Why `delete` ? Please at least document!
@mossop could you confirm the correct english naming for this method?
It allows to clean it up on firefox quit instead of OS shutdown/reboot. 
On Windows I'm not even sure it is emptied on restart :/
Oh wow. That is FANTASTIC. **&lt;face palm /&gt;**
Ah! ok.
It looks like this alert will never happen, since it's after the return.
can we get rid of it now, or is does generateKeypair invoke callback before function completion?
s/assyncronous/asynchronous/
Revert this change in your current branch, create a new one and apply it there.
Yeah. Copy paste and in a rush. I like yours too. I'll switch out. I should put in in utils as well.
session_start() creates a session or resumes the current one based on a session identifier passed via a GET or POST request, or passed via a cookie.

So the session_start() should have it in all code.
Does it make sense to specify a `:source` when `:local` is set to `true`?
@vrthra there needs to be a space between -d and device
This is a bit pedantic, but shouldn't this be `::UnboundMethod.method_defined?(:owner)`?  `@klass.instance_method` returns an `UnboundMethod`.

I think that owner is either present or absent on both `::Method` and `::UnboundMethod` but it still makes sense to check against the class that the object below is actually an instance of.
Don't define methods in the describe/it blocks.
I did it (see the latest commit), but I'm not sure it's worth it, because it adds the complexity to manage the directory (mkdir, rmdir).
could use 'else if' here
Hi, sorry about that. I have now added the license informations. I am not sure if they are included in this Pull request, but I cannot make a new Pull request....? (First time making a pull request!)
I won't comment much on the helpers because they will evolve further anyway, but: the comment lies. Instead of "exit": command-exit-code (integer) the exit key returns boolean.
Feels bloaty to generate a new it() for each file. Any reason it can't be in one it()?

```js
it('should generate files', function() {
  [
    '.editorconfig', 
    '.gitignore'
  ].forEach(function(el) {
    helpers.assertFile(el);
  });
});
```
I must have been half a sleep here,  that's the only explanation for not noticing this - Fixed in the last commit
This method should be removed.
To check (not sure about myself), but $filter ( $filtered ) may throw some CS problems with the PSR-2 conventions. Anyway, for consistency, please remove spaces before and after in function calls (the same apply for previous line : attach(new StringToLower())
Same comment as above. Also, this method needs a docblock.
BadMethodCallException sounds good here
You don't need use `@static` please remove all.
I wouldn't track this in the response. I'd instead do a hash map of splobjectid => flag -- you can then check the given `$response` object against that to see if it has been sent.
use this other tag:

```xml
<programlisting language="php"><![CDATA[
$code
]]></programlisting>
```
do you need this assignment?
Try to get a True/False instead of count using bool.
the { ?
Should be rbacService.
done.

also I use self-closing because it’s easier to distinguish closed elems:

```
<input>
<input />
<div></div>
```
ok, didn't know that, will use crossroads.js then, next weeks will be busy, so it may take some time...
This started syncing my `~/Downloads` to `~/Reference`
It would be better to use a different char(e.g.  `-`, `_` or `|`) rather than using `x`.
It should look like this:
```
that.applier.requestChange("popupMatches", newArray);
```
OK.. I added it because the AnnotationDriver throws the specific exception, whereas the XML and YAML drivers throw the common one. I thought it'd be more consistent and easier to test if they all threw the same type, but happy to go with the consensus.
i think its too early to limit ourselves to commons 2.4 yet. i guess this will not just be ignored but provoke an exception with 2.3?
changing this section of the test to something like the following proves that you have leaky variables.

```javascript

    test("should collect error information and report to Sentry", function() {
        var isSupported, frame, caughtErr;

        try {
            giveMeAnError();
        } catch(err) {
            caughtErr = err;
            Raven.captureException(err);

            ok(typeof(label) === "undefined");
            ok(typeof(data) === "undefined");
            if (err.stack) {
                isSupported = true;
            } else {
                isSupported = false;
            }
        }

        equal(ajax_calls.length, 1);

        var data = JSON.parse(ajax_calls[0].data);

       // ...
```
"Support for building/installing guile: default build procedure, and set correct CPATH."
Empty line before this?
use name here instead of eb['name'] ?
No spaces inside `()`, and generally put if results in their own `{}` block.
It tells the admin interface to not put any extra blank inlines in the list. If extra is 2, for example, the list of inlines would look like http://cl.ly/image/1M2A0J02340e

The link for adding new locales is still there, so new locales can still be added, it just doesn't have any until you click it.
Nit: alphabetize.
Please remove the trailing slash after /security
Smart. I'd say "Profile", but yeah.
`!important` is bad, you should find a better selection to do what you want here. Also, isn't this already the `body` font?
Can you test that this is the actual lat and lng and not a toString of a location object.
I imagine we don't actually need to add another table here - it's probably enough to just declare another `Manual` or `Article` class  **inside** the test class. This will create a model class namespaced to the test class but that uses the same table name by default. Take a peek at how I've done there here:

https://github.com/norman/friendly_id/blob/master/test/slugged_test.rb#L138
Are you working this (remove param dict from RDT create) with Chris?
We need a place to keep track of the pid otherwise the process becomes unmanaged.
Ah good point - I was thinking I needed this for macros, but the definition of a macro is the only case that needs to be preserved.  Removed.
Ok. You can have a look at the various `io` sources, they all define a dedicated clock, for instance there:
  https://github.com/savonet/liquidsoap/blob/master/src/io/gstreamer_io.ml#L36
Yes sir – I pushed a new commit that escapes the pattern matching characters.
i think since rails 3.07 the right format is something like

```erb
class='label label-<%= tag %>'
I was running it multiple times in the same checkout, so I found it necessary to add

    echo "Ensuring clean setup"
    rm Gemfile.lock

here
I also use it in the Encore translator and IIRC some other library catalogs, too.
This is a general MARC issue - the editors of edited books are turned into contributors rather than editors. 
However, there are cases where contributors really are just contributors, so I first check if there is an author anywhere.

I suggested changing that in MARC itself - and I'd still be happy if that could be done - but that suggestion wasn't received with a lot of love. We could take another stab at it. I find it rather irritating that, e.g. all edited volumes from the LoC come out wrong with the editors as contributors. @simonster any thoughts?
I think we should refactor a bit
Maybe this should be a class field, as this is needed here and there for the zk specific APIs
Why just return if it's interrupted?
Oops, that was a warning wiget i put in while it was being developed to remind me to finish it, it should be removed now
This default value shouldn't be in production code.
@tmartin You're right, we understand we are going to get a **string type prefix** but the `s` at the end of type should be removed. So, @gnugat Good naming :)

For the case where `$length` is null, I agree with @tmartin too. If the function accepts null as value, then it should be optionnal by adding `= null` to the prototype & btw, for this case, it makes sense according to my above comment.

@gnugat returns `LONG` by default because it takes my previous code as base for this PR. My initial intention was to not limit user if he uses a Fridge type & not provide options for this type.

For example, if you use the BLOB type without options then, IMO, you should get the biggest BLOB the platform can provide.
Github is not allowing me an option to close this. (The later request has been merged). This is just a note to other contributors *not* to try and merge this request. Will close as soon as github starts working properly.
can be deleted once definitions are deleted
Yeah, I just ran the example query and the return value says "certname".  We did this because for the UX testing it was really confusing to be using one term on the input side and a different one on the output side.  We should probably switch them both to "node", but for now I think that being consistent is probably the most important thing.
Supplementary?  Just throw 'em in with the fishes :smile:
I think I got it, so >> produces signed ints, >>> produces unsigned. Apparently IE7 doesn't support it. So I guess it depends, cause if you do -2.5>>>0 you get 4294967294, which probably isn't what you want.
Having this as part of the file-specific functionality would probably be good as well.
this can just be `..., [name, port], callback);`
I'm not entirely sure if this method should pass on an error  if a user can't manage a content item.
I can understand that the code becomes smaller doing it this way, but it feels a bit abusive.
This doesn't seem to be used any more
Better to add a ``timeout``
``start() ``
It should also cancel the previous request, not just the timeout - so that the callback is not fired. 

Also, this uses Mediawiki style spacing :P
Because if they should've been done, the current migrations wouldn't use a 0-padded 3-digit format. I didn't design it and I wouldn't have done it in that manner, but it is pretty obvious nonetheless.
Щас, попробую, массив индексов нужен для того, чтобы по-моему записать в него новые уникальные значения от 0 до х, поэтому во втором цикле проверяется на уникальность вот.. Я конешн наверно намудрил)) торопился.. а на след день узнал что дедлайн продилили но времени все равно уже небыло переделывать
А разве множество индексов не упорядочено? Не получается ли здесь, что вы не случайный индекс берёте, а с 0 по length-1?
Вот он,  этот метод в init
Он вызывается из init с дефолтным значением 100
Так что при создании объекта [[Cell alloc] init] его DNACell2->DNA.count = 100
Вроде думал правильно так )
В общем случае, правильно было бы получать количество элементов в зависимости от размера массива

```
count = [_DNA count] * X / 100;
```
@witem дело не в "нужно". Категорию вы вольны объявить где угодно, лишь бы перед этим был #import файл_класса.h. Ну и, соответственно, реализовать свою категорию. 

Другое дело, что при разработке, если делать всё в отдельных файлах, в Xcode всё будет "как на ладони" - слева у вас будет список всех файлов, по названию каждого из которых будет понятно, что за ним скрывается. Например, в файле Cell.h - по-любому будет класс Cell, ну и подклассы, возможно. А в файле Cell+Mutator.h - объявление категории Mutator к классу Cell. если посмотрите, Xcode сам обучен называть таким образом файлы, по команде New File вам будет предоставлен выбор - создать новый класс, категорию, расширение, протокол, и так далее. Слева - всегда что-то вроде "оглавления".
Согласен - проще, но на тот момент я не знал как взять этот элемент, что нагуглил, то и использовал :)
здесь не ошибка, потому что мы выбираем случайное число от 0 до 2, т.к. выбор всего из трех оставшихся элементов, исключая тот, значение которого совпадает с текущим
this is just my ocd but `msg = msg || 'explicit failure'` would be fine, doubt anyone will pass `'0'` etc
Shouldn't this be more like the SQL `between` operator which is inclusive?  Or maybe you just use `between` like this:

    where(['created_at between ? AND ?', start_time, end_time ]).order("created_at ASC, #{self.primary_key} ASC")
I was agreeing with you that, at the code level, preconditions are about setting them up rather than just checking whether they have already been fulfilled.  :-)
I would suggest turn reflow off by default
It was just for modularity as their logic are reverse. If you want them to be in one method, let me know and I will do it tomorrow.
I am not a huge fan of adding an extra argument. I'd rather think of a way to have a smarter “preload"
This does seem more explicit to me. Nice.
This toString needs to mimic what you just did in PersonName.toString().  This should be a machine readable string that is helpful in debugging, not somethign the end user sees.  This method should return toStrings from person name
Ahhh! The field is old! Need a break ;-)
Find an example command which also has arguments. Hint: "uname -a"

Have the students manually execute the same command on slave and then compare the results

root@slave:~# uname -a
Heh, I just saw that you deleted this in the other PR. Nevermind...
Please include all the parts of the sentence in the internationalized block.
It doesn't makes sense because ruby-info give me rbenv if rvm isn't install (this plugins isn't shown if rvm doesn't exist), mine is a plugin for rvm not for ruby.
default is 5 or 2?
indeed. Adding it is a syntax error. These are not array elements but arguments.
should be via https://github.com/php-fig/fig-standards/tree/master/accepted
So we're eventually setting `transform_params` to use `in_postal_code`, which means, I believe, that `InPostalCode` will eventually be sent over the wire? That's incorrect.
Add comment explaining that order is crucial here, just for extra airbags and seatbelts.
newChannel foreach setupChannelInternal
Why do you define __init__ if it does not do anything?
In [this test](https://github.com/rails/rails/blob/master/railties/test/application/configuration_test.rb#L249-292) `authenticity_token` is not sent in `patch` and `put` requests. Now exception is raised in this case (previously session reset was happened and test passed), so we can either skip verify or send authenticity_token to make it pass now.
Yes...if you don't have to worry about free_symbols then that is the
quickest answer to your query.
Oh, I didn't notice that one solution is the conjugate of the other.  That doesn't really matter, though, as they are both solutions (I think any addition of `I*pi*n` will be).
in WIMT, a -1 can be stuck in front of any affine in the series and the inverse will be applied for the affine. we need to think of a good way to do this.
Also, this isn't triggering a deprecation warning:
```
In [1]: from scipy.misc import radon

In [2]: a = random.randn(100,100)

In [3]: r = radon(a)

```
I expected a deprecation warning in line [3].
А потом ты изменишь чо в комбобоксе, и ноль станет валидным. Поэтому от
этого кода плохо пахнет.
Need reindent
yeah, even I thought, so earlier.  Had put it first under katello/organizations , but got an error called cyclic dependency, 
 But seeing used somewhere else in the code, thought it was ok, to use it from katello.tests
This should be:

```javascript
if (count === null || count === undefined)
    count = points.length;
```

If `count` is not passed, it will be `undefined`, and it would be nice to make that parameter optional. You don't need the `var` because `count` has already been declared (but may have the _value_ `undefined`) in this scope by including it in the argument list. `NULL` is defined by Cappuccino, but in regular Javascript code we prefer to use the standard `null` defined by Javascript itself.
maybe i should throw catch for consistency though...hard exit is ugly.