labofsahil/github-event-dataset-2012 huggingface dataset:
adding active directory example for recursive group membership support

Added example to documentation to support nested groups in Active Directory (groups as members of groups)
synced with latest mojolicious deprecations

please update cpan
thanks tobi
Bugfix

Fixed to work properly when many pages with both footer and header are created.
Internationalization docs update

Related to https://github.com/symfony2admingenerator/AdmingeneratorGeneratorBundle/pull/230
And +1 to the commit message :-)
@mdomba CDbException does not replace the {error} with the params, you need to add it manually. Simply concatenate strings.
Thank you! Can you add germanMessage to the export list?
Ah sorry, this commit was already pushed with Ablu's review: ba22f31c36a845553c495ce4604826532ed55600
OK, thanks, let's merge it.
@mscdex: Yes, via close() and listen().  Get the address when you pause, and then when you .resume(), listen on that address again.

It's better not to have this here because a) clearly no one needs it (it's been broken for a very long time, and no one seems to have even noticed), and b) pause/resume have another meaning in the Stream interface.
He meant " and not ' ^^
I would prefer that you extract a private method for example _index_difference(original, sorted_list)
Is there a reason this print statement is here or was this left here accidentally? Noticing some extraneous output while testing.
i think you did not read the second point correctly. it says "there should
not be two or more windows of the same app shown in the gnome3 panel window
list. if you have two or more windows, and you click on the app icon, the
utility window should show up in the front."

i cannot do this because hime already has several windows for setup. so
this is not related to this commit.

i can design a better ui for hime. but i think tetralet will say "we need
to discuss", and caleb- will say "this makes it difficult to sync with
gcin". XD

On Sun, Feb 5, 2012 at 4:37 AM, Tommy Tsim <
reply@reply.github.com
> wrote:

> Ref. http://developer.gnome.org/hig-book/3.0/windows-utility.html.en :
>
> > Utility windows, such as palettes and toolboxes, normally have borders.
> They do not contain a menu bar, a toolbar, or a statusbar.
>

> A utility window should not appear in the panel window list unless it is,
> or may be, the only window shown by an application. Otherwise, the utility
> window should be raised above the application when the application window
> itself is selected from the window list.
>
> In my experience, "unless it is, or may be, the only window shown by an
> application" clause does not applicated for hime-setup. You could imagine
> the situation by omitting the "unless clause". Feel free to modify the code
> to satisfy the "unless clause". :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/caleb-/hime/commit/8beaa68a98a67a254ff9d4976272c0473eb18214#commitcomment-933888
>
And might I add, the warning that was attempted to be fixed by MYPGM is a bug in GCC. This was fixed in 4.6.2 as far as I can see. (I had a gcc-bugzilla number somewhere, but I cannot find it)
I mean in the context of [3027](https://github.com/symfony/symfony/issues/3027)
#2509
That problem is already fixed it just hasn't been pushed yet.
This is actually incorrect. Inline methods shouldn't be marked as inline here; they should have the inline keyword where they are defined (i.e. at the bottom of the header).
Though using the inline keyword here produces no error message; but it has no effect.
So I can drop rhapsody as well.
@nahi: Maybe you can help me with this? I prepared the Gemfile so that everything would run automatically with TravisCI. That's why I replaced the local paths with the respective git repositories. Works like a charm when I run either

bundle install
bundle exec rake spec

or

rbx -X19 -S bundle install
rbx -X19 -S bundle exec rbx -X19 -S rake spec

But if I try

jruby --1.9 -S bundle install

then it won't package the gem correctly, aborting with the message that it can't find Manifest.txt nor lib/kryptcore.jar.
That's really weird. As far as I understand, bundler tries to do a 'gem build <gemspec>' with the files retrieved from the
git repo. If I run 'jruby --1.9 -S gem build krypt-core-jruby.gemspec' in the krypt-core-jruby directly, then it will work as expected.
But when bundler downloads the files, it somehow won't generate the manifest and it won't compile the Jar using ant.

Have you got any clue or even a solution?!
IIRC it is to keep eio from blocking the event loop too long.
Hmm, OK
is it save in this situation to cast split to ColumnFamilySplit without even checking first?
ya +1 for clearer code comment :D
I'm disappointed by this commit, unless it fixes a bug. Does it fix a bug? I also wanted to get rid of the code duplication, but @rwldrn had asked me to wait till grunt was integrated before making further changes to the test suite. My plan was to have php invoke grunt, or something like that, each time the test suite were run, and only when you don't have PHP use the document.write fallback hack.

We had discussed in a jquery meeting that PHP was a requirement, but that we should make a base attempt at supporting testing without PHP. This change prioritizes the fallback method and loses any gains from having PHP installed.

The point of using PHP was to concatenate the source down to a single file, which also gets us testing the same source code as the built version of the code. Instead, we're back to including multiple files via document.write in every case. Also, I needed that single include file to test some work I'm doing on a ticket that uses a lazy loaded jQuery. When a lazy loaded script uses document.write to write multiple script tags, execution order is not preserved, so I guess I can't test that now :-/.
This makes a very noticeable/sizable flash/jump in the scrolling when you expand a tree node because the expansion instantly creates a whole screen height's gap between nodes before collapsing down to the children that actually exist. If this height is needed, maybe we could have it conditional on if there is no priorPreload?
Hi,

> Ah, in my defense ;)
> 
> -> isohybrid --help
> Usage: isohybrid [OPTIONS] <boot.iso>
> 
> Options:
>    -h <X>          Number of default geometry heads
>    -s <X>          Number of default geometry sectors
>    -e --entry      Specify partition entry number (1-4)
>    -o --offset     Specify partition offset (default 0)
>    -t --type       Specify partition type (default 0x17)
>    -i --id         Specify MBR ID (default random)
> 
>    --forcehd0      Assume we are loaded as disk ID 0
>    --ctrlhd0       Assume disk ID 0 if the Ctrl key is pressed
>    --partok        Allow booting from within a partition
> 
>    -? --help       Display this help
>    -v --verbose    Display verbose output
>    -V --version    Display version information
> 
> Report bugs to <pj.pandit@yahoo.co.in>
> 
> There is a -o option (version on openSUSE 12.1)
> 
> At least I am not completely insane ;)

