Issue with changing mean function for GPRegression object

I have time series of gene data and I am building GP regression models in a loop for each gene. For each GP model I am specifying the kernel, gaussian noise and the mean function. I am then storing each GP object in a dictionary for later reference.
The code is as below:
for gene in micro_array_data_csv.columns:

    Y = np.array([[a] for a in rna_seq_data.loc[gene]])
    kernel = GPy.kern.RBF(input_dim=1, variance=0.5, lengthscale=4)

    kernel.lengthscale.constrain_fixed(1.15)
    mf = GPy.core.Mapping(1,1)
    mf.f = lambda x:Y.mean()
    mf.update_gradients = lambda a,b: 0
    mf.gradients_X = lambda a,b: 0
    mean_functions.append(mf)

    m = GPy.models.GPRegression(X,Y,kernel, mean_function=mf)

    m.Gaussian_noise.constrain_bounded(0, np.var(Y), warning=0)
    m.optimize(messages=False)
    m.optimize_restarts(num_restarts = 10, verbose=0)

    matrix_genes.append(gene)
    fig = m.plot(title = "{}".format(gene))

    plt.plot([2,6.5,24],[Y.mean()]*3, color="r", marker="o")
    gp_models[gene] = copy.deepcopy(m)

Inside the loop I get the following plots printed:

when I later plot them by using the objects in the dictionary I observe that the mean_function of all plots has changed and is made equal to the mean_function of the very last plot:

How can I change this behaviour and make each object retain their original mean_function?

Interesting ... this sounds like a bug where the mean function is not being copied in the deep copy. Is that consistent with the behaviour you observe?

Yes, that is exactly what I have observed

Great, would you be able to create a minimal example that recreates the error? (i.e. one that doesn't rely on your data).

This is a minimal example where you can observe the error:
  import scipy.stats
  import numpy as np
  import GPy
  GPy.plotting.change_plotting_library('matplotlib')
  import copy

  time_series = [np.random.normal(5, 1, 100), np.random.normal(0, 1, 100)]
  X = np.array([[i] for i in range(100)])
  gp_models = {}

  for i in range(2) :
      Y = np.array([[a] for a in time_series[i]])
      kernel = GPy.kern.RBF(input_dim=1, variance=1, lengthscale=1)

      mf = GPy.core.Mapping(1,1)
      mf.f = lambda x: Y.mean()
      mf.update_gradients = lambda a,b: 0
      mf.gradients_X = lambda a,b: 0

      m = GPy.models.GPRegression(X,Y,kernel, mean_function = mf)

      m.optimize(messages=False)
      m.optimize_restarts(num_restarts = 10, verbose=0)

      fig = m.plot(title = "Gaussian Process {}".format(i))

      gp_models[i] = copy.deepcopy(m)

  gp_models[0].plot(title = "Gaussian Process 0")
  gp_models[1].plot(title = "Gaussian Process 1")

Hi, thank you for the answer. I understand the source of the misbehaviour now. But what I am actually trying to do is to set the mean function equal to the mean of the data according to their Y values, so that in regions where there are no data the mean function goes to the Y mean value instead of defaulting to 0 which for some reason was happening before I set the mean_function parameter. To give some context: In my case here X values are time points, and Y values are gene expression values in that particular time point, and for each time point there are 3 gene expression values obtained by the measurements of 3 different replicates. Can you suggest me a feasible solution to obtain this and make each GP model maintain their respective mean function?

The normal approach to doing that would be removing the mean from the data.
If you set normalizer=True, it doesn't that, but also sets the scale.
Perhaps creating your own normaliser to remove the mean?

Ok so I set the normalizer = True and now the mean doesn't tend to go to zero in regions where there are no data. Does that mean the when normalizer = True it doesn't subtract the mean to the data hence mean function doesn't go to 0 (that's a bit counterintuitive if I understand it right)? Can you please explain this to me in some more details what actually happens when normalizer is true or false?

It subtracts the mean before training (and normalizes the scale of the data) but it adds back in the mean and scales back up before giving you the output. Does that make sense?

So if normalizer = False, it doesn't subtract the mean and doesn't go through some normalisation before training? But then how is it possible that in the final output the mean function tends to 0 in places of the graph where there is not data, as opposed to what happens when normalizer = True?
This is what happens when normalizer=True:

This is when normalizer = False:


Proxmox VM Upgrade - Reboot Loop when HDD attached

Describe the bug
When starting the the raspiblitz VM on Proxmox with the attached HDD with the blockchain data, the VM reboots after some seconds.
To Reproduce

REPAIR - MIGRATION, saved the migration file, stopped the raspiblitz-VM
Seen that it would better to use UPDATE, so

Restart the raspiblitz VM - UPDATE - RELEASE - UPDATE NOW  - saved backup file - stopped the raspiblitz-VM with the old version

Prepared VM based on Debian 12.8 and build sdcard - first with branch v.11.1 and HDD (with blockchain and old data) attached

Restart raspiblitz VM - after about 10 seconds it reboots. When I manage to login I see the rebooting dialog counter. When I press cancel I get the message with the ssh login, but it continues with shutdown and reboot.

Prepared VM based on Debian 12.8 and build sdcard - now with branch dev and NO HDD attached. I could login but it asks for the HDD. As soon as I attach the HDD it also falls into the reboot loop.

(I've restarted my old raspi VM version again, so I can change on the HDD of do different preparing steps, if necessary)

Hmm .. so there are some "reboots" in the boostrap script .. we would need to pin down which it is.
For this can you do the following:

Boot up, connect/patchthru HDD and let reboot.
Disconnect HDD on bootup
Login per SSH, CTRL+C to terminal and cat /home/admin/raspiblitz.last.log
Paste the last lines of that log file here for debugging


bug

Please select the area your Bug applys to. (Multiple selections are Possible)

[x] Onboard. Initial vault setup and import/export pods
[x] Create. Note creation, lookup, snippets and templates
[x] Retrieve. Backlinks, references, graph view
[x] Structure. Refactoring, multi-vault and schemas
[x] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information)

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of >Dev:Dendron: Open Logs here

If you have a specific issue, please submit a ticket. Closing this


Use pip3 instead of pip to upgrade pip, wheel & setuptools

I tested this by deploying a 3-node cluster on AWS and it worked fine.

:+1:


Private link ga

Adding Private Link troubleshooting articles

@ramandhillon84 : Thanks for your contribution! The author(s) have been notified to review your proposed change. @rdhillon

Hi @ramandhillon84. Microsoft authors are expected to submit content to the private repository, azure-docs-pr instead of this public repo, so the content can be staged and validated by the current validation rules, which aren't available in the public repo. We will close this PR.
Please submit this PR in the private repo. See the following references for help:

Should I work in a private or a public repo?
If you need help getting set up to work in the private repo, see Fork and clone a reposistory.

Thanks.
@KumudD


[Bug]: Rocblas errors out on fp16 gemm operations

Describe the bug
rocblas returns rocblas_status_internal_error when rocblas_hgemm is called on mi100:
rocBLAS error: Tensile solution found, but exception thrown for { a_type: "f16_r", b_type: "f16_r", c_type: "f16_r", d_type: "f16_r", compute_type: "f16_r", transA: 'N', transB: 'N', M: 256, N: 256, K: 64, alpha: 1, row_stride_a: 1, col_stride_a: 256, row_stride_b: 1, col_stride_b: 128, row_stride_c: 1, col_stride_c: 256, row_stride_d: 1, col_stride_d: 256, beta: 0, batch_count: 1, strided_batch: true, stride_a: 0, stride_b: 0, stride_c: 0, stride_d: 0, atomics_mode: atomics_allowed }
Alpha value 7.21875 doesn't match that set in problem: 1
This message will be only be displayed once, unless the ROCBLAS_VERBOSE_TENSILE_ERROR environment variable is set.

To Reproduce
The issue can be repoduced with rocblas-bench:
rocblas-bench -f gemm -r h -m 256 -n 256 -k 64 --transposeA N --transposeB N
Expected behavior
rocblas_hgemm should work for any valid input
Environment

Hardware
description

CPU
MI100

GPU
EPYC 7402P

Software
version

rocm-core
v6.2.1

rocblas
Self compiled @rocm-6.2.2 tag

Hi @IMbackK. Internal ticket has been created to investigate this issue. Thanks!

i just compiled rocblas with tensile as of 013a4b7cbe31146925f55c62149190c45a561640 and this issue seams to have been fixed in the git version.

I am getting this when building Tensile in rocBLAS for gfx1030 on Arch Linux, while it works when building the same source tree with the same CMake parameters from a ROCm Docker.
 SCLK 2660 MHz, max. MCLK 1000 MHz, memoryBusWidth 32 Bytes, compute capability 10.3
maxGridDimX<PHONE_NUMBER>, sharedMemPerBlock 65.5 KB, maxThreadsPerBlock 1024, warpSize 32
-------------------------------------------------------------------------------

rocBLAS info: Time taken to complete rocBLAS library initialization is 438 milliseconds.

rocBLAS info: maximum library size per device is 0.392167 GB.

