labofsahil/github-event-dataset-2012 huggingface dataset:
add back functions used in some Windows DLLs

- I will also prepare a pull to update the dmd samples.
Test Fixes

portal/dbupgrade/sampledatalatest/documentlibrary
portal/dbupgrade/transfersampledatalatest/documentlibrary
portal/dbupgrade/sampledatalatest/blogs
portal/dbupgrade/sampledatalatest/wiki
portal/dbupgrade/transfersampledatalatest/blogs
portal/dbupgrade/transfersampledatalatest/wiki

This should somewhat fix the document library lar tests. However, the issue where the wrench icon stops working will still cause failures. This should fix the rest of the lar tests though.
mentioned looking at instance log for dot warnings
Moodle 23 stable

This fixes some issues in Grid course format:

- Fix upgrade code failing on DB upgrade 
- Add section release criteria compatibility
- Fix section images getting overcompressed in grid format
Just a bit of housekeeping...

This should get you back on track to making future edits easier.
Fixing URLs

There were a few urls on the main page that weren't properly pointing to the internet.
gles2 working

i have been working in the same branch so this contains the commit from the previous PR, sorry!
Mostly PHP functions on manage_account and Cupcakes
Added Arquillian OpenShift 1.0.0.Alpha2 highlights
Детектим раржпеги в отдельных (аж ЧЕТЫРЁХ!!!) тредах

Это не просто костыль, это <h1>МЕГАКОСТЫЛЬ</h1>. Почему-то внутри песочницы, в которой работает скрипт, нельзя создавать рабочих. Ёбаный стыд.

Зато работает быстрее, да. Намного. Таким же макаром надо сделать вычисление хэшей изображений.
Предупреждения компилятора

<b>JSC_WRONG_ARGUMENT_COUNT</b>
Pull Req: Fixed issues #7 and #18

Fixes for:

GHI #7: User has to enter "local:///" in the content src of the config.xml file to open the local page in the simulator otherwise it shows a blank page

GHI #18: Compilation error shows when author tag has "rim:copyright", "href" or "email" specified
Fixed a broken link to Spring Security docs
F L A S H
GROUPLOVE - Itchin on a Photograph
JSON support

add support for JSON serialization which causes problems with generics
Sdk download + integration test

Added automatic sdk download from github and integration test
I forgot to remove the unnecessary 
	&lt;properties&gt;
		&lt;env.SDKVERSION&gt;${project.version}&lt;/env.SDKVERSION&gt;
	&lt;/properties&gt;
in the pom. To be removed.
Updates to threading libraries, UnityEditorBinding, etc
Removing a redundant function that is now moved into the lib/ui/invocati...

...onlist module in WP
Declare MSMFB_OVERLAY_COMMIT in msm_mdp.h

This fixes the following compile error:
hardware/qcom/display/libhwcomposer/hwc_external.cpp: In member function 'bool qhwc::ExternalDisplay::commit()':
hardware/qcom/display/libhwcomposer/hwc_external.cpp:541:26: error: 'MSMFB_OVERLAY_COMMIT' was not declared in this scope
Fixed several parameters on several methods
hiding sortable field

In order to use sortable field in sonata_type_collection it need to be visible, which is really 

annoying.
I have created small patch, so such a code will work

protected function configureFormFields(FormMapper $formMapper)
    {
        $formMapper
            ->add('position',null,array('attr'=>array("hidden" => true)))
Update Readme to include syntax highlighting.
Spree 1 0

Gem updates for Spree 1.0
Make the reference to the synchronized queue an atomic reference
Change the SynapseJavaClient to not output anything to the console

Instead, it uses it's logger, as it should.
Retain removals & README update

I'm adding this to one of my projects and noticed two retain statements that can go away with ARC enabled.  There was also a reference to what I think is an old class name in the README.  Thanks for open sourcing this!
jQuery 'noData' event for autocomplete.

jQuery 'noData' event for autocomplete. Triggered when sql-query returns no results. It's useful to show messages, somethink like "sorry, do data found" etc. 
To bind a callback function jQuery("#autocomplete_id").bind("railsAutocomplete.noData", function(){ alert('no data:('); })

Guys, please review. I find it useful for myself, hope somebody else will find it useful too.
Restore possibility to give a block to autocomplete

This isn't only useful to set a different json encoder. It adds the possibility of aggregating the items and returning a different set of autocomplete options.
common: hack: increase system size to be able to compile
Added the previous checks to OK, with changes to reset values
Change comment to read TRIAL VERSION if trial release

Otherwise, there's no indication of that in the file.
reproduced SPR-8955
Fix auth support
Fixes to README and parse_svmlight_mapper.py

Fixes a number of README typos and a bug in parse_svmlight_mapper where comment lines in the original example SVMLight files where not ignored, resulting in positive instances with no features.
Syntax Error correction.

Error in gem spec for 'machinist.'
Added support for new firmware XXKI3 (also extracts nvram_net.txt_murata...

... for setting data calls up).

See bug comment https://bugzilla.mozilla.org/show_bug.cgi?id=710493#c5 .
Runnable examples - preprations for Phobos and other fixes

Various changes in order to prepare scripts for phobos' runnable examples
Also some changes in UI.

This pull needs to be merged first, before Phobos' ones

Demo as always at http://dlang.dzfl.pl
[WIP] Javascript Validation

Making a pull request out of this so discussion is simpler.

Idea:

1. Have a javascript library that mimcs the validation rules of the server side Symfony validation.
2. Have a way to expose certain parts of the validation metadata to the client side.
3. Combine validators and metadata to allow client side validation.
Added the default page controller.

Don't forget to update the routes.
Update Ukrainian/main/DrmProvider.apk/res/values-uk/strings.xml
Moved Italian to i18n/it.yml

Hi.  Pushed the Italian translation live as well.

Thank you!
Sanity checking for existence of $scopeSpecified
trad

J'ai fait particulièrement attention à la ponctuation ce coup-ci.
Je voulais ton avis pour ceci car il n'est pas toujours évident de retrouver les successions des phrases => à la ligne : 
21)  CHRISTA
{thinkingA}Why shouldn't she be?
J'ai supposé que cela faisait suite à : 
22)  LEE
{SadA}I know, I know. I just want her to be safe. (je veux juste qu'elle soit en sécurité)
Dans ce cas, la traduction peut être : "pourquoi ne le serait-elle pas ?"

Idem pour : 
32)  CHRISTA
{angryA}Yeah, same here.
Je suppose que c'est la réponse de Christa à Lee qui dit : 
33)  LEE
{SadA}I've got things to do.
Si c'est la cas, le sens pourrait être  "Oui, moi aussi" mais dans le doute j'ai préféré une traduction plus approximative : "Oui, pareil ici". C'est compréhensible mais peut-être pas optimal.
Change parent container from .clearfix to .control-group as this is more...

