labofsahil/github-event-dataset-2012 huggingface dataset:
Coding standards

Hi Aditya,

Wow, fantastic job on this game! Really amazing how close it gets to 'the real thing' ;-) . Brings back some good memories... Thanks for that! All we need now is an EVA installer ;-)

I was reading through your blog where you stated you would be interested in turning this more into a 'community project'. I'ld love the idea of that. So, as a start, i was looking through the source and had some ideas. 

First of all, i think as a team project there should be some sort of 'coding standard'. Jslint is pretty strict so imho a good starting point. This commit gets the source to comply to this coding standard. (You've made a couple of commits since i checked out and started coding, but i guess it's pretty straightforward for you to re-implement these features, if you like the idea...) A couple of things i fixed along the way:

- Moved 'var' declarations to the top of the functions and remove doubles and added missing ones.
- Fixed indenting, mixed tabs and spaces (that pretty much messed up the 'diff', sorry for that)
- Bugfix in function 'preloadImage' (typ0)
- Added a few todo's , e.g.
   -  'Infantery.draw' (function drawSprite doesn't exist?)
   - Bugfix in function 'vehicles -> moveTo -> processOrders' (var collDirection doesn't exist?)
- And a few more minor cleanups, no big things...

While scrolling through the code, i had some ideas too i'ld like to share with you:
- What about implementing a nice debugger-console-thingy. This way we can uncomment all alerts() / console.logs(), give them a priority and disable / enable them with a single flag! 
- Splitting cnc.js into modules would be nice and give some structure to the code. How about settings up a simple js-loading-mechanism en setup a build process to minify and build releases? Or do you have such a script already? Some node.js script or something?
- And while we're at it, we make it Google Closure compatible to squeeze the last bit of speed outof the browser ;-)
- I did not have enough time yet to look into this, but i have feeling there's pretty much 'code duplication' that could be removed... Are the any quick wins in that, that would make the code more structured too?
- And how about the copyright issues? Any thoughts on that?

Anyway, just interested in your view on this. I love the project and would like to help this to the next step. Can't wait to get the original music back, and how about an online websockets based C&C MMO :-))) Thanks again for your efforts!

Tim
On django 1.3.1 custom_admin_media_prefix tag return function (as callab...

...le object):

<link rel="stylesheet" type="text/css" href="<function admin_media_prefix at 0x8a5d3e4>css/changelists.css" />
Underscore required

I added an underscore

I hope this pull request is correct
Fixed azure related documentation issues.
Fixes unbreaking this extension

No changes to phpbb were needed.
Cast not needed -- the DAO should take care of casting.
Again, I'd prefer this to be a view function.
Same here, maybe you can conside return "False" instead string "false"
This should have been done inside CVaapi::open
Ideally this bytesperpixel stuff should be a query added for the renderer. We'd still need to ifdef there for gl_bgra, gl_bgra_ext and etc.. but at least we could consolidate it in one place. Also, it would be smarter there as we could check for extensions. so it wouldn't be gl vs gles, but has_bgr vs !has_bgr.

Will do up a pr when I get a chance.
On 27.02.2012 15:01, Krzysztof Kulesza wrote:
> Po zmianach z nazwą z beautify_bankaccount na format_bankaccount brakuje modułu lib/Smarty/plugins/function.format_bankaccount.php
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/lmsgit/lms/commit/ce397222979994dc32607562a6fd44a415d34b4e#commitcomment-1014980
> 
W Smarty 3 można funkcje wywoływać bezpośrednio nie trzeba wrapperów
tworzyć.
Must have forgotten about this a long time ago!
THANK YOU, just thank you
The CMDLINE NSTextField is not controlled by the View.xib file... I was intend to use interface builder to test the styling and forgot to remove that xib file...

The actual code that control the focus ring is in XGlobal.h, the `#define CMDLINE_HAS_FOCUSRING`, I made a quick commit and now its disabled by default...

Please help me check if my commit patches the issue.
Skvele, dalsi riadok z bootstrapu moze ist prec:

    Debugger::$email = $container->parameters['application']['webmaster'];

Ale stale este jedna vec k absolutnej dokonalosti tomu chyba. Ak implicitne nastavim v configu productionMode:

    parameters:
        productionMode: true

Configurator by mal byt vygenerovany tak, ze v initialize() by sa mal prestavit Debugger aj Configurator. Zatial teda musim rucne to poriesit v bootstrape:

    # Update Debugger and Configurator with local parameters from container
    $productionMode = $container->parameters['productionMode'];
    $configurator->setProductionMode($productionMode);
    Debugger::$productionMode = $productionMode;
"cache" is something that you configure yourself, so it shouldn't be added. The others are ok.
I updated it, but still having problems. Based on below, it looks like a DNS configuration issue. @friflaj ?

    ~/src/backlogs.github.com (gh-pages) $ curl -D- redminebacklogs.net
    HTTP/1.1 302 Moved Temporarily
    Server: nginx
    Date: Fri, 09 Mar 2012 21:29:22 GMT
    Content-Type: text/html
    Connection: keep-alive
    Content-Length: 154
    Location: http://www.redminebacklogs.net/

    <html>
    <head><title>302 Found</title></head>
    <body bgcolor="white">
    <center><h1>302 Found</h1></center>
    <hr><center>nginx</center>
    </body>
    </html>

    ~/src/backlogs.github.com (gh-pages) $ ping www.redminebacklogs.net
    PING backlogs.github.com (207.97.227.245): 56 data bytes
    64 bytes from 207.97.227.245: icmp_seq=0 ttl=48 time=981.790 ms
    ^C
    --- backlogs.github.com ping statistics ---
    2 packets transmitted, 1 packets received, 50.0% packet loss
    round-trip min/avg/max/stddev = 981.790/981.790/981.790/0.000 ms

    ~/src/backlogs.github.com (gh-pages) $ ping redminebacklogs.net
    PING redminebacklogs.net (38.101.213.227): 56 data bytes
    Request timeout for icmp_seq 0
    ^C
    --- redminebacklogs.net ping statistics ---
    2 packets transmitted, 0 packets received, 100.0% packet loss

    ~/src/backlogs.github.com (gh-pages) $ host 207.97.227.245
    245.227.97.207.in-addr.arpa domain name pointer pages.github.com.

    ~/src/backlogs.github.com (gh-pages) $ host 38.101.213.227
    227.213.101.38.in-addr.arpa is an alias for 227.192-26.213.101.38.in-addr.arpa.
    227.192-26.213.101.38.in-addr.arpa domain name pointer notset.namecheap.com.
Probably not.

Using -p, however, causes it to error out on line 235 on 1.9.
Please update the Todo for translators- otherwise they will not translate the changes in HELP files or MSG.
Also the description "FAI finish height fixed for tasks shorter than 100km. " is going to be used as-is in changes.
Is it enough?
Is this workaround still needed?
`slicer serve tutorial_04.ini`

slicer expects the config file it seems.
Hmm well I hate to see the repeated code so I'd prefer to consolidate it, despite the 2-3% performance issue.

The other simplification would be to convert:

```javascript
		while(obj = objects[idx++]){
			type = obj.type;
			if (~type.indexOf('/') && !(type in seen)){
				seen[type] = true;
				mids.push(type);
			}
			var mIdx = 0, mixin;
			while(mixin = obj.mixins[mIdx++]){
				if (~mixin.indexOf('/') && !(mixin in seen)){
					seen[mixin] = true;
					mids.push(mixin);
				}
			};


into

```javascript
		while(obj = objects[idx++]){
			var types = [obj.type].concat(obj.mixins), tIdx = 0, type;
			while(type = types[tIdx++]){
				if (~ type.indexOf('/') && !(type in seen)){
					seen[type] = true;
					mids.push(type);
				}
			};


I guess keeping mixins and the primary type in separate lists is a mistake.   Probably if obj.type was a string like "dijit/form/Button,dijit/form/Mixin1" it would simplify the code overall a bit further?
TODO : サブプロジェクトにしよう
yes, it's exactly the same. I copy it. I forgot to delete this in my commit.
Wollte mal ausprobieren, wie das mit Kommentaren aussieht.
Does port.cpp need to include update.h? I don't think we'll be registering update callbacks here, right?
(08:29:08 PM) tomeofgreen: made a simple change : https://gist.github.com/1936760
(08:30:25 PM) tomeofgreen: Than got rid of the error, but not it just spits back the *node* without my pushes fromt he dolist. 
(08:30:59 PM) zokosiman: <HTML>You're not defining newNode as anything, I Think.
(08:31:12 PM) zokosiman: <HTML>you just have it available int he let( ((newNode))
(08:31:19 PM) zokosiman: <HTML>shouldn't you end up doing like
(08:31:49 PM) zokosiman: <HTML>(setq newNode (list (gensym "garden") (format nil…..)) (push newNode *nodes*))))
(08:32:43 PM) tomeofgreen: I think you on the do with that
(08:32:54 PM) ***tomeofgreen has not seen setq only setf
(08:32:58 PM) zokosiman: <HTML>oh
(08:32:59 PM) zokosiman: <HTML>setf
(08:32:59 PM) zokosiman: <HTML>sure
(08:33:02 PM) tomeofgreen: lol
(08:33:06 PM) tomeofgreen: cool
(08:33:30 PM) tomeofgreen: I'll try that out and see what I can find, thanks for the input dude.
(08:33:36 PM) zokosiman: <HTML>npbro
(08:33:47 PM) ***tomeofgreen is going on a redbull and sushi run
And the Jaguars do oscillate when trying to reach a 0-setpoint.  It happened last year as well.  This year when we did our tuning we'd type 1000 into the SmartDashboard, watch it get there, then type '0' to stop it but it would then oscillate about 0.  After that we'd quickly hit "disable" in the driver station, and that's what actually stops the motor.
btw: I'm wondering why you only see it with this file...
fpr the issue, you can try fix #issue in commit but not unfortunatly.
It's used in calculating distance. tan(theta/2) = (FOV / 2) / distance
Are any of these lines different?  Is there a "right" way to merge these xml conflicts?
Ser det bra ut @oleander ? :)
@Th0ar 这个统一一下~
I'm almost certain this didn't need to be a merge commit.

## Josh's PROTIPS for avoiding merge commits:

* git pull --rebase remote branch
* git pull --ff-only remote branch
* git checkout -b new-branch && git pull --rebase remote branch && git checkout master && git rebase new-branch

Remember, it's rare that you need a full-on merge, and rebases and fast-forwards leave the cleanest revision histories!
Make sure you credit et65 as well. http://pastebin.com/Y22qwdtZ
I guess you could use a private repository… That way, you wouldn’t have to worry about anything. I don’t know of a better solution though.
Yyour favourite accessor naming convention here ;-)
But that is only for sorting. I want to use the qsort method which sorts a SGVector
still not allowed?
nothing wrong on it, if its disabled by default :)
I have just tested it, and right that setting SetClientControl(m_pBase, 0) works better :D You cant control the vehicle as passenger, and can leave it directly too, not just through driver. Still cant cast tho, as passenger.
I'm wondering if we shouldn't  put the two inputs together;  have a smart input that allows the user to toggle between a rich text editor (for quick, "regular" use) and a text area that takes HTML (for advanced use).  @JonFerraiolo, what do you think?
we don't have a mixin for this?
Why the `.column:last-child, .columns:last-child { float: right; }` in grid.css? 

