labofsahil/github-event-dataset-2012 huggingface dataset:
Added one more piece for hidden fields, hides table row.

Forgot to include this on my other patch. Sorry, I'm new to github and collaboration
Just made a commit that should demonstrate what I was trying to express -- by having a CharSetUnion operator you can avoid punching the inversion functionality down into the baser Set type.
这个又不影响安全....就像你说的屏幕在900px下的没几个一样，禁用js的也没几个
Removing troll?  What a joke.  Get over yourself.
I couldn't find any misbehaviors by passing such query strings, so please just open a new issue if you find any.
This is causing an error during `rake assets:precompile`.  I will open an issue.
Anotace se vždycky parsovaly ze zdrojáků, ale kvůli php-acceleratoru se zavedla i tahle metoda - podle všeho se rušení neplánuje.

ad změna názvu) v celém frameworku se pomocí getParams() dostává asociativní pole klíč=>hodnota, na tom nevidím nic špatného. Kvůli zpětné kompatibilitě moje úprava zpracuje správně jak seznam parametrů, tak asociativní pole. Co je na tom v nepořádku?
Yes, I thought about that. Since it's just "basic inlining" it's not implemented yet.
Roots will be undergoing some big rewrites soon in terms of structure (not what it is/does). So you're correct but we might as well just wait until then to fix a lot of these issues.
>>> 'hold' in ('hold')
True
>>> 'stud' in ('hold')
False
>>> 'draw' in ('hold')
False
>>>
Teach me master, I don't really know what is wrong now.
!!! Fetched file: v3.2-r4_master.diff.gz VERIFY FAILED!
!!! Reason: Filesize does not match recorded size
!!! Got:      663041
!!! Expected: 640198
That's really bad. Passing string to `setTimeout` means that `eval` is called on it. And it's not just wrong, but also totally unnecessary in here.

You can simply do:

    setTimeout(resetScroll, magicNumber);

BTW, `magicNumber`? :) You could use more understandable variable name ;)

Anyway, as `resetScroll` is only used in this one place it can be even inlined, to make it simpler (and avoid this global you created above):

    setTimeout(function () {
        sacScroll = true;
    }, magicNumber);
Buena práctica HTML: Usar siempre etiquetas en lowercase, no UPPERCASE.
Hi!
Nice to meet you, too.

I'm pretty excited to have found your project and been able to join it so early.
I have been looking for a Javascripty way of making music in the browser for a while, and haven't really found anything that I liked. I took a Computer Music course in College a while back and we used CSound back then. I was almost ready to reimplement a CMusic or Chuck compiler in JS. And then I found flocking ;)

I actually have a minor in DSP from my college days, but since that was almost 5 years ago it might be a little rusty.
None withstanding that, I'd be more than happy to help you with taking flocking forward.

As far as the live editor goes, I didn't actually look around a lot. I have some experience with processing and remembered their live editor @ sketch.processing.org. So I looked and found that they used ace, actually back then it was still called bespin. 
AFAIK, Orion seems a little complicated and not as shiny as the other two.
From what I can see, ace and CodeMirror have almost the same feature set, so I don't think it matters much.

If you want I can implement another version with CodeMirror and we'll see which one works better.
As far as the branching goes, I totally agree, but as I am still kinda new to github, I wasn't aware of that.

Regarding the boilerplate stuff, that's exactly what I thought. As you can see I already did that in the editor, naming the flocking object synth which gets automatically played when the button is clicked.

I think the editor is pretty feature-complete, if you'd like me to add something more, let me know, if not please feel free to pull the code into your repo.
FYI, there's an EnumValidator class the does this.
I check this spell (32663) with old rev R2: 1354 & Mangos : 11855 , show this error :
ERROR:HandlePetCastSpellOpcode: Creature (Entry: 28511 Guid: 1000237) isn't pet of Player Naytdk (Guid: 172172) .
please check .
is regex really needed?
simple strtoul will do the work, even if you need error checking.
Good points - much appreciated. I'll start putting together some tests to check these items.
うーむ、難しいですね。RequestFlowのままで良いとするとRequestFlowをちゃんとRequest継承にしないとSimpleAuthだと上手く動きません。
Requestって名前が付くくらいですし、本来ちゃんとRequestを継承するのが筋なんでしょうね。
We could wrap it which is why it requires "a little work"... but there might be other solutions as well.
I know there's no unit tests on the old cron jobs, but I'd be +1 on some unit tests for these now.
Should hint at grunt usage in README.
Looks slightly strange that one line gets the repo type, and the next appears to assume git (GitServerRoot).
Right on, man. I love this stuff.
Thanks for clearing that up! I was confused about that section of code. Also, I commented out the ccase2 in the most recent revision since I realized it was not used anymore..check it out! eclipse IDE refactoring is fun.
Hmm, that doesn't make much sense - I'll write something better.
end with a period?
With uninit, only one component will get to use it. If multiple components has the function uninit the last one will be the one that gets called right? This is why the RemoveComponent event was created instead.
Uh? The QR code is executed when the page has been downloaded.
...and I don't think that was enough.  Trying more stuff...
Ah, okay. The fix with lower case extensions is all commited and checked in.

Woosh!

On 2/28/12 10:29 AM, Micah Walter wrote:
> Oh sorry, all the extensions should be lowercase jpg ( S3 is case sensitive )
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/straup/collection/commit/0b75be025f3f14103f5ad6935b16772188f1c2d7#commitcomment-1022319
>
Urs,

