g++ (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0 changes

This PR is a work in progress, needs to be reformated and also the multiple decls need to be refactored but this is a starting point for people wanting to use a modern gcc. It now linkes for cuda 12

@jmikedupont2,
Thank you for your engagement with the project. It's important to note that the DCGM build system is designed to function specifically within the build container. This setup includes building all third-party dependencies and placing them in the /opt/cross directory while constructing the dcgmbuild docker image. This process is particularly tied to the versions and headers of Cuda.
Please be aware that we cannot approve changes that disrupt this established scenario.

I just added in the the vendor you can just ignore that but look at the changes for upgrading the compiler because there is the changes needed for the format strings to upgrade your compiler version

I would appreciate it if you could provide more details regarding the changes you require from GCC 11.2, 11.3, or 11.4. The formatting-related changes are difficult to isolate from the pull request, making it challenging to understand your requirements. Please note that we do not use std::format and opt for fmtlib instead.
The current commits included in this PR make it impossible for us to merge it. If you encounter any issues with the builds, please create individual issues, so we can track efforts and indicate if a PR would be acceptable/desired. However, please bear in mind that we won't accept changes that disable Cuda 10 and 11 in any circumstances or break the containerized build environment.

Here is a cleaner PR that only has the C++ changes.
Note that newrandom.h was not found anywhere.
https://github.com/NVIDIA/DCGM/pull/130

I refactored the cmake changes to its own branch in case someone needs that, marked and closed the pr for the record. Hope that helps someone in the future.


Fix unittests & solr response error handling on python3

Hi there !
Following issue #159, I went working a bit on this project

First I had to fix the solr installer to allow travis to run the tests
Then I fix issue #159 and add an unittest test about it
Finally, I had to mark as expected faillure test_extract given it was weirdly failing under travis

Here is the stack trace of the error (or see https://travis-ci.org/touilleMan/pysolr/jobs/71444801, the error is present both on python 2.x and 3.x)
======================================================================
ERROR: test_extract (tests.client.SolrTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/client.py", line 513, in test_extract
    extracted = self.solr.extract(fake_f)
  File "pysolr.py", line 932, in extract
    files={'file': (file_obj.name, file_obj)})
  File "pysolr.py", line 331, in _send_request
    raise SolrError(error_message)
SolrError: [Reason: lazy loading error]

My guess is travis is missing a library, but I couldn't figure really what...

It would be great merging and releasing a new version, downstream some django-haystack PR are blocked because of this issue https://travis-ci.org/django-haystack/django-haystack/jobs/76260939

looks ok :+1:

:+1:

I've corrected my branch with the try/finally block :+1:

The Solr errors turned out be caused by a change in file paths between Solr releases – the path in solrconfig.xml used to load the content-extraction library (solr-cell) from contrib/extraction/lib was one level off. I changed it to define the solr.install.dir property in the hopes that this will avoid similar problems in the future:
https://github.com/toastdriven/pysolr/commit/16c993459de0139f007bcd9ed35ba15b9eac16cf#diff-9588af6c9d46310c2f216d9e14a0b959R55

After that change, the only commits left on this PR were squashed into https://github.com/acdha/pysolr/commit/cf0fa493ea73217a427475a5c6a88fcf65220168 which ran successfully:
https://travis-ci.org/acdha/pysolr/builds/82057531

cool :+1:
The only remaining trouble is the travis status button on the README which points on a non existing target (https://travis-ci.org/toastdriven/pysolr)

Yes – unfortunately, only @toastdriven can check the box to enable that CI build. I'd prefer to transfer this repo to the django-haystack organization so it doesn't depend a single person.

cf0fa493ea73217a427475a5c6a88fcf65220168 was merged awhile ago but I just added some credits to the AUTHORS file in bf6d8b648738c575c9067b765b6f233c4921e7dd.
If you have any testing or suggestions for the README, I'm getting ready for the v3.4.0 release.


Update vehicle_create.md

add small information to declare dependencies in docs.

Are you Sure async is needed? I dont See a reason for that.

Yeah tried it out myself.

When async is required for it a bug report should be opened with sample resource and reproduction steps since this isnt intended.


clarify requirements words

Update appendix Z.10 to

Point at general Intro E, indicating that there is no intention to conflict with the general conventions IHE has
Add a sentence indicating that StructureDefinition profiling is the more dominant method of conveying constraints.

this replaces CP-ITI-1262


Flexible hash engine

We may want to:

Support hashing for GPU
Have an API that allows passing in a custom hash engine. Callers can take advantage of their own hardware to speed up hashing. Alternatively, allow passing the hash as input

I think 2 is done via #188 and GPU support should not be as part of v1 release. Ok to close and open a new feature request for GPU hashing? Or leave this as is but just exclude from milestone?

LGTM

Made #201 for the GPU case and can close this now


Debug tool opens with F12

Not sure whether this changed recently or it always has been or it's just me. But it's a bit annoying because F12 open Chrome dev tools and abap2UI5 debug tools.
Also inconsistent with this info.

https://github.com/abap2UI5/abap2UI5/blob/dd0121578e4f996d5bb5e52312e43a9ba495f872/app/webapp/Component.js#L39-L41

it used to be as it sais ctrl + f12

yes no recent changes.
mhh i am using mac and press "control+F12" therefore i thought the info is correct. and it also does not overlap with any other shortcuts on my side.
Are you using windows? what are you pressing?
open minded for changes when it conflicts with something on other machines...

ah i got it! the event handler is just listening for "F12". without realizing that, i always used control+F12 and therefore the chrome debugger never starts, but the abap2UI5 debugger tools did.
mhh yes then its an issue. maybe we just need to make the event handler for keys more precise and let it listen to both control+F12. if someone wants to give it a try, feel free, otherwise i can put it on my todo list.

I can give it a try. Shouldn't be too hard as zEvent has a ctrlKey property.

What's the current dev workflow for such a change? Manually changing Component.js and z2ui5_cl_app_component_js?

Ah, nice. And then the frontend repo branches cloud and classic are updated via GitHub Actions. Very nice!
One last question, how will the standard branch in the frontend repo be updated?


Broken server render

When i import the react-google-chart the server rendering broken.
"Document is not defined"
How can i use the module without this error?

Unfortunately this is a known issue. I will work on it when I have some time that frees up, meanwhile PRs are always appreciated if you think you can contribute a fix :)

Thanks @RakanNimer i will try do this.


Add --json flag to generate command

This MR adds a --json flag to the generate command. JSON is a format that, in general, is a little easier to work with than YAML. For example, Nix has fromJSON, but there's no fromYAML (this was my main reason).
Feel free to let me know if there's anything you'd like to see changed or if you have any ideas on how to do this better.

I also took the liberty of adding meta.mainProgram for e.g. nix run and lib.getExe.

@Misterio77 could you please have a look at this if you have some spare time?


Enable runtime roll forward

To allow running when .net 3.0 is not installed but later version is.

Merging to unblock migration process.


feature: OSIS-70-remove-filterKey-list-accounts-parameter

Functionality change.
Now we list accounts which are not created through VCD as well.

/approve


DBZ-1464: fixed NPE when op is not u/i/d

if the received event from mongoDB has anything but op=i, op=u, or op=d, it can result in a NPE for example with op=n (no op).
https://issues.jboss.org/browse/DBZ-1464
an alternative fix would be to change recordEvent()in RecordMaker to return 0 if op is not found in operationLiterals
Can we backport this to 0.9 and release a patch on that branch?
Signed-off-by: Lev Zemlyanov<EMAIL_ADDRESS>

@gunnarmorling and @jpechane could you have a look please?

Thanks a lot, @levzem! Squashed, rebased and applied.
Note I've merged this into master, i.e. it'll be part of the 0.10.0.CR1 release next week. We don't have any further 0.9.x releases planned at this point.

And thanks for reviewing, @patilharshal16!

Hello,
By curiosity, when do you see a noop in mongo oplog? Like, what's the scenario?
Thanks!

@renatomefi - I am not sure. This issue was raised by a customer.
according to here, "n" for "no-ops", used for changes in the database or collections which don't result in a change in the stored data


conda install --file requirements.txt -c conda-forge times out when running from a bash step in an azure pipeline on an ubuntu vm.

https://dev.azure.com/bionanogenomics/Bioinformatics/_build/results?buildId=1183&view=logs&s=96ac2280-8cb4-5df5-99de-dd2da759617d
trigger:
- azure-pipelines
- vault

resources:
  repositories:
  - repository: pybionano
    name: Bioinformatics/pybionano
    type: git
    ref: develop

  - repository: svconfmodels
    name: Bioinformatics/svconfmodels
    type: git
    ref: azure-pipelines

pool:
    vmImage: 'ubuntu-latest'
    #name: conda

variables:
  BUILDID: $(Build.BuildNumber)

steps:
- checkout: pybionano
- checkout: svconfmodels

- bash: echo "##vso[task.prependpath]$CONDA/bin"
  displayName: Add conda to PATH

- bash: |
    cd $(System.DefaultWorkingDirectory)/svconfmodels/
    conda env create -n bionano3 -f bionano_python3.0.yml python=3.7

    source activate bionano3

    cd $(System.DefaultWorkingDirectory)/svconfmodels

    conda install --file requirements.txt -c conda-forge

    cd $(System.DefaultWorkingDirectory)/pybionano

    pip install .

    cd $(System.DefaultWorkingDirectory)/svconfmodels

    pip install .

    cd $(System.DefaultWorkingDirectory)/pybionano/tests/unittests

    pytest -v

    cd $(System.DefaultWorkingDirectory)/svconfmodels/tests/unittests

    pytest -v

  displayName: 'Unittests'

The following NEW packages will be INSTALLED:

  _openmp_mutex      pkgs/main/linux-64::_openmp_mutex-4.5-1_gnu
  libgomp            pkgs/main/linux-64::libgomp-9.3.0-h5101ec6_17
  py-xgboost         conda-forge/linux-64::py-xgboost-0.90-py37_4
  xgboost            conda-forge/linux-64::xgboost-0.90-py37he1b5a44_4

The following packages will be UPDATED:

  ca-certificates                      2020.12.5-ha878542_0 --> 2021.5.30-ha878542_0
  certifi                          2020.12.5-py37h89c1867_1 --> 2021.5.30-py37h89c1867_0
  imbalanced-learn                               0.6.2-py_0 --> 0.7.0-py_0
  lightgbm                             2.3.0-py37he1b5a44_0 --> 3.1.0-py37h745909e_0
  numpy                               1.18.4-py37h8960a57_0 --> 1.19.4-py37h7e9df27_1
  pandas                               1.0.3-py37h0da4684_1 --> 1.1.4-py37h10a2094_0
  pytest                               5.4.2-py37hc8dfbb8_0 --> 5.4.3-py37hc8dfbb8_0
  pytest-cov                                     2.8.1-py_0 --> 2.11.0-pyh44b312d_0
  pytest-forked                                  1.1.2-py_0 --> 1.3.0-pyhd3deb0d_0
  pytest-xdist                          1.32.0-pyh9f0ad1d_0 --> 1.34.0-pyh9f0ad1d_0
  scipy                                1.4.1-py37ha3d9a3c_3 --> 1.5.3-py37h8911b10_0
  xlrd                                   1.2.0-pyh9f0ad1d_1 --> 2.0.1-pyhd8ed1ab_3

The following packages will be SUPERSEDED by a higher-priority channel:

  gfortran_impl_lin~ conda-forge::gfortran_impl_linux-64-7~ --> pkgs/main::gfortran_impl_linux-64-7.3.0-hdf63c60_1

So, there you have it.  Looks pretty straightforward to me but it stalls on line: conda install --file requirements.txt -c conda-forge
Never gets further than the single line: gfortran_impl_lin~ conda-forge::gfortran_impl_linux-64-7~ --> pkgs/main::gfortran_impl_linux-64-7.3.0-hdf63c60_1
Any insight would be praiseworthy.

hey @cbladd , what does the requirements.txt look like, does it contain large packages to be installed?
I'm wondering if you could try conda install a small single package first to see if there's a networking issue or not.
otherwise it maybe be hitting the time out limitation.


Fix poisson to support lam array

We were not supporting arrays for lam parameter ...
~merge after #4907~

rebased, PTAL

Sorry.  I meant 'shape': [..., None] in testing.parameterize to pass size=None to distributions in the tests.

Updated 😄

Jenkins, test this please.

Jenkins CI test (for commit 64320e927b7b3417588449603c8ae37f20d3b653, target branch master) succeeded!

I think we can merge this :D


Nightly is not up to date with the latest commit

I'm trying to test out the latest commit cb37dd18 of reviewdog/reviewdog inside a GitHub action without installing it from the source. I wanted to use the nightly branch to do this. I, however, noticed that the nightly build is still on 67c34030. It looks like you have set up the gorelease Github action, this it only runs when a pull request contains a label. I was, therefore wondering what the currently recommended way to update the nightly branch is?

Ah I see it is being released but failing I will take a look.

This issue should be fixed in #6


Update rancher-charts/neuvector chart to support updated approach for PSPs

Parent issue: #40331

Implement the changes specified in the parent issue for the neuvector chart.

Create a new chart version (102.0.0+up2.4.0)
Add the context in the app-readme.md file about the changes related to PSPs for running within Rancher Manager. Example content is in the main parent issue linked above. Please note that all the other charts are getting the same value name, but we are not expecting that change for our SUSE projects (aka NeuVector), so the language may need to be updated.
Add a validation psp install check in the chart so that if someone tries to enable PSPs on a cluster that doesn’t have the capability, then it would error out. Note: This is recommended for the upstream NeuVector chart and not just our “patched” upstream as it could be updated to true and it would cause immediate errors on a k8s 1.25 cluster. Code on how to do this is in the parent issue.

Cleanup note:
Can you please also remove this line from the Chart.yaml as this line implies that there is custom UI for this feature chart which doesn't exist according to the UI code?
https://github.com/rancher/charts/blob/dev-v2.7/charts/neuvector/101.0.2%2Bup2.4.0/Chart.yaml#L13

Note: The new chart version should be 102.x.y - Also, if you plan to release only this version in Rancher 2.7.2, please remove using make remove any older 101.x.x versions that you may have in release.yaml earlier for 2.7.2 release

@prachidamle I am about to make PR today for the changes mentioned in this issue for 2.7.2 release. Do we need to remove older version 101.x.x or up to us to decide.

Created a PR to fix the issue. The fix includes

Added global variable to set PSP
Updated questional yaml
Added PSP validation yaml
Removed other psp configuration from yaml
Updated psp.yaml template to include global psp value
removed UI component from Charts.yaml
Updated app-readme to include PSP detail
Bumped version to 102.0.0

https://github.com/rancher/charts/pull/2395

@selvamt94 yes ideally we should release only one version with Rancher manager - if you do not intend to release multiple Neuvector versions with Rancher 2.7.2, we should remove the older 101.x.x versions in the release.yaml https://github.com/rancher/charts/blob/dev-v2.7/release.yaml#L18

PR merged

Pass verified in rancher 2.7.2-rc8
this chart was tested in https://github.com/rancher/rancher/issues/39856


Add README to the repo

Hey @jared-rimmer @sciortino @poloaraujo! This is looking great, just one request to add a README to the repo with some basic info. I created a similar issue on https://github.com/brooklyn-data/dbt-kustomer/issues/10.

I see https://github.com/fivetran/dbt_klaviyo/blob/main/README.md and https://github.com/fivetran/dbt_klaviyo_source/blob/main/README.md. I'm not necessarily requesting that level of detail but enough for someone to know what the repo is for and how to use it would be great.

I dont see Fivetran kustomer repos but the klaviyo should give you a good idea of some stuff to include 😄 .

Closed by https://github.com/MeltanoLabs/dbt-klaviyo/pull/7


HBASE-24593 [branch-2.2] Fix the maven compilation failure for nightl…

…y build

Need to wait https://github.com/apache/hbase/pull/1927 merged firstly.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m  0s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
5m 10s
branch-2.2 passed

+1 :green_heart:
compile
0m 27s
branch-2.2 passed

+1 :green_heart:
checkstyle
0m 37s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m  6s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
branch-2.2 passed

+0 :ok:
spotbugs
1m 16s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 15s
branch-2.2 passed

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 26s
root in the patch failed.

-1 :x:
compile
0m 23s
hbase-client in the patch failed.

-1 :x:
javac
0m 23s
hbase-client in the patch failed.

+1 :green_heart:
checkstyle
0m 33s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
shadedjars
2m  9s
patch has 20 errors when building our shaded downstream artifacts.

-1 :x:
hadoopcheck
1m 18s
The patch causes 20 errors with Hadoop v2.8.5.

-1 :x:
hadoopcheck
2m 37s
The patch causes 20 errors with Hadoop v2.9.2.

-1 :x:
hadoopcheck
3m 58s
The patch causes 20 errors with Hadoop v2.10.0.

-1 :x:
hadoopcheck
5m 21s
The patch causes 20 errors with Hadoop v3.1.2.

-1 :x:
hadoopcheck
6m 47s
The patch causes 20 errors with Hadoop v3.2.1.

+1 :green_heart:
javadoc
0m 23s
the patch passed

-1 :x:
findbugs
0m 24s
hbase-client in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 25s
hbase-client in the patch failed.

+1 :green_heart:
asflicense
0m 11s
The patch does not generate ASF License warnings.

28m  5s

Subsystem
Report/Notes

Docker
Client=19.03.11 Server=19.03.11 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1928

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux dbed0bc2d319 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1928/out/precommit/personality/provided.sh

git revision
branch-2.2 / e7f17fe68d

Default Java
1.8.0_181

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-compile-hbase-client.txt

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-compile-hbase-client.txt

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-shadedjars.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-javac-2.8.5.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-javac-2.9.2.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-javac-2.10.0.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-javac-3.1.2.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-javac-3.2.1.txt

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-findbugs-hbase-client.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/artifact/out/patch-unit-hbase-client.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/testReport/

Max. process+thread count
96 (vs. ulimit of 10000)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
5m 14s
branch-2.2 passed

+1 :green_heart:
compile
0m 26s
branch-2.2 passed

+1 :green_heart:
checkstyle
0m 39s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
branch-2.2 passed

+0 :ok:
spotbugs
1m 15s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 14s
branch-2.2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 42s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
checkstyle
0m 35s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m  2s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
25m 23s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
0m 24s
the patch passed

+1 :green_heart:
findbugs
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m 46s
hbase-client in the patch passed.

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

59m 54s

Subsystem
Report/Notes

Docker
Client=19.03.11 Server=19.03.11 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1928

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 07eda24f9e17 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1928/out/precommit/personality/provided.sh

git revision
branch-2.2 / 1bbe250a2d

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/2/testReport/

Max. process+thread count
305 (vs. ulimit of 10000)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1928/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Ping @Apache9 take a look about this?


Chilling Water is missing the Technician Tera Boost

Chilling Water is missing the Technician Tera Boost when used by a Tera Pokemon
Without Tera
252+ SpA Mystic Water Kyogre Chilling Water vs. 236 HP / 116 SpD Amoonguss in Rain: 49-58 (22.3 - 26.4%) -- possible 5HKO after Sitrus Berry recovery
With Tera
252+ SpA Mystic Water Tera-Water Kyogre Chilling Water vs. 236 HP / 116 SpD Amoonguss in Rain: 66-78 (30.1 - 35.6%) -- 97.7% chance to 4HKO after Sitrus Berry recovery
35.6%/26.4% = ~1.33
There is only the ~1.33 boost from Tera, no power increase to 60 BP like with other moves lower than 60 BP

Nope, that calculation is right. The BP boost from Tera is the last BP modifier applied. Kyogre is holding a Mystic Water, an item that applies a BP modifier earlier and enough of one to make Chilling Water equal to 60 BP, so no Tera BP modifier is applied.


Provide overview of all external scalers

Provide overview of all external scalers that are available.
This information should be based on Artifact Hub - https://artifacthub.io/packages/search?kind=8&sort=relevance&page=1
Kept is using a similar approach: https://keptn.sh/docs/integrations/
Relates to #412

I like this idea @tomkerkhove. Do you think we should write a script to pull from Artifact Hub?

Nope, they allow us to embed everything :) But I'll ask the CNCF to help us to revamp the whole experience with filters based on the categories.

Are the cards used to display scalars on the homepage, the kind of card you'd like used on the Scalars page too?
Screenshot from the homepage:

We don't have a strong requirement on the info but would be nice to surface the name, availability and categories that we'd like to introduce.

@tomkerkhove I'm taking a look at this one.
/assign @thisisobate


SDK functions crash when called without a callback

Right now, there are no checks to make sure a callback has been passed to an SDK function before attempting to call it. In a situation where someone wants to call a function in a "fire and forget" manner, and does not provide a callback function, this causes the application to crash.
We should check to make sure callbacks are set before attempting to call them.

Upon further investigation and consideration, we will not be supporting calling SDK functions without a callback.


fixing lookup error from gcc/mingw

small change to make nana compile with mingw again.
best regards, jan
^

Thank you for your contribution


Require qemu dependencies for other architectures

In order to make the user experience as fluent as possible for
non-intel arches supported in testcloud as well let's require
corresponding qemu packages.

Hm, or should this rather go to testcloud instead?

Could those be "weak" dependencies? Currently we don't require even x86_64.
Also those are not available on rhel-8 (nor epel)

By chain of dependencies.

Do you suggest to recommend it directly?

Would be better, IMO. tmt recommend it for better UX so why depend on 3rd party chain.

Makes sense, updated.


Add Akitax to Token List

your token doesnt currently meet the liquidity requirements to be tokenlisted under the meme section. Keep it up!

closing PR for now, please re-open when liquidity is near 6 figure range on trader joe LP. We would like to see healthy and sustained volumes before tokenlisting


Failed to lookup symbol 'readBarcode'

This issue exists in latest commit (4ebc637dfb3208f72c9c8adda813e1f58d3ed99a) and not version 0.10.0
1. Underlying Exception
Path
/Users/user/.pub-cache/git/flutter_zxing-4ebc637dfb3208f72c9c8adda813e1f58d3ed99a/lib/generated_bindings.dart
Exception
Exception has occurred.
ArgumentError (Invalid argument(s): Failed to lookup symbol 'readBarcode': dlsym(RTLD_DEFAULT, readBarcode): symbol not found)
Offending code
late final _readBarcodePtr = _lookup<
      ffi.NativeFunction<
          CodeResult Function(ffi.Pointer<ffi.Char>, ffi.Int, ffi.Int, ffi.Int,
              ffi.Int, ffi.Int, ffi.Int, ffi.Int, ffi.Int)>>('readBarcode');
// ---------------------------------------------------------^

2. User-Facing Exception
Path
/Users/user/.pub-cache/git/flutter_zxing-4ebc637dfb3208f72c9c8adda813e1f58d3ed99a/lib/zxing_mobile.dart
Exception
Exception has occurred.
_CastError (type 'ArgumentError' is not a subtype of type 'Code' in type cast)
Offending code
await zxingProcessCameraImage(image, params: params) as Code;
// --------------------------------------------------^

3. Usage
Init / Dispose
// StatefulWidget state property
final Zxing zxMobile = ZxingMobile();

// called from initState
await zxMobile.startCameraProcessing();

// called inside dispose
zxMobile.stopCameraProcessing();

processCameraImage
// `CameraImage` object comes from `startImageStream` from `camera` package
// https://github.com/flutter/plugins/blob/729c14a9243ebd2b16d7ec2a63b4e54661b00641/packages/camera/camera/lib/src/camera_controller.dart#L415
Code zxBarcode = await zxMobile.processCameraImage(
    cameraImage,
    params: DecodeParams(
        tryHarder: false,
        tryRotate: true,
        tryInverted: false,
    ),
);

4. pubspec.yaml
offending versions
flutter_zxing:
  git:
    url: https://github.com/khoren93/flutter_zxing.git
    ref: 4ebc637dfb3208f72c9c8adda813e1f58d3ed99a

working pubspec.yaml
flutter_zxing: 0.10.0

Platform Information

iOS

Flutter 3.3.9 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b8f7f1f986 (9 weeks ago) • 2022-11-23 06:43:51 +0900
Engine • revision 8f2221fbef
Tools • Dart 2.18.5 • DevTools 2.15.0

Another question if you may, which one is more recommended when using flutter_zxing with camera package?

processCameraImage (parameter is CameraImage)
readBarcode (parameter is image bytes InputImage -> byte list property Uint8List?)

Thanks for reaching out.
Since version 1.0.0, the flutter_zxing package uses submodules, so specifying a specific version in the pubspec file is not possible. To use the main branch, you will need to manually clone it locally using the command "git clone --recursive https://github.com/khoren93/flutter_zxing.git". Once the package is cloned, navigate to the "scripts" directory using "cd scripts" and run the script "sh update_ios_macos_src.sh". After this, you can add the flutter_zxing package as a local dependency in the pubspec file by specifying the path of the cloned local repository. For example, if you cloned the package in your home directory, the path would be
flutter_zxing:
  path: ~/flutter_zxing

For the second question, it is highly recommended to use the processCameraImage method, rather than the readBarcode method. The reason for this is that processCameraImage is specifically designed to work with the camera stream, and it processes the stream in a separate isolate. This allows for more efficient and reliable barcode scanning, as well as improved performance and reduced memory usage. However, the readBarcode method can still be used to read barcodes from local image files, such as those from the gallery or from a URL. It takes as parameter InputImage which is a object with properties like bytes (Uint8List) and it's more appropriate for use with static images rather than a camera stream.

Closing for now as this is already fixed.


http: add idempotency key checker, inmem LRU for now

This adds a working implementation of handling X-Idempotency-Key and
does so for local dev / beta without any additional deployment.
I plan to add a redis implementation, but when we need it later on.
Fixes: #15

Codecov Report

Merging #17 into master will increase coverage by 2.09%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   33.52%   35.62%   +2.09%
==========================================
  Files          12       13       +1
  Lines        1375     1398      +23
==========================================
+ Hits          461      498      +37
+ Misses        826      806      -20
- Partials       88       94       +6

Impacted Files
Coverage Δ

main.go
0% <0%> (ø)
:arrow_up:

pkg/idempotent/lru/lru.go
100% <100%> (ø)

http.go
56.06% <71.42%> (+5.21%)
:arrow_up:

transfers.go
42.52% <88.88%> (+7.37%)
: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 e11539c...e4195ce. Read the comment docs.

cc @wadearnold


Code refactored according to PEP8 style

@dexter1691 please review.

@DESHRAJ I know I'm nitpicking but it might be a good idea to run pep8 against the source. Output of pep8 --statistics -qq app/* (how often each error was found):
38      E101 indentation contains mixed spaces and tabs
2       E111 indentation is not a multiple of four
7       E114 indentation is not a multiple of four (comment)
9       E115 expected an indented block (comment)
8       E116 unexpected indentation (comment)
2       E127 continuation line over-indented for visual indent
21      E128 continuation line under-indented for visual indent
2       E202 whitespace before ']'
6       E203 whitespace before ','
3       E222 multiple spaces after operator
98      E225 missing whitespace around operator
129     E231 missing whitespace after ':'
86      E251 unexpected spaces around keyword / parameter equals
3       E261 at least two spaces before inline comment
2       E262 inline comment should start with '# '
56      E265 block comment should start with '# '
5       E266 too many leading '#' for block comment
1       E271 multiple spaces after keyword
2       E301 expected 1 blank line, found 0
15      E302 expected 2 blank lines, found 1
18      E303 too many blank lines (2)
3       E401 multiple imports on one line
136     E402 module level import not at top of file
302     E501 line too long (95 > 79 characters)
7       E701 multiple statements on one line (colon)
8       E703 statement ends with a semicolon
3       E711 comparison to None should be 'if cond is None:'
443     W191 indentation contains tabs
94      W291 trailing whitespace
7       W292 no newline at end of file
57      W293 blank line contains whitespace
4       W391 blank line at end of file
4       W503 line break before binary operator
2       W601 .has_key() is deprecated, use 'in'

pep8 --show-source --show-pep8 app/* will give you detailed line-by-line checks. Not that all need to be fixed, but it might be useful to keep in mind going forward.

@abhshkdz Got your point. :+1: Will fix it. Also I was thinking to use flake8 after writing the test cases in travis CI build so that it will automatically check for the pep8 styling.

That'd be great! :+1:


Error when using Manual.py

I got the script up and running on OS X so I wanted to see if I could get it running on my Synology. Here is the error i get why I run it.
DiskStation> ./manual.py -i ../../Movies/Wanted\ (2008).mkv
Traceback (most recent call last):
File "./manual.py", line 10, in
from readSettings import ReadSettings
File "/volume1/Videos/Downloads/mp4_automator/readSettings.py", line 5, in
from babelfish import Language
File "/volume1/Videos/Downloads/mp4_automator/babelfish/init.py", line 13, in
from .converters import (LanguageConverter, LanguageReverseConverter, LanguageEquivalenceConverter, CountryConverter,
File "/volume1/Videos/Downloads/mp4_automator/babelfish/converters/init.py", line 6, in
from pkg_resources import iter_entry_points, EntryPoint
ImportError: No module named pkg_resources

Your python installation does not have setup tools installed.
Follow the instructions here to install it:
https://pypi.python.org/pypi/setuptools#installation-instructions

That worked!! Thank You.


Tests failing in CI with obscure Tensorflow Error

Describe the bug
Tests are failing on CI with the following error message:
tensorflow.python.framework.errors_impl.AlreadyExistsError: Another metric with the same name already exists.

To Reproduce
Check recent Actions logs eg, #149 or this
Expected behavior
CI should not fail unexpectedly
Screenshots
Environment (please complete the following information)

OS: Unclear
Python Version: Unclear
c3-toolset Version : dev

Additional context
Add any other context about the problem here. You can also suggest possible solutions and relevant implementations & edge case details.

This, uh, seems to have fixed itself?
cf. https://github.com/lazyoracle/c3/runs/4174123057


Geofencing

Hi, I am using the example code of geofencing, it does not seem to work the broadcast receiver is not being called, I've tried many solutions nothing seems to work

Hi, I have the same problem and change the Lines: 233 and 236 in MainActivity.java:
Intent intent = new Intent(getApplicationContext(), GeofenceBroadcastRutaReceiver.class);
// We use FLAG_UPDATE_CURRENT so that we get the same pending intent back when calling
// addGeofences() and removeGeofences().
mGeofencePendingIntent = PendingIntent.getBroadcast(mContext, 0, intent, PendingIntent.FLAG_UPDATE_CURRENT);
==========
Context mContext;
@Override
protected void onCreate(Bundle savedInstanceState) {
mContext = this;
}

Please tell me the best solution for Geofencing. I use a ForeGroundService with a notification on Top instead of the IntentService as it wasnt working. and have a PendingIntent to receive the entry/exit events . But still My geofencing solution doesnt work at all.

Same thing here, geofencing doesn't trigger when the app is closed.
Geofencing only works with foreground service.
Would be great it someone at google could make a statement.
@pramodit do you use the BroadcastReceiver like in the sample?

Now it works fine for me . I used a JobScheduler and scheduling the job every now and then as this background thread gets killed in Android Oreo due to Background Execution Limits . @DevEddy  Please try with the JobService . It worked like a charm to me

Hi @pramodit
I am glad that it works for you now.
Could you provide some snippets how you're doing this? I would be very thankful.

Sure @DevEddy This is how i am doing it . I use this JobService and reschedule it every 1 secs in Android O to make sure the OS dont kill my service.
/**

JobService for Geofences above Android Oreo

@author Pramod
@since 16-Sept-2018.
*/

@TargetApi(21)
public class OreoJobService extends JobService {
private JobParameters mParams;

//Assuming it takes maximum 5 seconds for
private String TAG = "OreoJobService";

public boolean onStartJob(JobParameters params) {

    this.mParams = params;

    String command = params.getExtras().getString("command");

    if (command != null && command.equals("stop")) {

        this.endJob();
        return false;
    } else {

        if (MyApplication.getInstance().geoFencePendingIntent == null) {

            if (params.getExtras().containsKey("fromJobScheduler")) {

                String geoFences = MyApplication.getInstance().fetchGeoFences();

                if (geoFences != null) {

                    List<Fence> fencesList = new Gson().fromJson(geoFences, new TypeToken<List<Fence>>() {
                    }.getType());
                    MyApplication.getInstance().createGeoFences(fencesList);

                }

            }
        } else {

            Log.e(TAG, "onStartJob: scheduling job ");
            this.scheduleJob(10000);
        }

        return true;
    }
}

void endJob() {
    this.jobFinished(this.mParams, false);
}

void scheduleJob(long interval) {

    ComponentName serviceName = new ComponentName(this.getPackageName(), OreoJobService.class.getName());

    PersistableBundle extras = new PersistableBundle();
    extras.putString("command", "start");
    extras.putInt("fromJobScheduler", 1);
    JobInfo jobInfo = (new JobInfo.Builder(VariableConstants_Fortees.GEOFENCE_JOB_ID, serviceName))
            .setExtras(extras).setRequiredNetworkType(JobInfo.NETWORK_TYPE_ANY)
            .setMinimumLatency(interval)
            .setOverrideDeadline(interval)
            .build();

    JobScheduler jobScheduler = (JobScheduler) this.getSystemService(Context.JOB_SCHEDULER_SERVICE);
    assert jobScheduler != null;
    jobScheduler.schedule(jobInfo);
    this.endJob();
}

public boolean onStopJob(JobParameters params) {

    return false;
}

}
###################### And here's my logic for scheduling ##########################
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
        ComponentName serviceName = new ComponentName(this, OreoJobService.class.getName());
        PersistableBundle extras = new PersistableBundle();
        extras.putString("command", "start");

        JobInfo jobInfo = (new JobInfo.Builder(VariableConstants_Fortees.GEOFENCE_JOB_ID, serviceName))
                .setExtras(extras).setRequiredNetworkType(JobInfo.NETWORK_TYPE_ANY)
                .setMinimumLatency(1L)
                .setOverrideDeadline(1L)
                .build();

        JobScheduler jobScheduler = (JobScheduler) this.getSystemService(Context.JOB_SCHEDULER_SERVICE);

        try {
            jobScheduler.schedule(jobInfo);
        } catch (IllegalArgumentException errorMessage) {
            errorMessage.printStackTrace();
        }

    }