It breaks a simple layout like this:

    <div class="row">
        <div class="two columns">…</div>
        <div class="three columns">…</div>
    </div>

The three columns div floats to the right and leaves a seven columns empty space between both divs.
Can you add a detection for >= 1.0.14 in FindOFED, please?
Oki.

We will have a 1.0.1 version soon I think, with a GLPI module update in the
next week I think/hope :)


Jean

On Wed, Feb 29, 2012 at 10:55 AM, Gerhard Lausser <
reply@reply.github.com
> wrote:

> The contact_groups were delivered as a string, not as a list. So Multisite
> treated the string like a list:
> ![contact_groups](
> http://picpaste.com/pics/contact_groups-NaxZDEyD.1330509208.jpg)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/f7af7c45ac29648badcde93bcf9349408ee2e445#commitcomment-1025306
>
oh yes, can be useful :)

On Fri, Mar 9, 2012 at 10:19 AM, Andreas Karfusehr <
reply@reply.github.com
> wrote:

> ok emacs ist hardcore ... http://validator.w3.org/ is also a useful tool
> to debug html code
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/131b678e799e23dca28b5a11ff6f6bef93f7b7fa#commitcomment-1064844
>
インターフェイスを実装する形にして、メイン画面がインスタンスを複数保持する形にすれば、
ボールやアイテムの操作が楽に出来るかも。
Be careful with IGC files: IGC files in real life contains *lots* of errors of all sorts, and it's much better to simply ignore invalid records than to abort the file.

