labofsahil/github-event-dataset-2012 huggingface dataset:
downloads.html is updated

with the links to the zipped source code from Github
Added text to readme

Hi i edited readme text
work started on support for types of issues
Make OSM tile server configurable.

This patch makes it possible to configure the tile server used for serving the OpenStreetMap maps.
Add README with hints about the Tile Usage Policy (http://wiki.openstreetmap.org/wiki/Tile_usage_policy). A list of public available tile servers can be found here: http://wiki.openstreetmap.org/wiki/TMS
You can also configure the default zoom level.
Tiny rspec indenting fix

Just spotted this...
apply upstream patches

these two functional changes have been applied upstream since the fork (2.30.1)
http://git.gnome.org/browse/libgnomecanvas/log/
Workinprogress

* Updated links to work with other markdown parsers (self closing tags sometimes act funny)
* Added structKeyExists
* Added looping through a structures keys
Tests for when an N is present in input
Switching build to Gradle

Hi,

Run "gradlew" to create the plugin zip at "build/teamcity". It is also created as Gradle artifact of "server" module and installed to local Maven repo at ".m2/repository/org/jfrog/teamcity/server/2.1.x-SNAPSHOT/server-2.1.x-SNAPSHOT.zip".
Italian language updates (merge and new items)

Italian language updates (merge and new items)
premiere integration
Address issues from PR 14

Here are some fixes to asjobs.js to clarify things a bit.
jQuery Mobile 1.2.0 compability

I found some problems in the slide menu when using it with jQuery Mobile 1.2.0 (to be honest, I haven't tried it with any older versions of jQM as I just started out implementing the slide menu in my project.
The problem being that whenever you click or tap in the active page with the menu expanded the header of the page would jump to the right (by as many pixels as the width of the menu itself). 

I found code in jqm.slidemenu.js that looked like a workaround (removing the class .ui-fixed-hidden on click) but this did not work for me. However, by adding the data attributes tap-toggle=false and update-page-padding=false to the page div it works as expected.

Feel free to implement the changes if it doesn't break anything else and if the behavior I introduced to the page is allright.
Simplify array parsing

Hi,

I tried to speed up array parsing a little and simplify the code.
Commit 9ab5d9ed475fafa8b958b54a16da191be27df034 does both, while commit fbb5128d666a98fa9b9b9e6659bf3f045fbf4d44 simplifies the code a lot, but unfortunately also makes it slower...
Cherry-pick whichever you like ;)

Best regards
  Martin
SOS-821 User is able to write a message to a non-existing member
Fix error in manual deployment instructions

It should be RPLUGIN.SRC.node not RPLUGIN.SRC.nodejs
RPLUGIN.SRC.xxx must match -t xxx.
Fixed issue adding same value twice if value was not initially an array on Document::setMultiValue call
MAVEN-43
Prevents currency_position error notice on metabox.php

Prevents the following error notice when there is no currency_position
option present:

Notice: Undefined index: currency_position in
.../wp-content/plugins/easy-digital-downloads/includes/metabox.php on
line 100
fix issue 141
Decrease sprite file size

We do this by specifying depth when creating the sprite and using optipng if it exists.

Added a test sprite task for convenience.

:metal:
Move files into folders to keep consistent

just a cosmetic change:

* Passenger
* Redis
Updated fork of Kuroir to include more JS/HTML/JSON coloring
Updated ElasticSearch module

- Fixed float for Bytes metrics. Uint16 is not egnough space for bytes metrics
- Updated authors file
- Removed IP module requirement. ElasticSearch can now determine the local node
- Added additional metrics for collection.
[Generator] Introduce a `render` method

 Introduce a `render` method that generates content without beeing forced to put the result in a file.
Update three.dart
Remove content-type templates

As referenced in #4 this product has document_view, file_view, image_view, folder_listing, folder_tabular_view and folder_summary_view templates in skins folder. But it also marks Documents, Files, Images and Folders to be ILicenseables so the viewlet registered for ILicenseable objects also gets rendered when viewing those objects.

Moreover, when you install this products its skins folder gets installed on top of the skin path, so if you need to customize those templates you have to copy from contenlicensing instead of from Plone...

With these changes, all unneeded templates from skins folders are removed and NewsItems are added the ILicenseable interface (which was missing, because newsitem_view already exists in skins folder :S).
Move the crowbar.yml layout to v2 [2/27]

Recent changes to add the database required us to update
the layout version.

I also added a doc skeleton while I was touching everything



 crowbar.yml                                        |    2 +-
 crowbar_framework/doc/clouderamanager.yml          |   12 ++++++++++++
 .../doc/default/clouderamanager/barclamp-info.md   |    5 +++++
 .../doc/default/clouderamanager/licenses.md        |   10 ++++++++++
 4 files changed, 28 insertions(+), 1 deletions(-)
add that _source need to be enabled to use the update API
Connector GUI

Updated teh Connector GUI to include a new testbox that allows the user to add information about the port to which the client should connect to in addition to the IP address of the PCE server
Typo mastertips

quelques modifs / typos sur masterstips
Improve the readme

condense gem install and use curl instead of cp since most people wont pull down the repo.
Bezier Correction & setImage

Introduced the ability to turn on Bezier correction for smoother line drawing.

Also you can load images as part of the PaintView so that images can be modified and erased (rather than just putting a PaintView over an ImageView).

iOS only for the moment as I need this for a tablet app.
AVR-ASM-Sublime

Syntax definition for AVR ASM assembler.
Allow more text in input dialog field.

The amount of text that can be entered in the input dialog is currently
capped at whatever can be displayed at a time, i.e. a few characters less
than the console width. This is often insufficient when a torrent should
be added via URL at standard terminal sizes.

This commit introduces paging for the input text field. The input string
is split into pages, each as long as the width of the text field. The
"page" displayed is dependant on cursor position. This only affects text
and cursor drawing, i.e. only a certain portion of the original string
will be passed to addstr.
freshclam: fix change in the behaviour

Freshclam (and most probably clamd too) changed its behaviour: it drops its root privileges before writing the PID-file. This made it impossible to write directly into /run.
The advantage: freshclam+clamd can clear their PID-files itselves
The disadvantage: the systemd service file doesn't work anymore
This is why I add a tmpfiles.d config file and changed the path of both programs (freshclam and clamd) to /run/clamav.
For clamd nothing changes (except /var/run to /run).
update to support swfupload v2.5.0 event handlers.

Hi Adam,

I have just update your code to support swfupload v2.5.0 events. Coz i am using your jquery plugin for swfupload :D and found that is a little bit out-dated.

Cheers,
Kenny Cheng
Silence a few strict LLVM warnings
Initial logging support

Added initial logging support, also changed references to "log file" to "plex log file" to avoid confusion between plex logs and plugin logs.
Autorizacao de Paciente pra medico
Fixes: The Observer Pattern

As I'm reading the book, I'll send fixes in batches. This is the first one :) First commit represents changes you certainly want, other ones represent changes you probably want.