The implementation wasn't quite correct, I think it did end up messing up
Infernals slightly with that merge. I finally corrected it today, though I
think there's still some changes I am going to make to clean up the code a
little bit and make it more intuitive for users.

-Sean

On Wed, Feb 1, 2012 at 11:38 AM, Urs Reupke <
reply@reply.github.com
> wrote:

> @JMobius, @the-smith,
> in the light of the current discussion on Infernals, is this correct?
> Shouldn't Infernals resonate with either nothing (sidebar p173: 'lose all
> magical material affinity') or with vitriol-tainted materials?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/anathema/anathema/commit/25ac3b18c6d5b76b2ec0f7ce7158273371195448#commitcomment-923727
>
they should
Since we don't seem to get Kohsuke's attention this way, I've opened an issue: https://issues.jenkins-ci.org/browse/JENKINS-12327
## Usage

```javascript
sandbox.use( 'misc-core-module', function(module){
   // At this point, the sandbox granted access to 'misc-core-module'
   // You are given a copy of the core module via param
   module.foo();
});

```
короче. вот моя наивная попытка сделать первые правки... надеюсь больше так глупить не буду
for empty you could bypass `reduce` and exit early `if (list == 0) return 0`
>3< 嘛 小小的建議，各個版本的RM對素材格式的要求都好麻煩，如果可能的話希望你的版本能同時兼容XP和VX的人物圖形和地圖土塊。這樣會很方便開發起來還很容易
<xrds:XRDS
                xmlns:xrds="xri://$xrds"
                xmlns="xri://$xrd*($v*2.0)">
                <XRD>
                     <Service priority="1">
                              <Type>http://specs.openid.net/auth/2.0/return_to</Type>
                              <URI>http://pypi.python.org/pypi?:action=openid_return</URI>
                     </Service>
                </XRD>
                </xrds:XRDS>
постоянно делать коммиты это не очень хорошая идея, т.к. оно тормозит при перестройке индекса. Лучше или коммитить по окончанию перестройки, или использовать commitWithin
With a basic WAMP configuration, you might need to uncomment mod rewrite in your httpd.conf file for this to work properly.  I don't know about the default apache configuration in mac/linux evironments.
Use single quotes or double quotes consistently (e.g., 'up' not "up") throughout. The google pylint tool will catch stuff like this.
We've already moved to postgres, SQLite was slow as balls. Is it worth the effort keeping this general in case we want to move away from postgres?
This is merged with a compromise for the formatting issues - you both have valid points
No language is used.
Wait we had Easter eggs?
disregard my previous comment, you have to restart the session for it to take effect.
I still don't know if this is really so useful.
stop placing returns at useless locations kthkbye
Sorry for that. I quite new to Contributing, Will make sure i do it better next time.
I think this code does not perform what you’d want it to perform: the thrown error will not be wrapped in a `Promise`.
You should replace it with:

```scala
case _ => throw Errors.BAD_RESPONSE
```
Also,
Are you using commandline or gui (if commandline, then I can show you how to quickly separate this commit into a specific branch rather than use the master branch)
I don't know why but locales for comments doesn't work, I supposed it's work for the latest version only ?
http://lists.gnu.org/archive/html/bug-bash/2008-05/msg00040.html
Hmm, I’m not so keen on this. Names like “English”, “Francais/Français” etc are meaningful to humans, I don’t think they should be used for internal language codes.

Also the locale codes allow for different regional variations, e.g. “en_GB” versus “en_US”, “fr_FR” versus “fr_CA”, that kind of thing. There’s a reason such codes are used for identifying locales in GNU systems; I think you should use them too.
The above condition doesn't look right..

For legacy multi slave clusters you should be testing "@multiSlaves" ..
For new clusters, whether slavesPerNode = 1 or >1, you should be reading the groups..
I [did](https://github.com/Compass/compass-rails/commit/bddaaad7184e9a8135edf487e4a2281bd8411533#L18R51)! The problem is that the sprite load path is different.
URL Filter пока в стадии недо-API: какая-то дополнительная работа с ним выльется в кучу проверок на JS (тормоза), поэтому проще выдать число равное тому, что на кнопке быстрого отключения.
The tell me why is still here https://github.com/cgeo/c-geo-opensource/blob/master/main/AndroidManifest.xml#L14
log fine
it's for auto detecting the path of programs, rather than hard coding them. i switch between multiple systems often, so it's useful. try it out in the scratch buffer.
We should probably rename this file to something else, just to avoid confusion with this file and the actual popcorn.js file
I have it fixed, but pushing to github from the wireless here isn't working. (probably a weird firewall) I will push it later.

As you might expect, this improves performance.
преобразование массива коллекции в объект. пришлось делать костыль чтоб связать.
Oops. ;)
prefer insertRow(i) (or insertRow( rowCount() ))
i was wondering why not allow overloadSetter and overloadGetter pass any additional parameters? For examples, what if i wanted to call an overloaded method ::set(attributes, options) where attributes is a key value pair object with the actual keys and values that i would like set and options may be another object with options for the method. I thought it was useful and implemented my own by simply applying the additional arguments to the call.

var slice = Array.prototype.slice // in closure scope
...
self.apply(this, [ k, a[k] ].concat( slice.call(arguments, 1) )); 

and when not usePlural