IGC parsing is also much easier with regular expressions. See, for example:
  https://github.com/twpayne/igc2kmz/blob/master/igc2kmz/igc.py

Regards,
Tom
Sure installers will either install dependencies (windows) or express them (linux ).

I have to test this under ubuntu. But it's quite sure if should configure the build first (and use a real build process by the way)
Well, Fortuna is only defined for 128-bit cyphers...for any larger size it'd need to be re-engineered.
caution when using non ascii characters in code (EVEN IN PYTHON COMMENTS !) the ` character made the daemon.py crash
тут можешь добавить before_filter и в него передать имя метода который он будет вызывать при каждом обращении к контроллеру, в методе просто вставишь @product = Product.find(params[:id]) и удалишь эту строчку из всех экшенов, так как она будет вызываться в бефор фильтре, например так before_filter :loader, :except => [:index, :create] тут еще указываешь где загружать этот метод не нужно, ну и сам метод loader в самом низу контроллера
c5ea31dd2b3c92798f1ccb6e9672ce50a9e255d7
I had problems with my gems of Rails 3.2. Then, I "blocked" the <code>activesupport</code> version to solve my problem in up to date this project. But, I guess that works fine in all <code>activesupport</code> versions.
Yeap, I figured it out again yesterday ;) Now almost all works as before ... almost ;)
r+, this isn't cheesy, it's clear and simple ;-)
@koshke - It now passes R CMD check! Thank you! You are a lifesaver!
oops, sorry. Won't happen again. Shall I move it or is it good?

This was very useful for workflow when combining things in flatiron.
Thanks for the credit! Awesome btw. I crashed shortly after I tweeted about it.
Odd variable names. ```value``` is an attribute, ```new_value``` is a method. Especially when you declared your ```sample_property``` class with first argument named as ```method```, it is good to pass this argument using variable with the same name.
Good question. The category names are based on the categories, and the pk we're getting is from the forms which are based on the categories. If the form is tampered with, however, it could break.
Isn't it supposed to be `-L` if not (see comment #2)
you should just use `device.device.name`
Another general comment: there's no transaction right? So what happens when two users read the same query and update the same time. Please provide the semantic. In the long run, we'll have to address this issue.

It's fine to leave a TODO(twoask) there.
Yes, indeed. I needs it to be backwards compatible with some older code. 
I realize it's  double as _handle  also does a statSync but '_handle'  should never throw.
still not right :) I don't think you want to add 0.5 to the freq before you divide it. 

CVGetHostClockFrequency() * 0.001 + 0.5
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.
Please only throw Error objects.
you changed the logic here during the reformatting. Is it intended ?
shouldn't this be only single backslashes? so the regexp is /[ \.\*]/
haha, yeah np, you translated pretty good, but the phrase was all messed up
If I want to browse the documentation locally is it working  ?
Wooo hoo!!! You added the multicraft code!
 I was reviewing it all, it seems like your adding chunks to it! (Inspiration from dscraft?)
what's the purpose of this ?
@skowyong thanks a lot!
On Sun, Jan 15, 2012 at 5:16 PM, imsoftware
<reply@reply.github.com>
wrote:
> Thank you.
> ... i don't know what you mean with the storage part, but i only want to use it for simple authentication for me and some friends, eg. @
> http://stackexchange.com/users/login?returnurl=%2f or http://www.livejournal.com/identity/login.bml or Drupal
> Is there the storage part required?
No, don't worry. The storage part is the system enabling third parties
to tell the openid provider additional and non-standard attributes.
This would be awesome to use, but sadly no service available at the
moment does this (and no reader uses it).
> Now i've tested the old version and i wasn't able to run it ... arrr... i've to try it again...
The old code is all but nice. there are some problems with the
templates, some of them are solved in my version in some early
commits... You should try mine. the buggy part is the storage part, so
I think it ought to work better than the old version if you don't use
the storage.

If you want some advice I'm available! :)

MMo


-- 
Marco Montanari
mail: marco.montanari@gmail.com
office: marco.montanari@labs.it
other: mmo@mmo.it
mobile: +39 3335267086
web: http://www.mmo.it
unnecessary check, please remove for consistency
There is some examples in [marketplace](https://github.com/KnpLabs/marketplace/tree/master/src/Marketplace/Provider/Controller) but it is not providers meant to be reused, just a way to organize things

@ubermuda built one culin with the old LazyApplication implementation which required a patch to be able to be reused several times. The new way using providers was (at least partially) an answer to his feature request but he has not updated the code after that to use the new cleaner way.
This guy should probably be `li3_queue\storage\Queue`. Having classes directly inside vendor namespaces is a violation of PSR-0.
Still getting some digit changing bugs on Heroku. Possibly related to messy digit code?
Thank you,

it´s working again!
Nothing much... Just hadn't anything better to do...
> I agree here, the multiple inheritance for testApp was surprising, I don't
> feel this is a good solution. A separate object would be better imHo.

Yes.

I was also surprised by the multiple inheritance in ofxMidi
Yes, it was Carton. @leto recommended it to us. If it is not as mature as I've imagined, we can shelve it for now, just like we did with Composer support in the PHP builder after we realized it has issues that affect pretty much everyone. Lets hear what @leto thinks about whether or not we should try to support Carton in the initial release.
sounds good to me. better than have 3 gemfiles, no?
Setting `active: true` triggers the event, so this seems to amount to a double-trigger of `removeUnread` in `channel_tab.js`
Remember to put bug numbers in the commit message so we can easily mark them as fixed (this lets QA know when it's available on the dev site)
Bien vu !  :)  Je l'avais aussi corrigé, mais sur une autre branche  ;)
I get an exception when I click any of the links in the `/system/` page.
Ooo, cool. I will take a look.

Sent from my Windows Phone - apologies for any typing mistakes.
________________________________
From: Dean McDonnell
Sent: 15/02/2012 11:01
To: Gary Ewan Park
Subject: Re: [VBForums-Viewer] - Temporarily corrected the bug by hitting the back button on the Welcome Screen.  Next to find a better way to do this. (f42e176)

I Gary, this problem was left over from Sendu and it was my first attempt a purging the backstack. I have since written a better method for this and popped in into the WinPhoneKit.Navigation project on my account.

---
Reply to this email directly or view it on GitHub:
https://github.com/gep13/VBForums-Viewer/commit/f42e17628493981a5b2f147264260a3108e7fec3#commitcomment-968876
That's correct. Hmm, how did this lockfile get checked in?
<3

Didn't want to show their AFK status though, eh?
Fixed derp here.
Wait what are you thinking? That will send "This player is currently ignoring all private messages." In the sender's language lol
Interesting link. Looked odd, but seems to work. It is a redirect though, right?
I agree long-term, but I don't think we should require new-style before all browsers implement it; that would cause lots of test failures for what is after all only a pretty small issue.
probably some cruft from when I first wrote this function. Rip the line out.
I was trying to back-port the state transtition error reporting for to
Rails 2 (which the README says is ~> 1.0)

On Sat, Feb 11, 2012 at 12:30 PM, Aaron Pfeifer
<reply@reply.github.com>
wrote:
> That's interesting, I wasn't aware of that.  I'll have to take a look at the Rails 2 integration and see what's up.  Anything you could point to in particular that's broken before I dive in?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gregwebs/state_machine/commit/a0842cdadf8a9a366edea9a7c68687059a3cf674#commitcomment-956967
св алкаш ебаный а димитро тощий пед...
floats are not the same as decimals. It woul be better to give the full name, "de ponto flutuante" (or "números de ponto flutuante")
Navegue online
I think "autovalores/autovetores" is more standard, although this nomenclature is also used.
Putting isInterface()  check to isInstantiable()  won't work.  isInstantiable()  already returns false for interface. Exception occurs because glueCodeClass.getSuperclass()  returns null. I may put several null checks instead checking  isInterface()  but this seems to be more intrusive refactoring.
Can you try again with the latest version? ;)
It will print a message before the error...
I know the menu system sucks, one of my next commits will be a rewrite of the menu engine (right now it's written repeatedly across multiple files)
Ahh, now I understand this better. This line makes sense for all git versions since 1.5.
Build 357.  Minor cleanup.
Removed from collabs, you're no longer wanted here rcp.
Awesome -- thanks Kyle, let me know when you want me to test it out.
Stop committing whole .pro files again... you don't have to commit all files if your line endings have changed >_<
Can we use `===` instead of `==`?
I'll actually be removing all CSS gradients as I'll be restyling the UI. Thanks for the heads up though!

btw, great work on the emulator.
http://www.boost.org/doc/libs/1_48_0/libs/ptr_container/doc/tutorial.html

Il faut qqs trucs... clone(), new_clone(), non_copyable..

Et ensuite le transfert entre containers:
another_zoo.transfer( another_zoo.end(), // insert before end 
                      zoo.begin(),       // insert this animal ...
                      zoo );             // from this container

// pour retourner un container entier, sans recopie du contenu:
std::auto_ptr< boost::ptr_deque<animal> > get_zoo()
{
    boost::ptr_deque<animal>  result;
    ...
    return result.release(); // give up ownership
}
...
boost::ptr_deque<animal> animals = get_zoo();
(BTW)

Do you know a way to have a method declaration like this?
How to make sure a hash is part of the first hash and not part of the second one?

```ruby
def method_with_multiple_hash_instanciation(hash1 = {}, hash2 = {})
  p "hash 1 : #{hash1.inspect}"
  p "hash 2 : #{hash2.inspect}"
