fix topic name in rviz config

Types of PR

[ ] New Features
[ ] Upgrade of existing features
[x] Bugfix

Link to the issue
Description
How to review this PR.
Others

@yamacir-kit Do you have any idea about the build fail?

This branch is derived from the fairly old use-autoware-auto-msgs.
This is a resolved issue with the current use-autoware-auto-msgs, so run git merge origin/use-autoware-auto-msgs to this brunch.

Thanks. I did


Sample for DBTreeView from ServoyExtraComponents

Hi,
A colleague of mine wrote into Servoy Forum asking for help with DBTreeView
https://forum.servoy.com/viewtopic.php?f=22&t=23640&sid=c4b2c568632d22c9e4121bb56c69d4a6
I tried to find out how to do it but didn´t work. Do you have any simple sample on how to build a DBTreeView working with 3 tables via relations?
Regards

Solved at Servoy´s Forum


fail to install JSX on a fresh MacOS machine

I have no previous installation, on my host machine, no scripts left from old installations, and no containers.
I follow the 3-steps of the "Interactive Container" installation process as described on https://github.com/threefoldtech/jumpscaleX/blob/development_jumpscale/docs/Installation/README.md#interactive-container
It fails always at the same step:
 glendc@gdcwmbp  ~  curl https://raw.githubusercontent.com/threefoldtech/jumpscaleX/development_jumpscale/install/jsx.py\?$RANDOM > /tmp/jsx ;
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 19121  100 19121    0     0   209k      0 --:--:-- --:--:-- --:--:--  209k
 glendc@gdcwmbp  ~  chmod +x /tmp/jsx;
 glendc@gdcwmbp  ~  /tmp/jsx container_install
DOWNLOADED INSTALLTOOLS TO /tmp/InstallTools.py
provide secret to use for encrypting private key:
provide secret to use for encrypting private key (confirm):
provide secret to use for encrypting private key:
provide secret to use for encrypting private key (confirm):
 - Docker machine gets created:
af23f54815c5281268dcc02828f4fbcd54ce2528e9ca4a43c3b7174594e4ebb7
 - Docker machine OK
 - Start SSH server
cat: /root/.BASEINSTALL_OK: No such file or directory
 * Starting OpenBSD Secure Shell server sshd                             [ OK ]
 - Upgrade ubuntu
Get:1 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
Hit:2 http://ppa.launchpad.net/wireguard/wireguard/ubuntu bionic InRelease
Hit:3 http://archive.ubuntu.com/ubuntu bionic InRelease
Get:4 http://archive.ubuntu.com/ubuntu bionic-updates InRelease [88.7 kB]
Hit:5 http://mirror.unix-solutions.be/ubuntu bionic InRelease
Get:6 http://archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
Get:7 http://security.ubuntu.com/ubuntu bionic-security/universe Sources [190 kB]
Get:8 http://archive.ubuntu.com/ubuntu bionic-updates/multiverse Sources [4867 B]
Get:9 http://security.ubuntu.com/ubuntu bionic-security/main Sources [150 kB]
Get:10 http://archive.ubuntu.com/ubuntu bionic-updates/main Sources [368 kB]
Get:11 http://security.ubuntu.com/ubuntu bionic-security/multiverse Sources [2776 B]
Get:12 http://security.ubuntu.com/ubuntu bionic-security/multiverse amd64 Packages [4172 B]
Get:13 http://security.ubuntu.com/ubuntu bionic-security/universe amd64 Packages [725 kB]
Get:14 http://archive.ubuntu.com/ubuntu bionic-updates/universe Sources [331 kB]
Get:15 http://security.ubuntu.com/ubuntu bionic-security/main amd64 Packages [580 kB]
Get:16 http://archive.ubuntu.com/ubuntu bionic-updates/universe amd64 Packages [1245 kB]
Get:17 http://archive.ubuntu.com/ubuntu bionic-updates/main amd64 Packages [889 kB]
Get:18 http://archive.ubuntu.com/ubuntu bionic-updates/multiverse amd64 Packages [7235 B]
Fetched 4748 kB in 3s (1610 kB/s)
Reading package lists... Done
Building dependency tree
Reading state information... Done
11 packages can be upgraded. Run 'apt list --upgradable' to see them.
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following packages have been kept back:
  linux-headers-generic
The following packages will be upgraded:
  base-files bash iputils-ping libglib2.0-0 libssl-dev libssl1.1 openssl
  redis-server redis-tools tmux
10 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
Need to get 6177 kB of archives.
After this operation, 24.6 kB of additional disk space will be used.

(Reading database ... 55691 files and directories currently installed.)
Preparing to unpack .../base-files_10.1ubuntu2.5_amd64.deb ...
Unpacking base-files (10.1ubuntu2.5) over (10.1ubuntu2.4) ...
Setting up base-files (10.1ubuntu2.5) ...
(Reading database ... 55691 files and directories currently installed.)
Preparing to unpack .../bash_4.4.18-2ubuntu1.2_amd64.deb ...
Unpacking bash (4.4.18-2ubuntu1.2) over (4.4.18-2ubuntu1.1) ...
Setting up bash (4.4.18-2ubuntu1.2) ...
update-alternatives: error: alternative path /usr/share/man/man7/bash-builtins.7.gz doesn't exist
(Reading database ... 55691 files and directories currently installed.)
Preparing to unpack .../0-iputils-ping_3%3a20161105-1ubuntu3_amd64.deb ...
Unpacking iputils-ping (3:20161105-1ubuntu3) over (3:20161105-1ubuntu2) ...
Preparing to unpack .../1-libglib2.0-0_2.56.4-0ubuntu<IP_ADDRESS>_amd64.deb ...
Unpacking libglib2.0-0:amd64 (2.56.4-0ubuntu<IP_ADDRESS>) over (2.56.4-0ubuntu<IP_ADDRESS>) ...
Preparing to unpack .../2-libssl-dev_1.1.1-1ubuntu2.1~18.04.4_amd64.deb ...
Unpacking libssl-dev:amd64 (1.1.1-1ubuntu2.1~18.04.4) over (1.1.1-1ubuntu2.1~18.04.3) ...
Preparing to unpack .../3-libssl1.1_1.1.1-1ubuntu2.1~18.04.4_amd64.deb ...
Unpacking libssl1.1:amd64 (1.1.1-1ubuntu2.1~18.04.4) over (1.1.1-1ubuntu2.1~18.04.3) ...
Preparing to unpack .../4-openssl_1.1.1-1ubuntu2.1~18.04.4_amd64.deb ...
Unpacking openssl (1.1.1-1ubuntu2.1~18.04.4) over (1.1.1-1ubuntu2.1~18.04.3) ...
Preparing to unpack .../5-redis-server_5%3a4.0.9-1ubuntu0.2_amd64.deb ...
Unpacking redis-server (5:4.0.9-1ubuntu0.2) over (5:4.0.9-1ubuntu0.1) ...
Preparing to unpack .../6-redis-tools_5%3a4.0.9-1ubuntu0.2_amd64.deb ...
Unpacking redis-tools (5:4.0.9-1ubuntu0.2) over (5:4.0.9-1ubuntu0.1) ...
Preparing to unpack .../7-tmux_2.6-3ubuntu0.2_amd64.deb ...
Unpacking tmux (2.6-3ubuntu0.2) over (2.6-3ubuntu0.1) ...
Setting up iputils-ping (3:20161105-1ubuntu3) ...
Setting up redis-tools (5:4.0.9-1ubuntu0.2) ...
Setting up libglib2.0-0:amd64 (2.56.4-0ubuntu<IP_ADDRESS>) ...
No schema files found: doing nothing.
Processing triggers for libc-bin (2.27-3ubuntu1) ...
Setting up libssl1.1:amd64 (1.1.1-1ubuntu2.1~18.04.4) ...
Setting up openssl (1.1.1-1ubuntu2.1~18.04.4) ...
Setting up tmux (2.6-3ubuntu0.2) ...
Setting up redis-server (5:4.0.9-1ubuntu0.2) ...
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of restart.
Setting up libssl-dev:amd64 (1.1.1-1ubuntu2.1~18.04.4) ...
Processing triggers for libc-bin (2.27-3ubuntu1) ...
 - Upgrade ubuntu ended
Reading package lists... Done
Building dependency tree
Reading state information... Done
mc is already the newest version (3:4.8.19-1).
git is already the newest version (1:2.17.1-1ubuntu0.4).
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
getaddrinfo 3bot: nodename nor servname provided, or not known
getaddrinfo 3bot: nodename nor servname provided, or not known
getaddrinfo 3bot: nodename nor servname provided, or not known
copy installer over from where I install from
Warning: Permanently added '[localhost]:9122' (ECDSA) to the list of known hosts.
Warning: Permanently added '[localhost]:9122' (ECDSA) to the list of known hosts.
 - Installing jumpscaleX
Reading package lists... Done
Building dependency tree
Reading state information... Done
python3-click is already the newest version (6.7-3).
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
Connection to localhost closed.
DOWNLOADED INSTALLTOOLS TO /tmp/InstallTools.py
Fri 19 08:50:29 InstallTools.py  -2999 - repos_get                          : get code:jumpscaleX:threefoldtech (['development_jumpscale'])
Fri 19 08:50:29 InstallTools.py  -2999 - repos_get                          : get code [git] (first time): jumpscaleX

EXCEPTION

        jsx                                                          : 562 : cli()
        /root/.local/lib/python3.6/site-packages/click/core.py       : 764 : return self.main(*args, **kwargs)
        /root/.local/lib/python3.6/site-packages/click/core.py       : 717 : rv = self.invoke(ctx)
        /root/.local/lib/python3.6/site-packages/click/core.py       : 1137: return _process_result(sub_ctx.command.invoke(sub_ctx))
        /root/.local/lib/python3.6/site-packages/click/core.py       : 956 : return ctx.invoke(self.callback, **ctx.params)
        /root/.local/lib/python3.6/site-packages/click/core.py       : 555 : return callback(*args, **kwargs)
        jsx                                                          : 299 : installer.install(sandboxed=False, force=force, gitpull=pull)
        /tmp/InstallTools.py                                         : 2944: self.repos_get(pull=gitpull)
        /tmp/InstallTools.py                                         : 3002: Tools.code_github_get(repo=sourceName, account=self.account, branch=self.branch, pull=pull)
        /tmp/InstallTools.py                                         : 1742: rc, _, _ = Tools.execute(C, args=args, die=True, showout=False)
        /tmp/InstallTools.py                                         : 1255: original_command=command,
        /tmp/InstallTools.py                                         : 1417: raise RuntimeError(msg)

Fri 19 08:51:06 core.py          - 762 -                                    : EXCEPTION:
Could not execute: 'bash /tmp/jumpscale/scripts/62544.sh'stderr:
    Cloning into 'jumpscaleX'...
    ssh_exchange_identification: Connection closed by remote host
    fatal: Could not read from remote repository.

    Please make sure you have the correct access rights
    and the repository exists.

CANNOT CONTINUE
Connection to localhost closed.

