labofsahil/github-event-dataset-2012 huggingface dataset:
(#12630) Run puppet agent as a service

Originally we were planning on installing puppet agent as a scheduled
task, but there are issues with this, such as the task scheduler service
itself may be disabled, may be configured to run as different user that
doesn't have sufficient privileges, etc.

This commit adds a wrapper script to run puppet agent as a service using
the win32-daemon gem. The Service Control Manager invokes the daemon.bat
script, which runs `rubyw.exe daemon.rb`.

The daemon ruby process sits in a loop executing `puppet agent` every
`runinterval` seconds, the value of which is obtained from puppet itself,
and can be configured in puppet.conf.

Windows services cannot write to stdout/err. Instead they are supposed to
write to the event log. But since we don't support that yet, the daemon
logs to a file in var/log/windows.log. By default the daemon logs at the
`notice` level.

In order to pass arguments to the service, you can either use the Service
Control Manager GUI or do the following from the command line:

  sc start puppet --debug --test --server foo.com

If --debug is passed as an argument, then the daemon will log at the
`debug` level.

This commit also updates the msi to install the daemon as a service. The
service's name is 'puppet' and its display name is 'Puppet Agent'. During
an install, msiexec will automatically stop the (old) service, install the
new files, and start the new service. The service is installed as an
automatic service, meaning it will start when Windows starts. It is
difficult to make this configurable from the command line, but can easily
be done using Orca/InstEd MSI editors.
Basic dragging

I have spent about 4 hours on this, the dragging is still not working. I want to push my changes to you, hope you could find the solution. I added a cube on the plane, you could test the dragging on that. 

BTW, I think we should keep the shift+drag=rotating function as a backdoor for debugging purpose, as long as it doesn't interfere with other functions.
import must be at top of build.sbt, or SBT will complain

I've improved the comment about "import".
LPS-21348 Deleting a document and its shortcut at the same time causes an error
replaced view and post ingest hooks
convert $output to array to fix implode() error
Jellybean

Fixed it! :)
Added something about the Cartesian `for` loop

This documents JuliaLang/julia#330.
DATAREDIS-82

PersistenceExceptionTranslator wraps unknow exceptions in a
RedisSystemException, this causes other spring data projects like JPA
to have their exceptions systematically wrapped.
LPS-25643 Child pages are re-created after deleting them from live
LPS-24057 apply default comments for format-javadoc init option
Fix exception when trying to render an inline scaffold from a namespaced controller

Fix exception when trying to render an inline scaffold from a namespaced
controller (e.g., render :active_scaffold => 'admin/scaffolds/admins') when
using AJAX to insert the rendered scaffold instead of render_component.

As discussed here:
http://groups.google.com/group/activescaffold/browse_thread/thread/e6a7d2b705974609/6bfd935a479f44cc?show_docid=6bfd935a479f44cc&pli=1
LPS-25504
LPS-25166
LPS-25592 Retain question and answer flags
ganglia link on node page

when using the Dell Hadoop RA, the only access to the crowbar UI is via a 'public' IP.  by default the ganglia link on a node page points to the 'admin' IP of the admin node.  this commit changes the link to use the 'public' IP if it's been configured, otherwise is falls back to the 'admin' IP.
Simple syntax fix in budget controller

Removed colons (:) from when statements in  case session[@sort_name][:key] @ lines 154-160 for compatibility with Chiliproject 3.0.0 in 1.9.2. Nothing complex, tested and working.
Option for panel popup on save

After using the plugin for a few days it started to bug me that I had to hit the JSLint keyboard shortcut to see _what_ errors it found when I saved. This seems like something other people might want, so I made it a setting :)
So dalle
Allow slave reads to be turned on by default.
changed location for local scripts and css to be correct
INT-2393 Add Sonar Support (Gradle Plugin)

For reference see: https://jira.springsource.org/browse/INT-2393
INT-2399 fix processMessage for service-activator

JIRA: https://jira.springsource.org/browse/INT-2399.
1. ServiceActivatorFactoryBean#createMessageProcessingHandler: force 'processMessage' method-name parameter into constructor ServiceActivatingHandler for MessageProcessor.
2. Test for check 'handlerMethods' in ServiceActivatingHandler's MethodInvokingMessageProcessor.
3. Integration test for <service-activator> with invalid Groovy inline script.
Add composer.json

I inferred the license based on other Symfony2 Doctrine bundles. I also guessed on the authors based on the very small commit history after this was separated from Symfony itself.

Please double-check the deps before accepting.
Updated readme

Signed-off-by: Brian Gilbert <brian@realityloop.com>
Support Arduino 1.0.

Please add support for Arduino 1.0.

This time the pull-request is being done correctly, without me being lazy. :)
LPS-25728 / LPS-25729

http://issues.liferay.com/browse/LPS-25728
http://issues.liferay.com/browse/LPS-25729

No failures are expected.
Bootstrap 2

Modified to utilize Bootstrap 2.0.1.  Still needs some work, the sidebar isn't rendering properly, but the major modifications to support the new release have been made.
Add curl output

For issue 15
updating labels, server fixtures and tests for tc Server v2.8
Replace Zero with Data.Void.

The `Zero` type as defined in Common isn't actually empty. This patch replaces it with `Data.Void`.
Next scm change

1. Fixed CI build failure issue: define a default value for property scm
2. Updated script "configure" to make it has a consistent command line argument with "configure.bat"
Pull changes from orloffv

on click backdrop, on press escape trigger cancel & close
Adding blackberry.io.filetransfer.upload

