[Feature Request] Are there some way to customize summary row or display html

Existing Component
是
Component Name
Table
Description
hi~ I want to show total with fixed row at the bottom of table, and it seems the only way is summary row.
But I also want to customize content with html, for example, i want to display two-line content in the cell.
Is possible that support summary slot or richtext in summary row?
你好，我想问一下，element现在有方法可以实现table组件最后一行固定的同时，支持自定义cell的内容吗（比如支持多行数据）?
如果可以，希望可以考虑一下支持summary/footer slot，或者可以在summary/footer行支持富文本 ~

Found any solution for this?

@eele94 Did you manage to figure this out?

@eele94 Did you manage to figure this out?

Nope, in the end only used the show-summaries and summary-method(getSummaries) method as described in the documentation

@motishaku did you find any solution or way around ?


How to use this plugin

can u tell me how to use this plugi in? what is snaptoken?

Please read official documentation here:
https://mobile-docs.midtrans.com/#transaction-flow-using-snaptoken-flow


Can't compile on Windows due to gdscript_vm.cpp taking a long time to build

Godot version:
4
OS/device including version:
Windows
Issue description:
Cant compile under windows with mono.
Hangs on gdscript_vm.cpp
Related to that commit
https://github.com/godotengine/godot/commit/cf4079cb5f51a40c8370a84f5131f944c398f4b8
Steps to reproduce:
Compile under windows with mono
Minimal reproduction project:
Not necessary

Are you using MSVC or MinGW to compile Godot? If you're using MinGW, try using llvm-mingw instead.

MSVC

Btw its just by building under the target release_debug.

Here the compiler is hanging

