labofsahil/github-event-dataset-2012 huggingface dataset:
Changes the implementation of the post function

refer issue #91.  I arrived at the same solution independently (issue #119).
proposed change of block error message container from P to DIV, to allow...

... better css display property control
ARC & NIB compatibility
Code formatting consistency only

As suggested, here's a pull request only for the style-related changes:

Made use of spacing, semicolons, and if-statement breaks consistent.
Also changed somewhat-misleading "trigger" function name to
"optionallyTrigger".
Another simplification of default.rb

There was an unnecessary block of code left which replaced 'rails/all' with single includes. Using the -O option to skip ActiveRecord during creation makes rails generate multiple require statements.

will_paginate is _not_ a requirement per se (there are other solutions) and should therefore not be included by default. Adding 'neo4j' to the Gemfile makes bundler auto-require it on application startup (no reason to require it in application.rb)

Note: The last commit (and pull from your side) did for some reason not get published on your gh-pages.
php-github-api is not a v3 api

php-github-api is not a v3 api
heatmap now from blue to white to red

w/a new param dynamic_range=true sets range as now, otherwise, use min=0, max=100
Bug 715092 - Mozmill environment should return the correct exit code

For details see:
https://bugzilla.mozilla.org/show_bug.cgi?id=715092
Update master

Github er sært. Anyway, tror jeg har pillet ved SNES.
Jb dev boost aries
Release 6.x-1.1

Hello,
Yesterday I released version 1.1 for Drupal 6. It contains quite some enhancements end improvements.

Regards,

Lode Vanstechelman
[TIMOB-7511]Camera returns low resolution images when in edit mode.