... common in Bootstrap 2.0+
Sounds good. One more question, if I may. How do you envision to secure the health check HTTP endpoint (basic auth, perhaps)?

Thanks,
D.

Sent from my iPhone

On Mar 2, 2012, at 16:36, "Marvin S. Addison"<reply@reply.github.com> wrote:

> To add to the context of my previous comment, the assumption is that most institutions have some kind of enterprise monitoring framework in place that can do HTTP health checks with status code evaluation as a minimal feature.  I know most frameworks can do much more, thus the X-Monitor headers for them.  I'm open to other bells and whistles if they would come at minimal cost.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/serac/cas/commit/30e2882469647e5e6cb44e1f33f574b8c8dcf5bd#commitcomment-1037477
These used to be in alphabetical order.
Yep, I agree it's a shame we had to do this.
@rafaelfranca ah, thanks. I can see one broken test on my machine (railties/test/application/initializers/frameworks_test.rb:160) but I couldn't reproduce the one broken on travis-ci (railties/test/application/route_inspect_test.rb:72).

I'm running ruby-1.9.3p0 like travis-ci, and ran `bundle exec rake test` in both `/` and `/railties/` so I'm not sure what's going on. Do you have any clues?
...initialize for an IPython console

make change on both lines
well dmd uses 'typedef uint64_t StorageClass;' defined in mars.h.
so what to do?
@bnoordhuis  o['variables']['strict_aliasing'] is always false. Fixed in 031e9b3c2da1dd02a45ccdc8cf4639bdf952eace .   I can pay off my debt, Ben  :-)
Ty změny jsou super, byla tohle poslední změna API Latte? Možu si začít hrát? :)
To jsem asi nikdy nepotřeboval.
#ignore
Hi, I've added some more comments. This is coming along nicely!
One thing I forgot is that the changes to EmberServices should also be in a separate commit.
I don't understand the question, sorry. Which file are you after?
did you intend to add a newline here? same at line 69
People might expect their close callbacks to run through.  And in some scenarios it's possible that close callbacks will do something to increment the refcount (e.g. create a new tcp connection), which would have kept the loop running.
You can get rid of the conditionals and just do this:

``` ruby
format.json { render :json =>{:response => signed_in?}.to_json }
```
We decided to go with whatever was the Vanilla spec. Anything more should be handled and/or provided with a plugin.
Why the choice of returning 0 instead of the time they logged in?  hasPlayedBefore() covers testing for first time joining, surely the other values can be accurate to the timing?
We should remove this commented block before submitting
This has the side effect that we get the "export" prefix when doing `brew install -v foo 2>&1 | gist`. (Maybe it would be better to have a switch for this behavior?)
I know but there were 2 ClampInBounds(): one that changed the calling instance and one that created a new one.
To me it is very ambiguous if GCC uses the correct one. Also the following code was legal
`const float3 foo;  foo.ClampInBounds();`
What does nothing, but when you don't check the const  definition of foo you would assume it changes foo itself.
Thank's a lot.
Same is true for all hover elements.  Keep rules separate for now..
Twas decided it was better way back, wish I could find my logs
这是个测试题目
============

1. 测试
2. 测试
I don't quite understand this after looking at it for a minute. Can you put a comment explaining what this sanity check is? Also I don't think "normalized" is what you want here, it has a specific meaning... maybe "adjusted"?
The above change does breaks repeated STARTS.  Unfortunately, the original author change is completely un-necessary, and is supposed to ad support for clock-stretching.

Per the Atmel manual.
[...] the TWI will hold the SCL clock low during the wake up and until the TWINT Flag is cleared (by writing it to one). Further data reception will be carried out as normal, with the AVR clocks run-ning as normal. Observe that if the AVR is set up with a long start-up time, the SCL line may be held low for a long time, blocking other data transmissions.

One can infer that the SCL clock is held low until the TWINT flag is cleared, which occurs in the twi_releaseBus() method.  This needs to occur after the callback has been completed.

