The continue is no more needed ?
We can even go for something like:
```java
                Map<Integer, JsonArrayEntry> multiResource = multiResourceMap.get(resourceId);
                if (multiResource == null) {
                    multiResource = new HashMap<>();
                    multiResourceMap.put(resourceId, multiResource);
                }
                multiResource.put(resourceInstanceId, resourceElt);
```


I've added that the method renames the method it delegates to to the commit note.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (399 of 399 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)


Looks good to me.


to apply


Кстати нужно бы и это в switch засунуть


sure, might make sense to do an inline comment explaining the behavior for the last popped `vk`, alongside a pylint disable statement for the use of a loop iteration variable.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28804/testReport/) for this commit**


`.topbar-nav-svg-caption` vs. `topmenu`. Should not the naming be more consistent?


Qu'est-ce que t'as fait lol tu sais que c'est public


Since you have a default value it seems like you meant to set `optional = true`.


We could have both, ie: we cat set the default to the current URL and content and just allow the users to change it if they want but don't enforce it.


Why do we need to check if `resourceElt.getName().startsWith("/")` ?
The spec seems to say that this should not happen.


檔名改成 etc/mail_account_sysop 才看得出來是帳號站長


Then why not simply cast to string and be done with it? `(string)$variable` will do exactly that!


What about adding this diff to your commit?
```diff
diff --git i/turf/c/platform_detect.h w/turf/c/platform_detect.h
index 5e79149..65e2a9a 100644
--- i/turf/c/platform_detect.h
+++ w/turf/c/platform_detect.h
@@ -83,7 +83,11 @@
         #endif
     #elif defined(__powerpc__) || defined(__POWERPC__) || defined(__PPC__)
         #define TURF_CPU_POWERPC 1
-        #define TURF_PTR_SIZE 4  // Only 32-bit detected for now
+        #if defined(__powerpc64__)
+          #define TURF_PTR_SIZE 8
+        #else
+          #define TURF_PTR_SIZE 4  // Only 32-bit detected for now
+        #endif
     #else
         #error "Unrecognized target CPU!"
     #endif
```


great! lgtm, thanks a lot!


I have amended the commit note to include the method is renamed in the new branch.


To be fair I'm not sure I should be credited. I mean I appreciate it and all but my version was more of a refactor and considering how much the two codebases [differ](https://github.com/Anahkiasen/cookie-monster/tree/master/src) I doubt any of my changes made it into your fork :d


Indeed. Fixed in #13729.


remove


This is such a pain. I almost want to try to reproduce that library in a
clear license. I am just going to accept the pull request as it appears the
Debian project and C Profile projects have both used the code in that
sense. I am wary of "Freely Distributable" being compatible with "Ported to
PHP" but on the other hand, the author seems to be quite supportive of this
sort of use.

On Tue, Mar 15, 2016 at 9:33 AM, Frank Wikström <notifications@github.com>
wrote:

> Yes, I understand your concern. The license is not very clear (but
> apparently the SciPy project was ok with it).
>
> Relevant link: https://lists.debian.org/debian-legal/2004/12/msg00295.html
> .
>
> It's listed as "freely distributable" at
> http://www.sai.msu.su/sal/B/0/CEPHES.html
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mossadal/Probability-Distributions-for-PHP/commit/f2e68932284d447df0e975d5fe7b0e8845549bd2#commitcomment-16712534
>


if you use hinting, i.e. public function Manager(iStorageEngine $storage_engine, iCrawling $crawler) Ides will work better to find the function declarations in other classes/interfaces


Why not bubbling up the exception?


:-D


@scytacki  Thanks for your feedback on this.  Made the suggested changes in this commit.


A million thumbs for this.


That's a good suggestion. I'll add this. Thanks.


Use `GATKProtectedMathUtils::rowMeans`


My experience with this has been that it will just lead to faulty measurements. It would allow the user to configure the test in a way that is not correct and leading to us reporting (wrongly) that psiphon is not reachably while it in fact is.

This is for example the case with the http_host test, which in my opinion has lead to a lot of unusable results.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9-NoThreads: OK (395 of 398 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.25%25-yellow.svg)


Use `GATKProtectedMathUtils::rowMeans` followed by `.map(Math::sqrt)`.


- next 라는 인자를 받는 callback 함수
    - 미들웨어 사이에는 많은 의존성이 존재하기 때문에 next() 를 호출하여 다음 미들웨어 호출

> 아주 조금 대략적으로 이해가 되긴 하는데.. 다음(next) 미들웨어를 어떤 방식으로 호출하는지, 호출해서 어떻게 사용하는지는 모르겠음


looks good please merge


fixes :))


Personally, I'd skip this whitespace and the one after the overridden method.


please don't use this on production code, we're testing a security patch to this project.


Sounds good to me.


Yes, within the scope. The matrix library does not have a set(data *) method yet, but I could add it. The _gyro object does not change, the assignment operator copies the data from the temporary Vector3f created from the voter data.


I think it would be worth adding column means and variances to `GATKProtectedMathUtils`.  It comes up elsewhere in our code base.


I see you've disabled arm64 4K page support, was that necessary for your booting or just an optimization?


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2171 of 2171 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)


Wow, that really is counter-intuitive, I didn't notice the docs mention this. It does get refactored out later - so might leave as is.


@lasote Reverted in 03199f26cbabbb6649137da6d46657d5d9b80b56. It's easy to revert and change these things so please try to stay polite; we're all doing our best here.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28805/testReport/) for this commit**


Это точно не должен быть div


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


done.


This is very naive... Must be redone by someone with better insights.


no that was stupidity ;)
my commit right after that fixes it..
btw i tried all toolchains. same results..

the thing about logs. i cant get the freaking kernel to create a last_kmsg.. i tried your pstore way in the defconfig.. there's nothing created..


@bbcrddave I take it that this comment about string cast is not an issue anymore?


These last commits are so nice ;)


This contradicts openSUSE specfile guidelines: https://en.opensuse.org/openSUSE:Specfile_guidelines#Permissions


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28806/testReport/) for this commit**


Here possible conclusions should be added instead of 'Based on the ouput data what conclusions can you draw?'.


This fixes #6


