labofsahil/github-event-dataset-2012 huggingface dataset:
But why does Visual Studio care how ccache names files in the cache? The name is a ccache implementation detail.

Sure, the .o prefix for object files in the cache was chosen because object files in Unix environments typically have this extension, but it has nothing to do with what they are called in the build directory. (Actually, the object files had no extension at all in the cache in ccache 2.4.)
We already have the default filter.
Indeed this cache should make sure that existing userdata values are reused when still available. There are two reasons for doing this:

* Performance, though this is probably pretty much irrelevant. Creating a user data object requires allocating memory on the heap, which is relatively slow compared to using a light user data (a light user data is equivalent to passing an integer around).

* Equivalence. This is the somewhat more important reason. A user data object is only equal to itself. In particular, two user data objects that point to the same object will not compare as equal. This cache makes sure that this won't happen, since when there still is an existing user data referencing the same object, it will be reused.

I guess I shall mention these points in the class documentation.

I'm not sure what you mean by reducing the number of different stacks. I want to reduce the number of different Lua states, but this is change is unrelated to that.

As for being comfortable with Lua metatables and registry... it took me a while to get it right. :-)
ah ok, i'll drop the dash then :)
Good extension! But maybe use `OutputStream` as _dataPath_ parameter. So you can
* render it into a ByteArrayStream to hold it in memory
* render it some placed accessed not via the filesystem, but via another protocol like jdbc, ftp, etc
Note that the CAN IDs have been reversed from yesterday's configuration in order to facilitate wiring.
looks like we're getting

    File "/data/www/addons-dev.allizom.org/zamboni/apps/bandwagon/urls.py", line 4, in <module>
       from stats.views import collection_stats

    ImportError: cannot import name collection_stats
Можно ту же информацию вынуть из ключей paysystems.
Well, the bug only occurs on PHP 5.1.6 from my knowledge...
Perhaps it should be:

    escaped = (c == '\\') and not escaped

To prevent cases where one has a value like:

    \\"quote"
Also look at ```pip install --user package```, it will use your PYTHONUSERBASE by default ;-) So now it's easy to install Theano on your ~/.local !
my gossip learn dual spec has option_id 1 and i think thats right
http://pastebin.com/G63pDydM
14 such crashes in the last 7 hours.
cool symbol
a combine between mangossymbol and R2
te va estable con la ultima actu? se me cae cada 10min con los ultimos commit y antes duraba hasta 20h :S
Also, this property is missing `defaultValue`, right?
double "it"
с одной стороны это правильно что общий код вынесен в application controller, но получается что ВСЕ методы могут отвечать на format js. Мне кажется это не совсем правильно. Я бы оставил здесь только html формат, а уже непосредственно в контроллерах указывал бы и формат js, при чем для определенных методов.
It should like this whitout cms
<code>include '../system/sparks/curl/1.2.1/libraries/Curl.php';</code>
It is not a good idea to put default values here, you should either let the exceptions escape or fail with a well-defined error message. Missing "package" and "version" fields are errors and should be reported properly to the user.
Maybe User.where(:id => ..).first would be better here because it won't raise AR::RecordNotFound and just return nil. It would be simpler to handle such cases.
Melhor fazer result.redirectTo(this).index(); 
Assim não dependemos de uma String
It is better to move all netperf analysis code to an external module, so it can be used stand alone, like it was done with iozone.
also slightly annoying about this.  resque has built in hoptoad support, so upgrading this means we need another gem to get resque airbrake to work again :(
After thinking a little bit more, it will be clearer to write:

separator="comma"
separator="void"
More changes in "set-ref-template" branch:

More comments on the code:
https://github.com/gao-yan/pacemaker/commit/2bcfb7e5f5940c2f3d105b70701571af049c9c2b

Removed the unnecessary re-setting of variables:
https://github.com/gao-yan/pacemaker/commit/f643a60e3fb2a655a30d05e465bc111f1ee7c656
This mutates the hash during the `each_item` iteration, which seems like it might be a bad idea.
Okay, then I will try to import the required data as test fixtures from the full_dump.php into our stub services.
I would say that's expected. What i understand so far, lock_ttl locks an edited resource for xxx seconds. This prevents the resource from being edited by another user, in case the user editing it at first becomes inactive (logout or anything). I'm pretty sure you don't want your resources locked for an hour :)
This sounds like a general issue with NPM or the JSHint package. Is it broken on this merge’s parent (351b516359) or that parent’s parent (04d48e8592)? If no solution can be found, try a `git bisect` to find the offending commit (I cannot reproduce the error locally).
This is not correct. We support full unicode for filenames and latin-1 for PDF contents. Restricting both to ASCII for watermarking is not acceptable.
And also add `rico` folder in `[exclude]` of `build/full.cfg`?
Awesome. Do you think I should test against `ruby-head` as well, or do people only use released version?
@markrickert much appreciated :)
Lioncash kirjoitti 29.1.2012 kello 0.13:

> Ah, figured out why, GMEPlugin is missing the following:
> 
> @Override
> public int getIntInfo(int what){
>     return N_getIntInfo(currentSong, what);
> }
> 
> After adding that, subsongs began working again.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/alankila/droidsound/commit/2bdc76a8af8b104fb7e8d09d910ca2c89c4afeb3#commitcomment-911354