This fix uses the cropRect info to get the best possible resolution for the edited image.
Test is in JIRA
[TIMOB-7511](http://jira.appcelerator.org/browse/TIMOB-7511)

FR should include running this across current IOS line (iPAD1, iPAD2, iPhone 3GS, iPhone4, iPhone4S)
Process dropDatabase errors

Adds handling of errors produced by `dropDatabase`. If additional error information is available (e.g. `assertion` and `assertionCode`), those will be copied to the error object provided to the callback.
mingi pealkiri

test :D
Fixed uldum creatures with the first letter of the name 'B'.
Add migrations for import and add some active scaffold stuff [2/29]



 ...120801015019_barclamp_import_clouderamanager.rb |   24 ++++++++++++++++++++
 1 files changed, 24 insertions(+), 0 deletions(-)
Allow other test environments

Include other test environments
Various performance improvements.

- Defer customLinks allocation.
- Avoid copying strings in attributedTextWithLinks.
- Cache the result of the link color selector check.

Please see the individual commits for descriptions.
Better pointer format specifier mismatch message.

Fixes [issue 7480](http://d.puremagic.com/issues/show_bug.cgi?id=7480).
Upgraded Twig to the latest version

I just upgraded the Twig to version 1.5.0
Add jasmine-fixture to the set of bundled extras

jasmine fixture is sexy. Way sexier than jasmine-jquery for setting up fixtures.

I mean, check out the affix method. zomg. I drooled all over my keyboard when I saw it.
Limit options

Hi, Rubiety

I made some changes in validates_lengths_from_database to allow put limit options into ruby code instead of put limit into db schema/migration.

Would be nice if you will merge this changes.

Thanks Robin
some inequality tests to check ampersands are not getting swallowed

Also 'foo&bar' -> 'foo and bar'.  This is maybe not important, but happily works with my fix (because it introduces spaces on ampersands)
Please, consider pulling it in...

Fixes #25
LPS-25715

please backport to 6.1 for licensing
LPS-24812

LPS-24812
asthma management project
maybe this should be removed

this line is not exist in original arrays.xml

https://github.com/IceColdSandwich/packages_apps_ROMControl/blob/master/res/values/arrays.xml
line 299~303
I added a dash to your readme. Makes sense to have one there.

I really just wanted to try this out though. You don't have to keep it. :)
Update DefyParts/res/values-pt/strings.xml

Update for Portuguese translations
Recompile tess2 for armv6l and minor make file fixes.

All polygon examples now work!  Fixes #51 and all examples that draw polygons, paths, etc.
The log in can distinguish between admins and clients and each will be brought to their pages. the admin can also now update the about the baker page.
Small fix in CSS

Hi there,

There seemed to be a small bug in the css file. Using Firefox, the display window spans the whole page, and using the custom renderItemHTML function as provided, this extra length was turned into extra height. Either way, forcing "display: absolute" on the UL fixes this problem.

Thanks for this app - it's awesome!
Documentation refactored

I believe that for new users current documentation is a bit hard to follow so I propose this refactoring of documentation. I haven't deleted anything, I've only added some info on getting started with SonataAdminBundle and tried to describe installation procces as short as possible :). Improvement ideas and thoughts if any are very welcome ...
Фикс для обновления капчи

Алсо, отключил проверку обновлений, по-моему она работает как-то не так. У меня с самой свежей версией из транка постоянно возникает окошко об обновлении.
add Toggle Comments support

see: https://gist.github.com/864839
Sigur Ros - Hoppipolla

The soundtrack of the universe.  There's a music video, but it has worse quality audio.
Bruce Springsteen - Badlands

The Boss has to be at every list of fuckingawesomesongs...
handling landing on a specific picture. getting newer photos
Ics

battery mA fix
Extended the timeout for the search test
libaudio: Updating device volume as per UI volume level

- Volume level in the call is not matched to UI on audio output
device change
- Current computed volume level is not getting updated to the
device whenever device change happens.
- Updating the device volume with current computed volume
irrespective of device change.

CRs-Fixed: 291369
(cherry picked from commit 6f99b7b6ccea5c0e8c2fd791752eea0de91bdc9d)

Change-Id: I3a7bf882d45345a7b470bc7dde9331dcb10901b1
Changes to code to reflect Moodle's coding standards.
Avoid PHP direct array access

In order to make the command integrator work with all Magento installs, avoid PHP 5.3 only direct PHP function array access.
Also add a concrete example to the readme.md.
Added ability to set a function which can generate more complex html for each option
Changed comments to PHPDoc comments "/**" and removed whitespace

My Netbeans IDE didn't show any documentation for the twitter functions.

After a quick look in the source i saw that the functions where using "/*" instead of the official "/**"
Causing the tooling to ignore al the nice @param typehints etc.

see http://en.wikipedia.org/wiki/PHPDoc for more info
Compatibility with ActionBar (or SherlockActionBar) and not force background

The actual code doesn't work fine if the root view has not background or the activity has a Action Bar.
Allow $LOCAL to be overridden by specifying PREFIX variable, and ensure ...

...that $(LOCAL)/lib and $(LOCAL)/include exist and are directories.

Example:

    mkdir /opt/custom
    make install PREFIX=/opt/custom
Add Vapor.js

Why?  When performance and minimal loc are important, vapor.js just cannot be beat.
Improved code for restoring exported role 'id'
remove TitleProvider and use getPageTitle() from support library

for more info look at the commit message from 81065a6e
Added README and some small naming changes

Hi, I have added a README to explain the approach of adding a Characterization test to ensure that the refactoring doesnt regress the existing behaviour.
morgue

attention ce fichier ne contient que la 2e partie de la trad
Not sure why this happened, but removing the gameTab call fixed it.  That call doesn't need to be there anyway as it'll be called in FindNormalRandoms.  I tried debugging and narrowing down the issue, but I'm mind-blown and I don't want to lose sleep over it.  It no longer clicks the main screen when logging in, and that's all that matters.

I suspect this is what's causing the Alcher to "walk" to random places on the screen.
Should escape $attachmentId.
This is not needed, OptionSet has a method to output usage details. You can supply descriptions to parameters in the options definition.
Yes, the test are broken now.

I could put the 'ordered' argument into simper, which would save the extra documentaion for summary.simper, however this is inmy opinion not that intuitive. Can´t the two documentation be combined?
Well, if there are further comments it can be changed. It looks ok to me and it is a good improvement to have *something* written. Thank you.
Hi Matt,

Looking into it. Thanks.

Bruno.

2012/1/27 Matt Crider <
reply@reply.github.com
>

> Hey Bruno -- not sure how, but this appears to cause problems with OCS.
>  Try clicking on the 'Site Administrator' link from user home --
> authorization fails and you get redirected to the conference home page.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/pkp/pkp-lib/commit/f3c9cfdea7bb16ac14efee7f54a693d96ef5fe20#commitcomment-909940
>
No, but I noticed that the libev guy committed a fix for this issue a couple of days after the initial version of my patch.
I lied, I moved everything from here to the API, didn't add events here.
Dir.exists? seems to be an alias for File.directory? http://ruby-doc.org/core-1.9.3/Dir.html#method-c-exists-3F
private/final
so was that a fix to #42?
Apologies, I will pay attention to this in future.
Hi, thanks for the feedback. In principle I agree that it is probably better to have more extension points than implementing everything in the core. But in this case (checkout handling) I can hardly imagine many more use cases, and that a shared workspace is required to reasonably use this option is well documented.

> But what if another build environment can't do concurrent writes to a shared workspace?
If someone really wants to checkout for configurations and not for parent, he will hit the same issue I fell over in my first attempt: SCM polling as a build trigger will not work. This is not obvious for anyone not familiar with jenkins internals, exposing it as an extension point might lure more people into this trap. Also this use case is not that beneficial since is saves only one instance of the repository, while the other way around it saves multiple instances (assuming that noone would configure a multi configuration project with less then two actual configurations)

> But what if another build environment can't do concurrent writes to a shared workspace?
Then the user needs to work around it (e.g. by running the builds sequentially). The feature clearly warns the user that the slaves need to share the same workspace, the implications should be clear to the user. Jenkins is a sophisticated tool for technical people, so while I would not expect the end user to have insider knowledge in Jenkins (e.g. required to understand why discarding parent checkout disrupts build trigger SCM polling), I would expect him to understand those thing not Jenkins specific.

I can try to work on extension points for this purpose, but it will surely take time. I'm neither too familiar with Jenkins, nor am I mainly a Java developer, nor do I have any experience with groovy or jelly to add the required UI options; I'm working as build/test automation engineer to support a project for multiple platforms (some embedded) and a frequently changing set of feature branches.

For now I created a new branch from my fork, thors-integration, which contains my modifications and will be re-based for new official jenkins-releases, so I will keep my patches in sync with mainline. In our company I'm using this fork now. Of course this cannot be a final solution, but it will take some time before I could start working on your proposal, and it would definitely require me to learn more about Jenkins internals, jelly and groovy. And looking at the ticket JENKINS-4960 there are some others that are waiting for such a feature as well.
@envygeeks `select-editor` is a Debian (and derivatives) specific command was built to make the (Debian-and-derivatives-only) `sensible-editor` command easier for newbies to use — If we want pry to support the `sensible-editor` mechanism, we can just try to call the `sensible-editor` instead of just `editor` (which is yet-another Debian-specific-editor-configuration-related command).

See: https://github.com/ConradIrwin/pry/commit/7772c738a5f5a3b0677e5e5af485f8c0ec35a578
Maybe this should be static.
Oh, you did later. Perfect, thanks!
Yes, it would be much more cleaner if it can be done in Jump() itself.
520? and 250? those are correct, right?
The first one would be easy to filter with something like `/^(.+\/)*(\d+-\d+-\d+-)?([^-].*)(\.[^.]+)$/`, but I don't really see the point: Currently the only validation that Jekyll does on the filename is the date, there are no other validations (you could have a file called "2012-01-01----foo.markdown" or even "2012-01-01-2012-01-24foo#@bar.markdown")

So why should we add those restrictions?
Ahh, thanks, was not aware of that.
Is it true that you're allowed to pick up two tiles away? Ablu was saying one tile.
Just realized this should really be behind a waffle switch to we can easily turn the promo off when MWC is over and revert back to the Aurora promo without another code push. I may need help with that.
I thought it would be a nice shortcut if one wants to force URL recognition. Accessing file:// scheme URLs is not that common, I think. The // syntax is also used in HTML (but for I slightly different purpose, I admit).
Why increase this when a new thread is spawned to run the task anyway?
This commit doesn't seem to do anything. Appears to just be "pseudochanges".
For future reference: column is spelled column.
just curious why line 117 is blank instead of fn(exception)... thanks!
改成page的一个方法的话，还能用wiki_get_write_permission()，以及将要改成group方法的wiki_is_in_group()吗？
it took me a few seconds of reading this to realize it was a purely semantic change; perhaps in the future I'll take the time to de-dense-ify it, but I don't see a reason to screw with it for now.
oh my god all spaces converted in tab o.orevert this plz -.-
This is kind of a big chunk of code. Could it be pulled out into a helper function?
on Tue Feb 14 2012, Mike Anchor <reply+c-966538-fac31d356722063bf93b585de0cb6c1fb168952d-44065-AT-reply.github.com> wrote:

> Without really knowing a great deal about inline functions, how is
> removing static and keeping the function in the source for Lion better
> than allowing Lion to provide it's own version?

On second thought, maybe it wouldn't.  I don't remember why I thought
that was better.

-- 
Dave Abrahams
BoostPro Computing
http://www.boostpro.com
Sure was hacking with fb on it last night. Back it out or fix it. Your call.

Sent from Mobile.

On Feb 16, 2012, at 5:29 AM, Ben Francis<reply@reply.github.com> wrote:

> Your manifest.webapp is a mixture of Google Maps and Facebook and we want to use touch.facebook.com, not m.facebook.com. But this won't work yet due the framebusting bug we're working on. Suggest backing this out and putting it in a pull request?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/andreasgal/gaia/commit/fc4bd2ccf110841a8396273d39048800c8dd78f4#commitcomment-973707
CompileError: nvcc compilation of /tmp/tmp2ru5rp/kernel.cu failed
[command: nvcc --cubin -arch sm_11 -I/usr/local/lib/python2.7/dist-packages/pycuda-2011.2.2-py2.7-linux-i686.egg/pycuda/../include/pycuda kernel.cu]
[stderr:
kernel.cu(7): error: identifier "uintptr_t" is undefined
have you post that rewrite in mangos Forum?
This defines values as default or overwrite config setted in with "cmake ..  -DPREFIX=/opt/mangos -DNDEBUG=1 -DCMAKE_BUILD_TYPE=Release -DACE_USE_EXTERNAL=1 -DUSE_STD_MALLOC=1"
A link to the git repository seems better :)
you could make this a jquery function?
oops missed one.
Actions 全放在一起……
@stof Awesome. Really awesome. Thanks much.
Kevin and Tony,
Agreed. Generally always better to centralize stuff. Just wanted to make sure Tony (if he committed) brought the changes into a master based branch (via cherry-pick) before pushing it to sourceforge.
-brady
Ha, ha, ha! Just because it worked out of the box with my IDE. I spent only a few hour on this.
May not want to accept this pull request until it has it's own package directory.
Distribution management section should be this only going to 3rdparty.  Cheers


	<distributionManagement>
        <repository>
            <id>atlassian-3rdparty</id>
            <url></url>
        </repository>
        <snapshotRepository>
            <id>atlassian-3rdparty-snapshot</id>
            <url></url>
        </snapshotRepository>
    </distributionManagement>
Active Record here.
or `self.data['description'] = ''`
this is redundant. because if line 448 passed (we were successfully redirected to `edit_url`) that means that there were no form errors.
I did test this with myself using http://www.hidemyass.com since so it would not be local to me and another guy on a different ISP tested too.  You will have to create a Firewall: NAT: Port Forward to open WAN ports HTTP - HTTPS to Redirect target IP to ports HTTP. Your Dynamic DNS hostname will now be redirect from HTTP to HTTPs if HTTP is used when HTTPS Protocol is enabled for the router webConfigurator.
should this be `password`?
It will load only for IE8 and older versions.
Yeah; it makes sense.

Thanks,

On Fri, Mar 9, 2012 at 5:00 AM, Philippe DENIEL <
reply@reply.github.com
> wrote:

> Commit 5f65ea58b6fc1000b5bfdfdc3a5dcb632dbc3ae2 fixes this.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jvrao/nfs-ganesha/commit/8e0c6d87b05e5cfb2f70c403d97130bcb6201122#commitcomment-1065448
>



-- 
Jvrao
---
It feels good to be important... but, it is important to be good....
плохой комментарий к коммиту. он говорит о том, что изменилось, но не зачем оно так изменилось. что main() поменялся, и так видно
Yes, I made manual tests, but you're right, we should have a big page written in markdown with lots of cases to check that there's no problem with complex examples.
I will add a complex check.
0. Chrome 8 has less users than WebTV. There's no sense in adding code to support a 98-generation deprecated browser.
one thought i did have would be to use it in config.ru, but then we lose the ability to use AppConfig :(
![ydig](https://www.joindiaspora.com/images/404.png)
why are you creating a new `<table>` for each `id`? seems like you should be creating a new `<tr>`.
Seems I do...the edit listing view flakes out otherwise
Hmm ... is this because there is no way to craft a request that is definitely processed by an app engine proxy after the reprioritization of app engine vs. trusted peers?
looking into that now
That's correct, fixed now,
I will create a test case and apply your fix.
Going on a limb here and approving... wondering where Dmitry disappeared.
Search and replace fail :(

Should refactor the '4' out *somewhere* as a constant--not just so it can be changed more easily but so someone reading the code has any idea what '4' is.
During install I get an error that class Install can't be found on line 4.  The error goes away when I comment out lines 4 through 6.
Please expand max here too.
There is no test in this pull request.

User's email is not updated, althought it's not needed to fix the issue it might be better to do the change update_from_AMO as in https://github.com/mozilla/FlightDeck/pull/104
The *it "should traverse a relation"* spec is hanging for me in a manner suggesting the connection pool is exhausted. If I comment out the other specs above, it works, and they pass without it. It appears to be hanging on the reload of link 1. Is this spec passing for you locally?
The only issue I've had before was with the android_prebuilt repo syncing but since then has seemed to have fixed itself.
Thanks.... :)
Maybe, but then it runs the risk of some one uploading a cached copy with the wrong urls. This way it is only changed locally no issues about cache files.
元のソースコードは下記の通り。StartIdx, toIdx, stpの値によって昇順または降順に探索するループとなっている。

    For i As Integer = StartIdx To toIdx Step stp
        Dim id As Long = Tab.GetId(i)
        If id > -1 AndAlso Not posts(id).IsRead Then
            Tab.OldestUnreadId = id
            Exit For
        End If
    Next
"...the name of the language is included in the tag.." is less cumbersome than my original and probably no understanding would be lost.
It will probably be moved to public records soon after a definite decision is made...
This adds when renewing a session (for example from a disconnection) to enable the corresponding widgets.
Originally this was enabling  everything, either  the contact was blocked or not.
why are you simulating a left click here hijacking the event id and emitting another signal (which will be catched by this method, if i am right (?))
Actually, maybe that makes sense - since it's help for the module settings rather than the module itself.
> чем обоснуешь использование id формы для доступа к элементу? лучшее решение?

Может быть не лучшее.. Возможно, лучше на класс повесить, чтобы все формы с этим классом подхватывались js-скриптом.
Disabling the buttons NewTask and ImportTask is ok. But the Backlog should always be present. It should be possible to drag undone tasks from the scrum board back to the backlog. So we have to disable the possibility to drag tasks from the backlog onto the scrum board if the sprint is in the past.
Because the route command can hang. Unless through some form of magic it's been fixed by the faries in freebsd 8 or 9.

It was a current issue on 1.2.3 with freebsd 7.

It blocked all filter reloads from that point on. 

typed on a tiny touchscreen, why exactly?

Ermal Luçi <reply@reply.github.com>schreef:

>Can we please stop dumping the route table!
>Just use route -n get if needed!
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/bsdperimeter/pfsense/commit/1716682b1d1a6efa6e97bb0874e89e6132c55dfd#commitcomment-933017
You need to define the index in the embedded document.
ARGGG... I am not sure what happened but now I can't push changes to the main branch. Sigh... :-(

I get time travel rejection when I push AND I can't pull your changes... no joy on both fronts :-(

eddie@virtualchurchill.com
Hey guys,
I didn't mean put your files in the README. I meant put your files in the project folder.
This should fix #012.

See my comments.
Weil wir dadurch die Möglichkeit haben auch in Contao den MinestarPlayer zu benutzen. Die Gruppe holt er sich bei einem Offline-Spieler direkt vom GroupManager.

Das ganze entschlackt Contao dann doch... ziemlich :)
this comit breaks about 10000 tamarin testcases.
I don't know if it's correct, but if i move the i++ statement inside the while loop after the check, 
all broken testcases are working again:
<pre>
@@ -2225,9 +2237,9 @@ void SyntheticFunction::acquireScope(const std::vector<scope_entry>& scope)
 	auto end = scope.end();
 	for(; it != end; ++it)
 	{
-		i++;
 		if(i > mi->body->init_scope_depth)
 			break;
+		i++;
 		func_scope.emplace_back(*it);
 	}
 }
</pre>
Just pushed a change to revert this change back to how it was originally, should be right now
html and body tags are not required as we have already used these tags in base.html
Why not to write smth like "Copyright (C) 2006 till now"?
Not really a change, just a rename
After the patch? Thought I got them both...
![this](http://imgupld.lunaticoutpost.com/graphic/images/2011/July/08/95B8_4E16EEE5.gif)
This looks like crap.
That looks suspiciously like we didn't implement `continue` at all before, right?
Perhaps a nice motivating example would be to give an example of the collections hierarchy when parallel collections were added there weren't yet any Gen traits? For example, in the old design, you could have some method on Seq which you could pass a ParSeq to. (Not sure if it's overkill to add all of this discussion though.)
Because It could be Modified between the time you Define it and the time you use it.
+1 .. link_tagret ... nice one
Thank you for your co-operation
Great! 
I'll start work on several quest fixes that were just waiting for this.
in db achievement_criteria_data  entry with type 2 needs to be checked if achievement requires source or target. 
if source(example realm first class,race) type needs to be updated to 21
I disagree with this codestyle and few scripts.... just for easy view say Event type and Action type... Name - Event - Action. I don't use creature name in commit...coz entry is the key for me
Doenloads no Google Code
Hi,

transaction_id exists in the paypal response only when an actual transaction has been made. 
In my own project I get transaction_id after made payment request with request_payment method and use this id when getting IPN related to this transaction.
Having a cost for rail destruction is insanity. Good fix @master-lincoln
good job, could be done on SAI but its still a great job
This check is for non-static inner classes. Since you need a reference to parent class to instantiate them, there is a decision to be made.Shell we try to instantiate parent first, then discovered class. This seems to be messy. Other option is completely ignore the class. I've chosen to instantiate super class. Some of test cases from spring module were using anonymous inner classes, which were extending top level annotated class. Pre-fix behavior would have instantiated them. This approach is most compatible with existing code.

While is there for cases when inner class extends inner class which extends top level class.

As to isInstantiable, I see the value of adding it as third check. Would you agree?
I see some unproper stuff here...

for example '#include <linux/version.h>' should have been removed.

You might want to try the one on my repo, is slightly different of your and works perfect so far. :)
Not sure if this is going to fix the problem I'm having.  That problem being a large portion of my debugging text not showing up in debug.log.
Yes, I've made the notification icons widget vary its width, and, though I've not hit this case yet, it looks like when the row runs out of space it would cull the entire widget (or some other widget) - it'd be better to shrink this one first. So, perhaps a flag or two that say: this widget is stretchy horizontally/vertically.
this.audioInternalBuffer should not have existed.
@candidasa  This style of inline commits will not be detected by phpDoc parsers, please stick to standard phpDoc.
Спасибо, что ответили, но я, честно говоря не понял. Я полный ноль в программировании и не могу понять то, что вы написали.

