labofsahil/github-event-dataset-2012 huggingface dataset:
Minimal maintenance update

Hey,

In this pull request you'll find code that basically eliminates maintenance by generating all parts that change on the fly.

There currently are about 5 branches of compass-wordpress that all contain near identical changes, namely;

 * Updated styles and layouts from Thematic
 * Updated version of Compass

What I did is write a Bash script to generate a simple extension, and some copy-paste Ruby to use the version number of the installed version of Compass.

I hope you like the changes.

Pepijn de Vos
Use factory.SubFactory when possible.

This allows to specify the values of some fields of factory-generated fields:
  MessageF(user__email='foo@example.com')

Also, the SubFactory is evaluated when the Factory is instantiated, not when the module is imported (which was the case before this commit).
zsolt.staging_DL
Cleaning up the new app provided to get started quicker
Screenreader space fix

There's an issue when voiceover reads out the internal navigation links in the main section pages (strategy, research, etc...). It gets confused because there is no space between the number and the text, reading them out as one word.

This is just to add that space in without disrupting the layout.
Fix broker connection to use configured value not default.

As per subject.
Refactor.mergedict

Merged RedisDict and PyramidRedis classes into a single RedisSession class
Bug 813841 - [email] Explicitly sized HTML Body content appears to scroll under message envelope

* Sync mail api fixing in https://github.com/mozilla-b2g/gaia/pull/6711
Support manual account creation

@asutherland: this PR isn't ready yet, but in order to maximize the limited amount of time we have to go over the changes, I'm issuing it now. Currently, this just changes things so that we use the existing account settings when recreating accounts. That way, manually-configured accounts can be properly recreated.
BUG: Fix PostgreSQL error when creating a table for namespaced data object
adding a usage section to readme

adding a how to use section in readme
Fixed directory listings while using a 404.html

There was a problem before where showing the 404.html page took precedence over showing the directory listings. This problem should be fixed, as it now adds a bit of data that lets ecstatic know whether the request is from a directory listing request, and ecstatic can use that to know whether to show directory listings instead of the 404 page (the 404 originates from the lack of an index page in the directory).
mvnDebug binding
Update Mercurial to v2.1 and introduce constants for version numbers

We are still using Mercurial 1.9.3 in our Mozmill environment. With 2.0 and 2.1 a lot of improvements have been introduced. So lets upgraded.

I have build environments locally and tested them on all platforms. I haven't found any regression in Mercurial.

Also this patch adds constants to the top of the files to ensure we can modify it way better in the future.
Ensure test duration is output with a decimal point

Currently, if running tests on an emulator/device with a locale which uses decimal commas instead of decimal points, the test duration attribute is output with a comma.  This can upset JUnit XML parsers, certainly that of Jenkins.

Now the locale is fixed to English when writing out this XML attribute.
Tested before/after on an emulator with German locale.
Avoid sorting the working set and requirements when it won't be logged.

Merge Ross Patterson's speedup patch for zc.buildout[1] and
buildout.dumppickedversions.

[1] http://svn.zope.org/zc.buildout/trunk?rev=124059&view=rev

It's reported to result in considerable (over 2x!) speedups in some cases:
http://mail.python.org/pipermail/distutils-sig/2012-January/018268.html

Closes https://github.com/Simplistix/buildout-versions/issues/3
(Android) Added changes to fix TIMOB-3158, basically checking if the activity is nu

Added changes to fix TIMOB-3158, basically checking if the activity is null or isFinishing() before doing the show.  (Please note first time ever doing this so I may have screwed something up).
Fix trash should delete

This commit fixes posts/pages that are trashed aren't removed from the index.
Fixed namespace for Spree::Core::FileUtilz.
Updated crowbar.yml to support the CM4.1.1 package set. [1/1]

Updated crowbar.yml to support the CM4.1.1 package set.



 crowbar.yml |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

Crowbar-Pull-ID: 81adf58aa4ed2717d74ef381f54d941971391762

Crowbar-Release: fred
Automatically enrol users on relevant courses

In a new self enrolment instance
Prodmix + Warehouse

hier die beiden Samples. Ich hoffe ich habe das mit Nuget etc. auch richtig gemacht.

Gruß,
Marian
Small Enhancements added.

1-st commit is IE fix in 'var' declaration.
2-st commit is showHandles/hideHandles moved to addHandles/removeHandles. And I added showHandles/HideHandles with a timeout. I also added Flip/Flop functions.
Check type of showAsAction before reading value

Fixes issue in GB where it thought showAsAction was a reference to a drawable if no value was set.
Bug fixes for bug 716808 and bug 716799

Bug 716808: pair link spacing and placement on success screen.
Moved pair link to bottom of screen. Also made link style changes, propagated to other links

Bug 716799: screen rotation restarting J-PAKE
Set manifest attribute android:configChanges to control orientation changes from activity, instead of restarting activity (default).
trivial fix typo in Makefile

If I'm not mistaken.
LPS-21110

http://issues.liferay.com/browse/LPS-21110
LPS-24131

rotty3000
LPS-21303 merging site pages variation doesn't work correctly

- As the workflow status was publish, after merging, new variations were automatically published
- LayoutBranches were not created when merging
- If two variations had the same name it was too confusing, now we handle repeated names better
$DB global overwrite

Fixing a very odd problem where the $DB global would get overridden and set to a admin_setting_configcheckbox instance.  It was tracked down to the quickmail settings.php
solved issue#6 and #9

hi.

