labofsahil/github-event-dataset-2012 huggingface dataset:
adding SublimeFormatSql

SublimeFormatSql formats long SQL statement to a more readable form by using [python-sqlparse library](http://code.google.com/p/python-sqlparse/).
Add Windows key bindings
refer to jobEntity instead of trying to add jobEntity->getId()

Set Job for Arg using the entity instead of the entity id, otherwise Doctrine complains about spl_object_hash() expects parameter 1 to be object, integer given.
These are the changes I needed to make to get puppet installed.
LPS-25769
pull-request-263

gpr open
LPS-24752
LPS-25525 Apply LPS-22593
Test Fixes

socialofficehome/navigation
LPS-24584

LPS-24584 Unable to add a Page Template within a Site Template, @JorgeFerrer should review this
0.9.9.14

changed initialization block to work with refinery 0.9.9.14+
Editorial/admin updates

Changes to cover the following tickets:

* https://mozila-webmakers.lighthouseapp.com/projects/97101/tickets/48
* https://mozila-webmakers.lighthouseapp.com/projects/97101/tickets/49
* https://mozila-webmakers.lighthouseapp.com/projects/97101/tickets/50
Freebsdsupport
Flat structure

Flat structure for wsdl support
ADM-125.  

As per recent discussion on JIRA ADM-125.  Spring Batch Admin context files to remove versions from SI URLs in order to use with SI 2.1

See: https://jira.springsource.org/browse/BATCHADM-125
Fixes for use of Irssi Connectbot on Asus Transformer (including the new Prime) tablets with hw keyboard.

will match on build.model to start with 'Transformer'.

-Removed URL Scan behaviour from the search key to repurpose as ESC/ALT key globally.
-ctrl-1..0 = F1-F10
-Fn works (PgUp, PgDn, Home, End)
-shift+1..0 works (!, @, etc)

Patches were taken from https://github.com/lorantkurthy/connectbot/ and modified.
Review 4.3 && 4.5
修改了文件的命名，像一本书，都按次序来。。

修改了文件的命名，像一本书，都按次序来。。
Add proper formatting to birthdays in activitypage

Currently, date format in birthdays block in activitypage is force-formatted. It should use language-specific date format.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4517
New header and various tweaks
kleine verbesserungen in der default language
Small Typo

It's Sal, not Sol. Also, hi, this is an interesting idea. Wiki format might be more appropriate than GH.
Add Splunk Adapter

Splunk Adapter has both inbound channel adapter and outbound channel adapter to read/write data from/into Splunk. The adapter used Splunk SDK to get data in/out of Splunk.

INTEXT-21
Improvement

- Optimization of the code (in using $this instead of calling $(this) several times, and in using this.id instead of $(this).attr("id"))
- Use a relative position for the DIV instead of an absolute (so it makes to work even if the elements are hidden during the initialization)
- Wrap the DIV into another DIV to make sure we don't have a blank space due to the relative position (see http://jasonseale.hubpages.com/hub/Relative-Positioning-Leave-Blank-White-Space-on-Webpage)
Corrected name of zshrc file in list of dotfiles

I assume this is a typo, unless there are multiple name variations for this file? My local installation of zsh has created a ~/.zshrc file.
Add exception handling to the XML parser

The XML parser now catches the exceptions that might be thrown by xml.etree.parse()
gerber batch writer + tests

Apologies for not merging upstream... I tried a few times with different merge.renameLimit values, but git status subsequently yields "modified: library/kicad/parts/XILINX.lib". As the file doesn't exist, I'm unsure how to correct the issue.

Anyone else seen something like it?
fixed the change from foreca
Bescos tommy

Bonjour,

mon projet est terminé.

Cordialement,

Tommy.
Pull Req: Refactor event code

* Move event-specific code into their own extensions so that blackberry.event can remain lean
* Developer needs to whitelist the respective extensions in order to use the event, blackberry.event will be made automatically available without having to manually whitelist it (in separate packager pull request)

JSDocs change: blackberry-webworks/WebWorks-API-Docs#118
Changes whitelisting to allow disabling web security

Fixes blackberry/BB10-WebWorks-Framework#392

    Reviewed By: NOBODY (OOOPS)
    Tested By: Tracy Li <tli@rim.com>

    Modifies the WebWorks Framework to disable all web security when the
config is set to.
fix omx build in jellybean...

more pull requests are required.. 2 more files
Remove buggy urllib2 dependency.

urllib2 crashes, urllib doesn't.
Refactor StorageLocation

1) Inject the S3 client.
2) Make a single call to list the S3 objects prefixed by the same node ID.
Implement promote version
TIMOB-11174 Added new contacts functionality examples

Tests for The mobile test.
Update code to remove use of Titanium.UI.orientation

Test Phone > Orientation.js to see if it does not cause any problems. 

__MAKE SURE TO TEST ON BOTH iOS and ANDROID__
Adding Johan to the supporters
refactor the parsing of SQL files

The same logic was in two different places. Agglomerated them into a single function.
Minor change in submit button

Now submit button has blue background instead of grey
First release

UI changes ready for first release of diagnosis capture in Rwanda
validation updates and styling fixes
FIX: Deprecated use of Object::add_extension and has_extension

After updating to the recent 3.0.3 release, the JS snippet was not being added to my pages. I tracked this down to the line 34 of `GoogleConfig.php`: the condition `Object::has_extension('SiteConfig', 'GoogleConfig')`was not being working.

Looking at Object class I’ve seen that Object::add_extension and has_extension are being deprecated in SS 3.1, so I tried changing the code to `SiteConfig::has_extension('GoogleConfig')` and it worked.