that's also crap because I call it with "-offset" which is
incompatible with "--offset" what the heck are those people
thinking ?? I'm really fed up with this crap. How should
we make this to work in any case and that's just because
those /zensored/ don't respect any option rules

It's not you it's all isohybrid's fault

Regards,
Marcus
-- 
 Public Key available
 gpg --keyserver gpg-keyserver.de --recv-keys 0xCCE3C6A2
 -------------------------------------------------------
 Marcus Schäfer (Res. & Dev.)   SUSE LINUX Products GmbH
 Tel: 0911-740 53 0             Maxfeldstrasse 5
 FAX: 0911-740 53 479           D-90409 Nürnberg
 GF:  Jeff Hawn,Jennifer Guild, Felix Imendörffer                  
 HRB: 21284 (AG Nürnberg)       Germany
 http://www.suse.de
 -------------------------------------------------------
this is 86 characters long
When this switch is depressed, what will the shooter rollers do?

The [javadocs](http://www.wbrobotics.com/javadoc/edu/wpi/first/wpilibj/buttons/Button.html#whileHeld(edu.wpi.first.wpilibj.command.Command)) claim that whileHeld will "cancel" the FlywheelRun command, implying that its inherited "cancel" method will be called, which we know will not disable the motors on its own nor call the end() method.

So any idea what will happen in this instance?
I guess I did not understand that it's similar. From a superficial look it looked quite different.

The better. Let's use what works. 

I don't think a stand alone module is better, that's not what I meant. That's fine.

If you want to send me some sample code, I'll be happy to roll some Common Tests for the exceptions.

Thanks,
Henning
Why do we need to pass anything like guiOptions or platform here? We can just query that inside the method via ConfMan, since we know the target from the first parameter.
Something went wrong with the merge. In order to close RAILO-554 is has to be re-integrated (wrong location of brackets).
Do I create another push? It's just IE compatibility is important for Windows platform (not every admin would agree to install something in addition to stock IE).
Oh hmmm, could you do an illustration with a 3x3 grid?
I think you should be consistent a keep the same abstraction level within one method. You already had evaluatestring and runprocedure outside the main evaluate method. Too big an evaluate method would also obscure its clarity imho.
Naming the sub evaluates as evaluateX still keeps hinting to the recursive nature, although maybe not so much in your face.

Regardless I still think switch makes the different paths much easier to read than the conditionals hidden as tests with if methods.
maybe is better to use db table without hard-code hackig this spell changing
all from sniff? GJ :)
I wonder whether we should allow for PyUnicode here as well?  I presume we want a utf8 string for searches etc?
Would also be great to check if extension can be build and loaded into Slicer properly. 
Checking with the existing extensions a first test, building EMSegment as a real extension will allow us to validate that all path are properly set. If the following code is still working:

* https://github.com/Slicer/Slicer/blob/master/Base/QTGUI/qSlicerScriptedLoadableModuleFactory.cxx#L77
* https://github.com/Slicer/Slicer/blob/master/Base/QTCore/qSlicerLoadableModuleFactory.cxx#L63

It should be fine.
I don't think copy and pasting classes from one library into another library is a good idea. What if someone wants to use ignition (e.g. for the widgets), but use AsyncHttp instead of IgnitedHttp? Of if they use some other library that depends on AsyncHttp? They will receive duplicate class errors when linking both libraries. This is what happened when we bundled Google Guava's MapMaker classes with Droid-Fu; people then complained that they weren't able to link any other 3rd party library which in turn also had a dependency defined on Google Guava. The result is dependency hell.

Moreover, I'm not sure why it's necessary to have support for multipart HTTP in ignition. Android builds on Apache Commons HTTP, so does IgnitedHttp, and there's a library available for Commons HTTP which adds multipart support. If you send multipart requests, you can use Apache http-mime and Apache mime4j. In fact, that's what we do in our app. ProGuard will then take care of stripping out any unused parts of any given library you add.
Removed getObject: c16857b723
\begin{description}[Décentralisation]