Therefore, the need for the custom 'twi_stop_slv()' is not necessary, and in fact the routine is identical functionally to the twi_releaseBus() method.  Removing lines 416/417 above is the correct fix.  Without it, the Arduino will not respond correct to repeated start commands, which is the most common way of accessing I2C sensors (read data from a register, which is specified via a write followed directly by a red).
I was having problems with the # symbol. Since the host files have that in to symbolize comments, they have to be there and Jekyll formats using the # to signify some sort of heading.
because shapeshift form = 0x0D = SHADOW DANCE
r+ for an immediate change, i'm still going to file a bug for a rewrite.
At the risk of looking like a begger, PLEASE try to fix the currencies bug as soon as you can, this update is somewhat pointless as players can simply relog and have their valor points maxed out, thanks :(
es können ja aber nur nur beliebig lange tabellen damit erstellt werden. die darstellungsform ist ja unabhängig...
REGEXPOWER, für was brauchst du die hier?
http://www.youtube.com/watch?v=ltM5jHIJFw4
I have tested this with all four flavors of OBJ files:
* containing only vertices (v) where faces are defined as 'f v1 v2 v3' (this used to crash)
* containing both vertices (v) and texcoords (vt) where faces are defined as 'f v1/vt1 v2/vt2 v3/vt3'
* containing both vertices (v) and normals (vn) where faces are defined as 'f v1//vn1 v2//vn2 v3//vn3'
* containing vertices (v), texcoords(vt) and normals (vn) where faces are defined as 'f v1/vt1/vn1 v2/vt2/vn2 v3/vt3/vn3'
r+ w mine and @cvan comments
Raise errer at "yardoc --languag ja."
Since improve document.

Thank you for yhara!
These need to be inside the bootstrap package, not sure why there are 2 of them... but only the one in bootstrap gets called!
https://github.com/SpoutDev/Vanilla/blob/master/src/main/java/org/spout/vanilla/protocol/bootstrap/handler/BootstrapIdentificationMessageHandler.java
I agree completely, @paulirish. Issue created: #106 <3
You'll have to be more specific with your bug report.
Nice catch. I found some other stuff that needs to be changed to fully get this working with two instances. I'll commit them to my fork in a few minutes.
I don't understand this commit. You've edited old builds? Is this a new version of GeoCommons - should it be a new provider?
у тебя вроде уже есть RSpec но ты добавляешь factory girl во встроенные тесты - не очень понятно. Ну и я уже писал про assosiation
Thanks for the explanation. Bundles are going to be very powerful!
You've got a loose space here.
(Also, camel-case variable names is something else PEP-8 doesn't really like...)
`hub_page_title` is for the developer hub. you can use `mkt_page_title` for public(ish) pages.
@tardyp: This commit breaks '/users', and doesn't update the relevant docs in `docs/developer/webstatus.rst`.
I'd be up for anything that does have CamelCase, unless those Camels are other models in zamboni.
Never seen that, no. addons.models imports amo though, so I don't see why it should be a problem.
We will try our best to skip small commits like this one.
I fixed this issue in acb582a3281adc2c586ef769c4830168bccdf29f and driftnet now builds/works again.
Don't need to list system dependencies like this.
Can you please also add a test for looking up java:app/AppName and also enhance the tests to use @Resource(lookup="java:module/ModuleName") and @Resource(lookup="java:app/AppName"). Long back in 7.0.x days, I remember running into problems injecting these names, so having these tests will be handy. You can create a separate pull request for that if that's easier for you. Doesn't have to block merging this pull request upstream.
subqueries usually perform pretty bad. Did you check the explain output for it?
@Ahuraa: I know^^ - Gunship ist WIP and don't work blizzlike @moment. I working on it - Therefor it's optional and don't need to be done by players. If U don't want to use until it's done revert this commit.
You've really separated structure from presentation!  Try making your left html use your right css and the other way around.  Very cool!
If you send me a pull request, I'll be happy to add this to my module.
Could you take a look at this?
I don't seem to manage to get the view's bounds, if I uncomment this line it will make the shadow invisible, but this would be a great performance improvement.

Thnx
Those two are from kaelima.
I fixed the issue @ 61dc7f9f479603bfe8de7105ef064a1c8591fe92

I didn't know there's this counter functionality - thanks for showing it :)
DOKU_TPL also appears in several files under lib/ dir. Shouldn't those be changed too?
确实这里为了语义添加了一些原文没有的意思，我觉得这也许是不同语言的表达习惯不同，我更倾向于在可接受的范围内牺牲原意取得好的表达效果。如果这样的翻译不能接受，也请尽管修改。至于上游我毕竟不是native
speaker，暂时先不去管。
在 2012-2-29 上午8:49，"Jiang Xin" <
reply@reply.github.com
>写道：
>
> - amend 统一译为修补。后面的语序调整是好的： 或者您可以用  \"git reset HEAD^\" 整个删除该提交。
> - 翻译时“信”和“达”的度真的难以掌握，你添加了文中没有的内容，如：“因而并未接受”，“更好的处理方式是”。
> - 和英文原文尽量匹配最好，如果缺失语义欠完整，建议上游修改英文原文。
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/thynson/git-po-zh_CN/commit/167d4ac5484ad5a30e416a8880048211ee9fe450#commitcomment-1023838
Hi Tyler,
I can't add a Unit Test for this method because Jan Berkel removed /changed all Unit Tests for ShadowSQLiteDatabase. I believe he is refactoring a lot behind SQLiteDatabase Shadows.
Hi where is retriman ?
Cual era el bug? duraba menos , duraba mas? no entiendo todos esos numeros y cosas raras :C
Solid change :P
OK Jeroen, now I must learn how to remove last commit from my forked repo :)
If you switch to using StringBuffer you can look into using the insert() function which would greatly simplify this whole section.
Because some databases only allow "ORDER BY" on a selected column, the final query becomes something like:
    select distinct doc.fullName, category.priority from ....
