labofsahil/github-event-dataset-2012 huggingface dataset:
`fs.watchFile` [doesn't work on Windows right now](https://github.com/joyent/node/issues/1358) (it throws when you call it), and it seems [a bit unclear whether it will eventually work](https://github.com/joyent/node/pull/2049) or if we're supposed to use `fs.watch` instead.
No problem. I'll fix & test tonight.

On Saturday, January 28, 2012, NuclearW <
reply@reply.github.com>
wrote:
> @gmcferrin Give it a test run please, before we pull it.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/gmcferrin/mcMMO/commit/42ddf648b1151231ebfd96698f05dc55ccfea015#commitcomment-911410
>
our delays are for sure some months old, getting closer to that 1 year. It wasnt that bad back then, as we didnt have a day uptime :D But now, already for long we barely have any crashes, we just have to reboot cause delay. Also we are based on udb, I just take some parts of ytdb. I dont think the db part can cause the delay.
yeah, definitely needs to be alphabetical.
Awesome! With an alias, this works like a charm: `switch foobar`
typo: Methoen
Dann würd' ich nicht von Methoden reden, wenn's genau eine ist...
Yep, It checks forever if a lx_core_lock is set with sleep each 3 seconds (8 times).  So the minimum wait is 3 to 24 seconds if lx_core_lock works properly. But if it fails, it is indeed a endless loop.
Way to the point the finger ;)
This should probably be more specific: "None of our ES tests should use fixtures..."
or you could even make it a jQuery function
Yay for some action!
Since you do not use any of the closure properties of nested function definitions, would it not be better to have these functions (_fill_bit_pos_vector, _compute_ghd_aux and _init_tableau) defined externally (and not redefined each time GHD is run)?