To align the description environment to the largest described word.  It's not done automatically ... and it made me sad during the presentation.
Please note that we are working on [a separate doc site](https://github.com/clojurewerkz/monger.docs) similar to http://rubyamqp.info and this documentation in the README won't be around for too long. But we can keep it there for now.
This has already been done. See paradoxxxzero/qtile@890832b480cc2f11faf6c05b35b77b42e5750566.
Well, if someone asked for a specific number of eigenvalues, I think we should return that or fail. This only really matters for matrices of sizes 2x2 to 7x7 or something. Let's not worry about it.

-viral



On Jan 8, 2012, at 11:14 PM, Stefan Karpinski wrote:

> Shouldn't we maybe just call that for them instead of failing?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/d4d995bd64d9a04914052678aef901f8f9c3bcb2#commitcomment-847782
Could shorten that to `(mappings (divisors i) i)`.  The `Fn` impl of `PersistentMap` includes an arity with default arg.
Mit welchem Commando kann man das Rendern?
I cannot follow your link. Thank you anymore.
Needs 73 and 74
I wonder if maybe we're relying on a selection change event that only gets fired if we explicitly set the selection?  (just guessing)
You stole my migration number!
this was a bug anyway :)
I see, thanks for the tip. Coming from OO languages I've been trained to use the more general verb.
So I took a stab at portability - but don't have a Windows system with which to test.  Where I used eventfd/read/write to do AOI, I ifdef'd in CreateEvent/SetEvent/ResetEvent.  Seems to be the equivalent functionality.  See upcoming commit...
Before, when both slacks were null, we stopped with an ``AssertionError``, but now, we pass. 
I don't think it matters, though, since Assertions are disabled anyway.
просто это exception, а не Controller->View, вот я и того

http://www.google.ru/codesearch#HVBvXbZbclo/uriplay-persistence/trunk/src/main/java/org/uriplay/persistence/servlet/StatusCodeExceptionResolver.java&ct=rc&cd=12&q=extends%20SimpleMappingExceptionResolver&sq=

вот нашел решение в лоп! statuscode внутре Exception
I don't have a simple enough example right now, but if I create a widget with a parameter that is a list of references to other widgets, I get extend running in an infinite loop during extending of unrelated objects.

This commit fixes the bug in Google Chrome, as does replacing AjaxSolr.extend with jQuery.extend, so there's definitely an issue with the simplified extend function in AjaxSolr.
I replied to this in the main pull comment. Anyhow if you didn't see a negative one, why specify it as integer in the format string? ( line 175 )

```cpp
const char* spelldifficultyentryformat = "niiii";

```
"addon" should be "add-on" when used for copy
Locale based.
una vez que entra no me redirige al catalog, me redirige a edit_customer_path, pero es buen la observación ^^
Con el tema de comprobar con el alert que aparezca, no se como comprobar el mensaje que tiene dentro, es una de las tantas preguntas a hacerte ^^
suggested syntax:

`- if authors[conversation.id].present `
@sarahmei I see.  Being the type of person who likes to know beforehand, I am not too happy with this communication approach.  But on the other hand, (a) Diaspora is in Alpha and so I am actually test-driving it, and (b) you guys put in work to make this happen, so I accept it.  I do still hope that Diaspora's approach to communication - and maybe decision-making - will change somewhere down the road, simply because I prefer things done differently.  But, as I said, at the same time I am thankful for the work you continue to put into this.  And I am looking forward to learning more about this UI change.
cool, texting
Response to inline comment
doyou have an example of how you would do it?
So is there any way of knowing who this all powerful admin users are?
Still the same crashes over and over again ... You just swithced Null into 0 which is basically the same ...
isnt it just firefox? (same for chrome ^)
Can you add a private constructor to prevent any possible instantiation:

```
private GestureLibraryFactory() throws IllegalAccessException {
    throw new IllegalAccessException();
}
```
I pretty sure that it's better use `window.document` in the invocation at the bottom b/c using just `document` there lists it a as global on jslint.

But I'm still debating between:

```javascript
window.Response = (function($, window, document, undefined) {
   // ...
}(jQuery, window, window.document)); 
```
or simply the technique that jQuery itself uses:

```javascript
window.Response = (function($, window, undefined) {
   // ...
}(jQuery, window)); 
```

We only need to reference `document` once inside (when we cache the `$(document)` selector) so maybe the latter is better. Either works.
Hi Evert,

I found 2 or 3 while reading code and trying to understand permissions and ACLs. Then I decided to fix them and send you a patch, but thought I invest a few more minutes to find some more ;-)
I'm using PHPStorm, there is a Spellchecker available as Code Inspection. So I can easily run that Inspection on the whole project. I only checked the lib/ folder and went through all results quickly.
I highly recommend PHPStorm. Which IDE are you using?  PHPStorm is free for Open Source Projects...
Ty SignFinder
The usage of block generator is the same as content generator. In the Gii module, Click on the menu block generator and fill the block name.
HEH... 8 \
jo jeweler generiert en Gemfile wo auf gemspec referenziert wird.
wir habens damals rausgenommen. macht ihm nach hinein aber
keinen sinn, da wir die dependencies doppelt pflegen müssen ...
im gemspec stehen alle nötigen dependencies drin.
is this necessary or would jcr:lockIsDeep be enough?
I tested and reported all by using UDB, which in my point is more blizzlike than ytdb and much more complete in almost all points
Well, those ones could be added also (though, with direct database access).
Maybe "minion" could do some checks once in a while. (If I understood minion role correctly).
Ошибочно съехавшая dependency
This isn't working properly for me unless I revert the filetype back to 'htmltwig'
I appreciate the desire to keep the dependencies to a minimum (it's far better than the opposite "including everything you can" philosophy), but I'm not sure that I see many advantages to pulling multi-json in like this.  You still have an external dependency on either json-gem or yajl-ruby; so this isn't removing external dependencies entirely.  In the app I'm working on that uses json-schema I need to use multi_json for some other stuff, and I'm concerned about potential weird, subtle problems if two versions get loaded (i.e. the gem-installed version loaded via `require "multi_json"` in the app code, and the version vendored and loaded here).  Vendoring a gem so as to avoid it as an external dependency (rather than relying on rubygems doing what rubygems does best) seems like not a good idea in general to me.

That said, ultimately, it's your project, so feel free to do what you want :).  I can always maintain my own fork without this.
ok thanks for the explanations, I've added them to the JIRA issue.
:trollface: :thumbsup:

Sorry the rendering fix couldn't make it into final. Maintenance releases should come fairly quickly and I hope to get that resolved ASAP.
Так быстрее отображается. :)
old bug) is fixed =)
Commenter pour expliquer pourquoi on ignore le  `|` stp!
digit returns -1 for invalid character.  maybe us isDigit to verify?
The one thing that is different is that I removed the updated_at, created_at and id from the project associations in the JSON for the Client#staff_plan_json.

