redhat version fix

The redhat version numbers now have the os version appended, which when combined with this version number validation means that 0.28.0-1.el7 is rejected by the module, but 0.28.0-1 won't select the package correctly from the yum repo.
The options are: either remove the version number validation in init.pp, or transform the centos version number to include the .el5, .el6 or .el7 suffix.
In the interests of making it easier to have one version that applies to all environments (debian, redhat, windows, etc) I've opted for the latter.
I'm not sure what side-effects this might have, so let me know if you think this is a terrible idea.

Nice! I couldn't figure out why my runs were failing after updating the repo with yum seeing the new software fine.
The only piece I had to make a modification was here:
default     => "${sensu::version}.el${operatingsystemmajrelease}"
Not sure if my puppet version (1.7.1) is causing it, but I don't have a os_maj_version fact.

@rgeniesse cheers for the heads up! $::operatingsystemmajrelease seems to work fine for me as well so I'll use that instead 👍

blurgh. I feared this might happen. I'll take a look and see if I can work out whats wrong

@andyroyle @rgeniesse sorry for the confusion here, allow me to explain.
Sensu 0.27 is the first Sensu release built by us using Chef's Omnibus tools. One effect of this change is that version strings in the RPM metadata now include a .elX suffix, where X is the major version of Redhat. Switching to Omnibus builds also necessitated this change of repository structures, and in our restructured repository we have renamed old package files for consistency, but have not modified the old package version strings to add the new suffix.
This means

From 0.27 onward, both file names and version strings in RPM metadata will include the .elX suffix
Versions prior to 0.27 include the .elX suffix in their file name but do not include the suffix in RPM metadata

In the Sensu Chef cookbook I've implemented a helper method for generating appropriate version strings, perhaps a similar implementation might be useful here.

Thanks @cwjohnston, that clears up the behavior I am seeing. I just upgraded all my Sensu agents today, mix of CentOS 5/6/7, using the code @andyroyle provided. They all worked.
My only issue now is we have a few Amazon Linux nodes out there, but I haven't had time to dig into how to resolve those. Maybe a fact override somewhere. Or maybe they'll just have to sit on 0.26.5 for now.
Open to any final solution and happy to test people's ideas.

This looks great to me. Can you fix the conflicts?

rebased 👍

@andyroyle @jaxxstorm I think merging this has broken tests introduced in https://github.com/sensu/sensu-puppet/pull/645/:
  1) sensu package setting version with redhat platform version suffix should contain Package[sensu] with ensure => "0.29.0-1.el7"
     Failure/Error:
       it { should contain_package('sensu').with(
         :ensure => '0.29.0-1.el7'
       ) }

       expected that the catalogue would contain Package[sensu] with ensure set to "0.29.0-1.el7" but it is set to "0.29.0-1.el7.el7"
     # ./spec/classes/sensu_package_spec.rb:53:in `block (5 levels) in <top (required)>'

Automatically appending the OS major version without the ability to override it makes it impossible for folks like @rgeniesse to install the el7 package on many RedHat derivatives, e.g. Amazon Linux where the osmajorversion would be 2007.
Changes in #645 were intended to allow the version parameter value to include the elX suffix so that users of RedHat derivatives can specify the elX suffix that aligns with their platform's major version . Maybe there's some middle ground possible between these two approaches?


WiFi respones

This plug in is very nice, thank you for working on it.
Wanted to know if you have your TV connected with Wireless or cabled with ethernet?
I have mine just connected with Wifi, the TV will turn on but it will not respond that it turned on or not.

Hi,
i have my tv connected over WiFi and everything works fine. Could you more describe your issue?

Sure.
After configuring the MAC address and IP, which i have set to a static on my router, when I try to turn the tv on with the home app on iOS. The TV will turn on but it will not respond and show that it is on in the home app. Trying to change the volume just shows no response.
If the tv is turned on with the remote and I look in the home app it is not showing as on.

Thanks for the info. Well, i can't reproduce such issues on my device but i will have a look at it and see if i can do something about it. But at first it looks more likely to me like an issue with your setup.
Could you check if the same happens over LAN cable?

In finally got some time to work on this, and I did get it working noticed that the TV was prompting from accepting permission. Once approved it worked great. But I have found it is prompting each time the server reboots. Do you know how to find the keyFile? I'm hoping adding the keyFile will stop the prompts each time. I was looking in the TV settings with out luck and searching the internet on how to find it.
Thank you.

Hi,
thanks for letting me know.
You can configure the location where you want to save the keyFile in the config, just use the keyFile field for that and specify a path.
Greets


Bluetooth: tests: shell: br : increase rfcomm dlc stack size

For the shell RFCOMM case, these is a stack overflow problem when do disconnect, so need to increase the CONFIG_BT_RFCOMM_DLC_STACK_SIZE. After testing, the stack here requires at least 388 bytes. With an additional 10% buffer, so set this size to 420.

