Add missing bool type test for HivePartitionFunction

@mbasmanova
It seems that typed hash function for bool is implemented but there is no test for that case.

Ah I see now how it works now. This PR is automatically merged. I will pay a attention next time.

@amaliujia I merged the PR and updated the description. Thank you for the contribution.


[Feature] Game points/ranking system

In the review on Monday, people mentioned that it should come with some motivating system. A leaderboard or ranking system could be sufficient for now.

Fastness: this is not recommended as players might rush through the game without reading the materials or discussing with peers, by randomly picking answers
Point: need to calculate points based on score and rate?
Ranking: currently can be a global ranking for all players. Maybe displayed both locally and in game center
Metrics: what categories are a player best at?

GameCenter resources page: https://developer.apple.com/game-center/

https://www.codemag.com/Article/1603081/Integrating-Achievements-and-Leaderboards-into-Your-iOS-Game-Using-Game-Center

https://medium.com/@alexanderruzmanov/custom-game-center-leaderboard-in-your-ios-game-ee6358409bb0

Use game center for global leader board; in app make a metric page for performance.
Add game center button to show the leader board.

https://developer.apple.com/videos/play/wwdc2020/10145/

Game Center Profile

leaderboard, profile, achievements.

The old GameCenter <13.0 was a VC with 3 tabs (via segmented control): leaderboard, achievements, challenges

https://developer.apple.com/documentation/gamekit/
https://developer.apple.com/documentation/gamekit/gkleaderboard
https://developer.apple.com/documentation/gamekit/gkachievement

https://developer.apple.com/design/resources/


Compute and Network services return resource IDs with different casing

Describe the bug
When running az network nic list --query "[?virtualMachine.id=='<ID>']" the CLI processes case-sensitive.
To Reproduce
You'd need to look up Private IP Address by VM ID
vmList=$(az vm list --query "[?location=='westus']".id -o tsv)
for vmId in $vmList; do
   echo "$vmId"
   az network nic list --query "[?virtualMachine.id=='$vmId']".ipConfigurations[].privateIpAddress
done

Expected behavior
It should not handle case-sensitive.
Environment summary
Azure CLI version 2.0.43

The name of the resource groups returned through a VM List operation will always be upper-cased. This is the currently designed and expected behavior for VM List when no resource group is provided. When a resource group is provided then the response will use the casing of the resource group that was provided in the API.
The Resource URIs are not case sensitive. So when comparing any two resource URIs to determine if they are the same resource, then we recommend using a to_upper() or to_lower() function to get consistent casing of the URIs before the compare. Azure does not support two URIs that are exactly the same except for casing.


Improve lsp-ui-peek support

In doom emacs, activating the lsp tool means that lsp-ui is used as well.
The current version of the themes, specifically tomorrow-night which i am using (so not 100% sure about the other themes) has problems such as:

The peek layout is not distinguishable (background, border) from the buffer containing it.
peek highlight face isn't set on all of the themes apart from acario-(dark|light).

Relevant faces :
https://github.com/hlissner/emacs-doom-themes/blob/1f51ed72282fffc0459dd3e2d059592e8a62c3b6/themes/doom-acario-dark-theme.el#L243-L245
I think a :foreground highlight for lsp-ui-peek-highlight would be a nice default, and for the sideline-symbol-info, reverting the doom-blend call (comments bg 0.85) to have a better fallback in 256-color environments would be enough imo

It is actually a problem because now when I switch themes I have very bad colors for the lsp-ui features

@ema2159 Can you edit the title as it's related to lsp-ui-sideline and not really lsp-ui-peek ?

I don't use lsp that much. I've noticed there's a lot of lsp faces missing in the base file though, I'll need to give it a check soon.

I added a few there https://github.com/hlissner/emacs-doom-themes/pull/412

Can be closed (by #412  )


500 page

General
Closes #9
Implemented a new page that can be displayed on critical errors.

I have resolved the conflicts and removed unnecessary code, please observe my last commits on this branch.
I expect that you check every commit such that you do not check in files that have only been reformatted anmd remove debug statements.

@sschiessl-bcp
I have checked commits after the merge and how "Page_500" component is working after an error is thrown. And I have found out that it is broken. For fixing it: I modified "componentDidCatch" method in "AppInit" component, and rewrote "Page_500" component.


Use Pipenv

Use pipenv instead of pip+virtualenv to simplfiy the workflow. Pipenv is now the standard tool recommended by the python community.
https://pipenv.readthedocs.io/en/latest/

 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.

Travis was fixed indeed lol

Thank you.
There is another PR for pipenv support here: https://github.com/duo-labs/cloudmapper/pull/160
I unfortunately haven't had a chance to look into it yet.  My schedule should be clearing up in a week for me to look into these.

Sure :)
I was a bit more strict though :) I've removed the requirements files and fixed the Travis support :)
You'll decide!

The whole ecosystem is moving to pipenv...also, it was tested :)

I'll try to explain. I have a few PyPi packages that do the same thing
The setup.py is used for PyPi packaging, Pipfile is used for development:

Pipfile provide a kind of high quality replacement for the requirements.txt and requirements-dev.txt and provide also much more for the maintainers if this code.
To ship the package to PyPi and provide command line util out-of-the-box, you still must use the setup.py file to declare your dependencies and define console_scripts for running cloudtracker to work from the command line.

The best practice I've found so far is as follows: define the dev dependencies in the Pipfile, and the actual dependencies in the setup.py file. Then install the package with -e into the Pipfile.
The end result in my projects it looks like this: Pipfile and setup.py. You can see that this package (algosec) I maintain, is also published to PyPi.
Let me know if you need more help, it should be pretty straight forward.

You can see that the Pipfile there declares only the dev dependencies, and depend upon the setup.py for regular dependencies. That way I have minimal efforts to keep it all in sync.

Thank you for the explanation.  Sounds like the best practice is to provide both, so if I move CloudTracker to pypi in the future, this pipenv work is still both usable and desired to keep around.

Yes. This is the best practice and way superior to the requirements.txt
files. For packages like your I also don't commit the Pipfile.lock file.
It is better mostly because the guys of Pipenv made a great job to make it
very comfortable tool for developers. Just read about it quickly and you
will see.
On Thu, Sep 20, 2018 at 9:37 AM Scott Piper<EMAIL_ADDRESS>wrote:

Thank you for the explanation. Sounds like the best practice is to provide
both, so if I move CloudTracker to pypi in the future, this pipenv work is
still both usable and desired to keep around.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/duo-labs/cloudmapper/pull/187#issuecomment-423250418,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADtXZnIIVAXVAS5bZ6hZoG-bJOT7rrSvks5uc8QzgaJpZM4WeXo_
.

--
Almog Cohen
https://htmlsig.com/signatures/01ADE8F6/edit
[image: Linkedin] http://linkedin.com/in/almogcohen
US:<PHONE_NUMBER>

I've merged the changes up until today into this branch so you can accept and merge it. (I've added numpy to the pipfile and merged all previous changes to the repo).

There are a few more changes and improvements you can keep doing after that, but it would be a good starting point for the repo.


Bazel ERROR

Hello,
After following the instructions for compiling the C++ code in Docker, I get the following Bazel error:
`$ docker build -t qcc4cp:latest .
DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/
Sending build context to Docker daemon  220.1MB
Step 1/27 : FROM debian:11
---> 873181a41b99
Step 2/27 : LABEL maintainer="Abdolhamid Pourghazi<EMAIL_ADDRESS>---> Using cache
---> 883cfe428df3
Step 3/27 : LABEL maintainer="Stefan Klessinger<EMAIL_ADDRESS>---> Using cache
---> 1bba1a411a0b
Step 4/27 : ENV DEBIAN_FRONTEND noninteractive
---> Using cache
---> de5d6fb41216
Step 5/27 : ENV LANG="C.UTF-8"
---> Using cache
---><PHONE_NUMBER>b9
Step 6/27 : ENV LC_ALL="C.UTF-8"
---> Using cache
---> f3bd7ed972c8
Step 7/27 : RUN apt-get update && apt-get install -y --no-install-recommends            build-essential                 git             libpython3-dev          python3                 python3-pip             wget
---> Using cache
---> c058f7ccc312
Step 8/27 : RUN python3 -m pip install absl-py numpy scipy
---> Using cache
---> 3ae8eb401763
Step 9/27 : RUN useradd -m -G sudo -s /bin/bash repro && echo "repro:repro" | chpasswd
---> Using cache
---> b6b00da09dd4
Step 10/27 : USER repro
---> Using cache
---> eb773b7aa90a
Step 11/27 : RUN mkdir -p /home/repro/bin/
---> Using cache
---> 5119a047c1a3
Step 12/27 : RUN wget -O /home/repro/bin/bazel  https://github.com/bazelbuild/bazelisk/releases/latest/download/bazelisk-linux-amd64
---> Using cache
---> a7f664877276
Step 13/27 : RUN chmod +x /home/repro/bin/bazel
---> Using cache
---> 8c4c771990c8
Step 14/27 : ENV PATH="/home/repro/bin/:${PATH}"
---> Using cache
---> c51c34440086
Step 15/27 : RUN mkdir -p /home/repro/sources/
---> Using cache
---> 3a898b65d78f
Step 16/27 : WORKDIR /home/repro/sources/
---> Using cache
---> 8439b2897039
Step 17/27 : RUN git clone https://github.com/qcc4cp/qcc.git
---> Using cache
---> ba3894a520d1
Step 18/27 : WORKDIR /home/repro/sources/qcc
---> Using cache
---> 58b7d18fe6fb
Step 19/27 : RUN sed -i 's/python3.7/python3.9/g' WORKSPACE
---> Using cache
---> 67ee9c8e5681
Step 20/27 : WORKDIR /home/repro/sources/qcc/src/lib
---> Using cache
---> dfca1d6daa5d
Step 21/27 : RUN bazel build all
---> Running in d09b99177dde
2023/12/18 14:16:12 Downloading https://releases.bazel.build/7.0.0/release/bazel-7.0.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping:
WARNING: --enable_bzlmod is set, but no MODULE.bazel file was found at the workspace root. Bazel will create an empty MODULE.bazel file. Please consider migrating your external dependencies from WORKSPACE to MODULE.bazel. For more details, please refer to https://github.com/bazelbuild/bazel/issues/18958.
Computing main repo mapping:
Computing main repo mapping:
ERROR: /home/repro/sources/qcc/WORKSPACE:1:21: //external:third_party_python: invalid label '/home/repro/sources/qcc/python.BUILD' in attribute 'build_file' in 'new_local_repository' rule: invalid target name '/home/repro/sources/qcc/python.BUILD': target names may not start with '/'
ERROR: /home/repro/sources/qcc/WORKSPACE:8:21: //external:third_party_numpy: invalid label '/home/repro/sources/qcc/numpy.BUILD' in attribute 'build_file' in 'new_local_repository' rule: invalid target name '/home/repro/sources/qcc/numpy.BUILD': target names may not start with '/'
ERROR: Error computing the main repository mapping: Encountered error while reading extension file 'toolchains/local_java_repository.bzl': no such package '@@rules_java_builtin//toolchains': error loading package 'external': Could not load //external package
The command '/bin/sh -c bazel build all' returned a non-zero code: 1
$ `
I get the exact same error when running the compilation outside of Docker following README.Linux.md. I am on Ubuntu 22.04.
I am not a Bazel user and Google so far didn't help.

