labofsahil/github-event-dataset-2012 huggingface dataset:
Added column attributes

This change adds the feature mentioned in issue #29. There are no tests and docs yet. However, I wanted your opinion on the fix before I went ahead with those.

Due to the naming conflict I mentioned in #29, I named the attribute `col_attrs`. I am a little wary of the changes I made in `rows.py`, but the template needs access to the `BoundColumn` object in order to get at the column attributes.

Any thoughts?
Sublime Command Palette support

Hi,

This patch adds "Format SQL statement" command to Sublime Command Palette (reachable via cmd+shift+P).

Can you please apply this patch if you don't mind SublimeFormatSQL to be accessible though Sublime Command Palette?

Kind regards,
z4y4ts
Includes gems or dependencies that are followed by an inline comment.

Initially the following line would not pick up rails as a gem:

  gem 'rails', '3.2.0' # This gem is not picked up!
RaquelVP y RalfSaldana

Solucion de Kata
Added documentation for orientation API for blackberry.app

Added documentation for https://github.com/blackberry/BB10-WebWorks-Framework/issues/243
udaykadaboina

my Intro :)
MediaStoreCompatibility implementation for capturing photo feature

カメラ機能の様々な実装のコンパチビリティ実装。
Fixed issue with edd_format_amount()

Fixed an issue with the `edd_format_amount()` function in the [`formatting.php`](https://github.com/pippinsplugins/Easy-Digital-Downloads/blob/master/includes/formatting.php) file.

A check was being run if the decimal separator was a comma and if it was, it was returning a period instead of the comma causing the Purchase Receipt to display a period instead of a comma.

This has been tested and is working correctly on the Downloads page, Checkout page, Purchase History page and the Purchase Receipt.
Updated edd.css fix css buttons Issue #198

Pippin, the previous button css was not to flash ;)
Added French language files

I translated the strings with Codestyling localization plugin. I saw another person offered to translate the plugin , so he's welcome to proofread!
(maint) add new answers for DMV

DMV requires new answers for installation.  First pass at adding
this to answer file generation.
[TIMOB-7644][TIMOB-7152] iOS: Memory is not being released when a tab is removed from tabGroup