rocBLAS error: Tensile solution found, but exception thrown for { a_type: "f16_r", b_type: "f16_r", c_type: "f16_r", d_type: "f16_r", compute_type: "f16_r", transA: 'T', transB: 'N', M: 128, N: 2, K: 32, alpha: 1, row_stride_a: 1, col_stride_a: 4096, row_stride_b: 1, col_stride_b: 32, row_stride_c: 1, col_stride_c: 128, row_stride_d: 1, col_stride_d: 128, beta: 0, batch_count: 40, strided_batch: true, stride_a: 524288, stride_b: 64, stride_c: 256, stride_d: 256, atomics_mode: atomics_allowed }
Alpha value 7.21875 doesn't match that set in problem: 1
This message will be only be displayed once, unless the ROCBLAS_VERBOSE_TENSILE_ERROR environment variable is set.
rocBLAS status error: Expected rocblas_status_success, received rocblas_status_internal_error

More details:
https://gitlab.archlinux.org/archlinux/packaging/packages/rocblas/-/issues/2
This piece of code shows the actual problem:
_Float16 one_f16 = 1.0;
_Float16 alpha_f16 = typedInputs.alpha;
printf("one_f16  : %f\n", static_cast<double>(one_f16));
printf("alpha_f16: %f\n", static_cast<double>(alpha_f16));

uint8_t* one_bytes = reinterpret_cast<uint8_t*>(&one_f16);
uint8_t* alpha_bytes = reinterpret_cast<uint8_t*>(&alpha_f16);
printf("one_bytes  : %02x %02x\n", one_bytes[0], one_bytes[1]);
printf("alpha_bytes: %02x %02x\n", alpha_bytes[0], alpha_bytes[1]);

printf("one_f16 equals alpha_f16: %d\n", one_f16 == alpha_f16);

// This only builds inside Tensile
std::stringstream inputValue;
inputValue << typedInputs.alpha;
printf("stringstream alpha: %s\n", inputValue.str().c_str());

Output Tensile Docker (in ContractionSolution.cpp)
one_f16  : 1.000000
alpha_f16: 1.000000
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 1
stringstream alpha: 1

Output rocblas Arch (in tensile_host.cpp)
one_f16  : 1.000000
alpha_f16: 1.000000
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 1

Output Tensile Arch (in ContractionSolution.cpp)
This is the error.
one_f16  : 1.000000
alpha_f16: 2.406250
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 0
stringstream alpha: 7.21875

The incorrect value of the alpha_f16 print changes every invocation in contrast to stringstream, which gives us the incorrect value from the error messages, 7.21875.

I am still getting this when building Tensile 4.43.0 (013a4b7cbe31146925f55c62149190c45a561640) in rocBLAS rocm-6.2.1 for gfx1030 on Arch Linux, while it works when building the same source tree with the same CMake parameters from a ROCm Docker (rocm/pytorch:rocm6.2.2_ubuntu22.04_py3.10_pytorch_release_2.1.2).
 SCLK 2660 MHz, max. MCLK 1000 MHz, memoryBusWidth 32 Bytes, compute capability 10.3
maxGridDimX<PHONE_NUMBER>, sharedMemPerBlock 65.5 KB, maxThreadsPerBlock 1024, warpSize 32
-------------------------------------------------------------------------------

rocBLAS info: Time taken to complete rocBLAS library initialization is 477 milliseconds.

rocBLAS info: maximum library size per device is 0.392167 GB.

rocBLAS error: Tensile solution found, but exception thrown for { a_type: "f16_r", b_type: "f16_r", c_type: "f16_r", d_type: "f16_r", compute_type: "f16_r", transA: 'T', transB: 'N', M: 128, N: 2, K: 32, alpha: 1, row_stride_a: 1, col_stride_a: 4096, row_stride_b: 1, col_stride_b: 32, row_stride_c: 1, col_stride_c: 128, row_stride_d: 1, col_stride_d: 128, beta: 0, batch_count: 40, strided_batch: true, stride_a: 524288, stride_b: 64, stride_c: 256, stride_d: 256, atomics_mode: atomics_allowed }
Alpha value 7.21875 doesn't match that set in problem: 1
This message will be only be displayed once, unless the ROCBLAS_VERBOSE_TENSILE_ERROR environment variable is set.
rocBLAS status error: Expected rocblas_status_success, received rocblas_status_internal_error

More details:
https://gitlab.archlinux.org/archlinux/packaging/packages/rocblas/-/issues/2
The below test results were done with the Tensile version shipped with rocBLAS 6.2.1.
This piece of code shows the actual problem:
_Float16 one_f16 = 1.0;
_Float16 alpha_f16 = typedInputs.alpha;
printf("one_f16  : %f\n", static_cast<double>(one_f16));
printf("alpha_f16: %f\n", static_cast<double>(alpha_f16));

uint8_t* one_bytes = reinterpret_cast<uint8_t*>(&one_f16);
uint8_t* alpha_bytes = reinterpret_cast<uint8_t*>(&alpha_f16);
printf("one_bytes  : %02x %02x\n", one_bytes[0], one_bytes[1]);
printf("alpha_bytes: %02x %02x\n", alpha_bytes[0], alpha_bytes[1]);

printf("one_f16 equals alpha_f16: %d\n", one_f16 == alpha_f16);

// This only builds inside Tensile
std::stringstream inputValue;
inputValue << typedInputs.alpha;
printf("stringstream alpha: %s\n", inputValue.str().c_str());

Output Tensile Docker (in ContractionSolution.cpp)
one_f16  : 1.000000
alpha_f16: 1.000000
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 1
stringstream alpha: 1

Output rocblas Arch Linux (in tensile_host.cpp)
one_f16  : 1.000000
alpha_f16: 1.000000
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 1

Output Tensile Arch Linux (in ContractionSolution.cpp)
This is the error.
one_f16  : 1.000000
alpha_f16: 2.406250
one_bytes  : 00 3c
alpha_bytes: 00 3c
one_f16 equals alpha_f16: 0
stringstream alpha: 7.21875

The incorrect value of the alpha_f16 print changes every invocation in contrast to stringstream, which gives us the incorrect value from the error messages, 7.21875.
So while still in rocBLAS the _Float16 is interpreted correctly, but after arriving in Tensile it's garbage, even though it has the same byte value.

Can you try --compute_type f32_r? It is more accurate and probably more efficient.

@babakpst The same rocblas-bench invocation does succeed using --compute_type f32_r. The issue though is that this won't help with the compatibility of llama.cpp or ollama, since they try to use f16_r.
See
https://github.com/ggerganov/llama.cpp/issues/10234
https://github.com/ollama/ollama/issues/7564

Your results are quite strange
I configured rocblas with:
cmake
    -B build
    -S .
    -D CMAKE_BUILD_TYPE=None
    -D CMAKE_CXX_COMPILER=/opt/rocm/bin/hipcc
    -D CMAKE_TOOLCHAIN_FILE=toolchain-linux.cmake
    -D CMAKE_CXX_FLAGS="${CXXFLAGS} -fcf-protection=none"
    -D CMAKE_INSTALL_PREFIX=/opt/rocm
    -D CMAKE_PREFIX_PATH=/opt/rocm/llvm/lib/cmake/llvm
    -D amd_comgr_DIR=/opt/rocm/lib/cmake/amd_comgr
    -D BUILD_WITH_TENSILE=ON
    -D AMDGPU_TARGETS="gfx908:xnack-;gfx1030"
    -D Tensile_LIBRARY_FORMAT=msgpack
    -D Tensile_TEST_LOCAL_PATH="$srcdir/$_tensile_dir"

on archlinux using tensile 013a4b7cbe31146925f55c62149190c45a561640 and rocblas a203ccb76e3e15c47e9617d7761e8c0f9081e6f5 on arch linux and this works fine here, but i will reopen the bug for your sake for now.

Hello @lubosz, thank you for reaching out. It appears that this issue may be specific to Arch Linux. Unfortunately, Arch Linux is not included in the official ROCm compatibility matrix, and the "official" ROCm package on Arch Linux is not maintained by AMD, so we can't guarantee its functionality. You might consider submitting this issue to the Arch Linux community or trying the ROCm Docker image as an alternative.

@lubosz wanna give rocm-llvm 6.2.4 in [extra-testing] a spin?

@IMbackK I did use the same with the only difference setting AMDGPU_TARGETS="gfx1030" and using the internal Tensile of rocblas and I was able to reproduce the issue. I have done this in the PKGBUILD, so it should in theory produce a working build for you? https://gitlab.archlinux.org/lubosz/rocblas/-/tree/update-tensile

No it needs to be git rocblas AND tensile for it to work
also to be clear @taylding-amd  i can repoduce this issue on ubuntu when self compileing the 6.2.4 rocblas/tensile tag so its not related to arch linux persay but rather its a real bug in rocblas/tensile that depends on compiletime paramters.

@svenstaro Yes I have tried rocm-llvm 6.2.4. I have the assumption that the problem does not lie in the PKGBUILD, since changing the tag to rocm-6.1.0 fixes rocm-llvm. More details on the Arch issue.
@IMbackK Didn't consider rocblas git! that! Will give it a try.

