Can't install fenics with conda or mamba

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Recently, when trying to install fenics with
conda install -c conda-forge fenics

or
mamba install -c conda-forge fenics

We get the following error:
2023-09-26 11:56:16.351Z: #6 0.701 Traceback (most recent call last):
#6 0.701   File "/opt/conda/bin/mamba", line 7, in <module>
#6 0.701     from mamba.mamba import main
#6 0.701   File "/opt/conda/lib/python3.11/site-packages/mamba/mamba.py", line 49, in <module>
#6 0.702     import libmambapy as api
#6 0.702   File "/opt/conda/lib/python3.11/site-packages/libmambapy/__init__.py", line 7, in <module>
#6 0.702     raise e
#6 0.702   File "/opt/conda/lib/python3.11/site-packages/libmambapy/__init__.py", line 4, in <module>
#6 0.702     from libmambapy.bindings import *  # noqa: F401,F403
#6 0.702     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
#6 0.702 ImportError: libarchive.so.13: cannot open shared object file: No such file or directory

Our Dockerfile is:
FROM mcr.microsoft.com/vscode/devcontainers/miniconda:0-3

RUN conda install -c conda-forge mamba
RUN mamba install -c conda-forge fenics  # <-- fails here

RUN apt-get update && apt-get install -y libgl1

RUN pip install festim==0.11.1 matplotlib meshio[all] ipykernel h-transport-materials==0.12.7 numpy==1.24

Installed packages
# packages in environment at /opt/conda:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                        main
_openmp_mutex             5.1                       1_gnu
boltons                   23.0.0          py311h06a4308_0
brotlipy                  0.7.0           py311h5eee18b_1002
bzip2                     1.0.8                h7b6447c_0
c-ares                    1.19.0               h5eee18b_0
ca-certificates           2023.08.22           h06a4308_0
certifi                   2023.7.22       py311h06a4308_0
cffi                      1.15.1          py311h5eee18b_3
charset-normalizer        2.0.4              pyhd3eb1b0_0
conda                     23.7.4          py311h06a4308_0
conda-content-trust       0.2.0           py311h06a4308_0
conda-libmamba-solver     23.5.0          py311h06a4308_0
conda-package-handling    2.1.0           py311h06a4308_0
conda-package-streaming   0.8.0           py311h06a4308_0
cryptography              41.0.3          py311hdda0065_0
fmt                       9.1.0                hdb19cb5_0
icu                       58.2                 he6710b0_3
idna                      3.4             py311h06a4308_0
jsonpatch                 1.32               pyhd3eb1b0_0
jsonpointer               2.1                pyhd3eb1b0_0
krb5                      1.20.1               h143b758_1
ld_impl_linux-64          2.38                 h1181459_1
libarchive                3.6.2                h6ac8c49_2
libcurl                   8.1.1                h251f7ec_1
libedit                   3.1.20221030         h5eee18b_0
libev                     4.33                 h7f8727e_1
libffi                    3.4.4                h6a678d5_0
libgcc-ng                 11.2.0               h1234567_1
libgomp                   11.2.0               h1234567_1
libmamba                  1.4.1                h2dafd23_1
libmambapy                1.4.1           py311h2dafd23_1
libnghttp2                1.52.0               h2d74bed_1
libsolv                   0.7.22               he621ea3_0
libssh2                   1.10.0               hdbd6064_2
libstdcxx-ng              11.2.0               h1234567_1
libuuid                   1.41.5               h5eee18b_0
libxml2                   2.10.3               hcbfbd50_0
lz4-c                     1.9.4                h6a678d5_0
ncurses                   6.4                  h6a678d5_0
openssl                   3.0.10               h7f8727e_2
packaging                 23.0            py311h06a4308_0
pcre2                     10.37                he7ceb23_1
pip                       23.1.2          py311h06a4308_0
pluggy                    1.0.0           py311h06a4308_1
pybind11-abi              4                    hd3eb1b0_1
pycosat                   0.6.4           py311h5eee18b_0
pycparser                 2.21               pyhd3eb1b0_0
pyopenssl                 23.2.0          py311h06a4308_0
pysocks                   1.7.1           py311h06a4308_0
python                    3.11.4               h955ad1f_0
readline                  8.2                  h5eee18b_0
reproc                    14.2.4               h295c915_1
reproc-cpp                14.2.4               h295c915_1
requests                  2.31.0          py311h06a4308_0
ruamel.yaml               0.17.21         py311h5eee18b_0
setuptools                67.8.0          py311h06a4308_0
six                       1.16.0             pyhd3eb1b0_1
sqlite                    3.41.2               h5eee18b_0
tk                        8.6.12               h1ccaba5_0
toolz                     0.12.0          py311h06a4308_0
tqdm                      4.65.0          py311h92b7b1e_0
tzdata                    2023c                h04d1e81_0
urllib3                   1.26.16         py311h06a4308_0
wheel                     0.38.4          py311h06a4308_0
xz                        5.4.2                h5eee18b_0
yaml-cpp                  0.7.0                h295c915_1
zlib                      1.2.13               h5eee18b_0
zstandard                 0.19.0          py311h5eee18b_0
zstd                      1.5.5                hc292b87_0

Environment info
active environment : None
            shell level : 0
       user config file : /home/vscode/.condarc
 populated config files :
          conda version : 23.7.4
    conda-build version : not installed
         python version : 3.11.4.final.0
       virtual packages : __archspec=1=x86_64
                          __glibc=2.31=0
                          __linux=5.15.0=0
                          __unix=0=0
       base environment : /opt/conda  (writable)
      conda av data dir : /opt/conda/etc/conda
  conda av metadata url : None
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /opt/conda/pkgs
                          /home/vscode/.conda/pkgs
       envs directories : /opt/conda/envs
                          /home/vscode/.conda/envs
               platform : linux-64
             user-agent : conda/23.7.4 requests/2.31.0 CPython/3.11.4 Linux/5.15.0-1041-azure debian/11 glibc/2.31
                UID:GID : 1000:1000
             netrc file : None
           offline mode : False

@jhdark

This is an error initializing mamba, so definitely a failed install of mamba and/or conda (specifically, mamba was installed but a compatible libarchive dependency is not installed). So it's the conda install mamba line that fails, but the second line is where the error occurs. I imagine you'd see this same error with mamba install <anything>.
This dockerfile also fails with the same error:
FROM mcr.microsoft.com/vscode/devcontainers/miniconda:0-3

RUN conda install -c conda-forge mamba
RUN mamba install -c conda-forge mamba # should be a no-op

My guess is it's the mixing of conda-forge and defaults channels that results from installing a single package form conda-forge when everything else has come from defaults. Try adding conda upgrade -c conda-forge --all before the conda install mamba step.

Hey @minrk , I tried adding the conda upgrade -c conda-forge --all but doesn't seem to fix the issue. However, there is an error that comes after the RUN conda install -c conda-forge mamba line which is:
Error while loading conda entry point: conda-libmamba-solver (libarchive.so.13: cannot open shared object file: No such file or directory)
I tried installing this package manually with RUN conda install conda-libmamba-solver, but this didn't help either. Not sure if this helps narrow down the issue. I understand this has nothing to do with fenics, but if you have any other suggestions to try that would be great

Sorry, you need --strict-channel-prioirity to trade out all of the defaults packages for conda-forge ones:
FROM mcr.microsoft.com/vscode/devcontainers/miniconda:0-3

RUN conda upgrade --strict-channel-priority -c conda-forge --all
RUN conda install -c conda-forge conda mamba
RUN mamba install -c conda-forge fenics

