Trying to install `preact-cli`, which depends `"preact": "^8.1.0"`. No success because of

```
> preact@8.2.2 post install /app/node_modules/preact
> node ./config/donation-message.js

module.js:491
    throw err;
    ^

Error: Cannot find module '/app/node_modules/preact/config/donation-message.js'
    at Function.Module._resolveFilename (module.js:489:15)
    at Function.Module._load (module.js:439:25)
    at Function.Module.runMain (module.js:609:10)
    at startup (bootstrap_node.js:158:16)
    at bootstrap_node.js:598:3
```


Do we still need rmi?


No, just use {}.


Thank you very much for the answer. Please tell me where you can find slightly older version?


Or `balances[msg.sender] += extra;`, depending on specs.

Trying to uncover unconscious choices.


register, deregister recordAdhocTicketExit
author:Tien Huynh
reviewer:Satwant
moderator: Iqbal


Ah OK. Bon, si api-adresse.data.gouv.fr est supide au point de retourner une liste pour une seule addrese, explique-le dans un commentaire s’il te plaît :upside_down_face:


Hi again! I'm am on LineageOS. I found [THIS patch](https://github.com/LineageOS/android_device_motorola_griffin/commit/7710f7443906f8c5031b0afce5fa75ca6acffb41#diff-813d8fba57c8900923f28c4cacd11bd2) and now it works perfectly fine!
Thanks a lot for the help!


Anti-pattern. Make your state changes before contacting another address. Avoid re-entrancy, avoid TheDAO.


What were you thinking, past Jessica??


develop (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2158 of 2158 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 may be misreading the code, but with this change it seems you're introducing a distinction between an element's having hild nodes and its having attributes. But attributes are child nodes too - just attribute nodes. Or is there a difference from the perspective of the DOM? (Or am I misinterpreting?)


Hello. This solved my immediate issue, so first of all - thank you. I've been thinking (🤕) and since I'm a beginner at Nix and NixOS - perhaps it's just my ignorance. If so please forgive me. To the point:

What if after installing Atom user upgrades Git and then collects garbage (`nix-collect-garbage`)? Wouldn't it leave a symlink pointing to non-existent file? Or does Nix take care of that?

Also - and it's a rather theoretical point - what if Git introduces a breaking change to it's CLI? Sticking with `git` binary shipped with Atom (and patching it) would keep a responsibility for that on Atom's developers. If we symlink, it seems to get non-determinable.

Again, maybe I just don't understand how it all works. I'm always happy to get schooled :)


I would `var half = 25;`. Just to be safe.


I would `var extra = 1;`


You should check the event in here.


Omg... what a mistake. I get it now, the if is wrong. It should be: `$fieldDescription->getLabel() === false`

In fact, this does apply when the label === false or when label is missing (null)?


Change this sync call into async.


@EmilyyC This will affect all themes. Also, I believe this hides all the blocks. We just want the UCLA course menu to not display content if it is in the Snap theme. See https://github.com/ucla/moodle/blob/master/blocks/ucla_course_menu/block_ucla_course_menu.php#L110


`.strictEqual(expected.toString(10), ...)`


Wait, let me think about it...


@abhishekjain88 could you please also verify that complete integration tests are working with this change?


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2138 of 2138 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)


You disclosed the passwords in the create phase. No longer private, no longer a challenge.


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

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


Here, you're detecting the truncation issue but you're not accounting for the extra funds.

consider

`balances[msg.sender] ++; // to return the change`


Aïe, je suis certain que ce n’est pas la bonne manière de faire. Ballader `sys.argv` dans les vues de Django dans une application web qui peut devenir pas très petite, ça peut causer des problèmes :upside_down_face:

De plus, en général il faut mieux utiliser des mocks en faisant de l’injection de dépendance à la Java ou monkey-patch  les choses à la Ruby pour avoir des comportements différents entre les tests et la production. Tu vois le délire ?


Anti-pattern, don't change the state after an interaction with external contract.


I can make a template properties file, which will have one of each service commented out including all parameters and copy it there. Also wile i was making the scripted i noticed that the logo is not included inside the compilation of the .jar and it is a folder up from where sw.sh/bat is executing. Should we put it inside the jar ?


Also beware `delete` does not do what you think it does if you struct has a mapping or an array.


s/hild/child


I wouldn't hurt to check that payee1 and payee2 are not empty. Oddly enough, you can check `payee1 == 0` or the more explicit `payee1 == address{0);`


Почему он лучше push ?


Revert the transform to .Result and update pattern matching appropriately.


Presumably, the creator need not give passwords. Being the creator should be enough.


Hey guys, sorry for long respond time in general, i m on vacation since 2 weeks :-).
About review process, and to it is fine as it is, as i already said it is your baby :-) and thats why i accept everything you suggest at least in most of the cases if it is not very weird :-)
"Please take all of our comments in good faith" : -I do guys of cause i do :-) I m always willing to lear stuff so go on :-) I have to do it myself at my work almost everyday :-)
"...It's not like we are infallible either - three of the commits I've got lined up for the next.." - I ll try my best to find enough time to react fast, but it is a bit difficult for me with 2 little kids and work and wife :-(  or :-)


Can we move these command templates to a designated class? We've discussed a few potentially useful template classes: ExogeniMngtCommands, RyuCommands, DockerCommands, and SafeClientCommands.


It is a fair point, in something so simple I don't see it as an issue, but I see the potential harm this pattern can cause in a bigger project with more moving parts.


Consider doing `keccak256(` only once.


Can we put what inside the loop in a separate method and call that method here?


I didn't like this myself to be honest, the only thing I was trying to keep in mind was what if one of the recipients doesn't have any eth of their own in order to call the withdraw function.


Since we create *num* slices here, the name of this method should better be *createCustomerSlices*.


https://github.com/sonata-project/SonataAdminBundle/blob/3.x/Datagrid/ListMapper.php#L120

You are right, it was wrong, but already fixed by: https://github.com/sonata-project/SonataAdminBundle/pull/4602


Add `if (results.size() == 0) {continue;}`. We don't want the whole thing to throw an exception just because the consent configuration may be a bit outdated.


@rnunez80 you need to sign Plone Contributor Agreement in order to merge this pull request.

Learn about the Plone Contributor Agreement: http://docs.plone.org/develop/coredev/docs/contributors_agreement_explained.html


Quirk


commands issue again.


This ifs is needed here (before the Engine's ProcessMessage call) to set up the EPD ID according to topic, from what we received the message 1; and 3; because Engine is not processing topics by itself and will do all actions on the last used EPD ID - possibly the bug of the Engine.
Also simulation is done here too


this was copied straight from the solidity repo.  I feel like the lowercase contract name denotes that it is meant to be used by other contracts and not on its own.


i think we should also check that the parameter `enabled` is not `false`.


Indeed. In the DOM model, attributes are not considered child nodes.


Only 1 pending remittance per exchange?


I think it would be cleaner if you defined a compatible `randbits` in the the exception handler:

```
try:
    from secrets import randbits
except ImportError:
    import random
    randbits = random.SystemRandom().getrandbits
   # Or, if you want to use a new SystemRandom every time:
   # randbits = lambda nb: random.SystemRandom().getrandbits(nb)
```

Not only is this less code, but I suspect there is a (probably negligible) performance improvement by eliminating run-time comparisons for data known at import-time.

(Sorry for the drive-by critique.)


Again, copied from solidity standard library: https://github.com/ethereum/solidity/blob/develop/std/owned.sol


`# street`


The earlier version was more appropriate, because if we need to change the key, we don't need to change the hash, instead, we just change the environment variables.


AFN2017 (jasondegraw) - x86_64-MacOS-10.10-clang: Build Failed

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


`require(`


moving `donation-message.js` to root folder should do the work


`if ((icmpRec->icmp6_type == ICMP6_ECHO_REPLY) && (ntohs(icmpRec->icmp6_id) == pid) && (ntohs(icmpRec->icmp6_seq) == (seq - 1))) {
cout << "code " << ntohs(icmpRec->icmp6_code) << endl;
cout << "ICMP ECHO REPLY" << endl;
minBuff = lengthBuff;

}`


How do you prevent disclosed password reuse?


Thank you, but the lack of a 32-bit dll results in an error message displaying any time a 32-bit application is opened, such as Spotify.

```
Error code 0x00000002: The system cannot find the file specified.
Routing functionality couldn't be initialized in the starting process.
```


Change this sync call to async.


Since it is basically %2 the only possible value besides 0 is 1.  also half is guaranteed to be the largest multiple of 2 for msg.value so I thought it made sense to use that instead of 2. (9-4-4) has to be better than (9-2-2-2-2).


You cannot add gas to wei. You can add gas * gasPrice to wei.


Nothing


altera essa senha mano brow xD


TeamCity Code :: 1. Development :: DDK :: Feature [Build 437](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=29574&buildTypeId=Code_Development_Ddk_Feature) is now running


It's not necessary per se, I guess I'm just used to always looping where I can.


You could check addresses are not 0.


@afshinm Something about this change (or another to this file in the last 24 hours) breaks onbeforechange. Reverting to the older version I downloaded last night restores the functionality.

Using an alert box, it appears that targetElement is now always undefined when calling the onbeforechange function, where before it was only undefined on the first step.

I'm using the targetElement ID in an if statement to control the opening/closing of a collapsible menu. I have to verify which step is next before opening or closing the menu.  The option to start the tour is actually in the menu itself, so if targetElement === undefined, the menu will be closed by javascript before the tour begins. Once the tour gets to the menu, it will automagically open and close at the appropriate times during the tour.


This was abandoned in favor of the SimpleSplitter.sol contract


TeamCity Code :: 1. Development :: DDK :: Feature [Build 437](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=29574&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Tests passed: 70 Build time: 00:01:33


Ok, great then ! ;)


good point, I was originally doing the math in the child contract and then made these methods


jjajjaja no funcionaba


fixed in 703e048.


Did not catch this one earlier, but these sync calls are ugly. Use `module.exports = function(deployer, network, accounts)` instead.


not my usual patter, I think I posted the link and then got in a cycle of finding things I wanted to change and would do it as fast as possible and then once I pushed I would find something else


вызывается эта страница из формы в  tickets.phtml
код:
...
<input type="hidden" name="ticket_id" value='.$block->getNextFieldFromCollection('ticket', 'ticket_id').' />
...
На странице tickets пользователь только нажимает на кнопку(кнопка содержится в форме с методом пост) нужного ему тикета, а уже в этой форме имеется hidden input, где сохранен ид этого тикета. Если я правильно понимаю, то пользователь не имеет доступа к полю ticket_id


About removing the fallback cases, I personally think it is not very good way because the user that have no backround knowlege will not try to activate any logs or something like that at least the most of them. My experience with customer tells me that they just remove everything or will ask for help in best case. - Just a side note :-)
But as you said the migoto does not try to do it in other cases, thats why that is something that breaks out of general rules and would be kind of unexpected. So maybe you are right and we should remove the fallback.
 I ll do that as soon as am at home again, so in 3 days. I ll also start to work on the sampler feature.