And thats my build settings for vscode:
  {             "label": "mono-build-binary-release",             "type": "shell",             "command": "scons",             "group": "build",             "args": [                 "platform=windows", // Change to your current platform                 "target=release_debug",                 "tools=yes",                 "module_mono_enabled=yes",                 "-j20"             ],             "problemMatcher": "$msCompile",             "dependsOrder": "sequence",             "dependsOn": [                 "mono-build-release-temp",                 "mono-build-glue-release"             ]         },
    `     {
        "label": "mono-build-release-temp",
        "type": "shell",
        "command": "scons",
        "group": "build",
        "args": [
            "platform=windows", // Change to your current platform
            "tools=yes",
            "target=release_debug",
            "module_mono_enabled=yes",
            "mono_glue=no",
            "-j20"
        ],
        "problemMatcher": "$msCompile"
    },`

      {             "label": "mono-build-glue-release",             "type": "shell",             "command": "bin\\godot.windows.tools.64.mono.exe",             "group": "build",             "args": [                 "--generate-mono-glue",                 "modules/mono/glue"             ],             "problemMatcher": "$msCompile"         },

How much time did you wait during the linking process? It can take a very long time on the master branch (up to an hour).

Issue is gone since the laster rebase from the master branch. Ticket can be closed.

With a CL that identifies itself as Microsoft (R) C/C++ Optimizing Compiler Version 19.36.32532 for x64, from Visual C++ 17.6.0, identified by SCons as MSVC 14.3, it happens to me, with this command line:
cl /Fomodules\gdscript\gdscript_vm.windows.editor.x86_64.obj /c modules\gdscript\gdscript_vm.cpp /TP /std:c++17 /nologo /MT /Gd /GR /utf-8 /bigobj /Zi /FS /O2 /W4 /wd4100 /wd4127 /wd4201 /wd4244 /wd4245 /wd4267 /wd4305 /wd4514 /wd4714 /wd4820 /EHsc /WX /DTOOLS_ENABLED /DDEBUG_ENABLED /DNDEBUG /DNO_EDITOR_SPLASH /DWINDOWS_SUBSYSTEM_CONSOLE /DWINDOWS_ENABLED /DWASAPI_ENABLED /DWINMIDI_ENABLED /DTYPED_METHOD_BIND /DWIN32 /DMSVC /DWINVER=0x0601 /D_WIN32_WINNT=0x0601 /DNOMINMAX /D_WIN64 /DVULKAN_ENABLED /DGLES3_ENABLED /DMINIZIP_ENABLED /DBROTLI_ENABLED /DZSTD_STATIC_LINKING_ONLY /DUSE_VOLK /DVK_USE_PLATFORM_WIN32_KHR /DGLAD_ENABLED /DGLES_OVER_GL /Ithirdparty\libpng /Ithirdparty\glad /Ithirdparty\volk /Ithirdparty\vulkan /Ithirdparty\vulkan\include /Ithirdparty\zstd /Ithirdparty\zlib /Ithirdparty\brotli\include /Iplatform\windows /I.
It's taken around 10 minutes, but has completed.

@RandomShaper Can you reproduce this after removing the bin/ folder? See https://github.com/godotengine/godot/issues/77968.

I can't try again shortly since I've already changed the build types, etc. However, I can confidently say that the issue is not related to link time. (I have issues with never-ending linking that I fix by Ctrl-C, deleting the .ilk file and linking again, but that's aside.) This problem with compilation happens even if I run the pasted command manually, outside the build system.


Conversion of Sprite using region to MeshInstance2D broken

Godot version:
3.1-stable
OS/device including version:
Windows 10
Issue description:
When converting a Sprite that uses region to MeshInstance2D, the resulting mesh is broken:

Steps to reproduce:

Add a sprite with region to scene
Click "convert to 2d mesh"

Minimal reproduction project:
Get Minimal reproduction project and try to convert the sprite.

The conversion works correctly when the origin point of the region is (0,0)

Still valid in 2a0de6d

For anyone interested in fixing this, the problem is create_from_image_alpha method in BitMap, which ignores the region rectangle. I tried adding create_from_image_rect_alpha, which would create a BitMap from sub-rect of an image, but got stuck on byte magic. Here's some patch with my work: 0001-Tried-to-fix-mesh-from-sprite.zip


Ignore  prepare_zpool test on failure?

Hello, I am almost done with packaging Shavee for NixOS. However, I see a unit test that needs root permission. This isn't supported is there an alternative or can you ignore this specific test?
error: test failed, to rerun pass `-p shavee --bin shavee`
error: builder for '/nix/store/nr969jrh81fln3j79ipbpkyhkx4f1wyd-shavee-0.5.drv' failed with exit code 101;

Hi @jasonodoom, I have updated PR #26 so the unit tests that depend on the root permission to gracefully exit in case the root permission is not granted, instead of causing a panic and fail the tests. Please give it a try and if resolves your issue, we can close this case after that PR is merged.

Thanks @kiavash-at-work  looks like that worked! Can you also do the same for dataset_does_not_exists_test? That one is failing too:
failures:

---- zfs::tests::dataset_does_not_exists_test stdout ----
thread 'zfs::tests::dataset_does_not_exists_test' panicked at 'ZFS and ZPOOL tools must be installed. Test terminated early!: Os { code: 2, kind: NotFound, message: "No such file or directory" }', shavee-core/src/zfs.rs:451:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    zfs::tests::dataset_does_not_exists_test

test result: FAILED. 12 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 431.3

Then you can re-package and I will build from that and if everything is fine from there (which it should be) I'll create a PR in nixpkgs.

I made the ZFS units test to exit gracefully too. Please check the updated code again.

Thanks @kiavash-at-work all tests are passing. Can you please tag this so I can work from it?

@ashuio as the maintainer of this repo can only generate a new tag/release after merging the PR.

noted. I will wait patiently then 😄

Sorry for the delay, the commit is merged and tagged.

no worries @ashuio thank you. It looks like the file hash test are now failing. Can you or @kiavash-at-work please resolve?
     > thread 'filehash::tests::get_filehash_unit_test' panicked at 'called `Result::unwrap_err()` on an `Ok` value: [114, 198, 213, 139, 67, 213, 120, 43, 163, 13, 255, 110, 191, 190, 203, 251, 149, 28, 124, 127, 251, 14, 155, 0, 181, 106, 178, 75, 235, 244, 197, 242, 51, 230, 184, 174, 82, 132, 6, 213, 33, 91, 98, 211, 223, 239, 228, 8, 236, 29, 70, 188, 168, 158, 241, 251, 66, 152, 199, 6, 79, 85, 98, 12]', shavee-core/src/filehash.rs:643:30
       >
       >
       > failures:
       >     filehash::tests::get_filehash_unit_test
       >     filehash::tests::remote_file_hash
       >
       > test result: FAILED. 11 passed; 2 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.19s

Hi @jasonodoom,
I am not able to duplicate this error. Can you please run these units tests using this command
$ cargo test --package shavee_core --lib -- filehash::tests --nocapture
and share the output for our review?

Hi @kiavash-at-work This seems to be the issue: thread 'filehash::tests::get_filehash_unit_test' panicked at 'called Result::unwrap_err()on anOk value
In a local environment this error isn't present but for the case of nix packaging it appears.
failures:

---- filehash::tests::get_filehash_unit_test stdout ----
0
1
2
3
4
thread 'filehash::tests::get_filehash_unit_test' panicked at 'called `Result::unwrap_err()` on an `Ok` value: [114, 198, 213, 139, 67, 213, 120, 43, 163, 13, 255, 110, 191, 190, 203, 251, 149, 28, 124, 127, 251, 14, 155, 0, 181, 106, 178, 75, 235, 244, 197, 242, 51, 230, 184, 174, 82, 132, 6, 213, 33, 91, 98, 211, 223, 239, 228, 8, 236, 29, 70, 188, 168, 158, 241, 251, 66, 152, 199, 6, 79, 85, 98, 12]', shavee-core/src/filehash.rs:643:30
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- filehash::tests::remote_file_hash stdout ----
0
thread 'filehash::tests::remote_file_hash' panicked at 'called `Result::unwrap_err()` on an `Ok` value: [114, 198, 213, 139, 67, 213, 120, 43, 163, 13, 255, 110, 191, 190, 203, 251, 149, 28, 124, 127, 251, 14, 155, 0, 181, 106, 178, 75, 235, 244, 197, 242, 51, 230, 184, 174, 82, 132, 6, 213, 33, 91, 98, 211, 223, 239, 228, 8, 236, 29, 70, 188, 168, 158, 241, 251, 66, 152, 199, 6, 79, 85, 98, 12]', shavee-core/src/filehash.rs:447:26

failures:
    filehash::tests::get_filehash_unit_test
    filehash::tests::remote_file_hash

test result: FAILED. 11 passed; 2 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.19s

error: test failed, to rerun pass `-p shavee_core --lib`
error: builder for '/nix/store/a8frqy4bf51wdy701pq3bdmp75sl3kn8-shavee-0.5.1.drv' failed with exit code 101;
       last 10 log lines:
       > thread 'filehash::tests::remote_file_hash' panicked at 'called `Result::unwrap_err()` on an `Ok` value: [114, 198, 213, 139, 67, 213, 120, 43, 163, 13, 255, 110, 191, 190, 203, 251, 149, 28, 124, 127, 251, 14, 155, 0, 181, 106, 178, 75, 235, 244, 197, 242, 51, 230, 184, 174, 82, 132, 6, 213, 33, 91, 98, 211, 223, 239, 228, 8, 236, 29, 70, 188, 168, 158, 241, 251, 66, 152, 199, 6, 79, 85, 98, 12]', shavee-core/src/filehash.rs:447:26
       >
       >
       > failures:
       >     filehash::tests::get_filehash_unit_test
       >     filehash::tests::remote_file_hash
       >
       > test result: FAILED. 11 passed; 2 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.19s
       >
       > error: test failed, to rerun pass `-p shavee_core --lib`
       For full logs, run 'nix log /nix/store/a8frqy4bf51wdy701pq3bdmp75sl3kn8-shavee-0.5.1.drv'.

In a local environment this error isn't present but for the case of nix packaging it appears.

It seems that the packager doesn't allow curl to access the internet so the unit tests that depend on it fail.
I prefer not to remove these unit tests and instead have a condition in the unit test code that detects it is being run by nix packaging and skip the tests that depend on the internet. Is there any indication in the packager that can be passed to the unit test code (like ENV variable)?

Please give this a try and it works in your build environment I will submit a PR. Otherwise, I need your help to fine tune the logic.
I added this to check for existence of buildPhase ENV variable to skip the test if it exists (its value is not checked).
        // Check for Nix package builder before running the unit test [issue #27]
        if std::env::var("buildPhase").is_ok() {
            eprintln!("Nix package environment detected. Unit test is skipped.");
            return;
        }

@kiavash-at-work , thanks for working with me on this.  I was able to test it locally via buildPhase = "echo"; Fingers crossed but it's passing and everything should be good to go. And of course with the removal of buildPhase the tests fail as expected.
Can you update to check for the existence of meta instead? The current use of buildPhase  will probably receive scrutiny as it's not logical.
running 3 tests
test args::tests::input_args_check ... ok
Dataset Password: test tests::password_mode_hash_test ... ok
Dataset Password: test tests::integration_tests ... ok

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.27s

     Running unittests src/lib.rs (target/x86_64-unknown-linux-gnu/release/deps/shavee_core-9fcc89409d48fbc9)

running 13 tests
test filehash::tests::get_filehash_unit_test ... ok
test filehash::tests::remote_file_hash ... ok
test tests::port_check_test ... ok
test zfs::tests::zfs_list_test ... ok
test tests::parse_file_size_arguments_test ... ok
test zfs::tests::zfs_create_load_unload_key_test ... ok
test zfs::tests::zfs_mount_umount_test ... ok
test zfs::tests::zfs_name_validation ... ok
test zfs::tests::zfs_to_string_test ... ok
test filehash::tests::local_file_hash ... ok
test zfs::tests::dataset_does_not_exists_test ... ok
test password::tests::test_hash_argon2 ... ok
test logic::tests::file_key_calculation_test ... ok

test result: ok. 13 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.20s

     Running unittests src/lib.rs (target/x86_64-unknown-linux-gnu/release/deps/shavee_pam-e0030c1c026ab0c3)

running 1 test
test args::tests::input_args_check ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

   Doc-tests shavee_core

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

Finished cargoCheckHook
checkPhase completed in 1 minutes 15 seconds
installing
Executing cargoInstallHook
Finished cargoInstallHook
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/6ka2dsm1x088w3mfyz1xa5mwclv3lh8m-shavee-0.5.1
strip is /nix/store/dkw46jgi8i0bq64cag95v4ywz6g9bnga-gcc-wrapper-11.3.0/bin/strip
patching script interpreter paths in /nix/store/6ka2dsm1x088w3mfyz1xa5mwclv3lh8m-shavee-0.5.1
checking for references to /build/ in /nix/store/6ka2dsm1x088w3mfyz1xa5mwclv3lh8m-shavee-0.5.1...
/nix/store/6ka2dsm1x088w3mfyz1xa5mwclv3lh8m-shavee-0.5.1

This is very good that we found out the root cause of the test failures (lack of access to the internet by the builder). However it is not clear to me your proposed usage of meta.
Would you mind putting together your proposed solution as a simple algorithm (in any language that you prefer, even bash is ok) and I will try to write it in Rust?

Unfortunately, I am not familiar with the Nix packager. Are any of these meta attributes exposed to the shell environment?
No worries, I am still new to this myself and wanted to learn more that's why I wanted to contribute to Nix packages.

Not unless you use nix-env
But all of the phases such as postInstall and buildPhase are  exposed via stdenv
[nix-shell:~/code/shavee-dev]$ echo $postInstall
cp target/release/shavee $out/bin/ cp target/release/libshavee_pam.so $out/lib/security/libshavee_pam.so

Regarding postInstall, I don't know how it is exposed to the unit tests of the rust code. Is it a package builder directive or shell environment set during the package build?
It's exposed via the standard build environment (see above).

The below is the current postInstall which executes these instructions after compilation. In this case the binary and  the library file are being copied to their necessary paths.
I am not using  buildPhase but  postInstall Your addition worked when I tested it but should be replaced with posInstall:
    // Check for Nix package builder before running the unit test [issue #27]
        if std::env::var("postInstall").is_ok() {
            eprintln!("Nix package environment detected. Unit test is skipped.");
            return;
        }

There doesn't need to be any additional steps besides that. No additional logic on the package itself is necessary and would be wasteful. Essentially, an environment is created, the source is pulled and the package is built within that environment. So we can expect everything to work as postInstall is also accessible and in use since it's necessary here to copy the binary after compilation.
I tested with buildPhase and not postInstall to avoid the copying of the binary.
If you prefer, you can also run this on your end:
# default.nix
with import <nixpkgs> { };

rustPlatform.buildRustPackage rec {
  pname = "shavee";
  version = "0.5.1";
  src = ./shavee;

  cargoSha256 = "sha256-iNGn5KknSNgazFSu6Nur7AkKVb6qKMxuFwTdCz5djWU=";

  nativeBuildInputs = [
    pkg-config
  ];

  buildInputs = [
      libiconv
      linux-pam
      makeWrapper
      openssl
      openssl.dev

  ];

buildPhase = ''
  echo
'';

  meta = with lib; {
    description = "A Program to automatically decrypt and mount ZFS datasets using Yubikey HMAC as 2FA or any File on USB/SFTP/HTTPS.";
    homepage = "https://github.com/ashuio/shavee";
    license = licenses.mit;

    maintainers = with maintainers; [jasonodoom];
  };
}

Just run nix-build

Thank you for the complete and detailed description. The change was simple. If this looks good to you, I will submit a PR.

LGTM, thanks so much! Please don't forget to do the same for get_filehash_unit_test

@jasonodoom i have merged the solution and created a new release, I have changed the tag name form 0.5.1 to v0.5.1 so you might want to edit that on your end too.
Let me know if this works for you and I can close this issue.

@jasonodoom No worries, I updated it, can you check and let me know?

@ashuio good to go, thanks! The rest is up to me.


Add python 38 classifier and black code formatter spec

Coverage remained the same at 100.0% when pulling d9b4d6c6199d26e4e963720d28acafe61997859b on add-python-38-support into 23c73318447f578a4a24845c5f43574ac7b414e4 on master.

Coverage remained the same at 100.0% when pulling d9b4d6c6199d26e4e963720d28acafe61997859b on add-python-38-support into 23c73318447f578a4a24845c5f43574ac7b414e4 on master.

Coverage remained the same at 100.0% when pulling d9b4d6c6199d26e4e963720d28acafe61997859b on add-python-38-support into 23c73318447f578a4a24845c5f43574ac7b414e4 on master.

Coverage remained the same at 100.0% when pulling d9b4d6c6199d26e4e963720d28acafe61997859b on add-python-38-support into 23c73318447f578a4a24845c5f43574ac7b414e4 on master.

Coverage remained the same at 100.0% when pulling d9b4d6c6199d26e4e963720d28acafe61997859b on add-python-38-support into 23c73318447f578a4a24845c5f43574ac7b414e4 on master.


How can I add space between columns?

Describe the solution you'd like
Please add a new props for making space between columns!!

Currently you can add margin or padding in child views.

Closing this since this is currently possible and I am not sure the better solution would be 🤔


Create templates, sample code, and instructions for using user profile JSON API

The Javascript and Handlebars template used by the updated profile demo site can serve as the basis for this. Then we'll just provide a few additional templates that have the content arranged in various different ways and some instructions for using them. We can host this content on a new public page on the Metadata Rails app.
Note:  The current profile demo site is using the beta version of the Metadata app. Any sample code that we provide should use the production version of the Metadata app.
Aha! Link: https://penn-state-university-libraries.aha.io/features/MDDB-3

This was also done a long time ago.


Update GraphQLController to use ObjectMapper to map ExecutionResult

Prior to this PR, the Webflux version of graphql-spring-boot did not use GraphQLObjectMapper to map the execution result. This meant that the error messages could not be customized by creating a custom GraphQLErrorHandler.

Thanks for your contribution!


Support prebuilt k8s DIND images

This will make it easier to use kubeadm-dind-cluster for Virtlet devenv & CI.
Provide prebuilt image for k8s stable (1.5.x).
Try to make an image for k8s 1.4 if this is not too difficult.

Fixed in 4c6e371fa1c89bb0f92d92525df0cf22abe50f45


[Bug] Setting custom link provider doesn't remove the embedded one

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
https://microsoft.github.io/monaco-editor/playground.html?source=v0.50.0#XQAAAAK3AgAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscw2RAeOJuetpg8d5BCIdogK6CBmgyKHIOVjH6uO-oA-RA-rcpQdGBlgwMT7NDeGjyzFmNRmwpTWfh6GmWBZkAhEPmOlk69puNs02ZToWdVFwS92CvH_k9MlQ7xBm0NFhP5KEwfRmYsSPambdtpl-tZTME5kYf0muDJsyIxi_ouU9AenmLF2HVua7ZTKbwdUUjNF3X3qR_mL8MxkrznQsYH8-OH0ogxgOJSPcpR0s0klmzoK8z6fZS-rkMsyIFKo2pwhN2gLmio8WVMA12LI_cykPKg5b3cP7x0Q32t2zoJQUBQQg_pU-bADL9GXBbdW3ezdI1zu4YEcTTK5g1BqWxEW_Nq3CJ6GMaPRxnazVioDCE4ua0SGAgWSSRqL2NffLtmx_kXOtJjhIgDcxBMgoNYhW7lkBpLWKDbDdoDZ-WQB7lK1XBD9u9y_FZnfFF-AUi3UJ_Cmw9Y_GPbyXqBiJ09Ac3sQycCN0nEG30wl2xeaRJzJgZYx3THD_vzGn39-TGoA4h0Hr1ZW4HCGXPWXt1Jy8Nc4jm29JihbwWZpVzp__2YXLc
Monaco Editor Playground Code
const value = /* set from `myEditor.getModel()`: */ ` https://www.google.com> <https://www.google.com>`;

// Hover on each property to see its docs!
const myEditor = monaco.editor.create(document.getElementById("container"), {
	value,
	language: "javascript",
	automaticLayout: true,
});

monaco.languages.registerLinkProvider('javascript', {
	provideLinks(model) {

		const links = model.findMatches(
			// @ts-ignore
			/<((https?|ftp|file):\/\/[\w!#%&+,./:;=?@|~-]*[\w#%&+/=@|~-])>/g,
			true,
			true,
			false,
			undefined,
			true
		).map(({range, matches}) => ({
			range,
			url: 'https://www.google.com/'
		}))

		return {links}
	}
})

Reproduction Steps

Create monaco editor instance,
Register link provider for the same language,
See that links not detected by custom provider are detected by embedded link detector

Actual (Problematic) Behavior
When i registered custom link provider by monaco.languages.registerLinkProvider that searches for link that are enclosed in <>. The embedded link detector still works, and detects link that are not enclosed in <>.
Valid:

Not valid, but still detected:

Expected Behavior
When custom link provider is registered, embeded link provider shouldn't work
Additional Context
I think it's beacuse editor worker sets it's own linkProvider and still works, even when client version register its own linkProvider

Found solution. When calling monaco.languages.registerLinkProvider, you have to create selector like this: {language: 'javascript', exclusive: true}.


[Bug] [CDCSOURCE] When synchronizing the entire database, an error will occur if the table name of the target table is a SQL keyword.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
[dlink] 2023-12-12 01:13:00 UTC ERROR com.dlink.cdc.sql.SQLSinkBuilder 303 lambda$build$0 - Build test.user cdc sync failed...
[dlink] 2023-12-12 01:13:00 UTC ERROR com.dlink.utils.LogUtil 46 getError - org.apache.flink.table.api.SqlParserException: SQL parse failed. Encountered "user" at line 1, column 13.
What you expected to happen
排查之后发现是在创建插入的SQL语句时出错，因为没有在表名上加反引号源码位置com.dlink.utils.FlinkBaseUtil#getCDCSqlInsert

How to reproduce
使用SQL关键字做表名
Anything else
No response
Version
0.7.3
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

thx. I will fix it.


"error":"dial tcp: missing address","server":"podinfo-redis:6379"

Issue
Attempting to test some redis monitoring and the podinfo main application can't seem to connect to the redis pods,
values.yml
cache: "podinfo-redis:6379"
# Redis deployment
redis:
  enabled: true
  repository: redis
  tag: 6.0.8

deployment logs
❯ kubectl logs -n podinfo deployment/podinfo
Found 2 pods, using pod/podinfo-65d549885d-f8d2j
{"level":"info","ts":"2022-04-11T23:02:22.027Z","caller":"podinfo/main.go:150","msg":"Starting podinfo","version":"6.1.2","revision":"","port":"9898"}
{"level":"info","ts":"2022-04-11T23:02:22.032Z","caller":"api/server.go:265","msg":"Starting HTTP Server.","addr":":9898"}
{"level":"warn","ts":"2022-04-11T23:02:22.032Z","caller":"api/cache.go:159","msg":"cache server is offline","error":"dial tcp: missing address","server":"podinfo-redis:6379"}
{"level":"warn","ts":"2022-04-11T23:02:52.032Z","caller":"api/cache.go:159","msg":"cache server is offline","error":"dial tcp: missing address","server":"podinfo-redis:6379"}
{"level":"warn","ts":"2022-04-11T23:03:22.036Z","caller":"api/cache.go:159","msg":"cache server is offline","error":"dial tcp: missing address","server":"podinfo-redis:6379"}
{"level":"warn","ts":"2022-04-11T23:03:52.032Z","caller":"api/cache.go:159","msg":"cache server is offline","error":"dial tcp: missing address","server":"podinfo-redis:6379"}

I'm able to nslookup the podinfo-redis service which returns the correct address. I also spun up a test redis-cli container and was able to connect successfully.
redis-cli test
root@redis-cli:/data# redis-cli -h podinfo-redis -p 6379
podinfo-redis:6379> exit
root@redis-cli:/data# redis-cli -h podinfo-redis -p 6379 --stat
------- data ------ --------------------- load -------------------- - child -
keys       mem      clients blocked requests            connections
1          866.85K  2       0       7556 (+0)           641
1          866.85K  2       0       7569 (+13)          642
1          866.85K  2       0       7573 (+4)           642
1          866.85K  2       0       7580 (+7)           642
1          866.85K  2       0       7584 (+4)           642
1          866.85K  2       0       7588 (+4)           642
1          866.85K  2       0       7592 (+4)           642
1          866.85K  2       0       7596 (+4)           642

Seems the values.yml documentation could be a bit more clear. After messing around with a few things adding tcp:// in front of the cache value may have resolved this. Might be worth putting this into the code rather than relying on the helm chart entry.

@stefanprodan I believe adding tcp:// in front of my cache definition may have resolved this as I'm getting cache hits now. May be something to fix in the helm chart docs or in code

Yeap tcp was removed in https://github.com/stefanprodan/podinfo/pull/180 and that broke podinfo.... Thanks for digging into it.


Media Plugin (WP8.1 SL) : How to avoid picture to be saved in "Camera Roll" album ?

Hi,
Thanks to the Media Plugin, it's really easy to take picture within Xamarin Forms Apps.
However, I'm facing a small issue on Windows Phone Silverlight 8.1.
When taking a photo, it's properly saved to the local application storage but I noticed that the picture is always saved to the "Camera Roll" album... Under a different name.
I would like to know whether it is possible or not to prevent saving in the "Camera Roll" album ? And thus save the picture only in the local application storage ?
Thank you and thank you for your work !
EDIT : Before you ask, [SaveToAlbum] is not set and I also tried with [SaveToAlbum = false] ;-)

Unfortunately this is a limitation in Windows Phone 8 SL: https://msdn.microsoft.com/library/windows/apps/hh394006(v=vs.105).aspx
On Windows Phone 8, if the user accepts a photo taken with the camera capture task, the photo is automatically saved to the phone’s camera roll.
I will document this.

Okay thank you :-)
Thanks for your time & your work !


TrustKit cleanup

In this PR we're tailoring TrustKit's codebase, for Simperium Usage:

Nuked Remote Reporting Tool
Removed TrustKit Demo App
Nuked Subdomain Policy + embedded static libs

Needs Review: @astralbodies

Unit tests pass locally. Do you want to reference this PR in each of those comment blocks? It may not be super obvious as to why they're commented out if you're expecting them to last long.
:shipit:

Good call @astralbodies, adding a comment there. Thank you!!


Implementation of global flag fQtActive

This small pr implements a global fQtActive boolean that is set
to false in init.cpp, but is set to true if gridcoin is started via
main() in qt/bitcoin.cpp. This flag replaces the QT_GUI macro flag
conditionals in init.cpp to ensure that daemon functionality is correct
when a one-shot build is done of both the daemon and GUI version of
Gridcoin.
Without this change, in a one shot build the daemon will not work properly
unless --without-gui set in configure, which then requires two separate
builds to build both the daemon and gui version.

It is a quick fix... any suggestions on a permanent solution? We are not going to be able to use the QT_GUI flag unless autotools can remove the flag to compile gridcoinresearchd (only) when it compiles both.

This fix is similar to what Blackcoin did and should be able to be applied to staging for a release. We can test it along with the general tests of staging for the next release.
We need to clean up the initialization code a little more, but I think that belongs in a different PR that will receive more thorough testing on the dev branch before pushing to staging, as that will have more regression risk associated with it.

Ok. That is weird. Why is travis complaining about fQtActive not declared in scope when it is declared externel in bitcoin.cpp, and why does my build succeed locally?

Ok. Here are the remaining places where the QT_GUI flag is used.
qt/bitcoin.cpp:186:#if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:11:#if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:325:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:339:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:363:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:380:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:404:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:428:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.cpp:452:    #if defined(WIN32) && defined(QT_GUI)
qt/bitcoingui.h:9:#if defined(WIN32) && defined(QT_GUI)
qt/sendcoinsentry.cpp:65:       #if defined(WIN32) && defined(QT_GUI)
qt/transactiondescdialog.cpp:44:#if defined(WIN32) && defined(QT_GUI)
txdb-leveldb.cpp:391:        #ifdef QT_GUI
txdb-leveldb.cpp:487:        #ifdef QT_GUI
txdb-leveldb.cpp:628:#ifdef QT_GUI
wallet.cpp:542:#ifndef QT_GUI
main.cpp:4611:    #if defined(QT_GUI)
main.cpp:8824:    /*#ifdef QT_GUI
I would say we are safe to leave everything in the qt subdirectory alone, because if the main in bitcoin.cpp is not called, none of that code is active anyway.
It turns out there is already a global flag bGridcoinGUILoaded that is in global_objects_noui.hpp and is initialized in init. I am investigating using that one rather than the new flag I created.

bGridcoinGUILoaded is only in effect once the full qt is loaded. A flag that is set immediately would be better suited.

I noticed that. I am encountering a problem when trying to remove the QT_FLAG reference in main and wallet, because as soon as I declare the extern, I get duplicate definition of shutdown(), etc at the linker stage. This is probably because of a namespace/declaration conflict.
Should I try to resolve that by adding the original definition to global_objects_noui.hpp instead? This is where the bGridcoinGUILoaded is declared.


Update logic for categorising expenses

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/40668
PROPOSAL: N/A
Tests

Create a new account and login
Click on the Fab, then track expense and create a track expense request
Actionable whispers should appear in personal chat
Click on the Categorize option.
Verify that an option to select a category appears and selecting it creates a request successfully.
Repeat steps 1-3, and click on the Share with accountant whisper
Verify that you are redirected to the request confirmation page and upon confirmation, you're able to see an invite members page.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Create a new account and login
Click on the Fab, then track expense and create a track expense request
Actionable whispers should appear in personal chat
Click on the Categorize option.
Verify that an option to select a category appears and selecting it creates a request successfully.
Repeat steps 1-3, and click on the Share with accountant whisper
Verify that you are redirected to the request confirmation page and upon confirmation, you're able to see an invite members page.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Having issues with my Android build.

Android: mWeb Chrome

https://github.com/Expensify/App/assets/30054992/40968f10-a580-4908-8da6-d2aef00aa54c

iOS: Native

https://github.com/Expensify/App/assets/30054992/8c3a5e25-300e-4ab4-ad79-5ab3f4914ffe

iOS: mWeb Safari

https://github.com/Expensify/App/assets/30054992/64ffeed3-45e7-498d-9f33-eae3921171ca

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/30054992/0640a9ae-0e12-4d6f-aae3-39827ae3da4d

MacOS: Desktop

https://github.com/Expensify/App/assets/30054992/95a7b1e7-d2be-40fd-a73f-9f2dbc6d5bed

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/thienlnam in version: 1.4.67-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.67-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.67-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Rendered map chunky/incomplete?

Hi,
I've set this up for 2020 on Ubuntu 22.04.3 LTS, as by the instructions, using an planet extract of Germany from Geofabrik. I'm currently hosting it at: tiles.emilius123.de. -The data seems absolutely incomplete/"chunky", even though the import ran without errors and the extract should be, on my half, complete. (I've checked the MD5 checksum)
What could be the error here? I would be thankful for any help.

Can you post a screenshot showing what the tiles look like?


No download link to Docker Machine for Linux

I've been trying to look at https://docs.docker.com/machine/ https://docs.docker.com/machine/get-started/ and https://docs.docker.com/linux/started/ and can't find a link to Download Docker Machine for Linux.
Is Docker Machine only available for OS X and Linux?

Linux description was at https://docs.docker.com/machine/install-machine/ - but there should be a link to this page from probably all 3 of the above pages.

Hi on the last page that you referenced, you download the command using curl $ curl -L https://github.com/docker/machine/releases/download/v0.4.0/docker-machine_linux-amd64 > /usr/local/bin/docker-machine.

I wouldn't expect download links to be available in the intro documentation and getting started guides for machine.

Yes we used to have a link to the GitHub releases.  I'm not sure what happened.  I will add that back.  For now, you can get the latest release at https://github.com/docker/machine/releases

hi again the download link is on install-machine page web|source, it's in the first paragraph.

Docker Machine is supported on Windows, OS X, and Linux and is installable as one standalone binary. The links to the binaries for the various platforms and architectures are available at the Github Release page.

I agree with the sentiment of this thread (installation of the binary directly should be clearer), but I'm closing this issue as a dupe of: https://github.com/docker/machine/issues/1680

I've commented on #1680.
Once you hunt around in the menu and see "Install" and realize this also has a sub-menu for Docker Machine you are good to go, as it then includes the curl commands.
The problem is finding the page https://docs.docker.com/machine/install-machine/ if you are on any of the Docker Machine pages. Just because it belongs under the global "Install" section does not mean that there shouldn't be cross-links to it from other sections.


Should base field in misa be renamed to MXL for consistency?

How about XL instead?  This is more consistent, but highlights that it is a little different than the other fields, in that it controls the current mode's XLEN, rather than a less-privileged mode's XLEN.

MXL it is


Campaign | Flyers: Submenu

As an app user I want to be able to click on Flyers in the menu bar #185 to open up a submenu for flyers. The submenu has a single option #270.
Acceptance criteria:

[ ] 1st position: "Routen" (no link)

Further information:

the submenu option can be toggled on or off and is initially off.
#270

Note: Menu updated with #342

Prototype: https://www.figma.com/proto/7wE0Iveas3GGb8u4oL0714/App-Design-Mitglieder-App-(1)-(Copy)?node-id=2670-9749&t=b8swD5IOAFfZ0OSo-1&starting-point-node-id=2425%3A17429
API endpoint reference: https://github.com/verdigado/gruene-api/issues/82
Pollion reference: https://github.com/pollion/sunflower-app/issues/136

@leagerndt @JuliaFreitagBGSt
Please review issue description and move to "Ready for Development" if ok.

Updated to reflect Release 0*


Update README.md

I've added another Screensaver resource. It's similar to the other Audubon collection, but I am slowly hand cropping the plates to the appropriate dimensions and de-saturating each so that they look really good on the Supernote.

Thanks for adding the screensaver!


"alpha-2.html" test report (cbd7787)

@ron666
Chrome & Firefox (Last Linux / Windows / Android)

No debug for Stop, Back 30 sec., Forward 30 Sec, Mute on both Players
Dailymotion: Cant seek video
Dailymotion: On click speaker shows that error

mediaelement-renderer-dailymotion-iframe.js:270 Uncaught TypeError: Failed to execute 'dispatchEvent' on 'EventTarget': parameter 1 is not of type 'Event'.(…)(anonymous function) @ mediaelement-renderer-dailymotion-iframe.js:270

Dailymotion: On slide volume shows that error

mediaelement-renderer-dailymotion-iframe.js:270 Uncaught TypeError: Failed to execute 'dispatchEvent' on 'EventTarget': parameter 1 is not of type 'Event'.(…)(anonymous function) @ mediaelement-renderer-dailymotion-iframe.js:270
mediaelement-renderer-dailymotion-iframe.js:277 Uncaught TypeError: Failed to execute 'dispatchEvent' on 'EventTarget': parameter 1 is not of type 'Event'.(…)(anonymous function) @ mediaelement-renderer-dailymotion-iframe.js:277

Vimeo: Still visible controls on switch source
Facebook: Still visible controls on switch source
RTMP: On click speaker control cant mute

On Play Video, Audio Player

audio.ogg: File not found
Soundcloud: On switch to that source, shows

visual-single-sound-8b3fe4e9.js:3 Uncaught DOMException: Failed to execute 'getImageData' on 'CanvasRenderingContext2D': The source width is 0.(…)getImageData @ visual-single-sound-8b3fe4e9.js:3(anonymous function) @ visual-single-sound-8b3fe4e9.js:1c @ widget-dc77d738.js:2fireWith @ widget-dc77d738.js:2i.(anonymous function) @ widget-dc77d738.js:2(anonymous function) @ widget-dc77d738.js:2c @ widget-dc77d738.js:2fireWith @ widget-dc77d738.js:2(anonymous function) @ widget-dc77d738.js:2c @ widget-dc77d738.js:2fireWith @ widget-dc77d738.js:2i.(anonymous function) @ widget-dc77d738.js:2(anonymous function) @ visual-single-sound-8b3fe4e9.js:1dispatch @ widget-dc77d738.js:2m.handle @ widget-dc77d738.js:2

and breaks the controls
api.js:1 GET https://w.soundcloud.com/player/?url=https://w.soundcloud.com/player/?url=h…w_comments=true&show_user=true&show_reposts=false&visual=true& 404 (Not Found)load @ api.js:1sc.(anonymous function) @ mediaelement-renderer-soundcloud.js:233mediaElement.changeRenderer @ mediaelement-core.js:383setSrc @ mediaelement-core.js:268setFn @ mediaelement-utility.js:35(anonymous function) @ VM7344:2(anonymous function) @ alpha-2.html:254

?url=https://w.soundcloud.com/player/?url=https%3A//api.soundcloud.com/tracks/207370321&auto_pl…:1 Uncaught TypeError: Cannot read property 'call' of undefined(…)t @ ?url=https://w.soundcloud.com/player/?url=https%3A//api.soundcloud.com/tracks/207370321&auto_pl…:1(anonymous function) @ VM7330 widget-dc77d738.js:17window.webpackJsonp @ ?url=https://w.soundcloud.com/player/?url=https%3A//api.soundcloud.com/tracks/207370321&auto_pl…:1(anonymous function) @ error-698b3c1d.js:1

MsEdge
On Load
Some warnings on JS syntax
HTML1402: Character reference is missing an ending semicolon ";".
alpha-2.html (114,102)

HTML1402: Character reference is missing an ending semicolon ";".
alpha-2.html (114,113)

HTML1402: Character reference is missing an ending semicolon ";".
alpha-2.html (114,121)

Webm: On click shows "Error loading this resource"
Vimeo: Still visible controls on switch source
Facebook: Still visible controls on switch source
Dailymotion: On click speaker shows that error

SCRIPT87: Invalid argument.
mediaelement-renderer-dailymotion-iframe.js (270,11)

Dailymotion: On Slide volume shows that error

SCRIPT87: Invalid argument.
mediaelement-renderer-dailymotion-iframe.js (270,11)
SCRIPT87: Invalid argument.
mediaelement-renderer-dailymotion-iframe.js (277,11)

Soundcloud: Doesnt play and shows that error

SCRIPT5022: IndexSizeError
visual-single-sound-8b3fe4e9.js (3,15859)

audio.ogg: renderer not found

IE 11
On Load
Same warnings than MsEdge

Dailymotion: Controls Dissapear
sintel_vod.mpd: No renderer found
manifest.mpd: Error loading this resource
webm: Renderer not found
ogg: Renderer not found
Souncloud: Same errors than MsEdge

IE 10 w/o Flash
Same warnings than MsEdge & IE 10
and...
Make sure you have Flash enabled; otherwise, download the latest version from https://get.adobe.com/flashplayer/

mp4:sample.mp4: undefined (Doesnt Play)
Dailymotion: Same errors than MsEdge
Switch to Dailymotion loses controls
stream.m3u8, x36xhzz.m3u8: No renderer found
audio.ogg: No renderer found
Soundcloud: Doesnt Play

IE10 w/ flash

manifest.mpd, sintel_vod.mpd, audio.ogg: No renderer found

IE 8 w/o flash

On Load: Make sure you have Flash enabled; otherwise, download the latest version from https://get.adobe.com/flashplayer/
mp4, webm, m3u8, mpd, mp3: No renderer found
Youtube: Doesnt Play and

  Object doesn't support this property or method  www-embed-player.js, line 1217 character 190
  Exception thrown and not caught  base.js, line 7360 character 17

Dailymotion: Breaks page (adds new player), doesnt play, and...

  Object doesn't support this action  34b48c80e9545b010c89.dmp.js, line 2 character 31676
  Object doesn't support this property or method  34b48c80e9545b010c89.dmp.js, line 1 character 1

Soundcloud: Doesnt play and...

  Object doesn't support this property or method  widget-dc77d738.js, line 2 character 6604
  Object doesn't support this property or method  widget-dc77d738.js, line 1 character 1
  Object doesn't support this property or method  207370321&auto_play=false&hide_related=false&show_comments=true&show_user=true&show_reposts=false&visual=true, line 1 character 2540
  Object doesn't support this property or method  207370321&auto_play=false&hide_related=false&show_comments=true&show_user=true&show_reposts=false&visual=true, line 1 character 1

IE8 w/ Flash

Vimeo: Plays but shows that errors

  Expected identifier  player.js, line 2 character 669
  'Vimeo' is undefined  mediaelement-renderer-vimeo.js, line 109 character 4
  Expected identifier  moogaloop.js, line 2 character 101

mpd, ogg: No renderer found
Facebook:  Expected identifier  iR5joTXdUYz.js, line 669 character 6412
mp4:sample.mp4: Plays but shows broken

Soundcloud: Doesnt play and...

  Object doesn't support this property or method  widget-dc77d738.js, line 2 character 6604
  Object doesn't support this property or method  widget-dc77d738.js, line 1 character 1
  Object doesn't support this property or method  207370321&auto_play=false&hide_related=false&show_comments=true&show_user=true&show_reposts=false&visual=true, line 1 character 2540
  Object doesn't support this property or method  207370321&auto_play=false&hide_related=false&show_comments=true&show_user=true&show_reposts=false&visual=true, line 1 character 1

Thanks I'll fix any issues so one more est can be performed

@ron666 when you fix that issues i will perform a new test

@isantolin Most of the issues have been resolved; some issues cannot be fixed, since they are related to the iframes themselves. We are thinking of dropping support on IE8 and older browsers in this version. The IE10 error is expected since you deactivated it


Update events.md

@duynguyen modifed by checking https://www.timeanddate.com/worldclock/meetingtime.html?month=3&day=11&year=2021&p1=283&p2=268&iv=0

@duynguyen modifed by checking https://www.timeanddate.com/worldclock/meetingtime.html?month=3&day=11&year=2021&p1=283&p2=268&iv=0

thanks @Yu1986 , LGTM.

thanks @Yu1986 , LGTM.


AWS Wrangler S3 reading parquet, writing to DynamoDB - Unsupported type numpy.ndarray

I am trying to read parquet into dataframe with AWS wrangler, while writing this data to DynamoDB its erroring out with unsupported type error - Unsupported type numpy.ndarray for value.......
wr.s3.read_parquet(path=s3_path, dataset=dataset, chunked=True)

and writing like

wr.dynamodb.put_df(df=df, table_name=table_name)

Is there a way, I can convert the ndarray type to dynamoDB list ? I dont want to lose the readability of the array if I use np.tobytes() to write and np.frombuffer() to read again. Also, users will not have access Numpy to read the data from DynamoDB, they will be using dynamoDB API for data read

Could you please provide an example of the data you are trying to read. It does not need to be the exact data, a dummy example with a data frame would help us debug this, thanks

["APS3200", "APS3240A", "APS3240E", "APS3400"]
or

errors out like this
"errorMessage": "Unsupported type \"<class 'numpy.ndarray'>\" for value \"['APS3200' 'APS3240A' 'APS3240E' 'APS3400']\"",
  "errorType": "TypeError",

The error is raised here by the boto3 dynamodb batch writer which does not recognize the data type that you are trying to write (i.e. a numpy array). It only supports native python types such as integer, string and list.
import awswrangler as wr
import numpy as np
import pandas as pd

df=pd.DataFrame({'id': ["1"], 'key': [np.array([1, 2, 3])]})
wr.dynamodb.put_df(df, "test_table")  # error raised

raises the error you mentioned. But converting the column to a list solves it:
df["key"] = df["key"].apply(lambda x: x.tolist())

wr.dynamodb.put_df(df, "test_table")  # no error raised

thanks @jaidisido this solution will fulfill out requirement


cleanup: Remove support for Delete and Unmounting v1.1.0 PVC

as v1.0.0 is deprecated we need to remove the support for it in the Next coming (v3.0.0) release. This PR removes the support for the same.
closes #882
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>as per @gman0 request https://github.com/ceph/ceph-csi/issues/882#issuecomment-609925244 we are keeping the support to specify the rootpath and monitors in cephfs NodeStage.

Looks good to me. Definitely needs mentioning in the release notes!

will take care of it. Thanks

Added dnm so that @gman0 @ShyamsundarR can take a look at it.

@Madhu-1 thanks! I'll have a look in a bit.

I've tested Node{Stage,Publish}Volume locally with mounter and rootPath in volume context, everything seems to be working fine.
The code looks ok as well, but I have a suggestion. internal/cephfs/volumeoptions.go implements, besides other things, functions for creating volumeOptions/volumeIdentifier from following sources:

CSI's CreateVolumeRequest: newVolumeOptions
volume ID: newVolumeOptionsFromVolID
static volume: newVolumeOptionsFromStaticVolume
and lastly, v1 context: newVolumeOptionsFromVersion1Context

Now that support for v1 is discontinued, name of the function newVolumeOptionsFromVersion1Context IMHO loses its meaning. Maybe if not now, it surely will in the future. Would it make sense to rename it to something more descriptive? It creates volumeOptions based on the presence of list of monitor addresses in the volume context - so how about newVolumeOptionsFromMonitorList or something like that? A year from now it might not be all that clear what those options in v1 volume context were, so a better name would definitely help, and also it would guard against accidental deletion of this function during some cleaning of legacy code.
If you agree, then those two comments here could use an update: newVolumeOptionsFromStaticVolume gets mon IPs from the supplied cluster info, and newVolumeOptionsFromVersion1Context (or rather newVolumeOptionsFromMonitorList) retrieves those IPs directly from volume context.

Thanks for confirming @gman0, will get this in and will send a patch to rename the functions.

@Madhu-1 super, thank you!

lgtm! Not scoring as I did not do a detailed review, changes and removal looks fine. Thanks.


OCPBUGS-22974: [release-4.14] manage-security-groups: Only add SGs to LB members (#2455)

Previous code for manage-security-groups were adding the SGs to all the ports attached to the Node. That doesn't make a lot of sense, we only need this traffic opened to the LB members and we only add one port of the Node as the LB member. Moreover we've even tried to add SGs to the ports that had port_security_enabled=false which obviously failed.
This patch makes sure that we only add the SG opening NodePort traffic to the port that is actually plugged into the LB. Moreover it adds a check for port_security_enabled to make sure that ports with it disabled are ignored.

/jira refresh

/jira refresh

/label cherry-pick-approval

/label cherry-pick-approved


get unexpect color

input
import { Hct, argbFromHex, hexFromArgb } from '@material/material-color-utilities';

export const hexToHct = (hex: string): number[] => {
    const hclColor: Hct = Hct.fromInt(argbFromHex(hex));
    return [hclColor.hue, hclColor.chroma, hclColor.tone];
};

export const hctToHex = (hct: number[]): string => {
    const hctColor: Hct = Hct.from(hct[0], hct[1], hct[2]);
    return hexFromArgb(hctColor.toInt());
};

const [hRotate, cTarget, tStart] = [209, 2, 73]
console.log("initial：", [hRotate, cTarget, tStart])
console.log("initial to hctToHex ：", hctToHex([hRotate, cTarget, tStart]))
console.log("initial to hctToHex to hexToHct：",hexToHct(hctToHex([hRotate, cTarget, tStart])))
console.log("initial to hctToHex to hexToHct to hctToHex：",hctToHex(hexToHct(hctToHex([hRotate, cTarget, tStart]))))

output
initial： (3) [209, 2, 73]
initial to hctToHex ： #b4b3b3
initial to hctToHex to hexToHct： (3) [214.8962657784951, 1.6994043852090706, 73.02219038271852]
initial to hctToHex to hexToHct to hctToHex： #b4b3b3

I wonder why 1 and 3 are different and 2 and 4 are the same. Is there some adjustment in the transformation?

Because there may be a gamut clipping (chroma reduction) in the conversion from hct to sRGB, to let the color fall in the sRGB gamut. HCT can express any impossible colors, so we should do chroma reduction to let it correctly display.


Use abacus-moment in integration tests

#228
[#113294537]

Coverage increased (+0.07%) to 93.259% when pulling 9da0766ee0231037ebbac081981e0787a350613d on SAP:ctp-tests into 6386975bba4ab05df574f1a2f2736c6929d46990 on cloudfoundry-incubator:master.

Coverage increased (+0.04%) to 93.23% when pulling fa0653871f00857d19a5921768384996e8326617 on SAP:ctp-tests into 6386975bba4ab05df574f1a2f2736c6929d46990 on cloudfoundry-incubator:master.


Extend auction on last-minute bids

I would like to request the following feature:
Ability to automatically extend auctions for 5 min if a bid comes in within 5 minutes until the end of the auction.
Thanks 🙏

@flyingrabbit-lab - please extend setForSaleNftControl with flag "extendAuctionPeriod" (unix timestamp, same type as auctionFrom). This is undefined by default. If undefined it behaves as it's implemented at the moment.
If set, extend auctionTo and auctionLength with "extendAuctionPeriod" period.

Delivered


linux support(SteamOS)

im using Bazzite.gg's OS, which is a fork of steam os. will steamos be supported?

Outside the scope of this project but there is Linux drivers for activating eGPU. I don't know if bazzite integrates it or how well it works. All things to ask other people.


RefreshIndicator should animate out vertically if no refresh is triggered

Steps to Reproduce
Pull down to get the RefreshIndicator to show, but not enough to trigger the refresh(). Now let go. The refresh indicator animates out using scale instead of leaving the viewport from the top. This makes it unclear if the refresh occured or not.
See how gmail on android for how it should work.

/cc @HansMuller

I have a fix for this. It should be available later today.


Map null in map to undefined for JS database

@nbransby re https://github.com/GitLiveApp/firebase-kotlin-sdk/pull/448#issuecomment-2044330150
If its specific to updateChildren on JS, this should fix it.
Im not sure how JS is supposed to handle null in a map though. Like in EncodersTest.encodeDecodeGenericClass, is it supposed to be
"nullableBool": null or "nullableBool": undefined?

I should note that if I go back to the commit before the Serialization changes where merged, I dont see anything that would do this mapping either.

Yes I had a look myself and couldn't see anything either. All I know is rolling back fixes it, I'll have to investigate it more

Lets see if we can write a test that reproduces it I guess

So far, on master. I havent been able to reproduce using:
@Test
    fun testUpdateChildren() = runTest {
        setupRealtimeData()
        val reference = database
            .reference("FirebaseRealtimeDatabaseTest")
        reference.updateChildren(mapOf("lastActivity" to null))
    }

    private suspend fun setupRealtimeData() {
        ensureDatabaseConnected()
        val firebaseDatabaseTestReference = database
            .reference("FirebaseRealtimeDatabaseTest")

        val firebaseDatabaseChildTest1 = FirebaseDatabaseChildTest("aaa")
        val firebaseDatabaseChildTest2 = FirebaseDatabaseChildTest("bbb")
        val firebaseDatabaseChildTest3 = FirebaseDatabaseChildTest("ccc")

        val values = firebaseDatabaseTestReference.child("values")
        values.child("1").setValue(firebaseDatabaseChildTest1)
        values.child("2").setValue(firebaseDatabaseChildTest2)
        values.child("3").setValue(firebaseDatabaseChildTest3)
        firebaseDatabaseTestReference.child("lastActivity").setValue(1)
    }

given
{
  "rules": {
    ".read": true,
    "FirebaseRealtimeDatabaseTest": {
      "values": {
        ".write": true
      },
      "lastActivity": {
        ".validate": "!newData.exists() || newData.isNumber()",
        ".write": true
      }
    }
  }
}

Whereas reference.updateChildren(mapOf("lastActivity" to "bla")) does fail

I found a bug in OnDisconnect specifically, will make a patch for that once I have testing figured out

Closing cause real fix is here https://github.com/GitLiveApp/firebase-kotlin-sdk/pull/494


Undefined variable $allowRemember

ErrorException
Undefined variable ^
VENDORPATH\lonnieezell\bonfire\src\Views\Auth\login.php at line 26
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
if I change it to @$allowRemember  the next error is
CodeIgniter\View\Exceptions\ViewException
Invalid file: "master.php"
SYSTEMPATH\Exceptions\FrameworkException.php at line 33
26     public static function forEnabledZlibOutputCompression()
27     {
28         return new static(lang('Core.enabledZlibOutputCompression'));
29     }
30
31     public static function forInvalidFile(string $path)
32     {
33         return new static(lang('Core.invalidFile', [$path]));
34     }
35
36     public static function forCopyError(string $path)
37     {
38         return new static(lang('Core.copyError', [$path]));
39     }
40

I'm getting exactley as you are. Just to put it into context, I have been running CI4 in this project for about 6 months. I installed Shield about 2 months ago and installed Bonfire today, all through Composer. This may be an incomatable use case but I'm still investigating. Maybe a config file hasn't been updated properly in the Bonfire install process.

So, master.php is the theme. For some reason Bonfire can't find it. However, I ran out of time. Already spent a few hours on it but I have to get back to work. I've removed Bonfire and may revisit in a few months.

You might want to look at the shield config that is set by a fresh bonfire install. E.g., Shield views are not automatically compatible with Bonfire themes.

Not sure if this has been resolved but I had same issue today after installing Bonfire2 for a trial. Resolved it by commenting out the following line from \App\Config\Routes.php file.
// service('auth')->routes($routes);
This line was initially part of shield setup which I had before installing Bonfire2.


Update README.md

@vinayak017  i am rejecting this PR. this has not contribution to the community or users. to start contributing I suggest to view open issues under the issue tab with "good first issue" label


Version UI Updates: add working version to version table

Description

Add the working version of the experiment as the first row with all actions replaced by "create version"
User Impact

n/a
Demo

n/a
Docs

n/a

Codecov Report
Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

apps/experiments/views/experiment.py
0.00%
3 Missing :warning:

Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!

🚨 Try these New Features:

JS Bundle Analysis - Avoid shipping oversized bundles


Added S3 Config Credentials

It is possible to pass S3 credentials and bucket with command line.
So it could be used as a "one liner".
Take a look if it makes sense for your use-case.

Unfortunately, this change doesn't quite work in my case. I have s3cmd configured already, thus passing on credentials and server directly to the command is something I don't need and which should be optional.
https://github.com/opencast/opencast-rpmbuild/blob/c213d84ad9857c95bce9626fe6ebfa3151ae96c4/.github/workflows/build-rpms.yml#L92-L104
This also misses a few adjustments e.g. in the requirements section of the readme which change with this pull request.
Can you make the new arguments optional and adjust the documentation?


Using xlink:href attribute with SVG <use> will throw type error, but this is legal in the specification

Describe the bug
In specification, the xlink:href and xlink:title attributes of SVG <use> element are legal, but they throw type error in JSX.
Type '{ "xlink:href": string; }' is not assignable to type 'UseSVGAttributes<SVGUseElement>'.
  Property 'xlink:href' does not exist on type 'UseSVGAttributes<SVGUseElement>'.

The fill attribute is the same as xlink:href:
Type '{ id: string; fill: string; "fill-rule": string; "xlink:href": string; }' is not assignable to type 'UseSVGAttributes<SVGUseElement>'.
  Property 'fill' does not exist on type 'UseSVGAttributes<SVGUseElement>'.

Your Example Website or App
https://stackblitz.com/edit/solidjs-templates-qt8jjs?file=src%2FApp.tsx
Steps to Reproduce the Bug or Issue
In the example, you can see the xlink:href and fill attributes throw type error.
Expected behavior
Not throw type error.
Now I refer to MDN and write a type to avoid type errors:
// types.d.ts
// SolidJS
import { JSX as _JSX } from 'solid-js';

declare module 'solid-js' {
  namespace JSX {
    interface UseSVGAttributes<T>
      extends _JSX.CoreSVGAttributes<T>,
        _JSX.StylableSVGAttributes,
        _JSX.ConditionalProcessingSVGAttributes,
        _JSX.GraphicsElementSVGAttributes<T>,
        _JSX.PresentationSVGAttributes,
        _JSX.ExternalResourceSVGAttributes,
        _JSX.TransformableSVGAttributes {
      x?: number | string;
      y?: number | string;
      width?: number | string;
      height?: number | string;
      href?: string;
      /** @deprecated */
      ['xlink:href']?: string;
      /** @deprecated */
      ['xlink:title']?: string;
    }
  }
}

