[Bug] GetConnectionTimeoutException when Querying External Table

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
Doris Version: 2.1.1 (recently upgraded from 2.0.0)
Java Version: 1.8.0_402
What's Wrong?
When querying JDBC external tables in 2.1.1, we reached this error after a while:
org.jkiss.dbeaver.model.sql.DBSQLException: SQL Error [1105] [HY000]: errCode = 2, detailMessage = (xx.xxx.xxx.xxx)[CANCELLED]UdfRuntimeException: Initialize datasource failed:
CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10, maxActive 10, creating 0
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:133)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeStatement(SQLQueryJob.java:583)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.lambda$1(SQLQueryJob.java:492)
	at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:190)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:499)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.extractData(SQLQueryJob.java:947)
	at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:4057)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:123)
	at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:190)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:121)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer$ResultSetDataPumpJob.run(ResultSetViewer.java:5164)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: java.sql.SQLException: errCode = 2, detailMessage = (xx.xxx.xxx.xxx)[CANCELLED]UdfRuntimeException: Initialize datasource failed:
CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10, maxActive 10, creating 0
	at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:129)
	at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:122)
	at com.mysql.cj.jdbc.StatementImpl.executeInternal(StatementImpl.java:763)
	at com.mysql.cj.jdbc.StatementImpl.execute(StatementImpl.java:648)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:330)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:131)
	... 12 more

What You Expected?
successfully querying the external table.
How to Reproduce?

In Doris 2.1.1, create a JDBC connection resource and add a valid external table
Open multiple connections to Doris and query the external table.

Anything Else?
Here are the following things we tried:

Increasing connection_pool_max_size, helped to increase the time before we hit this error again, but even increasing the max pool size to 10000, we still eventually hit this error. We suspect that there is a leak and the connection pool management.
Decreasing connection_pool_max_life_time and jdbc_connection_pool_cache_clear_time_sec also seemed to help increase the number of queries before we hit the error again, but also do not solve the issue ultimately.
Interestingly, when we update  connection_pool_max_size to any value (could be smaller than the previously set value), Doris seems to clear all the useless connections from before and we can immediately query the external table again.

We have not tried: Installing Doris 2.1.1 from fresh to reproduce this error, the bug may be a result of unexpected behaviors from the version upgrade from 2.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I would like to know the following situations

How many queries for external catalog can you have at most at the same time?
Are you using the alter catalog statement to adjust connection_pool related parameters?
After adjusting connection_pool_max_size to 10000
The error reported is the new CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000, creating 0
Are you in a multi-FE environment? When an error occurs, have you paid attention to whether the connected FE is the master FE?
Have you tried re-creating a catalog and specifying connection_pool_max_size as a larger value, such as 100, in the properties of the created catalog? Then run it for a period of time and observe whether there are the following errors: CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 100, maxActive 100, creating 0

I would like to know the following situations

How many queries for external catalog can you have at most at the same time?
Are you using the alter catalog statement to adjust connection_pool related parameters?
After adjusting connection_pool_max_size to 10000
Is the error reported a new CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000, creating 0, or is the error still CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10, maxActive 10, creating 0. Pay attention to the difference between 10 and 10000. the difference
Are you in a multi-FE environment? When an error occurs, have you paid attention to whether the connected FE is the master FE?
Have you tried re-creating a catalog and specifying connection_pool_max_size as a larger value, such as 100, in the properties of the created catalog? Then run it for a period of time and observe whether there are the following errors: CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 100, maxActive 100, creating 0

Majority of queries in our production requirements require joins from external tables. In most cases we have no more than 50 queries at the same time.
Yes. Specifically, we use the statement ALTER RESOURCE abc PROPERTIES("connection_pool_max_size"="1000"). After updating the value to a different one, we were immediately able to query the external table under this resource regardless to the size we set it to.
The former. The new error would be CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000.
Yes, currently we have 3 FEs, all followers. We observed the same error regardless if we were connected to the master node. We also see that the IP reported from the error is always from the BE nodes. Furthermore, we hit the same error when we only had 1 FE.
Yes. We observed that dropping and re-adding the resource and table has the same effect as using the alter statements.

I would like to know the following situations

How many queries for external catalog can you have at most at the same time?
Are you using the alter catalog statement to adjust connection_pool related parameters?
After adjusting connection_pool_max_size to 10000
Is the error reported a new CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000, creating 0, or is the error still CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10, maxActive 10, creating 0. Pay attention to the difference between 10 and 10000. the difference
Are you in a multi-FE environment? When an error occurs, have you paid attention to whether the connected FE is the master FE?
Have you tried re-creating a catalog and specifying connection_pool_max_size as a larger value, such as 100, in the properties of the created catalog? Then run it for a period of time and observe whether there are the following errors: CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 100, maxActive 100, creating 0

Majority of queries in our production requirements require joins from external tables. In most cases we have no more than 50 queries at the same time.
Yes. Specifically, we use the statement ALTER RESOURCE abc PROPERTIES("connection_pool_max_size"="1000"). After updating the value to a different one, we were immediately able to query the external table under this resource regardless to the size we set it to.
The former. The new error would be CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000.
Yes, currently we have 3 FEs, all followers. We observed the same error regardless if we were connected to the master node. We also see that the IP reported from the error is always from the BE nodes. Furthermore, we hit the same error when we only had 1 FE.
Yes. We observed that dropping and re-adding the resource and table has the same effect as using the alter statements.

This seems a bit tricky. I also tested it using the method you mentioned, but I did not reproduce the problem. Is there any way I can contact you so that we can synchronize in time?

I would like to know the following situations

How many queries for external catalog can you have at most at the same time?
Are you using the alter catalog statement to adjust connection_pool related parameters?
After adjusting connection_pool_max_size to 10000
Is the error reported a new CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000, creating 0, or is the error still CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10, maxActive 10, creating 0. Pay attention to the difference between 10 and 10000. the difference
Are you in a multi-FE environment? When an error occurs, have you paid attention to whether the connected FE is the master FE?
Have you tried re-creating a catalog and specifying connection_pool_max_size as a larger value, such as 100, in the properties of the created catalog? Then run it for a period of time and observe whether there are the following errors: CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 100, maxActive 100, creating 0

Majority of queries in our production requirements require joins from external tables. In most cases we have no more than 50 queries at the same time.
Yes. Specifically, we use the statement ALTER RESOURCE abc PROPERTIES("connection_pool_max_size"="1000"). After updating the value to a different one, we were immediately able to query the external table under this resource regardless to the size we set it to.
The former. The new error would be CAUSED BY: GetConnectionTimeoutException: wait millis 5002, active 10000, maxActive 10000.
Yes, currently we have 3 FEs, all followers. We observed the same error regardless if we were connected to the master node. We also see that the IP reported from the error is always from the BE nodes. Furthermore, we hit the same error when we only had 1 FE.
Yes. We observed that dropping and re-adding the resource and table has the same effect as using the alter statements.

This seems a bit tricky. I also tested it using the method you mentioned, but I did not reproduce the problem. Is there any way I can contact you so that we can synchronize in time?

I was kind-of able to reproduce it using a fresh 2.1.1 by spamming with multiple connections, but I had to use different queries, otherwise, it seems to only use 1 connection. The thing after I hit this error, the state of the "active connection count" seemed to be stuck even when i closed the previous connections.
Anyways, we can discuss on 微信 if you'd like, thanks! id: feer4847

I've encountered the same problem and can reproduce it through a demo project, please add me on wechat, id: ziyanTOP @zy-kkk

same problem.

same problem.

Version 2.1.6 fixes all known connection issues. Please upgrade and test. This issue has been unanswered for a long time. I will close it first. If there are still problems later, please feel free to open it.

@zy-kkk  Does 2.1.6 resolve https://github.com/apache/doris/issues/34168?

@zy-kkk Does 2.1.6 resolve #34168?

Are you experiencing memory leaks due to using Jdbc Catalog? If so, a fix will be released in 2.1.7

Yes, but we are using JDBC external table, not Catalog. I think the underlying mechanism is the same.
Please let me know which commit fixes this memory leak in 2.1.7, thanks!


CSS maraton #1

Coverage increased (+0.3%) to 42.489% when pulling f91a85dd09680683535bda44f641abcd6bce45a4 on bjornreppen:stats into 34c5e513b1b32862386ddeab32ac1cba80af5a03 on Artsdatabanken:master.


Add test for X509 sign TSB cache regression.

See Issue #19388.

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

Test for  #19393

I had trouble getting X509_REQ to fail, so I have omitted that one..
These are based off tests written by @davidben (I have omitted the more complex manual signing code for now)

These tests should be failing currently..

Thanks @davidben for finding this and writing some tests

I had trouble getting X509_REQ to fail, so I have omitted that one..

You're probably running into the same thing I ran into, which is that merely changing the sigalg isn't sufficient for X509_REQ. Certificates and CRLs contain an extra copy of the sigalg in the TBS area, so by merely changing the signature algorithm, you're exercising the cache. But CSRs don't, so you need to change something else about it.

Rebased..

CI failure is relevant.

ping @slontis could you please resolve the CI failure?

The test has been moved to recipe 25 instead of a new 60 one.. So needs reapproval. EC disable check added to the .t file.. It can be moved later if the test file ends up with non ec tests at some point.

This pull request is ready to merge

Merged to master branch. Unfortunately this won't cherry-pick cleanly to 3.1/3.0, would you please create a PR for 3.1 branch?


[Help wanted] Usage with ngrok

Are you familar with ngrok?
I have a problem with that tunnel, my events in dev mode is repeated (retried) by twitch.
I'm used helper for getting tunnel for eventsub client
func GetAppTunnel(ctx context.Context, cfg *config.Config) (net.Listener, error) {
	if cfg.AppEnv != "production" {
		tun, err := ngrok.Listen(ctx,
			ngrok_config.HTTPEndpoint(),
		)
		if err != nil {
			return nil, err
		}

		return tun, nil
	} else {
		tun, err := net.Listen("tcp", ":3003")
		if err != nil {
			return nil, err
		}
		return tun, nil
	}
}

