v1.16.5 router is bind on port 80 and 443 regardless of router_http_port / router_https_port config

for my setup it is necessary that port 80 and port 443 are not occupied by the router.
so change the router_http_port and router_https_port, that worked flawless in version 1.15.2, today i  upgraded to version 1.16.5 now regardless of the config the router tried to bind port 80 and 443.
# docker ps
1.15.2 behavior (my setup use 8080 for port 80 and 4443 for port 443):
<IP_ADDRESS>:8080->8080/tcp,
<IP_ADDRESS>:4433->4433/tcp

in version 1.16.5 somehow the "default" ports are also bind (regardless of the router_http_port config):
<IP_ADDRESS>:80->80/tcp,
<IP_ADDRESS>:443->443/tcp,
<IP_ADDRESS>:8080->8080/tcp,
<IP_ADDRESS>:4433->4433/tcp

is there a way to prevent the binding for port 80 and 443 in ddev version 1.16.5 ?

Hi @brannow - I'm unable to recreate the behavior you report. Perhaps something else is listening on those ports (not ddev), or maybe you have more than one project running, and the other project(s) don't have the router port set, so use the default?
If you can give more information I'll be happy to chase it with you. Useful information would be the output of ddev list and docker ps -a. You might also hit http://localhost with your browser and see what the response is, find out what's listening there.
$ ddev --version
ddev version v1.16.5

rfay@rfay-mbp-2017:~/workspace/d9$ ddev config --http-port=8080 --https-port=8443
You are reconfiguring the project at /Users/rfay/workspace/d9.
The existing configuration will be updated and replaced.
Found a drupal9 codebase at /Users/rfay/workspace/d9/web
Ensuring write permissions for d9
Existing settings.php file includes settings.ddev.php
Configuration complete. You may now run 'ddev start'.
rfay@rfay-mbp-2017:~/workspace/d9$ ddev start
Starting d9...
Building ddev-ssh-agent
Creating ddev-ssh-agent ... done

ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Pushed mkcert rootca certs to ddev-global-cache/mkcert
Creating volume "ddev-d9_nfsmount" with local driver
Building db
Building web
Creating ddev-d9-db ... done
Creating ddev-d9-dba ... done
Creating ddev-d9-web ... done

Creating ddev-router ... done

Ensuring write permissions for d9
Ensuring write permissions for d9
Existing settings.php file includes settings.ddev.php
Successfully started d9
Project can be reached at https://d9.ddev.site:8443 https://<IP_ADDRESS>:55042

rfay@rfay-mbp-2017:~/workspace/d9$ curl http://localhost:80
curl: (7) Failed to connect to localhost port 80: Connection refused
rfay@rfay-mbp-2017:~/workspace/d9$ curl https://localhost:443
curl: (7) Failed to connect to localhost port 443: Connection refused
rfay@rfay-mbp-2017:~/workspace/d9$

Hi @brannow - I'm unable to recreate the behavior you report. Perhaps something else is listening on those ports (not ddev), or maybe you have more than one project running, and the other project(s) don't have the router port set, so use the default?
If you can give more information I'll be happy to chase it with you. Useful information would be the output of ddev list and docker ps -a. You might also hit http://localhost with your browser and see what the response is, find out what's listening there.
$ ddev --version
ddev version v1.16.5

rfay@rfay-mbp-2017:~/workspace/d9$ ddev config --http-port=8080 --https-port=8443
You are reconfiguring the project at /Users/rfay/workspace/d9.
The existing configuration will be updated and replaced.
Found a drupal9 codebase at /Users/rfay/workspace/d9/web
Ensuring write permissions for d9
Existing settings.php file includes settings.ddev.php
Configuration complete. You may now run 'ddev start'.
rfay@rfay-mbp-2017:~/workspace/d9$ ddev start
Starting d9...
Building ddev-ssh-agent
Creating ddev-ssh-agent ... done

ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Pushed mkcert rootca certs to ddev-global-cache/mkcert
Creating volume "ddev-d9_nfsmount" with local driver
Building db
Building web
Creating ddev-d9-db ... done
Creating ddev-d9-dba ... done
Creating ddev-d9-web ... done

Creating ddev-router ... done

Ensuring write permissions for d9
Ensuring write permissions for d9
Existing settings.php file includes settings.ddev.php
Successfully started d9
Project can be reached at https://d9.ddev.site:8443 https://<IP_ADDRESS>:55042

rfay@rfay-mbp-2017:~/workspace/d9$ curl http://localhost:80
curl: (7) Failed to connect to localhost port 80: Connection refused
rfay@rfay-mbp-2017:~/workspace/d9$ curl https://localhost:443
curl: (7) Failed to connect to localhost port 443: Connection refused
rfay@rfay-mbp-2017:~/workspace/d9$

Hi,
thx for the super quick reply!
here some meta infos:
linux kernel 5.10.1-051001-generic
Docker version 19.03.6, build 369ce74a3c

current ddev version (working version for me):
ddev -v
ddev version v1.15.2

i create a working test environment:
mkdir test
cd test
ddev config
-- everything default, just hiting enter
ddev start

next thing I check the binded ports via netstat and docker ps
docker ps
e727fe154289        drud/ddev-router:v1.15.1                             "/app/docker-entrypo…"   9 seconds ago       Up 8 seconds (healthy)    <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp   ddev-router

sudo netstat -tulpn | grep :443
tcp        0      0 <IP_ADDRESS>:443           <IP_ADDRESS>:*               LISTEN      7961/docker-proxy

notice here port 80 and 443 is successfully binded (default config) so far so good.
ddev poweroff
nano .ddev/config.yaml

how i change the router ports to:
router_http_port: 8080
router_https_port: 4443

and the same exercise as above
ddev start
docker ps
e727fe154289        drud/ddev-router:v1.15.1                             "/app/docker-entrypo…"   9 seconds ago       Up 8 seconds (healthy)    <IP_ADDRESS>:4443->4443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp, 80/tcp, <IP_ADDRESS>:8080->8080/tcp   ddev-router

sudo netstat -tulpn | grep "docker"
tcp        0      0 <IP_ADDRESS>:8026          <IP_ADDRESS>:*               LISTEN      11851/docker-proxy
tcp        0      0 <IP_ADDRESS>:4443          <IP_ADDRESS>:*               LISTEN      11875/docker-proxy
tcp        0      0 <IP_ADDRESS>:8036          <IP_ADDRESS>:*               LISTEN      11810/docker-proxy
tcp        0      0 <IP_ADDRESS>:8037          <IP_ADDRESS>:*               LISTEN      11796/docker-proxy
tcp        0      0 <IP_ADDRESS>:32775         <IP_ADDRESS>:*               LISTEN      10273/docker-proxy
tcp        0      0 <IP_ADDRESS>:32776         <IP_ADDRESS>:*               LISTEN      10445/docker-proxy
tcp        0      0 <IP_ADDRESS>:32777         <IP_ADDRESS>:*               LISTEN      10502/docker-proxy
tcp        0      0 <IP_ADDRESS>:8080          <IP_ADDRESS>:*               LISTEN      11784/docker-proxy
tcp        0      0 <IP_ADDRESS>:8025          <IP_ADDRESS>:*               LISTEN      11863/docker-proxy
tcp6       0      0 :::32773                :::*                    LISTEN      10404/docker-proxy

we see nothing is bind on port 80 and port 443 as expected, everything is working fine on version 1.15.2
now i upgrade to version 1.16.5
ddev poweroff
curl -LO https://raw.githubusercontent.com/drud/ddev/master/scripts/install_ddev.sh && bash install_ddev.sh

ddev -v
ddev version v1.16.5

now I start with the same config.yaml as above with the changed ports
ddev start

docker ps
3e09fca657a8        drud/ddev-router:v1.16.2                             "/app/docker-entrypo…"   8 seconds ago       Up 7 seconds (healthy)    <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:4443->4443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp, <IP_ADDRESS>:8080->8080/tcp   ddev-router

sudo netstat -tulpn | grep "docker"
tcp        0      0 <IP_ADDRESS>:8026          <IP_ADDRESS>:*               LISTEN      16766/docker-proxy
tcp        0      0 <IP_ADDRESS>:443           <IP_ADDRESS>:*               LISTEN      16804/docker-proxy
tcp        0      0 <IP_ADDRESS>:4443          <IP_ADDRESS>:*               LISTEN      16790/docker-proxy
tcp        0      0 <IP_ADDRESS>:8036          <IP_ADDRESS>:*               LISTEN      16752/docker-proxy
tcp        0      0 <IP_ADDRESS>:8037          <IP_ADDRESS>:*               LISTEN      16738/docker-proxy
tcp        0      0 <IP_ADDRESS>:32778         <IP_ADDRESS>:*               LISTEN      14483/docker-proxy
tcp        0      0 <IP_ADDRESS>:32779         <IP_ADDRESS>:*               LISTEN      14613/docker-proxy
tcp        0      0 <IP_ADDRESS>:32780         <IP_ADDRESS>:*               LISTEN      14625/docker-proxy
tcp        0      0 <IP_ADDRESS>:80            <IP_ADDRESS>:*               LISTEN      16818/docker-proxy
tcp        0      0 <IP_ADDRESS>:8080          <IP_ADDRESS>:*               LISTEN      16726/docker-proxy
tcp        0      0 <IP_ADDRESS>:8025          <IP_ADDRESS>:*               LISTEN      16778/docker-proxy
tcp6       0      0 :::32774                :::*                    LISTEN      14691/docker-proxy