It would also make the main function line easier to read, as it would begin with the first operation.
isn't it supposed to be http://guardian.mossycobble.com:8080/nexus/content/groups/public/ ?
Thanks for digging in, looks great!
That should be `Wordless::register_helper("ThemeHelper");`
Ha! As always, I should drink my Coffee **first**.
There's another minor fix on top of this in #64
Yes, I hope it will merged any time soon.
TGI gone. I wondered what the purpose of this was! +1
Odseparowałem menu, bo będzie takie samo dla wielu podstron, więc po co sie powtarzać w widoku ;P
I should have mentioned. The above class has a composite primary key.
So the owner is nullable, but the field of the composite primary key is NOT NULL.
All sleeps after repository.removeResource were there to assure changes have been flushed in the database (we didn't know that was the cause, but found sleeping avoided the error). Good you noticed and removed them now they are of no use at all.
why did you remove the if? i mean, is there a particular reason (note: i didn't re-read the entire module)
No! I'm surprised to see it there to be honest. Shall I remove it and send a new pull-request?
здесь self необязателен (self обязателен только если мы устанавливаем значение, то есть для setter-ов)
added cmbootanimation
This is the same log that's been displayed for https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/XWiki.java#L1784
askForCardChosen is good enough, please refer to the first two lines of Room::askForCardChosen
Yep, I copied the code from google, so I think it works
For the purposes of maintainability, I think it makes more sense to have the "fall back" if x12gs03 isn't set here rather than in the x12_gen_class.
Doesn't it make sense to return $claim->x12gsreceiverid() here instead of null?
Would make a lot of sense IMHO. (and move the comment here)
+//In some clearing houses ISA08 and GS03 are different

+//Example: http://www.acs-gcro.com/downloads/DOL/DOL_CG_X12N_5010_837_v1_02.pdf - Page 18

This way the appropriate value to use is sorted out "early" and anything which uses the claim.class has this fixed, and not just gen_x12
Denis-

Please ask Carsten.  He upgraded it via commit - https://github.com/LimeSurvey/LimeSurvey/commit/0641eef16ac8445726be37526cfe927ed03633ac

This commit is just an automatic merge of  that one.
since default areaIds set is empty, why CONFIG_BOOL_RESET_DUEL_AREA_ENABLED==true by default?
Just a small observation, you can avoid complex weirdness in classes by just `dup`ing a class. Here's an example from dirty tracking:

```ruby
card_class = Card.dup
card_class.class_eval do
  mass_assign_any_attribute true
end
@card = card_class.new(:first_name => 'Sam')
```
proc nepouzijes `DIRECTORY_SEPARATOR` ?
I see. sockjs is a virtual (namespaced) package which contains sockjs.tornado and might contain other packages (sockjs.cplex for example). I added 'sockjs' package back and it now works as expected.
If I add that git config then I can keep using "git add ."? Please don't make me have to "git add -p" everything I ever write at EY.
Hi,

I added my email now. :D

I'm definitely willing to get this merged back. I am in the middle of a project though that requires my focus, and all commits I make are because I need them right now. I have updated some documentation, but haven't committed that yet. That, together with the changes facebook recently (http://developers.facebook.com/blog/post/624/) made (we can't rely on the fbsr_ cookie any more) makes that I have some more work to do on this project. Hopefully I'll have this fully working in production somewhere in February. I'll cleanup a bit then and let you know! 

If you want to merge back stuff you like already, please feel free to do so! If you have specific questions, let me know.

Tino
I guess having asserts in a test case could help :)
What's strange is that the JSON string returned by the new method is exactly the same than the string created by applying .to_json to the output of the previous method used in client.rb

http://cl.ly/0I0w3M2M312E3L1D041H
From the coding conventions:
"No 80 column limit or midstatement newlines."
The default behaviour is explicit in the constructor, setting ```defaultCopies``` to 2. This code means that, only when that function is used to get a file descriptor, the variable will be unconditionally set to 1, for whatever reason.

I don't understand the whole meaning of ```defaultCopies```, but according to the facts I have, and the fact that this is being changed elsewhere (when dealing with clusters and file parts), I believe this ok.

Facts:

* ```defaultCopies``` is the number of replicates a file will have. 1 means it's not possible to replicate (thus, we have that failure), 2 or more means we can replicate (or is already replicated) and it'll find N nodes to replicate to (or not replicate at all). 0 is not possible, and only happens if the file didn't have that value (but had ```redundancy```) in the first place.
* Some places set the value to 1 on external files, which makes sense, since you can't replicate an external file, nor you have copies of it in your own local DFS.
* Other places set ```@redundancy``` to 1 if ```defaultCopies``` is either 0 (previously non-existent) or more than 1, meaning that being redundant is the default behaviour.
* Redundancy is determined by the number of copies, if I force ```defaultCopies``` as 1, on the next round, a file that had redundancy, will not have it any more (it's still persisted as ```@redundancy```).

I'm not sure this is the best place, but it seems like a good change. If other parts of the system fail because of this, and I can't see why they would after looking at all the places that use this variable, we should fix it in the right place. To me, it looks as if that line was added in a rush to fix another bug, just as other lines I showed you earlier.

However, this does't fix the underlying issue of IPs being zero. But that issue will be revisited when we implement URI naming system, so I'm not too concerned about it now...
>If he decides to fire a SPAM event, the Bayes should react as if the user is clicking on the SPAM button in the comments list: Learn, then moderate or move to recicler.

That's exactly my point :) the normal workflow of the plugin never moves a comment to moderation after the plugin is saved. block() and moderate() are only called in the frontend_save_comment-event and control the flow of the comment the user just submitted. When a user clicks on spam in the commentlist, the comment is always 
1. learned as spam
2. optional: copied to the recycler
3. deleted
Tests updated on newest commit to this branch. Since the recipe rules seem to apply pretty broadly, I reverted the change I made to the linter in that regard.
but but... this is done randomly while at that value of drunkness, not instantly
Shauren is right, revert!
ah yeah true, that's probably a good methodology to follow (keeping unused code commented.)

hm.. I'm not sure if my_goals would be appropriate. the methods aren't exclusive to "your" goals, they're used for all the goals on the site. I think the best option would probably be to just change the default method backbone fetches, and just change it to fetch or whatnot. thoughts?
it's still in the commit history...
I apparently cannot type... "remote" == "remove"
This comment can be removed any time - no purpose for it at all
After changing it into assertThat based on points, you could look at some of the collection matcher that I have written. They should help with this.
Does your code really use ActiveMQ? I don't think so.
Pankaj: Thanks for the suggestion. Did not think through that too much since the service that uses the pdftohtml executable that runs locally on my ubuntu VM. But point taken :)
Again, this is the only place where MESSAGES are used. If MESSAGES used symbols as hash keys then the line could be translated to:
@message = [ MESSAGES [params['DE039'].to_sym].presence.......
Performance consequences would probably be minimal.
Could someone else have a scan through this.   The code is written in an odd confusing way (eg. says check if overlaps and then code is (if !IsOverlap)

I think Arion is right and that is all that is needed (subject to the pre/post periods being taken into account ???)

Is this sufficient for the duplicate detection logic to work???

Just need a clean set of eyes to give this the once over
swizle i think your right one of the main solar eclipse spells calls for that aura...332
Nope, the bridge was lacking some things when I wrote the consumer
tests are failing because you forgot to commit these fixture classes, etc Author and Paper are not found
avantage ? boost::ptr_list est un shared_ptr directement ? si c'est le cas bien sur en effet !
你确定是 点击项目(TARGETS) 图标，而不是点击项目(PROJECT) 图标？在我的 xcode 里，是点击 PROJECT 图标。
If you want this to work there are other hardcoded references to 'home' in the same file that will need amending.
I tested this one and it works as expected. Whenever the location changes the map is updated and centered on the location. 
Now we have to include the latitude/longitude in every ScanResult and add it to the stuff being uploaded to the form and it's done. :)
I wasn't able to reproduce the issue we have in production. So I created a
stress test script which modifies a compute object; clearly I noticed
conflicts on computes. I have logging enabled and as soon as I reproduce a
complete multi host environment.
Der Footer hat eine feste Höhe, das ist nicht so gut. Die Höhe des Footers soll sich an der Browserfensertgröße anpassen.
Den Footer in den html-Bereich einbinden, dann mit background-position nach unten verschieben
html
{
    background: url(Kneipenquartett/images/footer_bg.jpg) repeat-x;
    background-position: 0px 750px;
}
in der Footer Klasse die Höhenangabe raus
.footer
{
  
  margin: 0;
    padding: 0;
    margin: auto;
    margin-top: 0px;
}
It absolutely better, but I copied that file as is from http://snippets.dzone.com/posts/show/4571
I'll change it. 10x
I just fine tune it.
I can add other governors without problem
Please only write debug-output when the debug flag is set. 

$c->log->debug will always output the string to STDERR.
So this is the basic thing - the css is _really_ rough, so I'll be tweaking it here and there - but all the html is there ready to go.
I'm using rails 3.2.1. As far as I know, if the file does not have a
sass or scss extension, it will not be processed as sass. When I first
upgraded to this version of active scaffold, my scaffolds had no
colors at all. This was in development and production mode. Renaming
the file with an scss extension fixed the problem. Also, if someone is
not using sass, using @import defeats the point of the asset pipeline.

I do not have a custom active scaffold colors file yet, as I have
always used an active scaffold overrides css that is in my asset
manifest after active scaffold. Maybe that is the difference?

Are you precompilng assets in production mode?

I will test again, but according to asset pipeline documentation a
sass file should have a sass or scss extension.

On Feb 3, 2012, at 11:52 AM, Sergio Cambra
<reply@reply.github.com>
wrote:

> I have tested, both with *= require active_scaffold and @import active_scaffold and it's working
>
> - with sass, including active_scaffold with "*= require", @import line is in generated file
> - with sass, including active_scaffold with "@import", active_scaffold_colors is included in generated file
>
> I'm using rails 3.2.1, are you using rails 3.1?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/activescaffold/active_scaffold/commit/686f43c1fbd76224be9b32505ecde6d553a0d26d#commitcomment-930986
yeah .. that sounds like the way to go to make it generic ..
you should inject "doctrine_phpcr" instead along with the name of the document manager
Sorry to have been sprinkling these all over the client code. Its a sure sign that we're overdue for some organization. On the other hand, I'm finally building small federations and coming to understand what it is like to move around in them. Then I look at the console log to see what the code did on my behalf.
it'll be nice to dynamically generate these files so we don't have copy and paste between them all, but that's a task for another day.
Still http://test.thebkkingdom.com:8080/job/TheSocialNetwork/lastFailedBuild/console
thanks,  I need to think about maven2 and 3
you missed the ifeq for OCOM_HARDWARE here
te has pasado tio
@wenqing @endJunction @TomFischer I'll remove the constructor from the interface.
I would use the Arr class to remove it, it might not be there and therefor result in a php error/notice.
jsLint found an unused var.
Seems to work in my testing. I don't want to go through Y.DOM here for performance reasons, but I guess I could if it's really a problem.
Original for deleteIcon while deleteIcon feature had removed
True. The problem with langs is or at least the problem I experienced was that, I wanted to be able to only detect, so that I could throw back the error. Lang only has support for parsing correct input, or taking the default. 

Taking the default won't work in this case, because there's a possibility that the value provided could be the same thing as the default and I didn't want to, on behalf of the client, assume an evaluation order which is something they ought to be able to configure responsibly.
This property doesn't make sense here I think, since there is no data?
that is sad
Great thanks. Also cmp the poms.
it will be, once we update to scala 2.10’s next milestone
I think that is what we need to do for supported versions anyway.
Should be documented as non-threadsafe since it doesn't use CAS/locks to ensure that there are no lost updates
I thought I had added a new comment here.  The ones below are necessary class methods.  If you don't like the name, we can fix that.  They are used in more than one place, so DRY suggests a method somewhere, and this seems the right place for it.
I'd try hard to avoid it if we didn't already have custom doctest parsers, but in our case, the overhead is tiny.  The ellipsis looks ugly to me because it's not clear to the reader what's being ellipsized.
Fair enough. I might pass the message into the method and return the return code from it, but I might just as likely do what you did as well.
In builtins/test/unit/base_builtins.py, there's a PulpClientTests base class that can be used instead of unittest.TestCase. That sets up most of the necessary pieces for testing these, including a self.context that's an actual context object with the appropriate test contents within. For example, it includes a prompt that's configured to write to a recorder, so it will capture written tags and written content.

Ping me if you want more details, I'm glossing over it a bit in this comment.
Btw, as your code relies on the Symfony 2.1 Process component, you could use the ProcessBuilder class to make this easier
you moved the file into emrio, so lets update this.
I don't remember if I said this last time, but these should just be YAML or JSON files.
`Box` is only one direction, I think. When would you ever want a `Prompt-TagTop` to be a subtype of any `(Prompt-Tag s t)`?
i'd say styles that are same for all themes shouldn't be added to themes
It's unfortunate that we have to return an object instead of an image.  Does it make sense to do something like this instead?  (Also: do you have any references so that I can learn about the width/height bug?)

```js
img.setAttribute('width', img.width); 
img.setAttribute('height', img.height);
```
Hm... I guess in some browsers that would override stylesheet / css rules?

-- J
So this is Breadth-First-Search, pre-order traversal?
I don't see why this needs to be a two-liner
Why is this required here? It seems like it should be done in stats.js if even.
The `__block` specifier no longer prevents retain cycles under ARC. This should be changed to `__weak`.
Yeah, I'm not convinced that's The Right Way to handle the potential for category-smashing. I'd prefer we prefix those dudes.
Yeah, that can be organized better, you're right.
The world is such a strange place, try again
Not needed, either.
Proč ne raději

    new Nette\Callback($this, 'templatePrepareFilters');

?
I haven't tested it.  I think you are correct.  My understanding (from looking at the code) is that the functions just need to be added to numpy_api.py, correct?

If I do that, I should probably just remove the "#if NPY_TRACE_DATA_MALLOC" verbiage, rather than try to discover that in the code_generators.
Thanks for updating the docs and adding this. I believe this could be a one-liner (ie like the others below), and `ActiveRecord::SchemaDumper` should be inside backticks for proper markdown formatting.
Thanks -- I did not know that trick (and it's even in Ruby 1.8.7).

I'll make this change after waiting briefly to see if there's other refactoring advice that I should include.
ensure the +debugger+ gem is listed
Currently, the test for `destroy` in `persistence_test` doesn't test failures. This is only done in `callback_test`. I added the exact same corresponding tests for `destroy!`.

Moreover, there is no accessible model that cancels destruction in the callbacks; these are inlined in `callback_test`.

If you want, I can:
1) Move some model out of `callback_test`
2) Move some `destroy` testing out of `callback_test` into `persistence_test`
3) Move the corresponding `destroy!` testing.

These behavior seem pretty intertwined to me (as only callbacks can make `destroy` return false, unless I'm mistaken), so I think that as long as the test is in either file we're ok.

Thanks
The fields are available on the message object (object here implies instance of the class). For example:

```javascript
var point = new Point();
console.log(point);
// ^ Prints out
// {  x: null
// ,  y: null
// ,  z: null
// }
```
relative imports? ;)
Wow, you got Option, Seq, Tuple2, Either, and Any in the same type.  One might think one should receive a prize for that.  (However, one should not: if there is a prize, it is for not doing that.)
I think it's much clearer with this docstring.  I would nevertheless suggest removing the sections "inputs" and "outputs", and instead describing `s` like this: "a dictionary to which the function will add the resulting mapping of variables to subtrees".  You may also consider endowing `s` with the default value of an empty dict.  Since, apparently, you always copy dicts before adding values to them, you shouldn't have any issues with the default value `{}`.

I understand that I'm probably exhausting your patience by paying this much attention to documentation; that's because I have the strong opinion that, in modern high-level programming, comments are as important as code (yeah, [literate programming](http://en.wikipedia.org/wiki/Literate_programming)).
I think it comes from PEP 8 actually.
Just to be clear, I did try to replace it internally just by overriding `numbered_symbols`. The simplification of constants routines got really confused. Probably I did it incorrectly...
Maybe you were thinking of the gcd behavior that seeks the lcm of the denominator:

```

>>> gcd(S(1)/4,S(1)/3)
1/12
>>> gcd(S(7)/4,S(35)/3)
7/12

```

That behavior hasn't changed.
add space around `+` in `o+1`
See http://stackoverflow.com/questions/750172/how-do-i-change-the-author-of-a-commit-in-git/1320317#1320317
-1 to this change.
Yes. It is the main place which care me with this PR it is dependence on prev operations.
Frankly I **did not want** to obtain the same objects indeed.

But they are created the same and saved to the cache only in this one place. And it confuses all the cards
So instead of `(3*z, e, 2*x, 2*y, d, z**2, f)` I obtain `(3*z, e, 2*y, 2*x, d, z**2, f)`
I don't understand this comment. Why am I not allowed to provide superfluous morphisms?
Ну а чего непонятного то? ) Вроде бы так и написано, что если нужны закрытые репозитории, то надо платить, а открытые проекты можно выкладывать в любых количествах бесплатно. 
Может так:
GitHub — это коммерческая компания, которая взимает плату с учётных записей, использующих приватные репозитории, но любой может хоть сейчас получить бесплатную учётную запись и разместить сколько ему угодно открытых проектов. Мы быстро рассмотрим, как это делается.
?
Any suggestions on how to do this without hard coding a version number in that needs to be updated every time we update the framework?
Yeah, that's OK.
Yeah, i meant the same thing. :-)
Dont forget if(function_exists('mb_strlen'). See any other components that use any third party php extensions.
I think we should have some sanity validation here for the keys and values. Otherwise things will silently fail when least expected.
Maybe I'm crazy, but is there any reason to not just make `i` a uint32_t and avoid the static_cast stuff?
...did not log anything.
These 2 assertions seem a little overkill? You're basically testing the `where` method on an ActiveRecord model?
Genau... Weil die Group Navigation die Gruppe aus dem Kontext nimmt... und diese natürlich nicht mehr gesetzt war...
ok, then :)
That will raise a exception. 
```jinja
{% set user = user|merge({name: "Igor"}) %} 
```
my bad, will change.
I can't think of any downside that is worse than the -1 special value.  Documentation of it would just need to explain any value < 0 means unlimited.. I suppose we could actually build the REST interface to have both a set limit to an integer value and set it to unlimited via different url or parameter; that way, users/admins don't need to worry about the implementation details
Example Queries for Profiler Data
we shouldn't document behavior for 1.8 and before specifically.
These tests should be wrapped in a context to group them and describe what area they're testing, and it looks like `filterCompleter(@completer, ["story"]) # Force installation of @onVisitRemovedListener.` should be in a setup block
giving

period at the end
Yeah, should probably just cast to boolean.
`$user_row['user_has_posts'] = (bool) $db->sql_fetchfield('post_id');`
@var type missing.
Please use the phpBB function `unique_id()` since that is what it is there for.
It is wrong.

On 14 Oct 2012, at 16:55, Nathan Guse <notifications@github.com> wrote:

> In phpBB/language/en/email/notifications/quote.txt:
> 
> > @@ -0,0 +1,20 @@
> > +Subject: Topic reply notification - "{TOPIC_TITLE}"
> > +
> > +Hello {USERNAME},
> > +
> > +You are receiving this notification because "{AUTHOR_NAME}" quoted you in the topic, "{TOPIC_TITLE}" at "{SITENAME}". You can use the following link to view the reply made.
> This is the way it is in topic_notify.txt, what phpBB originally used. I do not know if this is correct or not because of British English can be rather odd.
> 
> —
> Reply to this email directly or view it on GitHub.
>
We need to have this be the burnt in CA cert OR the test cert if '-i' is passed in.
fixed in 0484674
Asta pentru că nu a fost nicio challenge, corect?
I think I'd put a warn: here so people are aware we're throwing away the base object and just copying their input.
Lines 149 - 207 have lots of repeated logic. Suggest abstracting the shared code to a helper method and pass the button type (e.g. ConfirmReferToDocButton), instead of repeating all the lines. 

For example, you could create a helper method showButton: function (<object> button, <bool> isVisible ) which shows or hides appropriately.
should ivy still be listed here?
Cool. Would it be possible to accept a target extent?

Additionally, I think it is worth mentioning that this function is only really useful for visualisation purposes. Because of the scheme, range and original values are preserved, but the mean & standard deviation will now be meaningless.
Should that Error be raised only if known_fail == True?
The warn_type flag is gone because it doesn't make sense under the new interpretation of how to handle integers and gradients.
I did it for op. It is easier as I set the infrastucture. I agree we should do it in the file too, but in the case of the block dim, I think they never changed up to now and this won't affect performance except that it could in the future allow us to remove an outter for loop in the gpu function and thus freeing register.

I don't have the time for this now and think there is a bigger payoff to spend time elsewhere before trying to change this one.
Perhaps we want to limit the precision to %.2d or %.1d even?
Break lines inside the parens.
I know this is not something you did directly, but the function name in the spec doesn't match the actual function name.
Ad 1. Most of builtin nodes refer to either settings.DEBUG or settings.TEMPLATE_DEBUG for raising exceptions, none use logging. I don't see a reason to treat this error differently than any other runtime error in templates.

Ad 2. Ok.
@lest @ndbroadbent `redirect_to` will not stop the destroy, it will only set status/location/body and return true.
https://github.com/rails/rails/blob/master/actionpack/lib/action_controller/metal/redirecting.rb#L65
@gibson042, updated.
Once per animation on startup - I'm not that concerned with it.
Can we perform DROP TABLE IF EXISTS & CREATE TABLE here?
Yes, a boost-style frontend to the database RNG is desirable.
nitpick: this should be __VCSCOMMAND_H
This file generates a list of files that need to be packed in the installer, and fills in the template accordingly. Could use some documentation in the future...
Is it hard to support this? It'd be nice to merge this in with this supported. (I know that Weibo doesn't support this, that's too bad)
Okay. Wish you guys were using tabs though, so everyone can use whatever tab width they want. I hate how that's the C# default.
This end span tag should line up with the one that begun it.
This reads weird, having the assignment in the if condition. Something like...

```ruby
@format = nil
matches = /.(\w+)$/.match(@name)
if matches 
  @format = matches[1]
end
```

or

```ruby
@format = nil

if /.(\w+)$/ =~ @name
  @format = $1
end
```

Would be preferable.
Oula, fait attention, tu as supprimé plein de choses dans cette fonction qu'il fallait garder (cf derniére MAJ de kratisto)
Is this indented 1 character too far or is that just an artefact of how its displayed in the diff somehow?
Is this comment relevant to the API documentation? Seems more like a implementation detail of reddit that a PRAW developer should (1) already know, or (2) not be concerned with.
might as well say immutable?
I don't think this case is as useful anymore. In the code above the motivation is to save a StringBuilder if we have no qualifier. In this code we're going to construct a string anyway so you might as well not have this code
don't bother with the constant, it's easier to read as just "<"
This needs the `@synopsis` treatment too, but I think you can just use `@synopsis <taxonomy> --<field>=<value>` and then just point to `register_taxonomy()` in the man page.
We could probably merge this convert(DatabaseDocument<?> and the convert(LocalDocument) to a simple convert(Document), no? Since Document extends the JsonDeserializer interface, this shouldn't be a big problem.

Seems easier to do that than to commit a bunch of Database-API changes and implement those all over :)
This logic is messy, and given it is being modified so heavily here, should be rewritten completely in order to supply a regular set of named (boiled) events reflecting these workflow points rather than a "rat's nest of callbacks". The "fire error on no data" logic is also standard and should be consolidated with the corresponding logic in datasource.js line 328 (currently also implemented in a private piece of logic)
Ah, looked at Yammer and Instagram. I'll switch it.
Soooooo muuuuuuch maaaaagic. I think readability would be greatly increased with by hiding these magic numbers and seemingly-arbitrarily-grouped tuples behind some symbolic names.
Please remove the commented out lines.
Ohh...sorry. I see where I messed up. Nevermind. This block looks good.
Move this ``triggerHandler`` inside ``sortWorkingSet``.
Yeah, you're right. At first it was interfering with unit tests, but as long as all of the update notification tests are run in a separate window, this check isn't needed. Removed.
@jason-sanjose Added a note in 00ec608 that explains the extent of the problem.
We could further clarify this by renaming `_topUsedColors()` to `_collateColors()` (since it doesn't actually scan the document for the colors that are used in it--it just does the collating and sorting).
"X or _smaller_ number of files."
why are no symbols better? I've never seen a rule either way
Re-reading that I came across more snarky than I realized, my apologies :-) (but this clearly the most important thing ever ;-))
yeah looks like it was a bug
Note to self: This is a lie, because the default is t, not nil. So update the documentation to reflect the default.
Why do you remove the property? If really needed, please remove the commented code, we can still go back to it using the version control
is this method used ? It it is, doesn't it just hide a DriverException behind an IndexOutOfBoundsException ?
I looked and was wrong. The client only checks for `status_code != 200`, and in other places for `result['status'] == 'error'`. So we should be good to correct this. And I shall.
_password_locator refers to the password field when you firest create your BD account
_login_password_locator refers to the password field when you sign into your account.
The devs have changed some locators with the new push to stage.
Can you make this the regular style of addEventListener?

```
input.addEventListener( "blur", function( e ) {
  __addListeners( e.target );
}, false );
```
On closer inspection, it looks like this file has a lot to do with user data. Perhaps a title which represents that makes the most sense.
Do we need to include util/lang, doesn't look like its used any where
This will always be an array?  It can't ever been a string?
Ha, right.

For some reason I thought you could make one function call and pass everything in.
change this command name to `trim_comment`
Could the member 'enable' be the first one?
細かい点ですが、 else 節を使って無いので if ではなく when を使ってもらえませんか?
I couldn't get a ToManyfield to work. The problem with it is that it affects hydrate, too. I don't want PUT/PATCH for speaker, category, or tags affecting videos. As near as I can tell, it's much easier to do the work in dehydrate.

And yeah, there's some duplication, but it's three pieces and it's pretty minor. Sure, we can construct a class hierarchy. I'm on the fence as to whether this is a compelling reason.
Let us rename the class to MergeStreamDataConsistency
Database is intentionally left out. As user may want to (and they often do) use it for some other projects on the same machine and they may not want to restart it.
In case they want to restart it, they can put this line into /etc/katello/katello-service:
SERVICES="postgresql mongodb $SERVICES"
Fixed in adc7e62.
这方法没地方调用？
@laanwj You convinced me, the important part is not via which method an array is initialized as long as we just initialize it :).