How tight on memory are you with the platforms that you're targeting for? Generally, it's better to avoid special-casing applications and having conditional defaults, if a single default value can cover most cases. Maybe consider removing the conditional default for HFP in subsys/bluetooth/classic/Kconfig and just have a single default of 512? (again, I don't know the memory situation for your platforms, so please correct me if you think this is a bad idea)

@jhedberg My platform is based on NXP's EVK1170, which uses an ARM Cortex-M7 core.
With a default single-core configuration of 512 meet my requirements.  And @lylezhu2012 How about this proposal using single default of 512 by removing the  conditional default for HFP ?

And @lylezhu2012 How about this proposal using single default of 512 by removing the conditional default for HFP ?

Yes. I think we can do it.

And @lylezhu2012 How about this proposal using single default of 512 by removing the conditional default for HFP ?

Yes. I think we can do it.

OK, I will update the PR with the proposal


GitLab pipelines issue running nitric up on shared runners

Bug Report
Issue
On GitLab pipelines, with shared runners there are issues connecting to the containers.
Steps
Steps to reproduce the behavior:
With latest docker (27~) there are connection errors.
deploy:
  image: docker:latest
  services:
    - docker:dind
  rules:
    - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
    - if: $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
  variables:
    PULUMI_CONFIG_PASSPHRASE: $PULUMI_ACCESS_TOKEN
    PULUMI_ACCESS_TOKEN: $PULUMI_ACCESS_TOKEN
    AWS_ACCESS_KEY_ID: $AWS_ACCESS_KEY_ID
    AWS_SECRET_ACCESS_KEY: $AWS_SECRET_ACCESS_KEY

  before_script:
    - apk update && apk add --no-cache curl bash
    - curl -L https://nitric.io/install?version=latest | bash
    - export PATH=$PATH:$HOME/.nitric/bin
    - curl -fsSL https://get.pulumi.com/ | bash
    - export PATH=$PATH:$HOME/.pulumi/bin
  script:
    - nitric up --ci

With docker in docker recommended version docker (24~) there are resources that are not deployed. (i.e. API gateway)
deploy:
  image: docker:24.0.5
  services:
    - docker:24.0.5-dind
  rules:
    - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
    - if: $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
  variables:
    PULUMI_CONFIG_PASSPHRASE: $PULUMI_ACCESS_TOKEN
    PULUMI_ACCESS_TOKEN: $PULUMI_ACCESS_TOKEN
    AWS_ACCESS_KEY_ID: $AWS_ACCESS_KEY_ID
    AWS_SECRET_ACCESS_KEY: $AWS_SECRET_ACCESS_KEY

  before_script:
    - apk update && apk add --no-cache curl bash
    - curl -L https://nitric.io/install?version=latest | bash
    - export PATH=$PATH:$HOME/.nitric/bin
    - curl -fsSL https://get.pulumi.com/ | bash
    - export PATH=$PATH:$HOME/.pulumi/bin
  script:
    - nitric up --ci

I have also tried adding these variables, which do not fix any of the above:
DOCKER_HOST: tcp://docker:2375
DOCKER_DRIVER: overlay2
DOCKER_TLS_CERTDIR: ""

Expected
It should work.
Environment and setup information

Nitric CLI Version: v1.50.2
Operating System:
Nitric dependencies and their versions, such as SDKs or Middleware: Python SDK
Cloud providers you are deploying to and their version<EMAIL_ADDRESS>
Other info
Errors with latest docker:

False success with docker 24 (missing API gateway output/deployment):

I have done some investigating and the WSL fix of using the eth0 interface IP does fix this issue. The question is ensuring it does not break other environments.


filename & directory names maybe should a little bit shorter

fatal: cannot create directory at 'spec/vcr/GitHubChangelogGenerator_OctoFetcher/_fetch_closed_issues_and_pr/when_API_ca
ll_is_valid/returns_issue_with_proper_key': Filename too long

Thanks for reporting.
This is a VCR issue. We’re auto-naming these.
Perhaps there’s a way to avoid making long names!


Add choosealicense-cli, isitup and licen

ping @alebcay  Hi, I've added some small stuffs written by myself, please have a check, feel free to remove any of them.
ping @xuxiaodong I've also updated the chinese edition, please have a check.

English changes look okay, waiting on approval for Chinese changes.

No problem.


KAFKA-15853: Move AuthorizerUtils and its dependencies to server module

Blocker for #15103 - Moving AuthorizerUtils and Session into server module
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi @mimaison can you review this pr please?

It's next on my list once https://github.com/apache/kafka/pull/15246 is merged. It will probably require you to rebase (sorry).

FYI I merged https://github.com/apache/kafka/pull/15246 and this indeed needs rebasing

Just rebased, it was an easy one


Peer to Peer Code Review: Module 2 Week 1 (refactor to use JavaScript classes)

Hi @kal2944 👋

The project is almost complete now, great job!! But there are a few things to consider in order to make your project even more similar to the wireframe design.

The 'remove' buttons don't have a shadow cast as shown in the wireframe design.
The font and overall design must match the wireframe design given.

Again, Great Job🥇 for your amazing work!
Debby

well, Noted! thank you so much!


how can I upload single file?

...using your utility?
src: [     'conf.html' ]
does not work

This has been answered numerous times.
Here's one of the answers: https://github.com/zonak/grunt-ftp-deploy/issues/102


"light" switch in home kit does not keep "on" state when used with "special device" "http call"

(Thank you very much for such a useful project)
Please describe your problem
Which Homematic actor/sensor you want to control ?_
I want to control a third party device using http call via Node-Red running on CCU3.
Which settings y've choosen for this actor/sensor in hap-homematic ?
I added "special device" http according to https://github.com/thkl/hap-homematic/wiki/special-devices#http-call
What happens
Device appears in Home Kit just fine (e.g. as "light bulb" if such type is chosen) and action is triggered when tapped on it however after a ~second device in Home Kit returns to the initial state (e.g. off).
The problem is I don't know what value/json should the http call return to modify the state of the actor.
I tried returning True, 1, ON and also JSON {"on":true, "off":false} but the result is always the same: after a second device's state in Home Kit returns to "off".
What should be happend
If "special device" of the http call type supports changing the state of Home Kit device it is mapped to the documentation should mention this and provide example of the value the http call should return. Or if changing state is not possible, documentation should mention such limitation.
Since hap-homematic does a GET call on the endpoint configured in http call I assume there is no separate call for getting current state (e.g. GET) and to issue an action (e.g. POST).
Steps to reproduce this issue
I will add interaction steps in debug mode if needed but please, if possible, explain first if http "special device" supports changing permanently the state of the mapped Home Kit device - because if not then this issue should be a feature request (apart from remark on documentation of the http call "special device" being too brief).

Hi, you are right, there is documentation missing. I struggled today with a POST request that include a Bearer token (part of the header information when sending the request via curl). No idea where / how to put this into the setup.
Maybe @thkl or @sailor79 can give some hints / answers to above request.

Currently the http service will not react on the result of the call. The isOn Request from HomeKit will be served always with "false". So that is the reason why your light icon will turn off after HomeKit sends the next query.
The http service was designed as "a Push to action".
For implementing a valid state getter there has to be a second url for requesting the current state of the device and maybe parsing logic to get a valid state from the http request.

Thanks Thomas, I defined my use case per CCU virtual device + script.

Thank you for all the answers. I ended up using node-red for home kit integration.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 414

Questions ?
Please refer to the Backport tool documentation


Use clown

I just rebased on master, restored the removed env variable in .travis.yml and ran the tests. LGTM! :+1:


fix to allow docker swarm download image automatically

Hello,
Please see the feature request for more details.
https://github.com/openanalytics/containerproxy/issues/52
Thanks,
Robin

Thank you very much for this contribution! This is now part of ShinyProxy 3.0.0 which we are releasing today.
See https://github.com/openanalytics/containerproxy/commit/e6de16534ef6a29c20526a2178ecb47680c327bf
I also listed you as contributor on our release notes: https://shinyproxy.io/downloads/


hippocampal microinfarct

If this is a request for a new term, please provide us with the following information.

Preferred term label (e.g., asplenia)

hippocampal microfinfarct

Synonyms (e.g., absent spleen)

Textual definition (the definition should be understandable even for non-specialists. Include a PubMed ID or URL(s) to refer to any relevant article(s) that provides information about the suggested term).

an infarct located in the structure of the cerebrum involved with memory storage and spatial navigation that can only be observed microscopically

Any additional notes to clarify the meaning or use of the term. Please provide a PubMed ID or URL(s) for the source of this information.

https://jamanetwork.com/journals/jamaneurology/fullarticle/776636
DOI: 10.1001/archneur.57.6.869

Suggested parent term (Please look in the hierarchy of the MP using the MP Browser or other browsers such as OLS and find the place in the MP where the new term best fits)

cerebral microinfarct, abnormal hippocampus morphology

If you would like a nanoattribution, please indicate your ORCID id or provide a label such as MGI:sbello (organization, first initial-last name)

https://orcid.org/0000-0003-2307-1226

As hippocampus is defined in the parent "abnormal hippocampus morphology" I just used hippocampus in the definition of this term rather than embedding the definition of hippocampus in the definition.
MP:0021006


CLI: Develop a revised wizard for adding an interaction that is more automatic.

An improvement to the current CLI wizard for interactions need to be implemented.  Essentially the following steps should be pursued:

[x] Prompt the user for automatic or manual interaction creation
[x] For manual follow the current code path
[x] For automatic develop a new code path that somewhat mirrors what the company automated code path performs

Steps for the automated code path include:

[x] Prompt for the file to be added to the interaction, this should include some configuration for the target bucket
[x] Upload the file to the targeted bucket
[x] Fetch all companies from the backend and provide a checkbox for the user to select which company the interaction belongs to
[x] Link this interaction to the selected company
[x] With the company selected populate key information from the linked company into the interaction object.  This will largely be geographic data, linking information, phone number, etc.
[x] Fetch all studies from the backend and provide a checkbox for the user to select which company the interaction belongs to
[x] With the study selected populate key information from the linked study to the interaction object this is particular to the linked_study property
[x] Link this interaction to the selected study
[x] Prompt the user for the balance of the properties to set
[x] Register the object in the backend

Much of the logic primitives are already available to do this.

This work has also led to the improvements of the company cli by ensuring there are choices for company role and region.  It is likely additional improvements will be ported over from interactions to companies over time like the ability to pick an associated study.

This issue has been superseded by other issue in the CLI/JS API.  Closing issue.


Hot restart using anonymous routes breaks browser back button

Invoking a hot restart breaks browser back button functionality for apps using anonymous routes.
This is related to https://github.com/flutter/flutter/issues/73793 and might be related to https://github.com/flutter/flutter/issues/70299
Steps to Reproduce

Run an app using anonymous routes (example)
Press the "view details" button to invoke a Navigator.push()
Invoke a hot restart
Press the back button
See error below:

Error: Assertion failed: org-dartlang-sdk:///flutter_web_sdk/lib/_engine/engine/navigation/history.dart:239:12
_isOriginEntry(state)
is not true
    at Object.throw_ [as throw] (http://localhost:53091/dart_sdk.js:5296:11)
    at Object.assertFailed (http://localhost:53091/dart_sdk.js:5232:15)
    at _engine.SingleEntryBrowserHistory.new.[_unwrapOriginState] (http://localhost:53091/dart_sdk.js:147111:57)
    at _engine.SingleEntryBrowserHistory.new.tearDown (http://localhost:53091/dart_sdk.js:147171:79)
    at tearDown.next (<anonymous>)
    at http://localhost:53091/dart_sdk.js:38561:33
    at _RootZone.runUnary (http://localhost:53091/dart_sdk.js:38418:58)
    at _FutureListener.thenAwait.handleValue (http://localhost:53091/dart_sdk.js:33395:29)
    at handleValueCallback (http://localhost:53091/dart_sdk.js:33955:49)
    at Function._propagateToListeners (http://localhost:53091/dart_sdk.js:33993:17)
    at _Future.new.[_completeWithValue] (http://localhost:53091/dart_sdk.js:33835:23)
    at async._AsyncCallbackEntry.new.callback (http://localhost:53091/dart_sdk.js:33858:35)
    at Object._microtaskLoop (http://localhost:53091/dart_sdk.js:38676:13)
    at _startMicrotaskLoop (http://localhost:53091/dart_sdk.js:38682:13)
    at http://localhost:53091/dart_sdk.js:34209:9

Logs
[✓] Flutter (Channel master, 1.26.0-2.0.pre.251, on Mac OS X 10.15.7 19H114 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.251 at /Users/ryjohn/code/github/flutter/flutter
    • Framework revision 5bc725bea8 (4 days ago), 2021-01-08 08:12:36 +0100
    • Engine revision 2e5833b7c5
    • Dart version 2.12.0 (build 2.12.0-179.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0-rc1)
    • Android SDK at /Users/ryjohn/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 51.0.3
    • Dart plugin version 202.8346

[✓] VS Code (version 1.52.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.1

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • Mac OS X 10.15.7 19H114 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 87.0.4280.141

! Doctor found issues in 1 category.

logs
Error: Assertion failed: org-dartlang-sdk:///flutter_web_sdk/lib/_engine/engine/navigation/history.dart:239:12
            _isOriginEntry(state)
            is not true
                at Object.throw_ [as throw] (http://localhost:61163/dart_sdk.js:5306:11)
                at Object.assertFailed (http://localhost:61163/dart_sdk.js:5242:15)
                at _engine.SingleEntryBrowserHistory.new.[_unwrapOriginState] (http://localhost:61163/dart_sdk.js:147586:57)
                at _engine.SingleEntryBrowserHistory.new.tearDown (http://localhost:61163/dart_sdk.js:147646:79)
                at tearDown.next (<anonymous>)
                at http://localhost:61163/dart_sdk.js:38980:33
                at _RootZone.runUnary (http://localhost:61163/dart_sdk.js:38837:58)
                at _FutureListener.thenAwait.handleValue (http://localhost:61163/dart_sdk.js:33778:29)
                at handleValueCallback (http://localhost:61163/dart_sdk.js:34338:49)
                at Function._propagateToListeners (http://localhost:61163/dart_sdk.js:34376:17)
                at _Future.new.[_completeWithValue] (http://localhost:61163/dart_sdk.js:34218:23)
                at async._AsyncCallbackEntry.new.callback (http://localhost:61163/dart_sdk.js:34241:35)
                at Object._microtaskLoop (http://localhost:61163/dart_sdk.js:39095:13)
                at _startMicrotaskLoop (http://localhost:61163/dart_sdk.js:39101:13)
                at http://localhost:61163/dart_sdk.js:34592:9

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.334, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.334 at /Users/pedromassango/Code/flutter_master
    • Framework revision 333dc5d88a (5 hours ago), 2021-01-14 06:56:54 +0100
    • Engine revision 1b5e16cda2
    • Dart version 2.12.0 (build 2.12.0-223.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.2.5
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 87.0.4280.141

! Doctor found issues in 1 category.

logs
Error: Assertion failed: org-dartlang-sdk:///flutter_web_sdk/lib/_engine/engine/navigation/history.dart:239:12
            _isOriginEntry(state)
            is not true
                at Object.throw_ [as throw] (http://localhost:61163/dart_sdk.js:5306:11)
                at Object.assertFailed (http://localhost:61163/dart_sdk.js:5242:15)
                at _engine.SingleEntryBrowserHistory.new.[_unwrapOriginState] (http://localhost:61163/dart_sdk.js:147586:57)
                at _engine.SingleEntryBrowserHistory.new.tearDown (http://localhost:61163/dart_sdk.js:147646:79)
                at tearDown.next (<anonymous>)
                at http://localhost:61163/dart_sdk.js:38980:33
                at _RootZone.runUnary (http://localhost:61163/dart_sdk.js:38837:58)
                at _FutureListener.thenAwait.handleValue (http://localhost:61163/dart_sdk.js:33778:29)
                at handleValueCallback (http://localhost:61163/dart_sdk.js:34338:49)
                at Function._propagateToListeners (http://localhost:61163/dart_sdk.js:34376:17)
                at _Future.new.[_completeWithValue] (http://localhost:61163/dart_sdk.js:34218:23)
                at async._AsyncCallbackEntry.new.callback (http://localhost:61163/dart_sdk.js:34241:35)
                at Object._microtaskLoop (http://localhost:61163/dart_sdk.js:39095:13)
                at _startMicrotaskLoop (http://localhost:61163/dart_sdk.js:39101:13)
                at http://localhost:61163/dart_sdk.js:34592:9

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.334, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.334 at /Users/pedromassango/Code/flutter_master
    • Framework revision 333dc5d88a (5 hours ago), 2021-01-14 06:56:54 +0100
    • Engine revision 1b5e16cda2
    • Dart version 2.12.0 (build 2.12.0-223.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.2.5
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 87.0.4280.141

! Doctor found issues in 1 category.

@johnpryan Hello. The example link you provided seems to not be working anymore. Could you either provide a working minimal sample that reproduces the issue or confirm if it's been fixed?

Verified this issue by using anonymous routes code sample from https://medium.com/flutter/learning-flutters-new-navigation-and-routing-system-7c9068155ade. Followed steps to replicate using latest master version and observed that the reported error doesn't occur.
https://user-images.githubusercontent.com/67046386/223400659-2b7315af-348a-43b6-a07c-fb6eced83221.mov

 anonymous routes runnable code sample

import 'package:flutter/material.dart';

void main() {
  runApp(Nav2App());
}

class Nav2App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: HomeScreen(),
    );
  }
}

class HomeScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: ElevatedButton(
          child: Text('View Details'),
          onPressed: () {
            Navigator.push(
              context,
              MaterialPageRoute(builder: (context) {
                return DetailScreen();
              }),
            );
          },
        ),
      ),
    );
  }
}

class DetailScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: ElevatedButton(
          child: Text('Pop!'),
          onPressed: () {
            Navigator.pop(context);
          },
        ),
      ),
    );
  }
}

I am going ahead and optimistically closing this as fixed in latest versions.
Channel master, 3.9.0-1.0.pre.72


Using external POI data

How much thought have you put into using none OSM point of interest data?
I've been producing a data set of shops, it's "scraped" from there websites (only when they use structured data, and following robots.txt).
In the UK we use it as QA.
Anything in OSM but not the list is probably old OSM data.
Anything in the list but not in OSM is probably a shop but not mapped.
Copyright wise, it's not compatible with the ODbL, when editing OSM we need an alternative source, but I doubt any of these shops would complain if you used them as an overlay. You could probably even get explicit permission if you tried.

Hi, thank you for your message. We try to simplify our own database process ad keep believing OpenStreetMap is a good poi data sources. We also use TripAdvisor in UK for some categories (hotel, restaurant, activites).

My dataset is now deprecated and I work on All The Places which does similar.
I too hope OSM POIs will become the best, I work on them daily. But currently to get the best data you'd have to conflate multiple sources like you are already doing with Tripadvisor.


Fixes #606 - Add error condition to the FLOW record on disconnect.  M…

…oved the counter-flow linking to a time prior to attempting to create the connection.

Codecov Report

Merging #607 (0aa5ee4) into main (dffbdde) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #607      +/-   ##
==========================================
- Coverage   25.05%   25.05%   -0.01%
==========================================
  Files         117      117
  Lines       28951    28960       +9
  Branches     4687     4690       +3
==========================================
+ Hits         7254     7256       +2
- Misses      20727    20736       +9
+ Partials      970      968       -2

Flag
Coverage Δ

unittests
25.05% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/adaptors/tcp_adaptor.c
2.23% <0.00%> (-0.03%)
:arrow_down:

src/router_core/route_tables.c
15.92% <0.00%> (+0.27%)
:arrow_up:

src/router_core/address_watch.c
75.00% <0.00%> (+1.13%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dffbdde...0aa5ee4. Read the comment docs.


pid-file doesn’t remove on container shutdown

When i shutdown the container, the pid-file from the log-volume isn‘t removed. If i start the container back, cronicle cant start because there is already a pid-file. If i disable redirect the log-dir to my local filesystem it works.

I also have this on my Synology Diskstation.
It's quit annoying because it it means one has to manually delete that file every time.

I also have this on my Synology Diskstation.
It's quit annoying because it it means one has to manually delete that file every time.

Running cronicle in a container requires some additional adjustments, you can try this image
https://github.com/soulteary/docker-cronicle
more detail: https://github.com/jhuckaby/Cronicle/pull/80#issuecomment-986231283


opencv problem

Ok, so whenever i type in 'pip3 install opencv' this error message shows and i have no clue what it means or what i have to do. Anyone know what to do? Im on Windows 10.

did you ever fix this??

Yea i did, you type pip3 install opencv-python
And it sold work

Thanks, it worked.


There is a lack of the administrator's ability to see how many files a user has

There is a lack of the administrator's ability to see how many files a user has
Better yet, if administrators could view lists of user files and grant permissions
But the lack of this function is especially critical when transferring files from one blocked user to another active user - as a result, the administrator does not understand whether the blocked user had files, how many there were, and whether they were successfully transferred to the active user?

You have already been answered on our forum. Please don't post the same thing on all our resources.

Thanks. I'm sorry


Adds on-close to pf-notification-drawer component

closes https://bugzilla.redhat.com/show_bug.cgi?id=1554809
does not modify default behavior of clicking the bell to open and close tray
So we have this real narrow edge case of 20-30 pixels where yah can end up with this view:

In efforts to avoid this purgatory, unable to close notifications, proposing we do this, add a notification list close button (what this pr does):

cc @serenamarie125 yah ok with this change?

@serenamarie125 Yep ! just adding it, didn't know it was a recent change! why yah were tagged in, goooood to know 👍

@miq-bot add_labels ux/approved

@miq-bot remove_labels ux/review


Configuring Ollama server URL

I see here that the Ollama server URL is hard coded. Would you consider adding support to connect to an Ollama server that isn't hosted on the default URL?

Yes. I am planning for that.
It will be in the shape of "resolver" a little similar than the Auth model. Might be like a "endpoint resolver."
This way, combined with the "model mapper," dev should be able to route any model to any adapter to any endpoint.


repo settings: Enable "delete head branches" on PR merge?

Suggestion details
Wonder it it might be a good idea to turn on "delete head branches" in the repo settings.
I noticed when my PRs are merged, my branches hang out there:

With my projects, we frequently turn on "delete head branches" which really helps in terms of the collaborators within the project to reduce branch complexity, but frequently even with open source contributors, to just help reduce later technical debt on their stale branches.  It doesn't yet look like this repo is using any fancy branching strategies (dev, release, etc) but even if that's done later, if those get protected, delete head branches feature won't touch those so they can ensure those aren't affected (accidentally deleted with a PR merge for example).
Just a possible suggestion!

Additional details
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-the-automatic-deletion-of-branches

Hmm it was already on 🤔

I think it might not work on forks, it seems to work on local branches https://github.com/pyrsia/pyrsia/pulls?q=is%3Apr+is%3Aclosed+label%3Adependencies


[10.x] Fix kebab-case attributes not passed to children component data

Fixes #48956.
Today, when doing <x-test {{ $attributes }} />,  kebab-cased attributes are ignored (for component class constructor parameters and anonymous component @props)
Class components
For compiled class components I have added ->toProps() to attributes array which convert attribute names to camelCase (diff). Kebab-case attributes are now passed to component constructor parameters. I have updated a test case to assert the data passed to Component::resolve() method.
Anonymous components
For anonymous components I have updated the data() method to transform attributes name to camelCase. I have added a test case for this method.
Attributes cleaning
In compiled component tags, I changed except() to exceptProps() to remove kebab-case attributes based on component constructor parameters (diff). I have updated a test case with user-id instead of userId to ensure kebab-case attributes are removed.
Naming considerations
The  new ComponentAttributeBag::toProps() method may be not explicit enough, I can rename it if needed.

See my comment here: https://github.com/laravel/framework/issues/48956
I'm afraid this will break all sorts of other use cases people may already be doing.

@taylorotwell Noted. To me it was a needed fix for the "attributes spread" feature, it's seems non-complete by supporting only single word attributes. Related PRs :

https://github.com/laravel/framework/pull/36240
https://github.com/laravel/framework/pull/35742


Added Outzone and Turbo OutRun support

Just wanted to let you know I called the longtrack lankhor1_longtrack as there is another variant I will be adding in the next week, which I will be calling lankhor2_longtrack


Specify correct dependency versions for 350 build

Closes https://github.com/NVIDIA/spark-rapids/issues/9263
We need to specify the slf4j version per shim, otherwise we can get duplicate versions and that breaks logging in the unit tests.
I also noticed that the parquet-hadoop version needed updating.

build


feat: #546 Make mysql pvc volume name configurable

Description of the change
Adds the ability to optionally configure the volume name for the mysql pvc. This allows helm upgrade --force to be used after the initial deployment.
Existing or Associated Issue(s)
Closes #546
Additional Information
Linting charts...

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 trillian => (version: "0.3.0", path: "charts/trillian")
------------------------------------------------------------------------------------------------------------------------

"sigstore" already exists with the same configuration, skipping
Linting chart "trillian => (version: \"0.3.0\", path: \"charts/trillian\")"
Checking chart "trillian => (version: \"0.3.0\", path: \"charts/trillian\")" for a version bump...
Old chart version: 0.2.5
New chart version: 0.3.0
Chart version ok.
Validating /Users/jonathand/src/helm-charts/charts/trillian/Chart.yaml...
Validation success! 👍
==> Linting charts/trillian
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed
------------------------------------------------------------------------------------------------------------------------
 ✔ trillian => (version: "0.3.0", path: "charts/trillian")
------------------------------------------------------------------------------------------------------------------------
All charts linted successfully

Checklist

[x] Chart version bumped in Chart.yaml according to semver. Where applicable, update and bump the versions in any associated umbrella chart
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[x] JSON Schema generated.
[x] List tests pass for Chart using the Chart Testing tool and the ct lint command.

thanks, have small nit request and did you tests the force with this change? can you provide a log or something?

I did test with the change and it seems to work - sort of. I was kind of hoping if you provided a volume name that it would dynamically create the volume when necessary with that name, but that doesn't appear to be the case. Instead, when a volume name is given, it expects that volume to already exist. So as long as you have a volume with that name - it works. If you don't - the pvc stays in a pending state and the app never comes up.
I could add a persistent volume resource to the chart and have it be created conditionally. Or just leave the expectation that if a volume name is specified then something else needs to create the volume in order for the chart to work. Any strong opinions?

Well.... I ran into other problems manually creating the persistent volume in order to pass in a volume name. I'm going to go ahead and abandon this PR and corresponding issue in favor of not using the --force flag. That seems to be a better approach.


[AppleAppStore] 401 Unauthorized

The use of the bearer token isnt working anymore.
[error] => UnexpectedResponseException (401): Unexpected response from upstream
    [context] => Array
        (
            [id] => 310633997
            [p] => ipad
            [country] => US
        )

@captn3m0 ping


Update README.md - Add Admin environment variables

Add

GITNESS_PRINCIPAL_ADMIN_EMAIL
GITNESS_PRINCIPAL_ADMIN_PASSWORD

to the docker run command. Without this, the admin user is not setup

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Use notifications to enable features

This issue is to discuss and gain consensus on the concept of using notifications (similar to the alert system) to activate features instead of using a hardcoded blockheight in the source.
I propose that:

We use a message to activate features instead of hardcoded source
Said message may only come from Exodus
Said message will contain the transaction type, version, and blockheight for go-live
Said message can be superseded by a new activation message with a later blockheight, in the event that we discover a critical issue and need to push back activation

This provides us with more granular control of feature activation, but most importantly allows us to do releases more frequently without always guaranteeing everything "beforehand" as it were.
Thoughts please :)

There should be a hardcoded minimum/grace period, between the time the notification was received and the activation, so that users are guaranteed to have a chance to update, and as security buffer, in case keys got compromised. 1000 blocks?

NACK on Exodus, at least at this very moment: I'm still waiting to get a contact info, to fill the blanks for the alert system. Other than that though, Exodus would be suitable.

I suggest to move the following into rules.cpp:
- enum TransactionType
- enum BLOCKHEIGHTRESTRICTIONS
- bool IsAllowedInputType()
- bool IsAllowedOutputType()
- bool IsTransactionTypeAllowed()

Maybe the Exodus address and marker, too?
Once done, the height restrictions could be transformed into a datatype, which allows inserting new entries.
Ideally, rules are "per network", i.e. "main", "test", "regtest".

I'm -1 on this idea because it seems too centralized of a solution. I realize there is already of level of centralization in the software release process, but this makes it explicit in the protocol.
I'm wondering if there is some variation of an idea that @zathras-crypto brought up in the all-hands that might work. That idea was using the version number of a particular transaction type.  For example once 90% of simple sends are using version n of that transaction a new feature will become enabled within x blocks.

I'm -1 on this idea because it seems too centralized of a solution.

Did you know about the alerting system, which was deployed with 0.0.9? It can be used to show messages in the status bar, and remotely shutdown (outdated) clients.. :S
It's not a good feeling to have that power, but the usefulness outweights the cost in my opinion. To make it less intrusive, there is the command-line option -overrideforcedshutdown, as well as options to add or ignore  "alert senders" (via #68).
Now regarding the transaction enablers: it's equally centralized than hardcoding a number, but it has the potential to prevent disaster, and similar to the alerting system, users should have the option to add or ignore such messages.
I like this idea of signaling via some mechanism, though I think transaction volume can very, very easily be exploited.

equally centralized than hardcoding a number

With a hardcoded block number there are two checks on centralization that don't exist with a signed transaction enabler:

Users must voluntarily install the version with the new feature and activation block number.
Anyone can fork the software and make a version with a particular feature and activation block number.

In a technical sense, those two checks remain with transaction enablers, but we've now set an expectation among users that the enablers are the way to do upgrades, pushing them further towards centralization.

Sorry, what are you referring to with "signed transaction enabler"? "Enable via notification" or "enable by send-version-signal"?

Users must voluntarily install the version with the new feature and activation block number.

And likewise users must install an updated client explicitly, which supports new features, which are enabled via notifications.

Anyone can fork the software and make a version with a particular feature and activation block number.

... but we've now set an expectation among users that the enablers are the way to do upgrades, pushing them further towards centralization

For what it's worth: anyone could still fork the software and edit the whitelisted notification sources. Or simply overwrite them via the configuration file, but I see your point.
I mean, the initial idea is certainly not intended to increase centralization, but to refine the activation mechanism. Instead of a hardcoded number, defined by a central party, the central party would have the ability to set the number on the fly. I proposed that "after the notification, there should be at least 1000 blocks" to make it less "arbitrary", and to define some boundaries.
In the long run I'd like to move into the direction of "user defined protocol settings", where users could "vote" and influence values like STO fees, but at this point I wouldn't consider something like this as feasible option to enable new transaction types at this point.

Just wanted to add a note, a good point was made about liability for whomever sends such activation messages, because that entity could be deemed as having control (and thus some liability) over parts of the Omni layer.
Thus if we do go the messaging activation route (and I still advocate we do) I think the best route would be for activation messages to come from a multsig address controlled by the Omni Foundation, such that the foundation (and not any individual developer) is the entity responsible for activating features.
Hope that makes sense :)

And @msgilligan just to re-iterate @dexX7 sentiment, everything we do is override-able by the user - we're setting the 'default case' (which is required for consensus).

multsig address

Yes, please. And if this is going to happen, then the alert keys might as well be replaced.

But just to underline: I see @msgilligan's concern, though I don't see alternatives, and thus suggest we move on with the proposal, unless you have anything else in mind than "signaling via simple-send-version".

Yes, please. And if this is going to happen, then the alert keys might as well be replaced.

I'm not so sure on the alert side - it could take an extended period of time (ie days) to contact all parties (board members? whoever is nominated I guess?) and handle signing of a transaction "from the foundation" - that's a non-issue for a pre-planned event like feature activation but in the event of something going wrong we might lose valuable time that we could be warning users...

Ahh, that's a good point.
Right now there is no handy way to create P2SH raw transactions from within Omni Core, and #77 does the trick, but I'm not sure, if this should be a priority at this point.
What do you suggest?

I still advocate for a P2SH address from the foundation for feature activation as it doesn't really matter if this takes a little time, I just meant regarding alerts specifically we should keep that a bit looser.

I started with #116, but if you already have a PoC, please don't ditch it. This is one of the things I don't see it as a bad thing, if we have more than one potential implementation.
From a higher level, I think we should specify the mechanism first. Here are a few notes, I have in mind right now:

it should probably follow the format of other transaction types

we could easily add a few lines to parse the packet (like interpret_SendToOwners())

likewise, there should be a logic handler, similar to the other logicMath_XX() methods

I suggest to use type = 65534, version = 65535, which would be similar to the notification transaction (i.e. count backwards)

What's unclear to me right now is how the actual transaction might look like. Probably something like:
[type=65534] [version=65535] [feature identifier] [activation height]

See my comment on #116 but to note:

From a higher level, I think we should specify the mechanism first. Here are a few notes, I have in mind right now:
it should probably follow the format of other transaction types

Agreed.

we could easily add a few lines to parse the packet (like interpret_SendToOwners())

Agreed.  see interpret_Activation() on my branch 0.0.10-Z-DexX-FeatureActivation.

likewise, there should be a logic handler, similar to the other logicMath_XX() methods

Agreed, see logicMath_Activation() on my branch 0.0.10-Z-DexX-FeatureActivation.

I suggest to use type = 65534, version = 65535, which would be similar to the notification transaction (i.e. count backwards)

Agreed.  CMPTransaction class has been expanded a little to handle the data for the feature ID and the activation block.

What's unclear to me right now is how the actual transaction might look like. Probably something like:
[type=65534] [version=65535] [feature identifier] [activation height]

Not probably like, exactly like my friend!! hehe :)

I've looked over #129 and it looks extremely good.

I'm wondering, if we should move away from allowing everything on testnet/regtest.
I think ideally all upcoming features should be explicitly enabled, to allow testing, and to do test activations on testnet. Similar to the alerts it should be possible to add or ignore certain senders, and the min. and max. interval on testnet and in regtest mode should be more flexible, say for example min. two days on testnet, and min. 3 blocks in regtest mode.

Now thinking a bit far in the future: if we get to a point, where black holes and feature activation fits together, and we establish a concept of "local consensus" - that means: "I only care, whether a specific action succeeds, and this can be determined with a much more limited view, and I don't to need to know everything" - then we may move into the direction of "features as plugins", which may not even be provided by the foundation/core team. Or rephrased: I imagine it might be possible to get to a point, where features are no longer hard forks, but opt-in, and this could open a lot of possibilities.

I've looked over #129 and it looks extremely good.

Thanks dude :)

I'm wondering, if we should move away from allowing everything on testnet/regtest.

I agree with this - I'm on my way out now but I'll make the appropriate changes soon...

Now thinking a bit far in the future...

That's a very interesting idea - confine certain actions to only a subset of the state - not sure how that would work in practice but definitely agree it would open up some further potential!

Upon a new feature is activated, should that display a message in the client so that the user can  accept new terms and conditions; thus reducing any liabilities by the parties activating the feature,

@stebansaa brings up an important point: we currently don't have any visual notification that a new transaction type  might be activated at some point in the future.
While I don't really see the need for another disclaimer/warning (we show one on startup on mainnet), it would probably good to know, given that, at least in the context of the MetaDEx, new features and actions are available.
Maybe we could show one info dialog, when an activation message was successfully processed, and another one, once the feature is activated?

@stebansaa brings up an important point: we currently don't have any visual notification that a new transaction type might be activated at some point in the future.

I'm not sure I see the problem - along with an activation we would send an alert out would we not?  This would display a message to the user along with shutting down their client at the live height if the client doesn't support the new feature.
Perhaps there is some crossover between notifications/alerts and activations, and I certainly don't disagree with more info for the user (such as on receipt of activation message etc) but perhaps we should think of the ideal process we want to see, and that perhaps adjust the alerts and activations to work accordingly?

The feature activation via notification was added with #159. While I think we should continue the general discussion about this mechanism, I'm going to close this issue for now, and suggest to open a new one, for the specific topic to discuss, once there is something to discuss. :)


ui-router-extras breaks my unit testing?

QUESTION:
- Why are my tests failing when ui-router-extras (not normal ui-router) is install?
- How can I use ui-router-extras and still have my tests pass?

If you want to install this quickly use yeoman + angular-fullstack-generator + bower install ui-router-extras
I found a similar issue with normal ui-router.

Luckially, ui-router normal works just fine with my testing.
After installing ui-router-extras I get an ERROR

If I uninstall ui-router.extras it this test passes just fine:

UPDATED for beforeEach module of $urlRouterProvider TEST
Heres my test:
'use strict';

describe('Controller: MainCtrl', function () {

  // load the controller's module
  beforeEach(module('morningharwoodApp'));
  beforeEach(module('socketMock'));

  var MainCtrl,
      scope,
      $httpBackend;

  // Initialize the controller and a mock scope

  beforeEach(
    inject( function (_$httpBackend_, $controller, $rootScope) {
      $httpBackend = _$httpBackend_;
      $httpBackend.expectGET('/api/things')
        .respond(['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express']);

      scope = $rootScope.$new();
      MainCtrl = $controller('MainCtrl', {
        $scope: scope
      });
    }),
    module(function ($urlRouterProvider) {
      $urlRouterProvider.otherwise( function(){ return false; });
    })
  );

  it('should attach a list of things to the scope', function () {
    $httpBackend.flush();
    expect(scope.awesomeThings.length).toBe(4);
  });
});

Here's my karma.conf
module.exports = function(config) {
  config.set({
    // base path, that will be used to resolve files and exclude
    basePath: '',

    // testing framework to use (jasmine/mocha/qunit/...)
    frameworks: ['jasmine'],

    // list of files / patterns to load in the browser
    files: [
      'client/bower_components/jquery/dist/jquery.js',
      'client/bower_components/angular/angular.js',
      'client/bower_components/angular-mocks/angular-mocks.js',
      'client/bower_components/angular-resource/angular-resource.js',
      'client/bower_components/angular-cookies/angular-cookies.js',
      'client/bower_components/angular-sanitize/angular-sanitize.js',
      'client/bower_components/lodash/dist/lodash.compat.js',
      'client/bower_components/angular-socket-io/socket.js',
      'client/bower_components/angular-ui-router/release/angular-ui-router.js',
      'client/bower_components/famous-polyfills/classList.js',
      'client/bower_components/famous-polyfills/functionPrototypeBind.js',
      'client/bower_components/famous-polyfills/requestAnimationFrame.js',
      'client/bower_components/famous/dist/famous-global.js',
      'client/bower_components/famous-angular/dist/famous-angular.js',
      'client/app/app.js',
      'client/app/app.coffee',
      'client/app/**/*.js',
      'client/app/**/*.coffee',
      'client/components/**/*.js',
      'client/components/**/*.coffee',
      'client/app/**/*.jade',
      'client/components/**/*.jade',
      'client/app/**/*.html',
      'client/components/**/*.html'
    ],

    preprocessors: {
      '**/*.jade': 'ng-jade2js',
      '**/*.html': 'html2js',
      '**/*.coffee': 'coffee',
    },

    ngHtml2JsPreprocessor: {
      stripPrefix: 'client/'
    },

    ngJade2JsPreprocessor: {
      stripPrefix: 'client/'
    },

    // list of files / patterns to exclude
    exclude: [],

    // web server port
    port: 8080,

    // level of logging
    // possible values: LOG_DISABLE || LOG_ERROR || LOG_WARN || LOG_INFO || LOG_DEBUG
    logLevel: config.LOG_INFO,

    // enable / disable watching file and executing tests whenever any file changes
    autoWatch: false,

    // Start these browsers, currently available:
    // - Chrome
    // - ChromeCanary
    // - Firefox
    // - Opera
    // - Safari (only Mac)
    // - PhantomJS
    // - IE (only Windows)
    browsers: ['PhantomJS'],

    // Continuous Integration mode
    // if true, it capture browsers, run tests and exit
    singleRun: false
  });
};

This unit test fails as soon as the $state service is initialized.  ui-router-extras initializes the $state service, which is why inclusion is breaking your unit test.
If you do not include ui-router-extras, the following line in app.js will bootstrap $state and cause your test to fail:  app.run(function($state){})  This isn't really specific to ui-router-extras.

UI-Router is synchronizing the browser url with the proper state when it bootstraps.  It is matching the 'main' state by default and, thus triggers the transition and the fetch of main's template, which $httpBackend is complaining about.
$urlRouterProvider.deferIntercept(); will cause ui-router to delay url <=> state synchronization until you call $urlRouterProvider.listen().  Put that in your unit test initialization code and you should be good to go.
@matthewharwood  please try this and verify if this works for you.

Hey @christopherthielen  I'm a super Noob with testing(literally never have written them).  That being said,  I tried to add $urlRouterProvider.deferIntercept(); and it threw the error saying
Error: Unexpected request: GET app/main/main.html
Here's how I did it
  beforeEach(
    inject( function (_$httpBackend_, $controller, $rootScope) {

      $httpBackend = _$httpBackend_;
      $httpBackend.expectGET('/api/things')
        .respond(['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express']);

      scope = $rootScope.$new();
      MainCtrl = $controller('MainCtrl', {
        $scope: scope
      });
    }),
    module(function ($urlRouterProvider) {
      $urlRouterProvider.deferIntercept();

    })
  );

On a lighter note, I was able to fix this from a 200 point bounty on stackoverflow Stack Question.
but I don't know if this is the intended method.  It honestly feels more like a hack (but again I have no idea whats going on).
  beforeEach(
    inject( function (_$httpBackend_, $controller, $rootScope,  _$state_) {
     //injecting state and spyOn(state,go)/transitionTo fixes this GET error.
      state = _$state_;
      spyOn( state, 'go' );
      spyOn( state, 'transitionTo' );
      $httpBackend = _$httpBackend_;
      $httpBackend.expectGET('/api/things')
        .respond(['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express']);

      scope = $rootScope.$new();
      MainCtrl = $controller('MainCtrl', {
        $scope: scope
      });
    }),
    module(function ($urlRouterProvider) {
      $urlRouterProvider.deferIntercept();

    })
  );

Could you explain why? Moreover, if there is a better method could you answer the stack so I can mark it correct?  #forthefuture

Here's what I changed the controller spec file from angular-fullstack to:
describe('Controller: MainCtrl', function () {

  // load the controller's module
  beforeEach(module('uiRouterExtrasKarmaBugApp'));

  beforeEach(module(function($urlRouterProvider) {
    $urlRouterProvider.deferIntercept();
  }));

  var MainCtrl,
      scope,
      $httpBackend;

  // Initialize the controller and a mock scope
  beforeEach(inject(function (_$httpBackend_, $controller, $rootScope) {
    $httpBackend = _$httpBackend_;
    $httpBackend.expectGET('/api/things')
      .respond(['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express']);

    scope = $rootScope.$new();
    MainCtrl = $controller('MainCtrl', {
      $scope: scope
    });
  }));

  it('should attach a list of things to the scope', function () {
    $httpBackend.flush();
    expect(scope.awesomeThings.length).toBe(4);
  });
});

@matthewharwood  can you check out this comment and let me know if it helps you?
https://github.com/angular-ui/ui-router/issues/212#issuecomment-69974072

closing for inactivity


Support App Groups

It would be great to have some documentation on how to set this up to use app groups.

Do you mean something like this?
userDefaults.set("key2", [1, true], "group.com.company.app", (err, data) => {
  if(!err) console.log(data)         // Save success
})

^
userDefaults.set(kay, value, suitName, callback)


Suggestion:  Convert Affixes to work as Attributes as well

This would allow cross mod support as well as custom game support with commands.

Looking into that more - affixes cannot work as the raw attributes that they actually utilize, since the attributes are not actually part of the entity, and adding arbitrary attributes to entities is unsupported behavior.


9/13 Publish

@Taojunshen,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/azure.common.client_factory.yml
:white_check_mark:Succeeded

docs-ref-conceptual/python-sdk-azure-authenticate.md
:white_check_mark:Succeeded

docs-ref-services/event-grid.md
:white_check_mark:Succeeded

docs-ref-services/virtualmachines.md
:white_check_mark:Succeeded

xrefmap.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


training time for "main.py"

The paper says "The training process takes about 20 minutes each image on one NVIDIA GeForce RTX3090 with a batch size of 1."
I used two RTX3090. However, it already cost three hours and it doesn't seem to stop.
For "v1-finetune.yaml”, which paramater specifys the number of epochs in the training process?

The paper says "The training process takes about 20 minutes each image on one NVIDIA GeForce RTX3090 with a batch size of 1." I used two RTX3090. However, it already cost three hours and it doesn't seem to stop.
For "v1-finetune.yaml”, which paramater specifys the number of epochs in the training process?

As for me, if you dont not Ctrl+C, with the default yaml, it won't be stop

In main.py, change
trainer_kwargs["max_steps"] = opt.max_steps
to
trainer_kwargs["max_steps"] = trainer_opt.max_steps
you will can use max_steps in the v1-finetune.yaml file

In main.py, change trainer_kwargs["max_steps"] = opt.max_steps to trainer_kwargs["max_steps"] = trainer_opt.max_steps you will can use max_steps in the v1-finetune.yaml file

Change max_steps in v1-finetune.yaml doesn't work. Directly set trainer_kwargs["max_steps"] = xxx works for me.


Added google translate widget.

This is not ready yet!
I copied the google-widget-specific styling over from Cannabis (which looks good there), but here, it has the effect of hiding the widget.  Need some help from @kkoryaka who likely styled the Cannabis one.
I'm also not convinced my choice of a separate .scss file is a good one, but don't see an obvious alternative.

Also, curious if there is a way to fold the styling into the cagov-component itself?
Also: https://www.searchenginejournal.com/google-translate-widget-is-free-again-for-some-websites-to-use/370104/#close
This says free usage of the widget is restricted to "government, non-profit, and/or non-commercial websites whose primary focus is on COVID-19 response" (which does not describe drought).

@jbum In general, this looks good. Just need to resolve the problem with the missing sass file mentioned above.
I share your pain regarding the need to add CSS override files; I've spent a lot of time over the past couple weeks doing the same, and then refactoring the design-system component to pay down that debt. Happy to show you how I approach this if useful.


Feature/2020.07.30/telemesh mesh v2 code coverage

Description
Converted app compact to  androidx and added new features including one-to-one content sharing and Group messaging (Create group, add/remove members). Also added Mesh v2 support. And also passed the travis-ci build with minimum code coverage.
Fixes # (#119 )
Type of change

[x] One-to-One content sharing
[x] Group create, add member, leave and remove  member features
[x] Mesh v2 compatibility added
[x] Android build version updated
[x] Broadcast message simplified
[x] Travis-CI build passed

How Has This Been Tested?

[x] Test A
Manually tested Group messaging and One-to-One file sharing.
Between nodes, they can send content from the chat page.
User can create a from the home page and add a member or change the group name too.

Test Configuration:

Firmware version: Android
Hardware: Android OS 7.1,9,10
Toolchain:
SDK: 30

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

This pull request has a conflict problem. And it is hard to merge. So I'm closing this pull request.


Add support for openldap plugin

Since version 5.5, there is a plugin in collectd that supports monitoring of OpenLDAP instances through the backend database monitor.
This PR adds support for configuring this plugin.

This looks fine. @ortegaga can you squash this into a single commit? Thanks

thank you @ortegaga! and thank you @nibalizer for reviews


Documentation: Default role names to map to LDAP groups

I am trying to use openLDAP groups to set roles in bookstack.
I cannot find in the documentation what the default role names are so I can create the appropriate LDAP groups that map to them.
What are the default role names? Are they:
admin
editor
viewer
public
I was looking for something in the documentation that described the default role names and permissions, but I could not fine any such documentation.
Also the LDAP documentation describes the rules for mapping names, but does not describe the default role names.
If the default role names listed above are correct, then my understanding is that if I create LDAP groups by the same name and set the variables in environment listed below, then, for example,  a user in the LDAP group "admin" would be set to the role of "admin" in BookStack.  Is this correct?
LDAP_USER_TO_GROUPS=true
LDAP_GROUP_ATTRIBUTE="memberOf"
LDAP_REMOVE_FROM_GROUPS=false

Also the LDAP documentation describes the rules for mapping names, but does not describe the default role names.

Yeah, I thought listing out the default roles could cause more confusion as they are only a default and will depend on the instance setup and somewhat the BookStack version. The rules for the mapping of names remain the same though so you can just apply the rules to the default roles.

What are the default role names? Are they:
admin
editor
viewer
public

Yes, That looks correct for a fresh install.

my understanding is that if I create LDAP groups by the same name and set the variables in environment listed below, then, for example, a user in the LDAP group "admin" would be set to the role of "admin" in BookStack. Is this correct?

Yes. Alternatively you can use existing LDAP groups and enter their names (Lower-cased and hyphenated) in the External Authentication IDs options when viewing a role in BookStack.

Will now close this


Use first AZ in list of AZ's for infra for singleton bosh director

[#145562177] - https://www.pivotaltracker.com/story/show/145562177

Fixed @krishicks


Added license header for apache 2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: (Re-)Enabled ReadMe Utility to run offline without failing

Description

Improved error handling (to not throw but handle the error)
Improved robustness for functions that try to access unavilable content
Updated diverse error messages

Pipeline Reference

Pipeline

 (unrelated, WAF)

 (unrelated, WAF)

 (unrelated, WAF)

 (unrelated, WAF)

 (unrelated, WAF)

Type of Change

[x] Update to CI Environment or utilities (Non-module affecting changes)
[ ] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

@AlexanderSehr all looks good, but 1 question I have, wouldn't we want this to fail (exit code 1 etc.) if it cannot pull these data sources?

Maybe, maybe not. Let's consider the following scenarios:

Jack works on the utility while flying (i.e., without internet access) and wants to test the changes. Turns out, Jack cannot change his changes because the readme script would not run because it cannot pull in external references. Jack is blocked and would've preferred the script just returning a warning to notify the user that the generated readme may not be complete
Sombeody accidently deletes any of the sources this scripte expects e.g., in the AVM repository because that person did not know they're used. All module owners are now blocked and cannot run their tests anymore because of the external reference being missing

(any references to real people are unintended and purely coincidental)

@AlexanderSehr all looks good, but 1 question I have, wouldn't we want this to fail (exit code 1 etc.) if it cannot pull these data sources?

Maybe, maybe not. Let's consider the following scenarios:

Jack works on the utility while flying (i.e., without internet access) and wants to test the changes. Turns out, Jack cannot change his changes because the readme script would not run because it cannot pull in external references. Jack is blocked and would've preferred the script just returning a warning to notify the user that the generated readme may not be complete
Sombeody accidently deletes any of the sources this scripte expects e.g., in the AVM repository because that person did not know they're used. All module owners are now blocked and cannot run their tests anymore because of the external reference being missing

(any references to real people are unintended and purely coincidental)

Thanks @AlexanderSehr, will any of the CI tests fail though if the README is not up to date? As before publishing we will need the README to be fully formed and not missing any parts, right?
Therefore I’m wondering if this should be added as a switch rather than a default, so folks are aware of what they are doing and they will need to re-run it to fully form it before the publishing and PR can be merged.
Thoughts?

@AlexanderSehr all looks good, but 1 question I have, wouldn't we want this to fail (exit code 1 etc.) if it cannot pull these data sources?

Maybe, maybe not. Let's consider the following scenarios:

Jack works on the utility while flying (i.e., without internet access) and wants to test the changes. Turns out, Jack cannot change his changes because the readme script would not run because it cannot pull in external references. Jack is blocked and would've preferred the script just returning a warning to notify the user that the generated readme may not be complete
Sombeody accidently deletes any of the sources this scripte expects e.g., in the AVM repository because that person did not know they're used. All module owners are now blocked and cannot run their tests anymore because of the external reference being missing

(any references to real people are unintended and purely coincidental)

Thanks @AlexanderSehr, will any of the CI tests fail though if the README is not up to date? As before publishing we will need the README to be fully formed and not missing any parts, right?
Therefore I’m wondering if this should be added as a switch rather than a default, so folks are aware of what they are doing and they will need to re-run it to fully form it before the publishing and PR can be merged.
Thoughts?

Hey @jtracey93,
that's correct. In the pipeline, due to internet connection, they will need to make sure the ReadMe was run without any errors/warnings. The static tests will fail, to be precise. 💪
That's how the script started anyways - it was just that later additions (of even more external content) were not implemented in the same way and threw errors instead of warnings. This PR just re-establishes the status-quo.
Now, we can of course change that. We'd need to add a new switch parameter like -IgnoreWarnings, -OfflineMode or something like that and slightly increase the complexity of the script as this parameter has to be passed into various functions. But it should work.
Subjectively I don't see toooo much value in it as it is very obvious if the script cannot access the internet (not the least does git show you that many parts of the readme were deleted), but I of course understand where you're coming from. (:

@AlexanderSehr all looks good, but 1 question I have, wouldn't we want this to fail (exit code 1 etc.) if it cannot pull these data sources?

Maybe, maybe not. Let's consider the following scenarios:

Jack works on the utility while flying (i.e., without internet access) and wants to test the changes. Turns out, Jack cannot change his changes because the readme script would not run because it cannot pull in external references. Jack is blocked and would've preferred the script just returning a warning to notify the user that the generated readme may not be complete
Sombeody accidently deletes any of the sources this scripte expects e.g., in the AVM repository because that person did not know they're used. All module owners are now blocked and cannot run their tests anymore because of the external reference being missing

(any references to real people are unintended and purely coincidental)

Thanks @AlexanderSehr, will any of the CI tests fail though if the README is not up to date? As before publishing we will need the README to be fully formed and not missing any parts, right?
Therefore I’m wondering if this should be added as a switch rather than a default, so folks are aware of what they are doing and they will need to re-run it to fully form it before the publishing and PR can be merged.
Thoughts?

Hey @jtracey93, that's correct. In the pipeline, due to internet connection, they will need to make sure the ReadMe was run without any errors/warnings. The static tests will fail, to be precise. 💪 That's how the script started anyways - it was just that later additions (of even more external content) were not implemented in the same way and threw errors instead of warnings. This PR just re-establishes the status-quo.
Now, we can of course change that. We'd need to add a new switch parameter like -IgnoreWarnings, -OfflineMode or something like that and slightly increase the complexity of the script as this parameter has to be passed into various functions. But it should work. Subjectively I don't see toooo much value in it as it is very obvious if the script cannot access the internet (not the least does git show you that many parts of the readme were deleted), but I of course understand where you're coming from. (:

Fair, I dont want to create extra work unless we need to. So happy to just keep that in our pocket, if we need to add it 👍 But as a alternative for now, can you add a note to the warning saying something like "you'll need to re-run this when the URLs are accessible so the CI tests will pass"?
@AlexanderSehr

(...)
Fair, I dont want to create extra work unless we need to. So happy to just keep that in our pocket, if we need to add it 👍 But as a alternative for now, can you add a note to the warning saying something like "you'll need to re-run this when the URLs are accessible so the CI tests will pass"?
@AlexanderSehr

Sure thing. Adding a code comment and will work on it when I find a moment

anybody? 😄


payment issue

hello, every time i got payment issue like this one
2017-07-19 11:00:36 [Payments]      [hush] Insufficient funds (0.00259442) to process payments (12.49980225) for 1 blocks; possibly waiting for txs.
hush-cli getwalletinfo
{
"walletversion": 60000,
"balance": 0.00259442,
"unconfirmed_balance": 0.00000000,
"immature_balance": 12.50030000,
"txcount": 138,
"keypoololdest":<PHONE_NUMBER>,
"keypoolsize": 101,
"paytxfee": 0.00000000
}
hush-cli listunspent
[
{
"txid": "d281fd212b3396c8f7bb56dfe817b7f19515cfcb58ab8d6270964a59974c4429",
"vout": 0,
"address": "t1WrJucfj3ySefztzjY3dhARKGDu6HcQjy6",
"scriptPubKey": "76a9148e5af22e64e4ce8d03b8d6a43daff513146408f688ac",
"amount": 0.00259442,
"confirmations": 731,
"spendable": true
}
]

Mine does the same exact thing, Payments do finally go through at about 140-150 confirms. But it does not show it is sheilding.

2017-07-19 15:53:04 [Payments]      [hush] Insufficient funds (0.00259442) to process payments (24.99970225) for 2 blocks; possibly waiting for txs.
hush-cli listunspent
[
{
"txid": "d281fd212b3396c8f7bb56dfe817b7f19515cfcb58ab8d6270964a59974c4429",
"vout": 0,
"address": "t1WrJucfj3ySefztzjY3dhARKGDu6HcQjy6",
"scriptPubKey": "76a9148e5af22e64e4ce8d03b8d6a43daff513146408f688ac",
"amount": 0.00259442,
"confirmations": 841,
"spendable": true
},
{
"txid": "04af7a196171b43ac03e9ae29ec6087b370b6668bcddf0c79335376d6a60206a",
"vout": 0,
"address": "t1Pd626U121hzHCT7VtGZBLVXSmvFgTDW35",
"account": "",
"scriptPubKey": "76a9143f11e64100b775c8e2694ce7782151b325cf3bad88ac",
"amount": 12.50000225,
"confirmations": 58,
"spendable": true
},
{
"txid": "1cf0980ab8b91fce88f39bacf8cbeba062e1fc8b1b720dfe1d7cfb5273fcc3e3",
"vout": 0,
"address": "t1Pd626U121hzHCT7VtGZBLVXSmvFgTDW35",
"account": "",
"scriptPubKey": "76a9143f11e64100b775c8e2694ce7782151b325cf3bad88ac",
"amount": 12.50000225,
"confirmations": 57,
"spendable": true
}
]
hush-cli getwalletinfo
{
"walletversion": 60000,
"balance": 25.00259892,
"unconfirmed_balance": 0.00000000,
"immature_balance": 0.00000000,
"txcount": 141,
"keypoololdest":<PHONE_NUMBER>,
"keypoolsize": 101,
"paytxfee": 0.00000000
}

@Tmotta I would guess yours is due to blocks per transaction. Default it 3, so if you have 1 block nearing payment and a new block matures neither will payout until the 2nd block is matured. Default being 3 you could have up to 3 blocks waiting to be paid

@madbuda is there something I can do that shows the block as sheilding? I thought it was supposed to Show on webpage stats it was sheilding like it shows paid?

@Tmotta the blocks show on the graph stats page with red text and the number of confirmations they have.
Possibly do some math based on how many blocks you pay together and how close each immature block is?? you can source the data from it.stats.pools[pool].pending.confirms

i set it to 1 block per payment right now, and still got same issue.
2017-07-20 08:32:11 [Payments]      [hush] Insufficient funds (0.00289124) to process payments (12.50005144) for 1 blocks; possibly waiting for txs.
hush-cli listunspent
[
{
"txid": "d281fd212b3396c8f7bb56dfe817b7f19515cfcb58ab8d6270964a59974c4429",
"vout": 0,
"address": "t1WrJucfj3ySefztzjY3dhARKGDu6HcQjy6",
"scriptPubKey": "76a9148e5af22e64e4ce8d03b8d6a43daff513146408f688ac",
"amount": 0.00259442,
"confirmations": 1239,
"spendable": true
},
{
"txid": "85d588e051387d980ce969a84de24dcc15d13fbbfd81977fd85d8b86bc09e469",
"vout": 2,
"address": "t1Pcvk4ZueZZqnn2KEWUTdKupi16SPCn6To",
"scriptPubKey": "76a9143f0a27e659f6802dc8b87d3cb51f46fa1c3f587f88ac",
"amount": 0.00029682,
"confirmations": 351,
"spendable": true
}
]
hush-cli getwalletinfo
{
"walletversion": 60000,
"balance": 0.00289124,
"unconfirmed_balance": 0.00000000,
"immature_balance": 0.00000000,
"txcount": 144,
"keypoololdest":<PHONE_NUMBER>,
"keypoolsize": 101,
"paytxfee": 0.00000000
}

What does z_gettotalbalance return?

{
"transparent": "0.00289124",
"private": "12.50014919",
"total": "12.50304043"
}

Funds are in your private wallet... DID you change z address or so?

nope, it's just happen... 3 days works fine, and after i get issue, nothing changes, after N reloads z-nomp it can be fixed, but next block again same thing.
idk, maybe issue couse i got 32threads(2xE5 processors)

Are you waiting long enough for the shielding transaction, it takes time..
Try rpcqueuesize = 128 in your coins hush.conf and try again

ok, looks like it solved, but might be not....
unshielding started.
can i not use shielding? because by default it was turned on, i didn't change it.

You cannot. Nice server you have :-)

oooh, it's not solved, after unshielding i get same error
Insufficient funds (0.00289124) to process payments (12.50005144) for 1 blocks; possibly waiting for txs.
getwalletinfo
{
"walletversion": 60000,
"balance": 12.50294043,
"unconfirmed_balance": 0.00000000,
"immature_balance": 0.00000000,
"txcount": 145,
"keypoololdest":<PHONE_NUMBER>,
"keypoolsize": 101,
"paytxfee": 0.00000000
}
listunspent
[
{
"txid": "d281fd212b3396c8f7bb56dfe817b7f19515cfcb58ab8d6270964a59974c4429",
"vout": 0,
"address": "t1WrJucfj3ySefztzjY3dhARKGDu6HcQjy6",
"scriptPubKey": "76a9148e5af22e64e4ce8d03b8d6a43daff513146408f688ac",
"amount": 0.00259442,
"confirmations": 1294,
"spendable": true
},
{
"txid": "85d588e051387d980ce969a84de24dcc15d13fbbfd81977fd85d8b86bc09e469",
"vout": 2,
"address": "t1Pcvk4ZueZZqnn2KEWUTdKupi16SPCn6To",
"scriptPubKey": "76a9143f0a27e659f6802dc8b87d3cb51f46fa1c3f587f88ac",
"amount": 0.00029682,
"confirmations": 406,
"spendable": true
},
{
"txid": "fc5054349325ef638c77dfee06a6af82b952d467e086b5ef41932b68eb59ffc5",
"vout": 0,
"address": "t1Pd626U121hzHCT7VtGZBLVXSmvFgTDW35",
"account": "",
"scriptPubKey": "76a9143f11e64100b775c8e2694ce7782151b325cf3bad88ac",
"amount": 12.50004919,
"confirmations": 6,
"spendable": true
}
]

How long do you wait when you see this??

~10-20min.
payment every 10min.
still got same issue

afte i mannualy spend this amount, all works fine, but  it's not autopayment at all

Insufficient funds (0) to process payments (12.50005144) for 1 blocks; possibly waiting for txs.

Weird, have TeamViewer or so, pm me credentials

ok, i check src, and find command for listunspent, it must be  listunspent 100 99999
and return this
[
]
because this transactions didn't get 100 confirmations

Thanks, to all, i found fix.
in pool config minConf must be 10, it was 100
"paymentProcessing": {
"minConf": 10,

I have "minConf" set to 10. Block was mined yesterday and still is "pending". I have these errors:

[Payments] [zclassic] Shield balance 12.4134
[Payments] [zclassic] Insufficient funds (0) to process payments (12.4131) for 1 blocks; possibly waiting for txs.
[Payments] [zclassic] Error trying to send z_address coin balance to payout t_address.{"code":3,"message":"Amount is not a number or string"}
[Payments] [zclassic] Error trying to shield balance Nan {"code":-3,"message":"Amount is not a number or string"}

I have checked balances with zcash-cli:

address: 12.41350000
zAddress: 0
tAddress: 0

I am having similar issue.
zcash-cli z_gettotalbalance
{
  "transparent": "1852.65500519",
  "private": "11275.24218136",
  "total": "13127.89718655"
}

But still, Insufficient funds (25.82018425) to process payments<PHONE_NUMBER>3); possibly waiting for txs.

sounds like you have the coin configured wrong, possibly using the same t_address for address and t_address in pool_config/coin.json

I have error
Insufficlient founds (......) to process payment (......); Possibly waiting for txs
What i need do fix :(

Basically the error Insufficient funds is not an error. It's just a notice that currently there are not enough funds matured+shielded+unshielded to send payment for the last block(s), but this will fix itself.
I think the error message should be removed from z-nomp or at least it should be marked with different color, not red. Most of the misunderstanding about this is that people do not get that for each mined block, they have to wait

100 blocks for maturing
10 more blocks to confirm shielding
10 more blocks to confirm unshielding
and wait until 3 last found blocks are confirmed this way.

Only then the payment can be sent to miners.
If the server has slow hardware, the shielding and unshielding takes very long time to execute, delaying this even further.

i am deploying bitcoinz pool and my config:
Coin:
{
"name": "Bitcoinz",
"symbol": "btcz",
"algorithm": "equihash",
"requireShielding": true,
"txfee": 0.0001
}
pool:
`"coin": "bitcoinz.json",
"coin": "bitcoinz.json",
"address": "t1PPKiaskZsSXZb.....",
"_comment_address": "a transparent address to send coinbase rewards to and to transfer to zAddress.",
"zAddress": "zcNus9QdXx3skhE5KzQx9.......
"_comment_zAddress": "a private address used to send coins to tAddress.",
"tAddress": "t1JrZXDy5N......",
"_comment_tAddress": "transparent address used to send payments, make this a different address, otherwise payments will not send",
"walletInterval": 2.5,
"rewardRecipients": {
"t1diyf84HoFYztg.....":1.5
},
"tlsOptions": {
"enabled": false,
"serverKey":"",
"serverCert":"",
},
"paymentProcessing": {
"minConf": 10,
"enabled": true,
"paymentMode": "prop",
"_comment_paymentMode":"prop, pplnt",
"paymentInterval": 20,
"minimumPayment": 100,
"maxBlocksPerPayment": 1,
"daemon": {
"host": "<IP_ADDRESS>",
"port": 3333,
"user": "xxxx",
"password": "xxx"
}
},`
but always show the error
017-10-07 04:50:39 [Payments] [bitcoinz] Insufficient funds (0) to process payments (12312.5); possibly waiting for txs.
my balance
ubuntu@ip-172-31-34-70:~/bitcoinz-pkg$ ./zcash-cli z_gettotalbalance
{
"transparent": "12312.5001",
"private": "0.00",
"total": "12312.5001"
}

@jcreyesb this issues is closed, if you are having problems please open your own issue

@madbuda thanks for info :)
I had the same problems
after which I "maxBlocksPerPayment": 3,
set to 1, the matter was done

hello guys, cam anyone help me with this problem on my Hush pool
[Pool]      [hush] (Thread 4) Share rejected: {"job":"cccd","ip":"::ffff:<IP_ADDRESS>","worker":"t1aPXQqsm9fFZzhFotsYeJyvve3SNgURCQH.Rig18","difficulty":2.4,"error":"incorrect size of solution"}
Thanks in advance

The miner isn’t submitting correct solutions... this happens with a bad
miner or when someone is trying to spoof a real miner.
I’d suggest you post an issue on znomp regarding this
On Wed, Dec 6, 2017 at 9:47 PM savskii<EMAIL_ADDRESS>wrote:

hello guys, cam anyone help me with this problem on my Hush pool
[Pool] [hush] (Thread 4) Share rejected:
{"job":"cccd","ip":"::ffff:<IP_ADDRESS>","worker":"t1aPXQqsm9fFZzhFotsYeJyvve3SNgURCQH.Rig18","difficulty":2.4,"error":"incorrect
size of solution"}
Thanks in advance
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/z-classic/z-nomp/issues/168#issuecomment-349846166,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFwhE5U7rjjD9PVBiiJByyjPs5R8_Zoks5s91HRgaJpZM4OcXp-
.


Drop Tape RSEs from the MSTransferor input location

Fixes #9943
Status
ready
Description
Do not use _Tape RSEs as a valid current location; and don't use them as a destination for input data placement either.
Is it backward compatible (if not, which system it affects?)
yes
Related PRs
Bugfix to: https://github.com/dmwm/WMCore/pull/9759
External dependencies / deployment changes
none

Tests went fine, merging.


Show total execution time

Hello,
This PR shows the total execution time of all tests. It's a nice information to show.

It's a nice information to show.

We don't add things on the merit of it being "nice to have". Does it have any actual use-case for every test run? I think knowing the test time can be useful once in awhile to make sure your tests aren't getting too slow, but then you can just do $ time ava. I don't want to see the time every time I run a test and I don't really see how it's useful either.

In my opinion some users want to see the total execution time, if you don't want show the time for all users could be a good solution have a flag, that way only users which set the flag will see.

@brunoqueiros If users will need to use a flag (for example, --show-time), they might as well just:
$ time ava

which will also show total execution time.

In my opinion some users want to see the total execution time

Why?

OK, if you doesn't see that is a useful feature I'll close the PR.


Beekeeper: Remove need of Index from internal logic

Remove Index from stream msgs, etc.

https://github.com/ethersphere/beekeeper/pull/87

https://github.com/ethersphere/beekeeper/pull/87


Can I build a catalog of all data already available in the network? Or a list of shares available? (e.g. could I query a network to get all mounts available and use them as a scheduling target?)

I've been using a silly bash command to list my own inputs/outputs. Can be adapted to all jobs like this:
List all Inputs
bacalhau list --all -n 100 --output json | jq '.[].Job.Spec.inputs[]'

List the output CIDs
bacalhau list --all -n 100 --output json | jq '.[].State.Executions[].PublishedResults.CID' -r | sort -u

For my use case (keeping track of the versions of a dataset), I was thinking to make it a Bacalhau job, put it back to IPFS as a CSV and add a pointer in GitHub to the last one.


timing-suite failed on s390x

55: FAILED (0 / 1 tests (0 skipped))
55/60 Test #55: timing-suite .....................***Failed    6.76 sec
Timing selftest ...................................................
Full buildlog: https://kojipkgs.fedoraproject.org//work/tasks/332/20080332/build.log
Any ideas how to fix this? This issue affects only the timing-suite on s390x.

Hi @mortenstevens I suggest you run the timing suite, with the verbose parameter, to have more information of the failure, for you to continue debugging:
./test_suite_timing --verbose

We are already using ctest --output-on-failure -V

As a data point, mbedTLS succeeded building on s390x on Debian:
https://buildd.debian.org/status/fetch.php?pkg=mbedtls&arch=s390x&ver=2.5.1-1&stamp=1498131591&raw=0

@jcowgill Thanks for uploading the build report
@mortenstevens please run ./test_suite_timing --verbose which will run the specific test_suite with output, which will be easier to debug, and not the full test_suite packages.
On successful run, the output should be:
./test_suite_timing --verbose
Timing selftest ...................................................   TIMING tests note: will take some time!
  TIMING test #1 (set_alarm / get_timer): passed
  TIMING test #2 (set/get_delay        ): passed
  TIMING test #3 (hardclock / get_timer): passed

PASS

----------------------------------------------------------------------------

We would like to focus on the specific test that fails

ARM Internal Ref: IOTSSL-1490

I think I'm not able to provide the requested information. As Fedora package maintainer we have only API access to Fedora build servers. So I can change only something like this:

Hi @mortenstevens
please try the following changes:
in tests/CMakeLists.txt change
add_test(${data_name}-suite test_suite_${data_name})

to
add_test(${data_name}-suite test_suite_${data_name} --verbose)

after that, build the tests with debug:
cmake -D CMAKE_BUILD_TYPE=Debug .

and after that, run the test:
ctest -R timing-suite -V --debug

Is this something you can do?
I think this will show you at least what specific timing test fails

@RonEld
Yes, thank you. It's strange. We are not able to reproduce this build issue. Maybe it was an issue related to our s390x build servers. Here is the output:
Now, it's building fine:
https://kojipkgs.fedoraproject.org//work/tasks/3079/20183079/build.log
55: Test command: /builddir/build/BUILD/mbedtls-2.5.1/tests/test_suite_timing "--verbose"
/builddir/build/BUILD/cmake-3.8.2/Source/cmCTest.cxx:142    Current_Time: Jun 26 10:41 UTC
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:678 55: Test timeout computed to be: 9.99988e+06
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55: Timing selftest ...................................................   TIMING tests note: will take some time!
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:   TIMING test #1 (set_alarm / get_timer): passed
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:   TIMING test #2 (set/get_delay        ): passed
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:   TIMING test #3 (hardclock / get_timer): passed
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55: PASS
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55: ----------------------------------------------------------------------------
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55:
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:61 55: PASSED (1 / 1 tests (0 skipped))
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:703 1/
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:705 1
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:721 Test
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:728  #55:
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:729
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:761 timing-suite .....................
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:763 Testing timing-suite ...
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:221    Passed
/builddir/build/BUILD/cmake-3.8.2/Source/CTest/cmCTestRunTest.cxx:263   8.76 sec

@mortenstevens Thank you for your update
I'm glad this issue was probably resolved(although I still don't know exactly what was the root cause)
Please close this issue, if no further action required
Regards

We believe that this issue no longer exists, therefore closing it.
If you are not happy with this, please reopen


New-HTMLImage - Image background color modification and including None/transparent as an option

With more email clients allowing for dark mode themes, When inserting an image, it uses white as the default background color when the image is inserted.

This turns out to be an outlook dark mode issue.
Section {
    $SectionStyle1 = New-HTMLSectionStyle -RequestConfiguration
    $pos = $SectionStyle1.SectionHead.IndexOf(".")
    $rightPart = $SectionStyle1.SectionHead.Substring($pos + 1)
    Add-HTMLStyle -Placement Header -Css @{
        "$($SectionStyle1.SectionHead)" = @{
            'display' = 'block'
            'background-color' = 'white'
            'text-align' = 'center'
            }
        } -SkipTags
New-HTMLImage -Source 'https://Site.domain.org/header.png' -Width '800' -Height '173'  -Class $rightPart
}

Note also that unlike other functions, I needed to remove the leading period from the class name generated or the function would not accept the class.