labofsahil/github-event-dataset-2012 huggingface dataset:
Made some minor changes to FOXML.inc

Foxml.inc assumed that you always passed in a transform and a document,
in some cases I wanted to be able to use FOXML.inc without a
document or tranform, so I added checks, now if they are NULL
they will not be included.
initial Ingest work.

Ready for review
Readme fix

Ditto.
Use NUPACK3.0 rna1999 material.

I made a couple of quick patches to use -material rna1999 rather than -material rna37 since the parsing doesn't handle the deprecation statement for the old material syntax. It would be good to make this available for people who want to use this code with the latest version of NUPACK.
Starling上で表示するProgressBarをDisplayList上で表示するものに置換え

これにより、ProgressBar画面の背景やバーのイメージをGPUにアップロードする必要がなくなり
パフォーマンスの改善が期待される。

また、Rootの役割がアプリの初期化と画面遷移だけになり、スッキリする。
Haskell support

Just added support for searching Haskell documentation using Hayoo! (or Hoogle, if one prefers)
Reset TemplateProcessor

New BundleStores were being registered to the SearchPath, but the RepositoryTemplateProcessor only looks at the storelist once, so they were not getting picked up.

A reset fixes this, but requires ugly ContextClassLoader reset.

Also included small change to dashboard js to allow remote access. (perhaps this was intentional ?)

PS love your work, makes life much easier
[Lighthouse-62] Displaying a tag list and archive views. 

Have had to repatch the pre_get_posts filter so we need to be very careful this hasn't broken sessions on staging before I push it live
Typo fix

Just a couple of fixes for typos in the POD.
NewRelic initialization fix for non-production
LPS-25595
Ported MMS fix from CM7. Trimmed MMS fix code.

I tested this commit and verified that it fixes MMS.
Update README.md

added closing paren
Update 2.2.md

修正了声明含有元素数组的bug
第二课，我修完的版本。
Temporary solution for problems with dual View call for ProfilingDebugPanel

Related issues and requests:
https://github.com/django-debug-toolbar/django-debug-toolbar/pull/204
https://github.com/django-debug-toolbar/django-debug-toolbar/issues/267
https://github.com/django-debug-toolbar/django-debug-toolbar/issues/209

Soon I will finish the panel that will gather all the statistics on calls, not only for the View.
It will start in process_request and will stop in process_response, as well as will drawing a graph and will provide a file "*.profile" to download.
Added callback support to .fn.twitter

Wanted to use build in markup generation, but needed a callback. Thought this could be useful if anyone wanted to fade in content, start a slideshow, or call relayout on isotope or similar tools after markup insertion.
Change to set charset to avoid mojibake

