Indentation could be improved with 2 backspaces, else it's looking good!


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/f1a75b66-203d-4d78-9391-c6c86d4ded44 "JobMatcher"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/3fea10cb-b4d9-4628-9899-2891d2fdef69 "JobMatcher") deployed [JobMatcher](https://github.com/Mbouzon/JobMatcher/releases/tag/deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-dev-20170921-190809 "deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-dev-20170921-190809") to [dev](https://console.bluemix.net/apps/618b653e-f3e1-4811-bcb4-2935fb6798fc?env_id=ibm:yp:us-south "ibm:yp:us-south:marcio.bouzon@br.ibm.com:dev"), including this commit


Method name should start lowercase, and indentation should be one level outer (I guess...)


@ricki-z Works well now? Haven't been able to merge the progress into our branch yet because of time constraints so I haven't been able to try this out.
I personally don't mind having passwords in the form in the config page in clear text. There are so many other security concerns (limited or no ssl capability in combination with weak basic authentication) that for practicality's sake it would even be easier not having the password masked, you know what I mean? :)


Nice! No useless variables, clean code! Awesome!


Yes I agree that it will be better to do that. I can work on it once we start to move pex files to a separate repo :)


Indentation! Huge spaces from 11 to 15! Else looks pretty cool!


This conditional expression is being evaluated twice:

1. Line 6
2. Line 22

Hmm, it might be a place to refactor and create a variable to hold the conditional expression's result.  Then use that variable in place of it.

What are the benefits?

1. The variable will _tell you_ what it's doing.  That makes the `if`s more readable.
2. If you need to change the conditional expression, you only have to do it in one spot.  That reduces errors of _forgetting_ to do it in the other spot too.


https://github.com/Code4SocialGood/c4sg-web/issues/1300 image sizes issue merge pull request


+1 for the function name :)


![](https://github.trello.services/images/mini-trello-icon.png) [NFR26](https://trello.com/c/VscRkN7L/33-nfr26)


Huuuuuuuuuuuuuuuuuuge spaces! (i know that you are still going to work on this, but still: INDENTATION!) 👍


Eso faltaba


Good job! Clean code, easy to read! AWESOME!


This is owner-only; it should probably be owner, admin, or dev.


I dislike doing the msg sender as a string instead of just sending the entity and using its name.  Converting the underscores to spaces is pretty easy, and then talking npc's don't require manually typing out their name on every print.

I guess either way I'll still wind up using my own version because I do a lot of faking one type for another (like a "say" message that reaches everyone in "party" without looking like it was  a party member, which means an extra argument to the function)


{
2	  "directory": "bower_components"
3	}


The code wont be finished today. :( Therefore, we have to present next week. This is better in order to get 100%, anyway ;)


Indentation could use two backspaces at lines 9-11, else looks cool! Good job!


edit coz I no talky right:

I guess it doesn't matter either way since I'll still wind up still using my own version because I do a lot of faking one type for another (like a "say" message that reaches everyone in "party" without looking like it was a party member, which means an extra argument to the function)


Some unnecessary empty lines, else looks pretty good!


Created multidev environment [natalie-sandbox#pr-paragrap](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-paragrap).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-paragrap-natalie-sandbox.pantheonsite.io/)


Nice, descriptive variable names, and your code contains every necessary logical steps


You could bring the } here, so you don't have so many unused lines!
Else is pretty awesome! GOGO BUNNIES!


Good job! Nice and clean, good naming! Awesome!


The `itemtype` is more of a [`ImageObject`](http://schema.org/ImageObject) as everything within this `div` describes/supports the image itself.


Remove the `itemprop`.


Add the following microdata:

```
itemprop="contentUrl"
```


Okay, this is very strange, but I found out what's causing this: It's the top of the stacked sector part at X 5800 Y -22000 - when I remove it or at least remove the stacked sector things, the actors work - and the sneakable soldiers don't get waked at the beginning, just as it should be.

I tried moving the sector around without any luck. I also removed the sector and kept the stacked sector things, still the same problem. Only removing the "Upper Portal" things fixed the problem. No idea what's going on though...


2 spaces before "Or"


This patch also adds a `visorchannel_destroy(dev->visorchannel)`, which isn't mentioned in the comment.  Rather than add an "Also..." in the comment, perhaps this belongs in a separate patch?


please next time commit all the files separately :)


These lines are related to #2013.


@GuardianRM looking at the combo test you started (http://tests.stockfishchess.org/tests/view/59c3f3a30ebc5916ff64b5d1) it seems to perform much worse than your original patch. So I think continuing with the original patch is best.


Created multidev environment [nt328-cubear-vol2#ci-1](https://dashboard.pantheon.io/sites/d4ae58db-149f-403d-be1d-315cc7f64f2a#ci-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-1-nt328-cubear-vol2.pantheonsite.io/)


@markwkidd Why did you remove my new Windows pages from the index?


Did you intentionally delete the parens around the last expression here to clarify the operator precedence?  I personally would rather see the parens.


whitespaces missing, especially before/after curly brackets :)


Hi @PabloSichert
please remember to reference the respective issue from your commits.
We would like them to show up in the issue's timeline, and we sometimes want to identify all commits for a particluar issue (via github API :triumph: ) in order to cherry-pick them.
Of course, particularly with this one commit-issue it's not a problem, but..you know :-)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2160 of 2160 tests passed, 0 test warnings)

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


whitespaces missing and unnecessary blank lines


it could have been more simple from the other way around, like:
return string.substring(0, 1) + "*" + changeXChars(string.substring(1))


It's your call whether to propagate the "success: reference kept via unmatched get_device()" comment, but it helped me to understand that we are indeed keeping a reference on purpose.


I think I deleted them to save space (and not go over 80 characters). I am also not sure what the new checkpatch script will do with (), I can put them back in and find out and if they don't violate the space or checkpatch, I will leave them.


This change from Forge default implementation of fluid handling to the BuildCraft custom one seem to have broken the filling of stacked containers #3729


such wow! that's elegant!


it could have been easier in forward direction, like:
return input.substring(0, 1) + "*" + starAdder(input.substring(1))


done :)


Thanks, I will add the comment back in.


nothing to disapprove here 😁


Isn't there room to merge these onto a single line now?


I've fixed the order with this commit: 1c897d0
Regarding int? and uint?: uint? would be better, but the other properties are int?. I know there is inconsistency but changing the type now would make existing code incompatible.


I am not sure, this works, the base case should return a value, no? could you check it? otherwise, the indentation is nice :)


Apologies if I'm mistaken, but isn't this already overriding when laststatus is explicitly set to 0 and 1?

```
if &laststatus < 2
```

I tested `set laststatus=0` and `set laststatus=1` in my vimrc and neither has any effect with vim-airline enabled. It always switches on.


I will try to leave it under the loop, but not the same line as the assignment.


nice fibonacci code :)


Do you know why my linter flagged the new __toString method as "wrong"?


İyi iş!  :D


Ok, so try permissive. That's the only difference between your and my build


