example data folder

I am attempting to manually run natrix - just wondering where I can access the example_data folder, i.e. the example_data.csv, example_data.yaml and example amplicon datatset.
Thanks!

Hello,
Thank you for using our workflow :). Please use the Illumina data and either the Illumina_swarm.yaml or Illumina_vsearch.yaml along with Illumina.csv. If you are working with nanopore, use files and folders with the prefix "nanopore". Additionally, please use the dev branch as it contains all the latest changes.
we update the readme as soon as possible.
feel free to ask for any queries.
best,
aman

Hi Aman,
I was also wondering if you have an example Nanopore.yaml for data that has yet not yet been demultiplexed? e.g. the yaml file corresponding to the Nanopore 18S V9.csv?
From my understanding of your paper, the natrix2 workflow for Nanopore data is Pychopper (demultiplexing) > CD-HIT (clustering) > Medaka + Racon (error-correction) > Minimap2 > Vsearch (chimera removal) > taxonomic classification (e.g. blast). However, interpreting the .yaml file is a bit confusing given a lot of the parameters listed (I think) are for Illumina processing. I am a bit confused, for example, where the pychopper parameters are in the .yaml file.
Thanks

Hi @katfiishn,
Unfortunately, our pipeline does not currently support demultiplexing for Nanopore data. I recommend employing Nanopore-specific demultiplexing tools, such as Guppy, before proceeding with our amplicon workflow. If you want to make use of Pychopper for reorientation of your sequences, it's essential to conduct demultiplexing without adapter trimming prior to integrating it with our pipeline. If you don't want to integrate Pychopper, please trim your sequences during demultiplexing.
In Natrix2, Pychopper serves the sole purpose of identifying forward and reverse primers, ensuring uniform orientation for all reads. This step ensures consistency in the orientation of the reads.
For an enhanced Nanopore experience, I suggest utilizing the "dev" branch of our pipeline. This version includes updates specifically tailored for Nanopore data, optimizing the workflow. Additionally, the configuration files have been updated to provide clearer and more intuitive Nanopore- and Illumina-specific options.
Feel free to reach out if you have any further questions or require additional clarification.
Best,
Dana

Please have a look at the Nanopore.yaml configuration file within the 'dev' branch for a more intricate understanding of parameters specific to Nanopore or Illumina. Please refer to the following link: Nanopore.yaml - dev branch

Hi Dana,
I ran the pipeline successfully on my Nanopore dataset using the dev version.
However, I noticed a critical error in the clustering output:
The vsearch_uc.txt file does not match the vsearch_clusters_names.txt and subsequent output into the vsearch_table.csv
From my investigating, the vsearch clustering (vsearch_uc.txt) is correct in how it has clustered the sequences (I specified 96% clustering), however I think there is something wrong with the write_clusters_uc script - as it has not clustered the correct sequences from the vsearch_uc.txt file.
I have attached both txt files. For example, in vsearch_uc.txt - on Line 5 vsearch indicates that 2879;size=8 and 5022;size=91 should be clustered together (at 98.6% similarity) - 5022;size=91 being the centroid. However, in the vsearch_clusters_names.txt file on Line 4: 2879;size=8 has been incorrectly outputted with 5;size=26. As such, their count data has been incorrectly merged in the successive vsearch_table.csv. I found this has occurred throughout the dataset and has incorrectly merged both the sequences and count data from different species, when they were never meant to be clustered together following the
vsearch_uc.txt
vsearch_clusters_names.txt
vsearch_uc.txt file.

Hi @katfiishn,
Thank you for your feedback! We always appreciate it when people bring issues to our attention.
I've made the necessary changes to the corresponding scripts (merge_clust_results.py and vsearch_uc.py) in the dev branch. Please replace these scripts accordingly. To avoid rerunning the entire pipeline, it's sufficient to delete the "finalData," "mothur," and "clustering" folders in the output directory. This way, only the clustering and subsequent steps will be repeated.
Best,
Dana

Hi Dana,
I re-ran it with your new scripts. The vsearch_clusters_names.txt seems to have created duplicates (see line 4 + 5 of attached file) and is still incorrectly clustering based on the results of the vsearch_uc.txt. The pipeline was not able to produce the vsearch_table.csv with the errors in the vsearch_clusters_names.txt.
vsearch_clusters_names.txt
vsearch_uc.txt
Thanks for your help!

