labofsahil/github-event-dataset-2012 huggingface dataset:
An issue to write a documentation page? I can do that first thing tomorrow morning.
если все время будем ждать чтобы "поправил кто-то" - то это надолго. учитесь у Raynar, которого достало ждать имплемента Duel Reset и он его просто сделал сам :)
is the `action` column's width locked? That'll be an l10n nightmare.
Typecast (if not null) is done in typehint, so initialize code does not need to typecast again, just check simplified check `if (null === $array) $this->array = array();`
You know, we might end up using this on our server... or at least, I might merge the changes over onto our build of Nethrar. I'm curious why you rewrote the inventory code; were the Bukkit functions not functioning for you? At least in the case of the StorageCart?
The idea in the spec about removing the ref : {} from the data-dojo-props would come into play here, because the thought was to look for dojox.mvc.at to know it was a binding.  If we keep the ref: in the data-dojo-props then we could allow a custom implementation.  And that may be a good reason to keep it, in addition to potential performance concerns.
Second comment probably should say "Hide..".
Yes, it's all merged already :)
Good catch, fixed with 1.1.9 version.
The line
    this._children[i].destroy();
Causes an error in the asteroids demo when the spaceship is hit by an asteroid.
And thanks again :)
Yeah, I was using the full '.git' from the GitHub web interface, which let me checkout but not commit.
error: src refspec develop does not match any.
error: failed to push some refs to 'git@github.com:Erkan-Yilmaz/Mink.git'

tried also different push commands, but fails :-(

will continue later
cool. sounds fine with me :)
I can't see where TOTAL_CHRONO is ever used. It's initialized to zero and never referenced anywhere. 

Unless someone has a reason for this var to exist it can probably be removed.
should probably organize object return types in a better manner
typo: createPacakageSnapshot should be createPackageSnapshot
Optional tip:
log4j is hasn't seen any active development since 2006 IIRC, it's dead. Prefer using Logback (AKA Log4j 2.0 because Log4j, SLF4J and Logback have been written by the same guy, Ceki). Logback implements the SLF4J interfaces directly (in fact, they are pretty much the same project).
I am using Logback in drools-planner's test and examples, copy paste from there. It's basically just replacing the dependency and upgrading log4j.xml to logback(-test).xml
no `+` is necessary and a period could be added to the end.
C'est pas bientôt fini de pusher du vide :]
compiling this project reports that RMMapViewDelegate.h is missing
Here I'm using Linux Mint 12 Lisa 64. Python 2.7.2.

It takes sometimes to load the video, maybe that could be the reason? Is it displaying the "MIssing plug-in" text on the screen?
Why such a big piece of commented code instead of deleting it? If we need this later on we can go to the previous version in github
Was the static analyser complaining about the variable's name, or was this personal taste?
Actually it would be great if someone with a strong understanding of Java concurrency reviewed all the FitNesse synchronization code. From the little I've seen, there isn't a  consistent approach and there are probably potential race conditions lurking in a number of places.
:) Reviewed.
@main-- This is the multiverse "bug" people kept reporting with...
/mv delete oldworld
/mv create newworld
/mvtp newworld
Then after the tp they see what appears to be the old world?
It's necessary remove some creatures for this Pit of Saron event, because right now there is 3 Scourgelord Tyrannus, for example.
Zelfs in Nederland werk ik veel offline, dus lijkt dat me geen goed plan. Kan je beter het bestand locken zodat niemand je eigen settings overschrijft.
Yes...they are swapped in mmap builder...
so instead of changing this in mmap extracting
we change the way we load it according to how it is build...dont worry
i tested the changes..mmaps works now
It's not a list of every prefix created by every vendor. After all, they
are all created by vendors.

I suggest you re-read the page you referenced and compare it to what this
function does. ;)

On Mon, Jan 16, 2012 at 8:58 PM, Marc G. <
reply@reply.github.com
> wrote:

> That's a draft prefix used by Opera which is a "vendor". So yes it's
> missing :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cinsoft/My-Library/commit/b31bbd33fea52616b9893d887a810fef3981017e#commitcomment-872837
>
One to many params?
I fell this is getting called because I get event.id's correlating to `SDL_WINDOWEVENT_RESIZED`.

Perhaps modifying this will fix the issue.
For 2. 
the best thing I know is __traits(compiles, (){
T dummy=void;
visitor(dummy);
});

But anyway you are right, I was looking at the diff only and was just wondering what a template without constraints is doing here.
Adding 'udp_burst' in MEASUREMENT_TYPES in measurement.py makes the new task type available to the dashboard

I also added param4 and param5 that let you specify destination port and direction.
LGTM. Just two minor nits. If you can address those I'll merge in tomorrow.
gentoo is a build-from-source distro and its .ebuild package files are
only plain text with package source urls, dependencies and commands to
build it. Also file named Manifest contains hash of file that will be
downloaded. All you need to do is create an "official" source
download, in github or sf or anywhere. Btw, please change .pro to make
it work with freeimage by default first