Adding the blackberry.io.filetransfer extension and implementing the upload() function.
This allows users to upload a file from the device to a server.

Issue:  blackberry-webworks/BB10-Webworks-API#26
Pull request for docs:  blackberry-webworks/WebWorks-API-Docs#126

Viewable docs:  http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-file-upload/view/blackberry.io.filetransfer.html
Multiple uploader instance support - Issue #21

SWFUpload supports this (http://demo.swfupload.org/v220/multiinstancedemo/index.php), and now S3SWFUpload can too. 

There are a number of requests for this and an open ticket, which this should resolve: #21
SWC-255: move logic to where we're clearing out fields, and add unit test
Fix for broken build.  Unused search index fields are now deleted from the search indedex.
timob-10208: Kitchensink: ScrollableViews - Required buttons are not displayed on the view.

https://jira.appcelerator.org/browse/TIMOB-10208
Test steps in JIRA.
Sauce REST API Changes

Hi Matt, I've completed the modifications to the SeleniumFactory-for-Python library.  I've made the following changes:

* Added a Wrapper class to SeleniumFactory.py.  This class will wrap selenium/webdriver instances that are created when the SELENIUM_DRIVER is specified, and provides helper methods for marking a build as passed/failed and to set the build number.
* The Sauce session id is output to sysout, which will be parsed by the Bamboo/Jenkins plugins
* Added SauceRest.py which provides helper methods for invoking the Sauce REST API
* Added sauce_ondemand_test.py which provides unit tests to verify the behaviour of the wrapper invocations of the Sauce REST API

Let me know if you're happy with the changes.

Thanks,

Ross
Support for Padrino

Simple gem, well executed, thanks!

Here's a patch to support Padrino. Since Padrino is built on Sinatra, the Sinatra check succeeds, but then the settings variable is nil, and the app tanks.

This just makes a check with Padrino, then uses the Padrino root method. 

Note, it doesn't quite operate the same as the Rails.root, (see https://github.com/padrino/padrino-framework/issues/730) but what I have checked in works.

H
Ajout ergonomique

Désormais lors de l'appel à la méthode LocMenu::afficher(), l'écran du terminal est préalablement affiché.
Korean translation :  quiet hours(strings)

update quiet hours(strings)
correct custom brightness(strings)
Fix for issue 4

It also includes a change in localization files for recordings on meeting detail.
updated all packages repository links and fixed some markup issue

Hi all,
    I updated all the packages repository to link to github. Then I've done some markup cleaning, no more warnings during sphinx-build. I've updated all the references to the dexteritydevelopermanual to point to readthedocs. Can you merge it?

Cheers,
Giacomo
I am a horse

2. kierros. Ei mitään ihmeellistä tässäkään.
Worker_install geaendert, sodass er nun nach Status agiert
Fixes to title, author, etc.

Yeah, I messed it up, also I keep committing to the wrong branch now.
Refactoring

Refactoring to allow different propagations of requests and clean up the transaction management. Also split up responsibilities.
That's just how it's done. See for example: https://docs.djangoproject.com/en/dev/topics/http/urls/#example

Come to think of it, the only advantages I can find are:

* a trailing slash means you can have a form with `action=""` so that when you submit the form it reloads the same URL but as `POST`. Otherwise you have to repeat the URL in the `action` attribute and that ties you down to one submission URL per form snippet.

* Django always appends a trailing slash to unfound lookups and tries again. E.g. /owfjwofijweffd is redirected to /owfjwofijweffd/ so by being consistent and always using a trailing slash you avoid the excess redirect on (or just before rather) 404 errors.
Should be escaping variables put into XML.
MongoSessionFilter应该是更好的选择
@gfosco I can't agree more... stupid Rails. Powered Rails on the other hand, they are pretty awesome, but you do need to get a lot of gold.
{LogInAsLabel, "Сиз"},
  {CurYearLabel, "Жыл"},
  {CurSemesterLabel, "Семестр"},
  {CurWeekLabel, "Апта"},
  {CurExamLabel, "Сынак"},
Je to jen součást testu
Can remove the todo also now ;)
Oops. I should have done this when I committed the update to the PHPUnit bootstrap. Sorry about that. :-)
oh!!!!
これむっちゃやりたかったです!!
ありがとうございます!!
抱歉,刚才提交的时候,没有看到你写的,就重复提交了这个问题
"I didn’t know who wrote the file at the beginning, so I didn’t shout at YOU, i just cleared it, and I didn’t care who really wrote the code at the beginning, and it doesn’t matter."

As you say, it doesn't matter who wrote the code, but it's still very insulting towards him.
maybe make a comment about how this is for migration, roughly how it migration works, 10k view, or some other thing that explains why there is a 090x2 version
I know that was driving you crazy...
Well, we need to asl Lluis for this, as it is touching the global module. Don't think there's going to be a problem, but we will ask just in case (I will hang out tomorrow in irc.gimp.net to ask him about this).
While using `_` is common in functional languages, it's surprising to Ruby programmers. I'd suggest you use `#last` here.

```ruby
command = m.params.last.match(/^!(\S+)/).to_a.last
```
I would put this into a separate function into circus/util.py, because we need the current directory in several spots
The convention is to have a full lowercase name, that's why I wrote that ;)
MSVC appears much more challenging.
Correct! Seth called it. Thanks for fixing.
upps thanks for fixing this
@rsa =)
Сергей, действительно все проблемы с рыбалкой решены (на данный момент).
Я же не "от фонаря" взял этот код, сам долго и упорно тестировал и дебажил. В результате это вылилось вот в эту квинсенстенцию.

