labofsahil/github-event-dataset-2012 huggingface dataset:
2 Minor changes

* wrap line
* use strings instead of symbols

basically #1 minus puts removal
(#12153) Adds list of supported browsers to README.markdown

Self explanatory. Puppet Dashboard supports the following browsers:

Chrome (current versions)
Firefox 3.5 and higher
Safari 4 and higher
Internet Explorer 7 and higher

Fixes http://projects.puppetlabs.com/issues/12153
Configuration cleanup.

- Remove option to create .rackspace_cloud file on gateway server.
  - Rename CC_AUTHORIZED_KEYS to AUTHORIZED_KEYS.
  - Rename RACKSPACE_CLOUD_SERVER_NAME_PREFIX to SERVER_NAME_PREFIX.
Dynamic port allocation

Implemented dynamic port allocation. Added junit test.
Updated Scripts to Version 27... Remove PrePlus ko modules as the Pre2 versions work... Kept PrePlus WIFI driver...

...re2 as they work... except WIFI kept PrePlus driver...
resalta alerta de exito en mensajes
Changes to the behavior of the inline image viewer

I created a system for preserving the original link (when necessary)
without the trouble of loading entire webpages to mark history.

When necessary, the original link is saved to a property called
hrefOverride. If that is detected during expand creation then
mouseover/mouseout handlers will be attached to swap the href on demand.

This works well but it can cause the link to turn blue on mouseover.
Without marking the history this has been made nearly impossible to
prevent in the name of user privacy.

I also went back to the original link on thumbnails so there is a fallback method for all links.

----

On a less closely related change, galleries now display per image captions/titles.
agregando cantidad de fotos en vista de galerias y pep8 de todo el paque...

cantidad de fotos y pep8 a todo el paquete
global leak protection

This is a questionable change, but it prevents the unintentional leaking of globals in when misuse goes undetected.

Addresses issue christkv/node-mongodb-native#524
Remove annoying print when tests run.

Remove annoying "Installing indicies for ..." prints when running tests.
VideoPlayer Sample update for new iOS VideoPlayer implementation

Updates the VideoPlayer sample to exercise the code represented by [pull request 206](github.com/mono/MonoGame/pull/206).

In particular, Looping, Pause/Resume, Mute/Unmute, and automatic frame dropping are tested fairly thoroughly.
Update README.md

Updating the Contribute and Feedback section for continuity between repositories.
fixed incorrect success message on role grant

previously:

#keystone-manage role grant Admin adminUser
SUCCESS: Granted Admin the adminUser role on None.

now:

#keystone-manage role grant Admin adminUser
SUCCESS: Granted adminUser the Admin role on None.
Acceptance tests are passing now

Plenty of api changes as well :)

New acceptance test and unit test libraries.  Left the existing test library alone
Added trd? method

Hi, i just thought, using this gem that i need this method to verify if given a domain exist or not the subdomain.

Example:

```ruby

>> require 'public_suffix'
=> true
>> a = PublicSuffix::parse("www.google.it")
=> #<PublicSuffix::Domain:0x101a7a790 @trd="www", @sld="google", @tld="it">
>> a.trd
=> "www"
>> a.trd?
=> true
>> b = PublicSuffix::parse("google.it")
=> #<PublicSuffix::Domain:0x101a59a40 @trd=nil, @sld="google", @tld="it">
>> b.trd
=> nil
>> b.trd?
=> false

```
revert "do not build camera.cooper and libcamera"

as a consequence of adapting the overlay in framewroks_native
add link to put mapping API in config mappings page
Add require and fix spelling issues

This will attempt to fix the broken Travis CI build, by adding:

    require 'support/guitar'

...to the top of the Mandolin class.  It is important that we can load this class as a separate file to test the use case where a re-loading of a parent class would break subclassed indifferent access support.
restructured again, removed project files
Added a package for LESS highlighting + comment toggle support.
Support * keyword in .feature files
Add property integrationTestsSystemProperties

Add property integrationTestsSystemProperties to store some default tycho properties for integration tests

Currently only sets property swt.bot.test.record.screencast to false but also enable setting of this property from command line
JBIDE-13088 - jdt dependency added to dummy pom.xml because of OSX issue
Re-upload files if they've changed since the last upload

Re-upload files if they've changed since the last upload (based on size and timestamp). Previously it just skipped existing files. Now it'll re-upload them if they've been modified.
Small typo in 2-the-view-layer.md
LPS-25072

Fixes the original issue, and does not cause LPS-25456
[Form] Update MongoDBQueryBuilderLoader for bridge interface

This implements an additional interface method introduced in symfony/symfony#3156.
Fixed URL ;)
New file

All right!!
Fixed a typo: "localStorage.remoteItem()" should be "localStorage.remove...

...Item()"
Update README

Wnated to check update file
Make DISABLE_MEMORY_POOLING work with ARC

This moves the DISABLE_MEMORY_POOLING macro to hide SPPoolInfo completely when it's set. That gets projects using ARC and Sparrow working. purgePool is still visible and uses the previous noop implementation.
Allow to change variables to get your own styles

It is impossible to change variables for bootstrap now. This fix will help to get ability to change variables
Исправления для компиляции

Исправляет все варнинги за исключением `JSC_WRONG_ARGUMENT_COUNT`.
Fixed issue #92 [<access> with no <feature> allows all features]

Fixed issue #92 [<access> with no <feature> allows all features]
+unit tests
Delete the old coffee branch
Expanded the list of ignored files and added directories too

I added some common development dot-files to the ignored file list. Also mirrored over the ignored-file check so it catches directories too. 

