labofsahil/github-event-dataset-2012 huggingface dataset:
@hawesie: All the other patches I mentioned are various copies of the same as far as I know. Visually looking at the my gist (3.4.1) and your gist (3.4.2), yours is a super-set, so if this PR goes in, then mine shouldn't be needed. I'll put a note to that effect in the other PR.
no to just `wrapper`, no need for _underscoring stuff.  functools docs has good example names to use.
+1 looks good
Nice! Thanks, Kevin.
It's not a standard yet, it's still in Working Draft. But it's also something IE8 would never implement. That it would break is surprising, since reserved worlds are allowed object properties according to the ECMAScript standard.
Bump :)
ets に expire つけるべきじゃない？ mnesia 使うくらいなら ets でイイと思う。むしろここは間違って eleveldb とか使ってみたら？（永続性あるよ
Joe Wakeling
oh, actually, I should fix this test instead of removing it.. it's the one place where we do flip_bit tests. I'll update it and add a flip_bit(sig) clause.
I was playing with the code to understand it (I'm a beginner). Already corrected it, thank you!
s/idea/ideal/

;)
omg I'm scared
this `class` seems unnecessary
? works for me...
when i compile this version i get 500 errors :) on vmap4extractor and i can't create the tools
eh well double counting requires more study there are too many flows. hehe
убери эту строку вообще
Do you still working on mmaps?:]
@Brianetta, I've done a bit more thinking and I've decided to change the name back to "sidereal". Thanks for your input.
And what is the meaning of the angle? I.e. are 0 and 360 normal text, and is 45 degrees slanting up or slanting down?
Det har inte att göra med `0 * -1`, det har att göra med `0.0 * -1`. Av någon anledning tycker ruby att Floats kan ha ett minustecken framför en nolla.
What is the new build procedure from checkout to binaries on linux?
Hi,

after this commit (& maybe the one before) setting the option forceFitColumns to true results in an empty grid.
The code from jan,18. works...

Thanks 
Robert
Thanks for clarifying. Merging..
this must delete i have do this but always here
ああ、そうでしたね。
カレントディレクトリ以外に移動しましょう。
翻译成 头 ，很崩溃。这个HEAD是个重要概念，个人认为保持原味比较好
engrish.
Ca mériterait un test unitaire :D
edit_refinery_blog_admin_blog_post_path is the correct one and that's
what is currently generated if the blog were doing things properly.
It's a work in progress and it's all going to make sense. I'd rather
not have to use a private fork :-)
the node cells can be brought back, 
but Jon asked for absolutely no colors
check for null here
This would certainly prevent the exception from being thrown.  

I guess I am curious as to why your client would only want to receive a subset of the arguments that the server is sending?
Heh, didn't catch this one because I had already deleted login.php in my local copy (working towards the single-page login).
当存入的Map的值是个集合的时候该怎么取？ 
Map<String,List<User>> saveMap=new HashMap<String, List<User>>();
List<User> user1=new ArrayList<User>();
user1.add(new User(1,"测试1",1));
					
List<User> user2=new ArrayList<User>();
user2.add(new User(1,"测试2",1));
saveMap.put("user1",user1);
saveMap.put("user2",user2);
//存入Redis
boolean isSuccess = client.hashesMultipleSet("test1", saveMap);
用这个函数存入，这里存入的是一个集合。
我想问取的时候是不是用个 client.hashesGetAll(User.class, "test1");  这个函数吗？ 这个函数只支持Class，这个时候我要怎么样取出来呢？
One other thing -- I made a couple of specific comments about it but the issue of hard-coded sizes (especially in pixels) is general and pervasive. I think testing with different window sizes and screen resolutions may show some problems with that.
Seems like your problem is to get NPM running on windows and not with statify.js itself.

Maybe this guide will help: http://www.hacksparrow.com/install-node-js-and-npm-on-windows.html

Once NPM is installed and working you can install statify.js with a simle ```npm install -g statify``` from the NPM registry without making any changes to the code. If you just want to use it, you don't need this git repository or the fork at all.

PS: deleting licensing info is still kind of lame ... especially if it's a license that basically permits you to do almost anything you like with the code.
No ! Thanks ! Added the wrong file to the commit !
Vaya, lo lamento, yo siempre he agradecido vuestro trabajo, sois muy buenos y espero que sigáis haciendo un buen trabajo, aunque sea en privado.

Un saludo, habéis echo bien :)
Carsten-

FYI, I found a few dozen strings in Expression Manager that were not being translated.  A patch from yesterday fixes that (but we should ask the translators to look at them).
No need for html and head tags
lol, so the problems with the cdma kernel
Do you have a pointer to a discussion where upstream discusses this issue? I saw nothing besides the JIRA issue. It did not seem to me like the upstream devs commenting there were against any change in particular.
throw MaksStringException... is going to always be handled at least as well as throwUnexpected ...
Hi chris!

Thanks for the pull request.. Actually adding this functionality is a good idea.. I only don't really see the need to have this extra link from the QUISwitch to the controller; as is now it's leaking the controller, as the switch will keep that reference in forever. 

The _QEntryTableViewCell already has a reference to the QuickformTableView, which has the controller. I guess we can make the call from there. 

I can take a look at this later today or over the weekend, lets see! Thanks for helping!
Can we get a new gem pushed with this fix?
Awesome use of HTML5 elements!
"chestDropRate" : [
    {
      "name" : "foo",
      "sequence" : 0,
      "value" : 0
    }
  ]

works but

"chestDropRate" : {
      "name" : "foo",
      "sequence" : 0,
      "value" : 0
    }