Thanks, good catch.

Been changing shit around too aggressively. Still, more deleting of code is required…

-- 
Antti
Just to clarify, the dr_id is actually the date_reminder id number which is then linked to the receiving docs?
`<user ||= current_user>` 这样的写法还蛮常见的吧，并且有完整测试覆盖了，应该没问题吧？
Passing the additional args changes the api that is used for this function call. Also, not be be a downer about the difference in performance, but it isn't a real good practice to optimize in isolated instances, if this idea of passing around arguments (view, StateManager) were an accepted thing to do in the codebase then I could see this difference, but otherwise there is not benefit to it.
Photoshop mockup of proposed design
It changed in e4f4326e2857586e1e63bbd2ab394e38444d24bb (9 months ago)
actually, do we need this block at all?  they can probably count on their own. 

either way is ok with me tho!
хм, как бы consistency не пострадало...
Получается мы в вёрстке не прописываем, что это будет за элемент, а в коде js вдруг решаем, что он будет таким-то блоком...
А можешь пример полезный?

Многим блокам кстати ещё надо всякие дополнительные параметры для инициализации...
so……感觉把smart减肥了就差不多鸟
This fixes it for me.
Hi JC,

Thanks for integrating these changes :)


Here is the error message from gcc:

/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp: In function ‘bool<unnamed>::checkDataObjectType(const voDataObject&, const char*)’:
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: expected primary-expression before ‘>’ token
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: expected primary-expression before ‘)’ token
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp: In function ‘bool<unnamed>::checkDataObjectType(const voDataObject&, const char*) [with TYPE = QObject*]’:
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:197:   instantiated from here
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: could not convert ‘voDataObject::data() const().QVariant::canConvert’ to ‘bool’
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: in argument to unary !
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp: In function ‘bool<unnamed>::checkDataObjectType(const voDataObject&, const char*) [with TYPE = vtkVariant]’:
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:227:   instantiated from here
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: could not convert ‘voDataObject::data() const().QVariant::canConvert’ to ‘bool’
/Users/AJG23/GITROOT/Visomics/Base/Testing/Cpp/voDataObjectTest.cpp:82: error: in argument to unary !

Arnaud
yes it is the id of the custom report view for your space
I agree completely, to be honest. I wanted to get your commits in last night and on a whim decided to try vendoring the lib instead of doing something smarter. What I think I'll do, which will probably add more code cruft but be a bit more flexible, is use MultiJSON if it is available, otherwise fall back to the old method I had used. That way you can choose your engine using MultiJSON as always and my duplication won't interfere with any other usage of the MultiJSON lib elsewhere in a project.

I'll get this out later today.
Neat! More techs! :D

Python generating Groovy definitions out of Blender shapes for use inside Java. Me likey.
2all Marble users (including widget users): you should edit/remove your $KDEHOME/share/config/marblerc to make "raster" the graphics system being used now.
Any reason for having suppressed this property ?
Yes, it is needed. Flash is not only used for storing messages, you should probably check if something is a string or responds to to_str before printing it.
Sure. I think the following would work either:

```ruby
flexmock(Adhearsion::Events).should_receive(:trigger).once.with('/exception', StandardError)
```
Been a while since I worked on this, why do we need to coerce serverlist to a string?
Натолкнул меня на это решение патч в browser.js  https://github.com/operasoftware/browserjs/blob/master/desktop/browserjs-12.00.js#L676
Я не знаю от чего зависит проседание производительности у меня, мб зависит от процессора, мб просто комп слабый. Но лаги реально довольно жесткие.
This seems not ok with design , see http://packagist.org/packages/Aura/Autoload . You can see the previous one , which looks good with the design.
Oh thx, will wait for new gem ;)
BTW: i have problems setting a conditional breakpoint. What is the correct expression at this location for a cond. breakpoint with the condition geocode == GC.... ?
Would it work without the groupId ?
This looks fine.
@joahua IE6 also seems to consider `font-size:0` to be `1px`
Can't select it right :(
A few things...

- Can we revert changes to the spacing throughout this method?
- Use "||" instead of "or" -- that's the convention used throughout the library
- `method_defined?` also checks protected instance methods, so no need to call `protected_method_defined?(method)`
- Let's move the arity check to the if conditional so that there are only 2 cases: if the method is defined and the arity is 0, call the method without arguments... otherwise, call the argument with all arguments.
Btw, I find these git tools very useful in my own practice:
 * 'git rebase -i' - interactive rebase allows you to merge, split and reorder commits. Rebase is not recommended on a live branch, but in case of this branch I'd recommend you to use it to prettify commit log.
 * 'git add -i' - interactive add. *Very* useful tool to check in only what you really want to check in. I even prefer it over "git citool" in many cases.

I also find that it's better to never 'git add' whole directories. It will prevent you from accidental check in of wrong files. Ad always double-check file list when you write commit message - file list to be checked in is shown there.