After that using it like:
	appTun, err := helpers.GetAppTunnel(appCtx, cfg)
	if err != nil {
		panic(err)
	}

	appAddr := lo.
		If(cfg.AppEnv != "production", appTun.Addr().String()).
		Else(fmt.Sprintf("eventsub.%s", cfg.HostName))

	eventSubHandler := handler.NewHandler(services)
		go func() {
                         // manager here is *eventsub_framework.SubHandler
			if err := http.Serve(appTun, eventSubHandler.Manager); err != nil {
				panic(err)
			}
		}()

	fmt.Println("Listening on %s", appAddr)

I am familiar with ngrok's command line interface but not familiar with the Go library. However, the code you posted makes sense and looks like it should be no different from receiving HTTP requests on the local port.
Some thoughts:

My understanding is that ngrok has a web interface which lets you inspect requests and responses that go through your tunnel. If you look at this interface, are you seeing the notification requests from Twitch go through? If so, do you also see a response? If not, maybe put some wrapper code around the eventSubHandler.Manager handler to intercept and log what's happening, e.g. implementing an http.ResponseWriter that logs the calls to Write and WriteHeader.

 I expect this to make a difference given the GetAppTunnel seems perfectly reasonable.

My theory behind why #2 was needed was this: your proxy must not be sending the HTTP response from the server over the proxy because it is still waiting for more of the response. i.e., setting Content-Length: 0 and responding with a 204 would indicate to the proxy that the response is finished. I use my own tunnel system (which I do NOT advise you use, it's really sketchy and not production-ready) https://github.com/dnsge/leap and I have it force the connection to close per-request: https://github.com/dnsge/leap/blob/0ed301834c87bf5826bc66f0711f11a9e3280e2a/server/proxy.go#L16 so it would make sense that I don't experience the issue with leap while you do with some other tool.

I'm assuming that your problem of retrying events means retrying notification events. If the retries are occurring on subscription verification, perhaps a patch like so could fix the issue? (similar to #2's fix)

diff --git a/handler.go b/handler.go
index d1ad0da..30a6476 100644
--- a/handler.go
+++ b/handler.go
@@ -6,6 +6,7 @@ import (
 	"github.com/mozillazg/go-httpheader"
 	"io"
 	"net/http"
+	"strconv"
 )

 const (
@@ -175,6 +176,7 @@ func (s *SubHandler) handleVerification(w http.ResponseWriter, bodyBytes []byte,

 	if s.VerifyChallenge == nil || s.VerifyChallenge(headers, &data) {
 		w.Header().Set("Content-Type", "text/plain; charset=utf-8")
+		w.Header().Set("Content-Length", strconv.Itoa(len(data.Challenge)))
 		w.WriteHeader(http.StatusOK)
 		_, _ = w.Write([]byte(data.Challenge))
 	} else {

Verification seems to be ok, it happens only for notifications.

@Satont Could you check out #6? If it fixes your problem I'll go ahead and merge it in.
As for your comment on dnsge/twitch-eventsub-bindings#1, I already made a new release bumping the twitch-eventsub-bindings version but I'll make another when (if) #6 gets merged. I have no clue if you can mix dependency versions so I'll just make a new release on this module whenever I update the bindings.

Didn't I write in a previous comment that I changed it and it worked for me? But yes, it still needs to be tried again to be sure.

Yep, it's working just fine, no retries.


Python Interactive is blank.

Issue Type: Bug
Python Interactive is blank.
When I Run Cell in pyhon script, then Python Interactive is open.
But there is not any results in Python Interactive.
And I try again to put 'Run Cell', then the message'executing cell' is shown.
But never finish and show result.
By the way, I try to open jupyter notebook, the same situation happen.
VS Code version: Code 1.50.0 (93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3, 2020-10-07T06:10:52.432Z)
OS version: Windows_NT x64 10.0.19041
Remote OS version: Linux x64 4.19.128-microsoft-standard

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (9.63GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 4.19.128-microsoft-standard

CPUs
Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 x 3600)

Memory (System)
12.37GB (10.64GB free)

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

remote-wsl
ms-
0.50.0

python
ms-
2020.9.114305

This repository does not contain Python language features. Please submit this issue at microsoft/vscode-python

I submit again here.
https://github.com/microsoft/vscode-python/issues/14342#issue-718003957

@k-wht1023 Please close this issue as well


java.sql.SQLException: sql injection violation, dbType mysql, druid-version 1.2.16, multi-statement not allow

mysql 8.0
druid 1.2.16
springboot 2.1.8
mybatis-plus 3.1.2
问题sql语句
<insert id="overwriteData">
        replace into pdd_account_order(shop_name, order_id, money, account_create_time, style_id, materials_name, materials_num, materials_money, is_divisible)
        with pdd_account as (
            select shop_name, order_id, create_time, income, pay, account_type
            from pdd_account_excel a
            where a.account_type in ('交易收入', '退款')
            group by shop_name, order_id, create_time, income, pay, account_type
        )
        , pdd_account_res as (
            select a.shop_name, a.order_id, sum(a.income + a.pay) as money, max(create_time) as create_time
            from pdd_account a
            group by a.shop_name, a.order_id
        )
        , pdd_order as (
            select shop_name, order_id, num, style_id
            from pdd_order_excel o
            group by shop_name, order_id, num, style_id
        )
        , pdd_order_res as (
            select o.shop_name, o.order_id, o.style_id, ifnull(m.name, '--缺少物料--') as materials_name, sum(o.num* ifnull(m.num, 0)) as materials_num
            from pdd_order o
                     left join pdd_materials m on o.shop_name = m.shop_name and o.style_id = m.style_id
            group by o.shop_name, o.order_id, o.style_id, ifnull(m.name, '--缺少物料--')
        )
        , pdd_order_res2 as (
            select r.shop_name, r.order_id, sum(r.materials_num) as materials_num
            from pdd_order_res r
            group by r.shop_name, r.order_id
        )
        , pdd_order_res3 as (
            select o.shop_name, o.order_id, o.materials_num, a.money
                 , a.money/nullif(o.materials_num, 0) as price
                 , if( MOD(a.money*10000000, nullif(o.materials_num, 0)) =0, 1, 0) as is_divisible
                 , a.create_time
            from pdd_order_res2 o, pdd_account_res a
            where o.shop_name = a.shop_name and o.order_id = a.order_id
        )
        select r.shop_name, r.order_id, r3.money, r3.create_time, r.style_id, r.materials_name, r.materials_num
             , r.materials_num* r3.price as materials_money
             , r3.is_divisible
        from pdd_order_res r, pdd_order_res3 r3
        where r.shop_name = r3.shop_name and r.order_id = r3.order_id
    </insert>

我发现带有with语句就会报错，已经尝试了能查的方法，这是我的yml配置文件截图

发完整的出错堆栈来


[Bug] share_plus dependency issue

이슈 설명 (Issue description)
Flutter : 3.24.0
Dart: 3.5.0
kakao_flutter_sdk: ^1.9.5
kakao_flutter_sdk_user: ^1.9.5
share_plus: ^10.0.2

Because share_plus >=10.0.1 depends on web ^1.0.0 and kakao_flutter_sdk_common 1.9.5 depends on web ^0.5.1, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_common 1.9.5.
And because kakao_flutter_sdk_auth 1.9.5 depends on kakao_flutter_sdk_common 1.9.5, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_auth 1.9.5.
And because kakao_flutter_sdk_user 1.9.5 depends on kakao_flutter_sdk_auth 1.9.5 and no versions of kakao_flutter_sdk_user match >1.9.5 <2.0.0, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_user ^1.9.5.
So, because rock2you depends on both kakao_flutter_sdk_user ^1.9.5 and share_plus ^10.0.2, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on share_plus: flutter pub add share_plus:^10.0.0

share_plus 최신버전과 kakao_flutter_sdk 최신버전이 서로 의존성 이슈가 맞지 않습니다.
이에 수정이 필요해보입니다.
앱 ID (App ID)
1107113
플랫폼 (Platform)
Android, iOS
디바이스 (Device)
Samsung Galaxy S9+
Version
^1.9.5
Flutter SDK
3.24.0
재현 방법 (Steps to reproduce)
kakao_flutter_sdk: ^1.9.5
kakao_flutter_sdk_user: ^1.9.5
share_plus: ^10.0.2

코드 샘플 (Code Sample)
No response
Logs
Because share_plus >=10.0.1 depends on web ^1.0.0 and kakao_flutter_sdk_common 1.9.5 depends on web ^0.5.1, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_common 1.9.5.
And because kakao_flutter_sdk_auth 1.9.5 depends on kakao_flutter_sdk_common 1.9.5, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_auth 1.9.5.
And because kakao_flutter_sdk_user 1.9.5 depends on kakao_flutter_sdk_auth 1.9.5 and no versions of kakao_flutter_sdk_user match >1.9.5 <2.0.0, share_plus >=10.0.1 is incompatible with kakao_flutter_sdk_user ^1.9.5.
So, because rock2you depends on both kakao_flutter_sdk_user ^1.9.5 and share_plus ^10.0.2, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on share_plus: flutter pub add share_plus:^10.0.0
Process finished with exit code 1

Flutter Doctor
flutter doctor -v                                                                                 ✔ │ took 4h 10m 57s │ at 20:45:31
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6.1 23G93 darwin-arm64, locale ko-KR)
    • Flutter version 3.24.0 on channel stable at /Users/shinjaewoo/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (3주 전), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at /Users/shinjaewoo/Library/Android/sdk
    • Platform android-34, build-tools 35.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.91.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.92.0

[✓] Connected device (6 available)
    • SM G965N (mobile)               • 2193d76caf0c7ece          • android-arm64  • Android 10 (API 29)
    • 신재우의 iPhone (mobile)            • 00008130-001240622483401C • ios            • iOS 17.6.1 21G93
    • 신재우의 iPad (mobile)              • 00008103-000671983ED9001E • ios            • iOS 17.6.1 21G93
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.6.1 23G93 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.6.1 23G93 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 128.0.6613.84

[✓] Network resources
    • All expected network resources are available.

• No issues found!

안녕하세요
이슈 확인했고, 빠른 시일 내에 대응하도록 하겠습니다.
급하신 경우 pubspec.yaml 파일에 아래와 같이 명시해서 임시조치를 해서 사용하실 수 있을 것 같아요
 dependency_overrides:
  web: ^1.0.0

그리고 이슈와 별개로 첨부해주신 pubspec.yaml 파일을 보면 kakao_flutter_sdk와 kakao_flutter_sdk_user 모두 의존성을 추가해주셨는데요, kakao_flutter_sdk의존성을 추가하는 경우 kakao_flutter_sdk_user 를 포함한 kakao_flutter_sdk 관련 패키지 모두 의존성 추가하기 때문에 중복해서 명시할 필요 없는 점 참고 부탁드려요


How to judge conditions before drag and drop?

First check https://github.com/SortableJS/Vue.Draggable/blob/master/CONTRIBUTING.md
How to judge conditions before drag and drop?
Step by step scenario

I need to decide if I can drag before I drag

//templete
<draggable
            v-model="leftColorList"
            item-key="element.id"
            v-bind="dragOptions"
            group="color"
            @start="dragStart"
        >
//script
    function dragStart(element:any) {
      console.log(element)
    }

Cannot get the binding element, how to write, to determine whether can be dragged


Youtube vidoes don't show up on iframe

When I try to add an iframe using a youtube video's link, the iframe appears white and the video is inaccessible. Am I doing it wrong? Or is it just an issue with YouTube itself?

Ahhh.. you do need the embed link not the regular one. Should've done it programmatically though.
Thanks a lot for your valuable feedback!


Changing the structure of actions, will be incompatible with older versions of the html

I wanted to change actions from the current

actions
name : url
name : url

to

actions
id : str
displayname : str
url : str

this will allow us to add meta data about the actions, such as the type of parameter.  (int / datetime)
I'd like to be able to popup a date picker for schedules, or a slider for countdowns.

actions
id : str
displayname : str
url : str
argname1 : "start"
argtype1 : "date"

This would also allow us to move the 'canceljob' setter to an action, which is better I think.
thoughts?

Definitely something to consider. I'm planning to spend some decent time over the next week developing a more sane replacement for the web interface. Something that would also be compatible with adding in unrelated devices at some point even (such as smart light bulbs, etc.).
This is something for #10 too.


Specific route stops working when deployed

Describe the bug
I have 3 routes in my app:

/routes/charts/+page.svelte
routes//favorites/+page.svelte
/routes/[mode]/+page.svelte

Using the vite dev server everything works as expected. Both /charts and /favorites lead to their specific pages and everything else is captured by the slug [mode].
However, when deploying the frontend (using ASP.NET Core in this example) the /charts stops working. Its now also captured by the slug, leading to the wrong page.
Reproduction
I have created repo with a minimal reproducible example here: https://github.com/koryphaee/SvelteBug. It should be very easy to reproduce: just clone and run docker compose up.
Logs
No response
System Info
I made sure that everything is up-to-date:

System:
  OS: Windows 11 10.0.22631
  CPU: (16) x64 AMD Ryzen 7 5800X3D 8-Core Processor
  Memory: 45.22 GB / 63.91 GB
Binaries:
  Node: 21.0.0 - c:\programs\nodejs\node.EXE
  npm: 10.2.0 - c:\programs\nodejs\npm.CMD
Browsers:
  Edge: Chromium (127.0.2651.74)
  Internet Explorer: 11.0.22621.3527
npmPackages:
  @sveltejs/adapter-static: ^3.0.2 => 3.0.2
  @sveltejs/kit: ^2.5.18 => 2.5.18
  @sveltejs/vite-plugin-svelte: ^3.1.1 => 3.1.1
  svelte: ^4.2.18 => 4.2.18
  vite: ^5.3.5 => 5.3.5

Severity
annoyance
Additional Information
No response

closing as duplicate of https://github.com/sveltejs/kit/issues/12739 . See https://github.com/sveltejs/kit/issues/12739#issuecomment-2425524871 for a workaround using a matcher to specifically exclude the other routes in the dynamic route.


Crashes on vttest

Crashes on vttest 1 (cursor movements).  Most other tests need work too.
There is some discussion on alacritty's vttest issue thread that may be useful.  https://github.com/jwilm/alacritty/issues/240

This is really useful, thank you. I wasn't aware of vttest until now. I'll give it a run tomorrow.

No problem, I am always happy to see new terminals come up.  Yours is quite popular, hope it fares well.  FYI vttest is a bit of a bear to get through, but once done you will know the vt100/vt102 manuals inside and out. :)

Wow, even iTerm fails these.

Oh, and I've just realised you have to switch to 80x24 before you attempt these, for anyone trying it.
Easiest way to do that is ctrl + shift +d/cmd + d and resize.

I crashed the first time too.  :)  (I crashed again long after passing vttest when I logged into a VMS system and ran the LSE editor.  Man those DEC guys were brutal...)

