Allow multiple istio.test.skips

Otherwise its impossible to skip subtests sanely.
Instead of --skip'TestFoo|TestBar' we can do --skip 'TestFoo' --skip 'TestBar/subtest' and not skip the entire TestBar.
This is not possible with the current regex match
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

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

Test name
Commit
Details
Rerun command

unit-tests_istio
c7374e347ffb7d7c9941aad94730ded64acb0e55
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
e35159272b6dbfb24914d5d8ca02711f1dfffbaf
link
/test integ-security-multicluster-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
e35159272b6dbfb24914d5d8ca02711f1dfffbaf
link
/test integ-security-multicluster-tests_istio

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

/retest

In response to a cherrypick label: #30842 failed to apply on top of branch "release-1.9":
Applying: Allow multiple istio.test.skips
Using index info to reconstruct a base tree...
M	pkg/test/framework/resource/flags.go
A	pkg/test/framework/resource/matcher.go
A	pkg/test/framework/resource/matcher_test.go
M	pkg/test/framework/resource/settings.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/test/framework/resource/settings.go
CONFLICT (content): Merge conflict in pkg/test/framework/resource/settings.go
CONFLICT (modify/delete): pkg/test/framework/resource/matcher_test.go deleted in HEAD and modified in Allow multiple istio.test.skips. Version Allow multiple istio.test.skips of pkg/test/framework/resource/matcher_test.go left in tree.
CONFLICT (modify/delete): pkg/test/framework/resource/matcher.go deleted in HEAD and modified in Allow multiple istio.test.skips. Version Allow multiple istio.test.skips of pkg/test/framework/resource/matcher.go left in tree.
Auto-merging pkg/test/framework/resource/flags.go
CONFLICT (content): Merge conflict in pkg/test/framework/resource/flags.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Allow multiple istio.test.skips
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #31070


BULK VALIDATION missing descriptions 6

Docs Build status updates of commit ede16bb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/UCMA/sending-and-receiving-page-mode-messages.md
:bulb:Suggestion

Details

Skype/UCMA/receiving-page-mode-messages.md
:white_check_mark:Succeeded

Skype/UCMA/recorder-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/recorder.md
:white_check_mark:Succeeded

Skype/UCMA/reference-applications.md
:white_check_mark:Succeeded

Skype/UCMA/reference-topologies.md
:white_check_mark:Succeeded

Skype/UCMA/reference-topology-with-high-availability-and-a-single-data-center.md
:white_check_mark:Succeeded

Skype/UCMA/reference-topology-with-high-availability-and-multiple-data-center.md
:white_check_mark:Succeeded

Skype/UCMA/reference-topology-with-limited-high-availability.md
:white_check_mark:Succeeded

Skype/UCMA/registering-for-call-events.md
:white_check_mark:Succeeded

Skype/UCMA/registering-for-conversation-events.md
:white_check_mark:Succeeded

Skype/UCMA/remotepresenceview.md
:white_check_mark:Succeeded

Skype/UCMA/security-considerations-in-ucma-5-0.md
:white_check_mark:Succeeded

Skype/UCMA/sending-page-mode-messages-asynchronously.md
:white_check_mark:Succeeded

Skype/UCMA/sending-page-mode-messages-synchronously.md
:white_check_mark:Succeeded

Skype/UCMA/server-platforms.md
:white_check_mark:Succeeded

Skype/UCMA/session-dialog-resiliency.md
:white_check_mark:Succeeded

Skype/UCMA/setting-up-the-development-environment.md
:white_check_mark:Succeeded

Skype/UCMA/shutdown-and-termination.md
:white_check_mark:Succeeded

Skype/UCMA/signaling-layer.md
:white_check_mark:Succeeded

Skype/UCMA/signalingsession.md
:white_check_mark:Succeeded

Skype/UCMA/sip-uris.md
:white_check_mark:Succeeded

Skype/UCMA/speechrecognitionconnector-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/speechrecognitionconnector.md
:white_check_mark:Succeeded

Skype/UCMA/speechsynthesisconnector-quickstart.md
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.

Skype/UCMA/sending-and-receiving-page-mode-messages.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@lindalu-MSFT
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!


How to change the size of the feature vector (currently 16)

I am trying to modify the size of the feature vector.
By default, it is 16. I am trying to use a feature vector of 3 for example.
I modified the file "gaussian-grouping/scene/gaussian_model.py", by changing line 52:
self.num_objects = 3
But, when running the code I get this error:
RuntimeError: Given groups=1, weight of size [150, 3, 1, 1], expected input[1, 16, 479, 639] to have 3 channels, but got 16 channels instead
What other part of the code should I modify in order to use a feature vector of 3?

Hi, thanks for watching our work!
For changing dim of the feature vector, you can refer to this issue.


Updates needed for iPython notebook

Issue #662 and PR #746 give an iPython script to demonstrate the API. Some comments in this PR thread were not resolved, however.

I'll take a look at doing these updates.

This is done. @jeromekelleher, you can close.


Merge pull request #1 from ParsePlatform/master

Update

Current coverage is 91.25%

Merging #1242 into master will decrease coverage by -1.77% as of 86dd536

@@            master   #1242   diff @@
======================================
  Files           83      92     +9
  Stmts         5287    5800   +513
  Branches       965    1056    +91
  Methods          0       0
======================================
+ Hit           4918    5293   +375
- Partial         10      11     +1
- Missed         359     496   +137

Review entire Coverage Diff as of 86dd536

Powered by Codecov. Updated on successful CI builds.


Handle input in multiple data formats

Currently offline data analysis accepts NumPy arrays and Pandas dataframes as input. We can leverage the MNE package to accept a variety of data formats. See https://mne.tools/dev/generated/mne.io.read_raw.html

Great point @toni-neurosc! We could add a separate function to
https://github.com/neuromodulation/py_neuromodulation/blob/fd2b112f9647e7160cbae8f8b2e1b09f7f627018/py_neuromodulation/nm_IO.py#L49
and maybe call it read_raw_mne and add it to nm_IO. Additionally it would be great to add a test maybe test_mne_raw_io that utilizes the brainvision file py_neuromodulation/data/sub-testsub/ses-EphysMedOff/ieeg/sub-testsub_ses-EphysMedOff_task-gripforce_run-0_ieeg.vhdr.

Fixed by #329


Added get_open_scenes_roots API to Editor Interface to retrieve all opened scenes root nodes

Added a single API get_open_scenes_roots  to EditorInterface to retrieve all opened scenes root nodes instead of scene file paths.

Changes applied to editor_interface.h and editor_interface.cpp
Documentation added to EditorInterface.xml

Thanks for contribution. Please squash the commits as it's required by our pipeline (see https://docs.godotengine.org/en/latest/contributing/workflow/pr_workflow.html).

@Chaosus Done, I'm not sure if I did it correctly, This was my first time :D Let me know if ther's still a problem.

Can we get this change merged? :)


Windows Support

Really excited to try this

Awesome! I now wonder how you found this project :)

Well, I started following you after seeing Porcupine, and here I am :)

If you are stuck on Windows, you can use WSL for now. It is basically the Linux terminal but inside Windows.

Gonna try that, thanks!

I tried one of your examples:

And this occurred:

First screenshot: vscode uses pylance, because it is convinced that test.jou is a Python file. Maybe there is a way to disable pylance for .jou files, but leave it enabled for actual Python files?
Second screenshot: You need to run test.jou with the Jou interpreter ./jou. For example, ./jou test.jou. The commands to compile it are in the README. You are trying to run it with bash (maybe with . test.jou or ./test.jou or bash test.jou).

First screenshot: vscode uses pylance, because it is convinced that test.jou is a Python file. Maybe there is a way to disable pylance for .jou files, but leave it enabled for actual Python files?
Second screenshot: You need to run test.jou with the Jou interpreter ./jou. For example, ./jou test.jou. The commands to compile it are in the README. You are trying to run it with bash (maybe with . test.jou or ./test.jou or bash test.jou).

Okay, thanks so much!

Let me know if you need more help or if you manage to run some Jou code :)

What are the problems with supporting Jou on windows? C and H are cross-platform right?

The biggest issue is getting a C compiler installed. Installing a C compiler on Windows is possible, but last time I tried, I spent a few days on it and gave up. Maybe you'll have better luck.
That's the hard part. Everything below is details that I'm sure we will figure out if we get a C compiler to work on Windows.
To invoke clang, there is currently a Makefile, a configuration file that contains commands that the make program runs. My impression is that Windows doesn't really have anything similar to make, and many projects instead use a huge and annoying framework like cmake (not to be confused with make), or just do whatever their IDE likes to do (e.g. cpython has a bunch of visual studio project files).
The Jou compiler also uses a few functions that aren't available on Windows, such as dirname(), but making it cross-platform should be easy.
Jou's tests are written as a bash script. I could probably rewrite the test script in C.
Jou's tests use valgrind, a tool that doesn't work on Windows. However, it is already possible to run the tests without valgrind, so this shouldn't be a problem.

I actually managed to get a C compiler to work on Windows. Maybe there will be Windows support within the next few days/weeks :)

I managed to run some Jou programs on windows, without using WSL. My work so far is in #103. Tomorrow I will rewrite my notes into some instructions you can follow :)

(closed accidentally)


Server Crash on Client Join (1.12)

Hey just a crash to report. I have a whole bunch of mods installed so it might be something from another one of them but from logs it looked like it was from tickdynamic. Let me know if there is anything else I can provide to help with debugging!
Forge: <IP_ADDRESS>3
Crash Report:
https://pastebin.com/ZKFDRZyD

Does this happen every time a client joins? And have you tried with just TickDynamic present to see if the issue still occurs?

Closing due to no response


How can I report a potential security vulnerability?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

To whom it may concern,
I have found a potential security issue in Youtube-DL. I'm wondering what the best way for me to report it is?
Kind regards,
Gamer191

Related: https://github.com/ytdl-org/youtube-dl/issues/30016

If you were to clone the repo and exercise the git log function, a suitable email address might be revealed.
Having said that, I am sceptical that any yt-dl bug would present a sufficiently serious vulnerability, in the absence of an underlying OS or Python vulnerability, to merit private disclosure.

Whoops, I forgot about this, I will send an email now

Having said that, I am sceptical that any yt-dl bug would present a sufficiently serious vulnerability, in the absence of an underlying OS or Python vulnerability, to merit private disclosure.

It almost certainly doesn't merit private disclosure, but I'm not a security expert, so I'm not going to make that call

