Improve connect header and navigation

There are a couple snags I run into when navigating around Connect, and these observations come from using app.blockstack.org:

From the index, if I click Settings then it shows my usernames, but there is no option to go back.

On that same page, hovering over a username looks like I can click it, but clicking a username does not appear to do anything. It would be great if this could expand to show additional info such as the full ID or similar.

After clicking View Secret Key from the settings page, it was not immediately clear that clicking Copy Secret Key would take me back to the settings page. It would be helpful to have a back button, or alternatively, a Settings link at the top-right to match the format of the Choose an account page.

This could be important especially if the user is on a public device and decides they do not want to copy the secret key for some reason.
Also, after clicking Copy Secret Key, I'm taken back to the settings page with no way back to the Choose an account page.

One easy solution could be to keep the icon / app name at the top-left and link it to the choose account page, then use the top-right to change options based on context.
A quick mock-up is below, and this would avoid the need for a back button as well as make the top header for each page consistent.

cc @jasperjansz since he's currently working on designs that could impact this screen and incorporate the feedback here


Updates support options, adding StackOverflow, fixes #872

The Problem/Issue/Bug:
OP #872 mentions adding StackOverflow links to docs, this PR adds those references.
How this PR Solves The Problem:
Manual Testing Instructions:
Release/Deployment notes:

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

@andrewfrench We often do as Rick has done here, approving a PR with some suggestions, which means that we're asking for simple followup commits to be pushed in by the author, but they don't need further review.


chore: Publish all javascript packages using bash script

Description
The set of scripts allow you to build and publish all javascript packages.
Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Failing tests are not related to changes made here

Closing this as #3029 has been merged.


chore: clean flag

Since the introduction of transformation rules, this logic is no longer necessary. The skipForeignNode flag was undocumented before, making its removal straightforward.
Pull Request

What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[x] you linted your code (make lint)
[x] you linted your code (make unit)

See make help for a description of the available targets.

/m


Fossilize: Fix multiple crash and bad exports

Description
Each change address a different issue that resulted in fossilize exports that either crashes or produce unintended results.
The code are tested, but I'm still not confident about the code structure:

For foz: Encode sampleMask as array, is it a good idea to put the type conversion right inside ConvertSDObjectToFossilizeJSON?
For foz: Reconstruct specialization constants with correct layout, should we instead make the SD reader export the spec constant buffers so we can export the pipeline byte-accurately?

I'm sorry for the delay, but I've pushed a revision to address the comments. I've also added a change to remove the sorting of spec const reflection variables — in the current code they are sorted by construction and an explicit code confuses the reader.


fix: accidental ownership with specialization

The old compiler tried to avoid borrowing that could inhibit mutual TCO, but the implementation was wrong and in an unnecessary way:

ownVar is called but it can only mark variables of the current function, so the wrong variable was marked
there was no mutual recursion as the additional declaration came from specialization
the old compiler only supports TCO for self-calls anyway

Thus we restrict the borrow inhibition to self-calls. More extensive TCO could be considered in the new compiler.
Fixes #4240

!bench

!bench

We deemed the parser-specific slowdown (which effectively must be random chance given that the removed heuristic did not do what it intended to do) acceptable in exchange for a clear fix to the code and a modest overall stdlib boost.


SyncBot: Squash support rooms

SyncBot should be able to support monitoring both community and premium support workflows in one flow.

Duplicated by #297


Gateway MVC: Path seems not passing through the request body