Speaking about this commit, Sylvain is right - it would be extremely helpful if you split it into logical pieces. Because we can merge .gitignore changes, but we're not going to merge deletion of all those files. This is a good idea in general, but it should be "mainlined" first. Otherwise it will be a nightmare to maintain for us - breaking automatic rebases. Also we must check that all those deleted files are generated correctly if you use Makefile for the build.

Could you please use use interactive rebase to fold, reorder and split commits into logical pieces? Then we could merge most commits into our branch and propose deletion of auto-generated files for inclusion into mainline. Think of commits in this branch as of a set of patches you're submitting to bug tracker.
Che, y porque los tenias como simbolos? Era por alguna versión de rails?
Thanks for udpating the rake file - I thought that both namespaces :spree_warehouse and :db made sense but didn't realize that it needed both.

Also, what's up with the different new_admin_visual_code_path syntax?
@coffejunk thank you for the feedback!
The fix for needing to check numRows() might get moved to the engines.
In your commit, you renamed one var $nv to $newVersion in line 450 in 
the hook, but did forget to change the other 2 ones before, one as the 
defined in 447, the other already in use in 448. I tried to tell you in 
the forum thread, but had no luck making myself understandable.
So I decided not to rename your commit back to $nv, but let $nv be 
$newVersion at all. Sorry for that.
config.hq_fm is set to 1.
Hi Yuri, is the RSS bot already working in WiaB.net?
I've tried adding wave-dev-atom@waveinabox.net to a wave without success. Is that the correct robot address?
Unfortunately,

```<label>label: <input></label>```

Is rendered differently than:

```<label>label:<input></label>```

So, while most whitespace can be collapsed, inner spaces cannot be safely trimmed.
Original code:

if (floatA != floatB)

Modified code:

if (!islessgreater(floatA, floatB)) => if (floatA == floatB)

Definition has been changed.
Yes, sorry.

Temp fix: https://gist.github.com/1576465
There's already a cast (to ChunnkHeader *) needed. You can cast to ChunkHeader* then subtract 1 ....
It might be by design -- see #1559 . Herecomments are `expressions`, not comments -- isn't that the correct way to think of them?

Clearly `-> ### a ###; x = 0` is a bug, but `### a ### b` is not so clear.
r+ looks good. Merge it in
Hi glyptostroboides,
thank you for infos. I'm sorry, maybe it would have been more polite to drop you a line about forking, but mine is actually more or less an experiment (with GITHub, with Python and with GTK...). I'll follow your advice, though, using the develop branch as base, while waiting for/helping on a C rewrite. I cannot guarantee my full participation, I have not much spare time, but I would be glad to do all I can to help. It would be great to have something suitable for Debian too, maybe I can help on that side.

I've seen bumblebeed and I found it very interesting, I'll try it ASAP. 

As you maybe noticed, being currently on Debian (using debumblebee) I don't have appindicator, thus I stripped it off from bumblebee-indicator. My idea was to write a gnome-shell extension and/or a "status icon" using persistent notifications if available ([1], [2]). I was planning to use Dbus for communications, so it's great to me to hear that you too want do it: I'll wait for a specification on bus name and interface.

[1] http://people.gnome.org/~mccann/docs/notification-spec/notification-spec-latest.html
[2] http://git.gnome.org/browse/libnotify/tree/tests/test-persistence.c
All I did was indent this, the main work on this commit was documenting
Thank you Nan, it's somewhat important for us to be able to track changes through product lifecycle...
@stof What is the possible use case when it breaks anything?
I don't think there will be much if any impact.  Historically, most people seem to use ICS export from Bio-Formats as an interchange format when going from ImageJ to Huygens or Volocity.  In my limited tests, Huygens doesn't care in what order the dimensions are written, and obviously neither does Bio-Formats.

Absolute worst case, we have to tell people how to explicitly change the ordering during conversion (relatively easy in ImageJ, but not fool-proof).
Just a note that would be nice if the API/usage/implementation is updated for FlexSelect ...

I would recommend having all existing <SELECT> elements that are to be transformed be laid out with a hidden visibility directly. The reason is there exists a slight jump as the SELECT element is removed and replaced by the INPUT. In the case where no default value is selected and the select element has values this causes a jitter and you can see the first element of the select (briefly).
En este commit estaría bueno, mover algunas cosas a vendor/assets (que deberían funcionar exáctamente igual), pero ayudaría a distingir entre el código nuestro y el de otros.

En una segunda instancia nos permitiría cambiar más facil el código externo (vendor) o sacarlo de la aplicación como una gema.
@FernFerret main--/SerializationConfig@855aafa7bfe4171d3190e2a95090884874243625:

 * Validators: They validate property-changes (unfortunately only when the property was changed using ``setProperty(String, String)``). They have the power to accept the change, modify it/the new value or abort it.
 * Virtual properties: Something like the ``time``-property in ``MVWorld``. They're **not** saved, however they behave just like normal properties: You can use ``setProperty(String, String)`` to set them and so on.

