labofsahil/github-event-dataset-2012 huggingface dataset:
Any instance of an included file not needing cfoutput would be very rare and using a terrible implementation, most likely using Evaluate.  And there really is no need to attempt to reinvent what cfinclude does (wrap around it to extend it - yes, like what cfwheels does) but cfinclude quite rightly does not transfer the cfoutput status from the current file onto the one being included.

I don't think wrongly highlighting non-cfoutput hashtags is the solution for the rare case someone is using a framework that uses the cfm files in a non-standard way.
Hi,
Suggest posting it on the forum since Kevin (yehster) will likely want to look. Also, rather than posting only the commit, suggest also posting the branch (since there are many commits on that branch), which is needed if somebody wants to test the code. For example, if you post it to the same branch as you have been, then the branch link is: http://github.com/tajemo/openemr/commits/datedReminders .
thanks,
-brady
This just makes memcached not compile for any version lower than 2.1.25; not adds support for.

    gcc -std=gnu99 -DHAVE_CONFIG_H -I.  -DNDEBUG   -g -O2 -pthread -Wall -Werror -pedantic -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls -MT memcached-sasl_defs.o -MD -MP -MF .deps/memcached-sasl_defs.Tpo -c -o memcached-sasl_defs.o `test -f 'sasl_defs.c' || echo './'`sasl_defs.c
    sasl_defs.c:145:20: error: ‘sasl_callback_ft’ undeclared here (not in a function)
    sasl_defs.c:145:37: error: expected ‘}’ before ‘sasl_log’
    sasl_defs.c:106:12: error: ‘sasl_log’ defined but not used [-Werror=unused-function]
    cc1: all warnings being treated as errors


$ pacman -Q libsasl
libsasl 2.1.23-9
sgs.ai_slash_prohibit.leiji 真的工作了吗？我没改之前见过n次一名角色杀张角然后被劈掉2体力。。。
这样的话牌的显示次序会乱的吧……虽然也不知道会成什么样子……
I agree but t.Logf doesn't display anything if the tests "pass". When
the tests that are skipped skip, they "pass"; so no output. This leads
to false positives which is why the pq tests are broken.

On Fri, Jan 27, 2012 at 8:59 AM, Brad Fitzpatrick
<reply@reply.github.com>
wrote:
> This is kinda gross, but fine.  t.Logf is really the right way to do output from tests (the only way that gets buffered and intermixed correctly).  But if this is the only output of the test, it doesn't matter much.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bradfitz/go-sql-test/commit/1766e220792df9c501fb5e23dbd96ef7811c749f#commitcomment-908781
But you can do emitting already too:

    $socket->sockets->emit();
I think this part should just be moved, and replace the code I suggested in this patch:

```
$copy{$map->{upload_id} } = $id;
```
Use c.getMilliseconds for timeout config
I think having a convention is good. It is just hard for me as in almost every programming language a dash is representative of subtraction.
You are removing unrelated code here. Need to be careful not to remove others code. This is likely because your github repo:
1. Is not uptodate
2. Relying on a huge merge commit to update the code (rather than correctly maintaining a git repo)

See http://www.open-emr.org/wiki/index.php/Git_for_dummies
Note - Google Translate charges per character sent.  So, although it won't translate the contents of HTML tags, if you send them, you'll get charged for processing them.

The changes from this commit does properly deal with the new EM syntax (so it uses the EM SplitStringOnExpessions() command to separate out expressions, and properly re-joins them.

If we want to reduce the translation cost, we could parse the string parts that do need to be returned for HTML (e.g. create a DOM object) and just pass the non-tag content; but then we'd need to properly re-construct the HTML based upon the return values.
Meterme esto cuando podáis ;)

https://github.com/Radicalmangos/devmangos/commit/4e11985ba499ea606e85179b250691e8702bcbb4
@Bananeweizen Ouh, the first time i really have to say: please revert ,-) Each cache parsed in GCBase.parseMapJSON() is now display as a Mystery in the Live Map. And in 80% (?) this is not true. As far as the bitmap parsing doesn't work 100% correctly (i doubt it will ever do so) we don't know exactly what type a cache has. We shouldn't pretend that every unknown cache is a Mystery.
Any reason for this debug call? This looks to be called quite often and can produce a lot of messages, so it would be a good idea to remove it
getHeight return 256 and 256 != 255
get open\closed issues functions
Thanks Tim!  I'm obviously going to quickly, but I was worried about these regression bugs getting into the RC.
hi mike. what time is New version upgrade ?
Part | Points
------------- | -------------
base | .75
Adv | 1
*Total* | 1.75

Fails some tests. :cloud: 

```
======================================================================
ERROR: test_normalize_many (__main__.NormalizeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 129, in test_normalize_many
    self.assertEquals(normalize((0,0,0,1)), [0.0,0.0,0.0,1.0])
  File "/Users/jacklaxson/Projects/homework-cs112/Grug16/hw08/math_funcs.py", line 39, in normalize
    vector[n] = vector[n] / distance
TypeError: 'tuple' object does not support item assignment

======================================================================
ERROR: test_normalize_two (__main__.NormalizeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 124, in test_normalize_two
    self.assertEquals(normalize((1,0)), [1.0,0.0])
  File "/Users/jacklaxson/Projects/homework-cs112/Grug16/hw08/math_funcs.py", line 39, in normalize
    vector[n] = vector[n] / distance
TypeError: 'tuple' object does not support item assignment

----------------------------------------------------------------------
```

