v1/management.cattle.io.settings/system-default-registry 404 not found on New clusters deployed with Rancher 2.5.8

What kind of request is this (question/bug/enhancement/feature request):
The rancher terraform provider complains of 404 not found when trying to use rancher2_app_v2 (install an app) on a cluster as the v1 API location does not exist - https://mycluster.local/k8s/clusters/myclusterID/v1/management.cattle.io.settings/system-default-registry
On 2.5.7 rancher clusters created DO have this v1 API location, however once upgraded to 2.5.8 (or a new install of 2.5.8) any new clusters deployed will NOT have this API location which breaks at least this specific terraform resource.  I am submitting the Issue to the rancher Issues as it seems to be a change in rancher, if that is an intended change then I can open up an issue with the rancher provider to support whatever the new API/method is to get that info, if it's NOT and intended change then this is the right place :)
Steps to reproduce (least amount of steps as possible):
Create vsphere Cloud Credential in gui
Create vsphere Node Template in gui
Create vsphere cluster in gui

Kubernetes "v1.20.6-rancher1-1", Rancher 2.5.8, Ubuntu 20.04LTS nodes
wait for cluster to provision/sync.
Get Cluster ID

Result:
Browse to https://mycluster.local/k8s/clusters/myclusterID/v1/management.cattle.io.settings/system-default-registry and get a 404 error
The API location doesn't exist, so it is not necessary to use terraform to validate it doesn't exist.
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):
This behavior is seen on a fresh install of the rancher 2.5.8, on kubernetes v1.20.5-rancher1-1 (setup using rke v1.2.8) on Ubuntu 20.04LTS nodes (3 of them).
As well as an 2.5.7 cluster upgraded to 2.5.8

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"clean", BuildDate:"2021-04-08T16:31:21Z", GoVersion:"go1.16.1", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.5", GitCommit:"6b1d87acf3c8253c123756b9e61dac642678305f", GitTreeState:"clean", BuildDate:"2021-03-18T01:02:01Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

(paste the output here)

At previous Rancher versions, global settings were synced on all downstream k8s clusters API, but not anymore from v2.5.8. So global settings as system-default-registry, has to be got from local cluster always.
Submitted PR at tfp to fix the issue https://github.com/rancher/terraform-provider-rancher2/pull/663

@rawmind0 is there a workaround about the system-default-registry issue? How can I create it manually? I understand your PR will change the behavior of the provider to use the global rancher default registry?

@vkruoso , yes, the PR is changing the behavior of the provider to use the global rancher default registry. It's already included at released Rancher2 tf provider v1.15.0

Please, reopen issue if needed

Awesome! Thanks @rawmind0!


modules structure

#243

@kalbasit before merging, check the issue #243 to see if we have to do anything else. If yes, then add it as to TODO list in this PR and I'll take care of it.


LDAP.md Doc has  a FIXME comment to be cleaned up.

Environment

Python version: 3.8
Nautobot version: all

Steps to Reproduce

Load https://nautobot.readthedocs.io/en/latest/configuration/authentication/ldap/#install-system-packages

### Install System Packages

!!! danger
    FIXME(jathan): With `wheel` packages asserted, let's make doubly sure these development libraries even need to be installed anymore.

First Danger block.

Expected Behavior
Danger message verbiage to be cleaned up.

Observed Behavior
Some verbiage includes fixme and has a name assoicated, should be cleaned up.

Fixed by #1073.


Added sobol initial design

Issue #, if available: #278
Description of changes: Follow up on #280 . Fixed issues with optional dependencies for Latin and Sobol designs.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #283 into master will decrease coverage by <.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
- Coverage   89.78%   89.77%   -0.01%
==========================================
  Files         118      119       +1
  Lines        3818     3835      +17
  Branches      427      427
==========================================
+ Hits         3428     3443      +15
- Misses        309      311       +2
  Partials       81       81

Impacted Files
Coverage Δ

emukit/core/initial_designs/__init__.py
100% <ø> (ø)
:arrow_up:

emukit/core/initial_designs/sobol_design.py
88.88% <88.88%> (ø)

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 f9284cc...45a4ad3. Read the comment docs.

Thanks! You would it need to push this now though, I can't do it anymore


filenames in error messages refer to the temp file

I get error messages like
 Plugin.hs    56  35 error           bios: Ambiguous occurrence ‘pluginRules’
 It could refer to either ‘Development.IDE.Plugin.pluginRules’,
                          imported from ‘Development.IDE.Plugin’ at /tmp/haskell-lsp36476/Plugin.hs-00162-1656586003122053330.hs:33:1-63
                       or ‘Ide.Types.pluginRules’,
                          imported from ‘Ide.Types’ at /tmp/haskell-lsp36476/Plugin.hs-00162-1656586003122053330.hs:40:1-26
                          (and originally defined
                             at /home/alanz/mysrc/github/alanz/haskell-language-server/src/Ide/Types.hs:45:22-32) (lsp)

This does not allow my editor (emacs) to jump to the location in the actual file affected.
Using  hie-8.6.5 --version
Version 1.1, Git revision a4f726de33fdec4555e9ca049a9c3aaa9e230dbb (3814 commits) x86_64 ghc-8.6.5

I've seen those file names in vscode lately too.


Building/testing rosidl_runtime_rs with pure cargo produces linker errors

For the rosidl_runtime_rs package, running cargo test produces a linker error complaining about not finding -lrosidl_runtime_c. This is despite sourcing the ROS 2 distro's install directory before.
A workaround is to set the LIBRARY_PATH environment variable to the directory containing librosidl_runtime_c.so:
LIBRARY_PATH=[directory containing librosidl_runtime_c.so] cargo test

I think this is only happens for this package since it doesn't depend on rclrs. For other packages, they seem to inherit the linker search dir set in rclrs/build.rs from their dependency on rclrs, even when rclrs is only a transitive dependency. I'm adding a build.rs to rosidl_runtime_rs in #80.


pip-compile fails to override the duplicate package version

Our organisation has 200+ repos,  and it maintained global constraint file which is an online url contains all pins versions like Django<2.3.
In all repos it is defined on top.
-c https://raw.githubusercontent.com/......./common.txt
Now orgnization is shifting to django3.2, but on running upgrade command it fails.
local file has following content. it has 2 entries
-c https://raw.githubusercontent.com/......./common.txt ( it has content django<2.3)
django>2.3,<3.3

In this example common file bring django<2.3 but I expect the 2nd entry will get the priority but latest pip fails to resolve the conflict Could not find a version that matches django<2.3,<3.3,>2.3,
Previously it was working fine.
Is there any thing I am missing or its expected behaviour or is there any way to override the local file version or give priority ?
Thanks

If I'm understanding right, this is expected behavior.
The order of the lines in the input file doesn't matter, and the -c should constrain and restrict any matching packages to the common.txt versions, while satisfying the rest of the file.
And it can't resolve a single django version that's both less and more than 2.3.
I'd' probably split into multiple constraints, e.g. common-django3.txt, common-django2.txt, common.txt and start including them in the projects. Or use branches with one common.txt.
Alternatively you can use the version specifier syntax to carefully disallow what you don't want globally, but a single version spec might not fit or be sane ultimately.

Hey, it's been a while since any new discussion here.

Is there any thing I am missing or its expected behaviour or is there any way to override the local file version or give priority ?

AFAIU the behavior is expected: specifying impossible constraints yields an impossible match.
I don't know exactly what behavior you'd want for "overriding," but you could first compile the common constraints to the project's output file, then compile your project's requirements (without setting the common constraints) over that same output file, and I think that might achieve what you're after. You would be losing the guarantees of setting a proper constraints file.
I'm going to close this, but if you want to propose a well defined overriding/prioritizing feature, please do so! I don't think line order will be suitable, since line order doesn't matter to pip itself, so it would be unexpected and/or difficult.


Can monitors return to “active” automatically?

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I monitor my websites with Uptime Kuma
A site may go down, but come back up 15 minutes later.
In that case I would expect the monitor to return to “active”. It would be nice to receive an email as well.
Instead, I apparently need to log in, pause the monitor, and then resume it again to get it back to “active”.
Am I missing something?
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.15
💻 Operating System and Arch
Docker image on Ubuntu
🌐 Browser
Firefox
🖥 Deployment Environment

Runtime: Docker version 27.3.1, build ce12230
Database: sqlite/embedded
Filesystem used to store the database on: EXT4
number of monitors: 16

That is a bug.
Is the bug reproducible?
What monitors are you using?

Yes, it happened twice.
It is a monitor on a Nextcloud instance https://URL/status.php returns a JSON, but I did not add a JSON check in the configuration.

Monitor Type: HTTP(s)
URL:  ...
Heartbeat Interval: 1800
Retries: 10
Resend Notification if Down X times consecutively: 0
Method: GET
Body Encoding: JSON
Body: [empty]
Accepted Status Codes: 200

I now tested it with another server, but could not reproduce the issue any more.
I have now deleted the monitor and recreated it.
Let's see what is happening.

The copied monitor works well.
Thanks for helping.


File::close(self) -> std::io::Result<()> for the std

Gitoxide shouldn't rely on File::drop() after writing files as it squelches errors. Instead, use io-close but push for the ability to close files to be added to the standard library.
Creating a PR for it would be the way to go.

Are there places in gitoxide where the squelching of errors could be avoided, or meaningfully lessened, by doing an explicit flush before dropping a file that is open for write?

A quick search revealed that flush() is called in a couple of places already. It would also be a question if close() is necessarily flushing synchronously as well and thus would detect all possible errors - those related to flushing and those related to closing.
One might also think that flushing before closing (explicitly or implicitly) would always lure out errors related to the prior write.


Rails 7 Update

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Superceded by: https://github.com/alphagov/smart-answers/pull/5892


new 0.5.0 version

peft 0.5.0 required by snowflake-ml-python

