Yes.  In OSX, strikethrough and underlined are not attributes of the font.   They are attributes of the text that is being drawn with the font.  If you look at the NSFont object, there is no settings for underlined/strikethrough.   There are attributes to query where the underline should be placed (although nothing for strikethrough thus the 0.6 on my other pull request).   Thus, setting an wxFont with an underlined style on any OSX control will not result in any underlined text as the underlying OSX font doesn't know about it.   The text that would be drawn with the font needs to be attributed with that information.


Updating to master of fork changes.


`for (int i = 0; i < fieldNames.length; ++i) {`


:smile:


I know it's useless now, but just checking that we'll still need right?


Menda: Has it, yes, but cannot install it. Read [here](https://github.com/manjaro/artwork-menda/blob/master/Menda/README-Firefox)!


great work adding extra methods to enhanced validation.


I know it's useless now, but just checking that we'll still need right


this is because when browser changes size to remove issue i opened " Before and after resizing turtle position changes #258 "
we have following container(1) main canvas four activity.js(2) we passed this canvas to arguments of trashcan(3) argument of turtles(3) blocks (4)palettes while calling** init** function in activity.js. So all must have to become windows size while resizing.So "=" operator for them to become windows size.


@1ec5 @friedbunny I don't think that nil attribute values are possible for any of the "stock attributes" so I went with this approach.


My mindset was doing a reference to a propertyDescriptor return.

It seems weird to check if an accessor property is writable or not.


Should go into `maintenance_0.10.x`?!


Seriously, @hans ? After writing this many boolean masks, you'd think a typo would be out of the question.. :)


Because you shouldn't change it. It's just there to make the other values below shorter.


Nice work adding extra methods.


this is well organized and well thought out code. congratulations! a little less white space in the future. thats my only guidance. keep kickin butt.


fair point :) right now, the app_rep constructor inlines whenever the function and all the arguments are known at compile-time.


Nevermind.. its an old commit and in `0.10.3`, just the comment was recent..


@Nevcairiel: Will that help for the apparently related ticket on MPC-HC's Trac?


so a lint of a fix that was a feat?


That is kind of confusing. `CREDENTIALS_HOSTNAME` is actually a pattern that's used in setting up the nginx config files. If you look at the value below, it's not actually the hostname.


scaladoc for jobId