Describe the bug
Path seems not passing through the request body, for instance:
spring:
  cloud:
    gateway:
      mvc:
        routes:
          - id: transcoder
            uri: http://localhost:8089
            predicates:
              - Path=/transcoder/v1/**

http://localhost:8089 is a Wiremock server. The following is the Wiremock logs:
                                               Request was not matched
                                               =======================

-----------------------------------------------------------------------------------------------------------------------
| Closest stub                                             | Request                                                  |
-----------------------------------------------------------------------------------------------------------------------
                                                           |
POST                                                       | POST
[path] /transcoder/v1/break                                | /transcoder/v1/break
                                                           |
{                                                          |                                                     <<<<< Body is not present
  "active" : true                                          |
}                                                          |
                                                           |
-----------------------------------------------------------------------------------------------------------------------

And there is the following exception:
java.io.EOFException: EOF reached while reading
	at java.net.http/jdk.internal.net.http.Http2Connection$Http2TubeSubscriber.onComplete(Http2Connection.java:1587) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadSubscription.signalCompletion(SocketTube.java:648) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.read(SocketTube.java:853) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowTask.run(SocketTube.java:181) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$SchedulableTask.run(SequentialScheduler.java:207) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:280) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:233) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.signalReadable(SocketTube.java:782) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadEvent.signalEvent(SocketTube.java:965) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowEvent.handle(SocketTube.java:253) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.handleEvent(HttpClientImpl.java:1467) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.lambda$run$3(HttpClientImpl.java:1412) ~[java.net.http:na]
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) ~[na:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.run(HttpClientImpl.java:1412) ~[java.net.http:na]

Sample
The following is a sample where you can find a test GatewayissueApplicationTests that is failing.

i alse meet the same problem, my springboot version is 3.2.0 and the gateway version is org.springframework.cloud:spring-cloud-starter-gateway-mvc:4.1.0

the sample was missing a @RequestBody, though the issue is still valid.

I just tested the sample with implementation("org.springframework.cloud:spring-cloud-starter-gateway-mvc:4.1.1") and it is still failing

the sample was missing a @RequestBody on the body parameter

I don't follow, where exactly should I put the @RequestBody on the body parameter? The sample code has:

GatewayissueApplication main entry point
GatewayissueApplicationTests test
application.yml gateway config file

There is no @RestController or similar there. Maybe we are confusing this sample with other issues?

I'll look after the holidays

I face similar problem when using ProxyExchange from the spring-cloud-gateway-mvc. The use case is the following - we are just passing the @RequestBody parameter from one service to another via ProxyExchange. But seems the payload is empty.

Any updates on this issue?

looking now

If I remove rest-assured and wiremock and send it to httpbin, the body is sent just fine.

Anyone else able to recreate the issue without wiremock?


Sample wanted for Kotlin/kotlinx.html

About the library

Description:   Kotlin DSL for HTML
GitHub: https://github.com/Kotlin/kotlinx.html
Official website: https://github.com/Kotlin/kotlinx.html

What the sample could do
Generate the HTML for an email
Wants to contribute the sample?

Say so in a comment here
Follow instructions in CONTRIBUTING.md
Submit your pull request!

Would love to contribute with this :)

@dector Do it! 😃


x/gamm: Add check for correct input asset denoms to CalcJoinPoolShares

Closes: #1906
What is the purpose of the change
Our current implementation of CalcJoinPoolShares checks to make sure the number of denoms sent in for an all-asset join is correct, but has no check for whether those assets are actually the correct denoms. This makes it so that entirely invalid inputs (i.e. correct number of input tokens but incorrect denoms) successfully make it through the entire set of all-asset join calculations until they are incidentally caught in the single-asset join section due to a check in updateIntermediaryPoolAssetsLiquidity.
https://github.com/osmosis-labs/osmosis/blob/85529c76683f2d2f2ffb2121125baf0e39463f84/x/gamm/pool-models/balancer/amm.go#L368-L370
This PR adds an additional check to the input tokens that makes sure they are of the correct denoms so that the function never assumes invalid inputs are correct for calculating all-asset joins.
Brief Changelog

Adds a check similar to DenomsSubsetOf that makes sure that the input denoms are part of the pool being joined (Note: we cannot use DenomsSubsetOf here because we have a denom->PoolAsset map instead of a set of sdk.Coins to represent pool liquidity)

Testing and Verifying
This change is already covered by existing GAMM tests.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (no)
How is the feature or change documented? (not applicable)

LGTM

Do we have a test case already testing this path? If not, we should definitely add one.

I believe this test should cover it but happy to add another one with more assets/mixed assets if you think it would be useful:
https://github.com/osmosis-labs/osmosis/blob/2007babdee01e9bb7743ccadeff097cdab9b7236/x/gamm/pool-models/balancer/pool_test.go#L121

We should get a changelog entry in.

Added in recent commit!

LGTM

Do we have a test case already testing this path? If not, we should definitely add one.

I believe this test should cover it but happy to add another one with more assets/mixed assets if you think it would be useful:
https://github.com/osmosis-labs/osmosis/blob/2007babdee01e9bb7743ccadeff097cdab9b7236/x/gamm/pool-models/balancer/pool_test.go#L121

We should get a changelog entry in.

Added in recent commit!

Just making sure we don't miss anything - was that commit pushed? Not seeing a new test

Just making sure we don't miss anything - was that commit pushed? Not seeing a new test

This test was already there so it's not a part of this PR. The link I included was directly from main


zfile: fix build and runtime errors

As reported in #1147, the build fails if either zlib.h or lzma.h are unavailable.
Additionally, the feof check in zfile.c is backwards and causes a "truncated file" abort after searching the first zipped file.

FWIW, I addressed this issue with a simple workaround at #1146 : https://github.com/ggreer/the_silver_searcher/pull/1146/commits/bf375de7b1177df7eb08eddc13c33c2e188c81c5 .
Your fix is clearly more complete, however, it does clutter the code quite a bit.
IMHO the best fix would be to refactor zfile.c such that it doesn't need gazilion #ifdefs. Or, alternatively, take the approach in my PR and just tie fopencookie to both lzma and zlib availability (and further remove the remaining #ifdefs - which I didn't do in my PR). There is a fallback anyway, so I think this approach is not too terrible.
Code with so many #ifdefs is not fun to maintain...

Good point; I know the pain of dealing with code that's #ifdef'd to hell.
I can refactor zfile.c to put all of the zlib/lzma specific code into their own functions with a wrapper to check cookie->ctype, which would clean up zfile_read a lot.

I tried adding a layer of abstraction (2 layers of inline functions, with stubs #define'd for unsupported modes), but it might be even worse than littering zfile_read with #ifdefs.
Perhaps your solution to just require both zlib and lzma to support fopencookie is better. (has anyone  benchmarked whether fopencookie even gives a noticeable performance boost?)
The alternate version is on my zfile-fix-redo branch: aswild/the_silver_searcher@24995eb239799e52ae09d47b7520d50a3b8b606c
Regardless of the above, I think we do need some form of my second commit to fix the feof handling, that missing ! totally breaks zfile searching unless the file sizes happen to line up nicely. (aswild/the_silver_searcher@09b0bd8545d08d17ef70893fa15ae644e0bb8b2f)


Generic linked list implementation

We need it badly, now. Two major requirements are:

no memory allocation for list nodes (i.e. nodes are embedded into structures),
any structure may be easily linked by several lists.

There're a few solutions:

Linux Driver Development chapter 11.5.
FreeBSD queues

Just choose bare minimum interface and present it as a header file, don't start implementation yet. I'd go for doubly linked circular list variant - it seems to be the most versatile.

I just created the pull request being the first minimal step to implement this feature.

We need those pesky lists now - please prepare a patch ASAP :)

It'll be done today :)

I'm still waiting...


Changes Default Javascript Prompt to Custom Popup  for New Circuit

Describe the changes you have made in this PR -
I have Changed the  default Javascript Prompt to Custom Popup  for New Circuit as per #779
Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 81.774% when pulling 4ad21d8fc0ed75e4415008bf882633eb1dfab787 on Prerna-0202:NewCircuitPopup#779 into a978b46a15ad68aee6a19924b1d35bbe73eccae1 on CircuitVerse:master.

@gr455 @shubhankarsharma00, Please review my pr...


Add safeTxHash to the history/queue summaries

Context
We're using safeTxHash as an internal id on the client side. safeTxHash works well for us because it's not changing throughout the life cycle of a transaction.

to track things like Pending Status
to open individual transactions from a URL (aka tx deep-linking)

Problem
safeTxHash is present in the detailedExecutionInfo when fetching the details of an individual tx, but not accessible in the list (History and Queue).
Currently, we're relying on a dirty hack to check if a tx is pending in the list (by parsing the id field 🙈).
Feature request
Please add safeTxHash to the History and Queue endpoints. It should be on the top level of each transaction item, alongside the id field.
E.g.
{
  "id": "multisig_0x5f9ee776B85f1aFF123f935C48C528D00a62B0ad_0x204bd8ec34dad230927a095763e98844ca687e6aebc2148b38a8edcb8ca35b16",
  "safeTxHash": "0x204bd8ec34dad230927a095763e98844ca687e6aebc2148b38a8edcb8ca35b16",
  "timestamp":<PHONE_NUMBER>000,
  ...
}

@iamacook please correct me if I'm wrong – we've discovered that the id is changing throughout the life-cycle of a single transaction, whereas the safeTxHash remains the same. Hence we picked the safeTxHash.

That might be problematic for deep links that should be valid "forever", but why would this be problematic for tracking the pending state?
I think we should align here what you want to do and also explain the functionality of the id.

If I remember correctly, the safeTxHash was chosen because of the bug we encountered regarding transactions temporarily being "CANCELLED". Retrieving transactions by their safeTxHash did/does not seem to return "CANCELLED" as often.
Upon executing a transaction, the txStatus was being locally overwritten to "PENDING" and then updated according to the CGW. Hence, it was temporarily "CANCELLED" in the UI.
As we are polling for summaries, by adding the safeTxHash there, we would not need to parse the txId to determine if the transaction was successful. If the txId doesn't change during the lifecycle of a transaction, we could indeed rely on that instead for deeplinking/status tracking.
I altered the way pending statuses are tracked this week so that the state isn't mutated. It would relatively easy to pivot to using the txId instead as we now store a hashmap ({ [chainId]: Set<safeTxHash> }) of pending transactions.
There was a mobile kick-off this week in which we discussed using the txId as all transactions have them compared to the safeTxHash. There is an open issue to improve the deeplinking where this would be considered.
If the txId is persistent, I don't see a reason for our reliance on the safeTxHash now we have altered the status tracking. My only question would be, if are we alright with the long URLs or could we minify them in some way? Although, that is more related to deeplinking.

URL length doesn't matter IMHO.
I'm closing this, then. We'll use the id for both deep linking and the local pending status. Will reopen in case we notice problems with the id again.

@rmeissner, can you confirm whether the txId remains the same from transaction creation, through execution to success (for all transaction types)? I'll update the deeplinking v2/create a status tracking tweak issue accordingly.

@iamacook not sure I understand your question. We currently to not allow to fetch creation information for a Safe via the client gateway. If this is required please create a new issue and link to the epic for which this is required, so that we are aware of the flow.

Thank you for the clarification. Since my question, I have made the required alterations to not use the safeTxHash anymore.


Optionally invert oversized images

Description
Adds a debugInvertOversizedImages flag and a VM Service extension to toggle it.
If the application draws images that are more thandebugImageOverheadAllowance (which defaults to 1024, i.e. 1kb), the image will be drawn with colors inverted and filpped vertically, as well as reporting a Flutter Error (which is useful for failing tests - although if it would be better to let a golden comparator fail the test we could change that to a debug print).
/cc @liyuqian @jacob314 who reviewed a related patch to this.
/cc @zanderso @jonahwilliams for tool - I chose ! as the key for this since we're running out of letters, but could be convinced to use pretty much anything else that makes sense.
/cc @jamesderlin who was interested in this
/cc @tvolkert who suggested something like this at one point
Related Issues
Continues work in https://github.com/flutter/flutter/pull/59877
Addresses https://github.com/flutter/flutter/issues/56332 some more
Tests
I added the following tests:

Test that we invert color the image when this is set
Test that we flip it vertically
Test that we print the right help for it
Test new service extension related wiring.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

Here's what Shrine looks like with it turned on:

Should we be rendering lower resolution images in Shrine? 🙂

@tvolkert - yes.  /cc @willlarche

@dnfield We'd love that! If you can't make that fix, please create a ticket on the /gallery repo. And this would be good info for a blog post!

The color-inverted-and-vertically-filpped images don't look dangerous enough to me as I find the Shrine screenshot to look quite cool! How about adding a semi-transparent overlay with black and yellow bars (similar to those with overflow)?

@liyuqian we could do some other effect but my thought was that this should be visually obvious to someone who knows what image they're expecting to see. I'm not sure if the yellow bars would trigger confusion about layout overflows. WDYT?

I'd tend to agree with @dnfield that we should use a different signal for a different problem.

Yeah I think app developers will know what the images should look like and will notice the inverted/flipped images right away.

I see. Then let's go with the color-inverted-and-vertically-filpped images :)

So is any brave soul willing to LGTM this? :)


[Update Request]: ImageMagick.ImageMagick - SupressMsg-Property

What type of update are you requesting?
Something else not listed
Current Package Identifier
ImageMagick.ImageMagick
Package Version
<IP_ADDRESS>
Please describe the changes you would like to see
The current version of ImageMagick was re-created after been removed.
Unfortunately the suppressmsg propertiy was lost with this change:
https://github.com/microsoft/winget-pkgs/commit/1c79516c3c729bd93574e8ee4fd845c6a3e8cafd

I'm seeing the PR, hence the issue could be considered closed.


How to pass focus to component?

Thanx for the great job
I would like to pass focus to the component as common TextInput (TextInputRef.current.focus()

I found a solution
add ref prop to textInputProps (ignoring typescript warning)

@samloba can you elaborate on how you solved this issue? I would also like to programmatically pass focus to the component using refs


Audit of Fugue-sql documentation

The Fugue-sql section is roughly done, but needs a full audit. Relevant Github files are here: https://github.com/fugue-project/tutorials/tree/master/tutorials/fugue_sql
The Fugue-sql section on the docs website is here: https://fugue-project.github.io/tutorials/tutorials/fugue_sql/index.html

Closed due to staleness. There is an ongoing major update to Fugue 0.8.0, so the FugueSQL docs will be reviewed as part of that.


Click AltasTexture in theme Inspector will not unfold it

Godot version: 3.2.3 stable

OS/device including version: Windows 10 20H2

Issue description: If using AltasTexture in StyleboxTexture in theme, click the AltasTexture in the inspector will not unfold it. Check the Minimal reproduction project.

click AtlasTexture:

then StyleboxTexture will fold and can't change AltasTexture's property:

Steps to reproduce: Open new_theme.tres, and click AltasTexture in StyleboxTexture in the inspector. The StyleBoxTexture will fold.
Minimal reproduction project:

AtlasTexture.zip

Duplicate of #28341 (same underlying issue)
As a workaround, you can right-click the stylebox and select Edit.

Duplicate of #28341 (same underlying issue)
As a workaround, you can right-click the stylebox and select Edit.


Hotspots

Is it possible to add tracking hotspots ?
Something like this
https://codepen.io/pixelthing/pen/EgAJWr

@mpvlc Yes. This has been something that was on my mind before and I started working on this long ago but just couldn't finish it coz of time issues. Hopefully, I can find time sooner and work on it. It is already added to the roadmap of this package. If anyone is willing to contribute, I'd be happy to accept their pull request changes coz I don't know how long it will take for me to find a good time to work on this feature.


Upload the full-resolution poster images to the storage service

On LIB_WDL_DDS, there's a folder of poster images LIB_WDL_DDS_POSTERS. We have copies of these images in the storage service, but they're heavily downsized (because they were retrieved through the DDS poster service).
We should upload the full-resolution copies to the storage service.
You can work out which image goes in which bag by looking at the MP4 files in the Reencoded_Videos directory.
Part of https://github.com/wellcomecollection/platform/issues/4187

See Slack conversation: https://digirati.slack.com/archives/CBT40CMKQ/p1582908641064600

Related: https://github.com/wellcomecollection/goobi-infrastructure/issues/255


Add version: Helix.Helix version 22.12

- [ ] Have you signed the Contributor License Agreement?

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree


启动之后报错响应数据：request to https://api.openai.com/v1/chat/completions failed, reason: Client network socket disconnected before secure TLS connection was established

request to https://ai.fakeopen.com/api/conversation failed, reason: Client network socket disconnected before secure TLS connection was established
加了代理也是报错

request to https://ai.fakeopen.com/api/conversation failed, reason: Client network socket disconnected before secure TLS connection was established
加了代理也是报错

解决了,是科学上网的问题

开启了代理就可以了嘛

request to https://api.openai.com/v1/chat/completions failed    能拦截不让页面显示嘛

配了vscode的代理，不开全局也能跑了


PIC-2240

. Added hearing event field filter.
. Added test to verify the behaviour.
. Added jsonpath plugin
. Updated local stack docker name to avoid conflict.

Ready for the review


Bug: Issues with yarn 4.5.1

Describe the bug
after installing with CLI with yarn 4.5.1. The rest work
yarn create message-kit
Restarting 'dist/index.js'

node:internal/modules/run_main:122
    triggerUncaughtException(
    ^
Error: @xmtp/message-kit tried to access @xmtp/node-bindings, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: @xmtp/node-bindings (via<EMAIL_ADDRESS>Required by: @xmtp/message-kit@npm:1.1.7-beta.21 (via /Users/fabrizioguespe/.yarn/berry/cache/<EMAIL_ADDRESS>
    at makeError (/Users/fabrizioguespe/fa4/.pnp.cjs:9143:34)
    at resolveToUnqualified (/Users/fabrizioguespe/fa4/.pnp.cjs:10781:21)
    at Object.resolveToUnqualified (/Users/fabrizioguespe/fa4/.pnp.cjs:10961:26)
    at resolve$1 (file:///Users/fabrizioguespe/fa4/.pnp.loader.mjs:2031:31)
    at nextResolve (node:internal/modules/esm/hooks:748:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:240:30)
    at handleMessage (node:internal/modules/esm/worker:199:24)
    at checkForMessages (node:internal/modules/esm/worker:141:28)
    at process.<anonymous> (node:internal/modules/esm/worker:160:5)
    at process.emit (node:events:518:28)

Node.js v22.11.0
Failed running 'dist/index.js'

Workaround:

Use yarn 1

added
nodeLinker: node-modules


simplify UpdateClient proposal

Description

closes: #176

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration tests
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #181 (05d7e67) into main (f412334) will decrease coverage by 0.02%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
- Coverage   79.91%   79.89%   -0.03%
==========================================
  Files         107      107
  Lines        6409     6405       -4
==========================================
- Hits         5122     5117       -5
- Misses        929      932       +3
+ Partials      358      356       -2

Impacted Files
Coverage Δ

modules/core/02-client/types/proposal.go
83.92% <ø> (+1.96%)
:arrow_up:

...ht-clients/06-solomachine/types/proposal_handle.go
100.00% <ø> (ø)

...s/light-clients/09-localhost/types/client_state.go
93.91% <ø> (ø)

...ght-clients/07-tendermint/types/proposal_handle.go
80.48% <69.23%> (+2.30%)
:arrow_up:

modules/core/02-client/keeper/proposal.go
84.31% <77.77%> (-6.80%)
:arrow_down:

modules/apps/transfer/types/trace.go
80.88% <0.00%> (+0.03%)
:arrow_up:

modules/core/02-client/types/height.go
90.90% <0.00%> (+0.90%)
:arrow_up:

modules/core/04-channel/types/channel.go
77.55% <0.00%> (+4.02%)
:arrow_up:


Restore adios2 CI

This PR is fixing

the CI test io_read for ADIOS2 builds
2decomp when using the intel compiler and ADIOS2
the subroutine decomp_2d_io_fin, adios2_finalize was called at the beginning and should be called at the end


responsive site

it should work and look better on mobile and older browsers.
also: accessibility.
wham bam

Hey there! I did a pass on the mobile CSS which I can submit as a PR shortly.
As far as older browsers, how far back specifically were you thinking? The site's layout is fairly simple, so I'm curious to know if you've gotten reports of it not working well in older browsers. (Or if traffic to the site shows that the support is still needed.)
I can also look at the accessibility of the site. If I tackle all three of these things, would you like them all as separate pull requests?
💕 Mocha so I'm trying to give back in a small way. :)


docs: explain deployment.environment impact on service identity

Why
To clarify a likely misconception about service identity in the case of multiple environments.
What
Explain that deployment.environment does not impact service identity.
References
Fixes https://github.com/open-telemetry/semantic-conventions/issues/119
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] CHANGELOG.md updated for non-trivial changes.
[x] schema-next.yaml updated with changes to existing conventions.

I think this is a good description of the current state. Maybe we should also add a note that service.namespace could be used to distinguish the services. A more major change would be to make the environment part of the service identity, although IMHO we should then rename the attribute to service.environment (or service.deployment_environment?)


mergeSTR can't merge haploid data from HipSTR output

Hi,
Recently I have been processing some haploid vcf files from HipSTR output and I find I can't merge haploid data through mergeSTR, and the errors are as follows:
TypeError: ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced(line 400 in mergeSTR.py)
I know dumpSTR didn't support haploid, is mergeSTR the same way?
Best,
Weixiang

I encountered a similar problem when merging the output vcf file of expansionHunter with mergeSTR.
~tr_harmonizer.py", line 209, in InferVCFType raise TypeError('Could not identify the type of this vcf')
TypeError: Could not identify the type of this vcf
Have you solved this problem?


[TextField] Outlined renders gap with label = empty string

Regression for outlined text-field with label="".

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

(buggy is the value)

Expected Behavior 🤔

(fine is the value)
Steps to Reproduce 🕹

https://codesandbox.io/s/trusting-snow-8ses9
Steps:

Use a TextField with label=""

It works when omitting the label prop at all (or with label={undefined}).
This problem seems to occur since v4.9.0 - it works fine with prior versions!
Context 🔦

I wanted to display the input without a label and stumbled across this.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.2

React

Browser

TypeScript

etc.

I think we want to stick with string => some label, undefined => no label semantics. It's simpler that way instead of separately handling empty strings. We would also need to document these because "empty string" is not obvious.
With regards to regressions: This would also apply to label={0} which doesn't make much sense at all.
To avoid churn we could special case label === "" and remove it in v5 to avoid type overloading and two-values-doing-the-same thing API issues.

I can pick it up if no body mind


Web API Authorization Manager Error

While debugging a Web API error, I cleared my web token and used Authorization Manager processes to create a new one.
When having performed that the first time, the "input" error was cleared but a new error is logged
[1/27/2023, 9:24:53 AM] [homebridge-xbox-tv] Device: <IP_ADDRESS> XBOX, not authorized, please use Authorization Manager.
I've performed the Authorization Manager process a handful of times, with the same result.
e.g. my latest is
            {
                "name": "XBOX",
                "host": "<IP_ADDRESS>",
                "xboxLiveId": "redacted",
                "clientId": "5e5ead27-ed60-482d-b3fc-702b28a97404",
                "xboxWebApiToken": "M.R3_BAY.86b2cf91-947e-e5f7-57be-83dbda9f97e4",
                "disableLogInfo": false,
                "disableLogDeviceInfo": false,
                "enableDebugMode": false,
                "webApiControl": true,
                "volumeControl": 0,
                "getInputsFromDevice": true,
                "filterGames": false,
                "filterApps": false,
                "filterSystemApps": false,
                "filterDlc": false,
                "sensorPower": false,
                "sensorInput": false,
                "sensorScreenSaver": false,
                "enableMqtt": false,
                "mqttAuth": false,
                "mqttDebug": false
            }
        ],
        "_bridge": {
            "username": "0E:E7:D2:24:BA:D4",
            "port": 57008
        },
        "platform": "XboxTv"
    },

copied from from the Authorization Manger URL
http://localhost:8080/auth/callback?code=M.R3_BAY.86b2cf91-947e-e5f7-57be-83dbda9f97e4
I've restored the original web token using a backup, but the original token gives the error as well

Sorry but I can not reproduce this behavior, tested on 2 consoles both clear web token, generate new one all akcept saved restarted and all working correct, no any error. Check your homebridge/xboxTv/ folder for token_xxxx file.

Thanks for the lead!
Am I looking in the right place?
I do not see a token_xxxx file under the
/var/lib/homebridge/node_modules/homebridge-xbox-tv/
tree
nor under the
user "homebridge"'s home
/home/homebridge
Thanks for any counsel!

Thanks for the lead!
I found an authtoken_192168140 file in /var/lib/homebridge/xboxTv
it is zero bytes
thoughts?

So, it’s not stored, are you saved it after create, read carefully what authorizations manager display as tips.

Thanks for the suggestion!
I proceeded again though all the steps; this time taking screen shots of every step if you need them.
After completion, I checked the authtoken file. Its time stamp was updated at the moment I did the authorization followed by a save, but again, zero bytes.

The I also confirmed the new web token is in the new config. Looks like the auth file was touched during this latest authorization, but nothing written to it.

Completed the authorization process again after updating to homebridge-xbox-tv v2.4.8.
again the auth file was touched during authorization, but nothing written to it
the file is owned by user homebridge with -rw-r--r-- permissions

Can you record video of this process.?

Prior to the process with current token

Token cleared

Start authorization pushed

URL launched

Cut and pasted into field

Save button pushed

New token showing under plugin settings

Directory showing the auth file was touched during this process but zero bytes

You don’t press activate console on last screenshot

derp … Thanks. I didn't notice that button changed labels and needed to go back to the first button to do the last step LOL. I appreciate your time. Looks like the original issue's cause may have needed a re-issue of a new token. Will continue testing.

Can't find your "donate" button LOL

I don’t have donate button, so I close this issue as solved.

follow-up question:

am I looking in the correct place for "display tips on top of authorizations manager"

Yes, marked place changed during press buttons

Much appreciated—you are very gracious. Have great weekend!


Backport 2.7: Fix miscalculations of maximum record expansion in mbedtls_ssl_get_record_expansion()

This is the backport of #1915 to Mbed TLS 2.7, fixing #1914.

@mpg @k-stachowiak Please re-review.

retest

The failing CI can be disregarded, as that's my fault in reconfiguring the CI name.
This PR can therefore be considered to be passing and ready for merge.


fix: Add appname parameter

Description
My organisation uses the following format for its apps:
com.other_organisation_name.app
Currently the cli only allows for setting the organisation name, while the appname is derived from the folder name.

so when I run very_good create coolcomp --org "com.coolcomp"
the bundle id comes out as com.coolcomp.coolcomp.

What I actually want is com.coolcomp.app
To achieve this currently, I have to rename all instances of com.coolcomp.coolcomp to com.coolcomp.app.

closing in favour of #280


Convert to Swift3 (Xcode 8 beta 6)

+1 For this

@zeusent @RemyDCF I merged this pull request into swift3 branch. Feel free to use it.
I will merge this changes into master once release version is out.

Thanks!

Awesome! Thanks!

Hi, I think it is time to release the swift3 branch. Will be easier to maintain cocoapods.

@illyaBakurov done! Merged into master.
created swift2.3 branch for old version


fix: variable types for PHPStan 1.9.0

Description

PHPStan 1.9.0 reported errors.

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
[x] Conforms to style guide

Thank you @kenjis


[intel-mkl] Update to 2020.0.0

Fix #12963
Update intel-mkl to the latest version 2020.0.0
Note: No feature need to test

This looks good, any reason you still have it marked "draft" ? Thanks!

/azp run

Thanks for your contribution!


Fetch the underlying beanfactory, fixes #1657

Hey! Are you able to write a test that fails without this change?

I've pushed a couple of tests, one shows that the current behaviour would fail without the fix

@JBodkin - thanks a lot!


Apache HttpClient instrumentation

Hello,
I have working POC of Apache HTTP client 1 instrumentation. I would like to move it to contrib org. It is implemented with standard Http[Request|Response]Interceptor 2 3. It works also for httpasyncclient.
Repository name could be:

java-apache-httpcomponents - This could group all instrumentations related to Apache HC. Currently I can think only about opentracing-apache-http-interceptors artifact.
java-apache-httpclient

cc @bhs @yurishkuro @wu-sheng @objectiser

@pavolloffay fyi we have Apache HTTP instrumentation in jaeger-client-java. It uses our internal context propagation mechanism, but may be of interest.

@pavolloffay 👍👏 httpclient may be a better name. More popular I think.

@yurishkuro thanks, I completely missed it.
Here is dependency tree of jaeger-apachehttpclient.
[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ ot-sandbox ---
[INFO] sk.loffay.test:ot-sandbox:jar:1.0-SNAPSHOT
[INFO] \- com.uber.jaeger:jaeger-apachehttpclient:jar:0.17.0:compile
[INFO]    +- org.apache.httpcomponents:httpcore:jar:4.1.2:compile
[INFO]    +- org.apache.httpcomponents:httpasyncclient:jar:4.1.2:compile
[INFO]    |  +- org.apache.httpcomponents:httpcore-nio:jar:4.4.5:compile
[INFO]    |  \- commons-logging:commons-logging:jar:1.2:compile
[INFO]    +- org.apache.httpcomponents:httpclient:jar:4.1.2:compile
[INFO]    |  \- commons-codec:commons-codec:jar:1.4:compile
[INFO]    +- com.uber.jaeger:jaeger-core:jar:0.17.0:compile
[INFO]    |  +- com.uber.jaeger:jaeger-thrift:jar:0.17.0:compile
[INFO]    |  |  \- org.apache.thrift:libthrift:jar:0.9.2:compile
[INFO]    |  +- io.opentracing:opentracing-api:jar:0.15.0:compile
[INFO]    |  +- com.fasterxml.jackson.core:jackson-databind:jar:2.7.4:compile
[INFO]    |  |  +- com.fasterxml.jackson.core:jackson-annotations:jar:2.7.0:compile
[INFO]    |  |  \- com.fasterxml.jackson.core:jackson-core:jar:2.7.4:compile
[INFO]    |  \- org.slf4j:slf4j-api:jar:1.7.16:compile
[INFO]    \- com.uber.jaeger:jaeger-context:jar:0.17.0:compile

It's not optimal.
Other thing if there is redirect it would produce two spans as children of context get by TraceContext parentContext = TracingUtils.getTraceContext();. If there is no parent context spans would be in different traces.
example output:
inject(io.opentracing.mock.MockSpan$MockContext@6442b0a6, io.opentracing.propagation.Format$Builtin@60f82f98, com.uber.jaeger.httpclient.ClientRequestCarrier@35f983a6)
MockSpan{traceId=1, spanId=3, parentId=2, operationName='GET', startMicros=1489660386665000, finishMicros=1489660386776000, tags={http.status_code=302, span.kind=client, http.url=/search?q=httpClient, peer.hostname=www.google.com, peer.port=-1}, logs=[]}
inject(io.opentracing.mock.MockSpan$MockContext@46d56d67, io.opentracing.propagation.Format$Builtin@60f82f98, com.uber.jaeger.httpclient.ClientRequestCarrier@d8355a8)
MockSpan{traceId=1, spanId=4, parentId=2, operationName='GET', startMicros=1489660386804000, finishMicros=1489660386942000, tags={http.status_code=200, span.kind=client, http.url=/search?q=httpClient&gws_rd=cr&ei=4WnKWLmALsjoaqCkv6gI, peer.hostname=www.google.cz, peer.port=-1}, logs=[]}
MockSpan{traceId=1, spanId=2, parentId=0, operationName='parent', startMicros=1489660386359000, finishMicros=1489660386945000, tags={}, logs=[]}

No parent:
inject(io.opentracing.mock.MockSpan$MockContext@6442b0a6, io.opentracing.propagation.Format$Builtin@60f82f98, com.uber.jaeger.httpclient.ClientRequestCarrier@35f983a6)
MockSpan{traceId=3, spanId=4, parentId=0, operationName='GET', startMicros=1489660604869000, finishMicros=1489660604979000, tags={http.status_code=302, span.kind=client, http.url=/search?q=httpClient, peer.hostname=www.google.com, peer.port=-1}, logs=[]}
inject(io.opentracing.mock.MockSpan$MockContext@46d56d67, io.opentracing.propagation.Format$Builtin@60f82f98, com.uber.jaeger.httpclient.ClientRequestCarrier@d8355a8)
MockSpan{traceId=5, spanId=6, parentId=0, operationName='GET', startMicros=1489660605002000, finishMicros=1489660605118000, tags={http.status_code=200, span.kind=client, http.url=/search?q=httpClient&gws_rd=cr&ei=u2rKWOSjOoT5apf7juAO, peer.hostname=www.google.cz, peer.port=-1}, logs=[]}

I don't like to create instrumentations which already exist see https://github.com/opentracing-contrib/meta/issues/15. I can see some benefits in having it here.

Here https://github.com/pavolloffay/java-apache-httpcomponents is my impl.

Object spanAttribute = context.getAttribute(ACTIVE_SPAN);

so it isn't using span manager?

no I will add it in the moment. At the same time I'm not super happy with it because it will cause breaking changes once in-process propagation is "solved" at standard level.

@pavolloffay just catching up on this – sounds like we shouldn't actually create a new repo yet, but let me know when you think the time is right for that. Thanks.

I would like to move this forward. I have spent more time on it and current instrumentation looks even better. Now it instruments all requests including errors like UnknownHostException.
thanks goes to @adriancole
Repository can be java-apache-httpclient

@pavolloffay https://github.com/opentracing-contrib/java-apache-httpclient/invitations
Thank you!

Closing, it has been merged and released.


Fix upgrade and version check

What type of PR is this?
Bug
What this PR does / Why we need it?
This PR fixes how we fetch binaries for upgrades and well as enforces the version of backplane CLI to not be static.
Which Jira/Github issue(s) does this PR fix?
https://issues.redhat.com/browse/OSD-15761
Special notes for your reviewer
Pre-checks (if applicable)

[x] Ran unit tests locally
[x] Validated the changes in a cluster
[ ] Included documentation changes with PR

/label tide/merge-method-squash

/lgtm


Recent Astro-first Community Resources

Description (required)
Added the recent UNIQUE content I've been publishing involving Astro.
Confirmed by Sarah that this is the place where I may propose to add the recipes: https://discord.com/channels/830184174198718474/830184175176122389/1229477006979240016

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note
Locale

src/content/docs/en/community-resources/content.mdx
Source changed, localizations will be marked as outdated.
en

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@sarah11918

Thanks for providing even more content @rishi-raj-jain !

There's a lot more incoming!
I've resolved as per your comments.


Not ready for Ionic 2 final?

Hi, I built an app with Ionic 2 (RC3) and the great ionic-audio, and Ionic 2 final just released :)
It seems that ionic-audio is not compatible with this final version (I remember I had the same problem when trying to Ionic 2 RC4). Would be wonderful if it could fit the final version :) (and the docs could reflect it too).
Here's what I get :

Runtime Error
Error in ./AudioPlayerComponent class AudioPlayerComponent - caused by: Cannot read property 'isImgsUpdatable' of null

Stack
TypeError: Cannot read property 'isImgsUpdatable' of null
at Img.update (http://localhost:8100/build/main.js:60904:39)
at Img.set [as src] (http://localhost:8100/build/main.js:60874:22)
at Wrapper_Img.check_src (/IonicModule/Img/wrapper.ngfactory.js:25:22)
at View_AudioPlayerComponent2.detectChangesInternal (/AppModule/AudioPlayerComponent/component.ngfactory.js:313:17)
at View_AudioPlayerComponent2.AppView.detectChanges (http://localhost:8100/build/main.js:99083:14)
at View_AudioPlayerComponent2.DebugAppView.detectChanges (http://localhost:8100/build/main.js:99176:44)
at ViewContainer.detectChangesInNestedViews (http://localhost:8100/build/main.js:99268:37)
at View_AudioPlayerComponent1.detectChangesInternal (/AppModule/AudioPlayerComponent/component.ngfactory.js:121:14)
at View_AudioPlayerComponent1.AppView.detectChanges (http://localhost:8100/build/main.js:99083:14)
at View_AudioPlayerComponent1.DebugAppView.detectChanges (http://localhost:8100/build/main.js:99176:44)

Ionic Framework: 2.0.0
Ionic Native: 2.4.1
Ionic App Scripts: 1.0.0
Angular Core: 2.2.1
Angular Compiler CLI: 2.2.1
Node: 7.4.0
OS Platform: macOS Sierra
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36

Thanks !
(if interested, the app is available at http://app.captainweb.net/)

You are right. I haven't had the time to upgrade yet 😞

It's done. Upgraded to Ionic 2 final. There are a few changes related to the setup, please check the updated docs:
http://arielfaur.github.io/ionic-audio/2.0/index.html

Amazing!!! Thanks a lot man, cheers!


Pull to-device event list out of returned tuple

OlmMachine.receiveSyncChanges returns an array of [device messages, room key changes], so emit "to_device.decrypted" with that instead of the entire array.
Fixes regression introduced by #287.
Signed-off-by: Andrew Ferrazzutti<EMAIL_ADDRESS>Checklist

[ ] Tests written for all new code
[x] Linter has been satisfied
[x] Sign-off given on the changes (see CONTRIBUTING.md)

I intend to add test coverage for this to prevent further regressions (whether self-inflicted or induced by dependency changes).

On closer examination, the regression wasn't caused by #287, but by a change in the rust-sdk crypto crate which isn't yet in a release of the Node bindings (change = matrix-org/matrix-rust-sdk@98882b9c23bdf03f875abf79060937047425a4c3, merging PR = matrix-org/matrix-rust-sdk#2142).
With that said, this PR will be needed only after a new release of the rust-sdk Node bindings is cut.

Note that this applies on top of https://github.com/turt2live/matrix-bot-sdk/pull/330 due to requiring an update of the rust-sdk bindings, which necessitates handling the omission of the Sled crypto store.


Bug: Scrub Relocation Issues

You already know about these, I'm just making this a new issue since you've reduced some bugs down.
Basically, right now, if you trade Land Title Deed before you trade Moon's Tear the scrub in South Clock Town will be gone. This allows the player to somewhat sequence break (aka going out of logic) if they have Deku Mask to get the South Clock Town Day 3 chest without Moon's Tear if they don't have MM's full hookshot. This also makes some cycles where you plan to trade Moon's Tear but... already traded Land Title Deed somewhat annoying because it means you can't trade the tear until you cycle reset.
In addition, there's currently no scrub relocation of any kind in general. Can't have the bean scrub in SCT to buy beans from without Deku Mask (if you found a bean in the world first), for example. Can't have the Goron Village scrub in Southern Swamp to buy their item without Goron Mask (so you could actually buy Goron Mask from the scrub if you have Moon's Tear, Land Title Deed, and Swamp Title Deed).
I'm well aware these scrubs are annoying from a programming perspective but I am just noting this down here in case you ever get the desire or need to want to change/fix it.

Bug fixed for Land Title Deed trade here https://github.com/OoTMM/OoTMM/commit/b2a85f963dcda7f1f0e1a60fa5db21603e5346de
Probably should make a separate issue for scrub relocation as it's not really a bug but more so just not working due to whatever Nax had to change to make them stay where they are and what not.


Remove unused dependencies by individual crates

Using a small tool of my creation to remove unused dependencies; I've seen that cargo-udeps was used, but I've found that it has a few imperfections, as it cannot detect unused dependencies of individual crates within a common workspace. This shouldn't bring large build time improvements, but it change the graph of crate dependencies and thus cause some little wins due to cargo pipelining effects. I haven't observed any improvement outside the range of noise here, unfortunately.
The tool I'm using can be imprecise, so I'm curious to see if CI will be happy with the changes...

Excellent! Have you made the tool public?

Works great, many thanks :)
Maybe it is going to have a positive impact on:
https://github.com/uutils/coreutils-tracking/#binary-size-evolution
(refreshed tomorrow)

Excellent! Have you made the tool public?

It's open source and in my repos :eyes: but not yet officially public! I've wanted to write a small blog post announcement, explaining how it works and its limitations first.

Very cool! Looking forward to it!


Governance. Vote. There is no verification on "The account is already delegating"

Description
When user vote, while delegating - the app won't check that account is already delegated
Steps to Reproduce

Delegate in all tracks
Try to Vote to any referenda

Expected Behavior
Verification - "Can't Vote - The account is already delegating"
Actual Behavior
There is no Verification
Environment
https://github.com/novasamatech/nova-spektr/commit/7144562e489600adfe97da3cdc7ba05ac6b29843

Status verification: ✅
What has been verified:

Multisig Verification
NovaWallet Verification
PolkadotVault single Verification
PolkadotVault multishard Verification ❌


Can't find discord bot link, to add to a server.

Looking forward for reporting a security issue:
Please report security issues by sending an email to<EMAIL_ADDRESS>instead of raising a GitHub issue.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser: [e.g. stock browser, safari]
Version: [e.g. 22]

Additional context
Add any other context about the problem here.

Hi there,
The 100 Days of Code bot is available in the freeCodeCamp Discord server but is not available to add to other servers at this time.


Introduction to Figma Part 11 : Shadows

Description
Description

Add shadows to your designs using Figma

Domain
UI/UX
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@ShudhanshuSingh please assign me this issue

@soma-v go with the issue


Add query parameter for JSON file URL

Users should be able to navigate to a url like https://json-space-analyzer.com?file=<file-url> and see the visualization for that file.
This will allow people to share links to specific file visualizations with each other.

Actually this should probably use hash parameters so that the URLs don't ever get sent to the server.

This was partway done in 9cc7423e5a5f2e047acfbdd06af06370aedda207, but I still need to add an input box for pasting links, and should also add better error handling for invalid URLs in hash parameters.

See #8 for the input box task


Remove the 'bookmarks' app completely

There is a django app bookmarks that can be completely removed since it's not used at all: https://github.com/rtfd/readthedocs.org/tree/e9d0d2ff5b70e1c0b58be512a6cf10e7c11930b3/readthedocs/bookmarks

Consider that removing this app could help us to not depend in django-pagination since it's used in some of the bookmarks templates.

@humitos looks like this can be closed

This was solved on #3663


✨ Introduce local_storage mode for cloud instances

https://www.notion.so/laminlabs/Data-sharing-through-hybrid-local-remote-LaminDB-instances-e01025ed0e3e4f06948c4390894f20a8
Replace the initial draft:

https://github.com/laminlabs/lamindb-setup/pull/717

Needs:

https://github.com/laminlabs/laminapp-ui/pull/605

I guess local instances also should have local storage, but it should be the same as root. Right?

I guess local instances also should have local storage, but it should be the same as root. Right?

Yes! The naming choice is slightly confusing for local instances, but simpler than saying "hybrid" for cloud instances.
The point is to make it easy for users to keep their data local also when working with cloud instances.
They'll have a switch on the hub called "Keep data local" or "Default to local storage" or similar.
I'm pretty sure we'll not allow to have purely local instances on the hub going forward.

It is not actually clear than what is the difference between local and cloud now. Because a local instance can change its root to cloud but the local storage stays the same i guess. So now every instance can have cloud storage and local storage. So there is not much difference i suppose.

It is not actually clear than what is the difference between local and cloud now. Because a local instance can change its root to cloud but the local storage stays the same i guess. So now every instance can have cloud storage and local storage. So there is not much difference i suppose.

You are correct. I'm simply trying to achieve a more convenient UX for users who want to default to local storage working on a common server. I'll give this yet another refactor because the UX is still confusing.


Fix placeholder text issue in Menu Items when input loses focus

I want to merge this to fix placeholder text issue in the Autocomplete component in Menu Items.
This fix covers:

typing text by user
choosing items from drop-down list

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-4374 Placeholder text issue in Menu Items when input loses focus

The logic behind this version:

If an input for the Link field is invalid for onBlur – clear the input as opposed to leaving it, as it was previously.
Cancel the loader if input area loses focus. For this we need some way to track onBlur state. I’ve left that part as it was.


How to get query value?

Currently, I use URI.parseQuery((new URI()).query()).a to get value of a.
Is there any more semantic way like (new URI()).getQuery('a')?

there are a couple of options, the one I'd go for is
console.log( URI().query(true).a );

This is good. Thanks.


Video issues on Android with Chrome 38.0.2125.102

The bigPlay element is no longer displayed on Android videos with the latest Chrome. It appears Chrome has stopped emitting an initial pause event for videos, which triggered displaying the bigPlay element:
media.addEventListener('pause',function() {
    if (!mejs.MediaFeatures.isiPhone) {
        bigPlay.show();
    }
}, false);

I've worked around this by showing the layer after a success event is emitted by mediaelement:
if BrowserDetect.isAndroid()
    $('.mejs-overlay-play').css('display', 'block')

Maybe it would be a good idea to always display bigPlay with mobile devices, as none seem to autoplay without setting flags?
Cheers

@adrian-gray Closing old issues. If you'd like to reopen this, let us know. Thanks!


winpython editable install

Installing mne in WinPython works but the editable install with "pip install -e +url" doesn't work as expected. If I edit any function it is not updated.
To reproduce issue download winpython 2.7
install mne using WinPython Command Prompt
'pip install -e git+https://github.com/mne-tools/mne-python#egg=mne-dev'
edit any function
use import mne in spyder
and then use the function to see if it got updated

for this to work you need git installed. if it's not the case please close.
Otherwise we have a bug to fix.

I have git installed ... I have a clone in the src folder in the winpython directory.

ok. What is the error you get?

I do not get any error. I just want to edit some of the functions in mne and then run my analysis which I am not able to do. If I run the function it runs everything except the lines I edited. It seems like an install in the non-editable mode.

ah. You're not doing it the right way. If you want edit the code you should
start with:
http://martinos.org/mne/dev/contributing.html#using-git
and in particular:
http://martinos.org/mne/dev/contributing.html#setting-up-the-fork-and-the-working-directory
hope this helps

I tried this but this installs the files in documents somewhere and is not contained within the winpython directory. Is there a way of editing setup.py to install within the winpython folder?

When doing python setup.py install, a correctly configured Python setup will make it so that when you do import x later it will import the installed version. If it does not, it indicates that Python isn't configured correctly. When you run python does it actually execute the WinPython version of Python? Do you have another Python interpreter installed or configured? Have you sucessfully installed any other packages using python setup.py install?

Ok so I installed it using "python setup.py develop". I omitted the --user argument and it installed in the current directory but still I have the same problem. If I edit the script then it doesn't get updated. Is the --user argument necessary for it to work?

Depends on your python setup. --user means install only for the current user, omitting it typically does it system-wide. If you have write permissions to the system-wide Python, it might not matter which one you should do. But --user should always work AFAIK, so you could try that one.

I am running spyder inside the winpython folder so I think it is using the same winpython version. If I uninstall then I cannot import it in spyder so i think it is the same version.

I just installed with --user but still same issue:
I run the following
chNos = list(range(1,32))
estr = 'E'
chanNames = [estr+str(chNo) for chNo in chNos]
info = mne.create_info(chanNames, 250.0, ch_types=None)
then change mne.create_info: delete everything in the function and replace with return 0.
If I run the function again I get an info object and not 0.

Ahh right, you wanted an editable install. You'll need to use --develop, and possibly also --user. Then when you open Python, do:
import mne
mne

and it should print out where it imported MNE from. It's possible it's getting an older installed version, which you might need to delete for it to find the --develop version.

ok so I figured out what the problem is - I have to close spyder and repoen for the changes to reflect. I used import mne again without closing spyder but doesnt work. I used mne = reload(mne) which also doesnt update the function.

you discovered a standard behavior of the python virtual machine.
if you edit an imported file or module it will only be updated in a new
interpreter. it's for efficiency reason as otherwise import caching would
not work fine. Rather than closing spyder just open a new interpreter.
HTH

Checkout ipython %autoreload
Many google sources
2015-04-29 21:38 GMT+02:00 Alexandre Gramfort<EMAIL_ADDRESS>
Closed #2034 https://github.com/mne-tools/mne-python/issues/2034.
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2034#event-293362564.

yup just got that working. Thanks :)


XCAT connecting multiple QEMU sessions

Please check with me. Will pass power system details where QEMU is installed on Power system. Using that you need to connect to XCAT.
Basically create multiple QEMU and try to connect XCAT using that.
As discussed with XCAT team, openbmc may not have XCAT support yet. Please check and open issues accordingly.

Feel free to add your estimate accordingly.

Feel free to update your estimate.

http://wiki.qemu.org/Documentation/Platforms/PowerPC
You can get more details about QEMU on PowerPC in the above link

http://wiki.qemu.org/Documentation/Platforms/POWER - Right one

Shared the system detail where QEMU on Power is installed.

We could successfully install XCAT and able to add FSP/openpower system. Now need to work on openbmc system.

Task already completed as per @prkatti1  closing


Quickfix: Update VMware template used in ci

Why is this PR needed?
The tfvars used in ci for VMware jobs is outdated and is using a VMware template with known issues with cloud-init which may cause problems when rebooting machines.
What does this PR do?
Updates the VMware template in the tfvars file used for ci jobs
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

@bergmannf added a comment in the PR description.


Updating Ali's postprint-pledge-for-linguists

Updating Ali H. Al-Hoorie's postprint-pledge-for-linguists in accordance with his re-edit of the pledge.
This PR consists of changes to a markdown file that word the pledge terms more concisely.
Ali has looked at the edited version of 2022-06-06-postprint-pledge-for-linguists.md, and he agrees that the PR version matches his intentions.

Updating Ali H. Al-Hoorie's postprint-pledge-for-linguists in accordance with his re-edit of the pledge.
This PR consists of changes to a markdown file that word the pledge terms more concisely.
Ali has looked at the edited version of 2022-06-06-postprint-pledge-for-linguists.md, and he agrees that the PR version matches his intentions.


ARROW-7344: [Packaging][Python] Build manylinux2014 wheels

@ursabot crossbow submit -g wheel

@ursabot crossbow submit -g wheel

Thanks @kou, had to leave a few things in there just to make sure you were paying attention ;)
I've made your suggestions, PTAL.

@kszucs will do, also will add 3.8 to that verification (should it also have 3.5?)

@ursabot crossbow submit -g wheel

I'll merge this.
@wesm We updated release verification scripts for Linux, macOS and Windows. But they aren't tested yet because we don't have uploaded RC artifacts. You may have a problem when you use the release verification script for Windows. Sorry. We may need to fix release verification scripts on the next release process.

@kou no problem, I'm used to there being some problems with the verification scripts around releases, so we can fix them when it's time


v3.0.6 - ship list

Major Updates:
CSS
JS
Docs
Build

There probably will not be any more updated in the 3.x line.


chore(dependencies): update peerDependencies

Closing, lodash.debounce was not needed as a peer-dependency, at least for now.


fix: Add publish-docs stage to Travis build

Re-enable the scripts/publish-docs script by adding a stage to update the docs/ submodule from probot.github.io.
See and fixes probot/probot.github.io#321.

Because there isn't a good way to locally test the PR, we'll only know if everything is working as intended in production :smile:.
I replicated the steps from the scripts/publish-docs script as best I could and encountered zero errors; the steps from script/publish-docs don't have to be changed.

It's currently failing: https://travis-ci.org/probot/probot/jobs/631900147?utm_medium=notification&utm_source=email


Can I disable some rules?

Did not find anything with google, I just want to disable false positive on fs.readFile(filename, 'utf-8')  (https://github.com/facebook/flow/issues/416)
can I do it somehow?

as of 0.12.0, you can add an option to your .flowconfig which sets up magic comments that suppress flow errors on the following line.
at facebook, we do this:
[options]
suppress_comment=\\(.\\|\n\\)*\\$FlowIssue

and then would do something like
/* $FlowIssue - flow doesn't type fs.readFile properly as of 0.12.0 */
var content : string = fs.readFile(filename, 'utf-8');