```python
    if not (distance == 0):
        for n,v in enumerate(vec):
            vector[n] = vector[n] / distance
        
    return vector
```
You need to create a new list to return if you're receiving a tuple, honestly thats a silly reason for this to break but its in the testcase.

```
FAIL: test_separater (__main__.OutputTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "adv_nims_tests.py", line 51, in test_separater
    self.assertEquals(separater(range(1)), "-" * 22, "expected 22 -")
AssertionError: expected 22 -
```

```python
>>> import adv_nims
>>> adv_nims.separater((range(1))
... )
'======================'
>>> adv_nims.separater(range(1))
'======================'
>>> len(adv_nims.separater(range(1)))
22
```

Ok that failure is not a bad one and won't count against you.
Nope, item also missing on 3.3.5a
I didn't mean it that way either. If only you had properly read my reply...
Your product needs a serious revision, to ensure that you can not ...
Sorry if I have offended .. but you have a staff who can not communicate even with those who can help in your hard work ...
*Closes #5112
Hmm... I have no idea what I'm doing.  I meant to save this and continue it later.
Alright, all merged!
This should fix issue #6
Shouldn't this check for the environment? The bootstrap file is loaded without protection from web/index.php as well.
wszystko chyba masz dobrze tylko stara wersja repo bo widzę na Jenkinsie ze masz stara wersje AsciiCharacterClassOperator.java
Boh, funziona ed ho 3 pull request aperti da quasi un mese -.-
Hi,
thank you for your notice! I fixed it and finally test is done.

2012/1/6 Michael Klishin
<reply@reply.github.com>:
> Private github URLs won't work on travis. Our VMs don't have your private ssh key.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/kengonakajima/travistest/commit/f06472d07becc85dce117efbff4226802cbc527e#commitcomment-841891



-- 
Kengo Nakajima / 中嶋　謙互
kengo.nakajima@gmail.com
http://twitter.com/ringo
090-4278-0336
Yes but that part's just for building the gem.. it inserts the right version into the gemspec so that when we push the gem up to rubygems.org it has the current version..
YEA! Jenkins works now! :D
The originator of this issue has tested the fix in an sdk I provided using this pull request. The user is not longer able to reproduce the issue. PULL ACCEPTED
Samenvoegingen van een Engels en Nederlands woord zijn niet zo mooi. Daarom liever homepage dan home pagina.
а почему поле публичное?
User caps are managed a bit differently
minor detail... missing a `]` in the example

```js
define(['knockout', 'knockout.mapping'], function(ko, mapping){
```

also, make sure that a module id of `"knockout.mapping"` actually works.  its possible that some loaders see the `.` and think that `mapping` is the extension and may leave off the `.js`.  since you're not likely to test this with all loaders, at least make sure it works with the one you use - you probably already did but just wanted to make sure.
logstash-web doesn't have a config file
@jhinkey - Fixed all of these issues. You can double check and we should be good to go.
transit method return dojo.DeferredList object, so wait for the DeferredList object resolved and then do next transition in the waiting queue.
Please bump PR; after that I will merge this :-D
@ryanlecompte: Thanks for pointing out the first thing I would have said had we continued this conversation elsewhere :)
Thanks Stella. Corrected in trunk.
for rails 3.1 better then renaming css to ERB is renaming it to .css.scss and using image-url("") in place of url()
ALLEEN_WIFI
Derby doesn't support locks on FOR UPDATE?..
I should probably factor out some of the boilerplate in these tests, but I'll fold that into the review comments.
Nice, GCMods
Thanks for ironing these capture edge-cases out, nice stuff as usual !
Try to get consistent ordering of attributes. These two are mirrored!
So then phpBB automatically assumes >= ? because some extensions that work for 3.1.0 could also work for 3.1.1 and (maybe even) 3.2.0
Great insights,

I was thinking about something like TTTAttributedLabelStyler wrapped by TTTAttributedLabel, but your extended version definitely makes sense, it is way more modular.

I'll try to pull this together tonight.

By the way, do you know of any good implementation of a Markdown/Textile parser in Objective-C?
What build breakage exactly? When I built with both interfaces in place, things seemed fine.
There is no x|b (i.e. x|a) witch aren't composed of primes of c 
If x|b and x∤a (i.e x not in c) then b∤a (reductio ad absurdum)
Зачем это в IO?
small hint -->   texteditor -->  tabs to space  4 spaces :D  codestyle :O
FWIW, this doesn't actually test that the `str(level)` was used.  It checks that it returns the user's name plus ` (foo)`.
no i rollback with this sql update on my world db. works fine, the repro is bad
Error prone maybe, but you do a cut & paste anyways. Quickly removing a line after the paste does not seem to be such a big overhead. 
An alternative would be to remove all email addresses and maybe list the authors in the top level package.html file so that they appear here - http://docs.jboss.org/hibernate/stable/beanvalidation/api/

Looking at the above URL it seems that we are missing some package level docs anyways, but I think @gunnarmorling has already mentioned that :-) I can have a look at the docs. I started anyways with some modifications there
also use $ git update-index --assume-unchanged web/hourly-predictions/scenario-template.json
`for ( $i = 0, $count = count( $parts ); $i < $count; ++$i ) {`

