labofsahil/github-event-dataset-2012 huggingface dataset:
Fix Rails 3.2 deprecation warning

"DEPRECATION WARNING: The InstanceMethods module inside ActiveSupport::Concern will be no longer included automatically. Please define instance methods directly in ActionController::Base instead."

The warning blamed Draper in our case and never mentioned breadcrumbs_on_rails, because Draper loaded ActionController::Base and indirectly triggered this.
Mas vale tarde que nunca.

kata de diciembre hecha con python y con pytddmon
d2vzw: cleanup after VRBLK3 update and remove gps

gps blobs not needed anymore per: http://review.cyanogenmod.org/#/c/27813/
had a derp on the last push for libacdbloader (was defined twice and i only removed one)
add sec_keyboard keyboard daemon

Change-Id: I5767efa82bbc2342616ad1a8a55cbc39a43a40a2

Hi,
This is a little keyboard daemon that is necessary for the keyboard dock to work on Galaxy Tab 7.7.
It sets serial port (baud rate etc) so the kernel driver can communicate with the keyboard.
Update BBM manifest permissions
Fix list diplay

Hello,

I've made two minor corrections in the ZCML part.

**Problem**

On this page: http://collective-docs.readthedocs.org/en/latest/components/zcml.html

We can see that the two lists `It provides:` and `Downsides of ZCML are:` are not correctly displayed because there is no blank line before the items and the previous line.

**Solution**

I just added the missing blank lines ;)
Pitchfork music player support
Remote and componentsfromfile

This patch adds 2 functionalities to the checker (CLI):

- Ability to inspect remote paths out from own codebase: If one existing full path is provided, it's checked.
- Ability to rely in one static file containing all the plugins and subsystems available, instead of extracting them from current codebase.

This 2 functionalities shouldn't affect normal (wen/cli) self-execution at all, but are required by the CI server in order to be able to check different branches (with different plugins/subsystems) in any readable location (the CI server own git clones).

For your consideration, TIA and ciao :-)
[TIMOB-7582] Fixed typo for online variable.
Changed the attribute. From ID to AGE.

Changed the attribute. From ID to AGE.
martianrocket sending changes to saurabh
Copy

Some more changes to speakers and sponsor section.  Updated logos, and changed CfP text such that it implies the site is live rather than coming soon.
various updates to sensors, drive code
add cooper defconfig
Make cucumber tests pass

Definitely needs some more tests. hopefully I'll get some time this weekend. But this makes the cucumber tests pass.
6.x

adding solr db entries working
ahah'd form elements with ahah behaviors now triggering
some code style fixes
S3 is a conclusion
EmailResponseDAO
models kinda implemented. Looks pretty scary atm
Fix for questing in Pangaea

Fixed questing in Pangaea
Perdition will likely also need fixing but I don't have it unlocked yet so cannot confirm the 'tab_<name>' to add
normals example modification

used ofMap instead of more opaque direct version and cached matrices to avoid multiple printouts
cleaning up devApps

i've cleaned up devApps a ton and moved a few relevant examples into examples.

* AdvancedImageLoading: i'm going to work on it for a bit, it seems to be broken.
* fboTester: i think @kpasko is going to do some porting of this.
* ofSynthFloaties, synthExample, synthSequencingExample: should be removed or ported by @jonbro
* VboExample is pretty awesome, and should be ported to examples. but needs a little more work.
* VboiPhoneExample i can't guarantee this builds. if @obviousjim or @julapy could test this, that would be great.
Almond module loader!

Switching to this as a future proof value add for Ripple-UI.

Zero cost now but gives us some amazing features such as :

- AMD module loading
- Loader Plugins
- Smaller deployment size, almond is specifically designed to work in a single file deployment.

@kwallis - can you start getting this into the OSS licenses and such? once that is done I will update the license file in this branch to take out browser require and add in almond.
Updated navigation styling to be consistent with the classic...

... Wordpress theme version.
Nalexander/test global session

No sense doing more on this.
updated text

Hey Ross, I have working text now. Also touched the CSS a bit. Still waiting on an updated logo from my designer, will push that once I have it.
Ajaxloader

i added a dynamic loading of calender events for my own little site because there were to many entries at once. maybe you want to pull these changes into your work.
re-requesting pull after removing HTML version and cleaning up copyright

re-requesting pull after removing HTML version and cleaning up copyright
Update Contents/API/Class_Action/Method_getImageUrl.md
Add support for Last-Event-ID

Adds support for Last-Event-ID. This is two fold. First the, ability to
send an event id to the client is added. Second, a new method is added
that is called with the Last-Event-ID if a connection is resumed.

This implementation has the limitation that the id has to be set before
data or a generic event is sent. This is because these two methods
currently finish a message (CRLF CRLF flush). Two alternatives
would be to either overload these two methods or introduce a dedicated
method that finishes a message. They both seem less attractive.

 - Add EventSource#onResume

 - Add Emitter#id
Give me your stuff
simple python reformatting
Turn on Utils and Network Menus [1/5]

Utils Menu
   Import Barclamps (add or update barclamps in a live system!)
   Export Logs
   Export Chef Objects

Network Menu
   Switch View (shows connections to the switches)
   VLAN View (shows network barclamp vlan config)

Documentation pending.


 .../app/views/support/import.html.haml             |    2 +-
 crowbar_framework/config/navigation.rb             |   18 +++++++-----------
 2 files changed, 8 insertions(+), 12 deletions(-)
decode url parameters

Added decodeURIComponent calls around url parameter strings.
Lps 23622
LPS-24880 Javadocs missing exception info for DLSync
shows the more section of each participant inline instead of in an alert box