Нужно решать проблему с LOS, а точнее с твоей добавкой - TerrainInfo::CheckPathAccurate(...)
Конкретный пример: игроки жаловались, что, что за тупость - я хил, стою ниже ступенек, мы рейдом бъём Саурфанга (который чуть выше), а мне пишится, что цель в не зоны действия. Ну действительно неправильно.
Откатил последние правки и написал так:
        if (tstZ < (INVALID_HEIGHT + M_NULL_F) || tstZ > (MAX_HEIGHT - M_NULL_F))
        {
            ++errorsCount;
            break;
        }

        tstZ += (DELTA + DELTA_Z);

с этим проблем нет теперь.
Martijn, I can agree that the first version is less likely to lead to an ImportError on recent versions but my point here is not about programming, is about logic...because of the dependency between them, if zope.component is missing then zope.app.component is even more missing and thus that try/except is useless.

+1 for the getUtility

just my 2¢
Klingt vernünftig :-)
Seriously... You copy/paste code to point out that you must copy/paste code. You can do better than that. And why should GUI feature match the same than CLI? If it wanted just that then I wouldn't have started this whole thread!

GUI already collects a list of files its going to check - it can expand that a bit (one function call) to get the extension. Then there are lots of possibilities - the basic thing of comparing extensions of course, but without platform-specific ifdefs etc. Then it could allow e.g. using user's configuration for the extensions to allow needed customization (and solve things like that problem with `.C` and `.c` extensions). If you think GUI as CLI with windows you don't get anywhere and can claim whole GUI is unnecessary. Based on development activity it might be?

One problem with GUI development is exactly this - people don't think past what they want CLI to do. So how could GUI add any of those neat features when it is limited to the very limited "CLI is everything we need to support" -thinking. lib should do its base task of checking given file and let clients to all the rest.
Which clients you are talking about? Once per 10 seconds updates are useless for everybody.
Messed up line breaks in my-defuns.el somehow.
after this commit GM's may teleport to test map. if this need for regular player - need implement custom areatriggers.
Thermaltake, тебе бы из десятков писем приходящих ежедневно приходила такая же чушь, ты бы тоже злился!
You're right. I'll remove.
make sure you use the branch collisionAlg_v2

check it by "git branch

Best regards,
Yu-Shiang Wong



On Sun, Jan 8, 2012 at 9:20 PM, Simon <
reply@reply.github.com
> wrote:

> When I press SPACE key , the program just collapse immediately.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sssnight/qtgl2d/commit/d01f7bdfbe40fc9373a68487c64a4ba776cc3f48#commitcomment-847480
>
Don't forget chef attributes are hierarchy, not just flat map. Consider `default['mysql']['tunable']['max_allowed_packet']`.
Wondering why not just use getAttribute("scrollable")... what do you think?
Arkane, perfecto amigo, buen trabajo.
this was left as not being autoreleased because somehow, the attributed string will be over released when unserializing. I've yet to hunt down this bug.
@chochos That's why it says "prefix", not "postfix" ;)
Be patient, I'm working on it...
technically `.text` is not an attribute. Also unlike other browsers a cloned script node in IE can be executed again once it's source is set. Also a script node in IE can be recycled and executed many times, as well as executed when appended to a document fragment.
Woohoo!  Sane defaults.
If the ``length`` is optional, how can one know how many e.g. integers are returned in the list,
in order to be able to walk through the list?

I suggest we just make the ``length`` parameter mandatory.
It's perfectly fine to call `hClose` on a closed `Handle` according to `hClose`'s documentation.  Do you have any other reason for giving this warning?  Should there not be any other reason, the API could be simplified by returning only the `Handle` and allow the user to manually call `hClose`.  Or at least this scary warning could be removed =).
That's just for flexibility. I am not using the ability to replace the global keymap (as in take actions out of it) but I thought maybe a porter would maybe want to do this? It's pretty hard to defend so yeah I agree with you it is probably not relevant/needed and I will remove it.

I really don't want to tackle the idea of "adding actions to the global keymap" as I cannot come up with a good way to do this. Inheritance is supposed to be achieved through transparency anyway. There's nothing fundamentally wrong about having 2 or more global keymaps in the stack.
I hope this does not mess up anything else. Test ran fine. But I am not too sure of any other place using this view for AJAX.
Hey.
Why 8 messages?
For me it's enough 1, explaining what you don't like.
Could you explain please, what did you mean.
Vladimir

----- Original Message -----
From: "OndraZizka" <reply@reply.github.com>
To: "vratsel" <vrastsel@redhat.com>
Sent: Wednesday, February 1, 2012 10:58:10 PM
Subject: Re: [jboss-as] Metrics DS tests (d484e28)

Uh oh... please not this way. Read https://docs.jboss.org/author/display/AS71/AS+7+Testsuite+Developer+Guide and fix.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/vratsel/jboss-as/commit/d484e282d990218f9aa9653f337cdaa0b2710447#commitcomment-924209
Yeah travis told me already ;) I got a half-patch here, just one last issue and not sure why it fails.
How would you check to see if Debug is being installed at CMake time (or should I print that at install time somehow)?

