labofsahil/github-event-dataset-2012 huggingface dataset:
+1 on `format`.

I think that `__str__` should fall through to the current `string()` method with default args.  I usually think of `__str__` as something that provides a human-friendly version of an object.
其实可以，不过现在暂时不想要area和发布人。没想到更好的办法，我再想想。
Missing leading slash on your example?
I did want to note that I discussed removing the -ldl with gmaxwell / sipa and a few others before submitting this pull.

Are static builds still being used? They not for the official binaries, to my understanding at least.
I get it from the Measure plug-in who is a copy from the GXP version should it also changed ?
The API spec has an example for submitting POST data.

Regarding PATCH in specs, rack-test added support for it 9 months ago, but hasn't seen a new release for about a year :(. We could use the github master version instead I guess.

https://github.com/brynary/rack-test/pull/33
https://github.com/brynary/rack-test/issues/60
I think it would be good to explicitely differentiate between Google tracker issues and Rietveld issues.
For example, i'd change the description of SetIssue to 
"Set this branch's Rietveld issue number.  If issue=0, clears the issue."
In fact, i'd go as far as to change function names to [Set|Get][Rietveld|Tracker]Issue, and all variable names to Tissue (or Gissue) and Rissue for even greater clarity.
This looks slightly messy to me - there's both a `val` and a `def` called `remoteCommandLine`, both are open to being overridden; together they seem to behave like a single method with an optional parameter, but if you do override one you'll just break the relationship between them. (That 'relationship' currently exists as code repetition.)
Could they become one method with an `Option[Credentials]` parameter?
typo: `donrun`
Is this macro equivalent enough then or do we want the caching?

On Tue, Oct 2, 2012 at 12:46 PM, Bussonnier Matthias <
notifications@github.com> wrote:

> In IPython/frontend/html/notebook/templates/page.html:
>
> >  <!DOCTYPE HTML>
> > +{% macro static_url(name) -%}
>
> static_url was tornado defined macro that was adding a ?v=file_sha for
> caching. With is doable in Jinja2 but not worth it if i remember correctly.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ipython/ipython/pull/2363/files#r1743023>.
>
>



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
I have not used JS exceptions before, but it probably makes sense to start doing so.  Does JS have a notion of exception types we should/could use?
by the way, when you hover to rename, je css jump a little.
No, this adds cost to all instances only for the benefit of a doctest.  Instead, the test should be changed, to not depend on the order (it can do a `set` check on both keys and values, for example.
@mattstep @dharmamike this is a template option i.e. for the entire group.  Do you think we should call this floatingIpPool? and then for node server created, it can pop an ip off this pull during creation?
can you move these to .config.AbiquoProperties ?  new convention
Perhaps it might be helpful to introduce a `NIPagingScrollViewController`?
The heck, let's return!
ah ok, does it mean if I make the change in base urls.py I can drop the privacy bit of, of all of these URLs?
Yeah. It's just the order from the bug, and they're still discussing, so it'll change soon regardless.
If you want to save a few lines, you could do:

```python
form = SMSSendForm(request.POST or None)
if request.method == 'POST' and form.is_valid():
   # ...
```
oh clever fix to that annoying one.
I guess this will need to change before we merge it in... :)
this function already is a transaction.  Using named savepoints is a technique of doing nested transactions.  Honestly, I'd rather have the MySQL "insert or update" statement.  I embrace the python "ask for forgiveness rather than permission" method rather than the "look before you leap".  A nested transaction is of the former, your code suggestion is the latter.
Ah, sorry. Ran an old without the plugins section. Maybe adding a deprecation warning would do it?
Great catch, fixed and added a test.
Good points.

I'm ok with the current implementation then (I don't think boolean parameters are evil  ;)).  @igorw?
"모래언더의 모래"
should be
"샌즈 오브 듄"
Why doesn't this case (no USE_SCALERS) check the bytesPerPixel?
-> puts "bar"
While I appreciate the reduction in LOC from removing .idea, I don't like that these sorts of formatting changes are now going to be inevitable.  Is there any way to specify a pom.xml _and_ a code style?
I'd use the property instead of the getter for performance: $this->prefix (same line 55)
Please remove the phpdoc on private properties
No. Adding a violation on other property paths than under the current scope does not sound reasonable to me. Do you have any use cases for that?
extra empty line to be removed
Agreed.  

Aren't they just checking if utc is not nil?  I think utc would have to be an NSString since that is what it comes in as.  I think we could remove all confusion if we say -  if (utc) {...}

The iTuneU CS193P instructor did those types of checks all the time.
But what's the point of passing the splitview to the block, if the block is a property of the splitview?
Re overcommit, I see the point. Overcommit by default to yes so there's not much change to ask for but if memory becomes scarce it would come down to OOM v. agent stops working. I think redis is a pathological case because it makes extensive use of malloc -- unlike our agent so I think it's not a clear-cut win for threads.

Interestingly Instagram was complaining about fragmentation caused by redis and its constant calls to jemalloc causing the issue you mentioned (needing to "reserve" ram on a redis box).