But even that doesn't result in a working mamba, which I don't really understand, since it's now failing with:
0.548       File "/opt/conda/lib/python3.11/site-packages/mamba/mamba.py", line 877, in _wrapped_main
0.548         configure_parser_repoquery(p._subparsers._group_actions[0])
0.548       File "/opt/conda/lib/python3.11/site-packages/mamba/mamba.py", line 805, in configure_parser_repoquery
0.548         p = sub_parsers.add_parser(
0.548             ^^^^^^^^^^^^^^^^^^^^^^^
0.548       File "/opt/conda/lib/python3.11/argparse.py", line 1192, in add_parser
0.548         raise ArgumentError(self, _('conflicting subparser: %s') % name)
0.548     argparse.ArgumentError: argument COMMAND: conflicting subparser: repoquery

which I think means that conda and/or mamba have not actually been installed properly.
Downgrading conda to 22.x resolves the issue, I don't understand why, because when I install the same versions that don't work in an isolated env, I can't reproduce the problem. So you can pin that for now:
FROM mcr.microsoft.com/vscode/devcontainers/miniconda:0-3

RUN conda upgrade --strict-channel-priority -c conda-forge --all
RUN conda install -c conda-forge conda==22.* mamba
RUN mamba install -c conda-forge fenics

Essentially, what this is doing is removing all conda packages and replacing them with ones from conda-forge, so you might be better off using a lighter base image and installing miniforge, since that should avoid having the unused files from the original  miniconda install (which is all of them) in the image layer history.

fwiw, I found that it's a very recent regression in conda-libmamba-solver: https://github.com/conda/conda-libmamba-solver/issues/303, so that's what should be pinned back, not conda itself:
FROM mcr.microsoft.com/vscode/devcontainers/miniconda:0-3

RUN conda upgrade --strict-channel-priority -c conda-forge --all
RUN conda install -c conda-forge 'conda-libmamba-solver<23.9' mamba
RUN mamba install -y -c conda-forge fenics

Ok this is brilliant, its works! thank you so much

@minrk there is some more explanation here
https://github.com/conda/conda/issues/13148#issuecomment-1735930256


Fixed Duplicate code - 1

Solving #1262
Part - 1
I have solved the first using closure and a function. Please give me feedback so that I can finish the other parts.
Thankyou !

This PR has had not activity for more than a month. Can we close it?

This PR has had no activity for more than a month. Can we close it?

This PR has had no activity for more than a month. Can we close it?

@mihir1739 If you plan to get this merged in reasonable time, please let us know.

I'm closing this PR due to the lack of activity. @mihir1739, feel free to open it again in case you want to continue working.


GRID should probably have an audio interface built into itself.

Löve2D's audio interface does lack in certain departments.
Features that probably should be in the interface:

Volume based on distance?
Sound effects - underwater, etc. (This is not a required, but still useful feature).
Equalization, Bass boost, etc?

(This is just an idea, not a requirement!)

You're right. This was planned but I didn't stub it out yet.

It's funny, because we give an example for entity:emitSound() on the front page!

Dismissing this for now.


Pode and letsencrypt

How to bind a letsencrypt certificate in Pode?
I got a certificate using Posh-ACME, but I can't associate it with Pode. If possible, give an example.
OS: Windows 10
Posh-ACME: v3.12.0
Pode: 1.6.1

Hi @soulruins,
Would you be able to give some examples of what you've tried? The Add-PodeEndpoint and the New-PACertificate calls mostly.
From looking at their docs, New-PACertificate seems to create a cert.pfx and  fullchain.pfx - both with a default password of poshacme (unless you specified -PfxPass).
You should be able to use either of those files:
Add-PodeEndpoint -Address test.example.com -Port 443 -Protocol Https -CertificateFile ./fullchain.pfx -CertificatePassword poshacme

Hi @soulruins,
Would you be able to give some examples of what you've tried? The Add-PodeEndpoint and the New-PACertificate calls mostly.
From looking at their docs, New-PACertificate seems to create a cert.pfx and  fullchain.pfx - both with a default password of poshacme (unless you specified -PfxPass).
You should be able to use either of those files:
Add-PodeEndpoint -Address test.example.com -Port 443 -Protocol Https -CertificateFile ./fullchain.pfx -CertificatePassword poshacme

Oh!!! Many, many thanks! It's working. I didn't study the documentation well enough. Thanks!


Finetuning on KITTI

Hi,
I am impressed by the outcome of your work on multiple datasets. However, I wonder if you can offer some training details about finetuning on KITTI to achive 72.8% on test set since the amount of data on KITTI is highly limited. Sincere thanks.

To further explain my confusion, I wonder if you can explain the training details such as
1.Whether you pretrian the dataset on Cityscape or Mapillary
2.During training on KITTI, I saw the original paper says you only trian 90 epochs. Is the number referring to 90 epochs for KITTI because the iteration times would be quite few
Looking foward to your reply.

Sure, thank you for your interest in our work.

Yes, we pretrain our model on Cityscapes. We found it is very effective, because both datasets are captured in Germany cities and the data distribution maybe similar.

Yes, the 90 epochs is for KITTI, so the fine-tuning procedure is very fast. As we said in the paper (and also in the code), we use 90/10 as train/val split to fine-tune on KITTI. As I vaguely recall, our best model on KITTI is the 66th epoch model trained on val_2 split.

Thank you for your prompt reply , it helps a lot. But still I have a few more questions :
1.Since you didn't publish the result on KITTI with baseline, I wonder if the biggest reason your model showed such a fine performance is that it has an extremely high baseline( and it surely drops a little on KITTI due to the distribution difference)

I wonder if you employ exactly the same video propogation and label relaxation on KITTI like what you did on Cityscape

3.I'm curretnly working on the problem of having nice generalization performance on small datasets(exactly like KITTI) , so any advice from your previous experience or general idea would be greatly helpful
Again genuinely thank you for the reply

And last one , you said the model performed best at the 66th epoch trained on val_2 split , what do you mean by that? How can you tell it would be the best one?
Thank you.

Yes, I'm sure my baseline is high. But due to the submission policy of KITTI benchmark, I can only submit once for one publication. So I didn't try my baseline on KITTI. You can try and submit it on yourself.

I didn't use video propagation for KITTI because of two reasons. One is KITTI is too small, there is no diversity, so no need to propagate. Second reason is although KITTI has raw videos, but we don't know the corresponding video frames. So there is no way (at least to my understanding) to easily use video information. I used label relaxation, just as I did to Cityscapes.

From my side, I think two things. (1) Try some self-supervised or weakly-supervised pre-training on large driving dataset. The more data you see, the more robust your model is. (2) Try not to overfit.

I test many models on the whole training dataset to see which model has the highest mIoU. Because KITTI has only 200 images, it's pretty fast to do this kind of evaluation to pick the "best" one. But it may not be the best due to train/test distribution difference. It just seems to perform the best to me.

Hope this helps.

Thank you again for the generous help!

I wonder if you still have the configuration script for finetuning the model on KITTI because I tried many of them and the miou always dramatically drop (compared with Cityscape).

That' s really helpful . Just out of curiosity, if the training Iou is around 65 ,how come it reaches 72.82 at the final test set ?

Two things:

The training IoU is single scale score, while the final submission uses multi-scale sliding evaluation strategy.
Train and test set distributions are different. For example, our Cityscapes model achieves 82+ on standard validation set, but achieves 83+ on test set. So it's fine, as long as the validation score and test score are close.

@kwea123 @bryanyzhu  Please let me know if the issue has resolved.

@karansapra I think the issue has been resolved. You can close it for now. They can reopen it if something comes up. Thanks.

Sure, thank you for your interest in our work.

Yes, we pretrain our model on Cityscapes. We found it is very effective, because both datasets are captured in Germany cities and the data distribution maybe similar.
Yes, the 90 epochs is for KITTI, so the fine-tuning procedure is very fast. As we said in the paper (and also in the code), we use 90/10 as train/val split to fine-tune on KITTI. As I vaguely recall, our best model on KITTI is the 66th epoch model trained on val_2 split.

Hello @bryanyzhu ,
I am trying to finetune this code on KITTI dataset, however while dumping the results i could observe that it dumps original segmentation mask(binary). At the time of cityscapes it dumps colorize mask, but it is not working that way for KITTI dataset. I used the same kind of data loader as cityscapes. Can you please suggest me what could be the issue??
Thanks in advance.


MSRV Change

I've been getting CI failures when building my magnus-dependent library on Rust 1.61. It looks like the regex crate bumped their MSRV to 1.65 (https://github.com/rust-lang/regex/commit/8e13494bc898db42c64ef6a750203b1e6ce47214). Since regex is a dependency of rb-sys, building magnus on Rust 1.61 seems to fail now.

Magnus doesn't use any features from after Rust 1.61, and while the way the dependency versions are specified allow for using newer versions of those dependencies, those newer versions are not required. You can set a requirement on the version of the regex crate pinning to an older version to get things building on Rust 1.61 like so:
[dependencies]
magnus = "0.6"
regex = "= 1.9.6"

I can only really promise the Minimum Supported Rust Version is valid with the versions of the dependencies at the time of release for a specific version of Magnus. You should always be able to check those by looking at the Cargo.lock at the tag of the release, e.g. https://github.com/matsadler/magnus/blob/0.6.2/Cargo.lock
This is guaranteed as the tests on CI will aways be run against the MSRV (as well as the latest version of Rust), with the version of the dependencies in the Cargo.lock (I also run the tests against the MSRV when developing locally, but that's slightly less of a guarantee as sometimes I forget).


Configure fips on the bastion node of openshift cluster

To configure the bastion node of openshfit cluster with FIPS enablement on PowerVM.

As a part of fips enablement we have already enabled fips on master nodes and worker nodes, In this PR we are going to enable fips on the bastion node.

@yussufsh :
Please review the PR.

@yussufsh : Can you please review the PR.

@gauravpbankar  There are syntax errors in the code. When you are making any further changes I want to see a full run log with the required variables.
Please assign PRs only when you have a working code otherwise it is a waste of time for reviewers.
@prb112

Hey @Gaurav / @yussufsh yes - I totally agree. please ping the log to Yussuf and I

I am going to close this PR, if needed please raise a new one.

still needed, new PR will come with new dev


Missing Organization Feature: EnhancedEmail

My organization enable Email to Case. Previously we successfully installed the package but now it asking for EnhancedEmail  to be enabled.

Hi @unidha,
Yes, the conversion tool requires Enhanced Email feature enabled because the code has checks to avoid trying to convert attachments belonging to EmailMessage object.
Probably not the best solution in place as there are other objects that have to be checked like EmailTemplate and Solution which don't seem to support Files either. You can track progress towards a new feature to let admins configure which parent object types should have attachments converted at https://github.com/DouglasCAyers/sfdc-convert-attachments-to-chatter-files/issues/15
Thanks,
Doug


Add Flashbots harvester contract

FlashbotsHarvester pays the miner directly by forwarding ether to block.coinbase. It takes a minHarvestExpected argument (measured in want) and reverts the tx if the strategy harvests less than that
FlashbotsHarvester: 0xA9C0252dcB258F41ABF99348F9C103bedac678fA
Also modified StrategyConvexStakingOptimizer.sol to return want gained instead of a harvest data struct

Superseded by #220


Profile picture not showing up

[

](url)

Steps to reproduce:

Create account
add photo url
connect widget on your own website
Send message from Website
Reply to message from app.papercups.io

Fixed in https://github.com/papercups-io/papercups/pull/84


Automatic PR of restapi_auto_mcardosos_graph into dev

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

Closing until we release v14


Spelling fixes

Codecov Report

Merging #143 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #143   +/-   ##
=======================================
  Coverage   91.16%   91.16%
=======================================
  Files           5        5
  Lines        2116     2116
=======================================
  Hits         1929     1929
  Misses        187      187

Impacted Files
Coverage Δ

scour/scour.py
92.78% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


ci: Fix golangci lint checks

Description
Checklist

[ ] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
CI is enough
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

I wonder why this wasn't caught by CI before it was merged

The golangci lint wasn't showing any failure logs today and failing for everyone.

Enabled auto merge

/test gke-upgrade-tests gke-qa-e2e-tests


[Bug]: Theme Btn alignment for 1024px and 1440px

Describe the bug
The Theme button is not aligned properly for 1024px and 1440px width
To Reproduce
No response
Expected Behavior
No response
Screenshot/ Video
1024px

1440p

Desktop (please complete the following information)
Device: desktop
OS: windows
Browser: Chrome
Additional context
No response

Hey, i would like to fix this.

I'm Working on this :)

pls review my PR ( Navbar Fix #245 )


Add privacy statement and Slack link to footer

Signed-off-by: David Espejo<EMAIL_ADDRESS>

This website does not use cookies or any other tracking technology.

I don't think this is currently true. We embed youtube videos, which will track users:
https://www.kiroule.com/article/use-video-embeds-in-hugo-theme/
There is a workaround proposed in that article and there are privacy settings that we can set in the config.yaml:
https://gohugo.io/about/hugo-and-gdpr/

This website does not use cookies or any other tracking technology.

I don't think this is currently true. We embed youtube videos, which will track users: https://www.kiroule.com/article/use-video-embeds-in-hugo-theme/

Right @waciumawanjohi just removed that statement after confirming with legal.
The content is now aligned to what other VMware-initiated projects actually display


formatted long string in screen name and description

Description
Formatted screen name and description so that the Delete screen button stays in screen when the name and description of the screen is too long.

Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General
Read the full guidelines for PRs here

[x] Added the correct label: < feature | enhancement | bug | dependencies | infrastructure | ignore-for-release | database >
[x] Assigned to a specific person, civiform/developers, or a more specific round-robin list.
[ ] Created unit and/or browser tests which fail without the change (if possible)
[x] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary

User visible changes

[ ] Followed steps to internationalize new strings
[ ] Wrote browser tests using the validateAccessibility method
[x] Tested on mobile view. See mobile device mode
[x] Manually tested at 200% size
[x] Manually evaluated tab order

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #5001

Did a quick manual test from your branch and it seems the problem is still there:

Might be good to add a browser test with a snapshot.
Also, did you see the comments on the issue?  What would you think of limiting the number of characters on the name and description fields?

I think that we need to limit the max widths for both the block list on the left and for the block info display.  Then, we also need to make sure the delete button is always within the container.  Honestly, this page needs a major style clean-up, so everything is made a little bit more challenging.  I think fixed max-widths would do the trick for now.


Check nans, satellite

Detailed Description
Check for nans
Context

don't want to keep any results if the there are nan's in satellite data
this should be done in India too

Possible Implementation

check in the UK, that there are no nans

We should only check the UK area
ds_sat.sel(
x_geostationary=slice(-996_133.85, -480_064.6),
y_geostationary=slice(4_512_606.3, 5_058_679.8),
)


nomad-driver-podman: 0.3.0 -> 0.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hashicorp/nomad-driver-podman/releases.
meta.description for nomad-driver-podman is: "Podman task driver for Nomad"
meta.homepage for nomad-driver-podman is: "https://www.github.com/hashicorp/nomad-driver-podman"
meta.changelog for nomad-driver-podman is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.0 with grep in /nix/store/sqk3zf7m3rc8l6hckycy0l7b52vn5paq-nomad-driver-podman-0.4.0
found 0.4.0 in filename of file in /nix/store/sqk3zf7m3rc8l6hckycy0l7b52vn5paq-nomad-driver-podman-0.4.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nomad-driver-podman

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sqk3zf7m3rc8l6hckycy0l7b52vn5paq-nomad-driver-podman-0.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nomad-driver-podman https://github.com/r-ryantm/nixpkgs/archive/5d3a4224eb62fd821456929759fa24008c6327ba.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sqk3zf7m3rc8l6hckycy0l7b52vn5paq-nomad-driver-podman-0.4.0
ls -la /nix/store/sqk3zf7m3rc8l6hckycy0l7b52vn5paq-nomad-driver-podman-0.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nomad-driver-podman

Maintainer pings
cc @cpcloud for testing.

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

1 package failed to build:

nomad-driver-podman

Building fails while it fails to fetch dependencies
go: downloading github.com/godbus/dbus/v5 v5.1.0
go: downloading github.com/json-iterator/go v1.1.12
go: downloading github.com/Masterminds/sprig v2.22.0+incompatible
go: downloading github.com/bgentry/speakeasy v0.1.0
go: downloading github.com/posener/complete v1.2.3
go: downloading github.com/prometheus/client_golang v1.12.1
go: downloading golang.org/x/mod v0.6.0-dev.0.20220106191415-9b9b3d81d5e3
go: downloading github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd
go: downloading github.com/modern-go/reflect2 v1.0.2
go: downloading github.com/Masterminds/goutils v1.1.1
go: downloading github.com/Masterminds/semver v1.5.0
go: downloading github.com/google/uuid v1.3.0
go: downloading github.com/huandu/xstrings v1.3.2
go: downloading github.com/imdario/mergo v0.3.12
go: downloading github.com/prometheus/client_model v0.2.0
go: downloading github.com/prometheus/common v0.34.0
go: downloading github.com/beorn7/perks v1.0.1
go: downloading github.com/cespare/xxhash/v2 v2.1.2
go: downloading github.com/prometheus/procfs v0.7.3
go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.2-0.20181231171920-c182affec369
github.com/hashicorp/nomad-driver-podman imports
        github.com/hashicorp/nomad/nomad/structs imports
        github.com/hashicorp/vault/api imports
        github.com/hashicorp/vault/sdk/helper/jsonutil imports
        github.com/hashicorp/vault/sdk/helper/compressutil imports
        github.com/pierrec/lz4<EMAIL_ADDRESS>read tcp <IP_ADDRESS>:59982-><IP_ADDRESS>:443: read: connection reset

Building fails while it fails to fetch dependencies

https://logs.nix.ci/?key=nixos/nixpkgs.183577&attempt_id=c46d667d-7367-4f8d-92c5-89aa7f6ff5fd
https://logs.nix.ci/?key=nixos/nixpkgs.183577&attempt_id=99d78259-6d5a-4041-99d1-04d4649cf328
It built on both linux ofborg and I can build it locally without any issues?
@ofborg eval


Port ifremer features

started addressing comments in https://github.com/Kitware/dive/pull/1348/commits/584394abdd76192a8ad3f01521203660ad7d5d9b

Hey @BryonLewis
Do we have any more issues to address regarding this work?

I believe I'll need to re-review it.  I never had any actual 3D display before (I could get the renderer up and see the new panel but there wasn't a visbile display of  3D tracks before).  If I can find some time for testing to make sure the feature works properly when given some test data it should be good.  Just been a bit busy with some other projects.

Thanks for the update Bryon. I'm going to send you test data that hopefully should work out of the box to test the feature by the end of next week. (I'm on vacation right now)

Trying to revive this PR @BryonLewis
Noticed that the attribute value lookup for multicam is based on cameraStore.getAnyTrack, which cause issues if you have paired tracks with only one of them containing attributes. This happens if you have the stereo3d_* stuff only on the left camera dataset


Arduino sensor example fixes for newish click-to-play Audio API rules

Oof, while #166 was just merged, I had to make some changes since then to this to get it to work with the new Web Audio API click-to-play permissions.
(Refs; https://github.com/publiclab/webjack/issues/88, processing/p5.js-sound#322)
I wrapped it in some extra code, which is not super simple:
https://github.com/jywarren/p5js-webjack-sensor/blob/90de143c93fc501b1ad50a4cd18b370edd2fd71a/webjack-p5-wrapper.js#L1-L16
The resulting receiveSensorData(fn) method is not bad though, as in this sketch:
https://editor.p5js.org/jywarren/sketches/rkztwSt8M
function setup() {
  createCanvas(800, 800);
  noStroke();
  fill('#ff00aa');

  receiveSensorData(function draw(data) {

    console.log(data); // output the values to log
    // data[0] is the 1st value, data[1] 2nd, etc.

    // draw stuff! Browse http://p5js.org/reference/
    background('#ddd');
    ellipse(200, 200, data[0]+10, data[0]+10);

  });
}

@catarak @lmccart (or anyone else who has input i guess?) what would be the best p5js-y way to fix this?
I'm 💯 game to contribribute a fix, but should I make the receiveSensorData() method includable via a script element, or maybe pre-concatenate a version of WebJack that has that method included, specifically for p5js usage? (we already require including the basic webjack.js, so that'd minimally increase complexity)
And, how does this syntax look? Passing a function to this new receiveSensorData(fn) function looks roughly similar to writing a draw() method but it's a little weird that it's within the setup() function.
And there's the slight weirdness that in receiveSensorData() i'm running parseFloat on everything because it's returning string values (but I could probably solve this in follow-up by better recognizing and parsing numbers vs. non-numeric chars, and release a new version of receiveSensorData)
Anyways, happy to help, and if the above looks good I'll submit a PR to a new version which has receiveSensorData()!

I'm 💯 game to contribribute a fix, but should I make the receiveSensorData() method includable via a script element, or maybe pre-concatenate a version of WebJack that has that method included, specifically for p5js usage? (we already require including the basic webjack.js, so that'd minimally increase complexity)

