labofsahil/github-event-dataset-2012 huggingface dataset:
Feature/multisites

Fin des modifs demandées & fin des bugfixes, sur IP et sur Montreal (sur l'autre repository).

A savoir ; il reste un peu de JS sur la home mais pas beaucoup, en gros les 3 fonctions qui servent à charger en AJAX les projets dans le slider, à afficher le loading, et à initialiser le slider ; j'ai voulu les sortir de là pour les intégrer au slider.js, mais cela causait des bugs et le slider cessait de marcher, j'ai donc temporairement laissé ces 3 fonctions en attendant de trouver un moyen de les sortir proprement.

Une chose est à tester toutefois ; la traduction française, car je n'ai pu le faire moi même, n'ayant pas accès aux trads Transifex... Cela concerne le fameux problème des guillements sur les trads dans le JS ; j'ai donc remplacé les quotes js par des guillemet, et afin d'éviter tout problème j'ai ajouté l'option |escapejs qui permet de protéger les caractère sensibles d'une valeur afin déviter tout bug JS.
J'ai donc des lignes de la forme :    
 $("#quick_project input[name=title]").example("{% trans 'My project title'|escapejs %}");

A priori, selon la doc, cela devrait marcher, mais n'ayant pas ces fameuses trads, je n'ai pas pu le voir de visu.

Voila voila,
Pour le reste tout est corrigé / nettoyé, et testé sous IE8 Chrome et FF, pas de bugs à signaler.

Du coté dev, il reste donc :

Pour IP et Montréal :
-Coder les différents listings du menu déroulant "Projets"

Pour IP seul :
-Coder le système de news (le bloc central avec titre, blabla, tags et photo)
-Coder le système derrière le "derniers posts du blog" et "derniers twitt"
-Coder les vues pour le système d'inscription à la newsletter
-Corriger le bug de l'url "/None/" des icones pour les topics
Use new library (PyGithub) which uses Github API v3

Also, use or create github labels, including mapping "Type-Defect" -> "bug" and "Type-Enhancement" -> "enhancement".
Fixed the cache handling.

A different cache file must be used for each domain, otherwise the
data will be wrong when loading a second domain.

To avoid cluttering the root cache folder with many ``bazingaExposeTranslation.*`` files (2 files per domain per locale), I moved all cache files in a subfolder
Bug #2756. Fix the problem with the Null values coming from Oracle (a nu...

Fix the problem with the Null values coming from Oracle (a null value is set by Oracle for each empty string!)
Quelques petites modifs

Avec explication en live
Улучшенная предзагрузка изображений

Единственный минус - теперь не поддерживается распознавание раржипегов в лисах < 18, хотя это можно исправить. Впрочем, теперь любая фича, используящая воркеры не будет работать в Firefox'ах < 18.
Added QR Code support

This change generates a QR Code for the URL that is exported to pdf. As QR Codes normally only make sense on printed documents I decided to add it directly to the PDF. To get the QR Code in the PDF you can use the @QRCODE@ replacement pattern within the desired template.
Upgrade plugin to version 2.0.4 of Grails
Pull Req: blackberry.io.filetranfser

blackberry.io.filetransfer extension allows you to upload files to a server or download files from a server.

This pull request includes the implementation of upload and download function.


- [Issue](blackberry-webworks/BB10-Webworks-API#26)
- [CI packager build](http://ci0000003863287:8080/hudson/job/BB10-Webworks-Packager-next-filetransfer/)
- [Viewable docs](http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-filetransfer/view/blackberry.io.filetransfer.html)
- Docs pull request
Pull Req: Refactor all extensions so that the namespace is not hardcoded

* Refactor all extensions, each extension now has its own manifest.json file which indicates whether it is global (like blackberry.event), its own namespace, and dependencies
* Changed require.js so that client.js can require other client-side files (provided the required files don't have other dependencies)
* Get rid of blackberry in the `http://localhost:8472` URL
1. bluedroid lib back from cm7, no dbus errors in log, less bat drain sh...

...ould be

2. libmoto_ril.so back from cm7, no pain with GSM modem not turning off after switching gsm/umtc

Signed-off-by: Oleg Pronin <olegpronin777@gmail.com>
start plastometr2d
Bval-241 method validation

I am pushing this request in the name of Gunnar to be able to send feedbacks
Fix for session instance error.

The error received:

Catchable Fatal Error: Argument 2 passed to FOS\FacebookBundle\Facebook\FacebookSessionPersistence::__construct() must be an instance of Symfony\Component\HttpFoundation\Session, instance of Symfony\Component\HttpFoundation\Session\Session given, called in /var/www/RealUP/app/cache/dev/appDevDebugProjectContainer.php on line 914 and defined in /var/www/RealUP/vendor/bundles/FOS/FacebookBundle/Facebook/FacebookSessionPersistence.php line 24
Custom temp folder

With these changes the user can select a different temp folder to manage the gitolite repository. If this custom option is ignored, the old behaviour still works and the plugin uses Dir.tmpdir.
In my case this patch is mandatory because the /tmp folder in my server is mounted with the noexec flag and the plugin doesn't work (it causes a 50x error)
SQL injection tests.
Develop

Refactorings.
Nightingale 1.11

add ability to import various video formats into nightingale
Update/Remove/Authenticate
Fix blocks tests, add block timeout and cancel tests

Block tests were not actually running to completion (the success and fail blocks were never actually running during unit tests).  Added a wait loop to wait for all block tests to complete.  

Also added tests for block completion and timeout.
armstrong/master
added instructions for installing with pip
Add Apahce License
change process kill command to working
INTSAMPLES-53 Update README.md

For reference see: https://jira.springsource.org/browse/INTSAMPLES-53
korean translation 4.3b2

transparency part
Multiple scopes

Your library is nice, just added support for multiple scopes, as I needed it for a project
First translation is done. Maybe will check by me later.
Italian translation

Hi, I've made the Italian translation for this document.

Cheers
Giulia
Fixes the issue with 'getOutputOfSysCommand' is not defined
Fixed menu link and added i18n variable

Fixed: Link on github in DefaultMenuBuilder.
Added: Layout welcome variable in Ru i18n.
Ultrasonic Sensor testing
Added missing 'is' argument to 'default_filters' attribute 

Recent versions of Moose require a 'is' argument. The error message is this:

    Attribute (default_filters) of class Moose::Meta::Class::__ANON__::SERIAL::4
    has no associated methods (did you mean to provide an "is" argument?)
Upgrade to Rails 3.2.1

I found your project and thought it was a nice starting point for a potential project of mine.
Ancestry and NestedSortable do fit quite well together.
I brought it up to the latest rails and thought you may be interested.
Thanks!
Reestruturacao do codigo

- O codigo R foi movido para o arquivo oligocnvCode.R;

- O arquivo README contem uma breve descricao;

- Bug Fix: match()
We called this "getOccurrence" for the event exception case and I think it make sense to ask an event for an event occurrence regardless of the nature of this occurrence (exceptional or not, etc)
I agree! Baby-steps is where it's at. However, we should at least have some tests if we're going to land this in master otherwise we're begging for trouble further down the line if it evolves. And if it's just a goofy experiment prototype, we can always develop a brand spanking new app and install it with pip and append it to `INSTALLED_APPS` in settings/local.py
(__ or AppLocale)
total should support `Infinity` for calculated results that don't have a finite end point.
@bnoordhuis - Thanks! Merged in 2f759a7.
AFAIK, 3.3 does not _require_ 5.3, it only supports it. @zombor, can you confirm that?
IMO, it's much simpler and more explicit to instead use a check for the initial request, rather than a test for two different return types..

It seems really wrong to have files return either this:

~~~
array(
    '/tmp/filename1.ext',
    '/tmp/filename2.ext',
);
~~~

or this

~~~
array(
    'file1' => array(
        'name' => 'filename1.ext',
        'type' => 'image/jpeg',
        'size' => 12345,
        'tmp_name' => '/tmp/abcde',
        'error' => 0,
    ),
    'file2' => array(
        'name' => 'filename2.ext',
        'type' => 'image/jpeg',
        'size' => 54321,
        'tmp_name' => '/tmp/edcba',
        'error' => 0,
    ),
);
~~~ 

The formats and features are completely different, and pushing $_FILES into files() seems to give no advantages...
Thanks Jon! Fixed in @272016a5677e3bd1f5372980e64a5252a7ee31ec.
I propose we fix this with pull request #956.
ぎゃっぱぎゃっぱ line note
Now you can see my latest post in a very basic form
We should also update the other two views.
Maybe better use core.dom.Node.assertIsNode() and remove both, the notNull and the isTrue(nodeType) check
Maybe add some comments explaining why this is done so nobody will "optimize" this away in the future by accident :)
I'm not crazy about the fact we've added  yet another layer of complication to the file grid implementation.
@petervandenabeele The following steps is my confirmation step :-)

```
$ git clone git://github.com/kennyj/rails.git
$ cd rails
$ git checkout fix_4749
$ cd ..
$ rails/railties/bin/rails new app --edge
...
Using railties (4.0.0.beta) from git://github.com/rails/rails.git (at master)
Using coffee-rails (3.2.0) from git://github.com/rails/coffee-rails.git (at master)
Using jquery-rails (2.0.0)
Using rails (4.0.0.beta) from git://github.com/rails/rails.git (at master)
Using sass (3.1.12)
Using sass-rails (4.0.0.beta) from git://github.com/rails/sass-rails.git (at master)
Using sqlite3 (1.3.5)
Using uglifier (1.2.3)
Your bundle is complete! Use `bundle show [gemname]` to see where a bundled gem is installed.

$ vim app/Gemfile
...
group :assets do
  gem 'sass-rails',   :git => 'git://github.com/rails/sass-rails.git'
  gem 'coffee-rails', :git => 'git://github.com/rails/coffee-rails.git'
...

```
I'm replying to the issue.

On 31.12.2011 04:20, Aaron Meurer wrote:
> Since series doesn't seem to play well with essential singularities (like with exp(1/z)), can we generalize the workaround I suggested in the issue:
> `exp(1/z).subs(z, 1/z).series(z, 0, 6).removeO().subs(z, 1/z)` (I guess if we can make that work, we should implement it in the series code, not here, though)?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ness01/sympy/commit/28de3b6004ad93270dd6f3c38039060c76a1ea49#commitcomment-829268
>
Wow, has this ever grown :P
there is trailing whitespace on the end of this line
Ну попробуй заюзай 2 несовмещенных :)
Epic Fail >< xD
Ok, weird. I just changed my `NameError` into a `NoOptionError` since that seemed to be more specific to the error and the error message was essentially the same as my manual one. I then decided to remove this check (srsly -.-), since I deemed the `NoOptionError` raised by `get()` sufficiently meaningful. Testing the change revealed, though, that `get()` didn't raise an exception in this case (I don't get why) which results in this diverted JSON error: `TypeError: expected string or buffer`.