I'll try to put it in again and see how it goes.
My mistake, I will fix this right away

On 02/19/2012 02:10 PM, Kenichi Saita wrote:
> What is "GitExtensions/bin/Debug" file added by this commit?
>
> This file prevent me to build GitExtensions.exe(debug version).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spdr870/gitextensions/commit/27223fa7c9b8b38f700886e91fad93cad1ceca28#commitcomment-982471
>
>
Den rename bitte rückgängig machen
Un-comment and use `assert !`...
It goes over by 1 character. How hard is the 80 character limit?
querry -> query

"Each player also exposes a popcorn.youtube.canPlayType( container, url ) that developer can use to querry if their data can be played or not"

Should probably be something like:

"Each player also exposes a canPlayType( container, url ) method that the developer can use to query if their data can be played or not."

Can you also update the players now adding in the canPlayType method? We should also link to it here when we mention it.
Don't forget to rerun cmake or this will still not work.
I believe this was a mistake: the onmessage on `Socket.prototype` is a handler that can be used to extend the functionality of engine.io. as such, it should imo receive the same information as the 'message' event.

the ws complicane @guille was referring to has to do with the `socket` object in the `WS.prototype`, referring to it's own websocket socket. Since both objects (the `socket` and `Socket`) have an `onmessage` handler, I suppose those two were simply mixed up by the author.

I recently tried to figure out the same thing and started to sketch a simplified call graph with dot. you can find the file in `/docs` and a rendered version [here](http://oi44.tinypic.com/2h7qnpd.jpg)
Branch created : BUG-3732-Digital_Everywhere_can_cause_locked_CAM
omg... thanks :D
Was this just moved from the wrong plugin section?
Superb! Can't wait for the day when the spells are not hacked all over the core anymore
Fontes -> As fontes
The container should contain ThrowableCaught objects. Events are transient objects and should only be created when fireEvent is called.
Nice work Willem! I'm starting to get into Git a little bit ;) I do think it is quite a hassle on windows machines though, maybe we could create a homework assignment like "submitting your code to github", for getting the ssh keys installed on each machine will probably take too long ;)
on line 14, int i is Initialized for limited tries for unsuccessful log-ins.
Comment all 3 out..
http://pastebin.com/4Gpb8tCD
Could not print in landscape mode due to swapped PdfOrientation enum.
This test passes on 32 bit.
Visual Studio issues a warning on Level 4 with:
"Your class was padded due to alignment"
56 is 64 - 8 (pointer size on 64 bit)


On Wed, Feb 1, 2012 at 6:31 PM, mikhail lozanov <
reply@reply.github.com
> wrote:

> Why 56?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kingofthebongo2008/vitosha/commit/3a039ef7590a7852fd817296d4d460e1cc94274e#commitcomment-922963
>
Always try to use `===` instead of `==`.
Read more about it [here](http://www.jslint.com/lint.html#eqeq) or [here](http://geekswithblogs.net/brians/archive/2010/07/03/quality-equality-with-javascript-quotquot-gt-quotquot.aspx).
We also note this in our code guidelines section in the [README](https://github.com/blakeembrey/jquery-lifestream#readme) of this plug-in.

It's also a good idea to run your final code through [jslint](http://jslint.com/) or [jshint](http://www.jshint.com/).
These will spot these kinds of things for you.
Is there some reason to not have this in a TestClientRecord class?  It is only testing ClientRecord, after all.
It would be easier to read and maintain if this was just one PHP block, like:

    <?php
      if (...) {
        if (+) {
          $result = ...
        } elseif (-) {
I prefer the former - (harvard-university-of-xyz) - that way all
harvard styles are together, makes sense to me.

On Wed, Jan 11, 2012 at 2:47 PM, Rintze M. Zelle
<reply@reply.github.com>
wrote:
> @adam3smith, we have quite a few harvard-(some university name) styles. Do we have any preference on what format to settle on?
>
> harvard-university-of-wolverhampton.csl or university-of-wolverhampton-harvard.csl ? The former is currently more common.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/citation-style-language/styles/commit/af442165b3c202828d2232901415d452616ae723#commitcomment-858794



-- 
------
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
:-) Any idea if this one fails in any browser?
You're missing one set, plus I just merged some other changes that might conflict. Mind redoing this fresh? I'll merge it as soon as you do!
only partially, ie: 911

this really is not the appropriate way to be doing it lol
It would be good to keep that next event section even if it's just the
date. I'll get confirmation of the speakers now and then we can update it
ASAP. Looks good!

On Thu, Feb 2, 2012 at 11:19 PM, rodnaph <
reply@reply.github.com
> wrote:

> Craig, what's the right thing to do?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/unified-diff/unified-diff.github.com/commit/50bd3bb6f29fb076a055271fdfb76e307cdf0556#commitcomment-928290
>
okay, i will try :)
Original fix by Mook : http://bugzilla.songbirdnest.com/show_bug.cgi?id=22246
němělo by tady být  `$this->status = static::STATUS_PUBLISHED`? :-)
my ready?() assumes VMware Tools is installed in the VM.  You're correct ,we have to consider when VMware tool is not installed.

If VMware tool is installed ,  check  "poweredOn && ipaddress"
If VMware tool is not installed ,  check  "poweredOn"

BTW,  VMware Tools is now free to use and distribute:  http://www.vmware.com/download/eula/tools_policy.html . So we may expect VMware Tools to be installed in every vCenter managed VMs.
Correction. I just meant that I didn't want to test `metadata` by calling `metadata` in these tests, if I could help it.
why are those simply accessors and not fog attributes?
For completeness and jsLint's liking:

```javascript
var doc = Y.config.doc,
    SUPPORTS_TOUCHES = !!(doc && doc.createTouch);
```
JSlint doesn't detect any others within the ScrollView suite.
Make sure $function is_callable, and is either a string or an array with element 0 being a string. Otherwise throw InvalidArgumentException. We don't want to accept anything we can't use or generate a string representation of.
Doesn't really matter, but set it after the `update_data` call. It's just so the in-memory instance matches the new DB value for the rest of the request.
Decided to move the method impls one level up to the abstract ticket registry.
By default we typically log WARN and above so doing a check around those is not super useful.  Our recommendation is any DEBUG or INFO with complicated concatenation.
Yes I commented on it too, because I forgot to remove the 'a' in the method name before testing.  Also mentioned that I forgot to set Random(9) back to Random(25)
Why log slot twice?
might want to rename this to `enterBarrier` for clarity
No extra line is the preferred way.
and that was of course with

    `reader`
When a remoteActorRef is created. The result is cached in the  
RemoteActorRef.
I think it is worth repeating "enqueue requests from different senders" here, to avoid misunderstanding
Change 3. to 2.
The "output" is declared on line 320. It's the XmlWriter which is pointing to the file defined by "fileName".
Can't we re-use the direction finding from odf.py?  Why are the peaks computed in the ODF function?
Where would you like me to put it?
Also technically that example is for writing an extension not implementing a new extension point within ckan.

Generally the idea is to improve the codebase as we go :)
Looks pretty straight forward :)
why this method again?
Are you sure we want this?
The idea of the context and SPNL tasks is that rate can change. Unlike Hadoop that iterates as fast as possible, SPNL long term goal is to be an incremental processing over production data. We need a way to give more or less resource to running jobs, and the way I imagined it was the rate limit. I think the throttling and unthrottle functions should be inside the context, I agree, but the context need to be mutable.
This should be made configurable, or at least put a comment to do so.
is this function still in use anywhere?
Why change the order of these lines?
skipt --> skip
tatal --> total
> the proper replacement for that use case is `getDefaultAddress`