see issue #23
bug fixed queues and stacks

Stack and queues did not transverse at all
Update RabbitMQ.Client.MessagePatterns/Configuration/AMQPConnectionSetti...

...ngsSection.cs

Corrected sample config XML (was missing '<connections>' tag).
Changed front page popular ideas layout to resemble the specs.

Minor issues in positioning the bottom voting distribution bar.
Test Fixes

portal/permissions/webcontent
portal/permissions/usecase
portal/permissions/messageboards
[properties] Renamed ci.* properties to review.*.
Updating directories creation workflow
added PHPSpecBundle dep
remove cfsetting from list of omissions

this tag was completed a while ago, just never removed from the readme
Layout frags

I wanted to wait to get at least all of the fragments done in the resources tab but it looks like that's going to take a bunch more work and this branch is miles away from master.  So before I start on the next round of layouts we may as well merge.
Pull the new manifest with extra dependencies.

I added missing dependencies for stuff that was already depended on in the asd-file. Additionally, there was one missing dependency for the navp_controller that is started in the launch-file.
Android extension 1.0.0.Alpha1 blog

Android 1.0.0.Alpha1 ready
Cleanups and bring up to scratch with Skate and Blade
ImageProvider should pass the size of the image and honor the ratio
Remove newlines from handlebars/minispade filter

The reported line-number for errors is incorrect in the Chrome/Webkit developer console with them included.
Фикс для loadThread

@VisualKei отпишись, проявляется твоя бага или нет. Если да, я тебе дам специальный билд, где скрипт будет вести логи и мы сможем точно определить в чём дело.
Улучшение readCfg

Теперь скрипт сохраняет только изменённые параметры. Очень полезная добавка к замене `(un)eval` на `JSON`.
Дополнения и исправления.
Pull Req: Fixes Issue #53

Fix for issue #53 - Packaging fails when the <access> element contains a single <feature> element

Unit tests also added.
Morisse thibault

Bonjour Monsieur,

J'ai un souci avec un de mes tests lié à la gestion des cookies. 

Je suis allé sur ce site http://stackoverflow.com/questions/5335183/racktest-cookie-help pour savoir comment tester les cookies, mais malheureusement en rajoutant le "before :all clear_cookies", mon test ne passe plus et je n'arrive pas à trouver le pourquoi du comment... Si j'enlève le clear_cookies, mon test passe.

Pourriez-vous m'éclairer s'il vous plaît ?

Merci beaucoup,

Bon week-end,

Thibault
added support for lazy load of multiselect data from ajax/callback

This pull request adds support for loading multiselect data lazily, either via AJAX or a callback function. The data is loaded once the multiselect is clicked on for the first time. The data format is assumed to be JSON, although it would be relatively straightforward to add support for XML datasources as well.

I've added qunit tests for the new functionality as well. Let me know if there's anything that's missing. I've also added a lib - mockjax - which I use to mock ajax calls on the unit tests.
Gingerbread

disregard the data/sounds/OriginalAudio.mk  changes.  

This is a cm7 commit that allows the use of a motorola world phone. 

https://github.com/CyanogenMod/android_frameworks_base/commit/264ebb11f3436de100bfe3a1c473f2d0821a6739
list.action was not translated

added PL translation for list.actions
Issue with call to SUPER::got_error

The params passed to SUPER::got_error method include $self as the first two params instead of just the first param.

Need to shift off the first value of @_ before calling SUPER::got_error to fix the problem. Otherwise the SimpleHTTP got_error method dies since the params are not in the positions it expects.

There are other places where the same issue with calling a SUPER method exists but this is the only one that was causing us problems at this point.

Sorry if I got the flow wrong on the pull request. My first time working with github.
Optionally preserve `where` clauses when calling the `count` method

Hi,

I've added an optional second parameter (boolean) to the `count` method. If this parameter is `FALSE`, the `_clear` method is not called. Handy for code such as:

```php
<?php
$this->mongo_db->where(array('foo' => 'bar'));
$total_count = $this->mongo_db->count('my_collection', FALSE);

$this->mongo_db->limit(100);
$results = $this->mongo_db->get('my_collection');
```

Cheers,
Stephen
add !default to the variables. 

it make the variables overridable even when declared before importing bootstrap-sass-scss. Without these they are 99% useless.

I have also created a "compass" branch while trying to make it a true compass extension. Let me know what you think: 
https://github.com/idflood/bootstrap-sass-scss-compass/tree/compass

good job!
Fix NoTabs test failure

[NoTabsTests] was failing on t/00-assure-deps.t in dzil test --release; also added git repo info.
Switched from Binary to cereal, allowed switching between lazy and strict ByteString, fixed some bugs

See email for details; microbenchmark results here: https://gist.github.com/6d958ade26ff70c2cce0
Java version of solutions
PVR Userspace update 1.8 DDK (rowboat)

PowerVR GPU Userspace Binaries Update (1.8)
-fix documentation link
Prevent export of spock plugin

This was breaking applications using Grails 1.3.x
remove bkup file i left
Reinstated tiny icons for non-html links

These are based on Andrew Harris' implementation in the 2007 templates with icons renamed and Open Document Format files added
Да, наистина $lifetime трябва да стане $keepMinutes
don't work for me :(
Maybe you should catch both if location does not exist OR mirror does not exist, and then don't rely on the other models at all anymore and just print out the IDs.
we like to draw with vietnamese noodles. we are phographers.
const CStdString &

(or possibly better just to use const std::string & )
Hey Luke,