Adding extra attributes to help enhanced validation. Nice work


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - i386-Windows-7-VisualStudio-12: OK (2119 of 2120 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


ddd


Yes, this is better, but I'm not convinced about the indentation yet, I was thinking more like this:
~~~
v:event         Data related to an |autocommand| event as a Dictionary. The
                available keys differ between different events and are
                specificed at the documentation for each |event|. The possible
                keys are:

                    regname      requested register ["x] or the empty string for
                                 an unnamed operation.
                    regcontents  text stored in the register as a list of lines.
                    regtype      type of register as returned by |getregtype()|.
                    operator     The operation performed. Unlike |v:operator|, it
                                 is set also for an Ex mode command. For instance,
                                 |:yank| is translated to "|y|".
~~~


Do we need to implement this method? When is this actually used?

I don't recall implementing this for any of the previous steps, @overtron please correct me if I'm wrong


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16718](https://teamcity.mesosphere.io/viewLog.html?buildId=141496&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:03:42


nope, fixed


Opened GHCi, it seems parsec works as I expected:
```haskell
> import Text.Parsec
> let test1 = parse (string "") ""
> let test2 = parse (string "" >> many anyChar) ""
> map test1 ["", "1", "12"]
[Right "",Right "",Right ""] -- 1
> map test2 ["", "1", "12"]
[Right "",Right "1",Right "12"] -- 2
```
(`""` at the right in definitions of `test`s is a source filename used only in error messages: [docs](https://hackage.haskell.org/package/parsec-3.1.9/docs/Text-Parsec.html#v:parse).)
1 shows it parses `""` successfully, and 2 shows it doesn’t consume from the input, so `many anyChar` can parse the rest.

Anyway, I don’t think anyone would be using `text("")` as a ‘literal value’—if only in dynamically generated parser, where string could occasionally be empty, then this is an issue.


No, thats entirely unrelated.


Hi - can you give a few words on the changes you have made here? Perhaps it could be a contribution to the original repository.

Thanks,
/Thomas.


thank you! :+1: this probably fixes https://bugs.php.net/bug.php?id=71135


Changed shebang to bash in 13125c62481a8d46db93a6443b76159f33510357.


Looks good to me, I'll merge later tonight, thanks!


This is working great for me, thanks! Can you please rev the version number and update pypi?


Interesting this route. You could have left the `required: true` on each rule as well. Nonetheless I like your approach to this solution.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2151 of 2152 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1002 of 1002 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-28.94%25-green.svg)


:thumbsup:


dont call onResize() function while initialize because all container are not set yet


cioè togliere il float e basta no?


i just put old code in comments just ignore


BTW, is there any reason why Foundation is build using a Debug build_mode? Wouldn't it make more sense to build it in Release mode?


Ok.


What does this line of code do?


@dting
Not sure what you mean here. Where are the original change lists?


I don't think `TeachersDashboard` is used anymore. Remove this and the component itself? @rogerhutchings


"wybierzesz IP z listym" => "wybierzesz IP z listy"


The location of the OrientDB should be coming from the Consul


@SandeepApsingekar  Actually these attributes should be private and we need to add setter & getter functions for these attributes


The 'no_' prefix is a bit weird from the API perspective:
```python
double_negative = make_mask(PQ, no_cloud_shadow_fmask=False)
```
I'd prefer something like this:
```json
cloud_shadow_fmask:
  bit_index: 13
  value: 0
  description: Cloud Shadow (Fmask)
```

Or we could parse the 'no_' prefix and expose both names through the API


I musst support @gdude2002 here it looked indeed better


@artych Do you think this branch and the Ember_2.2 branch of yours are stable enough to make a pull request to Addepar?


and this is the second comment.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Clean clean clean! This code is TOO CLEAN! Great work!


looping it up here will cut down on duplicate logic.


Great job extending this to enhanced your validations.


@ChrisJefferson : `cygpath` is an bash interface to [`cygwin_conv_path`](https://cygwin.com/cygwin-api/func-cygwin-conv-path.html) which GAP kernel can call directly, without resorting to `cygpath`.


That's not very nice either. I'd prefer a more verbose way of specifying enumeration values:

```json
land_obs:
  bit_index: 9
  value: 1
  description: Land Obs

sea_obs:
  bit_index: 9
  value: 0
  description: Sea Obs
```
API examle
```python
confusing_sea_mask = make_mask(PQ, land_obs=False) # everyone knows not land is sea
clear_sea_mask = make_mask(PQ, sea_obs=True)
```


I think you have the basics down already here at the beginning. great! You dont need this much white space, though. "beautify" will fix it for you.


Mrm...I'm not a big fan of the GLib logging for a few reasons, one is that it prints the program name and `CRITICAL` before hand, but that's usually redundant if you're looking at it from the terminal, and syslog/journald also capture the program name too.  I guess I basically don't like the default log function.

I would prefer a g_printerr ("error: %s", local_error->message) here, like we do elsewhere, or at least in ostree.


'no_band_7_saturated' for consistency? Crazy, but consistently so


I wanted to write test..


I applaud your multiple commits and messaging. Great work! 'Beautify' before you commit so others can read it, too.


Any particular reason this is needed?


@vaa Your 'if' expression read as "if date of event is NOT AFTER current date, then ...". A bit complicated, isn't it? If you will use **Date.before** code would be more readable.


Update commits


Testing


Yep. Needs to be an absolute url


reack


Thanks. It probably isn’t the biggest deal in the world, just something I’ve been trying to mind as we refactor stuff.


Oops, missed that.


CSS file is in real trouble. Line 7 doesn't target an attribute, class or id.


@dting
B/c the result of `is NaN(0)` returns `false`. So I had to add an additional boolean statement to include the case if `num === 0`. This was the best way I could think of doing this.


If you use guards then there is no need to use matchcontinue:
case ... guard Expression.expEqual(e1,e2) and (i1 == i2) then inClockKind;

Also, you compare e1 with e1, which might not be what you want! Same for the cases below.


OK, more work is needed, but -12 elo with some time losses is a progress.


to_string is C++11, which we try to avoid in assimp. Would you mind changing?


When running `bin/make-libjsass-darwin.sh` from the repository root, the script fails here:
`bin/make-libjsass-darwin.sh: line 34: cd: bin: No such file or directory`


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2363 of 2395 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.66%25-yellow.svg)


love your hw1 site! great idea! 'beautify' your code before you commit. initial summary can be a company name or idea so you have a jump point for future reference


Fix typo


I think it got replaced. It should be ?> to close php code.


This was actually Episode 20 not Episode 19


What about:

```
self.db = db or self.db
if self.db is None:
    raise TypeError('db is required for initialization.')
```


When running `./make-libjsass-darwin.sh` from the `bin` directory, the script fails here:
`./make-libjsass-darwin.sh: line 35: cd: ../src/main: No such file or directory`


Hi Eko,

I will try your code, as soon as I'm back to the Lab Thursday this week - until then, yes, you should try I2C. It's generally much more reliable and faster.

Patrick


This file is missing a license header.


That returns the dreaded `The redirect uri included is not valid.` though.


The single argument to `TempDir` is the directory to create the temporary directory under, and the directory you specified doesn't exist. Use `None` instead (or use `IMP.test.get_tmp_file_name` to make a single temporary file). I'll make it default to `None` since anything else makes little sense.


:-DDD And I thought you are just adding arbitrary new keys, as you like :-DDD


Your test proves only that dao method `getEvent` is called when you calling service method `getFutureEvent`. It doesn't test that you get events only from future not from past. To test this case you need make integration test.


Why removing the testcase?


You targeted well and have the concepts well in hand. Note: beautify before you commit to reduce white space. so humans can read it.


For anything thats not required to be public, you should put in the .m file instead

We only expose things that other classes need


Why do twice the same?


also you can remove the readonly attribute as well


this is wrong - should be a string


You do not need ?> for this framework. There's another, more obvious mistake.


should also be a string...


One is supposed to be SetToggleState(false). Will fix


you should use `self.pullToRefresh` which uses the property instead of using the ivar directly.

Except for init and dealloc, in general you want to use property access. There several reasons for this if you interested in knowing why..


Thanks.  Will remember and use the `env` technique moving forward as well.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16720](https://teamcity.mesosphere.io/viewLog.html?buildId=141503&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:10:32


same thing here (self.pullToRefresh)


I know you're not adding validation yet, but you may want to add `validates :user_course_enrollment, presence: true` for `EnrollmentModuleAssignment`.  Otherwise I think this can blow up if `:user_course_enrollment` is nil.


great commit and message work ethic! Code is ALL ABOUT details.


Should this also be inside the transaction?


What's the recursion for?

Also, what was the reason for the MSWP change?


@juwlee - Do you happen to recall if there were actually known issues w/ using this gem on ruby 1.9.3? Thanks for any insight!


started knowing about Trello website
https://trello.com/


мб `Prepared`?


When a stream ends during a delay for loading a fragment, the fragment might then not be loaded correctly. This can be resolved by moving the delay from the FragmentModel to the FragmentLoader. It works by allowing the xhr request to be generated right away, because if the stream ends then there might be a problem generating the request. The delay is then inserted just before sending the xhr request.

After this change, there will no longer be a need for hooking the PLAYBACK_SEEKING event in the FragmentModel.


cc @TiarkRompf


ok


Congratulations!


Not needed?


Wheres Day3Homework?


`serverBaseUrl` should be something like:
```java
String serverBaseUrl = req.getScheme() + "://" + req.getServerName() + ":" + req.getServerPort();
```

Right now, the URLs are in the form:
`http://localhost:9898/swell/account/sem/swell/account/sem/avatar/c2VtQGxvY2FsLm5ldA_1456179424753.png`

As you can see, the chunk `swell/account/sem` is repeated twice.


@dting
I created this based on my anticipation of a feature we could possibly implement in the future, after further thought, I agree that this is not necessary and may be a little over the scope for now.


I'm really amazed that the presentation compiler barfs on this. It is basically the simplest example of a macro.


We still have the unsolved problem of how to set the username for ec2. I am testing centos images, and they all have different usernames (ec2-user, centos, root, etc).


No worries, it is always good to discuss, in order to understand the issues of each other.

I guess, in order to get the stable versions to work on Travis, I guess you'd need to re-run all builds of the stable packages in order of dependence. The problem seems to be that the ``NumPy`` version of Travis has changed in between the builds, and you are trying to mix packages (wheels) compiled against different versions of numpy.

 Although... I am not sure that re-compiling the stable versions of the packages would actually be sufficient. When building a stable version on Travis, it still takes the latest versions of the dependent packages. Hence, in order to work 100%, you would need to compile the latest stable **and** the latest master branches of all packages that you want to have inside bob in the order of [dependence](https://raw.githubusercontent.com/idiap/bob/master/dependencies.png), and wait for one package to have uploaded its wheels in order to start the next one.

However, it might be easier to push new versions of all C++ packages (again, in order of dependence, including waiting time). In this case, we might adapt the ``bob_new_version.py`` script in order to build only against the stable versions of the dependent packages. In this case we make sure that a stable bob package compiles and works with all the other stable versions of the dependent packages.


@ellismg I can't seem to find a list of runtime dependencies for CoreCLR, any pointers here?


This is why I don't like building locale keys through concatenation -- it's really hard to find where these are used (e.g. by grepping the code).


by "simplest" I mean "simplest generic" macro.


I don't see any places in DecimalOperators that use DECIMAL without parameters. Also, I don't think we want to introduce that notion, as it's not clear what that type means (it's similar to the issue of `varchar` vs `varchar(x)`).

In particular, `DECIMAL(p) == DECIMAL(p, 0)`, but what values of precision and scale are implied if none are specified?


@bgruening   I updated the help on this.  Thanks for pointing it out.


actually, maybe not... I wonder if the implicit parameter is the thing that confuses it. Hmm. Could try returning a `List[T]` (implement as `Nil`) and then there is no need for an implicit.


great work! you clearly understand the core concepts here. Note: commit messages are to help your team (now) and YOU (in the future). Try to make them more descriptive. If its too much to remember, commit more often. Don't worry, GIthub isn't clingy; commit as much as you want.


This is about the closest thing I have found

https://github.com/dotnet/coreclr/blob/master/Documentation/install/get-dotnetcore-dnx-linux.md


Those wight spawners will make it **really** tough :P


Lettering broke, so I updated it to work


Can't hensonAnnotationMirror be null?
some elements will enter here not because they have HensonNavigable annotation, but because they have InjectExtras


We can install it with a change to `.travis.yml`.


Is `coxpcall` supposed to support extra arguments? The coroutine version below does (by accident, because `copcall` uses the same code). For now the native code path does as well. The [HTML documentation][1] doesn't mention any extra arguments ...

  [1]: http://keplerproject.github.io/coxpcall/index.html#reference


approved


Wrong URI.


OMG, thats the Wolrd's biggest code change on CS!


trying to fix MTP


I forgot to remove the comment "//"


we enter here, only HensonNavigable Annotated elements can run this code.


See my comment on https://github.com/stephanenicolas/dart/commit/9b7fde82d71b0b10ea9dc930bb32de32fef59e09#diff-582777e2e4956b57da959c349422e25cR125


...what the hell? Linux RobotBuilder apparently royally screwed up the diff


You already have these concepts well in hand. Great work! This is beautiful code! (Don't hesitate to help those that don't)


FixCrashWithZoneAndSystemERV (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2105 of 2109 tests passed, 549 test warnings)

 * 553 tests had: AUD diffs.
 * 483 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.81%25-yellow.svg)


@gut that's another thing we will never use LLVM? Note that  i put a #if __x86_64__ #else not_implemented() but, you can revert if you want. Not a problem.


Hey @slavapestov, @egranata and I banged out most of the memory reader this weekend that LLDB can eventually use. What do you think of `ReflectionContext::getTypeRef` and `TypeRefSubstitution`?


This is the name of the group, so I thought this was the standard for naming the packages with composer ? I can add a NON OFFICIAL statement in the title of the repositories if you prefer ?


@dting
These are fixed in other pull request.


(Also sorry for the large patch and it needs some general cleaning up as well)


@dting
Same as in comment above.


Exploring how to use Trello for our sudoku project


I guess there's a small issue here because not all languages capitalize titles.

Best regards,
Mr. Localization Party Pooper


Nope, this means that in idle state we check every 0.5 sec if the server sent us any answer for our requests... If it did, then we flush those updated info in the EP files.


[inst/benchmarks/profviz-wtc_sig.R:16:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L16) *style:* **Commas should always have a space after.**
```r
d1.ar1 <- arima(d1[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[inst/benchmarks/profviz-wtc_sig.R:17:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L17) *style:* **Commas should always have a space after.**
```r
d2.ar1 <- arima(d2[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[inst/benchmarks/profviz-wtc_sig.R:34:1:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L34) *style:* **Variable and function names should be all lowercase.**
```r
registerDoParallel(cl)
^~~~~~~~~~~~~~~~~~
```
[inst/benchmarks/profviz-wtc_sig.R:48:33:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L48) *style:* **Commas should always have a space after.**
```r
        .init = array(dim = c(3,3,nrands)),
                                ^
```
[inst/benchmarks/profviz-wtc_sig.R:48:35:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L48) *style:* **Commas should always have a space after.**
```r
        .init = array(dim = c(3,3,nrands)),
                                  ^
```
[inst/benchmarks/profviz-wtc_sig.R:50:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L50) *style:* **Commas should always have a space after.**
```r
          y[,,x$r] <- x$a
             ^
```
[inst/benchmarks/profviz-wtc_sig.R:52:9:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L52) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
        }) %dopar% {
        ^
```
[inst/benchmarks/profviz-wtc_sig.R:53:43:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L53) *style:* **Commas should always have a space after.**
```r
          list(r = r, a = array(dim = c(3,3), (1:9) * r))
                                          ^
```
[inst/benchmarks/profviz-wtc_sig.R:70:3:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L70) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {seq_len(1000000)},
  ^
```
[inst/benchmarks/profviz-wtc_sig.R:70:20:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L70) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  {seq_len(1000000)},
                   ^
```
[inst/benchmarks/profviz-wtc_sig.R:71:3:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L71) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {1:1000000},
  ^
```
[inst/benchmarks/profviz-wtc_sig.R:71:13:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L71) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  {1:1000000},
            ^
```
[inst/benchmarks/profviz-wtc_sig.R:82:36:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L82) *style:* **Commas should always have a space after.**
```r
    out1 <- apply(ts, MARGIN = c(1,2), quantile, 0.95, na.rm = TRUE)
                                   ^
```
[inst/benchmarks/profviz-wtc_sig.R:86:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L86) *style:* **Commas should always have a space after.**
```r
      out2[i,] <- row_quantile(ts[i,,], 0.95)
             ^
```
[inst/benchmarks/profviz-wtc_sig.R:86:37:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L86) *style:* **Commas should always have a space after.**
```r
      out2[i,] <- row_quantile(ts[i,,], 0.95)
                                    ^
```
[inst/benchmarks/profviz-wtc_sig.R:92:13:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L92) *style:* **Commas should always have a space after.**
```r
      out3[,i] <- row_quantile(ts[,i,], 0.95)
            ^
```
[inst/benchmarks/profviz-wtc_sig.R:92:36:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L92) *style:* **Commas should always have a space after.**
```r
      out3[,i] <- row_quantile(ts[,i,], 0.95)
                                   ^
```
[inst/benchmarks/profviz-wtc_sig.R:92:38:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/benchmarks/profviz-wtc_sig.R#L92) *style:* **Commas should always have a space after.**
```r
      out3[,i] <- row_quantile(ts[,i,], 0.95)
                                     ^
```
[inst/examples/example-plot.biwavelet.R:34:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-plot.biwavelet.R#L34) *style:* **Commas should always have a space after.**
```r
par(mfrow = c(1,2))
                ^
```
[inst/examples/example-plot.biwavelet.R:42:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-plot.biwavelet.R#L42) *style:* **Commas should always have a space after.**
```r
par(mfrow = c(1,2))
                ^
```
[inst/examples/example-pwtc.R:15:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-pwtc.R#L15) *style:* **Commas should always have a space after.**
```r
par(mfrow = c(2,1), oma = c(4, 0, 0, 1),
                ^
```
[inst/examples/example-wclust.R:14:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-wclust.R#L14) *style:* **Commas should never have a space before.**
```r
w.arr[1, , ] <- wt.t1$wave
        ~^
```
[inst/examples/example-wclust.R:15:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-wclust.R#L15) *style:* **Commas should never have a space before.**
```r
w.arr[2, , ] <- wt.t2$wave
        ~^
```
[inst/examples/example-wclust.R:16:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/inst/examples/example-wclust.R#L16) *style:* **Commas should never have a space before.**
```r
w.arr[3, , ] <- wt.t3$wave
        ~^
```
[R/check.data.R:31:8:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/check.data.R#L31) *warning:* **no visible global function definition for ‘check.datum’**
```r
  y <- check.datum(y)
       ^~~~~~~~~~~
```
[R/check.data.R:34:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/check.data.R#L34) *warning:* **no visible global function definition for ‘check.datum’**
```r
    x1 <- check.datum(x1)
          ^~~~~~~~~~~
```
[R/check.data.R:44:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/check.data.R#L44) *warning:* **no visible global function definition for ‘check.datum’**
```r
    x2 <- check.datum(x2)
          ^~~~~~~~~~~
```
[R/check.data.R:61:47:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/check.data.R#L61) *style:* **Commas should always have a space after.**
```r
    if (class(x[, 1])[1] == "Date" | class(x[,1])[1] == "POSIXct") {
                                              ^
```
[R/phase.plot.R:38:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L38) *style:* **Commas should always have a space after.**
```r
  phases[-a.row,] <- NA
                ^
```
[R/phase.plot.R:39:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L39) *style:* **Commas should always have a space after.**
```r
  phases[,-a.col] <- NA
          ^
```
[R/phase.plot.R:43:27:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L43) *style:* **Commas should always have a space after.**
```r
      if (!is.na(phases[i,j])) {
                          ^
```
[R/phase.plot.R:45:32:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L45) *style:* **Commas should always have a space after.**
```r
              alpha = phases[i,j], col = arrow.col)
                               ^
```
[R/phase.plot.R:76:60:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                           ^
```
[R/phase.plot.R:76:63:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                              ^
```
[R/phase.plot.R:76:66:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                                 ^
```
[R/phase.plot.R:76:69:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                                    ^
```
[R/phase.plot.R:76:72:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                                       ^
```
[R/phase.plot.R:76:75:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L76) *style:* **Commas should always have a space after.**
```r
  X <- (par()$usr[2] - par()$usr[1]) / par()$pin[1] * c(x1,x2,x3,x4,x5,x6,x7)
                                                                          ^
```
[R/phase.plot.R:77:60:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                           ^
```
[R/phase.plot.R:77:63:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                              ^
```
[R/phase.plot.R:77:66:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                                 ^
```
[R/phase.plot.R:77:69:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                                    ^
```
[R/phase.plot.R:77:72:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                                       ^
```
[R/phase.plot.R:77:75:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/phase.plot.R#L77) *style:* **Commas should always have a space after.**
```r
  Y <- (par()$usr[4] - par()$usr[3]) / par()$pin[2] * c(y1,y2,y3,y4,y5,y6,y7)
                                                                          ^
```
[R/plot.biwavelet.R:114:29:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/plot.biwavelet.R#L114) *style:* **Commas should always have a space after.**
```r
                     "cyan","#7FFF7F", "yellow",
                            ^
```
[R/pwtc.R:89:40:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L89) *warning:* **no visible binding for global variable ‘MOTHERS’**
```r
  mother <- match.arg(tolower(mother), MOTHERS)
                                       ^~~~~~~
```
[R/pwtc.R:91:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L91) *warning:* **no visible global function definition for ‘check.data’**
```r
  checked <- check.data(y = y, x1 = x1, x2 = x2)
             ^~~~~~~~~~
```
[R/pwtc.R:105:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L105) *style:* **Commas should always have a space after.**
```r
  y.ar1 <- arima(y[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[R/pwtc.R:106:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L106) *style:* **Commas should always have a space after.**
```r
  x1.ar1 <- arima(x1[,2], order = c(1, 0, 0))$coef[1]
                      ^
```
[R/pwtc.R:122:20:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L122) *style:* **Commas should always have a space after.**
```r
  y.sigma <- sd(y[,2], na.rm = TRUE)
                   ^
```
[R/pwtc.R:123:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L123) *style:* **Commas should always have a space after.**
```r
  x1.sigma <- sd(x1[,2], na.rm = TRUE)
                     ^
```
[R/pwtc.R:145:20:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L145) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.cw_yx1 <- smooth.wavelet(s.inv * (cw.yx1), dt, dj, wt.y$scale)
                   ^~~~~~~~~~~~~~
```
[R/pwtc.R:146:20:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L146) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.cw_yx2 <- smooth.wavelet(s.inv * (cw.yx2), dt, dj, wt.y$scale)
                   ^~~~~~~~~~~~~~
```
[R/pwtc.R:147:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/pwtc.R#L147) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.cw_x1x2 <- smooth.wavelet(s.inv * (cw.x1x2), dt, dj, wt.y$scale)
                    ^~~~~~~~~~~~~~
```
[R/smooth.wavelet.R:51:37:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/smooth.wavelet.R#L51) *style:* **Commas should always have a space after.**
```r
    wave.pad[seq_len(m)] <- wave[ii,]
                                    ^
```
[R/smooth.wavelet.R:68:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/smooth.wavelet.R#L68) *warning:* **no visible global function definition for ‘convolve2D_typeopen’**
```r
  swave <- convolve2D_typeopen(twave, rev(ker))
           ^~~~~~~~~~~~~~~~~~~
```
[R/smooth.wavelet.R:71:41:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/smooth.wavelet.R#L71) *style:* **Commas should always have a space after.**
```r
  swave[keep.start:(keep.start + n - 1),]
                                        ^
```
[R/wclust.R:40:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *style:* **Commas should always have a space after.**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                    ^
```
[R/wclust.R:40:27:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *warning:* **no visible global function definition for ‘wdist’, Did you mean 'dist'?**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                          ^~~~~
```
[R/wclust.R:40:42:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *style:* **Commas should never have a space before.**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                                        ~^
```
[R/wclust.R:40:43:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *style:* **Commas should always have a space after.**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                                          ^
```
[R/wclust.R:40:55:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *style:* **Commas should never have a space before.**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                                                     ~^
```
[R/wclust.R:40:56:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wclust.R#L40) *style:* **Commas should always have a space after.**
```r
      dist.matrix[n,j] <- wdist(w.arr[n, ,], w.arr[j, ,])
                                                       ^
```
[R/wdist.R:35:31:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L35) *style:* **Commas should always have a space after.**
```r
  u <- wsvd$u[seq_len(nfreqs),]
                              ^
```
[R/wdist.R:36:31:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L36) *style:* **Commas should always have a space after.**
```r
  v <- wsvd$v[seq_len(nfreqs),]
                              ^
```
[R/wdist.R:38:61:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L38) *style:* **Commas should always have a space after.**
```r
  Lnk <- t(u[, seq_len(nfreqs)]) %*% Re(wt1[seq_len(nfreqs),])
                                                            ^
```
[R/wdist.R:39:61:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L39) *style:* **Commas should always have a space after.**
```r
  Ljk <- t(v[, seq_len(nfreqs)]) %*% Re(wt2[seq_len(nfreqs),])
                                                            ^
```
[R/wdist.R:49:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L49) *style:* **Commas should always have a space after.**
```r
      (u[,2:NROW(u)] - v[,2:NROW(v)]))))
          ^
```
[R/wdist.R:49:27:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wdist.R#L49) *style:* **Commas should always have a space after.**
```r
      (u[,2:NROW(u)] - v[,2:NROW(v)]))))
                          ^
```
[R/wt.bases.R:32:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wt.bases.R#L32) *style:* **Put spaces around all infix operators.**
```r
    morlet =rcpp_wt_bases_morlet(...), # original wt.bases.morlet(...),
           ^~
```
[R/wt.R:75:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wt.R#L75) *warning:* **no visible global function definition for ‘check.datum’**
```r
  checked <- check.datum(d)
             ^~~~~~~~~~~
```
[R/wt.R:81:20:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wt.R#L81) *style:* **Commas should always have a space after.**
```r
  sigma2 <- var(d[,2])
                   ^
```
[R/wt.R:107:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wt.R#L107) *warning:* **no visible global function definition for ‘wt.bases’**
```r
    wb <- wt.bases(mother, k, scale[a1], param)
          ^~~~~~~~
```
[R/wt.sig.R:42:40:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wt.sig.R#L42) *warning:* **no visible binding for global variable ‘MOTHERS’**
```r
  mother <- match.arg(tolower(mother), MOTHERS)
                                       ^~~~~~~
```
[R/wtc_sig_parallel.R:24:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L24) *warning:* **no visible global function definition for ‘get_minroots’**
```r
  mr1 <- get_minroots(lag1[1])
         ^~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:25:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L25) *warning:* **no visible global function definition for ‘get_minroots’**
```r
  mr2 <- get_minroots(lag1[2])
         ^~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:28:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L28) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
  d1 <- cbind(ntseq, ar1_ma0_sim(mr1, lag1[1], ntimesteps))
                     ^~~~~~~~~~~
```
[R/wtc_sig_parallel.R:36:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L36) *style:* **Variable and function names should be all lowercase.**
```r
  NUMC <- getDoParWorkers()
          ^~~~~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:36:11:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L36) *warning:* **no visible global function definition for ‘getDoParWorkers’**
```r
  NUMC <- getDoParWorkers()
          ^~~~~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:48:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L48) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
                      }) %dopar% {
                      ^
```
[R/wtc_sig_parallel.R:58:26:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L58) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
      d1 <- cbind(ntseq, ar1_ma0_sim(mr1, lag1[1], ntimesteps))
                         ^~~~~~~~~~~
```
[R/wtc_sig_parallel.R:59:26:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L59) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
      d2 <- cbind(ntseq, ar1_ma0_sim(mr2, lag1[2], ntimesteps))
                         ^~~~~~~~~~~
```
[R/wtc_sig_parallel.R:71:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L71) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
      sw1 <- smooth.wavelet(s.inv * (abs(wt1$wave) ^ 2), dt, dj, wt1$scale)
             ^~~~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:72:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L72) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
      sw2 <- smooth.wavelet(s.inv * (abs(wt2$wave) ^ 2), dt, dj, wt2$scale)
             ^~~~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:96:5:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L96) *warning:* **no visible global function definition for ‘row_quantile’**
```r
    row_quantile(rand.rsq[,j,], sig.level)
    ^~~~~~~~~~~~
```
[R/wtc_sig_parallel.R:96:28:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L96) *style:* **Commas should always have a space after.**
```r
    row_quantile(rand.rsq[,j,], sig.level)
                           ^
```
[R/wtc_sig_parallel.R:96:30:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc_sig_parallel.R#L96) *style:* **Commas should always have a space after.**
```r
    row_quantile(rand.rsq[,j,], sig.level)
                             ^
```
[R/wtc.R:97:40:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L97) *warning:* **no visible binding for global variable ‘MOTHERS’**
```r
  mother <- match.arg(tolower(mother), MOTHERS)
                                       ^~~~~~~
```
[R/wtc.R:100:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L100) *warning:* **no visible global function definition for ‘check.data’**
```r
  checked <- check.data(y = d1, x1 = d2)
             ^~~~~~~~~~
```
[R/wtc.R:115:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L115) *style:* **Commas should always have a space after.**
```r
  d1.ar1 <- arima(d1[,2], order = c(1, 0, 0))$coef[1]
                      ^
```
[R/wtc.R:116:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L116) *style:* **Commas should always have a space after.**
```r
  d2.ar1 <- arima(d2[,2], order = c(1, 0, 0))$coef[1]
                      ^
```
[R/wtc.R:128:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L128) *style:* **Commas should always have a space after.**
```r
  d1.sigma <- sd(d1[,2], na.rm = T)
                     ^
```
[R/wtc.R:129:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L129) *style:* **Commas should always have a space after.**
```r
  d2.sigma <- sd(d2[,2], na.rm = T)
                     ^
```
[R/wtc.R:134:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L134) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.wt1 <- smooth.wavelet(s.inv * (abs(wt1$wave) ^ 2), dt, dj, wt1$scale)
                ^~~~~~~~~~~~~~
```
[R/wtc.R:135:17:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L135) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.wt2 <- smooth.wavelet(s.inv * (abs(wt2$wave) ^ 2), dt, dj, wt2$scale)
                ^~~~~~~~~~~~~~
```
[R/wtc.R:153:16:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.R#L153) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
  smooth.CW <- smooth.wavelet(s.inv * (CW), dt, dj, wt1$scale)
               ^~~~~~~~~~~~~~
```
[R/wtc.sig.R:59:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L59) *warning:* **no visible global function definition for ‘get_minroots’**
```r
  mr1 <- get_minroots(lag1[1])
         ^~~~~~~~~~~~
```
[R/wtc.sig.R:60:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L60) *warning:* **no visible global function definition for ‘get_minroots’**
```r
  mr2 <- get_minroots(lag1[2])
         ^~~~~~~~~~~~
```
[R/wtc.sig.R:63:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L63) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
  d1 <- cbind(ntseq, ar1_ma0_sim(mr1, lag1[1], ntimesteps))
                     ^~~~~~~~~~~
```
[R/wtc.sig.R:79:24:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L79) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
    d1 <- cbind(ntseq, ar1_ma0_sim(mr1, lag1[1], ntimesteps))
                       ^~~~~~~~~~~
```
[R/wtc.sig.R:80:24:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L80) *warning:* **no visible global function definition for ‘ar1_ma0_sim’**
```r
    d2 <- cbind(ntseq, ar1_ma0_sim(mr2, lag1[2], ntimesteps))
                       ^~~~~~~~~~~
```
[R/wtc.sig.R:92:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L92) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
    sw1 <- smooth.wavelet(s.inv * (abs(wt1$wave) ^ 2), dt, dj, wt1$scale)
           ^~~~~~~~~~~~~~
```
[R/wtc.sig.R:93:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L93) *warning:* **no visible global function definition for ‘smooth.wavelet’**
```r
    sw2 <- smooth.wavelet(s.inv * (abs(wt2$wave) ^ 2), dt, dj, wt2$scale)
           ^~~~~~~~~~~~~~
```
[R/wtc.sig.R:95:16:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L95) *style:* **Commas should never have a space before.**
```r
    rand.rsq[, , r] <- abs(smooth.CW) ^ 2 / (sw1 * sw2)
              ~^
```
[R/wtc.sig.R:111:13:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L111) *style:* **Commas should always have a space after.**
```r
    result[,i] <- row_quantile(rand.rsq[,i,], sig.level)
            ^
```
[R/wtc.sig.R:111:19:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L111) *warning:* **no visible global function definition for ‘row_quantile’**
```r
    result[,i] <- row_quantile(rand.rsq[,i,], sig.level)
                  ^~~~~~~~~~~~
```
[R/wtc.sig.R:111:42:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L111) *style:* **Commas should always have a space after.**
```r
    result[,i] <- row_quantile(rand.rsq[,i,], sig.level)
                                         ^
```
[R/wtc.sig.R:111:44:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/wtc.sig.R#L111) *style:* **Commas should always have a space after.**
```r
    result[,i] <- row_quantile(rand.rsq[,i,], sig.level)
                                           ^
```
[R/xwt.R:75:40:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L75) *warning:* **no visible binding for global variable ‘MOTHERS’**
```r
  mother <- match.arg(tolower(mother), MOTHERS)
                                       ^~~~~~~
```
[R/xwt.R:78:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L78) *warning:* **no visible global function definition for ‘check.data’**
```r
  checked <- check.data(y = d1, x1 = d2)
             ^~~~~~~~~~
```
[R/xwt.R:93:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L93) *style:* **Commas should always have a space after.**
```r
  d1.ar1 <- arima(d1[,2], order = c(1, 0, 0))$coef[1]
                      ^
```
[R/xwt.R:94:23:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L94) *style:* **Commas should always have a space after.**
```r
  d2.ar1 <- arima(d2[,2], order = c(1, 0, 0))$coef[1]
                      ^
```
[R/xwt.R:105:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L105) *style:* **Commas should always have a space after.**
```r
  d1.sigma <- sd(d1[,2], na.rm = T)
                     ^
```
[R/xwt.R:106:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L106) *style:* **Commas should always have a space after.**
```r
  d2.sigma <- sd(d2[,2], na.rm = T)
                     ^
```
[R/xwt.R:127:9:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L127) *warning:* **no visible global function definition for ‘ar1.spectrum’**
```r
  P1 <- ar1.spectrum(d1.ar1, wt1$period / dt)
        ^~~~~~~~~~~~
```
[R/xwt.R:128:9:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/R/xwt.R#L128) *warning:* **no visible global function definition for ‘ar1.spectrum’**
```r
  P2 <- ar1.spectrum(d2.ar1, wt2$period / dt)
        ^~~~~~~~~~~~
```
[tests/testthat/test-convolve2D.R:7:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L7) *style:* **Commas should always have a space after.**
```r
  y <- c(1,1,2)
           ^
```
[tests/testthat/test-convolve2D.R:7:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L7) *style:* **Commas should always have a space after.**
```r
  y <- c(1,1,2)
             ^
```
[tests/testthat/test-convolve2D.R:9:27:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L9) *style:* **Commas should always have a space after.**
```r
    convolve2D_typeopen(x,y),
                          ^
```
[tests/testthat/test-convolve2D.R:10:18:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L10) *style:* **Commas should always have a space after.**
```r
    convolve2D(x,y, type = "open"))
                 ^
```
[tests/testthat/test-convolve2D.R:15:12:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L15) *style:* **Commas should always have a space after.**
```r
  y <- c(1,1,2)
           ^
```
[tests/testthat/test-convolve2D.R:15:14:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L15) *style:* **Commas should always have a space after.**
```r
  y <- c(1,1,2)
             ^
```
[tests/testthat/test-convolve2D.R:17:18:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L17) *style:* **Commas should always have a space after.**
```r
    convolve2D(x,y, type = "circular"),
                 ^
```
[tests/testthat/test-convolve2D.R:20:1:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L20) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-convolve2D.R:21:1:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-convolve2D.R#L21) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-plot.biwavelet.R:27:1:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-plot.biwavelet.R#L27) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-rowQuantileC.R:41:22:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                     ^
```
[tests/testthat/test-rowQuantileC.R:41:24:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                       ^
```
[tests/testthat/test-rowQuantileC.R:41:27:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                          ^
```
[tests/testthat/test-rowQuantileC.R:41:29:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                            ^
```
[tests/testthat/test-rowQuantileC.R:41:31:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                              ^
```
[tests/testthat/test-rowQuantileC.R:41:33:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                                ^
```
[tests/testthat/test-rowQuantileC.R:41:35:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                                  ^
```
[tests/testthat/test-rowQuantileC.R:41:37:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                                    ^
```
[tests/testthat/test-rowQuantileC.R:41:40:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-rowQuantileC.R#L41) *style:* **Commas should always have a space after.**
```r
  data <- matrix(c(1,1,NA,2,3,3,3,5,NA,NA), nrow = 1)
                                       ^
```
[tests/testthat/test-wclust.R:17:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wclust.R#L17) *style:* **Commas should never have a space before.**
```r
w.arr[1, , ] <- wt.t1$wave
        ~^
```
[tests/testthat/test-wclust.R:18:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wclust.R#L18) *style:* **Commas should never have a space before.**
```r
w.arr[2, , ] <- wt.t2$wave
        ~^
```
[tests/testthat/test-wclust.R:19:10:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wclust.R#L19) *style:* **Commas should never have a space before.**
```r
w.arr[3, , ] <- wt.t3$wave
        ~^
```
[tests/testthat/test-wclust.R:28:37:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wclust.R#L28) *style:* **Commas should always have a space after.**
```r
  expect_equal(dim(c$diss.mat), c(3,3))
                                    ^
```
[tests/testthat/test-wtc_sig_parallel.R:9:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc_sig_parallel.R#L9) *style:* **Commas should always have a space after.**
```r
d1.ar1 <- arima(d1[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[tests/testthat/test-wtc_sig_parallel.R:10:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc_sig_parallel.R#L10) *style:* **Commas should always have a space after.**
```r
d2.ar1 <- arima(d2[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[tests/testthat/test-wtc_sig_parallel.R:39:34:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc_sig_parallel.R#L39) *style:* **Commas should always have a space after.**
```r
    expect_equal( dim(out), c(42,64) )
                                 ^
```
[tests/testthat/test-wtc.sig.R:9:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc.sig.R#L9) *style:* **Commas should always have a space after.**
```r
d1.ar1 <- arima(d1[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[tests/testthat/test-wtc.sig.R:10:21:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc.sig.R#L10) *style:* **Commas should always have a space after.**
```r
d2.ar1 <- arima(d2[,2], order = c(1, 0, 0))$coef[1]
                    ^
```
[tests/testthat/test-wtc.sig.R:39:34:](https://github.com/vsimko/biwavelet/blob/2a14ee222bcab833a685a99a086363bf3149a7a3/tests/testthat/test-wtc.sig.R#L39) *style:* **Commas should always have a space after.**
```r
    expect_equal( dim(out), c(42,64) )
                                 ^
```


This represents a giant leap forward! Great work! You're getting it.


will fix


Do you mind splitting up the patch first? I think the memory reader and substitution stuff is mostly decoupled?


Actually is needed, took out by mistake


@kyleries Not Juwon, but the reason why Ruby 2.0.0+ is required is because TLS 1.2 is becoming required for PayPal API connections. Currently, only Sandbox requires TLS 1.2 but the production PayPal environment will require it in the "near" future. You can find out more at the [PayPal SDK TLS update repository](https://github.com/paypal/TLS-update) or the [PayPal TLS microsite](https://www.paypal-knowledge.com/infocenter/index?page=content&id=FAQ1914&expand=true&locale=en_US). You may also be interested in the [SSL Certificate Upgrade](https://www.paypal-knowledge.com/infocenter/index?page=content&id=FAQ1766&expand=true&locale=en_US).

There are some unofficial patches for Ruby 1.9.3 that support TLS 1.2 but they are unsupported by Ruby  core dev itself. (My understanding is that Ruby 1.9.3 is unsupported as of early last year for any fixes including security patches).


Done! Switched to TempDir with 01bfef8.


These are dead debugging changes - I'll remove these.


Yeah - it should. Thought the destroy couldn't be rolled back but it can


Trying to resubmit my work again


can you talk about the choice of behavior based test validation here.  it's pros and cons and why you chose it?


Didn't think about it at the time, but this can be moved above for a microscopic performance boost (even above `global`).

#returnearly


This is just a sketch - we'll need the definition of this to be in the C library interface.


You're finding work-arounds and I applaud that. Making code work, no matter what is the core of a coder. Very impressed. You'll need to figure out file structure to move forward though. Watch the videos I sent you.


This update was done incorrectly. What happened is that those files failed to download because OSRM_RELEASE variable was not set. The latter problem will happen locally and I'm working on fixing that in https://github.com/Project-OSRM/node-osrm/pull/148. The former problem of incorrect checksums I'll fix by reverting this change in `develop` /cc @lbud


I'm curious, why do you split out the prototype instead of following the CommonJS pattern of returning an object?


I see your point... However this would require changing both the agent and the server, so I would like to this only in place of a name significantly better than this! Any suggestions?


Please remove your password from here and change it immediately wherever you used it


Sure, let me see what I can do.


rerere


erererer


Fixed in 6da64ec


Seems like I've forgot to update the use when I've reverted the definitions after @goyakin's review in https://github.com/anba/ecma262/commit/b595f473aab57ba8191ef157409cc9d7864b3b9a#commitcomment-15628036


> Do you have a suggestion for another key to turn into the capslock key?

What do you think of left shift : `key <LFSH> { overlay1=<CAPS> };`
I tried it and that work great.

> I think I will add the second row to your maps to be consistent.

I didn't managed to make the media key work as F key (and vice versa) with the overlay key, so I don't know if it's possible on Falco. Even if it work it's really useless to have 2 F keys rows I think and that lead to my next comment.

> For consistency sake, we should do something similar for the falco ones and have two keyboards where the media keys are F keys.

> Since beta2, our default keyboard made the F keys media keys by default (without a special key).

What do you think of keeping only the F key with the overlay on numbers key ? That work well and that allow us to have access to all 12 F keys. Honestly what the point of having 2 F keys row ?


Now, how do we get it to emit a recursive definition of `star_loop` :)


@snackycracky: I updated the PR. Any suggestions?


Why you dont set it in your cluster definition? You have username field for all providers such as:

ec2:
   ami: ...
   username: root

baremetal:
   ips:
      - ...
   username: centos


Don't translate dates. They are translated automatically (assuming you add the locale and dateformat strings to your lang.it.ini.


yorick is a contributor 3!


Addresses #3


typo: boring


Missing `If Type(_P_) is String` check, cf. assertion in step 1 of CanonicalNumericIndexString.


How about `/<a\s+[^>]*\b%s\s*=\s*([\'"])(.*)\1[^>]*>/i`? Still not as good as a DOM library but might handle some edge cases.


Let's change them here.


Does this have a test?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16723](https://teamcity.mesosphere.io/viewLog.html?buildId=141516&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:03:53


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#YoU2H16nSWGtLAEJQULquA


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16724](https://teamcity.mesosphere.io/viewLog.html?buildId=141517&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:05


The change lists that add these lines of code.


`s/host/address/`

This [matches upstream parameter names][1]. I'd also support `s/listen/bind/` but meh.

[1]: https://docs.python.org/2/library/basehttpserver.html


yes, will do


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1005 of 1006 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.9%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-29.0%25-green.svg)


Ok, then the help message needs to be updated :) ... How about " [...] how often the agent checks for RPC responses and applies them if any were received ..."

Also, the option name has a typo : iterval -> interval


How about request_endpoint_details()?


Good call! I will update


@BriceShatzer, we used the live edit mode for the same reasons.

For us Daux is hosted on a server that I'm not usually logged in to, so it's a lot easier to just hit edit and fix up a doc instead of having to go through 6 steps to get into the server to tweak an incorrect link.

At this point it's all just a matter of development time and for someone to do it. I can see why it was removed (the original implementation was pretty naive and done in a way which made it difficult to maintain, and daux was going through multiple huge restructuring projects), but hopefully we'll see it come back soon.

What's nice about daux is that if you're using it, minimal features is probably exactly what you're looking for, so you can live with running an old version because it already does everything that you need it do (hopefully),


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


I guess you should run it through `htmlspecialchars_decode()` here because it is HTML encoded. Or maybe `html_entity_decode()`


Why?


So the table on the [joda docs](http://www.joda.org/joda-time/key_format.html) would leave me to believe the original code would produce the same format as the ruby CA (and would match with the [java Date class](https://docs.oracle.com/javase/7/docs/api/java/text/SimpleDateFormat.html)). Though from my experiments below I can reproduce the issue reported by fiddyspence in [SERVER-1007](https://tickets.puppetlabs.com/browse/SERVER-1007).

```clojure
user=> (require '[clj-time.format :as t])
nil

user=> (require '[clj-time.core :as time])
nil

user=> (def custom-formatter (t/formatter "YYY-MM-dd'T'HH:mm:ssZ"))
#'user/custom-formatter

user=> (t/unparse custom-formatter (time/now))
"2016-02-22T22:29:22+0000"

user=> (def custom-formatter (t/formatter "YYY-MM-dd'T'HH:mm:ssZZ"))
#'user/custom-formatter

user=> (t/unparse custom-formatter (time/now))
"2016-02-22T22:29:32+00:00"

user=> (def custom-formatter (t/formatter "YYY-MM-dd'T'HH:mm:ssZZZ"))
#'user/custom-formatter

user=> (t/unparse custom-formatter (time/now))
"2016-02-22T22:29:44UTC"

user=> (def custom-formatter (t/formatter "YYY-MM-dd'T'HH:mm:ssz"))
#'user/custom-formatter

user=> (t/unparse custom-formatter (time/now))
"2016-02-22T22:30:04+00:00"
```

From Chris Price's comments in that ticket it looks like we'd rather go with the way it actually works now. If that's the case... I still don't know why this works on some folks' computers and not mine or yours... I'm looking to see if what versions of joda and friends I have installed.

And if we're keeping the output, after figuring out why this fails for you and I, we might want to update the test not to fail and change the format to ZZ so that it matches the documentation.


Practice was actually on 2-21-16.


Sweet!


Sloppy, so sloppy. Thanks!! :heart:


Managing /etc/motd


I would love to try this to harass my colleagues? Can you give more information how to install this? I have my token but I don't see the config.js and I'm not sure where to put the files locally. Any tips?


Thanks, I thought i had tried that before. But maybe that was because of the bug it didn't work.


left-over element?


FixDatasetFiles (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2118 of 2119 tests passed, 5 test warnings)

 * 5 tests had: AUD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


Hardcoded path. I think we have something in leap.common to get the config folder.


The actual code changes were done in the first commit


That way we don' merge a changelist that won't build into the repository.


In fact, the return value is not about false or true, but now about number of processed characters, see this [thread](http://forum.arduino.cc/index.php?topic=361411.0) in the Arduino forum.


554 5.7.1 is a separate message. The error number was not modified as part of this commit and the postfix and postfix-rbl filters are intentionally different. If you'd like to create an issue for adding that log line to this filter, please provide the entire log line. But this commit had no impact on your log line being matched or not.


k


Fixed in 7ecb8b1.


Work performed:

- verified commit# 9b8304752cc9bacf29815187cd578c444da6a9a0 .

- verified commit#792f6d5ac13c223c904453c01386d1f89d4f2a4f.

- re-applied commit# 8c1f6755d2da14de484d07748b727509c18bdc6f to stable codebase . Recompiled. Retested.

- modified commit# 74e10be20141485054b9bca76158cd3abfd82fda “Add Photo app crash” (relocated class file to SAV module, updated layout reference) then re-applied to stable codebase. Runtime crash fixed by adding more Proguard rules for ResourceIDs. Also missing ‘FitnessAPI’ Play services import in wrapper app caused runtime crashes.

- re-applied commit# 8c2f92ef0deb2cf91be3b793dd8e675ff68983df


Add a comment: 'Generated by generateRulesIndex.js'


@diptanu ok -- I am running the master branch.  Two questions: 1) which is correct above, `docker.auth_config` or `docker.auth.config`.  2) Should I be looking in the docker.log or the nomad output for an error?


The newline fix should ideally belong to the previous commit.


Yes, doing a forced recursive delete is a very dangerous operation in bash. The right way to remove a symlink is to use `unlink`. However, if this does not work on cygwin and we _must_ resort to this, we should do it under a platform switch so that the other platforms are not affected.


hehe, we know how to do that with LMS ;)


why would we need to here, though?


Hey @ZogStriP, I just noticed this branch. Is this going to be a new base importer?
It looks a lot faster than the old one. Can't wait to give it a try with the phpBB3 importer. :wink:

It's great to see that you aren't using `OFFSET` in your database queries. :+1:

Are you planning to add the ability to skip already imported items like it is possible with the current base importer ([`all_records_exist?`](https://github.com/discourse/discourse/blob/master/script/import_scripts/base.rb#L197-L214))? I guess with the "no offset" queries it would even be possible to store the last imported IDs in the `plugin_store_rows` table or something like that. That would make skipping already imported rows a lot easier and faster than it is now.


you don't want this here... the logic of this if statement is saying ` if (args.bitstreamVersion < VERSION_ENTITIES_LIGHT_HAS_INTENSITY_AND_COLOR_PROPERTIES) {` --- that means... it's an old bitstream... which... by defintion should not and don't include `PROP_SURFACE_RADIUS`... so this logic is wrong...

But it turns out that we could actully get rid of this whole if statement... since we no longer have old binary streams for the old verisons... so you don't really need this at all... but I'd leave that unchanged for now... just don't include your new property here...


done

On Mon, Feb 22, 2016 at 2:49 PM, Luke Faraone <notifications@github.com>
wrote:

> s/host/address/
>
> This matches upstream parameter names
> <https://docs.python.org/2/library/basehttpserver.html>. I'd also support
> s/listen/bind/ but meh.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/herb/grouper/commit/2b8ec0f50284b90080342b89552814ca9b514bf7#commitcomment-16254398>
> .
>

--

herb


thx that u change it ;)


Good catch. I just recently found that, I forgot to update the github page.


Hi,

Is there a specific reason to use @Inject decorator instead of type information ? Should we do the same at the application level ?

Regards & thanks
Kapil


@ErikEvenson Please use `docker.auth.config` as the key in the client options. And the error would be in Nomad logs.


Other classes are like `diffins`, `diffmod`. Maybe this one should be `diffhref` to maintain consistency?


the placement of this prop no longer matters, but it's customary to add new properties at ~ line 168/169... where you see the comment that says...

```
    // ATTENTION: add new properties to end of list just ABOVE this line
```


This caused a strange bug where a lone grimscale/warleader was giving two damage. When there was no other attacking murlocs, `oldMurlocsThatCanAttack` went down to 0, and -1 that gave you -1. Therefore, you got damage -= 1 * -1, which is equivalent to damage += 1!


Looking closer, there's a bug with this approach. Looking for `</font>` in a string like `<</font>` returns false. You don't "rewind" when you find a mismatch and miss scanning portions of the array.


I love it!


I will test it asap if I see better results I will merge this change into piccolo-dev because all performance improvements are welcome :)


Won't this fatal in < 4.4 since `WP_REST_Request` won't be defined? Could we keep the callback in the REST class and just call `WPCOM_Liveblog` from there?


This likely won't work. What you are doing is mixing up packages now. The author of the PPA has his own packaging (ppa:beineri/opt-qt551-trusty) you can't compare the upstream names with his naming.

For example the qt5xmlpattern5 thing is called in his PPA as: ```qt55xmlpatterns``` and there is only the debug package ```qt55xmlpatterns-dbg```. I guess everything is in ```qt55xmlpatterns```.
See: https://launchpad.net/~beineri/+archive/ubuntu/opt-qt551-trusty/+packages


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16725](https://teamcity.mesosphere.io/viewLog.html?buildId=141546&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:07:16


Awesome. How much smaller does this make the stdlib? There are tons of unspecialized stubs that just shift dozens of generic arguments up and down the stack a few spots.


Have we taken any measures against XSS here?


In terms of collapsing levels, we should be able to, no? (But I guess you're saying we don't need it here, because `star_loop` is already compiled on it's own?)


Can you please expand the comment a bit? No need to add a Haddock comment to every definition, just explain why we need a `Writer` monad.


Ah, I completely missed that the source files did have a license :D


Who is currently at the helm for MSpec now that @danielmarbach is no longer piloting the project? I think it is time this PR was merged, as it is a trivial but correct documentation change.


@slavapestov - ok, split this up. `TypeRefSubstitution` and `ReflectionContext::getTypeRef` are the two main pieces.


Sadly, we can't use anonymous functions since WordPress still supports PHP 5.2 :(


` 	breakfast $TARGET`
Thats me every morning


I am confused about how to recognize of the migratory group's demography information through GPS data , for example, whether there are children who needs education service.


Might be cleaner just to call the endpoint `entries`? The between would implied by the times.


Oops, missed that influxdb had been updated.
:+1:


God I love open-source. I'll merge tomorrow!

- Ludvig

> On 22 feb. 2016, at 19:13, Ed Morley <notifications@github.com> wrote:
>
> Maybe? I think the current way is quite wordy for something that is readily expressible in a single paragraph.
>
> I was approaching it from the point of view of a literal API parameter list, but that page is more informal, so I agree your suggestion makes more sense. PR updated :-)
>
> —
> Reply to this email directly or view it on GitHub.
>


Oops, I take back all of the above - we're already installing a log handler that uses syslog.


Semangat petualang terus berlanjut :+1:


Why do we need to add this static var?


Good stuff changes


TrinityCore rev.  ecd74a8  2016-02-22 16:47:42 +0100 (HEAD branch)

Now you can't destroy the Cocoon


Here's all the runtime dependencies for the native components from CoreCLR:

```
libc6
libedit2
libffi6
libgcc1
libicu52
liblldb-3.6
libllvm3.6
liblttng-ust0
liblzma5
libncurses5
libpython2.7
libstdc++6
libtinfo5
libunwind8
liburcu1
libuuid1
zlib1g
```

Some of these are specific for components like SOS (so the runtime would boot if they were not present) but this is everything you'd need so the loader would be happy.

For CoreFX, the list is much larger:

```
libasn1-8-heimdal
libc6
libcomerr2
libcurl3
libffi6
libgcc1
libgcrypt11
libgnutls26
libgpg-error0
libgssapi3-heimdal
libgssapi-krb5-2
libhcrypto4-heimdal
libheimbase1-heimdal
libheimntlm0-heimdal
libhx509-5-heimdal
libidn11
libk5crypto3
libkeyutils1
libkrb5-26-heimdal
libkrb5-3
libkrb5support0
libldap-2.4-2
libp11-kit0
libroken18-heimdal
librtmp0
libsasl2-2
libsqlite3-0
libssl1.0.0
libstdc++6
libtasn1-6
libwind0-heimdal
zlib1g
```

I generated these lists by using @bartonjs's ldd + grep magic, augmenting it a little to roll things up:

`ldd -v *.so | grep so.*: | cut -f 1 -d: | xargs dpkg-query -S | cut -f 1 -d: | sort -n | uniq`


Could you please do such things on a branch and not break master?


I'm guessing I did it wrong, hahahaha. Whoops, my bad.


Using Nomad v0.3.0-rc1 (1d9c65ba83e9ff764eb30b759cb310d6458aa389) these are my nomad logs:

```
$ cat /var/log/nomad.log
    Loaded configuration from /home/docker/nomad-config.hcl
==> Starting Nomad agent...
==> Nomad agent configuration:

                 Atlas: <disabled>
                Client: true
             Log Level: INFO
                Region: global (DC: dc0)
                Server: false

==> Nomad agent started! Log data will stream in below:

    2016/02/22 23:04:17 [INFO] client: using state directory /var/lib/nomad/client
    2016/02/22 23:04:17 [INFO] client: using alloc directory /var/lib/nomad/alloc
    2016/02/22 23:04:17 [INFO] fingerprint.cgroups: cgroups are available
    2016/02/22 23:04:17 [INFO] fingerprint.consul: consul agent is available
    2016/02/22 23:04:21 [WARN]: fingerprint.env_gce: Could not read value for attribute "machine-type"
    2016/02/22 23:04:21 [INFO] client: setting server address list: [192.168.99.116:4647]
    2016/02/22 23:05:24 [ERR] driver.docker: failed pulling container 212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0: API error (500): Get https://212181198242.dkr.ecr.us-east-1.amazonaws.com/v2/vbenergy/vbez-web/manifests/2016.02.19.0: no basic auth credentials
    2016/02/22 23:05:24 [ERR] client: failed to start task 'api' for alloc '6afca143-fb07-9fe6-c471-97ff535fbf6b': Failed to pull `212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0`: API error (500): Get https://212181198242.dkr.ecr.us-east-1.amazonaws.com/v2/vbenergy/vbez-web/manifests/2016.02.19.0: no basic auth credentials
```


Probably just the difference between running from the fork


The following doc(s) have been deployed to the [1338simple](http://1338simple-static-docs.pantheon.io/docs) Multidev environment:
- [source/_views/doc.html](https://github.com/pantheon-systems/documentation/commit/190ed00e53f130860de3c91024fbb7f4d38c681a/source/_views/doc.html)
- [source/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/190ed00e53f130860de3c91024fbb7f4d38c681a/source/assets/css/main.css)


See #77


OK, I see you added a comment in this patch.


I would also add a link to a sample JSBin they can use: https://jsbin.com/cagaye/edit?html,output


It should build in both debug and release, although our build script is esoteric and we could have messed something up.


This is `/home/docker/nomad-config.hcl`:

```
client {
   options {
      "docker.auth.config" = "/home/docker/.docker/config.json"
   }
}
```

and here is a redacted `config.json`:

```
$ cat /home/docker/.docker/config.json
{
	"auths": {
		"https://212181198242.dkr.ecr.us-east-1.amazonaws.com": {
			"auth": "<auth key>",
			"email": "none"
		}
	}
}
```


Maybe ### ? I feel ##s are a bit yelly, but that's a personal preference


This change adopts ruby map magic instead of a larger code block with additional intermediate variable assignment for `new_pids` and incorporates all the logic into two lines of code.  It groups the update of the solr_parameters all together at the end of the method.  Otherwise this code behaves the same way.


+shindigo It is in the camera movement subsystem.


lgtm


extra ```"``` at the end after factor)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16726](https://teamcity.mesosphere.io/viewLog.html?buildId=141549&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:07:12


![screen shot 2016-02-22 at 6 10 43 pm](https://cloud.githubusercontent.com/assets/5640772/13236011/a0220a4c-d98f-11e5-8d7d-2c6fd85787b5.png)


Style/RedundantFreeze: Freezing immutable objects is pointless.


what was wrong with //?


Did you mean to change this line to `return err` too? Otherwise, the change in line above doesn't seem to have a point.


I'm doing some logging here with a 00606TX transmitter as well.  Here's what I have so far, and now actively acquiring the gfile*.data files as well.  (NOTE: Some of below read temperature  on the receiver may not match the recorded transmit record, but do believe the majority matches.  Just guessing part of the binary data is split and parts read backwards?)

00827 AcuRite Weather Station

RECEIVER
Model: 00827RX (00827 Weather Station Receiver)

Transmitter
FCC ID: RNE606TX
Model Number: 00606TX

https://fccid.io/RNE606TX

35F (Binary 00100011)
[01] {32} 74 80 0e 1e : 01110100 10000000 00001110 00011110

34F (Binary 00100010)
[01] {32} 74 80 0d 33 : 01110100 10000000 00001101 00110011

33F (Binary 00100001)
[01] {32} 74 80 06 29 : 01110100 10000000 00000110 00101001

32F (Binary 00100000)
[01] {32} 74 80 02 aa : 01110100 10000000 00000010 10101010

31F (Binary 00011111)
[01] {32} 74 8f fb c0 : 01110100 10001111 11111011 11000000

30F (Binary 00011110)
[01] {32} 74 8f f7 74 : 01110100 10001111 11110111 01110100

29F (Binary 00011101)
[01] {32} 74 8f f1 2e : 01110100 10001111 11110001 00101110


Move to new file LogHoursViewModel.cs


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1564 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.71%25-green.svg)


@parkera Do you mean it is always built in both modes? Or do you mean it could be built in either mode?


ack, its neat


Looks good!


We were doing some experimentation with the constants to try to get auto-mode driving the right amount.


Probably! To be honest, that was the first time I've ever pushed a tag to a forked repo (with or without `apm`, so the error is mine, haha ^^)


ya i don't fully know what we are supposed to be doing i apologize . I did go through the power points and remade them.


Since the controller's responsibility is primarily behavioral, I felt like a behavioral approach was more appropriate.  Compare to the service tests, which are more functional.  In this case, without the behavioral approach, I might have had to add a value to the controller that only serves the purpose of passing this test, which I felt was a code-smell.

Pros?
* Mocking dependencies allows one to isolate the core functionality
* Easier to not explicitly worry about concrete instances
* Cleaner controller

Cons?
* Mocks can be :rainbow: magical :rainbow: and have unknown side-effects
* Mocks can be expensive to create & maintain


Suspicion to sql injection vuln - need to be changed to parameterized query...(on the word query)


It needed some time to do the reset for some reason.


maybe add an "expected sthg like 'git://github.com/giantswarm/repo' here"


Can you remove the changes related to Migrations? This will most likely go in another project.


@ErikEvenson Can you please remove the `https` from the `https://212181198242.dkr.ecr.us-east-1.amazonaws.com` in the config.json? Or alternatively in your image configuration in the nomad job spec can you specify `https://212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0 ` instead of `212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0 `

In the code, we are doing a map lookup to find out the authConfiguration, so that might be the problem.


This is definitely a great improvement. If you say it's default for Nexus devices, then I don't see a reason for not using it... :+1:


Put in separate view model file


Why isn't this `Exclude="@(TestAssemblies);@(CoreTestAssemblies)"`? If I update TestAssemblies, I immediately need to update this.


@pborenstein Thanks for your work. I few suggestions. Move the inner functions into their own files. Why? Because every time fish "loads" `iterm_shell_integration` it will create definitions for each of the inner functions and it does not have to. Also, recall fish does not have "private" functions, so while I see your intent to make these "private" in the traditional sense, they are most definitely _not_ private.

One thing you could do, is _erase_ the functions at the end of `iterm_shell_integration` since they will be reloaded anyway every time `iterm_shell_integration` is invoked.


Made changes to settings.xml:
Hide Tasks when the previous task is set to "none"
Hide Events when the previous event is set to "None"
Fixed Event "Test Command" field (changed 'visible="!eq(67,0)"' to 'visible="!eq(67,None)"')


Yes, i can check this very soon .. even for both LCD Types.
Should i cherry-pick this commit or can u provide some prebuilts ??


This change adopts ruby map magic instead of a larger code block with additional intermediate variable assignment for new_pids and incorporates all the logic into two lines of code. It groups the update of the solr_parameters all together at the end of the method. Otherwise this code behaves the same way.


TBH this is probably faster than invoking the `set_color` builtin to produce the ANSI color codes, but is there a reason for not choosing to use the more readable `set_color`?


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16727](https://teamcity.mesosphere.io/viewLog.html?buildId=141571&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:03:55


Thanks. I'll submit a new pull request that adds the clarification.


Fixed in the (soon-to-be-uploaded) latest commit.


Changing the image configuration in my job spect led to:

```
    2016/02/22 23:17:59 [ERR] driver.docker: failed pulling container https://212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0: API error (500): Error parsing reference: "https://212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web" is not a valid repository/tag
    2016/02/22 23:17:59 [ERR] client: failed to start task 'api' for alloc 'b599958a-5d2f-607e-e234-7152af457e00': Failed to pull `https://212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0`: API error (500): Error parsing reference: "https://212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web" is not a valid repository/tag
```

Removing the https in the `config.json` led to:

```
    2016/02/22 23:20:50 [ERR] driver.docker: failed pulling container 212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0: API error (500): Get https://212181198242.dkr.ecr.us-east-1.amazonaws.com/v2/vbenergy/vbez-web/manifests/2016.02.19.0: no basic auth credentials
    2016/02/22 23:20:50 [ERR] client: failed to start task 'api' for alloc '91d747b9-890d-42d2-b870-9f2fbe5f0d7e': Failed to pull `212181198242.dkr.ecr.us-east-1.amazonaws.com/vbenergy/vbez-web:2016.02.19.0`: API error (500): Get https://212181198242.dkr.ecr.us-east-1.amazonaws.com/v2/vbenergy/vbez-web/manifests/2016.02.19.0: no basic auth credentials
```


It takes time to do the encoder reset for some reason.


what changed the dns entry?


Ok, will do!


I cannot be sure that my change corrected the display issue.


reack


This looks great! Good job bromigo!


isn't `something` an undeclared variable reference?


> "supertab"

Young man this is your not father speaking. What is this new fangeleged **supertab**.
Is it a double tab or is it simply so many tabs that it makes a new line??
**_Kevin is pooping into his fruit and nut mix, but his poop is sultanas's's's'_**


remove


Naming convention typically ends in ViewModel


I checked and this gets optimised away. The reason I made this change is I hope to make more obvious to the compiler there are things that could get vectorized in here


Sure, can upload hwcomposer if it's needed. http://xda.mister-freeze.eu/XDA-files/ChronoMonochrome/misc/hwcomposer.montblanc.so.zip


Typo!


@ErikEvenson Ok I am going to launch a ECR myself now and test this on AWS myself. Thanks for helping to debug this, I will report here what I find out


all right, will do!


sqdqsd


First of all, thanks for doing this. Maintaining a super popular OS project is no small feat.
Now to the point, could you please publish this change to npm? I installed swagger with a `npm install -g swagger` and when I create a hello-world project, I get the version without this fix.

Thanks!


I don't know why, but here is how is displayed this warning box on the [current version of this page](http://scikit-learn.org/stable/modules/cross_validation.html#leave-one-label-out-lolo):
![weird_display_bug__scikit-learn_cross_validation](https://cloud.githubusercontent.com/assets/11994719/13236287/bb2e4a38-d9c3-11e5-9bb3-3bab36529590.png)

The `labels` and `:class:LeaveOneLabelOut` are wrongly displayed, and the **bold** should be not be there.


@diptanu -- thanks for your attention on this!  Using ECR would be an important feature for us.


which bug?


You're a bitch ass


Typo here!


抄袭可耻啊喂！


@michelleN Thanks for the clarifying text.


FixCrashWithZoneAndSystemERV (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2105 of 2109 tests passed, 549 test warnings)

 * 553 tests had: AUD diffs.
 * 483 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.81%25-yellow.svg)


Please not log statements in the final code


Did not delete constructor. Do this again.


It shouldn't need to be setuid anything. :-)


This is fixed in https://github.com/ruby-rdf/rdf/commit/474a3c9bd84bb6ad8828d1a83ddfad9c2da447f7.


Maybe use `MarkdownCommon`?

This adds useful stuff like inline code and tables and all that good jazz.


The version issue is causing travis to fail, and we haven't had time to track it down.  If you can help with that, I'll push a version.


I just thought it would be useful to be able to easily see what your existing tab stops are when you're noodling with documents.  I find .__repr__() really handy in general.  I have no more justification than that.

I wasn't planning on add_tab_stop() returning anything.  But I guess you're right that would be useful when building your tab_stops.


Surely Cabal already has code to test for this? Would prefer to use Cabal's test for this case here.


This is awesome thanks!


I doubted about the `btn-group` being unpronouncable, but I get your point. Lets go with this version.


The display issue has already been corrected in November, the change below does nothing (but does not break anything).

The changes above are true changes though.


I've added this note to give us a little explicit leeway about improving our definition of "merge" for the default repository.

A relatively simple improvement would be, e.g. to allow a transaction with a `#graph_name` to merge as long as data within its graph hasn't changed.

My sense is that if we can guarantee that the result of query and write operations would not have been changed by the diff since transaction start time, that would be a non-breaking change allowable in the 2.x release series.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (2119 of 2120 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


cc: @gkellogg @bendiken


I have forgotten about *test data*, I will add that


@msteinfeld Yes, v7 should be upgraded too. In the meantime, you can manually apply this patch on v7 and it should fix your issue too.


Is something broken in the previously published 0.13.11 -- if so, should one wipe it from the local cache?