labofsahil/github-event-dataset-2012 huggingface dataset:
Fix broken specs

This fix answers issue #17
https://github.com/RailsApps/rails-prelaunch-signup/issues/17

Here are the failing specs when I ran `rake spec` right after cloning to my local repo
```
~/rails-prelaunch-signup $ rake spec                                                                                            ruby-1.9.3-p194@rails-prelaunch-signup  [5s] 
/Users/pmc/.rvm/rubies/ruby-1.9.3-p194/bin/ruby -S rspec ./spec/controllers/home_controller_spec.rb ./spec/controllers/users_controller_spec.rb ./spec/mailers/user_mailer_spec.rb ./spec/models/user_spec.rb
..........FF.......




Failures:




  1) User password validations should require a password
     Failure/Error: User.new(@attr.merge(:password => "", :password_confirmation => "")).
       expected valid? to return false, got true
     # ./spec/models/user_spec.rb:70:in `block (3 levels) in <top (required)>'




  2) User password validations should require a matching password confirmation
     Failure/Error: User.new(@attr.merge(:password_confirmation => "invalid")).
       expected valid? to return false, got true
     # ./spec/models/user_spec.rb:75:in `block (3 levels) in <top (required)>'




Finished in 0.33531 seconds
19 examples, 2 failures




Failed examples:




rspec ./spec/models/user_spec.rb:69 # User password validations should require a password
rspec ./spec/models/user_spec.rb:74 # User password validations should require a matching password confirmation




Randomized with seed 37430




rake aborted!
/Users/pmc/.rvm/rubies/ruby-1.9.3-p194/bin/ruby -S rspec ./spec/controllers/home_controller_spec.rb ./spec/controllers/users_controller_spec.rb ./spec/mailers/user_mailer_spec.rb ./spec/models/user_spec.rb failed




