✨ Output in automatic provisioning like 0.10

What are you trying to do?
Dagger 0.11 made changes to the console output which I believe to be a large downgrade.
What I used to do:
I would run my pipeline without the dagger binary, simply running python main.py. This would print the output from the container as well as my program output (i.e., calls to logger.info()) to the console in real time. Once the process was complete, all the container logs + my own would still be clearly visible for reference.
I found this approach to be the best for 2 reasons:

I did not have to install the dagger binary, I just needed to make sure that dagger-io was installed in my Python virtual environment.
It produced output that was by far the most understandable, readable, and as a result, useful.

The Downgrade
With dagger 0.11, simply running python main.py produces only the program output and none from the containerised process itself.
The best alternative is to run dagger run -vvv python main.py, but I find that to be significantly worse than running python main.py with dagger 0.10 due to the following points:

It requires the dagger binary to be installed (extra work)
The output is split into 2 sections, "container output" first and "program output" second, which is harder to make sense of.
The container output is significantly summarised and not nearly as verbose as that generated in earlier versions.

Why is this important to you?
Seeing the output of a program is incredibly helpful for pretty much every part of the development process.
How are you currently working around this?
I am not upgrading to dagger 0.11, and using 0.10 instead.

You can also increase verbosity to debug level:
dagger run --debug python main.py

And you can swap the TUI with the older output:
dagger run --progress=plain python main.py

So it boils down to not wanting to install the dagger CLI beforehand.
\cc @vito

I wasn't aware there was a difference, thanks.
Can you see if this helps (pyhthon -u)?
dagger run --progress=plain python -u build_binary.py ...

Does not make a difference, see output:
$ dagger run --progress=plain python -u build_binary.py --program solana-exporter-go --tag v2.0.2
1: in exec docker start dagger-engine-63121e921843d412
1: dagger-engine-63121e921843d412

Connected to engine 6398f17bfe5b (version v0.11.1)
2: in starting session
2: OK!

3: in dagger run --progress=plain python -u build_binary.py --program solana-exporter-go --tag v2.0.2
3: {"level": "INFO", "time": "2024-04-22 12:50:49,859", "message": Running build_binary for program SOLANA_EXPORTER_GO and tag v2.0.2, building to .builds}

4: in Query.checkVersionCompatibility
4: 10:50:50 DBG Using development engine; skipping version compatibility check.

3: in dagger run --progress=plain python -u build_binary.py --program solana-exporter-go --tag v2.0.2
3: {"level": "INFO", "time": "2024-04-22 12:50:50,632", "message": Building SOLANA_EXPORTER_GO binary with branch/tag v2.0.2.}
3: {"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ubuntu:22.04@sha256:6042500cf4b44023ea1894effe7890666b0c5c7871ed83a97c36c76ae560bb9b}

3: in dagger run --progress=plain python -u build_binary.py --program solana-exporter-go --tag v2.0.2
3: {"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Using Dagger cache.}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Updating APT_GET}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Executing container command: apt-get update}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,637", "message": Container successfully executed: apt-get update}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,638", "message": Executing container command: apt-get upgrade -y}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Container successfully executed: apt-get upgrade -y}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Executing container command: apt-get install -y apt-utils}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Container successfully executed: apt-get install -y apt-utils}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Installing the following dependencies: ['git', 'curl', 'build-essential', 'tar', 'file']}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Executing container command: apt-get install -y git curl build-essential tar file}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Container successfully executed: apt-get install -y git curl build-essential tar file}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning repository ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning certusone/solana_exporter [v2.0.2] into solana_exporter-v2.0.2}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Executing container command: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,672", "message": Container successfully executed: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Installing dependencies ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": No project-specific dependencies required, skipping ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Adding tag v2.0.2 as a 'TAG' environment variable}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing program-specific build steps.}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Installing go}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Downloading and installing go 1.21.0}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing container command: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Container successfully executed: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Executing container command: tar -zxf go.tar.gz -C /usr/local/}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,697", "message": Container successfully executed: tar -zxf go.tar.gz -C /usr/local/}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Building solana_exporter v2.0.2}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Executing container command: go build -o build/solana-exporter ./cmd/solana_exporter}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Container successfully executed: go build -o build/solana-exporter ./cmd/solana_exporter}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Exporting binary ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Moving into build ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Including the following hard-coded files for export: ['solana-exporter']}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file (checksums.sha256) in container ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Executing container command: sha256sum solana-exporter}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,726", "message": Container successfully executed: sha256sum solana-exporter}

3: in dagger run --progress=plain python -u build_binary.py --program solana-exporter-go --tag v2.0.2
3: {"level": "INFO", "time": "2024-04-22 12:50:53,734", "message": Checksums: 074d6d8f362212935ba35c683e0e6384fa624137f2765692236ce1b95ea41f76  solana-exporter,        }
3: {"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Wrapping up container:['solana-exporter', 'checksums.sha256'] to container:export-package-vessel.tar.gz ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Executing container command: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Container successfully executed: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Executing container command: du -sh export-package-vessel.tar.gz}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,789", "message": Container successfully executed: du -sh export-package-vessel.tar.gz}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,798", "message": Exporting export-package-vessel.tar.gz [6.0M] to host ...}
3: {"level": "INFO", "time": "2024-04-22 12:50:53,887", "message": Successfully exported container:export-package-vessel.tar.gz to host:.builds/solana-exporter-go-v2.0.2.tar.gz}

✔ connect 2.4s
✔ Container.from(address: "ubuntu:22.04@sha256:6042500cf4b44023ea1894effe7890666b0c5c7871ed83a97c36c76ae560bb9b"): Container! 2.8s
  ✘ remotes.docker.resolver.HTTPRequest 1.2s
    ✘ HTTP HEAD 1.2s
  ✔ HTTP GET 1.2s
  ✔ remotes.docker.resolver.HTTPRequest 0.3s
    ✔ HTTP HEAD 0.3s

