feat: update Multisig view with Accordion

Closes #167

@Tbaut  So the current to do that needs clarification (the rest is fixed):

New transaction button position (sync the design)
3 dots button (not present in the design in the multisig view, but is present in the header view. The issue is that we don't show the Header view for standalone multisig)
Always show a balance on multisig (imo it stretch  the multisig selector too much that it looks disproportionately )
Make margin left on Signatories list (Probably better to clarify with designer)

Your point 1 and 2 need to be discussed with @sweetpea22, let's have a meeting.
I answered the point 3 in the related comment above.

Make margin left on Signatories list (Probably better to clarify with designer)

we can talk about it too with @sweetpea22
You marked it as resolved, but I can still see the behavior from https://github.com/ChainSafe/Multix/pull/313#discussion_r1300688535


Cannot use routes that contain %2B

I have the following routes:
options.Conventions.AddPageRoute("/packages/details", "packages/{id}/{version}/{targetFramework}");

This works great for urls like /packages/foo/1/net
However, if the targetFramework contains an escaped + as in: /packages/foo/1/net%2Bother then IIS (hosted as an Azure web app) says:

The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

And it never runs my code. You can see it in action here: http://fuget.org/packages/sqlite-net-pcl Just click the yellow button.
The same URL works fine on Kestrel.
I expect IIS to behave as Kestrel since the URL is well-formed and matches my route definition.

What do you see in the application logs for that request?

I think IIS Request Filtering will filter out '%2b': https://blogs.iis.net/nazim/use-of-special-characters-like-in-an-iis-url.

Applog says:

@jkotalik my god that's terrible. How do you turn all that off?

I ended up creating a web.config file and put this in it:
<?xml version="1.0" encoding="utf-8"?>
<configuration>
    <system.webServer>
        <security>
            <requestFiltering allowDoubleEscaping="true" />
        </security>
    </system.webServer>
</configuration>

It would be nice if Kestrel and IIS had the same defaults. I can't test on IIS and getting these deployment surprises isn't nice.


Nothing happens when select image to upload

Android 7.1.1
Smartphone Sony Z5 Compact.
Zulip self-hosted installation.
Photo with camera also nothing not send to chat.
Thanks.

@zulipbot claim

Hmm interesting. Thanks for the report!
I don't think we've seen reports of symptoms like these before, and uploading images is a pretty frequently-used feature; so I suspect there's something different about your server that is part of what's happening here. If so and if we can identify that, then we'll be in good shape to figure out how the app should handle it.
It sounds like you're interested in debugging this yourself, which is great. :) I have a couple of suggestions of things to try for comparison:

Join chat.zulip.org (the Zulip community's own server), and try uploading there; #test here is a good stream to send test messages to.
Make your own test organization on zulipchat.com, and try there. That's the major cloud Zulip service, run by Zulip core developers (including me), with a pretty different configuration from chat.zulip.org.

@gnprice I will be try debug this problem, thanks.

We just fixed in 3529c3155 a bug that, at least on iOS, was causing the "use camera within app" part of these symptoms. It's possible the same bug was the cause of those symptoms here, but I was never able to reproduce the issue on Android (only on iOS, 100% of the time there), so I'm not sure.
The "select image to upload" part of these symptoms shouldn't be affected by that fix, though, so there's definitely still an open bug here.

(that commit was from #2972, for cross-reference)

@gnprice When will be release in to Google Play?

@frg-tarasov I've just sent a version with this fix (17.1.98) to all Google Play users!
When you get a chance, please try that version out and see if either or both of these symptoms ("select image to upload" not working, and "photo with camera" not working) are resolved for you.

@gnprice Thanks, i now will be test this issues.

@gnprice I'm sorry. Now i try send photos and problem stay.
After i check nginx log and see that nginx return error for upload request with exceed body size limit.
Change nginx setting client_max_body_size resolve both this issues.
I close this issue, thanks.


[FEATURE] Are there any plans to add "update" and "delete" methods to the EmbeddingStore interface?

The current top-level interface for vector storage, the EmbeddingStore interface, only has methods for adding (add/addAll) and querying (findRelevant). Are there any plans to expand this interface in the future to include "update" and "delete" methods to accommodate more scenarios?

Hi @coderyjr yes, this is planned, but I am not sure when exactly it will be implemented. Any help would be much appreciated!

Duplicate of https://github.com/langchain4j/langchain4j/issues/301


There are some problems on custom key

[Vue warn]: Error in event handler for "match": "TypeError: undefined is not a function"
found in
   <keyboard
            @match="matchGame"
            layouts="123456|7890{←:backspace}{ok:match}"
            v-model="awayPkCode"
            :maxlength="4"/>

It looks like the error is emitted from your event handler i.e. matchGame.

my handler
     matchGame() {
        console.log('.....');
      }

@MartyWallace error info
[Vue warn]: Error in event handler for "match": "TypeError: undefined is not a function"

found in

---> <Keyboard> at node_modules\vue-keyboard\src\keyboard.vue
       <ElCol>
         <ElRow>
           <VueJsModal> at /Users/yev.vlasenko2/Projects/vue/vue-js-modal/src/Modal.vue
             <ElContainer>
               <ModeTraining> at src\components\mode-intro\mode-challenge.vue
                 <App> at src\App.vue
                   <Root>

rebuild then ok 。thanks


(Suggestion) Legacy edition-esque statistics screen

this is not a suggestion forum; join the discord to suggest things


(URGENT) FIPS PUBS MR scheme wrong (NIST.FIPS not NIST.FIPS.PUB)

See https://github.com/metanorma/nist-pubid/blob/main/README.adoc

See these:

MR
DOI

NBS.FIPS.PUB.34
NBS.FIPS.34

NBS.FIPS.PUB.35
NBS.FIPS.35

NBS.FIPS.PUB.36
NBS.FIPS.36

NBS.FIPS.PUB.36-1975
NBS.FIPS.36-1975

@ronaldtse we have FIPS PUB from NIST and NBS (examples: NIST FIPS 100-1 and NBS FIPS 101)
Should it be NIST.FIPS.100-1 and NBS.FIPS.101 as a result of conversion? Is it the same for Final PubID?

Should it be NIST.FIPS.100-1 and NBS.FIPS.101 as a result of conversion? Is it the same for Final PubID?

Yes I believe so.


History: Speed up acquisition of shards on ring membership change

Whenever the ring membership changes, the history engine re-evaluates the shards it owns and subsequently acquires new shards, releases shards no longer owned. This whole process today, is performed by a single go-routine that goes over the shards from 1..N serially, calling UpdateShard (CAS) to claim ownership of new shards.  This is not great with large number of shards, say, 16k, when the median latency of UpdateShard is in the ~100ms range i.e. When nodes go down, the history engine quickly disowns the shards, but slowly acquires new shards. This can make shards unavailable for a short period of time during node failures or deployments.
Short Term:
Proposal: Spin up multiple shardControllers per history node
Today, all 16k shards are handled by the same shardController, but it doesn't have to be. We can have a configurable policy, say, max of 4k shards per shardController. This will both (1) Speed up acquisition of shards (2) Reduce lock contention
Long Term: Come up with a design for graceful shard handoff

This is supported for matching now and more history improvements are coming soon


Kubernetes Dashboard Generates Significant Amount Of Traffic

Bug description
Our organization migrated to Podman Desktop ~1 year ago away from Docker Desktop to avoid licensing costs. So in my context, I have hundreds of developers with Podman Desktop installed.
We have encountered a scenario with Podman Desktop 1.14+ (Specifically with the new kubernetes dashboard functionality), where developers with a valid kubeconfig in their path started generating a significant amount of traffic to the kube API server of multiple clusters causing stability issues for those cluster control planes (IE outages), after discovering the node agent these requests are made with (node-fetch) we blocked the user agent to restore stability to the clusters and effectively block podman Desktop.
I believe the default behavior of this functionality either needs to be ratcheted down so its not making so many API requests by default, or conversely make the dashboard refreshable manually with the option to enable automatic refreshed.
Operating system
MacOS 14
Installation Method
Installer from website/GitHub releases
Version
1.14.0
Steps to reproduce
No response
Relevant log output
<redacted source IP> - - [22/Nov/2024:04:30:32 +0000] "GET /k8s/clusters/<redacted>/api/v1/namespaces/<redacted>/services?watch=true&resourceVersion=5515861643 HTTP/1.1" 200 240 "-" "node-fetch/1.0 (+https://github.com/bitinn/node-fetch)" 386 21.571 [cattle-system-rancher-80] [] <IP_ADDRESS>:80 240 21.571 500 aa6661c2be0b717ae64a7c7929d3cb7e

Additional context
No response

Thanks @MichaelGerhart for this report.
I'll develop in an upcoming message (to try to find the root cause of the problem), but as a first workaround, a possibility to not block the user agent at the server-level would be to "de-activate" the kubernetes functionality from some or all Podman Desktop installations, by changing the path of the kubeconfig file for these installs, to indicate a non existing file, so the functionality will be disabled on these installs.
Users can change it by going to Settings > Preferences > Kubernetes > Kubeconfig, and setting a non-existing file (/tmp/non-existing for example)
This way, some users may still have access to the Kubernetes functionality in Podman Desktop, while desactivating for the suers not willing to use it.

The traffic should be greatly reduced with https://github.com/podman-desktop/podman-desktop/pull/10335 (only the current context will be monitored)

The traffic should be greatly reduced with #10335 (only the current context will be monitored)

Closing as fixed, please reopen it if it still happens with the upcoming version 1.16


Feat: ability to customize the endpoint operationId in documentation

Hi!
First of all, thank you for your effort and time in this project!
I wrote a small change in this Pull Request so we can override the generated operationId for each endpoint, while maintaining the previous behavior.
This is not a final implementation; I actually would love some pointer into how to implement this correctly;
I wrote this so we can discuss if the change is valid, at work we are using some libraries that use the operaitonId inside the swagger definition, so being able to predict how the endpoint operationId is named would help us.
Again, thank you for your time.

@john-schmitz ,
perhaps utilizing the Endpoint::getOperationId() could also be useful in the following place (not sure yet):
https://github.com/RobinTail/express-zod-api/blob/fa47a91403ec7b70ec86c668c28dbab977e06b7d/src/integration.ts#L97-L98

@john-schmitz ,
perhaps utilizing the Endpoint::getOperationId() could also be useful in the following place (not sure yet):
https://github.com/RobinTail/express-zod-api/blob/fa47a91403ec7b70ec86c668c28dbab977e06b7d/src/integration.ts#L97-L98

@RobinTail Sorry, I did not understand what you meant by this. Could you please expand it?

Thank you for the contribution, @john-schmitz . The PR looks good. A couple of changes requested — let me know if they are unclear.
Could you please also clarify, why it might be important to have the operation id specified manually? I think forget where it's displayed or involved.
Thank you

To expand on it, we export the API definition so multiple clients in different languages can consume our API using generated client code.
Consider this endpoint: GET /cake/{cakeId}. The generated operation ID would be something like: GetCakeCakeId.
For more complicated paths, it gets confusing when referencing a specific endpoint in our daily conversations and when consuming the generated client code for the API.
Its easier to write api.GetCakeById than api.GetCakeCakeId
So, we prefer to reference specific endpoints by the operation id.
@RobinTail

Anyway, I addressed most of your comments; if I missed something, please let me know!

Sorry, I did not understand what you meant by this. Could you please expand it?

I meant that there is a feature of generating the client types (Integration), which is using the same approach for naming the IO types of endpoints. I assumed that the featured customization can also be utilized there. But it can be done later.

🚀 12.3.0, @john-schmitz


Data augmentation by random scaling and random mirroring

Hi to all !
Might you have information on by the random scaling and random mirroring functions at ImageReader.py, in which amount are we augmenting the training data?
I mean if i have a training set with 600 images, and if i run the fine_tune.py --random-scale --random-mirror command, how many training images will i have in the end?

will there be any augmentation, so the training set will be more than 600 images or
the current 600 images will be just randomly transformed so will i still have 600 images in the end but with transformed versions?

thanks in advance,
s.

Dear @DrSleep, thanks for your reply. I guess that 2NSC should be 2S*C, shouldn't it?


Depricated addListener used in useMedia

Just want to mention that the useMedia hook are using the deprecated event addListener here https://github.com/streamich/react-use/blob/master/src/useMedia.ts
It should be updated to use the new listener. See https://developer.mozilla.org/en-US/docs/Web/API/MediaQueryList/addListener
I haven't had time to do a pr. sorry.

Hello @TommySorensen, I have fixed it and raised a PR regarding this issue. Let me know if you see any issues with that PR.

@iAmmar7 Looks good to me 👍


Google assets excessively slow to load; Maps crash Ladybird

While browsing Google owned domains, specifically Google.com, Youtube, Gmail, Google Maps image assets are taking exceedingly long to open. Google Maps simply crashes the browser. Startpage, DuckDuckGo function fine however.
Reviewing the terminal read out, this seems to be related to not having a decoder in place, however it is a JPEG image format, initially thinking that it might have been a WEBP instead. Startpage.com logo image is SVG formatted and loads fine however.

Hi there! This bug report has way too many things going on at the same time. If I read this right, what you are reporting is essentially:

google.com takes a long time to open
YouTube takes a long time to open
GMail takes a long time to open
Google Maps image assets are taking exceedingly long to open
Google Maps crashes the browser
There may be some issue with not having an image decoder in some context

We are aware that many websites take a long time to open. Many parts of our stack are slow and unoptimized. We don't require bugs about every site that loads slowly :)
The main actionable issue I see here is how Google Maps crashes the browser, so let me repurpose this bug for that, and see if we can fix it.


Mi Smart Kettle Pro (yunmi.kettle.v12) has no urn

Heyho,
i tried to add my Smart Kettle into Homeassistant, and get the following error:
2021-12-27 09:05:28 DEBUG (MainThread) [custom_components.xiaomi_miot] Setup xiaomi cloud for user: xxx, 1 devices 2021-12-27 09:05:28 INFO (MainThread) [custom_components.xiaomi_miot] Xiaomi device: ['Mi Wasserkocher', 'yunmi.kettle.v12'] has no urn
Seems like this device isnt supported? Is there a way to help you to include it?
Best regards, Marko

Not currently supported without urn.

Featurewise, it seems to be identical with this one here:
yunmi.kettle.v13

Execute this shell command on the terminal of  your HA host:
miiocli -d device --ip 192.168.lan.ip --token xxxxxx raw_command get_properties '[{"did": "miot", "siid": 2, "piid": 1}]'

After a long search i found out, miiocli doesnt support bluetooth device like the kettle, but only wifi devices :-/
When using a BLE Scanner, i find these informations

Try this: https://github.com/custom-components/ble_monitor

Try this: https://github.com/custom-components/ble_monitor

It doesn't work because kettle PRO requires an active connection and ble_monitor only supports passive listening to BLE advertisements. See https://github.com/custom-components/ble_monitor/issues/398 and https://github.com/custom-components/ble_monitor/issues/509 for more context.

This component is powerless, because it can only obtain the device state through LAN or Xiaomi cloud (urn is required).

@al-one any particular reasons why this issue was closed?
As @Japander mentioned, yunmi.kettle.v12 seems to be almost identical with yunmi.kettle.v13 which I suppose is already working.


Refactor get topic names and types

Connects to ros2/ros2#361

@wjwwood I noticed a few jobs with uninitialized warnings that seem related to this PR:
http://ci.ros2.org/view/nightly/job/nightly_linux_release/478/warnings22Result/
https://github.com/ros2/rmw_connext/pull/231/files#diff-9bba412d4597100825da2b2307403d08R1092

Hmm, I fixed that warning in Fast-RTPS:
https://github.com/ros2/rmw_fastrtps/commit/027139a2103af9ae0a1e2e05d00258edfb627688#diff-8225b098f1ec4debcdcee03d65ae95d5R2906
I guess I missed this one. I'll make a pr to fix it.
Not sure how that got passed CI...


View-Only Links name field allows unlimited input

Steps
1. Make an new view only link
2. Paste as much text as you want into the name field.
![screen shot 2015-08-13 at 3 24 36 pm](https://cloud.githubusercontent.com/assets/1322421/9259649/9b10879c-41d0-11e5-886d-5ec47bd95641.png)
Expected
Limited input like on other Title fields (e.g., Project/Component titles).

The question will be how many characters will be reasonable. @jlcohoon

This is a duplicate of https://github.com/CenterForOpenScience/osf.io/issues/4034

I'll close, thanks, didn't see that one when I searched


Remove delegation

on supporting the proposal, overriding the delegation
on selecting 'remove delegation' from delegate select.


Remove iteration process in feature_select

Looking at this code in feature_select.py
code location here
# iteratively passing normalized data
    for norm_data, feature_file_out in io_files:
        feature_selection(
            normalized_profile=norm_data,
            out_file=feature_file_out,
            config=config_path,
            log_file=log_path,
        )

In addition, the feature_select.smk module also takes in a list of profiles, therefore removing snakemake parallelization here
It would be best to remove the for loop and take advantage of snakemake's parallelization.

Update: This is related to #41. Using a single input for the feature_select module is causing issues. Currently, the script requires a list of file paths, which is removes  CytoSnake's multiprocessing (handled by Snakemake). However, we want users to have the option to do feature selection across all plates or choose specific features for each plate while maintaining flexibility.
This will be added into the next release project roadmap.


Configure logging at runtime

Enable configured logging and bump version from 2.4.1 -> 2.4.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 89.676%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/auth_service/main.py
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
1937

Relevant Lines:
2160

💛  - Coveralls


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ABI/API checker doesn't match overloads properly

This build failed the ABI/API check, but it shouldn't have - the actual changes were adding const to a non-const method, which is always safe.  It does this correctly in some cases, but the 3 at the top match one overload against another and claims the parameters change (which is a breaking change).
log
Problems with Symbols, Medium Severity  3

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
idempotent_mutation_policy.h
namespace google::cloud::bigtable::v1
[+] AlwaysRetryMutationPolicy::is_idempotent ( google::bigtable::v2::Mutation
const& p1 )  1
⇣
AlwaysRetryMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v125AlwaysRetryMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

              Change                                 Effect
1 Parameters list has been        Recompilation of a client program may be
  changed.                        broken.

[+] IdempotentMutationPolicy::is_idempotent ( google::bigtable::v2::Mutation
const& p1 )  1
⇣
IdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v124IdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

              Change                                 Effect
1 Parameters list has been        Recompilation of a client program may be
  changed.                        broken.

[+] SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::Mutation const& m )  1
⇣
SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v128SafeIdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

              Change                                 Effect
1 Parameters list has been        Recompilation of a client program may be
  changed.                        broken.

to the top

Other Changes in Symbols  8

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
idempotent_mutation_policy.h
namespace google::cloud::bigtable::v1
[+] AlwaysRetryMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 )  1
⇣
AlwaysRetryMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v125AlwaysRetryMutationPolicy13is_idempotentERKNS_8bigtable2v224CheckAndMutateRowRequestE

         Change          Effect
1 Method became const. No effect.

[+] AlwaysRetryMutationPolicy::is_idempotent ( google::bigtable::v2::Mutation
const& p1 )  1
⇣
AlwaysRetryMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v125AlwaysRetryMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

         Change          Effect
1 Method became const. No effect.

[+] IdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 )  1
⇣
IdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v124IdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v224CheckAndMutateRowRequestE

         Change          Effect
1 Method became const. No effect.

[+] IdempotentMutationPolicy::is_idempotent ( google::bigtable::v2::Mutation
const& p1 )  1
⇣
IdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v124IdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

         Change          Effect
1 Method became const. No effect.

[+] SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 )  1
⇣
SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v128SafeIdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v224CheckAndMutateRowRequestE

         Change          Effect
