labofsahil/github-event-dataset-2012 huggingface dataset:
splitting out remote node logs

i found it easier to troubleshoot nodes by splitting out their logs into separate files based on their hostname, as opposed to digging through /var/log/messages.  this commit creates the dir /var/log/nodes/ and configures rsyslog to send each nodes logs to their own file in /var/log/nodes/%HOSTNAME%.log.
JENKINS-12744 - Execute a command before timeout

Allow execution of a Shell or Batch command just before the effective thread interruption.
Korean translation : Menu unlock(strings)

update for removed menu unlock
Modify to use osfamily instead.
INT-2418 Pool Size and NIO

https://jira.springsource.org/browse/INT-2418

Add documentation about setting the pool-size to a minimum of
3 when using-nio="true".
Add pin_on and pin_off

So now its possible to enable special pin.
Also, much easier to scan using bash rather "echo 1 > fire".

Have used this way scanning & checking is new USB device visible:
   for a in `seq 160 176`; do echo $a > /sys/gpiotoggling/pin_on; sleep 1; echo -n $a - ; lsusb | wc -l; done
Fixes issue #1

Issue #1
Fixed create_from_issue route for GET link

Hey, thanks for pulling in the last patch. I didn't realize the `create_from_issue` route was the endpoint of a normal link, i.e. a `GET` request. This fixes that.
Updated DSConverter.py with better checks for PDF

We used to check the return value of PDF2SWF and assume that if it
returned 0, then the conversion was successful. This is not always
the case. If the PDF has security turned on PDF2SWF will output:
FATAL   PDF disallows copying and return 0. This was causing odd
error messages in the logs.

Added a check to make sure the file exists. We also now log whatever
PDF2SWF outputs to STDOUT if the file doesn't exist so that we have a
reasonable log message.
Update README.md

La till beskrivnig för Sublime Text 2 med indentering
adding ernstchan.com
Release of the WebWorks Ant Build Script to Open Source

Ant Build script to simplify application development with Lint checking, code optimization, building for multiple SDKs, easy integration with IDEs, and deployment to device or sim.
Use a working ACL

It took me a really long time to realize that there's a bug in the existing implementation:

A post is shared, and is viewable in your profile, and shows up in your main stream, but **it's not pushed to any of your follower's streams**. This commit fixes that.
Added Vector3 and Vector4
Aw1.1

scripting - first version
Modeler bugfixes
Pull Req: UI Elements - Build scripts and resources

This patch adds the build scripts from the UI elements branch to the next branch. Additionally, the source for the 3rd WebView and HTML resources are included.
HTML Dialog implementation with blocking 

This change addresses several issues that are in need of a Dialog. It traps the DialogRequested event and ensures that execution does not continue until the dialog is either accepted or canceled. It will fix issues #165, #163 and unblocks issue #146.
Enhances stability for teamwork situations on same network

Uses LoggerSetupBonjour() to connect to a specific NSLogger
host with the given Bonjour name. Useful in networking
situations where multiple NSLogger hosts are running.
Teddy knox

you should pull my changes please.
Inclusão do script de criação do db
Develop

Using single dispatchServlet.
remove-lowercasing-32

Removed all the optional lowercase versions of selected text, link text, etc. Removed option. Fixed small problem with Twitter putting U+200F into their links. Updated README.

Fixes #32
Fix numeric input arguments

getLog() throws exceptions since the argument check has been prefixing it with another "-"
Attempted to use on barebones install, needed some stuff installed.

Greetings!

I installed a fresh ubuntu vm, I installed perlbrew 5.16.0, and I attempted to use your project.  I had to install the following modules:

Template
Data::Dump

So I added them to Makefile.PL !

Cheers!
Feature singleton controller
Master config xml

Changed version number and fixed minor xml invalid issues.
Update source/members/member_profile.txt


Added section to explain increasing minimum password size for password reset form
Fixed: spelling mistakes on icon files.

Fixed: spelling mistakes on icon files.

Create CSS File: jquery.mobile.customicons.css with the css needed to use these icons inside of jQuery Mobile. For Example <a href="#now-playing" data-theme="a"
data-icon="disk" data-iconpos="top">.

Changed: JSON to use quotes eg. pause is now "pause".

Fixed: jb.series to jb.latest to save future confusion. Ammended the html
to match.
Fixed: jb.latest.get so that it uses the same look and feel as the other
list items in the shows section.

Changed: the footer tab items so that they now have icons.
Changed: list view items icons to thumbnails.
fixed source location
Update patch-4

Added ":" to internationalization functions as in some languages (for example in french) a non breakable space is needed in front of ":" ";" "?" "!".
Corrected some words and filled missing text. ;)

I'm going to currect more files if I will find. :)
Fixed Modules Permissions Controls Dragging
added test for finding Position
Docs

Initial work on issue #3
Update site/sr-YU.com_hwdmediashare.ini

Done serbian latin
Gemspec

Saw your post on the Dev Google group and replied but not sure if the message got through. I made this gemspec branch a little while ago and it works fine on rc4. I did this quick, got sidetracked on other work and never got around to writing any tests for the changes I made, but hopefully this will speed things up getting page_factory ready for Radiant v1.0
Database.java erweitert um 6 weitere Methoden/ServerSocketThread dementsprechend geändert
switched from PointXYZRGBNormal to just PointNormal (compiles, but no co...

...lor for now)
List master fixes into feat-twitter-theme
Adding Raizlabs to the list

Thanks.
Update form and list

This commits fix embed/referenced document display in lists and forms.
Backbone migration