Screenshots or Videos
No response
Platform

OS: Mac OS
Browser: Edge
Version: SolidJS latest

Additional context
I am a big fan of SolidJS, I have cloned the codebase and saw that the JSX type jsx-runtime.d.ts is copied from dom-expressions.
I'd like to fix this type error, but I don't know the jsx-runtime.d.ts is copied from dom-expressions/src/jsx-h.d.ts or dom-expressions/src/jsx.d.ts, which is the finally type?
Could you please tell me how can I fix it?

Oh I am sorry, the xlink:href attribute is deprecated, so it's should not be fixed, but maybe the type error of the fill attribute should be fixed?

Oh I am sorry, the xlink:href attribute is deprecated, so it's should not be fixed, but maybe the type error of the fill attribute should be fixed?

Typings are here,
https://github.com/ryansolid/dom-expressions/blob/main/packages/dom-expressions/src/jsx.d.ts
theres also the h ones https://github.com/ryansolid/dom-expressions/blob/main/packages/dom-expressions/src/jsx-h.d.ts

Oh I am sorry, the xlink:href attribute is deprecated, so it's should not be fixed, but maybe the type error of the fill attribute should be fixed?

Typings are here, https://github.com/ryansolid/dom-expressions/blob/main/packages/dom-expressions/src/jsx.d.ts theres also the h ones https://github.com/ryansolid/dom-expressions/blob/main/packages/dom-expressions/src/jsx-h.d.ts