as you can see 80 and port 443 is binded + port 4443 & 8080
if you can cannot verify this behavior,
this might be a good thing for me so i must simply wipe ddev and docker (configs) and reinstall everything and hope its gone.

Hi,
thx for the super quick reply!
here some meta infos:
linux kernel 5.10.1-051001-generic
Docker version 19.03.6, build 369ce74a3c

current ddev version (working version for me):
ddev -v
ddev version v1.15.2

i create a working test environment:
mkdir test
cd test
ddev config
-- everything default, just hiting enter
ddev start

next thing I check the binded ports via netstat and docker ps
docker ps
e727fe154289        drud/ddev-router:v1.15.1                             "/app/docker-entrypo…"   9 seconds ago       Up 8 seconds (healthy)    <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp   ddev-router

sudo netstat -tulpn | grep :443
tcp        0      0 <IP_ADDRESS>:443           <IP_ADDRESS>:*               LISTEN      7961/docker-proxy

notice here port 80 and 443 is successfully binded (default config) so far so good.
ddev poweroff
nano .ddev/config.yaml

how i change the router ports to:
router_http_port: 8080
router_https_port: 4443

and the same exercise as above
ddev start
docker ps
e727fe154289        drud/ddev-router:v1.15.1                             "/app/docker-entrypo…"   9 seconds ago       Up 8 seconds (healthy)    <IP_ADDRESS>:4443->4443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp, 80/tcp, <IP_ADDRESS>:8080->8080/tcp   ddev-router

sudo netstat -tulpn | grep "docker"
tcp        0      0 <IP_ADDRESS>:8026          <IP_ADDRESS>:*               LISTEN      11851/docker-proxy
tcp        0      0 <IP_ADDRESS>:4443          <IP_ADDRESS>:*               LISTEN      11875/docker-proxy
tcp        0      0 <IP_ADDRESS>:8036          <IP_ADDRESS>:*               LISTEN      11810/docker-proxy
tcp        0      0 <IP_ADDRESS>:8037          <IP_ADDRESS>:*               LISTEN      11796/docker-proxy
tcp        0      0 <IP_ADDRESS>:32775         <IP_ADDRESS>:*               LISTEN      10273/docker-proxy
tcp        0      0 <IP_ADDRESS>:32776         <IP_ADDRESS>:*               LISTEN      10445/docker-proxy
tcp        0      0 <IP_ADDRESS>:32777         <IP_ADDRESS>:*               LISTEN      10502/docker-proxy
tcp        0      0 <IP_ADDRESS>:8080          <IP_ADDRESS>:*               LISTEN      11784/docker-proxy
tcp        0      0 <IP_ADDRESS>:8025          <IP_ADDRESS>:*               LISTEN      11863/docker-proxy
tcp6       0      0 :::32773                :::*                    LISTEN      10404/docker-proxy

we see nothing is bind on port 80 and port 443 as expected, everything is working fine on version 1.15.2
now i upgrade to version 1.16.5
ddev poweroff
curl -LO https://raw.githubusercontent.com/drud/ddev/master/scripts/install_ddev.sh && bash install_ddev.sh

ddev -v
ddev version v1.16.5

now I start with the same config.yaml as above with the changed ports
ddev start

docker ps
3e09fca657a8        drud/ddev-router:v1.16.2                             "/app/docker-entrypo…"   8 seconds ago       Up 7 seconds (healthy)    <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:4443->4443/tcp, <IP_ADDRESS>:8025-8026->8025-8026/tcp, <IP_ADDRESS>:8036-8037->8036-8037/tcp, <IP_ADDRESS>:8080->8080/tcp   ddev-router

sudo netstat -tulpn | grep "docker"
tcp        0      0 <IP_ADDRESS>:8026          <IP_ADDRESS>:*               LISTEN      16766/docker-proxy
tcp        0      0 <IP_ADDRESS>:443           <IP_ADDRESS>:*               LISTEN      16804/docker-proxy
tcp        0      0 <IP_ADDRESS>:4443          <IP_ADDRESS>:*               LISTEN      16790/docker-proxy
tcp        0      0 <IP_ADDRESS>:8036          <IP_ADDRESS>:*               LISTEN      16752/docker-proxy
tcp        0      0 <IP_ADDRESS>:8037          <IP_ADDRESS>:*               LISTEN      16738/docker-proxy
tcp        0      0 <IP_ADDRESS>:32778         <IP_ADDRESS>:*               LISTEN      14483/docker-proxy
tcp        0      0 <IP_ADDRESS>:32779         <IP_ADDRESS>:*               LISTEN      14613/docker-proxy
tcp        0      0 <IP_ADDRESS>:32780         <IP_ADDRESS>:*               LISTEN      14625/docker-proxy
tcp        0      0 <IP_ADDRESS>:80            <IP_ADDRESS>:*               LISTEN      16818/docker-proxy
tcp        0      0 <IP_ADDRESS>:8080          <IP_ADDRESS>:*               LISTEN      16726/docker-proxy
tcp        0      0 <IP_ADDRESS>:8025          <IP_ADDRESS>:*               LISTEN      16778/docker-proxy
tcp6       0      0 :::32774                :::*                    LISTEN      14691/docker-proxy

as you can see 80 and port 443 is binded + port 4443 & 8080
if you can cannot verify this behavior,
this might be a good thing for me so i must simply wipe ddev and docker (configs) and reinstall everything and hope its gone.

@rfay some good news.
it seems it is a issue related with docker v19,
after i force upgrade to Docker version 20.10.2, build 2291f61 ports are correct.
for me this issue is now /closed

@rfay some good news.
it seems it is a issue related with docker v19,
after i force upgrade to Docker version 20.10.2, build 2291f61 ports are correct.
for me this issue is now /closed

I'm glad it's working! Don't forget that you have to ddev poweroff when upgrading every time to make sure the router gets rebuilt.

I'm glad it's working! Don't forget that you have to ddev poweroff when upgrading every time to make sure the router gets rebuilt.


Sparse GAT isn't tested properly

Describe the bug
GAT tries to uise subclassing to share tests between dense and sparse models, but accidentally defines a function instead of a subclass:
https://github.com/stellargraph/stellargraph/blob/ec132647e5cf43ff683e3f2e72e18ac6daa98202/tests/layer/test_graph_attention.py#L625-L627
To Reproduce

Look at test_graph_attention.py

Observed behavior
The sparse tests define a new function that is ignored.
Expected behavior
The sparse tests should declare a subclass, like class TestGATsparse(Test_GAT):
Environment
Operating system: all
Python version: all
Package versions: StellarGraph ec132647e5cf43ff683e3f2e72e18ac6daa98202
Additional context
N/A

Ops


Add wildcard match to acme domains

What does this PR do?

Allows matching of hosts like x.traefik.wtf to ACME wildcard
certs *.traefik.wtf

Motivation

Tested wildcard acme v2 and noticed that it was not properly selecting my wildcard domains

More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes

@oldmantaiter could you explain more in details why you added those lines in getCertificateForDomain() ?
			if strings.HasPrefix(domain, "*.") {
				selector := "^" + strings.Replace(domain, "*.", "[^\\.]*\\.", -1) + "$"
				if domainCheck, _ := regexp.MatchString(selector, domainToFind); domainCheck {
					return domainsCertificate, true
				}
			}

@emilevauge for sure.
I was testing the new ACMEv2 merge and created a certificate for (as an example) *.foo.bar and have backends that route based on Host:x.foo.bar  or Host:y.foo.bar. These were not properly getting the stored ACME wildcard cert because this function only looked for and exact string match

@emilevauge This code snippet has been repeated in the acme package 3x already, and in server.go, do you think that we should refactor this to be a more unified function, to check for matches better? rather than re-duplicating code?

@dtomcej I noticed the same, I can combine if that is easier

I am agree with @dtomcej .
I guess we can transform the list of DomainCertificate into a map[string]*tls.Certificate instead of a []string in the loop instruction (by using the metod Domain.ToStrArray as keys and the DomainCertificate.Certificate as values).
Then, we can use acme.searchProvidedCertificateForDomains method instead of copy/paste the regexp matching.
@oldmantaiter Do you think you can do it? Or do you need some help?

@oldmantaiter
Many thanks for your PR.
Because this fix is very important, I did the pattern matching refacto in the way to generate quickly a new Release Candidate.

@nmengin Thanks. There is one more place I saw this pattern matching code (provider/acme/provider.go) so I was going to put the function inside provider/acme and reference it from the acme package.

@oldmantaiter I saw it too and I am currently analyzing a solution to integrate all of them.
I think it will be in an other PR.
Many thanks for your feedback :+1:


Change landscape ListView into protrait

Change landscape ListView into protrait

I can't sign yet so I didn't build the apk.