Thanks for the heads up.  I never got those changes working in a
satisfactory way; in fact, IE's rule count didn't match my count using any
reasonably complex stylesheet, so I couldn't use it for the purpose I
intended (prevent IE from silently failing when there are too many rules
per file!).  A better method for solving the problem came to me later on...
I wrote a simple application to count rules using IE's own engine.  That is
available here if you are curious:

https://github.com/mhh/CssRuleCount


Matt

On Sun, Feb 19, 2012 at 4:00 PM, Luke Page <
reply@reply.github.com
> wrote:

> Hi,
>
> Just been looking through some of the branches for dotless.. I was
> thinking this would be nice as a plugin (It could count selectors too).
>
> A note - my recent changes use Rule for media queries, so the logic in
> this shelveset will not be correct if you update...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mhh/dotless/commit/fa8e5d3bf3898dc9a201e006c1725b321d6beecf#commitcomment-983033
>
Feel free to move these various definitions around. I don't know if there's a specific section of code for customization variables, for instance.

And please suggest distinct names for the custom variable and the hook function!
It is good to add a separate paragraph in the commit message with a rationale for the archives.
The compression is not in the streamed file, but in the file written to the filesystem and nginx/apache will know how to serve it only if you configure it. So it would never work by default.
esqueceu a versao da gem hein mano...
spaces are needed for line-break
Should be 2012
Actually it looks like (format "%S" nil) and (format "%s" nil) do exactly the same thing, so I misinterpreted some other error with %S I encountered.  I sent you the push a few hours after the fact and I can't remember what it was now other than something in the error handling was a bit off. I'll see if I can trigger it again.
OK, I thought you gave him or something.
If it didn't happen that way I apologize. :)
I'll merge this, although postgresql was not explicitly supported at the time of release of beta3, it will be by beta4 though.
might be possible, indeed :)
Reference https://github.com/mxcl/homebrew/pull/7688

The change was added at the request of another user who said that `make install-world` would include contrib.  After reading the docs, it appears that one must first run `make world`.  I'm trying it right now.
why is this #include needed?
My fork has merged your commit now.
Porque esse metodo ?
I think our clients got out of sync, but hopefully this change the right thing:
https://github.com/danvk/dygraphs/commit/e46e3d09ab519d3c154eca0df217b921d0e5b500

Thanks for the fix,
  - Dan
so we won't have tags on documents at all? only on revisions?
This Db works with Skyfire Original 4.0.6 core and their Rebase 4.0.6 core.
Nice! How about making this explicit in the definition.rb ? It defaults to vagrant.pub but if you want you can override it and it will use this logic.
Very funny, I've never heard that one before. You mean _bike-shed_ as in _kitchen-sink_?
i don't know now .
i will be create test patch after researching in random movement codes .
i'm busy now :)
You don't know what liquid is behind glass with this, right? It is bad.
Thanks Serge, will there be a 2.5 to 2.6 update soon? :)
Mistake of table, this goes in auth not characters, sorry.
I repeat in the table auth
I’m confused, is this a <1.7 example or a 1.7+ example? If the former, it should not have called require() earlier; if the latter, it should be using `DataGrid` directly.
We used to run quick tests after rebuilding. Is that gone now? It would be nice to have some sanity tests that can run in a few seconds as part of the build process. This goes a long way in preventing broken builds.
@HosipLan  :+1: :)
Mal ne Frage: War das so gemeint bei Issue #47
Since subtypes is related to skillsubtype, django required this to be unique. Unless I am missing something.
"the command line arguments are parsed and the their setting sare stored in ``self.config``" (let's use passive voice here -- this way we can avoid talking about which component actually parses the arguments.
а чем тбб аллокатор винде не угодил?
там еще не один этот объект, еще колонны и телепорты. они тоже влияют.
https://gist.github.com/1791267
Thanks for the explanation. I didn't get most of it, but it seems useful. ;-)
"Thanks for the explanation. I didn't get most of it, but it seems useful. ;-)"

:-D

It is. ;)
I did:

$ python setup.py install --root=/home/share/software/

and then both (in sfepy source directory):

$ ./simple.py examples/diffusion/poisson.py # running local
$ simple.py examples/diffusion/poisson.py # running installed

and (outside sfepy source directory):

simple.py /home/share/software/usr/share/sfepy/examples/diffusion/poisson.py

work. 

So to fix your issue I would like to know more:

- you use current git version, right?
- how did you install sfepy? The command you used looks like a run from the sfepy directory...
- if you look into sfepy/__init__.py, there is "in_source_tree" flag and "data_dir" variable - could you print them and re-run the failing code?
Other option to set size, we can use properties. So, user can set the size
Hm, on second sight, that's not a a good idea either, because you can want an interactive session even when there is a command.
@DarkArc that was for melee, not projectiles =P
Wewt.

DriveWithJoystick probably shouldn't have this much responsibility, but competition is too close to worry about cleanliness! :(
Opsss :) OK I thought you forgot.
Whatever you did worked. I no longer get the error.

Thanks,
Christian.
OH YEAH :UNICORN:
It's not ready for merge yet - it's actually quite different from your version, so let me work on it a bit more and then we can decide if we want to override your stuff. Is that ok?
then update duh
Hello world *
Sounds *familiar* to me. ;)
https://github.com/jezhiggins/android/commit/60e9ec685a4553bec3958aad22ef5cd05c638e5d
I would argue that for level 2, this is good enough. If there is something, smash it, otherwise, move along.
It gets more tricky when the captives get introduced later and you have to know what occupies the tile.
Now it's true that enemy? does not get introduced anywhere else, but it seems to be equivalent to !empty? and captive?

