labofsahil/github-event-dataset-2012 huggingface dataset:
the immediate mode (above, without the array argument) only works if you're certain that the class has already been loaded.  Otherwise, I think you'll get an error.
Forgot to mention: Fixes #1215
Use AMD format.
Thomas,

I think I see what you are getting at, instead of having the files in your XBMC "TV Show" being Symlinks back to your SIckbeard Download directory you want it the other way around. That way you can easily delete the files in your Download Directory without also deleting the files in your TV Show directory (since if you delete the actual file you break the symlink and have a orphaned file).

I ran in to the same type of issue in my own setup and ended up changing from Symlink to Hard Link:
http://www.cyberciti.biz/tips/understanding-unixlinux-symbolic-soft-and-hard-links.html

A hard link is two separate files linked to the same inode, this way we save disk space and can delete either file.

If this sounds like what you are looking for I can post my changes to use Hard link rather than Symlink.
I still want to show the pretty tooltip if you forget the `http(s)://` - just remove the suffix part of the regex
Could you please only upload icons and images that we relay need? I don't think we need a background image for example ;)
I know it looks odd, but the other switch format is actually PEAR CS.
What's this for?
I mean, why?
This is what is required:

 All certifications must require that a Complete EHR or EHR Module developer conspicuously include the following text on its website and in all marketing materials, communications statements, and other assertions related to the Complete EHR or EHR Module’s certification: “This Complete EHR is 2011/2012 compliant and has been certified by an ONC-ATCB in accordance with the applicable certification criteria adopted by the Secretary of Health and Human Services. This certification does not represent an endorsement by the U.S. Department of Health and Human Services or guarantee the receipt of incentive payments.”;

We could just add that to the copyright file, since the project IS the marketing material. adding an about link might be an OK approach, however the title bar is starting to get crowded.

I think it would be good overall to have the fact that we are a certified EHR right on the main screen, that's good for the project.  And I think it's only fair to have and OEMR.ORG link in a visible location.  Open-EMR.org list all over the place already.
а надо ли тут целый класс ради двух функций?
It looks like the error is in the JNI C code. Breaking out the Apple USB API to try and fix
So I keep struggling with blending. I created a new class which can play concurrently more animations but such blended animations look really bad with MD2 model.

Not sure if I'm doing something wrong or if it's simply that MD2 models were not created with blending in mind (for example different animations have different lengths). 

Do you remember how it was in the original Quake 2? Did characters sometimes do multiple animations at once (like walking blended with shooting)?

For example here enemies seem to always stop before shooting (so that always just a single animation is played):

http://crystalin.dyndns.org:8080/GwtQuake.html

But maybe it was different in the original implementation? I think single animation only restriction would interfere a lot with multiplayer, human players can walk and shoot at the same time. How was that animation handled for characters of other players which can be seen from 3rd person perspective?
I think I might prefer `CameraGizmo` too, since the `Gizmo` looks kind of cute too :P
ah I see line 111
Adding a .gitignore file like this to the tree is not a good idea as the files it needs to contain will be different for everyone as it needs to include temporary and settings files used by the IDE, file browser, etc.

I'd rather recommend creating a separate build directory for yourself and then pass the path to the source instead of '.' when running cmake.
@koshke - Sadly, I believe this change breaks compatibility with v0.8.9, which doesn't have a `coord_transform()` function.
Maybe this is already on your to-do list:
I am thinking of having a long background noise file - like ppl talking at a party - and then selecting a random section with the length of my generated wav thereof, to merge. The lesser resource intensive way would be, if the WavFile class did not read the entire data stream upon opening. You could then just pass an offset parameter to the merge function and use fseek() - somehow ;-).

P.S.
In case you didn't notice: I made line comments at:
https://github.com/dapphp/securimage/commit/71b9e4aaa65d7ffcef3524f5ca80fea24d4dc330
https://github.com/dapphp/securimage/commit/d87714a715ef2254d01e4146e2962a06c1a41418
This should do the trick ...
https://gist.github.com/1678609

I don't actually sure it compiles :(  someone else will have to do the debugging on this one.

But I think the general idea is simple enough. This way we'll at least see if that's the actual problem.
maybe {local, ?MODULE}?
I do not agree with this commit : FSAL_DIGEST_SIZE_HDLV* are not the same as sizeof(p_in_fsal_handle->data). 
The digest is (by definition) smaller than the FSAL_Handle
@ozten - Thanks for the info, I have to explain the useJavascript flag better.  I added the `useJavascript: false` for the `unsupported` and `cookies_disabled` dialogs so that users who see the dialogs do not have to download any Javascript.  This flag is not affected by the user's browser or any settings, so users who do not have Javascript enabled are not going to receive a different set of resources to those that do.

My reasoning for keeping JS from being served to these two dialogs is that users who lack support for localStorage or postMessage have a good chance of being on an older phone where downloading a large resource may be a concern.

Since users receive the same resources whether JS is enabled or not, does the `Vary` concern still come into play?
normally thumbnails are not shown.
i think most of the time brender is rather used to render animation, not so much to check and output video formats...
Which permissions does this user have?
tomorrow, maaaan!
Nice, thanks.   Do I correctly recall that 'callable' disappears with Python 3?
We should write a test for this or it won't get accepted into the core robolectric project.
<version>1.1-R2-SNAPSHOT</version>
Gotcha.  Thanks for that. :)
I would assume that the param should be $limit = null, otherwise we have the same problem as before
This did the trick for me too, thanks :)
HI,

