Add an #[unsorted] attribute to allow certain cases to be ignored

Add #[unsorted] attribute to be used inside of blocks marked with #[sorted]. The attribute allows for cases marked with it to be ignored by the checks.
The attribute can be used as #[remain::unsorted] or #[unsorted].
The attribute is not a real attribute macro and is stripped when the checks for #[sorted] are done, but is labelled as such due to the usage of it.

Example 1
#[remain::sorted]
pub enum TestEnum {
    Aaa,
    Bbb,
    #[remain::unsorted]
    ThisIsIgnored,
    Ccc,
    Ddd,
}

Example 2
#[remain::sorted]
pub enum TestEnum {
    Aaa,
    Bbb,
    Ccc,
	#[unsorted]
	#[doc(hidden)
    __Nonexhaustive,
}

Here's an example of my use case in a Discord library I'm working on.

Example
/// Buckets grouping [rate limited] routes.
///
/// [rate limited]: https://discordapp.com/developers/docs/topics/rate-limits#rate-limits
#[remain::sorted]
#[derive(Clone, Copy, Debug, Eq, Hash, PartialEq)]
pub enum Bucket {
    /// Route:
    /// ```text
    /// /channels/{channel.id}
    /// ```
    ChannelsId(ChannelId),
    /// Route:
    /// ```text
    /// /channels/{channel.id}/invites
    /// ```
    ChannelsIdInvites(ChannelId),
    /// Route:
    /// ```text
    /// /channels/{channel.id}/messages
    /// ```
    ChannelsIdMessages(ChannelId),
    /// Route:
    /// ```text
    /// /channels/{channel.id}/messages/bulk-delete
    /// ```
    ChannelsIdMessagesBulkDelete(ChannelId),
    /// Route:
    /// ```text
    /// /channels/{channel.id}/messages/{message.id}
    /// ```
    ChannelsIdMessagesId(ChannelId),
    /// Route:
    /// ```text
    /// /channels/{channel.id}/messages/{message.id}
    /// ```
    ///
    /// This has a separate bucket from [`ChannelsIdMessagesId`].


feat: Make compatible with Ts Projects

I noticed that the package is not fully compatible with Ts projects as it suffers from Type negation.
TODO

Rewrite the declaration file to make it compatible with Ts

Resolved in 2.0.3


Notification pops up even when no tests are run

🐛 Bug Report
Jest notifies me as follows that 100% of 0 tests passed when I save any file in my watched directories, even if there are no tests associated with it. What is the utility of this?
To Reproduce
Steps to reproduce the behavior:

Run jest with --watch --notify
Save a file that is untested
Notification like so shows up and console says No tests found related to files changed since last commit.

Expected behavior
Notifications are only shown when tests are run

jest 23.1.0
node 8.11.2
macOS 10.13.4

PR welcome!
I think a notification might make sense? But it should say "no tests ran" or something.
Code is here: https://github.com/facebook/jest/blob/master/packages/jest-cli/src/reporters/notify_reporter.js

Happy to pick this one up.
What's the proposed format as currently the notifier expects a message and a title?

Title maybe "No tests were run", and an empty message?

My thinking was that there should be no notification unless some tests are run. What’s the point of a notification that says nothing ran?

Yeah this makes sense to me as well as the expected behaviour. Could simply check the total tests run and exit early from the notifier logic if numTotalTests === 0?

@thymikee @

I think such "empty" notifications may be annoying, especially when fiddling with git a lot. I'm OK with not displaying it whenever there were no tests run.

Please, add an options to notify in case of faled test only

Failed only is already an option: https://facebook.github.io/jest/docs/en/configuration.html#notifymode-string

@SimenB sorry, not noticed (

Closed via #6495


update(config): Improve http output, add compression and keep_alive

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
This PR adds two new configuration options for the http output:

[X] Post compression
[X] Connection keep alive
[ ] Batching

Which issue(s) this PR fixes:

Fixes #2955
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

feat(userspace/falco): falco administrators can now configure the http output to compress the data sent as well as enable keep alive for the connection. Two new fields (compress_uploads and keep_alive) in the http_output block of the `falco.yaml` file can be used for that purpose. Both are disabled by default.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

43ac9cd feat(http_output): implement support for compressed upload
ef96640 feat(outputs_http): implement keep alive

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sgaist
Once this PR has been reviewed and has the lgtm label, please assign andreagit97 for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Recommend not marking this as feat and instead as new feature that doesn't break the old default behavior. More thoughts?

What do you mean?
A feat does not necessarily impose a breaking change. In this case, no breaking change in the default behavior is to be expected, right?
I think we are already ok with this one.

LGTM label has been added.  Git tree hash: 1c103b6386c1efb7bbb25b90049771303ae62101

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold I think @sgaist is still looking into possibly adding batching :upside_down_face:

@incertum I am currently waiting on an answer with regard to the batching part which will likely require a bit more work and design discussions.
What could be done is to split things and implement said batching in a follow up patch so the benefits of compression and keep alive are already made available. This would only require a bit of rewording of the pull request message.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Don't if the batch will be really useful, very few targets accept that. On the other hands, having more control on the http output would be a really nice thing:

allow to set the User-Agent
allow to set the Headers (really useful for Authentications either)
allow to set the Http Method (PUT/POST)
allow to use mTLS for the authentication

@Issif based on the current code of the http output, the following are already available:

User agent setting
mTLS configuration

When you say authentication, are you thinking about setting a username/password or a token in the authorization header or something more involved like oauth ?


Improved struct conversion

The package now converts directly to a struct instead of to json first
Fixed minor issues

Conversion to structs has fundamentally changed. Concerns issue #43.
Maybe it fixes the issue.


[SPARK-44268][CORE][TEST][FOLLOWUP] Add test to generate sql-error-conditions doc automatic

What changes were proposed in this pull request?
This is a follow up PR for #41813, change test to automatic generate sql-error-conditions doc. Like other GOLDEN_FILES test, if the doc not match will report a error during test.

Why are the changes needed?

make sure the error-classes.json sync with doc
and developer sync doc easier.

Does this PR introduce any user-facing change?
No

How was this patch tested?
add new test.

Merged to master.


story/TYTA-98/securize-critical-backend-routes

Description

[x] Refactor of user model to change user role implementation
NB: I added an email validation in the backend but not a big change, I would try it 😃
[x] Edit auth middleware to add role in request data
[x] Normalize middleware naming
[x] Edit routes with authorization middleware
[x] Edit swagger with new results and update route name when is necessary
[x] Add some comments in routes to quickly see what a route is for

No merge without me please, I touch a lot of file!

I read all, need to test it tomorrow morning but seems nice. Nice implementation of admin role !


Detect RBENV_DIR properly from file argument to ruby

I believe there are 2 weird behaviour relating to RBENV_DIR.  This should fix both 2.

file argument does not follow symlink (yyuu/pyenv#404)
file argument works weirdly if first argument doesn't contain / (yyuu/pyenv#479)

Both can be reproduced with the following setup.
$ mkdir -p ./foo
$ echo 2.2.3 > ./foo/.ruby-version
$ echo 'puts(RUBY_DESCRIPTION)' > ./foo/test.rb
$ mkdir -p ./foo/bar
$ echo jruby-<IP_ADDRESS> > ./foo/bar/.ruby-version
$ ln -sf ${PWD}/foo/test.rb ./foo/bar/

If the first file argument to ruby interpreter is a symlink, it'd be better to follow the link.
$ ruby ./foo/test.rb
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
$ ruby ./foo/bar/test.rb
jruby <IP_ADDRESS> (2.2.2) 2015-07-21 e10ec96 OpenJDK 64-Bit Server VM 25.72-b05 on 1.8.0_72-internal-b05 +jit [linux-amd64]

RBENV_DIR just not working if the first file argument doesn't contain /.
$ cd ./foo
$ ruby test.rb
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
$ ruby test.rb ./bar/test.rb
jruby <IP_ADDRESS> (2.2.2) 2015-07-21 e10ec96 OpenJDK 64-Bit Server VM 25.72-b05 on 1.8.0_72-internal-b05 +jit [linux-amd64] # <-- this would be better to be 2.2.3

Resolved conflict.

Thank you for the implementation, explanation, and the ensuing discussion, but I remained unconvinced that this is a change I want in rbenv, and in the interest of keeping rbenv stable, I'm going to close this. Furthermore, I stopped resolving symlinks in most places within rbenv: https://github.com/rbenv/rbenv/pull/1428
I believe that if you call ruby /path/to/file.rb, the /path/to/.ruby-version should be respected and the actual destination of the /path/to/file.rb symlink shouldn't really matter.


🐛 [Bug]: redis: client is closed error

Bug Description
I am using RedisStore in my application and this error comes from nowhere. On my pc when i try to get some key from Redis "redis: client is closed" comes sometimes. But on the server its always disconnected.
// Redis Client
var RedisStore = redis.New()

// When accessing redis
var redis = database.RedisStore.Conn()
var state = redis.Get(ctx, fmt.Sprintf("state.user.%d", user.ID))
if state.Err() != nil {}

How to Reproduce
Steps to reproduce the behavior:

Create new RedisStore
Try to access some key from store

Expected Behavior
It gives the value from store without error
Storage package Version
v1.3.4
Code Snippet (optional)
package database

import "github.com/gofiber/storage/redis"

var RedisStore = redis.New()

Checklist:

[X] I agree to follow Fiber's Code of Conduct.
[X] I have checked for existing issues that describe my problem prior to opening this one.
[X] I understand that improperly formatted bug reports may be closed without explanation.

@alparslanahmed You are using the default store:
var RedisStore = redis.New()

Is your redis instance at localhost:6379 and doesnt use username, password, and default DB?

Also update your import, you are 2 major versions behind. The current module is here go get github.com/gofiber/storage/redis/v3

@alparslanahmed You are using the default store:
var RedisStore = redis.New()

Is your redis instance at localhost:6379 and doesnt use username, password, and default DB?

Yes it is

@alparslanahmed You are using the default store:
var RedisStore = redis.New()

Is your redis instance at localhost:6379 and doesnt use username, password, and default DB?

Yes it is

Can you upgrade to V3 and try again?

@alparslanahmed Were you able to figured out your issue?

I upgraded to v3 but issue persist :(
package database

import "github.com/gofiber/storage/redis/v3"

var RedisStore = redis.New()

@alparslanahmed Can you share your logs?

This is the error, which logs you need?
redis: client is closed

@alparslanahmed How are you running Redis?

I run redis on Ubuntu, same server with the application.

Redis running as systemd service

package main

import (
	"github.com/gofiber/fiber/v2"
	"github.com/gofiber/storage/redis/v3"
	"log"
)

func main() {
	store := redis.New()

	app := fiber.New()
	app.Get("/hello", func(c *fiber.Ctx) error {
		err := store.Set("hello", []byte("world"), 0)
		if err != nil {
			log.Fatalln(err)
		}

		sendText, err := store.Get("hello")
		if err != nil {
			log.Fatalln(err)
		}

		return c.Send(sendText)
	})
	log.Fatalln(app.Listen(":3000"))
}

We need a reproducible example, without it we cannot make or help a fix
please try to create a small example where the error is reproducible

@alparslanahmed Did you figured out your issue?

@ReneWerner87 this can be closed


There is no code published

I know this is obvious, but I'd like to have a place that I can subscribe to for changes.

Hello all,
I am very interested in implementing your solution for UAV lidar point cloud registration.
Any update on a release date for your code ? :)

Hi @chrisranderson @AurelienBrn,
Thanks for your interest in our work!
Sorry for my late response, since I am busy with the ICCV submission during this period.
We are going through the final inspection of the code, and it is expected to be released at the end of March :)
Best

I  am interested in this work. I wonder when the codes will be made public.

Hi @tangbohu @chrisranderson @AurelienBrn ，
Thanks for your patience, the code has finally released :)
Enjoy~


Extensive use of JdkDynamicProxies

Current implementation of Ratel both on client and server side uses many dynamic proxies, e.g.:
com.payu.ratel.client.ClientProxyDecorator.decorateWithMonitoring
com.payu.ratel.client.RatelClientProducer.decorateWithCaching
com.payu.ratel.client.RatelClientProducer.decorateWithRetryPolicy
com.payu.ratel.register.ServiceRegisterPostProcessor.decorateWithMonitoring
com.payu.ratel.proxy.monitoring.ServiceInvocationHandler
It also applies to hand made interceptors, e.g.
com.payu.ratel.client.RatelHessianProxyFactoryBean.invoke
This has negative impact on manageability and performance. Ideally, there should be single proxy on client side and single on server side and all decorators should be implemneted as interceptors injected to the two proxies.  Most probably org.springframework.aop.framework.ProxyFactory should be used for this purpose.
This issue is about changing the implementation so that the above condition is met

PR #59 resolve this issue


Instrumentation of Node.js application developed in typescript

We are planning to develop Node.js application in Typescript and we want to do the instrumentation using opentelemetry. Can you please help us with the documentation or sample project for the same.

There are several examples in the root of this repository as well as documentation at opentelemetry.io. If you have specific questions not covered by the documentation you can ask them but this question is too general to be actionable.

Maybe a Typescript getting started version might be useful
I have the same problem here


HBASE-28246 Expose region cached size over JMX metrics and report in the RS UI

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 50s
master passed

+1 :green_heart:
compile
2m 46s
master passed

+1 :green_heart:
checkstyle
0m 44s
master passed

+1 :green_heart:
spotless
0m 43s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 53s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 41s
the patch passed

+1 :green_heart:
compile
2m 44s
the patch passed

+1 :green_heart:
javac
2m 44s
the patch passed

+1 :green_heart:
checkstyle
0m 47s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
9m 23s
Patch does not cause any errors with Hadoop 3.2.4 3.3.6.

+1 :green_heart:
spotless
0m 41s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
2m  4s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 20s
The patch does not generate ASF License warnings.

34m 34s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux b081a5571b3b 5.4.0-166-generic #183-Ubuntu SMP Mon Oct 2 11:28:33 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 6e421e9d94

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
81 (vs. ulimit of 30000)

modules
C: hbase-hadoop-compat hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  8s
master passed

+1 :green_heart:
compile
0m 58s
master passed

+1 :green_heart:
shadedjars
5m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 48s
the patch passed

+1 :green_heart:
compile
0m 58s
the patch passed

+1 :green_heart:
javac
0m 58s
the patch passed

+1 :green_heart:
shadedjars
5m 21s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 34s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 38s
hbase-hadoop-compat in the patch passed.

-1 :x:
unit
31m  2s
hbase-server in the patch failed.

54m 26s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b467c76c0399 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 6e421e9d94

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/testReport/

Max. process+thread count
1815 (vs. ulimit of 30000)

modules
C: hbase-hadoop-compat hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 26s
master passed

+1 :green_heart:
compile
0m 46s
master passed

+1 :green_heart:
shadedjars
5m  2s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 32s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 15s
the patch passed

+1 :green_heart:
compile
0m 46s
the patch passed

+1 :green_heart:
javac
0m 46s
the patch passed

+1 :green_heart:
shadedjars
4m 59s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 29s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 33s
hbase-hadoop-compat in the patch passed.

+1 :green_heart:
unit
219m 46s
hbase-server in the patch passed.

242m 23s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 55dbe3ede641 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 6e421e9d94

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/2/testReport/

Max. process+thread count
4667 (vs. ulimit of 30000)

modules
C: hbase-hadoop-compat hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/2/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 54s
master passed

+1 :green_heart:
compile
0m 58s
master passed

+1 :green_heart:
shadedjars
5m 14s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 35s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 36s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
shadedjars
5m  8s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 34s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 36s
hbase-hadoop-compat in the patch passed.

+1 :green_heart:
unit
223m 59s
hbase-server in the patch passed.

248m 25s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c56b350fb341 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 6e421e9d94

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/3/testReport/

Max. process+thread count
4717 (vs. ulimit of 30000)

modules
C: hbase-hadoop-compat hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5565/3/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


How can we show suggestion by querying server ?

Dynamically querying the server and show suggestion.

please read the wiki page or have a search.
You can use the remoteFIlter callback to do that.
https://github.com/ichord/At.js/wiki/How-to-use-remoteFilter


[Feature Request] Add EC2 Instance Name Tag to CW Alarm.

I see adding it to the instance name tag as something very useful. Among my testing codes I was able to make this improvement, I must investigate again to incorporate it into my main Branch.

Solved in the commit https://github.com/aws-samples/amazon-cloudwatch-auto-alarms/commit/42e23d13e03e93b81c56cb1b095746b23d11ec74


RTL Metro UI css v3

Using RTLit library to add RTL support on the entire framework
metro-rtl.less is the main file to generate RTL output

http://forum.metroui.org.ua/viewtopic.php?id=69


The loss in log

The loss print in log is the mean in the log interval?

Yes, the printed loss is the mean in the log interval.


Add var.create_origin_bucket

This adjustment allows for a slight improvement in functionality -- it's
now possible to have this module create the origin bucket, but manually
specify the name to use.
This change works around a problem where it's not possible to use a
computed value for the origin bucket value. For example, if you create
an S3 bucket using the Cloudposse S3 bucket module, you'll see an error
like:

Error: Invalid count argument
on .terraform/modules/my_cdn/main.tf line 312, in data "aws_s3_bucket" "origin":
312:   count  = local.enabled && (var.origin_bucket != null) ? 1 : 0
The "count" value depends on resource attributes that cannot be determined
until apply, so Terraform cannot predict how many instances will be created.
To work around this, use the -target argument to first apply only the
resources that the count depends on.

Unfortunately this fix just moves the corner case, rather than
eliminating it. If a user specifies an existing bucket for
var.origin_bucket, they must set create_origin_bucket to false or a
spurious bucket will be created.

what
I ran into this problem while deploying a simple CDN as follows:
module "label" {
  source    = "git::https://github.com/cloudposse/terraform-null-label.git?ref=0.24.1"
  # ...
}
module "origin_bucket" {
  source  = "cloudposse/s3-bucket/aws"
  context = module.label.context
  # ...
}
module "cdn" {
  source  = "cloudposse/cloudfront-s3-cdn/aws"
  context = module.label.context
  origin_bucket = module.origin_bucket.bucket_id
  # ...
}

why

It's sad Terraform has no ability to completely squash this corner case. But it's good to eliminate the possibility of accidentally building non-reproducible Terraform configurations

just wondering if you set origin_bucket = module.label.id and set a depends_on, would this work without this pr change?
module "cdn" {
  source        = "cloudposse/cloudfront-s3-cdn/aws"
  context       = module.label.context
  origin_bucket = module.label.id
  # ...
  depends_on = [module.origin_bucket]
}

I also think this is very useful in case we want to override the origin bucket name while still allowing this module to create it.


[Community] Encourage blogs surrounding product thinking in regards to platforms

Contribution Description
There is a hypothesis that "platform as a product" may benefit from some baseline discussions around product thinking in general.
This will track one or more blog that we see our community start to put out around their experiences in this area.
Related Working Group (WG)
Platforms
Contribution type
Blog (Default)
Why TAG App Delivery?
We want to encourage more community discussion which requires lowering the barrier of entry for both understanding what we are working on (product thinking in my situation) and being a contributor (writing blogs vs full blown white papers in this situation)
Related projects/technologies
Platform-as-a-Product research in the WG
Affiliation disclosure
This is encouraging multiple blogs, each author will need to complete this accordingly.
Additional collaborators
David, Michael, Dominik, Atul, Budha
Additional information
No response

Thanks for opening this issue @abangser!
The first outline we worked on for a blog article is about "Why Talking to Users Matters" - Identifying users, talking to users, focusing on problems instead on solutions and why you should care about all that.

I'm interested in the topic, too, and I am happy to help!

I've written a few blogs on this topic, and I would love to help on this.

Glad to hear there is excitement here! I think the goal behind blogs is that they are lower barriers to entry for content and for authors. And that they don't require as many reviewers / soak time as something that is more formally published. Nor do they need to be fully unique as different ways of writing/perspectives can be welcome. With that in mind @Rotfuks @earth2marsh @bryanrossUK while you are welcome to collaborate, I would recommend also feeling free to write your own blog that you think brings value to the TAG readers!

Hey @abangser - will these be publish these on the CNCF blog or just via our own platforms?  If it's our own platforms, what's the best way to promote the CNCF WG in the blog post?  It would be good to be able to tie the post to the WG in some way, but I'm not sure what's required in terms of comms guidelines for the CNCF.
I've got outlines for two blog posts that I'll draft and share here this/next week.

Hi @bryanrossUK, great question and idea to be wider in our reach! So we only have full control over our own blog, so I think it is fair to say that posting there is our number one objective. That being said, raising the profile of this WG to the CNCF blog would be great and if we have some key points we want to share there we may be able to add them.

fyi, just posted a brief Linkedin article on this topic

Thanks for sharing @dgem!
Would you like to see that on the TAG website? I think it has a bit of a social media vibe to it, but some really small updates to intro/outro would make it a good fit if you'd like!

In trying to get alignment on current WG priorities, I am going to suggest we close this topic. But before doing so, I want to give 1 week for anyone who is interested to revive the focus on this, else we can reopen in the future when we have the right people/energies to commit to this topic (or create an associated blog post).
Thanks!

Per my last message, I am closing for now but am happy to reopen this conversation if it becomes higher priority / has a sponsor to drive it!


Push API

Building on #5
Check out https://developer.mozilla.org/en-US/docs/Web/API/Push_API and see if it applies to the CSG

[ ] Remove any code that was written to implement Slack

Push-notifications or Slack? The jury is still out on this one.
Slack could be easier to implement...

Push notifications require service workers or Celery or other external components

Rather than implementing Slack or a push API, a RESTful API would be much more handy, as it could be used to hook up external plug-ins, such as a Slack interface plug-in, or a push notification plug-in.

Hey,
Seems like you made a big progress these days. Gotta try it out soon!
For this topic: I've planned to write a Telegram bot once I've set it up. So I'd really appreciate a common API for accessing the data. But I can't really judge on this problem. REST usually is more simple to access but it could be more difficult from client side to witness edits or something like this. So it could lead to busy waiting. But I've never worked with Push API and I don't know how much integration it has among languages and how much effort it is to integrate.
So from my opinion REST would be sufficient but maybe not the best choice in all use cases. If you do this do whatever is more simple :+1:

That sounds great! When I picked up this project from where I left it, I thought of implementing a Slack bot which the users could interact with. The Slack bot would be called every now and then by a Cron job and would then query CleanSys over its API.
This way, neither Celery nor an external MQTT server such as RabbitMQ would be needed.
I'm not quite sure yet which functionalities and data to expose over the API. Here are some bullet points that came to my mind. Feel free to contribute/critique!

No administrative functionalities
All enabled Schedules
All ScheduleGroups
All Schedules belonging to a ScheduleGroup
All Cleaners currently affiliated with a ScheduleGroup
All current and coming Assignments of a Schedule
All current and coming Assignments of a Cleaner
All Tasks and their states of an Assignment
Set the state of a Task (done by whom)
Create a DutySwitch request
All open DutySwitch requests which a Cleaner is able to accept
Accept a DutySwitch request
Warning messages for the admin, such as "A Schedule's Assignments are running out"

A first REST API has been implemented using the django rest framework which makes API development surprisingly easy.
The API can be viewed in the browser under api/ and supports read and write requests (write requests requiring sign-in).
The REST API will be left at this state for now until a concrete need emerges for it.

I think the list of API features is quite complete from my view on the system. But the point of setting the state of a task seems for me quite necessary. Since there are no further restrictions by the applications itself for setting the state of the task as I can login as any user the API should reflect this behaviour and just accept it by giving it the name of the task and maybe the user assigned. Same for DutySwitch requests.
The last bullet point with warning messages for admin seems more like an e-mail thing to me rather than something I would need in an API.


Ability to animate turbo image like fast image.

I had a animation function with fast image which goes like below.
const AnimatedFastImage = Animated.createAnimatedComponent(Fast);

and it worked. Like i use it to scale up and down the image which gives a nice animated feel when users enter a screen.
const AnimatedTurboImage = Animated.createAnimatedComponent(TurboImage);

The animations do not work properly for turbo image but other features work flawlessly.
Let me know if you could add this feature. It will be a nice addition. If this is done please add it to the docs/ReadMe so people can use it as well.

@Gautham495
Can you try v1.12.2 ? let me know the result. Thanks.

Working perfectly. I tested in ios now. But should test in android.

Is there any other way i can dm you lol. I am going to replace fast image with your library completely.😇🗿. What about the linear gradient library? I want to use it for easing gradients and grayish gradient replacing blur views.

I don't use Twitter or X. Do you mind closing it since it works with react native reanimated?

Closing this as its working. Please add this to the docs to make this the ultimate library with animated support🐐


Update Intro Tutorial: Using a List View

Now that FlatList and SectionList are coming up in 0.43, we can recommend their use over ListView in Using a ListView.

I got problems to test the FlatList - " UnableToResolveError: Unable to resolve module react/lib/ReactPropTypeLocations "
I think is about the react module it self.  I'm using ""react": "^16.0.0-alpha.3"" that's recommended by the yarn installer.  And I'm using "react-native": "0.43.0-rc.2".
Just to be sure, I really need to upgrade the React module to use 0.43.0-rc.2? Thanks! Best regards.

I'm using FlatList on React Native 0.43.0-rc.2 and React ~16.0.0-alpha.3, but did not run into this issue.
	"dependencies": {
		"react": "~16.0.0-alpha.3",
		"react-native": "0.43.0-rc.2"
	},

Yep, I think few modules need old dependencies :/
Installing as "fresh" react-native init some app it's okay but when I insert old modules to my actual working progress app. It ask for old dependencies from react 15 or 14 not sure the version but it's older.
I think that I have to wait for stable dependencies...

I think we should close this issue, because now RN 0.43.0 is release.

The issue requires the tutorial to be updated, which has not yet happened. We can leave it open.

@hramos If you could let me know what needs to be updated in the tutorial, I can take a jab on it

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


chore(deps): update tubenhirn/semantic-release-azdo docker tag to v4.1.3

This PR contains the following updates:

Package
Update
Change

tubenhirn/semantic-release-azdo
patch
v4.1.2 -> v4.1.3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


Directly applying the vanilla Transformer checkpoint to the Informer fails

Hi,
I am wondering that if directly applying a well-trained vanilla Transformer checkpoint to the Informer (which I think is possible since there is no additional parameters in Informer), could Informer have relatively good performance during the inference? Based on our experiment results, directly applying vanilla Transformer's checkpoint leads to bad performances (almost like random guess).
I understand that informer is initially used in the training setting, but I still believe that informer can do direct evaluation since there is no additional parameters. Could you help me to solve this?
Thank you!

@cookieminions @zhouhaoyi Anyone can help me for this? Thanks!


Allow users to curate GTFS feeds, and stop bundling them by default

Fixes https://github.com/headwaymaps/headway/issues/32
Fixes https://github.com/headwaymaps/headway/issues/66
Fixes #17
Transit is no longer built by default to allow people to manually curate their GTFS feeds. Using all applicable GTFS feeds was causing problems (see #66) so I'm disabling transit in the default build flow, and allowing users who want to curate GTFS feeds to re-enable it. This is a much nicer way of setting up builds I think and makes more sense with the philosophy of Headway. Easy builds, and it should work every time. A round-trip was added to fetch capabilities.txt on every place card load but that can be optimized out easily later on.

Great, thank you so much, will try and let you know ASAP.


docs: Improving state type with interface

Documentation link: https://nuxt-typed-vuex.danielcroe.com/store/state.html
Current example:
export const state = () => ({
  // AVOID: This results in emails being typed as never[]
  emails: [],

  // This is correct
  emails: [] as string[],
})

But we can make it even prettier with an interface:
interface IState {
  (): {
    emails: string[]
  }
}

export const state: IState = () => ({})

Or there is some issues with interface? I just used interfaces myself everywhere, instead of type casting with as syntax.

I prefer the inline type definition in the docs at the moment, but your approach certainly works too 😊


Update an entity with a non persistent child.

Hi everyone,
I have an entity called Order and this has many Items, when I try to update Order, where my items is not persisted yet, I have an updated Order, but my added Item doesn't persist. The solution that I found is persist the Item before I try update the Order. Is there a most easy and correct to do way?

Let me guess, you have been working with hibernate 😄
You shouldn't think of objection as an ORM, but more like a query builder on steroids. I should probably remove all references to the word ORM from the documentation. You have to explicitly start every database query and nothing like this is done automatically (eager, insertGraph and other "bulk" methods are exceptions to this rule). The whole point of objection is that what you see is what you get; minimal amount of magic. You really need to know SQL (not saying that you don't) to use objection.
I'm working on a updateGraph method #211 that would do what you want, but that isn't ready yet unfortunately.
If you are looking for a true ORM that attempts to abstract away SQL, sequelize is probably the best option in the node world.

First, You guessed :)
I will persist the Item before update Order, provisionally, until your updateGraph is not ready, and study Sequelize too.
Thanks very much.


