Integration Test Record

Fixed the Integration test, because the req.body of createRecord has been changed. Comment all the functions about the has LinkAccount in the createRecord and comment the unit test and integration test that is testing the situation of LinkAccount

[ ]  Does the code work?
[ ]  Are functions, methods, and variables adequately named?
[ ]  Any test on the new function?
[ ]  Is it well tested? It should pass all jest tests.
[ ]  Is the code clear and easy to understand?
[ ]  Does it meet the team guidelines and style guides?
[ ]  Is there any duplicated code?
[ ]  Is documentation on functions, methods, classes, contexts, and behaviours adequate?
[ ]  Are there better or simpler solutions?
[ ]  Is there any performance issue?

Updated backend record route & controller testing functions


Hello,w hen will stream Chinese support/?

Hello,w hen will stream Chinese support/?

tomorrow.

Please see
https://k2-fsa.github.io/sherpa/onnx/pretrained_models/lstm-transducer-models.html#csukuangfj-sherpa-onnx-lstm-zh-2023-02-20-chinese
We are adding more pre-trained models in addition to LSTM transducer.


Replace the use of Globals with Singletons

Currently the validator and util are using global values to store and persist data akin to a singleton.
https://github.com/jondavid-black/AaC/blob/050ca9ae28230c11c54c1e4163480037be14fc30/src/aac/validator.py#L12-L13
https://github.com/jondavid-black/AaC/blob/050ca9ae28230c11c54c1e4163480037be14fc30/src/aac/util.py#L16-L18
AC:

[ ] Validator.py globals are converted to singleton(s) that's defined and used inside the validator.py
[ ] The util.py globals are converted to a singleton(s) and moved to a descriptive module, like acc_spec
[ ] The singletons all expose a clear function that will allow users to clear the current instance (largely for testing use)

Upon further research, this issue appears to be flawed. Python doesn't seem to support singletons in the way we'd expect, though it does seem technically possible.
Since the validator globals were dealt with, I think the real remaining issue is that util is being used to supply the core spec when we should instead move that to its own module

I've addressed the remaining issues related to this issue in #93. Refer to that issue from now on.


Extend the BLS library to support also min_pk using macros

Extending the BLS code to work with both modes.
All objects exist for both modes, under different internal modules: min_sig and min_pk.

Ok I see, so the fastcrypto dependency will be something like fastcrypto::bls12381::min_sig::BLS12381KeyPair, and the end user doesn't know anything about macros etc?

@kchalkias yes, similar abstraction to what we had so far, just with additional modules to differentiate between the modes


chore(main): release 0.0.8

:robot: I have created a release beep boop
0.0.8 (2023-08-16)
Features