On 2/17/12, Alessandro Portale
<reply@reply.github.com>
wrote:
> I am glad that it works for you. :) And I will help hosting an ebuild
> download if you explain me exactly what is needed.
>
> Btw., if you want to have much broader image format support, you could
> enable FreeImage support. Just remove the '#' before "DEFINES +=
> FREEIMAGE_LIB" in posterazor.pro and install the freeimage development
> libraries on your system. Then again qmake && make.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/aportale/posterazor/commit/0813fa8243cad064e8f05bc6fe84b409c15db436#commitcomment-975144
>
thanks for the hint.
TabGroupie is not made to handle that addon. It works with the buildin tabgroups.
If you want to use that addon there is a other plugin here http://code.google.com/p/dactyl/issues/detail?id=533
Also made a trac ticket entry:  
http://core.trac.wordpress.org/ticket/19931
Testing the working
But what if you don't really know exactly what things are called? Browsing is a natural way of discovering.

Don't know about later revisions, but when I committed "autocomplete-stay-open" fix, it opened when the input got focus, and closed on blur, for example if you tabbed or clicked anywhere else.
angular.forEach
yes, even this file is going to be involved in mock testing with compiled angular...
Здесь скорее нужно проверять наличие :RubyConstant а не :all_symbol
Put back the following code
```
# try to compile an mpi program to check if compiler is already mpi	  	
check_cxx_source_compiles(
"#include <mpi.h>\n#include <iostream>
int main(int argc, char* argv[]) {
MPI_Init(&argc, &argv); int rank; MPI_Comm_rank(MPI_COMM_WORLD, &rank); MPI_Finalize();
return 0;
}"
CF_MPI_COMPILER_AVAILABLE )
```
Sorry to be the slow kid, here, but I think you may have solved a problem I've been having (http://wiki.makerbot.com/forum/t-423130) with this commit. Does that mean I should download the latest version from here: http://replicat.org/download ? Or I should download just this file from the "downloads" link above and replace the relevant file in my install?

Thanks in advance for the clarification :-)

Rosalyn
Is the problem pointed out by you regarding our git usage is corrected in this commit?
Yeah, tester will eventually be removed. I just had it in for some debugging.
ahah, i did not see that. I could try that next time!
We've never had an occasion to call a method like this. Remember how back when we had these, we only ever called `insertBefore`?

Also, the reason I chose the naming scheme `adopt` and `disown`, distinct from the DOM/jQuery `insert`, `remove` and `detach` naming scheme, was that you don't need to call `detach` before calling `insert`, but you usually have to call `disown` before calling `adopt`. The exception to the latter is `createBefore`, since nodes are born orphans, so to speak, so if `createBefore` called `insertBefore`, it would either be needlessly rearranging former parent and siblings it doesn't know don't exist, or `insertBefore` would be dangerously different from jQuery's `insertBefore`. But `createBefore` calling `adopt` makes sense, since it's born an orphan.
Why not? I tested the script and it works fine. You'll just have to match the guid of the postbox (xxxxx) with the guid of the related parcel (yyyyy).

insert into gameobject_scripts (id, delay, command, datalong, datalong2, comments) values
(xxxxx, 1, 9, yyyyy, 600000, ''),

After all spawning the parcel and summoning the postmen are completely different events and can be implemented independently of each other.
[Fixed](https://github.com/sstephenson/prototype/commit/fe31e6bc2126101fd907d8c36bb52fb0a681c35b).
Hmm, I suppose that's positive, but not what I expected: `( -1 >>> 0 ) === 4294967295`
поля класса лучше инициализировать в списке инициализации конструктора
while you're here, can I ask you something..

been busting my head about getting the whole InitD menu in Performance in RomControl

I tried adding init_trigger.enabled to system/etc, but I don't see new options.. do I need something more?
Looks good, but it would be even better if we try keep the coding style the same in this project.
I don't really care what style is used as long as it is consistent, by looking at the code it seams i have used:
Lower case keywords (as instead of AS)
Logical operators as in C, && instead of AND
Space after the comma in calls and function definitions, function f(a, b, c) and f(1, 2, 3)
Or more general: if the code looks like the code in tftpserver.php im satisfied :)
I don't understand how this really works. From the point of view of the Ceylon compiler, what is the type of `c` in:

    public static int fromJavaChar(char c)
Fix the indenting...
changed to md5, pushing the code into the repo

On Sun, Feb 19, 2012 at 1:33 AM, nakuldhotre <
reply@reply.github.com
> wrote:

> are we suppose to use md5 here or murmur ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/nakuldhotre/dedupcache/commit/6746603bd5f4708e998aa0e924a587ec0401c1ea#commitcomment-982315
>
Isn't it simpler to use $recursive param of mkdir to create the structure ? 
http://php.net/manual/fr/function.mkdir.php
fail? nah. Epic fail sql/updates/*world*/2012_03_06_02_*world*_queststatus_seasonal.sql
Maybe it would be safer to do the sorting after the merge. In theory there could be cases where the merged list is not properly sorted.
This applies of course also to line 111 and 155.
thx man
Detalle de menor importancia:
Aquí si querés asignarle un valor vacío, al ser un título, conceptualmente correspondería el string vacío "", o bien un valor null, si es que el "" tiene su propia interpretación.
El valor NaN se refiere en realidad a "not a number". En caso de ser necesario el uso este valor especial, se escribiría simplemente NaN, sin comillas de ningún tipo. Al ponerle comillas, representa el string "NaN".
Especially as I think we only use one of them ...
Just exposing the lists might be good enough (since this is for testing).
I wonder if writing a GraphLayoutCallback stub implementation & using Mockito.spy on it might be clearer?
Let me add that the irony in that commit message having wrong command was not intentional.

DERP
jCnv - Javascript Canvas library
================================

## About

