Node.js  uvwasi team Meeting 2023-11-22

Time
UTC Wed 22-Nov-2023 19:00 (07:00 PM):

Timezone
Date/Time

US / Pacific
Wed 22-Nov-2023 11:00 (11:00 AM)

US / Mountain
Wed 22-Nov-2023 12:00 (12:00 PM)

US / Central
Wed 22-Nov-2023 13:00 (01:00 PM)

US / Eastern
Wed 22-Nov-2023 14:00 (02:00 PM)

EU / Western
Wed 22-Nov-2023 19:00 (07:00 PM)

EU / Central
Wed 22-Nov-2023 20:00 (08:00 PM)

EU / Eastern
Wed 22-Nov-2023 21:00 (09:00 PM)

Moscow
Wed 22-Nov-2023 22:00 (10:00 PM)

Chennai
Thu 23-Nov-2023 00:30 (12:30 AM)

Hangzhou
Thu 23-Nov-2023 03:00 (03:00 AM)

Tokyo
Thu 23-Nov-2023 04:00 (04:00 AM)

Sydney
Thu 23-Nov-2023 06:00 (06:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+uvwasi team+Meeting+2023-11-22&iso=20231122T1900
or https://www.wolframalpha.com/input/?i=07PM+UTC%2C+Nov+22%2C+2023+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1_AL-pDzgfxCLYx043ULGwPTRne-FOS0JPI3uJvrCDf0/edit

Agenda
Extracted from uvwasi-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/node

WASI preview 1: fd_readdir ignores cookie argument #47193

nodejs/uvwasi

What's new with uvwasi in 2023 #201
Changes to WASI spec in snapshot_preview2 #59

Invited

uvwasi team: @nodejs/wasi

Observers/Guests
Notes
The agenda comes from issues labelled with uvwasi-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/91612484369
For those who just want to watch:
youtube admin page: https://www.youtube.com/my_live_events?filter=scheduled

@guybedford @nodejs/wasi, @phated, @ospencer unfortunately with the timezone change this conflicts with the TSC meeting this week.  I suspect that the US Thanksgiving might impact attendance as well.
I'm going to move to next week. Let me know if that was not the right thing to do.

closing, new issue will be created for meeting next week.

This sounds sensible to me!


Dispaly problem in Firefox 61

Hi,
anybody have a display problem when you split an image in Firefox, some vertical line appear in the split image. Don't have this problem with Chrome/Safari/Webkit

Thanks

This is likely not a Firefox exclusive, but more of a responsive issue/rounding error. Some browsers will have this more than others, and certain factors exacerbate it like exact screen / element size or if you're using a retina display.
If you resize your window slightly, the line should disappear. Adding .split-cell { margin: -1px } may help, but with opacity that can leave odd borders.
Does that help?


A required class was missing while executing azure-functions-maven-plugin:0.2.0:package: DocumentDBInput

Plugin name and version
azure-functions-maven-plugin:0.2.0
Expected behavior
mvn clean package should succeed.
Actual behavior
Report "A required class was missing" error.
Detailed Logs
[ERROR] Failed to execute goal com.microsoft.azure:azure-functions-maven-plugin:0.2.0:package (package-functions) on project survey-report-apis: Execution package-functions of goal com.microsoft.azure:azure-functions-maven-plugin:0.2.0:package failed: A required class was missing while executing com.microsoft.azure:azure-functions-maven-plugin:0.2.0:package: com/microsoft/azure/serverless/functions/annotation/DocumentDBInput
[ERROR] -----------------------------------------------------
[ERROR] realm =    plugin>com.microsoft.azure:azure-functions-maven-plugin:0.2.0
[ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
[ERROR] urls[0] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-functions-maven-plugin/0.2.0/azure-functions-maven-plugin-0.2.0.jar
[ERROR] urls[1] = file:/Users/ankit.choudhary/.m2/repository/javax/enterprise/cdi-api/1.0/cdi-api-1.0.jar
[ERROR] urls[2] = file:/Users/ankit.choudhary/.m2/repository/org/eclipse/sisu/org.eclipse.sisu.inject/0.3.0/org.eclipse.sisu.inject-0.3.0.jar
[ERROR] urls[3] = file:/Users/ankit.choudhary/.m2/repository/org/apache/maven/maven-builder-support/3.3.3/maven-builder-support-3.3.3.jar
[ERROR] urls[4] = file:/Users/ankit.choudhary/.m2/repository/org/eclipse/aether/aether-util/1.0.2.v20150114/aether-util-1.0.2.v20150114.jar
[ERROR] urls[5] = file:/Users/ankit.choudhary/.m2/repository/org/sonatype/sisu/sisu-guice/3.2.5/sisu-guice-3.2.5-no_aop.jar
[ERROR] urls[6] = file:/Users/ankit.choudhary/.m2/repository/aopalliance/aopalliance/1.0/aopalliance-1.0.jar
[ERROR] urls[7] = file:/Users/ankit.choudhary/.m2/repository/org/codehaus/plexus/plexus-interpolation/1.21/plexus-interpolation-1.21.jar
[ERROR] urls[8] = file:/Users/ankit.choudhary/.m2/repository/org/codehaus/plexus/plexus-component-annotations/1.5.5/plexus-component-annotations-1.5.5.jar
[ERROR] urls[9] = file:/Users/ankit.choudhary/.m2/repository/org/sonatype/plexus/plexus-sec-dispatcher/1.3/plexus-sec-dispatcher-1.3.jar
[ERROR] urls[10] = file:/Users/ankit.choudhary/.m2/repository/org/sonatype/plexus/plexus-cipher/1.4/plexus-cipher-1.4.jar
[ERROR] urls[11] = file:/Users/ankit.choudhary/.m2/repository/org/codehaus/plexus/plexus-utils/3.0.20/plexus-utils-3.0.20.jar
[ERROR] urls[12] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-maven-plugin-lib/0.2.0/azure-maven-plugin-lib-0.2.0.jar
[ERROR] urls[13] = file:/Users/ankit.choudhary/.m2/repository/org/apache/maven/shared/maven-filtering/3.0.0/maven-filtering-3.0.0.jar
[ERROR] urls[14] = file:/Users/ankit.choudhary/.m2/repository/org/apache/maven/shared/maven-shared-utils/3.0.0/maven-shared-utils-3.0.0.jar
[ERROR] urls[15] = file:/Users/ankit.choudhary/.m2/repository/com/google/code/findbugs/jsr305/2.0.1/jsr305-2.0.1.jar
[ERROR] urls[16] = file:/Users/ankit.choudhary/.m2/repository/org/sonatype/plexus/plexus-build-api/0.0.7/plexus-build-api-0.0.7.jar
[ERROR] urls[17] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure/1.7.0/azure-1.7.0.jar
[ERROR] urls[18] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-resources/1.7.0/azure-mgmt-resources-1.7.0.jar
[ERROR] urls[19] = file:/Users/ankit.choudhary/.m2/repository/org/slf4j/slf4j-simple/1.7.5/slf4j-simple-1.7.5.jar
[ERROR] urls[20] = file:/Users/ankit.choudhary/.m2/repository/io/reactivex/rxjava/1.2.4/rxjava-1.2.4.jar
[ERROR] urls[21] = file:/Users/ankit.choudhary/.m2/repository/org/apache/httpcomponents/httpcore/4.4.5/httpcore-4.4.5.jar
[ERROR] urls[22] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-storage/1.7.0/azure-mgmt-storage-1.7.0.jar
[ERROR] urls[23] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-network/1.7.0/azure-mgmt-network-1.7.0.jar
[ERROR] urls[24] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-compute/1.7.0/azure-mgmt-compute-1.7.0.jar
[ERROR] urls[25] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-graph-rbac/1.7.0/azure-mgmt-graph-rbac-1.7.0.jar
[ERROR] urls[26] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-keyvault/1.7.0/azure-mgmt-keyvault-1.7.0.jar
[ERROR] urls[27] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-keyvault/1.0.0/azure-keyvault-1.0.0.jar
[ERROR] urls[28] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-keyvault-webkey/1.0.0/azure-keyvault-webkey-1.0.0.jar
[ERROR] urls[29] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-batch/1.7.0/azure-mgmt-batch-1.7.0.jar
[ERROR] urls[30] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-trafficmanager/1.7.0/azure-mgmt-trafficmanager-1.7.0.jar
[ERROR] urls[31] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-dns/1.7.0/azure-mgmt-dns-1.7.0.jar
[ERROR] urls[32] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-redis/1.7.0/azure-mgmt-redis-1.7.0.jar
[ERROR] urls[33] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-appservice/1.7.0/azure-mgmt-appservice-1.7.0.jar
[ERROR] urls[34] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-locks/1.7.0/azure-mgmt-locks-1.7.0.jar
[ERROR] urls[35] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-eventhub/1.7.0/azure-mgmt-eventhub-1.7.0.jar
[ERROR] urls[36] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-cdn/1.7.0/azure-mgmt-cdn-1.7.0.jar
[ERROR] urls[37] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-sql/1.7.0/azure-mgmt-sql-1.7.0.jar
[ERROR] urls[38] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-containerinstance/1.7.0/azure-mgmt-containerinstance-1.7.0.jar
[ERROR] urls[39] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-containerregistry/1.7.0/azure-mgmt-containerregistry-1.7.0.jar
[ERROR] urls[40] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-containerservice/1.7.0/azure-mgmt-containerservice-1.7.0.jar
[ERROR] urls[41] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-cosmosdb/1.7.0/azure-mgmt-cosmosdb-1.7.0.jar
[ERROR] urls[42] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-search/1.7.0/azure-mgmt-search-1.7.0.jar
[ERROR] urls[43] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-msi/1.7.0/azure-mgmt-msi-1.7.0.jar
[ERROR] urls[44] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-monitor/1.7.0/azure-mgmt-monitor-1.7.0.jar
[ERROR] urls[45] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-servicebus/1.7.0/azure-mgmt-servicebus-1.7.0.jar
[ERROR] urls[46] = file:/Users/ankit.choudhary/.m2/repository/joda-time/joda-time/2.1/joda-time-2.1.jar
[ERROR] urls[47] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-mgmt-batchai/1.7.0/azure-mgmt-batchai-1.7.0.jar
[ERROR] urls[48] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-client-runtime/1.3.0/azure-client-runtime-1.3.0.jar
[ERROR] urls[49] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/rest/client-runtime/1.3.0/client-runtime-1.3.0.jar
[ERROR] urls[50] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/retrofit2/retrofit/2.1.0/retrofit-2.1.0.jar
[ERROR] urls[51] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/okhttp3/okhttp/3.4.2/okhttp-3.4.2.jar
[ERROR] urls[52] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/okio/okio/1.9.0/okio-1.9.0.jar
[ERROR] urls[53] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/okhttp3/logging-interceptor/3.4.2/logging-interceptor-3.4.2.jar
[ERROR] urls[54] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/okhttp3/okhttp-urlconnection/3.4.2/okhttp-urlconnection-3.4.2.jar
[ERROR] urls[55] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/retrofit2/converter-jackson/2.1.0/converter-jackson-2.1.0.jar
[ERROR] urls[56] = file:/Users/ankit.choudhary/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.7.2/jackson-databind-2.7.2.jar
[ERROR] urls[57] = file:/Users/ankit.choudhary/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-joda/2.7.2/jackson-datatype-joda-2.7.2.jar
[ERROR] urls[58] = file:/Users/ankit.choudhary/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.7.0/jackson-annotations-2.7.0.jar
[ERROR] urls[59] = file:/Users/ankit.choudhary/.m2/repository/com/squareup/retrofit2/adapter-rxjava/2.1.0/adapter-rxjava-2.1.0.jar
[ERROR] urls[60] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-client-authentication/1.3.0/azure-client-authentication-1.3.0.jar
[ERROR] urls[61] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/adal4j/1.1.2/adal4j-1.1.2.jar
[ERROR] urls[62] = file:/Users/ankit.choudhary/.m2/repository/com/nimbusds/oauth2-oidc-sdk/4.5/oauth2-oidc-sdk-4.5.jar
[ERROR] urls[63] = file:/Users/ankit.choudhary/.m2/repository/javax/mail/mail/1.4.7/mail-1.4.7.jar
[ERROR] urls[64] = file:/Users/ankit.choudhary/.m2/repository/javax/activation/activation/1.1/activation-1.1.jar
[ERROR] urls[65] = file:/Users/ankit.choudhary/.m2/repository/net/jcip/jcip-annotations/1.0/jcip-annotations-1.0.jar
[ERROR] urls[66] = file:/Users/ankit.choudhary/.m2/repository/net/minidev/json-smart/1.1.1/json-smart-1.1.1.jar
[ERROR] urls[67] = file:/Users/ankit.choudhary/.m2/repository/com/nimbusds/lang-tag/1.4/lang-tag-1.4.jar
[ERROR] urls[68] = file:/Users/ankit.choudhary/.m2/repository/com/nimbusds/nimbus-jose-jwt/3.1.2/nimbus-jose-jwt-3.1.2.jar
[ERROR] urls[69] = file:/Users/ankit.choudhary/.m2/repository/org/bouncycastle/bcprov-jdk15on/1.51/bcprov-jdk15on-1.51.jar
[ERROR] urls[70] = file:/Users/ankit.choudhary/.m2/repository/com/google/code/gson/gson/2.2.4/gson-2.2.4.jar
[ERROR] urls[71] = file:/Users/ankit.choudhary/.m2/repository/commons-codec/commons-codec/1.10/commons-codec-1.10.jar
[ERROR] urls[72] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-annotations/1.2.0/azure-annotations-1.2.0.jar
[ERROR] urls[73] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/applicationinsights-core/1.0.9/applicationinsights-core-1.0.9.jar
[ERROR] urls[74] = file:/Users/ankit.choudhary/.m2/repository/eu/infomas/annotation-detector/3.0.4/annotation-detector-3.0.4.jar
[ERROR] urls[75] = file:/Users/ankit.choudhary/.m2/repository/commons-io/commons-io/2.4/commons-io-2.4.jar
[ERROR] urls[76] = file:/Users/ankit.choudhary/.m2/repository/org/apache/httpcomponents/httpclient/4.5.3/httpclient-4.5.3.jar
[ERROR] urls[77] = file:/Users/ankit.choudhary/.m2/repository/commons-logging/commons-logging/1.2/commons-logging-1.2.jar
[ERROR] urls[78] = file:/Users/ankit.choudhary/.m2/repository/commons-net/commons-net/3.6/commons-net-3.6.jar
[ERROR] urls[79] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-functions-java-core/1.0.0-beta-3/azure-functions-java-core-1.0.0-beta-3.jar
[ERROR] urls[80] = file:/Users/ankit.choudhary/.m2/repository/org/reflections/reflections/0.9.11/reflections-0.9.11.jar
[ERROR] urls[81] = file:/Users/ankit.choudhary/.m2/repository/com/google/guava/guava/20.0/guava-20.0.jar
[ERROR] urls[82] = file:/Users/ankit.choudhary/.m2/repository/org/javassist/javassist/3.21.0-GA/javassist-3.21.0-GA.jar
[ERROR] urls[83] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-storage/5.4.0/azure-storage-5.4.0.jar
[ERROR] urls[84] = file:/Users/ankit.choudhary/.m2/repository/com/fasterxml/jackson/core/jackson-core/2.6.0/jackson-core-2.6.0.jar
[ERROR] urls[85] = file:/Users/ankit.choudhary/.m2/repository/org/apache/commons/commons-lang3/3.4/commons-lang3-3.4.jar
[ERROR] urls[86] = file:/Users/ankit.choudhary/.m2/repository/com/microsoft/azure/azure-keyvault-core/0.8.0/azure-keyvault-core-0.8.0.jar
[ERROR] urls[87] = file:/Users/ankit.choudhary/.m2/repository/org/zeroturnaround/zt-zip/1.12/zt-zip-1.12.jar
[ERROR] urls[88] = file:/Users/ankit.choudhary/.m2/repository/junit/junit/4.12/junit-4.12.jar
[ERROR] urls[89] = file:/Users/ankit.choudhary/.m2/repository/org/hamcrest/hamcrest-core/1.3/hamcrest-core-1.3.jar
[ERROR] Number of foreign imports: 1
[ERROR] import: Entry[import  from realm ClassRealm[maven.api, parent: null]]
[ERROR]
[ERROR] -----------------------------------------------------
[ERROR] : com.microsoft.azure.serverless.functions.annotation.DocumentDBInput
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginContainerException

@JunyiYi Thanks for reporting. The releated PR is here: https://github.com/Microsoft/azure-maven-plugins/pull/128.
For any other users who encountered this problem, please set the the version of azure-functions-java-core to 1.0.0-beta-2 in the pom file.

Just tried to create Azure function with Java and Maven.
expected mvn clean package to succeed
got such error:
[ERROR] Failed to execute goal com.microsoft.azure:azure-functions-maven-plugin:0.2.1:package (package-functions) on project iothubsuperpoc.functions: Execution package-functions of goal com.microsoft.azure:azure-functions-maven-plugin:0.2.1:package failed: A required class was missing while executing com.microsoft.azure:azure-functions-maven-plugin:0.2.1:package: javax/xml/bind/JAXBException
In pom.xml I see that:

azure-functions-java-core is 1.0.0-beta-2
azure-functions-maven-plugin is 0.2.1

Apache Maven 3.5.3
Java version: 10.0.1, vendor: Oracle Corporation
Java home: C:\Program Files\Java\jdk-10.0.1
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

@maadtechie Currently the Maven plugin only support Java 8.

@jdneo Any plans to support Java 11? Java 8 is going the way of the dodo soon.

@nicenemo Not yet. The reason is that one of the plugin's dependency only supports on Java 8. We have created an issue at their repo but they will not support Java 9, 10, 11 very soon.
More details are here: https://github.com/Microsoft/ApplicationInsights-Java/issues/674


Esword pro combat

About the PR

Eswords can no longer be EMPd (people complained about it).
Eswords will now block bolas, syringes, spears, pies and similar projectiles.
Also fixed syringe gun having muzzle flash and syringes flying at weird angles.
Why / Balance

Bola is a hard counter to esword currently, you just throw it and mag dump a person, it's too easy. Also it makes sense that esword that can block projectiles, can block thrown items too.
Technical details

Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl: Aviu

remove: Eswords can no longer be EMPd.
add: Eswords will block bolas, spears, syringes, pies and similar stuff.

We should probably also make northstars cheaper / better because at this point they are dogshit

We should probably also make northstars cheaper / better because at this point they are dogshit

True, they should be 20-30tc.

maybe northstars could provide some sort of shock protection? IMO losing 40tc AND your shock protection makes buying northstars such a horrific choice

Ah, yes, let's undo the PR which tried to make em fair and make em even worse.

But pies and maybe bolas should be un-reflected

I feel bad for those syndies when I ensnare and magdump them.

The CL say it "will" block. Does that mean it applies the reflect chance, or will it always block? The latter seems op.
On the subject of the bola issue, I think they should need to be hit to negate, as opposed to some rng or simple immunity.

The CL say it "will" block. Does that mean it applies the reflect chance, or will it always block? The latter seems op. On the subject of the bola issue, I think they should need to be hit to negate, as opposed to some rng or simple immunity.

It always blocks.

I'm not a fan of blocking 20 spears simultaneously.

Could put it on cool down, so you block melee or thrown items, but only once per x seconds.

Could put it on cool down, so you block melee or thrown items, but only once per x seconds.

This is how melee blocking works on WD, might port it later I guess.


微信服务号在Azure上部署，SSH段80端口无法监听

Load config success
[INFO][2023-05-01 12:32:24][plugin_registry.py:23] - Plugin Selector_v0.1 registered
[INFO][2023-05-01 12:32:24][log.py:32] - [Selector] inited
[WARNING][2023-05-01 12:32:24][log.py:39] - Failed to import plugin createimg
Load config success
[INFO][2023-05-01 12:32:24][log.py:34] - [MultiChannel] Start up chatgpt on wechat_mp
[INFO][2023-05-01 12:32:24][wechat_mp_channel.py:39] - [WX_Public] Wechat Public account service start!
Bottle v0.12.25 server starting up (using AutoServer())...
Listening on http://<IP_ADDRESS>:80/
Hit Ctrl-C to quit.
[ERROR][2023-05-01 12:32:25][log.py:47] - [MultiChannel] Start up failed on wechat_mp: Couldn't listen on <IP_ADDRESS>:80: [Errno 13] Permission denied.
[ERROR][2023-05-01 12:32:25][log.py:45] - App startup failed!
[ERROR][2023-05-01 12:32:25][log.py:50] - Couldn't listen on <IP_ADDRESS>:80: [Errno 13] Permission denied.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/twisted/internet/tcp.py", line 1361, in startListening
skt.bind(addr)
PermissionError: [Errno 13] Permission denied
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "app.py", line 42, in main
start_process(channel_type[0], args.config)
File "app.py", line 23, in start_process
raise e
File "app.py", line 20, in start_process
channel.startup()
File "/home/linyar/bot-on-anything/channel/wechat/wechat_mp_channel.py", line 42, in startup
robot.run()
File "/home/linyar/.local/lib/python3.8/site-packages/werobot/robot.py", line 686, in run
self.wsgi.run(server=server, host=host, port=port)
File "/home/linyar/.local/lib/python3.8/site-packages/bottle.py", line 767, in run
run(self, **kwargs)
File "/home/linyar/.local/lib/python3.8/site-packages/bottle.py", line 3175, in run
server.run(app)
File "/home/linyar/.local/lib/python3.8/site-packages/bottle.py", line 3026, in run
return sa(self.host, self.port, **self.options).run(handler)
File "/home/linyar/.local/lib/python3.8/site-packages/bottle.py", line 2930, in run
reactor.listenTCP(self.port, factory, interface=self.host)
File "/usr/lib/python3/dist-packages/twisted/internet/posixbase.py", line 495, in listenTCP
p.startListening()
File "/usr/lib/python3/dist-packages/twisted/internet/tcp.py", line 1363, in startListening
raise CannotListenError(self.interface, self.port, le)
twisted.internet.error.CannotListenError: Couldn't listen on <IP_ADDRESS>:80: [Errno 13] Permission denied.
^CException ignored in: <module 'threading' from '/usr/lib/python3.8/threading.py'>
Traceback (most recent call last):
File "/usr/lib/python3.8/threading.py", line 1388, in _shutdown
lock.acquire()
KeyboardInterrupt:
linyar@ChatBotV1:~/bot-on-anything$ python3 app.py
Load config success
[INFO][2023-05-01 13:10:42][plugin_registry.py:23] - Plugin Selector_v0.1 registered
[INFO][2023-05-01 13:10:42][log.py:32] - [Selector] inited
[WARNING][2023-05-01 13:10:42][log.py:39] - Failed to import plugin createimg
Load config success
[INFO][2023-05-01 13:10:42][log.py:34] - [MultiChannel] Start up chatgpt on wechat_mp
[INFO][2023-05-01 13:10:42][wechat_mp_channel.py:39] - [WX_Public] Wechat Public account service start!
Bottle v0.12.25 server starting up (using AutoServer())...
Listening on http://<IP_ADDRESS>:80/
Hit Ctrl-C to quit.
[ERROR][2023-05-01 13:10:42][log.py:47] - [MultiChannel] Start up failed on wechat_mp: Couldn't listen on <IP_ADDRESS>:80: [Errno 13] Permission denied.
[ERROR][2023-05-01 13:10:42][log.py:45] - App startup failed!
[ERROR][2023-05-01 13:10:42][log.py:50] - Couldn't listen on <IP_ADDRESS>:80: [Errno 13] Permission denied.

AZURE已经打开了80端口
但是依旧无法监听

用sudo python3 app.py,  你的错误是权限不够


Feat/gate placement

This is the first change required for #226.
An intermediate type, GatePlacement, is added as a drop-in replacement for the AbstractGate type. It replaces the use of AbstractGate in QuantumCircuit. This means that all code downstream of  QuantumCircuit now interacts with GatePlacement instead of AbstractGate,
There are two parts remaining.

The GatePlacement interface needs to be trimmed to not include the gate interface. (and all downstream code of QuantumCircuit needs to be updated accordingly)
We can remove the connection information from the AbstractGate interface (and all the gates).

We are busy with talks about if this is indeed the way forward. I marked the PR as a draft to signify that even though all the comments will soon be addressed, the PR will not be merged as soon as possible.

I've rebased to the latest changes in main, and I've done the rename (AbstractGate -> AbstractGateSymbol, GatePlacement -> Gate).
@alireza-najafi @caffeine-storm The code is ready for another look.

I've updated this PR with the latest changes in master. I didn't fix the coverage yet. I'll do that next.


New test for a possible bug with the 'map' function of the Stack

I think that there is a bug with the map function of the Stack. If I'm not mistaken, this test should pass but currently it is failing.
For some reason the iterator of the map function of the Stack is reversed, which IMOO can very inconvenient for performing certain kinds of operations. If this is intentional, I think that there should be a test for this uncommon behaviour and it should also be explained in the documentation.
An example of how this could be annoying:
Imagine that you have a stack where you are keeping track of the latest moments when a user has switched directions in a game, the Stack would look something like this:
const directions = Stack.of(
  { direction: 'LEFT', moment: 95 },
  { direction: 'UP', moment: 90 },
  { direction: 'RIGHT', moment: 80 },
);

Then, you also have in another variable where you keep track of the current "moment", like this:
const currentMoment = 100;
And now we want to get the "vectors" of the directions. Basically we would like to get a Stack like this:
const expectedVectors = Stack.of(
  { direction: 'LEFT', len: 5 },
  { direction: 'UP', len: 5 },
  { direction: 'RIGHT', len: 10 },
);

I would expect the following code to return a Stack like the one above:
const vectors = directions.map(({ direction, moment }, idx, iterator) => {
  const nextMoment = idx === 0 ? currentMoment : iterator.get(idx - 1).moment;
  return { direction, len: nextMoment - moment };
} );

Unfortunately, since the iterator is reversed, the previous code will return a Stack like this:
Stack.of(
  { direction: 'LEFT', len: -5 },
  { direction: 'UP', len: -10 },
  { direction: 'RIGHT', len: 20 },
);

In other words, I find it very weird that the following 2 lines are not equivalent:
Stack.of('a','b','c').map((x, idx, iterable)=> iterable.get(idx) + (iterable.get(idx + 1) || '')).toArray();
Stack.of('a','b','c').toArray().map((x, idx, iterable)=> iterable[idx] + (iterable.get[idx + 1] || ''));

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

@facebook-github-bot-1 I just signed up the Contributor License Agreement, sorry about that. Thanks!


Update Gradle plugin to v.1.5.0.

let's wait with this until https://github.com/square/spoon/issues/300 is resolved

@ligi Sure. No problem.

@ligi Is Spoon ready meanwhile?

looks like they released it right one day after @ligi suggested to wait for the release :)

Yes it is out now - we might now even go to 2.0.0-alpha3 - instant-run is pretty nice - the only draw-back I see currently is that you have to get a newer version from sdkmanager through jitpack.io

I am using alpha and beta version locally, too. I would not commit them in this case, though.

Was just saying as @cbalster was suggesting it here https://github.com/criticalmaps/criticalmaps-android/pull/134 and I blocked at this time - but I think with alpha3 it got pretty useable and I gained some confidence in this plugin

Rebased.

:+1:


Significant Lagging after Picking Up Item

Description
On picking up item from tree, game significantly lags and ultimately freezes
How To Reproduce
Steps to reproduce the behaviour:

Go to tree
Click 'E' to interact
Experience lags and slow down

Expected Behaviour
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional Context
Occurred after attached merge, changing how player sprite is updated when holding item: https://github.com/UQcsse3200/2024-studio-3/pull/225
Suspected error caused by this merge due to known functionality of other interaction components from previous main versions
Temporary Fix
None at present

After investigation, wasnt working before the given merge either, @IWontCauseAFuss could it be related to your changes in the inventory display?

More info, wasnt working before that merge either, @Michael could it be related to your changes in the inventory display?

Error found, in PlayerActions some old cod meant to freeze player on some interactions became uncommented, was trigger event in KeyboardPlayerInputComponent that messed with movement. Submitting fix, just commenting out trigger


increase the timeout for service discovery

Increase the timeout for service discovery, from 60 seconds to 3 minutes. I was hitting this while launching apps on the iOS simulator (on a slower macbook).
@cbracken @xster

ptal - reworked this to remove the timeout on service port discovery

Does it indicate that something is happening while waiting? If it's going to take 3 minutes some times then we're gonna need a spinner if we don't already have one...

Does it indicate that something is happening while waiting? If it's going to take 3 minutes some times then we're gonna need a spinner if we don't already have one...

I agree; I don't know about this case. Spinners should already be showing for our long tasks at app launch, but I don't think we have all the cases covered we should.


A11y fix nav

🍿  PR to primarily fix #535
🎟  Also fixes #652 & #12
Note: Jest-axe landmarks rule is now disabled. We're going to have to be a bit more careful about how we use semantic HTML  now.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


pull request for e2e test at 12/06/2016 03:26:01

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Small image drawing

Small images are resized beyond their size. As an result of this, SVGSaver thinks that they were smaller than their original size. He draws line on a bigger width and height. IndexOutOfRangeException is triggered because bound are smaller.

Fixed with setting proper image stretch.


Description of Checkbox is missing in menu on website

http://www.material-ui.com/#/components/checkbox
is a link that is missing in the drawer on the left side of the website.
I found "Checkbox" only by reading the chapter about "List".

How is that missing? I see it under the switches section.

Notice that the new search bar on the next branch should help a lot finding what you are looking for.


Year navigation

Before Submitting: Double-check that you are running the latest version of the plugin. The feature might have already been included 😄

up-to-date
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I'm trying to backfill an old online diary into obsidian and match the date stamps from the past with daily notes. In the plugin, I can only go back one month at a time, and I'm trying to go back twenty years.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I want to click on the year and input the desired year to go back to it, or else some other way to navigate to a specific month and year.
Describe alternatives you've considered
There might be a way to do this outside of the calendar plugin by editing some hidden code to set a date for the daily note, but I haven't yet learned of it.
Additional context
Add any other context or screenshots about the feature request here.
none

Well, in checking back at my database I see now that the notes aren't appearing in the calendar. I moved them out of my daily notes folder and into their own folder specific to the archiving project, which seemingly removes them from the calendar. I'm not sure I can get to what I'm looking for at the moment...

The Obsidian 42 - Jump to Date plugin can help with the first issue. https://github.com/TfTHacker/obsidian42-jump-to-date  Once you "arrive" you can use Calendar's "reveal active note" command to show the correct month in the calendar.
On the other issue, how are you structuring your archive? Are the archive folders subfolders of your daily notes folder?

@oldjove thank you for this. I think it's my structure. I have daily notes in a folder within my vault. I'm collecting notes and ramblings from different platforms and incorporating them into my vault. I decided to create folders to contain the specific notes from these platforms, but in doing so I've removed those notes from the daily notes folder and seemingly therefore the calendar?
Ideally, the calendar might disentangle from daily notes and there would be some way of stamping a meta time tag to anchor the note within my personal chronology. I'd like to evenutally recapitulate the goings on in my life at certain times in the past (10 or more years ago), like I will be able to do moving forward with the daily notes.

@trijste no worries. A couple of thoughts on this -- not sure if they're helpful. What I do is assign a date format in Daily Notes as follows: YYYY/YYYY-MM/YYYY-MM-DD -- so every daily note is automatically created in folders for the appropriate year and month. If the folder doesn't yet exist, this will create it. This makes all the notes much easier to manage, especially because mine go back many years. If I want to link other non-daily notes to particular dates, I just enter the relevant date as a link. This can be easily achieved using the Natural Language Dates plugin. You just hit the shortcut key (default is @) and then type the relevant date in, duh, natural language. Then on the date in question you can look at the backlinks to see all the notes referring to it.
Another thought: if you want to track where notes and ramblings have come from, why not use tags rather than folders?

@oldjove Thanks again! I've converted to your style. I am using tags as well, but am still developing some methodology to keep it organized. Appreciate you writing here. I'm closing this issue :)