You don't wanna debug that, so I'm gonna keep the check and just make it into a more specific `NoOptionError`. So much on the effects of not checking options ;)
The tarball packages are uploaded to the AUR repository. I committed the package file just to keep a release record - they aren't accessed for anything.

The Archlinux package system is very simple. The package is a small bash script (PKGBUILD) which clones the github repository and calls whatever build commands are required.  That's why it should just work as if manually pulling the clone and cmake, etc.
@petebacondarwin see https://github.com/kanso/kanso-utils/commit/71b43545e44432130b3d70af2248924db9bde191 - I've added better support for dynamic modules in the new version of kanso-utils. This is available from kanso-utils v0.1.5 in NPM
This patch is already applied to the master branch
The new GetPlayer function should probably also be used for the group online / offline status. There was a commit who fixed player being displayed as offline while he is teleporting by directly using the hashmap because the old GetPlayer was only return players currently in the world
Did you tested if this works too if the unitsync (or spring itself) scans for new files and i.e. wants to read the dependencies?
I might look some more into the way the timer is calculated. Thanks for including this and thanks for continuing to work on opvp ;)
crash with same error except ('rwb') where ('rw') is mentioned
Если вызвать этот метод после clear() получим java.lang.NullPointerException
Also this light thing should come out of isSpeedGood(), and instead happen in a command that calls isSpeedGood().
How about YOU commit some code?
Every block has an entity attached to it, it can either be null (if the material doesn't have a tile entity) or an entity with the specified controller :)
this seems to be a legit fix - but I wonder: can we use `localStorage` instead?
Friend, why not to use the same deleting guids?
Hi Mathieu,

that `$0` points to the local script path. If you followed the README and checked out deliver into `~/.deliver`, then `$0` in this case would be `~/.deliver/bin/deliver`, regardless of your current working directory.

There is a separate [`DELIVER_ROOT` variable](https://github.com/gerhard/deliver/blob/d875138b33553266801a89c036fc6177158222b8/libexec/config#L10) which must resolve to the root path of the app which you're delivering. In that context, a local `.deliver` file is important because otherwise the utility couldn't load your app's settings. I am thinking to do away with this requirements, [see my notes](https://github.com/gerhard/deliver/blob/e9c08eeed02f4d83a446ff756f64cd0b30584b30/examples/gh-pages).
What's up with Dart then? Splitting their focus?
a bit lower is code 
http://img6.imageshack.us/img6/766/53705028.jpg

why not remove it and combine it like this?
http://img40.imageshack.us/img40/8043/74801895.jpg
why you think its lunar festival?
Some people on our server say, that lags appeared like 1 year ago, when we opened ulduar and SotA. Some say, that it appeared just like 6 months ago :/ Cant say. In the beginning we thought that its just our problem, some instance script stucking, or so, so its not reported anywhere. These are just rough guesses tho, so dont take the 0,5 year and 1 year for granted.
:) calculation not may be maked more often than the map|unit refresh period (100-200ms)
yes. i'm planning rewrite all ->exitVehicle() expressions on aura correct removing. now player exit from vehicle, but affected by controlling aura :)
да запросто, вот как тут - ГМ вылазит за пределы карты мира (-17000, -17000 - это оно и есть) и говорит namego кому угодно. сервер лег, потому как клиент может вылезти за пределы карты из ДБЦ, а сервер - нет.
factory_girl - изменила структуру файлов. Возможно у тебя не последняя версия - лучше обновиться и использовать новый возможности
bueno y cual es el problema aqui?
大大我测试 IE了 但是不行啊 没有自动刷票 没有自动提交  没有高亮 没有音乐 非常辛苦啊
different indentation.
is it necessary to have this prefix for the id ?
rhyme intended
I think all of them. First I noticed on supermales because they had letters on their costumes.

http://planetquake.gamespy.com/View.php?view=Quake2.Detail&id=83

Seems like everything is mirrored, not just skins, for example characters hold weapons in their left hands.

Check ogre here:

http://glge.org/demos/shadows/

vs here:

http://alteredqualia.com/three/examples/webgl_morphtargets_md2_control.html

I was trying to do some workaround - what did the trick was inverting X, reversing vertex order in faces (ABC => CBA) and rotating around Y in other direction. But this is weird, your code is nice and simple, it should just work.

At least we are not alone, these folks also have mirrored ogre:

http://www.openprocessing.org/sketch/11113
I did see that, but new users have sending enabled by default. Maybe sending messages to Eve should be disabled by default?

---------------------------------------------------------------------------------------------------------------------------
In accordance with The Companies (Registrar, Languages and Trading Disclosures) Regulations 2006:
*	Registered name:			 The American School in London Educational Trust Limited
*	Registered number:			 784923
*	Place of registration:			 England
*	Registered office address:		 One Waverley Place, London, NW8 0NP
---------------------------------------------------------------------------------------------------------------------------

Email has been scanned for viruses by Altman Technologies' email management service - www.altman.co.uk/emailsystems
this is better added in the functions.php file like:


add_action( 'init', 'fd_scripts_register' );
function fd_scripts_register() {
  if ( !is_admin() ) {

     wp_deregister_script('jquery');
     wp_register_script('jquery', get_template_directory_uri().'/js/jquery-1.7.1.min.js');
     wp_enqueue_script('jquery');
    
     wp_register_script( 'foundation', get_template_directory_uri().'/js/foundation.js', array('jquery') );
     wp_enqueue_script( 'foundation' );
    
     wp_register_script( 'app', get_template_directory_uri().'/js/app.js', array('jquery') );
     wp_enqueue_script( 'app' );
    
     wp_register_script( 'modernizr', get_template_directory_uri().'/js/libs/modernizr-2.0.6.min.js' );
     wp_enqueue_script( 'modernizr' );
            
  }
}

add false for head inclusion or true for footer inclusion, stylesheets also can and should be done like this,and it makes it easier when using option pages and meta boxes.

add_action( 'init', 'fd_css_register' );
function fd_css_register() {
  if ( !is_admin() ) {

     wp_register_style('style', get_stylesheet_directory_uri(). 'style.css', false);
     wp_enqueue_style('style');

     wp_register_style('foundation',get_stylesheet_directory_uri(). 'css/core/foundation.css', false);
     wp_enqueue_style('foundation');
    
     wp_register_style('app',GENEVA_I7_DIRECTORY . 'css/core/app.css', false);
     wp_enqueue_style('app');

  }
}  

obviously change paths to suit file structure.
Then why'd you write it that way?  haha  ... `git show 7720020e`
can you not import this at the top of the file?
Should NzbDrone search for missing past missing episodes?
current description is kind of misleading since we still download them if they show up in rss, but we don't search for them.
i renamed OPTASLIST module to LF
Why require a separate project to fix this?
@dkubb that could be also useful.
remove the spaces between commas because our minifier doesn't do it (I don't think)
f.exists() is what makes it expensive.