which in theory shouldn't behave differently, since it would still differentiate on the fullName.
buffer.find looks for M anywhere in the string, but then buffer.substr(1...) assumes the M was the first character in the string.

If we can assume that commands always START with the primary GCode command, then we can save a lot of time by converting this buffer.find's into buffer[0] and buffer[1] checks.
The debug code in ```CacheDetailActivity``` is also unnecessary because you can see the values in the debugger. ;)
Java uses "Short-circuit evaluation" so I don't know why it's not safe ,-)
You're probably right. Feel free to change it.
Yes -- the functionality itself *is* invalid ... since it can't be relied upon cross browser. 

Basically what I'm saying is that setting Object.prototype properties is "undefined behavior" according to Backbone, because of IE bugs. We *can* add a test that asserts something about the undefined behavior in certain environments ... but I don't think we gain anything by doing so. 

In any case, if you feel strongly about this -- feel free to add it back in as you suggest -- I'm flexible on this point.
Yeah.. I actually just read that from the status code doc. Will fix tmr
confirmed options control over keyboard shortcuts. added to readme as well.
You don't need floor, `/` does integer division automatically.
Oops, I got the feeling something went wrong here .. what did I do?
I don't know what's going on here but I think the issue is in winston. After setting the log levels 'debug' no longer works?...
This must be a typo for sure, right?
I'd hardly sacrifice the main feature speed. It's a well-known heuristic that I applied in research papers before. To quote Burt Reynolds: "Trust me. I know what I'm doing." ;-)
http://www.wowhead.com/spell=35321/gushing-wound
Forget that.
@geraldalewis: try this one out instead: `/^(?:\\\\|[^\\])*\\(?:|[0-7]|[0-3][0-7])[0-7]/`
If no other middleware sets headers, this event handler would never run. We'll want to explicitly set headers we rely on, and clean headers we don't want.

I didn't see removeHeader throwing an exception, but I think this is more bulletproof in case that behavior changes (or was different in the past).
``master`` is not an existing version. and you should target releases, not the latest dev version (the dev version will continue to change after the release of the bundle so it won't stay compatible forever)
Jupp! "According to my request"
thank, yes i misunderstood this sentence 
>still return true for all other flavors and for all non-JellyBean desserts
will fix it next days
Automatically generated `hashCode()` method. I didn’t look carefully at this magic. The ‘Find Bugs’ utility says, it is bad to override `equals()` but not to override `hashCode()`.
-> /b2m/init/help
this should be actually an event as it can happen from multiple places
the even should be called openLink
There is a better way of naming elements. Probably..
See here: http://help.github.com/remove-sensitive-data/
Send a pull request and I'll merge it in.
I don't know what this is supposed to do, but this is not a valid version name, so your package can not be crawled by packagist anymore. I would remove it and rather encourage the lessphp author to update his code to newer standards and add it to packagist.
This is a merge conflict message that would need cleaning up before a  merge.
I'm not sure why you need a new interface.  I would expect the callback side to be implemented inside the sort algorithm.  That should allow the sort interface and activity to remain the same.
Looks like you used the web interface of github to edit the file. Avoid so you don't have problems editing the file. My file uses the original encoding of wired people.
Can you say a few words about what this does? I didn't write config.ru.

I am told that the wordy ruby convention for constructing path names avoids trouble on microsoft platforms. Are we putting ourselves at risk there?
A possible fix would be setting a null byte at position `r` or `BUFFER_SIZE - 1` if `r` >= `BUFFER_SIZE`
Bro I need you to commit less. The commit history has a lot of messy experiments. I feel like this should be happening outside `master` maybe in your own fork, maybe in a branch.
bence iceweasel olmasın.Firefox veya chrome olabilir.Çünkü iceweasel gtk sorunları çıkarabiliyor
This is very useful new feature to FSB. Thank you for contributing to the project!

I encountered an issue when testing the import. 
Environment: Win 7 64. MetaTrader: AccentForex Trader 4 installed at its default path: C:\Program Files (x86)

When importing data I received the following error messages: 

Importing EURUSD240 ... Error (Access to the path 'C:\Program Files (x86)\AccentForex Trader 4\history\Butenix-Server\EURUSD240.hst' is denied.)Importing GBPUSD240 ... Error (Access to the path 'C:\Program Files (x86)\AccentForex Trader 4\history\Butenix-Server\GBPUSD240.hst' is denied.)Importing USDCHF240 ... Error (Access to the path 'C:\Program Files (x86)\AccentForex Trader 4\history\Butenix-Server\USDCHF240.hst' is denied.)Importing USDJPY240 ... Error (Access to the path 'C:\Program Files (x86)\AccentForex Trader 4\history\Butenix-Server\USDJPY240.hst' is denied.)

When I copied the whole HST directory to my Desktop  everything past smoothly.
When I run FSB as administrator and imported from original place, everything was smoothly.

...

Another issue: When I tried to import the files again from other location, I received this error message:
cannot access the file 'C:\Users\Miroslav Popov\Documents\Visual Studio 2010\Projects\QANTau\Forex-Strategy-Builder\bin\Debug\Data\GBPUSD240.csv' because it is being used by another process.)

Best regards,
Miroslav
Ok, I got the point...
For that reason I have made the possibility to have a recently added widget on one of the custom playlists...
If a user wishes to have it seperated, then he can create a playlist himself, name it movies and he is good to go...
See here:
https://github.com/mad-max/skin.moddedconfluence/commit/a34f22f141ee429bc7996171d25d2e96addfaaa3

IMHO this means more flexibility for the users as not every user wants to seperate it...
I know you do, and you know that I do, but this needn't be the majority :-)