upstream
[requirements] (https://github.com/huggingface/peft/blob/v0.5.0/pyproject.toml)
skip test_prompt_encoder_warning_num_layers and upstream issue raised, see https://github.com/huggingface/peft/issues/1066
skip some windows test (test_feature_extraction_models.py or test_decoder_models.py or test_encoder_decoder_models.py) relaying on pytorch LAPACK - it needs pytorch fix , will talk with snowflake if it is good enough for now

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps and --no-build-isolation.
WARNING: clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
ERROR: clone/recipe/meta.yaml:53: missing_license_family: The recipe is missing the about/license_family key.
ERROR: clone/recipe/meta.yaml:53: missing_summary: The recipe is missing a summary
ERROR: clone/recipe/meta.yaml:53: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: clone/recipe/meta.yaml:53: missing_dev_url: The recipe is missing a dev_url
WARNING: clone/recipe/meta.yaml:53: missing_description: The recipe is missing a description
Errors were found

Is abs.yaml missing?

Is abs.yaml missing?

no, this dependency for snowflake-ml-python which goes to defaults

It didn't pick up the latest build of pytorch 2.0.1. It used 2.0.1-cpu_py38hb0bdfb8_0while the latest is2.0.1-cpu_py39hb0bdfb8_1`.

I don't know why it's not picking up the latest build. I'll investigate further.

Windows passed now. Not too sure what happened.


Problem with SMPTE ST 2094 App 4, Version 0

Hello!
I am having problems with HDR10+ SMPTE ST 2094 App 4, Version 0 metadata.

Starting from version 1.3.1 the --skip-validation flag was added, which allows to extract and parse ST2094-40 version 0.
And indeed it is extracted without problems, and you get a perfect json.
But the problem is when injecting those metadata, it seems that hdr10plus_tool always injects them as version 1 instead of keeping and respecting the original version (version 0 in this case).
For example:

MediaInfo of the original source:
HDR format: SMPTE ST 2094 App 4, Version 0, HDR10 compatible

MediaInfo of the video injected with HDR10+ metadata extracted from the previous source:
HDR format: SMPTE ST 2094 App 4, Version 1, HDR10 compatible

Would it be possible to make hdr10plus_tool not modify the version (and not modify anything in general) to get a result identical to the original source?
Thank you very much for everything!
Regards.

P.S.
It should be noted that in the json file the version is the correct one:
  "JSONInfo": {
    "HDR10plusProfile": "B",
    "Version": "0.0"
  }

So I would guess that the problem will be in the inject command and not in the json file.

The problem seems to be here:
https://github.com/quietvoid/hdr10plus_tool/blob/main/hdr10plus/src/metadata_json.rs#L325
I have tried changing 1 to 0 and then compile a new executable with cargo build --release.
And indeed the new video with this injected metadata now already appears as version 0 instead of 1.

Obviously I have done this just to test, since this is wrong because the version will always be 0 (then version 1 would convert it to version 0).
The ideal would be to make application_version dynamic and get that value from the metadata, instead of it always being a fixed value. But I don't know how to do it.
Best regards and thanks again!

P.S.
In order to test it I also needed to change 1 to 0 here, otherwise I got an error:
https://github.com/quietvoid/hdr10plus_tool/blob/main/hdr10plus/src/metadata.rs#L197
https://github.com/quietvoid/hdr10plus_tool/blob/main/hdr10plus/src/metadata.rs#L203

Version 0 is legacy and not supported by the HDR10+ LLC specification.
It shouldn't be used.

There's no good reason to support the legacy version.


Running mxnet aborted.

2016-07-26 20:05:53,076 Node[0] start with arguments Namespace(batch_size=256, data_dir='./dataset/', gpus='0,1,2,3', kv_store='local', load_epoch=None, lr=0.05, lr_factor=0.1, lr_factor_epoch=10.0, model_prefix=None, network='inception-bn-28-small', num_epochs=200, num_examples=61479, save_model_prefix='./model/mxnet')
[20:05:53] src/io/iter_image_recordio.cc:67: Loaded ImageList from ./dataset/train.lst 61479 Image records
[20:05:53] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: ./dataset/train.rec, use 4 threads for decoding..
[20:05:53] src/io/./iter_normalize.h:103: Load mean image from ./dataset/mean.bin
[20:05:54] src/io/iter_image_recordio.cc:67: Loaded ImageList from ./dataset/test.lst 42393 Image records
[20:05:54] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: ./dataset/test.rec, use 4 threads for decoding..
[20:05:54] src/io/./iter_normalize.h:103: Load mean image from ./dataset/mean.bin
2016-07-26 20:05:54,776 Node[0] Start training with [gpu(0), gpu(1), gpu(2), gpu(3)]
[20:05:58] /home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:05:58] src/operator/./pooling-inl.h:80: Check failed: (param_.stride[0]) == (param_.stride[1]) Only same stride is supported now
[20:05:58] /home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:05:58] src/engine/./threaded_engine.h:295: [20:05:58] src/operator/./pooling-inl.h:80: Check failed: (param_.stride[0]) == (param_.stride[1]) Only same stride is supported now
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
  what():  [20:05:58] src/engine/./threaded_engine.h:295: [20:05:58] src/operator/./pooling-inl.h:80: Check failed: (param_.stride[0]) == (param_.stride[1]) Only same stride is supported now
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
start.sh: line 3: 16067 Aborted                 python -u train_music.py --data-dir ./dataset/ --save-model-prefix ./model/mxnet --batch-size 256 --num-epoch 200 --lr-factor 0.1 --lr-factor-epoch 10

My mxnet.version = 0.5.0
So, what to do with this?
Below is my net's configuration:
    data = mx.symbol.Variable('data')
    label = mx.symbol.Variable('softmax_label')
    #first conv 596*256
    conv1 = mx.symbol.Convolution(data=data, kernel=(128,4), num_filter=256)
    #first max pool 149*256
    maxpool1 = mx.symbol.Pooling(data=conv1, pool_type='max', kernel=(1, 4), stride=(1, 4))
    #second conv 146*256
    conv2 = mx.symbol.Convolution(data=maxpool1, kernel=(1,4), num_filter=256)
    #second max pool 73*256
    maxpool2 = mx.symbol.Pooling(data=conv2, pool_type="max", kernel=(1, 2), stride=(1,2))
    #third conv 70*512
    conv3 = mx.symbol.Convolution(data=maxpool2, kernel=(1,4), num_filter=512)
    #third pool 35*512
    maxpool3 = mx.symbol.Pooling(data=conv3, pool_type="max", kernel=(1,2), stride=(1,2))
    #below is for global pooling (only mean)
    meanpool1 = mx.symbol.Pooling(data=maxpool3, pool_type="avg", kernel=(1,35), stride=(1,1))
    #fc1 2048-unit
    fc1 = mx.symbol.FullyConnected(data=meanpool1, num_hidden=2048)
    relu1 = mx.symbol.Activation(data=fc1, act_type="rely")
    drp1 = mx.symbol.Dropout(data=relu1,p=0.5);
    #dropout1
    fc2 = mx.symbol.FullyConnected(data=drp1, num_hidden=2048)
    relu2 = mx.symbol.Activation(data=fc2, act_type="rely")
    drp2 = mx.symbol.Dropout(data=relu2, p=0.5)
    fc3 = mx.symbol.FullyConnected(data=drp2, num_hidden=40)
    #loss
    net = mx.symbol.LinearRegressionOutput(data=fc3, label=label, name='mseloss')

And after I set stride[0] to be the same as stride[1]: new error:
2016-07-26 20:13:10,738 Node[0] start with arguments Namespace(batch_size=256, data_dir='./dataset/', gpus='0,1,2,3', kv_store='local', load_epoch=None, lr=0.05, lr_factor=0.1, lr_factor_epoch=10.0, model_prefix=None, network='inception-bn-28-small', num_epochs=200, num_examples=61479, save_model_prefix='./model/mxnet')
[20:13:11] src/io/iter_image_recordio.cc:67: Loaded ImageList from ./dataset/train.lst 61479 Image records
[20:13:11] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: ./dataset/train.rec, use 4 threads for decoding..
[20:13:11] src/io/./iter_normalize.h:103: Load mean image from ./dataset/mean.bin
[20:13:11] src/io/iter_image_recordio.cc:67: Loaded ImageList from ./dataset/test.lst 42393 Image records
[20:13:11] src/io/iter_image_recordio.cc:212: ImageRecordIOParser: ./dataset/test.rec, use 4 threads for decoding..
[20:13:11] src/io/./iter_normalize.h:103: Load mean image from ./dataset/mean.bin
2016-07-26 20:13:12,440 Node[0] Start training with [gpu(0), gpu(1), gpu(2), gpu(3)]
[[20:13:16] /home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
[20:13:16] 20:13:16] /home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
/home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
[20:13:16] /home/username/mxnet/dmlc-core/include/dmlc/logging.h:245[20:13:16] [20:13:16] : [20:13:16] src/engine/./threaded_engine.h:295: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
/home/username/mxnet/dmlc-core/include/dmlc/logging.h:/home/username/mxnet/dmlc-core/include/dmlc/logging.h:245: [20:13:16] src/engine/./threaded_engine.h:295: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
245: [20:13:16] src/engine/./threaded_engine.h:295: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
terminate called recursively
  what():  [20:13:16] src/engine/./threaded_engine.h:295: [20:13:16] /home/username/mxnet/mshadow/mshadow/./././dot_engine-inl.h:431: Check failed: dst.size(0) == sleft[0] && dst.size(1) == sright[1] && sleft[1] == sright[0] dot-gemm: matrix shape mismatch
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.terminate called recursively
start.sh: line 3: 16315 Aborted                 python -u train_music.py --data-dir ./dataset/ --save-model-prefix ./model/mxnet --batch-size 256 --num-epoch 200 --lr-factor 0.1 --lr-factor-epoch 10

please update to current master

Thanks, it works.


Add NAND NOR XOR table in mathematical reasoning section with their info

Is your feature request related to a problem? Please describe.

Add NAND NOR XOR table in mathematical reasoning section with their info

Describe the solution you'd like

A clear and concise description of what you want to happen.

I would like to work on this, will you assign me?

@sairish2001 can I work??


Add ability to ignore a file

Hello,
Could it be possible to add a feature to ignore a file so imports will not be ordered ?
For instance, in my project we used previously "prettier-plugin-organize-imports", and they added a feature later to ignore a file from being formatted by the plugin, by adding the following comment :
// organize-imports-ignore
Full explaination could be found here, and related commit also here, that consists only of having an if condition to check if the file constains that comment.
Thank you.

For the whole file or for a section / single tag?
There's also always the option if just disabling the sorting for specific files inside the .prettierrc.
If you're interested in that I can give you an example.

Thanks for your reply.
We finally manage to ignore some files completly by using the .prettierignore and .prettierrc

