Add support for dark mode in React Native

In other words, re-create the template files using the latest code created by the React Native CLI.

Fixed by #937.


Update dependencies

Codecov Report
Patch and project coverage have no change.

Comparison is base (0e08ad5) 100.00% compared to head (732b9e3) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #369   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          311       311
  Lines         9961      9961
  Branches       975       975
=========================================
  Hits          9961      9961

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update to 0.3.1a

Well this is the fix for my issue, they just released an update.

Just came here to see if you'd put up the PR yet :)

Just came here to see if you'd put up the PR yet :)

Haha ^^

Thanks for pointing this out. This is kinda sub-optimal though since the change is just in the beta and it's not like having an extra skip which was still more or less compatible. I added a version selector now so that it works correctly for both beta and stable: 9c692d1


m_UseMultiThread in AdvancedImageToImageMetric not initialized

The member m_UseMultiThread in the AdvancedImageToImageMetric is not initialized which causes ValGrind errors.
Adding the line:
this->m_UseMultiThread = false;
to the constructor will fix this.

Thank you for reporting this issue!


Bug in signatures page

Got this bug when navigating in to the page after providing the password

The only way to get rid of it is to refresh the page, navigate away and return did not fix the issue

did you get did from approve signatures (looks like that from the log)?
if so, i tried to reproduce but didn't get it

It seems to be a specific state the page is on when this happens, I also can not reproduce this.
So closing it for now.


feat: added support for wait_till in classic clusters with new variable is_vpc_cluster

Description

Added support to wait on cluster creation for IKS/ROKS classic
Added new variable is_vpc_cluster (by default true) to differentiate between Classic and VPC

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

:tada: This PR is included in version 1.19.0 :tada:
The release is available on:

GitHub release
v1.19.0

Your semantic-release bot :package::rocket:


fix: packageManager config set error

Description

Additional context
https://github.com/vitejs/vite/pull/5133
What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

https://nodejs.org/api/packages.html#packages_packagemanager
need to set the version of packagemanger, or it will get the error like:

When set the version, the corepack can work as normal:

@patak-js Yes, sadly this is correct :(
I would like that npm/node would support like only declare the package manager or e.g. pnpm@6 or pnpm@=>6 or<EMAIL_ADDRESS>But packageManager NEEDS to be set to the EXACT version. This field gets also bumped by dependabot / renovate!

The field only get read out when you set corepack enable on your system. After enabling this flag, you are not possible to do dev stuff in vite repo => bumping to p4

The field only get read out when you set corepack enable on your system. After enabling this flag, you are not possible to do dev stuff in vite repo => bumping to p4

And this config only be used at node v16.9.0 or after


Add Cypress tests and documentation

This PR addresses some things requested by the maintainer of NGINX Proxy Manager

Cypress automated tests
Documentation for setting up SSO with OIDC
A more user-friendly error message for when a user does not exist in NPM and they are attempting to sign in via an IdP
Fixing a typo in the "hint" description so that the name of "Redirect URL" matches the field label in the modal.

@oechsler I just added some additional UI-based end-to-end tests. These tests test logging in with a test user when OIDC is both enabled and disabled

Hey @chutch1122, I completely missed the fact that you made further changes with your last push and merged them directly. If you think a review would still be beneficial, I can revert the changes and reopen this PR. Apologies for the mix-up on my part.

@oechsler Feel free to take a look at it. I'm just opened up a new PR anyway to merge in changes from the upstream develop branch. There were some changes to error messages at some point after the last time develop was pulled in that was causing the build to fail


type that is imported twice with different aliases is removed

Expected Behavior
i have code like this:
import a.b.Class
import a.b.c.Class as Class1
import a.b.c.Class as Class2
...

code that uses Class1
...
code that uses Class2

and while this is pretty strange its valid and works.

Observed Behavior
when i reformat that code with ktlint the second import is just removed, so the code no longer compiles.
If ktlint really wants to remove the duplicate import it should probably change all usages of Class2 to Class1 in the body of the file.
I hope this is obvious and easy to reproduce, if not i can try to create a sample project to reproduce it.

Nice find.
Ktlint should keep both imports in my opinion. Rewriting the code to eliminate one of the imports is not what people would expect from a linter.


Reduce the bloat the kstore package brings in.

Describe the Problem Statement
Either switch to lighterweight kube client, or move the kstore impl into a different binary.
Describe the Enhancement
No response
Alternate Solutions
No response
Additional context
No response

This was completed by moving this functionality to another process, nexd-kstore


p1_HierarchicalAttention_model_transformer.py

I have tried your code in text classification in github. Thanks for your sharing.
However, I come across with a problem when running the test code in “a05_HierarchicalAttentionNetwork/p1_HierarchicalAttention_model_transformer.py”.
In the test code, you have set all the input docs the same. But why the model can predict diffirent label for the docs with same content? And why the accurracy will become 1 after a few iterations?
Also, we find that when we give a different input content with same labels in the HAN model, we always get a same result on the test dataset.
Looking forword for your reply.

hi,
input_x = np.zeros((batch_size, sequence_length))
-->should be input_x = np.random.randn((batch_size, sequence_length))
test() function is used to check whether model can run without error, and if possible,
we want to show that model can have some ability to learn simple task in a very short time.
so you can design a new input and output as a toy task.
I used this model for a text classification task, it can works. so you may try to let the model pass toy task first.
toy task: I've just updated this file.

发件人: dulusir<EMAIL_ADDRESS>发送时间: 2018年7月18日 16:18
收件人: brightmart/text_classification
抄送: Subscribed
主题: [brightmart/text_classification] p1_HierarchicalAttention_model_transformer.py (#71)
I have tried your code in text classification in github. Thanks for your sharing.
However, I come across with a problem when running the test code in “a05_HierarchicalAttentionNetwork/p1_HierarchicalAttention_model_transformer.py”.
In the test code, you have set all the input docs the same. But why the model can predict diffirent label for the docs with same content? And why the accurracy will become 1 after a few iterations?
Also, we find that when we give a different input content with same labels in the HAN model, we always get a same result on the test dataset.
Looking forword for your reply.
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/brightmart/text_classification/issues/71, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASuYMCgDdRw2DhGfdox0clAepyJ0omqEks5uHu9YgaJpZM4VUJY7.

how the model tells the docs with all zeros from each other?


option for disabling hash navigation

Is there an option for disabling hash navigation? looking through the source, it appears the answer is no. Since hash now appears to be part of the core module, I'd be interested in adding that as an option.

1+ for this.
When using livereload during development everything gets all foobarred when the browser refreshes on pages other then the first. Think it has something to do with the hash not working properly. 😟

@eads @popkinj It cannot be disabled or easily hacked around without forking right now. I agree though that it should be. I'll include this in the 2.0 list, but as you may be able to tell this (deck.js) isn't high on my priority list right now.

Thanks @imakewebthings .. Understood.
Appreciate the response. ☺

@popkinj For what it's worth, here's how we're handling it: https://github.com/nprapps/lookatthis/blob/master/posts/test-project/www/js/app.js#L342

Thanks @eads . That should help a lot.😊

Thanks @eads it helped a lot :)


Inconsistent runes caused by alternate URLs

If a user has some followers following them via a their raw dat:// ID, and others following them via an alias like a hashbase URL, some mutuals will be incorrectly displayed as non-followers (with ~ instead of @).

I've just noticed fix_port addresses this but I'm unclear whether it's a complete solution.

I don't think it is, but it's a step toward. People need to stop sharing their hash base urls.. Or set it into their dat.json

Is there any way to "merge" or associate these URLs? Don't 100% understand everything about the dat:// protocol, so maybe this nonsensical, but I think I remember someone making a commit which replaced hyperlinks to hashbase URLs (e.g. in the sidebar) with the associated dat:// links. So, is there any way to make it so when someone copy-pastes in a hashbase URL, it looks up the associated dat:// and adds that to the list in portal.json instead?

Actually, yeah, I'll have a bash at this just now.

Never mind, looks like @cblgh's fix_ports patch already takes care of it.

does it happen automagically though? (so that hashbase urls never exist in portal.json)

ohhhhh oh i just realized we could basically run what fix_port does whenever the user saves.
i the idea late last night but doing too drastic changes after midnight is never really a good idea *yawns*

I believe that this is fixed. I haven't had this issue since the past couple of PRs.

I'm still seeing this, but I think I have a fix for it

Okay, I've figured out a fix but it's hugely inefficient and probably overkill for a minor issue like this. Plus, the problem should hopefully resolve itself as people upgrade their clients, so this can probably be closed for now.

I think it's not working for me; I'm on dat://maxhaessle.in/ and rofollow says deam ( dat://6f2b20227b414fa602c5f2ea9ded7d37819f44117c5b338af949279534d79317/ ) is following me; on their dat it says @maxhaesslein so that's correct but on my dat its ~deam so the wrong rune get's displayed.
I'm using this method to redirect dat://maxhaessle.in to my real dat ( dat://1c16b383863398cc028a3d18e7ab807b8e8b43f3c7d5a81a698355f830e89d3f/ ) -- so I think it's just a redirect, not a copy of my original dat like hashbase does and maybe that is causing the issue for me.

So I fixed it for me -- see this commit.
The problem was, that my own dat was saved in r.portal.data.dat as the unresolved uri.
I'm not sure about my changes in portal.js though, maybe someone can have a look at it?

This is still an issue for me. Is there anything I need to do to fix it?
I see people who are following me with a @ on dat://1327ca6d4caf3c4665f5b82d72a5dc716ddf24f58f253e857c5703b9b6a69838/, but when using dat://elifessler.com/ I see only ~ (as no one's following me using that URL).

close per #124,


every other docker run is failing

${{ github.base_ref }} appears to be null on some runs, investigating

warning message to be fixed, not related to the error:
Input 'version' has been deprecated with message: The version input will not be supported after May 30, 2020. Use buildx-version instead


Added support for CORS while running on a different host

Goal
Fixed  #19 - Cross Origin Resource Request error when the agent is running on a different host.
Design
Implemented HTTP OPTIONS method handler with relevant Access-Control-Allow-Origin headers to support CORS.
Changeset
Added
New function to handle OPTIONS method.
Tests
Manually tested & verified by running the agent in a remote host.
Discussion
Linked issues
Issue #19
Review

For the submitter, initial self-review:

[x] Commented on code changes inline explain the reasoning behind the approach
[x] Reviewed the test cases added for completeness and possible points for discussion
[x] A changelog entry was added for the goal of this pull request
[x] Check the scope of the changeset - is everything in the diff required for the pull request?
This pull request is ready for:

[x] Initial review of the intended approach, not yet feature complete
[x] Structural review of the classes, functions, and properties modified
[x] Final review

For the pull request reviewer(s), this changeset has been reviewed for:

[x] Consistency across platforms for structures or concepts added or modified
[x] Consistency between the changeset and the goal stated above
[x] Internal consistency with the rest of the library - is there any overlap between existing interfaces and any which have been added?
[x] Usage friction - is the proposed change in usage cumbersome or complicated?
[x] Performance and complexity - are there any cases of unexpected O(n^3) when iterating, recursing, flat mapping, etc?
[x] Concurrency concerns - if components are accessed asynchronously, what issues will arise
[x] Thoroughness of added tests and any missing edge cases
[x] Idiomatic use of the language

Thanks for the contribution @vsaravind007 and sorry it took a while for us to look at it. I've made a minor modification in #21 which adds headers to GET and POST requests.


chore(codec): disable vp8 vp9 default

Codecov Report

Merging #90 into develop will increase coverage by 0.4%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop     #90     +/-   ##
=========================================
+ Coverage     71.8%   72.2%   +0.4%
=========================================
  Files          200     200
  Lines        10618   10618
=========================================
+ Hits          7624    7667     +43
+ Misses        2994    2951     -43

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayerDataSourceListener.h
0% <0%> (-100%)
:arrow_down:

framework/demuxer/AVBSF.cpp
65.93% <0%> (-2.2%)
:arrow_down:

framework/demuxer/avFormatDemuxer.cpp
80.87% <0%> (-1.57%)
:arrow_down:

mediaPlayer/player_notifier.cpp
81.31% <0%> (-0.55%)
:arrow_down:

mediaPlayer/player_msg_control.cpp
83.6% <0%> (ø)
:arrow_up:

mediaPlayer/MediaPlayer.cpp
70.62% <0%> (+0.21%)
:arrow_up:

framework/demuxer/play_list/HLSStream.cpp
71.05% <0%> (+0.32%)
:arrow_up:

framework/demuxer/play_list/HLSManager.cpp
73.3% <0%> (+0.39%)
:arrow_up:

framework/codec/ActiveDecoder.cpp
86.01% <0%> (+0.51%)
:arrow_up:

framework/filter/ffmpegAudioFilter.cpp
82.43% <0%> (+0.97%)
:arrow_up:

... and 3 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 f823d27...1729318. Read the comment docs.


create SriovNetworkPoolConfig and configure imagepolicy

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 40.977%

Files with Coverage Reduction
New Missed Lines
%

controllers/drain_controller.go
1
68.06%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
5588

Relevant Lines:
13637

💛  - Coveralls


fix: Prevent many cases of Tasks treating non-tag strings as tags

Description

fix: Disallow semi-colon (';') in tags
fix: Disallow [ and ] in tags
fix: Disallow 0-length tags, that is a # followed by invalid tag characters
fix: Disallow digit-only tags

The only known remaining limitation in tag-parsing is that the colour in the following is treated as a tag, as it is not understood to be inside formatting.
- [ ] <mark style="background: #FFF3A3A6;">Make phone call 10:50 today</mark>
Motivation and Context
To almost fully address #929.
How has this been tested?
Adding new tests and running existing ones.
Screenshots (if appropriate)
Types of changes

Changes visible to users:

[x] Bug fix (prefix: fix - non-breaking change which fixes an issue)

Internal changes:

[x] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

I've done some manual testing of this and it is still not as accurate as I would like.
See the attached file
Parsing Tags.md
When it is viewed with this new code, there are still multiple "tags" recognised by Tasks that Obsidian does not highlight - confirming that they are not valid Obsidian tags:


[WIP] Add SslContext validation

Motivation:
In some case, servers can start with miss-configured SslContext and
fail when users request ssl/tls connections.
If servers refuse to start in that case, users can fix their
configuration before severs start.
Modifications:
Add VirtualHostBuilder.validateSslContext()
Result:
Fixes #1844

Hello, here is my draft.
And I'm not sure I'm on the right way to go. Please see my questions.

If SSLEngine is created without exceptions, is it guaranteed that SslContext is well-configured?
I read some articles including official docs but couldn't find answer.
If not guaranteed, what configurations do I have to validate manually?
I added some test cases and it seems that most of configurations are already validated by other classes before validateSslContext() method is hit.
I cannot reproduce initial bug report in my test code. Because when I try to load KeyStore without proper key, it throws a exception, which means the server fails to start. How do I create KeyStore with invalid key?

Thank you in advance :bowing_man:

One more question,
I added JKS file to resources directory for test purpose and source code contains its password.
Is it okay with any code security issue?

@jyblue wrote:

If SSLEngine is created without exceptions, is it guaranteed that SslContext is well-configured?

I think so. If not, we can add more checks later.

I cannot reproduce initial bug report in my test code. Because when I try to load KeyStore without proper key, it throws a exception, which means the server fails to start. How do I create KeyStore with invalid key?

You have to try to reproduce the problem with 1) a PEM format private key and 2) OpenSSL enabled (which is enabled by default, but you'll have to check using Flags.useOpenSsl()).

I added dummy JKS file to resources directory for test purpose and source code contains its password. Is it okay with any code security issue?

I don't think it has any security implication if you used a self-signed certificate and if it's not encrypted with your personal password. :smile:

We have validation of created contexts here
https://github.com/line/armeria/blob/master/core/src/main/java/com/linecorp/armeria/internal/SslContextUtil.java
I think the new checks here look the same as what we do so maybe they don't help with the issue. I guess we need to reproduce the error case we first to know for sure.

@anuraaga Thanks for good hint.
I think I found the reason.
In this case, null is given to KeyStore.load() as a password parameter and KeyStore.load() didn't perform password integrity check.
https://github.com/line/armeria/blob/45c1633e1cdea43fcfc729a03739dcdb8ad0b49c/spring/boot-autoconfigure/src/main/java/com/linecorp/armeria/internal/spring/ArmeriaConfigurationUtil.java#L500

https://docs.oracle.com/javase/7/docs/api/java/security/KeyStore.html
A password may be given to unlock the keystore , or to check the integrity of the keystore data. If a password is not given for integrity checking, then integrity checking is not performed.

If steps reproduced,
KeyStore.load(keyStore, null);  // not performed password check
...
// server starts
// wait for ssl/tls request
...
KeyStore.getEntry(password); // password check, exception occurs and server fails

I will write a test case to reproduce it.

Thanks @jyblue! Looks like we are going into the right direction. Please keep us posted.

Hello, here is my research and code updated. PTAL :grinning:
Test conclusion
If key store password is not given to PKCS12 key store,
a server is built, running without exceptions, but fails when ssl/tls request come.
Internal cause
When netty.SslHandler is trying to decode client's first packet, a exception occurs.
I think SslHandler doesn't have a valid access to key store (not for sure).
https://github.com/netty/netty/blob/d8b1a2d93f556a08270e6549bf7f91b3b09f24bb/handler/src/main/java/io/netty/handler/ssl/SslHandler.java#L1329
Validation
I added validation method which tries dummy ssl/tls request,
and it produces same exception as original error.
failed to validate SSL/TLS configuration : javax.net.ssl.SSLHandshakeException: error:100000ae:SSL routines:OPENSSL_internal:NO_CERTIFICATE_SET
java.lang.RuntimeException: failed to validate SSL/TLS configuration : javax.net.ssl.SSLHandshakeException: error:100000ae:SSL routines:OPENSSL_internal:NO_CERTIFICATE_SET
	at com.linecorp.armeria.server.VirtualHostBuilder.build(VirtualHostBuilder.java:843)
	at com.linecorp.armeria.server.ServerBuilder.build(ServerBuilder.java:1432)

Note
JKS key store doesn't produce any exception when key store password is not given.
FYI
Bug reproduction and detail report
https://github.com/jyblue/ArmeriaIssueReproduction1844

Please remove the WIP status if it's ready for reviews. :wink:

You also have to release the two SSLEngines you created during validation:
try {
    ...
} finally {
    ReferenceCountUtil.release(serverEngine);
    ReferenceCountUtil.release(clientEngine);
}

Here is my update.

Rename validateSslContext to validateKeyStoreNullPassword
Because it is confused with VirtualHost.validateSslContext
Make validateKeyStoreNullPassword to be called inside of tls(KeyManagerFactory, TlsCoustomizer)
Making the validation more close to user-side, so users can easily find their miss configured key store password

It seems @Ignore annotation on testJksKeyStoreWithNullPassword() doesn't work.
I will take a look and fix it.

I fixed @Ignore to @Disabled, and ready for next review.
Thank you in advance.

Thank you for review, again. Here is my update.
I fixed items from previous review
and removed 6 deprecated method (had a chat with @trustin for this)
Please see below removed method list.

ServerBuilder.sslContext(sslContext)
ServerBuilder.sslContext(protocol, keyCertChainFile, keyFile)
ServerBuilder.sslContext(protocol, keyCertChainFile, keyFile, keyPassword)
VirtualHostBuilder.sslContext(sslContext)
VirtualHostBuilder.sslContext(protocol, keyCertChainFile, keyFile)
VirtualHostBuilder.sslContext(protocol, keyCertChainFile, keyFile, keyPassword)

Nice work, @jyblue! I've updated the PR description and commit message for you. :wink:


Maintainers wanted

We would like to get contributors who would like to maintain and evolve this library
The current maintainers, @thelinmichael and I, have been busy lately with other projects and aren't able to keep this library up to date. There are some PRs stacking up, issues that aren't getting answered, and there is no clear plan going forward on what to do with the library.
If you are interested in maintaining the library, please add a comment to this issue.

I've worked a fair bit with this library now and would be happy to maintain

@thelinmichael can you add @tigar as a maintainer?

Might be a bit late to this thread, but I'd love to help out if I can.

I've been using this library for a quite a while now. I'm also willing to help contribute in any way I can

@JMPerez @thelinmichael I would like to be a maintainer of this package. I see several open PRs and Issues submitted by others that I would like to resolve, as well as my own.


Mismatched MAC signature for Blob Batch API when / in blob path

Bug Report

import path of package in question: github.com/Azure/azure-sdk-for-go/sdk/storage/azblob
SDK version:

github.com/Azure/azure-sdk-for-go/sdk/storage/azblob v1.1.0

output of go version

go version go1.20.2 darwin/amd64

What happened?
When calling the Delete Blob Batch API with a blob path that has a /, I am getting AuthenticationFailed errors with The MAC signature found in the HTTP request ... is not the same as any computed signature. However, if I call the same API with a blob path that doesn't have a /, the request works fine. Also, if I call the single blob delete API with a / in the blob path, it works fine.

What did you expect or want to happen?
A blob with a / in the path can be deleted by the batch API.

How can we reproduce it?
I made a gist that reproduces the case, with some logs: https://gist.github.com/robherley/64e656f8326b70ecd9cafec65f9e68f4

Anything we should know about your environment.
I was unable to reproduce this in Azurite, but have tested this with multiple storage accounts on blob.core.windows.net

Placing a breakpoint at: https://github.com/Azure/azure-sdk-for-go/blob/0cda95c7a7e55361d9602a7c8a141eec584f75cc/sdk/storage/azblob/internal/exported/shared_key_credential.go#L96
The value is:
DELETE\n\n\n\n\n\n\n\n\n\n\n\nx-ms-date:Thu, 28 Sep 2023 20:38:01 GMT\n/robstorage123/go-sdk-repro/foo%2Fbar%2Ffile2.txt

Which looks correct, but I'm not sure how the subrequests escapes are supposed to be handled on the azure blob side.
Weirdly enough, I was able to get it working my commenting out this line from the client that escapes the name:
https://github.com/Azure/azure-sdk-for-go/blob/a50bf6de8d5d19367660473632b81fe35d7b2e4c/sdk/storage/azblob/container/batch_builder.go#L47
Which resulted in a stringToSign as:
DELETE\n\n\n\n\n\n\n\n\n\n\n\nx-ms-date:Thu, 28 Sep 2023 21:04:31 GMT\n/robstorage123/go-sdk-repro/foo/bar/file2.txt

And that works 🤔 which technically matches the error response that uses unencoded / characters:
<AuthenticationErrorDetail>The MAC signature found in the HTTP request '1Z4JigwM6+SBWsHckBnT/6fzoQWw5GBKZA2ULkYxNM8=' is not the same as any computed signature. Server used following string to sign: 'DELETE

x-ms-date:Thu, 28 Sep 2023 19:10:12 GMT
/robstorage123/go-sdk-repro/foo/bar/file2.txt'.</AuthenticationErrorDetail>

Hi @robherley. Thanks for reporting this bug and providing detail explanation on it.
I have created a PR to fix this. It will be part of our next release.


New cask-scripts suggestion

Hello Victor,
First off, thanks for your very handy cask-appcast and cask-check-updates scripts.
I have an idea for another script to check for dead Casks based on the idea that if the homepage is dead, then the Cask probably is also. I used this in a PR to Hombrew-Cask here.
I do not have the coding know-how to create a self-contained script like yours, so I thought it might be something you would be interested in creating.
Basically, I have the following simple url_check.sh script:
while read -r LINE; do
      read -r REP < <(exec curl -sSI "$LINE" | head -n 1)
      echo "$LINE: $REP"
done < "$1"

and I run the command
brew cask _stanza homepage > url_list.txt ;
sed -i '' 's/https/http/g' url_list.txt ;
bash url_check.sh url_list.txt > url_check.txt ;
grep -v 'HTTP' url_check.txt > dead_url.txt

which (eventually) outputs a list of (potentially) dead homepage url's for each Cask to the file dead_url.txt. The command brew cask _stanza homepage > url_list.txt lists the homepage stanza of every Cask in the file url_list.txt, for example:
http://www.sweetscape.com/
https://play0ad.com/
https://mycodingsucks.com
http://www.suavetech.com/0xed/
https://pc.115.com/
http://1clipboard.io/

which are the first five url's from the Casks, and I added the fake url https://mycodingsucks.com. The command sed -i '' 's/https/http/g' url_list.txt replaces any https's to http (so curl doesn't complain about authentication), like so:
http://www.sweetscape.com/
http://play0ad.com/
http://mycodingsucks.com
http://www.suavetech.com/0xed/
http://pc.115.com/
http://1clipboard.io/

The script bash url_check.sh url_list.txt > url_check.txt then checks each url, and outputs the following file:
http://www.sweetscape.com/: HTTP/1.1 200 OK
http://play0ad.com/: HTTP/1.1 301 Moved Permanently
http://mycodingsucks.com:
http://www.suavetech.com/0xed/: HTTP/1.1 200 OK
http://pc.115.com/: HTTP/1.1 200 OK
http://1clipboard.io/: HTTP/1.1 200 OK

and also outputs curl: (6) Could not resolve host: mycodingsucks.com to the terminal. Finally, the command grep -v 'HTTP' url_check.txt > dead_url.txt outputs each line of url_check.txt that did not return the first line of the command curl -sSI "URL" | head -n 1 (implying an error) to the file dead_url.txt, which is the required list of potentially dead Cask homepage url's, like so:
http://mycodingsucks.com:

Using this in my PR above, I obtained a list of approximately 25 url's, which I then went through and found that 20 actually corresponded to dead Casks (which I removed in 2 separate PR's), and about 5 that returned an error as curl requests were not allowed.
Now, what would be great is if it could output a file with the Cask name and homepage, much like your outdated.csv list, but I have no idea how to do that.
This is certainly not something that would run every day like your cask-check-updates script, but maybe once a week or something?
Apologies for the long post, and please feel free to ignore it if you are not interested.
Thanks!

Thank you @miccal for your suggestion and a very informative description with a great example. I really appreciate that!
This script will definitely be really useful and I will surely add it to the repository as soon as possible with your suggested behaviour and configure the server to run it every week.
You are awesome! 👍

No problem, I just wish I had the know-how to do it myself - my bash and ruby coding skills are quite amateur. I do most of my coding in Maple and Matlab!

Script cask-homepage has been added, that includes suggested behaviour. Basically, it outputs a list of all homepage URLs that might have some issues. The ones that will have an 'error' status will be the (potentially) dead casks.
The script can automatically fix homepages, but I strongly recommend to fix only those casks which definitely need some fixes by specifying casks to check.
Thanks again for your suggestion.

Wow thanks Victor, that is fantastic - you made my simple bash script look extremely amateur!
I also had the idea of listing the possible redirects, but the https availability and trailing slashes I had not thought of.
I notice that your list is quite long - I will leave to to you to submit the fixes.
In your doc/cask-homepage.md  and bin/cask-homepage files there are a few instances of slush instead of slash.
I hope yourself and the other Cask maintainers find it useful.

Thank you for noticing the slush typo. Fixed now. 👍
Yes, at the moment the list is definitely long since the script is yet immature and requires a bunch of improvements and bugs fixing. Soon the generated list will become much shorter after making few tweaks with redirects reporting since a lot of those are not necessary.
Thanks again for your suggestion, I really appreciate that!


Reboot based on Band change (or parameter)

I'm having a lot of issues with T-Mo services, and this app does help.  If I'm having network issues, I quickly look at the app to (1) see what tower I'm on [ECGI Value] and (2) what band I'm on.  If ECGI <> x, reboot.  If [BAND = B2] reboot or if [BAND <> B66] reboot.  Essentially, this is what I do manually to correct the tower or band issue, typically a single reboot gets me back into an "acceptable" level.  What do you think?

You would definitely need the ability to disable after X amount of reboots or it could  put the router into an artificial boot loop.

Closing in favor of #56


Add upstream problem list to some hotspot testcases

Close  #2712
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Grinder with jdk_jre: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1029/testReport/


chore: Enable mdsnippets tool

Description
In other projects I have used mdsnippets for embedding machine-generated files in to markdown docs, to ensure that the docs are always tested and up-to-date.
This branch is an experiment in using that on that Tasks code, and I've created the PR so that GitHub Actions will run.
Motivation and Context
To simplify the writing of user and developer documentation for this project.
How has this been tested?
It will be tested in the new GitHub Action added here.
Screenshots (if appropriate)
Types of changes

Changes visible to users:

[ ] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[ ] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (prefix: docs - improvements to any documentation content)
[ ] Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[x] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[ ] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Closed... I ended up implementing this in a few small incremental steps, culminating in #1248 - which does successfully enable mdsnippets on main now.


What is vote_data in verify_slashable_attestation?

What is vote_data referring to in verify_slashable_attestation? I don't see this field used anywhere else.

it looks like it is leftover from SlashableVoteData.  I think it should read:
        domain=get_domain(
            state.fork,
            slot_to_epoch(slashable_attestation.data.slot),
            DOMAIN_ATTESTATION,
        ),


Inaccurate Documentation (maybe)

I am using Visual Studio 2019 Profession and I cannot see the "Remove Unused References" option on a new dotnet core project / solution.
If this applies to a specific version of VS 2019 or there are other factors affecting the availability of this feature it would be useful if this article provided some detail on when this feature should be available.
Thanks,
Malachy

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

ID: 83570d18-e0b6-0097-6966-1ffd6856955e
Version Independent ID: 2a191933-b3ab-60ae-3e31-f1fb129ceefa
Content: Remove unused references - Visual Studio
Content Source: docs/ide/reference/remove-unused-references.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @mikadumont
Microsoft Alias: midumont

I just realised that this is only available for version 16.10; I was on 16.9.
I still think that this version requirement should be highlighted on the documentation.


Add support to TailwindCSS and style the web

The application is currently badly styled.
Add support to TailwindCSS and style the web

In another PoC


add a network connectivity helper

It takes the offlines and it gives them to you.
I am willing to bikeshed on the names, ps.

The code LGTM. The name installNetworkWatcher is probably too generic. It could make it sounds like the API is also watching for network connection changes ('wifi', 'cellular', etc). As the API is just for watching for offline changes, maybe we should name it that, like installOfflineWatcher?

updated to the new name


remove possibility of base-ambiguous IPv4 octets with leading zeroes

Hello!
Problem
I was noticing that it is possible to construct octets of three digits with a leading zero. In many programming languages, a leading zero in an IP address octet (or any integer) indicates this octet should be parsed in base 8, like so: 010 should be 8 in decimal. The check in this library for determining if an IPv4 octet is acceptable only looks to see if the value of the octet is < 256, and if it is comprised of three or fewer digits, but does not currently attempt to determine the appropriate base for the octet, and assumes it is decimal.
Solution
Since you currently reject octets written in hexadecimal and do not parse them with the function I updated, and additionally reject octal octets which are greater than three digits, this change seems to be in line with the intent of the library: reject all IPv4 address octets which are not written in base 10.
Why would anyone want to use base-ambiguous IP addresses with leading zeroes at all?
There's a number of times in which an attacker might wish to obfuscate an IP address they are passing to an application that wants to know if the IP is valid, part of a private range, etc. A great example of a time switching out the base for an IP address might be useful is server-side request forgery (SSRF).
Additional References

similar issue in NodeJs node-netmask where the impact is determining whether an IPv4 address is public or private would fail
node-netmask technical writeup.

While I chose not to request a CVE at this time since it mainly affects functionality which checks whether something is an IP address and not whether that IP address should be handled any particular way (which would be more of a security issue to my mind), I would be happy to do so if you feel it would help move this fix along :)