Notice there is api method for treasure carry speed pirate.getCarryTreasureSpeed()
Better save game.getSailOptions(p, p.getInitialLocation(), 1 to a variable and use this instead of duplicate code


**nice project**
-----
> make file


I have completely no idea why, but Visual Studio decided to edit several files. I have no idea why, but it's still editing them for me. I'm sorry for so much unneeded changes.


better late then never


[R/guess.R:33:28:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/guess.R#L33) *style:* **Trailing whitespace is superfluous.**
```r
  if (!is.null(short_code))
                           ^
```
[R/guess.R:48:46:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/guess.R#L48) *style:* **Trailing whitespace is superfluous.**
```r
  UseMethod("icd_guess_short", x[[icd_name]])
                                             ^
```
[R/valid.R:400:46:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/valid.R#L400) *style:* **Trailing whitespace is superfluous.**
```r
  icd_is_valid_major.icd9(x, whitespace_ok) |
                                             ^
```
[R/zzz.R:22:1:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/zzz.R#L22) *style:* **lines should not be more than 150 characters.**
```r
    "Welcome to the \"icd\" package for finding comorbidities and interpretation of ICD-9 and ICD-10 codes. Suggestions and contributions are welcome at https://github.com/jackwasey/icd .
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-current-guess.R:13:1:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-guess.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-current-guess.R:63:1:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-guess.R#L63) *style:* **lines should not be more than 150 characters.**
```r
no_pts <- structure(list(visit_id = integer(0), code = character(0), poa = structure(integer(0), .Label = character(0), class = "factor")), .Names = c("visit_id",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-current-openmp.R:45:75:](https://github.com/jackwasey/icd9/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-openmp.R#L45) *style:* **Trailing whitespace is superfluous.**
```r
          if (packageVersion("testthat") < package_version("0.11.0.9000"))
                                                                          ^
```


One of the questions you will have asked yourselves is, "what order do the members of a class go in"? The answer is here : http://imgur.com/yzGKHHz - so you'll see fields are placed before methods adn ctors. So these three lines should be the first things in file.


get_status_code is easier to use if it doesn't raise an exception


I pushed a workaround (proper fix would be in distutils, to distinguish CFLAGS from CXXFLAGS) that strips -Wstrict-prototypes from the C++ compiler arguments, commit 96cfaefa8a0d0b6cda353046717de9e6421dff65.


revert


@trociny I hit failures in TestJournalEntries.* while running under valgrind.  Perhaps this explains why rbd-mirror daemon stops replaying a live journal stream.


[R/guess.R:33:28:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/guess.R#L33) *style:* **Trailing whitespace is superfluous.**
```r
  if (!is.null(short_code))
                           ^
```
[R/guess.R:48:46:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/guess.R#L48) *style:* **Trailing whitespace is superfluous.**
```r
  UseMethod("icd_guess_short", x[[icd_name]])
                                             ^
```
[R/valid.R:400:46:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/valid.R#L400) *style:* **Trailing whitespace is superfluous.**
```r
  icd_is_valid_major.icd9(x, whitespace_ok) |
                                             ^
```
[R/zzz.R:22:1:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/R/zzz.R#L22) *style:* **lines should not be more than 150 characters.**
```r
    "Welcome to the \"icd\" package for finding comorbidities and interpretation of ICD-9 and ICD-10 codes. Suggestions and contributions are welcome at https://github.com/jackwasey/icd .
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-current-guess.R:13:1:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-guess.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-current-guess.R:63:1:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-guess.R#L63) *style:* **lines should not be more than 150 characters.**
```r
no_pts <- structure(list(visit_id = integer(0), code = character(0), poa = structure(integer(0), .Label = character(0), class = "factor")), .Names = c("visit_id",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-current-openmp.R:45:75:](https://github.com/jackwasey/icd/blob/d2d967c0c88e6f34be49f7ce928bef3132077db0/tests/testthat/test-current-openmp.R#L45) *style:* **Trailing whitespace is superfluous.**
```r
          if (packageVersion("testthat") < package_version("0.11.0.9000"))
                                                                          ^
```


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


yes, otherwise the default is a 1x1x1 nodebox, obstructing the entire doorway.


was that supposed to be left commented out?


mind spelling out Any for readability and exhaustiveness check?


add newline


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (399 of 399 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)


hmm, I'm thinking `no-self-use` is weird enough that we should have to disable it in places where we do it by design


fyi @mdtusz


Another similar cards are Bahamut Shark and Horse Prince, which use "...ターン", but don't check action before the effect active.


I meant update alpine version :)


imo the install link should be pointed to only the snippets associated with the angular-styleguide or better still fork the repo, create a PR and push the snippets.


.take/.drop


I suggest to fix indentation so it will be more readable
you can combine both checks to a single if with condition1 && confition2


@mdtusz looks ok?


Do you know what your doing? This is for msm limiter


I've updated these tests to use one of the other values.


so 21 uhr?


Same applies to Ext4 backed only Btrfs one is not affected (obviously). I can extend the patch so that it covers also the ext4 case if found desirable.


This didn't change anything. PhantomJS builds are failing for lots of people due to download overages on bitbucket: https://github.com/ariya/phantomjs/issues/13953


hmm for now we can try running an adb shell with a constant cat /proc/kmsg > /sdcard/kmsg.log running until it reboots perhaps that will show us something


Heh, so this is my bias. `no-self-use` is not something I've done unintentionally, but it's something I do at times.


Also, in PDS we're prefixing member fields with m_ (and static member fields with s_)


setAttribtes :)


Yeah it should be removed, oops 32c5342


ну алерт то был так показом) примером


break should be continue


**Very important** clip path get translated if they are applied to a `<g>` that contains a translate transform. Hence, the `x=0` and `y=0` attributes here.

Ref: http://codepen.io/etpinard/pen/qZqxyM


`r_list_free(rx_list)` will free the elements in it, right? I did not check very carefully since `rx = r_regex_new (tok, "");` also allocates. So i believe we neither have leaks before nor now ^_^


each оставь в темплейте а тело сюда


thats a great idea!
you thing its doable with a terminal emulator?


no need to check if p1.equals(tmp) already checked in previous line


Looks good to me.


Count() returns 0 from a an empty IEnumerable, so you can write this more simply as
 m_iwordCount = listInput.Count();


This was committed by me, for some reason it displayed it as unknown.


[R/from_archie.R:17:10:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/R/from_archie.R#L17) *warning:* **no visible global function definition for ‘read’, Did you mean 'head'?**
```r
  aml <- read(aml)
         ^~~~
```
[R/from_archie.R:19:9:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/R/from_archie.R#L19) *warning:* **no visible global function definition for ‘new_context’**
```r
  ct <- new_context()
        ^~~~~~~~~~~
```
[R/from_archie.R:20:59:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/R/from_archie.R#L20) *style:* **Put spaces around all infix operators.**
```r
  ct$source(system.file("archieml-js/archieml.js", package="rchie"))
                                                         ~^~
```
[R/from_archie.R:31:1:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/R/from_archie.R#L31) *style:* **lines should not be more than 80 characters.**
```r
             identical(try(httr::status_code(httr::HEAD(x)), silent=TRUE), 200L)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/from_archie.R:31:68:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/R/from_archie.R#L31) *style:* **Put spaces around all infix operators.**
```r
             identical(try(httr::status_code(httr::HEAD(x)), silent=TRUE), 200L)) {
                                                                  ~^~
```
[tests/testthat/test-aml.R:4:4:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L4) *style:* **Place a space before left parenthesis, except in a function call.**
```r
for(file in list.files(pattern = "\\.aml")) {
   ^
```
[tests/testthat/test-aml.R:7:42:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L7) *style:* **Variable and function names should be all lowercase.**
```r
    expect_identical(imported[-c(1, 2)], fromJSON(imported$result))
                                         ^~~~~~~~
```
[tests/testthat/test-aml.R:11:4:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L11) *style:* **Place a space before left parenthesis, except in a function call.**
```r
for(file in list.files(pattern = "\\.aml")) {
   ^
```
[tests/testthat/test-aml.R:13:1:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L13) *style:* **lines should not be more than 80 characters.**
```r
  test_that(paste0(imported$test, " (with fromJSON arguments) (", basename(file), ")"), {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-aml.R:14:1:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L14) *style:* **lines should not be more than 80 characters.**
```r
    expect_identical(imported[-c(1, 2)], fromJSON(imported$result, simplifyVector=FALSE))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-aml.R:14:42:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L14) *style:* **Variable and function names should be all lowercase.**
```r
    expect_identical(imported[-c(1, 2)], fromJSON(imported$result, simplifyVector=FALSE))
                                         ^~~~~~~~
```
[tests/testthat/test-aml.R:14:82:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L14) *style:* **Put spaces around all infix operators.**
```r
    expect_identical(imported[-c(1, 2)], fromJSON(imported$result, simplifyVector=FALSE))
                                                                                ~^~
```
[tests/testthat/test-aml.R:21:40:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L21) *style:* **Variable and function names should be all lowercase.**
```r
  expect_identical(imported[-c(1, 2)], fromJSON(imported$result))
                                       ^~~~~~~~
```
[tests/testthat/test-aml.R:26:30:](https://github.com/ropensci/rchie/blob/e3db7a9ad589c84e0db2f3915a4f4676d6767f0b/tests/testthat/test-aml.R#L26) *style:* **Variable and function names should be all lowercase.**
```r
  expect_identical(imported, fromJSON("{\"key\":\"value\"}"))
                             ^~~~~~~~
```


Yes


Good :-)


Ah right, that's trixy—in that case I'd suggest either two approaches:

1. Use Metaphysics instead of Gravity to get the artwork data (with it's image.url helper) e.g.
![image](https://cloud.githubusercontent.com/assets/555859/13785900/cdc55940-eaab-11e5-9a15-5b128f1fd5f7.png)
2. Require `'backbone'` and `artsy-backbone-mixins` to do an ugly `image: ArtsyBackboneMixins.ArtworkHelpers.imageUrl.call(new Backbone.Model(artwork))`. That way it at-least couples it to just the libraries and not the client-side code.

1 would be nice b/c it could involve less `async` fanciness (and you could use GraphQL to limit the fields you save), but 3 might an easier solution to implement right now.


Comment would be clearer as "but probably better than assuming there are as many segments as targets"


@richard-scott are you sure you have an empty `/etc/hosts` file? could it be it's not empty, but it hasn't got a newline at the end of the file, which causes the hostname (by default `pi` to show up at the beginning of the command prompt, where it's easily overlooked)


traversed with 'd'


no it happens even if its enforcing..
my latest build seems to be the most stable so far (i used an old fashioned way of repacking the kernel) but it has acquired a new bug: when i try to access fingerprints or any lock method it automatically reboots.. i ised it on the vr for 50 mins and didnt reboot.. but for one of my testers it gave her 3 immediate reboots..

your patch for permissive is fine..i used my own "easy/lazy" way to do it also (just adding a "new value =0) in selinufxs.c but the results are pretty much the same.. reboots.. and usually after the 1st reboot then it goes down hill..


might be, but chances are terminal emulator will get killed too early to get anything useful
i wonder if it's related to the type fail panic here:
https://github.com/jcadduono/nethunter_kernel_herolte/commit/cbef699ec73d390b73debf82156dac08e93111a3#diff-fdfd84e5fdea26ce2af73fb1db994dbeR1504


Mind adding a test for failures of prefixes that have *s in them?
/foo/*bar/baz in particular.


a


Can't we just do this:
```
const inputElm = React.findDOMNode( this.refs.input );
if ( inputElm ) {
  inputElm.focus();
}
```
and render this
`<Input { ...this.props } ref="input" />`


平 Set = {平, 怦, 㛁, 評, 坪, 伻, 评, 抨, 砰, 秤, 萍, 呯, 岼, 泙, 玶, 枰, 胓, 蚲, 軯, 鲆, 鮃, 駍, 苹, 閛, 匉}


latex-documentation (michael-okeefe) - i386-Windows-7-VisualStudio-12: OK (2126 of 2135 tests passed, 19 test warnings)

 * 18 tests had: AUD diffs.
 * 11 tests had: RDD diffs.
 * 10 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 9 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 7 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


It's should be 'Content-Type' (capital C and T) regarding to [W3C spec](https://www.w3.org/Protocols/rfc1341/4_Content-Type.html)


:+1: Thanks.


ZFS_AC_CONFIG_USER_TIRPC should be updated to be consistent with the existing `user-*` m4 files.  My suggestion would be to model this function on [user-libuuid.m4](https://github.com/zfsonlinux/zfs/blob/master/config/user-libuuid.m4) .  You can modify it to automatically detect if the tr-rpc library is installed and check the `xdr_control()` is provided, then set `HAVE_TIRPC` accordingly.  Optionally you could further extend it to allow `--with-tirpc/--without-tirpc` to passed to override the default autotdetected value.


I've added comments. Each comment comes **after** the line it refers to.


Is it necessary to remove the conditional restart?


sorry I didn't catch this error before .. I just see the test failing now on my CI...


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


@wolfmanjm I don't like this, and I know it fails for multiple hotends. Is there a better way to get the designator? I see that the TemperatureControlPool constructor does it, but it uses a uint of "name" and I don't see an obvious way to get all that into a module?


let me build a quick one with enforcing and ill try to access fingerprints (which gives reboot 100%) and see what happens..if enforcing is actually stable (as i originally thought but not anymore) then the answer is the permissive patch..

btw the way i used to do the permissive thingy was this:

https://github.com/friedrich420/S6-Edge-Ael-Kernel-MM/commit/4baaccbe1e667b31bded48a5545a6ff18b096b13


Yeah this is just a hack to make weak globals work safely in Rust (they're unstable). This should definitely move somewhere else in the compiler before stabilizing.


Since this commit (with the help of git bisect), jenkins don't returned subversion changeSet in api XML.

Is this behaviour due to a bug or a functionnal reason and  should I create an issue ?

The "how to reproduce" is the following :+1:

- Create a simple job wich export an SVN and do "echo $WORKSPACE"
- Call xml api (like http://localhost:8080/jenkins/job/test/1/api/xml?depth=3)
- In the given XML the changeSet xml is no more present.

Before this change, the changeSet is present.


nuked


TeamCity jonas-requests :: WindowsBuild [Build 39](https://build.neohq.net/viewLog.html?buildId=584042&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **SUCCESS**
Summary: Tests passed: 33401, ignored: 138 Build time: 2:31:13


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28807/testReport/) for this commit**


The sentiment of the comment is still valid, but it's no longer relevant since, if setSource's argument evaluates to false, an alternative code path is now followed which does not use the value of the argument.

Note that '' evaluates to false so before this PR src was not set anyway.


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


Fixed (issue #102) in commit 5d82d63d57a0ca84adc27de117f2bcdc019be337 from Main DMON Repo.


typed_function_ordering (lefticus) - Win64-Windows-10-VisualStudio-14: Build Failed

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


/cc @victorhurdugaci @pranavkm what about the performance repo?


Old brokerages require closing the position and issuing a separate order to sell short, but those who provide helpers (IB) which automate this I imagine the buying power is modeled from closing the position and opening a new one. #2 seems to ignore the size of the order though and I'm not sure summing the requirement-percentages will work as expected -- and multiplying the current holdings by initial margin requirements doesn't feel right? If the maintenance and initial are different (futures) #1 will also be wrong but since all our assets have the same maintenance/initial requirements it hasn't been an issue.

Maybe its closer to something like this? (Where order quantity is fraction in new direction)
```(security.Holdings.AbsoluteHoldingsValue * GetMaintenanceMarginRequirement(security) ) + GetInitialMarginRequiredForOrder(security, order) + portfolio.MarginRemaining;```


I strongly recommend not adding comments. There are two reasons
a) comments tend to get out of date and become misleading
b) code should be easy enough to read without a comment. The corollary is that if you feel a comment is necessary, it may mean that you should change your code to make it more readable

There are places where comments are necessary, where it helps explain something that the code cannot convey. But these are few and far between and not here!


Hm... Maybe not. I'll have to try.


I can't recall, but there's no need for it - have removed from updated commit.


Yeah, that's probably a good idea.


i reverted the whole permissive patch not just disabling it in the defconfig (ocd reasons..)


C'est cool Mylène est parmis nous :)


Have added your correction in the commit note subject. Is there an example of the "null object" pattern within smart answers?


develop (lefticus) - 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-warnings-yellow.svg)


That seemed to fix it!  Thanks!


#Fail


What about `Math.sqrt(new Variance().evaluate. . .` in order to avoid writing a one-off standard deviation?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


This isn't as pretty when compiled!

<img width="1071" alt="screen shot 2016-03-15 at 1 00 09 pm" src="https://cloud.githubusercontent.com/assets/5882053/13786447/f68a4028-eaad-11e5-8e58-a120a3a9018a.png">


v1.2.7 :)


@Koenvh1 I just realized that this line does nothing, since `.Replace()` returns the modified string, instead of manipulating the original string.  Should this be `cityName = cityName.Replace(" ", "");` instead?


@Alhadis Thanks for the tip! Added it to my Atom user stylesheet :smile:


thanks ;)


I would stick to use our activeSyncMachine, why reverting my recent change?


[HZR] 2.3 AC-Web User views Hot Zone Report


I think we don't have `develop`. We only have `master`


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


Great linq statement.
I'm not sure "newword" is a great variable name, as it doesn't describe its contents well.


Bienvenue Mylène


You can do with it whatever you want.


@ctennis thanks! Fixed up the commit and updated master.


We can't use static strings for the `ref` part, because we have more than one of these fields and `ref`s are global, so there's a conflict and an error.
I believe the `! prevState.toggled && this.state.toggled && prevProps.value === this.props.value` condition condition is needed - the use case is the issue you've raised. That is - when the Organization field was non-empty on load, the field was focused unnecessarily. This condition checks if the update was a result of the user clicking the toggle link or if the `props.value` changed (if the latter - don't focus).


still freaks me out!!


I know this is from before your commit, but could you add some code the deregister these listeners when the element is destroyed


Ok thks for thé heads up


suggestions: capitalize `Frag`


From 2016 fsPHENIX workfest: switch the fsPHENIX macro to use Cesar's new field map with piston + up-to-date sPHENIX yoke design. More details in /phenix/upgrades/decadal/fieldmaps/fsPHENIX.2d.README


@Kris-LIBIS sorry to hear that. I haven't been actively working with Ruby so I haven't kept up with the changes.

If anyone figures out a way to make this gem work with Mongoid 5+, I'm open to contributions.


Changes look good, just address my comment and you'll be good to ship


RTL8188EU also fails - I suspect all of the RTL* packages are going to need a PKG_SOURCE_DIR fix. :(


maybe more idiomatic rust ?
```rust
                        let basic_str = node.get_basic().map_or(String::from("unknown"), |basic| format!("{}", basic));
```


develop (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed (0 of 391 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-0.0%25-red.svg)


Developer loads hot zone report in right side article


Only necessary to use parentheses when defining a constant that is an expression. For example:
#define MXT_FOO_BIT (1 << 3)
but not
#define MXT_FOO_SIZE 24


typed_function_ordering (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed (0 of 391 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-0.0%25-red.svg)


You probably should just terminate compilation if the option is None in the callback itself, because the option will only be `None` if typeck fails before MIR build. I don’t think we can properly do borrowck if typeck fails.


Looking at more recent logs, I think the problem was not that failure is hard to read, but that success is. Because of all the test output, it looks a lot like we're supposed to evaluate that output ourselves. I think that merits a separate task to silence that test output when not in debug mode. To be clear: LGTM for these changes. That was just an observation.


no need to typecast here


update_travis_toolchain (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed (0 of 390 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-0.0%25-red.svg)


This change has been refactored out in the new branch based a previous comment to an earlier commit.


lgtm


The authoring domain is not needed, because it's not publicly accessible under it's own name. In this case it's going to be accessed as `frontend_domain` via varnish rules.


Well, but it does the same job it resolves the value and it reads well

  resolve defauls.field


That is interesting. I will test it out with a webcam. Thanks for the comment.


Would prefer you defined block_size in size_t rather than changing count to int


is 1.2.7 working for you ... I am getting below error when I try run my app:

```bash
ReferenceError: regeneratorRuntime is not defined
    at utils.js:6:1
    at Object.<anonymous> (utils.js:44:16)
    at Module._compile (module.js:435:26)

```


"show" is the wrong word, I feel


and actually, we don't even need the second part since we use it in a format later, so just using `unwrap_or` should be enough:
```rust
let basic = node.get_basic().unwrap_or(String::from("unknown"));
```

and then maybe you don't even need to use a temp variable (but you can keep it if you think it's clearer that way)


What do you mean manually? The above resolution is for API url cal and this one is for route link creating. Two different things


Use the 'RegisterAssemblyModules' method to get all the existing modules in one hit.


Again, you can leave out the call to Any() as results.Where returns an empty IEnumerable if it is given an empty IEnumerable. So the line can be

var finalwordlist = results.Where(x => x.Count() == results.Max(c => c.Count())).Select(a => a.Key);


Hey! What "blahblah!" -.- ^^.
Thanks for the fix :).


Misunderstanding of requirements. We want to *set* the i2c block size when doing other commands, not have a command for displaying it


Sure.


when are the `Index` projections used with anything *other* than slices? can this just be an assert? Oh, I guess for fixed-length arrays we need to insert the zero?


Crypto test enabled in 66a12354264905fe888edf1269253d140c9bb564.  Agreement with the Python code enabled and tested in be3eb9205ae2aeb6adf0b1cd038162d6f9bb3e6b.  Test vector source cited in d60f30449870d0aee4b0ad85705566f43f4bf590.


Can we not just contain the clip path within the element being clipped?

e.g.

```html
<g clip-path="url(#clip)">
    <clipPath id="clip">
        <rect x="0" y="0" width="100" height="200"></rect>
    <clipPath>
</g>
```

Then the clipPath should remain positioned relative to the element it's applied to?


output:index(input, 1, torch.LongTensor(indexes))


I think that you can use variables on the stack here; they don't need to be fields.


tcp is the one odd protocol that needs it's tls listener wrapped up in a keep alive. Make it the special case so that the default is to just pass the listener to tls.


It still is being used to know when to strip parameters, nothing changed with that. And dynamic defaults are still returning static values it just for simplicity of defining defaults since we are in dynamic language here.


In my latest version, I included "import database.CrimeDbSchema.*;" so every time that I needed to use CrimeTable, I did not have to have the CrimeDbSchema right before it.


my main complaint is that I'd like to see specialized tests, but I guess that given that you are actually switching on MIR in general, there isn't really much need for that anymore in this patch series, so lgtm


Maybe use the word `pre-existing`


So, the set_new_questions method works fine when you click "begin"
But when you click "next", it instantly submits the new question and the new answers as the answer to itself and says, sorry, that's an incorrect answer.
Is there something I can clarify to not make this happen? Is js hearing the click of "next" and "submitter" at the same time?


Yes, pretty much only arrays and slices.


This isn't proper use of `id` as you can't expect it to be unique. Rather, add specificity by including more tag and class names.


test size and commit it


No, the markdown versions work just fine. See [here](https://github.com/MDAnalysis/MDAnalysis.github.io/blob/post-nglview/_posts/2016-02-28-release-0.14.0.md) for an example. Please change them back.


its not the permissive patch.. the enforcing kernel has the exact same behavior.. reboots..


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28808/testReport/) for this commit**


seems ok.


chapter 15 + chapter 16 not success.


Ha, never mind. :)


This will start a new Job. This is most likely NOT what you want here. If first-page-mode is enabled and the current job has signaled that the content for the first page is available, then you already know that you can read the content via the "/content" handler. So why another job start?


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2074 of 2132 tests passed, 58 test warnings)

 * 52 tests had: EIO diffs.
 * 57 tests had: ESO small diffs.
 * 17 tests had: MTR small diffs.
 * 57 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.
 * 6 tests had: MTR big diffs.
 * 7 tests had: Table small diffs.
 * 3 tests had: ERR diffs.

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


Bare en liten bidrag fra meg :) Du har virkelig stått på, Jonas. Hva
gjenstår? Poengsystem? Cool! Jeg også likte ideen om å ta imot spillernes
navner og bruker dem i gui'en. Er det mulig?

On Tuesday, 15 March 2016, Arxcis <notifications@github.com> wrote:

> Yes! Dette trengte vi ;)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub:
>
> https://github.com/raglew/EnkelSpill/commit/9e9c757eb5aa002c17c78f86b8af6afa8751b029#commitcomment-16712508
>


The `ref`s are not global actually, they are tied to the component.

I think the `if` statement should remain, my feedback was to the use of `callback` in render method. It seems React refers to using a string for `ref` as a legacy. I don't know when that happened and frankly, I don't see why a callback method would be better :/ It seems your method is the React's recommended way now, so I won't push for it but I'm still not convinced how does that make anything better.


My apologies! The reason I changed it is because I went to `brew home conan` and the GitHub site didn't give me any additional info as to what Conan was. I saw the username on GitHub was conan-io, and when I went to conan.io the upper-right corner said "Fork me on GitHub" and pointed right back to the GitHub page. So that's why I thought that was the actual homepage!

Thanks @mikemcquaid for the quick response on both the PR and the reversion!


Except for my comments on the highlighting, everything else looks good. I think the expansions in the text are good clarifications.


great, so probably sometime tima related... cat /proc/kmsg save us


Next target: SkyNet


**[phpUnit_trunkIntegration run failed](http://jenkins.wikia-prod:8080/job/phpUnit_trunkIntegration/9535/testReport/) for this commit**


The whole "if (isFirstPage)" and this else block makes no sense. The server will not send a "CONTENT_AVAILABLE" status if the first-page-mode is not enabled.


The scenario is somthing along the lines of having other ruby / gem on the PATH
it will then pick up "gem" from that and use the wrong gem and gem path. if that makes sense?


ok.


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


Are we planning on doing anything other than updating the entry point? For such a small change I feel nervous forking and labelling as our own. Perhaps issue a PR to the upstream repo and list the PR in our `package.json`?


This `.gitignore` rule looks okay, but you've still committed all the class files. Any idea what happened?


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


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28809/testReport/) for this commit**


TDIL: stop using `git commit -am` cuz you forget to add new files


default.html in _layouts


ok


Horse Prince says "AFTER you synchro summoned [...]", Shark says "[...] for the REST of this turn."
I've just noticed that we have an outdated text in our system for Vulcan ... that one actually says "but for the REST of this turn".
The current translation ygorg has for Sangan says "[...] you cannot activate the effects of monsters with the same name DURING that turn."
So, just like Duality says that you can't special summon during the turn, this card says you can't activate the effect of the monster during that turn. Thus I still think we need some kind of check for this.


In my local Jekyll they didn't. The liquid tags are ugly but always work.

Did you test locally?

--
Oliver Beckstein
email: orbeckst@gmail.com

Am Mar 15, 2016 um 10:09 schrieb David Dotson <notifications@github.com>:

> No, the markdown versions work just fine. See here for an example. Please change them back.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
>


Decidir si aceptar propuestas!


Typo?


OK.  If you want to make the C++ tests less verbose by default, that would be fine by me, as long as there is still an easy way to get the full output they currently show (e.g., a DEBUG environment variable as you suggested -- in this case, the C++ tests do not follow any standard practice and are just ad hoc programs that return zero or nonzero).


im on it but my phone is messing with me telling me its not unauthorized in adb connection.. i need to check the settings


Fix

```
docker run -p 9000 -v $HOME/export:/export -v $HOME/.minio:/root/.minio minio/minio
```


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1583 of 1583 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.88%25-green.svg)


style nit: } else { instead of continue so that it's visually clear these are mutually exclusive paths


Are you guys open to pull requests?
I'd really like to help with cleaning


Are you suggesting that we parse null as "" inside OffsetCommitRequest itself?


the older code was equivalent, no? just more generically written? can we add some sort of assertion here that the repr is just an integer? do we know that the discriminant isn't a `u8` or some kind of integer that has the wrong size?


True, but the Organization had also listed half of today's effects as mandatory instead of optional: it would be better to check the Japanese wording directly. Has the Wikia page been updated with the errata yet?


Forgot to tag! @klq


That [didn't work](https://travis-ci.org/educreations/django-ormcache/jobs/116177567#L198). I'll actually test locally this time.


Changing to "print"


cs


When we get to lines 27-33, your comment is becoming useful, as the code is completely obvious when you read it. Rather than write a comment, though, extract the lines in a method and give it a useful name, e.g. `MostCommonlyUsedWords(IEnumerable<string> words)`


Maybe I'm not remembering this correctly... Does authoring have it's own domain?


If you do not have any special thing in the override method, you can remove it.


Я тестировал этого провайдера у себя и для удобства ставил версии, теперь когда провайдер станет обще доступным, то имеет смысл начать версии с начала и с 0.

У меня была версия на пару минут, сделал изменения - новая версия, проверил...


As mentioned, this causes problems elsewhere through the code. This was the simplest way to avoid too many other alterations.


seems like this is actually loading *all* constants, not just fat ptrs -- but I guess the only case where we were indirect before was fat ptrs?


it will free if rx_list->free = free. if its null it will not free anything :)

> On 15 Mar 2016, at 17:52, 刘智猷 <notifications@github.com> wrote:
>
> r_list_free(rx_list) will free the elements in it, right? I did not check very carefully since rx = r_regex_new (tok, ""); also allocates. So i believe we neither have leaks before nor now ^_^
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub:
> https://github.com/EasyHard/radare2/commit/8d835ed5e2fc387c9665566aa1d5a60f3d903784#commitcomment-16713046 <https://github.com/EasyHard/radare2/commit/8d835ed5e2fc387c9665566aa1d5a60f3d903784#commitcomment-16713046>


xóa linh đi hị


@enisoc Please have a look at this.

From what I saw in the test we currently always enter lame duck and do not exempt the special cases you mentioned in the comment?

Let me know if you disagree.


Why did you change indentation?


Can fail at response[0]?


let's hold off on generic constructors for now, and just make NewUnixServer()  and NewUnixTLSServer. It's not clear that other protocols wouldn't require some kind of special treatment like tcp.


You could use a lambd instead of the single-use rank_cmp so we don't have to go searching for the implementation.  Or, at least put rank_cmp just above this declaration.


No, `trans_get_discr` would try to `Load`, but we always have the right-size discriminant (it's the same type `trans::adt` computes for a C-like enum), and it's an immediate, `operand.immediate()` makes sure.


With python 3.5 i'v got next

...
2016-03-15 17:27:24,479 - pyswagger.core - INFO - resolving: [#/paths/~1pet~1{petId}/post/parameters/1]
2016-03-15 17:27:24,479 - pyswagger.core - INFO - resolving: [#/definitions/Tag]
2016-03-15 17:27:24,480 - pyswagger.core - INFO - resolving: [#/definitions/Tag/properties/name]
2016-03-15 17:27:24,480 - pyswagger.core - INFO - resolving: [#/definitions/Tag/properties/id]
2016-03-15 17:27:24,480 - pyswagger.core - INFO - request.url: //petstore.swagger.io/v2/pet
2016-03-15 17:27:24,480 - pyswagger.core - INFO - request.header: {}
2016-03-15 17:27:24,480 - pyswagger.core - INFO - request.query: []
2016-03-15 17:27:24,481 - pyswagger.core - INFO - request.file: {}
2016-03-15 17:27:24,481 - pyswagger.core - INFO - request.schemes: ['http']
<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Pet><id>1</id><name>Tom</name><photoUrls><photoUrl>http://test</photoUrl></photoUrls><tags/></Pet>
Traceback (most recent call last):
  File "/home/cd/data/projects/python/pyswagger/test_swagger.py", line 61, in <module>
    body=pet_Tom
  File "/home/cd/data/projects/python/pyswagger/pyswagger/contrib/client/requests.py", line 55, in request
    raw = six.StringIO(rs.content).getvalue()
TypeError: initial_value must be str or None, not bytes

To fix it enough to replace six.StringIO with six.BytesIO.

But i think here the problem with encoding. When response will be in not utf-8. Client will try to decode binary like utf-8.

For binary format encoding not important. But for json and xml we need make deal with.

If client not working with encoding who will work with it?


I am mildly surprised to see this code **here** and not in trans. Surely the trans code can figure this out based on the fn being called?


@kaweesi , no I didn't look for any improvements to make, I was only aiming to release the code that had been committed a while back and had been in a snapshot for a long time.  Are there improvements you would like to see made to this module?  Hap


Thanks for the PR. Some small tweaks in the comments above.


@kaweesi , no I didn't look for any improvements to make, I was only aiming to release the code that had been committed a while back and had been in a snapshot for a long time.  Are there improvements you would like to see made to this module?  Happy to discuss.


Wait, what kind of problems do apostrophes cause in vim?


then... тут асинхронно


this seems fine.


#if instead of ifdef


Très joli :+1:


oh oh I see, this is on the incoming parameters. hmm, ok


#3 This fixed that issue


Good. Exactly how this was intended to be used.


This if-statement makes no sense here. This code would only make sense if the first-page mode was limited to the first page. But it is not. Dont try to guess the state from the report-progress-event, let the CahingHTMLOutput update that status directly.

The way the system works, we allow the user to select which page he wants to see. This is controlled by the "accepted-page" parameter. When the reporting engine has generated that page, we signal this fact by setting the status to "CONTENT_AVAILABLE".

You should do that in the CachingPageableHTMLOutput.CacheListener when you have seen the accepted page.


occurr


I am suggesting in Protocol.java we set the default to "" for the affected Schemas. That handles converting nulls to "" in the protocol instead of needing to be done manually.

This is how it was handled in the client id fix:
```java
new Field("client_id",
   NULLABLE_STRING,
   "A user specified identifier for the client making the request.",
   ""));
```


Even in a single-mon cluster, we may need to expand the persistent features.  Should this go in finish_election() instead?  Or, put it in MonmapMonitor's on_active?


I deleted the `develop` branch yesterday after merging the SpeechToText PR.

We found that the `develop` branch significantly slowed us down. There were bugs we fixed in `develop` that were ready to merge into `master`, but lived side-by-side with code that was _not_ yet ready to merge into `master`. Because we had to wait for all of the code to be ready to merge, a lot of important, quick bug fixes were delayed.

Moving forward, we would like to work closer to `master`. Our model is:
1. Branch from `master`.
2. Add new code or fixes to the branch.
3. Submit a PR to merge the branch into `master`.

That allows us to move quickly with small bug fixes. Larger feature changes would just have a longer-lasting branch while under development.


You should use extension to make your code more clear. Example:

```Swift
extension MoviesViewController: UITableViewDelegate, UITableViewDataSource {
    // Implement methods of table view
    func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        // Your code
    }

    func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        // Your code
    }
}
```


Hi~I'm HaoYu


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - i386-Windows-7-VisualStudio-12: OK (2139 of 2139 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)


after you set the total-pages here, also update the status to "CONTENT_AVAILABLE" to signal that it is now safe to query the page.


Too many blank lines here.


I did. I get syntax highlighting locally with the markdown blocks. I think we should keep them.

David

On 03/15/2016 10:15 AM, Oliver Beckstein wrote:
> In my local Jekyll they didn't. The liquid tags are ugly but always work.
>
> Did you test locally?
>
> --
> Oliver Beckstein
> email: orbeckst@gmail.com
>
>
> Am Mar 15, 2016 um 10:09 schrieb David Dotson <notifications@github.com>:
>
> > No, the markdown versions work just fine. See here for an example. Please change them back.
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly or view it on GitHub
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub <https://github.com/MDAnalysis/MDAnalysis.github.io/commit/a5c19b5a820dbb741c74c4d9ea67177a7208d018#commitcomment-16713654>
>


Nit: this should have a comment giving some example source code and showing what is happening :)


Yeah I think you're right... You can see it in a later commit https://github.com/Connexions/cnx-deploy/commit/9e4b0df370a89787f6987b62cae80180d78ecac6


Sorry! The filenames are named after the URL, so to make that one stand out I was going to give it the correct name, but while I was naming it I realised I didn't know the uniqueID so I put blahblah in place of the ID! Apologies!


Par contre, j'ai pas trouvé de moyen simple d'intégrer le code dans "index.html" par simple appel (type iframe, embed, link import etc), ça pose des problèmes de glissière sur le côté...
Du coup, c'est mieux de faire les modifs directement dans le fichier d'index pour le moment ;-)


Looks like unnecessary blank line. Don't put a blank lines after { and before }, the prefered format should be

```Swift
someBlock {
  someOtherBlock { // no blank line here
  }
  // blank line here is okay
  someVar = someMoreCode
  ...
  // blank line here is okay
  someFinalCleanup() // no blank line at end of block either
}
```

In short, in a block, no blank line right after the start { and right before the }. Blank lines are welcome anywhere else in the body though.


This update is good. But it's not quite correct. This would drop support for potential development use cases; and that's not a problem since it's not a focus at the moment. Webview doesn't communicate directly with authoring, right? This setting should probably be ``host: location.hostname,``, as it is in the webview codebase.


Yes, `trans_load` doesn't *actually* load by-reference operands, maybe it's badly named?


What about OFFSET_COMMIT_REQUEST_PARTITION_V0 and OFFSET_COMMIT_REQUEST_PARTITION_V1?


Just to clarify, this function is determining the remaining margin, or buying power, given a particular order direction, but not a full order. So if we were long and wanted to go short, what is the maximum margin available to me. I had the same thought you did, though, about using a fractional order size but I feel it just doesn't make total sense for this function.  I tend to lean towards #2 (with mods) but think we just need to write down some examples and test it. #2 makes the most sense to me for the following:

When you initially purchase something it costs you in initial margin, but when you hold it costs you in maintenance margin. If you're flipping your direction we need to account for the fact that it's free margin to go towards zero (closing a position), so in order to do this we need to account for the cost basis, which would be holdings cost * initial requirement. On the other side of the zero crossing that margin is technically a credit, but not for the full initial, since the position was only consuming maintenance. So we need to add in the holdings cost * maintenace.

So maybe most correct is a hybrid of the many suggestions. Spoof a market order to represent the 'closing position' to zero in order to compute the initial margin that we would get for free (we get this for free since liquidating doesn't require margin).
```C#
var closingMarketOrder = new MarketOrder{Quantity = securtiy.Holdings.Quantity};
var freeMarginForCloseOrder = GetInitialMarginRequiredForOrder(security, closingMarketOrder);
```
Now we need to account for the added margin from the liquidation which would equal the current maintenance margin for the security
```C#
var freedMarginFromLiquidation = GetMaintenanceMargin(security);
```
Now adding in the portfolio's outstanding remaining margin we get the following:
```C#
var marginRemaining = freeMarginForCloseOrder + freedMarginFromLiquidation + portfolio.MarginRemaining;
```

Once again, I think we just need to add some good examples to have any confidence in this, but this is where my thought process led for the flip case.


You can also remove 2 override functions if you don't do any special thing in them.


Ok, so we can abandon this commit then, yeah?


@mike-koch Yes, it should. Never knew that (or perhaps the behaviour in VB.NET is different).
The thing is, it never finds a space, because `while ((line = readCity.ReadLine()) != null)` works with both Linux and Windows line endings (and city names do not contain spaces).


@ntwieneke  Great choice, always shipit!  -klq


Ah, got it. Makes sense to me.


No worries! And again, thanks for the fix :). I felt incomplete without my cat ;).


@adzenith It's ok. Sorry if I was hard.


This ``512`` should be the ``SECTOR_SIZE`` constant defined somewhere (if it's not already).


It seems like this name could be better.  diff_not_in(other) maybe?


I _think_ I'm doing this the way Jose suggested. This PR points to the upstream branch, but I'm not sure what we do next.
https://github.com/biocore/emp/pull/56


Fixes Solaris-Skunk-Werks/solarisskunkwerks#9


I'll remove it.


But, there is no need to wrap 2 exceptions if `NoForemanResourceAvailableException` would implement `ExecutionException`.


Er, shouldn't it be ```@drawable/fossasia```?


Think he means https://github.com/TCPR/Thermos/blob/master/patches/net/minecraft/world/chunk/Chunk.java.patch#L280


shouldn't there be an `idx += 1` here?


Doesn't need to be. It's only registering a module, it's not relying on one being registered.

We can make ngtemplates register a template module named after the module with `.templates` suffix, then there's no required order of any of the script files.


I'd remove this assertion, if there's a problem the next assertion will fail


@ntwieneke
Make sure to add line breaks for your code. Right now there are none.

You answer is correct. Or you can just type ```python --version``` in your terminal, that way you don't need to start a python interpreter.
-klq


in particular, something like:

This argument was "de-tupled", so now we have to reconstitute the tuple locally.


Since we don't talk to pre-hammer mons (right?), we can probably drop this condition?


ok


I'd suggest:  The first two EFI partitions are necessary to ensure the device can still boot after troublesome events such as a power loss during a software update, or errors during deploymnet of a new EFI combo application.


(ideally a bit more detailed)


That user expect this config to be presented on global config page in cloud section not in any custom global config section.


OK


@kraj : This breaks all non raspberrypi builds (AGAIN!!!).

Could you perhaps try "include recipes-core/images/rpi-basic-image.bb" instead of "require" ?

Or maybe base wpe images on top of images available in oe-core ?


Why not sprt [0,4] ?


Dont query the content-entities to find out the page-number. The ReportProgressEvent has a "page" property that you can query directly. It will tell you the page the report is processing. Note that this "page" is the current page the report is working on. So while the first page is filled, this will return 1. You will be able to deliver the first page as soon as the "page" property is greater than 1.

So to check whether your content is ready, use

"reportProgressEvent.getPage() > acceptedPage"

as condition.


I'd remove this one as well, same reason


it'd be nice if we had an example to show this case; but this is basically saying that if you have `(i32, Box<Trait>, u32)`, that will get passed as `(i32, *mut Trait, vtable, u32)`?


@ntwieneke
Great choices! I have not personally used Pycharms before. But several people in the bootcamp are using it.
-klq


I think so, I guess we don't stress this enough during bootstrapping.


ok, could use some comments


This happens on a per-mon basis, right?  That means I can't abort an upgrade without recreating mons.

Should this instead get set when the quorum features include jewel?


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28810/testReport/) for this commit**


Why change this?


Hey, it's deleted now.  I missed the notifications in my github folder. I'm going to delete my comments soon.  Yeah, it was someone trying to make some advertising spam.  I contacted GitHub support and they deleted it. I thought it might be a vulnerability of some sort (i.e. somehow the page would be executed).


We have actually forked this to
https://github.com/rwanda-rbc-emr/openmrs-module-addresshierarchyrwanda to
upgrade openmrs platform to 1.11.5

On Tue, Mar 15, 2016 at 8:20 PM, Michael Seaton <notifications@github.com>
wrote:

> @kaweesi <https://github.com/kaweesi> , no I didn't look for any
> improvements to make, I was only aiming to release the code that had been
> committed a while back and had been in a snapshot for a long time. Are
> there improvements you would like to see made to this module? Happy to
> discuss.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/PIH/openmrs-module-addresshierarchyrwanda/commit/55c1e718be94afa407bf49792ba7b38bd4748001#commitcomment-16713796>
>

--

*Kaweesi Joseph*

*Jembi Health Systems NPC* *|  **Uganda*

*Mobile: **+256784994313** | Office: **Software Developer** | Skype:*
*kaweesi.joseph*
*E-mail: kaweesi@jembi.org <kaweesi@jembi.org>*

*This e-mail contains proprietary and confidential information some or all
of which may be legally privileged. It is for the intended recipient only.
If an addressing or transmission error has misdirected this e-mail, please
notify the author by replying to this e-mail and then deleting same. If you
are not the intended recipient you must not use, disclose, distribute,
copy, print or rely on this e-mail. Jembi Health Systems NPO, its
subsidiaries and associated companies is not liable for the security of
information sent by e-mail and accepts no liability of whatsoever nature
for any loss, damage or expense resulting, directly or indirectly, from the
access of this e-mail or any attachments hereto.*


See if commit 948c1895d6d23d14ba46f6e7f4346202a69b8ade handles what you're trying to do, just as well.  You'd just "ln -s ./vmware ./vmplayer; sudo ./vmplayer install" and it should install as vmplayer instead of vmware.  If it doesn't work, let me know!


ok: f1_nu has the smallest magnitude of the range of negative values of a1, f2_opp_nl has the largest magnitude of the negative range of a2


change to: ... might want to add other partitions for special purposes, for example..."


As the project owner, can't you delete any comment, or just yours?


change to "different"


Sorry this took so long -- I just assumed I would get notified of pull requests, and just saw this.


OK, next on my list.  Please let me know when you think there is no more
point to be raised and I'll work through it.  Also, if there is any tooling
available to ensure whatever guidelines you want to be followed,
in an automated way, please let me know as well.


This check is only checking if ASTSPOOLDIR exists. It needs to be ```$temporaryAudioFile.'.wav'```


I'm actually a huge fan of the excessive flowers, but I'm starting to think it might be a bit too much. Placing the shrubs first is a good idea for sure and might be just what it needs.


Hey, are you having any luck with Cogeco -- it'd be awesome if you got it working.


Yes, because *each of those 3* is a separate argument, although `*mut Trait` would include the vtable pointer too, you mean `*mut u8, *const VtableForTrait`, right?


change: "it is sufficient to copy" to "simply copy"


Sorry, I had not noticed I had any pull requests -- merging now. Thanks!


This is a system to track software bugs. Stop abusing it for your personal troubles. Go talk to a mirror if you have such a huge want to communicate.


ok


@ntwieneke
Great job! Good explanation of the xargs command.


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


The following doc(s) have been deployed to the [simpleurls](http://simpleurls-static-docs.pantheonsite.io/docs) Multidev environment:
- [_docs/content-delivery-network](http://simpleurls-static-docs.pantheonsite.io/_docs/content-delivery-network)
- [_docs/logs](http://simpleurls-static-docs.pantheonsite.io/_docs/logs)
- [_docs/migrate](http://simpleurls-static-docs.pantheonsite.io/_docs/migrate)
- [_docs/new-relic-analysis](http://simpleurls-static-docs.pantheonsite.io/_docs/new-relic-analysis)
- [_docs/newrelic](http://simpleurls-static-docs.pantheonsite.io/_docs/newrelic)
- [_docs/settings](http://simpleurls-static-docs.pantheonsite.io/_docs/settings)
- [_docs/unsupported-modules-plugins](http://simpleurls-static-docs.pantheonsite.io/_docs/unsupported-modules-plugins)
- [_docs/user-dashboard](http://simpleurls-static-docs.pantheonsite.io/_docs/user-dashboard)
- [_docs/user_dashboard](http://simpleurls-static-docs.pantheonsite.io/_docs/user_dashboard)
- [source/_partials/navbar.html](https://github.com/pantheon-systems/documentation/commit/bca7942f758a1e6f1ab2b2f6d43ff4d36781d91b/source/_partials/navbar.html)
- [source/_partials/search_box.html](https://github.com/pantheon-systems/documentation/commit/bca7942f758a1e6f1ab2b2f6d43ff4d36781d91b/source/_partials/search_box.html)
- [source/_views/doc.html](https://github.com/pantheon-systems/documentation/commit/bca7942f758a1e6f1ab2b2f6d43ff4d36781d91b/source/_views/doc.html)
- [source/docs.html](https://github.com/pantheon-systems/documentation/commit/bca7942f758a1e6f1ab2b2f6d43ff4d36781d91b/source/docs.html)
- [docs/_guides/sendgrid](http://simpleurls-static-docs.pantheonsite.io/docs/_guides/sendgrid)
- [docs/_guides/two-factor-authentication](http://simpleurls-static-docs.pantheonsite.io/docs/_guides/two-factor-authentication)
- [docs/_guides/visual-diff-with-wraith](http://simpleurls-static-docs.pantheonsite.io/docs/_guides/visual-diff-with-wraith)
- [docs/_guides/wordpress-commandline](http://simpleurls-static-docs.pantheonsite.io/docs/_guides/wordpress-commandline)
- [_docs/access-management](http://simpleurls-static-docs.pantheonsite.io/_docs/access-management)
- [_docs/add-ssh-key](http://simpleurls-static-docs.pantheonsite.io/_docs/add-ssh-key)
- [_docs/change-management](http://simpleurls-static-docs.pantheonsite.io/_docs/change-management)
- [_docs/clone-new-site-backups](http://simpleurls-static-docs.pantheonsite.io/_docs/clone-new-site-backups)
- [_docs/database-workflow](http://simpleurls-static-docs.pantheonsite.io/_docs/database-workflow)
- [_docs/delete-account](http://simpleurls-static-docs.pantheonsite.io/_docs/delete-account)
- [_docs/delete-site](http://simpleurls-static-docs.pantheonsite.io/_docs/delete-site)
- [_docs/deploys](http://simpleurls-static-docs.pantheonsite.io/_docs/deploys)
- [_docs/faq](http://simpleurls-static-docs.pantheonsite.io/_docs/faq)
- [_docs/generate-ssh-key](http://simpleurls-static-docs.pantheonsite.io/_docs/generate-ssh-key)
- [_docs/getting-started](http://simpleurls-static-docs.pantheonsite.io/_docs/getting-started)
- [_docs/getting-support](http://simpleurls-static-docs.pantheonsite.io/_docs/getting-support)
- [_docs/lock-environment](http://simpleurls-static-docs.pantheonsite.io/_docs/lock-environment)
- [_docs/machine-tokens](http://simpleurls-static-docs.pantheonsite.io/_docs/machine-tokens)
- [_docs/multidev](http://simpleurls-static-docs.pantheonsite.io/_docs/multidev)
- [_docs/organization-dashboard](http://simpleurls-static-docs.pantheonsite.io/_docs/organization-dashboard)
- [_docs/organization-faq](http://simpleurls-static-docs.pantheonsite.io/_docs/organization-faq)
- [_docs/organizations](http://simpleurls-static-docs.pantheonsite.io/_docs/organizations)
- [_docs/pantheon-workflow](http://simpleurls-static-docs.pantheonsite.io/_docs/pantheon-workflow)
- [_docs/platform-considerations](http://simpleurls-static-docs.pantheonsite.io/_docs/platform-considerations)
- [_docs/quicksilver](http://simpleurls-static-docs.pantheonsite.io/_docs/quicksilver)
- [_docs/required-reading](http://simpleurls-static-docs.pantheonsite.io/_docs/required-reading)
- [_docs/resetting-passwords](http://simpleurls-static-docs.pantheonsite.io/_docs/resetting-passwords)
- [_docs/security](http://simpleurls-static-docs.pantheonsite.io/_docs/security)
- [_docs/select-plan](http://simpleurls-static-docs.pantheonsite.io/_docs/select-plan)
- [_docs/site-access](http://simpleurls-static-docs.pantheonsite.io/_docs/site-access)
- [_docs/site-owner-faq](http://simpleurls-static-docs.pantheonsite.io/_docs/site-owner-faq)
- [_docs/site-payments](http://simpleurls-static-docs.pantheonsite.io/_docs/site-payments)
- [_docs/sites](http://simpleurls-static-docs.pantheonsite.io/_docs/sites)
- [_docs/team-management](http://simpleurls-static-docs.pantheonsite.io/_docs/team-management)
- [_docs/update-payment-method](http://simpleurls-static-docs.pantheonsite.io/_docs/update-payment-method)


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28811/testReport/) for this commit**


EFI is in quotes because only one is really EFI, the other does contain EFI data, but it's not formally marked as EFI.
Yes, I was trying to convey a lot, with 2 double quotes.


I've added an explanation for this in the commit note.


Szkic strony do poprawki. Dołącze wytyczne


Por quê?
Você pode dar exemplos?
Também seria bom colocar um exemplo de código em que Perl se destaca das demais.


Arrgh, typo :p
Thanks for the heads up.
This issue most probably crept in as i made a new branch before pushing changes :/


Not sure I follow. Yasnippet is the Emacs package needed to use these snippets, and the snippets themselves are already in a PR...can you clarify? Thanks!


@sils1297 I think we need `abspath` here as `__path__` is returned to `__path_list__` which is returned to `path_list` or `glob_list` which is `collect_files` which calls `icollect` and checks if `os.path.isfile` at https://github.com/coala-analyzer/coala/blob/master/coalib/collecting/Collectors.py#L67


Confirmed ok now, thx!


latex-documentation (michael-okeefe) - Win64-Windows-7-VisualStudio-12: OK (2125 of 2135 tests passed, 19 test warnings)

 * 18 tests had: AUD diffs.
 * 11 tests had: RDD diffs.
 * 10 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 9 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 7 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 3 tests had: MTR small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


@knosach  any luck on leaving the original 500ms as @deruelle  suggested, so we can merge the changes?


FWIW I tried a generalized version where I take an SCC, divide it in parents and children (where a parent is any node in the SCC that also has a submodule/subpackage in the same SCC, and children are the rest), and then apply sorted_components() to the graph restricted to the children only; I then add the parents back in reverse order (so the most ancestral package would go last).

For the sqlalchemy cycle this gives a perfect result (sorted_components() on the children returns all singletons).

But for a specific SCC of interest with an initial size of 578, this identified 15 parents and chopped off 39 other modules, still leaving an SCC of 526 nodes. That's disappointing. So I'm going to try harder, looking into sorting within subpackages first.


Description... should this better be something like this?
`"A cross-platform virtual tabletop software for multiplayer card games"`
or `${PROJECT_NAME}": A cross-platform virtual tabletop software for multiplayer card games"`


Damn. My bad.

What about another hunk?


`x` should be `self.matrix`. Sorry.


Checked the codegen for these tests, and this LGTM


I also like wikia more since that is usually more accurate, but sadly no text has been edited there yet :/


I've amend the commit note length.


and also fixes issue #43


ln 17 should be deleted and ln 16 to be restored, otherwise it jumps to wordpress.org and tries to index those pages


can't we just call `tcx.item_has_attr` or whatever?


This `’` should be `'`. What I get for copying from a PDF.


@daN4cat & about invoicing: just take quick look at Austria and Italy.
In both countries (more in Austria and here especially a 'must' starting not later than 1.1.2017) every shop which has a electronic cash register has to hand over an invoice!
In Italy this requirement was stopped in Summer 2003, there now the Invoice has to be hand over to the customer if he wants that.
Germany may follow the next 2 years with such a regulation.

Therefore I recommend to let the admin define this setting in the config, eventually on a 'per user base'.
With such settings each store owner can handle how the country legal regulations are defined.


Hello @andresgutierrez, why has it been removed ? Some "not very well" written Bundles from packagist uses it, and it's really much more proper to have only one autoloader instead of two (composer's one and phalcon's one).


Thanks! Uploaded :)


add in a test paramater, so the host doesn't actually shut down.


Likely. Neither method will work for closures cross-crate, sadly.


here goes..

[kmsg.log.txt](https://github.com/jcadduono/nethunter_kernel_herolte/files/174468/kmsg.log.txt)


it'd be nice to have this pulled out into a function so we can just do `mir_for(def_id)`


@misscs Becki thought we may want to start using it more, so i'll leave it unless they decide otherwise


Whyyy? :) This brings back a lot of unused `this` variables.


Procure em algum congresso nacional coisas relacionadas a linguagens e modelos de programação para robôs.
Esse aqui também é bom:
https://www.rfc1149.net/download/documents/scpr2008-08-Baillie.pdf


I am still confused what we shall do. We eventually want to add benchmarks for weighted paircounting as well. What about reverting the benchmarks to the original one, and defer a massive benchmark 'jump' or 'invalidation' in 0.18, where we probably will clean up the test cases for spatial as well?


If you see an acceptedPage of less than zero, then your JavaScript client is still invoking the parameter-handler with renderMode=XML. In async-mode this is no longer needed as you will receive the total-page-number via the status from the job-manager.

So instead of pretending that the user requested the first page, stop your report-viewer from paginating while querying parameters. You can find the source of that call in "_updateReportContent" in "reportviewer.js". When the async mode is enabled, there is no need to fetch the page-count via that method.

And once you have fixed that, restore the old code here, as you wont encounter that state here anymore. However, old clients that are not aware of the async-mode should continue to generate in a non-caching fashion and the old code achieved that.


@misscs I moved trial-blue into the trial_warranty module config file


use always mailTextArea.setId method and declare it in styling.css, like:
'#'mailTextArea{
-fx-font-size: 23px;
-fx-text-alignment: center;
-fx-border-color: #00FF00;
-fx-border-width: 3px;
-fx-effect: dropshadow(gaussian, black, 10, 1.0, 100, 100);
-fx-alignment: center;
-fx-text-fill: green;
-fx-fill-width: true;
}


Slowly realization with BFS algorithm.


this list misses sushi samba


Why is this required? Can info be NULL here?


I agree that would be a nice refactor, but I wonder if we can leave that to another PR, this one is really about moving state out of the flow and into a calculator class that can be unit tested, and to remove all the variables passed between outcomes and variables.


`trans::collector` has that, but it doesn't populate everything `trans` needs, for some reason (I forget what kind of function I hit, might have been a closure?). cc @michaelwoerister


= for endless debug


Can you see eagles in the park?


As I've added an explanation about the guard statement for accessing `passport_data`, I wonder if it's worth repeating in every commit where the same pattern is used?


I feel like this should go into another module `mir_cache` or something


Fixed.


Individual time pr. dimension? What's the advantages?


This code seems wrong!
Because we need to traverse whole column too! we are traversing just the seed length and then its jumping to the next row.

Also, "column" variable should be "row" variable but that doesn't matter :D :P


k


lol


Also...can/should this be a `Cow`? I guess not worth the trouble.


Would putting it in `trans/mir/mod.rs` work for you?


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


ok modulo nits


remove this extra whitespace change.


As discussed in person, how about we add a `visitDataDefinitionStatement()` method to `StatementAnalyzer` and create the row there thereby remove this if statement?


Why do we need to check the cntr pointer here?

Wouldnt this ignore any counter update errors?


The problem with the conditional restart is that most of the time we're not changing the supervisor config, and archive, authoring and publishing are already running, and they're not getting restarted.


typo: true -> tree


Same comment as above.


Should fix the "no newline at end of file" issue, here and in the other file.

I guess we should add a lint check for missing newlines at end of file, in general.


Testing with all relevant codecs is a good idea, but this is the wrong way to do it. Arbitrary is for generating a random sample of values from some (effectively) infinite set. When you have a small number of values, you want to test every value, especially when each value represents a different code path, as they do here.

So I think what you want here is something like:
```scala
def printAndParseTests(codec: PathCodec) {
  ...
}

"PosixCodec" should {
   printAndParseTests(posixCodec)
}

"WindowsCodec" should {
   printAndParseTests(windowsCodec)
}
```


Absolutely, although not as many eagles as you would see in Alaska, there are a few mating pairs with nests.


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


return value ignored?


It's incorrect and I was making very similar changes already. Practically speaking it doesn't make a difference right now.  visitStatement() never gets overridden, so ultimately it always ends up calling visitNode().  But Insert extends Statement, so if someone were to create a visitor that overrides visitStatement() from the AstVisitor,  but does not override visitInsert(), then insert should get the visitStatement() behavior too.


Are those 2 indentation changes really needed?


`Cow` has mutable semantics, so no.


Just writing random questions


[tests/testthat/test-corrplot.R:34:42:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L34) *style:* **Only use double-quotes.**
```r
  expect_equal(corrplot.mixed(M, lower = 'circle', upper = 'number',
                                         ^~~~~~~~
```
[tests/testthat/test-corrplot.R:34:60:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L34) *style:* **Only use double-quotes.**
```r
  expect_equal(corrplot.mixed(M, lower = 'circle', upper = 'number',
                                                           ^~~~~~~~
```
[tests/testthat/test-corrplot.R:35:58:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L35) *style:* **Only use double-quotes.**
```r
                              low = L, upp = U, plotCI = 'rect'), M)
                                                         ^~~~~~
```
[tests/testthat/test-corrplot.R:36:42:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L36) *style:* **Only use double-quotes.**
```r
  expect_equal(corrplot.mixed(M, lower = 'number', upper = 'circle',
                                         ^~~~~~~~
```
[tests/testthat/test-corrplot.R:36:60:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L36) *style:* **Only use double-quotes.**
```r
  expect_equal(corrplot.mixed(M, lower = 'number', upper = 'circle',
                                                           ^~~~~~~~
```
[tests/testthat/test-corrplot.R:37:58:](https://github.com/vsimko/corrplot/blob/b79dfa9a88b008c7114b4d963613a63e7a685d89/tests/testthat/test-corrplot.R#L37) *style:* **Only use double-quotes.**
```r
                              low = L, upp = U, plotCI = 'rect'), M)
                                                         ^~~~~~
```


shouldnt b & c be always non-zero? If so, shouldnt we assert?


Extra "translate"


+1  (We need to still document the -i <imagename> option because the M3 (399) build does not have the script change in it, while the 405 build does).  The curse or rolling releases - how to keep the documentation "correct" when the tools change)


@daosgc de momento sólo vamos a soportar youtube


Can we merge this with next commit?


The 64-bit MSBuild Version is located under `ProgramFiles(x86)`

However the path to the 64-bit msbuild version still is not correct according to this article i have found: https://blogs.msdn.microsoft.com/visualstudio/2013/07/24/msbuild-is-now-part-of-visual-studio/

> On 32-bit machines they can be found in: C:\Program Files\MSBuild\12.0\bin

> On 64-bit machines the 32-bit tools will be under: C:\Program Files (x86)\MSBuild\12.0\bin

> and the 64-bit tools under: C:\Program Files (x86)\MSBuild\12.0\bin\amd64

I will change the code to this


Thanks


Rather than adding this, why not make the new constructor you've added take a ProjectionTree to initialize from, so that the new ProjectionTree is fully initialized upon construction?


It's the chacha8 test vector we'd really care about, but only chacha20
comes from an independent source citation.   Automatic derivation of the 8
test vector from an alternative implementation would be a good start.

Also, it would be irresponsible to use an in-house RNG implementation
without doing basic tests of its statistical properties, e.g. via diehard.

Best,
Alex

On Tue, Mar 15, 2016 at 1:08 PM, riastradh-probcomp <
notifications@github.com> wrote:

> Crypto test enabled in 66a1235
> <https://github.com/probcomp/crosscat/commit/66a12354264905fe888edf1269253d140c9bb564>.
> Agreement with the Python code enabled and tested in be3eb92
> <https://github.com/probcomp/crosscat/commit/be3eb9205ae2aeb6adf0b1cd038162d6f9bb3e6b>.
> Test vector source cited in d60f304
> <https://github.com/probcomp/crosscat/commit/d60f30449870d0aee4b0ad85705566f43f4bf590>
> .
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub:
>
> https://github.com/probcomp/crosscat/commit/6535957d2de7194a649f56eb7a9e2409d1f231c7#commitcomment-16713477
>


The comment above this doesn't match this particular constructor. I would suggest moving this down and giving it a comment for what this particular constructor is for.


@dwaynebailey It seems that the typo just mutated to a new form.


Can this be squashed with the first commit?


``+ ${route['height_diff_up']}``
=> I think there should not be a space after ``+``(ditto for ``-``above).


sock_pe_new_rx_entry always return 0


Can we split the part of the security group and squash it with the first commit where the security group is added?


Obrigado, farei isso.

Hoje não irei pra sua aula, preciso estudar pra um teste de cálculo 4
amanhã.

Valeu Chico.
On Mar 15, 2016 14:39, "fsantanna" <notifications@github.com> wrote:

> Procure em algum congresso nacional coisas relacionadas a linguagens e
> modelos de programação para robôs.
> Esse aqui também é bom:
> https://www.rfc1149.net/download/documents/scpr2008-08-Baillie.pdf
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/HandelSM/Ceu-arduino/commit/5e043b32daddcaded1e2d14e546aa481cb4b370c#commitcomment-16714320>
>


Hello, I'm working on a fork of this project and I realized that the xapi_type, which will fill the object definition, is hardcoded here:
protected static $xapi_type = 'http://lrs.learninglocker.net/define/type/moodle/';

I'd like to know if it's still correct since, when used in a course context, it then points to a page that returns "Status: 404 Error: Error in exception handler."

I feel it would be more appropriate for a course object definition URI to refer to the tincan definition here:
https://registry.tincanapi.com/#uri/activityType/1

Please let me know if my statements are correct,
looking forward to hearing from you,
best regards,
Marcello


Oh, I think you fixed the issue I was hunting for a few days.
What did you do exactly (what did you rename it to)?


@hannesmuehleisen i don't think `sqlsurvey` is used anywhere anymore  <grin>


Can you explain in the commit message that when one uses "protocol = tcp" no defining the `from_port` and `to_port` will result on only opening the port 0.


Yeah, there needs to be a way to tell supervisor to restart one of the services. Unfortunately that doesn't exist yet... Let's put a ``FIXME (15-Mar-2016) ...`` in to mention that we need more fine-grained restarting of supervisor managed processes.


Fixed in 97ff77417c3a8ce10951.


I've tried to capture that with the second line in the commit note:
```
The value returned by `supporting_documents` changes if the user visits
the birth_location question and enters a value that is not the united
kingdom.
```
I guess the key difference is we're moving the setting of the value from a specific point in the flow, to the calculator where the method can be invoked at any point, so it needs to return a valid value if birth_location hasn't been answered.


How about renaming this to `rowCountQueryMetadataRow()`? I would even add a one line comment explaining what this method does. Something along the lines of "Build an output row in the format expected by DESCRIBE OUTPUT for row count queries".

Also, you extracted this into a method but you didn't use it on line 362.


Never put real tokens in code


update


very good


missing case test for accordion group without options


The compilation server gets in the way. Technically we only have to keep these variables within their functions, i.e. not if we inline them. I don't really know how to distinguish that though.


Line 509 already checks if info is NULL


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - 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)


I can't remember if it was fully fixed or if the fix caused another issue. I'll try to have another look at this over the weekend if I'm able.


Ahhhh. Just found an error.
`const Bitboard Center =
 110   +       (Us == WHITE ? in_front_bb(Them, RANK_5) & (FileCBB | FileDBB | FileEBB | FileFBB)
 111   +                    : in_front_bb(Us  , RANK_4) & (FileCBB | FileDBB | FileEBB | FileFBB));
`

Should be
`const Bitboard Center =
 110   +       (Us == WHITE ? in_front_bb(Them, RANK_5) & (FileCBB | FileDBB | FileEBB | FileFBB)
 111   +                    : in_front_bb(Them  , RANK_4) & (FileCBB | FileDBB | FileEBB | FileFBB));
`


I would say "... without an Apple Developer Program membership" instead, since you still require a certificate, it's just now transparent to you.


please create a example in the docs for the same


I tested this on Development. It does not parse 'output' to the results page, instead, it is printing it on the console.

[15/Mar/2016 15:13:18] "GET / HTTP/1.1" 200 1101
add
dad


@shadylane318 Great to see you shipping code! Thank you for doing the work.


I've amended to the commit note to try and explain this, I'm not sure how clear it is:
```
The value returned by `supporting_documents` changes if the user visits
the birth_location question and enters a value that is not the united
kingdom, so to cover the scenario where the method is invoked on a
calculator without this value set, we add a `blank?` check to return
a correct default value.
```


Meh :(


Thanks, this has been corrected in the new branch.


And check this out:
```
makeComponents : List (Random.Seed -> a) -> Random.Generator (List a)
makeComponents cs =
  Random.list (List.length cs) Random.splitSeed
    |> Random.map (List.map2 (<|) cs)
```


Not part of this PR, but we should update this statement. iOS 8.x is minimum required as of cordova-ios-4.0 (which is cordova-cli 6.x?)


Maybe we could simply remove Meta.This when inlining them, hmm...


You might want to check if the conn_factory is in fact HTTPSConnection, it may not be the case with an http proxy.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2050 of 2170 tests passed, 179 test warnings)

 * 254 tests had: EIO diffs.
 * 183 tests had: ESO small diffs.
 * 102 tests had: Table small diffs.
 * 73 tests had: MTR small diffs.
 * 113 tests had: ESO big diffs.
 * 54 tests had: Table big diffs.
 * 36 tests had: MTR big diffs.
 * 15 tests had: ERR diffs.
 * 1 test had: AUD diffs.

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


You still need to have an Apple ID however, which is not stated in the PR (although stated in your PR description).


Can we continue calling them `Buffer`s within `Bucket`? It is an implementation detail one abstraction level away that `Buffer`s are implemented with `StructArray`s.


Fixes #15


if the storage is __m128*, this operator[] is a gcc extension and won't work elsewhere


Personally, my view would be that, if we are doing significant new work to upgrade to 1.11.5 that we consider abandoning this module in favor of the widely used "addresshierarchy" module.  The addresshierarchy module was written based on addresshierarchyrwanda, and is a part of the reference application, so moving to it will make any future adaptation of OpenMRS 2.x or Bamni more straightforward.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 19152](https://teamcity.mesosphere.io/viewLog.html?buildId=166212&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:05


Ok, I believe what I wrote above was correct logically, but I flipped the logical check in the implementation.

Here's what's supposed to happen:

`bcBuffer` is allocated in `GetSerializedBuffer`. If the allocation fails in that function, then the buffer is released and a failing error code is returned.

https://github.com/Microsoft/ChakraCore/blob/master/bin/ch/ch.cpp#L156
```
    if (hr != S_OK)
    {
        // In the failure release the buffer
        if (bcBuffer != nullptr)
        {
            delete[] bcBuffer;
        }
    }
//...
    return hr;
```

So if a failing `hr` was returned from that allocation, then the buffer has already been deleted and the pointer returned in the `byteCodeBuffer` parameter is still `nullptr` because it is never assigned the allocated buffer. This becomes the value of `bcBuffer` in the caller, which is `nullptr` in the failure

`bcBuffer` can only be freed in the case that the call to `GetSerializedBuffer` was successful. So these checks should read:

```
if (hr == S_OK && bcBuffer != nullptr)
```

This way the buffer will be freed if we reach this clean-up block on a non-error path, and otherwise will not be freed because there's nothing to free.

I'll push a commit to correct this logic.

Thanks for the review @obastemur


comment should be updated as this is now a point in time, not a duration in the past


I think we need to separate the simd_emulation_ and non-emulation mode and have the emualted use the storage with [] directly and return {{... }} while the other use the [] ont he pack and not the storage and use the init-list ctor of pack.


I have updated this with your suggestion.


imo we should keep this. This tests that the created team has a valid `slug` irrespective of the method used. If somebody was to change from `slugize` to something else in future, you won't have a failing test.


Ah, thanks a bunch :)


@goranche my /etc/hostname file is the blank one.

My /etc/hosts only has one line for localhost as follows:
```
root@pi:~# cat /etc/hosts
127.0.0.1 localhost
root@pi:~#
```
As you can see, I have no "pi" entry in /etc/hosts and as I'm on dhcp I would have expected to have:
```
127.0.1.1 pi
```
in /etc/hosts
odd.


yeah, I noticed that, but it's not relevant anymore since after adding a visitDataDefinitionStatement, I can revert all the changes to DescribeOutput code.  This method will go away.


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


this is unneeded if we return {{ .. }} see how it's done in load in emualtion mode.


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I would prefer an interface for `Buffer` that entirely encapsulates and hides the `StructArray` implementation.

```js
function Buffer(attributes, type) { ... }
```

This interface would necessitate creating a new instance of `StructArrayType` per instance of `Buffer`; however I doubt that'll be a perf bottleneck.


I wonder if these and the rest would be better written with `__class__.__name__`?  I've seen cases where methods have been moved between classes, but the strings not updated..


Nice - thanks for sharing - would be good to merge something like this into the PR.


@brandonsavage This is Day 1 homework. Sorry for including my PHPStorm .idea directory :(


@brandonsavage This is Day 2 homework


Do you mean that order quantity should be equal to `security.Holdings.AbsoluteQuantity` ?
Replacing the implementation of `GetInitialMarginRequiredForOrder` in the equation, the only difference from #2 is the addition of the orderFee.
This is the `GetMarginRemaining` method, I'm not sure if it should include order fees.

#2 can also be written as:
```
security.Holdings.AbsoluteHoldingsValue * GetMaintenanceMarginRequirement(security) +
security.Holdings.AbsoluteHoldingsValue * GetInitialMarginRequirement(security) +
portfolio.MarginRemaining;
```


@ilayaperumalg @jvalkeal I just added one more commit on this same branch: https://github.com/markfisher/spring-cloud-deployer/commit/3e9063f640f5663737cfb461011916cbd54f3ab0


I think we're assuming users will need to create new worker dirs (thus new buildbot.tac's) when switching from buildbot-slave to buildbot-worker, right?  Otherwise, we may need to think harder about this commit.


Can we use `supervisorctl` to restart the specific services?