SSL-timeout, we may be ok with threads. The broken code has a PySSL_ALLOW_THREADS macro around the handshake in Modules/_ssl.c so it would release the GIL, get stuck in handshake and never re-acquire the GIL (or so the code reads to me).

I'll stop my advocacy for processes now. It was born from having experienced enough multithreading bugs in the past to make me cautious.
will chat with you about this
Does testing for A > 0 even make sense anymore here?

How did you arrive at 1000 for the upper bound?
呃，我猜是成员变量的标记吧。我看Android很多example也是这么定义的。
那好，我也统一成coverImage, titleLabel, authorLabel, descLabel
I find the `*~` ignore to be essential; especially if your working with those editors that _autosave_ your work.
Below Andrew was using an in finite loop around `hang` instead of hlt.  I have no idea why one is better than the other.
added a team activity feed well, we can turn this into tabs later
You shouldn't repurpose an existing message like this. The message id becomes a key into a kbase article and if the message is repurposed the kbase article may be misleading.

Instead, create a new one. If the old one becomes unused, that's ok. If unused commenting it out is ok. (Leaving it in the file commented out is a good way to show the history and remove peoples' temptation to reuse the id #.)
yes you are correct. that make sense to create once
What browsers are you seeing this happen in? Can you provide a demo page showing this? Browsers should report `<body>` when no element is focused, assuming the document has loaded. If you click on a button to open the dialog, then the button may not have focus, but that's fine because as a mouse user, you don't care if focus goes back to the body. If you tab to the button and press enter to open the dialog, then the button should be the active element, which is what we want.
There is a global instance of TestSwarmContext available as `$swarmContext`. Ideally this method would become a regular method and all callers become context-aware, but for now it is easiest to work around it by using the global instance.

In most cases it is the same instance anyway since Context is just a good pattern to follow but in this case it will usually be the same instance. And besides, it won't lose any state that it didn't have before because it was static anyway.

See for example `swarmpath()` in `inc/utilities.php` for a similar case.
background service would be killed so I think you may want to listen telephony messages as well as sms messages instead of events. Sorry for this big new amount of work :(
according to the style guidelines (for gecko/gaia?) this should be
````js
if () {
}  else if() {
}
````
Don't do that please :)
There is a typo ;)
We must forget about using `shared/js/custom_dialog.js` as is going to be deprecated,  you are doing it well appending the dialog html in the app, but that does not mean that in the communications app we can not host a similar js file for mananging this huge amount of dialogs :D

I think that the major concern y this PR is having some dialogs in dialer applications that are being used from other comms applications

