labofsahil/github-event-dataset-2012 huggingface dataset:
fix hog.py - orientation calculation and visualization

skimage.feature.hog has 2 bugs in

1) orientation calculation

current implementation

```py
orientation = arctan2(gy, (gx + 1e-15)) * (180 / pi) + 90
```

returns orientation in interval (-90, 270), because arctan2 returns angles in (-180, 180)... my proposed fix is:

```py
orientation = arctan2(gy, (gx + 1e-15)) * (180 / pi) % 180
```

2) visualization

```py
                    rr, cc = draw.bresenham(centre[0] - dx, centre[1] - dy,
                                            centre[0] + dx, centre[1] + dy)
```

bresenham() expects parameters in "y,x" order, correct code is:

```py
                    rr, cc = draw.bresenham(centre[0] - dy, centre[1] - dx,
                        centre[0] + dy, centre[1] + dx)
```


I tested both on this usage case:

(hog, hog_img) = skimage.feature.hog(img, visualise=True, normalise=True, orientations=8, cells_per_block=(3, 3))

my testing image:
http://pavel.campr.cz/public/scikits-image-hog-fix/src.png

master code - result image with HoG visualization:
http://pavel.campr.cz/public/scikits-image-hog-fix/master_hog_visualise.png
http://pavel.campr.cz/public/scikits-image-hog-fix/master_hog_visualise_with_src.png

my proposed fix - result image with HoG visualization:
http://pavel.campr.cz/public/scikits-image-hog-fix/fix2_hog_visualise.png
http://pavel.campr.cz/public/scikits-image-hog-fix/fix2_hog_visualise_with_src.png

when compared, e.g. on the head, some orientations are currently lost, because orientations are in interval (-90, 270) , but processing is done on (0, 180) only... the rest of angles is lost and not counted into histograms

first bug (in orientation calculation) rotated orientations by 90 degrees, second bug (in visualisation) switched X and Y, so that orientations were shown perpendicularly to real angles... those 2 bugs rotated (wrongly) the angles twice by 90 degrees = 180 degrees, so the visualization seemed to be fine, unfortunately
New session type for Facebook deep links handling

Fix for broken FB deep-lnks
Fix deprecation warning for new installations
Fix FacetedModelSearchForm

FacetedModelSearchForm inherits from ModelSearchForm, which means that the selected_facets kwarg is not popped anywhere down the chain until it gets passed to Form, which explodes because its not expecting it.

The fix is simple: make FacetedModelSearchFrom pop the "selected_facets" kwarg just like FacetedSearchForm.
Menu entry bug fix.
1.  Start with a url like:  http://somehost.com?foo=ONE&foo=TWO&foo=three

Hi,  I had problems getting the signature to work when I had multiple parameters with the same name.  The following patch resolved my problem.  Thanks!


1.  Start with a url like:  http://somehost.com?foo=ONE&foo=TWO&foo=three&count=100
   
2. oauth.js, _prepareParameters line 245 converts this to:
   
   {'foo[0]': 'ONE', 
    'foo[1]': 'TWO', 
    'foo[2]': 'THREE', 
    count: '100' }

3.  oauth.js, _normaliseRequestParams line 137 needs to be fixed to NOT include the bracket notation in the string-to-sign:

  argument_pairs[i][0]= this._encodeData( argument_pairs[i][0]);

  changed to 

  argument_pairs[i][0]= this._encodeData( argument_pairs[i][0].match(/^[^[]*/)[0] );
URL title 플러그인 외

- 대화 중 나타나는 URL의 title을 표시하는 플러그인. 과거 #langdev 채널에 있었던 것.

- 데비안 패키지 지나치게 단순화된 버전 정보 변경

- .gitignore
Fix getClientCredentials when no client_secret sent

client_secret doesn't have to be provided to gain an access token.

Prior to this fix the script would throw undefined offset errors with no client_secret when using GET or POST, this commit just gives it null if not set.
Timob 11697: Added hooks to support SSL certificate store module

http://jira.appcelerator.org/browse/TIMOB-11697

Test case in Jira.  

Note that I have not added documentation for these hooks since they are only used for native modules and we have plans to eventually move this into platform.  I created a ticket for this here: http://jira.appcelerator.org/browse/TIDOC-899
[TIMOB-9400] iOS: Remove children when context is shutdown

Test is in [JIRA](http://jira.appcelerator.org/browse/TIMOB-9400)
Added dropdown to support different currencies.
Added reduce and group options to views
adding tests with unicode chars

Some of them don't work with JRuby 1.6.7 with 1.9 mode. This is part of my effort to make the world [a better place](http://www.youtube.com/watch?v=BWf-eARnf6U&ob=av3e) intridea/multi_json#25.
NYC DATA
Unit test refactor

I fixed the problems introduced in b16 mentioned in #1047, and rebased against master.
Describe BadRequest errors better

Mostly a debugging tool, useful for distinguishing what particular bit got wedged in the request. The filesystem paths are never exposed, merely the chunk already visible in the querystring.
Init the action bar during findViewById

Ensure that the ActionBar is initialized when findViewById is called.

Closes #623 and #748.
Resolves #358. Switched Update and Close Ticket buttons.

See #358.
Support for creation and removel of Images added

This commit adds support for creation and removal of images. I tried to follow the logic that your code has for server creation.

Let me know what do you think.
Fix if value in parameters is not NSString type
LPS-23296 - Control Panel menu background gets pushed into portlet background

 http://issues.liferay.com/browse/LPS-23296
Swaped default bootstrap icons with FontAwesome icons

* Unable to integrate `less-rails-fontawesome` through the normal gemspec workflow
* Applications using WoopleTheme must add `less-rails-fontawesome` to their own Gemfile
LPS-31442 Breadcrumb is show in places where it shouldnt
LPS-26266 Recycle Bin

Hi Brian, 
- front-end has been reviewed by @natecavanaugh
- tests have been reviewed by @migue
- javadocs have been reviewed by @jhinkey 
- the rest of the commits have been reviewed by @zberenty, @caorongjin or me.
Updated readme with workarounds for gsl gem install errors

For the people getting errors installing `gsl`
Complete support for banners and phrases.

Обновлены методы работы с объявлениями и фразами, а также соответствующие структуры.

1) Реализована иерархия структур различной степени детализации (в зависимости от режима возвращаемых данных). Это сделано в целях того, чтобы не объявлять большее количество полей в одной общей большой структуре Nullable потому-что они возвращаются не всегда. Теперь если данные присутствуют в структуре, значит они возвращаются и имеют смысл. Т.е. более строгая типизация и структур.


2) Для редактирования записей существует соответствующий тип данных.
EditableBannerInfo - он содержит только те поля, которые непосредственно необходимы для при сохранении объявления на сервере.
Чтобы отредактировать существующее объявления, его необходимо предварительно клонировать в данный тип (реализация правила того, что редактирование изменений необходимо проводить в отдельном объекте, а не в оригинальном, ссылки на который могут использоваться в различных частях приложения).