I'll explain some stuff that might not be obvious, in order of appearance :P

### The Observer Pattern

`gridUpdate` is called before it's defined. Also, I noticed that sometimes you define named functions and sometimes you assign anonymous functions to variables. As far as I know JavaScript, they are pretty much the same, so I suggest you pick one, because it's kind of confusing. For example, `gridUpdate` is an assigned anonymous function, other grid functions are named.

The function `extend` is naming it's parameters in the wrong order, because the function seemed like it was extending an extension. Also, the parameter order is unlike other `extend` functions out there, they all start with the object first, **then** the extension, so you might wanna change that.

I renamed `ratingTemplate` to `ratingsTemplate` because the ID is named that way.
Add Dexterity compatibility

And fix an indexing bug in the process.
Implements fb Like Box portlet, refs issue #5

Portlet implemented, test coverage on fblikebox.py is 100%
Additional fix to issue #161: handle bare repository case

In a bare repository, git rev-parse --show-toplevel is empty.

In a non-bare repository, git rev-parse --git-dir returns the .git dir.

While the initial fixed the non-bare case, it broke the bare repository case.  Now if we don't get a toplevel dir, attempt to use the git-dir.
Options for lazy people

I'm lazy so, I wanted to make less code
That's not an important improvement, but you can do

    validates_lengths_from_database except: :avatar

instead of 

    validates_lengths_from_database except: [:avatar]
Fix failing travis tests
A whole bunch of cleanup to make stuff not die until restart
Patch 1

Add hdpi as the last locale and "should" prevent device from using "hdpi" as locale on first boot. This depends on patch-2.
Fix pagers
A minor change to use the default system look and feel.

JRuby apps look pretty unsightly on ubuntu using the basic look and feel. This commit ensures that the Velvet GUI always uses the system's (less ugly) default.
Docs issue

A minor issue in documentation. resp.json is actually an instanceof JSONElement
added a way to change the default template in the configuration file

With this method, you can change all the default template in the configuration file

```yml
white_october_pagerfanta:
    default_template: default_translated
```
Fixed incorrect building link, i.e. Building git » promotion » name #10 links to  http://localhost:8080/job/git/promotion//name/
Repo commits and hooks

Includes support for repo commits and hooks.
fixed Pylons version info

Fixed pylons version in setup.py
auto saving new photos
The Great Merging

I've now folded all of cmp_pq's features into bmark_pq.  Well, almost all — see the comment starting on line 75 of bmark_pq.  Any thoughts about how to deal with that problem are welcome.

Also, I fear I'm now the blameworthy person for more than half of the 1330 committed lines in this repo.  Sorry about that.
Invoke ServerState.LOADED after restarting by console command.

I wasn't sure if this is how you wanted this implemented, but if the intent of the LOADED serverstate is not just for the initial server loading, then this fixes the problem when using the restart command.
fix pixel cookers theme and default theme

now display correctly in both (copy from home_monitoring_controlling_project)
adds for setting wallpaper in blackberry.system extension

Fixes issue blackberry/BB10-WebWorks-Framework#398

Docs: blackberry-webworks/WebWorks-API-Docs#240
Update compiler flags for extensions
Add signals after successful processing of payment
Alternative syntax (embed & reference) example for the yml-mapping reference

I had to reverse a bit the YmlDriver to find a way to customize the fieldName for a reference/embedded document. Maybe it could be cool to share the tip.
Plugin should display a useful error message when it can't find the specified room

This tripped me up today: the plugin doesn't explicitly tell the user when it can't find the room specified in the config.  This could be due to a misspelling of the room name or, as in my case, that the provided API token doesn't have access to that room.

The code already contained the correct error message, but it was being suppressed by some odd exception handling code.  

Thanks for reading this far, and for maintaining the plugin!

Best,

