labofsahil/github-event-dataset-2012 huggingface dataset:
numeric-range-filter with wrong properties names.

The documentation on numeric-range-filter have mentions to the properties from_inclusive and to_inclusive.
This properties do not exist the correct are include_lower and include_upper.
Added feature to feature dependency to scala-ide

It is needed to correctly instal scala-ide when installing
the insynth plugin.
Added single player aquarium in traditional layout

See screenshot: http://i.imgur.com/Z0rJl.png
puzzle suggestion number 8

Hope you like it.

btw, at the beginning of line 6 of pzzlr-template there is an apostrophe. Is this purposed?
Buildout travis fix
Git was committing MacOS X .DS_Store files

These files aren't necessary and shouldn't be committed. I added a .gitignore file which is the standard file Git uses to exclude files that are needed for personal use, but ones you don't want to commit, like log files, OS specific filesystem index files, etc.
2nd test pull request

Removing original human body tiff.
send() really shouldn't fail silently when getting an unknown data type

I made this edit from the github UI, so you should run it before merging
Improved tatoeba script

Preprocessing is dead slow...
myPhoneDesktop for Dropzone App Extension (Send to iPhone)

v1.0 for details and how-to see: http://bit.ly/myphonedesktop-dropzone
Build core and php from git
fixed ofMesh saving colors and normals, added binary save

i tested all these changes against MeshLab, which uses VCGlib internally to parse the ply files.

i was considering replacing the save function entirely with the binary version, but decided against it because the load() doesn't support binary yet.

if this pull request can be done, it will help a ton with my next class at ITP where we're covering 3d. having built in load/save for ply clouds and meshes makes a huge difference.
completion block

Hey, thanks for releasing this as open source.

I added a completion block to the CPAnimationStep. see if you dig it.

r.S.
INTSAMPLES-76 Wait For Server

Add utility and @Before methods to all tests
to wait for the server to start before starting.
Moved unit tests from .Net 4.0 sln to .Net 4.5 sln
All classes in separate files.
Use HTML syntax highlighting for HTML in the documentation
Add support for key_type
LPS-25289
check-repo-permission
Typo Fix

* minor typo fix in tinymce init block
Arquitectura del proyecto

arquitectura del proyecto en 4 capas
Zeus rebase 12_11_12_a

This pull request contains code for parsing a new sequence named "BestFitSequence".
link to updated SR1a extras site >> master
Improved extraction of annotations from comment

This PR is related to [PR-94](https://github.com/pulse00/Symfony-2-Eclipse-Plugin/pull/94) of Symfony-2-Eclipse-Plugin. It should be merged after the PR in the other repo is merged.
Knowledgebasedemo1

plugins/knowledgebase/usecase/demo1
build-test.xml
build-test-plugins.xml
portal-web/test-ant-templates/ant-script.xml

http://issues.liferay.com/browse/LPS-24829
Test Fixes

controlpanel/blogs
controlpanel/bookmarks
controlpanel/sites
portal/controlpanel/usergroup
portal/controlpanel/messageboards
portal/controlpanel/webcontent
added count comments in NewsBundle
Update 4.1.md

代码运行时,无法正确输出
修改:
func login()下添加一行
r.ParseForm()
3.1 modifications, updates sentences relating to .png files
Minor refactor of comment model template

Cheers,
  Lee :beers:
Fixed typo under "What is 'el'?"
basic obj support

Merge it.
Fixed IE Javascript error
Addition for annotation based toggles

Ryan,

Thank you for your help in this process.  Please let me know if you have any questions or concerns!

Thanks,
Michael Minella
Fix edit button display in delete template
error with '

3 ' cause nice error, removed one(i think because you want to enter "empty" value, and work ;)
fixed northwind tests, changed namespace

Fixed some bugs in northwind tests and added new version of Northwind.Common.dll

Refactored namespace of MySql dialect test project to ServiceStack.OrmLite.MySql.Tests

Additionally I fixed a string escape bug in MySqlDialectProvider (unit tests are included).
Remove unused makefile from ui.dialog extension

This file was left over from the native extension.
Implement notification feature.

* Not ready to be tested, waiting for unit/functional tests.

Implement notification in blackberry.notification and attach the object to window.

Fixes issue blackberry/BB10-WebWorks-Framework#166

[Docs Link](http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-notification/view/topics.html)
Add missing execute()

...as pointed out by fREW himself during his presentation at YAPC::NA 2012. :)
Added login/logout links to application.html.erb

yeah buddy
Adapter

As discussed per mail, I added some extension point using the adapter pattern to make it easier to integrate with other frameworks.
The Zend adapter for the access to request params and cookies could probably be improved as I'm quite sure they have an abstraction for cookies too but I never worked with ZF and so someone else will have to improve it later. Currently, it only does special things for the request param (like the previous implementation)

Sorry for the whitespace changes. My IDE is configured to strip trailing whitespaces automatically when saving. You can view the diff without them by adding ``?w=1``.

The Symfony2 bundle will be available soon
Adding Group 3 info
wanted to add a complete android project with cordova 1.6.1. includes jar files
Added a DebugMap class to print multiple properties, added comments to the Debug class
Без ошибок в коде

Без ошибок в коде
New Dialog methods for Navigation Mode

This pull request contains new signatures for functions that display the specialized dialogs introduced in Navigation Mode - date/time picker, color picker and single/multi select.
Adding back in a sneaky little comma that got away from me in the README
Two commits, for better integration in CI tools