Tasks: TOP => spec
(See full trace by running task with --trace)
```
add some repos

collective.cmcicpaiement
collective.footerportletmanager
collective.portlet.nivoslider
useless but fun goodie, based on a joke in the perl community

rafl is so everywhere, he should be easy to find, especially with DDG! This goodie keys off his name and displays a random fact about the ubiquity of rafl. See here for more info: https://metacpan.org/module/Acme::rafl::Everywhere
French localization file
LED notifications.

LED notifications.
Update app/helpers/bootstrap_flash_helper.rb

the biggest difference between "x" and "×"
Fixes an error with asset compilation

The following error is fixed:

    rake aborted!
    Invalid CSS after "...nd-image:url(""": expected ")", was "/assets/twitter..."
      (in /home/phoenix/investopresto/app/assets/stylesheets/application.css.scss)
    (sass):1311
[WIP] Finishing the planned functionality.

The clean up PRs were a step in the right direction, but because ZF2 is now 
stable, I will continue with a more aggressive clean up. 

Changes:
- The profiler and related parts are now located under the Profiler namespace.
- Its uses the event manager to decouple functionality like the collecting 
  process from the profiler.
  - New profiler events: `bootstrap`, `collect`, `access`, `finish`
  - The `bootstrap` event allows modules to do thing only if the profiler is
    enabled. The matcher will listen to this event, with a higher priority, and
    stops further propagation if the defined pattern does not match.
  - The `collect` event is used to start the collecting process. It will also 
    be used flush the reponse early, if possible.
  - The `access` event only occurs if any profiled data should be send or was 
    requested from the browser, like the toolbar. It will be triggered in the 
    `GuardListener` or during the controller dispatch (see ZendDeveloperTools 
    `AbstractActionController`). If the access was denied (
    `ProfilerEvent::denyAccess`), the rendering will be interrupted or in the 
    controller case, it will return a 404 response. This is done to prevent 
    the module from exposing itself thru the internet.
  - The `finish` event is used to send the toolbar/FirePHP console output or 
    save the collected data.
- The toolbar will support ajax based application, which is currently not the 
  case, but the initial toolbar loading will still happen without JavaScript.
- Simplified the `CollectorInterface`. A collector can now implement additional
  features such as `ToolbarTemplateProviderInterface` (ugly name, I know. Maybe shortened to TemplateProviderInterface).
First working version of spheroidal scattering
[jruby] dont validate peer when using jruby, quick fix for this lib
[FIX] Not sure why the : was there, it broke the step.
Merge fix for 'random boo compiler errors'

Fix is clearing array class cache of generic classes that we are deleting when unloading an image.
Removed GetSatisfaction
Style changes

Some style changes for motorways, waterways for *_link roads to be drawn behind other roads (I found it unsightly where a _link terminated on a roundabout)
don't force thrift in classpath


check Thrift class after protobuf so that MultiInputFormat 
does not require thrift on classpath unless needed.

Tested with loading data with ProtobufPigLoader() with out thrift in classpath.
Added support for yesterday features (http://www.wunderground.com/weather/api/d/docs?d=data/yesterday)

...r/api/d/docs?d=data/yesterday)
Remove rotation and erosion.

1. Erosion doesn't really make sense in the context of CZML, it's just a demo feature in Cesium right now.
2. Billboard rotation can't really be expressed in 3D with a single double, we'll revisit it in the future.
Minor updates to threejs module

Updates to Quaternion, Camera, and PerspectiveCamera. Added OrthographicCamera. Feel free to remove &lt;script&gt; from .gwt.xml
Set up as a PyPI package

As per [#2335](https://tracker.receipt.com/issues/2335), turn this into a PyPI package. Now it's up on the PyPI site at http://pypi.python.org/pypi/GraphiteReporter. 

You can install it with `pip install GraphiteReporter` and use it with `from graphitereporter.graphite_reporter import GraphiteReporter`, because Matt and I couldn't think of a better module name.
autocomplete additions

Just FYI, added launchManual() and refresh() methods, as well as an afterItemSelect callback.
submitted my portion and created a groupII folder
removed need for versions.cfg, merged to milestone

I removed versions.cfg from "extends"
Versions can be set in the buildout.cfg versions section OR in setup.py. Also removed duplicates.
Event delegation and triggering of original event

- Using event delegation, to support dynamically generated html (Ember, Backbone, Batman.js etc).
- The original event is triggered upon confirmation, to avoid callbacks.

Feel free to cherry-pick however you like — from nothing to all of it!
Get gatein-portal-container artifact in a more generic way

I wanted to test my portlet on Tomcat instead of JBoss. So I changed few things. Among these, I had to change the artifact "org.jboss.portletbridge.testing.gatein:gatein-portal-container:war:jboss-as7:1.0.0.Alpha1" in gatein-arquillian-extension.

I tried to change it in a more generic way. With these changes, I only have to add a gatein-portal-container dependency in my pom.xml, whatever the version is.
The only constraint with this version is that the final name of the artifact must start with "gatein-portal-container". Not sure this is the best way, seems a little weak, but I didn't find a better way so far.

WDYT ?
Added Instructions about the I18n support to README

Just added some instructions about the I18n support, e.g., including and using translations and creating custom ones.
Fix: entry for infoPList file in xcode project settings might be absolute

In newer versions of the Xcode IDE the path to the infoplist is prefix with ${SRCROOT}. With that change the plist file is denoted as absolute path.

In this case we must not add a prefix (pointing to the folder in that the xcode project is built) to the infoPList file.
Pass request to related serializers

Related serializers may need access to the request to properly serialize
a child resource.  For example, reverse() in djangorestframework.reverse
uses request if available to return an absolute URL.  While the parent
resource has access to the request to generate the absolute URL, the
child resource does not.

A colleague discovered that child resources of a resource were not generating an absolute URL.  It was due to not having the request to pass to DRF's reverse() which calls request.build_absolute_uri() if request is present.

This patch fixes the issue by passing the request to related serializers since they are separate objects that do not inherit the request.
Russian translation, fix player scores.
Replaced an empty level with Siege (a new level)

As the topic says, I simply changed the empty level Horizontal8.bmp by a new one Siege.bmp 

SIEGE LEVEL
You have to siege a castle with MULTIPLE RESOURCE-TILES. Be the first one to get 100% of the resources and make sure your opponent is not taking the others !!
Removed 'Update Levels' button

Because the game already loads extra levels by default, there is no reason to use it. 'Update Levels' is also misleading and suggests that the game will download new versions of the maps. The code is only commented out just in case.
Moved back button to a better location
Fixed selection and custom UI Example
Fixes outputDir build option to be correctly utilized

*) Fixed outputDir build option, which was not being applied when files were written 
*) Now normalizing the outputDir value upon set
*) Creates output dir(s) before writing output
HHH-4439 - Override auditing behavior of properties defined in @MappedSuperclass

Patch and tests for HHH-4439 JIRA issue.
Link: http://opensource.atlassian.com/projects/hibernate/browse/HHH-4439

Regards,
Lukasz Antoniak
Minor improvement on exception messages for matrix dimensions

Hi, 

Just thought it would be cool to have matrix dimensions specified in the exception - to have an operation against nonconformant matrices crash with "Matrix dimensions must agree: op1 is 2x1, op2 is 1x3" rather than just "Matrix dimensions must agree". 

So, as Math.Numerics is so great, I decided to go an extra mile and changed (where applicable) all the places where you throw that exception. Hope it will be helpful. 

It's my first use of Git at all, so if there's anything I could have done better from "the process point of view" please let me know!

Thanks,
Andrew
Fix issue #13.

Implements creating inactive users (with appropriate docs section).
lol

fb
Updated requirements checks
Added Jekyll-powered github page based on new design

I've converted Ollie's (@Oliverrandell) files to use Jekyll. Github will render the templates into static files.

The design is based on James Sann's (@HooverDST) design.

At the moment, the only "dynamic" part is the news section. You can add a post (preferably in markdown format) in the _posts folder, and a summary will appear in the news section, as well as a full version in its own permalink page.
this is 7.x-1.x

Try this. There might even be a tarball at http://drupal.org/project/commerce_dps if the robots are still awake.
Config file is never read fix

Just change the arguments in the call of phantomjs helper ([issue #18](https://github.com/jasmine-contrib/grunt-jasmine-task/issues/18)).
Implement sign in feature

Add sign in feature acceptance tests and steps implementation using devise.
typo

Another question:

**I got no segmention faults when I run segfault.py......**

It looks like numpy has changed a lot
Fix build in PHP 5.4
Namespaced events and immediate transitions from onEnter in default state

Hi. 

I've added a couple of features:

- namespaced events were not working correctly because the namespace part wasn't being included in the lookup.
- I want to be able to immediately trigger an event in my onEnter callback in my defaultState which I also subscribe to as an exit from that defaultState. This way I can set up an 'initialization' state which the machine can self-transition out of, never to be returned to again. 

What do you think?
Reworked the gun system to handle hostage scenarios better.

Ported and adjusted with permission from Lost Worlds.
Remove Win9x references & more

## Done:
* Remove references to unsupported systems (WinMe is also not supported, right?)
* Update / remove some system requirements
add DB Dialects to config

add DB Dialects to config
Fix bug which cause errors when using STI classes

Found that when implementing acts_as_async in Ringadoc ;-)
Provisioning Yumrepo["varnish"] and ensuring the latest varnish version available
This should integrate Fb checkin
Rudimentary Diff Submission and Display

Diff display works great, diff submission is incredibly untested.
Use GET request for api key regardless of other datas request method.

Allows api keys to be sent as a GET request regardless of what the other datas request method is. For example when using I/O Docs the api key is always sent as a GET request regardless of what the HTTPMethod is set to. This causes codeigniter-restserver to throw a Invalid API key error on methods that use a request other than GET.
Added Version 1.0 of PAST&E Project Plan.doc and .pdf, and the WBS.mpp
Remove 20 mile distance option
added Year Validator

Basic validator to check for a year matching yyyy (4 digits).

I think we should keep this pull-request open to work with as a feature-branch.

//review @jone
inline assembler support for AVX instructions


 * AVX
 * AES
 * FSGSBASE
 * FP16C
 * FMA
dbieber's page

added a page for dbieber. hope I didn't screw anything up.
LPS-25607 Guew user can see the "Edit Structure" in Template
r28.01

MVC Map structuur volgens Life
Add Project.add_to_digest

This allows libraries to modify the generated tmpdir name so they
can make the pipeline rebuild if the library version changes.
Modified version to support Django 1.4+ project layouts.

Modified version to support Django 1.4+ project layouts.
Fix changes to data being lost in after_get callback for single record queries

Here's a basic example of a callback I'm using:

```php
protected function convert_date_to_dmy($data)
{
	if (is_object($data))
	{
		$date = DateTime::createFromFormat('Y-m-d', $data->date);
		$data->date = $date->format('d/m/y');
	}
	else
	{
		$date = DateTime::createFromFormat('Y-m-d', $data['date']);
		$data['date'] = $date->format('d/m/y');
	}
	return $data;
}
```

Even though this will return the correctly converted date format, that data is only currently used in the ```get_all()``` / ```get_many()``` / ```get_many_by()``` methods. This quick fix makes the ```after_get``` callbacks work as expected for the ```get()``` and ```get_by()``` methods too.
Fixing ClassCastException using testClasses

This fix the issue 204 I posted: http://code.google.com/p/maven-android-plugin/issues/detail?id=240

Bye

Tristano
Preliminary classes & teachers support

I'm just getting this out there so that frontend people can start working on it. Note that because I wanted the smallest frontend possible for testing, some things (i.e. authentication) are missing; they should be pretty easy to add once we have the frontend going.
Fix issue #6 - data/greeting race condition (w/testcase).


Patch includes testcase.
Update Form/Type/AuthorizeFormType.php

Patch for compliance with latest Symfony 2.1 updates.
Add macloader for t769.
syndication done for pone 4.3 in this product to be able to use it in plone 4.1 and 4.2
Pull in nova_dashboard fixes and keystone id fix. [1/2]



 chef/cookbooks/nova_dashboard/recipes/server.rb |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)
actual version with merged refinements 

... :)
French translation and edd.pot updated
Increase index.html test coverage and update test with internal JSON resources
Timob 7681 1 8 x

This is actually to fix TIMOB-7681 and replaces PR 1486.
SRS bounding box adaptions
Adding extra formating options for toFormat

You can now use toFormat to get short and long form day and month names.
Update src/mod_Loymonium.java

:)
Part12 fixes for clarity

Fixed two things :

  * the first example didn't work, the duplicate name '`HttpKernel`' makes the import impossible. I understand that this example is not supposed to be followed but it's here and most people will make the modifications to `sr/Simplex/Framework.php` and `web/front.php`. For the sake of don-t-panic-the-newbie, all the code presented should work ::)
  * Linked to the first point, expose the final code of `src/Simplex/Framework.php` to make sure everybody gets it.
J'ai rajouté des accents sur tes colonnes, t'as un clavier azerty mec ;)
Fixes and tweaks for v0.1
Fix warnings.

- Removed some bad code that was attempting to remove a view controller from a
  view.  This eliminates a warning.  However, the surrounding code should be
  re-examined carefully -- the comments not match the code and it is actually
  unclear what the purpose of the code block is.  It looks like the code should
  remove the canvas view from the dictionary, but that is not actually what the
  code is doing.  It seems to merely decrease the reference count which may
  result in the canvas view's reference count dropping to zero before it is
  removed from the dictionary.

Reviewer: @aogilvie
two bugs related to filenames fixed

Hi Jay, I saw your bug description on the Sublime Text 2 forum the other day and fixed the bug (and another one for filenames containing single quotes).

Best
Nathanael
Add support for empty files inside tar files

When trying to untar empty files before this patch, it claims that it have an unsupported block and it hangs.
fixing browsing plate error, close #7862

Hi, could you please merge bug fixed in sprint 7.
Fix copy_file and a typo. Fixes #244
Restcallbugs

Fixes for a number of bugs (include #16 and #26)
First few tests improved and fixed.
Lps 25135

minor code reformatting @sustacek
Changed Links to buttons to avoid page refresh.

I changed the close buttons from links to buttons.
This prevents the page from refreshing when you close a message. 

From the bootstrap docs....
```html
<div class="alert">
  <button class="close" data-dismiss="alert">×</button>
  <strong>Warning!</strong> Best check yo self, you're not looking too good.
