@Rotonen Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.intelligenttext/6/violations
```
plone/intelligenttext/tests.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/tests.py:11:5: T003 print function found.
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:21:10: C812 missing trailing comma
plone/intelligenttext/transforms.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/transforms.py:22:9: P103 other string does contain unindexed parameters
plone/intelligenttext/transforms.py:22:80: E501 line too long (278 > 79 characters)
plone/intelligenttext/transforms.py:23:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:23:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:27:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:27:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:63:45: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:68:20: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:69:32: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:73:26: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:37: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:54: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:90:11: T000 Todo note found.
plone/intelligenttext/transforms.py:97:16: S001 found module formatter
plone/intelligenttext/transforms.py:107:16: S001 found module formatter
plone/intelligenttext/transforms.py:150:18: S001 found module formatter
plone/intelligenttext/transforms.py:176:17: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:176:44: C812 missing trailing comma
plone/intelligenttext/transforms.py:179:7: T000 Todo note found.
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@russburnett @lulachen We need to explain why we have the blocks. And we need to be clear whether there was a Team by Resident block or just a Team block with information about Resident as a potential moderator or something. The table below looks like it could have been blocked by both Team and Resident (so we have some pairs for example).

```
> (table(TeamXResident=interaction(providers2$Team,providers2$Resident),Treatment=providers2$Email,useNA="ifany"))
             Treatment
TeamXResident No Yes
    Blue.No    4   5
    Gold.No    5   6
    Red.No     5   4
    White.No   5   6
    Blue.Yes   1   1
    Gold.Yes   1   1
    Red.Yes    1   1
    White.Yes  2   1

```


I think you should use `parse()` here. `getResponse()` is only meant for the main page template (including cache headers etc.)


2 ?! At 3 Numbers ??!?


Oh right! This is the best solution.


I understand. Thank you for the explanation!


These are legacy API that exposed legacy MBP code. I think it is better to deprecate and remove them. They can be re-created if there are clients for them, but hopefully using newer MBP code.


Hello, i need your help. How do i use the multi-selection?


![](https://github.trello.services/images/mini-trello-icon.png) [Neues Icon für MCE](https://trello.com/c/zPjYTtfG/6-neues-icon-f%C3%BCr-mce)


yep. Dar s-a stins laptopul ca nu mai avea baterie,parca am lucrat la task 3.


Make a directory?


@Rotonen Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.intelligenttext/7/violations
```
plone/intelligenttext/tests.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/tests.py:11:5: T003 print function found.
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:21:10: C812 missing trailing comma
plone/intelligenttext/transforms.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/transforms.py:22:9: P103 other string does contain unindexed parameters
plone/intelligenttext/transforms.py:22:80: E501 line too long (278 > 79 characters)
plone/intelligenttext/transforms.py:23:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:23:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:27:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:27:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:63:45: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:68:20: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:69:32: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:73:26: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:37: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:54: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:90:11: T000 Todo note found.
plone/intelligenttext/transforms.py:97:16: S001 found module formatter
plone/intelligenttext/transforms.py:107:16: S001 found module formatter
plone/intelligenttext/transforms.py:150:18: S001 found module formatter
plone/intelligenttext/transforms.py:176:17: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:176:44: C812 missing trailing comma
plone/intelligenttext/transforms.py:179:7: T000 Todo note found.
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


You should change the definition instead of creating arbitrary parameters just for dependency injection. See https://github.com/terminal42/contao-assets/blob/tags/src/DependencyInjection/Terminal42AssetsExtension.php#L50-L53