Я не знаю как в контроллере что-то передать, но все же решил немного поковырять. Открыл imgae.entity.class.php и добавил туда класс:
<code>
    public function getTitle()
    {
        return $this->_aData['album_title'];
    }
</code>

и

<code>
    public function setAlbumTitle($data)
    {
        $this->_aData['album_title'] = $data;
    }
</code>

Чуда не произошло -))

Не подскажите в каком направлении двигаться? И было бы правильнее по дефолту передавать инфу об альбоме просматриваемого изображения.
Latest version already has a fix (original code is in fedorahosted), but I will likely revise it for your version which has a cleaner way to do it.
In that case the app will freeze if called from UI Event Dispatch Thread, since there is no checking condition for that. (isEventThread).
i implemented the container approach now, and it works.

can you create an issue with the other aproach? while it would be nice, i don't know how to do it and think there are more urgent matters to adress right now.
It seems to be missing as well on the HAML and SLIM version.
Also added a ton of comments, because commenting your code is something everyone should do :D
Note that `_METHOD_PARAM` is on `Request`, so the above's wrong
Hey, cyanogen, why this change?  It messes up the default Auto Brightness settings and makes the Vision behavior inconsistent.
Very nice bro is there still a border around the notification drawer with the bg color fixed ? Also the navbar needs to enabled for nexus by default otherwise it should be disabled for other devices.  Working on that overlay now for you
bwaaaaaahhh
Curious why you changed this to have the Window created on demand instead of re-using? Does this perform better on Android? I noticed it was not changed for the iOS version.
Cool, yeah personally I think it makes sense to just remove the `--prompt` option completely.
Sorry, I've been reading too much about AMD so I am in an eliminate all global variables mode :-)  You may choose to ignore me.
That's the canvas size itself including the navbar. I'm not sure that it can be relative to the navbar itself as it includes it. In SmartParams.Simba, Smart is initialized with those values or else it will be too small and the canvas will get cut off.
Also ImageSetClientArea does that already. It changes the canvas size virtually so whether or not I change those values, it won't make a difference.