Review style of file paths, change them to os.path.join and pathlib.Path

Some file paths do not work between different file-systems i.e. Mac/Linux/Windows.
Please change paths to work on ALL systems by removing system specific paths and replacing them with the proper python calls to generate system-friendly paths.

initially triage and check the setup directory files and the examples directory files to make sure they work -- afterwards pull request when necessary if someone else messes up file paths

What is this? EDGAR/dataloader_neal.py
Can we delete it from your branch? @NC-Zhao

There have been path fixes available for over a week now that you haven't pulled so you're recoding stuff that other people have done already.

What is this? EDGAR/dataloader_neal.py
Can we delete it from your branch? @NC-Zhao

That was the dataloader with the path fixed only to my PC. It should be deleted.

I assume that sys.path.append('../') does the same as sys.path.append('..')?


want to Modify Assasin's  creed card

Issue Type:

[x] Bug Report
[ ] Feature Request
[ ] Documentaion

Describe the bug
some animation  is missing
Possible solution
I want to add some animation in these card.
Screenshots

Do add before and after images in PR description


MAINT: add optional packages for tensorflow and tensorflow-cpu; update documentation

Description
Fixes #261
Changes
poetry add --optional tensorflow==2.7.0
poetry add --optional tensorflow==2.7.0

Update documentation. I updated all pip install sites - but maybe only some need to be updated?

