labofsahil/github-event-dataset-2012 huggingface dataset:
I assume you're wondering why we need to `bind()` these methods.

The deal is that the methods get added as listeners to the argument streams (well, to a set of `Valve` instances on top of them). If they weren't bound, then the event callbacks would get called with an unpredictable `this`.

I'll add some comments (here and in the other equivalent spots).
What if an app wants to use long press on the ESC key?
Changed to new-style formatting.
`astropy.io.votable.table.from_table` will create the `VOTableFile` from a `Table`.  I don't think there's a need for the following lines, other than `to_xml` to write out.
It could still be done for Numpy: PyFITS still maintains Python 2.5 support (though PyFITS 3.1.x will probably the last release series to do so).  In PyFITS this isiterable() function actually has something like
```python
try:
    from collections import Iterable
    if isinstance(obj, Iterable):
        return True
except ImportError:
    pass

try:
    iter(obj)
    return True
except TypeError:
    return False
```

It's a pretty minor thing, but maybe I'll submit this to Numpy nontheless.
我觉得约束是一样的，但是写和阅读的时候少一些干扰。
It's global either way. Global functions aren't much better than global variables...

A function with __attribute__((constructor)) can provide the same behaviour, but it's a GCC extension (non-standard) and using a global variable is much cleaner.
@ADmad i added spaces and committed.
Need a space after the `=>`
please fix the coding standards. The curly brace should be on its own line for methods and classes
This could be simplified.
```php
return (array) $ids;
```
This should be a static variable if possible
The same as above `beginning_of_year`.
Does this need to be careful of packages that might have "Rcpp" in their name?
It's actually okay to pass NULL to free(): " If ptr is NULL, no operation is performed." (from man page).
Could you put `return []`on a new line, it is easier to spot.
This is the only thing I don't like. I get that the remote can have a `NULL` fetch spec, but we could do better than forcing an empty string to mean "default fetchspec". Not too sure of how, though.
Documentation. Specially regarding the semantics of the different types.
So maybe something like `40 tweets in the last 24 hours have replies` ?
this import should be below tempfile
Do you really need to use `str()`?  `'%s'` already does that anyway.
``It _is_ triggered``
Ha! I didn't know you could do that. Is it better than `input()`?
An alternative way to do this is:

    try:
        addon = Addon.objects.get(pk=addon_id)
    except Addon.DoesNotExist:
        return

This also keeps it to a single db query, whereas the `exists` and the `[0]` actually does 2 separate queries.
I would also check that the content type of the result, if we expect to have json here.
I don't think @jerel wrote this file! You can put your own name in here if you like, or remove this block of comments entirely.
This was all deleted
@bschussek  Keeping the `FieldType` should be enough for BC, no? We don't need to extend it for built-in types.
you should write the ``f`` in lowercase for consistency
please fix the CS. A space should be added between ``foreach`` and the brace and spaces should be removed inside the braces
you have an extra space here
why are we type hinting \Closure here anyway? this is not recommended .. 
see http://php.net/closure
Can you remove the group?
I agreed that in this pull I shouldn't do such kind of thing, but look in another places of code. It mixed up single quotes(') and double quotes(") everywhere. So, I thought that using only double quotes will make code more consistent.
also needs a configurable worker pidfile.

that will obviously need to be a string with a single `%d` present
ditto for these.
Is there something bulletproof protecting a thread in this pool from ever dieing?
Why store the id instead of the photo object?
It's not obvious what `includeND` means, `includeNeighborhood` is clear. Also `filterDuplicatesFromArray` seems that it's general purpose method, it should be `filterDuplicatesVenuesFromArray`.
Single quotes in the ruby tag preferred.
Missing comment terminator/trailing `*/`
Sample usage of the avoid:

```
def f(x):
    return x*x

if __name__ == '__main__':
    pool = Pool(processes=4)              # start 4 worker processes
    result = pool.apply_async(f, [10])    # evaluate "f(10)" asynchronously
    print result.get(timeout=1)           # prints "100" unless your computer is *very* slow
    print pool.map(f, range(10))          # prints "[0, 1, 4,..., 81]"
```
Same goes for shelly console
Requested assistance from @pelson on this.
Please consider adding 75 to that list of numbers!
define as a member and put creation in setup

(Note that variable names are lower case starting)
Fine by me. It was originally just "board", and *that* was certainly no good.
I'd love some suggestions on a better data structure for the decorators.
stylistically, top down comments are preferred to EOL comments:

```
// See #7960
zIndex: 90
```
лучше б тогда userIdList был set'ом, а не list'ом
There should indeed be a `free()` here.
wOOt!!

thanks for checking the typo, right now is not used for this pr but will be used for the next one to get the icon for canceled downloads.

will fix it anyway :)
I will continue the review later, needs to leave the building.
Replace L80-83 with

      this.settingGroups.splice(
        this.settingGroups.indexOf(this.defaultGroup), 1);
