Terragrunt in Centos

I'm trying to execute terragrunt in a Centos container to bring AWS stack which is built from below snippet.
FROM centos:latest
RUN yum install -y curl unzip
RUN curl -O https://releases.hashicorp.com/terraform/0.11.8/terraform_0.11.8_linux_amd64.zip \
    && unzip terraform_0.11.8_linux_amd64.zip \
    && mv terraform /usr/bin/terraform
RUN curl -O https://github.com/gruntwork-io/terragrunt/releases/download/v0.17.0/terragrunt_linux_amd64 \
    && mv terragrunt_linux_amd64 /usr/local/bin/terragrunt \
    && chmod +x /usr/local/bin/terragrunt

Output
[root@7f7055edd444 governor]# terragrunt plan
/usr/local/bin/terragrunt: line 1: syntax error near unexpected token `<'
/usr/local/bin/terragrunt: line 1: `You are being redirected.'

The problem doesn't seem to be with terragrunt as it works if we consider linuxbrew based install and some explicit package is missing.
Do we need to install any specific package in Centos to proceed?

Hi there,
The problem clearly is that you are not downloading the binary of neither Terraform and Terragrunt in your Docker image (your error is for Terragrunt as you invoke it).
Change the curl -O command with wget and you are good to go.

I'm trying to execute terragrunt in a Centos container to bring AWS stack which is built from below snippet.
FROM centos:latest
RUN yum install -y curl unzip
RUN curl -O https://releases.hashicorp.com/terraform/0.11.8/terraform_0.11.8_linux_amd64.zip \
    && unzip terraform_0.11.8_linux_amd64.zip \
    && mv terraform /usr/bin/terraform
RUN curl -O https://github.com/gruntwork-io/terragrunt/releases/download/v0.17.0/terragrunt_linux_amd64 \
    && mv terragrunt_linux_amd64 /usr/local/bin/terragrunt \
    && chmod +x /usr/local/bin/terragrunt

Output
[root@7f7055edd444 governor]# terragrunt plan
/usr/local/bin/terragrunt: line 1: syntax error near unexpected token `<'
/usr/local/bin/terragrunt: line 1: `You are being redirected.'

The problem doesn't seem to be with terragrunt as it works if we consider linuxbrew based install and some explicit package is missing.
Do we need to install any specific package in Centos to proceed?

In your case curl -O downloads the HTML page of HTTPS permanent redirect. Change your curl -O command to wget in the Docker image file and your are good to go.

my bad, i was in the assumption terragrunt link point to a file and not https page. Thank you so much for the clarification!


Compile Error: Cannot declare class XXController, because the name is already in use

Hi,
Have declared my grapqlite.yaml in Symfony
# Read more about GraphQLite available options at: https://graphqlite.thecodingmachine.io/docs/symfony-bundle

graphqlite:
namespace:
controllers: App\GraphqlController
types:
- App\
I have created a controller  GraphQLTestController with simple hello function
    #[Query]
    public function hello(string $name): string {
        return "hello $name";
    }

Who work well, after some problems ;)
After I tried to create a new Controller Product (ProductController), since my project have crashed with the error:

Compile Error: Cannot declare class App\HttpController\GraphQLTestController, because the name is already in use

I already change namespace unsuccessfully, it's very strange error ;)

This looks like a Symfony issue to me.  I'm guessing that compile error is related to your service container's complication pass.  Please make sure you're not registering this namespace twice.


yaml-dsl: enable flow mode deserialization by default

Some code may be removed with camel 3.11 , see CAMEL-16693


librealsense: 2.42.0 -> 2.43.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/IntelRealSense/librealsense/releases.
meta.description for librealsense is: "A cross-platform library for Intel® RealSense™ depth cameras (D400 series and the SR300)"
meta.homepage for librealsense is: "https://github.com/IntelRealSense/librealsense"
meta.changelog for librealsense is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-hello-realsense had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-save-to-disk had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-depth had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-color had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-distance had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-pose had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-pose-predict had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-pose-and-image had a zero exit code or showed the expected version

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-convert passed the binary check.

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-enumerate-devices passed the binary check.

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-fw-logger passed the binary check.

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-terminal passed the binary check.

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-record passed the binary check.

/nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin/rs-fw-update passed the binary check.

6 of 14 passed binary check by having a zero exit code.

6 of 14 passed binary check by having the new version present in output.

found 2.43.0 with grep in /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0

found 2.43.0 in filename of file in /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0

directory tree listing: https://gist.github.com/99f69a864c105e81f593fb60c271af2c

du listing: https://gist.github.com/560b49dfd87c33cb56c896e717988717

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

107 total rebuild path(s)

32 package rebuild(s)

32 x86_64-linux rebuild(s)
31 i686-linux rebuild(s)
21 x86_64-darwin rebuild(s)
23 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
crlfuzz
deja-dup
deno
duplicity
duply
gnss-sdr
gnuradio3_8
gnuradio3_8Minimal
gnuradio3_8Packages.gnuradio
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradio3_8Packages.rds
gqrx
inspectrum
librealsense
librealsenseWithCuda
librealsenseWithoutCuda
or-tools
python38Packages.mypy-protobuf
python38Packages.ortools
python38Packages.pyrealsense2
python38Packages.pyrealsense2WithCuda
python38Packages.pyrealsense2WithoutCuda
python39Packages.mypy-protobuf
python39Packages.ortools
python39Packages.pyrealsense2
python39Packages.pyrealsense2WithCuda
python39Packages.pyrealsense2WithoutCuda
qradiolink
tflint
wifish
wpa_supplicant

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0
ls -la /nix/store/156cq44h4mis7xyvl4i7l1xgm9pd2kzn-librealsense-2.43.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @brian-dawn for testing.

Result of nixpkgs-review pr 118511 run on x86_64-darwin 1

3 packages marked as broken and skipped:

librealsenseWithCuda
python38Packages.pyrealsense2WithCuda
python39Packages.pyrealsense2WithCuda

3 packages failed to build:

librealsense (librealsenseWithoutCuda)
python38Packages.pyrealsense2 (python38Packages.pyrealsense2WithoutCuda)
python39Packages.pyrealsense2 (python39Packages.pyrealsense2WithoutCuda)

So all I can say for my process is I followed the instructions to build locally then tested it. It seems the build is working fine at least on linux? I have been using 2.43 on unstable so things seem to have built fine. Is it maybe just darwin that failed?
I'm curious to know more about this process too :) maybe I should be disabling darwin for this package if it's just going to keep failing?


allow dismissing service alerts [CA-1053]

This adds a 'Dismiss' link to service alerts, which allows users to suppress the alert for 1 day. Details are as follows:

This only affects service alerts (the notifications published by comms)
Pressing 'Dismiss' will hide that specific alert for 1 day, even across page reloads or new tabs.
The identity of an alert is based on its full contents, so any changes to an alert will be considered a new alert (bypassing the previous dismissal)

Since we're using LocalStorage to save the dismissals, some caveats apply:

If the user opens an incognito window or a different browser, their dismissals will not carry over.
In the event we run out of space in LocalStorage, previous dismissals might be removed, meaning alerts could show up again (this is unlikely, but possible)
If the user clears their browser data (cookies, etc), it will clear previous dismissals, meaning alerts could show up again.

Would you mind adding what testing you did as part of implementing the feature to the PR description?

Would you mind adding what testing you did as part of implementing the feature to the PR description?


Custodial account UI

This PR updates the custom session UI to include custodial accounts.

Depends on #451

Had to rebase this branch after updating the custom-connect-ui branch with the latest changes from master

Can we merge this into the custom-connect-ui branch and then merge https://github.com/lightninglabs/lightning-terminal/pull/451 as a whole? Looks like we got a bunch of reviews already (or do we want to wait for @dstrukt's review?).

Agreed @guggero


Update E3SM Diags environments