self.apply(this, [ a, b ].concat( slice.call(arguments, 2) ));
I have traced down the errornous logic to SaxParser.js parse(). After the change it no longer treads <body> level <p> elements as valid; instead it simply takes their text content. Thus, all <body> level <p> elements are collapsed and new lines removed.

parsing:p
SaxParser.js (line 167)
isValidElement:false isShortEnded:false
Of course I meant "commented out" by default
Been meaning to do this, but it'd be better to organize within a namespaced folder... i.e:

```
lib/turing_tarpit.rb
lib/turing_tarpit/tape.rb
lib/turing_tarpit/scanner.rb
...
```

I'd expect `lib/turing_tarpit.rb` to use `require_relative` to load all the files in the `lib/turing_tarping/` folder. Definitely would accept a pull request for that if you don't mind doing it.
Well spotted.  Yes it was a lurking bug.

Previously when creating the heaps it would have created one for 2048, 4096 and increasing sizes up to heapsize/20.  However if the heap size is low so that heapsize/20 < 4096 it would have meant an allocation request between heapsize/20 and 4096 might have got confused.  The change of 20 to firstFractionHeap is similar.

It allocates all allocations up from (maxheapsize.. (maxheapsize/20, 4096) to that same heap rather than potentially creating extra ones with little benefit.
Shouldn't this check be moved up to line 126?
some of the responses you replaced, replaced real iphone 4S responses. Not a big deal, but I like to try to keep it as close to the real thing. Some of the ones you replaced are alternate responses for the same question, so we need to KEEP BOTH not just replace ones that already exist. When I return home Im going to start on making multiple responses for the same question.
Tried to compile with  following errors

[ERROR] /home/pslegr/development/AS7/EXP/as7-console/gui/src/main/java/org/jboss/as/console/client/core/StandaloneGateKeeper.java:[13,7] class StandaloneGatekeeper is public, should be declared in a file named StandaloneGatekeeper.java
[ERROR] /home/pslegr/development/AS7/EXP/as7-console/gui/src/main/java/org/jboss/as/console/client/core/DomainGateKeeper.java:[13,7] class DomainGatekeeper is public, should be declared in a file named DomainGatekeeper.java

Is there a typo in the class name ?
StandaloneGatekeeper vs. StandaloneGateKeeper.java
あーなるほど…そういう事ですか。了解しました。

getDocumentMethod()メソッドは1度しか使われてないですが、APIとして公開されているものなのでスニペット等で再度使われると無駄に$_REQUEST['q']を作る処理が走るので、どこか別のところに置くのもいいかもですね。
例えばDocumentParserクラスのイニシャライズの時とかです。
(多分、問題はないんじゃないかな…)
This breaks non-node.js compatibility because no `module` obj exists
There is an SvgWidget class you can use and return.
I switched to python-mode and implemented the expansions based on py-* functionality rather than python-*. Let me know what you think :)
Yep, I think this is key to F5 v11 support, and in general, the providers should either be for a specific version or update them to support at least v10 and v11. I'm not too concerned about v9.
Sorry, my editor removes trailing whitespaces.
Thanks, Kris.  I've touched this up and cherry-picked it at 56026863e8dc0cb40d68846b3cbbd0e8c43381a0.  Cheers!
Where do "0.00001" and "0.000001" come from?
Yes. "asleep" doesn't mean much for a phone, since it is still on and can receive calls.
You don't need this line here because you are using it below to mark the LI that has been completed. By having it here all the LIs get the .listed class.
The Liferay IDE logo in Eclipse - should highlight Liferay IDE icon (the white cube).
Nope, it's not a sugar, it actually changes behaviour. The demo
piecharts2.html will work in a different way after this patch lands.

On the other hand I don't think the current API makes sense. What is does
is in a situation where you have this config:

values: [10, 20, 30]
hrefs: ['abc', 'def']

you will have a pie chart with the *largest* elements linked, because in
the middle of creating it the code sorts the values, but doesn't update the
hrefs order.

I think we should break this 'feature' and introduce new saner behavoiur.
The other part of my email was the sugar I'd like to go towards. But this
would be after the stabilisation phase.

Thanks,

Karol

On 21 February 2012 15:04, Richard Zurad <
reply@reply.github.com
> wrote:

> Makes sense. I'll see about merging it into `next` for the 0.5.1 release.
> I want 0.5.1 to have as little API changes as possible and just address
> stability (lockdown of the API and thorough testing/docs) and bugs, but
> since this is sugar, I don't think there will be a problem including it in
> next.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DmitryBaranovskiy/g.raphael/commit/a64f080fa07206021a914f76887c56f25a92129d#commitcomment-992399
>
fix in commit https://github.com/eric-wang/dojox_application/commit/8a94e8b000bb05b95796cc6216858711a5db31c2
Good job. You're doing what I should have done at first! ;)
There is an updateImage() method in this class to help and remove some code. It takes a path! :)
Compilation error:

```bash
  Building modules, stage 2.
  MODPOST 15 modules
ERROR: "wifi_set_power" [drivers/net/wireless/bcm4329/dhd.ko] undefined!
  Kernel: arch/arm/boot/Image is ready
make[1]: *** [__modpost] Error 1
make: *** [modules] Error 2
make: *** Waiting for unfinished jobs....
  AS      arch/arm/boot/compressed/head.o
```
need to add trim()
Yeah.  I figured it would be easier to just do it than explain it. :)
the Uint8Array will let me use bson.js in the browser aswell. need it for the game demo and should let me split bson out as a seperate npm module
Why is the username and password on GitHub?
`Logger.info`.
did you modify them when you rebased? i just applied them as-is.