Anyway, looks mostly like a matter of taste :)
I'll keep my empty? implementation for now on my side.
I keep getting the "You already have a sword" message. (but I have none)
`*dr` must be escaped
Not needed default timeout is 1800
I think initializing `Event` instance giving `Logger` is better because `Event` does not require other parameters of `@@configuration`.
If you like to keep `@@configuration` private, do not catch error here, and catch them in `tick`.
In a few places, there are nested if statements that all try to add to a result.

It might be clearer in the code to return $result when no more things can be added to it (the above seems to be one)

That also lets you swap the code from being
if () {
 if () {
   if() {
   } else () {
   }
 }
}

to
if (!condition) {
   return $result;
}

if (!condition2) {
   return $result;
}
... etc

This tends to be a lot cleaner/more readable.
s/defult/default
users != developers ;)
Can we use `if @user.email.present?` instead of of `unless`, it reads nicer. :)
I'd say that the vardefs in the client's Sugar instance are amiss.  You can find them in <sugar_root>/modules/<Module Name>/vardefs.php.

vardefs are metadata that define the fields and relationships of a module.  The gem depends heavily upon that metadata to be correct for things to work properly.
problem is here 

C:\xxx\mangosR2-mangos-b48e1e6\src\bindings\scriptdev2\VC100\100ScriptDev2.vcxproj.filters : warning  : The Filter File "100ScriptDev2.vcxproj.filters" could not be read. Unload the project "100ScriptDev2.vcxproj", to load the file.
C:\xxx\mangosR2-mangos-b48e1e6\src\bindings\scriptdev2\VC100\100ScriptDev2.vcxproj.filters(1863,5): The Project File could not be loaded. The Start-Tag 'ClCompile' in Line 996 Position 6 doesent match with the Finish-Tag 'ItemGroup' . Line 1863, Position 5.
(Translated)
Лог конфы прочитай. У меня не падает и все нормально переключает. Пусть еще кто-нибудь из пользователей затестит.
TADAAAA!
I'm not sure this is the right place to fix this. In particular, op["[]"] should work on most non-table values which define an __index metamethod. Hence, fix it in tree's __index metamethod?
too late.. I couldn't ignore it ;-)
Oops. Watch out for this one...
Why not also zend_db_table_select?
Could be nice.
i keep typing 'contollers' instead of 'controllers' -- phooey!
Great :-)
I wish you a lot of success with your law&programming classes!
Don't hesitate to experiment more with AnkiDroid.
There are a lot of things to do: http://code.google.com/p/ankidroid/issues
If you want to try a small, easy task to perform, just ask me.
And thanks for using AnkiDroid :-)
Nicolas
Can you tell me what the errorMessage is that is being generated in this case? I'm curious where the ENOTFOUND is being generated. I'm guessing the message will read something like `"Failed to process the file '%s'. It doesn't exist in the workdir, in the HEAD not in the index".`

I'm curious to know more because I suspect this could be a change in behavior stemming from my attempt to ignore submodules (i.e. a submodule might be leaving the status iterators in an inconsistent state).
jaja :D   Hmm, pienso que a la mejor no estan muy claras estas instrucciones :S
Ahaha oui. Il faut régler ton éditeur pour qu'une tab = 4 spaces (soft indentation). C'est la "règle" dans le monde de l'open-source (en tout cas, sur Sf2)
Doesn't look like I'm getting my Internet back soon, so I'll get back to
that tomorrow. It's not master, so no need to worry. Meanwhile, if
necessary, travis-hub can be reverted to the previous version of
travis-core and redeployed to stagging.
Das ist schlecht... In deinem script waren recht viele compile errors drinne...
Hmm, I'm getting differences on 1.2.3
Glancing at the jquery code, I believe this is specifically for expando properties on DOM elements, not arbitrary objects. Either way, should run into this in IE testing if it doesn't work. Thx for the heads-up.
Part | Points
------------- | -------------
base | 1
*Total* | 1

Passes tests. :sparkles: 

Don't keep your test code in the final copy next time please, it slows down my workflow. It's good to have, but comment it out!
아.. 제가 로컬에서 띄울때도 postgres로 데이터 보면서 하거든요. 그래서 저렇게 해놨는데.. 흠.. develope까지는 올려놔도 괜찮지 않을까 싶네요.  

그나저나.. 이렇게 깔끔하게 환경 미화를 해주시다니.. 캬.. 좋아요.  

--  
KeesunBaik


On 2012년 2월 22일 Wednesday at 오후 9:02, YongKwon Park wrote:

> 괜찮을려나?
>  
> 개발 및 운영서버 설정파일은 깃헙으로 형상관리 안하고 서버에 두려고 일부러 안 올렸는데.. ㅎㅎ;;
>  
> maven profiles 설정에 보면 어떻게 하려고 했는지 보일게얌!
Thanks! You rule!
Because I'm an idiot?

Fixed.
FWIW, this document talks a bit about using : for mass commenting, but not the first line.  OS X definitely does understand it at the top of a file though - http://wiki.bash-hackers.org/scripting/basics
My mistake, I should have gotten used to it by now.
box is a list. you're calling it with ()
I'd rather you extract all the required properties here and leave the reference to configurationValues alone to save memory and improve readability
Apparently Github comments hide the angle brackets so I don't know if you will be able to make sense of the above comment
We had resize support before - please check how the timeline and the flash graph do this. The event & event handler might not be necessary.
This had me confused for a good half hour... derp.
@Ussmaan, why do you have to be the black sheep asking all the crappy questions?
Regarding "plan" and TODOs, I added milestones and created issues in GitHub Issues of each repo. I don't know, IMO the granularity it offers should suffice, what do you think?
Hi, Nik.