Hope it helps!
no :( because of assigning selectableFriends to selectedContacts and you delete selectedContacts[xxx] checking items, you are deleting element in selectableFriends as well
yep, we can use the Name || number part, and the photo logic I just added. Carrier logic is not important for the list
Next PR will have all this.
Label should be "Brazilian Portuguese" or "Portuguese (Brazil)"
why do we need an arbitrary class name here?
We can do it some magic with:
`label:not([for]) {...} `
In that case when you use a for you get the default label style :D
Is it not supposed to be a different building block? Or does those buttons have something special that make them looks different?
I am fine either way -- picked RebaseAbort as it is easier to find next to Rebase
I'm going to reboot into Windows and try to disprove that "Sinatra won't build on Windows" ... but in any case that's different from this. 

"rake" means "rake spec" which runs unit tests and smoke tests, and makes sure you don't have any syntax errors in your step (or .mw or whatever) files
I couldn't get that to work.  I assumed it was to do with how the for-compr is translated; it was late night, etc; but I'll look again.  Actually I don't have to look again, I think I typed p <- future.  This is why we need examples.
Purely style, but any reason to favour these over pry?
This dummy wire is here because spi.write returns before the data is out thus the CS will get released in the middle of the last byte.
Consider a case/switch here, since you're always comparing `infrastructure` with a string.
You should definitely be filtering `app_name` to avoid possible security problems (e.g., if their app's name has periods in it).
https://github.com/AppScale/appscale/issues/105
Do all of these need to be global constants? Any reason why they can't be class constants inside DatastoreProxy?
I think inside of lib it should further be in a folder respecting the 3rd party libraries you downloaded. Whatever they originally named it.

If we leave it as lib/uploader.py and in the future we add another 3rd party library that happens to have a uploader.py file it'll overwrite this one. Further separating the folder will avoid this kind of namespace issue.

I recall it was originally already in a folder representing the proper name I would put it back in the same folder name.
># text node

Yes alright.
again, the "chat" in getChatChannels() is redundant
The reason is probably some Google-related nonsense to do with passing on PageRank.
why not begin/end ?
Yes - the `fetchall` closes the cursor, which drops locks in SQLite.  The `whereclause` should ensure there's only one result.  The alternative, `fetchone` and then `close`, is just a little more cumbersome.
No need for `if` here actually
It would be useful to still mention that the `root.html` template is used for the welcome page.
let’s stick to functional programming style “one assign per variable” that gracefully reduces bug count

```coffeescript
handler = _.find Backbone.history.handlers, (handler) -> handler.route.name is name
if handler?
  handler.route.reverse params
else
  false
```
I think you should only do it in ``execute``
So either remove this first download attempt (or better, dont, see if CN_Match will work first)... and fall back to the SAN validation. (though Im not sure this method of SAN validation is  ok yet.)
AFAIK, used classes are imported via `use` clause, so did I. Is that wrong?
fixed. test added.
Because I was not aware of what the composer json printer can (quickly hacked this together at ignition). However if you could quickly tell me if it supports outputting [that format](https://github.com/ktomk/composer/commit/cfc55329750af11e479fc35b806c0c3b90823ee3#diff-2) (including wordwrapping at column 80), I'm happy to adopt if it can.
this change isn't necessary.
remove this route as you don't need it
set this to something that is not in the DOM already, previously `"Lorem ipsum dolor sit amet"` was used.
Would you mind moving this out to another pull request? I'm not yet, if this will cause side-effects by lazily loading / executing some of the code.
Pushed

On Mon, Aug 20, 2012 at 9:37 PM, Hilary Ho <notifications@github.com> wrote:

> In index.php:
>
> > @@ -167,11 +167,15 @@ function LeadingZero(Time) {
> >  	</header><!--end div menu-->
> >
> >    <div id="content_container">
> > +<<<<<<< HEAD
>
> No need to make any pull request. Just push to "fixes" branch of yours.
> Und??
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nvcHilary/CoinCod/pull/3/files#r1413689>.
>
>



-- 
Regards,
vincent
How long does it usually take before the administrator password is available via the API?  I've seen it take over 10 minutes to be available via the management console.  How often will be not be able to retrieve the Administrator password?
There are named exceptions in both the Runtime, and the Generator. We should use them :)
Never change CS in the AlternativeCodingStandardsBehavior, **never**!
Wouldn't this cover up a malformed data file? Shouldn't it error in this case?
You probably want to make sure to restore $, to the old value afterwards.

It would be very weird if because of that somebody would get a spec failure in another spec depending on whether this spec has ran or not.
Should this be .text() instead of .html()?
Changed to add "suggests" and "require-dev", and to move the dep out of "require".
Move "implements" keyword up a line.
I'm wrong you must use `else if` to catch the posibility of $query=null and $options=null;

Can you add a test case with this posibility (both arguments = null)?
; should not be here
Add // CraftBukkit start/end
It's sucky, but it helps maintain things between updates
Classes should be named with CapWords style. [reference](http://www.python.org/dev/peps/pep-0008/#class-names)
But why changing the title ?
I would not change the behavior in this pull request unless there are good reasons to do so
my bad, forgot about them
+1. We generally just use src unless this is absolutely necessary.
You're right. I remove them.
Does this work better than putting the if on the `<tr>` above it, or do you just like the more template-like syntax? Looks fine, I'm just curious because I haven't seen it before.
Rename the variable. FQN is not obvious. Rule #5: Do not abbreviate.
I'd recommend: `$proxyClassName`.
Also, align the `=` signs.
I'd still leave this method here commented.
They are the reference we have when we implement the Concrete Table Inheritance (1 table = 1 class), which we call as "UnionSubclassPersister".
this should not be needed, phpcr-implementation should be enough.
we probably should let the use specify the extension he wants
Change H2XADS1
Right, it uses XPC so it's stateful. We can ignore this comment.
Bad form to check in pending tests. You can delete this file.
What's troubling is you found the need to count them all.  I wanted a break between sections.
Then you might want to do something like:

```python
assert not u.api_key.key
assert p.get_api_key()
```

Or something that demonstrates that `get_api_key()` did actually create it.
I'll stop commenting on indents but they'll all weird in this class. Fix?
The triumphant return of `makeLinkUrl`
Would be nice to just blow away all the track events and plugins here cept footnote. Like I did here.

https://github.com/mozilla/popcorn-js/pull/175/files#L0L9

Otherwise, this example does not run.
why are we overriding this method ? we have i in page.py
Unit test is OBE
Hi! Someone else will have to look at the backend, but I am here to be a pest about frontend. *g*

Can you make the classes here "abandoned complete-no iswip" instead? iswip is the class that says, "This symbol indicates the status of the item," complete-no says, "This item is not complete," and then abandoned would explain why the work isn't complete. Basically, there are instances where one might want to apply a style or script to all incomplete items, so it's handy if they all have the complete-no class.
A thought: you could get these mappings from `httpexception` instead of hardcoding them here.
file.add(asdfasDF)
Unless you tested this with the IE bookmarklet, it's probably best not to mark it IE-compatible until the automated tests get to it.
- call admin css from admin form
This should be `BLUE` without the quotes. It's a constant that evaluates to `"#6495ED"`
Well the lookup any, you should probably just add a barren ReadOnlyProduct with just a name/id.
This also matches if someone calls their local favicon file "httpfavicon.ico". I am not in favor of a mere substr test
I think i covered this case in the tests.

And as the commit detail states, this "fixed" quoted string regex was found on the php-parser lib
@mbax Thought since there's a byte[] needed it would be way more readable using a string. Changed it to a single byte now. Hope that fits. For sure feel free to change anything you need - I'll fix up the client then.
@AutomatedTester suggested that the mouse was 'moving' horizontally first rather than vertically if you leave it to it's lonesome, causing the hover to go away. I really  have no proof that it's helping / hurting but it does make this method suitable only for hover menus that open down. if i encounter any hover menus that open in any direction other than down, i'd add a parameter dir=[up|down|right|left] and have the move by offset do the right thing.
`text` is not defined at this point. You probably meant `rel`
I don't believe this will do what you want. By doing a `count` on `result-set`, I believe that you'll end up forcing evaluation of the entire sequence (which may defeat the purpose). Also, I'm not sure that you can iterate over a result-set multiple times like that...unless it's a scrollable cursor, which i don't think we're using.

Seems like this could be simplified to just use `take`:

    (-> (take limit result-set)
        (convert-result-arrays)
        (vec))
I think the default version should be None, so it autodetects, not 1.4.1. That way, 1.5 support will be automatic for existing code
Possible typo. Shouldn't this be 'ant' instead of 'any'?
IIRC, This will be addressed in the Authz work
I think it's probably safe to day that something has gone wrong if the config value cannot be found. I think we can just blow up in that case.
It's true for the one below as well, but that is a longer query. So maybe it's not necessary to do anything here.
I think it makes sense to do this in a dedicated search.js file as well.
This end point should go in oae-config
This doesn't really verify that the image has been cropped properly and saved in different formats.
Ch. 5 vi 
(The text for this section comes from Ch. 12 in the current UG.)
Вы забыли проверить что именно **заменяете** нуклеотид. У вас возможна ситуация когда, например, "С" поменяется на "С" , что по сути и не замена вовсе)
Можно подробнее почему не работает?
Запись эквивалентна записи int count = round(DNA_LENGHT * percent / 100);
Т.к. числа целые округляется и без round.
Или я не прав?
I have renamed this to `lintFunctionNode` in cb9a8cad4318840aa0cc915134ae166d2e21475a
I realize this class is for testing references to sharded documents, but should we also test the ability to use something like `simpleShardReference.$id` as a shard key?
Can you please include details about how to get this key? ie sign up for an account over at https://something.com
Specifically, these `if` statements are confusing. Perhaps all local provider stuff should be in its own `describe` block.
Look Ma! A missing trailing new line :)
Yeah, in theory this could be automated by splitting connectivity itself to discover the disjoint sets. I'll see if this takes too long, and if it does, just scrap the feature (it only saved 5-10% time if I remember correctly).
one thing we should keep in mind while adding more code is to keep the test suite as fast as we can (we could already do better). When it's not required use very few epochs as here. Maybe events[:4] is good enough. Besides this it's good !
Damn you french and your odd punctuation rules ! Corrected.
This should be setDefaultOptions() as of Symfony 2.1. Also, an example configuration should be included here IMO.
It seems dangerous to overwrite the value. The point of the lambda is that it is run every time it's needed instead of being statically set.
remove the extra parenthesis
Maybe it's not that important but @param is not documented
Probably needs to be conditional
This needs to stay, you're building on an old version of bukkit
fixed? Did you fix this?
`contains?` ei toimi sekvenssien kanssa sillä lailla kun voisi olettaa. Se nimittäin palauttaa löytyykö indeksi sekvenssistä.
Oh, I see. Wouldn't hurt to add this to the Create javadoc. Same for the rest...
I think this log should stay in handleTopologyAffectedCommand
Excellent point, commit coming right up...
I think the original indentation of this whole chunk was better; I don't think that the end of this block should be indented the same as "points".
I added a function to the User class which returns all the users on a given committee, and changed NumberOfVacancies and NumberOfPositions to refer back to this function.  Now they are both only 3 lines long and we have another helpful utility function!  Lemme know if this is a good alternative to what you said.
yeah...if we have the api, then we don't need mozwebqa in the setup/teardown, but it is otherwise problematic.
get a circular reference otherwise, middleware references response :) I thought it was icky too btw
ScenarioTest
If it's dead code, just delete it. If we need it later we have an SCM for that. If not someone will forget why it was there in the first place (see Lava Flow Antipattern).
Is there reason to use (or even have) the `OpenLayers.String.startsWith` method?

If it is a confusing use of `!`, I would think `url.indexOf("http") !== 0` would suffice here.  If we don't care about confusion, I'd even go with `!~url.indexOf("http")`.
As a side-note, this will be supported when the Canvas v5 spec is implemented (with [hit regions](http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2012-March/035239.html)).
This takes the unordered trends output, sorts it for each trends stat, and figures out what the end period of the timeseries is.
I think the comment should be specifying that CSS is being added to head, not body.
@bp323 : I think it's acceptable to continue using double quotes inside of strings.
@jsloane Can we add a `;` at the end of this line?
I know it's not necessary but it will help keeping the diffs more clear.
@gsilver could you add an empty line at the end of this file again? That way git won't complain
It's great we have a lot of comments here, but I think a lot of them are too verbose and will be pain to maintain. For example the one above duplicates the comments next to specific entwines and is going to very quickly become outdated. @adrexia could you go through the file and clean this up a bit?
I looks like palliative solution.

You may use Shellwords.escape.
Stops the server if it is running
Should the return type not be "PHPUnit_Framework_Test" instead of "Test"?
лучше просто getId
This corrects the 45 degree border on the arrow
ZeroClipboard10.as should add it too.
You forgot the docblock.

BTW the loci is reached when I try to access some property that just doesn't exist.
Is it added elsewhere?  I chose to add it here, after the digest was calculated, so that other changes that might move an annotated warning would not affect the value of the digest, but would still be alerted during annotation calculation.  Does that make sense?
`thin start` works here, too.
Thanks @stephpy - this one I haven't modified yet (the code is copied directly from the ORM version at the moment).
Right, will add that change.
No, in this case redirect back to the contact form page.
Why don't you pare XML only once?

```
horoscope_xml = parseString(urlopen(horoscope_url).read())
```

Is there maybe even some function which reads from file descriptor? So you do not have to `read()` it first into the string for `parseString` to consume it?
I am not sure what have you changed here except indentation?
You like this on its own line? You moved it again there. Then OK, let it be. ;-)
Empty lines between CSS blocks.
Confirmation and activate? ;-)