mkdir() is not removed, but moved to the constructor, so the directory will always exist and I don't think the behaviour is changed
Line 253 is indeed a bug, but even with this bug it works on my side.
Great that you found the solution to get it working for you.
Ah, hm, you should be able to do a manual clone from Github though...

You can either:

1. clone from github: git clone git@github.com:lassevk/DropChest.git
2. download the ready-built binary, built from the next-to-last changeset, and the last changeset is not interesting for anyone else either, it's just some debug messages

Make sure you remove any old versions of the DropChest.jar file from your plugins folder, and then drop in my replacement and you should be good to go.

Vennlig hilsen

Lasse Vgsther Karlsen
lasse@vkarlsen.no

On 27. jan. 2012, at 16:11, sorklin wrote:

> Unfortunately, i'm not narrowtux. So I can't do anything about the main code base.  I've been following your fork because I'm trying to get it working on our servers.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/lassevk/DropChest/commit/200bbe49ae3ac4476ffd12d4108791e8d382ed08#commitcomment-908398
Two start tokens in a single regex?  That seems wrong.  Try replacing with a disjunction of the two things you want supported.
Why taskAssignArray?? I thought it's feedback already....
The other rename commit died in a sketchy rebase, so I'll repost @Brianetta's comment on that here:

>I'd still recommend Sidereal.

