labofsahil/github-event-dataset-2012 huggingface dataset:
Constraint evaluate fix

This commit should fix a lot of liar assertions. Let's consider the following test example:

````php
<?php
class MyCompany_Module_Test_Config_Config extends EcomDev_PHPUnit_Test_Case_Config
{
    /**
     * Assert that this module's front name is 'foo'
     *
     * @return void
     */
    public function testModuleFrontNameIsFoo()
    {
        $this->assertConfigNodeValue('frontend/routers/module/args/frontName', 'foo');
        $this->assertConfigNodeValue('frontend/routers/module/args/frontName', 'bar');
    }
}
 `````

`````xml
<config>
    <phpunit>
        <suite>
            <modules>
                <MyCompany_Module />
            </modules>
        </suite>
    </phpunit>
    <frontend>
        <routers>
            <module>
                <args>
                    <module>MyCompany_Module</module>
                    <frontName>foo</frontName>
                </args>
            </module>
        </routers>
    </frontend>
</config>

`````

Before this commit the test will be passed.
Get build working with GHC 7.4.1

Also needs a prompt release, if possible :-)
Fix Nagios issues injected by Andi's code checkin. [1/4]

Commented out what appears to be a missing file that needs to be checked in.
Added comma to role.


 chef/cookbooks/nagios/recipes/server.rb |    4 ++--
 chef/roles/nagios-client.rb             |    3 +--
 2 files changed, 3 insertions(+), 4 deletions(-)
Performance fix of Enigmail and signed label features

I've fixed the performance issue with Enigmail. It works faster and seems to work fine so far.
I needed to split registerActions() to register attachment actions.

In addition, I modified signed label features.

Could you review this?


Performance fix for opening threads with Enigmail

It takes long time to open a thread which contains some encrypted
pgp/mime messages with large attachments. When opening a thread, the 
entire contents of encrypted pgp/mime messages are streamed instead of
just picking the information from Gloda. Streaming + pgp decryption does
cost time.


Refined signed label features with Enigmail

- show signed label of encrypted and signed pgp/mime messages
- fix setting title attribute since two `tag-signed` exist
- open security info alert dialog on clicking signed label for easier
  access
Vala Syntax Support

Hi, I just added Vala language syntax support to the sublime. Plz check it out ;)
Improve asset function
fixed path error in audio_sp.inc // jira ticket 689

I also noticed https://github.com/Islandora/islandora_solution_pack_audio/blob/6.x/audio_sp.inc#L183 which may be the same error?  Will have to talk to Gervais about that.
add Keywords support (pypi)

Map keywords from setup.py to categories in project during release (pypi)
Simple account cancelation

As discussed:

* Cancel button on profile page - Triggers an email to us
* Note on profile page about cancellation taking up to 24 hours
* Link to "Update your account settings" in broadcast emails

@sandal look over the code just to sanity check me, but I think this is OK to merge.
LPS-8498
LPS-24763 Optimize PasswordPolicyLocalServiceImpl by reducing service ca...

...lls
LPS-24442 should export repository element if path is already processed
Any UTF-8 character may be specified
Hi there!

I have made a few modifications here. Basically deleted unused files and added a .gitignore file. I am open to further explanations.

All of the best, Alexandre
Fixed backwards compatibility issue with how coffeescript compiles the is_valid_luhn method.  Older versions of ie can't access characters in a string with [].

Fixed backwards compatibility issue with how coffeescript compiles the is_valid_luhn method.  Older versions of ie can't access characters in a string with [].
Can now specify the % of a frames point cloud data to be notified about

...percentage of frame)
Iterating entities
Extended Description

No worries @skurfer for not picking up on it.

These small changes:

* add a proper extended description
* Improve the documentation (I've elaborated more)
* version bump
2.2 review

2.2 review
Fix CTV regex

Fixed the CTV regex for the now slightly different JavaScript.

Also made the regex non-greedy for quoted strings so it doesn't capture more than expected.
Nick JR plugin rewrite with siteconfiguration which uses #nickjr

Hi,

I've updated the plugin and the site configuration to use #nickjr in the call so we can use the right site configuration.
I also adjusted the dailyshow and colbert to march erverything except the #nickjr

Regards,
Eelco
"Compile and Display JS" command for Sublime Text 2

Added the equivalent of the Textmate "Compile and Display JS" for Sublime Text 2.
Added a couple of options

Thanks for the very handy and cleanly designed plugin. I had a small problem using it with an inline element that was right next to another character (a span inside quotation marks), and the problem was that animate() turns an inline element into an inline-block, which messed up my layout.

I've modified the plugin slightly to allow animation to be turned off. Consider rolling these changes into your own codebase.

Thanks,
Rob
Translation of header_account

Hi,
I added the translations of header_account.php in the language files. And I have translate it in french of course :)

And sorry if you see my old commit, I'm not a master of git.
Github page added.
Missing a comma in config example for secureChannel.definition
Fix an inconsistent role name in the docs
Stream now streams without needing to change the streamData. However, I ...

...had to add in a timeout.
Moved the DataFilter model from Portal to Indicators

Moved the DataFilter model into this project to facilitate command center tasks.
update for gradle 1.0-milestone-6
rowend/4-admin-error

Translate complete admin error folder
Make it work with latest version of AMQP

New API of AMQP is not backward compatible when it comes to channel management. The current code didn't work anymore because it tried to reopen channel and was not waiting for async creation to work.
Added Minor Organizational Cleanups to settings.py variables.

Made a few minor ordering changes to variables. Made newline structure consistent.
2 0 stable

This addresses issues #6 and #3.

I'm testing client_side_validations-simple_form on the 2-0-stable branch with client_side_validations on the 3-2-stable branch, using the default simple_form generators with zero modifications. Everything works great for the most part, except the `error_class` is placed on the wrong element. Since the bootstrap wrapper adds an additional wrapper surrounding the validated input, `wrapper_tag` and `wrapper_class` no longer match the same element.

This pull request removes the assumption that `wrapper_class` and `wrapper_tag` will always match the same element. The default simple_form wrapper continues to work as intended, as the matched element is the same.

This is what it currently looks like:

!(before change)[http://i.imgur.com/zryNX.png]

This is what it looks like with this change incorporated:

!(after change)[http://i.imgur.com/h8Jns.png]

It also correctly formats errors on forms tagged with the `form-horizontal` class:

!(horizontal forms)[http://i.imgur.com/DtYYQ.png]
Auto overwrite emptybuilder 2.0

port of #154 to 2.0

not tested on a project

[![Build Status](https://secure.travis-ci.org/jaugustin/AdmingeneratorGeneratorBundle.png?branch=auto-overwrite-emptybuilder-2.0)](http://travis-ci.org/jaugustin/AdmingeneratorGeneratorBundle)
AVL-Formeln & Source Code

AVL-Formeln:
Die fehlen, sind aber wichtig da die relativ willkürlich sind.
Muss also vermutlich korrigiert werden, da ich noch kein Code für den
AVL-Baum vorhanden ist. Hinweis im Dokument ist rot.

Source Code:
Ich hab meine eigene Implementation eingefügt.

PS: Zeichenkodierung ist mal wieder fehlerhaft, werde das noch
anschauen und auch bald committen.
Changed save log location, created activated event for addmessageform, and added reset and focus txtname
Added an Atom.xml file

Works in google reader and w3c validator.

Fixes #14
Added some things

Added "Lasting Appeal", considerations for screen res and intructions and a Sound component as well.

Also added white space between components for formatting and legibility.
Fixed errors, changed uploadSensorData parms, started recording real accelerometer values

-Fixed syntax errors
-Changed uploadSensorData parms (actual uploading still TODO)
-Added recording of accelerometer values
Move plugins (elasticsearch and community) to a central page

As discussed in #129, here is the pull request.
I think we can ask for users in the mailing list to contribute to this page (if I wrote something false or if plugins are missing).

Plugins will be shown now here : http://www.elasticsearch.org/guide/reference/modules/plugins.html
Changed autoload instructions in docs from require to fallback

This seems to work for loading the Mockery class that does not have a namespace rather than having to require it.
Refactored the functions for Figure 2, better using ggplot2 functionalit...

A better way to generate PC plots using ggplot2's 'facet' functionality.
added pr2_plugs and geometry_experimental to stack.xml
Same here:

```c
if (it->second)
      it->second->chatLog(event.getString("message"), BY_GM);