Maybe `EmailConfirmationProcessTokenForm`? And the above `EmailConfirmationSendTokenForm`?
Technically you don't need this, since you aren't setting `_net_lookup_counts`.
Indeed, there is a finite risk (although extremely low). Right now I am thinking whether this Facade Test could be replaced with another, more efficient, one.
Given the unit test `CreateWithAbstractionRequest_ReturnsResultFromDecoratedBuilder` isn't this test redundant?
But what if he sets his alias to Asphodan for example, and then Asphodan actually joins the game? I think a method which checks the player.dat file is needed here...
$targetProperty = $this->getProperty()->getTargetProperty()
А вот с кодировкой надо разобраться
i am used to use code formation like this:

if (something)
{
    getSize().y = Display.getHeight() * size.y / 100f;
}
else
{
    getSize().y = Display.getHeight() * size.y / 100f;
}

in my opinion this is hard to read:

if (something) {
    getSize().y = Display.getHeight() * size.y / 100f;
} else {
    getSize().y = Display.getHeight() * size.y / 100f;
}

but that are to coding styles. So sometimes i make extra spaces which makes it easier for me to read (to me). but i will remove them in the future.
I have to say, I don't know either. :)  I wrote this a long time ago (1+ year?) and looking at it, I can't say.  Not good.  The method works, but it should definitely be re-written.  Will add that to new commit.
The CSimpleFileFind is called with ```while (finder.FindNextFileNoDirectories())```, so most temporary file are not deleted automatically after one day. We need to make it work recursively here.
I'm guessing he meant the indentation.
Can we use __setitem__ instead here? That way zero values will be correctly ignored.
On Thu, Apr 12, 2012 at 06:14:49AM -0700, Raphaël Pinson wrote:
> > @@ -4,6 +4,7 @@
> >                      $opts=[],
> >                      $config='nginx/proxy.conf.erb',
> >                      $proxy_target) {
> > +  include nginx
> 
> The problem I have is that I call `nginx::proxy` in another class, and it
> doesn't include the `nginx` class, even if I tell it so, so the
> dependency on `Exec['nginx-reload']` is not met. Doing this fixes my
> issue.

Weird. The "scope" of ressources in the yaml catalog are completely flat.
Classes don't have any sort of importance at this point.

I guess the nginx class is just not included at (grep through the catalog
for nginx-reload to be sure), probably some code reorganization ended in
some sort of regression. I guess the node you're working on is not
rebuildable from scratch at the moment (ie: nginx won't get installed).

This patch is IMHO just the symptom for another problem. Sorry for being
picky :-)
I prefer the `blocked` approach - good idea.
having to create a new HashMap and copying stuff around for each association worries me a bit. It's good to focus on functionality at this stage but let's keep in mind to think about alternatives.

Some ideas - I didn't try them -:
- provide a iterator on column values/names (something like EntrySet) to avoid creating these arrays.
- have an alternative Map implementation thing to pass to the DBObject which is able to behave as needed without copying stuff around
Adapter not needed as of Python 2.5 (minimum version of Django 1.4).
This also needs documentation in the docstring for this function.

Documentation is also needed to describe the usage of this, probably in `docs/customizing.rst`.
"...and set *both* the height and width...". 

The picture does a great job of explaining what you mean, but maybe double up by mentioning explicitly that both the height and width are at 640.
This doesn't fail (but should) due to extra try-catch block in PigToThrift.
Also, now refers to kiji metadata.
nitpik

s/optionnal/optional
You should set the buffer tag indicated the buffer is in the memory,  see the macro ngx_buf_in_memory() . Otherwise, nginx will treat this buffer to be stored in the file.
Replace with a call to ngx_http_pagespeed_get_srv_config.
Ok. I'll get rid of it in 'internal' classes. Saves some typing I suppose.
put this in the else of if num_levels, expensive call that is not always used
well, the point is that PRs should stay separate to have one feature per PR. If the other PR is not merged yet, simply use KnpMenu 1.0 for now (and merge the other PR into master).
This is from some other pull request? Anyway, there were comments already for that. Please fix.
Perhaps we ought to add "approximately" before corresponds, just to be fully explicit?
I'll move it to the Offerextensions. Looks like the properties in the sdk object is for binding api values.
i looked in guava for this ... should have looked in our own ResourceUtils !
uh oh, i don't think we should be allowed to invoke stop after we've unmanaged it.
(it works currently, but shouldn't.)

suggest we:
- removeMember (as group)
- then stop
- then removeNode or unmanage

could maybe do the stop in removeNode ?
We should not need this line. Before_filter ensures that the user is signed_in before he can take any action.
Sbtconfig.txt the main reason for this patch is that we read that guy and
use its values on the command line
client reexports binding.namespace, so it's fine.
Just re-factored the code to fit into a single line and reconsidered:
```php
if (
    !isset($_SERVER['REMOTE_ADDR'])
    ||
    ($_SERVER['REMOTE_ADDR'] !== '127.0.0.1' && $_SERVER['REMOTE_ADDR'] !== '::1')
) {
```
What do you think about this proposal?

That would get rid of the suppressed warning, is a good deal faster and is psr-compliant.
Not sure if symfony does require `||` do be on a single line thought.
It's a good idea (no pun intended) to add this directory to your .gitignore ;)
That was just a mistake
Sry, haven't had time to look into this earlier.

It works, but: the commit message quite confused me. To me, it reads like "I didn't remove my emacs backup file from the test repo -- instead of removing it, I'll just raise expected_count by 9 and remove a check for ignored files and things will magically work again". Shouldn't the fix rather look like the following?

```
@@ -338,7 +338,7 @@ static void workdir_iterator_test( 
 void test_diff_iterator__workdir_0(void)
 {
-       workdir_iterator_test("attr", 15, 4, NULL, "ign");
+       workdir_iterator_test("attr", 14, 3, NULL, "ign");
 }
```

Thanks.
this should be ``if (!@chdir($docRoot))``
can this check be done with regexp instead?
Arrrr, everything is simple:
SmartVent<->Pipe<->Something
SmartVent (the one which is controlled remotely and does the pumping and cycling) is in my case in the airlock chamber
Pipe connects it to something
And Something in all of implemented cases is AirTank, but i offer using that simple Vent here in station, not pumping vent - we already have this as #1, just simple pipe cut open and protrude from wall.

1) Use airtanks/aircircuitry on asteroid - second is better, anyway is more good and right decision on asteroid.
2) Best way on-station is to connect airlock piping to station airpipes, but that is hard, because then we need air sorting machinery - for compressing we have to take air from station supply pipes and on decompression - use station scrubbing pipes.
3) Using simple Vent on station:
3a: makes pressure equalisation on pressurizing faster by pumping some of air from other side, still the fact what internal room is always bigger - not makes pressure very small inside
3a*: anyway - at first you'll mostly go out of the station, making pressure in said room higher, and it is returned back to normal on your return
3b: said in-station room would make sorting of air: fresh one comes from station-air-pipes-vents, excess one goes into scrubbers.
and here ... and more more more xD
commandArguments[0] присутствует в каждом if
I totally agree ... would you mind writing up a JIRA to put in a proper logger?
Instead of `Checking environment with:`, use `The following verification command:`
Please see new version here:
https://github.com/phil-schneider/mongo-java-driver/commit/b9fd84288cbe17238423a3a95dfe065a4e6a7fed
PEP8 issue: base.py:33:64: E225 missing whitespace around operator
I understand the need for the ability to create test users, but think that once we have other tests that use them that we can delete this initial test.
Need info: Is this being used? I think it's not in the API.
I am not sure what you mean by your remark, @amueller . 'foo is not None' is the right way to ensure that the test is not triggered only by the 'None' value.
I wonder is it's not better to check for symb.dtype.kind == 'i', so that you don't fail for int32 or int64.
I believe in the math package there is a max int constant you could use
Mockito is not enough since there are some final methods/classes and static initializers that need to be mocked. Powermock adds those capabilities to Mockito.
That regex is using tokens I'm not familiar with, but I suspect it's buggy in the face of paths with more than one / in them.
I see no reason why this can't be implemented
Remove cannot check what type of object is passed to it but it can test if it was added in the first place. Added a warning log if the user tries to remove an object that was never added.
Same as above for not having the image path relative to the location of the current js file.