perhaps its something specific to the zte blade then, as the errors seemed to be around wifi settings and translations.

i'll have another go at some point.
Right, I was aware of that. When using checkboxes, it makes most sense to
be implicating an OR in most use cases. However, $selected_terms there is
just being used (as far as I could tell) to indicate to the list walkers
what was already selected, so that it could properly check/select/change
the URL it the taxonomy term was part of the query. Exploding on only a +
sign would cause that to fail for a checkbox list of ORed items. Is
$this->selected_terms used for anything other than showing to the user what
was part of the request?

2012/1/11 Cristi Burcă <
reply@reply.github.com
>

> `?tax=foo,bar` means OR
> `?tax=foo+bar` means AND
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jlyman/wp-query-multiple-taxonomies/commit/383547c3b6948a82edd4b2e6bf0147b89ebc1753#commitcomment-857985
>
Hey Dyson - sorry it took me a while to get back to you. The biggest problem I've had in getting single sign out working is getting access to the session store in order to keep track of the session ID that corresponds to a given cas ticket.  This has been compounded by changes to the session store Rails code across versions.

The way it is working for me with this commit is to add this line to your application controller:
  include DeviseCasAuthenticatable::SingleSignOut::StoreSessionIdFilter

Did you try this? Or did you add your own before_filter?  I'm curious to know if adding this line to your ApplicationController makes my commit work in Rails 3.2.

I was planning to send a pull request for these changes but wanted to make sure it was working in my environment for a while before sending. Also - it would help to know that it's working Rails 3.2!

Thanks,
Jeremy
Missing closing back-tick "`" after "callback". Also, wrap "error" in "error event" in back-ticks.
I'll amend this.  
The "Recently Used" select is a original html  tag while  this select is imitated with <a> tag.  
As shown on [UX0.3, p.18], the width of the menu is bigger than the span .
Top level context should be the same as the handler.
Fantastic. I'll go ahead and close this issue. will also update the JIRA.
Yeah, there is #2226, and it was closed as invalid. The comment does not explain why.
Ok, I see the answer later down.
Yes, it's been there since 2.0, it's the shorthand syntax to create an Actor, can be dropped now when Roland has created the more rich DSL for it.
I agree with Viktor
Please restore to have them on one line each, saved vertical space.
Lets pull or comment this out
Yes i guess you are right, only the max limit makes sense as auto-managing.
or even simpler, ``set('aeiou')``
Just FYI, keep an eye when reviewing other people's stuff that the PS1 is consistent at $. Sometimes they get copy/pasted completely and pick up things like username or host, or even as simple as the writer thinking the convention is #.
Should be sMediaMounted instead of sExternalStorageState.
To fix this or our current development? We're miles ahead now and that will have to wait for post-Frodo and what will inevitably be a messy rebase and even longer review. There's fixes I'm still back-porting to keep Frodo-friendly but main development has diverged significantly.
okay, never sure what I have to use there.
also need to check for supported bin ops here
Yep, just pushed!

On Tue, Apr 24, 2012 at 10:32 AM, Gert van Valkenhoef <
reply@reply.github.com
> wrote:

> >  import org.drugis.common.threading.AbstractIterativeComputation;
> >  import org.drugis.common.threading.IterativeTask;
> >  import org.drugis.common.threading.NullTask;
> > +import org.drugis.common.threading.RestartableIterativeTask;
>
> I was missing this from the drugis-common pull request but I see that a
> commit just appeared.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gertvv/mtc/pull/5/files#r725159
>
This is a necessary workaround, because URL is "/Users/username" at first call, but in next calls next_url is used from the API response. next_url is a full URL, such as "https://api.github.com/users/username/repos?page=1&per_page=1". If we always prepend _url_base to url, we will take InvaliURLException. (By the way, I can remove warning sentence since it seems irrelevant. )
Why do you add Whitespace at EOL?
>when a *click
`(CGRect)` is unnecessary here. The type should be inferred from the variable being assigned into.
@galaxas0 I believe that information was in the Xcode 4.4 release notes.
Remove these @param and @return annotations if you aren't filling in the information about them
I think error message is likely not ideal. Maybe should look around what other parts of numpy give. C- and F-Order are possible here, though I could make it as a switch too if thats better style (I really don't code C much).
а это почему убралось?
I like it too, so I changed it already :)
could you use `args.concat`? and also remove the spaces after the `[` and before the `]`? Also I think is better to check `if configuration['username']`
@spastorino It was removed in 2.0.  I made an effort to remove edge case code like this that increased the complexity of the code with little benefit.
Well, the current implementation doesn't work with hashes with strings as keys. However, doesn't hurt make it work.