1 Method became const. No effect.

[+] SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::Mutation const& m )  1
⇣
SafeIdempotentMutationPolicy::is_idempotent
( google::bigtable::v2::CheckAndMutateRowRequest const& p1 ) const
_ZN6google5cloud8bigtable2v128SafeIdempotentMutationPolicy13is_idempotentERKNS_8bigtable2v28MutationE

         Change          Effect
1 Method became const. No effect.

polling_policy.h
namespace google::cloud::bigtable::v1
[+] GenericPollingPolicy<LimitedTimeRetryPolicy,
ExponentialBackoffPolicy>::clone ( )  1
⇣
GenericPollingPolicy<LimitedTimeRetryPolicy, ExponentialBackoffPolicy>::clone
( ) const
_ZN6google5cloud8bigtable2v120GenericPollingPolicyINS2_22LimitedTimeRetryPolicyENS2_24ExponentialBackoffPolicyEE5cloneEv

         Change          Effect
1 Method became const. No effect.

[+] PollingPolicy::clone ( )  1
⇣
PollingPolicy::clone ( ) const
_ZN6google5cloud8bigtable2v113PollingPolicy5cloneEv

         Change          Effect
1 Method became const. No effect.

FYI the abi-dumper tool hasn't had a release since 2017 and abi-compliance-checker since 2018. Maybe an upstream issue should be filed and nudge the developer to make a new release?