Dante
New patches for sleep, etc
PLFM-848
Add support for limiting access to groups in a specific grouping

We typically have multiple groupings for different events. Users are almost always in multiple groups.
To actually make Groups useful, support for groupings should be enabled. This will ensure that only those groups within the specified grouping are shown.

This should 'Just Work' out of the box once this setting is enabled.
Check for concept dictionary updates - https://tickets.openmrs.org/browse/META-257
ARQ-322 Review 1

Here are some suggestions from the first review.

- make test for valid GlassFish install dir more robust
- cleanup admin command execution
- change order of modules in root pom
- change name of common module
- change property glassfishHome to glassFishHome
- move http(s) protocol prefix to a helper method
- parameterized GlassFish version, update to 3.1.1
- fix typos
Priority Queue

This is a simple Priority Queue needed for fortune's algorithm
All stuff on the list for tomorrow

Custom format UI is still broken. I'm going to try to make progress on this before tomorrow, but no promises.
Version bump to indicate that module is 3.0 compatible.
createPage() and url parsing on load now handle remote site syntax

Man, I had to take an embarrassing number of stabs at this to get it working in the general sense, but here it is.  The main thing it shows in relation to the node server, is that now that we can make remote requests, they aren't implemented at all, so that is moving up on my priority list.

This commit also updates the computed style.css to reflect other peoples good work in style.sass.
modified:   Browser/Files/res/values-ru/strings.xml

Почти все переведено, осталось меньше десятка строк
Add my profile page

Please add this to the site
Tehtävien palautus
Small change to hide the API Key from being echoed

The script echoes the `user` and `api_key`, making it un-secure when
part of a deployment strategy (i.e. Capistrano recipe). I found the
most secured to be to store the `user` and the `api_key` on the
server where the application is being deployed as part of the setup
(i.e. /shared/path/rcfu/.CFUser, /shared/path/rcfu/.CFKey) and use
them as part of the deployment in a command like this:

	./scp-cloudfiles.sh.php \
		-u=`cat /shared/path/rcfu/.CFuser` \
		-k=`cat /shared/path/rcfu/.CFKey` \
		-c=<container> \
		-p=<path>
small typo fix

Just changed exercice to exercise in the affected branches
Ajax Requests panel - proof of concept

It's certainly not commit ready yet, but I wanted to open this pull request to facilitate feedback on this idea.

The code implements an "Ajax Requests" panel that lists out ajax requests that have occurred.  You can then click on one of the requests to replace your current debug toolbar with the toolbar rendered for that ajax request.

Would you consider integrating a panel like this?  If so, what sorts of changes would need to be made first?  I'm not very happy with storing the rendered toolbars in the session, so I'll probably have to come up with some other form of semi-persistent server-side storage.

Apologies if someone else has done something like this already - if they have I didn't find it.  I did find an old fork that did something like this (used the cache to store the most recent AJAX request and load that), but nothing that added a panel and allowed debugging multiple AJAX requests at once, which is probably a necessity in most cases.

Anyways, looking forward to your feedback.  Thanks!
Updated PT-BR Translations

Added missing translations for Portuguese Trebuchet
Prova 1 GitHub

Test
Speaker notes update

Updates to the speaker notes.
Thanks, Jason! I'll probably just whip up something simple for now then. Thanks for the notes!
aja tu q!
Use of XMLScan (0.3.0) https://github.com/GerryG/xmlscan
I write in epic thread
On Wed, Feb 08, 2012 at 06:33:53PM -0800, hkb261 wrote:
> I really didn't understand this part of the tests... I wasn't sure if it was testing HTML or status code responses which I  believe is the reason for the  fake_response asserts.  Hence my questions in class about this.  I can update these this weekend and fix them so they actually test both the proper status code being returned as well as proper HTML.

" the distinguishing feature of these tests is that when essential functionality in your code is removed (e.g. you comment out a line), the tests fail.
"

:)

(These are comments/discussion/"education", not dinging you on the HW itself)

--t

> On Feb 8, 2012, at 9:24 PM, C. Titus Brown wrote:
> 
> > So, here you check for stuff being in the actual html, but you don't anywhere below.  ???  These tests will pass if I eliminate returned HTML!
> > 
> > --- 
> > Reply to this email directly or view it on GitHub:
> > https://github.com/hkb261/meep/commit/dafb21e8a122b75219499d57009dcfcd8fd07852#commitcomment-947947
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/hkb261/meep/commit/dafb21e8a122b75219499d57009dcfcd8fd07852#commitcomment-947960