BTW: If you're interested in stuff like that, you might watch the branches within the next days...
I decided to create a new  branch and rip out some features that I do not use...and will do some minor modifications...just as you proposed...this will be for personal use, but maybe it fits your needs, too...
Or Javascript. And no, that's a horrible idea in real world code.
I'm not sure what you're saying here. Can you elaborate on what your actual problem is and raise a JIRA please on http://jira.grails.org/browse/GPRESOURCES
A load of integration tests are broken with this change (start the sample and run tests in sparklr2 and tonr2, or use the -P integration profile on command line). I think I know why (see comments inline on commit).
Uhh...yes?
ok thanks.
will try this for next build
thanks.  I swear I tried manually pushing over your files but they didn't seem to make a difference also; I'll try again.  

As for having to put them in the vendor/motorola/triumph/proprietary dir, I feel like an idiot.  For cm9 I had commented out the section in the mk that copied over the old files in the vendor/motorola dir.  See lines 76-82 here:

https://github.com/mantera/android_vendor_motorola_triumph/blob/ics/triumph-vendor-blobs.mk

However, I forgot to do that on the CM7 branch. Doh!

Ok, I just fixed that and pushed up the fix on my repo:

https://github.com/mantera/android_vendor_motorola_triumph/commits/gingerbread
Awesome find! I love that you're pushing changes back as the Open Graph protocol evolves. I just have a note on coding style here. While I love documentation and think that documentation is super helpful here, it makes it a bit hard (for me) to read here. In cases such as this I prefer to present the docs and then write the code. IMHO it makes both parts easier to read:

	/*
	 * Append a prefix to all keys.
	 * 
	 * Facebook prefix for 'admins' and 'app_id'.
	 * Open Graph prefix for all others.
	 */
	if ( in_array( $key, array( 'admins', 'app_id' ) ) )
		$property = 'fb: ' . $key;
	else
		$property = 'og:' . $key;
Please help me update the sources , by either git push or a pull request :)
Anyway , eagle wants to know how to delete a branch :P
file descriptors are not uintptr and not int.
maybe we instead of exiting with 1 exit with 0 or another value..

i use a custom beesh that exits here with 0 to build the lfs toolchain from our bee-lfs project..