end
```

```ruby
>> method_with_multiple_hash_instanciation :hash1 => {:content1 => "content"}, :hash2 => {:content2 => "content"}
"hash 1 : {:hash1=>{:content1=>\"content\"}, :hash2=>{:content2=>\"content\"}}"
"hash 2 : {}"
```

I think this is why, the password_test made it this way
coño si estamos 100% estables, solo os digo que en vez de usar el mangosdev para probar las cosas
useis el de radical
I think one of these should say "Elasticsearch".
фигня какая то...
Indeed, the could should be clean and easy to read... I will change it for the whole plugin (OPENGRAPH_NS_URI, opengraph_add_namespace, ...)
I pulled first...
Okay so I got all mixed up while trying to track the changes mantera and Whyzor had made to there kernel. I haven't update mine in a long time and needed some catching up, but I didnt realize that the changes when Whyzor merged with manteras branch I had already had these changes in my kernel. A big doh moment and I ended up changing back something I didn't intend on changing lol!
Better name for this, perhaps? sync.pair.tryagain.label?
there it is, i uploaded it to my fork instead of the main repo before.

El 16/02/12 12:11, lukasz karluk escribió:
> hi arturo,
> can't see this fix in the develop branch yet?
> is it going to come through as a new commit?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openframeworks/openFrameworks/commit/8bfde75beefc27e53c6cfb503db0035b0f828dfd#commitcomment-973373
You can get away with just 'tmp' and 'log'. One thing that's tripped me up a couple of times is adding to .gitignore -after- I'd added a file to the repository. A git command I've just started using is 'git ls-files' which shows which files are in the repository.
yippe!
Cool, thanks for cleaning this up
what exactly does minfile and check_filepages do?

as i crawled through various device repos (desire, ace, etc) i don't see this one set.
and why do u write " " (nothing) to check_filepages. shouldn't this be a bool (resp. 0 or 1)?

thanks
We should document the rationale for this option (also so I understand it: Isn't it usually enough to decide whether an action acts like a member, a collection, or new?).  We also need a test case.  From reading the code: could there be an exception when :nested_in and :additional_no_attribute are both set?  Lastly, it would be great if we found a better name for the option (if there is one).  :no_attribute_check already is horrible...
good call i will remove the rest of that patch.  LOL wow.
They're all working for me. All of the old versions are still in the bucket. I was just able to uninstall the SDK via PEAR, then re-install specific versions. Is there something more specific that's broken?
The problem is that you can write a passing test like this:

    def __unicode__(self):
        return "%s (foo)" % self.user

That's why I have all of those `get_random_<model>` functions in my tests.
Pessoalmente no tenho preferencia entre um ou outro, mas me parece que o
formtastic  mais usado atualmente *le parada para verificar rubygems* sim,
 mais usado. Acho interessante usar o que o pessoal da comunidade est
usando.
-- 
Thawan dos Santos Fidelis
Graduando em Cincia da Computao - UENF
Linux User #522067
@Thawan_Fidelis
well, the locale is optional when calling ``trans``. And you did it in ErrorAttrTypeExtension in the same commit :)
Please commit this. rails3_acts_as_paranoid is useless on rails 3.1+ - it gets into an infinite loop on any update without the above removed.
The documentation here seems out of sync with what the method actually does
_construct function is not necessary as it's used only once.
this class is not specific to the CAS protocol
Do we want an example here of par_loop syntax without matrices.  Something like:

op2.par_loop(user_kernel, set,
                             dat1(op2.IdentityMap, op2.READ),
                             dat2(set_map, op2.WRITE),
                             dat3(set_map(0), op2.READ),
                             global(op2.INC))

??
Or will that just confuse things?
Would break everything that this is used in, no?
Right, an FSM makes completely sense here.

> I’d also remove the `active` vs. `handleOption` duplication: just have  
> handleOrActive, where `None` means “active”

Oh yes, this is a remaining of a huge refactoring! Thank you, I will  
change.
http://www.assembla.com/spaces/akka/tickets/2228-handle-props-apply%5Bnothing%5D-as-props-empty
Thanks for clarifying. That is supposed to have the following comment
from Daniel Dyers code:

// This shouldn't happen as at least one the generators should be
// able to generate a seed.

Will fix to make sure it fits

On 2012/07/02 04:21 PM, Viktor Klang (√) wrote:
>> +    throw new IllegalStateException("All available seed generation strategies failed.")
> No, I'm saying that it will _always_ throw the IllegalStateException?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/akka/akka/pull/571/files#r1084499
We are using a blocking implementation of RCL. Also when/if we are going to support "reloading" I'd prefer to use a separate channel to do this.

The problem is that the RCL messages may be left waiting by the OrderedMemoryAwareThreadPoolExecutor for the users stuff to complete. And in the case of blocking RCL we "deadlock".

I like to be able to use idiomatic akka way for RCL communication therefore we have the separate system.

Alternatives? WDYT? Is this to much? Leave it be for the moment?
This is the old remoting, which should go away. This was just a short hack  
to make things work.
I could be mean and ask “why?”. But it was me who checked in this weird character combination.
Why will it only work for the first?
I assume that test conductor doesn't care about reuse. When all nodes have entered the barrier it is released and same name can be used again.
@rkuhn is that correct?
In `MultiNodeClusterSpec` this is named `cluster`
Got why the protected modifier.
Seems to be worthwhile doing.  It appears that there were a few dangling references to this class that weren't really valid anymore.
Nop, remove this please, we are not ready for 3. And all our build service / packaging / main computer don't have python2...
I'll take a look. I think I'll have to refactor this anyway as it's too slow.
This should be located alongside the other pulp.client imports above.
Even I wasn't thinking that :)

The render_* name is suggestive, no?
replace newCOntext.get with ctx ;-)
Damn intellij! ;)
Returns nothing has been removed so now if you try to find returns nothing, it returns nothing.
It works fine! I even wrote a test case for it, check it out.

Hadoop's version strings have thus far been two-digit after the first `.`. A first tenths-place release version of `1.10` seems reasonable to me.
This is mathematically wrong.
In every loop, foundArgument needs to be found in one of the columns, if it isn't the whole item gets removed, even if other arguments were found.
yeah, i wasn't totally happy about this, the alternative would be

```ruby
if (@result || (now > time_to_stop))
  @result ||= TIMED_OUT if (now > time_to_stop)
  return
