labofsahil/github-event-dataset-2012 huggingface dataset:
Its not clear that the return value is ```(x_pts, y_pts)``` - I was thinking the return tuple has a shape of ```(nx, ny)```... would you mind clarifying that a little?
I meant the following:

```
import shapely.geometry as shpgeom

shpgeom.Polygon
shpgeom.MultiPolygon
....
```

But as I said, you do not have to do that in this PR if you don't want.

Cheers,
No, not an epsg code.

I think the function name might be what's wrong here
because I *think* WMS servers can use an srs like "WGS84"
so I *think* that door is best left open?

Perhaps rename the function *from_string* or something?
Best is to just do `timeout = 10.0  # in seconds
Errata : jsonp can’t work (jsonp uses a script tag to embed JS, so only GET can be used).
i decided to put hte goal in the session's actiion this might be a bit annoying for users who dont want other people to know what they are trying to elicit..ie what they are researching. TODO we should start adding various levels to comments, what users can see, etc.
why not isAlias() so we can follow same semantics as with isRemote()
Okay, will implement that.
@steveklabnik: This should be "let's", right? Also, is it "changes" or "change"?
Whoops, that was a debugging line I left in when it wasn't building, because pdebuild erases the build dir when it fails, and you can't examine it.
I would change this to

```patch
+REBAR_BIN ?= $(shell which rebar || echo ./rebar)
```
`// roll back the setting value to notify the UIs that bluetooth has failed to enable.`
According to the platform people, this should already work or eventually work (and presumably not crash).  Let's call it, even if we crash.  If we do crash, we can file a bug against platform.  Please remove the comment about inline mode here.
Oh, I see. It is actually a <form> element and you're trying to prevent it from being submitted.  Where is the submit event coming from, do you know? I don't see any submit buttons...  Is it possible to just change the <form> to a <div>, or does the role="dialog" require it to be a form?

Its not a big deal, but my problem here is that you call getElementById() on line 37, then call addEventListener() on line 45, and then define the function here on line 262. This is a pretty trivial change, but it is spread out all over the code and there is no comment.  

I'd suggest doing it all in once spot with somethign like this:

// Don't let this form accidentally get submitted
document.getElementById('whatever').onsubmit = function handleSubmit(e) { e.preventDefault(); };
I like this module :)
`saveAlarm` ?
Actually the saved repeat modes doesn't apply to the single player, so I just used the static style "repeat-list" in HTML and set it to disabled. And I don't have to use the js code to disabled the buttons.
Let's add some spaces based on the indentation of the html.
Are you voluntary sneaking homescreen changes here? :)
immidiately -> immediately
That should not happen assuming you start the app with the gaia profile (after you created it via DEBUG=1) it turns off appcache completely.
window.setTimeout(cImporter.continue,0); work?
As you wish. You can open a thread and we can get some feedback, :)
can we please use a TargetDirectory and a @Rule with the method name instead?
Due to the way Java manages direct buffer, it's practically not easy to use direct buffers directly in most case, and thus it's not easy for a user to achieve zero-copy (impossible for most cases).  Therefore, I'd drop this section until we make some progress on this topic.
Why is the repo name HelloWorld here, while in DeploymentApis above, you use the test name (`DeploymentApisTest`) as the repo name?

Also, why do we need repo name at all? Can't we just get it by default from the github URL?
Looking at the implementation here, it's clear that this is a very dumb form of `pluralize` (far dumber than ActiveSupport's, for example).  It just appends an `s` if the count is not 1.  It's sufficient for RSpec's needs because the things it needs pluralized confirm to this simple logic (e.g. `second` => `seconds`) but it may not work for some cases an end user would use it.

Given that...it might be good to add a `@note` clarifying that it is not a general-purpose `pluralize`.

And actually, thinking about it some more: I'd prefer that we not declare these helper methods to be part of the public API.  They are useful internally for RSpec's usage, but I would like to feel free to refactor them w/o worrying about them being part of the official public API.  Maybe we can mark the module as `@private`?  I don't think there's anything here that end users would directly need anyway; even formatter authors could get by coding versions of these by hand if need be (particularly since they might have slightly different needs here).

@alindeman -- thoughts on this?
如果前面已经敲掉过都是侠客了。现在再说‘黑人侠客’不是重复了？
`.next()` and `.prev()` just return jquery objects, so they'll never be null—no need to guard them.
Might be time to extract this into a method, perhaps `unselectMessage`?
I think the default name should be settable. Also, I think it should contain the figure number so that you don't overwrite figures you just saved.

Could you add a `default_name` parameter to the function which defaults to 'untitled_%i'? Then the above line would change to:

```python
filename = default_name % fignum
```