The errors reported at the top are ABI breaks (which abi-dumper reports). It says
1 Parameters list has been        Recompilation of a client program may be
  changed.                        broken.

And const-ifying a member function is an ABI break.
So is there anything to be done for this issue? If not, we should probably just close this.

I am with @devjgm this is not actionable, closing.


fixes deprecated class name warning due to symfony upgrade 3.3

Fixes warning:
"PHP error:  Support of class name string was deprecated after version 3.2 and won't work anymore in 4.0. in /usr/local/sympl/sympl.app2/vendor/symfony/workflow/Registry.php on line 33"
See: https://github.com/symfony/symfony/blob/master/UPGRADE-3.3.md#workflow

@kadinho thanks for this.


feat: support dynamic app config and runtime config

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
The initial implementation of both runtime-config and app-config are using a deep frozen version of the build-time config so that it is safe to be shared across requests. However, both app-config and runtime-config are by design dynamic configuration sources.
Dynamic Runtime config
When using useRuntimeConfig() in an ambient context, it uses a shared, cloned, and mutable frozen object.
When using the new useRuntimeConfig(event), cloned object will be cached in the request event. Whenever we call the composable, the latest environment variables will be applied. This supports dynamic env updates as well as Cloudflare module/pages environment compatibility (https://github.com/unjs/unenv/pull/95)
Dynamic App Config
When using useAppConfig() in an ambient context, it returns a shared, cloned, and mutable frozen version of the app config. This is mainly for backward compatibility and is not shared with non-ambient versions.
When using useAppConfig(event), it returns a cloned and mutable version of the app config suitable to be dynamically extended per request without leaking context across requests.
Side note: Nuxt can integrate with the new mutable useAppConfig API of nitro by exposing it from the renderer to the rest of the app) and augmenting with next-specific app-config that might be tree-shaken from the nitro side. This way app config can be universally extended. Since useAppConfig from nuxt is already a context-aware composable it access to the event.
Other notes
Thinking today about dynamic config sources, we should have avoided the ambient context usage for such composables. This PR adds a new feature as an enhancement with fully backward compatibility and without breaking changes but i am thinking to slowly starting deprecating ambient usage and replacing it with a better and clear way for exposing that kind of build-time static configurations. Starting by updating docs and maybe a warning once there was the better alternative in place and widely adopted by Nitro and Nuxt.
This implementation is using klona which according to their benchmarks at least is one of the fastest possible solutions. structured clone as alternative, is not possible within workers. A defu based clone (https://github.com/unjs/defu/issues/90) could be used. But i am also thinking to implement change sets as a smart Proxy with lazy cloning per request. If we have it, it can be also useful for smart app-config hydration from server to client and only including changes.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

However this PR is fully backward/compatible with Nuxt, would be happy to hear feedbacks from @unjs/nuxt team members if there are real concerns about this direction and waiting for a while before moving this forward to edge.

Any feedback regarding this @danielroe ?

I'm planning to have a look today.

I really like this approach conceptually and think it makes sense 👍  Nice work.

Update: More thinking mutable config with ambient context, can be an unsafe change especially if go unnoticed by developers. For now, ambient context composables, remain frozen (but cloned).
This means that for now, nitro-dev and cloudflare presets, can only leverage the augmented environment variables as long as they explicitly pass event. Other presets, always have augmentation.
This is not final. In next steps, i plan to work on (mentioned above) proxy package that can enable us internally tracking and mutating a frozen public interface. This way we can avoid both clone costs and also internally apply envs even for ambient frozen public config.


Add Edge Bridge Extension

Add Edge Bridge Extension which enables forwarding of Analytics track events to Experience Platform Edge Network.

Add TS, Android and iOS implementation

Add tests

Update Sample App

Add Readme doc

Update Migration doc with sending data to Analytics options

Description
Edge Bridge View is added in the sample app with TrackAction, TrackState buttons.

The generic data now is converted into the Experience Data Model (XDM) format in the sdk log.
"xxxxx" AEPSampleApp[67646:2168359] [AEP SDK DEBUG - ] EdgeNetworkService - Sending request to URL https://edge.xxx.net/ee/or2/v1/interact?configId=xxx with headers: ["accept": "application/json", "Content-Type": "application/json"] and body:
{
"meta" : {
"konductorConfig" : {
"streaming" : {
"enabled" : true,
"recordSeparator" : "\u0000",
"lineFeed" : "\n"
}
},
"state" : {
"entries" : [
{
"key" : "xxxxx",
"value" : "xxxxx",
"maxAge" : 34128000
},
{
"key" : "xxxxx",
"value" :"xxxxx",
"maxAge" : 1800
}
]
}
},
"xdm" : {
"identityMap" : {
"ECID" : [
{
"id" : "xxxxx",
"authenticatedState" : "ambiguous",
"primary" : false
},
{
"id" : "xxxxx",
"authenticatedState" : "ambiguous",
"primary" : false
}
]
},
"implementationDetails" : {
"version" : "3.8.1+1.6.0",
"name" : "https://ns.adobe.com/experience/mobilesdk/ios/reactnative",
"environment" : "app"
}
},
"events" : [
{
"xdm" : {
"_id" : "xxxxx",
"eventType" : "analytics.track",
"timestamp" : "xxxxx"
},
"data" : {
"contextdata" : {
"customKey" : "value"
},
"action" : "action value"
}
}
]
}
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@emdobrin Updated the review comments.  Ready to be reviewed again.


Bug of Regular Express matching Report For Preprocessing LLVM IR

construct_struct_types_dictionary_for_file in inst2vec_preprocess.py cannot not handle the value in to_process, { i64, { i32, { x86_fp80 } }*, [7 x { i32, { x86_fp80 } }] }.
I try to fix it but the matching seems to complex? Could you help me?

Well, it might have to do with the type x86_fp80. I do not think this was supported in LLVM 6.0, which we wrote the parser for. Perhaps a specific corner case is necessary?

but which LLVM version do you use?


[BUG] Doesn't load @scene triggers on game load

When you first load into foundry VTT, scene triggers aren't triggering unless you leave the scene and return to it.
This is new only with the new patch, it was working fine in the past.

Any error on the web console ?

Try out 0.8.37 and let me know

Updated to 0.8.37 and its working again
No there were no errors in console related to the scene (although there were plenty of warnings about being unable to find drawings present on a different scene)

there is a module settings for tell to the module to not print these warnings if you want.
I'll leave open this ticket for further analyze.


Added a cmd line switch to control text that will separate each run

Not sure if anyone else would find this useful, but I like to see each run have some kind of separator between runs.
example.... gow ..... -S "-------------" ....

Also see 5348a1592279b319adbc84a561973222d27dba73.


Work towards merging TdsParser

This is for tracking purposes connected to #1261
I would like to merge TdsParser to a single code base netfx/netcore
The approach I'd like to take is this:

Do small PR's first which only change code style to align netfx/netcore codebases
Do small PR's that target specific things (e.g. change from Types to type aliases e.g. Int32 to int)
Do more PR's that align code but do need code changes

This way (I hope?) changes are small-ish and/or low-riskish, so that they can be reviewed fairly fast (much easier than "big bang" PR's IMHO)
Hope that makes sense, if not just tell me please :)

Align Newlines: #2954
Align whole comments: #2955
Fix typos (align) etc in comments: #2956
Align type aliases #2957
Align keywords #2958
Align var vs Type #2959
Align redundant namespaces #2960
ADP.IsEmpty -> string.IsNullOrEmpty #2961
Align BitConverer -> BinaryPrimitives #2963

Thanks for this @MichelZ - these bundles of changes look good to me.
Another pattern of changes will also need to be made at some point, across TdsParser and other core parts of SqlClient: Constrained Execution Region support exists in netfx, but was removed in netcore. This is (roughly) a no-op, but each instance adds ~45 lines to the diff and it's usually inside methods which are otherwise identical. It's probably not necessary as part of this cleanup, but it'd be a good idea to port the CER support to netcore before starting to merge identical methods.
On a wider point: I think it makes sense to use the TdsParser approach as a template for the core parts of SqlClient.

Thanks for taking a stab at merging this behemoth of a class 👍 I'll do my best to look at these PRs in a timely manner, but keep in mind we need two signoffs on them to get them merged. I'll also do my best to encourage my teammates to look at your PRs as well 😅
That being said, I think for the smaller changes, it might be more beneficial to combine a couple together - eg, comments and comment typos could be rolled into a single comment PR. I've been trying to structure my PRs such that each commit is a complete item. So, eg, if you had a PR for comments, one commit could be getting the blocks aligned, the second could be to fix the typos. You don't need to go back recall the PRs that are currently out, but it might be beneficial going forward. It's a delicate balance between too much overhead per PR and too much to review in a single PR.
My strategy for merging classes has been to create a "merge file" in the commit history of the PR. Basically indicate what blocks of code are from only one side of the merge. Then, merge work through each chunk of conflicts to either resolve as one or the other (eg, Int32 vs int) or leave with an #if NETFRAMEWORK sort of thing.


Duplication when replacement contains original string

It seems that when the replacement string matches the search, the substitution gets applied twice. The rule
donut --> pink donut
performs the substitution
donut --> pink pink donut.
As a workaround, I'm using
gibberish1 --> gibberish2
donut --> gibberish1
gibberish2 --> pink donut
which does in fact perform the subsitution donut --> pink donut because all the substitutions are done in order twice.
I'm using the version in the Chrome store. I'll try to fix it myself, but I thought I'd let you know in the meantime.

I see the same.  Slightly different ordering for reference:
When setting up:
donut --> donut life
I expect
donut --> donut life
but I see
donut --> donut life life

Hm that's interesting... To be honest, I haven't touched this project in years, and I'm swamped with work right now -- feel free to make a PR and I'll update the store version, though!

Yeah, I see the same. I have something like this...
^$.+ --> $& ndone
So it would take a string like
$ some-command
and output
$ some-command ndone ndone

I've submitted @djedi's patch that should fix this to the dev store; update should be out to y'all in a couple hours or so. @djedi, once you're on 0.0.2, could you confirm your behavior is resolved?
Thanks!

Nice on @djedi - thanks!

Yeah, it works for me. Thanks!

Awesome! Glad to hear it 🙂


rename CloudVolume.attach_volume to attach

Overview
We've never officially stated the name of the operation to attach and detach a cloud volume for supports. Either way, it is missing from CloudVolume base.

Renaming from CloudVolume#attach_volume to attach
renaming from CloudVolume#detach_volume to detach

Includes

[ ] https://github.com/ManageIQ/manageiq-api/pull/1156
[ ] https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/373
[ ] https://github.com/ManageIQ/manageiq-ui-classic/pull/8247
[ ] https://github.com/ManageIQ/manageiq-providers-amazon/pull/767
[ ] https://github.com/ManageIQ/manageiq-providers-openstack/pull/794

Concerns

Not sure exactly how CustomButton and supports works, but maybe the name is linked to attach_volume dynamically?
It looks like attach_volume has been the name of this functionality for a while.
Pretty sure this handles all the ui classic cases. (there were 3 main use cases)

Affects
This affects the api around cloud volumes that @MelsHyrule is putting into place

@miq-bot cross-repo-tests https://github.com/ManageIQ/manageiq-api/pull/1156 https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/373 https://github.com/ManageIQ/manageiq-ui-classic/pull/8247 https://github.com/ManageIQ/manageiq-providers-amazon/pull/767 https://github.com/ManageIQ/manageiq-providers-openstack/pull/794


Use TypeScript

tests fail due to registerMethod being called with arguments that do not adhere to type definitions.


Searching with English swim does not result in yâhyânam among the Cree results

Following up on #354, when one searches with the English verb swim ...
https://github.com/UAlbertaALTLab/cree-intelligent-dictionary/issues/346

... one does not get the Cree dictionary entry yâhyânam among the many results, even though that is for certain in the XML source, cf.
<e>
   <lg xml:lang="eng">
      <l pos="English">swim</l>
   </lg>
...
   <mg>
   <tg xml:lang="crk">
      <trunc sources="CW">s/he swims; s/he swims in s.t.</trunc>
      <t rank="15.1566" pos="V">yâhyânam</t>
   </tg>
   </mg>
...

@Madoshakalaka Here's the details of one of the issues I had run into, where we're not getting the match that we would want.

@aarppe
Our current descriptive-analyzer.hfstol file does not recognize yâhyânam, thus this word is not imported into the database when building it.
yâhyânam
yâhyânam        yâhyânam        +?

The latest .hfstol in #376 do recognize it
$ hfst-optimized-lookup newest.hfstol

yâhyânam
yâhyânam        yâhyânam+V+AI+Ind+Prs+3Sg

I'll rebuild the database when #376 is merged

Ok - other example cases are (vocative forms that are possible for [some] kinship nouns):
mosom
mosom	nimosôm+N+A+D+Voc+Sg	0.000000

nitanis
nitanis	nitânis+N+A+D+Px1Sg+Sg	0.000000

There are also some changes in the FST that have ramifications on the paradigms - namely the Unspecified actor forms for VAI verbs - so that is a further reason using the latest FST.

Ideally, this type of FST update should be doable with a simple invocation, every time that we have a new version of the FST that we've tested and want to make use of in itwêwina. Nevertheless, we do not want such FST updating to happen automatically, but always with on human initiative (this has its own issue #257).

When this is fixed, might it end up resolving also #369?

@Madoshakalaka Please do note @eddieantonio's observation in #376 about the new FST apparently breaking something in itwêwina?

The search with swim now matches with yâhyânam, which is the desired result.
@Madoshakalaka However, yâhyânam has two English definitions which are exactly the same.

Might that be due to some remnant from the old version of the dictionary database?


[BEAM-1088] Add missing job parameter to the submit_job_description.

Tested post commit test locally.

LGTM


Fix verify-gofmt

Signed-off-by: Chuck Ha<EMAIL_ADDRESS>What this PR does / why we need it:
FOUND! Thanks @ncdc for reporting this one. This has been bugging me for a while.
@neolit123 might want to be aware of this one and see if it exists elsewhere.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #70
Special notes for your reviewer:
After this first push I am going to have to actually fix the tests here as well.
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
/assign @ncdc
Release note:
NONE

/lgtm

Although I bet the CI job fails now!

/lgtm

yep that's a bug.
will update in the etcdadm and kinder repos.


[SPARK-34674][CORE][K8S] Close SparkContext after the Main method has finished

What changes were proposed in this pull request?
Close SparkContext after the Main method has finished, to allow SparkApplication on K8S to complete.
This is fixed version of merged and reverted PR.
Why are the changes needed?
if I don't call the method sparkContext.stop() explicitly, then a Spark driver process doesn't terminate even after its Main method has been completed. This behaviour is different from spark on yarn, where the manual sparkContext stopping is not required. It looks like, the problem is in using non-daemon threads, which prevent the driver jvm process from terminating.
So I have inserted code that closes sparkContext automatically.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manually on the production AWS EKS environment in my company.

Can one of the admins verify this patch?

ok to test

Thank you, @kotlovs .
cc @mridulm , too

Test build #137764 has started for PR 32283 at commit 814e391.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42292/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42292/

Jenkins also passed Java/Scala/Python tests and currently running irrelevant R test.
Merged to master/3.1. Thank you for your patience and persistence, @kotlovs !

Test build #137764 has finished for PR 32283 at commit 814e391.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137764/

I think we need a config to control this feature otherwise we run into problems on spark job servers running on K8S.
For details please check https://github.com/apache/spark/pull/39775


[REQUEST] Darkmode AD Explorer

Describe the solution you'd like
Add adaptive dark mode to AD Explorer

Starting on this

Added in 1.1 branch


moving the bundle tutorials to tutorials/bundle

Signed-off-by: Wenqi Li<EMAIL_ADDRESS>moving the content to a more visible folder
Status
Ready
Checks

[ ] Notebook runs automatically ./runner [-p <regex_pattern>]

Looks good to me! Maybe I should also transfer east-to-integrate tutorial in this bundle folder?

Hi @KumoLiu ,
No, please put your tutorial into tutorials/model_zoo/.
Thanks.


Update build.ps1?

It looks like this build.ps1 file is getting out of sync with the latest version?

fixed in #58


PoC for blocking TunnelVision

Task/Issue URL:
Tech Design URL:
CC:
Description:
Steps to test this PR:
1.

Definition of Done:

[ ] Does this PR satisfy our Definition of Done?

Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Closing this one as we used a different approach to address TunnelVision.


[RESOLVED] High ICMP ping response time

Problem started at 12:09:14 on 2023.07.14
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 750ms
Original problem ID: 32644
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23132&eventid=32644

Problem has been resolved in 4m 0s at 12:13:14 on 2023.07.14
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 32644


fix: minor sonar issues

Description

Some small sonar issues that have been flagged

:tada: This PR is included in version 1.17.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Setting up testrail-promise for e2e tests

Hi, I'm trying use testrail-promise for e2e tests.
My conf.js file looks something like this:
`const TestRail = require("testrail-promise");
let resultLeaker = {
suiteStarted: function (result) {
jasmine.results = {suite: result};
},
specStarted: function (result) {
jasmine.results.spec = result;
}
}
exports.config = {
specs: ['./**/login.e2e-spec.ts'],
capabilities: {
'browserName': (process.env.BROWSER_NAME || 'chrome'),
'version': (process.env.TEST_BROWSER_VERSION || 'ANY'),
'chromeOptions': {
args: ['--no-sandbox', '--test-type=browser', 'disable-infobars', '--start-maximized'],
prefs: {
download: {
prompt_for_download: false,
directory_upgrade: true,
default_directory: full_path
}
}
}
},
maxSessions: 1,
framework: 'jasmine2',
jasmineNodeOpts : {
showColors: true,
defaultTimeoutInterval : 300000
},
allScriptsTimeout: 120000,
baseUrl: 'url',
params: {
tr: new TestRail("host","user","password")),
},
// Setup the report before any tests start
beforeLaunch: function () {
return new Promise(function (resolve) {
  Reporting.beforeLaunch(resolve);
});
},

onPrepare: function () {
jasmine.getEnv().addReporter(Reporting.onPrepare());
jasmine.getEnv().addReporter(resultLeaker);
require('ts-node').register({
project: 'e2e/tsconfig.e2e.json'
});
browser.driver.manage().window().maximize();
browser.driver.get(browser.baseUrl);
},
// Close the report after all tests finish
afterLaunch: function (exitCode) {
return new Promise(function (resolve) {
Reporting.afterLaunch(exitCode, resolve);
});
}
};
My login.e2e-spec.ts has the aftereach block:afterEach((done) => {
const obj = {
'project_name': browser.params.projectName,
'run_name': 'Login',
'section_name': jasmine.results.spec.fullName
.replace(' ' + jasmine.results.spec.description, ''),
'suite_name': 'Login',
'title': jasmine.results.spec.description,
'status_name': (jasmine.results.spec.failedExpectations.length === 0 ? 'passed' : 'failed'),
'type_id': '1',
};
browser.params.tr.handleJasmineResults(obj, jasmine, browser.params.tr, done, browser);
});`
However, I keep getting TS2339: Property 'results' does not exist on type 'typeof jasmine'. Any comments on how to resolve it for e2e tests?

What version of Jasmine are you using?

If you are still having this issue please reopen this ticket.

Hey, did you find a solution for this. please provide it.

@iamtheitguy12 I did not. Do you have any more information?
It could be that the jasmine object doesn't like adding other objects onto it. So maybe in the resultLeaker object functions don't use jasmine.results.
It might be something to do with the onPrepare function not liking jasmine.getEnv().addReporter(resultLeaker); Try moving that just above the describe line in a test file and see if that solves the problem.


[Bug] Inoreader: Can't add more feed because credentials are invalid

Reproduce:

Add one feed for inoreader
a pop-up shows and say can't add feed

I tried to remove account and add again or just update credentials but no luck
https://github.com/Ranchero-Software/NetNewsWire/assets/41150432/58d3ade7-3dec-4cb3-803e-a16b4e4d1795

The issue is because I already have 150 feed, it is the maximum for free plan
Maybe we can have a better error msg and a counter on left bar, just to check how many feed we are following :)