I don’t understand why these two functions are not working as expected in 3.0.3, so I’m not sure that this commit can be applied, but it’s the only way I’ve found to make the module work.

Hope ite helps,
Juan
Minor fixes to unit labels

I just made some minor changes to display the correct unit labels at various places:

1. The CPU frequency unit was mHz; changed it to MHz
2. Memory and swap units were 'M'; changed it to MB
3. Changed temperature unit from just C to ℃
4. Network speed units in the menu are now displayed as either kbps or kB/s instead of 'k'
Tokat tehtävät

Oliko setin tai vektorin tuplaaminen tarkotus tehdä jollain muulla lailla?
Groovy devel

GraspableObjectList message moved from interactive_object_detection to object_manipulation_msgs. also some new .gitignore rules to ignore autogenrated files

be sure to merge this in at the same time you merge in the pull request i'm about to submit to pr2_object_manipulation
Search for existing theme page template when using 'show-on' => 'page-template'

Updated 'show-on' 'page-template' search function to check for an existing theme template if '_wp_page_template' is not defined.
Update Ukrainian/main/framework-res.apk/res/values-uk/plurals.xml
Bookmarklet can be loaded as Google Chrome Extension

Bookmarklet will be automaticaly started on load https://papertrailapp.com/ in browser.

How to load bookmarklet as Google Chrome Extension:
http://code.google.com/chrome/extensions/getstarted.html#load
German Translation

German Translation, now on the Master Branch :)
Added the ability to specify the amount of text to extract and index from an attachment.

By default, tika is only extracting a maximum of 100,000 characters from the uploaded file attachment. I have modified it so that on upload, you can specify the maximum amount of characters to extract from the document (specify -1 to remove any limit). 

Example usage:
{
    "my_attachment" : {
        "_content_length" : 500000,
        "_content_type" : "application/pdf",
        "_name" : "resource/name/of/my.pdf",
        "content" : "... base64 encoded attachment ..." 
}
}
Documentation

Fixed some typos in the documentation.
nvidia-settings -nt -q RefreshRate3
75,025 Hz

Just to be sure. I'm currently trying.
I'm thinking this upgrade might of caused some problems with the javascript, refer to my email that I sent so you know what I'm talking about.
Same as with INI reader.
It hasn't been really tested, although the compatibility of su3_tuple with the existing definitions has been checked. We're working on this currently, so you can expect it to be reliable in the near future. In fact, you may just start using it, assuming it does what it says on the tin. If there are cases where it doesn't, just let me know. It's straightforward and important enough that I'll hot-fix anything you find... :)
well, we want to trigger creation/termination of children via the resizer, and that should happen deterministically, no?
@jkbr, @gandaro no need for new issue actually

i'm using Ubuntu that sets `xterm` as the default emulator ... changing it to `xterm-256color` enables the desired style
Aby bylo vyhověno specifikaci a na nepodporované metody se reagovalo kódem 501. V praxi to ale spíš komplikovalo přístup přes CLI, pro kterýžto Nette\Application sice navržené nebylo, ale používá se.
Totally agreed that there's no cause for alarm. But it's good to know when there's potential for updates.
.CompareNoCase(".jpg") == 0
I was referring to this:

+    gemv = gpu_gemv_no_inplace
+    # As all input are transfered to the gpu, this allow to make all
+    # the gemv inplace.
+    gemv = gpu_gemv_inplace

There is no point typing two lines with gemv = ... in a row right?
just to make it symmetric with the following line...I changed it to 1 line.
Is  this machine independent?
Nope. Probably a bug right now if you manage somehow to place a lever near enough.
Btw it's better move defaults node creation to "finalize_expressions()" stage. But I prefer to put it off until next release.
And indeed, I think that special tags clearly are bad designed. I'll explain myself when I'm back on IRC.
Sorry guys i was kind of busy ;)

So, we start talking through Skype, in order to explain some issues about adding you to the community and how to manage the code/patches and "public" stuffs.

So, i gonna send you by internal github system my Skype and you already have my email i guess xD
@cscott Oh and I would love to merge yielder when you think it's ready enough.
Sorry guys.

I've been having a battle with Git and I thought that this was the final blow--one squashed commit to the master branch--, but is isn't even close.

This commit is missing the majority of my model code, it also contains tens of metadata files created by the IDE I've been using. I'll revert this commit and try again.
you need to keep BC, before it was varchar(128), with "string" you are changing this to varchar(255)

so instead of string use varchar(128)
Seems to be a to generic name for this special use-case
spelling: propagated
Should be `Float64`.
Looks + works fine
Oh, good point. yes.
This should probably take a const Common::String reference as parameter.
Missing comments here. As both methods seem to perform
the same goal.
My mistake on comment:

Removed quotes from parameter passed to $$.toType on function $$.html()
Just a thought, but does it make sense to move the style code in this .php file into a separate .css file?
please remove code that is not being used.
[this form](#L3R1238) is for the management pages - and handles replacing translations. this form just gets the initial values in for submission. we could most certainly benefit from a mixin so we're not redefining all these attributes - good catch.
Oh, now they are really messed up. 

I guess we should finally fix normals in `CylinderGeometry` for non-cylinder cases. I had once almost correct cone normals there, but then implementation changed completely so this got lost.
Very odd, it looks like some temporary debug / investigation code got committed accidentally.  The bit about obsid=8008 is clearly a one-off (with obsid 8008 being one of my favorites for testing). Hurray for version control.  I will fix this.
Ok, I think I'm done with the changes.. could anybody just double check? Thanks guys!
@Asyfupid 其实，更严格地说，你应该在 feature/avatar 分支来处理 #122
BTW, would be really good if you published somewhere how to use this thing :)
I never used the Remote Inspector and having a quick 2-min intro would help not only me (I guess I can figure that out) but all the folks that love PhantomJS.