Hi Marina,

this pull request includes 2 commits, both detected when integrating your report in the reports run by the CI server:

1) I've noticed that the CLI (and the checker in general) requires one moodle installation to be running (config.php, DB... and friends). That's not really a problem because the whole "ci" world is, in fact, one "local" plugin too, hence, it's always installed within one moodle site.

But, it needs to be able to run the checker against any other directory. The CI server continuously gets git snapshots from integration.git, and they are stored into different "workspaces", none of them being "runnning" moodle instances (and different from the "ci" moodle server where the local/moodlechecker is installed".

The first commit enables that, so it a full path is passed and it's valid (file_exists), then your checker runs against any "remote" directory (aka, not the own where it is installed. Seems to be working ok here.

2) In order to make it a correct checkstyle xml file, the xml output was missing the xml preamble and the root element. The 2nd commit addresses that by introducing one new attribute to the class, in order to differentiate the "main/root" element from the rest and insert the missing xml code. I know it makes the possibility to check multiple paths from CLI and the XML output somehow incompatible, perhaps we should restrict such xml output to just accept one path. For your consideration. But I needed well-formed XML for proper integration, so I did that way.

Soon I'll start to auto/pre-check all the issues submitted to integration with it, I'm just in the steps of generating the final report with all the information gathered.

Also, surely, we could offer such reports for 3rd part plugins. Surely we would need to run some more checks (all those already existing in your new "plugins database" plugin). But I really think that it is pretty doable. In fact, the "plugin database" itself could trigger the execution of the complete tests automatically or whatever. Let's see, but could be interesting to have all the checks for any piece of code centralized and applied everywhere.

Sorry for the blah, blah, blah, lol, just wanted to share how I'm viewing the "whole picture" :-)

Ciao :-)
Fix input not being parsed with the good date format. (issue #14)

Use the parse function of the datepicker
2e fichier exploration de Crawford
alternative: you can use ProjectSkyFires current authserver (master branch)
This might clash with the interrupt handler implementation (which uses hooks) if someone mistakenly defines both BUILD_LUA_INT_HANDLERS and LUA_REMOVE_HOOKS. Also, see the comment related to patching from https://github.com/elua/elua/commit/b174ab4757bea3cc71e0b75b7516f364609d2dcf
Thank you. My bad. 



On Jan 7, 2012, at 19:26, Alex Chaffee<reply@reply.github.com> wrote:

> bug: this should default to nil, not to "index.html", otherwise apps can't ever choose to serve "/" dynamically -- see issues #301 and #302
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rack/rack/commit/bd89414a45a5c1d2612b1790d482276f190fa14a#commitcomment-846699
If nothing else, this breaks all 6+ years of prior documentation on generating models and application. This seems inappropriate and unnecessary in a stable release.
Good news ;)

Well, now I see : basically, we didn't configure anything yet, main reason for this behavior.
I'll write an issue and explain how and why we'll work on this repository ASAP, a suggestion for our future flow.
There's actually an AMD branch in steal where this is being worked on: https://github.com/jupiterjs/steal/blob/amd/amd/amd.js

You should start from there.
Rails 3.1's asset pipeline should render the "asset" tasks (`lib/engine_tasks/assets.rake`) obsolete, no?
Sign into chat
	Call phone

 
   More
Sorry I meant "nullptr"
double backticks for inline literals
Changed, it makes more sense now.
Thanks, accepted
It has been agreed to keep the master stable.

I've got two regression reports regarding car handling from vdrift forum. I don't think I can resolve them in a timely manner. See also: http://vdrift.net/Forum/viewtopic.php?t=1735