or

```
$meta_count = count( $parts );
for ( $i = 0; $i <= $meta_count; $i++ ) {
```
If you use HTML5 tags, you will need [html5shiv](https://www.google.com/search?q=html5shiv+cdn).
I appreciate matching the S3 attribute name, but we use snake-case for any Ruby stuff consistently.  So, I would make this `version_id`.  Likewise, since it's a collection, I'd really make it `version_ids`.
There are a couple of other lang based tests in the system too. I just used the default use case that was given in the bug. They will both work technically the same way. But I'll add this test in the morning.
Makes sense, I was going off the assumption it should go here since the EhCacheBackedMap lives here.
I think it probably should raise an exception if you try to access the data of a temporary dat.
this will be fixed once you switch to CLQ
will add a comment explaining expected semantics
well, missing sender is one thing, wrong sender another.
it will when it has been called, this actor is stopped
comment explaining why
No code in rst doc please, put it in a scala file and import it.
why shutdown after start of successor?
Sure. I can add a check for the node address as well. But it can only be one.
sure, I think 15 is reasonable... I'm not sure I'm seeing the other line you're referring to?
So now we have two functions to convert values GSIM operates on: from values in a unit of IMT to normal-distribution space and back. Now they're called convert_imls() and convert_intensities() respectively. These names are not very much descriptive, so I accept suggestions about better ones.
That's not gonna work. If people give a str (not unicode) with "Fenêtre", the ê will give issues about encoding/decoding error. You must handle thoses cases, and remove the case in _set_text() then.
Remove subversion from dependencies
I actually started that way but decided this ended up being more easy to read.
I've made those changes in 54e8722

Over to you...
@tenderlove isn't a `reject{}.map{}` going to be slower - that's iterating the list twice, not that performance here is a concern.
this is highly confusing.  You're using the word "key", which has a precise meaning, to mean id.  we came up with "mark" to try to avoid this confusion.
bad bad bad.. will break rar/zip and others...  We must be able to encode full paths and have the not break url parsing of the full url. What you had before was right.

You can never ever url encode a full path, unless what you want is to make it fit in a component of a url.

comp1://comp2:comp3@comp4:comp5/comp6/comp7/comp8?comp9#comp10  and so on.
Yeah, because of the optional <name> tag.  No idea if it's used or not in the scrapers. @vdrfan you don't happen to know?
А почему именно так сделал?
а, нет. это можно
для чего нужна эта функция?
Would this be equivalent to other_zygotes -= self.prev_zygote ?
Just to keep the diff minimal. Otherwise the whole call chain needed to be adjusted for this return value:

`run_test -> make_cmd -> get_suites -> find_suite_path`

I think for exceptional cases like this it is fine to use throw; specially since I only catch the skip and nothing else.
closing over "this"
maybe is better to use toNumber as the method name and then static import it
Why enyo.job and not enyo.asyncMethod ?
this will throw a notice if there is no paths for the default namespace and you omit the namespace in the template name
Wow, really? I read it in one of the Clang release notes.... or am I confusing things... +____+
I like it! +1.
is exec imported?
It can be `"SQL"`. Some places use it.

https://github.com/rails/rails/blob/master/activerecord/lib/active_record/relation/finder_methods.rb#L184

https://github.com/rails/rails/blob/master/activerecord/lib/active_record/associations/preloader/has_and_belongs_to_many.rb#L16

Ping me when done.
I know, so how I said - I don't like Class.new syntax if we can use more readable `<`:)
Or should a deprecation warning be added to the *master* instead?
These assertions only test that the return value of `transform_keys!` is upcased. They do not assert that the transformation happens in-place. I mean something like:

```ruby
@symbols.transform_keys! {|key| key.to_s.upcase }
assert_equal @upcase_strings, @symbols

# a separate assertion for returning `self`
assert_same @symbols, @symbols.transform_keys! {|key| key.to_s.upcase }
```

Also, I don't think you need to test all 3 hashes (`@symbols`, `@strings` and `@mixed`) since in the block you're calling `to_s` on the key anyway.
Why extract this method. If all that it will do is assign a hash key to a value I prefer the see the assignment.
I think I'd rather keep this in since it is more explicit. We actually do mean to return nil in this case.
_ is a good variable name for "don't care"
Man, I almost feel we need a clone like this in the GenericCompanion classes that just `new`s a builder, pipes in the collection, and grabs the result.   I tend to do this kinda stuff a lot for some reason...
`Can be used as the null object for term names`
I don't know if we should test the exact value of the sort key.
...3 lines up: `i.is_integer` will return anything, but above it will return a 1. It will return whatever it was instantiated with since there is no canoncialization of the assumption values.
deprecation?
sure, you can always write the particle based function later that is a thin wrapper to this one

you could imagine a method to the particle class

Particle.inertia(point, frame)

I can also imagine a function that calculates the inertia of a group of particles and/or rigid bodies about a point that would potentially require this function.
Why not keep this and use it below...
shouldn't you then do something like:

inertia_of_particle(particle, <point or particle>, frame)

so that you can get the inertia of a particle with respect to another point or particle in a particular frame?
Because that is what the other matrices return. So until they all derive from Basic, I would like them all to give their properties in the same way. For those that are hashed, creating an object with ints or Integers is going to affect hashing...but if mutable ends up not being hashed then this is a mute point. But...there are tests that fail if mutable's hash is removed because of the custom `__eq__`, I believe. (If you defined `__eq__` then you need a hash, as I recall.) So...for the time being I want to make sure that what is expected from others is the same here.
The 0.001 should be a constant ``eps`` that is passed as a keyword argument, that is:

```
def flat(x, y, z, eps=1e-3)
```
Here I believe the default string representation will do:
```
Diagram({IdentityMorphism(Object("A"), ""): EmptySet(), IdentityMorphism(Object("B"), ""): EmptySet(), IdentityMorphism(Object("C"), ""): EmptySet(), Morphism(Object("A
"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g") * Morphism(Object("A"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g"
): EmptySet()}, {}, {Object("B"), Object("C"), Object("A")})
```
It is longish, that's right, but a lot of string representations of stuff are longish in SymPy ;-)
you don't need these final two parameters.
AFAIK the conditional performs better than rescuing an exception, so :+1: for that!

But I think you've missed the `render :new` in the else block. And I don't think we need to use `flash[:error]` in this case because the model will already populated with its validations errors.

Also we can pass the flash message along with the redirect and use some rails magic to automatically calculates the route for us:

```ruby
if task.save
  redirect_to(@course, success: "Task created! \o/")
else
  render :new
end
```
Done. 
cebc116
This is in test-server.js -- there are a LOT of other console statements there as well that are used for output within the Terminal.
For consistency, can you move the imports to the top of the file?
würde das umbenennen in $serviceNestingLevel
Is there anything that is better by doing this vs just passing in strings directly?
debug=profile ок
For each node type, there is an expected number of replicants as determined by the rule and an actual number that exist in the cluster. Segments are dropped until the actual matches the expected. In the event segments drop from one tier to the next, segments are not dropped from one tier until they are loaded in their correct tier. This is done by checking the total expected with the total actual.
Why not use databases as namespaces instead of table prefixes? (Seems typical in other projects)

Gian

On Nov 29, 2012, at 11:46 AM, cheddar wrote:

> In common/src/main/java/com/metamx/druid/db/DbConnectorConfig.java:
> 
> > @@ -41,4 +41,8 @@
> >    @JsonProperty("segmentTable")
> >    @Config("druid.database.segmentTable")
> >    public abstract String getSegmentTable();
> > +
> > +  @JsonProperty("ruleTable")
> > +  @Config("druid.database.ruleTable")
> > +  public abstract String getRuleTable();
> 
> I wonder if we should switch the table naming to be similar to the zk path naming. I.e. we can just specify a prefix unless we need individual overrides, at which point we use the individual overrides?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
don't know if it matters, but we might do a client.logout here just to be accurate - search bots are never logged-in.
In this case your ```if``` is equivalent (and more easily readable), yep :)
@erikfrerejean & @p : I have univ exams now.. That is why I am unable to respond and I am extremely sorry. I will be back in a couple of weeks and address all the issues put forth. Sorry again. Hope you can understand.
Please do not abbreviate unnecessarily. NUM should be written out as NUMBER, and CNT should be COUNT. It's only a few extra characters but it improves readability.
Regardless, all properties and methods on new classes should have a doc block. Can be as simple as:
```php
/**
* Avatar driver name
* @var string
*/
```
Nothing major, but spacing here is inconsistent. Why does one EVENT call have an empty line on each side and the other doesn't? Anyway, as I said, nothing big.
I don't think this comparison will ever return false?
This will conflict with #1108. Cache drivers need constructor arguments that are specific to them.
And this just happens to work because the user to be created/destroyed is 'user' rather than something more specific like 'test-user' and matches the hardcoded 'user' in user creation code.
Space after `while`
This feels too permissive. How about copying style management permission?
It is based on 'PCRE_UTF_SUPPORT_EXPLAIN', anyway, "if you" should be "if your".
decât să îi trimitem blind la homepage, să nu știe ce i-a lovit, mai bine punem filtru mass=True în get_object_or_404, kwargs. Măcar o să știe că 404.
Yes totally. I had basically copy-pasted this from WAI which uses a different convention, but will change
I think throwing a user error that's more descriptive here is ok
For consistency?