end
```

I'm not sure if that's clearer
This should be something like +stepToClearTextFieldWithAccessibilityLabel:
This should actually 404.
Good to get rid of this code, I think.
> it may be better to collect changed rows and process them once editor finished it's part

That makes sense. Should I just emit an event after `this.$onMultiSelectExec` finishes? Or is there a better way ?
Here it fails to set mError to a suitable message. Check with the other plugins to make sure the translation can be re-used.
This will change IconButton to always act like onyx.ToggleButton.  This could break existing apps, especially if you're using IconButtons as part of an enyo.Group.  I would either add a "toggle" property that allows picking the behavior or derive a onyx.ToggleIconButton.
Ah you're right, I misread, we would need something like `lazy map + find: [:a,:b].lazy.map {|s| ... }.find {|r| r}` but lazy is not in 1.9 and `for/break` is fine.
It would be better if `!($node instanceof Twig_Node_Expression_Constant || $node instanceof Twig_Node_Expression_Array)` check was outside the loop, even if it's done at compile time.
Incorrect indentation for this whole block of code.
Why is this saved?
This can be removed. NAPTR support will be in v0.8.0.
will address use of Container in a later revision
I would prefer insert an empty line between final fields and non-final fields.
`$(this.hash.replace('.', '\\.'))` is `$article` as far as i remember
Should there be a plain `else { error("unsupported order") }` here? (either instead of or in addition to the `else if` for `NPY_KEEPORDER`)
I will put a not in the file.
френдов убрать
Комменты надо бы исправить
icon,QTreeWidget --> icon, QTreeWidget
Yeah, it sucks to use it, but if `super` doesn't work...
You can use a custom deprecator object that responds to +warn+ and +deprecated_method_warning+. This is useful when you use +ActiveSupport::Deprecation+ in your own libraries and want to customize deprecation behavior.
I think you can remove the `expand_hash_conditions_for_aggregates` method now.
you forgot to add your name. See the other entries for a reference.
You have a point there, it would be good to get rid of this.

However, the opposing constraint is that when there is no custom Hashing object, some objects produce poor hashcodes for typical object ranges - for example (n: Int).hashCode == n.
The problem with this is that the upper bits of the hash code are usually always 0. In both concurrent and immutable tries this can cause a slowdown due to several collision levels. In parallel hash tables it causes a similar problem with parallel hash table construction.

So, ideally, we want to mangle the bits when a default is provided, and otherwise not mangle them and hope the user knows what he's doing.
There is no `println` after the `pop`?
That's an interesting idea. In the example below (t5314-return-reset.scala), both `cpsR` and `cpsA` are `None`, though, so no error would be flagged. Should that case be OK?
Test case with `Option#getOrElse` added.
Are you sure this will work on Windows systems?  I don't know, just checking.
I think it's the most useful when you have different system and that you don't care to keep track of the other. For example, if you have a system with mass/length/time and another with bytes, you don't want to mix them (you can, but you will have some useless extra information). Also the dict form allow to work with dimension names naturally, and avoid to have to remember the order.
Also if you have only the number of dimensions that you need, the methods that gives information on the dimensions and so on will return less, which is better. For example the matrix of dimensions will be only a 1x1 in case of bytes, 3x3 for mks...
To my knowledge it's not.  It would be a useful addition though.  It would go in the ntheory module.  We may have such gcd functions, or not.  You'll have to search.  I think we usually just do `reduce(igcd, list, 0)`.  Is it possible to do it more efficiently than that?
>
> okay! one int, one rational, one irrational and one float .
>
> how about this ?
>
>
If you want to keep the rational test, it should be S(77)/3 (compare the
difference between that and 77/3 in a console).
Yeah, randomize the result of flatten().  I mistook this line for that in the diff, because I didn't look too closely.
You may as well want to create a namedtuple `UWild`, for example.  In fact, you are technically doing the same thing, but relying on a more standard mechanism is, uhm, more organic.  I suggest this change mostly because I don't feel at ease when I see arbitrary string constants glued on stuff in this way.  While I'm pretty confident that such style should be avoided, feel free to disregard my remark if nobody else vociferates about this.
This needs to be updated with the version that I posted on the SO page.
Should be an `assert`?
The dict object is not cachable by sympy, as we spoke about in the topic https://groups.google.com/forum/?fromgroups#!topic/sympy/ga4AyRyE9Zw
You're missing 2 closed parentheses.
"should have"
Ah, I see you changed the listener to be a regular event subscriber - nice, thanks.
What is that hex value doing there?
add:
BBFS::Params.init ARGV
The change from CategoryFunction to ScanAggregation seems to be a base theme of this patch. Could you just re-iterate what the motivation behind this is?
Replication lag is too large if:
 - the amount of lag is growing, not remaining constant
 - you are using a secondary for hot-failover, and the lag is greater than the amount of operations you're willing to lose in a failover
 - you are using a secondary for read scaling, and the lag is great enough that your data will be inconsistent
