drawio-tools: Online CSV Viewer fails with 403 error

I am trying to import a CSV in drawio using the CSV tool tool (from drawio-tools)
I've tried with different hosts and csv urls (including the one suggested in the tool : https://jgraph.github.io/drawio-diagrams/diagrams/orgchart.csv) but It appears that clicking on the generated link always leads to a "Error loading file - Error 403" message in draw.io.
This seems to be caused by the proxy returning a 403 on csv files, as you can see:

403 : https://app.diagrams.net/proxy?url=https%3A%2F%2Fjgraph.github.io%2Fdrawio-diagrams%2Fdiagrams%2Forgchart.csv
success : https://app.diagrams.net/proxy?url=https%3A%2F%2Fjgraph.github.io%2Fdrawio-diagrams%2Findex.html

We'll replace #S with ?desc in the next release and update the csv link tool.

Great! Do you already know if ?desc be compatible with ?embed=1?
Right now it appears that #S is ignored in embed mode.

We'll add {action: 'load', descriptor: {format: 'csv', data: ...}} to support this in embed mode, eg.
{action: 'load', descriptor: {format: 'csv', data: '##\n' + '## Example CSV import. Use ## for comments and # for configuration. Paste CSV below.\n' + '## The following names are reserved and should not be used (or ignored):\n' + '## id, tooltip, placeholder(s), link and label (see below)\n' + '##\n' + '#\n' + '## Node label with placeholders and HTML.\n' + '## Default is \'%name_of_first_column%\'.\n' + '#\n' + '# label: %name%
%position%
Email\n' + '#\n' + '## Node style (placeholders are replaced once).\n' + '## Default is the current style for nodes.\n' + '#\n' + '# style: label;image=%image%;whiteSpace=wrap;html=1;rounded=1;fillColor=%fill%;strokeColor=%stroke%;\n' + '#\n' + '## Parent style for nodes with child nodes (placeholders are replaced once).\n' + '#\n' + '# parentstyle: swimlane;whiteSpace=wrap;html=1;childLayout=stackLayout;horizontal=1;horizontalStack=0;resizeParent=1;resizeLast=0;collapsible=1;\n' + '#\n' + '## Optional column name that contains a reference to a named style in styles.\n' + '## Default is the current style for nodes.\n' + '#\n' + '# stylename: -\n' + '#\n' + '## JSON for named styles of the form {"name": "style", "name": "style"} where style is a cell style with\n' + '## placeholders that are replaced once.\n' + '#\n' + '# styles: -\n' + '#\n' + '## JSON for variables in styles of the form {"name": "value", "name": "value"} where name is a string\n' + '## that will replace a placeholder in a style.\n' + '#\n' + '# vars: -\n' + '#\n' + '## Optional column name that contains a reference to a named label in labels.\n' + '## Default is the current label.\n' + '#\n' + '# labelname: -\n' + '#\n' + '## JSON for named labels of the form {"name": "label", "name": "label"} where label is a cell label with\n' + '## placeholders.\n' + '#\n' + '# labels: -\n' + '#\n' + '## Uses the given column name as the identity for cells (updates existing cells).\n' + '## Default is no identity (empty value or -).\n' + '#\n' + '# identity: -\n' + '#\n' + '## Uses the given column name as the parent reference for cells. Default is no parent (empty or -).\n' + '## The identity above is used for resolving the reference so it must be specified.\n' + '#\n' + '# parent: -\n' + '#\n' + '## Adds a prefix to the identity of cells to make sure they do not collide with existing cells (whose\n' + '## IDs are numbers from 0..n, sometimes with a GUID prefix in the context of realtime collaboration).\n' + '## Default is csvimport-.\n' + '#\n' + '# namespace: csvimport-\n' + '#\n' + '## Connections between rows ("from": source colum, "to": target column).\n' + '## Label, style and invert are optional. Defaults are \'\', current style and false.\n' + '## If placeholders are used in the style, they are replaced with data from the source.\n' + '## An optional placeholders can be set to target to use data from the target instead.\n' + '## In addition to label, an optional fromlabel and tolabel can be used to name the column\n' + '## that contains the text for the label in the edges source or target (invert ignored).\n' + '## The label is concatenated in the form fromlabel + label + tolabel if all are defined.\n' + '## Additional labels can be added by using an optional labels array with entries of the\n' + '## form {"label": string, "x": number, "y": number, "dx": number, "dy": number} where\n' + '## x is from -1 to 1 along the edge, y is orthogonal, and dx/dy are offsets in pixels.\n' + '## The target column may contain a comma-separated list of values.\n' + '## Multiple connect entries are allowed.\n' + '#\n' + '# connect: {"from": "manager", "to": "name", "invert": true, "label": "manages", \\\n' + '#          "style": "curved=1;endArrow=blockThin;endFill=1;fontSize=11;"}\n' + '# connect: {"from": "refs", "to": "id", "style": "curved=1;fontSize=11;"}\n' + '#\n' + '## Node x-coordinate. Possible value is a column name. Default is empty. Layouts will\n' + '## override this value.\n' + '#\n' + '# left: \n' + '#\n' + '## Node y-coordinate. Possible value is a column name. Default is empty. Layouts will\n' + '## override this value.\n' + '#\n' + '# top: \n' + '#\n' + '## Node width. Possible value is a number (in px), auto or an @ sign followed by a column\n' + '## name that contains the value for the width. Default is auto.\n' + '#\n' + '# width: auto\n' + '#\n' + '## Node height. Possible value is a number (in px), auto or an @ sign followed by a column\n' + '## name that contains the value for the height. Default is auto.\n' + '#\n' + '# height: auto\n' + '#\n' + '## Padding for autosize. Default is 0.\n' + '#\n' + '# padding: -12\n' + '#\n' + '## Comma-separated list of ignored columns for metadata. (These can be\n' + '## used for connections and styles but will not be added as metadata.)\n' + '#\n' + '# ignore: id,image,fill,stroke,refs,manager\n' + '#\n' + '## Column to be renamed to link attribute (used as link).\n' + '#\n' + '# link: url\n' + '#\n' + '## Spacing between nodes. Default is 40.\n' + '#\n' + '# nodespacing: 40\n' + '#\n' + '## Spacing between levels of hierarchical layouts. Default is 100.\n' + '#\n' + '# levelspacing: 100\n' + '#\n' + '## Spacing between parallel edges. Default is 40. Use 0 to disable.\n' + '#\n' + '# edgespacing: 40\n' + '#\n' + '## Name or JSON of layout. Possible values are auto, none, verticaltree, horizontaltree,\n' + '## verticalflow, horizontalflow, organic, circle or a JSON string as used in Layout, Apply.\n' + '## Default is auto.\n' + '#\n' + '# layout: auto\n' + '#\n' + '## ---- CSV below this line. First line are column names. ----\n' + 'name,position,id,location,manager,email,fill,stroke,refs,url,image\n' + 'Tessa Miller,CFO,emi,Office 1,,me@example.com,#dae8fc,#6c8ebf,,https://www.draw.io,https://cdn3.iconfinder.com/data/icons/user-avatars-1/512/users-3-128.png\n' + 'Edward Morrison,Brand Manager,emo,Office 2,Tessa Miller,me@example.com,#d5e8d4,#82b366,,https://www.draw.io,https://cdn3.iconfinder.com/data/icons/user-avatars-1/512/users-10-3-128.png\n' + 'Alison Donovan,System Admin,rdo,Office 3,Tessa Miller,me@example.com,#d5e8d4,#82b366,"emo,tva",https://www.draw.io,https://cdn3.iconfinder.com/data/icons/user-avatars-1/512/users-2-128.png\n' + 'Evan Valet,HR Director,tva,Office 4,Tessa Miller,me@example.com,#d5e8d4,#82b366,,https://www.draw.io,https://cdn3.iconfinder.com/data/icons/user-avatars-1/512/users-9-2-128.png\n'}}

Sounds good! Thanks!

This request would be great for us. Do you know more or less when it should be released ?
We are hesitating between spending time on trying to convert from CSV to XML based on your sources or just wait for the release.
Thanks a lot !

Hi @alderg, any pointer on when the new version should be coming out?
Thanks 👍


Since Chrome 71, chrome://net-internals/#events is removed.

Chrome 71 doesn't use chrome://net-internals/#events any more, insted it uses chrome://net-export, so this extension no longer compatible with latest chrome and has to be updated.

v0.0.3 fix it. Users should get this update soon through Chrome extension auto update.
Thanks for your report!


Creating analysis-ready data with xradar with QVPs (xradar)

Discussion of all topics related to QVPs

@openradar/erad2024-team , @wolfidan
I took the liberty to open this issue to discuss the QVP exercise. Could we start by identifying the contributors to this project? Please add your name and contribution as a reply to this comment:

@wolfidan , @jfigui : We will provide Pyrad config files to plot QVPs

Thanks, Jordi, for creating this issue. I will work on analysis-ready data; one application could be QVPs or QPE. The idea is to show the advantages of having data in CFRadial2/FM301.

By the way, @mgrover1 radar data has already been converted into ARCO format. How can I upload the data to the bucket?

Analysis-ready data could be conditioned to this issue since we might need to pass credentials. Let me know your thoughts @kmuehlbauer @mgrover1

@aladinor - the zarr file is now available on the bucket!
https://js2.jetstream-cloud.org:8001/pythia/radar/erad2024/zarr_radar/erad_2024.zarr

AFAICT this is up and running.


Add list of VM sizes to AKS UI proxy

Issue: https://github.com/rancher/rancher/issues/32587

Rebased


Wallpaper on multiple monitors being centered between them.

Describe the bug
If you have, in my case, two monitors and selects a wallpaper, the wallpaper is not one wallpaper per monitor but is centered split in half and stretches between them.
Steps to reproduce

Kill Windows Explorer process
Use Task Manager to open Cairo
Select a wallpaper using Cairo

Expected behavior
Duplicated wallpaper for each monitor, or and option for it.
Request: Ability to select an unique wallpaper for each monitor if possible.
System information:

Windows version:  11 10.0.22621
Cairo version: <IP_ADDRESS>259
32-bit or 64-bit: 64
In Cairo Settings > Advanced, is Cairo set as shell?: No

Confirmed, I have the same behavior.


Fix tf in bleurt takes all the memory (--amend)

Sorry that I accidentally missed the latter d in def :scream::scream::scream:. Here is the emergent fix patch.

I just pushed a commit which fixes the typo. I should have looked closer when merging, so this is my bad. This PR looks like it is for #8, so we should merge it separately. I should probably fix the CI/CD as well, but as you mentioned, we're having trouble with the LFS assets, so I have to figure out how to get that sorted before the next few steps. I'll take care of this tomorrow, when it's not a holiday!


Poll Command

Create a poll command using embeds.

Thanks for the PR, made some code comments. Also, do you plan to accept poll questions or answers with more than one word?

Yeah, all you have to do is wrap it in quotes:


bugfix: pgsql escape logic error.

Ⅰ. Describe what this PR did
pgsql add escape check logic error, ColumnUtils add escape with scheme.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2900 into develop will increase coverage by 0.04%.
The diff coverage is 91.11%.

@@              Coverage Diff              @@
##             develop    #2900      +/-   ##
=============================================
+ Coverage      50.09%   50.13%   +0.04%
- Complexity      3018     3031      +13
=============================================
  Files            597      597
  Lines          19292    19304      +12
  Branches        2347     2310      -37
=============================================
+ Hits            9665     9679      +14
+ Misses          8660     8640      -20
- Partials         967      985      +18

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/io/seata/rm/datasource/ColumnUtils.java
86.59% <90.90%> (+6.59%)
27.00 <0.00> (+7.00)

...o/postgresql/keyword/PostgresqlKeywordChecker.java
97.00% <100.00%> (+6.00%)
10.00 <0.00> (+4.00)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

... and 29 more


Set up dependabot or a similar tool to automatically check for dependency updates

As for now, I run ./gradlew dependencyUpdates once in a while and update the versions manually... let's make it more automated 🤖
This probably won't work for updates of IntelliJ itself (see https://github.com/VirtusLab/git-machete-intellij-plugin/issues/794), but probably will work for other deps (libraries and Gradle plugins) 📦

Actually... with Gradle version catalog (PR #938) in place, lemme close this one and open another one...

...https://github.com/VirtusLab/git-machete-intellij-plugin/issues/940


feat: Add shutter sound control

What

This PR Adds shutter sound control

Changes

This PR changes Add disableShutterSound option to takePhoto function (default: false)

Tested on

iPhone 12 Pro, iOS 15.4.1
Android 10 API 29 (Galaxy Note9)

Related issues

Fixes #311, #1044

Hey, thanks for your PR!
As mentioned in the previous PR (https://github.com/mrousavy/react-native-vision-camera/pull/1072), I'd rather have a prop called enableShutterSound that is true by default, rather than using reverse logic here. Makes more sense to me.
Also, I think this can be implemented on Android as well, so would be great if we could add that and don't have that limited for iOS only?

@mrousavy Thanks for the suggestion!

Hey @imjihun! Thanks for your edit, on Android it looks good.
But as I said in my first review, I'd want the thing called enableShutterSound instead of disableShutterSound. Then it is false by default, and the user can enable it if he wants. This makes more sense to me than the reverse logic. What do you think?

Hey @imjihun! Thanks for your edit, on Android it looks good.
But as I said in my first review, I'd want the thing called enableShutterSound instead of disableShutterSound. Then it is false by default, and the user can enable it if he wants. This makes more sense to me than the reverse logic. What do you think?

Personally, I think having the shutter sound on should be the default behavior. I have a need to disable the shutter sound but I would imagine it's not that common?

I think we should have it disabled by default, that's the default behaviour on both native iOS and native Android :)

Ah my bad I didn't know that. Makes sense.

@jeffBoom Closely following your PR, looks well done!

When will this PR be merged into main?

Thank you for the PR, I'm glad to see this. Are there plans to merge, @mrousavy? Anything needed from the community to get it over the finish line?

Hey thanks for your PR but this is fixed in V3!


Refresh token scenario - Single and A/P OCP clusters load test

First results on Refresh Tokens (CPU overhead, use them together with login/logout to stress the shared Infinispan resources, Ratio 1:10 login/refresh)

Single OCP Cluster with internal infinispan and external Aurora DB Refresh token tests with 300 users-per-sec:
./benchmark.sh eu-central-1 --scenario=keycloak.scenario.authentication.AuthorizationCode \
 --server-url=https://keycloak-kakella-keycloak.apps.gh-mkanis.jfth.p1.openshiftapps.com \
--users-per-sec=300 --logout-percentage=100 --measurement=600 --realm-name=realm-0 \
--users-per-realm=10000 --ramp-up=180 --refresh-token-count=5 --refresh-token-period=5

Gatling results:

Resource Utilization:

Load shedding observed with logs similar to this:
{"timestamp":"2023-08-14T20:51:12.303Z","sequence":11711,"loggerClassName":"org.jboss.logging.Logger","loggerName":"io.quarkus.vertx.http.runtime.QuarkusErrorHandler","level":"ERROR","message":"HTTP Request to /metrics failed, error id: 0d83d50a-c326-4c02-9286-6432593b1176-4","threadName":"vert.x-eventloop-thread-1","threadId":82,"mdc":{"trace_flags":"00","trace_id":"ba4dd3e838c294e3a88499af48b50701","span_id":"7ee6379f9dae297c"},"ndc":"","hostName":"keycloak-0","processName":"QuarkusEntryPoint","processId":1,"exception":{"refId":1,"exceptionType":"java.util.concurrent.RejectedExecutionException","message":null,"frames":[{"class":"org.jboss.threads.RejectingExecutor","method":"execute","line":38},{"class":"org.jboss.threads.EnhancedQueueExecutor","method":"rejectQueueFull","line":2127},{"class":"org.jboss.threads.EnhancedQueueExecutor","method":"execute","line":821},{"class":"io.vertx.core.impl.ContextBase","method":"executeBlocking","line":150},{"class":"io.vertx.core.impl.DuplicatedContext","method":"executeBlocking","line":138},{"class":"io.vertx.core.impl.ContextInternal","method":"executeBlocking","line":153},{"class":"io.vertx.core.Vertx","method":"executeBlocking","line":597},{"class":"io.vertx.ext.web.impl.BlockingHandlerDecorator","method":"handle","line":47},{"class":"io.vertx.ext.web.impl.BlockingHandlerDecorator","method":"handle","line":33},{"class":"io.opentelemetry.javaagent.instrumentation.vertx.RoutingContextHandlerWrapper","method":"handle","line":36},{"class":"io.opentelemetry.javaagent.instrumentation.vertx.RoutingContextHandlerWrapper","method":"handle","line":21},{"class":"io.vertx.ext.web.impl.RouteState","method":"handleContext","line":1284}

Single OCP Cluster with internal infinispan and external Aurora DB Refresh token tests with

200 users-per-sec
1:5 - login:refreshToken request ratio

./benchmark.sh eu-central-1 --scenario=keycloak.scenario.authentication.AuthorizationCode --server-url=https://keycloak-kakella-keycloak.apps.gh-mkanis.jfth.p1.openshiftapps.com --users-per-sec=200 --logout-percentage=100 --measurement=600 --realm-name=realm-0 --users-per-realm=10000 --ramp-up=180 --refresh-token-count=5 --refresh-token-period=5

Gatling results:

Resource utilization:

@kami619 - thank you for sharing these results. Could you please also share the dashboard of the Infinispan statistics? I would be most interested in the JGroup thread statistics. Thanks!

@kami619 - thank you for sharing these results. Could you please also share the dashboard of the Infinispan statistics? I would be most interested in the JGroup thread statistics. Thanks!

sure thing, I can add the screenshots for ISPN dashboards too

Active-Passive refresh token tests were performed with both 1:5 (login:RefreshToken) ratio as well as 1:10 (login:RefreshToken) ratio., in both cases we observed for the cluster with 6 Keycloak Pods and 8 CPU limit for each pod and with external ISPN server and external Aurora PostgreSQL db and liveness and readiness probes disabled,

The system scales well up until an overall limit of 1500 global req/s throughput and starts to load shed with 503 server side exception handling.
When OTEL is enabled, the regular Vert.x metrics endpoint gets rejected due to connection pool congestion with overall saturation and results in some errors on the server side.
We don't see aggressive CPU throttling due to the load shedding behavior of Keycloak however we see response times go above 2000 ms during these periods as shown in the below screenshot of the SLO dashboard during a load run when the load shedding happened.
Increase in refresh-token-count is proportional to the time at which the load shedding behavior happens and it seems to happen earlier as the count of refresh token's increase.

8/23 update was to repeat the tests that show slowness in response times once we start to see load shedding and upon confirming the behavior log a separate ticket to investigate the root cause for the slowness in response times.

Completion criteria for this issue:

Understand high response times, theory: When request queue is full (1000 entries), newly queued requests need to wait for their turn to complete
Have a CPU calculation base for refresh tokens

We found the 175 users-per-sec to be the optimum supported run rate for a 6 pod cluster with 8 CPU limit per pod which translates to 126 refresh tokens per vCPU or 378 refreh tokens per second per vCPU in a 3 pod clusters to match the calculations here.

full results for the 175 users-per-sec can be found here.

Beyond that we start to hit the limit of the system and at about 200 users-per-sec we were able to reproduce the requests being queued situation and that results in higher response times and load shedding starts to happen on the Keycloak server side.

Overall I think we achieved the objective of the ticket, if it lines up with the expectation we can close this one.
We have created subsequent tickets to,

https://github.com/keycloak/keycloak-benchmark/issues/516
https://github.com/keycloak/keycloak-benchmark/issues/517
https://github.com/keycloak/keycloak-benchmark/issues/518


Fix Tox installing Terra from PyPI

Summary
Solves the issue from https://github.com/Qiskit/qiskit-terra/pull/9477.
As explained in the code's comment, this results in pip resolving Aer's dependency on Terra via the local installation, not from PyPI.
Details and comments
How did I test this works?

I changed requirements.txt to set python-dateutil==2.8.2, then added to deps python-dateutil==2.8.1. That should cause a conflict: the deps stage will install 2.8.1, then when installing Terra locally, we'll try using 2.8.2. But due to Tox wonkiness not fixed until Tox 4.4, tox silently allows for that conflict to happen.
But, when adding -e . to deps as well, pip fails due to the conflict. This indicates that Pip indeed is installing from the local installation during the deps stage.

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 increased (+0.001%) to 85.361%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
68052

Relevant Lines:
79723

💛  - Coveralls

No longer blocked / ready for review :)


Add extra terms

Coverage remained the same at 97.944% when pulling b1008777a7a9add25842a850fe9749859efe1672 on zo:extra-terms into 1ab0381d187b04d9b0a15224c595fe0b327b6dc6 on Jason3S:master.


Adjust graphviz options for better layout with plot_gate_map()

Summary
This commit fixes an issue with the visualizations of some backends/coupling maps. The default neato settings worked well in most cases but for some graphs it was not generating a good layout. This tweaks the settings a bit so that layouts work with more classes of graphs.
Details and comments

ibm_fez

Based on that image I'm wondering if we want to scale up the fontsize a bit?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
22 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.02%) to 89.855%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/visualization/gate_map.py
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

qiskit/visualization/gate_map.py
1
6.95%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
2
88.35%

crates/qasm2/src/lex.rs
6
91.35%

crates/qasm2/src/parse.rs
12
96.23%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
65728

Relevant Lines:
73149

💛  - Coveralls

ibm_quebec

That second example looks like we should condition:
https://github.com/Qiskit/qiskit/pull/12770/files#diff-ff70fb816cde3f453d870891f47ede8245855a9e156228201a53fd613ad00659R1096-R1100
on if not qubit_coordinates and only use these graph attributes if we're doing an algorithmic layout.

In the last change, I changed to prism and I got the following results:
ibm_fez

ibm_cusconazca

ibm_torino

ibm_kyiv

If you are happy when them, I will update the image references and merge.

If you are happy when them, I will update the image references and merge.

Those look fine to me, the ibm_torino one is maybe a bit less pretty than ibm_fez and ibm_cusconazca but it looks fine. So let's just go with this.

It looks like the font size still needs some tweaking:

Here, another round of changes:


Any chance of getting Signal integrated?

Service
Signal: Signal Desktop
Link to your repository

Additional Information

+1
Signal is the one service that I rely on daily that I can't currently get in Franz.

+1

+1

+1

Ditto, although I suspect this is not trivial.  The other services (at least, all the ones I use) provide web clients, and Franz is just a great wrapper to that, using Electron.   Signal, however, doesn't provide a web client, and it's standalone client is an Electron app in itself.
If I'm mistaking and this is more trivial than I thought, then add me to the list of people who would love to have Signal integrated.

+1

We would need to make a web version out of the electron app and integrate that one into Franz. The code is open source and can be modified and distributed as we want.

+1

+1

+1

+1

+1

+1

+1

+1

+1

pretty please!!! only msg app I miss on Franz.. otherwise: franz is just the best and most important app I install on any computer straight away...
there seems to be: Signal Desktop is an Electron application that links with Signal
https://github.com/signalapp/Signal-Desktop

Would also really appreciate if signal would be useable via franz.

I would like this also. They already have an electron app. But I don't think they like to have alternate clients for their platform. It would be great.

same here, would love it.

+1

Hi, I would also love to see signal within Franz and I see there have been a lot of requests and duplicates for signal... Will it ever be included and is there any place to see the progress status for it? Thanks

+1

Building signal into franz would destroy the security of signal. It really shouldn't be done and it's hard to understate the non-trivial nature of the task. It's literally building an app within an app.


Another Pull Request with style changes to prompting bar with tooltips and Content Mode

Thanks for your feedback, I've done another pull request with the most recent version.
I see you managed to fix a lot of the issues I was looking at doing and the split componentry is such a huge help!
Cheers
Will

Thanks, I've reviewed this new version.
Did you know your app is live already for anyone to use: link
The quality of this patch has improved already, and glad you're figuring out the workflow to stay up to date on the latest changes. However, a couple of notes:

somehow the commit adds +131 lines of code and removes −129. this is quite large, and would be overwritten by future code that I write. This is because the code is formatted by "Prettier" (https://prettier.io/) on my end, while (VSCode?) on your end applies different spacing and all. This translates in a lot of code being modified, and the patch being harder to maintain because the 2 systems would keep changing the code back and forth. Is there a "Prettier" plugin you can install on your IDE? After installing, load the rules in the ".prettierrc" file, and the systems will be on the same page
Change 1. UI: I like the looks actually, but also see the value in the existing UI for developers / people that need more space for the edit box (to drop files in, etc). I'm thinking on whether I should make it switchable, with an ON/OFF "ChatGPT-like UI" option.
Change 2. Prompt: I'm thinking about how to enable people to publish their prompts ("Content Mode") and tracking this in #35. I like your change, but cannot formally merge it because of the "undetectable" implications to the brand :)

Here's what we should do: you can configure your IDE with Prettier so systems always minimize changes. I will not merge the pull request here (too many code changes, and there's a reason for the current UI to co-exist), but your fork of the repository has your UI and University-focused changes.
You should maintain and publish your version whi (I use Vercel, free plan, and you can give it a domain), which focuses on students and UI, and I will continue modularizing and cleaning the base code, and adding core features. If you have core features and improvements, I'll merge them into the master code base, otherwise, you can maintain them. Sounds good?

Ah, makes sense.
Yeah I was looking at it like "I didn't change that much". Turns out it is the formatter. I've used Prettier before but currently using Eslint so that's where that issue lies. Will change the formatter on my end for this project to keep consistency.
On the note of the toggle for the bigger text box, that sounds like a small task so I'll focus on adding that to my version so there's options in the UI.
My 2 cents on the Community Templates. The ideal goal is to have a marketplace style area that has different system messages that can be easily integrated for ultimate customization, however that could take a more intense hosting solution with the added database. I might start a small side project marketplace that prompts can be added to by users and voted up and down, and then I'll submit a pull request with an integration in the future that would allow people to select a prompt or upload a prompt.
So yup, I'll continue on my own fork for now and will definitely be in touch soon as your and my version progress.

i am super interested in this will
On Fri, Mar 31, 2023 at 1:17 AM Will Scott-Rodriguez <
@.***> wrote:

Ah, makes sense.
Yeah I was looking at it like "I didn't change that much". Turns out it is
the formatter. I've used Prettier before but currently using Eslint so
that's where that issue lies. Will change the formatter on my end for this
project to keep consistency.
On the note of the toggle for the bigger text box, that sounds like a
small task so I'll focus on adding that to my version so there's options in
the UI.
My 2 cents on the Community Templates. The ideal goal is to have a
marketplace style area that has different system messages that can be
easily integrated for ultimate customization, however that could take a
more intense hosting solution with the added database. I might start a
small side project marketplace that prompts can be added to by users and
voted up and down, and then I'll submit a pull request with an integration
in the future that would allow people to select a prompt or upload a prompt.
So yup, I'll continue on my own fork for now and will definitely be in
touch soon as your and my version progress.
—
Reply to this email directly, view it on GitHub
https://github.com/enricoros/nextjs-chatgpt-app/pull/37#issuecomment-1491309868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABO2LTFM6QUTVNPF4HNQ7BDW6ZSFXANCNFSM6AAAAAAWOAR7MQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks Will. Before I park this pull request: are you thinking of maintaining and hosting (e.g., Vercel) your fork?
If so, I can add a link to your fork from the main readme, to send University users your way :) Lmk.
Sounds great for the prompts, your feedback is great for #35 .

Hey
Yup, I'll self host this fork with a uni mode initially.
Gotta make some other modifications over this weekend then I'll flick you a link.


Request Example: LLM Compiler in LangGraph JS

Hey there, is there a plan to port the LLM Compiler example also to LangGraph JS?

Unlikely to be an immediate priority unless someone in the community would like to take a stab at it.


Add Groovy 4.0.8 and 3.0.14 to test coverage

Changelog for Groovy 4.0.8  [Released: 2023-01-19]
Changelog for Groovy 3.0.14  [Released: 2022-12-21]

It seems Groovy 3.0.14 still does not support Java 20, so this issue should be fixed after

https://github.com/gradle/gradle/pull/24019

Thank you for your interest in Gradle!
This is in the backlog of the relevant team and is prioritized by them.


BUG - Mock Server for oData V4

Hello,
I've detected the issue with MockServer for oData v4 (CAP) for a SAPUI5 App.
"devDependencies": {
"@sap-ux/ui5-middleware-fe-mockserver": "^2.2.6"
},
"ui5": {
"dependencies": [
"@sap/ui5-builder-webide-extension",
"@sap-ux/ui5-middleware-fe-mockserver",
]
}
I have an OData V4 (CAP) service call with the path: /Header?$filter=formats/any(f:f/FORMAT_ID eq 1) and not items/any().
This request works fine with real OData call and without a filter option without "not", but when I try to trigger it via a Mock server there is an error: Cannot read properties of undefined (reading 'operator')
Error stack:
info server:ux-fe-mockserver ODataRequest: GET /Header?$filter=formats/any(f:f/STORE_FORMAT_ID%20eq%201)%20and%20not%20items/any()
ERR! server:ux-fe-mockserver TypeError: Cannot read properties of undefined (reading 'operator')
ERR! server:ux-fe-mockserver at FilterParser.ALT (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:223:50)
ERR! server:ux-fe-mockserver at RecognizerEngine.orInternal (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:400:42)
ERR! server:ux-fe-mockserver at RecognizerApi.OR (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_api.js:133:21)
ERR! server:ux-fe-mockserver at FilterParser. (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:208:29)
ERR! server:ux-fe-mockserver at FilterParser.invokeRuleWithTry (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:118:33)
ERR! server:ux-fe-mockserver at FilterParser.wrappedGrammarRule [as boolCommonExpr] (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:131:38)
ERR! server:ux-fe-mockserver at RecognizerEngine.subruleInternal (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:422:37)
ERR! server:ux-fe-mockserver at RecognizerApi.SUBRULE2 (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_api.js:79:21)
ERR! server:ux-fe-mockserver at FilterParser. (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:266:38)
ERR! server:ux-fe-mockserver at RecognizerEngine.optionInternalLogic (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:212:27)
ERR! server:ux-fe-mockserver TypeError: Cannot read properties of undefined (reading 'operator')
ERR! server:ux-fe-mockserver at FilterParser.ALT (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:223:50)
ERR! server:ux-fe-mockserver at RecognizerEngine.orInternal (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:400:42)
ERR! server:ux-fe-mockserver at RecognizerApi.OR (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_api.js:133:21)
ERR! server:ux-fe-mockserver at FilterParser. (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:208:29)
ERR! server:ux-fe-mockserver at FilterParser.invokeRuleWithTry (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:118:33)
ERR! server:ux-fe-mockserver at FilterParser.wrappedGrammarRule [as boolCommonExpr] (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:131:38)
ERR! server:ux-fe-mockserver at RecognizerEngine.subruleInternal (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:422:37)
ERR! server:ux-fe-mockserver at RecognizerApi.SUBRULE2 (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_api.js:79:21)
ERR! server:ux-fe-mockserver at FilterParser. (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/@sap-ux/fe-mockserver-core/dist/request/filterParser.js:266:38)
ERR! server:ux-fe-mockserver at RecognizerEngine.optionInternalLogic (/home/user/projects/shelf-survey/app/shelfsurvey/node_modules/chevrotain/lib/src/parse/parser/traits/recognizer_engine.js:212:27)

FYI @nlunets


Broken debug build

Describe the bug
Performing a debug build after a release build followed by a make clean fails to build anything.
Reproduction steps
aconway@w3-hs1-112016:~/splinterdb$ make
...
...
...
aconway@w3-hs1-112016:~/splinterdb$ make clean
rm -rf build
uname -a
Linux w3-hs1-112016 5.4.0-97-generic #110-Ubuntu SMP Thu Jan 13 18:22:13 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
clang --version
clang version 10.0.0-4ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
aconway@w3-hs1-112016:~/splinterdb$ make BUILD_MODE=debug
Checking for mismatched config...No mismatched config found
aconway@w3-hs1-112016:~/splinterdb$

### Expected behavior

A debug build.

### Additional context

_No response_

Closing, since PR #388 fixed this.


feat: add support for custom keybindings and editor actions in the REPL

Towards #2647.
Description

What is the purpose of this pull request?

This pull request:

[x] adds editor actions
[x] adds support for custom keybindings
[ ] make existing actions configurable
[ ] adds settings, APIs and update docs

Related Issues

Does this pull request have any related issues?

This pull request:

partially resolves #2647

Questions

Any questions for reviewers of this pull request?

For the sake of user experience, I decided the key object can have 4 properties: name, ctrl, shift, and meta. In actuality, a readline key object also has properties like sequence and code, which mainly contain the escape sequence of the keypress.
In most keypresses, the name, ctrl, shift, and meta properties are correctly emitted and are enough to determine the action we have to trigger. But in some cases, for instance, CTRL+/, instead of emitting an event with { 'name': '/', 'ctrl': true }, it emits a sequence { 'sequence': '\x1f' }. So if a user has keybindings with CTRL+/ as the key, it won't work, as such a keypress is never emitted.
Now such cases might be limited, as readline only uses the sequence prop to identify two keypress actions, namely undo and redo.
Maybe we can handle these two special cases manually in code? Not sure if a robust solution exists.
Ofcourse, we can also punt the entire responsibility of giving the readline-ready key object, onto the user, but that's not user friendly IMO.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Another solution to the problem in OP is, we can expect the user to give us the ASCII sequence of the keypress instead.

Every keypress will always have a sequence prop unlike name, ctrl, shift and meta.
Now this is not user-friendly as the user is expected to know the control sequence of the keypress they are wanting to assign.
So, another approach would be to maintain a table mapping all possible keybindings to their corresponding emitted control sequences. Something like:
const CONTROL_SEQUENCES = {
    '\x01': 'Ctrl+A',
    '\x02': 'Ctrl+B',
    '\x03': 'Ctrl+C',
    '\x04': 'Ctrl+D',
    '\x05': 'Ctrl+E',
    '\x06': 'Ctrl+F',
    '\x07': 'Ctrl+G',
    '\x08': 'Ctrl+H',
    '\x09': 'Ctrl+I',
    '\x0A': 'Ctrl+J',
    '\x0B': 'Ctrl+K',
    '\x0C': 'Ctrl+L',
    '\x0D': 'Ctrl+M',
    '\x0E': 'Ctrl+N',
    '\x0F': 'Ctrl+O',
    '\x10': 'Ctrl+P',
    '\x11': 'Ctrl+Q',
    '\x12': 'Ctrl+R',
    '\x13': 'Ctrl+S',
    '\x14': 'Ctrl+T',
    '\x15': 'Ctrl+U',
    '\x16': 'Ctrl+V',
    '\x17': 'Ctrl+W',
    '\x18': 'Ctrl+X',
    '\x19': 'Ctrl+Y',
    '\x1A': 'Ctrl+Z'
};

But not sure if control sequences and their corresponding keys are standardized across emulators..?
How should we proceed?

https://github.com/nodejs/node/blob/main/lib/internal/readline/utils.js#L59
readline parses some keys based on xterm and gnome emulator behaviors (I guess). Maybe we can enforce xterm standards to parse keys? I suppose most combinations might emit standardized sequences..(?)

@Snehil-Shah Not requiring providing a sequence was the right call, I think, given that asking users to know or look up the control sequences for keybindings they want is undesirable. The current approach of a key object with Boolean modifier properties seems good; we could consider renaming meta to alt, as that the latter be understood by more people maybe?
Enforcing xterm standards for key parsing could be feasible. xterm is widely used and AFAIK many terminal emulators aim for compatibility with xterm, so this sounds worth investigating.
I also do think that it would be a pragmatic choice to handle the undo and redo special cases separately.

Now if we decide to go ahead and enforce xterm control sequences, we also have the flexibility to change the input format the user can give. So we can have something like the string "CTRL-E" as input, and map this to the corresponding xterm control sequence..

Went ahead and added parsing of unrecognized keypresses. Fortunately, there were not many cases to cover, the above problems were mainly around keys involving symbols.
@Planeshifter I think it would be better to stick to meta as the internal readline key object, also uses it. This makes it easier to map keybindings. And julia also uses meta instead of alt in their naming so we're not alone.

@Planeshifter Makes sense..

we could use the base package equivalents

We should go ahead and make those changes. Too easy to forget to do them later.

Maybe we can handle these two special cases manually in code? Not sure if a robust solution exists

@Snehil-Shah I'll defer to your judgement on that.

@Snehil-Shah Gentle ping regarding resolving the above comments.

@kgryte cleaned up some stuff. Regarding keybindings overlapping with system preferences, I don't see a way out.

I don't see a way out

@Snehil-Shah Maybe the way out is just clearly documenting that overlap may happen and that there is likely also going to be some platform variability, so MMV.

@kgryte should I update the docs in this PR itself? I'll raise another PR to add keybindings to existing actions..

@Snehil-Shah Let's punt updating the docs, and let the keybindings fly under the radar for the time being while we test out, and then we can add to docs in a follow-up PR.


DOF Far/Near Blur causes artifacts when glow is enabled

Operating system or device - Godot version:
Windows 7 64bit
Godot 3.0 built from July 18th source (although I've seen this in earlier versions)
Running on a GeForce GTX 760
Issue description:
When you have both DOF Far Blur (or Near Blur) and Glow enabled, the glow texture becomes corrupted, as seen in the gif below:

Steps to reproduce:
Just enable DOF Far Blur and Glow at the same time

for me on 4fe6b2a6744c4852cd3fdf0bc7a3437f8a16645f with those settings on, on linux and GeForce1060 it looks better but also not good:

Just wondering, is glow_bicubic_upscale enabled? I've seen a similar issue with this option enabled.

@kubecz3k this is a separate issue I think. The values need capping otherwise it wraps around and causes the strange colours. If you change the glow mode to Additive, it works, it's just the other modes that are broken at the moment

@Noshyaar yes, but if I disable it, I don't get the glow effect at all (it used to work when I tested it many months ago)

@kubecz3k That's #9106 I think.

Likely fixed by f5277e3. Let us know if it's not the case.


updated landing page and tools with consistent styling and better hover effects (UI LAYOUT IS ALSO FIXED)

Please make sure you do the following before filing your PR:

[✔ ] Provide a video or screenshots of any visual changes made
[✔ ] Run pnpm run check and make sure everything passes

https://github.com/user-attachments/assets/c4aa938c-96af-4be4-a731-d17df0798b61

@t3dotgg  I have tried to improve the UI a bit. Can you review it and merge the PR if you like it?


Terra 0.15.0 compatibility

When terra releases it's 0.15.0 we'll need to have an ignis release to update with some breaking changes terra is making in the release. We'll either have to backport the appropriate changes for an ignis 0.3.4 or push the current state of master out as 0.4.0.
For example, https://github.com/Qiskit/qiskit-ignis/issues/451 and https://github.com/Qiskit/qiskit-ignis/pull/400 are 2 examples where this is needed.

@nkanazawa1989 Can you look into updating pulse compatibility, and @mtreinish can you look into fixing any remaining issues with Marshmallow removal. Are there any other areas we are expecting failures?

Currently we have pulse related components in calibration and discriminator modules. I checked both and I think they are ready for the release (the calibration module is written with the basic instructions and no sample pulse dependency, and the discriminator module just calls a schedule instead of metadata and no actual pulse program generation).


runtime/race: second_test_passes failures

What version of Go are you using (go version)?
Commit 1abe9c1c73739786bb927342c4072e229affea8f
What operating system and processor architecture are you using (go env)?
Detected on the freebsd-amd64-12_2 and freebsd-amd64-12_3 builders, but I don't think it's specific to those.
What did you do?
I ran all.bash 1,810 times between the freebsd-amd64-12_2 and freebsd-amd64-12_3 builders.
What did you expect to see?
Everything passes.
What did you see instead?
I got the following failure 8 times (0.0044% failure rate). If I'm reading this correctly, the problem is just that it expects TestFail to happen in < 1 second and it takes more than 1 second (in this example 1.54s, but I have examples ranging from 1.09s to 3.00s).
##### Testing race detector
--- FAIL: TestOutput (11.27s)
    output_test.go:30: test midstack_inlining_traceback runs only on linux, skipping:
    output_test.go:30: test external_cgo_thread runs only on linux, skipping:
    output_test.go:83: failed test case second_test_passes, expect:

        ==================
        --- FAIL: TestFail \(0...s\)
        .*testing.go:.*: race detected during execution of test
        FAIL
        got:
        ==================
        WARNING: DATA RACE
        Write at 0x00c00001a2d8 by goroutine 8:
          command-line-arguments_test.TestFail.func1()
              /tmp/workdir/tmp/TestOutput4792514/008/main_test.go:9 +0x39

        Previous write at 0x00c00001a2d8 by goroutine 7:
          command-line-arguments_test.TestFail()
              /tmp/workdir/tmp/TestOutput4792514/008/main_test.go:12 +0x118
          testing.tRunner()
              /tmp/workdir/go/src/testing/testing.go:1440 +0x213
          testing.(*T).Run.func1()
              /tmp/workdir/go/src/testing/testing.go:1487 +0x47

        Goroutine 8 (running) created at:
          command-line-arguments_test.TestFail()
              /tmp/workdir/tmp/TestOutput4792514/008/main_test.go:8 +0x10e
          testing.tRunner()
              /tmp/workdir/go/src/testing/testing.go:1440 +0x213
          testing.(*T).Run.func1()
              /tmp/workdir/go/src/testing/testing.go:1487 +0x47

        Goroutine 7 (running) created at:
          testing.(*T).Run()
              /tmp/workdir/go/src/testing/testing.go:1487 +0x724
          testing.runTests.func1()
              /tmp/workdir/go/src/testing/testing.go:1840 +0x99
          testing.tRunner()
              /tmp/workdir/go/src/testing/testing.go:1440 +0x213
          testing.runTests()
              /tmp/workdir/go/src/testing/testing.go:1838 +0x7e4
          testing.(*M).Run()
              /tmp/workdir/go/src/testing/testing.go:1720 +0xa71
          main.main()
              _testmain.go:49 +0x2e4
        ==================
        --- FAIL: TestFail (1.54s)
            testing.go:1313: race detected during execution of test
        FAIL
        FAIL    command-line-arguments  1.699s
        FAIL
FAIL
FAIL    runtime/race    11.295s

@aclements could you take a look at this? Sorry I didn't find another way to contact with you:(
https://forum.golangbridge.org/t/about-lemma-2-of-hybrid-write-barrier/26253

@zhiqiangxu , that proof could be stated better. A stack is considered a single object for the purposes of marking.  So, B is either 1. a heap object, and the rest of the path has to be heap objects, or 2. it's stack, and all pointers from a stack either 2a. point to the heap, or 2b. point back to the same stack, in which case it's not a simple path and thus doesn't satisfy the precondition given on the path.

(Perhaps worth noting that there is at least one very special case in which the heap can point to a stack: channel operations in which the channel structure in the heap can point to the stack of a blocked goroutine.)

@zhiqiangxu , that proof could be stated better. A stack is considered a single object for the purposes of marking.  So, B is either 1. a heap object, and the rest of the path has to be heap objects, or 2. it's stack, and all pointers from a stack either 2a. point to the heap, or 2b. point back to the same stack, in which case it's not a simple path and thus doesn't satisfy the precondition given on the path.

Thanks for clarification, it makes much more sense now!
But as @ianlancetaylor stated above, heap can also point to a stack for the channel?


Changes to Insert DateTime at start of upgrade

Changes to support DNN-7280 for enhanced experience when upgrading.  Localization values used for UpgradeStarted.Text were obtained from Google Translate, and a few validated to be correct.  Would appreciate a review from native speakers

not sure how to process this - typically for translations you just add any new keys into the existing en-us resx file and then when I generate the difference files for the translators they provide the translations -I don't know if accepting the translations directly will cause any issues -in this case as it's the installation files which we ship I think it should be fine, but I'll probably check with the translators just in case

@cathalconnolly if I need to pull back the changes to the others by all means I'll modify my repro and update the pull request.  Wasn't 100% sure on this one for process language

There's only 1 change in the resx file, although most of the localized files seem changed. My suggestion would be to NOT include the localized values in the pull request and let the translators provide the new key.

@vmasanas Done!  I updated things to remove the other changes.


Adjusting for ambient Noise missing.

You might want to add this line after with sr.Microphone as source
r.adjust_for_ambient_noise(source, duration = 1)
Wasn't recording for me till I added this.
Windows btw.

It fixed the issue for you, but what does it do?

I read it on this website. it explains what setting dynamic energy to true and using adjust for ambient does.
according to this person, Using both together gives better results. so I tried it and it worked.
I kinda don't understand why, Since dynamic energy adjusts the value constantly, while adjust for ambient does it once. so It Should be fine with just using dynamic energy in theory.
https://www.codesofinterest.com/2017/04/energy-threshold-calibration-in-speech-recognition.html

Tried this too and the accuracy seems to have improved significantly. Thanks for sharing

It does seem to make it better ill try to do some testing to see how it reacts when you move from environment to environment and if it needs re calibration

This is now done in the code.


Use custom bond force for 1-4 interactions

Description
This PR adds support for scaled 1-4 interactions with custom nonbonded forces. These interactions are implemented via a custom bond force between the two particles.
Todos
Notable points that this PR has either accomplished or will accomplish.

[ ] TODO 1

Questions

[ ] Question1

Status

[ ] Ready to go

Codecov Report

Merging #17 (5263a72) into main (d7832e0) will decrease coverage by 0.79%.
The diff coverage is 70.58%.

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   76.25%   75.45%   -0.80%
==========================================
  Files           5        5
  Lines         240      273      +33
==========================================
+ Hits          183      206      +23
- Misses         57       67      +10

Flag
Coverage Δ

unittests
75.45% <70.58%> (-0.80%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

smirnoff_plugins/handlers/nonbonded.py
88.95% <70.58%> (-4.89%)
:arrow_down:

Continue to review full report at Codecov.

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


[Task] Rename Authentication Key and Chain

Description
Describe the task, this may be scoped beyond a single PR.
Instead of using the authentication field and key for DID messages, utilize a general verification method. As this removes the logic behind the authentication chain name, I request a rename to the Integration chain (Also in the code). The name of the key we use for signing DID messages may be anything, but we can default to #_sign-x, where x is the number of the key, incremented every time it is updated. This should not be enforced though.
Motivation
Describe the motivation for the feature to be developed.
We currently always use an authentication key for signing and updating DID messages. This doesn't fall in line with the standardized way of using the authentication verifiable relationship according to the DID standard. Instead, we should utilize a general verification method.
Resources
Link to any resources relevant for the task such as Issues, PRs, reference implementations, or specifications.
To-do list
Create a task-specific to-do list . Please link PRs that match the To-do list item behind the item after it has been submitted.

[x] Rename AuthChain and other references to IntChain (Integration Chain) in the code
[ ] Remove enforcement of signing with "auth key"
[ ] Edit examples to add verification method with a #_sign-1 key.
[ ] Edit account to standardize the #_sign-x naming instead of auth keys.
[x] Edit documentation to rename all references to AuthChain

Change checklist
Add an x to the boxes that are relevant to your changes, and delete any items that are not.

[ ] The feature or fix is implemented in Rust and across all bindings whereas possible.
[ ] The feature or fix has sufficient testing coverage
[ ] All tests and examples build and run locally as expected
[ ] Every piece of code has been document according to the documentation guidelines.
[ ] If conceptual documentation (mdbook) and examples highlighting the feature exist, they are properly updated.
[ ] If the feature is not currently documented, a documentation task Issue has been opened to address this.

@PhilippGackstatter I believe most of this is done, but I am unsure if we removed the enforcement of "auth keys" and the naming. Can you check and close if it is all done?


On slightly opening menu and again closing it ,its opening instead of closing

If we try to open slightly and close immediately so in that case slider always open instead of closing it.....
https://user-images.githubusercontent.com/71340355/115234292-7cdb8800-a136-11eb-857d-436582d80c10.mp4

Hi @RahulSathe458, you could set in sidebar config one of these options to prevent accidental open
    // Minimum swipe in px required to trigger listener: open
    documentMinSwipeX?: 10,

    // Range in px where document is listening for gesture: open
    documentSwipeRange?: 40,

When the sidebar is opening, until it is fully open, you can not close it.
If you need these feature PR are welcome!


⚠ Bofrost.fr has degraded performance

In 5f38ecf, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1410 ms

Resolved: Bofrost.fr performance has improved in a5319f2.


Error when running catkin_make

Hi and good day,
I noticed after I got the latest source code by running git clone, I received error message when running catkin_make "const struct darknet_ros_msgs::BoundingBoxes_<std::allocator >’ has no member named ‘boundingBoxes’" as per attachment.
Does that have anything to do with the latest commit?
Thank you in advance! :)

I'm sorry, silly me, it belongs to different path which is from dji_sdk_demo. I will close the ticket now. Sorry for the trouble!


how to launch a subset query?

hi, do logica have subset function? I want to compare one set to another, and return a boolean value showing whether the two sets are equal or subset?
eg:
SUBSET({ }, { }) = TRUE
SUBSET({ 1, 2, 3 }, { }) = TRUE
SUBSET({ 1, 2 }, { 1, 2 }) = TRUE
SUBSET({ 1, 2, 3 }, { 1, 2 }) = TRUE
SUBSET({ 1, 3, 5 }, { 1, 2 }) = FALSE
many thanks!

Hello!
We need to check membership in the second list for each element in the first list.
Here is the implementation and a test:
@Engine("sqlite");

# Returns 1 if a is subset of b and 0 otherwise.
# To call as a predicate call Constraint(Subset(a, b)).
Subset(a, b) = result :-
  result_or_null Min= (e in b :- e in a),
  result == Coalesce(result_or_null, 1);

# Testing Subset function.
Test(a, b, Subset(a, b)) :-
  (a == [], b == [1,2]) |
  (a == [1,2], b == [1,2,3]) |
  (a == [1,2,3], b == [1,2]);

Let me know if you have further questions.


Small PSP fixes

Host-namespaces and privileged containers work as you expect (defaults built into the templates.C
I have updated  host-network-ports to have the default min/max value.
Capabilities has an empty array as the default, but does allow the user to add more to that list. Do we want to hardcode that nothing can be added to that list?

Host-namespaces and privileged containers work as you expect (defaults built into the templates.C
I have updated host-network-ports to have the default min/max value.
Capabilities has an empty array as the default, but does allow the user to add more to that list. Do we want to hardcode that nothing can be added to that list?

existing LGTM


IcyTerm 7.7 crashes

It disconnects as soon as a connection is made, I looked at the log and find this entry each time:
ERROR - run_update_thread::read_data:

Updated the builds - should work now.


Update Linux docker sha

Changelog
Bug Fixes

Update Linux docker sha to use image with LLVM Lumen 12.0.0-dev (2020-08-04).

It passed the Linux compiler build and the rest of the failures are unrelated to the LLVM version issue, so merging


[Feature]: How to reuse createSnapshotSerializer in test helper

What problem does this feature solve?
The createSnapshotSerializer (https://github.com/web-infra-dev/rsbuild/blob/main/scripts/test-helper/src/index.ts#L17C17-L50) is battle tested and not coupled with Rsbuild. If possible, I could help to extract it to a new package and put it under rspack-contrib organization, and reuse it in Rslib 🤗.
What does the proposed API look like?
N/A.

I'm curious if this serializer can be used by both vitest and jest.
https://github.com/web-infra-dev/rspack/issues/7156
Rspack also needs this

I used to use a Jest serializer plugin in Vitest in other project. "Jest as a spec, Vitest as a implementaion", I believe so.

LGTM 👍

Any suggestion for the package name? rsserializer? 🤣

Maybe it doesn't need the rs keyword, just use anything you want 😄


Secure persistence: Contiguity of arrays may change after loading

After loading objects that contain numpy arrays, it can happen that the contiguity of the array is not the same as it was before saving.
The current testing does not catch this but we could add more checks here:
                assert val1.flags["C_CONTIGUOUS"] is val2.flags["C_CONTIGUOUS"]
                assert val1.flags["F_CONTIGUOUS"] is val2.flags["F_CONTIGUOUS"]

A peculiar things is that some of the arrays causing the error seem to be neither C, nor Fortran contiguous. And the loaded array is both C and Fortran contiguous. One example where this occurs is in cv_results_ of GrahpicalLassoCV.
At least for performance reasons, it would be better to conserve the contiguity of the arrays. One approach could be to store the contiguity of the array in the state and then restore it after loading, although I don't know how to get an array with neither contiguity.
As for the other flags, I don't know how important they are and if ensuring that their values are conserved is feasible or even desirable.

This should only be the case for object dtypes, since npy ensures the same contiguity.
We probably can do better in storing those numpy arrays anyway, something closer to the npy binary than what we have now.

This should only be the case for object dtypes, since npy ensures the same contiguity.

I haven't checked that. Anyway, should we consider this to be a bug?

We probably can do better in storing those numpy arrays anyway, something closer to the npy binary than what we have now.

With "those", do you mean object arrays? Not sure how easy that would be.

I think if the npy format is not keeping the contiguity, we can report it upstream, but for object arrays, we can find a better way to store them.

I checked and it seems the issue is not that the dtype was object. E.g., for GraphicalLassoCV, we get the error on cv_results_, because split_*_score values change contiguity from neither C nor F, to both C and F. The dtype is float64 and our get_state and get_instance methods save and load via numpy.
Interestingly, for the mean_test_score, we get no error -- it's both C and F contiguous before and after loading. So probably np.save and np.load work fine if the array is either C or F contiguous or both, but not if it's neither.
How we even get into this situation, I don't know.

What does it mean for a 2d array to not be C or F contiguous? Or are these 1d arrays that it doesn't matter whether they're C or F?

I checked, most arrays are 1d but there also arrays with shape (1, N), so technically 2d but practically 1d. I would assume that for those cases, the contiguity doesn't matter, but who knows if there are edge cases where it matters. For now, how about we add tests that check the contiguity, but only if x.squeeze().ndim > 1?

That sounds good to me.


Don't load JS files individually in debug mode

@tomaszdurka @vogdb
I would suggest to remove the individual loading of all library files in debug mode.
Doing so has some disadvantages like that the browser loads a lot of files one by one, which takes especially long on mobile devices.
The advantage of loading them individually is only that the line numbers match the ones in the original file. I don't think that is very important.
What do you think?
Related:
https://github.com/cargomedia/puppet-packages/blob/master/modules/cm/manifests/vhost.pp#L74

The idea behind it was to use ide breakpoints, debug code in there. Never fully worked.
What about keeping this functionality and simply produce same as on production? Otherwise we can skip it.

What about keeping this functionality and simply produce same as on production?

You mean to keep the code, but disable it?

Yes.

I would remove it then - it adds some complexity to the puppet modules, which will be tricky to understand in the future if we don't use it anymore?

Let it be.
Sidenote: Btw I think I have issue connected with all those vendor js/css somewhere opened.

You mean this one about assets? https://github.com/cargomedia/CM/pull/1557

@vogdb please let us know what you think - if it can/should be removed.

Sync between lines is not important. Almost never used it. Plenty of other ways to sync them for me. Lets do it.

...the proper way to do this would probably be to create a source map.


IPay interface

We want create a interface for all port classes, and user payment must be done just by call below methods.
Notes:

with and getRefId methods is optional.

<?php

namespace IPay;

interface IPayInterface
{
    /**
     * This method use for set price in Rial.
     *
     * @param int $amount in Rial
     *
     * @return $this
     */
    public function set($amount);

    /**
     * Some of the ports can be send additional data to port server.
     * This method for set this additional data.
     *
     * @param array $data
     *
     * @return $this
     */
    public function with(array $data);

    /**
     * This method use for done everything that necessary before redirect to port.
     *
     * @return $this
     */
    public function ready();

    /**
     * Get ref id, in some ports ref id has a different name such as authority
     *
     * @return int|string
     */
    public function getRefId();

    /**
     * This method use for redirect to port
     *
     * @return mixed
     */
    public function redirect();

    /**
     * Return result of payment
     * If result is done, return $this, otherwise throws an related exception
     *
     * @return $this
     */
    public function verify();

    /**
     * Return tracking code
     *
     * @return int|string
     */
    public function trackingCode();
}

If you see a problem or have a new idea please commented in this issue.

You know, this looks pretty good, though in a perfect world, I would like to call something like this:
Before sending client to gateway:
$myImplementation->set($price)->with($additional_data)->redirect();

Client is back, let's verify it.
$myImplementation->verify()->get();

So, this way is more readable. and I think we should return $this to leverage method chaining.
Any thoughts?

Interesting, actually this way is simpler and easier. I think we can't return false in chaining method, and if any error happen we must throws an exception.
In your example, you define all methods we needs, and i create another Interface for this approach.
Also i think we need a method for get tracking code.

According to @easteregg opinion, I create a new Interface

This looks really good. allow me to play with it for a few days. see if we missed something. I have a ZarrinPal class that could really use a helping hand to refactor. I'm gonna work on it and report it back here.
thanks @m-jch

Thanks. Also i currently developing Mellat port with this new interface, and if any new problem happen i reported this here.

I Found first problem. For example in Mellat gateway before redirect to port we need to access refId and with methods in last interface we can't do this:
$ipay->set(4000)->redirect();

And for this i create new method.
$ipay->set(4000)->ready();
$ipay->redirect();

In this way we can access to refId before redirect:
$ipay->set(4000)->ready()->getRefId();
$ipay->redirect();

Also i add a new getRefId method to interface, and update interface in first comment in this issue.

I change and commit IPayMellat class for work with new Interface.
Example of result:
Redirect to bank
use IPay\IPay;

$ipay = new IPay(IPay::P_MELLAT);
try {
    $refId = $ipay->set(1000)->ready()->getRefId();
    $ipay->redirect();
} catch (Exception $e) {
    echo $e->getMessage();
}

Verify payment
use IPay\IPay;

$ipay = new IPay(IPay::P_MELLAT);
try {
    $trackingCode = $ipay->verify()->trackingCode();
} catch (Exception $e) {
    echo $e->getMessage();
}

At current, unfortunately, IPayMellat class work with previous version of table, and i still waiting for
@Reza1607 to complete schema of new table.

I think,
We need a method to return the bank id.
This method returns the ID of the bank's transactions are in the table we can use it.
/**
* return bank id
* @return int
*/
public function getId();

The proposed database schema #12
@m-jch

Interface updated.
portId and _construct method added.

Interface is updated.


net: lwm2m: notify-messages aren't generated due to duplicate observation and bad timing

Describe the bug
If a resource is observed and if there happens to be duplicate observation(s) on that resource, the resource's content
might not be reported correctly anymore.
To Reproduce

Establish observation on a resource whose content changes regularly (for example: device uptime).
Have the resource's content reported to the server periodically (e.g. every second) by generating notify messages, either manually or automatically.
The smaller the report interval, the easier it is to reproduce the problem.
Trigger another observation request on the resource to cause a duplicate observation.

-> The notify-messages won't be generated periodically (e.g. every second) anymore.
What this also means is that, looking from the server/backend's point of view, many values of the resource will be missing.
Why?
When generate_notify_message() is called, the boolean active_tx_operation is set to true. In the meantime, an observation request is received by the device, and the resource's token is updated (while the device hasn't received the corresponding ACK for its notify-message yet).
Then the ACK comes and is handled by notify_message_reply_cb(), but since the token has been updated, notify_message_reply_cb() cannot find the observer anymore:
https://github.com/zephyrproject-rtos/zephyr/blob/48c83a1f4268e539afa03abe2ecaed43a0650b7e/subsys/net/lib/lwm2m/lwm2m_message_handling.c#L2893C1-L2907C1
        } else {
                obs = engine_observe_node_discover(&msg->ctx->observer, &prev_node, NULL,
                                                   reply->token, reply->tkl);

                if (obs) {
                        obs->active_tx_operation = false;
                        if (msg->ctx->observe_cb) {
                                msg->ctx->observe_cb(LWM2M_OBSERVE_EVENT_NOTIFY_ACK,
                                                     lwm2m_read_first_path_ptr(&obs->path_list),
                                                     reply->user_data);
                        }
                        notify_cached_pending_data_trig(obs);
                }
        }

if (obs) resolving to false is what we suspect to have caused the issue.

It seems to me that a natural consequence of updating the observer token here or here should be canceling any ongoing transactions. The question is how to tackle this best. We could simply loop over lwm2m_ctx->replies array and try to find a message with a matching token, or we could replace active_tx_operation with a struct lwm2m_message pointer, so that it could be retrieved from the observer context directly (non-null would mean there is an active transaction). Any thoughts @SeppoTakalo?

In a meantime, I've opened a PR with a fix proposal: https://github.com/zephyrproject-rtos/zephyr/pull/63458.
@hung-gcx Could you please give it a try?

@rlubos I just gave https://github.com/zephyrproject-rtos/zephyr/pull/63458 a spin, seems to have solved it (:
One titbit, not sure if relevant:

The value right before the (additional/duplicate) observation request is successfully reported to the server in a CON message (say the value is 300).
Observation request comes in, device responds with and ACK containing the previous value (300)
-> The same value is reported twice, but I'd say it's expected behavior.

@rlubos I just gave #63458 a spin, seems to have solved it (:

Glad to hear that!

One titbit, not sure if relevant:

The value right before the (additional/duplicate) observation request is successfully reported to the server in a CON message (say the value is 300).
Observation request comes in, device responds with and ACK containing the previous value (300)
-> The same value is reported twice, but I'd say it's expected behavior.

If the observed resource value did not change inbetween notification/new observe request, then I'd say there's nothing wrong here. The device has to respond to the observation request even if the resource value is still the same.


To-Do improvements

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

[x] https://github.com/Azure-Samples/azure-search-openai-demo-csharp/issues/20
[x] #52
[x] #44
[x] #42
[ ] #53
[x] #50
[ ] #45
[x] #46
[ ] #47
[x] #48
[x] #49
[x] #58
[x] #60

Closing in favor of #123


[Azure AD OAuth] AADSTS90013: Invalid input received from the user.

Occasionally, Azure AD OAuth fails with an error: "[AADSTS90013: Invalid input received from the user."
This issue might be due to random_secret generating strings containing characters that are not URL-safe.
https://github.com/Chainlit/chainlit/blob/main/backend/chainlit/server.py#L302

Thank you for the feedback! Can you take a look a this @tpatel?

@Hayato-Kosugi
We are investigating but we are not able to reproduce on our end.
Could you provide us with more information, for example what's the random_secret value that is responsible for the issue?

@Hayato-Kosugi We are investigating but we are not able to reproduce on our end. Could you provide us with more information, for example what's the random_secret value that is responsible for the issue?

You might be able to find it in the browser's cookies. It is stored in the cookies and then deleted on success. Since it is failing it should still be there.

The bug should be fixed with #442


Linter rule to catch known patterns which cannot be validated by what-if/preflight

There are various ways you can shoot yourself in the foot, and end up with a template that cannot be accurately validated by what-if. It's not made clear when editing a Bicep file which scenarios could result in inaccurate what-if/preflight.
If we could display warnings to the user, it would help inform and avoid bad patterns where possible - so the user has the ability to understand the changes they may need to make to improve validation. I think a lot of users would be willing to re-organize the code to ensure an accurate validation.
The main example I can think of is non-deploy-time-constant values being used to format the name or scope of a resource when passed in as a module parameter.

Closing this - I've done a more complete write-up under a separate issue.


Roll Flutter Engine from c58d87d62c20 to 2c88f9784fad (6 revisions)

https://github.com/flutter/engine/compare/c58d87d62c20...2c88f9784fad
2024-07-30<EMAIL_ADDRESS>Bump ossf/scorecard-action from 2.3.3 to 2.4.0 (flutter/engine#54208)
2024-07-30<EMAIL_ADDRESS>Roll Skia from 20e4dfabb6b2 to<PHONE_NUMBER>a3 (1 revision) (flutter/engine#54207)
2024-07-30<EMAIL_ADDRESS>Roll Dart SDK from e8eca6f05494 to 6ab9452297e0 (1 revision) (flutter/engine#54206)
2024-07-30<EMAIL_ADDRESS>Roll Skia from 56de22c39d0e to 20e4dfabb6b2 (1 revision) (flutter/engine#54205)
2024-07-30<EMAIL_ADDRESS>Roll Skia from 946ca24d03ab to 56de22c39d0e (1 revision) (flutter/engine#54204)
2024-07-30<EMAIL_ADDRESS>Roll Skia from 8167ff71af2c to 946ca24d03ab (2 revisions) (flutter/engine#54202)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Rakefile not found

No Rakefile found ...

I need more details here. There is a Rakefile in the repo, so if your installation of ruby or rake can't find it, it's probably something wrong with your environment specifically.

Probably. I will read the Archlinux wiki. More info when I get a chance.
Thank you!
On Wed, Sep 7, 2016 at 1:15 PM, Tyler Benziger<EMAIL_ADDRESS>wrote:

I need more details here. There is a Rakefile in the repo, so if your
installation of ruby or rake can't find it, it's probably something wrong
with your environment specifically.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tybenz/exemplar/issues/4#issuecomment-245334306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATmAq13EdnBgRBJtg2O7u4X8swou2va9ks5qnuMQgaJpZM4J3Ap7
.

Complaints about versions installed. I followed some forum advice, ran a couple of bundle updates and prefaced rake with bundle exec like I ought to and pimba! Thanks.


Running on Windows 10 (x64)

Hello All,
I am very new to Micro but think that it may hold promise for use in a highly scalable P2P application that I am working on since I really like what I have read so far although not sure what the difference between the "Micro" and "Go-Micro" repositories are at this moment.
Currently, I tried to follow a blog post "Building a global services network using Go, QUIC and Micro" (https://blog.gopheracademy.com/advent-2019/building-a-microservices-network/) [Dated 2019] to initial get up and running after compiling with Golang v1.19.3 on a Windows 10 (x64) which produced the micro.exe.
The blog said to connect with "micro.exe --peer" but there is no "peer" option and I do not know how to register and connect with micro.exe at this point?
Also, I am interested to know how well the micro network scales in addition to wanting to learn how to set up a private micro network for some development and testing purposes initially.
PS) I did find the Micro Discord Channel but there does not seem to be much activity.
Any help on these would be truly appreciated.
Best Regards and have a good day.

Old issue, but Windows is not well supported sorry


Package.swift must be updated if it's installed via Swift Package Manager

Hello, let me report an issue and a temporary workaround for it.
Environment

Xcode 13 RC
Parchment master as of now

How to reproduce

Install Parchment via Swift PM
Add it as a dependency of an app
Make an archive

What's the issue
Compile fails even though the codebase has #if canImport(...) barrier. It only happens when archiving as far as I see.

(Temporary) Workaround
As explained here, I had to folk the repo and specify platform versions for all products. See also the below diff.
https://github.com/rechsteiner/Parchment/compare/master...pot8os:spm-archive?expand=1
All errors had gone when I switch to my branch above in the Swift PM settings.
Consideration
Although it's a valid workaround, it bumps the minimum supported version of the library so I don't think it's an acceptable solution. The best option is that Xcode addresses the issue but I'm not sure when it becomes real. I'd be happy if you could share some thoughts.
Thanks!

Thanks for the detailed report! It would definitely be a shame to drop support for that many versions. I'll see if I can find some other workaround.

I have the same issue, since xcode and macos updated this morning

I have the same issue

Hello, let me report an issue and a temporary workaround for it.
Environment

Xcode 13 RC
Parchment master as of now

How to reproduce

Install Parchment via Swift PM
Add it as a dependency of an app
Make an archive

What's the issue
Compile fails even though the codebase has #if canImport(...) barrier. It only happens when archiving as far as I see.

(Temporary) Workaround
As explained here, I had to folk the repo and specify platform versions for all products. See also the below diff.
https://github.com/rechsteiner/Parchment/compare/master...pot8os:spm-archive?expand=1
All errors had gone when I switch to my branch above in the Swift PM settings.
Consideration
Although it's a valid workaround, it bumps the minimum supported version of the library so I don't think it's an acceptable solution. The best option is that Xcode addresses the issue but I'm not sure when it becomes real. I'd be happy if you could share some thoughts.
Thanks!

I tried but it doesn't work for me

Alternative Solution :
Instead of forking the repo please install using Cocoa Pods instead of Swift PM (remove from Package Dependencies if already installed). Then simply unlock the file PageView.swift for editing.
Change the line #if canImport(SwiftUI) && canImport(Combine) to #if !arch(arm), should archive without any issues.

Not work for me, any solution to use this library?

Alternative Solution :
Instead of forking the repo please install using Cocoa Pods instead of Swift PM (remove from Package Dependencies if already installed and Don't forget to clean build folder). Then simply unlock the file PageView.swift for editing.
Change the line #if canImport(SwiftUI) && canImport(Combine) to #if !arch(arm), should archive without any issues.
Basically put all the swift UI code between #if !arch(arm) / #endif

Not work for me, any solution to use this library?

Please try alternative solution

This seems to be a known issue in Xcode 13:

Swift libraries may fail to build for iOS targets that use armv7. (74120874)
Workaround: Increase the platform dependency of the package to v12 or later.

From: https://developer.apple.com/documentation/xcode-release-notes/xcode-13-release-notes
Hopefully it will be fixed before the official release, but for now I've created a xcode-13 branch that sets the deployment target to iOS 12 which fixes the issue.

A new version v3.1.0 is available now that should fix this issue 🥳 Let me know if anyone is still having issue


feat: Add address(0) check for addresses in parameters and constructors

Summary

Check if the address in the constructor is address(0)
Check if the address in the parameter is address(0)

Close issue: #92

the changes are ok, but must resolve git conflicts. I can approve it after that

the changes are ok, but must resolve git conflicts. I can approve it after that

It should be fine now.

@lukema95 thanks so much for your contribution! 🙌 One last request before we merge: please, run yarn generate, so Go files are generated from the updated contracts.

@fadeev After running yarn generate, I encountered an error: 'Error: Failed to compile IERC20Permit'. I'm not sure if this is a normal occurrence, as the IERC20Permit contract is an interface contract from the OpenZeppelin library.

@lukema95 created an issue for this: https://github.com/zeta-chain/protocol-contracts/issues/100
I think it shouldn't prevent merging this PR, since this error exists on main as well.

@andresaiello please, review.

Hey @lukema95 , sorry for the time coming back on this one.
For some context, we started working on new version for the smart contract architecture: https://github.com/zeta-chain/protocol-contracts/issues/198
As an improvement, it's still beneficial to merge this. So let me know if you want to solve the current conflicts in the PR and we can merge it. Otherwise we can close the issue as the v1 contracts will become deprecated.

Hey @lukema95 , sorry for the time coming back on this one. For some context, we started working on new version for the smart contract architecture: #198 As an improvement, it's still beneficial to merge this. So let me know if you want to solve the current conflicts in the PR and we can merge it. Otherwise we can close the issue as the v1 contracts will become deprecated.
It doesn't matter.


Published artifact is missing MIT license

Similar to: https://github.com/henryw374/time-literals/issues/7
We have a deps.edn project which looks like this:
{:deps {org.clojure/clojure {:mvn/version "1.11.1"}
        com.xtdb.labs/core2-api {:local/root "api"}
        com.xtdb.labs/core2-core {:local/root "core"}
        com.xtdb.labs/core2-client {:local/root "client"}
        com.xtdb.labs/core2-server {:local/root "server"}
        com.xtdb.labs/core2-datasets {:local/root "modules/datasets"}

        com.xtdb.labs/core2-kafka {:local/root "modules/kafka"}
        com.xtdb.labs/core2-s3 {:local/root "modules/s3"}
        com.xtdb.labs/core2-jdbc {:local/root "modules/jdbc"}
        com.xtdb.labs/core2-bench {:local/root "modules/bench"}

        integrant/repl {:mvn/version "0.3.2"}
        org.slf4j/slf4j-api {:mvn/version "1.7.36"}
        ch.qos.logback/logback-classic {:mvn/version "1.2.10"}
        cheshire/cheshire {:mvn/version "5.10.0"}
        com.cognitect/transit-clj {:mvn/version "1.0.324"}

        org.clojure/data.csv {:mvn/version "1.0.0"}
        org.clojure/tools.namespace {:mvn/version "1.2.0"}
        org.clojure/test.check {:mvn/version "1.1.1"}

        org.postgresql/postgresql {:mvn/version "42.2.20"}
        org.xerial/sqlite-jdbc {:mvn/version "<IP_ADDRESS>"}

        com.widdindustries/time-literals {:mvn/version "0.1.6"}
        io.github.cognitect-labs/test-runner {:git/tag "v0.5.0" :git/sha "b3fd0d2"}
        com.clojure-goes-fast/clj-async-profiler {:mvn/version "0.5.1"}}

;; ....

[WARNING] Licensing: The artifact com.widdindustries:time-literals:jar:0.1.6 has no license specified.

Is it possible to get the MIT license embedded in the release jar so automated licensing tooling can see it?

Latest versions of these libs have this
On Tue, 19 Jul 2022, 19:18 Steven Deobald, @.***> wrote:

Similar to: henryw374/time-literals#7
https://github.com/henryw374/time-literals/issues/7
We have a deps.edn project which looks like this:
{:deps {org.clojure/clojure {:mvn/version "1.11.1"}
com.xtdb.labs/core2-api {:local/root "api"}
com.xtdb.labs/core2-core {:local/root "core"}
com.xtdb.labs/core2-client {:local/root "client"}
com.xtdb.labs/core2-server {:local/root "server"}
com.xtdb.labs/core2-datasets {:local/root "modules/datasets"}
    com.xtdb.labs/core2-kafka {:local/root "modules/kafka"}
    com.xtdb.labs/core2-s3 {:local/root "modules/s3"}
    com.xtdb.labs/core2-jdbc {:local/root "modules/jdbc"}
    com.xtdb.labs/core2-bench {:local/root "modules/bench"}

    integrant/repl {:mvn/version "0.3.2"}
    org.slf4j/slf4j-api {:mvn/version "1.7.36"}
    ch.qos.logback/logback-classic {:mvn/version "1.2.10"}
    cheshire/cheshire {:mvn/version "5.10.0"}
    com.cognitect/transit-clj {:mvn/version "1.0.324"}

    org.clojure/data.csv {:mvn/version "1.0.0"}
    org.clojure/tools.namespace {:mvn/version "1.2.0"}
    org.clojure/test.check {:mvn/version "1.1.1"}

    org.postgresql/postgresql {:mvn/version "42.2.20"}
    org.xerial/sqlite-jdbc {:mvn/version "<IP_ADDRESS>"}

    com.widdindustries/time-literals {:mvn/version "0.1.6"}
    io.github.cognitect-labs/test-runner {:git/tag "v0.5.0" :git/sha "b3fd0d2"}
    com.clojure-goes-fast/clj-async-profiler {:mvn/version "0.5.1"}}

;; ....
}
If I run the Neo4j Licensing plugin, like so:
$ mvn org.neo4j.build.plugins:licensing-maven-plugin:check -DfailIfMissing=false &&
mvn org.neo4j.build.plugins:licensing-maven-plugin:collect-reports &&
mvn org.neo4j.build.plugins:licensing-maven-plugin:aggregate
...it complains about two libraries:
[WARNING] Licensing: The artifact cljs.java-time:cljs.java-time:jar:0.1.19 has no license specified.
[WARNING] Licensing: The artifact com.widdindustries:time-literals:jar:0.1.6 has no license specified.
Is it possible to get the MIT license embedded in the release jar so
automated licensing tooling can see it?
—
Reply to this email directly, view it on GitHub
https://github.com/henryw374/cljs.java-time/issues/10, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXNSDATKXSHKUT6Q4XX7QLVU3WN7ANCNFSM54A4QSMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


*: implement request header support

This pr adds support to customize request headers. We need it to examine cluster id in TiKV.
close #135

PTAL

@siddontang @overvenus @huachaohuang PTAL

LGTM

PTAL


Added dark_theme.html

Adding dark theme for the lib
Just added a new .html file to how preview in dark mode.

attaching screenshot for reference


Use Common CMakeUtils to build deps

Coverage remained the same at 96.313% when pulling ae3e6a90b7b38322e7d344e091cecff678c745d0 on CMake into 47fd08cdf432e418c17de23944ebc0b8e6c7a67a on master.


Shelly i3

Hi @windkh
any plan to support the Shelly i3?
Or is there a reason why this is not supported yet?
Thanks
Richard

Hi … the reason is quite simple. I do not own a I3.
Whenever I buy a device I implement the node red node to support it.
… I can not affort buying all devices in the shop at the same time.

As it's no longer on sell, I guess it will never be supported.
Thanks for your answer.

You can send me one. Then I can test the implementation and afterwards I send it back to you. That is all
i can offer

Another idea: I guess the I3 is similar to the shelly button but with more channels.
I could create a button node and you would have to test if it also works with your I3.
What about that?

We could try this. I send you already an email. But let’s try the solution and I will test it here.

I did not receive an emai

I send it to the web.de address linked in your profile. Maybe it’s sorted as spam?

please check out 4.4.0


doc/*.txt -> *.rst

Will close #910

[ ] Is it mergeable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

Can this branch be tested on RTD first?

@ctb Sure, I'll do a special build there

On Sun, Apr 05, 2015 at 04:14:51AM -0700, Michael R. Crusoe wrote:

@ctb Sure, I'll do a special build there
thanks - and good catch!

@ctb See http://khmer.readthedocs.org/en/doc-extension-rename/

Ready for review and merge

On Sun, Apr 05, 2015 at 04:31:56AM -0700, Michael R. Crusoe wrote:

@ctb See http://khmer.readthedocs.org/en/doc-extension-rename/

BTW, this looks good to me - go ahead and merge when ready.

Thanks!


Fastclick.js being blocked by AdBlock

Has anybody had any issues with the fastclick.js file being blocked by AdBlock in Chrome (49)?

:D fastclick used to be an advertising network (https://en.wikipedia.org/wiki/Fastclick - up until 2005 it seems!)

I have taken a look at the forums for EasyList, the large filter which AdBlock downloads regularly. They have since removed it from their filter, it was a mistake it seems.
If you go to filter preferences and then down to EasyList and actions, you can click "update filters". Should be good to go!

Thank you very much for the quick responses. Updating the filters resolved the issues.
Thank you once again.


Web Wallets and modern Web Platform APIs

Hey all,
I had a good chat with @ve7jtb, Mike Jones and @balfanz  at IIW about ways in which the browser could help Web Wallets, and wanted to take notes here to kick off a discussion.
I'm a browser engineer involved in the design and development of the FedCM API, the Digital Credentials API and other less related identity APIs, like WebOTP. I'm only partially involved in WebAuthn, but I think you all have good coverage there.
Here are some whiteboard screenshots I took with some notes:

I don't particularly know exactly what's the best way to help here, but I wanted to stop by to say "hi" and to offer any browser help you may have. I personally believe that the web should be a platform that is capable for developers of all types of applications more generally, but more specifically Wallets.
Having said that, there is at least a couple of ways that I think I might be able to help, to get the ball rolling:
(a) On Desktop, Web verifiers discovering of your Web Wallet and
(b) On Android, Native verifiers discovering your Web Wallet
I don't know if (b) is necessary or useful, but wanted to offer just in case. I think this can work equally well on Chrome on Android, but let me focus on Chrome on Desktop for a second.
Here is a demo of what (a) may look like:
You should be able to try this yourself this in Chrome here, with code that's currently checked in (some, behind flags):
https://code.sgo.to/2024/10/30/openid4vp-over-fedcm.html
What I'm imagining is that, in your Web Wallet, you could call IdentityProvider.register() to register itself in the browser storage:
await IdentityProvider.register(/** your configURL here */);

The browser prompts the user for permission to add https://wwwallet.org/ to its registry. The string used isn't great, because it is intended to be used by OIDC IdP rather than OpenID4VP Holders, but that's an easy fix.
Later, when a user is going to a verifier, it can call navigator.credentials.get({identity: ...}) and ask for "any" wallet (or specific ones too).
await navigator.credentials.get({
  identity: {
    providers: [{
      configURL: "any", // accepts any wallet
      type: "openid4vp", // that speak openid4vp
    }]
  }
});

The browser loops through every registered wallet, and sends a request with cookies to the wallet and asks for "give me all credentials you might have". With that, the browser is able to build the following credential selector across all wallets:
There are many variations of UX formulations here, but here is one example of what they could get:

Ultimately, though, I think you want this UX here instead, where the user can discover your wallet even before clicking on anything:

Up until here, because the selector is mediated by the browser, your wallet wouldn't know about the verifier, and the verifier wouldn't know about your wallet.
Upon the selection of a credential, your the browser sends the OpenID4VP request to your wallet, which it can choose to open a pop-up window to gather some user confirmation:

Upon confirmation, in a pop-up window that's controlled by your wallet, the wallet can call IdentityProvider.resolve() to return the OpenID4VP response back to the verifier, much like your native wallet would.
The verifier than is responsible for verifying the verifiable presentation:

Architecturally speaking, this prototype is happening with OpenID4VP over FedCM. FedCM is a browser API that is used as a binding to OAuth, and since OpenID4VP is an OAuth profile, it can serve as a binding to it. I can walk you through with what would be involved in using it, but wanted to first get a sense of the kinda of UX that it could provide before going much further.
Because FedCM is a browser-mediated UI, it can unify and reconcile well with WebAuthn and the Digital Credentials API. For example, I think it would be possible to add to the selector an option to get credentials from Android Wallets (e.g. via cross-device CTAP) in addition to Web Wallets (available locally on Desktop) in the same selector.
This leads to a good amount of open questions (e.g. what's the relationship between FedCM and Digital Credentials? how do matchers work? how do we combine the Digital Credentials call with the FedCM call?), but wanted to kick this off early to see if you all would have an interest in exploring taking advantage of modern browser Web Platform API -- as well as extending its shortcomings.

A prototype with simplified and practical UX choices could serve as a starting point to explore this topic, with wwWallet functioning either as a web wallet or a PWA app
To the list of questions above, I would add another concerning the role boundaries between the wallet and the browser in the areas of access control and consent management. Unless we significantly narrow down the usage scenarios and credential types, this could become quite complex in regulated ecosystems where advisory or mandatory disclosure policies may apply.
Let’s start simple and see where this takes us.


Binance future Minimum Qnt..  get minimum position size  fetch or calculate in python

Binance future get minimum position size  fetch or calculate in python

how to fetch or get or calculate Minimum Qnt..
anyone help me out ..
thank you ...

Get MIN_NOTIONAL filter information
        min_notional_filter = next((x for x in symbol_info['filters'] if x['filterType'] == 'MIN_NOTIONAL'), None)
        min_notional = float(min_notional_filter['notional']) if min_notional_filter else None

        # Get LOT_SIZE filter information for Min Trade Amount
        lot_size_filter = next((x for x in symbol_info['filters'] if x['filterType'] == 'LOT_SIZE'), None)
        min_trade_amount = float(lot_size_filter['minQty']) if lot_size_filter else None

        # Get current ticker price
        ticker_response = client.futures_symbol_ticker(symbol=symbol)
        if 'price' in ticker_response:
            ticker_price = float(ticker_response['price'])
        else:
            print(f"Symbol: {symbol}, 'price' not found in ticker response.")
            continue

        # Calculate the equivalent BTC quantity for the minimum notional value
        quantity = min_notional / ticker_price

        # Append symbol, min notional, min trade amount, and ticker price information to the list
        symbol_info_list.append({
            'Symbol': symbol,
            'Min Notional': min_notional,
            'Min Trade Amount': min_trade_amount,
            'Ticker Price': ticker_price,
            'Calculated Quantity': quantity
        })

Here's a breakdown of what each part of the code is doing:
MIN_NOTIONAL filter:
It searches for the filter with the type 'MIN_NOTIONAL' in the list of filters for a particular trading symbol.
Extracts the 'notional' value from the filter, converting it to a float.
If the filter is not found, sets min_notional to None.
LOT_SIZE filter:
Similar to MIN_NOTIONAL, it searches for the filter with the type 'LOT_SIZE'.
Extracts the 'minQty' (minimum quantity) value from the filter, converting it to a float.
If the filter is not found, sets min_trade_amount to None.
Ticker Price:
Retrieves the current ticker price for the given trading symbol using the futures_symbol_ticker method from the Binance API.
If the 'price' is present in the ticker response, converts it to a float.
If 'price' is not found, prints a message and continues to the next symbol.
Calculating Equivalent BTC Quantity:
Calculates the equivalent BTC quantity for the minimum notional value using the formula quantity = min_notional / ticker_price.
Appending Information to List:
Appends a dictionary containing symbol information (symbol, min_notional, min_trade_amount, ticker_price, and calculated quantity) to the symbol_info_list.