do not accept this pull request
Autoload priorities sounds like a good idea.
Thanks for the comments. Will apply those and resubmit.
Thanks buddy :-)
With the `ajax_change` event which you proposed? Or was it not you?
Indeed, we shouldn't hide bugs. We should fail more gracefully and more descriptively, though.
I am not sure I understand you. Do you have some sample code I can see?



On 20/01/2012, at 1:57, cmar<reply@reply.github.com> wrote:

> What do you think about altering Promotion#activate rather then registering for the notification? Since activators are already registered in core, it seems like it might be redundant.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree/commit/c534b5c7ecf85d09e1c065b44feb92cc1fedbaea#commitcomment-882310
You're right. It's a read-only attribute, though. Not sure how to achieve this, I don't see a way to set the system default encoding.
This is addressed in master
The more I think about it, the more I think toolTipsWordWrapped is dangerous and not scalable. It has a side effect of "trapping" the tooltips.
and the following is odd:
setToolTipsWrapped(true)
// here tooltips are trapped (installEventFilter), but toolTipsTrapped() return false 
setToolTipsWrapped(true) should internally call setToolTipsTrapped(true) then.
Code is duplicated, the installEventFitler and removeEventFitler are in 2 locations.
but the following would be counter intuitive:
setToolTipsTrapped(false)
setToolTipsWrapped(true)
-> now toolTipsTrapped() returns true, having the order of calls Trapped/Wrapped being important is a risk.

Also what happens when we'll add more features to the tooltiptrapper ? each option would have to install the event filter ? does not seem scalable.
So I would prevent toolTipsWordWrapped from installing the event filter, and let that work to toolTipsTrapped.

Jc, I wouldn't split the class in 2 . I believe the "wrap" feature is generic enough to be part of the "base" tooltip class. 

Danielle, if you don't have time to work on the cleanup I can do it, let me know.
This was tested with MinGW
@thibaudgg NICE. You are the man! Also, *adorable* working conditions!

Not sure if I ever announced this publicly, but I hope to be in the same scenario soon. :smoking:
Lindsay's tentative due date is July 3rd. Or the 4th. Or the 8th. Depending on who you talk to. Not confusing AT ALL (not like the numbers would be exact anyways).
Ochko

When do you need to deploy this?
Thanks for the info, will refactor tomorrow morning and send another pull request.

On Jan 22, 2012, at 5:58 PM, Trott<reply@reply.github.com> wrote:

> I'd recommend the fix that is in https://github.com/Trott/mwf/commit/510c83957eea8aa3c5cb5b6047ad7d4e16fc15a4 which uses the `Config` object rather than the constants.  Those constants are defined by the Config object for internal use.  They may vanish, get renamed, etc. in future versions.  (Of course, since that fact is not documented, you had no way to know that.)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/widukind/mwf/commit/fecf5896d1dea03b1f9b418c777b19e62d61879a#commitcomment-890980
Yep, exactly. BTW, you should edit your comment, it doesn't show-up well, remember, github use markdown even here. :-)
And this brings us are back to the question: Do we really care about python 2.5!?