Thinking about this a bit more: do you see any advantage in making these optional dependencies scikeras[tensorflow] v.s. just editing the docs to read pip install scikeras tensorflow or pip install scikeras tensorflow-cpu @metasyn ? I'm open to either and I think both need the same explanation blurb in the docs, I'm just trying to think what might be less confusing for a user.

Codecov Report

Merging #262 (e115731) into master (57918d6) will decrease coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #262      +/-   ##
==========================================
- Coverage   98.27%   97.87%   -0.40%
==========================================
  Files           7        7
  Lines         752      752
==========================================
- Hits          739      736       -3
- Misses         13       16       +3

Impacted Files
Coverage Δ

scikeras/__init__.py
88.88% <0.00%> (-11.12%)
:arrow_down:

scikeras/_saving_utils.py
96.73% <0.00%> (-1.09%)
:arrow_down:

Continue to review full report at Codecov.

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

Thinking about this a bit more: do you see any advantage in making these optional dependencies scikeras[tensorflow] v.s. just editing the docs to read pip install scikeras tensorflow or pip install scikeras tensorflow-cpu @metasyn ? I'm open to either and I think both need the same explanation blurb in the docs, I'm just trying to think what might be less confusing for a user.

I guess adding it as an optional dependency makes it more obvious, in my opinion, in terms of what is going on. It reminds me of using coverage[toml] or dask[complete] - where its expected that w/o that, certain things wont work. It would be really nice if poetry somehow supported an "either or" kind of constraint or something.
I'm happy to change whatever you prefer though.

