labofsahil/github-event-dataset-2012 huggingface dataset:
you can set a functions name using fn.__name__ = 'foobar'. in the decorator, you can set the resultant fn's name to the decorated function's name.
I thought we agreed that we were going to make this simple for users - and make this an enum, containing ALL, SAME_MACHINE, SAME_RACK and SAME_SITE.  So that people can pick TaskExecutionPolicy.SAME_RACK, for example, rather than having to implement that logic all the time in an interface.

Besides, the interface you've designed doesn't give the user enough information to decide which addresses are in which datacentre, etc.  :)
See screenshots in https://issues.jboss.org/browse/ISPN-1999
the StatementException gets logged on the server side. although I'd rather log it only as a one line WARN message, the problem is then how do I return erroneous result without throwing an exception ? Shouldn't we have something like isErroneous() in Result interface ?
I think the "No cache selected yet." message should be output in red, because it's not a name of a cache.
Ah, so it is.  Will fix this also.
I understand a foolish consistency is the hobgoblin of little minds, but shouldn't we stick with the normal setters and getters rather than introducing properties, so this will be exposed to the whole artist set/get introspection?  It seems like it would add confusion to have some properties sets as plain-ol-attributes-or-properties, and some as setters/getters.
Picky I know, but PEP8 encourages spaces between operators (i.e. ```if self.codec != 'None':```).
is this ok?
do we want this class to contain this method that sorts a specific area?
No I'm leaving it here otherwise the Calculator app may not be killed. if any step of the way fails then we need to leave the device in a clean state.
The test does not kill the app.
Just to add, I'm certainly not trying to be picky, I just know a couple of people who *hate* not having the option to *not* use a dynamic var. :p
We should write this up somewhere in the style guide then, since e.g. in Layer.Grid we have onLoadEnd, I don't like having multiple styles for this.
Great idea! I'll do that. 


On Nov 6, 2012, at 11:44 AM, Dan <notifications@github.com> wrote:

> In src/clj/forma/utils.clj:
> 
> > @@ -251,6 +264,10 @@
> >       ;=> (-1 -1 -1 3)"
> >    [bad-val coll & {:keys [default all-types] :or {default nil
> >                                                    all-types false}}]
> > +  {:pre [(if (or (nil? bad-val)
> This is a pretty hefty precondition, and it is used more than once. I might use a separate predicate function. It's a style preference, but it's common -- see the comments in this post. Here is an example that probably won't work as written, but can be used in the precondition:
> 
> (defn critically-bad?
>   [bad all-types full]
>   (let [total-nil (count (positions nil? full))]
>     (if (or (nil? bad) (pos? total-nil))
>       (false? all-types))))
> —
> Reply to this email directly or view it on GitHub.
>
Just a general comment: Can we stick to lowercase dashed for this stuff?
Compass built-in mixins have this naming convention, so makes sense for us to use it as well
looks good. you can remove that {1,1}, i think that's the default behavior.
not sure if those quotes shouldn't be html entities (e.g. &laquo;), what do you think?
to 0xffffffff bychom možná měli nahradit konstantou. Je to i v aktuálním materu, i když to komentuju tady.
I've added a section about tombstones to the LSM page.
My reading of the manual is while your request is executing umask stays at whatever you set it, and while this is going on all other requests in other threads are using your umask.
To be consistent with the other method names, what about renaming it to just `middleware`?
有關圖檔的連結, 可以這樣用

```haml
= link_to( image_tag(asset_path("footer_follow_fb.gif")), "#", :alt => "facebook 粉絲專頁", :size => '145x21')

```

這樣內容只需要顧及 app/assets/images/ 下開始的路徑就可以了
Are you sure ``new \Zend\ServiceManager\Di\DiStrictAbstractServiceFactory($di, \Zend\ServiceManager\Di\DiStrictAbstractServiceFactory::USE_SL_BEFORE_DI)`` has the correct type as "object<Zend\ServiceManager\Di\DiStrictAbstractServiceFactory>" is not a sub-type of "object<Zend\ServiceManager\AbstractFactoryInterface>|string"?
Bundler allows a certain group not to be installed with `bundler install --without=`. 
For example, `bundler install --without=development`, won't install all the `development` dependencies. 
As well in `production` environment it ignores `development` and `test` dependencies at all.

Please, let me know if you still would like to remove those debuggers dependencies. All the changes placed in b12050dbf3c058608ab751eb907ac549613c90f8 commit.
It should be easy to remove with `git revert b12050dbf3c058608ab751eb907ac549613c90f8`.
This could be just `open .`, right? Just wondering if I’m missing anything here. :)
I think we should keep the class names in line with webassets, i.e. without a prefix.
Please add a comment indicating that this method is only present to make Ant's JUnit test runner happy.
This is not correct way to do it. Tastypie supports [authorization and authentication](http://django-tastypie.readthedocs.org/en/latest/authentication_authorization.html). You should use that. So define a authorization class allows access to only user's notifications.
I am not really sure this works. `None` is defined in Django templates?
You can add bottom margin to form? ;-) E-mail verification form will be only one. ;-)
Ok I have one question. 
Django has all attributes in one list (query_terms).
But in MongoEngine we have operators, geo_operators, match_operators, custom_operators which are all local variables. Making all global is not the best idea I believe so how should I group them? Now I have a dictionary like this: `QUERY_TERMS = {"operators" : [list], "match_operators": [list]}` But I'm not sure if this is ok.
Always use HTTPS versions. What if site will run under HTTPS.
Translatable strings!
That's so weird! Labels/fields in iOS don't do that.
Not in the trunk for now. But it will help tests use the deep learning tutorial for speed comparison between c and gpu code.
Do radio buttons need the `Content` set? 1,2,3,4 doesn't particular mean anything to the end user.
Use `ENV.cc` instead of hardcoding cc here.
yes it is string starting at index 2  skipps "a." as index 0 would be on "a", and going for 15 chars. From a quick goole and test.
Please remove this phpdoc, it's invalid.
I added a documentation snippet from the official xml spec: http://download.chasepaymentech.com/docs/orbital/orbital_gateway_xml_specification.pdf
It would be quite nice be able to specify the lable for submit for default form_buttons (src/lib/simple_form_extension/form_buttons.rb). Like ```:submit_label => "something"```.
This looks like an optional parameter to me.
Instead of redirecting to unnecessary view, just render a template with intermediate page.
This line should remain as is because it's calling the method on the pages object and not the fuel_pages model
Yep, I know, see the comment above with the UI first, then functionality. Thanks!
Symfony does not use the ``@var`` annotation on private methods, and we don't use it either.
simply add the tag on the listener instead of searching the event manager and adding the method call by hand
use `@deprecated` :)
Why is _hibernate.dialect_ a required property?
So I assume this was broken up because Engage/People API has different options than the Tracker API? It doesn't have an event time or a test flag?
nit: Failed to stop the Selenium Grid hub
I don't understand this change. Why do you want to force a default disk write cache of 16MB. Optimally, the cache size is set depending on the amount of RAM on the machine (Amount of RAM / 8 is a common value). By setting it to 0 by default, libtorrent will determine the amount of RAM and set an appropriate value (And fallback to 16 otherwise).
Sorry for the sloppy commit. I couldn't do the change/commit properly and had to use the web interface. Mistakes were made; hearts were broken; lessons were learned.
also metal @rsanheim-free
I like the environment line to be the first of a settings file so dont getting confused between them
Please be more verbose. Users should know what happened.
Good, but please name variables the same as it is in Django so that it is easier to understand what they are:

```
var language_code = '{{ LANGUAGE_CODE }}';
```
I think this, and the event logger should actually be injected. We need to hit this in the refactor.
`startit` is a very vague name, please revise. You should also separate words in a symbol name using underscores or camel-casing.
I guess this raises a question as to what units we actually want `added_flux` (and thus the quantity returned by `drawShoot`) to be in.  I had thought, once the gain was included, that `added_flux` would also be in ADU... Is there a reason for it not to be?
Can remove this @brief.  Might be worth grepping for more of these.
That seems plausible (and quite parallel to C++).  If Jim doesn't have a chance to reply given his limited time for GREAT3 this week, I"ll just give it a try...

On May 7, 2012, at 5:09 PM, Mike Jarvis wrote:

>> +               in case it is not located at the center, which is the default assumption).
>> +    @param[in] strict If True (default), then there will be a run-time exception if shear
>> +               estimation fails.  If set to False, then information about failures will be silently
>> +               stored in the output HSMShapeData object.
>> +    @return A HSMShapeData object containing the results of shape measurement.
>> +    """
>> +    gal_image_view = gal_image.view()
>> +    PSF_image_view = PSF_image.view()
>> +    result = _galsim._EstimateShearHSMView(gal_image_view, PSF_image_view, sky_var = sky_var,
>> +                                           shear_est = shear_est, flags = flags,
>> +                                           guess_sig_gal = guess_sig_gal,
>> +                                           guess_sig_PSF = guess_sig_PSF,
>> +                                           precision = precision, guess_x_centroid =
>> +                                           guess_x_centroid, guess_y_centroid = guess_y_centroid)
>> +    if (strict == True and len(result.error_message) > 0):
>> +        raise RuntimeError(result.error_message)
> 
> I think what you want is at this point:
> ```
> try:
>    result = _galsim._...
> catch HSMError:
>    if strict == True:
>        raise
> ```
> Jim might correct me if I'm wrong though.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/133/files#r784370


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
Yeah, there are a few ways to pass along warnings to python users, but I'm not sure what the best one is in this circumstance.  But now that you mention it, it might be kind of interesting to have both ```draw``` and ```drawShoot``` return the fraction of the flux outside of the ```Image``` argument, so errors can be handled at the python level as you suggest.

This is clearly beyond the scope of this initial pull request.  I can open another one so we can discuss this elsewhere.
It's the same code as for AiryInfoObs.  I didn't repeat the derivation here, since it's the same thing, but your probably right that it's better to include it here as well.
It seems, that name is wrong for this method.
There seems to be some typos and half sentences here. I do not understand the content.
Thanks for the pull request.  Nice to do in a more portable way.
We all use spaces instead of tabs.  If you can resubmit your pull request with spaces, I'll pull that in, otherwise I'll take your suggestion and add myself.  But would prefer to pull your change in. :)
same comment here as the other
nice funny test :)
should caller be able to specify quarantine group?  seems potentially odd it being a child of this cluster.
(but fine ... this could be an enhancement)
Does this line really need to be moved up as well?
Instead of:

```
@expected_on.map(&:inspect).join(', ')
```

You can use:

```
to_sentence(@expected_on)
```

Check: https://github.com/frodsan/mongoid-minitest/blob/master/lib/matchers/helpers.rb#L4
Alright, though you could lust leave the PR open and update it. Preferably along with amending commits and force-pushing.
Erro de digitação: ``comum`` ao invés de ``comun``
It's part of new infrastructure in the Java SDK test suite that adds a new key to .splunkrc. If you don't have it here, you can't use the same .splunkrc for both. I've added a comment about where it came from and why it's not in the README.
Another little typo here.
I think it's better to fail explicitly than pretend everything's fine
From what you're saying, I gather that `git_vector` is probably not the type you're looking for. For your use case in the iterator code, you need something that:

1. Needs to be allocated in the heap, because it'll be contained inside a `git_vector`.
2. Needs to store integers, and not pointers like `git_vector` does.

This screams for a rewrite. Possibly store a linked list of integers inside the vector?
`5.4` ?
This is the exact kind of function that could some some detailed documentation. What does it do? ^^
The exact expression can be discussed, but characters that fork processes in the shell must be filtered out
Seems odd to have lowercase here, can we make this gramticaly correct? Trivial ass-hat request for sure. :heart:
Needed to force exit on this one
И здесь табы
Это нормальная ситуация.
Нет. Надо проверить, что это именно то выброшенное исключение. Тип совпадает (т.е. equals) и сообщение совпадает.
What are these for? Adding new dependencies will require partner teams to be notified.
You removed the trailing slash in the readme and the implementation, but not here. Did you actually run the code?
with the change of method name it makes a lot more sense. silly web design!
updated  and added validation check
Use the singular form here, to be consistent with `sample_weight`? cc: @pprett 

(I prefer the plural form though.)
I think documenting the parameters would be good, even though this is an internal function. same goes for ``_graph_is_connected``
It is a better way to do this; I  just didn't know about and didn't find it in the numpy documentation. That should make a difference on bigger datasets than the one I use right now.
Also, I think structured array is a new feature from numpy1.7, so I can't use them anyways.
Is there a reason you use ``calloc`` here which zeros out the memory, since you're just going to overwrite it a couple lines later?
Hrm, but we *are* talking about headers, are we. Have you seen a HTTP header with 5 megabytes of data?
**Disclaimer:** I'm 99% sure I wrote this example correctly but it's not yet tested. jsFiddle coming soon...
looks like a whitespace mistake
Add reference to obj before releasing width_ in case they are the same object
Appcelerator coding standards state that you must use == or != operator, i.e.

if (height_ != NULL)
getTypeIntersection.  It currently always returns a dataset.
Just my lazy. Fixing up.
CS: variable name should be camelCased
Hrm. So no multiple versions for CoffeeScript, then? Could be a source of confusion.
One of important aspects of connection strings is the error reporting for users, in the storage .NET SDK they verify `accountKey` by this:

    private static readonly AccountSetting AccountKeySetting = Setting(AccountKeyName, IsValidBase64String);
Hmm, yeah, geolocation is likely in the same boat as navigator.connection. It'll probably work just fine to put it in the clobbers list now.

inappbrowser will only work in the clobbers list so long as you export the open method and export only the open method. That's what you're doing though, so might as well :). Not a big deal either way.
J'aime pas trop ces noms de classe parce que tu confonds nommage et implémentation.
la classe ou tu mets un padding standard, tu devrais l'appeler .padded(comme ca on introduira un .padded-big ou .padded-smal pour d'autres valeurs de padding).
I don't think you want a "global" modifier -- you only need the first match, right?
`(not (empty? ...))` might be a better check.
Meh, freshdesk is sort of working. Has some issues. We could send to ``settings.MODERATORS`` instead and then point that at support@mirocommunity.org. But I don't think it's necessary. Gmail filters can properly split the error emails off from any others.
You don't need the text "Splunk Web -", apparently it's repeated.  Example:

	[-] 10.0.1.8:8000 SPLUNK_WEB - Splunk Web - /en-US/account/login - The connection timed out (10.0.1.8:8000).
Yes, it's difficult to tell intent of what you want data.size to be, and figuring out what you ought to get requires a lot of deep knoweldge of how appending an Integer to String works.
if size != ni_packet.length
These two checks can be combined like so:
```ruby
unless (1..254).include? datastore['UNIT_ID_FROM']
  fail_with(Exploit::Failure::BadConfig, "Unit ID must be between 1 and 254")
end
```ruby
We don't print to stdout, use print_good
Only name it "Universal" when you've actually tested every platform supported by the vulnerable software.
version isn't needed anymore
Looks like some of these files have a CRLF issue.
For the first one, I have to have a look.
For the second one: Felzenszwalb is the name of a person who published many algorithms. SLIC and Quickshift are the algorithm names. I could also call it "fast_graph_based", which is a common name.
I'll add another PR for this soon.
Can you explain a little what's going on in this block? It's a bit clever.
else should be on the previous line
Why not `Matrix4.IDENTITY.clone()` like we do elsewhere?
Probably `0.0` since I assume we intend floating point.
@shunter does your promise system for image loading have an impact on this class?
Should we also have a prototype version of this for consistency?
my previous comment may be a possible refactoring for solving this.
Since we decided to provide users with an initial chassis shape, we should disable `SketchingHandler` all together. `SketchingHandler` is designed to allow users sketching the chassis, not for the system to initialize a default chassis. Please move this code to other places that make better sense
Can this be a script CFC?
https://github.com/symfony/symfony/pull/3416
you can also get the scheme from the request
no need to continue the loop when modal window is found. add **break**
Not implemented on IOS
use require.on instead of addEventListener
Missing note for localstorage: DONE
http://requirejs.org/docs/optimization.html#setup. It would allow us to use AMD module loading and ship the code in a single file to work with non-requirejs projects. However, it's a pain to set up and feels like a premature optimization. 

Let's focus on the actual API for now and then worry about this a bit later.

In regards to tictoc, I'll embed it into Utils.
typo: his -> this
The way I wrote it yes, but really the line should be:

    ExecutorService executor = Executors.newFixedThreadPool( SIZE_OF_THREAD_POOL );

Going to change that.
That means we shouldn't do this.

The way we generate keys is:

* Observe that the server has no keys
* Generate and upload some keys, but don't store them locally
* Download keys from the server.

This way we're totally protected from race conditions: we *never* end up with keys locally that we didn't download from the server. Y'see?
I think "into" is the correct wording like "Save data (into / to)  the database"
No comma needed after the is.
You no longer need the `is_null` checks because you're doing them up above (good call)!
Na prática, você apenas tirou a action imagem, não é?
Missed this one. :)
Why do these have no recipient?
That would probably be possible but I don’t really like that solution as it will change the language of the node for the time it takes to render the page which I think is not acceptable. I would prefer a cleaner solution, i.e. get the English translation without switching the node’s language around. However, I have no idea if that can easily be done.
//???
you should use [``implementInterface``](http://fr.php.net/manual/en/reflectionclass.implementsinterface.php) instead of using an ``in_array`` check in PHP. The Reflection code is probably optimized for the lookup.
That would only work in PHP 5.4+. Shouldn't it work on PHP 5.3+ as well?
why are you remove this?
Have an example (URI)?
There essentially isn't anything that cares in the codebase AFAICT, but I
could be wrong.  I decided this made more sense.

--benkraft

On Wed, Nov 7, 2012 at 7:58 PM, Jordan Moldow <notifications@github.com>wrote:

> In esp/esp/program/modules/handlers/teacherclassregmodule.py:
>
> > @@ -736,6 +736,7 @@ def makeaclass_logic(self, request, tl, one, two, module, extra, prog, newclass
> >
> >              # Now that we're done putting together multiple resource type sources, listify!
> >              resource_types = list(resource_types)
> > +            resource_types.sort(key=lambda x: -x.priority_default)
>
> That's fine. I just wasn't sure if the higher number is the higher
> priority (lower numbers are higher priority with the lottery), and I'm too
> lazy to look into the code to find out if this is the case (or if the code
> currently doesn't make a distinction between higher and lower priorities).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/learning-unlimited/ESP-Website/pull/435/files#r2063893>.
>
>
In theory, this could be ported to MongoDB, etc. If we go by the Common persistence interfaces, DoctrineObjectManagerTypeDriver would be correct. Real mouthful though. :)
You'll run into problems here if ``DateInterval`` is the root object, e.g. ``$serializer->serialize($dateInterval, 'xml');``. 

Could you mirror the implementation of the DateTimeHandler here, and add a test for this?
if wrapped in parens like this, the start of this string should be bumped to the next line (so all that's on the first line is the open paren). otherwise it violates the exactly-four-space-indents rule
hah.  good point.  removed.
missing u in user
Introduces too much risk at this point. Created SAKIII-5100 for this.
Really any existing tag will be used if it exists, doesn't matter if it's product-specific or not. Also, formatting (single-line summary, etc)
I believe I've fixed these all over the place now.
I know I'm being nitpicky here, but that @@@ is awfully ugly. How about a straightforward "Bugzilla 8675" ?
These asserts help when porting. The idea is to get errors as early as possible when something is missing for a particular platform so it doesn't get overlooked. When a set of functions/types aren't present on a platform we can simply add an empty version block for them. That does mean 3 redundant lines for such platforms, but it'll help in the long run, since we don't have to go hunting when something is missing.
Right, so may be instead of going the two context approach we can simply check if expression string begins with '@' and throw IllegalArgumentException. 
However this approach would have to be constantly maintained in the event if SpEL evolves and provides other abilities to access AC data (beyond @)
I actually think there is still an idea discussed on yesterday's call that has not been implemented. It may or may not make the questions above irrelevant, and it may or may not have a vulnerability that I'm overlooking, but it's probably worth trying and running through the full concurrency tests...

When relying on the JMSMessageID being returned in the reply's JMSCorrelationID (i.e. the null correlation-key case), if TemporaryQueues are cached within the JOG and keyed by the Session identity - such that any cached consumers used within that Session are scoped by it as well, then any one such Session-bound consumer can only receive the reply it's waiting for OR a previously timed out reply that same Session had initiated a request for earlier. So, it can rely upon setting its own *exclusive* TemporaryQueue in the JMSReplyTo property of the request and then waiting on that queue for replies *without* any MessageSelector at all. Once a reply Message does arrive it can then *verify* that the reply's JMSCorrelationID does match the JMSMessageID of the request it had sent (else log a message and loop back to attempt another receive). It does not even need to wait on that JMSMessageID *before* creating (or retrieving from the cache) its consumer since it can grab the JMSMessageID after sending but before comparing against the JMSCorrelationID property of the reply Message that is actually received.
Don't these need to be ConcurrentHashMaps? If destroy() is called at the same time as a new thread invokes doSendAndReceiveWithTemporaryReplyToDestination, destroy() might get a ConcurrentModificationException.
It doesn't matter. This is the method that returns keys and the prefix is an internal detail so propagating it to the iterator() means I am passing the wrong keys even if its harmless.
I'll take a look; but why didn't the inbound use that technique?
Perhaps an autoloader could help.
プロジェクトの初期設定がよろしくなかった。
there is 2 semi colon here ;___; , other than that it looks good...
@bp323 Aren't we only using prettify on pages with an lhnav, so can't we discard these changes as well?
also see my comment above in regards to moving this class out of util-core
`eligible()` is supposed to mean something here ... as in "do we have enough information to reply?"  Since you are building the `reason` property in the `consume()` method, you might just want to return true or false based on whether the `reason` property has been filled.

The normal pattern for using a patient snapshot is as follows:

```
for (Obs o: obsList) {
    if (snapshot.consume(o)) { 
        // consuming the observation changed a property
        if (snapshot.eligible()) { 
            // the snapshot has come to a conclusion
            return snapshot.generateResult();
        }
    }
}
```

This requires a `generateResult()` method on the snapshot (which we are not doing yet) to return a formatted `Result` object.
I could go either way. I'd be happy to change this though, what do people think?
Expecting to fail. Failing if you fail to fail! Oh, what a tangled web you weave!

Also, not a big deal, but PEP8 and pep8 both want to see two spaces before the `#` of an inline comments
yeah, the music didn't start the second time, because nextMusicInterval was not reset. so in tick() no background music was started.
Only update NodeTranslations when no weight is set.
It should... That is the beauty of ARC

~Randall

On Dec 20, 2012, at 4:28 PM, Carlos Figueira <notifications@github.com> wrote:

> In test/iOS/ZumoE2ETestApp/ZumoE2ETestApp/ZumoLogUpdater.m:
> 
> > +//
> > +//  Created by Carlos Figueira on 12/17/12.
> > +//  Copyright (c) 2012 Microsoft. All rights reserved.
> > +//
> > +
> > +#import "ZumoLogUpdater.h"
> > +
> > +@implementation ZumoLogUpdater
> > +
> > +-(void)uploadLogs:(NSString *)logText toUrl:(NSString *)url {
> > +    NSMutableURLRequest *request = [[NSMutableURLRequest alloc] initWithURL:[NSURL URLWithString:url]];
> > +    [request setHTTPMethod:@"POST"];
> > +    [request setHTTPBody:[logText dataUsingEncoding:NSUTF8StringEncoding]];
> > +    [request setValue:@"text/plain" forHTTPHeaderField:@"Content-Type"];
> > +    
> > +    connection = [[NSURLConnection alloc] initWithRequest:request delegate:self startImmediately:YES];
> If I make it a property will it get nil-ed out?
> 
> —
> Reply to this email directly or view it on GitHub.
>
an alternative would be:
'#attributes' => array('class' => array('islandora-bookmark-hidden-submit')),
tokenizes
Yeah, I have no idea what I was thinking about here, anyway :P
Looks like this was to avoid a diners philosopher's problem, where >1 job are trying to lock sets of files that overlap.

Are you sure this isn't still needed?
This is actually being done on the next line, as ``Indexable`` has the attribute ``haystack_use_for_indexing``. This helps avoid errors where ``Indexable`` is copied to a different namespace/module & the conditional then erroneously fails.
Apologies for not making this clear before.
For further details, see Arthur's comment above.
Android does not fire this event.
I don't see any of these properties in android.
It's worth mentioning that image views have improper behavior on iOS with respect to % measurements (and possibly others) - % measurements are "% of image content" rather than "% of parent".
This method will resolve the passed in scheme / path, and use the Proxy's creationUrl if the path is relative
_Avanti a Dio!_ Enjoy the performance!;-)
Yes, @ugisozols was too eager merging this. The current code won't actually be idempotent.
I could add a key!  I'll make it something awesome.
I think this is the way how it is possible to run hook twice, does not it? Because you set it to `false`, then run `$template->assign_display` what runs hook second time, as `$this->useTemplateHook` is set to `false` for now.
need a test for this
Let's keep it simple for now :)

We can deploy to our EC2 instance instead of Heroku
crap variable name. please use something for verbose and descriptive.

like querystringParameters.
`(not (teen? age))` returns a boolean by it self.