I'll see if I can allocate some time to look into it again.
mostly ok, but the ; in the defines is useless, you want to have the semicolumn in the normal code flow instead. Also, whats the usefulness of all the ldc. -> ldc_* defines? 
Also, the blocktime is very needed for de-bouncing keys, although I doubt that its currently working. However it should, so adding the code for that is better than removing it. Originally, it was a non-code blocking denial of keypresses (per key) that discarded clicks within the blocktime after the last press. It also defined the automatic repetition time if you keep it pressed.
Yup, colored cluster markers back in FF + Chrome
http://groups.google.com/group/theano-users/browse_thread/thread/1723c95eed128a69 this doesn't do anything by itself but an optimization should be written so that the regular tensordot is replaced by this. Frederic Bastien mentioned that he will look into this.
I try on different emulators and devices. Still unable to reproduce. getContext() is always activity for me and runOnUiThread() works fine.
All done, thanks! Check the latest version of helper.
First committed line to github! Ha?!
``len()`` is a little bit of a non-Pythonism (don't you love it when I nitpick, I know, I know). ``if not getattr(obj, 'Results', None)`` would probably do the job.
This looks good. The only thing I'd say is to also remove the next pointer in hid_device_ since it shouldn't be needed anymore.
There is a FileMatchers class in sisu-litmus
Nice. I think this will get rid of some warnings in the test suite too. I don't think it liked the pre text.
Seems the comment and the see also section refer different function
这里的多余空行去掉
Ай яй яй!!!
@logicmd ok，我只是以为一般调用的时候至少应该出现函数名，比如：`search(:q => tag1,tag2)` （我猜的。。），如果不是的话就忽略我~
596c9c9
@sunaku, do you know if I'm implementing this `refresh` properly?

When I try to update the status bar using a basic telnet connection it usually takes more than 10 seconds. I would like to set this type of notifications for `Tork`. I guess a configuration helper similar to the following should do it:

https://gist.github.com/1831795
Good point, I think this helps readability!
So I went ahead and wrote that particular function myself. Here it is, if you are interested: https://github.com/vedang/emacs-config/commit/e9869e0111e3b6071c4363483e63d432f97511f1
Yes, there should not be zero-length vectors coming here, but one never knows, and it doesn't hurt anything.
ThumbnailGenerationEvent dosent generate tumbnails anymore, if them already exists. Force support (for regeneration button)
This makes Bulk add many times faster if there is images which already exists and have tumbnail.
I'm not understand reason of this patch.
This has been updated to remove the extra restoreSession code.  I have not done the XL() strings on the old text yet.  Also, it only works in FireFox, which was true before too, but not so obvious until to add multiple page + multiple fee sheets.  In both IE and Chrome the page length controls and the page breaks are broken in different ways.   Any thoughts on how to fix that would be interesting to hear. --Tony
the above can be replaced by

    var db = WarsiDatabase.instance();
    db.prepare();
In PsGet I also using similar technic to modify TabExpansion from module. However I am concerned of what situation when module is unloaded with Remove-Module? TabExpansion will remain changed. This always occurs when PsGet updates module. It forces it to reload. I am trying to revert it back in $Module.OnRemove event. But this will make things even worth if something already changed TabExpansion.

Idea that you use with Prompt seems to be much more reliable. If everybody can agree on common name of the global variable that will keep all prompts and remove them in $Module.OnRemove event, looks quite promising.

What do you think?
I think basically its a big GO GO if you arent considering fallback.
and caution if you are gonna use flashcanvas/excanvas
got to check this
Great job~
I should add: I want the full name, so that users of the pyspotify can easily turn on/off logging from pyspotify without having to know all the different logger names we use.
This introduced regressions that were fixed in 44356ee04efd7ae4d109129db724b9a01684faaf 

>.<
@themoob it's not mentioned in the commit message...
what do we need this for, actually? it'd break the architecture if used.
why do we need `base/users`?
onLoin ;)
Login

(And yes, I know this name doesn't matter...)
I think it is overruled because it was intended as osha only :)
```
+'descriptionmsg' => "Bundles several anti-spam extensions together, along with blacklists and other settings to provide protection from spam", // Same as above but name of a message, for i18n - string, added in 1.12.0
```

descriptionmsg is only needed if allow localization of the description message (in which case you need to have an i18n-file etc.). Having both ```description``` and ```descriptionmsg``` is redundant, remove the ```descriptionmsg``` one (which is invalid right now).
Your patch for drivers/usb/gadget/Kconfig has duplicates.
comments should end with periods.
Don't worry, it takes time to iron this stuff out. Will test and merge your change.
Ditto the above line. Should be:

```java
if ( def != null && ( result.getUserId() == null || result.getUserId() < 0 ) )
```
co to za zmiana? nic Ci nie popsuła?
You can move this line and line 37 since there is no code outside this block.
Yup, I'll be cleaning it up this evening though, so no worries :)
Day 1 of Ruby...Pumped.
Blank line before this "paragraph" of code.
Too much information in one line. Let's break that into two for easier comprehension?
Good point, I've added it and will be committed the next time I commit
Should be, "Made it not run as root"
add spaces after commas, break into 4 lines for clarity.
80 columns do you speaketh???
I do not disagree with your opinion VIncent, but I think you miss at least one important use case. In the extension we talk about recently, that is coded in Java and Groovy outside of wiki pages, we use code in velocity to do work that consist in customization of the main purpose. For exemple, creating forms, with validation, customizing livetables, etc... And I see no reason not the have the best tools available there. The idea is that these codes are more like content than API, and more easy to tune and access for those who use our extension.
Where you you suggestion I move the queue interface then?

On Feb 28, 2012, at 5:33 PM, Nate Abele<reply@reply.github.com> wrote:

> Hm, I wonder if my last comment just disappeared into the ether. It's not showing anymore. Anyway, I suggest moving this to `li3_queue\storage\Queue`.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/UnionOfRAD/li3_queue/commit/00a5e21092b8b94d20bfa5c355b2523176487d27#commitcomment-1023466
sorry @kikito but your pull request did not seem to be applied to the 2.0.0.rc branch. I wish I was a github expert... :-(
Do not merge with this commit.
and opcode CMSG_COMPLETE_MOVIE

replace to STATUS_UNHANDLED
Autocomplete.Suggestions.showSuggestions vs Autocomplete.Suggestions.show
что думаешь?
This commit effectively shortens the heavy code.
But if we consider optimization, dateWithNaturalLanguageString is actually a very slow method... and Apple documentation "strongly discourages" its use.

I consider refactoring the code, using some statically stored NSDateFormatter
Nah, I'd rather people scrutinize what I've done to make things better, as I tend to cut corners to get things "done enough" for myself.
1. that stylesheet is outdated, e.g. there's a change in `master` to the styling of the `textarea` using `clip` that is both way simpler and won't result in a 1px by 1px blinky cursor in IE

2. there's a lot of stuff that aren't immediately obvious at first glance, or there's a lot of stuff that's cryptic and doesn't make sense even upon careful reading it and needs to be commented? I don't want to comment every line, and it should be clear at least what each CSS rule is _supposed_ to do based on the class selector. You can always `git blame`, but please do mark things that you feel need commenting
It should be ```rstring.format```
Ralft,

Você está certo, eu vi a nota agora. Me tira uma dúvida, por favor.

Nessa montagem de usando SIGCB e emitindo pelo www.ecobranca.caixa.gov.br ambos colocam o codigo do cedente na posição 05-15 (nào tenho certeza aonde encerra) porém no Manual (CODIGO_BARRAS_SIGCB.PDF) na pagina 7 e no indice 4.1.8 mostra que a posição 05-05 é DV Geral e 06-09 fator vencimento e 10-19 valor documento.

O que está acontecendo ? Ja visse algo assim ? Estava pensando em reformatar para o padrão fornecido, mas depois que eu comparei o Ecobranca fiquei despreocupado, porém o mesmo não conseguiu gerar o exato mesmo boleto. Ficou parecido.
Could you solve this by introducing a BufferPool? The only expensive part of creating a BufferedFrameWriter is in allocating the 1MB buffer streams, so if you let a BufferPool handle that (which, for reconnects, will likely reuse orphaned buffers), you can move the delegate Stream into the constructor.

Thoughts?
Should be "decreased" the amount of stars.
This constructor is surplus. Please keep the POJO minimal. The empty constructor is used for deserializtion.
If you must have this constructor then make it call this() constructor with a default count value.
forgot to remove the original <title></title> block
This seems dangerous. Doesn't that mean it will silently attempt to write outside the sdram area if there is less than nr_chans*0x80000 bytes left?
Haha, I understand.  The original reason was because a possible employer suggested I set a github up with some of my projects during an interview.

I'll take that advice to heart, though.  Thanks
i think one of the great thing about osrm is the speed of routing. so i hope this can at least be an option. preprocessing is already fast, and at least for copenhagen we would much prefer maximum routing speed over preprocessing speed.
uy ansiedad
Wouldn't it be better if we had the z-axis values represented as well? If this is more for a human reading, rather than computer, then trying to visualize layers would be rather hard.

So for example on this map you would have two files:

### A Tile Type file
```
XXXXXXXXXXXXXXX
XX###########XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XXXXXXXXXXXXXXX
```

### And Tile Height file
```
000000000000000
006666666666600
005000000000500
004000000000400
003000000000300
003000000000200
001000000000100
000000000000000
```

I don't think we will ever need to have tiles overlayed.
If we do, I think it would be later down the road rather than right off the bat.
El .sql no lo he puesto pero tiene el muro de sindra y templates de los adds en 25.
> CDataStore__GetInt32(a4, (int)&v34);
  > CDataStore__GetInt64(a4, (int)&v35);
  > CDataStore__GetInt64(a4, (int)&v27);
  > CDataStore__GetInt64(a4, (int)&v30);
  > CDataStore__GetInt32(a4, (int)&v31);
  > CDataStore__GetInt32(a4, (int)&v29);
  > CDataStore__GetFloat(a4, (int)&v26);
Typo on here, it must be:
+            if (itr->first->IsImmunedToSpellEffect(GetSpellInfo(), effIndex))
Would it be better to introduce a scope instead?  That way query can't be used after it has been cleared.
Does this code really belong in daclient.cpp (linked into every dali client, afaik) rather than (say) daliadmin executable?
TINYINT(1) is the correct way to make a boolean. Note that BOOLEAN is an alias for TINYINT(1).
Instead of creating a test listener, you can mock a ThrowableEvent listener and then verify that notifyOfThroableEvent got called (e.g. using Mockito.verify with an ArgumentCaptor).
specifying required=False has really not effect. SchemaNode.required is a property. It returns False if you didn't specify a "missing" parameter.
it's not a major issue and i would love to see it work with rdoc, but if it doesn't, i'd prefer to switch to markdown and link to the last build result so people can actually investigate on the problem. it's not a big change and markdown is build for these things, so ...
Actually, what I meant from the comment was since we get rid of broken requests immediately can't we just check for the node.attrs.rid value in queued requests being less than session.rid?
We can return responses whose rid > this.rid (in case out of order requests arrive).
Not quite a sufficient fix. See issue #32
It seems dumb the a "none" vote is even tracked though.
Code review **accepted**
I'm not absolutely sure but I believe you would call tms.delete_transform(transform_id) here
Yes that would make sense, we should preserver backward compat with deprecation warnings + update whats new too.
In order to get access to HttpRequest GET/POST (or REQUEST for convenience), following examples should be used:

 - check the HTTP method:
```request.method == 'GET':``` or ```request.method == 'POST':```
 - get attribute value:
```request.REQUEST.get('key')``` gives value as a string or None or ```request.REQUEST.get('key', '')``` gives value as a string or empty string.

We could make few helpers to make the code prettier like here http://djangosnippets.org/snippets/826/. Alternatively string could be handled directly in the controllers.
even better: n_informative_features
Was ist der Sinn dieser Anweisung?
2012-03-07 13:11:05 [SEVERE] Error occurred while enabling BetterMechanics v2.0.0-prealpha (Is it up to date?)
java.lang.NullPointerException
	at net.edoxile.bettermechanics.utils.PermissionHandler.<init>(PermissionHandler.java:70)
	at net.edoxile.bettermechanics.BetterMechanics.onEnable(BetterMechanics.java:54)
	at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:215)
	at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:336)
	at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:363)
	at org.bukkit.craftbukkit.CraftServer.loadPlugin(CraftServer.java:342)
	at org.bukkit.craftbukkit.CraftServer.enablePlugins(CraftServer.java:324)
	at net.minecraft.server.MinecraftServer.t(MinecraftServer.java:422)
	at net.minecraft.server.MinecraftServer.a(MinecraftServer.java:409)
	at net.minecraft.server.MinecraftServer.init(MinecraftServer.java:223)
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:477)
	at net.minecraft.server.ThreadServerApplication.run(SourceFile:490)