Do you have a blog?
compress.sh should run each time browserid-locale.rpm is re-deployed. It should run afterwards. It slurps in the new json files.
"warning the" -> "warning if the"
OK, perfect I will submit this then
Ah, yes, thx. I don't work on r2 issues, only on things reported on my server's forums.
I used godi. Unfortunately, I can't build opa on a godi-based system, because some packages are in lib/ocaml/pkg-lib instead of lib/ocaml/std-lib, and the multiple-location thing breaks the build for some reason which I haven't been able to track down.
Hello Martin,

Do you mind if I merge this commit into my delimitMate repo?

Cheers!
Israel
ok, maybe move it into a util.py file and include - something like this: https://github.com/mozilla/firefox-flicks/blob/develop/flicks/base/util.py#L33
will fix this, although checkstyle somehow doesn't complain on my machine...
To use this system, the controllers must extend Webnel\Library\Controller\Controller.
On Thu, 23 Feb 2012 09:07:02 -0800
Urs Reupke
<reply@reply.github.com>
wrote:

> In that case, you need to ``git add .`` before committing, so all
> untracked files are added to git.

   OK, got it fixed, the Enlightened Mortal files are up.
Well god damn, youve been busy.
This seems to be referencing some other .js file within the current Theme folder (php_calendar). Is this required for something else to function?
Not according to the [release notes](http://kr.github.com/beanstalkd/2012/01/27/1.5-release-notes.html).
No problem :D As I noticed, somehow the commits and comments got misaligned a bit (some had unrelated commit messages with them) so I guess it's just the result of that slip.
鷳ܲܲҪˣʼ͹ûв۵̫ðɣҾֻdropԴʼ
 2012-1-8 1:35"bigeagle" <
reply@reply.github.com
>д

> Ҳһ뷨һԭͼǿһ°
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xdlinux/xdproject/commit/d967864e37647aea72f333bccb30c191c360d124#commitcomment-847157
>
If you're interested: https://github.com/probablycorey/mini_magick/pull/79
@Dinnerbone I am not familiar with GitHub so I'm not sure if there's an easier way to paste this here, but


    /**
     * Determines if the Player is allowed to fly via jump key double-tap like in creative mode.
     *
     * @return True if the player is allowed to fly.
     */
    public boolean getAllowFlight();

    /**
     * Sets if the Player is allowed to fly via jump key double-tap like in creative mode.
     *
     * @param flight If flight should be allowed.
     */
    public void setAllowFlight(boolean flight);

from https://github.com/Bukkit/Bukkit/blob/master/src/main/java/org/bukkit/entity/Player.java has a different definiton of these methods. If you still confirm what you said above (about only being like a permission to fly, not a way to) then I just want you to know the bukkit comments say otherwise.

I'm amazed by the network of way the Bukkit API works seamlessly with the net/minecraft codes btw :)
Great! That might come in handy.
Once unit tests pass. Paste results here and I will PR.
I see. Looks really weird. :)
This poses a bit of a dilemma. We either have to depend on people doing an ``npm install`` or depend on them to clone the ``q`` project in a peer directory of the project root. Either would suffice, but I prefer containment. Perhaps I should just embed ``q`` in the repository.
Thanks.  All my inspiration is stolen from the mind of @heathermiller.  I'm curious what she thinks of the re-layout of scala-sbt.org and all the crazy YAML/Jekyll going on.
typo "a lock active lock"
Ja. Eén permissie `{how:<mag planning maken>,who:jille,with:null}` zou kunnen.
RSA has fixed 15712cbf666bf0339ee640e9aa57ee27279b6f22
It's so common.
I think so too. :-)
Already fixed, sorry didn't see the pull request!  Should be fixed in 1.0.0.
Meto .gitignore , na pasta bin e gen é?
arregla eso del rogue xD con razón pegan tanto!
Do you need `createFactory`, you don't seem to use it.
Sorry if I wasn’t clear, Karim. Yes, the user should be shown the names of the languages in terms they can understand, absolutely. But at the same time these should not be the names that the code uses internally to identify those languages. It’s best if internal codes don’t depend on any particular language. I like the GNU codes because 1) they’re already in common use for exactly this purpose, and 2) they allow for regional variations, as I said. As someone living in New Zealand, who speaks an English traditionally more influenced by Britain rather than the USA, I don’t like the idea that “English” must mean “US English”. We have our own GNU locale code, en_NZ, though en_GB is mostly acceptable as a substitute. :)

Similarly, a French-speaking person from Canada or Switzerland might object to the idea that “French” must mean “French as used in France”. So somebody might do localizations for ShaderTools for fr_CA and fr_CH, in addition to fr_FR. And similarly with Spanish. And so on.
applied thanks
Тут видно эту точку. Точнее что ее убрали.
Кстати, какраз в CMS этой точки не видно, а у всех остальных страницах она присутствует.
> Jon, I've just checked. The TiddlyWiki version at http://tiddlywiki-releases.tiddlyspace.com/upgrade is 2.6.5, which is the latest version and the same as the version at http://tiddlywiki.com/upgrade/index.html .

This is because I manually updated it :)
Nice.

Could you please give the plot the same look and feel as the plot on:
http://scikit-learn.org/dev/modules/linear_model.html#bayesian-ridge-regression