```cpp
typedef container_type::iterator iterator;
typedef container_type::const_iterator const_iterator;
```
This is already in the "deprecation" notes.  It's been deprecated for a while, and is ripe for removal in my opinion.

I'm not sure how we'd put a warning on a property we always populate (seems noisy) unless we re-implement this as a property getter.  Not worth the effort in my opinion.
So its saying used the passed in title (or function),  but if there is
cached data, use that instead.  fetchData()  fetches data if needed
and puts it in  this.cached_title.

On Thu, Jul 26, 2012 at 10:25 AM, Eric D Helms
<reply@reply.github.com>
wrote:
>>              if (title && this.enabled) {
>>                  var $tip = this.tip();
>> -
>> -                $tip.find('.tipsy-inner')[this.options.html ? 'html' : 'text'](title);
>> +
>> +                if(this.cached_title){
>> +                    title = this.cached_title;
>> +                }
>> +                else {
>> +                    this.fetchData();
>> +                }
>> +
>> +                $tip.find('.tipsy-inner')[this.options.html ? 'html' : 'text'](title);
>
> Is the title here set by some value in fetchData?  Confused by setting title explicitly above, or calling the fetchData function if no cached_title without setting the title variable again.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Katello/tipsy/pull/1/files#r1244208
Same here, will it build for the mono build configurations?
MIssing `<value>` element
redundant with docalculatebmi. Should only have one of these methods, right? so remove this one if not in use
can remove.
I agree. Removing the duplicate file.
Again, yes, that makes sense.
This doesn't put *level_height* over *sigma*, as requested in #7.
That code was ok, too. The confusion was because when I was writing the original comment that described the calculation, I copy/pasted the comment for gx and edited it for gy -- and mistakenly left the piece that said "the first resulting dim goes last" when it actually goes second-to-last. The actual code correctly put it second-to-last, and the error was confined to the comment. 

You're right, though, that if the code had done what the comment said it was doing, it would have been the wrong shape. And I can report that verify_grad picks up on that and gives an error, because I encountered that quite a bit! But bottom line: it was a mistake only in the comment, not in the code (which I've since streamlined per your helpful suggestions, anyway)
Doesn't compare == 0 mean ONLY rename when the names are already equal ?!
Will this include the line number of the project file or DDL file in the error message?
This would be a nice setting for settings.py.
Unfortunately, I had added a few incorrect spacing in this file previously.