@jeremy @josevalim thoughts?
okay. Just checking :D
Here, is it ok?
We can only serve JavaScript to a page.
And what are the case where name is `nil`?
I was trying to avoid "defensive programming".  If i understood correctly the rules about using strings and symbols, there is a consensus that for actual table names one should use strings anyway, isn't this so?
Good catch, It must date back before Akka 2.0. Will fix it.
Unless you start messing with implementation in MethodSynthetics, never. Do you think a crash is better?
instead of another `if`, could we just the `else` below as part of the previous `if`? just asking.
I don't know what motivates these casts to MacroContext, but at least don't spread them around.  If you must cast, encapsulate the cast in one place.
would it make sense to do the same in protoTypeArgs?
Actually, this does work in the compiled docs, just not with the Github reST renderer. So never mind.
Actually, a better way to do this is just `Poly(expr, n).LT()`.
Uppercase ``Density``
Should this raise an Exception in some cases?
So this is the only place where `Basic.__new__` is actually used?
When I wrote this, I was looking at how things were setup to derive the equations of motion of a system in Kane. Since the bodylist was anyhow required there, I thought it best to use what the user already prepared for Kane.
suported => supported
documentation for mode options is missing in this method, please copy them from protected method `copyDirectoryRecursive`.
```
	 * <li>newDirMode - the permission to be set for newly copied directories (defaults to 0777);</li>
	 * <li>newFileMode - the permission to be set for newly copied files (defaults to the current environment setting).</li>
```
@samdark Which spelling exactly we should leave in entire framework codebase and docs? PDOish `rollBack` or `rollback`?
Why removing this class. We should keep this interface for the `boot()` method and service providers in Silex should extend this interface.
"chang**e**s!

Sorry, I take a break I have to eat if I want to be able to finish the review of the pull-request before dying of hunger! :P

Btw, when I said it's huge, I mean it in the very very good way! :D
fixnum and int are almost the same if we use them as parameters.
consider leaving only one.
"A basic,"  should be "An example," or similar.
s/box/rectangle/
"most uuids do not...": where is this information coming from?  There are time-based UUIDs that do have an ascending order.  See http://docs.oracle.com/javase/6/docs/api/java/util/UUID.html
as a general rule I tend to avoid/edit out casual admonitions (i.e. sentences that begin with "Note that,") because it's ineffective emphasis and usually a pointer that we can say something more clearly.
This paragraph  no longer makes sense (the second sentence does not reflect the operation of MongoDB.)  While the initial version is a bit verbose, I would revert and then delete/rewrite he redundant second sentence and leave the rest of the paragraph as is, which would resolve the issue without introducing a regression.
rervert, re: in person conversation.
I'd have these all start with :data:`<name>` collection holds .... rather than "this collection lists." 

Collections generally don't "list" things, in the same way that you wouldn't say a table in a relational database "lists" something.
ah weird. ok!
Agreed, that's a bit convoluted.  To make the tests work on windows, we'll need to mock out the registry calls, right now its a functional test which is actually hitting the windows registry on the box.
I doubt $request contains the phpbb_config object. :P Copy/paste fail?
cast forum id to integer
I am wondering if atexit (or invoking $this->stop() from atexit function) will be more reliable for this.
This was originally used to get around a bug in marking items read, but this was worked around via another method (the link to an unread item will link to a page that marks it as read, then redirects to the item).
@callumacrae You mean you want me to re-add this line break?
Multi-line comments are evil. Please consider this:
```cpp
/*
    some useless lines (big piece, surrounds)
    /* some useless lines (small piece) */
    FAIL
*/
```
Or even:
```cpp
/*
    some useful lines
    // good debug, bitches */
    FAIL
*/
```
Thanks! Frankly, I was getting a little frustrated with Sinatra at one point, and I threw DRY out the window for a short bit. I think I have my work cut out for me in terms of refactoring the app, but I will take this to heart, as well as what you said about the mountability. I was mostly looking to get something quick and dirty up for the short term. It's high on my priorities now, though.
I've created a new var in Utils called PAGES. In screener I use it like so:
```
Utils.PAGES.SCREENER = {
  page1 : 1,
  page2 : 2,
  //etc
}
```
This allows us to declare the page #s in the appropriate controller, but still refer to a common PAGES object. YMMV.
see: https://raxaemr.atlassian.net/browse/RAXAJSS-382
docstring.
including a statement that a cube can live in multiple pairs?
I made it gh-833
We need to add the MIT license here.
In this case, looks like it will be of relatively small length, so I doubt it will matter.
あれ? pushすれば勝手にpull request更新された気がするけど。
Это один заголовок, просто в одну длинную строку редактировать код не удобно. Писал по аналогии с http://ru.bem.info/tools/bem/customization/
I recognize this is not absolute - it is opinion, but it is hard for me to see how "loads file into a stringstream" is not approximately 100% redundant with

```c++
void loadStringstreamFromFile(std::stringstream &stream, std::string file);
```
TEST_NAME is discussed in "The test database" section of topics/testing/overview
AFAIK, it's only used for the reraise definition just below. You can find the exact same code in six: https://bitbucket.org/gutworth/six/src/175a03e21623/six.py#cl-294
the point isn't just the double-filtering, though; it's also the unfiltering step we need for using `#call` with markupsafe.