Interesting doc notation never seen this before. What parses it?
Why not just:

var lowerCaseWord = word.toLowerCase()

?
Should be simply `var id = message.id;`
What about: 

      fb.contacts = {
        get: function fb_contacts_get() {
        },
        
        save: function fb_contacts_save() {
        },
        
        saveAll: function fb_contacts_saveAll() {
        },
        
        init: function fb_contacts_init() {
           // and move the createStore function and the content of the init function here
        },
        ...
      };

I don't have a real strong opinion on fb.contacts = {} code but I think the init code can avoid one redirection.
I’d prefer using `<header>` here then. Quite a few lists in the Settings app for instance don’t use any `<h2>` element.
Finally I included the code in the parent's scope teardown and updated the child's suite suiteSetup by setup so they are executed before each test.
Is data-time-update only for querySelectorAll? Maybe you can apply the timeHeader class,  but it's just a personal opinion :)
Why do we need an iframe?
Totally agree. I'll change that
my problem with this approach was that bool(0) will return false whereas any other digit will return True.  I wanted this function to tell weather it succeeded or failed.  If a remote linux command returned 127 for example, it will mean true in python but it is not true the command was successful. What is your advice on this ? Would you like me to  make it available as instance variable like the stdout and stderr?
I will remove them.
This line is a bit long and hard to read. Would it be possible to break it up?
Oh. OK good I just wanted to check that that was the case, if it is I bet the reason why it was being called was during the meta-validation, so I think for that maybe we should just have `META_SCHEMA` be the default for `_schema` since that's going to be what's expected there.

Hopefully I'll get a chance to merge this tonight.
rename it to something like Redirect.to or whatever.
I mistook the line as open right now. I have no idea why I left post on init.
Still need to pull this out in to a constant.
Yikes, apologies. Will fix up indentation asap
Note that this file should now be in the package template. I'm not sure if it's best to pull in the latest package-template changes before, after, or in the current pull request though. @eteq - any suggestions?
Assets kommen später
You have 3 functions and logging errors in two places just to call 'self.botmaster.cleanShutdown'. In any case, 'botmaster.cleanShutdown' looks like it already logs what is going on (although it may be useful to log that the shutdown was triggered by receiving USR1.
This can be a simple `self.fail`.
isn't there a nicer way of doing this? I think I understand why you are doing it, but I'm not sure about this.

Also please can we delete rather than comment out functions.
why would this be included?
Shit! revert it ;)
btw, it's not required to explicit check for `eql?` if you know that variable will be boolean (or `nil` as it will evaluate to `false`)
so you can just
```ruby
 if @hasresult and @same
```
Oops, well spotted. Will replace!
@Mossop or maybe I should just remove "and typically prefix with `@` character" as in long term we want to get rid of those.
Is `normalize(id).slice(1)` equivalent to `normalize(id).substr(1)`?
If yes, substr is more explicit.
> You are already freezing Function.prototype. So that this additional lambda feature looks useless, isn't?

Nope without that one could do following

```js
const { Loader } = require('@loader')
Loader.prototype.message = {...}
```

which than can be consumed by something else. Freezing `Function.prototype` does not solves this as each function has fresh object as it's prototype.
So tested it locally and build gets executed in both cases. So it looks like we are fine here.
I would also suggest we get rid of `appendToList` and move its code in here. Any reason we need this as a separate function?
do we need this here? if so, why?
expectations shouldn't be set inside a let. If the let is never called, then the expectations are never setup and putting them in here makes it non-obvious that the expectations are there as part of the test.  Looking at the test, it looks like these aren't expectations, they are stubs.

You might extract the setup of the stubs to a method that describes what you are going to have setup. For example, `given_the_target_directory_does_not_exist`
Please see the `be_computed_by` matcher and write explicit examples, not loops.
Indeed, it was just copied from the previous spec.
I made the change in a new commit, what do you think?
and here it might probably be result
i actually disagree with this. Tooltip will break if inserted in the body triggered on a modal because of z-index issues. Just remove this line.
Here I suggest to create an other variable called `@modalBoxShadowColor` for instance (or something better, as you want), which has `@modalBorder` as value.

What do you think?
I'm not sure I understand why the `data` directory should be ignored. It, and everything under it, should be version controlled.
Should be PhpDateTime
Return? Not render? This is inconsistent with the method documentation
This if always is true
That said, git commands only operate on the local repository, with the exception of push.  So `git merge` could not modify GiveCampUK/GiveCRM unless you explicitly pushed changes made to the local branch to that remote repository.
Will undo. No idea why this was borked.
Why should you be able to set it? Just cancel it?
It's not idiomatic Ember to stash your objects inside sub-objects on the namespace. What you're doing here will break things like helpful `toString()` results. Instead of `App.Controller.Main`, you would have something like `App.mainController`. (Note also that instances are idiomatically lowercase and classes are idiomatically uppercase.)