>X: Beyond The Frontier has a stationary external camera as its only camera. Activating it dropped it in space, and it stayed in a fixed location, tracking the player's ship as it flew around (or, more usually, away).

>Sidereal might not be an extremely common word outside of astronomy, but it's well known and understood in that circle. In addition to this, it's unambiguous and easy to look up.

>Still, it's your code, and I'm not going to get in your way if you want to name it this way instead.

Yeah, I'm still somewhat on the fence about the naming, which is why I've kept it as a separate commit. You're right that "stationary" connotes that it may stick in the same position as well as ship-relative orientation—I hadn't considered that. 

It's not a huge deal either way, but my concern is that there isn't a wikipedia page dealing with the meaning of "sidereal" that specifically states that it means "star-aligned", and the existing articles are about sidereal time and other time-related concepts, so it may be difficult for the player not familiar with astronomy to make the inferential leap from that to a camera that stays aligned relative to the background stars. Another (less significant) is that it may look like a compound of "side" and "real" instead of one word and leave players confused; my own brain still parses it that way if I don't pay attention. 

It's not that I have a low estimation of players' intelligence, I just want them using as little brainpower as possible. :)
I can't remember! :)
looking at this now, I might have planned to suggest `marrangeGrob()` as a more general alternative.
I see you checked in `5/code/rsa/ks_primitives_unchecked.pyc`. Please remove it, together with any other .pyc files that might have made their way in. Afterwards, add `*.pyc` to `.gitignore`.
Are you sure this commit is actually useful. I must confess that I am no big fan of "pointer to pointer" types (things like char **).
Can you tell me more on that ?
They can keep their settings as the path to the config file stays the same (it has always been stored in the location where the other directories are now placed by default).
Trevor, please only use Scaml unless you really can't do it with Scaml.
The tests fail like this without the fix:

`  1) Refinery Updates Admin updates updates list shows two items
     Failure/Error: visit refinery.updates_admin_updates_path
     ActionView::Template::Error:
       uninitialized constant Refinery::Updates::Admin::Updates
     # /Users/djones/Code/resolve-digital-website-new/vendor/engines/updates/app/views/refinery/updates/admin/updates/_updates.html.erb:1:in `___sers_djones__ode_resolve_digital_website_new_vendor_engines_updates_app_views_refinery_updates_admin_updates__updates_html_erb__406718801257296969_70253743153500'`