@lapanoid: Looks like @mroch answered, hence closing; please reopen if it doesn't solve your issue.


🏷 Add Release Tag Builds

This adds a build step for Github which will build binaries for most major OS setups when a new tag is pushed.  This binary can be used with cargo-bininstall.

Thanks a lot!


support filter by net namespace

Network addresses may overlap between multiple net namespaces, which can interfere with our filtering.

assign netns ID to an existing netns to facilitate your proposed debugging?

By looking down the kernel source code net_namespace/peernet2id, netnsid is not retrieved from struct net neither struct ns_common.
inode is a good way to identify an net namespace. In kernel space, inode is the unsigned int inum field of struct ns_common. In user space, inode is the inode of the net namespace file, wherever the file is, like /proc/<pid>/ns/net, /var/run/docker/netns/<netns name>.
For example:
log of pwru: 0xffff8f26c23d4500 [isc-worker0000]     neigh_resolve_output<PHONE_NUMBER>061 netns=4026531992 mark=0x0 ifindex=2 proto=8 mtu=1500 len=62
inode of netns file: got netns: NS(4:4026531992)
(Go code to get inode: ns.UniqueID)

@Asphaltt Thanks! Using the inode makes sense to me. Would you like to work on this?

Fixed by #34.


Add syncit status to status command

