[qt5-webengine] build failure

Operating system
windows
Compiler
msvc
Steps to reproduce the behavior
vcpkg install qt5-webengine:x64-windows

Failure logs
Computing installation plan...
The following packages will be built and installed:
qt5-webengine[core]:x64-windows -> 5.15.9
Detecting compiler hash for triplet x64-windows...
Restored 0 package(s) from G:\Users\LuLujiang\AppData\Local\vcpkg\archives in 235 us. Use --debug to see more details.
Installing 1/1 qt5-webengine:x64-windows...
Building qt5-webengine[core]:x64-windows...
CMake Warning at scripts/cmake/vcpkg_buildpath_length_warning.cmake:4 (message):
qt5-webengine's buildsystem uses very long paths and may fail on your
system.
We recommend moving vcpkg to a short path such as 'C:\src\vcpkg' or using
the subst command.
Call Stack (most recent call first):
ports/qt5-webengine/portfile.cmake:4 (vcpkg_buildpath_length_warning)
scripts/ports.cmake:147 (include)
CMake Warning at ports/qt5-webengine/portfile.cmake:5 (message):
The qt5-webengine source was will be extracted to
G:/Cone/vcpkg/buildtrees/qt5-webengine , which has more then 35 characters
in length.
Call Stack (most recent call first):
scripts/ports.cmake:147 (include)
CMake Error at ports/qt5-webengine/portfile.cmake:6 (message):
terminating due to G:/Cone/vcpkg/buildtrees/qt5-webengine being too long.
Call Stack (most recent call first):
scripts/ports.cmake:147 (include)
Additional context
MITK requires a web engine of qt5 to compile, but I was about to install it when suddenly it stopped working. My request is that the directory remain unchanged, just provide me with an accurate solution

We recommend moving vcpkg to a short path such as 'C:\src\vcpkg' or using
the subst command.

可以了，但是

warning: The vcpkg G:\vcpkg\vcpkg.exe is using detected vcpkg root G:\vcpkg and ignoring mismatched VCPKG_ROOT environment value G:\Cone\vcpkg. To suppress this message, unset the environment variable or use the --vcpkg-root command line switch.
Computing installation plan...
The following packages will be built and installed:
qt5-webengine[core]:x64-windows -> 5.15.9
Detecting compiler hash for triplet x64-windows...
Restored 0 package(s) from G:\Users\LuLujiang\AppData\Local\vcpkg\archives in 250 us. Use --debug to see more details.
Installing 1/1 qt5-webengine:x64-windows...
Building qt5-webengine[core]:x64-windows...
-- qt5-webengine requires a lot of free disk space (>100GB), ram (>8 GB) and time (>2h per configuration) to be successfully build.
-- As such qt5-webengine is currently experimental.
-- If qt5-webengine fails post build validation please try manually reducing VCPKG_MAX_CONCURRENCY and open up an issue if it still cannot build.
-- If it fails due to post validation the successfully installed files can be found in G:/vcpkg/packages/qt5-webengine_x64-windows
-- and just need to be copied into G:/vcpkg/installed/x64-windows
-- Downloading https://sourceforge.net/projects/gnuwin32/files/gperf/3.0.1/gperf-3.0.1-bin.zip/download -> gperf-3.0.1-bin.zip...
-- Using cached python-2.7.18.amd64.msi.
-- Found external ninja('1.11.0').
-- Downloading https://download.qt.io/official_releases/qt/5.15/5.15.9/submodules/qtwebengine-everywhere-opensource-src-5.15.9.tar.xz;https://mirrors.ocf.berkeley.edu/qt/official_releases/qt/5.15/5.15.9/submodules/qtwebengine-everywhere-opensource-src-5.15.9.tar.xz -> qtwebengine-everywhere-opensource-src-5.15.9.tar.xz...
-- Extracting source G:/vcpkg/downloads/qtwebengine-everywhere-opensource-src-5.15.9.tar.xz
-- Applying patch common.pri.patch
-- Applying patch gl.patch
-- Applying patch build_1.patch
-- Applying patch build_2.patch
-- Applying patch build_3.with_missing_stdbool_include.patch
-- Applying patch workaround-msvc2022-ice.patch
-- Applying patch workaround-msvc2022-missing-include-in-chromium-ui-accessability.patch
CMake Error at scripts/cmake/vcpkg_extract_source_archive.cmake:159 (file):
file RENAME failed to rename
G:/vcpkg/buildtrees/qt5-webengine/src/5.15.9-f5cb7722d8.clean.tmp/qtwebengine-everywhere-src-5.15.9

to
G:/vcpkg/buildtrees/qt5-webengine/src/5.15.9-f5cb7722d8.clean

because: �ܾ����ʡ�
Call Stack (most recent call first):
scripts/cmake/vcpkg_extract_source_archive_ex.cmake:8 (vcpkg_extract_source_archive)
installed/x64-windows/share/qt5/qt_download_submodule.cmake:32 (vcpkg_extract_source_archive_ex)
installed/x64-windows/share/qt5/qt_submodule_installation.cmake:5 (qt_download_submodule)
ports/qt5-webengine/portfile.cmake:69 (qt_submodule_installation)
scripts/ports.cmake:147 (include)

because: �ܾ����ʡ�

??

You can try to clear this folder (G:/vcpkg/buildtrees/qt5-webengine/src), and then recompile.

You can try to clear this folder (G:/vcpkg/buildtrees/qt5-webengine/src), and then recompile.

你的建议我采纳了

Can recompilation be successful after clearing the files from the previous installation?


Invalid Complex formatting for floats (f32 and f64).

format!("{}", num::complex::Complex64::new(1.0, -0.0)) should yield 1+0i or 1-0i as result but instead 1+-0i is returned which is a mistake.
Here, real part is taken as 1 for demonstration purposes only.

Ugh. This is difficult because the formatting implementation is more generic than just float-like T, so we don't have any methods like is_sign_negative to detect the difference between +/-0.0. We only deal with negatives now by testing < T::zero(). I guess we could normalize == T::zero() to positive zero, as you allowed in your "should yield" possibilities -- not the best, but better than 1+-0i brokenness.

Yeah normalizing is a good patch-up maybe not a very great fix, but it should achieve the expected result regardless.

Just figured out that doing, format!("{:+}", number) would print the sign of the number whether it's positive of negative so, it should fix the problem.


Optimize constant protection runtime

Use int instead of uint, the previous implementation added extra conversions to long and ulong when comparing t == Mutation.KeyI*, and to int when accessing an array

