labofsahil/github-event-dataset-2012 huggingface dataset:
Heya

If u wanna used the ship from Booty Bay's to Ratchet, u leave the ship on Ratchet, u get a port to Ashenvale ( on heaven ) dunno why

Ratchet to Booty Bay's ( ship) works perfect

The Battleground Strand of the Ancients now total buggy, u can not used the relict ( to win the the BG, allys or hordes )
please fixed
thx
betaman2k
yes, but ng:app now takes an optional module name

2012/1/31 Marcello Nuccio <
reply@reply.github.com
>

> Isn't `ng:autobind` superseded by `ng:app`?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/angular/angular-seed/commit/02e4c215ebfb9d5a31a88e13928ac7d84d0c5a69#commitcomment-918407
>
poor for a test update
hi, i ve tried to set up the engine in my app but got the error below.
I had to add the YAML::ENGINE.yamler = 'syck' if defined?(YAML::ENGINE) manually.
I think adding it in your boot.rb wont fix the problem thus you should maybe add it in the setup guide: http://www.locomotivecms.com/support/installation/engine

Psych::SyntaxError in Admin/installation#show

Showing /Users/gho/.rbenv/versions/1.9.2-p290/lib/ruby/gems/1.9.1/bundler/gems/engine-c881750d78ce/app/views/admin/installation/step_1.html.haml where line #2 raised:
Yo no entiendo mucho :( Todava no se usar git. FOREVER ALONE.

On Fri, Feb 3, 2012 at 8:49 AM, Favio Manriquez Leon <
reply@reply.github.com
> wrote:

> jaja :D   Hmm, pienso que a la mejor no estan muy claras estas
> instrucciones :S
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mxlOS/presentaciones/commit/1e333dc3ef2843f9353ac1a4a893d02107719754#commitcomment-930979
>



-- 
Usa *Dropbox*, un servicio que respalda tus archivos en lnea
automticamente. Te ahorrar lgrimas, sudor y sangre. Si usas mi link para
inscribirte nos regalarn 250 Mb: http://db.tt/VFh3oBeJ
> main/res/values/strings.xml 100644 → 100755 

Why executable? :D
Overall, looking really good. Added line notes throughout.
`when-not` or even `if-not`
john mentioned in passing a while back something about how IE has a weird interpretation of `delete` on objects. I don't think this is a situation like that, but i recall the side-effect being unintuitive.
-mwindows is not needed here
cmake adds -mwindows when building the exe with add_executable(el_client WIN32 ...)
oo?
Yeah, it should probably be checking whether the login failed (for CAS error message), not whether it was successful, if we want it to raise an error if credentials are wrong.
ok makes sense, let me see how to remove this pull request
While I don't agree (why would people expect to find obsolete methods?), it can certainly default to false.
Is it ok that you don't have stat_sys_battery_5?
Please order your requires as followings: stdlib, gems, your files.
*.jsonp should be added to gitignore
nice, i'll edit http://www.readtfb.net/2012/01/15/handle-celery-dependent-tests-in-django-and-with-django-jenkins/ asap :)
serendipity_event_entrycheck/UTF-8/lang_de.inc.php  is missing the entry PLUGIN_EVENT_ENTRYCHECK_DEFAULTCAT now but the other language files still have it, riight?
Was this intended?
If you don't have javascript enabled then you shouldn't see the iframe at all, you only want the link. So adding the iframe with Javascript makes sense to me. It also helps keep the page as small and semantic as possible.
nit: C guarantees that all static/global variables are initialized to 0.  Is that what this is referring to, or is there a dependence on some gcc-specific behavior?

Sorry if this seems persnickety.
From being a pillar humping paladin on retail during WoTLK, I can say with out a doubt that spells should interrupt when you leave LoS or go out of range. Channeled spells however will continue to cast, unless you leave range.
I wonder if we should have separate versions for full and short ids (or is this handled at the level of the REST service)?
Strange.
Indeed if the cache is not stored you can refresh it by clicking on "Refresh". 
If it is already stored up to now the refresh just retriggers the store-procedure again.

Maybe this was done with intent but I don't know why ??
No, PWM_LEVEL_ADJUST limits the max pwm value irrespective of BL_MIN_LEVEL so it should probably stay at 226, unless Huawei got it wrong in the first place (which is not impossible).
Hmm, I just implemented this in JS like this: https://gist.github.com/1781216

I'll leave it to Alon to decide which implementation to take.  I'll write a test for it as well.
Ha ha. 
reponds_with_meme do |reponse|
   response.meme.to_json
end
I'm concerned about this too (looking forward to seeing the new approach). Generally, we should avoid anything that triggers eager initialization or that might lag behind something being configured within a BeanFactoryPostProcessor, especially if that BFPP is part of the core behavior that occurs with any SI usage. In this case, I'd not only worry about resolving the @ bean resolution (space added in case there is a 'bean' github user), but I'd also worry about property sources that may have not been established, and even the fact that some other BFPP might add channels, etc. I think it's a good idea to not do anything too proactive in a BFPP; unlike a BPP, there are too many things yet to occur and or too eagerly affected.
Did you fix the character creation crash i submitted? As much as you want to blame that on me, Check into it. I've been using WoW Servers for years. Theres either something wrong with your core, or the CataCrunch Rebase DB. Pretty sure it's the core. :P If you close a bug post you should atleast reply to it..
Ahhh ok. Sledgehammer but maybe the simplest option.
Merging this is going to be difficult...
Prefer to use the Tapestry CSS class : t-invisible.
This whole "Category for Country" part needs to be in a trans block with the category and country being passed in as parameters.
use this pic instead of the one now
http://dl.dropbox.com/u/23588174/saeindia.png.png
Ok I was missing the parenthesis >_<
I'm a nOob
May set it to 30ms later.
@aspee: Didn't flash it! But The UV module is still broken......
I think it happens because you work in the ps3mediaserver tree. This makes that the "Pull" command needs to merge the remote commits into your tree, seemingly making it a new commit.

You could experiment by using "Fetch & rebase" instead of "Pull".
This would revert your local changes, then fast forward the tree to the latest commit and finally reapply your local changes on top of that. To the outside world it would appear as if you did your work on the latest source.

There is, of course the chance that this will cause conflicts - the same chance as when you pull.
Do you need a separate prosody instance?
tested now and worked very well :)
bug fixed: MysSqlDialect  DoesTableExist does not work if tableName exists in other DB
What does it say?