from http://www.cplusplus.com/doc/tutorial/arrays/
"When declaring a regular array of local scope (within a function, for example), if we do not specify otherwise, its elements will not be initialized to any value by default, so their content will be undetermined until we store some value in them. The elements of global and static arrays, on the other hand, are automatically initialized with their default values, which for all fundamental types this means they are filled with zeros."
Wouldn't this work directly without strprintf, too?
Can use the same command line flag for these too...
BTW, just for information (this implementation is fine), instead of installing an event filter on the object itself, you can also override one of the *event methods ( see https://qt-project.org/doc/qt-4.8/qwidget.html#event ).
It is trivial, this can be changed if you consider it appropriate... in fact, I just added this here for consistency I suppose.
 
Cause the easiest way to 'hack' into generating enum values was to use PHPMyAdmin approach: to use the (already available) length logic, which achieves the same cause they have the same shape: ```string(255)``` as opposed to ```enum('a','b')```, but the enum value would end up being a string.

Because there was a need to understand the enum values and later validate (and handle) them, it was better it they were an array instead, so ```enumerationValues()``` and ```isEnumeration()``` were born, and the test inside ```describe()``` was done https://github.com/cakephp/cakephp/pull/1011/files#L1L320
just above `$created` is set to false so no need to set it again.
There should be a space before and after `else`. Please follow Cake's [coding standards](https://github.com/cakephp/cakephp-codesniffer).
o_O  @ tabbing
Can this be changed to if bcrypt: <blah> ? feels more pythonic
@Ocramius just putting a note here for pending taking ClassMetadata to the next level to be independent of persistence and to be used outside doctrine such as e.g. Symfony2 :+1:
I personally think it would be more clear (and less step definitions/duplication) if this was written instead as:

```
When I select viewable by everyone
And when I select members invitable by members
```
OK - I'll modify this.
Yes, someone reading the code needs to know that this method only is relevant if the header is present.
这里只需要用一个 obj 里作为 unique 验证即可
instantiating exceptions should/could be done inside the body of each Enum element (with an abstract method at the enum), it ensures you can't add a new element and forget to add an instantiation clause
I guess you are using `insert cell above` because when executing the focus goes to the next cell... 
Could you add a comment to explain it, it is not obvious when looking at the code.
Also, how does it behave at the end of a notebook ?

You could also add one line in one of the example notebook for user to discover it.
space before URL, and 'pattern'. Although 'history pattern' isn't very clear, I haven't come up with a concise term for the history mini-language. Maybe 'input history range'?
no, just 2008 StartYear(--Now)
oops.. removed and re-tagged 1.7.0rc2 ebda8a9e856f11ab0c8a4eb966cc380e86823fb0 .
They can use CSS on the rewrite target. the issue is that they need to apply the CSS for specific device. and they don't want to have ugly user agent hacks in their code. That's why they wanted solution from the html5 library.
Can be removed, as you're making save explicit in `git_remote_create`.
I considered that, but it just came in with ruby 1.9.2 - is Mongoid alright with that dependency for version 3.0?
We're gradually working on eliminating the base.html vs. base-resp.html duplication - but it will probably be around for a few more months.
Ah, this should be `{{` instead of `{%`. That's probably most of the test fails.
These function names all start with an underscore (i.e. `_user_num_solutions` etc.)
Then, shouldn't that be `r'@[0-9a-f]{8}\b'`or `r'@[0-9a-f]{8}\s'` or something so that it doesn't match `for@example123.com`?
Crap.  Left overs from double checked locking code that I was trying. Removed now.
Could this be renamed to `loadTemplates`, I think it is a bit clearer what the function does then.
Logcat truncates at 4096 so this accounts for some prefixing. Was hardcoded in the old logger as well.
What if such cache name don't exists in this array ? PHP would produce `Notice` error, which makes application dead (hint: sf2 error handler)...
``CommandProviderInterface`` may be a better name as implementing only this command does not make you a bundle
I don't see this used anywhere else.
I see, totally read over the second parameter of line 62.
spurious statement?
Le 18/07/12 11:59, David Coeurjolly a écrit :
>> @@ -333,7 +333,9 @@ void DGtal::Display2DFactory::draw( DGtal::Board2D & board,
>>
>>     ASSERT(Domain::Space::dimension == 2);
>>     for(ConstIterator it =  s.begin(); it != s.end(); ++it)
>> -    board.drawRectangle( (*it)[0]-0.5,(*it)[1]+0.5,1,1);
>> +    //board.drawRectangle( (*it)[0]-0.5,(*it)[1]+0.5,1,1);
>
> fine.. could you please cleanup your commented line ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/DGtal-team/DGtal/pull/422/files#r1186432
>
yes that done ...
I don't really know what you mean and am unable to reproduce this.
++j?   :-)
If this is specifically for the sidebar, can we make the selector more explict? There may be other tables using odd/even that shouldn't get this update.
Hm, it *shouldn't* passed. If you're changing state internally `deserialize` should not be called.
@hisakonog said that was there to hide/show the div.. we might need to think about which section should be open etc anyway so that users can find their settings quickly
Yeah, this is correct. The resizable test is checking for support of complex markup and the draggable test is checking that resizing is possible while draggable is enabled.
I don't see the way to pass in any parameters.
Problem is with this line:

https://github.com/jquery/testswarm/pull/246/files#L1L151

```php
$localSettings = require $localSettingsPHP;
```

Require returns true and so in this line:

https://github.com/jquery/testswarm/blob/master/inc/init.php#L160

```php
$swarmConfig = object_merge( $defaultSettings, $localSettings );
```
you merge defaultSettings array and Boolean object (true).

In this case scenario localSettings never loads.
delete these two lines
instead of doing this recursively, it's not easy to do in a row, and doing the setTimeout when filling a chunk?
:) better to remove this debugging
Yeah! I have to check internationalization all around the Dialer app ;-) Thanks!
Yup it was a duplicated rule
Hey guys! I was wondering if it'd be possible to add here a file for carrier/oem/phone manufactor (whatever) preferences as you are doing here with other stuff. I don't mean to add preferences in upstream, just an empty file and add it to the build process. That way carriers/oems/phone manufacturers could add their preferences in that file at build time. For example we (TEF) are about to start playing with building OTA updates and delivering them from TEF servers so we'll need to overwrite `app.update.*` preferences.
Put the image in a `image` directory
`preventDefault` and `stopPropagation` is currently not working in System app for the purpose of stopping other actions. But we put this here for now to remind us of that. Should be fixed in #1663.
Add a space between : and the rule.