the real goal of all this is to only have one layer of filter-wrapping at the very end.
OK with the patch but there's no need to rename the function to url2, please keep url. Also better if you keep Req as the first argument (less operations needed for the call). Thanks!
I am not sure this is a good idea as it will break resource management by context managers. Is it really necessary to open the port at instance creation? I think keeping the convention that `initialize` and `finalize` must be called by the user to open and close resources is good,
Change all of these to just be a single `AC_CHECK_HEADERS` (plural) and just list them.
Is it an API? I'm not sure and thus it probably should be moved into "internal" package.
Additionally ordered aggregates don't scale particularly well due to the need of establishing a single global ordering.  Not that I see any immediate alternatives.
The C++AL promises complete type safety so it needs to check anyway. An option might be to make FunctionHandle a template type where the template parameter is a function signature. Then the validation could already occur in the with the getAs<...>()  function. But I don't know if it's worth the effort for now.
Can you add doc comments to tihs?
For the record: #245
data files should not be added to github, please remove from your branch.
I think one of the stacktraces you were showing me suggested that Task doesn't have a request attribute. So this is probably wrong.
I am liking this. It gives the option of moving the search stuff to it's own `index.py` or `search.py` as well if your `models.py` is too cluttered.
(Uh, like that, only with tags in the middle.  :P )
This line didn't change the behaviour - was a more general comment on bottle behaviour, not on this specific pull request. I was just reminded by seeing the change here. Sorry to be confusing.
Thanks, I fixed that, also changed hah to sha1 and committed. 

Was having computer troubles and fat fingered something, I *think* I got it straightened out. 


On Sep 19, 2012, at 23:10, Jack Nagel <notifications@github.com> wrote:

> In Library/Formula/analog.rb:
> 
> > +  depends_on 'jpeg'
> > +  depends_on 'libpng'
> > +
> > +  def install
> > +    system "make DEFS='-DLANGDIR=\\\"/usr/local/share/analog/lang/\\\"'"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/bin"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog/lang"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/share/man"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/share/man/man1"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/etc"
> > +    system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog"
> > +    system "cp analog /usr/local/Cellar/analog/6.0/bin"
> > +    system "cp -R examples how-to images lang /usr/local/Cellar/analog/6.0/share/analog/"
> > +    system "cp analog.man /usr/local/Cellar/analog/6.0/share/man/man1/analog.1"
> > +    system "cp analog.cfg /usr/local/Cellar/analog/6.0/etc/analog.cfg-dist"
> > +  end
> Can't assume /usr/local, and you don't have to hardcode paths into the cellar, there are methods for this. Take a look at https://github.com/mxcl/homebrew/wiki/Formula-Cookbook
> 
> —
> Reply to this email directly or view it on GitHub.
>
Do we handle NaNs in the input data well? We should add a test case to check that. I guess that in this specific case, given that SPM uses NaNs to label out of brain voxels, these should be converted to zero by default.

Also, we should return a Niimg here.
It looks like this adds the `/` radius-width / radius-height. Shorthand. Could you please put it in a separate commit after your cleanup?
This will break the build on Windows because it's missing two closing parentheses. It should either be reverted or the unnecessary `Path::Combine` should be removed.
This loop seems unnecessary, since you either break out of it or return false after the first iteration.
Can you get the spacing to match the surrounding style?  Spaces before braces and between "for (" and "if ("
It's really the overall clarity.  This is a complicated feature being proposed, and I think we can find a clearer presentation.  I'm sorry not to be more specific.  If I get time soon, I may attempt my own re-write, but I find that waiting on my time to clear up hasn't been the best advice to patch writers recently.  :-/
Can you make this Class<?>...
Yeah, I was worried about that too.

@nciagra what are your thoughts about this?
this could be deleted.
Typically we set up our scss files with a wrapper, to make sure everything is scoped properly. In this case, you'd lead with

    .component.image-child-nodes {
      // rules
    }

That way all our rules stay confined to the component we want, and we don't have to keep typing `.component.image-child-nodes` a lot.

Take a look at some of the other component stylesheets if you're not clear about the pattern.
It looks like a good idea to check for this, but it seems like we'd want to check this for all cases. If so, then shouldn't this be the first check in this function?
DOH. That's what I get for making last-minute "code cleanup" changes without fully testing them. Good catch.
You mean *left* click, not *right* click, correct?
Is that first and second sentence in What is Brackets section? Take it as more artistic translation. I tried to keep the style and idea. I will send it to review and see what they think...
Please see https://github.com/dimagi/core-hq/pull/303/files#r2239341

When I run off the staging branch, a bunch of my saved reports that have a report_slug of daily_completions, daily_submissions display as "Deleted Report" in the saved reports list and fail to correctly send as emails (Display a message about the report being deleted since that case is handled)
It violates the rule: http://int.exoplatform.org/portal/intranet/wiki/group/spaces/development_standards/STC-0008
Static import this method.
If System.currentTimeMillis() is called too often, it will be a performance bottleneck. Consequently, from what I understand, we will have to be careful when firing events to the listener.
@tnm: In that case, I'd just go for `klass = Rugged.const_get err` for the first line. Errors and exceptions will lead you to the right place if there's a bad problem. Failure is good for the one line.
I'd say ‘, to the value “value”’
NOOOOOOOOOOO. The whole idea of going to MongoDB for User object is that we will have one object for everything: for user authentication data and for user profile data. All this should be combined into one mongoengine document.
I think that url can not be non hard-coded. :) if I try to adding it a name or something and calling it from {% url language %} i got errors.
This was not defined before?
yep, thats probably the best things to do (also to check load/domready order)
Hmm... the keys are identical here to how they're used around line 137. 