It's strange, but I still have that warning.
Unfortunately, I'm not an expert at all and therefore can't even imagine what's wrong.
All I see — the `authenticity_token` isn't sent in `params` hash.

The code is simple and i've double checked all the changes...
Maybe, I'll post my app here and You'll take a quick look at it, when it's possible?
This code used in two places at most, so it wouldn't take much time...
Not needed as you inherit this from the parent projects.
yeah, I haven't pushed the alt attributes and gettext, i'll leave the
template macros off for now because they are different enough that
refactoring them here is overkill.
There are few missing views & events that should be tracked:
/b2m/urls/open-prompt (when a question if the url should be opened is displayed)
events:
- [dialogs, cancel] (when user decided to cancel opening a url)
This breaks on certain browsers (I'd suspect on mobile it's pretty much everything that's non-Webkit), you need to check that console.log is available (it isn't on browsers like Firefox without Firebug being installed iirc), or just not log it at all.
I guess we can get rid of this input in favor of people manually writing their label + input for each checkbox. The code in the bootstrap app does this:

        f.label :draft, :class => 'checkbox' do
          f.input :draft, :wrapper => :inline_checkbox, :as => :boolean
        end

Which is very close to just using rails helpers.
Вот так работает
it could be null here so indexOf will raise an error! Please use model.get(q.key)?.indexOf
Huch da war noch ein Test Change dabei im Bezug auf die NPCs die durch das Gunship fallen. Kann aber drin bleiben. Habe nur die X3 Koordinate erhöht
wouldnt --key be a better name or --route
Definitely a necessary option. Good job.
you state requirement is php 5.3+ - don't `explode()`, use `microtime(true)` instead to get the time as a float.

