labofsahil/github-event-dataset-2012 huggingface dataset:
Well `main` is just like `require` it's just marks module you start from as an entry point one, so I guess you would just use `main` on module you want to use instead of `require` that's it. Although, you will still need to make a loader instance at some point somewhere, our thinking with @autonome was that you make have a loader instance available in `browser.xul` that could be shared across the window or app.

Unfortunately I think that this specific case is not very compatible with more generic case we need in SDK, there for I think it's better to have this lower level construct that people hacking on browser can configure to their needs and just use one loader instance.
Nit: Please give the function a name. You can keep as what we had before.
aOptions could be undefined. So please ensure it's at least an empty hash.
you should use `null` instead of empty string, same for the condition below
Whats wrong with this one? You want me to indent 'Detail' ?
是否需要保证返回的putFileRet不为null，在出异常的情况是否应该将e记录到putFileRet上。
That id name is more of a javascript convention than a css one, please try to stick to either `content-tabs` or `content_tabs`.
Mentioning the ring here confuses things.  Not sure I understand the second statement.  Is it one day going to be information different from `yz/index/Index`?
So do we actually want to just leave this as it was before, recording both the snapshotBefore and the snapshotAfter, until we implement better timed snapshots?  Is there a reason to prefer giving it an empty snapshot?
oh, well.  I guess you'll need to pass these into the DOM somehow?
Tab characters on lines 64 - 70.
This tags are not longer used in ZF2
`$count` Is incremented later in code depending on the result of division
Can't this just be elseif() ?
This would be better to remain in `Zend\Http\PhpEnvironment\Response`, as that class is meant to be an encapsulation of the current HTTP response environment within PHP -- in other words, that class should be checking for whether or not headers are sent already, etc. The response sender would simply check that information in the response object.
Can this method throw a runtime exception?
The HTML patterns would be many times slower then the JS ones. But I guess every bit helps.
IIRC it's not possible as they host only `beta4` at composer which is not usable in this case.
We probably want this to be automatic, no? We probably don't even need a function for it.
I agree. Suggesting Jetty is a good idea.
Actually, all Pebbles expect the session string to be present, and will use that session string to ask Checkpoint about things.
new classes should not use the underscore. The underscore was used in doctrine1 and at the beginning of the Doctrine2 development but they have been dropped in some classes when they were refactored and latest classes have been added without it (which is the case for the post flush event for instance)
Please check ``null !== $value`` instead. Using ``isset($value)`` looks weird to me as the variable is defined.
Je to 2.7.  Co som sa docital, tak __init__.py treba, ak to ma byt package (inak je to len modul):
http://stackoverflow.com/questions/5910379/importing-modules-in-python-and-init-py
Ak mate pocit ze to tam ma byt, tak to tam pridam (v tomto diffe som to zmazal).
not related, but that's weird to have this `.././` thing.
I'm pretty sure this is because `arguments` is a reserved word, so it needs to be accessed with []
imports on separate lines
hmm, if you don't change the meaning of the boolean, it should be ``$throwException``. Currently you catch when it is false.
Please put spaces after puncuation, like `.splice(0, 4)`.
to allow compatibility with other schemas we need to be very permissive in there
But we have to include the images in the static_banners directory, so the repo already has the hashes in it anyway. And this way we can add 3.6 banners to prod without having to go through IT (which would be an issue even if we had chief because it's a setting).
Inconsistent spacing between headers= here and response = , below
it follows the 301 to /community/  so we need the trailing slash
SkillsTest
This won't work on the localized version of the site. Furthermore I think that checking the text isn't the best way of determining state of the site.
Definitely worth looking at as will be using (and already am) using the same code elsewhere so no point having to change it multiple times when one method will do.
A small typo: resubscribe.

Also, please rebase against the latest master, so I can download, sign off, run tests, and push.

Thank you!
D

PS Apologies for this taking over 2 months to happen.
auto_link is deprecated in Rails 3.1 and has been put into its own gem: https://github.com/vmg/rinku 

What kinds of inputs is causing link_to issues?
I'll replace this with pairtraide.window_length to make this more obvious.
Is this just to simplify the table name or is there some other purpose?
Should this really be passed as a string? Seems a more like an int.
If you tell a student to write as a mixed number, you should only accept a mixed number.
I'd prefer to put this into a different file :)
Align this part of the code
What about patching using what's in master: https://github.com/rails/rails/commit/c5bf53822f5887c9b235ceef388b1c8a977bcc88
Invalid HTML, needs a charset:

```html
<meta charset="utf-8">
```
Please don't uppercase your file names!
I like this README file! Well done.
In my testing, this new implementation takes about double the time of the original. Given that and the fact that we call it on the result of a potentially large database query, I'd prefer to use the old implementation in the common case.
Is there a reason why we want the logic for deciding whether there is a "newer" version available to live outside of the puppetdb code?  I suppose the plus side is that we can change the rules whenever we want, and they will affect everyone w/o modifications to their software... the downside is that it means that the logic for the web endpoint has to be a little more complex.  I could see an advantage to having that web script be a really stupid, simple, logic-free thing.  Or even just static content.

I guess, after thinking about it, that I agree that the way you've got it now is probably best.  It gives us the most flexibility and whatever logic we do have to implement on the server side may be re-usable for other Puppet products in the future.  At any rate, it might be worth discussing.
Please explain this change
Friendly amendment:

> If you've developed your website locally, you likely have information in a local database that you would like to use.  Rather than trying to recreate all that on Heroku, Heroku provides you with a task that requires you to install the @taps@ gem.  Be warned, though:  Taps has been known to raise errors with Ruby 1.9.3. If you receive any errors or the transfer fails the first time, switch to 1.9.2 to be safe.
Note: We probably want something that works in a cluster without relying on node's cluster module.
On a related note, the act of signing the URI is much different than setting the URI on principal object. The correct thing to do may possibly be to set the URI on the object here as it can be cached, but move the signing of the URI to the API, as it is only needed for a user request.
Would be good to explicitly default the limit to 10 here, rather than piggy-backing on the contract of `AuthzAPI.getPrincipalMemberships`.
You'll need to have a `var log = require('oae-logger').logger('IO')` statement if you wish to use the oae logger
Would be better to extract the "magic" number in a constant field :-)
I see. Would it make sense to add some assert using the ``pool``? Or this is already covered by some other test?
Ch. 5 v
`3-1`、`3-2` 这种方法用在 group 上我没意见，但是用在 limits 上就不适合，如果只是用 `-` 代替 `,` 那也可以，但也是用在 `data-limits` 上，而不是 `data-group`。
these should still be in the `res.send.js` file, with `describe('"etag" setting', function...)` inside, then you can do `describe('when enabled', ...)` and `describe('when disabled', ...)` etc, the rest looks good
use a class instead please
I understand, but git likes the "+* text=auto"
По условию задачи емкость ДНК должна быть 100.
Чистоты кода =) если есть ворнинг — стоит задуматься, это никакого отношения к заданию не имеет, просто увидел и зарайзил хэнд, так что не обращайте внимания ;)
Опять же вместо того, чтобы явно указывать размер массива atcgArray, можно написать: 
arc4random_uniform(atcgArray.count)
Если понадобится добавить новую букву в массив atcgArray, то не надо будет искать и редактировать размер по всему коду.
And get rid of duplicate variable warnings
`NSAssert2` is super old and not needed anymore. `NSAssert` should work fine.
I agree. Maybe implement