```
Что в итоге принимается к серверу и выдаётся наружу?
You're absolutely right! I didn't know I could do that, I'm so used to setting the this object with the second param.
I fixed it, and the getObject typo as well.

I still have the feeling that these two functions can be refactored a bit but I still have a few things to before (like entering beta)

Thanks very much!
Nice!  I tried to get this through in a patch on the mailing list a while back, but it was rejected due to the fact that the menu isn't translatable (using our translation system). I agree with this change since a) it's better in English than not at all and b) it's not technically in Geany's core, it's a plugin.
Please, consider linking this file to luvit.io
should be val instead of key?
![](http://i.qkme.me/36fqv2.jpg)
Ok seriously this is just getting out of hand.. i believe it is just a different path to create a tenant in d5
Its not yet complete.
Missing features:
-Autotools support
-Icon support
-grabing support
-clipboard support
You can even make this at the same time a *real* list.
This might overwrite the file, how about using tempfile instead?
Ingenious. I wonder if we should define a sort of a "filler" that would work somewhat like an appender but would first fill the existing array. Then, if the array is static, the filler would throw, and if it's dynamic, the filler appends to it.
Thanks your confirmation !

I should rebased and updated to send PR.
I commited https://github.com/kennyj/rails/commit/c8c84397cdbd44f47ff9f8594a3dabfe8b3b1502 already.
oops... I guess eclipse shift+command+O auto-imports is not that smart...
Hmm, you were right. It's constant but it's not the same one. Sorry.
Well that's a relief, now quick, hide!
Mh, ok. Then I would go for/prefer checkboxes in the TopicListView. Btw, Wave had this feature as well, so you could do batch archiving.
This has already been fixed.  Please check the current source.
Even I was like: "WTF is this?" I must have mistyped it in script buffer instead of interactive buffer. =/
=( 

Buildfile: /opt/jenkins/jobs/bugswarm-connector-0.3/workspace/touge/org.touge.restclient/build.xml does not exist!
Build failed
Following DocBook spec thought, I sugest renaming it to PERSON, which will be more flexible.
I changed this to be consistent. Also having the word Invoice is super redundant.
I'm on Xcode Version 4.2.1 (Build 4D502) and am building against the Max OS X 10.6 SDK.

Is it just this line that isn't compiling?
That is true but that is only if it either happens in the first frame or the ball position is unacceptable at some point, otherwise it would only look in the area around the ball. In tests this doesn't happen so maybe it would be best to test it some more and change it if it becomes a problem? Also remember that the noise is unlikely to be persistent so it would fix itself in the next frame. Also line 226 provides some protection against this (the value can be changed if it proves to be a problem.
хех... большая часть слово-то такое только сейчас услышало.
Is this safe to do this way, doesn't it cost too much power to REMOVE all the item mods and re-apply them again for every person who revives?
Does this even do anything? Pretty sure I check if the blocks exists on the block watch list before adding them to toBeFelled.
Serge please use this settings http://www.winasm.net/forum/uploads/post-18-1201953773.png
Thanks for the suggestion. I am not using this code at the moment and sort of stopped working on it for now.
I am now doing grid searches using "apply async" by hand to do grid searches, as GridSearch was not flexible enough ;)
Oh!  That's a new wrinkle.  CC @benturner
No problem, I'll change it to strings.
Because tizoc said so.
Is this how I pull the latest code into my fork - i should take time to learn more github.
Personally don't see any need for getters in this case but please change as style dictates
is the only way? If a player never change his talents, the patch don't take effect?
It's a russism. Read that "less hacky" =)
#naicily done!
This will cause the attribute to only be set to false if you do:

  modelobj.attribute = 'FALSE'
Here's how it looks: http://img406.imageshack.us/img406/5678/qupzilla.jpg
Needs justification to 80 or 79 chars.
Great stuff!  I always just coded around these cases.  But much better to fix them here.
The server worker has a thread, if it is running, it is asked to stop

Stopping it is not an instant action though, the thread is not simply killed but is asked to stop itself in a good manner, so if we caught the thread in exactly that moment where it is not running anymore but is not yet done with stopping itself, this line is displayed

PS: this should not happen really since you can not click the button anymore to actually reach this code, but it is nice to have a complete and concise code to act on all possibilities
We can leave it blank, it works, else set a tagname. 
Not sure remove this line works nice
works like a charm, thanks
I am very interested in having translatable fields in Mezzanine. 
Could you please help explain how I might uses this with the blog model?
Shouldn't this return 0?
Otherwise this violates an important compareTo criterion:
a > b <=> b < a
(a.compareTo(b) > 0) == (b.compareTo(a) < 0)

If a and b have the same priority here, both compareTo calls will return 1 and the criterion is violated.

However, since this was explicitly changed from what I perceive as correct to the current state, I'm going to leave sk89q some time to comment before I mess with it.
For me, it seems like density should be a function: with discrete X, we can take Density(X)[2], so likewise for continuous X, we'd have Density(X)(2). How I think it should look:

    >>> x = Symbol('x', positive=True)
    >>> X = Exponential(10)
    >>> f = Density(X)
    >>> f(x)
    10 * exp(-10*x)

I'm not sure how multivariate stats works exactly. Can you give me some examples? My guess would be that it would return a function of multiple variables.
Hey Darren, are you sure this wasn't just a case of the plugin projects needing to be rebuilt?  The reason I wonder is that I didn't add those brackets in my warning cleanup... They were already there, and apparently working before.

It seems from experience so far that the plugins are only binary-compatible if built at the same time as the app.  This is because although the app calls the plugin functions through dynamic linking, the plugins call framework functions (ex: functions in AMScan) statically.  If the address of those functions changes without re-compiling the plugins, there will be crashes.

This also happens inside the framework with functions that are inlined.  If A.cpp inlines functions from another object in B.cpp, B.cpp changes, and B.h is #included in A.h, then make will take care of rebuilding A.cpp.    But if B.h uses forward declaration and is only included in A.cpp, then the make system won't rebuild A.cpp -- causing crashes when it calls that inline function at an address that is no longer correct.

As far as I know, the only surefire way around this is a full rebuild.
Fixed in bb36e5763aac20670a994d85578cd8faa70b1dc4.
Please use iterators in the future - see my merge cleanup commit, once I've pushed it.
Interesting as that deprecated issue does not occur on the Simple Machines wiki.  Although I can see why the define failed and it should of been pulled prior to committing.
maybe this reference can be eliminated using some trick?
Or maybe we can use 2 fixed array list, and just swap the old (end cleaned) one with the one full on new data, and so on?
In the 2° case you have to pay attention to NOT store around the reference, if you keep using the old one will be a tragedy trak the problem
it's an easy way to manipulate the prereq list. the lake frontent has a "link-only" function, which could not be realized with executable building block of cxxproject before this change. dependencies to source files must be deleted, but not dependencies to config files, mkdir, etc..

lake does the following:
theExeBB.prerequisites.delete_if {|p| Rake::Task::SOURCEMULTI == Rake.application[p].type}

or is there another way to archive this?
As a gesture of good will, I'm actually checking out emacs a bit this weekend. Chances of me switching over are low at this point. Muscle memory :)
TY.
Cache the result of `strlen(ticket)`.
We thanks Lucas Rolff, who have suggested us to include this feature. =)
This commit was placed by @djahren but for some reason glitched out.
We should probably remove use of eZINI (for 1 it is not properly setup in api context, as that is done in index.php)
Does Legacy use a lot of eZ Publis libs??
Indentation seems off here - should be a 4 space tab indent
passing a name in through the command line? maybe we should define the name in the plugin api itself to keep things calm. i mean, we're not passing in modes through the command line.
without () the filter will not work right
For tests we can use lots of asserts not only self.assertEqual
试一试Console.WriteLine();为什么前面必须加System?
@Guff does this look good to you? I'm not sure why you moved `width = w; height = h;` up in `Image::Tile`, but I moved it back down because I was only getting one tile due to the `tiles_x` and `tiles_y` calculation. (Sorry about the whitespace, moving those two statements was the only real change)
Je kunt als test proberen Jascha aan te melden voor het SpelLAN. Kun jij dat testen? Ik kom er vandaag niet aan toe.
return num > 0 ?
curious why 40?
here's where you are missing the .get()

submitScriptOnNode returns a future.  in order to block on completion, you need to call .get()
Think I fixed this and the problem I was addressing in 1.23.2: https://github.com/migurski/TileStache/commit/669bd5
Thank you.

Both the requests are merged into "development" branch
finally figured out what is failing :)

```ruby
assert_file "#{javascripts_path}/models/foo.js.coffee", /class\ TestApp\.Models.Foo/
```

see https://github.com/seattlerb/minitest/issues/73 (assert_match problem)
style-nit: For whatever reason, the code in JUnit doesn't tend to have a space before assignment operators. Could we make these changes consistent? Thanks.
Fixed. Thanks for the info.
It looks like the original test (which tested switching between apps) was replaced with one which only tests whether the welcome message appears.

Is it possible to still test switching between apps as well, or is that impossible with the new connect screen?
There's no global class to put them in.  They each have only one user, though, so I could stick them back in the MOSDOp[Reply] messages.  I liked having them here, though, since they're related to (a vector of) this type.  :/
For that  matter, you could provide a little SQL script that demonstrates the potential problem with PostgreSQL 9.0. I have 9.0 here and could easily test it.
Gee, thanks, it's getting near the final result.

Your solution returns:
ResultSetParser[scala.collection.immutable.Map[models.auth.User,List[models.auth.Permission]]]

So I'll probably need one last map-operation to get the final User with it's permissions out of it.
Thanks Peter.  I think h2 and javassist can be updated pretty safely without any code changes as well,  As well as the ebean pull request.  Let me know if I'm wrong and I'll take a look.
Oh, totally, that's why I kept this in my "personal" branch :P I wrote [a Ruby DSL](https://gist.github.com/1617301) for quickly generating TerrainControl configs, and TerrainControl writing over them with all of its comments was making things a little messy.

I definitely agree with you! This was just for personal convenience.
Xl's strings need to stay on one line (I know it's ugly in the code, but this is how it has to be so the translators see the entire string).
I agree, each step/point for the Web Installer should have a screengrab.  Given the general audience for WebMatrix, any visual aid is an advantage to the process.
These three lines:

```

  ok( !endFired, "end should not of been fired." );
  plus();
}