-- 
C. Titus Brown, ctb@msu.edu
Neznám přesně ten důvod proč je tohle potřeba, ale jestli to je něco obecně kvůly php54 nemělo by to být i na dalších místech kde se token_get_all používá? Např. [AnnotationsParser](https://github.com/nette/nette/blob/master/Nette/Reflection/AnnotationsParser.php#L251)
You can prepend to sequences, no need to convert to List.
Always `:8888`?
Good point, thanks for the reminder.
Reserved keyword problems or .. ?
>but here it doesnt matter much as the code base is not that big
The current repo size is 53MB in my laptop. Little stats:

    time git checkout echov2-dev
    Switched to branch 'echov2-dev'
    real 1.195      user 0.008      sys 0.040       pcpu 4.01
    time git checkout master
    Switched to branch 'master'
    real 44.571     user 1.272      sys 0.448       pcpu 3.85

>And please dont file any feature request issue for version 1.
I'm fine with this.

A list of issues from Redmine:
![http://i.imgur.com/6hELL.png](http://i.imgur.com/6hELL.png)
just in case.
Maybe use SC_READER_CAP_PACE_EID instead?

I suggest to use the prefix SC_READER_CAP_PACE_ for the 4 PACE specific capabilities.
SC_READER_CAP_PACE could be replaced by SC_READER_CAP_PACE_GENERIC
Why is `array_reverse` being used here?  aql files in higher level codebases are not cascading as expected.
Resolved in ac7430de37dee0bdb89fe7f34782a86703db5171 - thanks for caution :)
Ma quanti diamine di italiani ci sono qua dentro?
I am fairly ruby illiterate, but it looks like it retries by attempting to save the attachment again (presumably with the document version id it got from db.get(id) ), if the failure is due to a race condition in the big couch back end, and the put_attachement actually worked (and the version was updated to 3-xxx) then won't these subsequent retries always fail?
I personally would prefer
Player* player1 = NULL;
Player* player2 = NULL;

at least the cases where we indeed need two pointer variables of the same type are not so frequent, hence I think there is no harm to change them
Why did you remove this? Is there any roadmap to reintroduce the feature?

TY
Inline feature req: could this be put back in ~/Documents/Textual Logs? My Objective-C fu falls short of doing this myself..
Mit dem neuen Inhalt ist DHBW2 einverstanden
I cant believe that its been 8 months and this still has not been pulled into Rails. WAT?
This commit must not exist, see comment on 9a30fda8fe6a8ea9d04990b7ef61eab90b4960a8
"14" refers to the match length of the UL. Don't change it.
That's right, I just pushed a fix.
bien ouéj !
I figured we'd push that off to all the cleanup that'll happen when we switch to an IndexMixin.

If you think it's worth doing now, I can do that.
whoops :), 

fixing immediately,

thks,
It needs some CSS work (which I am useless with) on @ifesdjeen's part but in general content can be added even without it. We need to figure out a good way to organize code examples, everything else worked well with rubyamqp.info.
Not that I can see...does it work before this commit? I tried a trivial case and it worked. What are you doing exactly?
doch nicht jojo, sorry--> siehe andere commentare
Yes, I saw the other edits, and did not comment on them because they made sense.
I was just pointing out the introfuction of invisible padding.
You should update the docstring of the function.
hmmmm !
How my fork copyed to mangosR2 ?!!!
wow :) but anywhay, nothing strange. only one worked map thread - creature check MoveInLineOfSight around self.  may be in some maps we have auto-spawned scripted creatures?
Please make sure the first line of a commit message gives an indication of what it changes (without having to refer to the bug tracker or the rest of the commit message).
I'm surprised that this one is necessary.  The other sun vectors (AOSUNSA* and AOSUNAC*) occasionally have a magnitude of zero (e.g. Safe Mode) since they come from the OBC.  However, I can't foresee a situation where the sun vector based on ground-based ephemeris would do the same.
hehe yeah, ok, but doesn't it still need a certain light level to turn into water when breaking if it's not on top of water?
So as a example: if ice is on a dirt block, breaking it wouldn't create water would it? :/
Ah, I see. If we want to be pedantic, we could use the [formal style](http://www.oracle.com/technetwork/java/codeconvtoc-136057.html), which states the following:

> Four spaces should be used as the unit of indentation.

Personally, I prefer 2 spaces, where every tab character is always expanded to 2 spaces. However, is it necessary to convert the indentation of the entire file like this?
however, put spaces around `=` outside of function calls =P doesn't really matter here but...
could there be times when you want `null` or `undefined` to be copied?  same for the `mvc.to` block.
yes, I know and it will be a useful didactic experience.
probably a printf that warn the user should be enough to prevent it :)
Latin is website policy. We provide "local_name" with full UTF support. For first and last name we need to enforce only latin letters, ' and space. \w is more permissive by also including numbers.
We support many languages, like Scheme, Scheme and also Scheme!
поэкспериментировал с различными комбинациями: пришел к выводу что minimize это не решение - при нажатии на крестик окна - минимизировать его, если не стоит галочка tray, из опасения что пользователь потеряет окно...  Думаю оптимальным сделать так: если пользователь включил "hide on close", то включать галочку "tray" и не париться по поводу потери она. что скажешь?
Github's editor bug, sorry :(
The `char buffer[]` or the `string str` is redundant.  You could easily build the string into `str` from the beginning.    I would find it more readable that way, myself.

Just a thought.

```c++
    string str;
    str = "==> " ;
    str += m_filename;
    str += "<==\n";
    print_to_stdout( str.c_str() ) ;
```

Or you could even use stringstream:

```c++
std::stringstream ss;
ss  << "==> " << m_filename << " <==\n";
print_to_stdout( ss.str().c_str() ) ;
```
Please add comments to Q_PROPERTY :-) mostly to mention the default value
you don't need to do this. this is already taken care of in https://github.com/vaidikkp/zamboni/blob/9d7b75aeb95/apps/bandwagon/forms.py#L178
This is not a drive file please revert and use markdown* specifically.
Estou só terminando de me achar aqui no github....
Maybe explain what is deficient in the current approach
Just a quick note:   with CXF 2.6.0, this may no longer be necessary.   With my commit:

http://svn.apache.org/viewvc?view=revision&revision=1236893

