labofsahil/github-event-dataset-2012 huggingface dataset:
So I just discovered that CRL time checks can never be equal. The CRL time is always either before or after the current time. The logic is in x509_vfy.c (see http://code.google.com/p/repositorium/source/browse/trunk/myOpenSSL/crypto/x509/x509_vfy.c?spec=svn982&r=982#1749). So setting the last_updated to one second ago will solve this issue.
This is central to the issue that this bug is part of, namely how messed up how concept of scope and variable lookup is.  My next task is to work out some other oddities related to scope (`$::var` can never refer to a var defined in a node and so there is no qualified name for those vars). 

I'll have to figure out what `resource` actually is here in order to add anything to it :)
...but you just validated that options can only possibly contain one key, :arguments, right?  So, I think you should either use `validate_options` *or* duplicate the options into params.  Doing both means you do the same check for valid options twice.
This should also be in a before action.
Doesn't the f.seek(offset) below already raise this exception if offset < 0?

That means that this check should not be necessary. If the spec fails when not doing this check here, this might point at an issue with File#seek and not with File.binread. So that means this should be fixed in File#seek and not here.
Added a clarifying inline comment.
removed use ﻿Zend\Stdlib\RequestInterface make this is need. in the docblock only. cmiiw.
Is possible use `php` as namespace like the functions?
C'e' un po' di inconsistenza tra "module" e "modulo". Personalmente userei soltanto "module"
This statements are not used in the project, please remove them.
@michaelbarnard that was the code you wrote....
Will a Wither launch its 'missiles' when it thinks it is an EnderDragon? :p
-> `if ( tasks[i].completed ) {`
maybe we should have `on('change month', ` and `on('change year` granularity, as well as this one. not sure about the name but I can't think of anything better ATM
for better or worse :)
What about raising `postFlush` here instead? (and moving the triggering logic to its own protected method?)
this syntax will only work with closures in 5.3, not with any callable. but your constructor does not mandates using closures
axed the method instead.
Funny that you mention that: I've had this same idea but on second thought decided against it. The line you link to checks for FEED_DOMAIN, whereas this one https://github.com/jawher/pelican/blob/issue-550-atom-feed-should-include-all-articles/pelican/generators.py#L159 checks for SITEURL, i.e. a different key.
The feed generator uses both, and technically speaking, they could be different.
import as something? then use something(foo, bar) further on?
Make sure this line does not contain white space.
Maybe add comment indicating why? So somebody later on does not go throught the same process we did? :)
Needing to explicitly set oauth_params didn't feel exactly right in this scenario. When Server gets some Request love this will go away I presume.
Do you really need requests here? django-browserid should pull in requests on it's own.
We could make ircname required - but that would be a whole lot of logic that we would need to add somewhere else.
from installMysql import mysql_exec
mysql_exec("icinga-web", sqlpassword, 'CREATE DATABASE icinga_web')
Hmm. If a string of "#qqgg$-" is passed should that resolve to the absence of all color (black) or the presence of all color (white)?
+1 for follow.
No, _content.scss doesn't - fixed.
Should this include anything from the footer?  This seems to break the constraint that components should never include other components as that is what composites are for.
Could you do something like have a DIVISIBLE variable, so you don't have to keep comparing SOLUTION === 'Yes' and such?
This feels dirty but it's acceptable. The conversion of byte[] to String back to byte[] (happens in setCert) feels dirty.
please change in all places where this variable is used
It is possible: 
http://ruby-doc.org/core-1.9.3/Module.html
http://cl.ly/3v2R2L0W1y433A0Y2V3t
OK, but doesn't that mean the list of pages will have the proxies included but not the ignored? I don't see where you'd call this with include_ignored true.
Why did you add an annotation here? DurationElement doesn't seem to be exposed to the plugins
actually, this is the chatham JobID that gets passed to atlantic -- mind updating that?
This excludes `[0, 0]`, which could be a valid datapoint, but not `[37, None]` which probably isn't.

Rather filter out invalid `x` values and decide what to do with null `y` values based on widget config. (I want to exclude null values in some metrics, but treat them as 0 in others.)
This could also benefit from a postcondition about the results conforming to the limit.
This is not required to make the test pass therefore should be its own test
Is there a reason these assertions use string comparison instead of comparing the DateTimes directly?
Are you sure you don't mean the more common `>>>` zero fill right shift?
The type can be Comment
This has been removed in the latest master and replaced by the `LocalStrategyWrapper` in `api.js`.
Can these tests go through the REST endpoints ?
Luc, could you remove the ``@Ignore`` and try if the tests still don't pass (Iuli has been fixing one issue wrt to the find references tests, so maybe things are better now). Otherwise, I really need to fix this asap.
node_reference doesn't seem to be a valid module...
http://drupal.org/project/node_reference

