labofsahil/github-event-dataset-2012 huggingface dataset:
Exclude comment codes from annotations

Currently, comment codes are detected as ordinary links, and are thus annotated. This unfortunately results in issues like this: http://www.reddit.com/r/Minecraft/comments/wk47e/sheep_art/c5e1frh. If you click on the comment, the two nice image arrays of sheep and cloth suddenly explode into this mess of annotations. Of course, we don't need these annotations because these images are not links; they're simply placeholders (comment codes) so that the subreddit's custom CSS can set the link's background image.

This pull request adds a function that checks whether a link is a comment code (`isCommentCode`). This function is used in the method `keyFocus`, which adds annotations to links found in a comment.

Let me know if you think this should be added as an additional option that can be tweaked as opposed to a default.
Fixing payments history duplicated values

Cleans some duplicated code from previous commits.
Fixes template[s] conflictmentions

* The template namespace has not postfix s
* Share conflict mentions between affected plugins, added a forgotten argument
* replaced deprecated split() by explode()
Change IRC link to #zftalk since #zftalk.2 is now a closed channel

The topic on freenode #zftalk.2 channel is: "Channel closed, please use #zftalk" and channel is set to permanent moderation.
add rewritebase to htaccess

For some reason mod rewrite rules without rewritebase cause recursive internal rewrites and 500 error as result. 