It depends on whether this function would be useful for other users of webjack. If it does provide them with some usability improvements by removing boilerplate that they need to write, it would make sense for you to include in the webjack library itself.

And, how does this syntax look? Passing a function to this new receiveSensorData(fn) function looks roughly similar to writing a draw() method but it's a little weird that it's within the setup() function.

The callback pattern you are using now is naming the callback inline with the name draw, it is not strictly necessary although useful for debugging (but then not if you named it draw since it conflicts with p5's own draw function). I would suggest refactoring out the function, something like so:
function setup() {
  createCanvas(800, 800);
  noStroke();
  fill('#ff00aa');

  receiveSensorData(handleData);
}

function handleData(data) {

  console.log(data); // output the values to log
  // data[0] is the 1st value, data[1] 2nd, etc.

  // draw stuff! Browse http://p5js.org/reference/
  background('#ddd');
  ellipse(200, 200, data[0]+10, data[0]+10);

}

It feels more evented that way (feels like mousePressed() etc).

Another option would be to create a p5 library for WebJack, which could include receiveSensorData.

Yes, i was thinking of providing a wrapper library, as opposed to including
a p5js-specific function within Webjack -- that sounds like a great way
forward. And, I love the idea of breaking out the receiveSensorData()
function lower in the script, instead of providing it inline -- it
certainly does look more like how the 'draw()' method is used. Thanks for
the input! I can open a PR with this, then.
On Wed, May 1, 2019 at 1:30 PM Cassie Tarakajian<EMAIL_ADDRESS>wrote:

Another option would be to create a p5 library for WebJack, which could
include receiveSensorData.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/processing/p5.js-website/issues/463#issuecomment-488351008,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAF6J5BFFVSAQ57U4Y2Z2LPTHHQ7ANCNFSM4HJDY4QQ
.

Work in progress at https://github.com/processing/p5.js-website/pull/464/ 🎉


Make Footer responsive. 🫡

Add media queries to the footer. 🤩
Add more styles to the footer.

please assign this to me

Assigned to you @bamidelee

hey @Spyware007  can you assign this issue to me i can work on it


Uploaded .ipynb model building file

The file contains the technique to build the neural network model

Changes Accepted


Allow adding th title attribute

It would be useful to have a title attribute added to th elements. This attribute is used by the browser to set a tooltip on hover.
Although it's possible to add a custom thead, it seems to me that this breaks the automatic generation of sorting and filtering logic.
I followed the idea behind data-header-class attribute and created a data-header-title attribute. In addition, if no data-header-title is provided, the data-title attribute value is used by default.

Like the idea.
I'm going to tweak the pull request to allow data-title attribute be a binding expression. Similar to how header-class is now after the commit https://github.com/esvit/ng-table/commit/60de2066b9be1a33210767783354c11004d5e042

Sounds good! :+1:

This now adds a blank box (empty title attribute, title=" ") to each of my row headers on hover. See image below. How can I avoid this?


Fix issue with k8s.io/docs/concepts/configuration/secret/

Closes #23089

Preview page: https://deploy-preview-23090--kubernetes-io-master-staging.netlify.app/docs/concepts/configuration/secret/

@pjhwa Thanks for the helpful additions!
/lgtm
approve


package.json : Shouldn't main point to app/main.js ?

Hi,
Shouldn't the main property in package.json point to app/main.js instead of main.js?

Hi @hepiyellow ,
I think the app/ redirection is made by the start script

"start": "cross-env NODE_ENV=production electron ./app/",

Well, sorry for the un-informative question.
Here is my real question:
I added this VS Code launch configuration:
{
            "name": "Launch",
            "type": "node",
            "request": "launch",
            "program": "${workspaceRoot}/app/main.js",
            "stopOnEntry": false,
            "args": [],
            "cwd": "${workspaceRoot}",
            "preLaunchTask": null,
            "runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron",
            "runtimeArgs": [
                ".",
                "--enable-logging"
            ],
            "env": {},
            "console": "internalConsole",
            "sourceMaps": false,
            "outDir": null
        }

And it works only if I change package.json's main to be app/main.js instead of main.js.
When it is only main.js then I get pop-up window with an error saying:
"unable to find Electron app at [path to my electron-react-boilerplate]"
Is there something wrong in my launch config?
Using VS Code 1.11.2

I closed this because there wasn't a response from you so I assumed you resolved this. My bad. Try setting it to app/main.development.js. I'm just guessing. I haven't actually configured vscode with electron before

VS code looks for the main property in package.json.
I found this:
http://electron.rocks/debugging-electron-in-vs-code/
And it says:
"program - Your app entry point - same as main in package.json (needs to be explicit, setting "." won't do)"
Solution:
It also says for the launch config:
"cwd - Path to the program being debugged (in our case, that's root where program file is)"
So I set:
"cwd": "${workspaceRoot}/app",
instead of :
"cwd": "${workspaceRoot}",
and it works now.
I don't know if there are other implications of having the working directory at app/,
but for now it works.

I saw that there is a .vscode folder in this repository.
It might be a good idea to add a launch.json there.

@hepiyellow would be awesome if you could PR for this


DAshboard url for own cluster

Description
Changes proposed in this pull request:

dashboard url for own cluster instances does not contain kubeconfig id in the list of parameters

Related issue(s)

/retest


Bug: unique keys on "type" and "secret" in auth_identities table breaks create user functionality in sqlite3 DB

PHP Version
7.4
CodeIgniter4 Version
4.2.7
Shield Version
1.0.0-beta.2
Which operating systems have you tested for this bug?
Linux
Which server did you use?
cli-server (PHP built-in webserver)
Database
SQLite3
Did you customize Shield?
I use it within Bonfire2, non-customized.
What happened?
Migration code for the shield creates a table auth identities and adds unique keys on columns "type" and "secret".  It is my understanding that Codeigniter engine also creates indexes during migration on all unique columns. So, for example, in SQLite3 exported db you see this:
CREATE UNIQUE INDEX IF NOT EXISTS "auth_identities_type_secret" ON "auth_identities" (
"type",
"secret"
);
However, the "type" column stores non-unique values. So, again, in SQLite3 I get an error that I have described here:
https://github.com/lonnieezell/Bonfire2/issues/218
Steps to Reproduce
I guess it is peculiar to SQLite3 db.
Expected Output
Expected is creation of a new user without errors.
Anything else?
There might also be some problem with repeated lines inserted into auth_identities for a single user (which also throws an error on the "secret" being non-unique), but that might be some issue in Bonfire. I have not investigated further.

I am no longer sure this is a valid issue, since after update of Shield, Bonfire and Codeigniter to the latest versions today the issue no longer occurs...
Still somebody should check if creating indexes on those columns is meaningful.

I don't like them. Nothing about the table itself suggests that those should be unique, and I could imagine some identity platform with multiple secrets, some of which are not unique.
Also, in general: I like a dumber database and smarter code. Things like unique indexes get in the way of soft deletes and data migrations. They are "nice to have" in some scenarios but in general I think they do more harm than good.

OK, the secret column may have unique values but the type column has values in only a few categories, like "email_password". Did I understand it wrong that indexes should be created only on columns with unique values? This sql (part of exported Shield db) seems to treat both columns as unique:
 CREATE UNIQUE INDEX IF NOT EXISTS "auth_identities_type_secret" ON "auth_identities" (  	"type",  	"secret"  );

What needs to be unique is "type + secret" value; if the type is different, it does not matter if the secret value is the same.

Ok, I did not know about this possibility to create index on two fields... sorry for the trouble.


useLoadingState to swallow errors when given an errorHandler.

Coverage remained the same at 100.0% when pulling f12078755dd3fcf46d323e0bafd856efebd81eb0 on useLoadingState-error-handling into 45e7278f0ce069fa9f83fb4976a936579567bbe9 on master.


Hot Fix Bug

Bug on naming preventing to properly use the code.

:slightly_smiling_face:

lol, @Araq that's why your code highlighting didn't work when you reviewed it :)


