labofsahil/github-event-dataset-2012 huggingface dataset:
No, its an array of DumperCollection and Dumper Route instances. Not an array of DumperCollection or an array of DumperRoute.
Its also supported: see http://www.phpdoc.org/docs/latest/for-users/types.html?highlight=array
capybara-webkit は入れないんですか？
Is this a hard coded date?
Where is this deliver method defined?

Ah, it's a *private* method in the RolePrivateMessenger?

Feels weird to have this module knowing about private methods in another class. I get why, since it's mixed in, but I wouldn't expect my private methods to couple outside my class.

Objection registered. Carry on.
这个结构和目前结构无较大优势呀，无论性能还是写法上面。虽然分了区块，但是还要自运行，还要return function，也不见得比现在清晰，多是习惯问题。^_^
It is unrelated change, must be as separated pull request.

Common practice (maven and flexmojos flex config generator in flex do it) – if plugin produce or change virtual file content, it should force update it, so, user don't need to call "Sync" or wait auto file sync. May be, we need to update only out file, but not all out dir.
Was never initialized.
Also, perhaps too much rhetoric here for what is ostensible an academic paper?
you need to add `htmlContent = htmlContent.replace(/\x1b\[K/g, '');`
`build_string`
This could probably be simplified out to:
```python
if getattr(self, attr, None) is None:
```
We're using 4 spaces in the rest of the file
I thought about refactoring it to make it more testable, but I couldn't think of a better way that I wasn't sure wouldn't break it. So I left it for now, but it might be worth some investigation later, but probably after the move to Express 3. That might clean some of it up.
Why 'Բաբգեն  (Բաբկեն)'? Maybe 'Բաբգեն', 'Բաբկեն'?
That log message won't work as expected either. This is using a mix of the ```java.util.Formatter``` and ```java.text.MessageFormatter```. As @bstansberry this needs to be internationalized, but if it was a debug or trace message you'd want to use something like the following: ```log.debugf("Cannot delete cache %s %s, will be deleted on exit", file.isDirectory() ? "directory" : "file", file.getName());```
Needs to be implemented.
(restore the trailing new line at EOF)
I don't think this works accurately, since plugDJ sometimes returns the DJ map in random order.
Sounds smart to split up the namespace a bit :-)
Could you add type="text/javascript"? I forgot that :)
I believe you would need to 

    notification.removeEventListener('tap', captiveNotification_onTap);
    captiveNotification_onTap = null;
    NotificationScreen.removeNotification(id);

within this function.
If you don't want people to change that directly, you just need to do a getter that return an internal value of your class.
is this really working? where is it shown the dialog?
With asyncStorage instead of localStorage, you don't even need the JSON.stringify().  You can just store your object or array directly, and get it back with getItem() without calling JSON.parse()
That's annoying that you have to do that.  It sounds like a bug in the <audio> element to me.  Maybe ask on IRC in the #media channel if anyone knows about that?  

Why not just call the variable 'conversion' rather than having to write a comment explaining the name? :-)
Could it be a setting? (can be done in a followup)
Correction, should be `this.time.textContent = d.toLocaleFormat('%I:%M');`
Why are these commented out? We should either remove them entirely, or add a line at the beginning explaining why they're commented out.
two-space indentation everywhere, please
escape also
Let's remove the test?
} else
Why are we removing box-shadow?
Do you really need this onchange function? I think `settings.js` can handle all checkboxes with a proper `name`.
I'm gonna removed things not needed here.
containter -> container
We should follow how this is done for the UID search criteria because criteria can be more than one sequence number or range.
I though I would add this section as a "best off" and concise summary of our StackOverflow.

I'll add the link at the top of the page.
@albanlv you haven't closed the `p` tag
I tried that initially, but it wouldn't trigger the load event.
Yeah @filipewl I can do this, but I agree with you, I think it's not necessary this element be a link
`` ではなく <code> たぐ
これらのツールのいずれかは、あなたのネットワークの置かれたノードに関する情報を握っているツールかもしれません。
そこで、信頼されたツールへのインターフェイスを作ることと、そしてそれを Rundeck URL リソースモデルソースとして見せることが一番です。これは（ノード情報を管理している）tool のフォーマットから Rundeck のフォーマットへ変換するシンプルな CGI スクリプトとして実現できます。
↓
これらのツールは、ネットワーク内に配置されたノードの情報に関して認証を考慮しているでしょう。
従って、認証ツールへのインタフェースをつくって Rundeck の URL リソースモデルソースとするのがベストです。
ツールのフォーマットから Rundeck が理解可能なフォーマットへと変更するシンプルな CGI スクリプトで実現できます。
`deltaX` and `deltaY`

Is `cx` from Raphael?
9 ist faster than 10, so the ghost should be saved, shouldn´t it?
Should be: "Columns to query over"?
``` coffee-script
href = link.replace /;show:.+$/, ''
```
Do you think it will?
it seems that this would allow setting a property referenced by an empty string:

```html
data-bind=".oops"
```

which will try to generate a structure of:

```javascript
model._data[ '' ] = { oops : 'value' };
```