Created multidev environment [bigger-cyle#ci-1](https://dashboard.pantheon.io/sites/7d672e55-b2de-4c70-a461-431315717908#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-bigger-cyle.pantheonsite.io/)


![](https://github.trello.services/images/mini-trello-icon.png) [LeetCode #160](https://trello.com/c/593u64j1/96-leetcode-160)


Ok. I suspected that. Push then


@sahil21: Please put the why of the commit as well. Very important in the long run. Check the guidelines.


I thought the recommended multi-line comment block format was:
```
/*
 * comment
 * ...
 */
```
?
You should indicate that you are deliberately NOT doing kernel-doc format, because these are static functions.


The code on line 101 executes after the code on lines 104 and 106 because 101 is within a `setTimeout`. Look at the [whole snippet](https://github.com/bvaughn/react-virtualized/blob/28410af2d54e1755c325d9e3fecbb1b32d97287c/source/InfiniteLoader/InfiniteLoader.example.js#L96-L106) rather than this diff view. 😄


![](https://github.trello.services/images/mini-trello-icon.png) [LeetCode #229](https://trello.com/c/D19wHnTY/97-leetcode-229)


@lachellel - I think key usage should remain critical.

I don't see the reference to criticality in 7.1.2.3?

> keyUsage (optional)
> If present, bit positions for keyCertSign and cRLSign MUST NOT be set.

5280 says this:

> When present, conforming CAs SHOULD mark this extension as critical.


I think the new line 36 (the new intervening div) may be preventing the list items from rendering as Bootstrap dropdown menu items. On our site, they have now started rendering as normal links.


Yes it is except for the /net directory which I'm proposing is where visornic goes. Okay, I'll make sure I state that in the comment, thanks!


Hmm since we are in groovy I doubt we can call that function, but we will find a workaround, thx for looking in up though 👍


Ahh, ok.  Thanks.


Also, not sure where this happened but if you ask for `terraform version` on our 0.10, it says 0.10-dev and mistakenly claims that the version is out of date. Back with 0.8.8, it didn't append the -dev, so I'm guessing at some point they made it do that by default unless you ask it not to.


thanks @tylerjl !


hi bro you can help me?


Different times I needed things and I hope I could help others with my simple contributions.


Script to complete conservation status from DBPedia


Please checkout the Dokumentation and the prozess ;) without the bigger picture this comment does not help you ;) thanks.


@anna-zhernovkova

Do you use EditorConfig, ESLint and JSHint plugins for VS Code?


@juhyem don't forget your closing tags!  `</body></html>`


**INS 5386 - New event trigger added in payment service for UI**

[Deployment Steps.txt](https://github.com/KITSHarikrishnanSubramani/kits-deadpool-artifacts/files/1197583/Deployment.Steps.txt)
[Release_Note.txt](https://github.com/KITSHarikrishnanSubramani/kits-deadpool-artifacts/files/1197586/Release_Note.txt)
[03082017.zip](https://github.com/KITSHarikrishnanSubramani/kits-deadpool-artifacts/files/1197588/03082017.zip)


@HiddenRambler No problem, im not fussed either way :P


@sp-eldata Got it.


changed / updated in this later commit: https://github.com/uspki/policies/pull/145/commits/3f0c8fb63681fa89da6d1f7e9fde78e8c2523ba6


Processing name starting with b will need very special care. Here is my  `/sys/class/net` :
```
root@marvin:~# ls /sys/class/net/
bond0            docker0  veth0046f7d  veth44147db  veth5d383db  vethe3fd206
bonding_masters  eth0     veth0ce486b  veth5340447  veth7825e3f
br-cfa3ac0f6eb5  lo       veth1c53921  veth574eeb3  veth92c4968
```

The only interface I'm really interested in is eth0 all the rest is due to docker (including br-cfa3ac0f6eb5).
If you add a 2 IP limit then maybe you can sort the interfaces in reverse alphabetical in order to process the bridges last.

For VLAN I would simply drop any interface starting with a `e` or a `w` and containing a dot. At least that's an easy starting point (not perfect though).


I am missing something. What does it mean that "a test does something unexpected with an entity ID"?


Six?


Kyllä jatkuu jatkuu. Mi Max 2 varmasti hyvä laite. Viikon päästä 7.8.11 saatkin Mi 6:lle MIUI 9 ekojen joukossa. Pistähän viestiä minkälainen on sitten. :)


File renamed without changes.


When it is safe to remove, add a pull request.


Don't go down this path. This is a kang


Yes. I skipped it and saw after commit only


closes #24


The csgofast API is back online.


This in part belongs to Steve Kondik. Pick his commit


Created multidev environment [bigger-cyle#pr-address](https://dashboard.pantheon.io/sites/7d672e55-b2de-4c70-a461-431315717908#pr-address).

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


The line 160 check (`!metafile`) should be done here instead, so that errno (`%m`) is the right one (the one for `fopen`, not the previous one from `fileno`).


Cefca16, see issue: https://github.com/Unity-Technologies/GenericFrameRecorder/issues/1


If we didn't unlock the buffer lock on line 26 and line 31, like the following:
![image](https://user-images.githubusercontent.com/838924/28928350-8cad8292-789f-11e7-8a64-b184e9029df2.png)

It will cause the error "Maximum lock count exceeded"
![image](https://user-images.githubusercontent.com/838924/28928432-b9fe4bdc-789f-11e7-94de-6be9630c0523.png)

To solve this, invoke the unlock method when the buffer is empty and when read the EOF, then the buffer lock will be able to release properly:
![image](https://user-images.githubusercontent.com/838924/28928547-0858c62c-78a0-11e7-84b7-736938077ba4.png)


Use of "*" dependencies is usually frowned upon


What build are you using?


My personal opinion is that this patch just makes the code more fragile, and less readable.  But I guess fixing a checkpatch warning is more important.  ;-)


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2045)*


Great explanation. Didn't realise the dist would be read as a fallback.
Thanks! 👍


Ah, so that's 2.4 ghz? There's no way to fix it, we should wait when LEDE devs backport a newer driver.


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/5d60e65a-2eb9-47dd-8bc3-b34356ab2430 "simple-toolchain-20170803150016131"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/ad5b2ba7-53f7-4018-9a00-8e926d3fb0a4 "simple-toolchain-20170803150016131") deployed [simple-toolchain-20170803150016131](https://github.com/venkat-ramana/simple-toolchain-20170803150016131/releases/tag/deploy-ibm-yp-us-south-laurakuna10-dev-20170803-150750 "deploy-ibm-yp-us-south-laurakuna10-dev-20170803-150750") to [dev](https://console.bluemix.net/apps/d4f73aa5-ceb1-4870-895a-55bb11264211?env_id=ibm:yp:us-south "ibm:yp:us-south:laurakuna10:dev"), including this commit


사소한거긴 하지만.. parameter를 x_prev, y_prev나 x_next, y_next보다 좌표니까 pair를 써서 구성하는게 낫지 않을까
def ... (self, prev_coord, next_coord)


You rock!


We blocked on team x resident (so, 8 blocks in all). I believe the rationale for blocking by team was that providers within a team coordinate, communicate regularly -- and so there might have been some systematic variation between teams in how exactly the in-person intervention (encouraging the patient to register, etc.) was implemented. The rationale for blocking by resident, as I understand it, was that residents may have shorter-term, less stable relationships with their patients -- so they might have been less enthusiastic about encouraging patients to establish connections with them in the secure messaging system (and their patients might have been less enthusiastic about doing it). Does this all make sense?


You should make a data structure whereby you add up what a payee has received so far, instead of individual payouts added in an array. Like this the payee does only need ask to be paid once for all that it has received.
Moreover, here, each `Payee` costs 2 SSTORE...


> Processing name starting with b will need very special care.

We could use use a regexp like `br[0-9]+` to handle bridges separately. But as I see bridges will mostly be used for internal LAN and have a static IP, so it may be simpler to no include them.

> For VLAN I would simply drop any interface starting with a e or a w and containing a dot. At least that's an easy starting point (not perfect though).

There may be a case where you have IP on the VLAN interface like `eth0.101` but the main interface `eth0` doesn't have an IP. But this is a relatively rare case (only 3 supported boards have a DSA capable switch that requires setting VLANs) so we may as well wait for the feedback.


did you mean this? https://github.com/wikiwho/WhoColor/commit/00725170d7564eaaa9449a287d8a9a5a7ebefe3c


Do you want to base the ComboVar image to the Table image? Instead using false would it be better to use `!insertImage`?


Ah it was nothing


@Rotonen Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.intelligenttext/8/violations
```
plone/intelligenttext/tests.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/tests.py:11:5: T003 print function found.
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:16:29: S101 Multi-line construct missing trailing comma
plone/intelligenttext/tests.py:21:10: C812 missing trailing comma
plone/intelligenttext/transforms.py:4:1: I003 isort expected 1 blank line in imports, found 0
plone/intelligenttext/transforms.py:22:9: P103 other string does contain unindexed parameters
plone/intelligenttext/transforms.py:22:80: E501 line too long (278 > 79 characters)
plone/intelligenttext/transforms.py:23:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:23:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:27:9: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:27:27: C812 missing trailing comma
plone/intelligenttext/transforms.py:63:45: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:68:20: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:69:32: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:73:26: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:37: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:79:54: Q000 Remove bad quotes.
plone/intelligenttext/transforms.py:90:11: T000 Todo note found.
plone/intelligenttext/transforms.py:97:16: S001 found module formatter
plone/intelligenttext/transforms.py:107:16: S001 found module formatter
plone/intelligenttext/transforms.py:150:18: S001 found module formatter
plone/intelligenttext/transforms.py:176:17: S101 Multi-line construct missing trailing comma
plone/intelligenttext/transforms.py:176:44: C812 missing trailing comma
plone/intelligenttext/transforms.py:179:7: T000 Todo note found.
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Yes, Only 2.4G crashes. I tried hnyman's build, the same. Now I'm using my own build. But I can't fix this.


This is actually supported in V5. Just not added to the API reference yet.

Here's a definitive list of the V5 features - https://github.com/conjurinc/possum/tree/master/cucumber/api/features


Actually I pushed some changes to the Julia stuff that should make most things +w so let me know if things are working once it makes it into the Singularity image.


I am afraid it will not protect you against overwrite. You appear unaware that `now` is an alias for `block.timestamp`. This value is the same for **all** transactions in a block...
 It will be very possible for 2 transactions with the same sender, same value and same payee length to be in the same block. 💥 same hash!
That is a serious bug.


Sadly I agree with you, but it fixes a checkpatch. My bigger concern is I need to do some testing with devices that don't have drivers associated with them (i.e. Service Partition Recovery when we have rmmoded the visornic driver, or failed to load it to begin with).


I would rather you just do:
```
err = unisys_vmcall(VMCALL_CONTROLVM_ADDR, virt_to_phys(&chipset_dev->controlvm_params));
```
instead of essentially using the `addr` variable for 2 different purposes.


Interesting. These are weird. What makes this different treatment from line 20?


Actually, all of this should pretty much work in v5 with only minor modifications (maybe the routes or some parameters are a little different). Host Factory is definitely supported in v5.


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2046)*


Style thing. I would name it, if possible. It's possible. Generally, I incline to `bool success`.


Speaking of the TF packages, take a look at [this section](https://www.terraform.io/guides/running-terraform-in-automation.html#pre-installed-plugins) of their docs. Nix should probably build the plugins and symlink them into a directory, and then create a wrapper script that passes in -plugin-dir=/usr/lib/custom-terraform-plugins to the underlying terraform.


You could go nuts and `untrustedWithdraw(uint amount)`


why am i getting b5c168db error when using new files?


Will do!


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2047)*


It's important that when using lock objects, we're responsible for releasing the lock. It won't happen automatically as it would when we use the synchronized block, so the intrinsic locks we use with synchronized blocks are always released when the thread holding the lock exits the synchronized block. So this is one drawback to using lock objects in that we have to manage this lock process themselves by making sure we call the lock() and unlock() methods. If we forget to release the lock threads waiting for the lock will then start blocking forever.


![](https://github.trello.services/images/mini-trello-icon.png) [Lizenz eingefügt](https://trello.com/c/WmtBqfZ7/12-lizenz-eingef%C3%BCgt)


@fwhcat: how to avoid writing the MySQL password in the source code?
(The current password only allows to authenticate on my test database which is not accessible world-wide.)


期待，用dubbo两年了


No, `i` is not the `depositIndex`. It is the index inside `_payees`. Very different. `i` goes from `0` to `length1 - 1` on the first split, then from `length1` to `length1 + length2 - 1` on the second deposit.
On the other hand, `.push` returns you the new length. So `depositIndex = ...push(...) - 1;`.


HI,

I'm try use your libray with my arduino uno (16Mhz) but i don't get, because just run on with arduino mini?

Thank you

Matheus


@lachellel

Not sure removing "not to exceed 20 octets" is a good idea. Although perfectly fine from an DER and X.509 standpoint, I've seen more than 20 bytes in a serial cause problems.

5280 gives client software a free pass to break if it's >20 bytes:

> Certificate users MUST be able to handle serialNumber values up to 20 octets.  Conforming CAs MUST NOT use serialNumber values longer than 20 octets.


But this deposit array is a bad data structure anyway, you need to replace it.


@jwbowers I followed Russ' initial reasoning for focusing on the treatment period.  Per the redacted version of the initial analysis "We limit the provider-level analysis to the treatment period, even though we have data on some variables for months prior to treatment. This is because different providers joined the clinic at different times during the pre-treatment period (a missing data problem) and because we don’t know the histories of these providers prior to the treatment period (e.g., they may have had different roles or responsibilities in earlier months)."  I'll add this text into the draft of the reanalysis.


Yep! Thanks!


My point was since t1 can only be a scalar, computing a norm seems useless


You might want to review the parameters for the payload.


Maybe use `DISPLAY_MANAGER=nodm` or `DISPLAY_MANAGER=lightdm` instead of `yes`?


Thanks I got it now.
Good thing you have eslint in the project otherwise a typo can be unnoticed and babel doesn't give any exceptions.
Thanks for the component!


Providers were put into blocks and then treatment was randomly assigned within each block.   So, if a provider was assigned treatment, the provider would email all of his/her patients.  Ie, provider-level randomization where blocks are Teams x Resident


thanks this is now fixed on master


I think this will be gas-cheaper:

```
uint txAmount = msg.value / payees.length;
uint txRemainder = msg.value - txAmount;
```


Neither did I until I googled it :^)


> We could use use a regexp like br[0-9]+ to handle bridges separately. But as I see bridges will mostly be used for internal LAN and have a static IP, so it may be simpler to no include them.

Agreed !

> We could use use a regexp like br[0-9]+ to handle bridges separately. But as I see bridges will mostly be used for internal LAN and have a static IP, so it may be simpler to no include them.

As it's not critical (it's "just" a banner displayed), I would still vote for adding a check dropping interfaces with dots. But I'm going on vacation in a few hours so I may not think clearly ;)


Remove this commit later


wouldn't it be easier if you just submitted PR's for this. its more convenient to apply


I dropped people without Secure Messaging Names because they did not have secure messaging and did not have outcome variables.  For these individuals, most of the dataset included NAs.  Wasn’t sure if there was a better way to deal with them.


Wrong comment.


begging of the file --> beginning of the function


I just often forget to update the version in README.
It's up to users to decide which version they want to use.


Thanks! I have this commented out because I need to fix a deeper issue with my grammar first. I found while testing that "a" and "(a" actually put the parser into the same state, which has an EOF code in the follow set. For the first one, it makes sense. For the second one, it should not have EOF in the follow set because it needs a closing parens. There are many others that do work correctly, such as "a+\nb" will correctly try to ignore the newline. This is probably due to my R/R conflicts, which I found seem to be entirely due to my "expr expr" implicit multiply rule. I wish I could just take it out, but implicit multiplication is used too much. I'm currently following http://robertjacobson.herokuapp.com/blog/2015/01/28/the-grammar-of-mathematical-expressions/ to properly support implicit multiplication.


So you can use a different detail view in the CreateController as opposed to the ViewController without having to subclass.


You could add a `require(!active);` just to be on the safe side.


This should fix #26


Why did you make it `external`?


I believe @zahnno had a use case in mind and I see no reason not to allow such customization.


Also, re: the "-dev", the [TF build script](https://github.com/hashicorp/terraform/blob/fb50afb0a15e3735dd8c42e9c1fc8382ebde864d/scripts/build.sh#L47) overrides that variable to be the empty string when running a release build. Not sure if Nix should just patch [this](https://github.com/hashicorp/terraform/blob/fb50afb0a15e3735dd8c42e9c1fc8382ebde864d/terraform/version.go#L15) or not?


You're welcome!


These should use the "name" PHPDoc annotation.


Created multidev environment [bigger-cyle#pr-address](https://dashboard.pantheon.io/sites/7d672e55-b2de-4c70-a461-431315717908#pr-address).

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


Done! Thanks for catching that, guess it was just begging to be caught.


Semantically, it should be `require(`.


And BTW, `require(active);` is enough.


希望发布新版本，如果不合并该PR，app 同步结果校验将一直失败


`_txId` could benefit from being indexed too.
`_timestamp` need not appear in the event, it is already part of the log element. Save yourself some gas.


This may not be enough to know if the request is 3rd-party.

In the blok code-base, [I stored a src "top" host when the `requestDetails.frameId === 0`](https://github.com/mozilla/blok/blob/faac2281c48cd226b4fb8c4e22de588a02328c31/src/js/background.js#L89-L94). We'll probably have to do a similar thing here, to catch things like:

1. Main Frame wired.com -> 2. iframe doubleclick.net -> 3. js doubleclick.net request with potentially unsafe data

This code log the 1->2 request, but will ignore the 2->3 request, even though that's the one with the potentially unsafe data.


Oh and rename those `tx` without the `_`. That will be nicer when you access them in Javascript.


Though I wonder if the unsafe data will always pass thru 1->2 first?


ggg


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2050)*


Made a mistake on the commit message - it should read "Remove EventTickets details links from event details page".


Better you deploy a brand new one for each test.


Not sure what exactly you mean with jerk here - this is how it behaves on iOS with the current code in master

https://youtu.be/RowurKNKVq8

That focus bug is a skinning issue afaik


good catch... I can see this one causing headaches later :)


That's interesting! However, I'm not sure: Does Wolfram language/Mathematica support implicit multiplication or not? I'm asking because I've just started playing with the grammar too ;-)


Yes, it's "unknown" in a sense "not supported yet", i.e. we don't have an appropriate functor for such operators. Currently we can't have equality (or identity) comparisons with non-constant value, because otherwise logic of corresponding functor becomes quite complicated.


cool that's awesome .


Why not camelCase across the board?


Are you deleting semi-colons on purpose? I tend to leave them all on so I don't forget them when I add lines.


This is a synchronous call. Change it into an async one.


You should flatten your promise chain.


Here too is a synchronous call.


Excelent, thanks!


`_one_` on both lines?


The same thing is happening now in OCRw…
Is there a fix?


You have little control on an account balance so you should instead make some subtraction.


sorry should be `c:\cygwin64\bin\sh`


There is a function that converts arch package names into [name,slot] object. See `arch_to_pkg`.

Move 'official list calculation' into separate function so it can be used in other places (e.g. in lib/cleanup.rb).


|`line 14`|
|---|
It removes the default JAR archive added by the Java plugin.


What do you mean by 'b5c168db error'?


biutiful


I think it'd be good to mention authors (and with links, where available)


Wait what?? 3 promises launched side by side? Don't do that. Chain them.


Also add these chars to **StrippableText.cs**?


This is just what I needed! Hopefully it gets merged.


it's not necessary, especially for launch, but most (but not all) have GitHub repos (useful for viewing source and for reporting issues)


A proper identation of your Javascript will make it apparent.


"Try WebVR in your Browser" or "WebVR Sites in the Wild"


That does not sound like a balance.


It would be a bit more idiomatic to add

```
next if OFFICIAL_PACKAGES.include?(key)
```

above, at line 567/571


or since this is just a video, "What is WebVR?"


All the links I clicked on with github and the website seem to still point to this one and are 404s.


Thanks! 🎉


Created multidev environment [bigger-cyle#pr-address](https://dashboard.pantheon.io/sites/7d672e55-b2de-4c70-a461-431315717908#pr-address).

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


It is better to use `keys` here (that is array of [name,slot])


|`libraries/tools/gradle-tools/pluginMarkers.gradle:35-36`|
|---|
Could you explain in a one-line comment what this block does?


Also added nav bar throughout


I'm using the file with Subtitle Edit
The old one worked fine
Now when I use this I get
Tesseract b5c168db has stopped working
Or something like that…


@fernandojsg has a branch that works on Google Daydream (supports Chrome and Servo). @caseyyee could probably merge that branch in the `ablast` repo. And we could probably get Cardboard support with a few line changes.


if we do `next if OFFICIAL_PACKAGES.include?(pkgname)` above then this check is not needed anymore, right?


could use `display: inline-block` and reset the `font-size: 0` on the parent `ol`


I have a patch that fixes this ([see here](https://gist.github.com/tpunt/2ef158dcc2863446149d76fabaefe98e)), but it does require an additional hash table to track the strings being allocated. What are your thoughts on this @krakjoe? Do you see a simpler solution?


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/6b8b7163-1ec8-4616-82fa-aa467a17af03 "storybook-toolchain-v2"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/b84aaac8-8484-4a1e-902a-ebb526eb7998 "storybook-toolchain-v2") deployed [carbon-components-react](https://github.com/carbon-design-system/carbon-components-react/releases/tag/deploy-ibm-yp-us-south-chrisdhanaraj_org-Bluemix-Design-System-20170803-153735 "deploy-ibm-yp-us-south-chrisdhanaraj_org-Bluemix-Design-System-20170803-153735") to [Bluemix Design System](https://console.bluemix.net/apps/7bb2892b-d170-4ca7-8346-d058542dbb8b?env_id=ibm:yp:us-south "ibm:yp:us-south:chrisdhanaraj_org:Bluemix Design System"), including this commit


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2053)*


Please add `official packages` into `lib/cleanup.rb` as well. The cleanup script should remove all Quarry packages that are in standard repository already.


You might want to consider changing this to the following to help those new to Cocoapods:
```ruby
pod `TransloaditKit`
```

Note that since the pod hasn't been published I had to point directly to git:
```ruby
pod 'TransloaditKit', git: 'https://github.com/transloadit/TransloaditKit.git"
```


Fixes #19


https://docs.gradle.org/3.3/userguide/maven_plugin.html#sub:multiple_artifacts_per_project
Here it allows having artifacts which are published with separate POMs and Maven coordinates in a single project (the pom setup below). Will add a comment in the code.


"Play video"?


Something


since YouTube videos take a while to load, perhaps add a `background-color` to the parent container? and ideally also use a still photo (could be the same as the YouTube poster image) too.


@destijl
Good point!   There might still be links to tuf-spec.txt on the web, so we can't simply delete it.  Perhaps we can include in tuf-spec.txt a link to the latest version (tuf-spec.md).

The other option is to keep both tuf-spec.md and tuf-spec.txt in sync, but that would be painful.


Ohh oops, thanks for catching that. Time to change my password!


Inspirit is not the best (intuitive) demo to show IMO. onboarding is difficult.


*주석처리 된 함수들 이름 앞에 On이 붙어야 함
ex) void OnEnable(){}


`0px` -> `0`


점프 가능한지 판정을 OnCollisionEnter하고 OnCollisionExit 두 부분에서 판단하는데
문제가 될 수 있는게 위의 함수들 각각은 '모든' 충돌체에 대해 적용되는 사항이라서 여러개의 충돌체하고 작용할 경우 오류가 있을 수 있음.

예를들어 player가 바닥으로 떨어지면  OnCollisionEnter가 한번 호출됨.
벽으로 달려가 부딛히면 또 한번 OnCollisionEnter가 호출 될 것임.
근데 만약 이상태에서 player가 벽에서 멀어지면 OnCollisionExit 함수가 호출되고
allowJump는 false가 뜰것임. 그런데 현재는 바닥에 붙어있는 상태임.

해결책: 바닥 오브젝트들에게 고유한 tag를 달아줘서 if문으로 걸러서 처리
ex)
```
void OnCollisionEnter2D(Collision other)
{
   if(other.gameObject.tag == "Ground")
   {
      //Do Something
   }
}
```
*다른 방법으로는 Linecast 사용하는 법도 있음.


This paragraph goes after the first code sample, not the second, correct?


Let's say I made a mistake while writing a test, and the `'Q1'` from my test accidentally ends in a real database. This already happened multiple times in the past

This can not happen with `null`.

It also can not happen if we have something like `new DummyItemId( 'Q1' )` that is guaranteed to raise an exception.


I'd use the CSS Font Loading API to ensure that there's no flicker on load


Yes, emailing now.


this works! thank you


Yes it does. It's a useful feature but it makes the grammar parsing much
more difficult :).

On Thu, Aug 3, 2017 at 8:27 AM cznic <notifications@github.com> wrote:

> That's interesting! However, I'm not sure: Does Wolfram
> language/Mathematica support implicit multiplication or not? I'm asking
> because I've just started playing with the grammar too ;-)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/corywalker/expreduce/commit/d584530069e5af147d9f385e587d1b965b3815f0#commitcomment-23460325>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAEaMlxFvrKe9EprYRiWSPBrvskqM-rZks5sUebbgaJpZM4OsG78>
> .
>


This is an important state change to you should log an event.


`parse()` does not replace insert tags and does not minify HTML :-(


Actually this was on purpose to allow prospecting with no titles


이거 짱인듯ㅋㅋㅋ


![](https://github.trello.services/images/mini-trello-icon.png) [Test](https://trello.com/c/fjTD2Cuv/6-test)


There's an initialization in parser_init_stream() that needs to change:
```
int allocbytes = sizeof(struct parser_context) + bytes;
```
From the code before your patch, it appears as if `bytes` includes the `struct visor_controlvm_parameters_header`, which you have added to `struct parser_context`.  So I think this can change to:
```
struct parser_ctx *ctx;
int allocbytes = (sizeof(*ctx) - sizeof(ctx->data)) + bytes;
void *mapping;
```
(I just substituted `ctx` for `struct parser_context` for brevity and readability.)


Missing tests:

* should deploy already active
* should correctly save owner on deployment
* should not accept to split on empty list
* should not accept to split with 0 value
* should not accept to split if not active
* should make a straight contribution when only 1 payee, with correct events
* should make a straight contribution when only 1 payee and remainder, with correct events
* should make contributions with 2 payees...
* should make contributions when 11 payees and value is 11*n + 1
* should make contributions when 11 payees and value is 11*n + 10
* should make contributions with 254 payees...
* should not accept 255 payees
* should be possible to make 2 contributions to the same list, the same value in the same block without overwriting.
* should not accept to claim if not active
* should not accept to claim if no such payment
* should not erase payment if payee is contract that cannot receive Ether
* should allow claim with proper parameters
* should have to submit 2 claims for 2 different payments
* should not be possible to halt if not owner
* should be possible to halt if owner
* should not be possible to resume if not owner
* should be possible to resume if owner
* should not be possible to kill if not owner
* should be possible to kill if owner, active or not, with owner receiving funds


I would like to message you about this tab since it is very similar to C9 pro. I would like to know if you want to work together and try to get these devices working well.


Can you fix the formatting here so that AddOpenIdConnect is on a different line.


Actually no. Prospecting without titles should be possible too


|`line 14`|
|---|
Perhaps we could just disable `jar` task as another option


I must admit that here I'm relying heavily on existing data-flow subsystem, i.e. if `DataFlowValueFactory` managed to create a `DataFlowValue` for this expression, then effect system should work with this value and try to infer something about it. I'm also relying on the fact that later we can safely pass this `DataFlowValue` to `DataFlowInfo` methods (such as `equate`, `disequate`, etc.).
Generally, in my mind, I think of `ESVariable` as of some r-value. Is it incorrect?

Also, do you think I should explicitly check here if this is indeed correct case (from effect system's point of view) and threw away all other?


Hmm... two MFXInit() calls. Can't be combined as one call and if init fails, maybe return a platform variable value of -1?


Couldn't [balance-text](https://github.com/adobe-webplatform/balance-text/) be used to implement it?


No new modifications. Closing branch


|`line 8`|
|---|
Why do we need java plugin if we do not have any sources?


[R/plotting-helpers.R:907:3:](https://github.com/PredictiveEcology/quickPlot/blob/38a8cbe1202b02c61e2a10a196354091511947eb/R/plotting-helpers.R#L907) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/plotting-other.R:163:3:](https://github.com/PredictiveEcology/quickPlot/blob/38a8cbe1202b02c61e2a10a196354091511947eb/R/plotting-other.R#L163) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:171:3:](https://github.com/PredictiveEcology/quickPlot/blob/38a8cbe1202b02c61e2a10a196354091511947eb/R/plotting-other.R#L171) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:175:3:](https://github.com/PredictiveEcology/quickPlot/blob/38a8cbe1202b02c61e2a10a196354091511947eb/R/plotting-other.R#L175) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:387:3:](https://github.com/PredictiveEcology/quickPlot/blob/38a8cbe1202b02c61e2a10a196354091511947eb/R/plotting-other.R#L387) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```


I think you've accidentally called the aws_region, aws_container_name!


`_logFrom` or `logFrom`.


Receipt hash could be `indexed`.


Great work Michal ! We just need to adjust the prospect thing with no titles and we should be good.
Thanks !


So "at date" is never possible??


Should be printing out a warning about the failed registration, It should be known that an item/etc. is not being registered because it was *removed* from the registry during the callback.


komentarze zostały jeszcze z ćwiczeń


Closes #5


Visto. Pode seguir na linha, como sugerido pelo professor Celson.
Mas precisamos acelerar o passo senão não acabas a tempo.


'promieniowanie całkowite


It adds the `compile` dependency configuration that is by default included into the published POMs as a `compile` dependency.


I know this is a real nit, but why not just remove the need for the `!`, and do it this way:
```
if (channel->needs_lock)
	return visorsignal_remove_lock(channel, queue, msg);
return signalremove_inner(channel, queue, msg);
```


Ah, well this may fix things in Safari since that's been a problem forever.

Perhaps it makes sense to show a message to the user so they realize the file has been saved?


commit.


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2054)*


I know this is a real nit, but why not just remove the need for the `!`, and do it this way:
```
if (channel->needs_lock)
	return visorsignal_insert_lock(channel, queue, msg);
return signalinsert_inner(channel, queue, msg);
```


I like it..
The reason it was done the other way was how I developed the patch and found the errors from smatch.


How is this a challenge? The sender knows its address and the `_secret` supplied. Surely the sender can also do `sha3(itself, sha3(_secret))` and put that into `_receiptId`...


This "etc" is unclear to me --- what other kinds of things do you mean? Will it be obvious to your audience? If not, consider rewording to something like, "how to handle `null`, and other web scripting edge cases."


If we didn't remove the invocation of unlock method when the buffer is empty or when read the EOF:
![image](https://user-images.githubusercontent.com/838924/28930680-4f95f8a6-78a6-11e7-8c6e-f3682f3afee9.png)

This will cause the IllegalMonitorStateException:
![image](https://user-images.githubusercontent.com/838924/28930746-804fed58-78a6-11e7-975a-a9e17f7bea78.png)
The reason is that we're calling the unlock method when we don't own the lock and this is because if we try to release the lock that we don't own.


![](https://github.trello.services/images/mini-trello-icon.png) [(.25) Update TestNG to the latest and remove the manually-defined JCommander and the bintray repo](https://trello.com/c/1Lz51aUb/116-25-update-testng-to-the-latest-and-remove-the-manually-defined-jcommander-and-the-bintray-repo)


You can consider a function that allows changing these values.


It will gain in readability by writing over multiple lines.


@JacquesCarette We can do something like this cf8ef8738db1aac22fb7f6b0bf2c6e9d1294a4ed. Here we would store the names as just the single character and then render the (single character) name with a period.


I did this first, but... if we call MFXInitEx requesting for platform 1.19, it's going to fail with earlier platforms, a return of 0 doesn't necessarily means there is no support.
but you're right it's still not ideal. let try something better.


Great. Thanks!


Empty commit just so heroku could re-build the app and thus update the node version


`type` and `ram_type` aren't defined anywhere and cause compilation to fail on linux


Naah. Make it simple and just take `uint txCost, uint txDelay`.


This can help. Because it is multi-protocol, it is easier to see changes.
https://github.com/Hydreon/Steadfast2/compare/ac96592c5b087cbdd7a1d69e91bffc97d2fc5ccb...master


old driver


Yes, it's best to mention this in https://github.com/JetBrains/kotlin/blob/master/license/README.md


You will need `return true;` to make sense.


Isn't it too early to mention Elm's `Maybe` here?
I was trying to introduce Elm at the end on the post.


@radko93 I've also noticed that codepush doesn't have the gradle classpath in this gradle file:
https://github.com/Microsoft/react-native-code-push/blob/master/android/app/build.gradle

Not sure if I could remove that from this one or if it would cause issues.


This fixed the larger test case I had as well. 👍


delete all


A pass may have existed in the past, in that case it is better not to reuse the same hash.


Why not just merge #3?


No need to make it `private`, it is a `constant` function and you are only giving out info.


Thanks!


You have to use Disqus Migration Tools before you change `disqusURL` here, otherwise you existing comments won't be loaded.


Fiddled with the comments a bit


Throws an error, but it does work for me.


done


@csicar 2017#2017 SS heißt das ist die Klausur von morgen? Oder wie soll man den Namen interpretieren? 😂


test


`notBeforeDate(_createdDate) notAfterDate(_validUntil)` is such a weak test. In effect, you are asking the caller to pass in fake parameters that will pass this require. You might as well not check anything.


Make sense. How about this?

```
...how to handle `null`, how to build safe web app.
```


Hi, thanks a lot for this. I was able to get it running in MS Visual Studio 2017.


Same here, this should PR should be reverted.


This magic `1000` does not look good.


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2057)*


It is more like a `require(`.


In the previous commit, we issued these lock/unlock invocations scattered throughout the code. It could end up being quite messy so our code could be error prone and frankly difficult to maintain. The reason is that we're not using locks in the recommended way.

The best practice is to enclose the critical section of code within a try finally block, one reason for doing so is that we only need to call the unlock method in one place, the other reason is that it would be possible for code within a critical section to throw an exception that we're not explicitly handling using a catch statement and we want to ensure that we have released any locks we're holding when that scenario plays out.


Lol. All of that material was cringy.


I never used direct keys, but I will play with this code and give feedback in whatever way
(display configuration ticket entered with Unify/Gupta/Whatever)


Thanks for the quick update!


The main reason for this library is missing: Imho it's to specify layout settings by code rather than using the Contao core page layout stuff which is handy when building platforms like we have a few. I think that should be written here somehow because if I'm new to this README I don't really get why I should use this bundle if versioning is the only thing it does :) There is some background missing :)


@karussell - The `services: docker` is a copy paste left over from a previous project, I'll remove it since I don't think it's needed.

And to answer your question - No, Docker images are not supported currently, but they are on our roadmap.


without `the`


`To create a template partial`


yaay!


`an asset`


`a new version`


`a versioning technique`


I see... you may try checking other mfxPlatform member variable values to get an indication if there is absolutely no MFX support present. This will be slightly tedious, but worth a shot in trying to remove the same thing called twice.


Исправление


nuevaclase


Yeah, I think I've botched all of this. Ugh.


Weird. Ships are down as galleys = x right?


Changed.


`with setting the cache control headers so that they only expire after 1 year (which is the longest caching time as per HTTP specification) which will...`


`the Symfony Cache component.`


`How does it work? `


`the Contao back end.`


Claiming an expired remittance should not trigger the same event as a correctly claimed one.
Your `doTransfer` also takes a lot of parameters. Too much mental overhead, that looks like an opportunity to lose track of what these parameters are supposed to be.


This seems to be a breaking change. An existing control I have (Share Workflow) now throws an error when trying to load the page.

`Method not found: 'Void Rock.Web.UI.Controls.FileUploader.add_FileUploaded(System.EventHandler)'.`


Feature request: `body_top` and `body_bottom` would be handy. E.g. Facebook's pixel stuff needs to be right after the opening `<body>` tag.


`as previously` can be removed here.


`For better cache control`


`expire headers`


The specs was that there would be 2 passwords, and no check on the recipient.


Acctually, is pretty straight forward: you have to write the module's names (package names) of the files you wnat to load as DLLs in the settings


ok.
I'm curios where inside JavaPlugin that magic of converting `runtime` configuration to pom dependencies happens? So that if we stop applying java plugin from our js plugin, what should we do to provide the same publishing experience?


`an example`


I didn't remove it - I ADDED the "not to exceed 20 octets".  +1

We had it in the other profiles but not consistently across all.   I believe I also added similar verbiage to Section 7 directly.  When this merge goes in, the master will be updated consistently: https://github.com/uspki/policies/pull/145/files


`the versioning system`


`back end`


You cannot easily botch comments. As long as you don't delete them, they will always be in Disqus. The question is what URLs they are mapped to. You can always use Migration Tools to change URLs in a batch.


[R/plotting-helpers.R:907:3:](https://github.com/PredictiveEcology/quickPlot/blob/5a8bccfe3744c2af99918428989badf7bdd5cd00/R/plotting-helpers.R#L907) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/plotting-other.R:163:3:](https://github.com/PredictiveEcology/quickPlot/blob/5a8bccfe3744c2af99918428989badf7bdd5cd00/R/plotting-other.R#L163) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:171:3:](https://github.com/PredictiveEcology/quickPlot/blob/5a8bccfe3744c2af99918428989badf7bdd5cd00/R/plotting-other.R#L171) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:175:3:](https://github.com/PredictiveEcology/quickPlot/blob/5a8bccfe3744c2af99918428989badf7bdd5cd00/R/plotting-other.R#L175) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:387:3:](https://github.com/PredictiveEcology/quickPlot/blob/5a8bccfe3744c2af99918428989badf7bdd5cd00/R/plotting-other.R#L387) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```


in groovy it's rather idiomatic


updated


Don't know why, but the semicolon is still not there.


More like `require(`.


This was just so it would match the area joined message from local chat as well. Probably isn't needed anymore and could be replaced with the 2nd regex.
`areaRegexStr := ("^(?:[^ ]+ ){6}(\d+)\] : (.*?) (?:has) (joined|left) (?:the area.*)") `


Ah, good thing. I'll let it all sit for a bit, and will read and try to actually understand before delving back in. Thanks, as always, for your help!


are those packages that are 3rd party, how do you decide which to put there?


api version update needed?


Forgot to document changes to eval_client in commit message:

Prevent the GC from collecting EvalClient if the game manages to remove all other references to EvalClient.

Add connection acknowledgement.

Track more than one frame job.

Don't unhook Yield (allows eval_client to function in game).


It would be close, but not quite. Smart Wrapping (wrap style 0) tries the make the lines equal length, but ensures lines are never longer than the line above it. Smart Wrapping (wrap style 3) does the same, but shorter lines are on top. balance-text doesn't make those assurances.


camelCase


no, because ALPHA7 hasn't been tagged yet.


syntax error


Why the minimum holding period?
It adds complexity so there needs to be a good reason.


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/b09e3ee0-bc62-4bd7-bcf6-7b033dada6af "TriAzurUE"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/57bc70fc-3a85-4eb7-a477-d1a67ac25fb2 "TriAzurUE") deployed [TriAzur_code](https://github.com/Doncassino/TriAzur_code/releases/tag/deploy-ibm-yp-eu-gb-TriAzur-TriAzur-20170803-161739 "deploy-ibm-yp-eu-gb-TriAzur-TriAzur-20170803-161739") to [TriAzur](https://console.bluemix.net/apps/11b7850a-b04f-4546-975c-fb2662c615cd?env_id=ibm:yp:eu-gb "ibm:yp:eu-gb:TriAzur:TriAzur"), including this commit


This checklist is getting long, we really want a way to automate this, lol.


Yeah I gotta redo these tests. Stuff like `Collector` aren't actually exports of discord.js, but somehow the TypeScript libdef is still able to `import` them 🤷♂


And that's why you shouldnt be using it on master @Sandertv


hahahahaha


Why would it need to? That's done when the page is sent to the browser?


It would be simpler if the person creating it would just give either a timestamp set in the future, or a duration in seconds. And this function just takes it in, without convoluted calculations.


Testing


I don't believe this logic is correct. If a file is git rm'ed in the source content, the logic as written won't cause the delete to ripple through to the destination.


There does not appear to be a way for the owner to cash out the `txCost`, other than killing the contract, which also cashes out the current remittances...


did it this way: https://github.com/ph0b/gstreamer-media-SDK/commit/4c06db187898460d400eacb9f835c36861bf7de4


The transfer will fail and revert if that is not the case. You can probably dispense with this check.


Yes, you'll usually list as a DLL the 3rd packages you use. Here, read this post which has been very helpfull to me
_

> "For example, suppose you have an application built with several large libraries or frameworks such as jQuery, Angular or React which change relatively infrequently, plus some utility code such as underscore or lodash which you change very rarely. The Dll plugins enable you to put the utility code in one bundle, the frameworks in another and your application code in a separate one."

_https://robertknight.github.io/posts/webpack-dll-plugins/


@lassoan would it make sense to add a constructor that takes the three items? Would be quite convenient I think, instead of instantiating and then calling `SetSchemeValueMeaning`.


FWIW this broke Shopify's sharding code too :-)


A thread can actually test to see whether a lock is available using the try lock method, so if the lock is available the thread will acquire the lock and continue executing if it’s not available, the thread won’t block and alternatively can execute some different code.


Shouldn't be using what on master? I just update according the master branch at all times which works fine. This change is not considerable for backwards compatibility of plugins.


It means that only if that returns true (a lock is available), then we get the lock automatically then we’ll proceed to go through and executing the code.


@gshriki will return an error if no proxy.
change to demisto.get(demisto.params(),"proxy")


Make it a verb: `setContractSettings`.


If you are using the `latest` tag, chaning the pull policy won't do anything

https://kubernetes.io/docs/concepts/containers/images/
> The default pull policy is IfNotPresent which causes the Kubelet to not pull an image if it already exists. If you would like to always force a pull you must set a pull image policy of Always or specify a :latest tag on your image.
> If you did not specify tag of your image, it will be assumed as :latest, with pull image policy of Always correspondingly.
> Note that you should avoid using :latest tag, see Best Practices for Configuration for more information.


Make it a verb: `createCredit...`.


I would find it better to [omit this information also as a TeX comment](https://github.com/coccinelle/coccinelle/issues/109 "Delete a reference for a wiki in the manual").
Would you like to end the transformed sentence with a dot?


Oh wait is ALPHA7 on master in PM?

@Sandertv i was refering to you using ALPHA7 on masger in BlockHorizons projects


Here too, there is no way to cash out what came in. Unless there is a point to it, you may want to close deposits through the fallback.


Much better! Line 313, the check is needed... seriously?


Yes it is.


This modifier hides some calculation that is not apparent in its name.


This contract is very complex and not easy to read. That makes it dangerous for potential bugs to lurk and be difficult to detect.


@JonathanMeler proxy has a default value, so this shouldnt be a problem


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2058)*


@AlexPeshkoff I think these parameters should not be different in debug build, as they will cause debug built to be more limitted. I think it would be for the developer to adjust these locally to do tests.


good to know. I'll use it.


yep. and checked it with both states of the checkbox...


I suppose the idea was to evaluate _each_ of these properties lazily


Well, a `sequenceOf`/`listOf` of lazies will do, I suppose.


@fingolfin could you please make a release when possible? The current release uses `RecFields` and this now causes much noise in `make testobsoletes`.


I already feel uncomfortable with this array.


I'm not sure. I would expect the call to return MFX_ERR_UNSUPPORTED if the runtime is below 1.19.
but it's not documented this way and I don't have a runtime <1.19 to test today.
so I've chosen to keep it safe this way at least for now :)


Probably add a verb in the past tense?


@theHellDog @Praffesor стоит ли добавить обработку errors?


+코루틴 함수 공부 ㄱㄱ
코루틴 함수는 쉽게말해서 싱글스레드를 멀티스레드처럼 보이게 하는 효과임.
예를들어
```
void Update()
{
   float t = 0;
   if(Input.GetKeyDown(KeyCode.Y))
      while(!Input.GetKey(KeyCode.X)) ;
}
```
이런코드가 있으면 while 문 안에 갇혀서 프레임이 멈춰버림(싱글스레드의 비애)
이런 불편함을 보완해주는게 코루틴 함수.
update와는 별개로 도는 루프를 생성한다는 정도로 생각하면 됨.(사실은 눈속임)

주로 쓰이는 곳
* 아이템 사용 후 딜레이 구현할 때
* 죽고나서 리스폰 딜레이 구현할 때
* 특정 시간 또는 특정 수치만큼 오브젝트를 연속적으로 움직이거나 변형하고 싶을 때 등등

Update함수보다 효율적이라는 소문도 있던데 그건 잘 모르겠고

자세한건 구글에 '유니티 코루틴 함수' 검색 ㄱㄱ


Oh, sorry, maybe I commented on the wrong commit
Spell checking in OCR, when you permit change in every line, is not getting the "؟،" and the others are pancuactions…
It's considering them wrong


Poor man's debugging because Samsung Internet doesn't ship dev tools.


后端基础技术以及设施


It should be entirely possible to access VARIABLE, in fact that is already done for INDEX_DIR. That being said, function > variable for something like this, I guess...

But all this needs is an array of packages, not [package, slot]'s. I think. Or is this to match your comment about the pkgame --> key? I *think* that would result in building updated versions of outdated official packages, which is something that probably isn't optimal...


hm..


Ok sure. Will test the changes later on my Linux setup, mine is 1.17 for that.


On Thu, 3 Aug 2017, Markus Elfring wrote:

>
> I would find it better to omit this information also as a TeX comment.

And why would that be?

Maybe we will set up the wiki again some day, and then the comment can be
removed. Otherwise the connection wille forgotten.

> Would you like to end the transformed sentence with a dot?

No.  Not putting a dot after a file name makes things easier for someone
who wants to copy and paste it.

julia

>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the
> thread.[AAesml6noZayEA3PTrO22xnauaKgu2cXks5sUfOwgaJpZM4OsrYL.gif]
>
>
>


You actually had the order of `}` and `)` correct before! 😄

I used Chrome's Developer Tools on the index page to look at the Console, and I saw this message:

```
Uncaught SyntaxError: Unexpected token )
```


TeamCity OTA+ :: libats :: ci [Build 109](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=39905&buildTypeId=libats_Ci) is now running


Thank you! @chewydean


@Itileda typo


Take a look at your scanner element's HTML code here: https://github.com/Double-0-Se7en/ConnectPear/blob/77b83313e671aa7e84185ff8b0dbb76d3df100c5/index.html#L107

Is "scanner" an HTML ID or class? Are you trying to select a class or and ID with jQuery above?


thanks


You probably don't need `onlyOpen` on this `private` function.


I am so sorry my boyfriend installed a bunch of spy apps on my phone and I
was mad... I didn't u understand how this stuff works.. sorry to take it
out on you dan..

On Jul 25, 2017 1:23 PM, "junkie2100" <notifications@github.com> wrote:

> sorry to be the bearer of bad news meg but i have no idea who you are...
> also there shouldnt be any code in here that invades someones privacy,
> certainly none that i put in, this particular commit is actually just basic
> configs. also my name is dan and im pretty sure im the only one who gets
> this message so im not sure who youre trying to reach but yelling at me
> isnt helping anything lol
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/junkie2100/android_device_zte_warp3/commit/b441526e4112f9111b025b963f8fe0204cc79d14#commitcomment-23293076>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/Ac05VFQPYdWs6d6Dnn1eoPeTPt5u-n0Uks5sRiSUgaJpZM4OioZd>
> .
>


done


`next if ...` should be a style change that affects the for loop which cycles through dependencies and decides whether to add them to the list of pkgnames. This is a separate check for the pkgname itself, which ISTR was to prevent accidentally whitelisting an official package.


After you delete this `_orderId`, then `onlyUnpaid` will become true again.


TeamCity OTA+ :: libats :: ci [Build 109](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=39905&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Tests passed: 4 Build time: 00:06:02


@sleske OK will remove this branch.
Ping @pgrandin


"linted" instead of "lint"?


It's another one; fixed in https://github.com/Guite/MostGenerator/commit/1885239abfce46f38d7d0be5a08f5f10fa73fb1a


What's different about the selector used here? https://github.com/Double-0-Se7en/ConnectPear/blob/master/ratings.html#L66


does not compile


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2061)*


test


I don't think you added files needed for web management to the repository when you did the commit.


Might want to make the existence of the startup_hook and exit_hook functions in the hook scripts be optional. Call them if they exist but don't error out when they don't.
This change breaks the current google_cloud_hook (at least). It lacks these two methods.

They're not hard to add.


![](https://github.trello.services/images/mini-trello-icon.png) [Staubablagerung auf Modellen](https://trello.com/c/jrJNh7D8/52-staubablagerung-auf-modellen)


:D :D


![](https://github.trello.services/images/mini-trello-icon.png) [Staub Ablagerung](https://trello.com/c/ip9udRVB/35-staub-ablagerung)


This function exists for purely aesthetic reasons: https://github.com/expo/harvard-cs50-app/commit/3dd4866aadbd7ee068de7e4563b44241d00a1bec I like using modular scale better than arbitrarily picking font sizes that "look right".


![](https://github.trello.services/images/mini-trello-icon.png) [Staub Aufwirbelung](https://trello.com/c/IemHs8oK/36-staub-aufwirbelung)


There's an existing regex you can use (might want to move it into here as it kind of belongs here anyway) - https://github.com/hashicorp/consul/blob/master/agent/agent.go#L54-L55.


Oh yeah, seems completely logic now.
Alright, thank you!


@adrian-prantl nice to see work on `dsymutil` over here! Do you by any chance have a sense for how long the Apple `dsymutil` changes will take to back their way back to upstream LLVM?


More like `getProduct`.


Strange there can be only 1 merchant, and yet you use a `mapping`.


your tests seem to even pass against ReadableNumbers 0.0.2 (via Pkg.pin), though maybe you're not covering whatever the difference would influence?


compiled by running : g++ "main.cpp" -o CosmosQuest -march=native -O2 -s


The code in this commit is the closest you've gotten! That selector string `$("scanner")` has some issues though, if you're trying to select `<button id="scanner">`…


Very bad. A log changes the state of the blockchain but this is a `constant` function. So the expectation is that this function does not change the state of the blockchain, and yet it does. No `Log` in a `constant`.


why??


Oh and better make one event per action type. Not all action types may require the same number of parameters.
Plus, using a `string` for an action type is about the most expensive you can find.


Ok.  I will revert for now since I use direct keys.  Later I suspect I will come up with a patch.

> On Aug 3, 2017, at 9:05 AM, H.Merijn Brand <notifications@github.com> wrote:
>
> I never used direct keys, but I will play with this code and give feedback in whatever way
> (display configuration ticket entered with Unify/Gupta/Whatever)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


`h` is used in signature, but description was removed:

```diff
@@ -183,16 +183,9 @@ std::string const& AqlFunctionFeature::getOperatorName(
 //
 // . = argument of any type (except collection)
 // c = collection name, will be converted into list with documents
-// h = collection name, will be converted into string```


so im good?


Hi, what is the end point of the rest service.
I'm trying to access the service using localhost:8080/users/all and looks like I'm something in the uri.


lol


**Whoops!**

There were *2 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/fundamentals/native-hardware/capturing-images/index.md`](https://github.com/Google/WebFundamentals/blame/a8207d13691b949ad406534312fff447ee88fd31/src/content/en/fundamentals/native-hardware/capturing-images/index.md) - No `wf_blink_components` field found in metadata. Add if appropriate.
[`src/content/en/fundamentals/native-hardware/capturing-images/index.md#L107`](https://github.com/Google/WebFundamentals/blame/a8207d13691b949ad406534312fff447ee88fd31/src/content/en/fundamentals/native-hardware/capturing-images/index.md#L107) - Line must be at most 100 characters


*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/2062)*


You really should have benchmarks. If you did you might have noticed that there was  a reason the original used an explicit index loop instead of `range`. The later recognises UTF8 encoding and does extra work that is explicitly not needed/wanted here. The same benchmark mentioned on the old reddit comment shows that switching to range adds ~70% to the run-time (with a ~1500 byte string).


pokaże "undefined"
Nie wiem czy o to chodziło ale zmieniłem , żeby pokazywało "false"


@marta- Which JSON is line 4398 referring to? It looks like I need to allow the JSON to accept the new labels I created, but not sure where to look.


**Whoops!**

There were *1 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/fundamentals/native-hardware/capturing-images/index.md#L107`](https://github.com/Google/WebFundamentals/blame/a2bfcc81cd28175bf239370493a11623c2bd7395/src/content/en/fundamentals/native-hardware/capturing-images/index.md#L107) - Line must be at most 100 characters


You will reduce complexity by having 1 function for straight purchase, and another for copays.
A straight purchase needs 2 transactions?


@cdutra found another release we're relying on. Using this for the short-term, but may need to add another task input which would affect your pipeline as well.


That means the same person cannot order the same object twice in the same block.


Should be Patch Version 3. It will be fixed in new version.


Did you test this?


I set it to false,  but still getting errors of noImplicitAny


The function name is misleading when it actually decreases stock by 1.


tbh, i am not sure why there is a need to close the connection synchronously. But in the end, we should go away from the `async: false` approach as it blocks the current tabs main thread


@madoar we should find a common naming pattern when new scripts are committed to make the commit history more clear. I propose to use the application name only.
Currently it's a bit messy because we have commit messages like:
* Add game Caesar III
* New program 7-zip
* Star Trek Online


[tests/testthat/test_cleanup.R:4:1:](https://github.com/steinbaugh/basejump/blob/d23915fe59332e92a1a263d05a7693ec9bd76d1d/tests/testthat/test_cleanup.R#L4) *style:* **Variable or function name should be snake_case.**
```r
namedVec <- c(Item.A = "hello world", Item.B = "HELLO WORLD")
^~~~~~~~
```


Wow, youre killin it with finding these issues. Looks like it has been fixed upstream, but it'll take a bit of work to backport the changes


@guilleiguaran

Might this (or something similar) be backported?

I'm not familiar with how that works in Rails...


Hmm, you're right. What pattern do you propose?


Probably should be checked on placing the order too.


"Close" would still be better that the current situation. Can you open an issue so that this doesn't get lost and someone can make a PR?


Looks good to me


You should probably reorder your functions so that they appear in a natural way for the reader.
Here I see right away `payOrder`. It is only later that I know about `addProduct` or `addOrder`.


Should both "don't"s be here?


1. Директория является входным параметром для скрипта - логично её передавать через командную строку. Т.е. чтобы можно было вызывать скрипт так:


Ibid


So many notes to the developer just goes to show that this function is too complex.


Super Smash Bros Brawl (Wii) Replay File


I really do want empty list of parameters to be passed explicitly, because it's easy to forget that *sometimes* you have to specify parameters, and such bugs can be a bit hard to track. In fact, I've made one or two such mistakes, and it wasn't too fast to pin down the issue (especially given how stupid this bug is).


2. Сколько-нибудь длинные комментарии должны быть на отдельной строчке; в случае комментариев на той же строчке отступ должен быть в 2 пробела, т.е.:
```python
# комментарий здорового человека
dir = '...'

dir = '...'  # комментарий курильщика
```


Only bug fixes are backported.


As I said above: I think only application name is the best solution.
So:
* Caesar III
* 7-zip
* Star Trek Online


for uniformity's sake, maybe drop the v prefix when you next tag this


I think we should also add some kind of keyword to indicate that a script is added.


Hi,

This line change introduces a bug where the style is not applied at all. The apostrophe is somehow messing up the style( at least on Safari).

This is your name so it is not my place to make a PR. Maybe an issue should be raised with Stylish( an apostrophe in a comment should not break anything).

Either way, thank you for the awesome css!


[R/plotting-helpers.R:907:3:](https://github.com/PredictiveEcology/quickPlot/blob/234b5fb6a0ce98ef78c20e183060f380cb407223/R/plotting-helpers.R#L907) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/plotting-other.R:163:3:](https://github.com/PredictiveEcology/quickPlot/blob/234b5fb6a0ce98ef78c20e183060f380cb407223/R/plotting-other.R#L163) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:171:3:](https://github.com/PredictiveEcology/quickPlot/blob/234b5fb6a0ce98ef78c20e183060f380cb407223/R/plotting-other.R#L171) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:175:3:](https://github.com/PredictiveEcology/quickPlot/blob/234b5fb6a0ce98ef78c20e183060f380cb407223/R/plotting-other.R#L175) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^~~
```
[R/plotting-other.R:387:3:](https://github.com/PredictiveEcology/quickPlot/blob/234b5fb6a0ce98ef78c20e183060f380cb407223/R/plotting-other.R#L387) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```


Woohoo! This release also fixes crossing the poles too, but that's probably less of an issue than the dateline bit.

Also, I have a version that exposes the cpp interface so other packages can link it but I'm not quite happy with it AND I just started a vacation so it'll probably be a couple of weeks.

Anyway, thanks for letting me be part of this!


part 2, rather. *shrug*


Ok. How about "added" ;)
Like:
* added 7-zip


@shawnshaligram new release tomorrow.


Looks good to me


@hex007 What effect does this have?


@hex007 What's the effect of this?


#6028-PTAC-heating-coil-undersized-for-ventilation-load-when-zone-design-heating-load-is-zero (rraustad) - i386-Windows-7-VisualStudio-14: OK (2576 of 2577 tests passed, 0 test warnings)

Failures:

EnergyPlusFixture Test Summary
 * Passed: 498
 * 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)


same issue, this isn't known at compile time. we should include it if it might be used, probably it should be behind SINGLE_FILE?


Sounds good, what should we do when a script is updated?
Simply use `Updated 7-zip`?