```objc
- (void)start
{
    [super start];
    [self addEvent:[ZincDownloadBeginEvent downloadBeginEventForURL:self. httpRequestOperation.URL]];
}
```
I returned None, events because a couple lines up, that's what's returned if the number of events found is < 1. I can change it in both places if you'd like, but it seems like they should be consistent...
Hmmm.. right now this isn't supported. The problem is that you need to keep all the mt_spectra and weights for all epochs in memory and then compute relative phases for all pairs, so you can't simply average the output of the accumulator  function.

One option to support this could be to allow the accumulator function to return a list. But this would make the block-wise processing difficult.. I have have to think about how this could be supported. Let me know if you have an idea.
Yeah. The "FFT" method was actually my motivation behind allowing it to be a vector. With the FFT method, you just specify where you want the signal to be zero, and where you want it to be 1, which means you can specify the notch filtering across all desired notch freqs in one go. The IIR method just uses a "for" loop, since cascading IIR filters is usually a good idea.
+1 on n_lines, but I do like having node_order and start_pos, I think it is quite easy to use and flexible..
don't use assert but assert_true from nose. The reason is that some production system disable asserts.
isn't it a bit high?
Needs an @ since 1.9.2, 1.10.0
A couple of other things:  in trait Appendable, the weird equals() and hashCode() methods which just defer to super() can probably be removed.  I'll double check tommorow, but it looks like the original author was using these to prevent the usual  value-oriented behavior that case classes give.  Was working very hard to make the case class un-case-class like.