3) Думал над сигнатурами методов, их длина и количество перегрузок не совсем нравятся, но перебрав множество вариантов (включая работу через <T>), ничего лучшего не придумал.


Пример варианта использования методов:

```
var banner1 = service.GetBanner(63907290);
var banner2 = service.GetBannerWithPhrasesAndStats(banner1);
var banner3 = service.GetBannerWithPhrasesAndStats(63907290).ToEditable();

banner3.Phrases[2].Price = 2;

service.CreateOrUpdateBanner(banner3);
```
Fixed Web.MVC and Web.WebPages assemblies references

CommonJobs.MVC no estaba apuntando a los mismos assemblies que CommonJobs.MVC.UI, lo cual traía inconvenientes si el build se hacía en una máquina con .NET 4.5 y se instalaba en una máquina que no lo tenía.
Save the parent of the last project directory we opened.

Hey Arthur,

Small patch to save the parent directory of the last project directory we opened so that on the next Project->Open it will remember where you were last time.

I haven't changed the (is-mac) path (I don't have a mac to test on) so it won't work on a mac.

Steve
Bug 367036 - Add JSON to the testsuite

I added ChangeRequestJsonTests.java to the oslc2 testsuite. There are 16 new tests to cover application/json requests and responses. The work also involves adding helper methods in TestBase.java.

I tested the work against RTC 3.0.1.1 and CQ 8.0.0.2. There are subtle differences in responses between them in some requests. But both worked well. 

The tests failed against RIO-CM which is expected. 

There are some work left to do:
1) I just realized that there are other places that JSON coverage should be added. For instance,
    CreationAndUpdateJsonTests: is actually using XML 
    Should add ServiceProviderCatalogJSONTests and ServiceProviderJsonTests
    However, some of requests are covered by the helper methods - so I will just continue to address them.

2) I am seeking a more effectively way to parse JSON response (I am not very familiar with whether there is XML xpath type method in some library that allows me to get to the leaf key/value more effectively

3) right now, the calls in the constructor ChangeRequestJsonTests get called before each individual test methods which is very unnecessary - they should be moved to @Before as part of set up code. The reason I did not change this is that other classes do the same thing and I am not sure whether there is a particular reason to do things this way. I am just making a note here so we can enhance later. 

BTW, I ran two "git push" - the second one just contains fixes to remove unused imports. 

Thanks

- Yuhong
Hotfix/use rest message for link2

Improved the way click through works and replaced ~ character with | as well as included random IP addresses
Creando un modelo orientado a objetos

Los botones aún no funcionan y el dibujado en el canvas no lo hace bien (tengo que revisar el offset a la hora de pintar), pero esta es la idea que tenia de orientarlo a objetos. Lo he creado en un fichero nuevo para no tocar tu código a priori.
Cuando puedas me dices que te parece la idea.
Fixed RedisClient.StoreAsHash to use json instead of jsv

Hi Demis,

Quick fix for StoreAsHash functionality.

Dan
Moved post data field definition to its own method

Defining post data now comes from its own method, this allows us to hook in and add additional data as required.

Current functionality is not affected by this change
create kml log simplifier using line length

remove points which is the shortest line in the kml file again and
again until 200 points
Makefile: Use -rpath instead of LD_LIBARY_PATH for xpcom_abi

Hi, please pull my commit about rpath-linking. This simplifies the invocation of the xpcom_abi and fixes cases, where multiple library directories are needed (ubuntu once again).

Makefile:
* Instead of providing LD_LIBARY_PATH for the xpcom_abi program, we link the
  shared libraries using -rpath. This way, the executable will know where to
  find the shared libraries.
Allow to pass default request options