Adds performance test for React vs Vanilla

@PaulKinlan PTAL

@googlebot I signed it. On account of being a Googler and all that...


Implement JarInfo for cli jar command to store image parameters.

Part of #2796
JarInfo will hold all the image parameters derived from JarProcessor. The implementation is based on ContainerBuildPlan.

ContainerBuildPlan essentially does the same thing so don't need this intermediate class right now.


user-docs: Add box around instructions.

Adds a little bit of CSS and an example of how to use it for creating a little box around a set of instructions

(example):

@rishig

cool! The syntax looks good to me. In terms of styling, I think we'll want this to be pretty narrow (narrower than the surrounding text), and white background. I'm imaging the tips and stuff to generally be outside the box, unlike how slack does it.
But if those changes are hard, we can also merge this and work on the styling as follow-up.

How's this?

Left justification: Sounds good, but it sounds a bit like we are trying to make the box's size more like the surrounding text than you originally implied. Slack has their box the same size as their surrounding text, but the surrounding text is overall smaller. IIRC, there is an ideal column width that allows for better eye saccades, 45 to 75 Characters according to this article. Our current documentation width is about 110 characters.

Oops, sorry for the delay here.
I think making the surrounding text a lower column width and then making the box the same size is a fine solution. I agree it's a bit too wide right now.

new screenshot:

This lgtm visually.  One thing to think about: The HTML markup could be a lot simpler if we didn't intend to eventually support multiple sets of instructions (since the outer divs are a good syntax for that).  If we just want all of our numbered list blocks in /help and /api to look this way, we could e.g. just do styling for all <ol> blocks or have some JS code or markdown post-processor run to edit the HTML to put the wrapper divs around every <ol> in the document.
If we're more excited about that design (where this is automatic), I have ping a few folks who I'd expect to not have much trouble implementing this.

I suspect that we are a pretty long way away from having multiple-platform support in documentation, and therefore it might be nice to have this box automatically go around all the <ol> blocks. As long as we are okay re-implementing this later for multi-platform support, I would be down the the simpler solution in the short-term.

Will the ol solution work for something like

(i.e. text that is not a part of any <li>?)
We'll probably also want it to work with our tip syntax.

Because the source repository for this pull request was deleted, this pull request has been automatically closed by GitHub as an unintended side effect of renaming our default Git branch. If this change is still desired, please open a new pull request, referencing this one in the description.


Inquiry about Pretraining Dataset Construction Using Waymo Motion Dataset

Dear Luke Rowe,
I hope this email finds you well.
I am pleased to see that your paper has been accepted and that the related code has been made open-source. I am currently working on reconstructing a pretraining dataset based on the data in the scenario folder of the Waymo Motion dataset. To assist with this, I have included images showcasing the contents of the folder for better clarity.
Could you kindly share your approach or any insights on how you processed the data? Any guidance you can provide would be greatly appreciated.
Looking forward to hearing from you. Thank you for your time and assistance!
Best regards,
fengmozhang

Hi Fengmo,
Sorry for the late reply. We work directly with the nocturne-processed waymo data. The code to convert the raw waymo data (as shown in your attached image) to a nocturne-compatible format can be found here: https://github.com/facebookresearch/nocturne/tree/main/scripts/json_generation


question for importing custom maps in CARLA

<!--

Thanks for contributing to CARLA!

If you are reporting an issue, please use the following outline:

CARLA version: 0.9.15
Platform/OS: win10
Problem you have experienced:
Hello , sir. When I import my road network into CARLA. I would like to know if it is possible to import the .xodr file and the .fbx file into the release version of CARLA

Hello! yes it is possible to import custom maps at the moment, I leave the link to the documentation, so you can use it as a guide.
https://carla.readthedocs.io/en/latest/tuto_M_custom_map_overview/

OK,  thank you for you reply. I see the documentation, which shows that it can not to import a custom map if  I use a package version of CARLA in Windows.


This is driving me crazy: Error: Failed to find .env file at path: ./devOps/env/env.js

I have created a env.js file that i want to use with dynamic vars (that's why i'm using js)
I'm using the same logic in many other apps and it works fine with env-cmd
But on a specific project that has the same files and same folder structure, basically everything is the same. It gives this error:
Error: Failed to find .env file at path: ./devOps/env/env.js
Everything i do i doens't work, i removed node_modules, package-lock, gitignore, dockerignore, every .ignore
running this command: env-cmd -f ./devOps/env/env.js react-scripts start
what am I doing wrong?

same here

Also my react app is completely ignoring env-cmd with a simple env-cmd -f .env.android react scripts build.
@toddbluhm can you help?

same. works for pnpm env:cmd not with npm

Are you using this in an ESM setup? It seems the package currently doesn't support ESM and in this case fails with this error and hides the original one:
Error [ERR_REQUIRE_ESM]: require() of ES Module [...]\.env.js from<EMAIL_ADDRESS>not supported.
.env.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead either rename .env.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in [...]\package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).

    at Object.getEnvFileVars ([...]\node_modules\.pnpm\env-cmd@10.1.0\node_modules\env-cmd\dist\parse-env-file.js:21:33)

I got rid of this issue based on the previous comment: I renamed my env file from env.js to env.cjs.

Same here.
Remark: If I rename my env file from env.js to env.cjs, then I get rid of the message but the file is ignored. (this is even worst)

I renamed the file to .cjs and it worked!

Just finished updating the current master to use only ESM and updated all dependencies as well. The current master version only supports node >= 18 now. Give that a test if you like.


Add singleSelectionMode setting

Fixes issue #273.
Changes proposed in this pull request:
Add a "singleSelectionMode" option which causes the control/meta keys to be ignored.
This obviously conflicts with "persistantSelectionMode", currently singleSelectionMode overrides persistantSelectionMode.

Thanks!  Cool feature!


How to use custom messages and attributes when validating arrays?

Laravel Version: 6.10.1
PHP Version: 7.3.7

Description:
Let's assume we have such stub of form, where we print all tests information:
@if(isset($includeTests) && count($includeTests) > 0)

        Включити питання з наступних тестів:

    @foreach($includeTests as $test)

                    <input class="form-check-input mt-2 is-correct required-target" type="checkbox"
                           id="include[{{ $test->id }}][necessary]"
                           data-required="include[{{ $test->id }}][count]" name="include[{{ $test->id }}][necessary]"
                           @if(old("include.{$test->id}.necessary")) checked="checked" @endif>

                id }}][necessary]" class="form-check-label variant-text">
                    {{ $test->name }}

id}.count") col-auto @else col-2  @enderror">
                <input id="include[{{ $test->id }}][count]" name="include[{{ $test->id }}][count]" type="number"
                       class="form-control form-control-sm @error("include.{$test->id}.count") is-invalid @enderror"
                       placeholder="{{ $test->questions_count }}"
                       min="1" max="{{ $test->questions_count }}"
                       value="{{ old("include.{$test->id}.count", '') }}">
                @error("include.{$test->id}.count")

                    {{ $message }}

                @enderror

    @endforeach
@endif

And when we send request. The input will be in format:
array:5 [
  "_token" => "cHBtdOsyrJS2FwbUea1UZwzviVrIlp3wgn5exASa"
  "name" => "info1"
  "uri_alias" => "info2"
  "time" => "12345"
  "include" => array:2 [
    1 => array:2 [
      "necessary" => "on"
      "count" => "123"
    ]
    2 => array:2 [
      "necessary" => "on"
      "count" => "123"
    ]
  ]
]

I've created the form request class with method messages that, as i hoped, will change the default into my custom.
class CreateTestRequest extends TestRequest
{
   public function messages()
    {
        return [
            'include.*.count' => [
                'required_with' => 'Hoped message'
            ]
        ];
    }

    public function rules()
    {
        $rules = $this->baseRules;
        $rules['uri_alias'][] = 'unique:tests';
        return $rules;
    }
}

The $baseRules variable is like so:
    protected $baseRules = [
        // ...
        'include.*.count' => [
            'required_with:include.*.necessary',
            'nullable',
            'numeric',
            'min:1'
        ]
    ];

The question is: Why the

The include.1.count field is required when include.1.necessary is present.

message is printed, but not

Hoped message

If i define attributes method like so:
public function attributes()
{
    return [
        'include.*.count' => 'Count',
        'include.*.necessary' => 'Necessary',
    ];
}

The second first question is: What the heck it displays in the right way 'Count'  attribute, but not also 'Necessary'?

The Count field is required when include.1.necessary is present.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Generic Input/Output of DataFrames, Client part

This PR is the client part corresponding to PR #475 , it contains Python code to generate config/jobparameters to enable the user to use CosmosDb as Data Source and Online/Offline storage.
It adds "CosmosDbSource" and "CosmosDbSink" in the client lib, the former is to use CosmosDb as data source, the latter is to use CosmosDb as online storage.
There are still some open questions about using CosmosDb as offline storage, the main one is, CosmosDb requires every row/doc to have a unique key, but in the feature joining job, we still don't know what should be used as a meaningful key.
For this reason, using CosmosDb as an offline store is not opened in the public interface, even the function is usable internally.

E2E test should get updated to cover write features to cosmos db

Can we add test cases to show this in working?

The test failures are irrelevant hence merging the code.


Update scala3-library_3 to 3.1.3

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

Files still referring to the old version number
The following files still refer to the old version number (3.0.2).
You might want to review and update them manually.
.github/workflows/ci.yml
build.sc
docs/website/package-lock.json

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala3-library_3" } ]

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

Needs https://github.com/almond-sh/almond/pull/961.


Change to New Embedding Models

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

azd up

Any log messages given by the failure

The model 'Format: OpenAI, Name: text-embedding-3-small, Version: 2' of account deployment is not supported. (Code: DeploymentModelNotSupported)

Expected/desired behavior

I'd like to move to either the large or small embedding model on a new deployment, but receive this error after changing the embedding model. I've seen issues in the past where I needed to change the case sensitivity of a single letter of a model, but that doesn't seem to apply here. Any help would be greatly appreciated for the roll out!

OS and Version?

Windows 11 Pro

azd version?

azd version 1.6.1

Versions

Mention any other details that might be useful

My only change was the following in main.bicep:
param embeddingModelName string = 'text-embedding-ada-002'
to
param embeddingModelName string = 'text-embedding-3-small'

Please try changing version to 1 here:
{
name: embeddingDeploymentName
model: {
format: 'OpenAI'
name: embeddingModelName
version: '1'
}
sku: {
name: 'Standard'
capacity: embeddingDeploymentCapacity
}
}

Hi Pamela, you're a legend! That worked. I was eyeing that version number, but wasn't sure where to change it. Thank you! However, I notice that after changing the model to text-embedding-3-large I get the following error after running prepdocs :
Exception Details:      (IndexDocumentsFieldError) 0: The vector field 'embedding' dimensionality must match the field definition's 'dimensions' property. Expected: '1536'. Actual: '3072'. Parameters: actions
I tried changing all the fields that included text-embedding-ada-002 to text-embedding-3-large and also changed the field vector_search_dimensions=1536 to 3072, but still same error. I tried redeploying everything post changes and same thing. I'll keep digging and working with Copilot to figure it out, but the last bit of help will get us moving with a production environment on the new embedding model. Guess I'm a bit deprecation worried lol. Thank you!

