labofsahil/github-event-dataset-2012 huggingface dataset:
MappingSail implements similar functionality for Sails using a middle-person. URITranslator does a similar thing, using remote SPARQL 1.1 Update queries on any suitable Repository, so the individual statements are not processed on the application side if possible.

Currently I am rewriting an application where I plan to have users uploading documents directly to an RDF repository and then have the application rewrite the URIs using SPARQL queries so that the URIs in the repository for each document are unique. My particular application needs to be fairly high throughput and I am trying to set things up so that the application server is not a bottleneck in the process. Basically if the application server (which is likely to be a virtual machine in practice and used for more than just uploads) gets behind a little with translating it shouldn't stop new uploads from occurring.

I made it into a sesametools module in the hope that I could then use it in other similar applications without having direct dependencies. I have another application that needs similar functionality, although it will not be high-throughput.
Please use Strings.isNullOrEmtpy().

Strings has a lot of useful helper functions for String. Please use it as much as possible.
button should also be put as fields so that we can control them
I've given up on requirements/prod.txt. I still use requirements/compiled.txt and requirements/dev.txt
It's inferior to using `vendor/` because of the accuracy of versions. Also, with pip install, if I pin, say `mongokit==1.0` and it refers to `pymongo` in its setup.py does that mean that I get a very unpredictable version mismanagement. 

I'd like to see requirements/prod.txt removed. It's annoying to maintain. Especially since I don't use it.
> should I be aware of any performance tradeoffs here?
Good question. I'm not sure, to be honest. I'll see if I can whip together a straw-man performance thingy later today
For Linux? What kernel you have isn't relevant: it's all down to keyboard grab in X.

Anyway, somewhat useful, but a little confusing in that it minimises the game window (which, to me, only makes sense on a single-monitor setup with the game in full-screen mode; OTOH, pointer grab). Good that it has a cvar to control whether this is active.

Alt-Tab works fine when the console's open <em>but</em> not with either chat prompt. So open console, Alt-Tab away.
I'm concerned about the performance impact of this code, computing all supertypes of X every time we form the intersection of X with something else. I think we need to do some tests.
I'm not really sure that we should test for when the function fail. But leave it for now.

 cleaner interface for get_card/0 would be 
```erlang
{ok, card()} | {error, reason()}
```
where ```reason()``` could be ```no_cards_left``` or something similar
I got a fatal error / error about tearDown being protected, so jsut changes it w/o checking why.
I'll change it back and figure out why I got the error instead.
I think you want to use `Utils.insInstantiable` here
I would change the line to:
if(!empty($salutation) && strpos($field_value, $salutation) === 0)

makes more sense than ===true && ==0
@pbeeler No, http works too. It's open source, so https is not necessary for authentication. It's also faster since it is unencrypted.
Using cyan here to differentiate from red, green, and blue used by amplifyColors algorithm.
This is not an appropriate fix. The delete prototype[key] is a fix for IE where for (var key in Array.prototype) did not yield any results (maybe also affects others).
Don't mind this - first pass and all...
Turns out it was a perms issues. I fixed it by removing .npm in my user directory and re-tried npm install and it's good.
- 它們就會被實際經驗之水所充滿，而有了明確的形狀。
- 它就會被實際經驗之水所充滿，而有了明確的形狀。

改成這些概念如何

- 這些概念就會被實際經驗之水所充滿，而有了明確的形狀。
Apart from the remark above, this looks fine, and I am +1 for merge.
it wouldn't hurt to just do `this.element = document.id(element)` without checking the supplied type; document.id already does this for you. Additionally i'd recommend to replace $ with document.id for compatibility.
It's okay, see my general comment below - I'd also swap the init and event, because otherwise there's a tiny race window where you could exit after the event is set up but before git_threads_init is called
Rack in Rails 3.1 and flashes are all it touches, I believe.

On Jan 13, 2012, at 3:28 PM, Erik Michaels-Ober<reply@reply.github.com> wrote:

> Anything to look out for when upgrading from 0.13 to 0.14?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/thoughtbot/clearance/commit/214d1ddb6d806c47102117dcaee07b1f61a0ece7#commitcomment-866080
That's a good thing because we must not have any dependency on the html macro in default XE since we want the user to be able to not have the HTML macro and still have everything else working, see http://jira.xwiki.org/jira/browse/XRENDERING-27
@pmjones  apigen.org
go ask yourself! :P
ну норм. добавил эту правку и со шрифтом (14 строка. font-weight: normal !important;)
The new name sucks a bit..
@SammysHP 
Why did you add this call to finish() in the onCreate()? For me it causes the c:geo calendar process to exit while it is trying to show the dialog to select a calendar which then causes error messages about leaking a window.
If I remove this call to finish() then the new addon works for me.
now that the pendingPrepares are not final, I'm not sure that the "pendingPrepares" reference is correctly published or it needs to be declared volatile for that.
ok, can you reset and `push -f`ed over all my `DON'T PUSH ME BRO`s, can you push your `WIP-new-arch` back?
Isn't there a memory leak in GetStringFromObject()? getInstanceS() in GetStringFromObject() creates a new object which is not decReffed anywhere.