And all the `change['something something']` is matching all the column names defined in https://github.com/mozilla/balrog/blob/master/auslib/db.py#L598
Good to keep it though, yes. There are too many nested divs and things might get confusing otherwise
Should we close all of these tags, or is that not cool anymore?
I'm setting the width to match the container width, which can be less than the width of the screen with a small video. I clear the canvas on line 47 below
https and no butteruid
storeprojectData.media[ 0 ] just so this line isn't so long
Make sure this is filed.
you pass window.$ in as $ in the anonymous closure already.
Is there anything that actually uses this file? I tried pointing to it from the `test.html` and it looks like all of the urls are wrong. Is there a point to having this file setup like this, or is it misleading? `complete-config.json` makes sense in the `test` dir, so maybe we should just get rid of this config dir completely and just roll `default-config` into the `src` dir.
I'm really not entirely sure. While I'll remove the listener for the sake of good practice, will this just get GC'd since references removed completely?
`result.push(operations.substr(opIndex, currentOp.length));` will not create 2 additional arrays
Do you mean my implementation or the method in general? If you're reading large files, you should probably use `read`.
We should give these things better names. `clientOutgoingTimeout` and `clientIncomingTimeout` or maybe some slightly shorter version of that.
Why is the 'static' keyword removed?
Agree that things may cause breaking here.

I have not found a good way to deal with relative/absolute imports in a script that needs to serve both Py2 and Py3. Any advise or pointers on how to solve this? (From what I understood, as it is a multi-file module,  __init__.py should handle the module import not pyfirmata.py, but it does/did. )
Fixed in 	6535894
This is already covered in the title, so I think I'd nix this one because otherwise we've got it twice in the feed.
Just c -> return $ action c val
s/counter/mcounter/ (or just put it right into the case)
Pour rester cohérnet au niveau des dates => expiresAt. C'est bête, mais quand tu fais getExpire tu sais pas forcément ce que tu recois (une date ? un booleéen ?), alors getExpiresAt() tu sais direct que tu recois une date.
Maybe add the possibility to set this via options (auth needed or not)
But it's between #ifdef and #endif.
Does this site http://worldmap.harvard.edu/ use geonode?

If so, why there is no acknowledgement?  :-(

On Tue, Aug 14, 2012 at 12:43 PM, Jeffrey Johnson
<notifications@github.com>wrote:

> In geonode/maps/models.py:
>
> > +
> > +            map_layers.append(MapLayer(
> > +                map = self,
> > +                name = layer.typename,
> > +                ows_url = settings.GEOSERVER_BASE_URL + "wms",
> > +                stack_order = index,
> > +                visibility = True
> > +            ))
> > +
> > +
> > +            if bbox is not None:
> > +                minx, maxx, miny, maxy = [float(c) for c in bbox]
> > +                x = (minx + maxx) / 2
> > +                y = (miny + maxy) / 2
> > +                wkt = "POINT(" + str(x) + " " + str(y) + ")"
> > +                center = GEOSGeometry(wkt, srid=4326)
>
> We should be using Tim's helper method for this.
> https://github.com/GeoNode/geonode/blob/dev/geonode/utils.py#L290
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/GeoNode/geonode/pull/364/files#r1376366>.
>
>
Before, we had success and error; however, with pulpv2 an error at this level is only in connectivity to the client/agent.  As a result, we changed it to be 'general' (which captures success and other errors from client side) and 'rmi_error'.  The 'rmi' came from some pulp terminology about it be an error during the remote method invocation.
It would be better to put this String extension in src/lib.
eric pointed out that the assert is the odd part, and that it should probably be assert_equal.   I'm not really sure anything else it could be (since it was like that before), so i'll change it to assert_equal.
In Satellite that link could potentially be appropriate after reading through it, but in the upstream version, linking to a set of Terms of Use for a website that is not even the web application itself just doesn't compute.
That shifts the coordinate too, doesn't it? Or does cartodb ignore digits after 7 decimal places? 3.10000009 is arguably closer to 3.1000001 than 3.1000000, no?

In any case, good call about the negative numbers. Adding now.
i see. so this actually makes it back-compat, no?
Can this be rewritten as the 2-arity version with an empty option list? No sense in duplicating code.
Any reason not to use >= for clarity?
Yes, it was a bad joke... I'm from Argentina :')
What I ment, was the variable name to be consistent with all the other ```$indexes``` inside ``` lib/Cake/Model/Datasource/Database/Mysql.php```, pure cosmetic surgery.
Why not make this a method on the test case?  Seems like this will eventually cause a problem as a global function.
yes I think this can be improved
I'd simplify the vocabulary in the entire preceding paragraph.  We have a sizable number of ESL users who would find this hard to understand.
The problem is that MSTest would not copy git2.dll into the `TestResults` directory, so when tests that used LibGit2Sharp would fail because the dll is not in the path.

[Here's some documentation I found on the matter.](http://msdn.microsoft.com/en-us/library/ms182475.aspx)
Thomas Kluyver, on 2012-06-16 05:32,  wrote:
> Isn't it simpler just to do `HOME = os.path.realpath(HOME)`
> unconditionally? If it wouldn't trigger this `if`, then doing
> it unconditionally would make no difference.

good point, just applied this change

Min RK, on 2012-06-16 21:23,  wrote:
> Why not use existing utils.path.get_home_dir, where you fixed
> this the first time?

That function ends up returning a directory under windows, where
I guess $HOME isn't typically defined. It looks like in *this*
code, windows ends up getting a nonsensical HOME so that no
directory ends up getting replaced with ~.

Does that seem reasonable enough?
Maybe I'm missing something here, but as far as I can see, we're not able to use `included` here because:

1) these proxies have to be defined in the object's singleton class, so each decorator has its own proxies (which is obvious because defining methods in Draper::Base will propagate these methods to all decorators that descend from it, even to those that have no model related to ActiveModel whatsoever)
2) the `included` method is called before `initialize`, when we don't yet have an object instantiated, so we can't access its singleton class.