If you only want to disable sorting for a specific file (or file-matcher) you can do it in the following way:
{
  "overrides": [
    {
      "files": "unsorted.html",
      "options": {
        "attributeGroups": ["$DEFAULT"],
      }
    }
  ]
}

I'll mark this as resolved for now, if there are any other questions just comment here or in a new issue :)


feat: rewrite Voice module in typescript

Summary
Rewrote the module in typescript.
There are still some types such as Events that are using any, and it should have a more specific type, but its a good starting point.

Test Plan
run example on ios/android and checked that it works as before.

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[x] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

This is awesome! One thing that would make this great, is adding the type check as part of the ci/cd. Could you add that?

Sure, I'll add that tonight!

@safaiyeh should be good to go!


[graphql-request] print is imported as type

Describe the bug
Ever since #5798 fixed #5798, print is imported as type when setting useTypeImports config to true.
This breaks the generated code.

To Reproduce
Steps to reproduce the behavior:
https://codesandbox.io/s/aged-bird-qfoju?file=/types.ts:1556-1604

My GraphQL schema:

type Query {
  user(id: ID!): User!
}

type User {
  id: ID!
  username: String!
  email: String!
}

My GraphQL operations:

query user {
  user(id: 1) {
    id
    username
    email
  }
}

My codegen.yml config file:

schema: schema.graphql
documents: document.graphql
overwrite: true
generates:
  types.ts:
    config:
      rawRequest: true
      useTypeImports: true
    plugins:
      - typescript
      - typescript-operations
      - typescript-graphql-request

Expected behavior
There are no errors in types.ts
Environment:

OS: macOS 11.2.3
@graphql-codegen/typescript-graphql-request: ^4.3.1
NodeJS: v16.5.0

Additional context
#5798
#5855

#7059 is merged

Available in<EMAIL_ADDRESS>🎉
(fyi @MichaelDeBoey )


Question about contributor recognition after PR merge

Hi team! @alabulei1 ,
Thank you for merging my PR #60 ! However, I noticed that my contribution does not appear under the "Contributors" section of the repository. Could you please clarify if there is anything else I need to do to be recognized as a contributor?
Thank you for your assistance!

It is updated by GitHub. We do not know how they decide to display names in that section. Thanks.

It is updated by GitHub. We do not know how they decide to display names in that section. Thanks.

Thanks for answer.


Svelte 4?

Will this be compatible with svelte 4?

It should already be compatible with svelte 4, there is no peer dependency or anything that requires changes in the library


Can't pull from an array nested in an object nested in an object

I have this class:
var Thing = Astro.Class({
	name: 'Thing',
	collection: Things,
	fields: {
		obj: {
			type: 'object',
			default: function() {
				return {};
			}
		}
	}
});

Now, if I do this:
var thing = new Thing();
thing.set('obj.asd', {});
thing.save()
thing.set('obj.asd.arr', []);
thing.save();
thing.push('obj.asd.arr', 'admin') // this works
thing.save()
thing.pull('obj.asd.arr', 'admin') // this throws the following error:

TypeError: Cannot call method 'plain' of undefined
    at packages/jagi_astronomy/lib/modules/fields/pull.js:94:1
    at packages/underscore/packages/underscore.js:213:1
    at Array.some (packages/es5-shim/packages/es5-shim.js:575:1)
    at _.some._.any (packages/underscore/packages/underscore.js:262:1)
    at Function._.find._.detect (packages/underscore/packages/underscore.js:212:1)
    at packages/jagi_astronomy/lib/modules/fields/pull.js:93:1
    at runCallback (packages/jagi_astronomy/lib/modules/fields/utils.js:16:1)
    at next (packages/jagi_astronomy/lib/modules/fields/utils.js:46:1)
    at next (packages/jagi_astronomy/lib/modules/fields/utils.js:63:1)
    at next (packages/jagi_astronomy/lib/modules/fields/utils.js:63:1)

However, thing.obj.asd.arr is definitely not undefined:
console.log(thing.get('obj.asd.arr'))

[ 'admin' ]

In deed there was a bug. I've fixed it in the 1.1.5 version. I hope it's the last one bug in the pull method. I haven't made detailed tests for this method and that's why there were many bug there.

Thanks! It works now!

Good :)


display "no tokens" when connection succesfull, but user dont own any token

Hello, TN UI displays same message when "no tokens" or "connect error", it shows "Load" in both cases.
Its ambugiuous for user, better show different messages

@oleggrib this was intended to allow the users to try again once it had failed.  Where the application at this time doesn’t find tokens that have become available e.g. a NFT transfer completed, or off chain Token generated and assigned to that user.
We could as solution change this to;

load for when it’s a first request
retry for when it’s a further attempt to load the tokens

Please let me know your thoughts? @oleggrib

@nicktaras  we can show "Load" when token not loaded yet.
"no tokens" when empty token list received
user can use "Refresh" button to reread tokens

Hi @oleggrib,
Thanks for the feedback.
Version 3.2.0 introduces the following text configurations to allow you to flexibly manage the text, with regards to the scenario you have shared.

Auto Load tokens > "Load"
User Clicks to load tokens > "Retry"
When tokens are not found the notification utilises the text from 'noTokensFoundEvent'
When tokens are found the following options can be utilised, 'balancesFoundEvent' and 'nftsFoundEvent'
Off chain Tokens in this release utilise the 'nftsFoundEvent' text

uiOptions: {
          loadAction: "Load",
          repeatAction: "Retry",
          noTokensFoundEvent: 'No Tokens Found',
          balancesFoundEvent: 'Balance Found',
          nftsFoundEvent: 'Token(s) Found',
        },


⚠ Bofrost.at has degraded performance

In 0584681, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1545 ms

Resolved: Bofrost.at performance has improved in a666914 after 9 minutes.


Search tree: adopt templated rendering

Today the search tree is using ActionsRenderer which is basically a LegacyRenderer that has to create all DOM elements for each item. Much better to directly extend IRenderer and adopt the templated approach to rendering (see CallStackRenderer for example).

Awesome!

@roblourens FYI, I changed the Search tree renderer to not to use the Legacy renderer, instead use the templated IRenderer. Heard that you wanted to do some changes in this area, hence mentioning you.


Add option to preserve order in output

Currently order is not ensured, its whatever fulfills the Promise.race() first: https://github.com/voxpelli/buffered-async-iterable/blob/4d9bfd2c726ccbf465a3148ef71d37e5594ba505/index.js#L197
For modules like async-htm-to-string order is important

Solved in #29


Create virtualization.md for hindi language