`global` isn't strictly required here, as `COMMANDS` is a mutable object - did
you add it as hint for the reader?
Do you see any improvement with this change?  If yes what and how did you measure it?
Sweet good idea.

Sent from my iPhone

On Jan 21, 2012, at 9:16 PM, Glen Zangirolami<reply@reply.github.com> wrote:

> Awesome thanks! This slows down the server restart a little but I can deal with it. I split up the requirements file here: 5f082ca2f4649ed896a8130c80fe418059b95f02
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Codrspace/codrspace/commit/5ab93070fa536d5ef024bef15282aed7361c9352#commitcomment-889356
Some rates in the H_Abstraction library are not readable because the dictionary and tree files were not pushed. Running any jobs with the updated rates will throw an exception:

ERROR: jing.rxn.InvalidKineticsKeyException: unknown fg name: C/H2/Cb
    at jing.rxn.ReactionTemplate.readLibrary(ReactionTemplate.java:1500)
    at jing.rxn.ReactionTemplate.read(ReactionTemplate.java:1302)
    at jing.rxn.ReactionTemplateLibrary.<init>(ReactionTemplateLibrary.java:95)
    at jing.rxn.ReactionTemplateLibrary.<clinit>(ReactionTemplateLibrary.java:50)
    at jing.rxn.TemplateReactionGenerator.<init>(TemplateReactionGenerator.java:56)
    at jing.rxnSys.ReactionModelGenerator.initializeReactionSystems(ReactionModelGenerator.java:1248)
    at jing.rxnSys.ReactionModelGenerator.modelGeneration(ReactionModelGenerator.java:1312)
    at RMG.main(RMG.java:96)
ERROR: java.io.IOException: Error reading rate library. The error message is:
unknown fg name: C/H2/Cb
    at jing.rxn.ReactionTemplate.readLibrary(ReactionTemplate.java:1528)
    at jing.rxn.ReactionTemplate.read(ReactionTemplate.java:1302)
    at jing.rxn.ReactionTemplateLibrary.<init>(ReactionTemplateLibrary.java:95)
    at jing.rxn.ReactionTemplateLibrary.<clinit>(ReactionTemplateLibrary.java:50)
    at jing.rxn.TemplateReactionGenerator.<init>(TemplateReactionGenerator.java:56)
    at jing.rxnSys.ReactionModelGenerator.initializeReactionSystems(ReactionModelGenerator.java:1248)
    at jing.rxnSys.ReactionModelGenerator.modelGeneration(ReactionModelGenerator.java:1312)
    at RMG.main(RMG.java:96)
CRITICAL: Error in read in reaction template: H_Abstraction
CRITICAL: Error reading rate library. The error message is:
unknown fg name: C/H2/Cb
Change the xla() function to attr() around the $this_message['dueDate']  call. (since you do not want to translate the date)

Quick reminder (I know this is repetitious, but repetition is a good way to learn :)

Note the xla() and xlt() functions translate. Here is what they do:
xla() -> xl( htmlspecialchars('',ENT_QUOTES) )
xlt() -> xl( htmlspecialchars('',ENT_NOQUOTES) )

The functions that do not incorporate translating are attr() and text(). Heres is what they do:
attr() -> htmlspecialchars('',ENT_QUOTES)
text() -> htmlspecialchars('',ENT_NOQUOTES)
My idea is to generate random URLs for posting keys which would only last like 2 minutes exposed in HTTP. I don't think in that time an attacker could have time to make an attack, without knowing that an invitation is going to be generated. Using SSL also limits that only servers with a domain would be able to serve invitations.
yep, commented out for the reason to expose (visually) that there is this option/event; but to avoid being invoked if it is not set from outside. See here for example, https://github.com/mootools/mootools-core/blob/master/Source/Request/Request.js#L27, it's done there the same way.

I also thought about the reposition() option and I already guessed it was your convinient method for that purpose; so I'm fine with merging it into that.
Här kan man skriva kommetarer till koden
Well, in that case you don't know how to write a documentation,
`Monitr::log( <message>, <errorCode> ); // Default error-code: E_WARNING`

If you write:
`Monitr::log("Test");` it does not work.

If you change this line according to my code, then it saves you from getting an instance everytime you want to use log method and it makes it easier to use. 

But this is obviously a documentation problem. I don't know what is so hard about writing a simple example?
well, Dreadii my previously mentioned comment was only irony and kinda fun and if u don't udnerstand english irony please be quiet, are you not the one who merged with TC and didn't fix the Build for about 1 week... ;)
Incorporated this change in commit 9a5db5850c. Thanks. :)
tests fail otherwise. Why?
Need to remove check_output here as well.
Good. Because that's a very nice feature. Since I got it on a metaforum in 2006, that's something I can't really live without.
Still missing preview and auto-refresh on the discussion list though, but I missed it so much, I would have found the time to hack something together, even though I don't really do php :)

Thanks
Eilo merge this branch wit tc please.
Ja aber wenn du dem konstruktor einer BossAi nicht die Data mitgiebst funktionierts auch nicht unter trinity.
And where we are using LD_NOASNEEDED?
How else do you get/see default module?
Grmpf. No escape :-).
Interesting - is this actually necessary? I didn't get any errors on my side with it absent from the ns declaration.
So you would be all right with this being in release/1.8.2?
We're using pep 8's line wrapping throughout the codebase. Or at least, that's the goal. Until recently, I've been wrapping lines at 80 instead of at 79.
Hey there,