Note, I'd prefer lowercase or lowercase with underscores to mixed case variables.
Sockets are created on the fly now and pushed onto "socks", see `connect()`.
looks like it got in fine haha, weird I'll check it out after work
This regular expression doesn't work for some reason.
Please doc this function and any others where you made significant changes.
Add a `Raises` section and a `Returns` section.
First: this code doesn't contain the information either (the are in [:resoruces] key, here are just some app metadata), second: that's a lot of data to be stored in this method.

It was put into the method for a reason, so that when somebody looks at the method, he directly sees what parameters to expect. The validation was just secondary. With this change, the generated code looses a lot of readability from the client user point of view.
Isn't this done automatically be ModelForm if you pass in the instance?
Rename analysisCol & analysisRow to col & row
I'm wondering how a nil is getting through here anyhow?
I tried to name the `buildbot.scripts.foo` module after the cli name, as well as the function in it, replacing `-` with `_`.  I'll double-check that I got that right.
[This](https://github.com/tomprince/buildbot/commit/b056bd417ed9eb51ae9160b85382d478fff15ab3) fixes the test. I wasn't going to fix this in 0.8.6, because this failure wasn't a regression, and it changes calling conventions (of admittedly non-public code).
It seems to me that filename is already containing full path (at least in command line mode), so adding env.paths is probably not needed here.
wanted to test something forgot to take it off...
ExpressionRewriter
This is just scope variable that indicates to exit handler how to return error. The error is formed only upon exit.
[DocumentCloud workspace](http://www.documentcloud.org/public/#search/)はBackboneのモデルとコレクションとしてDocuments、Projects、Notes、そしてAccountsを利用して、Backbone.jsで作られています。もし、成り立ちに興味があるならば、Underscore.js と Backbone.jsは元々はDocumentCloudで使われていたコードを独立したJavaScriptのライブラリとして抽出したものであることを記しておきます。
This ended up getting overwritten by the latest change to issues anyways, but the old style was way too big one way and not big enough the other way padding wise.
it should not return true if target dir already exists i think.
Thanks, but `@zeus = ` is still useless.
style nitpick: can you move this inside the else if? having a break before the else is a bit odd
please remove comments like "was 1.0 and is fine...."
Maybe: The addon reads JSON from `CFX_OPTIONS_FILE` and performs tasks described by it.
Nit: Not sure it's even valid markdown we usually use

```
## install-xpi
```
Indeed, looks like a missed case part of the renaming of `complete_transition` to `used_address_as_primary`
@warner - Test coverage is far from complete in several areas, it looks like dialog.js is one of those areas where tests are lacking.
Why not in application/routes.php?
--
Nathaniel Higgins
http://nath.is

-----Original Message-----
From: Claudio Albertin <notifications@github.com>
Date: Sun, 12 Aug 2012 22:26:30 
To: nathggns/Scaffold<Scaffold@noreply.github.com>
Reply-To: nathggns/Scaffold <reply@reply.github.com>
Cc: Nathaniel Higgins<nat@nath.is>
Subject: Re: [Scaffold] Router (#4)

> @@ -46,3 +46,6 @@
>   * Boot Scaffold
>   */
>  require(SYSTEM . 'bootstrap.php');
> +

Bootstrap then? I'm just unconfortable with doing (custom) routing in a file that's not in the application directory by default. It could make the impression as you're not supposed to create custom routes.

---
Reply to this email directly or view it on GitHub:
https://github.com/nathggns/Scaffold/pull/4/files#r1359645
Damn! I completely misunderstood Propel cs. A review is needed.
These error checks should really check that they are getting the expected error, otherwise these can end up covering up a real problem. So changing to something like

````ruby
expect { @provider.execute_changes }.to raise_error RuntimeError, /command 'setm' not supported/
````
This actually tests it with the existing tests.  I didn't want to add another config file b/c it'd change a lot of tests.  The tests will fail with this change, but without my change to cli.rb.  I certainly can write a discrete test for it if you'd prefer.
Is there a reason here to be repeating the test for multiple strings?
tooltipID
Might be a good idea to create a case class for cache stats.
Unless you extend the DB writer, I don't see how you could format individual fields with `$columnMap` and `$separator` as the only options.

I see a few options here:

 * Add a `$dateTimeFormat` option to the DB writer, and default that to `YYYY-MM-DD HH:MM:SS` or UTC timestamp.
 * Allow the writer to take a formatter service, which opens several possibilities:
    * Ignore `format()` (like FirePHP ignores the date time format methods) and make a new method that can apply transformations to the event data array before column mapping and insertion.
    * Ignore `format()` and make a new method that simply transforms a DateTime value into a string (all formatters can use this method).
    * Change `format()` to return a mixed value, and let the DB's formatter receive the event data array, apply transformations, and return an array.
Ok, please log the BC breaks in the README.md
no class doc block
This is bad code - rewrite
I think this should be indented too
This should probably happen after the schema and host are checked.
I'm in agreement -- let's just match the trait. If we want to change the trait, that's another story.
Add a line space between assignments and control structures.
CS: alignments again :)
Does InMemory cope with this case? Lots of code in it assumes id is unique as a convention
Hej, sprav prazdny `__init__.py` a to by malo fungovat. Tiez pritom zmen ten server.py na `from infoboard.daco import Trieda` (vid vyssie).
If parser->method is HTTP_GENERIC, you need to avoid checking matcher[parser->index] or REALLYLONGMETHODNAME will have you walking off the end of the array.
Ok, let's add a comment here then to explain why it is commented out.
yes, Python's default args are executed at import time (which is why you don't ever want to use mutable default args).
definitely don't need pdb in production.
pattern for imports can be read about here:http://mozweb.readthedocs.org/en/latest/coding.html#python
Should we first be trying to save the data in the old location, and only saving it in ~/.zipline/data if the old location isn't writable?
No, just something I randomly chose.
Just delete.
@ohadlevy, the generated query returns params[:id] = nil, since there are no reports where host_id IS NULL.  Thus, the JSON response is 

  "error": {
    "message": "Couldn't find Report without an ID"
  }

Were you concerned that it was returning the max report id for all hosts?

The error message is misleading since it's the params[:host_id] that is invalid.  I could remove .try(:id) and then I would get 

 "error": {
    "message": "Called id for nil, which would mistakenly be 4 -- if you really wanted the id of nil, use object_id"
  }
Seems like you're changing the fully qualified file name to a locally qualified one - what's the reason behind this?
Added a comment:


    /**
     * Advances the seed (as if the problem number had been advanced) without
     * actually changing the problem number. Caches how many seeds we've skipped
     * so that refreshing does not change the generated problem.
     * @param {number} num Number of times to advance the seed.
     */
I don't think this would be necessary if we marked the option as required in the Berkshelf::Config

```ruby
attribute 'ssl.verify',
  required: true,
  type: Boolean,
  default: true
```

Do you think this is a better approach?
I did not even know it exists. It was also in the partycraft project.
Reactions were pretty negative to my suggestion of an ``Image`` sub-class on the list a little while back (https://groups.google.com/d/topic/astropy-dev/ReCISxQJgQQ/discussion) but I guess this will be a good time to revive this.
Why do you have a space between the closing bracket and opening parenthesis, but not anywhere else? xD
@timoxley 

Probably something you'd want to do in a custom command (not via core). Assuming this PR isn't accepted, I will just continue to use a custom Makefile each time I create a new component (i.e. I generally don't do `npm install -g ...`).
Yes, it should be better, at least for readability. Can't remember if the order of the parameters is really important. Actually, I guess that the answer is no since the keys provided in `operation_definition.php`  are only there to control which parameters are passed through the workflow process (ie: `$process->attribute( 'parameter_list' );`)
Problem is that in some cases `self::$handlerInstance` will be null. That's why I left this.
i guess this needs to be reverted
OIC, is that ruby convention?

I always explicitly return because, um, well, this exact issue, heh.

Cuz of my old sk00l perl sk1llz: http://www.devshed.com/c/a/Perl/Returns-and-Perl-Subroutines/2/
i think there maybe an extra `</div>` that has no matching opening tag. double check
What is a `paginator`? Could we please call this `pagination` and `PaginationRegion`.
Wouldn't you also need has_time here? This is why I preferred to default to True and only specify that we don't have a time when we explicitly don't extract it from the string.
I think so too. The code is readable: it does things if the file exists, it's size isn't zero and it's size is smaller than a maximum size. That's not a complicated condition statement. I don't believe in arguments "it will turn massive over time" - if it needs to be changed at some point then it needs to be changed so that it makes sense AT THAT POINT. Preparing something that may or may not happen in the future and that can anyway be done well if it happens doesn't make any sense.
I thought so too, but Å Ä Ö is a mess. replaceHTMLCodes converted &Ouml; to an apostrophe (!). This was the "easy" way out, and since we don't need to do any string comparison it works as intended. 
But I am hoping (and guessing) that there must be a better way to do it, until then...

Btw: my other pull request includes this one as well as paging and "Rekommenderat", maybe we can close this pull?
Plus I think we should consider using `&&` instead of `and`. What do you think?
request is a 3rd party non-oae module
We've been using `c` for content as well. But then again, I wouldn't really know what else to use for it.
Perhaps `co` or `comm` ?
Might want to ask some feedback from others as I'm terrible at naming things.
How about turning the two cases into one:
```scala
case _: VMDeathEvent | _: VMDisconnectedEvent =>
  vmDisconnected()
  reply(false)
```
You should use the composer phar already available on travis: ``before_script: composer install --dev``
i wonder if it is not over-engineering to have an interface for this. i don't see why people would want to implement their own nested matcher that is different from the default one, and if they really need to why they do not just do it. do we use the interface anywhere else than in the NestedMatcher extends ... line?
Minor coding style comment: constructor / destructor before any other methods.

Also, fix the indentation.
It's not clear why `make test` would start up a server and `npm test` runs mocha in the CLI. At the end of the day, both commands are supposed to be doing the same thing.

I would personally prefer `make test` and `make test-browser`, or something along those lines.
It's ok. I know it's not useful, but it's explicit at least.
Your indentation here is incorrect. Scopes like "public:" and "private:" are only indented two spaces as they are not true code. This makes it easier to fit good method names, and comments into the file.
@jamesarosen Hmm, I commented to that effect from my phone, seemingly didn't make it to Github.
[![Build Status](https://secure.travis-ci.org/toopay/CodeIgniter.png)](http://travis-ci.org/toopay/CodeIgniter)
А я вижу - ибо использую ее в 2х классах.
Здравствуйте, мне назначили вашу работу на ревью.
По условию задачи метод mutate принимает параметр - процент мутации от 0 до 100% при длине ДНК = 100 все в порядке, а если длина 200?
Pardon, это не мой код. Я не автор.
а оно и не выполняется, а выпадает assert
Not sure why you're using strncasecmp here. Just use strcasecmp.
So that any subsequent dependencies for the component being built can be looked up in the same lookups directory. Does that make sense?
如果 dest 不是 None 就不发消息了？
Indenting here is a bit weird.
same here rather that if (result != undefined) use if (result) . and the same for result.jobs.
Remove the FIXME. This is expected and reasonable given the dialyzer options we specify.
I don't think this is a method that we need to worry about, it's more of an implementation detail here.  The methods that do matter are the Result.discardingCookies ones.  If you'd like, I could make these methods overloaded, so they support both discarding by name, and with the entire cookie, but if I did that I think I'd still like to mark it @deprecated, since it's a bug waiting to happen.
Maybe use:

<pre>
<code>
import Data.List (find)
...
param key = find ((==key) . paramKey) `liftM` params
</pre>
</code>
use `this.rowCount` instead throughout?
This was already part of the document, maybe it can be moved in a separate PR. Once this PR has been merged it will be a lot easier to split up documents into precise chapters.  I think it is very important that the component documentation is complete and that there is no confusion between the 'book' and the component docs, since the book refers to Symfony2 in the context of Symfony2 framework and so could be confusing.  Anyhow, I'll take care of it in a separate PR.
Hmm, in my experience, this isn't very realistic. My model class usually just knows the filename (e.g. foo.jpg), but doesn't know if files are generally stored in a directory (e.g. `uploads/images`).

Perhaps we can not use this, and instead - at the very least - put the directory (e.g. `uploads/images`) in the custom `file_widget` block?
take care. You used ``$id`` as the key in ``$attributes`` whereas the id is something else.

you receive an array because you can apply the same tag several times on the same service (generally with different attributes). This is useful for instance to [register a service as listener on several events](https://github.com/symfony/symfony/blob/2.0/src/Symfony/Bundle/FrameworkBundle/Resources/config/web.xml#L27). You can see the [corresponding compiler pass](https://github.com/symfony/symfony/blob/2.0/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/RegisterKernelListenersPass.php#L27)

The main reason for the confusion is using ``$attributes`` to name an array which is not the array of attributes.
定义当对使用调色板的PNG图像，或者GIF图像进行变换时，是否需要保留图像透明度。
Cool, removing
This is  crappy line, I don't need this up in my business.
as discussed above, this whole file should be converted into an e2e test
Fixed in d7dea8a.
This is also a regression.  This answer was already given in master (albiet in a different form):

```py
In [42]: atan(S.Half).evalf()
Out[42]: 0.463647609000806

In [43]: print solve((2*sin(x) - cos(x)), x)
[-2*atan(2 + sqrt(5)), -2*atan(-sqrt(5) + 2)]

In [44]: (-2*atan(2 + sqrt(5))).evalf()
Out[44]: -2.67794504458899

In [45]: (-2*atan(-sqrt(5) + 2)).evalf()
Out[45]: 0.463647609000806
```
Вот это охуенно, батя грит малаца.
"Sesión."

Sería interesante mantener los links generados con `@Html.ActionLink` o `@Url.Action`.
Guarda también que CodeBase puede ser remoto mientras que Location es siempre local.
Can you double check this one, because I have the same file that is in the link you specified here
Please use brackets, multiline alignements
Thats a good remark, will rework my implementation to cover that.
I guess that if I call next() after this, I'd get the first element?
Please fix the coding standards. The space should be between ``foreach`` and the brace, not inside the brace
if the services are marked private, they will be auto removed unless another service has a reference to them.
Nitpick: Only need a single new line between methods.
Needs removing. Its not this that's causing the problem though.
No -- there is no way to blit directly to the screen/window hardware so there is no advantage to that approach.  It could be faked, but I'm not sure why...
spacing again.. you get the idea, I won't comment on all of them
Nope, it will add the extra commits to your original pull request
automatically. Thanks for fixing.
On Jun 4, 2012 10:23 AM, "Brian Jacobel" <
reply@reply.github.com>
wrote:

> >
> > -    fieldView->updateView();
> > -}
> > +      buttonLayout->addWidget(stopButton);
> > +      connect(stopButton, SIGNAL(clicked()), this, SLOT(stopDrawing()));
> > +
> > +      dockWidget->setWidget(buttonWidget);
> > +      addDockWidget(Qt::RightDockWidgetArea, dockWidget);
> > +
> > +    }
> > +
> > +    void FieldViewer::stopDrawing(){
> > +      keepDrawing = false;
> > +       qDebug()<<"Stopping WorldView...";
> > +      bot_locs->locs->stopListening();
> > +    }
> >
>
> Yup, I fixed indentation and whitespace in all the files tracked in this
> commit and pushed again. Do I need to make another pull request?
>
> --
> Brian
>
>
> On Mon, Jun 4, 2012 at 12:05 AM, Lizzie Mamantov <
> reply@reply.github.com
> > wrote:
>
> > >
> > > -    fieldView->updateView();
> > > -}
> > > +      buttonLayout->addWidget(stopButton);
> > > +      connect(stopButton, SIGNAL(clicked()), this,
> SLOT(stopDrawing()));
> > > +
> > > +      dockWidget->setWidget(buttonWidget);
> > > +      addDockWidget(Qt::RightDockWidgetArea, dockWidget);
> > > +
> > > +    }
> > > +
> > > +    void FieldViewer::stopDrawing(){
> > > +      keepDrawing = false;
> > > +       qDebug()<<"Stopping WorldView...";
> > > +      bot_locs->locs->stopListening();
> > > +    }
> > >
> >
> > And these extra whitespaces etc... just style things...
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/northern-bites/nbites/pull/289/files#r920584
> >
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/northern-bites/nbites/pull/289/files#r922320
>
XHTML spec says any element with no content (`<input>`) should self terminate -- `<input id="phone" ... />`
The response status is usually capitalized. Perahsp

```php
Http::response(500, 'Server configuration error');
```
ah cool, makes sense.
@bp323 Can we use single quotes in these changed lines??

We probably should still use the double quotes within the html (but not escape them - no `\`)

e.g.
```javascript
'prev' : '<span><div class="sakai_pager_prev"></div>'
```

instead of
```javascript
"prev" : '<span><div class=\"sakai_pager_prev\"></div>'
```
Could you fix up the indentation here?
Esta parte do envelope não ficaria melhor como parte do renderer? A view deveria ter a responsabilidade de fazer a "cola" da camada HTTP com os objetos de domínio, e não de gerenciar os metadados do envelope opds.
that would actually be fine - do we know if webkit is in error here, or is that sanctioned behaviour?
Can you explain what this `:final` flag is?
I've opened #221 for further discussion of whether WiredTiger should force transactions to roll back, and if so, when.
Docblocks missing from a lot of methods.
위와 동일합니다.
Mark, just a detail, i think it would be good to use the lastest release (1.4.5), are there any issues on using it?
- Could you add a ``@return`` annotation?
- Is it possible to add a ``@param`` annotation for parameter ``$redirect`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
Are you sure ``$this->validator->getRecursive()`` has the correct type as "Boolean" is not a sub-type of "Boolean"?
I just realized that this could have be made simpler by simply doing :
```js
if (isBlackberry) {
    e.rowData = data[e.index];
}
```
I think that would work.  Doesn't have to be done though.
no, I'm making another PR to refactor these. but github is stuck... so I
can't make the PR now...
I will from this branch once github unsucks...
https://github.com/rantav/hubot-yammer/tree/opengraph-refactor

On Thu, Oct 18, 2012 at 10:01 PM, Aurélien Thieriot <
notifications@github.com> wrote:

> In src/yammer.coffee:
>
> > - reply: (user,yamText) ->
> > -   @resolving_user_id user, (user_id) =>
> > -      if user_id
> > -         params =
> > -            body          : yamText
> > -            direct_to_id  : user_id
> > -
> > -         console.log "reply message to #{user} with text #{params.body}"
> > -         @create_message params
> > + send: (user, yamText) ->
> > +   if user && user.thread_id
> > +     @reply user, yamText
> > +   else
> > +     og_url = null
> > +     og_fetch = false
> > +     urls = yamText.match /\bhttps?:\/\/[^ ]+/
>
> You finally keep these here? :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/athieriot/hubot-yammer/pull/10/files#r1883218>.
>
>



-- 
/Ran
http://tavory.com
What I'm doing right now is just using a small `setFilter` helper that recieves an element, regexp and filter string as a param.
It's just reusing the code we had inside `setOpacity` and makes code more readable and concise, which I think is enough of an improvement. I don't want to go overboard with a small block of code that is only for oldIE
I'll change that phrase
Das ist überflüssig, oder?
Double checked, and apparently you can specify the patch-level.

    ">= 1.9.2.180"
I have problem here because of inheritance i guess.
it fails on fetching data when at is null, so when saving precipitation at==null and vice versa. please help
What exactly are you passing in?
Sadly, I do not know of a better way to do this. But are you aware, that this does not escape in proper way? It escapes only HTML, not also JavaScript or even jQuery selector syntax.

For now, I would leave `escape`, but add TODO for other types of escaping.
I was thinking. It would be interesting to store all horoscope data, also for the past. I am interesting to see how much they change through time or if our provides are re-using them and so on. So override only horoscope of the same day. You know how to change above query to store horoscope every day?
I liked 2 minutes limit here. If you will leave one, change comment accordingly (`minutes` -> `minute`).
Hm, I thought we are using lower case. Just make it consistent. ;-)
But i excpet that i have probelm from "Oktober" -> "October" or python this know??
Do we really need Mechanize for this non-remote test? So far mechanize seems to be only used in `remote_integration_helper.rb`. Afaik it won't be loaded in non-remote-test environment either looking at the Gemfile.
Wakelocks are reference counted, but the calls to the LockManager appear to be framed as idempotent.  If so, shouldn't all attempts to acquire a lock first check to make sure it isn't already acquired?
executables already knows about the bin folder and is just a simple array of binary's you want added to the path: http://docs.rubygems.org/read/chapter/20#executables
there is an extra > after header here
In all of the *Descriptor classes, the transforming of a descriptor or optionset into query parameters is the responsibility of that class. This is breaking from that convention. Is there a reason `CMWebService` should be responsible for breaking `CMStoreOptions` down into a query string?
thoughts for future: I'm starting to think that these two methods ( _entityCacheSize_ and _associationCacheSize_ ) are too Key/Value oriented.

They were trivial to implement on Infinispan, but it's getting complex here.
I think we fixed this earlier, but worth checking that this isn't done via normal Class.forName and uses PigContext instead
Move these variables to the block where they are used.
I'd argue it lies at the heart of all design principles.  If all software development is design, then this assertion holds. Either way, we should decide and either remove or replace 'all'
Flurry was the only option we wanted to log?
please wrap lines in the doc (see above)
Make sure all public calls have validate()
Is there a naming convention for advice methods?  begin is pretty generic.  Perhaps applyBeginAdvice?
Again, the ALT tag is empty. Accessibility aside, this will cause errors with the validator.
а чем новый парсер get лучше текущего? если ничем, зачем пустые изменения?
It would be for efficiency reasons.  The copy constructors for XTable and KTable are deep copies, not shallow, and there are some significant concrete objects in them (_array, _cache, _xwt) that aren't trivial copies.
you set a variable called buttonCount...might as well use it.
The pp_packing module has disappeared from the "Previous topic"/"Next topic" links. NB. The page itself is still there, and it's still shown in the "Modules in this package" boxes.
en fait je prefere ma phrase ... j'ai enleve le so par contre
delete this change
GROSS.
grumble.  My branch wasn't synced to HEAD of the opscode repo when I applied my patch.  The section you're commenting on wasn't part of my change.  I will merge opscode-cookbooks/nagios/master and get rid of the stuff that's not part of my change.  Thanks for the review and catching that.
I just wanted to keep this method` logic more-less similar to his brother from get_indexed.
But if you really want me to make renaming I can do it.
You're right. I've stripped it back down to seeing if self[name] throws an exception or not.
That's strange. I just created a new user and it doesn't show the table if no notifications are present. Signing up for a notification, the table is displayed as expected. Not sure what could have caused the problem.
If you think i'm flirting i assume you like it. Man that's not cool, stop it.
I changed it, what about now.
added the vars for the 4 buttons (^,v,<,>)
it does, look in configure.py
i don't like it either, but was trying to keep it < 80 chars
Если исключения не произошло, файлы тоже необходимо закрыть.
Есть такой класс -- Pattern. С его помощью можно получить Matcher, у которого есть метод group(int). Таким образом можно будет написать одно регулярное выражение, из которого можно будет по группам получить хост и порт.
Название не по CC
the ranking weights have already been created and put into the rankedSolverBenchmarkMap, no need to create them again (and especially not n*n times :). try reusing rankedSolverBenchmarkMap somehow
source_paths は要素中の keys.first と values.first しか使っていないので、最初から source_path と project_type からなる要素のコレクションにしちゃうのがいいんじゃないかと思うんですが、どうですかねぃ
Yup. If insert fails due to constraints (meaning that we already have the given id in the unique column), then we know it's safe to update. Should be good on a performance level too, as it doesn't need to regenerate indices upon updating.
Was this a random occurrence?
I agree that this is unfortunate. Should I add a sentence that says "you see this warning because you passed a square matrix to fit without setting self.affinity==precomputed"?
no. atleast2d will make it transposed
Ok cool, I'll change that.

Agreed on the shorter settings version, it is kind of confusing when they're both the same name. I'll go with `SECRET_KEY` and `ACCESS_KEY` if that's alright.
Are all the click event properties currently supported?  If not the yml needs updating
should also have support for StringObject
Why?  I don't remember discussing it.
I kind of agree.  

However the code will need restructuring if that happens - because if you add the row before resizing the index it may not be large enough if a spill is triggered - so both need to reallocate before the row is actually appended.  This way round it is safe.
I'll refactor when it gets to 5....
While you are at it, some parentheses might help make it clear whether the HTSIZE_LIMIT_PC is on the numerator or denominator...

maxRowGuess = (rowidx_t) (maxRowGuess / 100.0) * HTSIZE_LIMIT_PC; 

perhaps
I'd split this out so it's easier to read, i.e. construct the request context, then ```return render_to_reponse('main.html', req_context)```
you can just do:
```
return dict((k, v) for (k, v) in dict_.items() if v not in (None, ''))
```
This might happen if we're unable to extract a event from the file, correct? That's going to keep on happening, isn't it? Do we need a new place to move files that couldn't be uploaded? like a "dead" location, for manual intervention?
Ahh, makes sense. IMO this is complicated enough that it'd be worth turning into a gem. Perhaps it could aim to support a simple syntax:

    add_multiversioned_dependency 'activerecord', '~> 3.0.0'
Yes that's a good point! I think panel should have: `contentScriptWhen: 'ready'` set so it's content script is executed once document's elements are accessible and then it can register keyup listener:

```js
var textArea = document.getElementById("edit-box");
textArea.addEventListener('keyup', function onkeyup(event) {
 // same code as in original listener
}, false);
self.port.on("show", function onShow() { textArea.focus(); });
```
I think now after adding connection strings the check for emulated can be straightforward instead of current approach. In PHP we detect if it's emulated by this line of code instead of using `EMULATED` env variable

    return (strpos($this->connectionString, Resources::USE_DEVELOPMENT_STORAGE_NAME) !== false);
css paths and when you need to use xpaths try not to use the full path. If anything changes along the path it will break the code.
place the selectbyvalue in a try catch.
pls correct mistake in method name
This is a weak default password. :bomb: :cry:
Ok. WIl make the change.
In this method you don't disconnect, seems like there is a lack of "standarization" about connects and disconnects around the module....
PEP8 please
These tests don't seem to catch the bug. (The bug is present on master, but the tests don't fail.) It seems that `bw_lena` is all True because the input image is uint8, but it's converted to binary as if it's a float image (i.e. `bw_lena = lena > 0.4`). Could you test with a higher threshold (e.g. `lena > 100`)?
看来好久不说英文了，回去看friends:P
看来我还只适应Ruby, Python 这样两个字节的单词，哈哈
It may be better to avoid saying “select” here since that might make the user think of Vim’s visual or select modes.

Maybe just delete the phrase “select the word and”.
ali kash ina ro ba less misakhti az aval
vali felan bikhiyal vaght gerefte mishe
I removed the comment. It should be done in render because the camera could be changed after initialized frame and the view matrix uniform, etc. would not be updated.
always true, even for iPhone, could be removed
Setting titleOn/Off to an empty string removes title text.
Fixed in next update.
It looks like this treats 'open a b c' the same as 'open "a b c"' which seems very wrong.
this should be done before merging
We check url of each link in view in order to show it
Хотя непонятно что ты тут имел ввиду :)
> Is that the case with `union`s too?

No, `union`s are aliens from another planet, not stack-allocated objects!
(sorry for a bit of sarcasm).

```D
extern(C) @system nothrow void* alloca(size_t size); 

alias long T; // alias whatever you want here

void f()
{
	T t;
	assert(cast(size_t) &t % T.alignof == 0);
}

void main()
{
	f(); // ok (but may fail)
	alloca(1);
	f(); // fails
}
```
Opps. It looks like some debugging code was left in. Fixed.
Exactly. `isDynamicArray!S` is really necessary to distinguish between static string types and dynamic ones.
D's built-in string is  *an array of some character type*. It does not guarantee anything more than that.
At least, appending an element to static array will cause compilation error. Then almost no problem.
That's supposed to be
ˋˋˋif (!decodeIsWhite(s, j)) break;ˋˋˋ
That's me trying to make incremental but comprehensive commits bitting me in the ass. I missed it durring my re-read :/

Yeah, I wrote a function "decodeIsWhite", which, as you suggest, starts by checking if the codepoint is ascii, and then takes it from there.

I proposed 2 diffferent version of ˋdecodeIsWhiteˋ (you can see them in the individual commits). The first is a simple and conservative function, which gives a a good boost. The second is ore involved, but gets even more boost.

I'm actually out of town, so I won't be able to address any of the code immediatly.
Probably unrelated line. )
also can pass $event
does it work if you use "https" url here?
Not `setup.Constants`; just a constant in this file is fine. Sorry for being unclear!
Y U NO `dd` THIS LINE?
Need to get a different icon for this.
Why the word "Actual" is necessary here?
Tiny, not really even a bug, but maybe worth a comment...

It's possible (though unlikely, and kind of broken) to have more than one version of
python-rhsm/subscription-manager installed. Versions() will only return one (and I
suspect it's not predictable which it will return).

But honestly, I don't think it's worth the extra code to handle that case, so leave it as
is (maybe add a comment)
Well, anyway, I believe that TonyHash should be successor of Hash.
Actually, since you put so much of the logic into the `ImportResult` class, is this `write_result` function really needed? Seems like it introduces more overhead than it's worth.
I agree with @pmjones, **readability > writability**.
:+1:

Forget my [comment](https://github.com/php-fig/fig-standards/pull/16/files#r533056).
Why do we need to set this as a globally accessible variable?
the `.get(...)` doesn't need to be split across two lines to fit within 80 chars, be easier to read if it wasn't.
After some discussion on IRC, I have another proposal: Drop `NullPointerError` entirely and just attach RIP and the memory location that was attempted accessed to `InvalidMemoryError` and call it a day. You'll get a nice exception whenever you do an invalid memory access, and you can trivially ask (both programmatically and in a debugger) whether it's a null access.

Everyone wins?
QUERY: Move this into a helper class in ign_core?
. . . to open 'state'?
what's the purpose of the above line?
3rd __retry__ = 4th attempt
Missing priority parameter - sendRequestMessage below always uses the priority field instead of the priority derived from the message (if present).

Given that sendRequestMessage is called from lots of places now, we need tests to ensure that explicitQos works in all places (would have caught this problem).
The other chapters are referred to only by their id, without the '.rst' suffix. Can you please review?
Or maybe:
```
:ref:`SMTP transport <zend.mail.transport.quick-start.smtp-usage>`
```
(for internal links?)
`stand` or `self.stand`?
Avoid memory leaks by unsubscribing from Redis
I suggest you put these settings either all the way at the top (they're constants, after all) or in a separate file.

Since this is an example API, though, you can probably keep it inside this file.
I mentioned the mutability problems with returning arrays of doubles before.  What was the verdict on that?
Per Jay, the logic around what window to show should be in the presenter.  Opened https://sagebionetworks.jira.com/browse/SWC-272 for this.
Ahhh! Sorry for that! I'll fix it now.
@fannix: fixed in db71fb40c469bfae0a51267b2f187782c58195d5.
changed Group to group
Why a random one ?
These tokens are generated on the server and need to be copied from there.

Shouldn't it just be blank ?
Posso inclusive comentar as linhas que alterei, mas eu queria mesmo era comentar as linhas de vocês...
Not available on IOS
Чейнинг - это очень хорошо. Ты половину задания 4 пары уже выполнил ;-)
I think you should wrap in a try/catch and call fail if there's an error
Fixed with the addition of licenses to the base files
I think he just means that Citibank's cards are issued by MC rather than Visa.
Ill give it a try