If you *really* wanted to do this and preserve `toString()` functionality, you could make these instances of `Ember.Namespace` instead of plain JavaScript objects.
@ffesseler Can you update this to trim the value, and then reject the add if the trimmed value is empty? If not I can land this request and make the change myself.
請新增一行...
Also makes sense to take this out and maybe have a function that prints the date (just like the return value) and checks/puts era if necessary.
shouldn't the doc mention the interface instead ?
@dbu the suggestions should be changed to use a test explaining why it is suggested instead of displaying a version constraint IMO. It makes it better for users.
Are you not using JSHint? Wat.
Hi Gordon, this was not intentional and should not be deleted.  Thanks!
what service and where do you raise something?
Space after args,
This looks like too  many spaces for indent - should be 8.
We should throw a try/except block around the news stories, just in case the Google feed isn't reachable. 

```python
try:
    news_stories = find_news_stories(bill_number)
except Exception as e:
    news_stories = []
    logging.error("Couldn't parse google news: {}".format(unicode(e)))
```
Most of this locators are broken, please fix them so that they point to what they should.
Add a space between the href and the word 'can'.

<code><a href="http://reps.mozilla.org">Mozilla Reps Portal</a> can give</code>
Is this specific to the phonebook?
Why all of the indent changes?
Shouldn't this just be `CoverageCraft()`
навіщо дві опції, якщо юзається тільки повний шлях? зайве
why remove required? i think its still required ?
How is this related to this test code specifically? If this is a check for the general setup logic, then move this into GaiaTestCase.
MMGetBlackPercent and in mapwalk.simba?
I'm not so up-to-date with the standards, but will this do the same thing as `<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">` in all the browsers that we support (including like, IE8 and stuff)?
Оооо, теперь буду знать.  Сенкью.
You could just return in_array here.
Change "re pair" to "repair"
`assertSame` being faster and stricter, I only use `assertEquals` when you need a loose comparison, for instance when you need the integer `42` and the string `'42'` to be the same.
Use the short format:

```js
'dist/gruntjs.min.js': ['dist/gruntjs.js']
```
please move the opening bracket to the next line
We should add <code>@property</code> to this method since it returns a text.
the test have reported false positive if no messages where found.
If feedback_pg.messages is empty then the test dose not go in the for loop so the test is reported pass because it has no assert in that path
Not sure if we'd be creating/dropping CFs on the fly, but I take your point. We can keep it as is until we see a problem with (or Helenus fixes it).
As this function is only used in the `describe('comments'` section, it should be moved into that section.
If ``ScalaDebugTarget.setThreads`` it's only supposed to be accessed by this actor, then I'd remove this fields. THe reason is that I find it's too eas to forget to update its value and then ``ScalaDebugTarget.threads`` and ``ScalaDebugTargetActor.threads0`` may hold different data.
I will throw a custom exception, I think it makes more sense.
We might just want to put all the modules in their own modules directory. It'd make this code a bit simpler. The mod prefix makes me think these are apache modules.
We still have examples.
useless ;)
сначала так и было, потом решил на 0 не ругаться, потому что это допустимое значение - просто тихо выйти
создание массива вне блока гарантирующего успех вызова [super init] же
Неверно, должно быть !=  метод должен сравнивать свой ДНК и ДНК переданного в качестве аргумента объекта и возвращать количество позиций где символы ДНК не совпадают.
А зачем делается два массива? Ведь можно сразу инициализировать и заполнять _DNA
The MockCoursesDaoImpl is commented-out.  We probably ned it active by default.
In order to keep packets length
Hopefully I am not missing something but isn't this statement a duplicate of the below statement and not needed.
Erm... What?
code style fixes : `(Boolean) $options['interlace']`
I think we should throw and exception if the times are not compatible.
I think this function could be useful for some folks. In some data once I saw a consistent 23 Hz drift (?!?), so I could see why someone might want to band-stop (if a good SSP couldn't take care of it, for example, or if you wanted to calculate SSPs without that drift contaminating them, etc.). This is a trivial one to implement given the code we've written, in any case...
Wow, crazy...

On Sat, Jun 16, 2012 at 1:40 PM, Alfredo Aguirre <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,23 @@
> > +from django.core.cache import cache
> > +from django.http import HttpResponseBadRequest, HttpResponseForbidden
> > +import hashlib
> > +
> > +def throttle_view(func, methods=None, duration=15):
> > +    """If I was a better programmer, I would get this to work with
> passed arguments,
> > +    however - it's just not in the cards tonight. hence ...
> > +    TODO: make this thing actually take arguments
> > +    """
> > +    def inner(request, *args, **kwargs):
> > +        throttled_methods = methods if methods else ['POST', 'GET']
>
> In python mutable parameters are modified in place. It is saner to set
> them after, more about this: http://effbot.org/zone/default-values.htm
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla/webpagemaker/pull/434/files#r997836
>



-- 
@chmcavoy <http://twitter.com/chmcavoy>
http://lonelylion.com
Ah, I see.  Thanks.  I'll make the changes you suggested.
Fine, then.  Just look strange when reading codes.
Use `StandardError` instead of `Exception`

http://blog.nicksieger.com/articles/2006/09/06/rubys-exception-hierarchy
there is a missing `:` here.
checking nil is not needed.
I fix to compare Qtrue.
This is silly. I like it.
default should only be something that realy can be typed on console, shouldn't it? I am not able to type `null` neither could it be displayed somehow in the brackets. So we might need to force $default to be a string to ensure straight forward behavior...
This method needs a bit more explanation I think
用
`background-image: image-url("www/pics_icon.png")`

會比較結構化一點, 今天會教你這部, 記得問我
Pero la lógica ya está en VacationList...
Lo hago yo, no problem
Es cierto, no me quedó bien explicado, además puede resultar medio feo, pero la idea era decir que la empresa abandonó un poco esa idea de anotar todo y de sistematizar tanto la selección de personal. Si te interesa lo podemos chatear o hablar rapididto, en unos minutos me voy.
I kept the checked ParseException on purpose because I want the caller to be aware of the fact that a problem can happen when parsing and do something for this. I don't want to mask this potential problem with a RuntimeException.

Does it make sense to you ?
there's no more debug parameters for record_adapter::generate_subdefs
Both this file and the spec file should conform to our new and improved code standarsds; that is to say, 2 spaces per indent.  Thanks!
Jus copied this from your own version in role repository, that looked to me more updated :)