I solved issue #6 and #9, so please check the code.
if there is no error , pull it to your code.
Update external-sensor

newest version
Intscala 41 v4.0

INTSCALA-41 - Integrate Scala DSL application contexts with parent Spring application context when there is one
typo fix

changing "by the us" to "by us"
StorageObject.list() to use prefix

Changed the StorageObject to use the prefix parameter instead of path. Also updated the minor version because this might break compatibility.
Кнопка для очистки поля файл

Поддерживаются так же и имиджборды, в которых можно размещать несколько файлов (доброчан, краутчан, etc). Чуть позже добавлю кнопку для создания раржпега.
Исправление getReplyImgData

Все проблемы с отправкой файлов в режиме html5 теперь должны уйти
Updating working repo from blessed repo
changed the methods for localization
Some additional actions under selection items from suggestion list

* choosing suggestion by click on li item
* looping through items using navigations
Warehouse Service
remove unnecessary separator
Adding unique ID per mock object so that they aren’t equal (__phpunit_mockObjectId)

Adds a per mock object unique ID to prevent mock object from begin equal. Otherwise this test case will succeed per default nevertheless it is faulty:

```php
<?php
class Test extends PHPUnit_Framework_TestCase
{
    function testEqual()
    {
        $mock1 = $this->getMock('stdClass');
        $mock2 = $this->getMock('stdClass');

        $sut = $this->getMock('SplFixedArray');
        $sut->expects($this->once())
            ->method('offsetSet')
            ->with(0, $mock1);

        // This should fail
        $sut->offsetSet(0, $mock2);
    }
}
```
Update admin/sr-RS.com_hwdmediashare.ini

Finished  translation  admin/sr-RS.com_hwdmediashare.ini

regards, 
Goran
add in Pivotal Login test

Add This please
1.1.1 - group cleaning code

These are mostly feature I had in my own fork of the acegi plugin that we used to use internally.  I've reworked the plugins to be publicly available, but I needed the following features to reduce the amount of re-factoring as part of the upgrade.

I felt these features could be of use to others:

* Added example to documentation to support recursive groups in Active Directory (groups as members of groups)
* Adding additional LDAP group name cleaning to keep role names standard in `GrailsLdapAuthoritiesPopulator`
 * Prefix removal
 * Suffix removal
 * Dash to underscore conversion
 * Force Uppercase
* Added defaults to `grails-app/conf/DefaultLdapSecurityConfig.groovy`
* Updated documentation to reflect new options
* Added unit tests to test new feature
* Added configuration parameter initialization for `GrailsLdapAuthoritiesPopulator` in `SpringSecurityLdapGrailsPlugin.groovy`
* Added '...finished configuring Spring Security LDAP' statement to match the behavior of spring-security-core

Thanks,

--
Aaron
Added version command

Not convinced the last time we smelt crack we were running the correct version i.e. I haven't been able to recreate the situation. So I've added a version command so we can check what version is being run
как только будет патч :)
Did we really need to do this? Any font or IDE that cannot render an ellipsis, which is the only valid character here, needs to be thrown out the window.
Jeps.

Die --pidfile moest er ook bij anders werkt het niet, oops.
Typing error?
return $this->compiler;
The second part of this commit is necessary because Firefox and Chrome consider carriage returns inside of elements 'content', so the :empty selector does not find them.
Yeah, the template was actually incorrect.
// TODO: emit a cast of the scrutinee and a switch on the cast scrutinee if patterns allow switch but the type of the scrutinee doesn't
chan->fish_key[0] = 0;
man, I JUST committed almost the exact same code to my local git! I will compare and make it awesome
Running the tests I get

```
 tests finished: 3589 passed, 54 skipped, 114 expected to fail, 
7 expected to fail but passed, 34 exceptions
```

on 64 bit, Python-2.7, both in gmpy and python mode
Compiled successfully. but cannot run.
In the console, will show this: [UIWebView scrollView]: unrecognized selector sent to instance 0x???????
Yet it was the purpose of this Thread class and the special tags: one class for every kind of thread (pm and not pm)

If we change the way the PM feature is implemented, these special tags would surely become pretty useless.
Ok, so it wasn't actually functools.partial() causing the weird behaviour, but I like this change anyway.
Can't see when I'd want to use both of these simultaneously. Do you have a practical example? If not, I don't particularly like the idea of having a single object with two different behaviors based on flags.
you can change this to `<h2>{{ title }}</h2>`
it seems like the you may want to make this `type_pretty = unicode(site_event_type(amo.SITE_EVENT_RELEASE))`
Duplication with `Ordering.fromInt`
http://blog.gadgethelpline.com/wp-content/uploads/2011/08/free-MrBurnsExcellent.gif
It's available in libcork now.  [docs](http://libcork.readthedocs.org/en/latest/net-addresses/)
RIP Strongbad.

http://www.homestarrunner.com/sbemail50.html