Maybe GetStringFromObject() could be replaced by obj->toString() because many methods will only access th->data. This would also fix the leak. Otherwise you should do obj->incRef() in the (obj->getObjectType() == T_STRING) branch and the caller should decRef the return value (or return _MR<ASString> so that decRef happens automatically).
Added https://github.com/espeed/bulbs/commit/14676f3181c24646cb57c4688309379b84523784
Making the users look a bit prettier!
Lay off the stupid premature optimizations, Luke!
<code>accessible_attributes</code> te devuelve todos los permitidos, eso + <code>slide</code> devuelve un hash "limpio", o sea un hash con sólo los atributos que se pueden modificar por asignación masiva.
Array.new Byte.zero, digits
Hah, I started out with something along those lines, but added the "else" to make it clearer.

In general, looks good -- feel free to merge it to master when you get a chance.
MAKE BCHATMANAGER APILAYER COMPATIBLE. NOW.
Ben c¡est pourtant le minimum car chaque vertice doit être connecté à tous les autres vertices >.< Mais il y a une fonction pour creer un polygone à n cotes? :P Même s'il n'est pas compliquée à faire.
Naaan, mais je testerais peut ètre plus tard, je vais bosser un peu et j'ai une lettre de motivation à rédiger aussi x)
I thought about that too. But I thought this way would be better. What if you want a tree. The branches will be height 5 and the highest point of the trunk would be height 5 too. But the character should be able to walk under the branches but the trunk should be impassable. Another example, a bird files through on height 6 if we want this to happen without overlaying we would have to delete the previous tile and replace it with bird and then repeat every time bird moves. We would also have to do this with the character if we want him to move. And the Level Editor I will make will take care of readability, if we want to edit a previous map we can just load it up and switch between z axises and then display the entire map too.
<b>Chance</b> should be replaced with the proper value:

```DIFF
-(@Bag,20676,Chance,1,0,1,3), 
-(@Bag,20541,Chance,1,0,1,3), 
-(@Bag,20545,Chance,1,0,1,3), 
-(@Bag,20679,Chance,1,0,1,3), 
-(@Bag,20677,Chance,1,0,1,3), 
-(@Bag,20678,Chance,1,0,1,3),
-(@Bag,20552,Chance,1,0,1,3), 
+(@Bag,20676,22,1,0,1,3),
+(@Bag,20541,20,1,0,1,3),
+(@Bag,20545,20,1,0,1,3),
+(@Bag,20679,20,1,0,1,3),
+(@Bag,20677,20,1,0,1,3),
+(@Bag,20678,20,1,0,1,3),
+(@Bag,20552,19,1,0,1,3),
```
Just re-run the CMAKE before compiling as per Machiavelli's request. Thank you for the implementation :)
Fix them yourself? Or *at least* tell us what the errors are.
Hi, 

0.1.3 should be good to go :-) 

beruic <reply@reply.github.com> wrote:

>I can see you are making a lot of changes right now. Is it stable
>enough to use now?
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/flexiondotorg/oab-java6/commit/ebd4bde7c397cea69646624b64d712f1c7101249#commitcomment-874336

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
hooray! thanks. sorry. i'm swamped with info at moment. Will accept. Are you sure it doesn't hose the normal mvn build?
Ter
On Feb 10, 2012, at 12:10 PM, irvingc wrote:

> Hi
> 
> I actually did that at the same time as the pull request :-)
> 
> However I didn't put any reference to the pull request, do you need me
> to resubmit again? I guess I forgot to put the pull request as ID 3.
> 
> Irving Cordova
> http://irvingc.com
> 
> On Fri, Feb 10, 2012 at 10:50 AM, Terence Parr
> <reply@reply.github.com>
> wrote:
>> Hi!  Heh, could i ask you to submit also through click wrap license http://www.antlr.org/misc/feedback to keep license clean? then i can integrate.
>> Ter
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/irvingc/stringtemplate4/commit/7f6bb59859fea868343b8e410243ee7d8a4b1559#commitcomment-954851
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/irvingc/stringtemplate4/commit/7f6bb59859fea868343b8e410243ee7d8a4b1559#commitcomment-955093
Thanks Josh! I started a proper issue thread here https://github.com/thoughtbot/factory_girl/issues/264
Nein! That is a bug. Thanks for catching it. As I said I need to do some more testing on the Capture code. I'll go make the fix.
higher density of object = more force necessary to shoot object
Here is the crash log.


On Feb 14, 2012, at 12:46 PM, Zachary Waldowski wrote:

> Please do! Thanks for looking at the fork!
> 
> Sincerely,
> Zachary Waldowski
> Sent from my iPhone
> 
> On Feb 14, 2012, at 11:00 AM, rolinster<reply@reply.github.com> wrote:
> 
>> I just downloaded this fork and I am getting crashes on both the simulator and iOS device when I run the demo application.  I have Xcode 4.3 with iOS 5.1.  The demo runs for the most part and then crashes usually within a minute.  I can send you the crash logs.
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/zwaldowski/MBProgressHUD/commit/0fd4e4692577104f80a3f1f32ac06920dd8c2895#commitcomment-965083
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/zwaldowski/MBProgressHUD/commit/0fd4e4692577104f80a3f1f32ac06920dd8c2895#commitcomment-965604
This is necessary to serve ONC via http://localhost:8080/plugins/onc/root/
also adding the frostfire orb script in now...
Why a shared_ptr here? This is the only pointer to this d_geturl.
This will steal the `FunctionDeclaration`'s `returnType` in line `1148` because the reference to this `Type` is assigned to a container a second time.

