Suggestion: pip-installable Pyton package

Hi,
I'm a grateful user of svut (it is excellent work 👍) and am wondering if there is any desire to make this package pip installable and turn it into a standard Python package. If you are interested I'd be happy to take that on. Indeed, I took a (very opinionated) stab at it here: https://github.com/dpretet/svut/compare/master...sschaetz:svut:python-package where I am doing the following:

create a setup.py and pyproject.toml file which allows for this package to be pip-installable (and I think publishable to pypi but I have never done that)
move all sources into a sub-directory svut (this is somewhat of a convention for python packages)
rename the 2 command line tools to more of a Python convention snake_case format
expose the two command line tools via setup.py CLI tools -- this way once you install the python package you have the tools automatically in your path and don't need to mess with $PATH

The caveat is that I think I probably broke some of the shell scripts but I'm happy to fix them.
Cheers,
Seb

Hi Seb,
Thanks for your interest and the proposal. That's something I consider since months but didn't take time to do that.
The only worrying point is about renaming the executables, could you keep the name of the symlinks pointing them to avoid breaking existing flows? That would require a significant amount of changes among people using SVUT nowadays.
Except that, please go ahead, it will be a pleasure to accept your pull request ☺
I think it's possible but never did it, publishing a newer version thru GitHub actions would be very great.
Best,
Damien

Awesome -- I proposed a set of changes here: https://github.com/dpretet/svut/pull/24

Hi Sebastian,
I followed the instruction in the link you provided and everything is ready. Can you give a try please to be sure it's working?
https://pypi.org/project/svut/
Damien

Congratulations @dpretet -- it works like a charm!

Don't hesitate to request or propose new features. BTW I'm curious how and for what for you use SVUT ☺
Thanks again Sebastian!


KeyError: 'cu111'

I am trying to use CUDA 11.1 instead of 10.1 to use this but instead of setting thing sup, I get the error "KeyError: 'cu111'" and I'm not sure what to do.
Here is everything in the anaconda prompt pasted:
`
(base) C:\Users[My Username]>D:
(base) D:>cd D:\Installations\Funny Fullbody Deepfake\iPERCore-main
(base) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>conda create -n iperc python=3.6.6
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: done
==> WARNING: A newer version of conda exists. <==
current version: 4.10.3
latest version: 4.11.0
Please update conda by running
$ conda update -n base -c defaults conda

Package Plan
environment location: D:\Installations\Anaconda3\envs\iperc
added / updated specs:
- python=3.6.6
The following NEW packages will be INSTALLED:
certifi            pkgs/main/win-64::certifi-2021.5.30-py36haa95532_0
pip                pkgs/main/win-64::pip-21.2.2-py36haa95532_0
python             pkgs/main/win-64::python-3.6.6-hea74fb7_0
setuptools         pkgs/main/win-64::setuptools-58.0.4-py36haa95532_0
vc                 pkgs/main/win-64::vc-14.2-h21ff451_1
vs2015_runtime     pkgs/main/win-64::vs2015_runtime-14.27.29016-h5e58377_2
wheel              pkgs/main/noarch::wheel-0.37.1-pyhd3eb1b0_0
wincertstore       pkgs/main/win-64::wincertstore-0.2-py36h7fe50ca_0
Proceed ([y]/n)? y
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

To activate this environment, use

$ conda activate iperc

To deactivate an active environment, use

$ conda deactivate
(base) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>conda activate iperc
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>conda install pytorch==1.8.1+cu111 torchvision==0.9.1+cu111 torchaudio==0.8.0 cudatoolkit=11.1 -c pytorch -c conda-forge
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
PackagesNotFoundError: The following packages are not available from current channels:

torchvision==0.9.1+cu111
pytorch==1.8.1+cu111

Current channels:

https://conda.anaconda.org/pytorch/win-64
https://conda.anaconda.org/pytorch/noarch
https://conda.anaconda.org/conda-forge/win-64
https://conda.anaconda.org/conda-forge/noarch
https://repo.anaconda.com/pkgs/main/win-64
https://repo.anaconda.com/pkgs/main/noarch
https://repo.anaconda.com/pkgs/r/win-64
https://repo.anaconda.com/pkgs/r/noarch
https://repo.anaconda.com/pkgs/msys2/win-64
https://repo.anaconda.com/pkgs/msys2/noarch

To search for alternate channels that may provide the conda package you're
looking for, navigate to
https://anaconda.org

and use the search bar at the top of the page.
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>python setup.py develop
Cuda version is 11.1
Traceback (most recent call last):
File "setup.py", line 357, in
torch_torchvision_mmcv = platform_dependencies()
File "setup.py", line 237, in platform_dependencies
torch = get_torch_version(cuda_version_str, WINDOWS_PRECOMPILED_TORCH_CUDA_PAIRS, DEFAULT_WINDOWS_CUDA_TORCH)
File "setup.py", line 207, in get_torch_version
torch_version_str = default_torch_cuda_mapper[cuda_version_str]
KeyError: 'cu111'
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>python setup.py develop
Cuda version is 11.1
Traceback (most recent call last):
File "setup.py", line 357, in
torch_torchvision_mmcv = platform_dependencies()
File "setup.py", line 237, in platform_dependencies
torch = get_torch_version(cuda_version_str, WINDOWS_PRECOMPILED_TORCH_CUDA_PAIRS, DEFAULT_WINDOWS_CUDA_TORCH)
File "setup.py", line 207, in get_torch_version
torch_version_str = default_torch_cuda_mapper[cuda_version_str]
KeyError: 'cu111'
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>conda install pytorch torchvision torchaudio cudatoolkit=11.1 -c pytorch -c conda-forge
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: done
==> WARNING: A newer version of conda exists. <==
current version: 4.10.3
latest version: 4.11.0
Please update conda by running
$ conda update -n base -c defaults conda

Package Plan
environment location: D:\Installations\Anaconda3\envs\iperc
added / updated specs:
- cudatoolkit=11.1
- pytorch
- torchaudio
- torchvision
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
lerc-3.0                   |       h0e60522_0         140 KB  conda-forge
libdeflate-1.8             |       h8ffe710_0          61 KB  conda-forge
------------------------------------------------------------
                                       Total:         202 KB

The following NEW packages will be INSTALLED:
blas               conda-forge/win-64::blas-2.112-mkl
blas-devel         conda-forge/win-64::blas-devel-3.9.0-12_win64_mkl
cpuonly            pytorch/noarch::cpuonly-2.0-0
cudatoolkit        conda-forge/win-64::cudatoolkit-11.1.1-heb2d755_9
dataclasses        conda-forge/noarch::dataclasses-0.8-pyh787bdff_2
freetype           conda-forge/win-64::freetype-2.10.4-h546665d_1
intel-openmp       conda-forge/win-64::intel-openmp-2022.0.0-h57928b3_3663
jbig               conda-forge/win-64::jbig-2.1-h8d14728_2003
jpeg               conda-forge/win-64::jpeg-9d-h8ffe710_0
lcms2              conda-forge/win-64::lcms2-2.12-h2a16943_0
lerc               conda-forge/win-64::lerc-3.0-h0e60522_0
libblas            conda-forge/win-64::libblas-3.9.0-12_win64_mkl
libcblas           conda-forge/win-64::libcblas-3.9.0-12_win64_mkl
libdeflate         conda-forge/win-64::libdeflate-1.8-h8ffe710_0
liblapack          conda-forge/win-64::liblapack-3.9.0-12_win64_mkl
liblapacke         conda-forge/win-64::liblapacke-3.9.0-12_win64_mkl
libpng             conda-forge/win-64::libpng-1.6.37-h1d00b33_2
libtiff            conda-forge/win-64::libtiff-4.3.0-hd413186_2
libuv              conda-forge/win-64::libuv-1.43.0-h8ffe710_0
libzlib            conda-forge/win-64::libzlib-1.2.11-h8ffe710_1013
lz4-c              conda-forge/win-64::lz4-c-1.9.3-h8ffe710_1
m2w64-gcc-libgfor~ conda-forge/win-64::m2w64-gcc-libgfortran-5.3.0-6
m2w64-gcc-libs     conda-forge/win-64::m2w64-gcc-libs-5.3.0-7
m2w64-gcc-libs-co~ conda-forge/win-64::m2w64-gcc-libs-core-5.3.0-7
m2w64-gmp          conda-forge/win-64::m2w64-gmp-6.1.0-2
m2w64-libwinpthre~ conda-forge/win-64::m2w64-libwinpthread-git-<IP_ADDRESS>34.697f757-2
mkl                conda-forge/win-64::mkl-2021.4.0-h0e2418a_729
mkl-devel          conda-forge/win-64::mkl-devel-2021.4.0-h57928b3_730
mkl-include        conda-forge/win-64::mkl-include-2021.4.0-h0e2418a_729
msys2-conda-epoch  conda-forge/win-64::msys2-conda-epoch-20160418-1
numpy              conda-forge/win-64::numpy-1.19.5-py36h4b40d73_2
olefile            conda-forge/noarch::olefile-0.46-pyh9f0ad1d_1
openjpeg           conda-forge/win-64::openjpeg-2.4.0-hb211442_1
pillow             conda-forge/win-64::pillow-8.3.2-py36h10c25d6_0
python_abi         conda-forge/win-64::python_abi-3.6-2_cp36m
pytorch            pytorch/win-64::pytorch-1.10.1-py3.6_cpu_0
pytorch-mutex      pytorch/noarch::pytorch-mutex-1.0-cpu
tbb                conda-forge/win-64::tbb-2021.5.0-h2d74725_0
tk                 conda-forge/win-64::tk-8.6.11-h8ffe710_1
torchaudio         pytorch/win-64::torchaudio-0.10.1-py36_cpu
torchvision        pytorch/win-64::torchvision-0.11.2-py36_cpu
typing_extensions  conda-forge/noarch::typing_extensions-4.0.1-pyha770c72_0
xz                 conda-forge/win-64::xz-5.2.5-h62dcd97_1
zlib               conda-forge/win-64::zlib-1.2.11-h8ffe710_1013
zstd               conda-forge/win-64::zstd-1.5.1-h6255e5f_0
The following packages will be SUPERSEDED by a higher-priority channel:
certifi            pkgs/main::certifi-2021.5.30-py36haa9~ --> conda-forge::certifi-2021.5.30-py36ha15d459_0
Proceed ([y]/n)? y
Downloading and Extracting Packages
libdeflate-1.8       | 61 KB     | ############################################################################ | 100%
lerc-3.0             | 140 KB    | ############################################################################ | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: - "By downloading and using the CUDA Toolkit conda packages, you accept the terms and conditions of the CUDA End User License Agreement (EULA): https://docs.nvidia.com/cuda/eula/index.html"
done
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>python setup.py develop
Cuda version is 11.1
Traceback (most recent call last):
File "setup.py", line 357, in
torch_torchvision_mmcv = platform_dependencies()
File "setup.py", line 237, in platform_dependencies
torch = get_torch_version(cuda_version_str, WINDOWS_PRECOMPILED_TORCH_CUDA_PAIRS, DEFAULT_WINDOWS_CUDA_TORCH)
File "setup.py", line 207, in get_torch_version
torch_version_str = default_torch_cuda_mapper[cuda_version_str]
KeyError: 'cu111'
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>python demo/motion_imitate.py --gpu_ids 0 --image_size 512 --num_source 2 --output_dir "./results" --assets_dir "./assets" --model_id   "test" --src_path   "path?=./assets/samples/sources/man/man.png,name?=man" --ref_path   "path?=./assets/samples/references/fortnite.mp4,name?=fortnite,pose_fc?=300"
Traceback (most recent call last):
File "demo/motion_imitate.py", line 11, in
from iPERCore.services.options.options_setup import setup
ModuleNotFoundError: No module named 'iPERCore'
(iperc) D:\Installations\Funny Fullbody Deepfake\iPERCore-main>`