Also the tests contain their own declarations of these classes, which I didn't change, I'm not sure why this was done but I'll try to look into that tomorrow.  For example FakeRuntime, TemplateCompilerSpec, and TemplateUtilsSpec contain Html, Text (not Txt) which look similar.  The tests still pass though :)
Typo here: co<b>f</b>mposer should be composer.

And I like using the `require` command for this:

    $ php composer.phar require doctrine/doctrine-fixtures-bundle:@dev

This will update this dependency and add it to the list of requirements in the `composer.json` file (as well in the `composer.lock` file)
I found the "Yoda syntax" less natural at first as well. However, the "Yoda syntax" causes a syntax error to be reported, instead of allowing improper assignment to occur, when a programmer makes a mistake and types "=" rather than "==" (or similar). 

I couldn't find anything on the symfony coding standards page ( http://symfony.com/doc/2.0/contributing/code/standards.html )  listing this syntax style as a requirement, but both control structure examples on that page use this syntax style.
yes! Thank you!
let's pass just an empty obj instead - you create different handler explicitly for both proxy tests, so this one is used only for the other tests and they don't need any proxy configuration...
Почему ты сразу напрямую драфт айди присваиваешь?
Yeah,  I used a string so it would be easy to print out in logs.  The list is never amended, so it shouldn't be a problem functionally.  It looks like I took out the log messages i used during testing instead of down grading them to a trace/debug.  I'll add them back in.
why not moving the content inside the try/catch ?
Why not just make this an array to begin with? Also, might as well declare it up around line 7
'is' (and 'is not') compare identity, rather than equality. Since there is only ever one None object, using is and is not to look for None is faster than doing an equality check. Conversely, substituting is/is not in place of equality is never a good idea. It might work, but not always.
Loving the indent.
This one neither.
This is due to the paste config nonsense. The paste config loads the service class which loads the models. The chicken before the egg problem is due to the fact that the models need the db and the paste loading needs the models. The db is configured after te paste app I loaded. There is a long comment in the src about it. We need to discuss w the openstack common team. A sensible inbetween would be to have 2 configs, one for the paste and one with the Paramus for things like the db.
@galderz I'm not sure if it's possible, if the user specifies a concrete TMLookup instance for the default cache only, how do you create new instances for each other cache?
That should be no problem
I think you should break this line at the `+`.
Ack!  If this line is too long (is it actually > 79 characters?), then the only decent way I see to  shorten it is to use a temporary variable:
```python
invtrans = self.inaxes.transData.inverted()
xdata, ydata = invtrans.transform_point((x, y))
```
The problem with this, as far as I can see, is that you cannot guarantee what filetypes are valid until you know what backend is being used, and you cannot know what backend is being used until runtime. Therefore, for many of the backends, it would make sense for ```get_default_filetype``` to do something like:

```
def get_default_filetype(self):
    default = rcParams['savefig.format']
    if default in self.get_supported_filetypes().keys():
        return default
    else:
        return 'png'
    
```
This is kind of the part I don't fully understand. What function does 0.9 serve and where should I get the old bound. What's the difference between `self.datalim.bound` and `self.datalim.intervalx/y`?
Good thing the Meet & Discuss position isn't called the Meet'n'Discuss position. LOL what a fail.
Please scap the vars when doing this check
I had a bit of a look around and couldn't see anywhere where we could match up the src of the current image with the choices available on the 'pick wallpaper' page. I'll leave it to see if anybody else has any ideas for now.
'DIt is een levend document' might be better translated as 'Dit is een document in ontwikkeling'
size=16
Consolidating a timeseries really just means taking a bunch of unordered `period,value` tuples and putting them in order. The twist here is that the timeseries will include nodata values where there's a missing period thanks to the call to `sparsify` (a wrapper for `sparse-expander`).
Add precondition that (count coll) is greater than zero. Otherwise we might get a divide by zero here.
@bp323 I don't see what is wrong with the indentation, we should use 4 spaces, right?
All the other properties use tabs for indentation, this happened by a commit from @nicolaasmatthijs for content authoring.
that documentation is not really helpful... what is A / what is B (it would totally be OK just to write some formula / explanation once in the class doc not for the method).
Should LeftAndMain.BatchActions.js:onsubmit be also rewritten to use the new mechanism, as the handler above does?
Would you please review this comment carefully?
Michael, I fixed a bug that was preventing us from skipping default checkpoints when the file was already clean, and that meant I needed some way to force a checkpoint.   So, I added the "false" configuration flag to the WT_SESSION.checkpoint call.   I could hide it as an internal flag, of course, but thought it might be generally useful?
I'll add a big honkin' comment explaining what's going on here: it's subtle.
Ok, I'll cast them straight into the template vars.
Could you please inform me about the naming-convention for bundles, bodies, classes and vars? I didn't find clear guidelines for it. The only thing I've found was the last point mentioned in the Sketch coding guidelines, which says "...prefix them with a unique string...", which is pretty vague.
This line 574 has white space.
This does not mean you cannot have multiple forms on this one page?
In user data are all data including this:

```
meta: {
    code: 200
}
notifications: [
    {
        type: "notificationTray"
        item: {
            unreadCount: 0
        }
    }
]
```
I think that we don't need this data?
What does this mean? It is unclear from the name of the variable, or from the comment.
*authorisation* -> *authorization*
I was just searching for something like that, thx
`URLS['notifications']` -> `URLS.notifications`.