Besides (does not address the containment problem but who implements the type definitions), why do we not use the `WorthwhileTypesystem` here?
s/subtype/supertype
Bist du sicher, dass du hier zu WorthwhileTypesystem casten musst? Dank Dependency Injection sind wir ja eigentlich unabhängig von der konkreten Typsystem-Implementierung. Reicht es nicht, die Methode ITypesystem.typeof(EObject, TypeCalculationTrace) zu nutzen?
This fixes #21, not #20. Sorry.
@jbridson This is exactly the kind of magic numbers dance that we try to avoid with grid variables (applies to the whole commit).
If *any* numbers other than that are necessary, please document *why*.
For example: "Needs to subtract half the button width for center placement".
Otherwise other devs have no chance of maintaining this stuff.
> I am told that the wordy ruby convention for constructing path names avoids trouble on microsoft platforms. Are we putting ourselves at risk there?

The `expand_path` above works on both Unix and Ruby systems on all currently maintained Ruby versions. Ruby knows to threat the slash as a separator, even on Windows, otherwise common lines like `require "sinatra/base"` would not work there. However, this is more of a cosmetic change, whereas the second change should bring (though probably irrelevant small) performance.

The `map` feature is used in Rack to allow more than one endpoint (say serving a wiki from the same process as a Rails application). However, when using with the root prefix and no other endpoint, all it does is check for every request if the path is nested under the root path, which is always true, and hands it on to the controller.

Someone who want to serve more than one application from the same process is likely to write a separate `config.ru` file.
Nice eight you got there.
> Здрасьте, приехали.
А куда, по вашому, денутся переносы строки, нужныя для анализу вайп-детекторами?
Ежели б так все было просто, я б сразу текстконтент писал.

Всё на месте.
![http://i.imgur.com/wBIZ4.png](http://i.imgur.com/wBIZ4.png)
Why the different error handling?  Is all this content provider business just a shim until we land various tab things?
To be quite honest, there may and there may not. It works fine as it on
postgres (and presumably you've tested on MySQL), but I don't know whether
there may be issues with other DBs - e.g. Oracle or MS-SQL. Using the
Moodle abstraction layer should help prevent any weirdness on these.

Andrew

On 3 February 2012 12:40, Davo Smith <
reply@reply.github.com
> wrote:

> Is there any reason for doing this, other than a bit of a tidy-up? (Not
> that I'm against tidying up code - I just wanted to double-check I'd not
> missed anything here)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/andrewnicols/moodle-uploadpdf/commit/3b00b23550a46ea743749aaca7e43c2b37b132c8#commitcomment-930104
>



-- 
Systems Developer

e: andrew.nicols@luns.net.uk
im: a.nicols@jabber.lancs.ac.uk
t: +44 (0)1524 5 10147

Lancaster University Network Services is a limited company registered in
England and Wales. Registered number: 04311892. Registered office:
University House, Lancaster University, Lancaster, LA1 4YW
django-stackato is not required when you use `stackao run` in stackato 0.8.x or later.
Just realized these need to be immutable once set.
just for local feedback :)
Script by Jenova <Atlantiss>
testing still - can't seem to get this thing to show the new blog post and the changes.
É isso então? :P
Where is this attribute actually used?
Sorry if I submitted my update in the wrong way, this is my first time. The purpose is to improve touch response on multiplayer games as well as on other applications by increasing the maximum number of simultaneous touches to ten. My source from this comes from here: http://forum.xda-developers.com/showthread.php?t=891030. Thank you!
Task requirement 5 and use-case's 4th condition are not ready.
maybe I'm missing something, but where does create_results is defined?

while its already committed, isnt it more correct to implement the save method (as create ~= new(attributes) && save)
This is a _very_ good point! This logic may have never been correct... Do you think you could change the logic to check each efv for at least two replicates? I also agree with your suggestion to follow Benilton's advice - we suffer from far too many false positives if anything. Shall we go for min replicates - middle of the road? We'll do an exhaustive test - recalculating statistics for all experiments in Atlas and then run the sanity test to see how many end up with no statistics at all. If the number is minimal, we're good to go (we can perhaps consider unloading such experiments). Otherwise we'll need to see - either way it will be an extremely useful finding about our data - thanks for doing this analysis!
You should ask Dr.K. ;) I just didn't look close enough. No need for quotes, I'll change.
actors.remove(actorRef).foreach(_.apply(actorRef))

No need for the if block or anything

But mixing the produces and consumers in the same map is wrong, they deserve their own maps, then you do not need to store any functions either, just use 2 immutable sets, one for consumers and one for producers.
Thanks, Björn, your version won on the least-convoluted score.
Don't use return
How can ```newScore``` be less then zero?
You can use recover + get instead of the patmat
the method `Set.contains` is one of those which I prefer to write without the dot and parens
Sorry man, must've been tired
I don't get this. why call getRouteDefinitionHandler twice?
close in finally
just joking ;-)
this section is way too hard to read and looks very suspect to me.
todo fix: if ( $result[0] and count(static::primary_key()) === 1 )
maybe just - latest months data at http://data.gov.uk/data/site-usage
(non blocker, can be fixed later) When using the legacy templates the iframe is too short:

http://i.imgur.com/2dnLn.png
It's just for the name to be consistent with other functions in this file e.g. `package_show()` etc. Note that the API endpoint for this will be `/api/action/subscription_show` (does not contain 'get')
I'd personally make sure that both of these section already exist in the config, but at an enabled = true/false field to the section, defaulting both to false, instead of having to check that the section exists every time.
Might be beyond the scope of this PR, since maybe you just moved this code but didn't modify it. None the less...

Perhaps this could be done more simply with:

```
single_line_paragraph = paragraph.replace('\n', ' ')
```
Oh, I think failed_login needs a ! for the same reason as erorrs!
a little weird, but this functionality is changing fast and is hardly used.  So....√
You might as well use "if  (.. == 1)... else" here instead of the case
the phpdoc of the class should be placed just before the ``class`` definition
then it should not be public
The rest of GXM was converted to JSDoc as of 3406c8d9e
I think that's a good idea. With a content property, the developer could insert any amount of text, imagery or markup they saw fit.
Yes,  as you've noted in another comment this is a watch that needs to be run periodically. But, I'm not happy with this watch implementation. In fact this is not directly related to issue #33, but another precaution I wanted to put in. I'm taking the watchdog out, canary initialization takes care of issue #33 already. I'll re-think about the watchdog and add it later.
makes sense, meging
The 1.7R3 branch is incompatible with some of the processors (YUI, dojo). This we are stick with 1.7R2 for now, at least until the extensions module is split in independent submodules.
How about you do YUI and I do Dojo? Seems to be the best way towards 1.7R3, I would have to drop YUI in tapestry-wro4j if wro4j-extensions got split.
Missing new line before phpdocs.
Fixed, although only the top level displays normally
This should be removed. Having to bridge _into_ a `CFBridgingRelease` call is a good indication that you're doing something wrong.
Style: if arguments don't fit on a single line, make them line up, one per line.
actually, we don't need a merge it seems.
Do you mind putting `else {` on the next line to be consistent with the rest of the file?
If you want to prevent that to happen - changing the meaning of `where(nil)` - you can add a special name to the `opts` argument, lets say `:chain`, or even use `false` as a check instead of `nil`. This was done previously with `exists?` 359592bba6cb85b43ca8945917b1ca57fa7494a9. Just an idea :)
aise karoge
I thought usecases generate 'NonTemplateMemberImpl' where I override inDefinitionTemplates. Is itnot the case?
We should probably not throw such exceptions by default (see above), but maybe users might want to chose to throw such an exception. On the other hand, if we aren't using the exception, then it might be best to just drop that class.
I'm not sure anything can be definitively assumed (about anything.) "Let us take it as provisionally true that the following is unquestionably true..."

I can't speak with any authority about reflection, I'm just against the fire-both-barrels-and-pray approach to equality.
Why not name it `Throws` to be consistent with the name of the annotation?
I think this part can be dropped now. IIRC, the distributions now propagate nans, needs checking

>>> stats.t.sf([1,0, np.nan], 5)
array([ 0.18160873,  0.5       ,  0.        ])   #last should be nan
>>> import scipy
>>> scipy.__version__
'0.9.0'
should this just be `a`?
A link to wiki or a paper describing the algorithm.
focusses -> focuses
(Cond)Var->(Cond)Variable Fixed. `valid` is a lambda.


On Mon, Nov 19, 2012 at 7:38 PM, Aaron Meurer <notifications@github.com>wrote:

> In sympy/unify/core.py:
>
> > +    def __str__(self):
> > +        return "Var(%s)" % str(self.arg)
> > +
> > +class CondVariable(object):
> > +    def __init__(self, arg, valid):
> > +        self.arg = arg
> > +        self.valid = valid
> > +
> > +    def __eq__(self, other):
> > +        return type(self) == type(other) and self.arg == other.arg and self.valid == other.valid
> > +
> > +    def __hash__(self):
> > +        return hash((type(self), self.arg, self.valid))
> > +
> > +    def __str__(self):
> > +        return "CondVar(%s)" % str(self.arg)
>
> Maybe add CondVar = CondVariable (which maybe could then be called
> ConditionalVariable) and Var = Variable shortcuts so that they match the
> string values. I don't know what valid means. Is it useful to have it in
> the string value too?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1633/files#r2175773>.
>
>
To->Too
aguments->arguments;
period at end
Why making these changes? If needed, they're better included in a separate PR.
one empty line here would give better readability
e79e07d
CAn be remvoed, it's already beging registered in objectify factory
And then Add comments // Then with a brief explanation.  (this is great to see a test case.) 

We like to add // Given // When // Then with brief explanation for those who follow and iterate code for helping debug. I may also sue // And and brief too...
`calendar.leapdays()` can probably use it instead of this loop.

http://docs.python.org/library/calendar.html#calendar.leapdays
I think  having the availability feature of nydus enabled by default is a good idea. Since using roundrobin I would want to cycle through the hosts until I get an available one.
Should be `$app->finish(`
Replete?
I think the beginning of this paragraph about capped collections and the way the oplog works is the interesting part of this sentence that isn't located anywhere else. 