P.S.: If the answer is "CentOS use it":
Why the hell these idiots are serving OS with unsupported software?
Could also write this as `@capabilities || raise UnkownResponse, "Cannot read rets server capabilities"` (but I'm unsure which one I prefer :P).
Binding actions to url should not be done for security reasons.
With this change any unrelated site will be able to post a link to #/delete-course/<...>, and you'll delete a course if you'll click it while logged in.
Only bind actions to something you can't be redirected to.
One option is to use forms with submit buttons.
Another is to bind onclick handler to relevant anchors.
Merged by r34198. Thanks.
name = version tag in modinfo.lua
not bad for a start, but we see that it will require some work...
i was thinking about what to do with the positions of the labels...
i think we can safely put the labels next to the nodes, but we should try to put as much labels as we can higher (ideally to the middle of the edge) the labels could be either next to the edge (as now) or even interrupting the edge (like this: o----- label -----o ); you can calculate the angle between a node and its sibling to see whether there is enough space
another comment: the nodes where the words end should be encircled because you can have two words where one is a prefix of the other, e.g., "bar" and "barman"... then not all the words end at leaves... (another possibility is to have an end-of-word symbol '$' which you add automatically at the end of each word... then at "bar" there would be two different branches: "$" and "man$"... this will be used in the suffix tree so i guess it could be introduced already with tries... it's your choice)
Since the client code should be prepared for 0 being used here now, can't we just return 0 here?
Confirmed too.
Thanks for unit tests, really missed this.
This is for SQL Server 2008 to address the timestamp conversion issue.  Not applicable to MySQL.
oh weird! I didn't think you could make old style classes in python 2.6 :/
I get

```
checking for sgemm_ in -lblas... (cached) no
configure: error: Cannot find BLAS libraries
make[1]: *** [arpack-ng-3.0.2/config.status] Error 1
```
@richarddong 嗯，可以多个，凡是带@的都会送。
I guess the only significant change in this commit is that is PAGER is not defined or wrongly defined, then the usual suspects ('/usr/local/bin/less',
'/usr/bin/less', '/usr/bin/more') are attempted.
Написал им ивент http://ru.wowhead.com/npc=30270 не кастуют они, когда прислужники.
this shouldn't have been here anyway
However, the name "utcdatetime" isn't great either. Before this last edit, I made it always return a UTC date but naive. Maybe we just drop it.
throwing an exception would also be a BC break
please change s, ss, sss to something more meaningful 
usually `for(int i = 0; i<100; i++);` is standard enough to understand what i is; 
but `s`, `ss`, `sss` have no obvious meaning so the code will be hard to maintain. 

i suggest `target` instead of `s`
probably `path` instead of `ss`
and `files` instead of `sss`
It looks like you could use some work still on refactoring this to use other storage systems. For instance, I foresee that you are going to use loadSkillSettings in any system that you create. You may think of refining your interface (abstract class in this case) to micro manage a little. That will also document the code a little better into what you are expecting programmers to fork out when retrieving or sending to storage.

Just thoughts. I hope they help.
C'était quoi ça? XD
It's not strictly necessary (since this function defaults to `ia32` by default) but on my Ubuntu server at home I am getting:

``` bash
nrajlich@stan:~$ uname -m
i686
nrajlich@stan:~$ uname -p
unknown
```
btw, look at ticket #1203 where the user is bound to IE7 (corporate rules). He's probably not the only one. I also encounter workplaces with FF3.6.x 

http://www.assembla.com/spaces/tracks-tickets/tickets/1203
do we need an else here if it is returning already?
It works better this way on aquamacs: I am trying to find the best way to
go about things and csound-x in aquamacs separates the csd files into orc
and sco (orc and sco being the older way csound ran)

I have rolled back commits 3 times already trying to debug what will work
best, my next step is to try to use instrument names versus numbers so the
score has a bit more coherence than 1, main, 2, main, main etc

Also testing global variables for the next instrument.

All of this was supposed to be in the explanation of the commit so if you
no how to edit that after the commit it would be very helpful for me!

On Sat, Jan 7, 2012 at 8:40 PM, Tom Swirly <
reply@reply.github.com
> wrote:

> Coolness!  Does it still work... ?
>
> Let's talk tomorrow-ish about next plans, I don't want you to go off and do
> things at random...
>
> On Sat, Jan 7, 2012 at 8:38 PM, Austin Stone <
> reply@reply.github.com
> > wrote:
>
> > All csd files have been respectively name and put under .orc and .sco
> > extensions
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/rec/the-collapse/commit/44e9f41c4914d23d7dd197b0b8686b81f6763d68#commitcomment-846976
> >
>
>
>
> --
>     /t
>
> http://radio.swirly.com - art music radio 24/7 366/1000
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rec/the-collapse/commit/44e9f41c4914d23d7dd197b0b8686b81f6763d68#commitcomment-846978
>
Thanks for working on this

 1. We put a space between foreach and ( according to our coding standards
 2. Shouldn't the relationship be added before the $e entity is disabled just in case a plugin wants to do something with the entity before it is disabled?
with jQuery you can chain methods like this:

    $contributor_error.html('').removeClass('error');
да, исправил вернув удаленный cin.get()
Yeah, I am not certain, you could just have a catch all that says 
"ensure psql is in your path" type thing?

On 2012-03-08 11:59, sgilda wrote:
> Good point.
> I'm pretty sure the Linux install puts the link to the executable in
> /user/bin directory. I know I didn't have to modify my path.
> I'm not sure about the Windows install, but usually Windows installs add
> everything to the path. :-)
>
> I'll see what I can find and modify the docs accordingly though.
>
> Thanks,
> Sande
>
> On 03/08/2012 02:13 AM, tomjenkinson wrote:
>> Hi Sande, looks good to me, my only question (I haven't retested this on windows so I don't know for sure but)
>>
>> Create a database user
>>
>>       As the postgres user, start the PostgreSQL interactive terminal by typing the following command:
>>           psql -U postgres
>>
>> This is good *assuming* psql is in the path by default for a windows user? Are we sure it is in the path on Windows when it is installed? Might we worth pointing that out?
>>
>> Great work though, it looks really slick :)
>>
>> Tom
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/sgilda/quickstart/commit/a94ad1b2044f14b921200f02204fa57c9601f211#commitcomment-1057235
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sgilda/quickstart/commit/a94ad1b2044f14b921200f02204fa57c9601f211#commitcomment-1058092
I cherry-picked it.

On Thu, Jan 26, 2012 at 4:36 PM, Richard Hartmann
<reply@reply.github.com>
wrote:
> Why does this commit have a different SHA1 than https://github.com/RichiH/roundup/commit/4f842efb041e09bd7dd2a773fb030d6910609563 ?
>
> According to git show, they are exactly the same... Oh well, I will end up basing my master on yours so I can track what you decide to merge, anyway.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bmizerany/roundup/commit/33f589351746efddebde4446b6c1f7b527e5261a#commitcomment-906482
Every time User.objects.create is used, the id will be bumped up, so local_id won't always be 4.

Safer to use local_id=str(self.user_company.id)
不好意思。不喜欢爽引号。还是自己改了。 :em04
Thanks. I saw that happen once but hadn't looked into it yet.
I think you added this line at the wrong position, it should instead be just above the "foreach" at line 306
On Wed, Mar 7, 2012 at 10:30 PM, Gael Varoquaux
<reply@reply.github.com>
wrote:
> Where? I think that is it fine in the setup.py

Yes, sorry, I did not check, it looks like Yarik fixed that back in July.
Почему бы не сделать Ui_.Splitter_->setSizes ( QList<int>()  << size ().width () << playlistWidth ); ?
En testdata dan?

Op 6 januari 2012 16:55 schreef Hidde Jansen <
reply@reply.github.com
> het volgende:

> Zelfs in Nederland werk ik veel offline, dus lijkt dat me geen goed plan.
> Kan je beter het bestand locken zodat niemand je eigen settings
> overschrijft.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Nathriel/We-R-Asia/commit/64e8488f96bf99a5f1f189f5925dbbfb813bfa8e#commitcomment-844021
>
Oh, and NICE avitar! :)
"@param {jQuery<HTMLElement>} content" would be better, as it annotates more clearly what the expected type of the argument is.
That's what I thought, but @smeranda said https://github.com/kabel/wdntemplates/commit/2d617733daf59deac4dd3404851b4a0802f4eb57#commitcomment-922979
@sajjad-ser that looks cool.