Ok, I will change the test. getDefaultAddress is not merged in yet, so I  
will do this after that.
>   My thought was that 'rename' is clearly no change in content, so we can
>   exclude it. But in case there will be more event types in the future (no
>   idea what this could be), it won't harm to reload the config. With a
>   condition on 'change' we would miss these. But at the moment it would not
>   make any difference.
I just think that using `'change'` is cleaner, since we are actually watching
the file for changes. If there are ever other events added, we can adapt to
that, but even then it might not make sense to just exclude `'rename'`.
NOT a file creation. That log is confusing.
Probably not a huge deal here, but I've learned to fear this pattern.  It's usually better to just write out the for loop:

```javascript
var args = new Array(arguments.length);
for (var i = 0; i < arguments.length; i++) {
  args[i] = arguments[i];
}
```
Incorporated. And wow, did I ever screw up the initial implementation! I scheduled it to `resolve()` AND `reject()` after the specified `ms`.  So it was a race to succeed or fail.  I see you caught that in your fix as well.  Thanks!
unfortunately I haven't been able to figure out a way to get the method name to be accumulable.  I end with errors like this when I try to use it:

[error] /Users/imran/spark/spark/core/src/test/scala/spark/AccumulatorSuite.scala:33: ambiguous reference to overloaded definition,
[error] both method accumulable in class SparkContext of type [T, R](initialValue: T)(implicit paramMaker: () => spark.AccumulableParam[T,R])spark.Accumulable[T,R]
[error] and  method accumulable in class SparkContext of type [T, R](initialValue: T)(implicit param: spark.AccumulableParam[T,R])spark.Accumulable[T,R]
[error] match argument types (scala.collection.mutable.HashSet[Int]) and expected result type spark.Accumulable[scala.collection.mutable.Set[Int],Int]
[error]       val acc: Accumulable[mutable.Set[Int], Int] = sc.accumulable(new mutable.HashSet[Int]())
[error]                                                        ^



I could just make it a two arg method where the paramMaker is not implicit.  Or else maybe somebody can figure out the right scala magic to make this work, I am a little stuck though ...
Sounds too busy? :-) How about 10
Why remove the half? It is a 16 bit float.
`hlsize` needs to be `hitlist_size`.
Also, to post an file using the django client: https://docs.djangoproject.com/en/dev/topics/testing/#django.test.client.Client.post

```python
>>> c = Client()
>>> with open('wishlist.doc') as fp:
...     c.post('/customers/wishes/', {'name': 'fred', 'attachment': fp})
```
How about also adding these tests:
- when `ActionDispatch::Response.default_x_frame_options` is `nil`, the `'X-Frame-Options'` header is not set
- when the response header is already set, it's not overwritten by `ActionDispatch::Response.default_x_frame_options`