By contrast, the bit about "within one second," seems like something that we really ought not promise (because it's dependent upon other factors,) and should probably change.
not sure of the context of this versionchanged?
Nope doesn't work :( I have to mark it as safe otherwise it won't work. This is permanent behavior after updating the vendor library. Maybe a playdoh bug?
Maybe you can create a helper for the specific `strftime` format since you're using it in multiple places. If you do, place it in base.helpers which is more generic.
Why user_total_posts should differ from user_posts?
Too low-level for trigger_error. The message does not provide any new information to the caller thus you can simply return false on failure.
This is not going to display anything. If the empty value signifies that there is no custom html, it must be so documented on the interface. Otherwise this implementation does not make sense and should be removed.
This can probably be put in the other wrapper without any ill consequences.
Again, you probably shouldn't be doing this
@sadu well the developer cannot put a comma there without an error in IE. How about "Do not use" just to be clear, instead of using "cannot" or "should not".
> Using your web browser, visit the location where you placed

Maybe more gramatically correct:

> Using your web browser, visit the location where you **have** placed

or:

> Using your web browser, visit the location where you **just** placed
That doesn't exactly answer the question, it then just becomes, why does the previous one?
Why are you silencing errors here?
Trailing white-space
> any drawbacks on a situation when we call cmake . in root dir?

0. CMake (CPack, CTest) produces a pretty big bunch of files.
1. Polluting project root is dangerous (as long as we can override original files with build-time installs)
2. Polluted project root looks awful.
3. git -f clean in not a panacea as long as it can remove other (non CMake cache files)
actually, we want to allow parameters on POST, PUT, and DELETE, if for nothing else so that we can specify accept format as a query parameter for determining the resulting representation of POST and PUT
libutils?
Пробел после запятой. Название методо не соответствует возвращаемым данным
чем именно?
@delallea  I've added new dosctrings which hopefully are somewhat better. typeConstructor is a function that constructs a tensor of dtype float32. Usually it is TensorType (if dtype set to 'float32'), but when this class constructor is called by the ScanGpu optimization, then it is CudaNdarrayType.
maybe a comment on the first two parameters would be better. After reading the code, I understand that these two are mutually exclusive. However, if it's an old catalog, the second parameter bcryptShadowPassword does not mean what the name suggests but it's set to some value.
Ничего не понял. Разъясни.
page-header class already adds a bottom border equivalent to <hr/>
Alright - let's deal with this post-merge then. Are you suggesting that said future patch should depend on app-loading?
removeItem is used to remove a single item, so order isn't an issue.  The remove call in enyo.selection() handles shifting down the indices of other selected items that fall after this one.
This was kind of confusing to me at first as I thought you were describing what the Unit Test is checking for, but really you're describing how the parser should parse what we are testing. 

We could put some heading here like:

The WebVTT specification requires an cue setting align setting to be parsed as follows:
1. If the name of the cue setting is is case sensitive match for "align"
2. If the value of the cue setting is case sensitive match for "start" then let the cue's text track alignment be "start alignment".

We could use this format through all the comments.
I wasn't saying there was a negative consequence. You are waiting to retrieve getAttribute and elem.type until after the nodeName check, which is the fix I was thinking of for an issue with iframes in IE.
Good, I was going to file a jira for this.   I didn't expect to see it in the inferential statistics PR, but I'm happy to see it changing.

We should probably still have a jira to remove the "dev" prior to release.
Support for directories containing spaces!
Thanks, I was looking for the function that doesn't get grumpy when the path already exists.
Read RFC3744.. It's a bit dense when you first get into it, but it's totally worth it to get a better understanding of principals and privileges. ACL was a bitch to implement, but it's really a smart and awesome system.
Can I suggest you use the `vumi.log` module? It adds some helpers for different levels of logging.
Done. Using SkipTest at the module level and squashed into the first commit.
Could this be refactored to use one hash?
Can you use enumerate() here?
This and the above are trailing spaces. Trying highlight left to right. It appears when trailing spaces are removed sequential (line by line) it will replace a block.
doc error, never returns null
Indentation looks wonky.
I changed the signature to be the same as for the `hasItem` matcher (which is also used by `Each` internally).
Constants are usually written `PAGE_SIZE`
JSLint is unhappy about this `else` statement being on a new line.
You could remove the cmdShowDevTools var altogether and using chaining similiar to line 142 with setChecked()
It seems mixed: Firefox uses white on red, but Chrome and IntelliJ keep the text black (albeit in IntelliJ at least the red is quite pale).  You think it would look better if I make the background a bit paler?
Needs a description.
Pretty sure it'll be fine, but I'll doublecheck tomorrow and #dup it if now.
```python
# convert from windows path to segments
```

Comment is a left over from a previous commit. Thanks for letting me know. I've removed it and also changed the above one as suggested.
I was wondering about this.
This is going to be a problem.. 

There are currently already event handles that listen to 'beforeWriteContent', but at the moment, they expect to receive a stream for the fully updated body.. not the partial one.

The CalDAV plugin for instance, will use this to validate newly updated iCalendar files.

I need to think a bit about this..
Are you sure an empty string should be the last url tried?
`n` is unnecessary. fixed.
Qu'est ce que c'est que ce caractère ?
Done. Also what shall be done with java scripts and stylesheets? I changed stylesheet path to load from static. I left google jsapi to load from default URL. Line 13 and 20 are inconsistent. Should I wrap lines 13-17 in a single line or unwrap line 20? If later what indent to use?
Not good, you have overridden parent's default messages. You have to add to them (what if `super(LimitedDateTimeField, self).clean(value)` wants to use some message)?
don't need to do this
Make sure the "disabled" attribute is on this button too, otherwise, you can press "ok" before any data arrives and bork things.
I...... brilliant. Should have thought of that when going over this before.
probably copy pasta cruft.
the above lines need brackets:

`( _container.style.width = options.width || _manifestOpts.width[ "default" ] ) + "%";
m_pParser must be nulled.
Unit test for this?
Remove empty string at the beginning.
Yeah, I wasn't happy with that there given the name. I'm okay with a separate handler especially when looking further, I see that cleardeadlayers disconnects this handler anyway. Will adjust to a more generic name - there is most likely other cleanup that will need to be done and that can go there, too.
Outdated docs before they are even active. Powerdns moves vast :)

Habbie <reply@reply.github.com> wrote:

>> +            <varlistentry>
>> +              <term>tinydns-notify-on-startup</term>
>> +              <listitem>
>> +                <para>Tell the TinyDNSBackend to nofity all the
>nameservers on startup. Default is yes.</para>
>> +              </listitem>
>> +            </varlistentry>
>> +          </variablelist>
>> +        </para>
>> +      </sect2>
>> +      <sect2 id="tinydnsbackend-features"><title>Location and
>Timestamp support</title>
>> +        <para>
>> +The TinyDNS backend has support for location and timestamp. The
>behaviour should not be different from tinydns.
>> +PowerDNS has packetcache and querycache. Therefor it is needed to
>lower the <command>cache-ttl</command> to 0 for 
>> +correct operation with location support. Timestamp support is
>impacted by <command>cache-ttl</command> and 
>> +<command>query-cache-ttl</command>, as the packetcache is not TTL
>aware, which means that it responds with the same
>> +TTL until it queries the backend again. The query-cache might detect
>the same query and not call the backend code 
>
>I think the caches are TTL aware now :)
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/Habbie/powerdns/pull/20/files#r545349

--
Regards,
          Ruben
We can get the path dynamically to a variable, something like:

```
ALCHEMY_DIR=$(rpm -ql rubygem-alchemy | grep -o '/.*/vendor' | sed 's/vendor$//' | head -n1)
```
@iNecas comment was in regards to line 182 and adding default_locale to the validator
I agree - they should definitely have docstrings, although it's probably fine to leave off the `Parameters` and `Returns` sections if you clearly state that they are for use as function decorators.
@taldcroft: agreed, single string I don't like either. The tuples were just a suggestions. The simple truth is: I don't have a preference yet and am trying to see what fits best. 

I think your format has one big advantage: It requires the right number of arguments. Give one angle, it will yell at you. The tuple format doesn't do that without further checking.
Might be user-friendly to make this case-insensitive?
Scipy's fft is numpy's fft should not behave differently, as far as I know.  I can - and have - added a scipy FFT option, but I see no reason THAT should be modifiable at runtime, and I'm not sure how to test it.  I'll come up with some other way to test for 0==0
This now falls back to what was the original behavior only when it has an exception in the `parse_unit` call.  Does this mean we see a performance *decrease* in the cases where the full parsing kicks in (because handling exceptions has a performance penalty)?  I tbriefly ried to test this and didn't see a noticable effect, but I'm not entirely sure I hit this part of the code..
Nit:  if (addr.Set(dest))  entry.push_back()  would be clearer logic.
Is this conditional even needed, as Bind() checks IsLimited itself?
bitcoin.org has no ipv6 right now, and google turns up nothing interesting for github ipv6 migration, so...
Should be `TMP . 'session'` if this was to be merged in.
Should be $plusOrMinus
I tried and failed to not have this loop.
maybe I didnt try hard enough ?
I think that layers implementing `setFloor` should be on their own. We should thus not call `layer.redraw` here.
\callable? or just callable?
i love the term adminable!

this might work... then we dont need add_subgroup.. which feels stranege to me.. this way its just creating a group with or without a parent group. What do you think?
can[:create], Group, do |group| 
  if group.parent
    group.parent.administrators.include? user
  else
    # i guess anyone can create a top level group right?
    true
  end
end
Is there a reason to use a list here? Does ordering matter?
I would be fine with it being optional.  I like the idea of passing a dict
of options to the cell constructor.  Let's create a default options dict
inside the constructor and then use jquery's extend function to merge the
two:

http://api.jquery.com/jQuery.extend/

Cheers,

Brian

On Tue, Oct 2, 2012 at 1:53 PM, Bussonnier Matthias <
notifications@github.com> wrote:

> In IPython/frontend/html/notebook/static/js/codecell.js:
>
> > @@ -210,10 +211,33 @@ var IPython = (function (IPython) {
> >      };
> >
> >
> > +
>
> Oups, I wanted to make it optional, forget line 44, I'll remove it
> tomorrow. It is useless if you don't want continuation prompt.
>
> My though was to add a dict of option to cell constructor and eventually
> forward the relevant one to codemirror.
>
> Would this be more acceptable ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ipython/ipython/pull/2421/files#r1743775>.
>
>



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
I think they should also have a `__repr__()` method which just returns the path, so that it does something useful in the terminal as well. Or if we want to be clever, make an absolute path as a `file://` URL - at least in my terminal, those become clickable links automatically.
IMO, when there's no extra logic, map is still clearer than a listcomp (it's also faster):