We are creating a [NodeJS browser launcher](https://github.com/ekryski/launchpad) that can run a Browserstack API compatible server for launching browsers on a remote system.

I would like to use the node-browserstack package to test compatibility but the request options were hardcoded. I added a `defaults` key to the settings so it is possible to do something like this:

	BrowserStack.createClient({
		username : '',
		password : '',
		target : {
			port : 8080,
			host : 'localhost'
		}
	});

I already bumped the version, would be great if this is an acceptable patch.

David
NERDTreeStatusline support

followed up on [the issue](https://github.com/Lokaltog/vim-powerline/issues/79) of not being able to change NERDTree's status line with NERDTreeStatusline. 
Credit to [cperl82](https://gist.github.com/1742684) with his gist
update master

http://stackoverflow.com/questions/8646744/jekyll-plugin-not-working
Add boolean input
add collective.geolocationbehavior
classes/response.html, autoloader.html,  classes/file/intro.html, handers.html, advanced.html の翻訳

https://github.com/NEKOGET/FuelPHP_docs_jp/pull/22 を元に修正を加えました。
LPS-24893 Clear mbCategoryId parameter when threadId is used for RSS
support for tooltip-trigger with position:fixed

added new config option tooltips: fixed. this is needed if the trigger element, or any of its parents has position:fixed
Added class DARwInOPMotionTimerManager

This class is used to start the LinuxMotionTimer, which is nedded for the gait and motion managers.
Furthermore it ensures that it is activated only one time.
Fixed - JBIDE-13113 - JAX-RS validator moved into the root of preference...

...s

Setting (back) the correct id to the JAX-RS page, matching the category of the JAX-RS Validator page
use prawn settings when creating a new document

I had the problem that I wanted prawn to create PDFs in landscape page layout.

now you can do this with
```
set :prawn, { :page_size   => "A4", :page_layout => :landscape }  
```
Better DB for NEW SKYFIREEMU 4.0.6

The DB Based on Silvermoondb, ArkDB and SkyFireDB
Version 3.0 del refactor con unos configuradores

Hecho más peola
Update controllers/AdminRowController.php

Fix bug : add "container_id" element in post action (with a new form, the action send to post action)
But, if the element does not exist, the value is not put into the row)

And use extraParam to keep the container into the request
The documentation is a little bit more helpful, now.

This tackles issue #15.
Dat Inheritance

Tanks show up!
fix-close-icon-in-sidebar-92

I thought the X icon was causing a problem, but it seems it's not. So instead I fixed the issue with clicks on images not opening their parent element href. So now clicking the Fluidinfo logo or your avatar navigates the tab to the right place.

Fixes #92
Update website

We haven't used waf in ages. Use cmake more consistently and mention clay.
Course completed

Hi,
I have added the possibility to send email to admin also for a 'course_completed' event. I have also added the italian translation file.

Best regards,
Emanuele Tomasi
Fixing Gallery view, adding a JS fix and adding Auto Complete view

The float removed causes a non-full width grey gallery area, and results in wonky, stepped field elements in both Gecko and Webkit when multiple SBL+ fields are used after one another.

@jensscherbl has also made a fix to the JS and done an Auto Complete view which works like the Reference Link field.
Removed link to a demo which doesn't exists
Removes old javascript code
Rename zip archives for better discoverability and fix issues with a global installation of Mercurial/simplejson
Lerping

I've made quite a few changes since we last got together.  I updated some of our build system so that both "release" and "debug" builds can be made with the same set of makefiles.  Unfortunately, there's a bit more clutter now while things are building, and it /still/ isn't quite optimal - there's plenty of room for improvement there.

Anyway, "make" builds a release build, and "make debug" builds a debug build.  The debug build doesn't force the execution of unit tests every time, and is built with debug symbols allowing a debugger to be attached for runtime examination.

I've also identified and fixed the source of the error we encountered on Sunday.  It's also in this list of commits.  See the commit messages for more information.  The getVelocity() method returns the wrong value for now, but we'll get it right after we finish examining Bridson's code.

I'll have a handle on that by tomorrow night!
[Skeleton] Check if defined datetime fields like index.html.twig skeleton

show.html.twig doesn't check if datetime is defined previous use of |date filter.

Added the same check than already implemented on index.html.twig
Small change to avoid over-specifying the related field selector...

... Overcompensating for such a user error is not worth the performance hit (in my experience).

I have setup a basic JSPerf test to demonstrate the difference http://jsperf.com/over-qualified-selector-performance
Add inspection results controller and view
Added range support

Hi!
I added range support to your plugin, which, I think, comes in quite handy for huge files.
Best,
Nils aka Akron
Checks log file exists

Just checks log file exists, if not says so and exits gracefully.
Use the Upload Directory for Log Files

Instead of saving the log file in the plugin directory (where it likely gets obliterated by plugin updates?), save it in the user's upload directory.

This commit does that, but it brings up a few other issues.  The one I encountered was the sample code breaking on the admin page.  I get a notice that `wp_upload_dir`, the fuction I used to grab the upload directory, doesn't exist.  Which is should: it's in `wp-includes/functions.php` which gets loaded way early.  I suspect the issue is coming from doing ajax requests the wrong way (eg. using a custom PHP file and including `wp-load.php`).  WordPress has a nice [API for AJAX] (http://codex.wordpress.org/AJAX_in_Plugins) that you should consider converting to.
LPS-24668
JS minify

http://phpbbsocialnetwork.com/viewtopic.php?f=2&t=4165
Gemfile updates

This is a new version of this branch in which only the dev/test dependencies have the spermy operator (sorry I guess the force push removed the comments).

This is also rebased to the pairing_sessions_sections branch I just made -- that branch adds the meta_where gem, so this branch is assuming that's already there (since it probably should be applied after pairing_sessions_sections). So only the first commit below (fd8d369) applies, just ignore the rest.
fix build on natty... rebased

see my bitchy comments...  sorry about the bitchiness, but still.
Improvements + fixes

Hi,

I've just made some fixes and added a new language ;)
Correct folder name to match namespace

Behold! the problems of developing on a filesystem-case-insensitive mac :)
Visibility Fixes

Fixes similar to those for the shelf model - to do with docking and the relaunching of QS.
See http://groups.google.com/group/blacktree-quicksilver/browse_thread/thread/1834663216b386ef/9060465d4cc132bc#9060465d4cc132bc

These changes are almost exactly the same as those [here](https://github.com/quicksilver/Shelf-qsplugin/pull/3) and [here](https://github.com/quicksilver/Shelf-qsplugin/pull/2)
New admin inline

*Work in progress*

Moving to a drag-and-drop interface for sorting the related content.  Also going to add in a better delete.
Added Korean Language

Added Korean Language
한국어 언어 적용
Now Travis is watching you!
The GetFullPath stuff here is correct.
Put up that awesome website links browser (even if incomplete) on GitHub!
You don't need the "format" and "type" arguments when you create the texture, they specify the nature of the pixel data that you upload -- which is NULL here, we don't upload anything when we create the texture. Only the internalFormat is useful there.

Format and type shuold rather be passed to the Update function, if you also support different input pixel formats.
<select name="vote[select_limit]">
-      <option value="1">1</option>
-      <option value="2">2</option>
-    </select>
+    <%= f.select(:vote_type,[["单选","1"],["多选","2"]]) %>
我想在 controller 层，根据 select_limit 来判断 vote_type
I still believe we should collect headers under the only location -- include_dir. There is no name clash
You can change the commit log message to reflect the real issue that was fixed #222
@rishta you don't say? :D
Only CoffeScript allowed :P
Wait... so are you saying that there shouldn't be specialized classes for well-known frvs?
I think the new playdoh iterations have a settings/ directory. If we switched to that, then we could add a settings/sumolocale.py and put it there. Or something like that.
It's annoying not to be able to touch the file again... the keyword 'connection = ' in the _insert statement is not correct, it should be 'conn = '
free(pencoding) is not the good way to fix this leak (fails at compiling).
should be delete pencoding;
**BOARD_DIMENSION** makes sense. It's more, "these are the dimensions of the board" and less "the board has *this* many dimensions".
s/Upgrede/Upgrade/
Best thing to do in general is to use branches, and keep all your commits local to this branch.  You should still push up to github, so anyone can help with your efforts.

We're going to release an official 0.4.0 release very soon, so probably best to keep away from master for the moment.
where -> were

And you could probably make this shorter with just "Unable to create..." and it would be just as good.
There was "algorithm" eaten by github. Comment in last line should look like this: //std::replace is in algorithm header
bitte NaT2 so abändern, dass slashbox nicht mehr gebraucht wird! Das Slashbox-Package steht unter Tex-Live nicht zur verfügung. (Siehe: http://ctan.org/pkg/slashbox)
Alternative: Erklärung wie spezielle packages auf einem Mac installiert werden (Achtung, extrem schwierig!)
Starting from f903e7b723afd7f9900eb57f93d07bc612ec1818, brunch will check (in duck typing way) if dependency is brunch plugin and fail silently otherwise.
yeah i am having the same issue.
Will do.  I noticed today that ```definition.vagrantfile``` returns ```true``` instead of the value defined in the definition file.  When I access it via ```definition.instance_variable_get('@vagrantfile')``` it returns the correct value.  Same is true with ```include_files```.  Any ideas?
Jenkins and you are at war!
Great.

...from my mobile.
On Jan 26, 2012 12:43 AM, "Caolan McMahon" <
reply@reply.github.com>
wrote:

> @petebacondarwin see
> https://github.com/kanso/kanso-utils/commit/71b43545e44432130b3d70af2248924db9bde191- I've added better support for dynamic modules in the new version of
> kanso-utils. This is available from kanso-utils v0.1.5 in NPM
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kanso/modules/commit/2efdaa6b2356a53311bad37c5239aac46c8eb97b#commitcomment-902645
>
je viens de tester, c'est parfait!
tu peux renvoyer un tableau vide quand y'a pas résultats aussi?
http://localhost:3000/api/v1/candidates/search?name=void
merci.
Can be merged!
I'd be inclined to use whatever vcloud returns in its typed error format unless there's a compelling reason to add a typecode.
This should clearly be 'this.userId(user) == this.userId(annotation.user)'. Fixed in 3a912333ebb66ed417f739ca926d1a5dc15ca986.
That should have been #9850
I'm adding your patch. Thanks
Why not just only sleep if nextTime - System.currentTimeMillis() is positive?  Seems like a waste to sleep for 0 ms.
This should be in:

https://github.com/twilson63/nano/blob/master/nano.js#L546

Like all the others :)
look this rsa: https://github.com/raynar/mangos/commit/31046d1335e32b5b7e1c75e333713172e3c923e8  is it well written?

I think RemoveArenaAuras() isn't the best way beacuse it remove stealth and all buffs when duel starts. What would be the best way to remove only "debuffs"?
@KiriX, please, restart server :) in my tests almost all good in Shattrath and small dungeons at least.
nitpick: consolidating the background-image rule in a separate class would probably make the css more concise.  Would it also simplify or improve processing of the page?  Probably not any measurable difference?
One drawback to outputting directly is you cannot send headers/cookies/sessions which makes it no different than using the command line runner.
What about a new CLI parameter like: --language=c/c++ (mandatory?). This way the user should decide what code is checking.
What happens for mixed code? "If your project code is mixed between C and C++, for better results you should check separately the C files and the C++ ones." <- new clause for using cppcheck.
Numerical solutions of 2-D steady incompressible driven
cavity ow at high Reynolds numbers
E. Erturk1; ∗; † , T. C. Corke2 and C. Gokcol1
ah ok, thx for info :D
Anyway, the range's bug are fixed :D
http://www.ruby-doc.org/core-1.9.2/Kernel.html#method-i-trap "The characters “SIG” may be omitted from the signal name."
Yesterday Ben made the following comments on trac #13263, "ed, has there been much discussion in the open about these new APIs? i'm looking at _Controller as the first thing i opened up and its really unusual (possibly even just plain wrong) to automatically call startup like is happening here. that breaks the "contract" that is provided by widgets that startup is only called after a widget is guaranteed to be in the DOM. is someone familiar with dijit reviewing your new APIs? my concern is that if you don't have support from dijit (bill), then it may be hard to convince users to adopt mvc. "
Wouldn't it be great to exit the cmake with a >0 error code at the same time?
dsfghsdfghfdfgh
Je sais, je l'avais même fait. Mais j'ai eu un petit souci, l'arborescence de sources/test n'était pas reconnue dans mon IDE, et j'avais pas envie de passer du temps là dessus à ce moment là
works well, results in some strange behavior when scaling up and down the same uneven number: 50->51->50
previously had age values of 0, 10, 11, 50 will be scaled up and down towards 0, 9, 10, 50 for example.

Maybe it'll help by not applying mathematical rounding but by always either rounding up or down for all values?

This will of course not replace lost double entries on scaling back to the previous age, but maybe prevent the behavior mentioned above.
actually, '?' is for 'unknown' (obviously :P)
if it's temporary no need for caching.
yes :) , maybe that typos was in one of refactoring commits
Ok. Agor avamos configurar o eclipse. Nesse ponto eu ainda estou apanhando. hehehe
No need to matching group. This should also work: 

```ruby
space = /\s+/.match(string)[0]
```
For my information : did you make test with referrals ? (I ask this because you seem to have interest in fs_locations).
Check for empty list (running IN() without anything seems to be really expensive!)
Very good question, incanus - I've put it to the group: http://groups.google.com/group/route-me-map/browse_thread/thread/fbd12c48d5a5d1fd?hl=en

> It is, of course, still possible to conditionally use Vlad's queue stuff, and a relatively easy adjustment, but now's a good a time as any to have the discussion, I suppose.
is this whole thing finally correct now?   I never EVER had ANY intention of changing ANY code in this function ... at all.
This adds the following routes:

            setup_frontend_screen GET    /frontend/:id/setup(.:format)                        {:action=>"setup", :controller=>"frontend/screens"}
    content_frontend_screen_field GET    /frontend/:screen_id/fields/:id/content(.:format)    {:action=>"content", :controller=>"frontend/fields"}
                  frontend_screen GET    /frontend/:id(.:format)                              {:action=>"show", :controller=>"frontend/screens"}
Should this perhaps be configurable? People might not want to publicize the path to the buildbot. (Or perhaps even the host name, in a sufficiently distributed multi-master setup.
I'm assuming you haven't pulled this yet, because you would have noticed it was broken, I didn't actually export the new function.  I had copied over the patch I've been using by hand and did it wrong.  So I amended the patch to fix that, which should be ok since you haven't pulled yet, however the diff above is out of date now.
To shift to opting in simply change line 212 from:

~~~ js
if (!shived.methods && options.shivMethods !== false) {
~~~

to

~~~ js
if (!shived.methods && options.shivMethods === true) {
~~~
Typo on this line.  Not sure how to do a partial pull, or I'd just fix this while pulling.
oops, minified is not neccesary.  this was added before I understood oztens code.  will remove
This removed the head and body..
Typo, 'for'
this just looks messy..
Now you just need to capitalize Minecraft properly :)
Поднимаешь индустрию?
jo funguje to! stačí jen commit. Akorát se musím donutit dělat větve vždy, kvůli téhle malé úpravě jsem se na to vykašlal.
Haha. Very true, silly me. I'll fix.
Some notes:

1. I forgot to uncomment now.setToNow() in BamftActivity.menuClickFunction() -- please uncomment if you want real-time data.

2. I created a Location class that simply holds two doubles (lat, lon). I don't know why we didn't do this from the beginning, honestly. So in future code, please try to use a Location object instead of throwing two doubles around, it makes things neater :)

3. See MapHelpers.getDirections() -- I tried to build an all-purpose method for getting directions in google maps. It takes a source Location, a List of destination Locations, and a route type (see GoogleMapsConstants.java for the list). A lot more params can be added to the query, but I've only created the most basic one, as I've only used it for walking directions so far.
This made me think: we might need some of the CSS/SASS stuff to be conference-related too... I remember the background colors, for instance.. not sure how to do that..
Trying out adding notes to source code via GitHub.
again, reversing this to
if (!isHPCCNode && isSqlOrLdap)

may help clarify what I mean.
Now using it and looking at the code a bit more it makes sense; however it still seems a bit cumbersome for a casual user just needing a few more features than the built-in dns library gives.

One option would be to have some sort of "auto promote" option (defaulting to on!) so for casual users it's simpler, but being able to enable it for applications with high query rates.
This currently leaks out visible characters on to a page. Shouldn't it clean them up once it knows that there is support?
Hi,

can you explain this patch a bit? whats the reason to dont report this issue and providing the patch to the xbmc developers?
At a glance, can't see what changed, but it was always designed to pass the initial target to all listeners. The example demonstrates this.
Good find Jean. Not sure how to apply it to our situation here - that tutorial walks you through how to delete an entire file from a git repo, but we want to keep the file. I'm struggling to figure out the best approach to fixing it. Dave/Tom/Jean - any thoughts? Actually, Tom I don't want to know your thoughts...
Are we now doing an additional REPORT request on every "Session" just to check if the jackrabbit server has the right version number? Seems a little bit wasteful to me and IMHO we should only do this, if a multi-get request fails (which is the only place, where it does matter and a better error message could help)
I noticed some strange behavior with the examples and I think it was introduced alongside the codemirror editor. Try the following (should be reproducible with current master):
* open examples/dev.html
* open mouse-zoom example
* open basic-bubble (or any other example that looks sufficiently different from mouse-zoom)
* click the graph
The mouse-zoom example is shown again. I guess, there might still be some old event handlers registered or something like that.
I think it should better be:

"(Wie ?viel Uhr.*)|(.*Uhrzeit.*)"

This indicates that there can at maximum be one space between both words... your example would allow a lot of stuff between Wie and viel which could cause misinterpretation.. 

Please adjust your commit
Voor ons geheugen over de subtiliteiten van bepaalde veranderingen.
maybe this can help https://github.com/Radicalmangos/scriptdev2/commit/a37ffc6330ae600445482755a8694c29b9c65322
This should probably have a:

    cookbook "application"

attribute, no?
What if it's an explosion caused by a plugin? Those aren't caused by entities afaik...
on a line?
Warum github die umlaute zerschossen hat würd ich gern mal wissen: 

Folgende Änderungen hab ich vorgenommen: 


class.rex_form.inc.php

zeile 986

        if(strpos($element->getAttribute('class'), 'rex-form-read') !== false or is_object($element) === false)


zeile 1043

        if(strpos($element->getAttribute('class'), 'rex-form-read') !== false or is_object($element) === false)


zeile 1245

      	if(is_object($element) === true) {
          $element->setValue($this->preView($fieldsetName, $element->getFieldName(), $element->getValue()));
          $s .= $element->get();
      	} else {
      		$s .= $element;
      	}
Looks good.. I will pull in soon.
кто-то мне только что рассказывал про неправильные подходы к программированию ;))
can be rewritten as `$this->primary_key = (array) $options['primary_key'];`
jquery is required twice.
Make the fixes rebase them in so it looks like you coded perfectly and then we can check this into mozilla/master.  Our strategy shoudl be to commit back to mozilla master as much as we can without interfering.
This is not allowed from legal point.
I don't quite understand the negative margin trick - looks great, but maybe someday you can explain it to me :)
저도 잘;;; 일단 로컬은 로컬인것 같고..  
개발은 개발 서버 용일지도…?
dev.springsprout.org에 올려야하나..