Can you share the working sample if its possible?I am trying for this solution last one week

@pramodit Does your solution still work ? I have tried everything at this point

Yes it still works like a charm for me @kraghu

If you want it to make it optimised for every other locations , you could use a WorkManager which is compatabile for all regions such as for china, where there are no GooglePlayServices , it uses FirebaseDispatcher, AlarmManager for some devices and JobScheduler wherever necessary @kraghu  :)

JobService for Geofences project

please send me geofence project

Sure @DevEddy This is how i am doing it . I use this JobService and reschedule it every 1 secs in Android O to make sure the OS dont kill my service.
/**

JobService for Geofences above Android Oreo
@author Pramod
@SInCE 16-Sept-2018.
*/

@TargetApi(21)
public class OreoJobService extends JobService {
private JobParameters mParams;

//Assuming it takes maximum 5 seconds for
private String TAG = "OreoJobService";

public boolean onStartJob(JobParameters params) {

    this.mParams = params;

    String command = params.getExtras().getString("command");

    if (command != null && command.equals("stop")) {

        this.endJob();
        return false;
    } else {

        if (MyApplication.getInstance().geoFencePendingIntent == null) {

            if (params.getExtras().containsKey("fromJobScheduler")) {

                String geoFences = MyApplication.getInstance().fetchGeoFences();

                if (geoFences != null) {

                    List<Fence> fencesList = new Gson().fromJson(geoFences, new TypeToken<List<Fence>>() {
                    }.getType());
                    MyApplication.getInstance().createGeoFences(fencesList);

                }

            }
        } else {

            Log.e(TAG, "onStartJob: scheduling job ");
            this.scheduleJob(10000);
        }

        return true;
    }
}

void endJob() {
    this.jobFinished(this.mParams, false);
}

void scheduleJob(long interval) {

    ComponentName serviceName = new ComponentName(this.getPackageName(), OreoJobService.class.getName());

    PersistableBundle extras = new PersistableBundle();
    extras.putString("command", "start");
    extras.putInt("fromJobScheduler", 1);
    JobInfo jobInfo = (new JobInfo.Builder(VariableConstants_Fortees.GEOFENCE_JOB_ID, serviceName))
            .setExtras(extras).setRequiredNetworkType(JobInfo.NETWORK_TYPE_ANY)
            .setMinimumLatency(interval)
            .setOverrideDeadline(interval)
            .build();

    JobScheduler jobScheduler = (JobScheduler) this.getSystemService(Context.JOB_SCHEDULER_SERVICE);
    assert jobScheduler != null;
    jobScheduler.schedule(jobInfo);
    this.endJob();
}

public boolean onStopJob(JobParameters params) {

    return false;
}

}
###################### And here's my logic for scheduling ##########################
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
        ComponentName serviceName = new ComponentName(this, OreoJobService.class.getName());
        PersistableBundle extras = new PersistableBundle();
        extras.putString("command", "start");

        JobInfo jobInfo = (new JobInfo.Builder(VariableConstants_Fortees.GEOFENCE_JOB_ID, serviceName))
                .setExtras(extras).setRequiredNetworkType(JobInfo.NETWORK_TYPE_ANY)
                .setMinimumLatency(1L)
                .setOverrideDeadline(1L)
                .build();

        JobScheduler jobScheduler = (JobScheduler) this.getSystemService(Context.JOB_SCHEDULER_SERVICE);

        try {
            jobScheduler.schedule(jobInfo);
        } catch (IllegalArgumentException errorMessage) {
            errorMessage.printStackTrace();
        }

    }