that way it makes it easier to compare methods at a glance. Ideally, we should factor out the sample generation code.
i'll try to bring this topic on plone-developers list.

in short about jslint/jshint:
 - jslint is old, jshint is the "new cool tool" to lint javascript.
 - install nodejs on macosx which also installs npm (node package manager)
 - then `npm install -g jshint` should install jshint globally (-g is for global)
 - to run manually you invoke `jshint path path2 [options]` more about it read [here](https://github.com/jshint/node-jshint)
 - look [here](http://www.jshint.com/platforms/) for integration into editors
 - in [deco.init.js](https://github.com/plone/plone.app.deco/blob/master/plone/app/deco/resources/js/deco.init.js) i'm using it and you can borrow options ("/*jshint ...*/" part) or read more about them [here](http://www.jshint.com/options/)
I don't really get what this setting is for, mind elaborating?
This will error again on null type
This was an accidental commit, I was intending to do some more research into why it works before posting it, but any insight would be appreciated.

Code to replicate the original problem:

```
import matplotlib.pyplot as plt
import numpy

z = numpy.arange(1200).reshape(30, 40)

cs = plt.contourf(z)
plt.clabel(cs)
plt.show()
```
Here is similar feature: https://github.com/mborromeo/fnordmetric/commit/17474597864f0f73dd9628fb9e068012a7bf76e4 but it is still pending pulling.
really? skynet is the name out of fiction for the software made to rule the world.. the link with _humans_ won't work with _acme_ .. btw skynet.com doesn't exist, but acme.com does. I see that skynet is a brand of Telesat.. ok changing :(
there is still an issue here, `Content-encoding: deflate` has to use zlib.inflate, no zlib.gunzip.
Still no tests?
哈哈。。。 已经改密码了
the ? works great in such cases:

return longestTextLine == null ? 0.0 : getTextSize(longestTextLine).getWidth();
http://www.youtube.com/watch?v=a1Y73sPHKxw
Yeah. It still double procs w/ envenom.
Player auras are only applying to one Spirit Wolf/Treant.
am i going down a rabbit hole?
El key point 8 de los dias 4, 5 y 6 no está o no lo encontré? :S
I'm not against this idea, but :
 - instead of calling Platform.isWindows() I suggest that you should add a new method 'String[] getShutdownCommand()' to SystemUtils with custom command for windows 
 - there is already a couple of code which deals with process outputs, it seems that you don't need the output of the command, so  net.pms.io.Gob would simplify the code
You might want to consider removing the PDF generated from these files; it's almost comical: http://davetron5000.github.com/scala-style/ScalaStyleGuide.pdf
i've added a google docs for it
Tested and Approuved ! Thanks ;)
2010q1-202 is sourcery. And I find my current build extremely stable and ICS friendly ;-) I have built ICS CM9 ROM from source and customized it from tytung's cm9 and my tweaks ;-) And never had any su problems...
Strictly speaking, the scripts should be loaded at admin_print_scripts-$page (which means they'd have to be broken out into another function). But this is definitely better than what's there right now :)
Sry. Właściwie jezeli muliplayer miałby być "osobnym procesem" to mozna MultiPlayer.main()
Richtig, die wurden vom Generator so erstellt. Ich wäre dafür, solange wir nur ein paar Helpler haben, alle in eine Datei zu schreiben.
<3 this idea
So does the actual GBC bootstrap!
My presentation.
the way I decribed it also works, but maybe that makes it more clear
Note specific to this (more general) but please follow more strictly Dojo styleguide lines. For example (but there are maybe other issues) it must be 
```if(!parent){ 
```
not 
```if (!parent) {
```

This should not block the pull request but this must be fixed in a later request. Thanks.
It seems strange to use `live()` in this one place, where `bind()` seems to be the standard.  @hafriedlander can you help?
Now committed to topic branch under my fork. Commit c42cd2fe3a37778a2037d9799362838c3e09a182  Also created a new JIRA issue for this: https://jira.springsource.org/browse/SPR-9143
Get v5.5.0 - this fixes everything.
I did in deed switch to using onWorldChange - perhaps it's worth nuking 
setLastWorld as well, not really needed.

--------------------------------------------------
From: "sunkid" 
<reply@reply.github.com>
Sent: Monday, 20 February, 2012 12:50 PM
To: "Tad Hardesty" <tad.hardesty@suddenlink.net>
Subject: Re: [PermissionsBukkit] Added info and stub dump commands, fixed 
pom, added description to plugin.yml, added default configuration. (170efe2)

> Great, thanks!!
>
> btw - any love for https://github.com/SpaceManiac/PermissionsBukkit/pull/6 
> ?
>
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/SpaceManiac/PermissionsBukkit/commit/170efe2d355b3376d2fd134e73245084d678d639#commitcomment-986894
sorry. i am away from my laptop until sunday evening. will check it out when i get back
On 06.01.2012, at 18:02, bendavies <reply@reply.github.com> wrote:

> This breaks line 164 for me. Sqllite DBs no longer work.
> Uppercase vs lowercase.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/liip/LiipFunctionalTestBundle/commit/66268adf0eece5255d8dfab8625234bf75a65d38#commitcomment-844198
It does just what it does in MODX now, it just displays the version it was built for, it doesn't not allow you to install if you dont have that version installed. But it might (if its added) warn you if its not the version its built for.
How about feed:inbound-channel-adapter, inboundMailAdapterType, base-ftp-adapter-type and etc.
Search pattern whole project: **xsd:attribute name="channel".*?use="required"**
Is it separate issue?
Mantera your getting closer, but your still far, you should look at my changelog for V5.0
https://onmyee@github.com/onmyee/Xperia-X10-Kernel-2.6.29.git
Ok so Im trying to reorganize the physics system. Entity movement should be impulses while stuff like gravity will be forces. Gravity is turned off because I still need to implement terminal velocity.
I also want to take all the entity collision stuff into the entity class.
Good title!
Log instead of say
"so _you_ don't have to log in again"?
why return `None` and count on the caller to call `dist.as_requirement()` when we could just return that from here? Note that in the case we don't recognize the location as a VCS repo at all, that's what this function already does.
This method is begging for code comments, it's a mind warp, and hard to follow what's happening without them.
nominal?
BigInt(0).longValue() ??? What ever happened to 0L?
Yeah, but if we add another dep that starts with "akka-kernel" then we've accidentally messed things up
Do zeros need units?
Here is an example, maybe extreme:
In [65]: p1 = nhlib.geo.Point(0.0,0.0)

In [66]: p2 = p1.point_at(1.0,0.0,1.0)

In [67]: p3 = p2.point_at(1.0,0.0,179.0)

In [68]: p4 = p3.point_at(5.0,0.0,90.0)

In [69]: line = nhlib.geo.Line([p1,p2,p3,p4])

l = line.resample(7.0/2)


In [73]: l.points
Out[73]: 
[<Latitude=0.000000, Longitude=0.000000, Depth=0.0000>,
 <Latitude=-0.000000, Longitude=0.031476, Depth=0.0000>]

l should contain 3 points, but it contains only 2
Why the name change? context, data_dict is a code  convention use everywhere.
Good idea. I'll replace with a `MatchData`.
JM was right - should run limiter in FinalizeSamples() as mentioned on old DSP PR. Multiple streams can be mixed as can navsounds....
OK, no problem.
It's just a little faster in case that first find fails.
EMRio first looks at your previous job flow usage that you supply it.

Lets just say "EMR history" and rely on mrjob/boto to be the default source for it. Right now the tool that dumps job flow history into a json file isnt included in this repo, so you will just confuse people by mentioning file input. We can also add the tool :)
Yeah, I got lazy. I'll fix this when I get around to writing Hadoop runner tests.
`shouldDeps` должно быть достаточно
this is called in the reply thread...not sure this is thread safe with the main task execution loop
I agree. Will thing about how to implement it.
- use new with parens
Whitespace errors: leading tabs and overlong line.
A logical expression seems more reasonable here. i.e. not (AnalyseOk andalso QueryNoWarn)
in any case, doing so does not give acceptance in the HMC...
If they can be combined in any way, the style `(1 << offset)` should be preferred, but it sounds like this is just for some grouping, so that's okay.
Recent versions of Clang will automatically synthesize your properties for you, so this line is unnecessary.
Preferably use opts instead of o.
Oh, so there's not an inherent reason that it needs to be placed on the heap, it's just to make sure the stack isn't filled up by a potentially large allocation if NGROUPS_MAX happens to be huge.
Why the faux `do { ... } while (false)` loop?
Why not `this.bufferSize | 0` ?
I didn't liked the idea of FQN in all the code, but ok.
It's to clarify the meaning of unit in "cycles/unit". Maybe a short example would help, something along the lines " sixteen samples spaced two seconds apart, the frequency spacing of the spectrum would be 1/32 cycles/second (Hz)".
No worries, my dyslexia rears its ugly head. I misread the docs on RETURN_STRINGL. Thanks.
Above we are setting n = "{" + n + "}"

that would make "n" string. Will this work:  -> n.alias = a
тут тоже 2 таба
отступ в xml — 2 пробела
Ok, I'll try to format the min and max attrs to be a valid date string like I did in this line but I don't think we should try to change/format the value of the step attr because the value should be expressed in seconds and I guess it'll be awkward to pass something different than a number to the step attr, like a DateTime instance or something.

What do you think?
thx. removed
Yes, we should follow the naming convention in the lib files and test too.
We need to figurate out why you need to call html_safe here.

`link_to` already generates html_safe string. So call html_safe should not be required. If it is we need to fix.
Did you mean `expand_path`?
Should not be `remove_columns` here?
replace `currentOwner` by `fun.symbol` and drop the next line
I know these lists are probably small, but in the interest of performance, wouldn't it be better to fuse the two calls to `filter`?
What benefits? Modern compilers will inline small functions without prompting, plus you get type checking on the arguments and your eye can follow the flow of control without reference to some macro off in the woods. Really, I'm trying to break you of your macro habit, they simply aren't necessary for efficiency and when used like this obfuscate the code.
Maybe enum?
I think the deprecation warning should indicate that you should now use the Rails config.force_ssl option within it.  That will be more user friendly than just saying it's either on or off.
Since you are working with s single string, how about just using endswith? It seems like this would be faster but maybe the iterator is just as fast. I'm not sure.  Also, do we have to check for *whitespace* as the error message says or just newlines?

```
if text.endswith('\n\n'):
    n = len(text)
    text = text.rstrip('\n') = '\n'
    n -= len(text)
if n > 0:
    ...trailing newlines from...
```
Well...you are popping element 1 which is maybe the first symbolic element and maybe not. You just have to be clear about what you are trying to expand. How about `(a*b+c*d)**0.3`? And as far as indeterminism goes, if #1638 goes in you will have the `ordered` function to give you the arguments in canonical order.
It is a bit too late, as this is already merged, however this `except` should be `except AttributeError`. The way it is written now will catch even `ctrl+C`.

It is minor and it probably won't ever create a problem, however I am mentioning it.
Why is this?  Shoulnd't it contain y?
Does the different order here matter?  It looks like it used to be sorted.
Do you think this should be a method of `Density`?
it looks like you are duplicating code here...don't you have a function that does this sort of thing to minterms in the last PR?
changed to
```

    Represents the polyhedral symmetry group (PSG).

    The PSG is one of the symmetry groups of the Platonic solids.

```
This seems odd to me...Partition inherits from Union but now creates an object with FiniteSet. @mrocklin , is that correct?
1.Inserted a line after 3408
2. changed the method name from dual_matrix to dual
The best naming policy is to rename "registerMetaTag" into "appendMetaTag" and "registerMetaTagUnique" to "setMetaTag".
But this will breaks the backward compatibility.
If this pull request will be accepted you may consider it for Yii 2.0.
Pertaining to the java.sql.Timestamp questions:
Date.toString returns something like "Tue Oct 30 10:15:59 EDT 2012" while Timestamp.toString returns something like "2012-10-30 10:15:59.812", which Ruby likes better for parsing. What I SHOULD probably be doing is just sending timestamp.
fsdgsdg
Wouldn't it be better to subclass from a `dict` here instead?  You still get the type-sniffing benefits of the `Mapping` ABC and you don't have to implement most of the methods.  And I think the only method you'd have to implement would be `__setitem__`:

```python

class frozendict(dict):

    def __setitem__(self, k, v):
        if k in self:
            raise Exception('Boom!')
        else:
            dict.__setitem__(self, k, v)
```

But I may be missing something.
Here's my opinion on the twig stream method: The naming is way too generic, it should be `streamView`, `streamTemplate` or similar. But mainly, I think this is not something that should be configured in the controller. Why is this not an option that you can enable on the twig templating service, which changes the behaviour of `render`? That would allow a) those views to still be testable. b) the configuration not to take place in all of the controllers.

The current application `stream` however does exactly what the name says. The way it works makes sense to me.

That said, I see the value of consistency. IMO the way Symfony\Bundle\FrameworkBundle\Controller\Controller::stream() works should be changed. Either the name, or the behaviour as I described above. If that is not possible, I am open to butchering the current stream method in the name of consistency.
@stof Ah yes, good point

@fabpot I wasn't suggesting adding to Pimple/Silex, I was just suggesting another workaround for userland
if this change was intended you also need to remove the closing brace and afterwards correct the indentation
it would be great if you at least link us to information on how to set it up for ubuntu users for instance on having kde setup for this
On 4/30/2012 5:01 PM, Jeff Balogh wrote:
>> @@ -90,7 +90,7 @@ def message_validator(request):
>>           return request.errors.add('body', 'body', msg)
>>   
>>       # These are the keys we'll accept for messages.
>> -    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId'
>> +    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId', 'cryptoBlock', 'site'
> Right, the client would need to look up the site keyed by the queue URL if that's important. Should the site be a part of the encryption key?
>
>

No, it should be part of the message's HMAC however. The remote site has 
no idea how the client is storing information pertinent to how to 
decrypt the message. To that end, the remote site needs to identify it's 
source in such a way that the client can properly decrypt the message as 
well as prevent MITM attacks or a message potentially spoofing it's 
origin since the origin isn't being checked outside of the HMAC block.
Can we just use `infoPopUp' instead of `infoPopUpBinding' to be consistent with other knockout bindings?
I think this should be "one component holds coordinate data." (remove the bit about the other component, as its redundant)
Cool! I'll look into this. https://bugzilla.mozilla.org/show_bug.cgi?id=744761
It's actually reducing the amount of avatar settings you see. If you have for examle selected the local avatar it will only display the options of that avatar. It's actually more or less simplefying the avatar options.
1. Fixed (will appear in the next push)
2. Hum... That's more a comment that should have been sent towards the commit message... Should it be removed or changed?
Ah okay found it, see change in phpBB/language/en/install.php
Yes this is how it should be, see the mentioned ticket in the description http://tracker.phpbb.com/browse/PHPBB3-9364
Cast $user_id to int.
I am not a fan of "private" in this context for reasons described in another commetn.
Multiple lines please.
Delete "splitted in array", it is neither grammatically nor factually correct. Alternatively:

Holds the words entered by user, obtained by splitting the entered query at whitespace.
I see the problem. Previously, this was dealing with how to delete the user. Now it's focused on how to delete the posts. Users can be soft deleted with this feature, and posts can be soft deleted with your feature. I need to figure out a way to distinguish between the two, as this does not soft delete the posts; rather, it "soft deletes" (deactivates) the user and keeps the posts like normal.
i figured the client would exit and runit or similar would restart it.
It is behaviour change, I am not sure if everyone accepts it.

It is possible to check whether T is not ValueType and only then check for null.

For instance

```
isValueType = typeof(T).IsValueType.

ToString() { if (isValueType) ToStringValue() else ToStringReference() }
```

It could be great if you could do this in compile time also :)
I think this has to change to something ala '{{name}}ModelFoo'.
ну можно у баннера сделать вместо 1 записи shows/clicks, 2 - shows, shows_last, clicks, clicks_last, shows и clicks текущая сумма обновляемая нодой, а last'ы сумма на прошлый день. 
крону пересчитывать ничего не придется по сути, ему будет нужно взять 2 значения, разницу записать в статистику и приравнять last_shows/clicks к shows/clicks.