I think you emailed me about this, no? Any chance you wanna make this a pull request if it's something others will find useful?

Would be uber awesome! Thanks!
does this fix the request firing or just comments it to avoid interference with building?
great work!1
extract setXY(PointDouble) into some SVG utility library?
I love how you float the main content text to the left in the right-nav design  ... and to the right in the left-nav design. Great! Gives the reader lots of white space and sets off the text nicely.
Were there any known issues with these other update actions?

Are we changing them to be consistent when it should really just a special tweak for the firewall update behaviour?
I would have added the chunkHeaderSize before rounding up I think.
Great that we have the code split by exceptions (so that we know the exceptions that can occur). Thanks!
just making notes, & anyone else who wants to comment.
I think this should be clarified a bit so as to not completely disallow scrolling. I agree that main controls need to be tied to the edges of the viewport and not move outside it, but for content displayed in the app (e.g. messages in Gmail or a long document in Google Docs) scrolling can be important. Do you agree?
Retrieving archived services breaks because there don't exist
Service::Archive subclasses.  acts_as_archive doesn't define them.  So
rather than implement that, I decided to just effectively disable STI for
the Service::Archive class instead

On Thu, Jan 5, 2012 at 1:54 PM, Kevin Menard <
reply@reply.github.com
> wrote:

> Why is this change necessary?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/backupify/acts_as_archive/commit/e51fb2bbb5c29824889e5909d9fff0fcc076add3#commitcomment-841267
>
We should cache `$(this)`
Doesn't this mean we can remove the econ plugin dependencies from MV2?
@FernFerret You're right! I'm not at home right now, so feel free to push these changes yourself :)
Unfortunately I don't believe we can test this function this way.  Two equal JSON objects can produce different base64 encodings:

``` javascript
{"typ":"JWT","alg":"HS256"}
```
vs
``` javascript
{
  "typ":"JWT",
  "alg":"HS256"
}
``` 
They represent the same object, but will encode to different values.
Just having it enabled in the defconfig (not even the default governor!) was causing kinteractived to hang when in LPM.

A governor causing softlockup events even though it was disabled = immediate "kill it with fire" reaction from me.  It might be possible to fix the interactive governor so it doesn't act badly, but I just removed it.
:hover ?
That would be excellent!  When I started reviving the original hdpvrcutter.pl script I had no idea it would even get to this point.

I must say I've actually got a post-it on my monitor with #1 and #2 written on it, I suppose waiting for a rainy(ier) day.  I guess #3 is a logical/necessary extension that would work well with my next todo: allow the user to feed in the list of cut-points in exactly the format that mkvmerge takes, essentially allowing the cutting functionality to work without being coupled to MythTV.  I have some very older recordings that I just copied with commercials that I want to eventually edit.

That said, I happily invite you to submit additional pull requests :D

As far as coding standards...I can't say I ever thought I had any ;)  I suppose I try to adhear to "good practices," but I'm very open to ideas/changes that might improve things even more.  So, if you know any organizational or Perl related tricks, I'm all ears.

Cheers!

-Edward
This should read:

if not language == getattr(settings, 'DEFAULT_LANGUAGE', 'en')

We can assume, if no default language is provided, that we can selfishly set it to english.  If we *always* assume english is the default language, we're being a little anglo-centric :-D
ConfigParser allows values to contain expressions, so that values can be inherited, by calling config.update() for plugin, each previously loaded config file is reloaded, and all it's expressions are re-evaluated. Imagine a config with says:

```
[my_group]
search_path = %(value):other_path
```

By reloading the conf each time it could happen that the search_path gets messy like:

```
path_from_plugin_1:path_from_plugin_1:path_from_plugin_2:path_from_plugin_1:path_from_plugin_2:path_from_plugin_3:path_from_plugin_1:path_from_plugin_2:path_from_plugin_3:path_from_plugin_4
```

Instead of:

```
path_from_plugin_1:path_from_plugin_2:path_from_plugin_3:path_from_plugin_4
```
You have accessor methods for this, so make it private.
I'm not happy with it either; this was just what I hit upon to make the migrations run under 1.8 and 1.9.

I'm not sure requiring this at the top of every migration is the way to go even without the expand_path stuff, but I wasn't able to get it to work by moving the require 'lib/migrate' into various places in application.rb.  The file would get loaded (tested via the console) but the migrations failed.
the ``&&`` should be on the next line
Bug reported here: https://shopify.desk.com/agent/case/144380
The CONFIG_MSM7X00A_SLEEP_MODE=0
is back after you enter menu or compile.
you have to change it in code, the config is only reflection of what we have in code file.