![http://s11.allstarpics.net/images/orig/k/2/k2p7oxwa9fidawf7.jpg]
The docstring needs to say something about kwargs.  How does someone reading this docstring learn what keywords arguments are allowed?
Well RemoveAurasByInterruptFlagButSkip has custom checks for spells, to remove aura only on specific spell cast. For example Hot Streak proc applies aura to you and that aura should be removed only after casting Pyroblast, if we would cast Fireball aura should stay till it expires or Pyroblast will be casted. And such auras have AURA_INTERRUPT_ON_CAST_SPELL, that means remove only on spell cast. But AURA_INTERRUPT_ON_CAST means to remove aura by any cast, e.g. casting Mortal Strike or Fireball, it affects any cast, so such auras with INTERRUPT_ON_CAST should be removed because they are not designed to be removed only after cast of some specific spell.
Not sure what you are trying to accomplish. CCondition now looks like a binary event with auto reset. This does not solve the problem with the socket. It still can come out of wait with useCount != 0. From the socket's perspective the predicate is useCount. Since you allow to pass a mutex to the wait function, the solution in this case is quite simple: 12c221e8264f7739244d8c46a6ec9e854acd7542
I see the following options:
1)
Have a generic implementation of CCondition and pass the predicate as an evaluation expression e.g. by a function pointer or leave it to the upper layer. The latter might lead to wring usage of the class.

2)
Rename CCondition to CEvent and build the behavior needed on top of this. Don't pass the outer mutex into this class.
Looks like the isAtSetPoint() method is now deprecated?
Can we call this internalPort?  I think iPort is overdoing it on the abbreviation.
can you make this consistent with other acronym naming (MinCpuAssignmentPolicy)?
OK, but the logical step I'm missing is: wouldn't have been better to tell me "why don't we do x in way y?".
This way I "think" I know what you changed.

While I'm absolutely fine with the change in the text at the bottom of "usage.txt", I disagree with NOT providing a "-h" and "-v" options: those are very common in command line utilities, and they made a lot of sense to me.

Also, you got rid of a couple of files in the "linenoise import": while it's fine for us, having an exact replica of the linenoise repo means an easier way to track changes, given that is done by hand and not by git submodule.

I'm ok with you having the last word on what goes in (you are the maintainer and creator - you have all the right) but I'd appreciate a comment in the pull request instead of finding a mismatch between my branch and the upstream master, where (in theory) they should be identical.

Do you see where I come from?
Can we just use an if statement?

if (accounts)
  accounts.map...

is more readable and faster.

I'm not up on JavaScript so much these days but is there not a workable for(account in accounts) type of syntax? It's not semantically a map here (since the returned values are thrown away)
player cannot cast summon on invisible for his point.

Then we have an typo

     if (m_caster->GetTypeId() != TYPEID_PLAYER)

But I think you miss understand the problem. As i wrote: some mobs spawn positions are calculate right in FillTargetMap(), we should use this points and NOT make a new point with CheckPathAccurate(...)
во. то есть сразу с маунта прыгаем на вехикл, а потом когда с него слазим - пет неуправляем. да, это может быть - теперь хоть понятен источник. такое использование (через жопу) заранее хрен предусмотришь.
будем лечить, а пока надо врубить ансуммон пета. у кого уже испортился - прочистить базу (залить всем 0 в 6 поле базы, у кого там более чем 260).
Which vehicle?
We should probably make `readall(filename::String)` open, read all of the file into a string, close the file and then return the string.
Oh - I see. It could be a typo. I think you're right. Thanks.
this line is disappearing right away
Maybe we should create another branch to maintain for different screen size. :)
Intended as a fail-safe. Code _inside_ jclouds that requires a specific JcloudsVersion should have JcloudsVersion injected (as opposed to retrieving it using .get()) and arrange for a suitably prepared instance (created using JcloudsVersion("my-required-version")) available to Guice.
how about https://github.com/gc/mangos/commit/c357228f6093ce3b8ade48a0b977931099514cf2 , works fine.
Its just about those 1 or 2 spells in spellbook, not water elementals strength and other stats. Temporary water elemental should be there too, since on the net you can find many cases when someone accidentally removed Freeze spell from petbar, and people advised him to get it back from spellbook. Permanent pet doesnt have freeze, just temporary :p
@PSZ убрать скрипты, но конечно не все, а с вехиклов на БГ. tibbi уже делает.
@mosst, а чем я могу помочь, если у меня ничего даже похожего в скриптах нету?
our server was ok for like 23 hours, now suddenly cpu jumped to 120% (cause multithreading). Tried disabling AHbot too, but it was still happening at like 15+ hours uptime.
@billreed63: please review the changes to TreeCreateTool.js, DataGridCreateTool.js, and EdgeToEdgeDataListCreateTool.js. I believe I fixed some off-by-one bugs.
`{}` is a default value. Try this:

```clojure
(:foo {} :default)
```
I said to bluegr about this last night, but it is not trivial as the engine currently contains a large number of hardcoded filenames i.e. "DREAMWEB.r01" etc. spread around the source files.

Though replacing most of these is not a massive issue i.e. have a member string of the engine with the filename prefix selected by language i.e. DREAMWEB. / DREAMWSP. etc and do a local string construction / modify the function to accept just the part of the filename which differs and construct internally... but the full filenames are part of the g_roomData structure. This is not an issue directly, but Room members of this structure containing the full filename are saved/loaded, so some thought is needed as to writing these/rewriting on the fly when loading and saving....
If you wanted to switch to vertical automatically based on resolution, the place to do it would be here.
something like
if(top.document.width>1600) {toggleFrameOrientation()};

I don't feel like setting up the globals preference for this though.
Could you also make ironclad:fortuna acceptable here, for consistency with other MAKE-FOO functions in Ironclad?
this var seems to be unused; is that a bug or is it really not needed?
Getting back to this today.. will post a new branch with rebase and updates as needed
Ah, ok. The naming leading me astray. I would expect the Webfinger class to just do what it says. :)
Hi