This is a simple Javascript library in making which will be used for image manipulation and drawing. Simple app currently being developed using this library is [JCnvDraw](http://github.com/Hgazibara/jCnvDraw).
This class should be merged into `ActivityStarter` as they perform very similiar functions.
Please use consistent 2-space indentation, and follow the other style guidelines mentioned in the expertiza README - https://github.com/bbatsov/ruby-style-guide
this is not right just change it please
> oups… could be … but with so many CPUs you become lazy :)

That's why we don't have many at the lab :P
beautiful readme. I've never been so inspired in my entire life.
Nice. Those urls were a bit gnarly.
Holy crap notes.
*Ahh , your correct, I was thinking of another situation when I said d7, it
is broken there too ( this is a bug I need to file I guess but its pretty
late in d7 to fix but I dont use d7 on projects yet lol ) but where it DOES
work is in d6 prior to the DRUPAL_ROOT backport changes *
*
*
*
*
*Brandon Holtsclaw*
*
*


On Thu, Mar 8, 2012 at 5:40 PM, Robert Brown <
reply@reply.github.com
> wrote:

> Thanks for the comments. I understand what you are getting at and why you
> would need it for an external script, but going back to the D7 question:
> why would your externally bootstrapped script work in D7? It also defines
> DRUPAL_ROOT in index.php and has no if (!defined) statement in
> bootstrap.inc. Wouldn't you run into the same problem there?
>
> D7 index.php:
> http://drupalcode.org/project/drupal.git/blob/refs/heads/7.x:/index.php
> D7 bootstrap.inc:
> http://drupalcode.org/project/drupal.git/blob/refs/heads/7.x:/includes/bootstrap.inc
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/bholtsclaw/pressflow-6-plus/commit/044308d234db46c17f602145dcde69a428344773#commitcomment-1063451
>
agreed. Also why is it dojox.application.controller here? we should be using dojox.app.
Testing this...
Hi Shay, there are so many duplicated code fragments, didn't you consider refactoring?
Basically, this approach is the most (conceptually) simple way of implementing the singleton pattern in Ruby. Rather than creating an instance of class and then disabling its ability to create new instances, and rather than creating a module or class with module level methods and then putting all the state at that level but STILL disabling instantiation/use as a mixin, this approach simply creates a single instance of a bare object and directly adds functionality to it.