@pramodit  why you create geofences every time in service
 MyApplication.getInstance().createGeoFences(fencesList);
Because Android Oreo and above doesn't provide frequent location update,
Don't we need to call location update service in the background? I am confused :(

@RameshPokharel  Hey am not create the fences its just named that way. i am just registering them once for all . i get the fences from an API call and then i use them. if it's already created then i don't recreate the fences


site and nwis precip refactor

This PR refactors our site filtering and how we pass data around between the functions that use site data.
The initial site filter is now applied in the site fetcher rather than in a process. The site fetcher now outputs a SpatialPointsDataFrame.
nwis data is now pulled in it's own process step. It could be in a fetch if fetchers could depend on other fetchers, but apparently that will not work?
storm-sites-flood builds the stream gage point classes for visualization.
Downstream dependencies have been cleaned up a bit but there still may be some opportunities to clean up. e.g. storm-sites-flood is still used in two places but it's really meant for the visualize step.

fetchers can depend on other fetchers, just not processors

Maybe I don't know what I'm doing in some other way. I was getting errors when I had the first two process steps as the last two fetch steps.

There error I was getting was:
make[1]: *** No rule to make target `scripts/process/fetch.nwis_data.R', needed by `nwis-data_timestamp'.  Stop.
make: *** [fetch] Error 2