Yes- but apparently only to the python branch :-)

Med venlig hilsen / Best Regards


Poul Lindholm Christiansen
plc@cmcs.dk / +45 4050 9306

On 08/03/2012, at 12.14, Alex Sayle wrote:

> oh I thought I'd applied that patch ...
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/plc5700/mediatomb/commit/308a5f6796ab9ea23f489e36577f1274037e6165#commitcomment-1057974
Rewrote because array.indexOf shim removed, also uses loop short circuit. After all, bestIndex will never be better than zero.
No, that makes perfect sense. I'll look into doing it, after I've finished this next massive commit.
Seems I should have checked the API docs a bit more carefully...
Just wanted to ask about the current state =)
I searched for `.d =` and couldn't find any instance. Can you make this an `attr_reader`?

You should make it protected while you're at it, too `protected :d`.
Looks reasonable to me. I'll let other argue over if this is the right way to handle NA.
He has not yet added this to his repo (I checked), so it should safely merge.
@kennyj me neither. But I saw [in this method](https://github.com/rails/rails/blob/0d1df723c79300b73691d7785ee87d7b74caf591/activerecord/lib/active_record/model_schema.rb#L227)
I don't understand this fix.  The intent in the original test was to check that the value of `cached_main_body_text_folded` had changed.  The line changed here shouldn't be setting the value but testing it.  That was the intent anyway!
s/BEM/the BEM/ ?
s/command is to build/command builds a/
In the comments on that post, from tekkub:

"If git:// and git@ workfor you, stick with it. The biggest thing this will
help with are people behind insane firewalls and windows users (especially
tortoise git)."
On Feb 29, 2012 12:58 PM, "Koushik Dutta" <
reply@reply.github.com>
wrote:

> Also, since git blobs are immutable, it allows fancy things such as
> nginx/varnish caching to take place. That way the git servers themselves
> are never hit. In theory, it should be much more efficient protocol if
> implemented properly, and Github seems to think so too.
> Our build servers agree. They've stopped having sync failures once
> switching.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/CyanogenMod/android/commit/8327388862650b77646d7ae9e633df2bfae3ec9c#commitcomment-1027306
>
hi. there seems to be a wrong character in this line... let me know if you need me to add an issue about.
cheers.
Thank you. I will learn git.
Naming convention - method names are always PascalCase.
Please test again on latest revision (aka this one). I'm laggin like hell, can't see the crashlog yet.
this was ok and probably working as intended:)
There is no changes in this commit, I just wanted my name to show up
shall we throw a NotImplemented exception here?
Hey Mike,

After updated to latest release, I also noted that whatever i specify in the tilestache-server.py in the 'run_simple' function for (ip/app adress) & (port number), I cannot get to run the werkzeug/serving.py. It always shows a error message containing default ip & port(* Running on http://127.0.0.1:8080/). 

I guess it happens in the code snippet, 
-----------------------------------------------------------------------------------------------------
if os.environ.get('WERKZEUG_RUN_MAIN') != 'true':
        display_hostname = hostname != '*' and hostname or 'localhost'
        if ':' in display_hostname:
            display_hostname = '[%s]' % display_hostname
        _log('info', ' * Running on %s://%s:%d/', ssl_context is None
             and 'http' or 'https', display_hostname, port)
------------------------------------------------------------------------------------------------------
Is there any fix for this? Like, it was running for me from localhost, but when I host this from Heroku it gets this error.

One thing I could try is, without mentioning "Tilestache" installation in requirements.txt to heroku, I can try to upload my own modified Tilestache from local drive. But, I am not sure thats going to work. Let see!

-Thanks
Raj
Eher "Ungültiger Bildname"
And this commit is just undoing the previous glitch - I don't think anything actually changed.
imho, return ```self```, who wants ```undefined method ... for nil:NilClass``` exceptions?
Not to my knowledge. It won't matter for source distributions presuming we do git clean -xdf first. What we did before was the same thing I think. The directories were just hand coded.
Yeah looked like it helped some for a while, but it def wasn't a material part of the slogging 

On Mar 1, 2012, at 4:34 PM, Simon Murtha-Smith<reply@reply.github.com> wrote:

> bummer city
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/LockerProject/Locker/commit/87d6852032aca2d70565c5d07004c1fdb2982c78#commitcomment-1033668
Please remove this and possibly other sysouts

Running org.jboss.test.osgi.example.simple.SimplePackageAdminTestCase
1
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 31.705 sec
Useful! Makes me wonder though: 1) should other widgets be throwing custom exceptions, 2) if so, should the widget factory or our extension of it have the raise method, and 3) should such raise calls make it into our production releases?
cool. If you don't mind, I'll add that document.id stuff, OK?
It's irritating that this non-change means that just about the entire software has to be rebuilt.
P.S. learning git rebase is encouraged.
I'm keeping this allocate-before-once pattern.
Use binding
What was the motivation for doing this?