```
chk = map(max, _chunks(rlist, nrow))
```
It looks like this would cause a `KeyError` if the device is 'X11' - should it be protected somehow?
perfectly fine to have this test here; just a side note that you can also/instead test exception parsing in the expect tests
return Optional.absent()
I don't think there should be special cases for e != ee here.  Even if it is the identical edge object, it can have different weights assigned in the two graphs.
ok... but then why do we have the page set "FramesetSupport.PlayerCssProperties"  ?.. .Should we not check the page for a frameset and append that style to target embed element if on those platforms?
This is some seriously weird-ass code. This is calling for a rewrite; base64 encoding is a simple operation, it shouldn't look this weird. To begin with, drop the `outlen` parameter: we know the final size beforehand, so we can enforce the user to pass a buffer of the appropriate size.
Carlos is right, make this `_new`
the comments are nice but they might be too verbose to maintain

how about
A section with a body (  a hanging begin section tag, ws*, the close of the begin section tag, body or bodies, end tag  )  OR
A self closing section tag (  a hanging begin section tag, ws*, \}  )
This should be cached. The django cache framework is really easy to use:

https://docs.djangoproject.com/en/dev/topics/cache/?from=olddocs#the-low-level-cache-api

You should be able to just do `cache.set('devices', devices, 600)` assuming `devices` is the result of parsing the CSV file. That sets the time-to-live at 5 minutes (600 seconds).