Exited with status 2.

Did you change the viz.yaml field from processor to fetcher?

OK, got the fetchers where they belong now.


Replace Getting started with Zulip guide with Moving to Zulip guide

Following up on #31498, we should:

Replace links to /help/getting-your-organization-started-with-zulip with links to /help/moving-to-zulip. If it's just a link, let's rename the link to "Moving to Zulip" (including in the landing page footer). If it's in a sentence, let's check in about each one, to see if/how it should be reworded.
Replace links to sub-sections of that page with links to the corresponding dedicated pages.
Make a redirect

@laurynmm It would be great if you could take are of this one once https://github.com/zulip/zulip/pull/31498 is merged.


[PE-D][Tester B] Quick Start section in UG: incorrect command

add n/John Doe p/98765432<EMAIL_ADDRESS>a/John street, block 123, #01-01 is not a valid command as specified in the UG.

Labels: type.DocumentationBug severity.Low
original: ish1506/ped#1

Duplicate issue


Add gif export functionality, kindly please.

Add export function as gimp can do and it's the best professional and blazingly fast photoshop I have ever seen. Thank you.

it'd be sick to have but that'd practically double the development load, right?

if it's limited to an "export as gif" dialog where each layer is a frame and some settings it wouldn't be that much work. But that way I'm not sure many would use it.