@IMbackK I could reproduce your results! I am able to produce a working ROCblas at https://github.com/ROCm/rocBLAS/commit/a203ccb76e3e15c47e9617d7761e8c0f9081e6f5 with Tensile git, even though using the latest rocm-llvm from Arch!
Although building rocBLAS git did require a couple of changes for me.
They were most related to Tensile, the same Tensile tag that worked with rocblas-6.2.2... seems that it changed how rocblas is providing the CMake config.

I now also need to set CMAKE_C_COMPILER, otherwise the Tensile build will error out.
I now need to add /opt/rocm/lib/llvm/bin/ to my PATH, since Tensile tried to resolve the compiler without an absolute path
I needed to set -D BUILD_WITH_HIPBLASLT=OFF, otherwise Tensile wanted a more recent hipblaslt than available in Arch, which wasn't a dependency before.
I needed to add -I/opt/rocm/include to the compiler flags in TensileCreateLibrary.py

Anyway, doing a hopefully easy bisect now and will backport a fix to the rocblas-6.2.2 package.

Since this issue is now doubly confirmed to be fixed in rocm 6.3 (or whatever the next version will be called) im closeing it.

FWIW this was a bug in rocblas all along.
I was able to bisect the fixing commit on rocBLAS git:
c6bc09073959a2881a701b88ae1ed9de469354f1 is the first new commit
commit c6bc09073959a2881a701b88ae1ed9de469354f1
Author: amcamd<EMAIL_ADDRESS>Date:   Mon Apr 29 15:06:12 2024 -0500

    remove mf16c flag as f16 intrinsics _cvtss_sh, _cvtsh_ss no longer used

 clients/benchmarks/CMakeLists.txt | 8 --------
 clients/gtest/CMakeLists.txt      | 4 ----
 clients/samples/CMakeLists.txt    | 2 --
 library/CMakeLists.txt            | 5 +----
 4 files changed, 1 insertion(+), 18 deletions(-)

https://github.com/ROCm/rocBLAS/commit/c6bc09073959a2881a701b88ae1ed9de469354f1
This commit removes the -mf16c build flag from several targets, including Tensile. This fixes the issue.
It works with several Tensile versions and is independent of the Tensile source. So it also works with Tensile at the rocm-6.2.2 tag.

hmm that reaks of a compiler bug, maybe we should file against llvm

@lubosz great work! Wanna make an MR against the Arch package with the backport?

@svenstaro Opened the MR before I posted my results here ;)
https://gitlab.archlinux.org/archlinux/packaging/packages/rocblas/-/merge_requests/2


Test names (from specs2) are incorrectly displayed in the runner

the :: part should be trimmed.

Hi there! We're doing a clean up of old issues and will be closing this one. Please reopen if you’d like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.


Right click event is triggered for the bottom left corner of  'Settings' button

Issue Type: Bug
Rpeo Steps:

Open VS Code.
Right click the bottom left corner of  'Settings' button.