Did you run into problems or just decide that you like the explicitness more than you hate the redundancy?
dara los mismo errores en el dev no ves que las spells de icc en un sitio estan en un lado y en el r2 en otro al hacer el merge quedan duplicadas mil cosas } fuera de lugar brakes donde no toca y mil mierdas y ahora con la gran burrada que metieron de icc que ya teniamos es un infierno arreglar esto
Ah, good catch.  this way it will only change on one condition.  Thanks.
Note to self: Could have been check for empty(). When calling the page without any post data, and exploding the POST[id] string (line 29), causes an array with one empty item, hence the extra check for ids[0]==''
In general, I like to use the Game class in game.py rather than the raw document from mongo.  It tends to be a bit friendlier than the raw documents.  You can call game.GetPlayerDeck(name).TurnOrder(), for example, rather than building up the name to turn order dick. Likewise, you can get a game_state_iterator to iterate through the turns in order (and it handles possession and outpost correctly). On top of the client friendliness, it gives me a nice layer where I can change the game document format in the repository and then just worry about the one direct consumer, the Game class and it's related subclasses (PlayerDeck, PlayerDeckChange) and not have to worry about changing every analysis.
글쿤... 그럼 수정될때까진 놔둬야겠군~ ^^

용권형 수고하셨어요~ ^^

On Wed, Feb 22, 2012 at 10:02 PM, Keesun Baik <
reply@reply.github.com
> wrote:

> 테스트는 꺠짐;; testContext.xml이 없다고.. 꺠지는 테스트들이 많더군요.
>
> --
> KeesunBaik
>
>
> On 2012년 2월 22일 Wednesday at 오후 9:58, Miracle wrote:
>
> >  테스트가 깨지는 것 같아요.. ㅠ.ㅠ
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/whiteship/springsprout/commit/2bd83f494001b48e0c0545809a750c8f3dd0e585#commitcomment-995769
>
typo in the property name
My point is, that it is locking something, maybe not what you expected.
> additional_plugins is not only a VCS, it is also the source of
> Spartacus

Then maybe we should adapt the frequency of pulling this over to
Spartacus to a reasonable value which still allows for corrections like
this without anyone having to pull an overnighter.

Errors like this _are_ bound to happen again.
Sorta.
We're writing internally a "very simple and stupid" maven to work around the inability to decently integrate Xcode into Nexus.

The idea is to have some ruby classes that will provide simple maven like actions (clean, build, test, install, deploy) that are driven by configuration. 
The script is also clever enough to pull direct dependencies out of an internal Nexus.
When then wrap this with Rakefiles so we can build the projects or release them to nexus easily from dev boxes or Jenkins.

This will let us break our dependencies in static libraries that are resolved and downloaded at compile.

It's obviously tailored specifically for Xcode builds and very limited, but good enough for it's supposed to do.
Hi

I actually did that at the same time as the pull request :-)

However I didn't put any reference to the pull request, do you need me
to resubmit again? I guess I forgot to put the pull request as ID 3.

Irving Cordova
http://irvingc.com

On Fri, Feb 10, 2012 at 10:50 AM, Terence Parr
<reply@reply.github.com>
wrote:
> Hi!  Heh, could i ask you to submit also through click wrap license http://www.antlr.org/misc/feedback to keep license clean? then i can integrate.
> Ter
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/irvingc/stringtemplate4/commit/7f6bb59859fea868343b8e410243ee7d8a4b1559#commitcomment-954851
Awesome, I will merge.
premature optimization - the else block handles this case as well.
Why? Is it really desired behavior to leave products in the cart after order?
Opened an issue for now. It can be further discussed there #292.
0 and 1 are not magic numbers. It just says to take the first element. Should I introduce a constant int FIRST_FIELD_INDEX = 0? Sounds ridiculous.
The file was "detabbed" where the tab character was replaced with the equivalent number of single spaces (good practice to ensure code formatting stays consistent across text editors.)

I also used more readable syntax when indicating which data set (MCAR or Missing Informative) was being used.
I think that we don't need this empty paragraph :-)
I experimented last night with creating themes and found it wasn't all that straightforward. In terms of customizing colors, the colorscheme.vim file is not enough to provide. It seems you have to modify Hl.vim in powerline, which has a mapping of colors. I don't know why it's not possible to specify colors directly in the colorscheme. I'm going to try to get some more info from @Lokaltog

https://github.com/skwp/vim-powerline/commit/6172a1177f96fc2cd5d54e404180fe297a5d74c9
Haha yea, as a fellow vimmer I can understand :)
Hey Simon,

Get ready for a long-winded ramble :)

As I might have mentioned a few dozen times, my approach to making Bandit
Racer was to do it quick and dirty, and once I'm sure how the pieces are
supposed to fit in the end, rewrite it nice and clean. I think that at this
time I have a pretty decent image in my mind of how the game (or any
similar game) should be structured. Now I'm trying to extract core concepts
into a generic higher level game engine,  both for use in Bandit Racer and
for my future projects.

Roughly, high level goals are:

   - base implementations of core concepts, like game world, game object
   - thin abstraction over box2d to do away with most of tedious boiler
   plate, data type conversions between box2d and gamejs
   - resource caching/processing framework
   - infrastructure for game world serialization & synchronisation for
   multiplayer
   - renderer implementation to provide a viewport into the game world with
   ability to follow object, zoom, render objects in layers, some most common
   render implementations for rendering objects, animations
   - director / scene management
   - general scaffolding for a game

So, lofty goals :) All of this is already implemented in Bandit Racer in
one way or another. Just need to refactor away abstract concepts from
game-specific stuff, and clean it up a lot.

With this commit things I added:

Object - a base classs all game obejcts are derived from

Entity - a base class for a physical in-game object
World - base game world class that handles creation, destruction and
interaction of its objects, in a discrete and serializable way

>>> +Object.prototype.AM_OBJECT = true; //HAHAH IM A DUMBASS. srsly, dont
know better way.
>
> what do you need that for? to check whether an instance has
`engine.Object` in the prototype chain?
Yup. I don't remember why I neglected to google it at the time. Don't look
at it, my shame is overwhelming!