Accidentally uploading my .git dir to DropBox was funny the first few times, then not so much.
Corrección de bugs, bueno bichitos más bien

Algunos pequeños cambios que hice en algunos métodos que cuando por algún motivo fallaban lanzaban un NullPointerException "anónima" y era más complicado de debugear, ahora la lanza con un mensaje más explicativo.

Pero lo más importante, controlado un error con el método close(), que permitía seguir cerrando cuando no había ninguna instancia abierta quedando el número de instancias como -1, de forma que al abrirla se quedaba a 0 y daba error al "no estar abierta". El problema se da por despiste programando al cerrarla más de una vez, pero a mi me pasó y me mareé tela hasta saber qué pasaba, y lo mismo le pasó recientemente a JMPergar.

Bueno, y algunos cambios en el formato que ha hecho Eclipse automáticamente... de eso no tengo culpa :p

Espero que te guste, si no te parecen bien no pasa nada. Ya me cuentas.
Un saludo ;)
a few more changes: all extearnal attributes are in place and ComponentInstances should be ok.
Refresh

Fix 'sslverify' parameter usage, props sc0ttkclark
Fix checking of newly inserted checkboxes

- Check if the element is a checkbox or a selectoption, set as checked or selected as appropriate
Boolean typre changed with Equal Form Typea in ModelFilter

Bug fix: no
Feature addition: yes
Backwards compatibility break: no
Fixes the following tickets: -
Todo: -

See also: https://github.com/sonata-project/SonataAdminBundle/pull/666
Add query filters

The test I have written seem reasonable, but do not work. Could you help me with that?
I wrote pretty much the same tests against my actual project/model and they work without problem.

Other than that, this is a straight-forward addition of filter methods to the Query class.
timob-10818: KitchenSink: Android: Need to update the Activity Indicator and Progress Bar test

https://jira.appcelerator.org/browse/TIMOB-10818
Test steps in JIRA.
Initial Version - Facebook Ads API read only operations

This branch has been in progress for a while and I wanted to submit it even though it is not complete.

All of the read-only operations for the Facebook Ads API are here, but I haven't yet implemented anything else.  I wanted some eyes on this before proceeding.

Thanks!
Add lockscreen weather drawables
tweak key mappings
Little typo fix ;)
(1/2) 1% battery increments and no charging light on shadow

Fix the settings for the battd service in init.mapphone_cdma.rc to match those in the 605 OTA.

This allows us to use the 605 battd, which in turn disables the annoying charging light and makes 1% battery increments work.  Only use this patch with the 605 /system/bin/battd (included in a separate pull request for proprietary_vendor_motorola).  Without it, your phone will shut down immediately on boot because it thinks the battery is dead.

Discussion here: http://rootzwiki.com/topic/10191-updated-info-on-revnumbers-cm4dx-gbhdmi-out-1-battery-no-charge-led-fm-radio-charge-while-off/page__st__1250__p__426397#entry426397
Added option "data-graph-xaxis-labels-font-size".

=> Allows one to specify the font-size that is to be used for labels on the X-axis.
(Useful if you have labels that are too long, and, hence, take too much space on your graph)
Add support for a new firmware version.

My phone came with a firmware version that wasn't yet in extract-files.sh.
this a test

whee! hi ty!
Add hddtemp temperature.

Hello xtranophilist,

this patch adds support for the hddtemp utility. The current implementation
assumes that the hddtemp program can be called directly by
unprivileged users. This assumption is usually true for Debian and
Fedora based distributions, Debian uses setuid bit permission, Fedora the
consolehelper utility.

Sorry for the duplicate, but I don't know how I can add a pull request to an existing issue.

It should also fix:
Issue #32
hmm.. Looked at the example on your websites home, and notices that this...

... example was wrong
Completed Ajax Without jQuery Recipe

Created new Ajax chapter and added new Ajax without jQuery recipe.  Also updated authors list as well as wanted-recipes to remove completed recipes.  Corrected minor issue with math/random-integer recipe and ordered list issue with index.html file(s).
Small Idiom Fix

Saw some code that felt too javascript-y.  Thought I'd send a fix.
Various updates

For 7f951c9, see also https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Math/max#Example:_Using_Math.max
<@emsy> it doesnt' work kinda, but u can pick any raid weekly atm
Is this more efficient than separate change event subscriptions that update related attributes?  It seems like it is mainly to update the `details` attribute, which is of questionable value IMO (though I can be convinced).
Ah ok. I am always a bit cautious, because I have had trouble int he past with (non global/static) variables not being initialised. It is a shame that gcc doesn't optimise this.
@igorzi - I modified this test to run with v0.7 (master). Can you review koichik/node@e5438cd5?
This class doesn't need to be public, please keep it package-private.
agreed, this should have been in the changelog.
Fix bug  Edit permition
thanks for the fixes :)
This shouldn't change while the function runs, right? Why not just use this?
Why's that magic constant "RESULT" appended to the result buffer?
IMHO your examples should be updated :)
This should also continue to be installed, otherwise you break `macrubyc`. It depends on `llc` being in the MacRuby bindir. :/
I've routed around this change, so no updates are necessary in parrot.
So you don't really need to comment code to mark this as a fix, because the connection for the fix is automatically done eith the commit message and you can see the diff to previous versions in here.
It might be better to get appVPath to include the trailing directory separator...