It's the only useful thing gimp can do, not kleki. Is that enough motivation or should I try harder? :D


[sundown] Don't include headings within list items

A heading cannot be inside a list item.
Dependent PRs:

[x] https://github.com/apiaryio/sundown/pull/9
[x] https://github.com/apiaryio/markdown-parser/pull/11 I can update reference to master once dependent PRs are merged.

Fixes #322

Waiting for update on apiaryio/sundown#9

I have checked for any regressions which I was afraid we might have due to the changes in the parser. Everything is okay. But I cannot merge this yet, because the behaviour of the parser in the two conceptually similar test cases is quite different.
Normal header style
The header node is outside the block quote node.
+ root (type 0, data 0) - `` :0:59
  + header (type 4, data 1) - `GET /a` :0:10
  + list item (type 6, data 10) - `` :10:46
    + paragraph (type 7, data 0) - `Response 200` :12:14
    + code (type 1, data 0) - `a
` :30:6
    + quote (type 2, data 0) - `` :36:16
      + code (type 1, data 0) - `  b
e
` :38:14
  + header (type 4, data 1) - `B` :56:3

Setext header style
The header node is inside the block quote.
+ root (type 0, data 0) - `` :0:72
  + header (type 4, data 1) - `GET /a` :0:10
  + list item (type 6, data 2) - `` :10:62
    + paragraph (type 7, data 0) - `Response 200` :12:14
    + code (type 1, data 0) - `a
` :30:6
    + quote (type 2, data 0) - `` :36:16;50:16
      + code (type 1, data 0) - `  b
e
` :38:14
      + header (type 4, data 2) - `Heading` :52:16

@pksunkara seems like the limitation of the parser. I wonder if we need to worry about settext headers at all. Thoughts?

We do not know if some people are using setext headers. I will leave the decision to you to merge this PR or let kyle fix it.

@pksunkara what was the behavior before this PR? Is this a regression?

I checked this and it is behaving the same above before this PR fix. So, I am merging this now.


feat:retrieve registrants and number of models components relationshi…

…p and policies it registers w/signoff
Signed commits

[x] Yes, I signed my commits.

@MUzairS15 looks good to you?


Feature request/question: is tar support planned

Not sure tar is part of core utilities, but is there a plan to add?  Tar also likely has to be a part of standard library, but I did not find it there...  Any known implementation of tar in V?

No, tar is not part of coreutils.
It doesn't have to be part of the standard library.
An initial attempt to create tar support in V is here: https://github.com/csfore/vtar


Feat/docs

Created a code of conduct and contributing file, and also brought the README up to orbitdb standards. Added some keywords to the package.json (more needed). The README still needs usage info.
@RichardLitt

Thank you @adam-palazzo!


select without "realtime" doesn't work on new table

try to make a select on a table which hasn't been touched before and data won't show. Then try "realtime" and it suddenly works!

Closed as this was an issue with an older version written in Clojure


Migrate BSD + patents license to MIT or Apache v2.0, as done for React

Hi Facebook,
Could you consider migrating the license of this project too, as has been announced and done to React?

I'm checking internally for you.

Good news, the license change has been approved!


have problem on Ubuntu 17.04 64bit

Likely the OpenVPN package installation failed. Most commonly this happens because your container was out of memory and the script got killed.
Maybe you got more issues than that, but I can't tell. Not an issue with the script.

but I install on clean VM and my RAM is 1GB
How can it happend

I don't know what are you doing wrong but you are doing something wrong.
Try to install without modifying anything, the IP, port, nothing. Just run the script and press enter five times. If it's a clean VM, the installer will work.

I now tried with 16.04 ubuntu on Digitalocean and see without modifying


[feature] Modularize service logic and implement TCP (raw) and HTTP (POST-request) services

The current implementation only allows for a single service (which handles ESCPOS print requests, currently via MQTT). Add an abstraction to allow for multiple services to be configurable.

Implemented on master


added outputs to standard output

Add cli outputs to stdout with the fmt library which allows further processing of the outputs.

Thank you for your contribution.
🚀


Replace time with chrono

Removes the old time dependency and switches all use to the more modern chrono crate. Tests pass but would love a proper review from someone familiar with the codebase. :)
I'm a bit concerned about the previous use of time::empty_tm(), as the documentation is a bit lacking, but I'm assuming that since it's mostly used for setting issuers' touched, it's used to represent the earliest possible time, so I've replaced it with chrono::DateTime::MIN_UTC.
Closes #9

