labofsahil/github-event-dataset-2012 huggingface dataset:
Some hybrid/Cordova updates

- Removing lingering SFHybridApp.css calls.
- ContactExplorer gets the Cordova module invocation makeover.
- Fixed issues in ContactExplorer with refresh event not being consumed.
issue #35 fix

Not sure if it is as it should be, tho looks like this way solves issue without leaving artefacts in database.
New example using Hammelin

Couple of commits to improve application.rb, new data chart to make it compatible (till more work is done) with JFugue and a new example using the Hammelin gem (though it requires JRuby).
Account Callback Receiver Shouldn't be Unregistered on Low Memory
BUGFIX: notification emails sent individually to each member (issue #31)
ENHANCEMENT: customizable alternative field

allow the alternative field to be populated with a fully customized date
time string.

this will allow you to set the following config's:

{
altFormat: '', // format of the date in the alt field
altTimeFormat: '', // format of the time in the alt field
altSeperator: '', // seperator in the alt field
altTimeSuffix: '', // suffix in the alt field
}

if any of those config settings is omitted, it will fall back to the value of the normal field for this setting
if altFieldTimeOnly is set to true, which it is by default, nothing of the above settings will be used
LPS-24909
List Images

Parsing best candidate (or html) for find images.
You can use this for thumbnail, for instance.

You can use options for skip some formats, like this: 
<pre>
 # gif is default.
 @doc = Readability::Document.new("www.example.com", :ignore_image_format => ["gif", "png"])
</pre>

If you want, may use options for width and height:
<pre>
 @doc = Readability::Document.new(@bbc, :min_image_height => 400)
 @doc = Readability::Document.new(@bbc, :min_image_width => 100)
</pre>

After initialize, call **images** method:
<pre>
 @doc.images
 => ["www.example.com/images/1.jpg", "www.example.com/images/2.jpg", ...]
</pre>
see if this one works now

maybe this will work
Reindexing all models in the database

Bunch of tightly related changes within a single commit:

1) DeliveryMode is now determined at startup and then reused. This should improve performance somehow. (it has been read from configuration on every single event)

2) We can reconfigure handlers at runtime using setDeliveryMode setter. I have also added a method index(model, deliveryMode) to index a single entity using provided delivery method.

3) As a convenience, I have added 2 new delivery methods: (blocking) SYNCHRONOUS, which circumvents jobs and indexes from the same thread, and DISCARD, which ignores all events from persistence.
2 & 3 have made my previous modifications for issue #55 obsolete. I have reverted them as this is how I should have done it in the first place.

4) ElasticSearch.reindex method starts a job which reindexes the whole database using any deliveryMode you choose (default is SYNCHRONOUS, so the job does not spawn any children).

5) Collections used by Plugin are now thread safe. The module itself is not thread-safe as there are just WAY too many unsafe assignments and it would be too much work fixing them all, and it would probably also degrade performance.
Update de_DE/contact.php

Hey there,

I changed some words to more common ones, gave the form a CI and spell checked it. I just had a look over the german translation, sometimes the formal personal pronouns are used, sometimes the informal. For the user it makes a difference (For example You Tube uses the informal), I would like to recommend to change it to the informal. Just reply on my comment and I will change it everywhere in the german version.

Kay
Issue 7523 - Incorrect capacity for new T[] with non-zero T.init.

http://d.puremagic.com/issues/show_bug.cgi?id=7523
Trusted name color

Changed color of person's name to trusted color #127e0e whenever in trusted network
Add ameraController
Return the view that was instantiated by #view

When you instantiate a view through #view the view that was instantiated is returned allowing for chaining.

```
main.view(".header", new HeaderView).render();
```
Release/tucker

Added CSS on "onemenupage" projet listing, with the scss page.
I've also updated a h2 title color on the homepage, but i've not created the .css. Only sended the scss.
Makes earliest/latest date used in datepicker

earliestDate and latestDate didn't get carried over to the graphical
datepickers and to me that was not what I expected.
Place assignments in lecture folder ...

And update version of selenium-webdriver to work with current versions of firefox in assignment 5.1)
Replace PRIMARY keys in SN_FMS_GROUPS_TABLE by UNIQUE keys

As it was changed because of SQLite bug, update script should replace PRIMARY keys in old installations so that we do not have two different working installations.
Comedycentral

https://github.com/plexinc-plugins/Comedy-Central.bundle/issues/4
pull request

Nie wiem czy potrzebny ci ten pull request ale w razie co robie
Fixes and ability to create bodies withhout cpSpace.

Hi!

1) There was no main.m and i readded it - so it builds now ok.
2) Circle fixtures position was loaded from fixtureData, not circleData - it can look ok, when you have one body with one circle fixture, but when you use large complex bodies with circle fixtures inside - this breaks circle positioning.
3) I have a need of creating bodies & shapes without a knowledge of space they will be added to, so i copy-psated & changed a little createBody, so it returns now shapes, without adding them to space.
So they can be added later like this:
```objective-c
    cpSpaceAddBody(_space, anObject.body);

    for (NSValue *shapePointerValue in anObject.shapes)
    {
         cpSpaceAddShape(_space, [shapePointerValue pointerValue]);
    }
```

Best Regards,
Stepan
Arc 3.2 smp

Hello Vineet,

This is our latest version of SMP version.

You may use it as reference.

Noam
Updating context menus to better match the UI specification.

Still need to complete unit testing
Added a new file, testing pull request
Develop

PLFM-1463
Bring in some upstream changes

Pull in some changes from gerrit
Resolved the timeout issue and added asynchronous uploads for user feedback