Hello Kelly,
Thank you for reporting this issue and taking the time to provide a PR. I'll merge it straight away.
Marco

Thank you @bluemonk !


External Authz fails when server in a separate namespace

Bug Description
This was discovered while working on https://github.com/istio/istio/pull/37914. Specifically while porting the TestAuthorization_Custom to the new framework.
If the external authz server is created in the same namespace as the echo apps, then it works properly (this is the way the test works today). However, if you try to move the authz server to its own namespace, I see:
    authz_test.go:1298: failed calling a (cluster=cluster-0)->'http://b.echo1.svc.cluster.local:80/custom': call failed from a (cluster=cluster-0) to http://b.echo1.svc.cluster.local:80/custom (using http): 5 errors occurred:
        	* response[0]: expected response code `200`, got "403". Response: RawContent:       [0] Url=http://b.echo1.svc.cluster.local:80/custom
        [0] StatusCode=403
        [0] ResponseHeader=Content-Length:0
        [0] ResponseHeader=Date:Tue, 19 Apr 2022 16:53:45 GMT
        [0] ResponseHeader=Server:envoy
        [0] ResponseHeader=X-Envoy-Upstream-Service-Time:1

        ID:
        Method:
        Protocol:
        Alpn:
        URL:
        Version:
        Port:
        Code:             403
        Host:
        Hostname:
        Cluster:
        IstioVersion:
        IP:
        Request Headers:  map[]
        Response Headers: map[Content-Length:[0] Date:[Tue, 19 Apr 2022 16:53:45 GMT] Server:[envoy] X-Envoy-Upstream-Service-Time:[1]]

