labofsahil/github-event-dataset-2012 huggingface dataset:
8 is the lowest value the Wii will accept. I've tried testing different values, and higher ones were either the same or eventually worse. The overhead of the callback happening often seems to be negligible in our case.
Missing close ">"
*is sent
Remember you're not using only PostgreSQL driver, so removing these "insertid" call in these files probably break compatibility with other database engine, even if they don't work with PostgreSQL too.

Eng. Gabriele Pongelli.
I'd be very hesitant on line 11. apt-get may return "no deps to  install." or something.
> there are at least two commits
Yeah I considered whether to add checks for this but never bothered, I'll add it.
> all of them are non null
I will add a check to make sure the Commit isn't null, but in order to check all of them, I would have to remove the "break" in the loop, otherwise it will stop looking once it returns null, ie, no common ancestor found (and thus not all Commits in the enumerable would be checked). Not sure what you prefer to do?
Ok! It's understandable. Nevertheless, the form of our urls does not allow for what you describe here. We cannot just remove the "haystack-why-does-realtimesearchindex-sometimes-not-update-my-saved-object", because it is in the middle of the url. Maybe we should form the url so that the hash is before the slugified title.
init param should appear as Parameters here. See docstring of raw.py. My bad if it's not the case every where...
The function returns basically what `surfer.io.read_stc()` returns, hence "stc" in the name. There should be a better way to make plotting in PySurfer easier... I think ideal would be if an `mne.SourceEstimate` object could be directly submitted to a PySurfer plotting function, e.g. if `Brain.add_data()` would accept a SourceEstimate as first argument, of of there was a separate function for that, say, `Brain.add_stc`. Is that feasible, or is closer integration between the packages not desired?
It's actually under the methods section (note that that's the text for drop_bad_epochs() above it), not sure why they made it look like it's under attributes...
from .filter import resample

relative import
please shorten this line. Also, use a variable to hold the value of query.filter.toString().toLowerCase() rather than doing the operation more than one time. 
Another thing, is it necessary to use toString here?
Nit: In general I think the link text should include the verb. My preference for this would be:

```html
<a href="https://groups.google.com/group/mozwebpagemaker">Stay in touch through our mailing list</a> 
or <a href="https://twitter.com/#!/MozThimble">send feedback to us on Twitter</a>.
```

Counter-nit: that means the only non-linked word in that paragraph is `or`.
All code has meaning and purpose else it should be removed. In this case it is vital to Informative::Reinit() operation and must not be removed. If your compilation is producing an error, you should solve the error not comment out usefull code. In other words, this is the wrong way to fix an issue.
Openmrs convention says to expand this import
Prefer the opposite logic here. For example, if your code throws an Error, you'll log a success event. Better to initialize success to false, and set it to true after doWork returns.
I think it's really really clunky that you have to add these two parameters, which the IDE will claim aren't used in the function, just because the interceptor expects to rely on their value. Is there a better way of doing this?
When you attempt to create a pull request against of one branch into another on github, it won't let you create a duplicate.

Can we identify an existing pull request on github automatically? It would be nice to never need to worry about managing this
`read_attribute(:locale)` can return nil and calling #to_sym would raise an error; calling to #to_s will always work. You're absolutely right that `to_s.to_sym` would achieve the same; MRI calls (or at least used to call) the method intern and usually when I convert something to a string only to get a symbol I find it more explicit to say `to_s.intern` because that's how I think about the process, i.e., 'convert to string and intern the string'.

As I said, the benefit of the change is to avoid the potential NoMethodError in case the locale is not set. `to_s.to_sym` is just as fine and if you prefer this version, let's rename it.
I can't get behind this having `rescue nil`
I have updated it to reflect this.

I can't find a reference to it being a bundle's shortname or short name in the quick tour - I am all up for there being a consistent name for this in the docs though.
"Before you can start, .." (a condition is following)
Could this be an app setting, rather than a custom field?
Do we need all these statements in the test script?
These two lines seem to introduce changes that are unrelated to the enhancement you provide, so they should at least belong to a different commit.
Хз. На #qt сказали, что если const &, то передается как const &
Or this, which is just as cool:

```
[@move_a, @move_b].sort(&:ranking).last
``
mysql_fetch_row?
Would have loved to see all the specs implemented, but you definitely did enough to pass the assignment :)
Can you write a test for this ? see tests/lib/classes/module/console/test_schedulerState.class.php
This will be going in the master branch, I refer you to https://github.com/aurora-sim/Aurora-Sim/blob/master/Aurora/Framework/Utils/VersionInfo.cs#L50
Could you put a fancy name here like: "Chocolate I lØve You!"?
yeah probably not needed here
If we ignore it, why is it even used as a parameter?

On 13 August 2012 17:09, Adrian Nistor <notifications@github.com> wrote:

> In
> core/src/main/java/org/infinispan/configuration/cache/HashConfigurationBuilder.java:
>
> >      */
> > +   @Deprecated
> >     public HashConfigurationBuilder numVirtualNodes(int numVirtualNodes) {
> >        if (numVirtualNodes < 1) throw new IllegalArgumentException("numVirtualNodes cannot be less than 1");
>
> Is it worth validating numVirtualNodes if we ignore it anyway?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1236/files#r1363492>.
>
>



-- 
Manik

"In a decaying society, art, if truthful, must also reflect decay." - Ernst
Fischer
And what's wrong with the cache mode being included? :)

It has nothing to do with making it unique, but we did it to give some clues to management tools.
Actually the user could use a `PriorityBlockingQueue` if he supplied a `Comparator` (e.g. one that extracts the user-supplied `Callable` from the `DistributedTaskPart` and sorts based on some attributes of that callable).
Whilst I agree with you that there is a significant difference between SKIP_REMOTE_LOOKUP and "ignore return values", in this particular context(i.e. what's being marked as read in order to perform write skew check), making this distinction might confuse the users even more. E.g. consider key K which hashes to node A but *does not* hash to node B. The result of the following tx might be different if run on node A than run on node B, in the context of a concurrent update (i.e. K's value changes after the put operation and before the commit):
tx.begin();
cache.withFlag(SKIP_REMOTE_LOOKUP).put("K", "V")
tx.commit();

Now if this is run on node A, then the previous value would be returned (as it exists in the local container) and the tx would fail due to a write skew. But when run on node B, the same transaction won't fail. This connects the transactional behaviour to the place where the transaction is executed - which is again confusing.

Given the importance of this feature (ignoring remote values) it might be worth adding an explicit flag for this (IGNORE_RETURN_VALUES) that would be used by most of the users and would be easily to document and understand. And also keep the more esoteric SKIP_REMOTE_LOOKUP for advanced users that really know what's going on under the hood (cache loaders, locality of data, L1).
ahh right!
Just added a test for DIST and it works fine. In DistributionLogic it always creates an EntryVersionMap.
Resolved (sorry about that)
Wouldn't it be better to use `axes[0,0]`, `axes[1,0]`, ..., rather than using all these?
Is there any difference between `gc.ctx` and `self.gc.ctx`?
This PR doesn't introduce yet another special condition, it rather corrects the condition which already existed before (I even suspect the original condition was a typo, since the second clause was pointless). In this sense it doesn't hurt anything. Even if `path.get_path_collection_extents` would be working properly, this condition could be reasonable to keep.
It seems it would be more fair to acknowledge here that all but the last are designed to match their Matlab counterparts, or something to that effect (if that is the case).
@efiring said:
> Is there any reason not to remove most or all of the separate call definitions? It looks like they could be inherited from the base class.

Unfortunately some ```__call__``` methods use ```self.axis.get_view_interval``` others use ```self.axis.get_data_interval()``` so I couldn't comfortably make that change.
I may not be worth the tiny bit it saves, but there is a difference.

The initial code, which calls self.release_channels[-1] actually builds the array with all of the ReleaseChannels objects in it, which involves a call to selenium for each one. It then simply uses the last one.  The new code only calls selenium once, to return a single ReleaseChannels object, which is an object that corresponds to the last release channel on the page.
will fix, same for all
Oh yeah, I was trying to remember how we'd done the before - good call!

On May 8, 2012, at 7:49 PM, Sam Ritchie wrote:

>> \ No newline at end of file
>> +  (deseasonalize 3 [1 2 3 4 3 2 1 2 3 4 3 2 1]) => [1.1846153846153844 1.8846153846153846 2.8846153846153846 4.184615384615384 2.8846153846153846 1.8846153846153846 1.1846153846153844 1.8846153846153846 2.8846153846153846 4.184615384615384 2.8846153846153846 1.8846153846153846 1.1846153846153844])
> 
> Let's modify this test to use "roughly", like `(map roughly [1.1846 ,,,,])`, since it doesn't actually need to be THAT specific.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reddmetrics/forma-clj/pull/18/files#r793072
Again, good catch, but in this case we actually only need integers. Ceiling will handle that, but since we're loading up lots of data into CartoDB it's worth using the most efficient data type.

A good tweak of this ns would be to make lines 87-89 into a mini function.
Also the order of showSettings and widgetData is wrong
That's a lot of dead kittens! But I agree in principal.
Remove this line. No need it.
This the code that is supposed to handle applications setting truncate cursors to reference implicit or non-existent records.
Understood.  That change looks right to me.
Should also be non-nullable.
I think style_inherits_id and *_path should have the same basename, currently one ends with a "s" while the other doesnt
single quotes, same for the rest of the array keys.

also on lines 256+ below
Are you sure ``$checkTypes`` has the correct type as "array<integer,?>" is not a sub-type of "array"?
Is it possible to define the element type for the returned array (using ``array<SomeType>``, or ``SomeType[]``)? Please also check line 233.
Is it possible to define the element type for the array of parameter ``$titles`` (using ``array<SomeType>``, or ``SomeType[]``)?
Are you sure that ``$callbacks`` (of type ``*``) is traversable?
Please use `$::lsbdistcodename` instead.
Maybe this can also go on its own page
@caseywebdev probably more like :rage4:
Sweet.  Thanks @tgriesser!
I fixed a race condition between `next_tick` and *timer* in a commit below
Should this go into the [upstream repo](https://github.com/imageworks/pystring)? Also, there's two copies of pystring, one in `src/core/pystring/`, and the other in `src/apps/share/`
Nit: I'd prefer quoting the keys as well.
postMessages aren't supported cross-window in IE anyway. Yeah… :(
Same as above, cmd is the default filter.
This method is a little confusing, and while I don't see any SQL injection vulnerabilities here, it might be good to explain why, in this case, the string interpolation you're doing is OK so future updates don't accidentally introduce injection vulnerabilities (e.g. why it's OK to use Evaluation.order("#{sort_column} #{sort_direction}") in this case)
Same:

```
CLEAN_INACTIVE_USERS_INTERVAL = 1 # days
```
Finished script will be placed is home.js.
Didn't even noticed that poedit has changed the header.
Spell it in full. Not only `bid`.
Sorry, but my editor is doing this :/.
(I must say I am not sure if we are using underscored variable names or camel case.)
should split long lines with `\`
Codestyle, mon'
good to merge?


On Wed, Dec 12, 2012 at 11:37 AM, Greg Caporaso <notifications@github.com>wrote:

> In scripts/add_metadata.py:
>
> > +               make_option('--sample_header',type="string",
> > +                    help=('comma-separated list of the sample metadata '
> > +                          'field names. this is useful if a header line is not '
> > +                          'provided with the metadata, if you want to rename the '
> > +                          'fields, or if you want to include only the first n '
> > +                          'fields where n is the number of entries provided here '
> > +                          ' [default: use header from sample_mapping_fp]'),
> > +                          default=None),
> > +               ]
> > +opt_group.add_options(opt_options)
> > +parser.add_option_group(opt_group)
> > +
> > +def split_on_semicolons(x):
> > +    return [e.strip() for e in x.split(';')]
> > +
> > +def int_(x):
>
> What I worry about here is that most of the time in practice we end up
> with missing data values in sample mapping files (e.g., NA). This is
> especially common for example when using MIMARKS compliant metadata, so I
> think we'd end up overridding to allow error most of the time. I have to
> run right now - let me know what you guys think about how to handle this
> and I'll address later.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/biom-format/biom-format/pull/67/files#r2397160>.
>
>
… You know, like this.
¿Puedes combinar else e if con elsif igual que más abajo para seguir la misma convención?
double-plus-good!
I know, we have strict formatting rules.. but feel free to bring each enum to it's own line for better readability
I'm not familiar with `BytesRefArrayWritable`.. Is this size an indication of num columns / fields?
I left the ability to use env = xxx in watcher sections so old configurations are not broken.  If one of these sections is encountered it uses parse_env_str
only deleted the timer and close the connection here, didn't free object.
This declaration will cause problems on different architectures, depending on memory alignment.
You should use EigenSTL::map_string_Affine3d from here:
https://kforge.ros.org/geometry/eigen_stl_containers/file/d16897a145f6/include/eigen_stl_containers/eigen_stl_map_container.h
maybe we should cycle back to the first clip and play from there. That's how html5 video works.
picky: standard notation k_vars
Yeah. Likely some combination of incomplete refactoring and paranoia. Can remove.
These should all probably be *_params to be consistent with the rest of the package.
If you did this on behalf of your employer - would you mind expanding it to the complete company name (I guess that it's Daten-Verarbeitung-Tirol)
Sure, that's why I put "IMO" (i did not refer to International Meteor Organization, see http://vmo.imo.net/observer/code/CSETA/view :)

This is fine, but is Maven specific....
The user scenario is "configuration modification from multiple threads". The current nexus configuration is unable to handle that, as I wrote in the comment of this commit and in class javadoc.

Simply put: we know something is not working, and we also know, we cannot fix it without complete overhaul of configuration in core (see NEXUS-4909). So, this problem is not about this test, but more about config framework and it's deficiency.
Instead of doing random(), where the icons will change every time you refresh your list, you should use local id % 4.
Isn't this the non-type safe version?  Versus Collections.emptyList();
Ok.  With a big-ole note I'm good.
А зачем заменять в самом тексте hw=track на highway=track ?
я понимаю сделать так, чтобы ссылка указывала куда надо. Но кажется что менять текст введенный пользователем это surprising.
ah, much better.
I can't put the 1st 2 of those lines inside the switch statement because fluxFactor gets used afterwards.

But when I put from Solve onwards in as you describe, it works.  Question: why is it necessary?
get_full_path includes query string, good.
It seems unnecessary to delete these config sections. They don't harm the system if they are present.
Do we want this log4j.properties for software/webapp?
have made whole method synchronized
I am creating a pull request now.
Agree odd syntax with the white-space separating the args. I quite like 
the brackets though.
The syntax stems from the monterey code I "borrowed", which was 
originally written to aid some CLI parsing.

Changing it to a comma separator is worth while. I'll have a look at 
making that change this afternoon (before merging).

Medium/long term, we'll switch to a config file with json or such like, 
I expect...
ah, we forget the nice things groovy gives us.  if you just use a closure in the block below we probably don't need this.
(btw if you're eschewing groovy, you should declare this final.)
no reason to case, i think, if someone wants to claim something the pool doesn't know about.  it would be strange.
do we close when location is removed from parent ?
are there any other locations we should close, or might be leaking open ones ?
I'd prefer to use the callback or set_error_handler. Notices aren't always enabled and without the callback/error handler there is so much extra whitespace. We also don't care about the file/linenr in the Notice. We probably should add a convenience function for this in MongoLog actually...
Correct me if I'm wrong, but isn't this aggregating all the to, cc, and bcc recipients into a single list and sending to them?
Because in Ace, you can't delete a disabled app from its own namespace.
Same as with `web/app_dev.php`.
yeah, i fucked that up, that was one of my typos.

but i also don't like having 2 separate lines for different genders when
the vocabulary is actually the same...it's redundant, and really, how
important is it to type and retype 'Delegierte'?

anyway, there are other typos from earlier too.

On Sun, Dec 23, 2012 at 10:14 AM, john krauss <notifications@github.com>wrote:

> In vocab/6.exercise.txt:
>
> > @@ -47,7 +47,7 @@ die Betriebsleitung	company/plant management	n
> >  nicht sprechen will	 doesn't want to speak
> >  wenn die	if they
> >  die Staatsicherheit	(GDR) state police
> > -der/die	Delegierte	delegate (the one delegated)	n	-n
> > +der die Delegierte	delegate (the one delegated)	n	-n
>
> hmmm I've generally been making two separate lines, one per gender, for
> stuff like this...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/talos/germanforgreatgood/pull/5/files#r2496032>.
>
>
Thanks for the feedback - will update the next time I get a few mins. Will close request for now
nit: i understand why you want to align it this way but it sounds like from an old age to me :)
This also add the possibility to pass additional JAVA_CLASSPATH
well, the field **is** public, and subclassing would mean that you are forced to use the subclass to be able to use your extra field. And you need to be careful when setting the array in the child class. You cannot simply set the array in the property like here if you don't want to copy-paste the parent array
@francoisfrisch - Just pushed a commit to fix this issue.
Should this file have a `.js` extension? If not, can you change the usage comment?
Stack в однопоточной программе использовать - моветон.
Не все параметры проверяются полностью.
По этому исключению невозможно понять, что именно не так.
Just replace the whole method. It's not perfect but it is the OOP way.
It isn't, it's just that it's only a few steps further to just writing a plugin, which would have a lot more power.
you have a # here
I guess we should do that with BasicInfo and SupportInformation as well, correct?
Yes, yes it should! Thanks!
I'm still uncomfortable with `-b`, although I have no better idea. `-w` maybe? dunno :P
True, I'll see what I can do.
This is true, but we need to keep in mind that the strong basic strong rule ( the other can only be used in a some kind of path) is not very useful if alpha << alpha_max. In this case nearly all features are selected. In this case it can be much cheaper to use path then just fit for one alpha.
Why is this the case? Wouldn't that prevent to use `F1Scorer` for cross validation on a multiclass problem?
Agreed. The reverse approach seems more reasonable.
As far as I can see, this line you commented is the *actual implementation* of this function. 
Obviously I'm missing something?!
I don't remember at the moment, So I should check and put in a comment ^^
Please move this method into the `describe "Braintree::CreditCard.create"` so it's not in the global(ish) namespace. Thanks!
We use `@group` a little differently than normal to ease testing individual modules

All logical groupings in the userguide module should be listed under `kohana.userguide.XXX.YYY`
add padding around the * operator of both memeset lines.  don't know why astyle doesn't do it, perhaps it gets confused with the pointer/dereference operators.
Can you please import the class instead? :)
I've made this change to enhance consistency. 

We have already an enum named Scope to deal with @SharedPreference declared as an inner class of this annotation. This Scope class related to @Ebean was declared into the api package (org.androidannotations.api.Scope) like the helper classes and the @EBean annotation in the annotation package  (org.androidannotations.annotations).
Beware: I'm not 100% sure about this.
Why would you need to dump an expression directly to stdout in non-debug mode?
Is this here just for side effects?
You need a "return true" after this.
We most likely want to do this in __init__ now. So that we do not add listeners multiple times.
I would prefer to use `var` for all the high level code examples.
Hum isn't it only one value which is an array?
`promised(Array)(promise1, promise2).then(function(list) {`
where list is an array of resolved value of both promises?
`0 === stripos($version, 'dev-')` ?
rad, I am in IRC if you wanted to chat on other ways we could improve this!
The main assert method does take of these elements.
I like the reduction in LoC we get from this technique.
Well, we use PATCH everywhere else.  Had a big discussion internally and on Twitter back when I was designing the API.  Seems like a difference in the vague specs are interpreted.  

At least for now, I think going with PATCH for consistency is best.
I left the old code commented out so we can see what it usta do :-)
I do not agree with this. Please, tell me the concept you would do this?
Is there a chance the error will be UnprocessableEntity instead if this is called while the instance is resizing?
Not in generated doc, but how does this even compile:
```
var ImageryProviderError = function ImageryProviderError(imageryProvider, message, x, y, level, timesRetried) {
```
Why not?
```
var ImageryProviderError = function(imageryProvider, message, x, y, level, timesRetried) {
```
I created a duplicate Cartesian2 class (called ShadyCartesian2) that removed all error checking.

I then wrote a custom performance test which calls every Cartesian2 method with random data 90,000,000 times.  It does this in sets of 10,000,000 and interleaves using Cartesian2 and ShadyCartesian2 then totals and averages each 9 runs.  

Without any changes, it looked like ShadyCartesian was anywhere between 6% to 10% slower.  This seemed high to me so I looked into isNaN more specifically.  Changing isNaN to typeof 'number' calls brings them within 1.0% to 2.5% of each other.

So my official recommendation is to keep the error checking but get rid of isNaN calls (and update the coding conventions to mention not to use isNaN anywhere). This gives us almost the same level of error checking, the only exception is if NaN itself is passed into a function (typeof NaN === Number) the resulting components will be NaN as well, which is perfectly acceptable I think.  Thoughts?
did you mean ``unwarn`` when you wrote ``diswarn``?
Check again.. the help message uses compiler.. so defaultCompiler lasts just long enough to set compiler.
Should we start tests which are not found? Seems like this could lead to errors.
Block should have an `#ifdef USE_TI_WEBVIEW` around it, still; otherwise this will lead to crashes (`TiUIWebView` symbol might not be available, and besides, we then have the additional hit of subview iteration).
Maybe mention a node version (in addition to any npm packages as sptramer indicates).  I can't run it right now because `require('os').networkInterfaces()` doesn't work, for example.
This is the other method used for module import so I am keeping it for consistency.  Since this is an example, someone could change it if they like in their own config file.
This fork is already lagging the origin by a few years. My understanding is that we're not going to redistribute this library. When we do start tracking the original repo, we can either port these changes and take it in-house, or remove the Evolve-specific bits.
The `/.` is a bit confusing. Is that any different from this?

`cp -r "${sourceDir}/${dir}" "${destDir}"`
Erm, yeah. Fix this :-p
With this deletion, where are you emitting the 'PRIMARY KEY' constraint?
I think we need to either `require 'puppet/parser/functions'`, or break this constant apart into:

```
module Puppet
  module Parser
    module Functions
    end
  end
end
```

The intent is to avoid `NameError: uninitialized constant Puppet` or `NameError: uninitialized constant Puppet::Parser` errors that may arise in some of the scenarios where this file is evaluated.
it would have to be (str != null && !str.trim().isEmpty()) to be compatible with plexus-utils version... and I don't much care either way ;-)
What do you think about putting the answers file into /etc/foreman-installer/answers.yaml or something along those lines? That will make it easier when we start doing releases of the installer modules so we can put the installer into /etc/foreman-installer.
http://stackoverflow.com/questions/2566394/changing-the-input-type-in-ie-with-javascript

печальный жизненный опыт :(
Can we just use `X86_64` and `X86` instead? I really don't like how this turns into a version identifier forest.
This is perfectly fine, but you can also do
cat <<EOF | crontab -
MAILTO=...
0 * * * * * ...
EOF

This may make it easier to add stuff later, and avoids a temporary file.

Also, do you want to add a TODO about figuring out a more permanent email alias?  Feel free to catch me tomorrow, and I'll help set one up.  We should have an admin account for each of the machine roles, probably.  We have toby-admin already.
I'm not sure that putting @Inject on here is a good idea.
I did some work to make `@style/SyncTheme` inherit from the Fennec theme, so we automatically "look like Fennec".  We might not want this for settings, but we should discuss this.  In any case, `Theme.Translucent.NoTitleBar` will not look good on a `Theme.Holo` device.
Ok, finally, I found it in the test code. Guess it just seems odd to add a public method (that we may need to support). But if that's what's necessary.
Blanked it for now. I was planning to throw up some gh-pages at some point.
Change variable name to args to keep in line with other PSU code.
Forced to use console runner due to XUnit using 4.0 CLR with its MSBuild task.
well, my comment is not really accurate anymore as it is not a boolean anymore (because of ``force``)
typo in ``previously``
You should really use braces on if blocks like this, in accordance to coding conventions here: http://doc.silverstripe.org/framework/en/trunk/misc/coding-conventions
i think its better to not modify the original var as you parse through it ..
Sorry, just read #52. Yeah this is a hard one...
That quote should probably go.
Should this case be logged more aggressively? Seems like this should not happen and we would want to know if it does.
Message__&lt;?&gt;__

(compiler warning)
Diagnostics ?
This makes it very... confusing. Can we just squash it with rows 95-103 instead of splitting it up? Also, I think some of the options could directly be be placed in the parent module (except for the `objectManager` obviously).
Can you use path here.  Path is the namevar.  Name DOES work, but we prefer the namevar.
You can leave it here defined like this, but please don't use undefined in your code. I like this [description I found](http://www.gibdon.com/2006/05/javascript-difference-between-null-and.html):

*JavaScript will never set anything to null, that's usually what we do. While we can set variables to undefined, we prefer null because it's not something that is ever done for us. When you're debugging this means that anything set to null is of your own doing and not JavaScript. Beyond that, these two special values are nearly equivalent.*
What general JS library? We are doing django-datastream. It is Django based. This is what we are doing?
In headings, the namespace character needs to be escaped with a backslash, so it's ``Zend\\Filter\\Alnum``, otherwise sphinx won't render it.
Future proof.
Diego should the cfengine_version values be classes as well? and should it represent the minimum version, or the versions tested on?
Hi!

This piece of code causes huge problems with out-of-order packets, which may occur when using UDP via some intermediate network nodes. We are facing serious loss rate peaks (up to 30%) each time a message is out-of-order, like in the sequence [2,1]. In this example we expect after 3 to be the next seq number, but we get 1, so lostMessages = -2. The if yields lostMessages = 254.. which is quite a lot!

Accoding to http://www.wireshark.org/lists/wireshark-users/201112/msg00030.html, wireshark calculates packet loss via (SEQ_N - SEQ_N-1 - 1), which is exactly the same as lostMessages = message.seq - expectedIndex in your code. The difference is that they allow this value to be negative.

So maybe one solution would be to remove the "if" or just set lostMessages to 0 instead of 256.

Regards,
Tobias
i really don't think that's needed at all, though.

the purpose of allowing a function as `itemView` would be to let someone use a function to determine which item view to use... which is exactly what the `getItemView` function facilitates. 

i think the default implementation of `getItemView` should just be

```js
getItemView: function(item){
  return this.options.itemView || this.itemView;
},
```

If you see a real value in allow `itemView` to be a function, I'd like to hear why. I think it will have a tendency to cause problems with inaccurate reports from `instanceOf`, which is easy to have happen when using mixins and decorators. 

IMO - if JavaScript code relies on the knowledge of the type, the design is wrong.
@weierophinney, Well, for the specific feature that he has commented out here, init() would be the proper place to do it (bootstrap is too late for this to work properly). That said, we really just need to document it in the docs, where we can appropriately add more warnings about the potential performance pitfalls of doing stuff in init().
I should have made description that controller returns true if this key is already presented in database.
Got it. There are two large shapefiles, Financial_WGS84 and iran_road under ShapefileToCesiumLanguageTests/SampleShapefiles, which are the ones I received from Scott R. to test some less common types.  I'll remove those as there are other tests that convert the same shapefile types (just not read from a file). All the other sample shapefiles are < 190kb.
Personally, I'm not a big a fan of `XMLConfiguration`:

* I think names should be based on semantics instead of technical aspects. So the name should express *what* concept is modeled here and not *how* it is realized. Therefore I'd prefer `DefaultConfiguration` (because that's basically what this class is about, it represents the BV default configuration provided by the user), or maybe something like `BootstrapConfiguration`.
* Not having "XML" in the name leaves the door open for other potential kinds of default configuration in the future. Although I think we agree that this is not really likely, I'd still prefer to enable such an evolvement without having to rename things then.

Btw., if going with the current suggestion, I'd prefer `XmlConfiguration` over `XMLConfiguration` as it reads better IMO and allows for better variable names (`xmlConfiguration` instead of `xMLConfiguration`).
yeah that might be nice. Lets add it...
This is wrong just /mgmt/hosts/instances... no hostname in there
This is addressed in a couple of new commits. This functionality has been moved to a `module_spec_helper.rb`, and the puppet config parsing has been replaced by an environment variable
So this does not display a message if a message with the same content was displayed at any previous time?
That is, if the same message text is defined in the yaml file multiple times, only the first is ever displayed?
Rename "errors" to "userfriendlyErrors". (also server-side)
Needs standard copyright block at the top of the file.
About the `search` as we have this component right now in `input_areas we do not need to duplicate it. So i suggest to remove all the code related to this search form.
algo mas resumido, no mas de 50 caracteres... en todos los unicode
That's the difference between running update and running clean: this just clears out the version locks where clean clears out everything.  This leaves the temp directory so if a cookbook doesn't need to be updated then it won't be redownloaded.
Returns the current region setting for the device.
Performing the check this way (bottom-up on the window chain rather than top-down on the window chain) may cause a problem where modals "sandwiched" between non-modal windows cause rotation issues. Going to have to write a test to check for this.
kroll proxy -> KrollProxy (preferably with a `{@link org.appcelerator.kroll.KrollProxy}`
This issue will probably just not be solved for emulator. I'll look into this and see
how we can simulate weak references.
There's a syntax error on this line.  It's missing a comma on the end which will cause it to fail to compile.
It has been. The class is used in a single place - to extract WRAP token details from the response of authentication purpose.

The class for encoding/decoding is called System.Net.WebUtility.
what are event plugins?
Oups just see you do a PR on 1 commit only :(. good method is 1 PR = 1 branch
Foreach => ForeachStatement