{"level": "INFO", "time": "2024-04-22 12:50:49,859", "message": Running build_binary for program SOLANA_EXPORTER_GO and tag v2.0.2, building to .builds}
{"level": "INFO", "time": "2024-04-22 12:50:50,632", "message": Building SOLANA_EXPORTER_GO binary with branch/tag v2.0.2.}
{"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ...}
{"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ubuntu:22.04@sha256:6042500cf4b44023ea1894effe7890666b0c5c7871ed83a97c36c76ae560bb9b}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Using Dagger cache.}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Updating APT_GET}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Executing container command: apt-get update}
{"level": "INFO", "time": "2024-04-22 12:50:53,637", "message": Container successfully executed: apt-get update}
{"level": "INFO", "time": "2024-04-22 12:50:53,638", "message": Executing container command: apt-get upgrade -y}
{"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Container successfully executed: apt-get upgrade -y}
{"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Executing container command: apt-get install -y apt-utils}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Container successfully executed: apt-get install -y apt-utils}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Installing the following dependencies: ['git', 'curl', 'build-essential', 'tar', 'file']}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Executing container command: apt-get install -y git curl build-essential tar file}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Container successfully executed: apt-get install -y git curl build-essential tar file}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning repository ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning certusone/solana_exporter [v2.0.2] into solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Executing container command: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,672", "message": Container successfully executed: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Installing dependencies ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": No project-specific dependencies required, skipping ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Adding tag v2.0.2 as a 'TAG' environment variable}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing program-specific build steps.}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Installing go}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Downloading and installing go 1.21.0}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing container command: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Container successfully executed: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Executing container command: tar -zxf go.tar.gz -C /usr/local/}
{"level": "INFO", "time": "2024-04-22 12:50:53,697", "message": Container successfully executed: tar -zxf go.tar.gz -C /usr/local/}
{"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Building solana_exporter v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Executing container command: go build -o build/solana-exporter ./cmd/solana_exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Container successfully executed: go build -o build/solana-exporter ./cmd/solana_exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Exporting binary ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Moving into build ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Including the following hard-coded files for export: ['solana-exporter']}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file (checksums.sha256) in container ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Executing container command: sha256sum solana-exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,726", "message": Container successfully executed: sha256sum solana-exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,734", "message": Checksums: 074d6d8f362212935ba35c683e0e6384fa624137f2765692236ce1b95ea41f76  solana-exporter,   }
{"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Wrapping up container:['solana-exporter', 'checksums.sha256'] to container:export-package-vessel.tar.gz ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Executing container command: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
{"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Container successfully executed: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
{"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Executing container command: du -sh export-package-vessel.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,789", "message": Container successfully executed: du -sh export-package-vessel.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,798", "message": Exporting export-package-vessel.tar.gz [6.0M] to host ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,887", "message": Successfully exported container:export-package-vessel.tar.gz to host:.builds/solana-exporter-go-v2.0.2.tar.gz}

{"level": "INFO", "time": "2024-04-22 12:50:49,859", "message": Running build_binary for program SOLANA_EXPORTER_GO and tag v2.0.2, building to .builds}
{"level": "INFO", "time": "2024-04-22 12:50:50,632", "message": Building SOLANA_EXPORTER_GO binary with branch/tag v2.0.2.}
{"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ...}
{"level": "INFO", "time": "2024-04-22 12:50:50,761", "message": Fetching base image ubuntu:22.04@sha256:6042500cf4b44023ea1894effe7890666b0c5c7871ed83a97c36c76ae560bb9b}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Using Dagger cache.}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Updating APT_GET}
{"level": "INFO", "time": "2024-04-22 12:50:53,624", "message": Executing container command: apt-get update}
{"level": "INFO", "time": "2024-04-22 12:50:53,637", "message": Container successfully executed: apt-get update}
{"level": "INFO", "time": "2024-04-22 12:50:53,638", "message": Executing container command: apt-get upgrade -y}
{"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Container successfully executed: apt-get upgrade -y}
{"level": "INFO", "time": "2024-04-22 12:50:53,646", "message": Executing container command: apt-get install -y apt-utils}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Container successfully executed: apt-get install -y apt-utils}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Installing the following dependencies: ['git', 'curl', 'build-essential', 'tar', 'file']}
{"level": "INFO", "time": "2024-04-22 12:50:53,656", "message": Executing container command: apt-get install -y git curl build-essential tar file}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Container successfully executed: apt-get install -y git curl build-essential tar file}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning repository ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Cloning certusone/solana_exporter [v2.0.2] into solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,663", "message": Executing container command: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,672", "message": Container successfully executed: git clone https://github.com/certusone/solana_exporter.git --branch v2.0.2 --depth 1 solana_exporter-v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Installing dependencies ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": No project-specific dependencies required, skipping ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,677", "message": Adding tag v2.0.2 as a 'TAG' environment variable}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing program-specific build steps.}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Installing go}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Downloading and installing go 1.21.0}
{"level": "INFO", "time": "2024-04-22 12:50:53,684", "message": Executing container command: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Container successfully executed: curl -L https://go.dev/dl/go1.21.0.linux-amd64.tar.gz -o go.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,690", "message": Executing container command: tar -zxf go.tar.gz -C /usr/local/}
{"level": "INFO", "time": "2024-04-22 12:50:53,697", "message": Container successfully executed: tar -zxf go.tar.gz -C /usr/local/}
{"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Building solana_exporter v2.0.2}
{"level": "INFO", "time": "2024-04-22 12:50:53,703", "message": Executing container command: go build -o build/solana-exporter ./cmd/solana_exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Container successfully executed: go build -o build/solana-exporter ./cmd/solana_exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Exporting binary ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,710", "message": Moving into build ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Including the following hard-coded files for export: ['solana-exporter']}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Creating checksums file (checksums.sha256) in container ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,720", "message": Executing container command: sha256sum solana-exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,726", "message": Container successfully executed: sha256sum solana-exporter}
{"level": "INFO", "time": "2024-04-22 12:50:53,734", "message": Checksums: 074d6d8f362212935ba35c683e0e6384fa624137f2765692236ce1b95ea41f76  solana-exporter,   }
{"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Wrapping up container:['solana-exporter', 'checksums.sha256'] to container:export-package-vessel.tar.gz ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,778", "message": Executing container command: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
{"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Container successfully executed: tar -zcf export-package-vessel.tar.gz solana-exporter checksums.sha256}
{"level": "INFO", "time": "2024-04-22 12:50:53,784", "message": Executing container command: du -sh export-package-vessel.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,789", "message": Container successfully executed: du -sh export-package-vessel.tar.gz}
{"level": "INFO", "time": "2024-04-22 12:50:53,798", "message": Exporting export-package-vessel.tar.gz [6.0M] to host ...}
{"level": "INFO", "time": "2024-04-22 12:50:53,887", "message": Successfully exported container:export-package-vessel.tar.gz to host:.builds/solana-exporter-go-v2.0.2.tar.gz}

\cc @vito

Ah. I've just realized why this is.
This is a result of the specific way we do --progress=plain in dagger 0.11+. Essentially the plain output now uses a new streamingExporter (see #7069), but there's an issue where only spans that have associated logs are printed - so spans without logs don't get any output.
I think we need a bit of a refactor here, I'm going to look at this at some point soon, since the way that the logs are done with this make some of our tests produce enormous quantities of output, and it would be nice to avoid that.


disk_journal: overwrite ordinal files without truncating them

Otherwise a crash at the wrong moment can lead to an empty EARLIEST or
LATEST file.
Ideally this would be done using an atomic rename, instead of an
overwrite, but there seems to be a big (~25%) performance penalty for
that.
Issue: KBFS-1887

Thanks for the review, ready for another look.


samples: mesh: nrf52: amendment in States restoration implementation

For Genric OnPowerUp equal to 0x02 (Restore):
If a transition was in progress when powered down, the element restores the target state when
powered up. Otherwise the element restores the state it was in when powered down. This
commit implements above mentioned logic.

Removed redundant code.
Signed-off-by: Vikrant More<EMAIL_ADDRESS>

I will merge 3rd commit in first one today itself.

Removed the bug.

Please provide any review updates if any.

Hi @jhedberg
Please refer page no. 30 -> 3.14 -> Table 3.6 -> context under '0x02' row.
Previously, I had implemented thing which is not matching with Specs requirement.
I used to save, lightness & temp. value after transition get completed.
But Specs says, if (OnPowerUp == 2) & (transition is in progress) & (device get switched off),
then after device get switched on it should initialize itself with Target values of interrupted transition.
To happens this, if (OnPowerUp == 2), then we need to save target values on PS storage before transition start.

Ok.. I got your point.
I will add last 3 line of this code snapshot ... ( so that if default value = 0 then last value will not be remain as 0)
case STATE_RESTORE:
		light_lightness_srv_user_data.last = (u16_t)
			(light_ctl_srv_user_data.lightness_temp_last >> 16);
		light_ctl_srv_user_data.temp =
			(u16_t) (light_ctl_srv_user_data.lightness_temp_last);
		state_binding(ONPOWERUP, ONOFF_TEMP);

               if (light_lightness_srv_user_data.last == 0) {
                       light_lightness_srv_user_data.last = LIGHTNESS_MAX;
               }

Hi @trond-snekvik,
Thank you very much for your valuable review cum feedback.
As per your recommendations, I have reworked on commit 1. These are latest changes:

Create two separate entities of lightness_last & lgithness_temp_last_target to get store in PS storage
lightness_last will get store as soon as lightness value updates.
lightness_last will always contains non-zero value & this will remove bug in following case

  if (gen_onoff_srv_root_user_data.onoff == STATE_ON) {
 		if (light_lightness_srv_user_data.def == 0U) {
 			lightness = light_lightness_srv_user_data.last;
 		} else {
 				lightness = light_lightness_srv_user_data.def;
 		}
  }


fixing issue with modal on page

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.74%. Comparing base (fb6cbf9) to head (847e706).
Report is 3 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage      #61      +/-   ##
==========================================
+ Coverage   77.80%   83.74%   +5.93%
==========================================
  Files          10       10
  Lines        3100     3100
==========================================
+ Hits         2412     2596     +184
+ Misses        688      504     -184

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Airflow 2.0 support for Cloud Providers

How far away is support for Airflow 2.0?
What is your feature request?

GKE
AWS
Azure

A clear description of what you want.
Airflow 2.0 support for cloud providers
What alternatives have you considered?
Managed Services...

I've managed to get fargate and regular node deployments working for 2.0. I don't mind pushing a PR of an example deployment on AWS EKS. Note that airflow configurations have been decoupled further and renamed, old configs will be deprecated.

I've managed to get fargate and regular node deployments working for 2.0. I don't mind pushing a PR of an example deployment on AWS EKS. Note that airflow configurations have been decoupled further and renamed, old configs will be deprecated.


Continue crashes extensionHost

Before submitting your bug report

[x] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Arch Linux x64 6.10.7-arch1-1
- Continue: v0.8.50
- IDE: VSCodium
- Model: Version 1.92.2, Release 24228, Electron: 30.1.2, Chromium: 124.0.6367.243, Node.js: 20.14.0, V8: <IP_ADDRESS>-electron.0
- config.json:

{
  "models": [
    {
      "title": "Local Ollama",
      "provider": "ollama",
      "model": "phi3.5:3.8b-mini-instruct-q5_K_M"
    }
  ],
  "tabAutocompleteModel": {
    "title": "Local Ollama",
    "provider": "ollama",
    "model": "starcoder2:3b-q5_K_M"
  },
  "embeddingsProvider": {
    "provider": "ollama",
    "model": "nomic-embed-text"
  }
}

Description
Continue never starts. It eventually causes a popup to appear in the lower right to restart extensionHost
To reproduce
Eliminate extensions until only Continue is left and causes the error. Try with pre-release version and with different previous release versions. Try now on a new and clean installation.
0. codium --verbose

Install Continue extension.
Activate Continue: click on Continue icon.

4. Gets output:
...
console4.txt
Logs:
main.log
renderer.log
Log output
DEBUG Comments: URIs of continue on comments to add to storage .
workbench.desktop.main.js:3240 Extension host (LocalProcess pid: 5494) terminated unexpectedly.  No extensions were activated.
workbench.desktop.main.js:660  INFO Automatically restarting the extension host.
workbench.desktop.main.js:660  INFO Started local extension host with pid 5584.
webWorkerExtensionHostIframe.html:1 An iframe which has both allow-scripts and allow-same-origin for its sandbox attribute can escape its sandboxing.
workbench.desktop.main.js:660 DEBUG [Registering Log Channel] File does not exist. Waiting for 1s to retry. file:///home/guest/.config/VSCodium/logs/20240903T143220/window1/exthost/webWorker/workerexthost.log
workbench.desktop.main.js:660 TRACE writeTextAreaState(reason: focusgain)
workbench.desktop.main.js:660 TRACE [text file model] resolve() - enter file:///home/guest/programs/test1-w.-continue.py
workbench.desktop.main.js:660 TRACE [text file model] resolveFromFile() file:///home/guest/programs/test1-w.-continue.py
workbench.desktop.main.js:660 TRACE writeTextAreaState(reason: focusgain)
workbench.desktop.main.js:660  INFO Extension host (LocalProcess pid: 5584) is unresponsive.
workbench.desktop.main.js:3240 Extension host (LocalProcess pid: 5584) terminated unexpectedly.  No extensions were activated.
workbench.desktop.main.js:660  INFO Automatically restarting the extension host.
workbench.desktop.main.js:660  INFO Started local extension host with pid 5615.
webWorkerExtensionHostIframe.html:1 An iframe which has both allow-scripts and allow-same-origin for its sandbox attribute can escape its sandboxing.
workbench.desktop.main.js:660 DEBUG Comments: URIs of continue on comments to add to storage .
workbench.desktop.main.js:3240 Extension host (LocalProcess pid: 5615) terminated unexpectedly.   Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952
onWillAddFirstListener @ workbench.desktop.main.js:93
_event @ workbench.desktop.main.js:91
(anonymous) @ workbench.desktop.main.js:88
onWillAddFirstListener @ workbench.desktop.main.js:88
_event @ workbench.desktop.main.js:91
G @ workbench.desktop.main.js:166
createMouseController @ workbench.desktop.main.js:209
re @ workbench.desktop.main.js:209
j @ workbench.desktop.main.js:742
_createInstance @ workbench.desktop.main.js:655
createInstance @ workbench.desktop.main.js:655
createNotificationsList @ workbench.desktop.main.js:823
show @ workbench.desktop.main.js:823
doAddToast @ workbench.desktop.main.js:1918
(anonymous) @ workbench.desktop.main.js:1918
execute @ workbench.desktop.main.js:97
St @ workbench.desktop.main.js:97
(anonymous) @ workbench.desktop.main.js:97
workbench.desktop.main.js:660 DEBUG Comments: URIs of continue on comments to add to storage .
workbench.desktop.main.js:660 TRACE [text file model] resolve() - enter file:///home/guest/programs/test1-w.-continue.py
workbench.desktop.main.js:660 TRACE [text file model] resolveFromFile() file:///home/guest/programs/test1-w.-continue.py

See above files.

I have the same issue. Vs code installed with snap. Issue started with version 8.45.

I have same issue on ubuntu cinnamon vscode,insiders and VScodium

Same here on Fedora.

Yep. Same here
- OS: fedora 40
- Continue: xxx
- IDE: vscodium 1.92.2 (installed via official repo download.vscodium.com)
- Model: -
- config.json: -

continue.log

The issue come up after install the extension of v0.8.51 and v0.9.207.
The following info is from VS code about box:
Version: 1.93.0
Commit: 4849ca9bdf9666755eb463db297b69e5385090e3
Date: 2024-09-04T13:02:38.431Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.8.0-41-generic (Ubuntu 24.04)

I believe this is caused by https://github.com/lancedb/lance/issues/2195 https://github.com/lancedb/lancedb/issues/1437.

Exact same issue for me that started immediately after I installed the Continue extension on VSCode in Ubuntu 20.04. This is my info:
Version: 1.93.0
Commit: 4849ca9bdf9666755eb463db297b69e5385090e3
Date: 2024-09-04T13:02:38.431Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-122-generic

any fix here?

If your workstation is on a VM like mine (Proxmox), try to change processor type from x86-64-v2 to x86-64-v3 .  It requires avx2. That worked for me.

Same issue on macOS 15.0.1

Same issue on Linux Lite 7.2

The fix is to just use Zed (https://zed.dev). It's free, open source, works well and doesn't crash. It has smart integrations with Antropic, OpenAI and Ollama models.
I've been using it and recommend it.

any fix here?

The fix is to just use Zed (https://zed.dev). It's free, open source, works well and doesn't crash. It has smart integrations with Antropic, OpenAI and Ollama models.
I've been using it and recommend it.

same as @stone-w4tch3r Using 0.8.44 fixes issue on Debian 11

My Continue VSCode extension also crashes Extension Host, but only when I use the Remote - SSH or Remote - Tunnels -- It will crash the (local) Extension Host when the network is bad.
Even worse, with Remote - SSH that will trigger "cannot reconnect" error.
My solution is forcing it to run in the Remote Extension Host by setting.json (see https://code.visualstudio.com/api/advanced-topics/remote-extensions#incorrect-execution-location)
{
    "remote.extensionKind": {
        "Continue.continue": ["workspace"],
    },
}

I found this solution because vscode.dev/tunnel doesn't crash.
After that, it took me a few hours to verify the cause and find the method to force it to run in the remote.


About the configuration file

Is the ‘#’ of all configuration options in merecat.conf not a comment, but is it enabled by default?

They are supposed to illustrate the default value for each setting, but commented-out. Simliar to how SSH config files, and other similar UNIX tools.

@pioneerAlone if you are satisfied with my answer, maybe we can close this issue?

As I thought, I will configure it.
#cgi "**.cgi|/cgi-bin/*" {
enabled = false
limit = 50
#}
Changed to
Cgi "**.cgi|/cgi-bin/*" {
     Enabled = true
     Limit = 50
}
After that, execute [systemctl restart merecat.service];
Enter "<IP_ADDRESS>/cgi-bin/ssi" in the browser; the display is [404 Not Found]

It's definitely possible to run CGI scripts with Merecat, but /cgi-bin/ssi is not a very user-friendly example, nor is the above how to use it.  (See the man page(s) for more on Server-Side Includes, SSI).
However, http://<IP_ADDRESS>/cgi-bin/printenv should work more as you'd expect.
This is actually a very good topic for a blog post, so I'll try to write something up tonight (CET) to showcase the CGI capabilities with Merecat.  If you're still interested that is.

Hi again, I went ahead and wrote that blog post anyway.   Maybe you've moved on to some other web server by now, but here goes.  Maybe someone else will find this thread useful one day :)
https://troglobit.com/2019/08/Basic-CGI-in-C-with-Merecat-httpd/

thanks


Published NonNull Present

So, this may be against the spirit of nullaway and jspecify... However, I think that for a long time it'd be good to annotate any published "interfaces" completley. Meaning that @NonNull should be present on something that is public/protected. Bonus points if it also checks to see if the package is exported if a module-info.java is present.
I'm going to speculate this will be closed ;)

Not sure this is a NullAway issue really?  Like are you asking for an error message if @NonNull is not present?  If so then yes NullAway is not likely to report an error for that :-)

yeah, basically. Didn't hurt to ask?

yeah, probably not, I was thinking might be of your guys interest to implement. Could have 2 separate checks even... One that is "thorough" (public/protected must have something) and one that is just looking for NullMarked.
I could also ask EP itself.

let me know if you want me to re-open

I think it's very unlikely we would be able to prioritize implementing this any time in the near future.  So probably best to leave this closed for now.  If you or one of your colleagues has the cycles, I don't think it would be too hard of an EP check to implement yourselves and try out, and I'd be happy to give feedback on it


Microbit pr04

@0xc0170 ci/circleci failed?

Dont worry about that, I'll disable it again. Someone enabled it without defining tests.


17588 & 17900: Added weightBilled to DDP and DOP

Agility ticket
Agility ticket
Summary
Allowing DOP and DDP Service Items to have weight billed field.
Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against d582c7598a74c19fcc24795703a3c178a2969b18

Does DPK also need a partial weight field, per issue I-12164?

Not sure. I put DUPK and DNPK. I can add these. I had to cross referenced it with the names from a list so its possible.

I am not sure either - only asking on DPK because it was mentioned in the issue. Might be worth asking Holly?

Does DPK also need a partial weight field, per issue I-12164?

Not sure. I put DUPK and DNPK. I can add these. I had to cross referenced it with the names from a list so its possible.

I am not sure either - only asking on DPK because it was mentioned in the issue. Might be worth asking Holly?

If we are going off the list that Dan sent, then yes DPK should be allowed to have a billable weight input.

Just added DPK. I initially added DUPK must have looked past DPK.

Does DPK also need a partial weight field, per issue I-12164?

Not sure. I put DUPK and DNPK. I can add these. I had to cross referenced it with the names from a list so its possible.

I am not sure either - only asking on DPK because it was mentioned in the issue. Might be worth asking Holly?

If we are going off the list that Dan sent, then yes DPK should be allowed to have a billable weight input.

Just added DPK. I initially added DUPK must have looked past DPK.

They all get muddled together, I know haha

Does DPK also need a partial weight field, per issue I-12164?

Not sure. I put DUPK and DNPK. I can add these. I had to cross referenced it with the names from a list so its possible.

I am not sure either - only asking on DPK because it was mentioned in the issue. Might be worth asking Holly?

Just added DPK


"ReferenceError: require is not defined"

Noob here.  I ran npm i node-govee-led on the node red server.  I still can't find it in the pallets anywhere though.  And when I try to run the Run On example, I receive the following error:

ReferenceError: require is not defined

Sorry.  Got it working.  Not sure how to delete the issue above.

Sorry.  Got it working.  Not sure how to delete the issue above.


Variables used for modelling

Project Robyn
Describe issue
After having a look at the code of the project, I concluded that InputCollect.all_media attribute refers to the media variables (both paid and non-paid) to be used for modelling. But, here the paid_media_spends are used instead of the paid_media_vars. As a result, the modelling is performed based on the spend of the media campaigns, and not the exposure media metrics.
This can also be seen in the waterfall plot below, where only labels about the spend on paid media campaigns can be found (the plot is extracted by the demo.R script).

yes - this is intended, check the release notes for 3.6.0 https://facebookexperimental.github.io/Robyn/docs/releases/#360

Hello @kyletgoldberg and thank you for your response! Can I find more details about this change somewhere? As, if I understand correctly, there is no reason for providing exposure media metrics in input.

Hi,
Check the conversation here - https://www.facebook.com/groups/robynmmm/posts/1174300543338077/
We still recommend including the exposure metrics in order to identify places where variables should potentially be split into smaller variables in order to better understand these channels that may have poor correlation between spend and exposure.


Duplicate all modification on the original image

Create a new image based on the source image to duplicate all modification on it and keep the original dimensions.
In relation with https://github.com/MattKetmo/darkroomjs/pull/14 and https://github.com/MattKetmo/darkroomjs/issues/13

I'm closing this old issue since the work has been work in v2 (and v3 will be released soon).


MessageBundle is not work for some locale, e.g. zh_CN

yeah. I have found how this happened in java code : CasReloadableMessageBundle:67 in lastest master branch.:-)
the code blow :
final String filename = this.basenames[i] + "_" + locale.getLanguage();
should be :
final String filename = this.basenames[i] + "_" + locale.toString();
testCase:
if locale can be zh_CN, getLanguage method return 'zh' and toString method return 'zh_CN'.
according to project locale files rule, 'zh_CN' should be right.:-)

Good catch, thanks for the report.


Release Please

Need an APK file

This is just the backend for the app so not yet in APK form @JeseChan.


trying eqwalizer on elixir projects

We wanted to try eqwalizer on our elixir projects. I know it would be a lot better to implement a parser for Elixir projects, but as a first step we decompiled the beam files to erlang sources, and
we were running eqwalizer on those sources.
We have found some problems:

in the decompiled source there is a -file attribute:

-file("lib/asdfqqqq.ex", 1).
-module('Elixir.Asdfqqqq').

When this attribute is there eqwalizer simply ignores all problems. If I manually delete this line, it works.

there is also an info/1 function in the source:

'__info__'(module) -> 'Elixir.Asdfqqqq';
'__info__'(functions) -> [{double, 1}];
'__info__'(macros) -> [];
'__info__'(exports_md5) -> <<"\035S8}I7\030\232\236g-\023">>;

this last function clause will cause this problem:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Utf8Error { valid_up_to: 429, error_len: Some(1) }', crates/elp/src/bin/[reporting.rs:101](http://reporting.rs:101/):78
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

actually, when I manually fix these problems eqwalizer works quite well, and it is able to spot the problems in the code (of course the line numbers are wrong). One remaining problem is that I could not figure out how to tell to eqwalizer where the declarations of Elixir types are. So for instance:

-spec double({error, 1}) -> 'Elixir.String':t().
double(_x@1) -> 2 * _x@1.

Elixir.String type is declared in $ELIXIR_DIR//lib/elixir/lib/string.ex. Similarly, I want to add all files where elixir types are declared. How can I add those using project.json?
I have one more question...
Let's say I have my_app application, which uses my_lib as a dependency. In my_app.erl I have the following:
-include_lib("my_lib/include/my_lib.hrl").

I've tried it with this project.json:
{
  "apps": [
    {
      "name": "my_app",
      "dir": "",
      "ebin": "_build/default/lib/my_app/ebin",
      "extra_src_dirs": [""],
      "include_dirs": ["include"],
      "macros": ["TEST"],
      "src_dirs": ["src"]
    }
  ],
  "deps": [
    {
      "name": "my_lib",
      "dir": "",
      "ebin": "_build/default/lib/my_lib/ebin",
      "extra_src_dirs": [""],
      "include_dirs": ["_build/default/lib/my_lib/include"],
      "macros": ["TEST"],
      "src_dirs": ["src"]
    }

  ],
  "root": ""
}

elp is still complaining about being not able to find the include file:
  eqWAlizing                ████████████████████ 3/3                                                                             error: parse_error
    ┌─ src/my_app.erl:3:14
    │
103 │ -include_lib("my_lib/include/my_lib.hrl").
    │              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find include lib "my_lib/include/my_lib.hrl"

Do you have an idea what should I put into my project.json ?

We were trying to make it work using the following project.json :
{
  "apps": [
    {
      "name": "asdfqqqq",
      "dir": "/home/joe/Projects/asdfqqqq",
      "ebin": "/home/joe/Projects/asdfqqqq/_build/dev/lib/asdfqqqq/ebin",
      "extra_src_dirs": ["test"],
      "include_dirs": ["include"],
      "macros": ["TEST"],
      "src_dirs": ["erlangized_src/"]
    }
  ],
  "deps": [
    {
      "name": "elixir",
      "dir": "/home/joe/.manager/elixir",
      "ebin": "/home/joe/.managerl/erlang-OTP-23.2.3/elixirs/elixir-v1.12.2/lib/elixir/lib/elixir/ebin",
      "extra_src_dirs": [],
      "include_dirs": [],
      "macros": [],
      "src_dirs": ["lib"]
    }

  ],
  "source_root": "/home/joe/Projects/asdfqqqq",
  "otp_lib_dir": "/home/joe/.managerl/erlang-OTP-23.2.3/lib/"
}

Elixir.String.t() type is declared in the above-mentioned folder. With these settings, it still was not able to find the declaration of this type. Is it possible that to find the declaration eqwalizer also needs the erlang source code present in the src directory, or do you have any other ideas on what's going on wrong?

We don't provide any support for type-checking Elixir projects out of the box and we don't plan to do it on our own in the near future
Nevertheless, I think that it's feasible to to use it for Elixir, - but someone needs to drive it. And it's hard to estimate the amount of tech work.

Contributions are welcome.


feat: get_match_config returns with rule_id for stateful plugins

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #406 (e65707a) into main (81e467b) will increase coverage by 0.09%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #406      +/-   ##
==========================================
+ Coverage   40.23%   40.33%   +0.09%
==========================================
  Files          45       45
  Lines        6151     6151
==========================================
+ Hits         2475     2481       +6
+ Misses       3503     3497       -6
  Partials      173      173

see 1 file with indirect coverage changes


Bug: Tight list-items should not contain a paragraph

using mdast.parse():
mdast.parse("- xyz\n- abc\n- foo");

Yields a list AST node, with three listItem children that contain paragraph nodes.
Based on the Markdown spec for lists, there should be no paragraph node wrapping the text of the list item if there is no extra padding newline directly after the list item.
mdast.parse("- xyz\n\n- abc\n\n- foo");

Should emit a structure with the paragraphs inside.

Note that the markdown spec contains two parts: reading markdown, and compiling HTML. I read that sentence as being related to HTML.
Whether a paragraph is rendered depends on the output mechanism. In HTML and JSX, there shouldn’t be such a node of course, but other syntaxes might need it (e.g., man pages).
By always keeping block-level nodes inside list-items (e.g., a paragraph or a block-quote) instead of sometimes exposing inline-level nodes (e.g., links and emphasis) makes processing the syntax tree a lot easier.
I opt for keeping the paragraph inside list-items in all cases.
P.S. See how I implemented the logic in mdast-html ».

@wooorm Actually it does matter, the specific excerpt:

If list items are separated by blank lines, Markdown will wrap the items in  tags in the HTML output. For example, this input:
*   Bird
*   Magic

will turn into:

Bird

Magic

But this:
*   Bird

*   Magic

will turn into:

Bird

Magic

In https://daringfireball.net/projects/markdown/syntax#list

Thanks for your reply, @yaycmyk.
I still don’t agree. The input/output you posted can’t be compared to mdast, it can only compared to mdast and mdast-html or mdast and mdast-react, which both get this right.
Again, the most important reason for this is that the spec of the original markdown makes stuff very hard for plug-ins. Most importantly, it’s not just either-or, there’s also a huge difference between “pedantic” markdown / GFM markdown / commonmark markdown regarding to paragraphs, making things even harder.
Most importantly, what a paragraph means in mdast can be different from what a
 node means in HTML.
I understand your question, but I don’t agree on your conclusion (“It's definitely up to the parser to only add paragraph AST nodes when specifically needed.”). In my opinion, it’s the job of the compiler to handle conversion from a syntax tree to HTML, which in your case is your custom JSX compiler. That compiler is supposed to do this.

Most importantly, what a paragraph means in mdast can be different from what a  node means in HTML.

How so? If that were true, you'd just emit the contents as a TextNode and be done with it. Paragraph means something very specific.

And as a compiler, it doesn't make sense to have to do lookaheads through the tree so often to figure out if we have to unwrap children, with little information about the original structure of the markdown.
For example with the newline issue to wrap with
, if I look at the AST returned by mdast.parse(), I have no idea if the original markdown had extra newlines and that wrapping
 was intentional. I understand there are some differences between the different markdown variants, but this is kind of important and can materially change how the content should be emitted.

I think the most important concept for me is that there’s a difference between the syntax tree format, and the compiler. You constantly omit the use of plug-ins in your use-cases whereas they are the core of mdast. My goals with mdast are aligned differently from yours.
Also, you slightly hyperbolic in the case regarding “lookaheads through the tree so often”. Please check the example I linked mentioned above again.
An last but not least, mdast is an abstract syntax tree, not a concrete syntax tree. Not everything is stored, making the tree smaller and more easily manipulated, but loosing *some* information.


github: replace python versions by OS versions

Due to #586, we cannot test with anything by the OS provided python version. As such, let's test with multiple OS versions and against the respective LXD version for that OS.

#592 supersedes this PR.


fixBug: React.createClass is not a function

After React version 15.5 we can't use React.createClass, https://reactjs.org/blog/2017/04/07/react-v15.5.0.html, create-react-class fix it.

Coverage remained the same at 96.242% when pulling 2ad517993b3eb4d3ae6db3c7eadd6f6aeb9c5a68 on khaninD:fixExamples into 196ea66981d1e173a0a1a23ac71301683b1abc71 on FormidableLabs:master.

Coverage remained the same at 96.242% when pulling 2ad517993b3eb4d3ae6db3c7eadd6f6aeb9c5a68 on khaninD:fixExamples into 196ea66981d1e173a0a1a23ac71301683b1abc71 on FormidableLabs:master.

Coverage remained the same at 96.242% when pulling 2ad517993b3eb4d3ae6db3c7eadd6f6aeb9c5a68 on khaninD:fixExamples into 196ea66981d1e173a0a1a23ac71301683b1abc71 on FormidableLabs:master.

@khaninD -- Thanks so much for noticing this and reaching out to fix it!
I'm actually going to close this PR in favor of https://github.com/FormidableLabs/radium/pull/939 which just refactors to real classes so we don't need to add another dependency and get to modernize our examples. If you get a chance, I'd love for you to try out 939 and make sure that works for you!


Sample app

A minimal sample app would be nice - one activity / fragment per module to demonstrate the usage.
We did this also for SSO and IMHO a practical code sample is more worth (and up to date) as any documentation.
For example one activity that has themed UI components, another which shows how logging works (depending of the actually implemented featues mentioned in https://github.com/nextcloud/android-common/issues/56 of course).

Oh yes, absolutely 🙂 Thanks


Add NetworkX 3 compatibility

Wondering if there's any fundamental reason to disallow networkx 3? Happy to make a PR if it's just a matter of ensuring everything still works with the new API

no nothing fundamental.. just differences in the api and some worry about forcing people to upgrade is just as bad as not upgrading, so then wanting to make it work with either is more work than we wanted to do.


option to retrieve image from stage.exportImage

currently it always triggers a download

done 5ca95ef6577da48a2b9aa40c968eb6e37e4f068a
stage.makeImage( {
    factor: 1,
    antialias: true,
    trim: false,
    transparent: false
} ).then( function( blob ){
    NGL.download( blob, "screenshot.png" );
} );

ping @colin-combe

So ngl can send image data to Python too, right?

So ngl can send image data to Python too, right?

yes

@arose - thanks for this


duplication in turn lightOn

https://github.com/mkobuolys/flutter-design-patterns/blob/66707f0cfbba37da360b3b470d833c17a0062a84/lib/design_patterns/facade/facades/smart_home_facade.dart#L56C1-L56C1
hi, first of all, I want to thank you  for your amazing repo,
I just noticed that in the referenced function (stopStreaming) you turned the lights On which is already on, based on your previous function (startStreaming), and is there any reason to add  turnLightsOn()  again?
thank you in advance for your clarification
void startStreaming(SmartHomeState smartHomeState) {
smartHomeState.lightsOn = smartHomeApi.turnLightsOn();
smartHomeState.tvOn = tvApi.turnOn();
gamingFacade.startStreaming(smartHomeState);
}
void stopStreaming(SmartHomeState smartHomeState) {
gamingFacade.stopStreaming(smartHomeState);
smartHomeState.tvOn = tvApi.turnOff();
smartHomeState.lightsOn = smartHomeApi.turnLightsOn();
}

Hey, nothing specific, just to ensure the state is correct after calling stopStreaming(). In the current example (using switches to turn the mode ON/OFF AND preventing switching other modes while one of them is ON), I agree that this is redundant and could be removed from the method. However, in the code (let's say, you are building your custom UI and using this exposed facade), nothing prevents you from calling the stopStreaming() without calling startStreaming() beforehand, thus sometimes it makes sense to ensure the final state that you want to get into.

Assuming that my previous answer makes sense, closing the issue.


Use local resource for lookup

Hi!
Thanks for the great component! It helped me a few time already, but I came across a new request and I'm unsure how to solve this one.
The problem is that I'm trying to implement this as a part of a tag/token selector, and all the possible tags/tokens are already available locally and there's no actual need for an ajax request, but the component only supports remote requests.
Could this component be extended to optionally support local resources?

Hi @blackfyre!
Please check the issue #25.

Wonderful!
Thanks!


oauth_providers_example.dart: example needs to be updated

The example's comments give wrong instructions regarding the configuration of NHost's OAuth callbacks:
At line 10 it reads:
/// Success redirect URL: `nhost-example://oauth.login.success`.
/// Failure redirect URL: `nhost-example://oauth.login.failure`.

wheareas only one callback URL can be set on GitHub settings and no URL can be set on NHost's GitHub provider configuration.
Also at line 23, the sign in URL is wrong. It should look like:
/// Fill in this value with the backend URL found on your Nhost project page.
const nhostGithubSignInUrl = '$nhostUrl/v1/auth/signin/provider/github';

There also seems to be a mismatch between the expected query parameter name (refresh_token) and the actual parameter name used by NHost callback (refreshToken)

Hi,
I have updated all examples ready on this PR https://github.com/nhost/nhost-dart/pull/95
We are activity working to improve the SDK. stay tuned.

Hi, thank you very much for your work!
Will this improvements include the migration from the old backend URL discussed in the link below?
https://github.com/nhost/nhost/discussions/1319

Yes, I am working on it. I think we will make it in the next major release two.

@diegonc  Check this out nhost_sdk: ^4.0.0-dev.6 or this link  for supporting subdomain and region.

Unfortunately, using launchUrl with Facebook would block me from authenticating because of Embedded browser security check from Facebook.
Found a workaround to authenticate with OAuth in Flutter though.

Use this package https://pub.dev/packages/oauth_webauth specifically the BaseWebView widget
add  initialUrl with '${createNhostServiceEndpoint(subdomain: YOUR_SUBDOMAIN, region: YOUR_REGION, service: 'auth')}/signin/provider/facebook'
add redirectUrls with your callback URL (https://xxxxxx.auth.YOUR_REGION.nhost.run/v1/signin/provider/facebook/callback)

After successful auth, should return with the callback?code=

because it refers to success/failure callback settings

that part is incorrect, there is no success/failure URLs in hasura-auth, instead, if there is an error, the callback url will include an error query arg describing the issue


WIP: Bump CAPV to v1.6.0

Bumping CAPV to latest upstream release.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

After building the new CAPV resources locally from this branch, I was able to:

Create a cluster with the new CAPV resources using a bundles override ✅
Create a cluster with main, then upgrade using a bundles-override to the new CAPV version ✅

/lgtm

/retest


file resource's inspect_existing_fs_entry prevents managing permissions on char/dev files

Description
It is desired to use the file resource to manage permissions on files we are not creating, ala:
file '/dev/foo' do
  mode '0644'
end

The inspect_existing_fs_entry check in the file provider prevents this saying that it's the wrong type and can't manage it. This makes sense if we're trying to write contents, but otherwise, it doesn't.
Chef Version
12.6.0
Platform Version
CentOS 6 and CentOS7
Replication Case
file '/dev/vcs1' do  # pick any char device here
  mode '0640'
end

Client Output
Chef::Exceptions::FileTypeMismatch: file[/dev/vcs] ((irb#1) line 1) had an error: Chef::Exceptions::FileTypeMismatch: File /dev/vcs exists, but is a char device, set force_unlink to true to remove

Ping?

piiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiing?

@jaymzh Please don't do that.

@coderanger or @lamont-granquist  - what would you like to be the solution here... this continues to bite us and depending on how you think it should be fixed I may be able to allocate resources to fix it.

@jaymzh I just meant to not ping on tickets, fixing the issue is cool but I have no input on that :)

Well the thing I'm vaguely worried about is if you start managing file perms on non-files with the file resource its a slippery slope to wanting to create them with mknod and extend the file resource with flags for device and character special files and that makes me lean towards wanting a different resource for it.

so can you fix the logic in a generic way so it literally read in the code like you explained it -- where those checks get bypassed when we don't have content and aren't creating a tempfile?
that does make sense to me and then i can just go 'lalalalalalalalala i can't hear you' about char devices and not worry about it...

@lamont-granquist  if there's no content, then what's the problem? I think the check should only come into play if someone specified content

this is related to #6704 I believe... and unless it was fixed recently, I think it's still broken.


I validated the model on aflw2000-3d database, get bigger NME.

the result is:
[ 0, 30]	Mean: 7.839, Std: 2.902
[30, 60]	Mean: 9.314, Std: 4.167
[60, 90]	Mean: 13.345, Std: 8.840
[ 0, 90]	Mean: 10.166, Std: 2.327
is bigger than the paper mentioned.

hi , could you tell me how to calculate 3D NME?I don't know when I calculate 3D NME,how can I get bbox.
bbox_size = torch.sqrt((right - left) * (bottom - top)) is  right?

hi , could you tell me how to calculate 3D NME?I don't know when I calculate 3D NME,how can I get bbox.
bbox_size = torch.sqrt((right - left) * (bottom - top)) is right?

i have the same question withyou, how to calculate NME?that makes me big head hhhh:)

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

ok! I'll try it! thanks a lot!Have a great day!!! :)

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

hi,I am sorry to bother you again,do you get a similar NME result( according to the link method you have sent to me) to the author's PRNet paper? I use  the predicted result of 3D model 68 points(from _kpt.txt of some AFLW2000 test images )and ground truth(from AFLW2000 .mat file of pt3d_68), are they correct data to calculate the NME? Really hope to receive your reply. thanks very much! :) My mail box is<EMAIL_ADDRESS>求大佬回复一下啊啊啊啊 救救孩子

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

hi,I am sorry to bother you again,do you get a similar NME result( according to the link method you have sent to me) to the author's PRNet paper? I use the predicted result of 3D model 68 points(from _kpt.txt of some AFLW2000 test images )and ground truth(from AFLW2000 .mat file of pt3d_68), are they correct data to calculate the NME? Really hope to receive your reply. thanks very much! :) My mail box is<EMAIL_ADDRESS>求大佬回复一下啊啊啊啊 救救孩子

我只比较的前两维的数据，没有加第三维

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

hi,I am sorry to bother you again,do you get a similar NME result( according to the link method you have sent to me) to the author's PRNet paper? I use the predicted result of 3D model 68 points(from _kpt.txt of some AFLW2000 test images )and ground truth(from AFLW2000 .mat file of pt3d_68), are they correct data to calculate the NME? Really hope to receive your reply. thanks very much! :) My mail box is<EMAIL_ADDRESS>求大佬回复一下啊啊啊啊 救救孩子

我只比较的前两维的数据，没有加第三维

大佬，您是咋比较计算的？ 我是进行人脸重建运行demo.py，从position map获取了三维数据点存进了.txt文件，然后和AFLW2000数据集的.mat文件的三维数据进行比较，但是不是很对，前两维也不对，得不出作者的结果，可以指点一下吗？大佬方便留下联系方式吗，啊啊啊这个困扰我好久了！万分感谢！

@Shiyu555 @Nancyh-e
HERE: https://github.com/cleardusk/3DDFA/blob/master/benchmark_aflw.py

hi,I am sorry to bother you again,do you get a similar NME result( according to the link method you have sent to me) to the author's PRNet paper? I use the predicted result of 3D model 68 points(from _kpt.txt of some AFLW2000 test images )and ground truth(from AFLW2000 .mat file of pt3d_68), are they correct data to calculate the NME? Really hope to receive your reply. thanks very much! :) My mail box is<EMAIL_ADDRESS>求大佬回复一下啊啊啊啊 救救孩子

I wonder what is _ kpt.txt ? The third dimension of position map presents the spatial information ，I calculate NME directly with the predicted position map and the real position map.

it is a txt file to put the 3D 68 points predict datas from position map(in demo.py can find it). and how to use the predicted position map and the real position map to calculate NME directly ? Did you get a similar result to author?thanks a lot, hope your relpy!!!

@lijie2160 @Nancyh-e
hii, could you please send the NMEevaluation code that you have written to me. I have tried it, but there are still errors. thank you
My mail box is<EMAIL_ADDRESS>


CORS paths wildcard and allowed_origins_patterns format

Hello.
I have two quick question.
When I use:
'paths' => ['api/*']
I don't receive the CORS headers.
but using:
'paths' => ['*']
work as expected.
Why?
EDIT: Maybe adding in the documentation you can use wildcard in allowed_origin?
That would have saved me some time!

@perfectdeed Please look at the readme, carefully.

Please test with v2 and check if you have any headers added on that routes. Are you using a prefix perhaps?

The docs were updated.


Pthread aborting at Error， worker.js onmessage() captured an uncaught exception: RuntimeError: abort(OOM). Build with -s ASSERTIONS=1 for more info.

Any resolution?


bug: ionic build error with google services and play services

Every time I add google-plus to my application this error happens. Not sure how to get rid of it currently. any solutions. We came to far to have small problems like these. Kind of new to the android build.
Task :app:processDebugGoogleServices FAILED Found play-services-identity:11.8.0,

Bug Report
Ionic version:

[x] 4.x
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info:

insert the output from ionic info here
`FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugGoogleServices'.
> Please fix the version conflict either by updating the version of the google-services plugin (information about the latest version is available at https://bintray.com/android/android-tools/com.google.gms.google-services/) or updating the version of com.google.android.gms to 11.6.2.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 11s

> Task :app:processDebugGoogleServices FAILED
Found play-services-identity:11.8.0, but version 11.6.2 is needed for the google-services plugin.
Found play-services-auth:11.8.0, but version 11.6.2 is needed for the google-services plugin.
20 actionable tasks: 20 executed
C:\platforms\android\gradlew: Command failed with exit code 1 Error output:
Note: C:\platforms\android\CordovaLib\src\org\apache\cordova\engine\SystemCookieManager.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugGoogleServices'.
> Please fix the version conflict either by updating the version of the google-services plugin (information about the latest version is available at https://bintray.com/android/android-tools/com.google.gms.google-services/) or updating the version of com.google.android.gms to 11.6.2.
`

Thanks for the issue. I am going to close this as this is not a bug in Ionic Framework. Ionic Framework does not manage the Cordova build process.
For further help with this issue I recommend you post on our forums or our slack. Thanks!


Questions about the visual inspection

I followed the v2.0 branch documentation for the g(10) certification. I'm using MSI and AAD.
I have received the proctor sheet from the Drummond group for the g(10) certification, in the proctor sheet they mentioned that for tests standalone patient API, limited access app, EHR APP, and single patient API after selection of the scopes user should enter user id and password.
If I use my Microsoft account userid/password for this then after the first test user gets cached in the browser and for the next tests it does not ask for userid/password. I'm not sure if we should Microsoft account userid and password for this or not
Here right now proxy only asks for userid. Please see the screenshot attached. Also, the first checkmark doesn't have any value. Why it is empty?

I also have questions about the visual inspection, they are expecting a demonstration about client registration for single and multiple patients. Any idea how can I create clients for single and multiple patients? Do I have to show them the client registration steps of standalone patient API and Multiple Patent API?
Also, the user should know about the offline_accesss scope. How can I achieve this?
I will really appreciate your help on this. My deadline is very near for the certification.

I followed the v2.0 branch documentation for the g(10) certification. I'm using MSI and AAD.
I have received the proctor sheet from the Drummond group for the g(10) certification, in the proctor sheet they mentioned that for tests standalone patient API, limited access app, EHR APP, and single patient API after selection of the scopes user should enter user id and password.
If I use my Microsoft account userid/password for this then after the first test user gets cached in the browser and for the next tests it does not ask for userid/password. I'm not sure if we should Microsoft account userid and password for this or not Here right now proxy only asks for userid. Please see the screenshot attached. Also, the first checkmark doesn't have any value. Why it is empty?
I also have questions about the visual inspection, they are expecting a demonstration about client registration for single and multiple patients. Any idea how can I create clients for single and multiple patients? Do I have to show them the client registration steps of standalone patient API and Multiple Patent API?
Also, the user should know about the offline_accesss scope. How can I achieve this? I will really appreciate your help on this. My deadline is very near for the certification.

You can use the steps in the oncg10 document they show registration of a standalone and multiple patient API
The scope session restrictions are captured with user name, then password is entered and MFA is performed (if configured) then full scoping consent will be presented if it is the first time the user accesses the application which will include all scopes (offline_access, openid etc...) that must be consented to.
The first empty check box is simply allows you to select/deselect all scopes
You can adjust the code to fit your needs this is simply a basic implementation and again is presented as is.


Broken requirements URL in README.md

The requirements URL is broken and leads to a GitHub 404.
I'm having troubles apt-installing the system dependencies and having access to this file would be useful.

Having Rust and then via apt libgtk-3-dev and pkg-config should be sufficient.

The build fails with Requested 'gdk-3.0 >= 3.24' but version of GDK is 3.22.30 unfortunately. I don't want to break my system by compiling the C libraries manually.

You're trying to compile a crate then that requires the v3_24 feature of gtk or gdk, but your system's version is too old. Either try to get it to not require such a new version of gtk, or you'll have to update it.

(this is a bit off-topic for this issue anyways)

Indeed, just trying to help you here :) I'm not sure where the requirements page disappeared to (@GuillaumeGomez ?) but it also wouldn't have helped you with this specific problem.

The requirements page was removed a while ago (you can see the follow-up PR here when we fixed the URL). Now we just link to the GTK installation page.

Oh thank you @sdroege for mentioning the feature, I was actually trying to compile an example and didn't realize that the gdk >= 3.24 version requirement was controlled by a feature. I thought that it was an absolute requirement. Regarding "(this is a bit off-topic for this issue anyways)", I simply didn't wan't to annoy anyone else with unrelated discussion, but I'm really thankful for the help.

@edgarogh Do you have a suggestion where to add this in the documentation? Where would you have looked to find this? :)


[Dashboard Screen] - Remove unnecessary digits when viewing widgets (Time Series and Tarcometro)

Having unnecessary digits (eg 86.00) in the graph only makes it difficult for the user to interpret.
It would be useful to provide a field to limit the digits in the Time Series.

Gauges documentation states that it is possible to “cut” leading zeros and limit decimals with the parameters data-value-int="1" data-value-dec="0" in the html or valueInt=1 valueDec =0 in javascript. So it would be easy to provide a field for the user to limit the decimal places.

Functionality apparently simple to do, but we don't understand why it hasn't been implemented to date.


Sphinx Style Docstring Rendering Feature

Environment data

Language Server version: 2022.1.1
OS and version: Windows 11
Python version: 3.10.0

Expected behaviour
Sphinx style docstrings should be shown correctly.
Actual behaviour
Sphinx style docstrings are not shown correctly, like:

Thanks for the report. Will investigate this.

I can confirm that the issue is still present. Here is another example, which illustrates that of two methods defined in the same file and for the same class, only one has its docstring shown correctly.
The docstring is displayed correctly:

The docstring is not shown (despite being present in the source code):

VS Code version info:

Version: 1.66.0 (user setup)
Commit: e18005f0f1b33c29e81d732535d8c0e47cafb0b5
Date: 2022-03-30T05:50:14.623Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

@OmerFI what are you expecting?
currently it shows the __init__ text

I can confirm that the issue is still present. Here is another example, which illustrates that of two methods defined in the same file and for the same class, only one has its docstring shown correctly.
The docstring is displayed correctly:
The docstring is not shown (despite being present in the source code):
VS Code version info:

this issue should be solved in the latest 2022.4.0 version.. issue with panda type stubs

@bschnurr
When I use Numpy style docstring, it is shown beautifully, like:

But, this is not the case with Sphinx style docstring:

@bschnurr I've checked the latest versions of both regular VS Code and VS Code Insiders. In either case, the docstring is not displayed correctly (or rather at all, only the function's signature).
Regular version:
Version: 1.66.1 (user setup)
Commit: 8dfae7a5cd50421d10cd99cb873990460525a898
Date: 2022-04-06T14:50:12.141Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Insider's build:
Version: 1.67.0-insider (system setup)
Commit: b84feecf9231d404a766e251f8a37c730089511b
Date: 2022-04-11T05:15:52.676Z
Electron: 17.4.0
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

And if you stretch it:

if you go to def on convert_dtypes does it take you NDFrame class's convert_dtypes with docstring?

@bschnurr
When I use Numpy style docstring, it is shown beautifully, like:

But, this is not the case with Sphinx style docstring:

We currently try to detect the docstring format and do minimal changes to them so that they sort of appear like they do in raw text. VSCode treats the doctring text as markdown. see<EMAIL_ADDRESS>so for this example with no fixups it would be shown like this

parser code here https://github.com/microsoft/pyright/blob/main/packages/pyright-internal/src/analyzer/docStringConversion.ts

R you requesting something like this?
This is a Sphinx Stype Docstring

:param a: variable 1
:type a: int
:return: a
:rtype: int

to
This is a Sphinx Stype Docstring
(a: int) -> int

R you requesting something like this?
 I tried to find a correlation with what class the method resolves to, but could not. For example, pd.DataFrame.cummin and pd.DataFrame.abs are both defined in pandas.core.generic, both have a docstrings displayed by built-in help(), but only abs shows a docstring on mouse over and "Trigger Suggest".
Pylance v2022.4.0
To be honest, I am not sure how the things with Pylance and stubs. Maybe the problem I complain about is not the same issue.

VS Code is natively just rendering the docstring content as markdown in the hover box.  Ideally Python hover would be rendered via Sphinx so it's nicely styled regardless of the docstring format.  This might be an expensive operation, so a "light" parser that handles each of the common docstring formats and renders HTML in the hover box would be great to have.

I write a lot of my docstrings as markdown (so that I can use pdoc to generate my documentation), and have noticed that there's quite a few things that behave extremely poorly. These issues seem to be very difficult to condense down into a minimal reproducible example, and have lots of very finicky edge cases surrounding them. If I can get some easier things to reproduce

Nested indentation is super unreliable. The parser can't seem to match levels of indentation, and half the time it needs me to indent by >6 spaces to increase the indentation level.
I can't use markdown tables within the documentation. Perhaps this is a limitation of VS Code though.

Here's an example of the indentation issue:
def myFunction():
    """
    * This is my first dot point. Its indentation level
      is correctly assigned to be zero.

    * Now this is another dot point. Once again, its
      indentation level is zero, which is correct.

      * But now I want to have an indented dot point. VS Code kept the
        indentation for this at zero even though it is indented.

    * This dot point just resets the indentation level.

        * Let's indent it even more. Notice how it still doesn't work, even
          though this is now very clearly on a deeper level of indentation.

    * This dot point just resets the indentation level.

          * And now finally, after six spaces, it does a deeper level of
            indentation. But notice how this line has all the indentation
            removed and is no-longer part of the dot point?

          * Now, even though the indentation level is still six spaces, VS Code
            treats it as level zero indentation.

    * When I go back to the first level of indentation, there is no difference.
      This absolutely butchers a lot of my complex documentation, and makes it
      borderline unreadable in VS Code.
    """

This will render as follows (split into multiple screenshots as I can't increase the size of the popup):

I would massively appreciate a fix on this at some point soon, since it really breaks a lot of the documentation I write, especially for highly technical code that requires detailed descriptions.

VS Code is natively just rendering the docstring content as markdown in the hover box. Ideally Python hover would be rendered via Sphinx so it's nicely styled regardless of the docstring format. This might be an expensive operation, so a "light" parser that handles each of the common docstring formats and renders HTML in the hover box would be great to have.

I will be happy if this feature will be coming in the next releases

VS Code is natively just rendering the docstring content as markdown in the hover box does this mean that my issue described above is actually an issue with VS Code itself? I might want to copy my comment there too.

We currently try to detect the docstring format and do minimal changes to them so that they sort of appear like they do in raw text. VSCode treats the doctring text as markdown. see

If that is the proposal, it would be good if the changes remain more minimal.
There are some docstring formatting using indentation that disappear with the markdown parser, which is not predictable.
@MiguelGuthridge showed an example, here is another simpler one:
def foo(a: str, b: float, c: int, d: list[float], e: tuple[str]):
    """Test

    My function

    Parameters
    ----------
    a : str
        Et quam quidem sit distinctio aliquam deserunt occaecati voluptates. Laudantium nulla
        ea officia. Voluptatum rerum expedita ipsam dolor quia.
    b : float
        Molestiae aliquid libero. Dolores voluptate eveniet. Voluptas expedita rem doloribus
        possimus perferendis non molestiae.
    c : int
        Voluptas officia autem maiores temporibus nihil ea. Molestiae est doloribus ducimus
        modi quisquam. Velit quasi aspernatur aut dolor. Voluptas rerum aut vel debitis. Omnis
        magnam nobis consequatur mollitia. Nemo ea debitis animi et saepe dicta ut. Hic fugiat
        aut facere. Itaque ea eos deleniti asperiores neque quia. Molestiae architecto sed
        rerum sequi suscipit. Omnis sunt ut voluptatum esse. Consequuntur eaque consequuntur.
        Velit et aut ea sit. Quos mollitia omnis incidunt doloribus beatae aspernatur
        exercitationem consequatur est. Voluptates asperiores itaque. Excepturi nam omnis.
        Doloribus reiciendis voluptatem. Blanditiis eos earum cupiditate autem qui nihil ex
        omnis. Enim alias exercitationem et fugit tempora voluptates dolores vel.
    d : list[float]
        Dolores quia mollitia blanditiis incidunt ab sunt non rerum. Aperiam sed neque ut quis
        excepturi voluptas quibusdam odio voluptates. Harum ea necessitatibus quibusdam omnis
        explicabo placeat. Eum qui deserunt. Quis nihil iure dicta id vero doloribus.
    e : tuple[str]
        Quo fugit rerum et ipsum et repellat aut. Voluptas provident autem praesentium et
        molestiae. Nulla et atque voluptatem ut.
    """

Another example is the use of sphinx directives.
They just are not show at all.
def foo(a: str, b: float):
    """Test

    My function

    .. note::
        This is my note

    Parameters
    ----------
    a : str
        Et quam quidem sit distinctio aliquam deserunt occaecati voluptates. Laudantium nulla
        ea officia. Voluptatum rerum expedita ipsam dolor quia.
    b : float
        Molestiae aliquid libero. Dolores voluptate eveniet. Voluptas expedita rem doloribus
        possimus perferendis non molestiae.
    """

I am annoyed by the fact this never actually supported the basic syntax for **bold text** or *italic text*. Both * and _ should be supported.

Raw docstring content:
Default YAWNS configuration. All security constants (like keys) are undefined.

All keys must be prefixed with `YAWNS_` because this is added to the Sanic
object's configuration, and it can conflict with some configuration of the
Sanic object.

**DO NOT EDIT THIS FILE**, because it *will* be overwritten on upgrade.

make sure doc string builder can handle case listed here - https://github.com/microsoft/pylance-release/issues/5431 - as well.
https://github.com/apache/airflow/blame/07fe1d2a69cbe4f684a1989c047737c0686c4417/airflow/models/dag.py#L302

This should be handled when we finish https://github.com/microsoft/pylance-release/issues/5363. Sphinx docstrings should at least be supported.

Here's some examples of the coming changes.
PySimpleGui (which I believe the first screenshot was based on):

Tables

Emphasis problems:

Airflow (the example @heejaechang linked to)

I wish we could output html in the markdown but VS code is very limited there, so mostly we're just using indents/italics/bolding to highligh different sections.
For example that same function in the actual generated Airflow documentation looks like so:

This issue has been fixed in prerelease version 2024.6.100, which we've just released. You can find the changelog here: CHANGELOG.md

The fix for this issue is behind an experimental feature flag:
"python.analysis.supportRestructuredText": true

If you want to try out our new restructuredText support, enable this flag. It's behind a flag at the moment until we can make sure it handles all the possible Sphinx/GoogleDoc/Epytext scenarios that customers need. Please log additional issues if this setting isn't working out for you.
Thanks :)

The fix for this issue is behind an experimental feature flag:
"python.analysis.supportRestructuredText": true

If you want to try out our new restructuredText support, enable this flag. It's behind a flag at the moment until we can make sure it handles all the possible Sphinx/GoogleDoc/Epytext scenarios that customers need. Please log additional issues if this setting isn't working out for you.
Thanks :)

Is not working for me.
Trying the same lib (PySimpleGUI):

@Diogo-Rossi You should switch to the pre-release version of Pylance, it's working for me.

Thanks @rchiodo for all your effort :)

@Diogo-Rossi You should switch to the pre-release version of Pylance, it's working for me.

Thanks @rchiodo for all your effort :)

Right! I missed the prerelease detail. I  thought that 2024.6.1 was the version mentioned.
It's working! Thank you @rchiodo for the feature!


Move personal search preferences and match to openfoodfacts-dart #140 (uses new openfoodfacts-dart module, ready for review)

This is a continuation of @monsieurtanuki 's work for #140
This code works with an updated openfoodfacts-dart package, corresponding to this PR: https://github.com/openfoodfacts/openfoodfacts-dart/pull/138
It seems to be working fine.

@stephanegigandet Oh, I assumed it would be a 3-step process:

a reviewed and approved OFF-dart lib (I'll focus on that for the moment)
publish OFF-dart
remove the corresponding classes from smoothie and use the OFF-dart version

@stephanegigandet Oh, I assumed it would be a 3-step process:

a reviewed and approved OFF-dart lib (I'll focus on that for the moment)
publish OFF-dart
remove the corresponding classes from smoothie and use the OFF-dart version

Yes, I won't merge the smoothie PR until 1. and 2. are done. But I wanted to make sure that it worked, that's why I also tried the modified openfoodfacts-dart package with smoothie.

@monsieurtanuki : I have updated the PR to make it work with the new openfoodfacts-dart module.
Regarding the classes that we could remove, I don't think they are related to personal search, so it's probably to treat them separately in another PR. I think most of them were created by @PrimaelQuemerais , I do not now which ones have been replaced by new classes, and which ones are other features that we may need in the future.

Minor comments for a very smooth code review ;)
Please take 10 minutes about smooth_product_sneak_peek_view.dart: if we could get rid of that class and the related ones it would be good news. Or at least understand their purpose.

This class was created to display a custom view when long pressing a product card. It wasn't kept as a feature so you should be fine removing it

Thank you @PrimaelQuemerais

This class was created to display a custom view when long pressing a product card. It wasn't kept as a feature so you should be fine removing it

Even if we don't display in a long press, we may still want to display this information (how much the product contributes to the daily recommendent amounts) in the "Track" mode of Smoothie when we implement it though.


Use translations and a custom icon for the roles

It should be possible to give a role a real name and change the icon.

I've adjusted the Policy.yaml because my previous change could lead to problems.


Feature request: Synchro Shift notification

Garmin's implementation of Di2 support shows an on screen notification and beeps when upon the next shift Synchro Shift will shift the front derailleur either up or down. It would be amazing if Ki2 could do the same.
This is a similar feature to #70 but is only relevant when in Synchro Shift Mode 2.
Additionally, this feature requires being able to obtain the front derailleur shift points from the Di2 system. Since this can, in theory, be changed on the fly using the iOS/Android App this probably needs to be regularly updated or requested when it is required depending on latency.

Working on this for next update. As far as I can tell we don't need the synchro shift configuration, the shifting system will tell when the upcoming shift will happen on it's own.

That's fantastic to hear! Thank you!

Was this also closed by #76?

I believe so, still need to go out to test it before releasing. Also only the is only beeping (audio), there is no on screen (visual) indicator.

I believe so, still need to go out to test it before releasing. Also there is only beeping (audio), there is no on screen (visual) indicator for now.

Fantastic, very excited to try out the self-updater when this is released. The lack of visual indicator is probably the less useful indicator anyway (though I do find that the Karoo 2's beeper could be louder). Though, perhaps an option to add in the future. The preferences/settings might start to get complicated!

This is now available in release 0.9. The app should show a notification for auto-update or you can trigger an update via the settings.
Please give it a test when you go for a ride 🙂


python311Packages.pysolcast: relax responses

Description of changes
Fix build (https://hydra.nixos.org/build/246216115)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 282417 run on x86_64-linux 1

4 packages built:

python311Packages.pysolcast
python311Packages.pysolcast.dist
python312Packages.pysolcast
python312Packages.pysolcast.dist


Planned removal of crucial DOM events

Hello, lighthouse detects this library as depracated due to planned removal of some DOMevents:
DOM Mutation Events, including DOMSubtreeModified, DOMNodeInserted, DOMNodeRemoved, DOMNodeRemovedFromDocument, DOMNodeInsertedIntoDocument, and DOMCharacterDataModified are deprecated (https://w3c.github.io/uievents/#legacy-event-types) and will be removed. Please use MutationObserver instead.
The library is great, are you planning to migrate to MutationObservers instead? I think it would be worth giving it a try.

Hey @partiid!
Could you elaborate a bit more about the problem?
In this library none of DOM* events are used. Also - MutationObserver shouldn't be needed in the implementation either.

Hello, sorry for not replying. The screen above says it all. If noone of them are used, why lighthouse detects it as deprecated API?


Unreviewed commit on master!

Unreviewed commit(s) COMPARE was merged to master by grahamjenson.
Comment with appropriate number of :+1: to approve.

Reviews
0/1

More info

👍

Reviews
1/1

ClareCat
1

Pull request is reviewed and can be merged to master!
More info


Parameter NO_PARAM in request is invalid when creating FS backup policy through bicep

Please refer the feedback provided here: https://learn.microsoft.com/en-us/answers/questions/1287940/
and add a note or instruction to avoid the confusion in all the appropriate Azure Backup documentation. Thanks.

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

ID: 245cba1c-3685-79bb-15a3-a238fb6b627f
Version Independent ID: 441e44e7-5c77-0a80-797e-99398c665ba5
Content: Manage Azure file share backups with the Azure CLI - Azure Backup
Content Source: articles/backup/manage-afs-backup-cli.md
Service: backup
GitHub Login: @jyothisuri
Microsoft Alias: jsuri

Assigning to content author @jyothisuri for visibility, review and document enhancement as appropriate.

#reassign: @AbhishekMallick-MS

@KrishnaG-MSFT - as confirmed by the Product team, we've a template published for Azure Files backup configuration that can be used as reference.. Please check this article.
As there;re no doc updates needed, we're closing this thread. Thanks.
#please-close


Document supported table options

ping @adduc
Anything missing or can we merge?

I think so, although I'm not sure if there's been any changes to supported table options since 0.6.

I see. But even with it being not fully complete this would already be an improvement here.

@lorenzo Can we already merge this partial docs upgrade?


Creat a background

Steps

Setup: GitHub

[x] created this issue
[x] assigned myself
[x] moved the issue to the project board and added it to the appropriate
column

Setup: local

[x] pulled the most recent code from main/master
[x] created a new background branch for my feature

Steps to complete the task:

Before pushing my branch:

[x] pulled the most recent code from main/master to my feature branch
[x] ran code quality checks
[x] formatted the code
[x] fixed all linting mistakes

After pushing my branch:

[x] created a new PR linked to this issue
[x] make sure all of the checks passed
[x] assigned someone to review and merge my PR (if the task is ready)
[x] moved the issue to the appropriate column in the project board to review
it (if the task is ready)

all done


Assistance

@valzav I'd like to help if you are interested. My skillsets include:

Managing milestones
Prioritizing issues
Working with ZenHub to bring pipelines and order to github issues
UX mockups

I am comfortable with long distance collaboration. I use screenhero and slack for quick screenshares/chat to get in sync with remote volunteers. I'm interested and motivated to make time for this if you are in a position to let me help.

@wmbutler, yes this would be absolutely great to have you helping us to maintain this repo and maybe graphene toolkit's one and play a release manager role. I can send you invitation to dev slack channel to discuss more details.


Update shipping.py

More detail

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

Shipping is the vital process of moving goods from one place to another, involving logistics, packaging, and transportation. It ensures timely, cost-effective delivery and is essential for global commerce. If you're looking for reliable shipping services, consider [https://cheapmotorcycleshipping.com/].


Support maxLength in JSON Schema

Is your feature request related to a problem? Please describe.
My Redshift warehouse uses this tap, and is experiencing disk size bloat. It appears to be much bigger than I'd expect (2 orders of magnitude). I suspect this is due to target-redshift creating all columns as varchar(10000), despite my source data (Postgresql) having mostly varchar(256).
Redshift docs advise to use the smallest possible column size for data (see here and here).
Describe the solution you'd like
If the input stream supplies a "maxLength" property for a string field, use that as the size of the field created. This behaviour could be turned on via a config setting.
Describe alternatives you've considered
Not sure. Possibly running a bunch of "alter table" queries after ingestion.
Additional context
I would like to use this in conjunction with a tap-postgresql change described at https://github.com/MeltanoLabs/tap-postgres/issues/477

First raised at https://meltano.slack.com/archives/C069CQNHDNF/p1722991360991679

@techtangents you are welcome to open a pr for this issue!


Investigate possiblity of a operator button that can turn a driver cam feed in Elastic on and off for intaking.  Not sure it's possible tho.

ive looked into this, and the chances arent looking good to me, I think that the setDriverMode() method might be able to do this, we could probably tie it to a button. Problem is that no matter what I cannot actually get a camera feed in elastic, that needs to be solved before setDriverMode() can be tested

Liam, take a look at this WPILib page and see if we can use it's method to publish the feed from the NoteCam OrangePi to Elastic.
https://docs.wpilib.org/en/stable/docs/software/vision-processing/introduction/cameraserver-class.html

I also think these pages will be useful. https://docs.wpilib.org/en/stable/docs/software/dashboards/shuffleboard/layouts-with-code/index.html
Since we will want to add and possibly remove the camera widget to reduce the bandwidth

Elastic by default does not populate widgets (unlike shuffle board which does). You can add a camera stream to elastic from the CameraPublisher NT4 topic. Here is an example from simulation.

Liam can you try what Jon suggests above?  That would be a start to at least getting the streams on Elastic.  We can deal with the next step of turning it on and off after you get it streaming.

Ok so I poked around a bit and its actually pretty simple. We just need to change the streams [] NT4 field.

So we should create our own camera publisher that is pointed at a bogus URL possibly using the SendableCameraWrapper
then create a NT4 subscriber to get the stream url from the PhotonCamera we want
then use a NT4 publisher to overwrite the streams [] field of the SendableCameraWrapper.

Example:
NetworkTableInstance.getDefault()
.getTable("CameraPublisher/camera1")
.getStringArrayTopic("streams")
.publish();

@Marbledata can we test Jon's hypothesis above on Thur?  Looks promising.

Will work on it today
Get Outlook for iOShttps://aka.ms/o0ukef

From: char @.>
Sent: Thursday, March 14, 2024 3:22:37 PM
To: Team3176/Code_2024 @.>
Cc: Marbledata @.>; Mention @.>
Subject: Re: [Team3176/Code_2024] Investigate possiblity of a operator button that can turn a driver cam feed in Elastic on and off for intaking. Not sure it's possible tho. (Issue #36)
@Marbledatahttps://github.com/Marbledata can we test Jon's hypothesis above on Thur? Looks promising.
—
Reply to this email directly, view it on GitHubhttps://github.com/Team3176/Code_2024/issues/36#issuecomment-1998245114, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3NCU3KX7I4X2O2XOR3UTYDYYH2H3AVCNFSM6AAAAABENZIWSKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJYGI2DKMJRGQ.
You are receiving this because you were mentioned.Message ID: @.***>

I know that I need to use a NT4 publisher to overwrite the streams value of the sendablecamerawrapper, I just have no idea how to do that. even if i get a publisher, I dont know how to get it to change the streams value

Besides the bad choice of putting the switching button on transStick button 1, this looks ok.
@Marbledata can you take a control board with an attached photonvision unit and camera, verify functionality and then measure and report here the max perfmon values, please.


Is there a problem with WebChat channel - it's VERY slow

Hi @arieschwartzman, we are actively resolving a service interruption in web chat.

Hi @arieschwartzman, are you still experiencing the problem?


Cow: Watch internxt

Cow requests the watch of the watch_keyword internxt. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

internxt has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


sieve-of-eratosthenes algorithm

the Language used : C/C++

can you please add the label as hacktoberfest-accepted

can you please add the label as hacktoberfest-accepted

It is merged already so label is not required.


How to work on WebGL

It didn't work on WebGL. Is there any things I can do to solve the problem,such as a config file or a path rules?Thank you so much.

it should be work on WebGL. plz make sure there is no Cross-origin resource sharing problem.

it should be work on WebGL. plz make sure there is no Cross-origin resource sharing problem.