return right away?

return EtcdConfiguration.exists(ArakoonInstaller.ETCD_CONFIG_KEY.format(cluster_name), raw = True)


Thank you very much! Everything works now (also, the build number you just fixed). I'll run this with the full matrix and then merge to master.


use type related file grouping


Good catch... I see, I'm missing a `reconnect()` call at the end of the `launchQZ()` function.   Is this sufficient or should we do it differently?


Right, will be in the next release.


No obvious issues with your stylesheet. I looks mostly organized. One thing I noticed, I only see an HTML file and a CSS file in this repo. What about your sinatra server? Unless you have other plans for this, i would include all the files that make this site work together in the same repo.


hm exact didn't read the call closely enough


Hello Valentin, may I ask you why did you choose that peculiar
`+ (+steppedDifference)`
notation? It frankly seems rather pointless, given that it's applied to the output of parseFloat, at least it didn't break any tests once I replaced it with simple
`+ steppedDifference`
It also happened to break my build, which is naturally my problem only, and obviously that build process can be improved, but it kinda makes this expression potentially dangerous while bringing no evident value.


Do not push


@statwonk I think I've captured the answer to your original request on the wiki here:

https://github.com/jupyter/docker-stacks/wiki/Docker-Recipes#using-pip-install-in-a-child-docker-image


- 2.7除夕划水练习


Thank you


don't use trys here, and never catch assertion errors.

instead, refactor to read:

if not (1 <= month <= 12): return False

You'll find that you have a lot of ifs returning booleans, and can refactor by chaining them together, such as

return not (1 <= month <= 12) or (not 1 <= day <=31 if month in month31 else not 1 <= day <= 30) for most of the month checking, meaning you'd only need one more weird check, probably above this line,  for feburary.

Use whitespace and function decomposition to make it readable, such as creating a function 'valid_month(month)' and another 'valid_year(year)' as well as 'valid_day(month, day)'


> And you are overriding it, on the reconnect. Which I say we shouldn't.

How else would you suggest we loop?  Roll a new connection method for it?


Hi,
there may be a small typo in the quote of Jones role, which leads to an error.