If it makes the test pass could we squash these and commit them to CMake trunk?
high level idea here is we can cache minification of objects that don't change?
@carasuca What i mean is i need this implemented before I test the app and the whole infrastructure. Keep me posted.
As long as we can do .. in .pth, that works for me, and we should put manage.py in lib so there's less boilerplate in each site.
heh! did not see that one myself, yeah PRODUCT_PROPERTY_OVERRIDE += ... xD
As you agreed on the phone, the bug was fixed in the right way.

On Thu, Jan 26, 2012 at 2:37 PM, Ayende Rahien <
reply@reply.github.com
> wrote:

> Not really, no.
> Because that would result in the bug you mentioned.
> We should do that when we actually call Navigate, that is the action, not
> in the response to the action.
>
> On Thu, Jan 26, 2012 at 2:30 PM, Fitzchak Yitzchaki <
> reply@reply.github.com
> > wrote:
>
> > In the Frame control we handle any internal navigation, and IMO this is
> > the place to decide whether to forward the navigation to a new tab.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/fitzchak1/ravendb/commit/c6e12fec50d2f9d4f2801a60229cd9b173f0b35a#commitcomment-903980
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/fitzchak1/ravendb/commit/c6e12fec50d2f9d4f2801a60229cd9b173f0b35a#commitcomment-904002
>
This is so that if the user supplies the location of a config file via the CLI that's different from the defaults so that when LoadConfig(final String path) is called, it doesn't try to load the defaults.
c'est toi la modif !
I am in eResearch office today, and may be tomorrow afternoon. Please let
me know if you are around.

Regards
Amir

On Thu, Feb 2, 2012 at 9:14 PM, Nelli <
reply@reply.github.com
> wrote:

> Hi Amir,
>
> Sure. Are you at work on Wednesday or Thursday?
>
> Regards,
> Nelli
> Am 02.02.2012 09:07 schrieb "Amir Aryani" <
> reply@reply.github.com
> >:
>
> > Hi Nelli,
> >
> > I am a bit confused here? As far as I can see this is not the code that
> > you added to GitHub? So how did you get the message from Tim?
> >
> > If you are in Melbourne and have time, we can catch up. I will be glad to
> > have chat with you about possible improvements of the InPython code, and
> > related research work.
> >
> > Regards,
> > Amir
> >
> > -- sent from an iPad
> >
> > On 01/02/2012, at 22:39, Nelli<reply@reply.github.com> wrote:
> >
> > > No. I don't know why, but I haven't even thought about that. But I am
> in
> > > Melbourne next week and if I still have access to the computer I worked
> > on,
> > > I will try to improve the code with it.
> > >
> > > Thank you!
> > >
> > >
> > > 2012/2/1 Amir Aryani <
> > > reply@reply.github.com
> > >>
> > >
> > >> No idea!
> > >>
> > >> On Wed, Feb 1, 2012 at 11:48 AM, Tim Dettrick <
> > >> reply@reply.github.com
> > >>> wrote:
> > >>
> > >>> Any particular reason why you didn't use the [tempfile](
> > >>> http://docs.python.org/library/tempfile.html) library?
> > >>>
> > >>> ---
> > >>> Reply to this email directly or view it on GitHub:
> > >>>
> > >>>
> > >>
> >
> https://github.com/aaryani/CoreTardis/commit/2efeced5d7caac1332df9eaf16d058d7d750c337#commitcomment-920852
> > >>>
> > >>
> > >> ---
> > >> Reply to this email directly or view it on GitHub:
> > >>
> > >>
> >
> https://github.com/aaryani/CoreTardis/commit/2efeced5d7caac1332df9eaf16d058d7d750c337#commitcomment-921133
> > >>
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> >
> https://github.com/aaryani/CoreTardis/commit/2efeced5d7caac1332df9eaf16d058d7d750c337#commitcomment-922212
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/aaryani/CoreTardis/commit/2efeced5d7caac1332df9eaf16d058d7d750c337#commitcomment-924246
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/aaryani/CoreTardis/commit/2efeced5d7caac1332df9eaf16d058d7d750c337#commitcomment-925815
>
Hi Iulian,

I couldn't get sym.info.paramTypes to compile (see line 123 of
https://github.com/cheeseng/scala-ide/blob/master/org.scala-ide.sdt.core/src/scala/tools/eclipse/launching/ScalaTestLaunchShortcut.scala),
I don't like the line 124 too:-

defDef.vparamss.flatten.toList.map(valDef => valDef.tpt.symbol.fullName)

so if I can get sym.info.paramTypes to work it would be much cleaner.

Thanks!

Best Regards,
Chee Seng


On Mon, Feb 6, 2012 at 3:51 AM, Iulian Dragos <
reply@reply.github.com
> wrote:

> All type information should be retrieved using the `Symbol`. They hold the
> correct information, while the tree contains what the user wrote (meaning
> there is no type inference results, etc). So I would do:
>
>    val sym = defDef.symbol
>    sym.name
>    sym.info.paramTypes
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cheeseng/scala-ide/commit/86ae8bb98f8f3c6d0c6b26ce868de58e0b1978e7#commitcomment-934650
>
@dbu 2 questions regarding the Proxy implementation:

- why having a special handling for ``__set`` and ``__get``, defining them always and dropping totally any parent implementation ?
- what about implementation the same logic than added in ORM 2.2 to avoid loading the proxy if only the identifier was needed as you already have it: https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Proxy/ProxyFactory.php#L234 ?
If I'm not mistaken the error message needs to change.
Oh! Yeah I actually thought it was a typo lol. Didn't realize it was pulling in Base syncer functionality, although it makes sense. But since the parent module is "RSync" I somehow assumed it was coming from RSync, which can't be. Although quite obvious, might be an idea to make an exception in the a situation like this where the class name is the same as the class name of the class that it extends. What do you think?
Is there trailing whitespace after the }  ?
This breaks compatibility with Rails 2.3 (class_attribute is defined from Rails 3.0)
@geertdd does this patch include new strings?
It swaps CamelCase with table form (FooBar => foo_bar)
Doing that would require changing the call signature of `make_command` throughout all the plugins that make commands. Your style will work when the pattern is