Yikes, Thanks for letting me know.
Something appears to be wrong with the *.BUILD files... I'll investigate.
In the meantime, you can always run the individual Python files directly
(after setting PYTHONPATH accordingly). Things may just run a little
slower. Perhaps take a look at the first few paragraphs of:
https://github.com/qcc4cp/qcc/blob/main/resources/quickstart.md
I'll keep you updated.
R
On Mon, Dec 18, 2023 at 6:50 AM smartvinnetou @.***>
wrote:

Hello,
After following the instructions for compiling the C++ code in Docker, I
get the following Bazel error:
`$ docker build -t qcc4cp:latest .
DEPRECATED: The legacy builder is deprecated and will be removed in a
future release.
Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/
Sending build context to Docker daemon 220.1MB
Step 1/27 : FROM debian:11
---> 873181a41b99
Step 2/27 : LABEL maintainer="Abdolhamid Pourghazi
@."
---> Using cache
---> 883cfe428df3
Step 3/27 : LABEL maintainer="Stefan Klessinger
@."
---> Using cache
---> 1bba1a411a0b
Step 4/27 : ENV DEBIAN_FRONTEND noninteractive
---> Using cache
---> de5d6fb41216
Step 5/27 : ENV LANG="C.UTF-8"
---> Using cache
---><PHONE_NUMBER>b9
Step 6/27 : ENV LC_ALL="C.UTF-8"
---> Using cache
---> f3bd7ed972c8
Step 7/27 : RUN apt-get update && apt-get install -y
--no-install-recommends build-essential git libpython3-dev python3
python3-pip wget
---> Using cache
---> c058f7ccc312
Step 8/27 : RUN python3 -m pip install absl-py numpy scipy
---> Using cache
---> 3ae8eb401763
Step 9/27 : RUN useradd -m -G sudo -s /bin/bash repro && echo
"repro:repro" | chpasswd
---> Using cache
---> b6b00da09dd4
Step 10/27 : USER repro
---> Using cache
---> eb773b7aa90a
Step 11/27 : RUN mkdir -p /home/repro/bin/
---> Using cache
---> 5119a047c1a3
Step 12/27 : RUN wget -O /home/repro/bin/bazel
https://github.com/bazelbuild/bazelisk/releases/latest/download/bazelisk-linux-amd64
---> Using cache
---> a7f664877276
Step 13/27 : RUN chmod +x /home/repro/bin/bazel
---> Using cache
---> 8c4c771990c8
Step 14/27 : ENV PATH="/home/repro/bin/:${PATH}"
---> Using cache
---> c51c34440086
Step 15/27 : RUN mkdir -p /home/repro/sources/
---> Using cache
---> 3a898b65d78f
Step 16/27 : WORKDIR /home/repro/sources/
---> Using cache
---> 8439b2897039
Step 17/27 : RUN git clone https://github.com/qcc4cp/qcc.git
---> Using cache
---> ba3894a520d1
Step 18/27 : WORKDIR /home/repro/sources/qcc
---> Using cache
---> 58b7d18fe6fb
Step 19/27 : RUN sed -i 's/python3.7/python3.9/g' WORKSPACE
---> Using cache
---> 67ee9c8e5681
Step 20/27 : WORKDIR /home/repro/sources/qcc/src/lib
---> Using cache
---> dfca1d6daa5d
Step 21/27 : RUN bazel build all
---> Running in d09b99177dde
2023/12/18 14:16:12 Downloading
https://releases.bazel.build/7.0.0/release/bazel-7.0.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping:
WARNING: --enable_bzlmod is set, but no MODULE.bazel file was found at the
workspace root. Bazel will create an empty MODULE.bazel file. Please
consider migrating your external dependencies from WORKSPACE to
MODULE.bazel. For more details, please refer to bazelbuild/bazel#18958
https://github.com/bazelbuild/bazel/issues/18958.
Computing main repo mapping:
Computing main repo mapping:
ERROR: /home/repro/sources/qcc/WORKSPACE:1:21:
//external:third_party_python: invalid label
'/home/repro/sources/qcc/python.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/python.BUILD': target names may not start with '/'
ERROR: /home/repro/sources/qcc/WORKSPACE:8:21:
//external:third_party_numpy: invalid label
'/home/repro/sources/qcc/numpy.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/numpy.BUILD': target names may not start with '/'
ERROR: Error computing the main repository mapping: Encountered error
while reading extension file 'toolchains/local_java_repository.bzl': no
such package '@@rules_java_builtin//toolchains': error loading package
'external': Could not load //external package
The command '/bin/sh -c bazel build all' returned a non-zero code: 1
$ `
I get the exact same error when running the compilation outside of Docker
following README.Linux.md. I am on Ubuntu 22.04.
I am not a Bazel user and Google so far didn't help.
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASSDND4WU6INLTEHIV7HNQLYKBJZVAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DMOBTGYZTOMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Robert Hundt

Could you check which Python version you see in the docker container? The
docker file seems to expect 3.9 but you may have a more updated version.
The Python versions are sort of encoded in the WORKSPACE file:
new_local_repository(
name = "third_party_python",
build_file = workspace_dir + "/python.BUILD",
# Configure:
path = "/usr/include/python3.10",
)
new_local_repository(
name = "third_party_numpy",
build_file = workspace_dir + "/numpy.BUILD",
# Configure:
path = "/usr/local/lib/python3.7/dist-packages/numpy/core/",
)
You could try to manually edit those to point to the proper python
versions. I currently have problems even installing docker... I'll follow
up.
R
On Mon, Dec 18, 2023 at 7:23 AM Robert Hundt @.***>
wrote:

Yikes, Thanks for letting me know.
Something appears to be wrong with the *.BUILD files... I'll investigate.
In the meantime, you can always run the individual Python files directly
(after setting PYTHONPATH accordingly). Things may just run a little
slower. Perhaps take a look at the first few paragraphs of:
https://github.com/qcc4cp/qcc/blob/main/resources/quickstart.md
I'll keep you updated.
R
On Mon, Dec 18, 2023 at 6:50 AM smartvinnetou @.***>
wrote:

Hello,
After following the instructions for compiling the C++ code in Docker, I
get the following Bazel error:
`$ docker build -t qcc4cp:latest .
DEPRECATED: The legacy builder is deprecated and will be removed in a
future release.
Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/
Sending build context to Docker daemon 220.1MB
Step 1/27 : FROM debian:11
---> 873181a41b99
Step 2/27 : LABEL maintainer="Abdolhamid Pourghazi
@."
---> Using cache
---> 883cfe428df3
Step 3/27 : LABEL maintainer="Stefan Klessinger
@."
---> Using cache
---> 1bba1a411a0b
Step 4/27 : ENV DEBIAN_FRONTEND noninteractive
---> Using cache
---> de5d6fb41216
Step 5/27 : ENV LANG="C.UTF-8"
---> Using cache
---><PHONE_NUMBER>b9
Step 6/27 : ENV LC_ALL="C.UTF-8"
---> Using cache
---> f3bd7ed972c8
Step 7/27 : RUN apt-get update && apt-get install -y
--no-install-recommends build-essential git libpython3-dev python3
python3-pip wget
---> Using cache
---> c058f7ccc312
Step 8/27 : RUN python3 -m pip install absl-py numpy scipy
---> Using cache
---> 3ae8eb401763
Step 9/27 : RUN useradd -m -G sudo -s /bin/bash repro && echo
"repro:repro" | chpasswd
---> Using cache
---> b6b00da09dd4
Step 10/27 : USER repro
---> Using cache
---> eb773b7aa90a
Step 11/27 : RUN mkdir -p /home/repro/bin/
---> Using cache
---> 5119a047c1a3
Step 12/27 : RUN wget -O /home/repro/bin/bazel

https://github.com/bazelbuild/bazelisk/releases/latest/download/bazelisk-linux-amd64

---> Using cache
---> a7f664877276
Step 13/27 : RUN chmod +x /home/repro/bin/bazel
---> Using cache
---> 8c4c771990c8
Step 14/27 : ENV PATH="/home/repro/bin/:${PATH}"
---> Using cache
---> c51c34440086
Step 15/27 : RUN mkdir -p /home/repro/sources/
---> Using cache
---> 3a898b65d78f
Step 16/27 : WORKDIR /home/repro/sources/
---> Using cache
---> 8439b2897039
Step 17/27 : RUN git clone https://github.com/qcc4cp/qcc.git
---> Using cache
---> ba3894a520d1
Step 18/27 : WORKDIR /home/repro/sources/qcc
---> Using cache
---> 58b7d18fe6fb
Step 19/27 : RUN sed -i 's/python3.7/python3.9/g' WORKSPACE
---> Using cache
---> 67ee9c8e5681
Step 20/27 : WORKDIR /home/repro/sources/qcc/src/lib
---> Using cache
---> dfca1d6daa5d
Step 21/27 : RUN bazel build all
---> Running in d09b99177dde
2023/12/18 14:16:12 Downloading
https://releases.bazel.build/7.0.0/release/bazel-7.0.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping:
WARNING: --enable_bzlmod is set, but no MODULE.bazel file was found at
the
workspace root. Bazel will create an empty MODULE.bazel file. Please
consider migrating your external dependencies from WORKSPACE to
MODULE.bazel. For more details, please refer to bazelbuild/bazel#18958
https://github.com/bazelbuild/bazel/issues/18958.
Computing main repo mapping:
Computing main repo mapping:
ERROR: /home/repro/sources/qcc/WORKSPACE:1:21:
//external:third_party_python: invalid label
'/home/repro/sources/qcc/python.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/python.BUILD': target names may not start with
'/'
ERROR: /home/repro/sources/qcc/WORKSPACE:8:21:
//external:third_party_numpy: invalid label
'/home/repro/sources/qcc/numpy.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/numpy.BUILD': target names may not start with
'/'
ERROR: Error computing the main repository mapping: Encountered error
while reading extension file 'toolchains/local_java_repository.bzl': no
such package '@@rules_java_builtin//toolchains': error loading package
'external': Could not load //external package
The command '/bin/sh -c bazel build all' returned a non-zero code: 1
$ `
I get the exact same error when running the compilation outside of
Docker
following README.Linux.md. I am on Ubuntu 22.04.
I am not a Bazel user and Google so far didn't help.
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ASSDND4WU6INLTEHIV7HNQLYKBJZVAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DMOBTGYZTOMY>

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Robert Hundt
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20#issuecomment-1860802180, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEWY3BGL2K4YZJOAJ5GET3YKBNYZAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRQHAYDEMJYGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Robert Hundt