________________________________________
From: Elad Ben-Israel [reply@reply.github.com]
Sent: Wednesday, March 14, 2012 10:36 PM
To: Yosef Dinerstein
Subject: Re: [node-wintools] Parse XML output of WMIC. (#1)

> @@ -1,15 +1,23 @@
> -# wintools - Some windows tools for node.js #

The double `#` also works BTW...

---
Reply to this email directly or view it on GitHub:
https://github.com/anodejs/node-wintools/pull/1/files#r559384
This is a good suggestion, and actually brings up a related question. Should we just develop with the 1.2.x API? Or should we aim to be more backward-compatible?
Shouldn't you erase the other line, based on the comment above?
@carsomyr  - This is close to tty detection. It will only format the logs if they are being printed to stdout. No formatting will occur if they are being saved to a file
Yes, I know. Let's rework this in the next release of the app or if the ontology breaks the code assumptions (whichever comes first). I will keep you note in here so that we can refer back to it.
I don't understand why this `only_path` argument is necessary.
I feel like this should be a method on the view object. IIRC we already have to pass the request to the FlavorView object, so passing the marker wouldn't be too bad. It seems like having things like the "name" argument be part of the view class as well makes sense too.
@vblagoje, from the stack trace in the description it seems that we keep catching ExecutionException from a Future.get() and re-throwing it, wrapped in a java.lang.Exception. I think we can instead extract the cause of the ExecutionException and throw that.
I could but then @galderz prefers them to be squashed. So what is preferred practice du jour?
Oops, let me see
No snapshots allowed.
I agree with you. I'ill changed my code.
But IMHO, you are creating an application here, not a library or the framework.
Class 'GitList\Component\Git\Model\DiffLine' not found
Yeah, you're right that we probably don't care about the flags.

Using enums for `OR`able masks is a fairly common practice in Objective-C, though.
We have fixed the bug correct?
I just tried to remove `OpenLayers.Animation.requestFrame` here, and use the previous code. Based on your comment in the issue description, I was expecting to see loading placeholders in FireFox. But I'm actually not seeing loading placeholders in FireFox. So it looks like FireFox not displaying placeholders isn't related to the use of rAF. I really don't know why they're gone.

Also, I don't know whether we should/need to set the image visibility and opacity in a rAF callback. Are you seeing actual performance improvements? What browsers/devices? I think we should not introduce asynchronicity if not really necessary.

Also, I note that you trigger `loadend` in the rAF callback but still set `isLoading` outside the rAF  callback. Any reason for this? I tend to think that there's no need to trigger `loadend` in the rAF callback.
Human readable sizes would be nice, maybe

```php
Format::human_size($this->getConfig()->getMaxFileSize())
```

which might return something like `32MB` for instance
I need some small research on that - could you please check whether it computes all possible distances? Can it avoid computing some of them internally?
please add a white space after the cast and put the else on a separate line
control is deprecated, this should be kept as loop.
Doctrine annotations should be $StudlyCaps, like in:
Id
GeneratedValue
Column
Not sure this makes sense. Have you got some test cases fo what this does/doesn't escape?
1. we're not outputing HTML so why encode with HTML special chars?
2. Data from the db shouldn't be encoded with slashes ...
Ack, I guess a checkpoint of most files isn't particularly useful.  I'll change it to WT_ERR.
Take a look at the new commit, perhaps it makes more sense now.
Alternatively we can just use the luacrypto x509_ca stuff and get the X509_STORE out of it.
Sorry for this, it's my wrong auto-formatter configuration. But I still don't know what is preffered form for methods - with 'this' or without?
side note: I noticed in this code, and some of my code, that the "new_mapping" is not always convenient, so in EWF master, I made it part of the interface WSF_ROUTER_MAPPING_FACTORY (or something like that), so it is more flexible for new mapping.
Are you sure ``version_compare($install...IRED_PHP_VERSION, '>=')`` has the correct type as ``*`` is not a sub-type of ``boolean``?
Are you sure ``$attribute`` has the correct type as ``*`` is not a sub-type of ``string``?
Works if you set the package root directory in the editor preferences.
If i use the dart VM i get an error:

    library handler failed
    #import('package:unittest/unittest.dart');
it is unless code. if could create separete view file for it, for example
I'm not sure what you mean. Can you add more details?
Most Java developers are not familar with instance initializers. Recommend inlining the initial assignments to the associated variables.
`setReplicationFactor`?
Ok, will fix that but I thought we are using spaces that is why I added them.
You have here IntField, but above you have strings as keys, not numbers. Does this work?
Ok. That was bad idea. I hope now is better.
`user is None`.
Well, we dont even have minimal length. Do we even want this (Django generic forms do not have minimal length anywhere, just max.) ?
If you want I can check how it is done. Maybe it is good for passwords (to ensure strong passwords), but for username I think there should not be minimal length.
We could have. The questions is if it is reasonable. :-) I would like to have semantic structure. But yes, we could have order be a map from number of columns to order for this number of columns. But as I am asking myself (and you) is, is order really a part of settings? Probably yes. Maybe we can do it like that.
Please use `uuid4`.
This URL does not look correct, does it? ;-)
Namespace will need to be updated due to separate code file
ちゃんこなべとかいいなあ
We should make sure that an empty directory is also older than a day. maybe it was just created and no file yet?!
i think, SpellInfo const* spell = GetSpell(); to declarate pointer to class spellinfo
I changed the test side to match the implementation because it's easier and more reliable than having to add a yield for every template added to the suite.
```ruby
@coupons.select(&:applied?)
```
why not definining this in the XM file ?
Here I would suggest passing ``nestedMerged.getEntity()`` instead of ``null``.
it was breaking build on my ant 1.8.3. Dunno why.
typo: replace "=" with ":"
IIRC we have stuff in config to deal with different forms of 'True'
"In case of exception, instead of throwing, the decorated functions return a rejected promise holding the exception."
please add [%d] %s ... errno, strerror(errno)
meta is so ABSOLUTELY undspecific... wouldn't it be better using mixinDomain here? Would make, at least to a developer, kind of more sense. I really like the base idea but the name is kind of  ambiguous.
Remove type hinting.
Why does it need that font declaration at all? Should it not just inherit?
@thiagogabriel I had an Idea...