в твоем варианте крону придется именно считать. 
допустим через 3 месяца будет 40 активных баннеров работающих все эти 3 месяца. т.е. на каждый баннер по 120 записей в таблице статистике. проходит день. крон должен каждые 5 минут же чекать истек лимит у баннера или нет. т.е. каждые 5 минут, крон должен 40 раз просуммировать по 120 записей чтобы получить суммарные просмотры/клики.
помоему это явно лишняя работа и нужно чтобы в базе уже была сумма просмотров. так почему бы не апдейтить эту сумму напрямую нодой?
Зачем убирать вызов notFound?
Нода не должна заниматься этим.
Docblocks (also applies to other methods). Although you'll probably be moving this to the view model.
If that's your reasoning it's time to change it ;)
This is a sencha bug - fine for now, they will fix.
rather... first arg should be ```listenerStore```
and arg1, arg2, arg3 should be replaced by an array of stores (```targetStores``` or something). Please name them appropriately according to their role
Nit: "Use camelCase for variable and function names"
https://raxaemr.atlassian.net/wiki/display/RAXAJSS/Code+Style+Conventions+for+JavaScript
On Tue, Jun 26, 2012 at 10:16 AM, nouiz
<reply@reply.github.com>
wrote:
>> @@ -338,6 +338,12 @@ DllExport int CudaNdarray_reduce_min(CudaNdarray * self, CudaNdarray * A);
>>  DllExport int CudaNdarray_reduce_max(CudaNdarray * self, CudaNdarray * A);
>>
>>  DllExport int CudaNdarray_dimshuffle(CudaNdarray * self, unsigned int len, const int * pattern);
>> +//PyObject* PyArray_TakeFrom(PyArrayObject* self, PyObject* indices, int axis, PyArrayObject* ret, NPY_CLIPMODE clipmode)
>> +//PyObject*
>
> This was to see easily the numpy interface to this function. As it is also in the cuda_ndarray.c file, I'll remove it here.