</div>
```
See https://github.com/rtomac/robotframework-selenium2library/pull/68

Optimize get_selected_list_label, get_selected_list_value.

Create new functions select_from_list_by_index, select_from_list_by_value, select_from_list_by_label.
Create new functions unselect_from_list_by_index, unselect_from_list_by_value, unselect_from_list_by_label.
Those are faster than original select_from_list and unselect_from_list.

Hope I did well
Merged in changes I made without messing with defaults

I merged in my changes and kept zombie_passenger_killer.rb pretty much the same as your current master.

The diff of the files is here: https://gist.github.com/1777295

I needed to make forward commits to revert some of my changes to match yours, but I think you'll be happy.

the name 'reaper' is in no way permanent, please change it to whatever you'd like, or I can change it per your request.

I tried to fulfill everything on your TODO list.  Please let me know if I can do anything else.
Jellybean

fix for torch
This adds the whitelist & blacklist support for single object changes.

My current whitelist is at http://osm.trail.pl/edits_whitelist.txt.bz2, was a little big for committing to git.  I have no blacklist at this point.
claimed facts confirmed
Adding .gitignore file to project dirs.

* Added .gitignore files in project directories.
* bin/ and gen/ are ignored.
Entrega para o Adriano prévia - Caio

Entrega para o Adriano prévia...

Preciso do seu merge e que vc de um pull para que eu possa ter o conteúdo atualizado.
Changed Robot class initialization in constructor.

It now reflects the same procedure as in the simplest example of Darwin's SDK.
- added code to change current directory

Important remark:
The process needs to be run in /proc/self directory. Probably due to some shared libraries. Or something like that ^_^

I believe this solves issue 1.
Modified the Sample controller to run with the new Servo calibration
Fixed init of sensors extension
Fixed InvoiceService.addManualPayment documentation

The API documentation was missing Sample Request and iSDK examples for InvoiceService.addManualPayment (it was displaying information for InvoiceService.createInvoiceForRecurring instead.)
[doc] `jitsu logs`
timob-8177: Titanium Mobile Sample: Sample.Todo does not run on Mobile Web

https://jira.appcelerator.org/browse/TIMOB-8177
Add Request

Hi,
I'd like to support this manifesto and I added myself at the end of the list. I hope I did it correctly.

Cheers,
Stefano
Fix to admin approve on plugin

You may want to remove message_id altogether, but I leave that to you.
Fix wifi & usb tethering, add GT-I5500 support, other small fixes

Will require some changes to proprietary_vendor_samsung (pull coming soon)
Allow users to save screenshot by right clicking -> "save file as..."

Updated the extension to create a temporary file called screenshot.png that can be saved with right click -> save file as...
Removed the dependence on the regular expression match when converting dataURI to blob.
Updated README to explain how to install the crx into Chrome.
using 4 4 4

test line 2
Dev

Added Windows support and auto run Applescript (still not working perfectly tho).
At least he’s _trying_.
Defaults should be specified in the function's argument list (and have to be at the end of the required arguments as a result)
Could use a comment here that $articleId is an article ID to exclude, since that's not intuitive.
(Elsewhere we're using ===)
I couldn't verify because shortly after showing the pop-up, the client was crashing for another reason (the `setupSpriteDisplay` crash).

As far as I remember the pop-up just closes, giving you a chance to still chat and possibly get some help from a GM to warp you to safety. The Viewport will be gray as long as no map is loaded: https://github.com/mana/mana/blob/master/src/gui/viewport.cpp#L100
How about using an template listener which the user package could use if it's loaded and so include the avatar (which is absolutely important in the ACP x))? ^^
This function operates on a lower level, as it also applies to in-memory settings (note that `InstanceConfiguration` is no longer an ActiveRecord model) - so I figured persistence layer validations were inappropriate.
This changes the meaning of the code, aside from the switch from private to public.

The original code uses delegation to make `#attribute` always do whatever `#read_attribute` currently does — so if the behaviour of `#read_attribute` is later wrapped or overridden, `#attribute` changes too — but `alias` binds `#attribute` to the *original* definition of `#read_attribute`, so these two methods will go out of sync if the behaviour of `#read_attribute` is modified. This could cause bugs if `#attribute` is now part of the public API.