```
@deco
def foo():
    pass
```

But the pattern already set is

```
@deco()
def foo():
    pass
```
Are you sure that's Regex and not just you keyboard-bashing? :p Crazy!!
havnt tested compile. pls post errors here
You do not need to copyright it to oemr.org. Most of us copyright to ourselves or company; see other scripts for examples. If you desire, however, you do have the option of copyrighting it to oemr.org.
My find and replace skills are not up to scratch :(
Thank you for your work. ;P
Here's how blog routes will look - https://gist.github.com/1600844.
can you remove the SB drivers again from ATV and Ultra build, or is it such a USB Soundcard device?
Joda, det ser bedre ut, men kanskje det skulle vært www.blug.linux.no ? for å understreke at det er en URL?
Please send the correct type, maybe that already fixes it. And use our json_encode() wrapper:
{{{
$response = new SS_HTTPResponse(Convert::raw2json(array($return)));
$response->addHeader('Content-Type', 'application/json');
return $response;
}}}

Otherwise, use a standard encoding rather than custom stuff like ::lt:: - Convert::raw2html() should do.
You can decode this again with something like this: http://css-tricks.com/snippets/javascript/htmlentities-for-javascript/
shit.. this is a great idea!!
Html syntax highlighting didn't worked for me, until I changed the order of the filetypes, first *.twig and after that *.html.twig
I don't know, what did you do? Did I goof up something? If so, sorry.
```php
$versionAsText = $xpath->query('//table[last()]/td[1]')->item(0)->textContent;
```

I'm not aware of the convention Mailman follows, but if it's like PHP, you could go a step further, and just use version_compare() over the text.
You might want to revert it. Mixed up some stuff. Sorry :(
Bitte Codestyle beibehalten
I had assumed we would remove this feature entirely and use "likes." Do you see like/recommend as distinct enough to keep them separate?
Ok, I reverted this commit then..

On Wed, Feb 1, 2012 at 00:07, Daniel Black
<reply@reply.github.com>
wrote:
>>  The comments you added show exactly why this is a bad idea:
> yes it is a bad idea . Perhaps keeping all monitoring classes in the same module.
>
>> * this file resource shoudl require the user, group and the directory
> the user, group and directory are autodepends of file resource. Given these are all defined in the tribily module...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Tribily/puppet-mariadb/commit/614b8b376dcc02085d7d0cf94f146c369c349ff3#commitcomment-920399



-- 
Walter Heck

--
follow @walterheck on twitter to see what I'm up to!
--
Check out my new startup: Server Monitoring as a Service @ http://tribily.com
Follow @tribily on Twitter and/or 'Like' our Facebook page at
http://www.facebook.com/tribily
You should commit each change for itself, to ease up bug tracking abilities
Hi Kenneth, thanks for catching this!  Mark has merged this for the time being, but note that the description actually *has* been set in the docbook-reference-plugin \[1,2\].  For some reason, though, these tasks aren't showing up in `./gradlew tasks`.  I'll file an issue with the Gradle team about this shortly.

[1] https://github.com/cbeams/gradle-plugins
[2] https://github.com/cbeams/gradle-plugins/blob/master/docbook-reference-plugin/src/main/groovy/DocbookReference.groovy#L55
http://alltheragefaces.com/img/faces/large/misc-jackie-chan-l.png
Finally got the Markdown syntax proper this time.
This is an implementation detail, you are probably confused because you were expecting a hash from having built it with a hash. If you handed it to someone and said "this returns an array of anagrams" there would be no confusion because it is the interface which is important and the tests would cover the implementation. Also, this style of chaining methods together is fairly foreign to most OO developers as it is functional in nature. You could actually chain the whole method into a single line.

This is slightly easier to read:
```ruby
anagrams_with_letters = @hash.select {|letters,words| words.length > 1}
anagram_words = anagrams_with_letters.map{|a| a[1]}
```

You could also use an anagram class with the sorted letters and any words you can form in it.

Also, you read through these in backwards order.
HEISEI AI )))
Hi Nathaniel, I'm a user of cluster_chef and I asked a question to @mrflip about Cookbooks and cluster configuration should be cloud agnostic (https://github.com/infochimps/cluster_chef/issues/28), but doesn't get his response by far.  I see you are actively commiting the version 3 branch, could you help me on this question ?

my question is :

    it's great to see cluster_chef extends the support to various cloud provider.
    from the source code compute.rb of version 3 branch, i see it still only support ec2 :  raise "Only have ec2 so far" if cloud_provider && (cloud_provider != :ec2)
    Is there any sample code and guide to use Vagrant/Virtualbox?
    If i want to extents cluster_chef to support vSphere, could you give some hints on where to add code ?

Thanks a lot!  I'm eager for your feedback.
and sorry for piling up comments: i think we should have in the shared code all the logic to load blocks but inject a BlockRepositoryInterface or something, instead of accessing any document manager directly. for the cmf, this could handle different types of block locators (uuid for references, name for child names of current pages, user can write his own repository for special cases)
Du kunde också satt parentes runt subtraktionen
Espera que lo pruebe mas detenidamente xD.
Rickrolled :D
it's because timeout is not passed through the normalize_time() call in pf::config like the other date/time parameters are. See lib/pf/config.pm and add timeout to the list of processed variables.
now looks even better ;P