Describe your changes
create a new virtualization.md for Hindi lang
Related issue number or link (ex: resolves #issue-number)
#2231
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

This is a duplicate PR,  PR #1674 already exists for these changes

Closing this one, as it's a duplicate


needs tests

They would probably help with the refactoring I'm attempting to do on Issue #23 and Issue #24.
BUT
I don't wish to place this as a requirement on the 1.0 milestone (for macOS):  I just want to get on with "finishing" dotfiles so that I can begin to port / update it for Linux, especially SUSE.
There's an xUnit test framework for bash called shUnit
The code is at kward/shunit2

bats seems like another good candidate.


Is it possible to have blocking for toasts, so only 1 is renderred?

As the title suggests, is it possible to limit the number of toasts displayed to some integer value? So subsequent triggers of toast() will be blocked. I did not come across anything in the docs which implied this, so I was wondering if there's any way to implement this. Thanks! (great package by the way!)

Setting an integer limit is currently not possible. What is possible though is that  you can provide a fixed id to your toast. This will prevent concurrent toasts of the same kind.
toast("Copied to clipboard.", {
	id: "copy"
})

Would that work for your use-case?


🔧 [Refactor] ListTransaction must return the balance

🔧 Refactor ticket

ListTransaction must return the balance:
We get a listTransaction result after a successful sendGDD, but we don't get the balance. The backend need to deliver this in order for the frontend to display the correct new balance. Furthermore it would be wise to deliver the balance on every listTransaction call as well, since the amount you have displayed in the menu as balance might differ (because you sent or received new GDD) from your transaction list.

fixed with commit: https://github.com/gradido/gradido/pull/273/commits/b10886a8d359c1fd2b701722113674705c40d440
in PR #273


How to use the xarm package without moveit wrappers

Hi, we are on the process of developing a ros package that is meant to be usable with any moveit2 enabled robot arm. As hardware we have an xArm7 and would like to both interact with simulations and the real hardware using moveit's standard interface.
Our current goal is to simply to push a goal pose using Moveit's Move group script.
The current node we are trying to run is the following:
#include <memory>

#include <rclcpp/rclcpp.hpp>
#include <moveit/move_group_interface/move_group_interface.h>

int main(int argc, char *argv[])
{
  auto node_name = "arm_controller";
  // Initialize ROS and create the Node
  rclcpp::init(argc, argv);
  auto const node = std::make_shared<rclcpp::Node>(
      node_name,
      rclcpp::NodeOptions().automatically_declare_parameters_from_overrides(true));

  // Create a ROS logger
  auto const logger = rclcpp::get_logger(node_name);

  RCLCPP_INFO(logger, "node started...");

  // Create the MoveIt MoveGroup Interface
  using moveit::planning_interface::MoveGroupInterface;
  auto move_group_interface = MoveGroupInterface(node, "xarm7");

  RCLCPP_INFO(logger, "created MoveGroupInterface");

  // Set a target Pose
  auto const target_pose = []
  {
    geometry_msgs::msg::Pose msg;
    msg.orientation.w = 1.0;
    msg.position.x = 0.28;
    msg.position.y = -0.2;
    msg.position.z = 0.5;
    return msg;
  }();
  move_group_interface.setPoseTarget(target_pose);

  // Create a plan to that target pose
  auto const [success, plan] = [&move_group_interface]
  {
    moveit::planning_interface::MoveGroupInterface::Plan msg;
    auto const ok = static_cast<bool>(move_group_interface.plan(msg));
    return std::make_pair(ok, msg);
  }();

  // Execute the plan
  if (success)
    move_group_interface.execute(plan);
  else
    RCLCPP_ERROR(logger, "Planing failed!");

  // Shutdown ROS
  rclcpp::shutdown();
  return 0;
}

The first thing we tried was launching the xarm_moveit_config package with
ros2 launch xarm_moveit_config xarm7_moveit_fake.launch.py

And then running the node with ros2 run <our npackage> <our_node>. This would not work because the xarm package does not publish /robot_description_semantic which contains the .srdf file. We see that currently xarm_ros2 passes that as a parameter during the launch process.
Then, after reading a bit about launching Moveit Configuration. We using the following .launch.py file:
import os
from launch import LaunchDescription
from launch.actions import DeclareLaunchArgument
from launch.substitutions import LaunchConfiguration, PathJoinSubstitution
from launch_ros.actions import Node
from launch_ros.substitutions import FindPackageShare
from ament_index_python.packages import get_package_share_directory
from moveit_configs_utils import MoveItConfigsBuilder

def generate_launch_description():

    # Define xacro mappings for the robot description file
    launch_arguments = {
        # "robot_ip": "xxx.yyy.zzz.www",
        # "use_fake_hardware": "true",
        # "gripper": "robotiq_2f_85",
        # "dof": "7",
    }

    # Load the robot configurationl
    moveit_config = (
        MoveItConfigsBuilder("xarm7", package_name="xarm_moveit_config")
        .robot_description(
            file_path="urdf/xarm7/xarm7.urdf.xacro",
            mappings=launch_arguments,
        )
        .trajectory_execution(file_path="config/xarm7/fake_controllers.yaml")
        .planning_scene_monitor(
            publish_robot_description=True, publish_robot_description_semantic=True
        )
        .planning_pipelines(
            pipelines=["ompl", "stomp", "pilz_industrial_motion_planner"]
        )
        .to_moveit_configs()
    )

    # Start the actual move_group node/action server
    run_move_group_node = Node(
        package="our_package",
        executable="arm_controller",
        output="screen",
        parameters=[moveit_config.to_dict()],
    )

    return LaunchDescription(
        [
            run_move_group_node,
        ]
    )

However, MoveItConfigsBuilder is unable to interpret the xarm_moveit_config package.
WARNING:root:Cannot infer URDF from `/home/redacted/Lab/raven/raven_ws/install/xarm_moveit_config/share/xarm_moveit_config`. -- using config/xarm7.urdf
WARNING:root:Cannot infer SRDF from `/home/redacted/Lab/raven/raven_ws/install/xarm_moveit_config/share/xarm_moveit_config`. -- using config/xarm7.srdf
[ERROR] [launch]: Caught exception in launch (see debug for traceback): Caught multiple exceptions when trying to load file of format [py]:
 - ParameterBuilderFileNotFoundError: "File /home/redacted/Lab/raven/raven_ws/install/moveit_configs_utils/share/moveit_configs_utils/default_configs/stomp_planning.yaml doesn't exist"
 - InvalidFrontendLaunchFileError: The launch file may have a syntax error, or its format is unknown

In look for alternatives, we then tried using moveit_setup_assistant, feeding it the .urdf generated and published in /robot_description. Which would lead to the computer freezing and needing a reboot in order to get it back to work.
In short, we would like to use xarm7 both in sim and in real life by means of the cpp moveit move_group interface and we do not know how. We are not experienced with moveit and would really appreciate your help. Thank you.

Hi, we found a way to get this, we are basically using the same launch file as in xarm_planner -> _robot_planner.launch.py to launch our node. Whilst we setup the rviz simulation with  ros2 launch xarm_moveit_config _robot_moveit_fake.launch.py no_gui_ctrl:=true.
With this then we can use moveit_group in our node without a problem.
I understand that this issues is related to the lack of compatibility with MoveItConfigsBuilder
Still, we are left in doubt on wether this is the best or meant way to do things. Any help or advice would be appreciated.

Hi @gaspatxo, our moveit_config packages are not automatically generated by moveit setup assistant, and our xarm_moveit_config package does not obey the naming rule for MoveitConfigsBuilder inferring, since the package for xarm7 should have the name xarm7_moveit_config.  We designed our packages to present a more compact structure, because we have many robot models, if each model would have one moveit pakage, then "xarm_ros2" would appear to be bulky and full of redundant similar files (like our ROS1 repository).
For the above reason, most of our launch and configuration files are hand-edited instead of automatically generated, which may result in difficulties for MoveitConfigsBuilder analyzing. So I am afraid you may have to do additional work in order to fit xarm7 into a generalized software structure. I wonder if making a new (fake) xarm7_moveit_config package for MoveitConfigsBuilder while overwriting the default files in initialization with the real .srdf and .urdf files etc would work.
I am also glad that you have already found a way around. xarm_planner package is the one we use for move_group interface coding control. Since it will not be easy to transfer the structure of this repo to be compatible with MoveitConfigsBuilder, I guess you can continue going this way if it is OK for your application.

Hi @gaspatxo, thanks for your feedback. We started the development of ros2 support early, and by that time, there is no MoveitConfigsBuilder available and we could not foresee the way it interprets the package, which requires naming convention. We did the naming change to make the structure less redundant. After starting the launch files according to the ReadMe file, our package will still work in ROS2 eco-system with other ros2 enabled devices.
However, I agree with your point here. We will do some more thoughts to see if there is a better way to support MoveitConfigsBuilder and robot-agnostic development.

Just for completion. If you want to use your UFactory robot with the standard moveit API, here is a way to do it (mind that this is for humble, it probably works with any other ros2 branch but I do not guarantee it):

Copy _robot_planner.launch.py to the launch directory in your package
Optionally, though convenient, set the right default parameters for your hardware (this will spare you from specifying it in your ros2 launch command). In case of doubt you can check files such as uf850_planner_fake.launch.py
Delete the xarm_gripper_planner_node launcher:

    if add_gripper.perform(context) in ('True', 'true') and use_gripper_node.perform(context) in ('True', 'true'):
        xarm_gripper_planner_node = Node(
            name=node_name,
            package='xarm_planner',
            executable='xarm_gripper_planner_node',
            output='screen',
            parameters=[
                robot_description_parameters,
                {'PLANNING_GROUP': 'xarm_gripper'},
            ],
        )
        nodes.append(xarm_gripper_planner_node)

Modify the xarm_planner_node parameters to launch your executable:

line 32:
    node_executable = LaunchConfiguration('node_executable', default='<YOUR_EXECUTABLE>')

line 120:
    xarm_planner_node = Node(
        name=node_name,
        package='<YOUR_PACKAGE>',
        executable=node_executable,
        output='screen',
        parameters=[
            robot_description_parameters,
            {
                'robot_type': robot_type,
                'dof': dof
            },
            xarm_planner_parameters,
        ],
    )

Hi, we found a way to get this, we are basically using the same launch file as in xarm_planner -> _robot_planner.launch.py to launch our node. Whilst we setup the rviz simulation with ros2 launch xarm_moveit_config _robot_moveit_fake.launch.py no_gui_ctrl:=true.
With this then we can use moveit_group in our node without a problem.
I understand that this issues is related to the lack of compatibility with MoveItConfigsBuilder
Still, we are left in doubt on wether this is the best or meant way to do things. Any help or advice would be appreciated.

Hey i had encountered the same problem, i am trying to use c++ binding, hello_moveit.cpp on uf850, can you guide me through the solution? thanks


"WARNING:tensorflow:Variable += will be deprecated"

System information

Have I written custom code (as opposed to using example directory):  No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows 10.0.16299.1217
TensorFlow backend (yes / no):  yes
TensorFlow version:  1.13.1
Keras version:  2.2.4 to current
Python version:  3.7.1
CUDA/cuDNN version: NA
GPU model and memory:  NA

Describe the current behavior
When running the example "neural_style_transfer.py", getting a warning as below:
"WARNING:tensorflow:Variable += will be deprecated"
Describe the expected behavior
Deprecation warnings should be heeded and replaced with compliant code.
Code to reproduce the issue
Use existing example - examples/neural_style_transfer.py
Other info / logs
A typical output when running neural_style_transfer is below:
Using TensorFlow backend.
<>WARNING:tensorflow:Variable += will be deprecated. Use variable.assign_add if you want assignment to the variable value or 'x = x + y' if you want a new python Tensor object.

@prabindh Warnings are due to tensorflow which is going to release 2.0 (Huge shift from TF1.x) and deprecate TF1.x related API's in the future. You could disable those warning by using simple code given below. Add the following code at the top of your code to disable warning or info etc. Thanks!
import os
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '2'  # or any {'0', '1', '2'}
# 0 (all logs shown),
# 1 to filter out INFO logs,
# 2 to additionally filter out WARNING logs, and
# 3 to additionally filter out ERROR logs

Thanks. Agreed we could filter out all warnings, but then all other warnings would be masked, which isnt the right thing in my opinion. What stops us from moving to like below commit,
https://github.com/prabindh/keras/commit/063bfcbf43c0e315e807944caeeed07a44a0d33f

Any update ?


Dump the assessments to the evaluation CSV right after running a task, to avoid losing information if the evaluation errors

Part of #237

Closing in favor of https://github.com/symflower/eval-dev-quality/pull/241


kexec-tools: 2.0.25 -> 2.0.26

Description of changes
Update kexec-tools to current latest version, see: https://www.spinics.net/lists/kexec/msg30743.html.
This change was motivated by manual test run of https://github.com/tiiuae/sbomnix/tree/main/scripts/nixupdate, which pointed out that kexec-tools is currently not up-to-date in nix unstable.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@OfBorg test kexec


Add argument to skip showing pyvista plot

With this change, the pyvista Plotter object will be assigned to the pv property of the pyaedt ModelPlotter class, and it won't be shown. In this way, the user can further manipulate the pyvista plot before exporting it.

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

Project coverage is 34.60%. Comparing base (728c5a9) to head (ef85ebf).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4452       +/-   ##
===========================================
- Coverage   81.50%   34.60%   -46.90%
===========================================
  Files         109      109
  Lines       53196    53197        +1
===========================================
- Hits        43359    18411    -24948
- Misses       9837    34786    +24949


Chunk ... no longer exists

I'm using Nitrite database to store a million documents and then perform different read operations on them (for benchmarking).
After storing the documents, while trying to read all documents via collection.find().iterator(), I get the following exception:
java.lang.IllegalStateException: Chunk 162 no longer exists [1.4.196/9]
	at org.h2.mvstore.DataUtils.newIllegalStateException(DataUtils.java:765)
	at org.h2.mvstore.MVStore.getChunkIfFound(MVStore.java:953)
	at org.h2.mvstore.MVStore.getChunk(MVStore.java:935)
	at org.h2.mvstore.MVStore.readPage(MVStore.java:1943)
	at org.h2.mvstore.MVMap.readPage(MVMap.java:741)
	at org.h2.mvstore.Page.getChildPage(Page.java:217)
	at org.h2.mvstore.Cursor.fetchNext(Cursor.java:150)
	at org.h2.mvstore.Cursor.next(Cursor.java:50)
	at org.dizitart.no2.internals.DocumentCursor$DocumentCursorIterator.nextMatch(DocumentCursor.java:115)
	at org.dizitart.no2.internals.DocumentIterator.next(DocumentIterator.java:54)
	at org.dizitart.no2.internals.DocumentIterator.next(DocumentIterator.java:32)
        ...

Then, the next smaller read operations return more values than they should.(Not related to this problem)
I create the database with the default options and three indexes, and it occurs with and without enabled compressing. The used version is 2.1.0.
Is a million documents too much for Nitrite? The same data and test cases work fine with pure MVStore. Inserting only ~340 entries works fine too.

Could you please post the complete code here for both nitrite and mvstore alone to reproduce this error?

I've tested it with another dataset with ~270k entries that worked fine as well.
The project is really big and many classes are involved, so I hope the adapter classes are enough for you:
NitriteDB:
Constructing DB:
Nitrite db = Nitrite.builder().compressed()
			.filePath(storageDir + File.separatorChar + DB_FILE).openOrCreate();
NitriteCollection collection = db.getCollection(COLLECTION_NAME);
this.collectionAccessor = new CollectionAccessor(db, collection);

CollectionAccessor.java: https://pastebin.com/N63681vg
IteratorWrapper.java: https://pastebin.com/vpeXUMcE
MVStore:
H2MVStoreMultiMap.java: https://pastebin.com/V8wsUXP2
IteratorWrapper is basically the same, without the document converting.

MVStore example is not giving this exception is because you are creating only one map there. But in nitrite, it creates 6 maps - 1 for collection, 3 for indices, 2 for meta information.
Can you please do me a favor and create 4 mvmaps in mvstore example and insert data in all of them simultaneously in the same fashion? I just need to confirm if it is the volume of data that is creating problem or not. I might take this case to mvstore developers, so it will be better if I give as much info as possible.

I forgot to mention that H2MVStoreMutliMap class is instantiated three times.
But why do you want me to create only 4 mvmaps? What are the other two for then, because indexing should be enabled there too.
However, instantiating my MVStore multimap both 4 and 6 times works without problems.

I just want to make sure, you are creating 4 mvmaps in the same store and inserting 1 million of records in each of them parallely right?
I asked for 4 mvmaps, as other 2 are meta maps, it contains only few records, so does not contribute to this problem at all.

I just want to make sure, you are creating 4 mvmaps in the same store and inserting 1 million of records in each of them parallely right?

Yes, exactly.

Thank you. Let me re-create the scenario and see what can be done about this.

I'm currently trying to reduce the code for reproducing the error as well, and it looks like it depends on an outer layer that I didn't include here. I'll keep you updated if there maybe is a mistake on my side.

Nevermind that, I just didn't get the exception sometimes randomly.
So here is the reduced code that definitely threw one for me:
https://pastebin.com/kk9P06sG

Thanks a lot for the time. I am looking into it.

Fix is ready in 2.1.1-SNAPSHOT. Please test and confirm

I'm using Nitrite via Maven, but it's not available there yet?

https://oss.sonatype.org/content/repositories/snapshots/org/dizitart/nitrite/2.1.1-SNAPSHOT/

Thanks. Yes, now it works.


MongoDB connection not working in example app

Default configuration.
2016-08-31T22:34:29.533Z	Worker(6792/S1N1)	[example]	TypeError: Cannot read property 'insert' of undefined; /example/www/examples/mongodb/insertData.json/get.js:3:25; Domain.fnWrapper (\lib\impress.client.js:805:15); Domain.run (domain.js:221:14); \lib\impress.client.js:845:33; \lib\impress.application.js:143:9; FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)