Ok, but I think it should be removed both places. A url to the numpy
documentation would be more up to date, and less confusing to someone
who's grepping the code for *our* version of take.
Update it with voter deployment.xml
sic (*) github is eating backslashes
probably you're right, i have less info about pandaren, but it is possible.
Я так и не понял, для чего хранить имена классов в переменных
matchCount maybe?
I'm sure it is explained later, but at this point I'm left wondering (from a beginner's perspective) 'what is a URLconf.  Maybe just a parenthetical (explained below - assuming it is)
I just have come to like NamedTuples, and for this use case they seems to be perfect match. They have some advantages over dicts:
  - Usage is nicer - val.cols is nicer than val['cols']
  - They are immutable
  - There is no way they could contain more values than what is wanted - in effect they have a constraint on them
  - They have a little nicer printout for debugging purposes

But, this is stylistic issue and Django mostly uses dicts (because namedtuples haven't been available for long). So, there is no strong reason to use namedtuples if you don't want to.
You don't need to copy-paste -fexception twice. You can instead write somelike like

```
set(CLAY_CXXFLAGS "${CLAY_CXXFLAGS} -fexceptions")
```

after `endif`.
`== "interactive") ) {` => `== "interactive" ) ) {`
I guess I'll have to throw an ETOOTIRED exception, because I didn't understand this comment :-)

BTW, would it be better if the iterator had an ->empty method that could be used in this special case rather than hacking ->finished?
What's array_agg(1)?  What about ARRAY[1]?
See the filter idea above.

`is_future` used to be a less readable check, but like it is now, I agree that it's unnecessary to split it out. I like the `_unwrap_result` idea as an easy improvement without going all-in with filters.
Does json rpc spec split the code namespace into ``system errors < 0`` and ``app errors >= 0``?
Nit: There should be two empty lines between imports and code.
Can you pre-initialize this value so we don't have to make this check on every call?
`#{etc}/wemux.conf`
I'll do that tomorrow afternoon. It's movie time now.
Is this block identical in both branches of the conditional? Probably don't want to duplicate it.
Normally we don't leave version-specific comments on patches like this unless there's a change that a maintainer might need to know about right in the formula. If this is a comment about why you changed it, that usually would go in the commit message, or in the pull request comment. For instance:

"Old patch no longer worked due to X"
roger that.  updated.
This is needed to support WKT that has extra spacing between components, which is legal.
I think we should add a comment that references the Python issue. Then future us can know what was going on.
I think writing what you suggested would be more appropriate, though I still think it doesn't belong. Adding it begs the question of if we need to add something similar to the moderator remove functionality as well and anything else that has implementation specific oddities.

Then we either have to keep really up-to-date with our documentation update whenever reddit slight modifies how things operate, otherwise our documentation will become inaccurate in spite of a providing correctly working interface to those actions.
Same comments as for the singular translation helper.
Actually it would end like this:

```
      # Remove after Rubinius 2.0 is released
      DATETIME_FORMAT = RUBY_ENGINE == 'rbx' ? '%Q' : '%s'
      VALUE_FORMAT = RUBY_ENGINE == 'rbx' ? (value * 10**3).to_s : (value).to_s
```
Meybe support more than one space.

rule.replace(/\s+/g, ' ').split(' ');
I removed `virtual` from this because it is not being overriden in a derived class.
Filed a bug #1734
It's safer to use:
```
    if (sessionEditor) {
```
because it also checks for undefined.
Yep, I totally agree.  Maybe one of these days we can donate to webkit the faster implementation Flex used for VBox/HBox ;-)
Just a style thing, I prefer

