Fix/http header

cherry-pick from #1217
fix http disable normalize

Codecov Report

Merging #1219 into develop will decrease coverage by 0.06%.
The diff coverage is 30.00%.

@@             Coverage Diff             @@
##           develop    #1219      +/-   ##
===========================================
- Coverage    56.18%   56.12%   -0.07%
===========================================
  Files          268      270       +2
  Lines        25540    25548       +8
===========================================
- Hits         14350    14338      -12
- Misses        9816     9832      +16
- Partials      1374     1378       +4

Impacted Files
Coverage Δ

pkg/metrics/tls.go
0.00% <0.00%> (ø)

pkg/mtls/metrics.go
0.00% <0.00%> (ø)

pkg/stream/http/stream.go
18.84% <ø> (+0.04%)
:arrow_up:

pkg/upstream/cluster/cluster_manager.go
62.55% <100.00%> (+0.35%)
:arrow_up:

pkg/stream/stream.go
46.66% <0.00%> (-13.34%)
:arrow_down:

pkg/module/http2/transport.go
78.18% <0.00%> (-0.58%)
:arrow_down:

pkg/module/http2/server.go
81.77% <0.00%> (-0.15%)
:arrow_down:

Continue to review full report at Codecov.

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


Stream Removed is not received when someone leaves a conference

We see that the Participant Status changes to LEFT but no Stream Removed event is received.

This happen on android device only, cannot reproduce on Ios device. Problem is releaded with Android native SDK, during test I checked that react native  not getting stream removed events from sdk when participant left, only participant left event appear on react native.

I confirm that this issue is in SDK itself.
Fix is on its way.

As per internal communication fixes in the native SDKs should be delivered in 3.10.
Hence testing in React Native is needed.

Tested. Works properly.