--  
KeesunBaik


On 2012년 2월 22일 Wednesday at 오후 9:24, Miracle wrote:

>  
> 로컬, 개발이 정확하게 어떻게 나눠지는건가요? ^^
>  
> ---
Thanks Jeremy, will do.
Hey Riq,

I would change the setter as it could cause problems this way, if userData is pointing to the same object as userData_.

CCNode:284, inside the actual method...

  if (retainUserData_ && userData_ != userData) {
     [userData_ release];
  }
  
  retainUserData_ = retainData;
  userData_ = userData;

  if( retainUserData_ && userData_ != userData ) {
    [userData_ retain];
  }
I belive I have resolved this issue on the develop branch.  Once it is confirmed solved I will release a new version.
here it went wrong
Creating a module and then extending the current module with it is unnecessary. Just call define_method on self since self is a Module.
This should, perhaps, be a separate commit.

And depending on how we set up things, we can automatically update the version in the site, whenever a new release is made (with our release scripts).
Maybe use a CanvasBoundsEstimator with an internal Svg estimator for the height - the canvas one is a lot faster.
Word up. Zero complaints. :D
actually the evententry wasnt needed as each quest is only linked to one event and should only be linked to one event.
The error message wasn't clear to me that at least two symbols are needed
Aside from this typo, we also need to come up with a more official way of handling CLI environment relative to modular applications in ZF2. See the comments on [this](https://github.com/zendframework/ZendSkeletonApplication/pull/11) pull request on the ZendSkeletonApplication for more info.
Is this warning ever going to be seen by the user of the service?
wait a second, why did you pass err to get_bytes?  isn't err an ALSA error code?  I think I may have had that particular part right...
For testing, you might be able to just return the list.
Pluspoäng!
I'm going to close this PR. I like the other method better. New PR to follow.
29行已经return了，这行还能执行到吗？
@hany88 If you found a bug, please report it in Issues tab and please specify what version of script you currently have installed.
I published a version of the package earlier today which is node >=0.4.12
(that's the version I tested it with).

I'll need to set up an older environment to make sure it works with older
that that.

On Saturday, January 14, 2012, Gavin Uhma <
reply@reply.github.com>
wrote:
> How about "node": ">= 0.4.0"? I'm using node 0.6
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/contextio/ContextIO-node/commit/96f121832f3617ffefec945e74ea180ba2c284e7#commitcomment-867955
>

-- 
Founder, Context.IO - http://context.io
talk: 514-806-6730, meet: http://tungle.me/BrunoMorency, follow:
http://twitter.com/ContextIO, http://twitter.com/brunomorency
Homework 3
==========================
| **Part**  | **Points** |
|-----------|------------|
| base      |  1 |
| advanced  |  0.75 |
| clean code     |  0.25 |
| if statements | 0.25 |
| **Total** |          2.25 |

## prissybot.py
Well done.  You covered the entire assignment and added if statements.   Also, I'm glad someone figured out `time.sleep`.

The only issue is you didn't implement step 1 exactly, which would have forced you to do string manipulation to create the input prompt.  Here's what step one should have looked like:

```python
name = raw_input("Enter your name: ")
print ""
print "PrissyBot:  Hello there, %s!" % name
resp = raw_input("%s: " % name)
print 'PrissyBot:  You mean, "%s, sir"!' % resp
```

Finally, just a reminder, commas only work with print statements and they add a space

```python
>>> print 1, 3, 5
1 3 5
```

## AsciiArt.py:  
Not a bad example of ascii art.  Your variable names made it a bit hard to follow, like using `s` and `s1`.  Also, as a rule you don't use empty print statments, use `print ""` instead to print and empty line.

## fractions.py: 
Well done on this.  Obviously we hadn't gone over if statements yet but this is definitely a good use of them.  Also, thank you for checking for the divide by zero error.

I only have a few comments (not criticisms) about this file.  Your input phrases could be consistent but that is just a design choice.  Also, since you use commas there are a lot of excess spaces in your output.  By using string formating you could be:

```python
# what you have
>>> print t," ",n,"/",d
1   2 / 4

# cleaner
>>> print "%d %d/%d" % (t, n, d)
1 2/4
```

Also, in python if statements you don't need to wrap the conditionals in parentheses.

```python
# not needed
if (a == b):
    pass

# cleaner
if a == b:
    pass
````

## numberConversionsTable.py:  
Well done on this program as well.  Your code is well formatted and clean.  I can nitpick at the little things but overall really well done.  One thing I noticed in this file and others is how you parse integers.  You can write this as one statement instead.  For example:

```python
# two lines
n = raw_input("Enter a number: ")
n = int(n)

# one line
n = int(raw_input("Enter a number: "))
```
Changed the comment as part of https://github.com/brianchandotcom/liferay-portal/pull/3821
I'm not sold on haml yet.  Given the you and Dean likes haml I'd love to hear the arguement for.  My arguement against is:

1) not everyone likes haml and if you do it looks like it is easy enough to run this script
2) erb is faster (not much but it is faster)
3) some one-liners in haml are HUGE
4) everyone can read erb.  Not everyone can read Haml.  Given the ror-e is for the community it might make the erb folks unhappy.  I'm not sure if there is a gem haml2erb.
I like the sound of that :D
I think we could change `onComplete` function signature to `onComplete(res)`, add an example to `gh-pages` and tag version 0.3
Will test this...

ref:
http://www.malideveloper.com/developer-resources/drivers/open-source-mali-gpus-android-gralloc-module.php
That's a valid point too... the tension may be that composition is in order here.

@ericgj: I'll release the article tomorrow and announce on mendicant-research. That'd be a good time to bring up this concern.
I repeat, you're awesome. Thanks @citrus !!
thanks for the fix
There's a lot going on in this method and lots of nested clauses. Can any parts be abstracted out for an easier read?  Maybe create the `Runnable command` in another, private method?
This will — in theory, at least — go on to the next session regardless of the result of uploadClientRecord, which is asynchronous.

If you intend the operation of this stage to be:

```
init
  download records
    upload record
      advance
```

then you need to do the right chaining in the various success handlers of each delegate, or use a system of monitors with `wait` and `notify` to serialize the code.
Why was this merged?  It never passed a single test run of the pull tester.  Now it's breaking the master builds.  Please fix or revert, asap.
""I willl push her soon, i promise ;)""

:-D
After checking whats its actually used for in django, i think you're right.
My bad. I've changed it to NullBooleanSelect.

On Sat, Feb 4, 2012 at 12:25 AM, jmoldow <
reply@reply.github.com
> wrote:

> CheckboxInput doesn't work correctly for a NullBooleanField, since a
> CheckboxInput only lets you select from True/False, whereas a
> NullBooleanField should also be allowed to have a value of None.
>
> The default widget for a NullBooleanField is NullBooleanSelect. Is there
> any reason not to use that?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/learning-unlimited/ESP-Website/commit/bda0e217229f1a3fc2479e5ebc5a20a83511074e#commitcomment-931400
>
Er, seriously? All of those are either completely useless or counterproductive...

The first 2 only work, respectively, in LGE and Motorola ROMs. They do nothing outside those 2.
The SF settings are useless in CM7 since we don't use that generation of the qcom surfaceflinger.
The stagefright options are the default, all of them (except disabling scan, which is stupid anyway. Why would you scan with the slower parsers?).
The last 2 are meant for devices without GPUs...
I think it makes sense. though I am concerned a little bit about the unevenness of the API wrt return value (see ReturnValueDescriptor proposal above).
si varias veces
Test comment.

@djazayeri -- let me know if you get notified that I've made a comment here.
That's the job of the jQuery widget, not the Plone widget,
Adding config params so `grails doc` command works.
Thanks for the feedback..stay tune more to come... David

On Tue, Jan 3, 2012 at 6:27 PM, Paul Hawke <
reply@reply.github.com
> wrote:

> Nice to see a stable file path for the output :-)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/in-the-keyhole/khs-spring-batch-report-writer/commit/8f2761d2874961db2b958daa497d46d55c99ad79#commitcomment-835487
>
You should document this function.
It seems unintuitive to me that the returned array might include duplicate values. If a value exists in the intersection, there's no reason (other than implementation laziness) to re-add it to the intersection if it's encountered again.
I don't think we should add this comment to private members
What would be the difference between off and off-for-windows if it wasn't on for everyone else? :-)
Same here. Not 'system' but 'node'
Isn't this just liability, just use: message.withAttachments(message.attachments ++ myAttachments)
shouldn't this be DRY:ed with above?
well, you tell me why all other failed in this file are written like Promise.failed(...).future
Shall I change all to Future.failed?
this basically begs to be automated in some form
yes, was just going to say that this should be a dynamic port
done (removed)
testkit already adds dilation to things like `within` and `awaitCond`, so in this case it was multiplied twice, that's why I removed it here
I guess you meant `java.lang.Iterable`?
I think this is the wrong place for this, but I've only just started figuring out how to make the Rails parsing use a custom parser for other formats.  I think it is pretty straightforward, but a bit more research and trial is needed.
Hate to monkey-patch, and would have called the original highlight function, but the original throws errors if you call it.
Typo: "... in which case the*y* should ..."
I can just go with this for now. The `info` level used to be awesome and only print out when resources changed and performed an action. Now its basically silent and I had to drop down to debug to get anything useful.
Fixed, think it is a github.com bug that it does not mark this as an outdated diff.
a not why you do this would be useful
I have complemented this with lower-casing first so if anyone used upper cased values should still be ok

https://github.com/bthirion/nipy/pull/2
Needs gettext wrapping.
Since this method is so reliant on its exception handling as part of what its purpose is, a @raise clause would probably be a good idea in the docstring.
Good call. I'm punting on this for now since the name pre-existed (I just moved this method from pulp_rpm to here) and we need the time for more important things.
Was trying to test this earlier. Could you give any hints on how to trigger execution of the above please?
couldn't you use explicit constructor arguments instead of using ``func_get_args`` ? It would make the code more readable IMO (and easier to understand when reading the API doc)
This line should probably go. I didn't see any tests calling for its behavior, but I also didn't try taking it out.
you guys can put the attribution back if you want, I didn't want to presume...
I mean it should be <pre>
        if task is not None and r_task_log_dir
</pre>
As you're not allowing blank tasks, it's actually a comparison to None.
The ChannelHandlerContext is "bound" to the ChannelHandler instance so only "this instance" can change it. In 3.x this was different as the readable(..) flag was modified on the Channel and so was not "per" ChannelHandlerContext. So we don't need it anymore..
わかるわかる
Sure, here is a sample request URL that returns the XML: http://picasaweb.google.com/data/feed/base/all?thumbsize=160c&q=cats&max-results=25&bbox=-156.4453125,-44.087585028245,156.4453125,44.087585028245

There is a description tag that returns plain text and a summary tag that returns HTML.  For Worldmap, it was requested that clicking on a feature would bring up a larger thumbnail (whose image source is extracted from the summary) linked to the photo's page.  This is the way I kept it in the pull request, but maybe using the feed's summary HTML as is would be more suitable.
`password_get_info` will return `['algo' => 'unknown', 'options' => []]` on failure. Why was this chosen over returning `false` for example?
7. pád od ona je jí/s ní, takže to je správně. Myslím, že kdybych napsal ji, vynutil bych tím 4. pád, který by zde byl taky správně.
Right, I understand that, I'm just wondering if the implementation can be simplified (seems the `self.actual_value` and with itself is unnecessary).
Please don't do this :-)
Please, follow the [Coding Conventions guide](http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#follow-the-coding-conventions).

```
def public_method
end

private

    def private_method
    end
```
@jeremy Thank you. That shortens the code a bit!
Yes. Right
Not used by Rails, but this is public API. Changing args needs deprecation.
Maybe it's clearer as:

```scala
def shouldAddSynthetic(sym: Sym) = !sym.owner.isPackageObjectClass || sym.owner == context.owner
``
What do you mean "what it is used for"? Isn't the example below good enough?
should we do the same for Nothing in contravariant position?
I think this doesn't work --- if `first_only` is True, the cached function value is never used.
``FITPACK_`` --> `` `FITPACK`_ ``

In some older reST guides the backticks are missing, this one is correct: http://sphinx.pocoo.org/rest.html#hyperlinks
Indeed. I'll add a bug report. when I do a few more commits later.
There needs to be an empty line before the example.
I admitted it.

Frankly speaking I don't like many of the symplification-on-the-fly which are present. (although original xfail tests equivalence only)

But interesting, what do you think about in master:

```
>>> trigsimp(1/cot(x))
1/cot(x)
>>> trigsimp(tan(x)*cot(x))
tan(x)*cot(x)
```

Is it the case of `trigsimp` must work? (with notes about singularity)
This is weird since `y` could match to `1/x` in all three cases but it does only on the last case.
Ah, so again we come to the issue of separating the container types from the objects themselves.
Stimmt, ist immer schwierig zu entscheiden, ob man eher wörtlich oder frei übersetzen sollte... Wird geändert.
The problem is that if you're leaving a line blank it will be blank when translated no matter what the input is.
I see!
About the '**agnostic**' I had no idea of how translate into Japanese.
@tomdale You are definitely right. When I saw this issue first time I had not time to realize why bucket is null, but I think what @tchak run the into same problem and has more infomation about.
why using the ``@test`` annotation instead of prefixing the method name like everywhere else ?
The "new JPanel()" is redundant AFAIK.
IMHO, this check peers too much into the implementation details: we could one day group routes by verb in a map and no longer have a flat list of routes.
Good catch.  This means that we need separate caches for PreprocessedResource, but we could still have a signle cache for Resources (as these simply represent plain files).
I think we still need to check for `overdue`?