answer, yes.
While this is not technically wrong (agreed, you cannot use databases larger than 2GB) it implies that you are OK up to 2GB.  In fact, because MongoDB memory-maps all database files into contiguous regions of user-mode address space, you can run into trouble well below 2 GB of data.  It would be less misleading to replace "but cannot use databases larger than 2GB" with "but are limited to small databases".
what's with all the ' -> " replacements? ;)
One too many levels of indentation in this try / except block.
I believe that these `<span>`s are auto-generated by foundation. Are you sure that this should to be here?
This is a public repository.. so having the password and username for the website@klab.rw in the settings is probably not a good idea. :)
I'll pull this in to see how it affects the work on relevancy, but could you add a TODO to explain why the prefix and suffix arguments are useful once we get around to using them?

Usually it's best to only introduce extra complexity at the point that it's needed, but I know it would be a hassle to reshuffle this pull request.
https://github.com/phpbb/phpbb3/pull/608/files#r686275
Please check against other files and add new lines if necessary.
If $readable_forum_ids is listed as an array in the docblock, it should typehind and default to an array:
`function get_unlimited_reading_forums(array $unreadable_forum_ids = array())`
Looking at the mock, it seems that the result is an empty array, and if this is the case why was that chosen over e.g. false?
The format for  @ var annotations is:
```/*
* Description
* @var type
*/
```
includejs?
Never used in this function.
If it is the same class use self::
Hmm. I think some of the extra files and changes will disappear if master is merged in. I thought I might do this for you, but the conflicts are nontrivial. Maybe we can pair on this together Monday.
Can be removed until yage is at least compiled on Microsoft Windows platform.
This is the only line you use these macroses. Can you make test sources one styled, lets say use 0 and 1 here?
Agree but I am not a big fan of enhancements on the go. Everybody is free to open an issue for C++11 migration (which is preferable) but this should be approved before any code changes.
Can you name this `_event` so my small brain can understand what's going on? I only understand one syntax.
punting for same reason
но в этом случае нода будет делать 2 запроса к базе вместо одного. причем по факту нам в режиме реального времени нужна только суммарная статистика(для проверки лимита), а дневная статистика легко определяется из суммарной в конце дня.
поэтому я и хотел перенести создание статистики на крон, т.к. тогда вместо 288 insert...update в таблицу статистики для каждого баннера, будет только один insert на баннер от крона в конце дня.