Expect:
Nothing happened.
Actual:
Right click event is triggered.
More info:
There should display a tooltip for the 'Settings' button.
VS Code version: Code - Insiders 1.24.0-insider (d9299d952a59cb93a399eaf5bd38eb52e25fbdd4, 2018-05-16T05:16:02.701Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (4 x 2600)

GPU Status
2d_canvas: unavailable_softwareflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: unavailable_offnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarevpx_decode: unavailable_softwarewebgl: unavailable_offwebgl2: unavailable_off

Memory (System)
8.00GB (3.94GB free)

Process Argv
C:\Program Files\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-cosmosdb
ms-
0.7.2-alpha

azure-account
ms-
0.4.0

I like this behaviour as you can right-click, keep the mouse button down, go over an entry and release to trigger.


train_molnet.py not running

This program can't run because there is no longer a molnet.molnet_config folder.

Thank you for report.
Do you install master branch?
git clone https://github.com/pfnet-research/chainer-chemistry.git
pip install -e chainer-chemistry


fix: typo

Types of changes
What kind of change does this PR introduce? (check at least one)

[ ] Breaking change
[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactoring
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[x] Other... Please describe:

Description

fix: typo (lavel => label) [10d898da6b3e71e58ae55986023ba9c119145437]
fix: typo (separete => separate) [8f39d29a9a8cfb72126462320e939a92e2d741c1]
fix: typo (dammy => dummy) [bafadcbeea0d1bb62a30538ae07ead62a75e7e29]

Issue Number: N/A

@technote-space
Thanks for the fix.
I have two requests.

Run yarn snapshot command locally
Change the branch to develop

@technote-space
Thanks for the great contribution.


Update http4s-client, http4s-dsl, ... to 0.21.0-RC3

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-okhttp-client

from 0.21.0-RC1 to 0.21.0-RC3.
Release Notes/Changelog
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.
Have a fantastic day writing Scala!

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

labels: library-update, semver-pre-release

Codecov Report

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

@@         Coverage Diff          @@
##           master   #60   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files          24    24
  Lines         229   229
  Branches        8     8
====================================
  Misses        229   229

Continue to review full report at Codecov.

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


As a player, I want to be able to adjust the settings: volume, difficulty

This was done instead by @VegetableBirdB and @ZathrasXI


(BeCyIconGrabber) (GH-1384) add start menu shortcut

Description
This adds a start menu icon/shortcut and a package parameter to not add the icon. This nesitated documentation of the parameter, and addition of an uninstallation script to remove the icon.
The Icon is installed to the current user's start menu, not to all users start menu. This is because the current user start menu does not need administrator permissions to write to, and therefore allows the package to still be portable and installable by a non-adminitrator.
Motivation and Context

See #1384
How Has this Been Tested?

Tested in chocolatey test environment, both install and uninstall.
Tested on win10 machine in a non-administrative cmd.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

Thank you for your interest in improving the becyicongrabber package.
Unfortunately it is not possible for us to accept this pull request at this time, as the website for the software no longer exist and thus we can not update the package either.
If that changes anytime in the future, we can certainly merge this PR and update the package, but until that time comes I will unfortunately be closing this PR


error LNK2019: imGui::

Hello.
I try make project with Visual Studio 2019 and I got a lot errors  error "LNK2019: ImGui::".

Did you clone the repository recursively?
If not use the following command to do so:
git clone --recursive https://github.com/diharaw/SkyModels.git

Did you clone the repository recursively?
If not use the following command to do so:
git clone --recursive https://github.com/diharaw/SkyModels.git

Thank you, it worked.
But you missed file "teapot_smooth.obj", it is not in the folder "Data". I had to find it on the Internet on my own.


Bump dependencies

Modify the associated code based on the crate's versioning of dependencies.

Codecov Report
Attention: Patch coverage is 68.00000% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 48.17%. Comparing base (33d3bad) to head (5b7d834).

Files
Patch %
Lines

src/graphql/status.rs
42.85%
4 Missing :warning:

src/storage.rs
75.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #718      +/-   ##
==========================================
+ Coverage   48.15%   48.17%   +0.02%
==========================================
  Files          29       29
  Lines        8548     8552       +4
==========================================
+ Hits         4116     4120       +4
  Misses       4432     4432

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


Multinomial sampling of NUTS trajectories.

Summary:
Change how NUTS samples from each dynamically generated trajectory.
Description:
Right now NUTS uses a slice sampler to select each point which is a bit ungainly.  Sampling points multinomially using their Metropolis probabilities will streamline the code and make it easier to experiment with algorithmic generalizations.
Current Version:
v2.9.0

Along the entire trajectory including the original point?  And I take it this is the one that's easy to Rao-Blackwellize rather than multinomial sample.  Any a priori theory on how this will compare to the slice sampling?  I don't imagine the normalization for the multinomial is going to be too compute intensive compared to everything else going on.

Along the entire trajectory including the original point?

Yes, including the original point is important for avoiding periodicity.

And I take it this is the one that's easy to Rao-Blackwellize rather than multinomial sample.

That is part of the motivation, although we are very far away from anything close to an implementation.

Any a priori theory on how this will compare to the slice sampling?

Marginally (over the slice level or multinomial sample) they are the same so they shouldn’t yield
appreciable difference.  Having all of the probabilities at hand is technically more information
but we’re not using that yet.

I don't imagine the normalization for the multinomial is going to be too compute intensive compared to everything else going on.

No, it’s negligible.

OK, I think you should be in charge of what we do for sampling.
So I'm OK with this change.  I still need to review the code, though.

On Apr 7, 2016, at 6:14 PM, Michael Betancourt<EMAIL_ADDRESS>wrote:

Along the entire trajectory including the original point?

Yes, including the original point is important for avoiding periodicity.

And I take it this is the one that's easy to Rao-Blackwellize rather than multinomial sample.

That is part of the motivation, although we are very far away from anything close to an implementation.

Any a priori theory on how this will compare to the slice sampling?

Marginally (over the slice level or multinomial sample) they are the same so they shouldn’t yield
appreciable difference. Having all of the probabilities at hand is technically more information
but we’re not using that yet.

I don't imagine the normalization for the multinomial is going to be too compute intensive compared to everything else going on.

No, it’s negligible.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub


undefined is not an object ('evaluating 'this.state.trackInfoArray[0].path')

:beetle: Description

:beetle: What have you tried?

:beetle: Please post your code:
componentWillMount() {
    MusicFiles.getAll({
      id: true, // get id
      artist: true, // get artist
      duration: true, // get duration
      genre: true, // get genre
      title: true, // get title
      fileName: true, // get file name
      minimumSongDuration: 1000,// get track has min duration is 1000 ms (or 1s)
      path: true
    }).then(tracks => {
      let trackInfoArray = [];
      for (let i = 0; i < tracks.length; i++) {
        if (tracks[i]) {
          let trackInfo = new Trackinfo(tracks[i]);
          trackInfoArray.push(trackInfo);
        }
      }
      this.setState({ trackInfoArray });
      ToastAndroid.show(`There are ${trackInfoArray.length} tracks`, ToastAndroid.SHORT);
    }).catch(error => {
      ToastAndroid.show(`${error.message}`, ToastAndroid.SHORT);
    });

 var song = new Sound(this.state.trackInfoArray[0].path,'',
  (error) => console.log(song.isLoaded()));
 if (song.isLoaded())
  { song.setVolume(1);
    song.play((success) => { if (success)
      { console.log('successfully finished playing'); }
      else { console.log('playback failed due to audio decoding errors'); }
    });
  }

:bulb: Possible solution

Is your issue with...

[ ] iOS
[x] Android
[ ] Windows

Are you using...

[x] React Native CLI (e.g. react-native run-android)
[ ] Expo
[ ] Other: (please specify)

Which versions are you using?

React Native Sound:11
React Native:  60.5
iOS:
Android:
Windows:

Does the problem occur on...

[ ] Simulator
[x] Device

If your problem is happening on a device, which device?

Device:

please help me

You'll see that this.state.trackInfoArray[0].path is in your React code, not in react-native-sound. It's out of scope of what can be done here. I'd suggest posting on Stack Overflow.

okay i do thank you


Could you add an example with typescript support please?

Question
I'm using this example but I can't figure out how to make it to support typescript, memoize is the one that I can't get to work
https://github.com/react-native-community/react-native-localize/blob/master/example/src/SyncExample.js

@decim92 Hi 👋 Where do you struggle? If it's only to make memoize keeping the initial method signature, do:
yarn add i18n-js @types/i18n-js lodash.memoize @types/lodash.memoize

import * as RNLocalize from "react-native-localize";
import i18n from "i18n-js";
import memoize from "lodash.memoize";
import { I18nManager } from "react-native";

const translations: { [key: string]: () => any } = {
  en: () => require("../translations/en.json"),
  fr: () => require("../translations/fr.json"),
};

type MemoizeResolver<F extends (...args: any) => any> = (...args: Parameters<F>) => string;

const resolver: MemoizeResolver<typeof i18n.t> = (scope, options) =>
  JSON.stringify({ scope, options });

export const t = memoize<typeof i18n.t>(i18n.t, resolver);

const setI18nConfig = () => {
  const fallback = { languageTag: "en", isRTL: false };
  const { languageTag, isRTL } =
    RNLocalize.findBestAvailableLanguage(Object.keys(translations)) || fallback;

  t.cache.clear?.();
  I18nManager.forceRTL(isRTL);

  i18n.translations = { [languageTag]: translations[languageTag]() };
  i18n.locale = languageTag;
};

Thank you, I'm new to Typescript and this is of great help


chore: add image embedding test

Added this silly test to learn how to embed pngs in the svgs this library produces :)

Thanks a lot max 👍


Bug fix when scrolling to the last row, it used to jump to the bottom ri...

Bug fix when scrolling to the last row, it used to jump to the bottom right. Also if you had views exactly equal to an even fraction of the view then it wouldn’t scroll.

Thanks for your contribution!


Mismatched error message for Font Size of an Element

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Browser and Text zoom should be at 100%.

**
I kept trying different zooms and browsers and code order and got the same message. If I change the second p tag to
Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.
 the code passes.
It is confusing that the error message is about font size when the real problem is the color of the text.

It appears that the last test is designed to test the new paragraph you added and assumes that there is no class added to it i.e.
...
. This challenge could be modified to either give instructions to make both paragraphs red text or specify that the new paragraph shouldn't have any classes added to it.

The instructions do mention this

Also, please do not add any extra classes to your new element.

The test text could use an update to better reflect the updated test, and perhaps the word "extra" could be omitted from the mentioned section of the instructions to prevent confusion.

I think removing the word "extra" and putting a break between two messages in Note would be enough.
It's also possible to split the last test case into two test cases, one asserting that no classes are attached with test p element and the other to assert the font size is as desired.

Just to clarify, the issue is not with the instructions for the exercise. The problem is the error message gives the error as the font-size instead of the class on the second p tag.
Instead of the font-size message being highlighted as wrong, the second message should be marked wrong with some text like "Make sure each of your p elements has a closing tag and the correct classes".

I think it probably makes sense to have two separate tests.  There needs to be a test that ensures that there exists at least one p element with no classes, prior to the one that checks the size, since that check is dependent on it not having a class.

hi everyone
im a newbie on here
i had probs with this question. my code as follows

i changed to explorer fm chrome and got green light no problem
hope this helps

currently i am refreshing the page and  flicking fm chrome to explorer as i keep getting

erroneously

@SaintPeter, yeah, I propose two changes with respect two this issue:

Removing "extra" from note description.
Splitting last test case into two test cases, one for checking whether the p tag has any classes and other checking for font size.

@ankitaggarwal011 Sounds good.  Go for it!

@SaintPeter, opened a PR #6864. Have a look. Thanks!

@francisfrances thanks for sharing what you did about changing from Chrome to IE to be able to fix that issue, it did help.


Fixes npm link of the eslint-plugin-check-file plugin

Summary
This PR resolves a broken link in the README.
Background
I recently started working on my own Next.js boilerplate and found your project incredibly inspiring. Thanks for sharing your fantastic work, CJ! 💪

Thanks!


Update chart victoria-metrics-k8s-stack to 0.27.3

This PR contains the following updates:

Package
Update
Change

victoria-metrics-k8s-stack
patch
0.27.2 -> 0.27.3

Release Notes

VictoriaMetrics/helm-charts (victoria-metrics-k8s-stack)
v0.27.3
Compare Source
Release notes for version 0.27.3
Release date: 2024-10-11

Grafana chart: 8.4.9 -> 8.5.2
Prometheus operator chart: 11.0 -> 15.0
Human-readable error about Helm version requirement
Updated rules template context structure

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


[FEATURE] Supporting Matrix Notation

Hi @Kolaru ,
I was tinkering with MathTeXEngine and was trying to render a matrix.
Using LaTeXStrings.jl, I wrote the following snippet:
latex_string = L"""
$\begin{matrix}
1 & 2 & 3\\
a & b & c
\end{matrix}$
"""

And when I tried rendering it like so:
generate_tex_elements(latex_string)

I got the following error:
ERROR: TeXParseError: unexpected error while parsing
at position 7 (string index 7)
\begin{matrix}
1 & 2 & 3\\
a & b & c
\end{matrix}
      ^
Stack before
[1] TeXExpr :expr

Stacktrace:
  [1] texparse(data::String; showdebug::Bool)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:285
  [2] texparse
    @ ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:271 [inlined]
  [3] _broadcast_getindex_evalf
    @ ./broadcast.jl:670 [inlined]
  [4] _broadcast_getindex
    @ ./broadcast.jl:643 [inlined]
  [5] _getindex
    @ ./broadcast.jl:666 [inlined]
  [6] _broadcast_getindex
    @ ./broadcast.jl:642 [inlined]
  [7] getindex
    @ ./broadcast.jl:597 [inlined]
  [8] copy
    @ ./broadcast.jl:899 [inlined]
  [9] materialize
    @ ./broadcast.jl:860 [inlined]
 [10] generate_tex_elements(str::LaTeXString, font_family::MathTeXEngine.FontFamily)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/engine/layout.jl:295
 [11] generate_tex_elements(str::LaTeXString)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/engine/layout.jl:289
 [12] top-level scope
    @ REPL[12]:1

caused by: TeXParseError: unsupported command \begin
at position 7 (string index 7)
\begin{matrix}
1 & 2 & 3\\
a & b & c
\end{matrix}
      ^
Stack before
[1] TeXExpr :expr

Stacktrace:
  [1] _end_command_builder!(stack::DataStructures.Stack{Any}, p::Int64, data::String)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:233
  [2] macro expansion
    @ ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:256 [inlined]
  [3] macro expansion
    @ ~/.julia/packages/Automa/1KOLQ/src/codegen.jl:149 [inlined]
  [4] texparse(data::String; showdebug::Bool)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:283
  [5] texparse
    @ ~/.julia/packages/MathTeXEngine/zH12o/src/parser/parser.jl:271 [inlined]
  [6] _broadcast_getindex_evalf
    @ ./broadcast.jl:670 [inlined]
  [7] _broadcast_getindex
    @ ./broadcast.jl:643 [inlined]
  [8] _getindex
    @ ./broadcast.jl:666 [inlined]
  [9] _broadcast_getindex
    @ ./broadcast.jl:642 [inlined]
 [10] getindex
    @ ./broadcast.jl:597 [inlined]
 [11] copy
    @ ./broadcast.jl:899 [inlined]
 [12] materialize
    @ ./broadcast.jl:860 [inlined]
 [13] generate_tex_elements(str::LaTeXString, font_family::MathTeXEngine.FontFamily)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/engine/layout.jl:295
 [14] generate_tex_elements(str::LaTeXString)
    @ MathTeXEngine ~/.julia/packages/MathTeXEngine/zH12o/src/engine/layout.jl:289
 [15] top-level scope
    @ REPL[12]:1

I saw that this issue was somewhat similar to supporting \begin{align} statements in #25 .
So, out of curiosity, how difficult would it be to create support for rendering matrices in MathTeXEngine?
Is this something that is out of the purview of the engine or would it be doable?

@davibarreira following up from here: https://github.com/JuliaAnimators/Javis.jl/issues/448
How difficult do you think this would be to implement as well?
I don't really know where to look in the engine right now to get started, but I am assuming it may be something similar to how fractions are written using the engine?
Not too sure at this point.
Have any thoughts?

The main problem I can see is to correctly parse environnments, which is simply not supported currently.
I will need a bit of time to thing about it and how to properly represent the parsed information.
Then the layouting of the elements should not be too hard.
Overall it is definitely doable, I'll have a go at it (and other issues) at the end of the week.

I don't really know where to look in the engine right now to get started

If you want to have a go directly and need some help, don't hesitate to bother me here or on slack ;)
Generally speaking there are always two parts:

The parser (the hard one) : currently a poorly documented marvel of black-magic using Automa.jl. It gets the useful information out of latex string to be passed to the engine proper.
The engine (the fun one) : here most likely all you need is in this function https://github.com/Kolaru/MathTeXEngine.jl/blob/87729520b70a0f0cedceee70375d23080cf65cf8/src/engine/layout.jl#L20 that simply has a clause  for every possible supported construct and builds them recursively.

The good part about the second one is that it is pretty much pure geometry at this point so if you come up with the algorithm to layout the matrix elements in Javis (based on a reasonnable data structure for the elements, e.g. a julia matrix of latex elements), it should be easy to adapt it and include it in MathTeXEngine.jl.

@Kolaru this is a great explanation.
I took a look at Automa.jl - how fascinating!
It seems to use a nifty set of rules to parse out text.
Am I thinking correctly, you would use this approach to parse out the LaTeX to get each element of matrix?
I see what you mean as from there, putting each element into a corresponding Julia matrix would be rather straightforward.
I'll see if I can experiment with this on Thursday.
Thanks for the pointers - much appreciated.
Let me know if you have more thoughts or ideas as well, but I think experimenting with Automa.jl is going to be where I'll start!

Am I thinking correctly, you would use this approach to parse out the LaTeX to get each element of matrix?

You are, but there are of course some more tricks involved.
Since I had some time in the train and you look motivated to look at the parser, I just added a short dev doc for the parser here: https://github.com/Kolaru/MathTeXEngine.jl/tree/master/src/parser

This is brilliant!
Thank you @Kolaru - yes, I am feeling motivated so let's see what progress I can get done here. :)

Hey @Kolaru !
I had a look through your dev doc that you prepared.
That was perfect and I loved the final line: :slightly_smiling_face:

You can watch the rise and fall of the stack by passing showdebug=true to texparse. It is currently not as fun as to watch an old empire rise and fall, but beware, it is nearly as verbose.

I am now switching over to looking through Automa.jl so we will see what I learn there.
I did look at parser.jl and based on what I saw, it seems we need to first address the problem of LaTeX environment parsing as mentioned in #25 .
So, based on the parser logic, it seems like the following problems are present:

[ ] Recognize a command by \ presence
[ ] Perform a lookahead to disambiguate between command and environment (looks like the logic may be similar to things like \vec{v}vector LaTeX syntax
[ ] Find group { and } which again seems similar to vec parsing
[ ] Find the end of the environment

Some of these look either solved or partially solved, but I do not know.
So it feels like a first step is to address the problem of environment recognition then, we can move specifically to types of environments present.
What do you think?

Yes you mostly got it. The way I would go about it is

Register the \begin command as a 1 argument command i.e. with canonical form (:argument_gatherer, [:environnement, 1]). Everything is already implemented there, so once \begin{envname} is read, there will be (:environnement, ["envname"]) on top of the stack. (*)
While an :environnement expr is on top of the stack adapt the parsing and push the data to it to get something like (:environnement, ["envname", data...]). This is where most of the work is needed I believe.
Register the \end command similar to the \begin command, with e.g. (:argument_gatherer, [:endenv, 1]).
Add a special rule in the parser such that when an :endenv is found it terminates the :environnement currently on top of the stack. If the expression on top of the stack is not an environnement, or if its name does not match the one of the \end command, raise a parsing error. It means that there is either a dangling \end{envname} without the corresponding \begin or another unfinished construct inside the \begin...\end block.
Find out all the problems you only see when you actually implement the stuff :D

I think that should be pretty much it.
(*) This is what we do for example for \vec which is registered here (\vec is part of the combining_accents family): https://github.com/Kolaru/MathTeXEngine.jl/blob/2dd00d84ea29eb3952355958b1489c0b41b3cff7/src/parser/commands_registration.jl#L115-L119


⚠ Bofrost.be has degraded performance

In a688b2f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2380 ms

Resolved: Bofrost.be performance has improved in 7dae0eb.


adding support for datetime numpy arrays

While pd.apply() works for small datasets like the example from the docs
df['ADJUSTED'] = df.apply(lambda x: cpi.inflate(x.MEDIAN_HOUSEHOLD_INCOME, x.YEAR), axis=1)

it quickly falls apart if one tries to inflate long series because it inflates each value one at a time instead of taking advantage of numpy and pandas vectorization.
CPI already can handle numpy arrays and has both pandas and numpy as dependencies.

(100,000,000 rows in less than 2 seconds, pretty cool.)
The problem:
CPI takes year_or_month as either int or a date object and retrieves the corresponding source_index from cpi.db. This, as far as I understand, would need to be done for every item in the array therefore it would still be very time-consuming for very large datasets.
The solution:
I still don't have any solid solutions.
One way to approach this could be:

receive a numpy array of dates for year_or_month

clean it so they all have 01 as day of month

grab the unique values in this array of dates

even if you have 100,000,000 rows, you definitely don't have 100,000,000 different year-month combinations.

BLS' data goes back to 1913 (2017-1913=104 years, 104 * 12 = 1248 months + 10 months of 2018 as of now = 1258 unique values at most)

create a numpy array of those values matching their date (or a dict() to later use .map() on the dates array.)

map the source_index values to the array of dates

look up the CPI value for each of those unique dates and map it back to the original numpy array of dates

cpi.inflate() already just multiplies (value * target_index) / float(source_index)

numpy will take care of the rest

Even though most likely one would be inflating values to one specific year or month, this method could be applied to both year_or_month and to to inflate a series of values from a series of dates to a different series of dates.
The use:
The particular use I came up with was normalizing different types of incomes from public use microdata. For example, if I go to ipums and grab ACS data from 2000-2016 for incomes (earned wages, household income, farm income, social security, etc).
There are only 16 distinct years but if I use pd.apply() it would go row by row and it would simply never end:

I don't have a experience with sqlite so I couldn't put together a proof of concept but I hope this explanation is helpful.

I think I found a way to implement this method I was talking about but I'm not sure where it'd go in the code. I assume it'd be just another elif in inflate() where if the year_or_month cpi.inflate() receives is a series this would go into effect. Here's a (simple) example notebook I put together: https://github.com/chekos/cpi/blob/master/example.ipynb
The elevator pitch is:
If you receive a series of dates, just grab the unique values, look up the index (source_index), save it as a dict, map the values back to the series. Now grab the to value, if it's a series, do the same. Now you have 3 series (of values, dates to inflate from, dates to inflate to). They are all pandas series or numpy arrays so you can just return (value * target_index) / source_index like you would regularly.


Switched "read" and "write" definitions

read is the same as create and update and delete.
write is the same as get and list.
this didn't make sense, so I assumed it was a typo and flipped the definitions:
write is the same as create and update and delete.
read is the same as get and list.

Thanks for the correction!


Sidebar: Height of sidebar with menu-title overlap

height of sidebar no-bottom-menu with menu-title is overlap and cause scroll


Retrieve all versions of a secret

Describe the solution you'd like
It would be useful to specify objectVersion: "*" and have all versions of that secret retrieved. They could be mounted on the volume either as secret-name/version or secret-name-version.
The driver for this is the .Net Data Protection API. This allows use of certificates to protect the key. In order to provide backwards compatibility when the certificate updates you can pass in previous certs. At present this would have to be done by manually looking up the versions and putting them in the manifest.

Consolidating the feature requests in a single issue: https://github.com/Azure/secrets-store-csi-driver-provider-azure/issues/571


Fix cycle initializing IoTDBDescriptor

See these error logs during starting.

This is a bug caused by #7577
The root cause is that IoTDBDescriptor is a singleton and when the constructor is called, loadProperties() is called. However, there's a call of IoTDBDescriptor.getInstance() in convertMilliTimeWithPrecision(). As the initializing hasn't been finished, the getInstance() just returns null.
DatetimeUtils is a static class, so it's better not to depend on dynamic components, just decouples with IoTDBDescriptor through a method parameter.

@HTHou @HeimingZ PTAL


less modifyVars

Is it possible to use less modifyVars with fuse-box?

Hi, Could you throw some links here please?

Oh sorry, I forgot: http://lesscss.org/usage/#using-less-in-the-browser-modify-variables
Example: https://codepen.io/seven-phases-max/pen/dPYxzR

Gotcha, so the situation is .. that I am now in a complete re-write. I will keep that thing in mind, when moving the less plugin.

Okay, cool, thanks for your answer! What exactly are you re-writting? Do you already know about how long this will take?

@machiaveli88
https://github.com/fuse-box/fuse-box/tree/v4/src
fuse-box ;-)
ETA:
2-4 weeks: First version (without production builds)
2 weeks - production ready

Ah, cool, I'm very excited. :)

Okay, I think It's working already with less options:
LESSPlugin({         modifyVars: {           'color1': 'green'         }       })


Im not too sure if this is RoPro's fault or the patcher but server filters (premium) wont work

All of the server filters in the premium category do not work as intended.
Steps to reproduce the behavior:

Go to any game
Click on the Servers button
Scroll down to the filters and click any of the premium ones
See error

This was working as intended just yesterday

it only works as certain times atm. specifically, when i'm
online. will look into it later

It works now


Add ContextMenuBuilder argument for searching using DuckDuckGo instead of Google

Original PR is here https://github.com/electron-userland/electron-spellchecker/pull/138
And since this fork doesn't have the new electron (4), this actually runs on my machine and seems to work great :D

Fixed directly in patchwork instead https://github.com/ssbc/patchwork/pull/980


Add support for enableWhen with Coding type

This modification should make the enableWhen-Conditions work also with answers of type Coding.
I hope git doesn't screw things up.

Please be careful, looks like the renaming of appId => subjectId got also built into this Pull Request somehow

Okay I commited again, I thing in theory the new state should now be merged?


Make it so that streammenu.py takes in the COM port as the first argument

Right now it uses the config file to choose the right COM port. Right now, make it so that it is an argument of the command line instead.
ex:
python3 streammenu.py COM12

Will be include in v1.0.1


How to set the default value of scope when Authorization Request?

Expected Behavior
https://datatracker.ietf.org/doc/html/draft-ietf-oauth-v2-1-05#section-4.1.1
Current Behavior
scope should have a default value when authorizing
Context
If no scope is set when authorizing the request
Now when i make the authorization request the error occurs

Label


IP_STATUS_FILE and routing improvements

Obtains VPN public IP address from routing table
Logs and writes VPN Public IP address to /ip as soon as VPN is up
Obtain port forward, logs it and writes it as soon as VPN is up
Routing fully refactored and tested
Routing reads from /proc/net/route
Routing mutates the routes using ip route ...

Fix #123


Bug: Truncated summary entries don't work.

When the compact summary doesn't have room to put the entire filename, it ends up being truncated. Git will change part of it to .... When that happens, git_diff_explorer can't find the actual file path in order to show the diff.
I've no idea what an entry would look like if it is a moved file and also truncated like this.

Possible plan of attack.
Try to identify such entries by checking if they contain ... and don't exist.
In such cases, do the following:

Escape the entry for a regex.
Change the ... into .*.
Run the git diff --name-only command.
Grep those with the -x flag (--line-regexp, only matches whole lines) using the modified entry.

If only 1 entry is found, use that as the filename.
If zero entries are found, just go with the original entry.
If two or more entries are found, I'm not sure what to do. I'm tempted to just say to use all of them. I'm not sure how we'd be able to further refine the search.


REQUEST: Adding more than one path for "storage" volume

Would be nice to have a possibility to add more than one "storage" volumes to the container. I have a two "movies" folders placed on different hdds. Right now I need to copy the file which I want to edit to the "storage" volume to get access to it and after work is over I copying the file it to the "movies" destination on hdd a) or b).

This is part of the the container's configuration.  So you can add volume mappings you want using the -v parameter of your docker run command.

Hi, thank you for your reply.
I am deploying using docker-compose file.
My compose file is:
version: '3' services:   mkvtoolnix:     image: jlesage/mkvtoolnix     container_name: mkvtoolnix     ports:       - 5800:5800     environment:       - USER_ID=111       - GROUP_ID=222     volumes:       - /srv/disk1/CONFIG/mkvtoolnix:/config:rw       - /srv/disk2-part1/MOVIES:/storage:rw       #- /srv/disk2-part2/MOVIES:/storage:rw     restart: unless-stopped
In this case only one volume marked as storage can be added. When I am trying to add both error appears that volumes are duplicated - they are not because the path is different (different uuid). The "duplicated" message comes I believe from storage variable/indicator.

You need to map to different folders on the container side.  For example, you could use something like this:
volumes:
  - /srv/disk1/CONFIG/mkvtoolnix:/config:rw
  - /srv/disk2-part1/MOVIES:/storage1:rw
  - /srv/disk2-part2/MOVIES:/storage2:rw

You need to map to different folders on the container side. For example, you could use something like this:
volumes:
  - /srv/disk1/CONFIG/mkvtoolnix:/config:rw
  - /srv/disk2-part1/MOVIES:/storage1:rw
  - /srv/disk2-part2/MOVIES:/storage2:rw

The storage indicator can not be renamed - it is taken in consideration when creating the container. But I found the way I did not know exists - I am not super experienced in Docker. To add more directories to this (all?) container you need to create the unique folder after the main storage folder, eg.:
` - /srv/disk2-part1/MOVIES:/storage/movies1:rw

/srv/disk2-part2/MOVIES:/storage/movies2:rw`
Now it works correctly!

Thanks!

The storage indicator can not be renamed - it is taken in consideration when creating the container

I'm not sure what you mean by that, but note it's perfectly legal to create a container without a mapping to /storage.  And additions mappings are not required to be under /storage.
But anyway, the way you did is also another alternative that works.


HMR does not work when page require output bundles from multiple entries

Bug report

What is the current behavior?
HMR does not work when page require output bundles from multiple entries in webpack 5.
But it works in webpack 4.
If the current behavior is a bug, please provide the steps to reproduce.
https://github.com/toBeTheLight/issue-20211215

Require multiple bundles in page from two entries(single complier or multiple compliers):

// from { first: ['xxxx']} in entry config
<script src="./first.js">
// from { second: ['xxxx']} in entry config
<script src="./second.js">

change second.js
hmr work
change first.js
page request first.hot-update.js,but report an error Uncaught TypeError: Cannot set properties of undefined (setting './src/first.js') and a warning Update failed: ChunkLoadError: Loading hot update chunk first failed

What is the expected behavior?
Hmr of all files work.

Other relevant information:
webpack version:  v5.0.0~latest
I guess, in webpack5, hrm context of first.js has be overwritten by second.js(which is loaded later).
For example，in webpack5：
var currentUpdatedModulesList;
var waitingUpdateResolves = {};
function loadUpdateChunk(chunkId) {
	return new Promise((resolve, reject) => {
		waitingUpdateResolves[chunkId] = resolve;
		var url = __webpack_require__.p + __webpack_require__.hu(chunkId);
		var error = new Error();
		var loadingEnded = (event) => {
      // ......
		};
		__webpack_require__.l(url, loadingEnded);
	});
}

self["webpackHotUpdatewebpack5"] = (chunkId, moreModules, runtime) => {
  // ......
	if(waitingUpdateResolves[chunkId]) {
		waitingUpdateResolves[chunkId]();
		waitingUpdateResolves[chunkId] = undefined;
	}
};

After first.js emit hmr, function loadUpdateChunk run. And then, when self["webpackHotUpdatewebpack5"]  is running, waitingUpdateResolves in self["webpackHotUpdatewebpack5"] is the one be declared by second.js, and it is empty, so update of first.js failed.
When in single complier, using optimization.runtimeChunk can resolve this problem. But it is not available for multiple compliers.
And maybe relate to https://github.com/webpack/webpack/commit/887852f6df9774d52b017c0bd8781466966a64e7 of webpack 4?

Answered https://github.com/webpack/webpack-dev-server/issues/2792#issuecomment-808328432, please set optimization.runtimeChunk, you have multiple runtime on the same page, it is bad

@alexander-akait
Need to set different plugins and  different optimization.splitChunks config for different functional files, we must use multiple compliers.
But setting optimization.runtimeChunk doesn't work for multiple compliers, because __webpack_require__.h are difference from each compliers, although we can set runtime output file to same name.
see https://github.com/toBeTheLight/issue-20211215/blob/master/webpack5/package.json and run npm run compliers

You different unique names for compilers, just add name and output.uniqueName for each compiler https://github.com/toBeTheLight/issue-20211215/blob/master/webpack5/webpack.compliers.config.js#L3, it will generated different runtimes for different compilers

Multi compilers and multi entries was design for different purposes, you can't use multiple entries (from one compiler) on the same  page without optimization.runtimeChunk

And I noticed the warning under runtimeChunk in documentation

Imported modules are initialized for each runtime chunk separately, so if you include multiple entry points on a page, beware of this behavior.

Could I know why can't use multiple entries (from one compiler) on the same page without optimization.runtimeChunk, what would happen in bad case?
We have so many scenes using multiple entries on the same page from one compiler 😅, and they could not split chunks, because the output file may be use in current project and other projects by cdn（something like library）.🤔 Maybe I should split that kind of entries into difference compilers and set different output.uniqueName.

Could I know why can't use multiple entries (from one compiler) on the same page without optimization.runtimeChunk, what would happen in bad case?

Because each entry has own part of runtime code (i.e. chunks/code/logic/ they can have different public path and more), you can get problems (I can describe them, but there are a lot of them), you need to merge runtime and extract runtime in separated chunk, even more it will better for long term cache with hashes for production usage.
Note - you need do it only for configurations where you have multi entries on the same page, if you reuse bundle from other compiler just check you have unique name for runtime, we have output.uniqueName option for this, but by default generate name from package.json (the name field), so if you have monorepo most likely you will not face this problem

Find https://github.com/webpack/webpack/issues/13316, study later, thanks.


⠋  character not rendering

Windows Terminal version
1.7.3651.0 WSL 1
Windows build number
19043.1466
Other Software
Nanospinner
Steps to reproduce
run a program that uses nanospinner.js
the issue lays with in the characters that it uses like ⠋

Expected Behavior
a spinner character
Actual Behavior
it gave the question mark symble

FWIW That looks like conhost, the vintage console host, not the Windows Terminal. Also, 1.7 is almost 5 releases out of date - you might want to consider updating the Terminal 😉
Which font are you using? I don't believe that Consolas has the braille characters in it, and conhost doesn't do font fallback (at least, not the version in Windows 10)

I don't believe that Consolas has the braille characters in it, and conhost doesn't do font fallback (at least, not the version in Windows 10)

Yeah. I'm still on Windows 10 and can confirm that Consolas doesn't have braille, but Cascadia Code does, so switching fonts would be one option.


[GHSA-ghg6-32f9-2jp7] XXE in PHPSpreadsheet encoding is returned

Updates

Affected products

Comments
based on newly updated version of PhpSpreadsheet (1.29.1), https://github.com/PHPOffice/PhpSpreadsheet/security/advisories/GHSA-ghg6-32f9-2jp7

Hi there @oleibman! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

I am completely confused. Does this mean that, in addition to creating a new release and updating the security advisory, something else (i.e. the change that @chinh2597 wants to merge) also needs to happen to distribute the updated information to the "global advisory" list (and to packagist)? If so, I made a similar update to GHSA-wgmf-q9vr-vww6 - does it require this type of change as well?

sorry, this is my first time contribute, i just saw the different between https://github.com/PHPOffice/PhpSpreadsheet/security/advisories/GHSA-ghg6-32f9-2jp7 vs https://github.com/advisories/GHSA-wgmf-q9vr-vww6, and don't know how to make it the same so i create this PR, if this is not needed pls close it
cc @oleibman

@oleibman, you don't need to do anything more to get the global security advisory updated. When you make a change to a public repo advisory, the GitHub team will automatically be notified and we will make any appropriate changes to the global security advisory. @chinh2597 just beat us to it this time.


Feature/shipyard ready registry v2

Added support for registry v2 with authentification

Please check all of the log.Error and log.Debug.  I believe we log the error on return so we are probaly getting double logs with this.

Looks pretty good!  Thanks for opening!

Hi Evan. Thanks for the review. I just pushed the changes you requested. Thanks!

Hey @ehazlett, any progress for this merge?

I'll get this tested.  Thanks!

Hey @ehazlett,
Any news about the testing?
Waiting for your feedbacks.

Hey @ehazlett,
totally loving shipyard, please keep doing your excellent works. The only downside about shipyard I can think of is not supporting Registry v2. Really hope this feature could be merged into the master in the near future. Cheers!

Changes for  auth/ldap/ldap.go breaks ldap {username} substitution, this code should be placed before ldap bind invocation, otherwise return false,
Cheers

Awesome work, guys! Any progress with testing?

Looks good!  What is the "remarks" field in the table?  Also, the size is not being calculated.

Ok after addressing these we should be good to merge.  Functionality looked good; couple of nits for logging to reduce chatter.  I'm not sure why we need the "Remarks" column and we need to get the size calculating.  Other than that, it looks good.  Great job!

+1

@ehazlett the Remarks field is being populated in case we can't get the manifest properly.
We are working on getting the size, we need to get a different manifest with a different Header, hopefully, today it will be ready.

Hey @ehazlett . Thank you for the review. I added the changes so we will display the size for a repository, and the changes for the logging.

Great job @AndreeaDora !

+1
eagerly awaiting this functionality

appears all checks have passed! @ehazlett when will PR get merged?

@AndreeaDora thanks!  i'll take a look.

I'm unable to ping registries now.  I set up a local registry, pushed an image and tried to add it to Shipyard.  I receive "cannot ping".  However, I can curl it from the shipyard controller via an exec:

/ # curl -v registry:5000
* Rebuilt URL to: registry:5000/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to registry (<IP_ADDRESS>) port 5000 (#0)
> GET / HTTP/1.1
> Host: registry:5000
> User-Agent: curl/7.50.3
> Accept: */*
>
< HTTP/1.1 200 OK
< Cache-Control: no-cache
< Date: Wed, 05 Oct 2016 14:03:55 GMT
< Content-Length: 0
< Content-Type: text/plain; charset=utf-8
<
* Curl_http_done: called premature == 0
* Connection #0 to host registry left intact

actually wait -- this may be a DNS issue on my side; checking

ok this was a DNS issue on my part.
Can someone explain the "Remarks" field -- I'm not sure what it's used for.

ah i see above it's for errors.  i think this is fine.  lgtm

Thanks @ehazlett! great work everyone.

awesome!  pulling in AM  thanks to all for the great work
On Wed, Oct 5, 2016 at 11:24 AM, Eitan Schichmanter <
<EMAIL_ADDRESS>wrote:

Thanks @ehazlett https://github.com/ehazlett! great work everyone.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shipyard/shipyard/pull/852#issuecomment-251707842,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOJqa-pTC68-rChB1oXJydBVhx-1cUgsks5qw8FHgaJpZM4JcsNm
.

Hey I just noticed that the button for deleting an image from a registry has been removed. Is there any reason for that function not beeing implemented for registry v2?

Also I had some issues when trying to use shipyard with a v1 registry. It seems shipyard is not able to ping any v1 registry.

@ehazlett Hi. Earlier at https://github.com/shipyard/shipyard/pull/852#issuecomment-251684627 you mention what receive "cannot ping" registry.  Now I try to play with Shipyard and have same situation. But not sure this is related with DNS. In registry container have next logs:
http: TLS handshake error from <IP_ADDRESS>:56324: tls: first record does not look like a TLS handshake
I create local registry:2 with self-signed certificates. Maybe this related with SSL settings?


list_buckets() and list_objects() are returning empty lists from pachyderm using minio client #4187

Hi,
I am opening an existing issue 4187 as i could not see option to reopen after answer
@ysimonson, I have tried using list_buckets with minio library and works ok. I see a similar issue which is opened on minio-library and they have redirected to S3 incompatibility in the server implementation
Dear all,
when using list_buckets() or list_objects() on an account/bucket in pachyderm, the result is an empty list, allthough there are buckets/objects in that account.
s3_client = Minio(endpoint=s3_endpoint,
                  access_key=s3_accesskey,
                  secret_key=s3_secretkey,
                  region=s3_region,
                  secure=s3_secure)
the_bucket = s3_bucket
list_buckets = s3_client.list_buckets()  #Returns empty list
is_bucket = s3_client.bucket_exists(the_bucket)  #Returns True
list_objects = s3_client.list_objects(the_bucket)  #Returns empty generator object
[print(obj.object_name) for obj in list_objects]  #Throws the error below

Environment?:

Kubernetes version (use kubectl version):
kubernetes version v12

Pachyderm CLI and pachd server version (use pachctl version):
pachctl version
COMPONENT           VERSION
pachctl             1.9.2
pachd               1.9.4

Cloud provider (e.g. aws, azure, gke) or local deployment
custom kubernetes on azure

OS (e.g. from /etc/os-release):
Mac OS Sierra

Error on trying to access the objects from existing bucket
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/minio/parsers.py", line 104, in get_child_text
return self.element.find('s3:{}'.format(name), _S3_NS).text
AttributeError: 'NoneType' object has no attribute 'text'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "pachyderm_s3.py", line 75, in <module>
    main()
  File "pachyderm_s3.py", line 60, in main
    for obj in objects:
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/minio/api.py", line 906, in list_objects
    bucket_name=bucket_name)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/minio/parsers.py", line 238, in parse_list_objects
    is_truncated = root.get_child_text('IsTruncated').lower() == 'true'
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/minio/parsers.py", line 108, in get_child_text
    'Message: {}').format(self.root_name, name, error.message)
AttributeError: 'AttributeError' object has no attribute 'message'```

================================================

@chaitanyak52 Can you link us to the minio client issue that references the S3 incompatibility?

@chaitanyak52 checking in on this

Fixed in #4902, will go out with 1.11


two factor authentication domain name

Is it possible to configure the "domain" part or label of the generated QR-Code somewhere ?
Right now it seems to always be "example.net" if I scan the QR-Code.

The native 2FA integration in Zulip is unfinished (that's why it's turned off by default).  #10921 has some more details.
This is in Google authenticator?  It's a good question where that would come from; we don't have the string "example.net" anywhere in our codebase or in that of django-two-factor, the library we use for this.

Yeah, I think so, too.
Must be somewhere near https://github.com/Bouke/django-two-factor-auth/blob/d000074089ff02ee616a0c5f2e6b03d175f3b9ff/two_factor/views/core.py
line 558 I guess, but I am not very good with python and django tbh :-)
We tried to set OTP_HOTP_ISSUER in the settings.py, but that did not help.

I tried a bit around with the "Sites" Framework in the shell and I found:
In [7]: from django.contrib.sites.models import Site
In [8]: Site.objects.get_current().domain
Out[8]: 'collab.oss.ewetel.de'
In [9]: Site.objects.get_current().name
Out[9]: 'example.com'

Ok, found out how to change + save it with the shell. Now the QR-Code is OK :-)
I have no idea where/if the sites Framework is used in zulip though. Would be cool if it would be possible to set this in the settings maybe

Ugh, it uses the Django "Sites" framework.  We don't use that for anything in Zulip (other than, apparently indirectly, this).  I think our near-term fix might be to monkey-patch from django.contrib.sites.shortcuts import get_current_site to return effectively the realm.host value, since that's where to get the correct value for the current realm/subdomain.  Thanks for investigating!

Changing that issuer line to  issuer=request.user.realm.host seems to do the right thing.  So we just need to figure out a good way to make that change without forking social-auth.

Ahh, there's a better solution: If we just remove the Site Django middleware (which we're not using), it'll automatically do the right thing.  So I'll do that.


fix: Handle unhandled error in startResumableUpload_

Added error handling using stream.destroy

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #2494

Thank you for the contribution @cuyl, we appreciate it.


Code scanning alerts for Snyk Code SAST

This workflow tests your application for SAST vulnerabities and then presents them in the Secuirty tab of Github. It provides in-line details of where the vulnerability is found and provides details and guidance to fix.

Confirmed only the container scan Snyk feature works with Jsonnet. No further action


Separate weights for word embedding and lm-head?

Hi, thanks for providing the code.
I have a question regarding the word embedding and lm-head. In your code, both functions shard the same weight. I wonder if they can have separate weights?
Thanks for your help!

Hi,
The lm-head weights (word embedding) are used for decoding. You can add extra weighting parameters before lm-head to build a better decoder.

Hi,
thank you for the reply!
Wish you a happy new year :).


Commands and Output missing the tilde "~" symbols

Minor issue but the PowerShell commands/output are missing the tilde symbols:

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

ID: 18a5a4e5-2604-23c2-1374-68a6974170bc
Version Independent ID: f93809cc-eb5e-6b9b-ed58-c8811a8ee21f
Content: Install OpenSSH
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
Product: windows-server
Technology: administration
GitHub Login: @maertendMSFT
Microsoft Alias: damaerte

Please note: Those tilde characters used to be present in the document article, but were removed on April 8, 2021 (9 days ago as of this comment) in commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/1edf6dd35ae418216f5200dbd9deb0726f466ed6 by @v-dasis .
From a practical point of view, it is slightly unfortunate that the change was added without a public PR explaining the reasoning for this change.
Nevertheless, the change has been made by one of the MS Docs team members and therefore seems to be validated.

I can't say for certain but my guess is they thought the tildes were formatting anomalies and removed them however they are required or else the commands will fail and need to be reinstated.

I can't say for certain but my guess is they thought the tildes were formatting anomalies and removed them however they are required or else the commands will fail and need to be reinstated.

Fair point, I was a little bit uncertain about why the tilde characters were removed. Do you want to make a PR to recommend adding them back, or do you prefer that I do so?

Tildes have been added back - changes go live later this morning. My apologies...

Tildes have been added back - changes go live later this morning. My apologies...

Thank you for confirming the change. Good to know. 👍🏻

@walchst - Thanks for your dedication to our documentation. We sincerely apologize for the delayed response. The requested updates have already been made. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Laser scan data incomplete

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[x] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[x] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[x] Raspberry Pi 4
[ ] etc (Please specify your SBC here)

Which OS you installed on SBC?

[x] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[ ] etc (Please specify your OS here)

Which OS you installed on Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[x] Ubuntu 20.04 LTS (Focal Fossa)
[ ] Windows 10
[ ] MAC OS X (Specify version)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to reproduce the issue.

HERE

Copy and Paste the error messages on terminal.

HERE

Please describe the issue in detail.
I’m using the LiDAR scan data for obstacle avoidance using reinforcement learning. The range of the LiDAR scan is not complete from 0 to 359 degrees. Sometimes the len(msgScan.ranges) = 229 or 230or 232 but never 360 and also why does it fluctuate?
How can this be dealt with ?

Hi @rajalnagwekar
I predict that you are using LDS-02.
Refer below reply, it will help you to understand that situation.
https://github.com/ROBOTIS-GIT/turtlebot3/issues/952#issuecomment-2467139688

@rajalnagwekar Apologies for the delayed response. We have assembled a TF team to address TurtleBot3 issues and are actively. Please refer to @GyuH13 's answer for initial guidance. This issue will be closed due to prolonged inactivity. If the problem persists, feel free to reopen it, and we will address it promptly. Thank you for your understanding


Default bash script with provided git refs

Propose un bash script in order to retrieve metrics from the past :
input :

git url
list of git refs

output :

the report

algo :

foreah git ref

checkout
install and run minimal qa tools
install and run qatracker
generate data series

at the end, generate the final report with all data series

Note :

https://packagist.org/packages/gitonomy/gitlib

it must be another more genric project that could rad a configuration file with a list of git efs and run foreach one a bash script :

foreah git ref

git checkout
run script.sh

end


[Feature Request] Add prop for setting page title

Explanation
A descriptive title for each page is an important accessibility feature (W3). Neither Reach nor React Router handle this natively; it's left to developers to implement, and it is too often ignored. I think Reach, with its focus on accessibility, should address this.
Proposal
I'm proposing a title prop passed to the children of Router that will update the page title when that component is mounted. It would look something like this:
render(
  <Router>
    <Home path="/" title="Example - Example.com" />

 `${articleName} - Example.com`}
    />
  </Router>
)

The title prop would accept either a string or a function that returns a string. A function title will be passed any route params for reference.
If no title is provided, the page title would remain unchanged.
Thoughts?
I know that the intention of this project is to keep the API light, but I think the benefit of adding this feature would outweigh the cost. What do you think?

My opinion is that it's a bit slippery slope because there are so many other tags that should be set and we'd end up reimplementing libraries that already do this. E.g which usually work like this:
return (

    <XYZHeadLibrary
      title={props.articleName}
      desc={props.articleDesc}
      // and other tags
    />

Article content

);

My opinion is that it's a bit slippery slope because there are so many other tags that should be set and we'd end up reimplementing libraries that already do this.

That's a good point that I didn't consider.
However, page title has a particular role in accessibility; I think that would distinguish it from the rest of the head tags.

I agree that it would be a nice addition, however as @SeanMcP mentioned, it can lend itself towards feature-creep if this was handled by the router.
React doesn't disallow dynamically setting the title with native JS, but if wanting to do it in a more component based way, react-helmet has solved for this use case as well.
Gonna close, but the thoughts are much appreciated and thanks for using reach router!


TSA ARMA

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Continues towards #1404
What are the significant changes in functionality due to this change request?
Adds the basic ARMA to TSA algorithm (no VARMA, but including gaussianizing through CDF)

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

@alfoa tagging you in case you want to look at this as well

The ARMAs on all machines are typically within 1e-4, but there's always some random point that's much further off (2e-2 is the largest I found).
I'm not sure what to think of this yet. I want to test the signals themselves, but that's a pretty large margin of error.


CC states considered buffs

I dont know if all cc states work the same, but after prolonging buff duration, if character fails a save against for example grease, he will stay down on the floor for time prolonged by the mod.

My fix went in - wait for next release (anything later than <IP_ADDRESS>) and check if your problem is fixed! only beneficial buffs should now get prolonged

I think invisible CC states (the ones that do not show up next to your portrait) still are prolonged. (prone etc.)

We're still having issues with this related to "buffs" that a player would consider a negative effect that are applied by your character. Examples are failed skillcheck resulting in prone, or fatigue after barbarian rage.

Well then, it seems that the game doesn't mark those as negative... The only other option that comes to mind is to create a manual list of "buffs" that we don't want to prolong...
If anyone knows of a status effect that should not be prolonged other than prone or fatigue put it in this issue


docs: add example for including local catalog files

Hey, I just made a Pull Request!
I made a little update to the docs showing how to add local files and locations with the type: file as it was not really clear without looking at linked examples and the starter.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[X] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Sorry I messed up the branches using the Github sync from Fork Feature :/
I will open a new PR and link it here with a fresh commit.