> i was a bit shocked to see you overwrite the standard JS `Object`, i
would have called it `EngineObject` or something but i don't fully
understand what you do, yet :)
I prefer overwriting a key variable in the scope of object.js rather than
having to write engine.EngineObject a bunch of times :) Might be there are
some risks i'm unaware of. If so, let me know :/

This commit resolved the worst problem with BR code, lack of proper class
hierarchy and physics abstraction. My next immediate goal is to properly
enable sound, for which I'm thinking of Web Audio API with fallback to
Audio object. Did you consider implementing Web Audio into gamejs?

-- 
Domas
J'ai une erreur pour l'uplaod d'image, suis-je le seul ou vous avez la même ? erreur :
Warning: imagejpeg() [function.imagejpeg]: Unable to open 'ressources/data/Membres/admin.jpg' for writing: Permission non accordée in /opt/lampp/htdocs/Polyjoule_siteWeb/administration/ressources/scripts/php/functions.php on line 103
J'ai cherché sur internet, j'ai rien trouvé ... :(
That was an oops! Added the file now.
Hmm ... I'm not convinced. This commit introduces some backward-incompatibilities anyways. So, I think it's a good occasion to make things right. Also this is a pretty minor change in the code, and should be mentioned in release notes "View.renderers renamed to View.renderer_classes". Another solution is to write hacky code to support both of them, and deprecate 'renderers'.
Why did you kill this method?
I'm sorry, This change should be in SAND-244 and It is also obsolete.
```-Debug``` will always turn debug message on when running the app.
You could remove it and force debug mode with ```script/buysell_server.pl -d``` for development.
Ouch, sorry about that. Time for tests? :)
@schmittjoh A few months after @lsmith77 opened that Trac ticket, you said you moved the issue over to GitHub. Do you recall where it is? I can't figure out how to search for issues created by you. Just curious if this has been resolved in core (it was a 2.0.x ticket at the time).
:shipit: Haha, yeah! I'll write some additional things to increase clarity of the guide, i.e., the end event of the get request.  Even though, it's already extremely clear.
Kind of. But it doesn't depend on mime type only. It performs a series of tests on a file and outputs what kind of file it actually is. It's one hell of a tool to have if you're into reverse engineering. I'm surprised that you haven't used it yet. It ships with almost all linux distros. Read more here http://www.darwinsys.com/file/ . I tried to compile it for armeabi yesterday but didnt suceed. I'm not any good at cross compiling anyway.
Thanks, I don't think it is doing anything either. Since it was following the pershoots HC_32 config when turned on (which is turned on in his kernel for ICS), I thought it was worth a shot.  At first I thought it corrected things using GPS Test, but after more testing it seems to be a fluke.
Sorry here is the patch to the code that I applied.  image["Content-ID"] = "<image_%s>" % image_number
When I uploaded the correction and restarted PLone4.13 it failed to restart.   I will look forward to hearing from your colleague.
I changed how `processUncoveredFilesFromWhitelist=FALSE` is handled and restored the old functionality.
Would you mind removing the line break here and on line 96?
Ran out of time. Will continue when merged.
"fuzzy" = not sure about the meaning of original when out of context
I corrected the way jquery is inserted
> Another thing, to improve performance, the php.ini must be modified to load phar in cache. Use phar.cache_list directive.

I've update my host configuration an did see some improvements. The simple echo gained +33% but the TE und TE+DB outputs gained only marginally (<1%).
Trivial. Just import path changes.
表記が揺れてますね
@nvie to sum up, the final API would look something like:

```
q.enqueue(
    callable=func,
    args=(1, 2),
    kwargs={'a': 1},
    timeout=180
)
```
This logic seems duplicated verbatim, is there a good place to break this out so it could be shared by both modules?
Have you made sure `parse_str` can handle empty string without generating NOTICEs?
Sure. Though, as there's a *debate* around using static or not : http://slf4j.org/faq.html#declared_static, I think we should put that recommendation clearly in the CAS code conventions, don't you think ?
moved the comment remover in the OpenCL backend (3fba44b)
entire case can be rewritten as:

    case someOrigin: ByteString ⇒ 
       threadLocalDynamicAccess.dynamicVariable.withValue(getClassLoaderForOrigin(someOrigin)) { super.receiveMessage(remoteMessage) }
Would it be better (faster) to only call pop() once?
I'm not sure I understand what you are looking for. I tried to explain what it does:
"use the resizer defined in code if not defined in config"
I think ActorRefProvider and related things should not be user extendable yet
Also a good point, I think I prefer `whenStarting` and `whenStopping` to make it clear that the callback is executed before that overall action is finished.
Each pair is [:key, [:content, *:extras]], so [:name, [:content, :type]] (:content is the string with :substitute processed)