This also for the 100HZ or 300 you set before (deep in commits), it's still 100HZ!
Do we even need this class?  It is being called from Dashboard.java and uses code in Dashboard.java.  Mayve we can just move this method into Dashboard.java?
Actually I was wrong:
This is the proper copyright of the framework thing (from https://github.com/aptogo/OpenCVForiPhone)

# (c)2011 Robin Summerhill, Aptogo Limited. http://aptogo.co.uk
# This script is distributed under the GPLv2 licence.
# Based on work by Eugene Khvedchenya (OpenCV build) and Diney Bomfim (iOS framework)
I agree, Iw as just musing on its usefulness.
So this turns on kernel panic mode? When its off what happens? System crash causes hot boot? Maybe it should be expressed on. I just don't know
yep, totally fine with me (unless we wanna be pedantic about punctuation), merge away when ready
I probably should have put this in a midmap folder and I pit zero thought into style I just needed an icon to build.

We also need icons for fullscreen on and off in power menu
`return U.toJson(this);`
Yes, makes sense to have it private and still throwing the exception because the user will not interact with the method directly also the new method name makes more sense, just simpler.
The intention was to replicate as much as possible a standard session, since we can't predict what kind of tests the programmer will try to write. It could be easily changed though.
I think it's not a real use case in the OAuth module (value = null), though we could imagine just having the parameter without any value at a theorical level, meaning value = null. It's different from not having the parameter at all.
I have a 404 not found on this "Javadoc" link.
Why not, indeed. If everyone agrees, I am happy to use it.
closing over non-threadsafe vars and update them on other threads is a no-go for me
Should probably mention that it's a OneForOneStrategy.
What number of restarts should it be configured with?
Use of the trait is discouraged because of potential issues with binary backwards compatibility in the future, use at own risk.
Should be added to the ConfigSpec
you want more DSL syntax in-between?

    transform {
      case x => ...
    } using {
      case s: State => ...
    }
because all return an Option
you mean that I should take it out and get people to use copy?
I like that, I've actually used copy a couple of times in tests instead of these methods, cause the semantics are clear already instead
of some weird named 'withBody' etc. let me know.

Maybe all this with... stuff should be removed, ar least for Scala and leave things there for Java, and also leave in the methods for Scala that do something different than case class copy style
It's adding pluggability of failure detector and seed nodes.
I have started refactoring which will remove the need for a test Cluster extension instance, so problem will be solved in another way.
We don't use any unnecessary curlies. Newline is ok.
The variable "d" is in another function (get_norm_value) or I miss something?
I am not sure that the non-linear peak finding is useful with these models.  These models are also an exception (as Stephan said) in comparison to the standard models with fitting parameters. The code duplication is really minimal. We are very positive that it will be too much effort with massive memory costs to try to design this in a general way. So, please lets not go towards that direction.
```
``data.shape[-1]``
```
Technically, PEP-8 says the description should be aligned under context. But I say that more informational; I personally think this looks fine.
Why index 1 and not 0 (same on the next line)? I'd just do
```cpp
if (GetDirectory(item.GetPath(), items) && items.Size() > 0)
```
here because GetDirectory() might also fail.
Ненужно этого делать
these two lines are good
Actual image please :-)

For mrjob, we included the image in the repository and had the Readme link to the Github raw image file URL. That's how the elephant shrew is in the Readme in the mrjob repo.
Using the index of the first non-nil match group, we find which chunk subclass was matched and also returns the range of groups for the class.
I have taken this CSS change out, and will manipulate pointer-events myself in Cloud9. Thanks !
Maybe make this level_num
Please leave out `{` and `}` when you got only a single line body.
Well, if you apply my comment on ``getTests``, the dataProvider would also be defined in the parent class so it would not be an issue anymore
There might be a hack (I mentioned it in the forums, but I agree it is ugly).
I encountered an odd bug where a text field would still be tracking even if window was not key. It has since gone away, so I do see the point you make.
Property syntax is very yummy: `self._offscreenRightFrame`!
I suggested this because as long as a method acts like a getter for something, it makes sense to use it. (`TUIScroller`'s `isVertical` method.)
s/through/with/
No, I don't think so
My latest commit changes this to

```scala
var oldCount = 0L
var nextCount = tc.count()
do {
  oldCount = nextCount
  tc = tc.union(tc.join(edges).map(x => (x._2._2, x._2._1))).distinct().cache();
  nextCount = tc.count()
} while (nextCount != oldCount)
```

`nextCount` is initialized to `tc.count()` instead of `0` in order to halt after one round when the input is already a fixed point (when the input is a complete directed graph, for example).

I also added a `cache()` call when constructing the initial `tc` RDD.
No need to use this as a popup plugin here. You can just use the FeatureEditorGrid as item and add it to the popup.
Good catch. Another thing to take care of are relative URIs in the Location heade.
It's nit-picky, but please add a space after `#`.
I've searched a bit through the rails source: heredocs seem to mostly be used for code and marked up text like HTML.  One of the exceptions where it is used for plain text message is this: https://github.com/rails/rails/blob/master/railties/lib/rails/ruby_version_check.rb
but i find it weird that the indent of the message would depend on the indent of the code where the message is defined.  I also would not like to show off with a nicely formatted deprecation message among other deprecation messages :).

P.S. BTW, heredocs are weird ;).
No problem :). I remember @tenderlove arguing against it somewhere else, because it'd accept hashes like `{ api: :v1 }.flatten # => [:api, :v1]` making it a wider api.
This actually works without modification even when setting `inheritance_column` to nil: If STI is disabled, `class.descends_from_active_record?` returns true, `send` is not called, and `becomes!` behaves exactly like `becomes`.

If you disable STI, there's no reason to call `becomes!` anyway, as all the method deals with is STI...

I'm against bloating the test suite by writing a test for a problem that actually doesn't exist, what do you think?
Where is the `:kind` option?
On Thu, Apr 26, 2012 at 8:31 AM, Ronan Lamy
<reply@reply.github.com>
wrote:
>> @@ -438,8 +438,14 @@ def __init__(self, rules):
>>
>>  class InconsistentAssumptions(ValueError):
>>      def __str__(self):
>> +        from sympy.utilities.misc import filldedent
>
> No, the implementation of filldedent doesn't matter. It's the import statement that is the problem.