spm works in tandem with syncit (mutagen) and will even stop syncit sessions if it is installed when services are stopped.
It would be beneficial to add a syncit check on the status screen to remind when any syncit session is still running on that service.

Added in 0.13.0, can be disabled via --no-syncit.


Add docs for CLI usage

Add some documentation that explains how to use the assembler, what the flags do, and when one should use them.

Added CLI usage docs!


feat(preset-env): Upgrade browserslist-rs

Fixes

Fixed parsing query last 2 versions and .... (fix #4871)

Performance

Reduced build size by deduplicating browser names.

Thanks a lot!


Windows support

Could we get a Windows port of the cloudlibc runtime and an example cmake project illustrating how to build CloudABI binaries from Windows development hosts? So that we can grow the deployment space for CloudABI apps!

Pretty please?


Add an SBOM URI field to artifact events

Changes
Artifact packaged and/or published events may include a link to a published SBOM. Since there is no default storage location for such SBOM documents, CDEvents may help linking the artifact with its SBOM by including a link to the SBOM in the artifact events.
Partially-addresses: #132
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has the primer doc been updated if a design decision is involved
[x] Have the JSON schemas been updated if the specification changed
[x] Has spec version and event versions been updated according to the versioning policy
[x] Meets the CDEvents contributor standards

@xibz @e-backmark-ericsson rebased and addressed all comments.


Changing the folder structure, adding Authentication page

Changing Folder Stucture:
adding separate folders for models, controllers etc for better readability and maintaining code
Authentication Page
Login and registration routes are written but i didnt find any auth page
I will work on auth page and restructure the codebase.

@NOOB-3301 Take your time for this issue and if possible, try to keep updated with the fork. I'm assigning this issue to you, thanks

@NOOB-3301 @Pawankalyan2023 there's a bug in the code in which instead of making the api calls to the backend server it's calling it locally instead. Please change it to the server address (you can find the backend server link in navbar.jsx). I'll be updating the issue also so try checking the updated issue

you actually do not need the index.js folder anymore . i made 3separate files for starting index.js , db.js and app.js
index.js starting sever you have to  check the api/ folder.
and i changed some routes
for getting specific article /getarticle with post req
for getting all article  getAllarticle
for adding comment addcomment with post req
for adding artcle /addarticle with post req
this is the working

and this are the routes for the api to work

i think you have to change some of the routes. or please share your discord id @OkenHaha so i can explain there properly

I think I understand the confusion here after you're explanation. The server that was used to host the backend needed a script to run which was set to 'index.js' initially which was not under the 'api' folder. After examining, the package.json needed to change the scripts that was used to run i.e. from
"scripts": {
    "start": "node index.js",
    "dev": "nodemon index.js"
  },

to
"scripts": {
    "start": "node ./api/index.js",
    "dev": "nodemon ./api/index.js"
  },

I hope this clarifies the confusion. And my discord id is @okenk you can add me there and please leave a star for the repo.

i made a pr , with all the necessary fixes . if you have any question do ask me before merging ....?

I think I understand the confusion here after you're explanation. The server that was used to host the backend needed a script to run which was set to 'index.js' initially which was not under the 'api' folder. After examining, the package.json needed to change the scripts that was used to run i.e. from
"scripts": {
    "start": "node index.js",
    "dev": "nodemon index.js"
  },

to
"scripts": {
    "start": "node ./api/index.js",
    "dev": "nodemon ./api/index.js"
  },

I hope this clarifies the confusion. And my discord id is @okenk you can add me there and please leave a star for the repo. Do you want to make the pull request? @NOOB-3301

i made a pr , with all the necessary fixes . if you have any question do ask me before merging ....? @OkenHaha


Firewalld

Description of PR
Added firewalld configuration.
It is activated when the variable zabbix_agent_firewalld_enable is set to true.
Type of change
Feature Pull Request

Fail is not related to the addition of firewalld configuration support. What in this case?

Thanks! 👍
Yeah, failing CI is not related to your change but due to a container issue. So just merging this PR!


RpmITCase.java:49-52: This test is failing on Windows...

The puzzle 69-20779ab6 from #69 has to be resolved:
https://github.com/artipie/rpm-adapter/blob/493153a41dfc9dcae2bb43d8af290adf7d7acf47/src/test/java/com/artipie/rpm/RpmITCase.java#L49-L52
The puzzle was created by Kirill on 08-Apr-20.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@g4s8/z please, pay attention to this issue

The job #86 assigned to @carlosmiranda/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@g4s8 see PR #94

@0crat refuse

@g4s8 I give up on this. First of all, it seems to be strangely environment specific - I couldn't reproduce it locally even though I'm also on Windows. Then I've tried every fix and workaround and hack I can think of and nothing works on any of the Github Windows check builds.

@0crat refuse (here)

@carlosmiranda The user @carlosmiranda/z resigned from #86, please stop working. Reason for job resignation: Order was cancelled

The job #86 assigned to @ammaratef45/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat refuse

@0crat refuse (here)

@ammaratef45 The user @ammaratef45/z resigned from #86, please stop working. Reason for job resignation: Order was cancelled

@0crat refuse (here)

@ammaratef45 Job refused in 5 hours - no penalty, see §6

The job #86 assigned to @victornoel/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat wait for #115 to be merged

@carlosmiranda the file wasn't properly closed before the move happened on it. But the ATOMIC thing was also a good idea I believe, so I left it.

@victornoel good job. I did try to put the file input/output streams in try-with-resources too but my solution didn't work for some reason.

@carlosmiranda shame on me, I simply forgot to remove the DisabledOnOs annotation, actually the problem is not fixed ^^

@carlosmiranda finally found the solution: closing an xmlwriterstream does not close the underlying stream :|

The puzzle 69-20779ab6 has disappeared from the source code, that's why I closed this issue.

Job was finished in 38 hours, bonus for fast delivery is possible (see §36)

@sereshqua/z please review this job completed by @victornoel/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality good


Sample app

To showcase the new functionality of the 5.0.0 release, a sample app should be added to this repository.
A good example could be a rudimentary podcast app called EiffelCasts that allows searching, playback and marking podcasts as favorites. This would showcase both local and remote commands, state handling and could use public APIs.
It would provide a reasonably complex example with real-world challenges instead of a dummy app with no practical value.

Hi, any future plans on the sample app? Or the project is not actual anymore?🤔

Hey @3mph4515, I'm still planning to maintain the project but unfortunately I'm currently way too busy with job and family to actively work on it.
All that's basically left to do is documentation and this sample app. I'm also a bit behind on EiffelAmi, which I'd really like to release together with Eiffel 5.0.
I'd also need to check, if any new dependency updates from Kotlin Flow or AndroidX introduced some breaking changes.
Once I manage to get all of this done, I'll be more than happy to finally release the next major version of Eiffel.

Hey @piyushjohnson, thanks for the offer.
It'd definitely still make sense to provide some kind of sample application. If you want to give it a shot, please do.
I switched to working as a full-time Flutter developer so I'm not that entrenched in the Android ecosystem anymore. And with the current "global situation" I've unfortunately got even less time for side projects.
I still plan to keep Eiffel going though and more or less up to date, so I'd be more than happy to accept your help on a sample app.