Did you delete the current index? I did assume there'd be some issues with dimension hard-coding, but making a fresh index with the correct dimension seems like it should work. I'll see if I can try it today.

I did not, since I didn't add anything to it. I'll give that a go next!

Ah yeah then that's the error. You're trying to index a field with more dimensions than you set it up with.

Got it, and prepdocs completed at the default of 3072 dimensions. I found the dimensions under Fields in the Search Service. Super helpful to match them from both ends like that. Now if I originally deployed with 1536 and Azure's default is 3072, is there no way to change the default set by Azure in AI Studio or somewhere else? It seems the initial deployment's custom dimension of 1536 conflicted with what is set as the default in Azure. No worries if it's passed my original ask, ideally 1536 large would be nice.

Hm, I don't see dimensions in the deployment options, where is that? I was assuming you could specify it when calling embeddings.create(). Here's a notebook that created multiple dimension sizes using the same deployment:
https://github.com/farzad528/azure-ai-search-python-playground/blob/main/azure-ai-search-text-embedding-3-large.ipynb

I found these two field where you could change dimension before deploying. I had to change 1536 to 3072 to get the new large model to work without the error you solved above.
),
SearchField(
name="embedding",
type=SearchFieldDataType.Collection(SearchFieldDataType.Single),
hidden=False,
searchable=True,
filterable=False,
sortable=False,
facetable=False,
vector_search_dimensions=1536,
vector_search_profile_name="embedding_config",
),
SimpleField(name="category", type="Edm.String", filterable=True, facetable=True),
SimpleField(
name="sourcepage",
type="Edm.String",
filterable=True,
facetable=True,
),
SimpleField(
name="sourcefile",
type="Edm.String",
filterable=True,
facetable=True,
),
]
if self.use_acls:
fields.append(
SimpleField(
name="oids",
type=SearchFieldDataType.Collection(SearchFieldDataType.String),
filterable=True,
)
)
fields.append(
SimpleField(
name="groups",
type=SearchFieldDataType.Collection(SearchFieldDataType.String),
filterable=True,
)
)
if self.use_int_vectorization:
fields.append(SearchableField(name="parent_id", type="Edm.String", filterable=True))
if self.search_images:
fields.append(
SearchField(
name="imageEmbedding",
type=SearchFieldDataType.Collection(SearchFieldDataType.Single),
hidden=False,
searchable=True,
filterable=False,
sortable=False,
facetable=False,
vector_search_dimensions=1024,
vector_search_profile_name="embedding_config",
),
)

Hm, I don't see dimensions in the deployment options, where is that? I was assuming you could specify it when calling embeddings.create(). Here's a notebook that created multiple dimension sizes using the same deployment: https://github.com/farzad528/azure-ai-search-python-playground/blob/main/azure-ai-search-text-embedding-3-large.ipynb

Thanks I'll read through this, I'm still working through CS50/CS50P, so will take some time to figure out. Appreciate the title change to make it easier. Gen AI is very inspiring and thanks for making this accessible!


Add documentation of Game Events

The notes in #268 are probably a good starting point :-)

Will be part of respective recipe. Those are all setup as new issues for the type of analytic operations we will support.


TypeError: process_node() takes from 5 to 6 positional arguments but 7 were given