2016-08-31T22:35:04.496Z	Worker(6792/S1N1)	[example]	TypeError: Cannot read property 'collections' of undefined; /example/www/examples/mongodb/getCollections.json/get.js:3:21; Domain.fnWrapper (\lib\impress.client.js:805:15); Domain.run (domain.js:221:14); \lib\impress.client.js:845:33; \lib\impress.application.js:143:9; FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)

The connection seems to be fine on Mongo side though:
2016-09-01T01:34:18.503+0300 I NETWORK  [initandlisten] waiting for connections on port 27017
2016-09-01T01:34:23.859+0300 I NETWORK  [initandlisten] connection accepted from <IP_ADDRESS>:55969 #1 (1 connection now open)
2016-09-01T01:34:23.863+0300 I NETWORK  [initandlisten] connection accepted from <IP_ADDRESS>:55970 #2 (2 connections now open)

Line 88 of lib/api.db.js, changing
if (database.alias) application.sandbox[database.alias] = database.connection;
to
if (database.alias) application.sandbox[database.alias] = database;
fixes the issue for MongoDB, don't know if it breaks other drivers though.

Yes, it looks like incorrect behavior. Using application.databases.databaseName instead of an alias should work though.

@aqrln how to progress?

Working on that right now, related: https://github.com/metarhia/Impress/issues/601


Best way to  take input for date in redux-form?

Hey,
I have a form in which have few fields which take date is inputs. Now, I tried searching a bit and I see that input type "date" is not supported by most of the browsers and hence cannot use that.
Now, I am using react-datetime in few other places in my projects. Is it possible to use this in the forms too? In the end I just need the date in my values object under form right?
If not can you please suggest a good way to deal with dates under a form?

You can use any date picker. You need just rewrite onChange.
onChange={value => field.input.onChange(value)}

This issue doesn't belong to our GitHub issue tracker.
We request you to ask again at StackOverflow, using the redux-form tag, then post the question link here.
As usual, don't forget to explain your question well and show what you've got so far!
This is because the GitHub issue tracker is better suited for discussions related to bugs and improvements of Redux Form.
Meanwhile, StackOverflow is better at keeping popular questions visible, while separating feature-related questions from help-related questions.
Thanks for using Redux Form!


add tools to automatically sort js imports

following https://github.com/portainer/portainer/pull/3645#issuecomment-611638336
The following tools are a possible solution:
https://github.com/EmmanuelDemey/eslint-plugin-angular
https://eslint.org/docs/rules/sort-imports

Can we simply use the eslint sort import rule here? @chiptus

the problem with sort-import is that it doesn't auto-fix the problems. This is also the case with the angular one.
This one will fix the order (not for angular):
https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/order.md


Load Balancer fails to start

Hello,
I'm trying to run the playbook in a home server, but I'm failing to run at because the load balancer never starts (or so the playbook says... more on this later)
Ansible fails with:
TASK [Check openshift-4-loadbalancer-don.service] *******************************************************************************************************************************************************************************************
Wednesday 01 September 2021  21:34:02 +0100 (0:00:00.921)       0:00:09.955 ***
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Load balancer service openshift-4-loadbalancer-don.service is not running!!! Please check, run: systemctl status openshift-4-loadbalancer-don.service"}

systemd returns:
● openshift-4-loadbalancer-don.service - OpenShift 4 LoadBalancer (openshift-4-loadbalancer-don)
   Loaded: loaded (/etc/systemd/system/openshift-4-loadbalancer-don.service; enabled; vendor preset: disabled)
   Active: activating (auto-restart) (Result: exit-code) since Wed 2021-09-01 21:38:37 BST; 12s ago
  Process: 53950 ExecStart=/usr/bin/podman run --name openshift-4-loadbalancer-don --net host --env-host=true quay.io/redhat-emea-ssa-team/openshift-4-loadbalancer:master (code=exited, status=1/FAILURE)
  Process: 53903 ExecStartPre=/usr/bin/podman pull quay.io/redhat-emea-ssa-team/openshift-4-loadbalancer:master (code=exited, status=0/SUCCESS)
  Process: 53857 ExecStartPre=/usr/bin/podman rm openshift-4-loadbalancer-don (code=exited, status=0/SUCCESS)
 Main PID: 53950 (code=exited, status=1/FAILURE)

However, through cockpit I can see the load balancer is actually running:

But there isn't anything showing the logs. If I force stop the container, then the systemd unit keeps cycling it.
Any ideas what could it be?
Thanks!

I found the issue. My home server has only one NIC, and I'm not using it as a router, therefore the public_ip variable stated in the Ansible Playbook was not its real IP, but the external IP from my ISP provider. Changing the value to the private IP of the server works, the LB stays running, and the cluster creates successfully. The only change needed in the router is port forwarding to the server/LB IP.
However, this affects the public_ip sent to the DNS providers. If people are interested, I will try to send a PR with changes to make different the IP of the LB and the public_ip var for DNS.

Of course, if would suggest:

Run the installation with your private ip address  + dns_provider: none
and write an add-on to add the public DNS entries + letsencrypt stuff if needed.


SubSink in React project using typescript