EXCEPTION

        /tmp/jsx                                                     : 562 : cli()
        /usr/local/lib/python3.7/site-packages/click/core.py         : 764 : return self.main(*args, **kwargs)
        /usr/local/lib/python3.7/site-packages/click/core.py         : 717 : rv = self.invoke(ctx)
        /usr/local/lib/python3.7/site-packages/click/core.py         : 1137: return _process_result(sub_ctx.command.invoke(sub_ctx))
        /usr/local/lib/python3.7/site-packages/click/core.py         : 956 : return ctx.invoke(self.callback, **ctx.params)
        /usr/local/lib/python3.7/site-packages/click/core.py         : 555 : return callback(*args, **kwargs)
        /tmp/jsx                                                     : 242 : docker.jumpscale_install(branch=branch, redo=reinstall, pull=pull, web=web)
        /tmp/InstallTools.py                                         : 3558: self.sshexec(cmd)
        /tmp/InstallTools.py                                         : 3400: Tools.execute(cmd2, interactive=True, showout=False, replace=False, asfile=True)
        /tmp/InstallTools.py                                         : 1255: original_command=command,
        /tmp/InstallTools.py                                         : 1262: res = Tools._execute_interactive(cmd=command, die=die, original_command=original_command)
        /tmp/InstallTools.py                                         : 667 : % (original_command, returncode)

Fri 19 10:51:06 core.py          - 762 -                                    : EXCEPTION: ***ERROR EXECUTE INTERACTIVE:
Could not execute:ssh -oStrictHostKeyChecking=no -t root@localhost -A -p 9122 'cd /tmp;python3 jsx install'
returncode:1

CANNOT CONTINUE

Note that I do have an ssh-key loaded in my host system.

@zgorizzo69 helped me, and allowed me to install JSX as follows from my host machine (with the docker container already created by my failed installation, but manually stopped it prior to running these commands):
$ cd ~/sandbox/code/github/threefoldtech
$ git clone git@github.com:threefoldtech/jumpscaleX.git
$ cd jumpscaleX
$ git checkout development_jumpscale
$ cd install
$ python3 jsx.py container_install

After that is finished I am able to go into the kosmos from my host machine using:
$ python3 jsx.py container_install

we should know what the issue was.
The installer should be smart enough to figure out how to clone and which credentials to use.
are we sure the right branch was used for the jsx install tool?

are we sure the right branch was used for the jsx install tool?

100% sure it was the correct branch, if the correct branch is development_jumpscale that is, as is confirmed by the logs above

Now, Checking installation on a freash MacOS Mojave

On a fresh MacOS user:

ssh-keygen
ssh-add
ssh-add -L
added the key to GitHub
execute the following from docs

# download
curl https://raw.githubusercontent.com/threefoldtech/jumpscaleX/development_jumpscale/install/jsx.py?$RANDOM > /tmp/jsx ;\
# change permission
chmod +x /tmp/jsx; \
# install
/tmp/jsx container-install

and installation works Ok

Stacktrace from the point in the ticket
opy installer over from where I install from
Warning: Permanently added '[localhost]:9122' (ECDSA) to the list of known hosts.
Warning: Permanently added '[localhost]:9122' (ECDSA) to the list of known hosts.
 - Installing jumpscaleX
Warning: Permanently added '[localhost]:9122' (ECDSA) to the list of known hosts.
Reading package lists... Done
Building dependency tree
Reading state information... Done
python3-click is already the newest version (6.7-3).
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
Connection to localhost closed.
DOWNLOADED INSTALLTOOLS TO /tmp/InstallTools.py
Ok to use key: 'test_jsx_installer@MacBookPro' as your default key?
make your choice (y,n): y
Mon 22 14:46:10 InstallTools.py  -3003 - repos_get                          : get code:jumpscaleX:threefoldtech (['development_jumpscale'])
Mon 22 14:46:10 InstallTools.py  -3003 - repos_get                          : get code [git] (first time): jumpscaleX
The authenticity of host 'github.com (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is SHA256:nThbg6kXUpJWGl7E1IGOCspRomTxdCARLviKw6E5SY8.
Are you sure you want to continue connecting (yes/no)? yes
development_jumpscale
Mon 22 14:47:14 InstallTools.py  -1789 - checkoutbranch                     : Found branch: development_jumpscale
Mon 22 14:47:14 InstallTools.py  -3003 - repos_get                          : get code:digitalmeX:threefoldtech (['development_jumpscale'])
Mon 22 14:47:14 InstallTools.py  -3003 - repos_get                          : get code [git] (first time): digitalmeX
development_jumpscale
Mon 22 14:48:26 InstallTools.py  -1789 - checkoutbranch                     : Found branch: development_jumpscale
Mon 22 14:48:26 InstallTools.py  -2937 - install                            : link /sandbox/code/github/threefoldtech/jumpscaleX/Jumpscale to Jumpscale
Mon 22 14:48:26 InstallTools.py  -2937 - install                            : link /sandbox/code/github/threefoldtech/digitalmeX/DigitalMe to DigitalMe
Mon 22 14:48:28 JSBase.py        - 103 - systemfs                           : ***CLASS INIT 1: systemfs
- process         /sandbox/lib/jumpscale/DigitalMe/graphql.py
- process         /sandbox/lib/jumpscale/DigitalMe/__init__.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/__init__.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis/__init__.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis/GedisClientFactory.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis/GedisClient.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis/templates/GedisClientGenerated.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis/templates/_archive/ModelBase.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/multicast/MulticastFactory.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/multicast/__init__.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/multicast/MulticastClient.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis_backends/GunClient.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis_backends/GedisBackendClientFactory.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis_backends/__init__.py
- process         /sandbox/lib/jumpscale/DigitalMe/clients/gedis_backends/utils.py

possible reasons for that issue

wifi and ethernet both connected so ssh conflict happens
issue with GitHub cloning with ssh
ssh-key is not added to GitHub authorized key or conflicted

steps as mentioned above in addition remove nay old files in case old installation like ~/sandbox before installation

jsx version inside docker

3BOTDEVEL:3bot:jumpscaleX: git log -1
commit f1d139fb46c3ee7dd7933db781ae628ac7ce047b (grafted, HEAD -> development_jumpscale, origin/development_jumpscale)
Author: despiegk<EMAIL_ADDRESS>Date:   Tue Jul 23 18:11:26 2019 +0200

    specs for metadata


Layout issue when presenting on iPhone 6 Plus in landscape

If I try to present the popover form a button in landscape on iPhone 6 Plus I get a UIModalPresentationFormSheet instead of a popover and the content is not loaded.
The issue is reproducible with the demo app if you enable landscape orientation and fix the autolayout constrains of the two buttons.
Presenting in landscape on iPhone 6 and lower works.

Found a solution to this here:
http://stackoverflow.com/questions/30378249/uimodalpresentationpopover-for-iphone-6-plus-in-landscape-doesnt-display-popove/30481565#30481565
Sorry for not making pull request but the above solution requires iOS 8.3 so I'm not sure how you handle that with the current 8.0 requirement you have.

Yes, thank you for finding this bug and he solution as well. I have just merged a PR that handles this issue. You may get the latest version here or from CocoaPods in couple hours.
Thanks again! 👍


[HUDI-6874] Move configs for reading a file group to hudi-common module

Change Logs
This PR moves log merging and memory-related configs to hudi-common module so the common file group read can use them.  Configs in HoodieRealtimeConfig are removed if duplicate or merged to existing config if the functionality to control is the same.

Moved from HoodieCompactionConfig to HoodieReaderConfig: hoodie.compaction.lazy.block.read(with alternative key compaction.lazy.block.read.enabled), hoodie.compaction.reverse.log.read
HoodieMemoryConfig is moved from hudi-client-common to hudi-common module
HoodieRealtimeConfig is removed.

compaction.memory.fraction is merged with hoodie.memory.compaction.fraction.  Default value for MR is kept.
compaction.lazy.block.read.enabled is merged with hoodie.compaction.lazy.block.read.
hoodie.memory.dfs.buffer.max.size already exists in HoodieMemoryConfig.  Default value for MR is kept.
hoodie.memory.spillable.map.path already exists in HoodieMemoryConfig.
hoodie.optimized.log.blocks.scan.enable is moved to HoodieReaderConfig.
As a result, two config names are removed:

compaction.memory.fraction -> hoodie.memory.compaction.fraction
compaction.lazy.block.read.enabled -> hoodie.compaction.lazy.block.read

Impact
Code refactoring towards consistent logic in log merging and file group reading.
Risk level
none
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

CI is green.


TypeError: Function.prototype.toString requires that 'this' be a Function when trying to toMatchInlineSnapshot

🐛 Bug Report

To Reproduce
Steps to reproduce the behavior:
    expect([[], []]).toMatchInlineSnapshot()

Expected behavior
Does not crash with
    TypeError: Function.prototype.toString requires that 'this' be a Function
        at toString (<anonymous>)

      at toString (node_modules/apollo-env/node_modules/core-js/internals/redefine.js:38:95)
      at _isNativeFunction (node_modules/prettier/index.js:2008:28)
      at _wrapNativeSuper (node_modules/prettier/index.js:2015:28)
      at node_modules/prettier/index.js:40648:5
      at createCommonjsModule (node_modules/prettier/index.js:184:35)
      at Object.<anonymous> (node_modules/prettier/index.js:40620:27)

Link to repl or repo (highly encouraged)

envinfo
System:
OS: Linux 5.0 Ubuntu 19.04 (Disco Dingo)
CPU: (12) x64 Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
Binaries:
Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
Yarn: 1.15.2 - ~/.yarn/bin/yarn
npm: 6.9.0 - ~/.nvm/versions/node/v10.16.0/bin/npm
npmPackages:
jest: 24.9.0 => 24.9.0

ah okay this is only an issue when apollo-env is installed. I'll probably reopen with them then.


[Bug]: McAfee LiveSafe quarantines as false positive: JTI/suspect.196612!b9272b86fb70

The problem
McAfee LiveSafe quarantine false positive: JTI/suspect.196612!b9272b86fb70
What version of WAU has the issue?
Romanitho.WiGUI 1.7.1
What version of Windows are you using (ex. Windows 11 22H2)?
Windows 11 family 22H2
What version of winget are you using?
v1.4.11071
Log information
No response
Additional information
McAfee LiveSafe quarantines as false positive: JTI/suspect.196612!b9272b86fb70
might be worth submiting a consideration for allowlisting
https://www.mcafee.com/support/?articleId=TS102751&page=shell&shell=article-view

closing this. the issue should be in Winget-Install-GUI


adapta-gtk-theme: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change

Update to version <IP_ADDRESS>
Enable parallel building

Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!


Create stepup metadata entity with factory

The stepup metadata endpoint needed to be created. This change will make
sure that the entity is created.
Some minor adjustments where needed to allow optional fields.
Also the EB information is moved to a dedicated decorator as this is
won't be needed for all metadata endpoints.
https://www.pivotaltracker.com/story/show/169035362
I'll address adding decorator tests and the final implementation in a later PR
This is only a MVP to let the stepup metadata endpoint work.

Your concerns will be addressed in:
https://www.pivotaltracker.com/story/show/169257771


Fix ES URL for extension "NixPkgs Search"

Description
It appears the url used by the NixPkgs Search extension has changed, so I have updated it.
Issue: #3493
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Nice! I know the url has changed, but have no time to create a PR. Thanks for your patch

I haven't tested it yet, but the code itself LGTM