:white_check_mark: Build ConfuserEx 583 completed (commit https://github.com/mkaring/ConfuserEx/commit/ece199d335 by @GazziFX)

This code can be optimized with unsafe stuff
// NOTE: Assume little-endian
else if (t == Mutation.KeyI1) {
var v = new T[1];
Buffer.BlockCopy(b, (int)id, v, 0, Mutation.Value<int>());
ret = v[0];
}

where T : unmanaged // should be added at the head of method
...
unsafe
{
fixed (byte* ptr = &b[0])
Buffer.MemoryCopy(ptr + id, &ret, sizeof(T), sizeof(T));
}

// P. S. This code can be incorrect (written from phone) will try it tomorrow

Your unsafe code can't be used. All code in the Confuser.Runtime project has to be compatible with the .NET Framework 2.0. Buffer.MemoryCopy was added in .NET 4.6.

Does it make sense to support outdated frameworks like .NET Framework 2.0? I'd drop support of 2.0, 3.5, 4.0, and other frameworks before 4.6.1

The support for .NET 3.5 has to stay. It's supported until 2029. I currently don't see any reason to drop the support for old framework versions. In the 2.0 version it's significantly enhanced and supports old versions easier.

It's supported until 2029.

Where have you found such info? I've found the following: https://dotnet.microsoft.com/learn/dotnet/what-is-dotnet-framework. It says that support of .NET Framework 4.5.1 ended on 2016-01-12.

I currently don't see any reason to drop the support for old framework versions.

Due to issues similar to those pointed by @GazziFX

@KvanTTT I think he means that this project still support old frameworks. Anyways I can use Marshal.Copy method which should work on NET 2.0.

There are two distinct things that need to be differentiated:

The runtime ConfuserEx itself uses. This should keep targeting the current versions of .NET Framework for now. It will change later to .NET 5 or 6.
The version that is used by the runtime code of ConfuserEx that is injected into the protected assembly. As ConfuserEx is supporting everything from .NET Framework 2.0 up, this framework defines the set of methods that can be used.

The problem is that the way ConfuserEx is build and working, those two points are not really separated in the current master branch. Only the release/2.0 branch, has proper isolation between the injected runtime code and the runtime of ConfuserEx.
Regarding the 2029 date I stated. Find it on this page. The extended Support End Date for .NET Framework 3.5 SP 1 is set to the 9th of January 2029. The support for .NET 2.0 could be dropped now, but in the context of the ConfuserEx runtime there is no additional effort to keep .NET 2.0 supported, while 3.5 is supported.


Update label-tool.md

Fixed https error

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

approved

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

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


Write a C++ program to print multiplication table

Description
Write a C++ program to print multiplication table
Input  : 2

2 x 1 = 2
2 x 2 = 4
2 x 3 = 6
2 x 4 = 8
2 x 5 = 10
2 x 6 = 12
2 x 7 = 14
2 x 8 = 16
2 x 9 = 18
2 x 10 = 20

How to contribute

Save the solution in PrintMultiplicationTable.cpp file
Add PrintMultiplicationTable.cpp file in print-multiplication-table folder

Pls assign me... I'm new in hacktobetfest'22...i want to contribute so can you assign me this

Can you assign me this issue under hacktobetfest'22 pls

!assign

Hey @vishal2376,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign


Precompiled binary for iOS/macOS

Because I should customize the building process too much, I should build xcframeworks by my hand.
And, it's not good for casual users to build the binary on their own machine. They should be easily downloadable from some site.

https://github.com/espresso3389/pdfrx/releases/tag/first-apple-support


Changing ts twoslash to ts

Changing ts twoslash to ts in places without any errors or cuts

( to make sure you're aware of the consequences for these changes, this would remove all of the hover states from these code samples too, which is a bit of a shame and you might need to look at the text surrounding the code samples to make sure there are no assumptions you can do this )
(( if you are doing this to try save on speed for CI - I'd look into whether the caching for the results of the run is not working for example ))

The main problem is the build speed, yes; removing these comments at least halves the build time. If there's supposed to be caching, it definitely is not working.

Weird! This is the code which handles caching and returning the twoslash call: https://github.com/shikijs/twoslash/blob/main/packages/remark-shiki-twoslash/src/caching.ts#L4-L68
Maybe there are issues with it running on windows? This is running on CI for it, but maybe the cache lookup always fails?
It caches the results of the files into node_modules/.cache/twoslash - so check to see if they are in there at all, or maybe something is getting corrupted?

Well, in CI, we're not going to preserve that directory. But otherwise locally, I still have to clean build quite a bit to avoid broken caching in other places, so I'm sure I'm removing that one.

You don't preserve node modules in CI? That's is(/was?) default in GitHub actions for setup-node so I figured that was a safe assumption as that pattern is pretty well used in other npm modules in the ecosystem also - maybe that folder was the removed for this repos CI at some point?
For you locally, hard to say offhand - what commands are you running to clean the build?

You don't preserve node modules in CI? That's is(/was?) default in GitHub actions for setup-node so I figured that was a safe assumption as that pattern is pretty well used in other npm modules in the ecosystem also - maybe that folder was the removed for this repos CI cache at some point?

All CI caching mechanisms I know save the npm/yarn/pnpm cache only; saving node_modules itself is very error prone as it can contain symlinks and post-install artifacts that are machine or path specific. setup-node's caching (when enabled, it's not enabled by default) only saves a single package manager's cache.

For you locally, hard to say offhand - what commands are you running to clean the build?

I most definitely git clean -fdx and do a full reinstall and rebuild. It doesn't get cleaner than that, but it's the only way I can be sure that a local website build will contain the updated releases files, styles (we hit this bug recently), etc. I of course expect no caching at that point, but I certainly don't generally expect a website build to take 4 minutes on my local machine or 20 in CI...

That's all pretty surprising yep! It wasn't really like that when I was working on it, happy to help debug some of those and/or move the twoslash caching to work with how this repo is caching between CI builds too
I probably won't have much time over over September, but I'll see if I can get a repro for some of those locally


Include flow types into library

This removes the need of flow-typed, manual copying anything etc
Now types are provided with recompose

@TrySound can you help me a little, do I need to place index.js.flow into dist folder too, or placing it as it is will be enough


add resource status check after process manifests

to address: https://github.com/istio/istio/issues/22815

@richardwxn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test lint_istio

gencheck_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test lint_istio

gencheck_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test gencheck_istio

unit-tests_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test gencheck_istio

unit-tests_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test unit-tests_istio

lint_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
6e3fb2761181ead659482ad5491eb849d991b014
link
/test unit-tests_istio

lint_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test lint_istio

gencheck_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test lint_istio

gencheck_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test gencheck_istio

unit-tests_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test lint_istio

unit-tests_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test unit-tests_istio

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test unit-tests_istio

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test unit-tests_istio

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
bd9fd0ff5da69cbd80a1b9359b1a999a3f893e24
link
/test unit-tests_istio

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

release-test_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

release-test_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test release-test_istio

integ-local-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

release-test_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test release-test_istio

integ-local-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-galley-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

release-test_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test release-test_istio

integ-local-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-galley-k8s-tests_istio

integ-mixer-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test gencheck_istio

lint_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test lint_istio

integ-conformance-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-conformance-k8s-tests_istio

integ-pilot-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-pilot-k8s-tests_istio

unit-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-security-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-telemetry-k8s-tests_istio

release-test_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test release-test_istio

integ-local-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-local-tests_istio

integ-galley-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-galley-k8s-tests_istio

integ-mixer-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-mixer-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ffbea74c866988969ecf88558057783eb9488bad
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Overhaul the update UI

Now there are fewer lines of spew and the final update summary is
condensed. A typical update looks like
info: syncing channel updates for 'stable'
info: checking for self-updates
info: downloading self-update

  stable unchanged: 1.1.0 (hash-s-2)

r=me, exciting!

:pushpin: Commit ddaca07 has been approved by brson

@bors r+

:hourglass: Testing commit ddaca07 with merge ff5343e...

Oh there's probably no chance bors will succeed, so I'd just merge when CI is green.

:boom: Test timed out


[Blink] WebCL: Only set ENABLE_WEBCL in the relevant files.

This change (together with a follow-up in the Crosswalk repository
itself) moves the definition of the ENABLE_WEBCL macro to Blink's
features.gypi, so that it is only passed to the relevant modules and
bindings files intead of all the 15000+ files part of Crosswalk's build.
This part of the fix only adds the definition to the Blink side; the
Crosswalk patch will stop defining the macro for the other files.

@chunkyan @darktears

LGTM, this is part of my rcode to fix XWALK-6200


 How can i use this package in different recursive models? I have product, category, an brand models, all of this has product_id, category_id  and brand_id as foreign key.

Hi @josegus
Unfortunately there are no this feature. But you can extend the NestableService class and override config property with your custom config array.

are pull request allowed? I mean, a pull request making this option available. No breaking changes, just extend the actual config.

I definitely accept. Please add your tests.


Fix for specifying 'inbound_smtp_server'

This fixes:
NameError: name 'inbound_smtp_server' is not defined
When trying to define a inbound_smtp_server without doing MX lookup

Thanks!


No sound when playing video.

I have Nokia 6303i Classic runs on an S40 6th edition and when it is playing a video on Jtube theres no sound. I think it is a codec issue. I am playig videos at 144p and via browser option.  Can I fix this problem myself or is it a bug?

Maybe it could be... but here I have 201 and I don't have that audio problem... although some 144px videos in 3gp can't be downloaded

Maybe it could be... but here I have 201 and I don't have that audio problem... although some 144px videos in 3gp can't be downloaded

I have Asha 201 too and for me it is downloading the video first and then i can watch the video. But on 6303i i don't have to wait for download. Video plays instantly but sadly there's no sound

Read FAQ


localpv provisioner crash if the old PVs are not cleaned up

Description
When the old PVs are not cleaned up and there are references to the localpv storage class, the provisioner crashes
1 start.go:70] Starting Provisioner...
failure in preupgrade tasks: failed to get bdc bdc-pvc-0c179a7a-bae1-4e16-8fa5-cbcf61c6d735: blockdeviceclaims.openebs.io "bdc-pvc-0c179a7a-bae1-4e16-8fa5-cbcf61c6d735" not found

There are no bdc bdc-pvc-0c179a7a-bae1-4e16-8fa5-cbcf61c6d735 from the output of kubectl get bdc -A
Once all the old PVs are deleted, the openebs-localpv-provisioner pod started correctly.
Expected Behavior
localpv provisioner should not crash
Your Environment
OpenEBS 2.7.0

This error is generated when the provisioner fails to find BDCs for PVs while trying to add finalizers to BDCs of older (pre-v1.1) local-device volumes.
Orphaned PVs (with Retain reclaimPolicy) which do not have BDCs bound, are not usable, as the data may (or may not) have been corrupted.
Possible cause for the BDCs not being there could be manual removal of BDC resources.
These PVs may be overlooked when trying to add finalizers to BDCs created by older versions of OpenEBS.


v4.1.3 route transitioning does not remove previous DOM nodes

Hi there, thanks so much for your work on this library.
In your React Router example after a transition is complete the old DOM node is removed (which is correct behavior); https://codesandbox.io/embed/xo0lrqw2nz
In this example if I upgrade the library version to v4.1.3 (which is the latest version at time of writing) I notice the previous DOM nodes no longer get removed as they should. Here's my fork with the only difference being the version number: https://codesandbox.io/s/yv2458ljxv

Yep, sorry for that, it's the same issue as https://github.com/drcmda/react-spring/issues/43
The Transition component has a bug in the latest that prevents it from removing stale components. Fix will come up.

@nickmcmillan it should be fine now, @4.1.4


how to not use the _MD5 as the suffix of the filename?

          I still need use wikilinks in obsidian,but not on image~

and by the way how to not use the _MD5 as the suffix of the filename?

I use the MD5 as the filename but not need the suffix. could add this?

Originally posted by @Tback1 in https://github.com/Sergei-Korneev/obsidian-local-images-plus/issues/29#issuecomment-1613535086

and by the way how to not use the _MD5 as the suffix of the filename?

I still need use wikilinks in obsidian,but not on image~

You are allowed to edit the source-code.


Allow Returning Sets or Lists from Functions

Currently, if I have something like
#let null = math.op("N")
'''typst-sympy-calculator
@func_mat()
def convert_null(mat):
    return sympy.Matrix(mat).nullspace()
'''

in my Typst file, this function called on something like null(mat(1, 2; 2, 4)) will fail with
'list' object has no attribute 'subs'
presumably because it expects the return object to be a sympy object with the .subs() method. It would be nice to have a way to return a list of sympy objects, and have it rendered as a set, for example in this case
{ vec(2, -1) }

I'm sorry for replying so late. I've been quite busy lately and completely forgot about this matter until today.
It has been fixed now. You can reinstall it by using the command: pip install typst-sympy-calculator==0.6.0.


upgraded from v3 to V4

Hi, i have recently upgraded from v3 to v4 and my messages are just stuck in the queue. Any suggestions as to what could be causing this issue?

Try restarting azure functions and app service.

Hi thanks for that, tried that still same issue. Anything else i could try or check?.

Try restarting azure functions and app service.

Hi thanks for that, tried that still same issue. Anything else i could try or check?.

Hello @Qkhan463,
We aren't able to repro this issue at our end.
Can you please check logs and post us back.
Thanks


How to keep to new position of the zoomable view after the pan movement ends?

Thank you for this great library. I have a question, i would like to be able to drag an image and keep the new positon of the image (inside the boundaries)
here is how I use the component:

	<ReactNativeZoomableView
		ref={zoomableRef}
		maxZoom={1}
	        minZoom={0.3}
		zoomStep={0.1}
		initialZoom={1}
		bindToBorders={true}>
	<Image source={{ uri: currentQuestion.question_image }} resizeMode='contain' style={{ height: '100%', width: '100%' }} />
	   </ReactNativeZoomableView>

@medmo7 you can refer to this example https://github.com/openspacelabs/react-native-zoomable-view#example


[5.10.0] Getting hostname as the server host name when the authAPIURL is not configured

When using prompt function[1] with identity server 5.10 if the below configuration is not enabled(commented) in the <IS_HOME>/repository/deployment/server/webapps/authenticationendpoint/WEB-INF/web.xml , the host name is collected from the server host configured in the deployment.toml file.
By default this is commented .

  <context-param>
        <param-name>AuthenticationRESTEndpointURL</param-name>
        <param-value>https://localhost:9443/api/identity/auth/v1.1/</param-value>
    </context-param>

Ideally , it should be collected from the identity.server.serviceURL , if the above  web.xml configuration is not enabled.
Error scenario
When using the prompt funciton in their adaptive script. In our dynamic_prompt.jsp file it has the below code snippet
String authAPIURL = application.getInitParameter(Constants.AUTHENTICATION_REST_ENDPOINT_URL);

    if (StringUtils.isBlank(authAPIURL)) {
        authAPIURL = IdentityUtil.getServerURL("/api/identity/auth/v1.1/", true, true);
    }
    if (!authAPIURL.endsWith("/")) {
        authAPIURL += "/";
    }

According to the above implementation if the authAPIURL is getting from the web.xml file. Which will return null to the authAPIURL as it is commented by default. Due to that reason the authAPIURL will be build with the hostname value which is configuring in the deployment.toml file.
If the identity server deployment fronted by a loadbalancer  and as the  https://<IS_HOST>/api/identity/auth/v1.1/context/<id>  API required the MTLS to authenticate with Identity server, this returns 401 response because the request is served through the LB in this scenario and the LB uses SSL termination.
[1]https://is.docs.wso2.com/en/5.10.0/references/adaptive-authentication-js-api-reference/#prompttemplateid-data-eventhandlers

Note

Even though this issue was fixed in product-is, after the unification process, all the fixes have been lost. Therefore, all the related places need to be fixed again with the usage of internal_hostname when invoking internal API calls in identity-apps.
The git issue which tracked this issue previously - https://github.com/wso2/product-is/issues/11878


Port the function delta_file_sizes

We want to port the function delta_file_sizes from the mack project.
Here is a link to the function: https://github.com/MrPowers/mack/blob/cf250e29fa366cd7a0d31a0bc55edb1c74ebbd75/mack/init.py#L476
Reference to the origin of this function: https://github.com/MrPowers/mack/issues/16

@brayanjuls @MrPowers shouldn't we close this issue as the code is now merged


Can't teleport though open door

Hello, I have found some bug in the door BP, when I open the door and want to teleport through the door, I have to move my hand through the door first and after that, I can teleport further.
Another bug or maybe it's just my mistake, I have been working on archviz project and exported first test version, for some reason I cannot teleport there, ground just doesn't show any option for this. I'll look why I see it in the Editor and not in Win.EXE application.

Still unable to reproduce this issue (as mentioned in #10), please check the collision in your level to make sure there is nothing blocking the path


Document how QR codes can be used

Add a section to docs.helium.com to explain how QR codes can be generated and its capabilities in the mobile app.
https://docs.helium.com/wallets/app-wallet, add a new guide titled "Supported QR codes and usage guide"
Right now we can support several types of QRs:

Burn HNT - QR code from console
Payment_v2 - QR code with 1 receiver or more, with or without memo
add_gateway
assert_location_v2 (coming soon in a PR)
Wallet address QR code

Others on the horizon:

QR to follow a Hotspot

https://docs.helium.com/wallets/app-wallet/deep-links-and-qr-codes/


Implement Burst of Knowledge trinket

https://tbc.wowhead.com/item=11832/burst-of-knowledge this is the latest fad on mage discord. It gives -100 to the base cost of your spells for 10 sec. There's a log with Turkeybrah wearing it here if you want some confirmation on how it works in game https://classic.warcraftlogs.com/reports/xWdKZFLhYPJQADR8#fight=16

Can't read out mana cost of spells in logs so it's kinda useless.
Any screenshots/videos of this?

Thanks. I see you already merged it but here's some screenshots demonstrating mana cost reduction, for the record. https://discord.com/channels/253212375790911489/812641242222821376/865981715301662740

Went to BRD just to confirm it myself.


amusiveness, decrials

This pull request was generated by the 'mq' tool

/trunk merge

/trunk merge


[hotfix] Fix Java 11 target compatibility & add tests

This PR:

Fixes Java 11 incompatibility by unifying maven-shade-plugin versions
Automatically selects higher version Java target when using JDK 11+
Adds Java 11 into CI test matrix

Done, extracted common logic into flink_cdc_base and flink_cdc_migration_test_base, and kept minimal configurations in flink_cdc_java_8 and flink_cdc_java_11 workflow. @GOODBOY008 PTAL

Thanks for @GOODBOY008's review, fixed and rebased.

Apologies to @GOODBOY008, but seems modifying workflow names cause some side effects that breaks README building status badge. See #3807 for a hotfix.


FT Request a Change - Exceptional Expense Appeal (Student)

Description of Task:
Allow students to change their financial information
Context:

See #266 for context about general Request a Change Process

Acceptance Criteria

[ ] Introduce new 'form title' called 'Exceptional Expense Appeal (Full-Time Applications)'

[ ] Attach the following field/question to the form: "Do you want to submit an exemption request for exceptional expenses?"

[ ] Beneath this question, add the following two things:

free-input numeric field

A message beneath the free-input field which is always displayed, and which reads: "You can report exceptional expenses if they created financial hardship that affected your ability to start or continue studies, such as: medical illness or injury, family emergency (e.g. death or injury), natural disaster, unforseen legal expenses, court-ordered payments, expenses related to your or your dependant's permanent disability that are not covered by another funding source (uninsured)."

[ ] If student inputs anything in this field, display a warning banner below reading: "Your request will not be approved until the following documentation is added to your Student Aid BC account using the File Uploader:

A letter explaining your exceptional expenses, when the expenses occured, the amount of payment, and why these expenses were necessary during your study period

A copy of your medical / dental insurance plan proving that this expense is not covered under your plan

All relevant supporting documentation of your expenses (payment receipts, a copy of your bank statement, cancelled cheques, etc.)

[ ] Build the subsequent review screens (See Ministry Review Request photo and Student Review Request photo)

[ ] Change the workflow logic in the following ways:

If request is denied, do nothing

If field is left blank, do nothing

If any fields are inputted, and the request is approved, replace the appropriate workflow variables with the new amounts and generate new assessment.

[ ] See #2467 for other details about how request interacts with COE requests, other pending reassessment requests, etc...

Closing as this will be resolved under the request a change revamp epic.


StateWatcher watches and reports changed Pipeline State

This PR closes #32032 with a package private StateWatcher. It sends a GetJobStateRequest to the state stream endpoint of the Job Management service and reports any changes in State to listeners, invoking onStateChanged.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @Abacn or @ahmedabu98

@Abacn this is different in that the runner needs to launch an underlying Job Management service written in a different language and shut it down gracefully. The reason for introducing this PR is that relying on existing tools in Beam do not properly shutdown the gRPC channels, the code is too nested to safely refactor. This has caused errors in relying on proper shutdown, even when calling waitUntilFinish. The result is that the underlying process remains running without owndership and the only way to end the process is to use lsof, for example followed by kill.
Please review this assuming it is needed.


Can't define own templates

I set my templates in the config, but when I "generate" a commit message, my custom template doesn't appear.
"vscodeGitCommit.variables": {
    "prefix": "keke",
    "aliases": [
      {
        "label": "🚀 workspace upgrade",
        "detail": "Select when a workspace package is upgraded",
        "key": "5993179",
      },
      {
        "label": "🌈 style",
        "key": "9355052",
        "detail": "Select when beautify or style changes"
      }
    ]
}

Currently publishing a new version (2.1.0) that fix it, sorry for the inconvenience, it will auto migrate settings to be compliant with old ones


First go at defining our observables

@kirienko - I had to write down the equations for the presentation, so I put them in a tutorial notebook.
Could you proof read as a sanity check, please?
I think the summary in this notebook is useful to have somewhere.
Not sure how we should mix this with the existing pages and link to the new plot-specific function  documentation that you have created.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #168   +/-   ##
=======================================
  Coverage   79.13%   79.13%
=======================================
  Files           5        5
  Lines         906      906
=======================================
  Hits          717      717
  Misses        189      189

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 8bb43ea...c1192f8. Read the comment docs.


go-client 5.0.1 does not build with dep

I believe due to recent mutations in k8s.io/apimachinery, specifically https://github.com/kubernetes/apimachinery/commit/158150cb5856aaec3a319ff460ca4b93b715adda .
To reproduce:

Create $GOPATH/bug-repro, and copy examples/workqueue/main.go to it.
 Note in the output that it correctly locks go-client to ^5.0.1.
go install .

The compile will fail with:
bug-repro/vendor/k8s.io/client-go/tools/cache
vendor/k8s.io/client-go/tools/cache/mutation_detector.go:103:33: scheme.Scheme.Copy undefined (type *"bug-repro/vendor/k8s.io/apimachinery/pkg/runtime".Scheme has no field or method Copy)

dep doesn't parse the Godeps.json file that pins the exact dependency version. I recommend using glide at the moment. https://github.com/kubernetes/client-go/blob/master/INSTALL.md#glide

dep doesn't parse the Godeps.json file that pins the exact dependency version.

It seems dep does supports parsing Godeps.json now: https://github.com/golang/dep/blob/master/docs/FAQ.md#what-external-tools-are-supported
But we have other issues: https://github.com/golang/dep/issues/1124

Thanks! I've switched to glide, and I got a successful build. Thanks for the help, and for updating the documentation on dep (I originally used dep because INSTALL.md seemed to say it would work).


Outline the learning goals for our group.

Hi everyone,
We need to define the learning goals for our group.
The following link contains the learning goals document we need to prepare. Once it's ready, we will push it to GitHub: Learning Goals Document
Please, if any member has additions or suggestions, kindly add them as comments in the document or below. This will help us keep track of changes clearly, and there will be no need to review the entire document.

Hi everyone,
We need to define the learning goals for our group.
The following link contains the learning goals document we need to prepare. Once it's ready, we will push it to GitHub: Learning Goals Document
Please, if any member has additions or suggestions, kindly add them as comments in the document. This will help us keep track of changes clearly, and there will be no need to review the entire document.

For the goals, I only have one potential addition, which could be to incorporate a goal around reflection and feedback. After completing certain tasks or milestones, taking time to reflect on what went well and what could be improved can be valuable for continuous learning. Consider adding something like, "Schedule regular team retrospectives to discuss what worked, what didn’t, and how to improve processes for future tasks." This could enhance learning from experiences and help improve collaboration and problem-solving.
Feel free to add more ideas if you think of another suggestion.... 👇

This is a fantastic blend of technical skill-building and personal development goals! I really appreciate the emphasis on both mastering tools like Python, GitHub, and CI workflows, as well as growing essential human skills like communication and teamwork. The focus on applying skills in real-world scenarios, like integrating CI tools, is especially valuable, it ensures the learning is practical and impactful. This plan truly sets the team up for long-term success, Great work, everyone!😊


Clean Up Config Files

Description:
Clean up config files
Why:
Currently, we have many config files in the codebase. for example, webpack.config.js, webpack.dev.config.js, webpack.karma.config.js, tsconfig.json,tsdoc.json, karma.conf.js, api-extractor.json, etc..
We can categorize them to some directories

Can I try this issue??


Correction to "AzureWebJobsDashboard must be specified"

In addition, the properties AzureWebJobsStorage and AzureWebJobsDashboard must be specified as app settings in the site configuration.

Apparently, AzureWebJobsDashboard is not required, in fact it is best avoided if you are using App Insights for monitoring.
Please correct either page if necessary.

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

ID: a1ba8817-3e01-7e84-6c71-b98b131f2927
Version Independent ID: a7703386-9e83-1075-72ef-f9e63f3dc4de
Content: Automate resource deployment for a function app in Azure Functions
Content Source: articles/azure-functions/functions-infrastructure-as-code.md
Service: unspecified
GitHub Login: @ggailey777
Microsoft Alias: glenga

@ggailey777 @tdykstra There are a couple docs linked here that need updating so, I have included the both of you on this GitHub issue. Please update as necessary.

@ishepherd Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate. If there are additional details that you feel are pertinent to helping provide clarity, please feel free to add that information.

@ishepherd Thanks for reporting that - I've made the fix, and it should be live either this afternoon or tomorrow morning.


apt-get install commands should assume yes

This remove the manual enter in the auto-install script

Thanks!


When the training code will be released?

Hi, I just met your guys during CVPR 2024 in Seattle. Nice work! Could I know when the training code will be released? Thanks!

Thanks for your interest in our work. We plan to release it in a few weeks from now 😀

Thanks very much! Hope "a few <= 3" 😆


Getting "Unable to infer closure type in the current context"

I am getting this error while adding the following method after updating the pod
self.chatHubConnection!.on(method: "receiveMessage") { (message, String) in
             for mesDict in message {
                self.messageReceive(message: (self.convertToDictionary(text: mesDict as! String) ?? [:]))
            }
        }

Did you mean ‘message: String’ instead of ‘message, String’ ?
On Tue, Oct 22, 2019 at 3:34 PM djmunish<EMAIL_ADDRESS>wrote:

I am getting this error while adding following method.
self.chatHubConnection!.on(method: "receiveMessage") { (message, String) in
for mesDict in message {
self.messageReceive(message: (self.convertToDictionary(text: mesDict as! String) ?? [:]))
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/moozzyk/SignalR-Client-Swift/issues/77?email_source=notifications&email_token=AAK7JJEQY3OONC2QCG53S5LQP55WBA5CNFSM4JDZDPV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HTUW5HA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK7JJGL2ATRERKX67RDFSLQP55WBANCNFSM4JDZDPVQ
.

Thanks, buddy it works, earlier it was message, String. Thanks for the quick response!


Update PPP

Codecov Report

Merging #224 (7bec2b5) into master (fbe9204) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 7bec2b5 differs from pull request most recent head d6df0ed. Consider uploading reports for the commit d6df0ed to get more accurate results

@@            Coverage Diff            @@
##             master     #224   +/-   ##
=========================================
  Coverage     65.81%   65.81%
  Complexity      717      717
=========================================
  Files           114      114
  Lines          2542     2542
  Branches        294      294
=========================================
  Hits           1673     1673
  Misses          753      753
  Partials        116      116

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 fbe9204...d6df0ed. Read the comment docs.


Pls add google drive support

That kind of nonsense? Why do I need this in Windows Explorer?

 Also better file managment so. I think thats not hard
to you so pls add.
вс, 24 февр. 2019 г., 19:59 Ivan<EMAIL_ADDRESS>
That kind of nonsense? Why do I need this in Windows Explorer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/duke7553/files-uwp/issues/67#issuecomment-466779677,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AroSJ8tXTbeaTawAJYtneozCDUr_57uvks5vQpq1gaJpZM4bOiKx
.

Not sure I understand what the issue is?
I've installed Google Drive File Stream on my system (which you need anyway, in order to sync), and the drive correctly shows up on the side bar.
So again, I don't see what the problem is? See the image below, drive G:\ is google drive.

@carlosrfernandez I suppose he want it be directly embedded in, a bit as Onedrive is for the classical file explorer. But since it's supports it flawlessly as an external drive, i have no need of having it supported more and close the issue.

Thx
пн, 25 февр. 2019 г., 20:48 PoorPockets McNewHold<EMAIL_ADDRESS>
:

@carlosrfernandez https://github.com/carlosrfernandez I suppose he want
it be directly embedded in, a bit as Onedrive is for the classical file
explorer. But since it's supports it flawlessly as an external drive, i
have no need of having it supported more and close the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/duke7553/files-uwp/issues/67#issuecomment-467038360,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AroSJ9kX2LsA00Sh4xU3P_KAnIbzRTUpks5vQ_ehgaJpZM4bOiKx
.

Support for Google Drive is now available. You can download the latest version of Files from our website https://files.community/download


fix(schema): restore limit on line-length

Summary
Restores functionality of #7875 but in the correct place. Closes #7877.
I couldn't figure out how to get cargo fmt to work, so hopefully that's run in CI. Nevermind, figured it out.

Test Plan
Can see output of json.

Ah excellent! Thank you @henryiii!

Thanks!!


Update release tags for 1.3.17

Description
Update release tags for 1.3.17
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@prudhvigodithi @peterzhuamazon Do we not have automation for this yet?

No @gaiksaya but we extend this workflow https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/release-manifest-commit-lock/release-manifest-commit-lock.jenkinsfile having a new action for for final manifest tag update.

No @gaiksaya but we extend this workflow https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/release-manifest-commit-lock/release-manifest-commit-lock.jenkinsfile having a new action for for final manifest tag update.

I see! Do we have an issue for this? Looks like good enhancement. We can call that workflow after tag-creation workflow is finished.


add no_omitempty option to remote omitempty label from generated struct

For #682
add an new option "no_omitempty", which is default false to keep default appearance.
when call go_out=no_omitempty=true:./out.go, structs without omit_empty label for json will be generated.

I signed it!


Add LinuxBoot README.

cc @rminnich
cc @zaolin
cc @vejmarie
cc @dhendrix

We probably could add a "supported OCP hardware list" I liked Tram array with the various "working" feature from our latest hackathon ?

I messed up the table, please don't merge yet :)

Fixed it.

See preview at https://github.com/hugelgupf/OSF/blob/42006c1765880dfb91bedc2374eb9797af11909d/LinuxBoot/README.md

ping @rajeevsharma1?


Bug: Labels change on first press of the repeat key

Hi, I'm really liking the new repeat key functionality you implemented in 108222e, but I noticed that the labels change the first time the repeat key is pressed after "leaping". Subsequent presses of the repeat key don't change the labels.
I'm assuming this is not the intended behavior? To me, it makes more since for the labels to either always remain constant, or be updated on every repeat so that the nth leapable match can always be jumped to with with the same keystroke.

Thanks for the great plugin!

Another issue I just noticed is that the repeat keys still seem to be active even when you are no longer in the context of leaping. This leads to some weird behavior when using repeat keys with flit (with multiline disabled), as leap's repeat will take over and cause the cursor to jump to the next occurrence of the previous leap pattern.

I'm assuming this is not the intended behavior?

the repeat keys still seem to be active even when you are no longer in the context of leaping.

Well, that is the point of these keys, they repeat the last motion directly, just like ;/, for f/t, otherwise there is next_target/prev_target. If you s{char}{char} and then press ; (or whatever key you mapped), it actually exits the leap() call, and starts such a "cold" repeat (but with a trick, see below). I personally don't see much value in s{char}{char}<repeat><repeat>..., that is what labels are for, and why we show them ahead of time.

I noticed that the labels change the first time the repeat key is pressed after "leaping". Subsequent presses of the repeat key don't change the labels.

That said, if this annoys you a lot, you can set up an autocommand that maps ;/, as next/prev-target spec. keys after LeapPatternPost (so that they won't conflict with the search pattern), and removes them on LeapLeave. (add_repeat_mappings does exactly the same, but only for repeat calls, copy that with the obvious modifications, or just ask for help if needed.)

Off-topic: what is the colorscheme? Looks nice :)

Thanks for the thorough reply, it was really insightful.

the intended use case is doing a ;;;; streak, not s{ch}{ch};;; (for that, there is next_target/prev_target).

Oh wow, I didn't even realize next_target and prev_target were a thing. Even so, my OCD was killing me so I decided to implement your suggested fix. I'm not really that experienced with autocmds, but this is what I came up.
config = function()
  local sk = (require "leap").opts.special_keys
  local saved_next_target, saved_prev_target
  local id_1, id_2
  local function set_mappings()
    pcall(vim.api.nvim_del_autocmd, id_2)
    saved_next_target = sk.next_target
    saved_prev_target = sk.prev_target
    sk.next_target = ";"
    sk.prev_target = ","
    local function remove_mappings()
      pcall(vim.api.nvim_del_autocmd, id_1)
      sk.next_target = saved_next_target
      sk.prev_target = saved_prev_target
      id_1 = vim.api.nvim_create_autocmd("User", { pattern = "LeapPatternPost", once = true, callback = set_mappings })
    end
    id_2 = vim.api.nvim_create_autocmd("User", { pattern = "LeapLeave", once = true, callback = remove_mappings })
    return nil
  end
  id_1 = vim.api.nvim_create_autocmd("User", { pattern = "LeapPatternPost", once = true, callback = set_mappings })
end,

While this does seem to work, it unfortunately overwrites the default next_target and prev_target mappings... which leads me to wonder: if repeat mappings are enabled, is there even a point in having next_target/prev_target set to their own separate mappings? I think it might actually make more sense to have both next_target/prev_target and the repeat mappings to be the same, since ; and s<cr> do pretty much the same thing.

Hmm, it shouldn't... in flit, ;/, are mapped as next_target/prev_target by default, that should take priority. It has nothing to do with multiline.

Oh wait, I think there might be a bug with flit. When pressing , to jump to the previous occurence of a character, it's not possible to jump to an occurence before the location the cursor was currently at, which is not consistent with default vim f/F behavior. Additionally, spamming ; will keep jumping towards the next target up until the last occurence on the line, but , will still jump to the previous target once the last occurence is reached, which makes sense. Spamming ,, on the other hand, will only jump to the previous target until reaching first occurence after the starting position, at which point neither , nor ; will do anything anymore.
I also might've been confused because when there is only a single occurence of a character on a line, pressing ;/, triggers leap's repeat instead of jumping between the next and previous occurence of a character. But you were right that it has nothing to do with multiline.

Or you mean you want ;/, to function as repeat keys for Flit or Leap, depending on which one was the last motion?

I think that makes a lot of sense, and would align better with default vim behavior, since ;/, normally triggers "f/F<last found character". This would also lead to less confusion when there a single occurence of a character on a line and leap repeat is triggered, as just mentioned.
tldr; I think that it might be more intuitive for ;/, to trigger both next_target/prev_target and repeat either both leap or flit, depending on which one was used last to align more closely with default vim behavior. These would eliminate the need for  and .

Off-topic: what is the colorscheme? Looks nice :)

Cappuccin with these highlight groups :)
local colors = require("catppuccin.palettes").get_palette()

LeapMatch = { fg = colors.text, bold = true, nocombine = true },
LeapLabelPrimary = { fg = colors.pink, bold = true, nocombine = true },
LeapLabelSecondary = { fg = colors.blue, bold = true, nocombine = true },

I'm gonna close this issue now since I believe it will be solved by the repeat parameter hinted in #176.

I believe it will be solved by the repeat parameter hinted in https://github.com/ggandor/leap.nvim/issues/176

Nope, unrelated :)

Oh whoops I completely forgot what this issue was originally about, I was mainly referring to the inconsistent ;/, behavior when I commented that.

While this does seem to work for the most part, it unfortunately overwrites the default next_target and prev_target mappings...

Here is a clean solution:
local set_phase2_traversal_keys
do
  -- Set a counter in this closure and generate a unique autogroup each
  -- time, so that multiple key pairs can be defined (e.g. ;/, and s/S).
  local n = 0
  set_phase2_traversal_keys = function (fwd_key, bwd_key)
    local group = ('LeapSetPhase2TraversalKeys_' .. n)
    n = (n + 1)
    vim.api.nvim_create_augroup(group, {})
    vim.api.nvim_create_autocmd('User', {
      pattern = 'LeapPatternPost',
      group = group,
      callback = function ()
      if not require('leap').state.args['repeat'] then
        -- Note: `require('leap').opts` would dispatch to `opts.default`
        -- (see `init.fnl`).
        local opts = require('leap.opts').default
        local opts_cc = require('leap.opts').current_call
        if not opts_cc.special_keys then
          -- Copy the defaults - we need to keep the group switch
          -- keys - the `__index` method of `opts` will not fall
          -- back to `opts.default` for `special_keys` anymore if
          -- there is such a table in `current_call`. (And obviously
          -- deep-copy, don't overwrite the original.)
          opts_cc.special_keys = vim.deepcopy(opts.special_keys)
        end
        local cc_next = (opts_cc.special_keys.next_target or {})
        local cc_prev = (opts_cc.special_keys.prev_target or {})
        if (type(cc_next) == 'string') then cc_next = {cc_next} end
        if (type(cc_prev) == 'string') then cc_prev = {cc_prev} end
        table.insert(cc_next, fwd_key)
        table.insert(cc_prev, bwd_key)
        opts_cc.special_keys.next_target = cc_next
        opts_cc.special_keys.prev_target = cc_prev
      end
    end
    })
  end
end

set_phase2_traversal_keys(';', ',')
set_phase2_traversal_keys('s', 'S')

Actually, it might be worth including this in user.fnl.

While this does seem to work for the most part, it unfortunately overwrites the default next_target and prev_target mappings...

Here is a clean solution:
local set_phase2_traversal_keys
do
  -- Set a counter in this closure and generate a unique autogroup
  -- each time, so that multiple key pairs can be defined (e.g. ;/,
  -- and s/S).
  local n = 0
  set_phase2_traversal_keys = function (next_key, prev_key)
    local group = ('LeapSetPhase2TraversalKeys_' .. n)
    n = (n + 1)
    vim.api.nvim_create_augroup(group, {})
    vim.api.nvim_create_autocmd('User', {
      pattern = 'LeapPatternPost',
      group = group,
      callback = function ()
        if require('leap').state.args['repeat'] then return end
        -- Note: `require('leap').opts` would dispatch to
        -- `opts.default` (see `init.fnl`).
        local opts = require('leap.opts').default
        local opts_cc = require('leap.opts').current_call
        if not opts_cc.special_keys then
          -- Copy the defaults - we need to keep the group switch
          -- keys - the `__index` method of `opts` will not fall
          -- back to `opts.default` for `special_keys` anymore if
          -- there is such a table in `current_call`. (And obviously
          -- deep-copy, don't overwrite the original.)
          opts_cc.special_keys = vim.deepcopy(opts.special_keys)
        end
        local cc_next = opts_cc.special_keys.next_target or {}
        local cc_prev = opts_cc.special_keys.prev_target or {}
        if (type(cc_next) == 'string') then cc_next = {cc_next} end
        if (type(cc_prev) == 'string') then cc_prev = {cc_prev} end
        table.insert(cc_next, next_key)
        table.insert(cc_prev, prev_key)
        opts_cc.special_keys.next_target = cc_next
        opts_cc.special_keys.prev_target = cc_prev
      end
    })
  end
end

set_phase2_traversal_keys(';', ',')
set_phase2_traversal_keys('s', 'S')

Actually, it might be worth including this in user.fnl.


Component/record types

What is the process for defining new component/record types?
Do we want to support defined sets of components/records for certain communities (MME)?
Do we allow private components?
Why is components an array of objects if there is only one object?

By private I mean for internal use within organizations. For example organization X has a number of internal databases and wants to use this protocol but wants to use their own record type. The protocol as currently defined supports this but this is not official.

My vote would be no, since it would add complexity?
@harindra-a - Migrated

I think we can allow for groups to define their own components.
In the MME API, we specify that anything prefixed with an underscore is non standard. Allowing the same thing for component types should be fine I feel, and would allow for groups to define their own components. I don't think we need to call them "private" though, but that's more semantics.

It turns out this is pretty tricky to specify using JSON Schema, especially if I want to maintain draft-5 backwards compatibility to eventually auto convert to OpenAPI.
Given it's part of the written spec, we don't NEED to enforce this using JSON Schema.


Update file structure to follow electron-webpack guidelines

This change the file structure to follow the webpack-electron's  default one. The dirs under src/ are named after electron's processes: Main (previously, under app/) and Renderer (previously, under ui/).

@IGassmann did this actually come in with the file changes?
@liamcardenas once this PR is ready, you can merge at your convenience. Please run Prettier across the whole project right after merging.


TP2000-286 multiple workbaskets

TP2000-286 multiple workbaskets

Why

Tamato currently only offers a globally "active" workbasket, presenting a process bottleneck to users.
A system administrator is required to set the "active" workbasket.
There are opportunities to improve the process and UI around workbasket management and review.

What

This PR addresses the "Why" above by:

Allowing each user to concurrently make changes to their own, active workbasket (no more shared, active workbasket, though users can choose to work on a shared workbasket if they wish).
Providing users with the ability to activate their own workbasket for editing, removing the need for an administrator at this step.
Reorganising, extending and refining the workbasket UI views.

A user's active workbasket is maintained in their session and can, and normally will, be unique to that user.

Checklist

Requires migrations? No
Requires dependency updates? No

All looks good, just a couple of comments on tests, involuntarily said "ooh I like that" to myself as reading this.

Codecov Report

Merging #627 (5a7af5e) into master (2889072) will decrease coverage by 0.00%.
The diff coverage is 88.10%.

@@            Coverage Diff             @@
##           master     #627      +/-   ##
==========================================
- Coverage   93.02%   93.01%   -0.01%
==========================================
  Files         324      325       +1
  Lines       20485    20616     +131
  Branches     1637     1650      +13
==========================================
+ Hits        19056    19177     +121
- Misses       1131     1133       +2
- Partials      298      306       +8

Impacted Files
Coverage Δ

common/urls.py
100.00% <ø> (ø)

workbaskets/models.py
84.49% <ø> (ø)

workbaskets/urls.py
100.00% <ø> (ø)

workbaskets/views/ui.py
69.73% <75.78%> (+7.97%)
:arrow_up:

common/views.py
87.40% <90.00%> (+7.61%)
:arrow_up:

common/forms.py
87.41% <100.00%> (+0.48%)
:arrow_up:

common/tests/factories.py
99.34% <100.00%> (-0.17%)
:arrow_down:

common/tests/test_views.py
100.00% <100.00%> (ø)

settings/common.py
76.15% <100.00%> (ø)

workbaskets/forms.py
95.12% <100.00%> (+4.64%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Add the force_latest_compat and coverage inputs

This should not be merged until https://github.com/JuliaLang/Pkg.jl/pull/2176 is merged.

I used this repo to test composite actions when they were new and forgot to archive it. No need to update it. Sorry!


Correct DFR_HEIGHT and adjust the button height calculation accordingly

NAK, this breaks m1

How does this break M1s? According to https://github.com/AsahiLinux/m1n1/blob/main/proxyclient/experiments/touchbar_rainbow.py#L13, the height of the Touch Bar displays on M1s are 60.

The line stride is 64, and using 60 as height turns the image into cut up lines.

I added a separate variable for the stride. Can you check if it works on M1s?

Results in misaligned buttons on m1.

Can you test the new version please?

Can you test the new version?

Well, it doesn't even compile, :P But even after i fixed the compile error, the labels are still misaligned.

Well, it doesn't even compile, :P

Sorry about that, I forgot to amend the fix after adding it.
Could you please test it again?

I have noticed that you haven't actually did anything to the label drawing code so they will still be misaligned. Git pilot error? :P

I changed the x2 value of the rectangle passed to drm.dirty from DFR_HEIGHT to DFR_STRIDE, since I guessed that it could possibly be the cause.

Nope, that call just flushes the pixels from memory to the display controller. (The exact arguments don't even matter on m1, the driver does not do damage tracking and always sends a full screen frame)

I thought that the issue could be caused by the top 4 pixels not being flushed as the buttons get cut off on my Touch Bar too if I decrease x2 by 4.

 I think this version should work :).

 I think this version should work :).


InverseMelScale does not work in inference mode

InverseMelScale uses SGD inside so it does not work when the global context is no_grad or inference_mode. Or even when requires_grad=False would make it fail. This gives bad UX for inference.
There are couple of possible workarounds

Clone Tensor with requires_grad=True inside of InverseMelScale
Use other method such as SVD. (there was some discussion https://github.com/pytorch/audio/pull/366)

import torch
import torchaudio

trans = torchaudio.transforms.InverseMelScale(n_stft=10)

spec = torch.randn(128, 256, requires_grad=True)
trans(spec)  # pass
with torch.inference_mode():
    trans(spec)  # fail

spec = torch.randn(128, 256, requires_grad=False)
trans(spec)  # fail
with torch.inference_mode():
    trans(spec)  # fail

Traceback (most recent call last):
  File "foo.py", line 10, in <module>
    trans(spec)
  File "/miniconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/Development/torchaudio/torchaudio/transforms.py", line 472, in forward
    new_loss.backward()
  File "/miniconda3/lib/python3.8/site-packages/torch/_tensor.py", line 307, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/miniconda3/lib/python3.8/site-packages/torch/autograd/__init__.py", line 154, in backward
    Variable._execution_engine.run_backward(
RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn

Hi, @nateanl did you port nnls method?

Hi @spyroot , not yet. I'll work on it after the 0.12 release. Stay tuned!

Ratio of relative diff smaller than 1.000000e-01 is 7.614522473886609e-05
Ratio of relative diff smaller than 1.000000e-03 is 0.0
Ratio of relative diff smaller than 1.000000e-05 is 0.0
Ratio of relative diff smaller than 1.000000e-10 is 0.0
Ratio of relative diff padded than 1.000000e-01 is 7.145936251617968e-05
Ratio of relative diff padded than 1.000000e-03 is 0.0
Ratio of relative diff padded than 1.000000e-05 is 0.0
Ratio of relative diff padded than 1.000000e-10 is 0.0
Ratio of relative diff librosa than 1.000000e-01 is 2.3429299744748278e-06 <--- was my target
Ratio of relative diff librosa than 1.000000e-03 is 0.0
Ratio of relative diff librosa than 1.000000e-05 is 0.0
Ratio of relative diff librosa than 1.000000e-10 is 0.0
Ratio of relative diff my impl 1.000000e-01 is 0.0 < --- :) solved
Ratio of relative diff my impl 1.000000e-03 is 0.0
Ratio of relative diff my impl 1.000000e-05 is 0.0
Ratio of relative diff my impl 1.000000e-10 is 0.0

Hi @spyroot, do you mean you already implemented the nnls method? Would you like to open a pull request for it? We can help review after the PR is created. Thanks!

Yes, I used LBFGS and tested on GPU crazy fast and superb abs error with the original source. I'll do a pull request next week. But I had two fixes for two bugs in LBFGS, which I need to commit together.
I have one question about the current implementation. Torch does backward pass in InverseMelScale.
So if you do that in the training loop, it is a second backward pass.
So how is it intended to be used if you want to compute inverse in the training loop?

@nateanl thank you very much.   The reason I asked about the second case.   Imagine we are working in none GAN settings.
if you call a backward inside a loss function, that brake things a bit.
imagine a training pass. (pseudocodish)
for batch in batches:
loss=compute_lose(x) -> if compute_lose does inverse and it has backward()
self.scaler.scale(loss).backward()
self.scaler.step(optimizer)
self.scaler.update()
It is problematic.
Yes, we probably can formulate it as GAN.   i.e, you do backward() on the main optimizer than
then compute the Inverse.
Thinking a bit deeper. If Inverse computes a solution Ax=b and it is a Solution.
And we know that solution exists.
( in this case, by the way, if  you find a solution, you don't need to deal with Complex)
Why is it because the solution is float, i.e, you can find a solution for complex, complex -> the output float.
Then you can technically compute Inverse inside  But, why.   Why would you want to do that?
We imagine you need that Inverse for loss computation.   But if you do have a solution
You don't need to minimize anything.
Do you see my point? For example, in my case. I have to do this.
in compute_loss().  I am still trying to figure out the most efficient way to do this and avoid backward()
            with torch.no_grad():
                x1 = self.dts_inverse(mel_padded)
                x2 = self.dts_inverse(mel_out_post_net)

dts_inverse is my implementation.

@spyroot I see. So in your implementation, dts_inverse doesn't require gradient optimization. This helps solve the issue that InverseMelScale can't work in inference mode.
Regarding the usage in training, although we don't intend to optimize the InverseMelScale, the differentiability of the module is important. Take speech enhancement task as an example, some methods optimize the model based on waveforms instead of spectrograms. They indeed achieve some performance gain by doing that. In that case, we don't want the module is hard-coded as with torch.no_grad() because that breaks the chain of gradients.

I found another solution in librosa. in librosa.utils.nnls it uses np.linalg.lstsq which can be replaced by torch.lstsq. So that we don't need to re-implement the SGD manually.

Just wanted to know whether this is implemented by any chance.

Can this issue be solved by temporarily setting torch.enable_grad at the call site or inside the function?

Can this issue be solved by temporarily setting torch.enable_grad at the call site or inside the function?

torch.enable_grad works with torch.no_grad, but torch.inference_mode is more strict which doesn't record computation in the backward graph. Therefore the optimization inside InverseMelScale can't be run.

Addresed via #3280


camerad: free memory on failure

Perhaps we can refactor to automatically free when it goes out of scope?

created a new pr https://github.com/commaai/openpilot/pull/28807 to refactor to auto-free


BeaconReinforcement

I think this idea could be the future of SMP and I would definitely pay for this. My idea is for this  project to expand to BeaconReinforcement, a protection add on.
See what reinforcement is here: https://github.com/trevor99z/Reinforcement
The reinforcement region size would be determined by the Beacon pyramid. The amount of “breaks” would be determined by what blocks were used in the pyramid’s construction, all configurable in config. Permitted users can build/break/access with the region with ease, non-permitted would have to break through the reinforcement.
Along with the current GUI, there would be another tab (maybe a chest plate icon) where a user could add trusted members, purchase BeaconAlarms (with items, exp, or Vault. similar with how teleport costs work now), and also access block history. I think block history would be dope for dynamics as users could see what enemies broke at what time so they’re prepared to fight back.
If someone destroys a Beacon the region would start to “decay” by the reinforcement dropping one block-break every server tick. The destruction of the Beacon would also trigger the BeaconAlarms. During BeaconDecay all users within the region would receive the red tint that the world border gives you when you’re outside.
This would create a bury dynamic, where users would bury their beacons under layers of glass for protection. Griefers would target Beacons to make their raid easier.
Other details about BeaconDecay..
Would be cool if there was an animation. Maybe thunder?
BeaconAlarms include Weakness, Slowness, Blindness, Nausea, Spectral Arrow Outline, and Mining Fatigue.
TLDR;
BeaconReinforcement

Reinforcement sister plugin
Members
Block History
Decay
Discord Hooks
BeaconAlarms

While these are good ideas, they are outside of the scope of this plugin which focuses on fast travel using beacons.


Array.from returning empty array

using object.keys instead

@salzhrani nice catch. I was wonder what this getAssetGlobs function was doing. I'll publish an update.


Suggestion mini_installer.exe does not for path for installing

Why does mini_installer.exe not ask for where it should install ?

That is the behaviour inherited from upstream Chromium. Although I do admit it wasn't very good in the past (i.e. it would install x64 versions to Program Files (x86)).
Problem is there isn't any GUI for this program at this time, though it could be added.

How about documenting that?
I started mini_installer.exe, Chrome started and worked.
And I was surprised, when I found the installation in the User Folder.


More variant like samples.

Codecov Report

Merging #50 (b76b861) into main (132449d) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   81.50%   81.50%
=======================================
  Files           1        1
  Lines         411      411
  Branches       82       82
=======================================
  Hits          335      335
  Misses          9        9
  Partials       67       67

Impacted Files
Coverage Δ

include/matchit.h
81.50% <100.00%> (ø)

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 132449d...b76b861. Read the comment docs.


Hide autocomplete when the input is blurred.

Summary
Hide autocomplete when the input is blurred.
DISCLAIMER:

This is only for the post input. Other locations where this does not apply (yet):

Search
Edit post
Channel info form

Ticket Link
https://mattermost.atlassian.net/browse/MM-28453
Release Note
Autocomplete now gets closed when the input is blurred.

@matthewbirtch Let me know how you see it like this. I went the "easy path" which is taking in account the focus state of the input. The problem? If we hide the keyboard (at least in android), the input gets blurred, so autocomplete disappear. If this is a big issue, I can try to look for other options.

@matthewbirtch Let me know how you see it like this. I went the "easy path" which is taking in account the focus state of the input. The problem? If we hide the keyboard (at least in android), the input gets blurred, so autocomplete disappear. If this is a big issue, I can try to look for other options.

@larkox this works pretty well. The one case we will want to fix is the following:
When the input is blurred and the keyboard is not shown, if the user taps the @ button or the / button the character gets added to the input field, but right now it does not give focus to the input. We should make the input have focus immediately after adding the character to the input field when users press either of these two buttons. Is this doable?

@matthewbirtch Done here: https://github.com/mattermost/mattermost-mobile/pull/6613 There was some weird issue with autocomplete, but seems related to the server, not really the PR.


Reduce scope of interactive plot example

Reduces scope of example for interactive plotting to earthquake magnitudes in the last week within the state of California. Ideally, this reduces execution time for the website.

@lillythomas the notebook needs to be executed before saving and committing so that it includes the plot binary/html.

Make sure to execute the cell and save the notebook with the output. Right now the plot is blank.

Done


Check if the KUBECONFIG file is inside /tmp

If it is somewhere under /tmp, bail out explaining why. This has caused
a few silly situations where the user would save the KUBECONFIG file
under /tmp. Since bind-mounting /tmp seems like a wrong thing to do in
general, we at least bail out with a clear error message.
To do this we rely on a bash functionality so let's just switch the
script to use that.
Tested as follows:
export KUBECONFIG=/tmp/kubeconfig
./scripts/pattern-util.sh make help
/tmp/kubeconfig is pointing inside the /tmp folder, this will make it unavailable from the container.
Please move it somewhere inside your /home/michele folder, as that gets bind-mounted inside the container
export KUBECONFIG=~/kubeconfig
./scripts/pattern-util.sh make help
Pattern: common
Usage:
make
...

/lgtm - ship it!


getURL() inside Baseadapter

First i want thanks for your great library.
I implement SliderLayout inside BaseAdapter with ViewHolder()
onSliderClick works like a charm, but i need to get the right BaseSliderView at onMenuItemClick to receive some data that i put in before.
getCurrentSlider() returns always the next or previous BaseSliderView.
Have somebody an idea?
I know its caused by BaseAdapter, but have no idea how to solve this.

I solved it by building some temp variables at getView() after addSlider()
public static SliderLayout oddSlider;
public static SliderLayout evenSlider;
public static SliderLayout oddOldSlider;
public static SliderLayout evenOldSlider;

public static int oddPos;
public static int evenPos;
public static int oddOldPos;
public static int evenOldPos;

private int count = 0;

@Override
public View getView(int position, View convertView, ViewGroup parent) {

if((position%2)==0){
        if(count==1) {
            evenOldSlider = evenSlider;
            evenOldPos = evenPos;
            count = 0;
        }
        evenSlider = holder.slider;
        evenPos = position;
        count++;

    } else {
        if(count==1) {
            oddOldSlider = oddSlider;
            oddOldPos = oddPos;
            count = 0;
        }
        oddSlider = holder.slider;
        oddPos = position;
        count++;
    }

I saved Sliders by position (even and odd), thats the current/next Slider and the two previous Silders.
At my onMenuItemClick
     if((...getCurrentPage()%2)==0){
                baseSliderView = ...evenSlider.getCurrentSlider();
                pos = ...evenPos;
            } else {
                baseSliderView = ...oddSlider.getCurrentSlider();
                pos = ...oddPos;
            }

     if(...getCurrentPage() != pos){
            if((FlipFragment.flipView.getCurrentPage()%2)==0){
                 baseSliderView = ...evenOldSlider.getCurrentSlider();
                 pos = ...evenOldPos;
             } else {
                   baseSliderView = ...oddOldSlider.getCurrentSlider();
                   pos = ...oddOldPos;
              }
       }

If somebody know a better solution, please let me know.
Cheers

I use a picasso object and
    picasso = Picasso.with(context);


tests with @ShouldThrowException not correctly honored

I am currently struggling to try to run microprofile TCKs tests on wildfly.
Some of the tests are using @ShouldThrowException(value = DefinitionException.class, testable = true) where DefinitionException is javax.enterprise.inject.spi.DefinitionException.
While tests run well on a weld container, I cannot make them run with a wildfly container.
I suspect:

the expection to be wrapped but not unwrapped to check against the one defined inside ShouldThrowException
the failing undeployment (because the deployment failed as expected) to also cause troubles.

Enclosed is a simple reproducer case (I also put it on github: https://github.com/McFoggy/wildfly-failing-deployment):

mvn clean test -Pweld works as expected
mvn clean test -Pwildfly fails for the test expecting the deployment failure

reproducer app: failing-on-deployment.zip

To be clear - I made the API change in Arquillian that allows testable archives to throw exceptions.  Containers probably still need to do something to support this feature.
The change, as noted by @McFoggy is that @ShouldThrowException now has a testable attribute, false by default, that is meant to support enhancing the archive.

Looks like we do yes. It's new in 1.1.14 and currently wildfly-arquillian is targeting 1.1.13.

Actually the 2.0 branch is targeting 1.1.11. Is there a reason a new feature went into a macro release? Feels like it should have been a minor release IMO.

I'm going to close this issue and use https://issues.jboss.org/browse/WFARQ-36 for tracking purposes.


start prop doesnt work

passing in an object with x and y key value pairs does not move the draggable box

I have same bug:
start={{x: this.state.element_start, y: 0}}

I found solution:
this.refs.draggable_element.setState({clientX:this.state.element_start});

2.x now has a position prop.


Function App refresh required after swap

After swapping slots, a refresh of the app was required so that functions would appear in the correct slot location

This  is not an easy fix - since SWAP uses Ibiza code - this experience with the new redesign for Swap in Express.

This is by design for now & will be fixed by the re-design of Swap in Express.