You're very welcome!


Issues/28197 openthread build flags

Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/28197
Using generator expressions to fetch include directories, compile
options, and compile definitions from Zephyr interface.
This fixes #28197 and is a safer approach because the use of a generator
expression ensures that properties added after openthread libraries are
defined will also be included.
zephyr_get__for_lang() has a limitation that it will not be able
to return flags added to zephyr_interface after the function has
returned.
It also disables unused openthread libraries from Zephyr build.
This removes the need of setting Zephyr compile flags on openthread
targets that are not used by Zephyr.
This reduces number of build steps and build time significantly.
Before, 780 steps:
time ninja -Cbuild
[708/708] Linking CXX executable zephyr/zephyr.elf
135.08user 30.24system 0:46.21elapsed 357%CPU

After, 480 steps:
time ninja -Cbuild
[480/480] Linking CXX executable zephyr/zephyr.elf
84.02user 18.92system 0:30.72elapsed 335%CPU

@galak @rlubos please review.


How i can use scanQRCodes function ❓

Question
I cant use scanQRCodes function
What I tried
import { scanQRCodes, QrCode } from 'vision-camera-qrcode-scanner'

import AppLoading from '@src/features/AppLoading'

interface Props {
  postingJobId: number
}

const RNCameraScanQR = ({ postingJobId }: Props) => {
  // const dispatch = useDispatch()
  // let checker = false
  // useFocusEffect(
  //   useCallback(() => {
  //     checker = false
  //   }, [])
  // )

  const devices = useCameraDevices()
  const device = devices.back
  const format = useCameraFormat(device)
  const [permissons, setPermissons] = useState(false)
  const [qrCodes, setQrCodes] = useState<QrCode[]>([])

  const requestPermissons = async () => {
    const newCameraPermission = await Camera.requestCameraPermission().catch((error) =>
      console.warn('camera error:', error)
    )
    if (newCameraPermission === 'authorized') {
      setPermissons(true)
    }
  }
  useEffect(() => {
    if (!permissons) {
      requestPermissons()
    }
  }, [permissons])

  useEffect(() => {
    ;(async () => {
      await Camera.getAvailableCameraDevices()
    })()
  }, [])

  const frameProcessor = useFrameProcessor((frame) => {
    'worklet'

    const qrcode = scanQRCodes(frame)
    console.log('Processing Frame:', qrcode)

    setQrCodes(qrcode)
    runOnJS(setQrCodes)(qrcode)
    // runOnUI(setQrCodes)(qrcode)
    console.log('qrCodes', qrCodes)
  }, [])

VisionCamera Version
2.9.3
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

this function does not exist in the library itself, you need to install a plugin to be able to scan qrcodes,
https://www.npmjs.com/package/vision-camera-qrcode-scanner

i am facing same issue, this is how i resolve based on example vision-camera-code-scanner
`
import {
BarcodeFormat,
scanBarcodes
} from "vision-camera-code-scanner";
import { runOnJS } from "react-native-reanimated";
const onQRCodeDetected = useCallback((qrCode) => {
console.log(qrCode)
}, [])
const frameProcessor = useFrameProcessor((frame) => {
'worklet'
const qrCodes = scanBarcodes(frame, [BarcodeFormat.QR_CODE], { checkInverted: true });
if (qrCodes.length > 0) {
runOnJS(onQRCodeDetected)(qrCodes[0])
}
}, [onQRCodeDetected])`

Hey!
JFYI; VisionCamera V3 now includes a QR/Barcode Scanner! 😍  Check out the CodeScanner Documentation 🚀
Try upgrading to V3 if you can, it has a much more stable yet flexible foundation.
If you appreciate me dedicating my free time to improving VisionCamera and implementing features like the Code Scanner, please consider sponsoring me on GitHub 💖 to show your support.


Meeting-2022-11-07

Discussed offline with @Peyman-N, @UlrikeS91 , @apdavison and @lzehl

@apdavison or @UlrikeS91 feel free to merge


Bump prod CLI release for 0.8.1

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


add scraping time info / display it on the UI

The product details response has been extended with the scraping time information:
it can be found in the root of the response json:
"scrapingTime": "1535642080264"

The value is the timestamp when the data has been successfully refreshed for the provider.
This needs to be displayed on the ui below the curl command and above the details table

Implemented by #42


Add tests for Palom

The test data is 60MB. Is there anything smallee that we can use?

Some of these tools tend to fail on smaller images. @CameronFRWatson is that the case for palom?

@bgruening @alliecreason It's difficult to find .svs images under 30MB that still have features for the tool to align, and they can't be cropped due to the proprietary nature of the format so I can't make these ones smaller.

A bit of background.

Large test data makes handling github repos harder, especially for new contributors with slow internet.
Test data will be uploaded to the TS, where is also waste space - with 8000 tools this is a considerable amount.
test-data will also be downloaded into you Galaxy server where you install your tool - and this with every version again - so every version of palom will have 60MB of data on your Galaxy server --> they are actually used for some minor use-cases

We know this is not ideal and we have discussed for years now that it would be ideal to define in the Galaxy XML a URI to the data. So that planemo downloads the data from a URI and does not assume it is stored in git. However, we have not done this yet and working around these issues with various hacks. See also https://github.com/galaxyproject/planemo/issues/1268 for a recent conversation.
Another aspect that I wanted to point you to is that you can define tests that are failing. Failing in a specific way is good, we know the tool is working in theory and fails with a defined way and this you can define in the XML (https://docs.galaxyproject.org/en/latest/dev/schema.html#id92).
I don't know what the best way is before we have the URI feature. Feel free to merge, it looks good to me besides the large test data.


Added support for Groupdate 4

Hey @codesnik, this adds support for Groupdate 4 (and makes the existing tests pass). I use a similar approach in the ActiveMedian gem. https://github.com/ankane/active_median/blob/8d8313a35117643c60c317c54a0b831457cf0547/lib/active_median/model.rb#L54
Without this, calculate-all returns times instead of dates in keys. This also fills in missing data points.

And here's the process_result code: https://github.com/ankane/groupdate/blob/c74afce40d526cf849a364542ed89130d631a01a/lib/groupdate.rb#L19
It's essentially a no-op if Groupdate wasn't used.

Thanks @ankane. This works pretty good!
So after testing your PR my code starts working again and the format -options isn't ignored anymore. 🎉
MyModel.group_by_week(:created_at, format: "%W").calculate_all(:average_something)

So give my Thumbs-Up for merging this one @codesnik

sorry, guys, I have no time and/or will to even review PR's to that little gem anymore. Maybe I should just pass ownership to someone else.


Text composables: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-text-composables?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-text-composables#3
In which task and step of the codelab can this issue be found?
[Build a simple app with text composables]
[4. What is Jetpack Compose?] and following...
Describe the problem
Instruction: Composable function names ... The Compose function:...  NOT a nouned preposition: TextFieldWithLink()
Usage one step later (and also in the introduction video):
@Composable
fun BirthdayGreetingWithText(message: String) {
Text(
text = message
)
}
Contradicting....
Steps to reproduce?
Just check the documentation / codelab
Versions
(irrelevant)
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @PeterS-ZH
Thank you for writing in and bringing this issue to our attention.
The lesson has been updated to address it and the revised lesson will be republished again in the near future.
Hope you are enjoying the course!
Regards.


Introduced Lotus::Action::Params#get

What
Introduced Lotus::Action::Params#get as a safe way to access (nested) attributes.
Why?

Developers can safely access nested params without worrying of NoMethodErrors that can be caused by the subscriber notation. Eg params[:address][:street], if :address isn't sent, the action will blow up. The lack of this new suggested API (#get) will force developers to write defensive code to access nested attributes.
This serves as supporting feature for Lotus::Helpers' form helper. In case of param validation failures, a form should be able to be rendered again and keep the current values.

Usage
require 'lotus/controller'

module Deliveries
  class Create
    include Lotus::Action

    params do
      param :customer_name
      param :address do
        param :city
      end
    end

    def call(params)
      params.get('customer_name')   # => "Luca"
      params.get('uknown')          # => nil

      params.get('address.city')    # => "Rome"
      params.get('address.unknown') # => nil

      params.get(nil)               # => nil
    end
  end
end

Coverage increased (+0.01%) to 99.02% when pulling e9e8d872045945bc3f33911479af7e427eb668c5 on params-get into 1bbd4f14e318516c14cf1391a2b92a155156bef0 on master.

Coverage increased (+0.01%) to 99.02% when pulling e9e8d872045945bc3f33911479af7e427eb668c5 on params-get into 1bbd4f14e318516c14cf1391a2b92a155156bef0 on master.

Coverage increased (+0.01%) to 99.02% when pulling e9e8d872045945bc3f33911479af7e427eb668c5 on params-get into 1bbd4f14e318516c14cf1391a2b92a155156bef0 on master.

Coverage increased (+0.01%) to 99.02% when pulling 95e524651bde8ceda9957dddaf3d21c87f612e51 on params-get into 1bbd4f14e318516c14cf1391a2b92a155156bef0 on master.

Coverage increased (+0.01%) to 99.02% when pulling cf2a1cbdbcd6dc49d0d05b25e0481735470913d2 on params-get into 1bbd4f14e318516c14cf1391a2b92a155156bef0 on master.

Why we only use string for get a key?
I :heart: this approach...
params.get('customer_name')  # => "Luca"
params.get(:customer_name)  # => "Luca"

@vyper hash is nice but it does not represent the nested attributes very well. For that very reason, I think we should stick to string.

@jodosha :+1:


Name is already in use

Hi,
I've been using the name rsbackup for a similar backup tool since 2010. https://www.greenend.org.uk/rjk/rsbackup/
May I suggest that the more recent halimath/rsbackup be renamed, to avoid confusion?

@ewxrjk : I do understand your point and I agree, that the name has a lot of potential for confusion. Coming up with a good name, that describes a tool's purpose is a pretty hard challenge. Finding such a name that is unique on the internet is almost impossible. That's why (at least from my understanding), services like GitHub prefix repos with username/organization to provide uniqueness at the cost of a namespace (btw, there are 14 repos named rsbackup on GitHub).
I'm not making promises but I will think about a different - hopefully more unique - name.

bq. I'm not making promises but I will think about a different - hopefully more unique - name.
Thankyou.


Implement Builders RFC 0008: Jammy Builders

Describe the Enhancement

A set of stacks based on the Ubuntu 2022.04 LTS (Jammy Jellyfish) will be maintained by Paketo, per stacks RFC 0004. Paketo plans to distribute builders based on these stacks, per Builders RFC 0008.
Possible Solution

[ ] builder.toml that uses Jammy Full stack and doesn't include any buildpacks
[ ] smoke tests
[ ] full repo automation (see https://github.com/paketo-buildpacks/github-config/tree/main/builder)

Motivation

The RFC has been accepted. Time to implement!

Jammy full stack is live.


Deletion of blocks

Duplicates

[X] I have searched the existing issues

Summary 💡
Being able to press the Del key, or right clicking on the block & selecting "Delete" should delete the block. Neither options currently exist.
The blocks that depend on the one that's deleted can be disconnected from the rest before it, but the workflow must not be able to run while it's disconnected.
Start & end blocks cannot be removed, and a toast pop-up should warn of that.
Examples 🌈
No response
Motivation 🔦
Deleting blocks is necessary in case the wrong block type is added by accident, or when a user wants to reset a block's settings.

@itsWindows11 try backspace :)

Backspace isn't a well known shortcut for deleting items, maybe characters but a block isn't a character 🤷♂

Backspace isn't a well known shortcut for deleting items, maybe for characters but a block isn't a character 🤷♂

I think it just comes like this by default from react-flow because of macs, most don't even have a delete button 😆
With that being said, i do believe we can do better with handling this. I will mock something up for the context menu

I'm closing the issue as it works, the UI just does not indicate to the user what are the shortcuts so can be confusing if you used to windows. Instead i have created an issue to address this by better UI on this issue https://github.com/softflow24/data-river/issues/136


freopen() not implemented

NOTE: Low priority to implement this as we can modify stdin etc, anyway in GCCLIB

Fixed in F0024


go-dork: init at 1.0.2

Description of changes
Dork scanner
https://github.com/dwisiswant0/go-dork
Related to https://github.com/NixOS/nixpkgs/issues/81418

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

go-dork


kyverno policies

Addresses the following issues: openebs/openebs#3385
Signed-off-by: Nivedita Prasad<EMAIL_ADDRESS>Steps for installation & testing.
1.Install Kyverno using YAMLs or Helm in the cluster.
2. Apply Default kyverno policies for OpenEBS using kubectl apply -f
3. After that install OpenEBS kubectl apply -f https://openebs.github.io/charts/openebs-operator.yaml
4.Testing steps:
Check the bd kubectl get bd -n openebs
And, create cStoor pools using the above blockdevices.

@Nivedita-coder can we update the PR description with details like how to deploy the kyverno and what steps has been followed to tests these policies.

Done!

Merged into openebs/charts repo at https://github.com/openebs/charts/pull/242


chore: replace GitHub Actions deployment with Vercel

Use Vercel to run deployment builds instead of GitHub Actions. Nix is mostly useful for my dev environment, seems like Vercel has things covered on their end, hopefully...
This removes the GitHub Actions workflow definition and replaces the environment variables for the commit definition in the footer with ones defined by Vercel. If this key feature works, then the pull is free to be merged.

Confirmed working, Vercel project settings have been updated to properly run builds and comment each version on each commit and on GitHub PRs.


The weights you upload

Thanks for your generous open source. I just wanna know if the weights you offered is only for the "fpn". When I turn the "fea_mode" to "unet", there are errors for structure of network.

That's it! We only experiment with FPN.


[📚] App check installation doc mistake

https://firebase.flutter.dev/docs/app-check/overview/#installation
There after the paragraph

For legacy package imports, place the following ignore comment to hide Dart analyzer warnings:

I would expect to find these two lines
// ignore: import_of_legacy_library_into_null_safe
import 'package:firebase_app_check/firebase_app_check.dart';

But the ignore part is cut out

Also, I believe that necessary activate call step is missing from the instructions

I can help with this!

how does it work here? When one asks for assistance then is he forgotten?
https://github.com/FirebaseExtended/flutterfire/issues/6310


Branding

Added favicon and switched links back to blue.

Awesome!  Thanks!
Any chance you have a copy of the favicon.ico with a transparent background?  This one has a white background while the main macchina website is transparent.

@adamvoss good catch, I just added the transparent favicon. Thank you!

Hmm, something is up with that.  It does not render nearly as cleanly as macchina.cc:

Looking at the .ico in my file browser I don't see the same green though...

Oh that's odd, I'll try exporting it again and see if it changes how it renders. I wasn't seeing this on my end, but it might be something with the file.

Since there was an vector version of the logo from the first branding PR, I used that with https://realfavicongenerator.net (which I have used successfully on other projects) to generate a new favicon.ico.
Thanks for this!

That works for me. Thanks for making the adjustment on this @adamvoss!


refactor(shared): use spread syntax replace split

Use the benchmark to test.
Mac OS X 10_14_0
Node v10.16.0
set#split x 326,627 ops/sec ±1.36% (86 runs sampled)
set#spread x 714,082 ops/sec ±0.46% (91 runs sampled)
Fastest is set#spread

Chrome/76.0.3809.132
set#split x 287,831 ops/sec ±3.21% (57 runs sampled)
set#spread x 714,024 ops/sec ±1.01% (60 runs sampled)
Fastest is set#spread

At this point, wouldn't it just be faster just to start off with a single array that's got all the values as opposed to having 3 arrays spread into one?

This is for sure, but I'm not sure if the author has classified. Improvements were made while maintaining the author's original intent.


ZTGL implementation of zetaml is ahead of this repo

Update zetaml repository to how it is in the ZTGL repo.

Fixed with commit a80a8f334a84f007e576121dbdcb87a76108e5d0. Issue closed.


wget install fails - it requires curl to be installed

When I used the wget install command that is on the Install md file, the install failed because I didn't install Curl first.  The salt state to install kubectl requires curl.
We need to remove the wget install option and add a note that they must install Curl (if not already installed).
[...]
  {% set kubectl_snap_install = False %}
  {% set kubectl_path = '/usr/local/bin/' %}

  {% if kubectl_binary_install %}
    {% set kubectl_prefix = 'https://storage.googleapis.com/kubernetes-release/release/' %}
    {% set version = salt.cmd.run('curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt') %}    <======================
    {% if grains.os_family == 'MacOS' %}
      {% set kubectl_version = version + '/bin/darwin/amd64/kubectl' %}
    {% else %}
      {% set kubectl_version = version + '/bin/linux/amd64/kubectl' %}
    {% endif %}
[...]

Both methods are supported now


Grafana: fix queries of block cache hit panel

Signed-off-by: Liu Cong<EMAIL_ADDRESS>
What problem does this PR solve?
Fix the wrong block cache hit rate display in Grafana.
In Grafana, if we choose one specific instance in Grafana/tikv-detail, then the data of Block cache hit panel will be not right, the reason is the queries missed part of the filters by instance.
What is changed and how it works?
Add filter conditions to responsive Grafana queries.
Related changes
This is a related fix of https://github.com/pingcap/tidb-ansible/pull/1312
Check List
Tests

No code

Release note

Grafana: fix queries of block cache hit panel

/rebuild

/merge

/merge


Update pre-commit dependencies

[x] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1

What
Some pre-commit dependencies are too old and URLs are not valid any more.

Note that flake8 6.0.0 is not used yet as many of the additional_dependencies are not ready to use it yet.
Bump python version 3.8 in GitHub Actions so flake8 5.x can be used.
Add a few basic pre-commit hooks.

Why
Need to keep the tools updated.
How
Bump versions and fix URLs.
Test
Locally tested.

I can rebase on top of this change.


Update travis.yml to use new OS X image

Fixes https://github.com/libgit2/objective-git/issues/485
@joshaber you have to turn on travis for this repository, I couldn't do it as it requires admin access:
You can simply go to https://travis-ci.org/profile/libgit2 and add the checkmark for objective-git.
The build output can be seen at https://travis-ci.org/nerdishbynature/objective-git/builds/70506107

:metal:


Unnecessary constraint in package.json breaks dependencies in node < 0.10

What is the point of dropping support for v0.8? Got dependency errors for the simplest library (that needs to work in legacy system running older Node.js environment) just because of one of the deps (temp) depends on os-tmpdir. See no reason to break it  - a mistake or by intention?
https://github.com/sindresorhus/os-tmpdir/blob/master/package.json#L13

Node.js 0.8 is ancient and very few still support it. That being said, this should still work. You're probably just having problems with ^ versions. Make sure to upgrade npm to latest in your CI.

@sindresorhus Ancient but nothing much have happened with OS filesystems since Node 0.8, so therefore no point of locking it like that. Latest NPM not working no.

https://github.com/sindresorhus/os-tmpdir/blob/b2b1e366bb04a628372a0b86c271f4890b9b0527/package.json#L13 is not enforced and only result in a warning.


broken link detection does not handle case sensitivity and shortcut references

First, marksman treat link labels case sensitive.
For exmaple, the following markdown triggers a false positive warning.
[A][]

[a]: https://example.com

According to John Gruber's canonical description of Markdown's syntax,
link labels are case insensitive.

Link definition names may consist of letters, numbers, spaces, and punctuation — but they are not case sensitive. E.g. these two links:

[link text][a]
[link text][A]

are equivalent.

Second, marksman does not recognize shortcut reference links.
For example, the following markdown does not trigger any warnings.
[b]

Shortcut reference links were introduced by Gruber himself in a beta version of Markdown.pl, but never included in the official syntax description.
Most popular markdown parsers support shortcut referneces.
Also, CommonMark includes shortcut reference links in its specification.
; marksman --version
1.0.0-19de239

Thanks for reporting @weakish! Case sensitivity in link resolution should be fixed; I’ll look into it.
Re: shortcut links, shortcuts  a-la [foo] work when it comes to go-to definition, completion, etc (modulo case-sensitivity). There’s no diagnostic for shortcut links (which is IIUC the problem you mentioned above) because generally you can’t tell if something is supposed to be a shortcut link or not: it’s a shortcut link when there’s a corresponding link definition; otherwise it’s just text, e.g.:
* [TODO] Do A
* [DONE] Do B
* [REF] Do C

[REF]: url.com

If we emit diagnostics for every [foo] without a matching link definition [foo]: bar it’d create quite a bit of noise.


Feature #9674 - added checking High Alchemy value on 'Guide Prices'

IMO this seems quite a bit over-engineered. If we wanted a WidgetBuilder it should go into the widget API too. What do you think @abextm ?

My goal was to create code that is self explanatory.  Most of the plugins have messy code with a lot of comments and the code is really hard to understand. I could make this without WidgetBuildet but isin't it more readable now? I didn't make this class global becasue this is my first PR and I dont want to mess with actual API if I don't know that you will like this approach.

I think that the widget builder simply doesn't belong here. We discussed earlier on Discord, and to make a more proper widget builder would require some extensive api/injector changes which is out of scope here. I think you can just replace you usage with a method.

WidgetBuilder should be just part of Widget's api, which is an internal change and should be blocked on Widget getting an api that makes sense in the first place. Just call methods like every other plugin.
The GuidePrices and PriceCalculator classes should just be part of the plugin, they serve no function being split out imo
You are misusing Widget::name; it is for menuops, which your widgets do not have. Just keep a reference to the dynamic widget you create and update it
You should cache DecimalFormatters - they tend to allocate alot of extraneous garbage

I'll leave my comment from discord here. It's a lot of code when you could just move the ge price widget with widthmode = 1. Then you make 1 widget and sum the prices.

@Adam- @abextm @raiyni based on your feedback I made necessary changes. I'm looking for your feedback :)

What is up with all of the BigDecimal usage? The alch values are both 1) never a decimal, and 2) always fit in long