Fix configuration typo in snapshot-dependencies.md

Changed configuraiton to configuration in the "Enforce revisions synchronization" section of the table.

Hi Martin, sorry for the delayed response. Thank you for fixing this typo! I will now merge your request.


Added namespace creation to docker-registry-secrets Helm chart

Fixes issue where docker-registry-secrets Helm release fails to install
due to missing namespace. This commit adds a namespace field to the
values.yaml file with create set to true and name set to
jenkins-agents to create the namespace before installing the Helm chart.
Additionally, the imageCredentials.namespaces[1] value is updated to include the jenkins-agents namespace.
This resolves the issue and allows for successful installation of the docker-registry-secrets Helm chart.
Issue: #2278

Hi @Giridharan002 , it is really nice of you to propose a contribution.
However we cannot accept it for numerous reasons:

First, a pull request "out of the wild" is usually not the best place to start. Because a Pull Request is usually the "How", we require to agree on the "What" in the issue you would want to contribute to. We would want you to describe your solution (and how did you test it) as part of the issue https://github.com/jenkins-infra/kubernetes-management/issues/2278.

Second, the change you are proposing is not valid for helmfile:

in clusters/doks.yaml: failed to read doks.yaml: reading document at index 1: yaml: unmarshal errors:
  line 18: cannot unmarshal !!map into string
  line 22: field version not found in type state.helmStateAlias
  line 29: field secrets not found in type state.helmStateAlias

You might want to check the README (https://github.com/jenkins-infra/kubernetes-management#how-to-debug-deployments) and test with the helmfile command line beforehand.
It looks like you are mixing helm and helmfile concept: I recommend you to check https://helmfile.readthedocs.io/en/latest/#getting-started in order to get a better understanding.

Finally, why would you want to only apply the change on only one of our clusters, while other clusters would be the target.

Under these elements, I'm gonna close this pull request until we can agree on a solution on the issue before writing code.
I'm sorry for not being able to accept your contribution, but the infrastructure might not be the easiest way to get started on contributing to an open source.
If you are still interested, you are welcome to contact us on the IRC channel #jenkins-infra (https://www.jenkins.io/chat/) to discuss a bit more (prefer EU working hours).


Feature: Fetch messages sorted by a criteria on the API

As a user of the API, I would like to have the result sorted using the API 's sort_by and sort_order arguments.
See the API implementation: https://github.com/aleph-im/pyaleph/blob/main/src/aleph/web/controllers/messages.py#L58-L68

Solved by 2 PRs:

PR 1 on SDK
PR 2 on PyAleph


added order states

Added order states. Inputs welcome.

maybe OPEN_INITIATED -> OPENING and CLOSE_INITIATED -> CLOSING ?

looks good to me


Initial grouping of layers

WIP
also need to decide on how to do reordering when having groups, or maybe we decide not to do reordering when we have groups?

Coverage decreased (-0.7%) to 97.603% when pulling c23e60d0898731dba11e5f354fd20552ff117927 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage decreased (-0.9%) to 97.391% when pulling d8a3b9af4f8bf69c9296ad5797abc6cdc6e13274 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage increased (+0.03%) to 98.37% when pulling ac4e7d96e706d31fb6416a9bb0ef083a7becc833 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage increased (+0.03%) to 98.37% when pulling 50a404419dbe60dfc17b1548ec0f8dfa2b056a6a on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage increased (+0.03%) to 98.37% when pulling e8c8f24809b7ae4fe798f943e8703546cd066004 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage decreased (-1.2%) to 97.112% when pulling 1b6cc8a673391e3b3e9e8072925a2b19e9fcc9c9 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage decreased (-0.4%) to 97.968% when pulling 15cb281ed5f25e23b40e781aec29c2825a78ae9c on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage decreased (-0.3%) to 98.075% when pulling f474d28890658a8f83b31c63387db40d06c1cd6e on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

Coverage decreased (-0.05%) to 98.289% when pulling 67e7cc23876f3ef585181bb5e720c066076d8465 on bartvde:layer-groups into c9e34ca5dea40b913e1602fc781efc64601db00e on boundlessgeo:master.

@theduckylittle this is ready again for review
I incorporated your comment and also added support for exclusive groups with radio buttons instead of checkboxes

Coverage decreased (-0.05%) to 98.289% when pulling 0a710aed9e890eb9cf59d6aecc5c1a919a79403c on bartvde:layer-groups into 0caad898280bdc341da4022ed67ee11b1eb38e9a on boundlessgeo:master.

Coverage decreased (-0.05%) to 98.323% when pulling 0728a5cfceeabb2d6e35876c25b34d99d6e837ad on bartvde:layer-groups into d62993dc6f6b47f0f84cc34444d4dbee43b450f5 on boundlessgeo:master.

Coverage decreased (-0.05%) to 98.323% when pulling b366f3324e14f9b88c55eea472a267ac71a6f539 on bartvde:layer-groups into d62993dc6f6b47f0f84cc34444d4dbee43b450f5 on boundlessgeo:master.


Windows I18n directory problem

Because of the windows 10 custom named directories when using a not English language
Dirs like ./Desktop and ./Pictures do not exist on a not English system.
This might include the names of Desktop in another language
Note i have not tested English directories on a no English system but i guess it will fail.

After some stack overflow questions i found that those I18n names are kinda fake.
Closing issue because every dir name will be the same in every language


PySide dependency error

I followed your instructions to run and visualize copy_task.py in visdom, but am encountering some dependency errors. I am using Python 3.6.
First error when running python ./tasks/copy_task.py -cuda 0:
 File "C:\Users\alexander.d.payne\AppData\Local\Programs\Python\Python36\lib\site-packages\pyflann\bindings\flann_ctypes.py", line 171, in <module>    raise ImportError('Cannot load dynamic library. Did you compile FLANN?') ImportError: Cannot load dynamic library. Did you compile FLANN?
pip installed pyflann, and ran again:
 File "C:\Users\alexander.d.payne\AppData\Local\Programs\Python\Python36\lib\site-packages\pyflann\__init__.py", line 27, in <module>     from index import * ModuleNotFoundError: No module named 'index'
pip installed index, and ran again:
 C:\Users\alexander.d.payne\Documents\pytorch-dnc-master>pip install index Collecting index   Downloading ...files.pythonhosted.org/packages/7f/59/65da893e04f3eb49f73e6770e0999c57230669a484b14ca574154e9b75d3/index-0.2.tar.gz Collecting PySide (from index)   Downloading  ...files.pythonhosted.org/packages/36/ac/ca31db6f2225844d37a41b10615c3d371587677efd074db29855e7035de6/PySide-1.2.4.tar.gz (9.3MB)     100% |████████████████████████████████| 9.3MB 3.2MB/s     Complete output from command python setup.py egg_info:     only these python versions are supported: [(2, 6), (2, 7), (3, 2), (3, 3), (3, 4)] Command "python setup.py egg_info" failed with error code 1 in C:\Users\ALEXAN~1.PAY\AppData\Local\Temp\pip-install-aph21f59\PySide\
Even if I switched to Python 3.4 I don't see a torch option for that version at their website https://pytorch.org/, is there anyway around this? Thank you.

First off, never tested this on Windows :(
The index that pyflann is trying to import is this file, not the package.
I'm not really sure if pyflann is supported on windows. FAISS is not. I don't have access to a windows PC to test this out either.
I'd really recommend trying the deep learning ecosystem on Linux / OSX instead, simply because everyone else uses and develops on these platforms. You know, network effects.

I appreciate the quick response. I'll try it out in Linux, and I'll also let you know if I find a solution for Windows.

Closing this issue, Please open a fresh one if problem persists.


hoping to fix slick failure hanging whole JDK 10 build

see e.g. https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk10-integrate-community-build/84/

queued:
https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk10-integrate-community-build/85/

huh, it works. go figure


Split List View Widgets into "List View" and "Grid View"

List View Widgets had List View and Grid View stacked together, they can split into two
separate demos.
Introduced initial implementation for the same, Descriptions for both may need further discussion
Let me know any changes that need to be done
Screengrabs for the changes :
Grid View
https://github.com/workbenchdev/demos/assets/72352158/d6f2f7e4-05f9-4455-a5e8-62caf72248c5
List View
https://github.com/workbenchdev/demos/assets/72352158/b1fd5b2d-1133-42bf-98be-b2504c82331b
Part of https://github.com/workbenchdev/demos/issues/121

Arrange a large and dynamic list of items in columns

On it!
The documentation link for ColumnView points to https://docs.gtk.org/gtk4/class.ColumnView.html, should I split it into documentation and API reference similar to ListView and Documentation goes to : https://docs.gtk.org/gtk4/section-list-widget.html


Channels Guide

I have a m3u list that contains the show tv name in addition to the name of the channel. Is it possible to show it through a modification?

Yes, but you have to change the parser, which is not a piece of cake

Could you provide the example?
пт, 27 июля 2018 г., 0:24 Fabián Delgado<EMAIL_ADDRESS>
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sudo97/Roku-IPTV/issues/5#issuecomment-408239829, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADjhTOMqDwhpSqThPah7Vb33WwK5YRmgks5uKjOYgaJpZM4VZRDX
.

what is this?


Add compact mode

Using this for a while I frequently ran into issues when the terminal was pretty slim, e.g. at the bottom of an IDE. So I've created a compact mode that saves 8 lines of precious terminal space.
I've added a new config option that allows to switch the style between full(the 'old' style), compact (the new one added in this PR) and auto, which toggles the compact style at a height of 14 lines (at 14 lines there are still 5 entries visible with full style). auto is used by default.
compact style

full style with my VS Code terminal for color reference.

This is great, thanks for the work here! 🚀


Bug: Owned crew stats differ on shared profile

Numbers in stats differ between General and Shared profile

I think that actually might be caused by not properly updating the shared profile (seems related to #372). So I guess that is some browser or server cache issue


attempt to fix the lack of archetype-catalog.xml publication by the release build; lets make a normal jar and see if that helps

attempt to fix the lack of archetype-catalog.xml publication by the release build; lets make a normal jar and see if that helps

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/251/
Test PASSed.


Add gitea.trust_bundle parameter for specifying TLS CA bundle configmap

Currently, there is no way to add an additional trusted CA bundle into the Gitea pod. This PR adds an additional parameter for specifying the name of a ConfigMap that will be mounted under /etc/pki/tls/certs.

Thanks for opening this, @adamgoossens 👍! My pair of pennies:

We'll need to expose trust_bundle as a field on the CRD here: https://github.com/RedHatGov/gitea-operator/blob/main/config/crd/bases/redhatgov.io_giteas.yaml#L46
Should we be using a Secret instead of aConfigMap for cert data?
Could we use something more self-evident for the property name, like trust_bundle_secret? As is, I feel it's easy to misinterpret what the property is for.

Thanks for opening this, @adamgoossens 👍! My pair of pennies:

We'll need to expose trust_bundle as a field on the CRD here: https://github.com/RedHatGov/gitea-operator/blob/main/config/crd/bases/redhatgov.io_giteas.yaml#L46
Should we be using a Secret instead of aConfigMap for cert data?
Could we use something more self-evident for the property name, like trust_bundle_secret? As is, I feel it's easy to misinterpret what the property is for.

Hi @andykrohg , thanks for the feedback!
Regarding the CRD - will do. Same for the name, although based on my next comment how about trust_bundle_configmap?
I don't think we need a Secret as the trust bundle only has public CA certificates and no keys. That also allows the user to take advantage of automatic injection via the ConfigMap, although that's OpenShift only. Let me know what you think!

Hi @andykrohg , thanks for the feedback!
Regarding the CRD - will do. Same for the name, although based on my next comment how about trust_bundle_configmap?
I don't think we need a Secret as the trust bundle only has public CA certificates and no keys. That also allows the user to take advantage of automatic injection via the ConfigMap, although that's OpenShift only. Let me know what you think!

@andykrohg what do you think of this with the changes in place? I can use this as a pattern to add trust stores to other components, if the team is happy with the approach :)

@andykrohg what do you think of this with the changes in place? I can use this as a pattern to add trust stores to other components, if the team is happy with the approach :)

@andykrohg should this be re-based against the develop branch instead?

@andykrohg should this be re-based against the develop branch instead?

Indeed it should. That may not be part of the workflow for much longer, but it still is for the time being.

Thanks @andykrohg . I've made the camel case changes and ready for another review :)

Lgtm, but we're still waiting on CI conversion from Travis to GH actions 😭 are you still working on this @jharmison-redhat ?

I'll try to finish it today. PTO so shouldn't get distracted by things.
For now if you need to cut a release you should be able to manually execute Travis steps - if not, text/signal me. Will be closing work accounts shortly :)