I use Ubuntu 22.04 with Python 3.10.12. When compiling outside of docker I
change the paths to ones that exist on my system. When compiling inside
Docker I leave the paths as they come from GitHub and let the Dockefile to
modify them.
My Bazel version is 7.0.0, which is a major upgrade that came out only last
week according to the release notes. Was there perhaps a backwards
incompatible change in what the new Bazel expects on its input?
On Tue, 19 Dec 2023 at 21:17, Robert Hundt @.***> wrote:

Could you check which Python version you see in the docker container? The
docker file seems to expect 3.9 but you may have a more updated version.
The Python versions are sort of encoded in the WORKSPACE file:
new_local_repository(
name = "third_party_python",
build_file = workspace_dir + "/python.BUILD",
Configure:
path = "/usr/include/python3.10",
)
new_local_repository(
name = "third_party_numpy",
build_file = workspace_dir + "/numpy.BUILD",
Configure:
path = "/usr/local/lib/python3.7/dist-packages/numpy/core/",
)
You could try to manually edit those to point to the proper python
versions. I currently have problems even installing docker... I'll follow
up.
R
On Mon, Dec 18, 2023 at 7:23 AM Robert Hundt @.***>
wrote:

Yikes, Thanks for letting me know.
Something appears to be wrong with the *.BUILD files... I'll
investigate.
In the meantime, you can always run the individual Python files directly
(after setting PYTHONPATH accordingly). Things may just run a little
slower. Perhaps take a look at the first few paragraphs of:
https://github.com/qcc4cp/qcc/blob/main/resources/quickstart.md
I'll keep you updated.
R
On Mon, Dec 18, 2023 at 6:50 AM smartvinnetou @.***>
wrote:

Hello,
After following the instructions for compiling the C++ code in Docker,
I
get the following Bazel error:
`$ docker build -t qcc4cp:latest .
DEPRECATED: The legacy builder is deprecated and will be removed in a
future release.
Install the buildx component to build images with BuildKit:
https://docs.docker.com/go/buildx/
Sending build context to Docker daemon 220.1MB
Step 1/27 : FROM debian:11
---> 873181a41b99
Step 2/27 : LABEL maintainer="Abdolhamid Pourghazi
@."
---> Using cache
---> 883cfe428df3
Step 3/27 : LABEL maintainer="Stefan Klessinger
@."
---> Using cache
---> 1bba1a411a0b
Step 4/27 : ENV DEBIAN_FRONTEND noninteractive
---> Using cache
---> de5d6fb41216
Step 5/27 : ENV LANG="C.UTF-8"
---> Using cache
---><PHONE_NUMBER>b9
Step 6/27 : ENV LC_ALL="C.UTF-8"
---> Using cache
---> f3bd7ed972c8
Step 7/27 : RUN apt-get update && apt-get install -y
--no-install-recommends build-essential git libpython3-dev python3
python3-pip wget
---> Using cache
---> c058f7ccc312
Step 8/27 : RUN python3 -m pip install absl-py numpy scipy
---> Using cache
---> 3ae8eb401763
Step 9/27 : RUN useradd -m -G sudo -s /bin/bash repro && echo
"repro:repro" | chpasswd
---> Using cache
---> b6b00da09dd4
Step 10/27 : USER repro
---> Using cache
---> eb773b7aa90a
Step 11/27 : RUN mkdir -p /home/repro/bin/
---> Using cache
---> 5119a047c1a3
Step 12/27 : RUN wget -O /home/repro/bin/bazel

https://github.com/bazelbuild/bazelisk/releases/latest/download/bazelisk-linux-amd64

---> Using cache
---> a7f664877276
Step 13/27 : RUN chmod +x /home/repro/bin/bazel
---> Using cache
---> 8c4c771990c8
Step 14/27 : ENV PATH="/home/repro/bin/:${PATH}"
---> Using cache
---> c51c34440086
Step 15/27 : RUN mkdir -p /home/repro/sources/
---> Using cache
---> 3a898b65d78f
Step 16/27 : WORKDIR /home/repro/sources/
---> Using cache
---> 8439b2897039
Step 17/27 : RUN git clone https://github.com/qcc4cp/qcc.git
---> Using cache
---> ba3894a520d1
Step 18/27 : WORKDIR /home/repro/sources/qcc
---> Using cache
---> 58b7d18fe6fb
Step 19/27 : RUN sed -i 's/python3.7/python3.9/g' WORKSPACE
---> Using cache
---> 67ee9c8e5681
Step 20/27 : WORKDIR /home/repro/sources/qcc/src/lib
---> Using cache
---> dfca1d6daa5d
Step 21/27 : RUN bazel build all
---> Running in d09b99177dde
2023/12/18 14:16:12 Downloading
https://releases.bazel.build/7.0.0/release/bazel-7.0.0-linux-x86_64...

Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping:
WARNING: --enable_bzlmod is set, but no MODULE.bazel file was found at
the
workspace root. Bazel will create an empty MODULE.bazel file. Please
consider migrating your external dependencies from WORKSPACE to
MODULE.bazel. For more details, please refer to bazelbuild/bazel#18958
https://github.com/bazelbuild/bazel/issues/18958.
Computing main repo mapping:
Computing main repo mapping:
ERROR: /home/repro/sources/qcc/WORKSPACE:1:21:
//external:third_party_python: invalid label
'/home/repro/sources/qcc/python.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/python.BUILD': target names may not start
with
'/'
ERROR: /home/repro/sources/qcc/WORKSPACE:8:21:
//external:third_party_numpy: invalid label
'/home/repro/sources/qcc/numpy.BUILD' in attribute 'build_file' in
'new_local_repository' rule: invalid target name
'/home/repro/sources/qcc/numpy.BUILD': target names may not start with
'/'
ERROR: Error computing the main repository mapping: Encountered error
while reading extension file 'toolchains/local_java_repository.bzl':
no
such package '@@rules_java_builtin//toolchains': error loading package
'external': Could not load //external package
The command '/bin/sh -c bazel build all' returned a non-zero code: 1
$ `
I get the exact same error when running the compilation outside of
Docker
following README.Linux.md. I am on Ubuntu 22.04.
I am not a Bazel user and Google so far didn't help.
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20, or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ASSDND4WU6INLTEHIV7HNQLYKBJZVAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DMOBTGYZTOMY>

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

--
Robert Hundt
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20#issuecomment-1860802180, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABEWY3BGL2K4YZJOAJ5GET3YKBNYZAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRQHAYDEMJYGA>

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Robert Hundt
—
Reply to this email directly, view it on GitHub
https://github.com/qcc4cp/qcc/issues/20#issuecomment-1863484429, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOSDPMXDAUR26HXCCO33YQLYKH75VAVCNFSM6AAAAABAZUXNSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRTGQ4DINBSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Tracker Index Issue

Hello,
Im using two trackers on my unity project which i have set the device index to 3 and 4. However whenever I unpair one tracker and then pair a controller and unpair that controller to pair tracker, the tracker device index isnt 3 or 4. Is there anyway I can override SteamVr Indexing for the tracked objects or at least get the index of the paired controller and assign it? I have also tried the SetDeviceIndex(int index) it doesnt work like that neither

I have fixed this problem, if anyone else has this problem, don't hesitate to ask me :)

What was the cause fo the problem?


Auto-publish project to nuget

@azurecoder @sandy-may  Mostly a question as I don't know much about adla deployment. From what I've seen deployment happens by uploading DLLs to the server. Do we need nuget here at all in this case?


fix: svg props.

:tada: This PR is included in version 1.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Set per-address address settings to limit max size

(This requires using a connector per address, created on the broker after the queue is created,
in order to ensure the autolinks do not fire before address settings and queue have been created
explicitly, as triggering the links will cause implicit creation of queue)

Closes #973


Inference on a trained model.

Hello,
I have trained the TrimNet on the clintox dataset on the FDA_APPROVED task and it is not very obvious how to inference on new string data, can you show how to infer the model on a single SMILE string?
Thank you

If you have trained your model, you will have an triner(instance) of Trainer (class) in trainer.py.
Then, try this, I write this in text model, so, there may have bugs in these lines of code
`
from dataset import atom_attr, bond_attr
mol = Chem.MolFromSMILES(smiles)
node_attr = atom_attr(mol)
edge_index, edge_attr = bond_attr(mol)
data = Data(
        x=torch.FloatTensor(node_attr),
        # pos=pos,
        edge_index=torch.LongTensor(edge_index).t(),
        edge_attr=torch.FloatTensor(edge_attr),
        y=None  # None as a placeholder
  )
from torch_geometric.data import Batch
batch = Batch().from_data_list([data])
out = trainer.model(batch)

`

@yvquanli Thank you so much.
One question though: why is the model's output is a size of 2 ? I have only one task FDA_APPROVED, shouldn't it be a binary classification? I thought the last layer is a softmax but apparently, the output doesn't sum up to 1 (I checked your paper quickly and I didn't find a clue to the answer)

This is because the output actually is the logits you can change it to the final prediction by
probs=torch.softmax(logits)
pred=torch.max(probs)

Thank you, this is helpful!


Datepicker: "...ng-model" value must be..." on Clear action

Open http://angular-ui.github.io/bootstrap/
Ctrl+Datepicker
Open popup datepicker
Choose any date
Open popup datepicker again
Click 'Clear' button
=>
In console you'll see error "Datepicker directive: "ng-model" value must be a Date object, a number of milliseconds since 01.01.1970 or a string representing an RFC2822 or ISO 8601 date."

I have not expected any errors on clearing date (setting it to null).
At least in 0.12.1 I had not have any errors.

Having the same issue. Using angular 1.3.15.

I have this problem too and have had to downgrade to an older version to support nullable dates. Any other workaround?

Same issue with 0.13.0, but good with 0.12.1

I am running Angular 1.3.16 and get the same error message upon clicking 'clear' in the datepicker modal.
What makes it worse is that I am checking my forms $invalid property which is returning true even though the datepicker field in my form is not a required field, so a blank datepicker field should be valid, not invalid.

Probably worth mentioning the following:

If you open up firebug while viewing the DatePicker demo at https://angular-ui.github.io/bootstrap/, it throws the error.
In the example shown at the link, a clear function is set in $scope; however, in my setup, which is identical to that, the clear function is never called. Perhaps because the error is thrown prior to the clear function being called? I thought the clear function is called first if you click the 'clear' button on the datepicker modal, but that doesn't appear to be the case.

I have created a plunker, with documentation that describes the various problems related to this bug:
http://plnkr.co/edit/Iild15cgR9NXaX0iWjti?p=preview
This example aims to highlight the issue isn't just with the datepicker's model being cleared, but also how it affects form validation.

:+1: Same issue here.

+1

@askucher, please do not simply add +1 to issues as it adds no intrinsic value to the discussion and serves to only clutter up the thread.  That said, I'm not sure what you're trying to imply with your support of this issue as we've made the switch to only support JS Date objects due to how different browsers handle dates differently.

+1

@saurabh67, did you even read this thread?  the last post in this issue specifically asks you not do what you did.  you may use the response emojis to specify a reaction.  in the future, please be respectful of our time and processes by not being lazy.

It's not clear if this has been resolved. I'm still seeing the issue in Angular UI-Bootstrap 2.2.0. @icfantv When you say you've made the switch to only support JS Date objects are you saying that disallowing null as valid input for the datepicker is by design? Is there a workaround for date fields that are not required?


Memory Optimization Pass

Memory Optimization Pass
Use std::string_view in place of std::string in SymbolTable.
Also, optimize the storage by ensuring we don't store multiple
copies of same string.

This should be broken in down in various changes to be reviewable each addressing one of the things