What do you think about this? Instead of the `p` font.

It would fix #109

```css
  ul li {
      font-size: 14px;
      line-height: 18px;
      list-style: none outside none;
      margin-bottom: 1em;
  }
```
Fair.  The problem, though, is that it gets confusing with the *Example usage:* ends with a string.  The end quotation mark of the docs makes it all a little confusing.
Thinking about extendability:  How could a gamma kernel be included, for endog that is continuous but strictly positive?

Epanechnikov (?) or others have the advantage that they only need to be evaluated at points in the neighborhood, while gaussian requires all points. 
They might also be better with multimodal models (but I don't have any evidence).
I haven't found a good description of what has to be done with CJK, so I ported this class (necessary methods and constants) from ICU4J. I'll add some notes about that later.
great, thanks for fixing it.
you rock. thanks.

On Sun, Feb 26, 2012 at 2:07 PM, Moses Nakamura <
reply@reply.github.com
> wrote:

> >      return Sale(smap["name"], smap["sale"], smap["sale_key"],
> >                  smap["store"], smap["sale_url"], smap["begins"],
> >                  smap["image_urls"], description=description, ends=ends,
> > -                products=products)
> > +                product_urls=product_urls)
>
> ok, consistency is a reasonable argument, pulling now.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mnn2104/PyGilt/pull/1/files#r489586
>
- I wonder what Py_XDECREF will do with a ``git_treebuilder`` struct.
- The point of using ``Py_XDECREF`` instead of ``Py_DECREF`` is that you don't need to check for NULL.
Can you please give a bit more detail / background / motivation for this class?  It's a bit mysterious as-is.  A 2 line summary of the jira ticket would be enough.
I just kept it the same as before.  I don't really have an opinion about what makes sense for a default.
@alexlo03  No, there was a bug in the SDK signing procedure.
I have removed this file in the following commit.
Soory, Gary, I don't understand this sentence...
**are supported** ... **are set** ...
Maybe '...properties as 'true' respectively' ?
Please fix all spaces here and in the rest of the file.
Done. There's a new run() method now.
also should be a case statement
1. I've always done the opposite. It feels more natural (to me) to say "The radius is less than 0.15", than "0.15 is greater than the radius." It makes the variable the object in question.