First we shift off the main card.
The style of this test module does not match the style used in the rest of the codebase. Please fix that.
Should this change be separated into its own commit?
What happens here when mask in None?
Hmm, this should be catched in line 40 (https://github.com/okfn/ckan/blob/6ad7ec634dfde9fbbd206330b5327b0601509e6e/ckanext/datastore/plugin.py#L40). I see the problem. You have to remove the --plugin=ckan. I'll change the docs and improve the checking.
Same here, include repo_id/distributorid to describe what is being unbound. Even though this task is the agent call, that data is still part of uniquely identifying what's being unbound.
This is very much a guess as to what the original author intended, based on the variable name and public Fribidi API.  If this is not passed here, it will be uninitialized when read below.

The only other solution I could think of was to remove the buffer and read entirely.
Why do you need to do this anyway? Why can't you set all_priorities = cost.keys() and skip the deepcopy?
old-style class
This function takes a list of arg lists, creates `Popen` objects, connects the internal pipes together, and returns the dict with the proc and args objects the way `MRLocalJobRunner` expects. I'll document it better.

The idea is to move all the piping logic from spaghetti code in the runner into this small function. I call it a success. :-)
You are only testing for the network's starting IP, that's why you didn't find your errors.
Not sure this entire if-elif-else block is the best way to handle this. I would argue using getattr(requests, method.lower()) would be a better use case but we need to explore requests library more to be sure about that. Also requests implements PUT, PATCH, DELETE and some other HTTP verbs (see http://docs.python-requests.org/en/latest/user/advanced/#http-verbs) so we should at least support the ones necessary for CRUD operations. (PUT, DELETE, PATCH)
this is faster 
why to check for `tagMap` in an often called function when we know from the start if it is defined or not
This is just a picky style thing on my part... it seems like you probably want to put this statement (needs_next_dest_=false) *after* the moment in which you actually set the next_dest_. Putting these things after the moment in which you actually set next_dest_ avoids confusion if the setting process crashes for some reason. 

The way it is now, the code encourages confusion and accidents later if code were to be inserted between this line and line 156 that might rely on these pieces of information being up to date. Make sense?
You may want to also install the header file in the includes. That will help module developers use this functionality. That will happen down below in the Core/Utility/CMakeLists.txt file, maybe around line 66.
This is not recommended at all. Use the same logic as in master. (system.dispatchers.lookup might return a new instance for every call)
what about Twig_Function_Method ?
maybe:
return $this->templateClassPrefix . $this->loader->getMd5CacheKey($name, $index);
Why does user interaction need to be disabled?
I think these two examples are trivial and should be removed. The comment is enough. I think more of the problem with this actually boils down to `TUITextRenderer` itself, and the lack of information on how to use it.
Missed that one, but I generally do that.
You do need a space after `if ` though.
the window of opportunity is very tiny. what is the alternative that you think might be faster?
Why not always cache?
Should be done in `init` instead, by registering a listener for the target's "ready" event.
No need to have `defaultFormat` and `rssFormat` - `format` should be enough, with a default of "OpenLayers.Format.GeoRSS".
"mss" - is this a typo?
и в этих 4 вызовах тоже
there is a name space collision here. there is an argument to the __init__ function that is called isherm, so I will change it back to hermcheck.
we dont need the ```\s+``` between ```path``` and ```:```, do we?
please, use `dup`.
The `return` returns the candidate constant from the enclosing method which might not be the end of the chain, e.g:

``` ruby
class Alpha
  class Beta
    class Omega
    end
  end
end

class Delta < Alpha; end
class Beta; end
```

``` irb
>> puts "Delta::Beta::Omega".constantize
=> Alpha::Beta
```

The following implementation handles that - it's basically the same as yours:

``` ruby
def constantize(camel_cased_word)
  names = camel_cased_word.split('::')
  names.shift if names.empty? || names.first.empty?

  names.inject(Object) do |constant, name|
    if constant == Object
      constant.const_get(name)
    else
      candidate = constant.const_get(name)
      next candidate if constant.const_defined?(name, false)
      next candidate unless Object.const_defined?(name)

      constant = constant.ancestors.each do |ancestor|
        break constant if ancestor == Object
        break ancestor if ancestor.const_defined?(name, false)
      end

      constant.const_get(name, false)
    end
  end
end
```

The two `next candidate` lines could be written as: 

``` ruby
next candidate if constant.const_defined?(name, false) || !Object.const_defined?(name)`
```

I prefer the two lines as it makes the intention easier to see using the `unless` but I'm not particularly fussed either way.
Same here, also done in ebb906d23ae3212a
both lines are not equal, because `self.attribute_aliases = ..` is no plain variable assignment but a message send on its own, which will not be performed when calling `[]=` or `merge!`. calling `attribute_aliases=` is essential so that each model has it's own hash.
I fell into the same trap previously see: [7264223](https://github.com/mschneider/rails/commit/726422304cd28e05f6e8d5813d3b963e0f4a1042).
if I add the call to a many test fails
would prefer to explicitly returning a value of `true` or `false` here, to make the output value more explicit, otherwise it will be nil which rarely tells me anything.
Please move this changelog to the top.
But how is this particular change related to unused variables anyway? We want to test string equality there don't we.
`build` isn't so bad.   I do like `copyTo` in case you have to manually pass the Builder.  i.e.

```scala
val newVector = x copyTo Vector.newBuilder
```

Unfortunately, that requires *another* implicit from `Builder` to `CanBuildFrom`. which doesn't exist, and could be a horrible idea.    However, the above is what I have in mind, after I get some more testing in the second half of the idea.

```scala
val newVector = x build Vector.newBuilder
```

Is ok, but not as good.   Even though:

```scala
val newVector = x.build{Vector]
```

is alright.     Should we have a naming free-for-all on the ML?
Should be a comment explaining why this is commented
Probably `name` in the `Reification` case class should be made `TermName` instead of just `Name`. Then the necessity of this change will vanish.
Is the minimal spanning tree unique?
Isn't the summary line `Convolve two N-dimensional arrays.`?
Se above comment.
oh wow, im dumb. i totally forgot that this was C-style. i'm so used to Java.

should we use C or C++ style comments?
Yes, I know...so are we just going to let them find out the hard way, cursing sympy as they have to reboot their system because it hangs? :-) I suppose it will be memorable...
Yes it does.  `setup.py test` runs *all* kinds of tests, so it should be the only test command you need to run.
Also, if it's worth anything, the implementation here perfectly aligns with how permutations are dealt with numerically in LAPACK (they're represented as row vectors).  We're a symbolic library, not a numeric one, so this shouldn't be super important but it's worth bringing up.
As stated in the docstring at the top: `Note that the "level" (number of variables, i.e. parameter u+1 in distributedpolys.py) is never needed in this code.`

That is, the parameter `0` which is passed to `sdp_add` is called `u`. `sdp_add` never uses it, which is what my comment is trying to say. Maybe you can suggest a clearer wording?
Why you are doing this is_numeric() and later (int)? I think (int) or intval() will be enough. And dont know is it good to throw CException when checking range, may be just not set value when range is wrong.
On Wed, May 23, 2012 at 9:28 AM, Robert Martone
<reply@reply.github.com>
wrote:
>> @@ -34,7 +34,7 @@ video {
>>  /*
>>   * Prevents modern browsers from displaying 'audio' without controls
>>   */
>> -/* line 40, ../sass/normalize.scss */
>> +/* line 40, /Users/jgourgou/Sites/athena/Lu2/Lu/styles/sass/normalize.scss */
>
> hmmph!
>

Yeah, I don't know why this happened.  All I did was run "compass
watch"....?  I'll fix.




-- 
jimg at yojimg dot net
Superfluous check? r should always be zero with UV_RUN_DEFAULT, right?
How about destroying extensions cache only?
Why is it public?
Is there a transaction for these statements?
assets_version misseng.
Hrm, file's changes are all invalid IMHO
This message could be improved:

"An update to the Cloud Monitoring Agent has been downloaded and is ready to use. Please restart the agent."
#endif // GLOBAL_H
I definitely think so!
I like the current approach, since it is more future-proof against future code changes.
defining:

var dict;

and then using:

dict = {};
this.sortedModules[env] = dict;

would avoid constantly traversing the same chain repeatedly. The same approach for poslKey and lang would keep traversals to a minimum.
Вот тут начинается плохой код(с учетом того что я написал в StatisticsDao)
"Infecting a baby villager creates a baby zombie villager which is faster than a normal zombie and does not age." (http://minecraftwiki.net/wiki/Zombie - under release history). As far as normal zombies go, looking through the code, I can't see anything that indicates they do. There is `int d`m but that indicates how much longer a zombie is weakened by a golden apple for.
This line gives error in Visual Studio "ambiguous call to overloaded function". Replace sqrt(2) with sqrt(2.0f).
undefined gets printed on the registration card (as undefined is passed to corresponding id in patientCard.html)if I don't do this check.
You should be calling loginSuccess here - if the user clicks login, calling doLogin(), we want to see the AppGrid
> ...cannot be modified by any subsequent changes to the cube.
Perhaps we can't imagine how it might happen right now,
but that array is owned by another thing, and it can do what it likes with it.

The point is, we don't know, and shouldn't know, what a cube may or may not do to this array in the future.
I suggest, for best practice and future proofing, we don't share this array with another cube.
It's seems like a very undesirable dependency.
Do the current String-toTimestampType routines throw when they get extra (sub-nano) decimal digits, or is this uncharted territory?
I know compiler will optimize it out, in fact that's why I did it. And I knew of semicolon and decided its ok. But I did not think about the else case,  that is indeed dangerous and leads to hard to find bugs.

But I clearly saw that your existing do while did not work inside a for loop. I had to add "If not null" before my_free to make it work. It probably would have been enough to just  add { } for that loop, I can test that but it still means my_free is not safe how it is.

Its your call what to do but if you can't find a better replacement, would have to make a note to put my_free in {} in loops and go through code everywhere to make sure its there everywhere.
Why [2:]?
@cubiq do you want this in a cleaned up pull request? I'm not really sure of the function here- I'd just like to get things feeling closer to iOS. Does this change help that? cc @darvids0n
Will this be exposed publicly?


`function ()` => `function()`
The latter. In this case, I'm pretty sure that the run of similar values compresses well.
So you were talking about the new deferred object? That's the basis of the queue, you can not instantiate a real $.ajax() object and pass that back. You have to wrap the request function in the scope of another deferred object. This is all based off of @gnarf37's [work](http://gnarf.net/2011/06/21/jquery-ajaxqueue/) and seems rock solid. Thoughts?
PGXN_VERSION should also be configured.
hmm, yeah. The change @n1mmy added still can't address what I concerned about locally testing QQ auth service. Will do another drop after I am done with the auth part. Thanks for pointing that out though.
actually there was a change there which I tried a fix (https://github.com/raistlinthewiz/MonoGame/commit/a9696b85c39e80e6b801b5fc52f0320581c0aaa0 and https://github.com/raistlinthewiz/MonoGame/commit/f8274d6047158cf02155397acc5a2849df7c54d6), though slygamer provided a correct fix later for it (https://github.com/mono/MonoGame/pull/395).

So I decided to remove my changes back on that file which eventually broke the blame.

I'll be removing that annoyance now.
This should be reported upstream to libzzip as well.
using opt_prefix now.