Meaning jsx.d.ts is the same as jsx-h.d.ts, right?

With small differences but yeah, you can open a PR there to add the missing ones


Since THE WEIGHT CLASS I DOWNLOADED IS 80 and I need TO BE 2, how do I not load this layer for training, or what is a good idea

Prerequisite

[X] I have searched the existing and past issues but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version.

🐞 Describe the bug
Since THE WEIGHT CLASS I DOWNLOADED IS 80 and I need TO BE 2, how do I not load this layer for training, or what is a good idea
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for YOLOX:
size mismatch for bbox_head.multi_level_conv_cls.0.weight: copying a param with shape torch.Size([80, 128, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 128, 1, 1]).
Environment
size mismatch for bbox_head.multi_level_conv_cls.0.weight: copying a param with shape torch.Size([80, 128, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 128, 1, 1]).
Additional information
size mismatch for bbox_head.multi_level_conv_cls.0.weight: copying a param with shape torch.Size([80, 128, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 128, 1, 1]).

In training, simply set load_from to be the path of the weight should be fine, those mismatched weights will be ignored.


ENV var forwarding

In my understanding it is currently not possible to access env vars from the host environment, which seriously limits the usability of WinterJS on Edge - since you really need to read env vars for things like API keys.

This was already implemented yesterday in e6ef1cf0aa569a3f54b3bfb645e5f137c4bef79f.