@Adam- you have right, this wasn't necessary. Too much BigDecimals at my work :(

I replaced it with Long :)

The code should be structured so that the user can flip the config value on/off regardless of if the price checker is open or not and have it update accordingly

Can you explain this more? I don't get it right now :(

It means if you have the price checker open and flip the config value on, it should work.

What's the state of this PR? Is it ready to be re-tested?

It still doesn't work right if you flip it on/off with the price checker open. Maybe someone else can take it over since it looks abandoned.


readme writing changes for readability

I have made writing changes in the Readme file to improve readability.

Thanks so much for your contribution @tamkylet!


bug138/ICParser rework

Now handles ) very well. Needs Test + ~Refactoring + Deletation of old parser~.
Before testing, please execute ICHelper resetMailData. It should flawlessly parse all mails in your inbox, from different folders. But it takes some time (~20 second?).
Closes #138

It seems to be working great :D. Good job dude <3


make google_app_id non translatable string

Description

Mark google_app_id string as non translatable in order to build scripts doesn't generate errors about it.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

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

Thanks @DrMoriarty


Support Network CIDR Calculations

what

Calculate the desired network CIDRs for each account

why

Ensure each account does not have overlapping CIDRs

 Allocate Subnets


Show notes are not displayed

Hello!
I've noticed that the show notes of an episode where not being displayed when I went directly to the episode page.
How to reproduce