I have problem with getting proper angle for summoning creatures. Broken Frostmourne isn't summoned in front of Lich King, I don't know what I'm doing wrong ;/
This if statement is forcing users to have a username that's at least 25 characters long.
Erm, just saying... I now know your API key! :P ;)
Does anyone know what the second `w` in `chmod -R g+rww /var/apps` does?
Look at Multiverse/Multiverse-Portals@80768d2f1943fd15dc5e4276307822596d7dc72c, this is used there.
After this commit you cannot build the project if you define anything lower than android-target:11. The errors are:
/anki/res/values-v11/themes.xml:4: error: Error retrieving parent for item: No resource found that matches the given name '@android:style/Theme.Holo.Light'.
/anki/res/values-v11/themes.xml:5: error: Error retrieving parent for item: No resource found that matches the given name '@android:style/Theme.Holo'.
/anki/res/values-v11/themes.xml:6: error: Error retrieving parent for item: No resource found that matches the given name '@android:style/Theme.Holo.Dialog'.
bind css selector event on document by default if user not provides domNode.
No problem, I figured I would pick up some of the notices my dev system is getting from running with Debug turned on. Its a good way to start getting familiar with the code.
Чет плохое произошло, пользователю ты это выдал, а программа то не в курсе :) этот то эксепшен я бы пробросил наверх.
Fantomas42/django-blog-zinnia@b353fa8a9c91d1167b6ad68792225d4e41e36008

fix this issue.
Good catch! and sorry about that. a fix is coming right up.
I don't think these comments are that helpful as the type and variable names are probably good enough.  If they aren't, then lets pick variable names that are more descriptive
I also think that some of the helper logic inside of the domain class should be moved out to a service.
this commit should be merged with the previous one
+1 for constraintvalidators
I can't reproduce your issue so test this: http://review.cyanogenmod.com/#change,11913
It's highly recommended to use the Metasploit API, because in case the Ruby one is depreciated (for a better function or whatever reason), we don't have to fix code all over the place.  It would be nice if you could modify the code, and then a commit/push should update the pull request.
The spec only says `county`.
Level 2 has 2 objectives. Are you sure that's a good idea? I'm thinking it may result in multiplayer matches in which you don't see the enemy player.
new permissions aren't required at this point but will be soon
このブランチに修正コミットを積んでくれると助かります
@gvalkov :  see pull I submitted to your fork on how to remove the `import pip`
Maybe 'Show detailed information about an installed package'
I'm unsure whether we should support `shallow`, I saw that underscore does, and figured it might be useful… I do hate boolean arguments though :(
@author and @version for posterity ;-)
See this commit https://github.com/akka/akka/commit/31ace9e83f1a9dc361bc6d1aa4e79c4825e8e07e
If there is any drawback I'll revert.
So, shold it stay? Should I remove?
final override val ?
It is so I have a chance of testing 1. before 2. happens. I could also split up in two tests. Setting the duration to 30 seconds (to turn it off) in one test and normal in the other. WDYT?
if (headActor != null) try {
It won't matter, I'll experiment with that in master, would be nice to avoid accidental boolean-complexity
Works like a charm. Reads much nicer.
Yes. SupervisedAsk is a nice one. And the method could be askOf in order to underline the Props similar to actorOf?
ok, I'll change to private[akka]
Agreed, I also don't like that comment. That sentence in its original form needs to be read many times in order to be understood :)
I think there was an issue that lots of potential customers use IE 7 due to public sector issues
For consistency with the rest of the CKAN action api, I would make this accept either a string or a list of strings (as it does now).  But not try to parse a single string as a comma-separated list of strings.  If you want to allow passing in a comma-separated string, then you should probably provide a way of escaping a comma-literal.
Is there a reason you're using `.unbind()` here? It will remove all bound events from this element, which could be quite destructive, it also looks unlikely there will be any at this point.
What about "private" organizations - that could be useful (i.e. organization is not visible except to organization members)
I was pissed at css :P
No reason. Fixed ;)
done that way as a small act of protest.. "system.h" is such a dangerous include, so much safer with a prefix.
Hoe bedoel je? Ik heb alles van de enrise.com aan fonts gekopieerd voor de "enrise bootstrap"
Same here as above. Changing the raster data's `validMask` property is equivalent to changing the raster data, because the valid mask is used to determine which pixels exist and which not. Thus, this has an influence on the computed statistics and any other data processing.
This option should be split into two options (package and name)
Hm. Then I cannot publish the event, and the Endpoint actor will escalate  
the error. Is it enough, or do you need some special handling of the case?
the proper replacement for that use case is `getDefaultAddress`
Then modify the function? :wink:
Style: the backslashes need to line up at column 79.
Utilises simple_form
Why not keep the final here ?
Then remove the comments and add something like "// nothing to do" or "// NOOP"
there is no way to determine this except through graph path analysis. simply checking for output existence is not sufficient, need to check the inputs. but in the future we can do some graph path analysis (upto some depth). also this would assume that a user has not gone in and replaced things. we don't control the filesystem.
docskip
Thanks, I was looking for that function.
пустую строчку до и после
хорошо.
почему нельзя сделать один if ?
That's a fair assessment on the bang method, but the tests were written to match the others in the group.