Using this dev branch, how exactly do we change the route?

I am using the dev branch, but I can no longer use browserHistory.push . I have tried withRouter however, it does not work either.

Can you share some example how to programmatically change route? (I know the sample app uses <Link> but I am looking for programmatically way)

Thanks!


I think in my mind I saw these as utility methods to be used by others and that the event should come from those methods using it. i.e. Withdraw and Split


Here are the diffs from intro.js today and the one I downloaded last night (/usr/local/src/node_modules/intro.js):

```
> diff -u intro.js /usr/local/src/node_modules/intro.js/intro.js
> --- intro.js    2017-08-24 11:41:06.250999257 -0600
> +++ /usr/local/src/node_modules/intro.js/intro.js       2017-08-18 05:01:33.000000000 -0600
> @@ -368,16 +368,6 @@
>        ++this._currentStep;
>      }
>
> -    if (typeof (this._introBeforeChangeCallback) !== 'undefined') {
> -      var continueStep = this._introBeforeChangeCallback.call(this);
> -    }
> -
> -    // if `onbeforechange` returned `false`, stop displaying the element
> -    if (continueStep === false) {
> -      --this._currentStep;
> -      return false;
> -    }
> -
>      if ((this._introItems.length) <= this._currentStep) {
>        //end of the intro
>        //check if any callback is defined
> @@ -389,6 +379,10 @@
>      }
>
>      var nextStep = this._introItems[this._currentStep];
> +    if (typeof (this._introBeforeChangeCallback) !== 'undefined') {
> +      this._introBeforeChangeCallback.call(this, nextStep.element);
> +    }
> +
>      _showElement.call(this, nextStep);
>    }
>
> @@ -405,19 +399,11 @@
>        return false;
>      }
> -    --this._currentStep;
> -
> +    var nextStep = this._introItems[--this._currentStep];
>      if (typeof (this._introBeforeChangeCallback) !== 'undefined') {
> -      var continueStep = this._introBeforeChangeCallback.call(this);
> +      this._introBeforeChangeCallback.call(this, nextStep.element);
>      }
>
> -    // if `onbeforechange` returned `false`, stop displaying the element
> -    if (continueStep === false) {
> -      ++this._currentStep;
> -      return false;
> -    }
> -
> -    var nextStep = this._introItems[this._currentStep];
>      _showElement.call(this, nextStep);
>    }
>
> @@ -429,7 +415,6 @@
>      // re-align intros
>      _setHelperLayerPosition.call(this, document.querySelector('.introjs-helperLayer'));
>      _setHelperLayerPosition.call(this, document.querySelector('.introjs-tooltipReferenceLayer'));
> -    _setHelperLayerPosition.call(this, document.querySelector('.introjs-disableInteraction'));
>
>      // re-align tooltip
>      if(this._currentStep !== undefined && this._currentStep !== null) {
```


In the `div`, the second ampersand should be escaped.


Adding theme


@fredma @PeterSzegedi  got that bug - as it's been bugging me and i'm sure 247 will ask to reverse the direction of the sync ! I actually wrote the failing test , fixed the bug in debug once i got a real example to use and then n test and it passes ! thanks for @helena-patching  as i was also thinking of an ignore  for a certain user but that seemed to much of an assumption...


I think I meant this as a guard against a negative value.  While none of my uses will result in a negative value I think I saw a possibility to send in an amount that was greater than then balance.


I moved to doing this with the SimpleSplitter


@harouwu we see the same error consistently in internal testing.

I also see hundreds of lines like this in the log:
`I0824 17:21:36.598489   251 plan_executor.cc:436] Started executing plan.`

Which reminds me of this bug (https://github.com/caffe2/caffe2/issues/758), though it's not actually an infinite loop. @volkhin you fixed that issue - any ideas about this one?


One property file with one sample service for each type is enough.
Also it should include the email setting (they might be commented out)

Name it sample-config.properties

If you can include the image in the jar that will be fine.

Sent from my iPhone

24 Αυγ 2017, 20:26, ο/η Alexandros Kalavitis <notifications@github.com> έγραψε:

> I can make a template properties file, which will have one of each service commented out including all parameters and copy it there. Also wile i was making the scripted i noticed that the logo is not included inside the compilation of the .jar and it is a folder up from where sw.sh/bat is executing. Should we put it inside the jar ?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


develop (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1530 of 1530 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-39.28%25-green.png)


You could instead pass an option object


Breaking change


There is a lot of type checking here, maybe you can create a helped method that checks for all of this?


Hmm the validation checks are necessary to go in fallback: it is for the case of somebody enabled the upscaling with SCREEN_UPSCALING != 0 but the rest of stuff is not activated or contains wrong values.


Do you need these in the `__init__` also? or will `scene_starting` take care of that?


Add another overload


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - Win64-Windows-7-VisualStudio-14: OK (2746 of 2750 tests passed, 0 test warnings)

Messages:

 * 2 tests had: EIO diffs.
 * 2 tests had: ESO big diffs.
 * 2 tests had: MTR big diffs.
 * 1 test had: Table small diffs.
 * 1 test had: AUD diffs.
 * 1 test had: BND diffs.
 * 1 test had: MTD diffs.
 * 1 test had: Table big diffs.

Failures:

EnergyPlusFixture Test Summary
 * Passed: 520
 * Failed: 2

regression Test Summary
 * Passed: 607
 * Failed: 2

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


> Instead, people want to waste more time on HLE renderers with more patchwork 'per-game hackery', like it seems you and Gonetz prefer doing. Probably because both of you know nothing about the underlying RDP anyway so you wouldn't be able to do it properly in LLE like Parallel RDP. Which is fine. But then at the same time, don't casually brush off our efforts with Parallel RDP either.

> more 'hey this hack doesn't work for this or that game, so let's do another torrent of conditionals like 'if ZELDA then turn this off, if POKEMON then turn this on, etc, etc.'.

> they will just continue giving crappy HLE hackjobs the time of day, that they continue looking the other way with all sorts of per-game hackjobs

http://gliden64.blogspot.com/2017/04/major-modification-of-frame-buffer-and.html


If semantic versioning is followed, it should be bumped to 5.0.0 since you're completely replacing an API. If not, 🤷♂


perhaps have a default value something like:
`const isIsolatedDomain = req.x_wt && req.x_wt.ectx && req.x_wt.ectx.ISOLATED_DOMAIN || false;`?


There is no space after the Site's Short Name and the word and here.


@40huo
1. 打开文件的mode需要分清各种状态
2. 报告页里面的原有变量都理清楚，不要有调用被赋值的，比如没有漏洞的情况显示有问题，这块问题比较多，多花点精力看看


separate issue, but one to think about, we can offload internal communication so we should find a way to specify the protocol from an x-proxy-schema.  I wouldn't make it part of this PR but we should test this out a bit.


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/composer-category-chooser-is-above-composer-message/68704/4


Your right. Sorry about that.
I replaced the zip, please try again after re-downloading it.


Boa tarde Vanderson! Instalei o módulo cielo tudo certinho, mas quando preencho os dados e confirmo ele fica carregando e não do checkout... vc poderia me ajudar?


Please delete code instead of commenting it out. It makes your repo harder to grok.


In light of #122 shouldn't you change that to IE9 ?


Love it!

On Aug 17, 2017 2:56 PM, "thisisyourose33@gmail.com" <
rustywells1970@gmail.com> wrote:

> Check out “This is You 33 "Amelia"” from This is You 33 on Vimeo.
>
> The video is available for your viewing pleasure at
> https://vimeo.com/228790572
>
> If you like this video, make sure you share it, too!
>
> Vimeo is filled with lots of amazing videos. See more at https://vimeo.com
> .
> This is You 33
>


##This is a markdown file


This should come from the `PUBLIC_DATA_BUCKET` rather than be hard-coded.


Any way we can also test the job when there's a response?


у sphere не может быть этих параметров, если что. они cones принадлежат


Then put the validation in the ini parser please - it doesn't belong here


Actually #28, not #29. 😳


strlower() ?


I am going to take note of this so we can chat once this issue is behind us.


Quirk


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2783 of 2784 tests passed, 0 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: BND diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


just making a note that i found this page.


Make this an async test


it would.. but .. how this will look from now?
- deprecated action used by tons of scripts?
What is even worse... ids are not the same in each of DBs :)


Sorry dont understand what you trying to say :-( Do you mean the absence of the if (pDesc != nullptr) in the case that is it: I was following the specs of the function. The specs says the parameter can not be null.


Okay, thanks @knighthawkbro


Give our users some credit - they are gaming on a PC with 3D Vision - if they aren't willing to get their hands a little dirty they should probably be using a console playing in 2D ;-)

I think this should just be a log message followed by a DoubleBeepExit() - the beeps are there because even inexperienced users realise that means that something is wrong, and since they are tweaking the upscaling settings they should be able to put two and two together, even if they don't think to enable logging.


Is this for backwards compatibility?


Created multidev environment [jaypeak#pr-qa](https://dashboard.pantheon.io/sites/63ccddbd-731f-4f84-badc-3edee2e28541#pr-qa).

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


Typo: "... that I *now* ..."


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


Please verify when you tag someone ;-)


How does that `mapStateToProps` method work if we're not importing `store`?


When we were evaluating 1Password For Teams, I reviewed their whitepaper thoroughly, and they definitely have done their homework. Linking to it might be a good resource: [Ref](https://1password.com/teams/white-paper/1Password%20for%20Teams%20White%20Paper.pdf)


I *think* you can call `super(props);` which sets `this.props`, but I'm not 100% sure.


rip ^^


Ah good point -- it indeed doesn't work! Will fix.


Don't listen to the specs - programmers are idiots and will pass NULL where it doesn't belong. Sure, the function will fail in that case, but maybe they can cope with that. If we dereference that pointer we will crash instead, which they won't cope with.


I like it.


godoc fucked up, also pls remove dead commented code


Quirk


if you aren't actually combining multiple reducers this is unnecessary


mmmmm your reducer should represent your entire state, not a single action


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182004 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182004") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182009 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182009") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170824-182015 "deployed-2-env-Prod-20170824-182015") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182024 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182024") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