Please rebase to use the new test suite!

Rebased against develop and added a line to the README regarding having the CM in the same namespace at the Gitea CR.


Create eslint / prettier for backend/node package

Fixed by #24

Re-opening this since it doesn't work for me.

We are missing installing the required lib on the backend package.


just return if the event_list is empty when calling process_events_until_done

In the case where the event_list is empty, it errors out and gets this stack trace:
sh-4.2$ sudo cat /var/chef/.chef/local-mode-cache/cache/chef-stacktrace.out
Generated at 2019-08-16 15:12:23 +0000
ArgumentError: wrong number of arguments (given 0, expected 1+)
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/event_dispatch/dispatcher.rb:59:in `call_subscribers'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/event_dispatch/dispatcher.rb:84:in `process_events_until_done'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/event_dispatch/dispatcher.rb:31:in `enqueue'
(eval):2:in `updated_cookbook_file'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/cookbook/synchronizer.rb:290:in `sync_file'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/cookbook/synchronizer.rb:176:in `block (2 levels) in sync_cookbooks'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/util/threaded_job_queue.rb:52:in `block (3 levels) in process'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/util/threaded_job_queue.rb:50:in `loop'
/opt/chef/embedded/lib/ruby/gems/2.6.0/gems/chef-15.2.20/lib/chef/util/threaded_job_queue.rb:50:in `block (2 levels) in process'

Description
return sentinel clause to not do anything if the event_list is empty when process_events_until_done is invoked
Related Issue
https://github.com/chef/chef/issues/8825
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

@komealy It looks like you might have run rubocop or cookstyle on this PR based on the quote change at the top. Can you run Chefstyle on this file to make sure the style matches our style guide?

Upon further inspection, it looks like this sentinel clause actually should be looking at the element itself, and not the list. Before we move forward, I'll want to add another commit with some unit tests includes as well.

@tas50 I ran chefstyle and even added in the rules that it was enforcing into the rubocop.yml so my auto-format option in VSCode doesn't break me 😁
I also changed my sentinel clause to handle another scenario that I found. It needs to skip not just the first one, but any events that are actually nil.

Didn't mean to close it, the close and comment button is too close to Comment 🤦♂

@jaywhy13  I have not found a workaround other than this fix. My particular situation only happens once, then when I just try again, it works...

This is almost certainly a race condition caused by multithreaded access from the cookbook synchronizer to the queueing functions.  This is probably not the correct fix.

Looks like the proper fix is in #8950


feat:🎁 Run tests from buffer

Add the ability to run tests from a file or individual test methods.
This will be useful in large solutions where the solution has more than 100 projects.
It would also help if test discovery fails (I found this bug in a large solution with 25k+ tests, but haven't had time to tackle it yet) - so that I could run tests in a file or just the method I need.

There is a WIP implementation of this in #140

@molostovvs I think I might have something usable now. Would you want to test it?

Update options and add enable_buffer_test_execution = true as property under the test_runner section
Open the testrunner window
Close the testrunner window
Open a file containing a test
Navigate to the line which has the gutter sign
Press <leader>r
profit??

I'm with my family this weekend. will test this on Monday, thanks for your work!

Awesome, no rush :D

Just checked it out, works great!
Is there a plan to synchronize the test status from the status column in the buffer and in the Dotnet testrunner window?

I also have some somewhat weird symbols that apparently mean the ability to run and debug the test.

Yeah a couple things on my roadmap, either in this PR or future ones

Improve buffer indicators for runnable tests
The current icons kinda suck so i need better nerdfont icons, or use something like line highlights, virtual text etc.. Open to suggestions on this one

Sync testresults between buffer and testrunner

Add debugging so you can run <leader>d to launch debugger and start debugging the test

Update:

Added new gutter sign for test
Added F# support
Added bidirectional sync of status between testrunner and buffer

@molostovvs Do you mind retesting?

I've got character synchronization working like this:
if I run the test from the buffer, the characters in the Dotnet testrunner window are updated, but if I run the test in Dotnet testrunner, the character in the buffer is not updated until I do :e.
It seems to me that the symbol to indicate a test in the buffer should be “wide”, like flask symbol from Dotnet testrunner, because now if a test is successful, then on the line above appears the symbol “checkmark”, which is 2 times wider, which, as it seems to me, is not very aesthetic. Why not use the symbol from Dotnet testrunner?
And what about having a success/fail symbol replace the symbol for the test itself? I.e. replace the flask symbol with a “checkmark”

Fixed the following

Customizable keymappings
Gutter icon is same as test icon from testrunner
Status sign overwrites the gutter icon instead of being placed on the line above

if I run the test from the buffer, the characters in the Dotnet testrunner window are updated, but if I run the test in Dotnet testrunner, the character in the buffer is not updated until I do :e.

The sign placement is using BufReadPost to trigger. I changed this to BufEnter which might solve your problem. I didnt notice this because I have a check to ensure the buffer doesnt become stale when updated from other sources
-- auto reload file when contents are written from an external source. e.g github reset --hard
vim.api.nvim_create_autocmd({ "FocusGained", "BufEnter", "CursorHold", "CursorHoldI" }, {
  pattern = "*",
  command = "if mode() !~ '\v(c|r.?|!|t)' && getcmdwintype() == '' | checktime | endif",
})

I made all keybindings for testrunner and buffer running customizable now in options

I merged the pr now because I didnt want the branch to live for too long and the core features should be present and working. If you notice any bugs, improvements etc just open a new issue please


Fix color codes for grayscale mode

I found the cause of the issue I reported in #16. This has the fix, plus a minor spelling fix in a different function.

Looks great!
Thanks @thebitguru


Update client3:circe, client3:core to 3.8.0

Updates

com.softwaremill.sttp.client3:circe
com.softwaremill.sttp.client3:core

from 3.5.2 to 3.8.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.client3" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.softwaremill.sttp.client3" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #156.


[AutoPR vmwarevirtustream/resource-manager] Revert "regenerated all-api-versions"

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7286

(message created by the CI based on PR content)


pythonPackages.twisted: fix sandboxed build on Darwin

Description of changes
Enable __darwinAllowLocalNetworking.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Build on Darwin x86_64, sandbox enabled.
But this should target staging.

@smancill Thanks. I've updated the PR.


[BUG] ReactiveUserControl / ViewModelViewHost do not verticaly stretch wehn used with ItemsControll

Describe the bug
I bind  view models IViewFor<T> to a ItemsControl and enable scrolling by setting this in the ItemsControl:
<ItemsControl.Template>
 <ControlTemplate>
  <ScrollViewer>
   <ItemsPresenter />
  </ScrollViewer>
 </ControlTemplate>
</ItemsControl.Template>

When I use the same code for a scrollable  ItemsControl in a ReactiveUserControl and bind this  ReactiveUserControl to an outer ItemsControl the inner control is only scrolled horizontally and  not scrolled vertically. Vertically the height is not set stretched to outer controls height.
When I use a second ItemsControl.Template on the outer ItemsControl the ReactiveUserControl is bind to, the scrolling works within the outer ItemsControl and the inner content is stretched correctly.
Steps To Reproduce
Use the demo project and resize the window.
Expected behavior
The ItemsControl in the ReactiveUserControl scrolls horizontally and vertically when the outer size is too low to display the content. A secound definition of a  ItemsControl.Template is not requeired to achieve this. Therefor the height of the inner ItemsControl is stretched to the space of the outer control.
Screenshots

Showing the different cases in the demo app.
Environment

OS: Windows 10
UI Framework: .NetCore 3.0 WPF
Frameworks: Microsoft.NETCore.App, Microsoft.WindowsDesktop.App, Microsoft.NETCore.App.WPF
Pages: ReactiveUI (11.2.3), ReactiveUI.WPF (11.2.3)
Device:   PC

Try changing the HorizontalContentAlignment and VerticalContentAlignment to stretch.

On your ReactiveUserControl that is.


Clicking save save's the file twice

If I create a diagram in wavedrom, online, and then click "Save As", I have to click "save" twice before the dialog goes away, and by that time I've saved two copies of the *.json file that is my diagram.

what is your environment? OS? Browser version?

I'm using Ubuntu 16.04 with Firefox 72.0.2 (64-bit).
It used to work properly.  I'm not sure what changed.
Dan


dartDefine is not working well with values containing white spaces

In the build task, when you pass an argument's value that contains white spaces, it get split incorrectly here:

Probably you can get inspired with the Flutter's team solution to this issue, see here:
https://github.com/flutter/flutter-intellij/compare/master...devoncarew:flutter-intellij:fix_args_parsing
Basically they are using the ParametersListUtil.parse(..) method from the com.intellij.util.execution.ParametersListUtil package instead of dartDefineMulti.split(" ")

Hi, thanks for the bug report.
That function is a closed source of IntelliJ but I think it could be bypassed with the extraArgs= by passing  --dart-define-from-file.
I am currently been very busy, but I will think of a solution to consider spaces between the values as well.

I'll use a workaround to avoid using spaces. Thank you!

Test string "VALUE=foo bar character=Pinocchio id=12 name= hey 24 sheep role = sys    admin"
To fix this spacing issue, which could occur in --extraArgs and other flags as well. I think a separatorArg should be used with default value as "Space" (to support existing users, as it is currently the default value) if someone wants to use "space separated values" they should define the separator as well like ",".

@zacutti-basf you can use the workaround. I will keep this issue open for next release. I am a little busy so can't promise the next release date. But, I will try to fix it whenever the next release comes.

@zacutti-basf Hi, can you check if dart-define is working normally for you. It would really be helpful if you can validate and update me.
If this issue is fixed (so I can close this issue) on latest v0.3.13. Extension is already published on the marketplace.

@hey24sheep this is a breaking change which causes current workflows to fail. Could you please update README.md on how to use dartDefineMulti with latest changes?

@kightsonsanom it shouldn't break, the working isn't changed but how the value is processed and passed down. It should work as normal. Your values can be like "var1=value var2=value 2"
If this isn't working please let me know.
Also if this isn't working that means Analayze task must be failing as well for past few years. Because this change is just duplicating what Analyze task is doing for past few years.
Is anyone else facing issues with latest update?

@hey24sheep yes, it stopped working for me, I had to rollback to 0.3.6 version. I am not using Analyze task so I can't compare this

@kightsonsanom  thank you for logging the issue. Please give me a little time to fix it. I'll fix it at the earliest.

@zacutti-basf Hi, can you check if dart-define is working normally for you. It would really be helpful if you can validate and update me.
If this issue is fixed (so I can close this issue) on latest v0.3.13. Extension is already published on the marketplace.

hi @hey24sheep, thank you a lot for your effort and very fast fix. Unfortunately in my organization I've to request the extension upgrade and could take a while to have it approved, and then in case we have an issue could take a while for the version rollback. I'll try to create a project and make some tests in a personal laptop to give you feedback asap.

@zacutti-basf Hi, really appreciate that but latest update is failing (as updated by other users). I will make a fix within a day or two. So, you can test after that release for now I won't recommend upgrading to the latest version.

Hello, I also observed failing builds over the weekend, but I can confirm that the latest version 0.3.14 lets us build our app successfully with no further changes to our pipelines. Thanks @hey24sheep!

@ko16a46 Thank you for confirming. Web builds were failing so I have updated the extension again to 0.3.15 now. It will also fix the Test Task.
@zacutti-basf You can use the new argument separator input param (check the readme). This way you can define your own input separator and provide values like "ARG1= hello world, arg2= something with spaces,".
You all can test and check your builds. If something is not working fine, comment here and I will try to fix that. Otherwise, I will close this issue within two days of no activity, thanks.

Closing this issue as a new argument exists now separator which will solve white space issue. Use a different separator like | or ,.
If this issue is still not solved, feel free to reopen this issue.


SummaryType error if using simple_client_example.py

Describe the bug
Getting following error if using simple_client_example.py:
Traceback (most recent call last):
  File "/home/pi/toyota/simple_client_example.py", line 8, in <module>
    from mytoyota.models.summary import SummaryType
ModuleNotFoundError: No module named 'mytoyota.models.summary'

To Reproduce
Python 3.9.2
Run: python simple_client_example.py

Hello @dvogt23, thank you for reporting.
Unfortunately, I cannot reproduce the error. 😞
Which version of the mytoyota package are you using and how did you install it? Directly the latest version from the master of the repo or via pip the latest Pypi release version?

Hello @CM000n I did it with:
python -m pip install "mytoyota @ git+https://github.com/DurgNomis-drol/mytoyota@master"

Did you select the correct Python environment when calling simple_client_example.py?

Did you possibly use the python3 instead of python command at some point and got confused?

Ok, it works with your instructions, seems like smthg wrong with my old env 🤔
Thanks for your help :))


superset_config.py may need updating to latest keys in Superset

While debugging the issue of only seeing 2 Gunicorn workers, I noticed some of the keys in config aren't in use any longer:
WEBSERVER_THREADS -> SUPERSET_WORKERS
Not sure we can get around it with just -e SUPERSET_SUPERSET_WORKERS=2 - because it looks like it needs an integer:
https://github.com/airbnb/superset/blob/master/superset/config.py

Sorry, this slipped past me. I don't recall getting a notification for it. I didn't notice that the default config file had changed to much.
I'm thinking the best solution is to remove the config in this repository and force users to roll their own overrides with a superset_config.py file

And that's what I did.
I also ended up having to alter the CMD to include --address <IP_ADDRESS> to fix an issue with 0.15.0. I patched this on the master branch of superset so it should be removed in subsequent releases.
Thanks for letting me know, and sorry for any convenience in removing the default config.


How to bind control

So I wanna create a shortcut for my system monitor which would be Control+Shift+Escape. I haven't found anywhere in the documentation how to, though. How do I do this/is it even possible? If not, I'd really like to see this added.

https://wiki.hyprland.org/Configuring/Advanced-config/#binds

I'm sorry, thanks!


Graceful Error Handling

Currently we're leaving the API error raising to Faraday. That is not ideal since it's not possible to spot which Rubygpt call causes the issue.
Rubygpt must have it's own error handling.
Implementation Plan

Rescue from Faraday::Error (with it's child classes like Faraday::ClientError) in connection objects
Raise Rubygpt::Error instances with more spot-on details

Related: Raise an Rubygpt::NotConfiguredError when Rubygpt.chat.create is called before configuration.


Make curse pack types use downloadUrl if present

ideally you would merge #4 first :D

Looks good to me - have you verified that it works?

i have, and it seems to work & appears to be backwards compatible with files that don't have it as far as i can tell 🤔

Sweeet! I'll make a new release asap.


Quantum Kernel Implementation

Summary
Class that calculates a quantum kernel for use in scikit-learn machine learning algorithms for classification, clustering, dimensionality reduction and regression algorithms.
Details and comments
1. Background
The current QSVM algorithm is executed as so:
quantum_instance = QuantumInstance(BasicAer.get_backend('statevector_simulator'), seed_simulator=random_seed, seed_transpiler=random_seed)

qsvm = QSVM(feature_map=feature_map, training_dataset=training_dataset, test_dataset=test_dataset)

result = qsvm.run(quantum_instance)

and implemented in QSVM package, which consists of the following files: qsvm.py, qsvm_abc.py, qsvm_binary.py, qsvm_multiclass.py and qsvm_estimator.py.
Of these, only qsvm.py contains code that is novel, specfically, the generation of a kernel matrix that is used in a custom support vector machine algorithm (implemented in qsvm_binary.py, qsvm_multiclass.py and qsvm_estimator.py).
Many kernel machine algorithms for classification, clustering, regression and dimensionality reduction are already implemented in  scikit-learn. Common kernels are provided, but custom kernels can be specified. For example in sklearn.svm, you can define your own kernels by either giving the kernel as a python function or by precomputing the kernel matrix.
2. Proposal

Create a quantum kernel class in qiskit.ml.algorithms for use in scikit-learn algorithms.
Create a tutorial that demonstrates use of the quantum kernel class in scikit-learn classification, clustering, dimensionality reduction and regression algorithms.
Deprecate QSVM package.

Requirements:

Inputs:

feature map
x data
y data (default: none)

Options (for when y=none):

check that kernel matrix is positive semidefinite, and correct if not (default: don't check)

 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.Anna Phan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Initial comments by @stefan-woerner:

I think the results object now has a ’get_probabilities()` function in addition to statevector/counts. That has the advantage that it doesn’t depend on the type of backend that’s used and should allow you to simplify _compute_overlap.
the class doesn’t need to implement QuantumAlgorithm, so you don’t need to add the run method.
is_symmetric is hardcoded at the moment
we need to include some caching mechanism to make sure the same parameters are not evaluated twice (of cause that can be turned on/off)
if is_symmetric == True you implicitly assume that the input data is normalized, don’t you? Otherwise the diagonal doesn’t need to be = 1. I might be mixing up something here… but would be good to clarify.