2. I could. I used to love taking advantage of 0 == false, but over time I've come to the feeling that only booleans should be passed to conditionals.
This doesn't really belong here, does it? I mean this entire file inside this PR.
I know it wasn't changed in this patched, I just noticed it now :)

However, a return value of false means that the function call failed - not that "50% of it got applied, now you need figure out which parts"
Could you align that with the d?
nao pula linha aqui tbm
можна компактніше. ні, то винеси урл в окрему змінну. див. https://github.com/emeraldMaster/PortfolioBundle/blob/8a01f1e8605e28d762a483573ead38449b150a2b/Tests/Controller/CategoryControllerTest.php#L107
The problem with -ss is that it "[...]decodes but discards input until the timestamps reach position." so you waste a lot of time for decoding useless stuff. -itsoffset may not offer the same precision, but fair enough and with  better performances. And from what i understand, -itsoffset is only used on input file, so I doubt their would have any problem with it. Correct me if I'm wrong.
Yes - according to the docs: https://docs.djangoproject.com/en/dev/ref/models/querysets/#django.db.models.query.QuerySet.select_related
I would prepend the name with webinos- because it is as far as i can see only usable within webinos (or am i wrong :) )
Incorrect indentation here?
What is this? This should already be there... strange
Should you make it fTestNet ||   ... so it is running on testnet for a while before the switch on main net?
Huh, why twice?
`isHand`
I've made changes to the ContentAPI adapter to escape all the slugs and snac codes (see subsequent commit).
Для этого надо использовать [IdentityHashMap](http://docs.oracle.com/javase/1.4.2/docs/api/java/util/IdentityHashMap.html)
Есть ощущение, что это copy-paste.
Sorry, "working in the same line of work" is redundant (uses "work" twice) and too broad.
This is cool. I'll check with @shane-tomlinson who had some other ideas about how to best resize this dialog. He found some bugs with our previous method.
I am still getting a little bit of flakiness on this step but need to do some more investigation.
do we test this method?
Is that a +1 on `neural`, or on handling this later?
I'm didn't check the details but I think you can do the whole thing using ``np.bincount`` with the ``weights`` parameter.
Watch out with `atleast_2d`; it passes `np.matrix` objects through unchanged. I think `array2d` from `utils` is safer.
I would add a flag `deterministic_sign=True` by default to `randomize_svd` instead.
Why do we need all these Thread.sleep() calls? It sleeps for 1 second here, and then sleeps for another second in the queryCacheCheck method. Is this really necessary?
@AndrejMitrovic : This is the wrongest current dmd behavior. If variable has const or immutable type and CTFEable intializer, it will become manifest constant *implicitly*. Bug 3449 is a part of that *bug*. I think that is definitely bug, but Walter and Don doesn't agree with my argue.
since we don't do any specific font related parsing, we can rename this function as e.g. getArrayObject and use it whenever we want to parse Array
Brace should be on above line.
Yeah, I struggled with that issue too. On the one hand I don't just want to swallow it, on the other hand I prefer to to directly bubble lower-level exceptions. This is where I wish python had a predictable convention for chaining exceptions (e.g., create the CommandError with the IOError where it's available as e.cause or something like that. Maybe I should add something like that to command error?
Okay, I'll change that. I was thinking it may be a bit messy to maintain with the ifdefs.
The alternative is to build a list and make sure it'll be completely filled by the end and not double filled, a bit redundant with 'loadSubFiles' but that's only for safety and a temporary hack, shouldn't be too costly.

If you think we better be safe than sorry, I should change it.
You need to know the table size up front for that to work.
`if not versions:` is more pythonic.
should be node['platform']
The preferred way to create (or update) the database is to use "nova-manage db sync". Doing so when either $::nova::params::common_package_name is installed (or updated) or when sql_connection changes seems sensible enough to me. Does that answer your question?
The `value` parameter here is acting as a Control Couple. It would be better to send a different message for setting vs getting the HTML and remove this conditional.
Why does this test require setting up an HTTPS server?
Updated to do this. Required a change to EventManager::attachAggregate to also default $priority to 1.
Ah, that's a good one :). A comment to describe this case is probably fine for now, so we know why it's there.
I'm curious why you opted for pure javascript instead of something like `$('output_'+this.session_id).append('<div class="poweredBy">Powered by <img src='+$URL['powered_by_img']+'/></div>');
please use the version of this that's in the utilities class
That's just a throwaway line, will be gone as soon as we have service status in the api.
a few notes on CS issues .. there should be a space after `try` and `if` .. and there is a lot of extra empty lines.
When you use Rex::MIME::Message like I demonstrated, you can use send_request_cgi() instead.  This is exactly how to do that:

	res = send_request_cgi({
		'uri'     => '/en-US/manager/appinstall/_upload',
		'method'  => 'POST',
		'cookie'  => @auth_cookies,
		'ctype'   => "multipart/form-data; boundary=#{data.bound}",
		'data'    => post_data
	}, 30)
Bad indent, makes the intent unclear.
this won't work with Midgard
Gotcha.  OK with me.
Mixed tabs and spaces?
I believe we discussed this before, but I don't think these checks belong here.  What is the situation again?
It just a placeholder while async loading textures.  We'll see how it pans out.  I can actually envision multiple "defaults" or really built-in textures for different material components - diffuse, alpha, etc.  I can even imagine the ability for a material author to "create" (it would be cached) a placeholder texture, e.g., `{ components : "a", values : [1.0] }`.  We don't need to tackle this today, but I made a note on the [roadmap]().
Well, it's not totally broken in that it works on all valid input, it just doesn't catch all invalid cases.  I'll add a test case and try to fix it.
Changer order from here to upperest :-)
Either express or request lower-cases it, the actual header being set is upper cased right now.
How about a bit more readable blocks?

    col_num.times do |col|
      il = Magick::ImageList.new
      row_num.times do |row|
        il.push(Magick::Image.read(file_list[col][row]).first)
      end
      ilg.push(il.append(true))
    end