Write automated tests: Android Basics with Compose (Test fails if you don't specify Locale.US as a parameter to getCurrencyInstance()

What happened?
TipCalculatorTests fails.
Error message
expected:<[$2.00]> but was:<[¥2]>
Expected :$2.00
Actual   :¥2
Solution
Specify the Locale.US as a parameter to getCurrencyInstance() of the calculateTip() function like this:
return NumberFormat.getCurrencyInstance(Locale.US).format(tip)
Reason (guessing)
I live in Japan.
So default locale is Locale.JAPAN
(But I don't know why emulator shows "$2.00" without specifying Locale.US)
Code
MainActivity.kt line:135-145
@VisibleForTesting
internal fun calculateTip(amount: Double, tipPercent: Double = 15.0, roundUp: Boolean): String {
    var tip = tipPercent / 100 * amount
    if(roundUp) {
        tip = kotlin.math.ceil(tip)
    }
    return NumberFormat.getCurrencyInstance(Locale.US).format(tip)
}

Hello @taked4,
Thanks for reaching out to us and sharing your concern. The issue is fixed and closed.


Error using Elbow()

Describe the error
I tried to use the Elbow() object but I got an error.
Code and Error
Code:
class ElbowExample(Scene):
def construct(self):
triangle = Polygon(ORIGIN, RIGHT, RIGHT+UP)
elbow = Elbow(color=RED)
elbow.set_points_as_corners([ORIGIN, RIGHT, RIGHT+UP])
elbow.set_width(elbow.width, about_point=RIGHT+np.array([-0.1, 0.1, 0.0]))
self.add(triangle)
self.add(elbow)
self.wait(2)
Error:
...geometry.py", line 572, in init
in init
super().init(self, **kwargs)
TypeError: init() takes 1 positional argument but 2 were given
Environment
OS System: Win10
manim version: version of 23/02/2021
python version: 3.8.1

The problem is passing self to super().__init__, see https://github.com/3b1b/manim/pull/1419/commits/63b497c352eff38cd5b16b32fd9f385fd902a59f


Wrong Proposal Format error - something changed recently?

Expected behavior
To create a Proposal to Snapshot via the API
Actual behavior
Receiving the following error only today when trying to create a new proposal using the API
{ error: 'client_error', error_description: 'wrong proposal format' }
Steps to reproduce the behavior
Version -<EMAIL_ADDRESS>"0.3.77",
here is the object i'm sending
const proposal = {
                    space: 'fabien.eth',
                    title: 'test',
                    body: 'test',
                    discussion: 'test',
                    choices: ['Yes', 'No'],
                    start: startSeconds,
                    end: endSeconds,
                    type: data['type'],
                    network: network,
                    plugins: JSON.stringify({}),
                    snapshot: blockNumber,
                    metadata: JSON.stringify({ app: "snapshot.js" }),
                    strategies: JSON.stringify(strategies),
};

Here is the Error message
Sign {
  address: '0xFAD69Bd739c64cC8e3f1C3bb3B60fe4f160174Cc',
  sig: '0x0b692e6ce4a0c65274532daae3955e41c47b8bbee49f0ed28fc11cea3fc377663c1e4540ead8048334e258dad774280ee6ddba117d9157957f62691e20f3957b1b',
  data: {
    domain: { name: 'snapshot', version: '0.1.4' },
    types: { Proposal: [Array] },
    message: {
      space: 'fabien.eth',
      title: 'test',
      body: 'test',
      discussion: 'test',
      choices: [Array],
      start:<PHONE_NUMBER>,
      end:<PHONE_NUMBER>,
      type: 'single-choice',
      network: '1',
      plugins: '{}',
      snapshot: 19044092,
      metadata: '{"app":"snapshot.js"}',
      strategies: '[{"name":"erc721","params":{"symbol":"HVAXVC","address":"0x4E911626Fa378cC95fB1A26Cc272c070eF79e4b7","network":"1","decimals":0},"network":"1"}]',
      from: '0xFAD69Bd739c64cC8e3f1C3bb3B60fe4f160174Cc',
      timestamp:<PHONE_NUMBER>
    }
  }
}
{ error: 'client_error', error_description: 'wrong proposal format' }

I have also tried passing in app: "snapshot.js"  as a standalone key/value, same issue, so I don't believe it's that. (i'm using metadata as it's used in the version i have)

It seems to be an issue with the body text formatting.


feat: add swoole 5.1.5 release

update swoole with release 5.1.5

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


Thermostat zcl dev

Problem
Added thermostat app shell and updated cluster to support scheduling.
Required for TE5.
Change overview
Creation of thermostat app for linux.
At this time app has no functionality of significance.  Functionality to be added as scheduling features of cluster are implemented
Testing
Manually tested that the thermostat app runs.
Tested that the updated chip-tool app could pair with the thermostat app.
Verified that the new scheduling attributes appeared in the the chip-tool.  Verified that they returned their default values.
Verified the same in the chip-device-ctrl app as in the chip-tool

ZAP cnflicts keep coming back.  Killing PR


fix: downgrade httpx minimal requirement

Generated summary (powered by Graphite)

What Changed
Updated

httpx

Testing
Run the tests and verify that they pass. Also, test the functionality that depends on the httpx library.
Reasoning
This library is used in homeassistant and httpx library is a dependency in both modules. Reduced the minimal requirements to prevent issues with dependency hell

Current dependencies on/for this PR:

main

PR #7   👈

This comment was auto-generated by Graphite.

Merge Activity

Sep 14, 3:50 PM: @pjanuario merged this pull request with Graphite.


Send credential definition fails in ACAPy with request timed out

Recently the getting started test keeps failing with a request timed out
In ACAPy-0.7.5 I see ...
acapy  | 2022-11-02 12:30:20,222 indy.libindy.native.indy.services.ledger INFO 	src/services/ledger/mod.rs:241 | build_revoc_reg_entry_request() => Ok("{\"reqId\":1667392220222165525,\"identifier\":\"HALJ6zGdkkWVS28tRTMntC\",\"operation\":{\"type\":\"114\",\"revocRegDefId\":\"HALJ6zGdkkWVS28tRTMntC:4:HALJ6zGdkkWVS28tRTMntC:3:CL:118:default:CL_ACCUM:098a8108-ed63-414f-a318-6d72def3b589\",\"revocDefType\":\"CL_ACCUM\",\"value\":{\"accum\":\"21 124160E750AF0C97057CBCC9C84D3CD75E2BF4DBAEDF13C0498EABA8C7A32D4B9 21 122A76139F4C39E7876B38A704E709B87B7C6B027B6759C10B1557B483706C40E 6 7A9D0347E23CC34DB973206998FFED4861DDB20279BD5018DC08EDB451C95F4A 4 20EA523FA568D5D9237D21B81BC77F5D2170509D68C7286A5E23836833311243 6 56788113EFC464F383AD6950FD055593958FE5709831AC54B1F03450A4B3FF5B 4 38D481C88655BB6415D34A0388CCB6FE947FA8071C469D2FD09C500E2BF7AE80\"}},\"protocolVersion\":2}")
acapy  | 2022-11-02 12:30:28,226 indy.libindy.native.indy.services.pool.pool INFO 	src/services/pool/pool.rs:749 | Drop started
acapy  | 2022-11-02 12:30:28,226 indy.libindy.native.indy.services.pool.pool INFO 	src/services/pool/pool.rs:757 | Drop wait worker
acapy  | 2022-11-02 12:30:28,228 indy.libindy.native.indy.services.pool.pool INFO 	src/services/pool/pool.rs:760 | Drop finished
acapy  | 2022-11-02 12:30:33,233 aries_cloudagent.utils.http WARNING Upload error: request timed out
acapy  | 2022-11-02 12:30:44,038 aries_cloudagent.utils.http WARNING Upload error: request timed out
acapy  | 2022-11-02 12:30:54,936 aries_cloudagent.utils.http WARNING Upload error: request timed out
acapy  | 2022-11-02 12:31:05,939 aries_cloudagent.utils.http WARNING Upload error: request timed out

which suggest that ACAPy can't reach a certain endpoint. This may well be a configuration bad on my side, however I'd like the acapy error message to be more descriptive about what endpoint it cannot reach. Is it perhaps the tails-server that cannot get reached?

In the tails-server I see
2022-11-02 13:04:41,026 aiohttp.server ERROR Error handling request
Traceback (most recent call last):
  File "/home/indy/.local/lib/python3.7/site-packages/aiohttp/web_protocol.py", line 422, in _handle_request
    resp = await self._request_handler(request)
  File "/home/indy/.local/lib/python3.7/site-packages/aiohttp/web_app.py", line 499, in _handle
    resp = await handler(request)
  File "/home/indy/tails_server/web.py", line 85, in put_file
    genesis_txn_bytes, revocation_reg_id, storage_path
  File "/home/indy/tails_server/ledger.py", line 29, in get_rev_reg_def
    pool = await indy_vdr.open_pool(transactions_path=tmp_file.name)
  File "/home/indy/.local/lib/python3.7/site-packages/indy_vdr/pool.py", line 173, in open_pool
    await pool.refresh()
  File "/home/indy/.local/lib/python3.7/site-packages/indy_vdr/pool.py", line 58, in refresh
    await bindings.pool_refresh(self.handle)
indy_vdr.error.VdrError: Pool timeout: Request was interrupted
2022-11-02 13:04:41,031 aiohttp.access INFO <IP_ADDRESS> [02/Nov/2022:13:04:20 +0000] "PUT /8XwrAmugANgtsVJ8ScvBfo:4:8XwrAmugANgtsVJ8ScvBfo:3:CL:132:default:CL_ACCUM:c3a8d83b-cf6e-4978-a3fe-5ebca075289d HTTP/1.1" 500 244 "-" "Python/3.6 aiohttp/3.8.3"

Which tails server are you using?

bcgov/indy-tails-server-1.0.0
https://github.com/tdiesler/nessus-aries/blob/main/build-images.sh#L85

@WadeBarnes would you perhaps have an idea what is going on here?

Can the tails-server resolve the network (von-network) addresses and do you know what it's getting for that address?

It can resolve the named hosts from the compose file e.g.
indy@a22d5999d096:~$ curl -s --write-out '%{http_code}' --output /dev/null http://webserver:8000
200

The tails-server, would however not know how to connect to any of the other containers (i.e. there is no config for it)
https://github.com/tdiesler/nessus-aries/blob/main/docker-compose.yml#L125

There is no ledger config for a tails-server it gets it's info from the agents when they post info, AFAIK.  Tails servers are not tied to a network.  A single tails server can be used for multiple networks.

acapy, gets the genesis txs from 'host.docker.internal' like this ...
indy@10123eeca73d:~$ curl -s http://host.docker.internal:9000/genesis
{"reqSignature":{},"txn":{"data":{"data":{"alias":"Node1","blskey":"4N8aUNHSgjQVgkpm8nhNEfDf6txHznoYREg9kirmJrkivgL4oSEimFF6nsQ6M41QvhM2Z33nves5vfSn9n1UwNFJBYtWVnHYMATn76vLuL3zU88KyeAYcHfsih3He6UHcXDxcaecHVz6jhCYz1P2UZn2bDVruL5wXpehgBfBaL....

still, the tails-server fails to connect to something
  File "/home/indy/.local/lib/python3.7/site-packages/indy_vdr/pool.py", line 58, in refresh
    await bindings.pool_refresh(self.handle)
indy_vdr.error.VdrError: Pool timeout: Request was interrupted
2022-11-02 13:04:41,031 aiohttp.access INFO <IP_ADDRESS> [02/Nov/2022:13:04:20 +0000] "PUT /8XwrAmugANgtsVJ8ScvBfo:4:8XwrAmugANgtsVJ8ScvBfo:3:CL:132:default:CL_ACCUM:c3a8d83b-cf6e-4978-a3fe-5ebca075289d HTTP/1.1" 500 244 "-" "Python/3.6 aiohttp/3.8.3"

Is it perhaps trying to reach back to acapy on that IP?

What addresses are listed for the nodes in the genesis file?

[core@vmd94237 nessus-aries]$ curl -s http://localhost:9000/genesis | jq . | grep node_ip
        "node_ip": "host.docker.internal",
        "node_ip": "host.docker.internal",
        "node_ip": "host.docker.internal",
        "node_ip": "host.docker.internal",

Unlike acapy, the tails-server cannot resolve host.docker.internal for some reason
curl: (6) Could not resolve host: host.docker.internal

Ok, that works. Thanks for your help @ianco @WadeBarnes


go get: Use a specific SSH identity?

What version of Go are you using (go version)?

$ go version

1.17
Does this issue reproduce with the latest release?
I don't know.
What operating system and processor architecture are you using (go env)?
Darwin, amd64
go env Output
$ go env

What did you do?
I created two GitHub profiles, profileA and profileB with SSH access and setup ssh/config (below) to configure how the git command cloned repos.  Combined with a gitconfig that redirects HTTPS to SSH, this allows me to determine which SSH identity is used with which repo and it works well.
However, although go get "github.com/..." will use profileA to request modules from private repositories, go get "alt.github.com/..." fails with an error (below).  The problem is obviously that go get is looking at the raw path to determine whether or not it's a special github link, and doesn't know how to resolve the true path through SSH config.  I'm sure there are good reasons that it works this way.
Yet, my problem remains. Is there any way to make go get use a different SSH identity?

Host alt.github.com
	AddKeysToAgent yes
	UseKeychain yes
	Hostname github.com
	User git
	IdentityFile ~/.ssh/id_profileB
Host github.com
AddKeysToAgent yes
UseKeychain yes
User git
IdentityFile ~/.ssh/id_profileA

What did you expect to see?
I expected to be able to go get modules using the profileB SSH identity.
What did you see instead?
An error:

go get: unrecognized import path "alt.github.com/golang/go": reading https://alt.github.com/golang/go?go-get=1: 404 Not Found

The server has to respond to the https query to determine module roots/vcs system, your approach won't work.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


D435 realsense-viewer normal operation , but the sample program failed to run

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ D435 }

Firmware Version
(2.44.0)

Operating System & Version
{Ubuntu 16.04.7 (64bit)

Kernel Version (Linux Only)
( 4.4.38)

Platform
Manifold - 2G (SZ DJI Technology Co.,Ltd. ) (CPU: NVIDIA Jetson TX2 )

SDK Version
{ legacy / 2.44.0 }

Language
{C++ }

Segment
{others }

Issue Description
<Describe your issue / question / feature request / etc..>
I installed the SDK according to the instructions of here
and executed the command realsense-viewer. It seems that the program runs normally and the depth image can be displayed normally. When I execute examples/rs-align, the following prompt appears, other sample programs also have this error
RealSense error calling rs2_pipeline_wait_for_frames(pipe:0x662a70):     Frame didn't arrive within 15000

Hi @liu-jiaming   Do you have the RealSense Viewer streaming at the time when you launch the example programs, please?  If you do, then please try stopping the streaming in the Viewer and then launching the example.
The opposite is also true.  If you are currently using an example program then close it before enabling streaming in the Viewer.
The example programs are not dependent on the Viewer and can run without the Viewer being launched.

Hi @liu-jiaming Do you have the RealSense Viewer streaming at the time when you launch the example programs, please? If you do, then please try stopping the streaming in the Viewer and then launching the example.
The opposite is also true. If you are currently using an example program then close it before enabling streaming in the Viewer.
The example programs are not dependent on the Viewer and can run without the Viewer being launched.

I turned it off

Thank you.  Next, could you try the command rs-depth-quality in the Ubuntu terminal to see whether the Depth Quality Tool can be launched and used successfully like the Viewer can, please?

Thank you. Next, could you try the command rs-depth-quality in the Ubuntu terminal to see whether the Depth Quality Tool can be launched and used successfully like the Viewer can, please?

There is something wrong with the system. Recompiling and installing. When I execute the "./scripts/patch-realsense-ubuntu-L4T.sh" script, the following prompt appears, and then I skip this step. I'm not sure what this prompt means
Jetson Board (proc/device-tree/model): quill
Jetson L4T version: 28.2.1
Unsipported JetPack revision 28.2.1 aborting script

Thank you. Next, could you try the command rs-depth-quality in the Ubuntu terminal to see whether the Depth Quality Tool can be launched and used successfully like the Viewer can, please?

the command rs-depth-quality in the Ubuntu terminal to see the Depth Quality Tool can be launched and used successfully like the Viewe.
but when I run the rs-capture command, the following prompt appears
21/04 02:39:41,100 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,229 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,358 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,488 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,615 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,743 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:41,877 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
21/04 02:39:42,000 ERROR [548205945296] (synthetic-stream.cpp:48) Exception was thrown during user processing callback: \ufffd
RealSense error calling rs2_pipeline_wait_for_frames(pipe:0x9e61c0):
Frame didn't arrive within 15000

I note the error in your log Jetson L4T version: 28.2.1  Unsipported JetPack revision 28.2.1 aborting script.  I also note that you built from source with the Native Backend, since you used scripts/patch-realsense-ubuntu-L4T.sh
Your L4T version 28.2.1, which equates to JetPack 3.3, is quite old now.  The documentation for this installation method states that the installation script was verified with JetPack 4.2.3 [L4T 32.4.3], 4.3 [L4T 32.3.1] and 4.4 [L4T 32.2.1].
It is also worth mentioning that SDK 2.43.0 added support for Jetpack 4.4.1+4.5 (32.4.4 + 32.5)
Would it be possible to upgrade your L4T version to one of the above see whether it makes a positive difference?

Hi @liu-jiaming  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


fix: Add output of failed events to logs

Disclaimer: Thanks to the code provided by @Raffy23 in PR https://github.com/keptn-contrib/job-executor-service/pull/229
Fixes #234 by providing an appropriate error message

Unfortunately, we cannot easily fix #235, as it would require us to delete jobs (which we shouldn't do at the moment). I would dedicate the fix of #235 to the refactoring ticket of #244

Shouldn't we also implement a basic unit test for GetFailedEventsForJob?

Added two unit tests and rebased main branch


⚠ Secret Website 3 has degraded performance

In 3109d02, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2021 ms

Resolved: Secret Website 3 performance has improved in eb06130.


Consider building queries in QueryLibrary or building gaiaQueryLibrary

How to save the save OHDSI GIS queries? Follow this existing model for general OMOP queries
https://github.com/OHDSI/QueryLibrary

QueryLibrary or PhenotypeLibrary (or both?) for gaiaDB?
QueryLibrary could focus specifically on queries outside of the world of cohort building. PhenotypeLibrary could focus only on common queries used to filter cohorts.


FYI: COPR RPMs for sysbox for Fedora/CentOS/RHEL

Hi, just wanted to let you know I spun up COPR RPMs for Sysbox.
https://github.com/karellen/karellen-sysbox
https://copr.fedorainfracloud.org/coprs/karellen/karellen-sysbox/
Please review and let me know if you're satisfied with notices and LICENSES:
https://github.com/karellen/karellen-sysbox/blob/dc4588447732e50577f0832262fd67ac6976b7ac/karellen-sysbox.spec#L79-L81
If you guys release official RPMs that you will be supporting I will switch to those.

@arcivanov, thanks for doing this and for letting us know.
Wondering if you would be open to contribute these changes to sysbox-pkgr repository now that we have made it open source. You would need to make minor adjustments to plug in your rpm recipes within our Makefile building targets, but at first glance it looks like simple changes now that you already have done this work.

@rodnymolina
This is released under ASL 2.0 so I'll be very happy to contribute. However, this is specific to COPR infra and I'm not sure whether that's what you want.
As far as "plug in your rpm recipes within our Makefile building targets" one of the reasons I didn't do that (and I tried!) is that COPR is not capable of running containers for building, so I used -local building targets and extracted relevant portions of platform-specific setup from the Dockerfiles.

Oh, got it, I'm not familiarized with COPR so I missed that when I initially read your comment. And you are right, we heavily rely on containers to build (and also test) our images, so I guess the official COPR installer would need to wait. Having said that, we are still interested in creating an rpm installer, it's just that we haven't had the cycles for it yet.


Support for ONETOMANY for inline models

I have two models: Employee and Invitation. An Employee my have many invitations. That being said I would like to simultaneously create a new Invitation as well as an associated Employee. I cannot do that since the logic is limited to MANYTOONE relationships:
        for prop in target_mapper.iterate_properties:
            if hasattr(prop, 'direction') and prop.direction.name in ('MANYTOONE', 'MANYTOMANY'):
                if issubclass(model, prop.mapper.class_):
                    reverse_prop = prop
                    break

duplicate of #1405


wait_for_message is not very robust

Is your feature request related to a problem? Please describe.
wait_for_message only works if there is a single request in flight from a given behaviour.
Describe the solution you'd like
Generalise it so multiple concurrent requests can be filled without any message loss.

Discussed under [WD] Add concurrent requests support to wait_for_message internally. Considered not a priority


fix: not found operation should return 405

A proposed improvement which fixes #81

Codecov Report

Merging #82 into master will increase coverage by 0.36%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   94.04%   94.41%   +0.36%
==========================================
  Files           8        8
  Lines         168      179      +11
==========================================
+ Hits          158      169      +11
  Misses         10       10

Impacted Files
Coverage Δ

lib/middleware/operation.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add Write::appendIfMatches() and appendUnlessMatches() for taskWriteToFile.

Useful for setting up configuration files where the desired configuration may or may not already exist in the file.

Thanks!


Sonoff Zigbee bridge is not pairing correctly with Sonoff switch

I'm unable to pair Sonoff wireless Zigbee switch to Sonoff Zigbee bridge, I'm not sure if it's a compatibility issue or I'm doing something wrong:
When pairing the switch
ZbPermitJoin 1

I get the correct console response:
tele/tasmota_DC0B35/3330/SENSOR = {"ZbReceived":{"0x3330":{"Device":"0x3330","Manufacturer":"eWeLink","ModelId":"WB01","Endpoint":1,"LinkQuality":150}}}

Binding
ZbBind {"Device":"0x3330","ToDevice":"0x0000","Endpoint":1,"ToEndpoint":1,"Cluster":6}

Result:
tele/tasmota_DC0B35/RESULT = {"ZbBind":{"Device":"0x3330","Status":0,"StatusMessage":"SUCCESS"}}

But when toggling the switch, it's always OFF (power=0), I get the same response on every toggle.
ZIG: {"ZbZCLReceived":{"groupid":0,"clusterid":6,"srcaddr":"0x3330","srcendpoint":1,"dstendpoint":1,"wasbroadcast":0,"LinkQuality":124,"securityuse":0,"seqnumber":19,"fc":"0x18","manuf":"0x0000","transact":19,"cmdid":"0x01","payload":"0000001000"}}
ZIG: ZbZCLRawReceived: {"0x3330":{"0006/0000":0}}
RSL: tele/tasmota_DC0B35/3330/SENSOR = {"ZbReceived":{"0x3330":{"Device":"0x3330","Power":0,"Endpoint":1,"LinkQuality":124}}}

This github is for documentation of zigbee device support.
Any question or issue has to be done in tasmota github.

I guess this issue can be closed? But its not up to me of course. Just a friendly reminder.


Allow specifying version for from:gh-r

Related to b4b4r07/releases#2.
This change makes it possible to use syntax such as:
zplug "junegunn/fzf-bin", \
    from:gh-r, \
    at:0.10.7, \
    as:command, \
    of:"*${(L)$(uname -s)}*amd64*", \
    file:fzf

Thanks.

:-D


NestedScrollView : is there any way to get the innerController

My NestedScrollView's body is ListView Widget，I want to control the  NestedScrollView  body to scroll. i.e: auto scroll to xx offset.(_innerController.jumpTo(offset)).   I've know the innerController control the [Scrollable]-based widget body .   but I can't get the Controller。
Can this function be supported?

Hi @feiyang1235
This platform is not met for assistance on personal code.
Please see https://flutter.dev/community for resources for asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Is background scroll a feature of a bug?

If you open lity, the background is still scrollable.
The lity container is not of course, but if you open lity on mobile and try to zoom the picture, it sometimes get's difficult because of the scrollable body.
Currently I'm just hooking body -> overflow: hidden on open and body -> overflow: auto on close.
Is that something you're maybe interested in adding to the project itself?

This is more like a feature. Since i want to keep lity lightweight, i didn't implemented it. You need to implement that by yourself exactly how you did it.

I've release the new version 1.4.0 which now toggles a lity-active class on the <html> element.
Simply add
.lity-active,
.lity-active body {
    overflow: hidden;
}

if you want a non-scrollable body while lity is opened.
You could even improve this (eg. in combination with Modernizr) and only make the body non-scrollable on touch devices:
.lity-active.touch,
.lity-active.touch body {
    overflow: hidden;
}

Thanks, this is the best solution. Keeping the core simple while being easily extended and updated.


Uncaught runtime errors:

m-table-edit-cell.js:206 Uncaught TypeError: (0 , _styles.withTheme) is not a function
at ./node_modules/material-table/dist/components/m-table-edit-cell.js (m-table-edit-cell.js:206:1)
at options.factory (react refresh:6:1)
at webpack_require (bootstrap:24:1)
at fn (hot module replacement:62:1)
at ./node_modules/material-table/dist/components/index.js (index.js:113:1)
at options.factory (react refresh:6:1)
at webpack_require (bootstrap:24:1)
at fn (hot module replacement:62:1)
at ./node_modules/material-table/dist/default-props.js (default-props.js:26:1)
at options.factory (react refresh:6:1)

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Launch pyrpl without gui

Hey all,
Thanks for making this great library =) I made my PhD possible.
I am currently working on a headless linux pc with means that qt cannot be launched and it will cause an error. Is there a possibility to launch the pyrpl without the gui. I would like to use it from jupyter notebooks only.
Thanks in advance for your help.
Enjoy your days,
Nick

Hello,
The gui=False is not enough since I would like to get rid of Qt all together. However, the link you send is super interesting. I will have a look at that.
Thanks.
Enjoy your day!

From: michaelcroquette @.>
Sent: Friday, June 21, 2024 5:09 PM
To: lneuhaus/pyrpl @.>
Cc: Nick Sauerwein @.>; Author @.>
Subject: Re: [lneuhaus/pyrpl] Launch pyrpl without gui (Issue #511)
Hello,
Have you tried p = Pyrpl(hostname=hostname, config=config, gui=False) ?
I'm not sure if this will work for you however. If not you can checkout this fork of pyrpl : https://github.com/ograsdijk/StemLab.
—
Reply to this email directly, view it on GitHubhttps://github.com/lneuhaus/pyrpl/issues/511#issuecomment-2182935834, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BGU6ZWFPEHNPINT4MODMAY3ZIQ63ZAVCNFSM6AAAAABHFNCHWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSHEZTKOBTGQ.
You are receiving this because you authored the thread.Message ID: @.***>


docs(chinese):mirror

Signed-off-by: shaoanianche<EMAIL_ADDRESS>

Thanks for your contribution! :tada: :+1:


output.chunkLoadingGlobal can't handle string like 'xxx';

System Info
System:
OS: macOS 13.6
CPU: (10) arm64 Apple M1 Pro
Memory: 2.21 GB / 32.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.15.0 - ~/Library/Caches/fnm_multishells/11279_1676208385213/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 9.5.0 - ~/Library/Caches/fnm_multishells/11279_1676208385213/bin/npm
Browsers:
Chrome: 118.0.5993.117
Safari: 16.6
Details
Rspack output.chunkLoadingGlobal can't resolve the string like 'xxx'.
When we set the output.chunkLoadingGlobal as 'LOADABLE_LOADED_CHUNKS', the rspack output is:
(self['_LOADABLE_LOADED_CHUNKS_'] = self['_LOADABLE_LOADED_CHUNKS_'] || [])

then webpack output is:
(self["__LOADABLE_LOADED_CHUNKS__"] = self["__LOADABLE_LOADED_CHUNKS__"] || [])

The chunkLoadingGlobal as __LOADABLE_LOADED_CHUNKS__ was set by loadable-compnent. It ensuresloadable loaded normally.

Reproduce link
https://github.com/GiveMe-A-Name/rspack-chunkLoadingGlobal
Reproduce Steps

pnpm install
pnpm run build:all
watch the dist/rspack/web && dist/webpack/web

!canary


Consider handing PyPI package over to dwolfpi that hosts a more maintained py3 compat version

https://github.com/dwolfhub/zxcvbn-python/issues/23

/cc @rpearl

I no longer work at Dropbox and don't have any say over this repo at this point, sorry!

@rpeal could you add dwolfpi to https://pypi.python.org/pypi/zxcvbn ?

@rpearl could you add dwolfpi to https://pypi.python.org/pypi/zxcvbn ?

While I control the zxcvbn package on pypi, given that the IP belongs to Dropbox, please contact them. https://www.dropbox.com/contact

I just pinged @gvanrossum directly. I'm hoping he can help speed this up.

This is about ownership of the PyPI entry. Currently zxcvbn is unmaintained by @rpearl and python-zxcvbn is maintained by @dwolfhub

But even if I wanted to I couldn't change the PyPI entry's ownership,
because it's @rpearl's. Given that Dropbox itself uses @dwolfhub's version,
I think it's reasonable to say that if @rpearl agrees he can transfer
ownership to @dwolfhub, as long as you keep the original
https://pypi.python.org/pypi/zxcvbn/1.0 entry around (for those poor souls
who want to try out the original).
Or do you want an official legal opinion from Dropbox? I can ask our lawyer
but it will take a while. Let me know if you need this.

@gvanrossum I see no reason to remove 1.0. I'm sure there are projects still using it.
@rpearl my pypi username is dwolfpi FYI
Is it possible to add a message to the README in this repository to note that this version is deprecated? Just trying to avoid confusion.

OK, I added a note to the README (https://github.com/dropbox/python-zxcvbn/pull/20). @rpearl you have my blessing to had over the PyPI project to @dwolfhub.

Okay, with that word @gvanrossum / Dropbox, I've added @dwolfhub as an owner of the pypi project. Sorry for the confusion here; I just didn't want to hand over a thing which (while in my control) was developed solely while at Dropbox.

Thanks @rpearl! You did the right thing here.


ATLAS-3940 : Upgrade snakeyaml to a version without CVE-2017-18640

Maven package cassandra-all has transitive dependency on org.yaml:snakeyaml:1.11 which has CVE-2017-18640:https://nvd.nist.gov/vuln/detail/CVE-2017-18640
Raised a PR to Cassandra to upgrade the version of snakeyaml to 1.26 where the CVE got fixed : https://github.com/apache/cassandra/pull/736
Upgrade to the latest version once cassandra-all new release become available

CI passed.
https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/15/console
+1 for PR, @crazylab , Thanks for PR.


error TS2305: Module '"typy"' has no exported member 't'

when importing type
import { t } from "typy";

with typescript 3.4.3
I get the following error
error TS2305: Module '"typy"' has no exported member 't'.

6 import { t } from "typy";

looking at the index.d.ts this looks right.
t is actually a member of Type.
But creating a new Type and than calling t on it looks really clunky.
Not sure what to do here.

@analizapandac do you have any idea what this might be?

@flexdinesh As @burner mentioned, t is a member of Typy however it's also defined as the default export.
I'm not sure yet how to fix this one, I will check this out. For now, since you're using ES6 import style, you can use the default t.
import t from "typy"

I tried add
declare module "typy" {
	export function t(obj: any, nestedKeys?: string): Typy

to my index.d.ts
and Typescript is happy again

@burner You're right. Replacing typy with t instead solved the issue.
export function t(input: any, objectPath?: string): Typy
export default t;

"Schema, addCustomTypes"  should be exported differently then.

Yup, they should be included in the type definitions file as well since right now, both of these are not declared there.

did we miss any other types?
can you create a new PR or should I?

I think just these 3 needs to be changed. You can create the PR then we'll review it :)

Changes published in v3.2.2


PowerCommander - Add-KeeperSecretManagerClient can't return base64 configuration string

PowerCommander doesn't seem to have the option to return a base64 configuration string when creating a Keeper Secret Manager Client. Only the one time access token is provided as a return value. Example:
Add-KeeperSecretManagerClient -App “someApp”
Whereas, the regular Commader CLI can do the following:
secrets-manager client add --app someApp --config-init b64
Could you please add the ability to create base64 configs in the PowerCommander Add-KeeperSecretManagerClient function?
Alternatively, is there a current method in PowerCommander to convert a one time access token into a base64 config?
Extra Context
I am evaluating the Secret Manager product as a possible purchase add-on for my organization (we are a current Keeper customer).
Here's what I am trying to accomplish:
During an automated Infrastructure Deployment...

Create Shared Folders, Applications and Device Clients with PowerCommander
Invoke Keeper Terraform to create secrets for the Shared Folders (your terraform module requires a configuration, which is why I need the b64 value)
Use SecretsManager PowerShell plugin to pull secrets as needed (also using b64 config value).

It looks like this task belongs rather to secrets-manager project then to keeper sdk for .Net.
Secrets Manager already has a code sample of converting one time token to KSM configuration
https://github.com/Keeper-Security/secrets-manager/blob/a8187c3d1f215f1dbcdad9bccaa61a0d4de9b176/sdk/dotNet/SecretManagement.Keeper/SecretManagement.Keeper.psm1#L37
That configuration could be converted to base64 text as

Load KSM library
a. $module = Get-InstalledModule SecretManagement.Keeper
b. Add-Type -Path "$($module.InstalledLocation)\SecretManagement.Keeper.dll"
Get configuration from the token
a $result = [SecretManagement.Keeper.Client]::GetVaultConfigFromToken($token).GetAwaiter().GetResult()
Convert configuration to JSON
a. $js = $result.Data | ConvertTo-Json
Convert JSON to Base64
a. $b64 = [System.Convert]::ToBase64String([Text.Encoding]::UTF8.GetBytes($js))

I will transfer this issue to secrets manager repository

@sk-keeper - I believe this repo was the correct spot for this issue.
You are correct that KSM can be used to exchange a one-time token for a base64 configuration.
However, I'd like to see that ability added to PowerCommander. Specifically, the Add-KeeperSecretManagerClient function.
In full-fledged Keeper Commander CLI, you can get a base64 config back at the time you add/create a client. For example:
secrets-manager client add --app someApp --config-init b64
I would like that same ability in PowerCommander's Add-KeeperSecretManagerClient function.

KeeperSDK/PowerCommander and SecretsManager are two separates APIs.
In order to start using SecretsManager a one time access token should be created. It is done with KeeperSDK.
KeeperSDK does not know how this token is going to be used.
It is SecretsManager API that converts one time access token access token into base64 configuration.
Python's Commander distributes both Keeper and SecretsManager SDKs
It is OK for Python's Commander since it is not a library like .Net Keeper SDK.
We do not want to distribute SecretsManager SDK with .Net Keeper SDK.
Moreover PowerCommander targets .Net Framework while SecretsManager SDK uses .Net Core.
You mentioned in your original message:
Use SecretsManager [PowerShell plugin](https://docs.keeper.io/secrets-manager/secrets-manager/integrations/powershell-plugin) to pull secrets as needed (also using b64 config value).

It looks like you already use both SDKs in your setup. You can use SecretsManager SDK to create Base64 configuration from the one time access token.
Ideally SecretsManager team can expose that functionality as cmdlet in SecretsManager PowerShell plugin

@rdavisunr - If you are going to use both SDKs then extending SecretsManager SKD would be the easiest solution.
Otherwise we could try to convert one time access token into KSM configuration using direct KSM backend API.

It looks like you already have both SDKs in your setup. You can use already installed SecretsManager SDK to create Base64 configuration from the one time access token.

The part of the deployment that uses the Secrets Manager PowerShell plugin runs separately and in a different environment from the part that is creating folders, apps, clients and secrets.
Ultimately, I was just hoping for feature parity between Keeper Commander CLI and PowerCommander. But, I understand the limitations after your explanation.

We'll add that missing step (exchange one time token for ksm configuration) into Keeper SDK. It is just one API call.

PowerCommander v0.9.3 is published to PowerShell Gallery.
It adds -B64 parameter to Add-KeeperSecretManagerClient cmdlet.


BasicRxJavaSample issues : data  can not be refreshed!

when exit app and open again,click the update button,the data has not changed,still the old data.

@lizixian18 @florina-muntenescu This issue is fixed here . Please merge, if it looks fine.

OK，Thanks!

Merged. thank you for the PR!


Update keyboard shortcuts & full view

Add keyboard shortcuts for the 3 new views 🙌
Update the shortcuts list in settings
Don't let the details view zoom in to full screen (hiding all the metadata)

👌 no need to review this PR -- it's small enough 👍

Sweet! 🍬


Just make it work on Safari

I hereby file this issue as a todo item to make it possible to utilize the IndexedDBShim without having to modify it and call __useShim() etc. Just include the shim and then include Dexie and it should use the shim by default!
== Background ==
Ever since Safari came with its poor native indexedDB support, lots of users have been struggeling with making Dexie work on Safari. There have been some workarounds introduced into Dexie that makes it behave better in Safari but still not perfect. Many users feel that due to the poor indexedDB support built-in to Safari, it would be better to continue using the IndexedDBShim (targetting the WebSQL backend). However, this had become really tricky and the following issues suggests how to make the shim work on modern safari browsers:
Issue #68
Issue #65
Issue #63

Notice: The indexedDBShim is not 100% perfect but it seems that it is still better than the broken native indexedDB support. There are some issues with the shim as well but most common use cases works pretty well.

Fixed.
From now on, if you prefer to use IndexedDBShim before the native IndexedDB support on Safari, just include the indexedDBShim before including Dexie.js and it will use it instead of the native indexedDB.

Hi. Am using the latest Dexie and indexeddbshim. Included shim before Dexie as proposed, then it broke in Firefox, and in Chrome it used WebSQL as default.

Just include the shim for safari browsers.

Hi, in my webapp I want to use native implementation of IndexedDB in case of Chrome Browser, and the shim if Safari. If I include the shim before dexie, all browser will use websql over native implementation... There is a way I can use to force dexie to use the shim only if the browser is Safari (all implementations iOS/OSX, cause if always poor...) ?

Inspect the user agent and include the shim if it matches.
This is typically done server-side but you can also accomplish it client-side only:
<script>
if (is_safari) {
    document.write('<script src="https://rawgit.com/axemclion/IndexedDBShim/master/dist/indexeddbshim.js">\x3C/script>');
}
</script>

Actually I currently use this solution, but is so dirty :)
In my mind there is something like:

var db = new Dexie("MyDatabase",{ force_shim_if_safari:true });

Is it possible to implement something like this in future implementations?

It's possible in version 1.4.x to specify the implementations of indexedDB and IDBKeyRange to use:
var db = new Dexie ("MyDatabase", {
    indexedDB: window.indexedDB, // or the shim's version
    IDBKeyRange: window.IDBKeyRange // or the shim's version.
});


Cleans up remote controller class.

Removes most of the static methods in favor of instance props.

Codecov Report

Merging #74 into develop will decrease coverage by -3.84%.

@@             Coverage Diff             @@
##           develop      #74      +/-   ##
===========================================
- Coverage    85.96%   82.13%   -3.84%
===========================================
  Files           17       17
  Lines          577      610      +33
===========================================
+ Hits           496      501       +5
- Misses          81      109      +28

Impacted Files
Coverage Δ

src/lib/Constants.ts
100% <ø> (ø)
:white_check_mark:

src/core/Remixer.ts
90.32% <100%> (-0.99%)
:x:

src/lib/LocalStorage.ts
73.07% <100%> (-6.93%)
:x:

src/lib/Remote.ts
36.7% <33.33%> (-11.44%)
:x:

Continue to review full report at Codecov.

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

Codecov Report

Merging #74 into develop will decrease coverage by -3.84%.

@@             Coverage Diff             @@
##           develop      #74      +/-   ##
===========================================
- Coverage    85.96%   82.13%   -3.84%
===========================================
  Files           17       17
  Lines          577      610      +33
===========================================
+ Hits           496      501       +5
- Misses          81      109      +28

Impacted Files
Coverage Δ

src/lib/Constants.ts
100% <ø> (ø)
:white_check_mark:

src/core/Remixer.ts
90.32% <100%> (-0.99%)
:x:

src/lib/LocalStorage.ts
73.07% <100%> (-6.93%)
:x:

src/lib/Remote.ts
36.7% <33.33%> (-11.44%)
:x:

Continue to review full report at Codecov.

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


Fix: maintain and fix the quickstart notebook

What?
Maintain the quickstart notebook to support the latest jax, trained over the current jumanji rware ver and fix the evaluator code based on the recent changes to the repository

Hard to review the notebook,
What do you mean, @sash-a? 🤔


The neo-tree integration seems to be broken on small window width

Description
The neo-tree integration seems to be broken on small window width
https://github.com/shortcuts/no-neck-pain.nvim/assets/51330172/15c27e56-2ae9-46d6-a278-db52d2f5e693
Steps to reproduce

Set small window width (80 for example)
Toggle neotree
See that your code opened in wrong buffer

Expected behavior
Neotree integration will work with any width
Environment

Neovim version: Nightly
no-neck-pain.nvim version: latest
Plugin clash: neotree

Hey @Demianeen thanks for using the plugin!!
I've tried to reproduce myself but I'm not able to see any issue with the plugin, neither to see the bug on your video. Could you point to a timestamp where it happens?
Thanks!

Sure @shortcuts, on 9th second you will see that instead of focusing on buffer with a file, it focuses on left pane buffer that centers the main file buffer. In my case, setting width to 80. Config that reproduces it for me:
{
    'shortcuts/no-neck-pain.nvim',
    event = 'BufEnter',
    opts = {
      width = 80,
      -- --- @type table
      -- mappings = {
      --   -- When `true`, creates all the mappings that are not set to `false`.
      --   --- @type boolean
      --   enabled = true,
      --   -- Sets a global mapping to Neovim, which allows you to toggle the plugin.
      --   --- @type string
      --   toggle = '<Leader>unp',
      --   -- Sets a global mapping to Neovim, which allows you to toggle the left side buffer.
      --   --- @type string
      --   toggleLeftSide = '<Leader>unql',
      --   -- Sets a global mapping to Neovim, which allows you to toggle the right side buffer.
      --   --- @type string
      --   toggleRightSide = '<Leader>unqr',
      --   --- @type string | { mapping: string, value: number }
      --   widthUp = { mapping = '<Leader>un=', value = 20 },
      --   --- @type string | { mapping: string, value: number }
      --   widthDown = { mapping = '<Leader>un-', value = 20 },
      --   -- Sets a global mapping to Neovim, which allows you to toggle the scratchpad feature.
      --   --- @type string
      --   scratchPad = '<Leader>uns',
      -- },
      autocmds = {
        enableOnVimEnter = true,
      },
      -- buffers = {
      --   scratchPad = {
      --     enabled = false,
      --     fileName = 'NeovimNotes',
      --     -- set to `nil` to default
      --     -- to current working directory
      --     location = '~/Documents/',
      --   },
      --   bo = {
      --     filetype = 'markdown',
      --   },
      -- },
    },
  }

Thanks for the details, and I'm glad you like the plugin ❤ I try to make it as extensible as possible :)
I'm sorry but I've tried to reproduce your issue with many screen sizes etc. but I can't make it focus the wrong buffer, my guess is that you have an other plugin that conflicts with all of that 🤔
Alternatively, you can set autocmds.skipEnteringNoNeckPainBuffer to true which aims at re-routing focus on side buffers to the main one, which should fix your issue
I'm closing this issue unless you have more details, feel free to reopen!


Investigate Monitor Query tests timing out (14 test failures)

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1085694&view=ms.vss-test-web.build-test-results-tab

opened in wrong repo by accident


Rectified typos & grammar in docs

What does this PR do?
A few typos and grammatical errors were discovered and have been corrected.
/cc: @8bitmp3, @marcvanzee
Checklist

[x] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[x] The documentation and docstrings adhere to the
documentation guidelines.

Hey @Neilblaze, thanks for doing this!
There are a couple of empty lines in .rst files which are intentional as they align two pieces of code in a diff view.

@cgarciae rolling back changes for those two files. I also see a couple of empty lines in ../advanced_topics/linen_upgrade_guide.rst & ../guides/linen_upgrade_guide.rst too. Should I also rollback the changes for them too?

Yeah, please rollback anything in either side of a codediff block.

Hey, can you fix the pre-commit hook errors? Then I can go ahead and merge it.

Hi, could you rebase to the google/main branch and make sure all the tests pass? This is a large change and we don't want it to break head. Thank you!

@IvyZX Sure thing! I'll do it by EOD

@IvyZX done! ✅

hi @Neilblaze, seems like there's a trailing whitespace error

Codecov Report

Merging #3021 (b48c765) into main (332b7d0) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3021      +/-   ##
==========================================
+ Coverage   81.97%   82.00%   +0.03%
==========================================
  Files          55       55
  Lines        6002     6035      +33
==========================================
+ Hits         4920     4949      +29
- Misses       1082     1086       +4

Impacted Files
Coverage Δ

flax/core/lift.py
96.06% <ø> (ø)

flax/core/scope.py
89.80% <ø> (ø)

flax/errors.py
85.96% <ø> (ø)

flax/jax_utils.py
44.44% <ø> (ø)

flax/linen/linear.py
96.96% <ø> (ø)

flax/linen/transforms.py
94.14% <ø> (ø)

flax/metrics/tensorboard.py
92.85% <ø> (ø)

flax/serialization.py
94.56% <ø> (ø)

flax/training/common_utils.py
0.00% <ø> (ø)

flax/training/dynamic_scale.py
0.00% <ø> (ø)

... and 6 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


hive write error

flutter: Transition { currentState: ContributionState(PageState.initial, null, null), event: SetScores { score: Instance of 'ScoreModel' }, nextState: ContributionState(PageState.success, null, Instance of 'ScoreModel') }
flutter: Model Class: Closure: (dynamic) => MemberModel?
flutter: NoSuchMethodError: The getter 'account' was called on null.
Receiver: null
Tried calling: account
[VERBOSE-2:ui_dart_state.cc(199)] Unhandled Exception: Unhandled error NoSuchMethodError: The getter 'account' was called on null.
Receiver: null
Tried calling: account occurred in Instance of 'MemberBloc'.
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:54:5)
#1      MemberModelCacheItemAdapter.write
#2      BinaryWriterImpl.write
#3      BinaryWriterImpl.writeFrame
#4      StorageBackendVm.writeFrames.<anonymous closure>
#5      StorageBackendVm.writeFrames.<anonymous closure>
#6      ReadWriteSync.syncWrite.<anonymous closure>
#7      _rootRunUnary (dart:async/zone.dart:1362:47)
<asynchronous suspension>
#8      BoxImpl._write<…>
flutter: mapHttpError: SocketException: OS Error: Operation timed out, errno = 60, address = mainnet.telos.net, port = 64099
flutter: Transition { currentState: MemberState(PageState.loading, igorberlenko), event: OnLoadMemberData, nextState: MemberState(PageState.failure, igorberlenko) }

@n13 Was this issue resolved with the last merge?


Updated names.text

added my name to line 37

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.


Test to prevent commits containing binary blobs

This is an experimental pull request, as I'm testing out some Jenkins functionality at the same time. I will confirm when it is ready for merging.

Commits binaries, who does that?!

Apparently a lot of people. The size of the github repo download is 36.7 MB.
This is just a measure to prevent further bloat. We should ideally remove some binary files from our git history as well.


s/fireeye/mandiant

Rename all FireEye refs to Mandiant

This one should be good to go, all refs besides the capa-dev email are fixed.

Well, let me see if the tests should be fixed/addressed. Will do so tomorrow.


make_tester should not use filesystem at all

LcCatalog.make_tester() currently depends on a temporary directory. Removing this means:

removing path operations from workflows when _test is True
allowing indexes and backgrounds to be generated and used in memory without being saved

closed on 0.3


Skip the pipeline for a datatype if the Xena Matrix is found

I am using this in the instance. Let me know if we need it, I will add tests and docs. This is on top of #47.

In general, how do you know if the existing matrix is up to date and should be skipped? If you don't rely on some checks, you decide that arbitrarily. If we decide arbitrarily, we should just use a new script to be explicit about what exact project(s) and datatype(s) we want (like --projects and --datatype).

This command is kept for user convenience only and updating part is left to the user to check.
Suppose the pipeline is run and due to some errors (500 etc.), then either we can manually check or by unfinished.json and then execute the commands, but that would be somewhat more time-consuming.

f we decide arbitrarily, we should just use a new script to be explicit about what exact project(s) and datatype(s) we want (like --projects and --datatype).

Avoided this since this the command will be really long.

Is good to have a check to see what matrix is there. But I really don't think filename check is the right way to go. And it's not worth the time you saved if you are not sure the data is right. I assume retry works, no 4xx response and you don't have too many unfinished to do?

The ETL is done. Only setting the hub is left.


Typo

There's a typo in "Permanently Delete a Site" Step 3 - it says "yo" instead of "to".

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

ID: a3bd11d3-f2a4-2811-09bd-449c3c3607da
Version Independent ID: 2f47da99-02e6-3357-ed41-b28bd5190907
Content: Delete a site - SharePoint Online
Content Source: SharePoint/SharePointOnline/delete-site-collection.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

@tanktopArmy  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@tanktopArmy Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


ANC Register page

it contains the below features

anc data with opensrp table
filter option only with searchText
pagination
test cases

snapshot of ANC


It takes PdfViewer for uwp to work with INk

there is no pdfviewer that works with Windows Ink, it would be to count a PDFViewer for UWP

Are you looking for a PDF viewer UWP app that supports pen, or for a PDF viewer control that could be used in a UWP app?

A control PdfViewer for use with stylus

@Javier118 You could render the PDF as an image and put it inside an InkCanvas and render it back to PDF on save

But  it's better to have pdfviewer that works with Ink.

But  it's better to have pdfviewer that works with Ink.


#249- only import 'level' when it's actually used

i.e. if a developer provides their own createLevelDatabase then we will use that instead of 'level'
in order to support this for both node and browser, we need to switch to using dynamic import (instead of require, as that's not defined for node when "type": "module")
most of this commit is changing sync logic to async for that purpose (though it also allows for more flexibility when providing a custom createLevelDatabase)

@thehenrytsai can you publish an unstable version of this so @shamilovtim can consume?

@thehenrytsai can you publish an unstable version of this so @shamilovtim can consume?

This one can probably just get merged. I need the unstable of the other PR and this rebased into that PR first

@mistermoe and @shamilovtim, approved. Letting @dcrousso have the honor of merging. Feel free to merge if you can't wait.


[1.1] what’s the difference  matter sdk  from sdk of google home ？

Reproduction steps
google home log：
01-15 15:36:06.619  3630  3742 I native  : I0000 00:00:1705304166.619559    3742 chip_logging.cc:17] CHIP: DIS: UDP:[fe80::32de:4bff:fedb:8aa7%wlan0]:5540: new best score: 8
in fact,matter sdk has no  best score value 8；
and google home can add LLA interface wlan0.
so matter sdk  google home used has some unkown modify？
Bug prevalence
NA
GitHub hash of the SDK that was being used
1.1
Platform
android
Platform Version(s)
1.1
Type
Platform Issue
Anything else?
No response

so matter sdk google home used has some unkown modify？

Sounds like it, yes.


Bubbles off

Randomly bubbles will be off like in the attached screenshot
Running unc0ver 5.0.1 jb with version 0.4.1 of Notifica
apps im using that may/may not cause this issue are kalm, grupi, colorbanners3, tinybanners2

issues here as well https://www.reddit.com/r/jailbreak/comments/gqwbn4/question_any_one_else_experiencing_this_glitch/


Use a decorator to set metadata and a service to manipulate them inside a component

An issue to keep track of decorator  + service thing in one place.
Related to #22 #26 and #37

[x] use service to manipulate queries and mutations
[x] set metadata with queries and mutations
[x] use defaultClient

I'm going to put here my thoughts and plans for the future API change.

Also waiting for this issue of Angular

@kamilkisiela what's the status on this one? Looks like an issue was resolved on Angular-cli referenced by @Urigo and all other roadblocks out of the way -- saw that #165 review was never merged.


non validation of  in Svayambhu-CritEd.xml

Dear @ajaniak — what change should I make to make the validation problems concerning use of <ab> go away in Svayambhu-CritEd.xml?

PS The problem does not stand in the way of successful transformation...

You need to stick to your encoding model adding a <div> for your trailing <ab>. I have made the necessary changes to your file.

Thanks. Presently, the EGC (top of p. 27) states:
"For final colophons, this type of <p> or <ab> shall be considered the mirror image of text-initial invocations, in the sense that they may stand outside of the final <div> but before the end-tag of the containing element <body>. And, analogously to the case of <div>-level invocations, any colophons at the <div> level should be included before the closing tag of the containing element."
This does not seem to mean the same a creating separate <div> to enclose the <ab>.
Could you explain which encoding model you are referring to if not the above passage from the EGC? Should the EGC be modified?

I have added the <div>around the colophons and invocations.


Page size select's icon is hoverable but not clickable

Please provide a working example with a bug (git repo, codesandbox, source code at least, etc).
It takes a lot of time to set up environment to confirm an issue.
So I will resolve issues faster.
Configuration:

Web browser and it`s version: chrome
Operating system and it`s version:
vue-pdf-app version: 2.1.0

Steps to reproduce the problem:
see official demo
What is the expected behavior?
Once the icon is clicked, the select should be expanded
What went wrong?
When the icon is hovered, the cursor doesn't become pointer. Nothing happened if the icon is clicked.
Attachments (screenshots, links, etc)

workaround: place the icon under the select, and set select's background transparent

Closing as this repo not maintained any more


Can't use on WSL when connecting to local lotus node and local ipfs daemon "context deadline exceeded"

I am using WSL Ubuntu 20.04 on Windows 10. I can't get powergate to work by connecting it to my already installed lotus node and already installed ipfs-go.
I installed lotus, ipfs-go, pow, and powd. I waited for the lotus daemon to finish syncing then I launched ipfs with ipfs daemon, then the pow daemon in a screen session with powd --lotustoken <mylotustokenhere>.
So the lotus daemon is synced and running, the ipfs daemon is running, and the powergate daemon is also running connected to the lotus daemon through the admin auth token. I already staged a directory and got a cid (through powergate) with the command pow data stage path/to/dir and got a 'bafy' starting IPFS hash. The directory is about 5GB in size. I created a random data bin file through a dd command. I chose to make it 5GB because I noticed that a lot of miners want you to make the job for more than 4GB. I ran the command pow config apply --watch <CID> -o -c custom-072321-config.json. This was my custom-072321-config.json file.
{
  "hot":  {
    "enabled":  false,
    "allowUnfreeze":  false,
    "unfreezeMaxPrice":  "0",
    "ipfs":  {
      "addTimeout":  "900"
    }
  },
  "cold":  {
    "enabled":  true,
    "filecoin":  {
      "replicationFactor":  "1",
      "dealMinDuration":  "518400",
      "excludedMiners":  [],
      "trustedMiners":  [],
      "countryCodes":  [],
      "renew":  {
        "enabled":  false,
        "threshold":  "0"
      },
      "address":  "<my filecoin address with 400000000 attoFIL>",
      "maxPrice":  "100000000000",
      "fastRetrieval":  true,
      "dealStartOffset":  "8640",
      "verifiedDeal":  false
    }
  },
  "repairable":  false
}

But the job failed... IDK why?
This was my error that I got:
                 JOB ID                |             STATUS             | MINER | PRICE | DEAL STATUS
---------------------------------------+--------------------------------+-------+-------+--------------
            <my job id here>           | JOB_STATUS_FAILED executing    |       |       |
                                       | cold-storage config: getting   |       |       |
                                       | cid cummulative size:          |       |       |
                                       | getting cid cummulative        |       |       |
                                       | size: calculating data size:   |       |       |
                                       | merkledag: not found           |       |       |
<myusername>@<mycompname>:~$

So I removed all of the files from the IPFS daemon pinning system with ipfs pin ls --type recursive | cut -d' ' -f1 | xargs -n1 ipfs pin rm then ran ipfs repo gc to make sure that they were garbage collected. I ran ipfs add -r --pin=false path/to/dir and got a Q starting CID and ran the command pow config apply --watch <CID> -o -c custom-072321-config.json again with the new CID starting with a Q and I got the same error as above. So I figured that the original bafy starting CID and the Q starting CID were still describing the same directory. So I ran the original pow config apply --watch <CID> -o -c custom-072321-config.json command with the original bafy starting CID and after sticking on JOB_STATUS_EXECUTING for a long time, I got this error:
                 JOB ID                |                                                              STATUS                                                              | MINER | PRICE | DEAL STATUS
---------------------------------------+----------------------------------------------------------------------------------------------------------------------------------+-------+-------+--------------
            <my job id here>           | JOB_STATUS_FAILED automatically staging cid: adding data to ipfs: Post                                                           |       |       |
                                       | "http://<IP_ADDRESS>:5001/api/v0/pin/add?arg=%2Fipfs%2F<bafystaringCID>&recursive=true":                                            |       |       |
                                       | context deadline exceeded                                                                                |       |       |

What am I doing wrong? How do I store my file on Filecoin?

cold-storage config: getting cid cummulative size: getting cid cummulative size: calculating data size: merkledag: not found

The reason for that error is a misconfiguration of the Lotus node, refer here to see exactly what you need to do. Your error msg is different from the one mentioned in the FAQ, but is the same situation.
That configures Lotus to use the go-ipfs node to look for the data for making deals.

cold-storage config: getting cid cummulative size: getting cid cummulative size: calculating data size: merkledag: not found

The reason for that error is a misconfiguration of the Lotus node, refer here to see exactly what you need to do. Your error msg is different from the one mentioned in the FAQ, but is the same situation.
That configures Lotus to use the go-ipfs node to look for the data for making deals.


subscriptions: Minor Bug-Fixes.

Both the bugs are reproducible on CZO.
2nd Commit:
scroll_element_into_container is used in only 2 other places list_cursor.js and stream_list.js and both are implemented correctly.

3rd Commit
Previously:

Now:

Merged, thanks @ryanreh99!


Increase manifest retries

What

Increase manifest retries to 5 from 3
Adds additional manifest retry logging
Adds error message to response in response.metadata.error

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


[Feature Request] Dont keep user configuration files on reset

server keeps on reseting

Please express your opinion on the feature request for keeping user configuration files by commenting on #48, instead of opening a new issue.


dependence_plot for data with multiple labels error

I am plotting the dependence of two variables for one label in a multi-label dataset.
The offensive code is:
shap.dependence_plot('SOME_VALUE', shap_values[cls][idx_cls, :], X.iloc[idx_cls])
cls is the index of the class (an int) and idx_cls is an array of indices where the label was cls.
Now, there is an output plot, but without any labels, and the cell throws an IndexError. Here's the full error:
IndexError                                Traceback (most recent call last)
<ipython-input-47-b3e2be1f755c> in <module>
      4
      5 shap.dependence_plot('SOME_VALUE', shap_values[cls][idx_cls, :],
----> 6                      X.iloc[idx_cls])

~/anaconda3/envs/valley/lib/python3.7/site-packages/shap/plots/dependence.py in dependence_plot(ind, shap_values, features, feature_names, display_features, interaction_index, color, axis_color, dot_size, x_jitter, alpha, title, xmin, xmax, show)
    242             xlim[0] * np.ones(xv_nan.sum()), s[xv_nan], marker=1,
    243             linewidth=2, c=cvals_imp[xv_nan], cmap=colors.red_blue, alpha=alpha,
--> 244             vmin=clow, vmax=chigh
    245         )
    246         p.set_array(cvals[xv_nan])

~/anaconda3/envs/valley/lib/python3.7/site-packages/matplotlib/pyplot.py in scatter(x, y, s, c, marker, cmap, norm, vmin, vmax, alpha, linewidths, verts, edgecolors, data, **kwargs)
   2862         vmin=vmin, vmax=vmax, alpha=alpha, linewidths=linewidths,
   2863         verts=verts, edgecolors=edgecolors, **({"data": data} if data
-> 2864         is not None else {}), **kwargs)
   2865     sci(__ret)
   2866     return __ret

~/anaconda3/envs/valley/lib/python3.7/site-packages/matplotlib/__init__.py in inner(ax, data, *args, **kwargs)
   1803                         "the Matplotlib list!)" % (label_namer, func.__name__),
   1804                         RuntimeWarning, stacklevel=2)
-> 1805             return func(ax, *args, **kwargs)
   1806
   1807         inner.__doc__ = _add_data_doc(inner.__doc__,

~/anaconda3/envs/valley/lib/python3.7/site-packages/matplotlib/axes/_axes.py in scatter(self, x, y, s, c, marker, cmap, norm, vmin, vmax, alpha, linewidths, verts, edgecolors, **kwargs)
   4193                 isinstance(c, str) or
   4194                 (isinstance(c, collections.Iterable) and
-> 4195                     isinstance(c[0], str))):
   4196             c_array = None
   4197         else:

IndexError: index 0 is out of bounds for axis 0 with size 0

It looks like a matplotlib problem but I cannot figure our what it is.
Thanks

It is because dependence_plot takes a 2D matrix over many samples, not just a single row. The error here should be made better though.

I'm having the same issue with depemdence_plot!

I'm having the same issue with depemdence_plot!

Updating matplotlib solved the issue!

@thatlittleboy Candidate for closing?


done argument is not passed to job functions when using @Define

According to the agenda documentation, done is passed as the second argument to the method.
Currently it is not being passed.
See below for an example:
import { IngestionService } from './ingestion.service';
import { Queue, Define, InjectQueue } from 'agenda-nest';
import { Agenda } from 'agenda';

@Queue('ingestions')
export default class IngestionQueue {
  constructor(
    @InjectQueue('ingestions') private readonly ingestionQueue: Agenda,
  ) {}

  @Define()
  async ingestBatch(job, done) {
    console.log({job, done}): // done is undefined
  }
}

Thanks @AFSoftwareEngineering , will take a look


Input with type="number" allows letters in Firefox

Describe the bug
Steps to reproduce the bug

Go to https://eds-storybook-react.azurewebsites.net/?path=/docs/inputs-input--types
In the Number field, write "hei"

Expected behavior
If you're using Chrome, these characters would be ignored, but if you use Firefox, they will not. Issue seems to come from https://bugzilla.mozilla.org/show_bug.cgi?id=1398528, and Firefox does not seem to have any plan of fixing it themself.
Specifications

Version:
Browser: Firefox
OS:

Hello and thank you for the report!
EDS form elements does not handle input validation, since it would be very difficult to capture all use cases here. See the last comment in the bugzilla link, by Daniel Holbert, why this is the case and even chromes built in solution is incomplete.
Instead we suggest projects implement their own input validation, or use a third party solution such as react-hook-form like in our storybook example 🙂

That makes sense. However, it is problematic that an EDS component behaves so differently from one browser to the next. Since not all development teams use all browsers, this can cause long-lived bugs. I would suggest removing "number" as an input type and instead encourage developers to use pattern to validate their input.

My biggest concern regarding type="number" is that in Firefox, if you input "1.3" it works nicely, but if you write "1,3" it interprets it as if you've written nothing at all. This mistake is so easy for an end user to make, and they get by default no feedback about it. If the input is optional, you risk simulations are run with different input than intended.
And as a developer using Firefox and looking at this exact component for a time, it took a while before I realized this issue. I suspect many teams have not considered it at all.

I agree this is problematic, but this is a native prop on the native input inside the eds component, so it is not something we can do anything about really. I suggest using type="text" and have your own validation to make it consistent across browsers.
I see MUI does the same thing we do, while Mantine have made seperate components for number text and password. Seems like their custom numberInput is type="text" with their own home made validation built in kind of similar to firefox behaviour maybe (1,2 turns into 1, 1e24 turns into 1e+24). This might be something we could consider in the future

That is exactly what I did, and the issue is solved in my case. But it was time consuming and unintuitive. Perhaps a simply, temporary solution would be to update https://eds-storybook-react.azurewebsites.net/?path=/docs/inputs-input--types to not list type="number" as an alternative? And perhaps have an example of how to instead use type text with pattern validation?

Yes, we can write a note about why type=number is problematic and recommend to use type=text and homemade/third part validation instead for this in the storybook. And maybe make another example. Currently we already have an example how to validate number using react-hook-form 👍

@oddvernes The PR only updates the info for the TextField. Shouldn't the documentation for the Input component be updated as well?


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.9.276",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a832ee2f2145f57443b2d565f8cb5490e8339f42"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "a832ee2f2145f57443b2d565f8cb5490e8339f42",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "atm: iphase: fix possible use-after-free in ia_module_exit()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.9.276 by commit a832ee2f2145f57443b2d565f8cb5490e8339f42. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001460


UniTensor.arange(int lst) does not exist

It would be great if arange would work with different shapes then vectors as well.
In the draft, there is a codeline:
uT = cytnx.UniTensor.arange([2,3,4], labels=["a","b","c"], name="tensor uT")

It does not work, since arange only takes integer arguments as size. I would suggest that this is implemented, otherwise it is a bit complicated to use arange (then reshape, then relabel).

this is not consistent with numpy

I don't see why you can't reshape after create this

Sure, reshaping works. I agree that it is not so nice creating inconsistency with numpy and providing both versions (arguments int list or int) to do the same thing is often not a good idea.
In this case, I thought that it might be confusing to users though, because the typical initialization is very different for arange then.
uT=cytnx.UniTensor.ones([2,3,4], labels=["a", "b", "c"], rowrank=2)

works the same way for ones, zeros, uniform etc.
But for arange I need to create a UniTensor with the wrong/uninitialized attributes and then set each one by one
uT=cytnx.UniTensor.arange(2*3*4, labels=["a", "b", "c"], rowrank=2)
ut.reshape_([2,3,4])
ut.relabels_(["a", "b", "c"])
ut.set_rowrank(2)

This is very different from how we initialize UniTensors everywhere else (in the paper).

Why not do:
uT=cytnx.UniTensor.arange(2*3*4).reshape([2,3,4],rowrank=2).relabel(["a","b","c"])

I reopen this issue to discuss reshape and reshape_ for UniTensor.
In response to one of the referee's question, my current thinking is as follows.

cytnx.Tensor should behave like numpy.array, so it should support reshape with the same syntax.
It is not rewlly well-define how to do reshape for UniTensor. One should not apply reshape to symmetric UniTensor. For non-symmetric tensor, one can define in a way similar to Tensor. But there will be some ambiguities. For example, what is the default row-rank? For existing label, if reshape essentially merge them, how do we handle it?
It seems to me that we can drop the support of reshape to UniTensor. If something like that is really needed, then use get_block->reshape->put_block.

This is current behaviour. Depending on how you do the reshape, you got different default row-rank.
X = cytnx.UniTensor.zeros([2,3,4])
X.print_diagram()

Y=X.reshape(4,3,2)
X.print_diagram()

X.reshape_(4,3,2)
Y.print_diagram()
-----------------------
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
          ---------
         /         \
   0 ____| 2     3 |____ 1
         |         |
         |       4 |____ 2
         \         /
          ---------
-----------------------
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
         --------
        /        \
        |      4 |____ 0
        |        |
        |      3 |____ 1
        |        |
        |      2 |____ 2
        \        /
         --------
-----------------------
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
         --------
        /        \
        |      4 |____ 0
        |        |
        |      3 |____ 1
        |        |
        |      2 |____ 2
        \        /
         --------

I think current behavior of reshape will make row rank=0. Which we can just
change it to be new_rank/2
On Wed, Sep 4, 2024, 03:28 Pochung Chen @.***> wrote:

This is current behaviour. Depending on how you do the reshape, you got
different default row-rank.
X = cytnx.UniTensor.zeros([2,3,4])
X.print_diagram()
Y=X.reshape(4,3,2)
X.print_diagram()
X.reshape_(4,3,2)
Y.print_diagram()
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
---------
/
0 | 2     3 | 1
|         |
|       4 |____ 2
\         /
---------
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
--------
/
|      4 |____ 0
|        |
|      3 |____ 1
|        |
|      2 |____ 2
\        /
--------
tensor Name :
tensor Rank : 3
block_form  : False
is_diag     : False
on device   : cytnx device: CPU
--------
/
|      4 |____ 0
|        |
|      3 |____ 1
|        |
|      2 |____ 2
\        /
--------
—
Reply to this email directly, view it on GitHub
https://github.com/Cytnx-dev/Cytnx/issues/325#issuecomment-2328117460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFCX3SMHE5TR3W6W5YOMESLZU2ZBPAVCNFSM6AAAAABNTPH52WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRYGEYTONBWGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Popup closed while "mouse click inside but release outside" of the Popup block

Bug Report
Steps
I am creating a login popup component in React by using React Semantic UI Popup. I am using "Events triggering the popup on click". I was trying to copy the content in the Popup block, but the popup was closed when my mouse released outside of the popup box.
import React, { Component } from 'react';
import { Button, Message, Popup, Icon, Menu } from 'semantic-ui-react';

// code inside of render return function
<Popup trigger={<Button icon>LOGIN</Button>}
       on='click'
       className='login-popup'
>
       <div className='popup-main'>
              <Message attached='bottom'>
                        Log In
              </Message>
              <LoginForm
                  {...this.props}
              />
        </div>

        <Message attached='bottom'>
              Don't have account? Signup instead.
        </Message>
</Popup>

Expected Result
I would like to keep the Popup open unless the user clicks the mouse outside of the Popup
Actual Result
The Popup closed when the mouse clicked inside and released outside of the popup box.
Version
0.85.0
Testcase
I tested it on the offical website and still get the same result.

https://react.semantic-ui.com/modules/popup/#usage-click
I tried this example and everything works as expected. Please provide a minimal CodeSandbox example and detailed steps.

@layershifter
Thank you for your checking. I think I might not explain the issue clearly. I asked this in stack overflow (here). I updated my expected result as below. Hope the update makes more sense.
Expected Result:
The Popup box should not be closed if the mousedown event happens inside of the Popup, no matter where the mouseup event triggered.

Fixed in #3575.

@jiayimu
I came across your post while trying to resolve a similar issue. I eventually resolved it by setting one of the input fields in the popup to autoFocus. Prior to that, clicking on an icon or somewhere not specifically an input, closed the popup straight away.
I see the issue has been closed but thought the comment might be helpful in any case.


Re-run babel-extract

fixes https://github.com/pallets-eco/flask-admin/issues/1974

well that's a lot of test failures for something innocuous 🤔

Right, I think that's all sorted now.
@ElLorans @LeXofLeviafan - would you mind having a glance over this and seeing if you're happy with the updates? It shouldn't affect any translation strings other than:

Removing old strings that are no longer present
Adding new msgids with empty msgstrs, ready for translations to be put in
Adding equivalent msgstrs to the en file.

I haven't used Babel before to do any translation work, so would appreciate someone else looking over this unless I've done something unexpected/undesirable.

Note that unless you override the POT-Creation-Date timestamp within the template file with a static value (e.g. using sed or any equivalent after executing pybabel extract), this timestamp will be updated in all translations every time you run the generation script (even if that particular translation was not changed and you only wanted to make sure all translations are compiled); which would also modify all compiled translations as well.
Also, here's some (ad-hoc) translations for the "Delete record" string that were merged in buku (…can't say for sure if they're accurate though :sweat_smile:)
# de
msgid "Delete record"
msgstr "Datenzatz löschen"

# fr
msgid "Delete record"
msgstr "Supprimer l'enregistrement"

# ru
msgid "Delete record"
msgstr "Удалить запись"

Note that unless you override the POT-Creation-Date timestamp within the template file with a static value (e.g. using sed or any equivalent after executing pybabel extract), this timestamp will be updated in all translations every time you run the generation script (even if that particular translation was not changed and you only wanted to make sure all translations are compiled)

Interesting - good shout out - let me look at handling that 🤔
I'll pull in those translations, thanks!

Going to merge this as-is now. If anything needs tweaking or there are more translations, another PR is just a few clicks away :)

Right, I think that's all sorted now.
@ElLorans @LeXofLeviafan - would you mind having a glance over this and seeing if you're happy with the updates? It shouldn't affect any translation strings other than:
* Removing old strings that are no longer present

* Adding new msgids with empty msgstrs, ready for translations to be put in

* Adding equivalent msgstrs to the `en` file.

I haven't used Babel before to do any translation work, so would appreciate someone else looking over this unless I've done something unexpected/undesirable.

I confirm that now the strings are translated and everything looks right to me!


Out of bounds memory write in esp_nn_aligned_s8_pad_with_value (causing CORRUPT HEAP: Bad tail)

Hello, @vikramdattu !
We have successfully tested object detection and sensor time series models with ESP32-S3. However, when testing keyword spotting model, we ran into
CORRUPT HEAP: Bad tail
issue.
I started investigating, think that it might be the alignment problem again, described in https://github.com/espressif/esp-nn/issues/7, however it seems this is something different this time.
I peppered my inference and ESP NN code with heap_caps_check_integrity_all(true); and managed to find the source of the problem (I think):
The tail corruption is happening here
https://github.com/espressif/esp-nn/blob/12129cf04b09af0023127ca7551dc1a363344211/src/common/common_functions.h#L188
which is led from
https://github.com/espressif/esp-nn/blob/12129cf04b09af0023127ca7551dc1a363344211/src/convolution/esp_nn_conv_esp32s3.c#L257
The padding function is only used for ESP32-S3. I think the reason it only shows for keyword spotting models for us is that the dimensions are different and therefore there is a need for padding.
From what I understand the code here resizes scratch buffer? I don't think this is valid, even if it is within the arena (and for sure not valid when scratch buffer is allocated outside of the arena, like in our case). If the scratch buffer is inside of the arena, resizing it like that would likely cause corruption to neighboring buffers/tensors. If it is outside of arena, then there is no way to tell what will it write on.
Let me know if my reasoning here is correct and how can we solve this issue! Enclosed find the exact project to replicate the issue.
example-standalone-inferencing-espressif-esp32.zip

As an afterthought, it's likely this can be fixed by calculating correct size of scratch buffer beforehand.

Hello @AIWintermuteAI  thanks for reporting yet another issue.
Please find the patch attached and let me know if this fixes the issue.
memory_write_overflow.patch

Hi, @vikramdattu !
Thank you for the fast reply and help in resolution of the problem. I tested the patch and can confirm it resolves the problem. As soon you make the changes in the main branch, we'll pull them in (ESP32-S3 support was added to Edge Impulse yesterday).

Hi @AIWintermuteAI the fix has been pushed to the esp-nn repo.


Error deleting search index

In /Lucene/Admin/Index, the delete button calls /Lucene/Admin/Delete/Search.
But the Delete method in the AdminController expects (AdminEditViewModel model)
Should we replace by (string id) to retrieve the indexName to delete?

I think so.


GEOMESA-734 Change RuntimeException to IOException

Updated an exception to work within the GeoTools API.

merged as e241c95df99222707f6d3ec2b9f100806e2d09d0


UIUX design for custom roles in policies

Requirements
As a user of the Guardian, I would like to see the changes on the frontend that are related to issue #51.
Pre-requisite
Front End UX?
Definition of Done
Front End UX QA Demo
Acceptance Criteria:
Unsure?? @anvabr ?

At @mendelenvisionblockchain adding this to the Sprint 3 and assigning to you.

We will issue 52 out Removing Policy Engine to the next Sprint or two.

As per todays Grooming Session, we will  wait to do this ticket until we are doing the UX for the WHOLE Policy Engine.


COMPILING.md: make formatting corrections

added a \ to npcapsdk file location
explictly said mingw32 as it will not work with any other of the msys apps. creates corrupted file prone to memory leaks if you use mingw64

same as pr #33 just going to the correct branch


Git tool renders cyrillic commit messages in a wrong way

Commit example: https://github.com/nergal-perm/tg-core/commit/a7855f7688378310d1ada166d5569915dc58e57f.
It is shown in git tool like this:

The author name uses the same symbol set, but it's displayed correctly.
GT Version: v0.8.1334
OS Version: Ubuntu 21.10

Do you have experience using Iceberg? If so, could you check how it renders the comment?

Sure thing, I'll check it out. By the way, I've discovered that Author is represented by an instance of WideString but Comment is a ByteString. Could it be important?

 I’ll try and see whether we can do better.
Are those commits created inside GT, Pharo, or somewhere else?

Huh, funny. Thanks for investigating, I’ll take a look!

Is this an issue we can deal with it?

We should be able to deal with it; the problem is that vanilla Iceberg doesn’t handle encodings gracefully, so that might be a problem. I haven’t taken the time to properly investigate how to solve this yet, however.

Today (including with the new git CLI implementation) we can commit non-ASCII comments and view them, as well as use author names that contain non-ASCII characters. Encoding is UTF-8.


Add convert case feature back

Re discussion #215, add the feature back to convert method and parameter names to snake case.

The user can do their own modifications to the request before calling dispatch.
This was the code used in version 4.
def convert_camel_case_string(name: str) -> str:
    """Convert camel case string to snake case"""
    string = re.sub("(.)([A-Z][a-z]+)", r"\1_\2", name)
    return re.sub("([a-z0-9])([A-Z])", r"\1_\2", string).lower()

def convert_camel_case_keys(original_dict: Dict[str, Any]) -> Dict[str, Any]:
    """Converts all keys of a dict from camel case to snake case, recursively"""
    new_dict = dict()
    for key, val in original_dict.items():
        if isinstance(val, dict):
            # Recurse
            new_dict[convert_camel_case_string(key)] = convert_camel_case_keys(val)
        else:
            new_dict[convert_camel_case_string(key)] = val
    return new_dict


NuGet package is incompatible with latest version of Serilog

Unable to resolve dependencies. 'Serilog 2.4.0' is not compatible with 'Serilog.Sinks.Glimpse 1.5.14 constraint: Serilog (>= 1.4.204 && < 2.0.0)'.

Thanks for the note. The sink needs an update to current Serilog and Glimpse versions - a PR would be welcome if anyone has an opportunity to submit one.

Would PR #6 solve this?


Add DTLS session resumption tests and fix a bug in SSL ticket key update

Summary: This PR fixes #1968 and #1969.

retest

retest

CI passes. Disregard the 'mbed-ci-generic' result.

Re-applying the "needs backports" label as the backports haven't received enough reviews yet (per the label description: "backports are missing or pending review and approval"), so we can see at a glance what is currently blocking this PR from being merged.

I've provided the second review on the backport. @mpg I think this is now ready for merge.


Angular SPA SSR Template bug with Angular 6

I converted this template https://docs.microsoft.com/en-us/aspnet/core/spa/angular?view=aspnetcore-2.1&tabs=visual-studio to Angular 6. Everything worked fine initially, but then an update to @angular-devkit/build-angular npm package (from version 0.5.5 to 0.5.6) broke it.
The reason I'm not submitting this issue to the Angular repo is I think it's actually a bug in https://github.com/aspnet/JavaScriptServices/blob/dev/src/Microsoft.AspNetCore.SpaServices.Extensions/Util/EventedStreamReader.cs that wasn't noticed until now. Basically the AngularCliBuilder.cs waits for "ng build:ssr" to finish by watching the console output & looking for "Date" in the output. I haven't done a "diff" of the actual ASCII values in the console output between the two angular versions, but visually they look almost identical, except the new version has an extra newline, otherwise they look the same.
This is the error I'm getting:

Here's the visual difference between the console outputs:
working:

broken:

I need to research more, but I believe the issue is with this function in EventedStreamReader.cs:
        private async Task Run()
        {
            var buf = new char[8 * 1024];
            while (true)
            {
                var chunkLength = await _streamReader.ReadAsync(buf, 0, buf.Length);
                if (chunkLength == 0)
                {
                    OnClosed();
                    break;
                }

                OnChunk(new ArraySegment<char>(buf, 0, chunkLength));

                var lineBreakPos = Array.IndexOf(buf, '\n', 0, chunkLength);
                if (lineBreakPos < 0)
                {
                    _linesBuffer.Append(buf, 0, chunkLength);
                }
                else
                {
                    _linesBuffer.Append(buf, 0, lineBreakPos + 1);
                    OnCompleteLine(_linesBuffer.ToString());
                    _linesBuffer.Clear();
                    _linesBuffer.Append(buf, lineBreakPos + 1, chunkLength - (lineBreakPos + 1));
                }
            }
        }

I think it should call OnCompleteLine(_linesBuffer.ToString()); a final time at the end of the function if there's anything still in the buffer.
I've created a repo for reproducing this at https://github.com/speige/Angular6DotNetCoreSpaTemplateBug
The "master" branch is the original angular 5 template. The "Angular6Working" branch is an straight upgrade to v6 with the @angular-devkit/build-angular npm package on 0.5.5. The "Angular6broken" branch is identical except with @angular-devkit/build-angular on 0.5.6
The "Angular6Working" branch might be useful in order to upgrade the official template to Angular 6 (still in RC, but should be oficially released any day now).
On a side note, for Angular6 upgrade, a change should be done in
https://github.com/aspnet/JavaScriptServices/blob/dev/src/Microsoft.AspNetCore.SpaServices.Extensions/AngularCli/AngularCliBuilder.cs
            var npmScriptRunner = new NpmScriptRunner(
                sourcePath,
                _npmScriptName,
                "--watch",
                null);

The "--watch" is no longer a valid command line argument in Angular 6 CLI. I believe if you do "ng serve" it automatically does --watch, if you do "ng build" it doesn't.
I worked around this issue by changing the "build:ssr" command in my package.json from
    "build:ssr": "ng build --configuration=production --project=ssr"

to
    "build:ssr_ignore_dash_dash_watch_from_AngularCliBuilderDotCS": "ng build --configuration=production --project=ssr ",
    "build:ssr": "npm run build:ssr_ignore_dash_dash_watch_from_AngularCliBuilderDotCS"

which basically causes the extra command line argument to be discarded before "ng build" is called. Otherwise Angular throws an error about an invalid command line argument.
I'll test my theory with EventedStreamReader.cs & assuming it fixes this issue I'll submit a pull request.

@speige I tried to follow your instructions but unable to get the project building. have you tried with final angular 6?

Ok I downloaded/cloned the repo https://github.com/speige/Angular6DotNetCoreSpaTemplateBug/tree/angular6Working and after restoring npm packages tried to run the project, it didn't work.
Then I tried running below commands through command prompt in ClientApp and everything worked perfectly.
npm install -g @angular/cli

npm install @angular/cli

ng update @angular/cli

ng update @angular/core

Angular 6 support is being worked on at https://github.com/aspnet/templating/pull/515. This isn't in the 2.1 release but hopefully should be included in whatever version ships after (maybe it will be called 2.2, though I don't know if the version number has been decided yet).

Does someone know when will it be published?

@SteveSandersonMS, did you look at my pull request before closing this?
I understand that your normal upgrade process is being followed in https://github.com/aspnet/templating/pull/515 which is to take the latest angular repo & work backwards to make the .net version. However, I believe there's a bug in https://github.com/aspnet/JavaScriptServices/blob/dev/src/Microsoft.AspNetCore.SpaServices.Extensions/Util/EventedStreamReader.cs which needs to be fixed before SSR for angular 6 will be successful.

Yes, your PR is still active. I expect you are correct that this needs to be fixed. Thanks for submitting it!

It would be really great if we could have a fix anytime soon.

Any new ?

The fix for this particular bug is in pull request #1620 I believe they closed this issue as a duplicate of other Angular6+VisualStudioSPATemplate issues.
I've switched to my own modified version of the aspnet spa template because I couldn't wait for official support. I'm not sure the current state of the latest aspnet template or if it still has bugs or not. You'll have to research that on your own. Good luck :)

Thanks, I subscribed to the pull request. If I'm not mistaken, this only affects development, not production. If you are using one of the templates, you should be able to disable prerendering in dev like this:
options.BootModuleBuilder = null;
// Don't use the AngularCliBuilder
// options.BootModuleBuilder = env.IsDevelopment()
//     ? new AngularCliBuilder(npmScript: "build:ssr")
//     : null;

That's probably true, I haven't tried it. I just compiled Microsoft.AspNetCore.SpaServices.Extensions.dll myself off my forked code & I'm using that dll instead of the one from NuGet. I'll switch back to the official version once the PR is accepted.


Show informative error when account is not found

It's very common that querying wrong network (e.g. testnet is default but want mainnet).
Currently error is "[-32000] Server error: ..." with huge stack trace.
Instead should show error saying:
Account <account_id> is not found in <network_id> network.

Also if prefix on the account is .near show: Use export NEAR_ENV=mainnet to use MainNet accounts, if .testnet or .betanet same.

Sounds good, I think we were holding out hope that errors might return typed errors or special error numbers, but we can definitely look at the error message and use regex/string parsing to determine what to tell the user.

hey @mikedotexe can you pls check my pr https://github.com/near/near-cli/pull/532
on this issue

@ilblackdragon @mikedotexe Looks like this issue can be closed.
We have merged code from @rozum-dev + some aditional changes from me. Final version can be foud in this PR: https://github.com/near/near-cli/pull/602/files


remove gitbase binary

Signed-off-by: kuba--<EMAIL_ADDRESS>

You might want to add it to .gitignore.