I think this should be configurable by cinnamon-settings. :)
Yes, you are  right.
Yes, that was added as part of [the change from `dict.iteritems()` to `dict.items()`](http://docs.python.org/library/2to3.html#2to3fixer-dict).  It works fine without it, so I'll remove it.
Sorry I wasn't in touch--there's one more day of meetings ahead of me. I'll be much more sociable on Friday.
themes themes ?
I thought about that as well, I guess I could just check the that response file currently being processed is not the same as the one requested to be processed. I thought it would be useful to have response files in a response file, but maybe not. Does csc allow response files in response files?
We're rebuilding the abandoned plugins for R6 and beyond.  This is our helppages plugin, and after fixing I'm getting some errors, so I pasted for reference.
todo: split string by spaces in a loop to be able to select children i.e. "#test .test2 #test3 a"
I believe that's the case, but haven't looked at it in a while, but the
initial target was needed to provide a consistent mouse/touch interface.
Now we need to work in click/tap/taphold/etc. and will be complete.

On Sun, Jan 15, 2012 at 7:04 AM, Paul Connolly <
reply@reply.github.com
> wrote:

> I'll have to look at the example I wrote tomorrow (it's in work), and test
> to see if I can resolve this properly. It's most likely just a case of not
> using the fnEnd signature properly on my end. (I assume el would play the
> role of initialTarget?)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/gibletto/My-Library/commit/f11ea1fe7f729a649cac003c482cae4e4f16f950#commitcomment-868487
>
In discussing with Manu, normalization is moved to an external spec based on RDF. That will serialize a result, which will most likely be something in N-Triples like format. In any case, it's really just treated as a blob that is signed.
If we do put all the upstreams packages on github (and I'm not saying it's a bad idea), it would probably make sense to put them in a separate repository, and include it as a module.  That would make it easier to kill old versions in order to save space.

As for making it possible to build offline, I think that could be achieved simply by skipping the invocation of "wget --continue" when the file already has the correct size.  We could create a download script that's a little smarter than just doing wget blindly.
`for in` loops over all properties on the object including those inherited by the prototype, which can lead to unintended items being included.
I agree but it's important to return a 404 status code if the resource doesn't exist.
This is so gonna blow...
I'd prefer if you could use an easy-to read if statement.

```
if (Array.isArray(arguments[1])) {
   args = arguments[1];
   options = arguments[2] || {};
} else { 
  ... etc
```
This is also problematic:

const vecter TOUCH_INVALID_VECTOR = <0.0, 0.0, 0.0>
const vecter ZERO_VECTOR = <0.0, 0.0, 0.0>

--Adam

On Sat, Jan 7, 2012 at 11:08 PM, Strife Onizuka
<reply@reply.github.com>
wrote:
> It's too much work to track signatures on the wiki (which article of a set of articles that all use unarticled constants contains the signature definitions for those constants). The solution is to put the declaration in the template that builds table of constants used in all of those articles so that the signature shows up in all of the articles. Consequently any script that reads the signature block needs to ignore redeclarations.
>
> Example:
> https://wiki.secondlife.com/wiki/LlGetPrimMediaParams#Signature
> https://wiki.secondlife.com/wiki/LlSetPrimMediaParams#Signature
>
> I should make a template to make it easier to add signatures to the block and to track who is adding to the block.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BlindWanderer/lslint/commit/06977c8cefa28b76ef8ae7d9d16a6c0d30f1f01a#commitcomment-847217
嗯？改了咩？我只是问问 不过从还在用那些文件名那些词来看是没改吧……
It sounds like this suspends the entire process, instead of just this thread. Maybe pass a timeout of 0 and use sleep to wait instead?
Hi.I dropped my repo so I can reorginize my project.I'm going to make the library look better.
Sorry I totally missed this!  Did you get it sorted?

Dan
Shnoulle: This change caused http://bugs.limesurvey.org/view.php?id=5892
Can you please take care of it?
Attention à l'indentation here ;)
They just report form data, and are already given pid, encounter form ID, etc... but like I said, I'll handle it anyway.  Thanks.
это я делал просто из идеологических соображений, т.к. retRe означет что мы хотим получить регулярку.
Но т.к. далее в случае false мы корректно обрабатываем ситуацию, то регулярка не обязательна.
Scripts von dir benutze ich nicht soweit ich weis,  aber cata base scripts sehe ich immer gerne.
Hihihihihi I did it!
Side note: my feeling is that the "c:geo in English" option is a bad thing to have. It complicates the code for no benefit, overwrites a system preference for a purely visual change (this is not the same thing as the "date format" option which changes the way logs are *posted*). This also encourages people to set c:geo in English when a language is poorly translated instead of submitting fixes.
A uncompressed jquery might give you better stacktraces, good idea
recursive fibonacci with memoization -- nice, dude, nice.
Sorry for critics, all the rest is super cool :)
fixed. It was a bug that I introduced when I did a CCSequence cleanup for issue #1305.
https://github.com/cocos2d/cocos2d-iphone/commit/3419a408d2e0bb248ac72abecb91ba62c41d1a70
Gross :/

At least I was consistent.
Haha, that's pretty cool that we found a discrepancy in jQuery :-D