Good idea though.
Why you convert bool to string ?
good, just wanted to be clear about it ;-)
Oh yes! This is inside a config parameter, forgot to use it here. Thanks!
call to super.postStop() ?
This is a bit too opaque.
Yeah, just add additional commits.
note that _map is in the EnsembleSampler class.
`g` is imported but unused.
On this +1 instead. Very good refactor.
@tobes Agreed. How can I get the `data` attributes onto the `<img>` from within the template if the helper is returning this as html?
As with before, we can be PEP-8 compliant by adding a blank line below "import math".
This is an interesting case. Does the publish task even get created if this happens? Is it also marked as rejected? Is the rejected concept group-wide or can individual tasks in a group be rejected and not affect the overall execution of the group?
This part is a small copy/paste of Native.LoadNativeLibraryFromJar().
In theory I agree that ignoring exceptions is a bad pratice, but such pattern is very common in finally blocks when a stream has to be closed (because there should be no error here, and even if there is one we do not really care).
Isn't this done in the CDirectory class, so shouldn't be needed here?
Waarom 0666? HOewel hij wordt genegeerd, suggereert dit dat het readonly is.
Please remove references to virtual_field. It's not used.
Will the conversations codepath know how to reply if this header is not set?  It's required for normal endpoint operations but conversations probably supercedes it.
I think the hard part is just the search for, and combining of, methods. We can add an entry for every method that resolves to a particular handler into the map.
... there are so many magical unicorns out there that I have yet to catch. Thanks for showing me the way to yet another!
Is this always necessary or only when logging in with the email address instead of the user id? In the latter case, this should be an authentication hook example for Ingo instead.
I don't understand what you mean. I'm suggesting this:

```javascript
var now = Date.now();
if (!this._idleStart) this._idleStart = now;
var delay = this._idleStart + this._idleTimeout - now;
```

Which could be simplified further.
Also, doesn't this reassign λ00 = λ, φ00 = φ for every point, not just the first point as implied by the comment above?
Okay, I'll add that. However, if the user resource request cannot be satisfied (there are not enough free cores) we won't be able to tell the user in advance.
This is a bug because it will cause integer overflow for datasets with more than 2^31 - 1 elements. That's why we were returning a Long before. You should probably convert the sizes to Longs. Also, I'm not positive about this, but it's possible that Iterator.size() is slower than the while loop there. It would be good to test that.
Maybe mention the business day functionality specifically? That's completely new in 1.7.0 IIRC.
I think this should be reverted to return.  It's a void function.   Also, it's O.K to not put goto fail --- sometimes this is done in NumPy in a function before allocations have been made.
Not sure whether we care, but if it's desireable to keep the source files as ascii, I think this line could be written

```
expected = u"array(['\xe9'], dtype=object)"
```

and work on both py2 and py3.
I believe you could do this as a one liner :

    def as_json(options = {})
      root = options[:root] || include_root_in_json
      [...]

It would actually make more sense, as this makes options a hash at all times, so we don't have to manage it being nil.
@rafaelfranca, i remembered why i use `flatten`, because i found same method inside [reverse_sql_order method](https://github.com/rails/rails/blob/v3.2.8.rc2/activerecord/lib/active_record/relation/query_methods.rb#L394). 
So i thought this two methods should be symmetrical
And now we have a much better test + a test on the nil case. :-)
Why did you rename this function?
unreachable?
I'd probably remove the "Scala API" note. :-)
I think you are probably correct on that one.  Should I make the change and push back up to my repo, or do you merge and modify?  I've yet to do one of these 'modified' pull requests.
Have you actually tested this class. It seems that it returns a `Mul(cos(phi)*cos(psi))` objects, and that `BeamExpansionFactor` classes are never created.
>
>
> `expand_mul` will exactly help.
>
> I was hoping that Poly embedding will remove `expand` at all.
>
>
If you use polys, it will expand by default.
Yes the intent is to check whether this equality is always `True`. The issue referred by @rlamy about `bool(Relational)` does not mention any problem with `Equality`. In fact `Equality(x, x).__nonzero__` seems to do the right thing if you look at its definition https://github.com/sympy/sympy/blob/master/sympy/core/relational.py#L217 that basically corresponds to @asmeurer's suggestion.
You should include trailing commas a the end of long lists like this so that you can avoid fake "deleted lines" like the one above in figure additions.
If I remember correctly, the issue here was that mul was being applied before multinomial, so something like `x*(x + 1)**2` was not being expanded all the way.
No that's ok for now. But please consider it in the near future.
The amount of magic is rails is a bit intimidating. Thanks for teaching me about this.
Please choose a shorter test name, e.g. `test_broken_import`. Use a docstring/comment if you want to be more specific about what is being tested.
Since functions are writing to the transaction, they don't return anything.