It's also good practice to reset global state after each test (in your case `ActionDispatch::Response.default_x_frame_options = 'DENY'` will persist between tests).
:scissors: whitespace before `#`.
Don't need to check for the size. `args[0].is_a? Symbol` is enough.
Eta expansion is complicated; you don't want to reinvent it. Here's a longer test case. All the commented out variations still crash the reifier.
```
    def z1(x: Int) = x
    def z2(x: Int*) = x
    def z3(x: => Int) = x
    def z4[T](x: T) = x
    def z5[T](x: T*) = x
    def z6[T](x: => T) = x
    def z7(x: Int)(y: Int) = x
    def z8(x: Int*)(y: Int*) = x
    def z9(x: => Int)(y: => Int) = x
    def z10[T](x: T)(y: T) = x
    def z11[T](x: T*)(y: T*) = x
    def z12[T](x: => T)(y: => T) = x

    println(reify(z1 _))
    // println(reify(z2 _))
    println(reify(z3 _))
    // println(reify(z4 _))
    // println(reify(z5 _))
    // println(reify(z6 _))
    println(reify(z7 _))
    // println(reify(z8 _))
    println(reify(z9 _))
    // println(reify(z10 _))
    // println(reify(z11 _))
    // println(reify(z12 _))
```
Can we use `weak_<:<` from `Types.scala` for that?
You can have someone else fold this laundry:

`diagnostics count (_.getKind == ERROR)`
why is ``m2 is not ma.masked`` necessary? 
It's a bool not a boolean array.
Maybe use `asarray` here instead -- avoids making a copy.
Thanks. Can you add that explanation as a comment?
factores -> factored
On 11.06.2012 15:47, Christopher Smith wrote:
>> +.. autofunction:: sdp_sqr
>> +.. autofunction:: sdp_pow
>> +.. autofunction:: sdp_monic
>> +.. autofunction:: sdp_content
>> +.. autofunction:: sdp_primitive
>> +.. autofunction:: sdp_div
>> +.. autofunction:: sdp_rem
>> +.. autofunction:: sdp_quo
>> +.. autofunction:: sdp_exquo
>> +.. autofunction:: sdp_lcm
>> +.. autofunction:: sdp_gcd
>> +
>> +In commutative algebra, one often studies not only polynomials, but also
>> +*modules* over polynomial rings. The polynomial manipulation module provides
>> +rudimentary low-level support for finitely generated free modules. This is
>> +mainly used for Groebner basis computations (q.v.), so manipulation functions
>
> I don't see what q.v. stands for here.
>

http://en.wikipedia.org/wiki/Q.v.#quod_vides

Should I change it to "see there"?
done. Although this doesn't raise an exception if the inputs are bad.
I think the indentation here confuses Sphinx.
add issue number #1531 and make sure, changelog lines are sorted by number.
Do we keep only the title above the enum "The RevealType define..." and inline all descriptions?
The other views in kotti.view.edit have permission='view'.  Shouldn't it be 'edit' instead?
ok, kept the test

-------- Original Message --------
Subject: Re: [karaf] demos poms; karaf plugin (#6)
From: Andreas Pieber <notifications@github.com>
To: apache/karaf <karaf@noreply.github.com>
Cc: Andrei Pozolotin <Andrei.Pozolotin@gmail.com>
Date: Fri 17 Aug 2012 08:10:56 AM CDT
>
> In
> tooling/karaf-maven-plugin/src/test/java/org/apache/karaf/tooling/features/GenerateDescriptorMojoTest.java:
>
> > @@ -45,4 +51,25 @@ public void testReadXml() throws JAXBException, SAXException, ParserConfiguratio
> >          assert s.indexOf("repository") > -1;
> >          assert s.indexOf("http://karaf.apache.org/xmlns/features/v1.0.0") > -1;
> >      }
> > +    
> > +
> > +    /** FIXME xmlns 1.1.0 validation */
> > +    @Ignore
>
> but feel free to keep the test once the issue is here
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/karaf/pull/6/files#r1402466>.
>
Don't we still want to accept the case where self.radians == other.radians as equal? It seems confusing to have `__eq__` implemented but not have Angle(0) == Angle(0) be usable.
We should just do this change to the `default.xml`  as a separate PR and get that merged / tested.
I don't really care about the changes to ev.c because that file is going away soon but isn't something like the below less intrusive?

```c
#ifndef __GLIBC__
# define __GLIBC__ 0
# define __GLIBC_MINOR__ 0
#endif
```
Using GetCurrentThreadId() here is deliberate. GetCurrentThread() returns a pseudo handle that's not valid outside the calling thread.
Yes to the second question; will do for the rest.
s/reference/collected/?
Which leads me to ask: "Can I specify where the journal files will be written?" 
The answer is presently: no.
That makes sense. +1
I believe I pushed an update to fix this in my branch. @naderman, check that you have the latest stuff from my branch please :)
Guess this should be `_sql()` here?
Needs a newline at the end.
Course it will store properly, it's the old syntax.

I'm not sure that I can see the point in caching the object. It won't affect anything in any way unless the function is being ran more than probably 20 times a second
Sure, changed
The variable is passed to get_varref which will validate the variable name, no need to run a more complicated regex here.
aside: I see this code often enough that it feels like it belongs in a util module
`#ifdef _win32` ?
I’ve added this comment where it does not make any sense. Just wanted to say I’d not necessarily replace every imperative loop with a functional one
Will be done
А почему бы и не быть 100+ баннерам. Ну вернее почему не считать что это возможно.  решение по этой ссылке не подходит, потому что нужно указывать точное новое значение. Я же дал ссылку где подходящее решение
Using statements should be placed inside namespace declaration
We should move this to the `INancyContextFactory`, the engine should not have the dependency on the culture service. In fact, the `Create`method should be changed to take the `Request` object and this https://github.com/NancyFx/Nancy/blob/master/src/Nancy/NancyEngine.cs#L90 line should move into the factory as well  - long overdue
Trail never affects the world state, so it can't be this. Odd that the type
changed though.