I'll change it back to the original, but given that it's still an ongoing issue, I'm not going to bother bumping the version for this.
missing ``public`` keywords in this class (and implementation)
MMm, display our own page first. Then send them off to dropbox. Need to explain WHY they need to sign into dropbox before asking them to.
FIll in body tag and p tag in the above comment
Well, this may be unnecessary in the latest build, but back in the date of this patch, the SRT reader fails on all files, including ones exported by Aegisub itself. I added this dirty fix since my friend needed to get his job done. Maybe I will remove it in my next build.
Ty, I try it to do my best always, but sometimes........ :D Now I'm working on phase 1 on Goblins, so I stop to push here until I got all phase done
update to use 4.0.0-SNAPSHOT
Can you submit a pull request with this change?
I can't find SAI for www.wowhead.com/?npc=23859... didn't you forget to convert his script?
https://github.com/darkman1983/TrinityCore/commit/6053557de444ac90580507a64e4612c8488bdd3c

Worse?
It's hard to give the right credit as this patch has not been noticed by us before Retriman. Start contributing and we'll make sure you get the credit you deserve. Thanks Left4Info
A filter().exists() would be more efficient here.
Looks pretty straight forward. I'll take a look in a bit when I can hop into Linux.
Soooo just tried to flash the kernel by downloading from link and using
download manager to install, rebooted into recovery flashed and rebooted
and now just getting a Samsung boot loop. Power on Samsung for a few
seconds reboot same thing...same thing going into 3-finger recovery.
Throwing it in download mode to reflash recovery....Even if the download
was corrupt it shouldn't mess with BML8 (recovery partition) and then will
download again and try and reflash manually.

On Mon, Jan 9, 2012 at 8:35 PM, Paul Blair <
reply@reply.github.com
> wrote:

> The kernel is on my site. I've been running 100-1316, set with setcpu for
> the entire day with no issues...yet. I've been running benchmarks here are
> there and have played a game for about ten minutes. I thought about posting
> the kernel on the forums for people to test, but feedback from the general
> public is rarely very helpful.
>
> http://xda.toadlife.net/packages/CleanGB-MTD-Kernel-1510b5ac03.zip
> --
> Sent from my Android phone with K-9 Mail. Please excuse my brevity.
>
> dyehya <reply@reply.github.com> wrote:
>
> We can email through GitHub... That is awesome... If you can email me the
> compiled kernel, I can help test for stability especially under high load.
> Unfortunately with just 1.2Ghz enabled (1.1 disabled and not using 1.3+), I
> still have soft reboots randomly. Don't know if it is kernel or ROM issue.
> On Jan 9, 2012 4:00 PM, "Paul Blair" <
> reply@reply.github.com>
> wrote:
>
> > I might just remove the 1.4 step too. As I said 1.4 has never worked for
> > me either and since the whole point of my ROM is stability, removing 1.4
> > completely might be the prudent course of action. As for the voltages, I
> > don't have a lot of experience with voltages which is why I didn't touch
> > them.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851173
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851817
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851843
>
Lol. I was wondering about that. :D
-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

dyehya <reply@reply.github.com> wrote:

lol I just realized the problem *facepalms* I am using the BML version.
Didn't notice the filename on my phone...

On Mon, Jan 9, 2012 at 9:26 PM, Daniel Yehya <dyehya@gmail.com> wrote:

> Soooo just tried to flash the kernel by downloading from link and using
> download manager to install, rebooted into recovery flashed and rebooted
> and now just getting a Samsung boot loop. Power on Samsung for a few
> seconds reboot same thing...same thing going into 3-finger recovery.
> Throwing it in download mode to reflash recovery....Even if the download
> was corrupt it shouldn't mess with BML8 (recovery partition) and then will
> download again and try and reflash manually.
>
>
> On Mon, Jan 9, 2012 at 8:35 PM, Paul Blair <
> reply@reply.github.com
> > wrote:
>
>> The kernel is on my site. I've been running 100-1316, set with setcpu for
>> the entire day with no issues...yet. I've been running benchmarks here are
>> there and have played a game for about ten minutes. I thought about posting
>> the kernel on the forums for people to test, but feedback from the general
>> public is rarely very helpful.
>>
>> http://xda.toadlife.net/packages/CleanGB-MTD-Kernel-1510b5ac03.zip
>> --
>> Sent from my Android phone with K-9 Mail. Please excuse my brevity.
>>
>> dyehya <reply@reply.github.com> wrote:
>>
>> We can email through GitHub... That is awesome... If you can email me the
>> compiled kernel, I can help test for stability especially under high load.
>> Unfortunately with just 1.2Ghz enabled (1.1 disabled and not using 1.3+),
>> I
>> still have soft reboots randomly. Don't know if it is kernel or ROM issue.
>> On Jan 9, 2012 4:00 PM, "Paul Blair" <
>> reply@reply.github.com>
>> wrote:
>>
>> > I might just remove the 1.4 step too. As I said 1.4 has never worked for
>> > me either and since the whole point of my ROM is stability, removing 1.4
>> > completely might be the prudent course of action. As for the voltages, I
>> > don't have a lot of experience with voltages which is why I didn't touch
>> > them.
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> >
>> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851173
>> >
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851817
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851843
>>
>
>

--- 
Reply to this email directly or view it on GitHub:
https://github.com/toadlife/CleanGB-Kernel/commit/1510b5ac031c0d1ceb2787db5470f67e41e2ce25#commitcomment-851915
yep, copy/paste mistake, thanks
Never mind there is a better implementation for the keys in MojamComponent which use:
synchronizer.addCommand(new ChangeKeyCommand(index, nextState));

