labofsahil/github-event-dataset-2012 huggingface dataset:
added  option for rabbitmq backend

In the current implementation when an exception occurs during message processing, the rabbitmq backend does not send back any response to the broker. 

If you have set the consumer to more than one iteration, this means that the message will be broadcasted to the consumer not until the end of the iteration cycle has been reached. 

This PR allows to configure a rabbitmq queue to send back a  `basic_recover` to the broker, which will make the message be re-broadcasted immediately.
First blog update.
".changes" file entry now with correct translation of the word "created"
LPS-25605

LPS-25605 Adding an Organization with an admin with limited permissions may cause the admin to not be allowed to edit the org after creation
LPS-22706
Pull my readme

please
LPS-25516 Incomplete fix from LPS-22593
seems we never exposed the response content length field. Fix that.
Development up to date
check mechanism updated
Ejercicios StaticSetOfIntsExercises

Creacion de clase StaticSetOfInts, y de StaticSetOfIntsExercises con sus pruebas. Se hizo ejercicio 1.4.11, es el del encontrar la cantidad de elementos repetidos dentro de un array, el algoritmo es de O(Log2n) ooouuuuch! ;)
Latest style/template updates

Everything significant
Ok/Cancel buttons => Yes/No #50

this convert Ok/Cancel buttons to Yes/No on confirm dialog #50
Added real help texts to ideas
added 'PLS_Route' to single.php's loop-meta template call
Tidy ups and added a new 'copy result to clipboard' prefs

A twitter user gave me a very good idea to add an option to copy the calculator result to the clipboard.
There's now a pref in the 'calculator' prefs for this.

I've also tidied up the prefs (more information, moved things about and made the font bigger)

Does anybody have any idea why an `extern NSString *CalculatorResultPref` would have been used to just mean `@"Calculator Result Pref"`?
I changed it to a compiler const as I thought it was a bit pointless.
penndot

added dots for penndot cams
修改语句

不好意思啊，用git不久，不是很熟练，那个2.5实在不知该如何处理……
Added DFL-3588 test
Modeler functionality
[JENKINS-12373] JNDI LDAP properties affected by System properties.

Hi,

I implemented https://issues.jenkins-ci.org/browse/JENKINS-12373.
Maybe you wouldn't agree with the way I did it since it is hardcoded which System properties are used for JNDI LDAP properties.

Best regards,
Stepan Vavra
third times a charm :P
delete macos hidden files

more clean code!
Fixing broken invoke functional test

Reviewed By: Nukul Bhasin <nbhasin@rim.com>
Tested By: Tracy Li <tli@rim.com>
Fixing windows build

The Windows build was failing on next because it could not find the QtCore library.  The library path was added to configure-qsk but not picked up because the parameters passed into cmake (through cmd.exe) were not between quotes during execution.  This change fixes this by escaping the quotation marks in configure-qsk.
Prevent the auto deferring of networking and JavaScript when using run_when_backgrounded and/or the blackberry.push feature.

By default, webplatform will gradually slow down JavaScript and networking for apps running in the background.
This change prevents that if you’ve specified run_when_backgrounded in config.xml or <feature id="blackberry.push" /> in config.xml.

See issue https://github.com/blackberry/BB10-WebWorks-Framework/issues/206.
Documentation update request: http://github.rim.net/Microsites/HTML5-WebWorks/issues/282
Example for Jira issue

Demo of

https://jira.springsource.org/browse/SWF-1516
Allow specifying the serializer class in the render options (+ other tiny fix)

Also added attr_reader :options to the array serializer (for subclasses)
Modifications to manifesto

Modified section covering charging to attend events. Added new section on publishing list of attendees.

Added Mark Headd to list of supporters.
show-existing-tag-values-16

paparent:**approve**

This branch shows existing tag values in the popup. It also displays a notification window when the user visits a URL that they have tags on. Lots of code added here, sorry!  I gave in and opted to use jquery in the notification window, because I wanted to display ratings as stars and we do that using a jquery plugin. I don't think it's a big deal since jquery is packaged with the extension so no network call is needed to load it.

Fixes #16
More ICS overlays and such.
Oh, valid CCR's

Cleaned up the ccr exporter to produce valid ccr files.  Also added hook to allow the tests to validate ccr documents against the ccr schema.  This will need to be made available via an environment variable seeing how we cannot push the schema out due to licensing restrictions.
cleanup

Updated to 2.0, did some cleanup
Kata Ruby Sensei
Typo fix, plus couple of improvements.
Fixes made for the distributed-process with strict bytestrings
pull request

Added clicking to set new measurement position
added trace of previous predictions to view
CDMA addon

Would of done this earlier, but didn't understand that you moved it already.
README update

Updated the readme.
MINOR: Add two public static variables for switching  $show_status and $fb_id_only

Corrected previous pull request, not including the modified lang files.

Sorry for the inconvenience!
Support Thunderbird 11, fix translations and add Spanish one
Correction du 03/07

Corrections apportées.
Update to document (Reqs and Section 2)
Ch08

Traduction du chapitre 8 à réintégrer
As long as it's documented I don't think it should matter that much - my guess is 99% of the time you want a fallback of zero anyway.  I'd be tempted just to keep it simple and assume nothing after the number at all in order to be valid.
Since this change, there is a small bug now with DVD navigator-forced aspect ratio. 

It can be fixed by setting hint.forced_aspect value in CDVDPlayer::OpenVideoStream for streams of type DVDSTREAM_TYPE_DVD.:

In the if block when aspect != 0.0 you have to change this as follows:
  if( m_pInputStream && m_pInputStream->IsStreamType(DVDSTREAM_TYPE_DVD) )
  {
    /* set aspect ratio as requested by navigator for dvd's */
    float aspect = static_cast<CDVDInputStreamNavigator*>(m_pInputStream)->GetVideoAspectRatio();
    if(aspect != 0.0)
    {
      hint.aspect = aspect;
      hint.forced_aspect = aspect; // <------------------------- !!!!!
    }
what about ppc?
hehehe! thanks.
yes, good catch, will fix
I don't know what the base behavior is here, so I'm not sure.  Your original code had the base call first.  I'm happy to go with whatever you recommend here.  :)
I was going to send a pull request for the spelling mistake...
I personaly prefer:
`<h1><%= t(:editing_product) %> &ldquo;<%= @product.name %>&rdquo;</h1>`

it prevents all escaping problems
Ok, I will split of the two bugfixes into separate branches, and git cherry-pick other commits but this one.
Also note that in Python 3, you don't want to encode here, because `__repr__` should return (unicode) text.
That's not correct, is it?

automaticallyForwardAppearanceAndRotationMethodsToChildViewControllers returns YES if the OS automatically forwards. The method checks if we must do it ourselves. This change turns that around.
If you were going to store just an index of the item in the database then sure, though

```ruby
arr=%w(bronze silver gold platinum)
arr.map { |m| p [m, arr.index(m)+1] }
```

would probably be a bit cleaner. Using an integer to store the position within the array isn't an idea I'd favour for two reasons

  1. harder to see from looking directly at the database what is going on
  2. you need to make sure that you *never* change the order of options as you'll break the model
  3. with a database index on a small set of strings SELECT queries should be fast enough
sorry for very long Duration time.
zf2 don't recommend run tests All components at once.
(so I run each component tests as above)
now, I don't have good idea to reduce..
затупил :)
This is fine, of course.
Figured the use case for inheritance here was fairly limited and most things could be implemented by overriding the setter instead. Wasn't sure it merited the extra method call. Happy to change though, should I update kiall's references to _follow and _follow_headers to their respective getters while I'm at it?
I would love to help him testing it and finishing it, but it just appeared here in repo and I was surpised. But still I don't even want Neo_mat to be victim of your must-make-it-public. Everybody can make mistakes, but it's better to let him know, help him understand what he did wrong and let him correct it. This way you only disgust all people, who is trying to help.
This is looking very nice.
Thank you

but I didn't have enough time to do this

I did not use git

I downloaded neo4j-spatial-0.7-7-g01b5488.zip

and in m2eclipse add maven project

my OS is win xp
Someday there should be a permissions mechanism which would allow several people to collaborate on the schedule.
This was bad because here, the builder is already set to insert instructions into the current block (LlvmBlock)
Thanks rexeisen. I wasn't sure about including NSLocalizedString in the library itself as it complicates integration, but using NSLocalizedStringFromTable makes sense as you can keep the iRate strings separate. I'll include these in the next release. Thanks!
Yeah, that wasn't clear, but good to know
Awesome, is this headed for mainline?
nothing fixing per se but less code and duplication otherwise everything is functionally equivalent
Looks like copy/paste from Trinitycore without gving credit?
https://github.com/TrinityCore/TrinityCore/blob/master/src/server/scripts/Spells/spell_generic.cpp
yes but i don't now why isn't working :S
The last before the storm
Ich bekomm hier nen segfault
I give up on this, I'll revert that.
I think something even better would be:

wheels = [child for child in children if "wheel" in child.name.lower()]

--

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
William Woodall
Graduate Software Engineering
Auburn University
w@auburn.edu
wjwwood@gmail.com
williamjwoodall.com
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~



On Fri, Feb 10, 2012 at 11:08 AM, gilecheverria <
reply@reply.github.com
> wrote:

> I don't see the problem you mention about dragging the ground. Although
> when I first give a motion speed, the robot seems to accelerate too fast
> and stands on its rear wheels. Only the first time it moves, though.
> Let me know when you have the .blend files ready to put everything in the
> main branch.
> Cheers!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/davidhodo/morse/commit/53c25db340d5609538a34c37f6be527c6156ce13#commitcomment-954429
>
*But I have been used to Doxygen...
*
On Tue, Feb 14, 2012 at 4:34 PM, crab2313 <
reply@reply.github.com
> wrote:

> 这类GTK项目基本都是用gtk-doc的阿，
> 像gtk，glib，eog，gedit，
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kernelhcy/gtkqq/commit/365416c8877f10368cd9b237ea7e3371923b5cdf#commitcomment-963651
>



-- 

**
*

Best Regards,
**

Xiang Wang| Engineer of Linux

Gtalk: wxjeacen AT gmail DOT com <wxjeacen@gmail.com>

*
I found possible crashes.

[mr1611] oPVP have some crashes but http://pastebin.com/9ivRJRri crash is related to vehicles. Possible sources:

https://github.com/mangosR2/mangos/commit/6d13c3d0cc1294a64f26c9b652386d4e22eb5d8e
cmon, be more serious. 
@tibbi wrot:
"so you think its some script stucking in like Dragonblight.cpp, or other zone ?"

@rsa say something useful, so we can try :)
Meh I was drunk as fuck when I pushed that! 

Emailing from my iPhone 4S like a BOSS!

On 4 Feb 2012, at 15:40, Adam Fairholm<reply@reply.github.com> wrote:

> Hey Phil,
> 
> In my local install, adding the BASE_URI is doubling up on URI elements for sites that are in sub folders because link_tag() is adding the sub folder path. Before I push to the repo though, just wanted to check if that BASE_URI constant was serving a specific purpose that I'd be breaking by removing it.
> 
> Adam
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/pyrocms/pyrocms/commit/c400593235d71c81cdc64cb3c17e2707b49bc25f#commitcomment-933682
sir i read the article . wanted to ask if it is ciphertext only attack or known plain text attack ? . once i have the captured frames how to input them to the code ?
Overload File class and substitute the file name on the fly based on the game settings. Shouldn't be that hard.
The two lines above can come out.. they don't render outside of a block apparently.
Rendering an action with an object in attributes from Twig is not working anymore (http://symfony.com/doc/2.0/book/templating.html#embedding-controllers).

     {% render "AcmeArticleBundle:Article:recentArticles" with {'category': category} %}

If it is a normal result, what is the best way to call an action with an object in attributes?

Thanks and regards,
Whitespace padding around the array index
Thanks for your reply, it is a PATH issue indeed. Mac OSX Lion includes Postgresql in the system now by default. If you do a ```which psql``` you'll see ```usr/bin/psql``` instead of ```usr/local/bin/psql``` which is HomeBrew's correct one.

I fixed this by editing ```/etc/paths``` and putting ```/usr/local/bin``` to the head of the file. Now the pgrouting formula works!
Ok, clear :).
Better to use /usr/bin/autotest_db_babysitter...

Now I think /usr/bin/autotest_db_babystter can be very well renamed to autotest-monitor-watcher...
I think your Standard clarification above, and then stating Most clinics will want to use Standard OpeneMR is good enough. A more thorough explanation, such as contacting Rod can be placed on the wiki:
http://www.open-emr.org/wiki/index.php/Administration_Globals
Наверное "предложение"?
Please reformat this code. Write each key/value pair in separate line, `{` one line before.
Well @clebergnu, this looks good to me, what about you?
Hi John,
please look at issue #38, the removal was due to request/answer in that issue.

Also issue #37 is also interesting to look at to fully understand the change.


Diego


On Fri, Jan 13, 2012 at 9:01 AM, John-David Dalton
<reply@reply.github.com>
wrote:
> why isn't this being used?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dperini/nwmatcher/commit/74f1665efdeec3f8401cb999ee843178ed574e2c#commitcomment-863770
Dig it. This is exactly what I was trying to push on you. Nice.
And "a" is not always translated to the number in Chinese.

Ref:中文的常態與變態:
http://blog.yam.com/soliloque/article/4938270
What about proxies? X_FORWORDED_FOR
I like how you made the logo float here
Interesting, it's more practical than trying to remove the system wide grubby...
@credomane we believe so. It will require more testing, but the issue you're refering to is Multiverse/Multiverse-Core#122 which should be solved by this commit.
Hi

On Tue, Jan 17, 2012 at 3:42 PM, Antono Vasiljev <
reply@reply.github.com
> wrote:

> You can simplify stuff by adding AM_INIT_AUTOMAKE([foreign]). This way
> developer should not keep all this 90s ChangeLog AUTHORS and other obsolete
> GNU standart files.
>
> Personally i prefere README.md over README :)
>

Don't you mind to send a pull request ;)  ?
merde - Do not advance the iterator after the false value return. Because 'm_iterator' may be needed to examine the exit position or value that caused the break in the middle of the loop !!
Nitpickin here but line up the then statements.
Can we rename this to CodeFluxRobolectricTestRunner
no parens, you can use `sort!` here
There is a minor type line 193 with a period between items and manualjournals instead of a comma:
$valid_post_methods = array('contacts', 'creditnotes', 'invoices', 'items' . 'manualjournals');
should probably be
$valid_post_methods = array('contacts', 'creditnotes', 'invoices', 'items', 'manualjournals');
Where do the options come from?  I didn't know we could do this and I need to add this to the documentation...
Thanks Andrew. I find git to be horribly counter-intuitive. I can use it and kinda make it go, I just never know what the 'right' thing to do is. I'll move my work to a branch.

I'll look at that testing doc. So far things 'work' in that my own models with both cylinder(r=) and cylinder(ir=) seem to produce the right output.
我觉得 private_token 可以跑个脚本给没有生成的帐号都生成上，就不用留一个 if 逻辑了
A lot of nodeunit reporters - maybe all - use utils.betterErrors. This function works fine as long as the assertion error has values for expected and actual. If these values do not exist then the we a message such as "AssertionError: undefined undefined undefined" - this happens for example with the assert.throws() assertion. The purpose of this commit it to resolve this issue so that betterErrors provides what the assertion was trying to communicate. As it happens this change also helps with richer assertion libraries such as should.js, which have many assertions that do not include expected or actual.
cfquery can contain cfif's/sets etc (well pretty much any cf code)  SQL comments won't prevent that code from being executed.
uint64_t preamble_sec() const { return d_preamble_sec; }
https://github.com/pculture/vidscraper/blob/develop/vidscraper/tests/data/youtube/feed.atom
@wenzowski you are very welcome :-)
This is a proof of concept at this point, but yes that's what it does. We're discussing how best to offer it, and whether it'll be included in a release or offered as an external option you can include.


On Mar 7, 2012, at 11:30 AM, Robby Dermody wrote:

> So this means that browsers like iOS4 and Android 1.6 will get the "floating" toolbars that jQM 1.0 had then? Awesome! (The static positioning can be confusing for users if the mobile page is long.) Is this going into jQM 1.1 or 1.2?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jquery/jquery-mobile/commit/801e4aa6eed135aeacef7938cba274a3e46f2089#commitcomment-1051858
The new parameters were good enough on my machine. Like I said before, it won't work for everyone. So current recommendations on 1.5, is that you only enable middle-click. I am still investigating how to tune moveThreshold and minClickTimeout, so enabling touch-over-click or move-detection will experience some problems.
this is a style that both misko and I use everywhere in angular if a line with ternary operator needs to be wrapped.
Obviously going to make these command line options.
the reference to the overriding application.conf should surely honour the context-free spirit of the remainder of the invocation.

ie. ```-Dconfig.file=./application.conf``` should be ```-Dconfig.file=`dirname $0`/application.conf```
Oh .. sorry :)
I think thats a good idea I will try to refactor to reflect this.
A advice:

Put on .gitignore your files .idea/*

And make a .sql file with all the databases is totally useless. Kloxo has a function for create the databases dinamically from a simple file with the tables and columns.
ну, я лично предпочитаю camelCase, но важнее конечно, если будет выработан какой-то единый стиль.
P.S. какие-то проблемы с кодировкой
kadootje is foutieve spelling, het moet zijn "cadeautje"
I fixed it by using the original code for the array adapter to make a customers array adapter with a customized filter because when I made each listview item have an image and a text the default array adapter went crazy
I personally think that if new-style is required everywhere we should just test for new-style in all cases.
Probably better to catch the specific extension thrown.
Can this be sent upstream?
> warning: comparison between signed and unsigned integer expressions
Hi Dan, awesome to have you checking as well:)
Very good point.
I think the idea was that txComplete needs to be read in the synchronization to ensure we have the update value when we're in doubt - but indeed if it's true already I guess we can play unsafe as the value being true is by itself evidence that we already have read the last value.
I'm going to measure this combined with discussion at c65461e2f8fbdedf7dd92c950c1f4ce9a583b922
Yay, we can has proper UI and game modes soon! :D
seriously, two days to figure this out
It changed must be, my young padavan.
Sorry I didn't see this until after I had already pushed my version which added `er/mark-LaTeX-math`: 913176d3d60e6ea8a1eec08203bc87b9955e655e  Let me know what you think and how you want to merge the two.  

Also you shouldn't need to call `er--setup` anymore.
Hi there,
i'm running in this error 
Mar  8 18:03:02 localhost kernel: [    4.841437] input: so340010_kbd as /devices/virtual/input/input1
Mar  8 18:03:02 localhost kernel: [    5.230908] so340010_kbd: probe of 2-002c failed with error -1

is there an option to build to make this work? or it's simply broken? 
i'd bet they worked on first boot with this kernel.. i'm running 12.04 and have added --no-log to kern cmd line.. could be related?
Thanks Mambasoft, while I proceed further I realize how much work you did on this
Kind regards
Teambuilder/main.cpp uses #ifdef WIN32

Should use the same everywhere
for cm9 all is here ;-)
http://forum.xda-developers.com/showthread.php?t=1361859

BTW, looks like su problems started after r1.05 :-(
Committed from my Windows phone:-)
You mean moving the on_load hook out of this initializer hook? 

What is the symptom?
Why deleting "--drb"? Isn't it a nice thing to have automatic support for it on a non-JRuby?
I think that rather than changing the modification time of the parent item, the correct thing to do would be make sure that when plone.app.caching calculates what URLs to purge for the discussion item, that list of URLs should include the parent item's main view. This can be done with an IPurgePaths adapter. plone.app.caching actually includes one for CMF discussion items (plone.app.caching.purge.DiscussionItemPurgePaths); presumably we need to add something similar for p.a.d comments.

As an architect of both plone.app.caching and plone.app.discussion, @optilude may be able to confirm this. :)
@parndt asked me to comment here...

This ppa provides an old PhantomJS version (1.3.0). The current is 1.4.1. If people want to use [poltergeist](https://github.com/jonleighton/poltergeist), they need 1.4.1. There is now a [precompiled tarball for Linux](http://code.google.com/p/phantomjs/downloads/detail?name=phantomjs-1.4.1-linux-x86-dynamic.tar.gz) available, so the best way would be to download that and symbolically link the binary into the PATH.

FWIW I have been [trying](https://github.com/jonleighton/poltergeist/commits/) to get poltergeist tests running on travis, by installing phantomjs in a before_script. I'm having some trouble with it behaving erratically - it seems to sometimes connect to the Xvfb server and sometimes fail. If phantomjs can't find the X server it will just exit with status 1, but not show any error message (which is a bit annoying, but just a heads up).

Also: it's a bad idea to compile PhantomJS 1.4.1 against system Qt libs, explanation in the [poltergeist README](https://github.com/jonleighton/poltergeist/blob/master/README.md).
Please use the tagged 1.0.0 version with Symfony 2.0.
To samo tu. Poza tym zrób też test ze spacją jakąś lub końcem linii(`\n`), może nawet z tabulatorem (`\t`). Jeżeli `position` wskazuje na te znaki, a `position-1` na inny niż "biały znak", to też powinniśmy dostać `true`.
```version=origin/2.0``` should be added to the deps file of the packages that are provided on the website.
Just started a new project and got into this issue. Wasted an hour or so to get on the right way.
Holo is 3.0+, DeviceDefault is 4.0+. We choose Holo so that we'll get consistent styling when OEMs put crap like Sense and TouchWiz and MotoBlur on their phones.
Please split this:

  if (row1.length() == 4 &&
      row2.length() == 4 …
Ah, hadn't read the [discussion](http://www.ps3mediaserver.org/forum/viewtopic.php?f=6&p=64665#p64678) yet.
There are a couple more occurences in the code, I'll remove those as well.
At:
$message['TextBody'] = $this->_cakeEmail->message('text');

Is not it correct keep? :
$message['TextBody'] = implode($eol, $this->_cakeEmail->message());

See this commit the jboesch:
https://github.com/maurymmarques/postmark-cakephp/commit/7a8437ffc324807c931781a1cca3eec72c9e0b14
Also fixed some namespacing issues, included README inside gem as an rdoc, and made "VERSION" usable as an object, which IMHO is easier to deal with
blizbiggy: Exactly and if you correct these errors, you will see next more :-/ According to me, someone badly updated file characters_database, because previously everything worked OK.
You shouldn't commit this file, it's meant to be kept as a local file for each user to specify their build directories.
You *do* realize that supporting python 2.4 through 3.1 is going to cause us a lot of headaches?
We could either support 2.6 - future or 2.*, but supporting < 2.6 and >= 3.0 is madness.
(At least, that's the response I got in IRC when I asked how best to support 2.4 - 3.1)
Why are we calling 'close' on 21 and 22?
I have problems with YouTube playback as well actually. I'll be sticking with this, but let me know if you manage to get things sorted out. The OMX situation seems to be a bit messy right now.
HEERROOOOOO
you should change the typehint to the TranslatorInterface
Во многих местах возвращен serif, но вопрос о шрифтах на 2chan.net снова открыт.
The question is mainly "how the application using this specifying the list of values that could be valid"? Maybe can we provide an optional argument to look the values into.
The thing is HEAD should be implemented for anything that's a GET request or we're in violation of the spec.  But if this isn't exposed as  GET request then there cannot be HEAD.
(Also, use $currentJournal->getSetting rather than $journalSettings.xxx -- accessing $journalSettings directly is deprecated.)
According to Vitor they are pretty much arbitrary.
Hi Lluis

Thanks for the quick response.
I've tried removing these lines and the framework doesn't wire up the event. Am I missing something or should I be wiring it up some other way?

Thanks

Antony
This logic should simply be embedded within the `self.setup_py` property; `self.setup_py` should always point to the `setup.py`.
do we need a global variable?
Small typo: `plain`
Good catch.  Fixed in https://github.com/Jasig/cas/commit/21b598772fff0f8393a6dc426e542f09e23fd446.
Despite we shouldn't have a "MyControllerController" this would be safer as a preg_replace using "/Controller$/" (to make sure it removes only the one at the end of the string)
+ you can use __DIR__ instead of $dir
ok, but I tink there is a race when combining barriers with shutdown/remove. Got that failure on jenkins (akka-pull-request) now. We have a ticket for that, #2137.
Should we not run this spec with both the FDs?
what does it mean if ack() is called and tries == Marker?
please use a subtree of the default config here, otherwise there will be conflicts between this and real actor systems

    ConfigFactory.load().getConfig("migration")
You're most welcome, keep hAkking :-)
Yes, I'll de-expose the Gossip, but I'm not sure it's worth it to go all
the way to detailed change events, such as
MemberJoined, MemberLeft
I think the granularity of MembersChanged is good enough.

On Wed, Aug 15, 2012 at 9:17 AM, Jonas Bonér <notifications@github.com>wrote:

> In akka-cluster/src/main/scala/akka/cluster/ClusterDaemon.scala:
>
> >      if (PublishStateInterval == Duration.Zero) publishState()
> > +    publishMembers(oldGossip.members)
> > +  }
> > +
> > +  def publishState(): Unit = {
> > +    eventStream publish MembershipGossipChanged(latestGossip)
>
> Yep. I wrote that up in the ticket. You didn't see that? The Gossip does
> not help the user much. We should do the diff and publish actual changes.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/617/files#r1380314>.
>
>



-- 

Patrik Nordwall
Typesafe <http://typesafe.com/> -  The software stack for applications that
scale
Twitter: @patriknw
This will throw a TimeoutException instead of ClassNotFoundException if it times out, is that intentional?
I guess that every test that uses actorFor and the TestConductor needs to use a lazy val to avoid this problem.

Can we document this somewhere?
Ok, I will make sure.  I think that most of the message types will break tests if they don't work at all, and I can test that.
and the opening bracket needs to be on a new line
On Wed, Oct 24, 2012 at 7:57 PM, MrBago <notifications@github.com> wrote:

> In dipy/reconst/tests/test_dti.py:
>
> >
> > +    # Test fitting with different methods: #XXX Add NNLS methods!
> > +    for fit_method in ['OLS', 'WLS']:
>
> Having lowTri as a fit method was a hack for creating a Tensor from
> previous fit results. This should probably just be replaced with a
> constructor that returns a TensorFit from previously calculated fit values.
>

I think that's what it does. I will add some more documentation to make
this a bit clearer and a test or two.
I don't think we want to do this as h is template helpers and so available to templates.  I'm not sure we need this outside of that context.

Part of the thinking is that the toolkit will be a very stable interface so we want to minimize what is added.  This does not mean that I cannot be persuaded in this case maybe this is an important enough function.  I'd just really be sure before we add it that's all.

The idea is a stable interface for the extensions
I swear I commented on this before but I guess I missed the giant green button.

This doesn't have to be confusing. That parameter was added in this request, so if it's that confusing, change it. Suggestions:
"--details" - In other words, show me more info about the associations.
"--show-association-data"
"--show-associations"
"--association-data"

Or other such options. The term "metadata" is used all over when referring to unit information, it's probably better to qualify it in this case as association info.
I don't think so. It throws an exception if things don't validate, but I think that's about it. There's a bunch of #validate methods on other objects that do nothing at all.
Seq is enough, it wasn't initially as I was using -- to do the diff, but it's deprecated and found the filterNot contains way to do it. I'll remove the toLists and use only seqs ;)
I do not like that equality is defined by the toString method. The method could evolve for debugging and cause weird side effects.
En fait, il y a toujours une value dans RingNode, sauf si c'est un RingNode instancié pour faire une recherche. Bref, ce qui est retourné au client a toujours une value. Peut-être il y a aurait moyen de faire ca différent en utilisant l'héritage et en retournant toujours un type qui possède un value non optionnel.
Heh - guessing this was AE development day 3, but gnif might have a better answer.
Same here for both } and {
This sounds much better than before. Good job on making it more terse.

I would nit-pick on the usage of the words "though", "jobs", and "pretty" .. but that's up to you :)
```
else:
    super(MRJobRunner, self).__getattr__(name)
```

Otherwise `runner.pretty_much_anything` will evaluate to `None`
podmiane na... ? bo chyba nie dokonczyles ;-)
Well, he did say rules are there to protect children from falling ;-P
The second "// Comment 1" is really unnecessary
why is property better?
shouldn't this be while then?
there should be a space here
s/bundle/plugs/
I *really* don't think its worth it, I make about 70 lines of functions that are the same.  The other functions have some differences due to the fact that these are different markup languages as I outlined above.  Better for maintenance to keep two small self-contained parsers IMO, then you don't have to care if changing something will break some other parser.
Considering the colour is `CGColorRelease()`d below I'm not bothered enough to change it.
Please use layout constraints! I might be partial to them because I wrote them for TwUI, but they're a lot neater and forward-compatible with AppKit and UIKit.
Yes I know, but the question was more of what the IDL compiler will do.  It looks like the browsers do handle floats, for example:

new DataView(new ArrayBuffer(4), 1.1)

DataView
buffer: ArrayBuffer
byteLength: 3
byteOffset: 1
__proto__: DataView
I believe `session_size` should always be `>=16` and `<= 32`.  I think its safer to encode these limitations here, and fallback to openssl -- I want to be paranoid about parsing this kind of thing.
Looks great otherwise.  I use this feature all the time in node. (I've been working on mesh networks for the last couple months)
ah I see... makes sense then!
No... the iterator code is simply returning an incorrect multi_index for zero-dimensional arrays:

```
In [23]: a = np.array(0)

In [24]: a
Out[24]: array(0)

In [25]: a.shape
Out[25]: ()

In [26]: i = np.nditer(a, flags=["multi_index"])

In [27]: i.next()
Out[27]: array(0)

In [28]: i.multi_index
Out[28]: (0,)

In [29]: i.next()
---------------------------------------------------------------------------
StopIteration                             Traceback (most recent call last)
```

Everything here is fine, except that in line [28], it should return `()`, not `(0,)`. Returning `(0,)` just makes no sense in any case; it is not a valid index:

```
In [30]: a[(0,)]
IndexError: 0-d arrays can only use a single () or a list of newaxes (and a single ...) as an index

In [31]: a[()]
Out[31]: 0
```
I would much rather use a macro as I did in my original patch submission to free everything.

```c
if (!NETSCAPE_SPKI_set_pubkey(spki, pkey)) {
    goto cleanup;```

I was however told I should move away from using those
But, I thought only 'end' can confirm the timeout. Suppose we set happy to true on the first chunk, but second chunk timed out, we need to raise timeout error.
По стайлгайду это нужно попробовать заменить на цикл foreach
Not following. What comment?
Same comment here.  This is *way* too complex for what it needs to do.
Will do. Do you still prefer explicitly needing the &block parameter in the method def or leaving it off?
I would get rid of `create_order_node` and change this to:

``` ruby
        when Symbol
          table[order].asc
        when Hash
          order.map { |field, dir| table[field].send(dir) }
```

We don't need to validate the `dir` here as we're doing that earlier.
Two lines :scissors:
Can you add an example reflecting the end_of_days?
Moving this code from run_aux -> init_config breaks the run/2 public API. Why move it?
embarassing, will fix.
it's actually in a separate commit, that only does that
ok, I removed the `:func:`
yeah, it's necessary or all you see is vertical lines
Ralf Gommers wrote:
> It also seems to servo two separate purposes, defining a bracket or a
> bound. How about having two keywords ``bracket`` and ``bounds``
> instead of this one?

Yes, that's probably better.
continue means skip to the next iteration of the loop. This happens when the file name cannot be split into four parts (that is when there is something else on the web site we do not want to consider).
Maybe form_lagranges_equations instead?
I think you should write down what you personally think the behavior should be. Whoever comes by later change change it if they find a better solution.
or, since you need the index and the value you can use enumerate:

```
for i, ti in enumerate(terms[:-1]):
    for k, tj in terms[i+1:]:
        j = k + i
etc...
```
Perhaps you could copy over the code from the test to make this statement (and particularly the word "simultaneous") clearer?
Can you add two blank lines between the methods?
Renamed to `is_args`. We're still sensitive to people putting non `tuple/list/sets` into the `args` field of a `Compound` though.
ok, that's done now, too.
A further description of what majorizing is would be good (or a citation).
seems that you did not change the name of this property.
1.1.13 already
has to be `set`. fixed in 0955e63708df1bfe96377bb36330cc411e4ef94f
Better: "Einzelne Scopes können in der Form ..., mehrere **mit** ... angegeben werden."
unecessary doc
I think this should use the `pkg_resources` API (`resource_filename` and friends), like `pyramid.static` does.
__run_test_methods take care of failure_count and class_setup fixture is propagated to test methods, so it's taken care of for class_setup fixtures as well. But I think we need to bump failure_count for class_teardown fixtures.
The brackets here are correct, but look a little bit confusing. Can we add more parenthesis to make it more clear? 

E.g.

    if (!loadFlag) { 
      latch.countDown
    } else spawn {
      ...
    }
Instead, we just use `::` at the end of the previous sentence.
This is wrong. You should not exclude composer.lock
Convention: return -1 on error, 0 on success.
```
[L1106:C14] Don't make functions within a loop.
```
'this is a sting' !=  '\"this is a string\"'
as we agreed in discussion we will use new array based on the ARGV as a parameter for Run_In_Background.start
Prejudiced against ReturnType as a type parameter name?
I find the math for all of this rather confusing and wonder if the config and this code could actually be simplified by having the config be two things

1) the "origin" timestamp
2) the duration to check for termination conditions

Then you could just do something like

origin=2012-01-01T00:55:00.000Z
duration=PT1h

and it would check every hour on the 55th minute?
we could have includes for both.
This isn't in the header, it's with the viewonline/birthday/etc lists, where someone might want to output additional information blocks
Shouldn't be accessing the super global directly.
When you output a variable it is wrapped in an isset check, so an empty string is output instead. So in this case each variable should also turn into an empty string if it is not set. We just don't have behaviour like this in any of the other expressions.

Although actually the IF does something like this, the if doesn't fail just because one of the variables isn't set. It treats each unset variable as an empty value.
Yeah, callumacrae already explained it to me. Ignore this note, please.
Signature may be restricted. In fact signature restrictions are fairly common. I think the permission should be granted to everyone by default, consistent with existing behavior.
Have you tested sending emails with multiple languages? This looks good, but it should be tested.
I guess there should be spaces after comas.
Looking at the later parts (e.g., mroute), it's probably worth adding:

```haskell
-- | Function from path info, request configuration and actual request to a response
type RouteHandler = [T.Text] -> RequestConfig -> DCLabeled Request -> DC (Maybe Response)
```
You’re right, `rewind()`/`reset()` don’t really help to reduce ambiguity.

My original motivation to comment on this was: I found calling `reset()` and then `play()` looks a little confusing, and also adds some overhead by removing and re-registering the `tick` handler.
Yeah, that's a good point. I don't think there's really a better way to handle it after all.
Please separate the properties. Use

```ruby
should: %s[0].name == "NS01.XIF.COM"
should: %s[0].ipv4 =="63.240.200.70"
```

instead of

```
should: %s[0] == _nameserver.new(:name => "NS01.XIF.COM", :ipv4 => "63.240.200.70")
```

Also keep in mind nameserver names should always be lower case in the output.
malformed json (trailing ,)
any reason this can't be unified with the version from `within`? Looks essentially the same, except one variable name...
I don't think that this construction is thread-safe. The probability is close to zero, but 2 threads can end with locks on 2 different mutexes.
I know, I've seen it. I have fixed it but am waiting to push it until I have fixed some of the new game supports im working on
Indeed, I didn't even notice this one. Fixed.
ok....will change it soon.

On Tue, May 29, 2012 at 10:15 AM, nathanleiby <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,7 @@
> > +Ext.define('RaxaEmr.Outpatient.model.patientlist', {
> > +    extend: 'Ext.data.Model',
> > +
> > +    config: {
> > +        fields: ['firstName', 'telephone', 'city', 'state', 'disease',
> 'noofvisits', 'lastvisit', 'nameofdoc', 'id', 'age']
>
> work with astha (designer) to ensure you're showing all needed patient
> fields (There may be more which aren't yet pictured in the designs)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Raxa/Raxa-JSS/pull/60/files#r890333
>



-- 
Vikas Singh
Senior Undergraduate
Department of Computer Engineering
Institute of Technology
Banaras Hindu University
Varanasi
Ph: +91 9838438152
Not compatible with python 2.4, I think.
This is the justification for why connection_pattern is needed and not a "workaround."
с канканом сделаем: admin, player, editor, advanced (если чё, этого потом переименуем)
One of these DDL files is going to need to have at least one other partitioned table to test all the error handling cases.
I see now that this inheritance was not such a great idea. Do you agree?
I wasn't considering that utils.Pair is immutable and that's not at all how you are using your Pairs. I apologize.
I'm having trouble seeing why you go through the trouble of defining and using Pair instead of just using local variables named "first" and "second" and maybe implementing a few static functions like 

    static private String pairString( Object first, Object second ) {
        String firstString = (first == null )? "[]" : first.toString();
        String secondString = (second == null )? "[]" : second.toString();
        return "(" + firstString + " => " + secondString + ")";
    }

The Pair class doesn't seem to be of much help.
Well done, sir!
Method should start with lowercase letter
Missing spaces around `!`.
Open curly braces should be at the same line of conditional expression `if (...) {`
As above, regarding casting style.
This should be rewritten in plain SQL using assert.
If you agree we should have naming standards around internal functions these should be included.  E.g. "__bytestreamin"
Considering there are user agents with more than one set of parenthesis, this is a good protection to have.
Yep, I'm good with this!
Use full path in the test: `#{bin}/fon-flash`
Should be `system "#{bin}/normalize"`, to make sure we pick up the right one.
you can just delete unused code
According to our coding guidelines, we agreed to use braces even for one line ifs :-)
__DIR__ was added in 5.3.  Need to use dirname(__FILE__).
Done. See other comment.
I'll try to do it into vumi.utils
Can you delete these instead of leaving them commented?
Maybe it is better if this method receives a sid parameter and does something like: if (this._controllers[sid] { delete this._controllers[sid]; }
If this tuple is only used by GET_wiki_page for redirecting to a canonical URL, then I think we shouldn't implement that logic with a validator. Otherwise there's a technical tax on every use of VWikiPageName just for this one-off feature.

Would it be so bad for GET_wiki_page to just look at the request and see if the value of the 'page' parameter was changed by the validator?
That was what I was originally going to do, but I thought it might be a bad idea.  So...would it be so bad?
Okay - also agreed :-)
Is it necessary to expose a new ivar?
Thanks for updating. I will try to add some to the checkstyle rules although not all of them can be expressed as regular expressions.
On a purely stylystic note, this might be better:
```python
headers.update({
    'Accept-Encoding': 'gzip', 
    'Content-Type': opts.get('content_type', 'application/json')})
```
(breaking it out so each key is on it's own line for readability, and using .get() with a default rather than using the 'or')
`@` should be `this`
So there is a chunk rendering queue now? nice.
Oh, also: I don't think we'd have actually created 2 Document instances for the same file, because of the guard in the ctor.  Creation of the 2nd Document would have just crashed instead (which is at least less subtle).
The EditAgent now relies on a data-ld-edit event to enable/disable the editor + on DOM. characterDataModified to realize the actual changes.
It seems like calling the "_"-prefixed one here means we'll leak by leaving the entry in _popUps...
`{string}` should be lowercase here too
Change the copyright and the leading message ("Code Editor")
@jinghao: Wouldn't any decent compiler backend transform integer arithmetic with constant parameters into the most efficient representation on the target automatically?
Can you change it to `disabled="disabled"`? That's the convention.
Do we still want these in here?
Could you explain the comparison here? I am also afraid that `seriazlie_recurse`, which is quite hefty for large JSONs, is going to run for every JSON output, then get thrown out.
yep looks much better now.

`!!` will cast the returned value to a boolean and `!` would make sure we have the negation, but I think your solution is easier to read..
I did something in that order. I also moved .button classes up to body, since they will be used on all buttons? Or should for the momment leave them where they were?
Done, but this was implemented the way it was only to share errors with fields.py and since fields.py errors are now hard-coded, I can use the correct way of reporting errors here.
OK. You are right. But we have also one additional speciality: our portal will be accessible inside the wireless network. Will we want to allow it also to be accessible outside? ;-)
I Installed ruby by One-Click Ruby installer for windows.
Maybe, I think it has to be bad.
http://rubyinstaller.org/

---test.rb
class Hoge
	def print
		p "test"
	end
end

---main.rb
require 'main.rb'

h = Hoge.new
h.print

---ruby main.rb
C:/Ruby193/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in `require':
cannot load such file -- main.rb (LoadError)
        from C:/Ruby193/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:i
n `require'
        from main.rb:1:in `<main>'
Yes, let's remove -- we'll use our own embed stuff in anything.
Please remove the jquery-1.7.2.min.js file, since you're loading it remotely.
Let's move this inline style-sheet, plus all the ones in the test files below (i.e., merge it into one file), into css/manual-test-styles.css so that we get csslint working on it and don't have to duplicate for every test file.
Right, fixed (locally)
Should this be `parsed_body && parsed_body["errors"]` ?

if parsed_body is nil or false then, the way it's written here, it looks like it will try to call the [] method and throw an exception.

Also, while they work the same in other ways, the 'and' and 'or' operators have very low precedence, while the '&&' and '||' operators have very high precedence.  So if I understand right this code is effectively the same as:

`(return false unless parsed_body) or (parsed_body["errors"])`

which is probably not what you want.  Generally I would avoid 'and' and 'or' and prefer '&&' and '||' unless you are very confident about the precedence rules.
A runner is a Python module that contains functions. Each public function is...
This was suggested in the original paper as a sensible value.  In practice I don't expect the actual value to matter much since we will train the model to whatever we choose here.
这个没必要用switch，用if吧。
This should probably happen in a template
С `mocha-as-promised` здесь можно просто promise возвращать

```js
before(function() {
    return BORSCHIK.api({ tech: tech, input: inPath, output: outPath });
});
```
Nit: Could you remove these whitespace changes from the patchset, please?
This is cute if you happen to be looking at the diff, but I can't help but think this list can probably go away now.
This feature is required now for headpin; katello will catch up when it takes care of the pulp issue.
So if len(system) > 1 then we remove only first system?
I agree, but as we are starting soon with the 3.1 development, wouldn't be better to do a massive replacement there (probably with helper scripts) than  introducing noise in every 3.0 bugfix?
We'll need to localise this at some point