what if the file_path is /vpathxyz/page.aspx? It would still pass StartsWith(appVPath)
Name() is ok, it does not change runtime. 
It comes from the openair parser, airspaces with no names are dropped.
I'm checking all strings null termination inside airspaces now.
awesome! that works perfect
Extensions should not be responsible for creating cancellable background threads. This should be handled generically for all extensions.
What about: <code>setPreferredStrategy(LocatorStrategies.DISPATHCER_FIRST)</code> ?
Hmmm ... I like `consistent` than `conflict_free`.
Did you intentionally get rid of the import tips?
Personally I don't really mind (I think these should be in the coding style instructions, but I'm not sure they are already).
Looks good to me. Any objections @slavapestov @erg @mrjbq7 ?
Probably none, now that I think about it. I suppose I should probably change it back.
xD nose pero seria mas bonito comenzar a reparar todo lo que es pvp luego lo pve :(
Well, that's good for testing. Maybe Uint + Uint should always give Uint?
Weil man sonst "nur-definierte" Properties hat und die Definition eh in der Collection steht...man hat somit müll rumliegen
> How many more times do I have to say yes? I give up.
Sorry that was not obvious. Alright, I'll take that in mind.

Anyway, is it so evil that I won't change that until you have good points? I need proof!
Changed some specs to be more behavior driven instead of implementation driven.
True that, make a pull request ;)
maybe we can rename it as MasterSpec?
@crab2313 其实只要黑盒 webqq http package就可以了。
这个函数不是只有3个参数吗
Omg...
http://translate.google.com/#ru|en|http%3A%2F%2Fwww.wasm.ru%2Farticle.php%3Farticle%3D1010028
Is this something that is specific to **ListItemHelper**?  Seems like it is a generic function and should be moved somewhere where all widgets can make use of it.  If that is correct, please open an M5 issue to track this.
Workspace or runtime was prob the first singleton. Snap had to be hacked and should *not* be used as an exemplar
3.2.1 is out :)
arf, I confused ``$this->andWhere()`` and ``$this->queryBuilder->andWhere()``

And no, the field cannot be passed as parameter. Bound parameters can only be used for values
why during ? after
warnings for potential future formula upgraders, please feel free to amend / remove as needed.


```
1) libsoup 2.37.3+ requires glib >= 2.31
```

- this is version libsoup-2.37.2
- the current glib formula in hombrew is glib-2.30.2.tar.bz2
- before upgrading libsoup to any version higher, glib will need corresponding update


```
2) libsoup 2.37.2 looks for files which were last provided with gnome-keyring
   2.28.2 in the source package as:
       library/gnome-keyring.h
       library/gnome-keyring-1.pc.in
```

later versions of gnome-keyring, after gnome-keyring-2.28.2, have dropped the `./library` directory and no longer install the specific gnome-keyring headers or gnome-keyring pkg-config which this version of libsoup-2.37.2 depends on. i'm under the impression later versions of libsoup accomodate later versions of gnome-keyring.
... or is `forms` not iterable? confusing
This introduces another segfault in CDecoder::UploadTexture
in line 1316 transInput.src_surface = m_flipBuffer[index].outPic->render->surface;

I tried to also exclusively lock the decodersection in UploadTexture, but this does not really help -> fglrx segfaults with 0x0

Are there more api-locks to be set?
That's a problem related to haxe.
Haxe adds/modify some methods in Array.prototype (copy, insert, remove, etc.).
These methods' names are traversed by "for var in" which break quite a lot of things...
Looping using Array.length is safer.
Sorry to bother you with these little details, I am trying to port some of your examples to haxe and make them work :)
lxml warnings??

by raising a TypeError before?

```
  File "/home/ralf/code/mwlib/mathutils.py", line 138, in renderMath
    if not len(result) and output_mode == 'png':
TypeError: object of type 'NoneType' has no len()
```
I feel that a bit of text explaining the context objects' lifetime is missing.
Since EdaConfig is a GObject, when is it safe to unref it? When is it unsafe?
What happens when the user unrefs an object returned from 
``eda_config_get_default_context`` or ``eda_config_get_system_context``?
Ouais mais il a aussi dit que visual studio était le meilleur IDE du monde ... à partir de ce moment il a commencé à perdre toute crédibilité XD
Can't you still use reflection to instantiate a class with a private constructor?

I actually don't really care that much. Maybe someone wants to extend this with some real uses.....
That is purposely done: I intended ImageDecoder to maintain ownership of
the Surface. This is mirrored after the VideoDecoder way of handling its
Surface.
The rationale of the original behaviour was the following: if someone only wants critical notifications, he will also want to see when the problem goes away. In your modification, setting notif_severity to FAILURE will discard OKAYs and the admin will never know the failure was repaired.
Really?
I didn't have a deeper look but have a look at the signature of the function tag_bug_detach,  user_id is the fourth parameter.
function tag_bug_detach( $p_tag_id, $p_bug_id, $p_add_history = true, $p_user_id = null )
Oops... this wasn't meant to be in the commit!
I'll remove it and send you a pull request right away...
(@fat and other Bootstrap contributors, thanks for this awesome project, I'm a huge fan and we've gotten a lot of value out of it.)

As for your point "I don't think that generating a bunch of unused code serves the majority of bootstrap users", I'm curious about that. Do you mean "unused in the Bootstrap examples and docs?"  And I say this tongue in cheek, but why not just make this simple go with 1 column so you can get rid of all those unused span3's, 5s, 7s and 9s?   Is 12 col really justified when you're only using span4, span8 and span12?  