it's now possible to use the normal config:admin stuff to setup and configure the HTTP clients.   TLS stuff isn't there yet, but will be shortly.
Would it be correct to guess this is a broken merge commit of some sort? This doesn't look right at all.
@nahi: Boy, that took longer than expected. But I love it the way it is now!

- You can do ASN1.decode with DER or PEM interchangeably.
- If the underlying stream does not support a "rewind", it will still work by constructing a "SequenceInputStream" that consists of the lookahead plus the original stream.
- Of course this means a slight performance penalty (but much less than I thought) - those users that know with which format they are dealing may use ASN1.decode_der/decode_pem instead.
- Now the thing I'm most proud of: Krypt::PEM.decode returns an array of all the values it may find in a file (think of CA bundle file). You can also feed it a block, where the DER value itself, the "name" (such as "X.509 CERTIFICATE") and the current index is yielded.

You can see it in action via https://github.com/emboss/krypt-core/blob/685443010fdc31cf2b21ff62bebcc559ce2a97c9/test/scratch.rb.

What do you think of it? In my opinion that's as flexible as we can get. And it still performs way better than the old OpenSSL code. PS: The opportunity to read additional "header data" from within a -----BEGIN .... END----- block (sometimes there's some data just before the Base64-encoded stuff) is currently missing, but once needed it can be easily added. That would be another parameter in the block arguments of Krypt::PEM.decode.
could also write `if m.user == bot`
I would mock net http before this call to speed up the test case I have satellite internet and this takes forever
With the new look for published worksheets, this seems perfectly reasonable. (Although someone who knows this code a bit better should chime in.)
So if you select a parent task and hit OK, the dialog will be silently canceled instead of confirmed, without warning? That sounds odd to me. Can't we disable the OK button instead?
Is this to fix UA with broken label behavior?
Do you think the UoW will actualy calculate the dependencies? Wouldn't the Command or some kind of Dependency Resolver be responsible for that?
Ты уверен, что так, а не с отрицанием?
Requests honors `CURL_CA_BUNDLE` out of the box.
why revert taxi node?
Yea...  I know...  Gotta love IE.

-----Original Message-----
From: Peter 'Pita' Martischka [mailto:reply@reply.github.com]
Sent: 30 January 2012 09:29
To: John McLear
Subject: Re: [etherpad-lite] Fix #171 IE9 edit issue and #361 IE URL Last Letter issue (7096ce7)

wtf?

---
Reply to this email directly or view it on GitHub:
https://github.com/Pita/etherpad-lite/commit/7096ce7f1ef8962205e27db49284d440d834c26f#commitcomment-913599
This email and its attachments may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of the organisation from which this email originated. If you are not the intended recipient of this email and its attachments, you must take no action based upon them, nor must you copy or show them to anyone. Please contact the sender if you believe you have received this email in error. This email was sent by School Email - Safe Webmail and Hosted Email for Schools
In case it is not an instance of `phpbb_extension_controller_interface ` we should probably show an error. Random 404's will not help debugging the issue.
I put the localhost back, because the mgmt ops are done at server, therefore localhost is valid.
It would need packaging all the mgmt client deps into the war otherwise.
Right, but i think a user's first questions is going to be the trustworthiness of the app itself... that we're not sending their credentials to a hacker database somewhere or whatever.
"- there's also getNewline which could be added to $escapetool"

It's been added in the latest Velocity Tools:
- http://velocity.apache.org/tools/devel/javadoc/org/apache/velocity/tools/generic/EscapeTool.html#getNewline()
- http://velocity.apache.org/tools/devel/javadoc/org/apache/velocity/tools/generic/EscapeTool.html#getN()
406 status by default?
I rolled back to 1.9.2 because debugger on 1.9.3 is "buggy", :P
Hey, this actually broke the build. We can't use .isEmpty() in Java 1.5.
@peterbe My first thought was to use body_content_class but, this was again going in the direction of a double id selector. I reckon changing the container on the front page would be a better option then. We could keep the main_container id and add a common class for left floated containers.

I am totally with you on the use of !important. I never use it and it really should only be used as a very large resort in places where you have no control over the content you are attempting to style, such as with the simile stuff.

I will do the refactoring as mentioned above and update the pull request.
Forgot too add the util package to git. Should be pushed properly now
I know man. I appreciate it.
I believe you've nailed it. My private fork is sufficiently different to not have this issue. Could you add a test to the test suite which confirms it? I see there is alread a t/up_table_map.t. Your contribution will be credited in the "Changes" file.
Deal with sessions issue:
http://www.open-emr.org/wiki/index.php/OpenEMR_System_Architecture#PHP_Sessions_and_Browser_Windows