When Jenkins works on i18n environment, HipChatNotifier causes mojibake (http://en.wikipedia.org/wiki/Mojibake). This is the patch to fix the issue.
Fixes small issue with the monkey patching of the insert function
Update README.md

I tried the original html clone-request code for windows and it didn't work because of unescaped spaces.
This fix should work.
Removed an encoding warning on ruby 1.9

...ng option is ignored' warning on ruby 1.9 and above
added payment id to return url

hi,
I could not find a way to set the payment id to the return url.
I have added the paymentId to the return url in the express checkout plugin, this is for the issue #22.

thanks
parser update; writer update will be provided as a separate block

- generation: missing details are added as attributes (core components are changed)
- parsing: added Polygon header, expanded Pin struct, minor changes (like fixed templates)
Name tag is back in pom.xml
use active page width instead of document

Allows use of simple dialog when page width is not the same as document width
in this case the dialog does not correctly appear centred in the page.
vibrantmtd: Fixes SEND_END headset buttons
Multiplication and power

I added type families for multiplication and power.
Unfortunately, I was unable to find definitions that do not require UndecidableInstances. (In practice, no undecidability actually occurs.)
rowend/13-admin-settings

complete admin setting folder translation
step definition modification

I had to fill in the autocomplete text field in order for the step definition to work properly within selenium and webkit mode.

thx!
The callback parameter can now be a route parameter.

Hey guys,

I think this is more flexible. I can now put the callback parameter inside a route.

Thanks
add archive for chronological pages

archive is a dict where keys are years (where posts have been made)
and values are lists of months (where posts have been made) in the key year
my little description

In app/config.yml you can configure date and number formats

admingenerator_generator:
    twig:
        date_format: 'Y-M-d'
        number: 2
        decimal_point: ','
        thousands_separator: ' '
First pull request

Kyle,

I added files to config/import, lib, suite, and tests for the kfs branch just to get stuff going. I have a suite called "login" to be consistent with ks. It's pretty stripped down. I expect it to not successfully login. At this point, I want to checkin get the driver working. 

I wonder if you would have a look at my code in the pull request because when I run the driver command:
/Users/r351574nc3/projects/git/Open-Performance-Automation-Framework/lib/../lib/tsung-api.rb:254:in `[]': Symbol as array index (TypeError)
	from /Users/r351574nc3/projects/git/Open-Performance-Automation-Framework/lib/../lib/tsung-api.rb:254:in `add'
	from /Users/r351574nc3/projects/git/Open-Performance-Automation-Framework/lib/kfs/common/authentication.rb:32:in `login'
	from /Users/r351574nc3/projects/git/Open-Performance-Automation-Framework/tests/kfs/login.rb:31


I haven't looked to deeply into it, but is the tsung-api.rb proprietary to oae at all? I just wonder if I should be writing my own tooling for this or using what is there.
Library Additions 

I have added a couple variables and functions to assist in initializing on a separate database (besides $active_group default), generate DISTINCT and GROUP_BY options.
implemented the email class
Update es.lproj/ST2Preferences.strings
Saves answers submitted and displays saved answers

This branch adds a skeleton of the portal's Offerings controller and Saveables models to the dummy application. This allows the Lightweight Pages to carry forms and submit their Embeddables, which generate Saveables. The forms also look for Saveables matching their Embeddables, and if there are existing answers for the Embeddables, they are displayed.

N.B. this roundtrip is not Learner-aware: if there are concurrent sessions (more than one person using the dummy app at once) they both see the most-recent saved answers. This should go away when the engine is mounted in the main portal, however. (Checking that next.)
Issue 4779 - Add Callback example in interface to C docs

http://d.puremagic.com/issues/show_bug.cgi?id=4779
Update init.php

Updated 'show-on' 'page-template' search function to check for an existing theme template if '_wp_page_template' is not defined.
Allow developer options as a settings shortcut
Update SASS.plist

More predefined names.
I wanted to find a way to commit changes without rebooting.
So my first try was adding a function (called 'ptable_re_init') in order to "refresh" ptable without rebooting.
I'm still testing it, but it seems that it is working. Should I look for anything else in the code too for achieving it?
If $params['class'] is not yet set, this will generate a Notice.
In what case?  It shows just fine here.
This should be removed as Rockbiter fix seems to not use correct values.
actually, there should be a "use Traversable,…" at the top.
Snyggt jobb med en glowad krulle på slut-syntaxen! H4K20R FFS 1337
I didn't notice any problems when running the tests. Is there something else we should verify before submitting patches to rails?
well done! welcome and thank you for always improving rails and code habits!
agreed. pull req pls?
this line appears too long and should be followed by a space. Please check all instances of Example which should be Examples.
thanks! that was quick. no need for pull req i'll rebase and merge now.
Looks good to me. Although, being a contrarian the #ifdef Q_OS_MAC might need a comment to explain that on Macs the initial size of the dialog is too small causing display issues, whilst on Linux/Win it sizes fine, hence the #ifdef.

Also, we recently added an OperatingSystem global variable in Settings.h so you could use that, but it is moot, either way is fine.

I told you I was a contrarian.
i don't get it, why do you override DateTime#to_time?

     DateTime.now => #<DateTime: 2012-01-25T11:33:21+01:00 ((2455952j,38001s,996469000n),+3600s,2299161j)>  
     DateTime.now.to_time => 2012-01-25 11:33:25 +0100 

looks like offsets are allright? 

ruby 1.9.3p0
Technique for getting IP address found on Stack Overflow:
http://stackoverflow.com/questions/6458083/socket-io-get-clients-ip-address
Well I think they adjusted they API twice.

1) They introduced the new Listener interface to replace the Block/Player/Server/etc base classes (you don't method override anymore). Added the EventHandler annotation where you had to pass event = Event.class to the annotation.

2) Removed the event = Event.class annotation param because it could be figured out by reflection.

These are great changes, but the problem is they should have kept the old API working while introducing this new way of doing things.

For me (not for everyone, mind you), changes 1) broke my plugin. So I coded against the new but unfinished API. Then with the RB release they did 2) which broke my plugin again.

I guess it's the poison pill you have to swallow when answering the users' yells for regular dev builds.
I think string + undefined = undefined. Maybe remote.pathname + (remote.search || '')
I fixed SpoutPlugin yesterday @TomyLobo
idiot, I forgot to change the title of commit, but there it is. we just need to solve the environment issue... =/
Well if you wudn't touch it, I doubt anyone would have looked into that for long time. Nobody is dare to touch Unit.cpp because it's really weird place, just like dark side of youtube where you can watch some mexicans dancing and singing for 10 hours...
did i tell you, i use letters u,v,w,... for nodes? :)
'shoujld '? :p
Are you just testing?
Jeff, what are you talking about? You think that the ssh state should manage a user's home directory?
I'm not going to include this in the commit (I'll increment the $v_database before commit to sourceforge). In the future, recommend doing development work from the 'master' branch (see the version.php file of that branch) to ensure no issues when commit to sourceforge development branch(the master branch is where development is done and the version.php file there gives version 4.1.1-dev). In this case I know there are not issues, because the code rebases and testing I have been doing are on the 'master' branch.
You can hack up a SD DETECT signal easily enough.  On my sd card reader  (http://dl.dropbox.com/u/41900/Photo%2026-2-12%202%2019%2051%20PM.jpg), the 2nd pin form the top of the sd socket is SD DETECT, it just isn't routed anywhere, so i just solder a wire to it and use it as SDCARDDETECT.  On sdramps, a similar hack can be done.  i don't know the pin out of the socket used in sdramps, but "os1r1s" from #reprap has done it.  You can ask him on irc for the pin out.
I think we can do that - I don't think we currently care about the order in which controllers are loaded. Can I ask what OS you're using? That test doesn't fail for me (WinXP) or on the build server (Debian).
Tested, worked fine.
DELETE FROM `spell_linked_spell` WHERE `spell_trigger` = 23881;
INSERT INTO `spell_linked_spell` VALUES (23881, 23885, 0, 'Bloodthirst');

DELETE FROM `spell_bonus_data` WHERE `entry` = 23881;
INSERT INTO `spell_bonus_data` VALUES (23881, 0, -1, -1, -1, 'Bloodthirst');
wow, absolutely positive. I'll import ASAP
Unroll this and use & and | instead of shift.
Cool.  I have been working on some integration tests (really!) that
use vagrant and actually run the cap tasks.  If you're planning to do
some recap stuff I'll try and get them into working order and push
them.  Maybe to a dedicated branch.

On 13 February 2012 12:22, James Adam
<reply@reply.github.com>
wrote:
> Yup. I had forgotten too!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/freerange/recap/commit/60034c43036e105927f3ea85588998bca014a5a0#commitcomment-959965
also, could you please add a comment explaining why this specific number of iterations was chosen?  do we really need 6M iterations for Re = 5k?
thanks @raynar, you mean 1660, isn't it? Thank you
скорее всего достаточно было убрать && m_caster->IsFriendlyTo(target)
Ahh yes installer needs em. I often forget, but this was a quick fix for the pyrocms.com website.
I don't really know what you mean. There is no HMVC integration. You load them from module controllers the same as you load them from anywhere else.

A spark is like a library but has more bits. Lets not over think matters.

Emailing from my iPhone 4S like a BOSS!

On 29 Jan 2012, at 13:09, Pawel Martuszewski<reply@reply.github.com> wrote:

> Hi Phil,
> 
> Maybe it would be worth writing a quick post on pyrocms.com explaining benefits and usage of sparks in pyro hmvc (modular) structure?
> 
> So to show how sparks can be module depended or global etc. ? Hmm or maybe an interface for it too in the back-end?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/pyrocms/pyrocms/commit/f157c77ac042c00e842b2b853e5c42b6a7928f24#commitcomment-912487
Yep, that will do it, although the use cases surrounding AbstractTableGateway are currently fuzzy to me.  I'll merge, but this class might change a bit, or go away.
This make that some plugins won't work anymore (I mean the commands) :P
Like MCDocs or xLevel don't work.
Does this have to live in `lib`? It's not a library; `etc` or `share` would be more appropriate.
I think both are supposed to work if r is a string. No?
Keep in mind that you can use koji client inside any client side programs
I don't really understand why you need to do this.

    ocp-get upload NAME

should:

* look for NAME.opam
* look for NAME-VERSION.tar.gz

so what is the semantics of `ocp-get updoad DIR/` ?
Dave-

Thanks for the patch.

At this point the Configrity changes have made it to master and Configgy is now deprecated. 

I will check older branches and apply this patch if required.

Cheers,

Nick
Classic TShirt material..... :-)
Le fichier htdocs/admin/customfields.php doit etre renommé en htdocs/customfields/admin/customfields.php
Et dans le fichier descripteur du module tu mets
$this->config_page_url = array("customfields.php@customfields");
Ich habs geforked und div. Aenderungen eingebaut (z.B. Tschunk vom Camp 2011).

Als ich es auf dem Matomaten installiert habe,
hab ich micht halt gewunder :-)
Always use HTTPS URLs when referencing outside URLs, because otherwise there will be "mixed content" warnings if the site will run under HTTPS.
Squashed and pushed.
Failures:

      1) ActiveRecord behaviors with has_secure_password should be able to be authenticated
         Failure/Error: user = User.create!(:email => 'test@test.com', :password => 'foobar', :password_confirmation => 'foobar')
         ActiveRecord::StatementInvalid:
           NoMethodError: undefined method `error' for nil:NilClass: INSERT INTO "with_model_users_27596" ("email", "password_digest") VALUES (?, ?)
         # ./spec/active_record_behaviors_spec.rb:148:in `block (3 levels) in <top (required)>'

    Finished in 1.54 seconds
    55 examples, 1 failure
Reconfirmed for working for me.
Yeah, I was going to say, this seems a little silly when I can already do (Game instanceof Server) and be on my way.
need this the vmss ? for will work correctly?
hey shane, this is a lot of code.  Was it your intention that this goes into the train leaving for QA today?  like within two hours?  or would you rather this sit on dev for a week before going to QA?
On 28/02/2012 00:34, lukebyrne wrote:
> Hey Alex,
>
> The refactor looks good, I am guessing that the API stays the same. ie in order to set up a session token and make an APi call to markets it would be the same as in the README
>
> bf = Betfair::API.new(proxy, logging)
> session_token = bf.login('username', 'password', 82, 0, 0, nil)
> bf.get_all_markets(session_token, 1, [1,3], nil, nil, nil, nil)'

Yep, all that's unchanged.

I was thinking I might add some default arguments to #login so that if 
you're a personal user who just wants to get started, all you have to 
know is your username and password - it'll make noodling around in IRB 
that little bit nicer, too.

> Also can you fire me an email address so that I can send you some links to the BF Data. I also have a couple of scripts that I will flick you to read all the data into some db tables.

alex at blackkettle dot org will get to me.  Thanks!

-- 
Alex
For some reason tools don't take any damage when breaking blocks.
The configuration object used there is of type: org.infinispan.config.Configuration (deprecated). The reference should be changed to org.infinispan.configuration.cache.Configuration in order to work with not-deprecated config, but that's out of the scope of this fix.
In my oppinion, this should be the default behaviour. But users should still have the option to leave untracked files untouched. A possible use case: Imagine you want to test a new class together with the last commit, without losing the latest changes.
I think it's okay if it continues to run. Like with dashboard configs, there's going to be some expectation of delay of respecting changes to tasks on the fly.

The goal is to be running the tasks with smaller frequencies, e.g. tasks that run once a minute, before tasks that run hourly / daily etc. I think there's something in Scheduler#tasks_at_time about that.
isn't cpu limit min freq broken by removing this?
I'm curious to hear if there are tests of any browser implementations that use this.  In the [implementation](https://github.com/tschaub/openlayers/tree/utfgrid) I'm [testing](https://github.com/tschaub/openlayers/commit/0aa87052991f592f5ddf71f573f7a7559fc3efaa), things start failing at row 215, col 222.  This maps to 55296.  The range 55296 to 57343 is where some strict parsers fail to convert UTF-16 surrogate pairs to UTF-8.

Any hints on whether this may be a typo in my implementation, a limitation of my browser (same behavior in Chrome 17 and Firefox 10), or something that should be mentioned in the spec?  I'm serving demo.json (and my tests) locally.  Test page appears to be properly UTF-8.  Character encoding is not my strong suit.
> 6 months ago

Там этого текста уже нет давно. Но за исправление спасибо.
This was perfectly fine and intended. I werent thinking clearly
@Undergarun, Putricide is already doable on normal, I guess, I only reported problem with Abomination not able to attack Putricide. That's why I think it will not be able to cast slow on goos (adds). Also didn't check the power of Abomination (power from eating slimes). But even if these don't work the boss can be killed imho (if the rest of spells are working correctly - I didn't test with any raid).

About Frostmourne phase: I didn't come to this yet, but I read some posts that the teleport to the froustmourne area had some problems, that's all I know ;)
When I will come up on some problems then I will ask for help, thank You :)
ok, good.

