developerkit UART 0 OVER RUN

在developerkit板子上，使用rel_2.1.0版本编译并下载otaapp程序后，在命令行输入OTA测试命令：
[
OTA_APP "a154F8BTLOS" "test01" "ojjE88iqT2dilHgamHv0Zu4TKwCXcU38" "TVekLkQ7Es0frNNY"
]
会概率性出现“FATAL UART 0 OVER RUN ”错误，错误发生概率在80%以上
-------------------LOG INFO---------------
[2019-03-23 10:30:47.416]# RECV ASCII>
boot up ...
aos Bank2 bootup!
[2019-03-23 10:30:47.507]# RECV ASCII>
uart 1 enter uart_receive_start_dma instance 0x40004800[000050] sensor:  drv_acc_st_lsm6dsl_init successfully
[000060] sensor:  drv_gyro_st_lsm6dsl_init gyro do not need reset
[000070] sensor:  drv_gyro_st_lsm6dsl_init successfully
[000080] sensor:  drv_als_liteon_ltr553_init successfully
[000090] sensor:  drv_ps_liteon_ltr553_init successfully
[000100] sensor:  drv_baro_bosch_bmp280_init successfully
[000120] sensor:  drv_mag_memsic_mmc3680kj_init successfu
[2019-03-23 10:30:47.582]# RECV ASCII>
lly
[000130] sensor:  drv_humi_sensirion_shtc1_init successfully
[000130] sensor:  drv_temp_sensirion_shtc1_init successfully
         Welcome to AliOS Things

[2019-03-23 10:30:49.642]# RECV ASCII>
[002150] wifi bk7231 FW version is: FWVER:APP-1.1.6-20180928.1547
OK
[002170] Will connect via at cmd: AT+WJAP=CMCC-whxf,h9djj6ef
[002200] AT command AT+WJAP=CMCC-whxf,h9d986ef succeed, rsp:
OK
[2019-03-23 10:30:55.137]# RECV ASCII>
[007630] AT command AT+WMAC? succeed, rsp:
+WMAC:e47deb1a5f86
OK
[007640] mac result: e47deb1a5f86
[007640] Got ip : <IP_ADDRESS>, gw : <IP_ADDRESS>, mask : <IP_ADDRESS>
[007670] AT command AT+WMAC? succeed, rsp:
+WMAC:e47deb1a5f86
OK
[007680] mac result: e47deb1a5f86
[007680] Hello, WiFi GOT_IP event! at board/developerkit/aos/board_cli.c 210
[007690] Hello, WiFi GOT_IP event! at board/developerkit/aos/board_cli.c 210
[2019-03-23 10:30:58.728]# SEND ASCII>
OTA_APP "a154F8BTLOS" "test01" "ojjE88iqT2dilHgamHv0Zu4TKwCXcU38" "TVekLkQ7Es0frNNY"
[2019-03-23 10:30:58.803]# RECV ASCII>
OTA_APP "a154F8BTLOS" "test01" "ojjE88iq### FATAL UART 0 OVER RUN !!!
### FATAL UART 0 OVER RUN !!!
lHgamHv0Zu4TKwCXcU38" "TVekLkQ7Es0frNNY"

@kentzh Please update Wi-Fi firmware to 201812 instead of APP-1.1.6-20180928.1547(from above log), the guide is available on github wiki.

@kentzh please show whole logs to us. please input cli again if log shows '"# ## FATAL UART OVER RUN !!!". It is normal case for '# ## FATAL UART OVER RUN !!!', but abnormal for 80% failure.

After WIFI FW updated, this issue disappeared. Thanks!


Implement FieldUniqueness and ArgumentUniqueness cops

Introduces two new cops to prevent duplicate field and argument definitions, and fixes #49, with RSpec coverage.

New FieldUniqueness cop ensures no duplicate field definitions are within a type
New ArgumentUniqueness cop ensures no duplicate argument definitions within the same level- either top-level for mutations etc, or within a field definition block
Fix infinite loop errors within existing OrderedFields and OrderedArguments cops, which previously crashed when encountering a duplicate definition

LGTM! Thank you!


Question: Download problems

GUI 0.4
OS: Windows 7
Problem: Videos from www.dmax.de only in 1080p
I want the files in 540p
If I leave the default setting, the videos can be loaded in 1080p.
But if I set GUI -> Options -> Extra "--all-formats" I get all other available resolutions loaded, but without sound.
But as soon as I type -f 'bestvideo[height<=540]+bestaudio/best[height<=540]' in gui-> Extra i get an error and the log shows: ERROR: requested format not available.
Can you help me?
Its a problem with the homepage or do i something wrong?
thx

If you want 1080p videos, you need to install ffmpeg.
youtube-dl can't cope with 1080p without ffmpeg.

i have ffmpeg installed, does the software recognize it automatically or do i have to make some changes in options?

youtube-dl, not youtube-dl-gui, checks whether ffmpeg exists or not. There are no options to change.


fix .env path in js sample 03.welcome=users

Fixes #757
Proposed Changes
Fix right path to bot file in .env

@mehd-io Thanks for this PR! If you've got a moment, could you rebase against master, so i can merge it in? We had to make some changes the other day to remove a key from our git history, and one impact of that is that all in-flight PRs need to be rebased before we can merge them.
Thanks!

@cleemullins this is already fixed in #808, it should be closed along with the issue.

This PR is taken care of by PR #916


PutRelativeFile.ConflictingHeaders test deletes the original wopitest file, causing subsequent tests to fail

A fix is in progress. As a workaround, you can run the test groups manually and skip the PutRelativeFile test group.

This issue is now resolved.


Add base documentation

Closes #9

Codecov Report

Merging #16 (0ce352b) into dev (9bb4626) will not change coverage.
The diff coverage is 96.42%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##              dev      #16   +/-   ##
=======================================
  Coverage   97.30%   97.30%
=======================================
  Files           9        9
  Lines         260      260
=======================================
  Hits          253      253
  Misses          7        7

Impacted Files
Coverage Δ

src/plotting.jl
91.11% <ø> (ø)

src/utils.jl
90.00% <50.00%> (ø)

src/baselaplace.jl
100.00% <100.00%> (ø)

src/curvature/functions.jl
100.00% <100.00%> (ø)

src/laplace.jl
97.77% <100.00%> (ø)


Name all versions the same way

I feel stressed by the different version schemas you are using. Every release changes the version string. I have no idea why this is so inconsistent.
Could you build with "one" version schema that is always the same, please?
Not:

jdk8u172-b11
<IP_ADDRESS> (good)
<IP_ADDRESS>_openj9-0.9.0
11_28

But simply a semantic version like

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Creating a setup builder with such a version string chaos is not funny and now every ZIP has additional "random" version numbers.

Hi @alexhass - unfortunately we have to follow the version strings that upstream OpenJDK and OpenJ9 projects provide.  This ensures that tooling in the ecosystem 'just works'.


Added missing missing module logzero in docker

What current issue(s) from Github does this address?
What problem does this PR solve?
It enables docker to work correctly.
How did you solve this problem?
I tried to use the docker but it complained that logzero module was missing.
How did you make sure your solution works?
I tested in my own machine. Since it is an independent docker image, it should work anywhere else.
Are there any special changes in the code that we should be aware of?
No.
Is there anything else we should know?
Perhaps this should be adoped in the official python package, but I don't know where it is, and how to inform that in a better place :)

How did you use the docker image before for it to give an error about logzero? Afaik neo-boa doesn't use logzero, only neo-python.