```

have a tab at the front of each
On Mon, Feb 20, 2012 at 6:15 PM, Michal Čihař <
reply@reply.github.com
> wrote:

> Try choosing better identifier here, tabId is too generic and might
> possibly also occur somewhere else...
>

Replaced it with  'tableFieldsId'


> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ammaryasir/phpmyadmin/commit/ca6277bce5d8929286d4c1a001984b11c2578db8#commitcomment-985485
>
'NaN' is a special numeric value (http://www.postgresql.org/docs/9.1/static/datatype-numeric.html), so the code is syntactically correct. Besides, I think it makes the most sense, as the derivative of (i-e)/i goes to infinity as i approaches zero.
GOTTA LOVE DIFF
@alesj no worries I'm applying those changes already
Guys, wait I'm gonna be fixing some stuff in there shortly.
Possibly, I don't know what the code is supposed to be doing :)
How about this document? http://www.gnu.org/licenses/gcc-exception-faq.html
It would be very odd if somebody wasn't allowed to compile proprietary code with GCC...
Looks like this solutions, e.g. using C++x0 standards and shared_ptr will not work in Xcode on macos Lion.
See Qt::WA_DeleteOnClose instead.
Dispatcher.stop() should probably be called in ~pqImplementation() instead
Changes looks much cleaner now, thanks!

So, what is the current way with those coregen files? Does original files give correct timing? I'm sorry for asking stupid questions, as I have little knowledge about FPGA programming.
Well, we agree then.   But note that in this case, the DCM I commented out IS cascaded.   The s3a_icap clk signal is connected to wb_clk in u2_core.v.  wb_clk is the clk_div output of a DCM in the top level of the design.
Maybe there should be also a file for English language?
I think the DoRespawnGO functions use the time in seconds, not millieseconds.
Can you check please :)
I just tried it with `if [ -e ~/.rvm/bin/rvm-prompt ]; then` and it works for me. I don't have rbenv installed, so I don't know if that part works.
And supplementarily fixed in a47b5234a33cdd0885b5fb882be1d66429bd8e2a
Thank you for this temp solution
Kaelima, dont you trust me?
This looks good to me, FWIW!
This was tested under my Galaxy Nexus and Atrix kernel.  The funny thing is that I haven't tested on my G2x yet.  I often times update all of my kernel github sources at once with common patches.  I've been busy and my last G2x release was at least 1 month old.  I keep my sources current on github just in case I forget or something happen to my own local computer.
sure yeah
Looks good - some minor things in commit.
OK, fair enough. I was told that I could do it this way when I spoke to @ajpiano and @darcyclarke, so I figured it'll be solved by a buildstep later on (concatenation etc.), having everything in one file kind of begs for problems along the way, especially maintenance way (just my $0.02). I'll join this into `style.css` tomorrow.
Ah. Looks like the source files are in the main jar because they're part of zanata-common-api's main jar (probably for use with GWT).
This changed called for a major bump since `Connection` is exported.
Sorry for the lack of detail... Later I'll write an example, and I'll attach a screen capture
The NuSVR formulation used in libsvm does use C :-( See http://www.csie.ntu.edu.tw/~cjlin/papers/libsvm.pdf (section 2.5)
awesome! :)
-stan

--
Stanislas Polu
Mo: +33 6 83 71 90 04 | Tw: @spolu | http://teleportd.com | teleportation
is cool




On Thu, Feb 2, 2012 at 1:56 PM, Michael Petrov <
reply@reply.github.com
> wrote:

> We were going to have a slightly larger pull request in a few days with a
> few other improvements. Thanks so much for the work on this project so far!
> We will certainly be contributing back to the main source code.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/tenthbitinc/node-dynamodb/commit/ed3b590609e4a9c524159951fe00f0098daea17d#commitcomment-926248
>
What's the difference between this and `mat[10]` or `mat.at(10)`? Performance?
you can improve this using `Kernel#Array` or `Array.wrap`
Actually, I talked with the person that made the original change and it turns out it was because it did not compiler for him, probably because of using the wrong SDK.