(note this may cause an issue here, but recommend trying it)
Any reason _not_ to use `===` here? If we're checking for a string, then there's no need for type coercion, right? Looks like this was `===` back in [9883dc37c6f9caf](https://github.com/desandro/imagesloaded/commit/9883dc37c6f9caf#L0L67).
Put array('default20', 'blue_eagle'); into here and the template will work better -- getting images and php files from above templates if they do not exist in the current one.
please use explicit set when using templates.
Looks good, there are a few extra line breaks and formatting issues though. Also, I think the style for Factions is to have hanging braces, but you'd have to ask Brettflan about that. 

For the FPlayerJoinEvent, I'm pretty sure you don't need to override the setCancelled and isCancelled events.
Am 2012-01-07 11:08, schrieb zeitgeist87:
> Gute Idee ich hab die Aliases noch aus der Funktion herausgezogen.
> Sie sind immer noch Lokal, werden aber nur noch ein mal zugewiesen.

Ich würde vermuten, dass die Funktion dann nicht mehr im innersten Scope 
ist, dh dass zuerst dieser (erfolglos) durchsucht wird, bevor die 
Funktion dann im äußeren gefunden wird. Das würde dann die Optimierung 
wieder teilweise rückgängig machen... kannst du das bitte profilen, 
bevor du es committest?
What kind of context?
More info to why use this instead of msg::getObject?
The bleeding edge has this fix in it, so I *presume* that it does. Download from GitHub master (and not Riaforge or any of the release tags).
Caption should not be empty, see my other note.
You most likely want `(divisors.length - 1).downto(0)`
You don't need not_equal, Ruby automatically converts != to ! on ==
"geben" was correct.
as i said:
1) everywhere else used struct
2) additional indents because of public: private, etc..
What happens if you just change func_name to __name__ as @rgommers suggested?

http://projects.scipy.org/scipy/ticket/1576
forgot the ; at end
We are not your server service @aios. We NEVER claimed it to be stable or encouraged public use
2012_16_02_00_world_conditions.sql
2012_16_02_01_world_conditions.sql
2012_17_02_00_world_waypoints.sql

and ? ;p
From your link, in you check @ 3:26, the char start cast frostbolt at his target, in the middle of the cast, the target move of o LoS, the spell casting is cancelled (notice that the char moves after the spell cast is cancelled, not before).
А нельзя там никак менее уебищно, чем `init . init . ...`
?
Currently the only reason we return a resource is to keep the shortId along with any parent/child ids.  If we were to use the concept neighbourhood service and get all parents and children that way, it would then make more sense to just return a list of short ids from the hierarchy service and create the resource after the full id (and other attributes) are known.
cant we have files which changed after the last snapshot ? 

timestamp1 : 0001.bdb, 0002.bdb .. 000x.bdb
timestamp2: 0001.bdb, 0002.bdb .. 000x.bdb ... 0000y.bdb

I thought 0000x.bdb can still have more appends during timestamp1 -- timestamp2, can we check this for correctness ?
Generated file. The generator is further down.
If you’ll use `this` is some anonymous classes like `OnClickListener` or inline like `AsyncTask`, imagine what you get.
This plugin actually works for me in just one server (debian lenny i386) with debian munin packages.... fails with debian squeeze. there are several errors with munin-node-configure... lines 107 and 108 seems to be conflictive.
Also I don't like having those lines 12 and 13 within a php script...

Overall, this plugin should be entirelly and urgently translated to perl, aply some polishment, and Iet it work rock stable as the former shoutcast 1.9.8 plugin!
Sad I'm not a programer, but maybe we have someone that does it.
Lol, Edit warring
> Maybe we should revert C to be unscaled

That's simply meaningless: it breaks cross-validation where the number of
samples vary accross folds, or between parameter tuning and training the
final model.

> and add an scaled alpha parameter which is None by zero:

In the long run, I'd like to have a param, something like 'adim_params'
that converts all constants to adimensional value. If true, C would be
divided by C_min in SVMs/logistic regression. In lasso alpha would be
divided by alpha_max. In general we should try and find as much as
possible a way to make it so that the different parameters are not
modified by some trivial change such as sample size or regressor scaling.

G
CodeStyle ?
おっと…直します :)
missing the genlock code file.
thanks for your hard work.
Oh yeah, only line 315. There's a bug in breadcrumbs where if the page has a child, then when the child breadcrumb is rendered, the parent's breadcrumb function is called... but with the context of the child.
you should set autoload to true here to avoid registering the file twice if someone registers the extensions for the ORM and the ODM
Unfortunately, all the other wp_usermeta data such as user_twitter and user_organization isn't being stored in wp_usermeta when the user gets created. Also, it's not sending the user the confirmation email with their login and password. :(
Well, you could do != 0 to fix that.

Your way of doing it fails for all examples... so if you have <a href="http://google.com">Go to google</a>  it would try to do that relative.
Nevermind I was wrong
I personally prefer jsdoc-style documentation, e.g.

   /**
    * @param {nsIFile} source A folder to compress.
    * @param {nsIFile} dest The destination. The file is erased if it exists, created if it does not.
    */
It's a bug. Managed operation **getMessageGroupCount()** returns wrong number of groups...
Oh, I see what you're asking.

If the first `getContentProvider` fails, it'll throw a `NoContentProviderException`, and the constructor will fail.

If the first succeeds and the second fails, we want to catch it so that we can release the first, then we rethrow the exception. That's why only the second call is in a `try` block.
General documentation/comments guideline: _API documentation_ should go into the .h, while _algorithm/implementation description_ goes into the .cpp. Here, you have API documentation.
I'm assuming that you'll be wanting to persist both the GUID and the client name in the Android Account object. If not, you'd only pass back the GUID.

You can generate the GUID anywhere; here in the Clients stage is fine. The stage has access to a GlobalSession instance, and that's what the Clients stage should ask to save the GUID into the Account (via the SyncConfiguration).

Though see also my last paragraph, which says you can put all of this stuff inside SyncConfiguration: it would take care of retrieving the client GUID from the Account, and generating and setting one if it's not present.
It might be cleaner if there was a common function  - a bit too much scope for incorrect counts
Name is not unique. That's why I changed to the ID in the first place. 

We should match the length of the slug to the max length of the username. To make sure it is unique.
As far as I remember, the Analytics snippet should be put at the end of the page, since in this way it slows down the page load; why are you putting it inside <head>?
nice!! de eerste keer dat ie vrijwel helemaal soepel loopt op mn laptop in fullscreen. zelfs 7 cpu's en 150 ping (en custom map zodat de ai beetje moet sturen) doorstaat ie prima. super.
Test, :)
Just to let your know, you still have lots of code duplication in your maven poms.
This can stay as true
A commit earlier today should solve problem with ldpi devices (which I 
think the p350 is):
http://review.cyanogenmod.com/#change,11878