Since they don't return anything, it's better for them to return None than empty-string, so that we'll see TypeError if/when we do it wrong and try to use those return values. The Right Way to get a return value is to use the new $capture.
chunk?
`php -S localhost:8080 -t web web/index.php` should work
there is no such service
== NULL
prototype form
:option padingFactor:
All values in the QueryDict are lists:
request.POST
<QueryDict: {u'form': [u'rev'], u'title': [u'Modified test'], u'tags': [u''], u'show_toc': [u'on'], u'current_rev': [u'25', u'25'], u'content': [u'<p>foobar</p>\r\n'], u'based_on': [u'25'], u'csrfmiddlewaretoken': [u'ce0d0f1bf1352ecdcbebf4e23e5ad8bf'], u'slug': [u'modified_test'], u'review_tags': [u'technical', u'editorial']}>
oh, no that's right. this is only when there are no revisions left.
Shouldn't this just be `&&`?
`REGEX_VAR_ARRAY` ?
Unique key will ensure sphinx server can be used for multiple phpBB installations. I will generate a new key
Isn't "should not" a suggestion, whereas "cannot" command? I agree using "should not" in coding standards suggestions, but in a matter of not working code we should really use "cannot"...

IE error:
> Expected identifier, string or number

while the code was:
```
var obj = {
   a: 1,
   b: 2,
}
```
Print the warning and commit anyway, same as what commit-msg does for unacceptable commit messages.
might want to upgrade this to RC when adding all these dependencies? Or should we do that separately?
Because we don't use a namespace, the leading \ is not needed. Also, if you add a use statement at the top you don't have to fully resolve the namespace here.
you mean S_ prefix on template variables? this line mentions prefix on a php variable
This also seems to duplicate acp js.
Lots of logging at the INFO level here, probably needs to go debug and have a single message for the entire transaction.
This interval seems a little long- or at least I expected the agent would check for an upgrade close to startup.  I could imagine some clients never being able to upgrade.  Whats the rationale here?
please add an unit test for this piece of code
It's turtles, all the way down!!!!
@drewfish The things to consider are how where templates, partials, and helpers will be cached, and what type of sandboxing you'll want with these things.