about that crash, locking required in places where I want some auras to be removed?
docstring explaining what this is and why to use it
$config['log_threshold'] = 1;
that' s the point, I suppose that 
		if ( ! isset($this->_levels[$level]) OR ($this->_levels[$level] > $this->_threshold))
		{
			return FALSE;
		}
must skip loggin but it does not
Sorry Jeff, can you resubmit against current, not sure what is going on or if this is still valid
@pmjones I could be wrong, but that smells an awful lot like an [appeal to authority](http://en.wikipedia.org/wiki/Argument_from_authority). The fact that it is not _solely_ your personal preference does not in any way lessen the degree to which it _is_ a personal preference.
Беджик может потом пригодиться для количества заблокированного.
Это и другое - см. предыдущий коммит.
The names always been "SelfRestraint" the commit just updated the app window to display it correctly
Lgtm.
hi, cbuest

i fixed the classpath issue here: 6d9360b

BTW, i'm wondering why hard-coding "build" folder and put it to the classpath here?

actually you can easily get current user specified classpath via the super.getClasspath()
We already had the same problem in the GPX import (a dialog still shown, but the main thread doing an exit, causing problems), therefore I merged the suggested changes by campbeb.
I noticed this and actually just took the field entirely out in another commit earlier today.
No I'm just an idiot, comment's wrong, fixed
Cool! Bundler wasn't even around when I made this. Could you correct the gemspec and send me a pull request?
Si, si iba. Es para correr tus pruebas con los cambios que hice en el master...
I would use ```throw MakeStringException(INTERNAL_ERROR, "makeAbsolutePath: could not get absolute path of %s", relpath);``` in place of the EXCLOG/throwUnexpected()
ja mach ich mal sag mal gibt es einen weg blizzlike spawnpositions rauszufinden ohne zu sniffen?
Best repository ever?
It is not only the related posts... the setting enables and disables all Lucene
This looks unrelated?
Nice README update.
On Thu, Mar 8, 2012 at 1:12 PM, Rok Garbas
<reply@reply.github.com>
wrote:
> what about just passing "?diazo.off=1" in the url? it should be unthemed then

That only works in debug mode. That is, if I read the documentation correctly.
Those constructs seem to come up quite often with the embed loaders. I wonder if there is a better way to structure/represent them?
It would be tidier to just set ```$mtime = time();``` on line 69
or check ```if (isset($mtime))``` rather than testing on $file, in case the logic above changes
I have no idea what this does yet (I'm sure someone will either explain it to me or someone who does understand this will accept it), however, there seem to be a few minor style and general errors:

1. This line: var jakefiles = ["jakefile", /*"Jakefile",*/ "jakefile.js", /*"Jakefile.js",*/ "jakefile.j", /*"Jakefile.j"*/]; leaves an empty last element in the arrays (since you end the array with ","). This happens in at least 2 places.
2. Our style guidelines specify putting a space between = and having the opening curly brace on a separate line.
Je ne pense pas que les tangentes doivent être crées par défaut, ni même que la courbe d'interpolation ait besoin de garder une référence aux tangentes existantes.

En général, les objets respectent une certaine hiérarchie : la tangente a connaissance de ses arguments (la courbe notamment), mais la courbe n'a pas besoin d'avoir connaissance de la tangente.

Je suis d'avis d'incorporer l'objet Interpolation_polynomiale_par_morceau dès que les dérivées sont correctement gérées.

Les tangentes peuvent venir après.
OMG! I'll never believe it! Thank you, devs!
oh that's nice will test it... :)
Le sigh. How about you take a look at basically any outland/northrend zone folder in core? Most of them have it all hardcoded, and I mean because the OnGossipSelect hook is not called by any database table to my knowledge.
... there's a + 4 missing there (7 uint32 not 6) :P
Hmm I thought that if I'm scrapping web text then I should always return strings. I honestly don't know whether it's dumb or not. Should the code using the module handle that then and I simply return float?
I think it would make more sense to initialize direction as the positive normalized vector and set friction = self.friction * direction * -1.  This makes it more obvious that friction is opposite direction whereas before I was confused about why you negated direction.
Hi Derick! Please run the test without Matt's patch; I believe it will still work (and that Matt did something wrong in using sparse indexes). My own unit test did (https://github.com/hacklschorsch/mongo-php-driver/commit/054120a9edcd8a48a4a44856f51ef85cb9e5b576). Matt's code is broken, as he says himself BTW. Please see the mailing list eMail on that.

Florian
No No No, we need more boops!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
this should be an event not page view: let's call the even't "emailSpecified"
いあいあ！こっちのほうが直感的に分かりやすいっす！！ありがとござます！！

というか、あるぇー （・3・；
なんで僕こんな map して getOrElse とか無駄にイミフなことしてるんでしょうか？
これで、ほぼ標準コレクションAPI呼んでるだけなんでテストいらないじゃん状態＼(^o^)／
I think both works in the same way for java. Am I wrong?
I'd love to but the curl driver for request is all messed up and after spending days trying to hack around making it work I got bored and went with the "get it done in 5 minutes and do something that gets me paid" approach of file_get_contents().

It works most of the time for me. What issues are you getting?
Why won't you just use `:platform` option Bundler has?
Das geht kaputt - dieses Framework haben wir nicht im classpath!
Great! Certainly easier on the eyes! :)
It would be easier to read if this if-statement was in one php block.

    <?php
     if (!empty($_POST['number1']) && !empty($_POST['number2'])) {
      if ($_POST['function'] == 'multiply') {
        $total = (($_POST['number1']) * ($_POST['number2']));
      } elseif ($_POST['function'] == 'divide') {
      ...
Am I using the right service? ion-definitions/objects/cei/process_dispatcher_service.yml has the name as process_dispatcher_service.  When I changed it the generate_interfaces failed. 

This is what I was using to add the dep:
https://github.com/ooici/ion-definitions/blob/master/objects/services/cei/process_dispatcher_service.yml
oh yes! I cant wait for it... openengsb-root v33, openengsb-maven-plugin v2.4.0 and openengsb-framework v3.0.0 will completely change how we handle releases, notices, licenses, tests, ... (although the scripts will still remain for easy usage).
So here's the thing about party abbreviations: the reason I set them to 3 chars instead of one is that one isn't enough: see ftp://ftp.fec.gov/FEC/cn10.txt for a list of potential abbreviations used by the FEC, for example.
Nit: please remove this kind of TODO. If the default operation is fine, then it's done; otherwise, do the work!
see #837
This causes a compile error on droolsjbpm-integration:

[ERROR] /home/gdesmet/projects/jboss/droolsjbpm/droolsjbpm-integration/drools-container/drools-spring/src/test/java/org/drools/container/spring/MockAgendaEventListener.java:[21,7] org.drools.container.spring.MockAgendaEventListener is not abstract and does not override abstract method afterRuleFlowGroupDeactivated(org.drools.event.rule.RuleFlowGroupDeactivatedEvent) in org.drools.event.rule.AgendaEventListener

I 'll fix it
Dad would like to start baking on a per order basis not a per flavor basis hence the layout rearrangement. However during the transition period, both layouts are helpful.
Some notes:
* Use `https` for better security.
* The `json` extension will work just fine. But code will be clearer if you instead pass the header `Accept: application/json` and then you can omit the `.json`. This is also where we will do versioning in the future.
* The `login` call is used only once to get an `api_key` for a user/password. Store the `api_key` locally, and then you won't have to use the `login` endpoint anymore.
I didn't take them from nowhere. I added it myself ;-)
The class Renderer of the portlet has no methods to test.
The file in your repository is main.py, but this is pointing at main.app? Will that actually work?
Yes of course.

Whatever happened to `|blaha|` substitution in these blocks? The Akka version should be a replacement string.
How can fqn be null here?
ah, I thought it only supported `?`, nice!
no i cant think of any, will reorder
Looks good, but just overriding the things that are different from `.admonition` should work as well

```css
.warning {
  background-image: -khtml-gradient(linear, left top, left bottom, from(#ffffc4), to(#ffff00));
  background-image: -moz-linear-gradient(top, #ffffc4, #ffff00);
  background-image: -ms-linear-gradient(top, #ffffc4, #ffff00);
  background-image: -webkit-gradient(linear, left top, left bottom, color-stop(0%, #ffffc4), color-stop(100%, #ffff00));
  background-image: -webkit-linear-gradient(top, #ffffc4, #ffff00);
  background-image: -o-linear-gradient(top, #ffffc4, #ffff00);
  background-image: linear-gradient(top, #ffffc4, #ffff00);
  filter: progid:DXImageTransform.Microsoft.gradient(startColorstr='#ffffc4', endColorstr='#ffff00', GradientType=0);
}
```
This is one reason to name it Account, it corresponds to AccountController, and it pretty much does that.
I was not aware that you were waiting; didn't see the notification email on github. If you ever wait for me, best to email me, that way I'll be sure to see it.

Anyway, I'll write instructions.
### In order to comply the Kohana HMVC paradigm

```php
class Gleez_Upload extends Kohana_Upload {}
```
**instead**
```php
class Upload extends Kohana_Upload {}
```
Nitpick space issue
Worth storing here?  The MultiVoxel converter will do that for us.
I forgot that dictionaries implement the iterator protocol.... will change to what you suggest.
Wouldn't bother
%s would probably be easier than the str(..) also that will die on unicode where as %s seems to fair better
It is. I was going to move it to a method outside of the class but this seemed like a slightly more elegant solution.
This is where it's not clear, esp when coupled with Python's syntax. I read this and thought you were checking that status was not None.
It really is not as simple as this:

```python
class X(object):
    def a(self):
        print 1

def b(self):
    print 2

X.b = b
```

Instead, you have to do a thingy with the `types` module, which I will push momentarily.
Doesn't lack of a script path constitute a failure condition?
of course !
Makes sense. I'll make those changes & resubmit later today.
I think this will not fix the problem in general. This will only work if the graphUri is null (which is currently always the case, since the method is never called with a second param and it is a internal method). I think we can get rid of second param to this _execSparql method and replace the $graphUri with NULL. Would you mind to change this two lines? The I can merge it in directly from your fork. Thanks...!
align variable names in declarations, and put the * next to the variable name. for example:
```
  rtems_memory_management_alut    *alut_p;
  rtems_memory_management_entry   *current;
```
I was thinking more along the lines for things like in the librato library where they might want to calculate non-standard calculations in a single timer loop (similar to the counters loop above where its using the same key but different data sources. Though I can change it to timer_data easy enough.
This check can be moved into the `animations` block, so that the `redraw` invocations don't need to be duplicated.
I think I'd prefer "32-bit" and "64-bit" over "32 bit" and "64 bit" (note the hyphen).
This is an exceptionally crazy hack.  Why is it necessary?
Cela tato definice psana v pseudo jazyce configuratoru mi prijde dosti necitelna pro uzivatele. Nebylo by lepsi udelat normalni factory v plain php, aby to kazdy pochopil a mohl si ji pripadne upravit?
nitpick: `(PyArray_ISBYTESWAPPED(ob))`
Ah, I see. This should all be good, will take another quick look later
today and merge.


On 21 December 2012 13:18, Martin Filliau <notifications@github.com> wrote:

> In moxie/core/kv.py:
>
> > @@ -49,5 +49,18 @@ def set(self, key, value):
> >
> >      def setex(self, key, expiry, value):
> >          return self._backend.setex(key, expiry, value)
> > +
> > +    def healthcheck(self):
> > +        """Healthcheck query to the backend
> > +        """
> > +        # TODO this query (ping) is specific to Redis, it should be made generic at some points
> > +        try:
> > +            result = self._backend.ping()
> > +            if result:
> > +                return True, "OK"
> > +            else:
> > +                return False, "FAILED TO PING"
>
> You expect to receive "PONG" from Redis when you send the command "PING";
> if there was a connection problem (i.e. no response), this would throw an
> exception. "FAILED TO PING" means that the command didn't return True
> ("PONG"). But this is really to test the connection so we should never have
> this case.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ox-it/moxie/pull/9/files#r2488427>.
>
>



-- 
- Dave
Would this make more sense to be called ```--suppress_taxonomy_assignment```? Might be a bit clearer.
Same ... as earlier comment
Ha! It's just turning 11:00pm here. I take it you're back in Australia. :)
This should be `git checkout...`. I think you(me too) are used to our aliases.
With ```lstrip``` we don't need else part
Depends on your style :) Doesn't matter to me.
"PS: Is there a way in Scala to constrain annotations to a specific element like a class or a method like in Java?"

We should and don't yet enforce @target; it's a ticket somewhere.
Okay, best to document that, and pick parameter names that are consistent with the docs above for `collapseGuardedCases`.
IMHO this would be cleaner as two distinct cases  with two while loops rather than one while loop
The ``sides`` description in the docstring has a capital letter for ``Default``. Put a line ``sides = sides.lower()`` somewhere at the top?
I think it would be nice to have a short docstring explaining why this is not a particular case of `rm_id`.
I can make this change if you'd like. In general the public facing functions are promoted up to the rules module. This one isn't. Actually, I think this should move to some basic SymPy utility file, perhaps `sympy/utilties/iterables.py`?
They're generated.  If they're not correct then we have a serious problem :)
Building such objects could easily be a whole GSoC project, but also you could probably hack up a basic version that has no underlying knowledge but suits your needs.
Also in the other comment  above , I have addressed how the issue of a tuple can be dealt with.
remove null please. it's useless.
I cannot run this command now, I download your file and replace it.
First, to some extent it is a style issue, since I don't prefer `latex` and other markups contaminating my R code. Second, these are just defaults. So pulling them inside default files helps keep the code clean without cluttering them up with a lot of `if.. else` statements. For instance, if we implement `highlight.style` as an option, you can set it to `default` and the headers would automatically correspond to them. Just my two cents :-)
Application generator described above
Need issue `#` here.
Yes; they are in C99, but the VC++ 2012 compiler doesn't support them.

I don't know whether there is a better symbol to check.  
However, I also don't know whether `_alloca` will work on other compilers, which is why I did it this way.
This method is pretty big. Break out the invariant preservation into separate functions.
Loop 99 times or until the file is available and wait .1 s between each loop.
The NULL would get get propagated to the strlen call leading to a segfault. I figured it would be a good defensive change to protect against that type of crash, regardless of how the NULL/empty string got passed in.
````main_project```` implies that there are multiple projects, which isn't true for draft questions.  Rename to ````only_project````.
It's not just that one lazy worker, but all the other workers that the lazy one has infected with his laziness!
For 1.9, `convert` seems to return an object with the correct target encoding:

````
irb(main):058:0> i2
=> "El presidente del Congreso amonesta a Fabra, que pidi\xF3 excusas."
irb(main):059:0> i2.encoding
=> #<Encoding:ISO-8859-1>
irb(main):060:0> d = i2.detect_encoding
=> {:type=>:text, :encoding=>"ISO-8859-1", :confidence=>95, :language=>"es"}
irb(main):061:0> t = CharlockHolmes::Converter.convert( i2, d[:encoding], 'UTF-8' )
=> "El presidente del Congreso amonesta a Fabra, que pidió excusas."
irb(main):062:0> t.encoding
=> #<Encoding:UTF-8>
````

Is there a different encoding setting that we should be using in addition?  Or is that to cover the case where convert is unable to transcode to the target encoding?
(which is to say, don't document things that don't exist yet.)
/* ?
Why should it be off? It adds no benefits being off, adds some being on. It isn't the end of the world if its off. I believe we have the web.config file for IIS.

One of the advantages of front controllers is nicer urls. I don't see a point in having the default set to require app.php in the url.
You don't need intval here.
Shouldn't we always listen to the current promoted connection?
Verifica daca s-au intrunit conditiile de acordare a achievementului pe orice durata de 'days' zile(ma folosesc de ultimul seen pentru a nu verifica toate evenimentele de la inceputul jocului).
Maybe this is worth going to a separate helper?
`exec`
It seems that originally there was an [invalid date](https://github.com/weppos/whois/pull/169/files#L3L21). This case doesn't seem to exist anymore so I would simply change the code to

    time = $1.split(" ").last
    Time.parse(time)

without the begin/rescue statement.
I will like to preserve the output options through the cliOptions
Because this is what the ZF2 skeleton app tutorial told me
Thanks Rob.
Something I have not yet figured out is how to ADD new file/folder when they
don¹t exist (I can edit existing files).
I have the French translation for the Juser plugin but don¹t know how to put
them on GitHub.



De : Rob Clayburn <notifications@github.com>
Répondre à : Fabrik/fabrik
<reply+p-2008375-ace5d22bec485359dcb3a0559f46ef5cf3c0425c-1868381@reply.gith
ub.com>
Date : Thu, 01 Nov 2012 11:39:37 -0700
À : Fabrik/fabrik <fabrik@noreply.github.com>
Cc : Jean-François Questiaux <info@betterliving.be>
Objet : Re: [fabrik] Fix for the autologin function of the JUser plugin
(#507)

In plugins/fabrik_form/juser/juser.php:
> @@ -668,7 +683,7 @@ protected function setGroupIds($formModel, $me, $user)
>    $this->gidfield = $this->getFieldName($params, 'juser_field_usertype');
>    $defaultGroup = (int) $params->get('juser_field_default_group');
>    //$groupIds = (array) JArrayHelper::getValue($formModel->_formData,
$this->gidfield, $defaultGroup);
> -  $groupIds = (array) $this->getFieldValue($params, 'juser_field_usertype',
$formModel->_formData, $defaultGroup);
no problem! I added the pull request then reappled the (array) cast in the
following commit
 

 ‹
 Reply to this email directly or view it on GitHub
<https://github.com/Fabrik/fabrik/pull/507/files#r2008375> .
You shouldn't use hard-coded  OpenCV library jar name, instead please use something like:

    set(OPENCV_JAR_NAME opencv-${OPENCV_VERSION_MAJOR}.${OPENCV_VERSION_MINOR}.${OPENCV_VERSION_PATCH}.jar)

as it's already used in 'opencv/modules/java/CMakeLists.txt'
Agreed. The `break` is a hangover from the old in-line code.
you let this 'substring'-based id in, is it useful?
What thos last 3 print are mean to show?
ディレクトリ削除は、レガシーな方法だと再帰を使う方法もありますね。
https://gist.github.com/3128915
こんな感じで、削除できなければ false が返されます。
"letter, and" -> "a letter and"
data['component'] is being spliteed with | on line 172, so is it int actually?
Well spellId is being checked in this part:
```
		// Find spell of effect to be triggered
		spellId = GetSpellProto()->EffectTriggerSpell[mod->i];
		if(spellId == 0)
		{
			LOG_DEBUG("Warning! trigger spell is null for spell %u", GetSpellProto()->Id);
			return;
		}
```
and yeah, don't look at this as a complete patch, I just want to hear suggestions on what you think is better way to handle this, handling it in separate void that will be 98% copypaste of ProcTriggerSpell or handle it in ProcTriggerSpell with a check for EffectApplyAuraName as in gist diff I linked above.
`module.exports = {...}`

Здесь же один объект на весь модуль.
I'm not sure that I fully understand your point @rwcarlsen, but can offer that a key part of this whole pull request is to bring all of this kind of functionality to one place.  Agents are marked as "alive" but there is now a single way/place to do that.
See  https://github.com/django/django/pull/324/files  for updated version.
should be a comment not a docstring :-/
please don't use ``else`` when it is not needed (see the other platforms as I fixed their coding standards 2 weeks ago)
Hi Hitoshi,
     Thanks again for your effort to do code review for me. I think I already addressed all the issues you pointed out. 
If you have found more issues, let me know. Thanks.

All the best,
Kun


On Apr 5, 2012, at 9:43 PM, Hitoshi Harada wrote:

>> +        SET_VARSIZE(result, sizeof(int)*(doclen+2) + ARR_OVERHEAD_NONULLS(1));
>> +        result->ndim = 1;
>> +        result->dataoffset = 0;
>> +        result->elemtype = element_type;
>> +        ARR_DIMS(result)[0] = doclen+2; 
>> +        ARR_LBOUND(result)[0] = 1;
>> +        for(start_pos=0;start_pos<doclen;start_pos++){
>> +            memset(curr_top1_array, 0, nlabel*sizeof(int));
>> +            memset(curr_norm_array, 0, nlabel*sizeof(int));
>> +            if (start_pos == 0){// the first token in a sentence, the start feature to be fired.
>> +               for(int label=0; label<nlabel; label++){
>> +                  curr_norm_array[label] = rArray[start_pos*nlabel+label] + mArray[label]; 
>> +                  curr_top1_array[label] = rArray[start_pos*nlabel+label] + mArray[label]; 
>> +               }
>> +
>> +            } else if(start_pos == doclen-1){// the last token in a sentence, the end feature should be fired.
> 
> The code here and below looks so similar.  Avoid as many duplicate lines as possible.  The only difference looks like the addition of mArray, but I'm not sure.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/madlib/madlib/pull/103/files#r653665
>
This new function will exclude the discrete features used by ancestor.
For overall this code especially in this loop, it is more readable and good for performance to use int* variable rather than repeated ((int *) ARR_DATA_PTR(aray)).

<pre>
int *n_rArray = (int *) ARR_DATA_PTR(rArray);

n_curr_norm_array[label] = n_rArray[start_pos * nlabel  + label] + ...
</pre>
s/resued/reused/
Made this code run on Linux only.
Also, either this should be renamed `site_url_hash`, or the comment should clarify that that's what it is - otherwise you might think this was a hash of the site contents or something.
Should probably also use `std_cmake_args`
... we cannot trust xcode-select, so check additionally if xctools_fucked.
Shouldn't we propagate the errors here somehow instead of returning a probably odd sum result? Maybe setting `fireOnOneErrback=True` on the DeferredList is enough? I think even without that we'd get errors in the logs but I'd rather the caller gets to find out something went wrong too.
`setnx` already seems to exist in [txRedis](https://github.com/deldotdr/txRedis/blob/master/txredis/client.py#L111)?
Ahh. Sorry, meant fullhash.
This needs a comment as well
Can you explain what you mean by duplication?
I just followed the documentation,
http://doc.scrapy.org/en/0.16/topics/downloader-middleware.html#writing-your-own-downloader-middleware

My rationale was to implement a rfc2616-aware storage that based on the
request would return different values, which back at the middleware would
trigger certain behavior on either the scheduler or other middleware.

Originally I've added cache logic to both the middleware and the storage.
With my limited knowledge of Scrapy I don't see how we can have cache
policy totally decoupled from storage, since it will need to access
Response and Request headers.

Example, a 'no-store' directive should be handled at the middleware level
without storage ever knowing about it. But a 'Expires' header should be
handled at the storage level.

Perhaps we need two policies, one at the Middleware level and another at
the Storage level?
Yeah, I could go either way on that. As-is we call `newModule` in one place only which I like.
Yuck but it's temporary - I have a complete re-work of singleton wrapping planned.  And I checked, I believe this shouldn't have much performance hassle.
This type of ad hoc  escaping of XML entities is also error-prone. Your parser should provide an appropriate mechanism for doing this robustly.

Again, I expect that we will want to switch after the demo to a faster pull parser, perhaps written in C and bound to JavaScript. One of the Node expat wrappers may be the best option. From there, we can implement a reasonable JSON-XML binder on top (you'll notice this one is less than 200 lines and not particularly complex.)
this would be tracking the upload date instead I think, since you can't do much w/ the id for optimization, right?
Test should be named in the format: Method_State_ExpectedResult.

In this case a name in that format could be Add_ClassWithPropertyOfTypeThatHasDontSerializeAttribute_DoesNotSerializeThatType
Well, the `isHomeNode` check could be in `_setupAds`. As brittle, I mean if we then go implement DoubleClick or iAd or whatever other ad network, we have to reengineer the conditionals around AdMob.

I'm more concerned about it being hard to follow than it being brittle. I don't think this should hold up merging into develop.
I'm not sure how much that'd help (for me, at least): there are no closing tags and we only use two spaces, so I find it really hard to trace where blocks end. Haml would have the same problems.
Indentation is wrong on this line, and on lines 149-154 above.  Make sure you're using spaces everywhere (no tab chars).

In general, make sure your code passes JSLint (the indentation errors here would be flagged by JSLint).  To check this in Brackets just select View > Enable JSLint.
Yes, in the array case we're not using this parameter. Instead, we're reading the associated platform from each shortcut in the array. I can update the comments for keyCmds and platform to make this clear.
Maybe we should rely on ``brackets.config`` here to be more consistent with the other config options. Thoughts? Doing this dynamically works as well,  and may simplify the code base. I don't think we've spent enough time considering how different configuration options would be.
So this measurement sometimes includes disk I/O (to read file content or timestamp) along with parsing the text.  Disk I/O seems likely to be more noisy than other timing.  Would it be valuable to have another metric wrapping just ```_findAllFunctionsInText()``` so we could test parsing time in isolation?
In brackets I don't see the alignment issue though. It must be a tab before the equal sign.
That change doesn't work if I remove the fail() block code, but it does work if I leave the fail() block code in.

I'm not sure if I agree with #1451, so I'll add my comment there.
Actually, put `message_para` in coaster.views.
No, I think you got me wrong. What you need to do is:

```ruby
rb_yield(rugged_str_new2(branches.strings[i], rb_utf8_encoding()));
```
*Add user*? Better maybe *Create user*? *Register*?
Oh wait, Firefox supports standard transitionend. My bad!
I think it's better to use add/remove -- if the `mouseout` event ever doesn't get fired, you'll end up with the reverse functionality, right?
Don't make a variable for this here, just do it below in the caller:

```
utils = require( './lib/utils' )( { EMBED_HOSTNAME: EMBED_HOSTNAME, EMBED_SUFFIX: EMBED_SUFFIX }, stores );
```
Fix naming here too
Remove whitespace, please (spaces on this line, not the line itself).
Remove this line. Path.php isn't required (at least now).
From where `$newauctionprice` & `$oldauctionprice` are coming?
I don't see any reference creating of them.

Plus, it's a good idea to avoid creating loads of unnecessary variables
(when it's easy enough to work without creating/assigning them numerous times).

Consider this: Much smaller code & less chance for memory leaks.
```
if ($oldtotalbid <=200) {
    $newtoken = $oldtoken - 1;
    if ($oldprice >= 0.01) {
        $newprice = $oldprice - 0.01;
    }
    else if ($oldprice <= 0) {
        $newprice = $oldprice + 0.01;
    }
}
```