Commits say it all.
Added a translator and test. Also, added missing required nuget files.
(- fix) Moar.
my-boolean -tehtävä on nyt korjattu

Käytin nyt if:iä my-booleanin toteutuksessa.
Fix badge not showing "off" when Chrome is launched, line endings in myscript.js

Hey,

I wanted to fix two little things:

1. If I pause it, the next time I quit and relaunch Chrome the badge doesn’t say “off” anymore, even though it’s still paused.

2. It looks like the line endings in `myscript.js` changed partway through from LF to CR. This makes only the first part of the file show up when you look at it on GitHub.

    I talked to GitHub about it a couple of weeks ago when I was trying to figure out how your extension worked, and they said that it’s a bug on their end! But it might be a good idea to change the line endings anyway — I don’t think git handles CR line endings too well.

See you at HS.
Yo Dawg!

I put a Yo Dawg in your readme so you can read me while you readme.
Fix list array
Radio FM libs

Fixes #32. Also fixed some permissions.
this guy was the problem. was "on" to use the firebug extension for the closure compiler (extension is not working: only up to firefox 3 sth which doesnt support webgl....)
Yeah, we discussed it, and we'll brainstorm possible solutions in the 3.6.0 timeline.
you mean at Aura::HandleModAttackSpeed, I should also implement, and put something like 
GetTarget()->ApplyAttackTimePercentMod(BASE_ATTACK,float(m_modifier.m_amount),apply); , 
but for damage? cause it doesnt exist yet. Sth like ApplyDamagePercentMod ?
1) this commit breaks things - there is no v1.3.3 of factory_girl_rails on rubygems.org
2) why freeze it at all? Isn't it better to let users/bundler decided which version to use?
there is now node::SetMethod which is a templated function. It might be worthwhile to update the examples to use that instead of macros. Thoughts?
the yiic.bat should be removed as that is your local copy 

About the events... I'm not sure about the focus event... and in any case i don't like to bind to too many events

this can be solved by using the 'focus' event in the autocomplete options...

so the code would be:

```
			'options'=>array(
				'minLength'=>'0',
				'focus' => 'js:function(event,ui) {
					$("#'.CHtml::activeId($model,'tableName').'").val(ui.item.label);
					return false;
				}'
			),
```
Please send through a pull-request!
DUAL_XIM_ENTRY 是 N 年前遺留下來的無用 struct ，gcin 也早在 N 年前就停用它了，所以只有 xim_arr[0] 沒有 xim_arr[1] 之類的。
當初的想法是要讓單一 XIM server 同時支援不同 locale, 但實際上 XFree86 / Xorg 預設本來就支援單一 XIM server 不同 locale ，所以 DUAL_XIM_ENTRY 完全無用。

XIMS xims 只是用於檢查 IMOpenIM() 成功與否，不需要 global variable 。
Is it? I don't think I mind, just curious.
It's back: 7cee968
Namely `_VERSION`
excited for this to finally get merged in.
Отличный патч. Возможно стоит переделать использование $base_url.'/search.php на forum_link() в moz:SearchForm?
@fxn I think you're right.
Why are you comparing the block type id with the maximum number of tnt blocks allowed?

This blows my mind.
Unstead of calling unlink() on various occasion, it might be better to use atexit() to register a function charged to unlink. At least you'll be sure not to forget an unlink.
> you could do a .downcase.to_sym :-)