Sent from my iCheese

On 4/02/2012, at 4:36 PM, "Aidan M."<reply@reply.github.com> wrote:

> It can't read the file..No clue why :(
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/TheSocialNetwork/TheSocialNetwork/commit/c7617ea6f318044f070d364352738aee5d72e706#commitcomment-932573
Yes, I can ;). This is a work in progress, so I can comment out something here and there.

Your develop branch belongs to you, and not to castle. I don't know if you're experimenting, doing something proprietary work or anything like that. I'll wait you to update the castle's transactions repo with a version with windsor  and core 3.x, and then, we can ditch this branch.
XD ok :P
CodeTweek
Get rid of `.DS_Store`
This is such that the eclipse maven plugin stop bugging the developer about the goal/phase not being part of the default. Not needed per se, and has no effect on the config, but one less warning after all which is good for eclipse users.
Upon further review I see now why we did not use X.class.getName() in some cases.  The last two credentials classes are in other modules, and adding a dependency on them would create a circular dependency.  I used the component name for classes in the same module and made a code comment that explains why it's not done throughout.
It would be nice if you used the :ref: directive around commands:

    :ref:`tronview`

and backticks around UNIX commands and paths:

    ``less``

It's just a changelog, but consistently good internal linking makes documentation so much better.
typo "leight-weight"
Yes, I think I'm going to tackle this next...
I think it's common practice to Capitalize extractors
make "bytestrings" private
seen.values.toSet.size
lol at this typo, but there are now not enough arguments to this log line.
would be good to mention something like "use 0 to disable." in the help msg
right, gotcha.
I don't like this API.  I proposed a new one before, but it got "outdated".   Here was comment:

I see you use this often. what if api were more like:

ok? :update, :trait=>tagcode. that's fairly close to the english "is it ok to update the tagcode trait?" what do you think?
And what if orientation_dep is `False` instead of `None`? Anyway, you should comment on my fork, as this code is rather outdated. I modified it to make it PEP 8 compliant and a few other things. There's still a lot to do, so if you want you can send pull requests on my fork:
https://github.com/rubik/kivy/blob/multistroke/kivy/multistroke.py
is this a regression?
@amercader : I've added a release-cycle process document which I've copied from the dev meetup notes.  Would you mind checking it for correctness and clarity please?
Could you avoid the use of the variable `$1`? Some Ruby VM have problems with it (killed performance or wrong values in threaded environments). See <https://news.ycombinator.com/item?id=4658933>.
This code assumes that height will be same for all rows, which is not the case when data exceeds certain length. e.g. in your test case try something like this, 
gridOptions: {
..............
                collectionMap: function(models) {
                    models[0].set('required_field', "test with a very long string, so that rows becomes bigger then its default height.");
                    return models;
                }
}

Note: You'll also need to increase width of "datetime_field"
Make a test for duplicate errors, and fix it if broken.  Ideally, this wouldn't even be needed and it may not be because all the validations are re-ordered because of deeper changes.
I guess this changes behaviour in that musicplayer.* (except title) will return labels while video is playing.  I don't think this change is needed though?  (Same for the change on MUSICPLAYER_TITLE, GetVideoLabel() and VIDEOPLAYER_TITLE).
до 202 стабулируй
I couldn't find a decent way to test it
this should be

    if not isinstance(train_text, compat.string_types):

because python 3 doesn't define basestring. Thanks!
I think requests has a `response.text` attribute that does the same as this. I don't know what version of requests we require, though - it was added some time in the last few months.
Is name parameter necessary? As the implementation is service specific, this name is implicitly the service name.
On 07/06/2012 11:08 PM, Tuncer Ayaz wrote:
>>       (catch eunit:test([list_to_atom(Suite) ||
>> -                          Suite <- string:tokens(Suites, ",")], EunitOpts)).
>> +                          Suite <- string:tokens(Suites, ",")], EunitOpts));
>> +perform_eunit(EunitOpts, _Modules, {generators, Generators}) ->
>
> Yes, it's less to type. If there's both `foo_test` and `foo_test_`, should rebar run both the test and the generator?
> Any other potential ambiguity? Otherwise, ok.

No, I think running only the shortest match makes more sense. I see no 
other ambiguity.

    /Richard