т.е. вообще 2 варианта получается
 288/баннер insert...update в день к таблице статистики и высчитывание суммы кроном(еще куча селектов, и чем больше дней прошло тем их больше)
 288/баннер update'ов суммарной статистики в день и 1 раз на баннер в день высчитывание дневной статистики кроном

если ты хочешь в 1м варианте еще и держать сумму за предыдущие дни, то от 2го он будет отличаться только тем, что крон при проверки лимита будет каждый раз суммировать прошлую стату и текущую, вместо того чтобы сразу получать текущую статистику из базы. 


а вообще можешь написать как ты видишь работу этого модуля, а то я чет не могу понять что ты от него хочешь?)
Please don't use misc.nix. I want to get rid of it, not introduce a dependency on it.
s/is pass on to the /passed to the / ? (below as well)
out of scope for this commit, but : variations of this test should be run for all possible combinations of tiles in the app grid (or for all possible user permission profiles that determine allowed tile combinations)
Would prefer to keep this in-- don't know if we should put in caste or not. I took out caste for the demo, but at JSS they might decide they want it back.
Suggestion from Phil: why not leave the temporal resolution (i.e. all 240 years) but subset spatially. e.g. You could limit it to northern Europe and the files would shrink dramatically. NB. You'd have to update the corresponding description, plot title, etc. to replace references to global with your chosen region.
pep8 say that we should not put spaces here. It should be 
[strip_disconnected(g) for g in g_outs]