Can we create a 'sugar" module (or "pure_python" or something to that
effect) that has no dependency on sympy that can contain these sorts
of things? Or is there a better place to put filldedent, perhaps
core.functions?
Yeah, I was just cargo-culting nearby error code, I don't have a clue.
Oh, I think you're right, but when analog=False.  I will read more to make sure and update it.
That's a much better idea.  I'll change it in both welch and periodogram.
Note that you do have to keep the backticks. Then it will still turn into a link. The reST backticks try to resolve what's in between them as a function, and convolve() is right next to fftconvolve() so it will find it without any problems.

The ``:func:`` directives in the ``lsim2`` docstring also look unnecessary.
I guess tests are now in test_basic.py.  They don't look the same to me, though.
Otherwise, can you give an example of a problem case? Everything should define whether it is commutative or not -- if it doesn't and the existing code raises an error then that indicates an object that needs to answer the commutativity question.
I updated test1 with your changes and made a few more over that. Do
they look ok? (You can see which are the changes I made by looking at
https://github.com/smichr/sympy/commit/88e4fe014772e1a70b6446c868a85e527b704195

/c
Lines should be limited to 80 characters.
Sorry, I actually meant these comments. Those ones above can go in a module docstring.
Would it make more sense to put this in ode.py?

Also, why not add properties to make this have the form `Cn`, where `n` is an integer?  That might make some operations on constants simpler.

Anyway, I am +1 to the idea.  Is this used by default now?  If so, it should be documented, and an option to dsolve() added to just use regular symbols (this can be useful to users, but it would mostly be to make the tests sane).
Maybe say something about the fact that these functions are only for convenience and that the module is build around Series and Plot objects that can be called directly.
Probably. I wil try to change.
better name: send304Header().
this shouldn't be needed, objectify is in maven central
The below implicitly tests for non-nil. This test is redundant.
`# include <...>`
We don't use require relative.
add:
require 'file_utils/file_generator/file_generator_parameters'
You have a parsing parameters function, use it.
Make it return a hash and not write itself directly to the Params module.
Then after config file checks and loads merge the hash to Params module to sustain precedence.
I thought we were already caching Domain objects - why can't this just live in the cache for that?
I think we can strengthen this text a bit "a lot of looping" seems weak.
just comment? should we delete altogether?
see above nit: change to $('body').is('.edit.template')
Please indent under `id`
Yeah, not important. I was pondering how we were going to make sure that ENV["PWD"] was always prioritized over Dir.pwd, lest this be an issue some time and someone changes it without knowledge of the original reasoning.

+1 otherwise.
It's used. `login_forum_box($row);`
That's `$custom_html`: https://github.com/phpbb/phpbb3/pull/1100/files#L14R65
phpbb_style_is_active() alternatively.
Should probably do an array_intersect on $readable_forum_ids with the f_read forums so that this is not even an issue.
Probably shouldn't do this but throw an error instead. Not sure.
You should probably preserve the extension. I'd also prefer `_` instead of `.`.
I understand that this code may have been copied from someone else but if you can please add access control specifier (public/private) and a docblock. This applies to all functions in this class.
"WIP"

On 13 Nov 2012, at 09:29, brunoais <notifications@github.com> wrote:

> In phpBB/assets/javascript/autosave.js:
> 
> > +				continue;
> > +			}
> > +
> > +			break;
> > +		}
> > +
> > +		return data[keys[choice - 1]];
> > +	}
> > +};
> > +
> > +phpbb.autosave_save = function (key) {
> > +	try {
> > +		window.localStorage[key] = $('textarea[name=message]').val();
> > +	} catch (e) {
> > +		// Quota exceeded, should inform the user that their autosave isn't
> > +		// working
> Better solve this somehow. How about making a div that has that written appear under the textarea?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Globals should be on the first line of the method body
Query should be broken into multiple lines.
我又考虑了一下，是否叫username更合适一些，login有时有动词的意思。
Re #1: suggest renaming the test or commenting to let people know that it's a workaround/incomplete for the moment.
Re #2: you might try 1) backup all your changed code 2) git reset your cloned (local) repository 3) run js beautify on all the files you intend to change (this file, tests). commit and pull request this "style" fix. 4) add your functional changes to the file and then issue another pull request for "functional" fixes
Also, can you change the url: Host + '/....../patientlist/<uuid>?v-full' instead of the hardcoded uuid
Done.. Shall I send a new pull request??
Why not use ```numpy.fromfile()``` here as well?
I take it you agree with my derivation then?
If the old code was wrong, why didn't verify_grad catch it? Are there unit tests for these cases?
Is it intentional to have the same exercise in ``modes.txt``?
Can you expand on NYET explaining why a test is commented out. We're trying to not have commented out tests without explanations, because then they look like they might be accidental. I'm probably the chief offender. ;-)
Добавь внутри метода какой-нибудь комментарий вида `// stub`, чтобы не возникало желания метод удалить за ненадобностью ;)
My preference is to keep the explicit months_per_year notation so that we know what's going on here in the future.
Just created #15 for that.
I did that to reinforce the concept of code reuse which pervades CBVs. Since all but one of the previous examples had new templates and this was the exception I wanted to highlight said 'exception'.