Here are some thoughts for consideration:
In QSVM class there was a but a single method you could call to create the kernel matrix - it took the feature map, data and quantum instance and returned the kernel. Is there good reason to break this into a class - the class seems to do much only store feature map and quantum instance - which were not that hard to pass to a method. (Keep reading the next para though...)
The notion of having some method operating on the data, versus some other setup/configuration, is used in other algorithms where one party sets it up for later use when the actual problem data is available. Maybe the method here to build a kernel is directly callable by scikit-learn SVC to build one out on demand. In which case I can see the value of the class as it would used a similar way. If its just one party building out the kernel, and then passing it on, it is less clear to me the value of this class. Especially since what I see as a common case is trying different kernels on same data using different feature maps to see what works best. In this case having to keep passing the same data when building the kernel while the only thing that changes is the feature map, the class does not seem such a great fit to the use case.
Will you be building out a tutorial so we can see how this works/interfaces to scikit-learn. Maybe we should have a simple test case here that includes scikit-learn to ensure the commonly expected usage /integration continues to work. Perhaps this integration can be shown in sample code in the class docstring too so its easier to find for people.
In terms of the naming would one of build/construct/create as a name instead of 'evaluate' for the method that builds the kernel be better.

@woodsp-ibm The QuantumKernel can be configured, and then quantum_kernel.evaluate can be handed as a callable to many sklearn models (svm, clustering, ...). Thus, it makes sense to not only have a function, but an object that provides access to this (configured) function. The kernel object could also be set up to cache some results (not implemented yet) to increase efficiency, which requires it to be able to hold a state instead of just being a stateless function.
There is construct_circuit (which should be public, cf. @Cryoris' comment), while evaluate is evaluating the kernel function (which happens to be done via a quantum computer), so I think the name makes sense.

@stefan-woerner Yes the class makes sense if it can be used by a 3rd party - such as callable kernel method by scikit-learn. I was not sure if it could be directly used - maybe as WIP this use/integration is still being looked at. A tutorial or test case showing/confirming the integration workings would be helpful in this regard.
As to the name, I am not sure anything is 'evaluated' as I would use the word - computed for sure. But if it seems correct....

Don't forget to add a reno release note file too.

@woodsp-ibm I've now added a couple of test cases using the quantum kernel in sklearn.svm.SVC. I've also prepared a tutorial, which I'll create a PR for in qiskit-tutorials.

Be aware that qiskit tutorials are built etc against stable branch. At the next release the existing set of tutorials are expected to work still (and this will be tested at release) though they might emit deprecation warnings against the new release they MUST all work still. Once the new release is out they would be expected to be updated to use whatever function replaced the deprecated content etc. Hence the tutorials that are in there now for QSVM will need reviewing/altering for whatever is appropriate for the next release which new/updated ones.

Machine learning function has been moved to qiskit-machine-learning repository and a new pull request there Qiskit/qiskit-machine-learning#2 has been opened to add this function there now, rather than here in Aqua. As such I am closing this PR.


Use systemd_user_unit_dir from systemd.pc, if found

If pkg-config can find systemd.pc (e.g.
/usr/share/pkgconfig/systemd.pc), use its systemd_user_unit_dir for the location of psi-notify.service.
We override prefix to be whatever the user sets, which should work both for the distribution use case (prefix overridden to /usr) and for local compilation (prefix defaults to /usr/local)
Fall back to the old definition in case systemd.pc is not found.
Signed-off-by: Michel Alexandre Salim<EMAIL_ADDRESS>

Tested using mock on my Fedora machine

with this PR but no systemd.pc, a warning is generated
with this PR and adding BR on pkgconfig(systemd), no warning

Both builds succeed as before

Sorry, this one somehow slipped off my radar :-)
systemd_user_unit_dir is from v246 onwards -- about two years old since we added it in addition to the non-underscore. So probably alright, but maybe use the one without underscores just to have wider compatibility? :-)