you can fix in it setup.py
in terms of window, cu111 doesn't written in setup.py
so you open the setup.py and just add "cu111": "1.8.1" in DEFAULT_WINDOWS_CUDA_TORCH
Also you can add "1.8.1+cu111": {
"torch": "https://download.pytorch.org/whl/cu102/torch-1.8.1-cp{PYTHON_VERSION}-cp{PYTHON_ABI_VERSION}-win_amd64.whl",
"torchvision": "https://download.pytorch.org/whl/cu111/torchvision-0.9.1-cp{PYTHON_VERSION}-cp{PYTHON_ABI_VERSION}-win_amd64.whl",
"mmcv-full": "1.2.0",
"mmcv-dist": f"{MMCV_DIST}/cu111/torch1.8.1/index.html"
}
in WINDOWS_PRECOMPILED_TORCH_CUDA_PAIRS


chore: Add code fence languages to scripting-docs

Sorry this is kind of noisy, it was mostly scoped find/replace after looking at some of the sample code fences that were missing the languages, then doing find/replace in VS code.
I believe some of the changed lines where from mixed line endings in some of the files that weren't normalized after the .gitattribues file got added to the repo

@gewarren - I assigned this to you as a primary contributor. Might consider moving the commits from @nschonni  to the private repo so that you can take advantage of the validation and staging tools there.

I'm going to let @Mikejo5000 look at this one.

#sign-off

Next

#hold-off
The content team (@gewarren, @Mikejo5000) will move this commit to the private repository for validation and staging. Then, they'll close this PR.


Added GH action for backplane-2.4

Description
Added backplane-2.4 regenerate-bundle Github action job.
Related Issue
If applicable, please reference the issue(s) that this pull request addresses.
Changes Made
Added backplane-2.4 regenerate-bundle Github action job.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
/cc @cameronmwall @ngraham20
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/lgtm


Prevent redirecting to local hosts in Oracle

See neo-project/neo#2662 and neo-project/neo-modules#692. Turns out, net/http Client has CheckRedirect exactly for this purpose (but be careful to keep the limit on redirections number).

Also neo-project/neo-modules#694.

i suggest to customize http.Transport. i think it's the final solution kkkkkk

inside http.Client.Do, there is a dns lookup

It's not always the case due to connection caching, but I was thinking more about redirects (the original problem stated) and what can happen during the call processing.

this lookup result may be different with the one in URIValidator

That's true, resolving something in CheckRedirect or URIValidator in general doesn't mean that http.Client will use the same IP, so we can just say that the check was implemented incorrectly right from the beginning.
The way Client/RoundTripper/Transport/Dial things are structured it should be trivial to fix this though, we're TCP-level at Transport.DialContext(), irrespective of when and how DNS was resolved, checking and dialing there is easy.


Fix include order

Related to https://github.com/ament/ament_lint/pull/374

CI is green: https://github.com/ament/ament_lint/pull/374#issuecomment-1118963512


WebSSH fail to properly display/distinguish box cursor and bar/line cursor

Feature description

Hey! Great works! Webssh is by far the best ssh client on iOS I have tried. But I want to share some improvement ideas with you.

Webssh fail to properly display/distinguish box cursor and bar/line cursor, which make vim (also zsh-vi-mode)almost unusable. I need the cursor turn into a bar to indicate I am in normal mode.
here is what I trying to achieve:  https://www.youtube.com/watch?v=FcQjTXLrVUU
My setting is working on both iTerm2 on MacOS, ubuntu and Windows. Only Webssh will display the cursor as box regardless the setting.
I also tried the report term type as xterm, xterm-coler256. Not working either.
Thanks you!

https://github.com/microsoft/terminal/issues/4335#issuecomment-577355385
https://vim.fandom.com/wiki/Change_cursor_shape_in_different_modes
https://learn.microsoft.com/en-us/windows/console/console-virtual-terminal-sequences#cursor-shape
https://unix.stackexchange.com/questions/3759/how-to-stop-cursor-from-blinking

# https://www.youtube.com/watch?v=FcQjTXLrVUU
# Ps = 0 ⇒ Blinking Block
# Ps = 1 ⇒ Blinking Block (Default)
# Ps = 2 ⇒ Steady Block
# Ps = 3 ⇒ Blinking Underline
# Ps = 4 ⇒ Steady Underline
# Ps = 5 ⇒ Blinking Bar
# Ps = 6 ⇒ Steady Bar
echo "\e[3 q"

Hey 👋
After testing again it works on my side as expected (.vimrc) :
PastedGraphic-1.png
I’ve just followed https://superuser.com/a/1234770
Cursor is updated in insert / normal mode
Could you check again?
Have a nice day ☀


Issue 9 Fix

Fixed issue 9 https://github.com/oracle/weblogic-logging-exporter/issues/9
System will first check for system property, if not present it will check ENV variable, if not present it will default it.
Documentation talk only about global variable and not system property, I kept property read to keep retro compatibility.
Fixed also a type on README.md about the variable name.
Signed-off-by: Stefano De Angelis<EMAIL_ADDRESS>

Stefano, I'm going to Slack you the requirements to add your GitHub account to the @oracle organization so that we can accept the contribution.


[BUG] Can't load trained model in AU in Ableton