All kidding aside, just like Bootstrap doesn't apply all 12 columns of the grid to every page in the docs - you use a span4 here, a span8 there - we want to do the same but with the flexibility of tighter 24-column grid. I know that keeping the Kb low is important, but the grid is so foundational to Bootstrap. For sure, Bootstrap's docs have a great look that works well with wide spacing and large fonts. That doesn't work for many of our projects, because we don't do consumer-focused, public websites.

We are focused on enterprise and b2b related stuff; supply chain, inventory management, shipping etc., and suffice it to say that we would definitely benefit from more granularity than 12 columns.  We have a lot of data-rich, repeating elements that benefit from a tighter grid - e.g. widget-like stealth analytics, so we ended up having to create our own "grid within the grid". But it feels kind of dirty, and it's a PITA to maintain if we want to keep merging in Bootstrap's awesome enhancements. We'd like to be using Bootstrap's native grid, it's easy to not use all 24 columns if you don't need them. But it's not fun to maintain a grid that isn't natively supported or possible to create by default in Bootstrap. 

If that's out of the question, then maybe we can completely separate the current grid into a one page mixin, grid.less seems to makes sense since it's already there. Then maybe consider cooking up another mixin called grid-24.less for folks who need it.  Sorry about writing another chapter to War and Peace here, this particular issue just happens to be the one we care about the most...
+1 for adding a reference to the docs. Note that I added a few line comments above (I don't know whether you get separate notifications for them).
ok, but if we also have action helpers (as opposed to object helpers like gizmo), let's say having a class for dragging objects around the screen, would it also be named helpers? like. `DragHelper`?
oops, forgot about that, thanks
do we need a new `Support` group? and a migration?
@sambaker what do you think of the pattern being like `Awe.array(anArray).forEach(callback,thisArg)`? This would allow us to implement other interesting array operations in future
Good catch, thanks! TBH it doesn't matter (since node creation and starting is idempotent), but I like `before(:suite)` better.
Hi @cbeust,

Thanks for your reply, I will check it.

Brs,
Tim
Yes, I understood what it's good for. And to make sure that two people don't work on the same stuff, I'd definitely want a resource to be locked for more than 6 minutes - most phone calls need more than that... But okay, I can set that to another value in the system settings.

What made me ask is the fact that 6 minutes is a rather strange value - why not 5 or 10?
consider using ``super(BuildStep, klass)`` here instead of ``object``. The implication is that you are then also creating a ``PropertiesMixin``.
.
Then you also want to give args and kwargs to ``__new__``
i don't even know what happened here.
hehe thanks (it's weird to chat in source comments... :)).
It took me two days and a half to understand the algorithm and get this ugly result.
I'm debugging it and working on a cleaner version right now.
This is just the first prototype made for a bigger project full of 3D trees :)

About yours, I still can't manage to compile your sources on linux, so I didn't manage to see the result. Have you some screenshots somewhere ?
We gave it an object, so it knows that when the object is new it should go to the create action and when the object is persisted it goes to the update action. The [form helpers guide](http://guides.rubyonrails.org/form_helpers.html#relying-on-record-identification) contains all you need to know here.
Iam affraid, until @derekjones say yes, i will kept that one as is, since i copy those link directly from his suggestion.
FC work with TDB and YTDB
My fault, this is OK, an error was somewhere else in my code.
modification de l'index de machines (affichage de moins d'infos)
Using this commit for Assignment 5
Nethrar isn't 'extremely out of date' - while the last commit was in late December, the fact is there haven't been any breaking changes to it since then, and due to having a full time job, I've forgotten to update the 'latest build' number on the forum thread / bukkitdev project page. I tested last night, everything still works fine with 1.1-R3. Do you mind re-adding Nethrar support?
Reamer you should youtube of raids before u script anything only advice im gonna give ya cause your only wasting your own time
I feel a bit dumb but I can't say for sure why it's not needed anymore (it definitely was before!).
Wouldn't you want to include all the elements in the jQuery set, and not just the first? Those are the semantics I'd expect.

Also, there is a pretty serious memory leak that occurs each time the instanceof operator is used on a `window` or other host object in IE6/7. Don't know how often a `window` passes this way.
Laughed at this, because I do the same thing
Thanks, I need to look a little deeper into this before I merge (unittests for the issue and such), but this is indeed a pretty big issue
No, the 64bit define isn't there.

@sitzm See ^^
This looks good to me.  Can you add an example for how to use this in git-subtree.txt?
Inconsistent with Guose's description
The signal handlers are racy anyway.

1. it's not safe to do significant work in a signal handler; set a condition variable and return
2. you can't safely mix poll() and signal handlers without using the self-pipe trick

https://en.wikipedia.org/wiki/Unix_signal
http://evbergen.home.xs4all.nl/unix-signals.html
http://osiris.978.org/~alex/safesignalhandling.html
http://cr.yp.to/docs/selfpipe.html
They should be `warning:` or `note:` then; you will have to tell me which would work better in xcode 4.
Likewise:

    current_viewers = response["own3dReply"]["liveEvent"]["liveViewers"] rescue nil
    is_live = response["own3dReply"]["liveEvent"]["isLive"] rescue false

    self.update_attributes(:viewers => current_viewers, :live => is_live)
We can do that -- I keep them around as a build debugging aid, but I
can do the same thing by editing the cleanup function manually when
things need debugging.

On Sat, Jan 14, 2012 at 12:15 PM, Greg Althaus
<reply@reply.github.com>
wrote:
> I more meant to just delete them after the build.
>
> It may not matter I have to rebuild everything.
>
> Trash my vm
>
> Sent from my iPhone
>
> On Jan 14, 2012, at 11:44 AM, Victor Lowther <reply@reply.github.com> wrote:
>
>> the temp cache areas -- you mean the build and chroot trees?
>>
>> I used to have the build cache mount tpmfs'es on top of them for speed
>> and cleanup purposes, but that killed the build on machines with small
>> amounts of memory.
>>
>> On Sat, Jan 14, 2012 at 11:39 AM, Greg Althaus
>> <reply@reply.github.com>
>> wrote:
>>> Yeah. It was a nit. Im mostly done with cache update.  But my vm exploded.  I'm trying to resize it now.
>>>
>>> It help if the build killed the temp cache areas. they get huge.
>>>
>>> Thanks,
>>> Greg
>>>
>>> Sent from my iPhone
>>>
>>> On Jan 14, 2012, at 11:33 AM, Victor Lowther <reply@reply.github.com> wrote:
>>>
>>>> Arrgh, I thought I had caught that one.
>>>>
>>>> ---
>>>> Reply to this email directly or view it on GitHub:
>>>> https://github.com/dellcloudedge/crowbar/commit/f15f7ac2d6343bca5d2b25a33508924b9099c526#commitcomment-867494
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/dellcloudedge/crowbar/commit/f15f7ac2d6343bca5d2b25a33508924b9099c526#commitcomment-867502
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/dellcloudedge/crowbar/commit/f15f7ac2d6343bca5d2b25a33508924b9099c526#commitcomment-867509
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dellcloudedge/crowbar/commit/f15f7ac2d6343bca5d2b25a33508924b9099c526#commitcomment-867552
I think it can put off new contributors. I only have so much time. if I have to implement a solution 2-3 different ways up to the level of including passing tests, thats a lot of time wasted. If instead I had some guidance at the start as to what architecturally is a going be an acceptable solution, this would have cost me much less time. That means I'll have more time and more motivation to make other contributions. This process feels very inefficient to me.

as to adding another interface, the means modifying PlugableAuthenticationService which I think is CMF, not even Plone code.
mvn clean test 하면 테스트가 깨지는 것 같아요.. ㅠ.ㅠ
Purely style. Someone should really go through and edit/audit the source to go one way or the other ;)
That's why I use subtrees :-D
It was a workaround for a compiler bug (see: https://devforums.apple.com/thread/122059?tstart=15) with armv6 architecture.
Robert C. Martin - Clean Code
when using self-explaining names comments are not necessary (DCF_W -> WEIGHT_DCF77_SOURCE)
I would definitely turn this into a decorator. @private(commit.parent_set), etc.
We could just use text to write the maps but I think the engine will support Z axis and tile overlapping. So the level editor will help with visualizing the map and save us alot of time if we are going to have alot of maps.
Well, tell bio.
Can we `break` here?
haha sarcasm :P what HTML templates do you like to work off? is the html5 boilerplate good for you?
Using a String instead of Pathname seems simpler to me, would it break anything for you?
There is also a bug when closing the app. if you are drawing and ofImage you get a nasty crash trying to release ofImage
It took me a second to figure out what was going on here.  I think it would be clearer to initialize it first and then loop over one less value of precision, like so:

eff.prob = Probability.new{val = prob.hit}
for i = 2, precision do
    eff.prob = eff.prob / Probability.new{val = prob_hit}
end

If precision is 1 the for loop never executes (I just checked this).
Agreed.  You should submit your requests as pull requests if you haven't already.
One suggestion: `Array#include?` is O(N), whereas `Set#include?` is O(1).  If I need a collection to check membership, I tend to favor `Set` over `Array`.

That said, it's only 7 items in the collection, so I wouldn't expect it to make a big difference.
So why limit to 3.1 ?
Yes, there is an empty line, but there's no whitespace at all. ;)
Flash 3.0 is still not compatible with Arduino 1.0 IDE. I had to replace this line in Flash.h 

#include <WProgram.h>

with

#if ARDUINO < 100
#include <WProgram.h>
#else
#include <Arduino.h>
#endif
Im have one problem with CentosOS 5. Must disable  EasternKingdoms/BlackwingDescent/boss_maloriak.cpp  have compile error
Checked and agreed
Does this mean you're getting ready to publish it to the marketplace? If so you might want to check out [this comment](https://github.com/milaq/mumble-android/commit/e7dcbc10dd27c29cff7166a2fcd42dc963042cb3#commitcomment-788514) first.
Awesome work, but I don't see where `view_count` is actually returned, unless we're returning a cached value. Then again it's early (here) so I might just have missed it.
very nice! i don't fully grok it yet, but i can see it has more structure and plan than the code before :)

>> +Object.prototype.AM_OBJECT = true; //HAHAH IM A DUMBASS. srsly, dont know better way.

what do you need that for? to check whether an instance has `engine.Object` in the prototype chain?

i was a bit shocked to see you overwrite the standard JS `Object`, i would have called it `EngineObject` or something but i don't fully understand what you do, yet :)
You can replace this series of puts with a loop over the contents of the code set from config.  (See comments above.)
Lo he estado testeando esta mañana. Invocaba un porrón de nefarians pero le corregí eso. Lo demás no vi nada extraño, apenas cambia, salvo añadir script a los adds que estaban spameando un teleport que te mandaba al vacio y hacía imposible poder matar al boss.
https://github.com/openstream/SaferpayBusiness/blob/master/app/code/community/Saferpay/Business/Model/Cc.php#L100
All gluster related VDS commands should continue to extend from *GlusterBrokerCommand*, which should extend from *VdsBrokerCommand* instead of *IrsBrokerCommand*
What is the purpose of having two methods - one simply calling the other?
this does not override cmsettings values :)
it only sets the default value for a "new" user who hasnt changed the scrolling cache settings in cmsettings.