There are tradeoffs to this approach, and for the most part Ruby does not have an elegant option for building singleton objects. I discuss all the different approaches in [this Practicing Ruby article](http://practicingruby.com/articles/shared/jleygxejeopq).
This is just one change picked from chronossc/django-ldapdb
You got it. I'll release an upload to PyPi in the next few hours. Thanks for the report!
according to http://hg.python.org/cpython/file/2.5/Lib/inspect.py#l849 and http://hg.python.org/cpython/file/2.7/Lib/inspect.py#l1029
Will this break if `site.author.twitter` is null?
This method (build_query) is defined to behave exactly as the previous code in mixins, however there's something strange there: if args are provided it would be enough to use only pk in query - merging pk  with any other elements from kwargs has no effect. The comment was just to mark this (and indeed should not be present in the commit)

If args are present, it would be cleaner if this method by default returns simply Q(pk=args[-1]) - it was intentionally left to return Q object build with pk merged with kwargs, but I feel I should change it in the code candidate for the pull. What do You think?
> If only https://github.com/mozilla-services/android-sync/commit/5041e was in develop, it could have been used here :(

We can do that.
It is a lot more potential benefits if you could use more meaningful variables.
Why do you want to limit sleep time ?
This does break asset precompilation for me with error message `File to import not found or unreadable: active_scaffold_colors.` (Sass is used).
Их код можно написать лучше без увеличения количества кода и без умствований.
Например, функция splitEvery, которая не ловит вход 0 и честно делит на 0 тоже может считаться не бажной на основании того, что в моём коде ноль ей никогда не передают. Но better be on the safe side, особенно если это ничего не стоит. Непонятно, с чего бы и им не придерживаться.
Probably best to remove generator history. Startups will probably look at source.
Convert to using ComparisonChain: http://docs.guava-libraries.googlecode.com/git-history/v10.0.1/javadoc/com/google/common/collect/ComparisonChain.html
Remove the interval after "  :)
It sent me an email. Are you using sts-2.9.0.M2? Wondering if you got the right one since you installed it before I sent out the email with instructions on which one to get.
What benefit does having a tag that says that nothing is returned, when you can save a line of code and state that the return keyword is not present when the `@return` tag is not present.

Again, the draft [PHPDoc PSR](https://github.com/mvriel/Docblox/blob/develop/docs/PSR.md) says it is optional.

The [Wikipedia](http://en.wikipedia.org/wiki/PHPDoc) page states that the `@return` tag should not be used for a void / no return value.

The source code examples 2 and 3 on the phpdoc.org website clearly omit the `@return` tag when there is no return keyword. It also says says:

> The @return tag is used to document the return value of functions or methods.

There is no return value, so there shouldn't be a `@return` tag, just the same as you wouldn't include a `@param` tag when there is no arguments, or a `@global` tag when there's no global being used.

 > The datatype should be a valid PHP type (int, string, bool, etc), a class name for the type of object returned, or simply "mixed".

void is not a valid PHP type. At best, it's a pseudo-type, and only used to describe PHP functions itself for the purposes of documenting the C code that PHP is actually written in. It's not suitable for documenting PHP code.

The [PHP Manual](http://php.net/manual/en/functions.returning-values.php) clearly states that "If the return() is omitted the value NULL will be returned.". NULL != void, so by including `@return void` you're actually giving wrong documentation, which is worse than no documentation.
Interesting... the code which sync annotation sources should be able to parse HTML. Why not to use just text?
slightly ambiguous in English - do you mean 'a non-existing ann src' ?
This doesn't handle a case like `VCR.cassette_exists?(an_unsanitized_string)`.  VCR [sanitized cassette names](https://github.com/vcr/vcr/blob/master/lib/vcr/cassette/persisters/file_system.rb#L52-L56) before converting them to file names, and I think the same logic should be used here.

Maybe it could use [this helper method](https://github.com/vcr/vcr/blob/master/lib/vcr/cassette/persisters/file_system.rb#L41-L44)?

I'm also not sure it belongs here off of the top-level `VCR` object.  Historically, VCR always persisted cassettes to the file system but 2.2 added support for swappable persisters, so users could flexibly persist cassettes to any medium.  Adding another top level method that assumes VCR is persisting to the file system goes against that direction a bit.  How are you using this?
This is backwards, I think?
Is there some kind of formatting problem here to make the whole file appears different ?
same as above in terms of coding conventions
Same feelings here
how about a factor/percentage of nodes instead of a fixed number?
this should have an accompanying scaladoc comment explaining why we added the warning
That would make this:  publishTo <<= version apply Publish.akkaPluginPublishTo
that must be wrong name, it should not be latency, more like NetworkThroughput or NetworkIO
How will it be used from Java?
You're right. Will rewrite.
PlEASE STOP SENDING ME EMAILS!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Best Regards,
Alicia Harrison
Marketing Manager
PH 310-669-8800
Palos Verdes Footwear, INC
Volatile,Very Volatile,Volatile Kids,Volatile Kicks,
Volatile Handbags,Sbicca,Sbicca Vintage,Grazie


On November 15, 2012 at 3:38 PM "Viktor Klang (√)" <notifications@github.com>
wrote:


> 
>  In
> akka-samples/akka-sample-cluster/src/main/scala/sample/cluster/factorial/FactorialSample.scala:
> 
>  > +  // subscribe to ClusterMetricsChanged
>  > +  // re-subscribe when restart
>  > +  override def preStart(): Unit =
>  > +    Cluster(context.system).subscribe(self,
>  > classOf[ClusterMetricsChanged])
>  > +  override def postStop(): Unit =
>  > +    Cluster(context.system).unsubscribe(self)
>  > +
>  > +  def receive = {
>  > +    case ClusterMetricsChanged(nodeMetrics) ⇒
>  > +      nodeMetrics.filter(_.address == selfAddress) foreach { n ⇒
>  > +        n match {
>  > +          case HeapMemory(address, timestamp, used, committed, max) ⇒
>  > +            log.info("Used heap: {} MB", used.doubleValue / 1024 / 1024)
>  > +          case _ ⇒ // no heap info
>  > +        }
>  > +        n match {
> 
>  Why two different matches, looks confusing
> 
>  —
>  Reply to this email directly or view it on GitHub
> <https://github.com/akka/akka/pull/858/files#r2144298> .
>
Yes, will be done as part of ticket https://www.assembla.com/spaces/akka/tickets/2014
the promise returned by Patterns.ask() is fulfilled as a failure, including the exception
This is implemented now. I've also cleaned up as many of the Card::Codename references as I could.

You can DELETE when you review this.
Moved this, seems to be fine where it is, or are will missing another test?
@seanh Any particular reason for using ``table_dictize()`` rather than ``group_dictize()`` here?  (group_dictize() adds a bunch of related things I appreciate you may not want represented)
If this is new code then deprecation shouldn't be here - not sure how this fits with the group stuff
maybe ;p but you did call it pitfalls and hot to avoid

I suppose 

x.default bad x.default_value good
If I look at the output there is a html tag:
```

<!DOCTYPE html>
	<head>
	<title>phpBB MOD Pre-Validator Results</title>
    <link rel="stylesheet" href="style.css">
	</head>
	<body><div class="wrapper"><header><h2>Validation results:</h2></br></header><section>(Validating zip)<br /><br />
<br /><br />
```
The capital T strikes me as odd. It may not be wrong, just feels like it'd look better either with a capital O in of or lower case T in this.
It didn't feel like they applied. The epydocs should refer to the parameters, but the POST call doesn't accept any.
I was copied from old code.

Do we really need this double check? It was checked already in 'if (!results.GetDeviceOnLocation(deviceProperty, &prevDevice))'
Minor version is checked for 100% sure of correct detection. In case of win2k, broken emulators we can get different results.
"if ... **else** if..." will produce same compiled code as "switch... case... case....".
cause i don't want to loose precision going over m_subtitle which only has integers.
It should be:
`Cookie "%s" is not set, but should be.`
so we would just check this is the expected size?
```py
# Late bindings
```
new optional virtual attribute for the user account
This is now on the card too, was in User.
Another issue. I tested with https://github.com/arlolra/otr/blob/master/test/xmpp.js, setting the `fragment_size = 20` which generated over 30 messages during the AKE, and noticed that I was being rate limited by the gchat server. A more sensible size of `100` went through just fine. Does a `50ms` pause between firing the callback make sense here?
looping is not more fair than blocking `lock()`, and this should not be happening more than 10000 times per second
Sounds good, I'll get make the change and add it to the pull by tomorrow.
Why is this a macro?
Thanks for clarification!

I've changed it to the version with hyphens, so everything should be fine :-)
If a user passes a `bufferSize` of `n < 0` then `cb()` will be called w/o ever throwing an error.
Throw some appropriate error rather than RuntimeException.
Proč ```NOT isset($row['line'])```? Překlep?
Should be Nette\ComponentModel\IContainer.
Indentation is four spaces (not tabs, not two)
Got it, also went through and looked for other situations where I did something similar and updated.  Also added a MAKE_ZNODEL in the same spirit and coverted all but 2 MAKE_ZNODE to the length specified variety.
@TazeTSchnitzel Maybe I'm the only one who thinks dropping the body tag is a *terrible* idea.  I've voiced my opinion and that's all I can do.  I do agree that the text-format is designed to be human readable, but that doesn't mean we should rape our HTML.
удалю.)
а вообще это для фикса бага с контекстым меню #593
We avoid to use `assert_nothing_raised` since is better to assert the return value. Could you change it to something like [this one](https://github.com/Empact/rails/blob/dbfe388559ecc10a835da16caeffc82f66996ec4/actionpack/test/controller/test_case_test.rb#L788-789)?
@mmozuras  the current implementation returns the correct Julian calendar value for 6th July, 1253 - for what values does the current implementation return an incorrect value?

Also for the record, neither of these take account of [leap seconds][1] - they're irregular and can't be calculated with an algorithm.

[1]: http://en.wikipedia.org/wiki/Leap_second
Clearer as one line
"As of jQuery 1.7, the .on() method is the preferred method for attaching event handlers to a document." First line of the jquery bind docs http://api.jquery.com/bind =)
Nah, it does the exact same thing - cool - wow, you guys are fast.
Right. Please fo ahead
The first test refers to `clear` method of ActionDispatch::Request::Session and the second ensures that `destroy` method calls `clear` in its body. The second test needs more assertions to be correct, but I didn't find some quick way to mock the necessary stuff. We need to revise the mocks in this set of tests and cover all the defined methods. 

For now I will drop it and I will try to refine all tests in in a second iteration.
I'm thinking of the guy who is trying to debug why scala.reflect has different behavior from scala.frotz.  It's not like it's going to be obvious that scala.reflect is hardcoded to behave differently.  With a config file at least one can say "things which are treated differently in arbitrary-seeming ways are enumerated in this file" and one isn't left tracing through 100K lines of compiler source.
Shouldn't we also support the java convention?
This lookup could be abstracted in a helper method, it's duplicated right now. And the efficiency concern applies here as well.
The module docstring should be at the beginning of the file.
I'm starting to believe that the *gees declaration is in error: the two arguments are just tacked on to the Fortran routine prototype, so the routine gets called with the wrong signature. Apparently, this does no harm...

I'd suggest you just remove the two last arguments from the callstatement and callprotoargument. Things should still work OK.
Ah, it does continue after returning the reponse, I'll merge this in and update any other routes that may suffer as well.
do we want to try and standardize on 4 space indent when wrapping across the board? i kind of like it better
A more descriptive variable name then ddays would be helpful.
I did not understand the "XXX" message. Shouldn't you add the failing example as decorated test?
Why compare result.args instead of comparing the result?
Please add

```
from sympy.utilities.randtest import test_numerically

test_numerically(re(fresnelc(z)), fresnelc(z).as_real_imag()[0], z)
test_numerically(im(fresnelc(z)), fresnelc(z).as_real_imag()[1], z)
test_numerically(fresnelc(z), fresnelc(z).rewrite(meijerg), z)
```

etc
Oh wow :-( Sorry, fixing it.
Don't understand,  but [test report](http://reviews.sympy.org/pullrequest/1168) point out about those lines.

```
ValueError: line 16 of the docstring for sympy.utilities.iterables.capture has inconsistent leading whitespace: '-'
```
Why is converting to string necessary here?
I guess this is a little like using a `force` argument...if you don't know if it's real you will assume so?
The is_number() method had to be included since it was called during the caching workflow(from cache.py).  I believe the other methods could be implemented if needed by workflow.  

Plus, without evaluating the Tr it may not be possible to guess if the values are Integer or Number.
this TODO can be deleted
Мне кажется, что пользователь и учётная запись частенько рассматриваются как одно и то же. В сущности для GitHub'а так оно и есть.
Non-directories and non-readable directories couldn't be traversed and thus couldn't be removed.
please check the size of the array. Recurring theme.
Instead of all of this tricky logic - we should consider just using a click outside handler to close the tip when it is interactive....
I do agree that it's cluttering the interface though.  Also order by can take the _Field from the model.
instead of try on assert, just do:

if not link.writer in self.thread_names:
  print("Error: Link " + str(link) + " refers to a writer object that is not specified: " + link.writer)  
  sys.exit(-1)
Why not just make a local variable?  The object property isn't reused at all.  A local variable will be garbage collected at least.
@lsmith77 only bootable ones
How would that work? We don't want to override already set defaults, options, and requirements.
It might be helpful to have a "rule_version" field that essentially acts as an indication of what kind of data to expect from the payload
This appears in a couple of places and should be an included file. 

I'm not sure that this needs to be a list, but it needs some sort of terminal punctuation.
I'm going to leave this in for the moment. We were only able to confirm this process, for 2.1, as the process for getting it working on the wiki was somewhat less than clear on the service-related features in 2.0. We can touch base on it in the morning.
True! Nice catch ;)
Is there a ticket for this?
1. phpBB does not use CamelCase for js?
2. So you are asyn' to create a new ticket and add a dependency of this to that ticket?
Nu cred că e bine. Va da o singură dată, de la 1 la 2. Sau greșesc? Nu ar trebui ca upgraded să fie == level? Neapărat test.
This is a bit ugly. I think using 

> let conf = getRequestConf hailsRequest
> .. (browserLabel conf) \`leq\` ...

is a bit more verbose, but does not shadow names and is safer to interchanging order of types.
Minor, but can we use the convention from our other packages:

Data.ByteString as S
Data.ByteString.Char as S8
Data.ByteString.Lazy as L
Data.ByteString.Lazy.Char as L8
let me find out
I'm tempted to let `parseUnitString` only return the unit and not "amount". For two reasons: 
1) `parseFloat` is nothing we need to put in a helper function. It is already a well tested helper function itself and can safely be used. 
2) We don't need to allocate another object (to be able to return 2 values)

Code:

```
extractUnit: function (any) {
  // make sure it's a string and remove trailing whitespace
  var unit = String(any).replace(/\s+$/, '');
  // returns extracted unit or empty string
  return unit.match(/[a-z]*$/i)[0];
}
```

```
parseAngle: function(angle) {

  var radians;
  var amount = parseFloat(angle);
  var unit = extractUnit(angle);

  switch(unit) {
    case 'deg': 
      radians = amount * PI / 180;
      break;
    …
    case '': // default
    case 'rad':
      radians = amount;
      break;
  }

  return radians;

}

```

What do you think?
We should consistently use the sandbox Y instance.
а вот это уже гитхаб чудит, у меня там отступ есть)
not sure, I changed my initial opinion...
query the workspace/workflow table could be better but as actually these items are not indexed we only add dead code
throw an illegal argument exception saying that indexing of item without item is not supported should be the better option at the moment (this will prevent us in future to discover bug later if we start to index workflow/workspace items)iostath <notifications@github.com> ha scritto:> +
> +    	String toReturn="";
> +    	DCInput myInputs = null;
> +    	boolean myInputsFound=false;
> +    	String formFileName = I18nUtil.getInputFormsFileName(locale);
> +    	DCInputSet inputSet = null;
> +
> +    	//Read the input form file for the specific collection
> +    	DCInputsReader inputsReader = null;
> +    	String col_handle = "";
> +
> +    	Collection collection = item.getOwningCollection();
> +
> +    	if (collection==null)
> +    	{
> +    		col_handle = "db-id/" + item.getID();

Hello Andrea!
So if the collection is null, I have added to return null, and, in this case the DSIndexer has nothing to index
Do you think I should query the workspace and workflow tables to discover the right collection? Does DSpace in any case indexes also workflow or  workspace items?
Many thanks!
Rania

---
Reply to this email directly or view it on GitHub:
https://github.com/DSpace/DSpace/pull/55/files#r1609540
Simply not modifying the submodules fixes that ;) May be related to your (virtualbox?) setup that changes all your filemodes.
Done it in later commits.
style: remove whitespace
Style: please name with more clarity on its usage, rather than abbreviation.
Also, I don't see "bbb" being used anywhere other than in this definition. What is the intent of bbb?
How about we just ditch this whole class? The only thing it seems to add over the tests later in the file is a test of the netCDF-to-Cube translation. But that's also covered by `TestNetCDFSave.test_netcdf_hybrid_height` in `test_netcdf.py`.
Any reason for the change from single to double quotes?
We shouldn't use either! Leave it to `repr` so it can adjust as necessary (i.e. it prefers `'`, then falls back to `"`, then uses escape chars). Perhaps that's a detail point we should add the coding guidelines.

I'll fix the obvious transgressions in this file, but to keep the PR clean other fixes should be done elsewhere.
mixed adodb object type
should default values for $paths/$partials here be `array()` ?
In the chain, yes, you should avoid reusing the object. In a private method which is unused outside of a single plugin: you should totally use the object you already have.
`is`
Please #ifdef
When Django finds a regex that matches the URL of the current request, the specified view function is called.

"Python specified view" - not sure what that means here
The example is well-intended but maybe should do something else, or use the F object.
Hell yeah - fix the cause, not the symptom.
not sure if User. prefix is correct.  path would be django.contrib.auth.models.CustomUser.User.USERNAME_FIELD.  should is just be CustomUser?
This test doesn't fail either in current code. I suggest to simply drop it. No need to update the pull request, I'm going to commit the patch soon.
Please revert this indentation change
Good to know, thank you! I have had trouble with composer.phar downloads *not* always being up to date -- as recently as last week, I had an issue where it wasn't. I've had a lot of recent practice on that with my work on adding composer support to PHPUnit.

The new dependency-adding stuff in this set of build changes actually checks for a global composer, and does dev install, so I'll trim it down even further. Thanks!
k yo it's fixed. i replaced this one with another file and made 2 new tests
For the sake of readability, can you re-format this so there is no break-up from the `if` to the `else if` - thanks
Why did you change this block? Does your version yield a smaller jquery.min.js.gz?
DOM取得用コメント
In fact, madpack renames the old madlib schema if it exists. As far as I remember, the old MADlib still works afterwards.
This is not used right now, and shouldn't have been included in the PR. My bad.
Xcode changed this automatically. It was not intentional.
Is this a scrapy-defined method that is supposed to accept all these args? Otherwise it seems strange that it takes the "spider" arg and doesn't use it.
I don't know how we feel about defining HEAD inside of the devel block; it is typically reserved for "unstable, but not a moving target".
By the way...  xcodebuild is now part of the OS:    

    $ which xcodebuild
    /usr/bin/xcodebuild
    $ pkgutil --file-info /usr/bin/xcodebuild 
    volume: /
    path: /usr/bin/xcodebuild
    
    pkgid: com.apple.pkg.update.os.10.7.4.11E53.combo
    pkg-version: 1.0.0.0.1.1306847324
    install-time: 1339119116
    uid: 0
    gid: 0
    mode: 755
    
    $ xcodebuild -version
    Xcode 4.3.2
    Build version 4E2002

This is from my "Xcode only" install of 4.3.2

I don't think this is an issue here, but for the record, Xcode can still be downloaded directly from Apple and in that case, pkgutil is completely oblivious and should therefore never be trusted for identifying development environments.
Personally I dislike binding key-combos that are used in in desktop applications.
TRUE should be small letters
code commenté...
Et si MapManager.instance est null ?Ca va planter ?
Should be commented out in the repo so that attackers can't set-off a test-run (which consumes a lot of CPU)
please use only one tab when indenting arrays, and put every element on it's own line. This way you reduce the amount of whitespace
Bitte mehrzeilig notieren
We'll come back to that later, don't worry about it right now.
Don't really like the name. How about `timed` ?
You don't need this. You should use the standard `.hidden` class that already exists instead of duplicating it.
I moved "name" into Command. Whew, that was a lot of small edits everywhere to hit every command register. I also pushed on the command names into Strings.js while I was at it since I had to touch them anyway.

Can we talk more about the Menu and MenuItem classes and them being public on Fri? I still have the following functions on the prototype of MenuItem. I could make them just part of menu if the user identifies the MenuItem via ID instead of using a class.
	setCommand()
	isDivider()
	getParentMenuItem()
	getParentMenu()


-----Original Message-----
From: Narciso Jaramillo [mailto:reply@reply.github.com] 
Sent: Wednesday, May 23, 2012 11:10 AM
To: Ty Voliter
Subject: Re: [brackets] Menu, MenuItem, key bindings, and Command API (review only) (#927)

> +     * menu item or a native menu item if Brackets is running in a native application shell
> +     *
> +     * Since menu items may have a native implementation clients should create MenuItems through 
> +     * addMenuItem() and should NOT construct a MenuItem object directly. 
> +     * Clients should also not access HTML content of a menu directly and instead use
> +     * the MenuItem API to query and modify menus items.
> +     *
> +     * MenuItem dispatches the following events:
> +     *  click
> +     */
> +    function MenuItem(id) {
> +        this.id = id;
> +    }
> +
> +    /**
> +     * Retrieves the Menu object for the corresponding id. 

Now that all the functionality that the menus depend on is wrapped up in Commands, will clients ever really need to access Menu or MenuItem objects directly after creation (or even do things like listen for open/close events)? They can't really use open/close in order to decide when to update their enabled/checked state since those are going to be asynchronous. So I wonder if we just should consider Menu/MenuItem to be private implementation details--client code should do all their work in Commands and not try to work directly with Menu and MenuItem objects.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/927/files#r869823
This reflects the scenario in which a user has 2 c9 instances opened in the browser and switches between them during development. Listening to `focus` ensures that whenever the user switches to the editor we will perform a check to know whether the content of the document reflects the last revision we have on the list (the list of revisions gets updated constantly in all users/listeners). Does this explanation make sense?
This needs a check in case there is no `.watcher`. See the error in the discussion panel of the PR.
what you need is "command" to be async. Problem with that is that it will break API and every plugin has to be updated. What we could do instead is to have

`supportsCommand(user, message, callback)`

and

`command(user, message, client)`

the workspace.js could then first check if the plugins supports the command using the async API and if it supports it call the real handler.
So I think this is correct but want you to validate. This is the only `this`-to-`self` change that changes a reference, i.e. I think it was a bug before. I believe `this._selectedModule` refers to `cloudCare.dispatch. _selectedModule` (which as far as I can tell isn't a thing), while `self._selectedModule` points to an instance of `cloudCare.AppMainView`
I know this hasn't really been the team's style so far, but the "this" keyword is really only for readability. It isn't needed.

If you name your model's properties according to the scheme where public start with capital (FileLocation) and private start with an underscore, then it's easy to see that FileLocation = fileLocation is the assignment of the parameter to the model's property.

This isn't really something I would have expected you to change up when you came in to add an overloaded constructor, but something to keep in mind if you want to refactor in the future.
Isn't `get_profile` a method? I would be more careful here, like:

```
if request.user and request.user.is_authenticated() and request.user.get_profile() and hasattr('language', request.user.get_profile()):
    ...
```
replace `&amp;#39;` with an apostrophe
That's right, it's a ratio (100% of the font-size)
When does this code run when `BTN_ACTIVE_CLASS` is actually on the `classList` of `username` (and same with `userMenu` and `tooltip-no-transition-on` above)? Do you really need these checks?
Remove this line too... the next line already defines charset, and is cleaner.
Would you mind lowercasing the command here? No big deal if you don't - I can run cmake-bulk-decrufter after, just noticed this as I looked through.
This line has more than 80 columns.
Par défaut à true (puisque quand tu vas bannir une personne, forcément il sera par défaut à true).
we don't care.
I'm not sure, the RFC doesn't specify an order. But i could understand why you would want to do deletes first and then the updates.

I'm happy to change the order - i'm just not sure what the RFC states.
Why not just use the looping approach from the previous `tempname` with this call to `tmpnam?` I might just be missing something, but it seems useful to have everything boil down to a call to `tmpnam` here.
It ended up being easy to get rid of the try.
IMO this should file to prevent to use some code that is not supported while developing it
When touching this, we can rewrite it to errata_list.sort_by { |errata| errata.id.downcase }
@name.empty? would be a little nicer, not a big deal
not necessary: it works like this: 'file://' specifies the "protocol" and then the path follows, which might relative (without starting '/') or absolute (with starting '/' leading to '///' in a row): e.g. you can call curl file://some_file and it will search for the files on the pwd.

On the other hand it doesn't make sense to specify relative paths, and also localhost is not a typical example of a path to a file and listing 'file:///path/to/repo' instead would be probably more intuitive.
Is there anyway to detect that the source is already closed and to gracefully handle that case.  We were hit by such a problem last week.
In any case, are we ready to merge?
You are right! Fixing it!!!
Yup, I'll cut out the Where middle man.
OK, I'll change this.
yes, should use validator
This change is IMO a deeper one, as current release are still build with Qt 4.7.x. I would also love to see us switch to a newer version, but at least I (using Windows) can currently not work with 4.8.3 because of a qmake bug they introduced. 4.8.2 works pretty good and is used by the pull tester afaik, so 4.8.2 seems to be rather matured!
Check if sort_by (without !) exists.
Ok, if you say so ;)
Yeah I think `git_path` makes sense. I'll do the changes and rebase.
I feel like there ought to be a neater way to do this. How about `[s.lstrip('%') for s in items]`. Admittedly that would remove any number of leading `%`, but if we ever do have something with 3, that's probably the behaviour we want anyway.
The use of "remote tracking branch" seems different from what it means. HEAD cannot be/point to a remote-tracking branch. A remote-tracking branch is on which (unless you've changed the refpsec) lives under `refs/remotes/foo/` and represents the remote branch's state at last fetch. Here (and I think below) you probably mean to say "See if the current branch tracks a remote one".

We also have `git_branch_tracking()` which does this code block for you.
Hm, interesting. This is basically a broken configuration, as you simply cannot expect it to be usable when tracking any of the branches in either remote. I'd argue it's not worth supporting this kind of sillyness explicitly.

Even if we store the right entry in FETCH_HEAD, `git fetch two` is going to mess up what status, merge and rebase do.
I think we ended up deciding that `user_num_*()` takes a user. You updated the tests but not this here. I think?
do you need to sort this list to make sure the tests don't ever flake out?
There's `check_review_delete` already that's available to the templates. Do you need to make this?
Yeah, now that I think about it this will break when your beam is slightly less than 360 degrees wide. (qf1 === qf2, and it'll only do the first quadrant - and probably do it wrong). I'll have a bit more of a think about it when I get home. I also want to use tan to figure out actual slope values instead of dodgying it.

But for a torch beam or something, its surprisingly smooth.
This condition is too large. Might be splitted into variables or refactored into a Helper method
Code duplication with the `button()` method. Refactor in a protected method
Why is this import here? We should show an example of `tool` usage also.
This doesn't look like just a variable name, did you mess with indenting or something?
s/All though/Although/
Indentation ???
Aren't we going to use [phpunit](http://phpunit.de) for this?
nit: calling `toString()` is a waste of code
We are using camelCased names, not underscores. And why ``l`` ?
Hi,

I not find tests, but they start issue
https://github.com/FriendsOfSymfony/FOSFacebookBundle/issues/79
but fix not work :)

2012/3/22 Drak <
reply@reply.github.com
>

> > @@ -79,11 +110,35 @@ public function regenerate($destroy = false)
> >       */
> >      public function getId()
> >      {
> > -        if (!$this->started) {
> > -            return '';
> > +        return $this->id;
> > +    }
> > +
> > +    /**
> > +     * {@inheritdoc}
> > +     */
> > +    public function setId($id)
> > +    {
> > +        if ($this->started) {
> > +            throw new \LogicException('Cannot set session ID after the
> session has started.');
>
> Can you link to the specific tests it breaks please?  I tried to look for
> them but didnt see anything.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/symfony/symfony/pull/3493/files#r588565
>
>
Alright, will do
the return should be fixed too (it is even more important as it is the part used by IDEs and the api doc)
you can have up to 21 venues in this logic... 0-19 in this loop, plus the newVenue added below.
@adarshpandit Using Gang of Four terms, a decorator is an object that encloses a component object. One of their requirements is that it conforms to interface of its component so its presence is transparent to clients.
As the code is written (disclaimer: I didn't write it) the User.find_by_email will be called in this case when it returns null we try the Guest.find_by_email.
Because this is such a common requirement, I'm tempted to ask to to expose a context manager which has the following syntax:

```
with assert_warns(UserWarning):
    ax.get_extent()
```
Huh. I thought this worked okay in my Lighthouse test case, which has Bob and Charlie with default decks of 3 Estate 7 Copper. It doesn't seem to fail in that case.

For special treasures, my intention is to just not use this step (or maybe come back and fix it up if/when I need it). It's just shorthand for the way it used to be written in throne_room.feature, after all.
maybe we should get in the habit of something like {Promise.<Object>}