http://us3.php.net/manual/en/function.microtime.php
Are you actually using jslib?
Removing the old SVN repo broke my buildout -- was that really worth it?
That's much better. You are really improving
alsa libs are NOT for android, they were made for the Ubuntu and other linux distros by Cotulla, using them in android just wastes kernel space memory and nand space.
LGTM.  Good to see the text-mode flag also dropping out, now that the protobuf readers and writers seem to be pretty well debugged.
yes, now the test config was not used by test files
奇数列のみtileHeightの半分だけ、y軸方向にずらして描画させる。
Even though the stacktrace says the file is knockout-2.0.0.rc3, I updated the content to the released 2.0.0 version and still got the error.
awesome, looks good
Does not  make a sense
http://rubygems.rubyforge.org/rubygems-update/Gem/Specification.html#method-i-add_runtime_dependency
bra mellan klass
Não sei porque colocamos o formtastic. Pra mim, deveria ficar só o simple_form mesmo.
This _is_ the GB branch...
It's `django-nonrel/mongodb-engine`
no, i cant, need all repo files, i cant do that :(
C'est pas bizarre ça ?
В одну. Проверяется нет ли у ноды зависимостей, которые еще не появились в арче.
input/parent? What do you think?
It's better to use forceStringNoCtx() here.
Yes, because it is called from within loadClass method which is synchronized. We have a full blown lock here.
no need for the extra level of nesting here any more

    if (actor == null) {
    } else if (isNormal) {
    } else {
If you can't have an endpoint without a processor, they shouldn't be separate vars.
I don't like that syntax at all.
It's not type safe, and therefore doesn't prevent silly misstakes or help in refactoring

    "foo" must be('defined)
    cluster.convergence must be('definod)
1. Not needed
 2. This must be public
 3. This must be helper method
:O I missed that, was left over from internal testing, fixed now .
OK, I see those are just aliases.  Looks like I had the same comment last time :)
hmm are you sure we don't want to return the list of songs queued?  `post "/freeform"` seems to do that. i guess that's how hubot shows the list that was queued.
We should have some sort of reference to criteria in our general conventions section (may not exist yet, but it needs to). The "generic search feature" should link into that.
PEP-8 says for boolean values to not use the '==' or 'is' operator, but to just say "if kwargs['summary']:"
I actually thought about it, but it's only ever thrown/used by a method in this class and the message is the only thing that's ever output. I had no intention of using that traceback. Granted, that doesn't help that the situation looks like I missed something, which is bad in and of itself. I should either just do it or comment as to why I didn't, I don't like that I had to explain that to you.
I've noticed that booleans are more dependent on the db implementation, and thought this is simple and foolproof.  Maybe even a smaller fixed size field, is there a portable way to say 'one byte' ?
spaties rond .
A concise description of EMRio's purpose and functionality would be nice up here. It helps keep people from clicking away in fear of having to read too much.

Something like:

> EMRio helps you save money on Elastic MapReduce by using your last two months of usage to estimate how many EC2 reserved instances you should buy for the next year.
Good. Using tests as a reference is always a great idea.
This is a tad verbose. I think people using this tool are familiar with the pricing model, and if they are not, this should be covered in the readme.

I would just say that this module returns a usage cost for a given set of jobflows, reserve instance pool, and pricing scheme (or something like that)
this line should not be removed.
Er, this now says "Exchange (Exchange)" :)  I'm guessing you either want "WebDAV (Exchange)" or "Exchange (WebDAV)" -- whichever makes more sense in your locale.  Thanks!
ok, I've made it work in 0f623a1

As a bonus, I think this syntax helps understand the way the tag works better.
That's the point of `instancetype`-- we know it'll be an `NSValue *`.
Yeah, I hear you. I'll usually try to match the convention of the API – so I might do more C-like things with Core Graphics or GCD, and more Objective-C-like things with AppKit. But the most important thing is just whatever makes our lives easiest.
Also remove the <p>, I think ScalaDoc deals with blank lines by itself
Should this be `goto fail` instead? The mixing of `goto fail` and direct returns here looks rather suspicious.
ну это не значит, что это хорошо. оттуда его тоже надо будет убрать при случае
How can I fix this? Another commit or a new pull request?
I see! I'll make the changes and rebase 

Sent from my phone

On 2012-10-25, at 10:58 PM, Francesco Rodríguez <notifications@github.com> wrote:

> In activerecord/test/cases/finder_test.rb:
> 
> > @@ -28,7 +28,8 @@ def test_find_with_string
> >    def test_exists
> >      assert Topic.exists?(1)
> >      assert Topic.exists?("1")
> > -    assert Topic.exists?(:author_name => "David")
> > +    assert Topic.exists?(:title => "The First Topic")
> > +    assert Topic.exists?(:heading => "The First Topic")
> We can't do a mass conversion to 1.9 hash syntax in activerecord because it will affect many pull requests. Instead, we encourage people to write the changes or new code in 1.9 syntax. I don't think it will be more difficult, we don't have any agressive conversion plan, but this will make the transition smoother. Thanks.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Can you add some examples?
my fault, there was a test case creating a `DynamicMatchers::Method` with `model == nil`, which failed when I introduced the changes. That nil is now replaced by a stub (see 5797b39).
And why removing this one?
I think these tests will break. The message is different. Can we assert all the message?
yes i do. i'll fix that!
Remove FIXME as it doesn't make sense.
```
Migrates the expression into another mirror, jumping into a different universe if necessary.

This means that all symbolic references to classes/objects/packages in the expression
will be re-resolved within the new mirror (typically using that mirror's classloader).
```
Cool, I was looking for something like that but didn't find it.
unpacking a table with a nil value is returning no value?
true, I am happy to rename it to quickplot or something. Don't know if anyone else has an opinion.
Does is_MatMul not imply is_Matrix?
These were not really meant for public consumption. They should be stored as class functions inside Permutation but since they can just be imported from Permutations. I left the af part on as a reminder that one must be working with array forms.
Yes, use single backticks, like \`T = 1/2 m v^2\`.  Note that you need to use LaTeX notation here.
Can you elaborate on what the 'list of conditions' is?
again, hash in the type. Probably safest to just reuse basic.hash.
I suppose another way would be to work over the algebraic field. We don't have a Poly equivalent for rational functions.  Probably it could have a nicer interface.  But I can tell you that if you are used to working with such things that `Poly(a - 1).invert(Poly(a**2 - 2))` is not too unintuitive.
You could write this more verbosely:

L = m * v

where m and v are the mass and velocity in N of particle P, respectively, and L is the linear momentum of P in N.

The reason I complain about this, is because only people who have had to type Kane's notation in LaTeX really no offhand what "^N v^P" may in fact stand for. I would suggest minimizing Kane specific notation to all docs except the one's in the Kane class. This will ultimately make it easier for the average dynamics user to understand what is going on.
I think i have problems with my "gjslint  --strict" and "fixjsstyle --strict" because non of them warned about or corrected this rule.
i've run gjslint and fixjsstyle not to kill anybody :/
Defaults to "HTTP/1.0".
please close the bufferedreader when you're done with it.
Yes intentional. Previously these would break when run inside the `test` directory because it was no longer in your path (when it shouldn't matter). So non-relative (with 'test') is testing that you're running the test from `test/..`. We do have an absolute test, and `@relative` is still discovering `test_suite_subdir` like it would discover `test`, it's just relative to the test file now instead of your path.
I've added an inline link to the API documentation of the Symfony Serializer component.
I would use ``callable`` instead of ``mixed`` to be more accurate
KDE users don’t need to install anything :)
Doesn't seem to be the case, the older version used a NormalSummary still, not a Multivariate one.
Should it be `supplying` here?
I would prefer using `||` rather than `or`, http://craigrandall.net/archives/2006/09/ruby-coding-stds-etc/
Personally I think there is only so much we can support. Stupidity or incompetence should not be supported.

jQuery for instance doesn't support environments having extended `Object.prototype`. And test suites are generally more controlled than regular production environments where there is much more third-party code and mashups going on. 

However if we do want to support (older) ES3 based environments where a global variable `undefined` can be defined, we could simply add it to the QUnit closure (like we do in jQuery). That way the support is explicit, as opposed to implying it by avoiding `undefined`.
Can you abstract this if statement and the contents into some JS function, to avoid repeating the code. Just call it checkPassword(event) or something like that (don't worry about the event variable), then you can probably call it like:  $('#agreement_checkbox').on('click', checkPassword)
You didn't want to make this a VerifiedMigrationStep? Looks like a big change...
Order zones by `Zone.Order`
do we care about the extra space at the end?
NIT: I usually like to line up attributes if I make a tag multi-line.
I see what you mean. I'm going to have an API call on /people page and one /events page. So far we're not planning any other front-end to back-end interaction through API so I prefer to keep JS frameworks out of the picture for now. I'll definitely revisit this decision if we end up with more js.

thanks!
I was thinking about that exactly. I'll go ahead and do it.
Yes. Look at diff on github to find all similar code that shouldn't be here.
A tab was removed.
Does `$this->unlimited_reading_forums` have to be a property?
} on the next line.
@igorw This is this only way I could find to get new objects and load their dependencies automatically (copied from ContainerBuilder). If you have better ideas I'd love to hear them.
Please add a comment explaining what it does and I will research it.
any reason why it shouldnt be?
It would be good if this logic had comments explaining what's going on.
I just copied how it was already done.
The path can be stored in a class property in `setUp()`, it is the same everywhere so doesn't really make sense to redefine it every time.
Topic, probably.
if this is validating input, we should bail if the input isn't valid
Please:
- move import random on the first line (import order)
- leave two spaces between lines 19 and 20
If ``DARNER_VERSION`` is just a basic C string, the proprocessor (iirc) should merge all that into a single string.
Small point:  perhaps Y.JSON.stringify(urlParams) just to be safe.
again, use `in` instead of `has_key` everywhere
а разве 
> хранить в базе сумму за предыдущие дни

не равно 
>2 записи для показа

?
I think so, qt_[add|remove]Object can be called from different threads. If in doubt talk to Milian, he made the original code work with multi-threaded applications.
The formatting used in the rest of the code is to put the braces in the same line as the keyword: try {, catch (...) {
WHAT IS IT?!
So if github goes down, nobody would be able to build packages because of this ugly hack?
This list will have a queue of all patients who have PRESCRIPTION encountertype -- however, we want to make sure the patients who have had their prescriptions filled don't appear here. 

So, we need to POST another list for PRESCRIPTION FILL with the correct start/end dates, get the <uuid> for that list, and use notInList=<uuid> in the call right here.

That way, we can get the queue for patients who have had PRESCRIPTION but not PRESCRIPTIONFILL
Is the word strict still desirable?
Have these been pushed over to the puma's table (http://puma.nerc.ac.uk/STASH_to_CF/STASH_to_CF.html)? Is there a plan to do such a thing? If so, can we foresee the end of this dictionary?
This new behaviour renders the "native coordinate system" reference in the method docstring out of date - so that needs updating.

But that also raises the question, what code is using get_xy_grids (directly or indirectly) that's relying on the previous behaviour? Or needs to be extended to cope with the new behaviour?
said -> described

in -> on
I can't say for sure about all possible sets of axes. If people implement more axes sets, they should benchmark those and possibly make the optimization smarter.

I used the script bench_softmax_gpu.py that I sent to the mailing list recently. It uses the case of the max op that I implement here to do softmax operations. (The numerically stable way of doing softmax is to subtract out the max element of each row before doing the exponentiation) The softmax graphs used in this benchmark all get faster when this pull request is applied. In some cases the speedup is very modest, as in barely > 1X.  This benchmark uses a wide variety of input sizes so I am confident that this pull request makes theano faster across the board.

You are probably right that reduction is faster on CPU, even in the cases I am benchmarking. I think the main reason this pull request results in a speedup is that it yields fewer HostFromGpu and GpuFromHost operations.
However this should never be undefined unless there is a programmer error, since `tag` is required, and so this will always bind to the `tag` parameter passed to removeTag. So no need for `var` here.
use dedicated function for calculating layers/columns data
Field declarations above Meta would be consistent with what we do in database Models.
``if name.strip()`` ?
Code standards: should be `if ($Zip->locateName($search, ZIPARCHIVE::FL_NODIR) === false) {`
I agree. It should never get called given the rest of the code so there's not much point in it being there.
it should be ``null !== `` instead of ``!is_null()``
OK,I have just made it a non-optional argument.

2012/9/13 Lars Butler <notifications@github.com>

> In openquake/db/models.py:
>
> > +        match = re.match(r'^SA\(([^)]+?)\)$', imt)
> > +        sa_period = float(match.group(1))
> > +        sa_damping = DEFAULT_SA_DAMPING
> > +        hc_im_type = 'SA'  # don't include the period
> > +    else:
> > +        hc_im_type = imt
> > +    return hc_im_type, sa_period, sa_damping
> > +
> > +
> > +class HazardCurveManager(djm.Manager):
> > +    """
> > +    Manager class to filter and create HazardCurve objects
> > +    """
> > +
> > +    def create_aggregate_curve(self, output, imt,
> > +                               statistics="mean", quantile=None):
>
> Why is 'mean' the default? In my opinion, there shouldn't be a default at
> all. If you're creating aggregate curves, you should be specifying exactly
> what type, always. The name create_aggregate_curve itself does not
> explicitly have anything to with mean curves. It's a bit confusing.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gem/oq-engine/pull/874/files#r1598951>.
>
>
I'm okay with it as is, my suggestion was just to always test 100 + unit and skip the iterative loop, not 100% sure of how this impacts issue, and it seems gibson/mike have done more research on it.  If they say the way it's done is good, I'm inclined to agree
style nits: `window.scrollTo( 1, 1 );`
I'm going to start sounding like a broken record...

`jQuery.Animation.preFilter = function(...)`

`jQuery.Animation. tweener = function(...)`
Okay, we really need to resolve this and land something; we're already behind schedule.

Regardless of the whether the entry point is in ajax or manipulation, we have to expose and document an entry point here. I am not sure how an overridable hook is better for alternate ajax impls since they'll need to now punch in the hook. Plus, the stub will be left in manipulation.js. To me it just doesn't make sense to leave a reference to `$.ajax` inside manipulation and make code punch it out.

The method here is doing what it says on the tin, it's running scripts synchronously. The choice to avoid global events also makes sense, there is no asynchrony. So although it has limited utility and we wouldn't advise end-users to avail themselves of it, an ajax implementor needs to provide code for this case.

Any other people have opinions here?
`.selector` is going away in 1.9, right?
Yes, always private explicitly unless you have a compelling reason 
otherwise.

On 3/28/12 2:46 PM, ferhanture wrote:
>> +	}
>> +
>> +	private static int printUsage() {
>> +		sLogger.info("usage: [bitext-input-path] [filtered-output-path] [e-dir] [f-dir] [vocab-dir] [e-lang] [f-lang] [classifier] [classifier-threshold] [classifier-idOfPositiveClass]");
>> +		ToolRunner.printGenericCommandUsage(System.out);
>> +		return -1;
>> +	}
>> +
>> +	//	Map: (eSent, fSent) -->  (eSent, fSent)
>> +	//	Ê Ê Ê(vect1, vect2) = convert sentences into tf-idf vectors
>> +	//	Ê Ê Êcompute features for vector pair
>> +	//		 emit pair if confidence met
>> +	private static class MyMapper extends MapReduceBase implements
>> +	Mapper<LongWritable, Text, Text, Text>  {
>> +
>> +		PreprocessHelper helper;
>
> this applies to other variables as well, right? I don't directly reference any of them, so better keep them private?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lintool/Ivory/pull/25/files#r616986
>
What if other state saw no rows?  => div by zero
I'm always on mvim.

```
set listchars=tab:>-,trail:-
```
Needed for calling delete commands on Windows.
this won't work. because it's at module-level, this code gets run once when the module is imported, and the value will stay constant from then on. It will only change when the server is restarted. Have to put the call to `datetime.now` inside a function if you want it to actually change as the date changes.
This seems to be an unnecessary change.
HOMEBREW_PREFIX is what you're looking for, but what you really want is something like `HOMEBREW_PREFIX/'share/mailman'`, we don't want an e.g. /usr/local/mailman directory.
Pointless test, either make it test the software or just remove it.
Unfortunately postgres doesn't like $ or caps :( so leaving it as is.
This certainly sounds like a 403.. 405 may also be appropriate for read-only contacts.
Regardless, carddav clients suck and you should probably check at least OS X and iOS addressbooks to see how they behave in either case..
Да, тут и без дополнительных уточнений очевидно, что имелось в виду.

Я считаю, исходный вариант вовсе недопустим. Исправление может означать и готовый набор изменений, и процесс создания набора изменений. В первый раз «исправление», очевидно, употребляется именно в значении происходящего во времени процесса. В дальнейшем местоимение употребляется в неподходящем контексте (ясно же, что речь не о тестировании процесса исправления ошибки).

Словосочетание «тестирование ветки» часто употребляется при работе по схеме «одна задача — одна ветка», так что текущий вариант, на мой взгляд, допустим.

Предлагаю написать так:
> После тестирования ветки с исправлением сольёте её обратно и отправите в продакшн.

Отдельный вопрос — допустимо ли использование сленгового «продакшн»?
Use the Date.now() method when getting the current date in unixtime, since it saves a typecast.
ahh ok. muss die methode dann public sein?
Can you please refactor this to a more specific name, like `c.wiki_page`? Also, what if I wanted to use this validator to check an input wiki page name, rather than view a page? Can you move the initialization of `c` state values somewhere more specific to their use, perhaps a controller `pre`?
We use backports gem wich adds this method
Thinking about it, this if is unnecessary. I can just shift either way.
我这些东西就是写给你这样的人看的，什么java和gtk你折腾它们干啥呀，要装rJava包的话，用apt-get install r-cran-rjava，要装RGtk2的话就装r-cran-rgtk2，等等；Debian下的R维护者Dirk大人早已经把这些事情搞定了，不需要程序猿抓耳挠腮去折腾那些依赖。
One request though: assign-and-return in the same line gives me the jeebies.  Can we split it, please?
27 seems to be the header height and should be defined in a constant. Same comment for the FiF panel.
Hmm.  I think that code is sort of unintentionally working around a flaw in EditorManager.  I'd rather fix EditorManager directly.  But I think we could merge this initial pull request without fixing it, since Esc still works to close the inline editor (and closing with Ctrl+E behaves a little oddly even for regular inline editors, so this wouldn't be _that_ much worse).

So I think we should still remove the _colorPicker code, and then spin off a bug about closing with Ctrl+E.
Both true and false cases should be set here:

    this.closeOnSelect = (completion.match(/\/$/) === null);
JSLint nit: must put spaces around the `-`
That's exactly why I changed it to "on". It is not responding to a JS event in the code, but it is being called for the logical event, and could be hooked up to a JS event at some point.
After rearranging code above, this block can be moved into ``"tokenType === HTMLUtils.ATTR_VALUE"`` block above.
The `#555` color looks great on displayPath, but is too light for displayName (it's actually lighter than the non-matching text).  For displayName we should use a different CSS class that leaves the text black.
according to the security guidelines that we have from jan's presentation we should not use innerHTML
Noted ;).
Looks like you just copied and pasted from summary.html. Could instead reorganize the code (possibly using an {% include %} so that shared code is actually shared?
should this raise not implemented?
Yes. I can also just pass elTypes.shift() as an argument, instead.
This should go in a new file in auslib/util/
And, to make lint happy, I've just done this: https://github.com/mozilla/butter/blob/master/src/cornfield/module.js#L9
Reverse these quotes or escape `"``
mixin please
This is a way better style for leaving crap in. Worth more if it has a bug number with it, as in //todo (see #xxx)
Ah, okay. That makes sense. Will move the variables up and space them as in that example. Stay tuned...