[TIMOB-7644](https://jira.appcelerator.org/browse/TIMOB-7644)
[TIMOB-7152](https://jira.appcelerator.org/browse/TIMOB-7152)

Test cases in JIRAs.
FR: Search for 'Proxy' and 'Controller' in Instruments.
nginx_status edge case

Not sure if this will be useful to others, but this makes nginx_status idempotent. [This nginx-cookbook commit](https://github.com/gchef/nginx-cookbook/commit/1759f2508d18fa5f396f88266e1a0d4edcd14e5a) gives insight into how we're using it. Basically, nginx_status should be a standalone vhost entry which doesn't affect any of the other vhost definitions.
Lps 24411.sitemap canonical
Translation for 'monkey patch', 'request', 'timestamp', 'trailing space'.
iframe

...ith multiple tags.  ie. tag=['frame','ifram']
POM changes..
Adding a name in a different locale for: Test if ConceptNames are overwritten correctly - META-290

Adding a name in a different locale for: Test if ConceptNames are
overwritten correctly - META-290
Edited the Airkinect-Gesture-Lib to support Airkinect 2.2

Original Air-Kinect-Gesture-Lib did not work with AirKinect2.2 as the Skeleton joint properties are different from Airkinect older versions to the 2.2.
I have changed the code so that it works with AirKinect2.2.
initial request working
Added Caret-History package
Ruby is unlike a Banana

Facts:

1. This talk is much more interesting than what it seems from the blurb.
2. I am not good at naming talks.
setup work …
Needed a / at "/main.cfg" or you would get xxxMain.cfg in the config

Needed a / at "/main.cfg" or you would get xxxMain.cfg in the config folder not the /config/xxx/main.cfg
Added support for Clojure/CCW
New layout

Rough but working implementation of the new layout. Feedback welcome!
fix ckeditor select url and image links
Included header necessary for building TCO with GNU Runtime and modified

project to link against the png and xml2 libraries.
Se agrego Pry para el debug y en el gemfile se agrego pry para developmet group y simple_form para el manejo de twitter bootstrap en los formularios
Added brazilian portuguese validators translations
Include premium shows, since they can contain free clips. New icon.
LPS-24494
LPS-24685
CV

Segue meu CV para apreciação e possível contratação. Desde já agradeço.
Added xyz16 display space to both spi-vfx and spi-anim

The transform matrix was corrected to use the dcip3 white point.
Add unread count in navigation

See #26.

I've implemented a simple functionality by mark as read whenever users go to show page of [article | activity]. But I can't figure out to get it style. :smile:
Update the members.md

Hello, I've added myself as a student.
Edu bootstrap migrate redux

Three commits brings the edu_bootstrap module back in line with what's on the forge and what we're using in this weeks class.
Support for Tabs in Forms

With these modifications BootstrapForm supports also Tabs.

At the moment are only Tabs as shown in http://twitter.github.com/bootstrap/components.html#navs ->Tabbable Nav supported.
A lot of formatting changes for your pleasure

I also backported r6 from UP svn.
Small improvents.
Updated spanish translation

After I understand all concepts of this bundle (like snapshots), I updated this file to have a better translation of the bundle.
Refactor SituationQueryBean

As discussed a while back in issue #4, I'd like to propose cleaning up the service alert query API in the TransitDataService API.  To recap the discussion, the SituationQueryBean would be refactored to match the OBA / SIRI / GTFS-realtime "affects" clause model, where the query mechanism for alerts uses the same style of affects clauses as the alerts themselves.  This pull request implements the proposed change.

I know it's been months since we last had a discussion around this, but there are some bugs in the onebusaway-webapp admin interface for editing service alerts that would be resolved by this pull request.  However, it would also break any existing users of the SituationQueryBean interface, so care must be taken to modify any downstream clients after this change.

Thoughts?
3 variables and 3 changes :D

I changed 3 places of your version.
* no blank line
* current path changed to full path
* git-info moved to right as a tag like date or time

Now I added 3 variables, set them like blow to active all these 3 changes.
```bash
POWERLINE_CURRENT_PATH="%~"
POWERLINE_NO_BLANK_LINE=yes
POWERLINE_SHOW_GIT_ON_RIGHT=yes
```
or it will just be like what you did before.
$phpbb_hook->remove_hook should not be used

[URLs are not replacing for FMS ➽](http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4281&p=18457#p18453)
associated with #111, #106 and #84.

**Two ways:**

1. replace template hook functions for replace HREF in HTML code
2. Find some way how to solve correctly calling template hook for each parts for ```$template->assign_display```
Time for all activities

We should unify activities and statuses and add time to activities too
Do not show userlinks for anonymous user
Submodule compatibility

Symlinking is more complicated than necessary so I reordered the repository and adapted the README to a more streamlined setup process.
Readme changes, updating api and db schema
changes to struts.xml to fix dbexport issue with linux.

basically changed only struts.xml files row 404   dbexport.jsp->dbExport.jsp.. if there are more changes please ignore them... jazz didn't connect for some reason and i wasn't able to push with egit so i had to push with command line.   Tested with ubuntu and it worked.
"median" should trigger Average.pm

After all the median is also part of the output
Fixes of 2 issues and error happened in Grails 2.0.0

Please pull for fixes of 2 issues.

Did you see the following error? It happened when i run package-plugin in Grails 2.0.0

| Downloading: bcprov-jdk14-136.jar.sha1
:: problems summary ::
:::: WARNINGS
		[NOT FOUND  ] net.sf.json-lib#json-lib;2.4!json-lib.jar (0ms)
	==== http://repo.grails.org/grails/core: tried
	  http://repo.grails.org/grails/core/net/sf/json-lib/json-lib/2.4/json-lib-2.4.jar
		::::::::::::::::::::::::::::::::::::::::::::::
		::              FAILED DOWNLOADS            ::
		:: ^ see resolution messages for details  ^ ::
		::::::::::::::::::::::::::::::::::::::::::::::
		:: net.sf.json-lib#json-lib;2.4!json-lib.jar
		::::::::::::::::::::::::::::::::::::::::::::::

| Error Failed to resolve dependencies (Set log level to 'warn' in BuildConfig.groovy for more information):

- net.sf.json-lib:json-lib:2.4
Added ability to configure layout of the jesque screens

By default jesque screens were clashing with the layout of my current application.  I made them configurable, and created a layout called jesque that would be used by default.
Should return correct indices for multiple instances of the same URL

Currently a tweet like "www.example.com www.example.com" will cause (at least the JavaScript version of) extractUrlsWithIndices to return the indices of the first occurrence for all the occurrences.

```
twttr.txt.extractUrlsWithIndices("http://t.co http://t.co")
> [{"url":"http://t.co","indices":[0,11]},{"url":"http://t.co","indices":[0,11]}]
```
This is a more accurate Spanish translation. Native spanish speaker
Исправления для хрома.
Attepmting to fix layout breaks in i18n.markdown

Another try..
Fixing Scott's name in fourth from last paragraph.
Capistrano Task to restart `instrument_server` after deploy
Download the sdk release from sourceforge
Fixed small typo in method name.

Hi,
Artie, from Strategic Profits here!

I just noticed a small typo in the API docs,
which I have quickly fixed as well.

Best,

Artie
Making redirect work in facebook's canvas
Develop

fixes for PLFM-1425 and performance improvements for migrator
Fix for CONTRIB-3844 (First approach)

http://tracker.moodle.org/browse/CONTRIB-3844
Signed-off-by: jfederico <jesus@123it.ca>
Attempt at push

Test @ 12:35
Please review: Issue/53

Rebased onto dev. It should merge now.
Prevent exception if result not found
Minor fixes to player discovery code

Mostly for correctness; no actual issues were encountered yet.

 * Use regexp literals instead of string literals.

 * Compile the regexp once in _init() instead of doing it every time.

 * Escape `.` which is a wildcard in regexps, matching any character.

 * Make sure only "`MediaPlayer2.`" is matched, not "`MediaPlayer2Foo`"

 * Only act on additions and deletions, not owner changes. (DBus allows
   several applications to wait in a queue for the name.)
is there any specific reason for :method => 'post'?

Is there any specific reason for :method => 'post' while handling response with Rack::OpenID.build_header() ?

I'm using just this w/o devise database_authenticatable. After successful auth I'd like the user to be redirected right where they came from and Rails obviously complaining about "no such action X with method POST" after a (successful) redirect but with this additional param _method=post.

It's not really a pull request, just asking...
Aggiunga immagine

spettacolo sto committando su un mio branch!
Added SaveException to Client

This pull request is related to this request: https://github.com/ddeboer/DdeboerSalesforceMapperBundle/pull/9
Added Fundraise.com
When logging out, clear pg_fb_session. This fixes issue #70.
here they go:

http://forum.xbmc.org/showthread.php?t=124139
Nava2, you need the Delphi syntax mode. And you should always check if it compiles...
>I disabled rss a while ago (decreases idle load). So for me, it 
>seems to happen with the background addon updater.
>
>Base system is Ubuntu 11.10 Oneiric x86_64.
>
>Well, this commit is applied and I got 3 crashes. I can provide the 
>logs if you're interested.

yes logs please. nice to see osx is not the only platform this happens under.
ACTIVE is spelt with an E.

Why exactly are we adding features?
I didn't flag the other assertions on call_count, but they can also be replaced with 'assert_called_once_with()`.
We would have to replace all Set instances in CharacterSet with our wrapper type, though, because we would need to be able to union() any two sets -- inverted and non-inverted. If the plain Set object is not going to be used anywhere else in the code, then it might not be useful to maintain it as a separate underlying type.
!GetPlayers.size()
Inline and no precompiled Less?
Why, because \r\n was too compliant with the RFC? :)
There should be bitwise OR instead of logical
Issue https://github.com/rails/rails/issues/2560 is open with people wondering why some tasks don't show up in rake -T. Would you please drop a line there and possibly close the issue? Thanks!
This is just covering the cases you've tested so far, it might still fail if sys.stdin.encoding is something else. It might be easiest to temporarily set `sys.stdin.encoding` to 'utf-8' for the test.
Arguing my naming are we?
Not sure about adding a `test.tpl` to repo would be great. I have one in local and it does what I need. If both of use would use the same, package shipped `test.tpl` we would have conflicts all the time as we need it (mostly) for different purposes.
I think postfixing `unless method_defined?(:blank?)` after the method's `end` is the better way to do it.
Add a test if the cd command fails. Then exit and do not execute the rm
it was a goof.
Исправлен вылет psi+ в случае, если список пользователей для чата был пуст. Например при попытке открытия _чата_ с конференцией psi-dev если сама конференция уже активна.
this was a bad bug!
Hahaha. I don't care too much for doing that. Hitting those URLs to DL
can be traced. No thanks. Remember, I wrote a Python script to do a
similar thing. I will share it when I get the damn time to do ID3
tagging =D

On Mon, Jan 9, 2012 at 8:27 PM, Steve Regester
<reply@reply.github.com>
wrote:
> oh look. theres the code that does the stuff you want... whatever you do, don't use that code in your own private repo...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/egeste/ttTools/commit/2af417a95af14fabce130dbd88333cbccefa95b9#commitcomment-851989



-- 
Brett Holt
Web Developer
UIEvolution, Inc.
bholt@uievolution.com
<poźno już nie chce mi się myśleć>
a na chuj na potrzebne te komentarze :P
I think you mean "if there are fewer nodes than tasks"
Migrate this to above, per above recommendation.
I got the error with both 1.9.2 and 1.9.3, talking to an Alfresco 4.0 repository. All fresh installs, no monkey patching or anything like that. When I query the content stream data, the request method returns nil, but the block has the response. I am 99% sure if it happens with 200 codes, but it happens with error codes (>300) for sure. No idea what is going on, but I can easily reproduce it.

I got the inspiration for the solution from here: https://github.com/xaop/activecmis/pull/3, which describes a similar issue. I think might be a bug in Net::HTTP? I suspect the fact that document ids in Alfresco look like URLs is screwing things up, but I now I am just guessing.
I have crash at some summon spells. I think that it's from problem which you described.
@Reamer 
You have crash at this problem without your fix?

Короче, рса. Если у нас есть спелл или скрипт, который призывает НПС далеко от места, где находится игрок (по сути там, для которого данные карты не загружены) мы и имеем ошибку в призыве. У себя я видел несколько таких ошибок с крашами. Именно на призыве и именно на функции GetTerrain()->CheckPathAccurate( при обработке спелла призыва.

P.S: Кстати, говорят также, что сейчас крашится при юзании камней призыва...
Напрашивается рефакторинг! Очевидно копирование дух анонимных классов, вынеси в отдельную реализацию которая будет принимать в себя экземпляр Resource
chance 50% for 92295 ?
good night
Why not use decodeURIComponent?
Yes, it definitely does depend on your definition of minor and major. The whole patch hinges around this compromise - I don't see any other way to avoid making a HTTP request for every URL added to a room.

If you're not prepared to lose this functionality then this pull request can't really be used for JabbR. (Unless i've overlooked an alternative solution?)
I did spend a lot of time trying not to create this new virtual token but instead, use token rewriting like:
non_dquote -> ESC["\\"] non_dquote;
But I always cannot get the correct result from the tree walker.
These two chunks look good, but I think I see more places where we missed returning result and instead do TSS_SUCCESS:

obj_migdata.c:581
obj_migdata.c:659
obj_migdata.c:737
no, it's pretty cool. thanks for the patch. =)
The rebases are trolling me.
I: Program is likely to break with new gcc. Try -fno-strict-aliasing.
W: uterus strict-aliasing-punning ute-print.c:265
If using two persistence units and only listing specific classes to be handled by the persistence unit (in persistence.xml) getMetamodel would just return the first metamodel object it found, even if that metamodel was incorrect.

This caused the "throw new PersistenceException("No entity found by the name: " + entityName);" to be executed in KunderaQuery::initEntityClass depending on the order of persistence units.
Hellooooo... please see my last comment. :)
Try and limit these merges @ZNickq as we need more readable commit messages for update logs.
maybe, but we tried to support IE if possible and the nifty corners was used when css3 wasn't there. Couldn't we pospone changing (old) stuff until the next release? it makes more sense breaking stuff then.

Don't know if the fallback is fine in this case though
do you mind updating resources/NOTICE.txt to suggest that Suppliers2/Test patch classes taken from Guava?
i like the idea of `direction` being a bitmap
Additionally, these make a bit more sense as class constants on a File_DNS_Exception class.
Heh, no I didn't even see that.  Got excited when I thought of it and didn't even check.  But the current implementation looks to be Windows only, no?  There's definitely stuff for me to learn in there.

The files are in a different commit (716317e4ffef2a530f6266f954f5f7474a1a98c0) since I forgot to stash before merging...
So what is /opt/shinken ?

On Tue, Jan 3, 2012 at 9:42 AM, David GUENAULT <
reply@reply.github.com
> wrote:

> /usr/local is a more standard path i think.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/dd084b8c5ad25a0786fbe79733345d14cffbae2a#commitcomment-832932
>
Well, some parts don't. Do you have ideas for the issue I posted today as
well ( https://github.com/tsa/vu8/issues/4 ) I get a similar error member
functions in a class (vu8::Class).

On Sun, Jan 1, 2012 at 9:24 PM, nuisanceofcats <
reply@reply.github.com
> wrote:

> Thanks for issuing these bugs by the way it's great to have somebody raise
> windows problems for me.
>
> I'm surprised it even works for you :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/nicokruithof/vu8/commit/ce3d806c3a6b7d60657a55866762fec75fa06e3b#commitcomment-830491
>



-- 
Nico Kruithof
nghk.nl
nice )
squash the two commits, and I'll merge 'em in - thanks!
Not sure about that... I rather keep the scene graph as simple as it is now.

I think I'll go with

```javascript
mesh.applyMatrix( matrix );
```
couldn't you get rid of the line above and pass `status_code=302`?
Include a symlink to the javascript - file without the version
ok, can I be the party pooper here and say that, even in lowercaps, we probably should use different umm terminology? :)
why do you need `LIMIT 1`?
s-aska++
You can set maven to ignore tests.

    
    mvn clean install -Dmaven.test.skip=true
What about renaming the method as `getValidIds()` as well as the test name (and do the same in the FormBuilderTest).

The comments could be added into the array and displayed on failure.
As long as it fixes the bug - it is kung-fu for me :D

Thx dude!
Using the `isEmpty` method from the `String` class is more idiomatic.  I would also recommend throwing an `IllegalArgumentException` if the argument is an empty string.
I got excited and made a bunch of changes to the sequencer. I've pushed
them into a new branch for review before merge into master. Let me know
what you guys think.

https://github.com/overtone/overtone/tree/sequencer

You'll notice a couple of buttons that don't do anything yet (Mute, Solo),
but will. I've added a inst-param combobox that lets you select one of the
insts params for editing. Here's how it works...

   - alt-click or alt-click-drag on the grid to edit the param value.  You
   will notice the shade of the cells change. This may be better done another
   way.
   - ctl-click or ctl-click-drag to delete an entry/entries.
   - shift click to mute an entry entries.

I just started working on a popup toolbox widget that could be used in any
widget to modify the effect of a mouse event without using modifiers. I
think it will be pretty useful in the sequencer, and the piano-roll.

Thanks,

Kevin

On Sun, Jan 15, 2012 at 10:06 PM, Dave Ray <
reply@reply.github.com
> wrote:

> Glad it's working. I also like the way it looks and works.
>
> Jeff and I discussed separately setting volume in the sequencer so now
> that 3 people have thought of it, it seems like something we really should
> do. I can take a stab at it if no one else has started. I've got an idea
> for implementing a little popup label that show the current value while
> you're adjusting it.
>
> The parameter metadata is definitely an issue. Maybe it should be brought
> up on the list or something.  Having conventions (with API support) for
> common parameters that all the tools know how to use would be nice. It
> would also help with "note players". A player could automatically call
> `midicps` if a synth has a `freq` parameters.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/overtone/overtone/commit/063996cfaad9441b479f793ba1b57ea4d93e75b5#commitcomment-869702
>
Sorry, that was careless of me - I'll fix it up. Thanks for pointing it out.
Not familiar with localizers, but assume these are admin pages we aren't localizing on porpoise.
@skenaja, FYI WDTK categories now live in the WDTK theme at https://github.com/mysociety/whatdotheyknow-theme/blob/master/lib/public_body_categories_en.rb
you need to add this field to the "Basic Info" form: [`devhub/addons/edit/basic.html`](https://github.com/spasovski/zamboni/commit/14b1653#L5R151)
For now we don't associate member class with venue object, because it involves a whole bunch of changes in database structure and for our current version this is also unnecessary because we don't give user an option to view all his venues. We do this in V2.0 because similarly, tasks, manpower allocation, feedbacks, budget etc. can all be associated with member object. Currently our data structure is very elementary...
`1/0` compresses well; `Infinity` and `Number.POSITIVE_INFINITY` may also be overwritten in older environments.
Merged this change
The problem is we share some of the boiler plate code between V1 and V2 (see FeedServiceBase) and it complains if we don't have the method in here. I wouldn't have a problem throwing a NotSupportedException from here instead of doing any work.
#fff on #FFA4A4
 -> readability = 0
Missing space after comma in `[2,2]`
Thanks now we will check up)
This causes other issues, apparently Github is randomly failing on https requests. I've opened an issue with them.
Please also feel free to make a library/dated_reminders.php(or something like that) script to hold functions that may be used across scripts (the decision to do this is completely up to you).
Since it's a log, could use this (this is what controls access to Administration->Other->Logs:
```http://www.open-emr.org/wiki/index.php/Access_Controls_Listing#Users.2FGroups.2FLogs_Administration_.28users.29```
(admin,users)
Haven't given this much thought, but when I leave the line in, this is the error that I get:

~~~
WARNING:root:Warning: no results for FieldsForItem
Traceback (most recent call last):
  File "./gallery2picasa.py", line 334, in <module>
    main(sys.argv)
  File "./gallery2picasa.py", line 160, in main
    child = items.ChildEntity(gdb, id)
  File "/home/kvdb/gallery2_picasa/modules/items.py", line 71, in __init__
    Item.__init__(self, db, id)
  File "/home/kvdb/gallery2_picasa/modules/items.py", line 17, in __init__
    id, Item.TABLE_NAME, 'description', 'keywords', 'summary', 'title', 'originationTimestamp')
TypeError: 'NoneType' object is not iterable
~~~
same here.. uses_tmap == false on new objects
#daniellindsleyhascompetitionforrockingdahouse
Thanks for the version bump :+1:
the name is the key to access the form type, be it in the rendering (to find the needed block), in the inheritance tree (``getParent`` returns the name of the parent) or when adding a field in the builder (you could of course create a new instance each time but this is useless as the form factory reuses the same instance, and it would be painful when having dependencies)
This is a second `rescue Zlib::Error` that will never be reached.
(3) i should be able to run any "coverage" function from command line
without any code change. no more running projects from eclipse - everything
needs to be run from a jar.

how do i achieve this ?

On Mon, Feb 27, 2012 at 11:42 AM, anand kishore <
reply@reply.github.com
> wrote:

> (1) the function names should not contain "_" (not a big deal - just
> saying)
> (2) the code needs to be a separate project under sandbox (also mavenize
> it)
> (3) i should be able to run any "coverage" function from command line
> without any code change. no more running projects from eclipse - everything
> needs to be run from a jar.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/semanticvoid/reDygest/commit/6c6429dec6e748d20d5c2219e72f16dd08925f2a#commitcomment-1016672
>
I couldn't understand the explanation. I think a concrete example would be better (say with 3 classes and 2 SV).
Does this statement actually do anything?
ok,
I now see that the moment I turned on the "sharing" plugin - the homepage got back up.

Please check to see what else is needed so this is fully moved into a plugin.

Cheers,
Tal
I guess this needs updating now?
We kinda have to anyway (use Faye that is). All clients need to be notified when a game is created, so all clients need to subscribe to that event and the client who creates a new game will just publish the event to everyone.
I believe the script interpreter in that case wasn't getting recognized (no .sh extension) and my script was failing to run.
Wie wäre es denn, wenn wir die Plugins um eine Schnittstelle zum Abfragen der Adminmenülinks erweitern. Mit der Arraystruktur, die Jan in der web/index.php für die normale Administration eingebaut hat, sollte das doch möglich sein.

Wichtig wäre mir nur, dass die dazugehörigen Layer nur im Output vorhanden sind, wenn role_admin erfüllt ist...
again, "angeben" was correct.
Claro claro, es lo que dices tu, que se sumen. No le puedes meter 20 hender, sino que cada vez que lo casteas sumas un stack en un holder compartido.
Sapphiron's Ice Block workaround can be removed too.
oh, sorry, you're re-using the type, now I see.

Yes, a ```getTypeShuffle``` would look cleaner.
Hi,

This looks great, thanks! The:
`cd C:\Program Files\PostgreSQL\9.1\bin\`
should probably be:
cd C:\Program Files\PostgreSQL\9.1\bin\

i.e. without the ', do you agree?
Can't we remove this and validate every message being sent? (make more sense to me and cleans up the code)
WOW! nice work cant w8 to test this out :D im gona update my core now :)
thank you to Subv and Shauren
maanuel pickup machine gun and tra-ta-ta tra-ta-ta ;)
 
anyway thx kandera for bug finding )))
I've done a little bit more debugging (live on site I'm afraid).  It looks as though request.ssl? is returning false, preventing the cookie being set.

According to https://github.com/chneukirchen/rack/blob/master/lib/rack/request.rb#L70 a request is regarded as secure if it has a header set (such as `HTTP_X_FORWARDED_SSL`), or if the scheme is https.
Actually cancel that, we do need to set the needsDestruct flag.
“Notify” then.
support
+1
Perfect way, that's much better ;-)
блё ну как можно такой коммит называть таким названием??
По логике это должен быть класс, который хранит числа по модулю. То есть валидным является состояние, когда 0 <= num < mod, других состояний быть не должно. Поэтому в конструкторе передаваемый аргумент нужно как минимум брать по модулю, а можно ограничить диапазон валидных аргументов, бросая исключение, если num >= mod. Так же нужно проследить, что другие методы возвращают валидные объекты и не меняют валидность this. Тогда никакая нормализация нужна не будет.
Plugin of  0x29f2d80  is  0x2991550 (Battle  0x29d6318 ) 
Qt has caught an exception thrown from an event handler. Throwing
exceptions from an event handler is not supported in Qt. You must
reimplement QApplication::notify() and catch all exceptions there.

terminate called after throwing an instance of 'ContextQuitEx'
Aborted
Or we could add another page, like `editorconfig-format`.

Add a paragraph of comment like the following:

    @page editorconfig-format EditorConfig File Format

    @section format EditorConfig File Format
    ...

What do you think of this?
Not used for now...
oups, should be the answer to the other comment
Fixed in 7ded36d
is AHB really working on gio & cooper?
Why are you closing the DB here? Add a comment to explain… and close the DB in the two failure case methods, too!
Interesting...

I notice no difference in thumbnails or playback when I checkout this commit (e986194c68258befa450ed2900009d237ad21bb5) or the previous one (13e9341791da4ee2f58a6a7aff2319316f1859b8) on my computer.

What's more is that this commit reverts an [older commit](6edfca169f469b4057650853ad27e76a95fef25c) that was supposed to fix playback problems ([#1156](http://code.google.com/p/ps3mediaserver/issues/detail?id=1156)), so it basically brings back the source in its original state.

If thumbnails and playback were working for you before this commit but aren't working now, that means the older commit did indeed fix something for you. However [reports](http://www.ps3mediaserver.org/forum/viewtopic.php?f=11&t=12530&p=63509#p63507) indicate the older commit broke stuff for others. Which makes sense because new code was introduced. It is hard to imagine how reverting back to the original code would break stuff?!

Are you sure it is this commit that breaks your thumbnails and playback?
We will normally use the same controller for both logging in and out.
Not sure what this section is meant to do, but I moved it down as well, just in case.
If the QuerySet is already cached, wouldn't meeting_times.count() and meeting_times.ordered also be cached?
Added icons for each sub activity to be viewed along side the activity name
You probably don't want to use `round()` on this line because it is overwriting the formatting you've done with `number_format()`.
**although the ShapeObject file is a good class to have. That and the changes to BinPackingView could have been different commits (with the appropriate comments).
This line only checks if session.fs.file.expand_path(datastore["DOWNLOAD_PATH"]) is nil or not.  The problem is datastore['DOWNLOAD_PATH'] may return "", in that case 'path' will just return ""
This device does not use libaudio-qsd8, but the function in present in msm7x30
config_show_onscreen_dial_button is already true, so this is redundant
In Settings, has_dock_settings should be in bools.xml, not config.xml 
After merging this change, the settings application force closes on boot.
FM radio still has noise (bass effect) after 5 minutes.
How we will disable MBADRC filter for fm radio completely?

 new_post_proc_feature_mask = (ADRC_ENABLE | EQ_ENABLE | RX_IIR_ENABLE | MBADRC_ENABLE);
if (mFmRadioEnabled) {
                LOGI("Routing audio to FM Headset\n");
                new_snd_device = SND_DEVICE_FM_HEADSET;
                new_post_proc_feature_mask &= MBADRC_DISABLE;
            } else {
                LOGI("Routing audio to Wired Headset\n");
                new_snd_device = SND_DEVICE_HEADSET_STEREO;
            }
           
            
}
Note that L1 corresponds to 800mhz in the stock kernel. With the larger freq table in your kernel, L1 might be a value that is considered overclocking and might create a bottlenecking event. YMMV
This isn't really a good idea, as it results in queries which become arbitrarily long, due to the constant addition of spaces between $query and the (empty!?) $addition_string
Fixed the escaping of backslashes and also replaced gsub! with gsub to fix the side-effect bug that changed the input string
You can do $("input.marker-geo-mashup").each(function() {
not implemented yet but I figure we can use this handle to choose a alpha state for the bottom bar when opening the status bar, this could also respond to the color state I started playing with for the background
during video recording, pressing camera kay (short press) will auto-{focus,exposure}.

i missed that it is possible to chose video rec focus mode:) ( i don;t use video rec capabilities on the phone)
Hmm. This commit needs some minor fixes, because I used regular expressions for the replacement:

-    f.writeUint32BE('AWSV');
-    f.writeUint16BE(Serializer::CUR_VER);
-    f.writeUint16BE(0);
+    f.writeUint32_tBE('AWSV');
+    f.writeUint16_tBE(Serializer::CUR_VER);
+    f.writeUint16_tBE(0);

Oops. :)

If you haven't pulled the change in yet, I can rewrite the commit or I can just add an additional commit to fix those goofy bits.
I realised that hence I reverted it with force push :P

On 18 January 2012 19:20, Will Tisdale <
reply@reply.github.com
> wrote:

> This just initialises the variable as true. It doesn't enable face
> detection, in fact it will completely stop face detection from working, due
> to the code here:
>
> https://github.com/oxygen-rom-ics/android_packages_apps_camera/blob/c06e856f2cf7d5196c7cd26a98772576384c8f21/src/com/android/camera/Camera.java#L525
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/oxygen-rom-ics/android_packages_apps_camera/commit/c06e856f2cf7d5196c7cd26a98772576384c8f21#commitcomment-879287
>
let do_beefind check this... or just do it and let grep/find fail and check for error...
Using this means you couldn't add metadata by doing file.metadata['key'] = 'value'.  Also, you are exposing users to the X-Object-Meta- portion of the meta-data which kind of stinks.  It would be nice to abstract that complexity away entirely since it is definitely vendor specific.

Would probably be better if metadata was stored as an instance variable and loaded from attributes upon first access.
again, no need to check both of those conditions, the latter implies the former
This may not be accurate in IE, which does some weird thing with offsets where it adds an extra 2px or something. @msweeney would know better than me if this could be an actual issue; I remember him accounting for it in Y.Node.
Looks safe enough to me. Only thing I'd recommend is making sure now we're actually applying transformOrigin it doesn't cause worse breakage, since we're now actually exercising the refactor to push prefixes into Dom.setStyle.
There's no Access here, but I think I found and fixed all the misspellings.
val stop = Deadline.now + timeout.duration

and then use `stop.timeLeft` and friends.
`toString` is overridden in all cases and pattern matching not done, so I don’t see the value it brings. On the negative side we have the problem that `NormalChildrenContainer` cannot become a case class as is because of the resulting duplicate `apply`.
mutable queue and synchronized is not how we roll: this looks more like a ConcurrentLinkedQueue to me. The rest is thread-local, so what needs guarding?
In what way does this test verify that it works over NAT?
That is a very valid remark, and simplifies a lot. I have removed this sugar stuff and let user api be:

    .withRouter(ClusterRouterConfig(RoundRobinRouter(), ClusterRouterSettings(...)))

with some nice apply and constructors in ClusterRouterSettings
This is included in the next line automatically.
I was thinking about scala style guide; http://docs.scala-lang.org/style/scaladoc.html
When referring to the instance of the class, use “this XXX”, or “this” and not “the XXX”. For objects, say “this object”.

It's no big deal. I think the reason is to minimize the risk of outdated docs (renaming).
this shouldn’t be necessary
Yeah. True. Should be 'decrease'.
`make clean force html` is the equivalent (don't forget the indentation too ^^)
This may need some thought as do we want them the same/different for the api so actually that is what drives this decision - now that I think about it more

but yes the route in is the issue they may `diverge` as needed based on the is this a group or organisation
There is no self.run method; you named it status below.
While you're at it, why not use staticmethod?
We don't. The weight concept is unfortunately a bit spotty in the sense that I don't believe everything has a weight, or at least a configurable weight.

The rest of your questions are telling me I didn't properly define what weight meant or its relation to concurrency_threshold. It's an entirely fabricated number that doesn't correspond to threads or priority. The intention is that only X units of work can be done by the tasking subsystem at a time, where X is the concurrency_threshold. Each operation's weight contributes to that count of units, so three tasks of 2 weight each will occupy 6 units. 

Going deeper, the intention is that the weights should prevent starvation. If the total count was set to 8 and sync had a weight of 2, it's possible for syncs to totally occupy the system. By being creative with the math, we can ensure there's always left over and no one operation can fit into the full allotted space.
fixed but not yet tested and fixed.  Added a note in the more current request to add the test
the syntax is wrong; should be

```r
for (i in 2:15) {

}
```
Может после saveAs добавить unlink? файл не загрузился, а старый удален.
OBSERVER BEEP BOOP beep wwooomp
does this work that way? I want to remove the old setting.?
Uhm, I'm not really familiar with that kind of data structures of Python
but I understood that the `d for d ...' snippet deletes the directories
that `startswith` a dot, that is hidden files. That seems to work on my
setup, but I'm probably missing something then (not you).

When I was tweaking that code section the conditional structure made
sense to me in that position, but I'll have a closer look to spot actual
errors.

Thanks for the comments,


On 29 Nov 2012 14:11, xeor wrote:
>    In taggo:
> > @@ -176,6 +177,9 @@ class Taggo:
> >              count = 0
> >
> >          for root, dirs, files in os.walk(self.content_folder):
> > +         if self.ignore_hidden:
> > +                     dirs[:] = [d for d in dirs if not d.startswith('.')]
> 
>    Am I missing something? What are you using dirs for? Did you mean to
>    use 'continue' in your loop?
>    And loops should be merged if they loop trough the same thing.
> 
>    —
>    Reply to this email directly or [1]view it on GitHub.
>    [INLINE]
> 
> References
> 
>    1. https://github.com/xeor/taggo/pull/1/files#r2271783
<-- :metal:
Here is that method:
public void randomize(RandomProvider random) {
    this.value = checkNotNull(random).nextDouble() * (bounds.getUpperBound() - bounds.getLowerBound()) + bounds.getLowerBound();
}
This needs to be in a PR before this can be opened again.
no need for the braces here- single line statement.
How is this dependent on the order?
No, it is only a deoptimization.  It should be something like this:

```javascript
if (hello.sessionId.length <= 0 ||
    !this.server.emit('resumeTlsSession', hello.sessionId, callback)) {
  callback(null, null);
}
```

`emit` returns true if any listeners were called.
Neither, that's why wrote "I'd rather throw E_NOTICE in this case (but also keep this fix)."
Could you add a test checks that DISPLAY is not being set as one of the inputs inputs (when set through config)?
I'm curious: why the second push?
Well -- then one should do this change everywhere (right now this line is consistent to other usage of byte_swap_vector). Also the "4" should be changed to sizeof(NPY_UNICODE). So I think it's better to either keep it as it is, or create a patch that does this change everywhere in numpy. Well I don't know, either is fine with me.
это совсем не нужно? убрать тогда
ну замечательно, если мы сделаем break в строчке 1682, то оно тоже сработает на верхний цикл
ok, i will provide separate request for it then.
why not something like:
<%= link_to "Path", "#", data: {'route-helper': :_path} .......
I think so. They ensure that we don't assert against cached columns. If I remove the lines the test fails with:

```
test_change_type_of_not_null_column(ActiveRecord::Migration::RenameColumnTest) [test/cases/migration/rename_column_test.rb:125]:
Expected: true
  Actual: false
```
How is this ambiguous?
How is swing more important than building with the current release of java? I'm using java7 full time.  It's not super helpful to be sticking broomsticks through the spokes of my bicycle!
Maybe a different name would be more appropriate to emphasize the difference between `Type.memberType` and the thing here?
Is this an improvement over old impl?
"Bell polynomials of the second kind"
The tests that are already merged call the `.save` method with a `NamedTempFile` instance as an argument. Thus they do test that some `png` file can be produced, however they do not test the image for correctness.

The `matplotlib` guys do their testing using reference images saved in their git repo. They have a simplistic heuristic comparison function for raster images that I have reimplemented for our testing. It was voted against using this because it adds a megabyte of images to our repo (and is quite simplistic).
Also, lines should not exceed 80 characters

Spaces for pep8 here should include changes like ```u[k,k]``` to ```u[k, k]``` and ```#for the ...``` to ```# for the ...```
This is obviously a missing typo. Hmm, that is an interesting turn of phrase...
> I am basing each next branch on the previous one. So at the moment I
> have unpublished branch 3 that is based on this one.

Just to be clear: I think that the cleanest solution in terms of git
history and standard git workflow will be to merge this when it is
reviewed.
@cebe Thx, fixed.
here... If `unselectValue` is not set `hiddenName` is not used, would be good then to leave here just the check and adding of `[]` if needed to the `name`, and then move the `$hiddenName=substr($htmlOptions['name'],0,-2);` inside the next IF where it belongs.
i dont see any difference - is casting faster than converting type via dedicated function? again one more subjective opinion - my opinion as a developer is that this change is not justified, sorry:-)
Who owns the Callbacks here? Are we responsible for deleting them or are the subscribers?
Yes, except that _ needs to be imported from kotti_media.  I would also avoid using camel case here, let's just use a normal title with a space in it.
Removed this line per our discussion:

`There should be no return value for `applyHook()`. If someone really wants each registered callback for a given hook to update the argument, that can still be  accomplished by passing an object reference.`
why a *.beam at the top level? it's already in bin/.gitignore
Yep, that is mentioned later in the doc.
sqlalchemy should be able to figure out the schema from the Model definitions. I decided not to put `metadata.create_all()` in here because that would run some unnecessary sql statements every time a process starts.

Usually I would make the tradeoff for developer ergonomics, but avoiding setting up a db one time doesn't seem worth it. This is covered in the readme: https://github.com/jbalogh/push/#readme.
Style: split assignments and declarations.
Ok, I'll change it, but I quite liked the alignment with the // to guide you to the fact that there's no action for that case.
`{` should ne on the next line
You'll need a space on either side of the = sign.
`foreach ($this->get_avatar_list() as $cat => $null)`, then remove the unused variables
Alternatively you can rename the function to get_drivers and invert the meaning of the parameter.
Well, if it's in both places, it's easier on the administrator. But as long as it's available somewhere, then I'm okay with it.
Check against false.
Thats a new one. 2010.
Newline before this if statement please.
Please do not return 404 for all exceptions. In pretty much all cases that should be a 500 exception. Unless it's an HttpException, in which case the value can be taken from `$e->getStatusCode()`.
No there isn't, according to https://github.com/unknownbliss/phpbb3/blob/8443e37803586868ae2b4434230a5ef10b48c883/phpBB/common.php#L147

This is the last line, it needs a blank line following it.
One per driver?
Space after `//` please.
A bit belov. Try to use Ctrl+f :)
This seems to change nothing except the indenting. Is that correct?
good suggestion.

+1 for now assuming you need this in for something else.

[commander](https://github.com/visionmedia/commander.js), the cli option parser used supports [default values](http://tjholowaychuk.com/post/9103188408/commander-js-nodejs-command-line-interfaces-made-easy) as well. You may want to explicitly define defaults and initial values with it's api first and save a few `if(foo)`'s
Точнее lowerCamelCase
reformat to the same indentation as in other __doc__'s?
please capture any bug by creating a JIRA ticket and noting the JIRA ticket # here
Would recommend forking elmasse's library to your github, making
saptarshi's edits, and pushing to your github again. Otherwise, we have no
edit history on the library we're using.

After that, can include it as a git Resource in Raxa.
Can you add this line after this class:

Updates = OrderedUpdates

to don't break the interface?
I think using the existing "signed char" overload is more correct -- more consistent with java bytes -- bytes are SIGNED values in Java.
So, no need to cut and paste this function.
On the other hand, the error message should not reference specifically a " a string with length" -- ""a value with length" would cover both string and varbinary.
Typo: "unable to use to new"    Will "new replica placement" mean anything users? Maybe this should  say "INFO:  a more performant cluster configuration may be achievable - try configuring an odd number of sites per host."  ... Dunno. Seems like this message at WARN could generate questions.
Because TimeTrackable smells.
You should never do anything like this. I'll let you figure out what's wrong in this line, then please reply to this comment with your findings, thank you.
same problem here, hookAdminRowAction works with plain Model, as it has the same format either in Plugin and in core Model
I'll see if I can resolve that. In the meantime can you use `yetibot.util/bot-id` directly without require?
Shouldn't that be "Working Example"? I've never heard the term "Worked Example" and even if it's proper English it probably will sound odd for every non-native speaker.
I like the idea of a flag preventing alteration of related models - that'll need to be improved upon later on so you can re-enable it inside a single other appcache, as that's needed for migrations, but should be fine for now. Do you think it's worth putting that in the patch now, or waiting till after the merge? Also, perhaps we can interpret auto_register = False as this flag (as it means a very similar thing)?
Sure - although testing this purely in Django will be sort of a self fulfilling test - I'll also test with an actual Apache conf to make sure we're not just inaccurately mocking apache's expectations in our tests - but thanks for the suggestion - I still have a ways to go before I'm as multi-language aware as I should be.
should it really be a mutable public property ? It looks to me it should be immutable
conn.inRoom( x ) is if the user associated with conn is in room x, not if nickname is in room