This just got pushed to master @felipesabino ;)


I just submitted the pull request and this is what I have so far. If you
want to we can port the listen speech to a different client like
conversation.py instead of the console.py.

On Thu, Aug 24, 2017 at 9:54 AM, Keith Sterling <notifications@github.com>
wrote:

> Would make an interesting new client. Console is great for quick testing
> and interacting, maybe a Conversation.py client that is designed to be
> voice in/out way of using Programy
>
> Anyway, I think it looks really useful and another great addition to the
> code base which ever way it goes in
>
> K
>
> On 24 Aug 2017, 15:49 +0100, darylfung96 <notifications@github.com>,
> wrote:
> > I didn't expect that you would want a pull request lol. I will change
> that to an additional option so that everyone can decide if they want text
> or speech and then I'll send you a pull request :)
> >
> > > On Aug 24, 2017, at 6:24 AM, Keith Sterling <notifications@github.com>
> wrote:
> > >
> > > When is this becoming a pull request to master ;-), I'd love to have
> this on the mainline for everyone to use
> > >
> > > ―
> > > You are receiving this because you authored the thread.
> > > Reply to this email directly, view it on GitHub, or mute the thread.
> > >
> > —
> > You are receiving this because you commented.
>
> > Reply to this email directly, view it on GitHub, or mute the thread.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/darylfung96/program-y/commit/d04f60ecd192ad149ed33206b4194e1064fc01dc#commitcomment-23847955>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AXOu8rfOZIPDIL0FScreyUO4sPG4xftZks5sbY6dgaJpZM4PBPzL>
> .
>


Fixed, thanks!


usually, only one text is needed. In that case, u can set a single text and its same functionality as before. But when you have multiple texts, then its more intuitive and easier to use than the current 2 deprecated ones. Whats better, is that the system is the same as dbscripts, so its easier to learn as well. And also, nifty thing. There is a shitton of scripts where "no text" is a valid roll. My system can do that as well ^^. Not to mention, the same system can be used in SD2 to reduce amount of code ^^.


AddressProblemUnitTests (Myoldmopar) - i386-Windows-7-VisualStudio-14: OK (2749 of 2750 tests passed, 1 test warnings)

Messages:

 * 2 tests had: EIO diffs.
 * 2 tests had: Table small diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.

Failures:

regression Test Summary
 * Passed: 608
 * 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)


Hi @ZizzyZizzy, could you give me example code that you're having a problem with? Either paste it here or a codepen/jsfiddle would be great.  I can take a look.


Компиляция recovery v_3.1.1-0 (без использования dt.img) проходит без ошибок, но TWRP не стартует


Hi @accleo,
My apologies for the delayed response.

The photo subtest, while deployed to production, isn't actually enabled in production deployments at the moment. You have to perform a direct database manipulation to make it functional on the tangerine group. Are you using this subtest or simply experiencing the error on the server in general.

that error doesn't make a lot of sense to me because it indicates that the key is null. can you give me a little more context on your deployment?  e.g. is this a fresh install or upgrade, content language, anything that might help me understand your situation better?

thanks