This is something I did in the past but I'm trying to get rid of it.
I'm slowly moving the parser codebase to a normalized set of statuses. Unfortunately, I didn't have the time to document them yet (see #2, #5).

The current normalized list is

* available - not registered and available for registration
* registered - registered
* redemption - redemption period
* reserved - registered, reserved and not available for public registration
* unavailable - not registered but not available for registration (aka unassignable)
* inactive - registered but inactive (no DNS)
* invalid - invalid request

Of course, not all registries supports all of them.
It calls the copy constructor, rather than instantiating the object and then calling the =operator().  Basically it means it can be a little more efficient for some types.
Good catch - fixed now.
What about just declaring a `allowed_writer_methods` class method for `Virtus::ValueObject` and then in there perform this logic? Then since the `set_attributes` always accepts the "allowed_writer_methods" and you can make it so it just checks the attribute internally rather than having the list passed in explicitly.
Indeed. Need to use os.symlink.
I was having quite a few conflicts because all these were at the same level.  I tried removing some of the instances which don't seem like they're strictly necessary as they're provided by other instances, like `Pointed` and  `Foldable`, but then the tests broke for those type-classes broke because the instances couldn't be found.  Not sure why.

I also tried annotating the return type of the implicit defs, but on a few, such as `Pointed` and `Monad`, that caused SOE in the compiler.
Depends on whether you're providing a fully customized installer. And I don't see the point in not handling missing files in a bit nicer way? What does it cost, apart those two lines?
I'm really thrilled this got back to Jetty world. However please note that there's (at least one) bug:
https://github.com/jruby/jruby-rack/commit/debc8b73a2ca0ccdb121d9edfa3d517f720310c7
Ahh, indeed there's no problem here. Simply I misread.  Thanks for kind comment!
The code style is bad
I guess one question to ask is "what is considered arithmetic" in the phrase "all arithmetic is Int"? For example `ref(Array{T},i)` always gives type T, but should `ref(Range{T},i)` give `Int` on the basis that its elements are generated arithmetically?
[404](https://github.com/opengeo/gxp/commit/db9c865b9ab7f13be085f8649933f3dfb92304cb) !
Please don't combine multiple unrelated changes in a single patch because it makes it difficult to review.
speedup -> sped up
Good point. Yes, ok I'll change it and send new version of patch.
Create script for vehicle, there's a PassengerBoarded hook in CreatureAI, also need flag CREATURE_FLAG_EXTRA_KEEP_AI in db
I've never written a doctest before, I've always done nose/unittest type testing.  My understanding is doctests are executable code, but we don't have a DB to execute against I would imagine.  I guess I'll have to figure out how you do it for your other tests.

as for #1, you are right about db.query and a reference.
Yeah, this check is bogus. The diff below already covers it.
Windows deals just fine with unix-style paths, so you should perhaps just convert to unix-style paths here.
For Postgres, using "int4(#{Vote.table_name}.vote)" would fix the UNSIGNED bug since UNSIGNED is not a valid type in Postgres. This allows for values up to 2 billion. If the value is always 1, 0 or -1 then integer() is a valid call.

I'm now looking at ActiveRecord to see if there's something that can be used that is not database specific to abstract these casts.
functions which has leading underscore. don't you like? if so that, sorry ...
The `run()` method accepts an options object that defines how the tests should be run. I think this would be better as an option on that object.
Hi, that was a very good patch, but also I need to truncate the crumbs generated to some letters, I am using truncate method, but it does'nt seem to be working properly for me if I put anywhere in helper, where we are returning the crumbs. I am using it something like this truncate(name, :length=>10, :ommission=>'...'). Can you please provide me any solution.
You'll have to use a generated and seeded random from WorldGeneratorUtils to receive constant results. Right now, this creates stupid overlapping chunks and stuff.
this is terrible tree code
You are really putting a lot of thought in Livestatus, it is very professional.
Can you add some class documentation/comments to indicate the caching functions implementated (least frequency) and any additional details.
I know some of the other tests did this, but there's no reason we need to reverse this url and throw it in a variable if we're using it on the next line. also, `q=` is implied if there are no `GET` parameters.
за что мы так доки не любим? ;)
Hi Iliaal.

I guess you realized that this can never work, because you were trying to compare compressed length with uncompressed length. I made the same mistake and when I realized about it, I did the null char trick (though I know it breaks backwards compatibility). However, I've been working on using the 6pack to handle the fastlz data (as the original lib author does), not using files, but memory. This way there will be more consistency with the data, and the "memory reservation problem" would be solved. I haven't finished yet, but I'll let you know when I have it if you have interest. My worries are that this might not be as fast as using just fastlz, but I will do some tests.
I hope you don't mind if I use some of your code.

Jordi.
{active, once} にチャレンジしてみるべし
/^(?!Anonymous)$/
Yep!

You should probably

git rm .DS_Store *.pyc

-- no need to track binary files.  Also see my .gitignore e-mail...
This displays fine in Chrome but doesn't look right in Firefox.
is it important to rename this file to WebDriverInit.php ?   facebook module standards always put the require's inside an __init__ so this is just going to make it harder for me to maintain this repo with the facebook one.  could we leave this as __init__.php ?
http://www.youtube.com/watch?v=Tym0MObFpTI
Don't use C99 comments here, it needs to compile with `--ansi --std=c89 --pedantic`.
don't quite understand this one...prerequisites are already a read only attribute, you can enhance them, you can clear them, what do we need to have write access directly for?
This should throw an exception, not return true. "err on the side of caution" is dependent on how the result might be used.
Fixes the text issue in blurbs. Was also affection own works in collections.
Thanks, due to a number of issues with this function I simplified it to 09b39381ecff895d4b3d6f340390ae71af727f2a which also does what you suggested (just use -perm)
A fairly recent version of `node` (v8):

```
$ node --version
v0.6.7
$ node
> Infinity = 4
4
> Infinity
4
```

And I installed latest stable `node` just in case:

```
$ node --version
v0.6.12
$ node
> Infinity = 4
4
> Infinity
4 
```

And last time I checked v8 that behaviour existed, but you're right: it has now changed for me as well (using the latest Chrome beta build).
The spec says that with arrays the index "cannot be ref" -- 
<http://dlang.org/statement.html#ForeachStatement>.
So, InputRange shouldn't provide ref access to the index at all, right?
You don't need to specify the receiver here, and can remove the parens.
Thanks updated.
nice, fixed
先merge了再改吧
Maybe that the name "change_block_button" is not the best representative. A better name right now, does not come to me lol.

if change_block_button: is to maintain concordance with natural gtk set_sensitive function, if you put only one parameter everything in the toolbar will be changed in the same way.
Tar ännu ej hänsyn till bollens tjocklek
Thanks. I hadn't thought of that.

You might want to suggest it to jcw, as I used some of his code. https://github.com/jcw/ethercard/blob/master/examples/pachube/pachube.ino
Uups. "copy & paste"–ing gone bad.
I can recall some changes to libcec that required XBMC changes. You're sure it will build and actually work like intended with libcec-dev 1.1 ?
wizard_download_dir shall be check to make sure it make sense: if someone enters a value like /downloads, it will mess his NAS. This path value must reside on a share, synoshare can come handy to help do the work. (synoshare --enum and synoshare --get <sharename>).
Because at postuninst, the `deluser` is no more on the disk. Removed by DSM.
tu me kiff hein :D

Eze
Well, the value of the ID is in currval('whatever-is-the-sequence'), so it's referenceable, as long as the trigger/rule does so *after* the ship is inserted.
Yeah... I used R-6

Thank you.
Верни эту строку на место и пройди коан, заполнив __. Не меняй условия выполнения, заполняй требуемые пропущенные места.
A delete tromer :s
No worries, there are quite a few ways you can organize your configs and this update was needed because I had a project that already has an `assets` config which would cause issues. So it's only meant for the times when you need to pass in a config file other than the default `assets` one.
Corrected version which call the authenticate action. The previous commit was not calling this action.
> If `_check_build` is catching this, then maybe the further check is superfluous?  I have to admit I don't entirely understand why @amueller was seeing the problem he did...

I am not certain, but I think that a warning doesn't hurt :)
Instead of putting this on the prototype we can have it as a top-level function in this file (it'll be hid via closure at build-time anyways). This way if people inspect the `navigator.capture` object they won't see a juicy, undocumented `_capture` function.
this should be insert at index
why did you erase it?
![bummer dude](http://jamiedubs.com/screenshots/Waynes_World-Bohemian_Rhapsody_-_YouTube-20120221-220938.jpg)
WTF are you doing! You just demolished my board!
when I tried that I had some of the tets fails.  Well maybe we did not have the flatten previously...
Shouldn't this be `'subscribees'` which is the textarea field in the Mailman form, since `'subscribees_upload'` is the file upload field and you're just giving it a string with one email address instead of POSTing a text file with email addresses?
cool, i never knew about this .select()
Minor textile formatting change
More tabs & spaces
What do you mean by "state"? is that the same as node?  Also, lua does have a `break` statement, though as usual, I'd suggest that if you're linear searching, you should just be adding to your table with the name as the key as well, so it's just a table lookup.
I've submitted pull request #922 with a fix for this
How about Saurfang and Valithria?
from chest
Well you have right but i want to give the choice to disable it to allow all ips.
Anyways if you want to change it to array('127.0.0.1') its done :)
Note: this release will not work on the upcoming Rails 4.0 master:

http://intertwingly.net/projects/AWDwR4/checkdepot-193-40/section-26.1.html

The problem is that 'active_support/base64' is deprecated in Rails 3.2 and eliminated in Rails 4.0:

https://github.com/Shopify/active_merchant/blob/master/lib/active_merchant.rb#L33

I see a pull request with a potential fix:

https://github.com/Shopify/active_merchant/pull/245
I am currently in the process of trying to make the image upload crop feature to work. 
As trying to understand what is going wrong, it seems that 
if (false !== strpos($this->getMimeType(), 'image')) {
is wrong.
We could put instead
if (0 !== strpos($this->getMimeType(), 'image')) {
OR
if (false != strpos($this->getMimeType(), 'image')) {
I name unknown values quite randomly, it doesn't really matter. "Unk + type" is usually enough.

Btw, for auction structures check this https://gist.github.com/1579867
i have added some slides on QOM and the query builder to the PHPCR slides:
https://github.com/phpcr/phpcr/commit/88c322fe772b5e08f5ce79970b5869c839bf73ed
We can leave it as is for now - the visualization widget might not be used in Bio-Mixer anyways?
I am not sure if it is a good idea to change Graph while working on the SVG graph display, because it break compatibility to the Flash graph display.

Why was this change necessary?
Could be an OS specific issue. Im using Mac OSX Lion.
Oh sorry, I was a little clumsy, offending you was not my intention at all.
The way to indent is something very personal, and I often try to take the common habits of the projects.
I just take my own conventions when there is no common coding style in a project (even in th same file sometimes), by this is not the case of yours that uses the Redmine coding style it seems.
I'll take care of that I my future modifications. Sorry again.
Galder, applied the requested changes to both pulls
With my suggestions in the Location model you can just do this:

``` ruby
@location = Location.find(params[:id])

gon.locations = @location.delivery_list # assuming gon can handle a hash.

# or

gon.location_names       = @location.delivery_list.collect{|item| item[:name]}
gon.location_addresses = @location.delivery_list.collect{|item| item[:address]}
# If you need to do it this way I would have delivery_list build an array of Structs then you could do
# @location.delivery_list.collect(&:name)
```
Should be called just setAffinity() now.
Could remove the `else` by setting `doneTime.tv_sec` before the `usec` check.
Нулевая степень по определению единица. Единичный элемент зависит от операции. Можно добавить в функтор метод unity(), который будет возвращать единицу.
You are getting the value from the #property field, which is perfect, except your field's values aren't how they should be spelled in CSS.
I think you should disregard my previous comment. I got the error with 3.1.1 tag not with this commit.
and because a line should not go pass 80characters, I suggest to do the following:
moduleContributors << QString("Andrey Fedorov (SPL, BWH);
moduleContributors << QString("...");
moduleContributors << QString(...);
return moduleContributors
@cdogstu99, hey!
It's an addition to control the number of <thead> rows by 'thead_rows' setting. Actually nothing special until you really need it.
But why when leo IS an HDPI device ? Ace with a similar configuration (800x480 on 4.3') is also HDPI by default
it's in the path.  Note that the previous build attempt complained about it not existing and the current one doesn't complain.  Further, it's in the same directory as the other required .lib files, such as ws2_32.lib, etc.
Hey, what about using these values to overclock? They work realy fine in Franco kernel:

{ 1, 652800, ACPU_PLL_0, 4, 0, 217600, 2, 7, 200000 },
{ 1, 672000, ACPU_PLL_0, 4, 0, 224000, 2, 7, 200000 },
{ 1, 691200, ACPU_PLL_0, 4, 0, 230400, 2, 7, 200000 },
{ 1, 710400, ACPU_PLL_0, 4, 0, 236800, 2, 7, 200000 },
{ 1, 729600, ACPU_PLL_0, 4, 0, 243200, 2, 7, 200000 },
{ 1, 748800, ACPU_PLL_0, 4, 0, 249600, 2, 7, 200000 },
{ 1, 768000, ACPU_PLL_0, 4, 0, 256000, 2, 7, 200000 },
{ 1, 787200, ACPU_PLL_0, 4, 0, 262400, 2, 7, 200000 },
{ 1, 806400, ACPU_PLL_0, 4, 0, 268800, 2, 7, 200000 },
// { 1, 825600, ACPU_PLL_0, 4, 0, 275200, 2, 7, 200000 },
// { 1, 844800, ACPU_PLL_0, 4, 0, 281600, 2, 7, 200000 },
I am stupid like he'll... 
I have htc_ part and everything, but it still doesn't compile with that.. 

0002647302 <reply@reply.github.com> wrote:

>"Unlike backup tool, which uses the cm prefix to an existing lunch combo
>variable, this flag can apply to devices not currently supported by CM.
>The supported devices are manufactured by HTC, so instead of verifying
>a cm prefix, the htc_ component is used to trigger the append."
>
>^ it uses the htc_ part of your TARGET_PRODUCT choice to call it.
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/TwistedPlayground/android_build/commit/0eb05d656aca637193d036287671471e3d7e4dc6#commitcomment-912608
Hello, sorry if i have not answered immediately.
Thanks for the time in which you have developed and we hope for better time
Please, copy all language variables that you add in english in all language files. If not, it will be the mess for the other translator for see wich variables are missing in his language files.
Hello! I was thinking about replacing a plugin we use with the LHM for table migrations. I noticed the rubygems version is 0.2.1, and was just wondering if 1.0.3 isn't releasable yet for some reason? thx.
>  Are there any particular advantages you see for ReturnValueDescriptor?

Not a lot, It's more that having `isCascaded` and the related `ConstraintDescriptor`s of the return value on the `MethodDescriptor` surprised me a bit as awkward and not totally intuitive.

> Another question is how cross-parameter constraints fit into the picture.

Good question, but `MethodDescriptor` in this case is the clear host of that, just like `BeanDescriptor` host class-level constraints.
Habe mir "nur" den Quelltext und die Doku angeschaut.

Ich habe sehr starke Zweifel, dass es funktioniert. GTK und OSX 10.7 sind nicht die besten Freunde. Wenn ich mal Zeit habe werde ich das mal ausprobieren.

Am 31.01.2012 um 13:00 schrieb Matthaeus:

> Hast du das script eigentlich bei dir ausprobiert? Mich würde interessieren ob das so "ohne weiteres" auch auf MacOS geht
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/platzhaltr/easy.rapid.gtd-on-your-fs/commit/c053616e4728b7ce819fa753993a8b73941b6461#commitcomment-917963
But why not? If we move this line to the end of page this does not improve speed of page load.
dbc bug? maybe kernel bug?
Thank you for the link to Wikipedia. However, it should be placed in the README file itself.
Note: the second link is broken.
Yes can have a go. When you say "Shall we go for min replicates - middle of the road?" are we talking min 3 replicates?
Should this Makefile really get merged into develop branch?
You can probably switch to using the annotations for new tests.
You can drop "Since Akka 0.10, " that's ancient history :-)
I don't understand this line
Good point, fixed and pushed so that it is a fallback
So how about:

allChildren foreach {
  case a: InternalActorRef if a.getParent.path.address == address => system stop a
  case _ =>
}
wouldn't it be useful to include the `name` in the exception msg?
Yes, I thought that too, but it's not possible since the object field `actorCell` needs to be written to before the code in `init` and `start` is run to ensure proper visibility since we basically can start using it at any time after we send the `Supervise` message in `init`.
That's a really good question, and yes the value of lookup and underlying can change, but the critical change of underlying (swapCell) is protected by the the lock in the unstarted cell. And this test is only used from inside that lock.

Maybe there should be a comment that `cellIsBeingReplaced` and can only be used from inside the lock?
I assume that this is not what we're proposing our users use, right? Casting to internal API?
Note changed API for the Chunk::Foo.new calls.  Instead of the match, we supply the text of the whole match (first arg) and the class' range sliced from the groups of the whole RE.
va pour MozFR alors
Either move this down to be on line 278, or add a comment here saying where this will be used, so that we don't forget down the line.
We've got full CRUD on this action now. No need to limit.
thx √
This is because I already merged the smartname fix here.
Probably a one byte string is most efficient in db-space, same type as the types, and could be extended if needed.
Nice, I immediately understand what this object represents.
Already in assets.yml

    - public/javascripts/Imits/widget/Grid.js
    - public/javascripts/Imits/widget/SimpleNumberField.js
    - public/javascripts/Imits/widget/SimpleCheckbox.js
    - public/javascripts/Imits/widget/SimpleCombo.js
    - public/javascripts/Imits/widget/QCCombo.js
    - public/extjs/examples/ux/CheckColumn.js
but doesnt it [check](https://github.com/sam/harbor/blob/master/lib/harbor/application.rb#L1) if its using java and require the proper rack version?
Progress bar added to the message text.
it should not work any longer (i.e. return `None`), since `akka://` is only valid for the local transport from now on, right? Hence giving host:port is just wrong; the translation would the probably be to look for `Address("tcp.akka", "s", "host", 1)`, I think.
please check that the loader is instance of  Twig_AdvancedLoaderInterface
it is now set to TRUE by default in red/base/Class.js
Can you please add braces around this `if`?
typo, isVtrash should be isVtTash (capital T)
We actually don't do encoding arguments in luvit either. Can you remove the 'utf8' arg?
let's remove the X:MEMO blurbs, but keep the comments
Pas besoin de le changer, mais pour ton information tu n'as pas à mettre les parenthèses.
Utilise la route au lieu d'un path hard coded.
Why isn't this ``setSuccess(Channel channel)``?
@hjmjohnson are you ok with this change?
username, password should not have mandatory
Align this line with the previous one.
Hi,

did you push that sync because I can't see any commits about that ) .. or do I miss something?
оно точно не case-sensitive? в других кусеттингах вроде QReal пишется, лучше поправить
лучше запихать моторы в вектор и заменить три условия на цикл
Oh my!   The poor man's single-threaded lazy-val code...
Yes, good point. NonMajor is ok, but then by itself "NonMajor" doesn't sound like it means anything having to do with exceptions. I agree that it might be a good idea to make such a distinction between NonFatal and NonMajor (or whatever we call it) so sure-- maybe we give it more thought and I make another pull with whatever we decide?
I use vim. I haven't enabled spell check, sorry about lots of typos, spelling mistakes. Will fix them.
I think _merge is used only to calculate inversions and it doesn't add any other functionality. So it is not a good idea to move it to iterables.py.
Yes, that is correct. This functionality is primarily geared towards mechanical systems. The idea is to use the energy functions to determine the Lagrangian of a mechanical system and finally use Lagrange's method to derive the equations of motion for any mechanical system.
But the new behavior implemented here is simply wrong. This completely breaks substitutions! I am not against permitting differentiation like this, however the either `Subs` should be preserved, or `.subs()` should create it.
You might want to create OuterProduct's here. Just enclosing ``(state * Dagger(state))`` in parentheses will automatically take care of that.
My point was that I don't think 1/cot(x) should ever "autosimplify" to 
tan(x).

On 28.03.2012 08:01, Alexey U. Gudchenko wrote:
>> +    assert hsimp(sinh(x)**2/cosh(x)**2) == tanh(x)**2
>> +    assert hsimp(sinh(x)**3/cosh(x)**3) == tanh(x)**3
>> +    assert hsimp(sinh(x)**10/cosh(x)**10) == tanh(x)**10
>> +    assert hsimp(cosh(x)**3/sinh(x)**3) == 1/tanh(x)**3
>> +
>> +    assert hsimp(cosh(x)/sinh(x)) == 1/tanh(x)
>> +    assert hsimp(cosh(x)**2/sinh(x)**2) == 1/tanh(x)**2
>> +    assert hsimp(cosh(x)**10/sinh(x)**10) == 1/tanh(x)**10
>> +
>> +    assert hsimp(x*cosh(x)*tanh(x)) == x*sinh(x)
>> +    assert hsimp(-sinh(x) + cosh(x)*tanh(x)) == 0
>> +
>> +@XFAIL
>> +def test_tan_cot():
>> +    x = Symbol('x')
>> +    assert tan(x) == 1/cot(x)
>
> Yes. The only note that it is true only  for nonzero vallues, but as we adhere to the cancellation of `(x-1)/(x-1)` in silence, then this is the same case:
>
> ```
>>>> cos(x)/sin(x) * sin(x)/cos(x)
> 1
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/948/files#r613667
>
When «the change in formalism» occurs, `FlatRefraction` really doesn't change the ray or beam parameters, but the propagation in homogeneous space is determined by its index of refraction. Thus, suppose the situation with flat plate (e.g. n=1.5, d - thickness) in air. In the case of unimodular matrices, this system is described by two unity matrices, and one matrix describing propagation on distance d with refr. ind. n. In case of previous formalism, you have 2 matrices describing plate's interfaces (the depend on n) and 1 for free space propagation without account of refr. ind. This reasoning completely works both with geometric rays and Gaussian beams. Besides, the unimodular formalism is better for understanding in case of Gaussian beam. Here the curvature radius isn't changed when passing through the interfaces, but it starts to change when propagating through the space, and its diffraction simply depends on refractive index.
But, as you see, these two formalisms are equivalent (or similar), but the unimodular one has some advantages. The main one is the equality of matrix determinant to 1, which provides the possibility to check and control the correctness and accuracy of calculations in case of large optical systems. Besides, you should possibly saw the description of resonators, their stability etc. They are usually performed with the help of unimodular formalism and gaussian beams.
I think that is best.
OK, thank you.  Fixed for all section titles in `diagram_drawing.py`.
Some duplication here between how we're setting up the boundaries for the multipart form post and how we do it for feedback screen caps. Not urgent but going forward we should consolidate this into a generic method to handle binary uploads.
I am just curious on why subcomandante had those function declarations commented out, subcomandante, can you comment on that?
This could really use a test.
This page is for new features in cakephp 1.3 which still supports php 4, so the visibility keywords shouldn't be changed.
Brauchts das hier? Der Owner ist ja bereits im Camp selber eingetragen...
Yeah... this should be:

```ruby
gsub_file 'config/initializers/devise.rb', '# config.scoped_views = false', 'config.scoped_views = true'
```
will be done
That's the way I was going to go, but Oracle has a 30-character limit!
I'm going to rephrase what I said. There should be no url manipulation on the client side. Generate all urls on the server side and switch between them on the client side. To the client side urls should be opaque.
$tables?
You need to issue the same request before you create the user and assert that 'user' is not present.

'user' is a very poor choice for matching. Use something unique, say 'test-user'.
This is not a problem. This is available since PHP 5.2.
Swap arguments of =

t.topic_poster should be on the left side
So should I change all variables from subject to title?
Don't call it "Extensions Admin", we use "Administration" everywhere else. And this I would just call "Extensions Management"
Changed in ccdde5b15af868c32d655fbee2146ebee7c2b69e and made it more generic.
this was fixed in a different way in your previous pull request
I'm not happy about it either.  It's likely that jslint will have something (useful? good?) to say once this is delinted.
} catch (err) {
As a matter of style I like to keep semantics between null and undefined clear. Passing null here means arguments.length == 1 on the other side. That's different than allowing it to remain 'undefined'.
instead of having this 703 factor, why not just convert height_total_inches into height_meters.

Then you can do the calculation in the metric system?
BMI = (weight (kg) / [height (m)]2).
put {} around the if statement's executed code, so it's clear that this return only happens as part of the if statement.
avoid committing files that have mistaken spacing changes
OK - I think I've answered this by looking at the changes to test_regrid.py in #235.
You should call `callback(fn)` if no item as well. You would want the css loading to continue to the next item, if there is one.

Another fix would be to do a filter on `cssArr` to get rid of all falsy values.
isn't model::mod_time supposed to be part of the memKey also?
i deleted other parts and forgot do delete mgr there
Проверка на `techTargets[d]` — лишняя. Депенды у конкретной ноды могут быть про любую часть графа. А так получается, что часть депендов не добавится, хоть нода про них и сообщила.
I was trying to come up with a clean way of doing that, but it's fragile if anyone wants to add another option, or if an option that doesn't take an arg is added, then some offsets will be even, some odd. I wasn't happy with the result entirely either, and was thinking to maybe pull the strings into separate variables.

And then I thought I'd see what kind of feedback I got anyway.
Shouldn't this line be also under the "if" below it?
We've been discussing this on the note above, too - I don't want to use deepcopy, and I think modifying models so that auto_register = False means they don't touch anything at all would work well enough.
this is wrong. ``$startPos`` is either an integer or ``false`` when not used
Changed the loop function name to avoid confessions with the con.loop.
What about a test for <code>document.documentMode === 9</code>? it would also work.
interesting question. I'm not sure really. Leave this one as is I guess.
s/expectes/expects/
You can always pipe --version or --help through cat to make the status code 0.
We have a `which` helper method now (in utils.rb) that returns either nil or a pathname.
use pylab as pl
Oops, totally misread that one.  Fixed now.
Strange, the :ref:`criteria` bit causes Sphinx to complain about an undefined label, and if I grep the doc files for the string "criteria", nothing is returned. Might it be something else? Or is that from the main Pulp docs, perhaps?
This is not correct since the first call to `socket.on()` will remove the decorated method (`socket.on = on` overwrites the decorated method)

A correct implementation would be:

```javascript
var on = socket.on.bind(socket); // bind the on method to the socket
socket.on = function (event, callback) {
    on(event, function () {
        // your logic here
    });
};
```
Why would you want the developer the decide what routes need session?
Da du hier die Shortcutfunktion `$this->i18n()` des Addons verwendest, kannst du den Addonnamen ("debug_") vorne weglassen, also einfach `$this->i18n('log_activate')`.
Fixed in 851fb1eb7829cf210d5b09cc47c6e62cd58a8fc8
I note that this would only be called if BUILD_EXAMPLES is enabled.

How do you feel about an INSTALL_EXAMPLES target instead?
This doesn't seem to work. I believe this method requires an user_id or screen_name otherwise Twitter sends us
a "page not found" error.
Might need to add a comment that this is a legacy route
Why not just add the query object as an attribute of options where you create options right above?
It't notification, not warning.
I had a think about it and I think swapping to ConcurrentSkipListSet + ConcurrentHashMap (as a set) might be the way to go.

This would allow removal of snapshots if another dirty one arrived later.

The process would be something like

- Add to HashSet
- If there was an old value, remove it from SkipListSet
- Add to SkipList Set

The SkipListSet would use the distance value in the model to sort by
Nope. You're right. Needed it at one point, but forgot to take it out.
Seems like the folder should have both language and country (e.g. fr-FR) so that we can provide multiple translations for a given language such as en-US versus en-GB.
Extra space before "Quit"
I think it was "px" that had me confused. I think it should just say "fixed height". There are many ways to specify a fixed length other than px such as pt, cm, in, pica, etc..
Can we just remove this obsoleted test?  (And the JSLint declaration for 'xit')
Is there any case where lastSelectedLine would be undefined?  It seems like that can only happen if the line where the cursor/selection lies doesn't exist...
if you're not using it, remove it.
Consider an else block here that sets the sub path to the string in $parsingInfo['sub_path'].  This would allow you to route the URL somewhere physically different on disk if you wanted, and makes the behavior consistent with how the controller, action, and action_params arguments are processed.  Like so:

	} else {
		// Use the constant value
		$request->setControllerSubPath($parsingInfo['sub_path']);
	}
The difference in line 9 and line 19 is `nav-stacked`
Also, only one of these divs is visible at a time.  Bootstrap media queries auto-hide and reveal the divs responsively.
good point - the magic string is to ensure a consistent template for the hqcases_<md5> index name is created. putting in say, hashtag #defaultfoobar or something might make it not possible to collide with user input.  should make it an enum instead of magic stringing it everywhere.
yeah i don't think this is something we need to worry about until proven otherwise
the contract is such that if there's a None return, it means do nothing. the original intent was to have it such that you could do whatever the heck you want on tranform -> transport (as in the case for say, 1 doc emits 10 things to index vs 1 doc to index) - hence the desire to have a return value of what's been transformed, the reality doesn't make it easy for elegant code for that use case, so I'm fine with changing the contract to be "in place mod"
It's cool at this point.

But I use same expression as https://github.com/fluent/fluentd/blob/master/lib/fluent/command/cat.rb#L73 and https://github.com/fluent/fluentd/blob/master/lib/fluent/command/fluentd.rb#L104. I think it is good to use same expression to do same thing.
This is taken care by the method getQuestions() in the surveyPage object.
Is this code tested ? 

It seems to me that me.model does not exist (but we have me.getModel()) 
Moreover, there's no addFields method in a model instance.
Please add the missing parameter in the javadoc.
Should this be inheriting `rb_eException`?
Did we resolve that this needs to be in a setTimeout? I can't remember. I'd prefer it not to be, but... if we must, we must.
"the username to use when connecting to the database"
I'll stop now, but you get the point.
Random double slash?
This is specialized algorithm is addressed only this specific problem. The algorithm in the image.js does not take in account pixels "in between" steps of copy operation, this does. Which gives you this smoothing effect. Also, I don't this we have to use this algorithm in some other place.
Also, it's a little inconsistent to get these elements by id yet you pass in the #loadingBar into constructor.
Nice catch! Fixing right now.

---
Ryan Pitts
@ryanpitts
http://www.ryanpitts.com


On Thursday, September 27, 2012 at 7:36 AM, Ross Bruniges wrote:

> In source/people/templates/people/organization_detail.html:
> > @@ -11,7 +11,7 @@ <h1 class="maintopic"><a href="{{ url('organization_list') }}">Organizations</a> > <h2>{{ organization.name }}</h2> > > {% if organization.location_string_for_static_map %} > - <a href="https://maps.google.com/maps?q={{ organization.location_string_for_static_map }}"><img src="http://maps.googleapis.com/maps/api/staticmap?sensor=false&scale=2&zoom=13&maptype=terrain&size=350x250&markers=size:small|color:AD3121|{{ organization.location_string_for_static_map }}" class="map-inset-right"></a> > + <a href="https://maps.google.com/maps?q={{ organization.location_string_for_static_map }}"><img src="http://maps.googleapis.com/maps/api/staticmap?sensor=false&scale=2&zoom=13&maptype=terrain&size=300x250&markers=size:small|color:AD3121|{{ organization.location_string_for_static_map }}" class="map-inset-right"></a>  
> Mixed http/https content!
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/mozilla/source/pull/63/files#r1707537).  
>  
>  
>
This is missing `attributions`.
You probably also need a case for islink here.
Also I think we need this to use `:page => params[:page]` so we can page through search results.

And it would be nicer if the per_page value of '25' was a constant (perhaps returned by a helper function), since we're using the same value in lots of different places - that means we can change it in one place and we know it's always the same.  Or we might want to support a per_page value in the request parameters (but we'd still need a fixed default).
Yes, it was copied from web_steps
I would change 'started up' to 'started'
这个改了部分
有定义FALLBACK_ON这个宏，没必要用非的形式：
&& (ctx->ss_srv->flag & NGX_HTTP_SESSION_STICKY_FALLBACK_ON))
Talked to Johan IRL about this. The python server is just nice to have if you're doing JS/CSS development since the resources get packaged in the jar and won't be refreshed without a full compile; it's not necessary to run the Scala UI.
Can this use the preprocessing util?
fixed this to use .voided for now, I'll switch it when the trait changes.
why not ask for SOA?
One additional note - try your patch with full installation, e.g. with SELinux turned on. I am pretty sure SELinux will block you from spawning process, it may even block you from reading /proc/version. You may have to add a selinux rule for this.

To do this provision a new nightly installation, build the katello RPM with your patch, update it, restart and try it. Check SELinux audit log for denials.
Just curious, why do we need that?
You can't do ``{}`` in Python 2.6, you have to explicitly do ``{0:s}`` and ``{1:s}`` - that's why Travis is failing.
Ok, I've fixed this. I've renamed ``final_data`` to ``result``, and the return value to ``result_error``. I'm using ``result`` to refer to the ``NDData`` object rather than ``result_data`` because the latter could be confused with ``result.data``
Sphinx… slowly driving me insane.  Fixed.
On 26-Sep-2012, at 8:54 AM, nhmc <notifications@github.com<mailto:notifications@github.com>> wrote:


In astropy/cosmology/core.py:

>          """ Initializer.
>
>          Parameters
>          ----------
> -        H0: (float)
> +        H0: float


I think there needs to be a space either side of the ':' for the Parameters and Returns sections, otherwise the docs don't build properly. (Annoying, I know.)

—
Reply to this email directly or view it on GitHub<https://github.com/astropy/astropy/pull/365/files#r1696185>.
I mean this specifically for the sub-classes.  That is, with the syntax I'm suggesting, I could write sub-classes more easily:

```
class MyPercVarError(NDError):
    def propogate_add(lefterror, righterror, leftvalue, rightvalue, resultvalue, operanddata):
        return (lefterror + righterror)/resultvalue

```

as opposed to the current system, which requires this:

```
class MyVarError(NDError):
    def propogate_add(self, operand, result_data):
        lefterror = self.parent.error.array
        righterror = other_nddata.error.array
        result_array = result_data.array

        return (lefterror + righterror)/resultvalue

```

So the first version is much easier for someone to just write a simple propogation subclass without knowing much at all about the internals of NDError.  This may be good or bad, depending on your perspective :)

I see your point about duplicating information, though - I'm not attached to this idea, I just wanted to offer it as a possibility.