I'm not sure what spurred me to separate the tests using the next function.  In hindsight it seemed like a concise way to divy the tests.
That's what I figured. No problem!
Maybe year should be an Integer? It does not seem used, but I think it's better for POLS.
Memcache or Memcached ?
This could leak the return value of the `end` function, if it had one. Could you please change that to `end(data); return;`?

Better be fool-proof, and easy to read. Besides, the API could give a meaning to the return value of `camp.ajax.on`'s callback, in which case this code would be very confusing.
These should be tabs, not spaces.
This method can be replaced by a simple attribute accessor `attr_accessor`.
no doubt, and if it did it would throw the wrong message ;)
nono, ofc not! all stays the same. all is done automatically by the runtime. just if you need to unload the lib and reaload it, you need to use platform specific methods and thats the only case in which one would need the ReInit().
Either way is fine with me, but let's make it very noticeable since this is a concern that's both common and subtle.
Je l'ai déplacé dans mon dernier commit.
Redundant.  Class name tells what it does already.
Would this work if there are only positional arguments and the first is excluded?

The logic here is fairly convoluted; I wonder if it can't be cleaned up using set inclusion tests, etc.  I.e., the current logic goes "for each item in the parameters list, do a check to see if it should be included, and execute some behaviour depending on the outcome of the test", whereas an alternative may be "for each of the parameters, add it to the list with the appropriate transformation".
Could use `PyArray_DIM(arr, k)` here. Although I think the current code is actually more informative than the macro. Hmm...
I would put AttribDict into obspy.__init__ too ...
Alright, showing merge/commit now on my github fork, also took care of the variable declaration moving to beginning of block (unused node).
это к комменту ниже.
I think is will print a warning because `status` is not used
I think we need to check `logger.info? || logger.debug?`
Please rename this to `compiled_delegators`.
We don't need this `new`. Only `raise ActionController::UnknownFormat` should be fine
could this also work?

```
val params1 = if (isTailLabel) params else params map newParam
```
Perhaps we should have a code example of how the execution context uses this information.

Also, Java-compatibility-wise, would inheriting the `Runnable` instead of having a `self` type make more sense?
I'm thinking of a situation where a Java client inherits OnCompleteRunnable without extending `Runnable`.
An alternative is to remove `notAllowed("this statement")` below.
Needs ``plt.show()`` here and in second example below.
First of all, thank you for this module. I intensively use the Intervals.

Regarding this line:
I tried to test it 

```
>>> Intersection(Interval(2, 4), Interval(3, x))
ValueError: Only real intervals are supported
```
So it seems, that now this line is blind (not pass), because there are "Only reals' permited in the Interval's constructor, I guess.
At least I have not observed the example how to play with this line.

And it would be better to return Set, for those cases, as far as possible. `None` is often unexpected when we use resulted Interval in the future. 
E.g. we can use Max for this case.

`[0,3] & [x,6] --> [Max(x, 0), Min(3, 6)] --> [Max(x, 0), 3)]`

It is a separated question whether it is reasonable, because I can't imagine whether it's possible to have those evaluation in common cases, when all ends of the Intervals are symbolic or in complex cases.
But we doesn't lose object Inteval and, therefore can use, e.g. substitution: 

```python
>>> Interval(Max(x, 0), 3).subs(x, -1)
[0, 3]
>>> Interval(Max(x, 0), 3).subs(x, 1)
[1, 3]
>>> Interval(Max(x, 0), 3).subs(x, 5)
EmptySet
```
which are valid with original `[0,3] & [x,6]` cases of `x`.
should that be `an _eval_expand_hint() method.`
This is the problem with that PR. If `x + y` and `y + x` return different objects, it's not consistent any more to consider that `x + y == y + x`.

To put it another way, since we'll keep assuming that `x + y == y + x` and we won't be able to clear the cache every time an Add is created, the order of the args will still be effectively random, but instead of depending on the hashes, it'll depend on the content of the cache. I don't consider that an improvement.
`is_Mul` should still be `True`, right?
This does not seem right. (note that `exp_polar(I*pi) == polar_lift(-1)`)
if i call write, I'll overwrite src file. That's the thing I wanted to work around. I hate restoring files to state before test. And because this functionality depends on internal state of the object I cannot have separate agent for reading and separate one for writing as it is in config_agent-country tests.
sry, missed that last time. Your indentation is wrong, please use tabs instead of spaces.
this eem to be the problem ....
Is this even used? It is antiquated for two reasons:

 * OutcomeMeasure instead of Variable?
 * No WhenTaken?
This seems to be an unused variable?
Ahh, never mind.  There's a matching method I didn't see.
I see more benefits in passing DTO objects iinstead of passing all its properties . So, you will have something like:

```javascript
function CommuntiyViewModel(communityDto) {
   var self = this;
   self.id = ko.observable(communityDto.id);
   self.name = ko.observable(communityDto.name);
   // ...
}
```
A config file is not a requirement for running as a Windows service, but it probably makes it easier to manage (because you can just edit the external file and not mess with the settings of the Windows service itself).  --logpath is required, because without it all log output would go nowhere.  logpath could be specified in the config file if desired.
Unlike with `loaded_recipes`, I don't know of users accessing `loaded_attributes`, but one could argue they should be treated the same for consistency purposes.
If the avatar doesn't need custom HTML it will just return the src of the avatar with `get_data()`.
Get rid of the inline assignment.
Needs more explanation (comments) about what this is for
You can probably use sql_fetchfield('style_active') here.
Missing sql_freeresult.
Please avoid global variables in new methods. Instead, pass them as parameters or store them in class members.
Committed an updated version.
@joekain 