See #4408 for a related problem.
iirc because the deprecation is only on the 4.0 branch.
If everyone is doing it wrong, it's just a reason to read documentation.
ok, I will fix that.
Thought what if the body of an inline function is in a cpp file and you use link time code generation?
That must have slipped through my cleaning process.
I put asserts as a way of failing quickly because test suite for sure wouldn't catch every corner case (and recent sbt bug was an example of that). On the other hand bugs will be bugs and they will be discovered by users eventually (just might be a bit harder to locate). 

I will fix it soon.
Ah you are right.  The critical iterator is the DbIterator which has the polymorphic call sites removed.
emberjs/ember.js@1d964782c0620ce52c7cd574c5cf1b4a221384dc
Bon voila un petit push pour te montrer où j'en suis.. J'aimerais bien t'expliquer quoi, comment et pourquoi ce que j'ai changé, mais à l'écrit j'ai un peu la flemme.. Donc je te ferais un debrief orale bientôt. Pour l'instant les seuls trucs nouveaux sont dans _rpc/object_ si tu veux jeter un coup d'oeil : 

 - RPC est une classe qui convient à n'importe quelle usage de rpc
 - KadOHRPC étend RPC et correspond à notre usage des RPC (c'est donc là ou on check par exemple l'id dans les params)
 - PingRPC, FindNodeRPC, .. étendent KadOHRPC et correspondent à des adaptations pour chaque type de rpc (traduction de nos Peers, etc en 'rpc normalisées', check des paramètres etc..).