Wat?
```c
case 0x5Bu:
    6D7E40((int)&v245, this, a2, a5, a3);
```

leads to: 6C7C40


6C7260 is for 0x2E0C (11788), atleast in 4.2.2
thanks for clarification :)

cheers
Yeah I know, but handlers don't have a reference to the connection. Instead of finding out in which order the handlers are executed, and storing a ref up to the point where on_tls_init is called, it is clearer to provide the handler with what it needs. Same as the other handlers are set up. Ofc you could modify the signature and pass the connection, like the other handlers. But you don't need a connection to set up an ssl context, you need the io_service, so this is more direct!


Lars van de Kerkhof
lars@permanentmarkers.nl
tel 06-24352007

On 31 jan. 2012, at 12:34, megabyte1024<reply@reply.github.com> wrote:

> Do you need access to the io_service inside of on_tls_init? If no, then every connection has a reference to the io_service, for instance,
> <pre><code>void CTestHandler::on_open(connection_ptr connection) {
>    if (connection != nullptr) {
>        boost::asio::io_service &service = connection->get_io_service();
>    }
> }</code></pre>
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/specialunderwear/websocketpp/commit/77a2208501cb88957d9fa4483d33c00b0435cd7a#commitcomment-917903
Wojoo er gebeurts iets aan de robot!
Hooray ! :)
Why doesn't the _adSort in phobos just reference the one in druntime?
True, thanks for spotting that out. I never knew about using the triple equal signs in javascript, and I will definitely start using JSHint from now on.
Makes sense, thanks for feedback.
Yup thanks for reminding me of this, remembered hearing this somewhere before but seem to have completely forgotten.
Will fix with next commit.
well ``findRouteForUrl`` will usually need to be overwritten .. to handle a custom way to map urls, though of course this might not be necessary. but if we do not make this abstract .. we should review all the places where we currently specifically reference PHPCR ODM in the comments
when rmnet is loaded already it causes the data arrows to not reflect data throughput.
Hört sich aber iwie immer noch sehr uncool an. Vielleicht "Sonderpreise"?
Send another pull request?
It's weird, but it works due to the circumstance that it can escape %" to something. Should we put it back?
Awesome; thanks for changing that!
This commit has not been tested...  it's a work in progress
Shouldn't this be merged into develop as well?
arguments for type, pkgname, pkgversion and commit are not optional.. use '=' instead of ':'
I don't really understand this.  Could you describe a little more why this is needed?  I see the same block duplicated above.
Shouldn't this be `config.body = data`? To match what it does in Node.js?
An API change like this that changes the default behavior and that is not backward compatible REALLY needs to be in it's own pull request.

In addition, a change of this significance really ought to be discussed on the rib mailing list before hand.
"Single Lists" might be better than "Single"
I’m not currently fit enough to think through those fallback thingies, but all this will be marked “experimental”, so I think we can sit together at a later time and try to find a consistent spec (not saying that this wouldn’t match it, but …).
These two companion objects were put in place to get rid of strange errors during compilation.  I was able to duplicate with both idea and sbt.  I tried removing them and recompiling, and the issue is gone.
Why include this case at all then?
These give me the shivers
Perhaps add tests for the individual settings (and not only the resulting values)?
AbstractPartialFunction?
We don't need this now because lib/wagn/set.rb loads all the renderers as well as the sets.  (the view loading needs renderers to work right, so they come earlier)
A lot of this is my fault. I will at some point go back and clean it up, if someone doesn't beat me to it. But I think we should merge much of this in order to update the api and then go back and cleanup the backend after the release.
Small thing: when logging, specifying the arguments for the format string as arguments to the logging function means that the string will only be interpolated if/when the the log event is emitted. eg

```python
log.debug(u'User %(follower)s started following group %(object)s', follower=follower.follower_id, object=follower.object_id)
```
route matching may want an unescaped representation of the path.
Still seem to be missing output check.
Ok, i would then prefer that the whole start stop time calc is done there.
Don't have half in decode and half in get overlay.
Je zou je error_reporting even op e_notice kunnen zetten tijdens ontwikkelen, dan vertelt Php je dit al.