I looked only at the debug-output of django-admin. Adding psql planner cost and total execution time in parantheses now:

1. current master, unsorted (total cost: 19..27 time: 0.3..0.4msec)
```
DEBUG:django.db.backends:(0.001) SELECT id, name, intnames FROM routes
                          WHERE id IN 
                          (SELECT child FROM hierarchy
                           WHERE parent = 8497 AND depth = 2); args=(8497L,)
```

2. join with implicit id and subquery (total cost: 241 time 4.8msec)
```
DEBUG:django.db.backends:(0.005) SELECT r.id, r.name, r.intnames FROM routes r, relation_members m
                          WHERE m.relation_id = 8497 AND
                          r.id IN
                          (SELECT child FROM hierarchy
                           WHERE parent = m.relation_id AND depth = 2)
                           AND m.member_id = r.id
                           ORDER BY m.sequence_id; args=(8497L,)
```

3. join with explicit id and subquery (total cost: 40 time: 0.7msec)
```
DEBUG:django.db.backends:(0.004) SELECT r.id, r.name, r.intnames FROM routes r, relation_members m
                          WHERE m.relation_id = 8497 AND
                          r.id IN
                          (SELECT child FROM hierarchy
                           WHERE parent = 8497 AND depth = 2)
                           AND m.member_id = r.id
                           ORDER BY m.sequence_id; args=(8497L, 8497L)
```

4. without subquery using direct join (total cost: 0..89, time: 0.09..0.17msec)
```
DEBUG:django.db.backends:(0.002) SELECT r.id, r.name, r.intnames FROM routes r, relation_members m
                          WHERE m.relation_id = 8497
                          AND m.member_id = r.id AND m.member_type='R'
                          ORDER BY m.sequence_id; args=(8497L,)


                                                                   QUERY PLAN                                                                   
------------------------------------------------------------------------------------------------------------------------------------------------
 Nested Loop  (cost=0.00..89.20 rows=1 width=41) (actual time=0.090..0.174 rows=5 loops=1)
   ->  Index Scan using pk_relation_members on relation_members m  (cost=0.00..80.92 rows=1 width=12) (actual time=0.029..0.054 rows=9 loops=1)
         Index Cond: (relation_id = 8497)
         Filter: (member_type = 'R'::bpchar)
   ->  Index Scan using routes_pkey on routes r  (cost=0.00..8.27 rows=1 width=37) (actual time=0.006..0.007 rows=1 loops=9)
         Index Cond: (id = m.member_id)
 Total runtime: 0.243 ms
(7 Zeilen)
```


I don't have any explanation for the time difference in case 3 between direct postgresql and django-timing. But I'd prefer using the 4th variant utilizing the indices for the join...
`puppet = packages_array & ['puppet']` ? no idea if that syntax is actually better than this or not, just throwing it out there for discussion
I didn't know that was possible.  I can change it to that if you'd prefer
`this.loggedOutService`
Do you mean up in the intro section?  (as discussed in the line 4 comment above).