@alivedise Can you check the z-index will fit?
I can't really tell, but I think that error may mean that the argument is too long.

Making a bigger script that handles the backups sounds good to me.
@steveck-chung When your 'pending deletion' will be landed? Because we are dealing with this issue in order to get a 'full functional' version of the code... If your update will be over soon you could remove these lines and change with your new method ok?
The Cross-process mediastorage work has apparently landed. I rebased today and now I'm getting a crash. I filed: https://bugzilla.mozilla.org/show_bug.cgi?id=775591
class name is redundant
Do the `max_limit` and `limit` filters happen automatically somewhere?
the behavior that i initially had in mind was that the reverse mapping would just hold the first token to generate the id. but it doesn't really matter whether the reverse mapping holds the first token or most recent token that hashes to the id, in either case you may have a collision. changed in c5cc4a33424e1e97ff0f245b5475dbfc9cb7c973.
Sure it was =), tests are always good.

What do you think should I do here?
use an extra table to make your tests this break everything
~~~
それぞれのジョブオプションのエントリは可能な値をリモート URL からとるよう設定することができます。
~~~

~~~
それぞれのジョブオプションのエントリは可能な値をリモート URL から取得するよう設定することができます。
~~~
__sholud__ is a good friend of mine. His full name is __ebrahim sholud mediram__ and he worked with me on this request. It's his job to set and unset flags.
Sure. I'm really just trying to get one env working for this test, then I'll get the many envs going.  I'll probably add that yaml thing again (mostly because I know how much you like it)
Some of unit tests w/ inject didn't instantiate an Auth obj.  I fixed part of the issue elsewhere but I am leaving that in case it appears again when I match coverage.
Looks like there are still unresolved conflicts in this file
They are not required when used.
Not sure what this MAX_WAIT is for. Leaving in for backward compatibility.
Also, note that since this check is done 100+ lines in, even if the user passes in too many args, this script will still install curl and friends before aborting.
Raising exception now.
I think this method probably should remove every copy of its arguments, meaning it should use an algorithm similar to the one in `filter`. Further, the outer loop should go over the elements of `value` and the inner loop (which is hidden here inside the call to `indexOf`) should go over arguments, so that two adjacent but different removed elements will get consolidated within the same range in the change event.
Simple is better :-)