Describe the bug
I trained a model using the Colab provided, put the unzipped download in the Models/ folder but it doesnt show up in the plug ins. Only pre-shipped models are in the list (I'm guessing my model should be added to this list?)
System Info
Macbook Pro 2020, M1 Silicon, Monterey
Ableton 10
Audio Unit v 1.01
To Reproduce
Traine model using the Colab provided, put the unzipped download in the Models/ folder, hit refresh
Screenshots

Additional context
Anything else we should know?

same issue here
it's very easy to reproduce
here the file I put in the dir in model file of Magenta

Hi, could you try directly dragging the *.tflite file into your model directory, instead of the entire exported directory itself?

@Louismac @romanticamaj Please try again with the latest v1.1.0 release. As @wilzh40 mentioned, you would only need to drag the trained *.tflite into the models directory. We will update the Colab instructions to reflect this. Closing this for now, please re-open if the issue persists along with the trained model zip file if possible.


access model - ELKR side channels coded as 'ACCESSIBLE'

eg confluence of Elk and Baldrey/Bean just upstream of Fernie (linear_feature_id=706841412). Side channel portion of stream is 'ACCESSIBLE', should be 'POTENTIALLY ACCESSIBLE'

barriers below side channels are not being processed.
https://github.com/smnorris/bcfishpass/blob/main/scripts/model/sql/functions/update_barriers_dnstr.sql#L58


Broker tls migrations should be last in order

Fixing broker-tls-migrations by making them last in order

Coverage increased (+0.02%) to 86.867% when pulling 93585a48a2e36020335ff5b358282714792eb9f0 on broker-tls-migrations into 886610ca4893e744d2c8def77aea46faa036d2a0 on master.


chore(deps): bump https://github.com/zeebe-io/zeebe-cluster-helm from 0.0.143 to 0.0.144

Update zeebe-io/zeebe-cluster-helm from 0.0.143 to 0.0.144
Command run was jx step create pr chart --name zeebe-cluster --version 0.0.144 --repo https://github.com/zeebe-io/zeebe-full-helm.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign salaboy
You can assign the PR to them by writing /assign @salaboy in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[SMALLFIX] Fix #552 new contributor task

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4047/
Test PASSed.

@tieujason330 Thanks for this improvement! Could you update the PR title and description with more specific information?

Looks like PR #9203 is the same change. Should I pick up another task?

Closed because someone else already has a PR for this change


[Direct Feeds] [App] Broken connection - feed level and Update card sync

This is part of the Third party workspace feeds V2 (direct connections) project.
Implement changes from this section of the design doc.
Also lets not forget to update the card sync.

Noting for any BZ assigned later on, @allgandalf is being paid lumpsum for this project. No payment needed here!

Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue.

Updates: I've prepared the Draft PR. I just need to think about the test steps and prepare screenshots.

The PR has been opened for the review.


Skip RAPIDS accelerated Java UDF tests if UDF fails to load

Fixes #1726.
Skip the Java UDF tests if this isn't a precommit build and the UDF class fails to load.

build

Is this something that we want to have in all of our tests on different platforms?

Yes, that's being tracked by #1640


Add example using esbuild

Create a react app (vanilla?) that creates an nft and uses esbuild for compilation.

@lorisleiva can I get this issue assigned to me. I'll start working on it.


Too delay when you have a collection with more than 2 million of documents

Hi,
The library is so nice, and i would like to implement in a "big system". I tell you my problem.
I have a mongodb with a collection which has around 2 million of documents (rows). I have implemented a Django-mongo-datatables, following your example of https://github.com/pjosols/django-and-mongo-datatables/
It works so nice, when collection has ~50k rows, but when collection has ~1kk rows, each operation cost more than 1 minute.
In my production environ i expect to have 10kk-20kk or rows, so... i dont know what can i do.
I have debugged in local, and i found that the most delayed code was in this section
    def results(self):
        _agg = [
                {'$match': self.filter},
                {'$sort': {self.order_column: self.order_dir}},
                {'$skip': self.start},
                {'$project': self.projection}
            ]

        if self.limit:
            _agg.append({'$limit': self.limit})

        **_results = list(self.db[self.collection].aggregate(_agg))**

If i desactivate "sort" on datatables, the performance improve a lot, however if i click on "last row" on datatables it doesnt work.
Could you help me? Have you tested the solution with a big mongodb? Maybe i figure out my problem.
Thanks a lot.

@srPuebla you ever get this fixed?


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/66588

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/75700

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80946

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/107068

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/121223


CMake Warning Policy CMP0037

CMake reports:
CMake Warning (dev) at examples/CMakeLists.txt:4 (add_executable):
Policy CMP0037 is not set: Target names should not be reserved and should
match a validity pattern.  Run "cmake --help-policy CMP0037" for policy
details.  Use the cmake_policy command to set the policy and suppress this
warning.
The target name "test" is reserved or not valid for certain CMake features,
such as generator expressions, and may result in undefined behavior.
Call Stack (most recent call first):
examples/CMakeLists.txt:40 (build_example)
This warning is for project developers.  Use -Wno-dev to suppress it.

This was already fixed in master a long time ago.


Download page of website is down

The http://www.godotengine.org/wp/download page displays:
Binary Downloads
Warning: file_get_contents(http://<IP_ADDRESS>/build.html) [function.file-get-contents]: failed to open stream: Connection timed out in /home/okamstudio/godotengine.org/wp/wp-content/plugins/insert-php/insert_php.php(48) : eval()’d code on line 3

There should be a couple of mirrors tho right?
On Thu, Sep 3, 2015 at 10:05 PM, Sylvain Berfini<EMAIL_ADDRESS>wrote:

The http://www.godotengine.org/wp/download page displays:
Binary Downloads
Warning: file_get_contents(http://<IP_ADDRESS>/build.html)
[function.file-get-contents]: failed to open stream: Connection timed out
in /home/okamstudio/
godotengine.org/wp/wp-content/plugins/insert-php/insert_php.php(48) :
eval()’d code on line 3
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2420.

The download page don't show any mirrors.


meta: Revert "main" to "./lib/index", no dist (fix package on master branch)

Template boilerplate (click to expand):
Please be sure to read the contributor's guide to the GitHub package before submitting any pull requests.
Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.

Description of the Change

Revert the "main" field of package.json to point to ./lib/index.js, rather than ./dist/index.js, which doesn't exist on master branch. (The dist/ folder was was created on/for the v0.36.13 tag, take a look at that tag to see what that's about.)
Makes the package work again on master branch. We can go ahead and change it back again if we create the dist/ folder for a future release.
Note: per @mauricioszabo the command to do this is: ELECTRON_VERSION=12 babel lib -d dist --compact auto --source-type module, but this implies you have a working babel binary on your PATH, and I couldn't figure it out, personally.
Screenshot or Gif

N/A
Applicable Issues

Not opened as an issue on here, but the package is broken on master branch. This fixes it for now.

If package.json "main" field points to this non-existent folder, then the package is effectively broken.
This commit restores "main" to point to "./lib/index", as this seems to be working now in my tests.
We can pre-compile the dist/ folder again when tagging the next release, if we really want to, but I'm pretty sure Pulsar compiles the code on first run, so the stakes for doing so seem pretty small to me. (Impact should be limited to improvements in first startup speed, if I understand correctly.)

Note: according to Mauricio, the command to build the dist/ dir is: ELECTRON_VERSION=12 babel lib -d dist --compact auto --source-type module
But this is not trivially obvious to me how to automate at the moment, so I won't try to do so for now. This just gets the repo working on master branch again, without bothering with pre-compilation stuff.

Tangential note: I hope to see the way the tests run here be improved, since as far as I can tell a lot of the tests are actually failing, and it is running tests for the editor's own specs, not the package specs? Or something along those lines?
Until then, I don't really think the test results apply or say much about the quality of this (or any) PR.
Thanks to the efforts to get those set up, they are as good a starting point as any.

@DeeDeeG a note about tests, Version 2.1 of the our own tester is very broken. So no, these tests do not have any weight at all.
Although after some recent work on our tester we might now be in a position to update the version here to run the tests properly, will have to see if that could work as I forget exactly where progress on that last left off.

@pulsar-edit/core any thoughts on this?

Basically it's whether we want this package to be hackable and work in its master branch state, such as by using ppm link (by merging this PR)
or not merge this PR and have one less thing to change if we transpile out to dist/ dir and tag a release, but the state of master branch requires manually editing package.json to be functional in the mean-time.

It looks like the github version we're using (which is atom's still as of 1.101.2023.01.19003 x64), has an atomTranspilers key in the package.json - which is what appears to be used by @atom/babel7-transpiler to do an on-the-fly transpilation in conjunction with the .babelrc file to tell babel what to do with it
Our version also has this key, and should work as intended without problems - unless a file was removed by accident

@Spiker985 transpilation on-demand (live when loading the package in the editor) works, at least it does when loading the package located/symlinked at ~/.pulsar/packages/github, but it doesn't work when the github package is bundled/ a core package in the editor, for some reason. Either because the files aren't in the same place, or due to some of it being in app.asar, or just the app skipping transpiling core/bundled packages...?
So until someone figures out how to transpile a bundled/core package at run-time/on first run, we still have to transpile before bundling.

Strictly looking at this PR's change: We can write the transpiled files right back out to the lib/ dir, instead of to a separate dist/ dir. Then we wouldn't need to mess with the "main" field of package.json again! Regardless of whether we do or don't need to pre-transpile.
So I'd like to merge this if possible, to make hacking on this package easier.

Honestly, doing the transpilation ahead of time, and committing it is probably for the best. Since we want to get away from "magic" processes

Gotta say I agree with the sentiment here, can't remember all the details of this PR, but I'm on board with keeping build tool outputs in the actual repo to keep our life, and Pulsar build times simple

This PR is, make the package work when ppm linked.
And implicitly the thing to do after this PR  is to transpile out to lib/ instead of to dist/.
Like so:

Before: ELECTRON_VERSION=12 babel lib -d dist --compact auto --source-type module (Note: -d dist)
After: ELECTRON_VERSION=12 babel lib -d lib --compact auto --source-type module (Note: -d lib)

And sure, we could look into transpiling once and just committing from there, but babel's output isn't to pretty at the moment, IMO. So I would hope we could output something nicer if we're going to commit that once and for all.
I also assume that JSX transpiles out to something weird, but eh, that's an assumption.

Can I get an approve on this? It's a one line change.
What's the benefit?: this PR fixes a thing (making the project ppm link, and probably makes it installable directly from its GitHub repo URL, for that matter, without it being broken in that use-case).
What's the cost?: in exchange for the benefit above, this requires a very small behavior change (-4,+3 chars) from whoever knew the magical incantation to transpile this repo before, which I think was only ever Mauricio. The behavior change in question?: Use the -d lib flag with babel, not -d dist. (As fully spelled out in my comment above).
THANK YOU IN ADVANCE IF I GET AN APPROVE.

It's been almost three weeks, I am absolutely going to merge this -4/+3 chars commit that I view as a strict correctness improvement, if I don't get an approve or disapprove in the next week. Thank you for your understanding.
A reminder, or a clarification, (or a whatever, I don't even know anymore at this point): A lot of this conversation is out of scope for fixing this package for the ppm link and ppm install https://github.com/pulsar-edit/github workflows.
I won't improve the transpilation workflow overall in this PR, it is way, way way out of scope. This PR just means we will start transpiling out to lib/, not dist/, and if that doesn't mean anything to you as a reviewer, or if it's over your head, please don't worry about it -- since I know how to run the babel command, and so does Mauricio, and nobody else has attempted it that I heard of, so the only two people who could be affected (me + mauricio) will decidedly be fine. It is again a -4/+3 change to the babel command required after this PR.
If that sounds too good or too simple to be true, well it's not. This really is a tiny drive-by correctness fix. If it seems too simple, it's because it is too simple for the conversation that has been going on here. It ain't as complicated as all that. That's stuff we could hypothetically do, not stuff I want to do in this PR.
THANK YOU!!!
/losing my mind a bit

@DeeDeeG Seems I never fully reviewed the changes here, and got caught up on the content of the conversation around it. So sorry for dwelling on our of scope factors.
But this one in my opinion is good for a merge, so we can go ahead

Lol, disregard my rant unless anyone needed a clarification as to how this conversation diverged from my fairly narrow intend with the actual commit diff, the actual change in the PR.


Server Resource Pack does not take priority.

Hello! First time posting on github, please bear with me if there are any issues with my post. I have a very unique issue pertaining to the server hoplite,gg
The servers custom textures do not take precedent despite being higher on the list in priority in the resource pack folder. I am propsiing one of two possible solutions:

Add an exclusion condition to exclude specific items from having a CIT texture applied to them
Allowing Servers to take precedent (I have no idea if this is even possible)

How to repeat the issue:

Create a Custom Item Texture for a Diamond Sword with Sharpness 1
Log into hoplite,gg, open the recipe book and compare the texture for "Midas Sword".

Attached are the relevant screenshots. Thank you!

Apologies but this is exactly by design that it does that. CITs do not behave like other resources with vanilla pack layering, they have their own priority system that dictate how they are applied after all resourcepacks are done.
To revert to having the sword that's in the server pack, you'd have to have a higher priority CIT(either in the server pack or in a separate client pack) decide what to do with the specific item.

I dont believe that the server actually uses CIT layering, as these textures are visible on a Vanilla client. This works perfectly fine on a regular optifine instance.

Yes. Exactly as I said, CITs are on one level over resourcepack ordering. They are supposed to override what the server is doing in its resourcepack.


External cluster rancher bootstrap fails after initial successful cluster import

Rancher versions:
rancher/rancher: 2.0.8
rancher/rancher-agent: 2.0.8
rke: 0.1.9
Infrastructure Stack versions:
kubernetes: v1.11.2-rancher1-2
Docker version:
17.03.2-ce
Using https://releases.rancher.com/install-docker/17.03.sh
Operating system and kernel:
Ubuntu 16.04.4 LTS
4.4.0-124-generic
Type/provider of hosts:
Private on-premises OpenStack Cloud
Unless otherwise mentioned, all servers get a "public" floating ip that the control plane uses to route traffic to servers; the servers themselves do not see the floating ip's assigned to them
All servers have private ip addresses from <IP_ADDRESS>/24 range

Cloud 1 - Rancher Master cluster - Floating IP range: <IP_ADDRESS>/21
Cloud 2 - Customer cluster 1 - Floating IP range: <IP_ADDRESS>/19
Cloud 3 - Customer cluster 2 - Floating IP range: <IP_ADDRESS>/19

Inter-cloud networking happens via floating ip addresses
Used OpenStack flavor: 2 core CPU, 8GB RAM, 32GB disk
All IP addresses used in all the servers involved
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
5-node HA Rancher master cluster, created using this rke template
Floating ip's assigned from <IP_ADDRESS>/21 range
Steps to Reproduce:
Customer clusters have 3 control plane/etcd nodes, 3 edge worker nodes, and 3 worker nodes without a floating ip.

Customer cluster 1 name: ces-es-si-os-dhn-78
Customer cluster 2 name: ces-es-si-os-dhn-80

Customer clusters are imported as follows:
Step 1: Initialize cluster in Rancher
rancher clusters create --k8s-version v1.11.2-rancher1-1 --rke-config /tmp/customer_rke_cluster.yml ces-es-si-os-dhn-78
/tmp/customer_rke_cluster.yml
Step 2: Extract registration token
rancher clusters add-node --etcd --management --worker ces-es-si-os-dhn-78
Step 3: Bootstrap Rancher nodes
For externally accessible nodes (etcd/control plane/edge worker)
docker run -d --privileged --restart=unless-stopped --net=host -v /etc/kubernetes:/etc/kubernetes -v /var/run:/var/run rancher/rancher-agent:v2.0.8 --server https://mastercluster --token <previously_extracted_token> --ca-checksum <previously_extracted_ca_checksum> --address <floating_ip> --internal-address <private_ip> --etcd --control-plane/--worker
For non-externally accessible nodes (worker)
docker run -d --privileged --restart=unless-stopped --net=host -v /etc/kubernetes:/etc/kubernetes -v /var/run:/var/run rancher/rancher-agent:v2.0.8 --server https://mastercluster --token <previously_extracted_token> --ca-checksum <previously_extracted_ca_checksum --address <private_ip> --worker
Results:
Customer cluster 1 import
Imports successfully, logs:

Master cluster, rancher/rancher
Customer cluster 1, rancher-agent
Customer cluster 1, syslog

Customer cluster 2 import
Control plane nodes begin Rancher bootstrap, but bootstrap does not progress. During provisioning, Rancher indicates it cannot connect to etcd ports on customer cluster 2. No port checker pods are running on customer 2 on any node; however Rancher has attempted to start port checker nodes on Customer 1.

Master cluster, rancher/rancher
Master cluster, docker journal
Customer cluster 2, rancher-agent
Customer cluster 2, additional rancher-agent never present during first import
Customer cluster 2, second additional rancher-agent never present during first import
Customer cluster 2, syslog
Logs from customer 1, with rke-tools -pods running as port checkers.
Customer cluster 1, docker ps -a, see first two rke-tool images
Customer cluster 1, syslog
Customer cluster 1, docker service journal

Probably because you are using the same hostnames in different clusters, see https://github.com/rancher/rancher/issues/15456 and https://github.com/rancher/rancher/issues/15943.
Let me know if that doesn't solve the issue.


docs: Remove expected GA for Node.js builder

Summary
Remove the expected GA for Node.js builder since it's stalled on SLSA v1.0 support (#2499)
Testing Process
N/A
Checklist

[x] Review the contributing guidelines
[x] Add a reference to related issues in the PR description.
[x] Update documentation if applicable.
[x] Add unit tests if applicable.
[x] Add changes to the CHANGELOG if applicable.

Temporarily removed DCO app as a requirement.


BUG: Double subscribe button

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There are 2 subscribe buttons.

Assign this issue to me under GSSoC.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There are 2 subscribe buttons.

Assign this issue to me under GSSoC.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

preopend issue!


RPI4 RuntimeError: Unknown platform.

Hi,
I have a homeassistant on a raspberry pi 4 and I want to use this addon to read temperature and humidity with a DHT22 sensor. But from what I have read, there is no support for RPI4 in the older version of the adafruit library.
More is written about it here: https://stackoverflow.com/questions/64093805/adafruit-python-dht-raspberry-pi-in-get-platform-runtimeerror-unknown-platf
LOG:
File "/usr/lib/python3.11/site-packages/Adafruit_DHT/common.py", line 63, in get_platform
raise RuntimeError('Unknown platform.')
RuntimeError: Unknown platform.

The addon installs the upstream software, so as it's not supported there isn't much to be done here.


Dead link in Readme

"Why was this opensources"-link refering to a dead blogpost.

http://web.archive.org/web/20130317230047/http://blog.indragie.com/post/1-2-open-source


Create BFF Binary

Implements a urfave v2 CLI binary for running the BFF server.

Codecov Report

Merging #382 (b6f5163) into main (b5d88bb) will decrease coverage by 10.61%.
 Consider uploading reports for the commit 3e141fb to get more accurate results

@@             Coverage Diff             @@
##             main     #382       +/-   ##
===========================================
- Coverage   48.29%   37.68%   -10.62%
===========================================
  Files          77      151       +74
  Lines        8818     2367     -6451
  Branches        0      606      +606
===========================================
- Hits         4259      892     -3367
+ Misses       3799     1302     -2497
+ Partials      760      173      -587

Impacted Files
Coverage Δ

directory/pkg/gds/tokens/mock.go

directory/pkg/gds/store/trtl/index.go

directory/pkg/gds/secrets/token.go

directory/pkg/gds/client/profiles.go

directory/pkg/sectigo/creds.go

directory/pkg/gds/client/editor.go

directory/pkg/trtl/replica/prob.go

directory/pkg/gds/certs.go

directory/pkg/gds/backup.go

directory/pkg/gds/store/index/index.go

... and 218 more

Continue to review full report at Codecov.

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


[Feature Request] Separate blocks and clickable links in the footer

I love this easy to design and manage link in bio page, with no bloated interface or superfluous bells and whistles. However, there are two things I really miss.
First, it would be great if you could design different blocks. For example a paragraph "Follow me on ...", a second one a la "Contact me on/under ..." and a block "Legal requirement" for e.g. the imprint or the privacy policy. I'm sure there are many more purposes.
Whereas I personally always like to put the legal requirements in the footer. For this it would be great if .. would be converted into clickable links.
That being said, it's a great project. Keep up the good work ...

Thank you!  Let me think of an implementation that will scale in the future!


Unable to Download Patreon Videos

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2021.12.27. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
Tried to download a Patreon video that's uploaded straight to the site, ones I've downloaded before were embedded into the site with Vimeo but this one is fully uploaded to Patreon, and it just refuses to download.
Verbose log
G:\Cold Ones>"D:\New folder\Files\youtube-dl\yt-dlp.exe" -vU --cookies patreon.com_cookies.txt https://www.patreon.com/posts/we-were-tricked-59857679
[debug] Command-line config: ['-vU', '--cookies', 'patreon.com_cookies.txt', 'https://www.patreon.com/posts/we-were-tricked-59857679']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2021.12.27 [6223f67] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: none
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2021.12.27, Current version: 2021.12.27
yt-dlp is up to date (2021.12.27)
[debug] [Patreon] Extracting URL: https://www.patreon.com/posts/we-were-tricked-59857679
[Patreon] 59857679: Downloading JSON metadata
WARNING: "url" field is missing or empty - skipping format, there is an error in extractor
[debug] Default format spec: best/bestvideo+bestaudio
ERROR: [Patreon] 59857679: Requested format is not available
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1353, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1437, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1489, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2535, in process_video_result
yt_dlp.utils.ExtractorError: [Patreon] 59857679: Requested format is not available

Just curious can VideoDownloadHelper Firefox Addon detect/download it? Or is it DRM protected by Patreon?

Just curious can VideoDownloadHelper Firefox Addon detect/download it? Or is it DRM protected by Patreon?

Yes, that works :)

I'm experiencing the same error.
When did Patreon roll out their own video hosting?

Experiencing an "Error 403: Forbidden" when trying to parse site metadata on Patreon, also specifically for embedded videos in patreon feeds. I'm a subscriber and have proper access allegedly (gallery-dl is able to parse the posts and download thumbnail images for the same users/posts successfully with the same cookies). I'm on the latest version of yt-dlp as well.
C:\Videos>yt-dlp --cookies-from-browser firefox "https://www.patreon.com/XXX/posts" -v
[debug] Command-line config: ['--cookies-from-browser', 'firefox', 'https://www.patreon.com/XXX/posts', '-v']
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "C:\Users\temp\AppData\Roaming\Mozilla\Firefox\Profiles\xsw24k31.default-1572046729019\cookies.sqlite"
[Cookies] Extracted 1113 cookies from firefox
[debug] Encodings: locale cp1252, fs utf-8, out utf-8 (No ANSI), err utf-8 (No ANSI), pref cp1252
[debug] yt-dlp version 2<IP_ADDRESS> [e491d06d3]
[debug] Python version 3.8.6 (CPython 64bit) - Windows-8.1-6.3.9600-SP0
[debug] exe versions: none
[debug] Optional libraries: brotli, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [PatreonUser] Extracting URL: https://www.patreon.com/XXX/posts
[PatreonUser] XXX: Downloading webpage
ERROR: [PatreonUser] XXX: Unable to download webpage: HTTP Error 403: Forbidden
(caused by <HTTPError 403: 'Forbidden'>); please report this issue on  https://github.com/yt-dlp/yt-dlp , filling out the "Broken site" issue template properly. Confirm you are on the latest version using  yt-dlp -U
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\common.py", line 617, in extract
    ie_result = self._real_extract(url)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\patreon.py", line 240, in _real_extract
    webpage = self._download_webpage(url, user_id, headers={'Cookie': '.'})
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\common.py", line 900, in _download_webpage
    res = self._download_webpage_handle(
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\common.py", line 768, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\common.py", line 753, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\extractor\common.py", line 735, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\site-packages\yt_dlp\YoutubeDL.py", line 3591, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\urllib\request.py", line 531, in open
    response = meth(req, response)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\urllib\request.py", line 640, in http_response
    response = self.parent.error(
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\urllib\request.py", line 569, in error
    return self._call_chain(*args)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\urllib\request.py", line 502, in _call_chain
    result = func(*args)
  File "C:\Users\temp\AppData\Local\Programs\Python\Python38\lib\urllib\request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

(XXX is a placeholder, this happens with any patreon creator)

Seeing the same issue here, with a creator that produces mainly audio content

To update, this doesn't happen if I provide a direct link to a single post with an embedded video; it will parse and download that correctly if I provide each page individually. The above problem only happens if I try to download an entire page or feed of posts.


ActiveGate webserver's CA mounted properly in OneAgents pods

Description
Host filesystem is now mounted as read-only, so ActiveGate CA cert can no longer mounted in /var/lib/dynatrace/oneagent/agent/customkeys.
Switched to path expected by OneAgent: /mnt/dynatrace/certs/activegate/
How can this be tested?

Create dynakubre cloudNativeFullstack monitoring enabled
kubectl patch -ndynatrace dynakubes.dynatrace.com dynakube --type=json -p='[{"op":"replace", "path":"/spec/activeGate /tlsSecretName", "value":"dk-certs"}]'
OneAgent pods should be in Running state eventually

Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly

what happens if we are not running in read-only mode ?
shouldn't we use the "old" path or this path works in that case aswell ?

what happens if we are not running in read-only mode ? shouldn't we use the "old" path or this path works in that case aswell ?

depends on OneAgent version and this is why it's not yet backported to release-0.5 branch. Let's discuss it offline.


Mark the loadXML call as blocking

Fixes #27.  Source compatible for everyone who uses it implicitly and has an Async.
The need for Sync was offensive enough in JSON that we made a capability trait.  Note that we can relax it back to Concurrent in #25.

I really hate introducing constraint churn, with what appears to be a better solution looming in 0.24 (I'm working on a bench).  But what's here in 0.23 is just simply not safe.


registration order shouldn't matter

 Issue by sigmundch
Originally opened as dart-lang/sdk#17926

I'm still seeing issues where the registration order is not working properly in Dartium. For example, if a sub-element is registered before the parent element.

 Comment by sigmundch

Removed this from the 1.6 milestone.
Added Polymer-P-1 label.

 Comment by sigmundch

Added this to the 1.6 milestone.
Removed Priority-Unassigned label.
Added Priority-Low label.

 Comment by sigmundch

Removed Polymer-P-1 label.
Added Polymer-Milestone-Next label.

 Comment by sigmundch

here is an example were this is happening:
https://codereview.chromium.org/219943003/

 Comment by jmesserly

Removed Area-Polymer label.
Added Pkg-Polymer, Area-Pkg labels.

 Comment by sigmundch

We implemented a fix for classes in the same library. Cross-library ordering is not planned as part of a transformation, but could be done at runtime.
See: https://github.com/dart-lang/polymer-dart/issues/17

Added MovedToGithub label.

 Comment by sigmundch

Added PolymerMilestone-Next label.

 Comment by sigmundch

Removed Polymer-Milestone-Next label.


Use PHPicker if file type is image on iOS

Closes #35
I just copied the logic you previously had to use the PHPicker, but it's running only if the desired file type is an image.
Let me know if something else is required. Maybe a different screen in the sample to select an image or a pdf and thus check that the document picker is correctly run would be good.

Looks good! Thanks for your contribution.

Yes, I think it would be a good idea if you can add a new screen with a pdf picker to have both pickers in the sample.

Hey @MohamedRejeb 👋🏻
Thanks for that.
I added the screen to check the behaviour of selecting images or files and added a fancy switch to decide if the launcher should allow single or multiple selections.
This led me to find that the file picker is not working properly on Android. Do you want me to open an issue for that?

I'll merge it, I was just waiting for the build result.
Yes, please open an issue for that, I will check it out.


Add configuration option

This adds a configuration option in order to set the system and component ID of the SDK, as well as correctly set the MAV_TYPE.
FYI @bkueng @cmic0.

@julianoes @JonasVautherin Is this good to go?

Needs testing by @cmic0.

@julianoes I did test it and it worked. According to @cmic0 he still need to do changes on the firmware side when running avoidance.

@RomanBapst great thanks.


Vendor<EMAIL_ADDRESS>

How to categorize this PR?

/area open-source
/kind enhancement
/platform openstack
What this PR does / why we need it:
Vendor<EMAIL_ADDRESS>Release note:

NONE

/squash


Trouble with a Large Model

Godot version:
3.1.1 Stable
OS/device including version:
Windows 10
Issue description:
I have a large city of about 500k triangles that I can't seem to manipulate in Godot. I finally exported it in two OBJs (having tried all other formats as well as exporting in a single OBJ) and I was able to put both into a scene. The first half is for some reason gray, the other fully textured. When I try to manipulate either half, the editor stops responding (but doesn't crash).

Would be helpful if you provided the model here.

@OriginalAGP Can you still reproduce this in 3.2.2?

Closing due to lack of reproduction project.
Can reopen if posted.


Inbetween nested join resource is being lost in ?include

I just bumped in to this little gem. It's a bit tricky to explain so I'll just start with a few examples:

/list?include=x - :+1: Includes x
/list?include=x.y - :+1: Includes y (through x)
/list?include=x,x.y - :+1: Includes x and y (through x)
/list?include=x,x.y.z - :+1: Includes x and z (through y, through x)
/list?include=x,x.y,x.y.z - :-1: Includes x and z (through y, through x) but not y (through x) as I would have expected.

I'm guessing I get's lost somewhere in the AREL query or serialised away somehow but I just wanted to see if it's the expected results.

All the case you listed should be :+1: . I'll take a look at why y isn't being included.


defer waitForDomainGone in case of error

Inside logs of EdenGCP I see lots of invalid domain identifier messages.
Seems, we need to defer waitForDomainGone to hyper.Task(status).Delete step in this case, without it Task for xen variant will never ends.
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

Do you or do you not need the same in the PV case?

Good point, will check it.

Do you or do you not need the same in the PV case?

You were right, there is the same behavior for PV. Now it is inside the PR.


Update restrictions wording

Related to #314

@Aivean "Доступ ограничен только своими постами" should not be shown for the case when user has no posts

@Aivean "Доступ ограничен только своими постами" should not be shown for the case when user has no posts

Debatable. The restriction is still there, even if user doesn't have any posts.


fix(firehose): http2_adaptive_window

We currently multiplex all firehose connections on a single http2 connection, and under load we're seeing possible flow control issues, possibly a hyper bug. The default per-connection http2 window is small and equal to the per-stream window. Enabling adaptive windows should give us a much bigger connection window, and possibly solve the issue at the current loads. This sets the connection window to the maximum value, effectively disabling it.

This now sets the connection window to the maximum value, effectively disabling it. I was able to locally reproduce that this fixes the block stream stalling issue. I'm also convinced this is not a hyper bug but is http2 flow control working as designed, for our use case we should opt out of the connection level flow control.


Inconsistent notification display

Describe the bug, optionally with screenshots
On mobile, Tlon Internal shows as having unread messages in the group list.

When I click into the group, no channels have a blue dot indicating an unread message.

Expected behavior
I'd expect to see a blue dot on at least one of the channels.
Additional context:

Ship type - planet
Instance type - self-hosted on digital ocean
OS/browser - Chrome on iOS

I don't think this is #1075 - I think the bug here is it requires a hard refresh to see the blue dots appear for the channels.

#1426 as well?

I think #1426 is also a different issue


[MoveClassToFile] Remove unused usings

When moving the class to its new file, it gets all using statements inside the origin file and puts them into the new file. It would be cool if the unused usings are removed insde the new file (also, preferably sorted).

fixed here https://github.com/filipw/Strathweb.Roslyn/pull/6


Can't get the drain functionality to work in the browser

I'm trying to test out the drain functionality in the browser but I'm not sure how to set it up. This is what I tried
import ulog from 'ulog'

ulog.use({
  outputs: {
    custom: {
      log: function (...args) {
        console.log(args)
      },
    },
  },
})

ulog.set('output', '*=custom')

This didn't seem to work, and neither did doing
localStorage.setItem('log_drain', '*=custom')

I'm expecting the custom.log function to be called but it's not being called. How would I get this to work in the browser?

Please have a look at this codepen. It sets a custom output.
I do think there are still bugs. Setting the drain does not seem to work.
I will keep this issue open for now and report back here. Thank you for your interest in ulog and taking the trouble to report issues. It is much appreciated!

Hey thanks for your reply and glad I can help. Interestingly it seems that it works in the codepen example, yet it doesn't work in my nextjs sample app. I confirmed that its<EMAIL_ADDRESS>that I'm installing and I'm doing exactly this...
import ulog from 'ulog'

export const log = ulog('sample-app')

ulog.use({
  outputs: {
    custom: {
      log: function (...args) {
        console.dir('custom...', args)
      },
    },
    drain: {
      log: function (...args) {
        console.dir('drain...', args)
      },
    },
  },
})

ulog.set('drain', 'drain')

Note sure why it works in your codepen but not in my app. Setting ulog.set('output', 'custom') works as expected though. So only the drain setting is the issue.

I have redesigned the way outputs, formats and levels work together (or not!) several times. I think I just messed up between the betas.

I'm actually redesigning it yet again. I'm in the process of making static formatters (the ones that don't mess up the call stack) much more powerful and configurable. Formatters work on outputs and the drain and in the process of redesigning the formatting pipeline so it could benefit from static formatters I also am redesigning how outputs and the drain work. So please ignore the issue for the drain for a bit more; I'm working on it.

👍

@ifeanyiisitor Please check out the latest beta and let me know what you think.
Feel free to re-open if you still encounter issues.

Trying this again. What i'm doing is this
import ulog from 'ulog'

export const log = ulog('sample-app')

log.level = log.WARN

ulog.use({
  outputs: {
    drain: {
      log: function (...args) {
        console.log('drain...', args)
      },
      warn: function (...args) {
        console.log('drain warn...', args)
      },
    },
  },
})

ulog.set('drain', 'drain')

log('foo...')
log.warn('warn...')

In the browser, the call log('foo') gets sent to drain. But the call log.warn('warn...') doesn't and I don't understand why.


[sdl2] Fix pkgconfig: missing system lib dbus-1

CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:232 (message):
  Library "dbus-1" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:298 (vcpkg_fixup_pkgconfig_check_files)
  ports/sdl2/portfile.cmake:97 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:79 (include)

Fixes #13236.

LGTM, thanks!


Master integration tests

This PR is a major refactor of how the images are built to allow better testing integration moving forward.
The major feature added is to implement integration tests, by implementing two apps that conform to the integration test protocol defined at integration testing. These two apps are:

test-1.0 which is designed to test the aspnetcore-1.0 runtimes.
test-1.1 which is designed to test the aspnetcore-1.1 runtimes.

The apps are stored in two different directories:

tests/apps/test-1.*, these directories contain the source for the apps, the script tests/build_apps.sh can be used to build and publish these apps to their final destintation.
tests/integration_tests/test-1.*, these directories contain the published apps, including the necessary metadata files, ready to be used for testing.

The tools/test.sh script is included for convenience when running the tests locally. It will deploy and run the integration tests for you in a similar way as to how it is done with the go integration tests.
The runtimes are now grouped by major version, all of them built in parallel with a single cloudbuild.yaml file that also includes all of the structural tests, and in the future more functional tests.
The runtime builder directory is now also updated and also built with a single cloudbuild.yaml that runs the structural tests and functional tests during the build process.

PTAL


MPI processor support

For interoperation with existing MPI libraries, we need a way for Legion applications to run tasks on an "MPI processor".  My initial thinking is that an MPI processor would correspond to a communicator and the task would be run on all the ranks in that communicator.  The processor would have affinity to the system memories on all ranks included in the (sub)communicator, and it would be the problem of the caller/callee to put the right subregions in the right order in the task requirements.  Alternately, this might be a use case for asking for a region requirement on an entire partition and having the mapper select instances for each subregion of the partition.

@streichler : would you implement some specific Legion communicator class for this?

No, the idea would be that, at Realm initialization-time, the application provides one or more MPI communicators for which Realm processors will be created.  A task running on an MPI processor would be able to ask what its communicator is use that.  It can also create and use subcommunicators within a task if it needs to.  All operations on/to/with MPI communicators will be handled by the MPI library, not Realm or Legion.

OK, that makes sense. I will try to come up with a good use case example for this.


Reloading table data with non-ajax datasource

Since this is somehow extremely hard to do, I'm adding in another API method to do this,
based on the API.ajax.reload() functionality. I'm working on a Fork so that a minified version becomes available, but here's my quick fix:
  /**
   * Reload tables from their javascript data source.
   * Note that this function will
   * automatically re-draw the table when the remote data has been loaded.
   *
   * @param {boolean} [reset=true] Reset (default) or hold the current paging
   *   position. A full re-sort and re-filter is performed when this method is
   *   called, which is why the pagination reset is the default action.
   * @returns {DataTables.Api} this
   */
  _api_register( 'js.reload()', function ( data, resetPaging ) {
    return this.iterator( 'table', function (settings) {

      _fnProcessingDisplay( settings, true );
      _fnClearTable( settings );

      for ( var i=0, ien=data.length ; i<ien ; i++ ) {
        _fnAddData( settings, data[i] );
      }

      _fnReDraw( settings, resetPaging===false );
      _fnProcessingDisplay( settings, false );
    } );
  } );

Could you clarify why you found this hard to do? 'table.clear().rows.add( data ).draw()' would clear the table, add data and redraw.

Hmm, that works perfectly.
I meant to say, it's extremely hard to find how to do this easily. I've searched high and low for a simple 'reload', 'refresh', 'new data' method but could not find one. There is no documentation on how to refresh/reload any other data source then Ajax, which explains the many stack overflow and other forum posts I did found using google. It would greatly help if this single line to reload an entire table's data could be found in the documentation!
Issue resolved, fork deleted. Thank you for responding.

Hi,
To my mind, the problem with calling it a 'refresh' or 'reload' of the Javascript sourced data is that it is not reloading from the same source (which is what the Ajax call does) - but rather you are feeding the data into the API in order to add it to the table. There can't really be a reload of the data since DataTables doesn't know where the data has come from and therefore can't request new data.
Instead, you clear the table and then add new rows. That is exactly what you did using the private methods. What I don't quite get is why you didn't just use the public API to achieve exactly that :-)

Aha. I see, and I agree. :)
I guess people (like me) start looking for a reload method because somehow it is the best way to (conceptually) describe what you want to achieve finally.
I also think that 'reload' like methods are so ingrained (at least for me) that because I did not find them, I did not think about just clearing, inserting and updating the table as a solution.
For me, I think it is also because I thought that maybe some 'object pooling' system was used.
If it did, then clearing, inserting and then redrawing would circumvent the pooling system and be less efficient / fast.


fix: give local extensions unique names

This PR covers the simplest implementation for solving #62: just create a uniquely-named declaration for each extension, and use than name when referring to the extension if the reference is in the same file in which the extension is declared.

This adds a local name even if not used, I would use a similar strategy to the import map and do a 2 pass thing, first traverse normally and eventually create a map of "locally used extensions => unique identifier" and as a second step traverse and add the locally used extensions

Not sure this way would be ok, the order is important as you can't use a constant before having it defined, the uniqueDefinitions needs to be intertwined with the statements, exactly after the definition statement.
Roughly:
1st step create a map of used definitions storing the definition node
2nd traverse the source adding the renames after each definition

It should only add the assignments for constants intertwined with the original declarations. The only assignments it hoists to the top are functions.


#15 farm buildings

~Not ready yet - initial pass adding a battery farm.~
~I've stripped out the fuel consuption logic for now to simplify while debugging interaction.~
~Current issue is that food can't be hauled to the farm, and I'm a bit stuck on the Work/Jobs system. Tried to take inspiration from the Draincasket from VRE-Sanguophage, but still have hit a wall.~

I believe this is ready for review/QA.
Needs some input on build cost, and should it be powered or is the food input enough?
Farm will accept up to 16 animals. You attach hoppers to the farm to load in food. Consumption is calculated based on the animals contained, and is triggered daily. If the animals are appropriately fed that day, they produce once per day (if they can produce). At the moment, the output is just placed on the ground for collection.
If animals are unfed, they don't die, they just don't produce.
I also added a cheekey little easter egg - there's a "Release Animals" button, and an "Eject Animals" button.

Looks like I introduced a bug before committing, the job to take an animal to the farm is broken somehow. It's listed twice in the context menu, and doesn't actually do anything. No time tonight to look into it though.

Right, latest push - an approximation of the normal game hunger mechanics. If left hungry, pawns get malnutrition.
Production speed it now based on the normal mechanics too, taking hunger/health into account.
Should be trivial now to add in some randomized scaling to increase the time between productions.
Question - Should we produce whenever ready, or keep a list of produced things, and once a day spawn them in bulk? One set of hauling jobs vs a slow stream through the day might be better?

Scaling of the produce interval added. It's now doubled, but configurable in the defs.
I think all that's left before it's ready is to figure out the buggieness in the order patching that's causing doubling up of jobs.

And there's a bug where pawns that die of starvation don't spawn back properly.

I think you're right that once per day is better. I also think the malnutrition/starvation should be optional like it is for the others.
There's a bool in settings AllowNeedsDrop which lets the user make that choice. Extra perf for less real farming. I think no outputs with no food should always apply just wrap the needs bit in if (PawnStoragesMod.settings.AllowNeedsDrop).

And there's a bug where pawns that die of starvation don't spawn back properly.

I don't think we should allow them to die in storage. I saw in the code you have an auto-release, I love that. Could be optional via gizmo if you like but if the animals get to 0 food and can't be fed, just release them to graze seems like a nice option. I don't think they should get to 100% malnutrition before release though because I think that straight up kills them.

Updated so it respects needsDrop now.
Added logic to make it so that produce gets dropped once per day.
Pawns are now ejected at 75% starvation, and a letter is sent.
Still need to look into the orders/jobs bug, but I think other than potentially adding power usage, we're good! :)

Woo, on power usage I don't think that's necessary. Extra ticks and it's a farm, it should work for medieval (meme texture aside 😄).


Use UseServer() instead of UseKestrel() to allow override in tests

This is needed to fix the build. Explanation is in the comments.
Review: @mikeharder @Tratcher
Approval: @Eilon @DamianEdwards

:shipit:


The nightly builds have been failing since Dec 12th

The nightly builds have been failing since Dec 12th. The error is:
[ 75%] Performing build step for 'libtiledbsoma'
[  5%] Building CXX object src/CMakeFiles/TILEDB_SOMA_OBJECTS.dir/soma/managed_query.cc.o
[ 11%] Building CXX object src/CMakeFiles/TILEDB_SOMA_OBJECTS.dir/soma/soma_array.cc.o
In file included from /home/conda/feedstock_root/build_artifacts/tiledbsoma_1703213999729/work/libtiledbsoma/src/soma/soma_array.cc:33:
In file included from /home/conda/feedstock_root/build_artifacts/tiledbsoma_1703213999729/work/libtiledbsoma/src/soma/managed_query.cc:33:
/home/conda/feedstock_root/build_artifacts/tiledbsoma_1703213999729/work/libtiledbsoma/src/soma/managed_query.h:40:10: fatal error: tiledb/tiledb: No such file or directory
   40 | #include <tiledb/tiledb>
      |          ^~~~~~~~~~~~~~~
compilation terminated.
make[5]: *** [src/CMakeFiles/TILEDB_SOMA_OBJECTS.dir/build.make:76: src/CMakeFiles/TILEDB_SOMA_OBJECTS.dir/soma/managed_query.cc.o] Error 1
make[4]: *** [CMakeFiles/Makefile2:100: src/CMakeFiles/TILEDB_SOMA_OBJECTS.dir/all] Error 2
make[3]: *** [Makefile:136: all] Error 2
make[2]: *** [CMakeFiles/libtiledbsoma.dir/build.make:86: libtiledbsoma-prefix/src/libtiledbsoma-stamp/libtiledbsoma-build] Error 2
make[1]: *** [CMakeFiles/Makefile2:85: CMakeFiles/libtiledbsoma.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

branch nightly-build
Azure builds of nightly-build branch
Nightly builds generated by GitHub Actions workflow nightly.yml (actions log)

The last passing build was with commit 1b6688b13e337aa09b700399824663b209ac6e8b and tiledb 2.18.2-h1f358a1_1
The first failing build was with commit 198b78e08d8c8584633f94476cc36c4cd3959720 and tiledb 2.18.2-hc1131af_2
I don't see why these changes could have caused a problem:
https://github.com/single-cell-data/TileDB-SOMA/compare/1b6688b13e337aa09b700399824663b209ac6e8b...198b78e08d8c8584633f94476cc36c4cd3959720

The tiledb conda binary 2.18.2-hc1131af_2 (installed in the failing build) was created after merging my PR https://github.com/conda-forge/tiledb-feedstock/pull/225 that ensured the correct libcurl is installed at runtime (commit, azure)
INFO :: The inputs making up the hashes for the built packages are as follows:
{
  "tiledb-2.18.2-hc1131af_2.conda": {
    "recipe": {
      "cxx_compiler": "gxx",
      "cxx_compiler_version": "12",
      "gcs": "gcs_enabled",
      "google_cloud_cpp": "2.12",
      "libabseil": "20230802",
      "libcurl": "8",
      "libxml2": "2.12",
      "lz4_c": "1.9.3",
      "openssl": "3",
      "target_platform": "linux-64",
      "zlib": "1.2",
      "zstd": "1.5"
    }
  }
}

The tiledb conda binary 2.18.2-h1f358a1_1 (installed in the passing build) was created from the previous merge (commit, azure)
INFO :: The inputs making up the hashes for the built packages are as follows:
{
  "tiledb-2.18.2-h1f358a1": {
    "recipe": {
      "curl": "8",
      "cxx_compiler": "gxx",
      "cxx_compiler_version": "12",
      "gcs": "gcs_enabled",
      "google_cloud_cpp": "2.12",
      "libabseil": "20230802",
      "libxml2": "2.12",
      "lz4_c": "1.9.3",
      "openssl": "3",
      "target_platform": "linux-64",
      "zlib": "1.2",
      "zstd": "1.5"
    }
  }
}

So literally the only difference was replacing the runtime pin on curl with libcurl. And this should have had zero effect, because I confirmed that curl ended up getting installed into the build environment in both the passing and failing builds

I created a test branch in my fork, nightly-build-with-older-tiledb, to specifically test if using the older tiledb binary 2.18.2-h1f358a1_1 would fix the build error. Answer: no
https://dev.azure.com/jdblischak/feedstock-builds/_build/results?buildId=536&view=logs&j=656edd35-690f-5c53-9ba3-09c10d0bea97&t=986b1512-c876-5f92-0d81-ba851554a0a3
note: the osx and win builds fail simply because I hardcoded the tiledb build string to a linux-specific one

Next I tried using the last passing commit, 1b6688b13e337aa09b700399824663b209ac6e8b. This enabled building libtiledbsoma!
https://dev.azure.com/jdblischak/feedstock-builds/_build/results?buildId=537&view=logs&j=656edd35-690f-5c53-9ba3-09c10d0bea97&t=986b1512-c876-5f92-0d81-ba851554a0a3

Lastly I reverted the tiledb pin. This time it installed tiledb 2.18.3. And the libtiledbsoma build still passed, so I think the problem comes from commit 198b78e08d8c8584633f94476cc36c4cd3959720

The nightly builds are still failing with the same error. Here's the build from last night

Last night's build passed thanks to PR https://github.com/single-cell-data/TileDB-SOMA/pull/2019 from @nguyenv :tada:


Update install-azurerm-ps.md

Wow - that's critical information that's missing.

@sdwheeler - Can you pls review this PR.

The change submitted has totally wrecked the formatting. I cannot tell what the change was supposed to be. I am rejecting this PR.
@Rick-Anderson - what information is missing? Send me email with the details.

@sdwheeler It's been so long I've forgotten. I think the new formatting is much better. I know there is some critical information missing in the original, can you look at the rendered doc?

@sdwheeler  for some reason the formatting got messed up. If you run through the doc you'll see some needed information is missing.

@sdwheeler  maybe this was the missing info

[!NOTE]> If you have not previously set the PSGallery repository as a Trusted repository, you may see a prompt like the following:

  +```Untrusted repositoryYou are installing the modules from an untrusted repository. If you trust this repository, change itsInstallationPolicy value by running the Set-PSRepository cmdlet. Are you sure you want to install the modules from'PSGallery'?[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "N"): Y```

@Rick-Anderson that was it. I will get this added.


Include EDPM Ceph client files to the list of roles

This patch includes the edpm_ceph_client_files role to the list of the edpm roles.
It will be used to copy on the external compute nodes both ceph.conf and the
related keyring.
Signed-off-by: Francesco Pantano<EMAIL_ADDRESS>

/lgtm
/approve
I was going to ask why this is necessary because it already merged in tripleo-ansible [1] so I thought it would be inherited but now I see that edpm-ansible only has a subset of roles from tripleo-ansible. In that case, yes, let's please merge this.
[1] https://review.opendev.org/c/openstack/tripleo-ansible/+/865197


8.1.0 Segment sampling is bad

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?
skywalking 8.1.0

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.
package org.apache.skywalking.oap.server.analyzer.provider.trace.parser.listener;
segmentId.hashCode() It's going to be negative。SW_TRACE_SAMPLE_RATE is configured to be positive

example：
segmentid ：40ae9aaa370449bd80b61452fcf04154.125.16003124206517460
hashCode：-103557884

Requirement or improvement

Please describe your requirements or improvement suggestions.

Thanks for your report, this is a bug for backend sampling.

Could you try to send a pull request to fix this? Adding Math#abs should be an easy fix.

#5506

i am a newbie and want work on this issue so please could you point me in the right direction


Move all references to stdfs:: into files.cpp

This change makes us treat std::filesystem as if it were a system API, and making paths follow our "internals are UTF-8" conventions. This fixes numerous path encoding bugs where we forgot to say `vcpkg::u8path", and makes it easier to add a completely POSIX backend so that we can run on old Linuxes.
Adds a new type, vcpkg::Path, which is an analogue of std::filesystem::path, except which always is UTF-8 internally. This means we can't represent everything on the file system, but we don't consider creating file system objects with invalid unicode names reasonable behavior.

Only for personal interest: can you explain why this is done? To reduce include dependencies? :)

Only for personal interest: can you explain why this is done? To reduce include dependencies? :)

2 major reasons:

The status quo of "vcpkg is internally UTF-8 stored as char* everywhere" and "std::path considers char* the active code page" caused numerous encoding bugs where we forgot to say u8path, and this closes that hole.
We want to bring vcpkg to older linuxes and macos which don't have std::filesystem, so we need a pure POSIX backend for things; this localizes anything that would need to be part of that backend to files.cpp.

I think this change resulted in some include problem on our builds for conda-forge: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=363464&view=logs&jobId=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&j=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&t=933f325c-924e-533d-4d95-e93b5843ce8b
Would you have a second to check that?

I think this change resulted in some include problem on our builds for conda-forge: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=363464&view=logs&jobId=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&j=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&t=933f325c-924e-533d-4d95-e93b5843ce8b
Would you have a second to check that?

The Windows build failures there are probably expected; now that we ship binary releases we only worry about building vcpkg itself with current (e.g. 2019) on Windows (which also means we don't have to workaround std::experimental::filesystem's many Many MANY bugs). The MacOS one is more concerning; I'm currently working on making binary releases possible for older versions of that too by ripping out the std::filesystem dependency, but I didn't expect intermediate broken there.
Is that attempting to use "macOS-10.14" Pipelines machines and that's the trigger or is something else going on?

Great, I can upgrade the conda-forge config to use VS2019 if that's fixing it.
On conda-forge we're using the 10.9 SDK for macOS, but a recent clang version.
These builds used to work before and if I looked correctly then you were using that symbol before as well, no? Could it be just a missing include?

Missing include is entirely possible

Yes, this looks exactly like a missing include -- we aren't including <ios> in files that use std::io_errc::stream.

Could you try https://github.com/microsoft/vcpkg-tool/pull/158 and confirm if it solves the problem?

Could you try #158 and confirm if it solves the problem for MacOS?

Ping @wolfv

hey, sorry, I was off on the weekend :)
I applied the patch from #158 and that works on macOS! On Windows, I still have issues after switching to VS2019, although those seem to be in optional: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=364375&view=logs&jobId=a70f640f-cc53-5cd3-6cdc-236a1aa90802&j=a70f640f-cc53-5cd3-6cdc-236a1aa90802&t=f5d15007-a01c-5ad8-c9ce-4d519d3b275f

I was just making sure you were notified/ tagged since you weren't @'d before not being impatient :)
The windows one is probably still expected,  as indicated previously

Thanks, no worries at all :)

On Windows, I still have issues after switching to VS2019, although those seem to be in optional: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=364375&view=logs&jobId=a70f640f-cc53-5cd3-6cdc-236a1aa90802&j=a70f640f-cc53-5cd3-6cdc-236a1aa90802&t=f5d15007-a01c-5ad8-c9ce-4d519d3b275f

That log still says 2017

you're right. sorry. I just pushed another set of modifications that should properly activate VS 2019. Fingers crossed!

No problem, I broke you after all :)

all green now! thx


Add reorder tautomers function and accompanying tests

Reference Issue

Initial PR for Google Summer of Code 2020. Implements the reorder_tautomers function as detailed in Greg Landrum's blog post and adds a test file with a basic functionality test.
Any other comments? For Greg: as for the other changes, I'm guessing they come from the copied binaries, some of which were in beta?

@chriswzou : thanks for this! I will merge it after the CI builds pass


Load default directory from resources if there is no variable specified and relaunch with same arguments for privilege escalation

The current version of 4.3 broke the ability to specify default install directories using the TargetDir.os.xxx resource files as described in http://izpack.org/documentation/panels.html#targetpanel.

This fixes the problem by reading in and using the resource files if no TargetDir.os.xxx variable is specifically specified.

In addition, in the current version if there is a privilege escalation under Windows, any command line arguments specified are lost when the application is relaunched with a higher privilege.  This was fixed as was calling the correct java executable: javaw.exe for GUI and java.exe for command line.

I'm not sure why it's showing that the entire TargetPanel.java file changed as there were only 4 lines added.

Thank you for contributing.
Unfortunately, this pull request has build errors, can you review it, please.
Is there anyone specializing on the IzPack 4 branches to review this also?

Closing for inactivity.
Please reopen or send a new pull request if this is still relevant for you.


Add navigation

@doboianh thank you for your contribution

@doboianh thank you for your contribution

thank you for your support.


fix: brc-20 pending transactions

Try out this version of Leather — Extension build, Test report

I have endless brc-20 transactions in my wallet, bc of unprocessed error response from ordinals bot, wonder how we want to fix it, just remove tx from state here?

@kyranjamie wonder here if we need to clear order from redux state if it is not found?

@kyranjamie wonder here if we need to remove order from redux state if it is not found?

Yes, I think so. I recall having concerns about stale data when we built it, but we never got around to addressing. If API says it isn't there, we should indeed remove.


534507D2 - Condemned: Criminal Origins

Validation

[X] I've read the FAQ.
[X] The Xenia build used is from a Canary branch. (not UWP/MLBS/AlexVS/master/pull requests, etc)
[X] This issue isn't for help (tech support) with Xenia, or a game.
[X] I've checked if this game doesn't already have a compatibility report.

Wiki page (optional)
https://github.com/xenia-canary/game-compatibility/wiki/...
Xbox 360 Marketplace link (optional)
No response
Xenia Canary version
https://github.com/xenia-canary/xenia-canary/commit/2b49fc2ddc1c45d804cac14d8118f0f0d12d54ea
Issues
Fully playable.
Even at 1440p no flickering.
To unlock fps: change "MaxFPS" "45" to "0" in autoexec.cfg.
Log
xenia.log
Screenshots

Labels
state-playable

Where is autoexec.cfg. file??? I cant find it anywhere