Been a bit, but just checking if you saw the compat question above :-)


Download Link for Registration Committee

So i have uploaded a php script for the registration committee to download the excel sheet for having the data details
http://apratim.ccet.ac.in/php/dataFetch.php
this is the above link ... kindly check the same so that i can circulate the link to the registration commmittee.
The script is uploaded on the github also in the php folder

@co16353sidak Kindly see the Above link


[enhancement]: add controlnet openpose edit

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
drag and drop controlnet openpose node/bone
Alternatives
No response
Aditional Content
No response

Duplicate of #3957


SpacePin NullReferenceException OnDestroy

Description
OnDestroy (closing the SpacePin sample scene), this error occurs for every space pin ("CyanSphere", "YellowSphere", ...):
NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.WorldLocking.Core.SpacePin.OnDestroy () (at Assets/WLT/WorldLocking.Core/Scripts/SpacePin.cs:222)
Microsoft.MixedReality.WorldLocking.Examples.SpacePinManipulation.OnDestroy () (at Assets/WLT/Samples/1.4.0/WLT Examples/Scripts/SpacePinManipulation.cs:69)

SpacePin.cs at this version:
protected virtual void OnDestroy()
{
    AlignmentManager.UnregisterForLoad(RestoreOnLoad);// line 222, AlignmentManager is null
}

As well as:
NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.WorldLocking.Examples.SpacePinManipulation.OnDestroy () (at Assets/WLT/Samples/1.4.0/WLT Examples/Scripts/SpacePinManipulation.cs:70)

SpacePinManipulation.cs:
protected override void OnDestroy()
{
    base.OnDestroy();
    pinManipulator.Shutdown();// line 70, pinManipulator is null
}

Platform

Unity 2019.4.30f1
WLT 1.4.2
HoloLens 2

Hi Waffle, thanks for filing this. Unfortunately, I'm unable to repro this using the latest v1.5.0.
If you are still seeing this with the latest version, please reopen the issue. That would mean there is something you are doing that I am not, or vice versa, and I'll need to get more info on repro steps. But before spending our time drilling into that, it would be best to make sure it hasn't already been accidentally fixed as part of another refactor.
Thanks!

I am still getting this error when running SpacePin.unity. I just updated to MRTK 2.7.3, WLT 1.5.8 (Unity 2019.4.35f1).
NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.WorldLocking.Core.SpacePin.OnDestroy () (at Assets/WLT/WorldLocking.Core/Scripts/SpacePin.cs:272)
Microsoft.MixedReality.WorldLocking.Examples.SpacePinManipulation.OnDestroy () (at Assets/WLT/Samples/1.4.0/WLT Examples/Scripts/SpacePinManipulation.cs:69)

I've been downloading https://github.com/microsoft/MixedReality-WorldLockingTools-Unity/releases/download/v1.5.8/Microsoft.WorldLockingTools.Unity.Examples_wMRTKv2.7.2.unitypackage.
The Vehicle.unity scene is present, so it would appear the examples are up to date.