Created multidev environment [natalie-sandbox#pr-paragrap](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-paragrap).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-paragrap-natalie-sandbox.pantheonsite.io/)


spot on!

what do you mean under "spammy"?


niiiice.
Ken ik niet die 'requestAnimationFrame'
Ik had een lekkere old scool setInterval / clearInterval. Moest je zeker even van grinniken?


Would something like this be better for the last sentence of your comment?
> General philosophy -- declaration, space, pre-conditions (if needed), space, then rest of function.


> May I get access to the zlib project results? (zlib is an essential part of libpng)

That would be great!
We've set up [decompression fuzzing for zlib](https://github.com/google/oss-fuzz/blob/master/projects/zlib/zlib_uncompress_fuzzer.cc)
 ourselves long ago but nobody has looked at the coverage (it's far from perfect).

We'll need a confirmation the code owners (@madler?) that they don't mind.
Then just add yourself [here](https://github.com/google/oss-fuzz/blob/master/projects/zlib/project.yaml) as you did with libpng.


Really clean & tidy, easy-to-follow steps!


> Where does the fuzzer get zlib now?

It's just -lz provided by the Docker image, which is Ubuntu 16.04
If you want to fuzz libpng with a specific version of zlib, then
a) `git pull` that version in Dockerfile
b) build it and link against libpng in build.sh
(This will also solve the problem with msan)


Were you meaning to push this change?


The academy award for best commit message goes to mr.dreidel !


I don't have access to my computer for the coming days, so if someone can fix that I wouldn't appreciate. Sorry for inconvenience.


Not sure what you mean about being defined more than once. In this new code the list is only defined once.


Uh-oh.  Yes... Are we not supposed to push after a re-render?


Initialized new repo


thanks


>The macros to_visor_device and to_visor_driver are fairly, move them up...

Did you mean to say:
>The macros to_visor_device and to_visor_driver are fairly small; move them up...


yes, but that is currently the intended effect. I am trying to understand why it should only be changed if it is zero (as you suggested), when the default is 1 (so it won't be changed for users who leave it at their default)


>Move the includes to the files that actual use them.
actual --> actually


The following doc(s) have been deployed to the [speedtest](https://speedtest-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/guides/front-end-performance](https://speedtest-static-docs.pantheonsite.io/docs/guides/front-end-performance)


Dit is best wel generd ...


"remove io.h not from channel.h" "remove io.h from channel.h"


Hi, not caused by selinux, I'm checked that! Also 5 minutes ago I have added owner and group system.system to the init.qcom.rc

on early-boot
    exec /system/bin/sh /system/etc/init.qcom.wifi.sh
    chown system system /sys/class/leds/wled:backlight/brightness
    chown system system /sys/class/leds/wled:backlight/max_brightness
    chown system system /sys/class/leds/lcd-backlight/brightness
    chown system system /sys/class/leds/lcd-backlight/max_brightness

and that didn't fix isue! Found this in logcat:

09-21 21:19:37.667  4066  4066 F DEBUG   : *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
09-21 21:19:37.667  4066  4066 F DEBUG   : LineageOS Version: '14.1-20170917-UNOFFICIAL-amami'
09-21 21:19:37.667  4066  4066 F DEBUG   : Build fingerprint: 'Sony/D5503/D5503:5.1.1/14.6.A.1.236/2031203603:user/release-keys'
09-21 21:19:37.668  4066  4066 F DEBUG   : Revision: '0'
09-21 21:19:37.668  4066  4066 F DEBUG   : ABI: 'arm'
09-21 21:19:37.668  4066  4066 F DEBUG   : pid: 849, tid: 1312, name: PhotonicModulat  >>> system_server <<<
09-21 21:19:37.668  4066  4066 F DEBUG   : signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x54077200
09-21 21:19:37.668  4066  4066 F DEBUG   :     r0 aa03b900  r1 98362210  r2 acaa5c41  r3 00000000
09-21 21:19:37.668  4066  4066 F DEBUG   :     r4 983621f8  r5 00000000  r6 b3f9f3a8  r7 00000000
09-21 21:19:37.668  4066  4066 F DEBUG   :     r8 00000000  r9 9e043500  sl 12e38388  fp 000000ff
09-21 21:19:37.668  4066  4066 F DEBUG   :     ip 00000000  sp 983621f0  lr b2b5a259  pc acaa5c40  cpsr 600e0030
09-21 21:19:37.669  4066  4066 F DEBUG   :
09-21 21:19:37.669  4066  4066 F DEBUG   : backtrace:
09-21 21:19:37.669  4066  4066 F DEBUG   :     #00 pc 00000c40  /system/lib/hw/lights.msm8974.so (__ae_r+276)


nicely done! 👍


nice Power task, more whitespaces in SumDigit, please :)


OK, apologies. I thought the default was 0. I suppose my preference then would be not to have this, but I can understand that it is helpful for the average user.

I made an edit manually. Thanks for the response.


tell lo bitch i got all money yah


Kan dit niet gewoon met CSS?


Created multidev environment [natalie-sandbox#pr-paragrap](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-paragrap).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-paragrap-natalie-sandbox.pantheonsite.io/)


lcd-backlight is symlink to mdp virtual display, but wled:backlight is symlink to qpnp thing, so thing can't be fixed without kernel modification (which is I think much more work than reverting this commit)


Hopefully we will have v1 witness pretty soon. This will force your users to fall back to witness-in-P2SH


Thanks for reviewing. Actually, this is how it should work.

If you pass `?variant=ch` it should return all language variants with `ch` regardless of the root language. However, you can also specify `?language=de&variant=ch` to limit matches to only the `ch` variant of German.

What behavior did you encounter that is invalid?


Mega fgt


and replaced the raw URDF description by the xacro one


This is very awkward, like telling Picasso how to draw or something. May your blue period last forever :D 👯♂


![screen shot 2017-09-22 at 4 24 29 am](https://user-images.githubusercontent.com/31151578/30715240-8bdf9742-9f50-11e7-8a81-783b09557af8.png)
![screen shot 2017-09-22 at 4 28 52 am](https://user-images.githubusercontent.com/31151578/30715244-8d6b130c-9f50-11e7-83cb-f3d76f12c392.png)
What  NPM is and how to use it


Closes #1


TeamCity Glyssen :: Glyssen pull requests [Build *.*.626.8ad8c86e608ac18208fe1a3da13b8e5e9b67a7c4](http://build.palaso.org/viewLog.html?buildId=97955&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1728, ignored: 18 Build time: 00:09:15


This should be resolved in 856ebb5


The following doc(s) have been deployed to the [styleguide](https://styleguide-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/style-guide](https://styleguide-static-docs.pantheonsite.io/docs/style-guide)


nice :+1:


@vak2ve, this example from https://history.state.gov/historicaldocuments/frus1918/d455 demonstrates how we've been handling formal date statements (as found in proclamations and bilateral and multilateral agreements, usually with adjacent seals) in the `frus-dates-project`:
```xml
<closer>
<seg type="margin">[<hi rend="smallcaps">seal</hi>]</seg>
<dateline>Done in the <placeName>District of Columbia</placeName> <date when="1918-09-30">this
thirtieth day of September in the year of our Lord one thousand nine hundred and eighteen, and of the
Independence of the United States of America the one hundred and forty-third</date>.</dateline><lb/>
...
</closer>
```
After I finish date extraction/normalization in Q2, I will try to run a pass through the whole corpus for consistent handling on these types of date statements, with @joewiz's agreement on best encoding practices.

We'll add encoding examples to the Wiki on this issue and others so please keep your excellent questions and flags coming!


I'll make a build for you on Monday so you can check. Till then thanks for your work.


recursion is nice, but I would've used a more specific function name, like "bunnyCounter" or something similar


What about making this the default behaviour but having a "SkipHostnameVerification" option? I have a situation where we are using certs signed by our own private CA. The hostname verification won't work since we pre-generate certs using a CN that isn't an actual DSN/Host name. I still want to validate that other servers certs are at least signed by the same private CA as the client; but JUST want to skip the hostnameVerify check.


I will not look here anymore because you didn't ansvering to my questions.


Are you able to make granites work this way?


10 points to Gryffindor for excellent function naming !


Tested.  The values for $GATEWAY['daemon_host'] and $GATEWAY['daemon_port'] are null.


I didn't test it, that was just my impression from the readme description.


only this unnecessary extra blank line, otherwise nice :)


Yep


This (and other ~/ permissions) won't work if xapian-bridge runs as root.


This should be resolved in 88cdd67. It's in `#check_access`.


nice little counter :)


Try to format your code better so this is easier to read


> Ah, very nice design. I never quite got on the same page as you when we were discussing this verbally but now it is clear.

Yes, sometimes having code, even if it is incomplete, is helpful in providing clarity.

> I like the fact that the API level is a simple integer instead of attempting to come up with a meaningful name for each level. The man page can take care of explaining what each integer represents. I'm curious if you foresee the new log filter flag, log action, and process kill action all being represented by level '3' or would, for example, the logging flag and action be level '3' and the process kill action be level '4'? My question is assuming that the new flag and actions would all be supported in the upcoming 2.4 libseccomp release. If they were all initially present in the same libseccomp release, I'd think that representing them all with level '3' would be fine.

Yes, I agree, my thought was that all of the stuff you are proposing would be combined in level 3.


LocalNodeNetworkTry5 (XuanLuoLBL) - i386-Windows-7-VisualStudio-14: OK (2774 of 2774 tests passed, 15 test warnings)

Messages:

 * 14 tests had: AUD diffs.
 * 13 tests had: BND diffs.
 * 1 test had: ESO small diffs.

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


if you use 4 spaces-indentation, maybe here could be also applied...


As `curl_global_sslset` is in a `#ifdef USE_SSL` block, calling it when building without SSL support (as e.g. MinGW does by default) results in a linker error, visible when trying to build the sslbackend example. Probably there should be an implementation returning an error code instead in the `!defined(USE_SSL)` case?


LOL! 🥇


nice :)


what he said! your code is very tidy and i like the solutions 🐤


Try "postgresql-config list" to check, whether you set the PostgreSQL slot.
If it's not set, set it with "postgresql-config set 9.4", where 9.4 is your version. It should be something from 9.0 to 9.4 to work with PostGIS.


nice :+1:


The Lich King probably beats them all by a large margin. I think I've seen him like a dozen times while questing through Northrend in WoTLK... though I don't remember if he was there as an NPC every time.

I remember seeing him during at least 7 quests, and then there was his appearance in Trial of the Crusader, and later Halls of Reflection and of course Icecrown Citadel.

Imagine the outcry if they moved that NPC every single time! :wink:


easy to understand with variables (lastDigit, firstSegment). good one


this does not adds "*" - s, instead replaces x-s. maybe copy-paste issue :)


Then maybe this block should read as follows:

> The language variant code (e.g. `ch`; returns all variants from the `ch` territory)
The language variant written in English (e.g. `Swiss German`; returns the `de-CH` variant only)
The language variant written in the native language (e.g. `Schweizerdeutsch`; returns the `de-CH` variant only)


@NejcZdovc FYI with Redux this changes to `droppedOn.selectedRef.state.isFolder`, with props it was always undefined because it only checks ownProps in that case.  I'll fix in a commit I'm working on with another problem.


According to the Python style guide [PEP 8](https://www.python.org/dev/peps/pep-0008/#programming-recommendations)

> Don't compare boolean values to True or False using ==.
> **Yes**: if greeting:
> **No**: if greeting == True:
> **Worse**: if greeting is True:

I recommend changing to
```python
if not owner:
```


Created multidev environment [natalie-sandbox#ci-22](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#ci-22).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-22-natalie-sandbox.pantheonsite.io/)


I have modified the code for multiple columns of buttons, fixed the tkinter call for Python 2.7, etc.

[LWTouch.zip](https://github.com/anthonyeden/Livewire-Simple-Delegation-Switcher/files/1322547/LWTouch.zip)

There is an issue when subscribing to multiple outputs on a single IP.  Control is OK but Status is crosslinked.


@disti95 looks like your getting much faster :+1:


I like this solution! :)


Nice to see a forward-going else/else-if/else solution.
Clean, and understandable.


Spot on!


additionally, it does not use recursion. If you try and remove the inner string function, it does the same


Ok mate, no prob.


Skall funka nu, fick ända lite och lägga tillbaka lite grejjer. Kolla genom koden och finslipa om du hittar onödiga grejjer eller dyligt.


I'd assume that's the name of machine


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/f1a75b66-203d-4d78-9391-c6c86d4ded44 "JobMatcher"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/3fea10cb-b4d9-4628-9899-2891d2fdef69 "JobMatcher") deployed [JobMatcher](https://github.com/Mbouzon/JobMatcher/releases/tag/deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-test-20170921-195935 "deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-test-20170921-195935") to [test](https://console.bluemix.net/apps/e33766af-34ba-4ad0-aa3a-fd48d2045d92?env_id=ibm:yp:us-south "ibm:yp:us-south:marcio.bouzon@br.ibm.com:test"), including this commit


Pretty commit messages and names 🦊


How might you use the `ul` & `li` elements to structure this content?


We need all of our visible content to remain inside the `body` elements...easy fix!


this can just be `a` instead of `a href`


Great to see you playing around with different styles!


Nice work with the `h1` & `h2` element selections!


So, only `li` elements can be a child of a `ul`...something to keep in mind for next time!


Shall we check for timeout here?


How could you adjust the indentation here to help make it a bit easier to read?  any thoughts?


Recomiendo clonar nuevamente el proyecto en htdocs de xampp y así en dirección de servidor externa asignar la url: "_**http://localhost/Portal_del_Vecino/Portal_del_vecino/public_html**_"


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/b3392808-cedb-40d7-9da9-bc922fb8f90a "UI-yp-SimpleTC0920T215941"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/b53cc6f5-dc82-468e-858f-0b85b8f4a835 "UI-yp-SimpleTC0920T215941") deployed [UI-yp-SimpleTC0920T215941](https://github.com/idsb3t1/UI-yp-SimpleTC0920T215941/releases/tag/deploy-ibm-yp-us-south-idsb3t4-us.ibm.com-dev-20170921-020253 "deploy-ibm-yp-us-south-idsb3t4-us.ibm.com-dev-20170921-020253") to [dev](https://console.bluemix.net/apps/4181c554-27ac-453d-9af1-de10cff80da4?env_id=ibm:yp:us-south "ibm:yp:us-south:idsb3t4@us.ibm.com:dev"), including this commit


portYield still has potential problems, cannot just disable IRQ in entering portYield and enable upon exit.  and use 'ret' in portYield will also corrupt RA register.  It will cause problem eventually...


WARNING: Utility classes should not have a public or default constructor.


ERROR: [GradleCompatible] This support library should not use a different version (25) than the `compileSdkVersion` (26)
 There are some combinations of libraries, or tools and libraries, that are incompatible, or can lead to bugs. One such incompatibility is compiling with a version of the Android support libraries that is not the latest version (or in particular, a version lower than your `targetSdkVersion`.)


jjj


Mostly because ℕ is defined in basic-hott and doesn't work with the one in stdlib.


Right---I had this problem earlier as well if you all recall. I had to copy some definitions from the standard library since declaring our definition of ℕ as BUILTIN conflicts with the standard library's declaration of their ℕ as BUILTIN.


Okay but is this going to be added to uBO filter list. If no why?


please review


FixCrashWithZeroHeatLoadOnAirLoopComponentLoadSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1552 of 1552 tests passed, 0 test warnings)

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


Nice. 😄


Are you sure this fixes it? I just regenerated the documentation and it looks the same. Do you need a space there? There is a space on the corresponding lines on other spots I checked.

After you fix this can you regenerate the pkgdown page? there are instructions on how to do it in the readme of that repository.


ha ha


alo


hi. test.


@simonegu Since the `mavlink_helpers.h` does not include `protocol.h` by default, it leads to compile errors of the form:
```
mavlink_helpers.h:781:4: error: ‘MAVLINK_START_SIGN_STREAM’ was not declared in this scope
```
Is the user supposed to  provide the implementations for these new functions or should `protocol.h` be included by default?


I had forgotten.  What a massive pain.


test only.


@karmux You should add two renamed images in `BUILD_PLUGIN()`, in `plugins/VstEffect/CMakeLists.txt`. I think changing `logo.png` to `*.png` will be fine.


khong hieu


This change is not backwards compatible with Typescript projects that are already using statsd.
- require('hot-shots') does not resolve to StatsD
- the client (StatsD instance) is missing the `socket` method


Hi! I want to try your ArbitrageBot but im  confused where to put my api key? and how to configure the settings.thanks...


tét


change


hehe da


https://github.com/uBlockOrigin/uAssets/issues/471#issuecomment-310869935, your quote

> I would also like to clarify that I am not asking for going after all blank ads/elements but only when the content itself is affected

https://github.com/uBlockOrigin/uAssets/issues/471#issuecomment-310870148, my answer

> I agree with that, I just don't see it in this issue.
I would also argue that it is more than an annoyance if there are so many blank ads/elements on
a given site that it looks like whoever designed that site was either drunk or incompetent. Again not the case in this issue in my opinion.

I do not see how the content is affected in this case, now this is just my opinion and maybe  gorhill has a different one.


明白，多谢


Test


g


Realized later that I could've saved you some time by submitting my repro as a PR with a failing test instead. Will do so next time, although I hope there's no such occasion ;)


新购入中文版书籍，
**P56 二阶导数组成的Hessian矩阵用来和梯度下降方法比较，如何区分和各有什么好处不太清晰。**


as per http://lists.infradead.org/pipermail/lede-dev/2017-September/008818.html using more recent versions than 2017-02-11 will break vti/gre/6rd, so I reduced the previous mentioned commit to only include the compatible updates of netifd.
Probably  this is acceptable for LEDE-17.01


False alarm, apparently.  Either I saw the NTP issues on the OpenCSW buildbot (but those only showed up on 32-bit builders) or I saw bus errors on some tests I ran and thought they were SPARC/Solaris rather than PA-RISC/HP-UX (and they're not in the NTP tests, although the messages were near the NTP test messages).


Created multidev environment [sacdugd8#pr-ctools](https://dashboard.pantheon.io/sites/2c99b972-66bf-414f-9da5-5f3beb462bc8#pr-ctools).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-ctools-sacdugd8.pantheonsite.io/)


Procedure-Oriented now


BankHomeActivity


can you put out a new release for this as well @rm-hull?


Beyond1Airloop-Part2 (mjwitte) - i386-Windows-7-VisualStudio-14: OK (2767 of 2767 tests passed, 0 test warnings)

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


i'd like to extend `direccion` to handle things like stripping zip code or `Detroit MI` etc


I’m not quite sure I understand why this is needed. There is already the Subtexture. subtexturesFromAtlas method that does exactly this...


what?


can you add a new release for this as well @rm-hull?


@rm-hull is this commit intended?


that's a terrible idea! why would you do that? lol just kidding. impressed that you check code commits so carefully (!)

> On Sep 20, 2017, at 22:37, Jimmy McBroom <notifications@github.com> wrote:
>
> i'd like to extend direccion to handle things like stripping zip code or Detroit MI etc
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


This could be cleaned up to this and still do the same thing.
```rb
access all: [:index], user: [:show, :index], editor: :all
```


OK. Yes, I know about the name but didn't want to use it because this is not a getter per se. I can change the method to enable/disable. Initially, we were talking about setCache() with passing a boolean to it, so that was that

In Java it is obligatory to do getters and setters regardless whether they are private or not. Just objected-oriented stuff, every class comes prepackaged with them on IDEs


@grahamlyus Please, take a look in this changes and let me know your thoughts


## What is this for

Please explain this line better


Assignment 2


Created multidev environment [sacdugd8#ci-9](https://dashboard.pantheon.io/sites/2c99b972-66bf-414f-9da5-5f3beb462bc8#ci-9).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-9-sacdugd8.pantheonsite.io/)


# Commit Comment

Explain this commit better


While libtcti-tabrmd.so is not available as a rpm/deb package on most Linux systems.
We should do some tricks to make it possible to work with libtpm2-pk11.so on a system with out libtcti-tabrmd.so.

Here is one solution from strongswan project, they simply wrapped "tss2_tcti_tabrmd_init(tcti_ctx, &size)" with "#ifdef TSS2_TCTI_TABRMD":

```
/**
 * Initialize TSS2 TCTI TABRMD context
 */
static bool initialize_tcti_tabrmd_context(private_tpm_tss_tss2_t *this)
{
#ifdef TSS2_TCTI_TABRMD
	size_t   tcti_context_size;
	uint32_t rval;

	/* determine size of tcti context */
	rval = tss2_tcti_tabrmd_init(NULL, &tcti_context_size);
	if (rval != TSS2_RC_SUCCESS)
	{
		DBG1(DBG_PTS, "%s could not get tcti_context size: 0x%06x",
					   LABEL, rval);
		return FALSE;
	}

	/* allocate memory for tcti context */
	this->tcti_context = (TSS2_TCTI_CONTEXT*)malloc(tcti_context_size);

	/* initialize tcti context */
	rval = tss2_tcti_tabrmd_init(this->tcti_context, &tcti_context_size);
	if (rval != TSS2_RC_SUCCESS)
	{
		DBG1(DBG_PTS, "%s could not get tcti_context: 0x%06x "
					  "via tabrmd interface", LABEL, rval);
		return FALSE;
	}
	return TRUE;
#else /* TSS2_TCTI_TABRMD */
	return FALSE;
#endif /* TSS2_TCTI_TABRMD */
}
```


Not Ready for merge or review yet. Thanks!


restore support for gnome < 3.24 - fixes issue #2


Maybe a better check would be to see if we are at level 0. And if so only use symbol.


Intuition behind this?


I suspect this would be more useful if the user could override the query.


No, you're doing it wrong. You don't need to add a check here, it will just make it run 2 times slower.


# **_[Home](url)_**        **_[Furniture](url)_**
![0f51f347410dbdddbb7ea890bfec4f87--modern-furniture-design-contemporary-furniture](https://user-images.githubusercontent.com/32151404/30676767-69bca322-9ea5-11e7-83af-7906153d217e.jpg)


If data.watch is false, it will call pause(), but then pause will return immediately without removing observers... is that right?


Final Report


Update to 1.1


No need to push those registers, they are not used in the routine. You are just wasting more cycles.


you prepare for everything... even if a non-negative int is smaller than 0... :D


it works, I checked


thanks :)


when the exercise does not ask for it directly, I usually do not prompt the user for input, that is easier :)


thank you :)


That's fine -- sometimes it's easier that way when I have to backport patch (I'm sure git has ways to rebase things but I am not skillful enough to do those in a way I trust to be safe).


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/f1a75b66-203d-4d78-9391-c6c86d4ded44 "JobMatcher"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/3fea10cb-b4d9-4628-9899-2891d2fdef69 "JobMatcher") deployed [JobMatcher](https://github.com/Mbouzon/JobMatcher/releases/tag/deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-prod-20170921-200138 "deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-prod-20170921-200138") to [prod](https://console.bluemix.net/apps/5fd9d050-c40d-44c3-849f-54346daf7f84?env_id=ibm:yp:us-south "ibm:yp:us-south:marcio.bouzon@br.ibm.com:prod"), including this commit


@WaxCylinderRevival That tagging was my instinct too--though those instances can be a little unwieldy, and I thought I should hold off on making any changes that weren't extremely clear-cut in the context of the document until I'm a little more familiar with the dates project. So it's okay with if I go back and correct the few dozen documents in 1910 and 1911 that fit the above pattern?


nice solution. like it a lot!


nice, thanks!


Yeah sounds fine to me to disable issues here, this repo should barely change and as you say, if there were problems it makes more sense having them in the processor


@bparker06 oops! if I did so it was not intentional. I had a conflicted docs PR a week or two ago, with the conflict located in the docs index. I didn't notice your work at that time but it must have been when I removed it, if it was me. I promise not to do so again!


Sorry, you're correct. I was looking for the wrong thing, so hadn't noticed that it was _all_ now in init.

The issue about OpenWebIF still remains that (it will still report it as being in hours in OpenVix _etc._).


Well normally one does a PR first instead of pushing directly.

That said, I think one will need to build this locally. Though it looks like the CIs passed, the `skip` wasn't removed in the feedstock recipe. IOW the CIs ran, but didn't produce anything. My understanding is the build takes longer than the CI time limits allow. Hence why everything was disabled. So removing the `skip` will likely just cause all of the builds to fail. Can always be explored again though.


drawBot LOL


TeamCity libpalaso :: palaso-Linux64-lfmerge Continuous [Build 2.6.505.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97959&buildTypeId=PalasoLinux64lfmergeContinuous) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:07


TeamCity libpalaso :: palaso-linux64-libpalaso-3.1 Continuous [Build 3.1.474.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97960&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:22


TeamCity Glyssen :: Glyssen pull requests [Build *.*.627.c8fbaaa2cb555cb4d2e641ffba5bb42557ef5d2c](http://build.palaso.org/viewLog.html?buildId=97956&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1728, ignored: 18 Build time: 00:09:28


Hello,

You declare the option 'StagerRetryCount' in ' lib/msf/core/handler/reverse_tcp.rb' as an 'OptFloat' rather than an int. However, it is an int in all the other cases. Is this intentional?


TeamCity libpalaso :: palaso-linux64-libpalaso-2.6 Continuous [Build 2.6.1520.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97963&buildTypeId=PalasoLinux64v26Cont) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:07


Como se descargar Androrat


thanks:) i think if you use the WIDTH & HEIGHT, you won't get the center with these drawings, because the header / title of the drawing window is also included in the numbers :D


@cyriltata please check whether this is ok. couldn't find any other use of this method. since we seek to rename and don't want to rename the results table, it doesn't seem to make sense that we check for this. could also remove the second argument completely?


yep sorry, was relying on protected branches too much, so the push went through acidentally :/


Pardon me, I meant 'StagerRetryWait'.


I don't think this is the appropriate way to do this.  visor_check_channel() is more of an assertion-type function, and it treats it as an ERROR if the GUID in the channel doesn't match the GUID passed as an argument, and it logs a dev_err() message.  That means we are going to get a lot of dev_err messages logged because of this code, when in fact there is NO error; this is just normal searching for the necessary driver that will claim a channel.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.404.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97962&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


it's ok, but there are too many unnecessary empty lines


TeamCity libpalaso :: palaso-win-master Continuous [Build 4.0.96.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97966&buildTypeId=Libpalaso_PalasoWinmasterContinuous) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:07


Just a heads up, probably not a good idea to remove the license text since that violates the license.
https://opensource.org/licenses/EFL-2.0


nice, clear codes... congrat!!! really mean it! :) :+1:


Hmm not sure what happened. It looks okay on Heroku.


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/f1a75b66-203d-4d78-9391-c6c86d4ded44 "JobMatcher"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/3fea10cb-b4d9-4628-9899-2891d2fdef69 "JobMatcher") deployed [JobMatcher](https://github.com/Mbouzon/JobMatcher/releases/tag/deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-test-20170921-200805 "deploy-ibm-yp-us-south-marcio.bouzon-br.ibm.com-test-20170921-200805") to [test](https://console.bluemix.net/apps/b6ad378e-bde4-4d08-9791-843aadabfe2e?env_id=ibm:yp:us-south "ibm:yp:us-south:marcio.bouzon@br.ibm.com:test"), including this commit


Yes, your instincts on both accounts were spot on.

If you're comfortable doing so, feel free to adjust to follow the pattern. If not, feel free to continue leaving messages in the commits for me and for Joe.  Just let me know which way you'd like to proceed.


The following doc(s) have been deployed to the [speedtest](https://speedtest-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/guides/front-end-performance](https://speedtest-static-docs.pantheonsite.io/docs/guides/front-end-performance)
- [source/docs/assets/images/guides/front-end-performance/chrome-cookies.jpg](https://github.com/pantheon-systems/documentation/commit/9c02dc26e26e770fb26cffb4fb75a4620cd84658/source/docs/assets/images/guides/front-end-performance/chrome-cookies.jpg)


@vak2ve, if you decide to make the changes, one note: please add a line break at the end of the new `dateline`, once you move it into the closer.  Otherwise, the dateline and following signatures might not render properly.


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


nice, I especially like that you made effort to asking input


@LeandroRoberto *MUITO* obrigado por atualizar esse arquivo! :clap: Tinha me esquecido completamente! :cold_sweat:


Ah, I see. I haven't really found the solution for using width and height, but just wanted to use them as given.
and I meant height of course, instead of length.


"staging: unisys: viosrbus" --> "staging: unisys: visorbus"

> Removing the two wrapper functions visorchannel_create and visorchannel_create_guts for visorchannel_create_guts and using it directly instead.

change to:
> Remove the wrapper functions dealing with visorchannel_create(), and instead just always use a new version of visorchannel_create() with an additional parameter.


nice


The following doc(s) have been deployed to the [2885google](https://2885google-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/google](https://2885google-static-docs.pantheonsite.io/docs/google)
- [source/docs/assets/images/google-a-record.png](https://github.com/pantheon-systems/documentation/commit/10c183e5219e1616170eabf678ac7c34731c8fa1/source/docs/assets/images/google-a-record.png)
- [source/docs/assets/images/google-aaaa.png](https://github.com/pantheon-systems/documentation/commit/10c183e5219e1616170eabf678ac7c34731c8fa1/source/docs/assets/images/google-aaaa.png)
- [source/docs/assets/images/google-cname.png](https://github.com/pantheon-systems/documentation/commit/10c183e5219e1616170eabf678ac7c34731c8fa1/source/docs/assets/images/google-cname.png)
- [source/docs/assets/images/google-config-dns.png](https://github.com/pantheon-systems/documentation/commit/10c183e5219e1616170eabf678ac7c34731c8fa1/source/docs/assets/images/google-config-dns.png)


I'll go back, it's an easy fix! I'll doublecheck line breaks too--did I miss any in the Q3 volumes? If so apologies for the oversight and I'd be happy to go back and fix those as well.


Thanks @celm1990
Now it's fixed with 0b8ae7220b333a6ac1ef46498b7e3c477b86123e


I can only state my opinion here: the fact that checkpatch doesn't like these parentheses is totally stupid.


The following doc(s) have been deployed to the [styleguide](https://styleguide-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/style-guide](https://styleguide-static-docs.pantheonsite.io/docs/style-guide)


Couldn't the "pkgutil --check-signature" come back with a "Status: no signature" or "Status: signed by a certificate that has since expired" which would still pass the [ $? -ne 0 ] check?

Maybe use this instead:

pkgExpectedSignature=$(/usr/sbin/pkgutil --check-signature "$tempDir/NoMAD.pkg" | /usr/bin/awk '/Status: signed by a certificate trusted by Mac OS X/ { print $0 }' )

This would also catch if the certificate has expired or if the package was not signed at all.


stupid stupid stupid (sorry...)


Why are you still working on the CSSU website


Also why I am still an owner of the repo?


something is turning into the right direction... 👍


(not a problem at all)!


I think we should revert this commit since the namespacing/context ideas come later in the context guide. Using them here without explanation is confusing. Add an explanation would be premature since we tackle this later in the context guide. @bryanstearns can you send a PR to revert? Thanks!


#2 It show the list of items created on the API


`Also, use Numpy please and attach all the code you have used.` is clear to me, `Please use Numpy to attach all the code you’ve used.` is not.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.628.290181dbd1d6d4e173a4b96e564abbc6ab75b6db](http://build.palaso.org/viewLog.html?buildId=97968&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1728, ignored: 18 Build time: 00:09:13


Terrific!

Q3 is next on my list.  If you'd like to check first, that would be great.  But I know you have to balance new releases so I defer to your schedule priorities.


paranthesis --> parenthesis (last several patches)


sorry, this shall just count, not add the numbers... my bad. then it is perfect :+1:


I have a question.
You switch knp-snappy from ~0.1 to ~1.0, which is major I think.
And wrapper library have only minor version change.
Is normal practice? Is there some reason for that?

reagards,


#1 and #2


www.youtube.com 使用了不受支持的协议。  -.-我们的墙似乎换了套路


woo-hoo! it sure did that recursion thing! 🥇


Still, this feels like layering violation, vstr's aren't supposed to know about exceptions.


[src/prizing_from_observations.R:4:1:](https://github.com/ThorbenJensen/barter-market-pricing/blob/3a461f6e9ae74651b2612e4a6b6d714672ef2376/src/prizing_from_observations.R#L4) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[src/prizing_from_observations.R:5:7:](https://github.com/ThorbenJensen/barter-market-pricing/blob/3a461f6e9ae74651b2612e4a6b6d714672ef2376/src/prizing_from_observations.R#L5) *style:* **Put spaces around all infix operators.**
```r
  fit <-lm(c ~ a + b + 0, data = transactions)
      ^~~
```
[src/prizing_from_observations.R:6:1:](https://github.com/ThorbenJensen/barter-market-pricing/blob/3a461f6e9ae74651b2612e4a6b6d714672ef2376/src/prizing_from_observations.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```


Was looking for a decentralised way of letting participants know why they can't withdraw their money. Any better suggestions?

I figured that if things are so bad we have to pause the contract then a few extra pennies for gas would be no biggie -- plus they could use an empty string to keep the impact minimal. But perhaps I'm missing something or severely under-estimating the cost of a (non-persisted) string param?


@atscomposer Where did you get that result? My test showed that the command reported a failure when there was no signature present:

```
(aws) uss-enterprise:jhapi brysontyrrell$ pkgutil --check-signature /tmp/NoMADun.pkg
Package "NoMADun.pkg":
   Status: no signature
(aws) uss-enterprise:jhapi brysontyrrell$ echo $?
1
```


lua was ordered before other receivers on purpose. IIRC so you don't see game crap above luamenu.
Please don't change this.


We could do it better in the sys_xxx.cpp for x86 platforms to use some inline GCC to manually set the float flags.  This should be OK with clang?  I can look into it sometime.


Question 5


Ava


elegans, de kellett hozza egy felora hogy felfogjam, eleg volt mara :D


I'm 98% sure there was a reason why I did this order.
What was broken?


Understood, thanks. Did it for code clarity and maintainability (e.g. someone adds to the enum), on the assumption that gas cost would be negligible or optimised away completely.


tidy and compact solution, I like it 👍


I'm not sure if this is 'okay' or not, the cargo version is 0.1.0, but the only docs up are 0.0.1


If we use `--filesystem=/home:ro` would work, but yeah, not perfect.


It was in theory not an actual test. I did not have a package without a signature or and expired very available. If that throws an error 1 then great. Sorry for the confusion and not proving out my theory first.

________________________________
From: Bryson Tyrrell <notifications@github.com>
Sent: Thursday, September 21, 2017 4:26:21 PM
To: jamfit/NoMAD-Installer
Cc: atscomposer; Mention
Subject: Re: [jamfit/NoMAD-Installer] Switched to package signature verification (a246ed0)

@atscomposer<https://github.com/atscomposer> Where did you get that result? My test showed that the command reported a failure when there was no signature present:

(aws) uss-enterprise:jhapi brysontyrrell$ pkgutil --check-signature /tmp/NoMADun.pkg
Package "NoMADun.pkg":
   Status: no signature
(aws) uss-enterprise:jhapi brysontyrrell$ echo $?
1

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/jamfit/NoMAD-Installer/commit/a246ed0fe91eeab11e42ac9def22cc6ed8a74846#commitcomment-24471281>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ATh9UH1sKIEx0_9mtP1ezfdjTkIs45PYks5sksZtgaJpZM4Pf3Aw>.


[src/prizing_from_observations.R:4:1:](https://github.com/ThorbenJensen/barter-market-pricing/blob/f0ca0501e5b270b94ada52ca692d3bee3e05ef8d/src/prizing_from_observations.R#L4) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[src/prizing_from_observations.R:5:7:](https://github.com/ThorbenJensen/barter-market-pricing/blob/f0ca0501e5b270b94ada52ca692d3bee3e05ef8d/src/prizing_from_observations.R#L5) *style:* **Put spaces around all infix operators.**
```r
  fit <-lm(c ~ a + b + 0, data = transactions)
      ^~~
```
[src/prizing_from_observations.R:6:1:](https://github.com/ThorbenJensen/barter-market-pricing/blob/f0ca0501e5b270b94ada52ca692d3bee3e05ef8d/src/prizing_from_observations.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```


Last known good


![image](https://user-images.githubusercontent.com/32178575/30717397-c6b61014-9eea-11e7-80eb-93590afb6db8.png)

This doesn't seem to add enough value to be a contributor


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


@cjappl Sorry, I had to remove bidict. It was just adding too much to the runtime.

I don't understand why bidict takes so long to load. (About 10 times as long as Mido. Strange for what seems like a simple module.)


Why this line?


I'm trying to vet that it doesn't pass on an expired signature.


Yes it's possible to log in with this core. You can use a client from wowcircle, wodempire or even firestorm itself which is the origin of this core. Possibly it can be required that you need to tweak some client build number in the auth database.

Be aware though that the login procedure is different from stock trinitycore, the clues for this login are available in the  commits :)

Good luck.


Since interfaces cant have private members


Good stuff ;)


Hmmm. Only when paused gives the owner the chance to pause the contract indefinitely then claim all funds without first unpausing to allow withdrawals. Seems open to abuse - at least without more checks and balances on the pause function.

Idea of a 1 year cutoff for claiming winnings seems fair to me (subject to my TODO re: pausing). If I understand correctly, it allows for what is effectively garbage collection of the blockchain, and also a way to access accidentally deposited funds (from other suicides, or from before contract creation). What do you see as the downside? Just the 1 year time limit to claim funds, or is there more?


@cjappl  I looked around and found a fast and (fairly) clean way to include metadata in setup.py. This puts the metadata in `mido/__about__.py` and gets around the chicken and egg problem with dependencies by reading the file with `exec()`.


Mark, do you mind if I get access to the zlib oss-fuzz project?

Glenn R-P

On Thu, Sep 21, 2017 at 3:34 PM, Kostya Serebryany <notifications@github.com
> wrote:

> May I get access to the zlib project results? (zlib is an essential part
> of libpng)
>
> That would be great!
> We've set up decompression fuzzing for zlib
> <https://github.com/google/oss-fuzz/blob/master/projects/zlib/zlib_uncompress_fuzzer.cc>
> ourselves long ago but nobody has looked at the coverage (it's far from
> perfect).
>
> We'll need a confirmation the code owners (@madler
> <https://github.com/madler>?) that they don't mind.
> Then just add yourself here
> <https://github.com/google/oss-fuzz/blob/master/projects/zlib/project.yaml>
> as you did with libpng.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/glennrp/libpng/commit/04e16d28176b10abe4cb6e215802c2c39e45dea4#commitcomment-24469925>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABe25nBX7xGYvPSl_fyxmWUDb1Z6OKMEks5skrpMgaJpZM4PZtgr>
> .
>


Nearly - you missed updating the view as well


I'm afraid not. What is the usecase? Why can't you handle this on the model-side?


TeamCity libpalaso :: palaso-linux64-master Continuous [Build 4.0.374.8e3c5c621fcfeb166f0ce35e10568dfae003d9fb](http://build.palaso.org/viewLog.html?buildId=97964&buildTypeId=Libpalaso_PalasoLinux64masterContinuous) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 00:35:05


Now* checking* omg lol


Created multidev environment [ciccweb#pr-homepage](https://dashboard.pantheon.io/sites/cbc91b53-e053-4c27-8ad4-7502151a02ed#pr-homepage).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-homepage-ciccweb.pantheonsite.io/)


Ovo je nesto izbagovalo za dva HEADA , kod mene u sumblimu sve lepo , sima mi svedok!


thx


TeamCity libpalaso :: palaso-win-master Continuous [Build 4.0.97.54f94f49f8bd250712dedd1f5619e5cd3ec99dc8](http://build.palaso.org/viewLog.html?buildId=97971&buildTypeId=Libpalaso_PalasoWinmasterContinuous) outcome was **FAILURE**
Summary: Failed to start build Build time: 00:00:04


These names are not descriptive.


Hey @kaschioudi , it looks like I'm not supposed to do this...

It throws `vagrant@xmlps:/var/www/html$ PHP Fatal error:  Declaration of MergeXMLOutputs\Model\Converter\Merge::convert() must be compatible with Manager\Model\Converter\ConverterInterface::convert() in /var/www/html/module/MergeXMLOutputs/src/MergeXMLOutputs/Model/Converter/Merge.php on line 16`, I guess the convert function has requirements globally. I need to be able to do something like https://github.com/pkp/ots/commit/308916be7e94a3dfa79e6c65ade5611e47ad7aed#diff-ac7a4cad8df579477f769678be3f1511R163 in order to make the Merge module handle PDFs, though...


Can we start a thread about OpenWebIF in the dev room.


Okay, that fixes the problem.  Thanks.


LocalNodeNetworkTry5 (XuanLuoLBL) - Win64-Windows-7-VisualStudio-14: OK (2773 of 2774 tests passed, 15 test warnings)

Messages:

 * 14 tests had: AUD diffs.
 * 13 tests had: BND diffs.
 * 1 test had: ESO small diffs.

Failures:

EnergyPlusFixture Test Summary
 * Passed: 541
 * Failed: 1

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


what the fuck?


Ken Takata wrote:

> Please see this URL:
> https://github.com/vim/vim/commit/37c64c78fd87e086b5a945ad7032787c274e2dcb#diff-e66a539c610071d044099483d9dc2c40R70
>
> `shiftwidth()` came back to `&sw`.

This is in the cmake indent file.  Dimitri, can you fix that please?

--
        Very funny, Scotty.  Now beam down my clothes.

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


@martinezjavier Do you know where can I find md2man-roff for fedora?
I tried to build it from source with gem install md2man, but I'm getting some ruby errors:

[md2man]$ gem install md2man
Building native extensions.  This could take a while...
ERROR:  Error installing md2man:
	ERROR: Failed to build gem native extension.

    current directory: /home/tstruk/.gem/ruby/gems/redcarpet-3.4.0/ext/redcarpet
/usr/bin/ruby -r ./siteconf20170920-21018-15ncs3v.rb extconf.rb
creating Makefile

current directory: /home/tstruk/.gem/ruby/gems/redcarpet-3.4.0/ext/redcarpet
make "DESTDIR=" clean
rm -f
rm -f redcarpet.so  *.o  *.bak mkmf.log .*.time

current directory: /home/tstruk/.gem/ruby/gems/redcarpet-3.4.0/ext/redcarpet
make "DESTDIR="
gcc -I. -I/usr/include -I/usr/include/ruby/backward -I/usr/include -I.   -fPIC -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -mtune=generic -fPIC -fvisibility=hidden -m64 -o houdini_href_e.o -c houdini_href_e.c
gcc: error: /usr/lib/rpm/redhat/redhat-hardened-cc1: No such file or directory
Makefile:239: recipe for target 'houdini_href_e.o' failed
make: *** [houdini_href_e.o] Error 1

make failed, exit code 2

Gem files will remain installed in /home/tstruk/.gem/ruby/gems/redcarpet-3.4.0 for inspection.
Results logged to /home/tstruk/.gem/ruby/extensions/x86_64-linux/2.3.0/redcarpet-3.4.0/gem_make.out


`pkgutil` isn't returning an error on an expired signature for a package after I altered the time on my test VM to 2023, but it will on a signed app - so that's odd.

However, the purpose behind this isn't about the package's validity: the signature check is a consistent means of verifying that the package is intact as an incomplete download would invalidate it and fail the check. On that basis, I'm good with this change.


I would encourage you to name this `email-link` just like you did `contact-container` above


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


`<fieldset>` would be a good alternative to the `h5` here: https://www.w3schools.com/TAgs/tryit.asp?filename=tryhtml_fieldset


sweet! 👍


Great to see you playing with grid for your images!


Thanks for the explanation 👍


Just being silly 😄


I like the effect!


Great use of flex!


Nevermind, https://github.com/pkp/ots/commit/4b651951d4adc45fa67a0dc74db86d7b6a038b92 should do it...


Nice work with this selector!


This can go into its own file.


Great use of font awesome!


I would consider using an `h1` tag here...as it is the first heading in this "section"


Replace `;` with a space


Style wise I'd prefer ``z = sorted(zip(inds, key_list))`` over this two line approach.


taken care of


This does double-duty with the key in the mapping. Not needed.


We should throw an exception here instead of just printing a message.

```python
else:
    raise ValueError('Invalid value for computation_phase')
```


Can probably be replaced with `.owner != 0`.


TeamCity libpalaso :: palaso-win-master Continuous [Build 4.0.98.54f94f49f8bd250712dedd1f5619e5cd3ec99dc8](http://build.palaso.org/viewLog.html?buildId=97972&buildTypeId=Libpalaso_PalasoWinmasterContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3829, ignored: 191 Build time: 00:04:51


`require(!productRecords[productID].isAvailable);`


@luislobo omg thanks, been wanting to do that for a while and kept forgetting


`DifficultRotatations` -> `DifficultRotations` ?


😄 This looks great!  Awesome job with this!


`uint tokenName` ?


The parentheses in this expression aren't necessary. It can just be:

```python
return 1 - SSE/SST
```


You probably shouldn't use `ifconfig | grep eth1`. Interface can change.
Try this instead `$(ifconfig | grep -oP 'HWaddr \K.*' | sed 's/://g')`


:sob: Sorry I forgot how widespread the lib was


This could have been done by calling `mime.ParseMediaType(resp.Header.Get("Content-Type"))`, but the above code is much faster.


`Product storage product =` is also possible. You would just have to do `product.isAvailable = false;`.


I have no problem with that. Just taking it that one step further to validate the certificate as who knows if Apple will require that a valid certificate for all software to be installed in the future. Just food for thought I guess.

________________________________
From: Bryson Tyrrell <notifications@github.com>
Sent: Thursday, September 21, 2017 4:48:45 PM
To: jamfit/NoMAD-Installer
Cc: atscomposer; Mention
Subject: Re: [jamfit/NoMAD-Installer] Switched to package signature verification (a246ed0)

pkgutil isn't returning an error on an expired signature for a package after I altered the time on my test VM to 2023, but it will on a signed app - so that's odd.

However, the purpose behind this isn't about the package's validity: the signature check is a consistent means of verifying that the package is intact as an incomplete download would invalidate it and fail the check. On that basis, I'm good with this change.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/jamfit/NoMAD-Installer/commit/a246ed0fe91eeab11e42ac9def22cc6ed8a74846#commitcomment-24471751>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ATh9UCofjvnpjdyJCfXRGIrddkubUAaxks5sksutgaJpZM4Pf3Aw>.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-14: OK (2160 of 2160 tests passed, 0 test warnings)

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


I would move this line above `require(product.isAvailable == true);` to reduce the work miners have to do when faced with spamming transactions.
Also `...|| administrators[msg.sender] ||...`.


`restockProduct` does not have the "array ordering" undertones.


different solution than mine, but I like it this way


More like `getProduct`?


Don't prevent people from just collecting info.


Update Composer dependencies (2017-09-21-20-59)

```
Loading composer repositories with package information
Updating dependencies
Package operations: 0 installs, 4 updates, 0 removals
  - Updating consolidation/config (1.0.1 => 1.0.2): Loading from cache
  - Updating phpdocumentor/type-resolver (0.3.0 => 0.4.0): Downloading (100%)
  - Updating phpdocumentor/reflection-docblock (3.2.2 => 4.1.1): Loading from cache
  - Updating consolidation/annotated-command (2.4.13 => 2.6.1): Loading from cache
Writing lock file
Generating autoload files

Symfony Security Check Report
=============================

 // Checked file: /Users/ganderson/local/consolidation/Robo/composer.lock

 [OK] No packages have known vulnerabilities.

 ! [NOTE] This checker can only detect vulnerabilities that are referenced in
 !        the SensioLabs security advisories database. Execute this command
 !        regularly to check the newly discovered vulnerabilities.

```


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.405.54f94f49f8bd250712dedd1f5619e5cd3ec99dc8](http://build.palaso.org/viewLog.html?buildId=97973&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3835, ignored: 191 Build time: 00:04:25


Probably more like `require(msg.value >= product.price * productQuantity);`


No need to iterate over this array. It would be more efficient to use: `bool array_key_exists ( mixed $key , array $array )` and to loop over an array of the key names `['mls-number, 'taxes', ....]` testing for each specific key and then storing only the data.

    $keys = ['mls', 'price', ...]
    foreach ($keys => $key){
        if (array_key_exists($custom_fields, $key){
            $data[$key] = $custom_fields[$key];
        }
    }

For the labels let's move those to the Blade template since they are considered part of the "formatting" rather than the "data" or "logic". Same goes for adding on the "$" and "sq. ft.". I know my 30 year old self might have mixed all this stuff together when this site was originally written but that guy didn't know what he was doing ;)

Also if you could have this moved to the old_single_post_in_loop_data method in the Single controller that would be good too. You can also take the `$data` associative array and then cast it as an object like this:

    $data_obj = (object)$data;

PHP doesn't have an object literal syntax like JavaScript but we can do this:

    $obj = (object)[
        'property' => 'value',
        'property2' => 123
    ];

Then you can just access it like so: `$obj->property`.

Let me know if you have any questions.


How about checking for `productQuantity > 0` just to ward off trolls.


shortened more lines in one line (12): cool 👍


:D 🤘


TeamCity Chorus :: chorus-win32-master Continuous [Build 3.0.24.89f70d7689282642d4b3000822a2eb8f8e1e7878](http://build.palaso.org/viewLog.html?buildId=97974&buildTypeId=Chorus_Win32masterContinuous) outcome was **SUCCESS**
Summary: Running Build time: 00:01:38


`require(product.stock != 0 || product.stock >= productQuantity);` is the same as `require(product.stock >= productQuantity);` but more expensive.


This pull request returns exception. Fixed it here: https://github.com/rainlab/pages-plugin/pull/294


![](https://github.trello.services/images/mini-trello-icon.png) [xmeter v6.0.0](https://trello.com/c/YMQwMTYY/37-xmeter-v600)


Wait what? Where was `msg.value - product.price * productQuantity` accounted for?


To fix ambient display (doze) you need to apply this commits to kernel: https://github.com/TeamNexus/android_kernel_samsung_exynos7420/commit/e12d3c71e212f08184cc4ab5edf50a3bf7718002

https://github.com/TeamNexus/android_kernel_samsung_exynos7420/commit/3a9d1810e20fb977125719123648f460e8734fb4

And this to device tree:

https://github.com/TeamNexus/android_device_samsung_zero-common/commit/e5f60f4afadd8593c7f486f02784505c3ca9484a#diff-380e537acdaedd487ca1adb49d020f7e

Btw. your kernel is buggy (the new created one)


![](https://github.trello.services/images/mini-trello-icon.png) [extrajs-color v2.0.0](https://trello.com/c/llfbu1CZ/1-extrajs-color-v200)


I don't understand any of these changes. Why did you duplicate the constant? Why does BaseXClient have to be included directly?


I don't think it's necessary to create mean_y_local and count_local. I recommend just putting them inline when you create computation_output. See my comment below about my recommendations on how to write computation_output.


This doesn't seem like a useful FIXME to add?


See my comment below about my recommendations on how to write computation_output.


Would be good form to add an event that summarises this state change.


Nice. Maybe we should put them in ```/classes```? Whatever


tag v2.0.0-alpha


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


godspeed! :D


Awesome, nice one guys!


never mind, I was missing rpm-build.


On this very instance, I really hope it will never be necessary because you are questioning the underlying Ethereum platform.


Awesome!


`inserting` seems more Swifty, but I don't have an official reference or anything.


That was checked above already.


Tarea #18


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-14: OK (2160 of 2160 tests passed, 0 test warnings)

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


I also used to ask the user for input, but it makes things harder, so not doing it anymore


"Scala X" would look good on gear. Is `3cala` too funky? Distributing the article: "with the Dotty _team_ at EPFL and _the_ Scala team at Lightbend."


There is no point in doing a state change before a `revert` because that state change will be reverted. All it does is ask more work from the miner. In effect, your `if else` is doing `msg.sender.transfer(` over 7 lines. So use `.transfer`.


See subsequent commits. In order to redirect stdout to additional descriptor and redirect stderr to stdout I used nested blocks, see https://vds-admin.ru/shell-scripting/sluzhebnye-simvoly#codeblock_


fgfg


erro 401


The order of regular receivers changed unintentionally from BTF to FTB in 605cbd07, this just restores it.

Additionally, in 103.0 LuaInputReceiver was drawn last (front of queue, queue iterated BTF) whereas 958b6473 made it draw first.


An event would do here too.


Sure, no problem! https://github.com/phoenixframework/phoenix/pull/2511


You are restricting 1 exchange per first party. That is an artificial limitation.


Addresses #1


How do you prevent someone from reusing the same `secretCode` that someone has used and disclosed in the past? You need to protect your customers from an honest mistake.


The coverage report issue should be fixed now

https://oss-fuzz.com/coverage_report/fuzzer/libFuzzer_libpng_read_fuzzer/latest/f/%2Fsrc%2Flibpng%2Fcontrib%2Foss-fuzz%2Flibpng_read_fuzzer.cc shows "libpng 1.6.33beta03"


Why did you just break something that was correct? Now, all someone has to do is repeat the secret word that was passed in the `startTransaction` transaction. This is not a challenge, all tx data is public.


@JonasProgrammer I've attempted to regenerate CMake and rebuild all targets in the example file - Got quite successful, no errors at all :)


Wow, crazy! I can't believe it took so long. I may have to re-evaluate its use in another project I'm using it in.

I guess it's good that we figured out the whole version chicken-egg fiasco, that at least makes it easier to add other deps in the future if need be!


Don't let the compiler decide your data type. Use `uint`.


Beyond1Airloop-Part2 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (2274 of 2807 tests passed, 0 test warnings)

 * 533 tests had: BND diffs.
 * 90 tests had: AUD diffs.
 * 133 tests had: EIO diffs.
 * 51 tests had: ESO small diffs.
 * 40 tests had: MTR small diffs.
 * 35 tests had: Table small diffs.
 * 6 tests had: Table big diffs.
 * 3 tests had: ESO big diffs.
 * 5 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-81.01%25-red.svg)


I am good for either, we can always rename it around.

On Thu, Sep 21, 2017 at 2:04 PM, Ian Jennings <notifications@github.com>
wrote:

> Nice. Maybe we should put them in /classes? Whatever
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pubnub/chat-engine/commit/69ca9d9cbc4d397491365e2f1063bfe3214986f5#commitcomment-24472287>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AACxWPkLZMt0IYB0oihrzRp_8C2qwU-Zks5sks92gaJpZM4Pf6nb>
> .
>


I recommend rewriting this function to use list comprehensions. For example:

```python
avg_beta_vector = np.mean([site['beta_vector_local'] for site in input_list])

```


Hi Andrea,

If this is a message for me, I’ll need more context to make sense of this.

Cheers
Adam

------------------
Dr. Adam Zachary Wyner
Lecturer
Department of Computing Science
University of Aberdeen

From: Andrea <notifications@github.com>
Reply-To: rst0git/LegalRuleML-app <reply@reply.github.com>
Date: Thursday, September 21, 2017 at 10:03 PM
To: rst0git/LegalRuleML-app <LegalRuleML-app@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Subject: Re: [rst0git/LegalRuleML-app] Fix import of BaseXClient (bab9977)

I don't understand any of these changes. Why did you duplicate the constant? Why does BaseXClient have to be included directly?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://github.com/rst0git/LegalRuleML-app/commit/bab9977a18234f674151bb2149209f4127bd3546#commitcomment-24472225>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ACYKC01QBc5P20DtEgVc5gnY9_ZtrYwLks5sks83gaJpZM4Pf6h3>.

The University of Aberdeen is a charity registered in Scotland, No SC013683.
Tha Oilthigh Obar Dheathain na charthannas clàraichte ann an Alba, Àir. SC013683.


Linux version


Pandey Gaurav, September 21, 2017, 4:10 PM


Ah, I think I understand my confusion - it should get events first, but be drawn last.


@githubWyner12 No, it's more directed at Radostin and concerns the commit to which it is a comment.


ERROR! Twist is bless number 1 ...


I recommend using list comprehensions here as well. They should all be short enough for you to put them in line in the computation_output dictionary. For example:

```python
beta_vector_local = [site['beta_vector_local'] for site in input_list]
```


Made a commit. This is going to be cool. I finally understand enough to confidently say things, code things and predict things.


thanks, I'll fix it later


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


oh


Please see my comments above about using list comprehensions.


Why was this done? This broke search engine links for every tensorflow model project, making it very hard to find solutions - not to mention most readme files are now incorrect.


No need for parentheses here.


The `<tt>` tag is no longer supported. See https://www.w3schools.com/tags/tag_tt.asp for alternatives depending on what you want.


cc @JunTaoLuo


I think this expression would be cleaner as:

```python
MSE = SSE_global/dof_global
```


Beyond1Airloop-Part2 (mjwitte) - 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)


Python list slicing takes the first element up to and not including the last element. So `[1:4]` is taking `item[1]`, `item[2]`, `item[3]`. Sort example via Python REPL:
```
>>> a = [0, 1, 2, 3, 4]
>>> a[1:3]
[1, 2]
>>> a[1:4]
[1, 2, 3]
```


Are you builidng the image with docker? Interessting, maybe you could help out here:https://github.com/nextcloud/vm/issues/193?


Nice, this fixes my issue as well (#138).


Please see my earlier comments about the computation_output.


Leaking indev stuff onto production smh


Неплохо, можно делать PR.


to wcale nie jest test repo


Can you show a example on this plugin?


Ah, very good point. Yes the function needs to be present in all builds!


VAVDesignMinFlow6327 (mjwitte) - x86_64-MacOS-10.9-clang: OK (2765 of 2767 tests passed, 0 test warnings)

Messages:

 * 2 tests had: EIO diffs.
 * 2 tests had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 604
 * Failed: 2

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


On the next update can we add support for "us-gov-west-1" to the AWSAMIRegionMap? However, there seems to only be support for Amazon Linux - AMZNLINUXHVM atm.


No verifica el `user->id` con el `\Auth::user()->id`, tambien pasa para editar el pago


About Me


I should check to make sure 'ancestor' is not null. Should I check to make sure 'ancestor' is not null?


See #1906


This test doesn't ensure semantic conformance because it tests too little of the API, but if it was only meant to test syntactic conformance I wouldn't expect to see anything in the function body.  So the intention of the function and the test is unclear.


I recommend saving this as a new variable. Something like this.

```python
parsed_args = json.loads(args.run)
```


The server is running on a Raspberry Pi, and the client runs on any device that runs Python.


Created multidev environment [monthofmud#ci-18](https://dashboard.pantheon.io/sites/c1008179-94f3-4a56-ba0b-53c7f98ca3f8#ci-18).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-18-monthofmud.pantheonsite.io/)


Комментарии принято писать либо сразу перед строчкой к которой они относятся, либо в конце этой строчки.
```python
def get_names():
    url = '...'
    # Комментарий к фунции get_weather
    get_weather(url)

def get_names():
    url = '...'
    #                           ↙ тут два пробела
    get_weather(url):  # Комментарий к фунции get_weather
```

И нет ничего страшного в том, чтобы переименовать функцию для нового задания )


Indeed!

I found the culprit (`bidict/__init__.py`):

```
try:
    from pkg_resources import resource_string
    __version__ = resource_string(__name__, 'VERSION').decode('ascii').strip()
except Exception as e:  # pragma: no cover
    from warnings import warn
    warn('Failed to read/set version: %r' % e)
```

It's `pkg_resources` again. Fortunately it's only used here and is easy to replace:

```
def _get_version():
    import os
    with open(os.path.join(os.path.dirname(__file__), 'VERSION')) as f:
        return f.read().strip()

try:
    __version__ = _get_version()
except Exception as e:  # pragma: no cover
    from warnings import warn
    warn('Failed to read/set version: %r' % e)
```


Thanks!


와우 !!! 플라이하이에서 제공되어 수정된건가요?


a cane setta i parametri


Why are we using Python 3.4? Why not Python 3.6?


TeamCity Market :: INFRA :: Graphouse :: Graphouse on change [Build 159](https://teamcity.yandex-team.ru/viewLog.html?buildId=10779161&buildTypeId=MarketInfra_Graphouse_GraphouseOnChangeOpensource) outcome was **SUCCESS**
Summary: Tests passed: 14 Build time: 00:00:26


TeamCity Market :: INFRA :: Graphouse :: Graphouse on change [Build 162](https://teamcity.yandex-team.ru/viewLog.html?buildId=10779164&buildTypeId=MarketInfra_Graphouse_GraphouseOnChangeOpensource) outcome was **SUCCESS**
Summary: Tests passed: 14 Build time: 00:00:24


TeamCity Market :: INFRA :: Graphouse :: Graphouse on change [Build 161](https://teamcity.yandex-team.ru/viewLog.html?buildId=10779163&buildTypeId=MarketInfra_Graphouse_GraphouseOnChangeOpensource) outcome was **SUCCESS**
Summary: Tests passed: 14 Build time: 00:00:28


TeamCity Market :: INFRA :: Graphouse :: Graphouse on change [Build 163](https://teamcity.yandex-team.ru/viewLog.html?buildId=10779165&buildTypeId=MarketInfra_Graphouse_GraphouseOnChangeOpensource) outcome was **SUCCESS**
Summary: Tests passed: 14 Build time: 00:00:28


TeamCity Market :: INFRA :: Graphouse :: Graphouse on change [Build 160](https://teamcity.yandex-team.ru/viewLog.html?buildId=10779162&buildTypeId=MarketInfra_Graphouse_GraphouseOnChangeOpensource) outcome was **SUCCESS**
Summary: Tests passed: 14 Build time: 00:00:27


@tstruk glad that you solved your issue. There isn't a package for md2man for Fedora yet, I have a copr project with md2man packages and its dependencies (binman and opener) for both Fedora.

I should propose one of these days the packages to Fedora for review, but for now one could install with:

$ sudo dnf copr enable javierm/md2man-pkgs
$ sudo dnf install rubygem-md2man


I know you don't support this, but this commit breaks Google Play Services.

`09-21 22:50:43.531: W/.gms.persistent(8522): type=1400 audit(0.0:8814): avc: denied { execute } for path="/data/user_de/0/com.google.android.gms/app_extracted_libs/arm64-v8a/libgmscore.so" dev="dm-2" ino=1497646 scontext=u:r:priv_app:s0:c512,c768 tcontext=u:object_r:app_data_file:s0:c512,c768 tclass=file permissive=0`

I guess it could also break some games or other apps but, as always, security vs usability.

Btw, thanks again for you work and keep it up!

Running custom build CopperheadOS Oreo on bullhead here, with properly integrated OpenGapps :)

Cheers!


Created multidev environment [natalie-sandbox#pr-media-en](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-media-en).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-media-en-natalie-sandbox.pantheonsite.io/)


@iBublik Вот тут не совсем согласен - я использую имя атрибута внутри сообщения эксепшна, так что я не могу прокидывать только значение


Hi @JoEgg, nice start, just before you commit more information, would you first convert the text to plain text? so it's easier to read it from the program I will write, to automatically produce mp3 files.
`.docx` are less programmer friendly than `.txt` :-)


Так название переменной другое. Вот так сработает:
```python
keys = names_data[0]['Cells'].keys()
```


i only wanted to do the bare minimum to get it to work without bloating it since this has no retail use really (mobs with custom text use TextIDs)

there is a lua binding to get the entity's name also which can be used for the string param so w/e

i guess ask @teschnei if he feels it's worth stripping bad characters from name (personally i'd rather not since we have no use for this for debugging or retail purposes)


This is a good start, but can we formalize this into a set of unit tests? Can use either [unittest](https://docs.python.org/3.4/library/unittest.html) or [pytest](https://docs.pytest.org/en/latest/). Let's also use [coverage.py](https://pypi.python.org/pypi/coverage) to measure test coverage.


Тут идет перебор всех элементов и каждый раз в keys записывается одно и то же. Если добавить `break` то это безобразие) прекратиться на первом проходе.


I approve of this rationale. If you want to go cheaper, you can pass a `bytes32` only and give a way to the client software to find out the text behind the id.


Looks like a boo-boo to me! :)


@TheDiamondYT1 I would but everytime I try to decompile to C, it hangs at a certain subroutine and begins to lock up the program.


We aren't expecting any other exception to be caught. If we catch any unexpected exception, then this test needs to fail so that we can investigate what the problem is. Agree?


issue #17


Это немного на будщее, но все же:
```python
except (TypeError, ValueError):  # Всегда лучше указывать исключения который мы планируем прехватывать
```


If the intent is that this wraps the blocks, it should probably go just around the `$this->content` echo, or within the content.phtml. As it is it includes possible module-added markup as well as the next/previous/up links.


by "custom text" for mobs i mean when they speak during battle, not sure how far up my ass i pulled "custom text" from


I was thinking in terms of 2-step degradation of the contract. Code amber (pause) and code red (kill) if you want. Perhaps then you could have another degradation step so:

1. only withdrawals allowed
2. withdrawals stopped too
3. killed


Close #6


FWIW even `--filesystem=host:ro` would be an improvement over the current situation...

However while I'm sure that xapian-bridge itself does not read any files it's not told to by the API clients, it will resolve symlinks if the passed-in path is one. See https://phabricator.endlessm.com/T13099 and https://github.com/endlessm/xapian-bridge/pull/60 for more details.

Also, I believe that SDK1-and-prior eos-knowledge-lib passes the full database path as the host sees it to xapian-bridge when making the query (to avoid requiring xapian-bridge to care about the app ID, see https://phabricator.endlessm.com/T12444 and https://phabricator.endlessm.com/T11523 for more details).

In any event, I think the tl;dr; is that it's not realistic to expect xapian-bridge to work with filesystem permissions less permissive than at least `filesystem=/var/lib/flatpak:ro` -- and perhaps even `filesystem=/home:ro`, since I don't remember exactly whether `/home/$user/.local/share/com.endlessm.subscriptions/` paths are passed in case of a sharded app with a manifest instead...


If you have any idea, I'd be happy to hear. For now I might just provide .asm and C headers (.h).


just sayin I am disappoint son

if you have to bother using getName() you may as well be typing it into the message text to begin with though. we had that whole convo on discord. the entire reasons to do this at all was reduce ppl asking about it and reduce ppl breaking their updates with huge blocks of custom then bugging us for bugs we don't have. This accomplishes that so "good enough" but I like entity just sayin its name automatic if not null a lot better.


solves issue #5


While testing this whole thing I've probably spammed @sriemer with emails. I'm sorry.

For the future, I'm planning on writing a simple compile with `-fsanitize=address,undefined` and have it scan 3/4 different types. Hopefully I can actually scan another process inside a travis container.


I recommend breaking this out into its own function because it is complex.


> If we catch any unexpected exception, then this test needs to fail so that we can investigate what the problem is. Agree?

Right, the snippet I've provided does the same, just w/o the additional conditional logic 🙂


Probably want to add 3.0.0-ALPHA8.


> I know you don't support this, but this commit breaks Google Play Services.

CopperheadOS doesn't include Google Play Services so it's not something that's taken into account. This isn't the only change that's incompatible with what it requires.

> I guess it could also break some games or other apps

An app needs to be in `/system/priv-app` to be in this SELinux domain. It's only for the base operating system so it's not possible for this to cause compatibility issues with third party apps.


#6320-Some-_ExternalInterface--test-files-not-transitioned-properly-for-new-field-in-ZoneHVACIdealLoadAirSystem (rraustad) - i386-Windows-7-VisualStudio-14: OK (2768 of 2768 tests passed, 0 test warnings)

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


Amazing! Keep up the great work

![mindblow](http://www.reactiongifs.com/wp-content/uploads/2013/10/tim-and-eric-mind-blown.gif)


`s/type/typo/` WTF a recursive typo? 😆 🤣 🙃


WTF is this shit


Why do you have removed my work?

You have removed the function: getAutoTimerSettingsDefinitions
This function is needed in AutoTimerResource too.

See here ->
https://github.com/oe-alliance/enigma2-plugins/blob/0f298505f09759eb2971258dc01d23a622054415/autotimer/src/AutoTimerResource.py#L18

Please revert. !!!


В комментариях вначале файла или в данном случае можно в строке документации (докстринг - далее опишу что это) к функции стоит записать в чем домашнее заание заключалось. И мне будет проще, и когда будешь возвращаться к старому коду, будет легче понять, что он должен делать.

Докстринг - это принятый в Python способ добавлять описание к функции
```
def get_name():
    """Описание задания
    продолжение описания"""
    pass

# так его можно получить для функции
get_name.__doc__  # -> 'Описание задания\n    продолжение описания'

# можешь попробовать с известными функциями
print(print.__doc__)
```


Beyond1Airloop-Part2 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1553 of 1553 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-38.74%25-green.svg)


Pega la solución


My own custom version works like:
```lua
Target:PrintMuhShiz("text here", SPEAKER ENTITY HERE, MESSAGE TYPE, MESSAGE RANGE)
```

so:
```
Target:PrintMuhShiz("text here", NPC, SAY, PARTY)
```
Would transmit a SAY type message to all of Targets party members and look as if NPC's was talking. Code that strips the _ out is just a few lines. I had it its its own func but is easy to move to just a block in the existing one. I dunno if it can be lua'd I did it in C++

... I could prolly utils's it and have it pass input to what you have here but am lazy and don't wanna type npc names so I'll prolly just keep on using my custom block. Unlike newbies who hit us in chat I know how to not break my pulls.


Created multidev environment [natalie-sandbox#ci-30](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#ci-30).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-30-natalie-sandbox.pantheonsite.io/)


Yeah, the src dir is in dist. Our tsc root is the project root.


That's what i meant to add


Good point, I fixed the OS specific file stuff in a follow up commit.


You're right, they could be more descriptive. `EquipType` I would consider to be adequate, it's the equipment's type, a sword, an armor, etc, analogue to an item's type, which could be a usuable, equipment, etc. The 1 and 2 are a problem, but I couldn't think of anything better at the time, so I named them 1 and 2, like IMC did in the client.

While 1 seems to be the immediate type (e.g Sword or THSword), 2 apparently is the "category" (e.g. Sword for both swords and two-hand swords).


Check https://github.com/tchapi/markdown-cheatsheet/blob/master/README.md


Check https://github.com/tchapi/markdown-cheatsheet/blob/master/README.md for inline code.


Happy day!
:+1:


Parcial 1


I can't understand this testcase, it is level of Visotskiy or higher


My bad. Are there pros n cons in this case? Does it matter?


![](https://github.trello.services/images/mini-trello-icon.png) [For this challenge you will be sorting characters in a string.](https://trello.com/c/VmoQkb5G/2-for-this-challenge-you-will-be-sorting-characters-in-a-string)


You're doing *_init() twice, there's no reason to do this. That information is preserved across a fork.


I don't think this should be called db. There are a lot of databases. Maybe bdb?


PFFB registers for clockgating ?


1021 holland dr garland tc


arquivos de aula (nao é o que eu efiz) 190917


how about berkeley-db? it's the second name of the package


ok berkley-db will work


Great to see you remembering to use the `name` attribute!


How would you indent the `input` fields relative to the `form` element?


the rest of the path needs to be changed too: ...System/Runtime/Serialization/Formatters/BinaryFormatterHelpers.cs


@sikhub Good catch, thanks!


MultiPartialHVAC (mjwitte) - x86_64-MacOS-10.9-clang: OK (2766 of 2766 tests passed, 0 test warnings)

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


meu primeiro arquivo


Make it `internal` then to reduce your attack surface.


Not very expressive.


Oh, I see. makes sense. I guess `pkgdown` uses the official docs and not the roxygen code, then. thanks.


And check the event was emitted.


1.25 AM 😞
![giphy](https://user-images.githubusercontent.com/523221/30721525-218fac08-9f35-11e7-9d8e-ec53ffd80140.gif)


they should never have all used the foreign middleware, that's a typo.
All the post stuff needs to be refactored back to common code. but I'd do that in a pr separate from refactoring the middleware, of which there is much to do


sweet! 👍


Nice use of classes and ids throughout!


This could be a helpful tag for you in the future https://www.w3schools.com/tags/att_time_datetime.asp


Looks like the `iframe` could be indented relative to the `div` here


Created multidev environment [natalie-sandbox#pr-entity-e](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-entity-e).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-entity-e-natalie-sandbox.pantheonsite.io/)


Nice use of nested selectors!


👍  nice work with the grid area!


awesome : )


You can write these as one-liners,

`<li>Check-in at AirBnB in Akihabara</li>`


Nice work implementing some jQuery here!


happy to see you using `$(this)` !


Add link/credit to original Torrent class created by [adriengibrat/torrent-rw].

[adriengibrat/torrent-rw]: /adriengibrat/torrent-rw


Good luck!


This needs to remain.


ldap all the things.


Looks like a bad merge artifac @musically-ut


This function is not needed either.


Dank Meemur at ur servise


Minor typo "tasks scheduled"


Meeeemez


You should avoid looping an unbounded number of times because you will hit the block gas limit, and everything will be rolled back.
Your data structure is not adequate to the purpose.
Start simple. Alice and Bob.


Think about the remainder.


Avoid forcing a 2-transaction process when a 1-transaction process is possible.


I really like the changes you made to displaying the replytime (TimeSpan is a no brainer) so I've mirrored them in master and credited you.


FWIW my theory from the commit message doesn't seem to be true: I rewrote the lexer to not use a separate goroutine but it didn't help performance noticeably.


Wrong calculations, you are going to give away `msg.value * accounts.length / (accounts.length - 1)`. Me thinks it is not possible to give away more than you have.


To make it simpler, let's assume that alice is `msg.sender`.


It would be good to check that the addresses are `!= 0` to cover honest mistakes.


Well, this commit addresses rather issue  #389 than #423.


VAVDesignMinFlow6327 (mjwitte) - x86_64-MacOS-10.9-clang: OK (2765 of 2767 tests passed, 0 test warnings)

Messages:

 * 2 tests had: EIO diffs.
 * 2 tests had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 604
 * Failed: 2

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


Whyyy again


You are aware that any data that is in a transaction is public knowledge. The passwords have been disclosed at deposit time. Anyone can come and claim, there is no challenge.


redireccion a "Home" en modulo add proyectos, add actividades y add reuniones.


Why do you need this commit?


Yup, already addressed in #2891.


exact translate : 이름 (하강하는) -> 이름(내림차순)


"데이트 할 부가 기능이 없습니다." -> "업데이트할 부가 기능이 없습니다."


#6320-Some-_ExternalInterface--test-files-not-transitioned-properly-for-new-field-in-ZoneHVACIdealLoadAirSystem (rraustad) - Win64-Windows-7-VisualStudio-14: OK (2768 of 2768 tests passed, 0 test warnings)

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


So the final patch looks like this after the squshes:

~~~
diff --git a/sys/dev/random/fortuna.c b/sys/dev/random/fortuna.c
index e300c6ffb4ca..bc15f9b50faa 100644
--- a/sys/dev/random/fortuna.c
+++ b/sys/dev/random/fortuna.c
@@ -1,4 +1,5 @@
 /*-
+ * Copyright (c) 2017 W. Dean Freeman, CISSP CSSLP
  * Copyright (c) 2013-2015 Mark R V Murray
  * All rights reserved.
  *
@@ -232,16 +233,32 @@ random_fortuna_process_event(struct harvest_event *event)
 	 * during accumulation/reseeding and reading/regating.
 	 */
 	pl = event->he_destination % RANDOM_FORTUNA_NPOOLS;
-	randomdev_hash_iterate(&fortuna_state.fs_pool[pl].fsp_hash, event, sizeof(*event));
+
+	/*
+	 * We toss low entropy static/counter fields towards the end of the
+	 * he_event structure in order to increase measurable entropy when
+	 * conducting SP800-90B entropy analysis measurements of seed material
+	 * fed into PRNG.
+	 * -- wdf
+	 */
+	KASSERT(event->he_size <= sizeof(event->he_entropy),
+	    ("%s: event->he_size: %hhu > sizeof(event->he_entropy): %zu\n",
+	    __func__, event->he_size, sizeof(event->he_entropy)));
+	randomdev_hash_iterate(&fortuna_state.fs_pool[pl].fsp_hash,
+	    (const void *)event->he_somecounter, sizeof(event->he_somecounter));
+	randomdev_hash_iterate(&fortuna_state.fs_pool[pl].fsp_hash,
+	    (const void *)event->he_entropy, event->he_size);
 	/*-
 	 * Don't wrap the length. Doing this the hard way so as not to wrap at MAXUINT.
 	 * This is a "saturating" add.
 	 * XXX: FIX!!: We don't actually need lengths for anything but fs_pool[0],
 	 * but it's been useful debugging to see them all.
 	 */
-	if (RANDOM_FORTUNA_MAXPOOLSIZE - fortuna_state.fs_pool[pl].fsp_length > event->he_size)
-		fortuna_state.fs_pool[pl].fsp_length += event->he_size;
-	else
+	if (RANDOM_FORTUNA_MAXPOOLSIZE - fortuna_state.fs_pool[pl].fsp_length >
+	    (sizeof(event->he_somecounter) + event->he_size)) {
+		fortuna_state.fs_pool[pl].fsp_length +=
+		    (sizeof(event->he_somecounter) + event->he_size);
+	} else
 		fortuna_state.fs_pool[pl].fsp_length = RANDOM_FORTUNA_MAXPOOLSIZE;
 	explicit_bzero(event, sizeof(*event));
 	RANDOM_RESEED_UNLOCK();
~~~

@jmgurney and @badfilemagic could you please take a look at these patches?


"업데이트 만 표시"  -> "업데이트만 표시"


[Table] Improve scroll behavior in TableQuadrantStack (#1578)

Preview: documentation
Coverage: core | datetime


@mortezadadgar looks like this commit might be breaking NFC for me (Xperia Z5, pn547),

any issues for you after this ?


Just out of curiosity, what difference does this make?


Yeah, it was on purpose. That said, I have another patch in @OJ 's queue that normalizes it back to Int, mostly just for consistency (and people probably don't _need_ to specify pi for retry intervals :)


Sandbox folder used to learn how to load and modify projects


We actually have a way to blacklist specific PCI IDs with broken MSI implementations.  See the 'pci_quirks[]' table in sys/dev/pci/pci.c and the PCI_QUIRK_DISABLE_MSI and PCI_QUIRK_DISABLE_MSIX quirks.  Is the device currently using MSI-X instead of MSI?  If so, I would try PCI_QUIRK_DISABLE_MSIX first to see if regular MSI still works and if that also fails then fall back to PCI_QUIRK_DISABLE_MSI.


++ both this and its sibling 1bf4cbaffc7991b188cd1542eff58f8427b838f2. master +1 too already reviewed


[Table] Validation improvements (#1587)

Preview: documentation
Coverage: core | datetime


0.14.0 is marked as the [latest release](https://github.com/seatgeek/fuzzywuzzy/releases/latest).