I try to run distillation example but got error message.
Traceback (most recent call last):
File "/Users/maetee/Github/SFILES2/distillation_tower.py", line 62, in
img = pfd.draw(ctx)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pyflowsheet/core/flowsheet.py", line 176, in draw
s.draw(ctx, grid, minx, miny)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pyflowsheet/core/stream.py", line 22, in draw
points, startAnchor = self._calculateAutoRoute(minx, miny, grid)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pyflowsheet/core/stream.py", line 123, in _calculateAutoRoute
path, _ = finder.find_path(start, end, grid)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pathfinding/finder/a_star.py", line 94, in find_path
return super(AStarFinder, self).find_path(start, end, graph)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pathfinding/finder/finder.py", line 158, in find_path
path = self.check_neighbors(start, end, grid, open_list)
File "/Users/maetee/miniforge3/envs/sfiles/lib/python3.9/site-packages/pathfinding/finder/a_star.py", line 78, in check_neighbors
self.process_node(
TypeError: process_node() takes from 5 to 6 positional arguments but 7 were given

same here, even the example mentioned on the homepage returns the same error

Downgrade pathfinding:
pip install pathfinding==1.0.1


broken example link on homepage

congrats team this is an amazing project!
just a heads up that the "see examples" button at the bottom of the homepage seems to be a broken link. sorry, I couldn't figure out where it was supposed to link to submit a pr..

thanks @lmccart!
I will fix this now.


Can't control Wyze Lock Bolt YD_BT1 via Homekit

Describe The Bug:
Trying to lock/unlock Wyze Lock Bolt using Homekit, it's changed the status (Locked/Unlocked) in Homekit, but the actual Lock Bolt does nothing.
To Reproduce:

Expected behavior:
Homekit to lock/unlock wyze lock bolt
Logs:
[8/4/2023, 12:04:56 PM] [Wyze] Lock (Model: YD_BT1 (Mac: YD_BT1.095f8cb000ed72b829a0a34b657486e4 (Name: Front Door) not supported submit a request to https://github.com/jfarmer08/homebridge-wyze-smart-home/issues to have it added. If no issues are found

Plugin Config:
 {
            "name": "Wyze",
            "username": "XXX",
            "password": "XXX",
            "keyId": "XXX",
            "apiKey": "XXX",
            "mfaCode": "XXX",
            "refreshInterval": 60000,
            "entryExitDelay": 30,
            "showAdvancedOptions": false,
            "logging": "none",
            "excludeMacAddress": false,
            "excludedeviceType": false,
            "platform": "WyzeSmartHome"
        },

Screenshots:

Environment:
Homebridge v1.6.1
MacOS

There may be some technical limitations here since the Wyze lock bolt works primarily over Bluetooth (maybe even exclusively over BT) if that is the case this will likely require a large technical overhaul. I am curious if you are able to unlock this lock from anywhere over an internet connection or indeed you must be close to it?

You’re right. I can unlock only when nearby via BT… On Aug 7, 2023, at 7:30 PM, carTloyal123 @.***> wrote:
There may be some technical limitations here since the Wyze lock bolt works primarily over Bluetooth (maybe even exclusively over BT) if that is the case this will likely require a large technical overhaul. I am curious if you are able to unlock this lock from anywhere over an internet connection or indeed you must be close to it?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I know some homebridge plugins actually do use bluetooth like the switchbot plugin but I am not sure how complicated this gets and there is no infrastructure in place in this wyze plugin for bluetooth. I would say this will not be completed anytime soon unless @jfarmer08 has some insight I am not aware of. It could be done if someone has already reverse engineered the bluetooth communication between phone and wyze lock but I doubt that is known yet

Got it. Its helpful. Thanks for the insights. Appreciate that. On Aug 8, 2023, at 9:28 AM, carTloyal123 @.***> wrote:
I know some homebridge plugins actually do use bluetooth like the switchbot plugin but I am not sure how complicated this gets and there is no infrastructure in place in this wyze plugin for bluetooth. I would say this will not be completed anytime soon unless @jfarmer08 has some insight I am not aware of. It could be done if someone has already reverse engineered the bluetooth communication between phone and wyze lock but I doubt that is known yet
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Autocompleter system test

coverage: 94.319%. remained the same when pulling 3390bccd6a0262bb9cded9d2a32d15fc88b0728b on nimmo-autocompleter-systemtest into 234821eb8a044546f116de54ec1a4eace2690b08 on main.


Pdk integration

How can I integrate my custom pdk to OpenFpga? To do this what things should be done?

Hello @msaideroglu, to integrate our custom PDK/ Standard cell library you will have to update the OpenFPGA architecture file.
for example name="sky130_fd_sc_hd__inv_1"  in following line should be replaced with your own functionally equivalent  standard cell name.You need to do such mapping for the entire OpenFPGA architecture file, in order to get a correct tech mapped netlist.
https://github.com/lnis-uofu/OpenFPGA/blob/1fd9367ae9f7518fd041597e18c0650f1188dfee/openfpga_flow/openfpga_arch/k4_frac_N8_register_scan_chain_caravel_io_skywater130nm_fdhd_cc_openfpga.xml#L34

Hello @msaideroglu, to integrate our custom PDK/ Standard cell library you will have to update the OpenFPGA architecture file. for example name="sky130_fd_sc_hd__inv_1"  in following line should be replaced with your own functionally equivalent standard cell name.You need to do such mapping for the entire OpenFPGA architecture file, in order to get a correct tech mapped netlist.
https://github.com/lnis-uofu/OpenFPGA/blob/1fd9367ae9f7518fd041597e18c0650f1188dfee/openfpga_flow/openfpga_arch/k4_frac_N8_register_scan_chain_caravel_io_skywater130nm_fdhd_cc_openfpga.xml#L34

Is it not compulsory to mention verilog_netlist= "</Path to .v file of PDK>" in circuit_model?


Behavior param options

First try at implementing an interface to use option models!

as discussed offline, we will instead change _initiable() in make_behavior_option to include a model_controller key in the memory dictionary, and then for option_model.py we can add:
class _BehaviorOptionModel(_OptionModel):
    """An oracle option model for the BEHAVIOR env."""

    def get_next_state(self, state: State, option: _Option) -> State:
        env = get_cached_env("behavior")
        assert option.memory["model_controller"] is not None
        option.memory["model_controller"](state, env)


Join stops NodeRed

After pushing the Join command, NodeRed stop with this errors:
Found Sonos device B8-E9-37-5F-A7-72:6 at <IP_ADDRESS> 1 Nov 18:47:57 - [red] Uncaught Exception: 1 Nov 18:47:57 - TypeError: Cannot read property 'find' of null     at getTopology (/home/pi/.node-red/node_modules/sonos/lib/sonos.js:619:31)     at /home/pi/.node-red/node_modules/sonos/lib/sonos.js:1077:7     at Parser.<anonymous> (/home/pi/.node-red/node_modules/xml2js/lib/parser.js:303:18)     at emitOne (events.js:116:13)     at Parser.emit (events.js:211:7)     at SAXParser.onclosetag (/home/pi/.node-red/node_modules/xml2js/lib/parser.js:261:26)     at emit (/home/pi/.node-red/node_modules/sax/lib/sax.js:624:35)     at emitNode (/home/pi/.node-red/node_modules/sax/lib/sax.js:629:5)     at closeTag (/home/pi/.node-red/node_modules/sax/lib/sax.js:889:7)     at SAXParser.write (/home/pi/.node-red/node_modules/sax/lib/sax.js:1436:13)     at Parser.exports.Parser.Parser.parseString (/home/pi/.node-red/node_modules/xml2js/lib/parser.js:322:31)     at Parser.parseString (/home/pi/.node-red/node_modules/xml2js/lib/parser.js:5:59)     at Object.exports.parseString (/home/pi/.node-red/node_modules/xml2js/lib/parser.js:354:19)     at Request._callback (/home/pi/.node-red/node_modules/sonos/lib/sonos.js:1062:12)     at Request.self.callback (/home/pi/.node-red/node_modules/request/request.js:185:22)     at emitTwo (events.js:126:13)
This is my sketch:
[{"id":"aaeabd.e374654","type":"better-sonos-control","z":"ecc2de2b.8c965","confignode":"164fdcf2.cb59bb","name":"","mode":"","track":"","volume":"","volume_value":"","x":480,"y":160,"wires":[]},{"id":"ca4a84ea.beff28","type":"inject","z":"ecc2de2b.8c965","name":"","topic":"Bathroom","payload":"join","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":290,"y":160,"wires":[["aaeabd.e374654"]]},{"id":"164fdcf2.cb59bb","type":"better-sonos-config","z":"","name":"Bedroom","serialnum":"B8-E9-37-5F-A7-72:6","ipaddress":""}]
Where could the problem be?
Andrea

After an upgrade Sonos does no longer support the url http://{sonos'ip}:1400/status/topology which this application relies on

@80AAFF  Ok, thanks for the info


My titleFont and titleColor not change when selected

Version of the component: 2.1
Swift version: 3
iOS version: 10.2
Device: iPhone 7 Plus
Xcode version: 8.2.1

here is my code in state set

when i debug, i set selectedState background to orange and it worked but textfont and textColor not change.

sorry i update my pod and it was fixed.


sql: fix epoch ATX ID cache deadlock

Motivation
SQL query cache deadlocks b/c during transactions, SQLite lock is held while cache update lock is being acquired.
Description
Fixes the deadlock by bringing cached slice update out of the transactions.
Test Plan
Verified on mainnet

bors merge


busy_handler hook

Following the article about optimal performance for SQLite, one more thing missing in my opinion is the busy_handler hook. I have never worked with C bindings, nifs and the like, so I actually don't know if it is already supported or not. (search for busy_handler hook in the article)
A quote:

Luckily, in addition to the busy_timeout, SQLite also provides the lower-level busy_handler hook. The busy_timeout is nothing more than a specific busy_handler implementation provided by SQLite. Any application using SQLite can provide its own custom busy_handler. The sqlite3-ruby gem is a SQLite driver, meaning that it provides Ruby bindings for the C API that SQLite exposes. Since it provides a binding for the sqlite3_busy_handler C function, we can write a Ruby callback that will be called whenever a query is queued.

Is it already supported? If yes, how would I use it to write my own hook? And if not - what would it take to support it?

It is not available at the moment. Due to the pooled connection nature of DBConnection this is not possible.

We could introduce a way to define a function to be called using the sqlite busy handler, BUT the supervisor watching the process will die if it reaches the default timeout. Meanwhile, the query will still continue to execute in the background because it is a dirty NIF.
I'll have to think some more about this.

What are you trying to accomplish?
I'm just following the article to have the best settings possible on production. I don't have any issues yet myself. But the article does make sense to me, and it would be a shame if sqlite with ecto is less performant than with rails.

At some point, you still have to deal with a busy error or a timeout from the connection pool. The busy handler in that rails application is simply deferring when that happens and optimistically retrying until it exhausts itself. We can implement something similar but you still have to contend with the configured timeout when you execute a query with the repo.

That's understandable, that it won't make the system perfect.
If you can find a way to make the handler work - I think will go a long way towards sqlite adoption. It is a very cool database and the thought of having one service less to manage (postgres) is very very appealing to me.
If it's not possible because of the architecture - that's fine too. Unfortunately I don't know anything about nifs, to offer any help. But I'll educate myself in the coming days :)

@egze the way it is now it can handle quite a lot https://www.youtube.com/watch?v=huGVdGLBJEo but I do have plans to make it better.

I looked into this some more and I remember some of the reasons why I never implemented the busy handler callback.

You can not directly call an anonymous function from a NIF. You can only send a message and you only get a return value signaling that the message was sent. It does not block until the execution is done.
There is no standard library way to make the NIF execution sleep. We could use unistd.h but this rules out support on Windows.
This runs contrary to the Erlang creed of "let if crash". If a timeout error occurs a retry mechanism should be implemented by the application. This will also allow you finer control on when you want to do the retry and how to go about scheduling it.

Interesting. So the only way to have it, would be to compile SQLite where the callback is implemented in C, right? Would an extension be able to provide such a callback?

I don't recommend it.

Sorry to keep annoying you 😊 Why wouldn't you recommend it, if this interface if well defined by the SQLite authors? Or you don't recommend in general to bother with this hook?

I don't recommend it in the execution environment of Erlang or Elixir.
You can not call erlang/elixir code from a NIF. So there is no way to directly invoke a function or anonymous function. The only way to accomplish a callback into Erlang/Elixir is to send a message to a PID, this send is also a non blocking call. We have no idea if the call was processed. The only thing we know from the aspect of the NIF is that the message was sent. So there is no way to reliably have SQLite know if the function should return 1 or 0. Returning a 0 would cause the handler to stop and emit an SQLite busy error. Constantly returning a 1 would cause it to spin forever.

Yes, this I get that providing a callback in Elixir/Erlang is not the way to go. I thought that you wouldn't recommend doing it through an extension in C.
Anyway, let's call it a day. I might try it out with a C extension someday and then post a comment back here. Thank you for all the information you've provided.


fixed clamav.py datetime fromtimestamp function and update policy

There was two bugs preventing first run of the bucket antivirus definition update function:

according to https://stackoverflow.com/a/52508192/3476100 , if HeadObject is called on a nonexisting object, a different error will be raised depending on s3:ListBucket permission on the bucket. Solution to prevent update function from running into An error occurred (403) when calling the HeadObject operation: Forbidden is to allow the function the s3:ListBucket action on the bucket

The PR #63 introduced a bug when treating the exception from head_object. It is supposed to return a default timestamp when no object exists, but fromtimestamp is not actually a method from datetime but datetime.datetime

With those 2 corrections, the update function works again with first run.

 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.

 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.

I should note that the IAM new policy also works for me and I tested this in AWS on my account. So things are looking good!


[Discuss][C++] Replace MemoTable with a SwissTable implementation

Describe the enhancement requested
This is a follow up of https://github.com/apache/arrow/issues/36059#issuecomment-1592037527. There are many cases where we use a MemoTable, e.g. set lookup functions, vector hash functions, the count_distinct aggregate function, dictionary unification etc. Their performance can be boosted with a SwissTable.
We can either:

Use an existing swiss table library. This requires some work on dependency management. I recommend absl::flat_hash_map since they are the original author of swiss tables and we already has absl in our 3rd party toolchain.
Write one by ourselves. The current one in acero is too customized for the join node and it seems hard to extract a general hashtable from it. If I were to write one I would probably follow the structure of absl's and replace things like memory management and bit tweaking with Arrow ones.

What do you think? @pitrou @westonpace
Component(s)
C++

I don't know what a "Swiss table" is. Can you point to a general description?

More generally, I'm all for a faster associative table implementation.

Maybe we should first having an memory-usage / performance benchmark on this?

There's an Abseil's article and a CppCon talk that provide a good intro. The key idea is, when looking up a key via linear probing, it can check multiple slots at once.

Ah, so it's a hash table with SIMD-optimized lookup?

Maybe we should first having an memory-usage / performance benchmark on this?

Okay, I'll try to draft a proof of concept with the is_in kernel.

If you're going to use SIMD lookups, then I would ask that it relies on xsimd instead of intrinsics.

Ah, so it's a hash table with SIMD-optimized lookup?

Well it's a speedup even without SIMD instructions. To put it in a oversimplified term each slot occupies 1 byte so with int64 we can already check 8 slots at the same time. But SIMD can definitely make it even faster.

If you're going to use SIMD lookups, then I would ask that it relies on xsimd instead of intrinsics.

Sure, if we decide to write our own version.

Well it's a speedup even without SIMD instructions. To put it in a oversimplified term each slot occupies 1 byte so with int64 we can already check 8 slots at the same time. But SIMD can definitely make it even faster.

I see, thanks for the explanation.

Sure, if we decide to write our own version.

Well, we don't want to depend on Abseil, so our own version it should probably be :-)

There's an Abseil's article and a CppCon talk that provide a good intro. The key idea is, when looking up a key via linear probing, it can check multiple slots at once.

Thank you for explaining. I was also curious about what Swiss Join is. :)

Can you post benchmark numbers on string values? While hashing integers is nice, I'm not sure that's a dominant use case.

Sure. I chose Int32 because it was the only one tested with a large value set. I just wrote a similar set of benchmarks for strings and here's the result.
Before

After

Ok. I agree we could replace MemoTable with something better. However, we don't want to have a dependency on Abseil, so it will have to be reimplemented.

Ok. I agree we could replace MemoTable with something better. However, we don't want to have a dependency on Abseil, so it will have to be reimplemented.

Yes. I'll implement one using xsimd as you suggested. Also the swiss table may be worse than MemoTable on super small tables, since it checks 8 slots at once and reserves at least 8 slots on construction, as shown in the first two string benchmarks. We'll need further benchmarking to decide which hashtable to use in each individual use cases, or maybe with a dynamic dispatching mechanism.

I guess this might benifit the parquet dictionary encoding, which is widely used in parquet. If nobody move forward, I'll have a try next month

Yeah I'm afraid I won't have time to do this recently. Please have a try if you wish. Thanks!

Will the swiss table currently being used in acero's hash join (aka. swiss join) help on this?

The swiss table for hash join is very specialized and kind of hard to generalize. It can't be directly used for memotable.

Could I try to solve this issue?

@SGZW Anyone can work on an issue without asking for permission. Feel free to experiment and submit a PR when ready.


Ability to choose location of where debug.log is generated

Currently I have Twitch Leecher on my desktop and when I run it a debug.log gets generated on my desktop. Can we have the option to either choose where this log is generated or not generate the log file altogether?

no, you don't have twitch leecher... at least not mine. TL does not write debug.log files anywhere and must be installed with the installer. So if you have a portable version somewhere on your desktop, it is not my application.
The official release is HERE

@Franiac I definitely installed from the official release page. Have had it for at least 2-3 years. Every time I open Twitch Leecher now a debug.log file gets generated onto the desktop now when it never used to before. Seems to have been a result of the latest API change requiring us to login with our Twitch accounts

And please tell me how you got Twitch Leecher on your Desktop. Because the default install path is C:\Program Files\Twitch Leecher and the logfile gets written to the same directory where TwitchLeecher.exe is. So if TwitchLeecher.exe is on your Dekstop and all the other files of TL are not, you really have to tell me how you managed to do that. This all sounds extremely weird.

@Franiac I just created a shortcut to Twitch Leecher.exe on my desktop


Improve testing of load_from_uri

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

Project coverage is 100.00%. Comparing base (d0fcfa8) to head (76adf37).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines          548       548
  Branches        41        41
=========================================
  Hits           548       548

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


Nested questionnaireResponseComponent with enableWhen in repeated group is missing from questionnaire response.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1895
Description
The questionnaire response contains answers to all selected and enabled questionnaire items. However, when copying the questionnaire response, the use of 'getEnabledResponseItems' fails to copy the selected answers of enabled items.
when the type is a repeated group item, the questionnaireResponseItemParentMap is always empty in the findEnableWhenQuestionnaireResponseItem() API. This is because the buildParentList logic in EnablementEvaluator fails to correctly build the parent list for repeated group items.
Alternative(s) considered
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

I am adding end to end android test for this fix, but getting below error when Add item  get clicked second time to add more repeated group.
androidx.test.espresso.AmbiguousViewMatcherException: 'view.getId() is <2131230792/com.google.android.fhir.datacapture.test:id/add_item>' matches multiple views in the hierarchy.
Problem views are marked with '****MATCHES****' below.

Can we please review the PR? If the Android test gets fixed, I will push the changes; otherwise, I will have a separate PR for this Android test.


ci: move workflow check into actionlint, add reviewdog testing

There was a step in our main CI workflow that checks that bors.yml, build.yml and build_fork.yml generated properly from the source in build.in.yml. However, this check is only necessary for the (very rare) PRs which touch files in .github/. For that reason, I've moved the check into the actionlint workflow, which has a paths filter restricting to PRs touching those files.
While looking at this, I also realized that we can get reviewdog to automatically suggest fixes, so I added some steps to enable it for both actionlint and the workflow check.
One question: do we need these checks to run on push to master? According to bors.toml, these checks aren't required to pass for bors to merge things into master anyways. If we don't need it then I can delete quite a bit of code from actionlint.yml.

See https://github.com/bryangingechen/mathlib4/pull/13 for what the suggestions from reviewdog look like. Note that these suggestions are not integrated with the bot fix style command because making changes to workflow files requires more permissions than we currently give that bot.

I really like the idea of outsourcing the workflow check to actionlint!
You are way more knowledgeable of CI, so it feels a little weird to
maintainer merge
but this looks great, thanks!

Let me just ping @semorrison to see if they have an opinion on my question about whether we need to run these checks on pushes to master. (My preference would be to have this run only on pull_request events for simplicity.)

@bryangingechen, I think it's fine if you change this to only run on pull_request.
bors d+

bors r+


Enable parallel testing

This PR makes a number of changes to the test suite in order to allow it to run in parallel using the Rails 6 parallelism mechanism (multiprocess, fork-based).

[x] target flakey test
[x] refactor schema tests to use ephemeral location + cleanup
[x] address chrome driver modal issues
[x] enable parallel testing by default

The chrome modal issues I was seeing appeared to to go away when I allowed more leeway in capybara's wait timeout - presumably Chrome was bogging down a bit.

Sample runs, anecdotal only...
Before:
$ PARALLEL_WORKERS=1 bin/rails test
Finished in 1556.693891s, 0.6103 runs/s, 3.4098 assertions/s.
950 runs, 5308 assertions, 0 failures, 0 errors, 13 skips

After:
$ PARALLEL_WORKERS=2 bin/rails test
Finished in 731.754351s, 1.2982 runs/s, 7.2538 assertions/s.
950 runs, 5308 assertions, 0 failures, 0 errors, 13 skips

I'm ignoring the remaining rubocop reports, to avoid refactoring working code.

Out of interest @kenny-lee-1 / @ollietulloch - I ran this on my workstation (12 vcpus) and test runs take ~200 seconds. The limiting factor appears to be the schema pack assembling, which has to run once per worker that runs any tests from that file.


move ers to 2.0.0

Code Review Processes
New Feature Review
Before submitting a pull request for new review, make sure the following is done:

Design doc is created and posted here: [Insert Link]

README Checks

[] README has proper context for the reviewer to understand what the code includes, any important design considerations, and areas to pay more attention to

Code Checks

[] Add explanatory comments. If there is complex code that requires specific context or understanding, note that in a comment
[] Remove unncessary comments. Any comments that do not add additional context, information, etc. should be removed
[] Add natspec.
[] Scrub through the code for inconsistencies (e.g. removing extra spaces)
[] Ensure there are not any .onlys in spec files

Broader Considerations

[] Ensure variable, function and event naming is clear, consistent, and reflective for the scope of the code.
[] Consider if certain pieces of logic should be placed in a different library, module

coverage: 99.538% (-0.5%) from 100.0%
when pulling e9df31faaadd2c814428f34ddb9ae3b96b4333fb on cameron/audit-fixes
into abfe926c98b1dbb4f8031cdb97c2b14f5452976e on master.


Include email in the userData profile

No idea why it is not included in the response by default.
scope='email' has no effect.

Added in the new version


[FEATURE] Increase WebX DAO logo size

Detailed description
I find it small

same with footer

Context
It will be cool to increase the navbar and footer logo :)
Possible implementation
No response
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@faisalamin001 has opened a PR about it, there for considering it assigned for now