On Sun, Dec 23, 2012 at 8:43 AM, Chris Forbes <chrisf@ijw.co.nz> wrote:

> Not convinced that this particular case is wrong, but yes -- this is the
> sort of thing we're looking for.
>
>
>
> On Sun, Dec 23, 2012 at 2:02 AM, Matthias Mailänder <
> notifications@github.com> wrote:
>
>> In OpenRA.Mods.RA/Effects/Missile.cs:
>>
>> >  			Altitude = Args.srcAltitude;
>> >  			Facing = Args.facing;
>> >
>> >  			if (info.Inaccuracy > 0)
>> > -				offset = (info.Inaccuracy * args.firedBy.World.SharedRandom.Gauss2D(2)).ToInt2();
>> > +				offset = (PVecInt)(info.Inaccuracy * args.firedBy.World.SharedRandom.Gauss2D(2)).ToInt2();
>>
>> Is this conversion from Float to Int2 and then to PVecInt problematic in
>> case of rounding errors, @chrisforbes <https://github.com/chrisforbes>?
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/OpenRA/OpenRA/pull/2242/files#r2494947>.
>>
>>
>
>
@nathanleiby When we deploy this module, will we split it up with the rest of the code, or give CHW's the entire code, OPD, etc on their phone?
might be clearer with a comment like
```
// Once value of k is equal to the number of drug forms, we know all drug forms have been loaded. We can now create the encounter.
```
Reference to standard would help e.g. see cf.py
这里应该是将mouseout事件传递给地图。地图自己决定黯掉哪个块。
I guess we should also allow the actual python objects for True, False and None.
For instance, one may want to set  config.vm.lazy = True, instead of config.vm.lazy = 'True', so this method should let at least True, False and None through (and possibly account for '0', '1', 0 and 1 too).
True, though I'm inclined to leave this alone for now for time-pressure reasons. :(
This would be a nice generic function, possibly reusable, if the success and failure message values were passed in as arguments instead of hard-coded.
please rename PaginationMixin -> DataTableMixin
one space less please.
where is this define used?
With a little effort he could have made it funny at least :\
http://xkcd.com/221/
Я бы унёс `return arch.getChildren(levelNode)` в `getMergeBundles()`
those are the specs I mentioned I'll add as separate pulls ... as soon as this gets sorted out. 
I only removed them due rhino as that ain't implemented ...
This should be indented. Maybe it's an editor issue? You may want to check that you expand tabs. I'll check again on my end too.
Freudian slip...
"Instantiating a form instance with the passed POST variables, .." sounds simpler.
Something just came to my mind: How does 0o666 affect windows?
Sorry, I messed up my comments. This was meant for line 226 in parseLabel().
It seems that this is where the queue tries to transmit the data through the pipe, and always `dump`s the data to a file even if it is already memmaped....
`[ posX[i], posY[i] ]`
@rwldrn @dmethvin - Anyone object to this style?
Had to take these out. They were only used in the PrototypeRenderer20.
same here as my comments above under oauth1
There should be some info here about how to combine this with HTTP_SERVER_STATIC_DIR to get a custom client up and running. That's really the important part IMO and currently I think it could use some more docs.
We do this in sumo :) /me tips cowboy hat

We just make sure that the values are properly extracted from the db.
I'm aware of this, but that's why you are supposed to verify the changes before you push. Don't remove these for now.
`if ARGV.build_head?`
`s/assertSelectorHasText/assertSelectorDoesntHaveText` maybe :)
There is a reason for this.  There already is a  self.options which represents the actual options passed in.
requéte à vérifier
`[]` for output seems a bit odd
Stray debug or buffer bug work around?
Still has one `kw` that needs to be `kwargs`.
I don't think this one should be hard coded.
You taking assumptions about what the user will use as base Crumb.

I think it should be handled by the user as another before_filter or something like that.

The important thing to keep in mind is the ability to use I18n ;)
So how will you assure that the first set is 100,200 and not 100,201 if the order returned in ModuleAdapter for Class.getMethods() will differ between runs.  Using these magic values will actually not work, unless i"m really missing something.  Can you write this in a way that's deterministic?  I'm not seeing it.
Those variables should be in Notifier, not in eventHandler, I'll move them.
I don't think putting taxonomies and post types inside the theme is good practice. What if I'm writing a plugin?

The most flexible way would be to output directly to STDOUT:

Create a new file, wherever you want:

```
wp scaffold taxonomy foobar > wp-content/plugins/my-plugin/new-taxonomy.php
```

Append a post type to an existing file:

```
wp scaffold post-type movie >> wp-content/plugins/my-plugin/post-types.php
wp scaffold post-type actor >> wp-content/plugins/my-plugin/post-types.php
```
Duplicate service definition.
How about `actionStarted`, `actionFinished`, and `actionTaken`?
What if we change DocumentCommandHandlers.doOpen() to call DocumentManager.notifyFileDeleted() instead of calling removeFromWorkingSet() as it does today?  Then this fail() block would be unneeded (since notifyFileDeleted() calls closeFullEditor()).