![](https://github.trello.services/images/mini-trello-icon.png) [asdfghj](https://trello.com/c/C0AR9PDW/3-asdfghj)


Fixes a deadlock issue on windows 10 mobile, that in some instances appears when accessing CoreApplication.MainView.CoreWindow.Dispatcher. [The same issue discussed here](https://stackoverflow.com/questions/31839003/async-await-deadlock). After repeated testing, i have not been able to reproduce deadlock after fix, with the deadlock happening consistently without when scrolling through a 4 column GridView with image thumbs.


#2520943-WindowsCalc-Engine_Integration (vidanovic) - x86_64-MacOS-10.9-clang: Tests Failed (2099 of 2752 tests passed, 0 test warnings)

Failures:

EnergyPlusFixture Test Summary
 * Passed: 482
 * SEGFAULT: 40

integration Test Summary
 * Passed: 1
 * Failed: 611

regression Test Summary
 * Passed: 608
 * Failed: 2

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


oh yeah I forgot that you could pass `items = NULL` and have it return all items


moi aussi @TmpName mais comme le fichier n'est pas terminé je suis revenu en arrière pour ne pas avoir de conflit avec le "site identifier". je n'ai pas réussi à renommé le fichier. par contre à faire quand on intègrera le fichier ce sera à faire.
Non il te faut:
`sThumb = str(aEntry[0])`
`if sThumb.startswith('/'):`
    `sThumb = URL_MAIN[:-1] + sThumb`


![](https://github.trello.services/images/mini-trello-icon.png) [Realtime setup](https://trello.com/c/nTo1zAVt/56-realtime-setup)


There's is no usage of store from the import statement. The only 'store' in use is passed to mapStateToProps by connect / react-redux. As I understand it.


@sergeykolychev
Rerunning the test worked. The problem was with the environment itself.
Thanks a lot for your help!


Created multidev environment [jaypeak#ci-150](https://dashboard.pantheon.io/sites/63ccddbd-731f-4f84-badc-3edee2e28541#ci-150).

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


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182803 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182803") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182809 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182809") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170824-182814 "deployed-2-env-Prod-20170824-182814") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182824 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170824-182824") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Oh, duh. `store` is passed as an argument to that function. Reading is fundamental.


@loganmc10

See the source code notes -

 // Unfortunately, the N64 RDP doesn't *need* to know the framebuffer
 // height since it's implied by the scissor box.
 // It only renders scanlines.
 // We will need to estimate the real height and potentially flush out
 // everything if we guess wrong.
 // Just employ some crappy heuristic to make this sort of work.

At some point, there is still the matter that a hardware graphics API like Vulkan and/or GL doesn't deal in scanlines. This is unavoidable.

You're welcome to come up with a more generic solution if you know of a better solution. Somehow I doubt it though.


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/failing-tests-in-discourse-dev-due-to-multisite/68720/1


You don't, but I got warnings from my editor about setting new variables on self without declaring them in __init__.


I did this to make the props more explicit for tooling purposes. I was getting false errors about props.


@snicolet : actually, I hoped that this can be helpful against weaker opponents. But, it seems difficult to pick a variant that scales well...


I do believe x can be also other things than string in this function. I do believe it can be a function too. https://www.lua.org/manual/5.2/manual.html#pdf-load


Necro-ing 5 year old commits ftw


图片会自动居中，不必用繁琐的 HTML 语法，Markdown 即可 `![]()`。魔法在此：https://github.com/yihui/hugo-xmag/blob/412c0591715610b9ce78509a127de31526d83764/layouts/partials/footer.html#L1


@pb00068 : thank you, but it seems difficult to find a variant that scales well.


WTF dude...


Generally you don't want any tokens/keys anywhere in your code because this is saved forever. In the future I recommend you add it to a .env file and load that in memory.


Yes, but new descriptor objects are given all the time, so it should be no big deal.


It might make sense to try out putting `null` in Base now that the `Union` optimizations have landed, as long as we're okay with not having a convenient syntax for it during the deprecation phase.


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1530 of 1530 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-39.28%25-green.svg)


rien que çà sa suffirait

`sThumb = URL_MAIN[:-1] + str(aEntry[0])`


It's great this works. However, the structure of this code is still not the best that one can easily follow and manage. One thing we definitely want to pursue is to strengthen modularity of the slice controller code. *SliceCommon* is a significant first step toward achieving the desired modularity. As we've discussed, there are a few modules we should have in a slice controller: Network manager (for Plexus),  Ahab controller (SliceCommon is pretty much for it),  SAFE client. And now we have a REST server for each controller. The repo already contains most of the code needed to fulfill these functionalities.  What we need is to move code around to build up these four modules. With these modules, one can easily compose a slice controller. Besides, classes for a customer slice controller and a sdx slice controller  share common things from a generic slice controller.  This means we have a simple class hierarchy for slice controllers here.


Improve MouseDispatcher perf (#3387)

Demo: quicktests | fiddle


You're sorting the array of report type hyperlinks in the view, after they've been paginated in the controller. Is this what you really want? Imagine this is your original unsorted, unpaginated array of report types:

```ruby
['charlie', 'oscar', 'golf', 'papa', 'hotel', 'bravo', 'tango', 'alpha', 'sierra', 'delta']
```

Let's say they get paginated into two pages of five in the controller:

**Page 1**
```ruby
['charlie', 'oscar', 'golf', 'papa', 'hotel']
```

**Page 2**
```ruby
['bravo', 'tango', 'alpha', 'sierra', 'delta']
```

Now they get sorted in the view:

**Page 1**
```ruby
['charlie', 'golf', 'hotel, 'oscar', 'papa']
```

**Page 2**
```ruby
['alpha', 'bravo', 'delta', 'sierra', 'tango']
```

Q. Is this the correct result?
A. You need to sort the whole array of report types in the controller before paginating them to get the correct result:

**Page 1**
```ruby
['alpha', 'bravo', 'charlie, 'delta', 'golf']
```

**Page 2**
```ruby
['hotel', 'oscar', 'papa', 'sierra', 'tango']
```


Binary Sastantua to diff from


I don't think we want to continue when there is an error. Shouldn't we fail altogether ?


@brianchandotcom It's not quite the same; the original signature enforces a compile error with less than 3 arguments. This convenience method shouldn't really be invoked when there's only 1 or 2 string constants. With 1 string there's no need to concat, and with 2 strings, `s1.concat(s2)` should be used instead. If a general purpose `concat(String...)` is desired, we'd need to add short circuits for these special cases -- to avoid unnecessary `StringBundler` instantiation in case of reckless copy and paste of test code with fewer arguments.


The 10% opacity looks pretty faint to me:
![image](https://user-images.githubusercontent.com/4256376/29683021-03d80888-88db-11e7-9c6a-5eae1a75eac1.png)
Perhaps we could do something more like 50%?
![image](https://user-images.githubusercontent.com/4256376/29683043-197a638e-88db-11e7-8668-d3509c692421.png)


@hughsando tried this and last commit, and Windows worked OK. Howerver for Winrt got error when compiling Main.cpp

```
Compiling group: __main__
cl.exe -Iinclude -DHX_DECLARE_MAIN -Iinclude -ID:/bamtang/nmehd/extensions_201708/nme-toolkit2/sdl/include -ID:/bamtang/nmehd/extensions_201708/nme-toolkit2/sdl/include/configs/winrt/ -nologo /WX- /fp:precise -DWINAPI_FAMILY=WINAPI_FAMILY_APP -DHX_WINRT -errorReport:prompt -DHX_WINDOWS -GR -O2(optim-std) -Zi(debug) -FdD:\DisplayingABitmap\Export\winrt\haxe\cpp\obj/msvc19-rt64-nc/vc.pdb(debug) -Od(debug) -O2(release) -Os(optim-size) -FS -Oy- -c -EHsc -GS- -ID:/bamtang/nmehd/extensions_201709/hxcpp/include -DHXCPP_M64 -DHXCPP_VISIT_ALLOCS(haxe) -DHXCPP_API_LEVEL=331(haxe) -D_CRT_SECURE_NO_DEPRECATE -D_ALLOW_MSC_VER_MISMATCH -D_ALLOW_ITERATOR_DEBUG_LEVEL_MISMATCH -DHX_WIN_MAIN(main) -wd4996 ... tags=[haxe,main,static]
 - D:/bamtang/nmehd/extensions_201709/nme/project/src/winrt/Main.cpp
 - src/__main__.cpp
Error: Main.cpp
c:\program files (x86)\windows kits\10\include\10.0.15063.0\winrt\roerrorapi.h(204): error C2065: 'FALSE': undeclared identifier
c:\program files (x86)\windows kits\10\include\10.0.15063.0\winrt\roerrorapi.h(214): error C2065: 'FALSE': undeclared identifier
c:\program files (x86)\windows kits\10\include\10.0.15063.0\winrt\roerrorapi.h(228): error C2065: 'FALSE': undeclared identifier
c:\program files (x86)\windows kits\10\include\10.0.15063.0\winrt\roerrorapi.h(239): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(49): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(64): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(98): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(111): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(632): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(632): error C2789: 'status': an object of const-qualified type must be initialized
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(632): note: see declaration of 'status'
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(670): error C2065: 'FALSE': undeclared identifier
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(670): error C2789: 'status': an object of const-qualified type must be initialized
C:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt\wrl\wrappers\corewrappers.h(670): note: see declaration of 'status'

Error: Build failed
```


:+1: Very good.


develop (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2140 of 2141 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 610
 * Failed: 1

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


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

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


AddressProblemUnitTests (Myoldmopar) - Win64-Windows-7-VisualStudio-14: OK (2749 of 2750 tests passed, 0 test warnings)

Messages:

 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table small diffs.

Failures:

regression Test Summary
 * Passed: 608
 * 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)


Rendering to a 640x480 framebuffer at all times I fear would be too slow, so I don't see what whatever Gonetz was proposing there would work. Compute shading is very, very expensive, even for low resolution workloads like 320x240. Moving an entire RDP rasterizer to compute even moreso.

I think you and gonetz just don't understand that this is an entirely different ballpark to just doing yet another HLE GL renderer. You're welcome try and do it better, while still keeping performance up.


Logged Grow bug - https://github.com/grow/grow/issues/512


@tomkinsc if py.test is not installed when the easy-deploy script is run, this line fails.  Maybe install it in the conda env first?  Same for git.


accepted


This changes requires some changes to data files. Can you add data files to github and let us start to monitor changes ?


@reinhardt nope. I had troubles with zest.releaser when trying to do that release. Will do the release over the weekend.


fyi @ryanbr


From what I can tell, the source PropType indicates that a per-source header prop is not supported. We don't have any monitoring in place for comments left in commits as it happens, so #13697 is a better place to discuss this.


Initial index.html file empty place holder using Notepad++


Hi @maxiloc I need to update our config, I cloned the repo and created a new branch, but when I tried to push to origin, it doesn't allow me, how do I update the config??

We want to index every pages in this format (it's our documentation pages), https://doc.periscopedata.com/article/*

but https://doc.periscopedata.com/article doesn't have content,
how do I make the start_urls so it'll index correctly???

Thanks!

-----

{
  "index_name": "periscopedata",
  "start_urls": [
    "https://doc.periscopedata.com/article"
  ],
  "stop_urls": [
    "\\.pub$",
    "/newdoc/",
    "/doc/"
  ],
  "selectors": {
    "lvl0": ".doc-layout h1",
    "lvl1": ".doc-content h2",
    "lvl2": ".doc-content h3",
    "lvl3": ".doc-content h4",
    "lvl4": ".doc-content h5",
    "text": ".doc-content p, .doc-content li"
  },
  "min_indexed_level": 1,
  "nb_hits": 1974
}


Hi dude check this: https://github.com/ezio84/abc_device_huawei_angler/commit/0931bb92d51e55f06af27243471b01358c74d5a0

Haven't tried to build yet and haven't checked well the code but now in framework_base that config_nightDisplayAvailable is linked to that config_setColorTransformAccelerated boolean, so enabling this one could give some additional night mode options to play with also on angler.


@j-lawson as you're updating these, you may want to get into a habit of occasionally updating the NEWS file with any user-relevant changes. I'm trying to do this better with all the R packages. At least keep this in the back of your mind. Some examples:

* https://github.com/databio/simpleCache/blob/dev/NEWS
* https://github.com/nsheff/LOLA/blob/master/NEWS

I am trying to follow this standard on these: http://keepachangelog.com/en/1.0.0/


Created multidev environment [jaypeak#ci-151](https://dashboard.pantheon.io/sites/63ccddbd-731f-4f84-badc-3edee2e28541#ci-151).

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


@joseph-robertson there is now an upgrade_name column in the results.csv, so you can remove it from your post-processing script.


![](https://github.trello.services/images/mini-trello-icon.png) [initial commit yopta](https://trello.com/c/Ha5yy9jA/14-initial-commit-yopta)


Подключили


@vreuter where did you find 3.4 on rivanna? I only see 3.3


Nice, thanks for this fix :)


beautiful! i _really_ like handling this on the Panel instead of the first child. as always, great pixel-perfect attention to detail!


Переподключили


[R/methods-plotDEGHeatmap.R:48:26:](https://github.com/hbc/bcbioRnaseq/blob/32ae3dfce7f7159a7780bbafa079b3ed2fb78098/R/methods-plotDEGHeatmap.R#L48) *style:* **Integers should not be implicit. Use the form 1L for integers or 1.0 for doubles.**
```r
    if (length(genes) < 2)
                        ~^
```


Кнопка Фейсбук


Good point


Do you want us to comment on your repo, or are you going to create a PR for it?


Модель авторизации


Does this mean KAUI will now output his log into `killbill.out` ?


Без пустых читается плохо. И принято сначала подключать модули, потом описывать данные класса (attr_reader), только потом методы


develop (JasonGlazer) - i386-Windows-7-VisualStudio-14: OK (2141 of 2141 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)


@hideki Thanks for re-adding the DB14 release notes! I had overwritten them last week by mistake with a force push.


AFN2017 (jasondegraw) - x86_64-MacOS-10.10-clang: Build Failed

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


No I'm happy with this, you've obviously seen the light and decided to create a hacky HLE renderer like the rest of us.


[R/methods-plotDEGHeatmap.R:48:26:](https://github.com/hbc/bcbioRnaseq/blob/ea3d9dfb6b71777c96fd5a005762aa0349fdf1b4/R/methods-plotDEGHeatmap.R#L48) *style:* **Integers should not be implicit. Use the form 1L for integers or 1.0 for doubles.**
```r
    if (length(genes) < 2)
                        ~^
```


By not having an accompanying `*/` to close out this comment block, the JS in this file is completely borked.


missing new DataColumn()


Excelente, tome su badge.


Unneded <p>


I would add one to the end of line 26, right before `}`


vuri funnu gi


[R/methods-prepareTemplate.R:13:1:](https://github.com/steinbaugh/basejump/blob/eb454ed33fc0a6bfacf69d21776cc6ace000fbb1/R/methods-prepareTemplate.R#L13) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sourceDir Source directory, typically a URL, where the dependency files are
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Could you take a single hash?


You have the same case above at line 10, stick with just one.


Методы-команды не возвращают результата. Суть этого метода - выбросить ошибку, если что-то не так. Если исключения не выброшено, значит он отработал штатно. И как раз метод `valid?` преобразует результат метода `validate!` в логический результат (true если не было исключений, false если было).
В общем, эту строку надо просто перенсти в метод `valid?`, а из всех методов `validate!` убрать


Replace with sync call.


Title should say "Finished Edit Profile Page".


This is due to recent refactoring in broad institute/viral-ngs; `install_viral_ngs_conda_deps()` needs to also install `--file requirements-conda-tests.txt` Since pytest is now specified in that file.


@reduz Why was this commented?


All of your action and reducer setup looks good! You're just missing the final piece, which is `connect`ing your page components using `react-redux`. You can see more about that here: https://github.com/wbobeirne/nycda-ecommerce-server/wiki/redux-integration#the-connect-function


small typo here: `componenets`


There already is such a field.


yeah I tried to do that originally, but couldn't figure out how to sha with multiple args in web3 (to test against the contract). I see your comment about using a constant function, though, and that makes sense


And this is exactly what the balance field does.


Will this thing allow gradle to download and use packages from the Dev channel without specifying it in arguments?


Angrylion-based rasterizers are obviously not and never will be HLE. What is HLE, is trying to reverse RSP microcode snippets and then spitting out a bunch of high level GL functions to try to simulate the rendering.

I know that you are basically trolling right now, but honestly, even if it is for that reason, you are just highlighting that you dont understand the challenges associated with making a compite based RDP renderer still moderately fast. I repeat again, if we tried that approach as described by your precious Gonetz, it would not be good on performance. This is obviously not the same kind of hacks as what I was referring to when it comes to HLE renderers. If you had ever looked at those codebases, you could understand what I am talking about.


Ah yes, that would probably be it.


possible to merge shadowsocks and sshtunnel?


Place mkv before mp4 (alphabetic order)


`begin` добавляет уровень отступа (так же как if)


The reason I didn't use `docker container prune` is because that command deletes containers by creation time rather than exit time. So if you have a 6-day-old container which crashes at an inopportune time, that container could get deleted before you have a chance to inspect it. I would personally change it back until `docker container prune` gets smarter. Thoughts?

Also, you have to purge cronjobs to actually delete them...


Right.  But git isn't there either.  If easy-deploy is meant to be self-contained, maybe install git too?   conda-forge has a recent package.


i guess that the selector ought be "ion-input[currencyMask]" or "input[currencyMask]"


Ok.


Prefix them with `Log` to avoid confusion. `LogNewTransferRequested`.


`= 0` are `= false` redundant.


Consider passing this as a constructor argument.


OK, I will do that when we have an updated results csv.


`function requestTransfer`


It's under `/share/apps/R/openmpi/`


There is no enforcement logic on the comments, so either they should not be mentioned, or they stay in the event only.


Maybe we could set up travis-ci for viral-ngs-deploy?  Might also catch issues like Hayden just ran into.


Константы принято описывать вверху класса (после подключения модулей). Они все равно не могут быть приватными


@maddhruv Can you check problem with this commit?


My bad :smile:


В принципе у тебя регулярука итак не пропустит строки длиной менее 5 символов


I would recommend you wrap db facing operations into the model classes for example the bucketlist class could have a method to save the data which calls db.seesion.commit internally.. Same would apply for the other models too so that you get more DRY.


Same for `finalTarget`.


Please, refer to [the RR4](https://reacttraining.com/react-router/web/guides/philosophy) or [history](https://github.com/ReactTraining/history/blob/master/README.md) docs if you have RR related questions.
To answer your question, [`withRouter`](https://reacttraining.com/react-router/web/api/withRouter) should work and pass down a `history` property. If it's a route component (a direct child of `Route`), you don't even need to use `withRouter` HoC.
There is one difference with the previous history—you can't pass query object to `history.push`, only a query string. If you need this functionality, you'll have to install another library that can do that, like `qs`, and either use it directly in your component or via [`qhistory`](https://github.com/pshrmn/qhistory) that monkey patches `history`.


Adding `#include <stdbool.h>` fixed compiling but crashes at run start


I saw 2 spaces after this `==` :)


Use `require()`.


Should this not be submitted as a bug fix?  We already have ENPASSANT code in there for correctness and I would bet it's less useful than PROMOTION.


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-20170824-191658 "deploy-ibm-yp-us-south-chrisdhanaraj_org-Bluemix-Design-System-20170824-191658") to [Bluemix Design System](https://console.bluemix.net/apps/3618ef36-5a2d-44ed-8a38-10323939f293?env_id=ibm:yp:us-south "ibm:yp:us-south:chrisdhanaraj_org:Bluemix Design System"), including this commit


This line is not necessary as `transfer` is a reference.


Only because it is a `struct`.


</ThemeProvider>


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


SGTM


I don't think this is correct


Could we use the words text or talk as the option instead of an int ?


Nice how you can use a good samaritan to trigger the transfer.


What is an 'externally managed user' ?


This function name is hard to understand (what is `cbt`?), and the parameter names don't communicate what they are. It would be much better if they were renamed to say what they are.


Could you make this 1 hash and save 1 `SSTORE`?


David -- thanks for looking at this. Apologies, this might be an extraneous PR against the latest HEAD. I'd been working with beta3 and ran into the GenomicsDB truncation issue which inspired all this, then tried to patch against the current HEAD based on open issues and the current state of the code. If those other fields now match correctly between actual alleles and A/R in the header we should be all good. I'll compile and try testing with HEAD to see if these changes aren't necessary. Thanks again for the review.


Yes exactly.  Since it was previously a deck of actual objects, you'd have to use & to get the address to that item to assign to the pointer in wonderBoard[I].  Now, since they're both pointers, it's just an assignment operation with no qualifiers.


I believe `sender` is there for another future function?


**Uhmmm**..

> "Verificati commit cu comentarii."

I am testing it _right now_.

- A
- B
- C

![codetaplogo](https://user-images.githubusercontent.com/28351500/29684349-55de4e0a-891a-11e7-8085-6446bdd75ecc.png)


You can use `TransferRequest({ sender: msg.sender, target: requestedTarget, ... })` to avoid mixing up.


Doing control flow like this is pretty confusing. `plone` and `pltwo` aren't really separate functions, they should just be the bodies of the `if` and `else` blocks.


How about dealing with the fee straight away and not `SSTORE` it?


Are you sure the files are support to be named that way? It seems odd that they'd have `resource.language.<lang>` in the `resources/language/` folder. I would expect `resources/language/en_GB/strings.po` (or something like that).

---Alex


What is `one` vs `two`? These might benefit from better names, too.


In context, I can guess that `agl` is `agility`, but there's no reason to shorten these words. It's a lot better for your code to be clear when reading.


@orta Oh, I sent you a message on gitter about that when it happened and here it is just for the record

>  I was testing Peril and did the stupid thing of pushing to the wrong remote (the actual Peril git repo and master branch) instead of my heroku one. I immediately reverted it but wanted to let you know just in case something weird happened in those 10 seconds that my screw up was up

Anyway, as I said I reverted that wrong commit almost immediately and this PR has nothing to do with that 😉

And sorry again to all team the previous screw up 😥


Раз всего один метод класса, то лаконичнее записать так -
```ruby
def self.all
  @@stations
end
```


What is `check`? Should this be something like `.hasUpgrade()`?


Good luck at LTC!


cbt is short for combat.


I recommend deleting commented-out code like this. The code is always going to be in your git history, so you can reference it again if you need to. It's better to keep the most recent commit clear and readable.


@katzer Did you actually merge in the ios10 branch? I don't see it on the master commit list.


@mcostalba opinions on this ?

It did fail a [0,5]:
http://tests.stockfishchess.org/tests/view/5999697e0ebc5916ff64a8e0

Note that it only deals with the first promotion, i.e. pawn promotions later in the sequence of moves are not dealt with. I'm not even sure that would work in the current scheme, but I don't know this SEE code well.


Когда нам нужен быстрый доступ по ключу лучший выбор - хэш. А нам как раз надо поезда по номеру искать :)


semicolon


I'm using `qhistory`, here's an example with a `query` object:

```js
      this.props.history.push({
        pathname: '/sales',
        query: { ...this.props.location.query, customerName: name },
      });
```

or just pass a string as a first argument: `this.props.history.push('/home', { some: 'state' })`, second argument is html5 history's state.


one is player one and two is player two.
These objects are passed through the fight function to govern the fight order and dynamics.


This function really feels more like two separate functions `enemy.flee()` and `player.flee()`. Can you make this code more object-oriented?


hah - rockin, I guess I don't get notifications when it's reverted 👍

I added you to slack


1. Этот метод не должен ничего на экран выводить, он должен просто вернуть экзмепляр класса. За интерфейс отвечает другой класс.
2. Метод `select` всегда возвращает массив, а нам нужен один экземпляр
3. При использовании хеша метод был бы значительно проще


@aqw Yup, see http://kodi.wiki/view/Language_support. This is also the way https://github.com/xbmc/generator-kodi-addon does it.


149092157-AERC_updates_to_BSDF (vidanovic) - x86_64-MacOS-10.9-clang: Tests Failed (2348 of 2744 tests passed, 134 test warnings)

Messages:

 * 529 tests had: EIO diffs.
 * 166 tests had: ESO small diffs.
 * 354 tests had: ESO big diffs.
 * 192 tests had: Table small diffs.
 * 159 tests had: SSZ small diffs.
 * 240 tests had: ZSZ big diffs.
 * 305 tests had: Table big diffs.
 * 83 tests had: SSZ big diffs.
 * 53 tests had: ERR diffs.
 * 48 tests had: MTR big diffs.
 * 58 tests had: MTR small diffs.
 * 55 tests had: ZSZ small diffs.

Failures:

EnergyPlusFixture Test Summary
 * Passed: 521
 * Failed: 1

regression Test Summary
 * Passed: 211
 * Failed: 395

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


@killerwife im happy with your new system.. (i only asked ..if we planing to do anything with older one)...
Also can you pls give some info about that:
dbscript_random_templates
```quote
type 0 - strings
type 1 - relay dbscript
```
type 0
so from now this is used in both... DBScripts and ACID ..if im correct.
so if im using DBscripts? -- im using (target_id) textsid like these ones still 2000000875.
but if ill use ACID -- im using  (target_id) texts from creature_ai_texts like: -47 or -166 etc...
So we'll have negatives for acid and positives for DBScripts... correct?


You got it.


Okay! Thanks for the reminder!

On Thu, Aug 24, 2017 at 2:56 PM, Nathan Sheffield <notifications@github.com>
wrote:

> @j-lawson <https://github.com/j-lawson> as you're updating these, you may
> want to get into a habit of occasionally updating the NEWS file with any
> user-relevant changes. I'm trying to do this better with all the R
> packages. At least keep this in the back of your mind. Some examples:
>
>    - https://github.com/databio/simpleCache/blob/dev/NEWS
>    - https://github.com/nsheff/LOLA/blob/master/NEWS
>
> I am trying to follow this standard on these:
> http://keepachangelog.com/en/1.0.0/
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/databio/MIRA/commit/f4e7e7683bf37aafc8a3de6b9c9eecfa51017a0e#commitcomment-23853022>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AVRSxlA5bhO5ASw1T-1JKWGmT_dAzZjhks5sbcdhgaJpZM4PBypq>
> .
>


I find comments distracting and makes the code harder to read.


@emilsvennesson Weird... well, thanks for confirming. :-)

---Alex


Does it have any relation to https://github.com/vrtadmin/clamav-devel/commit/a83773682e856ad6529ba6db8d1792e6d515d7f1 and https://access.redhat.com/security/cve/CVE-2017-6419 ?


Why?  Shorter names mean you can see more of the code at once.
Couldn't a notes page be made to reference what the variables mean?


I'm not sure of the best place to put this comment, so I'm just putting it at the top of the `fight` function.

It looks like your code mixes a lot of detail and general fight concepts. Because it's so long, it's hard for me to "see the forest through the trees" and understand what the fighting logic looks like. For example, instead of keeping all of the venom logic in this one function, it might be better to split it like this:

```
if (venomOn()) {
  attackWithVenom()
}
```

That way you can see the battle logic "If venom is around, use it", but the venom logic itself can be self-contained. In general, look for ways to shorten this function by pulling out _related_ chunks of code into separate functions or objects. Don't just pull out code indiscriminately to make it shorter, but find _concepts_ that can be pulled out. For example, you don't want:

```
step1();
step2();
step3();
```

This is short, but doesn't tell you anything about what happens during fighting. It's much better to have your function tell a story:

```
applyModifiers();
player.attackEnemy();
if(enemy.alive()) {
  enemy.attackPlayer();
}
```

This is just an example, but it lets you see exactly what combat looks like without bothering you with the details. If you need to know the details of what `enemy.attackPlayer()` does, you can find that function and read it. Being able to [abstract](https://en.wikipedia.org/wiki/Abstraction_(software_engineering)) your logic and components is a huge part of writing code that's not only readable, but also easier to modify.


hello


Зачем эта строка? Счетчик должен увеличиваться при вызове register_instance


Зачем два отдельных счетчика в разных модулях?


Зачем этот метод?


added .DS_STORE


Yes, and if you are using it in SD2, you are using SD2 script_texts table. Only thing we couldnt decide is dbscripts_on_relay IDs. We think we will start adding from 1.


Already said but update the state before an interaction.


I see how that could be done.
But I could also combine them further and create one unified flee function removing half the code as redundant.


This commit seems to have broken simple line icons in some places, as pointed out by @av80r . will fix in #136 .


Your contract is not to be nice. It does the thing or it reverts. No cuddly `false`.


change link


И еще попробуй подключить модуль в 2 разных класса. Счетчик у них будет общий, а должен быть отдельный (у Train выводить число поездов, у Wagon - число вагонов)


Лучше писать так:
```ruby
private

def register_instance
  ...
end
```
Так сразу понятно, что все методы ниже - приватные


You probably can replace this `bool` with a test on other fields.


Consider adding whodunnit.


I'd at least rename them to `playerOne` and `playerTwo`, then. When I see names that are that similar, I read that to mean that they can be swapped with no problem. I see that they're swapped based on who has higher agility. Is that something that's better calculated in `fight` itself, so that `fight(playerOne, playerTwo)` and `fight(playerTwo, playerOne)` always do the same thing?


The yml syntax format was incorrect.
**Now Fixed**


I moved them because they haven't been part of 0.6.17, just merged after 0.6.17 was published


Consider indexing the fields which are reasonably expected to be listened to by interested parties.


File Deleted


A and w the wrong way round?


Constructor argument instead.


Это скорее конвенция. Так запись короче + у строк есть такой же метод (конкатенирует 2 строки)


Team update at 24 august of 2017


No need of the `restricted` modifier on a constant function.


Your contract is ripe for inheritance. `Owned`, `FeeCollector`.


Added: `isHexDigit()` -- prob should check if we already have?

Added: `_ColorizeString()` which is a glorified `memcpy()` except it increments the dest pointer to make "chainable()".


Dude, this is some SICK server side programming!


This cannot compile. `,`


![](https://github.trello.services/images/mini-trello-icon.png) [WPF events to tasks converter. Example: await AwaitableEvent.When(Button.Click, myButtonObject).](https://trello.com/c/xzg65jjh/12-wpf-events-to-tasks-converter-example-await-awaitableeventwhenbuttonclick-mybuttonobject)


Extended: `Colorize()` to mark up hex numbers prefixed by `$`

Fixed: `Examples:` so that the colon is marked up with the proper color.


Мозгу труднее воспринимать отрицания. Поэтому я бы заменил тут на `while i <= 100`


An unsigned int never is negative. You need to test differently.


Why would you accept a `deadlineBlock == 0` anyway?


Code should be self-explanatory as much as possible. Comments and references have their purpose, but greatly add to the difficulty of reading the code base. It's better for the code to be clear by itself. If you're worried about the amount of code you can see at once, that's a major sign that your problem isn't about long variable names, but about long and complex functions. I would focus on refactoring your code (which should _help_ with clarity) rather than shortening names (which _hurts_ clarity).

You might also look into changing the settings on your editor, or finding a new editor that has code folding, highlighting, etc. that can help you read the code better. For JavaScript, [Visual Studio Code](https://code.visualstudio.com/), [Atom](https://atom.io), [WebStorm](https://www.jetbrains.com/webstorm/), and [Sublime Text](https://www.sublimetext.com/) are all popular, as are many others.


Less scope for typo: `TransactionStruct({ origin: msg.sender, destination: destination, ... })`


dziekuje


Your test feels contrived.


Чтоб избежать дублирования, можно было так записать - `while (i = arr[-2] + arr[-1]) < 100`. Т.е. мы прямо в условии присваиваем переменную и тут же делаем проверку


Actually, there is some idea here. You could make a `calculatedCombinedHash` function.


Про пробелы не забывай. Все операторы должны быть окружены пробелами (`arr << i`)


dockertar-sha256-helper.go is our file, it should not go under `vendor/`


No need to check for `msg.sender` on a constant function.


And another one that returns all the info of a given remittance.


that thing you did on that one line of code... totes magotes terrible.


I suppose I can collect as long as there are still Ether in the contract. Bug.


missing the directory in the end, that is " .."


That could also work. You may find then that there are two layers of functions. A general `flee` that applies to both, and then specific `fleeOne` and `fleeTwo` functions that apply to only one type of player. An example would be something like:

```
function fleeOne(x) {
  flee(x);
  EVO.won += 1;
}
```

Keep in mind that although reducing code duplication is generally good, the _reason_ it's good is because it makes the code 1) easier to read and 2) less-likely to contain errors. Sometimes you may find that code is easier to read by leaving duplication, so play around with both styles and learn when it makes sense to do one or the other.


No, the problem is that the file is opened repeatedly when writing out each line...


Я по-прежнему не понимаю зачем эта проверка нужна. Если в `user_month` единица, то в цикл while не зайдет вообще. Можешь пояснить для чего она нужна?


1° passo:

Crie uma pasta com seu nome no diretório plataforma-mobilidade/Aula-03/Atividade/. (Em caso de homônimos # crie uma pasta com nome e sobrenome sem espaço entre eles ex.: vitor_silva/).


set_return_type is no longer in MethodBind, so I guess it should be moved to vararg method bind


I feel uneasy with this additional branch.


retira o espaço do nome e sobrenome


Same here, can collect as long as... Theft bug.


heyyyy. You're looping now :p


![](https://github.trello.services/images/mini-trello-icon.png) [Semantic UI Framework implementacija](https://trello.com/c/okxlLple/57-semantic-ui-framework-implementacija)


I think at least #149   and #150  should be fixed before doing a release. #150 is probably fixed already in plone.restapi. I didn't check lately.


I've removed the logging


If you got to this point and the video explaining how to use icomoon.io is not ready just download the following files and use them:

https://raw.githubusercontent.com/codetapio/my-cv-final/b815353768e75b1a9e51904a5a465d1000198891/src/_asset/font/my-cv.json
https://raw.githubusercontent.com/codetapio/my-cv-final/b815353768e75b1a9e51904a5a465d1000198891/src/_asset/font/my-cv.svg
https://raw.githubusercontent.com/codetapio/my-cv-final/b815353768e75b1a9e51904a5a465d1000198891/src/_asset/font/my-cv.ttf
https://raw.githubusercontent.com/codetapio/my-cv-final/b815353768e75b1a9e51904a5a465d1000198891/src/_asset/font/my-cv.woff


А в чем смысл так делать? По заданию надо порядковый номер в алфавите, а у тебя просто номер позиции в массиве выводится


@cesandoval FYI .DS_Store is a Mac OS X system file and is best [added to your global .gitignore](https://help.github.com/articles/ignoring-files/).


[R/RcppExports.R:26:1:](https://github.com/achubaty/grainscape/blob/57500a3aca46b23f0003a220a381384c6d2486cb/R/RcppExports.R#L26) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-MPG.R:73:4:](https://github.com/achubaty/grainscape/blob/57500a3aca46b23f0003a220a381384c6d2486cb/tests/testthat/test-MPG.R#L73) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:83:4:](https://github.com/achubaty/grainscape/blob/57500a3aca46b23f0003a220a381384c6d2486cb/tests/testthat/test-MPG.R#L83) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:93:4:](https://github.com/achubaty/grainscape/blob/57500a3aca46b23f0003a220a381384c6d2486cb/tests/testthat/test-MPG.R#L93) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:101:4:](https://github.com/achubaty/grainscape/blob/57500a3aca46b23f0003a220a381384c6d2486cb/tests/testthat/test-MPG.R#L101) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


В качестве отступов надо использовать 2 пробела


I don't know if we should add an additional iteration to the 17.1 cycle due to the delay on the 17.2 release ...

Other than that, @jasuarez , could you check whether this is OK with you?


your bro rocks!


Sign-up
Login
Create profile form, including
Custom index page for M4M, F4F, M4F, F4M.
From there - user can further filter their results

Order:
Rails app basic layout
Form to create user profile
Trello API / Instagram API integration


thanks for the catch!

*-Carlos*

On Thu, Aug 24, 2017 at 12:54 PM, Matthew Zagaja <notifications@github.com>
wrote:

> @cesandoval <https://github.com/cesandoval> FYI .DS_Store is a Mac OS X
> system file and is best added to your global .gitignore
> <https://help.github.com/articles/ignoring-files/>.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/MAPC/rental-listing-geolocator/commit/8589030dabdbda1dfb205bbb8d30fddd39b73001#commitcomment-23854153>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AB3KX4kPJTXbS42r1GJJuTPJff0XYL4Lks5sbdT0gaJpZM4PB23h>
> .
>


@colonelchlorine The x branch will be merged into master. The ios10 or 0.9-dev branch are only temporarily.


</div>


less or more than now?


`</div>`


No. The default is `true`. This means that if `enabled` is absent or its value is gibberish, it will be considered `true`. The correct condition is

```
if (!"false".equals(parameters.get("enabled")) && parameters.containsKey("fields") && uixName.equals(uix.getId()))
```
or, in a more logical order,
```
if ( uixName.equals(uix.getId()) && !"false".equals(parameters.get("enabled")) && parameters.containsKey("fields"))
```


Hi, does it means, before this fix, we could not use Remove method? Because I found panic when i call Remove.


I deleted my previous wrongly comment, I mis-read it.  :)


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2138 of 2139 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 610
 * Failed: 1

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


test


@sbeaver-netgate  ditto same as @doktornotor


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

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


Why do we want to fadvise away the page cache when direct io is not used?


Only Twenty-five people signed up.


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-20170824-020605 "deploy-ibm-yp-us-south-chrisdhanaraj_org-Bluemix-Design-System-20170824-020605") to [Bluemix Design System](https://console.bluemix.net/apps/e658b80f-51e4-4fb2-80de-0f413eb6e4f4?env_id=ibm:yp:us-south "ibm:yp:us-south:chrisdhanaraj_org:Bluemix Design System"), including this commit


再取得User的時候不是已經有roleid了?
如果在這裡去修改User.roleId不就會變成
本來帳號是 "會員" 結果一進來變 "超級管理員" 的現象(?)
Success ，我還真的沒寫到這問題，拍謝QQ

從 Windows 10 的郵件傳送

寄件者: Aaron
傳送時間: 2017年8月24日 上午 12:04
收件者: JackyHuang700/DotNetCore_React_Project_Plus
副本: JackyHuang700; Author
主旨: Re: [JackyHuang700/DotNetCore_React_Project_Plus] fix 會員編輯會噴調 (488ede3)

再取得User的時候不是已經有roleid了?
如果在這裡去修改User.roleId不就會變成
本來帳號是 "會員" 結果一進來變 "超級管理員" 的現象(?)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


initfile


Sure. We will add a checker to keep the R version stable.


value不一定是accesspolicy，可能是introduction或sms_sign_id


#6028-PTAC-heating-coil-undersized-for-ventilation-load-when-zone-design-heating-load-is-zero (mjwitte) - Win64-Windows-7-VisualStudio-14: OK (2748 of 2749 tests passed, 0 test warnings)

Failures:

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


sdfsdf


Created multidev environment [ajm-vanilla4#ci-9](https://dashboard.pantheon.io/sites/40d5b393-e81f-451e-a991-e2406d0ad7ec#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-ajm-vanilla4.pantheonsite.io/)


改成setAttributeValue比较好吧，因为不一定是policy属性


I see in `R kernel`, it captures sessioninfo by using `capture.output()`. Let me see how to return the sessioninfo() msg in `julia` without capturing it.


把accessPolicy=改成attributeValue=比较好，


这里的value可以去掉了


dfasdasdasdasd


是 npm 上的版本没有更新吧


I suppose.


xAuthToken太大，打印出来也看不出token是否合法，打出来没有意义，建议去掉打印xAuthToken


Use `doc.getXObject(VOCABULARY_RELOCATION_CLASS, "original", original, false)` to get the right object from the start.


Hi @chiragvels -

How about now? Sorry I hadn't tried "^v0.11.5.0", but this is working for me:

```
{
    "require": {
        "microsoft/bingads": "v0.11.5.4"
    }
}
```


That's an exciting change.


Just `1.4`.


return 是json吧？ 包含request_id 和topic_urn


@SungYeolWoo 맞습니다 ㅋㅋ 이렇게 쓴 이유는 `defaultValue`의 타입이 `Maybe T.Text`이기때문입니다.

`return $ if T.null configValue then defaultValue else configValue`의 타입시그니쳐를 생각해보면, `configValue`가 비어있다면 `IO (Maybe T.Text)`가 될거고 비어있지않다면 `IO T.Text`가 될수도있어서, `M.fromMaybe`를 사용하여 항상 `IO T.Text`가 될 수 있도록 한것입니다.

`fromMaybe`의 [타입시그니쳐](https://hackage.haskell.org/package/base-4.10.0.0/docs/Data-Maybe.html)는 `a -> Maybe a -> a`인데 두번째 인자가 `Nothing`이면 첫번째 인자(`T.empty`는 `""`와 같습니다)를 반환하고 `Just a`이면 두번째인자의 값을 가지고오는 함수입니다.

말씀하신걸 듣고보니 `inputConfig`의 시그니쳐가 `T.Text -> Maybe T.Text -> IO (Maybe T.Text)`가 되어야할거같기도하네요.


Hi,

I'm trying to debug a issue that efi firmware reads 32-bit fb base address, but the GOP actually uses 64-bit address.
How should I debug it? (printk didn't work)
Thanks.

pci reserves above 4G memory for the device, GOP also set the same FB base address
  - pci 0000:03:00.0: reg 0x10: [mem 0x410000000-0x41fffffff 64bit pref]

But actually efifb driver only get the low 32-bit address.
  ==> efifb_probe: screen_info is NOT VIDEO_CAPABILITY_64BIT_BASE
  ==> lfb_base 0x10000000, ext_lfb_base 0x0

Regards,
Jerry


Yes, I removed it because it was unused.


Site 1


Didn't notice this before. Cases 9-16 need to use `outputs_5a_b` since they're on the second bank of the IO expander. Then I think it should be all good.


User should login before create new campground as well as submit post request.


Created multidev environment [badcamp-2017#pr-contact](https://dashboard.pantheon.io/sites/8d658997-7a61-4db8-9be8-0f16b8b62022#pr-contact).

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


develop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2157 of 2157 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)


Update the campground Schema and populate it inside the create so the author data can be saved into database as well.


Allows execution on bare module without distance sensor attached. (Code as is generates constant WDT resets)


I was wondering what the current status of the inlet_profile branches was.  We need this feature for our research interests.  While it wouldn't be too hard to create our own implementation, I wanted to know if this feature is in a useable state in your version.

How buggy are these two new options (`INPUT_FILE` and `VEL_PROFILE`)? How general-purpose are they?


#2 done


develop (rraustad) - 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)


Thanks for the kind words.


The problem is that Base's underlying Libuv code is a little all over the place in terms of error-handling. Sometimes it's an AssertionError, sometimes a UVError, sometimes Argument. In any case, we're unable to get bytes off the wire, so in the next line we immediately abort processing the request.


@hkorth @yuriw
You can just revert  https://github.com/pfsense/FreeBSD-ports/commit/87f5da9d6b9225df0f22a0c17827034c2bd82f74.patch meanwhile (use System Patches or from shell).


@peterhoeg idk but I feel like any of these could convey it: 😅😶😑😓🤦♀🙈


<!DOCTYPE html>
<html>
<body>

<p>
Hello Word!
</p>

</body>
</html>


should this all be moved into patchcore instead?
it seems like it's not specifically tied to ticktack (except for the group thing, but maybe that doesn't actually need to be used in this code since this code is only used to trigger a load?)


这个测试是多余的，这个接口没有request parameter


不用这个测试，这个接口和displayname没啥关系


Thanks for update


Agree, reverted in commit fc6e90d


你之前这么写是对的，因为之前&query->options里面没有设置servers
设置servers是由下面的单独ares_set_xxx来干的


@Starefossen Did you face this error?

@jeff-knurek are you getting this error while just trying to create this chart with helm 2.5 or while auto upgrade?


@doktornotor could not apply via system patches


Yes, this uses XDG_CACHE_HOME now, as in #10


![](https://github.trello.services/images/mini-trello-icon.png) [(2) Setup messages basics](https://trello.com/c/6jWlSrIY/40-2-setup-messages-basics)


@Ponsen Try with v3.5.1
https://github.com/petermetz/cordova-plugin-ibeacon/commit/3bb2600d6c9b2103f4ce30222bac1edfafd826ab


![](https://github.trello.services/images/mini-trello-icon.png) [Add moment.js](https://trello.com/c/eXHMbqVK/41-add-momentjs)


![](https://github.trello.services/images/mini-trello-icon.png) [(8) Header items](https://trello.com/c/QhvEyCdc/39-8-header-items)


Created multidev environment [stevector-composer#pr-1](https://dashboard.pantheon.io/sites/5ed39b26-4e42-4bf3-a6d1-42898bead8bf#pr-1).

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


sssssssssssssssssssssss


最后不要用assertNotNull 来判断，没意义。要判断就明确判断接口请求参数，比如这里是diplay_name


Reverted for now, will investigate.


readme提交，程序名称和描述


尽量避免用assertNotNull来判断


你好，我想换看看


Does this work? Isn't the codebook array a vector of signed 16-bit integers?


请求成功的状态是可以预知的，为啥不用Assert.assertTrue(res.get("status") == 200);


PrepFor88 (Myoldmopar) - i386-Windows-7-VisualStudio-14: OK (2748 of 2748 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)


debug and add a share function


已fix


内部的字符串，已处理成静态不可变常量


@doktornotor I guess I never reverted patch for pfSene, is it System Patches, PasteMe URL from above and then ?


I found that some Android APIs actually give off errors by removing the redundant casts (such as Android API Level 24).


Created all Sprint 1 field entries and labels, created a displaybox to show field entries when tested out of IDE


Fetch and use the Revert button. Or see above. Otherwise post in the forums for help.


Did that and Revert does not show up


PrepFor88 (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed (2226 of 2742 tests passed, 90 test warnings)

Messages:

 * 606 tests had: EIO diffs.
 * 516 tests had: Table big diffs.
 * 11 tests had: ERR diffs.

Failures:

regression Test Summary
 * Passed: 90
 * Failed: 516

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


The problem was that it idn't found nme_managed_stage_create on ExternalInterface


Why bother with string. Use bytes32 directly.


Updating.


@leogermani
saiu no commit


Yeah I'm not so sure I believe that but we'll see I guess.  There are two opcodes being fetched which would mean 8T so the only way it can be 4T is if they are reading 16-bits at a time.


Nah. `require`. Don't be nice with a `false`.


Самым оптимальным решением было бы такое:
```ruby
alphabet = 'a'..'z'
vowels = %w(a e i o u y) # короткая форм записи массива, состоящего из строк
result = {}
alphaber.each.with_index(1) do |letter, index|
  result[letter] = index if vowels.include?(letter)
end
```
Преимущество в том, что мы не считаем индекс вручную.


This test should have come earlier, and done a `revert()` with gusto.


outra coisa, não precisa ser .php, pode ser .html mesmo....


@moethu could you also update the zendesk change log when pushing releases and hotfixes?


You are complicating things. It shows your data structure is maladapted.


I agree, having a comment about that somewhere in here would have been very useful.


and const > let
however, not all browser support let and const


How would you prevent the same secret words from being reused.


It’s a bad practice to include generated files in repository. There’s GitHub Releases and npm.


Does this test ever fail?


Lol yeah not sure that I understand it, but Im doing it!

On Thu, Aug 24, 2017 at 14:52 Elisha Garcia <notifications@github.com>
wrote:

> heyyyy. You're looping now :p
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/maxfieldegaj/web-workbook/commit/4f4eac185875d38d12040cd66db38a84a0147643#commitcomment-23854137>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AcSghXlQGySRXG3jZWCC9qfW2CYFaBtKks5sbdR_gaJpZM4PB2u0>
> .
>


See #1166


Do you know of some hidden list of the timings of these? Mike was very confident about it being 4T


This is the very thing I am trying to figure out how to do.


Transfer in a loop? DoS attack. Kill is a radical move and is not just a "flushing" of the remittances. It goes to the owner, boom. If you want a flushing of remittances prior to the kill, then you make an owner-only function that returns __one__ remittance. You can also make your `claimUnchallengedEther` function take a parameter to allow good samaritans to flush the expired ones.


Added Slice, Drop, DropRight, Take, TakeRight and began zip.


Move ``return`` here, then you won't need the ``else`` statement.


That can be added if someone wants it


Commit changes to helloWorld.java


Is that intended to have those escapes?


Initial Comment for helloWorld.java


command line argument for slowdownspeed should now be entered as an integer rather than alphanumeric. Previously you'd need to enter "F360". Now you enter "360"


@yarikoptic No idea. It would be a decision/problem in `tools/cast2script`


Adding some comments too


No need to leave the comment, you can delete the line.


TeamCity Code :: 1. Development :: DDK :: Defect [Build 117](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=29620&buildTypeId=Code_Development_Ddk_Defect) is now running


Do we need this function? Doesn't smsCompliance replace it?


@yarikoptic Found it; I can fix it and PR it to datalad/datalad


See #1150


Nice refactoring :)


[R/RcppExports.R:26:1:](https://github.com/achubaty/grainscape/blob/441f27c2ad3f3cf3e427950112221611201e547b/R/RcppExports.R#L26) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-MPG.R:73:4:](https://github.com/achubaty/grainscape/blob/441f27c2ad3f3cf3e427950112221611201e547b/tests/testthat/test-MPG.R#L73) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:83:4:](https://github.com/achubaty/grainscape/blob/441f27c2ad3f3cf3e427950112221611201e547b/tests/testthat/test-MPG.R#L83) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:93:4:](https://github.com/achubaty/grainscape/blob/441f27c2ad3f3cf3e427950112221611201e547b/tests/testthat/test-MPG.R#L93) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-MPG.R:101:4:](https://github.com/achubaty/grainscape/blob/441f27c2ad3f3cf3e427950112221611201e547b/tests/testthat/test-MPG.R#L101) *style:* **Commented code should be removed.**
```r
  #g <- simplify(g, remove.multiple = TRUE, remove.loops = TRUE)
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


It seems to me that you did the right changes,
but Scss wasn't converted to the CSS, as in css it still has previous address


TeamCity Code :: 1. Development :: DDK :: Defect [Build 117](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=29620&buildTypeId=Code_Development_Ddk_Defect) outcome was **SUCCESS**
Summary: Tests passed: 70 Build time: 00:01:32


Created multidev environment [ajm-vanilla4#ci-11](https://dashboard.pantheon.io/sites/40d5b393-e81f-451e-a991-e2406d0ad7ec#ci-11).

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


149092157-AERC_updates_to_BSDF (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (2386 of 2784 tests passed, 135 test warnings)

Messages:

 * 532 tests had: EIO diffs.
 * 167 tests had: ESO small diffs.
 * 356 tests had: ESO big diffs.
 * 192 tests had: Table small diffs.
 * 157 tests had: SSZ small diffs.
 * 240 tests had: ZSZ big diffs.
 * 307 tests had: Table big diffs.
 * 57 tests had: ERR diffs.
 * 85 tests had: SSZ big diffs.
 * 50 tests had: MTR big diffs.
 * 59 tests had: MTR small diffs.
 * 55 tests had: ZSZ small diffs.

Failures:

EnergyPlusFixture Test Summary
 * Passed: 521
 * Failed: 1

regression Test Summary
 * Passed: 229
 * Failed: 397

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


AddressProblemUnitTests (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2738 of 2738 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)


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2140 of 2141 tests passed, 0 test warnings)

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


graveyard code though.... seriously going to use that.


I see! But in `R`, why is `capture.output()` needed?


149092157-AERC_updates_to_BSDF (vidanovic) - 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.png)


@yarikoptic PR opened datalad/datalad/pull/1759


I'm getting a build error from this:

/home/pieper/slicer4/latest/Slicer/Modules/Loadable/Segmentations/qSlicerSegmentationsModuleWidget.cxx: In member function \u2018void qSlicerSegmentationsModuleWidget::onAddSegment()\u2019:
/home/pieper/slicer4/latest/Slicer/Modules/Loadable/Segmentations/qSlicerSegmentationsModuleWidget.cxx:405:7: error: \u2018segmentationNode\u2019 was not declared in this scope
   if (segmentationNode->GetSegmentation()->GetNumberOfSegments() > 1)
       ^


accepted


149092157-AERC_updates_to_BSDF (vidanovic) - i386-Windows-7-VisualStudio-14: Tests Failed (2351 of 2750 tests passed, 134 test warnings)

Messages:

 * 532 tests had: EIO diffs.
 * 358 tests had: ESO big diffs.
 * 167 tests had: ESO small diffs.
 * 191 tests had: Table small diffs.
 * 159 tests had: SSZ small diffs.
 * 240 tests had: ZSZ big diffs.
 * 308 tests had: Table big diffs.
 * 83 tests had: SSZ big diffs.
 * 59 tests had: ERR diffs.
 * 51 tests had: MTR big diffs.
 * 60 tests had: MTR small diffs.
 * 55 tests had: ZSZ small diffs.

Failures:

EnergyPlusFixture Test Summary
 * Passed: 521
 * Failed: 1

regression Test Summary
 * Passed: 211
 * Failed: 398

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


minor adjustments to wording/syntax


Thanks for the catch! Fixing it right away


Very interesting how the text its shrinking when I resize the window.
The background its really nice too.


Prepare Release 1.26.0 (#1494)

Preview: documentation
Coverage: core | datetime


- This bug has been fixed: https://trello.com/c/BM5x967N/1035-when-you-try-to-link-a-channel-the-picker-stays-after-you-finish-making-it-impossible-to-send


@yarikoptic webiste fixed as well. Regenerated all .sh files; only this one was affected.

Good catch.


What is this?


I removed some useless Miscellaneous colormaps and changed default colormap in commit c1e82f0.
These default choices remain subjective, but these variables can be customized by the user, as it may vary depending on the variable to plot (bottom, water depth, velocities), the target or display (within a publication, presentation, ...).


I agree but I think we should do it in another pull request since this file is already generated on `master` branch: https://github.com/asciidoctor/asciidoctor-reveal.js/blob/master/dist/main.js


los cambios se realizaron con éxito


wow


hi @lukeyeager, I don't think it's related but I'll take a look


If it's not a lot of trouble, I think it'd be worth trying bumping up the buffer to 480/576 all the time, since in my experience, the RDP is very performant on even modest GPUs, and the RSP is by far the bottleneck. If it fixes a stack of games without per-game hacks or heuristics at the cost of <20-30% performance, I'd say it's worthwhile, at least unless/until we find a better solution.


@taylorbarstow @ZECTBynmo Could you guys take a look. Tell me if it looks ok to you? I ran some tests locally it does not appear to be affecting Hubspot connector for me. Will Run more refresh tests tomorrow.


Solves #8


delete


Solves #3


Configuration a little further down contradicts this suffix.

```
 - provider: GitHub
   tag: Rubberduck v$(appveyor_build_version)
   release: Rubberduck v$(appveyor_build_version)
   description: AppVeyor build on [master] - https://ci.appveyor.com/project/retailcoder/rubberduck/build/$(appveyor_build_version)
   auth_token:
     secure: oh0SRnZVt0ABeWqFr7ut5TZkxFJjHmS/DZnJnp2HErQTCmX3O8RASJH3ZiYl11gz
   artifact: Installers\Rubberduck.Setup.exe
   draft: true
   prerelease: false  <<<<<<<
   on:
     branch: master
```


These functions are only called from inside a transactions os this change is not needed


Solved
https://github.com/cptechinc/soft-crm/issues/3


I would avoid cyclic dependencies between modules. As an example, `perc` uses `perc_reduce` which is also uses `perc`. Rationale is that your modules are more tightly coupled which can make it harder to write tests and other annoyances. In this case, it seems like the `gen` record related functionality could be fairly easily moved to their own module.


Solves #3
https://github.com/cptechinc/soft-crm/issues/3


Created multidev environment [wiegele#ci-59](https://dashboard.pantheon.io/sites/e5d4b65a-3be9-4852-bb2f-7b73d4f5eeaf#ci-59).

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