Open a browser tab and go to https://ladybug.dev/it-doesnt-have-to-be-crazy-at-work
Scroll down

What did I expect?
To read the show notes
What happened instead?
I only found the episode description but without the show notes
Screenshots
https://share.getcloudapp.com/Z4u5RWlo

This was also reported in https://github.com/ladybug-podcast/ladybug-website/issues/25#issue-548777961
I would like to help to solve this issue, but I'm a total newbie in web development; so, any help or hints on what to do would be appreciated.
To compare both cases (with show notes vs no show notes) I opened the "inspect element" menu, and this is what I found:
Elements
A whole section is present when show notes are shown (from sponsor to transcript, thick blue vertical line), but missing in the other case

Maybe the button is not relevant, but is something present when the show notes are not visible

Sources
Three .js files are listed when show notes are visible (big blue dots), but absent when not

The files are listed as inside a subdirectory when show notes are not visible. I'm not sure if this is relevant.

@semiramisCJ  Thanks for the interest and for informing me about the original issue!
I think you were in the right direction. This bug was confusing! The first request included the show notes, but as soon as the JS script was executed, it removed this section 🤷 to make it more challenging, this bug only happened in production and not development :-/
I investigated a bit and noted that this bug was coming from Gatsby so I followed their recommendation. Here is the PR to attempt to fix it
https://github.com/ladybug-podcast/ladybug-website/pull/71
Hope it helps!