Looks great, thank you! Your analysis about empty_tm seem correct to me (it's a more complicated version of "zero seconds from unix epoch", ie. zero days from epoch, zero weeks from epoch, zero ... from epoch) so your translation appears absolutely correct.


Fix mc logging and exit on first failure

Fixes https://github.com/minio/mint/issues/154

PR https://github.com/minio/mint/pull/139 is pulling functional test of mc which is taken care in that.

@balamurugana we need to release mint soon. Do you think #139 should be merged and be the part of mint first release?

@nitisht I am not saying we need to take/wait for https://github.com/minio/mint/pull/139 for the release

updated the PR @kannappanr

is travis expected to fail?

Updated the PR @kannappanr @deekoder


[Feature Request]: Ability to cancel all processes with one action

Is your feature request related to a problem? Please describe.
One of the most time consuming parts of using OpenOutpaint is when you have queued up lots of additional images with the + button but then already get the result you wanted early on and press "Y" to accept it. But then you're left with having to deal with the queue and having to keep pressing Interrupt over and over again as it tries to start the next iteration in the queue. This mechanic is the most frustrating since it can take forever to interrupt them all. Especially if the selected area was a large one.
Describe the solution you'd like
It would be great if after approving an iteration with the "Yes" button, that the rest of the iteration queue is removed.
Proposed workflow

Go to scenario where image is being generated and the menu is showing on the bottom of the image showing options for Y/N/+ etc
Press "Y" on the version you like which not only approves that iteration, but also automatically cancels any queued up iterations since they don't serve a purpose anymore at this point

Describe alternatives you've considered
If removing the remaining queued iterations after pressing "Yes" cannot be done for whatever reason I'm overlooking, then at least it would be nice to have an additional control in the bottom menu to just cancel the queue completely.
Additional context
No response

Thanks so much for the response zero! And I already said it in the discussions, but I'll say it again. I ADORE OpenOutpaint.
So [+] was never even designed for people to go nuts pressing over and over huh? But doesn't everyone do it hehe!
Well let me explain the psychology behind it. OpenOutpaint is an essential part of my daily workflow to fix weird eyes, hands, belly buttons and other things for the types of images I generate. Since I use it so much, I already know from experience that it might take 20 tries or more to get a wonky hand fixed. So it's already standard for me to go ahead and press [+] a gazillion times as soon as I start, knowing that I will most likely need a lot of iterations.
You might be asking "Well why don't you just click [+] once... see if you like the result... and if not, click it again?" But try and imagine how tedious that would be if this is something you do a lot of. Because you'd have to sit there doing click-wait-click-wait that for minutes on end. Sometimes it might take up to half an hour to get a weird hand or belly button fixed. By queueing a whole bunch of [+] presses, I can at least alt tab to check emails, chats etc and then come back to start cycling through the results.
And yes, I do things like:

Adjusting the denoising strength according to how different of a result I need
If a hand is especially tricky, I might try to completely cut it off and allow OpenPaint to create one "fresh", but that doesn't always work either, it's a case by case scenario.

And let me say that outside of weird eyes, belly buttons and hands, MOST other things are pretty fast and simple to get a nice result with and would never require a billion iterations like that. So if you've never had to cycle through 20 hand options, I can understand why this sort of case would come as a surprise!
Sorry for the long-drawn explanation, but just as you were surprised that anyone would use OpenOutpaint this way, I was also surprised, - assuming that everyone queues up the [+]'s hehe! So I really wanted to clearly explain how this kind of scenario can even happen!
Thanks so much for looking into it! It would be SUCH a time saver if I didn't have to sit there "Interrupting" over and over again.

aww much obliged for the kind words <3
so i just realized you might be very interested in the generate-ahead count functionality (discussion, pull request, manual - just scroll up a tad for the section on generate-ahead, it's the same for img2img) which should be available on both dream/img2img tools - i think it already does exactly what you want up to 100 images at a shot, and then if that's not enough you can push + after looking through all the results, and it already respects "queue cancelling" once a selection is made

actually i just realized with SDXLturbo (non-inpainting of course but hey) and the jump to 1st new on + option enabled, this thing kinda does firehose pretty successfully doesn't it

Thanks for linking the stuff about the generate-ahead stuff. It does look very handy! And if it respects queue cancelling beautifully then sign me up! The ability to cancel is all I ever really wanted to begin with.
OpenOutpaint is not working for me now so I can't test it yet. But dumb question: should the extension even be working right now? (or... is it actually a problem at my end after all?) Or is it just a matter of waiting for an update from you guys? Because when it stopped working after the last update, I just totally assumed (perhaps incorrectly?) that it was the Automatic1111 update that broke it and I just need to patiently wait for a OpenOutpaint update to become compatible again.
Anyhow, when and if I do ever get it to work again (or take the plunge with the standalone version), I'll definitely test this feature out. Thanks so much!

yeah, extension or standalone, it's definitely working as expected against webUI up to 1.6.x... you might want to reinstall webUI completely from scratch in a different folder somewhere just as a test, particularly if you've got a lot of extensions

Daaaammnnnnn really?

So installed a second Automatic1111 to another folder (did not reinstall Python or Git however, just automatic).
Set up my bat file like I usually have it, which is like this:

`@echo off
set PYTHON="C:\Users\andre\AppData\Local\Programs\Python\Python310\python.exe"
set GIT=
set VENV_DIR=
set COMMANDLINE_ARGS= --api --xformers --no-half-vae --medvram
call webui.bat`

Copied my models over
Started up, went to the extensions tab and installed OpenOutpaint from there (did not install any other extensions)
Unfortunately it behaved exactly like it did in my original install. I made a video for you so you can see it in action.

https://www.youtube.com/watch?v=0MU4aD0K7wg

well huh
yeah, that's assuredly not supposed to happen lol... in fact, in the entirety of openOutpaint's history, i don't believe i've ever experienced an actual OOM from any browser...
re: the 2nd install, yep, spot on, no need to reinstall git/python whatsoever, and if you wanted to try openOutpaint standalone  you'd just need to add --cors-allow-origins=http://<IP_ADDRESS>:3456 after --medvram in your COMMANDLINE_ARGS value to set webUI up, but i suspect it'll probably conk out the same way :/
in any case, for the time being i might possibly recommend trying edge since you're running windows and it should be preinstalled, or i can heartily recommend firefox lol, but tl;dr i SUSPECT this might be a chrom(ium) (and possibly extension) vs canvas thing somewhere?  if it's chrom(ium) without extension, you're the first to discover it, congratulations (?)
however if you want me to continue blah-blahing...
so until very recently, chrome and its downstream chromium dependents (i.e. every single browser that currently exists except firefox and safari) were suddenly acting quite foolish regarding HTML canvas operations (a,b,c,d), and in the past issues had arisen explicitly with extensions with a privacy focus, specifically duck duck go's extension which makes me kinda suspect some canvas skullduggery, since openOutpaint relies so heavily on HTML canvas and its use is sparing throughout webUI otherwise :/

OMG OMG OMG!!! Thank you thank you thank you! IT WORKS IN Edge JUST FINE!!!!!
See I totally thought that the recent Automatic1111 update broke it because that's exactly when OpenOutpaint stopped working. I'd been using it no problemo DAILY since the update in the fall. So of course I assumed that's what it was!
Thank you so much for the extra blah-blah too because even though I don't fully understand it all, it's still super interesting to know!

A HUGE MASSIVE thank you for taking time out of your day to try and help me with my questions. I hope that if there is anyone out there in the world that has been going through the same thing, that they stumble upon this conversation, learn a thing or two and that it might help them as well.

lmao no worries at all, browsers are such a giant catastrophic pile of moving parts that like 12 things might all get updated at once and there's a single config value that has ruined your week ;) glad to help in any case - and back to the original topic lol, is that generate-ahead sort of what you were looking for as far as "continual" generation/queue cancellation?

I tested it out and it's great! Really nice to simply put in a default generate-ahead count so I don't have to sit there clicking [+] over and over again. Really like the possibility to add the little * to one's favorites as well. But the best part of all.... You know what I'm gonna say... When I tell it to stop generating, it STOPS. Aaaahhhh bliss!
The amount of hours I would have saved if I'd only known about this feature earlier. See... originally I didn't really scour through the manual here on Github because I discovered this extension and how to use it from a YouTube video. So naturally was just using the basic tools I saw in that and was still a happy camper. But this feature makes me even happier.
Thank you again so much for everything!

no worries :) glad it's working better for your needs, i just kinda felt dumb yesterday because yeah of course your use case was 100% considered long ago, that's why generate ahead exists lol 🤦

I can totally see how you were bewildered, assuming everyone has read the manual and is aware of the feature. And even though I DID read the manual THIS time (because I was trying to find answers to my problem), that particular part that talks about "Generate-ahead" has no mention of it's bonus attribute: That it will obediently cancel/interfere when you ask it to. And of course that was originally the MAIN reason I was here to begin with. I never complained about having to click [+] over and over again. Rather that I had to click [Interfere] over and over. So I guess if there is anything useful that can be taken away from this, is perhaps to add a mention of that wonderful bonus feature in the Generate-ahead section of the manual? Or else to change OpenOutpaint to by default stop/interfere/cancel just as beautifully as it does if you have generate-ahead in use.


131 Error after placing SpaceEngineers-Dedicated.cfg in correct folder

Hi, I've followed the setup guide and used the start script to run the server. I've placed the world in the world folder, and the server config file in ./appdata/space-engineers/config/SpaceEngineers-Dedicated.cfg/ (which is a folder, not a file) however, the container keeps giving a 131 error code.
I have tried swapping the SpaceEngineers-Dedicated.cfg for the folder, and that results in docker mounting errors

Check permissions on the file. My first suggestion would be to swap the folder and file, but you have already tried that. Not sure whats going on with your setup. I would recommend deleting everything and using the start script to get a default world loaded, and then customize from there. If your still having trouble, reach out to me on the keen swh discord.

Oh, and make sure you run the start script with whatever user has uid 1000.


Axial Capacitor

Note: There are two variants: horizontal and vertical

hi @seveibar
Is it open to work on? can you please provide a doc to setup local server?

@seveibar worked on it #66. Please review it.


CI Update Teams-AI

Build.Reason:Manual by Saisang Cai
Build.Url:https://apidrop.visualstudio.com/Content CI/_build/results?buildId=365158&view=results

Opened too long time and got conflicts. Close this one.


update filip version

Currently the old filip version cause the error in Gateway. Following workarounds can solve this problem.
Specify the pydantic version in Dockerfile of the Gateway
RUN pip uninstall -y pydantic
RUN pip install pydantic~=1.10.0

Solution
Update filip version to >=0.3.0

Does not work for me - will look into this next week.


Detected bugs on this latest release

Hey there and thanks for a great plugin.
It seems this latest version introduces some issues which weren't present on the last beta that was available.
I haven't changed any code I've just updated the plugin, and all issues weren't happening before.
Before I had a v2.0 beta installed which seems to be not available anymore.
Is it possible to access any of the previous beta?
[ ] question
[X ] any problem or bug report
[ ] feature request
The plugin version:  (check one with "x")
[X] 2.0-beta3 (github)  ( latest one download today from https://github.com/mapsplugin/cordova-plugin-googlemaps )
[ ] 2.0 (npm)
If you choose 'problem or bug report', please select OS:  (check one with "x")
[X ] Android
[ ] iOS
Current behavior:
1 - When I open my map view the map_canvas element it's white, and the map doesn't show. I've placed some alerts and all plugin objects are available, the flow of the app keeps running, but simply the map doesn't show. (I have all other html elements that operate the map inside the map_canvas element)
2 - Also happened when I tap a marker and then another one, the infoWindows keept flickering.
Expected behavior:
1- (Like previously with beta) every-time I open the app and the map view, the map should appear.
2 - When tap a marker and another one only the latest marker should appear
Screen capture or video record:

Related code, data or error log (please format your code or data):
Nothing changed on my end after updating the plugin.
Thanks in advance

Please share your project files on github or others. I will check it.

I'll try to prepare the files so you can inspect better.
Meanwhile I've tried to comment all code except the map initialisation like this:
            app.map.map = window.plugin.google.maps.Map.getMap(div, {                 'mapType': plugin.google.maps.MapTypeId.ROADMAP,                 'disableDefaultUI': true,                 'controls': {                     'compass': false,                     'myLocationButton': false,                     'indoorPicker': false,                     'zoom': false                 },                 'gestures': {                     'scroll': true,                     'tilt': false,                     'rotate': true,                     'zoom': true                 },                 'camera': {                     'latLng': {"lat": myPos.latitude, "lng": myPos.longitude},                     'tilt': 0,                     'zoom': app.map.zoom,                     'bearing': 50                 },                 //https://developers.google.com/maps/documentation/android-api/style-reference                 'styles': [                     {                         "featureType": "poi.attraction",                         "elementType": "all",                         "stylers": [                             { "visibility": "off" }                         ]                     },{                         "featureType": "poi.business",                         "elementType": "all",                         "stylers": [                             { "visibility": "off" }                         ]                     },{                         "featureType": "landscape.man_made",                         "elementType": "all",                         "stylers": [                             { "visibility": "off" }                         ]                     }                 ]             });             app.map.map.addEventListener(plugin.google.maps.event.MAP_READY, function(){                 app.map.map.on(plugin.google.maps.event.CAMERA_CHANGE, function(position){                     app.map.zoomChanged(parseInt(position.zoom));                 });                 app.map.map.on(plugin.google.maps.event.MAP_CLICK, function(latLng) {                     app.map.followLocation = false;                 });                 app.map.map.on(plugin.google.maps.event.MAP_LONG_CLICK, function(latLng) {                     app.map.followLocation= false;                     app.map.suggest(latLng.lat, latLng.lng);                 });             });
And the same behaviour happens, the map normally is not visible.
It's really a strange behaviour because the map is shown some times (like 1 out 10 attempts)
Regarding the infoWindow issue here's a tiny clip that shows the behaviour happening with the latest update, when I tap the first marker the infoWindow appears as expected, but then if I start tapping more markers the video shows what happens.
Uploading IMG_8743.zip…
Thanks

I've also tried changing the background colour with:
plugin.google.maps.environment.setBackgroundColor("transparent");
Which seems to work because the map view (without showing any map) gets all black.
Also I've tried to execute this:
plugin.google.maps.environment.getLicenseInfo(function(text){ alert(text); });
And I get an alert with "OK"

Is this right?
Once again thanks.

I've also tried changing the background colour with:
plugin.google.maps.environment.setBackgroundColor("transparent");
Which seems to work because the map view (without showing any map) gets all black

This is not bug. Even if you don't install this plugin and set the background color as transparent for webview, you also get black color.

plugin.google.maps.environment.getLicenseInfo(function(text){ alert(text); });
And I get an alert with "OK"

Google changed the rule recently.
They removed the method from the Android API (I guess they reimplemented all code by themselves).
So in the Android, you don't need to display the license anymore.
But in the iOS, you still need to display the license.

Thanks for the reminder about displaying the license on iOS.
I don't know what other files I could share with you besides my map view.
We're using Telerik Kendo Mobile with Cordova version 5.0.0
And before the update the map always showed up properly, with the same markup and the same code after yesterday's update, the map just shows up very very few times.
map.txt
(renamed from .html to .txt in order to upload here)

Other things I've tried and didn't change the behaviour:

Different API keys for Android and IOS
Changed to Cordova version 6.4.0
Fixed an element which had two style attributes

Please(x3) save my time. Sharing project files on github (or other services) is only acceptable way.
I'm really busy person. Just share your code requires me to setup an application environment, and you use the JS framework, so I have to learn the framework.
I have no such a time (or if you pay $1,000 USD, I do that).
Sharing your project files on git repository saves my time a lot.
I can start to build your app, then test it. That is eligible for $1,000USD.
You just share an empty file, then what can I do?
I warned 3 times. Next time, I will close this thread mandatory (it means no more support)

Just to let you know, This morning We've updated the project with last night update and the issue seems to be resolved.
Once again thanks and sorry to make use of your busy time.

After a lot of code reducing and debugging I've manage to keep update this plugin until version 2.1.0.
The intermittent problem might have due to the some options I was specifying on getMap.
So after quite a bit of struggle, what worked for me was just using the bare minimum on getMap function.
app.map.map = window.plugin.google.maps.Map.getMap(div, {                         'mapType': plugin.google.maps.MapTypeId.ROADMAP,                         'controls': {                             'compass': false,                             'myLocationButton': false,                             'indoorPicker': false,                             'zoom': false,                             'mapToolbar' : false,                         },                     });
Further setting of options is done inside MAP_READY:
app.map.map.one(plugin.google.maps.event.MAP_READY, function(){ 	app.map.map.setOptions({ 		'gestures': { 			'scroll': true, 			'tilt': false, 			'rotate': true, 			'zoom': true 		}, 		'disableDefaultUI': true, 		'styles': app.map.getMapStyle() 	}); });
Hope it helps
Cheers


Renamed enable_code_completion_delay Issue #24352

Renamed enable_code_completion_delay -> enable_code_completion
Bugsquad edit: Fixes #24352.

For the reference, this setting was added in 6b1b3fb and the commit log does mention "a setting to disable autocomplete popup" even though it's incorrectly named.
Please change the name of the variable too to match the property, it's confusing right now. (Do it with git commit --amend to avoid adding a second commit just for that, see note on http://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#mastering-the-pr-workflow-the-rebase.)

Could you squash the commits together? See http://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#mastering-the-pr-workflow-the-rebase
Note that you need to use git push --force to update your remote branch after rebasing.

Actually I've pushed a different fix for this issue in 1039ba9ff. That setting was not just badly named, but also improperly implemented, so I prefer to simply remove it. If there are users who really want to disable auto-completion, we can readd a setting with a clearer name and implementation.


Removed before_reorganize_tests.png which was a copy/paste error

Shouldn't the image itself be also deleted?

Shouldn't the image itself be also deleted?

I believe I just did it, I hope it worked. Last time I did only git rm without git add since it was counterintuitive to do a git add for removed files and I guess that caused an issue.


MacOS not building properly

Describe the bug
Created Mostly for Tracking, however I couldn't get XLHA to build properly on my mac
Devices
Can test with PSP, and/or Vita
Versions
N/A
To Reproduce
Run the brew command to install the packages
brew install cmake gcc@10 boost ncurses libpcap libpthread-stubs
this may not be all the packages needed but it failed during the cmake portion before even trying to compile
Expected behavior
XLHA should build and run correctly on MacOS
Desktop:

OS/Distro: MacOS Monterey

Specs:
iMac (Retina 5K, 27-inch, 2019)
CPU - 3.7 GHz 6-Core Intel Core i5
RAM - 8 GB 2667 MHz DDR4
GPU - Radeon Pro 580X 8 GB

This may be just due to me doing something wrong, I will be continuing to debug however to see if I can get this to build properly.

edit: after installing the libraries that should be needed
CMake Error at /usr/local/Cellar/cmake/3.22.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find PCAP (missing: PCAP_LIBRARY PCAP_INCLUDE_DIR)
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.22.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  cmake/FindPCAP.cmake:48 (find_package_handle_standard_args)
  CMakeLists.txt:108 (find_package)

CMakeError.log
CMakeOutput.log

Try now

Works perfectly fine with dynamic linking and a disabled amfi.


ci: report bundle size on PR comments

Closes #123

Current dependencies on/for this PR:

main

PR #126   👈

This stack of pull requests is managed by Graphite.


Missing information about what CRC changes in the machine

The getting started guide does not mention anywhere e.g. "/etc/hosts" or "resolv.conf". I'd like, as a user, if I consider using CRC, to know beforehand, what it's going to touch on my laptop - both as root/sudo (/etc/*) and in my home directory.
Perhaps better not have this hard-coded in docs/getting-started but have a 'crc' sub-command to show it.
What I found out so far (by having /etc/ maintained in git and looking at my home directory, didn't check elsewhere nor the code):
crc setup:

Creates /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf
Creates /etc/NetworkManager/dnsmasq.d/crc.conf
Adds my account to the group 'libvirt'
Creates /etc/libvirt/qemu/networks/autostart/crc.xml
Creates /etc/libvirt/qemu/networks/crc.xml
Changes /etc/resolv.conf (and see also #1663 which I ran into)

crc start:

Changes /etc/hosts
Creates /etc/libvirt/qemu/crc.xml
Creates /etc/libvirt/storage/crc.xml

Both of them (didn't check which - likely setup):

Create $HOME/.config/systemd/user/crc* - user systemd units
Create $HOME/.crc - crc itself - perhaps should be made configurable somehow

Changes /etc/resolv.conf (and see also https://github.com/code-ready/crc/issues/1663 which I ran into)

This should have been resolved as we support setups with systemd-resolved. If not, this needs further investigation and more information to explain what the situation is and what you did to resolve this.

Networking setup is indeed more complex than that:
If using system networking
If systemd-resolved is used on the system

/etc/NetworkManager/dispatcher.d/99-crc.sh is created

If systemd-resolved is not used

Creates /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf
Creates /etc/NetworkManager/dnsmasq.d/crc.conf
/etc/resolv.conf is not modified by crc, this is a consequence of the configuration changes above, and an implementation detail of NetworkManager dnsmasq support.

If using usermode networking

/etc/hosts will be modified by a suid helper

If I remember correctly /etc/hosts is also modified in system networking even if it's not required there.

The sudo usage is only on the setup phase. We explicitly disallow the use of the root or administrator to execute CRC. If administrative actions are needed, this might be performed by the admin-helper. However, this is only the case on Windows.
One of my colleagues will fill you in on more details.

Creates /etc/libvirt/qemu/crc.xml
Creates /etc/libvirt/storage/crc.xml

We do not consider these changes performed by CRC. We ask qemu+kvm to create a VM and the daemon of the virtualization stack creates these files. It depends on the target distribution where these files are created or perhaps even additional changes are made.

Creates /etc/libvirt/qemu/networks/autostart/crc.xml
Creates /etc/libvirt/qemu/networks/crc.xml

With this we create our own bridge to prevent overlap or conflicts with the user specified network setup. This is only the case when 'system mode networking' is used.

crc on linux uses libvirt to manage its VM, and it creates a crc storage pool, a crc network, and a crc domain in qemu:///system. I agree how libvirtd stores this on the filesystem is an internal libvirtd implementation detail.


Make one of: test_kit_name_id,device_id,equipment_model_name required for all senders

We need all our senders to use valid device IDs at all times, because both we and the states use it for all kinds of other lookups and validations.    Currently we don't  always require a valid value.
There are several basic problems:

HHS Guidance on this issue is incredibly confusing.  There are multiple options for what you can use for device ID.
Many of our senders do not yet have their EUA (Emergency Use agreement) from the FDA, and so are not listed in the LIVD table.

Even thought HHS is confusing on this issue, we should be clear and simple.   So, we'll do the following:
For this story, in each of our covid related schemas, make sure that at least one of these is required (Cardinality ONE).

Our name
HL7 loc
Livd Location
Livd Name
Our Mapper

test_kit_name_id
?
column M
'Testkit Name ID'
livdLookup() mapper

equipment_model_name
OBX-17-2
column B
'Model'
livdLookup() mapper

device_id
OBX-17-1
column O,
'Equipment UID'
obx17() mapper

So, each sender must send at least one of the above values, with an exact string match against the appropriate column in the LIVD Table as specified above.
As long as any one of those is provided and is valid, we can calculate the other two.
OR, we also will allow the sender must match one of a small number of "Fake" allowed values, for cases where a sender does not yet have their EUA:_

Test_OTC_Device. (with the is_otc flag set to true in the LIVD-Supplemental table)
Test_Home_Device (with the is_home flag set to true in the LIVD-Supplemental table)
Test_Unproctored_Device (with the is_unproctored flag set to true in the LIVD-Supplemental table)
Test_POC_Device (not listed in the LIVD-Supplemental table)

To keep things simple, we'll allow any of those to be a valid value for test_kit_name_id, device_id, or equipment_model_name.   Why not just have a single fake value?  Because we need it for #2056.
Tasks

Review each schema, and change to enforce the cardinality: ONE rule for at least one of those fields.  Operations team must notify each sender of the impending change! Coordinate carefully with @anshulkumar-usds .
having the field filled in is insufficient - you also have to enforce that the value is valid from the LIVD table.    I suggest changing the mapper so it always runs, even when the field is non-empty.   See Carlos' #2270 and use mapperOverrideValues: true
Handle the 'fake' values.  If the mapper always runs, then do it in the mapper.
Simple rule:  If you give us a value, it better be in the LIVD table, or be one of our Fake values, or we give an error.
I suggest you try to combine livdLookup() and obx17() mappers into one new mapper that gracefully handles all three fields.
Some small senders won't be able to make these changes.   Carlos' new 'cleanser' work is meant to address those situations.
Add a check in the default qualityFilter to prevent these fake devices from feeding to states.  (Bug:  currently this means it will flow to the secondary feeds.  Bleh)

Notes

Note: the PDI schema is super confusing - it has a column they call Device ID that maps to our internal equipment_model_name, not to device_id.  Confusing.
You can do a Covid-result-metadata query and find the total list of all distinct test_kit_name_ids, and figure out which senders are sending invalid values  or no value at all.
When we roll this out live, we'll need to make sure there's no data queued for batch, because it may fail, due to the new validation rules.
Note:  You'll see that there is a device_id in our schema. If empty, it is filled in using the obx17() mapper.    Currently device_id is treated as an outgoing-schema field only.   However, it now appears we have senders sending us the device_id.
Note that currently, test_kit_name_id in our schema, if empty, is filled in using the livdLookup() mapper.    Its also a possible input to the mapper, so its playing the role of both the independent and the dependent variable, which is confusing.

@jimduff-usds to update ticket

@MikeC-A6


Problem in iconography settings

Iconography settings use a parent setting named : #smockIconsLibrary
which is not declared on OpenSmock.


Separate edit and view modes

Is it possible to add view mode, where user can only view diagram, zoom in/out e.t.c, but can't do any changes, and can do everything in edit mode?

Hi, I also just found out that there's an option for that, and I added types for it in a new version I just published. Have a look at the Storybook demo to see how it works:
https://marcveens.github.io/react-drawio/?path=/story/components-drawioembed--read-only
Let me know if this does the job for you!

At this moment i made it by creating 2 pages and in each one i import DrawIoEmbed but with different urlParameters and switch between view and edit modes by routing between this 2 pages. Not the best implementation, but it works. Cons is that we need to fetch DrawIoEmbed each time that we change mode and wait for its loading. If you have a better implementation, please, share it :)

I added an example of how to toggle between view/edit mode in Storybook: https://marcveens.github.io/react-drawio/?path=/story/components-drawioembed--read-only
I needed to add a key property to make it render properly. I'm not sure if that's only a Storybook thing, or if you also need it in a regular React application.


Refactor JFRResponse into a separate private class

There is some funk remaining, like the shared externs. This is an incremental commit on the way to moving the response-specific parsing into the response class. (so it will get cleaner as we go, but better to make the change now, before the branch diverges)


Discuss timing issues when publishing

Is this the same as #1038? @nilsml

I don't think it is exactly the same. This is more about having to wait to see the published results because of caching etc. This is "worse" than with AEM, so we should figure out exactly how this is working.

The current revalidate time is
120 seconds for all content coming from Sanity
1800 seconds (30 min) for the archived news
This means that for content that lives in Sanity, Next.js will try to re-generate the content at most once every 120 seconds.

We have adjusted the time to 60 seconds for the Sanity content.
So at the moment, the rules are:

For all content that lives in Sanity, Next.js will try to re-generate the content at most once every 60 second
For the search, we will run the query once every 5 minutes (#833 for improvements)

It is important to understand that the timing is a balance between performance, user experience and cost.  It is also important to understand the model we are using. I can recommend this article for a throughout explanation about ISR
It is an option to improve more if this is not good enough. We have discussed approaches like

Have different revalidate times for news articles and topic content
Use Next.js' new features for on-demand revalidation time for pages like the front page (for latest news component). Next.js will probably add more options to this in later releases
Create some components, like for example promo cards, with SWR or Next.js new features

Let's see how it goes with 60 seconds, if not I would suggest we use SWR for the news promotion at least.


[Bug]: 404 error at URL: /txns/rCU6S1iekH9vKrRUk6HL_UJVjFZ9E8cm5N75LQr7P-s

What happened?
A bug happened! Tranafering from my interal wallent to my phone wallet
URL
https://explorer.helium.com/txns/rCU6S1iekH9vKrRUk6HL_UJVjFZ9E8cm5N75LQr7P-s
What browsers are you seeing the problem on?
Chrome
Do you see the issue on Mobile or Desktop?
No response
Relevant log output
No response

sorry my issue


Asset version for combined assets

Greetings!
I cannot figure our whether it is possible to assign versions (packages) to combined asset files. This is what I want to do:
{% javascripts output="js/combined/main.js" package="core"
    '@AppBundle/Resources/public/js/*'
<EMAIL_ADDRESS><EMAIL_ADDRESS>%}
    <script src="{{ asset_url }}"></script>
{% endjavascripts %}

...
framework:
    templating:
        packages:
            core:
                version:              0.03
                version_format:       v%%2$s/%%1$s
                base_urls:
                    http: http://mydomain.com
                    ssl: https://mydomain.com

The desired output for the combined assets file would be
...
<script src="http://mydomain.com/v0.03/js/combined/main.js"></script>
...

Thank you in advance!

Seems like it can be achieved in the following way
{% javascripts output="js/combined/main.js"
    '@AppBundle/Resources/public/js/*'
<EMAIL_ADDRESS><EMAIL_ADDRESS>%}
    <script src="{{ asset(asset_url, 'core') }}"></script>
{% endjavascripts %}

Correct me if I'm wrong and there's a better way to do that.


Missing install instructions?

I wanted to try your repository but I believe I am missing some system dependencies?
 go get github.com/muesli/smartcrop
# pkg-config --cflags opencv opencv opencv opencv opencv opencv opencv
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
Package opencv was not found in the pkg-config search path.
Perhaps you should add the directory containing `opencv.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opencv' found
pkg-config: exit status 1

Any hints?

You're right, we need to update the install instructions for master still. In the meantime, you can try the latest stable build.

which is? :P

You can find them here: https://github.com/muesli/smartcrop/releases