One thing - from the code above you seem to be using spaces for your indentation instead of tabs.

Could you make sure to use tabs (4 spaces wide) instead of spaces please?

Thanks
Sean
Aco ich glaub das war zu viel "gefixt" :-)
Also SPELL_LEECHING_SWARM_HEAL in spell_generic.cpp does not work now - not heal caster - boss now ...
@joshk, I think that's a matter of taste, but I can change it if you want. I'm so used to unless-blank? that if-present? makes me stop and think for a while =)
Right but I have a column image_name so I thought it'd work but I
think I'm wrong.
It's probably worth stating when this changed. `RefinerySetting.find_or_set` appears to still be correct for the most recent gem release `1.0.9'`.
Wrong commit!
Crashing in movement system like hell:
http://paste2.org/p/1881010
http://paste2.org/p/1881148
check ur malygos script. the encounter resets on dragon phase event with the prevent function. dont works.
``` ruby
options.symbolize_keys!
```

?
Trying to make specs pass on 1.9.3 and this is the only one that isn't obviously related to the the includes in RSpec (https://github.com/garybernhardt/raptor/issues/17). I get a wrong number of arguments error from "routes requests through the record, presenter, and template"


    it "routes requests through the record, presenter, and template" do
      request = request('GET', '/post/5')
      rendered = FakeResources::Post.routes.call(request)
      rendered.body.join('').strip.should == "It's FIRST POST!"
    end

 &

    Raptor::Router routes requests through the record, presenter, and template
     Failure/Error: rendered = FakeResources::Post.routes.call(request)
     ArgumentError:
       wrong number of arguments(1 for 0)
     # ./spec/fake_resources.rb:48:in `initialize'
     # ./spec/fake_resources.rb:48:in `new'
     # ./spec/fake_resources.rb:48:in `find_by_id'
     # ./lib/raptor/injector.rb:16:in `call'
     # ./lib/raptor/injector.rb:16:in `call'
     # ./lib/raptor/delegation.rb:33:in `delegate'
     # ./lib/raptor/router.rb:193:in `respond_to_request'
     # ./lib/raptor/router.rb:17:in `call'
     # ./spec/route_spec.rb:9:in `block (2 levels) in <top (required)>'
This should really make it into the release notes for 2.12 or not? People currently using OpenLayers.Util.onImageLoadError need to get an alternative way of dealing with image load errors.
there is some syntax differences in SQLITE driver string like:
$dbh = new PDO('sqlite:/tmp/foo.db');
真不好意思，没有研究过Sawfish打开虚拟桌面的功能，我都是一个page的～
d'oh, was missing the :html => true opt. My bad, ignore me! and Thanks!
t'es une  bouse
Hi,

Could you provide some details on the bugs you encountered with ctrlp? If it was just the custom_ignore option then this was probably what you encountered: kien/ctrlp.vim#130. But if there was something else, I'd appreciate it if you could provide some details.

Thanks!
huh, did this get deleted?
Good catch! Would you mind making this a pull request so I can merge it into the master branch?
teeny weeny correction: "has an invalid", not "a invalid" ;-)
X\ nfr
A nice side effect is cpu usage is way down.
OK, so thinking about it, this overflow check is completely wrong. It doesn't matter that a sufficiently large Float cannot represent distinct integers. The fact is that the float represents *an* integer which we can convert it to. The above logic should be applied to Integer.float.
looks excellent to me! kudos
No es por ninguna versión, era porque en ese momento "me gustaba" así. Supuestamente es mejor usar símbolos cuando algo se repite y/o ejecuta mucho. Si hacés algo como

```ruby
100.times { "cadena" }
```

Crea 100 string anónimos, por lo tanto 100 reservas de memoria, 100 * cantidad de caracteres en RAM. Contra:

```ruby
100.times { :simbolo }
```

Que solo hace referencia 100 veces a la misma dirección. El tema con los símbolos es la tabla de símbolos, que crece y crece si abusás de eso.

Podés ver más sobre el tema acá: http://www.robertsosinski.com/2009/01/11/the-difference-between-ruby-symbols-and-strings/

También podés ver que no es para tanto la ganancia en rendimiento, a no ser que hagás un bucle de 10 millones de iteraciones =)

Bueno, yo abusaba, pero ya se me pasó...
Probably no need for a fork, I'd say. "Initial commit" makes sense, then you could see how it evolves. A fork to me means conceptually - 
"The original intent or current philosophy of this project differs from how I would like it to be"
or
"I'd like to take this in a potentially completely different direction and I'm not sure the author would agree with the direction"