throws an exception because it tries to iterate over the keys of a NSDictionary which returns NSStrings which then throw a selector not recognized.

On Feb 23, 2012, at 3:09 PM, Magical Panda Software wrote:

> Can you give me an example of how this crashes? I'd ideally like to make a test case for this, but I'm also curious since this hasn't crashed here for me in a while...
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004870
Line 219  (-) _guid(0)
I'd rename it to `MONGODB_DATABASE`. Is that named a database BTW in MongoDB. It looks like a schema in the relational world.
just re-add this sql


INSERT INTO `db_script_string` (`entry`, `content_default`, `content_loc1`, `content_loc2`, `content_loc3`, `content_loc4`, `content_loc5`, `content_loc6`, `content_loc7`, `content_loc8`) VALUES (2000000077, 'Greatmother Hawkwind gestures to the pitcher of water sitting on the edge of the well.', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL);

then the error will be fixed for the  20000000077
**TODO:** I accidentally removed that.
It was unnecessary. Use GitHub issues instead
I think it may break git-related stuff like git revert and git cherrypick. But other than that, the game looks fine :)
Should be "Autor".
This shouldn't make too much probs for l10n. It's going to make already done translations as fuzzy(I believe), so it's good to go with any forthcoming merge/push.
Why do you set ai_protocol to `IPPROTO_TCP` just to set it zero right afterwards?
同样的问题，如果是你的话你会怎么选择？
I was in the process of manually applying the commits of your pull request to my repo to test them and I found the following
using the subtitles above in a file named as `Sintel.2010.1080p.srt` with [Sintel.2010.1080p.mkv](http://www.sintel.org/).

Without commit 52e62203ea12c40628de1869882994ce1065446a the subtitles appear, with commit 52e62203ea12c40628de1869882994ce1065446a they do not appear at all.

My other test files with subtitles all work the same with or without the commit, this is the only one where there appears to be a regression.
Maak lijkt me meer standaard dan "creëer"?
i'm not sure, but if i cast something, what has to be in "can cast while walking" it will shutdown the server with Segmantation fault
Shouldn't this use the `activerecord?` method you defined below?
'singular_name' => _x('Project', 'post type singular name')
Will show the trash icon and then the singular name: Project
I forgot to commit the AndEngineExamples, will do so in a minute...
Table needs to be changed to div or something else. I don't think Web 2.0 uses table anymore.
Interesting. Could you add a test in the multiclass test file to do multiclass classification with ElasticNet / Lasso?
wow... your commit messages seem so ridiculous, then when I click on the diff, everything makes sense. haha.
Since it's currently unused, I'm going to mark it reserved and rethink it later. It's meant to allow for two future designs: early termination during shutdown when we detect that all contexts have parked for suspend, and letting the main thread make calls into JS land by invoking and waiting for the invocation to complete.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/31/2012 05:21 PM, Peter wrote:
> I've changed it to "NVIDIA Optimus and AMD Switchable Graphics",
> that should be clear right?
> 
> --- Reply to this email directly or view it on GitHub: 
> https://github.com/Bumblebee-Project/Bumblebee/commit/772308e3b62a25a91f1ecc0dc7de497d8f3559e0#commitcomment-920426
IMHO,
> 
we should call AMD's solution by one of the following terms

* PowerXpress (4.0) [2]
* BACON/BACO (Bus Alive Chip Off) [2]
* Dynamic Switchable Graphics [1]

[1]
www.amd.com/us/products/technologies/switchable-graphics/Pages/dynamic-switchable-graphics.aspx

[2] AMD internal slides:
https://docs.google.com/open?id=0B6HLw4COEpWLZWQ2YTEwOGMtMTE1Ni00ZmE0LTk5NzctN2E4OGFlNDViMWU5
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJPKIS5AAoJENBgQ48l36uhJh8H/AwrxQvMWNOl2kJ/J8vXDbNw
ffdhESWnJ+O+tPxr6tgXk7luINLLJXKRq3mnUyfISXsvgBKzkTxpSyp1ixIdTsx5
jS9BYuHjSI2+7MfgDr1gKIp1Zbm44rGSekm1SJWqvJxiRoSibJq8R64VKYn1/cbb
kwA4oA/urcofzJnFvNiovVOmuFZnEybJa5iq+n+ocJcoigpWLbL+/aj//pLJ+Psy
xDoVltSnHcIO3B4pAnR2oKjkAFYBgWH+R73sV66jU2Zxgifcqw2LUhNJeOS5hrv+
lib0+kjWPDfD5tILHfi2mTEN9dU1pFVyzg1Sa137Qyo7xekuXtMfWuRjR7/WJgE=
=f9OE
-----END PGP SIGNATURE-----
This was moved to line 518 to avoid multiple calls.
no files attached js/jquery.tokeninput.js, css/token-input-facebook.css
I ended up using *shudder* jQuery UI DatePicker and just styled it myself. Nice and fast, but crappy user experience for touch.
Why is this project "dead"?

Currently depends on childprocess a pretty old version of childprocess...
Again, we really prefer to handle this more gracefully.
marge this..
I'm not sure about the changes meant for "# Support for Spree < 1.0.0" because this plugin will be installed on
spree sites of version 1.0.0 and above.
I'll need to read about and do some upgrade to make sure it is working properly.
I was thinking about different sub-types of `Node`, such as `MethodNode`, `ParameterNode` etc. Likely we would need something like `Node#getKind()` then to determine the kind of node when iterating over a path.
All in all this needs a bit more time. I think we need something for the first draft, without an approach for representing results method validation doesn't make much sense.
I'm sure you've put some thought into this, but I just want to make sure 'open web pages' is the best name.

Does it include all tabs and windows? I can't think of anything better, but somebody else might.
Ok I saw it! ^^
i test and it works :D, thos method check_tiny used ( _gui_message_(self,text,conv)) don't be confused with self.session.gui_message(conv.cid, '', message)
We should bundle these components into some kind of "many-many relationship control".  Possibly a pre-defined, parameterised config?

    $config = new GridFieldConfig_ManyManyEditor('"FirstName")
ok, i will include this ifdef and soon made commit. also @rmcc please look at http://review.cyanogenmod.com/#change,12327
It should be enough just to scan the www-built directory, don't need the './www' one, but assumes you trigger a build first.
@cjmaclellan 

if initial time != 0 then

project the MRTI on the FEM data structures as the IC
Well, are you using cm 7.2 or still on 7.1 build? Try to update all and get more info about it.

On last 7.2 build i get usb.apk crashed... and after that crashed systemui. 
That was on problem.
Also, i explored about uevent and kernel. 
Quarx recomended to write module for usbnet_enable(defy has that in kernel... but how to build module i don't know, i have love experience at c/c++ lang. .
Why the change to wait a whole minute to restart? I have mine wait only 2 seconds.
Haha, yeah: https://github.com/CfABrigadePhiladelphia/the-other-philadelphia/commit/d94cf63a41e47b10cde9086083f4b29d6f4f56ba
Are you sure that this extension really maximizes new windows and hides the title bar?
Or is it just a "hello world" extension?
Yep. Makes sense.
Can we move this to the outermost level so if we need to add tests that do not require clearing the credentials or working within a tmpdir that they can be added?
or Fog#settings?
i tried https://github.com/nvk/t/raw/cli-bird.png i get a 404
https://github.com/yui/yui3/commit/8730c072d46f42ec5a075fac62d5930736ddb8e1
Isn't 60000 1 minute?
You can represent [^ ]+ as \S+ (an opposite to \s) to improve readability.
Test case for this change?
CONTOMIZE IT, DAMN YOU! CONTOMIZE IT!
then the router should monitor the UnreachableMember messages instead (that should give the same timely feedback, right?)
Just wondering when we should use dilation and when not to. In this case 30 seconds is ages anyway.
Should we revisit this URI pattern to harmonize on actor identifiers instead?

akka://system@host:port/asdasd ?
> What is the purpose of this object?

Contains message classes, constants and a handful of functions that are  
used at several places.
Hmm, just noticing the 'avoid_autolinking?', wondering if we need to have that in another place.
You are mixing different ( and ) code conventions, please use Mono code conventions everywhere
On Thu, Oct 4, 2012 at 3:15 PM, Stefan van der Walt <
notifications@github.com> wrote:

> In dipy/viz/projections.py:
>
> >      Returns
> >      -------
> > -    fig : figure
> > -        Matplotlib figure
> > +    ax : axis
> > +        Matplotlib figure axis
> >
> >      Examples
> >      --------
> >      >>> from dipy.data import get_sphere
> >      >>> verts,faces=get_sphere('symmetric724')
>
> Shouldn't get_sphere return a sphere?
>
>
You mean, this should read:

verts = get_sphere('symmetric724').vertices

And get_sphere needs to be changed, right?

—
> Reply to this email directly or view it on GitHub<https://github.com/nipy/dipy/pull/79/files#r1767626>.
>
>
It does indeed. Suggestions ;-)
It looks like the convention of _ before the method is being used for migrate_rpm_unit_repodata, should probably use it here too.
This is not relevant to this commit.

Please consider using your own global ignores for such things.
it is indeed. this is now included in the add-on
Решили оставить метод.
Als er 10.000 rijen in staan misschien wel?
default value is handled in setMode so there's no need to handle it here as well
Oh the shame! Fixed.
If a slot is not using a parameter, you can simply leave it out.
It's set to `YES` because the `beginTrackingWithEvent` method responded `YES`, and I do believe this how iOS does it.
@jwilling @jspahrsummers re-typing the view pointer does not satisfy the block's requirements, I'm changing the iVar, but there still needs to actually be an iVar.
@mkandrashoff If you didn't follow @creationix and @rphillips using error() is the pattern that has been sort of emerging:
https://github.com/racker/luvit/blob/master/lib/luvit/tls.lua#L499
Nie, nie je.

Keďže `Nette\Object` deklaruje, že `__get()` vracia referenciu, musia takto fungovať i podedené triedy (aj Nette\DI\NestedAccessor). Takáto konštrukcia zabezpečí, že `__get()` nevráti referenciu na službu, ale len referenciu na kópiu v premennej `$service`.
nit: potentiall should probably be potentially
спрятать в private
скобку на другую строку
Yes, I know that the old method is modifying the options hash. We didn't care about this when it was implemented, but I prefer that these methods do not modify the hash.
Unfortunately not. :cry:
Looks good. You could shorten the argument name as well, from `datetime_format` to just `format`.
Ok, will do later today
Make sure you call `super` here after doing your work. ([read more](http://jonathanleighton.com/articles/2011/initialize_clone-initialize_dup-and-initialize_copy-in-ruby/)).
What is this caret pointing at?
Chuck:  Sorry to be a bit prickly before on your observations about the use of Macros.   My tendency is towards getting as quickly as possible to working code.    Coupling that with most of the code being written when I knew less about the real reasons for object-oriented style and when it matters and when it doesn't often leaves me copying style instead of trying to improve it.   I'm sure I'll continue to be more focused on getting something working rather than style, but I don't want to discourage you (or really anyone else as I know you are unflappable) from voicing your stylistic opinions.
This import should be removed, it's always implied.
done


On Fri, Nov 16, 2012 at 2:01 PM, Ronan Lamy <notifications@github.com>wrote:

> In sympy/unify/usympy.py:
>
> > @@ -0,0 +1,109 @@
> > +from sympy import Basic, Expr, Tuple, Add, Mul, Pow, FiniteSet
>
> should be from sympy.core ...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1633/files#r2156216>.
>
>
Actually, just checked, and it changes it to `from io import StringIO`.
Aren't changes like this breaking completely the API? There was already a release with this api, so this may break other peoples code.
you forgot to rename the function in the doc
Consistency: use `char` here, not `unsigned char`
I'm not quite clear on what behaviour this is testing (or how many behaviours)
Es _pysqlite_, no _pysqlite2_. Me equivoqué. :P
Esto debería devolver un listado de Documents
The more I read these, the less the **Example:** form seems to be a little weak rhetorically and may present a challenge to understanding....
although this will work as !=, it would be nice to keep it consistent as !==
Opening tags are typically not an issue for code highlighters, but I guess it wouldn't hurt either.
$converted? I have a strong aversion to variables named "$temp".
Yes, this decouples phpbb_style from the implementation of the other ones, allowing them to be replaced. Particularly useful when testing, but in general simply more flexible.
Such as itself, yes.
We should probably record this info at the time of dump.
we are dying anyways, why free?
spațiu după %

Traducere la System, cu {% trans 'System' %} iar template trebuie să aibă {% load i18n %}
ăsta nu va merge. va șterge TOT gold-ul primit. Trebuie dat external_id.
dimension ?
това съм го пропуснала, предполагам трябваше да го има в Gemfile или да се махне от тук
should it be `if ($lat !== 0 || $lon !== 0)` instead ? (to allow < 0)
Yeah, I changed scripts.start to "server.js" for Nodejitsu deployments.
Changed this line.
@Wolvereness Is there a better way of doing that?
Just used the existing codes :) 
So you mean rather this way?

$app = JFactory::getAppliction();
$rowid = $app->input->get('rowid', '', 'string');
$url = 'index.php?option=com_fabrik&task=details.view&formid=' . JRequest::getInt('formid') . '&rowid=' . $rowid;
Why may to null ??? confusing...
This file is needed to avoid 'Proj prop-s -> Android tools -> Fix proj prop-s' on users side.
Why did you remove it?
Notices that this is the same change as submitted in https://github.com/NancyFx/Nancy/pull/712/ so this should definitely be removed.
Does this affect what happens when you run kmk? Does it do anything else?
I have done this currently in util for pharmacy. Will commit my code when it is in a working state.
It doesn't work if I do it like that. It throws this exception when I run the JUnit tests for saving a DrugInfo object:

ERROR - JDBCExceptionReporter.logExceptions(234) |2012-08-21 10:34:24,753| Value too long for column "DRUG BINARY(255)": "X'aced0005737200106f72672e6f70656e6d72732e44727567000000000000011d0200094c000b636f6d62696e6174696f6e7400134c6a6176612f6c616e672f... (409)"; SQL statement:
insert into raxacore_drug_info (drug_info_id, drug, price, cost, name, description, uuid, creator, date_created, retired, retired_by, date_retired, retire_reason, changed_by, date_changed) values (null, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?) [90005-135]
Good catch. Sorry I missed these in PR #31.
Then fix the merge to not do this? Is there an open issue for this?
cache多次调用的对象
What exception should we throw?
This does not fit my understanding of correct semantics for EXCEPT ALL. This looks like it's causing any single instance in the later tables to wipe out all duplicate matches in the first table -- I think it's only supposed to wipe out at most one original duplicate instance for each later (possibly duplicate) instance. 

Caveat -- this expectation of behavior is based on my past experience with Postgres. I went looking for a description of correct INTERSECT ALL and EXCEPT ALL behavior in the freely available "late draft" SQL 2003 standard -- and came up EMPTY! It only mentions them alongside EXCEPT and EXCEPT DISTINCT which are called out as being equivalent to each other but does not AFAICT describe how the behavior of the ALL variant differs from the other two. It even subtly suggests that except for the special case of UNION ALL, the ALL keyword does NOT prevent an implicit complete de-duping! Do you know of any other authoritative description of INTERSECT ALL or EXCEPT ALL behavior?

Barring the option of just ignoring the ALL in these cases...

Since the matching is done on all columns, maybe a tuple-to-duplicate-count map would be a better data structure to use for the ALL case.

Actually, for simplicity, we could again dump the templates here completely in favor of a simple m_isAll boolean check. In this case, the check would control whether to allow the duplicate count for a given tuple to exceed 1 or to "top out at" 1.
// DROP m_partitionValue -- not really used.
Hooray for replacing magic numbers!
Лучше явно указывать используемую версию `bem-bl` в свойстве `treeish`. Последняя стабильная версия на сейчас `0.3`.
We always have PCRE nowadays, no need to conditionalize.
Whitespace. Please check above as well.
Yep, you're right there! Fixed.
please write it as ``$notNull = $tableColumn['nullable'] === 'NO'``
Should be:

Test if parser fails when an unknown value is used for an otherwise correct 'rl' or 'lr' cue setting.
I wonder if we should harmonize any naming with WebKit here? https://github.com/WebKit/webkit/blob/master/Source/WebCore/html/track/TextTrackCue.h#L74
Which reminds me we should probably use the expando for the ajax anticache param too, so that two jQuery instances in the same window won't collide.
Sorry. It's my mistake. I double check with Yi and the src code. We needn't this anymore. Done. Thanks.
Existing points will have probability (weight) 0 of being chosen.
Support for Windows!
Do you know in which cases val is undefined here? If so can you add a comment for what those cases are?
Does 127.0.0.1 work? That's what we did for weibo.
The line between `md5` and `head` should remain.
Use `#{bin}/tal`
If show is deprecated add :deprecated => true to alias_action
Can we modify this to assume it will work from the PRAW location?
I like your goal of eliminating tedious, risky error checking in controller methods. However, we need to enforce some separation of concerns. I'd like to see an implementation where validators don't raise exceptions (so these validators can be used like any other existing ones). Someday we can have validators raising exceptions, but the way the code is laid out now doesn't support that in a manageable fashion yet.

I think the way that would make everyone happy would be to introduce a ```@wiki_validate```, and enhance ```Validator.set_error```, ```ErrorSet.add```, and ```Error``` to include an optional HTTP error code. We don't have an inheritence scheme in place yet for ```@validate```-like decorators, so you basically need to duplicate (or wrap) ```@validate``` -- which is easy -- and then scan c.errors and abort accordingly. I don't think that'll be too hard, but you'll also have to change all your apiabort calls to be explicit returns. Does this sound reasonable?
Looks like the describe_class let blocks here and below have extraneous whitespace.
Same package now. But the whole motivation for this PR is to move HttpConnection & HttpConnectionPool out of `internal` and into `com.squareup.okhttp`.
I did that intentionally as I wanted the errors in the unit test to be reproducible. Otherwise we won't know whether it's the changes to the code which breaks the tests or simply bad(/good) luck. But I see your point too, what do you think?
just `return this.json.dependencies[pkg.name]…`  that way the `if` is a single line statement and you don't need the else
So let it be written. So let it be done. Trailing :: and freestanding @ are now carnal sins.
Would this be better as `$GLOBAL_EXIT_CODE` or just `1`? In the event that we interrupt, we wouldn't want Travis or some other part of deploy to consider it a successful run.
When would this be overriden? I thought 'items' would just be for tourjsadapter?
Technically fine, but it should go in `/app/themes/toura/_mixins.scss'
инструмент =tool, in single form.
This could be simplified even more by chaining the initial creation with the .insertBefore(), .attr(), and .html() calls. I don't think you need the :first, either.

```js
        this.dialog = $("<div />")
            .attr("class", "CodeMirror-dialog")
            .html("<div>" + template + "</div>")
            .insertBefore($("#editor-holder"));
```
I think the group of z-indexes is easier to understand if they are in numerical order.
Part of the reason I kept these separate is in case we wanted to change any of these specific menus down the road. For instance, maybe in the next couple of sprints (or for Edge Code) instead of having How to Use Brackets go to an external URL in the browser, it loads up a dialogue box in the editor. 

I figured it was kind of an niche case, so I'm fine tweaking it.
``window.setTimeout(_projectOpen, 0)``
I could add this, although if not it would just evaluate to undefined which would still cleanly fail the test above.  What do you think -- worth adding?
Weird ... that's not even the latest version of that method :(( I won't say that ajaxorg/cloud9infra#1653 caused it to regress, but I'll think it very loudly.
I'd prefer `labels[!is.na(labels)]`
use a message like `otherwise.getClass + ": " + otherwise` to get the class and (possibly) instance info
垂直置中在 float 下其實是無效的
拿掉應該也不影響?!
UTF-8 strings are NULL-terminated arrays of characters. They require no special handling.
What about putting in Cloud9?

-Jimmy

On 3/18/12 10:24 AM, Zhai Ke wrote:
>> @@ -1,4 +1,4 @@
>> -package cc.mrlda.util;
>> +package cc.common.util;
>
> Oh, I didn't know we need to register every package. The reason I did that was I had another Mr. IBP (MapReduce Indian Buffet Process) package available, and those code blocks were shared among these two. It is fine for me to put those back to the cc.mrlda.util package.
>
> Ke
>
> On Mar 18, 2012, at 8:29 AM, Jimmy Lin wrote:
>
>>> @@ -1,4 +1,4 @@
>>> -package cc.mrlda.util;
>>> +package cc.common.util;
>>
>> Why moving into cc.common?
>> Shouldn't everything be under the mrlda.cc package hierarchy?
>> We don't own common.cc, so it's bad practice to put there...
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/lintool/Mr.LDA/pull/2/files#r572091
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lintool/Mr.LDA/pull/2/files#r572181
>
Can you split those two fields into multiline passing of arguments? So

```
foo = Bar(
   bla=blu,
   ...,
)
```
Indent. Twice. (Because of the block and because of the with.)
Well this is what i was talking about when you run the command ``./manage.py makemessages ``
One more thing, creating a user object when he come to the site is not yet implemented right?
Yeap will do
Extra space please.
nope we want document.head, as head is a snapshotted clone from earlier. We want all of the scripts with data-butter-include that weren't snapshotted.
moved after the end of the loop
I will send email about this after it's merged so everybody knows what is going on. I think once you know
```ruby
rescue => e
rescue StandardError => e
```
are the same, you don't forget it.

We could add StandardError to the all rescue => e, but it feels to verbose. And people will forget to add StandardError in future anyway.
It probably needs to be listed here is well, otherwise you can' not change just the locale and nothing else
Good catch, that should be fixed now.
The issue this is addressing is adding something like "aeolus-audrey-agent" gets incorrectly parsed as a NVRE and gets added incorrectly as a SystemTemplatePackage.  Without attempting to verify that the package is a real thing, I'm not sure I see any other way around this issue.
Just curious, why do you need to move this code?
nit: same line
I think we should have the ability to stream large files over HTTP without first copying them to local disk.  VOTable files can be quite large relative to their in-memory representation, for example, so it's generally much more efficient to just stream the HTTP.  But if there's some gotcha to making that work correctly, I'm willing to pass on it for now.
It definitely needs more work if it is to be used by other stuff. Using a simple straight line is not very accurate, but since one cannot really accurately predict a cone search without knowing the distribution of sources in the catalog, I did not put too much effort in this.
Good point - I'll put it back to `MAX_WIDTH`
I feel like this fun should be named `publish_supported` and the other one should be called `update_ring`.
You factor it out of the IF/ELSE statement too :)
This if is redundant, if fIsMine is false, we already bailed out.
As you said above, I just copied the code, which was there :).
It tells the uninstaller to uninstall the start menu items from the All Users location, not the (default) user local location.
Bad cast? Isn't the type of tb supposed to be `mach_timebase_info_data_t`? You're taking the address of a pointer to a `struct mach_timebase_info` here. Doesn't seem right.
`|| (!current_user)` is a bit confusing. "or not current user". How about `or logged_out?`
just call `close_motion!(nil)` here. and also try deleting `save` and see if it works.
Will.
I will back this change out.

I attempted writing a spec for GatlingAsyncHandlerActor, but it resulted in a significant number of changes that I felt were tangental to this change so I backed it out.  I just missed the addition to the pom.
maybe say `Paste Cell & Replace`?
Make sense, I should have thought of that. I just saw the same order on both, sorry.
The initializers? Should I try to write doc blocks for the already present elements, too (for example `NITextInputFormElement`)?
I don't really know the syntax, but I will try to mimic the blocks in other files.
Is 8174aa10e closer to what you had in mind? I basically introduced `void git__unescape(char *)`, so that `git_buf_unescape` now delegates to it. It avoids code duplication for the unescape logic.
All the posix functions begin with `p_` to note that they have been wrapped. Rename these declarations to `p_localtime_r`, and add a `#define` to the Unix `posix.h` that just calls `localtime_r`.
Any special reasons for dropping the `const`?
I also saw that you got rid of legacyDialogSubmit in a previous commit. There were still a couple references left, so I removed them.
Yeah, but only if the file doesn't exist right? So we might get one more false negative before we see the locale activated. This doesn't seem like a big enough deal to worry with. This should almost never happen anyway as these files are automatically created.
Oh right, fixed.
Are there other node possibilities? Also, the above two should be ``elif``s since it's either an h1, an h2 or a p.
When I run the tests, they all die on this line for me:

    FAIL: kitsune.apps.questions.tests.test_views:TestQuestionUpdates.test_no_update_vote
      vim +184 apps/questions/tests/test_views.py  # test_no_update_vote
        self._request_and_no_update(url, req_type='POST')
      vim +163 apps/questions/tests/test_views.py  # _request_and_no_update
        eq_(updated, self.q.updated)
      vim +31  vendor/packages/nose/nose/tools.py  # eq_
        assert a == b, msg or "%r != %r" % (a, b)
    AssertionError: datetime.datetime(2012, 7, 20, 5, 37, 36, 232107) != datetime.datetime(2012, 7, 20, 5, 37, 36)

My vote is that instead of using eq_, you check the stringified versions:

    eq_(updated.strftime('%Y%M%d%H%m%S'), self.q.updated.strftime(...))

It's possible there's something like this in kitsune already, but I don't know where it is offhand.
how about this instead:

```python
from socorro.external.postgresql.dbapi2_util import single_value_sql

# ...

    def breakpad_db_exists(self):
        return single_value_sql(
          self.conn,
          "select exists(select True from pg_database where datname = 'breakpad')"
        )
```
* the name of the function is far more descriptive of what it does.
* the sql returns either True or False directly rather than relying on counting lines in a result set
so the apps are not stand alone at all?  If a cron is to be run, it must be run through `crontabber` ?  I had expected these to derive from socorro.app.generic_app.App so that each one could have its own config file rather than crowding crontabber's configuration with the needs of all possible config options.
I'm not making string changes. I'm making tests faster. eat my :shoe:
I forgot to remove these lines (policy was not used) - done now.

For the commit the default policy is used: God->commit( ZYppCommitPolicy() );
Identation issues again
This can be done with QDir::fromNativeSeparators(str), should work for any string (path or url).
I would suggest to return the following:

return new Elastica_Response(json_encode($params));

Like this the response has the full request inside which could be useful.
Do we anticipate situations where CE != SE ?
The reuse of the finder is indeed a bug.
done or almost done in everzet/symfony#4
I removed all tests about ldap/ldap class
I would call it ``createFinder`` to make the name more specific, thus making conflicts with other interfaces less likely
this is not needed as the parent method (in the base Command class) is empty
you should use the short class name (the other solution would be the FQCN, i.e. with the leadaing ``\`` but Sf2 uses the short classname everywhere). The current code will break the autocompletion is some IDE (PhpStorm would consider it as ``Symfony\Bundle\FrameworkBundle\Console\Event\Symfony\Component\Console\Input\InputInterface`` for instance)
名前なぁー
くくり出すmethodが少いので適切なモジュール名を決めにくいと感じますが、なんかないっすかねぇ
should be find as unless.
Single quotes unless interpolating.
Ah... ok, I need to turn these into appropriate tests... issue incoming.
Changes made as requested. Good point about the `&rsquo;` it was my obsessive compulsive nature kicking in!
Good find here
I'm concerned about the fact that we're adding another event that will get fired unnecessarily if we don't have logging on.
I dont like this, when a new test is added there will be lots of starting and stopping going on for no reason.
This can run into problem if the deployment used a Logging-Profile based LogContext and that LogContext was unregistered just a few lines back (line 95 in this commit)
made sense to me, slightly unrelated to JBTM-1036 but I will let you off -P
why use the instance variables here? These are available as methods (created by the options like --conflicts)
Yes, that works fine. Done.
Very nice README!

I'd rather not maintain a separate README though since only very few settings are different.

* Some of the alternative phrasing for parts also in README can be incorporated there
* The parts the same will be redundant
* Sections detailing general server tasks (installing of packages, creating a MySQL user and granting privileges) can be left out imho.
верни оптимизацию только обратно plz
Use addEventListener instead of onload directly.
This `XXX` can be removed if we do

        *[data-keycode] * {
          pointer-event: none;
        }

in CSS. See https://developer.mozilla.org/en/CSS/pointer-events
@lco @jcarpenter 
For the issue, could we have a confirm button to let user set the specific time.
After we can receive the value change event of picker,
I will remove the confirm button.
Thanks.
Woot! thanks fo catching it.
I guess that `delete` doesn't like `currentDialog`, right?
You probably want to call this even if asyncStorage fails for some reason.  I think you can just call setItem() with two values and assume it is going to succeed. I'd just call setItem() and then call pv_addNewRepeat() right after, rather than doing it asynchronously.
the documentation block here for insertBubble seems like it wants to be split in two, with the specific comments placed before the appropriate functions.
Because of how l10n works you need to change the id of this string to something like remove-title-2 and use that instead.
Currently, we have four states in this IME
and we need three kinds of color to indicate different states besides the normal state.
We learned this from OpenWnn. 
In my opinion, these color is very useful to distinguish different states.

function km_showPendingSymbols(symbols, highlightStart, highlightEnd, bgColor); 
is enough by now. I am not sure if any other IME uses this feature.

If @timdream know a better solution or IME that uses less states, please let me know.
This function do exactly the same thing in #2606.

So the way to resolve the conflict with this one and that one would be

1. #2026 need to back out changes done on notifications
2. this pull need to back out changes done on statusbar

and things will work.
Of course not. `i` is first defined in line 51 of the same file.
nit: iframe is not needed.
I believe we want a timer to automatically close this after a timeout.  Although @caseyyee doesn't explicitly call it out in the wire-frames, he does link to https://wiki.mozilla.org/Gaia/Design/BuildingBlocks#Banner which mentions the timeout, and my impression from him has always been that we want this to disappear/fade-away.  We can worry about the fade-away later on.
Just for consistency, can you order the expressions in this |else| block in the same order as the one in the |if| block?
As `data-type="list"` has to be used on the list container, we could use `data-type="edit"` on the list itself.
Not an individual benchmark, maybe set of benchmarks?
Maybe add LOG.debug statements for each of these "action" methods?
`if slide.sound?`
Shouldn't this be maxClaimXLength not maxClaimVolume?
I know and I will restore it when you upgrade optimizer to work with edge veritas :)
Oh yeah, after you move the except up, you'll have to replace `pass` with `continue`.
Replace `kk` with a more descriptive name - columns?
> Oops! Google Chrome could not find api.4chan.org

sorry but i haven't sucked moot's cock
I agree that baseview should be in views.
@chuanchang The version check is necessary since virt-v2v only support RHEL(3,4,5,6) & windows guest, while the limitation is not on hypervisor or host system.  
@zhouqt The magic 3 is for RHEL3 system, which not support virtio disk driver. Well, your logical is right where the os_version judgement should be placed, but I also want this step to be checked and put the fdisk info into the log for further tracking. For now we only use info combine with error to check where fail, maybe we need do more.
why recreating a class to print json here ? PHP is able to encode in json, and if you want some pretty printing, Composer already have a class doing it for PHP 5.3 (PHP 5.4 has it natively)
no need to check here IMO.
typo => explicit
BackboneのViewは、コードというよりはほとんど慣習に近い存在です ー ViewsはHTMLやCSSについて何も決定付けることはなく、任意のJavaScriptテンプレーティングライブラリと合わせて使用できます。一般的なアイディアとしては、ページを全体を再描画せず、モデルが変更されたときにそれぞれが独立して更新できる、Modelに支えられたロジカルなViewとしてインターフェースを整理します。JSONオブジェクトを掘り、DOMの中にある要素を参照し、手動でHTMLを更新する代わりに、Viewの`render`関数をモデルの`"change"`イベントに結びつけることができます ー そうすると、UIの中のモデルデータを表示しているすべての箇所は、いつでも即座に最新になります。
Why do you check only m_handler for null value? What's about m_settingsItem?
Replace both of them by QScopedPointer
Casting array to string is not supported by PHP (thus `Array` with actual data discarded) and does trigger error notice. Created issue about that (before seeing this) https://github.com/humanmade/WPThumb/issues/47
ok, I will
Also not logging related, but this change is safe because socket.socket.recv() will return a string, so it should never have a negative length.
This seems like another weird case where separating the validation from coercion would be useful.

That is, normal config files go parse -> coerce -> validate. Yaml goes parse -> validate with no coersion necessary.
May be use `name` instead of `'test'` here and elsewhere.
I hesitate to do that since some code may try to load module to do the cleaunp in next turn of event loop to let all other modules cleanup first and in such case things will go wrong. I'm afraid there is no simple solution for avoiding leaks. I don't thin it's worth doing in this case.
I refactored this code. It was the pure equivalent of python code, but we can use JSON.stringify instead.
The goal here is to stringify the preference value.
nice, not to readers, this change will fix our under-reported metrics.

@zaach - should we log even if get data is present?
yep, fixed :)
make it `infile = self._expand_files(infile)` and remove the super on the next line. 

With these two changes (see one below on line 125) the class names doesn't need to be repeated.
awesome .. thanks!
@mmoll Is there a reason the `if` is outside the `setcode` block?
I think it might be a but cleaner to have
```
setcode do
  (Facter.value(:architecture) == "ia64") ? "ia64" : "parisc"
end
```
> Given a desire to have version.rb be as light-weight as possible, it seems simpler to keep version number information directly in the code.

I'm convinced.  There's not much of a difference in terms of difficulty between `git describe > VERSION` and `version=$(git-describe); perl -p -l -i.bak -e "s/VERSION(\s+)=(\s+)'(.*?)'/VERSION$1=$2'${version}'/" version.rb`  Pulling out the VERSION into it's own file was intended to make this a little easier, but it doesn't seem worth it.
but then when spomething is missing we will render the same message, btw, I hate that on GH...
In any case we don't need to read the littlechef_dir variable here, as it is being done again in `settings.py`
Speeking of `#container`, it probably shouldn't appear twice on the same page. IDs should be unique.
Agree with @marc-mabe here. In fact, why not have the cache as a static protected member instead?
Alignment is off here.
This here is to allow "/user/login?redirect=/some/route" to be requested. The value of the param is set only if it is present AND if the config allows it.
I don't know why this was left out since the authenticateAction() is actually looking for it!
ENTER_KEY
Other than the whitespace, which I'll fix, all of the ifdef'ing is consistent across all the Symas versions of the benchmark sources.  So I applied their changes.  The only real changes I made were the lines relating to configuration.
This rounds it to +-1 lines, which I think is acceptable.
this makes ``!isset()`` redundant .. but did you really mean to also ignore ``0`` ?
but what if the parent was previously persisted and there were no changes, we would continue to attempt to compute a change set. imho we should have a separate array to track for which objects we have attempted to compute a change set in this flush() call
This probably isn't necessary, because there's a way with simple form to include errors inline.
Specifically IE8 and below doesn't support it:

https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Array/indexOf#Browser_compatibility
docstrings?
if suff:
    if copy:
        tmppatch['copy'] = suff
    else:
        tmppatch['sourcepath'] =suff

?
goalf easyconfig for setuptools is missing
Regardless, it has become the generally accepted way to do it, in my experience. But the key point is to keep it consistent through the quickstarts.
should use Popcorn.dom.find()
нет необходимости составлять оглавления вручную, просто `.. contents::` все сделает
You aren't required to use the context-manager style call with `assertRaises` - typically you'd use that if you wanted to test something about the exception itself, like the contents of the message.  You could rewrite this test as (notice the lack of `()`):

```python
self.assertRaises(BadRequest, self.convo_management_service.read_conversation_type)
```

This is minor and I wouldn't expect you to go back and change all these, just for the future.
this is an example comment on the line of text itself. :cake:
oops didn't look at it. Will update that with RDL.
Ambiguous grammar: "displaying by hididng..."
Surprisingly not : https://developer.mozilla.org/en-US/docs/CSS/Media_queries
tip: always prefix all `self.client.login()` with `assert ` to always be certain it worked.
This style rescue makes me want to tear my eyes out, but nice to get a patch :)
Fixed all this stuff. The functions are now setUnselectedStyle, setSelectedStyle, and (un)setHighlighted. Also they're called the same things for lines and angles.
Can you put this 1 in a code tag so it looks pretty?
Use "" here, not %{}
@pmjones Yes @phpdocumentor was complaining . I discusses with @mvreil and he mentioned to add `\StdClass` as param
Put this up at the top with the `http` module usage.
also xml and yaml configuration are created inside the command, because they have dependencies on the conenction and the outputwriter.
A docstring might not hurt.
С-style cast в С++ не используют. Нужно использовать какой-то другой. Какой?
Well my proposal is to provide separate date and time. When a date is passed in via the command line no time is set, and therefore you wouldn't specify this argument. When a build ID is provided, a date and time can be extracted from it and both arguments can be provided.
Yeah exactly. There are no legal cases, so it will fail if it gets called. But there's a corresponding function in the resource query code, and I want to eventually combine them, so I wanted to keep the structure similar.
@rwldrn ok, I have been careless to use this.
It is for all types of resources, and it is extensible, as the administrator can add extensions to config.js. This statement is above when creating the schema:

`var resourceSchema = _.extend(createResourceSchema(), config.schemaExtension);`

A few other notes about extensibility:

a) If you need to add data that is stored, but not indexed, you can do so using the `extra` schema property. The users resource does this with the publicAlias.
b) If you need to add data that is indexed, but not stored, ES actually lets you just index arbitrary properties, and by default you can search on them
c) If you need advanced control over a field, you can make schema changes on the fly on ES, and you can update the schemaExtension to reflect that for your deployment

So, there are some options, depending on what your needs are.
Given how this is now being used, I would not return the original value if it can't be cast to a boolean. I would return an error, otherwise it's going be hard to track down bugs when code passes in invalid booleans.
Again, probably best to just perform the cache invalidation after the data has been updated in storage. If someone reads the principal after this happens, you are now stale again.
Do we need to check if comments[iii].createdBy exists, in case the comment has been deleted?
Fixed side note.

I'm not sure what the reasoning behind doing the checks in the API method is. I was under the assumption we'd like to prevent execution of functions as much as possible when the user is not authorized?
rewrote popup.js mostly
What is the purpose behind this migrate/rollback/migrate pattern?
isEqualToString работает быстрее при сравнении строк