@dirkf Could you please confirm that the email to report potential security issues is fieldhouse<>gmx.net?


Include g2pW into PaddleSpeech TTS

Thanks for the excellent work, @BarryKCL has included g2pW into PaddleSpeech , which greatly improves the accuracy of polyphone.
PaddleSpeech is an open-source toolkit on PaddlePaddle platform for a variety of critical tasks in speech and audio, with the state-of-art and influential models.
@BarryKCL converted the torch model to onnx and replace dependence transformers with paddlenlp.
Thank you again from the bottom of my heart ！😘
check:

pr: https://github.com/PaddlePaddle/PaddleSpeech/pull/2230
issue: https://github.com/PaddlePaddle/PaddleSpeech/issues/2196

@yt605155624 @BarryKCL
Thanks for your effort. Great job!
Could you need some help?
@BarryKCL
Great work of converting torch model to onnx.
Could I invite you to give a PR which replaces models with onnx to speed up?

@GitYCC not yet, but after merging and using, maybe we will find some problems, there is some little bugs in g2pw,  there are some words didn't in your 简体->繁体 dict (sorry I don't really understang taiwan or minnan lang), you can try to input this word "概念"，in @BarryKCL 's script, he added a "try catch" to avoid this bug, and use g2pM as backup, please check https://github.com/PaddlePaddle/PaddleSpeech/blob/aecf8fd3844371abcce5d337fab83aae6807285b/paddlespeech/t2s/frontend/zh_frontend.py#L186