This seems to duplicate the work done in the NLVIO class.  Does it need to be done in both places, of could one invoke the other so the code is not duplicated?  Maybe this update could be called after the sets are done on the NLVIO..
Appcelerator coding standard requires testing for NULL, e.g.,

if (itr != timeoutList_.end() && itr->second != NULL)
accéder à des données d'utilisateurs
typo "attribute"
This is really the crux of the commit @toe @rochestb.

dat-gui will give us the ability to modify game settings and values on the fly without having to modify code. It's visible in the browser: http://cl.ly/image/2O2A392S230C.
this is the case.

What notation do you use ?
Possibly, but not very likely is my guess.

You can't remove them from the keyed portion, you probably could from the payload.  You would need to add code to ensure that didn't happen.  Also you would need to ensure that dictionary[9].x indicated that field x was used from the dictionary.  I suspect it might not at the moment.

Since the dictionary operations don't stream you're unlikely to remove much.   You may just end up defining multiple copies of a global hash table instead of commoning them up.
readlink does not add a null terminator.
This is actually the problem line - I need to call super.before() after the assumeThat.
This is incredibly specific. It's very un-form-specific. 

Can you make that filtering outside the form instead. The views.py is often full of specific business logic oddities. Perhaps it's more complicated that this but it looks like you should be able to ...