Idem for ().
Also, why not test ndim ==1? The grad function seam to support it.
New logger target should be added to the corresponding log4j configuration file, in this case it's enterprise_manager_log4j.properties, so that people know it exists and can change it.
Take the habit of deleting code. This was clearly my mistake. Don't worry about this one - removing it in merge.
Yes. 
```
$ cabal help configure | grep -A1 constraint
    --constraint=CONSTRAINT        Specify constraints on a package (version,
                                   installed/source, flags)
    --preference=CONSTRAINT        Specify preferences (soft constraints) on
                                   the version of a package
```
The changes to this file cause a merge conflict.
Insert space here between kind() and end )
One more thought... we never want this auto-doubling value to get confused with a "real" scale, so this should be `"0.5"` or `".5"` (whichever is smaller) to ensure type mismatch in the `while ` comparison.
This is great. I hadn't gone through this logic yet. Nice work.
I think this needs to default to None, or on systems where "g++" is not the default compiler, builds will fail when --cxx is not specified.  I.e., Windows/MSVC systems.
No chance. Arrays in JSON are always mapped to lists in Python by `json`. Ref. http://docs.python.org/2/library/json.html#json.JSONDecoder
This should use single-quotes. Looks like something I added, but it's good to fix it here given you're fixing the double-space, too.
Please inline this patch
@jacknagel, agreed. Fixed now. Thanks for the feedback.
Same here:

    class AmbiguousCartridgeType < InvalidOption
         def initialize(cart_type, cart_types)
         def to_s
             message...
    end

    raise AmbiguousCartridgeType.new(cart_type, carts) if carts.length > 1
The lines are to long, not really readable.
style1 and style2 are not descriptive.
it is also better to combine these 2 ifs, because the idea is to display both or none.
If you create an appropriate ModelProxy , e.g. `self.outbound_message = self.manager.proxy(OutboundMessage)`, you can do `self.outbound_message.index_lookup("batch", batch_id)` instead of having to refer to the class and manager all the time. Not sure how big a win this is right now -- just mentioning it so that people know.
For consistency it should be `Return the front page submissions.`.
Good description but it needs a summary above it because it contradicts the method name. A good example would be:

```javascript
/**
 * Handle socket event.
 * 
 * Get the session ...
```
Das sollte aber doch mit "xy" nicht gematcht werden, sondern nur mit "*xy"..
Und es fällt ja sowieso raus, da dir3 ignoriert wird.
How? I can't put „anstelle“ in this sentence ;)
„Den Proxy als Socks Proxy statt als HTTP Proxy verwenden“ is ok for me too.
Instead of doing this conditionally I think the `@key_type_instance` and `@value_type_instance` should *always* be set in `#initialize`, defaulting to `Virtus::Attribute::Object` which should just pass-through the value.

I'd still probably keep the `#coerce_key` and #coerce_value` methods though, I like breaking that out of the main loop.
Whoops! I did say such things. Fixed.
`Dagger.draw()`?

(OK, I sheath this pun for good now ;-) )
Another `@since` tag above a `package` statement. I'll stop calling all of these out :-)
Nit: Can you move privates to the bottom below the publics.  It makes it more obvious what the public interface is and is makes potential future refactoring easier.
NO GO. A join(', ', ...) of the remaining fields could be ok.
This seems a bit more dangerous than a typical polyfill. With a polyfill, you either use the browsers default implementation or your own equivalent one. As a developer, you shouldn't be able to tell the difference.

Here, if the environment doesn't have the implementation and, let's just say, the developer doesn't know to include a platform file, everything will work great while they're developing in Chrome. Later, they'll deploy completely missing implementations to other browsers without even realizing it, thus, breaking the definition of a polyfill.
Fixed. Changed to "app.checkForUpdates".
I'm a little worried about how this works.  If anyone else has registered a keybinding for this command, it won't show up -- only ones passed here during menu item creation show up.  Similarly, if someone adds/removed a binding later on, the menu item doesn't get updated.

To solve the first problem, it seems like maybe KeyBindingManager should maintain a second map, from command id to list of shortcuts, so Menus can quickly find all registered shortcuts for the menu item's command.  For the second problem, KeyBindingManager should probably dispatch add/remove events.
Top-level functions should have comments describing usage, params, and return values.
Would it be less bug-prone to have _all_ the super calls be of this form?  Otherwise anyone adding a new arg in the future must remember to either add it to the call() args or switch call() to arguments().
good catch! ;)
I am not sure if we want the lists to wrap around. I haven't seen other tools do that. Maybe take a poll of other tools or ask around to see what others think?
Is this still necessary ?
I don't see such thing in client plugins using `ide.staticPrefix`
Yes, it seems like various places in the code desire access to the path for the controller, but are calling "getControllerName()" - doesn't make their intent very clear.  But, at a minimum, those places in the code need to be updated to additionally call getControllerSubPath() because they won't work when someone uses the sub_path feature yet :)
I thought we were taking this out?
Why is this needed now?
why in an h4?
has someone been reading too much bootstrap code?
No comments in french please :-)

If it shoudn't happen, maybe throwing an exception is a good thing.
Could you rename `sample` to `examples` - this is probably most common name for code samples in gems
I do not see Shims being used in here. Can we remove it?
I don't follow, what are you asking?
not sure. what if we would need to, e.g. re-group some draw operators or showText, remove useless content, change order of blend operations?
The empty <i> is a bit of a hack - could we go with a CSS solution that uses :before or :after instead?

I think the browsers we're targeting with this site should all be able to cope with such a technique...
basically it's the same.
timers in directories create recordings in directories.
my intention was to use this flag later for things like moving recordings from on folder to another.
how about calling it "bSupportsRecordingFolders". if you dislike it i will rename it to "bTimersSupportDirectories"
Use of "function" keyword is supposed to be bad style. It creates incompatibility without buying anything; you can just omit it.