add password-rotation Lambda module and utilize it in the examples/complete root module (#311) (97cfb65)

Bug Fixes

add eks 1.27 & gp3 storage class (#325) (b4ecfb1)
ci: Change the parse job in test and update workflows to always use main (#332) (2a92fae)
remove big bang dependencies (#325) (b4ecfb1)
stop using spot instances in the example root module (#329) (656cee6)
upgrade addon versions (#325) (b4ecfb1)

Documentation

adr: add ADR for how to trigger automated tests (#321) (49d1f77)

Miscellaneous Chores

deps: enable renovate updates in examples/complete/fixtures.common.tfvars for Zarf, EKS, aws-node-termination-handler, kubernetes autoscaler chart and underlying image, metrics-server, and calico (#309) (1e48f68)
deps: help Renovate get around Checkov's IP block (#324) (71f4534)
deps: Upgrade aws/aws-sdk-go from v1.44.293 to v1.44.299 (#305) (7cfec56)
deps: upgrade awscli from 2.13.0 to 2.13.1 (#313) (f4a7cae)
deps: upgrade awscli from v2.12.5 to v2.13.0 (#305) (7cfec56)
deps: upgrade defenseunicorns/build-harness from 1.7.0 to 1.8.0 (#313) (f4a7cae)
deps: upgrade defenseunicorns/build-harness from v1.4.2 to v1.7.0 (#305) (7cfec56)
deps: upgrade defenseunicorns/zarf from 0.28.1 to 0.28.2 (#313) (f4a7cae)
deps: upgrade defenseunicorns/zarf from v0.28.0 to v0.28.1 (#305) (7cfec56)
deps: upgrade github.com/aws/aws-sdk-go from 1.44.299 to 1.44.301 (#313) (f4a7cae)
deps: upgrade github.com/defenseunicorns/terraform-aws-uds-eks from 0.0.1-alpha to 0.0.2 (#313) (f4a7cae)
deps: upgrade github.com/defenseunicorns/terraform-aws-uds-vpc from 0.0.2-alpha to 0.0.2 (#313) (f4a7cae)
deps: upgrade github.com/terraform-aws-modules/terraform-aws-lambda from 5.0.0 to 5.3.0 (#313) (f4a7cae)
deps: upgrade golang from 1.20.5 to 1.20.6 (#313) (f4a7cae)
deps: upgrade gruntwork-ioterratest from v0.43.6 to v0.43.8 (#305) (7cfec56)
deps: upgrade renovatebot/pre-commit-hooks from 36.7.0 to 36.10..0 (#313) (f4a7cae)
deps: upgrade renovatebot/pre-commit-hooks from v35.147.0 to v36.7.0 (#305) (7cfec56)
deps: upgrade terraform from 1.5.2 to 1.5.3 (#313) (f4a7cae)
release 0.0.8 (6011188)

Code Refactoring

automation: refactor parse logic to make it a bit simpler (#323) (113758a)
automation: refactor the update-command workflow so that the bulk of the logic is in a reusable action, since it will now be used in 2 different places (#323) (113758a)
automation: simplify the if statements now that we aren't trying to run tests on push to main anymore (#323) (113758a)
pr-automation: simplify the GitHub context name that the E2E tests use so that we can require them regardless of how they were triggered (#323) (113758a)

Continuous Integration

main: delete the pre-commit-trunk workflow now that we aren't trying to run it on commits to main (#323) (113758a)
pr-automation: delete the auto-labeling workflow (#323) (113758a)
release: change release-please config to bump patch for pre-1.0.0 minor changes (#331) (a556560)
release: fix auto-tagging workflow previously released (#308) (12310eb)
release: update automated release process to use release-please (#326) (bd2cf4d)
test: Add ability to run Secure E2E and Insecure E2E tests separately, and only run Insecure test in commercial and Secure test in govcloud (#333) (bd2e23a)
test: change commercial primary test region from us-east-1 to us-east-2 (#313) (f4a7cae)
test: change commercial secondary test region from us-east-2 to us-east-1 (#313) (f4a7cae)
test: update the auto-test workflow to reflect the decisions documented in ADR 8 (#323) (113758a)

This PR was generated with Release Please. See documentation.

/test all

/test all

/test all

:robot: Release is at https://github.com/defenseunicorns/delivery-aws-iac/releases/tag/v0.0.8 :sunflower:


⚠ Processors has degraded performance

In 21346f2, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 49 ms

Resolved: Processors performance has improved in 7d501ec.


cannot format result.json

right click on result.json:

This has been fixed already, I believe originally in this commit: https://github.com/Microsoft/vscode-mongodb/commit/ae2b7f3be797fffb68399ff9d9b26faa6929112d


Support for System.Runtime.CompilerServices.Caller*

System.Runtime.CompilerServices.CallerMemberNameAttribute
System.Runtime.CompilerServices.CallerFilePathAttribute
System.Runtime.CompilerServices.CallerLineNumberAttribute

Steps To Reproduce
https://deck.net/e961191cff47e1ebe9472c8d378325f9
Expected Result
https://dotnetfiddle.net/s9fxHh
member name: foo
source file path: d:\Windows\Temp\15ti1t1k.0.cs
source line number: 4

Actual Result
member name:
source file path:
source line number: 0

https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/concepts/caller-information

Thanks for the request, and sorry for taking so long for reviewing it. This seems to be feasible, given source maps support is present, then there would be ways into getting source files number, and at least some path reference.


invalidfree when use _nss_dns_gethostbyname4_r

stack is
(gdb) bt
#0  0x00007ffff49567bb in raise () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x00007ffff4941535 in abort () from /lib/x86_64-linux-gnu/libc.so.6
#2  0x00007ffff7bfea66 in tcmalloc::Log(tcmalloc::LogMode, char const*, int, tcmalloc::LogItem, tcmalloc::LogItem, tcmalloc::LogItem, tcmalloc::LogItem) () from /home/app/libtcmalloc.so.4
#3  0x00007ffff7bf3644 in (anonymous namespace)::InvalidFree(void*) () from /home/app/libtcmalloc.so.4
#4  0x00007fffd0ac0240 in _nss_dns_gethostbyname4_r () from /lib/x86_64-linux-gnu/libnss_dns.so.2
#5  0x00007ffff4a01e0a in ?? () from /lib/x86_64-linux-gnu/libc.so.6
#6  0x00007ffff4a02e37 in getaddrinfo () from /lib/x86_64-linux-gnu/libc.so.6
#7  0x000000000051ec04 in Curl_getaddrinfo_ex ()
#8  0x00000000005205fc in getaddrinfo_thread ()
#9  0x000000000054bf4b in curl_thread_create_thunk ()
#10 0x00007ffff5bd0fa3 in start_thread () from /lib/x86_64-linux-gnu/libpthread.so.0
#11 0x00007ffff4a184cf in clone () from /lib/x86_64-linux-gnu/libc.so.6
and my way is by LD_PRELOAD
thank you for your  answer
@alk @tmm1

i found problem;
cause i use libcurl.a and libcurl.so in different library. modify .a to .so is correct.
really reason who know it can explain it,thank you


serde-derived Serialize/Deserialize impl for Value is not working as expected

The Value type is expected to serialize/deserialize from data it represents. For the following test code:
    fn test_deserialize_null_value() {
        let val = crate::core::Value::Null;
        let ret = crate::serde::to_vec(vec![], &val).unwrap();
        eprintln!("{:02x?}", &ret);
    }

Expected Output:
[f6]

which is the primitive value null, but
Actual Output:
[64, 4e, 75, 6c, 6c]

which is a 4-character UTF-8 text Null.
It looks like the generated implementation from the derive directives interprets Value as a externally tagged enum. Is it a mistake or a feature by design?

Ah, this is a bug. thanks for your report.

The 0.2.4 version has been released and it fixes this problem.

@quininer thanks for your prompt fix! 0.2.4 works perfectly now.


feat: allow cpu cores to be fractional whole cores and millicores

The Redpanda Helm chart is great in doing validations and ensuring sane values are used. The validation of CPU and Memory requests and limits is a wonderful safety net to ensure best experience with Redpanda. However, users should be able to go outside of recommended values.
While there is a way to override the chart's memory minimums, it's not possible to set CPU value to anything below a full core, due to current values schema constraints, and the way cores are passed to the --smp CLI flag.
Let's allow advanced users to experiment with Redpanda in K8s as they are able to do when running Docker images on their local workstations.

updating values schema to allow integers and strings for resources.cpu.cores
adding redpanda-cores-in-millis named templates to convert the resources.cpu.cores to millicores
adding redpanda-smp named template to calculate the value passed to --smp CLI argument, while defaulting to 1, and making sure to automatically set resources.cpu.overprovisioned: true when working with less than 1 full core, and round down to number of full cores when working with fractional cores over 1 full core
adding redpanda-cpu-warning named template to warn on lower than recommended value of cores, and adding it to the list of warnings

Related to https://github.com/redpanda-data/redpanda/issues/350.
Perhaps, when working with less than 1 full core the --smp flag should not be passed altogether?
Let's me know and I can make that change.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

current implementation prohibits setting CPU cores to less than 1 full core
❯ helm template redpanda redpanda/redpanda \
  --set config.node.developer_mode=true \
  --set resources.cpu.cores=200m \
  --set resources.memory.container.max=576Mi \
  --set  resources.memory.redpanda.memory=500Mi \
  --set resources.memory.redpanda.reserveMemory=64Mi
Error: values don't meet the specifications of the schema(s) in the following chart(s):
redpanda:
- resources.cpu.cores: Invalid type. Expected: integer, given: string

❯ helm template redpanda redpanda/redpanda \
  --set config.node.developer_mode=true \
  --set resources.cpu.cores=0.2 \
  --set resources.memory.container.max=576Mi \
  --set  resources.memory.redpanda.memory=500Mi \
  --set resources.memory.redpanda.reserveMemory=64Mi
Error: values don't meet the specifications of the schema(s) in the following chart(s):
redpanda:
- resources.cpu.cores: Invalid type. Expected: integer, given: string

@joejulian , I take it the failing version bump check will be ignored by whomever merges the PR? And I should not be bumping it myself?

Feel free to bump it, but if you don't, we'll help.

Feel free to bump it, but if you don't, we'll help.

@joejulian, is 2.10.0 alright (minor version bump)? Or do you prefer 2.9.2 (patch version bump)?

Good point. New feature so minor bump.


Luci-Theme: LuCI Master 24.319.72217~9fe1e34 is not providing theme selections in the theme dropdown

Is there an existing issue for this?

[X] I have searched the existing issues

screenshots or captures

LuCI Master 24.319.72217~9fe1e34 is not providing theme selections in the theme dropdown. This is preventing the any settings changes in the System Tab from being saved.
Actual behaviour

Opened System menu in LUCI
Changed router name <General Settings - Hostname>
Click Save and Apply
Received error - An error occurred while saving the form: Option "Design" contains an invalid input value. Expecting: non-empty value

Expected behaviour

Opened System menu in LUCI
Changed router name <General Settings - Hostname>
Click Save and Apply
Name Change Is Successful

Steps to reproduce

Opened System menu in LUCI
Changed router name <General Settings - Hostname>
Click Save and Apply

Additional Information
root@OpenWrt:~# cat /etc/os-release
NAME="OpenWrt"
VERSION="SNAPSHOT"
ID="openwrt"
ID_LIKE="lede openwrt"
PRETTY_NAME="OpenWrt SNAPSHOT"
VERSION_ID="snapshot"
HOME_URL="https://openwrt.org/"
BUG_URL="https://bugs.openwrt.org/"
SUPPORT_URL="https://forum.openwrt.org/"
BUILD_ID="r28085-6720c4ccba"
OPENWRT_BOARD="mediatek/filogic"
OPENWRT_ARCH="aarch64_cortex-a53"
OPENWRT_TAINTS=""
OPENWRT_DEVICE_MANUFACTURER="OpenWrt"
OPENWRT_DEVICE_MANUFACTURER_URL="https://openwrt.org/"
OPENWRT_DEVICE_PRODUCT="Generic"
OPENWRT_DEVICE_REVISION="v0"
OPENWRT_RELEASE="OpenWrt SNAPSHOT r28085-6720c4ccba"
OPENWRT_BUILD_DATE="1731585818"

LuCI Master 24.319.72217~9fe1e34

What browsers do you see the problem on?
Firefox
Relevant log output
<select id="widget.cbid.luci.cfg01e48a._mediaurlbase" class="cbi-input-select cbi-input-invalid" data-tooltip="Expecting: non-empty value" data-tooltip-style="error"></select>

Uncaught (in promise) TypeError: Option "Design" contains an invalid input value. Expecting: non-empty value
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:137
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:71
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:47
    save http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:47
    callClassMethod http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:137
    handleSave http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    handleSave http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    handleSaveApply http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    createHandlerFn http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:360
    handleClick http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:160
    bind http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:58
    render http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:47
    render http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:157
    addFooter http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:149
    __init__ http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    promise callback*__init__ http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    super http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:22
    ClassConstructor http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:12
    compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:182
    promise callback*compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:178
    promise callback*require http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:184
    instantiateView http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:360
    <anonymous> http://<IP_ADDRESS>/cgi-bin/luci/admin/system/system:51
    promise callback* http://<IP_ADDRESS>/cgi-bin/luci/admin/system/system:50
luci.js:137:348
Uncaught (in promise) TypeError: Option "Design" contains an invalid input value. Expecting: non-empty value
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:137
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:71
    parse http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:47
    save http://<IP_ADDRESS>/luci-static/resources/form.js?v=24.319.72217~9fe1e34:47
    callClassMethod http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:137
    handleSave http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    handleSave http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    handleSaveApply http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    createHandlerFn http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:360
    handleClick http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:160
    bind http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:58
    render http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:47
    render http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:157
    addFooter http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:149
    __init__ http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    promise callback*__init__ http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:147
    super http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:22
    ClassConstructor http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:12
    compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:182
    promise callback*compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:178
    promise callback*require http://<IP_ADDRESS>/luci-static/resources/luci.js?v=24.319.72217~9fe1e34:184
    instantiateView http://<IP_ADDRESS>/luci-static/resources/ui.js?v=24.319.72217~9fe1e34:360
    <anonymous> http://<IP_ADDRESS>/cgi-bin/luci/admin/system/system:51
    promise callback* http://<IP_ADDRESS>/cgi-bin/luci/admin/system/system:50
luci.js:137:348

You may have found something.
I see the normal selection between the Bootstrap variants.
But if I try to install openwrt-2020 theme, there is an error and the package does not appear into the dropdown selection list.  Neither is there any trace in /etc/config/luci, although there are openwrt-2020 theme files in /overlay/upper.
The error given is:
Executing package manager

apk add luci-theme-openwrt-2020

(1/1) Installing luci-theme-openwrt-2020 (24.320.57985~1a4b6ad)
Executing luci-theme-openwrt-2020-24.320.57985~1a4b6ad.post-install
1 error; 71 MiB in 309 packages

The apk install command failed with code 1.

Looks like it fails to run the post-install actions, or something like that.
The error message does not appear more clear even from ssh console with --verbose.

Looks like fallout from apk switchover?

I was able to install a  couple of themes via LUCI but they did not show in the dropdown

I tested a bit with r28102-65964c42f8 and noticed that

the script setting the config values to /etc/config/luci was placed into /etc/uci-defaults directory,   (https://github.com/openwrt/luci/blob/master/themes/luci-theme-openwrt-2020/root/etc/uci-defaults/30_luci-theme-openwrt-2020 )
but it was not being run along the theme installation.  opkg or apk should run it at installation.

When I then booted the router, the theme got changed and both themes were visible in the menu.
This is probably something that @aparcar and @Ansuel are interested, as it suggests that the apk package installation process does not currently do quite the same steps as opkg did.

The root cause may be

some kind of incompatibility between theme packages and https://github.com/openwrt/openwrt/blob/main/package/base-files/files/lib/functions.sh   , or
apk not running the default-postinst in /lib/functions.sh at all. (I tried adding debug statements there, but saw no output. Looks like that default-postinst is not reached)
cc @aparcar   Any ideas?

This should get fixed by https://github.com/openwrt/openwrt/pull/17023
Testing is much appreciated

@robimarko
I am not yet sure if this is solved at r28144-2784f78c6c, self-built for MT6000.
I tested installing luci-theme-material, and the config lines did not appear into /etc/config/luci.
The theme has been added to overlay on flash:
root@router6000:~# ls -l /overlay/upper/www/luci-static/
drwxr-xr-x    3 root     root          3488 Nov 21 20:40 material
drwxr-xr-x    2 root     root          3488 Nov 21 20:40 resources

root@router6000:~# apk list *theme*
luci-theme-bootstrap-24.284.01692~c68d5c8 noarch {feeds/luci/themes/luci-theme-bootstrap} (Apache-2.0) [installed]
luci-theme-bootstrap-24.322.80622~a403707 all {feeds/luci/themes/luci-theme-bootstrap} (Apache-2.0) [upgradable from: luci-theme-bootstrap-24.284.01692~c68d5c8]
luci-theme-material-24.322.80622~a403707 all {feeds/luci/themes/luci-theme-material} (Apache-2.0) [installed]
luci-theme-openwrt-24.322.80622~a403707 all {feeds/luci/themes/luci-theme-openwrt} (Apache-2.0)
luci-theme-openwrt-2020-24.322.80622~a403707 all {feeds/luci/themes/luci-theme-openwrt-2020} (Apache-2.0)

And the not-yet-executed uci-defaults script is visible:
root@router6000:~# cat /etc/uci-defaults/30_luci-theme-material
#!/bin/sh

if [ "$PKG_UPGRADE" != 1 ]; then
        uci get luci.themes.Material >/dev/null 2>&1 || \
        uci batch <<-EOF
                set luci.themes.Material=/luci-static/material
                set luci.main.mediaurlbase=/luci-static/material
                commit luci
        EOF
fi

exit 0

I likely figured one possible explanation:
LuCI themes do not define postinst function, but instead they directly add /etc/uci-defaults file, which was automatically ran by opkg installation, but apparently not by apk.
Likely the themes should be converted to declare postinst function in Makefile, like they actually do for postrm...

Do you think this is unique to themes?

I explicitly tested with luci-theme-material and uci-defaults script was for sure executed as UCI config was updated and newly installed theme was default, prior to these changes that would not happen and script would still be there.
But do note that you have to build the packages yourself as well as those that are currently in feeds are broken and it will take some time for SDK-s to get regenerated and then packages rebuilt.

For example with the 2020 theme, you can clearly see it works
root@Dynalink:/tmp# cat /etc/config/luci

config core 'main'
	option lang 'auto'
	option mediaurlbase '/luci-static/material'
	option resourcebase '/luci-static/resources'
	option ubuspath '/ubus/'

config extern 'flash_keep'
	option uci '/etc/config/'
	option dropbear '/etc/dropbear/'
	option openvpn '/etc/openvpn/'
	option passwd '/etc/passwd'
	option opkg '/etc/opkg.conf'
	option firewall '/etc/firewall.user'
	option uploads '/lib/uci/upload/'

config internal 'languages'

config internal 'sauth'
	option sessionpath '/tmp/luci-sessions'
	option sessiontime '3600'

config internal 'ccache'
	option enable '1'

config internal 'themes'
	option Bootstrap '/luci-static/bootstrap'
	option BootstrapDark '/luci-static/bootstrap-dark'
	option BootstrapLight '/luci-static/bootstrap-light'

config internal 'apply'
	option rollback '90'
	option holdoff '4'
	option timeout '5'
	option display '1.5'

config internal 'diag'
	option dns 'openwrt.org'
	option ping 'openwrt.org'
	option route 'openwrt.org'

root@Dynalink:/tmp# wget http://<IP_ADDRESS>:8000/luci-theme-openwrt-2020-24.326.01848~53e36e3.apk
Downloading 'http://<IP_ADDRESS>:8000/luci-theme-openwrt-2020-24.326.01848~53e36e3.apk'
Connecting to <IP_ADDRESS>:8000
Writing to 'luci-theme-openwrt-2020-24.326.01848~53e36e3.apk'
luci-theme-openwrt-2 100% |*******************************| 45812   0:00:00 ETA
Download completed (45812 bytes)
root@Dynalink:/tmp# apk add luci-theme-openwrt-2020-24.326.01848\~53e36e3.apk
(1/1) Installing luci-theme-openwrt-2020 (24.326.01848~53e36e3)
Executing luci-theme-openwrt-2020-24.326.01848~53e36e3.post-install
OK: 31 MiB in 172 packages
root@Dynalink:/tmp# ls /etc/uci-defaults/
root@Dynalink:/tmp# cat /etc/config/luci

config core 'main'
	option lang 'auto'
	option mediaurlbase '/luci-static/openwrt2020'
	option resourcebase '/luci-static/resources'
	option ubuspath '/ubus/'

config extern 'flash_keep'
	option uci '/etc/config/'
	option dropbear '/etc/dropbear/'
	option openvpn '/etc/openvpn/'
	option passwd '/etc/passwd'
	option opkg '/etc/opkg.conf'
	option firewall '/etc/firewall.user'
	option uploads '/lib/uci/upload/'

config internal 'languages'

config internal 'sauth'
	option sessionpath '/tmp/luci-sessions'
	option sessiontime '3600'

config internal 'ccache'
	option enable '1'

config internal 'themes'
	option Bootstrap '/luci-static/bootstrap'
	option BootstrapDark '/luci-static/bootstrap-dark'
	option BootstrapLight '/luci-static/bootstrap-light'
	option OpenWrt2020 '/luci-static/openwrt2020'

config internal 'apply'
	option rollback '90'
	option holdoff '4'
	option timeout '5'
	option display '1.5'

config internal 'diag'
	option dns 'openwrt.org'
	option ping 'openwrt.org'
	option route 'openwrt.org'

note that you have to build the packages yourself as well as those that are currently in feeds are broken and it will take some time for SDK-s to get regenerated and then packages rebuilt.

Thanks for explaining.
I forgot that the .apk package itself needs fix/recompilation.
I tested again with locally built .apk, and it works ok:
root@router4:~# apk add /tmp/luci-theme-material-24.191.51591\~5119fa2.apk
(1/1) Installing luci-theme-material (24.191.51591~5119fa2)
Executing luci-theme-material-24.191.51591~5119fa2.post-install
OK: 64 MiB in 318 packages

config internal 'themes'
        option Bootstrap '/luci-static/bootstrap'
        option OpenWrt2020 '/luci-static/openwrt2020'
        option BootstrapDark '/luci-static/bootstrap-dark'
        option BootstrapLight '/luci-static/bootstrap-light'
        option Material '/luci-static/material'

thanks for hint.
I will mark this solved, although it may take 2-3 days before the fixed packages are publicly available from the download site for all architectures.


Output formatting

One thing that seems to be missing from gofeed when compared to Feed43 is the ability to format the resulting RSS tags (title, link, description) with your own HTML. It would be great if you could compose your <description> output to include more than just what was matched from the source HTML file.
Here's a screenshot of Feed43's editing page for reference. In this example, {%1}, {%2} and {%3} represent content captured from the source page that is being injected into the resulting RSS xml fields, allowing you to control the format of the feed's content. I feel like the same thing could be done in gofeed using arbitrary {token_name}s instead of the fixed {title}, {link} and {description} shortcuts. Additional configs would be needed for Item.TitleFormat, Item.LinkFormat and Item.DescriptionFormat (or similar.)

Please consider this as a humble feature request. And forgive me if this is somehow already possible and I'm just reading the docs/code incorrectly. Thank you.

Thank you for your advice and support. I will look into this when I am not busy.


New release for qt6 support

Is it possible to have a new release for qt6 support?

Done. I've done some checks but I would ask you @looooo kindly to test the 4.0.1 and 1.6.1 bugfix releases.

thanks @VolkerEnderlein I wasn't yet able to test the new version a lot. Just a few things that came up:
for windows: it was not possible to simply replace coin4.0.0 with coin4.0.1. A rebuild of pivy and freecad is necessary. For the other platforms this is not necessary.

That sounds like we lost binary compatibility with version 4.0.0. It is not intended. I'll have a closer look on it over the next few days.


Don't clobber suffixesadd

This line currently resets the suffixesadd which can have unintended consequences and break other plugins / user configuration. We should append to suffixesadd instead of setting it (which clobbers existing things).

thanks !


Fix for #118. Provide separate deployment to retrieve a key from a URL.

@radcortez Hi Roberto thanks for this PR, sorry for a delay. I'm assuming this update will basically make sure that the test will run irrespective of how the application is deployed and its keys are initialized, correct ?

Yeah, the idea over here is that by using Arquillian multiple deployments, you could deploy an initial application that exposes the key and the second application that consumes the key depends on the deployment of the key app.

Hi @radcortez OK thanks. I've realized that some application servers (supporting single deployments like Thorntail or any other container which can't handle multiple deployments) may have problems. Perhaps we may split this test or come up with some other idea, otherwise I'll just mark it for JWT 2.0 where only UnresolvableKeyException without a tie-in to the time the keys are initialized, is checked...

Sure.
Another option could be to start a separate http server via HttpServer and expose the key over there. Not sure if we can plugin some sort of awaitability to wait for the server to get up and then deploy the Arquillian stuff. Might work.
I'll give it a try.

@radcortez It is a nice and neat idea, really is IMHO. I have a reservation now though about the issue validity. If the server fails at the startup if no remote location is available then it means it is not able to get hold of the rotated keys which is only possible by periodically refreshing a JWK set. If we have this HTTP server then what exactly we will test ? I mean it won't resolve the ambiguity about throwing or not throwing DeploymentException if no remote location is available.
Perhaps for 2.0 it could make sense to have such a test with the HTTP server emulating the case of returning a set with a non-matching kid, returning 404, etc, to support few variations around the remote key unavailability and verify that no matter what the client gets the same HTTP error, etc

Well, right now this was just to fix the issue for certain servers where the endpoint you use for the key is not available to query, because is part of the current deployment (and egg and chicken problem).
I agree that we need to discuss this further and figure out what should be the correct behaviour of the spec in these cases (404, wrong kid, etc).


[Nuxt 2 + Typescript] Reactive values are not unwrapped in component's template

I have installed the module with Nuxt 2 and transformed the whole project to use the new syntax (<script setup lang="ts">).
Is it the intention that all ref variables declared within the script block are intentionally left unwrapped in the component's template (that is, they can be accessed via .value)? I think either it is a misconfiguration on my side or a possible bug. I'll provide more details once I hear back on whether this is an intended behaviour.
As per below docs snippet, could it be a discrepancy between Vue 2 and Vue 3?

Closing - either seemed to be a temporary issue or impression caused by other accompanying errors. Ref values are correctly unwrapped.


Performance issue in @tanstack/query-async-storage-persister (and possibly #updateStaleTimeout)

Describe the bug
It looks like delayFunc in the packages/query-async-storage-persister/src/asyncThrottle.ts has some performance problem.
  const delayFunc = async () => {
    clearTimeout(timeout)
    timeout = setTimeout(() => {
      if (running) {
        delayFunc() // Will come here when 'func' execution time is greater than the interval.
      } else {
        execFunc()
      }
    }, interval)
  }

this is the 4x slow-down profile, each refetch call causes persistQueryClientSave to be called, and then delayFunc is the slowest function in the tree. All small yellow bars are setTimeout or clearTimeout calls.
My throttle timeout is the default (1e3), therefore it cannot be caused by execFunc (all those delayFunc calls happen within 1 second).
I try to patch my node_modules and replace the asyncThrottle with this:
function asyncThrottle2(func, { interval = 1e3, onError = noop } = {}) {
    if (typeof func !== 'function') throw new Error('argument is not function.')
    let currentArgs = null
    setInterval(() => {
        if (!currentArgs) return
        Promise.resolve(func(...currentArgs)).catch(onError)
        currentArgs = null
    }, interval)
    return (...args) => {
        currentArgs = args
    }
}
export { asyncThrottle2 as asyncThrottle }

This implementation is incorrect but fast.

After the patch, persistQueryClientSave now only costs 1 ms instead of 57 ms.
I think it looks like calling setTimeout or clearTimeout is costly. I think a better async throttle implementation can avoid calling them so often.
btw I also observed that #updateStaleTimeout method has a similar behavior.

also, this method behaves like delayFunc, simple but costs a lot of time (cancel & create timer).

Your minimal, reproducible example
n/a
Steps to reproduce
Sorry I didn't provide one, but my patch works so I think my analysis of the performance problem is correct.
I'm happy to try any new asyncThrottle implementation (by patch node_modules, so maintainers don't have to release a new version for this) in our project to see if that fixes the performance problem. Thanks!
Expected behavior
No performance problem
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Chrome 119
@tanstack/query-async-storage-persister 5.8.7
Tanstack Query adapter
None
TanStack Query version
5.8.7
TypeScript version
No response
Additional context
No response

I'm happy to try any new asyncThrottle implementation

if you have a better implementation that is also correct, please do contribute 🙏. The asyncThrottle function does have a decent test coverage.
Maybe we can also get inspired by other, already existing implementations?

https://github.com/sindresorhus/p-throttle

@Jack-Works would you consider contributing your patch back to the persister plugin?

https://github.com/DimensionDev/Maskbook/pull/11227/files

@Jack-Works would you consider contributing your patch back to the persister plugin?
* [DimensionDev/Maskbook#11227 (files)](https://github.com/DimensionDev/Maskbook/pull/11227/files)

Hi! I think our patch is obviously imperfect (it is over simplified and maybe even incorrect). I hope your team can find another throttle implementation and I'd like to try it.

Maybe we can also get inspired by other, already existing implementations?
[sindresorhus/p-throttle](https://github.com/sindresorhus/p-throttle)

@TkDodo Genuine question: why don't you use p-throttle? :) I'm just trying to understand what makes asyncThrottle special.

Hello @jhnns,
I was also wondering why not just use p-throttle, so I looked into the two implementations and they don't actually behave the same.
Take the following example:
const start = performance.now();
const fn = (a, b) => {
  const time = Math.floor(performance.now() - start);
  const result = a + b;
  console.log({ time, result });
};

const throttledFn = asyncThrottle(fn, { interval: 500 });

throttledFn(1, 2); // prints: { time: 0, result: 3 }
throttledFn(1, 3); // ignored
throttledFn(1, 4); // ignored
throttledFn(1, 5); // ignored
throttledFn(1, 6); // prints: { time: 500, result: 7 }

So to my understanding, asyncThrottle only keeps the last call waiting and ignores all the other calls.
While p-throttle keeps all the calls waiting and executes them in order:
import pThrottle from "p-throttle";

const start = performance.now();
const fn = (a, b) => {
  const time = Math.floor(performance.now() - start);
  const result = a + b;
  console.log({ time, result });
};
const throttle = pThrottle({
  limit: 1,
  interval: 500,
});
const throttledFn = throttle(fn, { interval: 500 });

throttledFn(1, 2); // prints: { time: 0, result: 3 }
throttledFn(1, 3); // prints: { time: 500, result: 4 }
throttledFn(1, 4); // prints: { time: 1000, result: 5 }
throttledFn(1, 5); // prints: { time: 1500, result: 6 }
throttledFn(1, 6); // prints: { time: 2000, result: 7 }

I didn't find a way to tell p-throttle to only keep the last call, so I don't think it can be used as a drop-in replacement for asyncThrottle.

@Jack-Works I made the PR https://github.com/TanStack/query/pull/7224 that suggests a new implementation. Can you please try it on your project and let us know if it fixes the issue?

fixed by:

#7224


TrailingCommaOnDeclarationSite shows incorrect line number and wrong error message for enums

Expected Behavior
Display the correct line number and error message.
Observed Behavior
The incorrect line number is displayed together with the irrelevant error message
* What went wrong:
Execution failed for task ':spotlessKotlin'.
> Error on line: 1, column: 1
  rule: trailing-comma-on-declaration-site
  Missing trailing comma before "}"

Steps to Reproduce
enum class Priority(private val value: Int) {
    PRIORITY_1(1),
    PRIORITY_2(2);

    companion object {
        fun getByValue(value: Int): Priority {
            return Priority.values().firstOrNull {
                it.value == value
            } ?: throw IllegalArgumentException("Invalid priority value: $value ")
        }
    }
}

Your Environment

Version of ktlint used: 0.48.2
Name and version (or code for the custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task): ./gradlew spotlessCheck
Version of Gradle used (if applicable): 8.0
Operating System and version: MacOS Ventura

This does not seem to be a problem in Ktlint. With Ktlint CLI I can not reproduce this problem:
$ cat <<EOF | ktlint-0.48.2 --stdin
> enum class Priority(private val value: Int) {
>     PRIORITY_1(1),
>     PRIORITY_2(2);
>
>     companion object {
>         fun getByValue(value: Int): Priority {
>             return Priority.values().firstOrNull {
>                 it.value == value
>             } ?: throw IllegalArgumentException("Invalid priority value: $value ")
>         }
>     }
> }
> EOF
21:04:35.087 [main] INFO com.pinterest.ktlint.internal.KtlintCommandLine - Enable default patterns [**/*.kt, **/*.kts]
<stdin>:3:18: Missing trailing comma before ";" (trailing-comma-on-declaration-site)

Summary error count (descending) by rule:
  trailing-comma-on-declaration-site: 1

Can you check with Ktlint CLI directly instead of using Spotless?

I have the same problem, and I could reproduce with Ktlint CLI using --format option:
cat <<EOF | ktlint --format --stdin
enum class Priority(private val value: Int) {
    PRIORITY_1(1),
    PRIORITY_2(2);

    companion object {
        fun getByValue(value: Int): Priority {
            return Priority.values().firstOrNull {
                it.value == value
            } ?: throw IllegalArgumentException("Invalid priority value: $value ")
        }
    }
}
EOF

09:55:00.513 [main] INFO com.pinterest.ktlint.internal.KtlintCommandLine - Enable default patterns [**/*.kt, **/*.kts]
enum class Priority(private val value: Int) {
    PRIORITY_1(1),
    PRIORITY_2(2),
    ;

    companion object {
        fun getByValue(value: Int): Priority {
            return Priority.values().firstOrNull {
                it.value == value
            } ?: throw IllegalArgumentException("Invalid priority value:  ")
        }
    }
}
<stdin>:1:1: Missing trailing comma before "}" (trailing-comma-on-declaration-site)
<stdin>:5:4: Unnecessary semicolon (no-semi)

Summary error count (descending) by rule:
  no-semi: 1
  trailing-comma-on-declaration-site: 1

Tnx. The --format was indeed the missing link to reproduce.

I can reproduce the problem with the 0.48.2 version. However, with current snapshot for the upcoming 0.49.x, I can no longer reproduce it.


Signature help request responds with data even after closing bracket

Signature help request returns with help information even after the cursor has moved past the function call brackets:

I'm implementing signature help for helix edior in https://github.com/helix-editor/helix/pull/1755, and to auto close the help popup the editor sends a signature help request again on common terminating characters like ) and ; upon which the server would ideally respond with None, which in turn closes the popup.

rust-analyzer version: rust-analyzer 17afa2e77 2022-01-24 nightly

Relevant part is here https://github.com/rust-analyzer/rust-analyzer/blob/e8edbb5d6fa43db893fc5eb2a3c75b7b23713919/crates/ide/src/call_info.rs#L42-L54


Rishabh Rathore

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one. Nice Profile btw 🚀


Wish for conda package

🚀 Feature
It would be nice with a conda package for this tool
Motivation
Conda packages simplifies installation and potential dependencies. It has become a popular way to install scientific python packages.
https://conda-forge.org/

Thanks @petrushy for the suggestion! We are actually working on a conda-forge package here:
https://github.com/conda-forge/staged-recipes/pull/9983
Still need to figure out a bad interaction between PyQt5 and PySide2...

Ah great :) I have unfortunately not experience in qt problems..

@jni @jakirkham can this now be closed? I think napari can now be installed with
conda install -c conda-forge napari

When we make new releases to pip will conda-forge then automatically start using the newest release or do we need to do something to make that update? What is the recommended way after / during the release process to make sure the new conda-forge release works? Do we need to add something to our test script plan in #929 @jni.
Apologies if these basic questions are answered in the conda-forge docs, but if we have to do anything at all we should add it to the napari release procedure docs otherwise i at least will forget.
Finally, it looks like both the feed-stock and the package are listed as BSD-03 (see here https://github.com/conda-forge/napari-feedstock), I just want to check that is ok and that we aren't actually distributing any LGPL Qt code ourselves here (in which case the license would have to be LGPL, as it will be for our bundled apps)

@jni I am closing this based on comments above

Yeah sorry for the slow reply. I think this is solved already.


disable stdin event validation for check

Events are not passed to checks via stdin.
Checks recieve a JSON construct that is a cutdown event, with just a "check" attribute.
For now just disable event Validation for the check plugin using an additional boolean to enable/disable event validation.

This LGTM. Thanks, Jef!


Edge case: Brackets within comments within DOCTYPE

I'm currently working with a file that starts off like this:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE kanjidic2 [
	<!-- Version 1.6 - April 2008
	This is the DTD of the XML-format kanji file combining information from
	the KANJIDIC and KANJD212 files. It is intended to be largely self-
	documenting, with each field being accompanied by an explanatory
	comment.

	The file covers the following kanji:
	(a) the 6,355 kanji from JIS X 0208;
	(b) the 5,801 kanji from JIS X 0212;
	(c) the 3,693 kanji from JIS X 0213 as follows:
		(i) the 2,741 kanji which are also in JIS X 0212 have
		JIS X 0213 code-points (kuten) added to the existing entry;
		(ii) the 952 "new" kanji have new entries.

	At the end of the explanation for a number of fields there is a tag
	with the format [N]. This indicates the leading letter(s) of the
	equivalent field in the KANJIDIC and KANJD212 files.

	The KANJIDIC documentation should also be read for additional
	information about the information in the file.
	-->
<!ELEMENT kanjidic2 (header,character*)>
<!ELEMENT header (file_version,database_version,date_of_creation)>
...

I'm getting the following error:
Unhandled exception:
XmlParserException: ">" expected at 18:21

I would guess this means that [N] is the part getting in the way, and that the parser recognizes the ] too early and expects an > to end the DOCTYPE element.

Thank you for reporting and analyzing the problem. The current implementation does not attempt to parse the DOCTYPE element and instead just tries to read over it. In this case that obviously fails, I'll be looking into it.
For reference, the doctype syntax is documented here: https://www.w3.org/TR/xml/#sec-prolog-dtd


Time Zone incorrect

Where do I adjust the timezone? The timezone showing on the Status page shows GMT-500, when the system should be in CDT. Typing date in terminal shows correctly: Tue  7 Apr 17:13:31 CDT 2020

CDT is -5 isn't it?

  You can close this.

How do you change the timezone in homebridge-config-ui-x.  the date/time/timezone is correct on the raspberry pi, but the timezone showing in system information is incorrect.


Bugfix/nullable types

Hi @ch4mpy. Thanks for sending this PR. I saw that you created a fork and you changed the package names.
Do you prefer to fork the repository and take ownership of the project?
I can close this project and add a note mentioning that you're maintaining an updated version of the plugin.

Ooups, sry for the packages, I forgot to restore it. At some point, I ran into some difficulties to publish on a private Nexus and changed it to sure I was pulling updated plugin...
Actually I'm not much more interested in the project ownership  as you are ;)
I just got things work for my use case and wanted to contribute back as your work saved me a lot of time (thank you by the way)

@ch4mpy: No problem at all. Feel free to update the package names and I'll merge this PR. Otherwise, I can do it over the weekend.

@cavorite: Hello, I believe I restored your group-id / package names. Ping me if if you need further changes.


Present Livemark on PyData

Overview
6:00-7:30 am UTC, Friday, Oct 29
It requires #165

DUPLICATE to #165


[armel tizen] Fixed RhpNewArray

@Dmitri-Botcharnikov
@BredPet
@jkotas
please review
With fix and with  https://github.com/dotnet/coreclr/pull/12550 we have successful execution of the "Hello" application before the next unimplemented EmitCode:
./Hello
 /share/CORERT/corert/src/ILCompiler.Compiler/src/Compiler/DependencyAnalysis/Target_ARM/ARMReadyToRunHelperNode.cs(80): method EmitCode: GetGCStaticBase is not implementedcrashed [1470068017] processname=Hello, pid=26776, tid=26776, signal=6Aborted (core dumped)


Can't build Dockerfile; Invalid Signature from Cuda repo

I recently had to clear out cached Docker images to make space for the massive Mathworks build process (which didn't work). Now, attempting to rebuild our Dave Docker image with:
./build.bash .

I get to step 5 in the build process and then it crashes:
W: GPG error: https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  Release: The following signatures were invalid: BADSIG F60F4B3D7FA2AF80 cudatools<EMAIL_ADDRESS>E: The repository 'https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  Release' is not signed.

This is a known issue: https://github.com/NVIDIA/nvidia-docker/issues/1369 but it is closed because nvidia-docker doesn't want to own it. They want it submitted to their developer forum. It has been: https://forums.developer.nvidia.com/t/cuda-10-1-install-gpg-error-the-following-signatures-were-invalid-badsig-f60f4b3d7fa2af80/79944
It looks like there are workarounds, but maybe the best fix is to move to using Rocker from a standard 18.04 base image.

Is this the link to the Rocker you meant? (https://github.com/osrf/rocker)

We have moved to using rocker so I am closing this.


[ISSUE #45] Add default system properties

fix [xpocket-jline] version from 2.1.0-RELEASE to 2.1.1-RELEASE for build failure.
init system properties for running in IDEs.


Junit-Spring integration IOexception on console

I have a question here. I am trying to access spring configuration file using junit, but it is failing before setUp() method with following errror:
10:33:21.494 INFO [main][org.springframework.integration.http.inbound.IntegrationRequestMappingHandlerMapping] Mapped "{[/getBillServiceImplReq],methods=[POST],params=[],headers=[],consumes=[],produces=[],custom=[]}"
onto public abstract void org.springframework.web.HttpRequestHandler.handleRequest(javax.servlet.http.HttpServletRequest,javax.servlet.http.HttpServletResponse) throws javax.servlet.ServletException,java.io.IOException

here is my Junit:
@RunWith(SpringJUnit4ClassRunner.class)
@ContextConfiguration(locations = { "classpath:/WEB-INF/upperSI/http/servlet-config.xml" })
@WebAppConfiguration
public class RestGetServiceImplTest{
@Autowired
private WebApplicationContext webApplicationContext;

private MockMvc mockMvc;

/**
 * Set variables and Objects which are need to execute the test-case.
 * @throws IOException
 */
@Before
public void setUp() throws Exception{
    this.mockMvc = MockMvcBuilders.webAppContextSetup(this.webApplicationContext).build();
        }

and servlet-config code im trying to access/load is :
<supported-methods="POST" path="/getBillServiceImplReq" mapped-request-headers="HttpHeaders.REQUEST_URL"/>

<!-- this is require to enrich the header to handle the content-type of
    type "application/json" -->
<int:header-enricher input-channel="getBillServiceImplChannel"
    output-channel="getBillServiceReqJsonImplChannel">
    <int:header name="content-type" value="application/json"></int:header>
</int:header-enricher>

<int:chain input-channel="getBillServiceReqJsonImplChannel">
<!-- Transformer is required to map the JSON payload in the Java object
    format which can be processed in the eChannel framework -->
<int:json-to-object-transformer type="com.nbad.echannel.common.model.GetBillType" object-mapper="jackson2JsonObjectMapper" />
 <!-- <int:transformer ref="loggingJSONRequest" method="logJSONRequest" /> -->
<int:service-activator>
    <bean class="com.nbad.echannel.service.account.impl.GetBillServiceImpl"><constructor-arg ref="getDepositType"></constructor-arg></bean>
</int:service-activator>
/int:chain>

Hi @razazaran !
First of all I don't see in this issue any Spring Integration Groovy DSL relevant info.
From other side I've just added MockMvc to the https://github.com/spring-projects/spring-integration/blob/master/spring-integration-http/src/test/java/org/springframework/integration/http/HttpProxyScenarioTests.java
And it works well.
So, provide, please, more StackTrace on the matter and supply some test-case to reproduce what doesn't work.
And, please, pay attention that message is 10:33:21.494 INFO [main]. So, what's going on there?
Thank you. And let us know how we can help you more.

Thanks Artem!!  see im trying to do integration testing of my spring-integration application. I have create Junit which loads application context using @ContextConfiguration(locations = { "classpath:/WEB-INF/servlet-config.xml" }), everything works fine and i get the required response in JSON,  but when i see STS console, i see this exception in blue , which I have pasted above.
This error comes even before setUp() method of junit is called.  Looks like @contextConfiguration is unable to load Spring integration beans.
Application context :

<int-http:inbound-gateway request-channel="getBillServiceImplChannel"
supported-methods="POST" path="/getBillServiceImplReq" mapped-request-headers="HttpHeaders.REQUEST_URL"/>
<!-- this is require to enrich the header to handle the content-type of
	type "application/json" -->
<int:header-enricher input-channel="getBillServiceImplChannel"
	output-channel="getBillServiceReqJsonImplChannel">
	<int:header name="content-type" value="application/json"></int:header>
</int:header-enricher>

<int:chain input-channel="getBillServiceReqJsonImplChannel">
<!-- Transformer is required to map the JSON payload in the Java object
	format which can be processed in the eChannel framework -->
<int:json-to-object-transformer type="com.echannel.common.model.GetBillType" object-mapper="jackson2JsonObjectMapper" />
 <!-- <int:transformer ref="loggingJSONRequest" method="logJSONRequest" /> -->
<int:service-activator>
	<bean class="com.eChannel.service.dueamount.impl.GetBillServiceImpl"><constructor-arg ref="getBillType"></constructor-arg></bean>
</int:service-activator>
<int:object-to-json-transformer content-type="application/json" />
</int:chain>

Sorry, I don't use STS, but as I said it is just an INFO. Since you confirm that everything works well, may be you just configure a logger in the STS for the WARN on the org.springframework category.
Let us know and close this issue if you don't have anymore problems

are these tags of spring integration in application context correct ?

I don't see anything specific. Just found that MVC include as redundant

Ok Many Thanks !!. Do you know how to debug the xml file so that I may know which spring integration element was called and which not.

XML config can't be debugged, because it makes sense only on the parser phase. In the runtime all those XML components are translates to specific Java object. And they already can be debugged.
I most cases they are MessageSource, MessageProducer or MessageHandler implementations.
Please, read more about JavaConfig (http://docs.spring.io/spring-integration/docs/latest-ga/reference/html/configuration.html#annotations) to get more info about Spring Integration internals.

Thanks for the info Artem. Cheers !! but this issue is still not resolved, though im getting proper response. : )

and yes I have seen you replied to similar question on stacoverflow.
http://stackoverflow.com/questions/24310609/spring-integration-http-endpoint-from-child-context-doesent-work

Sorry, I thought your issue was about that INFO in the logs, but now you refer me to an issue, when DispatcherServlet is in the child ctx.
For this purpose we really should have a separate ticket with proper info on the matter.

how to find whether dispatcher servlet is in the ctx ?? im not sure ... can u figure out from my code.

DispatcherServlet is, erm.., a Servlet in the web.xml of your web-app, or that one from Spring Boot, if you use spring-boot-starter-web, or that from JavaConfig for Servlet 3 spec.
But as I said: this question isn't relevant to this ticket.

when im trying to call spring integration code in xml... from junit class, i get this error....   I dont get any error when i deploy n then run SOAPUI tests.  i want to do this testing from Junit, not soapui.

OK. Let's come back to the start! Which error do you have from test?
Share, please, full StackTrace.

10:33:21.494 INFO [main][org.springframework.integration.http.inbound.IntegrationRequestMappingHandlerMapping] Mapped "{[/getBillServiceImplReq],methods=[POST],params=[],headers=[],consumes=[],produces=[],custom=[]}"
onto public abstract void org.springframework.web.HttpRequestHandler.handleRequest(javax.servlet.http.HttpServletRequest,javax.servlet.http.HttpServletResponse) throws javax.servlet.ServletException,java.io.IOException
this is the error im getting on console....  nothjing else. this error does not stop the flow, and i get proper response in junit.

It isn't error. It's just log message on the INFO level from the AbstractHandlerMethodMapping#registerHandlerMethod
this.handlerMethods.put(mapping, newHandlerMethod);
if (logger.isInfoEnabled()) {
	logger.info("Mapped \"" + mapping + "\" onto " + newHandlerMethod);
}

What is the reason to disturb us for nothing?

sorry I forgot to share the error message. my bad. I dont have the error message right now.


Add Exception metadata to the data sampling protocol

When an exception occurs, SDKs will sample the input element to the UDF that caused the exception and fill out the associated metadata.
addresses #26704

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

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

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

See CI.md for more information about GitHub Actions CI.

R: @kennknowles

Run Java PreCommit

Run Python_Examples PreCommit

R: @kennknowles ping

Run Java PreCommit

R: @robertwb

R: @udim


[EMERGENCY] Install licenses along with the programs

Describe the bug
Currently, most of the pacscripts don't install the license of the software that they are installing, this is actually illegal as you are basically coping the code without the license
To Reproduce
NA.
Expected behavior
Obvious
Screenshots
NA
Desktop (please complete the following information):

OS: NA
Version NA

Additional context
I am thinking of adding a message to the @pacstall/pacscript-maintainers to inform everyone about this.
#35  is a pacscript that installs the licenses, please follow similar commands to install the licenses.

Where would the licenses go?

I have a cool idea. There could be a directory somewhere, and pacscripts have an array of licenses, that correspond to a plain text url with the full license and downloads it

I will add the license stuff to my package like in my AUR package in a minute https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=soundux#n23
This should be fine.

Like I said #35 installs the licenses in /usr/share/licenses/$name just like the AUR.

Sure, I still think the license array would be cool, and, users could specify what type of licensed software they want to download, like portage

Currently, most of the pacscripts don't install the license of the software that they are installing, this is actually illegal as you are basically coping the code without the license

I don't understand where this "illegal" comes from.
Pacstall do not distribute the code or binary, it's just a tool(a series of commands) to make the end user able to download the code to there machine, build, compile and install it them self.
With GPL-3 you don't even need to accept the license to run the program, it only apply when you "modifying or propagating a covered work" all this have it's own section in the license starting like this

Acceptance Not Required for Having Copies.

You are not required to accept this License in order to receive or run a copy of the Program.

I have also look at the MIT and Apache license and I do not find a requirement to install a license.

I don't understand where this "illegal" comes from.
Pacstall do not distribute the code or binary, it's just a tool(a series of commands) to make the end user able to download the code to there machine, build, compile and install it them self.

Technically, you are required to receive a copy of the GPL, when you receive/compile a binary. And since pacstall is compiling and building the source, it is distributing it to the user.
Here are some FAQs

Basically since we are installing free software we are required to install the license, so the users get to know their rights.

Acceptance Not Required for Having Copies.
You are not required to accept this License in order to receive or run a copy of the Program.

It says accept not recieve. According to the GPL the user is supposed to receive a copy of it, along with the program itself.

It says You should have received a copy of the GNU General Public License along with Pacstall. (or any other GPLed program).

pacstall is just a program. It's a program that you can download and install onto your machine. Then, that program reads a config file where it sees how a given program can be downloaded (the pacscript file).
The user is then actually the one downloading that. That's the same as if you would say, a bash script with the command wget https://example.com/binary-thats-under-the-gpl in it would be illegal because it does only load the binary, not the license.
That's not the case. With that bash script, the operator of "example.com" is who is distributing the binary, and as long as the license is available on example.com, that's just fine. With pacstall, whoever operates the webserver behind the URL inside the pacscript is the one who is distributing the binary.
Yes, a package manager should install the program's license with the program. But it is not legally required to do that.

Thank you @Leseratte10 for you input! I am closing this issue then.


Contributing, goal/vision of the project.

As a Neovim user that has switched multiple times between my own Neovim config and SpaceMacs I really like this effort 👍
I'd love to help contribute but some things are still unclear for me.

What's the progress in the conversion process from spacevim -> spaceneovim
What work still needs to be done ?
How do you plan to facilitate discussion about what plugins to introduce and what neovim equivalent's to use for certain plugins ?
Seeing as you recently landed a commit titled: "adding vim support" how does this project relate to the original fork ? Is the goal to replace SpaceVim in the future when feature parity is reached ?

I think the later is really important, I'd like to prevent two separate projects whilst we could have a united effort in bringing spacemacs to neovim/vim.

Excellent questions/concerns! and I might have been a bit premature in sharing this repository before setting up some more clear goal posts (and the goals have also changed slightly since I began).

Spacevim itself was not particularly tied to regular vim (and should work in neovim), but before beginning my rewrite I thought I was going to have to use a lot of neovim specific API calls and such - that turned out not to be the case.
Currently smoothing out the install process and providing a nicer UX is on the plan, along with getting all the layers done. The layers are really the most critical one, but is luckily super easy to do - you just need to know what plugins you want.
With regards to discussions

Currently I imagine discussions will be separated into the two repositories. Core discussions such as install, updating and such goes in this (the spaceneovim) repository and discussiosn with regards to layers go in the layers (spaceneovim-layers) repository
Faciliatating neovim vs regular vim plugins is still up for discussion. Currently for things like the syntax checker you'd probably use +syntax-checking/syntastic for regular vim and +syntax-checking/neomake for neovim. Once you go into languages though, it could either be hidden behind has('nvim') or perhaps a more clear separation like say +lang/n-haskell for neovim and +lang/haskell for regular vim, although I'd prefer not to split them up too much.

Before getting familiar with vimscript/viml I assumed that supporting vim and neovim at the same time would be a lot of hassle, and would rather just focus on one thing - this though proved a lot easier than my initial estimate and therefore I decided to add vim support back again. There are a few has('nvim') calls here and there but all in all it's not so bad

I think you're very right that the latter concerns are quite important! Being able to have more community effort behind making the layers work perfectly is super important, and I'm contemplating renaming back to spacevim, since supporting both proved so trivial.

Oh, and with regards to the big difference between spacevim and SpaceNeovim is probably that SpaceNeovim only focuses on a distribution mode, while spacevim also supports just having keybindings enabled. SpaceNeovim also has a more clear separation of layers vs core.


Add back the default setting values and fix #739

fix #739

LGTM

LGTM

Disabling cron doesn't work for me. I added this to the config:
[cron]
ENABLED = false

@cez81 Yes. The default value should be enabled.

@cez81 maybe you could disable every item?
[cron.update_mirrors]
ENABLED = false
SCHEDULE = @every 10m

@lunny Yes, this works and disables all cron jobs.
[cron.update_mirrors]
ENABLED = false

[cron.repo_health_check]
ENABLED = false

[cron.check_repo_stats]
ENABLED = false

[cron.archive_cleanup]
ENABLED = false

If that is the desired behaviour config cheat sheet should be updated and  [cron]  removed from default app.ini

Actually, there are some different between bindata version and static version. It seems gopkg.in/ini.v1 has some magically usage. The child section will derive the parent section's variables. But for bindata version, there is no this magic. I will merge this currently and let's discuss how to resolve this problem in another issue.


Add an "Integrate Google Maps to an app" tutorial

Add a simple android tutorial that explains how to use Google Maps API with in your application. Include a simple application for the tutorial.
200 Points

F16 OK (200 pts)


Adding a patch

As Moment js announced their working is becoming leacy so replacing moment js with Date-fns format system which uses unicode time format.

thanks for your contribution :smile:


Improve test coverage of Parser and Command

resolves #290
This PR improves the test coverage of:

[x] Parser
[x] Command

@ljiazh3ng I'm gonna review this after today's mock PE yah.


De-dupe item-driven notifications

Update how we query for item-driven notifications to de-dupe them for:

mentions
replies
thread subscriptions
user subscriptions

users should recieve only 1 notification for any given item created by another user, following the above priority order
this is accomplished by querying for replies to current user, replies on subscribed threads, items from subscribed users, and mentions (if enabled for the current user), sorting those results by item id and priority order within item id based on notification type, then selecting the highest priority notification from each item id
these results are then union all'ed with all other notification types, unchanged.
Closes #453

Side note, this should also remove duplicate notifications if you're mentioned more than once in an item :)

Side note, this should also remove duplicate notifications if you're mentioned more than once in an item :)

That shouldn't have been an issue before unless we regressed somewhere along the way.

Side note, this should also remove duplicate notifications if you're mentioned more than once in an item :)

That shouldn't have been an issue before unless we regressed somewhere along the way.

You are right. I misread some of my notifications in the past (on one of the SN release posts). I thought it had 2 mention notes but it indeed only has 1.

Here's another LN invoice
lightning:lnbc1pjwlu8wpp5nva3yeggnf5kuqfd9azv6lyd2wmk5wmcsw93gn9ex0usc3kedt9qdqqcqpjsp54aueuuvrpkqqvnpv37cusn73708tls7ddctj42pekq0a5ztys7sq9q7sqqqqqqqqqqqqqqqqqqqsqqqqqysgqmqz9gxqyjw5qrzjqwryaup9lh50kkranzgcdnn2fgvx390wgj5jd07rwr3vxeje0glcllcn67flrvwvn5qqqqlgqqqqqeqqjqc27w5u7x20vyn85aypugpkfjxpmpppjrj8kxed2zzat9230swpkyxc4vgjltxajlp5yx5m4rfh7hytpjc2sccdd0hg9gav827n7hr2gqdlu09q
Should I just put these in the PR description when I open it? I think they're valid for a week upon creation.

If you can make them 2 weeks that'd put me less under the gun, but we can give it a shot.
If you're using an lnurl-pay compatible wallet an lnurl-pay code would be even better! (At least until we have non-custodial overflow on SN.)

Let me explore and see what I can come up with. I don't want to apply unnecessary pressure :)


fix/spatial-operations-reset-button

Hi both,
I've done a quick improvement on the 'SpatialOperations' sample.
When redoing the readme, I found that the reset button doesn't reset the selected operation (just the geometry).
Addition: added a 'NONE' operation, made it selected when the sample starts, and pressing reset selects this option again.
If you have a moment to review, I'd be thankful to hear what you think.

@JonLavi  how about
geomOperationBox.setOnAction(e -> {
        Geometry resultPolygon;
        OPERATION_TYPE operation = geomOperationBox.getSelectionModel().getSelectedItem();
        switch (operation) {
          case UNION:
            resultPolygon = GeometryEngine.union(polygon1.getGeometry(), polygon2.getGeometry());
            break;
          case DIFFERENCE:
            resultPolygon = GeometryEngine.difference(polygon1.getGeometry(), polygon2.getGeometry());
            break;
          case SYMMETRIC_DIFFERENCE:
            resultPolygon = GeometryEngine.symmetricDifference(polygon1.getGeometry(), polygon2.getGeometry());
            break;
          case INTERSECTION:
            resultPolygon = GeometryEngine.intersection(polygon1.getGeometry(), polygon2.getGeometry());
            break;
          case NONE:
            resultGeomOverlay.getGraphics().clear();
          default:
            return;
        }

        // update result as a red (0xFFE91F1F) geometry
        SimpleFillSymbol redSymbol = new SimpleFillSymbol(SimpleFillSymbol.Style.SOLID, 0xFFE91F1F, line);
        resultGeomOverlay.getGraphics().add(new Graphic(resultPolygon, redSymbol));
      });

Then you could get rid of the reset button

@tschie sounds good to me!
Had to modify it a little to make sure the geometries are reset every time a choice is made.
Reset button is now gone.
Would you have a quick look to see if it still behaves as expected?

@JonLavi Looks good. Only other things I can think of are

The image needs to be updated (no more reset button)
The label maybe isn't necessary. You could probably just show the combobox (without the background vbox)
The combobox could be a little wider so symmetric_difference doesn't get cut off

Good to merge when (1) is done. Up to you if you want to make the other changes.

Thanks @tschie, all those changes implemented.


add yourselves as watchers

@sedelson31 and @sili22 can you please add yourselves as watchers for this repo?

Yes - just did.

@sili22 any updates on this front?

Just did!


Working PR for Autoscaling Tutorial

I wasn't exactly sure how to lay the tutorial out so I'd consider this an initial thing which will be changed before merging.
I think we still need to update the development workflow and maybe do a full process-container tutorial
@mcollina @mcdonnelldean @pelger Let me know!

:+1:, we should also add this to my blog post.

also @Nss, as half wrote this bit!

@darragh-hayes @mcollina Does this still need to be reviewed?


EthernetClient::read(buf, size) always seems to return size

This function always seems to return the size value, no matter what I pass in. If nothing's available, or something less than size is available, it should return that number.
Is this function even intended to be called without first checking "socket-available" or some such, and then using that value for size?

Now I do see those errors and I know why, I didn’t add anything to keep track of what’s actually in the socket buffers. But yes you do have to call client.available() before doing reads, without calling it first the socket buffers won’t have data. Unfortunately I don’t have time to work on the fix for this, but it is something that should be fixed. I’ve slowly been learning to just use FNET by itself since I don’t like the standard Arduino Ethernet style. Not to mention the blocking nature that just wastes resources with how fast the Teensy 4.1 is.

Thanks for having a peek. I think I’ll take your advice and try to learn FNET’s non-blocking API. (See also #11 and #12 for Arduino API issues, which, as soon as I learn FNET, will become moot for me, but I think affect functionality.)

Well, I've been hunting down bugs in our use of ArudinoWebSockets on Teensy4.1 and since it uses NativeEthernet I've been chasing this bug hunt today down to bugs in this code.
AurdinoWebSockets makes the error of not always causing client.available().  They tried but didn't get it right if the WebSocket message is too large to fit into one packet.  I don't know why corrupted data is being returned.  Haven't figure that out yet.
I'll write this next problem as it's own issue, but I'll mention it here.
You have the variable _remaining in NativeEthernetClient.cpp to track how many bytes are available to read.  But when they fail to call accept() and just call read(), your code will go ahead and try the read even though _avaibale is 0.  Then when it manages to read data, you blindly subtract the 485 bytes (say) read, from _available, which allows it to go negative,  And once it's negative, available() stop working since it won't try to re-fill the buffers unless _aviable == 0. So it hangs forever, even when calling available() with available(), endlessly returning -485 bytes.


Feature request: Remove 'service' dimension from single_metric()

Use case
My team has a specific metric format used across lambda runtimes with the following dimensions: [Operation, Function]. I would like to use single_metric without 'service' added as a third dimension.
I would remove POWERTOOLS_SERVICE_NAME, but we also leverage the powertools Logger.
Solution/User Experience
The ability to exclude 'service' as a dimension from single_metric calls.
Alternative solutions
Support ignoring POWERTOOLS_SERVICE_NAME globally in Metric() since it is required by Logger.

Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (Python) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

Hello @nikevp! Thanks for opening this issue! I'd like to hear from other maintainers what they think about this, as disabling POWERTOOLS_SERVICE_NAME globally can be a problem and lead to unexpected behavior. Adding a flag to single_metric makes this very specialized and perhaps not the best developer experience.
I'm not saying that we won't analyze how to implement this or that I'm against it, but I don't have a strong opinion on this matter and I would like to ask my peers for help.
For now, if this is a mandatory requirement and is harming your production environment I might suggest you set the public service variable to None before single_metric's ContextManager flushes it. It's not the best experience, but it works as a workaround while we try to figure out a solution to this.
Something like this:
import os

from aws_lambda_powertools import single_metric
from aws_lambda_powertools.metrics import MetricUnit
from aws_lambda_powertools.utilities.typing import LambdaContext

STAGE = os.getenv("STAGE", "dev")

def lambda_handler(event: dict, context: LambdaContext):
    with single_metric(name="MySingleMetric", unit=MetricUnit.Count, value=1, namespace="TEST") as metric:
        metric.add_dimension(name="environment", value=STAGE)
        metric.add_dimension(name="xyz", value="123")
        metric.service = None

Thanks

I agree with @rubenfonseca – I think using @leandrodamascena's code snippet, you could wrap it in a function to ensure that you get what you want without having context managers everywhere in your code

Thanks for the quick reply @leandrodamascena. Your example works perfectly for my use case and I'm already using a singleton function for the logic.
I was running into issues Friday trying to pass service=None which fails due to https://github.com/aws-powertools/powertools-lambda-python/blob/fa235c0d7335265342f3f13d189d7e10af0ed0b6/aws_lambda_powertools/metrics/base.py#L18
Since you can easily remove the dimension before emitting, I no longer see any need for a code change. However, it might be worth noting in the logger documentation that adding $POWERTOOLS_SERVICE_NAME will automatically add a dimension to all metrics by default.

Hi @nikevp! Thank you for the feedback! I'm pleased to hear that you were able to resolve the issue.
We have a section in our documentation explaining that a new dimension will be created. While I believe it's well-explained, I'd appreciate your opinion on whether we should enhance this message.
https://docs.powertools.aws.dev/lambda/python/latest/core/metrics/#getting-started

Thank you.

Hey @leandrodamascena, I agree the metric ENV documentation is clear. On my side, I overlooked the implications of adding POWERTOOLS_SERVICE_NAME to the logger without considering the impact on metrics.
Here it does not indicate adding POWERTOOLS_SERVICE_NAME will impact metrics: https://docs.powertools.aws.dev/lambda/python/latest/core/logger/#getting-started
It's easy to catch in development by reviewing the logs, but only if you are setting up Metrics after Logger. If you setup the Metrics feature and later on add the Logger, a user might add POWERTOOLS_SERVICE_NAME without knowing it changes dimensions. This might not be a common migration pattern so I'm okay with the current documentation.
If you think it's worth revising to clarify the dependency, you might do something like:
There are some other environment variables which can be set to modify Logger's settings at a global scope. The POWERTOOLS_SERVICE_NAME environment variable is shared with other features and adds a default service dimension to Metrics.


Get user data from external source

Instead of only read a static json file to create users objects, it would be better to have the option to fetch the user data from an external source. This feature would allow interesting things such as load users from a database.
An optional object could be passed as argument in the FTPServer constructor to handle users.

thank u for your advice, i wrote this project when i learning workerman, so strictly, it is not a completely project, just a demo. i know lots of things to do let it to be a project.

It seems to be a simple feature. I'll try to implement this and create a pull request.


where is the 0060.zip

Where can I download the 0060.zip?  I used 0006.zip but only see radar scans in Rviz without images.

https://drive.google.com/drive/u/1/folders/1vHpkoC78fPXT64-VFL1H5Mm1bdukK5Qz
I applied the numerations for the files from http://www.cvlibs.net/datasets/kitti/eval_tracking.php
The scenario you search is now 0012

The link provided https://drive.google.com/drive/u/1/folders/1vHpkoC78fPXT64-VFL1H5Mm1bdukK5Qz only includes 13 files. 0060.zip is not one of them.  Do I miss anything here?

Yes! Previously called scenario 0060 is now scenario 0012

I see. Thanks.

No problem. Thanks for pointing out that bug


componets and data caching is not enabled when the route caching is not enabled

It seems like with this configuration (route.enabled=false) is data and component not enabled as well, because nitro plugin is not loaded
multiCache: {
route: {
enabled: false,
},
data: {
enabled: true,
},
component: {
enabled: true
},
},
After a long research, I tried to enable route because of this peace of code (maybe it is not exactly the bug, but it was my clue)
nuxt-multi-cache/dist/module.mjs:121
if (options.route?.enabled || options.cdn?.enabled || nuxt.options._prepare) {
addServerPlugin(resolve("./runtime/server/plugins/multiCache"));
}

Uff, good catch. And it didn't get caught by tests because they always enable all caches :man_facepalming: I fixed the bug and also adapted all tests to make sure this can't happen in the future.


Instructions unclear. Elbow caught in Roomba. Should I refactor?

.

So that's a yes? Been here for almost 24 hours, and I've lost all feeling in my fingers.

@bvanvliet tel://+911

According to the Roomba manual, you need to go to the hardware store and purchase at least 550 imperial inches of external vaccum hose pipe to extend from the top nozzle intake to the bottom exhaust vent of the Roomba, to provide enough reverse "blow" suction to dislodge your elbow from the narrow passage.
Before trying that out I would seriously considering patching the Roomba from nocode v.022.3246279324.32a to nocode v.0022.3246279324.33b but NOT v.022.3246279324.32c because that may cause the Roomba to short circuit leaving your elbow lodged for a short eternity. The compile instructions are documented in nocode.c.
Be sure to check the MD5 hash of nocode.c BEFORE the compiling the firmware, it should read out a string of perfect zeros, anything else means the file is corrupted, damaged, or corrupted and damaged.
It will also take at least 72 hours to compile nocode.c, so you may want to do all of this from a laptop connected to a free wifi station in your local emergency ward.

I fixed it, see my PR #1903

Okay so now there are two Roombas, and they've forced me into a small closet at gunpoint. I can see them through a small opening, and although I can't understand what they're saying, I do NOT like the way new guy keeps looking over here. There's just enough battery left on my phone to jump on here and see where I might have gone wrong, so I'm goin...
...
Imperial... damn.


test: speedup celery tests

SUMMARY
Speedup async celery tests by querying db in 0.5 s intervals instead of waiting 6s.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Hey! Could you look at it? @willbarrett @dpgaspar

Codecov Report

Merging #12885 (f350b57) into master (2adfb85) will decrease coverage by 1.51%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12885      +/-   ##
==========================================
- Coverage   64.89%   63.38%   -1.52%
==========================================
  Files        1024      489     -535
  Lines       50168    30197   -19971
  Branches     5148        0    -5148
==========================================
- Hits        32557    19139   -13418
+ Misses      17433    11058    -6375
+ Partials      178        0     -178

Flag
Coverage Δ

cypress
?

javascript
?

python
63.38% <ø> (-0.71%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/views.py
62.50% <0.00%> (-25.07%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/sql_validators/base.py
93.33% <0.00%> (-6.67%)
:arrow_down:

superset/db_engine_specs/base.py
79.59% <0.00%> (-6.38%)
:arrow_down:

superset/db_engine_specs/presto.py
82.25% <0.00%> (-6.28%)
:arrow_down:

superset/db_engine_specs/sqlite.py
90.62% <0.00%> (-6.25%)
:arrow_down:

superset/views/database/forms.py
83.33% <0.00%> (-5.56%)
:arrow_down:

superset/databases/commands/test_connection.py
84.78% <0.00%> (-4.35%)
:arrow_down:

... and 547 more

Continue to review full report at Codecov.

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

Hey! Could you look at it? @willbarrett @dpgaspar