Whereas my suggestions are just to improve the readability and code quality, whilst keeping the code fundamentally the same
If the iframe looks and works fine without javascript, how is that not a more graceful degredation than just having a link?
On Thu, Mar 8, 2012 at 1:53 PM, Laurence Rowe
<reply@reply.github.com>
wrote:
> plone.app.cmsui simply sets this in the view (see https://github.com/plone/plone.app.cmsui/blob/master/src/plone/app/cmsui/menu.py#L20). You need some way of signalling that a particular view is part of the UI and should be themed. I think the best way to do this might be to add an interface to the views that are part of the UI to signal that they should not be themed. This can be done from a simple ZCML directive externally to the view itself. An IPubAfterTraversal event can then set the header and changeSkin.

Are you saying that we should do the equivalent of
alsoProvides(someview, IOverlayed) or similar (in zcml, with browser
layer of course), and then in IPubAfterTraversal, if
IOverlayed.providedBy(context) you set the header and change the skin?
I just want to be sure because I like that idea.
Be sure to use backticks when commenting on code. You're maps will print out a lot prettier
[GitHub Flavored Markdown](http://github.github.com/github-flavored-markdown/)

```
XXXXXXXXXXXXXXX
XX###########XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XX#XXXXXXXXX#XX
XXXXXXXXXXXXXXX
```
This commit cause crashes:
http://paste2.org/p/1855093
closes #4440
It's applied only on documentation. It's possible to apply optipng on resources used in programs? And framework? I never see full sources, i don't know how to get all sources to build it.
I agree.  I did have one and then deleted it.  It would be cleaner with one.
Ok cool. Thanks for alerting me to `xvfb-run`, I'll put that as the recommended way in the readme.
Ok.  Do you want it to only do that when there is already at least one node on the graph?  It will be kind of weird but it will match with the pre-existing Graph code.
va bien el SkyFireEMU?
ooh. sounds good. I'll definitely try it.
Kyle: I believe we can do that as the only contributor as far as I remember was me, and I can provide the permission to remove my copyright notice. Let me know if I am wrong.
These ReadOnly locations should also be used for easy adding of links to external data (CEL files from other http services, etc.)  MUCH more robust way than manually setting location as something external!  Again, awesome work sir ...
perhaps it should have read "merge without creating an enabled link" ? Because in this case it has to at least to a merge once to create the pages based on the site template... the link is simply not enabled.
http://www.wowhead.com/spell=20700
That piece of shit was stealing work even before Trinity existed. Glad someone outed him.
About fucking time! :)
Yeah, doesn't work. I will try fix it tommorow.
if(!isNaN(ret) && ret.toString()==str) return ret;

Twitter ids are larger than JS integer limits, so automatic conversion to Number will create inaccuracies.
Where would i load the state back to the game.
It's for Webstorm ide. I'll remove it. I guess I can make that a system-wide ignore. That and .DS_Store.
dangerous, should be wrapped with { }
Thanks for this. I actually just saw this warning myself and updated this in a very similar fashion on my local copy. (Using pmpro- instead of paid-memberships-pro- for the prefix.)

I'm going to apply mine, but wish I could accept this just out of principle. :)
Why is this white space here?
I'm not sure how this change makes access to counter thread-safe - perhaps counter should be a volatile variable, or updates to counter should be synchronized.
I know. I might fix it when I'm bored sometime.
Coding style: we always brace conditional blocks (and cuddle else clauses). Source of bugs. Example:

<pre>if (foo == null) {
  bar.baz();
} else {
  bar.noo();
}</pre>
Test fails for me.

Failed tests:   testVirtualVideoActionInitializationCompatibility(net.pms.test.formats.FormatRecognitionTest): VirtualVideoAction is initialized as compatible with null configuration expected:<true> but was:<false>
Good addition to the harness.  I think it would be useful to mention this in the README so other users can take advantage of this.
target  폴더가 github상에 존재할 필요가 없을 것 같다.
I think some formatting got lost here. What is the difference between the 2 lists?
Makes sense. I'll take care of it. Noticed the build errors on the Jenkins
CI btw?

On 30 January 2012 10:25, Tom Christie <
reply@reply.github.com
> wrote:

> Okay yeah makes sense.
> So long as that builds in the docs fine that sounds good.
> Also re naming, I think it should probably move to CHANGELOG.rst.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/tomchristie/django-rest-framework/commit/f34ed6d1f3d8c1c4c5059beba944aa386e9a15ab#commitcomment-913588
>
The repository entry below is no longer present, and was removed post-0.3.0 b921e77e8ff851b2bb4b82d2048fffe0503cbea2

```
maven {
    url 'http://maven.teamcodeflux.com/external'
}
```
Already made the change.  Thanks.  It was only 1 line.  :)
yes, I should fix that.
Please see line 2225 of adm.js, it have property == "id".
True, but ```ByteStringCompact``` is ```private[akka]```. It was (and still is) done the same way for ```ByteString1```:

```scala
  private[akka] object ByteString1 {
    def apply(bytes: Array[Byte]) = new ByteString1(bytes)
  }

  /**
   * An unfragmented ByteString.
   */
  final class ByteString1 private (private val bytes: Array[Byte], private val startIndex: Int, val length: Int) extends ByteString {
```
Could be a good optimization. One for each address.  Will be the same every second - forever.
yes it is not silently - it has to be explicitly configured by user
Did you mean to leave the print statement?
That's precisely the fix I had in mind too, except I want to make it more generic (thus applicable to more than one platform). So I will not merge this pull request yet, I'll merge it only if the generic approach doesn't work (or if it turns out to be too ugly to implement).
Everything is good so far, the only step that is missing is the conversion of the values from log to the actual values (if a GMPE is used). If an IPE is used than no conversion is needed.
Those are the q_vectors who fill the q_space.
Insert spaces after colons, otherwise these render incorrectly.
Oh, yeah. That line does need it. Though it technically shouldn't reach that point.
m_effectsSlowDown should default to 1
lambda not used, but put this only in the function used.
Your examples should be sparse matrices (probably with different values).
Did you mean to insert this "r"?
change the name of this field, that would help with the ambiguity of some of the related methods bellow prefixed with "append"
what about appendedFields ?