Change return type of MelodicFigure::total_diatonic_change()

total_diatonic_change() should return a DiatonicInterval instead of a MidiByte. This requires implementing Ord for DiatonicInterval, among other things.

This isn't worth it. The dependencies all assume the MidiByte and I see no reason to change them too.


Reaction yield greater than 100%

Theoretically speaking, the percent reaction yield is supposed to be no more than 100%. However, due the impurities and measurement errors, it can go above 100%, e.g. https://open-reaction-database.org/client/id/ord-0624d5800c4b4df5ab74722e43784236#provenance. (I tried to look at the data from https://github.com/nsf-c-cas), but I was not able to find the right data source.
I am keeping this "problem" untouched for now. But please let me know if you have a different opinion. Thanks.
@skearnes @connorcoley

Does this actually create a conflict with other aspects of the system, e.g., a validation error?

No, everything works well. It passed the validation checking in validations module.

I believe that since the ORD is meant to capture what was performed & measured, it's okay if a reported yield exceeds 100%. We know there is error/uncertainty associated with all of these calculations


setNickname(server, nickname, user) not changing nickname

I have been trying to change a nickname with my bot however
client.setNickname(message.server, 'nickname', message.author);
has no effect.
I have got the bot to respond to users, this is the only thing that isn't working.
I am using the latest version of node (v6.3.1) and have installed discord.js.