So if you did not sync today yet, it would do that first and then recompile.

Op 18-01-12 19:12, pax0r schreef:
> Actualy my device - LG P350 - is not one supported by CM, I'm porting CM to it - is getting bootloops after this commit.
> The logcat of the error causing loop: http://pastebin.com/PX6X6HDg
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CyanogenMod/android_frameworks_base/commit/eb976f721b192802f06807eb460ba3dfcc6779cc#commitcomment-879045
teste do teste 02
Il faudrait mettre instantreact.com à la place. Nos apps mobiles sont uniquement en anglais :)
ELECTRON_BEAM_ANIMATION_ON, 0) == 1);
                                                                    ^ typo?
@ey-pairing - will tags give you what you need for this, or is there still a need for this line?
I've been watching the comments, but to be honest I don't think it's a huge deal. The code is annoyingly obscure, but I can see the point about not duplicating the whole of `add_option_group`. It seems to me that one of two things should be done:

1. Add a comment before the pop/insert, clarifying why it's done this way. This is a short-term solution, but good enough for now.
2. Longer term, probably in a subsequent commit, refactor the code to make `insert_option_group` the primitive, and implement `add_option_group` in terms of it (that'd be a one-liner using a hard-coded idx value, presumably).

If @gvalkov feels like doing the refactoring now, that's great, but I wouldn't hold up the commit for it.
We can't even use the default in Blizzard's tests (we always get an OS-provided open port for Blizzard servers being tested), so there's no coverage anyway.

I'm for having the same default for all cases. Doing that now.
"fileAcceptable" might be a better name.
You're missing the final keyboard in a number if spots.
I think it's safe to keep only keys that begin with `management` and leave out the rest.
We should probably have an AuthenticationBuilder class at this point.
I have no idea why it keeps doing that.
we agreed with Roland to keep the volatile var
Given the change to return the `Future` I proposed above, this should be changed along the lines of “asynchronous API, but you can make synchronous calls by using `Await.result` on the returned handle”.
Actually I can not change to this strategy yet, cluster actor name is not unique ;-)

private[cluster] class ClusterNodeMetricsCollector extends Actor with ClusterMetricsCollector with ActorLogging {

  val cluster = Cluster(context.system)

So we can refactor it when Patrik's PR is in master.
Then I argue that it's written in a suboptimal way since neither of you nor I touched this subject when we discussed rewriting it.
I can use the 'member-status' instead, and remove this method.
so, after a restart you don't want to use the handleOption passed in to the constructor, but still use isServer derived from the constructor param?
It should still mention that routerDispatcher cannot be a BalancingDispatcher.
lose commented code
Cool, will happily change it to test_dsi.
Can we drop the `sphere=None` here?
Done. I have kept all fail messages together though. Don't want to put DEFAULT_CONSUMER_PERMISSIONS in between them.
These would be better if they were the default values in the config instead of hard-coded here. See comment below.
The KEY_ASCII should be 0 I think - you're not passing an ascii value in, and this is supposed to take a char anyway (KEY_ASCII is 0xf100 so all you're doing here is setting it to 0...)
Yeah but it IS the correct list now. I think we'll just have to wait and see whether it causes any regressions. Note that as it was, it also caused eg. CFileItem to look for thumbs/fanart on eg. tcp:// (which obviously doesn't work since it's an ffmpeg protocol, not one of our vfs's). Agreed?
use a generic PigRuntimeException if there's one
Esta clase mejor borrala,,,
this won't grow very well. in the future, if the goober struct has more fiels to instantiate i'd rather have it be in its own statement
There has to be a better way of doing this. I'm really not a fan of instanceof checks as they cause the code to be worse. The same goes for boolean flags.

There must be a better way of doing this.
in this case it might be the same. But in general if you're acquiring keys from a different array than you're using them on, you're gonna have a bad time. And if we ever decide to drop timers from calculations or anything like that, this will bite us.
Shoudln't it be LogicException instead ?
Looks good (given that those exists, I don't know them all), but those should preferably be sorted alphabetically for easier updating.
Fixed that too! :]
There are 2 advantages to coalesced gradients and angles: The first, it takes 2 lines to achieve a very sleek table view cell look, as the example project shows. The second, it's still within scope (`UITableViewCell` has gradients for `selectionStyle`, and while implementing that, it literally took only 15 lines more to enable it for background and highlighted styles) of TwUI. Because of its tight integration with the class, it doesn't work AT ALL without being a part of `TUITableViewCell`. This is why I included it-- it's the *only* feature in `TUITableViewCell` that seems a slight bit excess, but as I detailed, the advantages outweigh the disadvantages, at least in my view.
They're there as public for now for debug reasons, and I do see why they should be private, so I'll commit that.
With break is perfect work. I resend.
This repeated getter call worries me slightly from a micro-optimization point of view.  Is it really necessary to handle updates to the listener array while we're in the process of removing all listeners?