I think if it's a practical point about installation, it might make more sense in the section where we install the bundles (or perhaps where we choose a content repository), but I guess it could also go in Preconditions or the intro at the top.
this should be a note for jackalope-jackrabbit like the note we have for doctrine in jackalope-doctrine-dbal
The script didn't work for me, when run in a different directory than clojure-scheme, unless I prefixed these three lines with the $CLOJURESCRIPT_HOME path
Good suggestion, but `$(LREF normalRNG)` would be better, as it'll create a link to the function.
uups, imported
Let's call it `_alignUp`. The lack of commonly needed integer mathematical functions related to powers of 2 just forced even you to make a mistake (you forget about *zero*). What about to add functions from [unstd.math](http://denis-sh.github.com/phobos-additions/unstd.math.html) to *Phobos*?
I guess the point was that 'j' is not used at all. Will drop it.
I really don't know; reading the Android docs at https://developer.android.com/guide/components/tasks-and-back-stack.html, I have no idea why we wanted a new task.  Kicking this to liuche.
still the question about the model
I'm assuming that this too is because MoneyFormatter has not been namespaced?
Also, not likeing the lower cased m on this class name. It does not match conventions.
Can't this be

    var batching = (ev.length > 2 && ev[2] === true)

Or if we dont' want the short-circuit return value

    var batching = !!(...)
does PSR-1/2 mandate public keywords for methods in interfaces?
Never leave in these default meaningless javadoc comments. Either make a meaningful comment, that conveys extra information, or have no comment.
Otherwise get this error when passing a date as a custom property:

    WindowsAzure\Common\Internal\InvalidArgumentTypeException: The provided variable 'value' should be of type 'string'
    ...\WindowsAzure\Common\Internal\Validate.php:74
    ...\WindowsAzure\Common\Internal\Http\HttpCallContext.php:301
    ...\WindowsAzure\ServiceBus\ServiceBusRestProxy.php:129
    ...\WindowsAzure\ServiceBus\ServiceBusRestProxy.php:149
Please avoid using global variables. See #301 for related changes. I'm missing the full context of this pull request, but it doesn't make sense to always disable Chef. Should be a configuration option instead?
Else on a new line please.

Also DownThemAll! will only use stuff it can download itself: http, https, ftp
ftp shouldn't be a metaurl
Can we store this in a variable instead of parsing the DOM to get the number?
Typo for Constructor
Consider moving to tear down.
Yea, I fixed that already.
How does this help? The rack response contains nothing to do with this call.
The recommendation is duplicated
デフォルト値は @DefaultValue アノテーションで事前にセットしておくのが良いと思われます。
そうするとここは Page(startPage, count) だけで済みます。
Рассмотрим пример использования пакета.
这三个diff_*.php太ugly。完全可以合并为一个。

另外，因为这些都只允许教师看，所以必须有权限保护
oh where projects-list is a file with all the projects you wanted to update?
This is a root-enabled-history pertinent change. Calling these out so your eyes don't glaze over while looking through the PEP8 changes.
is this test intended to ensure that unique nonces are generated under concurrent conditions? if so, seems like it doesn't do as expected because these 2 operations happen in serial
There's similar code in horizons/util/yamlcache.py:64, you could try to merge it into one and provide it somewhere for general use. That general function could have a parameter that is a function that processes single elements, the general function could handle everything about dicts, lists and tuples.
id = handle != null? handle.getId() : null;
This makes sense, it would be more complicated to check if the getMessage returned null or an actual Message. All getMessages now block until interrupted or a message is returned.
now that users' ids are their username in the datbase on the auth server, we dont need to look up usernames on teams (which often fails because the corpus's team isnt always set. although now that it is being set in the newCorpus function it should be fine.
This form is used to emphasize wrongness <http://www.englishpage.com/conditional/wereto.html>
There is no easy way to handle all the prompts in same index.html and seeing it in real action is for what we have it in separated files.
I think that index.html is enought understuble.

By the other way, we souldn't focus on this minor "issues" we could make this kind of reviews in another round, or maybe discuss it together in a meeting.
Your code is correct but @icholy's one is better imho
Think it's best to abort jobs if ever see a setting like this..
Could warn and reset largeMemSize to a proportion of global (if set), but I think better as errors, so they can reconfigure / change query settings.
in locals()?
Creates and returns a bitmap from resources, given its resource ID.
change it to extension
`Logger` as we go.
Yes, we have different coding styles for single method calls vs. chained sequences of method calls.
I suspect you meant art_xml here, or am I misreading how this works?
Isn't this from the bravo lib lights ? lol
Please add descriptions for the params, return value and Exception throws.
Abstracting isActive would make sense.

How should I proceed? abstract in this PR and then implement and PR it in the various children classes of all sonata-project bundles?
The above is still an issue.
The credentials returned from the auth service should include the first and last name and display name (can file a bug).
I don't refresh the page when it's changed because it seemed like that would be disruptive.
again the last line separated as a NOTE
Were you able to successfully get this as a non resident alien? We're you over 21?