For constant strings you can access like this.
OK, i return an object :D (with could could simply be moved to get_context you mean that all code move to panels.py?)
Damn it, there's always something when coding late at night . . .
Unless "int" is a standard I'm not aware of, I think this should be Integer.
Actually, the Rails API provides you an easy way to override that: define the helpers.submit.model_name.create/update dictionary key. I think these keys should also go to simple_form/*.yml file to have everything regarding forms in one place.
I think I'd rather have it raise `VariableDoesNotExist`. If the template author intended to override the url and had a typo in the variable name that might be difficult to track down. You can probably avoid this with `{% scribble 'header' FAIL|default:'something' %}`. This is a small corner case and I don't want to get too caught up in it as long as the behavior is known.
this is not case-insensitive
Un <ul> solo puede tener <li> como hijos, ¿puedes añadir <li> </li> a esta línea?
Stylistically I think I would extract a new method here for the keyToRead.equals() check to help with the readability.  Maybe something like:
````
    ...
    do {
        ...
    }
    while( resultSet.next() && isSameEntity( keyToRead, 0 ) );
    ...
}

private boolean isSameEntity(EntityKey keyToRead, int persisterIndex) {
    final EntityKey currentRowKey = getKeyFromResultSet( persisterIndex, getEntityPersisters()[persisterIndex], null, resultSet, session );
    return keyToRead.equals( currentRowKey );
}
Was Darragh's idea to use an `after_filter` instead. Way prefer this, means it can be skipped when necessary too.
Wouldn't you have to erase the register also in the other cases?
Isn't this an error?
Why isn't the process itself doing this validation and handling this directly?
event_handlers
Addressed a few lines further.
Replace with a call to ngx_http_pagespeed_srv_configure.
@sigmavirus24 you are the man.

Yes, I think that python3.3 is a reasonable additional target to python2.7. 3.1/3.2 less so.

once v0.1 is out, this whole thing will be rewritten for maintainability and reproduciblity.
хотя пишут что это тру-http. пусть будет, ок
I agree use drop to keep terms the same. I guess this means rename remove to drop in the various helper methods.
Basically everything. I'm not sure Archipel-core is using any configuration. It just provides the configuration system
Since the props are now translated into final variables we can remove it.  nice.
Is "whosul" a typo?
Thanks, I'll incorporate that in to what I had written.  

By the way, we actually fixed the convolve issue with stepk a little while ago so that it is (Sum stepk_i^-2)^-1/2 rather than min(stepk_i).  So I think that roughly gets the right value for convolutions of similarly sized objects.
Ah, good point. just pushed a change for that.
I'm curious, why is False the default?  Most of the time I'd assume people would want background-subtracted images.
(BTW, sorry for all the nitpicky comments on relatively minor things.   I think this is an amazing piece of work.)
This is a nice addition to the angle functionality.
going with list of pairs of addresses
I actually dislike the blah.tex.latexmain file (its ugly) and think it might be better to have something like a file called ".texroot" (thus hidden by default) that names the tex root. This is purely a cosmetic preference.
You don't need a google account to join the google group: https://groups.google.com/forum/?fromgroups=#!topic/google-groups-basics/P8NyycjobSs
I think this needs to be an unsigned long or it won't work for very high time values
good idea, i've tidied it up
Perhaps a better name would be anyEntityTypeIsNotAvailable
Same for braces.
Done. This should be ready for another look.
Retirar o comnetário do pdb
For i18n strings, you should pass in a dict:

    _("'%(title)s' is unavailable for re-order") % {'title': self.title}
On Fri, Aug 24, 2012 at 02:12:24PM -0700, Greg Sutcliffe wrote:
> > @@ -4,6 +4,7 @@
> >      reports        = foreman
> >      external_nodes = /etc/puppet/node.rb
> >      node_terminus  = exec
> > +    ca             = <%= scope.lookupvar("puppet::server::ca") %>
>
> that would leave you with "ca = <blank>". Would it not be better to do something like:
>
>     <% if scope.lookupvar("puppet::server::ca") -%>
>       ca = scope.lookupvar("puppet::server::ca")
>     <% end -%>
>
> which would omit the line entirely?
I think puppet::server::ca is always set (and a boolean) so I think
there's no benefit in an if around it.
well, I won't open a blog for this particular stuff but a PR for the composer doc, as it exists now...
Fair enough re-class variables.  I can see that over a period of time a namespace collision could cause havoc.  I actually rarely use them myself (but then I do use constants frquently).

By definition, anything bound to any variable that starts with a capital letter in Ruby is a constant.  Perhaps, not the definition you have in mind, but in the mind of Ruby you can easily see it:

ruby -e 'p Object.constants'

You can see this is mostly class definitions of core Ruby.  I don't really have qualms with your goal, but if you really desire constants to be unmodifiable (e.g. constant as in not changing state vs not being reassign a new object), then it might also be worthwhile to freeze the constant:

ruby -e 'Foo = {}; Foo.freeze; Foo[1] = 1'

The fact that you need to freeze to get this behavior tells me Matz did not really intend Ruby constants to be unmodifiable.  Like I said, I can understand why you may want to use constants in the way you describe but from my perspective (someone who implements Ruby fulltime), I got confused by the statement constants should never be modified.

I will update this pull request to use class instance variables.
One of the most common uses of GIT_UNUSED came from iterating over hash tables when you really only wanted the key or the value. So, I just reorganized the code and made separate macros for key, value, and key-and-value.  I'm not 100% certain that I chose good names here, so feel free to let me know if you have a preference.
yup yup...  i'm guessing vendor/* tells it that u want it to ignore vendor at the current dir level, otherwise it just thinks all vendor folders should be ignored
If this behavior wouldn't be usable without returning those columns, then this method should be `abstract`. This will force user to always implement it in own repository.
you should add that it requires the sundown extension (with a link to it)
Not a blocker, this can be a future extension, but I'd like to move to a convention where rotating into the definitive board orientation is done within the driver.  See PIOS_MPU6000 configuration->orientation.
author annotation not needed
I struggled with this one. I believe the public is fine here. For example, https://gist.github.com/2001639 this was the way to retrieve the current flow data, when the method were protected. Actually it's not sensible, you pass in the `Storage` and you set the key where to save the data. So you can always access the data anyway (by using the Storage directly e.g.).
Если threadsCount != 1 то все не уникальные мы отбросим на стадии последнего merge, иначе - тут
7.6 while Statements
A while statement should have the following form:

while (condition) {
    statements;
}

An empty while statement should have the following form:

while (condition);

Это точно исправлять? Вроде пустой while без скобок, хотя мой английский слаб и я могу что-то неверно понимать.
Please change the casing for tooltip container to be consistent with our naming elsewhere.
I don't think this is proper assert, what I expect here is a call to `listBlobs` API which gets all blob snapshots then assert that the created snapshot does not exist.
Should be JBoss Enterprise Web Server 2 or Tomcat 7.
it can not be $now. If it's now, you don't need to pass it to the function.

You should rename it to $date, should not ?
So are you planning to use `skimage.org` as the main domain name? I suspect that most people will start calling the project `skimage` if you do so...
from scipy import io

is more standard
Thanks. @ogrisel you made a comment earlier, so I though you knew what happened, sorry.
For the `assume_centered` arguments, I plan to make a new PR to address their consistency across the covariance module code.
<nitpick>full stop. Also maybe "Estimated covariance of latent variables" (or something?)</nitpick>
As we discussed, this interface may no longer make sense and could be folded into IServerRepo
Thanks for bearing with me. :)
it needs to keep top variable as the NativeControlObject property, so that when we implement 'left' property in sefLeft function we need to use also already set 'top' property, otherwise it will be reset to NULL.
These will change in future patches. All view objects will be embedded within a Cascades container.
make copy ctor and assignment operator private
You only need to include the class name for Qt classes, like so:
```c++
#include <QMap>
#include <QString>
```

I don't think there's an equivalent for cascades classes though
Also no whitespace here
Reply above.
I don't mind either way. We can also leave it as unsigned, since it's very unlikely (at least for now) that this will blow up 32bit storage.
Agree! Also, most of those methods below belong more to a directory/filesystem/utility object than a file one.
Should this be an Owned<IRowWriter>?
...otherwise the system will break once any value contains a non-ascii character.
You're twisting my words a little bit, but it's ok :-)

Your examples are strange. [`krsort()`](http://php.net/manual/en/function.krsort.php) is designed to sort arrays while maintaining keys (i.e. associations, as the name suggests) - hence it should not be used with numerical arrays (take a look at the [example provided in php manual](http://php.net/manual/en/function.krsort.php)). There are a [lot more array functions](http://www.php.net/manual/en/array.sorting.php) that will manipulate keys in an array, shuffling them around or changing altogether...  only a few of those functions are safe with numerical arrays (i.e. in a sense that you can still do a `for($x;$x<count(),$x++)` loop through them.)

>Any array with at least one string key is an associative array

I have never seen such (reverse) definition of associative array in my life ;) By this definition, the following ...

````php
<?php   
    $whatAmI = array(
        2012  =>  'Year When The World Ends' );
        -9920 =>  false 
    );
````
... is **not** an associative array. (!?)

Of course, first priority is the stability, so if any component relies on that wicked definition (@ralphschindler, chirp in ;) then we should investigate.
remove this and merge into the navigateto method.
if pass_hash is nil, you can simply evaluate pass_hash

if pass_hash
domains.uniq! so we dont enumerate the same domain from ALL and registry and potentially user specified DOMAINS repeatedly?
I think you mean ['"] without the |.
Sorry, but I don't know how to do that. What do I have to update? I have never used bento.
Sorry, I think this is what I wanted to suggest, but for some reason it came out very cryptically.
In my opinion the `Transformation` object makes it easier to work with transformations. All you need to do is an import of the `make_tform` function and you are done - instead of importing several different `make_*` functions plus their 'transform_*` counterparts. Maybe at a later point there will be implementations for more transformation types which would mess up the import process even more.

You also do not need to care about which matrix belongs to which transformation type, which would also not be trivial to determine once you made the transformation matrix. All you need to "keep in memory" is the tform object which is aware of its type. Additionally you can easily apply the transformation multiple times on different coordinate.

But if you say this makes absolutely no sense I have no problem with that. Maybe somebody else has also an opinion on this?
Can you add 2 more tests to each pair you have in this file, checking that when the model has `:validate => true`, then `@matcher.validate` passes but `@matcher.validate(false)` does not?
This has the same problem as #148, in that there is no `autoconf` in the main homebrew repo, so this will cause undefined behaviour, and we need to work out the best way of requiring the homebrew-alt version of autoconf.
Renaming `EquidistantCylindricalProjection` to `GeographicProjection` is a breaking change that should be mentioned in CHANGES.md, right?

Why the rename?  Geographic is certainty more friendly.  Is it equally precise?
I see - this is more general.  Perhaps it could be named better - `fromVertices`?
I think I asked before but why do we need this?  Is `shallowEquals` the right thing to do for positions?
Should we have a comment here indicating same number is okay because they are used for different `VertexArrayFacade` purposes?
{@link Image} is redundant here. JsDoc will already link to the type provided in {} after the @param tag.
Same problem here, you could end up moving to month 0.
On private static final Method targetMethod(final String methodName) let's refactor it to private static Method targetMethod(final String methodName) because private methods will be never overridden.
The two functions look identical to me (besides their names, of course).  Can you just recurse here?
Shouldn't this have platform: [iphone, ipad] since it's not supported by Android or MobileWeb?
IOS currently only supports type DIP and Auto for contentWidth. Also the minimum value of contentWidth is scrollView width. Have a look at the `handleContentSize` method in TiUIScrollView.m
I thought HTML5 didn't require closing "/>".
I'd rather use print(), it's the Darty way. In general, I don't want to give developers two choices to reason about for these web APIs. I'd prefer to use Dart API if it exists, and only fallback to a web only API if there's no Dart way to do it.

Regardless, since this isn't "how we'd do it in real life" can't we clean this up and include the error in the alert itself?
Yes, `"${name}.docroot"` should be unique and is more succinct, thus preferable.  

@akumria Could you please explain the following sentence in your comment:

"But not if ${name}-80 is also used for an IPv6 host."

I don't understand what is the problem that you are suggesting.
OK, fair enough. Let's make it consistent then. We can do this in a separate change after this one.
The new pattern of wrapping appears really often. Should it be injected into an Http wrapper object instead of doing this by hand everywhere?
Because the OpenEngSBForeignKey is an annotation and there are no methods for annotations AFAIK. And filterEngineeringObjectInformation is something only model related, in fact only Engineering Object related, which is placed in the EKB because of that. You can built up a solution where you use EDBObjects and never ever need this functionality.
As nit-picky as this is, this file should really be renamed to IRRrportsAPI.class.php
Following code could be moved to net.pms.util.FileUtil
@normann If you distinguish the flags ("inherited" and "extended" rather than "offlinescheduled"), the tests will become clearer.
I don't think telling it is a fork is really useful and it could be confusing. the bundle started as a fork but most of the original code has been rewritten now.
Are you sure? It's ignored in other doctrine bundles
Use a sensible name here, not an id
What has been changed here? whitespace?
Wenn wir zum Dokument auch die anzahl Worte kennne, lässt sich anschliessend den Preis einfacher rechnen!
Yeah, we generally prefer the warnings to be shown rather than suppressed. We should make sure we're receiving valid regex before saving it to the database.
This is not your code, but if we can and have the time we should move away from assigning variables in this way.
I would remove this and let the FilterException gives all details on the error coming from the process.
defining display plugin
That code is gone anyway
Here is what it look like once pushed
```
if (this.optimizeCorrelation || this.correlationKey != null){
				String correlationKeyToUse = "JMSCorrelationID";
				if (StringUtils.hasText(this.correlationKey)){
					correlationKeyToUse = this.correlationKey;
				}
				if (!this.cachedConsumers){
					logger.warn("Using optimized-correlation without caching consumers can lead to " +
							"significant performance degradation. Consider using the CachingConnectionFactory with 'cacheConsumers' attribute set to TRUE");
				}
				return this.doActualSendAndReceive(jmsRequest, messageProducer, messageConsumer, session, replyTo, correlationKeyToUse);
			}
```
please remove commented code -- no need for it
Can you use the default table name ? ``page__block``
Use `.get`. It supports negative indexes. So:

```
$('#placeholder').children().get(-1)
```

Of course in your case it would be better to do:

```
$('#placeholder').children(':last').datastream({ ... });
```
The definition of this set_variable_values2() bundle is missing.
This works --- and we didn't cover should_receive and should_not_receive, but I think that could make sense here. Just thoughts for later -- you could assert that the dealer does not receive a message to hit.
Option で囲む必要はあるのでしょうか。
case (by, null) => 
とかでよくは無いでしょうか。
in case a thread container is passed, you should not set it to ``$('#fos_comment_thread')`` first as this would do a useless DOM lookup (even if id lookups are fast, useless ones should be avoided when possible)
This is no longer used as it can be achieved inside the listener, but I think it is a good thing to add by default.

We'll need to add a listener by default on the thread container but provide a way to remove it.
github did some crazy whitespace voodoo when editing inline. Should I fix it?
sorry my mistake
Jay says "remove"
Well we probably want both right, I think that's what you said. so 2.7 and 2.6
You are right,.. I already deleted the last comment,... was a mistake on my side while reading the code
optional child selector parameter
Même chose qu'au dessus.
Done in 8586f33355223753808207aa606c01fbfb065a85.
I'm happy to change the behaviour to suit preference -- I realised when coding it that the no-arg version should do *something* and I wasn't sure what was best.

So autocheck() should return the current state, and autocheck(1|0) would set it?
うむむそうですかー
Yes indeed.  This means this does not work as expected in all the other tasks that use this.  I removed it from my tasks in 0108c04
remove the word "transition"
`Fast`
event -> events (below as well)
Since we want to report on `Procedure` records here, it might make sense to do:

```ruby
Procedure.joins(:patient_procedures) ...
```
Make that three times… and without a null or array length check.
There's only one extension, but I left the comment here because I expect we will need access to the configuration in the different actions.  In my next feature branch, I'm actually creating a plugin options class with static methods to get and set plugin-specific configurations so that we can more easily access values like this in different parts of the plugin.
Fixed everywhere in the project. That namespace is not added by default, and when VS generates method, it uses full class name.
У тебя скобка то на одной строке, то на другой - определись уж со стилем ;-)
Honestly, the main reason was rapidly prototyping where the collection wasn't ready for the View yet. But the CollectionView errors out without any collection.
I have uploaded new diffs with the changes.
Probably should add a comment describing this parses out the require statements and sends along the dependencies. Document the fact that it will not work for multi-level dependencies
Doesn't this mean clean-native is never called?
actually nevermind, just fix the formatting, there should be spaces between the { }
`(== (mod n divisor) 0)` would suffice as it returns a boolean.
To add on... and example of this would be `webview.executeJavaScript(ID, arg1, arg1)`.
Щито? Вы ж ведь знаете JS, батенька,
splice(X, Y) - удаляет из массиву Y элементов, начиная с X