Absolutely correct.
But it should raise an exception when queues connection are not the equal.
+1

- Please remove `console.log`
- Would be good if you could be more explicit. Instead of  `toDegrees(radians).toFixed(0) + 'deg'` just write down your expectation, which is `45deg`. Hard to verify otherwise. Then you could also get rid of  `toDegrees()`.
Why not leave version attribute?
again, remove dropdown for now if only one option
Don't change this, we need OUTPATIENT encountertype -- it seems you are trying to merge back an old version of Util.js
Heh, yeah I struggled with this too! :sweat:
The intent is definitely desirable.
And now you make me wonder if the 'icons' should be simply returned as that instead of converted to HTML...
We don't change it for now
Removed in e770e7c.
you don't need ``$data`` IMO. It could be inlined
My only other thought is that we might want to put in an iteration counter as a failsafe? Also arbitrary, but guaranteed to terminate.
No you can't. But that's the whole point: we know that you can call `Object` without `new` and be safe but we don't know that about some random global function `Foo`. If you really know what you're doing and don't need these warnings you can just disable `newcap`?
I see. That makes sense. Then I would suggest that we may want to include the matlab code here to help the maintenance of the code.
You could even add a basic "console" that dumps the live ws output to show that it is working?
Yes is a typo error
Good. I am still learning :-)
That sounds like a clean solution. I am not the best person to
implement that, however. I am still new to homebrew after all.


-- 
 Dipl.-Inform. Samuel John
- - - - - - - - - - - - - - - - - - - - - - -
 PhD student,
 Neuroinformatics Group, Faculty
 of Technology, D33594 Bielefeld

 +49 (0)1577 573 0 572
 mail@samueljohn.de
 sjohn@techfak.uni-bielefeld.de
 jabber: samueljohn@jabber.org
- - - - - - - - - - - - - - - - - - - - - - -

Am 28.06.2012 um 07:21 schrieb Jack Nagel
<reply@reply.github.com>:

>>     elsif File.exist? "#{xcode_prefix}/usr/bin/make"
>>       # cc stopped existing with Xcode 4.3, there are c89 and c99 options though
>>       Pathname.new "#{xcode_prefix}/usr/bin"
>>     else
>> -      # yes this seems dumb, but we can't throw because the existance of
>> -      # dev tools is not mandatory for installing formula. Eventually we
>> -      # should make formula specify if they need dev tools or not.
>> -      Pathname.new "/usr/bin"
>> +      # Since we are pretty unrelenting in finding Xcode no matter where
>> +      # it hides, we can now throw in the towel.
>> +      opoo "You really should consult the `brew doctor`!"
>> +      ""
>
> We could add an `DevToolsMissingError` exception and just ignore it in the appropriate places.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/13037/files#r1065757
Preferred way to do this is just:

```rb
cd 'src' do
  system "make"
  lib.install 'libosmpbf.a'
end
```
このinclude必要? joinsでsenderかreceiverに絞り込んでるのでは?
"RPM Support for Pulp"... I like that term. I can't quite put my finger on what it reminds me of, but sounds good.
There is an underlying Cython method you can use (look at `pandas/stats/moments.py`). Or port the array math to not use rolling_kurt like nanops.nanskew
Maybe it can test to make sure reactor is not None?
Why?  `for (var i = 0; i < 3; i++);` is perfectly valid syntax
Yeah, and it's also weird because the children would inject different sets than their parents.
The directory name is extension-graph. Shouldn't that match the artifactId ?
don't ask me, its unchanged line from your sources =)
Didn't close the CSV before. Could cause issues.
Can we use feature detection instead or does android lie and say it supports <audio>?
I think this should be

```
if (autoAddRemove) {
  $popUp.remove()
}
_popUps.splice(index, 1); // always remove from our set of popups to track
```
What happens if the pendingFileList promise is rejected? That means the promise to Smart Autocomplete will never be resolved--is there any potential bad effect of that?
Yeah exactly.  Hopefully the pace of the HTML spec changing will be relatively slow and we won't have false-negative failures here very often :-)
No, I don't think it is redundant. They're used to set additional element to be selected. eg. rgb format set an extra element . rgba to be selected.

But UI-wise, we don't have a rgb button. So I'm not 100% sure too. Garth, can you chime in on this?
Would be good to add docs with a brief description at the top of this module
typo: sartLine
Do we want to export this so that packages that depend on ggplot2 can also use it?
OK now i think I understand what you meant.
is synonym to 
```ruby
rescue StandardError => e
```
Why is this a circular reference?
is normalize a dead function now?
I think you'll need to leave "util/shims" in the list here; just loading it adds the shims properly. You don't need the Shims var though.
Still getting lint complaints about the -o- prefix:

### Linting CSS files
css/controls.css: line 11, col 3, Error - The property -o-transition is compatible with -moz-transition and -webkit-transition and should be included as well.
css/controls.css: line 11, col 3, Error - The property -ms-transition is compatible with -moz-transition and -webkit-transition and should be included as well.
Should move onLeaveDialog into the ui block here.
some indenting issues in these lines, should be moved over to the right.
Duplicate selector?
sure that works too
Oh, I had sort of assumed that pending_modified_values was a DB field and thus we had a getter and setter from ActiveRecord.  What you've got now is right.
why sync?
Is this import needed? That's the default I thought
Do we need these two commented out lines?
Проверка не имеет смысла, `PATH.resolve()` всегда вернёт абсолютный путь
can you please SesameDiesel here? we don't need two different instances
I was kicking up an error which sends me an email every time I logged out of the admin. Thank you so much for fixing this!
remove:
 - _welcome
 - _demo*
 - _configurator ?
I mean the message
missingDirsCommittedPaths.clear() should be done here. Not in commitMirroredConsumerPaths method
Going to refactor this.
For historical use, would it be worthwhile to add a comment as to why the 'default_notify_options' are needed for these controllers (organizations, password reset, user session...), but not others?   Is it because current_organization is not set?
@parthaa This is actually being done throughout the changeset.rb for add/remove of items to the changeset (e.g. template, product, package, errata, distribution...etc).  My assumption is that is the reason the original developer chose to use ! in the method name.
Good catch, I was refactoring this, it should be :generic error.
Also move this to private helper method
Should be the max height of our target devices in horizontal orientation.

I didn't add this number, but I will add a comment on it.
nit: add a name to your anonymous function
This is a little nitpicky, but the docstring here is a tiny bit confusing - it's supposed to describe what this is, not what *type* it is.  That is, maybe something like "The numerical value of this quantity in the units given by `unit`".
just to clarify, I'm not suggesting we totally reimplement everything - instead, move it into an attribute.  So the change is that instead of doing ``mask.sum(axis=5)``, we do ``mask.array.sum(axis=5)``.  Then you're never overriding the normal ndarray behavior, because someone can still do ``amask.array + bmask.array``, but ``amask + bmask`` does exactly what you're doing here.  That's why I'm thinking it's the best of both worlds.
yadda yadda ;)
The timer is single-shot.
Thanks, so it would be nice to remove that burden entirely from this pull and the code. If this is a one way ticket there is no need to keep BDB compatibility code in.
Nit-pick: Can you please remove ``.`` at the end, to ensure all that status tip formats match.
If the original fallback is unnecessary in windows XP or later (the current requirement for the core code), I'm fine with removing it.
Ah I didn't know CakTestCase::setUp() did that. Will remove it before merging.
CakeFixtureManager::fixturize() already sets ds to test and testing to true for the Model config stored in the ClassRegistry.  It could get confusing if we set this up again here.
Why is noLayerSelectedMessage missing?
So, this seems to be unrelated to the topic of the Pull Request.  I'm also wondering if it's the right way to handle a node going down.  The 'rexi_EXIT' and 'rexi_DOWN' messages are treated differently.
I couldn't find anything referencing it. I believe they all use jquery.captureDocumentWrite.js now
Do you need to do a check here to verify that the result came back with data, or is that handled implicitly somewhere in the core code? I can't remember.
You shouldn't need this type at all. JSON.NET is built in no?
Again, minimal diffs
As response can indeed be null in case of an exception, it should take an Option[Response]
@ninjudd Does this todo need to be addressed before merging into develop?
I'd also like to see at least a small docstring here.  Even though these are little utlities, they're starting to get 'real' enough that we might as well document them a bit more for use, even if it's just among ourselves.  It will make it easier for others to understand and improve the code later.
I'd rewrite this cell as:

```
from scipy.special import jn
x = linspace(0,5)
f, ax = plt.subplots()
ax.set_title("Bessel functions")

for n in range(1,10):
    time.sleep(1)
    ax.plot(x, jn(x,n))
    clear_output()
    display(f)

# close the figure at the end, so we don't get a duplicate
# of the last plot
plt.close()
```

It's a bit cleaner api-wise regarding matplotlib, and it completely eliminates the flicker. The trick is to do all plotting with the axis object, which doesn't force a draw call.  Then, we can do clear_output right before the display call, and most likely (unless it's a super-complex plot), it will finish before any flickering happens.

You can mention this and update the comment below accordingly, indicating how this will eliminate completely all flickering in most common cases (though it can still appear for sufficiently complex plots).
Ah, got it.  That was the result of doing the cherry-picking a bit too blindly.  Will do.
ah, there's the transformKeys :) still, same point wrt to assignments.
what do you mean? can you make an example?
I think this cannot be generated from IDL.
(Are there any other ways to avoid warnings?)
checking for `path('app')` doesn't do any different for other bundle.  Since stub already pass in the `$directory` (using `Bundle::path('bundle-name')`) we should use that instead.

`$stub = File::get($directory.DS.'stub.xml');` would do the job.
I would say wait for 1.2 to support newsgroup password promps.
No, the behaviour is also wrong. Clicking "Previous" on the first page of section B takes you to the first page of section A. It should go to last page of section A.
A bit prettier:

```javascript
$('<span class="btn-play"></span>').appendTo('#video-stage .player').click(function() {
```
Shouldn't it be 'this criterion' ? (I'm uncertain myself here)
why not simply os.path.join()?
@peterbe hm good point, I was following the existing style though (c.f. failureMessage) which is better than having it be mixed or touching unrelated things IMHO.
please use public static instead of static public
I would try to write it better (less code) ...
лиши #assets_version. потрібна штука.
ps. вирівняй значення. посунулись вправо
that needs to be reverted
Just curious: `Boolean`, but not `Integer`?
Should be `<=`?
rounded. and removed the now unused constants, which shortened the lines to the length of others in the file.
Thanks for correcting my silly error
Is reconstruction hyphenated? I thought I could use it in Scrabble ;-)
Left as a standalone function because it's useful in it's own right, but does it belong here?
Let's just change the PCCA script to return the arrays?
thx, will update
The test deliberately builds the tree in the 'wrong' order, to simulate things being added over time, not optimally - the nodes go: 1 2 4 10 8 3 9 5 6 7.