I will change it back to values-v11, since most apps on ICS use the action bar pattern anyway and it is actually quite handy!
These configs will not cause any problems on a kernel that does not suffer from MAJOR bugs.

If enabling this config causes reboots, you have MAJOR problems that need to be resolved.  In a properly working kernel, softlockup detection should never trigger under any circumstances.

I have had this feature enabled since the very beginning, it never caused a problem until I enabled interactive governor.

You'll wind up in a bad state if you band-aid problems (disabling softlockup detection) instead of fixing the root cause (fixing whatever is locking up).
Teleguy traducido a español: https://github.com/Agustin1010/Scriptdev2-Agustin/blob/master/scripts/custom/teleguy.cpp
I'm FAR from getting this done right, if I ever can Jay. Especially considering my Perl skills. No need for being aggressive.

On 4 févr. 2012, at 17:23, Jay Allen wrote:

> You know, in the future, you COULD send a pull request to let your upstream know you had modifications....  Not that it did any good when *I* did so to the movabletype account owner.  Or that time when I submitted all of those issues.  Pfft.
> 
> By the way, I wonder why you broke compatibility with MT 4 instead of simply providing a compatibility module?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/padawan/mt-plugin-field-day/commit/983c3533c77aa5831bbabf363f462400e2e42efd#commitcomment-932672
Doesn't work so well in Firefox...
same here too man
Hi TMorgan99,