Les RPC Query et Response sont traduites en 'rpc normalisées' pour  ensuite être adaptées au protocole dans le reactor.

Il me reste à adapter le reactor.
Thanks, should appear in the next commit.
Please do not use master branch, create your own. I will not merge a master branch next time.
Shouldn't it be `code.is_a?(Symbol)`?
I'm happy with this.
http://api.rubyonrails.org/classes/AbstractController/Layouts/ClassMethods.html#method-i-layout
I'm not too comfortable with these exceptions... anything else we can do instead of this?
Is proc different from Proc.new or it's just a style preference?
If you are not going to display anything, all that preceding work was wasted.  It would be nice not to indulge in huge amounts of effort to do nothing.
Could not the min and the max value be defined in the xml?
You're right, that was an error in the original doc.
@Ablu: Should be fixed in: https://github.com/Bertram25/mana/commit/8c14c15970bebbcb069147635c4873650c7171ac
"From" is dubious here -- we probably want "into" or something like that.
Is it "not a great idea" enough to file a bug to refactor it sometime?
We should add a deadband of perhaps ±0.1 on the joystick values.
This doesn't seem to check the particular assignment; just that the two tasks are assigned to the two nodes.  Can you add a comment explaining that?
HI Kevin,
 Thanks for your suggestion,I have made the required changes and also used the updated fork and committed the code in https://github.com/sphule/openemr/commits/mmfsystems-add-button-demographics-updated ,could you please check so as I could commit other bugs with the same updated fork.

Regards
Devender

----- Original Message -----
From: "Kevin Yeh" <reply@reply.github.com>
To: "Devender" <devender.singh@mmf.com>
Sent: Friday, January 6, 2012 7:57:44 PM
Subject: Re: [openemr] Committed changes in addlistitem.php as suggestion made by adding xl() function and addslashes funtion for code optimization (df7cfcd)

You should consider using json_encode
http://php.net/manual/en/function.json-encode.php
I think it would be less error prone than concating string and syntactically simpler, especially if you have more than one parameter. It's easy to mess up the quotes and slashes
So this becomes
echo  json_encode(array("error"=> xl('Record already exist') );

--- 
Reply to this email directly or view it on GitHub:
https://github.com/sphule/openemr/commit/df7cfcd7fcba9ca91c6fb739a4b4eea23f9671fe#commitcomment-843795
It seems like it'd be better to just go ahead and throw a ENOSYS here.
nice. works perfect.
[`fs.watch` is currently broken on OSX.](https://github.com/joyent/node/issues/1986)
I've checked target/test-data, now it's ok but for some reason 2-3 runs were blocked.
no spaces around = after help
No freezes or crashes related to LFG yet. @raynar should test #551

For me to master asap.
thx for explain
tst12, судя по ответам ты нифига не понимаешь о чём я!
"с личными разборками и наездами - все в сад." и это правильно =)
I do this before and i recompile it on windows (ad.exe & vmaps.exe) but same problem .
Can any one go to this NPC id : 18326 , and cast this spell : 32663 , after that check server log ?
I do revert to mr1611, but still crashing but a less :S
This is 1.9 style lambda syntax. Is mongoid giving up support for 1.8?
So you suggest separate them for now?
http://travis-ci.org/#!/symfony/symfony/builds/557840
Oh,  and bholes don't eat nuke-disks anymore and their sprite appears above the smoke they create and is semi transparent. If spirited properly (i.e. not by me!) it could look really good.
Think of all the kittens killed because of this commit message! The cute little kittens!
Hi,

regarding commented tests with message "uses real connection":
Tests containing this message require access to a real device, and will fail if there is no connectivity. That's why they are disabled.
In the case of proteus ROADM, if you still want to run these tests, you'll need to set up a vpn tunnel to the network where they are. Just speak with me and I'll give you access.
Same as comment #1
Does liveTV work in 0.24 + fixes?
Yeah, we could expand this later to use ADDON_WATERMARK_STORAGE or something and then import that class for watermark code. Currently it looks like Django only supports one storage class though.
<troll> you should know that apt >> yum </troll> ;)
This breaks things locally.  The version of gdata we used til now doesn't seem to have this module.