Hi, I can implements this in react projects or is only for angular projects?, I installed in my project of react and in the  compilation of webpack throw this warning
(Cannot find source file '../src/subsink.ts': Error: Can't resolve '../src/subsink.ts') and in my angular project not happen that ,thanks a lot

Hi there, in our project only appears this errors when we use the HMR (Hot module replacement).
Is related to sourcemaps.
@wardbell do you accept PR ?


Add all CodeCallbacks from server.dll

this was pain

I'd make this a new page and some also some code callbacks appear to not work idk


May I ask how to use this project to find the position of the screen where the eyes are focused

How to find the area on the computer screen that the eyes are looking at

How to find the area on the computer screen that the eyes are looking at

兄弟/姐妹，请问这个代码能显示眼睛看屏幕哪个位置吗


@JsonUnwrapped for SFM or map columns to inner object

I have a case where a stored procedure written by bad people returns 40 fields. I would like to map these fields to a more complex type. An example for the feautre I'm looking for :
If I get a ResultSet with fields : id, street, city, I would like the following to map correctly :
class Result {
   String id;
  @SfmUnwrapped
   Address address;
}

class Address {
   String street;
    String city;
}

I reference @JsonUnwrapped since this is how Jackson does it for JSON. If there is another way, please advice. I know I can just add an alias for each field, but that is a bit obtuse for the reader of the code.

There is a new feature that can enable by calling enableSpeculativePropertyLookupOnObject()
on the Mapper factory that will enable speculative lookup on object
https://simpleflatmapper.org/2019/11/14/v8.2.1.html
so the field street should in your case endup in address.street
could you try that on the last version 8.2.1?


Add indexed name auth provider

@ricardozanini, apparently the same provider can be added multiple times. This is the case for oauth because of the different flows. This way, we avoid conflict with the variable name. The attached open API spec reproduces this issue.


Add release notes for 0.4

Also update the contribution guide to include instructions for automatically synchronising release notes with the GitHub release page via chandler.

Closed by #226


Correction for raw affiliation Schulich School of Medicine and Dentistry, Western University, London, Ont

Correction needed for raw affiliation Schulich School of Medicine and Dentistry, Western University, London, Ont
raw_affiliation_name: Schulich School of Medicine and Dentistry, Western University, London, Ont
new_rors: 02grkyz14
previous_rors: 02agqkc58
works_examples: W4386156185;W4389828318;W4317722126
contact: 360fc43fadf4d17d1a27c27ffac30dcb:5cb6ad7193f6f0f1 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Bug - copy-pasting into fields when editing waypoint doesn't save changes

Couldn't come up with better title
Steps to replicate are simple:

Create waypoint
Open editor and try to paste in other name or cords or color - anything, really
If you do so, changes won't be registered

There is a way around - you just click on box and manually add space or just manually type anything into it and it will cause it to update and actually react to this
possible simple fix - when you click "Done" ("Готово") button it will check contents of fields and force them into waypoint

Good catch. There were apparently more issues with those TextWidgets but now in the next update hopefully they are fixed.


UB when getting robot for ID

UB can happen when the new Leaf::updateRobot() is called, since it sometimes returns an optional robot with nothing in it.

This looks like it creates an empty robotview... I guess it should return std::nullopt instead

Actually, std::optional's default constructor is equivalent to the std::nullopt_t constructor
https://en.cppreference.com/w/cpp/utility/optional/optional


Adjust Simple-Chart block wagtail layout

Additions

Made a new layout for the Simple-Chart wagtail block

Changes

Slight text change in one of the Simple-Chart fields

How to test this PR

Pull PR
yarn build -- ./runserver.sh
Go to http://localhost:8000/admin/pages/4481/edit/
Add Simple-Chart block and notice new layout

Screenshots

Resolves https://github.cfpb.gov/CFPB/super-regular/issues/383

I have some thoughts to share, some specific to the two-column layout approach, some are more general summaries of feedback we've gotten about the Wagtail admin that I'm including here because it seems like a good opportunity to make those changes.
As far as layout goes, I think the most useful thing to do is keep fields that are likely to be edited together as close together as possible. To a certain degree this is already true, but there are a few places where I think we can rearrange things to be slightly more logical. For example, my experience so far has been that axis labels are likely to change independently of data sources and structure, as a result of content clearance, so:
-Switch the position of "Transform" and "Y-axis label", so that the two axis label fields are next to each other.
The other thing that would help make editing simple is to match the order of the fields in the admin to the order they appear on the website, as much as possible. I don't think it makes sense to move the figure number field from where it is right now, since it is optional and I find that scanning for the chart title is the most important thing to look for when I'm editing pages with multiple charts, and it's about as prominent as it can be right now. But for the fields that appear below charts, the order and labeling should match what's live, and also how the information is arranged in content tables, to make it as simple as possible for content editors to copy and paste accurately.

"Credits" field should be renamed "Source credits" (to minimize possible confusion with the "data source" field)
Fields for information that appears below the chart should go in the order Source credits, Date published, Download text, Download file, Notes

This does require moving the notes fields which unbalances the columns a bit, but I think the tradeoff of page length is well worth it in terms of following the structure of how the content actually appears. It might also be nice to make the notes field full-width, for ease of reading longer note text.
The other things are just minor improvements to the helper text for some fields.

Change the helper text for "Data series" to: For charts pulling from a separate source file, include a list of the column headers (from a CSV file) or keys (from a JSON file) that match the data that needs to appear in the chart, in the format ["HEADER/KEY1", "HEADER/KEY2", "HEADER/KEY3"]. If the chart labels need to be different than the data source headers, include the correct labels with the format [{"key": "HEADER/KEY1", "label": "NEWLABEL1"}, {"key": "HEADER/KEY2", "label": "NEWLABEL2"}]
Change the helper text for "Filters" to: If the chart needs the option for users to filter the data shown, for example by date or geographic region, provide the JSON objects to filter on, in the format {key: "KEY", "label": "LABEL"}


Difference between bfgs.rs and bfgs of optimtool.unconstrain.newton_quasi

In compiled languages, using the method of state to prompt whether to initialize the parameter vector is a consideration for developers not to treat it as a dynamic tuning method, but to use it to adapt to compilation logic for embedded code?
https://github.com/argmin-rs/argmin/blob/a9e3eb7838e4a68994b452e840a04b7726cfa8e8/crates/argmin/src/solver/quasinewton/bfgs.rs#L157-L161
the above code may be same as the underlined code by given initial point:
gradient = np.array(res.subs(reps)).astype(DataType) # optimtool/unconstrain/newton_quasi.py#L64

https://github.com/argmin-rs/argmin/blob/a9e3eb7838e4a68994b452e840a04b7726cfa8e8/crates/argmin/src/solver/quasinewton/bfgs.rs#L200-L202
I have never occured such issue about not set in state, did function parameters make all the initial operations but stop the iteration because of not set in state?
https://github.com/argmin-rs/argmin/blob/a9e3eb7838e4a68994b452e840a04b7726cfa8e8/crates/argmin/src/solver/quasinewton/bfgs.rs#L236-L238
if I understand here, No parameters means bfgs.rs may stop iteration in intensive program because of line search, and when give a complicated function to adapt to an embedded program, a developer need many trials according to the exact problem which to process.

These are safety hatches in case a bug is introduced somewhere. Apart from the check in init, these should never lead to an error in normal operation. The one in init is a hint to the user in case they forget to provide a parameter vector.
This will likely become unnecessary when state handling is improved (#478).
Does this answer your question?

OK, that's really a considerable choice on handling future state.


Update sampling-event-data.adoc

Changed the description to be more simple and clear.

Thank you Cecilie, I'm merging


MB is a great guy once you get to know him

you might waste his time though
not deserve it
not appreciate it yet

Guy Sweet now says this.

Oh, so does Buddy Best. And the guardians.


All content not loading

I cloned the repo and followed the installation procedure. I have an apache server running on CentOS 7. When I go to /blockly/games/appengine/index.html I get the GUI to select the app. If I select an app from there (say Maze) I just get the directory contents of of /blockly/games/appengine/maze/. If I go to /blockly/games/appengine/maze.html, the app partially loads, but not the actual progress window:
https://www.dropbox.com/s/tyxith0poiwzhsr/Screenshot 2015-06-23 08.34.17.png?dl=0
I don't get any errors on the console or in the web logs (other than missing /common/storage.js). This feels like a web server configuration I'm missing. Any help? I'm trying to get this configured for a programming class that starts Monday where I won't have external network so I need to get it working self-contained. Thanks!

Does the offline download package work for you?
https://github.com/google/blockly-games/wiki/Offline

The offline version does indeed work! Thanks. What's the difference between the offline and the source compiled version?

The master branch of Blockly Games (old) is not compatible with the master branch of Blockly (new) at the moment.  This should be resolved soon.

I am actually getting a similar problem with the JavaScript not working properly and not showing/reacting fully. The offline version works okay but the build fails.

I want to do some changes on Maze, Also tried to use the build version but I have the same problem, also I did some changes on the maze.js with window.alert (make command was ok but the alert didnt works). I need to change the maze exercise and get the number of tries. Can I do that with the Offline version? Thank you, nice work

Its works! I used this pull
https://github.com/google/blockly-games/pull/16
The solution:
https://github.com/arjenpdevries/blockly-games/commit/0c9d6f954db2c1f9d803c43dd2d6be0bc107a8c5
Now I can build my changes


Workflow shows as failed, even if the latest workflow attempt of the job is successfully

Describe the bug
If a workflow job initially fails but a rerun of the same workflow job is successful, the workflow shows the state of the first attempt
To Reproduce
Steps to reproduce the behavior:

Run a workflow which should fail (due to wrong credentials).
gitactionboard shows the workflow as failed (correct), reflecting the state of the latest job.
resolve the reason why the workflow failed. (add correct credentials into github action secrets)
rerun the workflow job (attempt #2)
the attempt succeeds
gitactionboard shows the workflow still as failed (incorrect)

Expected behavior
gitactionboard should show the status of the latest attempt of a workflow job

Note the CCtray shows the evidence of the this issue as well
{
    "name": "fake :: Build and Deploy :: release / release-to-dev",
    "activity": "Sleeping",
    "lastBuildStatus": "Failure",
    "lastBuildLabel": "23",
    "lastBuildTime": "2023-10-25T11:31:11.000Z",
    "webUrl": "https://github.com/fake/fake-api/actions/runs/66402497/job/182159058",
    "triggered event": "repository_dispatch"
},

the lastBuildTime is the datetime of the 1st attempt, not the 2nd one.

Backgrounder https://docs.github.com/en/actions/managing-workflow-runs/re-running-workflows-and-jobs

Hi @globaltopmedia, could you please check your cache config? I just tried to reproduce it and it's working fine.
Until cache expires, it'll not fetch latest details from GitHub.


(Fix) Refactor setCrowdsaleInfo and remove lastCrowdsale from component state

Closes #551

Wrong review: try from another PR. I'll test again and let you know the results.

Ok. No such bug. But this is another one:

@vbaranov thanks for the catch, fixed


Add pack-missing-tests script to make it easy to spot missing pack tests.

Add pack-missing-tests script
Status
Functional - Request for comment
Description
Adds a script that makes it easy to spot missing or extra tests for packs that have changed..
Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)

Nice :+1:
Can you paste an example output of this script?

And since it appears to be informational only at this point, we should also update Travis config so it also runs on Travis.

I've cleaned up the output a little, here's some examples:
====== packs/alertlogic ======

Actions missing tests:
        scan_get_results_by_title.py
        scan_get_results.py
        scan_list_scan_executions.py
        scan_list_scans.py

Tests with no actions:

Coverage:
        Actions: 0/4

====== packs/orion/ ======

Actions missing tests:
        node_create.py
        node_discover_and_add_interfaces.py
        node_remanage.py
        node_unmanage.py

Tests with no actions:
        test_action_aliases.py

Coverage:
        Actions: 3/6

====== packs/duo ======

Actions missing tests:
        auth_auth.py
        auth_check.py
        auth_ping.py

Tests with no actions:

Coverage:
        Actions: 0/3

====== packs/cloudflare ======

Actions missing tests:

Tests with no actions:

Coverage:
        Actions: 1/1

The new output format can be extended to cover sensors too, if the file name of alias tests follows the manner as actions and sensors it can report on that too :-D

I'm hoping this will be useful for others, it started out a quick hack to spot test coverage holes in the Orion pack PR I'm currently working now.

@jjm Great, thanks.
Eventually, as @tonybaloney has pointed out in the past, it would be great if we can move all those scripts in some common place in st2 repository so users outside st2contrib can utilize them as well, but for now I'm fine with those scripts only living in this repository.

I'll merge this into master and hook it up to Travis inside master.
Thanks again.

No worries, automatic reporting of missing tests will be rather nice.
Some way of supporting scripts for testing / supporting packs outside of st2contrib would be very nice. I need to set up CI on our internal pack and a standard way of doing it would be great.

Made some improvements to the script (c5214c10b7638c7b2902609ac7fe1889337f7832) and enabled Travis check (f33d761d8bb76be7c88ee64ff48162343710ff08).

@Kami fwiw, SaltStack has done a reasonable job at this by creating a separate package on pypi for testing.

@tonybaloney Yeah, we created st2sdk in the past (https://github.com/stackstorm/st2sdk) which we planned to release as a separate package on PyPi.
We could potentially add those scripts and tools there or create a separate package (I can't really think of a good name for it from top of my head).


Change navbar message to reflect that we are using Crystal.

This branch is not set up to use Heroku.


docs: suggest snippets from the VSCode Marketplace

Hi, I'd like to propose a tiny addition to the docs if you're accepting.
I think this would be helpful to folks migrating over to Volar from Vetur who, like myself, don't really care about customizing their snippets, but just want some quick, ready-made snippets they can install with a click. Sarah Drasner's snippets are relatively popular, with Vue 3 support, and she's a member of Vue's core team so I suspect they will be well maintained. I read through #183 and #781 and appreciate the rationale not to include built-in snippets, but this seems like an easy alternative.
Happy to amend or close if you're not accepting PR's of this sort. Thanks!

LGTM, thanks for the update!

@jgaehring  Looks fine, but the templates are generated in this order.
<template>

<script>

<style>

And according to vue3, we should be sticking with the following order.
<script>

<template>

<style>


Update stable version to TiSpark 2.1

After this update, we should: update all documents and scripts that rely on latest TiSpark stable version or master version. Including docs&docs-cn/ansible/docker deployment/etc.,.....

/run-all-tests


SAK-49595 T&Q: Import/Export assessments with parts based on random questions

This has to be merged after #12210 and #12213

Thanks for the review!


bugdown: Autolinking websites does not work if prepended by multibyte characters.

OS: Ubuntu 16.04
Zulip: 1.7.1
autolink does not work that I expected like following example:
日本語http://example.com/

This URL with preceding multibyte characters.
expected result:
日本語http://example.com/

I found large regular expression as below:
https://github.com/zulip/zulip/blob/83f563aeff2358ba159b2349ab39382450e376a9/zerver/lib/bugdown/__init__.py#L1401-L1422
And I've created small patch for this issue:
$ git diff master
diff --git a/zerver/lib/bugdown/__init__.py b/zerver/lib/bugdown/__init__.py
index b1434c724..eb905fae8 100644
--- a/zerver/lib/bugdown/__init__.py
+++ b/zerver/lib/bugdown/__init__.py
@@ -1421,7 +1421,7 @@ class Bugdown(markdown.Extension):
         nested_paren_chunk = nested_paren_chunk % (inner_paren_contents,)
         tlds = '|'.join(list_of_tlds())
         link_regex = r"""
-            (?<![^\s'"\(,:<])    # Start after whitespace or specified chars
+            (?<![^\s\w'"\(,:;<])    # Start after whitespace or specified chars
                                  # (Double-negative lookbehind to allow start-of-string)
             (?P<url>             # Main group
                 (?:(?:           # Domain part
diff --git a/zerver/tests/test_bugdown.py b/zerver/tests/test_bugdown.py
index e8fab4c0c..b05c7bc33 100644
--- a/zerver/tests/test_bugdown.py
+++ b/zerver/tests/test_bugdown.py
@@ -1104,6 +1104,17 @@ class BugdownTest(ZulipTestCase):
             '
http://example.com/#settings/
'
         )

+    def test_normal_link_start_with_non_ascii_character(self) -> None:
+        realm = get_realm("zulip")
+        sender_user_profile = self.example_user('othello')
+        message = Message(sender=sender_user_profile, sending_client=get_client("test"))
+        msg = "あhttp://example.com/#settings/"
+
+        self.assertEqual(
+            bugdown.convert(msg, message_realm=realm, message=message),
+            '
あhttp://example.com/#settings/
'
+        )
+
     def test_relative_link(self) -> None:
         realm = get_realm("zulip")
         sender_user_profile = self.example_user('othello')

But this change will break a test like following:
FAIL: test_bugdown_fixtures (zerver.tests.test_bugdown.BugdownTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/python/3.4.6/lib/python3.4/unittest/case.py", line 58, in testPartExecutor
    yield
  File "/opt/python/3.4.6/lib/python3.4/unittest/case.py", line 580, in run
    testMethod()
  File "/home/travis/build/okkez/zulip/zerver/tests/test_bugdown.py", line 233, in test_bugdown_fixtures
    self.assertEqual(converted, test['expected_output'])
  File "/opt/python/3.4.6/lib/python3.4/unittest/case.py", line 800, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/opt/python/3.4.6/lib/python3.4/unittest/case.py", line 1173, in assertMultiLineEqual
    self.fail(self._formatMessage(msg, standardMsg))
  File "/opt/python/3.4.6/lib/python3.4/unittest/case.py", line 645, in fail
    raise self.failureException(msg)
AssertionError: '
&[17 chars]p://x<a href="http://x.com/xss.js<b" target[68 chars]
' != '
&[17 chars]p://xx.com/xss.js<b>
'
-
<script src=http://xx.com/xss.js<b>

+
<script src=http://xx.com/xss.js<b>

What should I do?
Thanks.

Thanks for the report @okkez!
We generally don't auto-link unless the http is preceded by a whitespace (or beginning of message). Is the issue that in Japanese or other languages URLs are typically written without a preceding space?

@rishig Thank you for replying.
In general, we Japanese people don't put preceding space before URLs.
Some web services/applications create auto-link for URLs with preceding multi-byte characters.
For example,

twitter
github like following

日本語https://example.com

gmail
redmine
slack
gitter.im
and more...

Great, thanks for confirming. I've added the appropriate label; hopefully someone will take it up soon.
To anyone working on this:

Note the patch above
Note that this likely does already work in the js markdown processor, and just needs to be fixed in the python markdown processor. But that should be verified.

Thanks again @okkez for the careful report!

@zulipbot claim

Yeah it works with slack. But one thing i noticed with slack is that, it works the same way even with ascii characters. Have a look.

Do we want the same?

After a bit of preliminary research, it seems that nearly all apps allow for multibyte characters before a fully qualified url, even if they're European characters, such as in ühttps://www.example.com.  Whether characters are allowed before something like www.example.com depends on the app.  The currently proposed change above breaks a lot of tests, so instead I propose that we do the minimal possible change that resolves the issue, namely allowing multibyte characters before a url.
The first multibyte code point is\u0080, and the last relevant one is \uFFEF (\uFFF0-\uFFFF are miscellaneous symbols).  Thus, to https://github.com/zulip/zulip/blob/32ede6a3552d20a638107d7a94221dc085e0d9e8/zerver/lib/bugdown/init.py#L1610
We could change the regex to
(?<![^\s'"\(,:<\u0080-\uFFEF])
I looked at the above contributor's, @dgarvit, change, and it seems that if this was the change instead, it would break no test cases, and simply require a new test case to be written to ensure that no future regressions happen (which was already done above).  Allowing multi-byte characters before an autolink seems to be quite universal, and should not create any unexpected or controversial behavior while still fully fixing the issue.  There's already so many ways to create a link (<www.example.com> is one easy example), that I don't think it's necessary to go overboard and allow single-byte unicode characters like Z to precede links as well.
If we are going to allow arbitrary characters before links, as the contribution above seems to intend to, it is also fine to just completely remove that one line of the regular expression, since we are no longer doing lookbehinds for the proper character to precede a link.

I think linkifying Zhttps://google.com is particularly nice for mobile or other fast copy-and-paste situations. I also think there are not any situations where someone submitted Zhttps://google.com and didn't intend https://google.com to be a link.
For the Zwww.google.com case, I agree we shouldn't bother separating out the Z from the www part if it is a lot of work.

That's a good point.  I was thinking of there being separate logic depending on whether we specify the protocol (e.g. https://) or not, but that would be complicated.  Now that I think about it a bit more, it does seem kind of difficult to type it but not intend to create a link.  I do wonder, though, why we were demanding that links start after either whitespace or one of the special characters in the code snippet above in the first place.  Putting a letter in front of a link so it doesn't linkify does not seem like something anyone would do.
By the way, zwww.google.com could conceivably be correct; it could be a subdomain of google.com if the site owners wished it to be.  I think it'd be really unreasonable to try and divine whether the user meant a subdomain zwww of google.com or meant to type www, it wouldn't even be an issue of the implementation being difficult.

why we were demanding that links start after either whitespace or one of the special characters in the code snippet above in the first place

I assume someone just copied the habit from how we do other things in our markdown. E.g. I'm guessing Z*this is not italic* in markdown.

I do wonder, though, why we were demanding that links start after either whitespace or one of the special characters in the code snippet above in the first place.

I suspect this is because we want the regular expression to capture the entire part of the string that might be a URL.  Perhaps the correct behavior would be to have a whitelist rather than a blacklist (i.e. letters/numbers/-/etc.) for what characters can appear at the start of the URL, rather than the other way around.  But if that doesn't make sense for some reason, I think adding multi-byte characters to that list seems correct.

By the way, zwww.google.com could conceivably be correct; it could be a subdomain of google.com if the site owners wished it to be. I think it'd be really unreasonable to try and divine whether the user meant a subdomain zwww of google.com or meant to type www, it wouldn't even be an issue of the implementation being difficult.

Agreed; aside from perhaps whitelisting "www" specifically, I don't think there's anything we can do to distinguish whether the "Z" is part of the domain name or not (in other words, for all the software knows, foo.com is just as valid a domain as zfoo.com).  I think we can only ignore leading single-byte characters if we have a clear protocol (e.g. Zhttps://google.com though I could imagine hardcoding www, under the theory that zwww type beginnings of URLs are very rare in comparison to www).  Which would primarily come up in the context of copy-paste on mobile.
@PentaHiggs is that enough info to answer the questions about what behavior should be correct?
(Thanks for all your thoughtful work on this so far!)


upgrade to spring 2.0

Codecov Report

Merging #40 into master will decrease coverage by 0.01%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   77.34%   77.32%   -0.02%
==========================================
  Files          26       26
  Lines         640      644       +4
  Branches       87       87
==========================================
+ Hits          495      498       +3
- Misses         98       99       +1
  Partials       47       47

Impacted Files
Coverage Δ

...epository/support/DocumentDbEntityInformation.java
76.08% <ø> (ø)
:arrow_up:

...ig/DocumentDbRepositoryConfigurationExtension.java
53.33% <ø> (-2.92%)
:arrow_down:

...umentdb/config/DocumentDbConfigurationSupport.java
62.96% <0%> (-2.43%)
:arrow_down:

...ore/mapping/BasicDocumentDbPersistentProperty.java
83.33% <100%> (ø)
:arrow_up:

...umentdb/core/mapping/DocumentDbMappingContext.java
100% <100%> (ø)
:arrow_up:

...entdb/core/convert/MappingDocumentDbConverter.java
68% <100%> (+2.04%)
:arrow_up:

...epository/support/DocumentDbRepositoryFactory.java
100% <100%> (ø)
:arrow_up:

...repository/support/SimpleDocumentDbRepository.java
91.37% <83.33%> (+0.15%)
:arrow_up:

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 b72cf3f...666c6f9. Read the comment docs.


Add emojis and spinners to console.logs as it's creating the files

https://github.com/sindresorhus/cli-spinners
https://github.com/sindresorhus/log-update
^These should take care of that for us

The module is so ⚡ fast that you don't even need a spinner


Hi, Are you still working with MOAI SDK

Hi @PeqNP
So sorry but I can't make the new issue at folked Moai repository. Are you still working with MOAI.
I want use MOAI for new Puzzle game project and have some problem
If you have time. Please give me a chance to ask some questions.
Thanks

I'm currently independent of MOAI. I do use MOAI though. Have you tried the
forums? They are surprisingly responsive. After you make a post, send me a
link to your post and I'll see if I can help. I'm still relatively new, so
no promises, but I'll at least take a look :)
Regards,
Eric
On Wed, Dec 21, 2016 at 9:14 PM, John Pham<EMAIL_ADDRESS>wrote:

Hi @PeqNP https://github.com/PeqNP
So sorry but I can't make the new issue at folked Moai repository. Are you
still working with MOAI.
I want use MOAI for new Puzzle game project and have some problem
If you have time. Please give me a chance to ask some questions.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PeqNP/GameKit/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABaemgIbf6V3OuO6o4d5W5yWE33ZvaDUks5rKgdJgaJpZM4LTpa-
.

💯 Thanks you @PeqNP


Cannot demangle a handful of symbols from a Linux build of Firefox

I've used the current master branch of cpp_demangle on the symbols produced by a local Firefox build on my machine and found 78 that cannot be demangled. The list of affected symbols is available here. These were generated by clang 14.0.5. As always with this kind of issues it's unclear if it's cpp_demangle's fault or the symbols were mis-mangled by clang.

I've fed the symbols into llvm-cxxfilt and the results are here. These symbols seem to share one of the two following conditions:

They contain parameters that are function pointers qualified as noexcept
They contain a at offset <number> component which I'm not familiar with.

Here's a couple of examples:
bool __gnu_cxx::__ops::_Iter_negate<int (*)(int) noexcept>::operator()<char const*>(char const*)
nsresult mozilla::extensions::AtomSet::Get<&(mozilla::extensions::WILDCARD_SCHEMES.<char const* at offset 0>)>(RefPtr<mozilla::extensions::AtomSet>&)

I did noexcept (the non-deprecated variants anyways).
at offset <number> hasn't actually made it into the spec yet: itanium-cxx-abi/cxx-abi#47


Instance type cx22 is not available (currently or anymore)

Packer fails when trying to build images since it's referencing instance type that is not available. I saw some updated from hetzner about instance types, but sort of missed the point, whether it was no longer available, or jsut currently not available.
I think it would be nice to have it dynamically pick available instance type with min resources that are needed to build the images, instead of having a hardcoded value.
https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/e171efcc9f3f2dd9f4f283fdf64b0793e4aaa259/packer-template/hcloud-microos-snapshots.pkr.hcl#L83

Nope packer is not failing on anything I've just rebuild the opensuse images using packer following readme guide and everything was just fine.
mkdir /path/to/your/new/folder
cd /path/to/your/new/folder
curl -sL https://raw.githubusercontent.com/kube-hetzner/terraform-hcloud-kube-hetzner/master/kube.tf.example -o kube.tf
curl -sL https://raw.githubusercontent.com/kube-hetzner/terraform-hcloud-kube-hetzner/master/packer-template/hcloud-microos-snapshots.pkr.hcl -o hcloud-microos-snapshots.pkr.hcl
export HCLOUD_TOKEN="your_hcloud_token"
packer init hcloud-microos-snapshots.pkr.hcl
packer build hcloud-microos-snapshots.pkr.hcl
hcloud context create <project-name>

Without providing more details from your side like screenshots of commands and so on I propose that issue to be closed

Thanks for confirming @ricristian. @virtuman It might have been a temporary issue on hetzner's side. Please try again with the method exposed above. Closing this for now.


[Enhancement] Use Bytes instead of Vec for Block to avoid copy.

Issue summary
Currently we are using a Vec<u8> for Block, because it's convenient given the signature of Blockstore::get.
However, there's a good performance boost to be found if we switch to Bytes instead, thus getting rid of unnecessary
copying of said bytes into a Vec.
Code in question: https://github.com/ChainSafe/forest/blob/10896c2c9f03f895e88cf6110d68a8ac38972df8/src/utils/db/car_stream.rs#L52-L55C1
Task summary

[ ]  convert Vec<u8> to Bytes in Block
[ ]  make sure that the above change plays well with the rest of the codebase
[ ]  run cargo forest-tool benchmark car-streaming benchmark to verify the performance boost with and without inspect arg

Acceptance Criteria

[ ] data field in Block struct is converted from Vec<u8> to Bytes to avoid copy
[ ] performance boost is validated by benchmarking car streaming
Other information and links

We need to figure out what we'll do with Blockstore::get implementation for car-backed stores as well as the cache that is also using Vec<u8>.

related to #3314

Note to discuss with @lemmih when back from PTO, based on this thread: https://chainsafeio.slack.com/archives/C0144LHN732/p1692632131928259

Yeah, so this is a quick and dirty solution to have an idea: https://github.com/ChainSafe/forest/pull/3366
In order for this to be interesting it would indeed be nice to employ Bytes when dealing with car-backed stores, for example. Currently not doable due to a variety of constraints, including Blockchain::get signature.

The code in #3366 does not make graph traversal faster. We must be all on the same page about this. It is called overfitting when you optimize a benchmark in ways that do not make Forest faster. It's easy to do this accidentally, and one must always be mindful of it.

I don't see how to avoid the copy. Closing.


Command string-return sending

Description of the PR
Most commands send a message at the end. This PR allows you to return a string (or a string[]) from a command to send it to the relevant channel (or edit the previous message in that channel, if cmdEditing is turned on). This works with subcommands, as well.
I changed a few core commands to take advantage of this new feature. However, it may be better to leave the core commands using msg.sendMessage explicitly, so they'd still work if the user disables command string-return sending.
Originally, I modified commandHandler.js directly, but then I decided that making it a finalizer might be better. Users can then easily disable this feature by transferring the finalizer and setting enabled: false.
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

A new finalizer, commandSendReturn.js, which sends the return value from commands, if it's a string or array of strings
(Possibly) a rewrite of the core commands to take advantage of this feature

Semver Classification

[ ] This PR only includes documentation or non-code changes.
[ ] This PR fixes a bug and does not change the (intended) framework interface.
[x] This PR adds methods or properties to the framework interface.
[ ] This PR removes or renames methods or properties in the framework interface.

The idea of this PR is nice, provides consistency with throw (which makes the command exit as a non-successful run and reply the user with whatever the reason is). However, it's quite limited in terms of options, and expanding them may be more expensive.

I'll see if I can expand it, without making it too involved to run on every command.

This breaks the consistency that Messages should be returned from Commands; Something that Finalizers and the commandSuccess events inherently expect to get. If this is even used once, this prevents any finalizer from it's previous capability of depending on the sent message.
That's before considering that this only works for content, and only creates extra overhead for commands that you need to send that isn't exclusively content.
Not to mention this is going too far with hand holding.

Can this be reopened please? I've changed things, and I think you'll be more pleased with it. It addresses the expected Message return, provides more flexibility to have more than just string content, removes all overhead for the check other than a single object property access, and is opt-in, so users can't rely on hand-holding too much.


Fix link

Link to Xilinx device plugin was incorrect.

Hi @PhilRW
If you are willing to sign the CLA, and can do that (online), we can review this pull request.

Hi there,
I'm doing this work through my work, and it seems we are a signatory. I'm wondering how to have my GitHub account recognized.

Does https://github.com/kubernetes/community/blob/master/CLA.md give you enough of a clue @PhilRW? You might need to adjust your affiliation via cncf/gitdm.

...Still figuring out why my employer says we're on the list but gitdm does not....

:wave: @PhilRW please sign the CLA. It's possible the commit has a different email address than you registered with. You can check with git --no-pager show -s --format='%an <%ae>' COMMIT HASH
Once reviewed you may need to change author info and force push.

My company did sign the CLA but we're not in gitdm yet.

/check-cla

@PhilRW sorry that the CLA is causing grief. Given the length this PR has gone unresolved I'm going to close it. Feel free to reopen it once the CLA is signed or reach out if we can help! Thanks!
/close