I'd rather have a different commit that fixes the whole thing in one go after this gets merged.
Wow, wrapping my head around this gigantic thread. I love having Django on github.

I assume these must be renderable programmatically because there will be a lot of them. Must they be renderable at docs-creation time? Alternatively, if we can stipulate that they are renderable in-browser, that lets us do some nice things there (but makes PDF generation more complicated, for example.

Off the top of my head, there are three good options for rendering the charts:

* **Graphviz**: simple, descriptive language for defining graph. Not much control over rendering output.
* [Nodebox](http://www.nodebox.net): A graphics library for python, analogous to Processing. [Nodebox-opengl](https://github.com/nodebox/nodebox-opengl) seems pip-installable, need to see that this works on all platforms. Pros: majorly nice control over rendering output. Cons: commensurate headache.
* [D3](http://d3js.org) or other web charting/graphics libraries: a nice way to do it in-browser. Cons: only in-browser.

Right now the commit isn't clear on much of a flow, it's just a list of steps in order. Perhaps an IRC discussion to help me get my bearings would be helpful,
nit: s/psuedonym/short form meaning
Yes, something like this.
Great explanation. I figured it was okay, seemed too obvious to be a bug.
bootstrap popover is not that handy
Wow.  (template getAs<double>()).
removed. done
Can't it be a singleton?
These files are generated by the Wix installer and thus not need to be on Git, ignoring here.
Moved everything that was in this class to the pluginloader.
I'm down with this, everything should ultimately be data driven with this monster class setting up everything properly itself and with virtually nothing needing be done in "implementation" subclasses.
At the time, that seemed just as much of a cop-out, but I suppose " 1 star(s)" does seem slightly more tolerable than "1 stars".
Perhaps this should be a comment `//` instead of a log? I doubt users care
Huh? I don't see that at all. Shouldn't it be

    if (binding instanceof SingletonBinding) throw new AssertionError();

If we're double-scoping, we have a big problem.
This seems weird. On the very next line after you call this method you call resetNextTlsMode
Maybe 'root' is better to use 'runs_as_root'  or similar - that makes the code better-readable.
`if ( !/optipng|jpegtran/.test( cmd ) ) {` ?
R的手册提名字和大概位置就可以了，不必列入文献，而且这里版本也老了
I am sorry to warn you (unofficially) that this use of the framework will fail in an upcoming release. If you need this functionality, please use the call fluid.model.mergeModel directly. It's a rather peculiar requirement though, especially since you are in an array that you made yourself, one wouldn't think you would need such a subtle merging algorithm. Couldn't you do this more simply?
I suggest:

    int result = m1.getName().compareTo(m2.getName());
    if (result != 0)
      return result;
    return m1.toString().compareTo(m2.toString());

...and delete `MethodSorter.compare()`

Also, I would modify the DEFAULT Comparator to delegate to NAME_ASCENDING if the hash code of the names are the same. That way, both Comparators are deterministic.
An array of ints would be simpler.
Well, that's what the Eclipse formatter did using the checked-in formatting
settings. I don't mind changing it though, explicit type arguments look
awkward in any case… ;-)
Another option that I like is to have

```
@Deprecated Timeout(int) {}
Timeout(long, TimeUnit)
static Timeout millis(long millis);
static Timeout seconds(long seconds);
static Timeout minutes(long minutes);
```

That way there's easy, non-confusable shortcuts for the most common options.
Document attributes better, these ones in particular.
Ah I thought @nciagra was just trying to point out that although `delete` and ` = null` are equivalent the second is latter, not that we should do both. I think we should just null out the values since it is faster and I would say only marginally less readable.
Mean to leave this `log` in?
Does this mean that we're setting up a screen that potentially never gets shown?
Yay! Good riddance flex-box!
Same here - this if statement can be removed.
Nit: use `{boolean=}` to indicate it's an optional param
Ditto: Appears to be unused... remove?
Too lazy to keep typing it out--please remove all dead code everywhere.
As discussed above, compilers are smart enough to do appropriate transforms of this type nowadays.  In fact, the (i<<1)|1 is a slight pessimization in my tests.

CF http://url.godbolt.org/shiftVsMultiply
Try to avoid hardcoding strings. You can use factory.getUniqueString()

If you need a variable lenth, you can extend getUniqueString to return a string of certain lenght.

No need to pass length for content... the method will solve this for you.
jquery extensions should return the result of this.each in case someone tries to do method chaining or assign the return value to an object
I am not sure if this is OK default. Because it is evaluated on module loading and not when field is used (which can be also multiple times).
maybe check number of returned documents, whether it's an array?
I dont think request should be passed here.

The request will be logged in the debug mode anyway. This method looks_for_errors in response and i dont think sending the request here is clean just because its used for logging/raising an exception.
This is meant to be testing that there is a new app.  Also, this step is pretty generically-named - if we're going to test via the UI then we need to be sure which page we're on, which means this step should probably navigate to the right page.  But actually I would generally prefer to do a more direct test, since it's so easy to do that with the app running in-process - just check using ActiveRecord that the new app exists.

If the step said that we could see the app on the page, that would definitely mean we should test through the UI, but it doesn't say that, so we can verify it however we like.
Note my updates above. Since `@page` already has a `@page.content` method (outputs raw file content body), the render method should be named `render_content`
Well, if this is the case, then I would either change the name of the variable or at least make the comment explicit to describe that it is not doing a true token count or index.
maybe we should delete this part ?
Kaksi pakotettua rivinvaihtoa tarkoittaa yleensä kappaleen vaihtoa ja ```<p></p>``` on kappale.
Can you make this (and the one in line 48) tank/test instead of share/test1. Or, change the commands above. Either way, the examples should match. :)
This is using a write access URL, please use a read-only version. Also, please add a 0.1 tag to your repo and reference that in the source.
This script should be added to bin directory as part of packaging. Please do the change in databus-dist/src/main/assembly/dist.xml
should 'your.domain' be filled in with the actual domain using the fqdn variable?
This should probably be pushed down to the model IMO, but it could wait till the status is also shown on the system group page.
I'm not a huge fan of writing class methods this way (much prefer the style of  'def  self.method_name'.  Any reason you chose this method over that?
Yea, I am getting the same error. You're right that it's probably not a good idea to put test gems into development and webmock illustrates this point. 

I think rspec-rails and parallel_tests are the only gems that need to be in both dev and test (that I know of).
Given the new terminology, should this be `Number of attached subscriptions exceeded quantity for %s`
`@return void` is not needed. You can remove it.
indeed, using the Monolog channel looks a good fit here
no, because its main purpose is to return whether is a badHostname or not. The return message is to not use  another different method for that.
Are the `json_bigz_const_t` and `json_bigr_const_t` types really needed? I find them quite ugly. Would it work if the definition was just `typedef JSON_BIGZ_TYPE json_bigz_t`, so `json_bigz_t` (and `json_bigr_t`) weren't pointers? This way, the Jansson code would use `json_bigz_t *` and `const json_bigz_t *`.
Isn't this kind of capitalisation against our style guide?
This will throw if frames[origin] is null
I understand what you're doing here, and it's pretty cool. But I think it's almost a little *too* magical, even for me.  I would sort of prefer to have optional dependencies imported at the top of the module within try/except statements.  In these cases the `except` block can just `pass` and this decorator can worry about just whether or not the optional dependency has been imported (perhaps by checking `sys.modules`).

This could also be extended, hypothetically, to support version predicates and use `pkg_resources` to check that the correct versions of each dependency are imported.  But that's an exercise for a later time I think.
This is not really checked below, or did I miss something?
I thought -fstack_protector-all makes the ssp-buffer-size param irrelevant; am I wrong?
Nit: Small indentation glitch.
Ok, can you add a comment to explain that?
`sizeof(struct kevent) == 12`?
Are you sure moving the call to `_initConstants()` after call to `parseParams()` won't cause any problems? (I haven't looked into those func. codes)
it is only destructive by choice (default false). And it avoids the second call to the method. I just followed the way Js::buffer() works. It also has a clearing param. For some other view blocks it might also make sense to be able to clear after reading.
return $readyFds > 0 would suffice here
Yeah. I liked the previous version because its more specific, but this one was shorter for the guard clause.
Instead of `withBranches = (v.ToLower() == "with-branches")`, do `withBranches = v != null`. Add line breaks.
I think unbuffered mode was something we needed when we were still using twisted, so maybe that's not applicable any more. We really need more Windows testers for things like this. Let's cook up something we think should work, then ping jstenar to test it.
Yes, thanks. I must have missed this one.
use of path style here is the "odd man out"  I can see why you'd prefer doing this vs 4 string params, but it would cause folks to so some string fanciness and also wonder if they should delimit or prefix and delimit with slashes.  I think it would be best to use either a type that contains the container and object name, or 4 string args..  As S3AsyncClient has the 4 args approach, and no-one has complained, probably this is the better.
agreed, seem pointless but didn't immediately occur to me how to do it better.
Still a bit rough but I think the idea is sound. I expanded on the idea of "onitem" to "onevent" "ontimezone" etc... so we can add better first class support for top level sub-components.
Instead of typecasting a `Closure`, you could just check with `is_callable`. It's more flexible that way.
To answer this myself: __Yet__ it's of no importance, as we're here in the clone-code which still silently assumes that the initial remote will be called `origin`...
What is this alway with the "always"? Don't you like singulars?
we use jasmie rather than the old test suite?