That would also get us halfway to fixing #1451 in the bargain...
Well, the point is that you should never, ever, ever use the global variable `ceEditor` anymore:

  * require("ext/editors/editors").currentEditor    - reference to the editor Plugin that currently has the user focus. In non-split mode: usually the editor plugin that is active in the selected tab.
  * require("ext/editors/editors").currentEditor.amlEditor    - reference to the active AML element that is, or wraps, the editor control. This used to be the `ceEditor` global, but is now kept as reference right here. For the image viewer, this will be the reference to an `<a:img/>` element and for ACE an `<a:codeeditor/>` reference.
  * require("ext/code/code").amlEditor    - an alias for the above, since `currentEditor` ALWAYS references to the active editor plugin and the editor instance is attached to the `amlEditor` property.
  * require("ext/imgview/imgview").amlEditor    - same as bullet above.
  * require("ext/editors/editors").currentEditor.path === "ext/code/code"    - to see if you're dealing with the code editor at the moment (to show/hide your plugin or something)
http://www.iheartquotes.com/api/v1/random?max_lines=1&format=json
A comment here would be welcome. I see that you are only considering the indices corresponding to different values and I guess the idea is to skip the flat part of the curve caused by the rounding problem in the exponential function we discussed a few days ago, but it is not clear at all what is going on from reading the code.
YOUR BRAIN HAS NEVER FARTED BEFORE???
Uh uh. Don't do it like this. This is much too broad.

I would recommend that you do a `ul` and `li` and then style that. Check how it is done in [nodewatcher](https://nodes.wlan-si.net/).
Empty line after docstring.
Allow also `datetime.date`?
Nope, no idea about that. Could you explain what exactly is wrong? Is the problem in using 2 backends for authentication?
`cli.js` already have a node hashbang, so this can be changed to `./node_modules/csslint/cli.js`
Spacing before `{`.
`options.containerClasses.split(" ")` needs some spacing
indentation 1 space off from the text on line 385? or vice versa?
test naming is improved. great.

next time I would take it even farther and test file and dir in two separate test methods.
You should reference a tag here. If there is no tags, you should ask author to add it using the [template](https://github.com/CocoaPods/Specs/wiki/%22Please-add-semantic-version-tags%22-issue-template)
print usage if args length < 2
GetTempPath and GetTempFileName are the functions to use on Windows. If you want to use tmpnam, I'm ok with you just marking it linux_only and making a windows_only version that raises a NotImplementedError. I''ll just open an issue against the windows version that it needs to be eventually implemented.
Can't we just check for a login? `proto://user@example.com/` is bad; `proto://user:pass@example.com/` is okay (if bad practice); `proto://example.com/` is presumed okay (though they could still be using authentication, they're most likely not)?
this is a bit easier to understand. changed.
updating environments is not supported on all subsystems (candlepin for example).
That is not explanation of the change - we would like to keep the defaults suitable for katello developers to start working quickly (copying this file to the right place and starting server). I vote for putting this back.
Every system command that requires --name param must also work with --uuid param. See other actions for examples. This would not work when you have two systems with the same name (which IS possible now).
Extra period at the end.
Nest this inside the `.search-box` block above.
Ok. I added `print(usnoa_cats)` after this line.
@mdboom - this should be straightforward. 

@astrofrog - I don't know why I used the word "table", that's just wrong.  About what kind of data this is, I'm not sure I follow.  Is the short-form explanation above insufficient?  If you have a suggestion for better wording in this docstring I'd be happy, I did struggle a bit with this.
I've played around with the decorator module a bit.  AFAICT it works pretty well.
@jrbasso its merge din already like this, but this is what you had in mind?: 90c32add9cea115ca7c6ffc4acc6f792565898a3

thx for the pointer
path info 的接口明显定义不对，应该通过返回 EntryInfo 的形式。
具体的key是限定于特定的backend的
Done. Fixed on 6f32e34
The event is polymorphic. How would you suggest I take advantage of that to move all this code out of the template?
I don't get this. So only on <code>http://fork.dev/en</code> the canonical will be <code>http://fork.dev/</code> (assuming EN is default)? On the other languages there is no canonical?
Note to self, this is temporarily. This should be fixed properly in the execute functions for each action (to be discussed).
On my box - I already had Git in the PATH for my user - might have been set up by either Chocolatey or msysgit. I'm pretty sure this should be kosher.
These should be named MOUSE_BUTTON_XXX_FLAG, not _MASK (a mask is an OR of many flags that can be ANDed with another value to remove other flags' bits)
as above, this should just be `iptest`
On Tue, May 8, 2012 at 2:05 PM, Min RK
<reply@reply.github.com>
wrote:
>> @@ -46,46 +49,45 @@ var IPython = (function (IPython) {
>>          return msg;
>>      };
>>
>> -    Kernel.prototype.start = function (notebook_id, callback) {
>> +    Kernel.prototype.start = function (notebook_id) {
>
> The general goal is to resolve the unnecessary coupling of kernels, notebooks, and cells.  Part of that is using the Kernel *alone* (no notebook, no cells) as a javascript library for remote execution.
>
> I should note that I did a test, and it does seem to work if you call `Kernel.start()`without specifying the notebook-id.

I have a vague recollection that I made sure the web service/handlers
behaved properly when no notebook_id was passed.  It should create the
kernel fine, but not associate it with a notebook.

>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1697/files#r791389



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
Should this increment errorCount and retry?
We may want to use `_isfile`, just for extra sanity?
Oops. Just add the `L` specifier to all keywords... `L")"`
There's no need to set it to anything, as we're not going to be using any of that information. The issue here would be whether `inflateEnd()` can be called twice on the same stream safely.