Is this part of another package now?
No this move was to fix the failing test :-P
Update this to the current master 17b1f5650bd329a315e88d6f07d449801db67d1d
shouldn't the constraint of 3 also be raised, to be able to work with timestamps
"кстати элемент с индексом (index + 1 + arc4random()%(kMaxDNAItems - 1))%kMaxDNAItems может хранить тот же символ и в итоге мы заменим "А" на "А" ;)"
Каким образом?
здесь вы заполняете каждую ячейку своим же номером, то есть ячейка 0 хранит 0, 1-1, 2-2 и т.д.
I really like how this is working out
Actually, now that I've played around with it, I'm changing my mind. The code you wrote accepts a **path**, not a url. I think we should use the name "kStartupPath" instead of "kStartupURL".

(I discovered this because I tried passing in "http://www.adobe.com" and the window came up blank.)
Instead of creating the user in here, create it outside of here and the just pass in the information.  Also, don't use username given that you're not actually using username anywhere.
Perhaps it's outside the scope of this PR, but how is `getIdentifierObject()` used? A MongoId is one possible type for an `_id`, but it can also be an object, string, number, etc. (anything but a numeric array).
This function doesn't reinitalize. See the `isset`. It is essentially identical to the ORM `enable` method
Do you use some flags with gofmt?  I can get the // to indent, but not add a blank space, I assume you mean above line 60.  Done.

The general rule is blank lines above all comments?
This is not correct - I will update this test to cover what I meant for it to test.
What is andalso? I've never seen that before and doesn't seem like it is an erlang BIF. Is that something we put in somewhere?
`(tar_excludes << ['.gitignore', "ext/#{packaging_repo}"]).flatten!`
Ah, my bad. Good catch.
Is there a cleaner way to do this?
private def acknowledgeDeliveryTag(deliveryTag: Long, remoteAcknowledgement: Boolean): Unit =
    for(c <- channel) {
        c.basicAck(deliveryTag, false)
        if (remoteAcknowledgement) deliveryHandler ! Acknowledged(deliveryTag)
    }
I think the name of this class should be Zikula_Doctrine2_Entity_AbstractCategoryAttribute because @ORM\DiscriminatorMap is category specific (and unchangable so modules can't use it).
Sorry, false alarm.  I forgot that in python 2.7, DeprecationWarnings are ignored by default:

```
$ python -Wd try_radon.py 
1.6.1
0.11.0.dev-94598b3
/Library/Frameworks/Python.framework/Versions/7.2/lib/python2.7/site-packages/numpy/lib/utils.py:139: DeprecationWarning: `radon` is deprecated!
  warnings.warn(depdoc, DeprecationWarning)
```
btw, _filldedent is now filldedent `from sympy.utilities.misc import filldedent`
stopping here for now until I hear from you about whether these changes should be made or not.
This brace belongs on the next line, Allman style.
please remove deprectaed package annotation
I'm thinking maybe users might want to modulate a few of these inputs. What do you think? Should we make grainLength and numGrains modulatable?

If so, you'll need to modify this code so that it reads from inputs.grainLength.output and inputs.numGrains within your unit generator's gen() function. You can see lots of examples through the code base, but in general if you want to support a control rate signal, you'll read the first sample from these inputs (e.g. that.inputs.grainLength.output[0]) right at the top of your gen() function. If you want to support audio rate inputs (which probably isn't necessary for these parameters), you'll read each value within the sample loop. Make sense? If not, I can send some sample code along.
this is also using the deprecated API
This is the command id of an old command? That might cause issues down the line, simply use a new command id.
So the hotRodWrapping field is only used for validation?
This can be `@(name)`.
This is `check_gc`:

```python
def check_gc(self, gc, fillcolor=None):
        orig_fill = gc._fillcolor
        gc._fillcolor = fillcolor

        delta = self.gc.delta(gc)
        if delta: self.file.output(*delta)

        # Restore gc to avoid unwanted side effects
        gc._fillcolor = orig_fill
```