Please give me a rationale for changing from home/page to static/home. Specific to your needs? Or a general practice which will benefit others?

-- Daniel
Afaik names should be ordered alphabetically by last name.
Ok. I saw them as 1 digit some spots and 2 in others. I can change it to use 3. 

--

Ben Jackson
bhjackson@gmail.com

@benjaminjackson
http://90wpm.com/about
http://linkd.in/benjackson

Tiny keyboard, tiny replies. Please frogive the occasional typo.

On Jan 31, 2012, at 11:19 PM, Derek Willis<reply@reply.github.com> wrote:

> So here's the thing about party abbreviations: the reason I set them to 3 chars instead of one is that one isn't enough: see ftp://ftp.fec.gov/FEC/cn10.txt for a list of potential abbreviations used by the FEC, for example.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/benjaminjackson/campaign_cash/commit/df7d02873e921303634affa94cc4c1cbc3c6ea53#commitcomment-921313
Why did you change the SDK version to 4.5.1 ?
Is this because of a player or SDK issue ?
Thanks.
rubygems.org への反映は週末2/25,26に行う予定です。
I feel I may have gotten too clever writing the next test.  It could've just gone "1".add.should == 1.

Also, I didn't think I was being too clever with a solution to your test being: self.to_i but then realized that would pass any test I was supposed to write, too.
FWIW: I am a vim geek, and I typically do use spaces instead of tabs. 

I could see that the dominant mode was tabs, not spaces, so I forced all spaces to tabs.

Not sure if this project has a whitespace policy.
You better put this key in a config file that's not shared on github...
This still suffer exactly the same issues than highlighted in the PR.
Well if I see that there're two object methods in which preg_replace was used than it would require to write e.g.:
$regex = new Regex($pattern);
$regex->replace($this->text, $replacement);

I had to do this three times, one time for the url, one time for the email and one time for the cacheCodes. So i write this code three times and if something changes I have to change this at all three places. In this case there're just two lines of program code. But generally I learned that you're supposed to prevent Copy&Paste code. This is the reason why I have implemented an extra method which is called each time.
In addition one of the highest aims should be to write reusable code and avoid writing code multiple times. 

I exactly tried to avoid writing the same code each time. Here a link to a description of the rule I tried to use: http://openbook.galileocomputing.de/oop/oop_kapitel_03_003.htm#mj637b1f986c2e9441d2d305cafc96f6ac
Good man, thanks! :)
This is an EXTREMELY BAD IDEA, for two reasons:

1) __iregex filters aren't efficient
2) this means that any outgoing messages with status 'Q'  and a connection that is an alpha string will REMAIN QUEUED, causing the messenger to hang

You should instead change line 73...if you MUST use iregex, change the line from msgs.values_list to msgs.exclude(...).values_list().