```python
# comment here justifying this hack...
specific_releases_only = ('Nightly', 'Aurora')
relevant_versions = [
   x for x in request.currentversions
   if x['release'] in specific_releases_only
]
form = forms.CrashTrendsForm(
        relevant_versions,
        request.GET
    )
```
Hi, no problem, it was just an automatic add from my IDE. On my side i don't care to appear anywhere. I juste move some code so I totally agree you stay the main contributor ;)
a scalarNode is a variableNode that accepts only scalar values (ie no array)
"Олон мөр нөхцөл шалгалтанд `?:` (гурвалсан оператор)-г ашиглахаас зайлсхий, оронд нь `if/unless`-ийг ашигла" гэвэл илүү ойлгомжтой
RESOLVED FIXED!
Ah. Redis encodes all the keys passed to it as utf-8 (or whichever character set is passed in to Redis' charset parameter). Woot.
Same as before, `$this->data['parameters']` may be better
I think so. The returned "object" from window.open call is another window reference: https://developer.mozilla.org/en-US/docs/DOM/window.open thus you can call .open on it
You can save the file and see if it takes effect (if you have "only apply formatting ti changed lines" enabled, edit the line)
use the instantiated obj rather than calling the page pages.methods directly.
@shacker interesting, thanks for testing it out, I agree, this shouldn't block the merge.
@shacker What was the reason for this change? Is it intended that the lhnav now looks like this: http://cl.ly/JXZa ?
is `print=silent` important?
@bodepd I'll add the docs !
1.upto(size) do |i|
I think this include line isn't needed, am I wrong?
No, you cannot. NiPy does not estimate the transformation matrix, you must explicitly declare it there.
Post conditional
Originally, I tried ``self._meta.cache.set(cache_key, bundle)`` here, and it gave the following error which I don't know what's going on.

(Pdb) self._meta.cache.set(cache_key, bundle)
*** PicklingError: Can't pickle <type 'cStringIO.StringI'>: attribute lookup cStringIO.StringI failed

Therefore, I decided to just delete the cache. As a result, the whole philosophy will be, only loading data to cache when we do get request. 

It will be nice that I can load the data into cache here to reduce a cache miss on the next request.
s/with message/with the message/
Is `interleaveTextureCoordinates` the best name?  How is this going to scale with new attributes.  I have my eyes on `occlusion factor` - precomputed ambient occlusion.  Perhaps `interleaveAttributes` or just `interleave`?
`=undefined`?
You should add the `clear_params="true"` line to your launch file, otherwise removing parameters will have no effect as the values still exist in the ros parameter server.
Change the **ID** of the container(`#tab1 -> #tab_container_table`) and also the corresponding closing comment.
I would actually put these in the test methods itself. dont worry about code redundancy. that's the idea.
sizeForFont must also take textPadding into consideration.
It would be much cooler to have a new event simply called `tap` and have it return the number of touch points as a property of the event object, but for the sake of matching the current API, I did this.
May need use a copy of data due to https://github.com/appcelerator/titanium_mobile/blob/master/android/modules/ui/src/java/ti/modules/titanium/ui/TableViewRowProxy.java#L188
Can we have a less ambiguous name like `horizontalWrap`
This was actually reworked in 8110...guess I forgot to backport it. It's updated now.
Ideally yes but until our whole system is powered by relative font sizing, I'd lean towards not doing so since it can cause some unknown outcomes if a users browser is set to a default font size higher/lower than 16px. I'm open to this discussion though. 

Sent from my iPhone

On Oct 12, 2012, at 4:41 PM, discojedi10 <notifications@github.com> wrote:

> In depot/plugin/_p-comments-a.scss:
> 
> >  } // END p-comments-a
> >  
> >  #reply {
> >      .form-footer { margin: 0; text-align: right; }
> > +    > header h2 { font-size: 14px; }
> Should this be a relative pixel size? 0.875em or 1em?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Is it really necessary to clear the input? The other commands don't do that. Doesn't the code clear the input after every command anyway?
This led me to think... perhaps we want an `appendAuthentication(request)` on the `SignatureType` enum? We'll have to pass the logger in which is messy... thoughts?
I didn't think of that. You're correct that it won't work in that case. I'll change it.
С целью?
@HakanS, what should actually be the date?!? The date of each core release?
*This* time, there is no way to avoid it. In particular, this is subject to bug 8847
> Anyways, I'll change it back. Want me to do it for repeat/cycle/take too?