i think we need to redesign our very simple exit code concept^^..
might be a nitpick but do we want to call this DBSubnetGroup instead, since that's how it's referenced in the docs (and so that the describe_db_subnet_group call matches the object name?
will make the change and do another pull request.
I know you probably copied this URL from the README, but I was thinking it might be better to link to https://dev.twitter.com/apps instead of https://dev.twitter.com/apps/new. I the user has previously created an app, they don't need to go through the whole process again from scratch. What do you think? Does that just make it more confusing?
Clearpass should not ever allow nulls to enter the map when it actually is in function. The unit test you refer to is probably misleading, and in likelihood should be modified to fix this behavior. While it shows that nulls are handled by the map instance in one way or another, in reality they never would be as proper values are intercepted by the meta-data populator. Clearpass would indicate that a password can't be retrieved, only by doing a lookup and failing to find the password. 

I certainly appreciate your patience and effort. Glad the native injection worked out. Would you be willing to document your approach on the wiki?
Will fix also.
That is wicked awesome. Didn't know about it. Is it in the docs?
or change the comment that `ec` no longer needed for Future construction
I'd probably not do it with .failed and then Await.ready + value but:

val e = intercept[AkkaCamelException] { Await.result(producer.ask(message)(timeoutDuration)) }
e.getMessage must be("failure")
e.headers must be(Map(CamelMessage.MessageExchangeId -> "123"))
Note - It's preferable to use raw re-usable functions in case you need this in other places.

    def akkaPluginPublishTo(version: String): Option[Resolver] = {
      akkaPublishRepository orElse {
        val name = if(version contains "-SNAPSHOT") "sbt-plugin-snapshots" else 'sbt-plugin-releases"
        Some(resolver.url(name, url( .....)
   }
Looks better :)
Top level renderer using the Content class to process the content chunks.  Note the _s version returns a String, whereas the other returns Content objects.
They can also be strings with file-names.
or even contrast_dim. OK
Good point. I am deprecating interleaved and adding a word of caution in the docstrings.
Docstrings on this and SubscriptionItem below please, would be good to have short docstrings explaining what Subscription and SubscriptionItem are intended to be and the intended relationship between the two classes
Same here, use the double ` to markup the package name.
This code is used to store/extract the trace context in Message.metadata which is a mutable map. I did not want to add a dependency directly to Message. I think I will simply move the content of this class into TraceFilter which is now an object and use Message directly as argument. In fact TraceFilter is probably the only user of this code anyway.
to be consistent with above; load_job_flows_from_amazon
No need for this option, since Jam doesn't use devDependencies
@fjakobs Could you tell me how to add a setting to ace so that anims are optional?
shouldn't we require that intervals.size > 0?
I don't think the plugins dir should be in the cache folder. If Smarty uses cache, that can go in there, but plugins should reside elsewhere. I don't have an idea yet of where it should go, maybe under APPPATH.'classes'.DS.'Smarty'.DS
Please change this to return early if initialization fails.
为什么要去掉`?>`?
for each of the outputs can we ensure that `outputs[...] = os.path.abspath(self.inputs. ...)`

so that people can specify `register.dat` or `register.mat`, etc.,.
Regarding quad precision, I cannot test this, but I cannot see a reason why it would not work: the hash method use the same attributes for all builtin dtypes.

Right now, I think we have decent coverage of the hashing logic to test the a == b -> hash(a) == hash(b). As for structured types, we already have some tests (look for tests using assert_dtype_equal/assert_dtype_not_equal), as well as for subarray dtypes.

I am not sure we want to hardcode hashes - we do not want to guarantee more than python itself, and given the recent discussion on randomized hashes, they may be even weaker than they are now (http://bugs.python.org/issue13703).

Pycon was nice, quite a few things we need to look at (https://us.pycon.org/2012/schedule/presentation/78/).
I've corrected with emalloc and 2xmemcpy() calls, being binary safe you mean to ensure the +1 to copy the \0 byte at the end?  That's right now as well.
mUi должен инициализироваться последним судя по порядку объявления полей в классе
This actually seems kinda silly. Is there not a unified way to access `request` on both the controller and the view?
I think this accessor is not being used anywhere.
There's no need to use `in?`, I think it's better to stick with `include?` and the inverse check.
No need to wrap this in a `Rack::Response` - we're not adding anything else to the response and you're supposed to call `Rack::Response#finish` to convert it back to an array when you're done with it.
Does this suggest the programmer to rewrite the offending code to `Test.this.bad1.selectDynamic("sel")` to get rid of the error?

Or does this mean that the offending code has been rewritten, and this produced an error?
I don't understand why this cannot be a val? Are we worried about stack space? Can the tpe's type symbol change during this method? If it's the latter, pulling out the `csym` out of the if changes behaviour, which I assume it didn't, so the motivation must be the former. (I agree it's clearer to have the head computed only once since only the tail differs from the then to the else branch.)
The problem is that we don't control scalacheck and I'm not super keen on maintaining local changes just to pacify the warning monster.
The entire `Apply(Select(Super(This(tpnme.EMPTY), tpnme.EMPTY), nme.CONSTRUCTOR)), List())` is a dummy, and it's the one and only possible representation that can be emitted by parser. 

This tree is quite convenient though, because:
  * It works as is for the case when no processing is required (empty ctor args for the superclass)
  * Stripping off the `Apply` produces a core that only needs rewrapping with applications of actual argss.

For some time I was thinking of using just `Select(Super(This(tpnme.EMPTY), tpnme.EMPTY), nme.CONSTRUCTOR))`, but that one required wrapping even if the superclass doesn't take any argss.
Crap, I'm not sure how this (and the common spree dependency) changes got into the pull. Fixing now, sorry.
On Fri, Apr 13, 2012 at 10:14 PM, Tom Bachmann
<reply@reply.github.com>
wrote:
>>
>>      def key(self, monomial):
>>          return (sum(monomial), tuple(reversed([-m for m in monomial])))
>>
>> +class ProductOrder(MonomialOrder):
>> +    """
>> +    A product order built from other monomial orders.
>> +
>> +    Given (not necessarily total) orders O1, O2, ..., On, their product order
>> +    P is defined as M1 > M2 iff there exists i such that O1(M1) = O2(M2),
>
> Ah. Well "monomial order" and "product order" are very standard terminology.


OK, so please just take my comments as plebian.
I went back and forth on this. Currently, the entropy function also handles cases where the object is of type 'numpy' or 'scipy.sparse'. Therefore, I left it out of the object.

If it belongs to the density object,  we need to decide how to handle the numpy/scipy.sparse types and how they could use this function.
For the sake of completeness, I've nevertheless added LaTeX pretty printing which exactly mimics Unicode printing.  It's not going to be hard to remove should need happen :-)
There was only 1 xfailed test. It is unxfailed and passing now.
I guess we need to put it in a finally block.
I've tried but I couldn't get the same animation effect with goog.fx.Animate. Due to the time estimation of the issue I was working on, I gave up and made minimum changes enough to solve the build problems. The primary objective for me is solving the build problems, then removing the remaining dependencies. If it's ok for you, I want to leave these as they are, for now. But if it's a 'must', I'll have to spend additional time for having the same animation effect.
similarly, replace `paste0` by `str_c` here

for `shQuote`, the argument `sh` is unnecessary because it is the default; what is more, it is wrong to paste `base` and `shQuote(fig.name)` here because you will end up with `base` not being quoted, e.g. `http://foo/bar/'figure/plot.png'`, so you need to paste the url's together before quoting them:

```r
fig.pos = str_c(shQuote(str_c(base, fig.name)), collapse = ", ")
```
No need for plural forms here for Russian translation.
@samdark @creocoder sure this does not break BC? There was `getSort()` for creating a new Sort class before, which does this:

```
			if(($id=$this->getId())!='')
				$this->_sort->sortVar=$id.'_sort';
```
This will not be called after this change anymore when sort is configured manually.
Also before it was possible to do something like this:

```
$db = new CArrayDataProvider(/* ... */);
$db->setSort(array('attributes' => array('name', 'example')));
// some code or if clauses
$db->setSort(array('defaultOrder' => 'name ASC')));
```

Not good code, but it was possible...
And add a short comment saying the first byte determines whether the stream is compressed or not.
In node 0.6.x, `fs.existsSync` will be `undefined`, so attempting to invoke it will throw an exception. This won't work. Please be sure to actually test this code in node 0.6.x as well as 0.8.x!
If you can, I'd update these method names to explain what the expected behavior is.  So...

```python
def test_pre_routing_returns_passed_in_key():
``` 

Or something like that.
Parenthesis everywhere :)
This is changed so the copy will have the content of the new version draft, but will still have the latest_published_same_number as the source question.
to *return* all documents with the value ``resturants`` in the ``type`` field near the ``<ex>`` point
yes. what's the expected result? why would you run it?
yay for refactoring into functions!
This feels a little odd just sitting out here in the wind. Move to before block, and @ohai ?
Should be escaped.
Needs a blank line added after it.
IMO, post and topic are not proper nouns, so they shld not be capitalized.

Also, I think better language would be like:
"%d reported posts" (same with topics).
and
"%d posts awaiting approval"

Also, does this fit properly with the new plurals system? I'm not sure how that works, so I just wanted to make sure.
Yeah, bantu has suggested I move all the plupload functions into a class and split them up.
No i just copy pasted the code ;) there are 2 breaks indeed. This line looks messy though, but since everywhere else it looks likes this i went with it.
@jcredding - can you add a comment above this line, with the description from the pickaxe link and the pickaxe link itself?  Once that is in place, I think this is good to go.

Thanks!
I'll add a proper check.
This looks pretty odd. Can you explain?
Could you add a URL to point to the origin of this approach and if possible add some comments?
do we want good resulting code or minimal diffs? votes please :)
Per the parenthetical placement here, don't the first two conditions cancel each other out?  Was the intention to group the second two?
No idea =)
Can use Util.dateTime(new Date())
move inside of var Util
 
```
Util = {
  ONEDAYMS = 86400000,
  /etc
} 
```
nitpick: Prefer meaningful names for all image files, e.g. "iconPlus.png", "iconQuestion", "iconYellowQuestion", etc. Is there a reason you are would prefer to use #s?
Yeah, I don't know if I can get onboard with this casting business. At around 134 million this cast will remove precision, and advanced indexing will become approximate on the GPU. Not really acceptable.
Spelling of test names lacks "Determinism".
It'll run the first VoltDB server process in the localCluster in-process, making it easier to debug. I think this line is a personal regression and I commented it out to make it line up with history. I'm not sure it matters all that much either way, but tests might be a smidge more reliable if it's out-of-process for jenkins.
ここは関係ないですね。diffに出たのは不明です。
Why do you copy self.asset_form.cleaned_data instead of using it directly?
I couldn't find an obvious way to deregister. I'd be much happier doing it that way than removing the match.
Please remove this ornament
it's in instmodel.h
It would be more elegant, and possibly more efficient, to use:

    template_postgis = getattr(settings, 'POSTGIS_TEMPLATE', 'template_postgis')
    cursor.execute('SELECT 1 FROM pg_database WHERE datname = %s LIMIT 1;', template_postgis)
    if cursor.fetchone():
        return template_postgis

This is how `QuerySet.exists()` is implemented.
s/Disaggregation/Scenario/
string list stuff has been moved to the bottom of the file, however I'm not sure if it should be in the public header or not.
Personally I don't like 'public/private/protected' to be indented, but maybe that's just me. I prefer that they're at the same level of indentation as the 'class' keyword, so that members/methods are clearly related (of course, you could have an extra intent for methods/members, but if we're trying to keep things within 80 columns, I don't know if that's a good idea)
Same, `$(...)`
Sorry I didn't notice this. 
I'll submit it later this day:) 

发自我的 iPhone

在 2012-9-26，10:49，Jason Mooberry <notifications@github.com> 写道：

> In t.js:
> 
> > @@ -32,7 +32,7 @@
> >  	function get_value(vars, key) {
> >  		var parts = key.split('.');
> >  		while (parts.length) {
> > -			if (!(parts[0] in vars)) {
> > +            if (!vars.hasOwnProperty(parts[0])) {
> Looks like you're using spaces and this was written in tabs. Mind fixing and resubmitting?
> 
> —
> Reply to this email directly or view it on GitHub.
>
`ways( function(` => `ways(function(`

This is one of the padding exceptions. I'll let you sort the rest out instead of littering with line comments <3
yea, nice catch.
Why not "Sys.openURL(uri);" from LWJGL? Works good.
Line 227 (btw - read the annotation =)) - http://java-game-lib.svn.sourceforge.net/viewvc/java-game-lib/trunk/LWJGL/src/java/org/lwjgl/Sys.java?revision=3778&view=markup
Please add test case?
We use Eigen's default storage order, which is column-major. It is therefore a good habit to also write loops in that order.

Better yet, use Eigen's specialized operations, e.g., [`unaryExpr ()`](http://eigen.tuxfamily.org/dox/classEigen_1_1MatrixBase.html#a23fc4bf97168dee2516f85edcfd4cfe7) in this case.
Not sure if this is better or worth it. I'm still a bit fuzzy on gst with respect to this stuff.
I'm not totally clear about the specificty rules, but I added `!important` because the `&:hover` wasn't working with out it.
Nit: This should be aligned with counts in the previous line.
We should really make these files pep-8 compatible…  (That's the official Python style guide.  It catches a bunch of stuff.  To run it, use:
pip install pep8
./bin/pep8 ispdb/config/configChecks.py
)
Does it require our autoconf on all platforms, or just with Xcode 4.3+? If it's the latter, we have a special `:autoconf` dependency to handle that case.
This is part of `std_cmake_args`
Also `patches` goes above `install` (by convention)
```{{ _('Make someone's day. Review a badge for') }}``` will also cause an error. String lesson continued :)

A string that is quoted with a single apostrophe eg. 
```python
'This is a string'
``` 
cannot contain another single apostrophe: 
```python
 'This isn't a string'
```
You can resolve this by using the double apostrophe for the string:
```python
"Who said that this isn't a string"
```
Pourquoi n'est ce plus une load function???
Pas d'accord. SSH est similaire aux protocoles Git et local pour ce qui est de la compression.
I agree, fixed that. How cool that github automatically adds that commit to this pull request! I guess pull requests are related to the branch, so when I change the branch it will be included in the pull request? (I am still new to githubbing.. ;)
prinzipiell macht die Idee sinn, aber die Stelle hier ist falsch...

Hier wird ja nur der link gerendert und nicht die default sortierung festgemacht.
es würde also nur beim 1. rendern ein link entstehen der nicht umsortiert (weil sortierung vorher = sortierung nachher).
Pretty skeptical about this approach. The extensionState object should contain the extensionId, so at very least we could include that in the pref name. But this seems like a hack to me anyway. We should use https://developer.mozilla.org/en/docs/Storage
`@returns requests 2` - returns at least two requests

-- wouldn't this be considered unexpected behaviour?
For code this is a simple, we add a  --length option to .perltidyrc.

Podtidy also has a --length option, but no .rc file option. So you are asking for a feature request from either Pod::Tidy (hard) or App::githook_perltidy (easy). Since I control the latter I built it into a recent version.
Another way to do this is to create pipe pair in the session when the connection is set up.
Then, you do not have to introduce the onconnect_attach_pipe method and the session termination does not have to terminate both pipe ends.
Probably a matter of taste... I would leave out `equalTo` as it is redundant and not more readable than simply `is(...)`.
OK, let's see if this works (sorry for making you test my changes like this :-) )

The selection is now done *after* the sorting.
`doResize` just basically says "have we gotten a mousemove event yet?," right?  If so, I think you could eliminate it entirely, since `newSize !== previousSize` can't be true without a mousemove anyway.

If you wanted to clean it up further, you could wait and not call webkitRequestAnimationFrame() the first time until you get the first mouseMove... but I wouldn't view that as mandatory.
Couldn't the callback be used to send the event, then?
Why not use the same pattern you have used in other places like ThriftServerDef.newBuilder().name("thrift")...
It might be faster to use Math.min and Math.max here.
This does not matter, all headers in monomap are obsolete anyway.

I will update all the license headers in monomap with the new ones from master in an other pull request.
male will be the default if the values is `None`, which may not be a good idea. Can we default to unspecified?
No, this is bad reason. There will be definitely also regular lists on our page. You should change default style of regular lists only if you want to design them in some way. Not for those special cases when we use list structure, but design them as no-list.
Removed  # Form settings comment.
If you're creating only one directory `FileUtils` is not really neccesary, you can use `Dir.mkdir('_posts')`
I think you want to use a period instead of a dash here.
nuke this boyo
remove whitespace please
Might as well keep spacing consistent since these are new files.
Yeah, that's right. I think I came up with this naming scheme at midnight...
This just caught my eye. Why is this happening in code? Couldn't it be part of the layout since you already separate the primary field from alternate fields?
Does this need to be here? I fear this will be left in without further consideration.
Inline style? Put in css above?
That's something at least - was your work CSS or background image based? If it's CSS (without the background image) based then, if it's possible would rather use that...
I added in the `return undefined` for clarity. I know it's redundant, but I think it makes understanding the code easier; you no longer need to infer what will happen if there is no `$prev` element.

Actually, I just realized that there's a bug in the if condition above. I'll need to check `$prev.length > 0` instead of `$prev` to confirm that there is a previous element.
增加make test命令以便运行测试案例
The #if #endif statements are not necessary.
%z -> %uz
The service name is hardcoded to zipkinui here.
`Map[String, EmailContent]()` should be sufficient
Chybný styl jako u předešlé aktualizace katalogu, má být "Probíhá aktualizace překladové paměti".
why use list? all deletes at least have to match on name, as far as I can gather
缩进有误
What about making this disabled by default, turning that on only if user wants that?
NM, I see: Only katello mode gets foreman.
yup, I may have messed up a merge but I had it for both create and update
Yes, it is quite nitpicky, but since I made you request the PR, I have addressed it with commit 05c941d .

Thanks! :)
why public ?

And please use camelCased names
I guess I was referring to this sentence:

```
You can also directly specify the units for both to resolve ambiguities
```

because you already showed two examples in the code section above that do this. Maybe just removing ``also`` would be fine.
Wow, I'm blind. Stared at that function looking for just that, and failed to see it.

So this test is then a little looser than it could be (since expected values may come from any step), but the expected values are all constructed such that it would be a pretty big coincidence for that to be the case. Works for me. :)
I think for this, a global is the best solution. Perhaps move it to util.cpp?
won't inv.hash work here?
这里不够简洁，不一定非要写 -1吧，编号从1开始也ok
never reused ?
remove open.
Are you using `as.character` to strip the names?  If so, `unname` would be more clear.
Can you document what this does in a comment.  Also rather than using hard coded number like `13` can you use the constants in IPython.utils.js?  Actually I think those haven't been merged yet, but will be soon - so maybe let's wait on merging this until that other PR (#1711) is merged.
Should DeprecationWarning be used?  This is a question for the IPython guys. DeprecationWarning is the correct warning, but due to a Python core design decision, it is not shown by default, so that almost no one would actually see it. What is your policy on this?
Here you need to cater for the use case that 

- the user is using stdin for the source
- the user has not entered a language (using -l)

The language cannot be inferred from the filename as there is none, so there needs to be a check and either:

- a default language
- an error message

Which do you prefer?

Agree with supported languages and the brittleness of prepending .
This would be a great example for the README, however it seems too specific to be part of the core functionality of the gem.
are these mappings still needed?
These tests should probably use mock to mock out the attributes that FirefoxDetails gets from ProductDetails so that they don't depend on whether product_details has the right info in it or not.