Why not something like:

```javascript
var listeners = this._events[type];
// LIFO order
for (var i = listeners.length - 1; i > -1; i--) {
  this.removeListener(type, listeners[i]);
}
```

The only use case I can think of where this would differ is if you did something like:

```
ee.on('removeListener', function (type) {
  ee.on(type, fn)
});
```

which is such a horrid anti-pattern it makes my brain break anyway, and will cause an infinite loop in this case anyhow.
Code duplication!  Please factor out the common list conversion as a separate method.
No need to specify types here (: K)
Ditto re. shallower indent

```
DAYS_INTO_WEEK = {
  :monday => 0
  ...
  :sunday  => 6
}
```
+1. Can remove these comments, then.
Continue using `test =~ ` syntax here and down below, even more if you don't need to use the match object.
Hm, I don't think it's necessary to remove the other description.
Ah indeed, it is in the EXIF specification. Could you just add a comment to explain this in the code ?
Um..I thought the idea was to find the first canonically smallest symbol and then cycle from it.  With that understanding  A*B*A*C*B  would continue to be A*B*A*C*B.  

Am I missing something?
Probably should be ```is_self_conjugate```.
On 31.05.2012 02:08, Aaron Meurer wrote:
>> +        """Returns True if ``other`` is a submodule of ``self``."""
>> +        raise NotImplementedError
>> +
>> +    def multiply_ideal(self, other):
>> +        """
>> +        Multiply ``self`` by the ideal ``other``.
>> +        """
>> +        raise NotImplementedError
>> +
>> +    def __mul__(self, e):
>> +        if not isinstance(e, Ideal):
>> +            try:
>> +                e = self.ring.ideal(e)
>> +            except CoercionFailed:
>> +                return NotImplemented
>> +        return self.multiply_ideal(e)
>
> Shouldn't it also return NotImplemented if this raises NotImplementedError (as above)?
>

Yeah I suppose ... changing it.
Is there a mathematical backing behind this mapping?  If not how about using strings or constants.  For example why not 

    TensorIndexType("some name", "symmetric" ... )

rather than 

    TensorIndexType("some name", 1 ... )
It's unnecessary for the last parameter. What if we add another parameter later and forgot the question mark for the former one?
Should we pass this maintenance issue or cover it for good?
user и account, по-моему, часто отождествляются, поэтому вполне понятно что есть что из описания :)
I think that any new method should contain `@since` phpdoc tag, but not sure whether it's needed in protected methods dedicated for internal use.
Whoever wrote this project has obviously never heard of string streams.
they also didn't know that constant string appending causes slow buffer
reallocations. There is still some evil in plLocalizedString if you want to
cry some more =/
"a string" is both more clear and more concise here than "the latter"
What about this parameters - do you think they should be configurable via options, removed or set to the values specified by default?
The problem is that `history.updateLocation` does not alter the history entry.  `history.replaceEntry` calls `history.updateLocation`, but it also updates the URL in the top history entry.

Changing this to `history.updateLocation`, the "form redirect" test case fails because the "referer" header points to the requested URL instead of the final redirected url.
Why the extra space.
Are you assuming that `marked_down_at` will be set to `0`, and that `marked_down_at + self.retry_timeout <= now`?  If so, That works in this situation, but will break for high values of `self.retry_timeout`, like if someone sets it to infinity.  I think it's probably more clear and better to do this:

```python
marked_down_at = self._down_connections.get(db_num, False)

if not marked_down_at or (marked_down_at + self.retry_timeout <= now):

Also, shouldn't we be giving a host back if we're `>=` now?
```
That's not my junk. That was part of the conflict, which I think was from a different commit. Do we need any of that commented code?
This needs to be fixed, as `@adapter` could be nil.
Superfluous now that libev is removed.
Why not abort()? Anything else is a fatal error anyway, right?
These lines were in the example that I  took as a base.
In the current implementation they are irrelevant.
Remains as a commented code to solve possible errors caused by future changes.
User_cache use User, it's a direct reference, so an import is needed.
Do not use indirect reference because it will be impossible to clean up the imports then
the function wraps the insert method and takes foo and bar arguments.
I think it's a good way to change this together with other files.
Would it not be better to activate the autosaving using a data attribute on the textarea? Supplying a unique save key via a data attribute would remove the need to munge the URL and make this function more reusable.
It is in a language string for the simple reason that the coding guidelines clearly state
> All messages/errors should be outputed by calling `trigger_error()` using the appropriate message type and language string.

If hardcoded language should be used in these cases than obviously the coding guidelines must be changed to reflect that.
This performs 24 dom traversals.
Can you remove this `<br />` and add some padding to one of the paragraph elements? Break elements are unsemantic as hell.
The statement "The ext directory does not exist" is incorrect in my opinion. No available extensions does not imply that the ext directory does not exist, because `all_available()` does other things as well.
Doesn't really matter why it is there, that function has been behaving like that since 2002
https://github.com/phpbb/phpbb3/commit/0b285c07f61b2c4be45ff4430fb8b9a0fa6329b2#L0R258
$avatar->enabled() makes as much sense as $avatar->is_enabled() but is much easier to type. Anyway, it's not a huge issue, I'm just used to it from messing around with Symfony.