Grabber replaces forward slash sign with backslash sign when it passes command to an application.

What steps will reproduce the problem?
1)Enter an appropriate command containing forward slash sign in the "Image" box in the "Commands" section of the options.
2)Start image download.
3)Check log.
What is the expected behavior? What do you get instead?
Grabber is expected to pass a command to an application as it is, without any modifications, with the exception of variables(%all% for example). But for some reason it replaces forward slash with backslash.
The command that is set in the "Image" box:
curl -F "user[name]=User" -F "user[password]=1234" -F "post[tags]=%all%" -F "post[rating]=%rating%" -F "post[file]=@%path%" localhost:9000/post/create
Command execution part from the log:
[05:45:45.108] Execution of "curl -F "user[name]=User" -F "user[password]=1234" -F "post[tags]=1girl artist name bangs blurry bokeh brown hair chromatic aberration close-up depth of field eyes closed face guweiz hair over eyes highres lips long hair nose original realistic scarf solo sunlight swept bangs text upper body watermark" -F "post[rating]=safe" -F "post[file]=@C:\476\c2519761d59bed31ea588976200c4359.jpg" localhost:9000\post\create"
[05:45:45.529] Images download finished.
[05:45:46.738] Batch download finished

As you can see the original "localhost:9000**/post/create" was replaced with "localhost:9000_post_**create", thus cURL is unable to upload an image to the imageboard.
What version of the program are you using? On what operating system?
Windows 8.1 9600
Grabber 4.6.0
P.S. By the way where can I find full list of variables(%path%, %all% etc) that Grabber accepts? It seems that only some of them are mentioned in WIKI(Available tokens).