I think it's probably a worth separate commit/PR to have the tests in this grouping do as you have described...not sure if I want to do that at the same time I'm changing the functionality.

@josevalim - Thoughts on this?
Use the Ruby 1.9 hash syntax
I think you can put everything inside one test only.
Do you mean the following code (a like) ?
```ruby
 ...
def register_task(pattern, task)
  @tasks ||= {}
  @tasks[pattern] = task
end

register_task /mysql/, ActiveRecord::Tasks::MySQLDatabaseTasks
register_task /postgresql/, ActiveRecord::Tasks::PostgreSQLDatabaseTasks
register_task /sqlite/, ActiveRecord::Tasks::SQLiteDatabaseTasks
...
```

If so, I agree with you.
Yes, so I think we need to delete the `:index` option here too:

```ruby
has_index = options.fetch(:indexes, true)
```
`if (kindErrors.nonEmpty)`
These empty lines can fight world hunger.
Do you think we need to catch errors here, or just crash if something bad happens?
Well, I only moved this code, I didn't change anything inside it and I'd rather consider that changing the implementation is outside the scope of this PR.
mutable default
Done. For the Fourier case too.
`Equality.__nonzero__` is on its way out too, though.
I would not use latex commands in doc strings. All docstrings need to be readable on the command line and should have minimal latex and ReST markup.

Also why do you quote m and not L and P?
Yes, I am aware of that, but things will become a bit clearer once there is a concrete proposal in 1.4's branch. The information is good.
Oh it's already there?  Please ignore then.
Can we move this import to the top?
Prefer flat code, e.g. `if (handle->fd == -1) return;`
loop_, not loop.
Здесь такое решение не прокатит, нужно сделать отдельный  uploader. так вот: `rails g uploader place_icon`

Это сгенерит новый uploader, сделай там что нужно, т.е. размер рисунка должен быть 50x50
So you're saying you don't know what it means?
mention the configurable journal interval. 

also it's not "reflected in your database," as it is "in the data files."
Is there any chance these IDs ever end up as part of a URL (not just a fragment identifier)? If so, that's a potential Unicode issue -- we might want to do something like Django's own built-in slugify template filter, which has a little Unicode-normalization song-and-dance to produce a readable but URL-safe result.
a vm to windows? :) haha that's what i usually do.
Instead of returning false and having to check for it all over the place, just throw an exception. It should never happen, if it happens somebody did something very wrong. That also allows the rest of the code to be simplified.
Indeed, I'd stick with arrays here.
You should probably start by creating a generic data cache test (i.e. not for the special case "file") and a generic sql cache test.
See: https://github.com/phpbb/phpbb3/pull/835/files#L1R1347

And I was told to implement something like this.
"Preparing a user's"
Unless I'm misunderstanding this regex, the periods need to be escaped like so:
`'#^avatar\.driver\.#`
Otherwise, doesn't a period match any character?
$user->lang() is a method. I actually need to change the $user->lang['INACTIVE_REASON_REMIND'] one to use () above.
Just delete this :)
c2e72f0
yes, sorry for the funny diff.. just look at the full file.

I sorted to compare with `.npmignore`
Could you remove the outdated code instead of commenting it out? 
We have a version control system (currently git) to keep the history.
sunbiz told me sometime back that we will check the range using concept resource using rest call

call should be made something like this:
HOST/ws/rest/v1/concept?q=bmi
This looks a bit fruity...
Needed this to aggregate by month on the ostia data set. (if you aggregate by month, your new points aren't necessarily going to be monotonic in time)
The thing is, g_cost does not exactly represent dx/dx, otherwise there would be no need to specify it in the first place, since it would have to be always one, and providing any non-one value would be incorrect.
Ever since Theano 0.6 we started to use the asynchronous capability of GPUs. This allows us to be faster but with the possibility that some errors may be raised later than when they should occur. This can cause difficulties when profiling Theano apply nodes. There is a NVIDIA driver feature to help with these issues. If you set the environment variable CUDA_LAUNCH_BLOCKING=1 then all kernel calls will be automatically synchronized. This reduces performance but provides good profiling and appropriately placed error messages.
it's ajax select / jquery ajax select plugins internal api.
create methods returns void ?!! I think this method should be called something like initialize, willAnnounce or something like that.
As long as we don't have an official JsonResponse in the core, I'm very hesitant to document it like that. Instead we should pass in the content_type and dump the content  on the spot in the render_to_response class as demonstrated here: https://docs.djangoproject.com/en/dev/topics/class-based-views/#more-than-just-html

Also, don't use `django.utils.simplejson` but just the `json` module.
same here than above
Yeah, my concern is that it's just not very good input. It doesn't have to be realistic, but it should be reasonable. I'll explain why.

1) You have an exposure model containing two assets which are separated by 100* longitudinally. One of them is in California, the other is somewhere in the Atlantic ocean off the western coast of Africa. That's a bit strange. We should try to define some test data which isn't so strange. Perhaps Vitor can provide us with simple sample data (following the geometry in the current hazard demos as an example).

2) Additionally, the `region_constraint` doesn't make much sense. It encompasses almost the entire globe. This kind of defeats the purpose of the constraint, which is meant to "crop" the exposure model (which can be quite large) to a small area of interest for the calculation. Given the large exposure models we will use as inputs, I think we should exercise this functionality with the demo. Please see `Risk Geometry Cases` for some illustrations about the region constraint functionality: https://docs.google.com/a/openquake.org/drawings/d/1Lqk4NIX9gmYsQpEovDTpUKB1mlxzk-ftRLd1EOvexKw/edit