[backport] Improve atomicAdd in histgram and sample

Backport of #1345

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 5382e4dd73acae391f2be0946e548b39b151ba87, target branch v6) succeeded!


Etcdv3: service kv.Range: how to get the newest events from the pointed resourceversion

Firstly, Thanks for your attention.
We can use service watch.Watch with resourceversion to get events that have happened(from the pointed resouceversion), but in common we like to get all of the newest events from a resourceversion in one request like kv.Range. and i have carefullly read the api reference doc，but have no idea how to implement it. can you give me some advices.

If you do not specify revision, watch will watch from current revision. If you also want to get the current kvs besides watch, you can do a range first, and then watch from the revision of the header of the range request.

@xiang90  thank you for replying
watch is a long polling action, and i‘d like to complete it in a short grpc request(like range).
if i want to get events from specified revision to current revision, how can i achieve it.

There is no way to achieve that for now.
We might add maxRev as an option for watch. Watch will terminate once it reaches the maxRev. To achieve what you want, you need to do a range on whatever key to get the current rev in header. And then you can do watch(/foo, withMaxRev(rev)). Then this watch will not block, it will just like a range that walks through previous history.
What is your use case for this?

if i want to get events from specified revision to current revision, how can i achieve it.

@helinbo2015 Can you tell us more about this? What is your potential use case?