Yes. Any situation where you're explicitly instantiating a template within itself and you don't need to be, please change it back. Anything not directly related to this pull request that you've done in other pull requests or noticed that others have done can (and probably should) be done separately, but definitely any case in this pull request where you've done that should be undone.
From what little I understand here. Code goes as follows:
take next power of 2 of `minCapacity` call it a `newSize`. 
(yeah this hacky loop  does this.. but only on 32bit platform 64 bit needs also a 32 bits shift)

Obviously the next nextSize < minCapacity branch is never taken (unless overflow is hit and that's kind of unlikely).
So newSize is truncated to never exceed half of the `range`.

To be honest in both cases it's used I'm not sure why it has this check for the half of the full range.
Same goes for the next power of 2 here. I could have understood the desire for page size granular allocation but just power of 2, dunno.
The first one sounds reasonable, but do we really need to worry about dates tens of thousands of years into the future, especially when FILETIME values are only used for the current time, and filesystem times?
Wait a second, the docs say `Ο(min(r1.length, r2.length))`, and 1 is certainly in `O(n)`, so this shouldn't be a problem.
this is a bit destructive :)
maybe throw an error if we have more than one child?
Line breaks, plz.
Pretty sure that assertion will get optimized out, because we're talking defined language semantics here, but whatever!
pack this code into a method please
You should still keep this class here.
I see the signtool docs say to use http but that's surprising to me -- I'd expect it to be https
so you're saying this now works with all versions of Refinery up until 3.0?
I fixed the other issues but I can't help with the Testing
The summary should be an English sentence not just a phrase.
The badge will disappear for that case.
I'm not entirely sure if we should use the term "repository" here since we already have the concept of mounting "repositories" and the concept of data "repositories" for usage of the DLStore.  At the same time, it kind of makes sense that this is a PortletFileRepository.  Also, are there potentially places where we are going to use this outside of a "portlet"?  What about the name AppStore (hahah… ok, I didn't realise that until I typed it) or an AppRepository?
We don't have to preserve Author anymore.
А часовые пояса мы будем учитывать?
Don't we want pushState to be enabled by default as long as we provide everything needed to manage <a href> elements with preventDefault and Backbone.history.navigate?
this is the extension used by the DefaultFileLocator
This is a little misleading -- `$row` is not a jQuery object until the next line. Maybe just declare a `$row` variable below and leave this as `row` to remove any confusion
1. A clarification comment would be fine
2. Yes, it is the C rule, and the doc should say so and should be corrected if it does not
3. It is semantically correct because by definition they are the same thing (not a signed word)
4. You don't need a version forest for ssize_t, since it is the same as ptrdiff_t. I don't know why sizediff_t would ever be different from ptrdiff_t
From what I gathered back when I poked around with this, the `NCCS` value in the glibc `termios.h` file is just a generic/portable value, while the `NCCS` value in `termbits.h` is the actual value Linux uses.