Hi @katfiishn,
Thank you for your patience and your feedback! I've made some changes to the scripts. Now, the vsearch_all_otus_tab.txt file is utilized for creating both vsearch_clusters_names.txt and vsearch_table.csv, replacing the previous use of the vsearch_uc.txt file.
Here's a brief overview of the changes made to each script:
[vsearch_uc.py](https://github.com/dbeisser/Natrix2/blob/dev/scripts/vsearch_uc.py): Updated to correctly handle cluster assignments using the vsearch_all_otus_tab.txt file.

[merge_clust_results.py](https://github.com/dbeisser/Natrix2/blob/dev/scripts/merge_clust_results.py): Adjusted to incorporate changes from vsearch_uc.py and ensure compatibility with the updated workflow.

[vsearch_clust.smk](https://github.com/dbeisser/Natrix2/blob/dev/rules/vsearch_clust.smk): Updated input file from vsearch_uc.txt to vsearch_all_otus_tab.txt.

Please test these changes with your dataset and provide any feedback or report any issues you encounter. The changes have been tested with our test dataset, so I don't expect any issues with your data.
Looking forward to hearing from you!
Best regards,
Dana

Hi Dana,
I incoporated the the three scripts and ran them. The rule write_clusters_uc executed correctly and has clustered the correct sequences producing a vsearch_uc.txt, vsearch_clusters_names.txt, vsearch_all_otus.fasta and vsearch_all_otus_tab.txt files (see attached).
vsearch_uc.txt
vsearch_clusters_names.txt
vsearch_all_otus_tab.txt.zip
However, there was an error in the next job clust_merge_results_vsearch which then caused the job execution to fail:
rule clust_merge_results_vsearch:
input: Nanopore_elasmo_eDNA_results_2/clustering/vsearch_clusters_names.txt, Nanopore_elasmo_eDNA_results_2/filtering/filtered_table.csv
output: Nanopore_elasmo_eDNA_results_2/clustering/vsearch_table.csv
jobid: 3351
Activating conda environment: /scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/conda/6bb4e448
Activating conda environment: /scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/conda/6bb4e448
Traceback (most recent call last):
File "/scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/scripts/tmpkvntjie8.merge_clust_results.py", line 20, in
swarm_dict[j]["sequences"] = filtered_dict[">" + seq_names[j][0]]["sequences"]
KeyError: '>1000_size_14_;size=1;'
[Tue Mar 26 12:49:50 2024]
Error in rule clust_merge_results_vsearch:
jobid: 3351
output: Nanopore_elasmo_eDNA_results_2/clustering/vsearch_table.csv
conda-env: /scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/conda/6bb4e448
RuleException:
CalledProcessError in line 44 of /scratch3/wes187/Natrix2_dev/Natrix2/rules/vsearch_clust.smk:
Command 'source /opt/conda/bin/activate '/scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/conda/6bb4e448'; set -euo pipefail;  python /scratch3/wes187/Natrix2_dev/Natrix2/.snakemake/scripts/tmpkvntjie8.merge_clust_results.py' returned non-zero exit status 1.
File "/opt/conda/envs/natrix/lib/python3.7/site-packages/snakemake/executors/init.py", line 2347, in run_wrapper
File "/scratch3/wes187/Natrix2_dev/Natrix2/rules/vsearch_clust.smk", line 44, in __rule_clust_merge_results_vsearch
File "/opt/conda/envs/natrix/lib/python3.7/site-packages/snakemake/executors/init.py", line 568, in _callback
File "/opt/conda/envs/natrix/lib/python3.7/concurrent/futures/thread.py", line 57, in run
File "/opt/conda/envs/natrix/lib/python3.7/site-packages/snakemake/executors/init.py", line 554, in cached_or_run
File "/opt/conda/envs/natrix/lib/python3.7/site-packages/snakemake/executors/init.py", line 2359, in run_wrapper
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message

Hi Dana,
I think the problem is with my filtered_table.csv. Let me fix this and get back to you!
Cheers!

Hi Dana,
This seems to have worked! Thanks very much!
Just wondering if there is a fasta file that matches the final vsearch_table.csv?
I see there is the vsearch_all_otus.fasta to match the vsearch_all_otus_tab.txt, but that obviously contains all sequences, not the final consensus cluster sequences that is in vsearch_table.csv.
Would I have to enable the mothur component to get that final fasta file?
Thanks for all your help :)

Hi @katfiishn,
In our pipeline, we utilize the sequence of the representative (centroid) OTU for subsequent analysis rather than the consensus sequence of the OTU.
If you intend to work with the consensus sequences outside of the pipeline, you have the option to include the --consout flag in the vsearch cluster rule. While we don't have plans to utilize the consensus sequences in our pipeline, I've created an example version of the rule for your convenience.
vsearch_clust.txt
Please rename the file extension from .txt to .smk after downloading it, as .smk files cannot be attached directly here. Also ensure it still meets the requirements for a Snakemake file.
I hope I was able to help you! :)

@katfiishn Hi I'm just started with the use of Natrix and it seems you were able to successfully use your Nanopore data.
I have data pod5 data from a single barcode and I did the basecall with dorado:
$ dorado basecaller --no-trim --barcode-both-ends --kit-name SQK-16S024<EMAIL_ADDRESS>./barcode04/ | samtools fastq -T '*' - > ./reads/BC04_A_R1.fastq
Could you please guide me through on how to successfully perform a run of the natrix pipeline using those reads as input.
hope you can help, best regards, Jorn.

Hi Jorn,
As Aman and Dana advised me, you need to have demultiplexed, un-trimmed fastq files as the input. I can't really advise on the installation process as I ran it on a supercomputer and we had to make some changes. However once you have it installed, you place the folder containing you demultiplexed files under the Natrix2 folder and then run the Nanopore.yaml script (you will need to edit the parameters on the script) to execute the run. It took a good day to install the conda environment containing all the dependent software and the run itself (once I could fix problems that came up) took a good 1-2 days to run through with the data size I had. I think overall it took 2-3 weeks to get the finalised OTU table and fasta file I was after (thanks Aman and Dana!!).
Cheers!

Thanks, @katfiishn and @DanaBlu for the bits of advice and goodwill. I hope I can get it to run properly.
best, Jorn


Proctored MC1

GH Indi Page and Review Ticket for following...
Start review ticket with study plan and study reflection for week, due EOD Monday 4/18

[x]  Finals-Quiz1 score/50 and corrections
[x]  Finals-Quiz2 score/50 and corrections
[x]  Create Task code/runtime and 1 min video are reviewed by another person, you may request Teacher review as time permits
Review ticket with Crossover Score on Monday 4/25 in class

Rohan Gaikwad 4.5/5.
Review ticket with study plan was done on Monday.
Test Corrections are shown and got over 50% on MC (49 on both!!!!!!!!). Well done!!!!!!!!!!!!
Create Task is well done and documented, review video has no talking and shows runtime.


[SPARK-35775][SQL][TESTS] Check all year-month interval types in aggregate expressions

What changes were proposed in this pull request?

This PR adds test to check sum and avg works with all the YearMonthInterval types.
Why are the changes needed?

To ensure the results of aggregations are what is expected.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

New test.

Test build #140049 has started for PR 32988 at commit 2aa268d.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44577/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44577/

Test build #140049 has finished for PR 32988 at commit 2aa268d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140049/

Test build #140051 has started for PR 32988 at commit 99a0a32.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44579/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44579/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44579/

Test build #140051 has finished for PR 32988 at commit 99a0a32.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140051/

Test build #140068 has started for PR 32988 at commit 36d7797.

Test build #140068 has finished for PR 32988 at commit 36d7797.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140068/

retest this please.

Test build #140070 has started for PR 32988 at commit 36d7797.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44595/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44595/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44596/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44596/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44598/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44596/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44598/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44598/

Test build #140077 has started for PR 32988 at commit 2844101.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44605/

Test build #140070 has finished for PR 32988 at commit 36d7797.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44605/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140070/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44605/

Test build #140077 has finished for PR 32988 at commit 2844101.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140077/


vfs: Make the filesystem query API confirm POSIX standard

Summary
specified here:
https://pubs.opengroup.org/onlinepubs/009695399/basedefs/sys/statvfs.h.html
Note:
need merge with https://github.com/apache/incubator-nuttx-apps/pull/325 together.
Impact
No, just change the struct and function name
Testing
Please ignore the following nxstyle false alarm:
fs/nfs/rpc.h:446:9: error: Mixed case identifier found
syscall/syscall_funclookup.c:99:3: warning: #include outside of 'Included Files' section

@yamt do you have more concern about the change? If no, please merge this patch and apache/incubator-nuttx-apps#325. The check failure has to be ignored since the change in nuttx and apps need sync to pass the compile.

@xiaoxiang781216 i think f_fsid is meant to be an ID for each individual file system, not file system type. how do you think?

@xiaoxiang781216 i think f_fsid is meant to be an ID for each individual file system, not file system type. how do you think?

Yes, you are right. After more thinking, I will keep both interfaces to improve the compatibility.

@ymat, the new patch implement statvfs/fstatvfs on top of statfs/fstatfs. Please review again.


CI: job cacher

+ npm run format:check

><EMAIL_ADDRESS>format:check
> prettier --check .

sh: 1: prettier: not found

https://build.libelektra.org/blue/organizations/jenkins/PermaPlanT-Folder%2FPermaplanT/detail/master/240/pipeline

Looks like a disk space issue?

I've wiped all the servers, caches, etc 😀

Do we have this plugin in jenkins? Jobcacher

I've wiped all the servers, caches, etc 😀

Thank you :heart:

Do we have this plugin in jenkins?

We can install any plugin if it somehow reasonably maintained (in particular: won't break future updates of Jenkins)

Jobcacher Might come handy with this https://github.com/ElektraInitiative/PermaplanT/issues/241.

Great, if you have experience with it we can give it a try. What would it cache that we don't cache yet?

Jobcacher Might come handy with this #241.

Great, if you have experience with it we can give it a try. What would it cache that we don't cache yet?
I dont have experience with it, I just thought maybe it can solve the caching problem and autoclean up or just restrict the cache space.

@lukashartl @fel115 What do you think?

Also possible solution, write a script that fails when the disk space is about to get short. With a soft and hard limit for example.

We already have monitoring and automatic regular cleaning but sometimes it hits the CI pipeline nevertheless.
A script that does some environmental checks is a good idea but I think it is rather low-prior. @4ydan can you start an issue with ideas what we could do? Also take the features and issues from Elektra into account.

A script that does some environmental checks is a good idea but I think it is rather low-prior. @4ydan can you start an issue with ideas what we could do? Also take the features and issues from Elektra into account.

Alright. #543


moveit indicating successful move but real robot not moving

I am developing a support package for the CX165L robot. I am able to visualize the robot in rviz, receive correct joint states for the real robot via the /joint_states topic, and plan/execute trajectories in MoveIt.
However, the real robot does not move despite MoveIt indicating the trajectory was successful. I have attached terminal outputs for the following:

rostopic echo cx165l_arm_controller/follow_joint_trajectory/goal

joint states before and after execution

output from moveit_planning_execution.launch with a planned and executed trajectory

This is a current state of the rqt_graph:

I am happy to provide additional files. @matsui-hiro please let me know if you have any ideas. If successful, I can create a PR to add CX165L functionality if you are interested.

I checked the terminal log, but no cause was found.
I would like more information.
If it's OK ,could you upload the launch file of CX165L and tell me how to run it ?
I think source code is here,but can't find a moveit execution launch file.
Also, it is possible that the joint values ware not sent to the controller due to the network environment.
I want your system information.(on native Ubuntu, or WSL, etc.)

Thank you for sharing the source code.
I ran the launch file in our environment and the same problem occurred.
After investigation, I found that the cause may be in the libkrnx (communication library with the controller).
Depending on the AS system version of the controller, sending joint values may fail. Therefore, please let me know the version of your AS system software.
And I plan to update the library to solve the problem.

Hi @matsui-hiro! Thank you for investigating the problem.
The AS system version is ASEB01010XX3S. Please let us know if there is anything we can do to help solve the issue.
This is the system environment on our E02 controller:
.*=== AS GROUP ===         : **ASEB01010XX3S** 2022/08/02 07:27
.*USER IF AS               : UASE01010XX3S 2022/08/02 07:27
.*USER IF TP               : UTPE01010XX3S 2022/08/02 07:25
.*ARM CONTROL AS           : AASE01010XX3S 2022/08/02 07:26
.*USER IF AS MESSAGE FILE  : MASE010XX3SEN 2022/08/02 07:27
.*USER IF TP MESSAGE FILE  : MTPE010XX3SEN 2022/08/02 07:25
.*ARM DATA FILE            : ARME01010XX3S 2022/08/02 07:25
.*KERNEL                   : _KNL103300000 2016/01/13
.*DRIVER                   : _DRV104800000 2018/02/08
.*RFS                      : _RFS20030X202 2021/08/31
.*=== SERVO GROUP ===      : SVE_08000006C 2021/06/09 14:52
.*ARM CONTROL SERVO        : ASVE08000006C 2021/06/09 14:47
.*SRV DATA FILE            : ASPE08000006C 2021/06/09 14:47
.*ARM CONTROL SERVO FPGA   : ASFE08000000A 2015/04/01 09:29
.*
.*Cpu board type           : 1VA
.*CF                       : 4GB CompactFlash Card 20180831 0000047E

The AS system version is ASEB01010XX3S. Please let us know if there is anything we can do to help solve the issue.

Thank you for your cooperation.
ASEB01010XX3 is  an E Series(OpenAS) software and some versions of E Series(OpenAS) cannot receive joint values with the current library. I will let you know when I have fixed and uploaded the library.

When can we expect the fix to be released? Is there anything we can do to help?

Updates are scheduled by June 9.
Please confirm the correction in your environment after the update.

Hi, @Landon-Getting.
I updated the library. Could you please try again with this?
When you try with CX165L, replace only khi_robot_control pkg.

Hi @matsui-hiro!
Thank you for the update. I have replaced the khi_robot_control package in my local workspace. I also recently upgraded to a different machine with Ubuntu 20.04 and ROS Noetic.
Although your change may have worked, I am now encountering different errors during bringup. Please see the terminal_log.txt.
I've attempted to solve the particular error below by switching via the /controller_manager/switch_controller service to position_controllers/JointGroupPositionController from the cx165l_controllers.yaml file in the bringup config.
[ERROR] [1686353857.803995864]: Could not load controller 'cx165l_arm_controller' because controller type 'position_controllers/JointTrajectoryController' does not exist.
[ERROR] [1686353857.804049420]: Use 'rosservice call controller_manager/list_controller_types' to get the available types

I will continue troubleshooting the errors and let you know if they are resolved. If you have any solutions, please let me know.

Hi @matsui-hiro,
My local machine was missing the joint-trajectory-controller library.
sudo apt install ros-noetic-joint-trajectory-controller
After installing, I was able to run the moveit_planning_execution.launch file and move the real robot!
Thank you for your help! If you would like, I can help create a pull request for the cx specific directories.

Hi, @Landon-Getting.

After installing, I was able to run the moveit_planning_execution.launch file and move the real robot!

I am glad to hear that you are now able to move cx !

Thank you for your help! If you would like, I can help create a pull request for the cx specific directories.

For the merge,  need to adjust the acceleration values in joint_limits.yaml and align the file format with the other robots.  After that work is done I will create a pull request. (the timeing is in plannning)
Thank you for your cooperation !

Thanks for sharing back @Landon-Getting ... This solved my issue, too:

My local machine was missing the joint-trajectory-controller library.
sudo apt install ros-noetic-joint-trajectory-controller
After installing, I was able to run the moveit_planning_execution.launch file and move the real robot!


fix: setting length of label property in mp_jsonout

Issue
#272
Intent
Prevent errors in json generation with meta enabled in mp_jsonout
Implementation
Set length for label to 350 (94 extra chars to allow for escapes and quoting)
Checks

[ ] Code is formatted correctly (sasjs lint).
[ ] Any new functionality has been unit tested.
[ ] All unit tests are passing (sasjs test).

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Closes #272


gnomeExtensions.dash-to-panel: 38 -> 39

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 97840 1

1 package built:

gnomeExtensions.dash-to-panel


List.generate for 2d/3d list bug

If we create a 2d/3d list by this way:
  List recipes = List.generate(
      999,
      (i) => List.generate(
          999,
          (i) => List<Ingredient>.filled(
              0,
              Ingredient(
                  name: '',
                  carbohydrates: 0,
                  proteins: 0,
                  lipids: 0,
                  fibers: 0),
              growable: true)));

Please assume that Ingredient is a simple Class.
When we try to access it:
print(recipes[0][0][0].name);
Arriving at this point:
print(recipes[0][0][0].
We need to see (access) the Class properties (like kcal, name, carbohydrates...); exactly as in a one dimension list.
However, at least in VS Code, the code editor display this only: hashCode, runtimeType, toString(), noSuchMethod(...)
When I try to create the list by this way:
List<List<List<Ingredient>>> ingredient =
      List.generate(999, (index) => [[]], growable: true);

This bug does not exists, but I have no idea on how to fill (give a dimension) the first and the second list...

@WizardingStudios
I think your issue is more suited for https://github.com/dart-lang/sdk/issues as it has to do with dart language and not Flutter, so please file the same in dedicated repo.
Closing from here.


One concat space

Thanks you @MathiasReker. I must close this PR because it doesn't bring any value.


1510 | Stone Game IV | C++

Pull Request
What have you Changed(must)
Added 1510. Stone Game IV (https://leetcode.com/problems/stone-game-iv/)
Issue no.(must) - #1167
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@swaingotnochill @JOKUE2002 @SSKale1

fine i will add

I have added the approach in the top

@SSKale1 @JOKUE2002 @swaingotnochill

@shouvikch97


Increasing test coverage.

Coverage increased (+2.63%) to 93.94% when pulling f29f1f46316da8a541943bbe901586a713775f67 on ws233:master into 051ed1c696922eb5545dfe4cd6099fb8fb8b064d on gali8:master.

Coverage increased (+3.61%) to 94.93% when pulling ae90c2ce87802f35aa220d6ff16dc55ec9ee6ffb on ws233:master into 051ed1c696922eb5545dfe4cd6099fb8fb8b064d on gali8:master.


Version for Linux?

I'm a Linux user, and also a steam deck user, and would love a build for Linux.

I can look into it. It should not be hard to do. ModBringer's main function is moving some files around.
However, I do not have a Steam Deck, to test on. I will likely need to rely on you to confirm if it works also on Steam Deck (which I think it would if I can implement it for Linux).
I'll look into it in the next few days and test it on my own system.

@LovecraftianGodsKiller Please follow this link and try it out: https://github.com/Squiblydoo/ModBringer/releases/tag/1.2.3
I tested it works as expected on my Linux system. I don't have a steam deck to test on there.
On Linux, you have to launch it from a terminal.
Open the terminal application, navigate to the download directory using the command cd (so for example cd ~/Downloads/Linux_modbringer/) then execute it with .\modbringer. It will then give you directions for setup.
(You may also be able to unzip it, then rightclick the directory and choose Open in Terminal).
I had never created a more advanced GUI and developing a new one is beyond the scope of the time I have for this project. :(

I have not heard back, but from my perspective, a Linux version now exists. If you identify any issues with it at a later time, please feel free to open a new issue.


Rework executor validation

The list of executors in configuration is intended to be read only. Using Sequence for this makes that more explicit, as Sequences cannot be modified.
This is part of a move towards using Sequence rather than List in many places (although the runtime type remains a list) because Sequence has better behaviour with subclasses of its parameter type: it is covariant, rather than invariant.
This PR rearranges the validation logic a bit so that there is no longer an arbitrary property setter for the executor property - it was always a bug to replace the executor list in a config once parsl has been configured, and removing this property setter enforces that more strongly.
Type of change

Code maintentance/cleanup

this was harder than i expected - i might get back round to it another time but it's not super high priority (as the inactivity shows)


Cinema template uses too small thumbnail

Unfortunately, the Cinema template uses thumbnails that are too small. 300px thumbnails are used on the first call. The display therefore appears pixelated / blurred.As soon as a product is called up, the correct 600px thumbnail is loaded in the overview.
This is even the case with the demo images
If I set object-fit: to scale-down instead of cover, as written in a previous post, the 300px is displayed in full and unpixelated in the small display. This would now also be correct for cover and with a larger thumbnail.

[public] Shopware automated response: This issue is linked to the internal issue: TCM-207
[created from TCM-207, comment 512598]


[cda.pl]  Unable to extract default player_json & No video formats found

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.11.15. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.11.15

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

PS C:\Program Files\mpv> .\youtube-dl.exe -vF https://www.cda.pl/video/17338600e
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-vF', 'https://www.cda.pl/video/17338600e']
[debug] Encodings: locale cp1250, fs mbcs, out cp1250, pref cp1250
[debug] youtube-dl version 2017.11.15
[debug] Python version 3.4.4 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
WARNING: unable to extract default player_json; please report this issue on https://yt-dl.org/bug . Make sure you are us
ing the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include i
ts complete output.
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest ve
rsion; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete outpu
t.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp10f7prtw\build\youtube_dl\YoutubeDL.py", line 784,
in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp10f7prtw\build\youtube_dl\extractor\common.py", lin
e 437, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp10f7prtw\build\youtube_dl\extractor\cda.py", line 1
80, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp10f7prtw\build\youtube_dl\extractor\common.py", lin
e 1075, in _sort_formats
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure y
ou are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and
include its complete output.

C:\Program Files\mpv>

Works for me here
$ youtube-dl -vF "https://www.cda.pl/video/17338600e"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-vF', u'https://www.cda.pl/video/17338600e']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.11.26
[debug] Python version 2.7.14 - Darwin-17.2.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.4, ffprobe 3.4
[debug] Proxy map: {}
[CDA] 17338600e: Downloading webpage
[CDA] 17338600e: Downloading 360p version information
[CDA] 17338600e: Downloading 720p version information
[CDA] 17338600e: Downloading 1080p version information
[info] Available formats for 17338600e:
format code  extension  resolution note
vl           mp4        360p
lq           mp4        480p
sd           mp4        720p
hd           mp4        1080p      (best)

Could you run youtube-dl -vF "https://www.cda.pl/video/17338600e" --write-pages and upload *.dump files?

Sure, no problem :)
1080- https://gist.github.com/rewida17/236666019d3e6fff06822fdcc21652bb
720-  https://gist.github.com/rewida17/6f1a7efc5bdf86b42887d247aab09825
360-  https://gist.github.com/rewida17/efaaf4614be658d6250111058ef45dba
"FullDump"  https://gist.github.com/rewida17/16f5ad15875f11ded860a71b7732c05c
IHMO at this time this issue can be closed. Also i can`t reproduce  this bug :/
After that error I was checked this on Fedora machine, And this works correct.
Sorry for wasting Your time...  Huge thanks for your great work.

No problem! It's not unusual for a website to go crazy :)


post data sync, the lnd wallet isn't unlocking

Version 1.7.0
It worked yesterday though, after accidentally throwing the main fuse off - rebooted and entered passwd and it continued - this morning the data sync finished but i'm having trouble maintaining open access to the lnd wallet, passwd C, -could it be that while loading block index the wallet isn't accessible maybe because of cpu/mem resources ?
i eventually managed to get "ok LND unlocked", after waiting a few minutes, ...but turned out to be false, it asked for the password C again -
on screen i can see the script has exited and message reads  "bitcoind is warming up .. loading block index"
under that
(exit after too much restarts/unlocks - restart to try again)
&
also .. i did a full upgrade, after all that i thought why not, see what gives -at least now i have the data on ssd, the most painful process [10days] -i did see a couple of warnings, the scariest one was the /etc/pam.d/users conf , which i didn't alter..   so what are my chances of actually logging in and test a tx today ?
: (
chances of me having to reflash the sdcard ?
; )

final-sync wait time over - unlocked
okay

: P
a full two weeks - it's over Mainet is accessible now - patience , bucket loads of it too... i think the best way to go about is to just let it run - wait till it says it's done - everybody gets all knotted up over the constant wallet unlocking and rolling forwards 10days , then indexing another three days . If not rebooting will only make the whole thing slower and risking corruption - pi4 has 4Gb of ram but ideally you would need 8Gb. i cleared out the caches every 2/3hrs the last two days had testnet synced in under 3hrs , as a test to see if it would come out the other end without much complaining - and it did , which encouraged me to go back and finish mainet sync  -- portfowarded bitcoind 9735 & lnd's 10009, enabled upnp on router - oh and i did choose the BTRFS option - i think ultimately it made a difference , in that the data after , a dozen reboots , was still usable , another thing i kept an eye out for was the temp
on the pi , i placed a small fan within 10cm distce which kept the temp under 47* - just remember - Raspiblitz is a 2weeks job , in the end you could be as chuffed as i am right now ...
case closed, on this end .


Missing sensors

I was using the sensors _loadOrEpsPower and _gridormeterpower for house load and grid load respectively. Having updated today, all the sensors are renamed and I can't find these two values.
None of the numbers for any entities seem to match.

Hi Nurgus,
I wanted to address a few points concerning the current Sunsynk API's integration with Home Assistant. I have identified several issues that I believe may be leading to the instability you've been experiencing.
Firstly, I found that the initial setup of the device as a service was incorrect, which may be contributing to the hanging/crashing problem with Home Assistant.
Secondly, it appears that some extraneous sensors were inadvertently included in the device display, which don't pertain to the Sunsynk integration. This is an area that certainly needs rectifying to avoid potential confusion and system overload.
Additionally, I realized that the integration needs to accommodate multiple solar systems, as a plant could potentially contain more than one inverter.
Furthermore, I am currently enhancing the Sunsynk integration to function effectively with the Energy Dashboard. I understand that this feature was not working as expected before, which may have caused some inconvenience.
I deeply appreciate your patience as I work on resolving these issues. Please feel free to share any additional feedback you might have. Your insights are invaluable in helping me improve this integration. I will keep you updated as progress is made and aim to rectify the situation as promptly as possible.

We also get the data now straight on inverter level
instantaneous_load = loadOrEpsPower
_instantaneous_grid_i_o = gridOrMeterPower
Let me know if I'm wrong, I'm still trying to figure out where to map each sensor correctly

Update
Instantaneous Grid I/O Total = gridOrMeterPower
Instantaneous Load = loadOrEpsPower

Hi, I'm impressed with how this integration is coming along and excited to get started using the service to control my Sunsynk gear!
Instantaneous Grid I/O Total = gridOrMeterPower
Instantaneous Load = loadOrEpsPower
The problem I have is that these sensors are giving me totally different readings to the ones I get from the sunsynk app.
In the screenshot, my house is pulling 7923w yet the Instantaneous Load sensor says 292w
From the grid it's pulling 26w and the Instantaneous Grid I/O Total sensor shows -1945w, These sensors are always wrong.
Instantaneous Battery and Instantaneous Generation sensors are always correct.
Let me know if I can provide any more information.

So I think what might have happened was the refresh timer was 30 to 60 seconds behind and might be the reason for why the data is not matching your app and in home assistant.
I will take a look at the Instantaneous Grid I/O Total as it might read the wrong sensor will keep you updated
This is how my data looks like:

I've been refreshing the app and comparing the numbers to this integration almost continuously since yesterday and there's been no moment where those two sensors have been anywhere near to matching up. Meanwhile the generation and battery sensors are spot on.
Something's definitely not right but I don't know how to diagnose it further.

This illustrates the problem. In Sunsynk connect the load is around 450w and then jumps to around 1500w with not much in between. In Home Assistant the load history is very spiky and doesn't match up at all.

Integrations > Solar Sunsynk > Go into the Integration and Click on re-download the integration.

Done, no improvement.

other then that, I can give you the old branch with the old integration code (This will be the last resort if all fails)?

Nah I want to help crack the problem. Let me know if you think of anything to test or any information I can provide.

Hmmm.... okay I'm going to use sunsynk virtual user and then I will check their system and see if I can see anything out of the ordinary.

I can see the differences now

Okay, updated the integration, check if your data is now updated to the correct values

Oh yes wow, that's fixed it. All sensors looking good.  What was the problem?

So it seems like there is more than one place you can get those sensor values. The ones I selected were the individual values for load and grid and the correct ones are calculated


[19.x] WFLY-13024 Integrate MP Fault Tolerance 2.1

Resolves
https://issues.redhat.com/browse/WFLY-13024
https://issues.redhat.com/browse/WFLY-13190
Upstream
https://github.com/wildfly/wildfly/pull/13003

Pre-check passed: https://issues.redhat.com/browse/EAP7-1428?focusedCommentId=13989088&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13989088


Added Tests for the Configuration Options.

Do you plan on adding more? or just merge this for now?

I'm going to add more for the rest of the options. The only test that's really 'finished' is the one for the indicator.
Too bad nothing is ever really finished. Just called a finishing point.

Also wanted people to start poking at the test.

I'm going to go ahead and merge. I'll do more work in master.

Please don't work in master. Using PRs helps keep us aware of the changes that are happening and allows a platform for discussion if we see things that can be improved.
Unit tests are important but not critical that they get merged quickly.

Okay. Sounds good with me.


Update grpc-api, grpc-okhttp, grpc-stub to 1.62.2

About this PR
📦 Updates

io.grpc:grpc-api
io.grpc:grpc-okhttp
io.grpc:grpc-stub

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

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.grpc" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.grpc" }
}]

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

Superseded by #383.


Document Intelligence updates

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-ai-documentintelligence


Fix app.version error, fix typos in ingresses

longer deadline for init jobs

No more Chart.version and App.version. There is only one version in Chart.version. This option set to latest in master branch to use latest images if install from master. Release process will override it to tag value: https://github.com/flant/loghouse/blob/master/.travis.yml#L17


Adding a homepage like feature with navbar

Is your feature request related to a problem? Please describe.
A homepage like feature with navbar and footer and other pages guides the user better
Describe the solution you'd like
a homepage with a navbar having elements like aboutus, contact,etc  and footer with contact and social handle info
Describe alternatives you've considered
none
Additional context
none

@ShambhaviSharma0110 Go Ahead!

cool! I'll start working on it then


helm init fails when namespace already exists

When running helm init on an existing namespace, I get:
helm init --namespace default
$HELM_HOME has been configured at $HOME/.helm.
Error: error installing: [namespaces "default" already exists, the namespace from the provided object "default" does not match the namespace "helm". You must pass '--namespace=default' to perform this operation.]

I think helm should skip namespace creation if it already exists

I agree with this. It doesn't seem like we should force users to create a new namespace just to use Helm.

+1

I'm having the same problem
Error: release funky-buffoon failed: namespaces "default" already exists
when running helm install.
version:
Client: &version.Version{SemVer:"v2.6.0", GitCommit:"5bc7c619f85d74702e810a8325e0a24f729aa11a", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.6.0", GitCommit:"5bc7c619f85d74702e810a8325e0a24f729aa11a", GitTreeState:"clean"}

@Anmorata can you please open a new ticket for this? Also we'd need the following information:

what chart you were deploying
how is your kubernetes cluster set up
how you installed tiller (assuming helm init)

cluster is set up in azure cloud by azure wizard. kubernetes version is 1.6.6 and tiller version is 2.6.0.
I'm deploying my custom build chart, which I build from helm create template. just added few my lines.
helm came with azure, I just upgraded

Thank you. Please open a new ticket. This looks like a different issue than what's described in the OP.


automatic peer TLS

Bare-bones; no cert verification, etc.

this looks great. Defer to @xiang90

Merging after 2.3 release

LGTM. Let's resolve the conflict and get this merged.


Test that entries and prefixes with same path are both displayed

Demonstrates issue in #46.

Sadly, there's no good way to detect whether both the outer/ folder and outer file exist due to limitations in the Hyperdrive-next API.
I'd rather not merge this for now unless there's a clear way to detect it in Hyperdrive itself.


Remove unused stop_reactor keyword

Coverage decreased (-0.01%) to 74.72% when pulling 784c2ba10a2671e20267fe9fae885c3c175b28d7 on remove_stop_reactor into 16f2dfd99e5894efb32dd1753c439ca8758ce1c7 on master.

Merging, there are not any functional changes here and grepping around for stop_reactor yielded zero results.


Implement new Designs for the home screen

Feature Description
Implement new Designs for the home screen
**Pre-requisites/ dependencies
-Designs
https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/379c47b1-90ba-488c-8763-e69db4b0768d/
**Task Break Down

Design home screen
Implement logic for home screen

Buy Bitcoin and notification stack is remaining

Verified this issue on keeper app v 1.1.11(296)


Android not triggering onLoadEnd if image width and height is 0

I was displaying a placeholder when the image is not yet loaded.
I'm doing something like this
{!this.state.loaded ? this.renderPlaceholder() : null}
<FastImage
            source={{uri: uri, cache: 'immutable'}}
            style={[{...size, borderRadius: 5}, this.state.loaded ? {} : {width: 0, height: 0}]}
            onLoadEnd={this.onLoadEnd.bind(this)}
          />
```

onLoadEnd I set the state of loaded to true. This works perfectly fine on iOS, but on Android the onLoadEnd event is never triggered when we have width and height of 0. So you end up with hacks to make it work on both Platform. I ended up ditching the 0 width and height and absolutely positioning the image behind the placeholder. This way you don't see a jump in the UI when the image gets loaded and the placeholder is destroyed.

remove {width: 0, height: 0}
works fine for me, use overlay view using position absolute

trying to use display: none, also faced this, onLoadEnd didn't get triggered.
so switching to opacity: <condition> ? 1 : 0 on my case.


Cleaner detection of out-of-gas situations

Currently, when contract runs out of gas, you can see the 'panic' message, but you don't know the reason. It would be great if we passed the clear information that this was caused by out of gas.
🤔 Rationale
This will make debugging a lot easier.

which command gives you that? it will help if you be more specific. This will help us fix it faster. Thanks :)

closing due to inactivity


Made BokehRenderer.server_doc method more general

The BokehRenderer.server_doc method was mostly used internally to create and attach a plot to a bokeh Document and return it. Apart from fixing a bug attaching the periodic utility, this PR allows the method to be called directly with a HoloViews object.

Ready to merge.

Looks good. Merging.


⚠ ClamAV has degraded performance

In fc85350, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 89 ms

Resolved: ClamAV performance has improved in 218828d.


xcursor implent inheriting themes

i went around moved things in CCursorManager added config checks a bit more so now it doesnt allocate memory etc depending on the config choice, reduced a bit of duplicated code, moved those into functions. and fixed a bit of inconsistencies when using hyperctl setcursor depending on hyprcursor theme/xcursor theme not found.
XCursorManager now supports Inherits from index.theme
also when testing i noticed asan reports a leak with hyprcursor, went back in commit history a bit and its been around for quite some time so its not something i introduced it seems. this technically means xcursor is now done? i think.
Direct leak of 110952 byte(s) in 207 object(s) allocated from:
    #0 0x555559991a05 in calloc (/home/tom/dev/Hyprland/build/Desktop-Debug/Hyprland+0x443da05)
    #1 0x7ffff7b7596e in g_malloc0 (/usr/lib64/libglib-2.0.so.0+0x7a96e)
    #2 0x7ffff7cc7ebc in g_type_create_instance (/usr/lib64/libgobject-2.0.so.0+0x43ebc)
    #3 0x7ffff7cb1392  (/usr/lib64/libgobject-2.0.so.0+0x2d392)
    #4 0x7ffff7ca549d in g_object_new_with_properties (/usr/lib64/libgobject-2.0.so.0+0x2149d)
    #5 0x7ffff473bfc0  (/usr/lib64/librsvg-2.so.2+0x33bfc0)

it spams
Indirect leak of 15 byte(s) in 1 object(s) allocated from:
    #0 0x5555599211d7 in malloc (/home/tom/dev/Hyprland/build/Desktop-Debug/Hyprland+0x43cd1d7)
    #1 0x7ffff4779a6d  (/usr/lib64/librsvg-2.so.2+0x379a6d)
    #2 0x7ffff44a8009  (/usr/lib64/librsvg-2.so.2+0xa8009)
    #3 0x295e6fffffffff  (<unknown module>)

so as before im here for judgement. i could technically drop the CCursorManager changes too and only go with the XCursorManager inherits change heh.

ok found a bug, investigating. hold it.

ok found a bug, investigating. hold it.

done :p was regarding env vars and reloading config from hypr to xcursors.

do you know jeremy clarkson

clang-format fail?

clang-format fail?

yeah im not sure how to fix that. lol im apparently newer versioned again then arch and the fail isnt telling me how it wants it >_<. im soon setting up a clang-format virtual machine on arch lol

clang-format fail?

yeah im not sure how to fix that. lol im apparently newer versioned again then arch and the fail isnt telling me how it wants it >_<. im soon setting up a clang-format virtual machine on arch lol

nvm i have an idea, push coming shortly.


Only for the first time?

class test {     public $a;     /*No hints*/ }
After you input the above code, you will not get any hints if you enter "Pu" or "func". That is to say, after I input "public" once, I will not appear again in class test. Can I only create 1 functions? It's really bad.

@DivenLee @kAlvaro try ctrl+shift+p and intelephense reindex. If still occur then... It's bug (I hope not)


Fix, Provider\Germany: Pentecost is not an official holiday - except in Brandenburg

Hi,
Pentecost was originally removed from the list of general holidays in #100.
That's because pentecost is not an official holiday (until today) - except in Brandenburg.
I don't know why, it was added again in #225.
So i don't remove pentecost from the list of general holidays again, but it is generally marked as TYPE_OTHER.

Hi @stelgenhof, Thanks a lot for your feedback. You are right, there was another test i missed. It's updated now.
I raised the memory_limit on my local machine to run all the tests.

@fbett I have fixed the code styling issues. Please rebase your branch. If all goes well, we can then merge your PR :)


InfiniteLoader has incoherent architecture for tracking loaded versus loading rows

I'm beginning to suspect that multiple people's different ideas about how to make InfiniteLoader work got accidentally mixed together in the course of merging PRs...
These quotes from the docs directly contradict each other:

InfiniteLoader is not a stateful component, meaning that it does not keep track of which rows it has requested to be loaded.

InfiniteLoader memoizes calls to loadMoreRows in order to avoid multiple calls with the same parameters while a user is scrolling.

This memoizer does literally "keep track of which rows InfiniteLoader has requested to be loaded".
A comment from a code example says

An entry (either LOADING or LOADED) can be treated as loaded as far as InfiniteLoader is concerned

I don't get this memoization.  If isRowLoaded should return true for both loading and loaded state, then InfiniteLoader should not need a cache/memoizer at all and can rely entirely on isRowLoaded to prevent redundant calls to loadMoreRows.
If isRowLoaded should not return true for rows that are loading, then a separate isRowLoading function should be required, or whatever API is necessary so that InfiniteLoader can be truly stateless and rely on the user's code to keep track of loading/loaded state.

What I am experiencing now is that rowGetter will be called for a row-index before is isRowLoaded and the thing will fail on the empty object (or undefined – I tried both) rowGetter returns. This thing should be either prepared to rowGetter returning nothing or probe each index with isRowLoaded before calling rowGetter. InfiniteLoader apparently follows some (ineffective) heuristics based on other parameters to decide when to call isRowLoaded.


Add howtos for each supported native device in whad-client doc

Add a page linked in documentation for each supported device detailing:

what it is
where to buy it
where to find the firmware source code
how to install this firmware onto the device

Firmware installation procedures have been documented (except for LoRa-e5-mini) in https://whad.readthedocs.io/en/latest/device/compat.html .


fix surefire dependency in surefire-fix branch

Fixed a surefire-plugin dependency in xchange-core

What does this fix?

It allows it to build. Without it, I get
[ERROR] java.lang.ClassNotFoundException: org.apache.maven.surefire.junit4.JUnit4Provider
during mvn install

no one else seams to have this problem
surefire dependancy is defined here: https://github.com/knowm/XChange/blob/develop/pom.xml#L511
in which directory are you running mvn install and which version of java and maven are you using?

no one else seams to have this problem
surefire dependancy is defined here: https://github.com/knowm/XChange/blob/develop/pom.xml#L511
in which directory are you running mvn install and which version of java and maven are you using?


azcopy copy symlink

Which version of the AzCopy was used?
azcopy 10.7.0
Which platform are you using? (ex: Windows, Mac, Linux)
Ubuntu 20.04
What command did you run?
azcopy cp  "https://hieustorage2.blob.core.windows.net/hieublob2/"    "https://hieustoreeu.blob.core.windows.net/hieublob2/" --recursive
What problem was encountered?
Symlinks are created as a text file containing the link
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
No

Hi @hieuhoang, thanks for reaching out!
The blob service does not support symbolic links, so I'm inferring that you are storing symbolic links as blobs that contain only the link?

Yes, that's how blobfuse (v. 1.3.6) create symlinks

fyi, here's a minimal example to reproduce the problem:
$ mkdir temp
$ echo content of file > temp/file
$ ln -s file temp/link
$ cat temp/file temp/link
content of file
content of file
$ azcopy sync "https://hieustorage2.blob.core.windows.net/hieublob2/hieu/temp"  "https://hieustorage2.blob.core.windows.net/hieublob2/hieu/temp.bak"
$ cat temp.bak/file temp.bak/link
content of file
file

Hi @hieuhoang, thanks for the info!
Could you please clarify the desired behavior? It sounds like you want blobfuse to recreate the symbolic link?

Hi @hieuhoang, thanks for the info!
Could you please clarify the desired behavior? It sounds like you want blobfuse to recreate the symbolic link?

yep, azcopy is not creating a faithful copy at the moment.

yep, azcopy is not creating a faithful copy at the moment.

Ok I see. I've reached out to my colleague who works on Blobfuse to help determine where the problem is.

Ok I see. I've reached out to my colleague who works on Blobfuse to help determine where the problem is.

thanks, let me know if I can help in some way. I'm also MSFT

thanks, let me know if I can help in some way. I'm also MSFT

Thanks @zezha-msft  for bringing this to my attention.
@hieuhoang ,this is a feature. The reason blobfuse only copies the link is our users want the symlink preserved. When you upload a symlink through Azure portal it uploads the contents of the linked file and does not preserve the link. Blobfuse is a virtual file driver and our users use symlinks so that the contents of the file can change they want the contents dynamic, in short, it is not a storage unit so it needs to preserve the link. If you want to show the linked file you need to mount the copied storage account again in blobfuse and make sure the link path exists. It may be better to use relative paths so that even if you copy to another storage account the linked path works.

Thanks @zezha-msft  for bringing this to my attention.
@hieuhoang ,this is a feature. The reason blobfuse only copies the link is our users want the symlink preserved. When you upload a symlink through Azure portal it uploads the contents of the linked file and does not preserve the link. Blobfuse is a virtual file driver and our users use symlinks so that the contents of the file can change they want the contents dynamic, in short, it is not a storage unit so it needs to preserve the link. If you want to show the linked file you need to mount the copied storage account again in blobfuse and make sure the link path exists. It may be better to use relative paths so that even if you copy to another storage account the linked path works.

Hi Nara,
Thanks for getting back to me.
I think you've misunderstood the issue. I am not using the Azure portal to upload a link.
I am using azcopy to try to copy the symlink but it creates a text file, not the symlink.
Yes, the symlink is a relative path, as you can see from the example. And yes, the link path does exist

Hi Nara,
Thanks for getting back to me.
I think you've misunderstood the issue. I am not using the Azure portal to upload a link.
I am using azcopy to try to copy the symlink but it creates a text file, not the symlink.
Yes, the symlink is a relative path, as you can see from the example. And yes, the link path does exist

ie. the link is not copied, it is converted into a text file

ie. the link is not copied, it is converted into a text file

@hieuhoang , I understood that you did not use Azure portal., and are using AZCopy . I was just using that as an example to say Blobfuse treats symlinks as a symlink file and will not copy the contents of the file. The symlink file created by blobfuse will be considered a symlink file only by Blobfuse. Any other tool will interpret it as a file whose contents is the link. So you cannot create a symlink and upload it using Blbofuse and expect it to work as an link if you don't remount it as a Blobfuse file system.

@hieuhoang , I understood that you did not use Azure portal., and are using AZCopy . I was just using that as an example to say Blobfuse treats symlinks as a symlink file and will not copy the contents of the file. The symlink file created by blobfuse will be considered a symlink file only by Blobfuse. Any other tool will interpret it as a file whose contents is the link. So you cannot create a symlink and upload it using Blbofuse and expect it to work as an link if you don't remount it as a Blobfuse file system.

In other words, will the copied link (currently just a file whose content is the linked file's location) start working as a symbolic link once the file system is remounted?

In other words, will the copied link (currently just a file whose content is the linked file's location) start working as a symbolic link once the file system is remounted?

yes when you mount it in blobfuse, it will be a symlink again.

yes when you mount it in blobfuse, it will be a symlink again.

thanks guys for your reply. I've just tried umounting and remounting. It does NOT recognize the symlink.
You should try to verify it yourself

thanks guys for your reply. I've just tried umounting and remounting. It does NOT recognize the symlink.
You should try to verify it yourself

I'm using blobfuse 1.3.6 if that helps

I'm using blobfuse 1.3.6 if that helps

@hieuhoang could you goto storage explorer or whichever tool you are using and verify if the metadata ( some tools call it user_property) contains is_symlink=true?  If it does not then AZCopy is not copying metadata and @zezha-msft should be able to help. I tested with my tiny copy utility to another storage account and it works. I specifically copy the metadata though.

@hieuhoang could you goto storage explorer or whichever tool you are using and verify if the metadata ( some tools call it user_property) contains is_symlink=true?  If it does not then AZCopy is not copying metadata and @zezha-msft should be able to help. I tested with my tiny copy utility to another storage account and it works. I specifically copy the metadata though.

good point. I've check the metadata for the source and target symlink files, they both have is_symlink=true.
I've also tried remounting the blobfuse file system but the problem persists.
If you are using your own utility to copy then it be doing something that azcopy isn't doing. Can you please try with azcopy to confirm

good point. I've check the metadata for the source and target symlink files, they both have is_symlink=true.
I've also tried remounting the blobfuse file system but the problem persists.
If you are using your own utility to copy then it be doing something that azcopy isn't doing. Can you please try with azcopy to confirm

sure, I will try with AZCopy but not immediately. It will take some time. After you mount the target in blobfuse what does ls -al in the directory containing the target symlink file show? Does it start with an 'l' like this below example of symlink copy?
lrwxrwx--- 1 nara nara     57 Jan 21 01:46 'symlinkfile (1).yml' -> /home/nara/NaraVen/azure-storage-fuse/azure-pipelines.yml
or does it start with a - "-rwxrwx---"

sure, I will try with AZCopy but not immediately. It will take some time. After you mount the target in blobfuse what does ls -al in the directory containing the target symlink file show? Does it start with an 'l' like this below example of symlink copy?
lrwxrwx--- 1 nara nara     57 Jan 21 01:46 'symlinkfile (1).yml' -> /home/nara/NaraVen/azure-storage-fuse/azure-pipelines.yml
or does it start with a - "-rwxrwx---"

$ ls -l link
-rwxrwxrwx 1 hihoan hihoan 4 Jan 21 01:59 link

$ ls -l link
-rwxrwxrwx 1 hihoan hihoan 4 Jan 21 01:59 link

i think i've spotted the problem. The key for the original file is
is_symlink
the key for the target file is
Is_symlink
That seems to be an azcopy bug in copying the metadata.

i think i've spotted the problem. The key for the original file is
is_symlink
the key for the target file is
Is_symlink
That seems to be an azcopy bug in copying the metadata.

only found it using the properties page on the azure website, doesn't show when using Storage Explorer

only found it using the properties page on the azure website, doesn't show when using Storage Explorer

Thanks @hieuhoang for the insightful observation!
@NaraVen we can't do anything about the casing unfortunately. The Golang http client changes the casing of headers before we could intercept the raw values. The solution may be for blobfuse to check the metadata value in a case insensitive way. Please let me know if that makes sense.

Thanks @hieuhoang for the insightful observation!
@NaraVen we can't do anything about the casing unfortunately. The Golang http client changes the casing of headers before we could intercept the raw values. The solution may be for blobfuse to check the metadata value in a case insensitive way. Please let me know if that makes sense.

hmm ok. I'll wait for you or @NaraVen to escalate it with the blobfuse team. At the moment, azcopy does not faithfully copy

hmm ok. I'll wait for you or @NaraVen to escalate it with the blobfuse team. At the moment, azcopy does not faithfully copy

ah, you were talking to Nara. I'll get out of your way. Please let me know when you have a solution

ah, you were talking to Nara. I'll get out of your way. Please let me know when you have a solution

No worries!
We've talked to the Go language team about this problem in the past, but unfortunately we didn't get a lot of traction on them providing a mechanism for us to fix this. So we have to live with this limitation for now.

No worries!
We've talked to the Go language team about this problem in the past, but unfortunately we didn't get a lot of traction on them providing a mechanism for us to fix this. So we have to live with this limitation for now.

We will change it in the blobfuse source code by the end of this week. However we will be releasing 1.3.7 only next month.

We will change it in the blobfuse source code by the end of this week. However we will be releasing 1.3.7 only next month.

@NaraVen Awesome! I'll close this issue then.
@hieuhoang please watch the upcoming blobfuse.

@NaraVen Awesome! I'll close this issue then.
@hieuhoang please watch the upcoming blobfuse.

great thanks

great thanks

@zezha-msft my 2 cent - the http header field names are case-insensitive, and some of them have special meanings:
https://en.wikipedia.org/wiki/List_of_HTTP_header_fields
If you're using them to send metadata then perhaps it would be better to format it like
Bobfuse-Header: is_symlink=true
rather than
is_symlink: true
which i assume that's what you're doing

@zezha-msft my 2 cent - the http header field names are case-insensitive, and some of them have special meanings:
https://en.wikipedia.org/wiki/List_of_HTTP_header_fields
If you're using them to send metadata then perhaps it would be better to format it like
Bobfuse-Header: is_symlink=true
rather than
is_symlink: true
which i assume that's what you're doing

@hieuhoang that's a good idea, and our plan for the future of metadata is along the same line: use the header value to store both the metadata key and value. However, this is an implementation that we hope the service could standardize so that all the tools out there could use. They are aware of this problem and should have plans for it.
I like your idea, but I think there may be a backward compatibility issue for blobfuse, namely the existing clients all have their links stored with is_symlink: true. It's up to @NaraVen but I think we should avoid breaking existing users and have blobfuse be more tolerant on the casing.

@hieuhoang that's a good idea, and our plan for the future of metadata is along the same line: use the header value to store both the metadata key and value. However, this is an implementation that we hope the service could standardize so that all the tools out there could use. They are aware of this problem and should have plans for it.
I like your idea, but I think there may be a backward compatibility issue for blobfuse, namely the existing clients all have their links stored with is_symlink: true. It's up to @NaraVen but I think we should avoid breaking existing users and have blobfuse be more tolerant on the casing.

for now we will leave it at is_symlink itself for the reason that @zezha-msft has mentioned above. Symlink maybe added to storage in future we will go back and comb our headers then.

for now we will leave it at is_symlink itself for the reason that @zezha-msft has mentioned above. Symlink maybe added to storage in future we will go back and comb our headers then.

just upgraded to 1.3.7 and looks like it's working. Thanks all


building with latest RC, main.css doesn't seem to be generated

From @RfanKD on October 1, 2016 6:48
Hi,
Thank you Ionic Team for the latest release candidate
but I seem to have a problem with I try to build with the sidemenu template
Short description of the problem:
building sidemenu project with latest RC and npm.
on first time starting the project up, the icons don't appear
when starting the project with ionic serve --lab, for a second, it seems the main.css has been removed from build folder. (In fact, the only file left in the build folder is polyfills.js)
Steps to reproduce:

ionic start --v2 myApp sidemenu
cd myApp
ionic serve --lab,(notice the icons don't appear) quit
ionic serve --lab, (notice the page1 don't appear, view console log to find that main.css seems to be deleted and not generated even when building is complete )

Copied from original issue: driftyco/ionic-conference-app#260

After review I am going to close this issue as a duplicate of #83


Updating ML version to 0.10.0

This PR fixes #15650

Pull Request Test Coverage Report for Build 922741484

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 43.369%

Totals

Change from base Build 922400433:
-0.002%

Covered Lines:
26602

Relevant Lines:
55738

💛  - Coveralls


Add Xonotic

This adds a Cask for Xonotic: The Free and Fast Arena Shooter

Add Xonotic.app v0.8.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

Thank you.


The solution to 49 doesn't work

I got an exception when running the 49th solution. The exception as follows:
ValueError: threshold must be numeric and non-NAN, try sys.maxsize for untruncated representation

Weird. It's still working for me using numpy 1.15.2.  What is your version?

Yes, you're right. It's working well under numpy 1.15.2 . The exception is caused by  numpy 1.16.3 which version I used in jupyter. So, it's a version problem.

I would be better if we find a solution that works on both version. Any idea?

I found the definition of threshold as follows:
threshold : int, optional
Total number of array elements which trigger summarization  rather than full repr (default 1000).
Nan always represents that it's not a number, so the solution of 49th doesn't work well under numpy 1.16.3. I think we should use np.inf which indicate a infinite number instead, and it works on both versions. And there may be a flaw here, the threshold was set to 1000, so a (16,16) matrix couldn't trigger the summarization. A (32,32) matrix could demonstrate the problem well. So the solution should be as follows:
np.set_printoptions(threshold=np.inf)
Z = np.zeros((32,32))
print(Z)

Nice. Can you make a PR ?

Try float("inf") instead of np.nan.


implement route53 hosted-zone query

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (48fc726) 69.94% compared to head (39fdd2d) 70.32%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
+ Coverage   69.94%   70.32%   +0.38%
==========================================
  Files          13       14       +1
  Lines         915      974      +59
==========================================
+ Hits          640      685      +45
- Misses        236      248      +12
- Partials       39       41       +2

Files
Coverage Δ

internal/client.go
40.00% <0.00%> (-1.27%)
:arrow_down:

service/route53.go
78.94% <78.94%> (ø)

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


Refactor member API

Coverage decreased (-0.05%) to 56.744% when pulling 17813a75f0d4b4547fc2f65ed29fa4b05fdac7aa on ywk253100:170519_member into d45ef78eaf5d5393b7b321b8601694c84b1898cb on vmware:master.


no function or associated item named from_fd found for struct TunTapInterface in the current scope

 $ cargo build --release
    Updating crates.io index
   Compiling tun2proxy v0.1.1 (/home/acheong/workspace/tun2proxy)
error[E0599]: no function or associated item named `from_fd` found for struct `TunTapInterface` in the current scope
   --> src/tun2proxy.rs:271:34
    |
271 | ...        TunTapInterface::from_fd(*fd, Medium::Ip, options.mtu.unwrap...
    |                             ^^^^^^^ function or associated item not found in `TunTapInterface`

warning: unused import: `std::os::unix::io::AsRawFd`
  --> src/tun2proxy.rs:20:5
   |
20 | use std::os::unix::io::AsRawFd;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

For more information about this error, try `rustc --explain E0599`.
warning: `tun2proxy` (lib) generated 1 warning
error: could not compile `tun2proxy` due to previous error; 1 warning emitted

 $ rustc --version
rustc 1.68.2 (9eb3afe9e 2023-03-27)

Since the CI had no issues during the compilation process (https://github.com/blechschmidt/tun2proxy/actions/runs/4725608560), it's likely that the dependencies were not updated. I have only recently sent a PR that integrates these features into smoltcp: https://github.com/smoltcp-rs/smoltcp/pull/776


⚠ GitLab Server has degraded performance

In ebf2c1b, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11497 ms

Resolved: GitLab Server performance has improved in 1720172.


Improve wording and formatting of future enhancements

What about the proposed solution to each of the planned enhancement? I think those are required too @heeeyi


How to use moment inside template?

I want to use moment.fromNow() in the angular material table in my template. How to do it?

This repo is for timezone shifts only. I think you need ask it in https://github.com/urish/angular2-moment - There's everything about moment namespace.
Anyway, I see there are no solution for you right now. You can write your own pipe for it by yourself.
Something like following code:
import {Pipe, PipeTransform} from "@angular/core";
import * as moment from "moment-timezone";

@Pipe({
  name: 'amFromNow'
})
export class FromNowPipe implements PipeTransform {

  transform(value: moment.MomentInput, withoutSuffix: boolean = false): string {
    return moment(value).fromNow(withoutSuffix);
  }
}

How to use:
<span>{{ momentValue | amFromNow }} </span> <!-- With 'ago' suffix -->
<span>{{ momentValue | amFromNow:true }} </span> <!-- Without it -->


Add comment to why we do batching in PerspectiveAPI.

I attempted to remove this in order to make retries easier, but noticed the slowdown.

Indeed no batch form attempts some recovery
https://github.com/googleapis/google-api-python-client/blob/369737102a0e944467b18617b2971db25bf85a6c/googleapiclient/http.py#L922-L933
The batch seems like it would be considerably faster
https://github.com/googleapis/google-api-python-client/blob/369737102a0e944467b18617b2971db25bf85a6c/googleapiclient/http.py#L1455-L1525
I think maybe we can implement a batch retry depending on the exception type at least
https://github.com/googleapis/google-api-python-client/blob/369737102a0e944467b18617b2971db25bf85a6c/googleapiclient/http.py#L1500-L1501
try:
    batch_request.execute()
except googleapiclient.errors.HttpError:
    # This would be errors with the request not the processing of the batch and we retry the whole batch

If one of the batch requests raises HttpError, I believe this would not make it up and instead be passed into the callback.
https://github.com/googleapis/google-api-python-client/blob/369737102a0e944467b18617b2971db25bf85a6c/googleapiclient/http.py#L1600-L1601


⚠ Bofrost.de has degraded performance

In b711532, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 829 ms

Resolved: Bofrost.de performance has improved in aba6d60 after 13 minutes.


Question regarding the detection example project under references/detection.

Hi, have been using the great detection example project created by @fmassa and had an issue when I created some tensorboard visualization code. I am doing only object detection and the bounding boxes in the targets looked completely wrong. After some debugging I pinned down the issue to the following line:
https://github.com/pytorch/vision/blame/eb7a0f40ca7a7e269e893c1a8ab5845085c8b219/references/detection/coco_utils.py#L64
This seems like a very deliberate line, so I am wondering what I am missing. Is this a different way of encoding bounding boxes other than simply doing bounding_box=(x1, y1, x2, y2)?
Thanks!

The COCO evaluation utils uses bounding boxes as (x1, y1, w, h), while the detection model uses (x1, y1, x2, y2), so this line is performing the conversion from one representation to the other.
Let me know if you have further questions

Oh my bad, don't know why I was positive that coco was x1,y1,x2,y2. Thanks a lot for the answer and for the great reference project!


Extension examples do not reflect new channel architecture

The extension examples won't work since they aren't aware of the new channel architecture introduced in v0.3. These should be re-written so extensions can be written using the new syntax.

this is now fixed in the latest release


Is this suitable for large scale dataset?

Hi! Thanks for your wonderful program!
In the "self._parse_function_train" function ,do you load all of your training dataset ?
I want to apply your code to video dataset，and should I load all of the training dataset in this function?
thanks!

Hi @feynman233,
please remember that the issue section should be used for bug reports on the dataset not for arbitrary QA. Please do have a look how tensorflows dataset api works (no it doesn't load the entire dataset at once in memory, instead it's queuing a specific amount of samples in RAM that can be defined by the user). And I have never worked with video data, so this is something you need to ask somewhere else.


Slider present value is not update

Slider present value is not update
if i write a code in viewdidload its working fine but if same code I call through any function slider Ui is not updated

Same happening with me. Did you find a solution for this?

I am also facing the same issue with this slider. Does anyone found any way out of this?

My solution was to change the slider minValue and maxValue in order to force it to redraw. So it can be something like:
        rangeSlider.minValue = 1
        rangeSlider.maxValue = 1
        rangeSlider.minValue = realMinValue
        rangeSlider.maxValue = realMaxValue

Hope this helps.

minValue and maxValue must refresh UI， but  selectedMinValue and selectedMaxValue no,  please fix this bug

I have the problem, somtime slider selected min and selected max malues shiw wrong data.How can fix it?


Add support for launchpad plugins

Hello @geert-janklaps,
I made a small addition to this repo to allow the local testing of Fiori Launchpad plugins (more info here, here and here). To load the plugins you currently need to add an standard html file to the app folder of your CAP project. Opening this page when running the CAP service will load the launchpad configuration from this plugin, as well as any launchpad plugins that are defined in the sapux list in package.json.
Best regards,
Jibbril

Hi @Jibbril,
Thanks for you contribution! Sorry for the delay, must have missed the notification of your contribution!
Will be publishing the updated version to npmjs right away!
Best regards,
Geert-Jan


[Update Request]: WinDirStat 2.0.1

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
WinDirStat.WinDirStat
Package Version
2.0.1
Please describe the changes you would like to see
There is a new release, no longer preview:
https://github.com/windirstat/windirstat/releases/tag/release/v2.0.1
See also https://windirstat.net/download.html for links to the same GitHub assets.

Since https://github.com/microsoft/winget-pkgs/pull/183283 is merged, I'll assume that this Issue is complete. If this is in error, please @ me and I'll reopen.


cmd/bosun: pass alert variables to unknown notification templates

#2275 allows to pass alert variables to action notifications.
This PR allows this for unknown and multiple unknown notifications.

hey, @kylebrandt , @captncraig
do you have any objections for merging this? we'd greatly appreciate that

It would be very useful to have this change


Including a "Required" parameter list for items

This will allow me to use openapi2proto.APIDefinition to generate JSONSchemas for tight request/response validation.
This project: https://github.com/chrusty/openapi2jsonschema is still a WIP, but you should be able to see what I'm trying to do there.

Thanks!


Expand Perf Suite Codebases, Expand Perf Suite Operations

The perf suite code typically was added for specific scenarios, but if you're trying to get a sense of how sourcemap emit has changed, it's hard given that not all projects use that functionality.
I think we should

Have perf suites projects enable emit
Have perf suites projects enable source map emit
Have perf suites projects enable declaration emit

In our current setup, we run benchmarks against a snapshot of a specific project, using the settings specific to that project. This is intended to cover different scenarios, such as:

Single-file output vs Multi-file output vs no output.
No source maps, vs external source maps, vs internal source maps.
Declaration emit vs. no declaration emit.
Emitting with and without comments.
Exercising different parts of the compiler (i.e., Compiler-Unions heavily uses union types to stress test those scenarios).

That way we aren't only focused on everything being a "kitchen sink" build, and we can catch performance regressions in non-"kitchen sink" areas (i.e., we don't want to slow down in code paths where we don't emit source maps). The current approach is somewhat randomized, but it covers most of the bases. I wouldn't want to have to run a benchmark against the same project for every permutation of every feature, as that doesn't scale.

@DanielRosenwasser Continue to run the perf suite as is, but additionally run the perf suite with declaration emit, source maps and declaration maps enabled. Basically produce two data sets.

What we can do is add new scenarios that point to the same sources but with different settings. That way old runs wouldn't be invalidated. So in that case we might have a "TFS+SourceMaps" scenario, etc.

Related: https://github.com/microsoft/TypeScript/issues/44251

This issue should also now track just adding new projects to the perf suite.

Would be great to have a (checked) JavaScript codebase in the perf suite.

Webpack?

Also related, but, I want to add two new hosts to ts-perf, vscode and bun.
The former we can pull via https://www.npmjs.com/package/@vscode/test-electron and then use ELECTRON_RUN_AS_NODE. I want this because electron's build of v8 has different options, mainly pointer compression, which impact the maximum memory usage of the process as well as the small integer optimization.
I also bun's new "as node" mode working with some tweaks to typescript (which are really just bugs in bun that don't have proper error messages). It's an interesting test as it's not v8, it's JSC.

Our hope is that we have this all in public, so things can be updated where needed by anyone. That and time / fragility are the reasons we haven't updated.
The snapshotting is hard to get rid of, because that's required for consistency in graphs without backfilling. However, I'm working on new infra that will be doing A/B tests, which means that we don't always need to be running a specific version if needed.

Drizzle? (see https://github.com/microsoft/TypeScript/issues/54939)

As of today, our benchmark suite can now run bun. For example: https://github.com/microsoft/TypeScript/pull/52656#issuecomment-1698316828
This is particularly interesting as it uses an entirely different engine (JSC) than Node/Deno (V8).
bun 0.8.1 is currently broken when running tsc (https://github.com/oven-sh/bun/issues/4327), but one can run 0.7.3 and it works. tsserver tests don't work, nor do tsserver startup tests, but given I'm using an old release, hopefully it is fixed in a future release.
I'm not yet sure if this should be a part of our main test suite; probably not until we increase our capacity a bit more.
Next up would be using VS Code's electron build via ELECTRON_RUN_AS_NODE, which has different performance behaviors due to pointer compression and memory caging; we don't currently test it, yet there's probably more CPU cycles spent in tsserver via VS Code than any other way to run TypeScript!

bun 0.8.1 is currently broken when running tsc (oven-sh/bun#4327), but one can run 0.7.3 and it works. tsserver tests don't work, nor do tsserver startup tests, but given I'm using an old release, hopefully it is fixed in a future release.

This is fixed in https://github.com/oven-sh/bun/pull/4378 (currently the canary build) which will become the release build (v0.8.2) tomorrow or the next day. The error was in some code assuming /dev/fd/${n} is mounted which it is not mounted by default in minimal Linux distributions like alpine. Our CI machine uses Ubuntu Server which mounts /dev/ by default, which is why we didn't catch it sooner. We will add an integration test that runs tsc --help.
The last few lines of output of strace tsc --help when tsc is run in bun:

Bun v0.8.1
Bun v0.8.2

Note how it is no longer calling openat() on /dev/fd/1 in v0.8.2

Thanks for the fix! There are other things that are broken (as noted above), but I have yet to go reproduce them. I'll send some bug reports once I get that sorted.

Hello from Prisma 👋🏼
Let us know if you're interested to hear from us about some complex types to test. It could be interesting to have some types used in Prisma Client Extensions, for example.
The performance test suite is located at https://github.com/microsoft/typescript-benchmarking? but is currently private? Do you plan to make it public one day? (Of course you might have reasons to not make it public, I'm curious here)

That repo is currently only build infra, which I plan to make public in short order as there's nothing sensitive.
The actual tool that does the benchmarking isn't ready to be made public and that's where the benchmarks currently live. I'm hoping I can fix it up to pull benchmarks from the public repo but it's not really made for multiple places to store benchmarks, or benchmarks that have to be cloned from elsewhere. But that's just details and I'm working on it. The new hosts were just a lot easier to add!

perf test vscode is now live: https://github.com/microsoft/TypeScript/pull/55267#issuecomment-1699851609
Next up is adding some more benchmarks and making public what I can make public quickly, e.g. the infra itself (but not the tool, sorry).

I'm happy to say that the new perf infra is now live and public: https://github.com/microsoft/typescript-benchmarking
I've been waiting to make this public as I did not yet have a good mechanism to support public benchmarks, but I got it all working in the past few days and now it's ready to go. The first public benchmarks are vscode, as well as self tests of TypeScript compiling itself (both tsc -p ./src/compile and tsc -b ./src).
Note that the public benchmarks are still split off from our old internal benchmarks; we have a lot more perf capacity than we used to have but still not enough to start adding lots that always run.
For now, they can be run via the perf test public preset. An example: https://github.com/microsoft/TypeScript/pull/56987#issuecomment-1881975364
This is the first time we've had new, actually error-free benchmarks in years, so I'm very excited.

I'm going to close this issue in favor of https://github.com/microsoft/typescript-benchmarking/issues/32 on the benchmarking repo.
This thread was refocused toward benchmark suggestions, but now that the repo that holds them is public, discussion and PRing can happen there.


Fix bug of calculation decodable immediate under arm architecture

1、
First,I generated such code for the InstanceCall instruction：
`
0x90d2a574    e59d0004      ldr r0, [sp, #+4]
0x90d2a578    e306e000      movw lr, #0x6000
0x90d2a57c    e340e010      movt lr, #0x10
0x90d2a580    e085e00e      add lr, pp, lr
0x90d2a584    e59eef8b      ldr lr, [lr, #+3979]
0x90d2a588    e3069000      movw r9, #0x6000
0x90d2a58c    e3409010      movt r9, #0x10
0x90d2a590    e0859009      add r9, pp, r9
0x90d2a594    e5999f8f      ldr r9, [r9, #+3983]
0x90d2a598    e12fff3e      blx lr
`
Because the offset of the data and target objects is too large, it uses decodable immediate.
2、
When I execute the code, the VM calculates the index of the data and target objects in the ObjectPool through CodePatcher, but the calculation result is wrong.
The wrong calculation logic is in the "InstructionPattern :: DecodeLoadWordFromPool" method of the "runtime/vm/instructions_arm.cc" file .

Thank you for the contribution!
The change is going through the CQ here: https://dart-review.googlesource.com/c/sdk/+/148340


fix: fixed security-bootstrapper Docker volume init semantics

fixes: #3851
Signed-off-by: Valina Li<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) NA; only Dockerfile, entrypoint.sh changed
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) NA; because it is fix for a WA

Testing Instructions

New Dependency Instructions (If applicable)

@vli11 , please complete the PR checklist with a why comment for those items not checked. Can simply be N/A because ...

I have not added unit tests for this bug fix because NA; only Dockerfile, entrypoint.sh changed

I have NOT opened a PR for the related docs change because it is fix for a WA

@vli11 , looking for you to edit the PR description and add those comment where it says (if not, why?)
example:

[ ] I have added unit tests for the new feature or bug fix (if not, why?) NA; only Dockerfile, entrypoint.sh changed

@vli11 i think you lost your Dockerfile changes.... please do not squash the commits before others finish your change reviews...

LGTM, but deferring to @bnevis-i & @jim-wang-intel for approval.

@lenny-intel can you dismiss your review?

recheck

Hi @ernestojeda how do we do a ci/cd re-build? .. want to re-run the build to see if this is just intermittent issue

"recheck" is correct. However, you may not have the correct permissions, so Jenkins might have ignored.

Hi @ernestojeda how do we do a ci/cd re-build? .. want to re-run the build to see if this is just intermittent issue

"recheck" is correct. However, you may not have the correct permissions, so Jenkins might have ignored.

weird, it used to be working for me though...


[otbn] Add spurious WE check to regfiles

See #12055 for context.

Rebased on top of recent blanking changes.

@GregAC @rswarbrick can you take a look at this one so that we can get it out of the way? It has been rebased onto the recent blanking changes...

I think we do want to OR the two halves of we_onehot in the bignum RF as if say we_onehot[2][0] is set and we_onehot[3][1] is set that would pass the checks but is definitely a fault (we'll never write to half of one register and half of another).
Not putting 'Request Changes' as I'm away tomorrow and it's a UK public holiday Monday so you can merge in the time whilst I'm away but please ensure this is sorted first.

Ok, makes sense. I have rearranged this so that the two halves are OR'd together before being fed into one checker instance, rather than checking the two halves separately with two checker instances. @rswarbrick can you take another quick look before we merge this?

Since this is just a re-index operation using assign statements, I don't think this should create any issues. There are no combo cells in between, so the we_onehot_unbuf get connected to the same physical nets as we_onehot.


[1.6]Fix typo

https://github.com/rancher/rancher/issues/10434

I noticed this issue when I fixed the config one https://github.com/rancher/rancher/issues/10247. However, my fix doesn’t work as I made a typo.
sorry about that.