with "default disabled" i get WAY better scrolling performance everywhere (incl. market, etc).

problem with the "disable scrolling cache on long flings" commit you mentioned is, that we have a rather poor scrolling perfomance even on short flings. "default disabled" works around that issue.

just try to add a widget to the homescreen - in the widget chooser list the performance difference between scrolling cache enabled/disabled is rather obvious.

cheers
how much more RAM did you get by that step ?

thx
Извиняюсь. Не до конца понял смысл кода.
for P1000L
Although you are right, this may prevent from working some existing plugins.
(Plugins that use pog_attribute_type for adding new functionalities).
I guess Joel will agree with you, and your contribution will be added as soon as possible.
Have a nice day.
I've pushed up a new commit merging your last version of master branch and applying your suggestions.
Out of interest, why did you break out the logging package? It is not only for _LoggerFactory_, right? The generated classes will go into this package as well, right?
Would you be able to amend the commit to only contain relevant changes, please? Would make the history and diffs a lot easier to read.
Actually, you can and it will set the `element.value` property by default which means you could remove the `else` block above as well. :)
@croby shouldn't we remove this logger line?
Oh, this was just a minor cleanup commit. Wrong commit message. :-)
Dear Mr. tprovot
Could you tell me what repo & build command is for ICS and what repo & build command line is for ics_mb525?
What is their differences on them (kernel or lens)?
Best regards,
Jeff
s/ quoted 'space' separated list/n array/
Yeah, sorry, I meant 3 replicates. As we are also replacing 'at least one efv with min X replicates' to 'each efv with min X replicates', I don't want to end up being over-restrictive. Let's re-calculate stats for all experiments with min 3 replicates per efv and see how much difference that makes.
os.getuid() and other calls used later in the function do not work on windows.
Use false instead of "false" for the default value of boolean type
I only exposed that for advanced configuration cases.  For example I didn't code support for the OpenID stuff, but someone might want to use it with these components, in which case you'd need to set this manually and point to custom serializers for the OpenID classes.  It's safe to set it to null since it's ignored in that case.
Yes it is, I added it to the top of WMU.js (where all the globals are declared) I need it in couple of functions across the code to do some stuff only when lunching WMU on special page.
It is ok. Our bread crumbs start with <.
good one :-)
I'm not sure if the `actorCellShouldStart` call is nicer since it's only overriden in TestActorRef, or if we should simply do `underlying.start` in `TestActorRef`. I'm afraid that the `underlying.start` call will be forgotten at some point during a rewrite, while this is more explicit.
Correction: 2.2: https://github.com/zeromq/jzmq/pull/109/files
The variant Viktor proposed will actually run the `refresh()`, while I was referring to making the sending of `RefreshCache` messages recurring. While those messages could potentially pile up, running the refresh job should be fine (if it is done synchronously). I still think that the `scheduleOnce` approach is the easiest to grasp (and also it does not leave state lingering around which needs to be cleaned up).
Right. Will fix.
val echo = system.actorOf(Props(ctx => { case x => ctx.sender ! x })) // ;-)
``implicit val timeout`` instead of param
It's probably easiest to close this PR and open a new for the squash (not a huge fan of push -f :-) )
If removed, all of the metrics multi-jvm tests fail because clusterView.clusterMetrics.size is always 0. So if we don't want it enabled yet then another test strategy is needed :(
Isn't it weird that the user cannot control which dispatcher is used? And that the same dispatcher is used for all AMQP consumers?
to make it idempotent. the registrar still has to get the register message