I think this change works. I agree with your assessment of it being more "obvious" even if it's just thin syntactic sugar.

It would be really nice if poetry somehow supported an "either or" kind of constraint or something.

It's not necessarily a poetry issue (in fact, poetry kinda support this usage when installing a local package via dependency groups). Really this is a pip / package ecosystem issue: https://github.com/pypa/setuptools/issues/1139

Thanks @metasyn !


Unable to play HLS streams

I am trying to playback an HLS stream using the emulator but I get the following error in the console:
[cast.player.api.Host] error: cast.player.api.ErrorCode.PLAYBACK/110 (Failed to execute 'addSourceBuffer' on 'MediaSource': The type provided ('video/mp2t; codecs="avc1.4D401F"') is unsupported.)

anybody familiar with this issue?
thanks

Seems to be an issue with Chrome not having all the codecs needed.
Using Chromium solves the issue: https://chromium.woolyss.com/


Print Function and Event selectors and signatures

Currently, this tool only uploads the function and event selectors to 4byte.directory. I propose we add some print functionality to this plugin such that the user can view the selectors and signatures for each contract in a table or list format.

I looked into this, and now remember that the plugin doesn't calculate selectors at all.  Rather, it creates a fake composite ABI by combining all of the ABIs in the artifacts/ directory, and uploads that to 4byte.directory.  The website handles the selector calculation.
So there's no technical reason to include this feature in this plugin.  No code can be shared.  However, I would like to add this functionality in some capacity - perhaps to the WIP diamond plugin?  Maybe even to ABI exporter?
Closing this issue, but open to discussion either here or in a separate chat.


Add Ability to Save Jupyter Notebooks

Currently the DockerfileJupyter copies the notebooks from the Launcher diretory to working directory inside the Docker container https://github.com/QuantConnect/Lean/blob/master/DockerfileJupyter#L37
The problem is that by default changes to these files won't persist on the disk after the container will be stopped. I tried using a volume to run
docker run -it -p 8866:8866 -v $(pwd)/Data:/home/Data:rw -v $(pwd)/Launcher/bin/Debug:/root/Lean/Launcher/bin/Debug:rw quantconnect/jupyter

and this solved the problem of persistance but then I wasn't able to load the Quantconnect packages inside the notebooks.
Is there a way we can get the best of both worlds, ie persistance of notebooks and loading of libraries? Currently I have to download the notebooks manually and rebuild the images everytime I want to stop the containers.

Please map the whole Lean tree:
docker run -it --rm -p 8888:8888 -v ~/Lean/:/root/Lean/ quantconnect/jupyter
and the notebooks will be saved at /Lean/Launcher/bin/Debug in the host.


Remove /api/v1/delete_series docs for 2.0

This endpoint has moved to /api/v2 (with somewhat different properties)
in Prometheus 2.0 and should now be part of a separate admin API page.

Hi Guys,
May I ask if this API support to delete series on certain period? like data with label job=xxx on last three hours. Since I just can't find where to set time period on this API of delete action by glimpsing patch, so I wonder if I overlook something.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Android doesn't show a blurred screen

Is it correct that Android does not show a blurred screen as is the case with iOS?
I have tried for example to run it on a Pixel 6, with Android 13 (API level 33), but only the screenshot is disabled, no blurred screen visible.

Yes it's correct. Because Android has native feature to disable screenshots, screen recording, casting but iOS doesn't. So, it's hidden with a blur view.


MsBuild property pointing to InstallDir\MsBuild

Hello,
I have a VSIX that when installed, will install some targets to the [InstallDir]\MsBuild directory:  https://docs.microsoft.com/en-us/visualstudio/extensibility/set-install-root
I then have a project template - I need the project to import the targets from there installed location (i.e the [InstallDir]\MsBuild directory).

  <Import Project="$(WHAT MSBUILD PROPERTY TO USE HERE)\MsBuild\DnnVsProjectSystem.props" />

Any ideas how I can solve this? Can I use vswhere?

I think MSBuildExtensionsPath was what I was looking for!


DateInput is not updating the value of the DatePickerDialog

Description
When using the DateInput or DatePickerInput with a DatePickerDialog, on change of the DateInput or DatePickerInput does not update the value in the DatePickerDialog. This can be recreate-able in the Saas-UI live demo. Consistently keeping the values in sync between the two components feels more user friendly.
Link to Reproduction
https://saas-ui.dev/docs/components/date-time/date-input
Steps to reproduce

Go to any live demo example
Edit the input date field to another month or year
Open the date picker dialog using the calendar icon
The date picker dialog will still show the current month and year

Saas UI Version
0.31.1
Chakra UI Version
2.2.0
Browser
Chrome 121.0.6167.184
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information

I stumbled upon a workaround. I don't know why, but when I extracted the DatePickerDialog into a private component, the DatePickerInput and DatePickerDialog stays in sync. Using the DatePickerDialog or wrapping it with one layer of Box doesn't work. See example below.
const MyComponent = () => {
    // private component
    const Dialog = () => (
      <DatePickerDialog>
        <DatePickerCalendar />
      </DatePickerDialog>
    )

    return (
      <DatePicker
        placement="bottom"
        granularity="day"
      >

          <DatePickerInput />

        {/* WORKS*/}
        <Dialog />

        {/* DOES NOT WORK
          <DatePickerDialog>
            <DatePickerCalendar />
          </DatePickerDialog>
        */}

         {/* ALSO DOES NOT WORK
         <Box>
          <DatePickerDialog>
            <DatePickerCalendar />
          </DatePickerDialog>
        </Box> */}
      </DatePicker>
    )
}

Interesting that this fixes it, thanks for the info. I'm going to investigate this now.

The value is actually in sync, but the calendar doesn't open on the correct month.

@tuantvu it seems like a limitation of react aria calendar. Found a fix, will release it later this week.

This was released in 0.12.23


CLI testing framework

This is a really simple test framework for cli.py that uses monkey patch to substitute out for the URL functions and just validates that requests get made to the proper URLs with the proper data being sent. If people think it's a good idea then it can be cleaned up and added.

Closing; see #367.


Fix memory leak when using Object(value:) for subclasses with List/RealmOptional properties

Calling [self init] from [RLMObjectBase initWithValue:schema:] ends up
calling a second non-convenience Swift initializer on the object, which means
that all of the properties get initialized a second time and the old values are
leaked.
Closes #2882.

:+1:


Spec version metadata

In section 2.5, the spec lists a series of metadata to help compatibility. Ought the spec version be part of this? Or is there some other way to inspect which version of the spec a particular instance of an OpenPBR shader is?
In addition, from reading it's not clear what the different metadata being mentioned are, or where they are applied? Would be great to specify all this explicitly.

We did add "The version of the specification implemented" to the suggested metadata.
The normal map conventions would be quite involved to properly specify (I think). So it would be for a later release, if we do it at all (it could be that this is outside of the scope of the spec permanently though).


add animated_media message renderer

I looked at my personal chat history for animated_media message types and found that all instances of these types contained an animated gif. I'm suggesting following changes based on the same assumption.
Please review!

thank you for the PR!


Ubuntu-latest Job fails after 1 h and doesn't display logs.