Indeed, pretty problematic.
I'll start working on a fix.

P.S. By the way where can I find full list of variables(%path%, %all% etc) that Grabber accepts? It seems that only some of them are mentioned in WIKI(Available tokens).

All of them but %path% are in this list. %path% is mentioned in the Commands wiki.

@Bionus

All of them but %path% are in this list. %path% is mentioned in the Commands wiki as it can only be used in that context.

Thanks for the information. There is one more thing I would like to know, is it possible to somehow exclude several tags from variable %allo%? You see, Myimouto and Moebooru both seem to support tag types(general, artist, copyright, character, circle, faults), for example **artist:**guweiz **copyright:**kill_la_kill **character:**matoi_ryuuko etc, and I want to pass them to cURL. But when I use %allo% I am unable to specify types for the tags that need them and if I try to use something like "post[tags]=artist:%artist%  character:%character% %allo%" artist and character tags will be duplicated with general type and and with specific type. And if it is not possible it would be great if you could implement such a feature in the future.  Plus it would be great if there were more variables without removal of forbidden symbols, afaik as it is now there is only %allo%.  Without this, variable %character% in case of a character with the name consisting of more than one word will be passed as two words divided by space to cURL, and during import this two words will be treated as two different tags.
Also, I noticed that Grabber sometimes adds symbols that shouldn't be there to the tags when variable %allo% is used.
curl -F "user[name]=User" -F "user[password]=1234" -F "post[tags]=>_ 1girl animal_ears black_panties blonde_hair bunny_ears elbow_gloves eyes_closed flat_chest gloves gradient gradient_background guweiz hairband highleg highleg_panties hug kantai_collection lips long_hair midriff miniskirt nose panties rensouhou-chan shimakaze_(kantai_collection) skirt smile striped striped_legwear thighhighs underwear yellow_eyes" -F "post[rating]=safe" -F "post[file]=@C:\00\01b39bd2012ba5e6f2e371108e4e9965.png" localhost:3000\post\create
In the example above before the tag batch begins there is ">_ " (right angled bracket, underscore, space), and this sequence of signs made it impossible to import the picture.  Is it normal? This picture can be found by entering two tags - guweiz and kantai_collection in Grabber when gelbooru.com is chosen as the source.