```
if (!enabled) {
  return;
}
```
We added logic recently in brackets.js that turns off quiet scrollbars in some cases on Lion...  Will this screw up the width calculation in that case, or are native Lion scrollbars the exact same size as our quiet scrollbars?

If we need to be more robust, I think we could do something like ```if( $(<jsTree's root node>).parents("quiet-scrollbars").length > 0 )``` to see whether quiet scrollbars are enabled.
I think "HTML-based on native" should be "HTML-based or native"
The dialog layout needs a bit of cleanup - the left and right padding don't match, and there isn't enough space between the label and input.
Still need to uncomment that and fix the tests. Forgot about that.

(the idea is that the worker should always keep the cache filled, but we don't stop performing database queries if the worker crashes)
Should include `{{ block.super }}`?
Oh sorry, app.template_filter can't be used in coaster. `blueprint.app_template_filter` used to not work earlier, but if it does now, this should be moved to baseframe.
do you need to adjust the defined size of rb_eRuggedErrors? It holds 2 additional elements now, right?
Basically, I noticed that every operation had a `dcpu.cycle +=` line, and I wanted to clean that up. The base cost gets applied after the switch block, but variable costs depend on the context of the operation, so those are left in the code.
Not a correct way to do this. Don't have URL through settings. This is still hard-coded URL. Just instead of template it is not in settings file.
Changed to lowercase.
I would put all search-related code into one external file. Now you have three (`query.js` and `search_bar.js` and `search_page.js`). Why? OK, I understand for search_page and search_bar (because page has additional code (but even there I see some code duplication I don't like), but why additional `query.js`?
You should also escape values you put into URL here! This is very very important. Remind me that I talk about this next time.

Find a correct filter and applied it.
agreed. might be good to have some kind of debug mode in templates eventually
Use uppercase for colours
Again, clarify order with a (...) so it reads more clearly.
Change pop() to shift() so that POST requests get processed in the order they were added (Add to back, remove from front).
I think it shall be something like:

```
charstring.splice(charstring.length - args[1]['arg'].length, args[0]['arg'].length);
```

Also it would make sense to introduce `arg1` and `arg2`.
> Well, I'm not familiar with JS, not familiar with pdf.js code base and have a lack of time, so it is very up to you how to split interfaces

Shall we use this pull request just as reference implementation or you want to address the review comments?

> 1) If annotation is visible, it showl'd be displayed.

Agree

> 2) Verification of digital signatures relies on regional standards and laws, so developers must have access to the following things: pdf blob, pages > enumeration, annotations enumeration, annotation dictionaries and streams.

I cannot not find any signature verification code in your pull request. Displaying the picture of signature without verification can mislead users and accept bad document as right one.
I agree. I've updated the test accordingly... Thanks!
This is good - I much prefer using real links rather thank JS to hit the back button.
Ordered lists have numbered bullets ;)
Isn't hash still optional?
Nit: absolute imports are preferred to relative ones
```js
return Array.prototype.splice.apply(a1, [from, to - from].concat(a2));
```