`pip install --upgrade gdata` seems to fix things.
Yes we should bundle an implementation.  But for windows, we also have to rewrite the call_compiler function to make it work.  And probably some other things too.  I mostly develop on Mac and test on linux.

If windows support is important for you, I will bump up the priority on that (which for now is: to do later).
I've looked at the code and I can't see any way of doing this without CMAKE_BUILD_TYPE being set properly without big changes. Does it work properly if you just run it with Debug or Release mode being passed into cmake rather than just as arguments to cmake_install.cmake (which seems like a corner-case)?
Definitely - can you make the change on your end and I'll pull the request?
excellent opportunity to use `<address>`
## Before

Array
(
    [module] => pages
    [amp;file] => pages.js
    [amp;language] => ru
)

## After

Array
(
    [module] => pages
    [file] => pages.js
    [language] => ru
)

Before correcting system answer "file not found".
This looks like a typo to me
this is already imported on line 5
Y U NO HAS TESTS????
such file streaming shoulnd't be required. place your files in an /files folder and the core will copy those files into /files/addons/...
Sorry, not good enough yet. Renaming an inner type or a side type having at-Data or any at-XxxConstructor will cause the first n letters of the annotation and the following code to be overwritten by the new type name , where n is the length of the previous name.
1 and 2: I hadn't thought of making it a plugin, but rather as a generic service provided by the core, but I can easily change that if you like. (I thought of configuring it internally via the settings object, but if you want it as a plugin, you'll get it as a plugin :)

3) My initial thought was that the engine itself would call cancel() on all it's tasks during shutdown... you don't like that?
makes you feel uncomfortable, doesn't it?
Excellent. Looking forward to your feedback.
this is the new model that Joel is working on that will handle the
relationship visualizations.


2012/1/15 Daniel Lobato Garca <
reply@reply.github.com
>

> I'm a bit confused, what does this mapvisualizations thing do?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rgoraya/wc_master/commit/7cd02f048b952f4a56cb8b8ad5e7217436035aef#commitcomment-868412
>



-- 

Sincere Thanks!

Goraya Raminder Singh
Bren School of Information and Computer Sciences
University of California, Irvine
Irvine, CA 92697-3425rgoraya@uci.edu
"Your app was disabled by a site administrator. If you have any questions, please email app-reviews@mozilla.org."
No idea, haven't decided on names yet :) I was going to talk to you about that
@markdotto: Why no `>` before the `.span*` definitions? I've run into problems with deep-nested `.span*`s within fluid layouts. See pull #1471.
i meant plr->player, codestyle,
 and snake traps is in scripts... 
i think your clear with the other traps haven't fully looked.
What's the purpose of client-auth? Who else would be able to publish messages?
@FernFerret That's the one! I've managed to reproduce that on my server accidentally.
Dreadii, when you attack Muradin or Saufang on the enemy gunship, your core crashes?
There is no need to do this since you are raising the method again.
OK, changed in all places.
sorry, you are right.  we will switch off withXXX in next rev
Actually the whole method will go away, see comments below
and why not keep the PHPUnit expectation for the message ?
1. I do not agree with 'needs to be fixed'.

why can't tests be order dependent after all? it doesn't hurt actually, and I have to see a refactoring case where it became a PITA.

2. Even if you don't care about the reason builds fail ('a.k.a needs to be fixed'), you are supposing that tests are not order dependent because of the 'order random'. But in reality you are just checking a random subset of random (hem..), you can't assert that with the color. 

A green build can then mean 2 things: 

* build is green
* build is order dependent

Which doesn't help.

3. When comparing builds on Travis or with git dissect, (2) will bite you while debugging at 12pm and make you cry your Mum that you really don't give a shit about (1)

I guess.
Yeah, Marius put up with my own learning curve when I did the same. ;-)

If you happen to be on Mac I'd recommend getting familiar with GitHub's UI for managing your branches: https://github.com/blog/1067-github-for-mac-1-2-snow-octocat

You'll also probably want to alias the git fetch/merge/push set of commands anyway.  I do it like this in my ~/.profile:

alias git_fetch_upstream='git fetch upstream'
alias git_merge_upstream='git merge upstream/master'
alias git_push_origin_master='git push origin master'
My Git skills are great...
\You could break out of the loop at this point.  It may be significant for reducing the number of iterations.
It works now I'm all synced up :)
If you wanted every thing in abc order just say so lawls
does this improve the performance?

explicit boxing it v.s. implicit boxing
FINALLY !!!!
Despite my numerous comments, code is looking good for a first OpenEMR submission. Most of my comments are dealing with translating and html-escaping strings. Please feel free to ask me for more clarification anywhere if needed. This review took awhile, so please address everything (either by modifying it or giving reason why not modifying it; doing this helps to keep track of what to review in the next revision).
thanks,
-brady
You shouldn't modify the style.css directly. This is generated dynamically by the assets/less files.
Yes it does, incredibly well! So I'm completely with you on this one and will indeed introduce a couple of _private functions. To be honest, the one to collect voc-names should be part of the taxonomy system anyway but as it looks the whole taxonomy system in D7 is not very mature anyway. As an example it turns out that taxonomy pages do support nodes only and no entities - what a shame. But there is already a team in place to resolve this for D8.
Hello, Mike