Restore the raw results and computed results to sql can resolve this problem, split the data and relationship.
This hack won't handle callables that aren't functions.
"that login the user" => "that logs the user in" @paulmillr
We should continue to use Backbone.Events.* here for clarity reasons. This is the mixins which defines the methods originally.
Of course `Backbone.on` is a reference to the same function, but if we write it this way a reader might think `mediator.publish()/subscribe()` have the same effect as the new Pub/Sub `Backbone.trigger()/on()` functions. But they shouldn’t be confused. If you publish an event in the context of the mediator, it has nothing to do with Backbone.on/off/trigger.
The second parameter must be set to ```true```.

http://php.net/manual/en/function.print-r.php
this line doesn't appear to be used for anything.
Could you please don't use comprehensions ? We banned them as it's was not obvious for people that are unfamiliar and they don't buy us much to be worth the traideoffs.

```js
let uuids = Array(5).map(uu) 
```

Is both shorter and simpler.
Unfortunately, this is a left over from the old required_email days. I believe this module is *only* used to authenticate a user who is logged in at the assertion level who picks a secondary email address and must enter their password. If this is the case, this branch may be unnecessary.
@benadida - Nice update to the mock!
The diff got confused and made it look like there was a catch all case for the path.  The below catch all (line 88 on the left) matches the if(cookiesEnabled) (line 77 on the left) from above.  But, a catch all with a console log is a good suggestion to aid developers who add new pages but do not add a corresponding handler.  Adding now.
Of course. :)
There were comment originally. Just stripped.
Can you add a blank line here.
Used elsewhere in the code (BasePeer), nto the purpose of this PR to change that.
Is there any need for having #! lines in spec files? Why not just leave it off?
The released version fails, the problem has been fixed in master but the gem has not been released yet, I'll change this as soon as omniauth is released again.
Unsure why we can't use a LinkedHashMap for this: http://www.scala-lang.org/api/current/scala/collection/mutable/LinkedHashMap.html
Again indentation. When I see this commit on github I see that you changed indentation ( personally I prefer using spaces to ensure that it is correctly formatted, but I am sure that @mvidner will disagree :)
I don't think that use try/catch here is a good idea. 