Но можно смело передавать `num` вместо `to`, тогда код будет такой:

```js
return Array.prototype.splice.apply(a1, [from, num].concat(a2));
```
good idea. Made that change and pushed it to this pull request.
To avoid the `reverse` and `mconcat` calls (both relatively expensive), you could write something like:

~~~haskell
Just b -> joinEvents evs (acc `mappend` b)
~~~

and

~~~haskell
joinEvents [] acc = (acc, True)
~~~
OK, but we need to fix this someday
Four nearly-identical one-argument macros sounds like a job for a two-argument macro and better abstraction.

```Julia
macro os_only(os, ex)
    if os == this_os()
        ex
    else
        :nothing
    end
end
```

Then wrap the OS detection code in `this_os()` (name bikesheddable).
Two things, since idx is a variable and not scoped to the just the for loop, general practice is to define it as a var at the top of the function.  Use of the -- operator is typically discouraged in Javascript.  See - https://fedorahosted.org/katello/wiki/JavascriptCodeConventions#ArrayIteration
This doesn't appear to be a new issue; however, should we be defining "All Errata", "Bug Fix" and "Enhancement" so that they will be translated?
At least TODO comment to replace the THE_VERSION with actual version. Or it's meant to be replaced by the build script? Anyway, some comment is appropriate here. Also consider useing <<HEREDOC for the message.
You should also define `#create` method
You forget `:architectures` here.
Now this is beautiful LINQ :smile:
Or, instead of `background.html`, use keyboard_manager.js to insert the special iframe (without mozbrowser) in the system.

You should do this instead since keyboard is a special class of apps, not background service nor foreground app.
missed the quote :P
not -> note
Although not as important in single-inheritance cases, please still try to use `super()` instead. For example `super(Fitter, self).__init__(model, fixed=fixed)`.  See the seventh bullet point under http://astropy.readthedocs.org/en/latest/development/codeguide.html#coding-style-conventions
My stars do I hate GitHub. One key slip and the comment you spent several minutes on writing disappears. Anyway.

The term "factory class" is technical. Since this is an example/explanation section, it should not be assumed that anyone knows what that means.

    The easiest way to create a coordinate object is to use the `Coordinates` class. The kind of coordinate
    is determined by the keywords provided. For example, if you specify "l" and "b" angles, the result will be
    a `GalacticCoordinates` object. If you specify "ra" and "dec" angles, the result will be an `ICRSCoordinates`
    object. Note that the `Coordinates` object is a generalization, and actually returns the kind of object
    appropriate to what you are requesting. (This is called a "factory class".)
I think @iguananaut is right here - we don't want to force external libraries to use the astropy logger (unless, as for affiliated packages, they want to, in which case they just use it directly).
Any reason this needs to remain in the code, but commented?