I think we should use something similar too.
Let's remove process_definition_id when you get the opportunity, OBE.
Right, I missed that indeed. If you've already got a good idea of what should be done I shall leave it to you, otherwise I can always have another go at it later, taking into account what you said...
Le 12/02/2012 12:44, Puddi Puddi ! a écrit :
> Bibizz dans ta partie il y a une erreur lorsque dans le code du tiny truc, il y a un caractère " " " ... bref je te corrige ça !
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/OlivierB/Polyjoule_siteWeb/commit/96ef74087c4df2c4a51b64c940eafadbef915a39#commitcomment-957625
Simon essaie de respecter le même design que les pages gestions des 
articles, gestions des membres,... ainsi que le même fonctionnement 
d'ajout et de suppression pour avoir le même schéma dans toutes les 
pages s'il te plait.
In the meantime I've found this: https://www.gnu.org/software/libextractor/ which includes a demo commandline program called `extract` which does the metadata reading for a ridiculous number of formats.
"M-x eshell" will create a eshell/ subdir in emacs-prelude, which should be ignored by git
This change should also the name to be extracted when json is on a single line. E.g. {'name':'test', 'something':'else'}
@sminnee @hafriedlander Can you please review? This caused some weird bugs like auto-adding all groups when we created a new member.
It was param1 now param2 ^^
maybe revert this (part2).
Mag ich nicht so. Das ist ungut programmiert :) Das ist wie im DataContainer oder so wo fix if-Anweisungen drinn stehen obwohl dieses Verhalten evtl. auch andere Felder/Funktionen möchten. Bin noch nicht so zufrieden mit dem :)
I hope this is a proper way to do this :-)
Actually, this method should be called getLiveLogoId() instead of getStagingLogoId().....
Again, extract these.
http://www.facebook.com/mgkyawswarwinn
much better pattern matching - good rewrite.  I'm tempted to benchmark and show it has same performance as previous version, just in case that was a botched optimization job.
Looks like you are missing the openssl gem.