Do you mind updating the jas mine tests

Will merge the changes soon
i think you need to do user.username in this case? or at least that's what the desktop template is doing.
Should this be assert os.path.isdir instead?
Should it be removed from the repo as well then?
All previous interfaces have been "Name*Interface*".  I think we should keep this standard convention.
Good idea to also extend it from Elastica_Param
Line too long?
@stof I'm really clueless about this.

What I'm trying to do is something like this:
- declare the cache drivers in the container
- declare a service builder, like what Cache.php does right now, which should take the given configuration for a driver and create it;
- make sure I don't get this twice in a row;
- keep the same functionality for people not using the DI component.

While I can split the Cache.php file into a CachePool.php, to act as a registry, and CacheFactory to create the instantiated drivers, I'm not really sure how this can be done for the DI component, more accurate, I'm not sure how to register all the needed cache services so that the you can call them from via the DI component. Did I skipped something from the documentation?

Thanks!
I agree, forgot them. Fixed now, thanks.
The difference between addRoute and addAllRoutes is not really clear. They seem to overlap.
Pardon, but what fundamental contribution did you make to the file?
Can you put everything in one line to be consistent with symfony CS?
`$constraints = (array) $constraints;` ?
sould => should
"Unrecognized options "strict_parameters" under "framework.router" " see commit #5149 and commit 03bbaaf325e1d29057d341551a8a3d3989ab330e
Can you make another PR just for the added tests? Thanks.
You are quite right, this has been fixed.
``exists`` instead of ``count``?
should have "(in development)" added.
Ah, so maybe we can't just use compute_etag since this needs to be a classmethod.  Maybe call it compute_etag_for_path to echo the existing RequestHandler method but emphasize that it's based on something other than "self".
As noted in the main description, the selector `"li.search-choice"` has no effect here. In fact, the [`siblings()` method](http://api.prototypejs.org/dom/Element/prototype/siblings/) accepts no arguments at all.
The changes to this class exposes a bug/problem which I believe has been there all along. This JcaAttributeWriteHandler is used as a write attribute handler for the attributes of 2 different resource definitions:

1) The ArchiveValidationParameters
2) The BeanValidationParameters