It looks like the last line restores it. Might that be the cause?
It might conflict and require a different solution on master -- but it needs to be here so the plotting command list can be autogenerated and moved off of the front page of the docs.
obviously would need a docstring.
> How does one know which port to use in the web browser URL (if you have to type it yourself rather than letting it get shown automatically)?

I think this is a stupid question... these ports are for the websockets stuff right?
Nit: pep-8 suggests that variable names be all lower-cased and not use camel-casing. So audioTag should be audiotag or audio_tag.
на уровне идеи - а может сделать интерфейс, который бы реализовывали оба этих класса и ожидать именно его. DBFull, например.
This is a hake, wsdl.exe generate for boolean fields always a is specified parameter. This has to be true, else the handling of the method is incorrect. At This means that there has to be at least 2 parameters. For example foobar(boolean a1, boolean a1Specified)
As far as I know, an interface can not be instantiated and they are used just to specify how to pass data. So what is the purpose of the constructor in there ?

If you need some sort of initialization, maybe an abstract class can serve this purpose better than interface.
Makes it look more like an enum.
There should be 1 line separating routes.
Why not just delete the last line of the csv file instead of using this function? Or is that a quirk of slurp?
@gsilver Could you replace this by a CSS gradient (in lhnavigation as well)?

In here we're relying on another widget (lhnavigation) to be present.
Although that is currently the case, we shouldn't rely on it.
Could you add two tabs here to fix the indentation?
"admin" is a user account which _does_ exist and which therefore should return a "204" status. LiteUserExistsServlet has a bug because it relies on Solr to find existing user accounts but the out-of-the-box "admin" account does not have a Solr index record.
SG_FREE(m_refcount) :)
That should be obviously wrong and cause invalid reads
involvement is spelt wrong.
does this method redefine itself??  sorry im not sure what `class << self` is referring to in this case.  i guess i don't understand why this method has to be defined dynamically.
this will call `done()` twice, it needs to be `if (err) return done(err), though in this case it could just be `.end(done)`
Yes, you're right.
Do you have a suggestion ? I wanted to call this method "addMiddleware", but I don't know if this fits in the "Silex naming" scheme.
Exactly, the existence of JSON allows the value to be an object, so the object test does not need to pass, right ?
@gavinking you're right, swt browser has support for javascript

But it seems that `SyntaxHighlighter` is not comatible with javascript runtime which swt provide, during DOM transformation I am getting errors. I don't know how to easily fix it :-(
Call this "Getting Started in Windows". Could you also change the title of the linux guide to "Getting Started in Linux", please?

Then, please add the following line under the title line:

group: "dev-docs"

This line is needed for the side navigation menus that I'm working on.
missing return. Plus, the grunt callback should get `false` to indicate the failure. `return cb(false);` Ideally, it should be just `return cb(err);` but right now we're forced to log the error and call the callback with a `false` value. Will change in grunt 0.4
Any specific reason why we need to defer this to next tick?
i think you're right, but we can handle that pretty easily. I'll add something later tonight for this w/ test coverage of course.
Break these up into two methods.
``` js
if (this.nodeType != 1) return this;
```
Curious as to why the function is declared elsewhere and not inline here? I would of though having the function declared here would increase readability?

Or is `addContext` also used elsewhere?
`notif_content` -> be verbose, write out fully.
See documentation on `block` tag for more information. Or documentation on template inheritance.
`AccountChangeForm` maybe?

We also have to decide: do we have users or accounts?
Those two could be simply only one view. On GET you display a confirmation page, on POST you send it.
Seems like with this `TEMPLATE_CONTEXT_PROCESSORS`.
You are right. I will check comment message instead of number of comments.
It was an oversight from my end. I'll update it in the next push.
Much better structure with the for loops!
an empty message with a color kills the client with java.lang.error
to unite in the general script it is better...
This is just a convention I user from Bundler's .gempec
Hmm... You're right, I should have rather go with \S+
Thanks, I'll fix.
.. in the `guests_self_registration.modes` field.
I wouldn't necessarily agree about the "costly" part -- it's really just about async operations generally, which aren't necessarily costly, just non-blocking .
If we are doing Google style in this file we should put the type on the same line as the function name if it is not too wide.
How does it behave?
The intention was to start discussion concerning an issue I'd observed.  Talking about code is easier when there is code to look at.  I could have opened an issue, but it would have taken me just as long to explain the problem as it did to just hack a quick solution based on code that works in OMZ-Robby.  I don't spend that much time coding in ZSH, so I'm aware that much of my code will need cleaning up or completely reimplementing; I don't believe that makes my input worthless.
joblib needs to work in batches, especially on Windows, see mailing list "playing with joblib" 10/10/2011 Alexandre's comments early on
Sorry I don't quite get what you're saying.  Did you mean param_nums?  
The only reason I didn't make it nums is because I haven't implemented a 
function that computes confidence intervals for more than one parameter 
at a time.

On 08/14/2012 09:41 AM, Skipper Seabold wrote:
>
> In statsmodels/emplike/elregress.py:
>
> > +        x0 = np.delete(self.params, self.param_nums)
> > +        if method == 'nm':
> > +            llr = optimize.fmin(self._opt_nuis_regress, x0, maxfun=10000,
> > +                                 maxiter=10000, full_output=1, disp=0)[1]
> > +        if method == 'powell':
> > +            llr = optimize.fmin(self._opt_nuis_regress, x0,
> > +                                 full_output=1, disp=0)[1]
> > +        pval = 1 - chi2.cdf(llr, len(param_nums))
> > +        if ret_params:   # Used only for origin regress
> > +            return llr, pval, self.new_weights, self.new_params
> > +        elif print_weights:
> > +            return llr, pval, self.new_weights
> > +        else:
> > +            return llr, pval
> > +
> > +    def ci_beta(self, param_num, sig=.05, upper_bound=None, lower_bound=None,
>
> These should all probably be *_params to be consistent with the rest 
> of the package.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/statsmodels/statsmodels/pull/406/files#r1372030>.
>
This might be getting ahead of the issue at hand with ELReg but it might make sense to make regression forced through the origin it's own module because unlike ELReg, it **is** a method of estimation as well.
Is this the desired API for passing query data as a string? I'm just guessing based on working backwards.
shouldn't be commited
Complicated if predicate, perform extract method.
The reason for sys:filter is to prevent classloader clashes between the application server and the web app.  Are we packaging a jar that includes com.sun.* anything?  That seems wrong.
Add a `BrooklynHelp` command to allow setting the generic type to `BrooklynCommand` in the builder
i liked the ability to apply filters immediately, and fluent manipulation of immutables.  gets cumbersome with guava.
Could you make this `extension`
Thanks, I realized that got mixed up and closed this pull request. Once I have it fixed, should I create a new pull request or is there a way to update this one?
I got the impression from the scope of work that there is meant to be a list of possible specializations somewhere? That would imply that this should be a reference to that list rather than a `CharField`?
What is this underscore variable?
Are we spinning here? wouldnt it be friendlier to sleep?
A one paragraph on what is the gem for would be great. I thought it is for Walkscore fetching, but apparently it also supports WalkShed. This is cool, but totally undocumented.
Is it important for this to be "stable" in the sense that it always comes out in the same order? I'm just curious.
Thank you. It'd be interesting to see what is the algorithm for converting an image to be under a given size.
I don't actually see any difference when I run it.https://github.com/w7cook/OpenBook/pull/36/files#write_bucket_465
It'd be best if it just CRASH()ed at this point, with a relevant message.
А приведение к нижнему регистру первой буквы?
Полезно сообщать в исключении, что есть дубликат.
How does that translate into settings from environment variables, then? Just because we have separate Configuration instances in the code, it’s still going to look by default for an environment variable named “oauth.uri”, isn’t it? Or am I just misunderstanding the whole thing?
Can someone explain this adding an inline comment?  I have a hard time understanding what it's supposed to do.
In the default kNN classification, it *does* cancel out. In fact, when using the default priors (normalized `bincount`), we don't even need to mess around with probabilities: we simply return the class `k` with the highest presence in the neighborhood of our query point. (See [above](#issuecomment-7129633) for the gory details.)

But if we allow other priors (which is the point of  issue 399), for instance flat priors (`P(C_1) = P(C_2) = ...`) then it won't cancel out.
There a typo here: oscilation not osscilation
Could always copy ZMQ on this one and just have a `write(data, [meta/codec], [more])`. "Meta/Codec" defaults to whatever and more defaults to false. Once you use a `write()` with a truthy "more", the next `write()` without a truthy "more" would flush the buffer and pack the multipart.

```javascript
if (topic) this.encoder.write(topic, codec, true);
this.sendToPeers(encoder.write(data, codec));
```
You can use haskey() http://doc.opensuse.org/projects/YaST/SLES11/tdg/haskey.html
It is not good for the robust code. When is(...) evaluated to `false`, IsExp::targ will be set to `NULL`. And if `IsExp::semantic` is called twice, the second call will Access Violation.
This does not seem to be used.
we made sure that activity indicator must be added to a window
I didn't mean to remove it from the class, just that this function should call another function in `TiPropertyMapObject` where the logic should be. You can pass `_valueModify` to `TiPropertyMapObject`'s function
Are we sure the parameters for a DELETE are supposed to be added to the HTTPBody instead of the URL? Not sure this is the case.
Retourne les conditions générales d'utilisation d'une databox pour chacune des langues disponibles
How about just include Lint::Tests, and then override that one method inside this test class?
I need to update caching. I'll do that later this evening.
".tgz" is a common substitute extension for gzip'd tar files
The ```isConstant()``` check aborts this case successfully, and none of the others, and none of the regression cases, too.
No I don't - that was one of the deliberate changes (the first in the list)
indeed, this is already supported, but I wanted to make this more explicit as a supported way to specified a "static" revset to build
ouch, that is a bad one ;)
Seems like this could pull up to a single line nicely, or at least avoid the intermediate variable.  If you are touching the code anyway, might as well do that, but not enough to change otherwise.
You can use the `PuppetSpec::Files` module to get the right sort of behaviour here automatically around the temporary file.

You should generally prefer a `let` method rather than assigning to a member variable in specs.  Even if that isn't consistent with the surrounding code - and feel free to clean that up at the same time, of convenient. :)
I just copied this file from `pythonbrasil8/templates/schedules.html`. :P
Changed in both on 3a02e14a65678aacd3cdadfc1722c313af8cac3e.
Erikoisesti failaa filediffi tässä.
service name.
I am out of the office until 20/08/2012.




Note: This is an automated response to your message  "Re: [ariatemplates]
Set the default sorting as Descending (#116)" sent on 16/08/2012 15:45:55.

This is the only notification you will receive while this person is away.
In over-the-shoulder review, we agreed that he combination of a singleton client instance & many modified duplicated parts-of-the-client added more danger than speed or convenience, so @yuhunglin will switch to getting a new client instance per use.
Sama kuin `has-a-living-author?`:ssa.
are you absolutely sure about this? seems to run contrary to our discussion the other day
Ya just about to retract my comment :) Thanks.
I think this comment L10-L15 can be deleted
Please don't use encode(), because that method doesn't exist in Ruby 1.8.
This is copied from psexec.  HD or someone else wrote that.
Why sorting a single number?
Just a reminder for the possibility to add sorting. Adding sorting requires specific knowledge on the index used in that particular search
I was in the process of responding to the other comment by @stof about auto-update being wrong. Glad you wrote this here.

First, I didn't know what auto update meant. :) Now that that is cleared up, it makes more sense. I didn't know that the `pacakgist:update` stuff was already taking this into account and recrawling these projects less frequently. Very cool.

Second, I saw two uses for this API endpoint; being able to do one off package updates and being able to use this endpoint as a hook for systems other than GitHub. (generic git hooks for homegrown systems, svn hooks, etc.)

The `update=1` param is along the lines of what I was going to ask about. Whether or not it would make sense to be able to tell the system if this was an automatic update (and likely to continue to be updated automatically) or if this was a one time deal. If you like that idea (since you just proposed it) I'll implement it that way.
This might just be for Chrome because I haven't checked other browsers yet, but the canvas width and height properties must be integers >= 0. Also, we use && because if you set one of them to zero, both are set to zero.
Same doc comments.
Is there a reason we call `getView` here rather than accessing `_view` directly?  There are a couple places in this file where we call getters like this instead of just accessing the variable.
i thought this was an internal doc comment and not exposed. I kept it in there for anyone future development. If this is a public comment, I'll remove it.
this *SidePanelController(View({...})..)* suggest the creation of a new view called SidePanelView. This is not the right way of using the View mixin.
Important question: One of the benefits of Anvil and integrating it with the CLI is that it will allow users to write unit tests for their apps. Should we add a configuration setting for being able to set the harness config, template, and test directories?
It may not be visible, but it's there. When calling .children, you will see it, and the layout algorithm will lay the child out, meaning it impacts performance.

MW supports add() on every single UI element, without exception.
Make n an argument here with a default value? Different sites may require different retry values?
Created issue, see #348.
ThemeRoller (one word)
Why create an empty prototype?
missing constant (global constants)
Confirmed. More efficient if this is not a regular expression.
We could use a better unit test as this one isn't adequate for Tim Sort. It never touches gallop mode when merging, and the stability test doesn't go beyond insertion sort. I'll work on a better unit test and leave this comment here as a reminder for the time being.
Or you could do:
auto a = function(Char[]) { };

`put` accepts a function too.
I think no. I want to have something like a module global variable inside the scope of the struct. Or does `immutable` imply `static` in this case?
Add as a named url?
Should be "Beginner" :-)
this shoudn't change
What do these various categories mean?
This is for MVC2 ASPX compatibility. Unfortunately, since we can't get rid of the razor reference, MVC2 users need to manually remove it until we can specifically target them in Nuget 1.8.
What's the deal with course_path AND submit_path pointing to the same string/URL ?
New RSpec yield matcher hotness:
```
subject { AssessmentFormPresenter.new struct obj }