Can you try `gem install openssl` and then try to publish again and see what happens?
Bah, I see now. Thanks!
Saw exactly this at the Poltergeist docs. Is this the missing puzzle in the Guard Jasmine README?
Thank you.
According to the N1 thread, that kernel with full hwa seems to be unstable so far.
`length` is a property, not a function.
eval is for slackers. it's not the best approach here.
@mattleff that's how I had the original query, but it's actually not correct. Correct me (again) if I'm wrong, but If today's date is Jan 17, 2012, then I want notifications that have a start date before or equal to Jan 17, 2012 and not ones that start after Jan 17, 2012. The same goes for the expire date. It's somehow backwards to how we think, right? :)
@KendallHopkins I think you're right I will try that.
No, tabs are valid. GitHub view just messes them up in here - they align perfectly well in your IDE.
Agreed, revised accordingly and updated pull request.
Agreed, should probably stick to `!whatever` instead of `not whatever` for consistency.
'sampleCharNames -- renamed (used to be called 'sc');' can go
<?php should also be split into <? and php for the same reason probably.
Didn't Geoff commented that disableWhen should better be a function?
Why do we care about passwords? Just doing `login($user)` seems to be all that is necessary for most cases (except where we're testing authentication itself).
See the line above my comment.
Re-add this page. See comment above.
Ah, the context to that link was not present in the diff. thanks
okay, color me clueless (I guess I don’t wanna know)
Wow, that's really mean
On Mon, Nov 26, 2012 at 8:09 AM, Patrik Nordwall
<notifications@github.com>wrote:

> In akka-camel/src/main/scala/akka/camel/CamelMessage.scala:
>
> > @@ -108,7 +108,21 @@ case class CamelMessage(body: Any, headers: Map[String, Any]) {
> >     * Java API
> >     *
> >     */
> > -  def getBodyAs[T](clazz: Class[T], camelContext: CamelContext): T = camelContext.getTypeConverter.mandatoryConvertTo[T](clazz, body)
> > +  def getBodyAs[T](clazz: Class[T], camelContext: CamelContext): T = {
> > +    val result = camelContext.getTypeConverter.mandatoryConvertTo[T](clazz, body)
> > +    // to be able to re-read a StreamCache we must "undo" the side effect by resetting the StreamCache
> > +    resetStreamCache()
>
> yes, that will not end well, body is Any and can be whatever mutable
> non-thread safe instance, so it's not much we can do about it.
> I would say that the recommendation should be that the user should convert
> the body to a real immutable instance in the first endpoint (actor) before
> passing it on to other actors.
>
> That is definitely a best practice (and people start doing that quite
naturally since the CamelMessage is hardly ever seen as part of the domain
and the normal immutable messages are easier to work with).

> This fix is still relevant, because you expect to be able to call
> getBodyAs several times (in same actor) with the same result (e.g. adding a
> debug log stmt should not break things).
>
+1

>  —
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/894/files#r2218705>.
>
>



-- 
Raymond Roestenburg

code: http://github.com/RayRoestenburg
blog: http://roestenburg.agilesquad.com
twtr: @RayRoestenburg
That's not Patriks fault, it's mine. You're right, something like this would be nicer.
```
case ((_, testResults), (_, multiNodeResults)) =>
  val results = testResults ++ multiNodeResults
```
i would like to see `updateRDY` updated (no pun intended) so that after recovering from a backoff it only sets RDY to a max of 1. This avoids the situation where you recover from a 120sec backoff w/ a RDY 50, succeed all of them and then drain your backoff to 0 immediately. In order for the exponential recovery to happen on the time scale expected only a single message should be RDY'd after a failure. 

The same sort of situation exists on the initial backoff side. if you are at RDY 50, and you fail all 50 at the same instance are you instantly at a delay of 120sec? One way to avoid that could be to calculate the delay time in finishLoop, and set a timestmp = time.Now().Add(backoffDelay), and ignore any success/failures till that time. (so that you are guaranteed the first 2sec backoff)

Also I've been thinking about the value of skipping backoff on the first failure, and only backing off after two contiguous failures.
Returns nil because of the println? Why use expect here?
nice method decomposition, good readability and provides ability to override at each level of granularity.
Capital D for consistency.
I generally like what you've done here but am not wild about having so much method logic repeated so many times.  Maybe the generalization you describe would be to just have a module_attribute called something like :configs, have the set files define the attributes, and then have the config method live in Set?
think carefully what is the value of `j:j+1`; if you are not sure, type `5:5+1`

note the precedence of `:` and `+`; use a simple example to verify it whenever you are not sure
The constructor function is as simple as it can be, no iterations, it just instances a already-processed Interface (OwnInterface) and returns it. Since this function is pontentially the one called more often.
This is a second blank line!
Here as well with the getClone
No need for the inner method, make the outer method @tailrec instead
Can you wrap the return statement with `{}`? Thanks.
Some like this:


    Some code

    {% block content1 %}
        Some default content
    {% endblock %}

    {% block content2 %}
        Some default content
    {% endblock %}

    Some other code
why not move this into fcitx-utils
This `if()` statement is redundant, it is handled by the `else` block already.
This is now fixed too :]
Here's a better source: http://stackoverflow.com/questions/2099830/unsigned-keyword-in-c
10.6 doesn't support `__weak` references. That's why I stuck to `__block`.
`return Undefined()` (not that it really matters but it's shorter)
Expensive. Make a copy outside of the hot path.
No, I think it's stupid.  But that's how fs streams work today, so that's what we get.  Changing it would be fine, but it should happen in 0.11.x
okay, the template string is dirty..
cleaned it now in my local branch, but there is still a reference to your own site `http://analytics.bluepyth.fr` here.. I'll google for the correct way to include piwik
should this method be private?
Why put in 3 extra words?
Le javascript doit être à la fin du body. Ça permet de charger la page plus rapidement. Le rendering peut se faire avant le chargement du JS.
Again, this could be removed. For local testing, python test_blah.py works fine and the module doesn't need to be mae executable.
is a naive timestamp really useable here?
here we would leak pkey, spki, spkstr_cleaned
The following should be logged:

* Cache miss
* Cache hit
* Failure to hit the cache
if (!cut) {
    return;
}
не покатит тут? очень много вложений иначе
On your comment about the request spec, I ended up having to do something similar to test creating a promotion that creates a line item. You should be able to rip that off as a starting point.
Gives one failure:

```
  1) Failure:
test_invert_create_join_table(ActiveRecord::Migration::CommandRecorderTest) [/Users/steve/src/rails/activerecord/test/cases/migration/command_recorder_test.rb:79]:
--- expected
+++ actual
@@ -1 +1 @@
-[:drop_table, [:artists_musics]]
+[:drop_table, ["artists_musics"]]
```

Obviously, this test could be changed.
Don't convert to a symbol here, it just gets converted [back to a string](https://github.com/rails/rails/blob/master/activerecord/lib/active_record/attribute_methods/read.rb#L71).
Doesn't this belong to the other `WithBackendTest`?
More followup, if I remove this logic from the `rename_table` method, but keep the new test checking that the sequence matches the new name (test-driven!), I see that the sequence for the `octopi` table is still named `test_models_id_seq`.

```
  1) Failure:
test_rename_table_for_postgresql_should_also_rename_default_sequence(ActiveRecord::Migration::RenameTableTest) [test/cases/migration/rename_table_test.rb:79]:
Expected: "octopi_id_seq"
  Actual: "test_models_id_seq"
```

State left in the database:

```
» psql activerecord_unittest
psql (9.1.4)
Type "help" for help.

activerecord_unittest=# \d octopi 
                                   Table "public.octopi"
 Column |          Type          |                        Modifiers                         
--------+------------------------+----------------------------------------------------------
 id     | integer                | not null default nextval('test_models_id_seq'::regclass)
 url    | character varying(255) | 
Indexes:
    "test_models_pkey" PRIMARY KEY, btree (id)

activerecord_unittest=# 
```
This code is different from `if size =~ %r{^\d+x\d+$}`
This is going to be propagated to all other tests when running the entire AR suite.
Hehe yeah ok, lets improve it so :)
`ActiveRecord::Base` is still the official way to create Active Record models. I prefer to use it in our tests.
Agree with @jeremy, we should keep config.action_dispatch.rack_cache to allow customisation of Rack Cache setup
Are you suggesting that it was broken in 3.2.0->3.2.6 also then?  My test passes in those,  the previous would have broken them.  

Logically,  what would be the reasoning that update_attribute should fail here?  ```0``` could be a perfectly valid ```zine_id```.  If  not then it is the user's responsibility to put a validation on it.
You can enclose these in a {{{code block}}} to get a monospaced font.
mats[0] AFAIK
I've got code working that does the check_func and replaces that macro.    I do not know how to replace the other macros and keep the benefit of storage variables and error-returning from within the other macros without major code re-design.   Eventually, this C-code might be replaced with Cython anyway, and I'd rather not spend more time on it.
`<ocd>` For future reference, please pad `==` with spaces (i.e. `tax_category == self`) just so it's a little "cleaner". `</ocd>`

I'll fix it when I pull this in.
This test will not fail in any way that I can foresee. I will fix it.
you sure you want to allow the client to pass in a bad/missing queue file? subtle bug?
i have no idea. used code from mpl. better safe than sorry?
@smichr,
I did not get this step. What is the significance of count_ops() here and when will it be nonzero?
> let's not let people get sloppy about this...it fails in master if you try to iterate over non-sets:
You are right. For some reason I was sure that set.union permits
lists. It actually permits them only as non-first arguments.

```
>>> set.union(set([1,2]), [3])
set([1, 2, 3])

>>> set.union([1,2], [3])
TypeError: descriptor 'union' requires a 'set' object but received a 'list'
```
Note that this doctest failure is the resyult of experiment.
Is it possible (mathematically, algorithmically) for ` _split_gcd` to split the source list into the two parts without dependability  of orders in source list?
Have it the seance for the algorithms that uses it, that they depends of the order of Add.args?
As I understand it is related with "sqrt_deenest".
What about

```

>>> binomial_expand((a*b*c)**3)
a**3*b**3*c**3
>>> binomial_expand((a*b*c)**.3)
(a + c)**0.3 + 0.3*b*(a + c)**(-0.7) - 0.105*b**2*(a + c)**(-1.7) + 0.0595*b**3*(a + c)**(-2.7) - 0.0401625*b**4*(a + c)**(-3.7) + 0.02972025*b**5*(a + c)**(-4.7) + O(b**6)
```
On 10.04.2012 11:20, Sergiu Ivanov wrote:
>> +>>>  from sympy.polys.distributedmodules import sdm_deg
>> +>>>  sdm_deg([((1, 2, 3), 1), ((10, 0, 1), 1), ((2, 3, 4), 4)])
>> +    7
>> +    """
>> +    return max(sdm_monomial_deg(M[0]) for M in f)
>> +
>> +
>> +# Conversion
>> +
>> +def sdm_from_vector(vec, O, K, **opts):
>> +    """
>> +    Create an sdm from an iterable of expressions.
>> +
>> +    Coefficients are created in the ground field ``K``, and terms are ordered
>> +    according to monomial order ``O``. Named arguments are passed on to the
>> +    polys conversion code and can be used to specifiy for example generators.
>
> specifiy ->  specify
>

Done.

> Sorry for my sluggishness :-( I do plan to finish reading in one or two days.
>

Sure, no hurry.
Removed `:math:`

About the usage of Polys, I tried, but

1. I tried to use IndexedBase object as abstract generators to poly (in spite of the mentioned modules intersection `exp`), but IndexedBase is non-commutative.

2. I have added the special tests, which I want to be passed. In particular with the implementation light abstract commutative indexed object (`IndexedLight`).

```python
>>> A = IndexedLight('A')

>>> A[1]
A[1]

>>> Poly(A[1], A)
>>> Poly(A[1], A, domain='ZZ[A[1]]')

>>> Poly(A[1], A) * A[1]
>>> Poly(A[1]**2, A, domain='ZZ[A[1]]')
```

it works as desired!
So I think that it is possible to include IndexedLight in the core somewhere, so that there is no conflict between the modules as sympy.tensor

3. I have tried to implement it but the bell functions failures.

It turns that the problems with commutativity:

```python
>>> Poly(A[1], A) * A[1]
>>> Poly(A[1]**2, A, domain='ZZ[A[1]]')


>>> A[1]*Poly(A[1], A)
>>> A[1]*Poly(A[1], A, domain='ZZ[A[1]]')
```

That is

```python
>>> Poly(x, x) * x    # works
Poly(x**2, x, domain='ZZ')

>>> x*Poly(x, x)   # doesn't works
x*Poly(x, x, domain='ZZ')
```

Even when I reverse `symbols[m]*self._bell()` to `self._bell()*symbols[m]` it doesn't works, I guess that Mul sorts arguments by hashes.

4. Additionally I have added the tests with tuples instead of variables. This is use-case to avoid substitution, when the bells polynomial is used for calculation of the value of them themselves, in sequences compositions (infinity), and in this case the calculations running directly. When Poly will be used, then it is impossible to use numbers as generators. But now I guess, that it may be to worth to use substitution still.

As conclusion, due to the (3) I can't implement Poly now.
Only [one line](https://github.com/sympy/sympy/pull/1181/files#L0R388) can be uncommented for it now.
I agree with you. I will implement prism class which will have better functionality and complexity.
```
seq[current_pos: current_pos + 1] = [c*a for a in s.args]
```
> so ist es möglich, entweder einen einzelnen oder ein gesamtes Array zu nutzen

da fehlt ein wort :) "einen einzelnen ...?"
the opening bracket should be on the next line
There's a bug in pivotal for it: https://www.pivotaltracker.com/projects/488193#!/stories/35616639
Is it common for config files to set a value to `nil` instead of deleting or commenting the unused key?
Doesn't need to be initialized.
Move this to the stylesheet.
And you could to the same here, just call $('input[type=password]').keyup(checkPassword);

Seems like the last 2 else's are the same and could be combined, like above.
this makes my head hurt, and while I accept that it may work (though I'm not totally convinced) it needs more explanation of what's going on here.
insert the documents as documents?
When you connect to a MongoDB database server, the MongoDB driver creates a

the driver part is really important.
Well, the code following this makes a new doc & rev in case of a redirect on the old locale/slug. So, this cache delete should get called also for the old locale/slug when that redirect doc & rev pair are saved. Haven't tested that, but I think that's how it should end up working
To solve the update problem (i.e., changing child slugs when the parent slug changes), you just not save the parent slug in here, but only the child slug. That means you'll need to change the view though where you map URLs to pages in the DB.
Please try to stay around 80 chars max per line
expression, not statement
should be `dirname(__FILE__)`
Form preparation is not optional. If we don't return a form at this point it won't display any data you might want to process in process_form(). That's also why it returns false here.
Should be $browser here, not $user->browser ?
E bine. Poate ar trebui denumita decrement_spell?
You're right about it should be a constant. However, I'm more comfortable doing the comparison using the `Gem::Version` comparator rather than just string comparison.
The formatting style we keep is documented here:

  http://twitter.github.com/effectivescala/#Formatting

(should be consistent with the surrounding code)
Not sure "mixed" is a type :). An example, or expanded description here would be nice.
Is this a debug line? This deep in the library nothing should be printing.
// For better MVC and use of Sencha, 
  // 1) we shouldn't be fiddling with HTML at all
  // 2) you should update the text field for the relevant component, and Sencha automatically handles updating the view
  // something like...
  componentNameForBmiText.text = updatedText;
We can extend our REST call to give the route of administration in initial GET call & store it for better and faster interaction.
again, this should be private, since you already have the ```setId()``` method. Don't want two public interfaces with duplicate behavior.
Agreed. Some/all of the 'no_unit' coords plus a number of the others look more like attributes.
Actually, no it's not simple to do what you're doing, and the fact that you messed it up proves that. Your update broke the gradient with respect to the cost, broke the system for handling disconnected input errors, and the system for detecting undefined gradient errors, not to mention that it made the behavior of grad method inconsistent with the documentation in the Op contract. Using your system would also make it really complicated for the known_grads (replacement for grad_sources_inputs) to work correctly for integer variables.

Ops need to understand what they are doing. The logic in Op.grad can do several very subtle things that are impossible to verify from the level of gradient.grad. I didn't create that situation; the types were already determined inside Op.grad before my pull requests this summer.
Is this robust? Can we count on it always being a string and not e.g. a NumPy dtype object?
EXPORT was not in this ticket. I'll remove it from the comment. Thanks.
Comment added just above this line to explain what it is in ae6ffd2e7ef69180560eaeae9a00f76cd3721691.
need to rebase to pickup the testing factor
I guess this change has been done so the full path is printed? This can be achieved and retain the link by just removing the `~` I believe. Same thing should apply to the below middleware class, having the links in is useful.
@jspahrsummers Specs are owned by its test case, so it needs to be weak. But lololdsdk.
What should be happening is that it should be ignoring the cue in cases where it is malformed, but some kind of error code should be emitted that you can check for. Since it's seg faulting it's unlikely that we are handling this correctly now so there will probably be no error code.

I don't know what the best solution for this would be right now if you want to get it landed. Maybe put in a comment that "No error code currently for this case", or alternatively fix the parser to spit out an error and not seg fault.

We should get some kind of consensus on what you should do for now because I do think a lot of these tests are good. The mere fact that the parser is seg faulting on them tells us that this is something we should be testing for. They've already done their job.
Not according to the Python logging docs. Print for building UI.

Sent from my iPhone

On 29 Jul 2012, at 17:00, Doug Winter<reply@reply.github.com> wrote:

>> +        """
>> +        host = args[0]
>> +        username = "ubuntu"
>> +        port = 22
>> +
>> +        if "@" in host:
>> +            username, host = host.split("@", 1)
>> +
>> +        if ":" in host:
>> +            host, port = host.rsplit(":", 1)
>> +
>> +        r = remote.RemoteRunner(host, username=username, port=port)
>> +        try:
>> +            r.install_yaybu()
>> +        except error.Error as e:
>> +            print str(e)
> 
> should use logger errors
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/isotoma/yaybu/pull/81/files#r1259534
OK. I will check it.
Good to add to the comments, would also be good to note that further improvement would be desirable, e.g. Ogita et al.
SGTM :-)
as it turns out dont_filter is only used in 2 places: the scraper's duplicates filter and here in the offsite middleware (which is the only place we can override) -- considering that the only way to stop the duplicates filter from filtering our requests is to set dont_filter=true, maybe the modification we need now is to just to remove the check for dont_filter in this middleware?
A further wrinkle is that it appears GEGL ships an OpenCL 1.1 implementation in `gegl/opencl`.
So this is ensuring that basically everything keg-only only leaks in if it's a dependency (even if e.g. linked)?
Thanks, @hecticjeff !
So actually it's partially a PRAW issue and partially a reddit issue. Not having the failure would have saved me a ton of time ;) The other two I've seen intermittent problems with and have not spent the time to investigate. Aside from the `local only exceptions` any failure or error is something that we need to be concerned with. I'll make a better effort to file issues for items that I'm looking into so you needn't duplicate the effort.
That's what I didn't get. It doesn't remove the method from the cache. It removes data from the cache.
I'd recomand binding the package to at least the current major version (major.x.x) since updates to that might bring in breaking changes, and those need to be controlled.
Could be a one-liner ;)
Jesse's OCD rule: `{@code }` and `{@link }` always go with `CamelCaseNames` like `Linker`. For lower case names like `linker` you don't need either.
Dan Imal is going to complain that this is missing a space.
should we return an empty string here, or `false` or `null` or similar?
If there's text on line other than prefix/suffx then I agree that the newlines should not be removed. If prefix/suffix is the only text on the line then it might be a fair assumption that user wants it removed. I could go either way.
Actually, "being careful not to overlap between the 2" is **not** the reason of the complications that you're referring to. The code that handles the context of the current cursor position has to take care of two different cases. 
 * add  a new empty string to the values array to indicate that we're in a location for inserting a new property value.
 * get the correct offset of an existing property value for the current cursor position so that the caller can get the correct filter string for code hints.

If we're going to make a big change as in your suggestion, I don't think this complicated part will get simpler than the current implementation.
You meant *up* in "scope", but *down* in the code, right? Done. Good catch!
This one needs escaping too
Since the UI won't update anymore when JSLintUtils fires "enabledChanged," and no one else is using that event anyway, can you just remove the remnants of it from JSLintUtils?
I'll file a separate bug to track individual document changes instead of assuming all open dirty documents have changed.
not sure I like the simplification of this paragraph, were you making these changes here because some are repeated below?
On older versions of Python. But I'm saying i someone sets log_level in their config file to logging.ERROR this patch would prevent that from taking effect because it's not in the dict you use to lookup. IOW, I'm pretty sure this would *force* the use of strings that are capitalized correctly instead of just trying to correct it for Python versions that don't support using strings.
Future proofing :)
If you want to try
https://github.com/caolan/async#somearr-iterator-callback
Similarly for wise here.
whew this line is rough. It already was before, but damn.
Maybe *Facebook prijava*?
No, I forgot to delete it.
Define our own field type which is limited by [choices](http://mongoengine.org/docs/v0.4/guide/defining-documents.html#field-arguments).
Should be 560x358 right?
might as well fix spacing
Does this only work with divs now? Or can things be <video> or <audio> still?
probably could do something like that.
Hmm. I think we should call this ``pricePoint`` in the public nav.mozPay() API because that's more in line with mobile speak.
Could you move log4j-loader to extras to make it clearer it's not standard sakai.
那个是我测试时下载的文件，没有删除就提交了。太大意了再重新弄下。
2012/8/15 xushiwei <notifications@github.com>

> In rs_demo.php:
>
> > +
> > +list($result, $code, $error) = $rs->PutFile2($key, '', __FILE__);
> > +echo time() . " ===> PutFile result:\n";
> > +if ($code == 200) {
> > +	var_dump($result);
> > +} else {
> > +	$msg = QBox\ErrorMessage($code, $error);
> > +	echo "PutFile failed: $code - $msg\n";
> > +	exit(-1);
> > +}
> > +
> > +$upToken = QBox\MakeAuthToken(array('expiresIn' => 3600));
> > +echo time() . " ===> Uptoken: $upToken\n";
> > +
> > +list($result, $code, $error) = QBox\RS\UploadFile($upToken, $bucket, $key, '', __FILE__, 'CustomData', array('key' => $key));
> > +echo time() . " ===> PutFile $key result:\n";
>
> 测试过了么？为什么 demo 还是 php5.3+ 风格的呢？
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/qiniu/php5-sdk/pull/6/files#r1380740>.
>
>
I would say it's safe to say that the system has at least *one* cpu if it's executing this code? Returning 0 is an ambiguous way to return an error if that is the intended use.  Someone may wish to loop ``` for x in range(0, grains['num_cpus'])``` and zero would make no loops.

Maybe I'm just exercising a lot of pedantry for a tiny corner case?
The precedence order of "||" is lower than "==", so the parentheses are not necessary.
you should enable the dso module, if you want to use a dynamic loaded module.
It does ook weird though. But it's very likely some implicit magic going on, which reduces somehow the need to create a new class.
Aligner le =.
Again, JavaDocs need fixing.
Are you sure this is working ? . Check the spelling of environments
This icon appears to be poorly aligned vertically.  Can you fix it? (hint: float right)
Yes. I think, nobody would keep them in sync right now. But validations can be still added if needed for UI. It's stil undecided if we do validations only in foreman or add them to the Foreman::models or add the same param validation to foreman_bindings as Apipie provides in foreman.
Looks like it was a victim of copy/paste/ide behavior.  Will update.
@iNecas, good catch.  The new lines were a copy/paste from the existing 'product' option.  Found a few similar occurrences and fixed with commit 8900f0baa8a352bd253f107b2681df1138cbf0f6
Also, the installer was having issues with foreman, and since the packages weren't in the repo, i disabled it so that people could install katello from our branch with puplv2 if needed.   This will be re-enabled before merging to master.
Very good catch! This could lead to nasty bugs. Introducing new function execute_command that will respect the setting (verbose, quite mode, log file) and making all "system()" statements to use it.
Good point. I think dd169e33e6cabeafc700daa54579e8235164c799 addresses this concern.
`var Datepicker = function datepicker(href, hrefCal) {`

add name to the function (for debugging purpose)
Really! cool!