Private CA Resolver and Cloudflare DNS cannot co-exist

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Add Cloudflare DNS resolver after running private step CA HTTP resolver
If I comment out LEGO_CA_CERTIFICATES and LEGO_CA_SERVER_NAME env vars then Cloudflare DNS works, but step CA can no longer issue certs
What did you see instead?
time="2024-04-07T19:34:34Z" level=error msg="Unable to obtain ACME certificate for domains \"jellyfin.real-address.com,jf.real-address.com\": cannot get ACME client get directory at 'https://acme-v02.api.letsencrypt.org/directory': Get \"https://acme-v02.api.letsencrypt.org/directory\": tls: failed to verify certificate: x509: certificate is valid for acme-v02-1.api.letsencrypt.org, acme-v02-2.api.letsencrypt.org, acme-v02-3.api.letsencrypt.org, acme-v02-4.api.letsencrypt.org, acme-v02-5.api.letsencrypt.org, acme-v02.api.letsencrypt.org, incident.letsencrypt.org, not step-ca" ACME CA="https://acme-v02.api.letsencrypt.org/directory" rule="Host(`jellyfin.real-address.com`, `jf.real-address.com`)" routerName=jellyfin-real-address-https@docker providerName=cloudflare.acme

What version of Traefik are you using?
v2.11.0
What is your environment & configuration?
## Jellyfin Labels

x-nio_io_labels: &nio_io_labels
  ## HTTPS Service
  traefik.http.routers.jellyfin-https.service: jellyfin-https
  traefik.http.routers.jellyfin-https.rule: >-
    Host(`jellyfin-${DEVICE_IP:-ip_addr}.nip.io`, `jf-${DEVICE_IP:-ip_addr}.nip.io`, `jellyfin.nip.io`, `jf.nip.io`)
  traefik.http.routers.jellyfin-https.entrypoints: websecure
  traefik.http.routers.jellyfin-https.middlewares: jellyfin-mw
  traefik.http.routers.jellyfin-https.tls: true
  traefik.http.routers.jellyfin-https.tls.certresolver: step-ca
  traefik.http.services.jellyfin-https.loadbalancer.server.port: 8096
  # HTTP Service Allowed!
  traefik.http.routers.jellyfin-http.service: jellyfin-http
  traefik.http.routers.jellyfin-http.rule: >-
    Host(`jellyfin-${DEVICE_IP:-ip_addr}.nip.io`, `jf-${DEVICE_IP:-ip_addr}.nip.io`, `jellyfin.nip.io`, `jf.nip.io`)
  traefik.http.routers.jellyfin-http.entrypoints: web
  traefik.http.routers.jellyfin-http.middlewares: jellyfin-mw
  traefik.http.services.jellyfin-http.loadbalancer.server.port: 8096

## UDP

x-udp_labels: &udp_labels
  traefik.udp.routers.jellyfin-client-discovery.entrypoints: client_discovery
  traefik.udp.routers.jellyfin-client-discovery.service: jellyfin-client-discovery
  traefik.udp.services.jellyfin-client-discovery.loadbalancer.server.port: 7359

## Recommended by Jellyfin Docs

x-recommended_middleware_labels: &recommended_middleware_labels
  traefik.http.middlewares.jellyfin-mw.headers.customResponseHeaders.X-Robots-Tag: noindex,nofollow,nosnippet,noarchive,notranslate,noimageindex
  traefik.http.middlewares.jellyfin-mw.headers.frameDeny: 'true'
  traefik.http.middlewares.jellyfin-mw.headers.contentTypeNosniff: 'true'
  traefik.http.middlewares.jellyfin-mw.headers.browserXSSFilter: 'true'
  traefik.http.services.jellyfin.loadBalancer.passHostHeader: 'true'

x-real_address_labels: &real_address_labels
  # HTTPS Service
  traefik.http.routers.jellyfin-real-address-https.service: jellyfin-real-address-https
  traefik.http.routers.jellyfin-real-address-https.rule: Host(`jellyfin.real-address.com`, `jf.real-address.com`)
  traefik.http.routers.jellyfin-real-address-https.entrypoints: websecure
  traefik.http.routers.jellyfin-real-address-https.middlewares: jellyfin-mw
  traefik.http.routers.jellyfin-real-address-https.tls: true
  traefik.http.routers.jellyfin-real-address-https.tls.certresolver: cloudflare
  traefik.http.services.jellyfin-real-address-https.loadbalancer.server.port: 8096
  # HTTP to HTTPS Redirect
  traefik.http.routers.jellyfin-real-address-http.rule: Host(`jellyfin.real-address.com`, `jf.real-address.com`)
  traefik.http.routers.jellyfin-real-address-http.entrypoints: web
  traefik.http.routers.jellyfin-real-address-http.middlewares: https_redirect

Traefik ENV Vars
services:
  traefik:
    image: traefik:latest
    container_name: traefik
    environment:
      # LEGO_CA_CERTIFICATES: /shared_files/ca_certs/root_ca.crt
      # LEGO_CA_SERVER_NAME: step-ca
      SSL_CERT_FILE: /shared_files/ca_certs/root_ca.crt
      CLOUDFLARE_DNS_API_TOKEN: $CLOUDFLARE_DNS_API_TOKEN

Add more configuration information here.
# Static Conf
# Configure your CertificateResolver here...
# ---
certificatesResolvers:
  step-ca:
    acme:
      email<EMAIL_ADDRESS>      storage: /shared_files/acme/acme.json
      caServer: "https://<IP_ADDRESS>:55888/acme/acme/directory"
      # caServer: "https://step-ca:55888/acme/acme/directory"
      httpChallenge:
        entryPoint: web
  cloudflare:
    acme:
      email<EMAIL_ADDRESS>      storage: /shared_files/acme/cloudflare_dns_acme.json
      caServer: "https://acme-v02.api.letsencrypt.org/directory"
      dnsChallenge:
        provider: cloudflare
        delayBeforeCheck: 10

If applicable, please paste the log output in DEBUG level
No response

Hello,
can you try to set the env var LEGO_CA_SYSTEM_CERT_POOL to true?

Hi thanks for responding, I left it out of what I pasted but I tried that variable as well, no difference. I will try it again and let you know.

Another way is to add the Step-ca root CA to the system-wide trusted root list.

Doesn't that involve making a new image?


Startup -> GrandOpening and BasicsPack

some changes to startup, including a new name
new pack based around basic cards

Hey, I really appreciate you contributing to the project, however I have to be the bearer of bad news.
With the quantity of packs we have in the project at this point it's important that we take a closer look at the packs and sadly the basics pack falls a little short of the required design standards.
The pack boils down to a combination of the base classes with none of their interesting mechanics mixed with a lot of screens. I've seen an archetype like this in Hermit which had genuinely creative effects, so it can definitely be done.
Long story short, I won't be merging this pack in its current state, sorry. If you really want to play with the idea some more, I'd recommend taking a look at Hermit and a critical look at what the pack wants to achieve while also keeping in mind that you need some form of scaling.


Make eventyay-video autodeployment work

There are currently 3 issues we should fix to make auto-deployment work:

Current deployment doesnt work when using the image eventyay/eventyay-video:development, we should fix this to make sure these images work.
Watchtower redeployments seem to fail with "npm not found" we need to investigate how watchtower does this redeployment and see how to fix this.
We changed ports in https://github.com/fossasia/eventyay-video/commit/8ca1a66009a7e788dad6ecd4198d3f0087b129b3, we should revert this change as it breaks the Dockerfile-dev.yaml builds and current docker-compose.yaml in root that are useful for local development. Then we should update the server nginx so the port matches in the reverted config: 8375.

Hi @marcoag,
on the third point, about the port.
In local, webapp will open a socket connection with backend and we do not use SSL for websocket connection.
But in dev and prod environment, SSL is required, and we setup SSL at nginx, so if we open web socket on port 8375, it will directly to backend, not through nginx and we got an issue here. the reason why we change the port connection that make webapp connect with backend via nginx to enable SSL.
For easy to understand:
webapp - open websocket to port 8375 -> backend (no SSL and will cause an error because webapp is running on SSL config)
webapp - open websocket to port 8443 on nginx -> nginx reverse proxy to 8375 to the backend, it working fine as currently.
So I suggest we should note somewhere in the document tell developer to change the port connection same with port opened by backend
cc @mariobehling

Hi @marcoag,
https://github.com/fossasia/eventyay-video/pull/136 to fix point 2 error when watchtower redeploy webapp.
for the first issue, could you please give me more detail how to preproduce or what is the log error of it.
cc @mariobehling

Current deployment doesnt work when using the image eventyay/eventyay-video:development, we should fix this to make sure these images work.

In the standard config when we merge a pull request into the development branch a new docker image gets built and deployed to https://video-dev.eventyay.com. So, instead of triggering the build yourself it gets triggered automatically. It can be tested by merging a PR into development.


Add yamlify or whatever function to marshal a content to YAML

I have YAML files inside /data directory, in my archetypes/services.yaml I want to do the following to automatically generate content files:
---
title: "{{ replace .Name "-" " " | title }}"
date: {{ .Date }}
url: /services/{{ .Name }}.html
---

{{ index $.Site.Data.oniontree.unsorted .Name |  yamlify }}

From the shell:
$ hugo new services/bbc.yaml

Unfortunatelly a yaml marshaller isn't available.

We have transform.Remarshal used for similar things on the docs site

---
title: "{{ replace .Name "-" " " | title }}"
date: {{ .Date }}
url: /services/{{ .Name }}.html
---

{{ index $.Site.Data.oniontree.unsorted .Name |  transform.Remarshal "yaml" }}

Did the trick. Why isn't it documented? I see you have already answered same question before: https://discourse.gohugo.io/t/display-yaml-as-text/19830.


[MacOS] supabase start - mkdir /Users: operation not permitted

Describe the bug
When trying to run supabase start or npx supabase start or sudo supabase start on a fresh project I keep getting the same error:
Error response from daemon: make cli opts(): making volume mountpoint for volume /Users/me/workspace/project/supabase/functions: mkdir /Users: operation not permitted

Running with --debug flag yields no useful infos.
Running supabase start -x edge-runtime works fine
To Reproduce
Steps to reproduce the behavior:
mkdir test
cd test
npm init
npm i -D supabase
npx supabase init
npx supabase start

Expected behavior
It should work.
Desktop (please complete the following information):

OS: MacOS 13.4.1 (22F82)
Version of CLI: 1.88.0, 1.92.2
Docker version 23.0.5, build bc4487a

Could you try creating this directory yourself before supabase start?
mkdir -p /Users/me/workspace/project/supabase/functions

There might be some permission settings for your /Users directory that prevents the binary from making changes.

Hi! The directory already exists.

Does it throw any error if you volume mount manually via docker cli?
docker run --rm -v /Users/me/workspace/project/supabase/functions:/tmp -it supabase/edge-runtime:v1.22.4

Just wanted to move from my windows machine to a new mac and now also stuck on this error 👍  @andreafdaf did you manage to resolve it somehow?