But perhaps it should be made even more difficult than that, to be a proper test.
wasn't the original plan to put the overhauled admin into a separate project anyway?
Wow! That's a big chunk of work! Nicely done! Can't wait to try it out ... :-)
This isn't XEP-0047, so a different callback name should be used, especially since it's that name is already being used.
This is possible, but not always what you want as if you resolve against AttributeDefinition it takes into account also default values.

what you can also use is
```java
attributeDefinition.resolveModelAttribute(ExpressionResolver.DEFAULT, model)
```
that works even if you don't have context
Like I said, we won't be needing it anymore.
这个是dream项目的domain
а зачем отдельно хранить replyTitle, когда он есть в reply.title ?
nit: extra line?
ok, cool!
Same thing here: I'd put this else clause in the event handler for the shuffle button.  That will simplify this function quite a bit.
It might not improve the UI response time if you still wait for all the messages callback for getMessage. If you want to make the change the bahavior, you need to change getMessages to callback every message in cursor, and your render function will only receiver one message data a time and you can insert/update contact into list right away instead of wait for the cursor traverse whole DB then callback the messages.
Do you mind removing the console.log, I would like to not pollute the log.
You went a little bit too far, this is still in use :)
Can you move the console.log statements behind a debug flag somewhere?
that was because I didn’t want to indent the whole JSON object, but just make it easier to see how many locales have been processed. That’s just for debugging though, I agree we don’t need this here.
why do we wait here depending on the number of friends?
ugh, I missed these `pack-icon-*` things. Are you using `.pack-icon` instead of `.icon`? If yes, why?
This is fixed with my commit https://github.com/jhford/gaia/commit/402e8cac99aae6de74de4660628183e758403109
Hi, this is added to test another issue for "select based navigation".
Take MDN as an example, 
https://developer.mozilla.org/en-US/

we have a <select> element at the bottom of the page for language selection.
If we select one of the languages with Gaia value selector, since the change would be triggered and cause the browser to reload.
We would find that we could not receive a blur event to hide the value selector.
why do you need to load those files at the end? Is 'defer' not working?
The list was required last time I tested, will check again
Is that intended behavior? If yes, maybe we should add a comment on why it is empty.
Does site always have at least once binding?
You're right, @alexch.  I successfully rebased with @nuclearsandwich's assistance this evening, and with the successful bundle install as of this morning, I can finally validate all my changes.  This probably needs to start with "step", which I'll validate tomorrow.  Once I've finished cleaning up, I'll close this and open the rebased version.
i dont know, it should not `try` to sign up, but say that the user wants to sign up :-D are you concerned because of the length of the name?
I think that could make this an implicit. All the TPipe methods (map, flatMap, etc) have different signatures than the equivalent ones in Pipe and Mappable so we should be ok.
Why not just one block with || in if?
This should be prevented in route setup stage and if still happening here, return an internal error
`namespace` is spelled wrong.
Don't use assert - use an AppScaleException.
That makes sense, that makes it easier for the client. I don't know why i made the start stop button at the beginning. Anything init can be done while client is being launched. I agree that there is no need for that button. 
On 2012-08-22, at 7:27 AM, Dennis Ideler wrote:

> In src/freeseer/frontend/controller/Client.py:
> 
> > +            self.status = 'Not connected'
> > +        elif state == 1:
> > +            self.status = 'Host lookup'
> > +        elif state == 2:
> > +            self.status = 'Establishing connection'
> > +        elif state == 3:
> > +            self.status = 'Connected'
> > +        elif state == 4:
> > +            self.status = 'The socket is bound to an address and port'
> > +        elif state == 5:
> > +            self.status = 'For internal use only'
> > +        elif state == 6:
> > +            self.status = 'Socket is about to close'
> > +        self.statusLabel.setText('Client status:' + self.status)
> > +    
> > +    def startClient(self):  
> After playing around a bit with the client, I think there's not much need for a Start/Stop button. Why not start the client as soon as it's opened and stop it when the window is closed? It will make the client easier to use. Also, the current implementation is buggy, so if you eliminate it then you get rid of the bugs without having to fix them. What do you think?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

Bora Sabuncu
might be worth using the existing lexer stuff:

```js
  color: function() {
    return this.rrggbbaa()
      || this.rrggbb()
      || this.rgba()
      || this.rgb()
      || this.nn()
      || this.n()
  },
```

though it's a little awkward since you'd have to initiate a lexer currently
The filter supports empty strings.
In this case you probably don't want to search for empty strings on an archive.
good, use n - 1 for population SD.
re.findall('/dev/vd\S+', s)