In the 4.7 Release Bionus has added adittional tag options. Once that is released you will be able to use %all:includenamespace,unsafe% to properly use tag types. As for the >_ tag there, what source are you using? I haven't seen that issue myself.

There is one more thing I would like to know, is it possible to somehow exclude several tags from variable %allo%?

The %all% variable does not contain duplicates. However, depending on your settings, it may or may not use different separators or underscore settings. If you are set-up properly (keep underscores and with a   separator), %all% and %allo% have the same behavior, except that %all% does not contain duplicates.
Another option would be to wait for the includenamespace and unsafe options (coming in 4.7.0, see the Filename wiki page) if MyImouto supports tag format "artist:guweiz copyright:kill_la_kill character:matoi_ryuuko".

Without this, variable %character% in case of a character with the name consisting of more than one word will be passed as two words divided by space

There's a setting to keep underscores in tags (instead of replacing them by blank spaces) in the option, called "Replace blanks by underscores" (in the "Save" category).

In the example above before the tag batch begins there is ">_ " (right angled bracket, underscore, space)

This looks like the >_< tag, except the right eye is missing for some reason, I'll investigate.

this sequence of signs made it impossible to import the picture

How so? Is MyImouto failing to create the image or cURL doesn't process the query? (I suppose it's the former)

@Flat

As for the >_ tag there, what source are you using? I haven't seen that issue myself.

I am using gelbooru.  There is a direct link to the picture in Bionus's post above. And here is how it looks imported to local Gelbooru:

And when you click on this tag Gelbooru can't find anything (standard message "Nobody here but us chickens!").
@Bionus

%all% and %allo% have the same behavior, except that %all% does not contain duplicates.

Do I understand correctly that %allo% can contain duplicate tags and %all% can not?

Another option would be to wait for the includenamespace and unsafe options (coming in 4.7.0, see the Filename wiki page) if MyImouto supports tag format "artist:guweiz copyright:kill_la_kill character:matoi_ryuuko".

Great!
Yes MyImouto supports the tag format above. AFAIK, its supported tag types are: general, artist, copyright, character, circle, faults.

There's a setting to keep underscores in tags (instead of replacing them by blank spaces) in the option, called "Replace blanks by underscores" (in the "Save" category).

Thanks, I will carefully read the manual.


documentation on quick start

This would be walking through each step of the tool.  Should be for CLI and Website   this should include the params and everything in the readme.

adding Michael on this as well, as he is looking to help with docs when he starts building apps


#4 Delete video card

Value Proposition
As a user
I want to delete videos in my feed
So that I won't see them in my feed anymore
Description

Acceptance Criteria

[ ] An delete button is on every video card at the top-right corner
[ ] On click on the delete button, the video card will be deleted
[ ] The data for the video card will be deleted as well

Tasks

[ ] Add a delete icon at top-right of every video card
[ ] Add onClick function to delete the video card
[ ] Write test to check if the video card is deleted after clicking on delete button

Complexity
M

Hy Vlady,
clean and simple. 😃
cheers
R.

Hey Vlady,
Well done! Just one comment: I still do not know what the little box at the bottom of the video card is supposed to be 🤔 maybe you can make that more clear in your scribble.
In your acceptance criteria you do not need to write what is already visible in the scribble (e.g. An delete button is on every video card at the top-right corner). Maybe it would be also nice if the video is not deleted instantly. When the user clicks on the delete button, a modal could open where the user needs to confirm deleting the video or he/she can abort the process. 🙂


Detecting When MongoDB Server is down

I was wondering what the recommended way of detecting/dealing with a mongoDB stop/start. I must admit mongodb is pretty stable but I need to still consider what to do. I call a reinit func to restart when I detect an Error .notConnected but this is complicated by the way MongoKitten behaves, if I cannot get a reconnection on the first error.

Some observation of 4.1.2 behaviour.
In my tests I get a  .notConnected Error  after trying to insert some documents the first time after mongodb was stopped.
But the second time I try inserting some documents I get .cannotSendData MongoSocket.Error
This enum is not public so is difficult to catch, explicitly.
The database.server.isConnected still returns true. But I can see that the number of openConnections is 0.
I have added a hasOpenConnection getter but maybe it should just include the $0.openConnections > 0 in the isConnected getter.  I do not understand why there is an array of hosts (I only have one)  so am a bit wary of changing it.
I also added public to the enum MongoSocket.Error
But maybe I'm missing something?
=====
/// Are we currently connected?
public var isConnected: Bool {
guard connections.count > 0 else {
return false
}
    return self.servers.contains(where: { $0.online && $0.isPrimary })
}

public var hasOpenConnection: Bool {
    guard connections.count > 0 else {
        return false
    }

    return self.servers.contains(where: { $0.openConnections > 0 })
}

This is an old issue that I didn't get to yet. With MongoKitten 4 and earlier versions the architecture wouldn't allow us to tackle this. I can add support in MongoKitten 5 by reading the pool array and its count.
For lazily connecting clusters I can imagine the following API:
enum ConnectionStatus {
  case connected, disconnected
  case unknown // Used when still connecting or disconnecting
}

let connectionStatus = cluster.status

Poking @Obbut for thoughts.


Feature Request: Remember Rows per page selection

Is this a bug or a feature request?
Feature Request
What did you expect?
When the Rows per page is changed from the default of 10 that the setting is retained on the same browser session. Would this be possible using a cookie or the local storage apis?

Yes, this would be a good one and should be fairly simple as all tables are using the same React component. https://github.com/brocaar/lora-app-server/blob/master/ui/src/components/DataTable.js#L17
This could work the same as the organization id storage: https://github.com/brocaar/lora-app-server/blob/master/ui/src/stores/SessionStore.js#L46.


stages/rhsm.facts: store osbuild version

This expands the rhsm.facts stage to always write the osbuild-version used to build the manifest to the RHSM facts file.

Closing this one until further notice, changes like these need proper motivation.


update to 1.35

Checklist

[X] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-webservice.
It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.
Please close this pull request and remake it from a fork of this feedstock.
Have a great day!

@notestaff 🔝 Please read this message and push from a fork in future, please.


Yes
Motivation
My project is a library. It supports four languages: Golang, Java, PHP and Node.js. So I need to show code in multiple languages in my documentation.
Pitch
I already noticed the discussion here: https://github.com/facebook/Docusaurus/issues/103. But it seems to be a little hacky. I still want to know if there's any official decision on this feature or other third-party recommendations? Thanks.

There is the problem of syntax though. How do we divine code block with language tabs ? :o