Update E3SM Diags environments (due to https://github.com/E3SM-Project/e3sm_diags/pull/551).
e3sm_diags_env_dev should be updated to e3sm_diags_dev:

https://github.com/E3SM-Project/zppy/blob/2008a8c0e2204537442994651d4778e267faa96b/zppy/templates/e3sm_diags_vs_model.bash#L10
https://github.com/E3SM-Project/zppy/blob/2008a8c0e2204537442994651d4778e267faa96b/tests/integration/test_environment_commands.cfg#L39
https://github.com/E3SM-Project/zppy/blob/2008a8c0e2204537442994651d4778e267faa96b/tests/integration/test_complete_run.cfg#L80

Also update the e3sm_diags_env_dev_20210827 environment.

This will be resolved in #195, which removes e3sm_diags_vs_model.bash, removes the environment_commands in test_complete_run.cfg, and updates the dev environment used in test_environment_commands.cfg.

Decided to resolve this issue with #207 instead.


mat-select disabled status doesn't refresh correctly

Bug
I'm creating a form in Angular with a mat-select input, bound to a FormControl, which should be disabled or not, according to a boolean, when the form is created.
What is the expected behavior?
The mat-select should be disabled/enabled according to the FormGroup disabled/enabled status.
What is the current behavior?
The mat-select stays enabled/disabled even when the FormGroup disabled/enabled status change.
What are the steps to reproduce?
step 1: The form is created and the FormGroup is disabled in the initForm method, according to the if condition. The mat-select is correctly disabled.
step 2: Click on the "Toggle" button change the boolean to false and calls the initForm method, so this time the FormGroup is not disabled ==> the mat-select button is still disabled.
https://stackblitz.com/edit/angular-qy1hqc?file=src%2Fapp%2Fapp.component.ts
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 5.2.9, Material 5.2.4

Closing as this is working as expected.  The form element is not changing its disabled state visually because rather than toggling the disabled state of the form control, you are creating an entirely new form group.  This form element is not a member of the new form group you are creating.  You can see a working fork of your stackblitz here .


[chore] remove mirai dataflow analysis to try to cut down on deps(#62)_29

Motivation
[chore] remove mirai dataflow analysis to try to cut down on deps
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
CI/CD tests are covered.

/canary

:sunny: Canary successful

/land

:lock: Merge Conflict

/land

@sahithiacn :exclamation: This PR is still missing approvals, unable to queue for landing

/canary

:lock: Merge Conflict


Stylix Roadmap

This tracking issue outlines my upcoming Stylix work, serving as a reference and personal reminder.

Clean up codebase to simplify future development and boost motivation
Add reliable tests to increase code confidence
Optionalize wallpaper functionality 2 5 early due to high demand
Support base24 scheme 4 to enable VIM highlight groups support
Support VIM highlight groups 3 to resolve styling inconsistencies
Implement slideshow 1 6 which will arguably be the coolest Stylix feature

The first two tasks are expected to be the most time-consuming, with the last one being the most technically demanding.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Write manual doc from deploying/ testing

[x] Create AzUrlShortener headless
[x] Create Azure Active Directory (AAD) components
[x] Configure the AAD components
[ ] How to deploy the TinyBlazorAdmin website
[ ] Configure the backend and the front

Trying to follow the steps in the current manual doc but I am not successful on closing the loop on the final step. https://github.com/FBoucher/TinyBlazorAdmin/blob/main/deployment.md#configure-backend-and-frontend-to-work-together
Same as one of the unfinished checks in this issue.
I'd be happy to update the doc once I have managed to follow.
The two keys to enter into appsettings.json; which IDs from the steps above are they? I assued it was the Tenant ID of my AD and the Client ID of the 1st registered app but that did not work for me.

@allcontributors add cmatskas for security, bug


What does "RuntimeError: frame == _episode->GetState()->GetTimestamp().frame" mean?

Describe our question or idea
When I run the challenge mode (routescenario), I sometimes get the error when call world.click(). The scenario running may not be influenced by it, but I am wondering why this happens.
Expected behavior
No error appears
Screenshots
Traceback (most recent call last):
File "/home/wenhao/scenario_runner/srunner/scenariomanager/scenario_manager.py", line 272, in _tick_scenario
CarlaDataProvider.get_world().tick()
RuntimeError: frame == _episode->GetState()->GetTimestamp().frame
Desktop (please complete the following information):

OS: Ubuntu 18.04
CARLA Version CARLA 0.9.6
Python version 3.7
Version  0.9.6

Hi,
when CARLA cannot perform the next tick due to whatever reason, CARLA raises an exception:
https://github.com/carla-simulator/carla/blob/4738f734de84f9547a6c30e70c4b840ecc43181c/LibCarla/source/carla/client/detail/Simulator.cpp#L122
Is this a reproducible problem?

Hi,
thanks for the information! It is reproducible, but there is only about 20% probability that this exception happens. I am still collecting more information about it. I will close this issue now and reopen it when I can provide more details!


Modifying log contents in hadoop-tools/hadoop-archive-logs/src/main/java/org/apache/hadoop/tools/HadoopArchiveLogsRunner.java

The following log line  LOG.info("Deleting original logs"); evaluated against the provided standards: 1. The log line does not include a parameter. It would be more useful to include the path of the original logs being deleted. 2. The log line does not include sensitive information. 3. The log message is concise and informative. 4. The log message is not for an exception. Due to the violation of standard (1), we would recommend a code change to include the path of the original logs being deleted.

Created by Patchwork Technologies.

👍


Selecting multiple default items not working?

Why isn't this simple example working?
http://jsfiddle.net/NdQbw/394/
Only the first item gets selected here?

Anyone?

MY bad, found out it is deprected and should instead use: $("#select").val(selectedValues).trigger("change");


Tables on mobile

Should probably scroll sideways instead of overflowing. (Subjective)

Working on this now. Searching for the issue.

Fixed.


Can't fully download the dataset

Hello! Thanks for your works.
I tried to download SeeingThroughFogCompressed from chrome or wget.
However, the download was said to be done before downloading about 5% of the dataset.
I have tried to download it a lot of times, but it didn't work.
Is there any way to reliably download the dataset?

Hi giho-sung,
can you explain your problem in more detail? Do you have an error message?
Typically the link gets replaced every 2-4 weeks due to technical reasons. Then it is necessary to register again to obtain a new link.
Other than that the server capabilities should be sufficient enough to download core zip files within a day. Can you start downloading single zip files? For example the Lidar zip is one of the smallest.
Best,
Mario

Hi! MarioBijelic,
These are the details of the error and what to do in the next step.
Detail:
The downloads usually ended up without any error messages such as it worked well.
But when I checked, the amount of data were 1 to 15GB of 500GB or 700GB.
Sometimes, I can see an error message on the network error.
Next:
(1) downloading single zip files in the same network as you suggested
(2) downloading directories in another network
I will post how it goes on later
Best regards,
giho

@giho-sung How long does download take for you?
My speeds are ~300-500 kbps for SeeingThroughFogCompressed.zip. I compared the download speed to NuScenes (at least 12 mbps) and it is much slower. I'm downloading from California, if that makes a difference. @MarioBijelic Any idea how long the data download usually takes?


Relax OpenSSL version requirements for GCM

GCM is supported by this version of OpenSSL (e.g. see here), but this doesn't seem to be reflected in the gem.

Updating your OpenSSL version seems better.

Agreed - but it doesn't seem unreasonable to include in the list of supported versions a supported version! I'm under impression that it's one provided by Ubuntu post-Heartbleed.

OK, then let's support all 1.0.1


[Bug] Error in downloading GSM8k dataset

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] The bug has not been fixed in the latest version.

Type
I'm evaluating with the officially supported tasks/models/datasets.
Environment
offical way to git clone repo and pip install -e .
Reproduces the problem - code/configuration sample
I am trying to eval gsm8k offline on my computer, but face an error in:
(opencompass) yang@DESKTOP-TNJH297:~/Downloads/opencompass$ python run.py configs/eval_demo.py
10/31 15:25:11 - OpenCompass - INFO - Current exp folder: outputs/default/20241031_152511
10/31 15:25:11 - OpenCompass - WARNING - SlurmRunner is not used, so the partition argument is ignored.
Traceback (most recent call last):
File "/home/yang/Downloads/opencompass/run.py", line 4, in
main()
File "/home/yang/Downloads/opencompass/opencompass/cli/main.py", line 299, in main
tasks = partitioner(cfg)
File "/home/yang/Downloads/opencompass/opencompass/partitioners/base.py", line 83, in call
tasks = self.partition(**model_and_dataset_args,
File "/home/yang/Downloads/opencompass/opencompass/partitioners/num_worker.py", line 69, in partition
dataset_size = self.get_size(dataset)
File "/home/yang/Downloads/opencompass/opencompass/partitioners/num_worker.py", line 141, in get_size
dataset = build_dataset_from_cfg(dataset)
File "/home/yang/Downloads/opencompass/opencompass/utils/build.py", line 13, in build_dataset_from_cfg
return LOAD_DATASET.build(dataset_cfg)
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/mmengine/registry/registry.py", line 570, in build
return self.build_func(cfg, *args, **kwargs, registry=self)
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/mmengine/registry/build_functions.py", line 121, in build_from_cfg
obj = obj_cls(**args)  # type: ignore
File "/home/yang/Downloads/opencompass/opencompass/datasets/base.py", line 12, in init
self.dataset = self.load(**kwargs)
File "/home/yang/Downloads/opencompass/opencompass/datasets/gsm8k.py", line 23, in load
dataset = MsDataset.load(dataset_name=path)
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/modelscope/msdatasets/ms_dataset.py", line 296, in load
with load_dataset_with_ctx(
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/contextlib.py", line 135, in enter
return next(self.gen)
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/modelscope/msdatasets/utils/hf_datasets_util.py", line 1359, in load_dataset_with_ctx
dataset_res = DatasetsWrapperHF.load_dataset(*args, **kwargs)
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/modelscope/msdatasets/utils/hf_datasets_util.py", line 929, in load_dataset
builder_instance = DatasetsWrapperHF.load_dataset_builder(
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/modelscope/msdatasets/utils/hf_datasets_util.py", line 1094, in load_dataset_builder
builder_instance: DatasetBuilder = builder_cls(
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/datasets/builder.py", line 371, in init
self.config, self.config_id = self._create_builder_config(
File "/home/yang/miniconda3/envs/opencompass/lib/python3.10/site-packages/datasets/builder.py", line 576, in _create_builder_config
raise ValueError(
ValueError: Config name is missing.
Please pick one among the available configs: ['main', 'socratic']
Example of usage:
load_dataset('gsm8k', 'main')
One solution to this problem is try to modify the dataset wrapper, but it's not the perfect way to do so. and for security after downloading, come back to the previous version.

hope it helps and better way to solve this problem
Reproduces the problem - command or script
python run.py configs/eval_demo.py
Reproduces the problem - error message
with read_base():
# from opencompass.configs.datasets.math.math_4shot_base_gen_43d5b6 import math_datasets
from opencompass.configs.datasets.gsm8k.gsm8k_gen_17d0dc import gsm8k_datasets
# from opencompass.configs.models.qwen2_5.hf_qwen2_5_7b_instruct import models as mymodel
datasets = gsm8k_datasets
Other information
No response

Which version of OC do you install? Try to pull the latest code


markup: Allow arbitrary Asciidoc extension in unsafe mode

This PR implements proposal #7698 which allows for arbitrary Asciidoc extensions in unsafe mode expanding the range of possibilities when using this markup language.
The proposal allows for arbitrary extension when SafeMode option is set to unsafe.  Any stricter mode would prevent arbitrary extensions and only those listed in the configuration would be allowed.

 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.gzagatti seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.gzagatti seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

How is this PR making untruested sites safer to build?

How is this PR making untruested sites safer to build?

By default AsciiDoctor has this security level selected. The only thing this PR does is add something very important to the integration with AsciiDoctor, the use of extensions.
If this PR is rejected, another way to use custom extensions should be allowed.

By default AsciiDoctor has this security level selected. The only thing this PR does is add something very important to the integration with AsciiDoctor, the use of extensions.
If this PR is rejected, another way to use custom extensions should be allowed.

Looks good to me, agree, @moorereason ?


fix: packageManager isn't configured properly

The recent node.js returns "Usage error" if packageManager isn't
semver. This commit fixes the versions to semver complient.
Also, commit-msg hook is still using yarn. Since the package manager
for this repository is pnpm, this commit also updates the hook.

Usage Error
❯ node -v
v18.15.0
❯ pnpm -v
8.2.0

# On the repository root
❯ pnpm -v
Usage Error: Invalid package manager specification in package.json; expected a semver version

$ pnpm ...

@riywo  That is interesting, the current value works for me doing the following as you described.
On the repository root
❯ pnpm -v
Outputs: 8.0.0
After upgrading from 8.0.0 to 8.2.0 locally it worked as well.
❯ pnpm -v
8.2.0
Does "packageManager" value support non-explicit version - like<EMAIL_ADDRESS>Would be ideal to not pin to explicit version so we don't need to maintain.
Are you using "corepath"? I added support but haven't tried it yet, curious if that is where the error is generating from.

@riywo If we can't use non-explict version, might just be best to remove the "packageManager" value for now, since is experimental and for this project not being used atm. Unless you are using it?
If you're using corepath and this feature that is great, and we can pin like you have. Let know either way, and good to go for this.

@JeremyJonas You mean corepack? Then, yes, I use corepack for pnpm.
It looks like this doesn't happen with locally installed pnpm:
❯ npm i pnpm

added 1 package, and audited 2 packages in 528ms

1 package is looking for funding
  run `npm fund` for details

found 0 vulnerabilities

❯ cat package.json
{
  "packageManager": "pnpm@8",
  "dependencies": {
    "pnpm": "^8.2.0"
  }
}

❯ npx pnpm -v
8.2.0

The version's description in the doc has nothing but it looks like pinned:
❯ cat package.json
{
  "packageManager"<EMAIL_ADDRESS>}
❯ pnpm -v
8.0.0

❯ cd ~
❯ pnpm -v
8.2.0

I would suggest not to opt-in packageManager now since it's experimental and only supports pinned version. Either getting rid of it or pinning the version is the only way to support corepack users at this moment.

@riywo Ok lets remove packageManager from package.json, that will be clearest fix. Thank you!

Closing this PR given it has been implemented in: #339


fix(CustomerSatisfaction): error on display images on storybook

Summary
Type

Bug

Summarise concisely:
What is expected?
Small bug on storybook with customer satisfaction.
Relevant logs and/or screenshots

Page
Before
After

url

If we select the third value, are the first three images enabled or only the first?


Trying to use 'columnrange'

Hi,
I have set more to true.
const chartOptions = {
  more: true,
  title: {
    text: 'Solar Employment Growth by Sector, 2010-2016',
  },

and in my  series I try and use columnrange
series: [
    {
      name: 'Installation',
      type: 'columnrange',
      data: mydata,
    },
  ],

but I get
Uncaught (in promise) Error: Highcharts error #17: www.highcharts.com/errors/17/?missingModuleFor=columnrange

missingModuleFor: columnrange

Any ideas why?
Thanks

I believe that "columnrange" uses highcharts-more. You can enable that with the more prop:
<highchart more :options="chartOptions" />

Let me know how it goes.


debug.print varA

https://docs.microsoft.com/en-us/dotnet/visual-basic/misc/bc30800

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

ID: a958f3e5-0207-69f7-4fc9-82ac080fa7fc
Version Independent ID: 8b02b646-01ba-5350-049f-1a31ec4780d4
Content: Immediate Window - Visual Studio
Content Source: docs/ide/reference/immediate-window.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @gewarren
Microsoft Alias: gewarren

Closing this now, please comment below if there are any issues.


Increase worker count

Great to see that the portable_queue was dropped in favour of limiting the queue size in the constructor!
I left a comment about potentially adding some logging to the blocking put call.


dotnetstatus-prod web app broken after 01/10 rollout

The dotneteng-status website broke after 01/10 rollout.
The changes that were in as part of the rollout doesn't seems to be impacting any related with the web app: [Rollout] Production rollout, 2024-01-10 by v-zheyan · Pull Request #1776 · dotnet/dnceng (github.com)
In order to get the site back online I did a swap with a the staging version (the old prod version) and that did the trick but after I put back the newest changes this broke again so I returned it to the oldest code, all this seem to be poiting out that the new code actually has something that is broken
I looked into the changing configuration and what make the most noise for me was that the working version has a managedServiceIdentityId value and the broken one has a null value in there.
we need to investigate more about this

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description- [ ]

The current active version is the one that belongs to https://dev.azure.com/dnceng/internal/_build/results?buildId=2344264&view=results which is just one roll out before the broken version.
The broken version is not on staging site (that is broken) and is: https://dev.azure.com/dnceng/internal/_build/results?buildId=2349473&view=results

I have been looking for a while and the only way to actually look into logs is by elevation, if not the information is very limited, I wasn't able to find anything without elevation.
The only difference significant difference I could found was the task version for the deployment, but also there were not additional logs or info that pointed that this could be the problem
I will try to look into this a little bit more with elevation, but also I don't have a lot more time on Ops left.

I will monitor this until next rollout, then decide if we can close this, like a one time thig, or add information if this fails again and put this back to the backlog

after rollout driven by @garath, looks like this didn't happen again! closing and celebrating 🥳


Update version: ForgQi.biliup-app version 0.4.1

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

Hi @vedantmgoyal2009,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries also be added to the previous versions?
(Automated response - build 582.)


amitesh- fix flexbox height

hey
I have changed style of gridStyle

Added media query.
tried to fix flexbox issue.
changed the title "React App" to "Algo Visualizer App".
corrected the readme.md documentation.
and solved some other relevant issues codes...

Thank u Hope it's solved the issue

Hey thanks for your contribution. Your Pr doesn't fix the flexbox overlay but I'll go on & merge it regardless based on your code cleanup. I have added the fix for the flexbox in my recent commit. Thanks


flutter_inappwebview the scroll position is changed when rotate the smartphone

[x] I have read the Get Started - Installation section
[x] I have read and done the Get Started - Setup Android section
[x] I have read and done the Get Started - Setup iOS section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version

Plugin version
5.3.2

Android version

iOS version

Xcode version

Device information:
Description
Expected behavior:
Current behavior:
Steps to reproduce

open webview
scroll to any position
rotate smartphone horizontally

Images
Stacktrace/Logcat

👋 @hung140120
NOTE: This comment is auto-generated.
Are you sure you have already searched for the same problem?
Some people open new issues but they didn't search for something similar or for the same issue. Please, search for it using the GitHub issue search box or on the official inappwebview.dev website, or, also, using Google, StackOverflow, etc. before posting a new one. You may already find an answer to your problem!
If this is really a new issue, then thank you for raising it. I will investigate it and get back to you as soon as possible. Please, make sure you have given me as much context as possible! Also, if you didn't already, post a code example that can replicate this issue.
In the meantime, you can already search for some possible solutions online! Because this plugin uses native WebView, you can search online for the same issue adding android WebView [MY ERROR HERE] or ios WKWebView [MY ERROR HERE] keywords.
Following these steps can save you, me, and other people a lot of time, thanks!

I searched and didn't find such issue


Explore GNAP-based OCM in ocm-stub

I'll add https://github.com/michielbdejong/ocm-stub back to the tests matrix
and then implement a GNAP-based version of the invite, share-with and public-link flows.

This evolved into #88


Show the week number on either side of the calendar

Before Submitting: Double-check that you are running the latest version of the plugin. The feature might have already been included 😄

Is your feature request related to a problem? Please describe.
I'd like to be able to move the week number column to the right side of the calendar. I use weekly notes to summarize all of my daily notes at the end of the week, and it would be beneficial to visually place this recap at the end of the week.
Describe the solution you'd like
It could be just a simple toggle in settings for left or right side or perhaps a dropdown with 'none', 'left', and 'right' (though I don't see any reason to change the way the current on/off setting works if not necessary).
Describe alternatives you've considered
As an additional (or alternative) option, hovering the mouse over the week number might highlight the week's full row in the calendar grid instead of just the week number itself.
Additional context
I'd be glad to help out getting a PR together but it seems that I don't have access to the repository where the base Calendar component is implemented.

Found the repository, disregard that last bit. Any objections to a feature as described? If not I'm going to start working on this tweak and will check back in with updates.

Implemented in https://github.com/liamcain/obsidian-calendar-plugin/pull/266 along with https://github.com/liamcain/obsidian-calendar-ui/pull/17


Speed up execution with Picamera2

Hey,
I've seen that the library PiCamera2 is using the threading module, could it be possible, if you use for example this module in your python script (imager taker or video recorder) to get additional execution speed or does the library already offer the maximum of speed?
It's just a question of interest, thanks.

Thanks for the question. You'd probably have to be a bit more specific about what you want to achieve. Picamera2 will give you frames pretty much as fast as the camera will deliver them, obviously it's up to the application how it then processes them. Some operations may benefit from threading within numpy or other libraries, or maybe an application could implement multi-threaded algorithms for itself.
The h.264 and hardware MJPEG encoders use built-in hardware, so there would be no benefit from threading there. The software JpegEncoder, when used for MJPEG, uses 4 cores already, so it's not clear to me that there are big and easy improvements available for video recording.

By default many operations actually run in the camera thread (cam.capture_file() for example), though the calling thread will normally want to block for it to finish. You can choose not to block for it to finish (wait=False), but at some point you will need to block to be sue the file has been written. Maybe this is what you want to do?
Delegating these operations to the camera thread is often convenient but sometimes users prefer not to make the camera thread do extra work. In this case they prefer to use their own thread to do the work. The easiest solution here is often to capture the request and then call save in your own thread.
Obviously you're also free to create another thread or task which will perform this operation outside your own thread. At this point, I think it's probably up to the application to implement the behaviour that it needs.

Hi, is there anything else to investigate here or should we close this issue? Thanks.


MTP-II versus MTP-2

My device (GOOJPRT) announces itself as "MTP-II" instead of "MTP-2" and is thus not picked up by the macOS application.
Is there anything I missed to select this device?

I can add that name as another option for that type of printer. Can you modify the code and confirm that it works after just changing the name? Each printer seems to have unique BLE UUIDs in addition to unique names, so hopefully just the name is different from the MTP-2.

I wish I could compile this. Unfortunately this is not possible without enrolling into the 🍎 developer program, which I can't ...

fixed


WARNING: opening from cache https://dl-cdn.alpinelinux.org/alpine/v3.18/main: No such file or directory

When opening in devcontainer

Network issues on laptop, unrelated to devcontainer conf.


Catch up

Summary by CodeRabbit

New Features

Updated CI workflow configurations, improving caching mechanisms and simplifying action versioning.

Bug Fixes

Enhanced consistency in commands related to Python package installations and updated linting processes.

Documentation

Updated changelog to reflect ongoing updates in the CI process and necessary changes for compatibility with Python 3.12.

Refactor

Improved readability and consistency of docstrings across multiple files.

Chores

Altered installation commands to include a new package requirement, uv, in setup scripts.

@coderabbitai please review again in full, that is, including updating all outdated conversations, they seem legit enough though can't be committed in batch due to the reference being outdated. Thanks in advance

OK, most of the nitpicks by CodeRabbit are ruff anyway, so basically updating each and every doctstring properly. (Which is also why I left ruff out for now, will add an issue to ensure someone picks this up).
As for the red crosses, nothing to see there:

[x] Black-ing changed some lines, hence the test-coverage dropping (actually there is still no testing
[x] CodeFactor issues ignored (they are interpreting the GitHub yamls wrong

If this is accepted/merged please tick the LOWEST checkbox on https://github.com/plugwise/python-plugwise-usb/issues/3 (Check this box to trigger a request for Renovate to run again on this repository) to see if all other PRs flow through now.

@CodeRabbitAI please approve, your ruff remarks are noted through #171 the changelog updates are placeholders anyway. Your timezone suggestions are noted in #173 for followup

Also replaced prettier with biome (see also https://github.com/plugwise/plugwise-beta/pull/695) and missing changelog entry as well as other pre-commit updates. This should supersede #157 and close it as incomplete.


Make list of appTypes include workflowapp for Single Tenant Logic Apps

Make list of appTypes include workflowapp for Single Tenant Logic Apps

As described in the following article: https://docs.microsoft.com/en-us/azure/logic-apps/set-up-devops-deployment-single-tenant-azure-logic-apps?tabs=azure-devops
- task: AzureFunctionApp@1
  displayName: 'Deploy logic app workflows'
  inputs:
     azureSubscription: 'MyServiceConnection'
     appType: 'workflowapp'
     appName: 'MyLogicAppName'
     package: 'MyBuildArtifact.zip'
     deploymentMethod: 'zipDeploy'
     ```

@azooinmyluggage @mijacobs

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#assign: @mijacobs

#sign-off

#unassign-reviewer:azooinmyluggage

#sign-off

@Stijn-Rutten Thank you for your contribution. We just published a new task reference that went live about the same time as your PR. The new task reference is built from the task open source repo here, and AzureFunctionApp@1 (and it looks like a v2 might be coming soon) don't list workflowapp as one of the choices and so it doesn't display in the task assistant for YAML pipelines or the classic editor for classic pipelines. Here is the new page for this task - I can add a note to that input parameter that is equivalent to your PR here.
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/azure-function-app-v1?view=azure-pipelines#inputs


[web] fix print warning

Codecov Report

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

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   89.74%   89.74%
=======================================
  Files          19       19
  Lines         897      897
=======================================
  Hits          805      805
  Misses         92       92

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 4d8dd41...6429f54. Read the comment docs.


[Bug] [Giveaway] The server responded with error 500: 500: Internal Server Error

Describe the bug
Bot errors out and disconnects upon 500 error
To Reproduce
long running tasks
Expected behavior
shouldn't get a server error, should run in the background.
Screenshots
N/A
Additional context
N/A

This seems to be a discord server side issue and not actually a problem with the code. Still looking for confirmation.

seems to have been a discord issue as far as I can tell. unable to replicate.


Run acceptance tests on Travis w/headless browser

This commit adds the running of browser tests to our TravisCI integration, using the headless mode of Google Chrome. Currently this requires a custom installation of google-chrome-beta (version 59)
although this functionality should be coming to the stable release soon.
The various Protractor-related files have been modified to look for a GOOGLE_CHROME_BINARY environment variable; if this is set, running the browser tests (using tox -e acceptance or gulp test:acceptance) will try to invoke that path in headless mode.
Note: this is a suboptimal way to configure that headless mode is desired, but I've punted on a better way pending some ongoing discussions with @sebworks about a larger refactor of Protractor config.
To test locally on MacOS, download the beta version of Google Chrome:
https://www.google.com/chrome/browser/beta.html
Then set the environment variable like:
GOOGLE_CHROME_BINARY=/path/to/Google\ Chrome\ Beta\ App.app/Contents/MacOS/Google\ Chrome

Additions

New acceptance testing matrix entry for Travis, and support for running Protractor with a headless browser.

Removals

Removed gulp test:unit:server to standardize on use of tox for running Python unit tests.

Testing

Look at the Travis logs! Follow steps above to reproduce with a local headless browser.

Notes

Unfortunately there seems to be an issue with calling setSize to set the browser size when using the headless mode with chromeDriver; I think this might be because of the need for Xvfb which is due to this open issue. See also this open issue to Travis.

Checklist

[X] PR has an informative and human-readable title
[X] Changes are limited to a single goal (no scope creep)
[X] Code can be automatically merged (no conflicts)
[X] Passes all existing automated tests
[X] Reviewers requested with the Assignee tool :arrow_right:

Unfortunately there seems to be an issue with calling setSize to set the browser size when using the headless mode with chromeDriver

You might be able to use the command line flag --window-size, like --window-size=1200,3000`.  A list of command line flags is here: http://peter.sh/experiments/chromium-command-line-switches/

lol lol, awww, this is too damn good.


Add Practice Exercise: parallel-letter-frequency

Problem specification:
https://github.com/exercism/problem-specifications/tree/main/exercises/parallel-letter-frequency
TODO:

generate the exercise with ./bin/fetch-configlet && ./bin/configlet create --practice-exercise parallel-letter-frequency
check if any of the existing tracks (examples below) implement the exercise and you can reuse the implementation:

Go: https://github.com/exercism/go/tree/main/exercises/practice
Rust: https://github.com/exercism/rust/tree/main/exercises/practice
Python: https://github.com/exercism/python/tree/main/exercises/practice

use exercise's canonical-data.json to see which test cases to implement:
https://github.com/exercism/problem-specifications/blob/main/exercises/parallel-letter-frequency/canonical-data.json

https://github.com/exercism/cairo/pull/254#issuecomment-2334748046


More information needed for Remote JMX

I want to enable remote JMX for profiling. But after I followed the instruction in the README, the server can not start.
part of the log:
[init] Starting the Minecraft server...
OpenJDK 64-Bit Server VM warning: CMSFullGCsBeforeCompaction is deprecated and will likely be removed in a future release.
Error: JMX connector server communication error: service:jmx:rmi://{{My server domain}}:7091
sun.management.AgentConfigurationError: java.rmi.server.ExportException: Port already in use: 7091; nested exception is:
	java.net.BindException: Address not available (Bind failed)
	at sun.management.jmxremote.ConnectorBootstrap.exportMBeanServer(ConnectorBootstrap.java:800)
	at sun.management.jmxremote.ConnectorBootstrap.startRemoteConnectorServer(ConnectorBootstrap.java:468)
	at sun.management.Agent.startAgent(Agent.java:262)
	at sun.management.Agent.startAgent(Agent.java:452)
Caused by: java.rmi.server.ExportException: Port already in use: 7091; nested exception is:
	java.net.BindException: Address not available (Bind failed)
	at sun.rmi.transport.tcp.TCPTransport.listen(TCPTransport.java:346)
	at sun.rmi.transport.tcp.TCPTransport.exportObject(TCPTransport.java:254)
	at sun.rmi.transport.tcp.TCPEndpoint.exportObject(TCPEndpoint.java:411)
	at sun.rmi.transport.LiveRef.exportObject(LiveRef.java:147)
	at sun.rmi.server.UnicastServerRef.exportObject(UnicastServerRef.java:237)
	at sun.management.jmxremote.ConnectorBootstrap$PermanentExporter.exportObject(ConnectorBootstrap.java:199)
	at javax.management.remote.rmi.RMIJRMPServerImpl.export(RMIJRMPServerImpl.java:146)
	at javax.management.remote.rmi.RMIJRMPServerImpl.export(RMIJRMPServerImpl.java:122)
	at javax.management.remote.rmi.RMIConnectorServer.start(RMIConnectorServer.java:404)
	at sun.management.jmxremote.ConnectorBootstrap.exportMBeanServer(ConnectorBootstrap.java:796)
	... 3 more
Caused by: java.net.BindException: Address not available (Bind failed)
	at java.net.PlainSocketImpl.socketBind(Native Method)
	at java.net.AbstractPlainSocketImpl.bind(AbstractPlainSocketImpl.java:387)
	at java.net.ServerSocket.bind(ServerSocket.java:375)
	at java.net.ServerSocket.<init>(ServerSocket.java:237)
	at sun.management.jmxremote.ConnectorBootstrap$HostAwareSocketFactory.createServerSocket(ConnectorBootstrap.java:864)
	at sun.rmi.transport.tcp.TCPEndpoint.newServerSocket(TCPEndpoint.java:666)
	at sun.rmi.transport.tcp.TCPTransport.listen(TCPTransport.java:335)
	... 12 more

I have already check the host and did not find any process that is using the port 7091. Besides, I turned off the firewall and tested it again, but the errors are the same.

part of my docker-compose file
    container_name: "MCServer"
    ports:
      - "25565:25565"
      - "7091:7091"
    environment:
      # ......
      # jmx
      ENABLE_JMX: "true"
      JMX_HOST: {{my server domain}}
    restart: unless-stopped

JMX is such a pain to get working. It took me many combinations to finally get that one example working.
In your case it's acting like another Java process within the container is also binding 7091, but that seems odd.

Have the same issue, not a fix but you can get it working by manually setting the options:
JVM_DD_OPTS: "com.sun.management.jmxremote.ssl=false com.sun.management.jmxremote.authenticate=false com.sun.management.jmxremote.port=1098 com.sun.management.jmxremote.rmi.port=1098 java.rmi.server.hostname=HOSTNAME_HERE com.sun.management.jmxremote.local.only=false "

Hmm, the ENABLE_JMX option does the same thing here:
https://github.com/itzg/docker-minecraft-server/blob/861aec6bd616cc0b3a84ca8c1cb6e2d10fa9f610/start-minecraftFinalSetup#L95
A port conflict sounds like a different issue though.

oh @jhlwscom I see you set yours to port 1098, can do the same with JMX_PORT looking at the broader code block
https://github.com/itzg/docker-minecraft-server/blob/861aec6bd616cc0b3a84ca8c1cb6e2d10fa9f610/start-minecraftFinalSetup#L87-L95


Minor: improve comments in temporal.rs tests

Which issue does this PR close?
Related to https://github.com/apache/arrow-rs/issues/6113
Rationale for this change
I saw a few comments that could be improved as part of reviewing https://github.com/apache/arrow-rs/pull/6071 from @nrc (🙏 ) so I made another PR
What changes are included in this PR?
improve comments in temporarl tests
Are there any user-facing changes?
No, just comments

Yep, that's better! Sorry I didn't fix before landing the earlier PR

No worries ! We are all just doing the best we can with what limited time we have. I appreciate all the help so far


Add example for frameworks

Introduction
It is not always clear for everyone that they shouldn't just output (echo + headers) the data in frameworks. This breaks the middleware/filters etc.
Proposal
Add an example to the docs. We could be more specific, eg. target PSR-7 and/or Symfony, or add a Streamed Response.
Alternative could also be to provide a method the generate a PSR-7 response in the code directly.

following your remark.. I've added a new method AbstractCsv::chunk to v9 to hopefully ease framework integration when creating a Response object.
V9 is in review period if you have other remarks now would be the perfect time :+1:


RSS feed on packages website has links with missing root slash

I've subscribed to the RSS feed for new packages on the https://nimble.directory/ but noticed that when clicking on a link, I get sent to e.g. https://nimble.directory/https:/nimble.directory/pkg/nimbug.
The reason appears to be the link urls are missing an extra slash at the root.
<link>https:/nimble.directory/pkg/nimbug</link>.

cc @FedericoCeratto

See
Duplicate of https://github.com/FedericoCeratto/nim-package-directory/issues/27


Documentation: FAQ.md and Repository Readme

How to categorize this PR?
/kind enhancement
/area documentation
/os garden-linux
What this PR does / why we need it:

FAQ created
Repository README.md

Let's see if this works:
/squash


PEP8 and Python library stuff

Hi,
Thank you for your great Python code. I like it a lot and worked on the code. This is a little bit a huge PR.
What I changed:

I did a PEP8 linting
I moved the files to a subdirectory and added a python setuptools and python requirements installation file

I don't know if you're interested in this, but if you are, here you have a PR. Feel free to approve or reject it.

hey keachi, thanks for your PR! i'm definetly interested into it and going to merge it. i'd just change the setup.py author things if you are fine with it.

Oh, sorry. Yes sure change it! I think I just copied from another project and forgot to change it. I absolutely have no problem with it, when you write down your name!


Incorrect width of full screen modal form fields

register endpoint + create-route  + deploy app
Shows smaller form fields (does the form have stepper-form class?)

This is still an issue in #2039

Could not reproduce


Run npm audit fix

Hey, I was just using this repo to build a small thing, figured I may as well update it.

@ikesau Sorry for delay.  Pull requests much appreciated.  Can you resolve the conflict if you have time? (looks like githubs auto vulnerability PR changed the package-lock file)

@Gaweph done!


Running docker on drives not mounted to /home

By default, docker images on SoS gets automatically maped (on a linux machine) via -v /home:/home -v /tmp:/tmp which works for files under the current directory. Currently I have some data located under  /median/$USER that I try to use with docker images but got a complain that the paths does not exist. Can this issue be confirmed on your end? How can we map the path better in that case? I tried symbolic link, did not work. The only way to ensure it works is to have everything under /home

There is an option volumes, which is mentioned in the reference.

Thanks for the pointer! I did not read those in detail because there is always extra_args one can fall back on ... I'm more thinking about trying to determine if some input files are not under /home, and thus automatically create volume mapping for the proper paths. This will make the workflow more portable (eg. when an extern drive is mounted on /mnt on one computer and /media on another). Is it straightforward to implement?
BTW there is a broken link on that reference: " (if properly configured under mac) "

I am not sure how to do that properly because directory names would be different inside and outside of docker, especially with externally mount directories. The script might also refer to path outside of docker, which can cause problems as well.
This is certainly doable but overly clever can bite us in some situations.

Okey, I can understand it. I'm cool to close the ticket if you deem it proper.


Provide helpful stack trace in WritableResourceStream

Personally I would pass a closure or null to the suspension to resume, e.g. static fn () => throw new ClosedException("The stream was closed by the peer");. It adds another function call to the stack trace, but the code seems less strange and Psalm should be more accepting of it.


Update docs for DefaultAzureCredential regarding AZURE_CLIENT_ID / managedIdentityClientId relationship

Add documentation to DefaultAzureCredential indicating that setting the AZURE_CLIENT_ID environment variable is equivalent to setting the managedIdentityClientId.

@christothes Can you provide some context here? What documentation are we referring to? What confusion lead us to make this clarification?

@christothes Can you provide some context here? What documentation are we referring to? What confusion lead us to make this clarification?

Sure - This issue was the catalyst for the issue (created for each language)
https://github.com/Azure/azure-sdk-for-net/issues/18253
Relevant documentation is here: https://github.com/Azure/azure-sdk-for-js/blob/726292875e5bd227e88958f54c72db1d86cea275/sdk/identity/identity/README.md#specifying-a-user-assigned-managed-identity-with-the-defaultazurecredential

Sure - This issue was the catalyst for the issue (created for each language)
https://github.com/Azure/azure-sdk-for-net/issues/18253
Relevant documentation is here: https://github.com/Azure/azure-sdk-for-js/blob/726292875e5bd227e88958f54c72db1d86cea275/sdk/identity/identity/README.md#specifying-a-user-assigned-managed-identity-with-the-defaultazurecredential

I think I might be able to do this this month. Worst case scenario I move it to next month.

I think I might be able to do this this month. Worst case scenario I move it to next month.


Adding option to return token when creating a user

Solves #77

Coverage remained the same at 100.0% when pulling ccce6d36d238c9db626191fa245de04714a4ec0a on DanMMX:autosignup into 5dac01630371932337ce383727a2490d11e7a1a8 on diegohaz:master.

Coverage remained the same at 100.0% when pulling ccce6d36d238c9db626191fa245de04714a4ec0a on DanMMX:autosignup into 5dac01630371932337ce383727a2490d11e7a1a8 on diegohaz:master.

Coverage remained the same at 100.0% when pulling ccce6d36d238c9db626191fa245de04714a4ec0a on DanMMX:autosignup into 5dac01630371932337ce383727a2490d11e7a1a8 on diegohaz:master.


Where is the official .proto file?

RTB protocol usually has its own official .proto file, For example google: https://developers.google.com/ad-exchange/rtb/downloads/realtime-bidding-proto.txt
There is one file attached to google code project(now unsupported) (https://code.google.com/p/openrtb/issues/attachmentText?id=14&aid=140008000&name=OpenRtb.proto&token=873ef9e3c79babbd442ff40906c26cd3) which is not official (a part of the issue).
So, where is OpenRTB official .proto file?

@aeryaguzov and others there is in fact an ongoing effort to put together standardized protobuf schemas for AdCOM/OpenRTB. This is in progress at https://github.com/InteractiveAdvertisingBureau/openrtb-proto-v2 for OpenRTB 2.X.


IntelliJ GroovyDSL extension for migration DSL

I have written a GDSL for intellij idea which would provide the code completion / suggestions for database migration changelog files (groovy) Read this [article]
(https://confluence.jetbrains.com/display/GRVY/Scripting+IDE+for+DSL+awareness) for more details
Other frameworks like spock and gparse also uses this to enhance code completion/suggestions for intellij idea

It can be further enhanced by the community.

This is fantastic! Can you target the 2.x branch?

sure.

Check #117 @jameskleeh

@jameskleeh
If you guyz wokr on this script further and make the changes to gdsl script while gdsl is already loaded by intellij, then you may need to delete the file ~/Library/Preferences//options/dslActivationStatus.xml to have ur changes picked up. Users will never need to do this, because they will never modify the original gdsl which is in classpath

Thank you very much for this contribution!


Generating sample table when updating database with MAGs - GTDB taxID and Accession Number

Hi there,
I am trying to update the GTDB -r207 database I have downloaded using Struo2 with my own MAGs. It is not clear how I get some of the information including "ncbi_organism_name", "gtdb_taxid" and "accession".
I have annotated my MAGs using GTDB-Tk. Using the FastANI I have de-replicated my genomes removing those with 95% ANI. This has left me with a ~ 3000 MAGs. Given that these MAGs are not close to any other genome in GTDB I don't understand how I can get a taxid? I have attached the current information I have from GTDB about my MAGs.
GTDB_MAG_Information.txt
Your help is greatly appreciated.
Kind regards,
P

You should get the GTDB taxids via https://github.com/shenwei356/gtdb-taxdump
I used that taxdump for setting taxids in GTDB-r207

Hi Nick,
Thanks for your response. A few things are still not clear to me. I have the GTDB taxids for r207 as obtained through the link above. But its not clear how I generated taxids for my own MAGs? I have used the below command which I found here
gtdb_to_taxdump.py
TaxID/gtdbtk.bac120.summary.tsv
https://data.gtdb.ecogenomic.org/releases/release207/207.0/bac120_taxonomy_r207.tsv.gz \

TaxID/taxID_info.tsv

This shows a taxid in the output file taxID_info.tsv.
How do I get the ncbi_organism_name and accession required for the databse update? I have confusion because most of my MAGs cannot be assigned a taxonomy beyond the genus level.
Many thanks
P

You could go from NCBI taxids for each of your MAGs to GTDB taxids, via gtdb_to_taxdump.py.
Another approach is getting the GTDB taxids directly from the GTDB taxdump created by https://github.com/shenwei356/gtdb-taxdump. You would probably need to create your own script for this, however.

Hi @nick-youngblut, I am experiencing a similar issue. I have a GTDB-Tk output file with GTDB taxonomies, but I don't have any TaxIDs. How do I go about this step of map taxonomy to gtdb-taxdump?
Thank you for any assistance you can provide.

This is what I ended up doing:
gtdb_lineages = df["gtdb_classification"].str.split(";", expand=True)  # create lineage dataframe based on gtdb_classification column

def get_sci_name_from_row(row):
    """This reads a Pandas Series (a row) and returns the lowest level scientific name."""
    trim = lambda s: s[3:]
    for value in row.to_list()[::-1]:
        if (value_fmt := trim(value)):
            return value_fmt

    return None

# Export to a text file
gtdb_lineages.apply(get_sci_name_from_row, axis=1).to_csv("scinames")

Now I run that with TaxonKIT (my --data-dir on TaxonKit is set for the GTDB r207 taxdump):
cut -f 2 -d , data/formatted/UGCMP_GTDB_scinames.csv | taxonkit name2taxid > taxids.csv

This gives me a text file with the TaxIDs from the custom GTDB taxdump. I hope it helps.
Best,
Vini


Prepare for 0.5. New Strings, Symbol instead of symbol...

...new getfield syntax, no longer compare with Chars, use BitArray and Array instead
of bitpack and bitunpack.

I think this one is ready now. The remaining warnings are from other packages.


simulation mesh.py regression test crashes on Windows

Describe the bug
The simulation example test mesh.py crashes, and Windows pops up this dialog:

Desktop

OS and version: Windows 10
VisIt Version: develop (3.2)

It also creates a memory dump file in %LOCALAPPDATA%/CrashDumps

I compiled against debug version of VTK libraries, and ran mesh sim through VS debugger, plotting the 3D mesh.
Crash occurs during serialization of the dataset (avtDataRepresentation::vtkToString) during write, down in this vtk method,
the SafeDownCast call returns a null pointer.
template <class T>
T* GetArrayRawPointer(vtkAbstractArray* array, T* ptr, int isAOSArray)
{
  if (isAOSArray)
  {
    return ptr;
  }
  T* data = new T[array->GetNumberOfComponents()*array->GetNumberOfTuples()];
  vtkSOADataArrayTemplate<T>* typedArray = vtkSOADataArrayTemplate<T>::SafeDownCast(array);

// ***** typedArray is NULL !!!! ****////
  typedArray->ExportToVoidPointer(data);
  return data;
}

The array being passed in is vtkComponentDataArray used/defined in SimV2 database plugin.

This isn't a Windows only issue. If you take a look at the regression results, tests 3 and 4 are blank, but they shouldn't be.
Other simulation tests were added to the skiplist after the VTK-8 port due to crashes (var, scalar, unstructured). They may be suffering from the same problem.

There are comments in simV2_GetMesh about problem with vtkDataWriter, such that a forced-copy (instead of zero-copy) of data arrays occurs.  However this is currently only happening for ndims == 2.  Allowing force-copy for all values of ndims still causes a crash, so I will need to investigate this further.
There is also logic in the file for creating vtkSOADataArrayTemplate instantiations when the underlying data coming from the sim is compatible.  This path isn't being used in all cases where I think it can be.  I will expand the use of vtkSOADataArrayTemplate.

Related to #3113

Have a fix, it's a patch for vtkDataWriter, basically incorporating some changes to the file from vtk-master.
I will modify bv_vtk to apply the patch, then do testing on linux and Windows.
@brugger, do you think this should go in 3.2?

@biagas Yes, it seems like it should go in 3.2 since it's fairly significant for libsim.


(poc) use eslint + cherow-ts

dogfooding our cherow-ts. 😊

Coverage remained the same at 89.736% when pulling f79aa9d0d15b72e434ad0ec6c3c72ced8c56a68e on Aladdin-ADD:eslint-ts into fb76275f55e7a25f23f3c8671245db0670f2ed21 on cherow:master.

@Aladdin-ADD This PR have been inactive for the last 15 days. Close it?


Ubuntu install instructions

Are there some ubuntu install instructions somewhere? am I missing them? the brew method only works on MacOS, not on ubuntu with linuxbrew.

I'm here also looking for Ubuntu instruction to install.

I found this [graphiql-app.AppImage](https://github.com/skevy/graphiql-app/releases/download/v0.6.3/graphiql-app-0.6.3-x86_64.AppImage) file on the release page.
Download and install to your Ubuntu, I think it should work fine.

Here's my solution. Probably not the most elegant one, but it works.
Also, it works for Linux Distros in general, I'm using Arch.
First, create a shell script -- in my case graphiql-app.sh -- with the following contents:
#!/usr/bin/env bash

NPM_START="/your/path/to/npm start"
GRAPHIQL_PATH=/path/to/graphiql-app/folder

cd $GRAPHIQL_PATH && $NPM_START

Notice that you'll need to change the paths according to your system. To find out your npm path, you can type which npm
Then you  create a .desktop file inside ~/.local/share/applications/:
[Desktop Entry]
Comment=
Terminal=false
Name=GraphiQL
Exec=/path/to/your/script/created/above/graphiql-app.sh
Type=Application
Icon=/graphiql-app path/build/icon.ico
StartupWMClass=graphiql

This way you'll have both CLI shortcuts (assuming your graphiql-app.sh is somewhere inside your $PATH and a GUI shortcut.
GNOME3 Example:

Before the guide from @trinaldi will work (see above), remember to run npm install and npm run build - otherwise you will get errors :+1:

Anyone mind turning this into a section in the README please? Thank you.


fix: Helm should emit an error when it cannot find a directory in a chart tgz

I figured out why the charts I created don't install, the tarballs put theChart.yaml inside the root of the .tgz file, not in a child folder. e.g.
# this one works
$ tar tf alpine-0.1.0.tgz
alpine/Chart.yaml
alpine/values.yaml
alpine/templates/alpine-pod.yaml
alpine/README.md

# this one doesn't
$ tar tf chaos-monkey-2.2.247.tgz
templates/
templates/chaos-monkey-svc.yml
templates/chaos-monkey-configmap.yml
templates/chaos-monkey-deployment.yml
Chart.yaml
README

Which then leads to:
$ ./bin/helm install fabric8/chaos-monkey-2.2.247.tgz
Downloading chart from: http://central.maven.org/maven2/io/fabric8/devops/apps/chaos-Fetched fabric8/chaos-monkey-2.2.247.tgz to /workspace/go/src/k8s.io/helm/chaos-monkey-2.2.247.tgz
Error: chart metadata (Chart.yaml) missing

Which had me quite confused for a while as when I opened the tgz file i found theChart.yaml file ;). (Also on a mac double clicking on a .tgz file generates the same folder layout ;)
I figure other folks may make the same mistake as me :) At the very least we should make it really obvious that a chart.tgz must include a single child folder inside for the Chart.yaml.
Right now the code assumes there's always 1 level deep folder:
https://github.com/jstrachan/helm/blob/96de0e35aa2eda4869a72de31570b59a4aa36a07/pkg/chartutil/load.go#L87
I wondered about adding a check first for a file called "Charts.yaml" and if so generating a warning that the chart should be in "foo/Charts.yaml" but still parsing it; otherwise then stripping the first folder from the file names?
Happy to submit a PR - just wondered which way folks preferred - accepting both styles of tarball layout; warning if no folder present - or always rejecting? At the very least we could give a really descriptive message of why the tarball should be rejected

I just submitted a PR that accepts a tgz with the Chart.yaml inside Chart.yaml or foo/Chart.yaml generating a warning if charts of the former are used. I figured its easy to accept slightly-wrongly packaged chart.tgz but we can then encourage folks towards the correct packaging

Did you generate this accidentally? I feel like we should be super strict with formats to make sure tooling can be consistent.

Here's the "light" spec for the chart format: https://github.com/kubernetes/helm/blob/master/docs/charts.md#the-chart-file-structure
The expectation is that the chart is always in a directory, that the directory name is always the name of the chart (e.g. matches name: in Chart.yaml), and that the Chart.yaml file is always inside of that directory.
Sounds like there's a bug somewhere that is not enforcing the directory part correctly.

@philips I generated them accidentally; I didn't see any docs at the time on that there should be a folder in the tar or what it should be called
@technosophos thanks -  the docs are extremely clear about the layout of charts on your local file system - they just don't say much about the layout of the tzg file. With hindsight I guess its obvious that the chart folder should be included in the tgz - I suspect though others will make the same mistake I made - so at the very least we should warn that the tgz file is invalid if it doesn't contain the directory

/cc @kmala

Here's the "light" spec for the chart format: https://github.com/kubernetes/helm/blob/master/docs/charts.md#the-chart-file-structure
The expectation is that the chart is always in a directory, that the directory name is always the name of the chart (e.g. matches name: in Chart.yaml), and that the Chart.yaml file is always inside of that directory.
Sounds like there's a bug somewhere that is not enforcing the directory part correctly.

oh sadness broken link

It was no longer relevant.


Index test fixes

TestIndexWriterExceptions was failing because deletes were not reflected when the index was opened with the reader. The reason for that was that the documents were being sent with leading 0 in the id (e.g. "id:01" instead of "id:1") but deletes were executed against id without leading zero. Simple oversight of .NET port, Lucene adds the base and i first before converting to string.
TestParallelCompositeReader needed to be adjusted to match ported code that uses anonymous inner class helper so the type name is different.

Looks like parallel composite reader fix is addressed by @Chand2048 in https://github.com/apache/lucenenet/pull/102, I will close this and reopen only with TestIndexWriterExceptions fix.

@laimis what do you make of @Chand2048 's comment about this test not existing in the original Java code?

oh yeah, thanks for pointing that out @synhershko. I see the test in Lucene, it is this: https://github.com/apache/lucene-solr/blob/lucene_solr_4_8_0/lucene/core/src/test/org/apache/lucene/index/TestParallelCompositeReader.java#L392


WIP - [#494] Replace danger-ruby with danger-kotlin

https://github.com/nimblehq/android-templates/issues/494
What happened 👀

[ ] Replace danger-ruby with danger-kotlin on templates' GitHub Actions workflow
[ ] Replace danger-ruby with danger-kotlin on generated project's GitHub Actions workflow

Insight 📝
N/A
Proof Of Work 📹
Will update

Closing for now due to unable to migrate Shroud to danger-kotlin


NUMA Aware Work Stealing

When running with pinned threads, prioritise NUMA local work-stealing (significantly) over stealing from further away cores.

Check https://crates.io/crates/hwloc to get topology information

C&P from Slack for reference:

lkroll  14:52
Alright, I’ve been looking into the NUMA aware workstealing and it’s a bit tricky. The only way I found to get some sensible topology information is via rust-hwloc(2), which is a FFI wrapper around libhwloc. So far so good, but the people who wrote the wrapper clearly have no idea how Rust FFIs are supposed to work and their interfaces suck balls. In particular some of their methods throw SIGSEGV because they try to dereference NULL pointers within an unsafe block (which isn’t exposed…the function itself is written as if it were safe).
14:52
Now, we could try to fix this…but that would probably mean we’ll end up maintaining rust-hwloc in long term (at least that is my experience with these kind of semi-abandoned projects)
14:53
And when I say ‘we’, I obviously mean ‘you’, since I’ll be leaving after December :D
14:53
How would you like me to proceed on this?^^
New
Max  14:56
I see, well it isn't critical to have right now anyways. More of a wish. Thanks for checking, I might have a look into it again in the future :smile:


Fix parsing of string literals which contain zero character.

Fixes ch08/8.4/S8.4_A5.js from test262.

Good to me.

LGTM


InitVar and deepcopy break PyTreeDef equality

Hi,
In some optimization process, I want to compare some new parameter values to old ones that I stored using a deepcopy. I get an error in the jitting of the optimization function because of the tree structure of my parameters being modified. See below a MWE, where we lose the tree structure equality, which is the root of the trouble in my complete program.
from dataclasses import InitVar
from copy import deepcopy

import jax
import equinox as eqx
from jaxtyping import Key

class MLP2(eqx.Module):

    key: InitVar[Key] = eqx.field(kw_only=True)
    layers: list = eqx.field(init=False)

    def __post_init__(self, key):
        self.layers = [eqx.nn.Linear(1, 50, key=key), jax.nn.relu]

    def __call__(self, t):
        for layer in self.layers:
            t = layer(t)
        return t

key = jax.random.PRNGKey(0)
mlp = MLP2(key=key)
params, static = eqx.partition(mlp, eqx.is_inexact_array)
print(jax.tree.flatten(params)[1] == jax.tree.flatten(deepcopy(params))[1]) # return False!
print(jax.tree.flatten(params)[1], jax.tree.flatten(deepcopy(params))[1])

Note that I found out the bug disappears when not using InitVar (probably less elegant so):
class MLP1(eqx.Module):

    layers: list = eqx.field(init=False)

    def __init__(self, key):
        self.layers = [eqx.nn.Linear(1, 50, key=key), jax.nn.relu]

    def __call__(self, t):
        for layer in self.layers:
            t = layer(t)
        return t

key = jax.random.PRNGKey(0)
mlp = MLP1(key=key)
params, static = eqx.partition(mlp, eqx.is_inexact_array)
print(jax.tree.flatten(params)[1] == jax.tree.flatten(deepcopy(params))[1]) # returns True!
print(jax.tree.flatten(params)[1], jax.tree.flatten(deepcopy(params))[1])

Is the problem really due to InitVar? Should I use something else rather than deepcopy?
Thanks!

Why do you need a copy of the parameters? All jittable functions should be without side-effects, and jax Arrays are immutable, so you can just store the original array and compare the new array to the old one instead of to some deep copy.
If you really do need to copy arrays, you could use https://jax.readthedocs.io/en/latest/_autosummary/jax.numpy.copy.html I guess?
So then you could do something like:
from dataclasses import InitVar
import jax
from jax import numpy as jnp
import equinox as eqx
from jaxtyping import Key

class MLP2(eqx.Module):

    key: InitVar[Key] = eqx.field(kw_only=True)
    layers: list = eqx.field(init=False)

    def __post_init__(self, key):
        self.layers = [eqx.nn.Linear(1, 50, key=key), jax.nn.relu]

    def __call__(self, t):
        for layer in self.layers:
            t = layer(t)
        return t

key = jax.random.PRNGKey(0)
mlp = MLP2(key=key)
params, static = eqx.partition(mlp, eqx.is_inexact_array)

params_copy = jax.tree.map(lambda x: jnp.copy(x) if isinstance(x, jax.Array) else x, params)

print(jax.tree.flatten(params)[1] == jax.tree.flatten(params_copy)[1]) # return True!
print(jax.tree.flatten(params)[1], jax.tree.flatten(params_copy)[1])

Hmm. This is really weird! I've poked at this a little bit and you're right, it's specifically the interaction of InitVar[...] and deepcopy.
I have no idea why this should be the case.


Update Configured SIR.json

Description
Initial values need to be float.
Parameter values need to be much smaller 1/1000 of non-normalized initial values
Resolves #(issue)

Does not look like it. It fails if they are not.


Add a policy-rc.d file on Debian systems to prevent service start on install

This becomes a lot more important in 14.04 and prevents the insanity that comes when Nginx creates a bunch of files on startup that don't agree with our Chef-specified values.

This approach is in violation of the method for using this file.

The /usr/sbin/policy-rc.d file must be managed through the alternatives
system (/usr/sbin/update-alternatives) by any packages providing it.

From: https://people.debian.org/~hmh/invokerc.d-policyrc.d-specification.txt
So I don't think this solution is the right one for now.


build: local build fails

Using npm install fails the build because of a missing dependency for run-p. I've then installed https://www.npmjs.com/package/npm-run-all globally only to run into the next error.
I am using Windows 11, npm 8.11.0, node 16.16.0
C:\Daten\Soren\static-cms-hugo-netlify-template>npm install
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
npm ERR! code 1
npm ERR! path C:\Daten\Soren\static-cms-hugo-netlify-template\node_modules\hugo-bin
npm ERR! command failed
npm ERR! command C:\WINDOWS\system32\cmd.exe /d /s /c rimraf vendor && node lib/install.js
npm ERR! node:internal/process/esm_loader:94
npm ERR!     internalBinding('errors').triggerUncaughtException(
npm ERR!                               ^
npm ERR!
npm ERR! Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'C:\Daten\Sören\static-cms-hugo-netlify-template\node_modules\pkg-conf\node_modules\find-up\' imported from C:\Daten\Sören\static-cms-hugo-netlify-template\node_modules\pkg-conf\index.js
npm ERR! Did you mean to import find-up/index.js?
npm ERR!     at new NodeError (node:internal/errors:372:5)
npm ERR!     at legacyMainResolve (node:internal/modules/esm/resolve:341:9)
npm ERR!     at packageResolve (node:internal/modules/esm/resolve:941:14)
npm ERR!     at moduleResolve (node:internal/modules/esm/resolve:1003:20)
npm ERR!     at defaultResolve (node:internal/modules/esm/resolve:1218:11)
npm ERR!     at ESMLoader.resolve (node:internal/modules/esm/loader:580:30)
npm ERR!     at ESMLoader.getModuleJob (node:internal/modules/esm/loader:294:18)
npm ERR!     at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)
npm ERR!     at link (node:internal/modules/esm/module_job:78:36) {
npm ERR!   code: 'ERR_MODULE_NOT_FOUND'
npm ERR! }

npm ERR!      C:\Users\Soren\AppData\Local\npm-cache\_logs\2023-01-17T18_44_29_259Z-debug-0.log

npm-run-all is already in the dev dependencies.
Do you have a repository you are working out of that I can double check the setup of?

It's just a local clone of this repo. Just git clone and afterwards npm install. Nothing fancy otherwise.

It sounds like it may be some kind of environment setup. Do you have a repo setup for this yet that I can take a look at?

I've updated to the latest Node LTS (18.13.0) and now npm install works fine.
Nevertheless npm start produces errors with webpack: Support for the experimental syntax 'jsx' isn't currently enabled.
I've fixed this by editing webpack.common.js and filing a PR.


Where is the loss function?

Hi
I want to train YOLOPX.
But, I can't find the loss function and train.py
When upload your loss function and train.py files?
Thank you.

@smilecode-github did you train a model on your data? How did it go in terms on performance?


Domain Names vs IDs - Inconsistency?

MMT now allows multiple domains with the same name.
Each domain has a unique id.
This command looks ambiguous:
./mmt add ibm "hello Mike!" "ciao Mike!"

How do I make sure I am adding the data to the right domain?
I this we have 2 ways:
1 - Domain names are unique
2 - IDs are used
Number 1 is my favorite because it is simpler for the user and it is also aligned with the way files are named in the training folder.

Hi @marcotrombetti,
You're right, at the moment that command seems ambiguous, in fact if it detects that there are two domains with the same name, it fails.
Having the name unique is not applicable in a real production environment: we cannot say to the user that he cannot name it's TMX "europarl" just because another user has already used that name.
The solution is: that command should prompt all the options in case of ambiguity and the user will select the right one (all the metadata of the domains will be printed).
Please note that in the API such "inconsistency" is not present because the ID is always used.

What about adding a user id to the domain name?

On 14 Dec 2016, at 09:34, Davide Caroselli<EMAIL_ADDRESS>wrote:
Hi @marcotrombetti https://github.com/marcotrombetti,
You're right, at the moment that command seems ambiguous, in fact if it detects that there are two domains with the same name, it fails.
Having the name unique is not applicable in a real production environment: we cannot say to the user that he cannot name it's TMX "europarl" just because another user has already used that name.
The solution is: that command should prompt all the options in case of ambiguity and the user will select the right one (all the metadata of the domains will be printed).
Please note that in the API such "inconsistency" is not present because the ID is always used.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ModernMT/MMT/issues/189#issuecomment-266974429, or mute the thread https://github.com/notifications/unsubscribe-auth/ABFS-qYlN2oiDA2RoiaGEtHuIiNcxzpXks5rH6owgaJpZM4Kqq3W.

@mfederico this is what we will introduce with the plugin implementation! :)

Update on this topic.
We haven't introduced the concept of user yet because we wanted to keep the MMT logic very simple and let the products interfacing with it to handle all the Authentication and Authorization  management.
In case of name ambiguity mmt add is now printing a message alerting this situation and let the user choose between ids. For example:
$ ./mmt add europarl "hello Mike!" "ciao Mike!"
ERROR Illegal Argument: ambiguous domain name "europarl", choose one of the following ids: [1, 4]

$ ./mmt add 4 "hello Mike!" "ciao Mike!"
SUCCESS - contribution added to domain "4"

Currently Domain has only id and name fields; we will include in this command any other additional fields we will put in the future in order to better distinguish between two domains with the same name.


Try to setup RDS to use the SecretsManager

We are currently manually adding the username and password to the SSM parameter store for our RDS instance. Try to setup and use SecretsManager (with 90 day credential rotation) for the RDS instance.
This will require that I am given permission to create Secrets via SecretsManager

closing since we will be using an S3 - Cloudfront distribution instead of Rails


CT-63 Change question and jobTitle field types in users

This PR fixes the issues migrating production user data

Number of characters was not supported in questions (action https://github.com/yldio/asap-hub/actions/runs/5187380563/jobs/9374300439#step:6:6402):

Number of characters was not supported in jobTitle (action https://github.com/yldio/asap-hub/actions/runs/5187380563/jobs/9374300439#step:6:7083):

Proposed solution
In order to fix it, I've changed the field questions from Array of short text to JSON and jobTitle from short text to long text.
After the change in this PR I ran the migration with prod data and we can see that all user entries were created without errors (action https://github.com/yldio/asap-hub/actions/runs/5523494198/jobs/10074498567#step:6:6161):

It is still possible to change open questions via UI.
https://github.com/yldio/asap-hub/assets/16595804/9a3541e0-c090-42da-8d27-7c2a2b2548c7

rebasing to master before merge


[Bugfix] Offline mode fix

This PR fixes a bug that I believe was introduced by the mistral model format support in https://github.com/vllm-project/vllm/pull/8168
The transformers_utils/config.py file now uses an unsafe file_exists check that will immediately fail in offline mode. This takes down all of our deployments where we set HF_HUB_OFFLINE=1 to ensure that we don't pull model files from the hub.
The changes here first add a check to look for the model revision in local cache, so that no call out to the hub is needed when the model is already cached (⚡ ⚡ performance improvement). Then, these changes guard against the OfflineModeIsEnabled exception so that we can properly loop through the different model types without crashing early.
I've added a test case that currently fails on main, which tries to load a model in offline mode. Getting the test setup correct is a bit janky, as far as I can tell the only way to enable offline mode on the fly is to recursively reload both transformers and huggingface_hub, since some objects are initialized at import-time that determine whether or not they operate in offline mode.
FIX (I didn't open a bug 😞 )

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Ah dang, my attempt to re-import things after the offline test is failing 🤔

I think it might be easiest to make a completely new test entry that checks multiple things with HF_HUB_OFFLINE set. This would allow you to get past the tricky patching

Yeah, separating out offline mode tests to run by themselves sounds way better. I think there's still the issue of making sure that the cached files exist ahead of time though, I imagine that CI step would have to invoke a download script first and then invoke pytest with offline mode enabled 🤔. I was hoping to find a more portable solution so everybody can just pytest this on their machines...
I'll hack away for a bit

I understand your thoughts, appreciate it and requesting changes for now to signify it isn't ready

Alright, hopefully this works- I've separated the test out into it's own module under tests/entrypoints and added a separate pytest invocation to the test pipeline for it. I've still kept the import hackery in so that the test is nicely encapsulated, and also fixed the last problem I was seeing with the imports and made the failure a bit more robust. (That's probably moot because I doubt many other tests will be added here, but I didn't want to leave it in an unhealthy state)
🤞  it all passes

docker: failed to register layer: write /opt/conda/envs/py_3.9/lib/python3.9/site-packages/transformers/models/maskformer/modeling_maskformer.py: no space left on device.

uhhhhh.... :(
I think I'll need somebody with write access to retry those failures?

Looks like all the failures now are due to no space left on device, and I see builds on main failing with backoffs on docker pulls :(
I won't trigger a rebuild here- don't want to make these CI stability problems worse

Test failures are unrelated.


Fix rotate clockwise angle

What's Changing and Why
In the document it says that image rotate is clockwise and it should be!
But the fact is the rotate method is anti-clockwise, So this is fix for the bug & update tests for both clockwise & anti-clock rotate.
for e.g
// This is should rotate the image 90deg clockwise ( to the right ),
// But the fact its being rotated anti-clockwise ( to the left )
image.rotate(90)

The bug caused by the rotation equation, The one is being used is anti-clockwise !
https://en.wikipedia.org/wiki/Rotation_matrix#In_two_dimensions
 ( Counter-Clockwise )
 (Clockwise )
What else might be affected
Nothing but for sure everyone notice this error and made workaround like use negative rotate angle to get clockwise rotate
// Before fix - workaround to rotate image 90 to the right
image.rotate(-90)

So when this fix is applied it should be
// After fix to rotate image 90 to the right
image.rotate(90)

Tasks

[x] Add tests
[ ] Add SemVer Label ( I couldn't tell if this will be major or minor !, Because it will affect a lot of people who was doing the workaround !  )

📦 Published PR as canary version: 0.16.3-canary.1100.1371.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>

@oliver-moran can you check that ?

I think we should just update the docs since this would require a breaking change

Closing in favor https://github.com/jimp-dev/jimp/pull/1062. Thanks for trying though! If you want to help with more issue I'm gonna be more on top of issues


[restock detection] should not show "Not in stock" if its not 100% sure after 403 errors etc

0.43.x
So restock will say "not in stock" when it cant be 100% sure

In this case because the text '403 forbidden' is the only text it can look at
for this URL https://www.newegg.com/g-skill-32gb/p/N82E16820374430?item=N82E16820374430

Better in this case is maybe Can't detect stock status if none of the "instock" or "out of stock" texts match


Multi-Series Grouped and Stacked Bar Chart

Using the stacked bar graph example i was able to create Stacked-bar graph, but seeing the current documentation it seems that its not possible to create Multi-series stacked bar graph.
My current stacked bar graph is exactly similar to one in documentation.

Stacked bar charts like the image above were released in v2.5.0


Docs.MS Release Updates for azure-core-auth

Update docs metadata and targeting for release of azure-core-auth

Docs Build status updates of commit 3cc87ff:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/core-auth-readme.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/core-auth-readme.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 2, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Property 'useStaticFilesLoader' does not exist on type 'TranslateService'.

Hi,
I am getting this error in my terminal window.
initializeTranslateServiceConfig() {
    let prefix = './i18n/';
    let suffix = '.json';
    this.translate.useStaticFilesLoader(prefix, suffix);

I am using "ng2-translate": "^1.8.0", what could be wrong?

Hello, the api has changed in 1.10, see the release notes: https://github.com/ocombe/ng2-translate/releases/tag/v1.10.0

Thank you for the information. I am trying to include it like this:
import { TranslateService, TranslateLoader, TranslateStaticLoader } from 'ng2-translate/ng2-translate';

@App({
  templateUrl: 'build/app.html',
  providers: [
    HTTP_PROVIDERS,
    provide(AuthHttp, {
      useFactory: (http) => {
        let config = new AuthConfig;
        config.tokenName = 'accessToken';
        return new AuthHttp(config, http);
      }
    }),
    provide(TranslateLoader, {
      useFactory: (http: Http) => new TranslateStaticLoader(http, 'assets/i18n', '.json'),
      deps: [Http]
    }),
    TranslateService, JwtHelper, AuthService, NoticeService, ConversationsService, MessagesService]
})

But getting error Cannot resolve all parameters for 'function (http) { '(?). Make sure that all the parameters are decorated with Inject or have valid type annotations and that 'function (http) { ' is decorated with Injectable.
I have changed code only in app.ts (using Ionic2), what modifications should I do on pages? Thank you in advance.

I think the error comes from:
provide(AuthHttp, {
      useFactory: (http) => {
        let config = new AuthConfig;
        config.tokenName = 'accessToken';
        return new AuthHttp(config, http);
      }
    }),

that should be
provide(AuthHttp, {
      useFactory: (http) => {
        let config = new AuthConfig;
        config.tokenName = 'accessToken';
        return new AuthHttp(config, http);
      },
      deps: [Http]
    }),

Thank you.


Dockerfile - sort & add bluez-deprecated

Hmmm
Should have merged this into a v0.0.11 branch (to be created).
Not a big deal for this one though (bug probability ~0)