appendedFieldsSize()
isAppendedFieldsNull() 
getAppendedField(i)
isAppendedFieldNull(i)
setAppendedFields(t)
resetAppendedFields()
...
I fixed alignment here.
This is too long for one line. String formatting with % is depracated [1] at Python 3.

[1] http://docs.python.org/3/library/stdtypes.html#printf-style-string-formatting
@fjakobs I need a isColumnFullyVisible method, but I couldn't find it. Horizontal scrolling was already broken. How can we fix?
No, because Class has a default `vars` object, so seeing as you always Class.extend({}) `vars` will always exist.
But it's just one method? I'll put it in the header anyway, but I think it's overkill for some reason.
Wow, how depressingly true. http://jsperf.com/argument-slicing
Ditto, don't need intermediate var
Can we revert these changes that add `QueryDelegation`, to concentrate on the connection ones? This one can be also send in a different pull request if you want. Thanks.
To get the symbol syntax into the string. Most of the time I could simply add `:` to the front, but `:"symbols with spaces"` would fail on that.
[this test](https://github.com/acapilleri/rails/blob/master/actionpack/test/controller/send_file_test.rb#L129) fails.
You might want to utilize "printAfterEachPhase" for this sort of thing in the future.  It makes the checkfile both testy and educational.
I only added def qualifier = EmptyTree to Ident sometime in the last few months, so it would be easy not know RefTree is usable this way.
I could not find evidence of this via `grep` on `src/compiler`.
apllies -> applies  typo
useless, remove all of this documentation. The method name is sufficient
In Python hiding method by "_" is just a convention, so technically there is no problem to call those functions.
Also, when the libraries or frameworks intermediate with each other they are often provide those methods on purpose to be interface.

E.g. sympy provide `_sage_` for sage (to convert objects to the sage properly) , `_latex` for printing and so on,

So clients of sympy (theoretically) can use `_to_mpmath` freely. I don't know about `_to_mpmath` and whom uses it, but as I wrote in 2418


```I searched a sources and:
There are no SymPy classes which have "to_mpi" or "to_mpmath" methods.
There are no code lines  with contains "to_mpi" string (except above Interval).
There are no SymPy classes which have "._to_mpi" methods.

But, there are two classes which have "._to_mpmath" methods - (base core interface), and a few lines which use "._to_mpmath". They are about "nsimplify" procedure.
```
As it is related the core, then it is better have one standard. 
Just in case you can also mark "_to_mpi" with `@deprecated` as and `interval_evalf`.
Most of the discussion happened on the issue page.
No, you need to use two backticks, like \`\`self\`\`. Otherwise it will be interpreted as math by Sphinx.
The benefit of this being special-cased in SparseMatrix is that the ```value != 0``` check happens before storage. 
MatrixBase._handle_creation_inputs creates a full dense array out of the callable function. For large sizes this is not desired. 

However, its not clear that this is the limiting factor. When testing this I found little difference in time between this branch and master; this surprised me. The issue is that sympify is being called on all the zeros. If you move the sympify under the value != 0 condition then you see dramatic speedups. A 1000 by 1000 matrix takes 500ms on my notebook vs. the old time of 8s 

What is in this branch is better than what is in master. I think that a small change to master would make this change undesirable though. 

I think the real solution is to just store and use the callable function - that's another issue though.
We discussed invert/FF related things a while back. See http://tinyurl.com/9nue2dq which might be helpful.
Right, fixing that...
Hmm, that's strange:

```

ruby-1.9.2-p290 > require 'ref'
 => true 
ruby-1.9.2-p290 > ConditionVariable.new
NameError: uninitialized constant Object::ConditionVariable
	from (irb):2
	from /usr/local/rvm/rubies/ruby-1.9.2-p290/bin/irb:17:in `<main>'
ruby-1.9.2-p290 > 
```
dont think this logic will help here. cause that || checks true (or not false) and existsSync would return false so itd be like double checking, pretty sure.
I guess it would make sense to explain which services are registered, I'll try to come up with something better.
@qbit: You can redo your commit (or commit on top and squash), then force-push it and post a "updated!" comment.
Есть у меня подозрение, что эта строчка вообще лишняя.

Тогда я еще просто копировал шаблон плагина из какого-то другого плагина не особо вникая что и как работает.
Use Params, just require the gem and define parameters with:
Param.parameter(name, default_value, description)
Usage:
Param.name

If there are bootstrap problems contact me.
s/updateZoomCache/updateAggregateCache/
Yes. I've pushed a fix for this.
Here you are using generateDigest/2 from ScannableUtils, but below you have your own helper function to do this,  aggregated_versions (which I prefer). Was there a reason why this was done in two different ways?
the `'` in `Network's` here causes `TemplateSyntaxError` at https://developer-local.allizom.org/en-US/search/xml
Oups, thanks for that :)
Think this is just caused by the diff engine assuming ISO-8859-1
Actually I agree with imkingdavid that this should be in the bootstrap due to being used all over the codebase.
There's some extra space between `'ROLE_ADMIN_FORUM'` and the `=>`. Why was that added?
Fixed, just wait for the push
i was thinking the runtime_dir/downloads would be the place for a temporary location. does the getUpgrade function move the binary into the correct location?
Could you add all of this logic to the pkgtype.py library?
There are lots of PRIMEs obviously. What should we name them? I don't like PRIME_INT because someone may want a different prime (for whatever reason) later, and then we are on to PRIME_INT_A, etc...