Then do `cache.get('devices')` to get the result, and it will return None if it doesn't exist, and you should set the cache then.

Make sense?
This test's name implies it tests sorting, but it only tests sorting by helpfulness.
This comment isn't clear to me. What would it even mean to renew "the responses fetched by self.app"?
an accidental slash
you could add this selector [above](#L0R369)
You are right, I didn't notice that. The reason I specified both is because we are interacting with both components directly. I think it's ok as is, possibly more future proof.
This should be before the request body, not after.
The name of this method is a lie; it does its work in the current thread.  Should rename to 'callbackInvoke' or push the threading into the method.
це костиль
hmm, actually, depending of the way you do it, it could make sense to keep this variable, and to add them in the matcher and the generator lazily: ``addRouteHandler`` would add the element in the array, and proxy the call only if the matcher and the generator are already initialized. Otherwise, it would be set only when initializing them.
Sh** I thought it would went unnoticed ! Any idea how to test in a simple way ('stack' should not be tested)
Can you move all of this code to an abstract sub-class, ``NumericNode`` for example?
That's a good idea, @amshali. That will remove the current photo-handling code out of the news template and into the Photo model.

Post probably doesn't need to be abstract, @ProCynic, since most posts simply output the `content` string. It's the special subclasses, e.g. `Photo` and `Checkin`, that require a different output.
There's this: http://wiki.worklist.net/wiki/index.php/Coding_Guidelines
It has a tiny paragraph about block indentation. What's the style you prefer?
大致同上，只需要\[ \]即可，没有任何强调或者其他，这两段内容一句话基本可以概括
It is slow. There may be a better way, but I haven't found it yet. The reason for this change is because the ShaderMaterial doesn't support size the way ParticleBasicMaterial does. Instead, it is implemented in the shader. I'll do some more digging.
Ik zou geen MVC gebruiken (geen controller). Controllers zijn goed voor afhandelen van requests. Aangezien dit geen online applicatie is bestaat er niet echt zoiets als een request. Misschien even kijken naar Model View Presenter:
http://msdn.microsoft.com/en-us/magazine/cc188690.aspx
http://www.cerquit.com/blogs/post/MVP-Part-I-e28093-Building-it-from-Scratch.aspx
Better check somewhere that the type of A is float!  Or better, let's go for our new strategy which Larry has just laid out in over(): create a templated implementation, but have the public version of histogram() just call the version which deals with float pixels.
Can create_function() ever return null?  If yes then this needs a check; if not the variable "bool ok" below has a redundant test.
Would have been nice if he actually used my getPing method in VanillaPlayer... :p
just so it doesnt conflict with Graph
We can even drop the trailing "nodes", otherwise it reads "Showing 4 nodes (filtering from 10 total nodes)"
The key point in my previous comment is "if something is holding the classloader for this class in memory." I don't mean that is the case today. Probably if that situation arose it would be a bug, although not necessarily. The main point is a thread pool whose lifecycle is not properly controlled increases the fragility of the system. And this is such a thread pool.

A ThreadPoolExecutor stops its threads when the shutdown() or shutdownNow() methods are called. Those methods are not called in this patch. That means the only time they are called is as part of gc via the ThreadPoolExecutor.finalize() method (which calls shutdown()).

Agreed, we need a discussion on a general thread pool facility. What its characteristics should be and how it should be exposed.
This attribute allows expressions, but there is no way to persist an expression, since in the xsd the element is "booleanPresenceType".
Hi Jonas,

I have a more elegant and simple solution, leaving it totally generic:

1) Changes on getScaleFactor can be rolled back (it is no more necessary to cap limits)
2) onScale must be changed in this way:

public boolean onScale(ScaleGestureDetector detector) {
	double center = viewportStart + viewportSize / 2;
	viewportSize /= detector.getScaleFactor();
	viewportStart = center - viewportSize / 2;
	
	// viewportStart must not be < minX
	double minX = getMinX(true);
	if (viewportStart < minX) {
		viewportStart = minX;
	}
	
	viewportStart + viewportSize must not be > maxX
	double maxX = getMaxX(true);
	double overlap = viewportStart + viewportSize - maxX;
	if (overlap > 0) {
		// scroll left
		if (viewportStart-overlap > minX) {
			viewportStart -= overlap;
		} else {
			// maximal scale
			viewportStart = minX;
			viewportSize = maxX - viewportStart;
		}
	}

	verlabels = null;
	horlabels = null;
	numberformatter = null;
	invalidate();
	viewVerLabels.invalidate();
	return true;
}