Hide an error returning an empty string can lead to errors when you are developing your App.
For me it's OK, Could you write send me a PR to merge in this branch?
since include is being treated as a function call, i don't think this space is necessary.  Same with call below too.
While that is true in the general case, I would be wary of messing with Cyan classes... they may make some assumptions
just for future reference: try to put unrelated doc fixes into separate commits. that way if we need to revert the code change, doc fix will be preserved.
matching quotes should be done with a back-reference on the right hand side.
Doctrine does not enforce any line length limit so you should keep it as previously
This module usually accepts both binaries and lists as inputs. Everything seems to work, except this line. It should probably be rewritten as:

    defp get_recursive_paths(dot, paths) when dot in [".", '.'], do: paths

Could you please try it and add a test using a list as arg? :D
I would have done a `''.join(name_root, page_num + 1, ext)` but that's fine :+1:
"GCC" can be replaced by toolkit.GCC  when pull request #38 is merged.
Shouldn't ConnectController extend Symfony\Bundle\FrameworkBundle\Controller\Controller?
Are data-numchannels and data-frequency new properties you are adding? Why not just use normal JS properties (audio.dataNumChannels)?
self.customized.release may cause a bug when update nil release

use try
I think exclude should default to something (None, I imagine?) Though I guess, without excluding anything, this becomes a no-op. Hmmm.
Should this be main_nav_link_list, to be consistent with the lists, below?
So with `ALWAYS_EAGER = True`, you shouldn't have to wait for elasticsearch to index... if elasticsearch _isn't_ respecting that because it has its own queue, we should see about patching our elasticutils as well so we can test without timeouts.

SUMO is all on es now, can you peek at any of their tests (or ask someone in `#sumodev`) to see if they're using timeouts or blocking queues to test? If we have to do this as a stopgap it's better than no tests, but I'm thinking we must have solved this problem already.
Any reason why the pause unit test is being updated here?
I'm not keen on the provide* naming of these methods. For the test writer the main interest is navigating to the page. I would use go_to_*_page