Personally, I would rather see a more "controlled" case, like so:

- Exposure model contains 3 assets, at (lon=-1, lat=1), (0, 1), and (0, 0).
- Region constraint a rectangle with the corner points (-1.5, 1.5), (0.5 1.5), (0.5, 0.5), and (-1.5, 0.5); this includes the first 2 assets and excludes the asset at (0, 0)

This is a pretty simple scenario and it properly exercises the region constraint, which is a very important feature.

Does that make sense?
Some of these params are missing the trailing colon.
All these funky loop conditions should no longer be necessary; I believe we can now trust `<NodeList>.length`.
Can we start moving all `var` initializations to the top of their scope, eg. out of `IfStatements`
Also, nitpick, can we call it attrs
A separate block is needed for each compiler.

>With :llvm the warning does not show

You mean the Homebrew warning? I see it:

```
$ brew install chicken --use-llvm
Warning: Using LLVM, but this formula is reported to fail with LLVM.
[snip]
```
On Tue, Jun 26, 2012 at 01:48:24AM -0700, Alexandre Abraham wrote:
> Gaël asked me to use pyplot explicitely

Did I? I must have been drinking

> and avoid aliases as much as possible (except for numpy -> np)

other exception: pylab -> pl

G
I don't think you have to add the unminified version here? It is just for license reasons that it has to be distributed in the packages.
I'm opening a separate pull request from a branch
ja es muss *.xy heißen. es ging rein darum auch das triviale verhalten zu prüfen...
The button only needs to be hidden not unbind and rebind
Ah, I should be more specific. You should either put a <p> before "Implementations" or put that word on the previous line.
I will fixt it.
It will stay at end of previous line.
if it errors anywhere wont that break this new callback pattern? perhaps pass in a custom res obj, or pass callback to error handler or something?
These inner functions are created each time extractAllSelectors(text). That's once for each .css file in the project for each selector search. Can this be factored out?
For folder and file sorting you need to call your custom sort() function here instead of $.map().sort().
Typo ("working set"). Might also want to clarify the subtle point that we can't use `getOpenDocumentForPath()` here because on startup we don't actually reopen the documents in the working set, so we're not guaranteed that all such documents are open.
I think the merging of history only affects undo, not the actual number of change events or change lists you get as part of those events (those are driven from the `textChanged` var, which is reset for each individual operation and is separate from the History object).
Yay! Async code hints!
use a relative path here?
You're talking about adding an additional method called getControllerPath?  I did create a getControllerSubPath...you see a need for another method?
This would be cleaner if these methods returned `Optional` instead of `null`.
User in models.py now extends mongoengine.django.auth.User.
Again, this can probably solved with the labels/options ticket
updated pull request with fix
we shall do this check only one time and cache the result. you can use "shadow()" type caching.
I'm not sure if this would even go there or there's already some other place where it would fit better.
As explained above GraphQuery objects can be expensive to build, and this is there so that we can just crate case class objects for a graph to give it a sparql view.
These two lines should get indented 2 more spaces.
还是挺好使的嘛
This clear should be present irrespective of whether consumerCommitPaths is empty or not.
This line should be changed to
```ruby
org if allowed_organizations.include?org
```

org readable is not the criteria for checking whether is his preferred org. Users for example is a user is given a permission to read a provider and nothing else in the org, he should be allowed to set that org as his default org. Org.readable is more an administrator thing.
This was added by me. The load path is added to be able to load `app_config` under same path as Rails would load them. It's there to avoid loading  `app_config` file twice with different paths. `app_config` contains only a simple delegator with deprecation warning which makes access to configuration through `AppConfig` working as well.
Should year be first?
this should probably be a `app.config.get('GOOGLE_CLIENT_ID', None)` and you should handle the case were None is passed to it. That way, if someone omits the setting an error won't be raised.
Hi Martijn -

Why did you delete the coercion into a string value for the lookup? This seems like it should cover all cases… if its not working for you, I need to get some test coverage in place for the issue
As this is not about the format of the actual error message but how the data is pushed into the specific api, I thought a formatter would be the wrong place.

I'll move the logic into the formatter.
this will duplicate many things (as ``$record['formatted']`` contains the normalized record). It should probably be limited to ``$record['formatted']``.

Thus, looking at the raven client, it seems wrong. the params array is meant to contain params to fill placeholders of the message through sprintf. This is not what the record is about. With your current implementation, the message sent to Raven will not contain any data of the context or extra array (and it will store its own timestamp instead of the timestamp provided by Monolog)
OK Good
Does that means, when you hit the sleep button to turn off the screen you will see first see the lockscreen and then the screen will be turned off?
According to our mumble discussion, removing the comment above the `gen_server:call` should be sufficient.
I was torn - it does make sure the functions run, but without checking the results it gives false security.
As I understand it @fanquake should be right here.
Well it is - if C++ would allow extending enums, it wouldn't need this :)

That said, updated the comment.
May I suggest when answering from a mobile don't quote the inital message :D. This makes things unreadable here.
I think it'd be better to print a message here, silently ignoring errors makes me nervous
Meh, just went with read(100)
Can someone confirm changing this won't have any unexpected side effects?
This would be a good place to exchange these macros for functions in your new CycLimits class.
This change is distinctly opposite our style guide.  http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Function_Declarations_and_Definitions#Function_Declarations_and_Definitions
This already exists in PathHelper