This makes the viewportSize calculation directly dependent on the scaleFactor gathered, so I guess that the setViewportSizeLimit now is useless.

Let me know your thought and tell me how can I put these changes here :-)

Best regards,
Marco
I think this would be "edit state" in proper English (same for all other states).

Would you document these `data-state` values please? They’re not as self-explanatory as you could think.
same comments here. I would suggest to name variables from dialogs without referencing the related functionality, maybe that helps to make variable names shorter
Yeah! I'm happy to hear that!
I really think you can get away by using a regular `mozNotification` here and get a ton a functionality for free :)
I you have callback issues, the bug if known and you can use https://github.com/mozilla-b2g/gaia/blob/master/apps/dialer/js/notification_helper.js
does this code works if I manage to click on the li itself instead of their children?

@crdlc any opinions?
I did this to put banner on top of the sprite animation. We will not use this piece of code in System app to show banners in any app, so it doesn't really matters.
Use - instead of camel cases to match the other css files of contacts.
Why did you also remove the data-action attributes?
@etiennesegonzac Why we were using `var number = message.imageURL.split('?')[1];` ? Could we use `message.sender`? I thought it was part of the 'icon hack'...
It's an indentation typo. Pushing a fix :)
I knew this was wrong when I did it :( I punted because I'd been failing at testing the redirect chain. I'll take another crack at it.
The other instance of the applicationFilePath problem above.
I notice functional tests are not run as part of build.cmd. Is that intended?
I *think* the convention being used here is that if the block return value is being used, `{}` will be used as delimiters. I might be wrong tho.
ok .. ^^
`maximum`?
Generally don't like 'continue', flip the condition and move next line in instead.
Catalogs link to catalog by default but without appending '- Catalog' to the title. This also unfucks /f/ which doesn't have a catalog but is linked by default anyway.
Had an extra return statement mixed in, edited.
the link should be fixed
Wouldn't this remove the default LibraryInstaller too ?
I thought about this on the way to work – so for the sake of clean code we could add a `dumpFile()` method to the interface which tar and zip use instead. And `dump()` could then throw a domain exception, and vice versa for the `ArrayDumper`. Just not sure if this is really worth it to be totally honest.

I think it may go over the top. ;)
blank? nil, empty string, whitespace or else?
Use the config module instead of doing it by hand. It allows per-environment overrides and is already included in Uptime.
Actually, this was deliberate. :)
I explicitly try to avoid the ternary operator in Ruby ... I think they are bad style.
It generally hurts legebility, especially if you have a `foo?` call or handle `:symbols`.

I might agree on the use of spaces though. ;)
indent: looks like 5spaces here instead of 4
\n ?:)
Ok, thanks! Since I have never worked with relative imports, I hope you don't mind me asking a probably somewhat noobish question. When I change this line to the one that @dwf suggested, I get a ```ValueError: Attempted relative import in non-package```. Should I wrap this in a ```try ... except``` statement as suggested in [this post](http://stackoverflow.com/questions/3616952/how-to-properly-use-relative-or-absolute-imports-in-python-modules) on stackoverflow? I'd rather not do this, but then I can't run the test suite in my local git repository, and I'd prefer not having to install the package just for this. Any suggestions how to best deal with this?
this test should mention the textarea usage in name imo.
Actually, that's the main idea behind this fix.
There is two windows involved here.
1/ the one already opened before running this test `browserWindow`
2/ the one opened and closed in this test `browserWindow2`

My idea was that this test might fail for some unexpected reason, before this patch,
when we tried to reset addon bar visibility just before trying to close it.
And we do not care much about this final state of this temporary window,
but we want to reset to original window state, i.e. `browserWindow`.

I may have to add some helpfull comments? Suggestions?
Please move `require`'s to the top of the file.
I'm wondering if this code should be moved to boostrap as it will adds more flexibility and control over the manifest structure.
So that we can change the manifest and this particular code easily at the same time.

A particular revision of cfx generates a particular manifest format.
If we decide to change manifest format and this code lands in firefox, we will have to pass a custom version of `resolve` and stop using this one. It will end up being a dead code.
"handle" and "cover" both sound very generic to me. As I said yesterday on IRC, the best I can think of is "attachTo". I like the way it reads like a complete command: 'attachTo:["existing"]'.
*awesome* updates to test coverage.  I did a cursory sampling, but love how many new areas we're testing.
This is accurate to what they were shown (instead of us re-parsing accept-language headers.
Why only the window and which window? The body element can absolutely have an onload event handler:
http://www.w3schools.com/jsref/event_body_onload.asp
would you mind using default names for properties here? It would be better portable to other projects then.
If this is an objection, perhaps the password for 10.8 could be formatted as "iterations:salt:entropy" or the like.  That would actually make operations faster, since we'd read/write the plist fewer times.
Cool how you kept this all to a fixed column width. Very legible. I dig.
@jmikola yes, looks good. And yes, agreed that the append behavior should be optional, but default to enabled.
I split the if/else branch into two protected functions: eventIntoColumn (array $event), mapEventIntoColumn(array $event, array $columnMap = null) . Very good suggestion.
Only if there are params