Better still, you could just change the Connection.objects.get_or_create login in handle_incoming, immediately discarding any messages that come in from non-numeric numbers (this would be preferred).
This is a major yuck factor for me as well. I think it's worth investigating why the server returns a response before the user is fully logged in.
Is there a reason for the release name change?
I believe the commit message explains.  I do use a `DecimalField`, but until the model has been saved the raw string is used.  Raw strings are passed in from the form when invoked via the view, thus this test.
Confirmed. You're right. We have `CDIEnricherArchiveAppender`, 
`ResourceEnricherArchiveAppender` etc. in the Test Enricher projects. We 
now have to review Container implementations for refactoring tickets in 
JIRA.

On 05-03-2012 21:49, Aslak Knutsen wrote:
> it used to work like that, but that's a long time ago..  :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/VineetReynolds/arquillian.github.com/commit/e47c5870086e8a4da57acd52d8c4696d657eac57#commitcomment-1043671
you also need to enter those files in the openengsb-framework.xml to be included in the generation. if you've done anything correctly a simple mvn clean install in the manual folder should create the entire manual; including your parts
Ignore this line... It's fixed below.
Try changing this to SND_DEVICE_SPEAKER to solve your speakerphone problem.
Hi Samuel - trying to implement your strategy on rails 3.2.1 and getting this stack trace when trying to list products:
```
activerecord (3.2.1) lib/active_record/relation.rb:500:in `with_default_scope'
activerecord (3.2.1) lib/active_record/relation.rb:166:in `exec_queries'
activerecord (3.2.1) lib/active_record/relation.rb:159:in `block in to_a'
activerecord (3.2.1) lib/active_record/explain.rb:40:in `logging_query_plan'
activerecord (3.2.1) lib/active_record/relation.rb:158:in `to_a'
activerecord (3.2.1) lib/active_record/relation/finder_methods.rb:159:in `all'
activerecord (3.2.1) lib/active_record/querying.rb:5:in `all'
app/controllers/products_controller.rb:5:in `index'
```
You can recreate easily in your example code by upgrading the gem file (I am also using sqlite):
gem 'rails', '3.2.1'
```
group :assets do
  gem 'sass-rails',   '~> 3.2.3'
  gem 'coffee-rails', '~> 3.2.1'
  gem 'uglifier', '>= 1.0.3'
end
```
Do you have any idea how to fix this?
Many thanks
I will check it. My idea was to make it somehow configurable, but if it's possible to load from POM we should definitely go with this option.
Just a spelling issue.
I tried picking everything from CM gerrit and it couldn't build.. I see you still use your overlay with chinese guy camera.. how did you manage to build it? or you builded camera with his overlay patch??
@krames - I think this seems great, but that it might be even better to be more specific right here, something like "The specified wait_for timeout ..." just to make it really obvious/specific. Thoughts?
OK thanks. I can now say for certain it's not needed in analytics.R :)
You don't have to define "template" as it's the same as "List"
Is "\" necessary?
Evan - are you asking to change the return to return false? It is not clear from your comment.
why Tuple2 instead of 2 params?
If that will make the code more awesome then I'm all for it
It will probably change the order on the class path while running the testkit tests, but it has no effect and is not intentional. Just residue from the first rewrite.
Nice catch. It should. Not referenced from anywhere.
I tried some experiments with registerExtension of a test instance of the Cluster extension. I thought it would work if I'd override equals and hashCode to make the two ExtensionId instances be treated as the same id, but that doesn't work because we use ConcurrentIdentityHashMap for the extensions in ActorSystem.

Why do we use ConcurrentIdentityHashMap and not ConcurrentHashMap for the extensions?

Please enlighten me if there is another way of doing it.
The encosing braces can be removed
To me you should do that  on different runs, thus without statistical dependence. I run it on the same contrast for testing, but if you want to test the sum of two contrasts on one model, you simply have to specify a new stat explicitly, not this contrast addition trick.
Note that the Contrast instance in itselm has no clude whether the two instance are statistically related or not.
I wouldn't bother adding this, we support the action API going forward and not the rest API
Might want to add in at least the consumer ID, but likely the repo and distributor ID as well, so there's more context around why we hit this inconsistent state.
And this makes the binding permanent.
I don't know how to improve this, as it stands the only way to get a file's duration is make this call. i.e. GetTvShow/MovieInfo doesn't populate m_strRuntime.

The biggest slow down is the fact file size is never saved in the db either, we stat for the file size currently.
No, because the method `update` (used in upsert) do not return `_id` and I need it.
set_locked, or set_no_access ??
so that the return value is the same with before
Does this relate more to your classes? Visible is really only different in the case of a modal view controller, which TUIViewController doesn't support at this time.
s/, it would be/It is/
s/a color/to a color/
Oh, yes... you're right
hello fiber!
I see what you mean.  This should cover it, but as I ask, is it enough? https://github.com/luvit/luvit/pull/383
Remove @Override annotations for jdk5
8 relative to the `if`, i.e., it need more indentation.
Might add version when deprecation goes to error.
Shouldn't have a semi-colon here - this actually blocks compilation for me