If you use a standard YUI module dependency `require: ['handlebars']`, then the `Y.Handlebars` object and any templates, partials, or helpers that have been registered to it, will only be available in that sandbox (I think, this should be tested because Handlebars is an odd duck since it's an embedded project in YUI.) This is what I'd suggest doing as a starting point. If you need more flexibility and control you might like to use Node.js-style: `require('yui/handlebars')`.
I don't think this is right.  The trick is that `config.prefix` might be empty, as we need to support this use-case (or at least we used to).
OK yeah, for the re-use of the `perf` variable it does make some sense.  (I've seen other cases where a comment does nothing except explain the next line of code, which is a bad comment in my opinion.)
indeed, this must be fixed, what is the procedure? I fix, commit, and make new pull-request? I wonder how python parsed that one, it looked like the result was correct.
это ты про таблицу статистики?
Да, твой вариант работает. И время выполнения пренебрежимо мало.

Мой вариант ты неправильно написал. Надо так
INSERT INTO tt (date,name,count) VALUES ('2012-11-14', 'name1',4),('2012-11-14', 'name2',10), ('2012-11-14', 'name3',40) ON DUPLICATE KEY UPDATE count = count + VALUES (count)

Причем ты дожен создать составной индекс на поля data и name. И мой вариант лучше ещё по одной причине. Когда у тебя не заведена ещё запись для нового дня или баннера - в твоем варианте будет ошибка. В моем же просто создастся новая запись с правильным числом показов и датой. И это преимущество очень ценно, потому что намного облегчет жизнь
I'd suggest mentioning ARMv5 as well ('...if you're working with ARMv5 or ARMv6 CPU instead...')
Ah, but I didn't say move to steam.xml. Should just be noted as steam only in the changelog (should have been clearer), and in the compatibility list.
There's still the other file =)
space also seems not to be used in sphere_searchlight, then. My copying of that was predicated on the assumption that it served a hidden purpose.
please remove the below file ext-all.js... it's already present:
https://github.com/Raxa/Raxa-JSS/tree/master/src/lib/extjs/
Если есть условие, которое дает вложение, а далее нет ничего, лучше заменить на обратное условие с выходом из блока.

Просто меньше вложений и, следовательно, читаемее.

А вот остальное, конечно, вопрос предпочтений.
Check for the actual permission that is needed.
Сюда бы тоже logging добавить
There's a check in #download if the tar exists before downloading.
Missing documentation.
Missing line break between param and return
Missing line break between return and throws
As we discussed, I have not yet seen a requirement that specifies that we need to distribute these calculations with Celery. I would prefer to first see a simple, single thread/process approach (that runs in the `control node` context) that just works. Then, we can let the heavy users pound on it and see how it performs.

Only then should we look at optimizations, because we don't yet know what the bottle necks are (memory, cpu, i/o, whatever).
`rel2abs_page_name( path, name ) {`
> to lists of package descriptions 

In this context, a package description is a `SourcePackage`, which is defined in `Distribution.Client.Types`.
No necessary, my mistake.
However, while the changes to webvtt_strerror make a lot of sense, it's already been explained why the error list is not a very good idea.

It's more code that we have to test
It's likely not going to be anywhere close to as good as data structures the user can provide for themselves
It's not actually even needed for the parsevtt test application, as all errors are logged to the console anyways (simply change the callback to return 0 rather than 1 if you don't want it to die on the first error it finds)
@miketaylr, still happening in Opera Next. To summarize: when a sub-resource like an IFRAME takes a while to load but the DOM of the page is ready, the other browsers report "interactive" as the readyState, while Opera reports "complete". 

Specifically, http://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#current-document-readiness says **non-normatively** : 

```
document . readyState
Returns "loading" while the Document is loading, "interactive" once it is finished parsing but still loading sub-resources, and "complete" once it has loaded.

The readystatechange event fires on the Document object when this value changes.
```
"Dieser Notiz Stichworte zuweisen oder entfernen" ist sicherlich näher am Original dran (tag_s_ also plural).
Ich muss gestehen ich habe bisher auch nur einmal drüber gelesen, also noch nicht wirklich Korrektur gelesen, hast du das gerade getan? (Für eigene Fehler ist man sowieso blind ;-) )
m3u does have the advantage that it is really simple and portable, and some custom ``# PLAYLISTNAME: foobar``  wouldn't actually cause any trouble for any other users for the m3us.
Your description says you removed this??
Same as above, this whole if block is not useful to a user and is more lines than the actual functional code.  Rely on exceptions thrown in the right place (update, REST) to report messages back to the user.  This whole method can be 4 lines:

    domains = rest_client.domains
    raise <exception, see comment above>
    domains.first.update(namespace)
    0