Description
I work on Kompact, an open-source Actor-framework in Rust. We recently migrated to Github Actions from Travis. We are trying to get a Code-coverage report produced as part of the CI pipeline, so this is not a job which was running and then suddenly started failing.
The Code-coverage job fails after 1 hour. No logs are saved for/viewable for the step which it fails on. You can view four similar failing jobs here and see what I'm talking about. We have tried to ensure that it's not failing due to faulty configuration/scripting/set-up on our end.
I found a similar issue (#1491) I suspect it is the same problem.
Area for Triage:  Rust

Question, Bug, or Feature?:  Bug

Virtual environments affected

[ ] Ubuntu 16.04
[✅] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
Expecting the job to succeed, or, fail and display logs.
Actual behavior
Failure and no log for the failing step.
Repro steps
The Job config is viewable here.
The tests ran during that job run for a long time. The testing includes networked integration tests, in which multiple sockets are opened and closed, for many of the test cases, and they are run multiple times against a feature-matrix. So the issue may be related to resource usage, as the related issue alludes to, but it is difficult to troubleshoot for us without access to the logs.

Hi @adamhass!
I've checked the provided failing jobs and 2 of them actually contain some data"

Use source-based code coverage Codecov #2 — a lot of ERRO slog-async: logger dropped messages due to channel overflow messages
https://pipelines.actions.githubusercontent.com/HIOnzKPM13Mp2mueOWpEoPDQaglt6dMP2qFrpGOn8g2OyHGIsi/_apis/pipelines/1/runs/17/signedlogcontent/3?urlExpires=2021-01-19T08%3A17%3A51.5954744Z&urlSigningMethod=HMACV1&urlSignature=yl4HUik8kFqjr6yYlr2bqUz12LO3NqMn5UNWByEJzg8%3D
Use source-based code coverage Codecov #3 — pretty straightforward error No space left on device
Could you please check how much space is usually used for the tests? This can be a reason for such behavior.

In the meantime, I'll try to find what's wrong with those 2 without logs.

Hi @adamhass!
I've checked the provided failing jobs and 2 of them actually contain some data"

Use source-based code coverage Codecov #2 — a lot of ERRO slog-async: logger dropped messages due to channel overflow messages
https://pipelines.actions.githubusercontent.com/HIOnzKPM13Mp2mueOWpEoPDQaglt6dMP2qFrpGOn8g2OyHGIsi/_apis/pipelines/1/runs/17/signedlogcontent/3?urlExpires=2021-01-19T08%3A17%3A51.5954744Z&urlSigningMethod=HMACV1&urlSignature=yl4HUik8kFqjr6yYlr2bqUz12LO3NqMn5UNWByEJzg8%3D
Use source-based code coverage Codecov #3 — pretty straightforward error No space left on device
Could you please check how much space is usually used for the tests? This can be a reason for such behavior.

In the meantime, I'll try to find what's wrong with those 2 without logs.

Managed to find backend service logs for runs without any data — the lack of logs is related to the disk space usage: Failed to upload diagnostics System.IO.IOException: No space left on device

Managed to find backend service logs for runs without any data — the lack of logs is related to the disk space usage: Failed to upload diagnostics System.IO.IOException: No space left on device

Thanks for the quick help. We can solve our problem with that information.

Thanks for the quick help. We can solve our problem with that information.


Allow to create OpenAI client without specifying apiKey or to be defined as an environment variable.

Confirm this is a feature request for the Node library and not the underlying OpenAI API.

[X] This is a feature request for the Node library

Describe the feature or improvement you're requesting
It would be useful to allow the creation of OpenAI client without specifying an apiKey (or OPENAI_API_KEY  as an environment variable).
The use case for that is:

Have a server with API compatible with OpenAI spec, instead of writing a custom wrapper. (This can save a lot of time and efforts)
Have the APIKey managed on the server
Use the OpenAI official client in the browser directly and change the baseURL for my custom server
If the api key is provided from the client, use the one passed in request. Otherwise, use the default one on the server.

As of today
Using this approach as of today, require to:

set a mock "apiKey" on the client
Set the flag dangerouslyAllowBrowser: true on the client
ignore the passed api key on the server side

and does not support the use case If the api key is provided from the client, use the one passed in request. Otherwise, use the default one on the server.. Would need to pass some header or another workaround to tell the server to ignore the key (which is not clean)
new OpenAI({
    baseURL,
    apiKey: 'my-mock-key',
    dangerouslyAllowBrowser: true,
  })

With the proposed change
new OpenAI({
    baseURL,
    dangerouslyAllowBrowser : true,
  })

Additional context
No response

Thanks for the suggestion; we prefer to keep this required, since most users hitting the API do require it.
You can pass apiKey: null or apiKey: 'not-needed-because-xyx' to explicitly not pass an API Kye.


How to feather or blur edges?

Hi there,
I'm looking for a way to feather or blur edges of a graphics object/sprite. What would be the best way to achieve this?
For example I would like to be able to achieve something similar to this:

Is there any way to do this while still having OK performance? I have tried various methods using BlurFilter (with blur value of 40 and 5 passes) and Masks, but it's simply too demanding to get the desired effect and hurt performance too much on tablets and mobile phones.
I would try out anything possible
I'm currently running v.3.0.8 and upgrading is not an option at this time.
Best regards

It lets me remember an old issue :  https://github.com/pixijs/pixi.js/issues/3139

Yes I got into that issue while searching but the difference is I'm not looking to necessarily mask my sprite in before doing the blur/feather. I'd like to do it on a graphic or unmasked sprite.
Did you solve it somehow?

I ahve an idea: take a Graphics, add BlurFilter, then SpriteMaskFilter on it, pass sprite image to SpriteMaskFilter constructor.

Though I recommend to cache blurred graphics into a texture:
var rt = PIXI.RenderTexture.create(W1, H1);
var spr = new PIXI.Sprite(PIXI.Texture.WHITE);
spr.anchor.set(0.5);
spr.width = W2;
spr.height = H2;
spr.filters = [blurFilter];
var tex = renderer.generateTexture(spr);

//use tex as a texture for mask sprite or masked sprite

What's mean of then SpriteMaskFilter on it , @ivanpopelyshev ?

@finscn If an element has mask and filters, mask is applied "inside" filters. You can manually add mask to filters array in the place you want it, just use SpriteMaskFilter for it.

any help please , am tring to do same thing here but filter not work ?
    var blur = new PIXI.filters.BlurFilter(10);
    var texture = new PIXI.Sprite(PIXI.Texture.WHITE);
    texture.width = 500, texture.height = 40;
    texture._filters = [blur]; // add blur to texture

    this.addChild(texture);
    this.mask = texture; // add texture as mask to the parent

#4451

closing as the original question was given an answer
@djmisterjon if you're still having problems, please open a new issue :)


uib-dropdown is lost focus when selecting an entry

When opening a dropdown, and selecting an entry from uib-dropown-menu, the focus is lost from the dropdown button.
My suggestion is to add the following row when the menu is closed:
line 2998:
toggleEl = $element.find('.dropdwon-toggle');
if(toggleEl.length){
toggleEl[0].focus();
}
Thanks.

duplicate


main.py: error: argument action: invalid choice: '--tasks semantic scene instance orientation' (choose from 'resume')

I'm trying to run the evaluation on NYUv2 (without pretraining on Hypersim), but I keep receiving this error, I had the problem discussed in https://github.com/TUI-NICR/EMSANet/issues/16
So I commented the line, run it again and receivedthe error:
main.py: error: argument action: invalid choice: '--tasks semantic scene instance orientation' (choose from 'resume')
It looks like the "--tasks" which is assigned as "semantic scene instance orientation", but it should be resume, correct?

It appears that the command itself may be broken, and the issue might not be related to the "--tasks" argument. Could you please provide the full command you're trying to execute so that we can understand the problem?

The exact same command that is suggested on the readme file. The fist command.
python main.py
--dataset nyuv2
--dataset-path ./datasets/nyuv2
--tasks semantic scene instance orientation
--enable-panoptic
--input-modalities rgb depth
--rgb-encoder-backbone resnet34
--rgb-encoder-backbone-block nonbottleneck1d
--depth-encoder-backbone resnet34
--depth-encoder-backbone-block nonbottleneck1d
--no-pretrained-backbone
--weights-filepath ./trained_models/nyuv2/r34_NBt1D.pth
--checkpointing-metrics valid_semantic_miou bacc mae_gt_deg panoptic_deeplab_semantic_miou panoptic_all_deeplab_pq
--validation-batch-size 16
--validation-only
--skip-sanity-check
--wandb-mode disabled


Removes the rm that caused inital build failures

On an initial clone of the repo, the rm would fail (even though it has a -r).  Removes this line as someone could run npm run clean if they wanted to do a clean build.

Coverage remained the same at 39.149% when pulling 524d7d9b2886ad3af49cd2f3aff8477b2af504c4 on initialFailures into 89bc44b3a17254527f36d579e24cfcb519c86824 on master.


Improve reporter output when watch mode runs test files without exclusive tests

.only is sticky in watch mode. If the watcher detects it in one file, and then needs to run tests from another, it only runs exclusive tests from that second file. If the second file does not contain any exclusive tests no tests are run. With the mini reporter the output in this case is simply "0 tests passed". This may be confusing to users.
Instead the reporters (mini, verbose and tap) should detect this scenario and output something like "no exclusive tests found".

I kinda think if there are .only tests we should abandon the whole smart rerun thing and just run all files with a .only whenever any change is detected (regardless of dependency graph). It is understood that .only is a temporary way of focusing on just a few tests. If a user has specified .only on two tests in separate files at the same time, we should trust that they know what they want and always rerun both.

I think we'd have to rerun all tests, in case you add a .only in another file. Of course if you then remove the .only it would run the full suite. May not be too much of a problem though.

I think we'd have to rerun all tests, in case you add a .only in another file

Unchanged files that did not have .only before will not need to be run again.

Unchanged files that did not have .only before will not need to be run again.

Oh good point!
We could simplify the implementation then:

Still track which files contain exclusive tests
When running specific test files, also run the files containing exclusive tests

We'll no longer need to pass options to Api#run().


refactor: cdp migration - processing pending tasks in bgQ

closes: https://github.com/customerio/issues/issues/11649
What does this PR do?
This pull request introduces a method that analyzes the Journey's SDK background queue to determine if any tasks from the Journeys SDK remain unexecuted. Should such tasks exist, it then fetches each individual task and executes them one by one. This ensures that all pending tasks are executed prior to transitioning to the CDP and all the pending tasks are sent to the CDP. Once a task is successfully executed, it is removed from the background queue.
This PR currently handles :

fetching pending tasks
executing them
sending data to workspace via (migrated) cdp module
added TODO note to the methods that are unavailable currently in segment and have to be added in our CDP SDK
Some edge cases like sending data of non-current identifier
Code cleanup
Tested with large data (> 300)
Automated test with large data (3000takss)

Pending:

NA

Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[ ] Assign members of your team to review the pull request.
[ ] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.
[ ] Wait until the pull request has been reviewed and approved by a teammate
[ ] After code reviews are approved and you determine this PR is ready to merge, select Squash and Merge button on this screen, leave the title and description to the default values, then merge the PR.

Could you share some more information about this PR? What work has been done and what has not been done?
Without this context, I am unsure what to review.
Quickly reviewing it now, tests need to be added, implementation is incomplete, and there are performance issues in this implementation.
I understand if more PRs will be made to address these concerns but with the information I have right now, I am unsure if that's true.

@levibostian Thanks for your feedback. I have updated the description with some detail. I hope it helps. Also to address the issue you highlighted with profileAttributes, I have considered that and updated the code. Let me know if you have any questions.

Looks good, some suggestions to increase confidence in migration tasks.

I would still suggest to do a peer and group testing of this feature since this is the major one. I am confident in it's working as far as I have tested but I do agree that dev testing is not enough at times specially when the feature is an important one like this ! We can do a test run once the team is back next week of all the modules that have been done. It can be a kind of UAT !


Custom resource as element in exported array in another resource - not saving

Godot version
4.0 beta 2
System information
mac OS 12.6
Issue description
Hello,
I'm attempting to set an in-built custom resource (MyResource below) via the editor inside an array of another parent resource MyArrayResource.

When I add one in and make changes to it (changing the default My Int from 42 to 43), it gets added as a Resource type - (perhaps I would be expecting a MyResource type instead?)

This change doesn't get saved though - when I refresh the view by navigating away from the resource then navigating back, it's now empty

I've attempted this as a simple field of type MyResource instead of an array and it does persist the change successfully.
Last thing - when I try to save an implementation of MyResource to a file and load that in (instead of it being in-built) I'm greeted by this message: The selected resource (Resource) does not match any type expected for this property (MyResource).

Just realised - I am able to get this semi-working if I export an Array[Resource] instead which is a decent workaround.
Steps to reproduce
See above, attached a sample project as well
Minimal reproduction project
Resource Array Bug Report.zip

The bug still persist on 4.0 beta 4, however I also get it with a node's exported array and I get a different error than yours:
Failed to set a resource of the type 'Resource' because this EditorResourcePicker only accepts 'MyResource' and its derivatives.

The bug also no longer happens after reloading the editor.

Just stumbled over this issue in beta 12. I can assign MyResources via the editor to the array, however, when running the project, the array will be empty when accessed in code. The array is shown correctly when I check the remote tree via the editor. I tried the mentioned workaround to export Array[Resource] rather than Array[MyResource] but with no effect.
Has someone figured out another workaround since this is progress breaking for my current project unfortunately.

I can't reproduce in current master. There has been many fixes for typed arrays, so this one might have been fixed along the way.

I believe i have this issue.
This is weird, cause i save my resources as .tres files, my ressource is like so:
CustomResource
¦-CustomResource
¦--Array[CustomResource]
This is weird because i save my resources with a @tool and the array of customresource is saved, i can even load it with my @tool script and it get loaded just fine, but at runtime it doesn't get loaded.
I'll investigate more to see if it's not a mistake on my part but for no it doesn't seemm so. Also i'm using the same function to load the resources in editor and at runtime.

Some test:
If i nest my resource like so:
CustomResource
¦-CustomResource
¦-Array[CustomResource]
(So instead my array of custom resources inside another custom resources is directly a child of the root custom resources) it does work.
I still don't understand why my first method works in editor but not at runtime.
I'll try with dictionary now

CustomResource
¦-CustomResource
¦--Dictionary{String:CustomResource}
Doesn't work either. Look like some limitation on the "recursiveness" of the resource loading? but only at runtime? Everything is fine in the .tres files tho.


Implement runtime fields

Codecov Report

Merging #21 (41fb093) into master (c84a806) will decrease coverage by 21.40%.
The diff coverage is 82.45%.

@@             Coverage Diff             @@
##           master      #21       +/-   ##
===========================================
- Coverage   61.14%   39.74%   -21.41%
===========================================
  Files          33       33
  Lines        3462     3510       +48
  Branches      484      491        +7
===========================================
- Hits         2117     1395      -722
- Misses       1166     2002      +836
+ Partials      179      113       -66

Impacted Files
Coverage Δ

...in/kotlin/dev/evo/elasticmagic/compile/Compiler.kt
44.82% <0.00%> (-37.94%)
:arrow_down:

...in/dev/evo/elasticmagic/compile/MappingCompiler.kt
78.46% <72.00%> (-21.54%)
:arrow_down:

...commonMain/kotlin/dev/evo/elasticmagic/Document.kt
86.45% <94.44%> (-0.81%)
:arrow_down:

...monMain/kotlin/dev/evo/elasticmagic/Expressions.kt
68.73% <100.00%> (-3.25%)
:arrow_down:

...ev/evo/elasticmagic/compile/SearchQueryCompiler.kt
60.54% <100.00%> (-33.83%)
:arrow_down:

.../evo/elasticmagic/serde/serialization/JsonSerde.kt
0.00% <0.00%> (-100.00%)
:arrow_down:

...otlin/dev/evo/elasticmagic/compile/BulkCompiler.kt
0.00% <0.00%> (-93.66%)
:arrow_down:

.../evo/elasticmagic/compile/UpdateMappingCompiler.kt
0.00% <0.00%> (-93.11%)
:arrow_down:

...otlin/dev/evo/elasticmagic/ElasticsearchCluster.kt
4.92% <0.00%> (-85.22%)
:arrow_down:

...asticmagic/transport/ElasticsearchKtorTransport.kt
0.00% <0.00%> (-85.00%)
:arrow_down:

... and 20 more

Continue to review full report at Codecov.

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


Use clean ruby hash syntax

Ruby 1.8 has not been supported for years

:metal:
I'm actually surprised there wasn't more use of the old syntax.


[Addition]: Cluster of 5 Self Witnessing Hotspots

Hotspot b58 Addresses
11DLRqXHJzzeSggxBU2H7hbDSPHqkGYCZJ6Sgm2qJMXaeoGZXcP
112dVjPtN3QKMmrfEbvoHHeh6yzGjAuocuwAedyyHC94MHWaph2k
112mXuCbzJXh6pUK66EiRdwJAFkEXXDb5ZFcJAQi1exxL51bG9Cy
11TRcBQCzjex8Q4HgMrTLpe26AZC2acJXTfkA1Fuc85E8RkX8m8
11a3jakG9RZn4gc8Z6KiaLDb2UgyMYxsDubs8GdVc57Jc91t8ZA
Discord Handle
Czar Osir#9967
Reason(s)
All this 5 hotsposts are not only talking only to each other, but they also each one have like 13-16 beacons everyday.
For example Ambitious Lava Dragon and Perfect Mint Hedgehog
this 2 are supposed to be at 25m with 8 DBi antena what definitley should be able to see others spots.

Also From RSSI/SNR we can directly see that this 5 hotspots are in some way manipulated

same as #6799 , already link to pr


Added pull request template

Fixes #851.
Playground at: https://github.com/bart-vmware/Spring-Config-Demo/compare/dummy-change?expand=1.

/cc @ccheetham

/azp run Steeltoe.All


npm i Doesn't Work

Using npm i -g hey-you doesn't work.
Error: ```/usr/local/bin/hey-you-cli -> /usr/local/lib/node_modules/hey-you/cli.js
/usr/local/bin/hey-you -> /usr/local/lib/node_modules/hey-you/index.js
/usr/local/bin/heyyou -> /usr/local/lib/node_modules/hey-you/index.js
/usr/local/bin/heyyou-cli -> /usr/local/lib/node_modules/hey-you/cli.js
/usr/local/bin/hey -> /usr/local/lib/node_modules/hey-you/index.js
/usr/local/bin/hey-cli -> /usr/local/lib/node_modules/hey-you/cli.js

<EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/hey-you/node_modules/electron
node install.js

/usr/local/lib/node_modules/hey-you/node_modules/electron/install.js:47
throw err
^
Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/hey-you/node_modules/electron/.electron'
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node install.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.```
Log: 2017-06-27T22_57_13_338Z-debug.txt

At first glance this looks like an issue with permissions. Do you own /usr/local/lib? (Do you normally have to install global modules with sudo?) Also, did Electron make a dir at ~/.electron with the right permissions (should look like drwxr-xr-x)? If not, mkdir ~/.electron && chmod 755 ~/.electron and let me know if that works. I've seen some possibly related bugs here, here, and here.

I do normally have to use sudo to install things globally. I did that the first time as well as after I did mkdir ~/.electron && chmod 755 ~/.electron
Running sudo npm install -g electron-prebuilt --unsafe-perm=true --allow-root from the last issue you linked worked.

Thanks! I added a note in the README about this.


404 error when deploying Nextjs App

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d2ucxgoxb4z8kg
AWS Region
eu-west-3
Amplify Hosting feature
Deployments
Question
I'm trying to deploy a Nextjs (v12) to Amplify.
All the checkmarks are green, but when I visit the URL (provided by AWS), it 404.
My build settings :
version: 1
frontend:
  phases:
    preBuild:
        commands:
            - npm ci
    build:
      commands:
        - npm run buildamp
  artifacts:
    baseDirectory: .next
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

The command npm run buildamp is simply next build
The build directory is indeed .next. I have no /src directory.
The App is quite big and old, with both TS and JS files. There are some dubious settings in tsconfig.json, jsconfig.json and next.config files.
What should I look into to find the source of the error ?
Thanks !

Hi @krirkrirk 👋🏽 , it looks like your app's platform is set to WEB and you're using Next.js 11 or older. The WEB platform can only be used with Next.js if your app is leveraging Static Site Generation (SSG). Can you confirm if your app is using SSG?
If your app isn't using SSG, then you'll need to change the platform for your application to WEB_DYNAMIC. You can do this by running the following command using the AWS CLI:
aws amplify update-app --app-id <APP_ID> --platform WEB_DYNAMIC --region <REGION>
You'll also need to update the framework for each branch to Next.js - SSR:
aws amplify update-branch --app-id <APP_ID> --branch-name <BRANCH_NAME> --framework 'Next.js - SSR' --region <REGION>

I'm trying to deploy a Nextjs (v12) to Amplify.

Whoops, I missed this first detail.
You can still follow the guidance above but since this is a Next.js 12 app, use WEB_COMPUTE instead. So your command would look like the following:
aws amplify update-app --app-id <APP_ID> --platform WEB_COMPUTE --region <REGION>

Setting the framework to 'Next.js - SSR' and setting the app platform to WEB_COMPUTE did the trick ! Thanks a lot @hloriii


Update android gradle config

fixes #247

@pacamara please review

Applied this fix and didn't work. Any hints?
I'm using gradle 4.4 and 3.1 plugin


Zend DevZone URL not being processed

Not sure why, but the Url plugin doesn't appear to respond to this URL:
http://devzone.zend.com/6178/20yearsofphp-a-timeline-of-tweets/
Relevant portions of the output from running it through curl:
09:20:42 ~ $ curl -v http://devzone.zend.com/6178/20yearsofphp-a-timeline-of-tweets/
* Adding handle: conn: 0x7fa891803c00
* Adding handle: send: 0
* Adding handle: recv: 0
* Curl_addHandleToPipeline: length: 1
* - Conn 0 (0x7fa891803c00) send_pipe: 1, recv_pipe: 0
* About to connect() to devzone.zend.com port 80 (#0)
*   Trying <IP_ADDRESS>...
* Connected to devzone.zend.com (<IP_ADDRESS>) port 80 (#0)
> GET /6178/20yearsofphp-a-timeline-of-tweets/ HTTP/1.1
> User-Agent: curl/7.30.0
> Host: devzone.zend.com
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Wed, 10 Jun 2015 14:20:45 GMT
* Server Apache/2.4.12 (Ubuntu) is not blacklisted
< Server: Apache/2.4.12 (Ubuntu)
< X-Powered-By: PHP/5.6.5 ZendServer/8.0.2
< X-Pingback: http://devzone.zend.com/xmlrpc.php
< Link: <http://devzone.zend.com/?p=6178>; rel=shortlink
< Vary: Accept-Encoding
< Transfer-Encoding: chunked
< Content-Type: text/html; charset=UTF-8
<
...snip...
<title>#20YearsofPHP - A Timeline in Tweets - Zend Developer Zone</title>
...snip...

Might be the ; charset=UTF-8 portion of the Content-Type header throwing off this check?

This appears to be due to an unrelated issue in the underlying IRC parser:
2015-06-10 09:14:37 NOTICE<EMAIL_ADDRESS>Parser unable to parse line<EMAIL_ADDRESS>PRIVMSG #nashvillephp :http://devzone.zend.com/6178/20yearsofphp-a-timeline-of-tweets/ []

Closing this issue out, as the Url plugin doesn't appear responsible for the problem. Sorry for the noise.

Thanks for letting me know :+1:


feature request: offline capable

Any plan to make databound offline capable with Service Worker and indexedDB?
Maybe it is a better idea to create another repo for offline capability.

Hello @wizztjh
Thanks for raising this :heart: :heart: :heart:
I never needed this myself, but if we can collaborate on the solution to this.
Could you setup an app and use your fork of databound so we could start working on this?

I would agree with @Nedomas to start this idea on the new project integrated with databound, not on this one.


No support for optimistic locks

Don't see how to achieve "select for update" using this orm. Doesn't it support implementation of  locking rows/tableS?

Hi @ajays20078
Could you let me know your use case, then maybe I could give some suggestions for how to do that.

Hi @jinzhu
Use case:
type Resource struct {
ID int,
status string
} where status is either available or allotted.
Now  if there are 2 parallel requests and if i have to select 5 resources and update them as allotted "select * from resource where status="available" limit 5"  . Select statement will return same 5 ids/resources to both requests as they are parallel requests and you are not doing "select for update"

Are you looking for:
db.Where("state = ?", "available").Limit(5).Update(Resource{Status: "allotted"})

select * from resource;
+----+-----------+
| id | status    |
+----+-----------+
|  1 | available |
+----+-----------+
1 row in set (0.00 sec)
var resource Resource
db.Where("status = ?", "avaliable").Limit(1).Update(Resource{status:
"allotted"}).Find(&resource)
panics:
panic: reflect: call of reflect.Value.MethodByName on zero Value
goroutine 16 [running]:
runtime.panic(0x6715a0, 0xc2080475c0)
/usr/share/go/src/pkg/runtime/panic.c:279 +0xf5
reflect.Value.MethodByName(0x0, 0x0, 0x0, 0x0, 0x6d8390, 0xb, 0x0, 0x0,
0x0, 0x0)
/usr/share/go/src/pkg/reflect/value.go:1306 +0xb1
github.com/jinzhu/gorm.func·005(0x0
http://github.com/jinzhu/gorm.func·005(0x0, 0x0)
/usr/share/go/bin/src/github.com/jinzhu/gorm/scope.go:174 +0xae
github.com/jinzhu/gorm.(*Scope).CallMethod(0xc208048200, 0x6d8390, 0xb, 0x1)
/usr/share/go/bin/src/github.com/jinzhu/gorm/scope.go:199 +0x2a3
github.com/jinzhu/gorm.(*Scope).CallMethodWithErrorCheck(0xc208048200,
0x6d8390, 0xb)
/usr/share/go/bin/src/github.com/jinzhu/gorm/scope.go:204 +0x40
github.com/jinzhu/gorm.AfterUpdate(0xc208048200)
/usr/share/go/bin/src/github.com/jinzhu/gorm/callback_update.go:70 +0x68
github.com/jinzhu/gorm.(*Scope).callCallbacks(0xc208048200, 0xc208054880,
0x9, 0x10, 0xc2080462c0)
/usr/share/go/bin/src/github.com/jinzhu/gorm/scope_private.go:297 +0x6b
github.com/jinzhu/gorm.(*DB).Updates(0xc208040b00, 0x689500, 0xc2080462c0,
0x7f90d7408d87, 0x1, 0x1, 0x7f90d7408f30)
/usr/share/go/bin/src/github.com/jinzhu/gorm/main.go:250 +0x139
github.com/jinzhu/gorm.(*DB).Update(0xc208040b00, 0x7f90d7408f08, 0x1, 0x1,
0x1)
/usr/share/go/bin/src/github.com/jinzhu/gorm/main.go:243 +0xb0
main.main()
/home/ajays/db/orm.go:13 +0x3a2
goroutine 19 [finalizer wait]:
runtime.park(0x413780, 0x8486a8, 0x847189)
/usr/share/go/src/pkg/runtime/proc.c:1369 +0x89
runtime.parkunlock(0x8486a8, 0x847189)
/usr/share/go/src/pkg/runtime/proc.c:1385 +0x3b
runfinq()
/usr/share/go/src/pkg/runtime/mgc0.c:2644 +0xcf
runtime.goexit()
/usr/share/go/src/pkg/runtime/proc.c:1445
goroutine 20 [chan receive]:
database/sql.(*DB).connectionOpener(0xc208048180)
/usr/share/go/src/pkg/database/sql/sql.go:583 +0x48
created by database/sql.Open
/usr/share/go/src/pkg/database/sql/sql.go:442 +0x27c
goroutine 21 [runnable]:
github.com/jinzhu/gorm.(*DB).print(0xc208040d10, 0xc2080475a0, 0x2, 0x2)
/usr/share/go/bin/src/github.com/jinzhu/gorm/main_private.go:36
created by github.com/jinzhu/gorm.(*DB).err
/usr/share/go/bin/src/github.com/jinzhu/gorm/main_private.go:26 +0x1a3
On Wed, Mar 4, 2015 at 12:32 PM, Jinzhu<EMAIL_ADDRESS>wrote:

Closed #400 https://github.com/jinzhu/gorm/issues/400.
—
Reply to this email directly or view it on GitHub
https://github.com/jinzhu/gorm/issues/400#event-245788690.

--
--
Cheers!
Ajay.S

@jinzhu
How do i get the IDs in the above query, i need the IDs whose status have been set to allocated.

@jinzhu
The command you mentioned translates to -
UPDATE resources SET status = 'allotted'  WHERE (status = 'available') LIMIT 1
Here you are not recording what IDs got affected, for that you need to do a select.
So you are essentially skipping "select on update" and thus loosing out on IDs that got affected, which is the data which needs to be returned(which resource got allotted)

You can't get ids with above query, basically you want to have a lock when update resource's state, yes?
You could do that with sync package or use raw sql https://github.com/jinzhu/gorm#raw-sql to do the work

Whats the point of using a ORM if i have to execute a RAW sql?
And sync will be app layer lock, not a DB layer, so your rows/table can still be updated.
This orm will be useless for transactions if you do not support locking.

Because even use raw sql with gorm to do that is still pretty easy
var ids []int
db.Row("select id from resources for update limit 5").Scan(&ids)
db.Where("Id in (?)", ids).Update(&Resource{State: "allotted"})

I don't think ORM could handle 100% cases, just use it to do those things that it good at.
Your case is not a common thing, and I can't imagine a good solution to improve gorm to make it fit your case well. (If you have ideas please suggest it)

@jinzhu
Your way of updating resource state does NOT work for concurrent request.
Consider the following sequence of execution
Request 1: var ids []int
Request 2: var ids []int
Request 1: db.Row("select id from resources for update limit 5").Scan(&ids) <- gets 5 resources
Request 2: db.Row("select id from resources for update limit 5").Scan(&ids) <- gets same 5 resource
Request1:db.Where("Id in (?)", ids).Update(&Resource{State: "allotted"}) <- updates 5 rows
Request2:db.Where("Id in (?)", ids).Update(&Resource{State: "allotted"}) <- updates same 5 rows.
You end up updating same 5 rows.
One way of implementing locks in ORM is,you can check before update, if the ID which you have read in memory is same as the ID on Disk.

'select for update' need to be run in a transaction, and it will prevent the select in second session until you commit current transaction, so you need to run above code in a transaction like this:
tx := db.Begin()
var ids []int
tx.Row("select id from resources limit 5 for update").Scan(&ids)
tx.Where("Id in (?)", ids).Update(&Resource{State: "allotted"})
tx.Commit()

This is exactly what i was trying to explain. We need the same behavior through ORM abstraction and not through raw query "select for update".

I think the above should be at least mentioned in documentation, as it is a very common use-case nowadays (who doesn't want concurrency, or does not implement it without considering race conditions?)
@ajays20078 shall we make a fork/pull request for this? I think it's very important to support it in the ORM (or stop using ORMs altogether)

its almost 2 year now.
Do we have the locking function now? because, concurrent is one of golang most wanted feature. let assume  we have lastbalance that we want to update concurrently, without locking we wont have correct last balance in the end of request.
example:
update account set last_balance = last_balance - nominal where id=?
when this query executed by 2 concurrent request, they both would get last_balance before update with same value
before update:
last_balance = 10.000
request 1 nominal = 1.000
request 2 nominal = 2.000
expected last_balance = 7.000
actual last_balance without locking = 8.000 or 9.000 depend on which request saved first, because:
10.000-1.000
10.000-2.000
If there are any idea to make this happen please share.
the only solution now i used is handling the balance deduction with database trigger
Thank You

@jinzhu,  can you reopen this issue?
I think it should be like this https://github.com/go-gorp/gorp#optimistic-locking

Guys, hey guys.


add support for muse.ai

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and am willing to share it if required

Region
global
Example URLs
https://muse.ai/embed/YdTWvUW
Provide a description that is worded well enough to be understood
Above link returns "Unsupported URL", but digging through the source (and network traffic with javascript enabled) returns the following .mpd file (which yt-dlp can download without any difficulty):
https://cdn-eu.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/videos/dash.mpd
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--restrict-filenames', '-o', '%(title)s-%(id)s-%(uploader)s.%(ext)s', '-w', '-v', 'https://muse.ai/embed/YdTWvUW']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481]
[debug] Python 3.11.4 (CPython x86_64 64bit) - Linux-6.1.28-with-glibc2.37 (OpenSSL 3.0.9 30 May 2023, glibc 2.37)
[debug] exe versions: ffmpeg 4.4.4 (setts), ffprobe 4.4.4
[debug] Optional libraries: certifi-3021.03.16, pycrypto-3.18.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[generic] Extracting URL: https://muse.ai/embed/YdTWvUW
[generic] YdTWvUW: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] YdTWvUW: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://muse.ai/embed/YdTWvUW
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/yt_dlp/YoutubeDL.py", line 1560, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/YoutubeDL.py", line 1688, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 710, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/generic.py", line 2568, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://muse.ai/embed/YdTWvUW

The example you provided won't do as it's encrypted.
Use the HLS format instead.
https://cdn-eu.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/videos/hls.m3u8
As for your embed link, it says Video Unavailable.

The example you provided won't do as it's encrypted. Use the HLS format instead.
https://cdn-eu.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/videos/hls.m3u8

Where did you get that link from? I have rather minimal skills at traffic analysis (meaning looking at things in a browser's Developer Tools window). I didn't see that. The (.mpd) link I provided works fine (for me) in yt-dlp.

As for your embed link, it says Video Unavailable. Please provide a valid link.

I just tried the embed link again, and it also opens fine for me. Maybe a geographic restriction exists that I don't know about (I have a United States IP address).
Of note also (looking at the Network tab of Developer Tools), the traffic shows links to the complete files (which I can download with wget, for example), even though the browser keeps returning status 206 (partial content):
https://cdn-na.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/videos/video-720p-video.mp4
https://cdn-na.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/audios/audio-196k-stereo.mp4

It must be geo-protected, I guess.
Also, I didn't "get" the link from anywhere. I literally just renamed the file itself (dash.mpd). Because hls.m3u8 made the most sense.
But without access the video on the website itself
That being said though. The video thumbnail is https://cdn-eu.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/thumbnails/video.jpg
But the video playlist where the video files are available is https://cdn-eu.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/videos/hls.m3u8.
So unless the video URL exists within the source code, the extractor needs a rename function in order to get the correct video URL, and then pipe this URL through ffmpeg.

Maybe a geographic restriction exists that I don't know about (I have a United States IP address).

This is why you don't fill in "global"...

Here's what I put together. But I'm stuck between a rock and a hard place.
My attempt here is to target the thumbnail class and rename "thumbnails/video.jpg' to 'videos/hls.m3u8" and then pipe that through FFmpeg.
from __future__ import unicode_literals
from yt_dlp.extractor.common import InfoExtractor

class MuseAIIE(InfoExtractor):
    _VALID_URL = r'https?://(?:www\.)?muse\.ai/embed/(?P<id>[^/?#]+)'
    _TESTS = [
        {
            'url': 'https://muse.ai/embed/YdTWvUW',
            'info_dict': {
                'id': 'YdTWvUW',
                'ext': 'mp4',
                'title': '2023-05-28-Grabien-1941111 (1)',
                'thumbnail': r're:https?://.*\.jpg$',
            },
            'params': {
                'skip_download': True,
            },
        }
    ]

    def _real_extract(self, url):
        video_id = self._match_id(url)
        webpage = self._download_webpage(url, video_id)

        thumbnail = self._html_search_regex(
            r'<img[^>]+src=["\'](https?://[^"\']+\.jpg)["\']',
            webpage, 'thumbnail', default=None)

        playlist_url = (
            thumbnail.replace('thumbnails/video.jpg', 'videos/hls.m3u8')
            if thumbnail else None
        )

        title = self._html_search_regex(
            r'<a[^>]+class=["\']player-title["\'][^>]+href=["\'][^"\']*["\'][^>]*>([^<]+)',
            webpage, 'title', default='Untitled Video')

        if not playlist_url:
            raise ValueError('No suitable video URL found')

        return {
            'id': video_id,
            'title': title,
            'url': playlist_url,
            'thumbnail': thumbnail,
        }

@pukkandan It turned out to be Firefox related. Not a geo block.

I took another look at this project. The website seem to block access to user agents that aren't Chrome based browsers. So I tried using recent Chrome user-agent. Still no luck grabbing the thumbnail url.
Not sure what else I can do here, with my limited programming skills. 😀 Sorry.

Maybe a geographic restriction exists that I don't know about (I have a United States IP address).

This is why you don't fill in "global"...

No indication existed of any geogragphical blocking (the site basically hosts videos for customers, and news sites generally want their content available everywhere).

Not sure what else I can do here, with my limited programming skills. grinning Sorry.

I have even more limited programming skills, but I have to ask if downloading the files in one piece (from the .mpd file) uses fewer resources (and reduces complexity) compared to downloading "streamed" (hls) versions.

Still no luck grabbing the thumbnail url.

Even the very short (non-javascript) version of the web page has this as part of the (non-executed, obviously) script:
player.setData(
{"description": "", "download": 0, "duration": 1291.03, "embed_domains": ["todaynewsafrica.com", "substack.com"], "fid": "04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0", "filename": "2023-05-28-Grabien-1941111 1.mp4", "height": 720, "ingest_video": 2, "ingesting": false, "license": "owned", "mature": false, "owner_name": "Today News Africa", "owner_shid": "8ytRPX2", "owner_username": "TodayNewsAfrica", "regions": ["na", "eu"], "size": 894950240, "svid": "YdTWvUW", "tcreated":<PHONE_NUMBER>, "title": "2023-05-28-Grabien-1941111 (1)", "twatched": 3164676, "url": "https://cdn.muse.ai/u/JdsD4tX/04a549c2aa68bdc90d9c6fe59913aa09d40404b9abfbc3a77a338426ec3590d0/data", "views": 5852, "visibility": "public", "width": 1280},
{time: args.get('start') || 0},
);

That snippet appears to contain all of the information needed (user id and video id) to download the video and audio files.
Also, this looks like another form of the video (although that page contains other videos, which makes extracting the correct video information slightly trickier):
https://muse.ai/v/YdTWvUW

@bashonly Great work as always, man!


Server crashes after saving world using Azure Files as volume

The server sometimes crashes after trying to save and backup the world. The result is a a 0 byte .wld file and the previous version as .wld.bak appears intact.
Might be related to using Azure Files as a volume.
Running it under Docker as opposed to ACI and without tmux, the crash returns a Win32 IO error. Need to grab logs from the tmux session to see if it's the same.

I'd promised you a copy of the error message I was getting when I ran the container locally. I'm not sure if it is actually related, but I'm sometimes getting this error when I exit:
Assertion at ../../mono/utils/mono-os-mutex.h:73, condition `res != EINVAL' not met

I found this in the mono repo:
https://github.com/mono/mono/pull/2299
In any case, I don't think the error message I'm getting has anything to do with this repo. Also, the save file seems to also be left intact for me.

Thanks for that. It's a different error from what I'm getting but might be related. I'll have a look.