On Sat, Nov 24, 2012 at 7:27 PM, Viktor Klang (√)
<notifications@github.com>wrote:

> In akka-camel/src/main/scala/akka/camel/internal/CamelSupervisor.scala:
>
> >          parent ! AddWatch(producer)
> >        }
> > +      producerRegistrar forward msg
>
> Why was this moved out here?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/893/files#r2215496>.
>
>



-- 
Raymond Roestenburg

code: http://github.com/RayRoestenburg
blog: http://roestenburg.agilesquad.com
twtr: @RayRoestenburg
There is, in `after`, L42
I thought about it - but will it really safe memory when asOutputStream is not used? I don't know how heavy a basic OutputStream instance is. On the other hand, if it's really lightweight, might a def even better than a val? Experts?
Personally, I am always unhappy when software tells me something like that... Can't we just print precisely whether pod2man is missing or not executable? The checks to distinguish "missing" from "not executable" are not complicated, after all :).
Its my IDE, looks fine in Netbeans, I will sort it now
why reintroduce then?
In IE8
Isn't this the job of Spnl to throttle the feeder if nothing was found ?? We kind of duplicate what SPNL is supposed to do here
Пока лучше убрать. Завтра обсудим, все остальное и решим.
also... I don't get what you mean... the try { } catch { } is/was formated the same way in HookDispatcher
I thought I needed to do all of this last part in more places, but even if not, I think the flow gets a bit better with this.
getter and setter functions are super-non-pythonic (also: slow). Just use a public instance variable.
since CSS classes are loaded for all the samples, can you change this to use a unique class name for this sample?
This property has to be declared before the main `@implementation` block in order for it to be synthesized.
This should be `HUGE_VALF`, per the [CAMediaTiming documentation](https://developer.apple.com/library/mac/documentation/GraphicsImaging/Reference/CAMediaTiming_protocol/Introduction/Introduction.html#//apple_ref/doc/uid/TP40004511-CH1-SW5).
or `-INT_MAX-1`
In this case, that's probably the better option, as it allows the cheap removal of settings (without switching the object to dictionary mode by using `delete`). Although you might want to use `typeof options.fd != "undefined"`.
, the `.connections` property is set to null (?)
Why is this called `cluster` and not `Cluster`? Odd...
Just a minor thing here: Instead of blocking the thread, you could actually return when the Future from RequestMasterState returns. I think you'd need to do this:

    val future = master ? RequestMasterState
    future.onComplete(state => completeWith { masterui.html.index.render(state) })
Same for this file.
Noted. I will make adjustments hopefully over the weekend.
What happens here if ``git`` is not installed? I think it should just not print the ``__version__`` string (e.g., ``1.5.0-dev``). You can check whether it is available with ``cogent.util.misc.app_path()`` Also, you should make systems calls with ``qiime.util.qiime_system_call()``. 

For example:

```
In [1]: from cogent.util.misc import app_path

In [2]: app_path('git')
Out[2]: '/usr/bin/git'

In [3]: app_path('not-a-real-program')
Out[3]: False
```
и вообще к методам бы побольше доков
@rafaelfranca I agree the inconsistency is confusing, especially if they don't know they're aliases.

My thought process was to only use `belongs_to` in the migrations if `belongs_to` was used in the model otherwise use `references`. What do you think?
It should be off. It is unnecessary.
I remember some discussion about this one but I didn't read yet. I'll try to read now.
cut trailing space :)
The method name is too generic. I'd rename it to something like `validate_order_args`. [I don't think the bang is necessary](http://dablog.rubypal.com/2007/8/15/bang-methods-or-danger-will-rubyist).
Here is a `:nodoc:` instead of deprecation.
Could we maybe do it the other way around? Make keepDocComments true when forScaladoc is set? I think the entire thing needs a bit of cleanup: we have `keepDocComments` and `forScaladoc` in `Global` and `isScaladoc` in `ScalaSettings`. A bit too many ways to achieve the same thing for my tastes...
https://groups.google.com/forum/?fromgroups=#!topic/scala-internals/qwa_XOw_7Ks
In case the indentation level is not desired, just move the if up to the definition line, problem solved.
"getOrElse(None)" is flatten, which points you toward the fact that if you flatMap on line 355 instead of mapping, you don't have to do either.
Hi Josh, sorry, that's Sublime automatically killing trailing whitespaces. But I think it actually removed whitespace from that line, as you can see it both removed and then added.
Thanks, but that seems to be another problem. Here we are dealing with internal-reflect-Select:

    [scala.reflect.internal] case class Select(qualifier: Tree, name: Name) extends RefTree with SelectApi

and

    trait RefTree extends SymTree with NameTree with RefTreeApi { ... }

and

    abstract class SymTree extends Tree with SymTreeContextApi {
      override def hasSymbolField = true
      override var symbol: Symbol = NoSymbol
    }
TeX is indeed supposed to only be used in the Notes section.
> `len(li)` maybe?
>
yes
Well it shouldn't be necessary to do the substiution (I can tell this confidently because I added in the `logx` parameter :D). What logx is passed as depends on what gruntz does. An expression `foo.nseries(5, logx=y)` yielding `log(log(1/x))` instead of `log(-log(y))` would be a bug. But I'd say let's concentrate on fixing the problem at hand.
this should probably be "x in coeff.free_symbols"
Anyway, it's removed now and a note like the comment above is added in the commit message.
The original location for the _eval_is_commutative (in functions) has
been restored.
There should be a docstring with one or two doctests showing how to use probs().
Also, in this one instance, I think L^S,etc was called for but would L1, L2,etc be preferred? I was unsure about that.
Better yet,

```
is_real = self.is_real
if is_real is False:
    return False
r, i = self.evalf(2).as_real_imag()
if i:
    if i._prec != 1:
        return False
    return None
if r._prec != 1:
    return r > 0
```

This way we don't discriminate against things that real can't answer on, e.g. `sqrt((3+I)*(3-I))`.
Maybe

```R
widths <- vapply(ps, function(i) diff(panel$range[[i]]$x.range), numeric(1))
panel_widths <- unit(widths, "null")
```
Good point. I think I'll make this a named scope on CourseMembership to encapsulate *how* a person is related to a membership inside that class. Ok?
why comment out this line?
please check the size of your array before accessing index
It's nice out today.
Could you add a comment to that effect lest others be as confused as I was?
nested() is deprecated as of 2.7: http://docs.python.org/library/contextlib.html, and it's unclear from the docs whether it supports actually getting the return value from any of the context managers to do stuff to it.
What about removing the `stream` method from Application? It does not add much value; the `stream` method from the Twig trait does add much more value.
Missing space after use
Because I'm ending poll loop once semaphore was fired. And if it wasn't - I'm continuing polling
See section 1 in style guid: https://sites.google.com/site/rndbbnorth/cheat-sheet/code-styling?pli=1
Pls addthe file author and short run description
Here should be a "HACK: XMPP specific values" comment
Replace `"` with `'`.
I probably ought to have a pre-commit hook checking for these things.
um yaa windows path does the same thing, what timeout should be created for waiting?
What does this have to do with the container?
Fixed in 299079b
Probably we should discuss indentation style here.
Still a bit confused. The sortedPaths var is an object, which has no inherent sort behavior. Object.keys likewise doesn't provide any guarantee of order, although it is an array. I'm willing to take this offline tho, just so I can more clearly understand the data structures and how they're operating here.
This is an app archetype, which contains a canned "top_frame" mojit, so I think what Isao has is correct.
+ 'json'
Вообще у массива есть нормальный метод push(). Да и нужно будет подумать как массив создавать запрос покрасивее
- Swap _object_ with _var_
Okay ... the changed turned out to be so easy to make that I just did it. :-) I've pushed an updated version of the patch.
return on Ajax calls is unreliable and should be avoided.
The method  will not return anything until the Ajax response is received and can't be tested without a wait
Should the use of "pp" (e.g. in "_pp_payload", "pp_field", and the like) be removed? Is this just a hangover from copying the pp module?
I don't think this is true "they fail to ensure easy access to the elements of the underlying matrix". What they are bad is changing the sparsity structure of the matrix. I.e. adding an element to in the middle of the matrix.
I'm wondering if it would be better to compute G and g as theano graph instead of here. This would let Theano optimize those graph. In particular, the dot of two round maybe have the possibility of being optimized outside of the loop to make bigger call to BLAS.