function x_file_put_contents($file_path, $data)
{
	if ($fh = fopen($file_path, 'c')) // 'c' is important
	{
		flock($fh, LOCK_EX);
		ftruncate($fh, 0);
		$rt = fwrite($fh, $data);
		fclose($fh);
	}
	else $rt = false;
	return $rt;
}

But... I think, Processing of writing is not considered but considers it more certain to check file size when reading conversely.
I'm not 100% sure we need this but ok. You should put this description in the source as comments. The comments you've added don't say this and they're not enough to understand. Thanks for the explanations.
Actually, I'm wrong. The client listeners are needed, or nothing is sent anywhere.
there's a very simple solution... simply use a fixed version :)
I think you mean 'opacity' :-)
This is currently not an API property; shouldn't it be one?
Это неправильный подход к программированию, если retRe = true значит мы хотим получить в результате регулярку, что бы там об этом не "думала" сама функция. Если она не будет соответствовать хоть какой-то спецификации то таким путём можно и до неуловимых багов дойти.
I agree, especially as it is one of ARD's main interest.
I think that it should appear fairly early in the docs that ARD tends to create sparse weights than Ridge.
`isset()` check seems good.
No this isn't good
Actually, GetStringFromObject is already implemented as
Class<ASString>::coerce(obj)?

2012/2/6 Antti Ajanki
<reply@reply.github.com>:
> Isn't there a memory leak in GetStringFromObject()? getInstanceS() in GetStringFromObject() creates a new object which is not decReffed anywhere.
>
> Maybe GetStringFromObject() could be replaced by obj->toString() because many methods will only access th->data. This would also fix the leak. Otherwise you should do obj->incRef() in the (obj->getObjectType() == T_STRING) branch and the caller should decRef the return value (or return _MR<ASString> so that decRef happens automatically).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lightspark/lightspark/commit/7b064c2377c55b83a3a53e014799a852161ba023#commitcomment-937466
I think you have a typo, it's 
git://github.com/primalmotion/Node.git
not
git://github.com/primalmotion/node.git
add comment to say it's for each test
What do we need curl for here? Is it required and Ubuntu / Debian don't have it by default?
This should be a label associated with the checkbox see my other comment on line 9. It is not sufficient to just put the string in the markup, it needs to specifically in a label tag and associated with the checkbox. Also, I think it should ideally come before the checkbox - think about linear content, see http://webaim.org/techniques/forms/ for more ways to make accessible forms.
yes, it's feature in Confluence to make sure the link will work even if the article is moved/renamed.
Glad to see you here Lord! :D
reviewed by bill
How about something like this?

```cs
private bool this_script_should_run(string script_name, string sql_to_run, bool run_this_script_once, bool run_this_script_every_time)
{
    if (this_is_an_every_time_script(script_name, run_this_script_every_time))
    {
        return true;
    }

    if (is_running_all_any_time_scripts && !run_this_script_once)
    {
        return true;
    }

    if (this_script_has_run_already(script_name))
    {
        return this_script_has_changed_since_last_run(script_name, sql_to_run);
    }
	return true;
}
```
If : simply means /bin/sh then I say we leave the detector alone, but default to /bin/sh.  This was the old default behavior.
``sc2reader.load_replays`` is actually implemented as a generator, so the following loop should have the same performance characteristics. Additionally, a reference to the data back is already attached to the replay. A cool trick for printing out player information is using the (also undocumented) ``player.format`` method as shown below. In addition to a player list, there is also a dual key (name and pid) dictionary lookup you can use to grab the players. Finally, the player class (this part is documented, w00!) has an ``events`` attribute that has already done some of this work for you.

```python
from collections import defaultdict

versiontounknown = defaultdict(set)
player_format = "{name} {play_race}"
for x, rep in enumerate(sc2reader.load_replays(replays)):
    config = rep.databack
    print x, rep.filename, rep.player[1].format(player_format), ":", rep.player[2].format(player_format)

    player1build = getBuild(rep.player[1].events)
    player2build = getBuild(rep.player[2].events)
    for build in (player1build,player2build):
      for e in build :
        abil = config.ability(e.ability)
        if abil["type"] == "UNKNOWN":
          versiontounknown[returnVersion(rep.build)].add(hex(e.ability))
```

Probably at the time you wrote this, the documentation was wrong or missing. It is slowly being filled in though.

Just thought I'd write in some notes while I looked around...
0...0x100 is how you do ranges in ruby. Also, this should be a 0.upto(0xFF) loop.
I write posts where clojure is not the main focus, for examples monthly event round ups.  Using just Clojure would add a number of posts I write about of which clojure is only a small part.  I didnt want to spam Planet Clojure.

If you are happy with the content you get from the Clojure tag alone, I am happy to use that.  If so, is the best approach to delete this pull request and create another one, once I have updated the URL?

I added the missing PlanetClojure tag off the London Overtone hackday, thanks for the catch.

Thank you.
I would suggest to use urand(0, 1) ? SAY_SLAY_1 : SAY_SLAY_2
even if your version will work generally as well :)
I guess I'll make a diff based on the current trunk in your Google Code repo.

While doing this, I can also have a look at implementing load_rom and load_archive in a way that does not require a modification to loadrom.c. 

There are two things I can already mention you could change though -

1.  in ym2413.c - there is this line -

 #define INLINE static __inline__

create an ifdef around this like this:

#ifndef INLINE
#define INLINE static __inline__
#endif

This is strictly done with MSVC in mind, where 'inline' is not supported for C89 - but _inline as a keyword is. (hence why I make a global define in my MSVC solution /DINLINE=static _inline

2. Remove the stdlib.h include in m68kops.h. This was causing problems with the Linux port on compile time. (probably due to it already being included prior either in shared.h or osd.h.
Durrr.... talk about a blonde moment. OK - it is a spell, so ignore that. HOWEVER, there is still no reason for text to be hard coded....
There seems to be a problem with SMART_ACTION_MOVE_FORWARD & SMART_ACTION_FORCE_DESPAWN
 When the creature despawns and respawns again, it appears in that point where it walked. So in these case the goblin respawns out of the cage and the cage doesn't reset its state.
 I was trying to fix another NPC that walks and then despawns and I have the same issue.
 
 Hope the information can help, and sorry for the bad english. Great job Malcrom!
This line causes this application to crash on the emulator and on the device. I don't know what the expected result should be.
Use node instead of node id?
when you type .ser info it shows you Uptime diff...
@jhinkey
Igor Spasić , where 'ć' is latin small letter C with acute(U+0107)
These will also be needed to be added the `jenkins_requirements`
My consensus ;) is unlabeled private first, followed by protected, followed by public.
But yes, I don't think we have same convention, I'm not too fussed, if change, but in this code base (Thor) the norm is as above.
This solves the problem of the usericon background colour in IE8, but does not solve the IE8 problem of not having the colour circle and the box of the colour picker itself.
That's why I switched to xvfb-run and even consider dropping the "headless" gem. It seems to do a decent enough job and clean after itself
Are the table creattion statements anywhere around here?
This still says #qsp.
autoreconf is not working with this? i dont have tried, but if it works we should use this
Agree, will pull soon...
Aproveitar e falar deste tipo de serviço, de você não precisar manter uma estrutura local de integração contínua.
Just out of curiosity, why are we using all lowercase hyphen&ndash;delimited strings rather than `camelCase`, `PascalCase`, or `snake_case`?
Yep, just seen that and adjusted in my merge. For the moment, I've made my temperatures and notification icon widgets 11px high, and the ongoing notifications widget 51px high, and the bucketing seems to work. This approach works well enough for me for now, but ideally, we'd have some more flexibility... it would be nice if ongoing notifications could be set to "the rest of the screen" below a 32px row or below an 11px row, without needing two versions of the ongoing widget.
I'm more concerned about running them easily than using `rake` specifically, so no specific complaints here.
They were only introduced a couple of weeks ago alongside the UploadField API, and never used externally.
Wow awesome. Thanks!!
why not access the parameter value via webRequest[name]? AbstractUrlMappingInfo.populateParamsForMapping should have populated the parameter values, thus

assertForwardUrlMapping("/test/1", controller: "grailsUrlMappingsTestCaseFake", action: "action1")  {
     id = p1
}

would be possible for 

"/test/$id"(...)  {
      id = { "p${params.id}" }
}
# Homework 3

| **Part**  | **Points** |
|-----------|------------|
| base      |  0.5 |
| advanced  |  0 |
| **Total** |  0.5 |

There is nothing specifically wrong with your code.  I would have liked different 3 different math problems but the standards are fine.  The one big thing is you didn't do step one exactly as specified.  You recreated everything it was testing except using a string prompt for raw_inputs.

```python
# prompting on the same line
n = raw_input("Enter a number: ")
```

Here is the full step one code.

```python
# Step 1
name = raw_input("Enter your name: ")
print ""
print "PrissyBot:  Hello there, " + name + "!"
resp = raw_input(name + ":")
print "PrissyBot: You mean, \"" + resp + ", sir!\""
```

The only other thing I noticed is just like `int` and `float` there is no space in `raw_input()` between the word and the parentheses.
Yeah, I would bet that you are testing with PostgreSQL < 9.1 without standard_conforming_strings on,  and that you are round-tripping through a string type instead of the bytea type.
I want a like button for Github commits.
This looks like you've taken away the ability to immediately scrub to a position in the audio when playing.  Are we sure we want to take away that ability?  I know this old way of doing it has turned out to be buggy, but there may be other ways to do it.  Also if iOS supports immediate scrubbing, Android should to.  An alternative would be to *only* do the `seekTo` call in `setTime` *if* the player is in a "legal" state (see the [Valid and Invalid States](http://developer.android.com/reference/android/media/MediaPlayer.html#Valid_and_Invalid_States) section of the Android docs).  If it's in an invalid state for a `seekTo` call, then you can at least set a private variable (named something like `initialScrub` to the value) and check for that in `play`, then `seekTo` there after `prepare` is called.
Wut? This help message is silly XD
Fixes an issue with not parsing some lnk files.
`TEST_BACKOFF_IN_SECONDS` is an int on line 39 in TestBackoff.java, was there a reason for this?
$object !== $admin->getSubject() should do the same.
I don't see any reason for moving that to the view... it was good here for re-usability purpose. Right now, I am using MongoDB, so a lot of custom code, I don't want to use the whole `View`, because many parts are not suitable - I use only `RequestMixin` and `ResponseMixin`, and basically I have to copy paste the whole `View.final` to my code, which is kind of stupid.