This is not a bad thing in itself, and perhaps some future convention can have special meaning for variables ".myFoo" as a result of that, just thinking out loud here.
Idea:  Turn debugdir into a property, then no function call is necessary, simply assigning to the attribute executes the code.
Alright, that was just a nitpick anyway and I don't have a strong preference. If "Beancounter" is the accepted standard name for these values, let's leave it as it.
I thought about using ports for the reader and writer monads, but I agree that what you've done is a better way to go about it. There should be a completely separate monad for consuming/producing with ports, should that be desirable to anyone.
前の値が異なる場合に新しい値をその変更を記録し、そうでない場合はこの属性への変更を削除する。
Ack! Yes, of course.
seems to be unused apart from being set?
Most tools handle this with a special output filename of '-'.
I think we need to either set this to the string "5" or typecast loglevel to a string on line 150 below.
Seems like lots of changes are required to the validator for yaml support

How about instead refactoring to separate validation from coercion?
I couldn't include the weiqi models in my test's as it was. I think the old way was just a bug right? 

However, I don't understand how it ever could have worked as it was.
refactor this long line in a method
Either way it's not something relevant here and let's keep it as is.
I just did not wanted to change name to avoid bigger diffs that are harder to review. But I can do that if it helps. Would you be happy with following ? 

```js

const Require = lambda(function Require(loader, module) {
  let base = module.path;
  .....
```
I'm sure you did some mistake you definitely can set function prototype (not redefine though) unless it's frozen. This works

```js
function foo() {}
foo.prototype = undefined

typeof foo.prototype
```

As of `prototype` argument it's undefined doing exactly what I want ;) But yeah I can remove it, it's just at first I thought I'd non `undefined`  prototype somewhere.
readability win!
@lloyd - Should this comment be updated before merge?
I don't fully know what the IDs are needed for, I just pulled the generation of the ID from out of the template code and into here.
If you want to add a comment for a method please JSDoc complient style '/** ... */'
As I said, properties are immutable. It will not be re-set by the following targets.
No worries @zacc - reflecting back what I primarily trying to get at is that method is returning a <code>MediaPage</code>. I felt that a <code>click</code> didn't convey that an instantiated <code>MediaPage</code> is what is being returned.
Because `setSchema` is already taken... That was the name I chose at start but it conflitcts with the already defined `setSchema` method in the upper `ScopedElement` class...
Ugh, `WIN32OLE` implements `.each` but doesn't mixin `Enumerable`?! That's dumb…
Shouldn't this, also, be an array of options?
Not a big fan of having an "exit" in the middle of a module definition...
In this usage, the `subview` name does now work as well! :unamused:

I wonder, though, if combining these two uses (creating subviews, styling/modifying existing subviews) belongs in the same method... not opposed to it, but giving it some second thought.
There are two commits. You can just merge the first one. But really you should not be messing with the installed set of gems without the knowledge of the user.
my bad, no clue why i re-assigned `re`:

```js
this.options.grep = 'string' == typeof re
  ? new RegExp(re)
  : re;
```
Why 10 here? Please use constant. No magic numbers
Sry it should be ... if (version_compare(\PHP_VERSION, '5.3.7', '>=')) { ...
put spaces around `=`
switch these two lines around.
Should be `master`
Do some of these need to be private member variables? They're only used in one place.
Good changes all around through here...except when we talked about it I plum forgot about the forced passing of an entity to the super.

In some ways, I think the core of events need to be heavily looked at. We are aiming for a "Entity component" model and this goes against it

Thoughts @W1ndwaker ?
Yeah it can, basically canTick is just structured like this
```java
if (canTick()) {
   tick();
}
```

every tick
Can you call it `active` instead of `left`? for consistency
Yes this could be done but it is not the way Maria is intended to be used. This is mostly due to the fact that I do not think it is a good idea to put templates in <script> elements in the HTML.
Korjattu: Näytetään vain merkitykselliset valinnat. Tekstejä myös muutettu.
Our PHP standards imply curly braces to be opened on a new line here.
you don't need ``$m`` here as you don't use it.
And you still need to escape ``/`` in ``$filter``. Changing the regex deleimiter does not remove the need to quote it. It only changes the character that needs quoting
I am not sure that we should call `expand` here, since it can lead to the execution of custom macros. If we want to keep this around, we should at least ensure we have a clean env and not the one given by `__ENV__`. For example, imagine this code:

    defmodule MyModule do
      use Another

      def check_valid_terms?(terms) do
        Macro.term?(quoted, __ENV__)
      end
    end

The `__ENV__` in the code above allows the execution of any macro in `Another`, since `Another` was used in that context. So, a developer could accidentally expose a macro in it. I believe in general it would be better to whitelist what we expect from `expand`. So instead of calling `expand`, we support `:__aliases__` and `:access` directly.
this is actyally empty?
Outline docstring. Maybe put it on a single line?
Wil this line ever be reached? log.exception raises an Exception...
I try to remove it, but I have to modify
https://github.com/bpaquet/http-parser/blob/master/http_parser.c#L2022.
I think the code is much cleaner with this state.

If you have a proper solution, I'm ok to implement.

On Tue, Jul 24, 2012 at 7:13 PM, Peter Griess <
reply@reply.github.com
> wrote:

> > @@ -1938,6 +1911,144 @@ const char * http_method_str (enum http_method m)
> >    return http_strerror_tab[err].description;
> >  }
> >
> > +static enum http_host_state
> > +http_parse_host_char(enum http_host_state s, const char ch) {
> > +  switch(s) {
> > +    case s_http_userinfo:
> > +    case s_http_userinfo_start:
> > +      if (ch == '@') {
>
> Why do we need the `s_http_userinfo_start` state at all? It sees to behave
> the same as `s_http_userinfo`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/joyent/http-parser/pull/118/files#r1227343
>
That's the beginning of this function: https://github.com/zalun/FlightDeck/blob/81cb8cbf7153d6a45d5af4db0e7cf491baa9a4b1/apps/amo/authentication.py#L37
We could make `form` a convention--but it's just bigger than was worth tackling right here. File a bug?
this changed.
Sure, I'll change it to something like that which is more concise.  The only thing is that we will lose the typeof check to make sure the callback is a function -- now we will just crash.  I don't really know why anything other than a function would be passed as a listener, but since it was there before I want to make sure it's OK to pull it.
Isn't it sufficient to simply determine the request method?  For example, using the following:

flowRequestContext.request.method

I can't imagine a case where there would be a POST response to anything but a POST request.
Scratch this - it doesn't need to be here.
Me too - wasn't sure your intentions.  Happy to collapse them both.
I might be in favor of switching these two lines.
so, I guess this should read:

// Create topic handle for the topic '/chatter' of type std_msgs/String.
var chatterTopic = new ros.Topic({

?
Take a look at http://guides.rubyonrails.org/migrations.html
This line allows an instance of MMCIF2Dict to behave exactly like a dict.
Maybe add a comment here: CORS headers needed because HTTP origins are both host and port specific, and the Ares IDE server and the filesystem service are running on different ports. (also, test to make sure that's actually true?)
The changes in this file is probably not needed, if the Persistence object is changed instead to truncate the value on set then this is not needed and we make sure the value in the db is not to long.
Hey Bob, this get attribute would be better placed as a property at about line 231 and then we can re-use it.
You can use `home_page.is_the_current_page` here which is pre-set with the details to assert that we are on the homepage. it shoudl be more robust as it's shared code.
These are all strings and should have double quotes.
this needs correcting
Maybe if we can be sure that we are doing things in the right way, we oould do:
self.deleteLater(self._single_file_watcher)

And if we are afraid of threads, we can also connect to the SIGNAL("destroyed(QObject* obj)"), and in the method (slot) where we receive that signal we can do the: self._single_file_watcher = None

What do you think?
`(not (nil? ...))` seems redundant here. `(to-date-time x)` should return a truthy value already.
According to http://stackoverflow.com/questions/7163061/is-there-a-require-for-json-in-node-js, you can just use require to load the json file, rather than having to read the file and parse the JSON. I'll create a follow up ticket to make this change.
Yes, absolute. I switched to _canManage instead.
If we remove this, what would be the version of the sbt library?
avoid breaks in loops.
про ООП я имел в виду не это замечание, да тут ни причем. 
В данном примере есть предрасположение для решения задачи в общем виде, поэтому логично было бы сделать все до конца. 
несомненно 100 совпадает с процентами, а если мы поменяем длину днк до 10.000 ? тогда 100% мутация даст лишь 10%.
Опять не правильно. Максимальное значение процентов - 100, и оно никак не должно быть завязано на [DNA count]!
Дим, удаляй такую фигню)
extra space after `if (` should be removed
commit name - dorkster?
Agree with @jonstorer
Can you improve the formatting here?  Thanks!
Commands that use `robot.respond` should be prefixed with hubot, ie:

    hubot klout <twitter name without @>

You'll also want some description of the command after this. That way, help will show something useful if someone says `hubot help klout`, ie:

    hubot klout <twitter name without @> - report klout score for twitter handle, and compare to competitors
No, right now I only touched the interface between KPV and K2pdfopt. Most bugs I fixed in k2pdfopt.c were in this part. So if I fixed the memory leak bug that originates the upstream code I would like to send it to upstream.
Why does the `MediaFragment` need to know that the logout dialog is showing?
Cool. Added this back in.
PS: I was thinking about commenting on this too.. but then I felt bad about nitpicking too much ;)
In this part of the code in the original file, it's assumed rng has already been converted to a np.random.mtrand.RandomState variable from what used to be an integer (or None or whatever)---note that the signs are pulled from calling rng.rand(). I just made it so that if rng is an integer, it should be converted to a flip/no-flip variable. I'll change it to a string, instead, so it's clearer.
I verified this exclusions were superfluous.
"At times, the system administrator may need to wrote code"?
Would `flat_map` work in this situation?
As long as you're fixing whitespace you could fix it around the hash rockets on the above two lines.
I don't think these tests are organized correctly. IMO there should be one describe called "Cross-browser salsita.* APIs should invoke the corresponding native chrome.* API". Then you will have:

it("should call content-accessible scripts directly")
it("should dispatch non-content-accessible scripts to the background window")
it("should invoke the callback, if any, for APIs dispatched to the background")

In beforeEach() you can set up the dummy chrome API, and in afterEach you can clean up (IMO just "delete chrome" will be enough).
the log dir is ``app/logs/``, not ``app/``
This is wrong. ``fingers_crossed`` is the valid type whereas ``fingerscrossed`` will fail
We should show what this template might look like.
`Doctrine's`
I would remove the last "bundle", as the name already ends with "Bundle". It reads strange :)
第一个逗号是句号。第一个句号是逗号。
回复是响应。
它和该模块换个位置
如何在Win32平台上使用Visual C编译nginx
加句号
从单个IP地址过来
=>
来自单个IP地址
超过的
=>
过速的

请求数
=>
过速请求数
What possible benefit is there from having this as an abstract base class compared to making in a object?
window.__coverage__, same below
We should have a method in the WikiManager API to get the list of templates, instead of iterating over all the wikis.
Yes - I'll fix it.
why x?
changed line to 

"case NonFatal(e) => log.warning("Safe operation failed. Swallowing exception [{}]", e)"

Is that what you meant?
Не путай, блядь, NSM и UAF (uplaod & forget)
То, что апи не предоставляет возможности узнать расшарен или нет файл
Move this to `package edu.nyu.library.citation.util;` and change name to AuthorNamer or CreatorNamer.
Sorry, I'll fix it soon (I had copied skeleton of dialog from export tasks
:D).

2012/6/13 SammysHP <
reply@reply.github.com
>

> > @@ -2442,4 +2457,61 @@ public static void startActivityGuid(final
> Context context, final String guid, f
> >          cacheIntent.putExtra("name", cacheName);
> >          context.startActivity(cacheIntent);
> >      }
> > +
> > +    /**
> > +     * A dialog to allow the user to set options for the export.
>
> Export?!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cgeo/c-geo-opensource/pull/1778/files#r974495
>



-- 
Ondøej Kunc
@dkubb @mbj @snusnu I clone mapper class (which just uses Mapper#from(self) to make sure the same mapper setup will be used and inject custom session behavior. I imagine we can use session finalizer that builds those classes and registers them for runtime usage (just like we do with relationship mappers).
Good idea. If the user isn't specified in qemu.conf, then I'll look at the
owner of /var/lib/libvirt/qemu

On Wed, Sep 12, 2012 at 10:58 AM, Andres Lagar-Cavilla <
notifications@github.com> wrote:

> In packagers/deb/nova-compute-gridcentric/DEBIAN/postinst:
>
> > +        chown(os.path.join(mount.path, 'vms'), uid, gid)
> > +        if mode != None:
> > +            chmod(os.path.join(mount.path, 'vms'), mode & 0666)
> > +        chown(mount.path, uid, gid)
> > +        if mode != None:
> > +            chmod(mount.path, mode)
> > +    except OSError, e:
> > +        # If the mountpoint doesn't exist or it isn't mounted, that's okay.
> > +        # We don't want the package installation to fail because somebody
> > +        # has bad /etc/fstab entries.
> > +        if e.errno != errno.ENOENT:
> > +            raise
> > +
> > +def main():
> > +    try:
> > +        passwd = pwd.getpwnam('libvirt-qemu')
>
> /etc/libvirt/qemu.conf is rarely a source of useful information. But
> looking at specific files would be optimal. E.g. /var/lib/libvirt/qemu
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gridcentric/openstack/pull/144/files#r1587831>.
>
>
How many spaces in indent? Remember that we use 3 space indent (see our contribution guide...)
If you are on a case-insensitive filesystem, you can do it by asking git to move the file twice by using a tmp name:

```
git mv JSMinFiler.php tmp
git mv tmp JsMinFilter.php
```

And if you choose a lowercase ``s``, please rename the test file to be consistent
I specifically intended to show multiple ways to create a 3d axes.  Having a 3d axes as a part of subplots was a new feature for v1.0.
These attributes should ideally have a sphinx style docstring. i.e.:

```
self.sing_tol = 1e6
"""If the gradient is bigger than this, we say it has a singularity"""
```
So are you trying to blame me for something horrible, or are you just selling yourself short? ;)
cleanUp is run on setUp and soon tearDown too. It will ensure tests start in a known state. I forgot that we'll also need to remove this from setUp in the dialect test.

Dave Hunt

On 5 Dec 2012, at 18:21, Schalk Neethling <notifications@github.com> wrote:

> In gaiatest/gaia_test.py:
> 
> > @@ -178,6 +178,9 @@ def setUp(self):
> >          self.apps = GaiaApps(self.marionette)
> >          self.data_layer = GaiaData(self.marionette)
> >  
> > +        # Disable sound completely
> > +        self.data_layer.set_volume(0)
> But will this then still ensure the sound is off when tests are run? Seems like it will only turn the sound off at the end of a test or set of test runs.
> 
> —
> Reply to this email directly or view it on GitHub.
>
you need to restore this locator because it's used below.

nb this technique of locating probably won't be used in the future but at the moment we are just trying to keep the test layout the same and migrate to webdriver :)
for the problematic sentence, i think there is no "correct translation" but from what i understood i would trasnlate it like 
"une des clef pour bien administer un projet est de réaliser qu'ecire du code pour soit meme est une mauvaise idée. Si des milliers de gens utilisent votre code, essayer d'écrire votre code le plus clairement possible, et de ne pas essayer de "faire le malin" avec une syntaxe exotique", the idea behidind this last sentence is to emphasis on the previous one, but the form itself is hard to translate as is .
partially match a URI identifier.
Consider adding an optional space between 'url' and optional ' or ".
This regex doesn't match any absolute link.
But good catch man - I'd completely forgotten why they exist.


On Wed, Oct 17, 2012, at 04:10 PM, Sam Ritchie wrote:

  In test/clj/forma/hadoop/jobs/forma_test.clj:
> @@ -151,3 +151,13 @@
>   2            [[4]]
>   0            [[2]]
>   50           [[52]])
> +
> +(fact
> +  "Check `unnest-series`"
> +  (unnest-series [[[1 2 3]]]) => [1 2 3]
> +  (unnest-series [1 2 3]) => [1 2 3])
> +
> +(fact
> +  "Check `unnest-all`"
> +  (unnest-all [[1 2 3]] [[4 5 6]] [7 8 9]) => [[1 2 3] [4 5 6] [7 8 9]])

  isn't this just map flatten?

  —
  Reply to this email directly or [1]view it on GitHub.
  [RkOat9Y1_FdXL0zywX1whpiVA_-tduUyZW7h3XwfS37sIWahrWHoTFgF1DTiGA4g.gi
  f]

References

1. https://github.com/reddmetrics/forma-clj/pull/244/files#r1873381
Unfortunately, we forced to set timeout as an optional parameter because we have the optional parameter agent_ids before. We can switch them and use the following definition:

def query_tags(tags, timeout, agent_ids=nil)

But in this case we need to modify all existing requests of query_tags.

I was thinking about it, but I didn't find the better solution.
Absolutely, The goal was to replace the old lists once this is done.
Can you expand this `if` to multiple lines to make it conform to the guidelines?
if (titles)
    delete titles

but usually we dont want to use delete but SG_FREE
On the other hand, using a different format would make it obvious which were ours and which were their's. But given our css selectors are also lower case dashed, it does make sense to leep these lower case dashed too.
Already merged, but good point.

I was thinking I'd like devs to understand how to set up a `sample.env` file checked into version control and it's relation to `.env` (which should not be).

Copying it for them risks not learning that, I feel.

But open to discussion.
Hi Keith,

> We could put a flag in the txn structure that we're about to return WT_DEADLOCK, and go ahead and release our locks so other things can proceed.
> 
> Then, we return WT_DEADLOCK on every cursor op until rollback is called, and if you call commit, we roll it back anyway.

I honestly don't see the point.  There is no problem with an application continuing a transaction after an update conflict -- it's slightly weird, but it certainly isn't fatal.  And if the application does roll back immediately, then this extra checking doesn't buy anything.

Michael.
Should the type for parameter ``$callback`` not be "callable|null" instead of "callable"?
I'll change it, just wanted to make sure I understood the changes :)
I've added TODO's in the code so it wont be forgotten.
Synonyms are great and all, but when you run out of them it probably means you've run out of things to say about that thing. My points was as much to stop beating us over the head with mistyness as it was to find different words for it.
"parameterized"
There's no way to do it that's built in to Jekyll's markdown engine. We could post-process it before we serve it, but that would require re-parsing the HTML for every request. We could also post-process Jekyll's output statically, but that would require some icky monkey-patching.
Mockery is required for tests
This goes broke if the `pred` function does not return actual boolean values. In this case, an alternative to `true?` would be `boolean`:
~~~clojure
(not (empty? (filter boolean (map pred a-seq))))
~~~
But this does exactly the same as:
~~~clojure
(not (empty? (filter pred a-seq)))
~~~
This is the same as:
~~~clojure
(not (empty? (filter pred a-seq)))
~~~
This won't work if `pred` returns `true` for falsey values, like `nil?` does:
~~~clojure
(boolean (first (filter nil? [1 nil 2])))
;=> (boolean (first [nil]))
;=> (boolean nil)
;=> false
~~~
This can be fixed by changing the `boolean` to `pred`. But then we also need to check whether the sequence is empty:
~~~clojure
(and (not-empty a-seq)
     (pred (first (filter pred a-seq))))
~~~
I tried doing something like

```
if env['uri']
  MongoMapper.connection = Mongo::MongoClient.from_uri(env['uri'], options)
  MongoMapper.database = MongoMapper.connection.db
else
  ...
```

However the tests seem to fail. Sadly, I don't have the time to implement this...
I’d prefer `ln -s /Applications/Xcode.app/Contents/Applications/iPhone\ Simulator.app ~/Applications/iOS\ Simulator.app` (i.e. have the symbolic link be named `iOS Simulator.app` instead of `iPhone Simulator.app`). Will change as I merge this in… Thanks!
The `-f` and `-o` options should be consistent, so I think this should be `:annotations`
Probably it's better that way, I'll call key and secret in authentication process then. Fixed
Ok, but view functions will be very complicated :S
It also does not work. I am getting:

```
TypeError: navigator.id is undefined

navigator.id.logout();
```
I know, but all of these are settings variables. I need BIG hint here. Do I have to change TemplateView to DetailView in HomeView class?
Ehm. Missing some more code here?
NO. Don't assign event handlers as this. And even don't create DOM elements as strings. This should be written as:

```
$('<a/>').addClass('close').click(close_advanced_map)
```
*le odgovori na ta e-mail* -> *lahko odgovoriš kar na ta e-mail*
Create a method on user document `name`, returning the name of the user, trying to combine first name and last name, or displaying only one if the other is not defined.
So please provide exactly what you have for your documentation definitions, what you try, why you get, what you expect.
Why does this have a reference to console.InMemory...
Was this pulled out for testing?
Right now TortoiseGit leaves hundreds/thousands of temp-files behind on normal use - not nice. So I think we should do this (we could increase the time to one week maybe). If files are still open, we do not delete them.

TortoiseSVN does the same w/o any users reporting problems.
Because this is something utterly boring and tiring to work on ?...
I think I like that we added this bit of logic as well, but is there a test for it?
This is slightly different than what I was proposing, but I think it'll work. Originally I was thinking that ``short_name`` would only be used when you wanted to display a shorter name of a business. I think ``display_name`` implies it'll be used everywhere it's displayed. With this method, I'd suggest using ``display_name`` as an optional field and defaulting to ``name`` when it's not entered, since it's ``blank=True`` anyways. This way we can avoid the migration to pre-populate the field too.
Context should be wrapper as RequestContext according to django docs: 
https://docs.djangoproject.com/en/dev/ref/templates/api/#subclassing-context-requestcontext

```context = RequestContext(self.request, context)```

This will propagate values also form various context tamplate processors.
Same here, as above
I don't agree. Not using the storage would be encouraging people to store the full object in session.
This change is better, but I like the `time_ago_in_words` method more than it's alias `distance_of_time_in_words_to_now` (It's just a bit shorter & nicer)
Ok finally nailed it, I was missing a `$r->handler('modperl');` statement.
how do we deal with something like pig using several of these in one job? the class conf values will clash.
"these" I guess
ah, i was probably looking at a different pinned version of bidpom
We should use triple double quotes for docstrings http://www.python.org/dev/peps/pep-0257/
It will be called from SpaceStation:: in #1391:)
SystemBody is a good place:).
Where is `locale` defined here?  I don't see any way that the body of this block will ever be executed.  Also, only one instance of `TwitterCldr::Shared::Resources` is ever created... I guess maybe I'm missing the point of this change.
Here I'd rather explicitly check if the package is selected for installation. Patch is basically just a dependency list of needed versions, it might contain several packages, some of them might be already manually updated so the patch might be half-installed.
Please name function in a more verbose manner. `fb` does not tell anything.
we only want to delete the unhandled message if the flag is set.. how about:

        handledSuccessfully = handler.handleMessage((MimeMessage) message);
        
        if (handledSuccessfully) {
            MailUtils.setDeletedQuietly(mimeMessage);            
        } 
        
        if (!handledSuccessfully && deleteUnhandledMsgs) {
            LOG.warn("Deleting unhandled message (check events and logs for more info)");
            MailUtils.setDeletedQuietly(mimeMessage);
        }
On Jul 4, 2012, at 12:12 PM, Mike Jarvis wrote:

>> @@ -18,42 +18,35 @@
>>      *
>>      * Convolve one, two, three or more SBProfiles together.
>>      *
>> -     * The profiles to be convolved may be provided either as the first 1, 2, or 3
>> -     * parameters in the constructor, or as a std::list<SBProfile*>.
>> +     * The profiles to be convolved may be provided either as the first 1, 2, or 3 parameters in the
>> +     * constructor, or as a std::list<SBProfile*>.
> 
> I actually removed the constructor that only took 1 argument, since it seemed pretty silly without the add() function to add additional profiles.  And the list argument is now `std::list<SBProfile>`  (no pointer).

Fixed and pushed.
these three params doesn't exist in method (code was copied), so please update description :D
I wanted to do that but defaultVal is a string, so ``defaultVal=Integer.MAX_VALUE`` doesn't compile; and neither does ``defaultVal=""+Integer.MAX_VALUE``. I could have declared my own constant and referenced that, but didn't seem worth it.
This line would make more sense as 
```
return !_.isString(string) ? '' : ('' + string).replace(htmlEscaper, function(match) {
```
Another `:locals` argument. Always use `@page.new_record?` to chose the button label, within the partial.
This looks like a breaking change for clients that handle HTTP 204 themselves.

In the PHP SDK, JobNotReadyException is a subclass of HttpException, which you might consider doing here, since it would preserve backward compatibility.
these are also ok. It was my error before. Sorry.
indentation isn't aligned now
splashpanel文件没有上传上来。
Yeah I don't mind, either are fine, but I thought Symfony used the latter. Just checked and the former is not used at all and the latter is only used in 4 tests, so it doesn't really matter.
withjs est tjrs défini non ?

:hand:  retire les lignes vides stp :facepunch:
Just noting here that anything above 70~ causes ridiculously black Tajarans, so that's why it's still here.
You could use str_len here, but even simpler would be just (pidfile != 0 && pidfile[0] != 0)
There's no real expectations so yeah, really only things that cause it to raise an exception, like a bad URL maybe, like you mentioned, would get caught.

I see that you were just modifying this and that it was already not tested.  Are you able to add some expectations, for example, that #authenticate passes with user.email and "password"?  And/or that user.valid? is true?  Or something like that.
LinkedBlockingQueue тебе в помощь.
Так как он thread-safe
Если передать параметры: "1" "1" "+" "1" - должна быть ошибка, а не 12.
Плюс, тут делается лишний поиск по мапу. Правильнее (если в мапе не может быть null-значений):
```
Object value = map.get(key);
if (value == null) {
  map.put(value, initial);
} else {
  map.put(value, value + initial);
}
```
Открывающая скобка ставится на предыдущей строке.
Всегда надо иметь возможность закрывать ресурс снаружи. См. интерфейс Closeable
Так не работает. Во-первых, разделитель - не только пробел. Во вторых, такое выражение ```1 1 + 1``` вернёт 12, а на самом деле это ошибка.
isLoaded() should never be false after a call to loadMovie() unless it failed.

We should find a way to handle asynchronous loading, for both URL and file assets together
unless you have a terribly good reason for needing boost, this is a big no for me
Read the function again.  This query is in the **save** method.
well, the idea of using a mask is that, when computing mean or percentile, you need to now how many pixels belong to the neighborhood, at the same time it allows you to set a mask on your image to selectively restrict the pixels being used.

so maybe a solution can be a helper function that returns true or false if the pixel is in the valid region (i.e. inside the image and inside the mask) ?
It would be nice to have it in `SGDClassifier too`. By the way, I recently realized that the learning rate and regularization don't affect the classifier in the case of the Perceptron (since the margin is at 0, the learning rate and regularization don't affect the consecutive updates...).
Ok sorry :)
Yeah I didn't know it could do that either, just had the feeling it should be possible ;)
I learned yesterday from stackoverflow that np.bincount is even more awesome than I thought ^^

To answer your question: The README says we require 1.3.
I'm also on 1.6.1. To bad we can't speed-test but I think it should be fine.

I think I would sort of prefer a backport. Though that shouldn't be a reason to delay merging.

Maybe we should leave it like it is an keep in mind / make an issue to replace it with choice at some point?
This remark should probably be moved to the part of the docstring in which you discuss the embed_solve parameter.
+1, at least as a constructor option (being able to adjust the number of power iteration would be a good to have too).

For exact sparse SVD there is also: `scipy.sparse.linalg.svds` based on ARPACK but I have no idea how scalable and stable it is.
See the fail through. This error will be exposed as a javascript error upstream. The actual stack will not but in production mode this isn't really helpful.
create() no longer exists, instead there are the following factory methods:

* timeUUID (v1)
* randomUUID (v4)
* md5UUID (v3)
* sha1UUID (v5)
myArray[ i ] = NumberTraits<TComponent>::ZERO; // faster ?
Understood, thanks Brian.
Assume that this can go as well?
Add NULL check for the parameter `string`, and remove unnecessary NULL checks in caller side.
That will increase code readability in `::toJson`.
openURL is part of this Jira task
We don't use mkbuild anymore so it could just be another advantage of moving to make. We'd get even more advantages once we move to qmake.
I'm not sure what you mean by within a method, i ran the formatter on this exact file and and it seemed to remove spaces in all occurences.  If there is actually an inconsistency, let me know with a specific example and i'll see if i can fix that in the formatter
this file is already exist in repo. unless we don't generate this file from makefile, we can update it
Move code to cpp
it appears this line was not passed in the formatter as there shouldn't be spaces inside the angle brackets.
Odd class hierarchy. Why not extend `MenuDrawer` directly? If it does not have the functionality it needs we should refactor the common stuff up as high as possible.
Ha! Let's remove credits ;)
Ønsker du virkelig å fjerne dette.
this is refactored in reLink branch, not actioned here
If you put the 'try' around the 'if' then you only need one try clause.
Either way, it needs to be installed so that you can type `lessc` anywhere. Ie. that it's on the PATH. 
Otherwise, you have to edit `COMPRESS_PRECOMPILERS` and explicitely set:

```
COMPRESS_PRECOMPILERS = (
    ('text/less', '/your/custom/path/to/lessc {infile} {outfile}'),
)
```
This can be reduced. This is enough:

```python
try:
    return re.findall(ooid_regex, input_str)[0][1]
except IndexError:
   return False
```
And a few colons.
why are you ripping out this?
Also this shouldn't be 1.0 because that is not the version number.
is this a part of registration?
consider how to we will check for timely in cases where we aren't logged in through the admin portal.
@shacker when I run this file through jsonlint, I get some errors.
Would it be possible to fix those?

It would also be good to convert these to 2 spaces.
Bingo - @default does the trick. Much cleaner now.
(However that might be because GitHub is having problems right now.)
Why are you wrapping this in a function?
So this one I will have to respectfully argue with you Mr. Lee though please know I mean no offense at all.

The way this module is working (As I understand it).  I'm spawning a service off of "cmd.exe" with /C "what ever command I want" as the service parameters.

I'm not sure how to grab the output of the commands I execute (do know if it can be done) so I cat them into a .txt file stored in the windows temp drive then read them back.

Though I'm sure someone smarter then myself could figure out how to do it.  It really is best I think if the module knows ahead of time what the name of the directory is.

Also this option has already been approved in my "psexec_command.rb" module.  So I would request that you please accept it here as well as I think it enhances the performance and capabilities quite nicely.

Is that alright with you?
RPORT should be an OptPort.  Or better yet, use the convenience wrapper ```Opt::RPORT(443)```
use vprint_status instead of this
72817 : ICONICS GENESIS32 / GENESIS64 GenBroker Service Multiple Remote Overflows
winrm_poke may throw nil due to a timeout since it uses send_request_cgi()... it needs to be aware of that.
do you mean I should rename the file?
Would a boolean `multichannel=` be acceptable?  I started implementing this, and there ended up being a lot of string parsing added that wasn't very readable as `color` got passed along.  Identical functionality but cleaner.
Ok, I'll convert all docs in this file.
That's a good point; though this bit of code was copied from the original Git back-end, so it might be worth reporting there too o.o
Not sure why this is different than `test_old_mobile_redirects` except it doesn't have `_old_` in the name. And those, for the record, aren't going away any time soon, no reason to refer to them as `old`.
Yes, the JSDoc bug still exists, which means everything must be `var NAME = function()  {}`, which means the name of the function is not actually associated with the Rvalue, which means that at run time, constructors are all anonymous functions.  The Chrome debugger figures this stuff out, but Firebug does not, meaning your call stack is always full of 20 "anonymous" functions.

If anything I'd say we should be doing `var NAME = function NAME() {}` everywhere, across the codebase.
Users will want to call `setPositions` with undefined or even one point.  I throw exceptions in Insight3D, and users didn't like it.  Instead, we should just not draw it.
i'm fine with removing this. an implementer would implement this in their delegate anyway if they wanted different behavior.
It looks like this means our only option is to test against the most-recently built (locally) version of the sdk, is that right?  If so, shouldn't we instead have an option to specify a different version for testing?
Tem opção de não instalar o coffee.
Se instala o coffee ele usa o app.js.coffee e não o app.js
Semantic discussion:  is the return necessary?  If so, should it have a [newline prepended][1]?  Should it be `return` or `return null`?

[1]: http://symfony.com/doc/master/contributing/code/standards.html#structure
Does this function take `bytes` as a string _or_ a Buffer?
oh, really off topic for a doc PR, but what if we would change this to findManyByRequest and pass the whole request? i.e. symfony 2.2 supports hostnamePattern on routes, so we might want to know the hostname that was used in this request...
This is only used once, why is it a separate variable?
Apparently can work with InputRange.
Yes, my reasoning was exactly as AndrejMitrovic has described.
app.removeBack = 4; //makes sense?
find in code font
Derp, nevermind.  It's length...  Specifying "-1" instead of the length of the substring is what I'm thinking...
Does the ad need to be removed from the view before it is released?
Could you clean up the javadoc or make this package-private?
Missing a comma at the end of this line...
Long comment is long and wrong.  We absolutely can't update the persisted meta/global.  The persisted meta/global should be *exactly* what we downloaded from the server, no ifs ands or buts.  I'm a little unclear on the flow here, so this might not be an issue, but the comment scares me.  Please clarify.
is this futureproof? if MAC_OS_X_VERSION_10_8 is defined, then is MAC_OS_X_VERSION_10_7 defined also?
We promise to return a string or a boolean... so please return false here, and an error message would help too
please use one use statement per class
Extracted phone() function.
ok I'll remove the get_class function.
The alias should be ``jms_serializer``. It has to be the same than the name of the helper, so that the behavior is consistent when using it in the full-stack framework (lazy-loaded) or with the engine of the component only (where the helpers are not lazy-loaded)
Can we use the hex value for this?

e.g. #0000ff

http://www.w3schools.com/cssref/css_colornames.asp
No; it's the same thread- I'll add a cautionary note.
Done, why didn't i copy what Gary posted :(
Prefer we don't do this kind of 1-line if statement, but not a huge deal.
ok nm i get it
That would indeed be a security issue. 

One solution could be a notice to the user to login with his credentials and merge the Facebook information with the existing account. That way we are sure the user is really in possession of the account.
good!  Much cleaner :)
I'm not really sure. When the shell is called using ```Console/cake``` I think it does a lot of the original inclusion for you, but maybe adding ```App::uses``` is better just in case.
remove all puts if it works properly
don't want to piss off our British friends, but `favorite` is still more widely used :)
not used yet?
here comes the no op interceptor. Now I am a little confused
take off the constant thing
Should be documented as this is the way to invoke the wrapper.
This has nothing to do inside the "workgroup" app, move it to the "project_sheet" app
We were trying to come up with a global id that we can subscribe to.  We would fire an event on that id for every event we cared about, which is Created and Destroyed.  Not the best in terms of performance not that I think about it.  Any thoughts on how to have global events?
geolocation needs to be tested as a part of this @tracyli please
Can this number not be hardcoded? You've checked with Ken about setting this number to be 5? Even if it's not changeable by the developer, it's still better to store that in a constant.
See comment about CSS. Fine doing it, but don't find it necessary.

It's also a bit wonky working with the Channel API with external JS.