This is really strange Chris, I just built the proposed Dockerfile (without logzero) and tried to run the compile script (with docker run -it -v ...). It didn't work, no feedback. So I bashed inside the image (with volumes mounted) and tried to execute the script, then it complained that logzero was missing. I installed logzero inside the image (with the same pip install that now failed on travis....) and it worked.
I "corrected" the Dockerfile with the logzero install, and then managed to use it with your docker run command.
If it is meant to work without logzero, this is really strange. It is so strange that this is the production Dockerfile we use at neocompiler.io (https://github.com/NeoResearch/neocompiler.io/tree/master/docker-neo-boa)
I'll try to understand it better, then I come back :)

Ok, that's a much better fix of course :)
It's better to destroy this PR if #48 is accepted.
Thanks!


inference error related to position in source files

Description
In my source code, I have 3 folders sorted alphabetically. The first and third folders contain the exact same code. The second folder exports a typed function with generics. Strangely, only the third folder fails a flow check, folder one passes.

Flow version: 0.98.1
Expected behavior

Source code order should not affect flow check.
Flow should infer the function type with only one generic.

Actual behavior

Duplicate code alphabetically after type definition fails flow check but if before, it passes.
Flow infers the function type with two generics.
Link to Try-Flow or Github repo: https://github.com/chasestarr/flow-reproduce-infer-error-position-in-source

Notes

If the Options type is in the same file as fn calls, flow infers correctly.
If the code is within the same folder, flow infers correctly.

Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/c-after/index.js:6:24

string [1] is incompatible with number [2] in type argument N.

     src/c-after/index.js
     3│ import fn from "../b-function/index.js";
     4│ import type {Options} from "./types.js";
     5│
 [1] 6│ export const Comp = fn<Options>("hello", options => 2);
     7│

     src/b-function/index.js
 [2] 6│   <N: number, T>(N, (T) => number): any,

Found 1 error

The issue no longer repros on newer versions of Flow, and it is likely fixed after types-first.


Add doubly linked list implementation

Hi. I have attached the class with Doubly Linked list implementation made by the pattern of the LinkedList class.
I would suggest first - to follow c# naming conventions: PascalCase for classes and second - implement data structures following .Net pattern, i.e. implementing IEnumerable and ICollection interfaces. It would make implemented class closer to native environment,
Cheers


Setup CI/CD pipeline

Title of ticket: Setup CI/CD pipeline
Dependencies
None - this is foundational
DOD

[x] Setup jenkyns pipeline - with github actions
[x] Automated build pipeline to dev and test (Adam: removed test, see comment below)
[x] All devs have access to the system
[x] OpenShift authentication configured in GitHub repo secrets
[x] Service account created in GitHub for logging in as GH actions.
[x] GA tests or build checks run on PRs.
[x] GA deploys build to "dev" OpenShift when work is checked into "dev" GitHub branch.

@acoard-aot where are things at with this?

I met with Sumesh and Saravan to go over the GitHub Actions (GA) pipeline
they used.
Here's the link:
https://github.com/bcgov/sbc-auth/tree/main/.github/workflows
I'm currently reviewing this.  I was going to start the task in ernest, but
it turns out I need to help deploy Patroni to the test env first
On Wed, Jun 30, 2021 at 3:26 PM molly pilchar @.***>
wrote:

@acoard-aot https://github.com/acoard-aot where are things at with this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/foi-flow/issues/109#issuecomment-871767011, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTSLJUN2G57PVKLEONRNDTTVOKYXANCNFSM46DFWT5Q
.

After beginning implementing, I suggest adding the following definitions of done.  It's fine if we don't, these can still serve as documentation of tasks completed.

OpenShift authentication configured in GitHub repo secrets
Service account created in GitHub for logging in as GH actions.
GA tests or build checks run on PRs.

Good news.  The "CI" part is done, and the "CD" part is completed for pushing a build to dev.  I've created the PR and will move this ticket to "Review." I will share this with the team more fully on Monday, maybe setup a meeting after discussing with team.
A minor issue I'll bring up with the 2nd definition of done item for deploying to "dev and test".  As discussed with Loren, we're drawing our implementation from sbc-auth.  However, their solution from what I can tell only deploys to dev.  After discussing with dev team, we think this is best approach for now (we can always revisit).  As such, I've crossed off "test" in the DoD description in the card.  Please let me know if this is an issue.  My thought is that for now test needs to be stable if we are using it for our demos.
This task was about creating a minimal CI/CD pipeline, and is complete, but there is a lot more to gain by continuing to work on it.  Many of these were in the sbc-auth project, but we just haven't had time to setup, and we will absolutely want to do (next sprint?). This includes:

[CI] Running Abin's unit tests for request management api
[CI] Running postgres migrations from request management api
[CD] Watching rollout status after build is complete
[CD] Sending push notifications of success or failure to Teams (? or RocketChat if teams doesn't support).

Additionally, we may wish to look at including FormsFlowAI in github actions, particularly the web component.

@acoard-aot I know there were some aspects of this body of work you wanted to make note of as we go forward. Let me know if you have any thoughts on how to do this (add to backlog or something less formal?) before this gets closed


Suggest to replace the PDF files with perma-links to documentation.

Currently the DFP contains PDF files. We discussed to replace the PDF files with perma-links to documentation.

This was done for the STM32U5xx_DFP only as a reference.
Would these pack-examples be "deleted" once we have the separate repositories?


JSON schema for data ingest for disease-model associations

See for example:
http://api.monarchinitiative.org/api/#!/bioentity/get_disease_model_associations
Click on 'Model' and then scroll to the Association object.


docs: update helm flag

In helm v3 the --name doesn't exist

Fixes #XXXX.

Description

Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@asdf2014 Could you check if the change is OK?


How to add QueryParams in route using .post?

In case I want to use .post method with a custom query param route such as: "/users/create?anonymous=true"

@imfree-jdcastro, you have to create a custom encoder. https://github.com/Moya/Moya/issues/1059 might be helped :)


Add ESLint and Prettier

Description of the change
Adds ESLint and Prettier support to backend repository.
A fair amount of ESLint ignore lines have been added to meet ESLint rules for now, as changing them may possibly involve a fair amount of code restructuring. A list of ignore lines that have been added can be seen in the "Changes made" section below.
This closes CommunityFocus/CommunityFocus#27
Fixes # (issue that will be closed by this PR)

CommunityFocus/CommunityFocus#27

Type of change

[x] New feature (non-breaking change which adds functionality)

Changes made

Add ESLint dependencies and config files.
Add Prettier dependencies and config files.
Reformat all code to pass ESLint and Prettier rules.
Add ESLint ignore rules to pass linting check until issues can be addressed:

File
Line
Rules Ignored
Description

server.ts
99
no-shadow
socket join event function parameter roomName shadows the roomName const declared outside of this event on line 80

server.ts
158
no-shadow
socket startCountdown event function parameter roomName shadows the roomName const declared outside of this event on line 80

server.ts
167
no-shadow
socket timerRequest event function parameter roomName shadows the roomName const declared outside of this event on line 80

server.ts
173
@typescript-eslint/no-empty-function
socket pauseCountdown event uses an empty anonymous function (empty functions are not allowed).

server.ts
177
@typescript-eslint/no-empty-function
socket unpauseCountdown event uses an empty anonymous function (empty functions are not allowed).

types/socket/types.ts
34
@typescript-eslint/no-empty-interface
type InterServerEvents is an empty interface (empty interfaces are not allowed).

types/socket/types.ts
40
@typescript-eslint/no-empty-interface
type SocketData is an empty interface (empty interfaces are not allowed).

apiController.ts
8
import/prefer-default-export
ESLint wants all files that have one function to use a default export. Since apiController may have more handler functions in the future, this rule should be disabled (rather than changing to default and then back from default once we add more controller functions).

destroyTimer.test.ts
43
@typescript-eslint/no-empty-function
mockTimer uses an empty anonymous function, but is currently marked as TODO and can stay empty for now.

destroyTimer.ts
18
no-param-reassign
line 19 deletes the key from the timerStore (delete timerStore[roomName]). ESLint rules discourage reassigning function parameters including delete.

startTimer.ts
43
no-param-reassignno-multi-assign
line 44 both reassigns a function parameter and contains multiple assignments in one line.

startTimer.ts
47
no-param-reassign
line 48 reassigns a function parameter.

startTimer.ts
51
no-param-reassign
line 52 reassigns a function parameter.

startTimer.ts
55
no-param-reassign
line 56 reassigns a function parameter.

timerRequest.test.ts
11
@typescript-eslint/no-empty-function
jest.spyOn mocks console.error with an empty anonymous function.

timerRequest.ts
51
no-use-before-define
emitTimerResponse and updateChecker both reference each other, meaning one of them attempts to access the other before it's assigned.

Screenshots/ Screenshare
N/A
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings or errors
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have used the appropriate file structure to keep my code concise and readable. I have created a new file for helper functions where possible

@riddlew Can you please merge this into the CommunityFocus:build/add-eslint-prettier branch after running npm run build:package


Allow configuring hover to dismiss

This feature exist already. You can put the extended timeout to 0 to make it more immediate.

Understood you wrong. If you want to disable the hover, you can put the timeout and extended timeout to 0.

neither of those accomplish the task I was looking for. It's fine though, I'm not using this anymore anyways.


NEBULA-794: Add diagnosis to check if websocket endpoint exists

The script fails when checking any ws: or wss: endpoints. Added diagnosis for when a websocket endpoint doesn't exist and when websocket URL doesn't exist.
URL doesn't exist
node index.js --endpoint=ws://localhost:4001/graphql
Diagnosing WS://localhost:4001/graphql
⚠  Connect Failed: Error: connect ECONNREFUSED <IP_ADDRESS>:4001
⚠  Connection refused (ECONNREFUSED)
Is the address correct?
Is the server running?

Endpoint doesn't exist
node index.js --endpoint=ws://localhost:4000/bad
Diagnosing ws://localhost:4000/bad
⚠  Connect Failed: Error: Server responded with a non-101 status: 400 Bad Request
Response Headers Follow:
connection: close
content-type: text/html
content-length: 11

⚠  Could not resolve (ENOTFOUND)
Is the address correct?
Is the server running?

Incorrect protocol
node index.js --endpoint=wss://localhost:4000/graphql
Diagnosing wss://localhost:4000/graphql
⚠  Connect Failed: Error: write EPROTO<PHONE_NUMBER>:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:332:

⚠  Protocol wrong type for socket (EPROTO)
Is the address correct?
Is the server running?

@William010x: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


gcloud container builds is deprecated

Need to update Jenkinsfile for Cloud SDK updates.
Current gcloud CLI command to Build and push an image with Container Builder :
sh "PYTHONUNBUFFERED=1 gcloud container builds submit -t ${imageTag} .
Need to update as:
sh "PYTHONUNBUFFERED=1 gcloud builds submit -t ${imageTag} .

Same modifications introduced in this PR.


Switch benchmark agent to newspec runner for testing

Description
Switch benchmark agent to newspec runner for testing
Issues Resolved
We want to switch to new spec of Benchmarkhost from 2 executors m52xlarge to 4 executors c54xlarge.
This is a trial run to check if that is good enough.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Waiting for:

https://github.com/opensearch-project/opensearch-ci/pull/339


webRTC SDK reConnection question

After the viewer and the master successfully establish a connection, the network status of one side is poor (Even disconnect the network for a short time), does the webRTC SDK has a reconnection mechanism, and how long can it be reconnected?
Environment:
Master in browser and use sdk-js, viewer use sdk-c.

@jiangjunzailushang yes, since our JS SDK only provides signaling, you can use ice restart. However, you need to be careful with browser compatibility, Chromium based browsers support this well but I'm not sure with other browsers or even old chromium based browsers.
For the viewer, there shouldn't be an issue, we have recently added this, https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/pull/457.
Also, there was a similar issue about this. You might want to also check this out, https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-js/issues/65.

Due to staleness and the question is related to browser APIs. I'll close this issue. Please feel free to reopen if you need further assistance.


Desired usage API v0.1 in README

I copied the usage readme from sqs-utils and drafted an usage example. send-message and send-fifo-message could have the same API. I tweaked handle-queue by:

removed visibility-timeout as I find that it's better to set that on the queue configuration and not through the client.
adding a s3-bucket parameter to queue-config to point to the s3 bucket that this queue should use as message storage. The idea is that we'll create and setup the S3 bucket separately once in our devops and then just pass the URL/ARN here. I don't want us to be creating infrastructure assets in client code.

Regarding (1)
Yes, that makes perfect sense.
Regarding (2)
I see your point. If you think that creating the bucket by hand wouldn't lead to too much administrative overhead later (cannot judge real world situation here) then this should be fine.

We use https://www.terraform.io to administer our infrastructure. So there's already another repo to manage that. I just want to keep things in one place.
Question for you @daisybytes on S3 bucket for the queues. Does it need to be one S3 bucket tied to each queue, or can multiple queues share the same S3 bucket?

@daisybytes if this PR looks good to you, could you approve and merge it please?

As of now I think the sqs-endpoint argument will need to remain, but I'll have a deeper look at that again

that's probably true. we probably left it there for some reason. I just don't remember.

We use https://www.terraform.io to administer our infrastructure. So there's already another repo to manage that. I just want to keep things in one place.
Question for you @daisybytes on S3 bucket for the queues. Does it need to be one S3 bucket tied to each queue, or can multiple queues share the same S3 bucket?

The working Java example ties the S3 bucket to the client configuration via which the queue later are created. The example only creates a single queue but I wasn't able to find any documented restrictions here. Must investigate/experiment but my current guess is that multiple queues can share the bucket.


Boolean operators in handlebars?

Is it possible to have boolean operators in handlebars? So for example, I want to respond when two slots have the 'yes' value in them. I only know how to check one variable in the response...
{{#eq slots.variable.value 'yes'}}Response{{/eq}}

In general, can you point me to where I can find out more about the handlebar syntax and its scope? Is it a subset of some existing language?
Thank you

the syntax you've written looks correct to me.
You can find more handlebars info at their own website: https://handlebarsjs.com
We also use an extra library for more functionality: https://github.com/helpers/handlebars-helpers
let us know if you're still having trouble with it.

Thanks! That extra library is great!


Write Frame OBUs instead of a pair of Frame Header and Tile Group OBUs

For non show_existing_frame frames, and seeing rav1e currently doens't offer
the option to write more than one Tile Group per frame, it's more
efficient to use Frame OBUs instead of a pair of Frame Header and Tile Group
OBUs.
This saves a few bytes per frame by removing the overhead from the superfluous
OBU header.

Coverage increased (+0.002%) to 80.795% when pulling 8c0255b464f9423769d25440a279263651b53770 on jamrial:frame_obu into edfc865445b18ceb86f176c51f063f67392ffae2 on xiph:master.

Could someone take a look at this?

AWCY run results: http://<IP_ADDRESS>/?job=master%402020-11-18T21%3A18%3A35.622Z&job=obu-optimization%402020-11-18T21%3A29%3A25.667Z

I was not expecting changes in encoding metrics. This simply saves a few overhead header bytes per frame.

I can't say why the metrics might have changed, and the analyzer is not hooked up to this AWCY instance, but I started a new run prior to the last commit on master which is not merged into your branch, in case it caused unexpected coding changes: http://<IP_ADDRESS>/?job=master-minus-one%402020-11-18T22%3A27%3A40.206Z

Just to follow up on the initial results, it turned out the "master" reference did not point to the origin repository as expected.


work around invalid content-range from azurite

When requesting zero byte files from azurite, it can generate invalid content-range headers.
See #1093 for an example.

There is an open work item that discusses this inconsistency.  https://github.com/Azure/Azurite/issues/1682


MqttClientSessionsManager should use lock

log by ClientConnectedHandler ClientDisconnectedHandler ClientSubscribedTopicHandler：
Client connected 4194328.
Client connected 4194328.
Client connected 4194328.
Client disconnected 4194328.
Client disconnected 4194328.
Client Subscribed 4194328 /devices/update//4194328.
info: Microsoft.Hosting.Lifetime[0]
the client can not receive subscribed topic
client is a rtos device，sometimes  subscribe  no effect when connected

can you create a test / proof of what is wrong? MqttClientSessionsManager is using concurrentdictionary so it should be good.

wait for a monent
i test
first  the client device is reconnecting
second start the server
at this monent, the device connected ,then subscribe the topic,most probable,subscibed not effect.
i will debug the  code.

mqttnet ver:3.0.8
bug Scene:
first：a rtos  device connecting to server  by a loop
second： start the  asp .net core 3.1 server
result：the client can not receive subscribed topic
i solve it by a lock HashSet，code:
public class Startup
{
private HashSet clientIds = new HashSet();
...
public void ConfigureServices(IServiceCollection services)
...
var mqttServerOptions = new MqttServerOptionsBuilder().WithConnectionValidator(t =>
{
bool res = deviceManager.OnLogin(t.Username, t.Password, t.ClientId);
if (res)
{
Console.WriteLine($"Authorized: ClientId: {t.ClientId}");
lock (clientIds)
{
if (clientIds.Contains(t.ClientId))
{
t.ReasonCode = MQTTnet.Protocol.MqttConnectReasonCode.ServerUnavailable; //.NotAuthorized;
}
else
{
clientIds.Add(t.ClientId);
t.ReasonCode = MQTTnet.Protocol.MqttConnectReasonCode.Success; //.NotAuthorized;
}
}
            }
            else
            {
                Console.WriteLine($"NotAuthorized: userName: {t.Username} password:{t.Password}");
                t.ReasonCode = MQTTnet.Protocol.MqttConnectReasonCode.NotAuthorized;
            }
        });

...
public void Configure(IApplicationBuilder app, IWebHostEnvironment env,
...
server.ClientDisconnectedHandler = new MqttServerClientDisconnectedHandlerDelegate(e =>
{
Console.WriteLine("Client disconnected {0}.", e.ClientId);
lock (clientIds)
{
clientIds.Remove(e.ClientId);
}
});

add the hashset,the log is:
info: Microsoft.Hosting.Lifetime[0]       Now listening on: http://[::]:35001 info: Microsoft.Hosting.Lifetime[0]       Now listening on: http://[::]:35000 Authorized: ClientId: 3211286 Authorized: ClientId: 3211286 Client connected 3211286. Authorized: ClientId: 3211286 Client disconnected 3211286. Client disconnected 3211286. Client disconnected 3211286. Authorized: ClientId: 3211286 Client connected 3211286. Client Subscribed 3211286 devices/update/_tcp.local./3211286.

i still dont get what is the issue here. why is the client establishing more than one connection at a time?

i still dont get what is the issue here. why is the client establishing more than one connection at a time?

the rtos device connect to server, failed  or initiative to disconnect by some error ,
and use the same clientId reconnect

@chkr1011  hi christian i created a minimal repro for this issue in form of a unittest.
my expectation is that 9 out of 10 connections are closed and the last one is working correctly.
        [TestMethod]
        public async Task Manage_Session_MaxParallel()
        {
            using (var testEnvironment = new TestEnvironment(TestContext))
            {
                testEnvironment.IgnoreClientLogErrors = true;
                var serverOptions = new MqttServerOptionsBuilder();
                await testEnvironment.StartServerAsync(serverOptions);

                var options = new MqttClientOptionsBuilder()
                    .WithClientId("1")
                    ;

                var clients = await Task.WhenAll(Enumerable.Range(0, 10)
                    .Select(i => TryConnect(testEnvironment, options)));

                var connectedClients = clients.Where(c => c?.IsConnected ?? false).ToList();

                Assert.AreEqual(1, connectedClients.Count);
            }
        }

        private async Task<IMqttClient> TryConnect(TestEnvironment testEnvironment, MqttClientOptionsBuilder options)
        {
            try
            {
                return await testEnvironment.ConnectClientAsync(options);
            }
            catch (System.Exception)
            {
                return null;
            }
        }

there are 2 issues here:
more than one client thinks it is connected
most of the clients run into a timeout instead of disconnect exception, this leads me to belive that the server is not properly closing invalid connections.

i use the  HashSet with the lock：
HashSet < string > clientIds = new HashSet < string > ();
the rtos device reconnect by 20ms interval.
the rtos device can receive topic in a keepAliveInterval, over the  interval, can not receive topic.

the server is not properly closing invalid connections.

@zhou881117 I already created a pr that should improve threading behavior of MqttClientSessionsManager.
there where some problems that in case of a session takeover the old code removed the new connection instead of the old one.
Also the old connection was not properly closed which could lead to timeouts in the client.
you can have a look at #1004 if you can create a testcase that is still not working please share it so we can fix it

@chkr1011 hi christian i created a minimal repro for this issue in form of a unittest.
my expectation is that 9 out of 10 connections are closed and the last one is working correctly.
        [TestMethod]
        public async Task Manage_Session_MaxParallel()
        {
            using (var testEnvironment = new TestEnvironment(TestContext))
            {
                testEnvironment.IgnoreClientLogErrors = true;
                var serverOptions = new MqttServerOptionsBuilder();
                await testEnvironment.StartServerAsync(serverOptions);

                var options = new MqttClientOptionsBuilder()
                    .WithClientId("1")
                    ;

                var clients = await Task.WhenAll(Enumerable.Range(0, 10)
                    .Select(i => TryConnect(testEnvironment, options)));

                var connectedClients = clients.Where(c => c?.IsConnected ?? false).ToList();

                Assert.AreEqual(1, connectedClients.Count);
            }
        }

        private async Task<IMqttClient> TryConnect(TestEnvironment testEnvironment, MqttClientOptionsBuilder options)
        {
            try
            {
                return await testEnvironment.ConnectClientAsync(options);
            }
            catch (System.Exception)
            {
                return null;
            }
        }

there are 2 issues here:
more than one client thinks it is connected
most of the clients run into a timeout instead of disconnect exception, this leads me to belive that the server is not properly closing invalid connections.

i use you test case, new master ver： a26b641
now is ok

[TestMethod]
public async Task Manage_Session_MaxParallel_Subscribe()
{
using (var testEnvironment = new TestEnvironment())
{
testEnvironment.IgnoreClientLogErrors = true;
var serverOptions = new MqttServerOptionsBuilder();
await testEnvironment.StartServerAsync(serverOptions);
            var options = new MqttClientOptionsBuilder()
                .WithClientId("1").WithKeepAlivePeriod(TimeSpan.FromSeconds(10));

            bool hasReceive = false;
            Action onReceive = () =>
            {
                if (!hasReceive)
                {
                    hasReceive = true;
                }
            };

            var clients = await Task.WhenAll(Enumerable.Range(0, 25)
                .Select(i => TryConnect_Subscribe(testEnvironment, options, onReceive)));

            var connectedClients = clients.Where(c => c?.IsConnected ?? false).ToList();

            //Assert.AreEqual(true, connectedClients.Count > 1);

            System.Threading.Thread.Sleep(20 * 1000); //over 1.5T
            var option2 = new MqttClientOptionsBuilder()
                .WithClientId("2").WithKeepAlivePeriod(TimeSpan.FromSeconds(10));
            var sendClient = await testEnvironment.ConnectClientAsync(option2);
            await sendClient.PublishAsync("aaa", "1");
            System.Threading.Thread.Sleep(3000);
            Assert.AreEqual(true, hasReceive);
        }
    }

    private async Task<IMqttClient> TryConnect_Subscribe(TestEnvironment testEnvironment, MqttClientOptionsBuilder options, Action onReceive)
    {

        try
        {
            var sendClient = await testEnvironment.ConnectClientAsync(options);
            sendClient.ApplicationMessageReceivedHandler = new MQTTnet.Client.Receiving.MqttApplicationMessageReceivedHandlerDelegate(e =>
       {
           onReceive();
       });
            await sendClient.SubscribeAsync("aaa");
            return sendClient;
        }
        catch (System.Exception)
        {
            return null;
        }
    }

i create a test case to test subscribe when time over 1.5 keepAlive.
ver 3.0.8 ,some times is error
now the new master is ok
`[TestMethod]
public async Task Manage_Session_MaxParallel_Subscribe()
{
using (var testEnvironment = new TestEnvironment())
{
testEnvironment.IgnoreClientLogErrors = true;
var serverOptions = new MqttServerOptionsBuilder();
await testEnvironment.StartServerAsync(serverOptions);
            var options = new MqttClientOptionsBuilder()
                .WithClientId("1").WithKeepAlivePeriod(TimeSpan.FromSeconds(10));

            bool hasReceive = false;
            Action onReceive = () =>
            {
                if (!hasReceive)
                {
                    hasReceive = true;
                }
            };

            var clients = await Task.WhenAll(Enumerable.Range(0, 25)
                .Select(i => TryConnect_Subscribe(testEnvironment, options, onReceive)));

            var connectedClients = clients.Where(c => c?.IsConnected ?? false).ToList();

            //Assert.AreEqual(true, connectedClients.Count > 1);

            System.Threading.Thread.Sleep(20 * 1000); //over 1.5T
            var option2 = new MqttClientOptionsBuilder()
                .WithClientId("2").WithKeepAlivePeriod(TimeSpan.FromSeconds(10));
            var sendClient = await testEnvironment.ConnectClientAsync(option2);
            await sendClient.PublishAsync("aaa", "1");
            System.Threading.Thread.Sleep(3000);
            Assert.AreEqual(true, hasReceive);
        }
    }

    private async Task<IMqttClient> TryConnect_Subscribe(TestEnvironment testEnvironment, MqttClientOptionsBuilder options, Action onReceive)
    {

        try
        {
            var sendClient = await testEnvironment.ConnectClientAsync(options);
            sendClient.ApplicationMessageReceivedHandler = new MQTTnet.Client.Receiving.MqttApplicationMessageReceivedHandlerDelegate(e =>
       {
           onReceive();
       });
            await sendClient.SubscribeAsync("aaa");
            return sendClient;
        }
        catch (System.Exception)
        {
            return null;
        }
    }`

test is ok.
tks!

@zhou881117 I assume this is done. So I close this ticket.


chore: remove Div Int and Mod Int instances, so mathlib can use different ones

Investigating one solution to Mario's RFC about the Int Div convention. zulip.
Just seeing what happens in CI for now.

I think if we are going to do this in core, we should move all three definitions to core as well and give them all a native implementation.

I think we've decided at the dev meeting last year that we're going to change / to ediv (like Python).

we should move all three definitions to core as well and give them all a native implementation.

I would already be happy if we just harmonized the division definition so that std4 doesn't redefine it.  Adding the two other ones can be a follow-up PR if that speeds things up.


Commit linting

Description
Task link: https://trello.com/c/O7zkMZIX/41-commit-linting-hook

Added commit linting via commitlint with Husky-enabled hook (commit-msg)
Updated lifecycle hook scripts to work for Yarn Berry (based on Husky Yarn docs)
Explicitly set introduction MDX as first in story ordering (see this comment)

Test Steps

Locally, test commitlint and make sure it leverages commitlint.config.ts

This can be tested by piping input to the commitlint CLI. For example, echo 'test: example' | yarn commitlint

Verify the Husky commit-msg hook automatically executes commitlint

This can be tested by adding a random file (don't commit it, just git add), then testing various git messages with git commit
CLI & config unit tests will be added later once this config is extracted to a shared repo (checklist item added to https://trello.com/c/O7zkMZIX/41-commit-linting-hook scope)

Verify the commitlint.config.ts config looks good, particularly the commit types
Verify introduction story is loaded first on page load

Storybook Introduction is not loading first. Still jumps to Accordion component.

Everything lgtm! Verified all test steps! Introduction does get loaded first on initial page load for storybook now, commitlint config looks great and linting worked great! Great job!

Storybook still loading Accordion for me.

Storybook Introduction is not loading first. Still jumps to Accordion component.

Hmmm... worked for me. You want to try and remove node_modules + and reinstall deps (or even just remove the .cache directory inside node_modules). Then make sure that you are initially loading localhost:6006 with no additional path appended.

Verified storybook loads introduction on page load. Needed to pull changes again.


can't open file

[ERROR] addons/advdupe2/lua/advdupe2/sh_codec.lua:468: unknown duplication format

error - [C]:-1
Decode - addons/advdupe2/lua/advdupe2/sh_codec.lua:468
3. InitializeUpload - addons/advdupe2/lua/advdupe2/cl_networking.lua:241
4. DoClick - addons/advdupe2/lua/advdupe2/file_browser.lua:421
5. OnMouseReleased - lua/vgui/dlabel.lua:206
6. unknown - lua/vgui/dmenuoption.lua:116

hold on I'll just use my magic crystal ball to see which file you're talking about

@Sermax18 we need the file that you tried this with or we can't help. Feel free to reopen the issue when you can provide us with it.


Is there any practical usecase of this project ?

Asking this because you and i both know very well that even if it is possible to run big models with lower vram usage still it is not practical to use the models like that because your brain cells will get damaged with that much slow inference speed so is there any practical usecase of this project where we can get any better advantage when running small models (quantized models) or anything else which  might be beneficial for us ?

Is there any practical reason to waste time asking such question ?
If there were no usecase then people would not use it and the project would die, but this is quite the opposite !
This is great work, valued by at least 5k people, so please remove any assumption about others in your sentence, and keep both your brain behaviour and belief yours.


SVG in basic gray palette and laser power

Hi,
I tried to open an SVG to GCODE project in VB6 and it doesn't work. I'm not sure if the source is complete. I want to modify SVG (inkscape) in basic gray colors (gray 50%> laser 50% ...) and adjust the laser intensity accordingly.

What's the error you got from VB6?
The app doesn't support intensity, you'd have to add this to the function yourself.
Cheers,
-av

Log file:
'0' could not be loaded
Line 0: The file C:\Users\Milan\Documents\SVG-to-GCode-master\src\frmExport.frm could not be loaded.

Version 1.2.7
(source code is extensive and always difficult to navigate, sorry for any mistakes)

detection of the opened SVG file (tested only by Inkscape) on the color of the object in the basic gray palette and scale (inch or mm)
the color of the object can be used as laser power (black / 255 = 100% power --- gray 50% / 127 = 50% power ...) with the possibility of additional changes in the Sheet (L = xxx) button right
G-Code export, possibility to set HW laser power (PWM), correction of coordinate notation (G0 X 1,00 to G0 X 1.00) with possibility to choose decimal places and reduction of repeating coordinates one after the other
export respects the scale units in which the file is open (G20 > inches and G21 > mm)
vertical scale adjusted for better clarity with information in which units are worked
attempt to repair the filling - in progress
3:07 PM 7/17/2017
Version 1.2.6
Heavily sped up importing of large shapes

Did you end up getting it working?  Would you like to make a pull request?

This is my latest version. Tuning is still enough, but there is no time. The latest version helped me solve my engraving intent. I hope it helps others. I can't create an installation file. To install, use a lower version SVGtoGCODESetupV1.2.6.exe  and then just replace the main program SVGtoGCODE.exe
New changes:

adjustment of rendering during selection (faster with a lot of data)
index of individual lines for orientation
change the scale inch <> mm just by pressing the button on the ruler
Added classic dezign puzzle
Brief description in the SVGtoGCODE_změny_V1.2.71.pdf document


Adaptor size increases significantly when PT with LORA done with modules to save has #lora_modules_to_save: #  - embed_tokens #  - lm_head

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Lora Adaptor size shouldnt be in GBs on completeing Lora Pretraining.
This happens due to following config:
lora_modules_to_save:

embed_tokens
lm_head

Without this it works fine.
There was Peft Library issue for this but it still seems to be there. https://github.com/huggingface/trl/issues/1287
Current behaviour
On PT Lora Training Gemma 2B it creates an adaptor of 4GB almost same the size of the base model.
This happens due to the mentioned modules to save config.
If they are removed it works fine.
Given below is the cfg file i used.
Steps to reproduce
Pretrain Gemma with Lora 2B with given config.
Check the adaptor size.
Config yaml
base_model: google/gemma-2b
model_type: AutoModelForCausalLM
tokenizer_type: AutoTokenizer

load_in_8bit: false
load_in_4bit: true
strict: false

datasets:
  - path: /content/test_txt_data-10exmpl.json
    type: completion
    field: text
dataset_prepared_path:
val_set_size: 0
output_dir: ./qlora-out

adapter: qlora
lora_model_dir:

sequence_len: 600
sample_packing: true
pad_to_sequence_len: true

lora_r: 64
lora_alpha: 128
lora_dropout: 0.05
lora_target_modules:
  - q_proj
  - v_proj
  - k_proj
  - o_proj
  - gate_proj
  - down_proj
  - up_proj
lora_modules_to_save:
  - embed_tokens
  - lm_head
lora_target_linear: true
lora_fan_in_fan_out:

gradient_accumulation_steps: 1
micro_batch_size: 1
num_epochs: 1
optimizer: adamw_bnb_8bit
lr_scheduler: cosine
learning_rate: 0.0002

train_on_inputs: false
group_by_length: false
bf16: false
fp16: false
tf32: false

gradient_checkpointing: true
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: False

warmup_ratio: 0.1
evals_per_epoch: 1
eval_table_size:
eval_max_new_tokens: 128
eval_sample_packing: False
saves_per_epoch: 1
debug:
deepspeed:
weight_decay: 0.0
fsdp:
fsdp_config:
special_tokens:

save_safetensors: True

Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
latest
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

yeah, this definitely seems like an upstream PEFT issue.


Add github action to publish docker image

What's the purpose of this PR
Add github action to publish docker image
Which issue(s) this PR fixes:
Fixes #4674
Brief changelog

add docker publish workflow
support arm64 architecture
update apollo release guide

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

sample run
sample image


Update docs, index new specs, and add eTRNG disclaimer

Works in tandem with https://github.com/chipsalliance/caliptra-rtl/pull/272

Partially addresses https://github.com/chipsalliance/Caliptra/issues/152

Required https://github.com/chipsalliance/caliptra-rtl/pull/274 because errors


Add total waste to categories

calculate sum of disposed and recycled waste and sum of recycled waste
add stacked graph showing disposed and recycled waste

added as aggregation of all waste categories and grouped by commune and year then concatenated with original dataframe


Explicitly require services to use our hosting

We've received some feedback that it's not clear (particularly to suppliers) that they should be hosting services using our platforms. This attempts to address that, and also provides an externally-useful email address to get more information.

@SteveMarshall the htmlproofer gives negative results for working links, I am looking into it on another branch, it can be ignored for now


Dynamic slicing on data-dependent value is not supported

Description of the bug:
Hi,
we want to convert wenet from pytorch to TFLite, but got the error
"torch._dynamo.exc.Unsupported: Dynamic slicing on data-dependent value is not supported"
after tracing the code, it seems the problem is below function
def reverse_pad_list(ys_pad: torch.Tensor,
                     ys_lens: torch.Tensor,
                     pad_value: float = -1.0) -> torch.Tensor:
    """Reverse padding for the list of tensors.

    Args:
        ys_pad (tensor): The padded tensor (B, Tokenmax).
        ys_lens (tensor): The lens of token seqs (B)
        pad_value (int): Value for padding.

    Returns:
        Tensor: Padded tensor (B, Tokenmax).

    Examples:
        >>> x
        tensor([[1, 2, 3, 4], [5, 6, 7, 0], [8, 9, 0, 0]])
        >>> pad_list(x, 0)
        tensor([[4, 3, 2, 1],
                [7, 6, 5, 0],
                [9, 8, 0, 0]])

    """
    r_ys_pad = pad_sequence([(torch.flip(y.int()[:i], [0]))
                             for y, i in zip(ys_pad, ys_lens)], True,
                            pad_value)
    return r_ys_pad

can you please help to give some advice~
thanks~
Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

Hi @markiicheng, hard to say what the most correct action is w/o some more context... I believe the offending code is y.int()[:i], is there any way to make that slicing static rather than dependent on i? Otherwise I believe at root it is a PyTorch Export issue: https://pytorch.org/docs/stable/export.html. Can you try to export independent of AI-Edge-Torch? -- if the issue is still there, then I believe you need to create an issue there but I do believe if you can find a way to do static slicing, it will resolve your issue.

Hi @pkgoogle,
you can get more model info from https://github.com/wenet-e2e/wenet.git, it's an open source audio recognition model.
It can be exported to ONNX by PyTorch Export. (https://github.com/wenet-e2e/wenet/blob/main/wenet/bin/export_onnx_cpu.py)
If this is the limization of ai-edge-torch, maybe we need to find another way. Because it is hard to ask user to modify their model  to meet the conversion tool's requirement.
Thanks~~

It seems like torch.onnx is using a different code workflow than torch.export ... do you have the script which produced the original error? Something that loads the model module and calls ai_edge_torch.convert(model.eval(), ...) on it? That will be helpful, thanks.

please refer to the below~thanks~~
import os
import math
import torch
from torchsummary import summary
import torchvision
import ai_edge_torch
import tensorflow as tf
import numpy as np
import torch.nn as nn
import torch.nn.functional as F

# ASR - wenet
model_path = "model_path"
input_shape = (1, 64, 80)
input1_shape = (1, 4, 187)

pytorch_model = torch.load(model_path)
sample_inputs = (torch.randn(input_shape), torch.tensor([1]), torch.randn(input1_shape), torch.tensor([1]),)

edge_model = ai_edge_torch.convert(pytorch_model.eval(), sample_inputs)
edge_model.export("edge_model.tflite")

please refer to the below~thanks~~
import os
import math
import torch
from torchsummary import summary
import torchvision
import ai_edge_torch
import tensorflow as tf
import numpy as np
import torch.nn as nn
import torch.nn.functional as F

# ASR - wenet
model_path = "model_path"
input_shape = (1, 64, 80)
input1_shape = (1, 4, 187)

pytorch_model = torch.load(model_path)
sample_inputs = (torch.randn(input_shape), torch.tensor([1]), torch.randn(input1_shape), torch.tensor([1]),)

edge_model = ai_edge_torch.convert(pytorch_model.eval(), sample_inputs)
edge_model.export("edge_model.tflite")

Thanks... where is the model_path? I'm not familiar with the "wenet" project... if i try to use it as in the github wenet.load('chinese'), It gives me a wenet model not a torch model. When I load it via code, is there a known location it saves everything to that I can load from? Or where can I find whatever is at model_path?

I think it's complicated to get the requirement from wenet github (model and needed files)
so I upload the all files to the google drive. you can download it and extract it to the same location of script, maybe it would work.

Thanks~

Hi @markiicheng, thanks I was able to reproduce w/ your files. Using our find_culprits function it can be shown this is a torch exportable issue:
import torch

from ai_edge_torch.debug import find_culprits

pt_model = torch.load("wenet_test.pt")

input_shape = (1, 64, 80)
input1_shape = (1, 4, 187)
sample_inputs = (torch.randn(input_shape), torch.tensor([1]), torch.randn(input1_shape), torch.tensor([1]),)

culprits = find_culprits(pt_model.eval(), sample_inputs)
culprit = next(culprits)
culprit.print_code()

More information can be found here: https://github.com/google-ai-edge/ai-edge-torch/blob/main/docs/pytorch_converter/README.md#debugging--reporting-errors
Output:
python find_culprit.py
# a lot output, same stack trace that you are getting
  File "xxxxxxxx/miniconda3/envs/aet_main/lib/python3.11/site-packages/torch/_dynamo/variables/lists.py", line 634, in __init__
    unimplemented("Dynamic slicing on data-dependent value is not supported")
  File "xxxxxxxxx/miniconda3/envs/aet_main/lib/python3.11/site-packages/torch/_dynamo/exc.py", line 221, in unimplemented
    raise Unsupported(msg)
torch._dynamo.exc.Unsupported: Dynamic slicing on data-dependent value is not supported

from user code:
   File "xxxxxxx/issues/ai-edge-torch/168/wenet/transformer/asr_model.py", line 111, in forward
    loss_att, acc_att = self._calc_att_loss(encoder_out, encoder_mask,
  File "xxxxxxxx/issues/ai-edge-torch/168/wenet/transformer/asr_model.py", line 147, in _calc_att_loss
    r_ys_pad = reverse_pad_list(ys_pad, ys_pad_lens, float(self.ignore_id))
  File "xxxxxx/issues/ai-edge-torch/168/wenet/utils/common.py", line 171, in reverse_pad_list
    r_ys_pad = pad_sequence([(torch.flip(y.int()[:i], [0]))
  File "xxxxxxx/issues/ai-edge-torch/168/wenet/utils/common.py", line 171, in <listcomp>
    r_ys_pad = pad_sequence([(torch.flip(y.int()[:i], [0]))

Set TORCH_LOGS="+dynamo" and TORCHDYNAMO_VERBOSE=1 for more information

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "xxxxxxx/issues/ai-edge-torch/168/find_culprit.py", line 13, in <module>
    culprit = next(culprits)
              ^^^^^^^^^^^^^^
  File "xxxxxx/git/ai-edge-torch/ai_edge_torch/debug/culprit.py", line 485, in find_culprits
    for search_result in _search_model(
  File "xxxxxxx/git/ai-edge-torch/ai_edge_torch/debug/culprit.py", line 389, in _search_model
    raise ValueError(
ValueError: Your model is not exportable by torch.export.export. Please modify your model to be torch-exportable first.

Can you please raise an issue with PyTorch? https://github.com/pytorch/pytorch/issues. If you have no more open items for this issue on our end, please feel free to close or ask any additional questions.

Hi @pkgoogle ,
understand, thanks for your help~~


Errors found in library_numpy notebok

Typo:
Exercises on specifying the data type.

Create an array of 100 booleans all set to True. The array should have dimensions 2x3.

The part "The array should have dimensions 2x3." should be removed

In Indexing, slicing, and iterating, the first sentence after the first python code starts with Unlinke.

In the final exercises, when integrating the function we need to be explicit about the density of x points. Because different point density gives slightly different results. "Slightly different" means sufficient enough to cause the test failure.

For the exercise on eigenvalues:

The solution uses eigh. If people use eig the test will fail because the order of eigenvalues is different. This can be overcome by testing for invariants like the trace and the determinant.
The exercise should ideally be parametrized in order for multiple matrices to be teasd

In Exercises on NumPy routines, exercise 2., tests fail if we do not use the fact that the matrix is hermitian: eig and eigh give same eigenvalues but in different orders, and eigenvectors are slightly different

In Exercises on NumPy routines, exercise 2., tests fail if we do not use the fact that the matrix is hermitian: eig and eigh give same eigenvalues but in different orders, and eigenvectors are slightly different

Thanks, we already noted this above


Generate/Import competition description from run

Ideally, one would be able to import a previous run as a competition in order to be able to run it again.

Ideally, there is a competition importer, whose UI / interface is also able to import the competition of a run (and discard the run information). Also inform the users about what has happened.
Officially, this will be the Competition Importer

This has been added on dev.
Currently, it is an explicit flag that must be set in competition import. Do we need an 'auto-detect' feature here?

Implemented!


chore: add tests

Closes #39

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 20.00000% with 8 lines in your changes missing coverage. Please review.

Project coverage is 51.29%. Comparing base (743698e) to head (6b26d8a).

Files with missing lines
Patch %
Lines

internal/output/render.go
0.00%
3 Missing :warning:

internal/output/table.go
0.00%
3 Missing :warning:

internal/output/json.go
50.00%
1 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   50.64%   51.29%   +0.64%
==========================================
  Files          17       17
  Lines         847      848       +1
==========================================
+ Hits          429      435       +6
+ Misses        402      396       -6
- Partials       16       17       +1

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


I don't see any MQTT messages in a client.

Hi,
I Like this multisensor a lot! I have one problem. ik don't see the messages in an MQTT client. like MQTT spy. When i subscribe to topic # i don't get any messages. I have owntracks on my phone and that works great in HASSIO mosquitto.
Can anybody help what to look for?
This is the serial monitor in arduino IDE.
...
WiFi connected
IP address:
<IP_ADDRESS>
Ready
IPess: <IP_ADDRESS>
Attempting MQTT connection...connected
Setting LEDs:
r: 0, g: 0, b: 0
{"state":"OFF","color":{"r":255,"g":255,"b":255},"brightness":255,"humidity":"0.00","motion":"","ldr":"0","temperature":"0.00","heatIndex":"nan"}
{"state":"OFF","color":{"r":255,"g":255,"b":255},"brightness":255,"humidity":"0.00","motion":"standby","ldr":"0","temperature":"0.00","heatIndex":"nan"}
{"state":"OFF","color":{"r":255,"g":255,"b":255},"brightness":255,"humidity":"0.00","motion":"standby","ldr":"60","temperature":"0.00","heatIndex":"nan"}
{"state":"OFF","color":{"r":255,"g":255,"b":255},"brightness":255,"humidity":"0.00","motion":"standby","ldr":"60","temperature":"21.60","heatIndex":"nan"}
{"state":"OFF","color":{"r":255,"g":255,"b":255},"brightness":255,"humidity":"67.70","motion":"standby","ldr":"60","temperature":"21.60","heatIndex":"16.64"}

Never mind. I now use this source.
https://github.com/ShawnCorey/ESP-MQTT-Multisensor-Authdiscovery

just wanted to follow up on this in case anyone else has this issue. as I had a similar issue and found my issue to be that I hadn't modified the PubSubClient.h from the PubSubClient library
modifying the MQTT_MAX_PACKET_SIZE to be '512' as mentioned in Ben's video: https://youtu.be/jpjfVc-9IrQ?t=10m42s
from:
#define MQTT_MAX_PACKET_SIZE 128
to:
#define MQTT_MAX_PACKET_SIZE 512


Invoice: UI fixes

Fix calculation

send BTC => hint must not be in BTC
Sender -> Receiver / Recipient / Payee

Remove multiple paddings

Handle overflow

Fix overlapping

disallow invoice from=to


Slow performance when open zarr file with numpy>2.0.0

What happened?
Hi!
I want to open a zarr dataset lazily.
On my computer:
With numpy==1.26.4 it takes around 1.5sec
With numpy==2.1.1   it takes around 5sec
It's also slow on an ubuntu machine.
Unfortunately, I don't really have the time to deep dive into the issue and pinpoint exactly what is the piece of code that takes much more time than before. As little as I tested, it doesn't seem to come from the http calls.
What did you expect to happen?
I expect that the time to lazily open the dataset is the same whatever the numpy version.
Minimal Complete Verifiable Example
import xarray
import time

top = time.time()
dataset = xarray.open_dataset(
    "https://s3.waw3-1.cloudferro.com/mdl-arco-time-035/arco/MEDSEA_MULTIYEAR_PHY_006_004/med-cmcc-cur-rean-h_202012/timeChunked.zarr",
    engine="zarr",
)
print(f"Took: {time.time() - top}s")
# with numpy==1.26.4: ~1s
# with numpy==2.1.1: ~5s

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.
[X] Recent environment — the issue occurs with the latest version of xarray and its dependencies.

Relevant log output
No response
Anything else we need to know?

It looks like with numpy>=2 we try to use cftime (which I didn't have installed in my environment), and if I then pass use_cftime=False I get:
OutOfBoundsTimedelta: Cannot cast 45757440 from m to 'ns' without overflow.

(for reference, time values are encoded as minutes since 1900-01-01 in the "standard" calendar with dtype int32, the values are between 45757440 and 64471620)
I wonder if this has anything to do with the changed dtype casting rules in numpy>=2?
cc @spencerkclark, in case you have any insight here

Thanks @keewis for taking a look—was this with xarray main?  This seems reminiscent of https://github.com/pydata/xarray/pull/9498#pullrequestreview-2307790147, which I fixed in https://github.com/pydata/xarray/pull/9518.  In other words due to an upstream pandas issue we inadvertently needed to fall back to cftime to decode times encoded with small integer dtypes with NumPy > 2.0; #9518 should have restored the old behavior with a workaround.

actually, no. Let me retry with xarray main and get back to you

I can confirm that xarray main does not have that issue, which means the next release will fix this.
Thanks for the well-written report, @renaudjester.
(as an additional comment, the link you've been using is actually on s3, which that by using that protocol you can save some additional time:
xr.open_dataset(
    "s3://mdl-arco-time-035/arco/MEDSEA_MULTIYEAR_PHY_006_004/med-cmcc-cur-rean-h_202012/timeChunked.zarr",
    engine="zarr",
    storage_options={"endpoint_url": "https://s3.waw3-1.cloudferro.com", "anon": True},
)

Great, thanks for confirming @keewis.

Thanks a lot :D
Super, I will wait for the next release then!
@keewis Thanks for the tips! Could you just point out to me why this is a bit more efficient?

as far as I can tell (and I'm by no means an expert on this), the S3 protocol is a REST API. This means that while it is possible to talk to it using just HTTP vocabulary, it doesn't allow you to be as precise when requesting data, so you'll have some overhead.

Do you have an ETA for the release of this fix?


Fixes the compilation error.

The project doesn't compile when coroutines are used as in https://github.com/Kotlin/kotlinx.coroutines/issues/591
Since this project is using the non-experimental version of coroutines,
I think the fix should be to update the kotlin version to any of the RC
versions.

LGTM, thanks for PR!


The ClusterAdmin api AlterPartitionReassignments can't support kafka V2.0

dear all:
pls pay attention to the issue ,and give me some solutions,thanks very much!
Versions
sarama v1.29.1
kafka V2.0
go 1.16.5
Logs
if i use config sarama.V2_0_0_0,got error:
failed to reassign partitions for topic:
kafka server: The version of API is not supported.
if use config sarama.V2_4_0_0 or sarama.MaxVersion,got error:
failed to reassign partitions for topic:
EOF
Problem Description
The ClusterAdmin api AlterPartitionReassignments can't support kafka V2.0
if i use config sarama.V2_0_0_0,got error:
failed to reassign partitions for topic:
kafka server: The version of API is not supported.
if use config sarama.V2_4_0_0 or sarama.MaxVersion,got error:
failed to reassign partitions for topic:
EOF

@systemcia I'm not sure what to suggest? That remote kafka API (AlterPartitionReassignments) was added in Kafka 2.4.0 so you would need to upgrade your brokers to Kafka 2.4.0 if you want to make use of it
Before Kafka 2.4.0 the only way to achieve the same (move partition replicas around) was to use the kafka-reassign-partitions.sh tool to submit a json file to zookeeper which listed the topic partitions and which brokers you wanted to host the replicas.

@dnwe ok ,got it , thank you !


Investigate feasibility of supporting a configurable reward schedule for staking

Since long-term returns are dictated by the reward rate schedule the protocol implements, we should add the ability for Playgrounds users to simulate different reward rate schedules
The reward rate frameworks like OIP-18 and KIP-3 don't offer specific guidance about when exactly the reward rate will be reduced, so we need to offer a more "open-ended" solution that allows users to configure when they think the reward rate will be reduced and what level it will be reduced to
We could bake in a "default" schedule based on conservative estimates for each protocol, and then allow the user to tweak these based on what they think would happen (both adding more "steps" and adjusting the rate of each step)
This will be a complex implementation, treat this as a discovery ticket to assess the difficulty and map out if this makes sense

hmm, we could create a table with two columns. Col A is reward rate tier and Col B would be when they think the transition will happen. Then we just loop through the rows for the growth forecast

I'll experiment with the idea. def doable

I had added a ticket that duplicates this. Instead I'll leave what I had written below and delete my duplicate.

As a User:
I'd like to be able to model a decline in rebase rate over time. This effect can be accomplished by just lowering the APY to expected average for that period see: Potted Meat's Emissions Predictions Dune page for Ohm. But we should give the user some controls for that.
Both Klima has a frame work proposed and passed that can be used for learning portions related to this.
Klima Framework described in KIP-3, rational explained in this medium article and modified for shorter Polygon block times in KIP-5


[Java] Fixed size message Broadcast Buffer + tests

This is a new broadcast buffer implementation.
It is different from the one already present in this library because although it allows variable sized transmissions it needs to know at configuration time the maximum allowed transmission size.
Thanks this restriction it allows precise counting of transmissions and losses on the receiver(s) side.
Most of the original semantic on the receiver is preserved and enhanced to allow different receive strategies, like extended receiveNext outcomes (LOSS, ANY_AVAILABLE, NOT_AVAILABLE) and explicit fast-forward methods (ie keepUpWithTrasmitter).
While on the transmitter side has been added a zero-copy transmission method in addition to the one
using copy.
The related CopyBroadcastReceiver has been changed as well to accomodate the enhancements of the receiver side.
Implementation-wise it is using a mechanics similar to the Fast Flow queues (aka using message indicators) with all the well-known performance benefits of them (I've some measurements using JMH to share if needed).
I have added 2 test classes covering most of the same cases of the original broadcast buffer and based
on tests already written for it.
Please review and If it will pass I hope it will be a good companion for the other awesome data structures of the library.

I'm looking forward to reviewing this. However it may take a little time to get round to reviewing as we have so much other things on at present.

@mjpt777 np and thanks!! The pr is here when the time will come :+1:

I'm closing this one, because it doesn't seem needed into Agrona :+1:


Show counterexamples ==> trailing spaces are deleted

VSCode 1.55.2
Dafny extension 1.4.0
macOS 11.2.3
When typing on a .dfy file while "Show counterexamples" is on, a space at the end of a line will be deleted unless a new char is added "soon enough".

This issue is also present in the Dafny extension 1.5.0.
https://user-images.githubusercontent.com/4139546/115805201-d807c600-a427-11eb-84be-9fd3fda56d78.mp4

Thanks for your report. Unfortunately, I am unable to reproduce. Neither my Windows nor my Ubuntu machine has such a behavior.
I have quickly scanned through the code if there is something suspicious, but I couldn't identify any location where any editor text is altered.
Although it's strange that this only happens if you have "Show counterexamples" enabled, but you may have some other extensions that interfere with this feature. Have you tried a clean VSCode installation?

Since I haven't heard back I assume that this is no longer an issue.


1.20.1, SS 2.1.2 - /dumpstructuresets command does not exist

The mod page exclaims the following: "Tip: you can dump all structure sets to a file in the right format by using the custom command /dumpstructuresets"
However, this command simply does not exist. This makes it a little tricky to adjust the structures I want when I can't be sure I get them all and have to manually check the /locate autofill list.

This feature was added in v2.2
This version will soon be available for 1.20.1
Have a good day

Oh

Oh

Will let you know when it's out!

Appreciated.

Appreciated.

Available for Fabric 1.20.1 on Modrinth


revdeps: add an option for prefx-based list

This is a kind of analog of Scan, when go run ./tools/revdeps -prefix would result in a list of things that depend on the packages with a given prefix, rather then a direct match.

This allows for queries like
$ time go run ./tools/revdeps -prefix -store ../_dep-graph-go-mod-41286 gopkg.in/src-d/enry
github.com/devspace-cloud/devspace/pkg/devspace/generator
github.com/okteto/okteto/cli/pkg/linguist
github.com/okteto/okteto/pkg/linguist
github.com/smola/bblfsh-benchmark
github.com/smola/language-dataset
github.com/src-d/engine/cmd/srcd-server/engine
github.com/tsileo/blobstash/pkg/gitserver

4.35s

or
$ time go run ./tools/revdeps -prefix -store ../_dep-graph-go-mod-41286 gopkg.in/src-d | wc -l
529

4.74s

and
$ time go run ./tools/revdeps -prefix -store ../_dep-graph-go-mod-41286 github.com/src-d | wc -l
53

4.40s

48914d1 solves the need for a prefix-based scan! 🙇


Override colorscheme for plugin nvim-treesitter

I'm using your monokai-pro colorscheme with Neovim (on Linux and OpenBSD) and I'm very happy with it :)
But with nvim-treesitter.lua plugin (also used with Neovim), I don't like italic style used for some items, especially for Markdown inline => https://github.com/loctvl842/monokai-pro.nvim/blob/master/lua/monokai-pro/theme/plugins/nvim-treesitter.lua
Is it possible and how could I override monokai-pro colorscheme for nvim-treesitter.lua plugin ? I would like to set italic = false for @markup.italic.markdown_inline and other items.

Yes sure you can use override function


feat: web app for previewing svg markup returned by BlissSVGBuilder

This adds another web application to the ./apps folder, specifically ./apps/svg-preview.
The app can be run on localhost and is started by the existing npm run serveAppsDemos script.  If so launched, use this following URL to access it:  http://localhost:5173/apps/svg-preview/svg-preview.html

@cindyli I have requested a review from you, but I think I should modify the main README and possibly add a README to the svg-preview folder in order to document how to use this app.  However, feel free to start reviewing the code while I work on the documentation.

@klown do you think if this pull request is still needed. Seems the SVG preview is already covered by the pull request that generates palette JSON files.

@klown do you think if this pull request is still needed. Seems the SVG preview is already covered by the pull request that generates palette JSON files.

No, not anymore @cindyli .  It was the inspiration for the SVG string inputs for the palette generator PR.
I am closing this one as overtaken by #25.


WIP: Add a new container/store module

Depends:

#105

The initial scope of this project was just "encapsulating" ostree
commits in containers.
However, when doing that a very, very natural question arises:
Why not support deriving from that base image container, and
have the tooling natively support importing it?
This initial prototype code implements that.  Here, we still use
the tar::import path for the base image - we expect it to have
a pre-generated ostree commit.
This new container::store module processes layered images and
generates (client side) ostree commits from the tar layers.
There's a whole lot of new infrastructure we need around mapping
ostree refs to blobs and images, etc.

This will close https://github.com/ostreedev/ostree-rs-ext/issues/12 once it has more tests and validation.

Open design question: right now I'm trying to strongly differentiate non-layered (i.e. pure ostree encapsulation, ideally GPG signed) from layered (from e.g. Dockerfile style build).  But, does that make sense?  Should we have a --layered option, or should we just drop the distinction and automatically apply a layered container if we're pointed at one?

Rebased :surfer: - now depends on https://github.com/ostreedev/ostree-rs-ext/pull/102
And when I tried this in the real world, I hit error: Preparing /etc: Tree contains both /etc and /usr/etc precisely because https://github.com/containers/buildah/issues/3523
So we need to teach our tar importer to drop that junk.

OK upgrades don't work yet, we're not storing tags correctly.

OK upgrades don't work yet, we're not storing tags correctly.

Fixed

Trying to use this to deploy the initial container, we also need a copy API I think so we can schlep the already-downloaded container from the cosa side into the VM without re-pulling.
This also ties into https://github.com/ostreedev/ostree-rs-ext/issues/115

OK lifting draft on this.  There's a huge amount of stuff left to do, but I think iterating on main will be better than having this sit in a PR.

I think @travier was going through reviewing this right now too, so could wait a bit before merging.

OK, though since this is all experimental, not critical path stuff we can always do post-merge fixups!


($authenticator) must be of type Symfony\Component\Security\Http\Authenticator\AuthenticatorInterface, Lexik\Bundle\JWTAuthenticationBundle\Security\Guard\JWTTokenAuthenticator given

Hello everyone.
I upgraded my symfony from 5.3 to 5.4
I tried using the guard and the custom_authenticaion
But no option really works
here is the security.yml which causes this error message when trying to call an endpoint after generating a Bearer Token
Symfony\\Component\\Security\\Http\\Authenticator\\Debug\\TraceableAuthenticator::__construct(): Argument #1 ($authenticator) must be of type Symfony\\Component\\Security\\Http\\Authenticator\\AuthenticatorInterface, Lexik\\Bundle\\JWTAuthenticationBundle\\Security\\Guard\\JWTTokenAuthenticator given
For example i send /api/login I get a Token and then i try to use /api/XYZ
Then the error happens
security:
    enable_authenticator_manager: true

    password_hashers:
        Symfony\Component\Security\Core\User\PasswordAuthenticatedUserInterface: 'auto'
        App\Entity\BackendUser:
            algorithm: auto
        App\Entity\User:
            algorithm: auto

    # https://symfony.com/doc/current/security.html#where-do-users-come-from-user-providers
    providers:
        # used to reload user from session & other features (e.g. switch_user)
        app_user_provider:
            entity:
                class: App\Entity\BackendUser
                property: email
        auth_token:
            id: App\Security\AuthTokenUserProvider
        jwt:
            id: App\Security\JwtUserProvider

    firewalls:
        dev:
            pattern: ^/(_(profiler|wdt)|css|images|js)/
            security: false

        login:
            pattern:  ^/api/login
            stateless: true
            provider: auth_token
            json_login:
                check_path: /api/login
                success_handler: lexik_jwt_authentication.handler.authentication_success
                failure_handler: lexik_jwt_authentication.handler.authentication_failure
            custom_authenticators:
                - App\Security\UserAuthenticator

        api:
           pattern:   ^/api
           stateless: true
           provider: jwt
           custom_authenticators:
               - lexik_jwt_authentication.jwt_token_authenticator

        easy_admin:
            pattern:   ^/admin
            lazy: true
            provider: app_user_provider
            custom_authenticators:
                - App\Security\EasyAdminAuthenticator
            logout:
                path: app_logout
                target: app_login

    # Easy way to control access for large sections of your site
    # Note: Only the *first* access control that matches will be used
    access_control:
        # Easy Admin Routes
        - { path: ^/admin/login,      roles: IS_AUTHENTICATED_ANONYMOUSLY }
        - { path: ^/admin,            roles: IS_AUTHENTICATED_FULLY }
        # Api Routes
        - { path: ^/api/login,        roles: IS_AUTHENTICATED_ANONYMOUSLY }
        - { path: ^/api,              roles: IS_AUTHENTICATED_FULLY }
        # Translations
        - { path: ^/translations,                   roles: IS_AUTHENTICATED_FULLY }
        - { path: ^/translations/grid,              roles: IS_AUTHENTICATED_FULLY }`

After this i tried using
           guard:
             authenticators:
               - lexik_jwt_authentication.jwt_token_authenticator

With
   enable_authenticator_manager: false
But this also does not work
What i can see is that when i add my jwt token to https://jwt.io/
It says
Invalid Signature
So i'm really not sure if this is a version problem or a config problem

Since you are using Symfony 5.4, you really should start using new authenticator instead of guard (and do not mix them together).
What is the different between auth_token and jwt provider? I saw your firewall config the login using auth_token provider and api using jwt provider.
I think you can just remove the custom_authenticators under login and api firewall if you have no custom logic during authentication.

Closing as explained.


fix(calendar): fixed dark mode

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7594

Relevant Lines:
8629

💛  - Coveralls


Add support for component-prometheus

Signed-off-by: Nicolas Bigler<EMAIL_ADDRESS>Checklist

[ ] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[ ] Link this PR to related issues or PRs.

Looks like this needs a make gen-golden-all to update the outputs to match Commodore >= v1.12.0 (due to projectsyn/commodore#689)

Done. (I actually neeeded to update my commodore version locally so that make gen-golden-all actually fixes this)


Rework own property enumeration

Properry name filtering is done during the enumeration.
Should use less memory, since no "sorting" phase, and should be much faster in some corner cases, e.g. when the indexed properties are not needed for a large typed array / large strings.

Please rebase.

@zherczeg, please rebase if still needed or close if it is not.


cargo-nextest: 0.9.33 -> 0.9.34

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cargo-nextest/versions.
meta.description for cargo-nextest is: "Next-generation test runner for Rust projects"
meta.homepage for cargo-nextest is: "https://github.com/nextest-rs/nextest"
meta.changelog for cargo-nextest is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.34 with grep in /nix/store/g6iwcd2yjdnh33z3k5jznpi7711k76r8-cargo-nextest-0.9.34
found 0.9.34 in filename of file in /nix/store/g6iwcd2yjdnh33z3k5jznpi7711k76r8-cargo-nextest-0.9.34

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-nextest

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g6iwcd2yjdnh33z3k5jznpi7711k76r8-cargo-nextest-0.9.34 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-nextest https://github.com/r-ryantm/nixpkgs/archive/f0ff2891f4671df563e23a5e212359294db8bd5f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g6iwcd2yjdnh33z3k5jznpi7711k76r8-cargo-nextest-0.9.34
ls -la /nix/store/g6iwcd2yjdnh33z3k5jznpi7711k76r8-cargo-nextest-0.9.34/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-nextest

Maintainer pings
cc @Ekleog for testing.

Result of nixpkgs-review pr 186407 run on x86_64-linux 1

1 package built:

cargo-nextest


Added Aws::EC2::Client config checking

If the environment variable DISABLE_AWS_CLIENT_CHECK is properly configured, now awsecrets will attempt to create an Aws::EC2::Client.new (discarding it latter) in order to validate if all configuration parameters are in place.
Documentation updated to reflect that.
Since that by default this will not change the previous behavior, the change should not have any impact clients unaware of DISABLE_AWS_CLIENT_CHECK environment variable.

Kind like messed up putting documentation changes together with those from #26 :-(
Let me know if that would be a problem (considering that the #26 is not approved).

@k1LoW , I've merged with #21 and things are looking good now!
Could you please take a look?

@k1LoW , I got a positive feedback from #21 .
Could please check and approve this PR?
Thanks!

@glasswalk3r Sorry !!!! sorry for the late reply !!

Released as v1.15.0.


End to End tests with patractlabs/cargo-contract

Issue summary
Ensure debug info works in browser

#44


Write/WriteRTP shouldn't block before SRTP

A Track shouldn't block until a PeerConnection is connected. It should
only send to PeerConnections that are fully established. v3 Track API
changed this behavior by mistake.
Related to pion/transport#116

Personally how do you feel about rolling back the context changes and
exposing SetReadDeadline and SetWriteDeadline?

That would be very, very helpful indeed.  From an API point of view it is
much cleaner (avoids carrying an extra parameter that is generally unused),
and the slight loss of expressivity is not a big deal in practice.
Pleae make sure that you can recover after a missed deadline, i.e. that
the operation can be restarted after it has returned with a missed
deadline error.  (Both bufio and gorilla/websocket have he issue that the
reader or writer needs to be closed after a deadline has been missed.)

This is what I think we should do, please correct me if you disagree/am missing anything.

Remove context, from pion/webrtc, pion/transport and pion/srtp. Make sure we keep these numbers available, and if they become better we can revisit.
Add SetDeadline/ReadDeadline and make sure they are tested/working as expected.

I personally really want to get v3.0.0 out. Announcing all these improvements is going to bring a lot of new people in that have ignored Pion because it didn't have what they needed before. The longer we wait, the more contributions we miss out on.
@tarrencev @OrlandoCo @jech @at-wat

Remove context, from pion/webrtc, pion/transport and pion/srtp. Make sure we keep these numbers available, and if they become better we can revisit.

As pion/transport supports both and has no performance problem, it can be left as is.


Install as app not working on Ubuntu 22.04 with Chromium

An icon gets created, but nothing gets added to the app menu or dock.
After "allowing launch" and launching it, it opens an empty Chromium window.

Version 100.0.4896.127 (Official Build) snap (64-bit)
It happens on stable and Pybricks Beta.

Regular "Shortcuts" also don't work, so this is probably unrelated to Pybricks-Code.

As a workaround, use Google Chrome.

Upstream bug: https://bugs.launchpad.net/ubuntu/+source/chromium-browser/+bug/1732482


Endpoint Summary Issue

After deleting an endpoint all of the endpoints summary will show wrong counts

count of endpoint's clusters and attributes need to be updated real-time when user changes the clusters state, but it doesn't work in this way except for first endpoint

The PR was not merged for this

The goal here is to now get rid of endpoint summary. We had long conversations on this and this is not really needed and not worth spending so much time on it. So get rid of cluster, attribute and command counts.

@brdandu I just tested this and it is fixed. Please reopen if you disagree.


Expose setting the cursor

This adds support for the set of cursors shared between macOS
and Windows. It does not include support for custom cursors.

At least on Cocoa there's the convention of treating the cursor as a stack, with push and pop operations for this use case. We could consider implementing something similar.


HTTP/1.1 400 Bad Request

I have a similar issue as described in issue #5.
I've added the following code to the composer.json file that's in the root of my theme
  "repositories": [     {       "type": "package",       "package": {         "name": "advanced-custom-fields/advanced-custom-fields-pro",         "version": "5.3.10",         "type": "wordpress-plugin",         "dist": {           "type": "zip",           "url": "https://connect.advancedcustomfields.com/index.php?p=pro&a=download"         },         "require": {           "philippbaschke/acf-pro-installer": "^1.0",           "composer/installers": "^1.0"         }       }     }   ],
I then added the ACF_PRO_KEY variable (with key) to the .env file.
Then when I run the command composer require advanced-custom-fields/advanced-custom-fields-pro:*, I get the following error message:
[Composer\Downloader\TransportException] The 'https://connect.advancedcustomfields.com/index.php?p=pro&a=download&t=5.3.10&k=b3jk...' URL could not be accessed: HTTP/1.1 400 Bad Request
My composer version is:
composer --version Composer version 1.4-dev 2017-01-22 19:01:35
When checking the url (with and without the GET params) in Chrome's Postman I get the same 400 Bad Request error.
What is causing this issue and how can I fix this?

Sorry for the late answer. I did not see the issue. 😞
Did you double check your key? I used the URL that you posted https://connect.advancedcustomfields.com/index.php?p=pro&a=download&t=5.3.10&k=WRONG_KEY and got a 400 as well (just tested it in the Browser). However, when I replace WRONG_KEY with my key, the download works just fine. 😕

I'm running into this problem as well on my mac and also on my server.
The issue seems to be that composer install try to install advanced-custom-fields/advanced-custom-fields-pro first and then acf-pro-installer.

@mickaelzhang Can you post the (relvant) contents of your composer.json? Because when you specify that ACF requires acf-pro-installer it should be installed before. I also think it is not the case for @rolandp because the URL in the error message contains a version and key parameter which gets added by acf-pro-installer. 😕

@PhilippBaschke Thanks for getting back to me and pointing me in the direction of the key. I think the problem was a space before the actual key in my .env file. I've got it working now without changing anything else.
Would it be possible to return a difference error code for an invalid key? Something like a 403?

I'm glad that your Problem was solved. The better error code is good suggestion 😌 However, the error code comes from ACF directly and I can't really influence it 😞

I'm using acf-pro-installer on several projects and it's great thanks! However on one project I'm getting this error all of a sudden and configuration is the same on projects that work and this that don't work... I'm out of ideas... any suggestions what to troubleshoot? 🤔

Glad that you like it @slackday. 😌  Did you double check your key? It would be really helpful if you could share your composer.json file. ☺

Thanks for you answer, sorry I missed it by a week didn't see the notification.
Here is my composer.json file https://gist.github.com/slackday/1f25e7cd73de3f5e4d0795f1b2916f44

Hey @slackday.
Thank you for your composer.json file. When I run composer install with your composer.json and my ACF_PRO_KEY everything works as expected (like it does on all your other projects).
Have you tried opening the URL that the error message returns in the Browser?
[Composer\Downloader\TransportException]
The 'https://connect.advancedcustomfields.com/index.php?p=pro&a=download&t=5.5.11&k=WRONG_KEY' URL could not be accessed: HTTP/1.1 400 Bad Request

You could also try to copy the key and compare it letter by letter with your key.

The url from the error message doesn't contain a key. It tries to access same URL as in the composer.json. I have the key in .env on server but in this case it doesn't get added same way as on other servers/localhost.

That is super strange 😱 Would you mind running clearing the composer files, installing again and then show me the console output of the install step?
In your project directory
$ rm composer.lock
$ rm -rf vendor
$ composer clearcache
$ composer install

Super strange yes. I only had this problem on server when running deploy.
I ran the deploy again to get the composer error message for you but I ran into other issues with the deploy.
I upgraded wp-cli from v0.24.1 to v1.1.0 to fix the issue with the deploy and ran it again. Successfully. And the ACF key was added aswell.
My issue on this server seems to be resolved. Thanks for all your help and taking you time with this.

For anyone having issues right now with this. It looks like ACF has broken the "t=" part of their download URL. Tested from composer, browser and from the account dashboard of ACF's site.
BROKEN
https://connect.advancedcustomfields.com/index.php?p=pro&a=download&k={ACF_PRO_KEY}&t=5.5.14
WORKS
https://connect.advancedcustomfields.com/index.php?p=pro&a=download&k={ACF_PRO_KEY}

I am having this same problem. Any thoughts on how to fix? @PhilippBaschke can we remove the "&t=xxx" from the installer plugin.

I reached out to ACF and they've fixed the problem on their end. @timalbert can you try again? Perhaps try clearing you composer cache first?

Awesome @codilechasseur! Thank you so much for your effort! ☺

Hey @codilechasseur and @PhilippBaschke - I am still getting the same error even with composer cache cleared. Just performed a test and it fails with the t=5.3.9 parameter, but downloads as expected when removed. Have you been able to get this working successfully?

I also can not download because of a string conversion problem. My ACF Pro key has an is equal sign ('=') as last character. On my Mac it is working fine but on my linux server it gets transformed to '%3D'. How can I fix this issue?

@timalbert Sorry for the very late answer. I did not have time to try it until now but as you can see in the ACF versions lists (My Account > See all versions) 5.3.9 is not available. I guess you have to install a different version ☺
@PSahner Can you paste the error output here? That might help to find where the problem could come from. Thank you! 😌 The encoding is probably from here but I can't think of a reason why that should be different on Linux than on macOS. 😕

This is the error I get:
Installing advanced-custom-fields/advanced-custom-fields-pro (<IP_ADDRESS>): Downloading [Composer\Downloader\TransportException] The 'https://connect.advancedcustomf
ields.com/index.php?p=pro&a=download&t=<IP_ADDRESS>&k=[THE_KEY]' URL could not be accessed: HTTP/1.1 400 Bad Request

And the string for the key has a %3D instead of = at the end. So of I copy and paste the URL in my browser it does not work. But when I change the %3D to = manually it works.
I guess the urlencode($value) part is responsible for that problem where it changes the = character to hex code?

I opened a ticket at ACF. Maybe they have an idea what to do about it ☺

Any movement on this?