@NiklasPor I solved starting over with my env but since I did it in a hurry I lost some work so please do it with ease, doing backups and double checking every step you take.
From what I understood it was some kind of conflict between Podman and Docker. So what I did was purging everything from my Podman and Docker installs, starting over with only Docker installed and the issue went away.
If unlike in my case you don't have Podman installed alongside Docker then I don't know what to advise you.
Hope this helps you out!

Thanks for the quick answer! interesting, is just started with a clean installed MacBook — we'll see 👀


git.api.getRemoteSources argument clarification

Testing #111518
I found 'providerLabel' and 'urlLabel' both confusing concepts for this command, I didn't understand what was going to be displayed in the UI. 'providerLabel' was especially strange to me, since as the caller of the command, I don't have much insight into what providers are available, and I didn't know why the provider itself wouldn't specify a label. I think I just need an example of when the caller would need to be able to tweak these labels

This command isn't really meant to be used outside, but as tight integration with some other features like devcontainers. No big need to document it fully, but thanks for the feedback!


👷[ci] Try and use an inline commenter for the flake8 checks

https://github.com/marketplace/actions/flake8-action is not up to date with the node version, and the fork allowing to use the path parameter does not seem to be working at the moment. Abandoning this for now, as the command line flake8, even if not as nicely integrated with GitHub, does work properly.


Added CocoaPods support

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master      #2   +/-   ##
======================================
  Coverage    92.3%   92.3%
======================================
  Files           1       1
  Lines          52      52
======================================
  Hits           48      48
  Misses          4       4

Flag
Coverage Δ

#SGCircuitBreaker
92.3% <ø> (ø)
: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 e5d72ed...114297a. Read the comment docs.


Added .git folder to ignored folders

Any .git folder should not be searched by Ghost and in this case it was checking the .git folder as a possible theme folder. This PR simply adds .git folders to the ignore list when reading a directory.
https://github.com/TryGhost/Ghost/issues/6140

Hi @StevenMcD thanks for the fix :+1:  Could I trouble you to please update the commit message using git commit --amend so that it matches this format, using that ensures that the PR and issue are properly linked. Thanks :grin:

No problem @ErisDS, my apologies for not spotting that the first time around. On it now

I removed my original branch, I'm going to close this PR and resubmit it. This time I won't delete my branch so quickly :smile:


Fix a bug in getitem

In the method OrderedSet.getitem, the index is checked
to be in range. In the previous implementation, an index error
was raised if the index was valid instead.

Thank you very much; quite an embarrassing bug! I wonder whether there's a name for these bugs, as in "off-by-one error;" what about "swap error?"


Added sky shader mode to visual shaders

Implementation of https://github.com/godotengine/godot/pull/37179 for visual shaders.

TY, fixed


Binding fails with target frameworks netcoreapp 2.1 and 2.2

This package works fine when I target netcoreapp2.0. But when I target anything above that I get
Microsoft.Azure.WebJobs.Host: Error indexing method 'UploadImage'. Microsoft.Azure.WebJobs.Host: Cannot bind parameter 'cosmosDbClient' to type ICosmosDbClient. Make sure the parameter Type is supported by the binding. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).

I have exactly the same issue

check if the package you deploy have non-empty extensions.json file. If not, update functions sdk to 1.0.26.

I tried this but still no extensions.json, and it still failed like before. See this gist for some example code
Do I need to create my own extensions.json that references the DI extension?

no, the extensions.json should be configured automatically by sdk. please share also the csproj file.

@miqm Thanks for the example I will look at it, I updated the gist with my csproj

Looks like you're using 1.0.24 - you should be on sdk 1.0.26 - have you tried upgrading? I know it doesn't work under 1.0.26

I duplicated your setup in the example, and the startup code runs fine but the binding to the [Inject] attribute still failes with 'Microsoft.Extensions.DependencyInjection.Abstractions: Unable to resolve service for type 'ObjectDetection.Services.ICosmosDbClient' while attempting to activate 'ObjectDetection.UploadImage'.' Hmmm, any idea here?

It's because you're trying to resolve ICosmosDbClient not from your namespace but from objectDetection.Services. Use CrackDetection.ICosmosDbClient in Inject and you should be ok.

Share your code if possible.

Share your code if possible.

@miqm Here is an upated Gist with the relevant source code.
Thanks

Not sure if this is will work, but try separate code with extensions into a library that function app will depend upon.


meta payload isn't being recognized on POSTs

Request
{
	"data": {
		"type": "user",
		"attributes": {
			"username"<EMAIL_ADDRESS>			"password": "f00b4r"
		},
		"relationships": {
			"environment": {
				"data": {
					"id": "641ce53d-c547-4d30-8d12-2d7fb2a25ab0"
				}
			}
		}
	},
	"meta": {
		"token": "c60e6f72-df7e-4197-ae84-fbf780bcdd8c"
	}
}

Parsed operation
{ op: 'add',
  params: {},
  ref: { id: undefined, type: 'user', relationship: undefined },
  data:
   { type: 'user',
     attributes:
      { username<EMAIL_ADDRESS>        password: 'f00b4r',
        environment_id: '641ce53d-c547-4d30-8d12-2d7fb2a25ab0' },
     relationships: { environment: [Object] } } }

this is pretty weird, have you been able to replicate this in the dummy test suite?

Didn't investigate fully, just stumbled upon it when trying to perform that kind of request on an API I'm working on.

'kay, sometime this week I'll take a look


Top Nav and Footer z-index issue on full overlay LockScreen

Top Nav and Footer should be hidden when going in full overlay mode.

Proper layout mode is now handled by the layout manager.


Windows thread support: Part 1

This PR adds support for threads on Windows.

Whoops, I messed up the rebase at first.

If it helps, here are some public resources on HANDLEs:

Why are HANDLE return values so inconsistent?
Why are kernel HANDLEs always a multiple of four?
Interprocess Communication Between 32-bit and 64-bit Applications
Pushing the Limits of Windows: Handles

I've rewritten the handle implementation based on what is actually guaranteed by the windows docs

If it helps, here are some public resources on HANDLEs:

To the extent that these are relevant for this new code in Miri, please add links to them in comments in the code. :)