`"Syntax error: Invalid CSS after "78%": expected expression (e.g. 1px, bold), was ";" on line 370 of /Users/jlfwong/code/blog/_sass/screen.sassn.sass"`
вообще комментарии к отдельным кускам метода тоже не помешают. сейчас тут просто какая-то магия происходит
We can say "ActiveRecord::SessionStore is extracted out of  Rails into a gem activerecord-session_store. Setting `config.session_store` to `:active_record_store` will no longer work and will break if the activerecord-session_store gem isn't available". More informative imo
Could these magic constants be documented as well?
It's a def because 2/3 of the boolean contributors will give the wrong answer if the symbol isn't initialized, so it is delayed as long as possible. Not that I know it comes up - one learns to do this sort of thing preventively when attempting to combat these brutal cycles.  It could possibly be a lazy val, but I am very suspicious of local lazy vals in most situations - the tests are cheap, so yes, I expect the lazy val is less efficient.
It is just asking for bugs.  Not at this second, but whenever it is changed.  If you want to put an accent on something, write a comment.  Code is for getting the logic right.

Here is another better way, maybe you'd prefer this.

    (wasDelayed, nowDelayed) match {
      (true, true) => case1
      (true, false) => case2
      (false, true) => case3
      (false, false) => case4
    }
Overloading of these methods can also cause problems, if the optimizations are based on name rather than symbol. But it looks like that is handled correctly now: https://github.com/scala/scala/blob/master/src/compiler/scala/tools/nsc/transform/PostErasure.scala#L55
Could you add a test with multiple `@throws` annotations on one method?
would args* be preferalble to a list here?

def linearmomentum(frame, args*):
    for item in args:
        ...
The problem with "groupby" is that Python already has a `groupby()`, in itertools, and it means something else. IIRC, this was already discussed when sift() was added.

I'm not too happy with sift's Swiss Army Knife interface but I don't want to multiply functions that basically do the same thing. My preferred solution would be to replace sift with the latest version of your groupby(), and change what needs changing in the callers.
The permutation of the last two indices controls the sign, right?
Did you intend to commit these changes? Were these lines causing problems for you? We should probably just specify the `debugger` gem instead.
This lookup should be removed as it relates to tables.php which is being removed.
Hi Nancy, if you edit the modules used, you also have to update the Makefile dependencies. $(OBJECT_DIR)/data_point_routines.o should depend on $(OBJECT_DIR)/coordinate_routines.o
Yep, that was unclear -- thanks, and fixed.
Add a comment on we are using heuristics (16 bytes) to determine string size ...
What's the difference between `{meck_val, term()}` and just `term()`?
@geemus - So that we can track clicks on our end to see how people are using it
Something like `imageMe` would have an `animate` arg it would condition off to set the as_filetype param?
You are only rendering 1 solution for each part?
is_a_collaborator = !collaborators.where{id == user.id}.first.nil?

Note: pull upstream/master before doing this.
Can it get renamed to `spacePath`, so that we can gradually phase out the usage of the term `domain`?
Weird. How does this even work on Ruby 1.8.7? I thought `force_encoding` was 1.9 only.
this is probably my fault, but we don't use the `::` form anywhere anymore.
isn't it that if the document appears in the result set more than once (due to hashing/sorting/query practices) 

the limitation you describe is really an elaboration of a general limitation of geo indexes
Will generate error, please, remove "j".
I don't think this comment is needed. The code itself is readable, and it just seems like it is the kind of comment where someone will change the amount of time, but forget to change the comment.
In the function definition you're specifying `$to_value` as an integer.
I completely forgot about that. Probably update will be broken with custom style. I'll fix it.
Because it's a parameter in the object and not a method like I find in the spec.
Manage attachment extensions sounds more descriptive to me.
This is an ugly interface. What is preventing cache dir only being given to the services that are filesystem-based?
This function does not simplify, and you don't need avatar prefix for namespacing either. Rename to reflect what it actually does.
Please consider moving the ternary logic into a variable for clarity. As it is, it can be difficult to understand at a glance.
`!preg_match('/^[a-z0-9]$/i', $path[1]) || $path[1] == 'api'`

Validation should always be allowing stuff rather than rejecting stuff (whitelisting vs blacklisting). Whenever the code tries to blacklist I assume it's wrong until proven correct (which this code now has been), and even then blacklisting is much easier to screw up unintentionally by someone modifying the code later than whitelisting. Blacklisting is also harder to review as you've just found out.
Should probably have a "return false" here. Currently, the hashtag is being appended to the url when clicking on the notification link.
Please communicate this very clearly on the wiki and to the support team, we will need a guide for reuploading prosilver if people deleted it.
Regarding bantu's comment for this method, perhaps passing a raw flag here would be sufficient.
We never remove the listener at any point. Won't this continue listening to two+ clients if there is a switchup in latency?
Oops, changing languages mid-day problem. I will fix this.
removed the stateFile logic.
E mult prea complicata implementarea asta.Am trecut cu vedera un detaliu care simplifica mult lucrurile.O sa o implementez din nou zilele astea
shouldn't `wouso.interface.chat` also be here?
Just wondering how this will work if we are inserting multiple id's at a time?
ah yes - you have to bind the function to 'this' other wise 'this' no longer refers to the class but the document window. I'll merge these changes and perhaps tweak that so that we can use _getSubElements()
Change this back to 2048. I don't recall why I modified this in the first place.
Is this cross browser? Maybe use JQuery to open a window instead.
The indentation is a bit off
Style: avoid abbreviations for code readability by new contributors
I don't see "arg" being used anywhere except in the first line... why do you need it?