A way to do it is to extend the markdown grammar to support it. Like said here, the text of the code block should be self-contained in the markdown instead of using external files. Then maybe we need to patch the markdown parser to support this new grammar. Javascript is then used to render it.
However, modifying markdown grammer is not always good. Another way is to put a placeholder like a reserved label <docusaurus-code-block-1> in markdown. Then define the code block #1 and its text in external files. The markdown parser won't need to be changed. Javascript can replace the label with actual code block controls in the HTML layer.

Hi, any update on this issue? @endiliey  @yangshun
GitBook and readthedocs already support this feature:

GitBook: https://github.com/GitbookIO/plugin-codetabs

readthedocs: https://sphinxcontrib-contentui.readthedocs.io/en/latest/tabs.html

@hsluoyz - we have a pull request in review that can add tabbing to a docs page. I imagine we could repurpose the final version of that to do something similar to what is being asked for here.

@hsluoyz Also see #103 - as part of that, we are looking to adding code tabs too.

@JoelMarcey  Thanks!
Never thought the PR will be so fast! We are really looking forward to this feature! I will test it first when the feature is ready :)

#1063 might be a better option.

Thanks for the status update! This PR is really amazing! Cannot wait to try it :) @fiennyangeln @endiliey

I believe tabbed code block is now supported on latest version

I will try it ASAP, close it now.


Issue 50: Remove bucket from Redis after deleting bucket/namespace

See #50

Would it really be that terrible to just FLUSHDB?

@steved re: FLUSHDB, if there are frequent config changes, we'd lose any rate limits stored in each bucket every time - we'd notice it more for buckets that have a very low rate-limit, like when using it to rate-limit based on IPs. /cc @corcillo
I could be convinced either way though - FLUSHDB will certainly make things much simpler.


EXPERIMENTAL: elements support

experimental Elements support

only one of Bitcoin or Elements supported(compile switch)
dynamic federation not supported
rewrite chain parameter implementation

fix: invoice creation


ValueError: No HTML body content found, please try setting the 'headless' flag to False in the graph configuration.

--- Executing Fetch Node ---
MarkupResemblesLocatorWarning: The input looks more like a filename than markup. You may want to open this file and pass the filehandle into Beautiful Soup.
soup = BeautifulSoup(html_content, 'html.parser')
Traceback (most recent call last):
result = smart_scraper_graph.run()
self.final_state, self.execution_info = self.graph.execute(inputs)
result = current_node.execute(state)
title, minimized_body, link_urls, image_urls = cleanup_html(str(document[0].page_content), source)
raise ValueError("No HTML body content found, please try setting the 'headless' flag to False in the graph configuration.")
ValueError: No HTML body content found, please try setting the 'headless' flag to False in the graph configuration.

pls verify your internet connection


Remove redundant key in env_vars.py

Removes redundant key from the set. Also reformats the comment to match other styles.

Makes sense, thanks for catching this and happy for your contribution 🤗


12th of July Updates

RAG Evaluation

100 questions
Types of questions:

60 on general trade
12 on growth/variation
28 on rankings

RAG evaluation results

Best combination tested so far: multi-qa-mpnet-base-cos-v1 (embeddings) + gpt-3.5-turbo (LLM)

Accuracy: 73%

Answers missing data: 9
Answers missing context: 14
Incorrect answers: 4

Average latency: 4.19 s

Out of the wrong answers:

24 were general questions
3 of growth (the lowest)
0 of ranking questions

We're preparing a presentation gathering the results of all approaches with more detail. Next week I'll be improving the RAG + LLM and evaluating the previous multi-layer approach with Pippo.

Fine-tune MAPE results:

Recap: Accuracy is 0% for all fine-tuned versions because numbers aren't right
Measuring the absolute percentage error by model

Measuring the median absolute % error on value qty, taking the 1.1B params model with one epoch as baseline (ft_tiny0), the error decreased on the 50 epoch trained model (ft_tiny2) and increased when increasing model size (to 7B params, ft_llama2). Sort of make sense, larger models are harder to finetune.
Fine tuning tinyLlama to produce API calls instead, the accuracy is 12%, many because it struggles with HS code numbers. Other than that, query accuracy is 89%. On the HS numbers the % mean error is about 12%. but again every time the model is query with the same question it returns a slightly different number.


fix: upgrade stacks packages

This PR exists as I ran into CI issues when updating @fbwoolf's similar-purpose PR.

@kyranjamie I think we need to get this merged some point soon? I have a lingering issue in QA that depends on the @stacks/transactions upgrade.


fix timing issue with fabric8-forge starting up before there is a valid gogs endpoint available

[merge]


Import react for new component

React is needed to be imported for every component, an import statement should be added to the new component file

@chiptus good point. Its a bit tricky to implement since you may be exporting the component into an existing file. Ill try implementing this ASAP

can look for the import statement, or otherwise if importing to existing file, don't add the statement

@chiptus yep. Also various version of import need to be taken into account. For example, Typescript users usually do  import * as React from 'react' while babel users do a regular default import.
@chiptus  wanna do a PR? Ill be glad to help

I'm also interested in this, I can also participate to the PR if needed.

I love the initiative! Ill be super glad to help out and point you guys in the right direction.
I'd start here: https://github.com/wix/vscode-glean/blob/7d6791f9595bb9be4e5619f566c7c67276599d87/src/code-actions.ts#L25
This is baiscally the implementation of the "extract to component" refactoring, and the part where extracted component is added to the target file

Managed to get a prototype working.
code-actions.ts
    await appendSelectedTextToFile(selectionProccessingResult, filePath);
    await prependImportsToFileIfNeeded(selectionProccessingResult, filePath);

    const destinationImports: string = getImports(selectionProccessingResult.text);
    prependTextToFile(destinationImports, filePath);

get-imports.ts
import { jsxToAst } from "./modules/jsx";
import { allText } from "./editor";
import traverse from "@babel/traverse";
import { File } from "@babel/types";
import { transformFromAstSync } from "@babel/core";

export default function getImports(code: string): string {
  let imports: Array<string> = [];
  let sourceAst: File = jsxToAst(allText());
  sourceAst.program.body = sourceAst.program.body.filter(bodyItem => bodyItem.type === "ImportDeclaration");

  const visitor = {
    JSXIdentifier(path) {
      console.log(path);
      console.log(path.node.type);
      if(path.container.type === "JSXOpeningElement") {
        imports = [...imports, path.node.name];
      }
    }
  };

  const ast: File = jsxToAst(code);

  traverse(ast, visitor);

  sourceAst.program.body = sourceAst.program.body.filter(imp => {
    imp.specifiers = imp.specifiers.filter(spec => imports.includes(spec.local.name));
    if(imp.specifiers.length > 0) { return true; }
    return false;
  });
  const importsFromSource: string = transformFromAstSync(sourceAst).code;
  return importsFromSource;
}

@williamluke4 Thanks alot! Im having a bit of a hectic week. Ill have look at it ASAP

@williamluke4 i havent forgotten about this! Its been a bit hectic. Ill get to reviewing this week! Thank you!

hey @williamluke4, we are back in business :) Can you please create a branch?
AFAIK, you want to import all missing imports, not just React?

BTW, there are several nuances that we needed take into account:
When Glean creates a stateful component, it extends Component. So we need to check if there is already an existing import and either add to it (in case the file already contain another component) or add a new one
@williamluke4

Hey @borislit Will have a go this evening

@williamluke4 can you create a branch? Thanks


Out-GridView throw exception for objects with property names containing unbalanced brackets or parenthesis

In PowerShell 7.0.0-preview.6
I was testing some more with Out-GridView and the problems with the columns in #11182 and #11184 and I found these other chars that in some combinations causes Out-GridView to throw and exception.
The examples here are all valid property names and work fine with Format-Table or Format-List so they should also be working for Out-GridView.
It's almost as if it is trying to evaluate the property name or something.
I tested multiple combinations of brackets and parenthesis to find what caused the different exceptions (Code is at the end).
ColumnName             Result  Exception
----------             ------  ---------
Control                Success
[BalancedBraces]       Success
[UnbalancedBraces      Failure Syntax error in PropertyPath 'Unmatched bracket '[UnbalancedBraces'.'.
UnbalancedBraces]      Failure Syntax error in PropertyPath 'Syntax error in Binding.Path 'UnbalancedBraces' ... ']'.'.
Empty[]Brace           Failure Syntax error in PropertyPath 'Syntax error in Binding.Path 'Empty[]' ... 'Brace'.'.
BalancedParenthesis()  Success
Unbalanced(Parenthesis Failure Syntax error in PropertyPath 'Unmatched parenthesis 'Unbalanced(Parenthesis'.'.
UnbalancedParent)hesis Failure Syntax error in PropertyPath 'Unmatched parenthesis 'UnbalancedParent)hesis'.'.
This)(Works            Success
ButThis][Fails         Failure Syntax error in PropertyPath 'Syntax error in Binding.Path 'ButThis' ... '][Fails'.'.

Steps to reproduce
[pscustomobject]@{
    'Bracket[' = "Some Data"
} | Out-GridView

[pscustomobject]@{
    'Paren)thesis' = "Some Data"
} | Out-GridView

Expected behavior
I expected the Out-GridView window to show the object with it's property name as column name and the property value in the column like Format-Table shows the following.
Bracket[
--------
Some Data

Paren)thesis
------------
Some Data

Actual behavior
Out-GridView:
Line |
   3 | } | Out-GridView
     |     ^ Syntax error in PropertyPath 'Unmatched bracket '['.'.

Out-GridView:
Line |
   3 | } | Out-GridView
     |     ^ Syntax error in PropertyPath 'Unmatched parenthesis 'Paren)thesis'.'.