Could you point out another way of calling `create_proxies` and yet have access to the object's singleton class?
I'm not sure what the problem is: isn't that intended? On Windows it'll always use the bundled copy to make building easier.
I added the definition of arguments + a little example
Removed the background-image (a dummy callout), and hence, needed to reposition the box to keep it in the same position as now.
I can be down with what the kids are doing these days. :p

I'll make the change!
Some more missing semicolons: lines 166 and 169.

Again, not a big deal (as in, it will usually still work), but it's definitely best practice to include them.
In production we have these dirs in /home/socorro:
fallback/
persistent/
primaryCrashStore/

We could use something more descriptive than "temp" but otherwise seems ok.
I wonder if there's a neat and simple way to compare a requirements.txt file like this with what's available on pypi.
Don't do a `git commit -a` - these got added automatically and should not be here.
Yes, there are no other data coming from _POST - password and activation code is readded separately below.
Modified to RuntimeError, but raising LoadError, is this OK?
This one uses Android's `Process` class to set the priority. I meant to check into the source as to whether or not it simply calls `Thread.getCurrentThread().setPriority`. Will do that now.
What is the root though? How do we know? ;)

We definitely need to reuse object instances though (such as the BindRequestListener instance). I'm going to fix this soon.
Done. Somehow check_cs is missing on master.
It is needed for the `FormExtension` tests. It is an inherited dependency from the Form `AbstractLayoutTest` base test class.
I would add the class on a single element in the dom (the common parent of these nodes) and then update the CSS to

```css
.collapsed-menu-parents #resume,
.collapsed-menu-parents #collector_content {
    margin-left: 60px;
}
```

Btw, this should allow to handle the menu with the same class too (by using the appropriate selectors).
And ``collapsed-menu-parents`` seems a weird class name.
is there no equivalent for this one ?
Isn't it clear enough that you can either pass an array or a single constraint?
the FormBuilder currently accept passing a FormBuilder in the ``add``. If we introduce the interface, it will have to accept any implementation of the interface, not only the FormBuilder implementation (otherwise the interface is simply useless)
style: use // comments except for scala docs.
also, I run M-x untabify on saving.
NSLogs here.
If you're finding by event name, I'd call the variable `event_name`
This can probably be just `build`
gharar shod comment hato dorost koni
```css

/* Menu section
-------------------------------------------------- */


```
maybe we should have a common practice for debuting messages... By the way, please use the trace mechanism instead of std:cerr..
Can you please tell me what this is about?
Ack, sorry.  I changed the filename for simple_output to complete_output since it's really meant to include every use case and not a simple example at all.
ok

maybe `qq.step()`? ;-)
nit: @params before @constructor and you don't need {ModuleLoader}
this should probably be done before we go to the effort of getting all the nodes and extended nodes
My convention for test entry points is `xxx-test`
lol indent
There are still some controller classes imported here. Please change as mentioned for the other file. The usage of ControlledProcessState should probably just be changed to use string constants
为哪样要注释。如果这些行不要，就删除。
fcontactToRender -> contactToRender
For getting the expected behaviour between states (explained in my first comment) I've moved all this 'callback' concept to 'events handlers'. This change is the key point of the code, with this new approach we are not going (unless there is some scenario missing) to loose the state change info.
Isn't it implemented?
I wonder if we can remove the need for this comment by adding some classnames. So for example if we add the class 'card' to li and the class 'icon' to img, that would make it very obvious and the comment would be not needed anymore.
`showWithPseudoEvent` ? The verb should be show here i think.

and `configuration` -> `config`

and function name
I will trace on this and see if I can make some update for it.
Also metadata from gecko is workable or not.
Isn't this identical to mozVibrate(100)?
What does the `-%}` mean vs. `%}`? I keep forgetting to ask.
Erm... you might want to change your password now? :-)

Also, since it's hard to do automated tests for a module like this (unless it's doing something that doesn't require a server, like parsing behavior) because there is no node.js IMAP server module, I generally don't include tests.
This class seems to have a lot of duplication with CustomBuilder. Is one of them meant to go away after the transition? If so, that's ok.
fixed :octocat:
~~~
既存の者を編集するためのリンクがあります。
~~~

~~~
既存の物を編集するためのリンクがあります。
~~~
This should just be private instance method.
why aren't we using zeroVector in the "zero : HLLInstance"?

Shouldn't we move it just about that, and just point to the zero vector there?
main => primary
Hmm, not sure about that.  If a chunk's observers goes to zero, it is queued for unloading.  This doesn't count as reaping.

Reaping is for chunks which have no observers.  For example, chunks that are generated due to populators are loaded, but have no observers.  This means that the number of observers can't drop to 0.
My mind is pretty blown right now if this is literally the only place in the entire code base that requests the info variable from an action.
Yeah, the tests are rather optimistic as well and don't cover the failure cases of these commands at all. This is definitely one of the areas that need solidifying. Also, there are no tests at all for the `-n`, `-N` and `+N` options, which is a little wild west. :)
It would be good to know why this is happening, rather than just hacking around it.
this would make it feel more like node's environment. plus some jsons might be generated by your tools and you perhaps wouldn't want them to know about module.exports.

I'm seeing problems emerge from making components work together with non-component code and the native json support would cut down some of the barriers.
There were to occurrences, in fact.