Also, you might instead rename this to something like ```navigateToPage(page)``` and take argument
```PAGES.familyList``` <-- "family"?
```PAGES.familyDetails``` <-- "patient"?
```PAGES.patientDetails``` <-- "illness"?
Thanks, I'll add this improvement in my upcoming change to support additional unit codes.
The changes will significantly tidy this code anyway, as I discovered the unit conversions can be unified in one place with considerable benefit.
"lone_name"?
As discussed offline, it's a shame we can't just make a custom grib wrapper, like in the test above, and send it to the grib rules.
Please consider creating a new issue for improved testability of the rules system, where we can pass in-memory fields
rather than having to test supply filenames.
Without it it went **boom**???
保持代码简单，遵循规范。争取在1000行内就搞定。
This line should be "assert result == 3."
This is a new function to support use of the new Op.connection_pattern
'import'?
Yes, but with the matching camelCase "rankLower" and not just "later".
Does buf need to be flipped before being used to create the volttable?
self.groups = groups instead of self.groups = [g for g in groups]
I know 1000 ms is plenty of time for the fsm's to complete, but should this give up after N tries?
you should add use statements for these classes
Got it. Pull request comin your way.

From: Ben Combee <notifications@github.com<mailto:notifications@github.com>>
Reply-To: enyojs/layout <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tue, 18 Dec 2012 10:39:49 -0800
To: enyojs/layout <layout@noreply.github.com<mailto:layout@noreply.github.com>>
Cc: Chris Patalano <chris.patalano@palm.com<mailto:chris.patalano@palm.com>>
Subject: Re: [layout] Fixed: (#114)


In list/source/List.js:

> @@ -96,7 +96,8 @@ enyo.kind({
>               onRenderRow: "rowRendered",
>               ondragstart: "dragstart",
>               onflick: "flick",
> -             onwebkitTransitionEnd: "swipeTransitionComplete"
> +             onwebkitTransitionEnd: "swipeTransitionComplete",
> +             ontransitionend: "swipeTransitionComplete"


Yeah, I'd pull it out of dispatcher just to avoid confusion.

—
Reply to this email directly or view it on GitHub<https://github.com/enyojs/layout/pull/114/files#r2452734>.
For now the changelog style is to explicitly credit contributors with a "Thanks to <name> for the patch" type sentence at the end of the line. If you don't want to be credited that's up to you, just wanted to make sure :)
Ага, поправил. Ну и ` p_value[:1] == '.'` тоже для единообразия привёл к `p_value.startswith`.
Admittedly subjective, but it's pretty easy to see that the following code is connection related. You don't have to comment things like that, IMO
Just because it's not the source, doesn't mean it shouldn't follow the style guide LN1-9
Somewhere i saw this before )) - https://github.com/libgdx/libgdx/pull/66
This change doesn't make sense.  I'll revert these format changes.
Reading existing hpp files, the convention seems like MADLIB_MODULES_CONVEX_ALGO_IGD_HPP
It should be ok as it stands, but I wonder why you didn't make it major.minor.patch
Remove this; Homebrew can glean it from the URL.
I think it would be better to just raise an error from get_riak_manager if `use_riak` is false and skip this bit and the `RiakDisabledForTest`?
argN is supposed to be an optional array as I see. If so, it should be marked as {Array} [arg]
Hmm ich glaube insgesamt trotzdem, dass wir den meisten Leuten einen Gefallen tun, wenn es standardmäßig "de" ist (man kann es ja auch sehr leicht ändern).

Falls man mit der Browser Sprache vergleichen will, kann man ja auch schauen ob die locale mit dem clang code beginnt oder so.
Es wird jetzt eine Exception geworfen, wenn die Cache-Datei nicht generiert werden konnte.
This makes the `({percentage}|{length})` clause non-optional. This regex matches lengths and sides, whereas I believe the spec requires a length *or* a side.

http://www.w3.org/TR/CSS21/colors.html#propdef-background-position
Seems like I shouldn't have had to do this, but it seemed the easiest way to get the integration tests to pass.
I prefer to avoid in-method comments. Code should be its own documentation inside a method ;)
The same applies here. You're taking a strong opinion on what the user is supposed to do in its app.
We're not sure that the index action will be the default one, etc.

Maybe it should go in Ariane's settings or something like that.
And most importantly, static checks!
This can be in a `<profile>` so that you can do `mvn clean test -Pspdy-tls` if you'd like.
Fixed by escaping unicode characters before transmitting through signal and unescape in slot. New multibyte Emoji caused this. It occurs if transmitted as str, however if the string in inside a list or dictionary it works okay. So for str cases, escape unicode first. I updated sendMessage, setting status, changing group subject and update contact status signals. I removed any count parameters you added.
could you change this back to var. we've consciously not used the comma operator in variable definitions.
Fixed in latest commit 2d886e29734f9f55b75db91095880c5d109ff678.
Safely?  i.e., is there code in this file that really does need that license in order to be safely included?
Should this be User.find?
No worries. Better to get it right!

I'll do it for PUT and POST only. That makes sense.

How about throwing an error if this._payload is set instead of just putting it in the if?
I know that expectHints() has been thoroughly test by this point, but won't putting expect(hintList).toBeTruthy() here prevent an exception in the case where hintList is undefined?
Not sure I understand this comment.  Was it supposed to get replaced with another constant?  (Placeholder for a placeholder, if you will :-)
A more precise translation would be "Fehler beim Erstellen des Datei Öffnen Dialogs. (Fehler {0})". The dash is not necessary.
I think you meant to use the `extensionPath` var in the call to `getDirectory()` here.
Changed to suppressToggleOpen