Start with the symbol table, just change the internal implementation if that is the core of the improvement. If there are string_views first to be returned, convert them to std::string when returning to keep the interface. So the firt change should only include the SymbolTable, its header and additions to the tests. Describe how the change will improve memory usage.
There seems to be changes adding mutexes implying some multi-threaded changes. Do that in a separate change.
There are other changes throughout that should also be addressed separately (e.g. avoiding string copies and instead assigning things to const std::string&. This is entirely separate aspect and should be broken into its own change.

Start with the symbol table, just change the internal implementation if that is the core of the improvement. If there are string_views first to be returned, convert them to std::string when returning to keep the interface. So the firt change should only include the SymbolTable, its header and additions to the tests. Describe how the change will improve memory usage.

Current implementation of SymbolFactory returns a reference to a std::string and calling site is usually holding this reference (correct/expected behavior) instead of cloning the returned string. Returning a std::string if I am storing a std::string_view internally would be reference to local variable. Plus, std::string_view to std::string is not an implicit conversion and so I can't just change the API's return type. Bottom-line, I don't think this transition can happen without touching every site where the SymbolFactory is queried. How do propose changing SymbolFactory alone would work?

There seems to be changes adding mutexes implying some multi-threaded changes. Do that in a separate change.

This is implementing the internal storage - multiple read, single write operation - since the storage itself is shared across multiple instances of SymbolFactory. This is the core of the SymbolFactory change.
I will address each of the concern raised individually.

Can you quantify the savings impact ?
Even if we have 1M symbols and would have an overhead of one allocated string per symbol instead of pointing into a congiguous area, we're still only talking about maybe 12 MiB overhad (or whatever sizeof(std::string) is) vs. this approach of having a few contigjuous areas.

The savings are multi-fold because the storage is shared across SymbolFactory instances. At least, that's the intent here. The cache backing is a fall-out from what I noticed when debugging black-parrot regression test. The SymbolFactory instance gets cloned nearly 600+ times so each string in the original SymbolFactory is cloned nearly that many times. Shared storage avoid that. The contiguous block itself is to prevent fragmentation when working with larger designs (and avoid putting all the burden on global allocator like tcmalloc).

I think we should address the string copies all over first by actually making the rest of the code base ready: have all parameters that currently accept const std::string & (or even std::string) accept string_view.
Then, it will be much easier to also have functions to return string_view as they can be processed by other places. At that stage, a change of SymbolTable to return string-views will be hopefully not requring much effort.

May be coming from the other end was better, or may be not. Either way most if not all of these files would be affected. I took the approach of attack the root and see what fails and fix that. May be other approach, attacking a class, a file, would have been granular but I wouldn't have been able to measure the overall impact of the change - which I am able to do now and I see some negatives that needs addressing. There are pros and cons to either approach - I just happen to choose one.

Attached is the complete report of comparison of two Surelog/master builds with two Regression builds that include this change. And, below is the quick summary (selected all tests which report over 1GB memory use).

Negatives in below table are savings in both time and memory
PyGen generated code reports faster runtime (relative to Tcl generated code) when compared to current master runtime.
Memory savings are not very high even for large tests. In some cases (like Google & YosysTests) the consumption is considerably higher. No memory leaks found in this change.
Interesting edits in this change are primarily in SymbolTable implementation.
Implementation includes no mutexes

Surelog/master builds: 2442 & 2452
Regression builds: 17 & 18

                            |                 |                 |    |                      |                       |                       |                         |    |    Surelog: 2442     |    Surelog: 2442      |    |     Surelog: 2452    |     Surelog: 2452     |    |    Surelog: 2442     |     Surelog: 2442     |    |    Surelog: 2452     |     Surelog: 2452     |
                            |  Surelog: 2442  |  Surelog: 2452  |    | PyGen Regression: 17 | TclGen Reggresion: 17 | PyGen Reggresion: 18  | TclGen Reggression: 18  |    |        vs.           |           vs.         |    |          vs.         |          vs.          |    |        vs.           |          vs.          |    |         vs.          |          vs.          |
                            |                 |                 |    |                      |                       |                       |                         |    | PyGen Regression: 17 | TclGen Regression: 17 |    | PyGen Regression: 17 | TclGen Regression: 17 |    | PyGen Regression: 18 | TclGen Regression: 18 |    | PyGen Regression: 18 | TclGen Regression: 18 |
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TESTNAME                    | TIME  | MEM(Mb) | TIME  | MEM(Mb) |    |   TIME   |  MEM(Mb)  |   TIME   |   MEM(Mb)  |   TIME    |  MEM(Mb)  |    TIME    |  MEM(Mb)   |    |    TIME  |  MEM(Mb)  |    TIME   |  MEM(Mb)  |    |    TIME  | MEM(Mb)   |    TIME   |  MEM(Mb)  |    |    TIME  | MEM(Mb)   |    TIME   |  MEM(Mb)  |    |   TIME   |  MEM(Mb)  |    TIME   |  MEM(Mb)  |
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
AmiqEth                     |    16 |    1018 |    21 |    1014 |    |       15 |      1013 |       19 |       1008 |        15 |      1006 |         20 |      1008  |    |       -1 |        -5 |         3 |       -10 |    |       -6 |        -1 |        -2 |        -6 |    |       -1 |       -12 |         4 |       -10 |    |       -6 |        -8 |         -1|        -6 |
Ariane                      |    50 |    1434 |    62 |    1432 |    |       49 |      1426 |       60 |       1426 |        50 |      1425 |         60 |      1428  |    |       -1 |        -8 |        10 |        -8 |    |      -13 |        -6 |        -2 |        -6 |    |        0 |        -9 |        10 |        -6 |    |      -12 |        -7 |         -2|        -4 |
AzadiRTL                    |    43 |    1066 |    55 |    1069 |    |       43 |      1065 |       49 |       1066 |        45 |      1064 |         54 |      1066  |    |        0 |        -1 |         6 |         0 |    |      -12 |        -4 |        -6 |        -3 |    |        2 |        -2 |        11 |         0 |    |      -10 |        -5 |         -1|        -3 |
BasicOh                     |       |    1066 |       |    1069 |    |          |      1065 |          |       1066 |           |      1064 |            |      1066  |    |        0 |        -1 |         0 |         0 |    |        0 |        -4 |         0 |        -3 |    |        0 |        -2 |         0 |         0 |    |        0 |        -5 |          0|        -3 |
BuildUVMPkg                 |   217 |    1608 |   239 |    1614 |    |      216 |      1608 |      216 |       1610 |       218 |      1603 |        238 |      1610  |    |       -1 |         0 |        -1 |         2 |    |      -23 |        -6 |       -23 |        -4 |    |        1 |        -5 |        21 |         2 |    |      -21 |       -11 |         -1|        -4 |
Compl1001                   |   114 |    2564 |   134 |    2571 |    |      113 |      2565 |      128 |       2566 |       116 |      2565 |        132 |      2565  |    |       -1 |         1 |        14 |         2 |    |      -21 |        -6 |        -6 |        -5 |    |        2 |         1 |        18 |         1 |    |      -18 |        -6 |         -2|        -6 |
CoresSweRV                  |    35 |    1565 |    43 |    1566 |    |       32 |      1553 |       38 |       1551 |        33 |      1554 |         41 |      1552  |    |       -3 |       -12 |         3 |       -14 |    |      -11 |       -13 |        -5 |       -15 |    |       -2 |       -11 |         6 |       -13 |    |      -10 |       -12 |         -2|       -14 |
Earlgrey                    |    29 |    1377 |    37 |    1378 |    |       28 |      1378 |       32 |       1374 |        29 |      1372 |         35 |      1379  |    |       -1 |         1 |         3 |        -3 |    |       -9 |         0 |        -5 |        -4 |    |        0 |        -5 |         6 |         2 |    |       -8 |        -6 |         -2|         1 |
Earlgrey_0_1                |    41 |    1948 |    51 |    1945 |    |       38 |      1944 |       44 |       1947 |        39 |      1941 |         48 |      1943  |    |       -3 |        -4 |         3 |        -1 |    |      -13 |        -1 |        -7 |         2 |    |       -2 |        -7 |         7 |        -5 |    |      -12 |        -4 |         -3|        -2 |
Earlgrey_Verilator_0_1      |    43 |    2031 |    53 |    2031 |    |       39 |      2023 |       46 |       2025 |        41 |      2023 |         50 |      2021  |    |       -4 |        -8 |         3 |        -6 |    |      -14 |        -8 |        -7 |        -6 |    |       -2 |        -8 |         7 |       -10 |    |      -12 |        -8 |         -3|       -10 |
Earlgrey_Verilator_01_05_21 |    66 |    1096 |    84 |    1096 |    |       61 |      1136 |       72 |       1135 |        64 |      1138 |         78 |      1137  |    |       -5 |        40 |         6 |        39 |    |      -23 |        40 |       -12 |        39 |    |       -2 |        42 |        12 |        41 |    |      -20 |        42 |         -6|        41 |
Google                      |    93 |    1554 |   118 |    1558 |    |       95 |      1908 |      107 |       1866 |        96 |      1923 |        112 |      1820  |    |        2 |       354 |        14 |       312 |    |      -23 |       350 |       -11 |       308 |    |        3 |       369 |        19 |       266 |    |      -22 |       365 |         -6|       262 |
Ibex                        |    30 |    1188 |    37 |    1194 |    |       28 |      1188 |       35 |       1187 |        29 |      1186 |         35 |      1189  |    |       -2 |         0 |         5 |        -1 |    |       -9 |        -6 |        -2 |        -7 |    |       -1 |        -2 |         5 |         1 |    |       -8 |        -8 |         -2|        -5 |
IncompTitan                 |    50 |    1990 |    59 |    1988 |    |       45 |      2005 |       52 |       2006 |        46 |      2003 |         57 |      2007  |    |       -5 |        15 |         2 |        16 |    |      -14 |        17 |        -7 |        18 |    |       -4 |        13 |         7 |        17 |    |      -13 |        15 |         -2|        19 |
Opentitan                   |    39 |    1642 |    46 |    1643 |    |       35 |      1634 |       40 |       1642 |        36 |      1640 |         44 |      1639  |    |       -4 |        -8 |         1 |         0 |    |      -11 |        -9 |        -6 |        -1 |    |       -3 |        -2 |         5 |        -3 |    |      -10 |        -3 |         -2|        -4 |
Rsd                         |    69 |    3762 |    81 |    3763 |    |       58 |      3753 |       69 |       3756 |        61 |      3753 |         76 |      3752  |    |      -11 |        -9 |         0 |        -6 |    |      -23 |       -10 |       -12 |        -7 |    |       -8 |        -9 |         7 |       -10 |    |      -20 |       -10 |         -5|       -11 |
Unisim                      |   123 |    1166 |   145 |    1167 |    |      116 |      1260 |      136 |       1247 |       119 |      1250 |        143 |      1261  |    |       -7 |        94 |        13 |        81 |    |      -29 |        93 |        -9 |        80 |    |       -4 |        84 |        20 |        95 |    |      -26 |        83 |         -2|        94 |
YosysOpenSparc              |    61 |    4092 |    75 |    4065 |    |       61 |      4097 |       73 |       4074 |        62 |      4194 |         74 |      4106  |    |        0 |         5 |        12 |       -18 |    |      -14 |        32 |        -2 |         9 |    |        1 |       102 |        13 |        14 |    |      -13 |       129 |         -1|        41 |
YosysSmallBoom              |    32 |    3263 |    37 |    3270 |    |       29 |      3240 |       33 |       3297 |        29 |      3254 |         35 |      3243  |    |       -3 |       -23 |         1 |        34 |    |       -8 |       -30 |        -4 |        27 |    |       -3 |        -9 |         3 |       -20 |    |       -8 |       -16 |         -2|       -27 |
YosysTests                  |    50 |     825 |    59 |     828 |    |       47 |      1196 |       55 |       1329 |        48 |      1169 |         58 |      1322  |    |       -3 |       371 |         5 |       504 |    |      -12 |       368 |        -4 |       501 |    |       -2 |       344 |         8 |       497 |    |      -11 |       341 |         -1|       494 |

@hs-apotell I really don't understand the purpose of this changelist.
Some tests show very minor memory savings, but some tests show huge increase of memory. Does not look like a good trade at all.

My suspicion is that you are keeping handles to memory in the symbol table so it does not show as memory leaks. It shows as memory used at exit. That memory is accumulating for each design parsed in batch mode.

Has the memory blowout been eliminated?

Nope. Just re-based it. I spent another day on it yesterday but no go. I don't see what/how the memory could increase so much. There are absolutely no leaks and I have even compared the life-time of SymbolTable from master and this branch. They are identical. Nothing obvious stands out.
I am also seeing a similar pattern with another change I have in the works. It seems like allocating bigger blocks of memory significantly bumps up the peak memory usage. Can't justify it but that's what it seems.

Maybe using mmap would help
https://stackoverflow.com/questions/41256789/is-memory-malloc-through-mmap-returned-to-os-immediately-after-calling-free
Large chunks might not be released back to the OS with new/malloc.
On Fri, Oct 29, 2021 at 2:40 PM HS @.***> wrote:

Nope. Just re-based it. I spent another day on it yesterday but no go. I
don't see what/how the memory could increase so much. There are absolutely
no leaks and I have even compared the life-time of SymbolTable from master
and this branch. They are identical. Nothing obvious stands out.
I am also seeing a similar pattern with another change I have in the
works. It seems like allocating bigger blocks of memory significantly bumps
up the peak memory usage. Can't justify it but that's what it seems.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chipsalliance/Surelog/pull/2067#issuecomment-955063726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APFYJ5GY7RNAKG4BBZO6LO3UJMIFBANCNFSM5F6NMIRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I will give that a shot at least to confirm the hunch. Will figure out a more portable solution (mmap calls are not portable across platforms) if that happens to be the problem.

Abandoned!!!!


some default Bootstrap 4 styles are not being set when using BootstrapVersion

If Bootstrap's version isn't detected and BootstrapVersion needs to be used (or if it's loaded asynchronously), the options iconBase and tickIcon aren't being updated to reflect proper Bootstrap 4 styling.

Released in v1.13.9!


[METADATA UPDATE] Global metadata changes

This Pull Request was generated automatically as part of a global effort to improve metadata quality. Please review and merge this Pull Request within 5 days. If you have any questions or concerns about these metadata updates, please contact<EMAIL_ADDRESS>If you are not the correct contact for this content and repository, please notify<EMAIL_ADDRESS>Fixing#2022-10-05?workitem=593828 Remove ms.technology value ews
If this Pull Request is not merged within 14 days, it will be automatically merged by our system to ensure the timeliness of the metadata update.Please notify<EMAIL_ADDRESS>if you would like Pull Requests for metadata updates to merge automatically in future.
After you accept changes of metadata used in reports and they're merged, standard dashboards and reports will reflect new values, so you'll need to use new filter values.If you have any custom dashboards or Kusto queries, you might need to update those also.
Why we make metadata updates
Metadata updates are a normal part of business.For example, we update our metadata to reflect a new product name when a product is rebranded.Metadata updates require content to be updated because some metadata values(slugs) are included in the content(YAML header).
For metadata used internally, these updates make reports cleaner, easier to use, more consistent, and more reflective of current business.For metadata exposed to Learn Platform users, these updates ensure our users are exposed to up-to-date information consistent with their overall Microsoft experience.
Frequently Asked Questions
Why does this Pull Request appear to be made by the Repository Owner? We open Pull Requests two different ways.For Git Repos with a permissioned Service Account, we open the request from our org's Service Account.For Git Repos that we either do not have Service Account permission for or the repo is in Azure Dev Ops(ADO) we are able to open the Pull Requests in an automated way, but only with the PR creator as the Repo owner.

Learn Build status updates of commit 5bde400:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

officedocs-dev-exchange-ews-proxy/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Fix AWS SQS host example

According to my experience trying to get an SQS connection to work, there should only be one protocol in this string.

Thanks @ethanpooley for your contribution. The url_pattern is a mandatory property.... except for this one. I don't think we can apply a JSON schema logic to omit url_pattern only for this case.


RuleExecutive + StructuralRules

Another try to generalize some rules to an format which is easier to modify and understand and document.

StructuralRules.structuralCompose1()
StructuralRules.structuralDecompose1()

Indeed, the compiler for the rule-language can indeed emit source code. This code describes the inference rules with data structures and lambdas/classes.
The advantage is that it's still possible to modify the rules from the outside if necessary.

The unit test fail
/nal/application/toothbrush.nal @11550
OutputContainsCondition OK: [$0.3727;0.0003;0.5886$ (^lighter,{SELF},toothbrush)! %1.00;0.18%]
OutputContainsCondition FAIL: No substring match: (^reshape,{SELF},toothbrush)! %1.00;0.06%
similar(1): (^reshape,{SELF},toothbrush)! %1.00;0.09%
similar(2): (^reshape,{SELF},toothbrush)! %1.00;0.22%
similar(7): (^lighter,{SELF},toothbrush). %1.00;0.06%
similar(8): (^lighter,{SELF},toothbrush)! %1.00;0.12%
similar(13): (^reshape,{SELF},#1). %1.00;0.31%
[ERROR] Tests run: 202, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 145.139 s <<< FAILURE! - in org.opennars.core.NALTest

Ah I see


Spike - filter logic

Summary
The filter logic is central to this feature, and we must figure out the best way to apply the filter.
Acceptance Criteria

Write a doc on filter logic
The logic needs to handle individual filters or multiple filters.
It must also have to handle individual term changes to a set of filter terms.
Doc should include API design to handle new endpoint.

@joshgarza  fyi that we have shifted our approach to search and landed on the following functionality for the current milestone - just want to make sure you're aware as you're working on this ticket. CC @TylerHendrickson
Searching is like writing an old, MS Word file that doesn’t have autosave. Just as you can:

Create a document
Open an existing document
test what you want to write in that document
save the document with the changes you like
Delete the document

so too you can

create a new search
Open an existing search
test out the search
Save the search with the changes you like
Delete a search

Blocking front end advanced search - primarily #1404


@ArraySchema is ignored in model property (array of abstract class)

I have the following class that returns an array of abstract classes that looks like this:
@JsonPropertyOrder({"answers"})
@Schema(name = QuestionAnswers, description = "")
public final class QuestionAnswersJson extends AbstractJson {
  @ArraySchema(schema = @Schema(oneOf = {XAnswerJson.class, YAnswerJson.class, //
      ZAnswerJson.class, BAnswerJson.class}))
  private List<AbstractAnswerJson> answers = new ArrayList<>();

  public QuestionAnswersJson(@Nonnull List<AbstractAnswerJson> answers) {
    answers = Preconditions.checkNotNull(answers);
  }

  public QuestionAnswersJson() {
    // for JSON-to-Java conversion
  }

  public List<AbstractAnswerJson> getAnswers() {
    return answers;
  }

  public void setAnswers(List<AbstractAnswerJson> answers) {
    answers = answers;
  }
}

The @ArraySchema annotation is completely ignored and the resulting class in the YAML file is an array of AbstractAnswers:
"QuestionAnswers" : {
        "type" : "object",
        "properties" : {
          "answers" : {
            "type" : "array",
            "items" : {
              "$ref" : "#/components/schemas/AbstractAnswer"
            }
          }
        }

But I would like to have something similar to this:
"QuestionAnswers" : {
        "type" : "object",
        "properties" : {
          "answers" : {
            "type" : "array",
            "items" : {
              oneOf:
                "$ref" : "#/components/schemas/XAnswer"
                "$ref" : "#/components/schemas/YAnswer"
                "$ref" : "#/components/schemas/ZAnswer"
                "$ref" : "#/components/schemas/BAnswer"
            }
          }
        }

Is that something incorrectly done on my side or is there a bug regarding @ArraySchema?

@AlexandrosG thanks for reporting this.
Trying to reproduce this with a similar test (as classes/ and test details are missing from the one shared):
@JsonPropertyOrder({"answers"})
@Schema(name = "QuestionAnswers", description = "")
public final class QuestionAnswersJson {
    @ArraySchema(schema = @Schema(oneOf = {XAnswerJson.class, ZAnswerJson.class}))
    private List<AbstractAnswerJson> answers = new ArrayList<>();

    public QuestionAnswersJson(@NotNull List<AbstractAnswerJson> answers) {

    }

    public QuestionAnswersJson() {
        // for JSON-to-Java conversion
    }

    public List<AbstractAnswerJson> getAnswers() {
        return answers;
    }

    public void setAnswers(List<AbstractAnswerJson> answers) {
        answers = answers;
    }

    public static class AbstractAnswerJson {
    }

    public static class XAnswerJson extends AbstractAnswerJson {
        public String bar;
    }

    public static class ZAnswerJson extends AbstractAnswerJson {
        public String tar;
    }
}

The resolved YAML is:
AbstractAnswerJson:
  type: object
  oneOf:
  - $ref: '#/components/schemas/XAnswerJson'
  - $ref: '#/components/schemas/ZAnswerJson'
QuestionAnswers:
  type: object
  properties:
    answers:
      type: array
      items:
        $ref: '#/components/schemas/AbstractAnswerJson'
XAnswerJson:
  type: object
  properties:
    foo:
      type: string
    bar:
      type: string
ZAnswerJson:
  type: object
  properties:
    foo:
      type: string
    tar:
      type: string

which matches your expected outcome I believe (oneOfs are wrapped into AbstractAnswerJson which is how resolving works currently).
If this is not the case, kindly provide a complete test with actual and expected output, otherwise please close ticket


Add support to ttnn async for executing models in threads other than the main thread

This is required by vLLM (as a result of https://github.com/tenstorrent/vllm/issues/15) since it attempts to execute TT models in threads other than the main thread when serving.

Changes merged @ https://github.com/tenstorrent/tt-metal/commit/34f1b629f4d4356455ccbaf63f81bf9c8f4ae9f3 should allow you to call TTNN async APIs from a forked thread.

vLLM async model execution works with your changes, thanks a lot Aditya


_dispatchable shortcuts for no args case

Currently _dispatchable exits early when no args are provided. Though if you're dispatching shouldn't that be the job of the method you're deferring to to handle (the no arg case).

Good point!

um... if there's no arguments, then we have no object to which we could delegate a function as a method.

Ah doh!


make roundCropper take effect

issuse #36

@AhamedBilal Please merge this, we need roundCropper to work properly


Create CONTRIBUTING.md

Description
To adhere to the tracking of contributors and contribution policy, we need this contributing page.
This is a first pass. still has a lot of holes and references to Ruby.
Reviewer Notes

[ ] Please fix the versioning (currently "???")
[x] add in instructions for installing Go, React, etc.
[ ] I bet I missed some major stuff

References

Github issue

@tinyels do you think the making and verifying code sections are also covered in our Development Guide?

And is code style our programming guide?


Broken TypeScript hover

open vscode.d.ts, go to line 3679
hover over setStatusBarMessage
🐛 the message show show 'Note' in italic but shows '*Note', debug and see that the provider returns an invalid string

I'm unable to repo this in insiders or building from master:

@jrieken Were any additional steps required to reproduce this issue?

Maybe the use of TypeScript 2.1.4

Thanks @jrieken. I'm able to repo using TS2.1.4, but it looks like TS 2.1.5 fixed this. I've updated the bundled version of TS to 2.1.5, and also updated our VSCode build to use TS2.1.5
Closing as upstream.


failed to read store from /srv/data/store.gob

Hi!

Permission on /opt/telepush is 777 and "data" folder is created while telepush start

I think the problem is that the bot will look for its data directory at /srv/data, but your mountpoint is only at /srv and it won't create the sub directory itself. So if you mount your volume directly to /srv/data, the issue might be resolved. Otherwise I can have a deeper look tomorrow.

 Thanks!


Skip null repositories

Accidentally deleted my branch for #22, here it is again.
atom.project.getRepositories() returns an array with an element for each project root, regardless of whether that root has a repository. So sometimes the returned array contains some nulls.

:+1: Thanks!


Select2 version 4

Hi
I am using Select2 version 4 and can't get this to work.
Is it configured to work with version 4?
Regard
M.

I am presuming so as I see on http://ui.themepixels.com/select2-skins/bower_components/select2/select2.css the version of select2 is Version: 3.5.2
Is there a skins planned for Select2 version 4?

The skin is for Select2 version 3.5. If given enough time I will create skin for version 4. Thanks

Thanks themepixels!

+1

@themepixels Hem, I think you should update readme. Because only select2 version 3.5 support this skins :|


Support for Puppet 6 with Windows nodes has been manually tested

Manual tests to run through:
Puppet 5 tests

[ ] Conjur identity via host factory, configured in Hiera, Windows Server 2019
[ ] Conjur identity via host factory, configuration hardcoded in manifest, Windows Server 2016
[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows Server 2012 R2
[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows 2016
[ ] Conjur identity via host / API key, configuration in Hiera, Windows Server 2012 R2
[ ] Conjur identity via host / API key, configuration hardcoded in manifest, Windows Server 2019

Puppet 6 tests

[ ] Conjur identity via host factory, configuration hardcoded in manifest, Windows 2012 R2
[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows 2019
[ ] Conjur identity via host / API key, configuration in Hiera, Windows 2016

In all tests, we are validating that if the server manifest includes conjur configuration and invokes lookup plugin, that it correctly sends the secret values back to the agent, i.e. we need to verify that the server can authenticate to Conjur and retrieve secrets / pass the values to agent.

As a note so we remember, I generated the above list by putting this model into web PICT:
Identity: Host factory, Host/API key Config: Windows Registry/WinCred, Identity files Windows Version: Windows 2012, Windows 2016 Puppet version: Puppet 5, Puppet 6
It then generated the above 5 scenarios to ensure all possible pairs of values are covered.

@sgnn7 @doodlesbykumbi how does this sound as a way to split the cases?

@sgnn7 will do:
Puppet 5 tests

[ ] Conjur identity via host factory, configuration hardcoded in manifest, Windows Server 2016
[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows 2016
[ ] Conjur identity via host / API key, configuration in Hiera, Windows Server 2012 R2

Puppet 6 tests

[ ] Conjur identity via host factory, configuration hardcoded in manifest, Windows 2012 R2
[ ] Conjur identity via host / API key, configuration in Hiera, Windows 2016

@doodlesbykumbi will do:
Puppet 5 tests

[ ] Conjur identity via host factory, configured in Hiera, Windows Server 2019
[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows Server 2012 R2
[ ] Conjur identity via host / API key, configuration hardcoded in manifest, Windows Server 2019

Puppet 6 tests

[ ] Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows 2019

I managed to get through 3 cases using 2019 windows containers. The puppet 6 test for Conjur identity via host / API key, configuration in Windows Registry / WinCred, Windows 2019 failed as a result of faulty instructions in the README.md. I have a PR to resolve this https://github.com/cyberark/conjur-puppet/pull/140.

Issues that came out of this round of manual testing:

#147 - When using Puppet server-side credentials, specifying both cert_file and ssl_certificate errors out on Windows
PR #140 - Update windows cred example in README.md

Unknown issue numbers: (@sgnn7 can you please add issue links later?)

Warning: Agentâ€™s registry did not contain path Software\CyberArk\Conjur. If this is the
first time using HFTs on this node, this is expected behavior. This appliest to both HFTs and Hiera
configs so we need to reword this message. We may want to bump it down to a "Notice" as well
if we don't expect agent direct config to be used more often than not.
In docs, examples of cert_path for Windows need to use C:\-prefixed paths since only PowerShell
natively resolves paths like /tmp/foo.pem.
In docs, examples of cert_path should use a path (e.g C:\vagrant\.tmp\conjur_ca.pem, not a "file" function
file('C:\vagrant\.tmp\conjur_ca.pem').
cert_file examples and definiton must include where this path applies to. Specifically, anything that
doesn't use wincred/registry on the agent implies that this path is on the server which isn't clear in the
docs at all.

@izgeri updated

thanks @sgnn7! closing this issue now.


Duplications in lists - Camps/Volunteers/Requests/etc

Duplication in current lists.
6k duplicates in volunteers list and 300 invalid phone numbers in dump (https://s3.amazonaws.com/keralarescue-datadump/20180818_0600/mainapp_tables.sql)
#de-dupe volunteer data
ALTER TABLE mainapp_volunteer
add normalized_name varchar(100);
ALTER TABLE mainapp_volunteer
add alternate_name varchar(100);
#upper names
UPDATE mainapp_volunteer
SET alternate_name = upper(name);
#remove special characters (also removes malayalam characters)
UPDATE mainapp_volunteer
SET normalized_name = regexp_replace(regexp_replace(alternate_name,'\W+',' ','g'),'( ){2,}',' ');
#duplicate list
SELECT normalized_name,phone,count(*) FROM mainapp_volunteer
WHERE normalized_name <>' '
GROUP BY normalized_name,phone
HAVING count(*) > 1
ORDER BY normalized_name;```

use list above to mark min or record id's as active='N' and put filter on queries to show only where flag is 'Y' or blank

same needs to be done for the other lists

Could you please explain your approach?

Looks like he's creating a new empty column normalized_name to copy the names and then also keeping a backup called alternate_name (except it's all saved as the upper case now). Now he's removing all the special characters using regex and I think trimming down multi-spaces in between to a single space. (although I think it should be \s{2,} than ( ){2,}. Still, his approach should work well too since parentheses makes it equivalent to \s.
Next, he's making a fetch of the records and rest is explained by him..

as mentioned by Saifur, alternate name is maintained since we have names in Malayalam and we don't want to lose that with the regex
before applying normalization, there is a need to de-dupe full table records
i have tried this on local db by making a stg table and a field to track flag for duplicate
UPDATE mainapp_request_stg
SET is_duplicate = 'Y'
WHERE id IN (SELECT id FROM
 (SELECT *, count(*)
 OVER
   (PARTITION BY
     district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others
   ) AS COUNT
 FROM mainapp_request) tableWithCount
 WHERE tableWithCount.count > 1)
AND id NOT IN (SELECT max(id) FROM
 (SELECT *, count(*)
 OVER
   (PARTITION BY
     district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others
   ) AS COUNT
 FROM mainapp_request) tableWithCount
 WHERE tableWithCount.count > 1
GROUP BY district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others);

SELECT max(id),district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others FROM
 (SELECT *, count(*)
 OVER
   (PARTITION BY
     district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others
   ) AS COUNT
 FROM mainapp_request) tableWithCount
 WHERE tableWithCount.count > 1
GROUP BY district,
     location,
     requestee,
     requestee_phone,
     needwater,
     needfood,
     needcloth,
     needmed,
     needothers,
     status,
     supply_details,
     needkit_util,
     needtoilet,
     detailcloth,
     detailfood,
     detailkit_util,
     detailmed,
     detailtoilet,
     detailwater,
     latlng,
     latlng_accuracy,
     detailrescue,
     needrescue,
     is_request_for_others;

Volunteer Duplicates , Requests duplicates and Person Duplicates are removed


Selected code has the same background color both in dark mode and light mode

Selected code in dark mode has very light background color which makes the text unreadable:

I guess this is also due to a similar kind of unintended keyword missing:
Linked inline code block is hard to read in dark mode:
(Something like Wikipedia)

But are excellent in light mode:

Or it might be the case that I've seen this for the first time.

@XRFXLP Thanks for letting me know. I simply didn't notice a code {} had a different color and didn't override it for the dark mode (maybe the update introduced it and I didn't catch it).

@XRFXLP The fix was deployed.


[feature request] Is there a way to specify currentColor

Some Wikimedians like Colorable SVG, so that you create one SVG and it can be rendered in several colours. librsvg2.48 implemented --stylesheet=custom.css see https://gitlab.gnome.org/GNOME/librsvg/-/issues/736#note_1119478 , does resvg has a similar functionality?

Edit 2023: I don't expect that Wikimedia will change it's renderer. (At least no one currently actively try to push it to switch render.)

No, it always fallbacks to black.
As for stylesheet injection: #277

No, it always fallbacks to black.

I see this is implemented in the SVG parser:
https://github.com/RazrFalcon/resvg/blob/b8938f406eb0a008e7403451a8dbcade63b02361/crates/usvg-parser/src/style.rs#L147-L150
After reading the comments at #467 I understand that #277 would provide a more complete solution, but maybe it's alright to add support for only setting a current color?

Adding custom currentColor or CSS injection would probably take the same amount of time. Therefore just wait for #277

svg_data = svg_data.replace("currentColor", color);

Just a simple replace after reading the SVG data to a string is working well for me. It's a pretty rudimentary solution and I'm sure there are edge cases but it works alright until a proper injection system is implemented.


DB11 cannot be added

I'm trying to add my Dahua DB11 into home assistant but I keep returning wrong user/password/address. I've changed the port number to 37777 instead of the default 80 but it still doesnt work. I've managed to connect through the Dahua ConfigTool  which means my authentifications are correct.
Is there something i'm missing
Home Assistant Core: 2021.10.6
HassOS  6.5

DB11 doorbells do not support native Dahua API, use the Imou Life cloud integration instead https://github.com/user2684/imou_life

Any News?


Support HTTPRoute Condition RouteConditionReady

Description:
Populate the GatewayConditionReady condition on the Gateway listener when its corresponding xDS resources have been programmed in envoy and acknowledged by it.
relates to https://github.com/kubernetes-sigs/gateway-api/issues/1833

the xDS protocol behavior around NACKs is defined here https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol#nack
we would need to

peek into the error_detail field within DeltaDiscoveryRequest https://github.com/envoyproxy/gateway/blob/0af4f190f98bba5ac84f939f4eb2dc7279971cf5/internal/xds/cache/snapshotcache.go#L306
if its empty, we get update the Gateway listener status, set Ready=True, and publish it using something like https://github.com/envoyproxy/gateway/blob/0af4f190f98bba5ac84f939f4eb2dc7279971cf5/internal/gatewayapi/runner/runner.go#L103
above involves also passing Gateways all the way to the xds-server, increasing memory footprint, we can optimize that in the future by only passing Gateway statuses

assigning this to @chauhanshubham who is interested in picking this one up, thanks !

It's a bit tricky at this time to determine and trace the NACK request error to a particular Route object.
Although we use delta requests, but the requests can be clubbed for multiple route objects in a single snapshot too.
Scenario 1:
Lets say we create httproute, tlsroute and then the gateway (containing http and tls route references) - in this particular order. The snapshot will be created at the last step -> when the gateway is created.
If only the tlsroute related resource had an issue, the httproute traffic would also not function, because of the snapshot error.
Scenario 2:
Create the gateway and httproute first, wait for envoy to be configured, and then create errored tlsroute configurations. This time around, the delta snapshot will error out. Now httproute traffic will work fine, but tlsroute would not work.
This means same configurations can lead to two different scenarios. Which in turn means that, assuming we complete this task, the route status cannot be deterministically set.
Will need to think more about this. Just posting some initial observations.

thanks for running these tests @chauhanshubham !
imo we should treat all scenarios like scenario 1 where we cannot determine the order of resources received and must update status of all child resources xRoutes to Ready:False if any error is encounted for a snapshot, because order can change pre/post EG restart

Also, an error doesn't really mean that the traffic on the xRoute would stop working. It's just that the errored configuration would be rejected, once tried after a valid configuration.
Should that be a Ready: true to Ready: false transition candidate.

since there is a 1:1 mapping of the RouteSpec to some xDS resource, it means the latest authored route config is not programmed/not ready imo


power tap bug? no canection to any cable of any kind

I love the mod but i found a bug that well makes it useless after you build the reactor you have the power tap but when you power up you make RF but you cant git it out even with a power tap no energer or "RF" will go into anything plz help me im using : CoFHCore-[1.7.10]3.0.0B6-32, and: big reactors-0.4.1A2 ill and a screen shot aswell someone plz help me

Please read the requirements on www.big-reactors.com. Big Reactors 0.4.1A2 requires CoFHCore-3.0.0B9 or above.


K8SPXC-643 restart node if it has non-Primary state

 ❨?❩

detect split-brain conditions and restart the node to
perform auto recovery

Test name
Status

storage
passed

scaling
passed

haproxy
passed

upgrade-haproxy
passed

recreate
passed

big-data
passed

init-deploy
passed

self-healing
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

upgrade-proxysql
passed

security-context
passed

limits
passed

self-healing-advanced
passed

monitoring-2-0
passed

smart-update
passed

demand-backup
passed

affinity
passed

one-pod
passed

operator-self-healing
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

pitr
passed

users
passed

upgrade-consistency
passed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

scheduled-backup
failed

PXC operator docker - perconalab/percona-xtradb-cluster-operator:PR-774-5a9d9312

Test name
Status

upgrade-haproxy
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring-2-0
passed

affinity
passed

storage
passed

one-pod
passed

self-healing
passed

auto-tuning
passed

self-healing-advanced
passed

proxysql-sidecar-res-limits
passed

operator-self-healing
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

scaling
passed

scaling-proxysql
passed

security-context
passed

recreate
passed

restore-to-encrypted-cluster
passed

big-data
passed

demand-backup
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

scheduled-backup
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed


Entering vr causes the vr camera to be darker if default pipeline was used on a different camera

Bugs
Repro: https://playground.babylonjs.com/#TAFSN0#36
Workaround: https://playground.babylonjs.com/#TAFSN0#37 (dispose pipeline before entering vr)

Expected result:
Entering vr should use the vr camera and no effects should be applied
Current result:
Entering vr causes the vr camera to be darker

This issue is caused by the existing behavior of the ImageProcessingPostProcess which modifies global scene properties eg. imageProcessingConfiguration.
Suggestion is when using the default pipeline to initialize with all cameras eg.
var defaultPipeline = new BABYLON.DefaultRenderingPipeline("default", true, scene, [camera1, camera2]);
Otherwise default pipeline's effect on the scene can be undone with: defaultPipeline.imageProcessingEnabled = false
 scene.autoClear = true;
See: https://playground.babylonjs.com/#CKFSVU#2


This edit deletes the extra word.

:white_check_mark: Organization membership check passed
Requester is not a member of the MicrosoftDocs organization.


create mojap-derived-tables-replication bucket

This pull request:

contributes to this ticket
creates mojap-derived-tables-replication and apc-bucket-logs buckets

Signed off by: Anthony Fitzroy<EMAIL_ADDRESS>

@AntFMoJ Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance
Guidance on approving these PRs is available at https://user-guide.modernisation-platform.service.justice.gov.uk/runbooks/reviewing-mp-environments-prs.html#process-for-approving-prs


read/pe should handle unaligned section addresses and sizes

The PE file format specifies that sections should be aligned, but Windows still loads them if they are not. We should duplicate the logic that Windows uses.

example file
possible method of handling alignment

I don't think there's any official documentation on how to handle this, and llvm-readobj doesn't handle the example file either, so this is a low priority unless you're interested in things like parsing malware.


Update README.md

The cd path is wrong (missing github.com), go build is unnecessary; go get already builds and install.

Oops! Thanks for fixing this!


Update cppcheck next time it is released

To work around a cppcheck bug, the ShaderTrap CI will feature applying a custom patch. This should be gotten rid of next time there is a cppcheck release, as the problem doesn't persist in cppcheck's main branch.

scripts/check_headers.py should be edited to remove cppcheck.patch from the list of excluded files.


Add single book option

By specifying a title (or regex), deal only with the books that match this regex

Fixed in commit 654d45e2ac11bc2dc33e577bb122d62634db7ce3


GIF is not rendered correctly on HTML (nbdev==2.3.4)

To display a test.gif on the notebook and on the rendered HTML I used  display.HTML(<img src='./media/test.gif’>)
with nbdev<2.3.4, nbdev_docs automatically creates the “media” directory (which contains test.gif) in the _docs folder and test.gif was rendered correctly on HTML.
However, when I upgraded to nbdev==2.3.4, nbdev_docs did not automatically create the “media” directory and test.gif is not shown in the HTML. (test.gif is displayed correctly on the notebook though).
Is it possible to add the feature that automatically prepares the “media folder” as in nbdev<2.3.4?

Should be fixed in latest release.


Missing dep for tests

Mocha is a missing dep when running the tests :)

I just cloned and ran install, then ran npm test, no issues... no references to mocha exist.

I also ran find for mocha, nothing in the project.  @Wardormeur if you can reproduce an issue, please link to the offending code and reopen.

I figured out why I guess :
when doing an npm isntall seneca-queue, it seems to take the one from github.com/mcollina/seneca-queue; which indeed redirect to this one on github, but probably don't for npm

@Wardormeur thanks for pointing that out.  I just published a new version with updated links and tests.

Thanks :+1:


SBC Media Bypass Features

Hello at all,
I discovered problems with media bypass and using teams applications like attendants.
There is one way audio till the transfer with external users.
I investigate it with the MS Support and the conclusion was, no media bypass when you will use Applications. Bad solution and in your list it is not clear with whitch vendor it works. In my case it was AudioCodes.
Please update the SBC list with this informations or better, fix it.
Media Bypass make sense for bigger installations. Workaround is for this applications exeptions and transcoding at the moment.

@sgtdonny Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@sgtdonny thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


OS Modifier (EMU): Add sshkeys to user config

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Change Log
https://github.com/microsoft/CBL-Mariner/pull/6824#discussion_r1406527775
as recommended, this is a follow-up separate pt for previous emu pr,
Does this affect the toolchain?
NO

add "OS Modifier (EMU):" or something to pr title?

Please also get sign-off from @dmcilvaney

oops sorry missed this comment. @dmcilvaney let me know if theres anything needs improve, thanks


RFC for alpa api authentication

issue #700

@rjiang-ptm  thanks!
@merrymercy @zhuohan123 please comment

Thank you for developing this great framework and for making the API public!
I was directed here by Hao, and I'm looking for programmatic access to the OPT interface. I believe will be used in the following ways:

Generation: Using the completions API to generate text given a prompt
Classification: Still using the completions API, but restrict the output tokens, and use their log probabilities for classification (e.g., this)

I hope the above makes sense. I noticed this pull request appears to need some work - Please let me know if there's anything I can help with or if you have any questions.
In case any of the requested features is not available, I'll be happy to help create it :)

We did some refactoring and introduce a multi-model serving controller.
The new priority logic can be implemented here https://github.com/alpa-projects/alpa/blob/32468ee67b1f9222fcabcc62c916c8326ee8ce9f/examples/llm_serving/launch_model_worker.py#L82-L108
The authentication can probably be handled as a middleware here
https://github.com/alpa-projects/alpa/blob/32468ee67b1f9222fcabcc62c916c8326ee8ce9f/alpa/serve/controller.py#L227-L245

@rjiang-ptm since we have agreed (in email) to proceed with Option 3. maybe we can start the implementation (w/ MBZUAI team)?

closed due to inactivity


Typo fix on chiperText variable

Just fixing typo on variable name.

https://en.wikipedia.org/wiki/Cipher
As the variables are in English I'm suggesting @danang-id not to merge this.
:-1:

I think I understand which variables name @TiagoGouvea  is pointing to. It should be decipherText instead of cipherText. However, your commit doesn't fix the variable name.

Agree with @lil5 . I'm not going to merge this.

I think there is indeed a typo on the variable's name on the example.
// this `chiperText` should've been `cipherText`
var chiperText = simpleCrypto.encrypt(plainText);


Sentry is not configured correctly

Bug Report
Expected behavior/code
When an issue is reported to sentry we want to know the environment and the release where the issue occurs
Actual Behavior
When an issue is caught and reported to sentry, we don't have the real environment and release information. Instead we have something like <property object at 0x7f2fc9fe0c20>
Steps to Reproduce

Configure your environment with a valid SENTRY_DSN
shut down the database
send a request to marsha. Without a database an exception will be thrown
look in sentry environment and release values.

Environment

Marsha version: 3.9.1
Platform: docker

Possible Solution
sentry is configured in settings.py. The method used a classmethod decorator and as I understand the cls parameter is not an instance of the class but the uninstanciated class. To configure sentry we call methods using a property decorator, calling them statically like we do return a property object and does not call the getter method attached to the decorator. Creating static method can resolve the issue or we have to find a way to instantiate the class.

The modification was done in this commit : https://github.com/openfun/marsha/commit/2eb8fd1f09a39c519d6eca89a69889fee83c855b


Update documentation to add privilege needed to manage cloud-init content

Hi,
I'm using version 3.0.1-rc1 of the provider against Proxmox 8.2.2 and followed the current documentation to create a role, user, and bind them together in Proxmox. When attempting to create a proxmox_cloud_init_disk resource in Terraform I got the following error:
proxmox_cloud_init_disk.ci: Creating...
╷
│ Error: read tcp <IP_ADDRESS>:51682-><IP_ADDRESS>:8006: read: connection reset by peer
│
│   with proxmox_cloud_init_disk.ci,
│   on vm-cloudinit.tf line 8, in resource "proxmox_cloud_init_disk" "ci":
│    8: resource "proxmox_cloud_init_disk" "ci" {
│
╵

After some digging in logs on the Proxmox host I noticed an HTTP 403 error in /var/log/pveproxy/access.log:
...
::ffff:<IP_ADDRESS> - - [03/05/2024:19:36:38 -0600] "POST /api2/json/access/ticket HTTP/1.1" 200 1016
::ffff:<IP_ADDRESS> - terraform-prov@pve [03/05/2024:19:36:38 -0600] "GET /api2/json/access/users?full=1 HTTP/1.1" 200 283
::ffff:<IP_ADDRESS> - terraform-prov@pve [03/05/2024:19:36:38 -0600] "GET /api2/json/access/permissions?userid=terraform-prov@pve&path=/ HTTP/1.1" 200 395
::ffff:<IP_ADDRESS> - - [03/05/2024:19:36:38 -0600] "POST /api2/json/nodes/radon/storage/local/upload HTTP/1.1" 403 -
::ffff:<IP_ADDRESS> - root@pam [03/05/2024:19:36:38 -0600] "GET /api2/json/nodes/radon/status HTTP/1.1" 200 818
...

It seemed the provider was not authorized to upload snippet content to the storage location. After browsing through the Proxmox Permission Management guide, I guessed the privilege Datastore.AllocateTemplate is needed. I modified the privileges for the role to include that privilege and was able to successfully create a proxmox_cloud_init_disk resource.
proxmox_cloud_init_disk.ci: Creating...
proxmox_cloud_init_disk.ci: Creation complete after 0s [id=local:iso/tf-ci-test-vm-cloudinit.iso]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

Please change the documentation so that this block...
pveum role add TerraformProv -privs "Datastore.AllocateSpace Datastore.Audit Pool.Allocate Sys.Audit Sys.Console Sys.Modify VM.Allocate VM.Audit VM.Clone VM.Config.CDROM VM.Config.Cloudinit VM.Config.CPU VM.Config.Disk VM.Config.HWType VM.Config.Memory VM.Config.Network VM.Config.Options VM.Migrate VM.Monitor VM.PowerMgmt SDN.Use"

...is instead...
pveum role add TerraformProv -privs "Datastore.AllocateSpace Datastore.Audit Datastore.AllocateTemplate Pool.Allocate Sys.Audit Sys.Console Sys.Modify VM.Allocate VM.Audit VM.Clone VM.Config.CDROM VM.Config.Cloudinit VM.Config.CPU VM.Config.Disk VM.Config.HWType VM.Config.Memory VM.Config.Network VM.Config.Options VM.Migrate VM.Monitor VM.PowerMgmt SDN.Use"

Thanks!

I'm not sure how content from this repo ends up in the Terraform Registry docs but I'm guessing this line is what needs to change per my recommendation above.

Also I just noticed that Datastore.Allocate is needed to modify existing cloud-init content. Without this, I received another HTTP 403 error in the /var/log/pveproxy/access.log:
::ffff:<IP_ADDRESS> - terraform-prov@pve [03/05/2024:20:10:41 -0600] "DELETE /api2/json/nodes/radon/storage/local/content/local:iso/tf-ci-test-vm-cloudinit.iso HTTP/1.1" 403 13

From my point of view, such approach, granting Datastore.Allocate, Datastore.AllocateTemplate for the root pool / or for all the disks, and many other roles/permissions,  is very risky and give for user too many permissions, thus dramatically increases blast area.
More secure option is to configure  fine-grained permissions and it can be achieved with user/token with privilege separation, my example is described in this post - https://github.com/Telmate/terraform-provider-proxmox/issues/784#issuecomment-2135266013


change operator alpha types based on SSCS experience

This addresses concerns about muxing version information from the initial pull adding the APIService controller and adds information to resolve https://github.com/openshift/service-serving-cert-signer/issues/6

/lgtm

godoc clarified

/lgtm

I've just realized that I made a mistake here.  The webconsole operator in openshift is affected by this and I failed to consider that.  Some lessons hurt.


Feature request: default .col behavior  of the direct .row descendants

Is a silly idea to apply the style .col on the descendants of a row container if none is explicted?
 Explicit styles means fewer overrides on our part. No plans to change this in v4 though.


Not enough PureScript

👹

somehow I missed this;
at the bare minimum, if this becomes a Real Thing, I want it to include flow


begin EPA load script in R

When I try:
docker exec -it -w /src/EPA gis_loader Rscript load.R
I get:
ERROR: dependency 'bit' is not available for package 'bit64'

removing '/nix/store/6wxz7fbsbj1qkxca8957f1bkr41h1n30-r-boot-1.3-25/library/bit64'
installing source package 'lubridate' ...
** package 'lubridate' successfully unpacked and MD5 sums checked
** using staged installation
** libs
sh: make: command not found
Warning in system(cmd) : error in running command
ERROR: compilation failed for package 'lubridate'
removing '/nix/store/6wxz7fbsbj1qkxca8957f1bkr41h1n30-r-boot-1.3-25/library/lubridate'
ERROR: dependencies 'bit64', 'lubridate' are not available for package 'RPostgres'
removing '/nix/store/6wxz7fbsbj1qkxca8957f1bkr41h1n30-r-boot-1.3-25/library/RPostgres'

I have a feeling this will require a slight modification to the gis_loader image, but I am not sure.

Had a meeting to review this PR. Tim to work on some of the changes discussed and will pass to me when ready or steam is lost.
RPostgres should work in our nixpkgs when this gets merged in: https://github.com/NixOS/nixpkgs/pull/121819

The load pipeline for EPA AQS data is functioning. I added jsonlite and httr in the list of R dependencies (in addition to DBI and RPostgreSQL). The pipeline is not necessarily pretty, but's a step in the right direction and opens several questions for consideration (somewhat notes in the inline documentation).


Add support for read-only files

This pull request adds support for read-only files.
Currently, licenceheaders crashes when input files are not readable or not writable.
This patch makes licenceheaders ignore read-only files (with a warning) as the new default behavior.
Moreover, it adds a new argument --force-overwrite to tell licenceheaders to try to change file permissions prior to license injection (i.e. make read-only files writable). On success everything works as expected and file permission are restored afterwards. On failure (insufficient file permissions), the file is ignored with a warning.
This patch also fixes the following:

Log an error when the file is not readable, prior to crash
Fix CMakelists.txt test case
Adds read-only test.sh test case
Add file permission comparison to tests validation
Fix a small typo in README.md

Files that are not readable are not affected by this patch, besides additional logs.
This is a proposal to fix issue #55.

Good idea, thanks!