Note that there are two different permissions for changing nicknames: changeNickname for the client's nick and manageNicknames for everyone else's. Are you sure your bot has these permissions?

The bot has both permissions, however the .catch(console.log) error is "Privilege is too low..." it says forbidden: true,
Is it something to do with my being the server admin?

Yes. If the bot's highest role is lower or the same as your highest role, you guys can't do stuff to each other.

Ok. Thanks for the help.


Updating assessment docs

Draft for updating content docs, closes #107

Rebased, also added a snippet at the bottom of the README to point to the (also new) docs/ folder.


Unexpected '@' in program

after updating pod, I am getting below error Unexpected '@' in progress in MGSwipeButton.m file.
how can I fix this?

The new version is using a XCode 9 feature (availability check syntax).
I have pushed a new 1.6.3 version to add backwards compatibility with XCode 8. Update the pod again and the issue should be fixed.

Update to 1.6.4 better, I missed a availability check in 1.6.3

@MortimerGoro , Thanks now it's working fine.


Remove the useless condition "variablesToInclude.length > 1".

The condition "variablesToInclude.length > 1" has no effect on the current statement.
This condition always produces the same result as the value of variablesToInclude.length was narrowed before.
http://findbugs.sourceforge.net/bugDescriptions.html#UC_USELESS_CONDITION

Coverage decreased (-0.009%) to 89.954% when pulling ce381a4fa5bf22b389af4db8f8d9c059ef7fff66 on BruceKuiLiu:useless-condition into eafb16c711d5cd79edad5fbb2055252acdb3825e on apache:master.


Use Slirp network binding plugin to confiugre VMs with Slirp interface

What this PR does / why we need it:
This PR changes Kubevirt network setup logic to delegate Slirp networking configuration to sidecar
using Slirp network binding plugin (introduced by #10272).
From the user perspective there is no change, configuring VM with Slirp remains.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Currently the Slirp network binding plugin sidecar image is hard-coded to quay.io/kubevirt/network-slirp-binding:20230830_638c60fc8
In follow up commit virt-operator should  set the image according to environment / Kubevirt configuration.
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: The Boy Scout Rule was considered and applied if needed
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to the kubevirt-dev was considered

Release note:

Kubevirt now delegates Slirp networking configuration to Slirp network binding plugin

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.27-sig-network

The goal to move the out of KubeVirt core is the base of this PR. With that in mind, there is a need to discuss and address the following points:
* Backward compatibility:

  * What will happen to existing running VMI/s? [Seems ok to me, but it needs to be explicitly addressed]

Running VM should not be affected because Slirp network setup is done when the VM is created.
New VMs will be created with Slirp network binding plugin sidecar.

Slirp does not support migration.

  * What will happen if the deployment does not have access to the public image? E.g. the operator needs to act explicitly to sync the images or even re-build them.

VM will fail to create since the sidecar wont start.
I don't understand what you mean by the operator to re-build the image.
Slirp plugin image is now integral part of Kubevirt, its part of nightly builds and should be  released on next release.
Unless is changed to be released on different pipeline.
Users who pull images directly from Kubevirt org should have access to the new image as well (registry authentication is not per image basis by for whole org).
In case images are mirrored to intermediate registry the user will have to mirror the new Slirp plugin image, this can be communicated in the next release notes / kubevirt-dev mailing list.

* One of the original goals is to extract out the maintenance of SLIRP, not just to take out the codebase. But if we keep it managed through the operator, this is a step back.

* Some of the changes may require a more transparent interaction with the community. Although the API is not changing, the backend may require adjustment from the operators.

The idea was to make this change frictionless for the users:

If the old Slirp API is used, the operator will pick the correct plugin image, pass it to virt-controller and create the VM with Slirp plugin.
If the new binding API is used, Slirp plugin image has to be specified in Kubevirt config

Not having the Kubevirt operator to pick Slirp image (i.e.: manage Slirp image) makes it a must for the user to set Slirp binding plugin in Kubevirt config.
Otherwise newly created VMs will fail to create.
Going with the latter should be announced with instructions about setting Slirp  plugin image in Kubevirt config in the next Kubevirt release.

Slirp plugin image is now integral part of Kubevirt, its part of nightly builds and should be released on next release.

If the code has not changed, we do not want it to be rebuild.
The point is to build only when needed and hopefully never or rarely.

this can be communicated in the next release notes / kubevirt-dev mailing list.

Yes, we need to communicate what is expected from the operator/admin in terms of SLIRP.
User guide, RN, mailing-list.

The idea was to make this change frictionless for the users:

If the old Slirp API is used, the operator will pick the correct plugin image, pass it to virt-controller and create the VM with Slirp plugin.
If the new binding API is used, Slirp plugin image has to be specified in Kubevirt config

Not having the Kubevirt operator to pick Slirp image (i.e.: manage Slirp image) makes it a must for the user to set Slirp binding plugin in Kubevirt config.
Otherwise newly created VMs will fail to create.
Going with the latter should be announced with instructions about setting Slirp plugin image in Kubevirt config in the next Kubevirt release.

I think we should require the image to be specified in the KV CR, even if using the old API.  Adding this image and others to the operator knowledge is not fitting our mid and long terms goals.

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.27-sig-network

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.27-sig-network

With this PR changes it become mandatory to register Slirp network binding plugin image in Kubevirt config in order to have working VM with Slirp interface.
If the image is not registered VMs with Slirp interface will hand on Pending phase until the image is registered.

Please update the PR message.

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.27-sig-network

/approve
Thanks!


Update Test Examples

Tests examples have been compiled with Cairo 1.0 but are already outdated.
For example core::felt has moved to core::felt252.
Tests should be up to date with latest Cairo 1 ABIs.
In order to make it easier to bring them up to date, tests should include simple cairo contracts.
These contracts are then compiled to their abi.json files.
Note that we do not wish tests to depend on cairo compiler, but only to record the contracts that generated the ABIs.
This task might require to modify the actual tests today and their ABI in favour of simpler ones if needed.
However, test cases will have to be expanded, and that's another issue

This item belongs to payment request #BCE367 on OnlyDust:

from thomas-quadratic to thomas-quadratic
7 items included
$500 for 1 day of work


cmd/compile: deadlock on syntax error

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

$ go version
go version go1.18 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/opt/go/bin"
GOCACHE="/Users/some_user/Library/Caches/go-build"
GOENV="/Users/some_user/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/opt/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/opt/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/some_user/Temp/base16-go/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/cz/mpwjzvhs2s5dfkk3nzj_m8b40000gn/T/go-build187003372=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.18 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.18
uname -v: Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_ARM64_T6000
ProductName:	macOS
ProductVersion:	12.2.1
BuildVersion:	21D62
lldb --version: lldb-13<IP_ADDRESS>
Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)