Take a look at the tests, everything (except VirtualProperties, I forgot to add them) should be demonstrated there.
Ich bin manchmal unsicher, wie andere mit dem Sprachgebrauch zurechtkommen. Ich bin Webentwickler und würde selbst vermutlich auch einfach Wrapper sagen. Dies ist aber eine Komponente, die auch Endanwender nutzen, und da kann ich nur raten, was verständlicher ist. Meiner Erfahrung nach sind das oft eher deutsche Begriffe, wenn's nicht gerade etwas sehr Übliches ist, aber manchmal überraschen mich meine Kunden auch.  ;)
in the long run, we could think about introducing an own (themed) options menu. The framework is already there
I think this .putIfAbsent thing might be new since the commit was written. Probably best to combine the two:

```java
MyCollection old = _collections.putIfAbsent(name, c);
if (old != null) return old;
else {
  if (cmd != null) getMongo().getDB("admin").command(cmd);
  return c;
}
```
Pretty sure this is changing in the new place table approach.
next is an view object, deferred.when is no needed.
Again, a bit too hasty here, I think.

testCryptoRecordFromCollectionKeys doesn't exercise encryption using the Sync Key at all, and this test does.

This test starts with a username and a sync key, and generates a wrapped bundle of keys, ready for upload. It then verifies that it can decrypt the payload and get a bundle of keys out, and that they're the same as the ones we've stored.

testCryptoRecordFromCollectionKeys simply tests that we can generate a pair of keys and serialize it to and from a CryptoRecord.

Please restore this test.
I'd rather lift both `getAccountGUID` and `getClientName` into the API contract for `clientsDelegate`.

Phrased differently: why should `SyncConfiguration` need to know about the GUID format for that delegate? Either the caller or the client store/delegate itself should take care of that.

(It also avoids us having to synchronize here…)
Why not move this to Utils and use it more consistently?

Can you give brief guidelines for logging and println throughout the project?  We seem to use lots of mechanisms and I'd like to know which is favored.
The proper solution is probably to first fix the network code that requires the `-Djava.net.preferIPv4Stack=true`, and only then lose the option.

The question is: which code requires that option and why?
This is a quality commit by quality people.
Maybe we need to include some CSS too? e.g.
{% block stylesheets %}
{{ block.super }}
<link href="https://ajax.googleapis.com/ajax/libs/jqueryui/1.8/themes/base/jquery-ui.css" rel="stylesheet" type="text/css"/>
{% endblock %}
Is the libuitls.so form cyanogenmod or stock rom?
git://github.com/DooMLoRD/Xperia-X10-Kernel-2.6.29.git
need to re-register CalendarRelation
You also forgot to remove libmmjpeg line from ace.mk.
I guess my main concern is that it makes projects look more "uncovered" and "bigger" than they actually are but If that is your observation than maybe thats still the better choice.

phplocs eloc counting only works with bytekit so thats not an option ether .. hmm. From my experience just telling people to turn of addUncoveredFilesFromWhitelist did the job. Maybe just turning that off by default in PHPUnit/Util/Configuration.php would be enough?

One of the inhouse libs (140kloc) would "grow" to +200kloc with this and in result we'd "loose" quite a lot of coverage, percentage wise.

Not a major issue but maybe it explains why I'm asking again :)
The very unprofessional tweak is required by Felipe when I found this facility is missing, he said "First just add a link at the beginning of the form so to remind him he need to make a design about it and also consider where to put this link",  then we could make a good implementation based on the design. :)
Tried fixing the mount usb issue, also tried fixing the display being too dim. Hopefully this works....
Torch is not working. I think it is caused by the Camera drivers we are using
Ohh, I didn't see that ;)
Salut David,

Le projet Thunderbird est reparti sur de meilleurs bases, tu pourras 
télécharger cette nouvelle version à partir de la branche 
AddressBookOverlay. Un entretien via skype serait il possible mercredi 
ou jeudi en fin d'aprem pour te faire un compte-rendu de l'avancement 
des deux projets ?