Version
master (head)

Additional Information
No response

Do we have a Sidecar restricting visibility of it maybe

Do we have a Sidecar restricting visibility of it maybe

No, the test is pretty simple. All there is to the deployment is: https://github.com/istio/istio/blob/master/samples/extauthz/ext-authz.yaml

Could you get the logs of the external authz server?

Just to follow-up, the logs were provided off-line. You indicated that the issue is that the authz server always allows requests from app a: https://github.com/istio/istio/blob/6da7a3193970e46ebe6cf748f6411f1de1d286ec/samples/extauthz/src/main.go#L82

@nmittler I think this is a different issue (deploying authz server in a separate namespace) and not related to my answer off-line.
I couldn't reproduce this by following https://istio.io/latest/docs/tasks/security/authorization/authz-custom/ (but deploy the authz server in namespace bar and sleep/httpbin in ns foo), so I suspect it's something wrong in the test code changes.

@yangminzhu were you testing with the "local" provider on "<IP_ADDRESS>", or the regular provider?
I'm testing with the regular provider. The authz server is in the ext-authz namespace, while the echo apps are in the echo1 namespace.  My patched mesh config looks like this:
accessLogFile: /dev/stdout
defaultConfig:
  discoveryAddress: istiod.istio-system.svc:15012
  gatewayTopology:
    numTrustedProxies: 1
  proxyMetadata:
    ISTIO_META_DNS_CAPTURE: "true"
  tracing:
    zipkin:
      address: zipkin.istio-system:9411
defaultHttpRetryPolicy: {}
enablePrometheusMerge: true
extensionProviders:
- envoyExtAuthzHttp:
    headersToDownstreamOnDeny:
    - x-ext-authz-*
    headersToUpstreamOnAllow:
    - x-ext-authz-*
    includeAdditionalHeadersInCheck:
      x-ext-authz-additional-header-new: additional-header-new-value
      x-ext-authz-additional-header-override: additional-header-override-value
    includeRequestHeadersInCheck:
    - x-ext-authz
    pathPrefix: /check
    port: 8000
    service: ext-authz.ext-authz.svc.cluster.local
  name: ext-authz-http
- envoyExtAuthzGrpc:
    port: 9000
    service: ext-authz.ext-authz.svc.cluster.local
  name: ext-authz-grpc
rootNamespace: istio-system
trustDomain: cluster.local

Here is example of an authz policy for HTTP requests to one of the echo apps:
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: custom-b
  namespace: echo1
spec:
  action: CUSTOM
  provider:
    name: ext-authz-http
  rules:
  - to:
    - operation:
        methods:
        - GET
        paths:
        - /custom
        ports:
        - "18080"
        - "18085"
  - to:
    - operation:
        methods:
        - POST
        paths:
        - /proto.EchoTestService/Echo
        ports:
        - "17070"
  - to:
    - operation:
        ports:
        - "19090"
  selector:
    matchLabels:
      app: b

For which the response appears to be forbidden before it even reaches the authz server:
failed calling a (cluster=cluster-0)->'http://b.echo1.svc.cluster.local:80/custom': call failed from a (cluster=cluster-0) to http://b.echo1.svc.cluster.local:80/custom (using http): 5 errors occurred:
        	* response[0]: expected response code `200`, got "403". Response: RawContent:
        [0] Url=http://b.echo1.svc.cluster.local:80/custom
        [0] StatusCode=403
        [0] ResponseHeader=Content-Length:0
        [0] ResponseHeader=Date:Thu, 21 Apr 2022 15:00:51 GMT
        [0] ResponseHeader=Server:envoy
        [0] ResponseHeader=X-Envoy-Upstream-Service-Time:4

I'm testing with regular provider as well. Can you include the both Envoy (enabled with debug logging) and the authz server?

By the way, I just noticed you have pathPrefix: /check in the mesh config, are you using your own authz server? The example authz server does not use this path, see the example config in https://istio.io/latest/docs/tasks/security/authorization/authz-custom/#define-the-external-authorizer:
data:
  mesh: |-
    # Add the following content to define the external authorizers.
    extensionProviders:
    - name: "sample-ext-authz-grpc"
      envoyExtAuthzGrpc:
        service: "ext-authz.foo.svc.cluster.local"
        port: "9000"
    - name: "sample-ext-authz-http"
      envoyExtAuthzHttp:
        service: "ext-authz.foo.svc.cluster.local"
        port: "8000"
        includeRequestHeadersInCheck: ["x-ext-authz"]

@yangminzhu I copied the /check from the current authorization_test

@yangminzhu I copied the /check from the current authorization_test

@nmittler I see, then it should be fine, did you have the chance to get the Envoy and authz server logs?

@yangminzhu I figured out what was going on. The echos were using sidecar to restrict egress to a set of namespaces. I've got a fix here: https://github.com/istio/istio/pull/38540


Does the Mac OS version have an unread notification count?

And if not, can it get one?
I used Coherence Pro to create a stand alone app for Android Messages on my Mac, but it doesn't give me a good unread count.

Good news on this, I'm almost there! Just putting some finishing touches on and it should be released soon.

@chrisknepper you are the best!


Our website doesn't have a way to publish news

We'd like to publish news every once in awhile. We'd like to publish news with a Flutter branded site/page of some sort. Examples of news might be updates to our dev cycle, updates to our IntelliJ plugins, big new features, etc.
These would be items bigger than a tweet, and more timely/announcement than a pure document.

From past experience one thing I'd be wary of with this is that you end up with pages that are stale. e.g. don't publish news for 2 months and now it looks like you've been out to lunch for 2 months.

We don't necessarily want to set up a whole blog: not sure how often we'll publish, and not excited to manage yet another site.
Perhaps we can use flutter.io's existing infrastructure, and add a "/news" subdirectory.

don't publish news for 2 months and now it looks like you've been out to lunch for 2 months

Agreed. We need to ensure we can publish at some reasonable schedule.
An idea for a solution is to just email the mailing list whenever we have something to say.

cc @mit-mit

An idea for a solution is to just email the mailing list

How many subscribers are there of that list?

361 members

Issue moved to flutter/website #1835 via ZenHub


Update README.md with struct compilation from Rust to Typescript example