I could be wrong though.
rewrote display switch links
Wrong spacing betweetn the type and the parameter name.
Just style: I write all object/dict keys in `'`.
"Вызываемый" можно спутать с "callable", но, пожалуй, это наиболее удачный вариант, жаль не получится использовать.
Can you remove abstract ?

Also you need to respect CS (see 4c9bff8a46c524f2e9247b569c49714f3334a97c)
Yea we can move it back into the main class but I think the plugin is still much more readable like this.

I generally do not like to keep it in the main plugin class unless the plugin is simple and compact.
following conventions, this should be user_id.
mock htmlPanel
Yes, that's true. I don't handle context change at the moment. We could just fake a template change whenever the context object is replaced. I wanted to have the strings, rather than Nodes, in the standard attributes (.body, .from_email, etc) to facilitate testing; and to not have to send() the email first in order to populate those values, to enable easy testing of emails using code like this:

https://github.com/aptivate/intranet-documents/blob/master/tests.py#L403
Only thing...  The different path might mess with people...
Import only modules. Put mongoengine into a separate section of imports.
yeah that's probably the right call especially if we decide to add OptIn to the SubscriptionList class
This declaration isn't used, and should be removed.
I'll move this check to the `@check` decorator.
Boom, done!
https://github.com/telefonicaid/Gaia-UI-Building-Blocks/commit/8cf11664208445c5abf9b8a9a143405a352e1900
I would remove or clarify the comment, it's only confusing as it is now.
This would seem to break the convention of a combined getter/setter (unless mech does it elsewhere, but I don't see it).  Is there a good reason to have the no-arg version set autocheck?
This is not true. Whichever property was set most recently takes precedence (i.e. the code `scrollView.disableBounce = false; scrollView.horizontalBoundce = true` will bounce horizontally, but not vertically).
We're probably going to want to be able to send some information into this page as well, e.g. error message(s), etc.
rspec実行時に下記エラーがでたので、その回避のために付けています。バージョンは変えてません。
<pre>
WARNING: Cucumber-rails required outside of env.rb
</pre>
Much sofistics solution for comments
> `thr.pCount`

Хех, работает неправильно, ибо эта переменная означает количество постов в треде, а не количество видимых постов. :<
When callback is triggered it also might be a failure to invoke, to not mislead with a log you can check if error is define - log failure otherwise - success. function (error) {if(error)...}
you are correct, clearly just going crazy
Why are these marked as TODOs?
Can just be .toHaveBeenCalled()
OK I can make a patch for webplatform easily. Syntax I believe is sourceOrigin, destinationOrigin, allowSubdomains
Very correct.
Please run a cs fixer http://cs.sensiolabs.org
might be that \Iterator must come first? i have some dull recollection that contrary to good reason, order did matter for this.