That's it.
T'as oublié ça
Так и есть. Поправлю.
description var names don't match the example
The following initializations should be refactored in another method (e.g. ``_init``) in the base abstract ``Notes`` class:

```javascript
Notes.prototype._init = function () {
    this._root = this.context.getRoot();
    this._notes = [];
};
```

The base ``Notes.prototype.start`` method would call this ``_init`` method.
The derived ``FloatNotes.prototype._init`` would call the base method and add its bit of functionality relating to the board setup:

```javascript
FloatNotes.prototype._init = function () {
    // call base method
    FloatNotes.super_.prototype._init.call(this);

    this._board = this._root.find('.board');
};
```

Now, the derived ``FlowLayoutNotes`` class would not need to do anything because it doesn't need to initialize any board.
Either of those is fine, too.  Your pick.  Thanks.
"`// Produce a random position, far away from the sombrero (used to spawn piranhas)`"
If we only show the checkmark when the status is STATUS_ACTIVE, then you could just remove it in all other cases.
Fixed. Code calls getFocusedEditor() both before and after retrieving data to make sure we're not holding on to a stale editor reference.
I think you can't always use `cursor.line`: when there's a selection, the cursor could either be at the start _or_ end of the range.  Seems like using `editor.getSelection().end` should work (as above, it's safe to use even when there's no selection).
This solution is good enough for now (or at least it's better than before). Our preferences stories haven't been refined enough, so I went ahead and captured the need for custom file filtering in the backlog. https://trello.com/c/OmPlSisf.
Use 'self.asssertNotIn('attack', column_names) instead. Save yourself some typing :-)
ah. maybe i'll have to edit them. ;)
Please use the new dialogs rather than old non-skinned dialogs.
Title should match header (h2).
Accessible for example under `/search`?
What happens if I try to use a non-flickr url here? The editor should probably report an error saying "use valid flickr urls or gallery ID's".
again, this is taken directly from the popcorn repo -- should it be updated even though the original is not?
We mostly use `echo` in this file, maybe use that instead
I thought so too, at first, but it doesn't look like it's necessary: https://developer.mozilla.org/en/JavaScript/Reference/Statements/throw
These line breaks are odd.  Why 3 separate lines?
Why is this getting added?
put `(function checkChromeWindows() {` befire this line and `})();` end the changed block. Also, could you mode that at the end of the file?
Ugh.  Fine.  But you should remove the last line since that might confuse people who don't know how IRC works and stuff.
Forgot to add: align them between oss and pro. Just as JD said

Thanks,
~t~ mobile
On Oct 26, 2012 9:39 PM, "Tamás Cservenák" <tamas@cservenak.net> wrote:

> Yes. Its used to align sisu and other deps as we had problems with that
> before
>
> Thanks,
> ~t~ mobile
> On Oct 26, 2012 8:26 PM, "Jason Dillon" <notifications@github.com> wrote:
>
>> In nexus-runtime-platform/pom.xml:
>>
>> > @@ -23,115 +23,114 @@
>> >      <relativePath />
>> >    </parent>
>> >
>> > +  <!--
>> > +  FIXME: This does not actually provide deps on the Nexus runtime, but only on the sisu/guice/plexus and basic support libraries.
>>
>> Fair enough, though I don't think that really warrants a separate
>> top-level pom just for that... but perhaps its needed by something in pro?
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/sonatype/nexus/pull/619/files#r1956129>.
>>
>>
>
Why not instead do something like this?:

    s = speaker(save=True)
    vid = video(state=Video.STATE_LIVE, save=True)
    vid.speakers.add(s)

    category_url = vid.category.get_absolute_url()

    resp = self.client.get(category_url)
    self.assertContains(res, vid.title)

    resp = self.client.get(s.get_absolute_url())
    self.assertContains(resp, vid.title)

Also, this test should really be broken up into two separate tests--one for the category and one for the speaker.
Now I see about the above two comments. Seems like you could change your function signature to:
TVar Int -> IO ()
Then you can remove the Maybe Int from tick and partially apply set (after switching the order of the arguments)
shouldn't deletes go before updates?
Hmm - ok - you've used tabs in your test files, and I have Vim set up for spaces.  You obviously want tabs, or just consistency?
Oo. Using an entire 8-byte float to store the time-zone seems excessive. Currently all values are stored in composite types as pointers to heap-allocated values, but the fact that offset is of an abstract type *forces* that representation. Once Jeff and I get around to implementing immutability of composite types, a field with a concrete type like `Float64` or `Int8` could be stored in the composite structure. Consider `tz::Int8` instead that is the timezone offset from UTC in 15-minute units. Also, making something like a timezone offset subject to the complexities of floating-point arithmetic seems like asking for all kinds of trouble.
Can you verify that the user actually has read access to the system?   system.readable?  should be an easy way to check.
Is this intentional? 

I dont see any typos, ACK for me as long as all cli tests are working :-)
Is this config setting used anywhere? Should we use it instead of ENV['RAILS_RELATIVE_URL_ROOT'] ?
You shouldn't have to call `cents.to_i` since you `#cents` properly returns a Integer/BigDecimal depending on infinite_precision's value.
See bdda369
How much more expensive is this if the string doesn't need to be escaped.
Bloody hell, I didn't see that. My bad.
Why has this block been commented out? Was this intentional? If so, can you just delete it if it's not needed.
As a more explicit comment that reference https://bugzilla.mozilla.org/show_bug.cgi?id=731746