This is mostly an optimization, did you tried this? Do you need more speed in your case?

This do not block the merge if this PR.
Yes deprecation warning please for 0.6.

Also, you can use the ordered dict class from gof/python25
break; here?
Don't you think This JobsReviewView should be called ExploringJobView or ReviewJobView ?
You can name the method - searchForJobsByLocation
What about renaming this test method to - applyForJobWithSelectedCV
This is mostly a style issue and not a pull-stopper.  (are there ways to make small tweaks/hot-fixes to a pull request in-situ?)
The quotes around 10 are slightly misleading, if you pass the string "10", it will fail. So value should really be 10 (integer) in this case. Maybe simply remove the quotes?
:runner: 
:runner:
:runner: 
:runner:
:runner: 
:runner:
:runner: 
:runner:
:runner: 
:runner:
You have to import `app` too.
Not long enough to bother with the let.

``` clojure
(clean-empty (merge-with difference mmap tombs))
```
What's the benefit of this change? I think we should use unquoted identifiers.
The variable name doesn't make sense to me.  Isn't it simply "scaleFactor" or something?
like, these change lines being present when there wasn't any change. Breaks `git blame` :/
Do we want a util function for this quote + encode type operation to make sure we keep it in sync everywhere?
What about cases with unknown json fields, or fields set to the same name as functions on the class, or __foo__ methods?
The example_base.html had:

```
  <body data-mobile-cookie="{{ settings.MOBILE_COOKIE }}">
```

Do we want that? I'm not sure what it does.
I think this isn't needed.
Can we combine these loops? Iterating over the list twice seems silly, especially constructing new jQuery objects twice.
What is the source of this patch? Has it been submitted upstream?
shouldn't we just selectFile:inFileViewerRootedAtPath: 

e.g.... selectFile:[workspace valueForKey:@"representingFilePath"] inFileViewerRootedAtPath:nil];
The name of the task follows ```{{ _(' made a new task for: ') }}```, so I'm not completely sure if this is correct?
Alright, and what do with the Java-API (i.e. don't have to explicitly provide ec (when you don't have one))?
Did a ticket get created for this?
Why is this static? Static is not good, not good at all. What happens with `ant`, when the compiler is executed multiple times in the same process?
ALWAYS LinkedHashSet ALWAYS

HashSet goes in the same pile as Vector and Hashtable as collections that shall never be used.
The compiler on OS X gcc version 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.11.00) Generates the following error without it:

    taglib/taglib/mpeg/id3v2/id3v2tag.cpp:77:34: error: default initialization of an object of const type 'const TagLib::ID3v2::Latin1StringHandler' requires a
      user-provided default constructor
    static const Latin1StringHandler defaultStringHandler;
                                 ^
    taglib/taglib/mpeg/id3v2/id3v2tag.cpp:84:22: error: definition of implicitly declared default    constructor
    Latin1StringHandler::Latin1StringHandler()
protection against my.cache[key]?
这几天忙学校的Project了没有顾得上改... Mac确实可以使用Aqua图形设备，但是我画图时会启动X11/XQuartz。此外，自己编译时，2.14+版本带aqua参数我就没有成功编译过，必须强行取消with-aqua不知道是问啥
fwiw, it's inefficient to return a default of now, if a date isn't known or available then there shouldn't be an at returned (return an undefined or null).

The hash of the json is compared to see if it changed before it's stored, so this means that it would /always/ be detected as a change and always rewritten, and before it's actually saved an at of 'now' is applied (after the hash check).

There could be other places where this is happening too, maybe even ones I did by mistake :)
These lines add events stored in the node to the browser menu. These events are generated in the debugger-extension.

This has no effect if no events are set (the debugger extension is not loaded).
In the fall through case, propName is blank. Is that OK?
typos ("an register" -> "and register", "no loner" -> "no longer")
The callers of this check truthiness, so trying to go to line 1 (return value 0) will be ignored.  Callers could explicitly check for undefined (or isNaN) instead.
typo: fucntionality
* English names please.
* `freq` is misleading.  I'd prefer `dt`
Same comment as for the BatesProcess
Actually, it will construct fine without one. However, it will log console warnings if you attempt to load records via the proxy and have not yet provided a url.
I think a dummy URL would actually be more confusing.
Not an URL but a String that looks like an URL.
If I want to raise an error on exception in views.py - what is the way to do it?
I am doing it like this `return debug.technical_500_response(request, *sys.exc_info())`.
No I just thought if someone wanted to make a visible element from hidden, he could change the class. But it is much easier to just change the atribute. I will delete.
It was not a bad idea to have label and field both in its `li`.