@xiang90 Thanks a million.

Can you tell us more about this? What is your potential use case?

i  had optimized kubernetes's list-watch function. when watch failed, relist will not catch all  of objects, instead returning objects that from specified version  to current version.
and as a solution, i had added a relist function(return objects from specified version to current version) in kube-apiserver component.

and as a solution, i had added a relist function(return objects from specified version to current version) in kube-apiserver component.

No. This wont work. Relist is only required when the history is compacted. If the history is compacted you have to do a relist. What you described will lead to inconsistent state.
If you want to optimize k8s storage, you should open an issue at k8s side. I am a maintainer of k8s. We can discuss there.


[BUG] content shakes during animation in TuiActionModule

🐞 Bug report
Description
Content shakes during animation. Adding will-change will fix such behaviour.
Reproduction
https://taiga-ui.dev/tui-action/Setup
https://user-images.githubusercontent.com/6787602/104334993-09f1be00-5504-11eb-8b2c-4991ad4354c2.mov
Versions

OS: macOs
Browser Google Chrome Version 87.0.4280.141
TuiActionModule

Additional context

@aalexeev239 thanks for spotting and providing a solution! Perhaps you will also be able to make a PR? :)

@aalexeev239 thanks for spotting and providing a solution! Perhaps you will also be able to make a PR? :)