What did you do?
Compiling github.com/tdegris/base16-go crashes the compiler. Instructions to reproduce the bug:
$ git clone https://github.com/tdegris/base16-go.git
$ cd base16-go/themes
$ git checkout a21c6cb
$ go build ...

What did you expect to see?
A compile error or a successful compilation. Sometimes, the compilation does return the following (correct) syntax errors:
$ go build ...
# github.com/tdegris/base16-go/themes
./base16-3024.go:23:3: syntax error: unexpected ) at end of statement
./base16-apathy.go:23:3: syntax error: unexpected ) at end of statement
./base16-apprentice.go:23:3: syntax error: unexpected ) at end of statement
./base16-ashes.go:23:3: syntax error: unexpected ) at end of statement
./base16-atelier-cave-light.go:23:3: syntax error: unexpected ) at end of stateme
./base16-atelier-cave.go:23:3: syntax error: unexpected ) at end of statement
./base16-atelier-dune-light.go:23:3: syntax error: unexpected ) at end of stateme
./base16-atelier-dune.go:23:3: syntax error: unexpected ) at end of statement
./base16-atelier-estuary-light.go:23:3: syntax error: unexpected ) at end of stat
./base16-atelier-estuary.go:23:3: syntax error: unexpected ) at end of statement
./base16-atelier-estuary.go:23:3: too many errors

What did you see instead?
# github.com/tdegris/base16-go/themes
fatal error: all goroutines are asleep - deadlock!

See the full stack trace.

CC @mdempsky @josharian

I wrote a demo that reproduces the problem.
package main

import (
	"fmt"
)

const (
	FileNames     = 5
	OpenFileLimit = 1 // if BufferCount == 0 and OpenFileLimit < FileNames, deadlock may occur
	BufferCount   = 0 // if BufferCount > 0, there is no deadlock for range p.err
)

type noder struct {
	err chan string
}

func main() {
	sem := make(chan struct{}, OpenFileLimit)
	noders := make([]*noder, FileNames)

	i := 0
	for i < FileNames {
		p := noder{
			err: make(chan string, BufferCount),
		}
		noders[i] = &p
		go func(c int) {
			sem <- struct{}{}
			defer func() { <-sem }()
			defer close(p.err)
			text := fmt.Sprintf("go func: %d finished", c)
			p.err <- text
			fmt.Println(text)
		}(i)
		i = i + 1
	}

	for _, p := range noders {
		for e := range p.err {
			fmt.Println("received:", e)
		}
	}
}

Thanks. I don't think changing the error channel to buffered fixes the issue though. It'll just push the deadlock to only affect when there are files with multiple syntax errors.
I think instead we need to prioritize that the semaphore is use to parse the first N files, rather than just any N.
I think this can be fixed by acquiring the semaphore in the loop outside of the goroutine. But then the loop itself needs to be on a separate goroutine too.

Thanks. I don't think changing the error channel to buffered fixes the issue though. It'll just push the deadlock to only affect when there are files with multiple syntax errors.
I think instead we need to prioritize that the semaphore is use to parse the first N files, rather than just any N.
I think this can be fixed by acquiring the semaphore in the loop outside of the goroutine. But then the loop itself needs to be on a separate goroutine too.

Thanks.
I'm thinking about your plan carefully.

I wrote another demo to show my understanding.
package main

import (
	"context"
	"fmt"
	"golang.org/x/sync/semaphore"
)

const (
	FileNames     = 5
	OpenFileLimit = 1
	BufferCount   = 1 // if BufferCount > 0, there is no deadlock for range p.err
)

type noder struct {
	err chan string
}

func main() {
	ctx := context.TODO()
	sem := semaphore.NewWeighted(int64(OpenFileLimit))
	noders := make([]*noder, FileNames)

	for i := range noders {
		if err := sem.Acquire(ctx, 1); err != nil {
			fmt.Printf("Failed to acquire semaphore: %v", err)
			break
		}

		go func(c int) {
			defer sem.Release(1)
			p := noder{
				err: make(chan string, BufferCount),
			}
			noders[c] = &p
			defer close(p.err)
			text := fmt.Sprintf("The goroutine func: %d finished", c)
			p.err <- text
			fmt.Println(text)
		}(i)
	}

	if err := sem.Acquire(ctx, int64(OpenFileLimit)); err != nil {
		fmt.Printf("Failed to acquire semaphore: %v", err)
	}

	for _, p := range noders {
		for e := range p.err {
			fmt.Println("Received:", e)
		}
	}
}

Here's the code I had in mind: https://go.dev/play/p/yKyrv7NiVB9

Here's the code I had in mind: https://go.dev/play/p/yKyrv7NiVB9

Thanks, I hadn't thought of that.


[GTS] Items bugged after restart if they are not take it out from Claim Trades before

It's a rare bug, that seems that happen when u buy an Item from GTS, and u don't claim it before a restart, I don't have too much details and have some doubts about how this bug can be reproduced, but seems when you buy the Item, and the item gets stored in the Claim Trade and you don't claim it, when the restart happen will appear in the GTS as a item selled by you as the same price you bough it, something impossible because you even didn't take it out from Claim Trades.
I will explain my doubts about the way to reproduce it in case won't work to discover how its the bug. The Item I bough was a Nature Modifier but, hat bug could happen with normal items (including Pixelmon ones), or happens only with the custom ones from Nebula? I had a free space in the GTS (I wasn't selling 3 things there) so, will happen if you have your selling space full? Needs to be at least 1 free space for the bug can happen? This issue happened during a Daily Restart, I think it's the same with crashes but I can't confirm at all. Those are all my doubts.
To Reproduce
Steps to reproduce the behavior:

Go to GTS
Buy a Nature Modifier (should work with all the modifiers, not sure all the Items).
Don't claim the trade.
Wait till a restart happens
Check GTS, the Item is not in Claim Trades and It's now in View Trades as the same price you bought it but as your name.

Expected behavior
The Item should stay in Claim Trades after restart, like happen with Pokémon u don't claim before.
Screenshots
Didn't take any screenshot, sadly, when Death joined to help me he is the one who discovered it was in View Trades, so I take it out to prevent other players buying it (after all I bought it because I wanted it). Didn't think into take a screenshot for the Tracker, sorry.
latest.log
This happened in Atlas, and my username is miguipkv in case is needed.

We'll need to first upgrade GTS to its newest version before re-testing. Forwarding to Jira.


The plugin does not support Gvim

In the Window10, I configured my _vimrc file, and restarted Gvim, but the bufferline did not appear . Then, I configured set showtabline=2, and the tabline appeared but no more change, which is just a tabline. However, when I open vim in the terminal, it works. So, I think this plugin is very good, but it seems not to support Gvim. I have no idea whether this is a problem with the plugin or lightline itself.
Here is a picture related to Gvim.

This is most likely solved by the second entry of the FAQ.
You can disable the graphical gVim tabline rendering by setting this option in your vim config:
set guioptions-=e


Object does not exist at path “/”

I've been following the tutorial. After setting up LocalTerra, when I run terrad keys add test1 --recover, and enter the mnemonic, I get Error: Object does not exist at path “/”.
Running ./terrad keys list returns [].
It looks like accounts were not migrated? Is there an additional step I need to execute?

do without --recover please.
accounts are not in your local keychain.


fixes assumption that tarball location is predefined

Description
fixes assumption that tarball location is predefined
fixes #123
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project.
[x] All new and existing tests passed.

Coverage decreased (-8.3%) to 82.308% when pulling 77eb1be08bbe1ab6ba0f9fd202a6e295718a7549 on gabrielcsapo:fixes-#123 into 544d0fea1fa3c2c2479dabd5b2137778715c8bd4 on local-npm:master.

Coverage decreased (-9.0%) to 81.627% when pulling 8738c8b658e75d0a0aa4d286f8a25ee051529d9b on gabrielcsapo:fixes-#123 into 544d0fea1fa3c2c2479dabd5b2137778715c8bd4 on local-npm:master.


Docker container EACCESS permission denied mkdir 'data/upload'

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
When I build my Docker image targeting the release stage and run the container, it throws an EACCES permission denied error.
👟 Reproduction steps

Build the Docker image targeting the release stage.
Run the container.