it 'yields' do
  expect { |block| subject.render_submission &block }.to yield_control
end
```
You might want to use nth-child selectors so you don't need to add 'odd'/'even' classes.
just `var getValue;` would be sufficient here
Should be 4. Pushed fix.
Please use Lukes File system utility to get the path.
Make the file name <sha1>.hdf5
Is there a reason to define function bodies in DDoc-only sections? This has lead to serios bugs with template functions in the past because of users using `-D` when building executables – for example, the version block for this very section got changed from `D_Ddoc` to `CoreDdoc` only very recently. Thus, I think it would be best to just avoid supplying implementations for such functions altogether, so that no mistakes can slip in in the first place.
Perhaps a global variable?
Fixed for the entire JDBC adapter with commit 7b16025
Same I feel as the explanation given above.
You have a nice way of explaining things to make them look scary ;)
And in all honesty id does look scary if enricher (SI) component could be used to modify any bean including the once outside of SI's scope.
Perhaps adding tests to show how it could be done and than fixing it to make sure it can't be done.
In any case I don't think its mergable at this point.
Add an assertion that the payload is a DOMSource?
Just replace this whole paragraph with

They give you the ability to persist data as well as retrieving data.
Replace null by void
Nice, that makes things a lot easier. Didn't know that form of open was deprecated, the book I had been reading was using that form. But thanks for the heads up.
@bp323 Can we remove the `=>`?
ok, only need 'and' instead of 'or'

Can you change it in this pull request or I should do it (if so - how?)
`battery.py` should be in the theme directory.
this looks like some excess baggage.
This delete *has* to execute or it could upset the VM environment right? In that case it should be in its own @test method of this class that runs_after this one but does not depend on it.
Is this the same for model resource of tastypie?

But what if I do not want a related field to be accessible through API, I still have to define? If not, then I would reword this to something like: *All related fields you want exposed through API must be manually defined.* (BTW, what is *resource file*?)
Stylistic comment: Imo, the function call, the heart of this part, is not as highlighted as it could be. This is not an error.

One could write a semantic comment like 
# show all messages and save their message mapping
That could make clear that the action happens here.

Another variation regarding the code:

msg_ids = show_message(..)
for msg_id in msg_ids:
 ....
What about the `include_displaynames`? Because of `[0]` in the new function, I would assume this could be used there, too.
typeof - оператор, а не функция - можно скобки не ставить
I think this is also true for ipaddr so I'll update the comment.
Great - much cleaner than reject
this is why 5.3 has array_replace
тут ты ее не сохраняешь, поэтому можно просто const Node*
This method (along with extract) should really be in an encapsulating class.
This should also be indented two more spaces.
It still isn't a great name, but I can't think of a better one, and it's definitely better than `isScalarTimeUnits`.
True, but this is not part of this PR. Please, create new issue dealing with this and hook file for sn-theme.
You must run install_sn.php, otherwise it will not update config value and ACP will display outdated version.
Кстати, если obj.recap != null, то pr.cap енто то же самое, что и pr.recap.
Енто видно из replyForm. Так що, вощемта, разницы нет.
any reason to remove all of these
Are we still planning to do this dynamically?
**Subir
Class comment missing
Tämä on juurikin miten se haluaisi tehdä. Tehtävänantoa sittemmin muutettu, map funktiota ei tosiaan oltu vielä esitelty.