the first disk is not always '/dev/vda'
Why + 70?
`$HOME/.composer` e.g. `/home/<user>/.composer` on *nix (maybe add OSX too?)
`$APPDATA/Roaming/Composer` - e.g. `C:\Users\<user>\AppData\Roaming\Composer` on Windows
Fixed all in master
IMO, you should run ``install`` after a checkout, not ``update``
we really don't use mustache, but it's nice to have.  Also, can you remove the `html` portion of the erb and haml entries? those can live without the html.
@randx Your concern is understandable, thank you for sharing it. However, the local variable `custom_port` will be initialized (with the value nil) even if `web_custom_port?` is false or nil. From http://ruby.runpaint.org/variables about local variables: "It is initialized if it appears on the left‐hand side (before the equals sign (U＋003D)) of an assignment expression, even if the expression does not actually execute. Variables of the latter sort have the value nil."
likewise, no need to comment out, just kill it
Maybe we should use © instead?
Are you updating the patch for that?
Forget what I said. It seems `NetUtils.asyncFetch` doesn't work with UTF-8 charset, even if I don't know why. Maybe the `nsIPipe` add something or I did something wrong. Here the code:

```javascript
function readAsync(uri, charset) {
  let channel = NetUtil.newChannel(uri, charset, null);

  let { promise, resolve, reject } = defer();
  let data = "";

  NetUtil.asyncFetch(channel, function (stream, result) {
    if (components.isSuccessCode(result)) {
      let count = stream.available();
      let data = NetUtil.readInputStreamToString(stream, count, { charset : charset });

      resolve(data);
    } else {
      reject("Failed to read: '" + uri + "' (Error Code: " + result + ")");
    }
  });

  return promise;
}
```

I expected to have the data returned properly encoded with the `charset` given, but is not.
@erikvold, if you can see something wrong in that code I'm willing to fix, otherwise I would say it's better use the current code with `channel` that works fine, and don't block the pull for that. What do you think?
Do we really need public method like this ? I think it would be better to just have `findMatchingNode(selection, node)` function used with in this module
Added comment elaborating more.
Please use relative require and move imports to the top of the file.
This case actually really only happens for the `create_templates` script, since it passes a `lastGen` timestamp based on `fs.statSync('build/templates.js').mtime`. If none of the individual template files have changed since then, it won't rebuild. Since it's not a long running process, it won't have the data cached in the closure, like it does when running the server.
Looks like this had already been fixed in winchan.js itself
that callable is actually the default. It'll compare the 0th element. What you have there is the same as `namespaces.sort()`.
ok, made two methods on PerformanceTestModel which delegate to SQLDataSet and make these calls.  Makes sense.
Please don't make it an anonymous function. Only change its name.
Yes. I just removed this file. Forgot about it. Rehan made almost the same file but in 'includes/macros.html'. I kept his.
I think there is a better code snippet for doing what I described in my previous comment probably... If it's true, let me know! I'll update accordingly!
The top level keys *are not* part of the general IPython message spec; what goes over the wire is supposed to only be the header / parent / content dictionaries. Min talked about this here https://github.com/ipython/ipython/pull/2000#issuecomment-6497952.
if making a pull request, please try to follow the code style. thanks!
there should be small explanation what is udev Key Value
Yes I guess it's overcomplication. Just wanted to make it return same results as `bccomp`. Will fix that.
Actually, the way matthew has it setup, EventManager must always be new (its set as not-shared).  So that behavior is correct, I think.
Changing an existing interface is a BC break.Separate the new methods into a new interface, and implement that in the concrete implementations. Test for that interface before doing operations on it in a public consumer.
Direi di tralasciare `Con il tuo IDE di sviluppo preferito`
This throws should be `Exception\RuntimeException`
The problem with making it a warning is that it will get lost in the noise of the script output and it will get ignored...
^ plus your javadocs are wrong.
I will look into what you pointed out, Art.

Thanks again for the help.
Maybe, but it worked! I did get RED, go figure.
top-vars is a personal preference thing.  I go back and forth, but generally err on the side of keeping related bits of code as close together as possible.
lets keep this as `postgres`
I dont like this, can you add a private method for this juggling instead and remove the = nuil? that makes the code much clearer.
Line breaks are flipped
again referenceable and versionable should have the exact same logic (ok, i admit they do not atm. but can you please clean this up while you are at it?)
Please add empty line afterwards.
Assumed so. That's the name of the PyPi package, it installs as "unittest2", like the unittest2 package does. But since now you ask, maybe we should investigate further.
Should be in line 623, the comment. Dunno what went wrong.
This we should redo all GPL headers. Pieter only has copyright in 2011, I only since 2010, Jens only since 2011.

nudded, if you're up to it, change it to (where everyone is included):