fix: sorted rooms.json by alphabetical order

#Description:
Sorted the rooms.json file in the temple config by alphabetical order.

@hndrewaall I will look into your suggestions. I believe that will be a good enhancement for this project to have.


Allow user to move between platforms that don't require migrations

Another incremental step to #52, we can allow users to move between platforms which are not API compatible but that don't require any database migrations.
Platform boundaries are defined in two ways:

Breaking API changes
Database migration changes

This roughly corresponds to the minor number in semantic versioning, but not always. Any time there is more than a bug fix, we give it a new gopher name and make it into its own platform.
Backend changes:

Write a function getNonMigratoryPlatformsAndVersionsList(platformId, version) that will return a list of compatible platforms and versions that don't require migrations.

Frontend changes:

Use  getNonMigratoryPlatformsAndVersionsList() to populate the version selection dropdown.
If the user selects a DIFFERENT platform, alert them (danger zone) that this choice certainly means they are moving to an incompatible version of PocketBase with incompatible API changes and that their client SDK will need to be updated accordingly.

@brewhousedigital let's hold on this one until we have an actual situation where this occurs. As of now, all platforms require db migrations.

moved to roadmap


Add jQuery validation example to the documentation

I've searched during a few hours to find a way to mimic the jQuery validation with the editor.
I've tried a lot of things but I think this one is the best one.
I've added a sample code to the documentation to show how to perform jQuery validation. If you need a jsFiddle to show you it works, please let me know.

Hey @BboyKeen, this is a bit too specific of an example to include in the documentation. Thanks for the submission though.


Refactor/update back and continue buttons

What (if any) features are you implementing?

Update "Back" and "Continue" buttons

What (if anything) did you refactor?

Include "Open Sans" font family
Float Back and Continue buttons left and right
Update App.css file and add the new css style
Apply the new css style to PreviousButton and ContinueButton

Any other comments, questions, or concerns?

Seems if we have translations setup correctly, we have to Edit/Update previousButton in Translations. We also need to make updates for other supported languages.

Looks good, just a couple small comments. @sydney-devine What do you think about adding a hover effect on the back button. @myang1010 You can change the translations locally by going to http://localhost:8000/api/translations/admin, searching the message id in the label field, clicking edit, editing the en-us option, and clicking save. You can also automatically translate it into other languages by pressing the translate button if it does not happen automatically.

I think a hover makes sense. However, the font on that is the mid blue I think. We're doing lavender on the other hover, but I think that might make the text too light. . . Open to suggestions on it.
Can I get an updated screenshot of this?

I think a hover makes sense. However, the font on that is the mid blue I think. We're doing lavender on the other hover, but I think that might make the text too light. . . Open to suggestions on it.
Can I get an updated screenshot of this?

Just an idea for hovering: consider using a lavender background or leaving it transparent?

And no hovering,

I think a hover makes sense. However, the font on that is the mid blue I think. We're doing lavender on the other hover, but I think that might make the text too light. . . Open to suggestions on it.
Can I get an updated screenshot of this?

Just an idea for hovering: consider using a lavender background or leaving it transparent?

And no hovering,

Let's do the hover but without the grey border.
For the "<" part of the back button, can we make it so it isn't so wide? Not sure if we need to pull an icon for this, but would like for it to be a bit smaller more similar to the comp. It just looks a little awkward right now.

Let's do the hover but without the grey border.

Hey @sydney-devine, just want to confirm that we want the one with lavender background without border? Something like this?

You can use this icon. React MUI has an easy way to add icons to buttons.

And thank you @CalebPena for the resource!

Let's do the hover but without the grey border.

Hey @sydney-devine, just want to confirm that we want the one with lavender background without border? Something like this?

You can use this icon. React MUI has an easy way to add icons to buttons.

And thank you @CalebPena for the resource!

@myang1010  - can we try this hex code for the hover highlight? #E6E7E8. I think the lavender is too dark with the mid blue. Can also try this #FBF9FC

@myang1010 - can we try this hex code for the hover highlight? #E6E7E8. I think the lavender is too dark with the mid blue. Can also try this #FBF9FC

Okay, here's how they look,

#E6E7E8

And #E6E7E8

The first one "#E6E7E8" is more obvious,  and the second one is a little difficult to notice.

@myang1010 Per slack thread, let's go with the original lavender

Could you pull from Marianna/734/typography-updates and use the updated style controller. Also change the base branch to Marianna/734/typography-updates

Okay, pull in the changes from Marianna/734/typography-updates and retargeting the base branch to the same one.

@CalebPena @myang1010 In staging, the "BACK" is still staying "PREV"
Also, can we increase the padding between whatever the input above the buttons is and the buttons themselves? Add another 50% of whatever is there. The buttons just seem really close to the inputs right now. Amount of spacing on Step 1 (see screenshot) is good reference.


Don't pixel align if the map isn't aligned.

If the map is not at an integer offset on the page, don't try to align texels.  This only applies to the vgl renderer.

Current coverage is 86.87% (diff: 100%)

Merging #646 into master will increase coverage by <.01%

@@             master       #646   diff @@
==========================================
  Files            87         87
  Lines          8546       8547     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7424       7425     +1
  Misses         1122       1122
  Partials          0          0

Powered by Codecov. Last update bd3727c...b9628d2


March 2024 Topics

Dual-funding BOLT Merged!

Credit: niftynei

V3 and some possible futures

Credit: instagibbs

Analysis of attempting to imbue LN commitment transaction spends with v3 semantics

Credit: sdaftuar

Human Readable Bitcoin Payment Instructions

Credit: MattCorallo

Basic vault prototype using OP_CAT

Credit: rijndael

Ecash and lightning via ZKCP

Credit: ajtowns


Allow HTTPS to be enforced

This might typically be done at a reverse proxy layer like nginx, but if you're running slackin on Heroku or Cloud Foundry or another PaaS, that may not be possible.
We're currently using Slackin at https://chat.18f.gov (hosted on an internal PaaS), and we very much need to comply with new federal requirements around HTTPS, which includes redirecting HTTP -> HTTPS, and setting an HSTS header.
Ideally, this would take the form of another couple of environment variables that turn this on, and looks for a header that indicates the original protocol. The closest thing to a standard protocol I've seen for this, that Amazon ELBs pass on, Heroku passes on, and various nginx configurations pass on, isX-Forwarded-Proto.

I am trying to get this done as well.
In the README it implies that all traffic may be HTTPS, but the app does not appear to self redirect to HTTPS


Better digit counting

https://play.rust-lang.org/?version=stable&mode=release&edition=2021&gist=0a6fb4522c0caf9a5c3addc23a3d9cb8
It's 3-5x faster and it's cleaner.
Why?
Because I have nothing better to do with my time.

Just so it's clear, the difference in performance is miniscule.
I'm just bored.


Update jest to 15.0.0

AutoMocking was removed.
ref. https://facebook.github.io/jest/blog/2016/09/01/jest-15.html

Coverage decreased (-5.2%) to 89.431% when pulling dc8d48deb2dc7fd390aa20499e0406570892e4ac on update-jest into c68823ecb49eb749a3c5f280a9f721d34c0eafe4 on master.


[Icon Request] Drone icon

[x] I'm sure this issue is not a duplicate?
[x] I want to create an icon request(if not, remove these lines below):
- Type: extension
- Icon Name: Drone
- Sample original Icon (32x32/svg/transparent whenever possible): have a PNG locally, can create an svg
- Extensions: .drone.yml

I think I have this worked up correctly on my fork, so I'm happy to submit a PR! :)

For those not familiar Drone is an awesome open source CI tool

Could you upload your png? Feel free to show us an svg icon, that would be awesome!

Here's the PNG, I can try for an SVG too, not the best with Illustrator though :p