Hey!, I opened a PR on this issue!
Please take a look!

@Panquesito7 Can you please assign me this issue, I have created an SVG for our logo and fixed the issue. Its looking great now. Hope you let me do this issue its just a min of work as I have created the logo and tested it 🙂.

^^ See the logo is still not fixed yet.

Hi, you can find our svg logo here! No need to create one.
Actually the issue is being assigned to @Rohitchaudhary13 #217 but need changes.

Hey, @mkubdev there's no SVG logo on that link that you gave 😕, all logos are in png format which will then going to blur out. I would recommend you to please check this SVG it can be useful 😊.

Hey, @mkubdev there's no SVG logo on that link that you gave 😕, all logos are in png format which will then going to blur out. I would recommend you to please check this SVG it can be useful 😊.

 Seem the svg was converted into png once inserted into the markdown... 🤔
Thanks a lot for this logo, maybe you can add it into the brand-asset repository?

This is how it's looking, clean right? 😃

This is how it's looking, clean right? 😃

Yup super cool!

Hey, @mkubdev there's no SVG logo on that link that you gave 😕, all logos are in png format which will then going to blur out. I would recommend you to please check this SVG it can be useful 😊.

 Seem the svg was converted into png once inserted into the markdown... 🤔 Thanks a lot for this logo, maybe you can add it into our brand-asset repository?

Okay man, And one more thing, Can I make a pr now for this issue with the right logo in it?! Thanks

This is how it's looking, clean right? 😃

~Yup super cool!~ Oh wait, the logo has a missed triangle :

Okay lemme trynna fix it rn.

@tarunfy i sent you an e-mail containing further infos. :)