@waterplea ☝

@waterplea ☝


BUG-000105205  Error Results not calculated. Check input parameters and try again when using image service with RLOS command in Military Tools Toolbar

Salesforce ID: BUG-000105205
Salesforce Submitter: Andrew Johnson
Salesforce Submit Date: 5/17/2017 7:56 AM
Salesforce Bug Type: Failure/Error
Salesforce Severity: Medium
Repro Data: (n/a)
Work Around: (n/a)
Product: ArcGIS for Desktop
Functional Category: ArcGIS for Defense
Client Platform: (n/a)
Version Found: 10.5
Planned Version Fixed: (n/a)
Comment: (n/a)

This should be resolved with #184

This issue was resolved in #184


Cellular response to protozoan

PMID:16369915; Knockdown of IRAK1 using IRAK1 siRNA resulted in impairment of TNF production by Leishmania donovani-infected macrophages.
PMID:19104650; Upon stimulation with TLR4 agonist, IRAK1 associated with MYD88 but failed to associate with TRAF6 in Leishmania donovani-infected macrophages.
Reported by: cusin
Original Ticket: geneontology/ontology-requests/10438

Added - but worth noting that Protozoan is a very mixed class (no longer supported by taxonomies/phylogenies), so maintenance of classes under this will have to be manual.


Decimal point doesn't work in spanish

My hunch is that it is because europeans use , instead of .

Opps wrong repo


add: add the copyright header

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
This PR adds the copyright header, which we usually add even in the sub projects.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Maybe the folder migration first https://github.com/kubernetes-sigs/kube-scheduler-wasm-extension/pull/18 for you have to modify again. but I'm ok with both approaches.

/lgtm
/approve
/hold
Leave it to you.

@kerthcet Sorry for bothering you again, PTAL 🙏

My pleasure.
/lgtm

/unhold


Increasing the log verbosity