How about FNV1_PRIME_INT?
I know this is going to be an inconsistent request, but if you're going to overload the parameters, please add a jsdoc here explaining it. Yes, the function should have had a jsdoc anyways, but for whatever reason it doesn't.
Ну и тут повторение кода
I suggest you use more descriptive var name, e.g. **setVideoCaptureNYI**
Use the getProviderUuid function here rather than your controller. Send a callback function as your 2nd parameter, then finally any parameters needed for that function.

(remove the duplicate function from your controller)
don't need htis include since it's in both the source and package recipes as well
The original problem report did some extra work setting up the test case because at the time slicing a cube removed the mask. Now that's not the case, the workaround mask fiddling stuff can be removed, leaving just what @esc24 described:
```python
cube = stock.realistic_4d_w_missing_data()
cube = cube[0, 2, 18::-1]
cube.coord('grid_longitude').circular = True
```
I've given this a bit of thought and I'm not convinced this is a good thing to do.
One way to look at Subs- functions is to consider them library of dumb processing, they ideally only do the operations which other, higher-level, functions know when to use and when not to use. Permissions checks (or filling up $context) should then not belong here. I think it may be a long road for us until we get there (there are about 40 occurences in the current code, of 'allowedTo', in Subs-xxx), but worth trying to keep this in mind.
Basically, the flow is: controller = receives a command, starts to figure out what's in it (params etc), checks permissions, then executes the thing. The 'executes the thing' is many times one or more calls to Subs-, to do the dirty work for us. Then sets up info for the template (i.e. fills $context), and gives up control.
At their turn, Subs- files stand for reusable database work, helper functions, some computation which ideally shouldn't start doing permission checks (those should be _before_), nor set up context (if possible, that should be _after_)... If you got here, you should be allowed to do that thing.

I should say, it's not imo a realistic goal to remove them or something, but the way I personally look at it is that perhaps we need two functions here, one to include mods or additional groups and another doesn't (hah!), or rather, perhaps we can pass the info $include_additional or something, as parameter to the function. The way createPost() is doing this, while a bit overwhelming, it might be worth a look.
I've reworked the conditions in that function and there are no longer orphaned conditions.  It will do a transfer if data is a ndarray (or something that can be converted to one).
Can you add that does op also have a structured grad defined bellow.
Ditto re `repr`
``preargs1`` is not used anymore after these lines, so no need to append to it
According to the Wikipedia, the EXCEPT ALL operator does not remove duplicates.  I know, it's not an official standard by any stretch of imagination, but still reinforce your understanding and not my implementation. Given that, the erase method simply needs to remove a single tuple in either cases (EEXCEPT and EXCEPT ALL) from set or multiset.  

            void erase(const TableTuple& tuple) {
                typename SetType::iterator tupleIt = m_tuples.find(tuple);
                if (tupleIt != m_tuples.end()) {
                    this->m_tuples.erase(tupleIt);
                }
            }

No need to specialize the erase behaviour then.
Revoked check for reserved/ETA tasks happens here:
https://github.com/celery/celery/blob/3.0/celery/worker/job.py#L185-186
get method returns VOID ??
Again, there's a decent amount of internal stuff going on here. Some documentation for how the test is expected to work would be nice for anyone reviewing it in the future.
Some param directives to describe the feed, prod, and tails, with units, would be helpful.
They can be set to lower case. They were kept uppercase just because of the structure of the Osciloscope manual (but instructions are not case-sensitive). 
I don't know if it was the correct way of declaring a private method using the decorator Action(). In the end, what the user will actually use is measure_mean, etc. Perhaps the Action() decorator should go in that definition? 
The function returns the value and not the index; it is a built-in oscilloscope function and not the python method min.
Extra space in front of `webvtt_cue_text_text_token_ptr` after `(`
That'd be great. It'd take care of #116.
You should yank lines you aren't going to use, or leave a note as to why they'll be eventually resurrected
Ahh, nevermind, I missed the reference to L506.  I'll explore that.
Fine by me, I just copied the moved that was already there.
Well, it depends what is our goal. resolving conflict is for me just bonus. Warning should be that you use already used shortcut.
And reason for full log of it is that noone care about it :)
Ugh.  I don't like `OPERATOR(madlib.+)` doesn't improve readability.

Off topic but I always wonder why we don't set search_path to madlib at the beginning of the driver function.
Can this `if` clause and the `getRefreshToken` function be made unnecessary if we change `Accounts.updateOrCreateUserFromExternalService` to not update fields that are already there? Specifically by making the following line dig into each property and construct specific $set directives? https://github.com/meteor/meteor/blob/master/packages/accounts-base/accounts_server.js#L222

(Not even totally sure this is a good change but want to understand this code better)
For for for ...  these seem to bug me for some reason :-)
This line of code assumes that a `SiteScan` is always for a single top-level domain. We need to validate this assumption with stakeholders before we move forward with it. I've added it as a "needs clarification" item in https://etherpad.mozilla.org/mobile-evangelism-tool-planning
It doesn't matter that the static libraries are missing.  `xmlstarlet` links dynamically by default.
```rb
%W[--prefix=#{prefix}
   --disable-debug
   ...
```
This line prevents macports or fink being an issue anymore.
These two lines can just be `bin.install 'hardlink.py' => 'hardlink'`
I think `#debug` displays the output only if `debug?` is true; in that case, `if debug?` is superfluous.
Commands don't exit, they raise.  Subclass and throw a known exception type (in this case, invalid argument)
Need an errBack here to finish the HTTP request if something fails, I think.
It might look crappy, but it's MUCH easier to grok
I suspect this is missing a `yield` and `@inlineCallback`
> Уточнять, что отслеживаются изменения «во времени» или «с течением времени», в данном случае вовсе не обязательно, поскольку это не является отличием