Each of these resource definitions have a same named "enabled" attribute (and possibly some more in future). When a write-attribute operation is issued, the implementation of this JcaAttributeWriteHandler just takes the attribute name into account to update the model. Now if the write attribute operation was issued on the "enabled" attribute then it just ends up updating a (potentially) wrong model. The changes in this commit would mean that it always updated the ArchiveValidationParameters resource definition while previously it used to update both ArchiveValidationParameters as well as BeanValidationParameters (the previous behaviour was wrong too).

This was caught by the failures in org.jboss.as.test.integration.jca.beanvalidation.DisabledValidationTestCase.
I am just setting up a new LDAP environment myself but from what I see here the {0} is the place holder for the users username, the  usernameAttribute I believe is from the config.
I see, the jmx test intended to have this attribute set to make r/w attributes writable, which perhaps isn't needed for this test currently. I think the real fix, since this is a storage=runtime attribute would be something like: https://github.com/kabir/jboss-as/commit/1897328eb98b38ebf3218f624bee65d6b315cecc, although I've not tried that with your branch
There can be issues here. The absence of a module name will not necessarily mean that the file path has been specified since the model management isn't doing any such validation while creating this service.
let's call it `DeleteDialog`
Do an early return. I don't think you want to consume all the keys with the evt.preventDefault()/evt.stopPropagation code that lives after.
these lines are already defined above, please remove them
sendEvent -> fireScreenChangeEvent
this logic should be in the onHideStart function passing pageHideBySwipe
/* For other modules to acquire */
NIT: the icon does not appear, you should use `pack-end icon icon-dialog`
nit: indentation seems broken
@fabi1cazenave I don't see such a problem, is just a matter of chossing another way to recreate this structure w/o using `label` element.
This structure works quite well, and is full HTML5 compatible:


&lt;form&gt;
    &lt;fieldset&gt;
      &lt;legend&gt;Work&lt;/legend&gt;
      &lt;section&gt;
        &lt;p&gt;
          &lt;input type="tel" placeholder="Email" required&gt;
          &lt;button type="reset"&gt;Clear&lt;/button&gt;
        &lt;/p&gt;
        &lt;p&gt;
          &lt;input type="tel" placeholder="Email" required&gt;
          &lt;button type="reset"&gt;Clear&lt;/button&gt;
        &lt;/p&gt;
      &lt;/section&gt;
    &lt;/fieldset&gt;
  &lt;/form&gt;
Question: if we have the bookmark section defined in saveBookmark.html, Should we remain this markup in index.html? thanks friend
It is not a problem, if a check is still running, there is no new
scheduling :)

On Sun, Oct 7, 2012 at 4:06 PM, xkilian <notifications@github.com> wrote:

