[sbt 1.3.0-RC2] Cannot read user classes from JMX registry with openjdk11

steps
sbt version: 1.3.0-RC2

Use openjdk11
Checkout wvlet/airframe
Run a test: https://travis-ci.org/wvlet/airframe/jobs/543948581#L567

$ ./sbt
sbt:airframe-root> ; ++ 2.13.0; logJVM/testOnly *LoggerJMXTest

[info] - should be registered *** FAILED ***
[info]   javax.management.RuntimeErrorException: java.lang.NoClassDefFoundError: wvlet/log/LogLevel$
[info]   at java.management/com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.rethrow(DefaultMBeanServerInterceptor.java:831)
[info]   at java.management/com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.rethrowMaybeMBeanException(DefaultMBeanServerInterceptor.java:842)
[info]   at java.management/com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.getAttribute(DefaultMBeanServerInterceptor.java:645)
[info]   at java.management/com.sun.jmx.mbeanserver.JmxMBeanServer.getAttribute(JmxMBeanServer.java:678)
[info]   at wvlet.log.LoggerJMXTest.$anonfun$new$2(LoggerJMXTest.scala:33)
[info]   at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
[info]   at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
[info]   at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:22)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:20)
[info]   ...
[info]   Cause: java.lang.NoClassDefFoundError: wvlet/log/LogLevel$
[info]   at wvlet.log.Logger.getLogLevelOf$1(Logger.scala:71)
[info]   at wvlet.log.Logger.getLogLevel(Logger.scala:77)
[info]   at wvlet.log.LoggerJMX$.getDefaultLogLevel(LoggerMXBean.scala:35)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[info]   at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[info]   at java.base/java.lang.reflect.Method.invoke(Method.java:566)
[info]   at sun.reflect.misc.Trampoline.invoke(MethodUtil.java:71)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[info]   ...
[info]   Cause: java.lang.ClassNotFoundException: wvlet.log.LogLevel$
[info]   at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:471)
[info]   at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:588)
[info]   at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
[info]   at wvlet.log.Logger.getLogLevelOf$1(Logger.scala:71)
[info]   at wvlet.log.Logger.getLogLevel(Logger.scala:77)
[info]   at wvlet.log.LoggerJMX$.getDefaultLogLevel(LoggerMXBean.scala:35)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[info]   at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[info]   at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[info]   at java.base/java.lang.reflect.Method.invoke(Method.java:566)

Corresponding code: https://github.com/wvlet/airframe/blob/962d6c2dd1d1acbf9d59a792701aa62b935d6226/airframe-log/jvm/src/test/scala/wvlet/log/LoggerJMXTest.scala#L33

problem

The same code worked before in sbt 1.3.0-RC1 with openjdk11

expectation

Fixing the classpass or the class loader to be compatible with openjdk11's behavior

notes

I can't reproduce this on mac or linux:
ethanatkins@Ethans-MacBook-ProZ:~/work/oss/sbt/community/wvlet/airframe$ java -version; sbt -Dsbt.version=1.3.0-RC2 ";++2.13.0; logJVM/testOnly *LoggerJMXTest"
openjdk version "11.0.2" 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)
[info] Loading settings for project global-plugins from global.sbt ...
[info] Loading global plugins from /Users/ethanatkins/.sbt/1.0/plugins
[info] Loading settings for project airframe-build from plugin.sbt ...
[info] Loading project definition from /Users/ethanatkins/work/oss/sbt/community/wvlet/airframe/project
[info] Loading settings for project root from build.sbt ...
[info] Resolving key references (36333 settings) ...
[info] Set current project to airframe-root (in build file:/Users/ethanatkins/work/oss/sbt/community/wvlet/airframe/)
[info] Setting Scala version to 2.13.0 on 34 projects.
[info] Excluded 7 projects, run ++ 2.13.0 -v for more details.
[info] Reapplying settings...
[info] Set current project to airframe-root (in build file:/Users/ethanatkins/work/oss/sbt/community/wvlet/airframe/)
[info] LoggerJMXTest:
[info] LoggerJXM
[info] - should be registered
[info] - should support setting log levels through JMX
[info] Run completed in 437 milliseconds.
[info] Total number of tests run: 2
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 2, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 3 s, completed Jun 10, 2019, 3:54:57 PM

It seems there was some issue at Travis CI. It's working properly now. Let me close this. Thank you for your time for checking!