Probably due to dynamic vhosts development config.
[zendframework/zf2#1293] Move view template down a directory

- View scripts for controller actions should resolve based on the module
  namespace
Don't blindly override item invulnerability when merging

use highest value possible. Fixes inventory drops disappearing when blown up by creeper explosion.
Pom to support sonar.

Need a separate pom because load-time weaving doesn't play nice with cobertura. So, we compile-time weave for the sonar build.
Wrote a JNI Wrapper for encoding Ogg Vorbis.

Hey, Here's my rough draft JNI wrapper based on your makefiles. Thanks for saving me that tedious makefile editing. The sample rates and quality are hard-coded, but it works. It's also always on stereo right now.

It's also dog slow. 14 seconds to encode an 11 second ogg.

Anyways, it's a start, and i don't see this code living anywhere else.
timob-7355: fixed children[] for tableview row

JIRA ticket: https://jira.appcelerator.org/browse/TIMOB-7355
Testing steps in comment
[TIMOB-7530] Fixed unwanted scrolling in certain applications.
FakeFS spec scaffolding

What remains is to employ it and that involves very personal tastes about where and how example files are stored, naming conventions etc etc.  Consequently I've not implemented that.  Rather the lead devs can implement an prototype example of the style/layout they want.  The helper methods should be self explanatory.

From a quick review this will substantially simplify the specs - no more mocking these things.
Bugfix for gitweb integration

Currently gitweb integration does not use configuration key "project". The attached patch fixes this behavior
Trigger error handling
Add. Added DeubFooter to middleware to help with debugging.
[Skeleton] Check if defined datetime fields same as index.html.twig skeleton

show.html.twig doesn't check if datetime is defined previous the use of |date filter.

Added the same check than already implemented in index.html.twig
Rewrite to convert to link based functionality.
The About popup displays #8230 rather than ellipsis

I know... this is a new low in mentioning the trivial.

When I display the About window/popup, the string:

-  About MythTV Android Frontend#8230

is on the top line.

Adding an ampersand in the about_title string allows the About popup to display what I suspect are the desired horizontal ellipsis.
mathjax re-rendering after edit

The Mathjax-Plugin did not rerender after the paragraph was edited. This fixes it for me. Also adds proper requests to the MathJax Queue.
added support for hive metastore remote mode. [2/2]

added support for hive metastore remote mode.


 crowbar.yml |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Crowbar-Pull-ID: c998ac2de47cdc38bd201d704eebe1518c0d7cd0

Crowbar-Release: fred
Added comment functionality.

Everything is green except one issue with Article deletion.  Run cucumber and you'll see.  Don't forget to rake db.
Update some old links to Android Market
Edited to use File.expand_path in requires

Fixes some issues for people using RVM
Develop

added trigonometry example and fixed a bug in ofPolyline::arc that wasn't drawing correctly when the angle was less than 360
Use the command bash builtin (see "help command")

... obviating the need for any hardcoded mvn path
Fix pdnsnd service

This makes pdnsd service depend on network.target, so that it starts only after the network has been fully configured.
Fixed issue where it was trying to save screenshots for non-request type...

... tests
Added additional code to check if an element that is NOT an array has at...

...tributes and if so add them to the output. Previous code ignores elements with attributes that aren't arrays.
Patch from Jim Brownfield to allow datasources to be enabled after add
Ticket/13 Make repo installable

Resolves some issues that broke the extension straight away. Now it can be uploaded to the `ext/` directory and not throw errors all over the place.
LPS-25254_Quartz_Shed_init_fails_sharding_actived
Deploy Version 1

This deploys the current state to the server
INTSAMPLES-77 - Fix Travel Example

For reference see: https://jira.springsource.org/browse/INTSAMPLES-77

* Use MapQuest Traffic Web Service
* Improve User Experience + Polishing
* Add support for multiple cities
JBIDE-13061 Fix for failing freemarker tests on JDK 1.7
Incredikernel ics
Added whitespaces before each double point in french translation file

French typography requires that we put a whitespace before and after a double point (":").
Added the whitespaces in the traduction file (mainly form labels).
LPS-24057 JavadocFormatter --update option

Adds option to "update" Javadocs for a method. In other words, if a method has some but not all of its signature elements (i.e. params, return, or exceptions) represented in its Javadoc, then the JavadocFormatter will insert the missing Javadoc tags. This should be especially helpful for an engineer to run after modifying signatures of existing methods.

To run, execute ... ant format-javadoc -Dlimit=SomeClass -Dupdate=true
QGraphicsView port
Completed nn translation, added nb translation and did minor corrections to english original. Market descriptions included here.

Hi nguillaumin!

I've completed the nn translation and added a nb ("Norwegian (bokmål)") translation. In the course of translation I also did some minor corrections to the original english: removed a space before a question mark, upper case to lower case in paranthesis, and removed superflous (I think) dot.

Market descriptions:
nn:
OSMTracker let deg spore turane dine, merke av viktige punkt ved hjelp av eit notat, foto eller stemmeopptak. Sporet kan du så eksportere til GPX-formatet som så kan brukast med ulike OpenStreetMap-verkty som til dømes JOSM. OSMTracker er inspirert av OSMTracker for Windows Mobile.

Spor turane dine med notat, stemmeopptak og foto for bidrag til OpenStreetMap!

nb:
OSMTracker lar deg spore turene dine, merke av viktige punkt ved hjelp av et notat, foto eller stemmeopptak. Sporet kan du så eksportere til GPX-formatet som så kan brukes med ulike OpenStreetMap-verktøy som til dømes JOSM. OSMTracker er inspirert av OSMTracker for Windows Mobile.

Spor turene dine med notat, stemmeopptak og foto for bidrag til OpenStreetMap!
Standard Test Cases.

Added Standard Test Cases.
Temp Stats Page
Table Client commit

•	the table client
•	relevant core updates
•	table tests
•	pom.xml is updated to reference apache commons lang
•	Updating useragent to v1.1.2
LPS-24843
Contact Maps Visualizer bug fix.

There was an exception raised when certain mouse actions were performed. I have fixed it and updated the plugin.
Federal units by region and vice-versa

Basically added the file `brasil/vocab/geo/regiao.py` and did the necessary modifications in `brasil/vocab/geo/__init__.py`, `README.rst` and `docs/HISTORY.txt`.
added validations groups Registration

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -
Update 1.1.md

在“系统类型”一行中，若显示“x64-based PC”，即为64位系统
原文为32位系统
Feature/master/add tcp provider
More from Medoff and blacklisting changesets from Kostik (see wiki)
Minor fix at the end of Chapter 1
Удаление повторяющегося кода.
Фикс для ютуба

Я помню, что эту штуку делал для нульчана, потому что там эмбеддер не всегда отображался рядом с картинками.
Ещё один фикс

После компилирования в опере аякс был сломан.
Fix for no longer works after a period of inactivity

https://github.com/richardwilly98/elasticsearch-river-mongodb/issues/10

Hello,

Yesterday i've modified elasticsearch-river-mongodb : this fix the issue #10.

Based on log ( I've turned on DEBUG onto ElasticSearch), 
i'm seeing that newer entry are added

<pre><code>
[2012-06-11 11:45:17,795][DEBUG][river.mongodb            ] [xxx] [mongodb][yyy] Update operation - id: zzzzzzzzzzzzz - contains attachment: false
</code></pre>

and with graph reports ( munin with https://github.com/revinate/elasticsearch-munin-plugins ),
there's activity for 'getmore' calls :  
http://imgur.com/jhbQW 

previously it was failing after 1 hour or so ( green line at bottom ).
Please apply my patches

See commits log
changes in music plugin

it's all related to this: https://github.com/emesene/emesene-supported-plugins/issues/54
Cleaned up and added helper functions

Moved each CTile* tile, and CMap* Map to the header, and just kept using the same one instead of constantly creating new.

Added a GetTile function to grab the tile, etc.
Newest bug fixes according to 'geda_test_result.zip' examples
Perello louis

Bonjour,

Comme je vous en ai parle en cours j'ai un souci sur un test qui ne passe pas dans spec/appli1_spec.rb.
Celui ci étant construit comme les deux tests le précédent je ne comprend pas ou se trouve mon erreur pouvez vous m'aiguiller ?

Pouvez vous me donner votre avis sur mon code afin que je puisse l' améliorer et finir.(Sans parler de factorisation de code car je sais bien qu'il y a plein de test ou j'ai oublié de le faire.)

Merci,

Cordialement,

Louis Perello
Updated blackberry.system and blackberry.identity to use webplatform's

With the PPS now in the webplatform, I've added a smaller helper class for reading from PPS objects located in lib/pps/pps.js. This request includes updates to blackberry.identity and the readonly properties from blackberry.system (excluding the methods which require PPS's delta functionality since we only can use FULL_MODE for now).
Omx for jb
A JavaScript extension template for developer

This template is designed to help developers get started writing their own WebWorks extensions. Download and modify the source and make changes throughout wherever you see a "STEP n" instruction.

Includes a unit test sample application.
The Initial version of an OpenEngSB - Visual Studio - Plugin
interpolated-mappings

Adding the ability to template variables we're pulling from LDAP.  This will allow people to combine multiple ldap fields to populate a socialcast field.
Make sure that MergeQuerySet slices are always slicable

MergeQuerySet returns an itertools.chain object which is missing behaviors that are expected (like subscripting and slicing)
Ics

I've pretty heavily modified the ICS calculator (Not quite sure what changed from ICS to JB - I started before that). You can quickly test it out here: https://play.google.com/store/apps/details?id=com.android2.calculator3

New features:
-History (as a dropdown)
-X,Y graphing
-Matrices
-Hexadecimal and Binary support

and a lot more. The package name changed so I could put it on the market.

I hope I'm doing this right. I'm not sure how I'm supposed to be sending my changes.
Fix org-mode todo and done faces

I've written a detailed explanation of how "bold" is intended to work in terminals, near the bottom of https://github.com/huyz/dircolors-solarized
Bold must not be disabled, otherwise we lose access to Solarized colors such as orange and purple; we just need to pick different colors, in this case simple "green".  Neither selecting "bold green" or disabling bold in terminals is Solarized-compatible.

Also, reverse-video does not work well with sparse trees.  So bounding boxes are better.
Test for has_many_through_association

Hi Andrew, 

As requested I've added a spec to your regression test suite. It should
fail with select_with_include 0.0.1 (without patches) but pass with 0.0.2.
Please pull if you like .. and tell me what you think.

All the best, 

- Johannes
Better UI for `attribute confirmation` fields

This pull request provides a better UI for confirmation fields – specifically it now validates `attribute` / `attribute_confirmation` only after tabbing out of the `attribute_confirmation` field. It involves two changes:

1. Don't trigger validation if the `attribute_confirmation` field is blank.
Currently as soon as you focus on the `attribute_confirmation` field, the corresponding `attribute` field registers an error – before the user has even had a chance to type anything. That sorta sucks.

2. Don't validate the `attribute_confirmation` field on keyup.
I'm not sure keyup makes sense here. We don't want the user seeing an error until they are finished typing and have blurred off the field.

Unfortunately, this *does not* fix the issue of the validation registering (at least arguably) on the wrong field. I believe the expected behavior would be for the validation to display on `attribute_confirmation` rather than `attribute`. As @bcardarella pointed out, however, this is how Rails validations work and so this feels like the wrong place to be addressing that issue.

@bcardarella: I've updated the tests as well. Let me know if you'd like me to make any tweaks.
Added coap doc

Kanske behövs mer? Isåfall får det fixas efter läsning.
Japanese translation of episode 320
ICS: update common device and add bluetooth main configuration

- this commit updates pulls in latest changes of TheWhisp, 
- add bluetooth main configuration to identfy device correctly and
- got back to google default gps configuration to avoid problems with intermediate position
rowend/13-admin-payment

translate admin payment folder
Gingerbread

Finish the translation of Chinese
Make email regex more forgiving

This is a somewhat more forgiving email regex, allowing TLD that are more than 4 letters long.

As you can see from this [list](http://en.wikipedia.org/wiki/List_of_Internet_top-level_domains) there are a couple valid ones that are greater than four letters (`.museum` and `.travel`).
New property to Client

Added property resultCheckDisabled and method disableResultCheck() to Client. 

This is related to this pull request: https://github.com/ddeboer/DdeboerSalesforceMapperBundle/pull/8
Add additional packages requirements to documentation

I was installing this other day on a new Arch linux installing and was getting a few warnings about missing themes when running `gnome-shell-google-calendar.py`.
Added support for Gnome Online Accounts

Instead of prompting for email and password use the Gnome Online Accounts introduced with gnome3. Additionally the account-email is stored in a json file.
Add timestamp to build line console output. 

Add timestamp to build line console output. Update AjaxMin version.
Requires PEAR Installer (version >= x)

Without upgrading pear before installing required tools, I see several "requires PEAR Installer (version >= x), installed version is y" and jenkins is missing some tools.
Intsamples 22

completed two samples satisfying this JIRA:

1. Showing how to use a different serializer/deserializer from the default CRLF
2. Show how to build a custom serializer/deserializer to map to/from your custom domain objects
Feature/social auth doc

There it is: the long awaited doc!
Began unit testing

Basic tests for embedding HTML5 style YouTube videos
Update Ukrainian/main/framework-miui-res.apk/res/values-uk-rUA/strings.x...

Перевел framework-miui-res.apk
[ARQ-719] Enable column sensing for FlatXmlDataSet
Use inverted color for selection

The current color used for text selection is much to close to the background color to be visible, especially when only a few characters are selected. By using the inverted theme's background color (base 3 for dark and base 03 for light) the selection is now much more apparent.
fix rawkey duplication BUG
Es muy buena idea. De este modo podemos ver además que cosas genera cada vista y averiguar cuales tenemos que reescribir. Por ejemplo veo que el logo de fondo en la pantalla de sign up se pone en [base/app/views/layouts/_devise_style.html.erb](https://github.com/ging/social_stream/blob/master/base/app/views/layouts/_devise_style.html.erb) (en social stream). ¿Debe estar eso ahí?
If password storage isn't improved, I'd suggest adding a FIXME here.
For consistency, it would be nice to use {fieldLabel ...} for all the <label> tags.
AFAIK Chadoe did this by purpose because we got problems with other runtimes.
@chadoe: you still remember?
this seems unnecessary?
Seems like it should be `'files:simpletype', $mimetype`, no? It doesn't look like you're passing the mimetype in at all.
There's only 1 right now, but should we be numbering them? There will certainly be more in the future.
I've made an issue for this: #43
In case of an empty string, better just return an empty array(). Alternatively, Let parse_ini_string() raise an error and throw it as exception.
:heart: Only issue now is that the flash looks like it's having a heart attack. I'll clean that up into something less epileptic.
@andyvb I was thinking more of the scoped attr\_accessible that Rails 3.1 gives us (attr\_accessible :attr1, :attr2, :as => :admin), but yeah, :without\_protection would also work and would probably be more backwards compatible. For 3.0 and earlier there's also a Railscast that teaches how to do something similar to 3.1's scoped attr\_accessible.
I'll backport this commit to 3-2-stable (if it hasn't been backported already).  I'm not sure about a release date for 3.2.2 though.
@pauldacus exactly
It was done here: https://github.com/harrah/xsbt/pull/258. The guy says there was NPE when using git on windows. I use mac and don't have access to any windows box so i can't tests what it does or does it work. I just trusted the previous code.
Awesome!  I'll get these pulled in a bit. Thank you!
And the file *does* open in Office Web Apps. So yeah, it appears that this is likely the shared strings issue. If there's anything I can do to help let me know.

On another note - I do not appear to be having any issues using the downlevel i18n. I've got four test failures in the gem but I see the same four test failures on the main branch as well, so those must be from some other bit of versionitis.
Собрал мастер, все работает.
AMD allows absolute paths, but that requires some configuration of the search path in the top-level HTML file.  I find that "no configuration required" relative paths are a better choice for library code, which can then just be dropped in.

I'm assuming your last sentence is referring to "rebasing the AMD changes on top of Narcissus HEAD, using the simplified wrapper"; I'll work on that today.
Ooopsies.  Reverted and tested, still seems to build alright.  LMK if you continue to have problems.

thx
ken
I have slept over the issue and came to the conclusion that we should make our XML parser compliant to the standard for the mere reason to apply the principle of least surprise for users of the framework. The plain existence of our discussion indicates that the current state is not optimal and will repeatedly bother people who stumble over the same issue. When implementing the quoting as defined in XML, we avoid those discussions in the future altogether.
Note to self: fix it.
Newbie sorry i know i'm lame but you have to learn sometime.
@games, if you cannot setup anticheat - it's your problem, not software. R2 passive AC perfectly detect HH - his tested on all HH features long time ago.
you can remove this `if` statement as the real fix is [here](#L3R907)
Maybe I am misreading the above note but it looked like that heading was the changelog for 1.1.1 (which evidently just hit maven central) but it says 1.1.2. If the heading was added for future changes to be added to it, my apologies for the noise.
We shouldn't be declaring variables in if blocks but rather at the top of the function
Here you can use the `&&` operator instead of `and`, but this will not change the code behaviour.
Sorry=)
Argh... this test never worked because of line 143. It _should_ be: 

    long taskId2 = addTaskResponseHandler2.getTaskId();

(see the 2nd 2? :/ )

I've corrected this in my upcoming commits. But this just wasted 2 hours of my time.. :_(
по координате "y"  в "-"  и "+" будет всё литься в логи. не зависимо от числа.
Я только не понял, проблема то в чем? Как возможно присумонить игрока и уронить сервер то?
yes that needs editing :(
Nunca he entendido porque poneis esto ... XD  debug?
A workaround is to just add 4 hours from Isyak (current time). We will get around 00:30 on the next day.
非常感谢！5555太感谢您了！
yep!! thanks++

I wrote the readme yesterday a bit late :-)
The bare 16s in this function should actually be requesting the block size from the internal cipher, right?
Great.


-------- Original-Nachricht --------
> Datum: Wed, 15 Feb 2012 03:41:35 -0800
> Von: alexkoon <reply@reply.github.com>
> An: Christoph Kutzinski <kutzi@gmx.de>
> Betreff: Re: [jenkins] JENKINS-12735 Version ranges in dependency graph resolution for up and downstream projects. (400c18b)

> I will try and revert back and recommit with a new pull request with
> changes as suggested.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/alexkoon/jenkins/commit/400c18b1d1d80523247d7800a4e374279dc2989c#commitcomment-969003
There's a bug, introduced either here or in one of the next commits:

Loading the option dialog the 1st time, change one value which is not corresponding to anything displayed and watch the model change
This only happens once on initial reload.
i am glad you have the !capped part as one won't need with padding.  will review.  we don't really want to add a knob but improvements are possible.
Hmmm, I like your idea about eliminating the need for SHAD-256. With the new properly-allocated seed, I don't need to bother SUBSEQing it, so I'm just going to remove the SUBSEQ and not bother with START/END.

I'll ASSERT that CIPHER exists, since a generator should always have one. This will let me eliminate CIPHER-NAME, since I can just REINITIALIZE-INSTANCE.

Great input!
Then the best_not_good = min(no_ok) will not be defined and it may raise an exception no?
No specs for this?
I forgot: THANKS thommey for the recipes.json file he parsed off the wiki!
Plus removed `Vector3.absoluteToRelative` and `relativeToAbsolute` as per discussion baf0f65e1db76cc95e3d1a3cf759861b807fa177 (no references to `Object3D` in `Vector3`, also this functionality is now subset of what `Object3D.applyMatrix` does).
a test for this method would be awesome
That should be amo_user
Then there is absolutely no benefit, and it is just sloppy.  Reading a long and down-casting to an int is bad enough, no need to make it worse.
I would create a series in the setup and use it for all the tests rather than creating it everytime, makes the test easier to read. you can call it something like  SeriesMonitoredWithBackLog, SeriesMonitoredWithoutBackLog ....
Can you add in the comment why you need this?
? it's pulled from google since this commit.
These functions are starting to get a bit hairy. Lots of setting and re-setting of expanded_xml and new_xml.
The re-setting gives me the feeling we're being sub-optimal.

If also feels like unpack_rsc_ticket_template() and friends should be returning xmlNode* instead of a boolean (and NULL instead of FALSE).

Could we get some comments about the general flow of these functions?
probably dont, I'm a bit retarded with such things
If possible, I'd put the lock-related code in an `#ifdef` block so that it's not used at all when `QL_ENABLE_SESSIONS` is not defined.
This way, we remove the requirement of linking to libboost_thread if we're not interested in it.
Excellent. Sounds like fun.
Don't you need exit; back here?
Thank you  for the commit, would you mind modifying this part of the commit so it only contains the additions of the nkConnect lines please ? (presumably you should also change your github secrets as you've just shared them with the 'world' :(
@lericson I didn’t that and just tested `?delete` subresource only, but when I passed `'delete'` into `args` parameter it was stripped in `canonicalized_resource()` method so authorization failed.  When I passed `'?delete'` into `key` parameter it was escaped (`'/%3Fdelete'`) in `canonicalized_resource()`.
Use xla() function (better to escape quotes also in all javascript code)
use xla() function (within javasscript, better to use xla rather than xlt)
Could you follow the established pattern in this class and move the above logic in a separate method called "getExecutionEnvironments()" and then set it like "bundleProjectDescription.setExecutionEnvironments(getExecutionEnvironments(bundleProjectDescription));"

In addition the getExecutionEnvironments() method should not override any existing execution environments in the manifest. It should just add the correponding Java execution environment if it is missing.
ungültiges Bild oder genereller Bildfehler
@davisagli, it's patched with something that takes *args and **kwargs, so the defaults dict is not set properly.
That's... the penlight. It's not a pen, it's a light.
will attempt to.
We're relying on mkdtemp to be unique inside the top-level `temp` directory. From the docs: "Creates a temporary directory in the most secure manner possible. There are no race conditions in the directory’s creation." The filename inside the directory is really inconsequential since a directory will be created per file downloaded. Later on, the file will be moved to it's final location using the path cleansing utilities.
I think we can postpone it for some time now,lets keep the complexity to minimum level.
Oh no ....

Need more work ....

jquery-ui integration ....
por qué no un `unless` en vez de dejar el bloque `if` vacío?
I dont quite understand how the name is too ambiguous, and what i think is that """ """ will be used for multiple lines of comments, and it is for the documentation of a class.
There are various options:

1. You could have a select list of cycles that the client could select from, or leave unselected for "no cycle."
2. It could depend on the context; e.g. an "add piece" form on a cycle page could include a hidden "cycle" input that is set to the cycle being currently viewed.
Отдельного не будет, пока оставлю, буду думать. Нет, причина в том, что если так продолжится дальше - он может немного хуево повлиять на работу плеера.
didn't work :S
Sorry, I was explaining it poorly.

Yeah, line 99 is the problem. If a block is given, nothing will be added to `res`, and an empty array will be returned.
I already removed the Data class model, so it's less relevant.
Top/Bottom.

If you start out with a menu like this that has dropdowns - http://i.imgur.com/FcTg3.png

When you collapse it, and open the dropdown, it looks like this -  http://i.imgur.com/rlQok.png
This change breaks the MapQuest layer in GeoExplorer.
This does not mean that it was a login error, could have been something else.
I'll tell you that tomorrow, now it's time to sleep for me ;)

sorry!
see you
Pascal

On Fri, Jan 6, 2012 at 12:09 AM, David Y. Ross <
reply@reply.github.com
> wrote:

> Now we're getting somewhere :)
>
> "cap deploy:setup" does in fact exist (and creates the shared directory).
> It doesn't seem to run automatically when you do just "cap deploy" which is
> ok. But I still don't see how to setup play to log to that directory... did
> you do that manually?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dyross/play-capistrano/commit/97154f1c9ea0d7d276875b2f0c92fd6ac40ecdc8#commitcomment-842020
>
ahh, youll see in a later commit I eventually found a label helper, but it wasn't allowing you to click on the label to focus the input, which was annoying. Looks like you're way is better
Oh, right - for reference: https://github.com/notanumber/xapian-haystack/issues/90
Breaks GCC compiling? Not at home to test ATM but forum reports indicate this is likely.

http://www.trinitycore.org/f/topic/5760-building-the-core-error/page__view__findpost__p__36567
http://stackoverflow.com/questions/2951273/pure-specifier-on-function-definition
you're right, I'll make a pull request !
mira la linea esa justo, pone pProto->Class == ITEM_CLASS_WEAPON y debería ser  pProto->InventoryType == INVTYPE_RANGED

Con meter este commit valdria ;)
There is just one particular problem on these atleast for me, I can't get the dummy in combat from mount by any means it perma reset to yellow, which don't allow me to apply the skills for some reason. There is one more problem the instructors don't take me as on mount with lance after taking the quests.
That was my fault writing sql code in my spare time working on a Saturday. Just re install the world db
Yes, it totally is.
ここにも `desc` があると嬉しいですなぁ
i know this change is a bit hard to read, my formatter messed up..
thanks for the info, i would say, open an issue :)
Before I make more comments I should say you are definitely moving in the right direction, and the work you did yesterday was good progress. If my comments sound critical I apologise. I am feeling energised and enthused by the direction this code is moving, and the new feature work it will enable long term, so I am keen to help the code improve.

Symbols in headers, including headers guards should not begin with underscores, as those names are reserved for the system and the C library. A better name for you header guard would be METASERVER_HPP or METASERVERNG_METASERVER_HPP depending on how cautious you want to be about keeping names specific.
Each researcher in this list should be linked to its respective resource (as you did in `find-researcher.html`).
You don't need to put it in event_combinations_gen_5.txt if there's only one sepcial move.
Okay, so this definitely wasn't what I wanted. I really wanted to hide this merge and commit from GitHub. Is that possible? What should I have done differently?
what the fuck is your problem?
Apologies. This was not intentional. Will remove comment. And other generated comments
CS: `catch` should be on the same line as the `}`.
This seems incorrect. As this file needs methods on ActiveRecord::Base, it should require it. Most users will be using bundler, which will require things in an unpredictable order.
@jmikola @stof how about just not putting null values into the array rather than filtering them out again afterwards?
While the goal of MiniColors is a minimalistic approach to a color picker, I'm not opposed to additional features that make it more usable.  This seems like a good concept, but I have a few issues that prevent me from accepting this pull request:

1. This should probably be a setting (disabled by default)
2. It's possible to get an improper value when using the textbox.  For example, try setting it to #ZZZ and then hitting done; read the value and you'll get '#' (acceptable values are a valid hex or empty).  In fact, this could probably be refactored to use the same logic that controls non-hidden input validation.
3. It's not aesthetically pleasing.  I'd like to see a better 'Close' icon instead of a button. We can work on this if it gets to that point though.
Ok, will remove and update later. Now one commit is only for one thing. I can update all AMD "issue" in another commit.
That sounds great François. All I'm saying is that you can do that the
right way (so it will work with both MT4 and MT5...and even eventually
MT6) or you can continue on with the plugin's short-sighted, barely
extensible programming techniques.

Instead of replacing MT 4-specific code with MT 5-specific code,
refactor it by **abstracting out the variable parts** into an MT4
compatibility module, create a similar MT5 module and then add a new
abstract compat module to provide the right code based on the MT
version.
You changed spacing. Also, you can always do git diff on a file to see this same output.
Hey Daniel, do you want me to merge this in?
It gives me an error! Check if the file exists!
Is there a reason we were hiding comments from users that are not online? The way it was people who were not logged in could not tell that commenting is an option.
Sorry it is just for debugging purposes ill add the documentation
К сожалению убивает все события внутри поста, поэтому возвращено назад.
_print_comment is gone from TB2, that was likely the TB1 vs TB2 change which broke things.

Work at `done_testing` time is better done by adding an [EventHandler](http://search.cpan.org/perldoc?TB2::EventHandler) for the [TestEnd](http://search.cpan.org/perldoc?TB2::Event::TestEnd) event.  This will guarantee your module will work even with non Test::Builder based modules.

As for adding a message to the log role, I don't understand your module to know what that's for, but it looks like you want to make, issue and handle your own event rather than co-opt an existing one.
I think it was removed in commit (c69ec15e967b6f00c5b6e05d476de5fbb5a398a5) though I am not sure
That's correct - backbone talks to the local couchdb, and the local couchdb handles replication with the remote server.

The general idea behind CouchDB is to offer a two-tier model - client and CouchDB. You can't really process incoming data on the server within CouchDB, other than to validate it. You can stream change notifications out to another process, if you really need to do some server-side work on data.

What I'm finding is that that's rarely necessary in many cases, if I approach things from the right angle. You have the utmost flexibility in your document structures, and they're pure JSON with a REST API. Once I stopped thinking about things in terms of relational database models, the document approach became clearer, and new solutions to old problems opened up.

You ought to take a look at it when you have some free time. It took me around half an hour to get CouchDB running on my PC and have my Backbone app interacting with it. Painless, and worth the effort, imo.
I think you probably should be using 'vars_post' instead of crafting the raw POST data.  See request_cgi (which is used by send_request_cgi) in Rex::Proto::Http::Client for details.
By default $res == the number of affected rows...
There seems to be no need to get the Last insert Id (fails with Oracle 10 and 11g)
Can you point me towards the other patches you're applying? I'll see if I can reproduce the conflicts.
Because if you don't have them specified explicitly you might end up with unresolvable deps such as trove. Left it hear to emphasize that they are really needed, but I can move them to parent if needed and put the comment here (it's anyway mentioned in the docs).
@hferentschik I agree. I'm going to add the methods simply to `Validator`.
Should I change this to `symfony/twig-bundle`? I believe I copied this from [FOSUserBundle's composer.json](https://github.com/FriendsOfSymfony/FOSUserBundle/blob/master/composer.json).

Technically, FrameworkBundle is still used for the `EngineInterface` type-hint in the controller, so perhaps both should be present.
ok bang it up then I guess
done. sorry for removing it.
Continuing thought from davglass/shifter#11:

This is where `Y.Template.Micro#precompile()` falls down, in my view, when used as a precompilation engine for `shifter`. It would be preferable to pass `{ "source": true }` or something as the options for `compile()`, and only `revive()` the generated function when `source` is `false`. (Conversely, when the `source` option is `true`, return only the generated source, as with `Handlebars#precompile()`)

```
if (options.source) {
    template = "function(Y," + (options.variable || 'data') +
        ",htmlEscaper){\n" + source + "\n}";
} else {
    template = this.revive(new Function ('Y', options.variable || 'data',
            'htmlEscaper', source));
}
```
Please inherit it from SimpleSplitList
Right, I'll check, but I think we don't need...
Yes, it has been moved to the parent pom.xml.
Perfect. I'll assign it to myself for now. Thanks :)
Do we really need that commented code in there?
aka arg._is_direct
Just to give the reasoning about why this is called '*AES128CounterRNGFast'.

The DefaultSeedGenerator requires Internet access (it goes through the 3 random sources).
If it isn't available, it slows down the initialisation.
The SecureRandomSeedGenerator uses the SecureRandom seed generator implemented by Java and shouldn't have a start up delay (unless you have evidence to the contrary?).

Perhaps the classes need renaming because now the AES256CounterRNGSecure is inconsistent with AES128CounterRNGSecure
idempotent receiver is the important thing, and that can be achieved with discarding duplicates, or having no side-effects, or side-effects ends with the same result
what if the cause is not serializable? endless loop?
Well it did come in handy, so I'll add some case objects.
so nice to be getting rid of this hardcoded stuff.
ya.  I think there's room for improvement in our error handling in this arena in general.
I think you want to say "I hope you enjoy it" here.
same auth issue as above.
clearContextInMessage
this is only for the season/tvshow containers, not the actual episodes. here episode_number means number of episodes. (PopulateObjectFromTag is not called on videodb folders).
preferable m_parser_split to match other naming and what it is we are using.
This is a general problem in xbmc. In JSON-RPC I also have to manually retrieve the stream details but there I have the advantage that I only have to do it if requested by the client. Will this metadata be cached by upnp until something changes or is it retrieved everytime a user goes into his video listing?
I think it would be safer if this was to default to false?!
Don't  forget  Docstrings. Let's try to be strict about pep8 compliance from now on.
methodSignatureEquals ?
I don't know how I feel about this plugin...
What's happening here. Please add doc comments wherever there is something "magical" happening.
Fix the comments. More below.
resetTimer can be called several times during one operation
it should not touch dom only set timeouts
is ``inbound`` == handle ne null? If so, we should remove the var
Minor conventions note – boolean properties are usually defined like so:

    @property (nonatomic, assign, getter = isProcessingChanges) BOOL processingChanges;

So the property can be used as either `if (layoutManager.processingChanges)` or `if ([layoutManager isProcessingChanges])`. This matches [Apple's coding guidelines](https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/CodingGuidelines/Articles/NamingIvarsAndTypes.html#//apple_ref/doc/uid/20001284-1001757).
Then wouldn't it be better to move this to `+load`, since that's only ever
*ever* called once?
Oops, debugging leftovers.
`.As<Function>()`?
Probably should use String.split() on this instead of bringing in the Scala StringOps class. Java's strings have a split() function already. It treats the splitter as a regex but it's fine for space.
Awful Python style.
This should be a comment instead of a string. This isn't a docstring, so the triple quote syntax shouldn't be used.
@jsalva: the `convergence_threshold` closes the bracket.
I'm not convinced this belongs in the module. How much of this could be done in test_pad?
Yes, but it should return ERR_NEEDMOREPARAMS.
On Sep 15, 2012 7:43 PM, "ElementalAlchemist" <notifications@github.com>
wrote:

> In txircd/user.py:
>
> > @@ -120,5 +120,17 @@ def irc_PRIVMSG(self, prefix, params):
> >                  if u is not self.data:
> >                      u["socket"].privmsg(self.prefix(), c["name"], message)
> >
> > +    def irc_NOTICE(self, prefix, params):
>
> I don't know. I mostly put the PRIVMSG function again. Though, now that
> you mention it, it does kinda assume there's a name and a message attached.
> Having said that, running it without parameters simply throws an exception
> (which seems to get caught somewhere, and the IRCd continues without
> issue), just like when an unimplemented command gets put in (which really
> should eventually return the proper numeric).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ojii/txircd/pull/5/files#r1613852>.
>
>
Ah sorry, I had overlooked the selectControl management when layers are added and removed. Looking forward to seeing this factored out to a separate plugin.
оставил, так как предупреждение было
здесь и ниже скобочки нужны
нет, эти методы. или что значит этот комментарий?
Table names are commonly provided as symbols with `set_fixture_class`. They're converted to strings at https://github.com/rails/rails/pull/4254/files#L0R439

I'd err on the side of being defensive here, in case an app or plugin is relying on the API.

(The original code makes it clearer that `fixture_table_names` acts as a default value, too.)
This method was created in the initial commit by dhh on 2004-11-23.
I was going for minimal change (only adding the new extension). I'm happy to commit a change to remove the word "Only" here and line 59 if you'd like.
Since we work with 1.9.3+ on master, can we use 1.9 Hash syntax please ? ^^ (sorry to «quibble» with that)
Are you sure? I get `array([1, 0])`? Moreover, [0, 0] doesn't satisfy the correct neighbour invariant...
spaces here
left unchanged for now..can go do that if it is warranted.
On Wed, Apr 25, 2012 at 9:07 PM, Tom Bachmann
<reply@reply.github.com>
wrote:
>>      from sympy.polys.polytools import Poly, factor
>>
>>      if a != 0:
>>          return _eval_sum_hyper(f.subs(i, i + a), i, 0)
>>
>>      if f.subs(i, 0) == 0::
>> +        if simplify(f.subs(i, Dummy('i', integer=True, positive=True))) == 0
>
> No it's 77 without the spaces.

Sorry, I see that I inadvertently copied the line numbers, too.
(and do so in the TirgonometricFunction class, perhaps?)
can `set` be used here, so `set(ans) == set(res)`?
On Sun, Jun 17, 2012 at 7:19 AM, Aleksandar Makelov
<reply@reply.github.com>
wrote:
>> -                else:
>> -                    coset_repr[ag] = Permutation(gen1)
>> -                sta.append(ag)
>> -                stg.append(gen1)
>> -                h += 1
>> +        >>> from sympy.combinatorics.perm_groups import PermutationGroup,\
>> +        DihedralGroup
>> +        >>> G = DihedralGroup(6)
>> +        >>> G.orbit_transversal(0)
>> +        [Permutation([0, 1, 2, 3, 4, 5]), Permutation([1, 2, 3, 4, 5, 0]),
>> +        Permutation([5, 4, 3, 2, 1, 0]), Permutation([2, 3, 4, 5, 0, 1]),
>> +        Permutation([4, 3, 2, 1, 0, 5]), Permutation([3, 4, 5, 0, 1, 2])]
>> +
>> +        See Also
>> +        ========
>> +        orbit
>
> What about leaving a blank line before the closing quotes for the docstring? https://github.com/sympy/sympy/wiki/Writing-documentation says this is recommended, though nobody raised an objection. Is there a new sympy policy on this issue?

I think following standard Python grammar/style conventions is always good.
Adds to readability for others, making it easier for developers to maintain.

> On a separate note, the References sections of docstrings should also be labeled as such, according to https://github.com/sympy/sympy/wiki/Writing-documentation ; I'll fix that too in the amended week1 branch.
>

I don't know if it is too much trouble, but if I recall correctly
there were some
docstring reference issues in the permutations.py module that needed
fixing as well.
You didn't write these functions but I think thery were
{{{transpositions}}} (which referred to
[4], whatever that is) and {{{__add__}}}, {{{__subtract__}}}. I don't
know what any of these methods
do and the docstrings don't help to explain them, as far as I can
tell. Can you please
take a look at them and see if you agree?


> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1319/files#r998393
The new assumption system decided it is True, btw

>>> ask(Q.integer(2.0))
True
I change it in new commit
not a big deal, but I think (because it gets 2 elements) that it would be more readable if structured better like:

```
$query->conditions=array(
   $child->relation->condition,
   $child->relation->on
);
```
What it we have 2 `Post` models in different namespaces and both models are used in the same form?
If I am not overlooking something here, $criteria can be null so you might get `trying to get poperty of non object here.`
I was thinking about removing the need for this one too
jack doesn't use bundler, right? Might as well remove these lines instead of leaving them commented out
We have not, where should I add this info?
Could you replace the magic number please?
Sollte von aussen (vom Controller) gesetzt werden
I find this method a bit confusing.  A LST is a *time*, not an angle, right?  There is a correspondance there, but given we have a `time.Time` object, we should probably accept that over an angle (or possibly both?)
I think it helps understanding this problem which is a bit hard to grasp, but since it's not ultra relevant to silex I will remove it.
What does this line above mean?
Shouldn't the user parameter get nuked as well?
You normally want to have the oplog be the same size on all set members.  I'd suggest adding a note saying this, and suggesting that the oplog be resized on all members of the set when resizing.
Again, just "built".
The two existing methods should probably be classes instead. The requirements could then be setup in the constructor and you could have one method per test.
Hmm where is this function used? Is it only accessed from classes in style? Then maybe we should just remove it from the template interface?
I don't know what the windows path does but the unix path does not do "shutdown", it immediately obliterates any running processes. A graceful shutdown path issues a kill, waits some time, then if nothing happens does a kill -9.
Please check if this fits into 79 chars.
Either $avatar_driver or $driver.
IRC: think about how to implement it for windows also.
yea, it matches on /proc/<PID>/stats which is 16 chars... thus rackspace-monit

# cat /proc/5670/stat
5670 (rackspace-monit) S
Ar fi bine să existe o constantă pentru 5-ul ăla.
Ideally this discussion should be cooked down into a single comment, something like:
```
// We have to heir() otherwise this.something in the model
// will pollute other instances of the model.
```
Awesome, that makes sense. It would be good to get _getSubElements() working in this situation but at least the current solution works. Thanks!
np, I'm enjoying adding support for more games.
Still marked as Windows platform
I know this line breaks the `Should_deserialize_list_of_primitives` test in `JsonBodyDeserializerFixture` because `s` will contain the value `"/Date(-59011459200000-0000)/"` which will not parse correctly
NIT: (Low priority) Might be more meaningful to add semi-realistic temp data, so that others trying the app get a sense of the functionality, even before it's complete.

E.g. Acetaminophen, 400 mg, <disp>, Yes, <labels>, <inhand>

<> because I'm not sure what these mean, so you add the label.
Should the second argument (`const char *symbol`) be declared as `ctypes.c_char_p`?
That token, "C0644" is part of the scp protocol.
You're right of course, thanks!
These are markdown-style literal escapes, not edoc-style.
so in essence, the "app" folder is special, all other folders will be referenced with the name of the containing folder.
Yes I think that would be nicer.
Perhaps we could still keep the `install` function and use it here, but not in the sandbox building?
* _calendar / _now / day(indexPath.row) ⇒ cellの日付
   * - (NSDate *)createCellDate:calendar now:(NSDate *)now day:(NSInteger)day