I think I create new release of luacov-coveralls in few days.
May be make `dkjson` as deps in rockspec instead of `lua-cjson`?
Fix of cjson is quite easy but bug still there :(


I got it now, this is done for implicitly throttle when populating the cache. In this case just a comment works for me. :+1:


nice


un commit


on the named groups


Hi Maryam, was there a particular reason you placed "sys.setrecursionlimit()" in the ifmain statement rather than at the top of your programs? Just curious. Also, your descriptions for Tasks 3-5 are all the same, and maybe they could be more informative . . .


Do you think we maybe should create some shims or links to these in `/usr/local/bin` maybe like `pip2` and `pip3` to make this a little clearer?


omg i know, right?


We only need to wrap with `Suggestion` for old string based API inside `data` method.


If we're moving this around, we could also put these docs on the respective field in the struct so the docs render next to them.


Nice organization, and it looks like you set yourself up to make media queries for different sized devices. Good job.


Delete the line


But you can common out these regex's - you repeat the same regex rules multiple times. Maybe introduce a single global regex rule that all these functions can use - some will use it for validation, some will use it for parsing and conversion.

Use of named groups in the larger global regex will keep it readable.


We only need `convert` here if we expect `data` may return string.


this is good!


Cool! No obvious issues and i like the proper indentation, very important. One note, I only see the html and css files, when pushing this live we'd want to include the sinatra server file, or the rails app (if you want to go that route).


113360729_HumidifierOASys (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2093 of 2093 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-100.0%25-green.svg)


@stomar: :+1:


Indeed you are right: it's useless. I use + in front of some variables to cast them to numbers (when they might be strings) but here, since `steppedDifference` is the result of `Math.round() * this.step`, it's already a number.

Can you provide a PR to fix this?


great tests!


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


also great tests


Cool, I don't see any issues and this is mostly organized. Adding media queries can help make your site more responsive across devices of different sizes.


or - или


Good indentation and i like that you're messing with font-awesome. Good job


@aw3s0me Should this not be the URL of the current installation?


Yes. Using dkjson is a good idea. Waiting for a new release of lua-cjson until someone else picks it up will just delay things.


Info : Le moteur tourne dans un sens pendant 5sec puis il s'arrete et reprend pendant 5sec dans le sens inverse


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


I find it disputable if adding `.npmrc` even to private repositories is a good thing.

E.g. `@livingdocs/server` has a readonly token checked into the repo. If I want to create a dist-tag I need to have a read-write token in the same directory as the `package.json`. Which then collides with the tracked token.

@masone This is related to what we talked about earlier.


哈哈哈哈哈哈哈哈哈


Correct. This goes with your other comment about copy/paste


I rebased and merged this for you in #362


Absolutely. Adding them to the READMEs for the images that have both py2 and py3 will help as well.

Would it also help to have conda2 / conda3 shims or does that just get confusing?


Hi Andre,

i am not sure but i remember that i got error when i put it just above the import sys. i could not understand your question about number3-5.

________________________________
From: Andre Moncrieff <notifications@github.com>
Sent: Tuesday, February 9, 2016 9:43 AM
To: biolprogramming/assignment-6
Cc: Maryam Foroozani
Subject: Re: [assignment-6] assignment 6 (4271caa)

Hi Maryam, was there a particular reason you placed "sys.setrecursionlimit()" in the ifmain statement rather than at the top of your programs? Just curious. Also, your descriptions for Tasks 3-5 are all the same, and maybe they could be more informative . . .

-
Reply to this email directly or view it on GitHub<https://github.com/biolprogramming/assignment-6/commit/4271caa80aac55fcef6fd81be737639786b5dbbb#commitcomment-15980944>.


This reverts part of LUTECE-1954. Is this intended ?


Dear @druxx,

we're facing a problem with this commit, see #186 - could you please comment?


We may also combine this with Suggestion shim, which I originally added for API compatibility only and to be removed in 2.0.

It might be OK if we have private method data conversion for Array/Object/String, but initially I thought it would be useful to have default data conversion for Array/Object/String cases with public API, so we can change them if necessary.


This is done mostly because SQLite has very light support for schema modification.  For tables likely to have a lot of data, it can make more sense to write dialect-specific code.


ok, nicer yes


The intention of the design is that any calculated LockPoints would always be valid until the reorg removed maxInputHash.  The way the code works is this would be true because if you had a relative sequence lock  > 0  (time or height based), that was based on an unconfirmed input, you would fail EvaluateSequenceLocks and the LockPoints would not saved.   Relative locktimes of 0 are sort of a special case and don't need to go into the computation of LockPoints.
I believe the code works correctly as is, but it is not going to be quite as seamless as I thought to allow the mempool to carry txs which are locked slightly into the future.  I'm going to see if I can fix that and make this code a little clearer.


Note: the is at least one required code change; stack had one overlapping instance: `(a, b)`. Which have to be tagged so for ghc 7.10. I'll try to release binary-tagged ASAP so you could verify it properly


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


Isn't that CVE-2016-1521? Source: http://www.talosintel.com/reports/TALOS-2016-0058/


you can use generator expressions here:

```
install(FILES "${QT_PLATFORM_PLUGIN_PREFIX}/qwindows$<$<CONFIG:DEBUG>:d>.dll"
    DESTINATION ${CMAKE_INSTALL_SYSTEM_RUNTIME_DESTINATION} (
```


We can make `data` method optional and skip this and the following line if it is `null`/`undefined`.


I think Lucas tried it, too.


It's OK where it works - autoloading can be delicate.  In many cases all SQLAlchemy needs to know is "there's a table with this name", in which case it's easiest to just give an empty table definition.


I think that this could be expanded to take any type of iterable, though I might need to change the name of the project! :laughing:


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1352](http://teamcity.monogame.net/viewLog.html?buildId=14693&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:20:22


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1353](http://teamcity.monogame.net/viewLog.html?buildId=14697&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Here `text` is a `{ label, value }` object wrapped with `Suggestion`.


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


Would it make sense to parameterize this function on table name and just call it with each of "connected" and "configured"?


113360729_HumidifierOASys (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (959 of 959 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-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-28.69%25-green.svg)


Oh, I see the difference in ForeignKeys now - never mind


So the good thing is that now all methods including this one have the same API.


This commit fixes the test repl_conformance_lookup.swift (listed in the commit message)


A fix for this has been merged in cakephp/cakephp#8211.


Sounds good.

Hmm...it seems to me that `conda` is already environment aware, but maybe we could do a better job pointing out how we have constructed the environments in places where it is less clear.


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


[R/iri.R:20:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L20) *style:* **Variable and function names should be all lowercase.**
```r
CalculateIRI <- function(profile, iri_coef, segment.length = 100) {
^~~~~~~~~~~~
```
[R/iri.R:23:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L23) *style:* **lines should not be more than 80 characters.**
```r
  K <- max(2, as.integer(.5 + .25 / DX) + 1)  # number of profile points used to compute mvg avg slope input (window)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:31:5:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L31) *style:* **Commented code should be removed.**
```r
  # num_segments <- length(profile) / num_samples_per_segment
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:32:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L32) *style:* **lines should not be more than 80 characters.**
```r
  profile_segments <- split(profile, ceiling(seq_along(profile) / num_samples_per_segment))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:39:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L39) *style:* **lines should not be more than 80 characters.**
```r
    # sliding window of profil elevations for calculating mvg avg slope (buffer of length K)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:52:5:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L52) *warning:* **local variable ‘iri_df’ assigned but may not be used**
```r
    iri_df <- data.frame()  # df for colleteing return values
    ^~~~~~
```
[R/iri.R:61:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L61) *style:* **lines should not be more than 80 characters.**
```r
      if (length(profile_segment) >= ix) { # if slope could be built within semgent
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:61:42:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L61) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (length(profile_segment) >= ix) { # if slope could be built within semgent
                                         ^
```
[R/iri.R:63:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L63) *style:* **lines should not be more than 80 characters.**
```r
      } else if (length(profile_segments) >= next_segm_id && TRUE) { # if slope needs to be built with samples of next segment and there is a following segment
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:63:68:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L63) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      } else if (length(profile_segments) >= next_segm_id && TRUE) { # if slope needs to be built with samples of next segment and there is a following segment
                                                                   ^
```
[R/iri.R:64:52:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L64) *style:* **Put spaces around all infix operators.**
```r
        y[K] <- profile_segments[[next_segm_id]][ix-num_samples_per_segment]
                                                  ~^~
```
[R/iri.R:65:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L65) *style:* **lines should not be more than 80 characters.**
```r
      } else { # if slope needs to be built with samples of next segment but there is no next segment or no following sample
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:65:14:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L65) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      } else { # if slope needs to be built with samples of next segment but there is no next segment or no following sample
             ^
```
[R/iri.R:80:12:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L80) *style:* **Put spaces around all infix operators.**
```r
        y[j-1] <- y[j]
          ~^~
```
[R/iri.R:88:32:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L88) *style:* **Commas should always have a space after.**
```r
          z[j] <- z[j] + ST[[j,jj]] * z_last[jj]
                               ^
```
[R/iri.R:91:26:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L91) *style:* **Put spaces around all infix operators.**
```r
      rs <- rs + abs(z[1]- z[3])
                        ~^
```
[R/iri.R:97:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/iri.R#L97) *style:* **lines should not be more than 80 characters.**
```r
    # determine avg rs by dividing by number of samples per segment and attach to result vector
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/rroad-package.R:30:1:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/R/rroad-package.R#L30) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-iri.R:7:10:](https://github.com/vsimko/rroad/blob/c8d7f8bd05d80a438d479af1ff467b70d558cd3f/tests/testthat/test-iri.R#L7) *style:* **Variable and function names should be all lowercase.**
```r
  iri <- CalculateIRI(profile, IRI_COEF_100, 20)
         ^~~~~~~~~~~~
```


Currently this is a hack to make something work at 14$/month, what would otherwise be 7$ per user.
We could introduce more tooling like https://github.com/deoxxa/npmrc to make juggling tokens and npmrc files easier, or go for the upgrade.


Перезаписаны последние изменения (коммит был сделан после твоего). Ты же откатил его обратно. Не надо было просто перезаписывать файл upgrade.sql (заменять на свой), а вручную указать свои изменения


put in pull request


I think we can leave this comment out -- if this becomes a problem with newer SQLAlchemies (and I don't see why it would) it's easy enough to grep for.


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


Ooop, I must be asleep. Thanks! Fixed.


instead of duplicating each of these methods, it would be preferable if we called into the base implementation. This will lead to less code duplication between this class and it's base class:
```C#
public override decimal GetMaintaenanceMargin(Security security)
{
    return base.GetMaintenanceMargin(security) * GetMarginCorrect(security);
}
```


Ok, thx


Was this before or after adding rHist?


WOW!
What's the URL to see your work?


The bottom of each README has a section about the conda environments. For example:

https://github.com/jupyter/docker-stacks/blob/master/datascience-notebook/README.md#conda-environments

But it can certainly be improved. Let's take this over to an issue or PR?


This is good.  I would use half-life with subscript like this $\tau_{1/2}$


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


“As an example” sounds a bit odd here in the context of the previous paragraph. We speak in quite a firm tone about integration and then present the integrated tool merely as an example.

Maybe something like “You can see this in action in Juttle Engine, our integrated environment…”?


I think this is a better idea.  I can see such references arising as beta users manually clean out DB tables, etc., and I think they would prefer to have Buildbot quietly make things right, rather than require manual intervention.  The builds table did not exist in this form in 0.8.12, so production-release-only users will not be impacted.


:cry:


I wasn't sure what the right package is so I placed it here.


I like the way you've factored this!


Is there more idiomatic way to construct the target URL?


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


"Schema"


`./examples/sqrt_main.cco` is not versioned, though.


you could cast your grandmother to a `QPalette&&` and the compiler would accept this.
f msvc doesn't allow the user-defined cast operators, maybe the type will have to be passed as explicit template argument to a variant of `QtCollider::get`.

fwiw, it smells like "C2440 can also occur for an incorrect use of a user-defined conversion" in the link above


Upstart scripts for #4


Here one could add logic to select a default client to return a redirect URL to... e.g. based on a client attribute or naming convention etc.


You could instantiate a no-o observer here so you don't need to do `if (observer) {` like we have done in other places.


![ocd](http://cdn.meme.am/instances/500x/41721590.jpg)

Thanks...


Ah interesting! Some questions:

1. Does that his work only for ES7? I had a check in there, but I can't seem to find one in the new code.
2. How are parentheses handled here? I'm sure it's behind the scenes, just curious to know a bit about it.


Yes, I only left this to be able to keep going. Would be great if you found a solution!


Done


This will fail for cases like `IchimokuKinkoHyo` which have a mix of `IndicatorBase<TradeBar>` and `IndicatorBase<T>`. It's kind of tough to dig in and get the real generic parameter, maybe it's sufficient to just check three times, `IndicatorBase<T>`, `IndicatorBase<IndicatorDataPoint>` and `IndicatorBase<TradeBar>` - that will cover all of our current use cases and probably 99% of future use cases as well. When we start using indicators with inputs other than tradebar/indicator data points we can make this a little smarter and crawl the inheritance heirarchy to find the correct generic type parameter and invoke the correct generic method


where do you use this method ?

anyway there is way how to read files in executor.

with h.executor().session() as ss:
    with ss.open_file("/tmp/aaa", "r") as fh:
         return fh.read()

anyway if you want such function, I would prefer start with 'open' to align with 'os' module.

and last I don't like that it ignores errors


Ah OK because in SQL only ' is allowed therefore only '' seems to be interesting. Your test covers the '' problematic :)


This code is removed again in https://github.com/michielbdejong/kinto-chrome/commit/04837b4ea361d1d0135fdeb29d946ed910127a66#diff-a7c88e4bf351928799b9036ef131dc57L207 but I'll apply the pattern in general in other PRs.


Better?
https://github.com/robinandeer/puzzle/commit/ded817158bddbd20723485219da3d78316c0f987?w=1


We could move the `ConstantIndicator<T>` check above the if:
```C#
if (field.PropertyType.IsSubsclassOfGeneric(typeof(ConstantIndicator<>)) continue;
```


Hi! Could you please tell me why `template_name` was not compatible with djrill?


@nzakas Your version check is left in previous commit, this one seems to be just adjustment on top of it: https://github.com/ternjs/acorn/commit/a8ce3a12394a454484edb8c66ac697532ffcb5f7#diff-3d557214191711f13dd9000000c8f305R237


Why IllegalState here ?
It is problem with argument, isn't it ?


@Martin-Wegner

Thanks a lot for all the help and the code review


What about Edge's properties comparision ?
There is a lot of properties.

What is the goal of equals here ?


I just want to make a note on the folder 'portfolio.html' we don't want our directories to have filetype endings, it can confuse others. Also, when creating a sinatra app to host this, we want all the html, css, and images in a public folder. Just a note.


Why isn't `Context.Rel.map_constr` used here?


This HTML seems good to me. I see some comments separating areas. Good indentation.


Nevermind didn't read properly. Not sure where we calculate white auto shots, considering its technically a spell


Sweet! Progress status!


@jasnell @TheAlphaNerd please consider this commit (already in 4.x-staging) for inclusion in the next 4.x release. It contains a fix for a race condition during npm install on Windows that has proven fairly severe (https://github.com/npm/npm/issues/9696). Thank you.


About the same time, but I cannot recall it precisely.


Although We can add the following code after line 556, to use `f_original` before `f = trigsimp(f)`.
`elif solns is S.Complexes:
        return S.Reals`
Then that assertion error won't raise.
What one should I do?


This is not a [json](http://jsonapi.org/) response.


Don't see any styling, not a big deal, I'm sure you're aware of this.


Perhaps we have to add an exception in SPELL_DAMAGE_CLASS_RANGED case for autoshot/shoot spells and add the rule


Small spelling mistake in commit message "taget".

Also just to confirm, this changes the install folder to `/usr/local/lib` rather than `install/usr`? It might pay to include what the change is in the commit message as well just to be clear.


This probably won't need glfw and boost


should probably reverse this accidental change...


I remember you mentioning why this was done, but would be cool to add to the changelog.


Save the output of the replace.


@RReverser thanks, I just saw the one above removed and forgot about the other one.


I am little afraid that these True/False will lead to same problem as we had in 'utilities.machine' module, and also in whole low/high_level test actions ...

I would bet that 'ip link set up X' produces some error message, and maybe some specific RC codes what went wrong. I would be nice to create exception taking type of error and message accordingly. it allows you to control flow of program you base on such library.

you always relay on logs, but program flow doesn't read logs to find out what was problem ... we have exceptions for this purpose.


hmm this isn't really needed in the Java API. It's only needed for scripting...


I am ok to include new methods in network module, but I am not sure about filesystem.


We should remove this `cat` because the config contains a password.


I see! Thanks for pointing that out.


Should be fixed by this:
https://github.com/square/okhttp/pull/2330/files


Should we specify the version?


Doh :D Fixed now.


Yeah Readable would be fine I think. Just FYI, I was imagining that we would split this repo into two packages: one for a versioning system on top of Aeson and one for a set of utils for Snap that enable higher level, REST oriented operations.


Ok, for `GetMaintaenanceMargin`...
In `GetInitialMarginRequiredForOrder` case, I need to account `orderFees` without margin:


what is this supposed to match?


Cred că asta se aplică doar la view-urile de Django, nu și la /static/css/main.css sau /static/js/main.js


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1353](http://teamcity.monogame.net/viewLog.html?buildId=14697&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:20:17


Yep, slight change there. If you're grabbing from the web each time you can get the web installers as well, which will parallelise download with install.

The PowerShell command you'll want is (should be - working from memory here):
`start-process -wait python-3.5.1-webinstall.exe "/quiet", "TargetDir=C:\Python", "InstallAllUsers=1"`

`pip` is included, but re-running `get-pip.py` is an easy way to upgrade - the bundled one is basically always out of date.


duplicated white-spaces:
```
preferredDuringSchedulingIgnoredDuringExecution  from podAffinity
```


unnecessary white line


And there's a lot of stuff that we have in Application that would (or at least could) move over into the snap-rest one. `Param` being one example.


hey, this newline is unneeded, that's also why GitMate comments on your commit :) if you just run `./coala` from the project root it should even automaticlaly fix that for you


Hmmm.... i hate xibs.


@mchandschuh  I just realized there are a few existing indicators that do not have the `ResetsProperly` test method, I'll add the missing ones and ensure they pass (probably will need the changes you suggested).


Even though the output string is in json format this response is still an html response. You need to set the `Content-Type` header to `application\json`. It can be done using `flask.json.jsonify` or by returning a tuple like:
```Python
return json.dumps(dict(todo)), 200, {'Content-Type': 'application\json'}
```


good organization, not too overdoing it with the styling. good job


Probably not understanding how this works.  I'd imagine mynetworks_style = host results in a list of /32 (V4) or /128 (V6) entries - which I thought was what we were getting already, which was the source of the bug?  For 8.6 we were getting things like: 127.0.0.0/8 78.31.107.98/32 [::1]/128 [fe80::]/64 which I thought would be more in sync with mynetworks_style = subnet ?


Didn't take.


Cool, so the indentation looks good. Make sure the <title> in the <head> is more descriptive. This is what Google's search bots check to know how and where to rank your page.


t


@icehess I know we are all guilty of this, but please try to make commit messages more useful to other readers. 'Fixed syntax on log line' or something to that effect.


to take topology key into consideration, you just need to use `node.Labels[softPodAffinityTerm.TopologyKey]` as the index of each count instead of `node.Name`.

```go
topologyKey := weightedpodAffinityTerm.TopologyKey
if node.Labels != nil && node.Labels[topologyKey] != "" {
	index := node.Labels[topologyKey]
	counts[index] += weightedpodAffinityTerm.Weight
}
```


I've done this before, used a bunch of <br> tags. But it's ugly. Use CSS to create the desired space.


They're probably going to freak out :-)


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


kk


Wtf??
Our cam is 3M


This is the only thing added to this file


I wonder if these lines are causing the timeout/fatal error during a fresh install?


One last thing, no .DS_Store files! Abandon Finder, it will make you a better developer.


I take it back, actually if you don't wanna a cache you can simple instantiate just the `OnlineFileSource`.


landscape.io reconnaît bien les imports servant de raccourcis, à condition qu'ils soient déclarés dans __all__


I guess this is not complete yet? Calling release() will result in only the first name from -releaserepo being passed to this method. That is, name here will always be a single name.


I'm currently rewriting this part (database cleanup) and will provide PR against master with cleanup changes.


rename this as it's checking more than being post-SPA?


I'm so sorry, I accidentally stopped the test, while trying to modify threshold :(


I even tested this with Arabic and it worked! :-)

All we want is for json_encode not to choke and your `mb_convert_encoding` trick seems to work okay for that


Copy paste error!


And this if branch logic can be moved into the one before it, so the final version would be like:
```go
topologyKey := weightedpodAffinityTerm.TopologyKey
if node.Labels != nil && node.Labels[topologyKey] != "" {
    index := node.Labels[topologyKey]
    counts[index] += weightedpodAffinityTerm.Weight
	if counts[index] > maxCount {
		maxCount = counts[index]
	}
}

```


We are making a contains check over the with Edge object which requires equals/hashcode to be implemented correctly.

Yes, we should also include those properties


could change :dancer:


I think this is a mistake here. You want to pass the full property value in since release(String,boolean) will parse the String into multiple names.


If you aren't using it, don't commit it.


Thanks for your notification.


Can we get a bugfix release?


this code doesn't do anything, try and not commit this type of code.


LOL thanks... you guys rock!


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


@rschmukler Wouldn't it make more sense to export the `PHANTOMJS_BIN` environment variable? This would be more dynamic for any `phantomjs-launcher` updates?


m_ scripts are used for one off testing when you don't have moodlecron running morsle yet


@jorisberkhout @ChaelKruip I have added the NL `dhw_normalized.csv` hot water profile to all regions, otherwise the merit order would not run correctly (P2H would look for the profile and not find it).

I realise we might not have country-specific hot water profiles yet, but wanted you to be aware that I've done this to ensure that the other regions can still be used.


I'm catching up! LOL


Middleware-ul adaugă header-ul Cache-Control cu valoarea respectivă la toate răspunsurile HTTP întoarse de aplicație. Am testat și se face refresh la css/js.


float?


Minor: variable name looks strange


I feel like I was forgetting something... `2.3.0.4`


needs a white line before this comment


TeamCity Civic Assembly :: Build backend + frontend [Build 86](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=903&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


https://github.com/anubis187/maven-jrebel-plugin/commit/bc0435c0a64bb596de1e4f5311517855a7e4418d#diff-600376dffeb79835ede4a0b285078036
https://github.com/anubis187/maven-jrebel-plugin/compare/patch-1


LGTM.


Cribbed this from the `.application` class


Let's remove output from the test.


just for my C++ education:

is this the constructor `std::vector<T>(first, last)`?


This is the only thing that changed in this file. Everything else is indentation


Yes, it is
We need to also change https://github.com/hazelcast/hazelcast-jet/blob/41e51b90ecf509119585b0f92b59289874240a7e/hazelcast-jet/src/main/java/com/hazelcast/jet/impl/dag/DAGImpl.java#L56 to `IllegalArgumentException`


same as above, should take topology key into consideration.


good organization and no obvious issues. I see some transform elements, nice! To take this even further we can use media queries to make this responsive across devices with varying sizes


Thanks Gabriel. Just to make sure this does exclude nested `_build` dirs right ? I think it's the only thing you want to watch out for.


Great indentation, very important to having readable HTML!


test


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/9083bdc3e5f96c5a0f5d64760510117db5ac129a


merging basho/riak_kv/rdb-kv679-moarlog = 9083bdc3 into borshop-integration-1344-rdb-kv679-moarlog


basho/riak_kv/rdb-kv679-moarlog = 9083bdc3 merged ok, testing candidate = 8bd2df9a


Skype id: mubashar_khan


nothing to to there!


@temoto thanks for the 0.18.2, but was there a strong reason to pull 0.18.1 from pypi? the 2 commits after 0.18.1 do not seem to be that bad?


Probably warning is reported here.


TeamCity Civic Assembly :: Build backend + frontend [Build 87](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=905&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


reknt


Thanks


TeamCity Civic Assembly :: Build backend + frontend [Build 86](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=903&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:38


Bit short on a setup to try, sorry. @jonathangray can you share some light please ?


So when we want this site to go live using a sinatra app, you'd want to have all your HTML and CSS inside of a public folder, and your images within an images folder inside that public folder. Right now I just see the HTML and CSS files. No worries tho!


Is assert forgotten?


also is `char const` == `signed 8 bit int`?

why not return `std::vector<char const>` if `dht_pkt_alert::pkt_buf` returns `char const*`, to avoid any possible confusion with string encoding? to sort of tell the programmer, hey, this is an 7/8 bit ascii.


Seems like you could avoid this allocation entirely and just write 'N's directly into `bases`


Fixed


Can a non-recursive function implementation (like I suggested one in [a commit on 22. March 2014](https://github.com/elfring/coccinelle/commit/6566af7dc51b9fcc7a5ee21728fb36bbeedb0bd6 "Bug #5: Support for file display with special characters in the name")) be executed a bit faster?


@DropsOfSerenity i need your input into this. Check out the [`saveImageToServer`](https://github.com/Core-Tech-Labs/LAZ/blob/26f3e6092b03e328742c2dd71ffdf864c8c6a6b3/resources/assets/js/Avatar/App.js#L28) method i added to the Avatar Cropper using ajax but its not saving the cropped image or any data I want.

What I'm I doing wrong here?


close #128


Shall we put here sth like:

```
	AfterEach(func() {

		Expect(cf.Cf("delete", appName, "-f", "-r").Wait(DEFAULT_TIMEOUT)).To(Exit(0))
	})
```
?


Should dev's RAILS_ENV stay as `production`?


Probably be good to say that they're an old grouping of policies, which we're expecting to deprecate soon.


After this merge, track size estimation is no longer reported on argument "-F".
Is this intended?


good organization, no issues. I like the use of transitions and specifying the browser prefixes. (-webkit-transition, etc.) To take this further use media queries to make the site responsive across devices with various sizes


I'm seeking the downloads for the kit and found myself here but still no downloads.I am not well versed in computer speak nor in many ways to use it,but learning very slowly and trying very hard.


crap


what?


Be careful with the indentation, there's no issue, it's just not good practice for CSS


Because of garbage collection and such it's hard to compare numbers with such a small margin (2-5%). In the recent performance tests and optimizations I was working on I was looking for changes closer to at least 50%... and in the recent batch the performance of find.one() with and without cache is about 3x (300%) as fast.

I'm not sure the separate thread approach will stay, they are other potential issues with it. Destroying the EC explicitly is a good point, I'll make that change right away. That will hopefully help with an intermittent issue I saw with the AssetReservationMultipleThreads tests. Using an explicit transaction may help with that too. Thank you for reviewing this and for the feedback.


I think newQuals is unnecessary since getBaseQualities() already performs a copy.


TeamCity Civic Assembly :: Build backend + frontend [Build 87](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=905&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 148 (1 new), passed: 24, ignored: 4 Build time: 00:07:04
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455035475787: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

##### there are 138 more failed tests, see build details
```


Can't we just remove these if they will no longer hold with flexible keys?


Should add the kit is the ciclop3D scanner kit,now back to searching for the downloads,I accept any advice and help with gratitude.


@brandocalrissian great question.

yes, `RAILS_ENV` controls things like asset compilation, logging, etc. We want all our cloud.gov installs to use `production` in order to exercise those features. The exception would be if we wanted to introduce a `config/environments/staging.rb` file if we needed to toggle some config that we wanted to exercise in one cloud.gov env but not another.


Hi,

I am utilizing the generated SDK's in the jasmine unit tests.
But when a 500 error is received from the API, then SDK starts retrying for the request.
Also there are 4 parameters that are being set internally in ms-rest code i.e.:

DEFAULT_CLIENT_RETRY_INTERVAL
DEFAULT_CLIENT_RETRY_COUNT
DEFAULT_CLIENT_MAX_RETRY_INTERVAL
DEFAULT_CLIENT_MIN_RETRY_INTERVAL

I want to know, how can i set value of these variables in my test scripts using the 'options' argument in the SDK method.

For eg: getdata('name of url resource', 'options', 'callback'). Let's say, this is my SDK method.

In above method, currently I am sending options as :

options = customHeaders : {
                            Authorization: 'bearer  authcode'
                             };

So how can i send the retry count or interval along with this options argument so as to control the request processing time ?

Looking forward for your early support on this.

Regards
Simran


hmm


# MyProgrammingLab_Solution_Java
Solutions to practical Lab Questions on Java


If you're using these li elements for spacing purposes, use CSS instead, it's much cleaner


I'm not put it in server, because it not work well now.
Just print something in the screen.
All contain reference from [here](http://www.nodebeginner.org/index-zh-tw.html)


Yes you are right, equals and hashCode are required because of HashSet.
But I think we need compare other properties as well


They will pass with flexible keys. I have broken out these changes from that work to guarantee it for 1.2, they will be uncommented for flexible keys and are spoilers for 1.3.


Cool, thanks! I'm still getting my head around the different ways the app adapts to environment variables.


Good

But putting it on server is a better way to learn, isn't it?

You can screw up, and learn from it :)


Be careful with indentation and spacing. It looks mostly good across the page, but these unnecessary spaces and improper indentation make reading HTML (or any code) difficult.


This is already fixed in the current master branch but thanks for having a look anyway.


@welvinyu Make a few useful pull requests (The amount needed will be based on how much and how important the content is) then you will be possibly added to the organization by @DefinitlyEvil


Dan,
I had a discussion with ssh maintainer Jakub Jelen, and we ended up with that getattr si enough. Here is patch which caused the issue.
http://pkgs.fedoraproject.org/cgit/rpms/openssh.git/commit/?id=af94f46861844cbd6ba4162115039bebcc8f78ba


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3195


Suggest: This technique requires that all functionality continues to be available when zoom is applied.
(Or something that says when it applies.)
Also, "Most desktop web browsers", mobile UAs don't.


:tada:


@arter97 https://github.com/rg3/youtube-dl#bugs.


See bf9d5428c53817cceae788877682739ca8f56a4d.
It seems that there is a reason. You can ask @k-takata for help about windows.


On Tue, 9 Feb 2016, Markus Elfring wrote:

>
> Can a non-recursive function implementation (like I suggested one in a
> commit on 22. March 2014) be executed a bit faster?

It's not recursive.  OCaml is a properly tail recursive language.

julia


Initial thoughts are that @jnurthen s suggestion is good.


I like that your commits are for small changes on only one file. This is good practice. However, you should not have a repo with multiple projects inside them. The portfolio site should be it's on repo. It's not good practice to have 'general' repo. Make them each specific to their project.


LGTM.


```
libglvnd $ mkdir build && cd build
build $ ../autogen.sh
Preparing the libglvnd build system...please wait

Found GNU Autoconf version 2.69
Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/\${ <-- HERE ([^ \t=:+{}]+)}/ at /usr/bin/automake line 3936.
Found GNU Automake version 1.15
Found GNU Libtool version 2.4.6

Automatically preparing build ... Warning: autoreconf failed
Attempting to run the preparation steps individually

Preparing build ... ERROR: aclocal failed

```

The deprecation warning is a due to a perl update. Not sure when the autotools people will update things on their end. That aside I might be missing something, yet with this approach/setup I can build pretty much everything X related.


cc. @bcamper. You might want to check on js as well.

@hjanetzek pointed out that it would be better to extend the tile bbox instead of shrinking it for "lineOutsideTile" test. I think it makes sense, and will avoid clipping of an actual boundary line.


We haven't anywhere else in the pipelines, so I didn't here.  It probably makes sense though.


I wasn't sure if this was a bug or not.
Ok, I'll open an issue ticket.


This should just be "rm -f", not "rm -rf". There shouldn't be any directories that end with .pyc.


Alright

[here](http://bemg.csie.io:8888/)


Бля, лол.


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


LGTM. Note that according to the ticket, "this work should be done as a pull request against the import-5.0-xml-into-postgres branch, not against master."


Nicely spotted. Will update the whole branch as we get confirmation on the AM_PATH_PYTHON question.


113360729_HumidifierOASys (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1511 of 1511 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-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.71%25-green.svg)


5382_PoolMakeUpWater (mitchute) - x86_64-MacOS-10.9-clang: OK (2053 of 2055 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: Table big diffs.

![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)


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


So I don't see a closing tag for the head element. It should close before the opening body tag. Also, the title element belongs in the head tag, not the body. Lastly, the title should be descriptive about the site. This is what Google's search bots use to rank your page. "Code Your Way" doesn't let a robot know that is your portfolio site!


You'll need to reset this file back to what it originally was before merging it into dev. The gradleHome value should not be statically set as it will cause problems when the project is built on other machines. For example, on my laptop, my android studio installation is not in C:\Program Files\Android\Android Studio.


later we're checking the `synchronous` variable to see whether to execute the callback synchronously or not. I think we should also do that same check here.


LGTM.
(These areas are not test at all. xcmd, I introduced may break them. But this is different topic.)


way* typo in commit :(


I don't think we need to have these Functor applications. The built-in `Hashtbl` module uses these comparison and hash functions.


Ditto about the `Hashtbl.Make`.


I don't have separate files for reformat and changes; splitting it up manually would be a waste of time... next time if I will do as you say!


Make sure you set the elements correctly, above ^^^ should be 'width' in lowercase.


Thanks, but i don't know why @syedhali didn't merger this


@galenwilk Can this be abstracted as an input parameter so we can have test and production db instances passed in?


@diracdeltas corrected that.


It can be confusing if we don't make sure we indent our code correctly. The above properties should be indented one level in.


@Tabjones  Cancer-less GP of the squared container...

![cancer-less](https://dl.dropboxusercontent.com/u/16526900/squared_shape.png)


A suggestion; just send a pull request and let the official maintainer handling the versionning. This change seems useful for other people.


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [source/_views/default.html](https://github.com/pantheon-systems/documentation/commit/45fb607480c9f7e4e1b225c6c9f9b24fa8ee2a1d/source/_views/default.html)
- [source/docs/assets/js/src/application.js](https://github.com/pantheon-systems/documentation/commit/45fb607480c9f7e4e1b225c6c9f9b24fa8ee2a1d/source/docs/assets/js/src/application.js)


I like the organization and you're not overdoing it with the styling, good job!


this is a work for human


Do you really need `CloudFunc` and `Util`? This files should be in commit if you need them


`iocmd`, I think


Good point, fixed.


Where you use this function?


Yes, that's the constructor.

char is a signed 8 bit and it's used like that. Using int8_t allow swig to properly put the `byte` type in the java interface, not `Char` that in java, is a 16 bit type.


You do not write json


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


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


Remove all unused variables, please


TeamCity DNN JavaScript Libraries :: CI [Build 143](http://teamcity.engagesoftware.com/viewLog.html?buildId=421&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 143](http://teamcity.engagesoftware.com/viewLog.html?buildId=421&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:17


Poor show.


Remove IIFE


remove comment


remove comments


"Aktivieren Sie die Protokollierung. Logs werden lokal gespeichert. Sie können sie bei einem Fehler mit dem Fehlerbericht mitschicken."
Cf. the English version, https://github.com/Steindi01/google-calendar-crx/blob/master/src/_locales/en/messages.json#L163


excellent, thanks!


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1522 of 1522 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-100.0%25-green.svg)


How about introducing maxTagsLine () like numTagsTotal and numTagsAdded?


Interesting. The kernel function could slightly generalised at probably no cost to something like
POSITION_IDENTICAL(list, obj [,skip, start])

find the first i in [start, start+skip..Length(list)] such that list[i] is identical to obj.

This might be useful in a bunch of related places.


I think that a better thing to do here would be to navigate to the modified element in the editor, so that the user could see how exactly it was modified - this would make the notification unnecessary. Also note that we fail silently if we are unable to locate build.gradle.


Can't wait to see this in a nuget package


//Hello


"Dann erstellen Sie einen neue Fehlerbericht und fügen die obigen Informationen dort ein:"

(Note honorific capitalization of "Sie", and adjustment of indefinite article to "einen".)


LGTM.


I prefer pwg.session.getStatus because:

1) this is already were we give info like "upload_file_types" and that sounds more the same kind of info compared to pwg.getInfos which provides "general statistics"

2) some day, we may have a "maximum size" different for each user


Cool! thanks! :+1:


That's a little more vertically compact, but I'm still not convinced it's useful, since there's no revision attached.


Perfect!


Elle est là \o/


Thank you for your time.


I have created https://github.com/Piwigo/Piwigo/issues/416 for Piwigo 2.9


love


Yes, it's redundand, it can be removed.


I chose not to use member initializers in `Matching::Options` because that would make it non-POD which in turn would prevent usage of member initializers inside `MatchingBase::Options` without a matching ctor in `Matching::Options`.


Well, this func and the local PodLister are temporary bridge. They will be gone in 3rd step where we swap out the cache implementation.


This is the original logic. See [here](https://github.com/kubernetes/kubernetes/blob/1032067ff98ec94515116c5fef9f108541486771/plugin/pkg/scheduler/factory/factory.go#L110).

As a second step, it only changes to use cache interface. But when we use GetNodeNameToInfoMap() we still list all pods from current podlister consisting of assumedPods and scheduledPods in modeler.


tu chyba powinno być H2


but i am not able to join


How will we remember to remove this test in the future when we have other tests in place?


with 0.13.0 and 0.13.1 and 0.13.2


you'll also have to enable the Travis webhook


Done!


remove comment


`Classic`?


you could convert `\` to `/`. Java supports both for the Windows file system.


I mean that someone could remove the initial directory and replace it as we're creating the file.


I'm not familiar with the debugger code. Is it correct to use `parent_env` or should it be `child_env` ?


5382_PoolMakeUpWater (mitchute) - 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)


Those changes are already done in PR https://github.com/c2corg/v6_ui/pull/148
=> you'll get a conflict when one of these PR is merged
=> you need to drop the commit either in this PR or this other


Did 2084234273b26ae0512e160c7831a827961505b1 work before this? If not, I think they should be squashed together


With BidAskUpdated you might need to add symbol with AddSymbol function.
Regarding TradeAdded. Did you test your account with mtm manager?


Sorry for my stumbled writing. I will make them more clear. Let me know if you didn't understand anything.


Velmi dobře, žes to konečně srovnal. :-)


Wow.  Excuse me for this direct commit to master @geoserver/team-geoserver.  In my haste to suggest an edit through the GitHub web UI, I completely forgot to check if it was creating a branch - thought this was going to result in a pull request.

Please feel free to revert if this breaks any process.  Also, I won't be offended if my push rights are removed.  I'm happy just to submit a pull request if I have any other proposed changes.


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [source/_views/default.html](https://github.com/pantheon-systems/documentation/commit/f8908d07e3574358d151ef158af8b050df3651eb/source/_views/default.html)
- [source/docs/assets/js/src/application.js](https://github.com/pantheon-systems/documentation/commit/f8908d07e3574358d151ef158af8b050df3651eb/source/docs/assets/js/src/application.js)


Nice catch to notice the base class does the same thing. (Also a good argument for using `override`, since I didn't even realize this was an override.)


Same reason as above.


protect me!


Make a new class for ingredient and then do the same thing for preparation.


Yeah.  Characters are free. :smile:


Say something about binding energy of nuclear particles


I wouldn't say the fix is required. I don't really know. That's just what I had to do to make it work.


fix #869


This willful violation is still there, just less obvious. Do you think it's worth keeping this note in some form?


Mistaken copy-paste. Already fixed in trunk.


this import should be before


Use a dummy EGL window.

https://github.com/linux-sunxi/sunxi-mali/blob/master/test/test.c


remove me


remove comment


yeah this sounds good :)


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


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


I forgot to remove that.Now it is done.I don't think that test-case is needed in solveset.


Ah, thanks, and sorry for not getting onto that soon enough

![](http://stream1.gifsoup.com/view1/4787838/plate-spinning-o.gif)


This is the APM that stock used
I guess you have done comparing FFOS source with this


I'll try that. Thanks. There are way too many ways to get simple text :). Also do you know any alternative for https://developer.mozilla.org/pl/docs/Web/API/Element/closest ? I need to get closest ancestor of element and this was the only function I've found, but to make it work I had to define manually JSDoc for this and in IE this function is not working.


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


чёрт...


This val should not be public. Either make is private or move into the method.


Thank you for your reply. It is working. My apologies, it just started firing the events correctly after I used the libraries in a fresh project. Thank you for your hard work, really neat job.


Didn't think of the colliding tokens. You're right about that. As a solution for now, you'll need to temporarily remove the npmrc in the repo :/

The alternative to having the token in the repo is to make every developer add it manually. I don't see that's feasible with all our clients. But I agree that it is disputable :)


The reply var could use a slightly different name here.  Just bare like this it has the same sense as response.  reply_to is better imo.  response_address would be better still perhaps.


+1, please. In the current version, shebangs are completely broken because of this.


remove


This is a very nice improvement.


Are you going to submit a patch?


Hi!

I wanted to try the newly merged feature \o/
I used the nightly build of mail app + oC8.2
is it compatible?
I guess not, I received an error on the command line
```
php ./occ
An unhandled exception has been thrown:
exception 'OCP\AppFramework\QueryException' with message 'Could not resolve OCP\IDb! Class can not be instantiated' in /var/www/html/lib/private/appframework/utility/simplecontainer.php:87
Stack trace:
#0 /var/www/html/lib/private/appframework/utility/simplecontainer.php(106): OC\AppFramework\Utility\SimpleContainer->resolve('OCP\\IDb')
#1 /var/www/html/lib/private/appframework/utility/simplecontainer.php(66): OC\AppFramework\Utility\SimpleContainer->query('OCP\\IDb')
#2 /var/www/html/lib/private/appframework/utility/simplecontainer.php(85): OC\AppFramework\Utility\SimpleContainer->buildClass(Object(ReflectionClass))
#3 /var/www/html/lib/private/appframework/utility/simplecontainer.php(106): OC\AppFramework\Utility\SimpleContainer->resolve('OCA\\Mail\\Db\\Mai...')
#4 /var/www/html/lib/private/appframework/utility/simplecontainer.php(66): OC\AppFramework\Utility\SimpleContainer->query('OCA\\Mail\\Db\\Mai...')
#5 /var/www/html/lib/private/appframework/utility/simplecontainer.php(85): OC\AppFramework\Utility\SimpleContainer->buildClass(Object(ReflectionClass))
#6 /var/www/html/lib/private/appframework/utility/simplecontainer.php(106): OC\AppFramework\Utility\SimpleContainer->resolve('OCA\\Mail\\Servic...')
#7 /var/www/html/apps/mail/appinfo/register_command.php(14): OC\AppFramework\Utility\SimpleContainer->query('OCA\\Mail\\Servic...')
#8 /var/www/html/lib/private/console/application.php(68): require('/var/www/html/a...')
#9 /var/www/html/console.php(68): OC\Console\Application->loadCommands(Object(Symfony\Component\Console\Output\ConsoleOutput))
#10 /var/www/html/occ(11): require_once('/var/www/html/c...')
```

Could you help here? Or should I open an issue?


The correct name of the heroku config is "COMPOSER_GITHUB_OAUTH_TOKEN". You need to change the above, it will not work


Minus in using ExprFinder is that I won't be able to define exactly which directory function is placed incorrectly.
So error message will be the following:
"Directory explorers [MAXDIR, IMAXDIR, MINDIR, IMINDIR] functions are not supported in ..."
I have made appropriate changes in new branch (DRILL-3944: ExprFinder). If you are ok with them, I will merge it into original one.
https://github.com/arina-ielchiieva/drill/commit/ea0167acd073dfd233b73ad9d455a3c091a096f8


Although it's just one line, it's a "copy-paste" from the other version. In case that some one modified the other version, those 2 versions could go unsynced. Should convert strings to columns and call the other version.


Don't love this, but it works. The other options entail code to change the fill color of the svg to white, which is meh. Both methods effectively restrict you from controlling the color of your icons


Oops! removed unused imports


This returns 'undefined'. Instead of changing the value of the _noun_ variable, _returning_ the new value will make it work.


Thanks, Xin!


Oh yeah!


Added commit message, clean up obsolete nvidia 304xx files


Can you explain what this is for?


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/531a2de8ef02751f53c649d453afe852a0d88de7/circle.yml)
- [scripts/deploy-wraith-diff.sh](https://github.com/pantheon-systems/documentation/commit/531a2de8ef02751f53c649d453afe852a0d88de7/scripts/deploy-wraith-diff.sh)


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2091 of 2093 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: Table big diffs.

![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)


This should likely be PyPDF2 here. Note that the build is now failing because it fails to install this version:

```
Collecting PyPDF==1.25.1 (from -r requirements/base.txt (line 8))
  Could not find a version that satisfies the requirement PyPDF==1.25.1 (from -r requirements/base.txt (line 8)) (from versions: 1.4, 1.5, 1.6, 1.7, 1.8, 1.9, 1.10, 1.11, 1.12, 1.13)
  No distributions matching the version for PyPDF==1.25.1 (from -r requirements/base.txt (line 8))
```


It makes script life a lot easier and on Java side it can be seen as a helper (no need to convert whatever String you get in input in use case where that's what you get).


I planned on rebasing before I did a PR, but next time I will do it before I do even my initial push.


Yep, currently it won't be clickable. Dunno if I'll change this, because the focus zoom animation will crop 2/5 of the banner.  With the latest commit the banner will fade out.


Why do we have the loop in the first place? I don't think we need it. Rather just have the single `setTimeout` call after launch.
If a site wants it to loop reconnects they can set that up themselves, but I don't think it is even needed as it doesn't take QZ tray that long to start up, even when it needs to search for ports.


## Hobby
Dessin
Sport


Looks like this now:

![image](https://cloud.githubusercontent.com/assets/4240292/12924164/c12d4382-cf26-11e5-88bf-228ae2ca95ba.png)


@MrDoni98 @seetch :smile:


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


unack, I think this needs a comment so nobody goes through here and shortens it later resulting in bugs


Note: They wouldn't be able to because the tests would fail.


also you'd rather want to do `arg_list is None` instead of `==` right, you want the identity of the None singleton


same, use `is` and add comment


I'm not 100% sure if this is safe. I messed around with something similar myself (though I think I used `rb_id2name` or something) and it kept segfaulting.


Small typo: should be "per-host".


hæ


Nice. Does this same change need to be made on line 138 as well?


Bättre commit-meddelanden kan du!


unack


Was it not possible to rollback to the previous version of the puppet module?


The line breaking here breaks the build. There can be no spaces or newlines between `<code>` and `Document</code>`.


ack


```
double currentDeg = Helper::radiansToDegrees(current);
double newDeg = Helper::radiansToDegrees(target);
if (newDeg - currentDeg > 0) {
    // right direction (clockwise)
   return true;
} else {
    // left direction (counterclockwise)
  return false;
}
```


Языковые курсы.md

разжевывать


If you refer to it this way, will it use all the bindings?
Because, in contrary to what you state in the commit, the facade is binded (otherwise it would throw an error).
Laravel itself (the docs) uses the global space for refering to facades.


Языковые курсы.md

... (полноценное троеточие лучше)


unack, you shouldn't need `--cov` as it's configured in `setup.cfg`


113360729_HumidifierOASys (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2061 of 2061 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-100.0%25-green.svg)


this should've happened in the previous commit right?


This is the wrong place to put it.
Document is using Blittable, not the other way around


same as above


What if you just make it the first list item?


>  I don't think it is even needed as it doesn't take QZ tray that long to start up, even when it needs to search for ports.

It does take long to start.  On Mac and Linux, it varies from 5-20 seconds, depending on system speed.  PC varies from 1 second to about 5.

> Why do we have the loop in the first place? I don't think we need it. Rather just have the single setTimeout call after launch.   If a site wants it to loop reconnects they can set that up themselves, but  I don't think it is even needed as it doesn't take QZ tray that long to start up, even when it needs to search for ports.

Well is it needed.  This is a highly demanded feature and "just hope it connects" doesn't work in this instance.  If you have design criticisms with the solution, please share.  This is a very [similar technique to what is used in 1.9](https://github.com/qzind/qz-print/blob/1.9/sample.html#L158:L166).


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


>It's not recursive.

The function "catline" is recursively called here, isn't it?

>OCaml is a properly tail recursive language.

The OCaml compiler might perform special software optimisations in this use case. So it might look more convenient to express an algorithm in a recursive way.

Would you like to compare the run time characteristics from [my suggestion for a linear function implementation "show_file_content"](https://github.com/elfring/coccinelle/commit/6566af7dc51b9fcc7a5ee21728fb36bbeedb0bd6#diff-a98fee8f4f68c2a7394c4c0efb79f644R289 "Bug #5: Support for file display with special characters in the name") with this one?


`...Pattern.compile(HIVE_TABLE_REGEX)...` doesn't seem right since `HIVE_TABLE_REGEX = "hive.table.regex";`

So with the `dbNamePattern` construction above.


I believe that a directory on HDFS does not take up a disk block as an ordinary inode-based one would - instead, the label is stored in an internal table on the head node.


why did you even do this???


This still doesn't really make sense. The algorithm either returns failure, or returns a {resulting URL record, resulting URL string} pair. If it returned failure, you can't get the resulting URL record. The original formulation was clearer.


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (958 of 959 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-28.69%25-green.svg)


Outstanding


unack, as mentioned before coverage should automatically be measured via setup.cfg m either we want to change that or the docs here


Why do you pass SetMessagesRequest when you only need updates ?


I have run some tests, selecting using the following select:
{code}
select * from dfs.`D:\\drill\\files\\testData` where dir0 = maxdir('dfs','D:\\drill\\files\\testData')
{code}
with enable_contant_folding option turned on and off.
Unfortunately, I haven't seen the difference in plans for both cases.

[plan_with_false_option.txt](https://github.com/arina-ielchiieva/drill/files/123444/plan_with_false_option.txt)
[plan_with_true_option.txt](https://github.com/arina-ielchiieva/drill/files/123445/plan_with_true_option.txt)

Could you please suggest where I should look?


No, `--cov` is not configured in `setup.cfg`
only the config file for coverage is set there. (Which now I realize shouldnt be there)

The reason for not adding `--cov` to setup.cfg is because it takes way too long. it shouldnt be the default behaviour (just like ./run_tests.py didnt do coverage without the -c flag)


Some Thelia shops are solding used or refurbished products. In these cases, the NewCondition value is not relevant.


This values doesn't exists in VM TEMPLATE, they are in monitoring info


On Tue, 9 Feb 2016, Markus Elfring wrote:

>
>       It's not recursive.
>
> The function "catline" is recursively called here, isn't it?
>
>       OCaml is a properly tail recursive language.
>
> The OCaml compiler might perform special software optimisations in this use
> case. So it might look more convenient to express an algorithm in a
> recursive way.
>
> Would you like to compare the run time characteristics from my suggestion
> for a linear function implementation "show_file_content" with this one?

No.  I trust the compiler to do the right thing.

julia


I don't really like the "input" parameter responseBuilder. I'd rather build a (partial) response and merge it with others in the caller code


`trySanitizeName` -> `sanitizeName`

try implies it may succeed or fail, in this case it will always succeed


So I moved it here and added new one REALMEMORY, which is reported by opennebula poll script. It is from virsh dominfo "Used memory" without compromising by resident memory.


For #39


I set MAXMEMORY to vm.allocated_memory


oh you changed it


If you still have steam on this, using the `new` keyword here on a hardcoded type is really no different than what you had before since you still can't reach in and replace that filesystem object. Better would be:

```php
class Foo {

    protected $files;

    public function setFilesystem(Filesystem $files) {
        $this->files = $files;
    }

    public function getFilesystem() {
        if (!$this->files) {
            $this->files = new Filesystem();
        }
        return $this->files;
    }

}
```

OR

```php
class Foo {

    protected $files;

    public function __construct(Filesystem $files) {
        $this->files = $files;
    }
}
```

OR

```php
class Foo {

    protected $app;
    protected $files;

    public function __construct(Application $app) {
        $this->app = $app;
        $this->files = $app[Filesystem::class];
    }
}
```

would all make it so that you can replace the filesystem instance


ack then, this is aceptable


Don't think it matters too much actually since cocoapods rely on checksums rather than versions. But could be good routine to update the podspec with each release.


I thought about this more and took a look at the Atom website and spec. The "spirit" of Atom is about indicating when a website is updated; I think there's still value in returning an empty feed (with the last-updated date being today's date).

Since this is now the behaviour of Butterfly, there's nothing more to do here. Thanks!


[R/iri.R:20:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L20) *style:* **Variable and function names should be all lowercase.**
```r
CalculateIRI <- function(profile, iri_coef, segment.length = 100) {
^~~~~~~~~~~~
```
[R/iri.R:23:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L23) *style:* **lines should not be more than 80 characters.**
```r
  K <- max(2, as.integer(.5 + .25 / DX) + 1)  # number of profile points used to compute mvg avg slope input (window)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:31:5:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L31) *style:* **Commented code should be removed.**
```r
  # num_segments <- length(profile) / num_samples_per_segment
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:32:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L32) *style:* **lines should not be more than 80 characters.**
```r
  profile_segments <- split(profile, ceiling(seq_along(profile) / num_samples_per_segment))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:39:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L39) *style:* **lines should not be more than 80 characters.**
```r
    # sliding window of profil elevations for calculating mvg avg slope (buffer of length K)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:52:5:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L52) *warning:* **local variable ‘iri_df’ assigned but may not be used**
```r
    iri_df <- data.frame()  # df for colleteing return values
    ^~~~~~
```
[R/iri.R:61:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L61) *style:* **lines should not be more than 80 characters.**
```r
      if (length(profile_segment) >= ix) { # if slope could be built within semgent
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:61:42:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L61) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (length(profile_segment) >= ix) { # if slope could be built within semgent
                                         ^
```
[R/iri.R:63:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L63) *style:* **lines should not be more than 80 characters.**
```r
      } else if (length(profile_segments) >= next_segm_id && TRUE) { # if slope needs to be built with samples of next segment and there is a following segment
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:63:68:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L63) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      } else if (length(profile_segments) >= next_segm_id && TRUE) { # if slope needs to be built with samples of next segment and there is a following segment
                                                                   ^
```
[R/iri.R:64:52:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L64) *style:* **Put spaces around all infix operators.**
```r
        y[K] <- profile_segments[[next_segm_id]][ix-num_samples_per_segment]
                                                  ~^~
```
[R/iri.R:65:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L65) *style:* **lines should not be more than 80 characters.**
```r
      } else { # if slope needs to be built with samples of next segment but there is no next segment or no following sample
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:65:14:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L65) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      } else { # if slope needs to be built with samples of next segment but there is no next segment or no following sample
             ^
```
[R/iri.R:79:12:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L79) *style:* **Put spaces around all infix operators.**
```r
        y[j-1] <- y[j]
          ~^~
```
[R/iri.R:87:32:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L87) *style:* **Commas should always have a space after.**
```r
          z[j] <- z[j] + ST[[j,jj]] * z_last[jj]
                               ^
```
[R/iri.R:90:26:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L90) *style:* **Put spaces around all infix operators.**
```r
      rs <- rs + abs(z[1]- z[3])
                        ~^
```
[R/iri.R:96:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L96) *style:* **lines should not be more than 80 characters.**
```r
    # determine avg rs by dividing by number of samples per segment and attach to result vector
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:125:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L125) *style:* **Variable and function names should be all lowercase.**
```r
CalculateIRIContinuously <- function(profile, iri_coef) {
^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:128:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L128) *style:* **lines should not be more than 80 characters.**
```r
  K <- max(2, as.integer(.5 + .25 / DX) + 1)  # number of profile points used to compute mvg avg slope input (window)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:136:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L136) *style:* **lines should not be more than 80 characters.**
```r
  # sliding window of profil elevations for calculating mvg avg slope (buffer of length K)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/iri.R:149:3:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L149) *warning:* **local variable ‘iri_df’ assigned but may not be used**
```r
  iri_df <- data.frame()  # df for colleteing return values
  ^~~~~~
```
[R/iri.R:172:10:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L172) *style:* **Put spaces around all infix operators.**
```r
      y[j-1] <- y[j]
        ~^~
```
[R/iri.R:180:29:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L180) *style:* **Commas should always have a space after.**
```r
        z[j] <- z[j] + ST[j,jj] * z_last[jj]
                            ^
```
[R/iri.R:183:24:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L183) *style:* **Put spaces around all infix operators.**
```r
    rs <- rs + abs(z[1]- z[3])
                      ~^
```
[R/iri.R:188:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/iri.R#L188) *style:* **lines should not be more than 80 characters.**
```r
    # determine avg rs by dividing by number of considered samples and attach to result vector
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/rroad-package.R:30:1:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/R/rroad-package.R#L30) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-iri.R:7:10:](https://github.com/vsimko/rroad/blob/2d32d1add2624b8bea849d78128771a6265f91a0/tests/testthat/test-iri.R#L7) *style:* **Variable and function names should be all lowercase.**
```r
  iri <- CalculateIRI(profile, IRI_COEF_100, 20)
         ^~~~~~~~~~~~
```


Somehow, I think that adding that `baseurl` broke the feeds’ urls:

`https://18f.gsa.govhttps//18f.gsa.gov/2016/02/09/tips-for-capturing-the-best-data-from-user-interviews/`


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/531a2de8ef02751f53c649d453afe852a0d88de7/circle.yml)
- [scripts/deploy-wraith-diff.sh](https://github.com/pantheon-systems/documentation/commit/531a2de8ef02751f53c649d453afe852a0d88de7/scripts/deploy-wraith-diff.sh)


Should be "person" without a ":".

`query.setParameter("person", p);`


IMO the default implementation should not filter anything, and let JAVA-1055 wrap it in an `ErrorAwarePercentileTracker`, possibly configurable.


vm.real_memory - this was filled from Memory Reporter MEM_TOTAL_REAL, but it is a bad value which is close to maxmemory. We need real memory setted by virsh setmem command


Make the organization the maintainer rather than just one person?


@shadow7 Not sure. I think authentication won't work right if I only chance the `FIREBASE_URI` value. I can try it this evening when I have a few minutes.


@64FF00 In general, we from @MrDoni98 in one project! :smile:


That's what I was talking about... :+1:


Fix, I added missing assingment


Also, if we keep this filter here, as discussed the comparison should be based on an `instanceof` operator.


Moved this to top for earlier use


Все замечательно.


Calculate free memory percentage including memory over provisioning ratio


no need for this


you can just return here


Please post to mailing list


The text is too long for the pages.


cut and paste fail


Thanks for trying this out!


Although this will show a notification (should be an alert?) it doesn't really help the issue that we were trying to solve here with the errors being swallowed and no immediate indication to devs, especially when testing.

It might be good to reach into the `settings.errors` object here and either log something to the console and throw an error with a useful message. Throwing an error with a useful message may be preferable as it will instantly halt the incorrectly set up test or incorrect usage during development which should be the only time we actually hit a validation error here.


Забыл про увеличение "i", цикл будет бесконечным...


what if anything fails in this method ?


Yeah


Hi Klaus,
I just tested everything again with the content of this repository and the example recording I uploaded for you - everything works as expected/as we left it here.
So it should not be necessary to tweak anything in the flow graphs to start out. If you feel like there's room for improvement in the demodulation, you should wait for working on that until your setup is running basically.

The ability to recover from CRC errors is depending on the maximum width of error patterns in the syndrome file. The file I uploaded for you is with a maximum width of 10 bit and totally fine for now.
If you use a lower width, you will get more errors during L2 decoding.
You can set up the maximum errorpattern width of the syndrome file using the parameter "errorwidth" of the "Layer2" Block in darc_rx.grc

The best way is to just call
python darc_udp.py
and then start the darc_rx.grc flowgraph. The L2 decoding (CRC check, FEC) will then be done inside the GNU Radio block, and the data will be sent to the Python script using the UDP sink block.
The syndrome file will be generated automatically by the L2 block itself.

The way you obviously did is to start
python darc_prototype.py <filename>
In this case, the L2 decoding will take part in python, which is much slower.
But be aware: Because the Python script is doing the L2 in this case, so you have to remove the Layer2 block from darc_rx.grc! The input for the script is L1 data, after Clock Recovery and Binary Slicing. Otherwise you would try to decode L2 two times, which will definitely not work.

But I see, there is room for improvement in the documentation ;)


Good work, but please remember that all columns (including nested columns) must be contained within a row. This should be a quick fix by adding the class on row to the divs created on lines 39, 44, & 49. Once you've done that, go ahead an merge :+1:


just checking if it changes


подключать нужно до файла style.css


Thanks, readme also needs a bump.


reack


I think you don't need a Set builder, you mostly have 0 or 1 entry in it. ImmutableSet.of() should work pretty well.


* @todo FIXME


If code is copied from somewhere, recommended to give reference to it so that it helps to check out the implementation details as well as review the licensing terms later.


In bash it gives me `-bash: pipestatus: bad array subscript` when I run `z`.

Bash is so icky and confusing and opaque with its errors, bash/zsh in combination so much more so, I think it makes a lot more sense to avoid any need to examine pipestatus than it does to make any attempt to abstract across them. It's largely gratuitous here. Nothing about filtering the directory list for those which actually exist has a semantically meaningful failure mode; it's just that bash conflates the last predicate application ("does this directory exist?") with the success or failure of the program execution ("did something go wrong?") despite these being utterly unrelated results.

I updated #178 with an approach which disentangles the above, avoids the pipestatus question entirely, and de-duplicates some code.


I think you have to set this after the `request.open` call. Otherwise this throws an exception.


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


> The __GLdispatchPatchCallbacks struct is there as an optional optimization, which can allow a vendor library to skip the overhead of looking up each OpenGL function in libGLdispatch's dispatch table. For example, Mesa could use some variation of the entry_generate functions to overwrite the entrypoints in libGLdispatch with its own dispatch code.

As Andy did a presentation on the project a while back, some of us read too much of what was said jumped on the "OMG you guys are going to fix the slow our (mesa) dispatch" boat. Thanks for the confirmation - we should be less hyped next time, silly us.

On the add/removeFoo front: I'd imagine it can be useful, but current implementation tries really hard to continue even in the worst case scenario. So if we consider the change it would require a serious rewrite. So those are ok as is, at least for now :)


There are lots of eRx files to hit as well.  Have you done a full grep?  Also, there is a query needed to pull the billing notes across.


you could at least test the error cases by mocking the parser


constructor must be between fields and methods


i'm picky, but be careful about spaces


Needs to be referenced/documented.


Actually it was a part of test code. Will remove it, thank you.


Yes, nailed that down in release 1.11.1.


(I should note I have no idea whether that code works in zsh, though I'm optimistic.)


goal here was to avoid coupling the method signature with the (complex) primitive types returned by getUpdates (Map<Function<...) and possibly handle business changes without changing method signature.

> Why do you pass SetMessagesRequest when you only need updates ?


whats going on here?


How about removing this absolute path? I had to use `sudo` to create symlink to avoid Exception.


Ok will remove it. Actually comments for myself, I still have 583 which use it


Class name should start from capital letter.


This is a fixture for https://github.com/jxcore/jxcore/issues/804 and it was tricky to find.

For an object:

```js
{
  end : undefined
}
```
`hasOwnProperty('end')` will return `true` even if the value is `undefined`.

The original node was using only `!this._events[type])`. After we changed to `!this._events.hasOwnProperty(type)` it wasn't sufficient.


>I trust the compiler to do the right thing.

I find this view questionable. How much do you care for run time characteristics?


^_^学姐你好！来学习下你写的东西。。。


Class name should  start form capital letter.


Maps.transformValues() is probably your friend


+(182349,1,31,18369,5,0,0,0,0,0,0,0,0,0,0,0,'search for 18369'), remove this line, its not needed
change (182349,2,0,0,0,18369,5,3,2000000814,0,0,0,0,0,0,0,''), to (182349,1,0,0,0,18369,5,3,2000000814,0,0,0,0,0,0,0,''),
if you are using buddy system you dont need to search for creature, + say text should occur almost immediately hence only 1 second delay, I fixed the quest in tbc and dbscripts work the same


@OHerasym switch with only one case [here](https://github.com/LuxoftSDL/sdl_core/commit/d25e6532695b17e5bf3a79db36a4d7e3d0608481#diff-810fe643d114d6f664e6cb9f4fb8a2e1R96). I'd rather replace it with `if {...} else {...}` if no other event id is expected.


Great! Thanks very much.


Do not put extra space here.


#python dice simulator

from random import randint

dice1 = randint(1,6)
dice2 = randint(1,6)

print "rolling the dice..."
print dice1
print dice2


Did a grep (in windows) did not get a hit on any ERX files.

I will make this change

#IfMissingColumn patient_data billing_note
 ALTER TABLE `patient_data`
 ADD COLUMN   `billing_note` varchar(255) NOT NULL default '';
 UPDATE `patient_data` SET `billing_note` = `genericval2`;
#EndIf


This generates other issues:
```
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package bower-asset/jquery could not be found in any version, there may be a typo in the package name.
  Problem 2
    - bower-asset/fullcalendar v2.3.2 requires bower-asset/jquery >=1.7.1 -> no matching package found.
    - bower-asset/fullcalendar v2.3.2 requires bower-asset/jquery >=1.7.1 -> no matching package found.
    - Installation request for bower-asset/fullcalendar 2.3.2 -> satisfiable by bower-asset/fullcalendar[v2.3.2].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion> for more details.
```

I'm changing it to ```@stable```, which works. See d72dfe82779cfe7c850a960a26a27b46d48957d3


Closes #1


@brevilo To my knowledge the code was never implemented for the stock BOINC environment.  We put it in at WCG's request and they used it internally.


@Wirlie you're changing the font here. That's really what you wanted to do, or it's an error?


@OHerasym another switch with only one case [here](https://github.com/LuxoftSDL/sdl_core/commit/0081ae8de61fac27a89539eb5e49278538f4ac20#diff-4ade7c5ef0855c9798ea29c7d1e28eccL181).


@harshavardhana I learned from https://github.com/minio/minio-go/pull/325 that the `CloudStorageClient` has been removed, so this no longer applies. Is that correct?


main.c:3:1: warning: return type defaults to ‘int’ [-Wimplicit-int]
 main(void)
 ^


Needs to be formatted before merging it to Develop/Master.
4 spaces instead of tabs.
I recommend to follow Airbnb Javascript coding style: https://github.com/airbnb/javascript
Or feel free to recommend me a better coding style.

Not a big deal but using a good IDE can reduce these issues.


This should be `kwargs.pop('config_dir', None)` here. Otherwise you get a `KeyError` when `config_dir` is not passed in the `kwargs`.


It might be that the MPlot itself is deleting the tools.  Need to double check how the plot is handling deletions and not deleting already free'd pointers.


Fixed this if to comparing with free memory including mem_over_ratio. So it is TRUE only if freemem is 5% under or 5% over desired mem_over.

Ex.
total_mem = 612MB
free_mem = 100MB
used_mem = 512MB
over_ratio = 40%
set new_mem = 716,8MB

New itteration
total_mem = 716,8MB
free_mem = 204,8MB
over_ratio = 40%

Without over_ratio
free_mem/total_mem = 204,8/716,8 = 0,29 * 100 = 29%, so its is under 35% (40-5margin)
Script is executed to do new memory calculation

With over_ratio
free_mem/total_mem = 204,8/716,8 = 0,29 * 140 = 40,6%, great, here is no change, skip new calculations

After this fix MEM_DIFF_TO_CHANGE never takes to effect, probably


it's worth noting that with this change stdout_lines still has BECOME-SUCCESS...


fixed


OK, done


Проверила, все верно.


The statement checks what IP protocol the tests were started on so the test can connect to the appropriate contact points.