@yt605155624
The root cause of this problem is because our model is trained on the Traditional Chinese (繁体) dataset. So, if we want to apply on the cases of 简体, I need to use package (OpenCC)[https://github.com/BYVoid/OpenCC] to convert them. But OpenCC still has some cases that can not be converted very well. Maybe we could seek a better method to do this conversion.

We have also tried opencc for 繁体 -> 简体 in PaddleSpeech TTS, but cause opencc has some bug when install in windows (not sure if this is still a bug now), we remove opencc and look up table now, you can check https://github.com/PaddlePaddle/PaddleSpeech/blob/0eb598b876f99bd26fa735577da92d46c45dc3fd/paddlespeech/t2s/frontend/zh_normalization/text_normlization.py#L81 , but I'm not sure if it works well for taiwan/minnan 繁体

The problem still exists. In order to convert the dataset into 简体, we need a Good 繁体 -> 简体 converter. If we have the 繁体 -> 简体 converter, our first problem has been solved. XD

Of course, I will submit the onnxruntime code as soon as possible.

Maybe just use the look-up table (https://github.com/PaddlePaddle/PaddleSpeech/blob/0eb598b876f99bd26fa735577da92d46c45dc3fd/paddlespeech/t2s/frontend/zh_normalization/text_normlization.py#L81 ) to solve this problem temporarily.

Of course, I will submit the onnxruntime code as soon as possible.

Thank you!

@GitYCC but even use a not too good 繁体 -> 简体 converter, you can also get a dataset (but maybe the number of available data will be reduced), I don't know wether the reduction of dataset will reduce the effect of model for 简体 g2pw

Actually, in my opinion,
no matter whether the 繁体 -> 简体 converter is good or not, if we just use the such converter, the effect of "pre-use on dataset" is same as the one of "post-use on changing input",
because filtered error cases of "pre-use on dataset" would not be trained on models and such models still can not deal with missing char. during conversion.

I'm not very familiar with NLP, I naively thought:

The G2P BERT will only have to be trained for polyphonic ones, not word such "概念"

Because people in mainland China use simplified Chinese, I naively thought that there might not be as many "missing chars" as traditional Chinese for the pre-trained Bert vocab, for example,  simplified "概念" maybe in BERT's vocab, but opencc cannot convert  simplified "概念" to traditional "概念", and even traditional "概念" in BERT's vocab, traditional g2pw still cannot deal will simplified "概念" input,

I have an idea to get a good look-up table. We can use the google translation to help us.
Like this way,

I will change the converter by this way in the future.

good idea

oh, I just found that, when input "概念", the error not because of 简体 -> 繁体 converter, but because there are not polyphone in "概念", so the texts output of prepare_data is [], so the input of bert is [].. maybe a empty judgment will fix this
sent before convert: 概念，
sent after convert: 概念，
sentences: ['概念，']
[] [] [] [['gai4', 'nian4', None]]
texts: []
onnx_input: {'input_ids': array([], dtype=float64), 'token_type_ids': array([], dtype=float64), 'attention_masks': array([], dtype=float64), 'phoneme_masks': array([], dtype=float32), 'char_ids': array([], dtype=float64), 'position_ids': array([], dtype=float64)}
[概念，] not in g2pW dict,use g2pM
sent before convert: 你我，
sent after convert: 你我，
sentences: ['你我，']
[] [] [] [['ni3', 'wo3', None]]
texts: []
onnx_input: {'input_ids': array([], dtype=float64), 'token_type_ids': array([], dtype=float64), 'attention_masks': array([], dtype=float64), 'phoneme_masks': array([], dtype=float32), 'char_ids': array([], dtype=float64), 'position_ids': array([], dtype=float64)}
[你我，] not in g2pW dict,use g2pM
sent before convert: 你好
sent after convert: 你好
sentences: ['你好']
char in polyphonic_chars: 好
['你好'] [1] [0] [['ni3', None]]
texts: ['你好']
onnx_input: {'input_ids': array([[ 101,  872, 1962,  102]]), 'token_type_ids': array([[0, 0, 0, 0]]), 'attention_masks': array([[1, 1, 1, 1]]), 'phoneme_masks': array([[0., 0., 0., ..., 0., 0., 0.]], dtype=float32), 'char_ids': array([580]), 'position_ids': array([2])}

bert-base-chinese_s2t_dict.txt

#11

Add g2pW to Chinese frontend PaddlePaddle/PaddleSpeech#2230

Hi, I'm not familiar with PaddleSpeech, now I only want to use g2p in PaddleSpeech to get the pinyin of sentences (in order to speed up the pinyin generation), could you give a tiny code example? Thanks a lot!

please check https://github.com/PaddlePaddle/PaddleSpeech/tree/develop/examples/other/g2p @beyondguo


Release 1.1.0 🚀🚀🚀

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


docs(example): alter refresh_example to use fb prefix in fb_login_protocol_scheme

Fixes #586

Note that this is not complete, there have been some other direct changes to the example that must be incorporated into the refresh_example.sh script so they aren't lost

https://github.com/thebergamo/react-native-fbsdk-next/commit/1cfb158e0c2bff16dc02d06d771692b11fae6299

After altering refresh-example to make that change, then refresh-example should be committed one more time, and then the results of a run should be committed (with all the updated example files...)

in fact https://github.com/thebergamo/react-native-fbsdk-next/commit/1cfb158e0c2bff16dc02d06d771692b11fae6299 did a good job - it updated the refresh-example script at the same time
So I'm running it again now just to update things and verify this change but then it should be good to go

Something appears to be broken in the example when it moves from react-native 0.75.x to 0.76.x and I haven't quite figured out what.
Relates to native module auto-loading as a likely root cause, and symptom is that the example app fails on boot with the Settings native module being null / inaccessible. Reproduces on android with true or false newArchEnabled in gradle.properties.


Add Negative Test for Express Flow-Typed Test

This commit fixes the following error when running the ./run_def_tests.sh command:
\flow-typed\definitions\npm\express_v4.x.x\flow_v0.32.x-\test_response.js
 1:1  error  Test has to include at least 1 negative Tests
According to the Flow-Typed documentation, this is a requirement.  See also:
https://github.com/flowtype/flow-typed/wiki/Contributing-Library-Definitions#writing-libdef-tests

What is  up with the single quotes being replaced by double quotes?  When I run git commit, it happens.  Also the following command-line output and my guess is some formatting task is doing this:
husky > npm run -s precommit (node v6.11.3)

25l[22:43:38] Running tasks for gitDir [started]
[22:43:38] Running tasks for package.json [started]
[22:43:38] Running tasks for *.js [started]
[22:43:38] Running tasks for gitDir [skipped]
[22:43:38] → No staged files match gitDir
[22:43:38] Running tasks for package.json [skipped]
[22:43:38] → No staged files match package.json
[22:43:38] prettier --write [started]
[22:43:39] prettier --write [completed]
[22:43:39] git add [started]
[22:43:39] git add [completed]
[22:43:39] Running tasks for *.js [completed]
25h25h[expressFix 63abcc4b] Replace double quotes with single quotes one more time.

Yeah we have prettier running as a post commit hook. It's helped make the code base a lot more uniform, but at the cost of some frustration (and messy PRs).

I am not getting the same errors on my local environment that is showing up when travis-ci runs the tests.  I wonder if there is a base image/container/VM that would allow me to have the exact same environment.

What kind of output are you getting when running the specs locally?

I have been running ./run_def_tests.sh from the root of the project which outputs details of the warnings and errors along with a summary:
✖ 361 problems (9 errors, 352 warnings)

npm ERR! Test failed.  See above for more details.


Object type not supported

When I first create the repository all the objects will pull down but when I go to move them to another system I get the error Object type not supported for SRFC, AQSG, IWMO and a bunch of others.

It seems that if the object has already been created that it will push and pull the changes.  If it has to create the object then that is when it has issues.  Is this correct?

Updated to latest version and still get the error:

@vtebano13 can you test SRFC serialization in your scenario?

Sorry been a little while.  I updated to the newest version and when I run the ZABAPGIT I get the error:

@vtebano13 see http://docs.abapgit.org/ref-supported.html
IWMO and IWSV is not natively supported by abapGit, but try installing https://github.com/larshp/abapGit-Plugins as noted in the bottom of the documentation

@vtebano13 are you sure you want to version these objects? I do not version them because those objects are automagically generated from IWPR objects in our case (however, you need abapGit-plugins for IWPR anyways).

closing, see https://github.com/larshp/abapGit/issues/75 instead


The songs will not automatically enqueue by selecting multiple music files and play

Description
When highlighting multiple music files and pressing enter to play, only one music file will be played and not enqueue all the music files selected.
Steps To Reproduce

Go to the folder containing music files.
Highlight multiple music files.
Press enter to play the selected music files.
Only one music file will be played, other selected music files will not be enqueued.

Expected behavior
All the selected music files should be enqueued and played in a list, but not only one music file will be played.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.376
Windows Version
Windows 11 21H2 22000.376
Relevant Assets/Logs
debug.log
debug_fulltrust.log

What program do you have as default for music files?

What program do you have as default for music files?

I tried with foobar2000 and Groove Music also same problem, and sorry for the mistake of windows version, it is win 11, not win 10.


slf4j 1.7.30

slf4j 1.7.30 was released in December 2019
https://github.com/qos-ch/slf4j/releases

Updating the parent pom version is insufficient. Log4j-slf4j-impl provides support for the EventData object that was removed in SLF4J 1.7.25. Log4j-slf4j-impl also specifies 1.7.25 as its version, so changing the parent pom probably won't accomplish what you want. Upgrading the version is probably fine in master, but that requires the EventData support be removed. We do not want to upgrade the release-2.x branch since that will break backward compatibility. However, user's can certainly upgrade to a more recent version of SLF4J (although it appears SLF4J is apparently no longer being maintained) so long as they don't require the EventData support.

obsolete.


The Home button

When I click the About of this website, I found that page is pretty great, but I did not find the button that I can come back to the Home page. So I think if the author can add a button that link to the Home page should be better.

Issue addressed in commits. Thanks!


Fix missing or incorrect return statements in UnCore.h

Methods involved are TArray::operator=() (which is meant to disable array copying), where the return type was incorrect, and FString::AppendChar(), which did not return a value at all. Wonder why this never came up before.

Nice catch! No idea why VS and gcc didn't catch these.

Merged in https://github.com/gildor2/UModel/commit/defc6d2406d46cea0381ca3861f2c866595092ec

Sorry, I was forced to roll-back automatic merge because it created an extra branch, and re-do the work by myself. Probably you'll need to sync your fork with my code by hands.

No problem! Yeah, it's a strange one, it only showed up for me when I __declspec(dllexport)ed a bunch of classes for a test, including these two. Maybe it doesn't even get compiled if it's FORCEINLINE and not used anywhere?


tlog synchronisation fails with: connect: cannot assign requested address"

I saw one of my passive minio getting redeploying forever.
After checking the logs I saw:
2019/03/13 13:50:52 [minio.29e99b48-a007-4f59-8335-4c6f12db2384]{"error":"dial tcp <IP_ADDRESS>:1032: connect: cannot assign requested address","level":"error","master":false,"msg":"failed to write transaction log","namespace":"d42e487f-52d4-4f55-ac3d-2ce7065a0b54_tlog","subsystem":"tlog","time":"2019-03-13T13:50:52Z","tlog":"<IP_ADDRESS>:1032"}

So for some reason the minio cannot connect to the its own tlog 0-db to write the metadata.

Here is the full logs of the passive container before it got redeployed: https://gist.github.com/zaibon/ae24cf08c74054b88359fae0007355cb
Seems here the problem doesn't come from a connection error, but some error during the replay of the tlog.

Can u reach this address from minio container?

@muhamadazmy yes the zerodb was actually reachable.
I've been search a bit about connect: cannot assign requested address, it seems this happens before the connection is even created so problem is not about 0-db reachability

I wonder if the don't have 2 different issues here.
Cause from my inspection of the logs it seems the error comes from lines like this: https://gist.github.com/zaibon/ae24cf08c74054b88359fae0007355cb#file-gistfile1-txt-L30
{"error":"open /minio_metadata/objects/awscli-mint-test-bucket-15348/prefix/directory/datafile-1-MB: not a directory","key":"awscli-mint-test-bucket-15348/prefix/directory/datafile-1-MB","level":"error","msg":"failed to write metadata","namespace":"d42e487f-52d4-4f55-ac3d-2ce7065a0b54","subsystem":"disk","time":"2019-03-13T14:34:08Z"}
The error is "Not a directory" and indeed it seems that  /minio_metadata/objects/awscli-mint-test-bucket-15348/prefix/directory is actually a file. I have no knowledge of how the layout of the metadata is supposed to look like but that doesn't sounds right here.

I am working on reporducing the issue now, and from the logs above, it might be bug in how we handle prefixes. Since s3 itself doesn't have concept of directories, it's rather prefixes it's possible to have a a/b as a file and also a/b/c as a file! but this won't work with our structures.
Anyway, once we reproduce the issue, we can be sure.

Updates so far:

awscli mint tests on master, although they seem to pass i can see some errors in the logs with the same error above. but they still report success anyway
it happens on the master minio, so it's not "for sure"  a slave or synchronization issue. the issue is detected because the slave just can't apply the same meta as the master, but it detects as failure to sync.
I will have to look deeper why we end up in this state, i will dive in mint tests for "awscli".
I think it might be related to how s3 actually support directories (via prefix + delimiter) as explained in the previous comment. in that case it wouldn't be possible to fix, and we will have just to ignore the error for now

https://github.com/threefoldtech/minio/commit/550a9084a72200d1104020e5bc0a11f90b45ddce


Ensure charts save project alias

This is a follow up to https://github.com/project-koku/koku-ui/pull/1949. Want to ensure charts save the project alias, similar to the Cost Explorer chart.
https://issues.redhat.com/browse/COST-1316

Codecov Report

Merging #1954 (08ab73c) into master (dbc1a5c) will increase coverage by 0.45%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1954      +/-   ##
==========================================
+ Coverage   71.17%   71.63%   +0.45%
==========================================
  Files         372      372
  Lines        6227     6821     +594
  Branches     1119     1466     +347
==========================================
+ Hits         4432     4886     +454
- Misses       1678     1914     +236
+ Partials      117       21      -96

Impacted Files
Coverage Δ

src/components/charts/common/chartDatumUtils.ts
57.14% <100.00%> (+2.38%)
:arrow_up:

...c/pages/costModels/createCostModelWizard/table.tsx
34.28% <0.00%> (-10.16%)
:arrow_down:

src/components/i18n/i18nProvider.tsx
57.14% <0.00%> (-9.53%)
:arrow_down:

src/pages/state/maintenance/maintenanceState.tsx
66.66% <0.00%> (-8.34%)
:arrow_down:

.../pages/costModels/createCostModelWizard/markup.tsx
27.27% <0.00%> (-8.03%)
:arrow_down:

...ges/costModels/costModelsDetails/utils/toolbar.tsx
77.50% <0.00%> (-7.80%)
:arrow_down:

...ostModels/createCostModelWizard/priceListTable.tsx
32.07% <0.00%> (-7.46%)
:arrow_down:

.../costModels/costModelsDetails/bottomPagination.tsx
82.60% <0.00%> (-7.40%)
:arrow_down:

src/store/common.ts
81.81% <0.00%> (-7.08%)
:arrow_down:

...ponents/reports/reportSummary/reportSummaryAlt.tsx
52.94% <0.00%> (-7.06%)
:arrow_down:

... and 124 more

Continue to review full report at Codecov.

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


Add autoheal-groups

By setting AUTOHEAL_CONTAINER_GROUP=mygroup you can limit the autohealing to a specific group of containers

how is this different to AUTOHEAL_CONTAINER_LABEL?

True, now that you mention it...


[Suggestion] Add Floating Compass

Hey,
it would be great if the maps have a floating compass which is visible all the time while moving the map.
Very useful for call outs.

Hey, thanks for the suggestion!  So, the map never rotates in the site so north is always pointing to the top.  You think the compass would just be a handy reminder?

Oh, right.
Yea, would be good as reminder. Doesn't need to be big, maybe the in game one at the bottom right corner and toggleable.

I'm going to try putting something on each map in a noticeable but non-static position (ie as you pan, the compass will pan away too).  That way it is a reminder, but doesn't clutter up each view, and won't require a new option in the menu (which I don't want to have too many of if possible).   I'll comment here once that is in.


Multifile Editor hints don't always match users code when running tests too quickly

Describe the Issue
If I enter a bad solution in the multifile editor and run the tests, the hint doesn't always match the hint my code should produce - seemingly when I run the tests too quickly in succession.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-by-building-a-cat-photo-app/step-1
Steps to Reproduce
On step 1 of the cat photo app, the code
CatPhotoApps
 should produce the hint: Your `h1` element's text should be `CatPhotoApp`...

Go to https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-by-building-a-cat-photo-app/step-1
It should have the default code in the editor:
Hello World

Run the tests
The hint should read The text `CatPhotoApp` should be present in the code. You may want to check your spelling.
Quickly change the code to
CatPhotoApps
 and run the tests again
The hint may still show the previous hint
If it doesn't, quickly use undo to change the code back to the default, and the hotkeys to run the tests
Hint may still show text for when using code
CatPhotoApps

Expected behavior
It to show the correct hint.
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

I'm currently using Chrome on MacOS 13.1. I've tried the change as quickly as I could, but cannot reproduce the error. I've attached the screen recording below:
https://user-images.githubusercontent.com/97413019/227628189-e59aee71-a234-4d45-8a9d-37e0a0251fb0.mov

This is difficult to reproduce, I am closing it for now. It is unlikely a camper will run into this problem.


Configurable job parallelism

I'm introducing a runner configuration option, concurrentJobs, to allow multiple jobs to be scheduled in parallel.

Good start

Good to see some work on this. The data dirs could be keyed by job ID or similar. Reaping them becomes a little tricky. Some folks may like to re-use build directories in some circumstances too (even though that's not always going to be safe). Just something to think about.

I feel like concurrent jobs for the same project in the same branch don't make sense at the moment. Maybe that should be the limitation.

I feel like concurrent jobs for the same project in the same branch don't make sense at the moment. Maybe that should be the limitation.

Hmm! That's going to be tricky to manage - an async.queue won't cut it any more, because more commits can arrive on a branch corresponding to an existing job even while the queue is unsaturated.
Concurrent builds on a single branch will be an issue if they trigger deployments, though.

The data dirs could be keyed by job ID or similar. Reaping them becomes a little tricky.

I had been thinking about maintaining an in-memory structure of data directories that are in-use and excluding them from the culling somehow. I think the runner can get away with something in memory because simple-runner jobs don't persist across service restarts anyway - otherwise I'd need to either parse the job ID from the data directory name or use some kind of flag file to identify active data directories.

Some folks may like to re-use build directories in some circumstances too (even though that's not always going to be safe). Just something to think about.

Re-using a data directory from one job to another? I'd thought that was what the cache was intended for. With the job ID in the data directory name, how does that work - or am I misunderstanding?

So there are two concepts, the cache and the data dir (as far as I know) and the cache is used by certain plugins but the data dir is where the project is cloned into and built. Personally I always copy the contents from the data dir to /var/www/<project name> when using strider locally.

Okay, I've revisited this to:

Use a custom JobQueue. It's an async queue that prevents the concurrent scheduling of tasks that share a key, namely a (project, branch) tuple. Otherwise, it runs parallel jobs up to a defined concurrency limit.
Prune recent data directories specific to each project and branch rather than globally. (This should also be helpful for #28.) Because jobs on the same branch don't run concurrently, this'll prevent Strider from deleting a build directory that's in use by a concurrent job.

Here's what the data directory structure looks like:
~/code/strider-simple-runner (parallel-jobs=)
$ find /Users/ashl6947/.strider/data -type d -maxdepth 2
/Users/ashl6947/.strider/data
/Users/ashl6947/.strider/data/deconst-deconst-control-master
/Users/ashl6947/.strider/data/deconst-deconst-control-master/job-56f29d31a141bc8723e5bf9a
/Users/ashl6947/.strider/data/deconst-deconst-docs-control-master
/Users/ashl6947/.strider/data/deconst-deconst-docs-control-master/job-56f29d30a141bc8723e5bf98
/Users/ashl6947/.strider/data/deconst-deconst-docs-master
/Users/ashl6947/.strider/data/deconst-deconst-docs-master/job-56f29d2fa141bc8723e5bf97
/Users/ashl6947/.strider/data/deconst-deploy-master
/Users/ashl6947/.strider/data/deconst-deploy-master/job-56f29d30a141bc8723e5bf99

I've also tried to be careful in extracting branch names from the job in a sane way because Job's ref field can hold arbitrary things. I'm testing for a few common keys (branch, fetch) but falling back to an ugly-but-safe "URI-encoded stable JSON representation of the whole thing".

Do you think this is ready for merge? Did you have any plans to write some basic tests for this JobQueue?

Nvm, you have tests.

I was just giving it a last look-over to see if there was anything obvious I'd missed. I don't think so :smile:

So it's probably a good idea to expose the configuration in Strider UI.

Yeah, agreed. The number of recent directories to keep should be simple enough to add. The level of concurrency is less clear to me.
Is there a place for installation-wide (rather than build-specific) configuration for runner plugins?

Hum, no but https://github.com/Strider-CD/strider/pull/681 would help when I get more time to finish it.

How is it configured now, an env var?

Yeah, CONCURRENT_JOBS.

It's a little generic, but I wasn't sure how far to go down the STRIDER_SIMPLE_RUNNER_... prefixing road. I can patch it if there's a pattern I wasn't following.

No, this is good. I documented it in the strider repo and updated the version of simple-runner.


Please help me! Unexpected "nothing" in line 0

Help me please

The error message is pretty self-explanatory, you should remove the nothing from your line 0.

@monyxie thank you, but that doesn't work, i got a none compiler error

Add Nocode to your PATH. It's what worked for me.

Try recompiling with --nonothing. Hope it helps.


fix(Makefile)!: add ledger build flag to build with ledger support by default

Overview
Mainly add ledger flag to add ledger support by default.
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Ideally, we had a test to ensure that the build flag works as intended. I think there is a mock-ledger (at least when coding the KMS that was what we used; albeit in Rust).

Codecov Report

Merging #2014 (a668ce6) into main (c66ab98) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2014   +/-   ##
=======================================
  Coverage   21.54%   21.54%
=======================================
  Files         126      126
  Lines       14285    14285
=======================================
  Hits         3077     3077
  Misses      10915    10915
  Partials      293      293

@liamsi has this PR been tested? In the absence of a test, can you please document in the PR description how it was manually verified?

has this PR been tested? In the absence of a test, can you please document in the PR description how it was manually verified?

I did only test if I can import a key from a ledger:
celestia-appd keys add my-key-name --ledger --index 2
which succeeded. Without the changes in this PR it previously failed. (note the index is optional; I just wanted to test with a different derivation path).

this LGTM! should we be worried about the docker build failing? should we set the default to false to avoid this for now until we fix docker?
#12 66.97 In file included from /go/pkg/mod/github.com/zondax/hid@v0.9.1/libusb/libusb/os/linux_usbfs.c:43,
#12 66.97                  from /go/pkg/mod/github.com/zondax/hid@v0.9.1/hid_enabled.go:26:
#12 66.97 /go/pkg/mod/github.com/zondax/hid@v0.9.1/libusb/libusb/os/linux_usbfs.h:24:10: fatal error: linux/types.h: No such file or directory
#12 66.97    24 | #include <linux/types.h>
#12 66.97       |          ^~~~~~~~~~~~~~~
#12 66.97 compilation terminated.
#12 106.6 make: *** [Makefile:62: build] Error 1
#12 ERROR: executor failed running [/bin/sh -c make build]: exit code: 2

#6 [stage-1 1/4] FROM docker.io/library/alpine:3.18.2@sha256:82d1e9d7ed48a7523bdebc18cf6290bdb97b82302a8a9c27d4fe885949ea94d1
------
 > [builder 5/5] RUN make build:
26.57 go: downloading github.com/go-playground/locales v0.13.0
26.57 go: downloading github.com/jmespath/go-jmespath/internal/testify v1.5.1
66.97 # github.com/zondax/hid
66.97 In file included from /go/pkg/mod/github.com/zondax/hid@v0.9.1/libusb/libusb/os/linux_usbfs.c:43,
66.97                  from /go/pkg/mod/github.com/zondax/hid@v0.9.1/hid_enabled.go:26:
66.97 /go/pkg/mod/github.com/zondax/hid@v0.9.1/libusb/libusb/os/linux_usbfs.h:24:10: fatal error: linux/types.h: No such file or directory
66.97    24 | #include <linux/types.h>
66.97       |          ^~~~~~~~~~~~~~~
66.97 compilation terminated.
106.6 make: *** [Makefile:62: build] Error 1
------
ERROR: failed to solve: executor failed running [/bin/sh -c make build]: exit code: 2

Circling back on this. I verified that I can add a key from Ledger
$ celestia-appd keys add ledger-1 --ledger

- address: celestia1xdc39xr027r78w4eeq6e7e35glgza9ka9rsds2
  name: ledger-1
  pubkey: '{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A+u46124WB1YIjLA+Ufriy5aQzcvdM4bily/Y32INEGV"}'
  type: ledger


KeyPress event absent

duplicate of https://github.com/go-p5/p5/issues/6.
if you want to pick up the slack, there were 2 attempts at fixing this:

https://github.com/go-p5/p5/pull/52
https://github.com/go-p5/p5/pull/53


Adjust MCC config and http timeouts

Timeouts have been adjusted as a mitigation. Hence, leaving the MCC auto-configuration via IoT conn string as-is.


Fix total_pages for StopSearcher when results are not on a per_page boundary

Jose Alvarez made this commit awesome.
Signed-off-by: Tim Mecklem<EMAIL_ADDRESS>

We should add the per_page > 0 check in the valid? method

@tmecklem It looks like @geofflane fixed this in a23b53f380b088d0876ece5627e9765c0b620ca0

@rockwood thanks for letting me know about that. I merged @geofflane's changes into the branch. Jose and I were on the fence about failing validation vs. overriding invalid params with the defaults, and I like what Geoff did better than where we last ended up.
I added in the kaminari gem for pagination and changed the StopSearcher to use it per @geofflane's suggestion in Trello.


Allow draft content to be returned

One of my company's clients wanted the option to autocomplete draft wagtail content too. This adds a way to do that by not using the .live() queryset when a custom queryset is defined.
I couldn't find any immediate usages for the objects view, but I modified that too, seems nice to have a custom queryset there.
Sorry for the linting, my editor did that. If needed I can re-apply my changes without the lint change.
Thanks for building this library!

Hi, just wanted to follow up and see if it's possible to get this merged?


Move samples from project root

There is a sample directory at the top of the project. These samples should move to under their specific topic.

Should be fixed now.


[DOCS]

Description
Add README.md file.
Screenshots
No response
Additional information
No response

Pls assign me this task.


Sync vendored version of more_itertools between setuptools and pkg_resources

I added this (more_itertools) here because it was listed in pkg_resources.extern.names.
The version corresponds to the version that already exists in the pkg_resources/_vendor the directory... But that is different from the version in setuptools/_vendor/vendored.txt. Maybe we should synchronise these versions so that it makes the life easier for downstream packagers "de-vendoring" setuptools?

Originally posted by @abravalheri in https://github.com/pypa/setuptools/pull/4320#discussion_r1574750784
Potentially other packages too.

For the record, downstream packagers don't pin versions to whatever was vendored. That would be completely unmaintainable.

This should already be resolved now that there's a single _vendor folder right ?

This should already be resolved now that there's a single _vendor folder right ?

Yes, I think so.


Error: "Failed to create icon" on flutter pub run msix:create command

I tried to build my sample application as windows installable file with this package but when I ran the final command flutter pub run msix:create then an error occurred.
I am using msix: ^0.1.12
Here are full error details or output from the terminal
C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\my_test_project>flutter build windows

Building with unsound null safety
For more information see https://dart.dev/null-safety/unsound-null-safety

Building Windows application...

C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\my_test_project>flutter pub run msix:create
getting config values..  [√]
validate config values..  [√]
cleaning temporary files..  [√]
create icons folder..  [√]
copy icons..  Unhandled exception:
fail to create icon C:/Users/Abhishek%20Kumar/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/msix-0.1.12/lib/assets/icons/icon.png: FileSystemException: Cannot copy file to 'C:\Users\Abhishek Kuma
r\Development\AndroidStudioProjects\TestProjects\my_test_project/build/windows/runner/Release/icons/icon.png', path = 'C:/Users/Abhishek%20Kumar/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/msi
x-0.1.12/lib/assets/icons/icon.png' (OS Error: The system cannot find the file specified.
, errno = 2)
#0      MsixFiles._copyIcon (package:msix/src/msixFiles.dart:271:7)
<asynchronous suspension>
#1      MsixFiles.copyIcons (package:msix/src/msixFiles.dart:38:33)
<asynchronous suspension>
#2      Msix.createMsix (package:msix/msix.dart:25:5)
<asynchronous suspension>
pub finished with exit code 255

C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\my_test_project>

hello Abhishek
Maybe the problem is file path with a space,
I'm not sure yet.
Please try to setup the 3 icons in your  pubspec.yaml like that:
msix_config:
logo_path: C:<PathToIcon><Logo.png>
start_menu_icon_path: C:<PathToIcon><Icon.png>
tile_icon_path: C:<PathToIcon><Icon.png>
with that the problem is still happening?

@YehudaKremer Thanks for your reply.
I tried the above solution but now another error occurred
Here's all command I ran
flutter clean && flutter pub get
flutter build windows
flutter pub run msix:create
Here's the full output
C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\beeta_lubricants>flutter clean && flutter pub get
Deleting .dart_tool...                                              18ms
Deleting .packages...                                                3ms
Deleting Generated.xcconfig...                                       2ms
Deleting flutter_export_environment.sh...                            2ms
Deleting ephemeral...                                               15ms
Deleting ephemeral...                                                3ms
Deleting ephemeral...                                                6ms
Deleting .flutter-plugins-dependencies...                            2ms
Deleting .flutter-plugins...                                         3ms
Running "flutter pub get" in beeta_lubricants...                    3.2s

C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\beeta_lubricants>flutter build windows

Building with unsound null safety
For more information see https://dart.dev/null-safety/unsound-null-safety

Building Windows application...

C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\beeta_lubricants>flutter pub run msix:create
getting config values..  [√]
validate config values..  [√]
cleaning temporary files..  [√]
create icons folder..  [√]
copy icons..  [√]
create manifest file..  [√]
copy VCLibs files..  Unhandled exception:
FileSystemException: Directory listing failed, path = 'C:/Users/Abhishek%20Kumar/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/msix-0.1.12/lib/assets/VCLibs/x64\*' (OS Error: The system cannot
 find the path specified.
, errno = 3)
Instance of '_StringStackTrace'pub finished with exit code 255

C:\Users\Abhishek Kumar\Development\AndroidStudioProjects\TestProjects\beeta_lubricants>

the file path comes from the config file .packages which is uri encoded.

the file path comes from the config file .packages which is uri encoded.

Hey, @micross thanks for this info.
What do I need to modify from my side? Can you please explain a bit?

@Abhishek0116  this line msix/main/lib/src/configuration.dart
the simple way is to remove the space in  Abhishek Kumar.

@micross Thank you for your inputs!

i update the code to use package_config.dart instead of .packages.
added Uri.decodeFull

@Abhishek0116 please check if the problem solved in version 0.1.14

Hello @Abhishek0116 Did the fix help you?

Thanks, @micross and @YehudaKremer I have fixed my issue with your help and tested my created Windows app successfully. As @YehudaKremer said that issue is fixed in version 0.1.14 and currently I tested successfully with version 0.1.15 so I am closing this issue.


no dashing2 cmp help info

Hello Daniel,
I compute sketches using dashing2 sketch and store sketch using - - outfile. There is a sketch and a sketch.name.txt. But I am not sure how to feed those to dashing2 cmp since no help is provided. I looked into the code and it confuses me. I can use -cmpout to have the distance but I want to check how long sketching and cmp take.
Thanks,
Jianshu

Hello Daniel,
time dashing2 sketch -k 11 -S 12000 --threads 24 --pminhash --topk 250 --cmpout phage_GPD_topK_250.txt -Q name.txt -F reference.txt
With and without the --topk 250 option, I have exactly the same output. Am I making a mistake? I am using the newest v2.1.11 for 512bw. Forgive me the sketch command read me is a little bit long/confusing.
Thanks,
Jianshu

Hi Jianshu,
Sorry for the wait! It's been a busy couple of weeks.
I've added in cmp usage (https://github.com/dnbaker/dashing2/commit/3b71c9cdb925aa582921e89a7cc66f62f773f9d4), so thank you for pointing that out.
You can pass sketched files to cmp, but you have to sketch all the original input files together. For example, something like this:
dashing2 sketch  <sketching options> -F filelist.txt -o stacked_sketch_file.
dashing2 cmp <sketching options> --presketched stacked_sketch_file.rc_canon.sketchsize1024.k32.SetSpace.DNA.opss

This way, it's broken into two stages (which you can time). This also makes it easier to work with larger collections, since the sketch matrix can be memory-mapped and therefore exceed system RAM.
Does that help?
I'll look into the topk 250 option results as well. I'm not sure what's going on there, but I'll let you know.
Thanks,
Daniel


feat(beerus_core): add get_storage_at + tests

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

What is the new behavior?

Beerus core can now call starknet_getStorageAt

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Codecov Report
Base: 34.09% // Head: 34.13% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (9d3476c) compared to base (f91a968).
Patch coverage: 34.78% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   34.09%   34.13%   +0.03%
==========================================
  Files          21       21
  Lines         393      416      +23
==========================================
+ Hits          134      142       +8
- Misses        259      274      +15

Impacted Files
Coverage Δ

beerus_core/src/lightclient/starknet/mod.rs
32.35% <0.00%> (-22.65%)
:arrow_down:

beerus_core/src/lightclient/beerus.rs
97.26% <88.88%> (-1.18%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Translation of the deploy file to Spanish

What kind of changes does this PR include?

Translated content

Description
Added implementation file translation to Spanish

Hi @Franqsanz 👋, welcome to the Astro Docs and thanks so much for your contribution. I would love it if you joined us in our Discord chat! 🙌
If you want to be an active i18n contributor, there you can grab the @i18n-gang-es role (#manage-roles channel) and join our Spanish translation thread, this way you can easily talk with other Spanish i18n contributors, ask for reviews, be aware of when someone needs a review and notify people of the pages you're going to translate to avoid duplicate effort.
You may also want to see our Translation Status Overview, there you can see the pages that are available to be translated (marked as i18nReady: true) and those that need updates.

Bienvenido @Franqsanz 🥳, gracias por la contribución!

Hi @Franqsanz, I'm really sorry for the delay, just reviewed your PR and LGTM! Welcome to the Docs' Face Pile 🥳, I hope to see you again with more awesome contributions! 🙌

Hello @Yan-Thomas , no problem, that's good news.
Thank you very much for accepting my PR, a pleasure to contribute!


Optimize ofClient method InstallServiceFlows

Parameters nodeLocalExternal and svcType are not very readable and scalable. Replace these two parameters with externallyAccessible. externallyAccessible indicates that if the Service is accessible externally, like NodePort, LoadBalancer and ExternalIP are accessible externally. ClusterIP is not accessible externally.

@hongliangl since the change is for #4866 and we are close to release, let's close this one and just use #4866?

@hongliangl since the change is for #4866 and we are close to release, let's close this one and just use #4866?

Sure, I'll close this one.


[BUG] Connection reset by peer is not handled well by retry policy

We observe Connection reset by peer being propagated as an error in tests and reported by customers.
Looking for holistic solution for this problem in the core.
References:
Customer reported: https://github.com/Azure/azure-sdk-for-java/issues/21066

Thanks for submitting this issue @kasobol-msft.
Just finished a rough investigation on this. It appears that when the expected response body of a request is void/Void the networking layer won't eagerly read the response body (as there isn't expected to be one). But, later in the call stack when converting the response headers, and body but there is none here, into the returned response there is an attempt to consume/fast-forward the response body. This is where the issue lies as doing this can result in a state when the remote closes/closed the connection triggering an exception to be thrown. Unfortunately, at this point the response has egressed past the retry policy, resulting in the exception being thrown to the caller instead of being captured and retried.

There seems to be few missing cases that we should cover:

buffered upload
download
pageable operations

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=900469&view=ms.vss-test-web.build-test-results-tab&runId=19372912&resultId=103744&paneView=debug

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=938985&view=ms.vss-test-web.build-test-results-tab&runId=20123708&resultId=101687&paneView=debug

Looks like there are still some gaps https://dev.azure.com/azure-sdk/internal/_build/results?buildId=954096&view=ms.vss-test-web.build-test-results-tab&runId=20421522&resultId=100738&paneView=debug

@kasobol-msft, has this happened recently or has this issue been fixed by #22647

This has happened recently, most likely because we exhausted retry count. We should follow up with aggregated exception and logs from retry policy.

Closing as this hasn't been a reported issue for a while and if this is seen again a new issue should be opened.


add Camera-Direction feaure

We can display direction of player and direction of camera separately by adding signature of direction of camera.
additional requirement

use signatures-x64.json with CAMERA signature
separate images to "player part" and "fan-shaped part" from "player.png"

Added!

Fixed insufficient signature mask.


chore: add 1.24.9-hotfix.20230509 1.24.10-hotfix.20230509 1.25.5-hotfix.20230509 1.25.6-hotfix.20230509 1.26.0-hotfix.20230509 1.26.3-hotfix.20230509

What type of PR is this?

What this PR does / why we need it:
This hotfix resolved kubelet issue when the volume is not detached after the pod and PVC objects are deleted
ref: https://github.com/kubernetes/kubernetes/issues/114207
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.973%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
2544

Relevant Lines:
4105

💛  - Coveralls


Sticky divider when showing Brew Properties.

When the Brew Properties editor mode is selected, dragging the editing:preview pane divider does not release when the mouse is released. One must hit reload to release.
This does not happen with Brew Editor or Style Editor modes.

Duplicate of #1388.


How to solve this problem

File "/home/water/Downloads/aster.pytorch-master/lib/models/attention_recognition_head.py", line 75, in beam_search
batch_size, l, d = x.size()
ValueError: too many values to unpack (expected 3)

@waterai12 Did you solve the problem?

Who can solve this problem? I can't fix it. Please tell me if you know how to fix it. Thank you so much!


Any idea why this is happening?

Working good, using two plugins, one for code blocks and one for tables and I'm seeing a lot of warnings in console:

Any idea how this could happen?

Please check #23 and #25 .

What is the solution? I don't really understand what I should do. I am not using SvelteKit btw

Oops, I'm sorry.
#17 & #18 are related issues.
You should write variables as props:
MdTable.svelte
<script>
export let type = undefined;
export let tagname = undefined;
export let position = undefined;
export let __index = undefined;
// HACK: ignore `unused-export-let`
tagName && type && position && __index;

// And you code ...
</script>


Incorrect selection

Issue Type: Bug
I have wxs file with many rows (about 7000). There are many similar strings in these rows. When I select part of string it highlight, but not always.
VS Code version: Code 1.34.0 (a622c65b2c713c890fcf4fbf07cf34049d5fe758, 2019-05-15T21:59:37.030Z)
OS version: Windows_NT x64 10.0.17134
Extensions (6)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

vsc-material-theme
Equ
2.8.2

CppSnippets
har
0.0.14

cpptools
ms-
0.23.1

csharp
ms-
1.19.1

rust
rus
0.6.1

/duplicate of https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aissue+is%3Aopen+find+results+limit we do have a limit of find/word results highlight.


Removing PDF button as they are not supported on Reference documentation

Docs Build status updates of commit ef8e292:
:clock10: Full build: the average full build time is 7 min(s) 53 sec(s), this is based on the last 760 build(s) for this repository.

Docs Build status updates of commit ef8e292:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add an option that controls whether lists are sorted

It would be great to have a setting which configures whether lists are sorted or not.
My rationale is that sometimes it is helpful to be able to organize list elements into groups with comments etc. For example, linter rule codes.

We are aiming to be similar to the Black formatter. The less configuration the better. The only kind of contribution I would accept is a fmt on/off comment block.

Okay, that's a fair decision. Would you accept an update to the docs which says so explicitly? E.g. a Philosophy paragraph.

Definitely 😃


iCloud3 v3, Beta 15 (Preliminary)

This is a preliminary beta 15 zip file.
icloud3-beta15.zip

Hi,
init.py is missing in zip file.
Fred

Fixed in beta15p2.

II've been testing with Beta 15 since today.
Before that I used the v2 for a long time.
Thank you for all your work @gcobb321


Create dynamic UI design for plugin parameters collection

The screen is generated according to endpoint declaration and builds a form to populate with values.
To start these might be:

Experiment dropdown (with search) -- PROBABLY if we don't add this to experiment UI
File dropdown inside experiment
string, multiline string, float, int.

@str-anger @jatinriverlane @safeamiiir @gabrielg42 pls review


Replace Infoblox.Override.FakeInfobloxEnabled by func IsFakeInfoblox() bool

I'd like to test the dns package more - currentlhy 20.4% coverage (of which infoblox is a part), and I'll use mock instead of FakeInfoblox class.
At the moment no test hits the fakeInfoblox section (https://github.com/k8gb-io/k8gb/blob/master/controllers/providers/dns/infoblox-ibclient.go#L41-L49).
To enter this section we use envar FAKE_INFOBLOX - in my opinion unnecessary. Entering the section (which is not technically used) is the only reason why we keep this variable.
I created a temporary function on Infoblox{}.IsFakeInfobloxEnabled() that says it is a fake infobox, instead of using a special variable in config.
ATM I can't say on 100% but probably delete IsFakeInfobloxEnabled() completely in a following commit when I inject IBRConnection mock in controller tests instead.
Signed-off-by: kuritka<EMAIL_ADDRESS>

will reopen after successfully mock Infoblox connection


feat: Upgrade jQuery

Major bump for jQuery. Everything seems to work just as well, did not find any issues during automated and manual tests.

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Plans to support longer sequences?

Right now, the model (correct me if I'm wrong) appears to be locked down to sequences of max 512, based on running & playing with the code (and this makes sense in the context of the paper).
Are there any near-term plans to support longer sequences?
Offhand, this would potentially require multiple issues to be addressed, including 1) allowing positional embeddings that can extend for longer or perhaps arbitrary lengths (with some degradation over longer lengths than it has been trained on, of course) (possibly using something like multiple sinusoidal embeddings, like in the original transformer paper?) and 2) containing/limiting the Transformer quadratic memory explosion (my first gut would be to try something like the techniques in "Generating Wikipedia by Summarizing Long Sequences" https://arxiv.org/abs/1801.10198).
Right now--from first pass--it seems like the way to use this over longer sequences is to chunk the docs into sequences (either inline with fixed lengths, or possibly as pre-processing on boundaries like sentences or paragraphs) and apply BERT in a feature-input mode, and then feed into something else downstream (like universal transformer).
All of this seems doable, but is 1) more complicated from an engineering perspective and 2) loses the ability to fine-tune (at least in any way that is obvious to me).
(Of course, having a model adept to longer sequences like in https://arxiv.org/abs/1801.10198 has model power trade-offs, such that it is plausible that the feature-based approach could still plausibly be more superior?)

We don't plan to make major changes to this library, so anything like that would be part of a separate project.
Our recommended recipe is exactly what you describe (it's what we do for SQuAD), but you can actually fine-tune on it normally (we just don't do it for SQuAD because only a few percent of SQuAD documents are longer than 384 do so it didnt matter. But we should have).
Let's say you have:
the man went to the store and bought a gallon of milk
And had max_seq_length = 6, stride  = 3, then you could split it up like this:
the man went to the store
to the store and bought a
and bought a gallon of milk

So from BertModel's perspective this is a 3x6 minibatch, but crucially you can reshape it after you get it back from BertModel.get_sequence_output() and softmax over all the tokens when you compute the loss (with some masking to make sure you don't double count the boundary words like to the store and and bought a). So you will be fine-tuning over the whole document end-to-end. The exact implementation is task-specific of course.

to the store a

Hi,
It looks like a good solution wherein a longer sequence is broken down into shorter sequences. I was wondering if it is feasible to apply the same technique to sequence of length ~100,000 tokens.
Also, could you elaborate more on reshaping from the implementation point of view?
Thanks.

Hi @oakkas
No, I haven't tried the above solution yet but I will resume this soon. Do you have any updates on this and would like to share here?
Thanks!

Hi again @vr25.
Not yet either. I was pulled into another project for now but hopping to start experimenting soon too.

Hey @vr25 @oakkas
Did any of you try it already?

Hi, I had to do classification of long texts, most of which had 500 - 1000 tokens, but some could contain up to 500k tokens.
So I did a system, heavily inspired by Jakob Delvin's comment above: split my 1024-token text into a minibatch of 2x512 tokens, then concatenated the 2 outputs of the CLS tokens 2 x 768 -> 1536 and put a regular classification head on top of it. Then finetuned the whole system end-to-end.
(The particular implementation was on FlauBERT from huggingface transformers trained in Pytorch.)
Due to very specific nature of my texts (lots of numbers, tables and other structures), I didn't do any striding. So basically, I had no attention span between the 2 individual parts of text.
For my problem this trick gave a meaningful perf gain, but it didn't change the world. The classifier was already doing quite well on truncated texts of 512 tokens, I just managed to push it a bit more. I also tried minibatches of 4x512 tokens. But this didn't give meaningful improvement as only ~5% of my texts were longer than 1024 tokens. These conclusions are task specific, of course.
We also tried to implement a BERT with an attention mechanism of the Longformer and compare with it: https://arxiv.org/abs/2004.05150
But for reasons unknown, the latter took significantly more time to train in our configuration, so we abandoned the idea. Note, however, that this long training is almost certainly not due to the Longformer itself. I'm very curious to see such comparison if someone makes it. :)

@vmaryasin Can you please elaborate why do you believe the long training process is certainly not due to the Longformer itself?
Thanks

@donglinz It'll get a bit messy and historical here. :)
There exist 2 French language BERTS: CamemBERT and FlauBERT with slightly different implementation. At that time, we haven't yet made a final choice of the model for the project. It turned out that due to the way attention layers are coded and accessed in 2 models, it was much easier to implement Longformer on Camembert. On the contrary, for historical reasons the above trick was implemented on Flaubert.
What we observed was that a Longformer took significantly more time to train. But later on we noticed that a large part of this delay was actually due to pure Camembert being slower than pure Flaubert in_our_particular_setup. This was the strangest thing and unfortunately, we didn't find the reason why. And as we already had a system which was working alright, we abandoned Camembert and did not try to reimplement Longformer on FlauBERT either.
So it's best to say, that I don't have any conclusion about Longformer. But I wanted to mention it above as it seems to be a good option and I'm curious to see the comparison of the two methods.

Hi, I am working on a NER task and I have a lot of sequences of text where the sequence_length goes above 512, is anyone having any idea of how can I tackle this.
One solution which I have thought of is: simply dividing the input sequence into multiple input sequences so that it fits the limit of 512, then apply the NER classification to these sequences and bind back the NER labels.
Has anyone faced a similar issue/ has found a better solution to this? Please let me know.

So I did a system, heavily inspired by Jakob Devlin's comment above: split my 1024-token text into a minibatch of 2x512 tokens, then concatenated the 2 outputs of the CLS tokens 2 x 768 -> 1536 and put a regular classification head on top of it. Then finetuned the whole system end-to-end.

Hi @vmaryasin what did you do for the samples that only had 1 output (aka fit into a 512 chunk)? Did you duplicate the first output or pad it somehow?

Hi @lucaguarro,

I padded the input text with zeros to 1024 length the same way a shorter than 512-token text is padded to fit in one BERT. This way I always had 2 BERT outputs.
I truncated the text.

I believe, those are specific design choices, and I would suggest you test them in your task. To be honest, I didn't even ask myself your Q1. As for the longer texts, I tried training a 4x512 model, but it wasn't any better, to say the least.
Below is my text length histogram, I should have probably tried the 3x512 instead...

Hi, we implemented the classification model for longer texts following the Jacob Devlin' comment:

repo is available here: https://github.com/mim-solutions/roberta_for_longer_texts
length of the input text is arbitrary, however for longer texts it needs proportionally more GPU RAM during fine-tuning, because all text chunks are fed to BERT as a minibatch
as default it uses base english BERT, however the pretrained model can be changed to any model compatible with transformers library - in the instruction it is described how to do it for polish RoBERTa model
we used it on a set of texts from polish websites - it gave a mild improvement over the model with simple truncation
we have not done more precise benchmarks yet
more technical details can be found here: https://github.com/mim-solutions/roberta_for_longer_texts/blob/main/docs/bert_for_longer_texts.md

Hi, I described more details about the above open source implementation in the Medium articles:

Part 1
Part 2

Repo is available here.

Hi, I described more details about the above open source implementation in the Medium articles:

Part 1
Part 2

Repo is available here.


empty example

its empty https://github.com/jfrog/jfrog-npm-actions-example/blob/main/.github/workflows/build-publish.yml#L1

https://github.com/jfrog/jfrog-npm-actions-example/blob/main/.github/workflows/build-publish.yml#L1

https://github.com/jfrog/jfrog-npm-actions-example/blob/main/.github/workflows/build-publish.yml#L1

its empty https://github.com/jfrog/jfrog-npm-actions-example/blob/main/.github/workflows/build-publish.yml#L1


feat: per-version uuid validation

adds specific validation for all uuid versions
should be 100% backward compatible

hey @broofa, I like your suggestions.
The main reason for this is because of a project specifically checking for v4 UUIDs, we were using the uuidv4 which deprecated and switched to internally redirected to uuid's validate, which wasn't fit for purpose in our project.
I think there is still value for people being able to call for per-version validation.

I've preserved the old commit if you'd prefer for me to redo it as a separate commit?

IMO so much fragmented source code is awful (hard to maintain). And this is exactly the reason I personally always avoid default exports.

IMO so much fragmented source code is awful (hard to maintain). And this is exactly the reason I personally always avoid default exports.

@TrySound I'm not a fan, but I appreciate the importance of consistent style rather than one specific style, so I'm happy to do it to stay in-line with the rest of the library's source.

The main reason for this is because of a project specifically checking for v4 UUIDs

Does this add enough value for enough users to warrant inclusion in this module, though?  Validation functions are trivially easy to implement, as demonstrated by the implementations in this PR.  It may be simpler/better to direct users to roll their own so they get the exact behavior they need (e.g. including validate() or not, or allowing multiple versions for something like const isNamespaceUuid = uuid => version(uuid) === 3 || version(uuid) === 5).
The cost of including these is mostly the API and documentation bloat.  While it may not seem that big a deal, this affects a lot (100K's or M's?) of users, 99% (???)  of whom won't need these methods.

IMO so much fragmented source code is awful (hard to maintain).

Does file fragmentation affect dead code removal ("tree-shaking")?   The current function-per-file schema is an artifact of the CommonJS deep-include approach in version 3 of this module, but that's been deprecated so there's no compelling reason for it.  Assuming bundlers are smart about removing dead code within a module we can structure the files in whatever way makes the most sense. For example, the v1-v5 methods are large and distinct enough they should probably be separate (I wouldn't expect much disagreement on that), but the other methods could go in a 'util.js` file.

This feature request has been raised a couple of times in the past, see https://github.com/uuidjs/uuid/issues/526#issuecomment-704951831 and https://github.com/uuidjs/uuid/issues/498#issuecomment-664896676 and my suggestion has always been to simply make use of the existing API and write out the one-liner
validate(uuid) && version(uuid) === uuidVersion;

that this pull request implements.
Now this PR adds 239 lines of code and I'm with @broofa in that I think this is really a lot of overhead to maintain in the future for something that can already be achieved by combining two existing API calls in one single line of code.

So unless we find really compelling reasons of why having to learn about another 4 methods of the API surface is significantly better for end users than to just combine two existing ones my tendency is to not accept this pull request.
I would, however, be more than happy to accept a pull request that improves the documentation and provides the above mentioned one-liner as a copy & pastable example.

@ctavan 160 of those lines are in the documentation, but I see your point.
I'm happy to create a new PR for the addition of example snippets for per-version validation.
Where would you like them in the docs @ctavan ?

@ctavan 160 of those lines are in the documentation and 47 are in testing, but I see your point.

This is precisely the point 😉 . It's documentation and tests that are tedious to maintain on the long run.

I'm happy to create a new PR for the addition of example snippets for per-version validation.
Where would you like them in the docs?

Where have you been looking for it (to then be disappointed not to find version-specific validation)?

Where have you been looking for it (to then be disappointed not to find version-specific validation)?

Hahah, I suppose by the validate and version methods. Though I feel it'd be out of place in the API Summary section of the documentation.
What about a new Additional Examples section @ctavan?

I agree with you in principle, but the question is if people will find it if it appears in another section?
I personally wouldn't mind just adding another short paragraph and another code block after the uuid.validate() example.

Makes sense 👍

Haven't runmd yet, but what do you think to this?
https://github.com/Mearman/uuid/commit/55d26edd216b707f1e111fd57e5275aadfe8a353

For my personal taste this becomes to verbose. I'd restrict it to one simple example:
import { version as uuidVersion, validate as uuidValidate } from 'uuid';
function uuidValidateV4(uuid) {
  return validate(uuid) && version(uuid) === 4;
}
const v1Uuid = 'd9428888-122b-11e1-b85c-61cd3cbb3210';
const v4Uuid = '109156be-c4fb-41ea-b1b4-efe1671c5836';
console.log(uuidValidateV4(v4Uuid)); // RESULT
console.log(uuidValidateV4(v1Uuid)); // RESULT

And I'd also not put it under a new section but just as a second example in the validate section.

Closed in favour of https://github.com/uuidjs/uuid/pull/543


Feature request: Add pole-zero plot

I don't know if it fits in this package, but it would be great to add pole-zero plots.
A basic example looks as follows:

Ideally, the scaling would be logarithmic or linear with the Option for different tik labels (dB or powers of ten) and the matching tiks/grid. Please make the positive/negative sections of a axis configurable to cover separate values (like $-10^5$ to $-10^1$ on negative x-axis and $10^-3$ to $10^1$ on positive x-axis).

Sounds great.
I think zpk format is sufficient. There are other solutions to convert a tf in zpk, so I think there is no need to do it in LaTeX.


Bdd list and query

Codecov Report

Merging #93 into master will decrease coverage by 1.23%.
The diff coverage is 90.9%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   95.91%   94.68%   -1.24%
==========================================
  Files           5        3       -2
  Lines         294      282      -12
==========================================
- Hits          282      267      -15
- Misses         12       15       +3

Impacted Files
Coverage Δ

server.py
93.75% <90.9%> (ø)

app/connection.py

app/server.py

app/models.py

app/__init__.py

app/engine.py

engine.py
100% <0%> (ø)

models.py
94.17% <0%> (ø)

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 e4e03ac...5cb46f7. Read the comment docs.


nestjs v7.x support

will it support nestjs v7.x?

@heavenlyteam Thanks!

will it support nestjs v8.x?

@gamedevsam Thanks for #9

Exceptional!


chore(main): release 1.4.2

:robot: I have created a release beep boop
1.4.2 (2024-02-03)
Bug Fixes

make default responses valid code (#69) (96d7dfa)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.4.2 :sunflower:


ANR in org.covidwatch.tags.android

Description: Crashed when launching the app from Androdi Studio
Device: Samsung A3 2017, Android 8
Crash logs:
https://gist.github.com/madim/7ae2998cac9051743a6d5c2925ea7630
Might possibly relate to: https://github.com/covid19risk/covidwatch-android/issues/80#event-3244660757

which branch is this error from?
I don't see "Context.startForegroundService" call (causing the crash) in the master or develop branch?

@prasad79 I'm not sure if it is related to this crash, but I was able to reproduce ANR in develop. It happens every time you launch the app from Android Studio
It is probably related to how we start and stop BluetoothManager (Service), because if we comment it out, ANR goes away

There is a bug indeed.
We should use only bindService and unbindService methods.

I'll update the library to the new version.
Plus fix some things related to this in the BluetoothManager


Add new variable to set the number of replica for the webhook

Creation of two variables: webhook.conversionReplicaCount and webhook.admissionReplicaCount to handle the number of replicas for the conversion and admission webhook.
By default, these variables are set to 2.

@floozy Thanks for the contribution, but this repo is for release purpose only, so please contribute to the longhorn repo instead.
cc @derekbit @weizhe0422


Can't add firebase-php to Symfony 3

1. Which version of kreait/firebase-php are you using?
2.3.1
2. Which version of PHP are you using?
7.0
3. What's the issue?
Im triing to add firebase-php to my symfony 3 project as say in this guide (https://github.com/kreait/firebase-php/blob/69496a13084710ff36d693439297e778dde178f0/doc/symfony.md) and i get this error:

There is no extension able to load the configuration for "firebase.config" (in /var/www/html/app/config/config.yml). Looked for namespace "firebase.config", found "framework", "security", "twig", "monolog", "swiftmailer", "doctrine", "sensio_framework_extra", "easy_admin", "fos_user", "fos_rest", "nelmio_cors", "nelmio_api_doc", "jms_serializer", "debug", "web_profiler", "sensio_distribution" in /var/www/html/app/config/config.yml (which is being imported from "/var/www/html/app/config/config_dev.yml").

4. Code that lead to the issue
config.yml
#FIREBASE
firebase:
    class: Kreait\Firebase\Firebase
    arguments: ["%firebase_databaseURL%"<EMAIL_ADDRESS>
firebase.config:
    class: Kreait\Firebase\Configuration
    calls:
        - ['setFirebaseSecret', ['%firebase_apiKey%']]
...

Parameters.yml
    firebase_apiKey: <my_code>
    firebase_databaseURL: https://<myurl>.firebaseio.com
...

Controller
namespace CronBundle\Controller;

use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Doctrine\ORM\Tools\Pagination\Paginator;

class CronController  extends FOSRestController implements ClassResourceInterface
{

    /**
     * @Route("/cron/firebase"), name="api_login" }
     */

    public function firebaseAction(Request $request){
          echo "HELLO";
          exit;
    }

[UPDATE]
I add config.yml content to config_dev.php and the error change a little:

There is no extension able to load the configuration for "firebase" (in /var/www/html/app/config/config_dev.yml). Looked for namespace "firebase", found "framework", "security", "twig", "monolog", "swiftmailer", "doctrine", "sensio_framework_extra", "easy_admin", "fos_user", "fos_rest", "nelmio_cors", "nelmio_api_doc", "jms_serializer", "debug", "web_profiler", "sensio_distribution"

Please help.
Thank you.

The guide you are referring to (kudos for finding it ;)) is targeted at version 0.x of the library and can't be applied to the current version.
If you're using PHP 7 anyways, you could also update the library to a 3.x release.
I have not tried it out, but you should be able to use the Firebase Factory class as a factory service.
Please try this and let me know about your progress - I don't know when I can give it a try, but as soon as I can, I will add a section to the docs and/or publish a bundle.


Make memory fence in ByteBufferGuard explicit

Description
Current implementation of ByteBufferGuard uses an implementation-based assumption that AtomicInteger#lazySet causes full-barrrier (ie [prior stores] happen before [following loads & stores] is required).
Since VarHandle appearance there have been explicit methods for performing memory fences including full fence thus it seems reasonable to rely on them rather than ungaranteed semantic.
It's worth mentioning that unlike #9824 which proposes to use new explicit access modes on the field, this does not change the type of invalidated or its access methods but simply replaces the old non-obviously invoked barrier with an obvious one.

Didn't know about it, but I like it!

I like it, but actually this class should go away at some point!

The whole ByteBufferGuard does not help too much to prevent code from failing and crushing the VM! But yes, it's a good idea to enforce more.
I played with that in the past: ByteBufferGuard never helped much, a full fence neither. This class was bullshit from the beginning as it only delayed the crushing of JVM a bit more on misuse.
So it should never have existed...

Sorry,
I'd suggest to not touch ByteBufferGuard at all, it did its job to throw AlreadyClosedException instead of segmentation fault consistently until memory access is compiled by Hotspot. Don't touch it.
W have to live with it only until Project Panama is in mainline JDKs. Unfortunatley as of today, not even 21 will ship with non-preview Panama, so tricky compilation to prevent preview warnings in #12294 is needed.

Sorry,
I'd suggest to not touch ByteBufferGuard at all,

No problems! Was glad to hear your review on this change)

Had to go back and re-read what all this was about in the beginning. I like the patch (nice API) but sort of agree with Uwe - the current solution is flawed (but working 99% of the time). The patch wouldn't change it to work correctly but it has the potential of having unknown side effects... Let's just keep the existing solution until a truly sound one is available.

I think it's a good idea to replace this tricky way to flush any CPU caches, such as using lazy set, with an explicit full fence. The purpose of a full fence is to provide a reliable memory barrier that works consistently across different architectures.
But if you're quite certain that the days of this class are numbered, then it's fine.
(By the way, we are still planning to test it in our implementation of the directory interface).

I tried the full fence a year ago, it did not help when code is optimized by hotspot. For sure it throws AlreadyClosedException when running in interpreted mode (so it helps users to figure out that their code is broken during testing). When in production, it is too late.
I will keep this open a few days. I would merge it (and also backport it), because I agree with @sherman: it can't get worse. The static method in VarHandle does not have any side effects, it just makes explicit what the AtomicInteger code did before implicit.

Please also add a CHANGES.txt entry in 9.7 section.

All done

Applied and cherry-picked on 9.x branch.


[Bug]: View column modes within the result set, and the data will not change after switching columns

ODC version
odc423
OB version

What happened?

What did you expect to happen?
The data within the column pattern will change
How can we reproduce it (as minimally and precisely as possible)?
View column patterns within the result set and then switch columns
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit 0850b7d9d7755eb697866971b85715cb77c60122
------
咏和
Tue Dec 5 09:58:24 2023 +0800

pass


Unable to Manage from Controller (Direct, not Manage in New Tab)

Bug Report
Unable to Manage from Controller (Direct, not Manage in New Tab)
System Information

Operating System (Including distribution name and version number) Ubuntu 20.04.5
AMP version and build date (Always use the version number, 'latest' is not valid!) <IP_ADDRESS> 27/10/2022 18:15
Which AMP release stream you're using (Mainline, Nightly or FastTrack) Release

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do? Manage a target or instance from the controller (via double click method)
What are you expecting to happen? I manage the instance
What is actually happening? ('Nothing' is not an acceptable answer!) "Unknown error" appears with [0] (MissingFieldException) : Login method requires username(String) argument in the console for the target.

Reproduction

(Controller has been up for 12 days)
Login to controller
Press manage on Target
See Unknown Error appear on the screen
See controller log show Authentication token for ampadmin requested by ManageInstance on behalf of ampadmin
See below error:
02:29:49
Authentication attempt for user ampadmin from <IP_ADDRESS>
02:29:50
Authentication success
02:35:08
Returned exception from API call Core/Login
MissingFieldException
[0] (MissingFieldException) : Login method requires username(String) argument.
at GSMyAdmin.WebServer.WebAttributes.() at GSMyAdmin.WebServer.APIService.()

Can I get a retest on this? I can't reproduce this issue on CI.

Closing due to reproduction failure. Re-open this if it occurs and can be verified not to be a configuration issue.


[Feature][doc] Add how to change standalone database type

Search before asking

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

Description
Currently, our standalone server doc only told us how to start the server in https://dolphinscheduler.apache.org/en-us/docs/latest/user_doc/guide/installation/standalone.html. and we should notice user the database is in memory and standalone server also support other type of database like postgre and mysql
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I think this issue is helpful to developers. If those who are new to dolphin know how to use mysql for standalone server, they will not lose all data after each time restarting the server. I'd like to take this one.


How can I use odas studio?

Nice to meet you.
I'm  using Respeaker 4 mic array now.
But I can't use odas studio.
I installed node.js and npm in my PC.
I don't know the case.
If you are familiar with how to use it, I want to know the details of the procedure for using it on a PC.

Thank you.

Hi @suekiti0327,
You state you have installed node.js and npm but require help with the installation?
What problem are you experiencing so that I can help you resolve the issue?
Did you follow the ODAS Studio installation instructions?
Did you use node.js v8? It is the required LTS version for ODAS Studio.

Hi @GodCed
For confirmation, I'm glad if node.js and npm can teach you how to install.
Also, I'm glad to share with you the steps after installing them.
When I install node.js and npm and execute npm start as root, I get an npm error.
I was following the installation procedure of odas studio.
I use node.js v8.
That's the answer to your question.
I apologize for the inconvenience, but thank you.

This is the situation when you get an error.

Please see the node.js installation instructions.
$ node -v should return v8.

Please see the node.js installation instructions.
$ node -v must return v8.

Please see the node.js instructions.
$ node -v should return v8.

In that case, how should I change the current version?
Could you tell me?


Electron version

4.0.0-nightly.20180821 works. What's up with the use of version 2?

We've been focused on the core of the application and haven't touched the UI in months. When we return our focus to the UI, which could be a while, we will likely upgrade the Electron version as one of our first tasks. Thanks!


Postgres (v15) TEXT[] is view as _TEXT in wields check

Hi, I'm just getting started with this lovely crate! Really like the ergonomics.
But for some reason my schema migration contains arrays of text that aren't being interrupted properly by the checking process in wields.
I get the following error:
public.users (High): The Column `permissions` is has changed, db_type: _TEXT welds_type: TEXT[]
public.users (High): The Column `roles` is has changed, db_type: _VARCHAR welds_type: TEXT[]

Yet in my schema, I do try using TEXT[]:
permissions text[] NOT NULL DEFAULT array[]::text[],
roles varchar[] NOT NULL DEFAULT array[]::varchar[]

All other fields work fine (including text and customs), just not the array ones. Any clues?

Congratulations! I think you found a bug :)
Would you mind sending me an example of your tables schema to help me get this fixed?
Best I can tell, the ORM is doing CRUD correctly on these types, just detecting changes where they don't exist. Is that what you are seeing?

Sure thing!
-- User Types
CREATE TYPE authority AS ENUM(
    'None',
    'Moderator',
    'Admin',
    'SuperAdmin',
    'Owner'
);

-- Users
CREATE TABLE
    users (
        id UUID DEFAULT gen_random_uuid () PRIMARY KEY,
        enabled BOOLEAN NOT NULL,
        disabled_reason VARCHAR(255),
        username VARCHAR(255) NOT NULL UNIQUE,
        email VARCHAR(255) NOT NULL UNIQUE,
        has_image BOOLEAN NOT NULL,
        name VARCHAR(255),
        surname VARCHAR(255),
        joined TIMESTAMPTZ NOT NULL DEFAULT now(),
        authority AUTHORITY NOT NULL DEFAULT 'None',
        permissions text[] NOT NULL DEFAULT array[]::text[],
        roles varchar[] NOT NULL DEFAULT array[]::varchar[]
    );

Quite a minor workable bug at least 😁 As a work arround in my test I'm just filtering out these mismatches like so:
let filtered_diff: Vec<welds::check::Issue> = diff
    .iter()
    .filter(|v| {
        v.kind.as_changed().map_or(true, |v| {
            !((v.db_type == "_TEXT".to_string()
                || v.db_type == "_VARCHAR".to_string())
                && v.welds_type == "TEXT[]")
        })
    })
    .map(|v| v.to_owned()) // iter .to_owned() differed
    .collect();

fix for this is up at
https://github.com/weldsorm/welds/pull/17

Awesome, I'll close this since you've already got a pull request to track this. Thanks for the awesome work you're doing and support with the crate!


Copy edit module 06

Mostly typos / slight word changes. Some formatting changes to properly display lists in HTML. Broke out some questions into a list to highlight them better.

Looks great, thank you!


High CPU Usage and Battery Life

The Ayaneo Next Lite has almost half the battery life compared to windows 11
CPU usage is also High

Alpha 2 will include basic power management that should help. In a future beta, we will include more advanced power management. Our goal is for everything to be automatic behind-the-scenes so you do not need to mess with manual TDP, frequency, undervolting, etc. settings yourself.

Alpha 2 has been released with initial battery improvements. I measured a 2x battery life increase when in the Playtron interface. Improvements while in-game will be coming in a future release.


fix oncontextmenu

prevent oncontextmenu only parent right menu

thank you

sorry,  I was engaged with my works before. And I didn't know how to merge branches in Github.
I am learning how to do that.
I will be merging your branches in the future.
I am very food.
Thank you very much.

Have I merged？
我去完全搞不懂Github 啊


feat(scan): fetch test types from API

closes #493

@maksadbek, please resolve conflicts and rebase onto next branch. Once that's done, I will be able to proceed with the review.


Revise build.gradle.

Added following features:

Specify JRE_CONTAINER version in eclipse
Add -sources artifact

test this please

BUILD SUCCESS.

LGTM.

BUILD FAILURE

retest this please.

retest this please.


[ENH] estimators failing check_regressor_against_expected_results

Describe the feature or idea you want to propose
no idea on these

[ ] "RDSTRegressor"
[ ] "RISTRegressor"

Describe your proposed solution
fix or create issue
Describe alternatives you've considered, if relevant
No response
Additional context
No response

Hello, I'd like to join the project by working on this issue.

Hi @ricor07, just a heads-up, this issue is a bit tricky, so if you are new to python/numba/git CI, this might not be the easiest place to start with.
The problem we're facing is that these two estimators (RDST and RIST) are failing a test (check_regressor_against_expected_results, see this file and this one to reproduce the results) on some of the platforms we are testing on (Ubuntu/Windows/MacOS).
The problem we are facing is that the expected results can differ (despite fixing a random seed) from platform to platform. The goal of the issue is to understand and fix this issue, but we didn't have much success on that for now. You can go in the test_config file to make these test run on the CI.


Add sr-only class to styles mixins, so that consuming apps without the class still see the correct behavior

See https://deploy-preview-10--vue-accessible-selects.netlify.app/?path=/story/selectsingle--primary

Closed in #13


Update to go-ipfs v0.4.7

generated commands.md and api.md
updated install.md

Thanks for taking care of this