Also the Out-GridView window pops up but in a loading state, but is otherwise responsive.

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.6
PSEdition                      Core
GitCommitId                    7.0.0-preview.6
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Testing code for combinations
I wrote this to see what different exceptions would be thrown
$teststrings = @(
'Control'
'[BalancedBraces]'
'[UnbalancedBraces'
'UnbalancedBraces]'
'Empty[]Brace'
'BalancedParenthesis()'
'Unbalanced(Parenthesis'
'UnbalancedParent)hesis'
'This)(Works'
'ButThis][Fails'
)

$teststrings | ForEach-Object {
    $obj = [pscustomobject]@{
        $_ = "Test"
        ColumnName = $_
        Result = ""
    }

    try {
        $obj | Out-GridView
        $obj.Result = "Success"
    } catch {
        $obj.Result = "Failure"
        $obj | Add-Member -NotePropertyMembers @{
            Exception = $_
        }
    } finally {
        $obj
    }
} | Select-Object -Property ColumnName,Result,Exception

The cmdlet was included in PowerShell 7.0 for backward compatibility only. No plans exist to improve something in the code. All new work will be in https://github.com/PowerShell/GraphicalTools


WIP: Update to the prerelease 3.15 version of spec

@mholo65 @TylerLeonhardt
This is a huge WIP based on the 3.15 spec https://microsoft.github.io/language-server-protocol/specifications/specification-3-15/

I might be breaking the build for the moment... :shipit:


CloudQuery Source Plugin?

Hi Team, hopefully this is right place to ask, if not, I'd appreciate if you can direct me.
I'm the founder of cloudquery.io, a high performance open source ELT framework.
Our users are interested in a Binance plugin, but as we cannot maintain all the plugins ourselves, I was curious if this would be an interesting collaboration, where we would help implement an initial source plugin, and you will help maintain it.
This will give your users the ability to sync Binance APIs to any of their datalakes/data-warehouses/databases easily using any of the growing list of CQ destination plugins.
Best,
Yevgeny

Thanks for the feedback, unfortunately we don't have enough resource to build plugin for all third party platforms.
However please let us know if you need help during building it and also it's recommended to open a topic at https://dev.binance.vision for this discussion.


HDDS-8039. Allow container inspector to run from ozone debug.

What changes were proposed in this pull request?
The KeyValueContainerMetadataInspector currently can be run by the option
-Dozone.datanode.container.metadata.inspector=inspect on datanode startup.
In this JIRA, we will allow to run with the ozone debug command.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8039
How was this patch tested?
Will add some new tests.

@errose28 , thanks a lot for reviewing this!  Yes, added some new tests.

I think in a follow up we might want to remove the code that can run this on datanode startup and just have it run from CLI for simplicity. Also, it looks like this CLI version only supports inspect mode. What's the plan for supporting repair?

Do you mean removing the option -Dozone.datanode.container.metadata.inspector=inspect for running the KeyValueContainerMetadataInspector on datanode startup?  Sounds good.
We may add a new command
ozone repair <subcommnads>

for running it in the repair mode.

@tanvipenumudy can you please take a look as well?

Thank you @szetszwo, LGTM +1

Thanks @szetszwo for the patch, @duongkame, @errose28, @sumitagrawl, @tanvipenumudy for the review.


Missing information about total size limit for a standard namespace

I was not able to find information about total size limit for a standard namespace. There is only information abut 1TB total size limit for a premium namespace.

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

ID: 17b26254-ce8c-76eb-c3e9-b80c3ecc45ba
Version Independent ID: fc16f0d4-dae7-8494-e25a-6c5398215b4b
Content: Microsoft Azure Service Bus quotas and limits - Azure Service Bus
Content Source: articles/service-bus-messaging/service-bus-quotas.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

@emulic
Thanks for your feedback! We will investigate and update as appropriate.

#please-close


utf8 CharacterEncoding - needed for deplay?

The current deplay script mentions the need to use the utf-8 CharacterEncoding - was it a necessity for the file to load after "deployment" or is it just a momentarily requirement for the copy?

[ ] does it stilll work even if ignored?
[ ] use sub-package-loading to load the original package with proper encoding by using a load.m that only uses encoding-invariant characters.

How did I do it in the past? Ofcourse only added
$CharacterEncoding="UTF-8"
into $UserBaseDirectory/Kernel/init.m

An automatic installation would include:

Has this installer not been used yet
Does the System support UTF-8
Is the SystemEncoding different from Encoding (= is there already a different encoding command in use?)
Append $CharacterEncoding = bla bla to $UserDirectoryBase/kernel/init.m
Quit[], so that the changes will be loaded on next execution

The thing is, on some systems, like on my partners mac laptop, the system character encoding is set to utf8, which does not require a manual entry into the user space kernel.m, but I need to on my windows. Not really a blocker really


Validate function omits callbacks error object.

During implementation if the validate function for my app, i notice that if i pass an error object back, regardless of what that error object is, it is not passed correctly to the Hapi auth handler callback.
var validate = function (decoded, request, callback) {
//    return callback("not nothing", false);
//    return callback({error: "Something TERRIBLE happened"}, true);
};

in the above case, the  first argument !null and should therefore be sent back to the handler for it to send it to the client.
See here: http://hapijs.com/tutorials/auth
The problem is here:
http://puu.sh/hgxHO/feae74cce8.png
As you can see the Boom.unauthorized is being sent regardless if we've supplied our own error object.
Possible (NOT PERFECT) fix:
http://puu.sh/hgyh7/8a5648275f.png

Hi @Think7 thanks for letting us know this was causing you grief.
The docs say we should use Boom to construct errors returned in the reply: http://hapijs.com/tutorials/auth#authenticate and we were keeping the message standard for simplicity.
But if you prefer to have the specific message that was returned from your validateFunc, would this be a good middle ground:
if (err) {
  return reply(err, null, { credentials: credentials });
}

Looks good!

Done. see 4.3.0 on NPM: https://www.npmjs.com/package/hapi-auth-jwt2
@Think7 thanks again for drawing our attention to this.
Please star this repo and follow to show our fellow Hapi people this module is loved. thanks!


Profile Gravatar image won't load

Description
When I access the Gitea profile, it takes a while to load the image. At the end of loading it shows the default image. Look:

I've already tried to access Gitea directly by IP to escape the Reverse Proxy. The problem continued.
Settings:
Environment variables in Docker:
...
GITIA__picture__DISABLE_GRAVATAR="false"
GITIA__picture__ENABLE_FEDERATED_AVATAR="true"
...

Gitea's app.ini (/etc/gitea/app.ini) file after creating the container:
...
[picture]
AVATAR_UPLOAD_PATH = /var/lib/gitea/data/avatars
REPOSITORY_AVATAR_UPLOAD_PATH = /var/lib/gitea/data/gitea/repo-avatars
DISABLE_GRAVATAR = false
...

Gitea Version
1.20.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
I use the official image for docker. Currently the image used is 1.20.3-rootless.
Database
MySQL/MariaDB

You enabled gravatar
DISABLE_GRAVATAR = false

So maybe you can open you chrome dev console to investigate whether your network is slow to visit Gravatar.

See some of my profile settings:

Note: I hid the real email address and put an example address.

Profile image loading test using Reverse Proxy.

Profile image loading test without using proxy.

Below I show more information about the test that did not use a proxy. Access is via IP and port. Example: http://IP:8084.

When I access the Gitea profile, it takes a while to load the image.

It means that ENABLE_FEDERATED_AVATAR="true" service fails.
I guess you don't need it.


App should not ask user to select backup directory if skip backup option was chosen

Description
App asks user to select backup directory even if skip backup option was chosen
Expected Behavior
Should not prompt user to select backup directory.
Actual Behavior
Prompts user to select backup directory
Steps to Reproduce
Create a local wallet
Context
Onboarding

Fixed via https://github.com/LN-Zap/zap-desktop/pull/2982