error_reporting(-1);
This is good, but i think you can re-phrase it to be a liiiiittle more concise.
phenotype_attempt.production_centre.try(:name).to_s
Not currently, and not on any system I work with, but Daniel built the facility into the wheel spec, so I'm reluctant to not support it when it's so easy to do so.
What's the rationale for returning None? That the pid might be gone when you ask for it eg. if the child dies?
Since it's a current issue to remove implementation from the header files, we should probably not add more instances of that. See issue #111 . https://github.com/cyclus/core/issues/111
But I returns the defaultAddress. You say it should be localAddressForRemote then?
No need for the extra braces in this if-else thingy
Would `NSNotFound` be more appropriate here?
My inner grammar-nazi tells me me to append 'Over Issues' to this title.
Yes, IsEmpty() is not required but isString is required, otherwise Utf8Value tries to convert  everything to string for example "null" which is not correct. So  IsString check is required
please add "newOutboundMsgBuf" as message of the NPE.
The .cmdline call was the best way I found to test the accuracy of the wrapping.  This is *probably NOT* what Nipype wants to have on the web documentation (at least it's not what is currently there).  Personally, I'd like to see more "command line"-based testing, but that's just IMHO.
``nargs == 0``, if defaults is a None comparison ``not self.defaults is None``.
> Maybe just inserting "when the" before "new base".

Done, thanks.
Charles Harris wrote:
> Don't really need the else since the other branch returns.

I tend to believe that the else makes things more readable in general 
but I can change this if you prefer. Is it to minimize the overall diff?
This should be a config option with your values as defaults (but note that the prefix should be `gxp_`, not `gx_`).
OK, but "HttpOnly" for exaple is sent as "HttpOnly" by ext/session and "httponly" by `setcookie()` - which one do I keep?
replace ``check_id_map.py`` with ``%prog``.
пришлось убрать, потому что при сборке ошибка вылетала на момент летней школы, как мне помнится.
закоментированный код
Removed the extra whitespace, but the original note is on the wrong line now.
Yes.
db:test:prepare => db:test:load => db:test:load_structure
db:test:clone => db:test:clone_structure => db:test:load_structure

Both tasks called db:test:load_structure.
haha sorry for the spanglish
Done! 9ad8d670ec53eab64f0e0f0a248c2b11ebd79cc8
See line 1307.

... the test may be bad, I'm not an expert at these tests. It fails without my code change and passes with it, though.
@carlosantoniodasilva you would think so, but from what I can tell that method is totally busted. Look for yourself and consider the situation where a model has:

 `:counter_cache => true`

messed up right? The problem is `cached_counter_attribute_name` should just delegate to [`Reflection#counter_cache_column`](https://github.com/rails/rails/blob/master/activerecord/lib/active_record/reflection.rb#L229) which is what I am doing directly. I looked at fixing it as part of this pull but it broke a bunch of tests so I decided it was out of scope. I would like to look at fixing it in another pull.
put a blank line above
Wdyt about describing it with something like:

```
Fix bug with presence validation of associations, which would incorrectly add duplicated errors 
when the association was blank. Bug introduced in 1fab518c6a75dac5773654646eb724a59741bc13.
```

Might still need some rewording :)
s/*/-/
They can, but they have to come from somewhere, right? I would expect to run into the missing class before the missing type.  But they can be TypeSymbols if there is some advantage.
Nice spot there.
It used to, but (and I'd forgotten I even did this) I rewrote it in java.  So it's all in scala.runtime.Statics.java.  Obviously if this is being changed around that file becomes very relevant.  Sorry it's not documented; I only meant it as an experiment in whether true static methods would be faster.


    scala> case class Foo(x: Int)
    defined class Foo

    scala> :javap -v Foo
    Compiled from "<console>"
    ...
    public int hashCode();
       0:	ldc	#63; //int -889275714
       2:	istore_1
       3:	iload_1
       4:	aload_0
       5:	invokevirtual	#23; //Method x:()I
       8:	invokestatic	#69; //Method scala/runtime/Statics.mix:(II)I
       11:	istore_1
       12:	iload_1
       13:	iconst_1
       14:	invokestatic	#72; //Method scala/runtime/Statics.finalizeHash:(II)I
       17:	ireturn
It seems useful, yes.
Maybe we should have a separate class in that case which composes 2 `Hashing`s together.
What property exactly is it trying to identify? Here's an implementation which should under normal conditions identify anything which originates in a jar under the running vm.  How exactly this will interact with symlinks etc. I don't know.

    class JdkClassIdentifier extends (String => Boolean) {
      private val javaHome         = sys.props("java.home")
      private val loader           = new URLClassLoader(Array[URL]())
      // Since I'm not sure of the intent behind the code I'm
      // writing, I don't know whether pre-filtering based on
      // package names is desirable.
      private val knownJdkPackages = "java javax com.sun sun" split ' ' map (_ + ".") toVector
  
      def apply[T](implicit ctag: reflect.ClassTag[T]): Boolean = apply(ctag.runtimeClass.getName)
      def apply(name: String): Boolean = {
        knownJdkPackages.exists(name startsWith _) || {
          val path = name.replace('.', '/') + ".class"
          loader getResource path match {
            case null => false
            case url  => 
              // At this point it looks something like
              //    jar:file:/Library/Java/JavaVirtualMachines/1.7.0.jdk/Contents/Home/jre/lib/rt.jar!/java/lang/String.class
              // getPath drops "jar:" so we remove "file:" - presumably if there is no jar
              // involved, getPath will drop "file:" instead.
              url.getPath stripPrefix "file:" startsWith javaHome
          }
        }
      }
    }
Yes, that's the right thing to do but we aren't there yet. I've created a ticket for that: https://issues.scala-lang.org/browse/SI-5819
"Plot real, imaginary components"?
I'm not really sure what is going on here.   Perhaps there are tabs in the original file.   These have been replaced with spaces now.
Maybe "Integer(ENV["API_PORT"] || ENV["PORT"])" ?
I personally think this looks a bit crowded. How about a space after the colon?
This isn't true in general. ``I**I`` is real, but you can change the base and get a complex number, e.g. ``(3*I)**I``.
Should be `S.Zero`, with a capital Z.
On Sat, Apr 7, 2012 at 2:45 PM, Tom Bachmann
<reply@reply.github.com>
wrote:
>>      else:
>> -        raise ValueError('(%s).series(%s, n=16) gave no terms.' % (f, x))
>> -    return series
>> +        raise ValueError('(%s).series(%s, n=16) gave no terms.' % (e, x))
>> +    if f == e:
>> +        return series
>> +    return 1/series
>
> Does this work? [I can't see how, but ...]


Serendipitously, I guess, since the following fails:

```
>>> eq=x**16/(x**20 + 2)
>>> limit(eq,x,oo)
oo
>>> eq=x**16/(x**16 + 2)
>>> limit(eq,x,oo)
1
>>>
```

Is the following true: `limit(z, x, x0) == 1/limit(1/z, x, 1/x0)` ?
OK, `.header.hi.tex` can go to a file like `default.css`. For `.header.framed`, I'd rather leave it as is at the moment.
Is @ really needed here? The fewer @ the better, in general.
Yeah so my question has been answered -- they both do the same thing, but there's a philosophical difference in their use cases.

I'm opposed to this because you no longer have 'one obvious way to do it.'  If I have to add some lines to a `let_before` method, do I also need to change its decorator to reflect the more complicated nature now? The purpose of the decorator seems incredibly subjective to me.

If we were to prescribe less complicated setup methods, why would we not just say 'use `setup`, but try to limit the amount of work you do'?

Additionally, I can't think of any case where being able to swap between `let` and `let_before` would be useful. There's no functional difference between the two, and one is clearly more performant.

The whole thing just raises more questions than it answers.
I'd pass the estimated block size in instead of the entire conf and just determine the size in ColumnSerDe.initialize.
"。" is duplicated.
Although I'd strongly prefer a regex literal here, either way it should be defined outside the function, since it doesn't depend on the returned anonymous function's input.
Guess its not, didn't realize that 'model' was a pointer to a stack buffer... It's on the stack, right? I rarely look into this kind of stuff (C,C++) and it's been a while...

I debugged with GDB, got a bit confused there when i saw that cpu_info->model referenced an invalid address before i realized that 'numcpus' had been overwritten from 1 to 0 after the call to sysctlbyname on line 264 and that loop was never entered...

In my mind i thought that since strdup takes (const char *) as an argument and didn't realise that 'model' was a pointer, so cast it..
Yes. It is used just to make the getAuthority method return a nicer result.
please resolve this.
ah, thanks for catching this too
Yes, I could not figure out what was happening at all. I did a complete search throughout all the files in the directory for is_enabled and it did not find any
Space between else and if
You should also probably make sure the class implements the interface. So:
```php
if ($method instanceof phpbb_notification_method_interface && $method->is_available())
```
Otherwise, there's no way of making sure you're not about to make a call to an undefined method.
"Path to the file to which access is controlled"?
No, because `empty(false) === true`.
bc7a826
Nu ar trebui să ne bazăm pe message_string. În schimb, la activity de la qotd, ar trebui să adăugăm action, ceva gen qotd-ok și qotd-wrong.
indentation mismatch?
а почему прозрачный? я красный люблю
I'll make these changes (even though I strongly disagree with the type-before-name documentation style).
this should probably be 2012 (or maybe 2011 - 2012). @everzet thoughts ?
This mutex lock is necessary and will not be removed.  It protects the bufferList.
Shouldn't this be "path" rather than ctx.Request.Path?
This test fixture contains a lot of dead code
Why the conversion to array?  Also probably better to use the log call that only creates the string if the log level is high enough, but it seems the SLF API doesn't support that.
This problem has stumped me for a while. This is a partial solution to allow building of expressions that need asciidoc. 

For asciidoc to work, it needs the docbook DTD and stylesheets in `${XML_CATALOG_FILES}` for when it calls `xmllint`. 

The `${XML_CATALOG_FILES}` environment variable is created by the `libxml2` `setupHook` from all of the `buildInputs`.  So this works for any expression that uses asciidoc as an input. 

But that's only half the problem. 

As you say, this does not work for users who install asciidoc to a their profile, as we don't run the setupHooks for them. 

So we can leave XML catalog management up to the user, suggest running asciidoc by `pkgs.myEnvFun` overrides or wrapper the a2x binary to use a default bundle of Docbook DTDs and stylesheets.  We need to be careful to allow the users to override the docbook version that we use.

How is this problem solved for perl, which has a similar problem with `${PERL5LIB}`? 

Does it make sense to call the setup hooks from the user's shell initialization, where $HOME/.nix-profile/bin is added to the path?
This is getApps for the smart apps-- Shivam, add 'laboratory' in the getModules function right above this.
Are these functions used elsewhere, other than login? If not, probably don't need them in global.js and can put them in your controller.
Don't call `tests.get_result_path`, it's done for you in `assertXMLElement`.
```_coord``` is undefined. More importantly, if ```src_coord``` is circular, but this coord isn't why are you treating it like it is and what modulus are you proposing to use?
Were you planning to fix these dead links in this PR, or later?
I generally prefer that functions that do re-allocation within the C interface are simple enough that they either pass or fail, and indicate that by returning a valid or NULL pointer.  The purpose of this convention is to relieve programmers of the burden of having to clean up memory mid-way through a computation when an error occurs. C doesn't make it easy to enforce RAII, so conventions help.

I won't block the merge over this, it's just something I wanted to bring up for your consideration.
Andrew didn't see a reason to support $ in role names.
These two patterns should have \a to anchor the search in order to avoid accidental matches when there's text between the pre-pattern and these patterns. Even if the code avoids the issue I think it's better to make the regexes bullet-proof against changes to the code.
typo of 'repair'
Ditto.

Or move the initialization of "doc" down so that there is no risk that code added at some point later between initialization and check breaks.
That str() call is most likely not a good idea on python3, using %d should work better.
this should be removed. It was used in the SVN area
Yes. Especially the `fragstate()` type. The RFC is quite nitpicky about what input is valid once a fragmented message has been received. The initializing and finishing state are required to ensure that the websocket_dispatch/4 function can handle the message correctly. In the initializing state it should initialize the payload buffer to the payload of the first message. In the finishing state it should append the payload, and dispatch it using the opcode of the first message.

The `bufstate()` type is added because the status of the input buffer was lost when the loop returned to the handler_loop function. Even if the input buffer contained a full message, the handler_loop function executed the receive expression to receive more input. The `bufstate()` type is a very simple state machine that enters the `nomessage` state when it has failed to decode a full message from the head of the buffer, it re-enters the `mustcheck` state each time more input has been read.
Could you extract this comment and create a geonode-purge.sh script?
Can you move the brace to the same line as the if statement (PSR-2 compliance)?
I worry about that, especially on an exposed method. First because the inconsistency will decrease compression efficiency, and second because aliasing issues can be so subtle... for example, the test at https://github.com/jquery/sizzle/blob/fbdc818520b93474c5eb0f35ce20df68fb369e4c/test/unit/selector.js#L275 should have been failing in IE *before* the commit fixing jQuery bug [#10799](http://bugs.jquery.com/ticket/10799), but wasn't because the NodeList returned by `getElementsByName("test")` had a `length` property of the HTMLInputElement, causing [`Expr.find.NAME`](https://github.com/jquery/sizzle/blob/d47b174ef344eb4ac30482ca6308893ba023f2d6/sizzle.js#L259) to erroneously return `null` instead of a single-element array.

However, we can replace all the `(elem = arr[i])` loop conditions with `(elem = arr[i]) != null` for no increase in gzipped file size if the former is too slow.
If we merge this file only, code will be broken.
This comment is unclear to me. If you change the name of a function that is only there for historical reasons, why not just get rid of it?
The assumption that the state will be an array of length 8 is somewhat suspect.  This will be the case when it is properly initialized by an aggregate, but individual functions of can be directly callable.  When called directly the parameters can be anything that a user provides.  

For instance, what is the behavior if a user calls:  ```select chi2_gof_test_merge_states(array[1],array[2]);```?

If this will reference initialized memory then it needs to be fixed.
Instead we could just `next(); return;` in case we don't rind Meteor.accounts.oauth1._services[serviceName]
Maybe add ^\s* and \s*$ around this regexp?
Just remembered that this should be unix timestamps instead of isoformat(). That also means every call to ".replace(microsecond=0)" can be removed.
Is there any open bug for that in debuntu? Reference it and create it if missing?
You can use a heredoc to avoid the really long line here:

```rb
cause <<-EOS.undent
  clang does not support unqualified lookups in c++ templates, see:
  http://clang.llvm.org/compatibility.html#dep_lookup'
  EOS
```
That loops through keys, you'd want `for data_dict in data.values()`
HAAAAA, pas de chemin absolut comme ça non mais :-p
#311 addresses this.
perhaps ... if min(days) < 1 or max(days) > 7: log.warn("Invalid combination for 'recurring': %r, 'days': %r (recurring_type, days)
I know you want a 1-liner, but I really don't like the poor grammar. If `Raised when a logged in user is not a moderator of the subreddit` doesn't fit, then I'd write, "Raised when the user is no a moderator of the subreddit."
`content_tag :span, "オススメ", class: "label label-info"`
とかやるときれいかもです！
Actually, at this level, the interface makes no guarantee as given in the existing text.  An aggregator at run-time decides that.  Implementors of this interface would implement either the codegen or the reflective (or other) strategy.   So I just removed the extra text.
Yeah - sorry.  You're right, it's clearer, though I'm just working against hitorical style, so it'll take a few to get comfortable.  Sorry.  

But in this case, I'm not sure I can guarantee which numbers will be in which sets, as I'm not sure there's any deterministic ordering of calls to the providers.  So it could be 1,2 or 1, 3 in either set.  Since method order is not guaranteed in JDK7, this test would break.
Please match surrounding style (space after if, omit braces where possible, spaces around operators.
heya @erictj could you add support for min_upload_date here, to pass in the newest we've seen, so that we don't re-crawl all the photos every time?
Also, should document the format of the shortcut.  Looks like ```string|{key:string, platform:string}```?
Spelling nit: "occurrence" with an e
Okay, I was following the "putting everything in 1 test so it runs faster" comment
5 seconds seems like a long time to wait. I think 1 second should be sufficient, since we're only waiting for the browser to refresh the page.
Nice suggestion! I make a minor change to your suggestion for consistency with other strings. I use "Live Preview: Click to disconnect (Save file to update)"
I will keep it back to `*`. Also changed for `*Makefile`.
Dont we want to solve this in treehugger? What if a language doesnt use a '.'?
But I wanted to show it the first time the user is using our IDE
Oh yes. This was for me to validate behavior, and was never meant to be in the final code. This is what really frustrated me about the accidental push.
It seems like user_id should be an explicit keyword argument here.
This is useless without `ModelForm`.
*created_time* or *create_time*?