👀 Expected behavior
The Docker container should run without any errors.
😓 Actual Behavior
The Docker container throws the following error:
uptime-kuma | Trace: Error: EACCES: permission denied, mkdir 'data/upload/' uptime-kuma | at Object.mkdirSync (node:fs:1414:3)
🐻 Uptime-Kuma Version
2.0.0-dev
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Microsoft Edge
🐋 Docker Version
Docker version 24.0.7, build afdd53b
🟩 NodeJS Version
20.x
📝 Relevant log output
uptime-kuma | Trace: Error: EACCES: permission denied, mkdir 'data/upload/' uptime-kuma | at Object.mkdirSync (node:fs:1414:3) uptime-kuma | at Database.initDataDir (/app/server/database.js:140:16) uptime-kuma | at /app/server/server.js:172:14 uptime-kuma | at Object. (/app/server/server.js:1571:3) uptime-kuma | at Module._compile (node:internal/modules/cjs/loader:1233:14) uptime-kuma | at Module._extensions..js (node:internal/modules/cjs/loader:1287:10) uptime-kuma | at Module.load (node:internal/modules/cjs/loader:1091:32) uptime-kuma | at Module._load (node:internal/modules/cjs/loader:938:12) uptime-kuma | at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12) uptime-kuma | at node:internal/main/run_main_module:23:47 { uptime-kuma | errno: -13, uptime-kuma | syscall: 'mkdir', uptime-kuma | code: 'EACCES', uptime-kuma | path: 'data/upload/' uptime-kuma | } uptime-kuma | at process.unexpectedErrorHandler (/app/server/server.js:1814:13) uptime-kuma | at process.emit (node:events:514:28) uptime-kuma | at emit (node:internal/process/promises:150:20) uptime-kuma | at processPromiseRejections (node:internal/process/promises:284:27) uptime-kuma | at process.processTicksAndRejections (node:internal/process/task_queues:96:32) uptime-kuma | If you keep encountering errors, please report to [https://github.com/louislam/uptime-kuma/issues](vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/code/electron-sandbox/workbench/workbench.html) uptime-kuma | 2024-01-15T06:57:36Z [] INFO: Cannot write to error.log uptime-kuma exited with code 0

The same src works on my local machine.

After I manually created the upload, screenshots, docker-tls directories in project src, and build the docker image, this error is resolved.

Build the Docker image targeting the release stage

What is the release stage?

I just noticed that you have written that you are running against 2.0.0-dev in the description.
Said version is not yet done (pre-beta) => not yet supported.
Please refer to #4171 for additional context


Rethinking the GraphBuilder

In the recent weeks, there have been a number of issues that are in some way related to the lsl.GraphBuilder class.

#173 (With regard to this issue, we discussed the question of what consitutes a change to a model vs. a change to a variable/node.)
#167
#93
#88
#87

I think it may be worthwile to have a look at the GraphBuilder and see how it might evolve in the future.
Three claims
To start off the discussion, I claim the following:

The GraphBuilder class is not in fact where we build the Liesel graph.
The GraphBuilder instead offers methods to change an existing graph or to change nodes in that graph.
A large chunk of the functionality of the GraphBuilder is concentrated in private methods that are being used in the GraphBuilder.build_model method.

Details on the claims
Claim 1
The typical way you build a model in Liesel is by initializing variables, connecting them with calculators. Only once in this process, and most likely near the end of it, do you call the GraphBuilder by adding the "terminal nodes": nodes without outputs that can be used to build the whole graph by recursively following their inputs. In my usage, I often have a one-liner of the form:
model = lsl.GraphBuilder().add(response).build_model()

Claim 2
The graph- or var-manipulating methods of the graph builder are:

GraphBuilder.replace_x (x can be "node" or "var")
GraphBuilder.rename_x (x can be "node" or "var")
GraphBuilder.transform

Claim 3
Here's an excerpt from the GraphBuilder.build_model method:
        gb._set_missing_names()
        gb._add_model_log_lik_node()
        gb._add_model_log_prior_node()
        gb._add_model_log_prob_node()
        gb._add_model_seed_nodes()

        nodes, _vars = gb._all_nodes_and_vars()
        nodes_and_vars = nodes + _vars

        model = Model(nodes_and_vars, grow=False, copy=copy)

What follows from these claims?
For me, it currently seems like the GraphBuilder may not need to be its own class. I think its functionality can be allocated as follows:

Var- and node-changing functionality can live directly on Vars and nodes.
Model setup can be done in the Model init. As a side node, the issue addressed in https://github.com/liesel-devs/liesel/pull/167 can be solved in the process by requiring users to  exclusively provide the "terminal nodes" to the model class, such that the graph is always built from the set of minimally required nodes by recursively following their inputs.
Methods for changing an existing model could become independent functions instead. This would only really apply to the GraphBuilder.replace_x methods, as far as I can see. I am aware that the current model class is intentionally static.

Concluding words
This issue is not intended as a fully worked-out solution to only be approved. It is instead intended to provide a basis for discussion about how the GraphBuilder might evolve.

@wiep @GianmarcoCallegher, my notes are included above. Feel free to comment for additions or corrections.


update elixir

This PR updates CI to use latest Elixir and Erlang versions and ensures Elixir is ~> 1.13 in mix.exs.

This PR is part of recreating #72 in a non-breaking way.
To avoid spamming this repo with PRs, I'm going to create the new ones only after the currently open ones are merged.

Related PR: https://github.com/ExHammer/hammer/pull/76


[Proposal] SOAR-0005: Adopting the Swift HTTP Types package

Motivation
See the proposal at https://github.com/czechboy0/swift-openapi-generator/blob/hd-soar-0005/Sources/swift-openapi-generator/Documentation.docc/Proposals/SOAR-0005.md
Modifications
N/A
Result
N/A
Test Plan
N/A

This proposal is now In Review, which will run until 15th September - feel free to leave your feedback either on this PR or on the corresponding Swift Forums post.
Thanks, and thanks @czechboy0 for the write-up!

Overall this looks good to me, and I believe it's the right direction for the ecosystem to move in 👍🏼
Just one question though: wouldn't it make sense to also make the HTTPBodys in ClientTransport and ClientMiddleware also optional?

Just one question though: wouldn't it make sense to also make the HTTPBodys in ClientTransport and ClientMiddleware also optional?

Great question, I actually tried to make that change, unfortunately unlike with request bodies (which OpenAPI documents as optional, and semantically, a nil body is distinct from an empty body), response bodies are required, and if there's no documented response body, it's considered empty. There's no concept of a nil response body.
The reasoning seems to come from HTTP semantics, where GET and DELETE requests are not supposed to have request bodies, unlike other methods, where a request body is allowed. In contrast, response bodies are never discouraged by the HTTP spec, so there's no nullability to communicate.
Consider the client transport implementing this and trying to make a decision of whether to return an empty or nil body - how do you make that? You can't, as there's no way to know whether the server sent a "nil body" or an empty body. That, again, is in contrast to a server transport, which can reliably, for GET and DELETE requests, provide a nil body (no need to even instantiate an empty HTTPBody value) without looking at the wire, as even if there are bytes, servers are supposed to ignore them for these methods.

The review period is now over, and the proposal has been accepted. SOAR-0005 is now Ready for Implementation.

Landing the proposals to avoid more conflicts, will update their status once the implementation lands.


Fixes DNS which appears to be prepending var.name to local.cluster_dns_name

what

DNS was changing when it shouldn't have been, it was the value of:

${var.name}-${local.cluster_dns_name}

I think this may have changed in https://github.com/cloudposse/terraform-aws-route53-cluster-hostname/releases/tag/0.6.0
This changes the parameter to dns_name and upgrades the version to get a stable DNS record
why
Fix broken DNS
references

Link to any supporting github issues or helpful documentation to add some context (e.g. stackoverflow).
Use closes #123, if this PR closes a GitHub issue #123

/test all


Allow execution of multiple scenarios

Allows to run multiple scenarios within one maven plugin execution with fine-grained control of include/excludes. Original behavior is retained by default.

Thanks!


Avoid unnecessary allocations

Since the Core AMQP type no longer uses BinaryData, we can avoid allocations from a few of the the SBMessage constructors.

/azp run net - servicebus - tests


Support non-RDF quads

As we're starting to explore non-RDF solutions for linked data and knowledge representation, it would be nice to restore the support which was once present in quadstore for instantiating a non-RDF version of the store.

Closing as not planned anymore. There's virtually nothing established outside of the RDF ecosystem for working on/with knowledge representation.


Added type annotations on the mkProgram functions

When starting to using the Program.mkProgram, Program.mkSimple the intellisense/compiler currently show generic type names like 'a which makes it hard to work out what your functions need to do:

So I've added the type annotations to make it a bit clearer:

As a side note while doing this I was also curious on the Program type:
type Program<'model, 'msg, 'view> =
    { init : unit -> 'model * Cmd<'msg>
      update : 'msg -> 'model -> 'model * Cmd<'msg>
      subscribe : 'model -> Cmd<'msg>
      view : 'view
      debug : bool
      onError : (string * exn) -> unit }

Does the view property need to be generic? I'm just getting started with Xamarin.Forms so there may well be an obvious reason I don't know... but I couldn't see a reason why it can't be defined as:
type Program<'model, 'msg> =
    { init : unit -> 'model * Cmd<'msg>
      update : 'msg -> 'model -> 'model * Cmd<'msg>
      subscribe : 'model -> Cmd<'msg>
      view : 'model -> ('msg -> unit) -> ViewElement
      debug : bool
      onError : (string * exn) -> unit }

This would then make the required function signatures clearer still? If you agree I can update the commit?

Does the view property need to be generic?
Maybe it's related to the StaticView part, but this seems more like an early code that haven't needed updating since then.
Makes sense to enforce a specific type annotation, clearer that way.

Ok, I'll update the pull request with the types updated.

Yes the generic 'view  is related to the half-elmish support
I wonder if we should duplicate out the half-elmish support into an entirely separate DLL.  Or deprecate/remove it - though there are still people who advocate it as a stepping stone.

If you were to remove it I guess the question is whats the target audience for Fabulous?. Maybe initially move it to a separate project/dll?

Today, most of the work goes to the full-elmish approach. We're not actively supporting half-elmish.
I think we still need to keep it around, but maybe we should go as far as putting it in a different Github project?
That way, we separate it from Fabulous and say we won't support it further.
But if people think it has its place, they will be able to maintain it.

Maybe initially move it to a separate project/dll? If you want to move I can add as part of this pull request?

I would prefer to merge this PR as-is. It's useful enough.
And do another PR later to add type annotation to 'view.

Merging.
We will separate the half-elmish in a another PR later, before changing type annotation for 'view


Sticky Scroll: Rust functions with multiple lines

Functions in Rust that span multiple lines are not shown correctly, regardless of the editor.stickyScroll.maxLineCount setting. For example, this:

Shows up as this, which makes the parameters disappear completely:

The same occurs with functions that contain a where clause:

Shows up as this, which makes the type bounds disappear completely:

Both are very bad and ruin the feature.
Can it be fixed, please?

Yes, thanks @aiday-mar and @RedCMD.
I think it would be awesome to show the complete function signature. Perhaps with a new config like editor.stickyScroll.maxLineCountPerLevel, which defaults to 1.


UTF-8 text garbling on opening file

When I open the file (bake.txt), the editor of vscode tab has a text garbling.
Attached file (bake.txt) has 1000 repeated same lines and no text garbling.
But, opened file in editor has garbled line, it's line number is always 586.
It is reproduced on chrome and safari.
Log file is here. ClientLogPane Log(Remote Server).txt

Web Browser: chrome & safari
Local OS: MacOS Catalina
Remote OS: Debian10
Remote Architecture: x64
code-server --version: code-server 3.4.1 48f7c2724827e526eeaa6c2c151c520f48a61259

thanks.

The picture shows around line 586.

I edited bufferSize:65536 to bufferSize:6553600 of /usr/lib/code-server/lib/vscode/out/vs/server/entry.js.
Text garblings is disappeared.
So, This bug may be related with buffer or data recombination between buffers.

Nice find!! I tried modifying the code to call VS Code's own readFileStream instead of handling it ourselves but the problem persists. Tomorrow I'm going to try out VS Code's remote SSH extension and see if it has the same problem to see if it's a bug in our implementation.

It works fine with the extension so it could be our implementation or it could be something in the web portion of VS Code, maybe with where it reads the stream returned by the server. :thinking:

@code-asher
Wow, This bug disappeared on ver.3.5.0.!
Do you know something about this.

I wasn't aware but that's great news! I'll go ahead and close the issue.
I wonder if it was accidentally fixed when the streams were modified to support multiple encodings.