The SVG logo is already available from the main site (http://readme.drone.io/logo.svg)

Great find!


Splitting options into separate file

There are quite a bunch of options at the moment, and more to come. So I think it's reasonable to split them into separate file for ease of use.

Coverage increased (+0.4%) to 67.347% when pulling a67204a04a155584bdc243510eb6d08cd468c204 on igsky:master into 50c2d576db8913bb0f0d29dc148a1d8886c11c80 on knq:master.

Coverage increased (+0.8%) to 67.813% when pulling 8c18c05611da5494d210e48f3fe6eeb9f50ba31c on igsky:master into 50c2d576db8913bb0f0d29dc148a1d8886c11c80 on knq:master.

Looks like my previous PR's failed due to some kind of external reason (server down or something like that).
I'm trying to keep commits atomic and easy to understand. Looking forward to hear feedback.


Integration cannot be loaded after 2022.12 update

Hi,
After upgrading to 2022.12 seems something is broken. The integration cannot be loaded after restarting HA and I got the following log errors.
Logger: homeassistant.setup
Source: setup.py:202
First occurred: 08:37:29 (1 occurrences)
Last logged: 08:37:29

Setup failed for custom integration ecowater_softener: Requirements for ecowater_softener not found: ['ecowater-softener==1.0.0'].

Logger: homeassistant.util.package
Source: util/package.py:98
First occurred: 08:37:29 (1 occurrences)
Last logged: 08:37:29

Unable to install package ecowater-softener==1.0.0: WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be795ba30>: Failed to establish a new connection: [Errno -3] Try again')': /musllinux/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7958130>: Failed to establish a new connection: [Errno -3] Try again')': /musllinux/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7959a80>: Failed to establish a new connection: [Errno -3] Try again')': /musllinux/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be79dd2a0>: Failed to establish a new connection: [Errno -3] Try again')': /musllinux/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7901840>: Failed to establish a new connection: [Errno -3] Try again')': /musllinux/
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be77d1300>: Failed to establish a new connection: [Errno -3] Try again')': /simple/ecowater-softener/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7b23b20>: Failed to establish a new connection: [Errno -3] Try again')': /simple/ecowater-softener/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7b23c10>: Failed to establish a new connection: [Errno -3] Try again')': /simple/ecowater-softener/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7b22fe0>: Failed to establish a new connection: [Errno -3] Try again')': /simple/ecowater-softener/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f1be7b23fa0>: Failed to establish a new connection: [Errno -3] Try again')': /simple/ecowater-softener/
ERROR: Could not find a version that satisfies the requirement ecowater-softener==1.0.0 (from versions: none) ERROR: No matching distribution found for ecowater-softener==1.0.0 WARNING: There was an error checking the latest version of pip.

I tried to redownloading the repository from HACS (main and the 2.0.0 too) and restarting HA … with no luck. The integration cannot be loaded after the restart.
If I delete the integration and I try to reinstall it … there is no success. It cannot be installed. Error 500
What it is weird is that the error is referring to version 1.0.0 when I already had updated it to 2.0.0  a long time ago.

One curios thing ... I installed a new Home Assistant and I made the setup from the beginning (not using backup). I installed HACS and the first integration I downloaded fromg HACS was the ECOWATER Softener ... I restarted and when I tried to install the integration from the "Integrations" section ... it was not possible.
I got error 500. "Internal Server Error"

Finally I move the NAS to a new location … I connected it to a different network and … homeassistant was able to load the Ecowater Softener integration without any problem… so the files that the integration needs (and that were deleted or corrupted after installing homeassistant 2022.12) were downloaded and installed correctly.
Now I moved again the NAS to home … and the integration was loaded Ok.
So maybe there is a problem with the router … that was blocking something (although I didn’t change anything recently) o maybe it was a problem with the ISP.
I am going to close this thread.


APT whitelist request for libmagickcore5-extra

This package is available in Ubuntu 14.04 but doesn't seem to be in 12.04


makes collapse to support two-way binding or hide exposed function(toggle,hide,show)

  @ViewChild(Collapse)
  private collapse: Collapse;

  toggleMenuItem(menuItem){
      // change by directive, instead model, the state of binding source will become inconsistent
      this.collapse.hide();
  }

There are isOpen property

there is no isOpen property in collapse.component.ts. could you point out the specific file?

not isOpen, sorry, check how it works at demo page:
https://github.com/valor-software/ng2-bootstrap/blob/master/demo/components/collapse/collapse-demo.html#L5


DEM changing of geoid not working

The function xdem.DEM.set_vref() does not seems to perfectly set the ccrs with my data. Or, .to_vref() should not use the ccrs created by .set_vref()?
To reproduce it:
GLO90_sub.tif.zip
#First, unzip the DEM .tif file in the attached archive,
import xdem
mnt = xdem.DEM(PathToDEM) # path of the .tif
mnt.set_vref('EGM08')
mnt.ccrs # prints +geoidgrids is set with EGM08, but ellipsoid is set to WGS84
print(mnt.data)
mnt.to_vref('WGS84')
print(mnt.data) # no elevation change noticed
In the example, we set the geoid to EGM08 and ask to transform it to WGS84 ellipsoid.
The mnt.ccrs prints ellipsoid WGS84 (not the geoid EGM08), and to_ref() uses the .ccrs attribute and not the vref value (which indeed is EGM08), so no transformation is done.
OS: Ubuntu 22
Environment : xdem (0.0.7) from the repository

Hello, here it is !
env_list_Luc.txt
I get this for your code : RasterioIOError: /home/beraudlu/miniconda3/envs/xdem/share/proj/us_nga_egm08_25.tif: No such file or directory

Found the issue ! You don't have proj-data installed. Run conda install -c conda-forge proj-data.
It's listed in the dependencies in environment.yml so the issue must have occurred when you installed xdem from the github repo.

What need to be modified in xdem is here. The condition to check that the file exist should be outside of the else block.

Thanks, I now have the good plot !
But my to_vref() still does not change the data.

Ok, I tried updating my pyproj version to match yours. I can reproduce your error with pyproj=3.4.0 (and 3.4.1) but it works fine with pyproj=3.3.1.
Maybe something changed in the new version.

Yep it works fine with pyproj=3.3.1 for me too, now. Thanks !

I have investigated to try to figure out what differs between the two versions of pyproj without success...
In either case, ccrs_init and ccrs_dest defined in the method to_vref are identical. Even ccrs_init.datum and ccrs_dest.datum are the same. I could not find anything about what pyproj does when initializing a CRS with the option "geoidgrids"...

Yes, geoidgrids was an argument on in its way to be deprecated I think :cry:.
Maybe we need to build the compound CRS ourselves now, I'll have a closer look next week!

Ongoing in #350

In either case, ccrs_init and ccrs_dest defined in the method to_vref are identical.

Actually ccrs_dest was being updated by a property call of vref, so the to_vref() function was behaving correctly (but poorly written, interesting to see the progress made on that side in 2.5 years!).
The reason it stopped working is because pyproj/PROJ now only transform the z coordinate if both input and output CRS are 3D. This is fairly new, which explains why everything still works on older pyproj versions. For instance, classic WGS84 CRS("EPSG:4326") does not work now, instead one needs to pass the 3D version of it CRS("EPSG:4979") (https://epsg.io/4979). See for example: https://gis.stackexchange.com/questions/436401/transforming-epsg4979-to-given-wkt-epsg6654, and several other issues in pyproj.
In addition to adapting to that new behaviour, I reworked the entire vertical CRS framework of xdem.DEM to be more robust and easy-to-use. However, I'm running into some issues to consistently cast the CRS back to 2D when they are already 3D and I need to create a CompoundCRS with a vertical CRS.
I opened a discussion in pyproj:  https://github.com/pyproj4/pyproj/discussions/1265

Actually, a good summary of that new behaviour is shown here: https://pyproj4.github.io/pyproj/stable/advanced_examples.html#promote-crs-to-3d
Update on the issue: I just added CRS.to_2d() to pyproj (simply needed to be wrapped from PROJ where it already existed). It will take some time until there's a new release and we can use that function in xDEM, so I'll finalize the PR with a NotImplemented error in the meantime :slightly_smiling_face:  (which will only apply to the case a CRS passed is 3D and need to be re-cast to 2D).


how to add  code intelliSense in vscode for python-docx

in vscode, some other third packages has code intelliSense, but not python-docx,  why ?

This has more to do with Visual Studio that python-docx I think. Try StackOverflow tagging with "VSCode".


Brightspace

Is there any deployment Guide to use within Brightspace as yet?

Hi @skmckenFVSD not yet but if you like to work through the Moodle/Canvas guides and develop and contribute instructions this would be great for the community.

@alphaz18 thanks for PR regarding the outh support for Brightspace D2L https://github.com/microsoft/Learn-LTI/pull/156
Could you also help us develop setup/config guides for D2L and PR changes for https://github.com/microsoft/Learn-LTI/blob/main/docs/CONFIGURATION_GUIDE.md to include D2L

@skmckenFVSD please could you help validate if this implementation for Brightspace D2L works on your implementation https://github.com/microsoft/Learn-LTI/pull/156

@skmckenFVSD please could you help validate if this implementation for Brightspace D2L works on your implementation https://github.com/microsoft/Learn-LTI/pull/156

@leestott Do you have any Guidance on adding it to Brightspace as yet?


Update Readme.md

update readme with the missing information and add some attractive components and badges in it.

Can you assign this issue to me,I can fix this

@pranjay-poddar  Can you assign it to me?

I would like to contribute to the particular issue, kindly assign me the concerned issue.


Latest Windows 10 updates from 12/11 seemed to have broken GPMDP

OS: Windows 10 x64
GPMDP Version: 4.6.1
Issue Descriptions: The latest Windows 10 update (KB4539684 and/or KB4533002, installed on 12/11) seems to have broken GPMDP. It's not stuck on the "Loading your music library" screen.
Steps to Reproduce: Open the app with the latest Windows 10 updates, I guess.

Coincidental, this is actually #3660


[W6.1d][W11-B1] Li Hong Sheng Gabriel

@lancehaoh
Some comments added. Please ack & close the PR after reading comments.

Alright. Thanks for reviewing my work.


Manually import critical code to Consolidate figures in distributions vignette

From PR https://github.com/poissonconsulting/ssdtools/pull/62

done with commit 41d0c65657b285a6984bd150fb207b1d65ac6897