Baptiste
Should this be `MultiJson.dump`? Same for the `respond_to(:load)` case using `MultiJson.load` below.
Why are extras treated differently from other dependencies (always recursively upgraded)?
Eclipse auto-drops a .project file in the top directory of a project workspace.  Since there were other IDE/repo patterns in .gitignore, I figured it was safe to add. If others are using Eclipse, they can have their own configuration without having to worry about clobbering.  It'd be fair to have this be part of a separate pull req though.
This line should below the "&Intel" line
Is 2^16 the maximum number of blocks allowed by CUDA?
Yes, we should definitely do that. Not allowing users to comment their kernels seems a silly restriction.
In fact, _is_global appears twice in the Arg class already too.
msg.getClass will be None or Some right?
I like the use of SGFCR. Good thinking.
should the return type be `FailureDetector`?
Might want to use Mapper instead of JFunction
kit it is
case object?
what happens if negative or 0?
These should be in a companion object like usual.
no, that was overlooked, there is a "code" missing.
why map + foreach?
how? `context` is an unused var since you got rid of `noConflict()`, or is this a Closure thing?
I'm fine with doing this differently, but see the comments on the pull request for just the refresh stuff.  It is connected, and maybe we should even move it up and add it to that  change.  It can't be that costly because this is the fastest branch we have (from the measurement of my tests runs, which isn't too accurate ...
Yes, but then we have lot's of 'raise' code we need to actually handle in some sane way, so why is this different.?
All that's happening here is we are using the updated AR apis for everything and we have no need for that helper anymore.  We are generally looking for a name and a set of types, and the new syntax handles that directly.  The new constants are arrays of strings (the codes)
Yea, the db must be cached somehow.
@jmarshallnz has already mentioned this earlier and this was my response:

I don't really like doing the sorting myself either. The reasons why I did it that way is that I did not want to change the original order of the streams as read from the container. Furthermore I figured that in most cases there are one or two streams to sort so using std::sort doesn't really seem like it will provide a huge advantage. But if we don't really need/care about the original order of the streams I'm happy to remove the custom sorting code, implement the needed method to pass to std::sort und use that one. I'm guessing @elupus will be the man to tell me whether we want to keep the original order or not.

So if we don't need to keep the original order I can switch to using std::sort. Otherwise I'd either have to use the custom sorting implementation or copy the whole CSelectionStreams object into a new one in which I can change the order (because it's just a copy).
Maybe OnFilter() ?
fixed using ThumbLoader in latest push. there's an issue with ImageFile.cpp opening thumbs but hopefully that's an easy fix. listings are much faster now.
No.  I added it because otherwise `rake test` would fail when attempting to require it.  I'm not sure how bones works, but `rake test` seems to run bundler and in so doing regenerate `Gemfile.lock`, so if shoulda isn't in the Gemfile (either directly or indirectly via the gemspec) `rake test` won't work.
thx updated.
You should use lists:foreach as the result is ignored.
Better:
```php
rtrim(str_repeat(',?', count($arrPks)), ',');
```
Add .php
That should be clear by declaring the argument as mandatory in the ParameterList
``call_user_func_array`` is 2x slower (5.3.19 linux, 5.4.9 windows). ``callMacroWithNamedArguments`` method with optimalization is 1.2x - 1.5x faster than method with simple call ``call_user_func_array``.
just #include "src/encodings.h" and make sure the include path is set as appropriate
Don't forget the spaces between the statement and the parentheses. Applies to the below as well.
Done, it's back to 0.2.0r1 now. I kept the r1 because SemVer defines it that way.
They're not yet used-- as the comment for the whole class states, I've yet to add in cell styles with labels and images. I'm working on that too, but I'd rather this PR go through first before I muck it up with half-baked code.
Why are you using `instancetype` here? We know it will be a `NSValue *`.
Indentation :trollface: here.
Ah, I completely agree with you.
The `.IsEmpty()` and `.IsString()` checks are not necessary, `String::Utf8Value` takes care of that.
OMG! str[0] = '\0'; after the malloc fixed it... crazy C!
sort.c is no longer generated, although I suppose a few extras don't hurt.
missing a  close folder comment /* }}} */ here :)
As we use C89/C90, declarations may not be mixed with code. Please move the declaration to the beginning of the block.
Instead of using double-quotes surrounding ``less``, use tick marks to show the command in a mono-space font. For example:

```
``less``
````
ModelNaming sounds better for me
@pixeltrix true.
Dalli::Client methods return true/false instead of string responses.
One last thing: wouldn't  `template.source.inspect` be a better alternative here since it would automatically escape quotes?
Shouldn't this require stay, pointing to AV? (I know it's autoloaded, but Rails does require such things, or not?)
sounds good, indeed :)
it creates fewer intermediate datastructures
I don't disagree with your comment. I'm just trying to keep records and commentary in pull requests clean for future reference. Thanks for understanding!
First of all, why is there a method called getMemberType which returns a ... Symbol? Why is there something called MacroContextPrefixType which is a... Symbol? Did I miss a memo? What justifies these confusing names?

Jason is right, doubly so about the method name.  Method in Types should be named accurately and used from more than one place; this doesn't seem to be either.  (I am sure there are plenty of others that fail one of those tests.) And we sure don't need the 7000+ line Types.scala attracting new code without clear cause.
This element uses the same data-hook as is in `payment_methods/_form.html.erb`. I think we should change it.
When is the second term of the disjunction ever needed?
@smichr @asmeurer 

I'm somewhat unsure about this. The problem manifests as follows: `gamma(3*exp_polar(I*pi)/2)._eval_evalf` throws some mpmath internal exception. The problem is that `_eval_evalf` evaluates the argument to 1 binary digit accuracy, which apparently is `-1`, and then evaluates gamma at -1, causing the exception. I believe calling `n` instead alleviates the problem, since then a special routine for evaluating the gamma function is called, which knows about the exception, and tries evaluating at higher precision. [Which works, of course.]

I wonder if something else should be fixed.
Missing a space after each comma.
does this look ok in the docs? I would have guess that where should be followed with double colon...but could be wrong.
It is used by default when outputting the results, however:

 - it is done by substituting all symbols starting with "C" *at the end of `dsolve`*. It is not at all used internally.
 - the fact that the names do *not* start with a "C" permits the solver to distinguish constants that were created internally (not instances of this class), from constants that were already present in the input. This is important if we are solving triangular systems, because otherwise the constants get mangled. In the future the internal methods should also use this class (as it subclasses Dummy, there will be no ambiguity based on name strings), however this would require quite the effort.
 - the "C" in front of it is added by the printing methods.
`::` -> `:`?
It is worth to use IndexedBase, but the module 'sympy.tensor.indexed' uses 'exp' from sympy.functions, which in turns uses this combinatoric module.
If null object is explicitly passed then question mark should be used.
Maybe i'll explain why he could eventually *hate* you :).
The idea of Fibers+eventmachine is great but at the same time is NEW and still not widely recognized and not supported in many areas. You know that there is no good PostgreSQL driver for EM at the time being?
So while you realize how good the idea is you soon realize that you have to code *A LOT MORE* by yourself.
In other projects, I've encountered strange (non-obvious) issues because of a missing `bundle exec` before, so I'd always recommend `bundle exec rake dist` for instructions like this.
Backup might not really be needed.  I think, if nothing else, that the added complexity required for it seems like more trouble than perhaps it is really worth.
This is broken. See how it was above. This mixin generates broken CSS if you use it that way since no browser recognizes all of these properties as a valid `transition-property` value. Instead you need to specify it multiple times so that at least one is completely right.
It means if there is an error it says if the previous command failed or not.

It just checks if composer.phar exists
Originally it was, but I couldn't get the alias to work as a Bourne script.

Justin Venus
On Aug 7, 2012 8:26 AM, "Ben Noordhuis" <notifications@github.com> wrote:

> In autogen.sh:
>
> > @@ -0,0 +1,16 @@
> > +#!/usr/bin/env bash
>
> Is it possible to make this a bourne (not bourne again) shell script? On
> e.g. FreeBSD, bash is not installed by default.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/joyent/libuv/pull/507/files#r1321014>.
>
>
Why did tests did not catch this bug?
Consider adding a test (should be one liner).
"only affect which members are chosen to read from" is clearer
specify the `geoHaystack` *index*.
Other modules are loaded as they currently are (via module ID rather than basename). The issue was that when the module was loaded by ID, it loaded the first submodule that was not set to hidden. The folders are controlled by the (hidden) viewfolders module (or something to that effect) and so even though it is first in the database, it is skipped when the modules are loaded by ID because it is hidden. This simply makes it load by basename so that it won't skip the hidden folders module, so folders will be displayed first instead of compose message.
Noted down. Will be removed in my next commit
This is an array.
> Just don't expect it to be as explicit as it is.

The name should reflect the purpose, not the implementation, as implementation may change.

With that, the name should be clear as to what it implies. For example, looking at the name of "Limited access" I have no idea what it implies.
To reproduce on this branch, with my code still in place, remove your profile and signature permission and try to navigate to the "Edit account settings" from the index.php. You wont be able to, as it displayes `You are not authorised to define a signature.`

So you see, the profile module is skipped, but the signature module isnt and therefor throws the error.
I don't like this wording. Perhaps
`This text will be displayed as a link to your website homepage in the board’s breadcrumbs. If not specified, it will default to ’Home’.` or something would be better?
Agreed. Sounds like a good approach to me. I'll bump it.
Missing yuidoc comments.

Can't the `type` parameter be inferred from the file extension?
Move to own file
Here is where you want to search on a provider by a patient, correct?
@alok and jming: try looking into the Cohort class in OpenMRS. I haven't used it, but it is a way of grouping patients together -- this could be what we want. (Not for this pull request specifically, but in the next few days.
Please name this method more specifically. Validate which field? And why are you validating in the controller, rather than the model? See info on Validations here:
http://www.sencha.com/blog/ext-js-4-anatomy-of-a-model/
I intend to remove this once pushing to the repo, just wanted to make it easier to review by not deleting at this stage.
Please revert this. Users can no longer copy & paste this easily.
I had some bugs in the way I added up gradient terms in my earlier PR. This PR fixes them. (@nouiz, this is the part that fixes the pylearn2 tests)
This is just extra work and noise, we already materialize the bytes in the EE, run it there on the parameter buffer and you are done.
This returns a list, which is converted into an array to be consumed in ClientInterface (as does the next method). Can we just make this method return an array?

The getSQLStatements() is consumed as an array in one place and as a list in another, where it is shortly converted into an array. It probably should also just return an array.
Just to suppress a warning. I tried to improve the comment:

    Prevent warning  about fis possibly being null below.
Should the abort message include the output_path?
use "Testing base asset fields" instead
Please keep aligned with the rest of list
I'll re-add the error message here since we already had it in the first place. I'm not convinced that this is the best approach in general though, and suggest would suggest most cases take advantage of the new `riak_core_capability:get/2` function to have a sane default.

The only place a user can provide a capability is by using the override settings. There really isn't an "invalid value" for `use/prefer`. An acceptable use-case of the override system is to force the cluster into a mode that isn't registered as supported by default. For example, we may have a code path that matches on three modes: `version1, version2, experimental-version3`. However, we only register `version1, version2` as supported capabilities for the given release, so the experimental mode is never negotiated. Yet, users (or our support staff) could use the proper app config override settings to force a node to use the "unsupported" version.
Can we just remove? Or does having the commented out tuples help with comprehension, do you think?
See above - use return code variable and call xmlFree only once for each node.
can parser or parser.document() be invalid pointers?
use same style as previous examples: "Uses a default ``template_name_suffix`` of ``_archive_today``.
Looks like you removed a chunk that was marked as a bug fix?
The warning was
    
    /Users/schopf/Documents/Development/fschopp-madlib/src/modules/linalg/average.cpp:173:45: warning: conversion to 'Eigen::MatrixBase<Eigen::Map<const Eigen::Matrix<double, -0x00000000000000001, 1>, 0, Eigen::Stride<0, 0> > >::Scalar {aka double}' from 'long long unsigned int' may alter its value [-Wconversion]

Converting to int32_t is certainly not going to make things better. The correct way is of course still to convert to double. The warning is because double only has 52 bits of precision, while uint64_t has 64 bit of precision for integers.
I thought the product of the PGXN deployment target would be a zip file. Sure, there are some differences in what it includes, but still the expected behavior would be the same as for other deployment targets like RPM or a Mac OS X package installer.
I try to put style changes in separate commits.
`if MacOS.version == :mountain_lion` (would also prefer the conditional to be inside the method definition, e.g. `DATA if MacOS.version == :mountain_lion`
Watch out the indentation
```_(Update from the flight deck: ' posted an urgent message:')``` will give an error. 

In django templates ```_()``` is shorthand for saying, please translate whatever is between the brackets. The bit that is between the brackets needs to be a string. Strings can be quoted with a single apostrophe ex. ```'This is a string.'``` or a double apostrophe ```"This is a string"```.

So, an example of using ```_()``` is ```_("Please translate this string?")```

If you need to use a variable like ```{{ satus.author }}``` in the middle of the sentence you can break the sentence in two or use blocktrans. It's better to use blocktrans, since splitting the sentence will make translation very difficult!

The above using blocktrans will look like this:

```{% blocktrans %}Update from the flight deck: {{ status.author }} posted an urgent message:{% endblocktrans %}```

Notice that the sentence is not a string anymore and is treated as normal HTML!

I hope this helps a little, if anything is unclear, please ask
Non, il ne me semble pas que j'ai modifié ces fichiers, ou alors des modifs mineures.
Когда я вижу "сливать одну ветку с другой в течение большого промежутка времении", лично у меня возникает ощущение, что одну ветку как-то очень долго сливают в другую. А тут нужно сказать именно то, что ветки на протяжении долгого времени сосущестуют параллельно и  периодически сливаются одна в другую.
The error code is used to differentiate between different precondition errors. We use RainError constants for the error type. The error codes are too specific to put them in the RainError and are used only in the unit tests. I think strings are better than integers in this case and ``cb`` comes from callback.
Why are singleton bindings being passed into this.
roger. Will get rid of it.
we will want to make a method to validate/clean group names. make sure they are slugged properly (all lowercase, no spaces, etc). that can be done in lue of URL encoding since our slug format is URL safe.
The problem with the `expected` attribute of the `Test` annotation is that the test will pass if any code in the method throws the exception. Usually it's better to use the `ExpectedException` rule so you know which method threw the exception.

In this case, I would do:

    tempFolder.create();
    tempFolder.newFolder("level1", "level2", "level3");
    try {
      tempFolder.newFolder("level1", "level2", "level3");
      fail();
    } catch (IllegalArgumentException e) {
       assertThat(e.getMessage(), containsString("level3"));
    }

Also, for completeness, we should add a similar test for the one-argument form of this method.
I thought you might say that! Entire dependency is <3kb. Horray!
And we should probably define this outside of function scope
Clean up this bit of dev logging.
This ideally will be the basis of a new button mixin—should I create it in this pull request? It already feels mammoth.
`clickDialogButton()` is already supposed to wait for the dialog to close. If this line is needed, something's broken there (I guess its no longer guaranteed that timeouts in one window are sequenced interleaved with timeouts in another window?).  We should try to fix `clickDialogButton()` itself instead of patching around it here.
We should put a comment at the top of the module documenting what events it emits.  (Similar to what we do for e.g. DocumentManager).
I wonder if the hint list ought to be closed when there are no triggered hint providers. I see the following scenario with the in-progress JavaScript provider. 

1. The hint list is open after typing the last character of a keyword (e.g., "var"); 
2. A space is typed, and the the provider's shouldShowHintsOnKey method is queried;
3. The provider's shouldShowHintsOnKey method returns false, and so triggeredHintProviders is empty; 
4. The hint list remains open and is updated by querying the "current" (but now outdated) provider's getQueryInfo method; 
5. Many new hints are returned because, from this position, in which the cursor is surrounded by whitespace, many tokens constitute valid hints.

Instead, it seems like the list should be closed when shouldShowHintsOnKey returns false and should only be opened again when that method returns true and provides a non-empty list of hints. Although the provider could attempt to detect this from within getQueryInfo by observing that the cursor is surrounded by whitespace, this would make it tough to handle explicit requests (via ctrl-space) for the hint list. Also note that handleSelect is not called in this scenario, so that method's return value doesn't have a chance to affect the list's status. 

Or is there another way I haven't considered for the JavaScript provider to handle this scenario?
Naming nit: DocClass should start with a lower-case "D" - docClass. This is true for the other two instances below.
I totally miss this requirement when starting out. as I got busy with 3 clients running my brain jumped to it being a requirement somewhere.
Such comments always looked useless to me. ;-)
No, we want only mongoengine-based authentications. Not also others. We have to rewrite others to work on mongoengine.
Above you seem to default to Hybrid, and here to ROADMAP
"Team *description*"?
I think I would rather see the try/except block moved up so that there's no chance of an ImportError or other error loading ipdb covering up the actual error. `call` could be `self.call` and assigned in `begin()` or `configure()`, for instance.
goto :/
cachedir. The default configuration puts it under */var/cache/salt/proc*.
判断NULL
中文版的文档，都用全角的中文标点符号吧。
Please describe what the input format is like.
probably a merger error.
There is no folder named "example", should be:
s.clean_path = ["Example", "Screenshots", "*ARTableViewPager*"]
I am not worried about performance I am worried about the maintenance of the webapi.js file.
There will be one extra .json file in homescreen that will include all the manifests for gaia.
Its not code and its auto-generated I don't feel like it adds overhead.

That said if you feel strongly about keeping everything in webapi.js I can include it there
but I don't feel good about mixing the auto-generated data with the hand maintained code.
This seems to be defined twice (see just above)
I know this is less efficient, but I find it convenient to pop up the debugger, set DEBUG to True and start getting tracing from there on out.  If this is all determined at import time that doesn't work.  Maybe I'll add a TODO that once the parser is well tested in the wild we change this to what you suggest?
Yeah, it was just because of `setLevel``, but I'm happy to change that. I don't want to change all the methods to semi-camel-case, but I can change this to ``set_color``. Any strong opinions?
I think there's a small amount of fragility here. Results for any single step are not guaranteed to be returned in any particular order. Because of the way inputs to that step are unique-ified, and because the number of inputs is low, and because this test system is not under heavy load, it's very likely that they will come out in lexicographically-key-sorted order. But, we could see this test flap on some systems.
I'm allowed to use 2 rows and I chose the exact format used all over the source, I think that is just fine.
GetProxy already returns the same information (addrProxy, used), although in a slightly different form. Either keep this function or GetProxy, but we don't need both.
I'd rather you just timestamp when it happened. That seems more useful to me.
Could you steal some sample files for these? Drop them in samples/ApacheConf/filenames.

After you rerun `rake samples`, it will tell you these are safe to remove from this list.
I'm pretty sure that taking the else clause in this case will set rmlen to be 1 too high. I think the test for !unrecognized should be inside the if clause and simply prevent setting *rmlen=0.
Also, we should think about how this can work in Windows. I'm sure it's possible, but it won't be quite the same.
should packaging be bundle by default?
Thanks, the example helped me understand.  This case is somewhat pathological, since there are a number of pitfalls already associated with DefaultWeightedEdge (e.g. you can't reuse an instance in two graphs, otherwise the weight will change in both), even without custom edge subclassing.

So it would be a shame to have to make the performance suck in general for this one rare case.

I can think of one hack which would allow us to avoid it.  In AbstractBaseGraph, we can use edgeMap to map to the corresponding edge in the second graph, and then use that in the equals/weight comparisons.  To access this, you could add a method in AbstractGraph which just returns the original edge, but in AbstractBaseGraph override it to do the mapping.  However, if the map returns something with class IntrusiveEdge, then ignore it, since it means it's a private object constructed by createIntrusiveEdge.

What do you think?  Does that work?  And is it too ugly?
OmniAuthに最初からついてくるdevelopment strategyだとformのmethodをいじれないぽいんですよね……。
Are all those `strip()` really needed?
I concur. Well spotted @kumar303
s/NoProcessorsRegisteredException/NoProcessorsRegisteredError
right, well spotted, I fix.
Ok, maybe we don't need it at all
Say `Guarded by 'this'` rather than `Synchronized on 'this'.` ?
Ok, I wasn't aware that there was an indent call. I'll make the changes
@bschussek why would the default group be cascaded instead of the current one ? And btw, wouldn't it duplicate the validation on the child if you cascade the default group several times ?
Please add `Use {@link OneOf} instead.` here.
could you replace ``mixed`` by ``callable`` ?
shouldn't you use an exception of the component here if you start introducing them ?
What about adding an ExceptionInterface in the component to be able to catch Finder exceptions more easily ? This would be consistent with many other components.
this comment is wrong. There is no reason to change the Output object as the commands are not about creating an Output object at all
Yeah my bad, I missed those methods...
Mhh ... not sure. As an example: I am trying to put this in Symfony for use in Drupal. All our use cases would not use any Content-Disposition header at all. I would like to make it easy to not have such a header. Would something like `$autoDisposition = null` that can be set to both DISPOSITION_ATTACHMENT and DISPOSITION_INLINE and would then have a Content-Disposition with the default filenames cover most of the use cases?
@stof [Yes, we can](http://www.phpunit.de/manual/3.3/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.errors) :)
AFAIK, this is not enough. ORM 2.2.3 could accept common 2.2.2. and the real 2.2.3 requirement is on Common in the bridge (thus, this does not change anything there as it is in the suggestion which is simply plain text)