I've just discovered this nice crate. There is one thing that made me pause, there is no example of Typescript that the crate would generate.
My suggestion / request is adding the example below in the README.md (I'm taking the current content of the section and add what I would like to add) :

use ts_rs::TS;

#[derive(TS)]
#[ts(export)]
struct User {
   user_id: i32,
   first_name: String,
   last_name: String,
}

When running cargo test, the TypeScript bindings will be exported to the file bindings/User.ts.
interface User {
   userId: number,
   firstName: string,
   lastName: string,
}

Something like that.

Maybe we can commit the bindings generated by the example?

Yes it would be great !


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1145 ms

Resolved: Bofrost.at performance has improved in 371b8d3 after 36 minutes.


Bugfix/knn validation

knn: Fix input validation issues

Some input validation was missing which is included now.
there was still a bug wrt ambiguous references to column names,
which was only partly handled previously.

We should remove the hardcoded schema names from the tests as well. If MADlib is installed in a different schema, they might create problems.

knn.sql lines 404 - 407 are not necessary anymore thanks to the expanded validation function.


Expose min/max port range for tx5 configureation

Thank you so much @neonphog -- this just adds onto your work to expose the configuration all the way up to the main tx5 config, which I'll then add to the conductor's network tuning params config

the only other concern I have is setting the default min & max ports to 1 and 65355 - I believe the allowed ephemeral ports are different for every OS and wouldn't want pion to try to grab a port that is not permitted. I'll see if I can find what its is doing under the hood and if it's impacted.


Unify net and regular container pull code

(aka "Avoid race between image pull and garbage collect - take 2")
Rewrite the net and non-net container pull paths to use a new
pullAndRunContainer function.  The key feature of this new functions is
that it takes and holds the PullLock until the new image is referenced
via a running container and thus protected from garbage collection.  The
previous version of this fix acheived a similar result through
uglier/duplicated code.
This change improves on the previous version by:

Releasing the pullLock earlier when pulling+running non-net
containers.

Rewriting the net pull codepath to use the same routine as non-net
containers.  There should be no functional change for net containers,
just less code duplication.

The change looks good, but I am obligated to ask about testing.  As in there is none.  Is this path covered by some test I am missing?  If not, can we think about how to ensure this stays in working order?

I think that this is the right re-factor, but I have some comments/concerns on the lock placement.

/cc @dchen1107

@anguslees  Thanks for your PR to help us improving our system. Like what we discussed before, Image GC is using "dangling=true" from docker to determine if a image is used or not, which is wrong. We already remove the current implementation of Image GC, and this PR is not required anymore. I am going to close this one. If I mistake anything from this PR, please send us a new one. Thanks again!


Creation of signup/signin page on frontend

Description:
There is no frontend folder for the site on this so will be initializing one and will be creating UI for the signup signin route.
It will be connected to the backend
BENEFITS:
User can register themselves
Would make it easier to login
Assign this issue to me under GSSoc'2024

can you assign to me this issue


Minor fixes

Minor stuff I hit while glossing over some docs...
Some other thing that bothered me a bit (made me stop and re-read a few times) is the compound noun "pipeline_s_ file(s)" instead of "pipeline file(s)" (pipelines plural), but I ... guess it's valid...? 🤔

p.s. sorry for the delay @omesser . Assigned you for visibility in https://github.com/orgs/iterative/projects/284.

Hey @jorgeorpinel,
Pushed some changes and answered some comments - please do another pass when you get a chance

Thanks @omesser !


How to change search engine

Describe the bug
.
How to Reproduce
.
Expected behavior
.
Environment
- DDG App Version:
- Device:
- OS:

Hey @Gamer750 I don't think this issue can be labeled as a bug. Maybe you can request this as a feature request.
Also after seeing your issue #4511, I went to DuckDuckGo android browser and searched for search engine change option, I was unable to see the option to change.

Thank you for reaching out with your question. Our DuckDuckGo browser is intentionally designed to use DuckDuckGo as the default search engine to ensure privacy for our users. As a result, the option to change the search engine is not available within our browser.

I've just created a fork of DuckDuckGo that allows to change the default search engine
Feel free to check it out or add your preferred search engine via a PR - https://github.com/lippfi/SearxDuckGo

@lippfi : this is great, thank you! However, not everyone is a developer and can add their own search engine. Any chance you could make it configurable so you can enter the search url?


[kvm] Fix network parameters generation.

Fixes 1290.

LGTM

Thanks!
The Semaphore failure was unrelated. See #1287.


Fix support for MTK devices

Currently most of the MTK devices are facing problems due to these issues :

[ ] Current size of the recovery exceeds 16mb partition which most of the MTK devices have, PBRP_GO was made for this but still it is not as perfect as possible.
[ ] Most of the MTK devices support older android versions (5.x-7.x), its been a while since other branches are regularly updated (They may face build errors also).
[x] Flashlight in many MTK devices have different path than Usual Qualcomm Devices, and often use single file path for controlling torch on/off. So, the flashlight trigger may not work in many devices.
[ ] More coming soon...

third issue should be fixed with last commit


Cannot create an HTTPRoute to ServiceExport/Import when underlying k8s service has the same name across multiple clusters

flowchart TD
subgraph config-cluster["configcluster
 "];
    HTTPRoute --> service-X-import
    HTTPRoute -..-> service-X-import
end
subgraph cluster2["cluster2
 "];
    service-X-import --> service-X-export --> service-X --> pods-X
end
subgraph cluster1["cluster1
 "];
    service-X-import --> .service-X-export -->.service-X --> .pods-X
end

Precondition
As presented on visual, we have 2 separate clusters: cluster1 and cluster2. It's not uncommon to have the same service with the same name to be deployed into multiple clusters to achieve HA/DR capabilities. Note that service name is the same in both cluster - specifically "service-X".
Use case
We want to have an HTTPRoute in a "config" cluster to route requests evenly to service-X in both clusters. To achieve that, we create ServiceExports in cluster1 and cluster2. Note that current implementation would look at ServiceExport-s names to map them to respective Services in the cluster. Then we want to create ServiceImports in a "config" cluster. Note that current implementation would look at ServiceImport names to map them to respective ServiceExports from cluster1 and cluster2. Then we want to create an HTTPRoute with 2 backendRefs - one for ServiceImport of service-X from cluster1 and one for ServiceImport of service-X from cluster2. It would look similar to
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: route-to-service-X-accross-clusters
spec:
  parentRefs:
    - name: service-network-name
      sectionName: http[s]
  rules:
    - backendRefs:
        - name: service-X-import
          kind: ServiceImport
          port: 80
        - name: service-X-import
          kind: ServiceImport
          port: 80

Problem
Because resource names are used to map to actual services, we have a conflict in the routing manifest with two identical service imports representing the same service-X deployed to multiple clusters 1 and 2.
Potential solution
Use annotations on resources to map to Services and fallback to resource names (for backward compatibility). The manifests would look like:
---
apiVersion: application-networking.k8s.aws/v1alpha1
kind: ServiceExport
metadata:
  name: service-X-export-1
  annotations:
    application-networking.k8s.aws/federation: "amazon-vpc-lattice"
    application-networking.k8s.aws/service-name: "service-X"

---
apiVersion: application-networking.k8s.aws/v1alpha1
kind: ServiceExport
metadata:
  name: service-X-export-2
  annotations:
    application-networking.k8s.aws/federation: "amazon-vpc-lattice"
    application-networking.k8s.aws/service-name: "service-X"

---
apiVersion: application-networking.k8s.aws/v1alpha1
kind: ServiceImport
metadata:
  name: service-X-import-1
  annotations:
    application-networking.k8s.aws/export-name: "service-X-export-1"
spec:
  ports:
    - port: 80
      protocol: TCP
  type: Headless

---
apiVersion: application-networking.k8s.aws/v1alpha1
kind: ServiceImport
metadata:
  name: service-X-import-2
  annotations:
    application-networking.k8s.aws/export-name: "service-X-export-2"
spec:
  ports:
    - port: 80
      protocol: TCP
  type: Headless

---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: route-to-service-X-accross-clusters
spec:
  parentRefs:
    - name: service-network-name
      sectionName: http[s]
  rules:
    - backendRefs:
        - name: service-X-import-1
          kind: ServiceImport
          port: 80
        - name: service-X-import-2
          kind: ServiceImport
          port: 80

This is where mapping between Service inside target group and ServiceExport is hardcoded: https://github.com/aws/aws-application-networking-k8s/blob/main/pkg/gateway/model_build_targetgroup.go#L177.
Potentially this might be the only place where changes need to be done. If we decouple Service and ServiceExport, the rest becomes unblocked autometicaly.


Fix circular require warnings

Place a conditional on the google/cloud require.
This will stop the circular require warnings when gems are
loaded by the google-cloud-core auto-load mechanism.

Warnings are currently disabled on all the tests, partially because there are many warnings in dependencies that we can't address. But also partially because of these circular require warnings. Resolving the circular require warnings will get us one step closer to being able to enable warnings on our test runs.

@dazuma I would really like your input on this. WDYT?

Does this syntax work? It doesn't seem to work for me:
% irb
irb(main):001:0> module Foo
irb(main):002:1> def foo
irb(main):003:2> end
irb(main):004:1> end
=> :foo
irb(main):005:0> defined? Foo.foo
=> nil

Oh, never mind, I forgot def self.foo...

Yeah, it works. It is safer to check for .new than it is to check for the module to exist, because gemspec loads the version and that creates the namespace. Even checking for Google::Cloud::Core wasn't as reliable as Google::Cloud.new.

Yeah, this seems fine, and it seems to work in my tests. I wonder if it would be better to create a constant like Google::Cloud::REQUIRE_MARKER or something specifically for this purpose so that it's more obvious what we're doing.

That said, this is just one way to guard against the circular require warnings. I'm open to suggestions on other approaches.

What is nice about this is that it doesn't require any changes to google-cloud-core, so we don't mess with that dependency at all.

Oh, good point. Yes then checking for .new sgtm.

Remove me from everything immediately or I will report ypu
On Thursday, September 13, 2018, Mike Moore<EMAIL_ADDRESS>wrote:

Place a conditional on the google/cloud require.
This will stop the circular require warnings when gems are
loaded by the google-cloud-core auto-load mechanism.

You can view, comment on, or merge this pull request online at:
https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2441
Commit Summary
Fix circular require warnings
File Changes
M google-cloud-bigquery/lib/google-cloud-bigquery.rb (2)
M google-cloud-bigtable/lib/google-cloud-bigtable.rb (2)
M google-cloud-datastore/lib/google-cloud-datastore.rb (2)
M google-cloud-debugger/lib/google-cloud-debugger.rb (2)
M google-cloud-dns/lib/google-cloud-dns.rb (2)
M google-cloud-error_reporting/lib/google-cloud-error_reporting.rb (2)
M google-cloud-firestore/lib/google-cloud-firestore.rb (2)
M google-cloud-logging/lib/google-cloud-logging.rb (2)
M google-cloud-pubsub/lib/google-cloud-pubsub.rb (2)
M google-cloud-resource_manager/lib/google-cloud-resource_manager.rb (2)
M google-cloud-spanner/lib/google-cloud-spanner.rb (2)
M google-cloud-storage/lib/google-cloud-storage.rb (2)
M google-cloud-trace/lib/google-cloud-trace.rb (2)
M google-cloud-translate/lib/google-cloud-translate.rb (2)
M google-cloud-vision/lib/google-cloud-vision.rb (2)
M google-cloud/lib/google-cloud.rb (2)
Patch Links:
https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2441.patch
https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2441.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@royrobertrobert1974 If you are subscribed to updates and you don't want to be, then go to the repository URL (e.g. https://github.com/GoogleCloudPlatform/google-cloud-ruby/ in this case) and click "unwatch" near the top right. We can't do that for you. It's your preference setting.

@royrobertrobert1974 You are watching this project. We didn't add you, you added yourself. Go to the main page of the project, click "Unwatch", and choose a new notification level.
https://github.com/GoogleCloudPlatform/google-cloud-ruby

loses to @dazuma again


"Optimal Denomination" problem of codechef

Aim
I want to add the code for "Optimal Denomination" problem of codechef (July Challenge 2021)
Contest ended on July 12th
Details
Problem name : Optimal Denomination
Problem Code : MINNOTES
Problem link : https://www.codechef.com/JULY21C/problems/MINNOTES
Please assign me this issue
LGM-SOC Participant
Programming language
C++

[ ] C
[x] C++
[ ] Java
[ ] Python

@Amit366 @ravikr126 @Ayush7614 Please assign this issue to me under the label LGM-SOC

@Amit366 @ravikr126 @Ayush7614 Please assign this issue to me under the label LGM-SOC for python


Add command for ARM based chip.

There is an example of command for ARM at the beginning but the second command is missing it.

Proposed changes

Related issues (optional)

Hi @kirhist, thanks for sharing the issue.
There is some issues with the formatting and the flag is not necessary here. For details, see https://github.com/docker/docker.github.io/issues/14517

Hi @craig-osterhout . Without this flag container fails to start with some Python dependency errors. If platform flag is undesirable option then some other options needs to be added.

@kirhist, thank you, I missed that the command hasn't been updated like the previous section.
If you use the following command, do you still get dependency errors? If yes, can you share the error output.
docker run -dp 3000:3000 \
   -w /app -v "$(pwd):/app" \
   --network todo-app \
   -e MYSQL_HOST=mysql \
   -e MYSQL_USER=root \
   -e MYSQL_PASSWORD=secret \
   -e MYSQL_DB=todos \
   node:12-alpine \
   sh -c "apk add --no-cache python2 g++ make && yarn install && yarn run dev"

@kirhist, thank you, I missed that the command hasn't been updated like the previous section. If you use the following command, do you still get dependency errors? If yes, can you share the error output.
docker run -dp 3000:3000 \
   -w /app -v "$(pwd):/app" \
   --network todo-app \
   -e MYSQL_HOST=mysql \
   -e MYSQL_USER=root \
   -e MYSQL_PASSWORD=secret \
   -e MYSQL_DB=todos \
   node:12-alpine \
   sh -c "apk add --no-cache python2 g++ make && yarn install && yarn run dev"

This new command works, thank you.

Thanks for getting back. Do you want to update your PR, or should I close it and open a new one to fix it?

Thanks for getting back. Do you want to update your PR, or should I close it and open a new one to fix it?

I updated this PR.


Update jupyter lab services to latest version

Fixes #7675
Just the jupyterlab portion. Widgets is still old version. Webpack is still old version. Those can wait till next debt week.

Codecov Report

Merging #7872 (662a437) into main (630af69) will decrease coverage by 0%.
The diff coverage is 59%.

:exclamation: Current head 662a437 differs from pull request most recent head 726ea91. Consider uploading reports for the commit 726ea91 to get more accurate results

@@          Coverage Diff          @@
##            main   #7872   +/-   ##
=====================================
- Coverage     70%     69%   -1%
=====================================
  Files        361     361
  Lines      22153   22182   +29
  Branches    3357    3366    +9
=====================================
- Hits       15585   15519   -66
- Misses      5211    5309   +98
+ Partials    1357    1354    -3

Impacted Files
Coverage Δ

src/client/datascience/commands/exportCommands.ts
34% <ø> (ø)

src/client/datascience/common.ts
72% <ø> (ø)

...ient/datascience/editor-integration/codewatcher.ts
68% <ø> (ø)

...rc/client/datascience/errorHandler/errorHandler.ts
58% <ø> (ø)

src/client/datascience/export/exportUtil.ts
54% <ø> (ø)

...ience/interactive-common/notebookServerProvider.ts
68% <ø> (+4%)
:arrow_up:

...atascience/interactive-window/interactiveWindow.ts
56% <ø> (+1%)
:arrow_up:

...cience/jupyter/jupyterCellOutputMimeTypeTracker.ts
77% <ø> (ø)

src/client/datascience/jupyter/jupyterDebugger.ts
19% <ø> (ø)

src/client/datascience/jupyter/jupyterExporter.ts
33% <ø> (ø)

... and 64 more


Entities becoming blacklisted

Minecraft v1.16.5
Forge v36.1.4
Menu Companions v1.2.3
I have recently noticed that mobs and the players are being added to the blacklist, and I suspect this is because of the Hats and Googly Eyes mods, since all the mobs that get added to the blacklist can have googly eyes or hats. Mobs that don't have hats or googly eyes render as normal.
There is no error printed out in the logs.

Minecraft v1.16.5
Forge v36.2.34
Menu Companions v1.2.3
Hats v10.3.3
Can confirm that Hats mod conflict with Menu Companions


Workspace info for debugging

Consider the following use case: I'm debugging a workspace scope problem, and want to know exactly what workspaces are currently open at any given point. Not only the currently active workspace (if any), but also any other workspaces that it is nested in.
If I have a reference to a workspace (and/or its name) I can check if it's open/active - but if I don't have one of those, I don't think there's a way to see what workspaces are open.
So, if have have: x.notifyScopeEntered(), y.notifyScopeEntered(), <determine here>, x.close(), y.close()
where <determine here> would let me know that y is active, and x is open external to y

Added here: https://github.com/deeplearning4j/nd4j/pull/2597


[Dev Task] Add a compile expression task

AC

[ ] Supports Ligo
[ ] Supports SmartPy
[ ] Command can handle CLI pipes and redirects
[ ] Annotation aware
[ ] Default to compiling to a file

Notes from the discussion:

taq compile-expression [expression goes here] - file as input, or Ligo expression
taq compile-storage
taq compile-expr | taq call [name] [entry]
taq compile-expr < input.jsligo | taq call [name] [entry]

Handling in #1166


Visual Entailment dataset?

Could you give me a Visual Entailment dataset download link?

Here: https://github.com/necla-ml/SNLI-VE


Conditioning on protein of different length than protein to be designed

Hello and great job on the tool! I'm trying to make a substructure conditioner based on one region of protein A and apply that to a similar region of protein B. However, because the proteins A/B have different numbers of atoms (I specified the same number of residues), I keep getting an error relating to tensor mismatch. Is there an intelligent way to pad/trim the pdbs so that I can do this?
Thanks!

Hello! Just wanted to ask again if there's any guidance I could get here.


91 currency pallet

Makes CurrencyId available from other modules
Uses orml_tokens and traits

Closes  #91

It is weird that so many files have conflicts, I merged dev/mileston-3 a short time ago. I will solve them and re push,

I thought it was because it was based on the main branch and that some changes there were not contained on the dev branch yet. But I now merged main to dev/milestone-3, and it didn't change anything. I also tried force-pushing the last commit again but this also didn't help...

Ready to merge from my side.


Feature / Betting Price Lock

Description

Type of change
Please check the options that are relevant. This PR introduces

[ ] Patch: Bug fix (non-breaking change which fixes an existing issue)
[ ] Minor: New feature (non-breaking change which adds new functionality)
[ ] Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Features Description

[ ] NA

Test Cases

[ ] NA

Documentation

Checklist:

[ ] I have added change type (major|minor|patch) in the PR
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (feature/upgrade-go@c4f3ec1). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                  Coverage Diff                  @@
##             feature/upgrade-go     #349   +/-   ##
=====================================================
  Coverage                      ?   11.70%
=====================================================
  Files                         ?      351
  Lines                         ?    45237
  Branches                      ?        0
=====================================================
  Hits                          ?     5296
  Misses                        ?    39456
  Partials                      ?      485

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Question: Custom UTLS Hello Specifications + the HTTP Transport Bridge?

Hello! I'm doing a little research into using this project, and while it seems to solve a problem I'm running into quite well, one limitation I'm running into is that UTLS's named Hello spec IDs aren't necessarily up-to-date, but to use a customized one (with a custom ClientHelloSpec) requires calling ApplyPreset directly onto an active connection, which is not something it appears this bridge exposes at all.
Perhaps it's just a rookie misunderstanding on my part, but can you help me understand if (and how) it's possible to implement custom client hello specifications while using this bridge?
Thank you so much!

Dear @SlvrEagle23,
thank you for your interest in this proof of concept! As OONI, we are going to use another strategy regarding how to integrate the uTLS library along with a net/http client. We have forked the stdlib net/http tree here https://github.com/ooni/oohttp and we are going to use this modified client along with uTLS for our measurements.
Regarding your question proper, I think it is possible to create a custom TLS ClientHello fingerprint and then set the UHTTPtransport.UTLSClientHelloID. The following link provides example code that shows how to do that: https://github.com/bassosimone/tlsfingerprint/blob/master/main.go. At the time when I wrote the code, the fingerprint in there was IIRC a very popular Chrome fingerprint.


add initialization for d_a

The variables "d_a" are not initialized, so the result during reduction is uncertain and may overflow to a negative number, causing the test to fail.

@ManjulaChalla Could you help to review and merge this PR when free, thanks.

Is there any reason this is fix is supposed to be merged into the master branch instead of development?

Sorry, I'm not familiar with the commit rules for this repository, should I submit a PR to the development branch?

Changed branch which I want to merge the PR into from master to development, see the PR.


Redirect screen contains fields, button visible to user

only visible for a short time, but would be nice if this is hidden

@SonnyBA or @Bartvaderkin , if one of you wants to investigate within the context of your current project (VIPS or OIP) feel free.

Discussed with @joeribekker and attempted on OIP acc ("
Note: CSP restrictions forbid inline styling so this isn't possible within the template ( https://github.com/maykinmedia/django-digid-eherkenning/blob/master/digid_eherkenning/templates/digid_eherkenning/post_binding.html ).
This will require a separate css file in /static/ to hide the form.

Maybe for non-frontend only apps, we an make a login form/button that directly performs the post to Logius instead of the separate screen?

Maybe for non-frontend only apps, we an make a login form/button that directly performs the post to Logius instead of the separate screen?

Having skimmed through the spec and SAMLv2 with POST binding, I don't think that's even allowed :grimacing:

If I look at mijn.haarlem.nl, they transition smoothly from login button to DigiD page without any intermediate screen.

@joeribekker I suspect they use a different binding then, see https://en.wikipedia.org/wiki/SAML_2.0#SAML_2.0_profiles

Can confirm, the removal of this screen can be achieved with a different binding (redirect instead of post).


Terminal: When typing in the integrated terminal, the cursor goes behind the header (which points to the previous command)

Does this issue occur when all extensions are disabled?: Yes

Version: 1.86.0-insider (user setup)
Commit:<PHONE_NUMBER>4e4c0fddbc1b3b728013744ae90840
Date: 2024-01-15T05:36:06.405Z
Electron: 27.2.1
ElectronBuildId: 26149897
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

Steps to Reproduce:

Open new Terminal in any folder containing files.
type command ls to list the files, any command works.
Once the tree is built up, run the clear or cls command.
Now the cursor will be pointing behind header.

Image:

Video:
https://github.com/microsoft/vscode/assets/38991749/1f86a26e-ddcb-44b9-84e2-6c6b2865caee

Root cause https://github.com/microsoft/vscode/issues/166864
/duplicate


libtensorflow libraries for >2.15

Issue type
Documentation Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.16
Custom code
Yes
OS platform and distribution
Windows Linux
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Is libtensorflow still being built for every version?
According to the documentation here it is, but according to the GCS bucket in that page which links to the bucket, there hasn't been any new builds for two years.
The last official version available is 2.15
Standalone code to reproduce the issue
No code involved to reproduce

Relevant log output
No response

Hi, @MattiasDC
I apologize for the delayed response, Thank you for reporting the issue regarding Nightly libtensorflow C packages availability. I see in this official documentation the status of Nightly Binary Official GCS marked as Status Temporarily Unavailable but we do have libtensorflow packages for stable version of TensorFlow available so at the moment I would suggest you to please go with stable libtensorflow C packages which you can download and extract it for more information please refer this official documentation
For future update regarding Nightly libtensorflow C packages availability you can see the status here : https://github.com/tensorflow/tensorflow?tab=readme-ov-file#official-builds
Thank you for your cooperation and patience.

Hi, @MattiasDC
Could you please confirm if this issue is resolved for you now, For future update regarding Nightly libtensorflow C packages availability you can see the status here : https://github.com/tensorflow/tensorflow?tab=readme-ov-file#official-builds ?
Please feel free to close the issue if it is resolved ? Thank you

Thanks for all the info!


automated doc build

This PR provides support for automated documentation via pydoc, which seems to be a particularly lightweight and well maintained solution for automatic documentation.

To do: module-level docstrings, which are rendered prominently.

The two points above are good, and we should keep the discussion going, but shouldn't delay merging to test the doc build.


Alerta.io Integration

Implements integration with alerta.io
The code has been tested with a local instance of alerta.
Usage:
stream
	.from().measurement('cpu')
	.where(lambda: "host" == 'serverA')
	.groupBy('host')
	.window()
		.period(10s)
		.every(10s)
	.mapReduce(influxql.count('idle'))
	.alert()
                .id('{{ index .Tags "host" }}')
		.message('kapacitor/{{ .Name }}/{{ index .Tags "host" }} is {{ .Level }}')
		.info(lambda: "count" > 6.0)
		.warn(lambda: "count" > 7.0)
		.crit(lambda: "count" > 8.0)
		.alerta()
			.token('testtoken1234567')
			.resource('serverA')
			.event('CPU Idle')
			.environment('production')

Configuration:
[alerta]
  # Configure Alerta.
  enabled = false
  # The Alerta URL.
  url = ""
  # Default authentication token.
  token = ""
  # Default environment.
  environment = ""
  # Default origin.
  origin = "kapacitor"

[ ] CHANGELOG.md updated
[ ] Rebased/mergable
[x] Tests pass
[x] Sign CLA (if not already signed)

@md14454 Looks great. Just a few things

Can you run go fmt on the changes, there are several formatting errors.
Can you add another alerta handler in the test with different options? Showing that the defaults can be overridden and that multiple handlers can be configured. something like

.alerta()
 			.token('testtoken1234567')
 			.resource('serverA')
                        .event('CPU Idle')
                        .origin('override')

Again this looks great, thanks for the PR!

@nathanielc I've updated the alerta code to select severity and status based on the Kapacitor level, as per your request. The CI tests seem to fail on my local machine and they previously did on circleci both due to a timeout when pulling boltDB from github; not sure why. Running build.py --test seems to work locally however.

@md14454 This looks good to go thanks. Can you go fmt the code and rebase? Then I'll merge. Thanks.
Don't worry about the tests, I have a few that are prone to error that I have been trying to chase down.

@nathanielc go fmt  didn't change anything and git rebase master comes back saying that the branch is up to date. Hopefully that means we're good to go; or I've missed something.

@md14454 Sorry I should be more specific.
You will need to run go fmt ./... in order for it to recursively format the sources.
And by rebase I meant rebase and squash all of your commits into a single commit. Here is an explanation of what I mean http://makandracards.com/makandra/527-squash-several-git-commits-into-a-single-commit Either method there works, I personally use the hard way because I find it easier and less prone to error.

@nathanielc That should be done now. Apologies I haven't coded in golang or used a rebase workflow before.

@md14454 no problem. Thanks again.


Change fgColor on hover

As far as I have read there is no option to change fgColor on hover. And it is difficult to change using css because fgColor is already given from .js file and its the top priority.

Same. Was hoping hover functionality could be added.

Not the most ideal solution but I got it working like this:
.social:hover :global(.social-svg-icon) {
    fill: #D95004 !important;
}

import styles from './social.module.css';
...
<SocialIcon className={styles.social} ... />

I'm using CSS modules. If you're not, tweak it appropriately.

In case others have this issue--just set fgColor to "currentColor". Easy peasy.

thank you for your contributions. I've update the README with your suggestions: https://github.com/couetilc/react-social-icons#how-do-i-change-the-color-on-hover


is downloading a video from ICloud Supported ?

I am using ICloud to store my old videos in camera roll. I don't see a method for getting video from ICloud. I only see cloudImageDownload(progressBlock: @escaping (Double) -> Void, completionBlock:@escaping (UIImage?)-> Void ) -> PHImageRequestID?
Please let me know how I can download cloud video.

@specc
Apple Document : request of video asset
https://developer.apple.com/documentation/photos/phimagemanager
https://developer.apple.com/documentation/photos/phimagemanager/1616981-requestexportsession

ah ok I will try it out. Is there a flag that will tell me whether a video is from iCloud or not ?

@specc
Use this.
public func tempCopyMediaFile(progressBlock:((Double) -> Void)? = nil, completionBlock:@escaping ((URL,String) -> Void)) -> PHImageRequestID?
76a280e50a751df370a0a979fbbd7b3f9cda281d


Create a cloneObject

you must clone the Object like that. If you try to clone it like the following
object = {bla: bla}
objectClone = object
and then try to edit it like
objectClone.bla = "blub"
You would also edit the real Object

Hey, @ArvedE I am closing this since it's a duplicate of functions we already have on the website. deepClone and shallowClone
Thanks for the effort though!


NuGetDocs Webhooks failing

The webhooks in the NuGetDocs repository are failing to publish to the NuGetGallery on a regular basis.

@csharpfritz can you provide me with admin access on that repo? Can't look into it currently.

I've disabled two webhooks that didn't look correct. If I'm right, we can delete them.
Not sure why the other webhook is showing as failed on GitHub, as the azure portal shows deployment got triggered, and succeeded.

Have updated the web hooks and verified they now work. @csharpfritz please keep an eye on it during the next docs updates - I expect it to be good now.

Latest updates have always returned green web hook calls. Closing issue.


[SPARK-41711][BUILD] Upgrade protobuf-java to 3.21.12

What changes were proposed in this pull request?
This pr aims upgrade protobuf-java from 3.21.11 to 3.21.12
Why are the changes needed?
The release notes as follows:

https://github.com/protocolbuffers/protobuf/releases/tag/v21.12

Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass Github Actions

Merged to master.

Thanks @srowen @HyukjinKwon


⚠ Bofrost.lu has degraded performance

In c88bf74, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1401 ms

Resolved: Bofrost.lu performance has improved in 940af47 after 6 minutes.


Comment?

Care to comment on the assertion made in https://github.com/princeton-nlp/tree-of-thought-llm#readme ?

Hey I implemented my own implementation of the paper, it blew up because people wanted to use it. They're mad they their repo didn't blow up because they didn't release the code for it 2 weeks. They want me to link their repo and are trying to force me even though I took absolutely 0 of their code. Implemented the entire paper based on 4 phrases, dfs, bfs, generate thoughts, evaluate thoughts


Numalogic v0.4a1

Codecov Report

Merging #131 (67d6e83) into main (7cada11) will decrease coverage by 1.07%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
- Coverage   83.55%   82.49%   -1.07%
==========================================
  Files          20       20
  Lines        1107     1137      +30
  Branches      121      116       -5
==========================================
+ Hits          925      938      +13
- Misses        157      177      +20
+ Partials       25       22       -3

Impacted Files
Coverage Δ

numaprom/tools.py
83.33% <ø> (-2.48%)
:arrow_down:

numaprom/udf/threshold.py
91.66% <66.66%> (-8.34%)
:arrow_down:

numaprom/udf/preprocess.py
88.63% <68.75%> (-11.37%)
:arrow_down:

numaprom/udf/inference.py
92.85% <72.22%> (-2.53%)
:arrow_down:

numaprom/udsink/train_rollout.py
85.36% <84.21%> (+1.15%)
:arrow_up:

numaprom/clients/sentinel.py
88.88% <100.00%> (ø)

numaprom/udf/postprocess.py
95.55% <100.00%> (ø)

numaprom/udf/window.py
95.08% <100.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove gitvalidation from travis

gitvalidation in travis has been problematic.  It is already
being done in Fedora and Centos so we have a reliable test
occuring there.
Signed-off-by: baude<EMAIL_ADDRESS>

@rhatdan @mheon ptal and merge

LGTM, it's failed twice in two days. Not worth the hassle.

@rh-atomic-bot r+


Shinigami: Fix cloudflare + pageListParse

Closes #334
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Remove the trailing space on Shinigami.kt. https://github.com/keiyoushi/extensions-source/actions/runs/7573763740/job/20641620721?pr=354#step:7:185


Can I use this on Windows?

Yes, as libtensorflow is available on windows you should be able to use the cppflow wrapper


Bump version to 0.2.18

Why
Includes revert to optional oneof change: https://github.com/replit/river-python/pull/47
What changed
https://github.com/replit/river-python/pull/47
Test plan

Current dependencies on/for this PR:

main:

PR #49 👈

This comment was autogenerated by Freephite.


Should decommission all the /rt/ aliases

... since we don't use rt anymore.
Could  replace them all with a link to github issues for rakudo

I think they were in an old apache.conf just deleted by @moritz  - but are linked to on https://rakudo.org/issue-trackers


Welcome Screen Passwords

All key generation methods rely on a password provided. There is no password field yet.

Add to "create new account"


Combine definition with related words in the definition popup

The definitions are now split into the definition and the related words. The definition popup should combine the two and the quiz screen should only use the definition.

Please add your planning poker estimate with ZenHub @Fattimo


Black / Strange Glitchy Terrain with Shader Patch 1.4.0

This happens with certain maps I don't know why. The one in the example is Toola: Frozen Tundra but this glitch affects certain other maps too.
For the glitch to occur, you have to be playing other maps first before selecting the affected map. Exiting and reopening the game, then playing the affected map first is my current workaround at the moment.

Thanks for letting me know. Is this https://www.moddb.com/mods/filefronts-hidden-gems/downloads/toola-frozen-tundra the correct map to debug on?

Yeah that's the map that I played on.

Genosis (G31) By YaNkFaN & a_speck_of_dust is another example.
Game bugged after playing other maps:

What the games is supposed to look like:


Add multiple power domain design in regression

Add a design in regression that would utilize the feature added in #1175
This design should have an internal macro that doesn't have the same power and ground names as the top level macro. Preferably has multiple power domains

We have caravel_upw now. However I think we should keep this and include more complex PDN examples


Logged out of Mauro Test Environment

Logged out of Mauro twice, the first whilst actively using the system (0 inactivity time).
I was creating an attribute term at approx. 15:00

created a new term on Attribute Terminology S CATEGORY CODE
closed the Attribute Terminology folder
clicked on + Data Element CodeSets
system was unresponsive
after about 30 seconds - clicked on + Data Element CodeSets again
logged out - back to the home screen

The second time there was an inactivity period of about 19 minutes (while I discussed this issue) at approx. 15:24.

Kate asked Naresh to chase this on 18/9/2024.

I am checking with @jamesrwelch about this. Mauro login sessions having a timeout feature is more general functionality that does not necessarily relate just to the Data Dictionary.

If the system was unresponsive, then it suggests something else was going on, but without seeing the logs, I don't know.  Is this a regular occurrence, or was it just a one-off?
I believe the default time-out is 10 minutes, although I'd need to check.  This is the time period of inactivity before sessions are expired - it is configurable per instance.

I haven't had this issues since, but will continue to monitor it.


More documentation

Especially of the function arguments. Should be automatically deployable...

Closing this because it seems that a good README seems to be Julia standard for good documentation.


Improve bci-init tests

Improve the systemd tests for the bci-init container by adding multiple
new tests for boottime, journald, timedatectl, loginctl, hostnamectl.
Related ticket: https://progress.opensuse.org/issues/111054

I don't know why the pipeline is complaining about the code not being black formatted, IMHO it is:
./format.sh --check
All done! ✨ 🍰 ✨
29 files would be left unchanged.

purge your .tox directly and run tox -e format

@grisu48 please add tox -e format --check to your pre-push git hooks :joy:

Did that while you have been complaining ;-) Should be good now, after the pipeline passes

@jlausuch @grisu48 the last pipeline (https://github.com/SUSE/BCI-tests/runs/6875884458?check_suite_focus=true) failed with:
E       AssertionError: Unexpected exit code 1 for CommandResult(command=b'systemd-analyze time', exit_status=1, stdout=None, stderr=b"\x1b[0;1;31mBootup is not yet finished (org.freedesktop.systemd1.Manager.FinishTimestampMonotonic=0).\x1b[0m\n\x1b[0;1;31mPlease try again later.\x1b[0m\n\x1b[0;1;31mHint: Use 'systemctl list-jobs' to see active jobs\x1b[0m\n")

This could result in a flaky test, but maybe we should add a HEALTHCHECK to init instead. WDYT?

I will have a look at this. Let me have another look, I would like to not push tests with flanky results. Marking as draft until resolved.

https://github.com/SUSE/BCI-dockerfile-generator/pull/9 should indeed prevent race conditions where the system target has not yet been reached. Thanks for the improvement.
I've pushed the black formatted test run and think we can proceed with this PR.

Could it be that you force pushed some of your improvements away?  Part of the latest changes got removed. Let me check, something went horribly wrong here.
Happy Monday btw :coffee:

Fixed. Please review again and sorry for the fuzz @dcermak


Performance testing for acapy

What is the most effective way for conducting performance testing on an acapy instance or an application built on acapy? Is there any framework? We are trying to calculate latency and throughput of transactions.

Ther is work currently happening in ACA-Py for implementing a load test generator. This is a Code With Us opportunity funded by BC Gov (details here) and awarded to Indicio.  We’ll be talking about the design tomorrow (Tuesday) on the ACA-Pug Meeting at 8:00 Pacific / 17:00 Central Europe.  With that capability, you will be able to run the load test generator as a set of holders (configurable how many, how many activities), hitting various “systems under test” Issuers and Verifiers, possibly with a mediator in front of the holders.
Another data point.  BC recently reloaded the “dev” version of OrgBook BC that holds about 2.5M credentials on start up.  The implementation is an ACA-PY issuer issuing all of the credentials to a single ACA_Py holder (a so-called “community wallet”). During the loading, without much tuning, the systems had a sustained rate of about 30 issuances/second.

For guidance on performance testing ACA-Py, please look at Aries Akrida, which provides capabilities for such testing.  BC Gov will be using Aries Adrida in the next short while to understand the performance characteristics of ACA-Py.


Graph model's memory usage

I have noticed that the memory usage of sigma.graph is very high. Processing large graphs leads to out of memory errors.
To give an idea, here is how sigma.js compares to node.js' graphjs module for a graph having 1210 nodes and 26409 edges, just to instantiate the graph:
# sigma.js when issue was opened
# process.memoryUsage().rss: 141.537.280

# sigma.js after a memory leak fixed in d821d8c
# process.memoryUsage().rss: 135.946.240

# graph.js
# process.memoryUsage().rss:  24.743.936

It's worth noting that graph.js also indexes the data for fast access.
I'd rather stick to using sigma.js because its API looks more straightforward to me. I have investigated the graph data and indexing structure in sigma.js
Here is the graph model's data stored for a very simple graph:

node n1 with one attribute attr:nodeAttr
node n2 having no attribute
edge e1 from n1 to n2 with one attribute attr:edgeAttr

{
  "nodesArray": [{ "attr": "nodeAttr", "id": "n1"},{ "id": "n2"}],
  "edgesArray": [{ "attr": "edgeAttr", "id": "e1", "source": "n1", "target": "n2"}],
  "nodesIndex": {"n1": { "attr": "nodeAttr", "id": "n1"}, "n2": { "id": "n2"}},
  "edgesIndex": {"e1": { "attr": "edgeAttr", "id": "e1", "source": "n1", "target": "n2"}},
  "inNeighborsIndex": {"n1": {}, "n2": { "n1": {"e1": {"id": "e1", "source": "n1", "target": "n2", "attr": "edgeAttr"} }}},
  "outNeighborsIndex": {"n1": { "n2": {"e1": {"id": "e1", "source": "n1", "target": "n2", "attr": "edgeAttr"} }}, "n2": {}},
  "allNeighborsIndex": {"n1": { "n2": {"e1": {"id": "e1", "source": "n1", "target": "n2", "attr": "edgeAttr"} }}, "n2": { "n1": {"e1": {"id": "e1", "source": "n1", "target": "n2", "attr": "edgeAttr"} }}},
  "inNeighborsCount": {"n1": 0, "n2": 1},
  "outNeighborsCount": {"n1": 1, "n2": 0},
  "allNeighborsCount": {"n1": 1, "n2": 1}
}

Observations:

nodesArray and nodesIndex are duplicate storage of the entire nodes dataset. Do we really need the array structure when we can iterate on nodesIndex object?
same remark as previous for edgesArray and edgesIndex
in nodesIndex, the objects' keys are the nodes Ids. Then we don't need to store the redundant id attribute.
same remark as previous for edgesIndex
inNeighborsIndex, outNeighborsIndex and allNeighborsIndex stores the whole edge data in the nested structure. Significant savings can be done by only storing the reference to the edges' id instead. Neighbors indexes can be used to check whether there is an edge between two nodes, and to lookup the id of the edge between two nodes. then in cases when it's needed a accessing edgesIndex by edge key can give the edges attributes.
inNeighborsIndex, outNeighborsIndex and allNeighborsIndex do store keys for nodes that have no edge. This is users extra space for nothing.
allNeighborsIndex is a concatenation of inNeighborsIndex and outNeighborsIndex. A method looking up both indexes and concatenating the result could do the same.
Neighbors count indexes: no redundancy, clean structure.

It seems to me that "fixing" points 1 to 5 above won't require lot of rework of the library, while and won't have affect the API too much, while providing significant gains.
Point 6 may require extra checks when using the indexes, I am not sure if its something suitable.
The effect of change suggested in point 7 should be further investigated.
For the sample graph used above, the graph model implementing the suggestions above would be so:
{
  "nodesIndex": {"n1": { "attr": "nodeAttr"}, "n2": {}},
  "edgesIndex": {"e1": { "attr": "edgeAttr", "source": "n1", "target": "n2"}},
  "inNeighborsIndex": {"n2": { "n1": "e1"}},
  "outNeighborsIndex": {"n1": { "n2": "e1"}},
  "inNeighborsCount": {"n1": 0, "n2": 1},
  "outNeighborsCount": {"n1": 1, "n2": 0},
  "allNeighborsCount": {"n1": 1, "n2": 1}
}

301 characters instead of 772 (spaces and line breaks excluded)
Is this something that the maintainers of sigma.js would consider worth being implemented?
Edit: added memory usage measured after memory leak fixed in d821d8c

How can I use the sigma.noIndex.js if I wasn't originally using sigma.min.js?


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/de-autobahn-feedstock#2.


router-link integration with custom components

Version
3.0.1
Reproduction link
http://jsfiddle.net/leopoldthecuber/forzLu80/38/
Steps to reproduce
Run the demo. In the demo I mainly did two things:

a custom component my-button, it has a prop content and a named slot icon
a router-link with its tag set to my-button

What is expected?
Props and named slots can be passed to custom component.
What is actually happening?
Props and named slots don't work

https://github.com/vuejs/vue-router/issues/1678 says props can be passed down since 2.8, but it still doesn't work
Default slots work fine, while named slots don't

is there any chance #1678 gets fixed? or any idea if it will ever be and when?

The recommended approach is to use the v-slot api of router-link. Making router-link functional would be a breaking change and thus isn't possible