Hmm, yes that is what my emulator does: unknown parameters are silently ignored.  Also, if you haven't already seen xterm's control sequences page: https://invisible-island.net/xterm/ctlseqs/ctlseqs.html  .  I find it handy in that it shows VT220/320/420 sequences in one place, a little easier than searching through those manuals separately.

Yeah, it's apparently my most visited webpage too, haha!

Crash fixed in #119
New issues for tests 1 and 2 at #120 and #121

The zombie rises again. :)  0.0.10 darktile release crashes on the VT102 tests (test 8).


Major issue

self.init = f"TWEET-{config.Resume}-0"

this is what pops up trying to use your modules I cannot automate it via the methods you describe using your code it does not run my way is insecure and I cannot release it like that in case a user injects malicious content into tweet fields profile etc so I want to use yours why does this happen on 2 diff machines just running your sample code after installing
No matter how I install it via pip3 or github link or what have you

Hi @devcoinfet, I'm not sure what you are asking for. May you elaborate?
If you get an error messages reporting self.init = f"TWEET-{config.Resume}-0", you have to run with Python 3.6 (at least) as described in the documentation.
Every information extracted from Twitter is an integer or a string, and there is not eval stuff in the code. I'm not a security researcher but I guess there are no issues with that here.


fix: Typescript Compile Errors

This PR fixes 2 issues with Typescript compilation in index.ts:

Removes the unused import Handler from elysia
For the second issue (property length does not exist on type never, calling isStringKey === false repeats the assertion what we have a string, so the code path for when it's truthy will never execute. I believe the intention was unsigned === false.

Hello @SaltyAom @bogeychan
Could you review this PR?


[ACTION NEEDED] Fix flaky integration tests at distribution level

What is the bug?
It was observed in 2.13.0 and previous other releases that this component manually signed off on the release for failing integration tests. See https://github.com/opensearch-project/opensearch-build/issues/4433#issuecomment-2026486279
The flakiness of the test runs take a lot of time from the release team to collect go/no-go decision and significantly lower the confidence in the release bundles.
How can one reproduce the bug?
Steps to reproduce the behavior:

Run integration testing for altering and see the failures.
Issues can be reproduced using the steps declared in AUTOCUT issues for failed integration testing

What is the expected behavior?
Tests should be consistently passing.
Do you have any additional context?
Please note that this is a hard blocker for 2.14.0 release as per the discussion here

It also looks like many of the manifests are still showing a Not Available status, related to the discussion here, but it's showing them even for fresh logs so it doesn't seem to be an issue of the manifests being stale.

Tagging @zelinh here to provide his inputs.

Here are some reasons that it may show Not Available. https://github.com/opensearch-project/opensearch-build/tree/main/src/report_workflow#why-are-some-component-testing-results-missing-or-unavailable
@Swiddis Could you share one situation that is showing Not Available so I can look into it in more details.

Could you share one situation that is showing Not Available so I can look into it in more details.

E.g. the 2.14 integration tests autocut, of the three most recent manifests at the time of writing, two of them are unavailable (most recent, second most recent (available), third most recent).

Could you share one situation that is showing Not Available so I can look into it in more details.

E.g. the 2.14 integration tests autocut, of the three most recent manifests at the time of writing, two of them are unavailable (most recent, second most recent (available), third most recent).

I saw these in both of the unavailable runs. Seems like the process is terminated because of timeout limit when we run the integ tests for observabilityDashboards ; therefore it didn't run through all the test recording process.
Cancelling nested steps due to timeout
Sending interrupt signal to process

Session terminated, killing shell...Terminated
 ...killed.
script returned exit code 143

https://build.ci.opensearch.org/job/integ-test-opensearch-dashboards/5856/consoleFull
https://build.ci.opensearch.org/job/integ-test-opensearch-dashboards/5844/consoleFull
Both of these jobs run for more than 4 hours; while the available one run only 1.5 hour.
Do you have any idea why these jobs run longer than usual? @rishabh6788 @gaiksaya

Hypothesis: The failing tests are flaky and the timeouts only happen if the tests pass (i.e. something later in the test suite is taking all the time). We only get the failure message when the earlier test fails and cuts the run short. Based on this hypothesis I made https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1250 to fix the flakiness, but I'm still not sure what's causing the timeouts.

For completeness I've checked the recent pipeline logs after the flakiness fix was merged, and am not seeing any integ-test failures for observability. https://build.ci.opensearch.org/blue/rest/organizations/jenkins/pipelines/integ-test-opensearch-dashboards/runs/5899/log/?start=0
I can find the interruption exception, but not the indication of what specifically is being interrupted (is some test hanging?):
org.jenkinsci.plugins.workflow.actions.ErrorAction$ErrorId: 5a075705-b450-4433-85c4-0b5d9991ba84
org.jenkinsci.plugins.workflow.steps.FlowInterruptedException
		at org.jenkinsci.plugins.workflow.steps.BodyExecution.cancel(BodyExecution.java:59)
		at org.jenkinsci.plugins.workflow.steps.TimeoutStepExecution.cancel(TimeoutStepExecution.java:197)
		at jenkins.security.ImpersonatingScheduledExecutorService$1.run(ImpersonatingScheduledExecutorService.java:67)
		at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
		at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
		at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
	Caused: java.lang.Exception: Error running integtest for component observabilityDashboards
		at WorkflowScript.run(WorkflowScript:317)
		at org.jenkinsci.plugins.docker.workflow.Docker$Image.inside(Docker.groovy:141)
		at ___cps.transform___(Native Method)
		at java.base/jdk.internal.reflect.GeneratedConstructorAccessor790.newInstance(Unknown Source)

Tagging @rishabh6788 to look in to the above failure ^

Currently just held up by #1822


fixmodify agg function nullability in PhysicalHashAggregate

Proposed changes
this is a follow up pr of https://github.com/apache/doris/pull/41576 , the previous pr may miss some agg function in AggregateExpression, this pr fix it.

run buildall

run buildall


Rework interpolation builders

For interpolations a builder is created, the current implementation works fine but there is two problems:

You can pass 2 times the same prop, this should at least give a warning because you are overriding a value without doing any work.
If a key is missing the error is terrible as the builder relies on generics to implement leptos::IntoView, so you don't know what key is missing, you don't even know that a key is missing.

First problem can either be done on expanded code with use of #[deprecated(note = "duplicated field foo"] when the field is already set, or by allowing to set the value only when T = EmptyInterpolateValue, or it can be done within the t! macro. First solution would be the better as a custom message can be created.
Second problem is going to be hard to implement, I'm not sure how to properly do it, the only way I see is to use a build method at the end that just serves as a check, but would have 2^N implementations, where 1 implementation would be with all values set, and the other with one or multiple values not set and use the deprecated attribute to give  warning on what prop is missing, and panicking in the body with the same message (would return () to avoid compilation error where the returned value does not impl leptos::IntoView).
Those 2 changes would massively increase the size of outputed code, and even if most of it would be thrown away after compilation it would still impact compiling speed, maybe gate it behind a feature? or only debug mode? I don't known the impact on compiling speed when config gating things, I'm assuming the compiler can just throw it right away.

So for the first problem, instead of outputing
struct Builder<__var_count> {
    locale: LocaleEnum,
    var_count: __var_count
}

impl<__var_count> Builder<__var_count> {
    pub fn var_count<T: ..>(var_count: T) -> Builder<T> {
         // set var_count
    }
}

It should output
impl Builder<EmptyInterpolateValue> {
    pub fn var_count<T: ..>(var_count: T) -> Builder<T> {
        // set var_count
    }
}

impl Builder<impl ..> {
    #[deprecated(note = "variable \"count\" is already set")]
    pub fn var_count<T: ..>(var_count: T) -> Builder<T> {
        // set var_count
    }
}

The error would look like this:
warning: use of deprecated method `i18n::builders::click_count_builder::<__count>::var_count`: variable `count` is already set
  --> src\app.rs:36:35
   |
36 |
{t!(i18n, click_count, count, count)}

   |                                          ^^^^^
   |
   = note: `#[warn(deprecated)]` on by default

Which is not that bad, it also seams like it is what the typed_builder crate does

implementing check function for all keys result from this error:
error[E0277]: the trait bound `(leptos::Scope, [closure@src\app.rs:36:13: 36:41]): leptos::IntoView` is not satisfied
   --> src\app.rs:35:5
    |
35  | /     view! { cx,
36  | |
{t!(i18n, click_count, count)}

37  | |         //
{t!(i18n, click_count, count = move || counter.get())}

38  | |         <button on:click=inc>{t!(i18n, click_to_inc)}</button>
39  | |     }
    | |_____^ the trait `leptos::IntoView` is not implemented for `(leptos::Scope, [closure@src\app.rs:36:13: 36:41])`
    |
    = help: the following other types implement trait `leptos::IntoView`:
              ()
              (leptos::Scope, T)
              (A,)
              (A, B)
              (A, B, C)
              (A, B, C, D)
              (A, B, C, D, E)
              (A, B, C, D, E, F)
            and 20 others
note: required by a bound in `HtmlElement::<El>::child`
   --> C:\Users\bapti\.cargo\registry\src\index.crates.io-6f17d22bba15001f\leptos_dom-0.4.10\src\html.rs:984:36
    |
984 |     pub fn child(self, child: impl IntoView) -> Self {
    |                                    ^^^^^^^^ required by this bound in `HtmlElement::<El>::child`
    = note: this error originates in the macro `view` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0277`.

turning to this error:
error: use of deprecated method `i18n::builders::click_count_builder::<i18n::builders::EmptyInterpolateValue, __var_count>::build`:
       Missing interpolations keys: [""]
  --> src\app.rs:36:13
   |
36 |
{t!(i18n, click_count, count)}

   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
note: the lint level is defined here
  --> src\app.rs:36:13
   |
36 |
{t!(i18n, click_count, count)}

   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   = note: this error originates in the macro `t` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `counter` (lib) due to previous error

The changes are now available under the debug_interpolations feature, this feature is not default for now to avoid breaking changes but will be in v0.2


Add a more informative footer

Current footer has three links and all partners logos, but eventually we want a footer that can act as a mini sitemap and fits existing UI.

Fixed links and closed with 5147daad7bfe60dc5059f64c522cb693f19aa7c8


IP filter applied on multiple scopes not working as expected and documentation doesn't cover this behaviour

https://docs.microsoft.com/en-gb/azure/api-management/api-management-access-restriction-policies#RestrictCallerIPs
It doesn't seem clear in the documentation, but from testing against the echo API to validate what I was seeing while trying to enable IP filtering at multiple scopes, it doesn't seem possible to do this, e.g. Product and API scopes or API and operation scopes.
I'm not entirely clear if it's breaking the filtering entirely or just only using the most specific scope and ignoring those inherited from the base policy. I don't think it matters if the base policy is placed before or after the IP filter statement within the inbound policy element.
I hope this isn't too vague. Thanks.

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

ID: 248a0753-0fa7-1cc0-d13a-d378ab182e95
Version Independent ID: 7efa247a-7184-6e7f-c1f5-3ff2531df357
Content: Azure API Management access restriction policies
Content Source: articles/api-management/api-management-access-restriction-policies.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@WillParry, Thanks for the feedback! We are currently investigating and will update you shortly.

@WillParry Apologies for the delay on this one. I believe the policy should follow the usual order. I did a quick test and can confirm the same.
Also, you could check out the trace for more details on which IP Filter is allowing and denying access. Could you share your policies and traces for tests that you have done?

@WillParry Just following up here... Could you share more details as mentioned in my last comment?

@WillParry Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

I'm sorry, I've been really busy . . on top of which, just as I feel like I don't have time to deal with this now and the issue is closed, actually it's suddenly become a bit of a priority again. So let's see if I can put together a bit more information for you @PramodValavala-MSFT

Does some who is assigned this issue need to reopen it?

@WillParry That is all right. We can continue our discussion here. I will reopen this issue.

@WillParry The effective policy is the order in which policies are executed. So in your case, the first ip-filter policy rejects the request before proceeding further.
So, having both in your case would never be an option since you don't have any overlapping IPs apart from <IP_ADDRESS> and <IP_ADDRESS>.
To have both, you would want to have a wider if not equal range setup in the product scope and specific ranges/IPs setup at the API level. You would have to move the <base /> policy before the ip-filter policy at the API level as well.
Currently, just changing the base policy will not work since the second ip-filter will reject it even if the first one doesn't.

I see, so this isn’t clear from the documentation at all I don’t think. It would be great if it could be updated. Thanks for explaining how it works.
From: Pramod Valavala @.>
Sent: 23 November 2021 05:20
To: MicrosoftDocs/azure-docs @.>
Cc: William Parry @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] IP filter applied on multiple scopes not working as expected and documentation doesn't cover this behaviour (Issue #83459)
@WillParryhttps://github.com/WillParry The effective policy is the order in which policies are executed. So in your case, the first ip-filter policy rejects the request before proceeding further.
So, having both in your case would never be an option since you don't have any overlapping IPs apart from <IP_ADDRESS> and <IP_ADDRESS>.
To have both, you would want to have a wider if not equal range setup in the product scope and specific ranges/IPs setup at the API level. You would have to move the  policy before the ip-filter policy at the API level as well.
Currently, just changing the base policy will not work since the second ip-filter will reject it even if the first one doesn't.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/83459#issuecomment-976176154, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE63AUCU7JXXVY5GHQ6XTQTUNMQBRANCNFSM5HOBV46A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@WillParry Glad I could help! The policy docs do call out the order in which policies are executed, and ip-filter doesn't affect the order in any way per se.

@PramodValavala-MSFT :

To have both, you would want to have a wider if not equal range setup in the product scope and specific ranges/IPs setup at the API level. You would have to move the <base /> policy before the ip-filter policy at the API level as well.

Surely if the ip-filter has overlapping address or address-range elements, then it won't matter what order the base policy is placed in the api scope. If the overlap is present, then it's allowed if it hits one policy or the other or both as they all cover the required address or address-range?
I still feel that this could be clearer in the documentation, but appreciate that you explained it further for me here. Thanks again.
👍

@WillParry The overlap is required (for at least clients with those overlapped IPs) because both policies will be evaluated. Policies are not skipped unless explicitly mentioned like for example the return-response policy which aborts the policy pipeline execution.

If the overlap is present, then it's allowed if it hits one policy or the other or both as they all cover the required address or address-range?

From my statement above, both will have to allow the request through. So, only IPs that are covered by both the policies will be allowed (the overlapped IPs).
I suppose having more details on how policy execution works in general should help clear up any confusion in this regard. I hope my comments here are helpful to clear up any other confusions. We are assigning this issue to the content author for review.

@WillParry - I added a note to the IP filter policy documentation to note the scenario of multiple scopes and policy evaluation order, and this will publish within a day or so. Thanks for raising the issue.


Missing mixins crash on startup instead of showing a notification screen

I had it originally setup so that when mixins are missing the game loads and then shows a notification screen with a download link for mixinbootstrap. Now the game just crashes on startup without providing any information on what went wrong.
I'd really like to see this behavior being reintroduced. I know the code for it is a bit weird and ugly, but the user friendliness part clearly outweighs that, especially since having to install core mod dependencies is not a common thing for a user to have to do on Forge.

I'll see what I can do. :) I had to tweak some of the coremod logic because the mod was refusing to load correctly when in the dev workspace, which made development a lot harder. However, I should be able to reintroduce this somehow.


Typographic issues

I was trying LightTable, but did not work for me, I figured I could be a bit helpful before removing it.
Typographies are displayed with two different kinds of issues.
Branch
stable, last release
To Reproduce
Kubuntu 18.04, Kde plasma version 5.12.9
Screenshots
This is how the main interface looks like:

Minor typography glitches everywhere. If I report issue, it opens github page and interestingly the issues appear also there.

This is what the "open file" interface looks like:

Desktop (please complete the following information):

OS: Kubuntu 18.04, plasma 5.12.9
Version 0.8.1, binary 0.34.5

Duplicate of #2343

dupe of #2368 and #2343


make fails; dep not found

Hello developer team,
I wanted to install the btrdb-server on Ubuntu 16.04, but the make process fails with in the step:
cd $(GOPATH)/src/github.com/BTrDB/btrdb-server &&  dep ensure
and fails with:
/bin/sh: 1: dep: not found
Please can you let me know what could be the issue and which other OS would be preferred?
Thank you,
Christoph

dep is provided by go-dep https://github.com/golang/dep


Add recipe for adding/replacing a Maven runtime config

What's changed?
Introduces a new recipe for adding and replacing Maven runtime configuration in .mvn/maven.config or .mvn/jvm.config.
What's your motivation?
It's important for Maven projects to standardize on certain runtime flags and make them available through checked-in files the SCM. All developers that check out the code from SCM will  automatically apply those flags. For example, you may want to execute the Maven build in parallel for better build performance and therefore introduce -T=1C in maven.config.
Anything in particular you'd like reviewers to focus on?

The Maven documentation is not very specific which notation should be used for separating a flag from the assigned value if there is one (no separation, whitespace, = character. In fact, you will find all notations across different pieces of documentation. The implementation allows for providing this option.
The Maven documentation indicates that new lines should be used to separate configuration for maven.config. For jvm.config, you should use a whitespace. The implementation takes this into consideration.
Currently, the implementation does not take into consideration the alternative flag notation e.g.  -U vs. --update-snapshots. Please advise if you'd need this case to be handled with the initial PR and optimally "how".

@timtebeek Thanks for the polishing and merging. All makes sense and gives additional guidance for future PRs.


Update container-instances-container-groups.md

Clarify resource limit must be less than resource request

@dlepow @dkkapur

@macols - Thanks for clarifying. LGTM
#sign-off

@dlepow: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


OperationCanceledException when modifying JS files

I get the following whenever I modify JS files which are being used for SSR.
<TraceRecord xmlns="http://schemas.microsoft.com/2004/10/E2ETraceEvent/TraceRecord" Severity="Critical">
	<TraceIdentifier>http://msdn.microsoft.com/TraceCodes/System/ActivityTracing/2004/07/Reliability/Exception/Unhandled</TraceIdentifier>
	<Description>Unhandled exception</Description>
	<AppDomain>/LM/W3SVC/7/ROOT-2-130709595736844036</AppDomain>
	<Exception>
		<ExceptionType>System.OperationCanceledException, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089</ExceptionType>
		<Message>The operation was canceled.</Message>
		<StackTrace>
			at System.Collections.Concurrent.BlockingCollection`1.TryAddWithNoTimeValidation(T item, Int32 millisecondsTimeout, CancellationToken cancellationToken)
			at JSPool.JsEngineWithOwnThread.RunOnThread(Func`2 method)
			at JSPool.JsEngineWithOwnThread.RunOnThread(Action`1 method)
			at JSPool.JsEngineWithOwnThread.Dispose()
			at JSPool.JsPool.DisposeEngine(IJsEngine engine, Boolean repopulateEngines)
			at JSPool.JsPool.Dispose()
			at React.JavaScriptEngineFactory.OnResetPoolTimer(Object state)
			at System.Threading.TimerQueueTimer.CallCallbackInContext(Object state)
			at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
			at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
			at System.Threading.TimerQueueTimer.CallCallback()
			at System.Threading.TimerQueueTimer.Fire()
			at System.Threading.TimerQueue.FireNextTimers()
			at System.Threading.TimerQueue.AppDomainTimerCallback()
		</StackTrace>
		<ExceptionString>
			System.OperationCanceledException: The operation was canceled.
			at System.Collections.Concurrent.BlockingCollection`1.TryAddWithNoTimeValidation(T item, Int32 millisecondsTimeout, CancellationToken cancellationToken)
			at JSPool.JsEngineWithOwnThread.RunOnThread(Func`2 method)
			at JSPool.JsEngineWithOwnThread.RunOnThread(Action`1 method)
			at JSPool.JsEngineWithOwnThread.Dispose()
			at JSPool.JsPool.DisposeEngine(IJsEngine engine, Boolean repopulateEngines)
			at JSPool.JsPool.Dispose()
			at React.JavaScriptEngineFactory.OnResetPoolTimer(Object state)
			at System.Threading.TimerQueueTimer.CallCallbackInContext(Object state)
			at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
			at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
			at System.Threading.TimerQueueTimer.CallCallback()
			at System.Threading.TimerQueueTimer.Fire()
			at System.Threading.TimerQueue.FireNextTimers()
			at System.Threading.TimerQueue.AppDomainTimerCallback()
		</ExceptionString>
	</Exception>
</TraceRecord>

It used to happen in the 1.3.0 occasionally but in 1.4.0 its much more frequent. When it does happen it crashes IISExpress and requires restarting the app.
This is only an issue when developing, I haven't ever seen anything similar in production. My serverside JS file is quite large (over 1mb) as it contains all of my external references and isn't minified, which may be contributing to the issue.
I think its probably due to the application timing out trying to access the file while the file is locked for write.

Sounds like an issue with JSPool (the library that handles pooling/reuse of JavaScript engines) so I've moved this to https://github.com/Daniel15/JSPool/issues/6.
In the meantime, you can try turning off pooling and seeing if that helps. To do that, call . SetReuseJavaScriptEngines(false) in your ReactConfig.cs file. This will make execution slower (as it needs to spin up a new JavaScript engine for every request rather than reusing them) but should avoid this issue.

Yeah that totally seems to fix the issue. Ill just leave it off in dev.
Thanks for the fast response :)


[r2.11-rocm-enhanced] Increase timeout times in test scripts

Increase timeout needed in //tensorflow/python/kernel_tests/linalg:linear_operator_circulant_test_gpu test
SWDEV-375542

retest Ubuntu-GPU-single please

retest Ubuntu-GPU-single please


Customized Fybrik deployment

If fybrik user wants to use his services as a replacement to Fybrik components, we have to allow him customization of Fybrik deployment.
The candidates to be an optional components are:

Katalog
opa connector
opa server
vault server

@roytman I think chart deployment of fybrik has the option to choose other components if desired, for example, https://github.com/fybrik/fybrik/blob/master/charts/fybrik/values.yaml#L66

The mentioned above mechanism allows to deploy or not katalog, OPA connector and OPA server. It was validated by @rohithdv .
If a user doesn't use Vault and his modules do not use it either, we might allow to deployment Fybrik without Vault.


Always a NullPointerException on Android API less than 19

DocumentFile.fromSingleUri causes a NullPointerException as it always returns null if android API is under 19.
private static File fileFromUri(Context context, Uri data) throws Exception {
        DocumentFile file = DocumentFile.fromSingleUri(context,data);
        String fileType = file.getType();
...
}

The minSdkVersion of the Library is API 19. Please use it if you target API 19 or higher.

Your default config indicates a minimum version of 15, I see that a previous issue asked you to make it lower. Either they didn't test it or something has changed.

public static DocumentFile fromSingleUri(Context context, Uri singleUri) {
        final int version = Build.VERSION.SDK_INT;
        if (version >= 19) {
            return new SingleDocumentFile(null, context, singleUri);
        } else {
            return null;
        }
    }

Apparently it seems that below API level 19, they return null by default. I am not quite sure when this was changed, but I increased the minSdkVersion  of the library because of it. Will upload it later on the day on Maven and jCenter.

it possibile retrive a DocumentFile from InputStream/ByteArrayOutputStream/byte[]?


Support for subsampling large images in Compose

Hello, thanks for maintaining this library! Would it be possible to make the sampling behavior of SketchZoomImageView available to Compose UI as well?

Sorry, there are no such plans at the moment. The most important thing now is to release the stable version first, and there may be later.

正在开发当中，还得一段时间，进度已经过半了

FWIW, I recently released telephoto for displaying zoomable images with automatic subsampling of large bitmaps: https://saket.github.io/telephoto
At the time of writing this, Coil and Glide are supported, with plans for Picasso in the future. Let me know if anyone wants to see Sketch supported as well.

memmeme

希望看sketch的支持支持！！

A separate zoomimage (https://github.com/panpf/zoomimage) library is here, supporting compose and view. The logic and experience of the two versions are completely consistent.
The new zoomimage library is completely reconstructed on the basis of sketch-zoom, and the user experience is comprehensively improved. In addition to fixing some bugs, it also newly supports the animation and clarity gradient of subsampling tiles and supports glide, picasso, coil, and other  image loader
It has now reached the beta stage of public testing. Welcome to the release page to download the APK and experience the effect.


[Issue] No way to input on NEX-6

OpenMemories-Tweak installs and launches just fine on the NEX-6, and I can scroll up and down on the first page with buttons. I can't do anything else as it seems the app expects a touchscreen, which the NEX-6 does not have. I've heard that on other non-touchscreen cameras it functions correctly, but seems to not detect the NEX-6 properly.

It turns out I was running an older version of the app.


fnm doesn't work without steam-run

Describe the bug
node binaries installed with fnm are not patched.
Tried both unstable and 22.05.
Steps To Reproduce
Steps to reproduce the behavior:
nix-shell -p fnm steam-run
eval "$(fnm env --use-on-cd)"
fnm use v16.0.0
node --version # broken
steam-run node --version
v16.0.0

Expected behavior
It should work without steam-run
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.0.9, NixOS, 22.05 (Quokka), 22.05.git.cf63ade6f74`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.05, nixos-unstable"`
 - nixpkgs: `/persist/home/pearman/dev/jankaifer/nixos/modules/nixpkgs`

My config: https://github.com/JanKaifer/nixos/blob/438d501cb890441c8747511cca80e279dfa04225/machines/vercelframe/configuration.nix

cc @kidonng

For now, I just have these aliases:
pnpm='steam-run pnpm'
node='steam-run node'
corepack='steam-run corepack'

But it feels hacky.

This is not a steam issue. It's more of a general unpatched binaries issue.
Anyway I've been using https://github.com/Mic92/nix-ld for while now and it solved all my issues with node.

This is expected when downloading pre-compiled binaries. You can use something like nix-ld or steam-run or others to work around this.

To me it seems like a broken package if it doesn't work unless you implement some workaround that is not documented anywhere (spent non-trivial amount of time by figuring out why it's broken and how to fix it).
Is it at least possible to add some "installation notes" into the official nixpkgs package search here?
Do you know whether this was already discussed somewhere (I was not able to find it)? This probably is cross-package issue that is not specific to fnm.

Feel free to open a PR that expands the description to mention this.
IDK, generally any package manager that installs binaries is broken like this.


Use XML namespaces

Mike Ash has a good article describing Objective-C class initialisers, which I used when writing this one.
I generally changed nodesForXPath:error: to nodesForXPath:namespaces:error:.
Everything still seems to work!
Some kind of unit tests would be nice.

Great - though I must admit to never having touched unit-testing in Xcode - there's always more to learn!


chore(docs): fix typo in readme

Description of changes:
chore(docs): fix typo in readme
Check points: (check or cross out if not relevant)

[ ] Ran swiftformat (from repository root): swiftformat Sources Tests
[ ] Ran swiftlint (from repository root): swiftlint Sources Tests and addressed all violations.
[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all targets using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[x] Documentation update for the change if required
[x] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Good catch.
LGTM.


Refine Training by Adding Additional Years to the Dataset

Task Description:
Enhance the DeepSensor model's training process by incorporating additional years of data into the training dataset. This refinement is expected to improve the model's performance and its ability to generalize and capture temporal variability more effectively.
Checklist:

Identify Additional Data:

[ ] Determine the additional years of observational data to be included in the training dataset (your call - maybe start small and go from there?)

Data Preprocessing:

[ ] Preprocess the additional years of data to align with the existing training dataset format.
[ ] Verify the quality and consistency of the dataset, ensuring that it is compatible with the DeepSensor model's input requirements. (This is pretty much already done - it's just about adding more years)

Update Training Dataset:

[ ] Integrate the additional years of data into the existing training dataset.
[ ] Ensure the dataset is correctly formatted and ready for use in model training.

Retrain the DeepSensor Model:

[ ] Configure the DeepSensor environment to use the updated training dataset with the additional years of data.
[ ] Retrain the model and monitor the training process, documenting any improvements or challenges observed.
[ ] Evaluate the model's performance after incorporating the additional data and compare it to the baseline model to assess the impact.

Documentation and Results:

[ ] Create a Jupyter notebook to document the process of adding additional years to the training dataset, including data preprocessing, integration, training, and results analysis.
[ ] Save the retrained model and relevant output files.

@eredding02 Likewise, feel free to add some info here about your extended training dataset and close the issue when you're happy with it  : )

@DaniJonesOcean I extended the training set to use data from 2007-2019. I found that the 2006,2022, and 2023 GLSEA3 data all had at least one missing date, so they were excluded from training and validation. I used 200 randomly selected dates throughout 2020-2021 as validation and the training consisted of 40 epochs.


Changes/corrections to Kubernetes-Mesos Getting Started

Targeting the following changes/corrections
WIP Don't Merge Yet

[x] Update Pre-requisites section with proper VPN Link
[x] Add to pre-req (knowledge and understanding of Mesos)
[x] Modify doc to bring up Mesos admin GUI up early for verification
[x] Modify docker run for etcd to address flag for etcd
[x] Clarify how to validate slave/pod tasks via GUI
[x] Reorder some instructions so instructions flow better
[x] Clarify mesos tail -f c7315c... instruction / maybe through GUI (Removed)
[x] Added additional headers in more places to separate content

Nice improvements, thanks @vladimirvivien.


Incorrect message schema reference

In this section, please remove the highlighted line.

In this section, please update the hyperlink to: https://learn.microsoft.com/en-us/microsoftteams/platform/resources/bot-v3/bot-conversations/bots-conversations#full-inbound-schema-example

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

ID: 82e5e411-8803-b85f-020e-2724b8a441a6
Version Independent ID: 751234e9-e420-d3ce-893f-02c572b68b81
Content: Channel and Group chat conversations with bots - Teams
Content Source: msteams-platform/resources/bot-v3/bot-conversations/bots-conv-channel.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Hi @surbhigupta12 - Could you please help here to update the doc as mentioned?


Fix setTimeout memory leak

Steps to reproduce

login user -> logout user
login user -> user disconnected ->  re-authenticate user

Expected behavior

previous logoutTimer (setTimeout) should be cleared

Actual behavior

logoutTimer still stays

Reason

this one will never be true https://github.com/feathersjs/authentication/blob/master/lib/socket/handler.js#L110 if the user was disconnected then tried to re-authenticate.

should clear the timeout here too https://github.com/feathersjs/authentication/blob/master/lib/socket/handler.js#L59 when socket is disconnected and user tried to reauthenticate.

Could you re-submit this against master?

I did here https://github.com/feathersjs/authentication/pull/653

Cool. This one is closed via #653 then.

@daffl can you fix the auk too because we have projects running on that version.

Other than Node 4 support, @feathersjs/authentication and related packages did not change so they should be backwards compatible and can be upgraded individually.
Eventually you'd probably want to upgrade anyway. The process is mostly automated and should be fairly quick. So far we haven't heard of anybody having major issues with it and it'll reliably get you the latest versions of everything.


expect: Improve report when matcher fails, part 12

Summary
For .toThrow and .toThrowError matchers:

Display not following expected label
Inverse highlight not-expected substring or pattern in received error message
Omit received error message for object instance because criterion is === strict equality

Residue

Improve report for error class if received error.name is not the same as error.constructor.name probably together with .toBeInstanceOf matcher
Breaking change to require .exec method in addition to .test method for RegExp

For more information, see discussion with @jeysal in #7795
Test plan
Change error message to more realistic "Invalid array length" in 6 .not snapshots
Chore in snapshot names:

replace .toThrow() and .toThrowError() with toThrow and toThrowError
replace strings with substring

Updated existing tests: all 60 names because of chore and snapshots:

expected

asymmetric
8

error-message
2

regexp
4

substring
4

See also pictures in following comment.

Example pictures baseline at left and improved at right
error message includes the substring:

error message matches the pattern:

error message is equal to the message property of the object:

thrown value matches the asymmetric matcher:


⚠ Bofrost.at has degraded performance

In ef6fbe8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 820 ms

Resolved: Bofrost.at performance has improved in e0122f6 after 19 minutes.


wrapLatestCommitMsg: fix behaviour in windows

While putting Environment.NewLine in output doesn't cause any problems expecting Environment.NewLine in input commit msgs causes problem in windows, so while keeping Environment.NewLine for output we change input processing to split by unix newline.
Fixes https://github.com/nblockchain/conventions/issues/123

@realmarv please review

Sorry Afshin, this PR didn't respect the KISS principle ;) I submitted a simpler fix here: 4c7790fd217c92e8f51e28aef84d056a29094674


Update the game over message

Update the game over message so people know how to play again!

Bekzodni qo'shdim


remove Create DAO Proposal when issuing an Audited Emissions in react app

Please hide the "Create DAO Proposal" button when issuing an Audited Emissions in react app.

DAO proposal button now always disabled for for audited emissions tokens on interface in https://github.com/opentaps/blockchain-carbon-accounting/commit/b3a7c3df71ca23d90f4b379b14b0e9013b50cb68

Can we remove the Create DAO Proposal button altogether, instead of just turning it inactive, when issuing audited emissions?


Checkbox + Radio are not properly vertically aligned

Description
The Checkbox and Radio components are not well aligned in the vertical direction, something which appears to be more significant when using Firefox. It would be good if the components were adjusted to better visually align vertically, as the current state almost appears erroneous.

Using Firefox 107.0.1 (64-bit)
Steps to reproduce

Go to https://primer.style/react/Checkbox
Go to https://primer.style/react/Radio

Version
Latest
Browser
Firefox

@mperrotti would this be impacted/fixed by: https://github.com/primer/react/pull/2701? 👀

Hey I am a newcomer in this project and I would like to contribute

Thanks for your interest to contribute, @govind15496! We already have a fix for this in the open PR #2701, but feel free to review other good first issues in this repo.

Closed by #2701


Jellyfin - Use API key instead of username/password

Description
Jellyfin has an API and it's very easy to request (and control) an API key that could be used by Homarr.
All of the *arr integrations already use an API key for access, and Jellyfin should too.
Priority
Low (Nice-to-have)

:+1:

I love to have this. I am using Jellyfin without a password and I can't add integration to homarr without password unfortunately.

Closed by https://github.com/homarr-labs/homarr/pull/127
This feature will be part of v1.0, see https://github.com/ajnart/homarr/issues/1993


chore: modify AgentBaker windows test

What type of PR is this?
/kind improvement

What this PR does / why we need it:

Modify workflows (clarify title and logs)
Support 23H2 in e2e pipeline.
Only use 2022 gen1 and 23H2 gen2 as default to reduce cost.

Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
2566

Relevant Lines:
3613

💛  - Coveralls


vault details - add dollar value for loans card

What would you like to be added:

Why is this needed:

@jtlntno-cake Will need design for this

Adding design link here for reference
Reference for different states of Loan cards here


Open Tendermint port 26657 to be able to access Tendermint Rest Interaface.

For the development of a monitoring and alerting tool it would be greatly helpful if the port was exposed so that I could access something like this : https://docs.tendermint.com/master/rpc/#/Websocket . Thank you.

We have our own RPC APIs that are exposed via the internal AF_LOCAL socket located in the node's data directory (called internal.sock). The used RPC protocol is gRPC with the CBOR codec and the exposed APIs try to be as agnostic to the underlying consensus protocol as possible (yes, we currently do use Tendermint, but we want to expose as little of that directly as possible).
These RPC APIs will never be exposed directly via TCP as some can be unsafe to use -- if you need this for a specific use case, a proxy should handle that translation, performing things like authentication, access control, rate limiting, etc.
If there is something specific missing from our RPC APIs, please file a more specific issue. At the moment there is no RPC-specific documentation besides E2E tests, example CLI utilities and the GoDoc reference for packages implementing the APIs.


Can i open iframe with a particular id in flutter web?

Actually i want to use html renderer but when i am doing so, i am not able to get iframe element from the document object in javascript using below code :
var iframe = document.getElementsByTagName('iframe');
but it works fine in canvasKit renderer.
Please tell if it is possible to get iframe or to start iframe with my own defined ID.

Hi, as far as I know, on the html renderer the iframe is somehow hidden inside the document's shadowRoot.
It has nothing to do with IDs (as long as you query by tag name and not by id).

After evaluating this, I could implement it but I'm not sure this is a good idea because the iframes are being created with a random ID for a specific reason, and that is because otherwise the user might forget to pass an ID, or might pass the same ID twice, and the iframe(s) will start to behave weird.
So the best advice is to use the methods from the webviewXController to do/call whatever you need, but keep the IDs random.
Closing this for now, please open a new issue if you need more help.


Hash rate performance feedback mandatory! [Feature]

🚀 Feature Hash Rate Performance Feedback

Motivation
There is no possibility to check performance yet, so why should anyone mine this especially with current market and electricity prices if there is no feedback?

I'm not sure about the plans for this feature, but you might be able to use what one of the contributors proposed in a PR:: https://github.com/AleoHQ/snarkOS/pull/2133.

The performance of each prover is listed on the leaderboard:
https://www.aleo.network/leaderboard


Pb with annotations

I think you should add
from __future__ import annotations

If not, you get TypeError: unsupported operand type(s) for |: 'type' and 'type' in python 3.9

Thanks. Pushed a new release.


Added homepage column option to csv_output

Added the homepage column option to csv output.

Merged. Thanks!


Ensure that invalid cache entries are always deleted

Codecov Report
Attention: Patch coverage is 33.33333% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 92.42%. Comparing base (8f72b13) to head (5b92bc2).

Files
Patch %
Lines

src/blueetl/cache.py
33.33%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   92.10%   92.42%   +0.32%
==========================================
  Files          45       45
  Lines        2508     2509       +1
  Branches      495      495
==========================================
+ Hits         2310     2319       +9
+ Misses        126      119       -7
+ Partials       72       71       -1

Flag
Coverage Δ

pytest
92.42% <33.33%> (+0.32%)
:arrow_up:

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

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


Search by ticker invalid?

Bug Report
If I search "META"; I get no response when I should be getting "Meta platforms"
If I search "ME"; I get multiple responses including "META"
Did I miss something?
Minimal Reproduction
let searchResult = await yahooFinance.search("META", {lang:'en-US',quotesCount:10,newsCount:0})
Environment
Node 18.17
yahoo-finance2: 2.4.2

Interestingly, If I write FB I get META 😅

Hey @Benjythebee, thanks for the report and for your patience.  I'm unable to reproduce this now.  Yahoo's services can be... finicky... at times.  It looks like they've fixed this on this end now.  Please feel free to re-open if you experience it again.


Feature Request: Blade Support for Patterns

Terms

[X] I have read the guidelines for Contributing to Roots Projects
[X] This request is not a duplicate of an existing issue
[X] This is not a personal support request that should be posted on the Roots Discourse community

Summary
With WordPress 6.0 it got very easy to register new Patterns by throwing them as PHP files into the /patterns folder: https://make.wordpress.org/core/2022/05/02/new-features-for-working-with-patterns-and-themes-in-wordpress-6-0/
That folder is located in the theme root, also with Sage 10. It would be great to have Blade support for the Pattern files and to move the folder to the other /views.
Motivation
Why are we doing this?
Patterns are very powerful and can be used for Block Templates.
What use cases does it support?
With Blade support it would be possible to build on the pattern system also with FSE outlook.
What is the expected outcome?
Blade support within Pattern files
Additional Context
No response

https://github.com/WordPress/WordPress/blob/a5cc224b3c5a5d983673f5f8ec4b3a7ae9e97f27/wp-includes/block-patterns.php#L225 unfortunately wordpress doesn't provider a filter for the patterns directory
i think that the ideal scenario would be that patterns would be created at resources/views/patterns and compiled to somewhere that isn't patterns. but since we can't do that, we would then have to gitignore the theme patterns directory.
it seems like whatever way we were to do this right now would be a bit janky. it would be nice if core would support customizing the path for this (and other) editor files and directories (like theme.json).

Closing for now. Poet by @Log1x supports registering patterns with Blade


style(container-creation): clarify ports mapping

additional changes to previous PR for #2864

@itsconquest please open a new issue and link to that PR. Also reference that previous issue in there.


Add Jungle Amulet and Talisman of Power line

@FantasmicGalaxy you should change this issue to only track the Jungle Amulet

Jungle Amulet is done, Talisman of power is handled here https://github.com/SkyCryptWebsite/SkyCrypt/issues/624
Closing


Segmentation fault while converting Bert-base-uncased with README command

System Info

transformers.js 7f5081da29c3f77ee830269ab801344776e61bcb
Operating System Mac OS Sonoma 14.5
Macbook M1
Python 3.11.5
transformers==4.33.2
onnxruntime==1.15.1
optimum==1.13.2
onnx==1.13.1

Environment/Platform

[ ] Website/web-app
[X] Browser extension
[ ] Server-side (e.g., Node.js, Deno, Bun)
[ ] Desktop app (e.g., Electron)
[ ] Other (e.g., VSCode extension)

Description
When I try to run the conversion script with the command python -m scripts.convert --quantize --model_id bert-base-uncased I get a segmentation fault.
(default) ➜  transformers.js-main python -m scripts.convert --quantize --model_id bert-base-uncased
Intel MKL WARNING: Support of Intel(R) Streaming SIMD Extensions 4.2 (Intel(R) SSE4.2) enabled only processors has been deprecated. Intel oneAPI Math Kernel Library 2025.0 will require Intel(R) Advanced Vector Extensions (Intel(R) AVX) instructions.
Intel MKL WARNING: Support of Intel(R) Streaming SIMD Extensions 4.2 (Intel(R) SSE4.2) enabled only processors has been deprecated. Intel oneAPI Math Kernel Library 2025.0 will require Intel(R) Advanced Vector Extensions (Intel(R) AVX) instructions.
/Users/maxfrax/opt/anaconda3/envs/default/lib/python3.11/site-packages/transformers/utils/generic.py:311: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  torch.utils._pytree._register_pytree_node(
Framework not specified. Using pt to export to ONNX.
Some weights of the model checkpoint at bert-base-uncased were not used when initializing BertForMaskedLM: ['cls.seq_relationship.weight', 'bert.pooler.dense.bias', 'bert.pooler.dense.weight', 'cls.seq_relationship.bias']
- This IS expected if you are initializing BertForMaskedLM from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
- This IS NOT expected if you are initializing BertForMaskedLM from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
Automatic task detection to fill-mask (possible synonyms are: masked-lm).
[1]    6178 segmentation fault  python -m scripts.convert --quantize --model_id bert-base-uncased

Do any of you have any pointer to pinpoint the root cause?
Reproduction

Download the repository as a zip
Extract the folder and move into it from the terminal
Run the README command python -m scripts.convert --quantize --model_id bert-base-uncased

Hi there 👋 Can you try in an environment like Google Colab? This might be an issue with your system/environment. Also, if you only want to use that model, you can use my pre-converted one here: https://huggingface.co/Xenova/bert-base-uncased


Added import cmd

This will probably fail against CI, but I can't run integration tests locally because train wifi.

Rerunning kismatic install plan with an existing cluster name doesn't fail
Panic when not enough arguments are provided

➜  out-darwin git:(import-cmd) ./kismatic install apply
Usage:
  kismatic install apply CLUSTER_NAME [flags]

Flags:
  -h, --help               help for apply
  -o, --output string      installation output format (options "simple"|"raw") (default "simple")
      --restart-services   force restart cluster services (Use with care)
      --skip-preflight     skip pre-flight checks, useful when rerunning kismatic
      --verbose            enable verbose logging from the installation
panic: runtime error: index out of range

goroutine 1 [running]:
github.com/apprenda/kismatic/pkg/cli.NewCmdApply.func2(0xc42015a000, 0x19d2f20, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/apprenda/kismatic/pkg/cli/apply.go:46 +0x3ae
github.com/apprenda/kismatic/vendor/github.com/spf13/cobra.(*Command).execute(0xc42015a000, 0x19d2f20, 0x0, 0x0, 0xc42015a000, 0x19d2f20)
	/go/src/github.com/apprenda/kismatic/vendor/github.com/spf13/cobra/command.go:746 +0x475
github.com/apprenda/kismatic/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0xc420091200, 0xc42015ab40, 0xc4200918c0, 0xc420091680)
	/go/src/github.com/apprenda/kismatic/vendor/github.com/spf13/cobra/command.go:831 +0x30e
github.com/apprenda/kismatic/vendor/github.com/spf13/cobra.(*Command).Execute(0xc420091200, 0x14, 0x16d4580)
	/go/src/github.com/apprenda/kismatic/vendor/github.com/spf13/cobra/command.go:784 +0x2b
main.main()
	/go/src/github.com/apprenda/kismatic/cmd/kismatic/main.go:29 +0x20a

Should be fixed.

Trying to add a new node as discussed by modifying the plan file:
➜  out-darwin git:(import-cmd) ./kismatic install apply dkoshkin-test

Validating==========================================================================
Reading installation plan file "clusters/dkoshkin-test/kismatic-cluster.yaml"   [OK]
Validating installation plan file                                               [ERROR]
- Worker nodes: Expected node count (2) does not match the number of nodes provided (1)
error validating plan: Plan file validation error prevents installation from proceeding

You need to provision first.

Looks like kismatic certificates command didn't get updated.

Updated how?

Oh, the help text?

Yes the help and the command itself, I would expect the command to use CLUSTER_NAME

Alright, addressed that as well. I was wonder if it was worth differentiating between a cluster name and a cert name. I opted not to merge the two, but if you think it makes sense, sure.


Query Builder

As PHPCR-ODM 1.3 has now been released we can now implement the QueryBuilder for the Document Manager.
For example:
        $builder
            ->setLocale('de')
            ->from()->document('page', 'p')->end()
            ->useStructure('p', 'overview')
            ->where()->eq()->field('p.structure#article')->literal('hello');

        $query = $builder->getQuery();

@dantleech Have you fixed your own comments here?

I am not sure there current form is the best API, what about the following?
Current
        $builder
            ->setLocale('de')
            ->useStructure('p', 'overview')
            ->from()->document('page', 'p')->end()
            ->where()->eq()->field('p.structure#article')->literal('hello');

Proposed, use structure instead of useStructure and structureField('p.article') instead of `field('p.structure#article'):
        $builder->structure('p', 'overview');
        $builder->where()->eq()->structureField('p.article')->literal('hello');

This now supports ->structureField('p.article') instead of field('p.structure#article').
Note: there is a current issue with orWhere() and andWhere() not sure if this is an issue with the PHPCR-ODM iteself.

orWhere problem seems to be a Sulu issue rather than a PHPCR-ODM one.

@danrot @chirimoya @wachterjohannes ready for review.

PHP Fatal error:  Class 'Sulu\Component\DocumentManager\Query\QueryBuilderConverter' not found in /home/travis/build/sulu-io/sulu/src/Sulu/Component/Content/Document/Query/StructureQueryBuilderConverter.php on line 33

Guess we need to merge the DocumentManager PR or update composer.json here.

Added dev-query_builder_final as DocumentManager version.

@dantleech Would have liked to review, but some of the tests are failing :confused:

Same as in https://github.com/sulu-io/sulu-document-manager/pull/60, need to have a deeper look at it. If you can fix all my comments I will have this deeper look next week.

Updated.

Tests are located in the old directory.

Updated and moved tests

Updated.

We are also updating to phpcr-odm 1.3 here, right? Then we could probably remove these lines: https://github.com/sulu-io/sulu/blob/7bbdcd18d50a320ad2c7e6286138a1b8a11d7901/src/Sulu/Bundle/CoreBundle/DependencyInjection/SuluCoreExtension.php#L44
The linked issue is already closed.

@danrot lets not remove those lines in this PR. It is a separate issue and I don't know what side effects that will have or if it will be picked up by the tests.

@dantleech Could you give a more elaborate example of how this would be used?

@webmozart it is based upon the PHPCR-ODM query builder, every thing here applies to this query builder, we add the useStructure and structureField methods allowing you to query based on the "Structure" fields - the things that hold the actual user content in Sulu.

@danrot thanks!


Have jenkins plugin publish scm information metadata when publishing artifacts

Based off of code base for #815.
The following metadata is now published with the jenkins plugin

scmUrl
scmBranch
scmRevisionNumber

rebased

Cherry-picked code and put it in #815


Handle too long dates passed to DateTime.parse

Since Ruby 3.0.3, this method raises an ArgumentError with a different message from the invalid date message, as mitigation for CVE-2021-41817.
ArgumentError: string length (150) exceeds the limit 128

This PR handles this error to return nil, just like when the date is invalid.

Thanks a lot for the review, @mikel! I'll get these changes implemented this week, I hope.

Implemented the changes requested 😊

Unfortunately, that is still an issue in 2024 with Mail 2.8.1 on 3.3.0.
That exception also breaks the POP functionality, as it crashes without option to rescue the individual mail:
mails = Mail.all delete_after_find: true

ArgumentError: string length (226) exceeds the limit 128 (ArgumentError)

          datetime = ::DateTime.parse(stripped)
                                      ^^^^^^^^
  from mail (2.8.1) lib/mail/fields/common_date_field.rb:17:in `parse'
  from mail (2.8.1) lib/mail/fields/common_date_field.rb:17:in `normalize_datetime'
  from mail (2.8.1) lib/mail/fields/common_date_field.rb:31:in `initialize'
  from mail (2.8.1) lib/mail/fields/common_field.rb:12:in `new'
  from mail (2.8.1) lib/mail/fields/common_field.rb:12:in `parse'

Any chance, this PR gets merged?
For now, we need manual intervention and delete the mail from the inbox.


Misc. UI Changes For Consistency with HN

[x] Highlight "hntop" in white on the hntop page (the original HN does the same for all pages but the front page)
[x] Show "Discuss" instead of "0 Comments" when there are no comments
[x] "N minutes ago" should link to the story page
[x] Username should link to user page
[x] Show domain after story title
Mobile/smalls viewports:

[x] slightly larger right margin  (just a few pixels it appears to me)
[x] wrap after a |

[ ] Do we want the same margins around the page content (grey background) as the original
[x] when you hover over a relative time (e.g. 2 days ago) on HN, it shows the iso timestamp.
[ ] usernames for new users appear in green.

Another one: when you hover over a relative time (e.g. 2 days ago) on HN, it shows the iso timestamp.

@johnwarden what exactly do you mean with:

"slightly larger right margin (just a few pixels it appears to me)"
"wrap after a |"

I don't think we need margins to be exactly the same as on HN. We have our own page layout which just looks similar.
New users appearing in green are not essential for launching on HN


Buttons get temporarily styled on SvelteKit before going back to unstyled

Once you load the page you're able to see the styled buttons but they immediately get unstyled
https://user-images.githubusercontent.com/30603437/133248943-7c035ca1-e240-4637-bc79-b2f3704cc8a2.mp4
Here's my setup
// __layout.svelte

  <SocialShare />

%svelte.body%

// SocialShare.svelte

Follow @itzami

Sponsor

This is by design to support browsers that does not have JavaScript enabled.
If you don't like it, try use data-text option. For example:

Sorry, misunderstood your issue. The problem is that SvelteKit is a front-end rendering framework that I think it apparently is override the renderred buttons. Typically there are two options:

Move the button outside of the app's DOM mount point, so that it becomes static and does not get reloaded.
Write a plugin for the framework that dynamically render the button.  For example, like react-github-btn.

SvelteKit isn't popular enough so I'm not considering to write a component for it, but feel free to implement one if you want to do so.


.

.

Duplicate of #44

can you please delete this issue?
sorry...

@moneytoo I'm sorry for bothering you, but can you please delete this issue? I can't do it myself


vafu and neca binaries not installed

Hi,
Wondered if I could get a pointer as when I run the tool I get the following errors:

Can't load siqs because yafu binary is not installed
Can't load neca because neca binary is not installed

There were a number of sage issues that were fixed by following another issue topic and installing the sage binary. However I could not find any details of the yafu and neca binaries.
Thanks in advance - K

I'll update the readme as soon as i can

@Kartibok:
I think the YAFU problem can be solved easily. I have a symlink from
RsaCtfTool /attacks/single_key/yafu
to /usr/local/bin
placed. Alternative you must Change your PATH ....
regards Laubi

The yafu binary in that location only yields a segmentation fault on my Linux x64 system.


Allow AniamtedIcon.State to be set on any ancestor.

Change animated icon to, when the icon is loaded, walk up the tree looking for an ancestor which has set the AnimatedIcon.State property. If one of these is found it subscribes to that elements property changed event for the AnimatedIcon.State property. This way we support AnimatedIcon.State being set on any ancestor of the icon. This does however require that the state property be set before the icon is added to the tree.

/azp run

/azp run

/azp run

@jevansaks do you anticipate any leak issues with this property changed event handler? I'm not removing the handler when in the unloaded event because of the issue where we can receive those events in the wrong order.  But does this mean that if you remove the animated icon from the tree it wont get collected?


Improper Spacing Between Word and Arrow

The spacing between the 'Question Papers' and the arrow icon is inconsistent or too tight, which affects the overall readability and design.

I propose adjusting the CSS to add proper spacing.
Please assign to me .

Hi @ShrutiDesai7  , can you assign this issue to me.

Hi , as i can see its still not fixed can you assign me this issue i would like to try  @ShrutiDesai7


⚖ Register Validator: OnThePluto

Network Choice
nemeton-1
Who are you?
OnThePluto
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"OnThePluto","identity":"801B6CC31C2431D4","website":"https://onthepluto.com","security_contact":"","details":"Lives in Cosmos (100% safe)"},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"okp41rusmaarg7l0khtucmm53grdghjjle8lnyk0afs","validator_address":"okp4valoper1rusmaarg7l0khtucmm53grdghjjle8ln33l5h3","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"FzxfJBsC91ZgeObKtZFdf58li9d58cllK2GpUwo14aw="},"value":{"denom":"uknow","amount":"10000000000"}}],"memo":"ead118d7cbe51cbabf5a77b69db7255512f41023@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AktRfGwV+v04NVogVNANHx0iWDXVPmyFLirF/9J0V44w"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["IG2AZqrYEyYXapOOAvfTWmEdTkaC/tqgVQJvd3bNJRNZrnCSyL3gpQr+rIGYS0jLrcCp1BePWY0qRLpaohZjjw=="]}

😉 Here is the corresponding PR: https://github.com/okp4/networks/pull/124


Support Grades view with 100% score to be inclusive in the x-axis

and not look like the course has students got more than 100%
Also cleaning up the UI

Done