@eatkins This issue happens sporadically. After running the same command multiple times, this issue happens.
leo@TD-0504:~/work/git/airframe> java -version                                                                   [0:03:46 Jun 11 2019]
openjdk version "11.0.2" 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)
leo@TD-0504:~/work/git/airframe> ./sbt                                                                           [0:03:49 Jun 11 2019]
[info] Loading settings for project global-plugins from idea.sbt,plugins.sbt ...
[info] Loading global plugins from /Users/leo/.sbt/1.0/plugins
[info] Loading settings for project airframe-build from plugin.sbt ...
[info] Loading project definition from /Users/leo/work/git/airframe/project
[info] Loading settings for project root from build.sbt ...
[info] Resolving key references (36298 settings) ...
[info] Set current project to airframe-root (in build file:/Users/leo/work/git/airframe/)
[info] Welcome to sbt 1.3.0-RC2.
[info] Here are some highlights of this release:
[info]   - Coursier: new default library management using https://get-coursier.io
[info]   - Super shell: displays actively running tasks
[info]   - Turbo mode: makes `test` and `run` faster in interactive sessions. Try it by running `set ThisBuild / turbo := true`.
[info] See https://www.lightbend.com/blog/sbt-1.3.0-release for full release notes.
[info] Hide the banner for this release by running `skipBanner`.
[info] sbt server started at local:///Users/leo/.sbt/1.0/server/eda4f13789b4b13dc0aa/sock
sbt:airframe-root> ;++ 2.13.0; jmx/test
[info] Setting Scala version to 2.13.0 on 34 projects.
[info] Excluded 7 projects, run ++ 2.13.0 -v for more details.
[info] Reapplying settings...
[info] Set current project to airframe-root (in build file:/Users/leo/work/git/airframe/)
Jun 12, 2019 12:04:32 AM wvlet.log.Logger log
INFO: Started JMX agent at localhost:55498
[info] JMXRegistryTest:
[info] JMXRegistry
[info] - should register a new mbean
[info] - should support class field
[info] - should handle nested JMX MBean
[info] - should avoid double registration
[info] - should support complex trait name
[info] JMXAgentTest:
[info] JMXAgent
[info] - should find jmx registry
[info] Run completed in 672 milliseconds.
[info] Total number of tests run: 6
[info] Suites: completed 2, aborted 0
[info] Tests: succeeded 6, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 3 s, completed Jun 12, 2019, 12:04:32 AM
sbt:airframe-root> jmx/test
[info] JMXRegistryTest:
[info] wvlet.airframe.jmx.JMXRegistryTest *** ABORTED ***
[info]   java.lang.NoClassDefFoundError: wvlet/log/LogLevel$
[info]   at wvlet.log.LogRecord$.apply(LogRecord.scala:32)
[info]   at wvlet.log.LogFormatter.format(LogFormat.scala:32)
[info]   at wvlet.log.LogFormatter.format$(LogFormat.scala:29)
[info]   at wvlet.log.LogFormatter$SourceCodeLogFormatter$.format(LogFormat.scala:143)
[info]   at wvlet.log.ConsoleLogHandler.publish(Handler.scala:15)
[info]   at java.logging/java.util.logging.Logger.log(Logger.java:979)
[info]   at wvlet.log.Logger.log(Logger.scala:144)
[info]   at wvlet.log.Logger.log(Logger.scala:148)
[info]   at wvlet.airframe.jmx.JMXUtil$.startAndGetAgentURL(JMXUtil.scala:107)
[info]   at wvlet.airframe.jmx.JMXAgent.<init>(JMXAgent.scala:55)
[info]   ...
[info]   Cause: java.lang.ClassNotFoundException: wvlet.log.LogLevel$
[info]   at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:471)
[info]   at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:588)
[info]   at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
[info]   at wvlet.log.LogRecord$.apply(LogRecord.scala:32)
[info]   at wvlet.log.LogFormatter.format(LogFormat.scala:32)
[info]   at wvlet.log.LogFormatter.format$(LogFormat.scala:29)
[info]   at wvlet.log.LogFormatter$SourceCodeLogFormatter$.format(LogFormat.scala:143)
[info]   at wvlet.log.ConsoleLogHandler.publish(Handler.scala:15)
[info]   at java.logging/java.util.logging.Logger.log(Logger.java:979)
[info]   at wvlet.log.Logger.log(Logger.scala:144)
[info]   ...
[info] JMXAgentTest:
[info] JMXAgent
[info] - should find jmx registry
[error] Test suite wvlet.airframe.jmx.JMXRegistryTest failed with java.lang.NoClassDefFoundError: wvlet/log/LogLevel$.
[error] This may be due to the ClassLoaderLayeringStrategy (Flat) used by your task.
[error] To improve performance and reduce memory, sbt attempts to cache the class loaders used to load the project dependencies.
[error] The project class files are loaded in a separate class loader that is created for each test run.
[error] The test class loader accesses the project dependency classes using the cached project dependency classloader.
[error] With this approach, class loading may fail under the following conditions:
[error]
[error]  * Dependencies use reflection to access classes in your project's classpath.
[error]    Java serialization/deserialization may cause this.
[error]  * An open package is accessed across layers. If the project's classes access or extend
[error]    jvm package private classes defined in a project dependency, it may cause an IllegalAccessError
[error]    because the jvm enforces package private at the classloader level.
[error]
[error] These issues, along with others that were not enumerated above, may be resolved by changing the class loader layering strategy.
[error] The Flat and ScalaLibrary strategies bundle the full project classpath in the same class loader.
[error] To use one of these strategies, set the  ClassLoaderLayeringStrategy key
[error] in your configuration, for example:
[error]
[error] set jmx / Test / classLoaderLayeringStrategy := ClassLoaderLayeringStrategy.ScalaLibrary
[error] set jmx / Test / classLoaderLayeringStrategy := ClassLoaderLayeringStrategy.Flat
[error]
[error] See ClassLoaderLayeringStrategy.scala for the full list of options.
[info] Run completed in 212 milliseconds.
[info] Total number of tests run: 1
[info] Suites: completed 1, aborted 1
[info] Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
[info] *** 1 SUITE ABORTED ***
[error] Error during tests:
[error] 	wvlet.airframe.jmx.JMXRegistryTest
[error] (jmx / Test / test) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 1 s, completed Jun 12, 2019, 12:04:36 AM```

It's probably because my code is trying to reuse JMX agent. Similarly to shutdown hook, in the current sbt configuration, we cannot reuse classes registered to JMX agent.

I've tested different strategies, but the result has been the same.
I guess sbt 1.3.0-RC2 cannot load classes defined in other projects.

Hmm. This seems like a bug. It shouldn't work sometimes. I'm reopening.

Actually, you were more or less right. I do find that the tests seem to work if I run with -Dsbt.classloader.close=false. The only other choice is to fork that project.


Failures after download, and duplicates

Often times it is just failing after downloading, which should have been a success point. it seems to randomly create an additional copy "original" "original (1)" despite proper settings. the behavior is not consistent.
the following is an attempt to download this URL
https://www.youtube.com/channel/UCDWLDVilBdxihGA55t2l8Iw/videos

to reiterate, it actually does hit 100%, the .tmp files show size, but then it fails to merge the .tmp streams and leaves behind a 0 file size and deletes the .tmp

The "original (1)" happens when there already files with the same name. If you download many videos at once, you don't have a chance to choose which files to overwrite, so it avoids doing it to be safe.

does it do that even when the option to 'skip downloads for files that already exist' is enabled?
also, on the page i linked, if you were to click the 'play all' to go one step further into a generated play list, the problem does not occur. this is the first sime i have seen this particular behavior. it does this in every test i have tried. only affects 1.7.12
Thank you, and be well

is it supposed to do that when the option to 'skip downloads for files that already exist' is enabled? it is in this enabled in the above example.

No, it should not.

@Hunanbean What I've found that works is using strings without spaces or special characters for the file name.
Example insteead of "Bla bla - Song (2019)" if I save it as "Bla01" it works.
Then I just rename it afterwards.

That is good to know, thanks. But, i am curious as to a shortcut to do that
on a list of 400 videos (like the 'inside unreal' list). Is there a place i
can enter format override commands, or does that advice apply more to
single file downloads?
On Sat, Aug 1, 2020 at 4:24 AM Alexey Golub<EMAIL_ADDRESS>wrote:

is it supposed to do that when the option to 'skip downloads for files
that already exist' is enabled? it is in this enabled in the above example.
No, it should not.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Tyrrrz/YoutubeDownloader/issues/125#issuecomment-667516587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNVR7D56TOJEHV7SDAAWS3R6P3QVANCNFSM4PNNLA4A
.

is this app support virtual-pn ？

@Bian-Sh I don't know what that is.

I believe they mean VPN, virtual private network. I cannot think of a
reason why it would not work on a fully functional VPN.
On Mon, Dec 14, 2020 at 12:40 PM Alexey Golub<EMAIL_ADDRESS>wrote:

@Bian-Sh https://github.com/Bian-Sh I don't know what that is.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Tyrrrz/YoutubeDownloader/issues/125#issuecomment-744697193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNVR7A33PGKEEDZU5SGKJ3SUZZ3TANCNFSM4PNNLA4A
.

I believe they mean VPN, virtual private network. I cannot think of a reason why it would not work on a fully functional VPN.
…
On Mon, Dec 14, 2020 at 12:40 PM Alexey Golub @.***> wrote: @Bian-Sh https://github.com/Bian-Sh I don't know what that is. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#125 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNVR7A33PGKEEDZU5SGKJ3SUZZ3TANCNFSM4PNNLA4A .

I just find that it works well ,so i was wrong,sorry 咯~


Add Java Endpoints quickstart

I need to build the docker image and push to gcr

I built and pushed the java docker image. It uses the flex Dockerfile which listens on 8080, so we will need to switch the container-engine.yaml to use 8081 for the esp.

@tswast Could you help Brad with his code?

Could you remove managedvms from your path & package names.   If you want to keep something there, it should be flexible, but I've been striking it to make things shorter.
Checkstyle says:
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/AuthInfoServlet.java:20:8: error: Unused import - java.io.PrintWriter.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/AuthInfoServlet.java:28: error: Import statement for 'com.google.gson.Gson' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/AuthInfoServlet.java:29: error: Import statement for 'com.google.gson.JsonObject' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:20:8: error: Unused import - java.io.InputStream.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:21:8: error: Unused import - java.io.OutputStream.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:29: error: Import statement for 'com.google.gson.Gson' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:30: error: Import statement for 'com.google.gson.JsonObject' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:31: error: Import statement for 'com.google.gson.JsonParseException' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.
/home/travis/build/GoogleCloudPlatform/java-docs-samples/endpoints/getting-started/src/main/java/com/example/managedvms/endpoints/EchoServlet.java:32: error: Import statement for 'com.google.gson.stream.JsonReader' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.

Looks good, and I'll approve if you at least try to using the new maven plugin, and if it fails, please report at:  https://github.com/GoogleCloudPlatform/app-maven-plugin

Brad assures me that all of Jeff's asks are done as well.

@googlebot confirm

Admin Approve


Add peer connection timeout

The process will not stop because there is a peer connection to a zombie peer that never completed a handshake:
INFO:squeaknode.node.peer_connection_worker:Connecting peers...
INFO:squeaknode.network.peer_client:Connecting to peer with address PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Making connection to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Got socket to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.node.squeak_deletion_worker:Deleting old squeaks...
INFO:squeaknode.node.squeak_controller:Syncing timeline with block range: BlockRange(min_block=2063547, max_block=2065563)
INFO:squeaknode.node.squeak_offer_expiry_worker:Deleting expired offers...
INFO:squeaknode.node.peer_connection_worker:Connecting peers...
INFO:squeaknode.network.peer_client:Connecting to peer with address PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Making connection to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Got socket to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.node.squeak_deletion_worker:Deleting old squeaks...
INFO:squeaknode.node.squeak_controller:Syncing timeline with block range: BlockRange(min_block=2063547, max_block=2065563)
INFO:squeaknode.node.squeak_offer_expiry_worker:Deleting expired offers...
INFO:squeaknode.node.peer_connection_worker:Connecting peers...
INFO:squeaknode.network.peer_client:Connecting to peer with address PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Making connection to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.network.peer_client:Got socket to PeerAddress(host='fooobar.com', port=18555)
INFO:squeaknode.node.squeak_controller:Syncing timeline with block range: BlockRange(min_block=2063547, max_block=2065563)
INFO:squeaknode.node.squeak_deletion_worker:Deleting old squeaks...
INFO:squeaknode.node.squeak_offer_expiry_worker:Deleting expired offers...

This could probably be fixed with a handshake timeout, and also a ping timeout.

https://github.com/yzernik/squeaknode/pull/995


v2.10 - The Plan

For v2.10, our focus will be on addressing several high-priority changes, feature requests, and a wide range of bug fixes.
While performance improvements were the main priority in v2.9, they will still be included in this release, but won't be the primary focus.
Regarding timeline, I found that a 2 month period was good. Let's shoot for the end of October for now, and we'll adjust as needed.
Changes
More flexible validation of generics

https://github.com/pydantic/pydantic/issues/10359

https://github.com/pydantic/pydantic/issues/9414

other similar issues, filter on https://github.com/pydantic/pydantic/labels/generics

https://github.com/pydantic/pydantic/issues/9467

https://github.com/pydantic/pydantic/issues/10315

Feature Requests

https://github.com/pydantic/pydantic/issues/9789
https://github.com/pydantic/pydantic/issues/9619

EDIT: deferred until v2.11:

https://github.com/pydantic/pydantic/issues/8394
https://github.com/pydantic/pydantic/issues/9731

Bug Fixes
I'm sure we'll work on many bugs that aren't listed here, but a few that come to mind at the moment are:

https://github.com/pydantic/pydantic/issues/8232
https://github.com/pydantic/pydantic/issues/7353 (and other url issues, we ran out of time for this in v2.9)
https://github.com/pydantic/pydantic/issues/9683

I've collected some issues in https://github.com/pydantic/pydantic/milestone/19 to serve as a guideline for work during this milestone. I don't think we'll complete everything in the milestone within the projected time block, but this provides some insight into issues that we recognize as important enough to address in the short term.
If you have an issue / feature request that you'd like included in this release:

Leave a comment on this issue - we're happy to consider your suggestion. We're much more likely to prioritize a feature request / change if there's been demonstrated high demand (think 👍 reacts, comments, etc).
If you really want a feature request / change, please open a PR! That's the best way to fast track adoption 🐎. We're excited to collaborate with you 🚀 !

What about https://github.com/pydantic/pydantic/issues/9686?
Did I miss this in the milestone?

Oh I see this is for pydantic-core.  Will it be part of the 2.10 release?

Probably won't get into v2.10 unless someone opens a PR with support for it - happy to review though! I've pinged DH re the most recent question to see if we can get the ball rolling there again.

Still TODO:

[ ] Document https://github.com/pydantic/pydantic/issues/9686 (@sydney-runkle)
[ ] Fix AnyUrl isinstance issue raised by @alexmojaki (@sydney-runkle is working on this)
[ ] https://github.com/pydantic/pydantic/issues/9467 and related issues (@sydney-runkle)
[ ] Merge pydantic PR for https://github.com/pydantic/pydantic/issues/9789
[ ] Finish up CoreMetadata refactor PR
[ ] Finish up KeyError Generic PR (@Viicos)

There's probably more, we can keep this up to date as we wrap things up in the next few days

What about #9782? I saw that it was postponed for v2.10

What about #9782? I saw that it was postponed for v2.10

We might take a look for 2.11. I'll take a look to gauge how much work it requires.

@caniko, regarding PEP 695 support, are you running into issues with the new type syntax (https://github.com/pydantic/pydantic/issues/8984) or with the new generics syntax?

The new generics syntax works fine, I was referring to the new type syntax.

Hi friends! We've finished much of the above - going to close this as resolved, given that we've moved forward with the beta release.
We're a bit behind on PR reviews given this push to get things out, so we'll be catching up on those soon.


Poorly document when wrong authentication method was initially chosen

You say the following.
Q: How do I resolve error: The user was not authorized?
The user was not authorized.

A: You can encounter this error if you try to access a project to which you don't have permissions. To get access, see Set permissions to access Analytics and Analytics views.

The user was not authorized.

This is so useless as information. What many of us do is the following: The first time we try to connect we chose a connection method and then discover that method does not work. PowerBI however only asks ONE time how to connect (as you also document), however we now get the error "The user was not authorized?" and we know it is because we need to find a way to forget the prior authenication and try again. However that is nowhere documented here. This error should explain what to do when you get this error due to having chosen the wrong method and now you want to try the right method.

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

ID: 9ea920e3-9644-ae0c-120d-db9e899ee192
Version Independent ID: 452bf06e-28f4-7815-72b3-16a4c2086e47
Content: Connect to Power BI Data Connector - Azure DevOps
Content Source: docs/report/powerbi/data-connector-connect.md
Product: devops
Technology: devops-analytics
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@blouwagie  - Thank you for this feedback. And apologies for the delay. I agree that more information should be provided. I'll work to get this article updated within the next few weeks.


Back button isn't displayed

Steps to Reproduce

Run this Gist on DartPad: https://dartpad.dev/71e1f185541eecc0a934723261b96901
Click "show details"
See that the back button doesn't appear:

@dnfield @willlarche Is it possible that the icon is getting optimized away since it's not used directly?

Web doesn't do any icon shaking. Does this reproduce on mobile/desktop?

Oh. It seems like whatever revision DartPad is on is not matched properly to the icon font it's trying to use.  See e.g.:
import 'package:flutter/material.dart';

void main() {
  runApp(
    MaterialApp(
      home: ColoredBox(
        color: Colors.white,
        child: Column(children: [
          Icon(Icons.play_arrow),
          Icon(Icons.ac_unit),
        ]),
      ),
    ),
  );
}

Shows the wrong icons in Dart.

Good catch, I'll move this issue to DartPad's tracker: https://github.com/dart-lang/dart-pad/issues/1627

Closing this one in favor of the DartPad bug. We can reopen if it turns out to require a fix on Flutter's end.


Support for Angular 7

@armved It seems to me this repository not maintained anymore, so if you want I created a new repository with support for Angular 7.
Here is a link to ngx-d3

Nice many thanks!


food vision

after using mixed precision the base model's or efficientnet model's first input layer dtype policy is getting changed into float16


Can we stop jdk12u nightly builds now development has finished for it?

Platform:
Architecture:
Please describe the issue that you are having with our binaries and if possible how to recreate it.

Builds have been stopped and we'll clean up workspaces as we discover them


Improve new user experience with installation/usage section in README

From the 'List of relevant files and modules' in the README, it was not clear to me as a new user how to set this up.
I've written my best (educated) guess so far in a new section. A few sentences about the expected output would make this even better.

Is present (and will be kept evergreen)


ERRO[0000] no configuration for command npm found

Used sample .envcli.yml from readme
When running envcli run npm init, error is:

ERRO[0000] no configuration for command npm found

Error is both on mac & ubuntu server 18.04.

It worked when I used the sample from /docs/ folder as the content of .envcli.yml file.
https://github.com/EnvCLI/EnvCLI/blob/894398d189df49837edde9ff102a278a92abe3d4/docs/examples/node.md

Yeah, you need to define each command you want to use in the .envcli.yml config. If you have any futher questions feel free to join the discord and ask there.


Limit the number of entities fetched per statement

Without this restriction, Cassandra can easily time out while trying to fetch many rows from the entities table.
This particular fix has the potential to increase the latency a bit in some cases, but some increased latency is better than a timeout.
Resolves #2943.

https://ocd.appscale.com:8080/job/Daily Build/5766/
https://ocd.appscale.com:8080/job/Taskqueue-e2e-Test/190/


Fix error when guild id is None.

About this pull request
I would say this is quite critical. Not sure why that checking was removed at this commit:
https://github.com/discord-py-slash-commands/discord-py-interactions/commit/b32d0e2da956bbc31066d7bdbe4240c97258eca3#diff-37afa4fe0663ae6e7e30366179bcffecedbdba72972e291cb682fec2ac4f0a24L538-L539
Changes
Add checking for when guild_id is None for command.
Checklist

[ ] I've run the pre_push.py script to format and lint code.
[ ] I've checked this pull request runs on Python 3.6.X.
[ ] This fixes something in Issues.

Issue: fixes #225

[ ] This adds something new.
[ ] There is/are breaking change(s).
[ ] (If required) Relevant documentation has been updated/added.
[ ] This is not a code change. (README, docs, etc.)

That was my bad. I thought I added if guild_ids to both of those.

It seems to me that with such checks empty list of guild_ids would be a valid input? Is that actually desired? I'm not sure but it could cause some other bugs down the line

https://github.com/discord-py-slash-commands/discord-py-interactions/blob/a7f468b66137726a2535a9d62177f6e25a8f8d2f/discord_slash/model.py#L315
It will always be modelled to an empty list always from what I can tell.

Then LGTM

Actually @artem30801 you are right. There are instances where it tried to .copy a None guild_ids, I think it best is to revert back to what was like before, at least for the hotfix. Then we see if we should consider later on where removing that guild_ids = guild_ids if guild_ids else [] has any benefit or not.

I think that
guild_ids = guild_ids if guild_ids else []
also could be changed to to
guild_ids = guild_ids or []

I think both have negligible difference, let keep with what prove works for now. We can do some optimisation on that after this hotfix.


Plugin error: [...] Error occurred in getTabProps

i have found 2 plugins with this error.
hyperterm-alternatescroll issue #5
hyperterm-tab-cwd issues #1

The relation between components got changed slightly in the 0.8.0 release, which means some plugins needs updates. Please track the issues you linked instead though, as this isn't a problem with Hyper's core.

is there an release info what changed exactly or how to fix this?
i dont see how its changed here https://github.com/zeit/hyper/compare/0.7.1...0.8.0


Plotter status not updating correctly

Describe the bug
Plotman, on a plotter only worker, is not starting a new Madmax instance as expected when the 1st has started to copy to destination folder.  Watching the plotting status on the /plotting/jobs page shows that when it gets to phase 4:1 it just stays there even after the copy to dst has started and it should be in phase 5:1.  Incidentally, the "wall" time also stops updating.  Also the "Plotting Speed" charts do not show the stats from the 2nd worker at all.
To Reproduce
I have a "fullnode" Machinaris container on the primary server, and a "plotter" worker on a 2nd machine.
The plotman config files are mostly default aside from dst paths and number of threads for madmax. (the scheduling bits are the same).
Plotting jobs on the "fullnode" work as expected.  When reaching stage 5:1 (copying to destination) a new plotter gets spun up while the copying proceeds.
Plotting jobs on the "plotter" worker never update the status when reaching stage 5:1, and therefore never spin up a new plotting job until the copy to destination has actually finished.
Expected behavior
Expected that plotter on 2nd machine starts a new plot job when the 1st one starts copying final plot to destination (stage 5:1).
System setup:

OS: Unraid 6.10.0-rc2 (with docker-compose)
Docker version number 20.10.9
Machinaris branch: test
Machinaris version number 0.6.9

Config
Plotting scheduling parameters (same on both workers)
scheduling:
# Run a job on a particular temp dir only if the number of existing jobs
# before tmpdir_stagger_phase_major tmpdir_stagger_phase_minor
# is less than tmpdir_stagger_phase_limit.
# Phase major corresponds to the plot phase, phase minor corresponds to
# the table or table pair in sequence, phase limit corresponds to
# the number of plots allowed before [phase major, phase minor]
# Five is the final move stage of madmax
tmpdir_stagger_phase_major: 5
tmpdir_stagger_phase_minor: 0
# Optional: default is 1
tmpdir_stagger_phase_limit: 1
    # Don't run more than this many jobs at a time on a single temp dir.
    # Increase for staggered plotting by chia, leave at 2 for madmax sequential plotting
    tmpdir_max_jobs: 2

    # Don't run more than this many jobs at a time in total.
    # Increase for staggered plotting by chia, leave at 2 for madmax sequential plotting
    global_max_jobs: 2

    # Don't run any jobs (across all temp dirs) more often than this, in minutes.
    global_stagger_m: 30

    # How often the daemon wakes to consider starting a new plot job, in seconds.
    polling_time_s: 20

Additional context & screenshots

Hi, thanks for the detailed report. Please provide plotman status output from the 2ndary plotter worker at stage shown above.  On the plotting system:
docker exec -it machinaris bash
plotman status

Please paste a screenshot of this status output from the plotman CLI.  Thanks.

plotman status command appears to hang and does not return anything at all:

Workers page:

plotman status finally returned when the plot finished copying to destination.  Executing again right after shows new plot started.

Hi, thanks for the detailed response.  Very interesting that the separate plotman status process on your Ripper machine actually hung for the duration of the copy.  As that is a separate plotman invocation, which simply spins through the container's process list looking for other running plotman and chia_plot processes, I am thinking there is some resource contention that is slowing/pausing the entire Docker container, not just the single Plotman job.
Secondly, you mentioned that the same plotting configuration on your Fullnode system does not exhibit this issue.  Since the code is the same, that indicates something is different between the two systems at a hardware/volume level.

What differences existing in the dst volume path on each system?
Is the dst path on each system an Unassigned Device in Unraid?
Or perhaps a drive share with parity?  If so, is caching turned on for that share?
Or is the dst path on Ripper plotter is a remote network share over the ChiaBoxZero fullnode?

I would recommend experimenting with different dst locations on the plotter.  The original Plotman author's design, particularly for remote plotting, was for:

tmp: to be SSD or RAM disk.
dst: to be a locally attached staging drive
Then use Plotman archving to transfer the completed plot from the staging location in dst over to a final location, either a remote server or a slow local drive.

Hope this helps,
Guy

Just found this, probably related:
https://github.com/ericaltendorf/plotman/issues/714
Should mention that host OS for secondary plotter is Ubuntu 20.04, and destination drive is mounted NFS share from primary Unraid server.
Going to switch to local dst when current job is finished.  Looks like maybe what I need to do is figure out how to get rsync setup for the archiver.
Edit: You posted as I was writing this.  I'll report back this afternoon when first job with local dst finishes, but I suspect we'll be able to close this as not a Machinaris issue.

Setting dst to a local path appears to have solved the problem.
Sorry to have wasted your time.
It's a shame though, seems like a pointless hop to get to the final destination.  And with the only drive I have available to use as a local dst currently, it's actually slower than copying over the network.  So what was a 1 hour copy delay from the time a plot finished to when it was harvestable, is now 2.5 to 3 hours.
Maybe I'll try leaving the finished plot on the tmp folder and writing my own script to move it to the Unraid share.
Anyhow, not a Machinaris issue.  Thank you.

Hi, no worries.  One thing to try would leaving the dst list empty in the plotman settings.  My understanding is that Madmax would then consider the plot complete and leave it in the tmp location (probably an SSD).  I believe you could then use Plotman archiving to transfer it from the tmp location to the remote system via rsync.  I have not tested this scenario myself however, so please take this with a grain of salt.
Cheers!


Chunking for telemetry sobject batch

Critical Changes
Changes

Organizations with more than 50 million Opportunities associated with open-ended Recurring Donations were seeing an error when the UTIL_OrgTelemetry_SObject_BATCH ran. We've implemented batch chunking to resolve these errors.

Issues Closed
Fixes #4119
New Metadata
Deleted Metadata

**lurch: attach W-031902

Tracking W-031902

@seanpat09 - Can you take a look at Changes and tell me if those notes are accurate? And does this use Primary Key Chunking?

@seanpat09 - Can you take a look at Changes and tell me if those notes are accurate? And does this use Primary Key Chunking?

@sam-knox The notes in Changes look great! And yes this, uses Primary Key Chunking (under the assumption that Primary Key Chunking is running a batch that processes records in order of their Primary Key, e.g. the ID field, in chained batches of 10,000,000 records each).

@seanpat09 and @sam-knox Keep in mind that generally "Primary Key Chunking" in Salesforce generally refers to bulk api processes with a Sforce-Enable-PKChunking SOAP header. This is a similar concept, but not the same thing as "PK Chunking" as defined here. https://developer.salesforce.com/docs/atlas.en-us.api_asynch.meta/api_asynch/async_api_headers_enable_pk_chunking.htm

@RadSFDeveloper @force2b Made some changes based on your feedback, let me know if there's anything else I should add. Thank you both again!


ReferenceError: window is not defined

To reproduce

Install Discord
Install BetterDiscord
Launch Discord

I did not use sudo as per #41

Running betterdiscordctl reinstall gives /usr/local/bin/betterdiscordctl: line 502: /home/usr/.local/share/betterdiscordctl/bd_map: Permission denied

You have to remove /home/usr/.local/share/betterdiscordctl since it was created with sudo and try again.

Reinstalled, got
usr@ubuntuhost:~/Downloads/bdinstaller$ betterdiscordctl install
Cloning https://github.com/rauenzi/BetterDiscordApp...
Cloning into '/home/usr/.local/share/betterdiscordctl/bd/0'...
remote: Enumerating objects: 12, done.
remote: Counting objects: 100% (12/12), done.
remote: Compressing objects: 100% (11/11), done.
remote: Total 12 (delta 0), reused 4 (delta 0), pack-reused 0
Receiving objects: 100% (12/12), 6.39 KiB | 6.39 MiB/s, done.
Installed. (Restart Discord if necessary.)

I don't see BetterDiscord options anywhere (restarted discord as well.)
Also tried betterdiscordctl reinstall

See #38


Make pheno_extract_candidates less top-heavy!

Reduce size of loaded data structures

CF here: http://stackoverflow.com/questions/25108014/python-defaultdict-for-large-data-sets

@Colossus I think this isn't too high on our priority list right now due to your #42 fix correct, right?
Can always re-open this

@ajratner Yeah sure, sounds good


:robot: Update dependabot.yml

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (6acf4e6) 99.45% compared to head (7184389) 99.45%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   99.45%   99.45%
=======================================
  Files          38       38
  Lines        1821     1821
=======================================
  Hits         1811     1811
  Misses         10       10

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


[zstd] Build error on x64-windows(only VCPKG_BUILD_TYPE is debug)

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.39.33523.0
vcpkg-tool version: 2024-06-10-02590c430e4ed9215d27870138c2e579cc338772
vcpkg-scripts version: 1318ab14a 2024-07-16 (28 hours ago)

To Reproduce
vcpkg install --overlay-triplets=...../triplets/x64-windows.cmake
Failure cmd
file(READ "${CURRENT_PACKAGES_DIR}/share/zstd/zstdTargets.cmake" targets)

Failure logs
-- Using cached facebook-zstd-v1.5.6.tar.gz.
-- Cleaning sources at Z:/git/vcpkg/vcpkg/buildtrees/zstd/src/v1.5.6-6d2acef635.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source Z:/git/vcpkg/vcpkg/downloads/facebook-zstd-v1.5.6.tar.gz
-- Applying patch no-static-suffix.patch
-- Applying patch fix-emscripten-and-clang-cl.patch
-- Applying patch fix-windows-rc-compile.patch
-- Using source at Z:/git/vcpkg/vcpkg/buildtrees/zstd/src/v1.5.6-6d2acef635.clean
-- Found external ninja('1.11.0').
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Using cached msys2-mingw-w64-x86_64-pkgconf-1~2.2.0-1-any.pkg.tar.zst.
-- Using cached msys2-msys2-runtime-3.5.3-3-x86_64.pkg.tar.zst.
-- Using msys root at Z:/git/vcpkg/vcpkg/downloads/tools/msys2/1e74ca60daa10104
-- Fixing pkgconfig file: Z:/git/vcpkg/vcpkg/packages/zstd_x64-windows/debug/lib/pkgconfig/libzstd.pc
CMake Error at ports/zstd/portfile.cmake:42 (file):
  file failed to open for reading (No such file or directory):

    Z:/git/vcpkg/vcpkg/packages/zstd_x64-windows/share/zstd/zstdTargets.cmake
Call Stack (most recent call first):
  scripts/ports.cmake:192 (include)

Additional context
vcpkg.json
{
  "name": "test",
  "version-string": "1.0",
  "port-version": 2,
  "description": "test",
  "homepage": "https://github.com/",
  "supports": "!(x86 | wasm32)",
  "dependencies": [
    "zstd"
  ]
}

triplets/x64-windows.cmake
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)
set(VCPKG_BUILD_TYPE debug)

set(VCPKG_BUILD_TYPE debug)

This is unsupported and must not be used.
https://learn.microsoft.com/vcpkg/users/triplets#vcpkg_build_type

TKS, I found these
Separate release and debug builds #1626
Add support for VCPKG_BUILD_TYPE = "debug" #38224


Add re-ranking script, including average doc embedding approach

After this merge, delete all branches.


Add a clear method to the factory

add clear method to abstract factory
add clear to the interface

Fixes #21

@winzou updated (I've also checked the "allow edits from maintainers" so if you have further change and want to do them yourself, you can).

That's perfect, thanks!


New lint check: CheckUnusedLabels

Check for unused labels blocks. This can help catching errors like:
() => {a: 2}  // Returns undefined, not an Object

Inspired by ESLint (https://github.com/eslint/eslint/blob/master/lib/rules/no-unused-labels.js)
Fixes #1128.

This change is

@concavelenz


[T4A4][T09-B2] Ashley Ong

Ready for review.

Accepting this PR without a review because another of your PRs
for this tutorial was reviewed in detail.
Few comments added. Check it out.


idea: Tab Index re-number>move content ?

i'm trying to do this, don't know if it will work:
with 4 tabs, i want to move contents(+title) of #3 to #2, and move
#2 to #3  > change order 0,1,2,3 to 0,2,1,3 for a better display,
without having to cut+paste contents and do all the work(!)
maybe if i just renumber tabs,save,close, re-load, it might work..
something one might wish to do in a build, anyway.

ah, it worked.re-numbered, saved, re-loaded, was ok. but
content didn't move. forget it, i can just changed Ownership
of elements, i'm sure it'll be ok now.

Please don't post ideas/new-feature-requests etc. here. This place is for bugs and issues.

sure thing
  From: Roman Kubiak<EMAIL_ADDRESS>
To: RomanKubiak/ctrlr<EMAIL_ADDRESS>Cc: subv0x<EMAIL_ADDRESS>Sent: Thursday, November 12, 2015 1:56 PM
Subject: Re: [ctrlr] idea: Tab Index re-number>move content ? (#14)
Please don't post ideas/new-feature-requests etc. here. This place is for bugs and issues.—
Reply to this email directly or view it on GitHub.


Latest dependencies + build with JDK21

Latest dependencies + build with JDK21, but maintains compatibility with Java 17.

👍

👍


Initial drop of code for the presentation server

Added some notes to #3098 of some of the questions/todos we'll have based off this PR... which of course we are just using to checkin initially.


Again:  AxiosError: Request failed with status code 429, reconnect in  306s.

Now i have again login problems. I config 306 seconds but after hours i get this error:  Login error: AxiosError: Request failed with status code 429, reconnect in  306s.

Please close this issues. I think the problem was initiate from a Home assistant integration

I am getting the same error.
@tobuser what was the issue with Home Assistant integration? Could you please provide a bit more details?
Is there a conflict, I am using Home Assistant and Homebridge and I have the melcloud-control enabled in both places.

The Melcloud error also occurs for me at 240s. I cannot yet say whether the problem is caused by Home Assistant. This is because I first had to completely delete various Melcloud integrations. I also have two Home Assistant installations at two locations, and I had to make sure that all Home Assistant integrations are deleted there as well. Then I had two installations of Melcloud Control at the two locations. All this can lead to the Melcloud error. Now I am testing the current Melcloud Control plugins. Unfortunately, some versions crash, which in turn leads to the Melcloud error. Let's see how it goes.


[twitch] ERROR: 'NoneType' object has no attribute 'format'

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 that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.06.21 (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 all URLs and arguments with special characters are properly quoted or escaped
[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 I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
Tried to download a Twitch stream like this:
yt-dlp 'https://twitch.tv/reddark_247'

What I expected to happen: It starts downloading
What happened instead:
ERROR: 'NoneType' object has no attribute 'format'

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] 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: ['-vU', 'https://twitch.tv/reddark_247']
[debug] Encodings: locale utf-8, fs utf-8, pref utf-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[42f2d40b4]
[debug] Python 3.11.3 (CPython x86_64 64bit) - Linux-6.2.13-arch1-1-x86_64-with-glibc2.37 (OpenSSL 3.1.1 30 May 2023, glibc 2.37)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.18.0, brotlicffi-<IP_ADDRESS>, certifi-2023.05.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1848 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>ERROR: 'NoneType' object has no attribute 'format'
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/yt_dlp/YoutubeDL.py", line 1555, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/YoutubeDL.py", line 1631, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 705, in extract
    self.initialize()
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 609, in initialize
    username, password = self._get_login_info()
                         ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 1335, in _get_login_info
    username, password = self._get_netrc_login_info(netrc_machine)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 1300, in _get_netrc_login_info
    cmd = self.get_param('netrc_cmd', '').format(netrc_machine)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'format'

Same problem here with same version

due to a regression in version 2023.06.21, already fixed in master, new release will made soon
workaround:

add --netrc-cmd ""or --netrc-cmd= to your command (depending on your shell)

or to get the patch, do one of the following:

run yt-dlp --update-to nightly if you are using an executable/binary
install master branch with pip if using pip installation
pull/build from master if running/building from source


[Bug]: RunInference GPU error: Expected all tensors to be on the same device, but found at least two devices, cpu and cuda:0!

What happened?
When using multiple processes (when we don’t use no_use_multiple_sdk_containers) in DataFlow jobs, this error has been observed
Expected all tensors to be on the same device, but found at least two devices, cpu and cuda:0!
The current solution is to add the experiment=no_use_multiple_sdk_containers flag.
We should look into how much benefit we get with multiple processes, as opposed to maxing out GPU utilization in one worker.
Issue Priority
Priority: 2
Issue Component
Component: sdk-py-core

.add-labels run-inference

is anyone working on this issue. If no you can assign me

Not at the moment. You can claim it with the take-issue command. See here for instructions.

.take-issue

.close-issue
not_planned

Likely was closed by mistake.


error in file Using lime for regression.ipynb

explainer = lime.lime_tabular.LimeTabularExplainer(train, feature_names=boston.feature_names, class_names=['price'], categorical_features=categorical_features, verbose=True, mode='regression')

ValueError                                Traceback (most recent call last)
 in ()
----> 1 explainer = lime.lime_tabular.LimeTabularExplainer(train, feature_names=boston.feature_names, class_names=['price'], categorical_features=categorical_features, verbose=True, mode='regression')
C:\Anaconda3\lib\site-packages\lime\lime_tabular.py in init(self, training_data, mode, training_labels, feature_names, categorical_features, categorical_names, kernel_width, verbose, class_names, feature_selection, discretize_continuous, discretizer, random_state)
148         self.mode = mode
149         self.categorical_names = categorical_names or {}
--> 150         self.categorical_features = categorical_features or []
151         self.feature_names = feature_names or [str(i) for i in range(training_data.shape[1])]
152
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

fixed in 7990a8af82fb31d0c599bf73e14854579a8eb9b0
Thanks,

super thanks
but this setup file is not working for Windows computer
some how python setup.py install is not working properly
, sorry
may you update The lime package  on PyPI
then it will be possible to use
pip install lime
Thank you very much in advance


The allergies exercise has some bad practices

[x] Rename allergic_to? to allergic-to?
[x] Rename list to allergies
[x] Use keywords instead of strings for allergens
[x] Use (some #{x} xs) idiom
[x] Use keep-indexed

Closing, since #69 exists.


fix: Only add missing user set actions to scheduling order

Description
If Api1 and Api2 are marked to be run on page load manually, and only Api1 is referred to in a widget, Api1 gets added twice in the scheduling order.
This has been fixed by only adding the difference between total user set actions and already mapped user set actions to the scheduling order.
Fixes #12596
Type of change

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

How Has This Been Tested?

JUnit test case
Manual testing

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=e27f19a


Terrible results

I trained the model by myself,but the results were terrible,there are always black things around eyes.

did you run this code  successfully? could you plz share ur code,appreciate


Macro optimisation of copyProperties.

According to  https://jsperf.com/object-keys-vs-for-in-with-closure/3 .

This does not appear to be an axiom.

@probablyup What browser you are using?
I was trying on several chrome versions and each time Object.keys() was faster.

Safari.
On Mon, Jul 17, 2017 at 9:09 AM Alexei Gontar<EMAIL_ADDRESS>wrote:

@probablyup https://github.com/probablyup What browser you are using?
I was trying on several chrome versions and each time Object.keys() was
faster.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack/tapable/pull/31#issuecomment-315750662, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAiy1rHuihy_KBpCUopDdmx0Xe3Mtj_Jks5sO10JgaJpZM4OBKuu
.

@probablyup Thank you for your comment.


How to run the project in Unity3d to get some visualizations like in your paper? Thanks

Property (_RadiationPatternMask) already exists. Use SetTexture instead.
can u help me? thanks

Hey,
Thank you very much for your interest in our framework. Could you please be a bit more specific on what you want to reconstruct? Is it a range plot over time or a Range-Doppler representation? You can find processing scripts written in matlab in the repository to estimate the range, velocity and angle of arrival of objects. You can directly use this scripts for plotting or adapt them for your purposes.
Hope that helps and cheers,
Christian

I don't understand how to get the simulate raw data and process it? Can u give me some advice? Thanks!

I guess you don't have a TCP/IP server running. Unity acts as a client and in order to receive or transmit data you need to setup a server. You can either do this in Python, Matlab or any other language you prefer.
You can also use the ROS Plugin instead to directly stream the data as a rostopic.

When I run this project in unity3d and simple_aoa_script.m in actave, I got below problems. Can u give me step-by-step advice? Thanks?

Up, I have the same problem


Subcomponents EG. <Form.Input />

Describe the problem
I may be missing the obvious but I cannot find a way to create sub components, is there something im missing or is it not a feature yet?
Describe the proposed solution
Have sub components in svelte
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

What do you mean? Like this? https://svelte.dev/repl/0e79b23dd3b74e56912bc3a9016f78c1?version=3.55.1

(not sure why you'd want to do that, but... you can)

Exactly what I was after, couldn’t find it online anywhere my bad, thank you


Docker mode doesn't work on Windows

Bug description
On Windows, process.cwd() includes a colon (:), which breaks the -v argument to the Docker CLI.
Status: Downloaded newer image for lostpixel/lost-pixel:v3.16.0
docker: Error response from daemon: invalid mode: \src\myproject.
See 'docker run --help'.
❌ Error: Command failed with exit code 125: docker run --rm -v D:\src\myproject:D:\src\myproject -e WORKSPACE=D:\src\myproject -e DOCKER=1     lostpixel/lost-pixel:v3.16.0
    at makeError (D:\src\myproject\node_modules\execa\lib\error.js:60:11)
    at handlePromise (D:\src\myproject\node_modules\execa\index.js:118:26)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async runInDocker (D:\src\myproject\node_modules\lost-pixel\dist\docker-runner\index.js:12:13)
    at async D:\src\myproject\node_modules\lost-pixel\dist\bin.js:32:9 {

Here is the relevant line in source code:
https://github.com/lost-pixel/lost-pixel/blob/6db31948aef97f10ea9c7b1a8fa8dd3a0ce2dd49/src/docker-runner/utils.ts#L23C5-L23C44
How to reproduce

Initialize latest Lost Pixel on Windows in OSS mode for any project (I used Storybook bindings)
Run npx lost-pixel docker
Observe the error message quoted above

Expected behavior
Docker mode generates the images and diff them as expected
Lost Pixel information
N/A
lost-pixel logs from CI
N/A

Thanks for reporting @dmitrc! mind creating a PR to fix this?

I can try to find time around the end of the week, but if you can get to it sooner, please be my guest.
From my brief research, the first value (before :) can stay as-is and the second one can be changed to be some arbitrary Linux path (eg /usr/workdir). Not entirely sure if that will cause cascading effects that would need to be updated accordingly.


bug: Swiping in menu while page inits breaks touch input

Bug Report
Version: "@ionic/angular": "4.3.0"
Current behavior:
On loading a complex ionic app on a slow phone and swiping from left to right while the page loads, the following error is printed (on the animation of the menu):
zone.js:682 Unhandled Promise rejection: Cannot read property 'offsetWidth' of undefined ; Zone: <root> ; Task: Promise.then ; Value: TypeError: Cannot read property 'offsetWidth' of undefined
    at Menu.loadAnimation (1hlqf399.entry.js:144)
    at Menu.onWillStart (1hlqf399.entry.js:184)
    at onWillStart (1hlqf399.entry.js:76)
    at tryToCapturePan (chunk-f56eaea8.js:483)
    at HTMLDocument.pointerMove (chunk-f56eaea8.js:459) TypeError: Cannot read property 'offsetWidth' of undefined
    at Menu.loadAnimation (http://<IP_ADDRESS>:8100/11.js:162:40)
    at Menu.onWillStart (http://<IP_ADDRESS>:8100/11.js:202:21)
    at onWillStart (http://<IP_ADDRESS>:8100/11.js:94:37)
    at tryToCapturePan (http://<IP_ADDRESS>:8100/0.js:497:13)
    at HTMLDocument.pointerMove (http://<IP_ADDRESS>:8100/0.js:473:38)

The code where the error happens is this code:
  async loadAnimation() {
        const width = this.menuInnerEl.offsetWidth; <- ! undefined error !
        if (width === this.width && this.animation !== undefined) {
            return;
        ....

After this exception it is no longer possible to tap anything.
Any following input will get the console message
chunk-f56eaea8.js:252 mousedown event dropped because of previous touch
Expected behavior:
The case of "menuInnerEl === undefined" should be handeled for strange timing errors like this and not create an error, thereby not blocking user input.

Hi there,
Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/17948.
Thanks for using Ionic!


Add instructions on how to compile the compiler from its source code

This was really useful, I wish it was added to the README of this repository.

with the current commit, stack init does not work. Thankfully, eh on the elm slack provided my with a custom stack.yaml that works:
resolver: lts-13.27
projects:
   - '.'
extra-deps:
   - containers-<IP_ADDRESS>@sha256:28ad7337057442f75bc689315ab4ec7bdf5e6b2c39668f306672cecd82c02798
   - http-client-0.6.4@sha256:a74b417345857d4250bdb6d4ac7fd4a89fd2ad8efab3164c1a2b843398d5c13a
   - language-glsl-0.3.0@sha256:85c1e7bf2cf5d6e604b7a2899c27e2935033425944db200798e57849e64d4c81
   - network-<IP_ADDRESS>@sha256:1ca79d81af02d7acd6032d5e6c9bde4618a8fdcfbe19bd42b49d420183975df0
   - time-1.9.3@sha256:8f1b5448722a12a952248b356c9eb366e351226543d9086a2da71270522d5f45
   - binary-<IP_ADDRESS>@sha256:ae3e6cca723ac55c54bbb3fa771bcf18142bc727afd57818e66d6ee6c8044f12,7705
   - directory-<IP_ADDRESS>@sha256:500019f04494324d1df16cf83eefeb3f809b2b20b32a32ccd755ee0439c18bfd,2829
   - process-<IP_ADDRESS>@sha256:77a9afeb676357f67fe5cf1ad79aca0745fb6f7fb96b786d510af08f622643f6,2468
   - unix-<IP_ADDRESS>@sha256:e69269a17b9fa26cb77f3f55e86c39d0a4940ccfa0c4bc20826919d2572076ad,3496

https://github.com/elm/compiler/issues/1899#issuecomment-467889877

Tracking in https://github.com/elm/compiler/issues/1949


Is there any shift right/left operation's implementation for typed_data Int32x4 on dart?

This tracker is for issues related to:

Dart core libraries dart:typed_data

I'm working on some crypto algorithms using SIMD on Dart. And I need to shift right all the lanes in Int32x4.
I have read the API Docs, and I have found the and/xor/or bit-wise operaion, but no shiftr/shiftl operaion.
Maybe I can implentment it as follwing:
import 'dart:typed_data';

void _run10000000TimesXorAndShuffle(int x, int y, int z, int w) {
  var ope = Int32x4(x, y, z, w);
  var ope2 = Int32x4(y, z, w, x);
  for (var i = 0; i < 10000000; ++i) {
    var c = ope ^ ope2;
    c.hashCode;
  }
}

void _run10000000TimesShiftRightAndShuffle(int x, int y, int z, int w) {
  var ope = Int32x4(x, y, z, w);
  for (var i = 0; i < 10000000; ++i) {
    var x = ope.x;
    var y = ope.y;
    var z = ope.z;
    var w = ope.w;
    var res = Int32x4(x << 1, y << 1, z << 1, w << 1);
    res.hashCode;
  }
}

void main() {
  var start = DateTime.now().millisecondsSinceEpoch;
  _run10000000TimesXorAndShuffle(0x918a57a0, 0x2095b753, 0x2041660, 0xe0e98a15);
  var end = DateTime.now().millisecondsSinceEpoch;
  print(end - start);
  start = DateTime.now().millisecondsSinceEpoch;
  _run10000000TimesShiftRightAndShuffle(
      0x918a57a0, 0x2095b753, 0x2041660, 0xe0e98a15);
  end = DateTime.now().millisecondsSinceEpoch;
  print(end - start);
}
// 30
// 7660

But it seems not efficient as the xor operation. So is there any other way to implement shiftr/shiftl operation on Int32x4, or will dart API add such method?

There is currently no shift operations on Int32x4.
If all you need is to do a left-shift by one, you can probably get around it as var res = ope + ope;.
Right shift, as you ask for, is much harder to simulate.

So will Dart SDK add such operators in the future? I think this is important for mathematical calculations using SIMD on Dart.


can't find the /bgp and /v1 in the etcd store

I read the confd template. It use many etcd keys like
prefix = "/calico"
keys = [
    "/v1/ipam/v4/pool",
    "/bgp/v1/host//NODENAME",
    "/staticroutes",
    "/rejectcidrs",
]

But when I use etcdctl show the keys I can't find any keys startwith like /calico/v1 and /calico/bgp
but the confd was render the configuration success! It seem the keys is exists in the etcd.
I read the document here it show that /calico/bgp and /calico/v1 is in the data model but I can't find it. Both use etcd v2 and v3 api.
I can only find the key what start with /calico/resources and /calico/ipam/
Is any thing I misunderstood? Thank you.
Expected Behavior

Should find the key like /calico/bgp and /calico/v1
Current Behavior

Can't find /calico/bgp and /calico/v1
Your Environment

libcalico-go version: 3.7.2
etcd version: 3.3.13

I think that document you linked to is the structure used in Calico v2.x, with the change to the v3 data model that document is only correct for the places where we still use the v1 data model, like in confd (I believe it is the syncer that converts our v3 data into the v1 data model).
We might need to just change the description of that document that it is the v1 data model and is not representative of how the data is written to etcd.

But I want to know why confd can use the key start with  /calico/v1 and /calico/bgp  not exist in the etcd, but It can render the template success!  I remeber that confd is direct get the key from the etcd and replace it with value in the template. I read the source code it seems you don't change the logic in the confd.

My understanding is that confd does not read from etcd directly but now only goes through libcalico-go which is responsible for a translation between the v1 and v3 data models. So the keys confd uses are still referencing the v1 data model and libcalico-go converts them to the v3 model.
@caseydavenport or @neiljerram, could one of you confirm my understanding here. Thanks

@tmjd Yes, that is right.

@konghui Our fork of confd has now diverged a lot from the upstream.  Whereas the upstream (kelseyhightower) confd performs a straightforward mapping from etcd content to the template files, I'm afraid our fork is now much more complex than that.
Do you have any more specific questions that we could help with?

Ok I get it .Thank you very much.


Question about the location vocabulary

OFA is an awesome job, and you maintain it meticulously! Thanks!
There is a little detail in the paper that I don't understand well. The problem is about Location vocab. Both text vocab and image vocab in OFA are pre-defined by word embedding or VQ-GAN. My question is, is location vocab pre-defined? I read the paper without finding a clear explanation. According to pixel2seq, each bin should be obtained by quantizing the input.
However, in OFA, image, text, and location vocabs are relegated to a uniﬁed vocabulary, which means they are all pre-defined. This confuses me about how the location vocab works.
Could you further explain how location vocab works? It may include the input and output work with location vocab. This will be very helpful to us.
Thank you again and look forward to your reply!   : )

@zzhanghub Thanks for your attention to OFA!
Yes, location vocab is pre-defined. We define a unified vocabulary in ofa_task.py. The corresponding code snippet is shown below, where <bin_0> - <bin_999> represent the location tokens.

When processing the input or output coordinates, we following pix2seq to quantinize the original coordinates into discretized values, these values will be mapped to the corresponding <bin_x>. The code snippet is shown below:

I have a related location question. I'm using the HF version of OFA and it's working well for captioning but I'm just trying regions and getting errors. When I try a visual grounding I get the following kinds of output.
"tensor([[    0, 58759, 58897, 58884, 58977,     2]])"
I assume that these tokens are the special bin tokens mentioned above. HF batch_decode fails with these tokens which I assume because it doesn't have them in the vocabulary. Do you have a working example of these being decoded correctly from HF?

 self.tokenizer.add_tokens("<mask>")
        for i in range(8192):
            self.tokenizer.add_tokens(f"<code_{i}>")
        for i in range(1000):
            self.tokenizer.add_tokens(f"<bin_{i}>")

Following on my above comments I tried reading the tokens to the HF tokenizer and per the above code and it now works.

 self.tokenizer.add_tokens("<mask>")
        for i in range(8192):
            self.tokenizer.add_tokens(f"<code_{i}>")
        for i in range(1000):
            self.tokenizer.add_tokens(f"<bin_{i}>")

Following on my above comments I tried reading the tokens to the HF tokenizer and per the above code and it now works.

Thanks for your comments!
Yes, in the previous version,  you could manually add these words in the way you commented. In the latest version, we have updated the ofa tokenizer, so there is no need to manually add these tokens.


Incorrect TOML parsing of multi-line literal strings for pyproject.toml

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
pip
Description
Apparently Dependabot is using the very outdated and poorly supported toml package to parse the pyproject.toml for some reason. It seems the people that wrote the Python helpers were aware that TOML support was included in Python 3.11, yet still use that third-party package here:
https://github.com/dependabot/dependabot-core/blob/7835cbdad710c359c689a22021d5daab77d09b3c/python/helpers/lib/parser.py#L17-L19
This causes a very ugly and hard to decipher error, if you are unlucky enough (as I was) to have a multi-line string literal in your pyproject.toml with certain characters that trip up the parser. I have not looked into the implementation of that toml package, so I don't know exactly what the cause of the error is. But it was frustrating enough to try and figure out, what caused Depenadabot to fail. Turns out, it was this array:
exclude_also = [
    '''if __name__ == ['"]__main__['"]:''',
]

Click here to see the full TOML file
[build-system]
requires = ["setuptools", "setuptools-scm"]
build-backend = "setuptools.build_meta"

[project]
name = "dependabot-pyproject-toml-error"
description = '''foo'''
version = "0.0.1"
requires-python = ">=3.12"
license = { text = "GNU General Public License v3 or later (GPLv3+)" }
classifiers = [
    '''Development Status :: 4 - Beta''',
    "Programming Language :: Python :: 3 :: Only",
]
dependencies = [
    '''pydantic==2.7.0''',
]

[tool.coverage.report]
exclude_also = [
    '''if __name__ == ['"]__main__['"]:''',
]

There are a few experiments with triple quotes here to figure out, what worked and what didn't.

Here is the traceback from the Dependabot error log:
Traceback (most recent call last):
  File "/opt/python/run.py", line 16, in <module>
    print(parser.parse_pep621_dependencies(args["args"][0]))
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/python/lib/parser.py", line 27, in parse_pep621_dependencies
    project_toml = toml.load(pyproject_path)['project']
                   ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/.pyenv/versions/3.12.5/lib/python3.12/site-packages/toml/decoder.py", line 134, in load
    return loads(ffile.read(), _dict, decoder)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/.pyenv/versions/3.12.5/lib/python3.12/site-packages/toml/decoder.py", line 213, in loads
    raise TomlDecodeError("Key name found without value."
toml.decoder.TomlDecodeError: Key name found without value. Reached end of line. (line 22 column 2 char 534)

See the full log here.
To reproduce, you can fork this minimal repo: daniil-berg/dependabot-pyproject-toml-error
Trigger Dependabot checks, see the error.
Remove the lines mentioned above from pyproject.toml, trigger Dependabot again, see it work without errors.
Needless to say, this is a bug because that string literal is valid TOML and meaningful for regular expressions etc.

It is surprising to me that the built-in tomllib is not used, since Dependabot already uses Python 3.12 by default.
If there are ways to use Dependabot with an older Version of Python, (first of all, please let me know how you can do that) then you could still use a package like tomli, which is faster, up-to-date, and has no issues parsing the pyproject.toml shown above. It works with Python >=3.8 and is actually the basis for the built-in library in Python >=3.11.
Maybe I'll find the time this weekend to provide PR for this. But since I am not sure, I wanted to leave this issue here.
dependabot.yml content
version: 2
updates:
  - package-ecosystem: 'pip'
    directory: '/'
    schedule:
      interval: 'daily'
      time: '12:00'
      timezone: 'Europe/Berlin'

What you expected to see, versus what you actually saw
Expected no errors.
Actually saw this.
Smallest manifest that reproduces the issue
# pyproject.toml

[build-system]
requires = ["setuptools", "setuptools-scm"]
build-backend = "setuptools.build_meta"

[project]
name = "dependabot-pyproject-toml-error"
description = '''foo'''
version = "0.0.1"
requires-python = ">=3.12"
license = { text = "GNU General Public License v3 or later (GPLv3+)" }
classifiers = [
    '''Development Status :: 4 - Beta''',
    "Programming Language :: Python :: 3 :: Only",
]
dependencies = [
    '''pydantic==2.7.0''',
]

[tool.coverage.report]
exclude_also = [
    '''if __name__ == ['"]__main__['"]:''',
]

Hello! I believe I'm hitting the same issue in my pyproject.toml.
The error I get is: /pyproject.toml not parseable
I'm even more of a ruby user than @daniil-berg, but since Python using tomllib not having any issues, I thought adding mine as a possible test case might help.

We also have problem with parsing

/pyproject.toml not parseable

The file updated automatically by poetry.

this seems like the same issue that im having. this is the error log that im getting:
updater | 2024/11/18 15:49:26 ERROR <job_918477550> Error processing opentelemetry-exporter-otlp-proto-grpc (TomlRB::ParseError)
2024/11/18 15:49:26 ERROR <job_918477550> Failed to parse input on line 169 at offset 1
 = "."

 ^

the only place that i have this in my toml file is the following line:
[tool.setuptools]
package-dir = { "" = "." }

does anyone know how to confirm if this is the same issue? if it is, it could be used as a test case for #10540


gui: installer: convey registering on a signing device isn't always required

As discussed in #545, in some cases we nudge the user to register the descriptor on their signing device although they might not have one. Those cases only ever arise when importing a descriptor (either when recovering from backup or participating in the creation of a descriptor on another laptop), since when the descriptor is created beforehand we can simply detect whether a signing device was used and thereby needs to be registered (implemented since https://github.com/wizardsardine/liana/pull/470).
Therefore, detect when the registration step arises as part of an import process and if so adjust the language to convey registration on a signing device may not be necessary.
Result:

Fixes #545.

In the future we could ask them beforehand whether they'll be using a signing device and only show them this step if so. But until we do that it's a minimal patch for the current misleading behaviour.

When importing the descriptor, user needs to store again the ledger HMAC. I am ok to make the step more easy to skip for the import descriptor process, but user with ledger will have to do it.

I don't understand your comment in the context of this PR?


Justified text is an accessibility issue

Description
And generally falling out of favor for readability reasons. https://www.boia.org/blog/why-justified-or-centered-text-is-bad-for-accessibility
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Formula
Internal issue ID
8f56bb9a-b557-47da-bd8f-f666a30c34e7
Paper URL
https://browse.arxiv.org/html/2312.12451v1
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

I think an important detail here is why it is an accessibility issue. As that link documents:

If text is justified, these users may see large, empty spaces between (or within) words — or the words may run together in a confusing mess of characters.

So one key question is testing if that problem is encountered over the exact CSS setup on arXiv. There are some provisions in place to help avoid that (e.g. using hyphens: auto; with a lang attribute set)


Display an input callout when long-pressing buttons

Similar to what Apple does when pressing the "n":

This can also be done by the library described in #92, CYRKeyboardButton.
Something super cool, that that library does not support out of the box, would be to show a menu in the expanded key cap view like how Gboard does when long-pressing their settings/switch keyboard button:

A simple way of showing a menu on long-press is to use a context menu. Let's say you have a custom action type of .settings like this:
extension KeyboardAction {
    static var settings: KeyboardAction {
        .custom(name: "Settings")
    }
}

Then in your keyboard view controller you can do:
    override func addKeyboardGestures(to button: KeyboardButton) {
        super.addKeyboardGestures(to: button)

        if case .settings = button.action {
            button.addInteraction(UIContextMenuInteraction(delegate: self))
        }
    }

extension KeyboardViewController: UIContextMenuDelegate {
    func contextMenuInteraction(
        _ interaction: UIContextMenuInteraction,
        configurationForMenuAtLocation location: CGPoint) -> UIContextMenuConfiguration? {
        UIContextMenuConfiguration(identifier: nil,
                                   previewProvider: nil,
                                   actionProvider: { _ in
                                    UIMenu(title: "",
                                           image: nil,
                                           identifier: nil,
                                           children: [UIAction(title: "Settings",
                                                               image: UIImage(systemName: "gear")) { _ in
                                                                print("Open Settings")
                                            }, UIAction(title: "Next Keyboard",
                                                        image: UIImage(systemName: "globe")) { [weak self] _ in
                                                            self?.advanceToNextInputMode()
                                            }])
        })
    }
}

which produces the following on long-press:

As I mention in #92, I have work in-progress here 🚀

What you show there looks extremely promising!

These kinds of components, like the callout container view, are something I think the library should provide to SwiftUI-based keyboards!

I think we should aim at implementing this in 3.1 or 3.2.

I have finally managed to put together a first implementation of a secondary input callout! See the screenshot below 🥳
I have also created a start set for secondary input actions, which currently supports English and Swedish, but can be easily extended with more locales.
I have merged this master and added it to the SwiftUI SystemKeyboard. Feel free to check it out and give it a good try. It still has some rough edges, but I'm really happy with the result so far.

I have finally managed to put together a first implementation of a secondary input callout! See the screenshot below 🥳
I have also created a start set for secondary input actions, which currently supports English and Swedish, but can be easily extended with more locales.
I have merged this master and added it to the SwiftUI SystemKeyboard. Feel free to check it out and give it a good try. It still has some rough edges, but I'm really happy with the result so far.

I have just released KeyboardKit 3.4, which has SwiftUI support for this.
I would love some extra eyes in the result and would love to improve the overall look and feel over time.

I have just released KeyboardKit 3.4, which has SwiftUI support for this.
I would love some extra eyes in the result and would love to improve the overall look and feel over time.


Update symbol_table

Brings symbol_table to the same version of hashbrown as egglog and has multi-threaded performance improvements. Follows up on PR#445 Update hashbrown.

Huh. cykjson seems to be very sensitive to hashing.

We saw regression on cykjson, but we think it is fine because this is just a dependency change and seems unavoidable. Also for context we are also considering removing symbol_table.


[Migration] Enable SCIM2 Group based User Filtering Improvements by Default

Purpose

The following SCIM2 group based user filtering improvements were done with a config to enable/ disable the improvements.

The SCIM 2 user endpoint with a groups filter returns the totalResults based on the total user count corresponding to the given group-based filter.
When invoking the user endpoint with a groups filter by passing a role (hybrid or system roles), it returns an empty list when role group separation is enabled, instead of returning users linked to the passed role.

With the following PR, these improvements are enabled by default in the product.

https://github.com/wso2/carbon-identity-framework/pull/5830

Therefore, these improvements should be noted in migration docs of future releases.

Related Issues

https://github.com/wso2/product-is/issues/12813

Related PRs

https://github.com/wso2/carbon-identity-framework/pull/5533
https://github.com/wso2-extensions/identity-inbound-provisioning-scim2/pull/547
https://github.com/wso2/carbon-kernel/pull/3906
https://github.com/wso2/carbon-kernel/pull/3951

Updated the migration doc


White/Black List Filtering and Multiple Registries Refactor

This is obviously pretty big, I'm going to try to follow this up with a test plan and try to point out the stuff that needs review attention. It's a little misleading in +/- because reorg'd a bunch of files into different packages.

Spoke with @rthallisey and @jmrodri re: apb-filter-design.md. Ultimately, it was supposed to be an implementation proposal that didn't belong in docs/ and fell very much out of date with that actual implementation. Plan is to remove the file since it's entirely inaccurate and follow up this PR with proper usage documentation.

@eriknelson ack from me once my comments are addressed

 (8) Multiple Registry Adapters configured in one Broker Instance

 (8) Multiple Registry Adapters configured in one Broker Instance


Better memory management and explicit mapping between imported GLTF MinFilter and Unity FilterMode.

[Edit: Apologies for the overlapping changes in this pull request. I assumed it would be a snapshot of the forks master branch rather than a live reference. I will use explicit branches for each PR in future.]
Improved disposal of temporary streams during import, particularly for non-binary files where multiple MemoryStreams are created but never explicitly disposed. Repeated loading large files, such as this one, causes my Unity Android app to crash very quickly.
A closer match for Mipmap variants of nearest point sampling. Although MinFilterMode.Nearest is supported, the MipMap versions are not. Here is an example that uses NearestMipmapNearest.

Thanks for your contribution! Content looks good, but could you match the project white space consensus? There are a lot of otherwise unchanged code segments that were reindented.

The IDisposable changes are great and the stream protection. The way that we handle loading data recently changed, so I'll make sure to take a look to make sure that we handle stream lifecycle correctly there.
I'll work on getting your changes checked in.

@stevenvergenz Sorry about the whitespace. I thought I had caught all the style problems before check in and I'll be a bit more diligent next time.
@blgrossMS Thanks for your help. I'm being a bit fussy about applying the pattern in this instance as the garbage collector will deal will all of this stuff eventually, but I've been chasing a memory leak and this was the first thing I thought to check.

Thanks for the PR, sorry it took so long to integrate


Memory issue

Hi,
I noticed that all the viewControllers are created at once. this can be a memory issue. Is it possibile to pre-load only 1 viewController per side?
Thanks

Currently there is no support for this. The reason is because the controllers are used to get the tab information (with the protocol IndicatorInfoProvider with the method that returns the IndicatorInfo object), so they all have to be created.
Hope it helps! Regards,

Thanks for your answer, this is actually a problem for me since I have several pages with a lot of images inside, so I need to recycle as much as I can. I was thinking about using a collectionView with the pagerTab on top, is there a way to pass the scroll to the pagerTab?
Thanks!

@johntiror Actually the views are not loaded, only the controllers. You could load the memory hungry items at a later stage (after viewDidLoad()), or some other technique..


support retrieving helm charts from private helm registry using a dockerconfigjson secret

We use a mechanism similar to that used by kubernetes code to pull docker images:
https://github.com/kubernetes/kubernetes/blob/ea0764452222146c47ec826977f49d7001b0ea8c/pkg/kubelet/kuberuntime/kuberuntime_image.go#L31
There are problems using k8s.io/kubernetes as a dependency.
One solution is to use https://github.com/vdemeester/k8s-pkg-credentialprovider , which is a "k8s.io/kubernetes/pkg/credentialprovider temporary fork"
Instead, I used a workaround suggested here:
https://stackoverflow.com/questions/59187781/revision-v0-0-0-unknown-for-go-get-k8s-io-kubernetes

This implementation seems costly in terms of the work done on each reconcile. How about adding an explicit field for the secret next to the chart field in FybrikModule? That way you would know that login is required and what secret to use (the keyring code is still required in case there are multiple credentials in the secret).
As for dependencies, using https://github.com/vdemeester/k8s-pkg-credentialprovider is fine by me if we can't easily extract just what we need.


Add more input audio formats

Add more audio input formats reported by users. Let's keep this open for a bit to collect feedback.
Thanks @zacwest to get things started.

As the information required to troubleshoot this is contained in the value of soundbar_audio_input_format, I've also reduced the log severity from warning to debug to make this less noisy.

This value also feels like it should be a bitmask, but I haven't found a pattern to decode it quite yet.

I'm starting to wonder if these bitmasks contain information besides just the audio codec, like the transport (HDMI/optical), speaker layout, etc.

If it would contain information about the layout, I'd expect that the values would vary among users with and without sub, no? But it definitely has some sort of pattern, I think it'd be great to get some more input (for atmos, dts) to see if that simplifies the puzzle :)

LGTM -- thanks. Curious to understand the model Sonos is using to determine the codes.


Playwright E2E runner step needs to setup CI env. variable to create junit test results file

Podman Desktop playwright tests need to have set env. var. CI in order to produce junit xml file. When the workflow is run, env. vars are not propagated into host (windows) machine. We need to do it manually.

Fixed by: https://github.com/odockal/pde2e-runner/commit/09c8b17ddca67b94c58eb9ac525bfc4125b276e7.


Zocalo issues on v4.0.0

In testing v4.0.0 we found that zocalo returned no results for https://ispyb.diamond.ac.uk/dc/visit/cm33866-2/dcg/9256812
Acceptance Criteria

We work out what went wrong
We have a good idea of how to diagnose these kinds of issues in the future

Partially addressed by https://github.com/DiamondLightSource/python-dlstbx/pull/210


Automatically link coverage reports in PR description

We currently have automatically generated code coverage reports created when unit tests are run, but these coverage reports are difficult to find and access. This PR updates the circleci workflow so that it automatically updates the PR description with a link to the latest coverage reports for each unit testing workflow.
Resolves #461
Coverage reports (updated automatically):

test_unit: 61%

Confirmed initial report shows up as expected, re-launched the tests to see if the report link is updated correctly when the report is already linked. Current link is to this report.

The PR "works" but I'm having trouble getting the way newline characters are represented to be consistent between when I run the script locally and on CI. I may try refactoring the script so it uses gh pr view and extracts the PR body, instead of using curl (which seems to modify special characters).

Reading the coverage report itself I made another issue #1880

Follow-on issue should be fixed by #1882.

Updated, deleted the placeholder to check it works.

Worked with no placeholder. Double-checking the with-placeholder case also still works, since I edited that if-clause as well.

Worked. Auto-merge should complete in 10-20 minutes when the SSH session I started closes.


Update nf-fileapi-getvolumepathnamew.md

Fix backslash notation

@hsebs : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hsebs : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Use lambda functions for sequencer message queue subscribers

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lambda_event_source_mapping
This will help us reach unlimited scale.

out of scope


LSP error about brace closing

in code from example
#include <interface99.h>

#include <stdio.h>

#define Shape_IFACE                      \
    vfunc( int, perim, const VSelf)      \
    vfunc(void, scale, VSelf, int factor)

interface(Shape);

clangd version 15.0.6 throws error:
Expected ')'

but when just compiling it with clang there is no errors

i investigated deeper and find out that problem was in restrict
clangd complaining only about bar:
void foo(void *__restrict x);
void bar(void *restrict x);

also gnu libc and musl libc uses __restrict instead of restrict
maybe you also can use __restrict here?
https://github.com/Hirrolot/interface99/blob/c81db7f76c70cd4a7fb1a55ba783dde47ecd93a1/interface99.h#L81

with this .clangd config everything works fine:
 CompileFlags:
   Add: [-std=c99, -xc]


(Bug) UserStorage

seems like a problem while init GoodWallet need to investigate this

@sirpy @AlexeyKosinski
I can't find any reported event related to the UserStorage issue in bugsnag. What should we do with this task?

you have a searchbox the above issue is taken from bugsnag so its there

I found the reported errors with UserStorage initialization.
There are two error message I found why the userStorage failed to initialize:

Wrong user or password
It is the issue with GoodWallet sign. I can't understand yet why there were a few situations for users when the password or user is wrong.

User is already being created or authenticated!
I look at the code and found some strange peaces (see screenshot):

why existingUsername (1 arrow on screenshot) is always false? then the condition will always go to the else block and we always create a new user
this.gunCreate (2 arrow on screenshot) cannot trigger error as creating users with the same username is allowed and not trigger an error. So the error occurred in this.gunAuth.

Considering all these things I think the problem might be with this.gunuser.leave() as the error tells that user is already authorized. Might be the logout is not executed successfully and we do not verify it. I found the note/recommendation from gundb author (https://gun.eco/docs/User#user-leave) to check if user is logger out by verifying 'user._.sea' exists. So we can try to add this thing somehow in our code.

@yaroslav-fedyshyn-nordwhale i've made a few changes to optimize(make it faster) login to gun. in another PR. #1361
so it bypasses the user/password auth which is slow because of pbkdf brute force prevention.
lets put this back in backlog and follow such errors in next releases

@yaroslav-fedyshyn-nordwhale
I've checked this on bugsnag and it seems to be only happening at /Auth/Recover
in userstorage constructor log.warn user/pass error and log.error other exceptions

Checked on the eWallet ver 0.16 for some time - from 03.03 till 11.03
The issue didn't reproduced


Debug and sensitive

Codecov Report

Merging #5 into master will decrease coverage by 0.99%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
- Coverage     89.67%   88.67%   -1%
+ Complexity      180      169   -11
=========================================
  Files            11       11
  Lines           426      415   -11
  Branches         43       43
=========================================
- Hits            382      368   -14
- Misses           28       29    +1
- Partials         16       18    +2

Impacted Files
Coverage Δ
Complexity Δ

...tlog-core/src/main/java/org/smartlog/SmartLog.java
89.06% <ø> (-0.65%)
31 <0> (-4)

...ore/src/main/java/org/smartlog/SmartLogConfig.java
70% <ø> (-5%)
8 <0> (-2)

...og-core/src/main/java/org/smartlog/LogContext.java
88.57% <100%> (-2.34%)
69 <0> (-5)

...-aop/src/main/java/org/smartlog/aop/LogAspect.java
76.92% <100%> (ø)
10 <0> (ø)
: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 8586e52...d29a683. Read the comment docs.


Store annotation data

stores annotation data in tables:

question-answer pairs
user reactions
user feedback
retrieved papers and scores

This adds a feature to #1


Fix for #3243

/azp run

/azp run

/azp run

@to-bar Good catches, copy+pasting at the end of the day is always a bad idear;)

/azp run

/azp run


Q: pws

I want you to act as a linux terminal. I will type commands and you will reply with what the terminal should show. I want you to only reply with the terminal output inside one unique code block, and nothing else. do not write explanations. do not type commands unless I instruct you to do so. When I need to tell you something in English, I will do so by putting text inside curly brackets {like this}.

pwd

curl "https://httpbin.org/json"
curl "https://httpbin.org/xml"


Q: why attrs instead of Pydantic?

This is really just a question.
I realize that attrs can be faster (with catters), but on the flip side Pydnatic is used by FastAPI, which IMO is currently the best way to make application backends in Python.

I have not use pydantic and when I started this project attrs was the stable thing and pydantic was still the new hotness. I haven't been given an overwhelming reason why one is better than the other. The ORM side of things does not require type validation (because data is typed coming from EdgeDB), which is often not case when deserializing from an API.  I'm not sure they're incompatible either.

Agreed on data validation not being necessary when coming from a DB. Pydantic has the ability to construct models that have validation without validating. So you can have data coming from the DB create objects without validation and return those directly to clients via API while using the same ORM to deserialize and validate incoming JSON.

I'm not sure they're incompatible either

This is an interesting idea I had not thought of. It is quite possible that you can dump an attrs object into a Pydantic Model, however that does still mean that you need to separately define your Pydantic Model. You wouldn't be able to make the DB schema the single source of truth.

One of the ideas I'm most pleased with is using EdgeDB's schema to define python classes/models. The dump attrs-into-pydantic path doesn't excite me or probably anybody else.
I think it would be possible to check for an attrs/pydantic flag. I imagine the inheritance pieces would be very similar so you could add class methods with the same decorator.

I think it would be possible to check for an attrs/pydantic flag

I think that would be very nice. I'm sure there are some advantages to attrs that I am unaware of. I haven't peeked at your internals so I don't know how feasible this is to implement, but if it is feasible, it would be very nice and would make EdgeDB play very nicely with FastAPI.

The majority of the work would take place here:
https://github.com/agritheory/edgewise/blob/testing/edgewise/registry.py
A flag would need to be added to merge_class and then do the pydantic version of _build_attributes. I don't know pydantic well enough to know if there should be versions of the scalar pieces as well. I do think that it makes sense to tackle #6 first so that we're working with JSON output from EdgeDB rather than some of the internal edgedb-python classes which I perceive are a little bit of a moving target.

That's interesting. Incidentally, Pydantic supports parsing from
deserialized JSON and even raw JSON strings.
On Wed, Dec 30, 2020, 12:43 PM Tyler Matteson<EMAIL_ADDRESS>wrote:

The majority of the work would take place here:
https://github.com/agritheory/edgewise/blob/testing/edgewise/registry.py
A flag would need to be added to merge_class and then do the pydantic
version of _build_attributes. I don't know pydantic well enough to know
if there should be versions of the scalar pieces as well. I do think
that it makes sense to tackle #6
https://github.com/agritheory/edgewise/issues/6 first so that we're
working with JSON output from EdgeDB rather than some of the internal
edgedb-python classes which I perceive are a little bit of a moving target.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/agritheory/edgewise/issues/12#issuecomment-752720208,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMPP6MDSNRWFVVYUNNM63SXNYFFANCNFSM4VN6ZPSA
.

In the context of the registry, its less about the serialization/deserialization problem than than correctly creating a model on the fly.
Here's what would need to implemented in the pydantic version of  _build_attributes.
https://pydantic-docs.helpmanual.io/usage/models/#dynamic-model-creation