I'm trying to run SPM on a dataset I unfortunately cannot share and I am getting the following error:
SPM12, version 7048 (standalone)
  MATLAB, version <IP_ADDRESS>8062 (R2017a)
   ___  ____  __  __
  / __)(  _ \(  \/  )
  \__ \ )___/ )    (   Statistical Parametric Mapping
  (___/(__)  (_/\/\_)  SPM12 - http://www.fil.ion.ucl.ac.uk/spm/

  Execution failed: /opt/spm12/spm_BIDS_App.m
  Error using spm_BIDS (line 56)
  BIDS dataset description could not be read.
  Error in spm_BIDS (line 56)
  Error in spm (line 1206)
  Error in spm_standalone (line 115)

Would there be a way to increase verbosity of the logs?

The error you get means that there was an issue with the dataset_description.json file: either the JSON parsing failed or one (or both) of the mandatory fields BIDSVersion or Name was missing.
If it does not help, could you at least share the dataset_description.json file?
I'm also going to update the app to a more recent version of SPM12 dev.


Support SWR

Updated: Since SWR doesn't provide hooks like useMutation in React Query, and it seems very flexible to use mutate in SWR, this PR will not provide SWR generated code about mutations

Add new argument Library to CLI for pointing out the specific library to generate code (choices: 'swr', 'react-query', default: 'react-query')
Update tests
Update README

Since SWR doesn't provide hooks like useMutation in React Query, and it seems very flexible to use mutate in SWR, this PR will not provide SWR generated code about mutations

Add new option --swr to command for generating code with SWR hooks
Update tests
Update the document

Hello @yuleicul, thanks for the contribution. A few points before we proceed:

Can we split everything related to SWR into it's own folder? Ideally we wouldn't have the 2 (RQ & SWR) mixed together in the same files. So it would be great if you can add a directory called swr and create files for everything that's needed and leave all the other files untouched if possible.
For the CLI options, instead of adding --swr, we should have a command to specify either one or the other. So something like rapini react-query --option1 --option2 ... or rapini swr --option1 --option2 - that way it will prevent the user from using rq options with swr and vice versa, so preventing confusion. (And keep the old way of specifying commands the same to be backwards compatible)

So far it's great, thanks, keep it up!

Hi @rametta , thanks for your review. Your comments are valuable and can make this project robuster. I'll try to refactor it in the next few weeks.

Hi @rametta, updated, please review. Thank you!


TEST

💥 Proposal
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)

/CPP


Wagtail Image Upload Doesn't Prune Browser Memory On Large Uploads

Issue Summary

Steps to Reproduce

Navigate to Wagtail Admin
Navigate to images
Attempt to upload via the browser over 3999 images; generally that should suffice to crash with an "out of memory" error.

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
When image uploads exceed a specific pre-defined quantity Wagtail image import could be adjusted to accelerate image importing by avoiding presentation layers on the webpage.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.10
Django version: 4.1.6
Wagtail version: 4.10
Browser version: Chrome - Version 110.0.5481.77

@iamstrained what browser? You have a version but not which one. My guess is chrome based on the number but please update the issue report.
Also - any screenshots and testing on other browsers would be helpful.
My guess is this is something that sits at the jQuery file upload plugin level.

Or is this a server side error?

@iamstrained what browser? You have a version but not which one. My guess is chrome based on the number but please update the issue report.
Also - any screenshots and testing on other browsers would be helpful.
My guess is this is something that sits at the jQuery file upload plugin level.

I have updated the Chrome; apologies. And yes, I believe it's a Upload Plugin issue. If a different modal/presentation process were adopted it would resolve the issue. I am for example uploading 7k AI.Art generated iamges and have to either do it on the side (programmatically) or you would need to be very patient on the web-uploader processing. [And you must manually adjust the page down after a certain quantity to keep it "firing" the upload.
I don't have a handy copy of the original "out of memory" error.
Work around has been - reduce batch size in the online version (<~2k works fine; takes way longer than a FTP + Programmatic sideload. But the Wagtail/Django DB doesn't enjoy getting sideloaded in the trials I've done.
(e.g., Image.Save() with a 'rb' load).

Or is this a server side error?

I believe this is a browser based issue after a certain point where the page is attempting to hold too much in memory. I would have to re-fire a large batch to try to re-prompt the error.


Create Project doesn't have any images or css

Have just installed using the create-project command
ran the install command
When i go to the admin area the stylesheets and images are missing (apart from the one logo)

@bretto36 It looks like an issue related to dumped assets (we're using Gulp). I pushed many minor & major changes recently so please download the newest version.


Handle DLLs for Symlinked executables

Description of the new feature / enhancement
With current zip support, the only types of installer allowed are:

Portable standalone exe files
Compressed installers

But what about supporting regular compressed binary packages, such as those having exe along with dynamic dll libraries?
Proposed technical implementation details
To use the dynamic dll libraries, they must either be:

in PATH environment variable
linked by specifying Files -> RelativeFilePath within the installer manifest

It is not possible to link dll files to make it work using the 2 approach, because the links are forced to have .exe extension.
This could be fixed either by:

Adding the dlls folder to PATH environment variable
Allowing to link dll files specifying something like NestedInstallerType: library. Then it would also be useful to provide pattern matching RelativeFilePath, such as *.dll
Instead of making a symlink inside WinGet\Links, make an executable bat with the command @call "path\to\exe" %*, then the working directory will be the same where the exe actually is (this would the easiest path in my opinion)

Another way, maybe easier would be instead of making a symlink inside WinGet\Links, make an executable bat with the command @call "path\to\exe" %*

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced. The DLLs aren’t needed in the installer files because they aren’t installers

Maybe I wasnt' clear enough. By binary files I mean plain binary, not an exe installer, that's the case for https://github.com/microsoft/winget-pkgs/pull/88065/files. What I mean is plain binary packages, which is just a bunch of exe files along with dll files, many development tools are deployed this way such as nginx, php, etc. And they won't run unless:

They are launched from the same directory where they were extracted, which a symlink will break
They are added to PATH
They are launched through a .bat file as I proposed: this strategy is used throughout many different packages.

If the community agrees with the bat implementation I would gladly like to contribute. I think it is much better than polluting the PATH or forcing the manifest to declare a whole bunch of DLL files which could also drive other side effects such as DLL versions conflicts.

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced. The DLLs aren’t needed in the installer files because they aren’t installers

Maybe I wasnt' clear enough. By binary files I mean plain binary, not an exe installer, that's the case for https://github.com/microsoft/winget-pkgs/pull/88065/files. What I mean is plain binary packages, which is just a bunch of exe files along with dll files, many development tools are deployed this way such as nginx, php, etc. And they won't run unless:

They are launched from the same directory where they were extracted, which a symlink will break
They are added to PATH
They are launched through a .bat file as I proposed: this strategy is used throughout many different packages.

If the community agrees with the bat implementation I would gladly like to contribute. I think it is much better than polluting the PATH or forcing the manifest to declare a whole bunch of DLL files which could also drive other side effects such as DLL versions conflicts.

I think you should look at the Paint.Net zip file more closely.
If you comment out all the other entries in the manifest, and then use winget to install this specific entry, it will still run perfectly fine

@Trenly, it does work if you install it and launch it through run dialog (Win + R) paint.net.exe, but that's because  somehow it looks like it follows the symlink working directory somehow, but if you try to launch it through cmd or powershell it won't work at all!
Although this is fine for "Desktop Apps", it is not the expected behavior for development tools and won't work for "CLI apps"

@Trenly, it does work if you install it and launch it through run dialog (Win + R) paint.net.exe, but that's because somehow it looks like it follows the symlink working directory somehow, but if you try to launch it through cmd or powershell it won't work at all!
Although this is fine for Desktop Apps, it is not the expected behavior for development tools and won't work for CLI apps.
But even for Desktop Apps you need to specify paint.net.exe which is not what I would expect to type in the run dialog, I usually just type paint.net, code, etc. I do not type the .exe, I think this is another issue with the SymLinking strategy.

It works fine for me from powershell when I use start paint.net

That's my point, start shouldn't be needed at all, you won't be running development tools using start by default, besides you probably won't be launching these development tools yourself but they will launched by other development tools which you don't have control over.

Ahhh, I see. Thank you for your patience and the clarification!

The issue title should be better renamed to something like "Handle DLLs for symlinked executables."

This could be fixed either by:

Adding the dlls folder to PATH environment variable
Allowing to link dll files specifying something like NestedInstallerType: library. Then it would also be useful to provide pattern matching RelativeFilePath, such as *.dll
Instead of making a symlink inside WinGet\Links, make an executable bat with the command @call "path\to\exe" %*, then the working directory will be the same where the exe actually is (this would the most easier path in my opinion)

Another solution would be to use shims, like Scoop or Chocolatey:

https://github.com/microsoft/winget-cli/issues/361

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced. The DLLs aren’t needed in the installer files because they aren’t installers

Maybe I wasnt' clear enough. By binary files I mean plain binary, not an exe installer, that's the case for Paint.Net. What I mean is plain binary packages, which is just a bunch of exe files along with dll files, many development tools are deployed this way such as nginx, php, etc. And they won't run unless:

They are launched from the same directory where they were extracted, which a symlink will break
They are added to PATH
They are launched through a .bat file as I proposed: this strategy is used throughout many different packages.

If the community agrees with the bat implementation I would gladly like to contribute. I think it is much better than polluting the PATH or forcing the manifest to declare a whole bunch of DLL files which could also drive other side effects such as DLL versions conflicts.

Use bat might cause problem while execute it in another bat script.

I really don't know, I'm not a BAT guru. But AFAIK @call does not cause any side effects inside other bat files, it won't change the running environment as a regular call and as the content is just a single call to an exe and won't change the any variable, it will be just like calling the exe itself which could also break/stop the bat itself.

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced. The DLLs aren’t needed in the installer files because they aren’t installers

Maybe I wasnt' clear enough. By binary files I mean plain binary, not an exe installer, that's the case for Paint.Net. What I mean is plain binary packages, which is just a bunch of exe files along with dll files, many development tools are deployed this way such as nginx, php, etc. And they won't run unless:

They are launched from the same directory where they were extracted, which a symlink will break
They are added to PATH
They are launched through a .bat file as I proposed: this strategy is used throughout many different packages.

If the community agrees with the bat implementation I would gladly like to contribute. I think it is much better than polluting the PATH or forcing the manifest to declare a whole bunch of DLL files which could also drive other side effects such as DLL versions conflicts.

Use bat might cause problem while execute it in another bat script.

I really don't know, I'm not that versed in BAT scripts. But AFAIK @call does not cause any side effects inside other bat files, it won't change the running environment as a regular call and as the content is just a single call to an exe and won't change any variable, it will be just like calling the exe itself which could also break/stop the bat by itself.
I suggested this solution because it is used by many development tools such as Anaconda, python tools, msys, osgeo4w, this is not a clever hack that came out of my mind.

iirc, the entire Zip folder is extracted and moved to the default install location. Take the Paint.Net Pull Request for example. If you comment out everything except for the zip-portable installer entry, it still installs and runs correctly, meaning that all the DLLs needed to run it are referenced. The DLLs aren’t needed in the installer files because they aren’t installers

Maybe I wasnt' clear enough. By binary files I mean plain binary, not an exe installer, that's the case for Paint.Net. What I mean is plain binary packages, which is just a bunch of exe files along with dll files, many development tools are deployed this way such as nginx, php, etc. And they won't run unless:

They are launched from the same directory where they were extracted, which a symlink will break
They are added to PATH
They are launched through a .bat file as I proposed: this strategy is used throughout many different packages.

If the community agrees with the bat implementation I would gladly like to contribute. I think it is much better than polluting the PATH or forcing the manifest to declare a whole bunch of DLL files which could also drive other side effects such as DLL versions conflicts.

Use bat might cause problem while execute it in another bat script.

I really don't know, I'm not that versed in BAT scripts. But AFAIK @call does not cause any side effects inside other bat files, it won't change the running environment as a regular call and as the content is just a single call to an exe and won't change any variable, it will be just like calling the exe itself which could also break/stop the bat by itself.
I suggested this solution because it is used by many development tools such as Anaconda, python tools, msys, osgeo4w, this is not a clever hack that came out of my mind.

If you run the second bat script directly in the first bat script, the remaining commands in the first script will not be executed after the second script is executed(even if you use the @call command in the second script). Unless the call command is also used when invoking the second bat script in the first bat script. But bat scripts from the network and the average users doesn't know this, because the difference doesn't occur if a exe file is invoked. In addition, Microsoft documentation shows that the use of the @call command is not recommended with redirection and pipes, so I'm not sure if this will affect the behavior of using redirection and pipes when running a bat script containing the call command as a command.

+1
I've got a .NET Core 7.0 app

but winget isn't capable of handling its XCOPY-Install scenario. Trying it as a 'portable' app
Installers:
- InstallerUrl:         https://github.com/DrusTheAxe/AppData/releases/download/2.0.0/AppData-2.0.0.zip
  Architecture:         x64
  InstallerType:        zip
  NestedInstallerType:  portable
  NestedInstallerFiles:
  - RelativeFilePath:   appdata.exe
  InstallerSha256:      7717bd116f15e6debfc581e6bde36ec8854e60eced1c9cb5c5496a7f640af0f3

only ends in tears

Looking forward to a solution

Since no one seems to have posted it here, there is a workaround, kindly described by @mdanish-kh  here:
"A not so pretty workaround (as it has the tendency to quickly fill up your PATH variable limit) is to install the application in a non-admin shell with developer mode disabled in Windows. In this case, WinGet cannot create a symlink and will resort to putting the entire package path (i.e., %LOCALAPPDATA%\Microsoft\Winget\Packages) into the PATH variable. That way all the required libraries, .dlls will be referenced properly."

It look like another problem with the winget while during install,  https://download.wsusoffline.net/wsusoffline120.zip

Well the PATH issue can be mitigated by having separate variable for winget package path eg WINGET so you can add stuff to path like %WINGET% or another option is to hold all winget path variables in WINGET variable and just add that to path. You can extend path by just chaining variables togather.https://stackoverflow.com/questions/34491244/environment-variable-is-too-large-on-windows-10

Any progress on this issue? Seems to be blocked many, many packages.

I believe we've resolved this with:

https://github.com/microsoft/winget-cli/pull/4816

It would likely involve uninstalling the package and reinstalling it (if it's installed and not working), but this should unblock a fairly large number of packages in the community repository.

https://github.com/microsoft/winget-pkgs/labels/zip-binary

Should be worth mentioning that existing packages affected by this issue like scrcpy needs to have their manifests  patched with this new option, and only then reinstalling them with the new winget version should work.

@denelon do you have an example of where specifically to add ArchiveBinariesDependOnPath and i'll go patch my packages today?


Use cross-env for environment variables

Use cross-env package to safely set unix-style environment variables on Windows.
Also bumps js-utility-belt to latest to be safe in the future.

The safeInvoke breaking change in the latest js-utility-belt doesn't affect this project, so going to merge.