> In shinken/objects/schedulingitem.py:
>
> >              # is service_max_check_spread or host_max_check_spread in config
> >              interval = min(interval, cls.max_check_spread * cls.interval_length)
> > -            r = interval * (random.random() - 0.5)
> > -            time_add = interval / 2 + r
> > +            time_add = interval * random.uniform(0.2,0.85)
>
> I can make it 0.0 to 0.85. Which is better than the old code which only
> did 0.5 to 1.0.
>
> It also means that for:
> 5 minute check_interval check, it will have 45 seconds for a check to to
> execute.
> 4 minute check_interval, it will have 36 seconds for a check to execute.
> 3 minute check_interval, it will have 27 seconds for a check to execute
> 2 minute check_interval, it will have 27 seconds for a check to execute
> 1 minute check_interval, it will have 15 seconds for a check to execute
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/naparuba/shinken/pull/629/files#r1780658>.
>
>
magic colors :-)
Seems unnecessary since LogStreamManager is single use
We really should move away from all those first class setting and switch to a prop bag. But we can do that separately.
This should be named $arrTemp to match the hungarian notation scheme.
Please add `type: "DELETE"`. `DELETE` is the way to go for destroying the session.
Went with startDate + Days(period) - Millisecs(1), as RichDate.upperBound only takes a string (I've been meaning to write a version that takes a RichDate, but not right now).
we dont have to split though, we can just use `\w+` as a match instead of a split
is this commented code necessary anymore?
Not sure what I can add to be informative
Add a `Returns` section.
Fixed. I replaced all of the hard-coded URLs with the url template tag.
Why add "isWorkingDir" parameter?
already inherited from droolsjbpm-parent pom
Why it's before product's type?
I see… so yes, will switch to a virtual host then.
You're not assigning something. Just use `||`.
Should `Row` be a `Control`?
Why make @kv_keys an instance variable?
Why update on `change:age`? That value is not used by any function on the toggle.
license block + "use strict"
we cannot authenticate this session until we've checked that the user is either using the same browser that they used to start the verification process, or that they know the password.

One implementation tactic would be to have this WSAPI return a 401 if there's nothing on the session indicating that they're on the same browser, and then have the frontend prompt for the password and retry the wsapi.
I wasn't quite sure what to do. Other places in the code we have a failure callback handy, but not here.

Any suggestions?
hehe. I can't remember the mathematical fancy name but multiplication doesn't need brackets :)
PEP8: Two blank new lines after imports.
Why @?
I just pushed a first pass at this:

https://github.com/rspec/rspec-mocks/commit/211743db22a44a5a42943ff2d0dff7ae8733260c

Originally, I was going to have the method return the raw original value, but I realized there are a couple edge cases here:

* What does nil mean?  That the constant was originally set to nil? Or that the constant was previously undefined?
* What should be returned for a constant that hasn't been stubbed?

I realized it's a lot easier to communicate each of the possibilities by returning an struct-like object.

I still need to document it, but I wanted feedback first.  Thoughts?
I agree. But with this change I need also little modify one spec. Check out new version :)
Change the second `%s` to `%r` and then remove the `str(...)` on the next line.
For get the program.opts value.
Geez .. forgot to remove it ;)
Thanks :)
This is not needed unless that you need customization.
Oh I see, I remember, that was an internal method that BubbleWrap depended upon. We sent a pull request and I believe it has been merged, so there is probably no need to add this method back.
){
it doesn't give the same result here reading the function. Intention here is to update the doc instance with the _id and new _rev. i'm surprised that the test pass.
Warum verwenden Sie FQCN für Parameter, aber nicht die return-Anweisung?
i left it in here for consistency .. that being said .. this code is missing in CouchDB /cc @beberlei
I don't understand the first `?` inside the first paren group. Does `?)` have a semantic meaning I've missed here? Otherwise, if `?` is 1-or-0-of-the-last-thing, and the last thing is `.*`, then that's the same as `.*`, isn't it?

I'm still drug-addled, beware.
```$options = ConfigDumperInterface::OPT_DEFAULT```
Maybe mention Solr  in this
The tarball is generated at build time, see output:

```
creating /home/kehoste/.local/easybuild/sources/p/python-meep/python-meep/dist
Creating tar archive
```

Produces:

```
$ ls dist/
python-meep-1.4.linux-x86_64.tar.gz
```

I guess I could be a little bit more strict here... Will fix.
single empty line before functions def in a class
No you don't. It's bundled in the quickstart directory.
Just use double quotes if you need a quote inside; that's preferred over using `\` to escape things.
I put a possibility to use a config option here, but it doesn't seem to have any effect, so I may have misunderstood something with the Activity API.
I'd prefer the dependency specified as `['~> 2.0', '>= 2.0.3']` instead of tightening the dependency.

Could you make this change to your pull request?
Ah no, I see now, isinstance works the same way and takes either a class or a tuple of classes.
cant we simply do class.to_s  or something like that?
Ditto Comment Above. These should be removed.
Check with the validator that you don't need to have the `action="."` in there.
We should also start using the `requests` library, it's already installed on production server.
Should refactor this and the [duplicate code](https://github.com/RiotGames/berkshelf/pull/68/files#L6R93) into a support function in spec/support
What frame are the apertures defined in? Pixel coordinates or world coordinates? There are telescope/detector combinations that this makes a significant difference. Does specifying pixels or world coordinates select the frame the aperture is computed for?
MarkDown syntax error.
Please, use ``assertEquals`` instead of ``assertSame``. I only use it for asserting that two objects have same references.
that is some fine MD text
Thanks for reporting,
this works, and is already tested.
Okay, so it looks like this is on Paypal's side. I doubt this bug will be fixed.
It doesn't use native so you could remove this :)