Ну тут всё же объясняется принцип работы других СУВ,  и к тому же это только первая глава, поэтому лучше ничего особо не сокращать и не выбрасывать. Оригинал тут был такой:
These systems (CVS, Subversion, Perforce, Bazaar, and so on) think of the information they keep as a set of files and the changes made to each file over time, as illustrated in Figure 1-4.

То есть тут, наверно, просто неудачно переведено, но смысл был именно в том, что есть файл и есть сделанные за время работы с ним изменения к этому файлу.
lower case the 'Please' please :smile:
Document that this must accompany @Provides? Or perhaps this should be an attribute of @Provides? 

Some options I haven't considered to carefully:
  @Provides(collection=true)
  @Provides(mapKey="foo")
  @Provides(unique=false)
I'm not sure if Classloading really captures what this is. Could you call this GeneratedCodePlugin ?
Rename to fTestClass or fClass please
use with open(path, "wb", encoding="utf8", newline="\n") as f:
and indent everything that uses the file
and pass the file to all the methods that need it
The mockup wasn't too specific, so I just made it the same as the nav bar.  That's where the gradient came from I believe. I can remove the gradient and the shadow.
Yeah, I thought for a little while about this but didn't have any good ideas. I have the same ambivalence you do, though.
Or better yet, "type".
I think we should rename the arg to maxLength
Fixed. I pass a custom sorting function to sort().
Is it not possible to write this in a livemarkup block, where we disable all these items when in readonly mode?

Like

```
<?lm
if (!require('core/ide').readonly) { ?>
   <a:item/>
   <a:item/>
<?lm
}
?>
```

It seems so tedious.
This regex seems very inefficient. log lines are short, perhaps it doesn't matter, but .*? three times and .* once seems far too loose to me.
generally you at least want to catch Exception, so you don't hijack things like keyboard interrupts
if the design doc is huge, this becomes a blocking operation, can we instead yield the dsign doc/view so it can be touched/iterated by an outside process? if we have 100 design docs, it'll do them one by one here.
I think it would be better if this used the the case xml generator libraries and submitted that. Otherwise this will be the only place in the code that doesn't create cases through that workflow, which, among other things, won't properly set the action blocks, and could prevent long term migrations that work off the raw transactions.
```el-get-self-update``` is forced to being synchronous :)
Should there be an `init` call here as well? This constructor would probably be better written as:

```objc
- (id)initWithURL: (NSURL *)URL {
    return [self initWithURL: URL withEncoder: [[XMLRPCDefaultEncoder alloc] init]];
}
```
The namespace is not CALDAV:, it's just a shortcut they use in the rfc. Use Sabre_CalDAV_Plugin::NS_CALDAV instead. Also, instead of the d: prefix, use cal:
Could xlim and ylim be documented in the same file to avoid this duplication?
OK. You are right. login view populates this variable.
Done! I throw default 403 error, is this OK?
Move this bellow just before the `mongoengine.connect`.
we could probably add a 
    if ( !window.jQuery ) {
      return;
    }

at the top of this file so we can skip assigning variables and reduce indentation
The above is all in single quotes, the rest of the file is double quotes.

Change the above to double quotes.
instead of two declarations just do:

    #video, #video iframe {
        height: 100%;
        width: 100%;
    }
Drop the butteruid here and in the line below.
Based on how you're calling this, could you not make this async?  You'll block on the `finishedCallback` in `attemptDataLoad` function, so this would give you non-blocking behaviour.  Maybe you want to block on this?
2 things here:

1. You can use String#strip! in cases like this, so `str = str.strip` can be replaced with `str.strip!` to strip the existing string object rather than create a stripped copy.

2. The comment says it handles trailing white space, but strip will remove both leading and trailing whitespace.  If you really want just trailing whitespace, use rstrip, otherwise update the comment.
Huh? No. By that logic, then every module or state that has its own options should have those options exposed in master/minion configs? Also, I'm not getting the part about "dependency injection"..., what does `__opts__` have anything to do with dependency injection?

About parsing. Sure, you can go fancy, but I think that's good enough for now..., well...,ok, I suppose making sure it has a exactly one dot in it would be a good thing to do.
We talked on IM and it sounds like this method was aimed to create a single exit point for messages in order to allow for validation / cleanup.  I'm suggesting we avoid this because it has the potential to become brittle.  Instead we should clean and validate user input right away (since we do have a single entry point).  That way we know that all the pieces to a message were kosher.
This might be a better method to add to BytesToString
This is the part which configure /foreman prefix. It is written exactly as /katello prefix. But /katello works and /foreman do not work. :(
This doesn't look intentional.  Is it?
First `\` is not required.
no, you always manage to have one report per transaction (request)
I like the extra hashes, makes it prettier and stick out more when viewed from the command line or text editor.
Keep notes to yourself to yourself
I agree with @nhmc that it makes sense to leave these as functions - this is "bread-and-butter" code for any kind of cosmology code, so quick ways of doing it are useful.
and that is why aligning assignments is bad ...
When removing the constant, I did not consider to remove this, will update.