:umbrella: The latest upstream changes (presumably #2236) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks for the PR! Looks like Windows is starting to catch up with our Linux support. :)
I am working on a review, but in the future it'd be helpful to break PRs into smaller chunks. For example, I think this could have started with just thread spawning and joining, without all the concurrency primitives. Smaller PRs are a lot easier to review so any effort you can put into splitting a change into chunks is much appreciated.

:umbrella: The latest upstream changes (presumably #2332) made this pull request unmergeable. Please resolve the merge conflicts.

Looking at your commits, I see "basic threading" and "condvar, parking and yielding" are separate commits. If you could split those into separate PRs, that would help a lot with reviewing.
Basically the first PR just needs to get concurrency/{simple,thread_local,tls*}.rs to work. That should be possible without new synchronization code I hope?

I separated out the sync changes and squashed these changes.

:umbrella: The latest upstream changes (presumably #2355) made this pull request unmergeable. Please resolve the merge conflicts.

(please use @rustbot ready once the PR is ready for review again)

:umbrella: The latest upstream changes (presumably #2379) made this pull request unmergeable. Please resolve the merge conflicts.

@rustbot ready

Please take care of the clippy warnings. ./miri clippy can run clippy locally.
And those CI failures are interesting, is drop order different between Windows and Linux?

@rustbot author

The TLS destructor order is different between platforms because std on Windows handles destructors itself, while Miri determines the order on other platforms. I don't think this is actually a problem, I just don't know how to make the test work.
@rustbot ready

Is the order also different across platforms on real rustc?
If yes, probably the best thing to do is to copy-paste the test, add ignore-windows in one file and only-windows in the other, and have the two reference files differ in their order, and have comments explaining the situation. The important part is that for both Windows and non-Windows, Miri uses the same order as real rustc.

I checked and it looks like miri is running the destructors in the wrong order. I'll make a separate PR for that.

Is the order even guaranteed?

I actually have no idea. :/

:umbrella: The latest upstream changes (presumably #2480) made this pull request unmergeable. Please resolve the merge conflicts.

I'd say for now just don't touch the Unix code, instead have separate tests for Windows and non-WIndows.

@rustbot author

@rustbot ready

Just two more comment nits, then we are good to go. :) Since review is done, please squash the commits together a bit.

@rustbot author

:umbrella: The latest upstream changes (presumably #2492) made this pull request unmergeable. Please resolve the merge conflicts.

@rustbot ready

Awesome, thanks a ton for all your work and patience. :)
@bors r+

:hourglass: Testing commit c466ac0b3ec200ac53b1ca8196c47261b692bda1 with merge 46da7485023e2b09b76039071d4969334aabbe91...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 46da7485023e2b09b76039071d4969334aabbe91 to master...


redefinition of enumerator 'SDL_JOYSTICK_POWER_UNKNOWN' errors with top of tree

(Many thanks for the go-sdl2 bindings--they're invaluable!)
With top of tree (3624de6c831d) I'm seeing the errors below. From a little investigation, this seems to have been introduced in 28145e5f; if I roll back to 253cca45719b63, it works fine. I think the issue is the #if !defined(SDL_JoystickPowerLevel) check there; I don't see anything that would actually #define that in the SDL2 code. (Not sure if the check should be #if !defined(SDL_joystick_h_), or if there's something more nuanced there.)
# github.com/veandco/go-sdl2/sdl
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:169:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_UNKNOWN'
    SDL_JOYSTICK_POWER_UNKNOWN = -1,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:101:5: note: previous definition is here
    SDL_JOYSTICK_POWER_UNKNOWN = -1,
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:170:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_EMPTY'
    SDL_JOYSTICK_POWER_EMPTY,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:102:5: note: previous definition is here
    SDL_JOYSTICK_POWER_EMPTY,   /* <= 5% */
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:171:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_LOW'
    SDL_JOYSTICK_POWER_LOW,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:103:5: note: previous definition is here
    SDL_JOYSTICK_POWER_LOW,     /* <= 20% */
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:172:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_MEDIUM'
    SDL_JOYSTICK_POWER_MEDIUM,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:104:5: note: previous definition is here
    SDL_JOYSTICK_POWER_MEDIUM,  /* <= 70% */
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:173:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_FULL'
    SDL_JOYSTICK_POWER_FULL,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:105:5: note: previous definition is here
    SDL_JOYSTICK_POWER_FULL,    /* <= 100% */
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:174:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_WIRED'
    SDL_JOYSTICK_POWER_WIRED,
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:106:5: note: previous definition is here
    SDL_JOYSTICK_POWER_WIRED,
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:175:5: error: redefinition of enumerator 'SDL_JOYSTICK_POWER_MAX'
    SDL_JOYSTICK_POWER_MAX
    ^
/opt/homebrew/include/SDL2/SDL_joystick.h:107:5: note: previous definition is here
    SDL_JOYSTICK_POWER_MAX
    ^
/Users/mmp/go/pkg/mod/github.com/veandco/go-sdl2@v0.5.0-alpha.3.0.20220906123024-3624de6c831d/sdl/events.go:176:3: error: typedef redefinition with different types ('enum SDL_JoystickPowerLevel' vs 'enum SDL_JoystickPowerLevel')
} SDL_JoystickPowerLevel;
  ^
/opt/homebrew/include/SDL2/SDL_joystick.h:108:3: note: previous definition is here
} SDL_JoystickPowerLevel;
  ^
8 errors generated.

Hi @mmp, thanks for the compliment (it means a lot!) and reporting of this issue. I have implemented a fix for this in the master branch.

Thanks for the quick fix--confirmed that it works!


SQUID 2.50 Tasks not importing correctly

This issue relates to the "four horsemen" (i.e. "Special U-Th-Pb") expressions for my test Task "Zrn 10pk Perm2 ConcU" (SquidTask_Zrn 10pk Perm2 ConcU.GA.zip) The first screenshot shows its (correct) expression-set, as viewed in SQUID 2.50's Task Editor:

The second screenshot shows this Task imported into Squid3 (Side-note 1: it'd be really nice to be able to Save these imported Tasks; I lose a lot of time because I have to use a lot of different SQUID 2.50 tasks to do the arithmetic testing, and every time I switch Tasks I need to re-import from scratch).
This shot comes from "Task-->Manage Current Task" (Side-note 2: I suggest suppressing the 'breadcrumbs' across the top of all these screens, at least temporarily. At present they are inaccurate pretty much everywhere!)

It is clear that expression for Uncor_208Pb232Th_CalibConst (yellow) is not being addressed correctly. I am working my way through all the Perms at the moment, and might be able to shed light on the overall pattern once I have done a few more... or it might be obvious to @bowring what's going wrong...
For what I am trying to do, I think I can work around this manually, at least for the moment... The version of Squid3 is 1.1.3.

Hi @sbodorkos ,  a little work around so to deal with the fact that you can't save an imported task is to save the squid project and then when you want to use that task again open the .project file and replace the data.

@NicoleRayner thanks, that is a really good idea. I guess I have always been a bit unclear on how to use Projects (I make one at the beginning of each Squid3 session because I have to, but I usually don't bother Saving them). Now it's clear that at least for as long as I am relying on SQUID 2.50 Tasks, I should be creating and maintaining one Project per Task... cheers


migrate on startup

Are these serial?

It doesn't specifically say in the docs, but looking at the source, it runs synchronously during init, so yes, this will finish before the rest of the application starts.


⚠ Bofrost.nl has degraded performance

In ea01c47, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 947 ms

Resolved: Bofrost.nl performance has improved in c393254 after 6 minutes.


Links not visible on Moto X Play running Android M

I recently saw this on a friends 6p, I'll try and get a fix in this weekend
On Sat, Jan 23, 2016, 12:23 PM Harshil Lodhi<EMAIL_ADDRESS>wrote:

[image: screenshot_20160123-112751]
https://cloud.githubusercontent.com/assets/1177554/12531904/540f1142-c22c-11e5-8626-89ed5f9f5aef.png
[image: capture]
https://cloud.githubusercontent.com/assets/1177554/12531906/780d77b4-c22c-11e5-8b4f-29776b59296b.PNG
—
Reply to this email directly or view it on GitHub
https://github.com/dinosaurwithakatana/hacker-news-android/issues/79.

Thanks @burntcookie90. I was working on a fix but couldn't test it because of slow internet at home(it needs 6.0 sdk).
Are there any important features enhancements which I can contribute to. I am usually free on weekends

The master / development branches are currently frozen (minus major but fixes). The application is currently undergoing a full rewrite on the branch feature/kotlin. Once that branch becomes stable I'll let you know. Thanks!


Add<EMAIL_ADDRESS>support

Android build fail on<EMAIL_ADDRESS>npx react-native init RN71 --version 0.71.0-rc.5
cd RN71
yarn add react-native-mmkv
yarn android

FAILURE: Build failed with an exception.

* Where:
Build file '/home/i/open-source/RN71/node_modules/react-native-mmkv/android/build.gradle' line: 173

* What went wrong:
A problem occurred evaluating project ':react-native-mmkv'.
> Expected directory '/home/i/open-source/RN71/node_modules/react-native/android' to contain exactly one file, however, it contains no files.

For authors of react-native-mmkv
Because of this: https://github.com/facebook/react-native/tree/main/android

same issue

any progress

you can use an early version from git
yarn add mrousavy/react-native-mmkv#feat/migrate-rn-71

Require to enable newArchEnabled=true in android/gradle.properties file

Does not work for me. I can't download the early version.

2.6.1 should fix this.

Fixed by #480 (released in 2.6.1.) :)

Thank you so much to everyone working on this repo ❤

🖤


Cleanup any container perms after #278 bug fix

Closes #279.
Not planning on merging until tomorrow.

Tried this and it worked as expected.


⚠ Bofrost.be has degraded performance

In 07e5871, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1837 ms

Resolved: Bofrost.be performance has improved in c3807e1 after 46 minutes.


Weird skyplot output

Not sure how this happened:

Ran this on 12 May 2023 at around 9:45pm PDT with the following command:
skyplot --tle ~/Desktop/tles/ISS.tle --time-start "13 May 04:00"

And the following TLE
ISS
1 25544U 98067A   23132.42640824  .00014922  00000-0  26744-3 0  9995
2 25544  51.6400 145.7781 0006355 339.7698 124.4230 15.50147384396243

Hopefully the rest can be reproduced from cached logs and alignment model.

OHHH I didn't even notice that this is showing the mount's actual path, since this was from a recent pass. I didn't even know that feature still worked. I need to adjust the color of that line so it has better contrast with the background.


"SyntaxError: unexpected character after line continuation character" pops up

Environment data
Build of Visual Studio Code - Insiders: 1.69.0-insider(user setup)
Build of python-Pre-Release: v2022.9.11681004
Build of jupyter-Pre-Release: v2022.6.1001741036
Build of pylance-Pre-Release: v2022.6.21
Build of jupyter Notebook Renderers: v1.0.8
Build of jupyter Keymap: v1.0.0
Steps to reproduce:
1.Make sure Anaconda installed in local
2.Open the file src/test/datascience/manualTestFiles/manualTestFile.py in VS Code
3.At the top of the file it will list the things that you need installed in your Python environment
4. On the first cell click Run Below
5.Interactive Window should open, show connection information, and execute cells
6.Check the "# %% Latex"
Expected behaviour
Execute cells sucessfully without errors

Actual behaviour

Sample code
# %% Latex

%%latex

\begin{align}

\nabla \cdot \vec{\mathbf{E}} & = 4 \pi \rho \\

\nabla \times \vec{\mathbf{E}}\, +\, \frac1c\, \frac{\partial\vec{\mathbf{B}}}{\partial t} & = \vec{\mathbf{0}} \\

\nabla \cdot \vec{\mathbf{B}} & = 0

\end{align}

I wonder when this stopped working. Did we perhaps escape this before?

/verified


Move custom targets to special level

Addresses part of https://github.com/adoptium/ci-jenkins-pipelines/issues/857

Quick test here:
https://ci.adoptium.net/job/Test_openjdk17_hs_special.openjdk_ppc64le_linux/5


PDF over 5080 mm (200 inches) not supported

PDF sizes over 5080 mm (200 inches) seems to be unsupported by Adobe Reader and will result in a blank page PDF (still works in the browser PDF plugin somehow).
I think the cause is the MediaBox exeeding 14400 which is not supported
/MediaBox [0 0 14400.00 2834.64]
Is it possible to limit the MediaBox output to 14400 somehow? Perhaps limits on wPt and hPt on line 274 jspdf.js might do the trick?
Perhaps it also can be combined with /UserUnit X.X as sugested in the forum thread below?
https://forums.adobe.com/thread/1003116?tstart=0
Example code that breaks in Adobe Reader (not browser version)
var doc = new jsPDF("landscape",'mm',[5081,200]);
doc.line(0,0,100,100);
doc.save("Test.pdf");

implemented with #2146


Forbid imports / calls, per file

Expected Behavior

I have one file that is should not use some types to avoid NoClassDefFoundError. THe import are allowed in other classes.
Could it make sense to have a file annotation that tells imports are forbidden (or even allowed in this file only)
E.g. in pseudo code
@file:ForbiddenImport(
    forbidden = ["git4idea.*"]
)

Current Behavior

Not supported today
Context

I have one file that is should not use some types to avoid NoClassDefFoundError. THe import are allowed in other classes.

Sorry I'm not sure I understand what you're saying here. Could you elaborate further?
There is already a rule called ForbiddenImport which does exactly that.

I'm aware of ForbiddenImport and the others, however this rule is global
``
ForbiddenImport:
active: true
imports: []
forbiddenPatterns: '^it.unimi.dsi.fastutil.*'

I would like to apply such a rule in a single class / file only.

This will be possible in detekt 2.0 where you can instantiate a rule multiple times.

Closing for the reason mentioned by Nico


Add bzlmod support

Enable bzlmod support and upload release to https://github.com/bazelbuild/bazel-central-registry

any plan?

Howdy! It's something that I hope to do eventually but don't really have any experience with bzlmod and so if someone out there really wants this then I'd be happy to review a pull request and learn!


Sync MiQ Alerts definitions with Middleware provid

This PR is a continuation of work on #9431 and #9441.
Context:
Middleware provider supports LiveMetrics instead of pull metrics into MiQ.
In a high level goal we want to add support for alerting.
Main strategy is to define Alerts and Alerts Profiles in MiQ, so from users perspective there is no change comparing as others providers.
In the case of the Middleware provider we are going to let the provider to know about these alerts definitions, so, in this way, the provider will be able to "detect" these alerting conditions and send back an event that we can link with the MiQ Alert.
This PR is adding the synchronization call on the provider once a new alert or a new alert profile is created or updated.

miq-bot warnings are discussed on #9441.
Basically rules changed after that class was completed so, new additions might break the code layout.

I would need these three PRs #9431, #9441 and #9460 to progress.
These are relatively easy to review.
Next tasks are on the MiddlewareProvider and Events subsystem but I won't send additional PRs to not create excessive dependencies between PRs until these ones are consolidated.

@lucasponce since #9441 is merged, please rebase this PR. thanks!

@miq-bot assign @Jiri-Kremser
@Jiri-Kremser please review. thanks!

LGTM

@miq-bot assign @chessbyte

👍 This looks good to me. There's still an open question of ManageIQ::Providers::Hawkular::MiddlewareManager vs. ManageIQ::Providers::MiddlewareManager, however.

Marking this as WIP for now @lucasponce.  Please update this PR when you return and we can get this moved forward.  Thanks!

@jrafanie @durandom I have reviewed your comments.
This feature is specific for Hawkular at the moment, so if no blocker I would move forward to have an end-to-end with the following PR that depends on it.
The suggestion to make a generic MiddlewareProvider call are goods and I think that we can do it in following tasks as there are more areas involved (not only this feature of alerting).
If you are agree, I can remove the wip state of this PR.

@gtanzillo @jrafanie @durandom please could you have a final review please ?
This PR is a serie of several ones to add end-to-end alerting support for middleware provider.
I would suggest to go forward and work on the generic MiddlewareProvider in a separate PR.
@miq-bot remove_label wip
/cc @abonas @pilhuhn @Jiri-Kremser

@miq-bot remove_label wip

@lucasponce Are there tests you can add for this feature?

@jrafanie not on this PR yet, once the following related PR are in sync, the idea is to add additional VCR ones.
As this is not a feature that can be tested in a unit-test manner, it requires additional integration of several pieces (alerting defined in MiQ using mock objects, sync with a provider with VCR cassettes, and some async eventing work as well).

@lucasponce Ok, I was more concerned about testing the controller code because I can't follow what it's doing before your changes to review it.  VCR tests won't help with the controller changes.

But in this case, the current controllers spec will check that there are not regressions.
At the end, the logic of this PR is that for specific alerting creation, the controller will place an async call via a queue to sync that info with the provider.
So, the ideal test is once we have the sync part complete, to create a test starting from the controller that check that is synced correctly.

@martinpovolny or @dclarizio Please merge if you're ok with the controller changes.  Thanks.

The new sync methods seem to gate everything but the middleware alerts and profiles, so I think the controller changes look good.


Registration of a new user fails in the method navigator.credentials.create

The user registration does currently not work due to that Streamlit wraps custom components in an iframe, which causes the call WebAuthn passkey creation process to fail. The navigator.credentials.create method is not allowed by default to be called within an iframe.
When trying to create a new user the following error is returned:
DOMException: The 'publickey-credentials-create' feature is not enabled in this document. Permissions Policy may be used to     delegate Web Authentication capabilities to cross-origin child frames.
This could possibly be resolved by setting the response header Permissions-Policy: publickey-credentials-create=* and setting the attribute allow on the iframe e.g.
<iframe src="https://example.com" allow="publickey-credentials-create *"></iframe>

The issue could be resolved by packaging up the JavaScript code with npm run build and running the JavaScript from the built source rather than the local development server.

The issue is resolved.


[FP]: False positive for CVE-2023-36415 in azure-identity

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:microsoft:azure_sdk_for_java:1.12.2:*:*:*:*:*:*:*
CVE
CVE-2023-36415
ODC Integration
None
ODC Version
10.0.4
Description
I see that vuln is actual for azure-identity (java) up to 1.10.2, but my version is 1.12.2 and CVE is still in the report (also tried azure-identity 1.13.3, the same thing)

any updates?

any updates?

see https://github.com/jeremylong/DependencyCheck/issues/7000#issuecomment-2417434590


can read/write folder settings in single folder setup

testing #31448

don't open a workspace but just a folder (old world)
have extension like below, it doesn't fail tho the test plan suggests something different
also, inspect doesn't set the workspaceFolder-value to undefined

'use strict';
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {

    let disposable = vscode.commands.registerCommand('extension.sayHello', async () => {

        if (!vscode.window.activeTextEditor) {
            return;
        }

        let value = await vscode.window.showInputBox({
            prompt: 'Insert font size',
            validateInput(input) {
                if (!/\d+/.test(input)) {
                    return "Input a number";
                }
            }
        });

        if (!value) {
            return;
        }

        let config = vscode.workspace.getConfiguration(undefined, vscode.window.activeTextEditor.document.uri);
        try {
            console.log(`inspect("editor.fontSize")`, config.inspect('editor.fontSize'))
            await config.update('editor.fontSize', Number(value), vscode.ConfigurationTarget.WorkspaceFolder);
        } catch (e) {
            console.error(e);
        }
    });
}

I have files this because it disagrees with the test plan. However, I don't necessarily dislike this behaviour...

To be strict, update to folder target in single folder workspace should throw an exception. Writing into FOLDER target is always not common and special. Hence, if a user intends to write into folder, this should go into folder target not to workspace target. Being setting file is same is an implementation detail.

I always set all values even though they are undefined. But Console is smartly skipping properties with undefined values.
defaultValue: config.default,
globalValue: config.user,
workspaceValue: config.workspace,
workspaceFolderValue: config.folder


Change C-arm to move source and detector simultaneously

Problem with the existing geometry implementation: The source and detector currently move independent of each other (as is done for a standard X-ray), but in a C-arm, the source and detector have to move together. This example  shows that the detector moves independent of the source:

Need to reimplement such that the rotation matrices are applied both to the source and detector. Also, use this opportunity to make the geometry match DeepDRR:

Note that DeepDRR reverses "the orbital movement and angulation" from the convention above (from the documetation):
Args:
    world_from_device: (Optional[geo.FrameTransform], optional): Transform that defines the device coordinate space in world coordinates. None is the identity transform. Defaults to None.
    isocenter (geo.Point3D): the initial isocenter of in the device frame. This is the point about which rotations are performed.
    isocenter_distance (float): the distance from the X-ray source to the isocenter of the CAarm. (The center of rotation).
    alpha (float): initial LAO/RAO angle of the C-Arm. alpha > 0 is in the RAO direction. This is the angle along arm of the Carm.
    beta (float): initial CRA/CAU angulation of the C-arm. beta > 0 is in the CAU direction.
    degrees (bool, optional): Whether given angles are in degrees. Defaults to False.
    camera_intrinsics: (Optional[Union[geo.CameraIntrinsicTransform, dict]], optional): either a CameraIntrinsicTransform instance or kwargs for CameraIntrinsicTransform.from_sizes

It would be nice to standardise between the two libraries. I've done a bit of wrangling to map between the two (I'm sure you have a much better understanding though!), but it case it's useful:

this issue was my attempt to reconcile the SDR variables - https://github.com/arcadelab/deepdrr/issues/80
if I change the order in which the rotation matrices are applied in DiffDRR AND use the mapping below, I get consistent behaviour (or put another can reproduce the pose of one library from the other):

Mapping to DeepDRR conventions
carm = MobileCArm(isocenter=volume.center_in_world, alpha=theta, beta=np.pi / 2 - phi, gamma=-gamma

Change Matrix Application Order in DiffDRR
diff --git a/DiffDRR/diffdrr/camera.py b/DiffDRR/diffdrr/camera.py
index dadcce9..6604b99 100644
--- a/DiffDRR/diffdrr/camera.py
+++ b/DiffDRR/diffdrr/camera.py
@@ -92,7 +92,7 @@ def Rxyz(rotations):
     R_z = Rz(theta, batch_size, device)
     R_y = Ry(phi, batch_size, device)
     R_x = Rx(gamma, batch_size, device)
-    return torch.einsum("bij,bjk,bkl->bil", R_z, R_y, R_x)
+    return torch.einsum("bij,bjk,bkl->bil", R_y, R_z, R_x)

 def Rx(gamma, batch_size, device):

If I make those changes / mappings, I can get consistent results (up to expected minor differences due to the way each renderer works):

(to be clear I'm not saying what I've done is necessarily the correct way to map between them, but it's the only way I've managed to get consistent results so far).

@hemmer Thank you very much for sharing this! I will retry your code with real CT. If all works well, I'll PR the changes in!

@hemmer Works perfectly! thanks very much for all the heavy lifting on your end. I would never have thought to reverse the order of matrix multiplications. I'll include a notebook showing how to do the conversion from DiffDRR -> DeepDR in the documentation.

Hello,
I did not understand well the mapping. In deepDRR I put as alpha the primary angle and as beta the secondary angle and gamma is 0. If I want to reproduce the same results with diffDRR in the following code gamma = alpha and phi = beta ?
@@ -92,7 +92,7 @@ def Rxyz(rotations):
R_z = Rz(theta, batch_size, device)
R_y = Ry(phi, batch_size, device)
R_x = Rx(gamma, batch_size, device)

return torch.einsum("bij,bjk,bkl->bil", R_z, R_y, R_x)

return torch.einsum("bij,bjk,bkl->bil", R_y, R_z, R_x)

Hi @fedeface98 , this part of the documentation should cover how to do the conversions: https://vivekg.dev/DiffDRR/tutorials/metamorphasis.html
Please ping me if any part doesn't make sense or doesn't work.

in this part for you alpha and beta are the primary and secondary angles?
I can't understand why you need to do np.pi / 2 - beta
def test_phantom_deepdrr(alpha, beta, gamma):
with Projector(
volume=patient,
carm=carm,
) as projector:
carm.move_to(
alpha=np.rad2deg(alpha),
beta=np.rad2deg(np.pi / 2 - beta),
gamma=np.rad2deg(-gamma),
degrees=True,
)
img = (
projector()
)  # The first run doesn't use updated parameters, for some reason?
img = projector()[:, ::-1].copy()
return img

yes alpha and beta are the primary and secondary angles, respectively
np.pi / 2  - beta is a correction factor used because DiffDRR initializes the camera on the x-axis while DeepDRR initializes it on the z-axis

Thank you, it is clear to me!
But then in the following function you inverse beta and alpha, why?
def diffdrr_to_deepdrr(euler_angles):
alpha, beta, gamma = euler_angles.unbind(-1)
return torch.stack([beta, alpha, gamma], dim=1)

i'm sorry, i probably misspoke --- i should have said beta is the primary angle and alpha is the secondary angle
the inversions aren't special in any way. they're simply helper functions to convert from diffdrr's convention to deepdrrr's convention
note that when using euler angles to parameterize a rotation, there are 12 equivalent conventions you can use (see this table). diffdrr uses one and deepdrr uses another, so the functions just perform some conversions

Thanks a lot!
I have a final question about the isocenter. In deepdrr the isocenter is put equal to patient.center_in_world that is different from the isocenter set up in diffdrr calculated as (torch.tensor(volume.shape) - 1) * torch.tensor(spacing) / 2).
I can't understand why.


bullet list and pre code

When doing this the code part will not be formatted with

.
* A test
* Another test

    Code Block

But if it like this (with double newline) it will work. I guess this is a bug?
* A test
* Another test

    Code Block

There are no bugs :) . See loose/tight lists http://spec.commonmark.org/0.24/#loose i think it worh to discuss spec-related questions at commonmark forum.
PS. use demo to provide permalinks for samples.

Thanks. I will remember your advice :)


Program stuck after All distributed processes registered, setting NCCL_P2P_DISABLE=1 freezes whole server

Bug description
Using the driver version
NVIDIA-SMI 545.29.02              Driver Version: 545.29.02    CUDA Version: 12.3
configure the environment using following commands
conda create -n test -y
conda activate test
conda install python=3.10 -y
conda install pytorch==2.1.0 torchvision==0.16.0 torchaudio==2.1.0 pytorch-cuda=12.1 -c pytorch -c nvidia
conda install -c fvcore -c iopath -c conda-forge fvcore iopath -y
conda install pytorch3d -c pytorch3d
conda install lightning  -c conda-forge -y
pip install rich hydra-core rootutils hydra-colorlog hydra-optuna-sweeper pydantic==1.10.10 opencv-python plyfile pandas icecream smplx scikit-image chumpy

given the code bor.py
import torch
import lightning as L
from lightning.pytorch.demos.boring_classes import BoringModel

ngpus = 2

model = BoringModel()
trainer = L.Trainer(max_epochs=10,
                    devices=ngpus)

trainer.fit(model)
print('Done')

run the code bor.py, the code stuck after  "All distributed processes registered"
I tried setting NCCL_P2P_DISABLE=1, but when I set this, the cpu seems to meltdown
Message from syslogd@sys at Jan  6 22:27:01 ...
 kernel:[  485.362134] watchdog: BUG: soft lockup - CPU#11 stuck for 27s! [kworker/11:1:509]

Message from syslogd@sys at Jan  6 22:27:29 ...
 kernel:[  509.586452] NMI watchdog: Watchdog detected hard LOCKUP on cpu 42

Message from syslogd@sys at Jan  6 22:27:29 ...
 kernel:[  513.353369] watchdog: BUG: soft lockup - CPU#11 stuck for 53s! [kworker/11:1:509]

Message from syslogd@sys at Jan  6 22:27:57 ...
 kernel:[  533.021241] NMI watchdog: Watchdog detected hard LOCKUP on cpu 16

The the server is unable to connect. I have to shutdown the power and restart the server.
So I can not set NCCL_P2P_DISABLE=1.
If the code is set to ngpus=1, it won't use ddp, then everything is ok.
I can reproduce the bug situation by the above steps.
What version are you seeing the problem on?
v2.1
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please
tning.pytorch` package, due to potential conflicts with other packages in the ML ecosystem. For this reason, `logger=True` will use `CSVLogger` as the default logger, unless the `tensorboard` or `tensorboardX` packages are found. Please `pip install lightning[extra]` or one of them to enable TensorBoard support by default
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/2
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/2
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 2 processes
----------------------------------------------------------------------------------------------------

You are using a CUDA device ('NVIDIA GeForce RTX 4090') that has Tensor Cores. To properly utilize them, you should set `torch.set_float32_matmul_precision('medium' | 'high')` which will trade-off precision for performance. For more details, read https://pytorch.org/docs/stable/generated/torch.set_float32_matmul_precision.html#torch.set_float32_matmul_precision

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

Fine, downgrading to 535.146.02 solved the problem. maybe the problem is within nvidia driver, not lightning.


Can shaping be reused for different line breaks?

Can a the result of shaping a Unicode string differ from concatenating the results of shaping the two parts, after splitting that string at a line break opportunity?
Or phrased differently: when making multiple layouts of an otherwise-identical paragraph by making different choices about which line break opportunities to take, can we reuse some of the results of shaping computation? This might come up for example when resizing a window or other container.
For CSS layout specifically, we sometimes need to compute the min-content and max-content intrinsic size of an element. For text, this means measuring the width of the widest line when taking respectively all or none of the break opportunities.
Assuming that shaping is somewhat expensive, it would be good if we can avoid doing it from scratch three times. (Twice for min- and max-content, then once more for “real” layout.)

What about in the case of soft hyphens, especially with Arabic (soft hyphens work with Arabic and leave the letter in its joined form even when broken).
Also kashida justification, but we're not planning on supporting that (yet?).

It’s quite possible that my question comes from a Latin-centric view and this doesn’t work at all in the general case, or not as easily as I’d hope.

Ok, this is a good question, because it touches on a lot of issues. The way Android solves the problem is a good model, but it has different constraints (legacy API) and it might be possible to do better. I'll briefly describe the Android approach, then ideas for how to do things better. We do want to treat shaping as expensive and avoid duplication of work as much as possible.
First, Android makes the assumption that shaping can be split on "word boundaries", that shaping results can be cached, and layouts can be assembled from these pieces. The definition of "word boundary" for this purpose is interesting because it doesn't correspond to anything in UAX 29 or similar. Basically, it's a space character or the boundary between two ideographs. This is essentially making the assumption that shaping does not happen across a space, and also that there's no kerning of ideographs. These are maybe 99.9% valid assumptions, possibly we want to tweak them (Android doesn't).
Line break opportunities overlap some with this concept, but not entirely. For non-hyphenated Latin text (and other similar scripts), and also CJK, they're similar, but then there are the exceptions. Hyphens are one, as a hyphen can (and often does) kern with the adjoining text. There are two schools of thought here, approximation based on adding the width of the hyphen, and exact measuring. Android does the latter. Note also that, even in Latin, there are line break opportunities caused by punctuation, and these also affect shaping.
So assuming the cache is fast, I think the Android approach is reasonable: just shape the text three times (min, max, and actual), and expect very high cache hit rates. The exceptions, like punctuation, will be different cache entries.
There is another possibility, based on retaining layout objects, rather than relying on the cache for persistence. To do this would involve exposing the boundaries for shape-caching. Then, a layout driver would iterate along both shape-caching and line-break boundaries, adding the widths to get a max intrinsic width, making the max width to get min intrinsic width, and potentially persisting the layout objects once line breaks are determined, reusing them when the boundaries align.
This would be a somewhat more complex API, so the obvious question is whether it's worth it. I think a lot of that has to do with the cost of cache lookup. One candidate for the API is for the layout method to return a sequence (can be an iterator) of individually cached shaped layouts, and the higher level driver reusing layouts when the boundaries align.
The challenge varies by script, but Latin has similar problems as other scripts. One of the most challenging is Thai (and similar Southeast Asian scripts), as line break opportunities are more frequent than shaping-cache boundaries. For this script, it's likely that it will need three shaping passes, and caching is not likely to be effective for max-intrinsic and actual.
One possible optimization that comes to mind is the underlying shaping engine reporting shaping boundaries (ie a guarantee that breaking the string at that boundary preserves shaping on concatenating the shaped substrings). I'll bring this up with @behdad when we meet later in the week.

cc @jfkthame, do you know what Gecko does here? I'm pretty sure Gecko has a word cache, but that's pretty much all I know about it :)

Firefox and current Chrome / Blink use a word cache as well, but disable the word cache if they detect that the space glyph interacts with the lookups for the script.  That's done using hb_ot_layout_lookup_collect_glyphs().
Blink's layout-ng rewrite removes caching and instead retains the shaping result on a per paragraph basis.  That's where assistance from HarfBuzz to avoid reshaping comes in.
There's some relevant discussion in https://github.com/harfbuzz/harfbuzz/issues/1463.  I'll summarize here:
HarfBuzz already provides a flag called UNSAFE_TO_BREAK.  That flag basically tells you, after shaping, which positions in the text have the property that shaping the two sides separately and concatenating results in the same output.  However, as I explained in https://github.com/harfbuzz/harfbuzz/issues/1463#issuecomment-448081494, this doesn't tell us anything about shaping of any other string.
It's possible to design stronger flags.  I'll continue in https://github.com/harfbuzz/harfbuzz/issues/1463 when I have a better proposal.

Yes, @behdad and I had a very good conversation on Friday where we went deep into this. I'm now encouraged to rely less heavily on caching (it has its own issues) and see if we can reuse more.
Since then, I've had an idea for an API which might be both nice and efficient.  From the object you get back, you can query for width and/or layout of an arbitrary substring (or, in the case of hyphenation, an arbitrary substring plus an additional hyphen character).
Internally, today that would use the HarfBuzz UNSAFE_TO_BREAK flag. Based on our conversation, I believe that if both start and end offsets have !UNSAFE_TO_BREAK, then that sublayout can be reused. Otherwise, a new layout is done of the substring.
My rough analysis is that this would work pretty well for Latin (most of the time, spaces and thus line breaks would be safe to break), and especially well for CJK.
This logic can then be refined as HarfBuzz evolves. I think the nice thing about the API I propose is that it's not too dependent on the details of how it's done under the hood. For example, even with a highly cache-centered approach, the style parameters (the "paint") can be interned in the first call, and then the interned paint can be used as part of the cache key. Doing it this way avoids the need to have an explicit API for interning (with associated lifetime and thread-safety concerns).

From the object you get back, you can query for width and/or layout

By the way, is computing the width cheaper than doing “full” shaping and layout? If not in amount of computation, perhaps in memory space for storing results?

By the way, is computing the width cheaper than doing “full” shaping and layout? If not in amount of computation, perhaps in memory space for storing results?

Yes, though this needs careful empirical measurement. At the very least, it can be done without allocating, while full layout requires allocating the result buffer. I think it's also a question of how heavily we rely on caching - if we go through HarfBuzz, the difference might not be that much, but if we hit in the cache the relative cost of writing the layout result might be significant.

https://github.com/harfbuzz/harfbuzz/issues/1463#issuecomment-505592189 looks relevant.

harfbuzz/harfbuzz#1463 (comment) looks relevant.

Indeed..
I haven't been able to reason about their model yet.  It's an interesting model (stop as soon as one "cluster" shaped the same as before...)  I will be thinking about it and implement something in HarfBuzz in the next couple months or so.


Hyperdrive gets stuck loading after adding a hypercore

So I add a hyperdrive, everything works fine, add watchers and load some files and everything is hunky dory.
If I then add a hypercore (with the intention to use the extension messages) the hypercore works 100% fine, but the drive gets stuck when loading files.
The expected behavior would be that my hyperdrive keeps working after the addition of another hypercore.

Would you mind posting a minimal test case to demonstrate this?

Gonna close this for now since it's inactive.
Feel free to post if you'd like it reopened. 😁


Misleading instructions

Where it says "Replace the <container-name> placeholder with any name that you want to give your container.", I found this did not work. I had to pre-create a container and use the name of the created container.
The same applies to this bit "Replace the container-name placeholder with any name that you want to give your container."

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7a000caa-6f6e-de6c-fb18-a342a6b1221b
Version Independent ID: 5f638d37-66bc-9987-35c0-8445170bd086
Content: Tutorial: Azure Data Lake Storage Gen2, Azure Databricks & Spark
Content Source: articles/storage/blobs/data-lake-storage-use-databricks-spark.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

Thank you for raising this @alexshipp-er ! Yes. the text is indeed incorrect. We've corrected this and the change should appear live shortly. Thanks again!

#please-close


Jared/updating metadata from script

@sbiswas96 Ready for review. This is a pretty big PR so brace yourself. I would say it doesn't need an extremely thorough line by line review but there are a couple primary things to look out for that I tried to account for.

No lost categories.
a. Originally "dataItems" was being lost but I made corrections in the script for this so we should be okay. They have just been relocated as the script alphabetizes them. This didn't have any impact on the sampleviewer.
b. redirect_from is another key category that if lost could be problematic. I didn't see any occurrences of this but would rather be safe than sorry.

Otherwise the updated information was pulled from the README.md which should be safe to assume is correct. I caught a few mistakes here and there and updated those READMEs as well in the process but once those were shored up I didn't see any problems.

@sbiswas96 Should have fixed up all the issues with the metadata.json files as well as the related readmes. I only have one outstanding question that came to my attention from a merge conflict with this but otherwise I think this is good to go!

@sbiswas96 Ran the script over the samples again and caught a few more mistakes on the readme's so I made those changes as well. Hopefully this is the last iteration!


Continue breaks with togethercomputer/llama-2-70b-chat

Describe the bug
It is not possible to use Together API with llama-2-70b-chat model
To Reproduce
Steps to reproduce the behavior:

Change config to
python  models=Models(      default=TogetherLLM(          api_key="api-key",          model="togethercomputer/llama-2-70b-chat"      )  ),
Send an instruction
Press Continue
See the error

Error
Traceback (most recent call last):

  File "/var/folders/zd/zfms33rs6gj781r191yw4_jw0000gn/T/_MEIO3tfVC/continuedev/src/continuedev/libs/llm/together.py", line 101, in stream_chat
    "content": json.loads(chunk[6:])["choices"][0]["text"]

  File "json/__init__.py", line 346, in loads

  File "json/decoder.py", line 337, in decode

  File "json/decoder.py", line 355, in raw_decode

json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File "continuedev/src/continuedev/core/autopilot.py", line 264, in _run_singular_step
    observation = await step(self.continue_sdk)

  File "continuedev/src/continuedev/core/main.py", line 326, in __call__
    return await self.run(sdk)

  File "continuedev/src/continuedev/plugins/steps/chat.py", line 67, in run
    async for chunk in generator:

  File "/var/folders/zd/zfms33rs6gj781r191yw4_jw0000gn/T/_MEIO3tfVC/continuedev/src/continuedev/libs/llm/together.py", line 104, in stream_chat
    raise Exception(str(line[0]))

Exception: b'{"status":"finished","prompt":["\\n<bot>: - Highlight code section and ask a question or give instructions\\n- Use `cmd+m` (Mac) / `ctrl+m` (Windows) to open Continue\\n- Use `/help` to ask questions about how to use Continue\\n- [Customize Continue](https://continue.dev/docs/customization) (e.g. use your own API key) by typing \'/config\'.\\n<human>: hello\\n<bot>:"],"model":"togethercomputer/llama-2-70b-chat","model_owner":"","tags":{},"num_returns":1,"args":{"prompt":"\\n<bot>: - Highlight code section and ask a question or give instructions\\n- Use `cmd+m` (Mac) / `ctrl+m` (Windows) to open Continue\\n- Use `/help` to ask questions about how to use Continue\\n- [Customize Continue](https://continue.dev/docs/customization) (e.g. use your own API key) by typing \'/config\'.\\n<human>: hello\\n<bot>:","max_tokens":1024,"temperature":0.5,"top_p":1,"frequency_penalty":0,"presence_penalty":0,"model":"togethercomputer/llama-2-70b-chat","stream_tokens":true},"subjobs":[],"output":{"error":"unmarshal output \\"\\": unexpected end of JSON input","result_type":"error"}}'

Expected behavior
It should not error
Screenshots

Solved, had to deal with different JSON formats coming in from different models (sometimes prefixed by "data: ")


Add command line option to pass extra build args

This is effectively the second part of https://github.com/jupyterhub/repo2docker/pull/1097
Instead of allowing an appendix for custom Dockerfiles, this adds the ability to pass arbitrary build args to the build process, which is much cleaner and more predictable than appending lines to the Dockerfile.
For the normal buildpacks this does generally nothing, since they don't expect any buildargs you can't already set via other switches, but for custom Dockerfiles it adds a range of new possibilities.

Thanks a lot, @TimoRoth! Would love for a few example use cases!

Our setup is at https://github.com/OGGM/r2d
Currently the custom Dockerfile setup is in the test-docker branch, but will eventually be merged into master now that this is merged.
We have a lot of intricate binary dependencies, and also profit a lot from using Pyston.
When trying to set this up with conda, we end up with a horrendously broken environment that fails most of our tests with obscure errors(see the chain of failed actions GH Actions builds for the master branch on that same repo) most of the time. While with our docker images it's no huge issue to get this working at all times, since we have very precise control about how everything fits together.


First Curriculum Rewrite

Moved all systems design and theory portions of the curriculum to relevant places for information security. trimmed the ethics section, moved databases so it's higher in rank order of importance in the curriculum, created our own Github badges, began rewriting the Final Project Section.

I intended to full-caps the badge, and I agree the next phase should be to muck with the server invites so they're not valid (we can replace them later). I Intend to work on the Code of Conduct tomorrow.

maybe in future commits, we should add a smidge more vertical space so it's easier to read?

Yeah looks good, I'll add in the cryptography content tomorrow.


In the line 240, the string 'conv_3' miss the %d

I think the author missed the '%d' to name the related net scope, so the correct code is :
net = slim.conv2d(net, 256, [3, 3], scope='conv3_%d' % (i+1)) NOT net = slim.conv2d(net, 256, [3, 3], scope='conv3_' % (i+1))

Can one of the admins verify this patch?

Please address the CLA. Thanks!

I have finished the CLA, then anything I need to do next?

Are the commits using the same email address you signed a CLA for? If now, you should change the email in the commit, and the CLAbot should pick that up.

@Eric2016Lv I think you have to comment with "I signed it!". Please try that.

I signed it!

@Eric2016Lv please verify the CLA issue as we can't merge the change otherwise.

@Eric2016Lv I did check the CLA database and you're not in. Can you check again? Did you sign via an organization?

I will close this pull request due to inactivity.
Please feel free to reopen once you have taken care of the CLA.

Acutally, I have signed the CLA.  The folowing link  I cliked and push the "I Agree" putton. Then, the reason why not successful which confused me. Would you help me?
https://cla.developers.google.com/
The attachment which is the picture that I took from the website, plz check.
Thx,
Eric Lv
2017-11-04
didi_lv
发件人：gunan<EMAIL_ADDRESS>发送时间：2017-11-04 11:35
主题：Re: [tensorflow/tensorflow] In the line 240, the string 'conv_3' miss the %d (#12578)
<EMAIL_ADDRESS>抄送："Eric2016Lv"didi_lv<EMAIL_ADDRESS>Closed #12578.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Is the email address you used to sign the CLA match the one you created the commit with?

No I used the gmail to sign it.

Now, I have added the gmail address into the Github email list. Please check it, thanks a lot.

If not, I will give up to sign it, too complex for me. anyway, thanks all for the help.

Can one of the admins verify this patch?

CLA bot usually checks the commit email address, but let's see if this fixes it.
Could you comment exactly with the words I signed it! in this thread?

All commits have to be made with an email that's associated with a CLA. If
there's more than one email in a PR, clabot won't be able to check it, but
it's message will change.
On Nov 4, 2017 12:37, "gunan"<EMAIL_ADDRESS>wrote:

CLA bot usually checks the commit email address, but let's see if this
fixes it.
Could you comment exactly with the words I signed it! in this thread?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/12578#issuecomment-341923955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjO_ZpLXIvgInv40dGmr22W-2smMkA7ks5szLzvgaJpZM4PCInT
.

I signed it!

Jenkins, test this please.

windows cmake issue is a known file availability issue.
Merging.


tr-ocr model gs demo Benchmarking

GS e2e Demo
test uses batched real inputs and produces batched outputs
this test is looped (takes several batches of inputs, produces several batches of outputs)
test evaluates results of TT model against validation data in selected data set
test is documented
how to run test (commands to execute)
what does the model do (high-level explanation in a couple of sentences)
description of what is input and what is output
this demo should load inputs and expected outputs from a data set on Weka (TBD which one)

Benchmarking of tr-ocr model is done ,
Data-set: tt-metal/models/sample_data/iam_ocr_image.jpg link
Metric: Character error rate (CER)
Note: trocr model getting low PCC, thereby affecting its accuracy during benchmarking.
commit link

Rebasing to latest main, the test hangs.
Able to reproduce the same results on main branch's older commit


add firebase example

Quickly set up authentication and a real time database via firebase.

Ref #820

@captDaylight Thanks for the example, is it possible to have next.js hook up with Firebase and have the data rendered server side with auth and private routes ?

@tezhub I'm new to Firebase, but having a look around it seems it can be hooked up. However, since firebase is run generally on the client maybe it's best to keep this example on the client. I can imagine firebase-server being a separate example but I'm open to hearing what others think!

is it possible to have next.js hook up with Firebase

@tezhub @captDaylight Yes, you can. Start from the custom-server example and work Firebase into that. Your main decisions will be:

Do I need firebase-admin or firebase?
Am I going to use Firebase's built-in Auth systems, or generate custom JWT tokens?
With either package, how will I want to store the Auth token/session & pass that info along to my database?

If you're going to use Firebase's done-for-you authentication, then you'll need a minimum of firebase-app and firebase-auth on the client. That's enough to give you what you need, as well as the tools to send the server what it needs to figure out who's who. Everything else can be done server-side.

https://github.com/zeit/next.js/pull/1756


Test performance of Enso via WSL

Summary
As Python and other "C-heavy" languages are not supported by GraalVM on Windows, we might want to run Enso via WSL. It is important to understand the performance implications of that, including file loading or any runtime performance overhead. If we come to the conclusion that this is the proper solution, then the WSL support would need to be added on Engine side, as it needs to work both in GUI as well as in batch (cmd) mode.
Specification

Create a test to load a big data file (1 Gb) and check the loading speed on Windows native and under WSL.
Run our benchmarks on Windows native and WSL and compare the results.

Acceptance Criteria & Test Cases

Results of tests

Reading 1GB file
I've generated a 1GB file filled with ascii character.
Tried measuring Enso's reading functions, however they are too slow to provide any meaningful results. See #1724.
So I've run Python's read_text on the file and compared between WSL and native Windows.

Time [s]
relative to Win

Win
9.63

WSL
28.5
+196%

(average over 30 runs)
Whether the WSL accessed file on mounted NTFS drive or read it from its local filesystem did not make significant difference.
 Results for v2 will come on the next day.
WSL v1
Reading 1GB file
I've generated a 1GB file filled with ascii character.
Tried measuring Enso's reading functions, however they are too slow to provide any meaningful results. See #1724.
So I've run Python's read_text on the file and compared between WSL and native Windows.

Time [s]
relative to Win

Win
9.63

WSL
28.5
+196%

(average over 30 runs)
Whether the WSL accessed file on mounted NTFS drive or read it from its local filesystem did not make significant difference.
Our benchmarks
Aggregated results (median over 10 runs)

citations

listings

read
processing
read
processing
makes processing
final processing

command time

WIN
4,096
0,216
1,196
0,128
0,7785
0,0435

26,8285

WSL
5,592
0,211
1,625
0,1315
0,737
0,048

29,2

relative
+37%
-2%
+36%
+3%
-5%
+10%

+9%

Individual run results

citations

listings

read
processing
read
processing
makes processing
final processing

command time

WSL
6,034
0,218
1,869
0,138
0,696
0,047

29,743

WSL
5,524
0,204
1,565
0,117
0,642
0,04

28,289

WSL
5,66
0,218
1,481
0,114
0,699
0,049

28,915

WSL
5,896
0,184
1,685
0,148
0,969
0,049

29,939

WSL
5,236
0,204
1,862
0,125
0,765
0,114

28,54

WSL
6,178
0,277
1,746
0,138
0,83
0,029

29,743

WSL
5,272
0,31
1,495
0,357
0,709
0,036

29,485

WSL
5,432
0,185
1,531
0,118
0,788
0,052

27,898

WIN
4,002
0,308
1,127
0,128
0,779
0,045

27,325

WIN
4,13
0,276
1,147
0,128
0,729
0,045

26,707

WIN
4,222
0,237
1,154
0,133
0,837
0,042

26,85

WIN
4,452
0,185
1,279
0,132
0,803
0,045

27,917

WIN
4,225
0,254
1,348
0,116
0,778
0,033

27,527

WIN
4,062
0,186
1,264
0,138
0,802
0,075

26,807

WIN
3,953
0,158
1,238
0,12
0,748
0,034

26,201

WIN
3,82
0,195
1,05
0,121
0,672
0,028

24,227

Summary
The processing performance is comparable across WSL and native Windows.
Reading CSV is about 35% slower.
Whether files and engine were located in WSL local filesystem on invoked through mounted NTFS disk made no difference.

Updated with WSL v2 results.

It looks like you've got an unfinished sentence in the big update. Can you clarify what you were trying to say?