Copyright 2009-2012 Stijn De Weirdt
Copyright 2010 Dries Verdegem
Copyright 2010-2012 Kenneth Hoste
Copyright 2011 Pieter De Baets
Copyright 2011-2012 Jens Timmerman
Copyright 2012 Toon Willems
"not submitted yet"
Shouldn't we use CDI injection, not EJB?
Yeah, this is due to Ruby's wacky implementation of Kernel.raise.  If you give it an exception *class*, the initializer is called with the message as the first argument, but the message is also stored on the exception.  If you have an #initialize for your exception, you may not want the message as a param - in which case you explicitly create the instance of the Exception.  The message is still stored on the exception somehow (possibly using a separate setter method), though according to what I read here: http://www.ruby-forum.com/topic/207448, the message and backtrace (optional third param to raise) are tucked away in inaccessible instance variables.  When you call #message on the exception, the message is returned even if it wasn't part of the initialization, since #message uses #to_s which uses the internal variables.  Magic!  And yes, this has been raised as an issue, and debated:  http://bugs.ruby-lang.org/issues/5898
```SDL.surfaces[surf].colors   = []; // index -> color``` is needed because function copyIndexedColorData does not check is SDL.surfaces[surf].colors defined. So if add this check into copyIndexedColorData this line can be removed.

```SDL.defaults.copyOnLock = false; // not supported yet``` is needed because for tests. I dont know for now how to set copyOnLock==false throught pre.js.
You should add a test that the search view doesn't return results with a totally empty search (at least if I'm reading the code in `views.py` correctly).
Put initialized but not assigned vars together at the end of the list.
old habits :D
Is there a way we can know what the full expected title will be rather than using contains? because contains is a little vague, albeit intentionally so.
I moved the author actions out of the work module and into the blurb because we don't want, need, or currently use them anywhere except a plain work blurb (so, not in bookmarks or on the history page). We might want to make the author actions into a partial on the off chance we do decide to incorporate it elsewhere.
logging the exception (`logger.warn(String, Throwable)`) would help because the stacktrace could be included in the logs too
well the right thing to do is change
if ($.inArray(baseErrorKeys, attr)) {
to
if ($.inArray(attr, baseErrorKeys) !== -1) {
but i thought it's safer to keep the original code there
This is probably not intentional?
should we check if orgs are enabled first?
Can you fix this? If the OPAC is in fact limited to local use, there's no purpose in including it in the general Zotero release.
You should have commented your function, like we did in other yunohost_*.py files, with params and returns.
it should be $expected, $result in cake2.x
Please put images in the assets directory, not on a third party site.
it was a copy from my temporary files, I still got he issue with github commits, will fix it.
Ignore this;  this comment should be moved to your other PR.
This should not be fixed ever, may can be an external language like JavaScript
сделать функцию от одной до многих.
И адекватный интерфейс обращения к этому.
True, at least the table models shouldn't be kept in memory as they're only needed when the data is being read. I'll check what the code would look like if the filed indexes aren't stored.
I've fixed this up, I think. Please have a second pass on it.

* If it's the admin or the user themself, show both displayName and publicAlias
* If it's a different user who has full access to view the profile, the publicAlias is simply supressed (displayName is left as-is)
* If it is a user who doesn't have access to view private details, the displayName is replaced with the publicAlias, and the publicAlias field is supressed
The way the documentation is worded on this method, it says it **expects** a stringified boolean, therefore only input that is "true", "1", "false" or "0" would be acceptable. When I see the code below that simply "tries" to change them to boolean using it, this makes some sense.
I'm not a big fan of the `@borrows` but that's like my opinion man. If we use this the Docs UI should properly link to the original documentation.
Why is it an edge case?
This feels like a pattern that will come back quite often. Perhaps it can be consolidated into content.isAdmin ?
Can't we just render the external logins as links ?
I'd prefer to keep ``createSourceFile`` over ``sourceFile`` because 1) it better express the intent (IMO, the side-effect is clear, while one could expect that ``sourceFile`` would not have side-effect), and 2) for uniformity in the codebase, we have several ``createXXX`` methods already.

I know you like short names, but in this case I think the extra verb adds clarity :)
The `default_river_lifetime` setting is missing
personally i would change this to `--hogan`, is it referred to as hjs often?
No problem for me :) What do you think @willdurand  ?
These header guards do not match the general convention for Cyphesis. Check other headers in this directory, and ensure they match.
Oh, one of your option help strings says it only works in EMR mode. In any case, let's get rid of the commented out code. :-)
Credits are done on the header
Пардон, не заметил. А что произойдёт если rnd два раза будет равным 99 ? Я к тому что был массив на сто элементов, выпал последний, вы сделали [indexes removeObjectAtIndex:rnd]; а потом ситуация повторяется.
Спасибо исправил - было бы хорошо если бы такие моменты были освещены в задании потому как сразу не понял что надо заменять именно на другой символ.
Честное слово, пропустил, вот плохо никто раньше не смотрел внимательно, а вообще выпадет эксепшен)) так как mutateElementWithNumber:(NSInteger)x в этом методе при большем 100 номером, вызовется эксепшен с сообщением, но я согласен с этим предложением)))
Please avoid trying to align '='. It's not maintainable long term. The usual convention is one space on each side of a mathematical sign.
This tests the invalid action it checks that the hunt did not get saved
I did look into sprintf and also ngettext for plural forms.  The full strings are not currently found by the translator though so submitted string is returned instead.  For now I'd just stay with the existing translated substrings, they should be fine for this, and if someone wants a project to go through and update translations that would be good.

Thanks for your input.  I've learned a lot.  Though not as much as there is still to go.
no need for ancient activesupport support :)
*args is the only way I know of to have a variable number of input arguments. I figured the docstring would clarify it for people.
I'm not bothered by this em dash, and it seems correct to me without a space. But I will ask @chloeatplay to decide what she wants to do here.