Not compiling on Linux

Let me kick off by saying what a great framework Neo4j-swift is to work with. I got it working quite fast and didn't have issues at all. While I'm now ready to push code to production, I get errors during building of my docker image. I think it has something to do with Bolt, but can't find the origin. Maybe good to mention that I use it with Vapor 4. 🙆
Do you have an idea where it fails?
Build commando (on m1 chip):
docker buildx build . --platform linux/amd64 -t {source}:{tag} --push
Log errors:
[1369/1373] Compiling Bolt CertificateValidatorProtocol.swift
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/Connection.swift:82:21: warning: initialization of immutable value 'result' was never used; consider replacing with assignment to '_' or removing it
#0 1001.4                 let result = response.map { bytes -> Void in
#0 1001.4                 ~~~~^~~~~~
#0 1001.4                 _
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:47:49: warning: 'forClient(cipherSuites:minimumTLSVersion:maximumTLSVersion:certificateVerification:trustRoots:certificateChain:privateKey:applicationProtocols:shutdownTimeout:keyLogCallback:)' is deprecated: renamed to 'makeClientConfiguration()'
#0 1001.4         var tlsConfiguration = TLSConfiguration.forClient(trustRoots: trustRoot, certificateChain: cert, privateKey: nil)
#0 1001.4                                                 ^
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:47:49: note: use 'makeClientConfiguration()' instead
#0 1001.4         var tlsConfiguration = TLSConfiguration.forClient(trustRoots: trustRoot, certificateChain: cert, privateKey: nil)
#0 1001.4                                                 ^~~~~~~~~
#0 1001.4                                                 makeClientConfiguration
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:68:35: warning: 'init(context:serverHostname:verificationCallback:)' is deprecated: replaced by 'init(context:serverHostname:customVerificationCallback:)'
#0 1001.4         let openSslHandler = try! NIOSSLClientHandler(context: sslContext, serverHostname: self.hostname, verificationCallback: verificationCallback)
#0 1001.4                                   ^
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:68:35: note: use 'init(context:serverHostname:customVerificationCallback:)' instead
#0 1001.4         let openSslHandler = try! NIOSSLClientHandler(context: sslContext, serverHostname: self.hostname, verificationCallback: verificationCallback)
#0 1001.4                                   ^                                                                       ~~~~~~~~~~~~~~~~~~~~
#0 1001.4                                                                                customVerificationCallback
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/SHA1.swift:103:17: warning: using '_' to ignore the result of a Void-returning function is redundant
#0 1001.4                 _=data.copyBytes(to: dest, from: range)               // Retrieve one chunk
#0 1001.4                 ^~
#0 1001.4
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/SHA1.swift:113:13: warning: using '_' to ignore the result of a Void-returning function is redundant
#0 1001.4             _=data.copyBytes(to: dest, from: range)                   // Retrieve remainder
#0 1001.4             ^~
#0 1001.4
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/UnencryptedSocket.swift:97:19: warning: result of call to 'connectTimeout' is unused
#0 1001.4         bootstrap.connectTimeout(TimeAmount.milliseconds(Int64(timeout)))
#0 1001.4                   ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/UnencryptedSocket.swift:142:13: warning: initialization of immutable value 'c' was never used; consider replacing with assignment to '_' or removing it
#0 1001.4         let c = cnt
#0 1001.4         ~~~~^
#0 1001.4         _
#0 1002.1 remark: Incremental compilation has been disabled: it is not compatible with whole module optimization[1371/1375] Compiling Theo BoltClient.swift
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:56:50: error: cannot find 'TrustSpecificOrRootCertificateValidator' in scope
#0 1007.5                 encSocket.certificateValidator = TrustSpecificOrRootCertificateValidator(hostname: self.hostname, port: port, trustedCertificateAtPath: certPath)
#0 1007.5                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:58:50: error: cannot find 'TrustRootOnlyCertificateValidator' in scope
#0 1007.5                 encSocket.certificateValidator = TrustRootOnlyCertificateValidator(hostname: self.hostname, port: port)
#0 1007.5                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:333:34: warning: enum case 'success' has one associated value that is a tuple of 2 elements
#0 1007.5                 case let .success(isSuccess, parsedResponses):
#0 1007.5                                  ^
#0 1007.5                                  (                           )
#0 1007.5 <unknown>:0: note: 'success' declared here
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:506:25: warning: initialization of immutable value 'error' was never used; consider replacing with assignment to '_' or removing it
#0 1007.5                     let error = BoltClientError.queryUnsuccessful
#0 1007.5                     ~~~~^~~~~
#0 1007.5                     _
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/ClientProtocol.swift:5:33: warning: using 'class' keyword to define a class-constrained protocol is deprecated; use 'AnyObject' instead
#0 1007.5 public protocol ClientProtocol: class {
#0 1007.5                                 ^~~~~
#0 1007.5                                 AnyObject
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:382:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt8' always fails
#0 1007.5         } else if let l = lVal as? UInt8, let r = rVal as? UInt8 {
#0 1007.5                           ~~~~ ^   ~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:382:56: warning: cast from 'PackProtocol?' to unrelated type 'UInt8' always fails
#0 1007.5         } else if let l = lVal as? UInt8, let r = rVal as? UInt8 {
#0 1007.5                                                   ~~~~ ^   ~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:384:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt16' always fails
#0 1007.5         } else if let l = lVal as? UInt16, let r = rVal as? UInt16 {
#0 1007.5                           ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:384:57: warning: cast from 'PackProtocol?' to unrelated type 'UInt16' always fails
#0 1007.5         } else if let l = lVal as? UInt16, let r = rVal as? UInt16 {
#0 1007.5                                                    ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:386:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt32' always fails
#0 1007.5         } else if let l = lVal as? UInt32, let r = rVal as? UInt32 {
#0 1007.5                           ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:386:57: warning: cast from 'PackProtocol?' to unrelated type 'UInt32' always fails
#0 1007.5         } else if let l = lVal as? UInt32, let r = rVal as? UInt32 {
#0 1007.5                                                    ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:396:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt' always fails
#0 1007.5         } else if let l = lVal as? UInt, let r = rVal as? UInt {
#0 1007.5                           ~~~~ ^   ~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:396:55: warning: cast from 'PackProtocol?' to unrelated type 'UInt' always fails
#0 1007.5         } else if let l = lVal as? UInt, let r = rVal as? UInt {

Hi Niels,
Thanks for the kind words.
I haven’t built using Docker, but that sounds like a use case that absolutely should go into the test pipeline, so let’s get started on that. :-)
Could you provide me with a minimal little project, no real hostname/username/password, to get more or less where you are? A little GitHub repo that I could check out and run your docker command would be really nice :-)
Cheers
Niklas

On 28 Sep 2022, at 06.37, Niels Koole @.***> wrote:
Let me kick off by saying what a great framework Neo4j-swift is to work with. I got it working quite fast and didn't have issues at all. While I'm now ready to push code to production, I get errors during building of my docker image. I think it has something to do with Bolt, but can't find the origin.
Do you have an idea where it fails?
Build commando (on m1 chip):
docker buildx build . --platform linux/amd64 -t {source}:{tag} --push
Log errors:
[1369/1373] Compiling Bolt CertificateValidatorProtocol.swift
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/Connection.swift:82:21: warning: initialization of immutable value 'result' was never used; consider replacing with assignment to '' or removing it
#0 1001.4                 let result = response.map { bytes -> Void in
#0 1001.4                 ^~
#0 1001.4                 _
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:47:49: warning: 'forClient(cipherSuites:minimumTLSVersion:maximumTLSVersion:certificateVerification:trustRoots:certificateChain:privateKey:applicationProtocols:shutdownTimeout:keyLogCallback:)' is deprecated: renamed to 'makeClientConfiguration()'
#0 1001.4         var tlsConfiguration = TLSConfiguration.forClient(trustRoots: trustRoot, certificateChain: cert, privateKey: nil)
#0 1001.4                                                 ^
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:47:49: note: use 'makeClientConfiguration()' instead
#0 1001.4         var tlsConfiguration = TLSConfiguration.forClient(trustRoots: trustRoot, certificateChain: cert, privateKey: nil)
#0 1001.4                                                 ^~~~~~~~~
#0 1001.4                                                 makeClientConfiguration
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:68:35: warning: 'init(context:serverHostname:verificationCallback:)' is deprecated: replaced by 'init(context:serverHostname:customVerificationCallback:)'
#0 1001.4         let openSslHandler = try! NIOSSLClientHandler(context: sslContext, serverHostname: self.hostname, verificationCallback: verificationCallback)
#0 1001.4                                   ^
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/EncryptedSocket.swift:68:35: note: use 'init(context:serverHostname:customVerificationCallback:)' instead
#0 1001.4         let openSslHandler = try! NIOSSLClientHandler(context: sslContext, serverHostname: self.hostname, verificationCallback: verificationCallback)
#0 1001.4                                   ^                                                                       ~~~~~~~~~~~~~~~~~~~~
#0 1001.4                                                                                customVerificationCallback
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/SHA1.swift:103:17: warning: using '' to ignore the result of a Void-returning function is redundant
#0 1001.4                 =data.copyBytes(to: dest, from: range)               // Retrieve one chunk
#0 1001.4                 ^~
#0 1001.4
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/SHA1.swift:113:13: warning: using '' to ignore the result of a Void-returning function is redundant
#0 1001.4             =data.copyBytes(to: dest, from: range)                   // Retrieve remainder
#0 1001.4             ^~
#0 1001.4
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/UnencryptedSocket.swift:97:19: warning: result of call to 'connectTimeout' is unused
#0 1001.4         bootstrap.connectTimeout(TimeAmount.milliseconds(Int64(timeout)))
#0 1001.4                   ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1001.4 /build/.build/checkouts/Bolt-swift/Sources/UnencryptedSocket.swift:142:13: warning: initialization of immutable value 'c' was never used; consider replacing with assignment to '' or removing it
#0 1001.4         let c = cnt
#0 1001.4         ^
#0 1001.4         _
#0 1002.1 remark: Incremental compilation has been disabled: it is not compatible with whole module optimization[1371/1375] Compiling Theo BoltClient.swift
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:56:50: error: cannot find 'TrustSpecificOrRootCertificateValidator' in scope
#0 1007.5                 encSocket.certificateValidator = TrustSpecificOrRootCertificateValidator(hostname: self.hostname, port: port, trustedCertificateAtPath: certPath)
#0 1007.5                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:58:50: error: cannot find 'TrustRootOnlyCertificateValidator' in scope
#0 1007.5                 encSocket.certificateValidator = TrustRootOnlyCertificateValidator(hostname: self.hostname, port: port)
#0 1007.5                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:333:34: warning: enum case 'success' has one associated value that is a tuple of 2 elements
#0 1007.5                 case let .success(isSuccess, parsedResponses):
#0 1007.5                                  ^
#0 1007.5                                  (                           )
#0 1007.5 :0: note: 'success' declared here
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/BoltClient.swift:506:25: warning: initialization of immutable value 'error' was never used; consider replacing with assignment to '_' or removing it
#0 1007.5                     let error = BoltClientError.queryUnsuccessful
#0 1007.5                     ^
#0 1007.5                     _
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/ClientProtocol.swift:5:33: warning: using 'class' keyword to define a class-constrained protocol is deprecated; use 'AnyObject' instead
#0 1007.5 public protocol ClientProtocol: class {
#0 1007.5                                 ^~~~~
#0 1007.5                                 AnyObject
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:382:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt8' always fails
#0 1007.5         } else if let l = lVal as? UInt8, let r = rVal as? UInt8 {
#0 1007.5                           ~~~~ ^   ~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:382:56: warning: cast from 'PackProtocol?' to unrelated type 'UInt8' always fails
#0 1007.5         } else if let l = lVal as? UInt8, let r = rVal as? UInt8 {
#0 1007.5                                                   ~~~~ ^   ~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:384:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt16' always fails
#0 1007.5         } else if let l = lVal as? UInt16, let r = rVal as? UInt16 {
#0 1007.5                           ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:384:57: warning: cast from 'PackProtocol?' to unrelated type 'UInt16' always fails
#0 1007.5         } else if let l = lVal as? UInt16, let r = rVal as? UInt16 {
#0 1007.5                                                    ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:386:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt32' always fails
#0 1007.5         } else if let l = lVal as? UInt32, let r = rVal as? UInt32 {
#0 1007.5                           ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:386:57: warning: cast from 'PackProtocol?' to unrelated type 'UInt32' always fails
#0 1007.5         } else if let l = lVal as? UInt32, let r = rVal as? UInt32 {
#0 1007.5                                                    ~~~~ ^   ~~~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:396:32: warning: cast from 'PackProtocol?' to unrelated type 'UInt' always fails
#0 1007.5         } else if let l = lVal as? UInt, let r = rVal as? UInt {
#0 1007.5                           ~~~~ ^   ~~~~
#0 1007.5 /build/.build/checkouts/Neo4j-Swift/Sources/Theo/Model/Node.swift:396:55: warning: cast from 'PackProtocol?' to unrelated type 'UInt' always fails
#0 1007.5         } else if let l = lVal as? UInt, let r = rVal as? UInt {
—
Reply to this email directly, view it on GitHub https://github.com/Neo4j-Swift/Bolt-swift/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABATZ3EE5Z3L7JR3RPG5ZDWAPDQFANCNFSM6AAAAAAQXMJMZY.
You are receiving this because you are subscribed to this thread.

Sounds awesome man! I would really love to get this working. ✨
Here is a clean vapor project. I created a plain vapor application and added the Neo4j swift driver in the Package.json file. Then I run the following docker command: docker buildx build . --platform linux/amd64 or if you're not a silicon chip: docker buildx build .
I've tested it with the package: it fails building. When I leave the package out it does build succesfully.
Hope this helps! Thanks a lot for already for looking at it 🙌

I think I found the problem. It seems to be with the dev/5.2.1 branch. For the new Certificate Validators there is no Linux implementation yet nor is there for the NIOTSConnectionBootstrap. Is there a way you could help me setting this up?


decoding defmt frames from C?

Is there any way to decode defmt frames from C?
Implementing it in C doesn't seem so easy.
Linking Rust to C seems to be one option but I don't want to change the build system.

Hi @jeffy1009, Thank you for your question!
As you said, one option is to create a thin wrapper around defmt-decoder which exports the necessary functions as C. See the rustonomicon for an example.
The other option is to use the JSON-mode of probe-run. Invoking probe-run with the --json flag will output JSON-frames instead of the human readable output to stdout. See the defmt book for how to use it.
You can then write a simple C program to decode the JSON and forward it to your application.
Both options are valid, but you probably know best which one works best for your use case.
👋🏾


Fix display of embedded Window

Fix Rect of texture to take window and stretch transform into account.
resolve #66809
resolve part of #72760
MRP: WindowsTestbed.zip (run and set "Window height" to 150)
There is no need for viewport_attach_to_screen for embedded windows, since their display is handled via Viewport::_sub_window_update.

Updated 2023-08-05: Draw black bars at the sides, that aren't covered by the content.
Updated 2024-12-16: Rebased on master for good measure. insert const expression, fix rounding issue.

Thanks!


Nrc workbench

update to current version

@enarcee thanks for your contribution, sorry I didn't see this until now. GitHub notifications somehow always get lost.
I also updated README.md, and @oliv3 contributed a pull-request for docker; I merged that before I saw this. Sorry! Perhaps you can rebase and force-push?


Electron Application

Can set settings from a pop up window. Not sure how we can build and distribute it yet.

fixes #7

This is what the settings window looks like:

Wow, thank you so much for your work. This looks great!
I just tested the prefilling part on a Mac and it works really well. I haven't tested to get to the beeping part though. Could anyone please confirm it's still working?
I will check the code a bit more thoroughly tonight when I will hopefully have more time and will try to merge it asap.
Again, thanks so much already!

Not sure how we can build and distribute it yet.

I have never used it, but we could use Github's release section to host the builds. But yeah, manually building could become a bit time consuming if there are frequent changes.  I would be happy to do the Mac builds every now and then, though.
Do you know if it's possible to build on Github or elsewhere somehow automatically?

I think I need to add a way that doesnt break existing users that run in node. It may just be a matter of updating the readme and add a script in the package.json

@jvolker I just restored the ability to run npm start to just run using node without having to build the electron app. I think this will mean we can merge this in and get other Pull Requests merged while we figure out how to package and distribute the build.

@jvolker I just restored the ability to run npm start to just run using node without having to build the electron app. I think this will mean we can merge this in and get other Pull Requests merged while we figure out how to package and distribute the build.

This is a great idea. Let's do this.
The code looks really good to me. Just want to test it one more time tomorrow morning as I need to stop working on this for today. Is it good to merge from your end?
I love your work! Thanks again.

I have addressed your feedback. I think its ready to merge.


Add missing GraalVM substitutions for full native image support.

Motivation:
Netty is used in a couple of contexts that aim to support native
compilation with Graals SubstrateVM, many of them wanting to have SSL
support.
In most of the scenarios, Netty event loops and buffers are used as well.
Two of those things are

The Neo4j Java driver (https://github.com/neo4j/neo4j-java-driver)
Quarkus-Netty extension
(https://github.com/quarkusio/quarkus/tree/master/extensions/netty)

At the moment these have to provide the same GraalVM arguments and the
same substitutions, especially all around the SSL Context, event loop
and buffers.
Modifications:
Provide the sets of substitutions that proved to work with
Neo4j-Java-Driver and Quarkus inside Netty. There are already a couple
of substitutions for Graal to be found, so no new dependencies have to
be added.
Substitutions will be provided to select an appropriate JDK ALPN
negotiator for the context and engine, the reference counting of the
buffers needs a static initialization and for the event loop, one cannot
use the plattform dependent Mpsc Queue.
Further more, native image properties have been added to mark all
classes that needs runtime initialization.
Result:
With those changes in place, a couple of project can stop running their
own substitutions completly and relay on Netty.
The change won't affect Netty on the JVM, it has only effect when run on
Graals SubstrateVM.

Ping @gsmet from Quarkus. Would you mind having a look respectively ask the appropriate folks? Thank you!

@cescoffier could you follow that one? We need to be sure everything is right here.

So what we want to do with this one ? I am far from a Graal expert

I can revisit where we left off next week during Easter holiday if there's still interest.

@michael-simons any update ?

Sorry, things went quite different here around and after easter…
I agree with the System properties and have removed them.
There was one minor polishing.
There's nothing more I can add to this right now.

Just a heads up https://github.com/oracle/graal/issues/1902#issuecomment-672847986


Reduce collection wrapping

There are several cases where low level APIs produce a basic IDictionary or similar, but then we wrap it in a IFoo : IDictionary to make it more user friendly. These wrappers cause lots of extra allocations on each request, resulting in lots of GCs.  The main culperates are request headers, response headers, query, form, and cookies.
Headers we can fix by moving IHeaderDictionary into the Features package, use it for IHttpRequestFeature.Headers & IHttpResponseFeature.Headers, and have the servers implement it directly. This is a little more work for the servers, but it removes a wrapper layer.  We'll still need the old wrapper implementation for tests though.
Query, Form, and Cookies we can address by having the callers create a collection that implements IDictionary and their own interface (IQueryCollection, IFormCollection, and ICookiesCollection), and having the parser add everything to the provided IDictionary.

Quick check on "why" the additional interfaces; is this because it is a different contract?

As in context exposes via new interfaces

https://github.com/aspnet/HttpAbstractions/pull/411


Feature request: watch configuration file

Hi again,
A useful feature would be to monitor the configuration file with inotify and restart when changes are detected.
An alternative could be to catch a signal (perhaps SIGUSR1) to trigger re-reading the config file.

It's possibly a bit more complex than just restarting, as it would probably be preferable not to disrupt connections that exist in both versions of the config file during the reload, while disconnecting connections that were removed from the config. But sure, doable :).


Map view empty in Öffi, other than Location Markers

I see that there's another issue (#7) here about Öffi crashing, which I don't experience. Regardless I thought I should post a new issue regarding my own current problem. It relates to another app which doesn't have a Bug Tracker so I thought I'd document it here and then e-mail him with a link to it - I hope that's OK.
I installed the most recent .zip to my Android device running CM12.1 (so Android 5.1.1), also with microg installed from F-Droid (up to date, AFAIK). Before I did this, the "normal" builds of Öffi in the developer's own F-Droid repo refused to install - afterwards, the install was successful.
There are a few modes in Öffi that can display a map, e.g. the one for showing nearby stations. The map pane is visible along with a location marker (indeed, when in an appropriate route mode, other location markers are also visible, e.g. for your destination too), but with no map content. I wondered whether there was a setting or something that I'd forgotten to properly configure but as far as I can see there isn't a settings menu for this Maps "package".
I will have time later in the week to try to get some actual debug output, and once I do, I'll post that in a reply. Also if there's any other information that would be useful, don't hesitate to ask. I'll provide that too once I have the time.

Oh, first clarification, I have actually installed this from the other Github repo by the same author, as this repo does not have release builds as the Readme claims. I had actually "confused" the two projects, though I am (mostly) confident that they are in fact the same project.
https://github.com/mar-v-in/MapsAPI

I'm not sure whether the developer intends himself to post in here, but nonetheless he asked me the following question, which seems apt to post (translated) here:

Who is it that's actually providing the OSM-Tiles for the MapsV1 implementation?

Your symptom description makes me think there is a problem with the Google server authentication (Maps V1 API key).

@schildbach, potentially related or helpful: I got a crash when "playing" with the map view this morning (specifically, I was zooming in and out), and Öffi spat out a debug log of sorts. I saved it, and have just now extracted it from my device: http://sprunge.us/IKBf

@aphirst this is a known problem with the old mar-v-in/MapsAPI implementation with latest Android versions due to changes in SD-card access (tile cache is written to sdcard)
The latest version (this repo) does not suffer from that problem, but has the problem described in #7, once it's fixed there will be a new release.
Long term goal is to redirect Maps API v1 calls to Maps API v2, so we can have all its nice features and focus work on a single, vector-tile-based Maps implementation.

As this issue is present in mar-v-in/MapsAPI, but should not be in this repos release, I'll close here.


⚠ ClamAV has degraded performance

In 2adce72, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 464 ms

Resolved: ClamAV performance has improved in d351858 after 536 days, 23 hours, 55 minutes.


bugs: test_delete_items

Failure in tests.test_routes.TestYourResourceServer.test_delete_items
  File "/usr/local/lib/python3.11/unittest/case.py", line 57, in testPartExecutor
    yield
  File "/usr/local/lib/python3.11/unittest/case.py", line 623, in run
    self._callTestMethod(testMethod)
  File "/usr/local/lib/python3.11/unittest/case.py", line 579, in _callTestMethod
    if method() is not None:
       ^^^^^^^^
  File "/app/tests/test_routes.py", line 204, in test_delete_items
    self.assertEqual(resp.status_code, status.HTTP_201_CREATED)
  File "/usr/local/lib/python3.11/unittest/case.py", line 873, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/usr/local/lib/python3.11/unittest/case.py", line 866, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError: 405 != 201


CC-438 : Changes as per node 18 migration and Dependencies are updated to required version of node 18 build

Updates :
Docker config files are added to have a new way of docker build and run using node 18 shared service image.
Updated unit test cases and the way constants are declared to match node 18 images.
Updated readme instructions.
Update on vulnerabilities reduced from 7 to 2 up updating few vulnerable package.
Updated dependencies to available versions as below.

Clean up's will be done as part of separate ticket

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!


python3Packages.async-upnp-client: 0.22.10 -> 0.22.12

Motivation for this change
Update to latest upstream release 0.22.12
Change log: https://github.com/StevenLooman/async_upnp_client/blob/development/CHANGES.rst
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

7 packages built:

home-assistant
python38Packages.async-upnp-client
python38Packages.openhomedevice
python38Packages.python-songpal
python39Packages.async-upnp-client
python39Packages.openhomedevice
python39Packages.python-songpal


Switch Eden workflows to 0.9.3-stable tag

In case we want to backport patches, 0.9.3-stable branch was created Ideally it should not happen, since tests are using branch and state at which they were released, but mechanism was broken, so branch was created
CC: @deitch @eriknordmark

Seems like it is still looking for danger. See https://github.com/lf-edge/eve/actions/runs/7891819915/job/21537489180?pr=3761

@eriknordmark can you run tests? Let's see how it goes


Setup locales for extension

Create the locale setup for the extension.
I've patched in just one locale zh but this can be copied for the others (see Extension localization for further details).
Review the function and parameter descriptions for accuracy as I'm not sure what's intended by the "L4, L5, L6" naming (levels I'm assuming).
Also, will need to verify that the zh code is sufficient or if zh-cn is needed to select the locale for that particular language.

@abchatra , @MariaOlekheyko

@ganicke  looks good to me. Good to merge? I have given write access to you as well. So feel free to merge.

If you specify just zh, then both Chinese Traditional and Chinese Simplified will be mapped to the same locale. I am unsure if that is ok for the translator. Otherwise you need to specify specific locale.
zh-CN
zh-TW

OK, we can merge and extend the zh locales if necessary. I need to put together an email and see what all the parties are think as far as the HoC contributions goes. They wanted to upload markdown to an en folder but I pushed back saying that we'll go the Crowdin route with this content.

Otherwise, it would be nice to put the docs in this extension and have a brief release cycle that auto-loads the extension and "plugs-in" the HoC docs.


fix: return (String, usize) on non-supported platforms

On non-supported platforms, e.g. FreeBSD, the mtu crate is a no-op.
fn interface_and_mtu_impl(_socket: &UdpSocket) -> Result<(String, usize), Error> {
    default_result()
}

Previously the function signature was off with its unix and windows equivalents.
This commit updates the function signature and adds a CI smoke-test for FreeBSD.

https://github.com/mozilla/neqo/pull/2099 landed, so we can update this.


Compatibility with dotnet v6

Is seems that devskim cli is available for dotnet v5 but not dotnet v6
Is it planned to upgrade it so it is dotnet v6 compliant ?

Yes. This is part of #390.  I've made good progress on the rewrite and I hope to have a beta of the rewritten CLI released next week.

@gfs thanks for your quick response :)

@nvuillam Good news. I have released a preview of DevSkim 0.7 on nuget - this uses .NET 6.  You can grab the latest with
dotnet tool install -g Microsoft.CST.DevSkim.CLI --prerelease.  Please let me know if you encounter any problems. I'm working for the next few weeks on rewriting the IDE extensions as well, so there will be a few weeks while it remains in a pre-release state.

Thanks a lot !
MegaLinter embeds only official releases, but we can try to embed DevSkim CLI in a sub branch until DevSkim for dotnet v6 is official :)

@nvuillam sorry for the delay. It's taking a bit longer than I'd hoped to resolve the extension issues.
Unfortunately I won't be able to dedicate much time to this for the next couple weeks. I may investigate if I can do a either a 0.7 cli or a 0.6 that supports .Net 6.0 released before completing the IDE extension refactors next week.
Thanks for your patience.

@gfs thanks for the update :)

We have spent a fair bit of time on this in the last two weeks and I think we're getting close. We've been having trouble getting the VS extension to build in the pipeline.
I believe I'll have a stable net 6 based release out next week.

NET 6 support should now be live on nuget with 0.6.30.


Strange behavior of TreeView control

Describe the bug

Suppose there is a folder named "test" and its subfolder "NewFolder"

When I remove the "NewFolder" folder, the following code is executed
Node.Children.Remove(RemoveNode)

Now the HasUnrealizedChildren property of the “test” folder is false
Therefore, the small arrow on the left should not be displayed, like this

The icon on the left shows the "folder opened" icon, which is caused by the IsExpanded property failing to automatically become false
However, when I manually set the IsExpanded property to false, the small arrow on the left appeared again, but the HasUnrealizedChildren property was still false, like this:

Steps to reproduce the bug

Steps to reproduce the behavior:

Create a folder with subfolders, like “test” and “NewFolder” mentioned above
Remove all subfolders
Set IsExpanded = false

Expected behavior

Without small arrow on the left
Screenshots

Version Info

NuGet package version:
2.3.200213001

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

November 2019 Update (18363)
Yes

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context
TreeView is indeed a bad control with many bugs!

@predavid fyi

@zhuxb711 if you reverse the order that you perform these repro steps in do you get the same result? for instance, set the isExpanded property to false and then remove the last child.

@StephenLPeters I have reversed the order

Set IsExpanded=false and then remove the last child

But although this TreeViewNode has no child nodes, it still shows the small arrow on the left

Okay thanks that is good info. So we know how to prioritize fixing this with all the work we are doing for WinUI 3.0, is this a blocking issue for you? Have you been able to find a work around?

@StephenLPeters
There is no solution for the time being, all I can do is to set HasUnrealizedChildren=false instead of IsExpanded = false after removing the last child node
Does this bug require WinUI3 to be fixed?


Blend Shapes rebuild on export

I'm not sure if this is Maya or the exporter but I've deleted a bunch of blend shape targets that are automatically created with an auto rigger. When I go to export the glTF new targets are created for each one that I previously deleted. Will send a file over if need be.
This target is created during export:

That is actually by design, Maya2glTF resamples everything, even the morph targets. We do this so you can apply modifiers like smooth after the morphing is done.
Now we only do this for the weight attributes connected to the blend shape modifier.
So not only the target shape should be deleted (Maya keeps the deltas in the blend shape modifier anyway even if you delete the target shape), but also the weight attributes must go.
One feature that is not yet in Maya2glTF, is to specify what blend weights should be considered static/setup and dynamic/animated. This can be useful when creating and exporting different characters from the same base setup. The exporter should consider all setup weights to create a new base mesh, and recompute the blend target deltas based on that.

One feature that is not yet in Maya2glTF, is to specify what blend weights should be considered static/setup and dynamic/animated. This can be useful when creating and exporting different characters from the same base setup. The exporter should consider all setup weights to create a new base mesh, and recompute the blend target deltas based on that.

Ya that would be a great feature.
What I ended up doing was using the auto rigger blend shapes to create new targets. I then used those targets to create a new blend shape. Then I deleted the old blend shape.
Does the flag -ignoreMeshDeformer flag ignore an entire blend shape node? Maybe I can do that to the auto rigger's blend shape and only publish my custom one.

Oh I think I misunderstood your issue. So you mean that new Maya targets are created, or that the exported glTF contains deleted targets? The former should not happen, if it does, please send me a scene so I can reproduce this.

Ahh ya sorry about that.
Yes when I delete a blend shape target from the shape editor it's gone. When I run the exporter it then creates a new target inside the same blend shape during export. This new target gets a name like "w[65]" from my screen shot and contains no weight information (Nothing moves when i turn the weight on or off). This also doesn't happen for every target. I'll send a file when I get home!

Oh that is really interesting...
This blend shape reconstructor is the only place where Maya2glTF modifies your scene: it disconnects the plugs to the weight attributes, setting each weight plug to 1 and others to zero, reading the output mesh, restoring the plug connections. I also do this for deleted shapes, since some artists deliberately delete these, but the blend modifier stores the deltas internally, so it keeps working. However in your case you also deleted the weight plug itself, so it is very strange that it is resurrected.
I am not happy with the fact that the exporter modifies the scene, but I don't know a better way to do this.

Does the flag -ignoreMeshDeformer flag ignore an entire blend shape node? Maybe I can do that to the auto rigger's blend shape and only publish my custom

Yes, it should do that, but is not very well tested. That's how we do it at WonderMedia, we create a second blend shape "setup" modifier that is excluded when exporting.


Auto-complete files and folder names using "TAB" key

Hi Team,
I have installed Myrtille on my windows 10 machine, then connect by my browser. Can you please provide solution to auto-complete files and folder names using "TAB" key. Currently, Tab key is used to enter comma in web browser.
Thanks in Advance

I don't understand your question, could you be more specific? The tab key can have many uses, depending on the context. Myrtille simply captures the browser keystrokes (tab included) and forward them to the remote side; there is no additional things to do apart that.

In the PuTTY(Windows), When we type "sh" and press TAB key, it completes the word i.e. show. This is the excellent feature of PuTTY or Power Shell to complete the words.

In the above scenario, Is it possible to work like autocomplete in the web ssh terminal using Myrittle.
I hope you understand. Let me know for more info.

I thought you were speaking about RDP. This SSH issue is related to https://github.com/cedrozor/myrtille/issues/204.
I will try to see if I can do something about it.

Yes, you are right.

Have you try to resolve this issue or any relevant solution?

This is in the todo list

Fixed in the upcoming 2.9.0 release. Please also note that autocompletion is depending on the shell being used and its configuration (https://serverfault.com/questions/196336/why-do-some-ssh-sessions-offer-autocomplete-and-some-not)


Update cosmjs 0.24-alpha.22

Closes #16

First a PO review.

:+1: lists a whole lot of clients and channels (I want to look at this code for the relayer)
Client numbering. I see Client 07-tendermint-0 4 times, then Client 07-tendermint-4,5,6.... Wondering why 1,2,3 are missing

Update: Same channels seem to be reported under each client. Maybe just remove all the client-xxx headings and show the connections. Then you can look at the details for each connection to see which client it is. Later we can work on proper groupings.

Minor: connections are numbered connection-0, connection-1, connection-10, connection-2. Since we know they are all now format xxxx-\d+ we could just order them by the last section (split last -, parse number, and sort numerically)
:+1: I was able to drill down to the channel level for eg. connection-6 and connection-8
It seems some channels only have acks, others only have sent packets. I wonder if this is the relayer issue?

First a PO review.

:+1: lists a whole lot of clients and channels (I want to look at this code for the relayer)
Client numbering. I see Client 07-tendermint-0 4 times, then Client 07-tendermint-4,5,6.... Wondering why 1,2,3 are missing

Update: Same channels seem to be reported under each client. Maybe just remove all the client-xxx headings and show the connections. Then you can look at the details for each connection to see which client it is. Later we can work on proper groupings.

Minor: connections are numbered connection-0, connection-1, connection-10, connection-2. Since we know they are all now format xxxx-\d+ we could just order them by the last section (split last -, parse number, and sort numerically)
:+1: I was able to drill down to the channel level for eg. connection-6 and connection-8
It seems some channels only have acks, others only have sent packets. I wonder if this is the relayer issue?


Next component's setup throw an error, turn back and last component crashed as well

Version
3.0.7
Reproduction link
https://cdpn.io/wzhzzmzzy/debug/poNZOMM/PNkvYXXQLgYA#/
Steps to reproduce

Open chrome dev console
Click "About" link
Click "Home" link

What is expected?
At last, <Home /> component should be rendered successfully.
What is actually happening?
<Home /> component missed, and there's an error:
TypeError: Cannot read property 'parentNode' of null

[Vue warn]: Unhandled error during execution of scheduler flush. This is likely a Vue internals bug.
Please open an issue at https://new-issue.vuejs.org/?repo=vuejs/vue-next

How should I catch and handle errors throw by setup function appropriately?

ur reproduction link shows 403 error.

Provide the editable codepen please: https://codepen.io/jh3y/pen/PoGbxLp

sorry, here is it.
https://codepen.io/wzhzzmzzy/pen/poNZOMM

I think this is intended in DEV,if errorCapturedHooks or appErrorHandler is not configured.

In CodePen, onErrorCapture in root could not catch the error that thrown by child.
But I tried onErrorCapture and app.config.errorHandler locally, they all can get it settled, both at ssr and csr.
Thanks a lot.


Update PercentEscaper to Support UTF-8

Fixes #10211
This PR updates PercentEncoder to support encoding UTF-8 characters.

/check-enforcer evaluate

/check-enforcer evaluate


Refs 4043: use content-template rbac

Summary
Adds content template permission to content template apis
Testing steps
Probably needs to be tested in stage after merge
Checklist

[ ] Tested with snapshotting feature disabled and pulp server URL not configured if appropriate

https://issues.redhat.com/browse/HMS-4043

this looks good, just needs some updated tests (unless i missed them)

added a small test, but we don't really test rbac for each handler, we more test the infrastructure around it.  Probably better suited for an api integration suite or iqe tests


Result limit is 250 packages

How could I get the next packages, when I need all the results for the given query?

You can read the api docs here https://api-docs.npms.io/. Does it answer your question?

Offset and limit? Sorry but somehow I am unable to find these parameters documented on that page...

See from and size.

Oops, the params are actually from and size. My mistake. Can this be closed?

Oh yes sorry! Somehow I was blind, thank you very much! ;)

No problem!


[<IP_ADDRESS>5] Bug:

Bug found testing Stacks Wallet for Web.
Error
Loadable: api.correct-nonce
Message: SyntaxError: Unexpected token F in JSON at position 0
Environment
Version: <IP_ADDRESS>5
Branch: feat/launch-misc
Commit: 6190d5f

SyntaxError: Unexpected token F in JSON at position 0
Version:<IP_ADDRESS>5
Branch:feat/launch-misc
Commit:6190d5f
If you believe this was caused by a bug in our code, please file an issue on Github.

SyntaxError: Unexpected token F in JSON at position 0
Version:<IP_ADDRESS>5
Branch:feat/launch-misc
Commit:6190d5f
If you believe this was caused by a bug in our code, please file an issue on Github.


Improve user experience with custom icon packs

Due to the new syncing mechanisms, all icon packs will have to be zipped into a file so that the syncing works smoothly. This behaviour works with the predefined icon packs but not with the custom created icon packs.
The improvement will include a better user experience to support icon additions to a custom icon pack through a zip file.
This issue was created based on #223

Hey @FlorianWoelki, just a quick question about this for my understanding: Does this issue mean that currently the "sync via zip file" feature does not work at all with custom icon packs? Or can I already add a zip file to a custom icon pack?
I'm asking because I'm experiencing the syncing issues described in #52 and just updated to 2.2.1 and am confused about how to use the new sync feature. ^^
Also, the only reason I'm using a custom icon pack actually is that the version of Lucide Icons in the predefined icon packs is quite outdated and a lot of new icons are missing in it. Are there plans to update the icon packs soon? I'll also gladly do a pull request for that if wanted. :)

Hey @FlorianWoelki, just a quick question about this for my understanding: Does this issue mean that currently the "sync via zip file" feature does not work at all with custom icon packs? Or can I already add a zip file to a custom icon pack?
I'm asking because I'm experiencing the syncing issues described in #52 and just updated to 2.2.1 and am confused about how to use the new sync feature. ^^
Also, the only reason I'm using a custom icon pack actually is that the version of Lucide Icons in the predefined icon packs is quite outdated and a lot of new icons are missing in it. Are there plans to update the icon packs soon? I'll also gladly do a pull request for that if wanted. :)

Yes you can already add a custom zip of your own custom icon pack and then use it. :)
I agree with the outdated icon packs, you can feel free to create a PR for that 🚀

just to be clear, this is not yet implemented per the FlorianWoelki moved this to Todo in Iconize Board 3 days ago bullet above? If so, are we currently unable to use custom icon packs with Obsidian sync? I just tried a zipped package and it didn't show in the sync logs. Workaround right now is using Syncthing to sync the icon pack to my phone

It should be possible to sync custom zip files in Obsidian sync. Maybe the zip file was empty?

Thanks for the reply @FlorianWoelki . No, I personally zipped the files myself, and confirmed it was not empty

Do you have something so that I can reproduce the issue? Because atm, I cannot reproduce it locally.

zipped_custom_icons_14.zip Attached is my zipped 14 icon custom pack. I drop this into /.obsidian/icons, and confirm it shows in the plugin settings as a custom pack. But it never shows syncing to my Vault, and subsequently never arrives on my phone. I've deleted and re-added multiple times, even deleting the vault from my phone.

Thank you! That's unfortunate. Do you use Obsidian Sync? Because obviously, normal syncing would mean it thinks when something gets added/updated :D You could also try using this approach

Yeah, I used to use Syncthing but now I have started paying for Obsidian Sync. And yes, I have already put my icons in .obsidian/icons but that does not solve the issue :/ Obsidian support doesn't have any answer to why this is happening, they seem to think it's something to do with the plugin. They said they've reached out to you but not response. Btw, I am just relaying the message here, not blaming you. I am just trying to get to the root of the problem :)

Well, they've never reached out to me, honestly. Then I need to reach out to them...

For Obsidian Sync, any non-recognized folders in .obsidian are ignored. The setting for syncing 3rd party plugins will sync files in the folders .obsidian/plugins/plugin-name, so this is why

You should not be storing thousands of icons in that folder, to avoid clogging up the syncing process. I've suggested changing the default unpacked folder to .obsidian/icons/, but it seems like that hasn't happened yet.
If you do have a packed zip file, then you'll want to put it in the plugin folder so it gets synced.

For Obsidian Sync, any non-recognized folders in .obsidian are ignored. The setting for syncing 3rd party plugins will sync files in the folders .obsidian/plugins/plugin-name, so this is why

You should not be storing thousands of icons in that folder, to avoid clogging up the syncing process. I've suggested changing the default unpacked folder to .obsidian/icons/, but it seems like that hasn't happened yet.
If you do have a packed zip file, then you'll want to put it in the plugin folder so it gets synced.

Got it! I will set the default path to .obsidian/icons/. Thanks!

If you're interested, I suggest possibly a way to pack and unpack through zips using this zip library. It will unfortunately bloat up your plugin's minified size by a few hundred kilobytes, but it may be worth considering.
See how we use it here:
https://github.com/obsidianmd/obsidian-importer/blob/master/src/zip.ts#L1

https://github.com/obsidianmd/obsidian-importer/blob/master/src/zip.ts#L1

Thank you for the suggestion. Zipping of the individual icon packs is already done, so that one icon pack is in one zip file. I think the safe approach would be to just set the default icon pack path to .obsidian/path

I apologize, but I am not following the solution. Do I need to zip up my icons and place them in .obsidian/icons? Or will it work without zipping them up?

Please follow up with the question in the other issue. This issue wasn't intended to be about the syncing issue.

Thanks for the reply @FlorianWoelki . However:

@lishid closed the other issue
And you specifically mention syncing in the original post of this issue

Due to the new syncing mechanisms, all icon packs will have to be zipped into a file so that the syncing works smoothly.

Yes, but it still can be discussed there. Maybe you can also ask the question in #52.

Ok I will not discuss the syncing here. However, the issue of whether we need to zip up our custom icons would seem appropriate to discuss here.
I just deleted all custom icons from both computer and phone. I then zipped up my icons and put them in .obsidian/plugins/iconize/icons. I then told Iconize to look in in tis dir but it wouldn't recognize the pack. Am I doing something wrong?

You only need to zip your custom icon packs. That's how I would do it:

Add a custom icon pack and add all the icons to this pack
Zip your custom icon pack where it has been created
Enjoy the custom icon pack

My apologies again, but I don't understand. I think there might be a language barrier in our communication. I have tried the instructions above but it does not work. I don't understand why I would create a custom folder, then zip them up inside of that folder. I am confused
I have the following questions:

Where am I supposed to tell Iconize to look for my icons?

When I zip up my icons, is the zip file supposed to be in the folder that previously contained them? For example

.obsidian/plugins/iconize/icons/custom_icon_pack/zipped_icon_pack.zip

No, you only need to zip the icon pack itself. At the moment, the plugin works through checking the icons in the zip file.

In my screenshot above, I made sure to show you that the icon pack is zipped, but the plugin won't recognize it.

I am going to take a closer look at this issue.

Resuming #272 here. Very interesting article and works fine on the plugin too 😃
https://nsisodiya.medium.com/hacking-font-awesome-for-custom-svg-icons-ea762118fa7b

A follow-up on #272. I don't know if the Icomoon solution has been mentioned before, but you can generate custom SVGs using the Font Awesome font library and they seem to be working seamlessly. The only issue I have in the past 9-10 days of usage is probably that the whole vault will take some seconds more to load, but it was a large one already, so...

@FlorianWoelki can you give us update on this?
I sync icons folder normally, don't add icons in settings but just add zip and all works fine. 👍

In my opinion, not zipping the icons, and just using folders with plain SVG files would make a lot of this easier. No need for zipping manually, nor automatically.
I'll copy this from my closed issue, as it was closed, and kinda duplicates this one: (https://github.com/FlorianWoelki/obsidian-iconize/issues/453)

I would like to have icons stored as individual plain text files, instead of opaque zip archive blobs, to version and sync them more easily.
I version my vault with git, and this would make that easier too.

(I already use a plugin (this one, but it's unmaintained: https://github.com/aidenlx/obsidian-icon-shortcodes), which stores each icon as a file.)

I second @NomarCub's point. The zip solution is great for bundling icon packs with the plugin, but there's much less advantage to bundling up a user's icon pack locally!

I would also prefer syncing of plain files. I made a folder with a hundred selected icons, so there's no big overhead.
An additional struggle is the naming though... because Iconize removes all white-spaces and special characters. So you end up with duplicate files in your vault and combined with syncing it's just all a big mess.
I've never seen this weird like renaming thing in other plugins - it's totally unclear to me why that should be necessary.
Anyways, the only clean way to sync and avoid duplicates is:

Remove all special characters and white-spaces from the names
Use a zip file
Store it in the vault directly (as in: not in the .obsidian folder)
Keep a different folder somewhere for working on the icons

Works so far.


Update stac-fastapi to v2.4.8

Related Issue(s):
None
Description:
Update stac-fastapi dependencies to v2.4.8.
PR Checklist:

[x] Code is formatted and linted (run pre-commit run --all-files)
[x] Tests pass (run make test)
[x] Documentation has been updated to reflect changes, if applicable
[ ] Changes are added to the changelog

I added issues for these things:
https://github.com/stac-utils/stac-fastapi/issues/611
https://github.com/stac-utils/stac-fastapi-elasticsearch/issues/159
https://github.com/stac-utils/stac-fastapi-elasticsearch/issues/158


KA Lite application must not load if kalite executable is not found.

@cpauya this fixes #327
I created an alert dialog to notify and will not load the KA Lite application if kalite executable is not found,

Tested and demoed, merging.


Make the location where the Docker image copies JARs configurable

In Dockerfile-3_11 and in Dockerfile-4_0 jars are copied as follows:
COPY --from=builder /build/management-api-common/target/datastax-mgmtapi-common-0.1.0-SNAPSHOT.jar /etc/cassandra/
COPY --from=builder /build/management-api-agent/target/datastax-mgmtapi-agent-0.1.0-SNAPSHOT.jar /etc/cassandra/
COPY --from=builder /build/management-api-server/target/datastax-mgmtapi-server-0.1.0-SNAPSHOT.jar /opt/mgmtapi/
COPY --from=builder /build/management-api-shim-3.x/target/datastax-mgmtapi-shim-3.x-0.1.0-SNAPSHOT.jar /opt/mgmtapi/
COPY --from=builder /build/management-api-shim-4.x/target/datastax-mgmtapi-shim-4.x-0.1.0-SNAPSHOT.jar /opt/mgmtapi/

I am working on Medusa integration with cass-operator for backup/restore support. Medusa needs access to /etc/cassandra/cassandra.yaml. I did some testing with creating a mount point at /etc/cassandra. The image copies the JARs into /etc/cassandra. The directory is then mounted over and the files are not accessible.
Can we change the location to /tmp and and have docker-entrypoint.sh copy them over to /etc/cassandra?

I would be careful with /tmp as well. Would placing the files under /opt/mgmtapi/hurt anything then use the entrypoint to copy into /etc/cassandra be a problem?

This is resolved as of version 0.1.15


Upload control-plane and node component logs to artifacts folder

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Upload control-plane and node component logs to artifacts folders through a DaemonSet.
Example usage in k8s.io/test-infra/kubetest/azure.go:
func (c Cluster) DumpClusterLogs(localPath, gcsPath string) error {
	if *aksCcm == false {
		return nil
	}

	// This script is localed in $GOPATH/k8s.io/cloud-provider-azure/hack
	cmd := exec.Command("./hack/log-dump/log-dump.sh")
	projectPath := util.K8s("cloud-provider-azure")
	cmd.Dir = projectPath
	testErr := control.FinishRunning(cmd)
	return testErr
}

Example script output:
Installing log-dump-daemonset.yaml via kubectl
daemonset.apps/log-dump-node created
pod/log-dump-node-482st condition met
pod/log-dump-node-r4462 condition met
pod/log-dump-node-wxt54 condition met
All pods from log-dump-daemonset are ready
NAME                  READY   STATUS    RESTARTS   AGE   IP            NODE                                 NOMINATED NODE   READINESS GATES
log-dump-node-482st   1/1     Running   0          7s    <IP_ADDRESS>   k8s-agentpool1-95854893-vmss000000   <none>           <none>
log-dump-node-r4462   1/1     Running   0          7s    <IP_ADDRESS>   k8s-agentpool1-95854893-vmss000001   <none>           <none>
log-dump-node-wxt54   1/1     Running   0          7s    <IP_ADDRESS>   k8s-master-95854893-0                <none>           <none>
Creating /Users/ernestwong/Desktop/k8s-agentpool1-95854893-vmss000000 for storing k8s-agentpool1-95854893-vmss000000's logs
Dumping kube-proxy-x5sth_kube-system_kube-proxy-99f19701f2a2f673f63c9a480f11f00cb135f015ba534e472d200ef08a465bc1.log
Dumping kubelet.log
Dumping docker.log
Dumping etcd.log
Finished dumping logs for k8s-agentpool1-95854893-vmss000000
Creating /Users/ernestwong/Desktop/k8s-agentpool1-95854893-vmss000001 for storing k8s-agentpool1-95854893-vmss000001's logs
Dumping kube-proxy-zmg49_kube-system_kube-proxy-0c4da5e0fe82a34cf23f4f9e44eb4e9c7bab24122989573eb43b91afe7cfdeb0.log
Dumping kubelet.log
Dumping docker.log
Dumping etcd.log
Finished dumping logs for k8s-agentpool1-95854893-vmss000001
Creating /Users/ernestwong/Desktop/k8s-master-95854893-0 for storing k8s-master-95854893-0's logs
Dumping cloud-controller-manager-k8s-master-95854893-0_kube-system_cloud-controller-manager-11b334faa9503d328946745d3efb4e31432916a7ffbaba64692995cbab3c0081.log
Dumping kube-addon-manager-k8s-master-95854893-0_kube-system_kube-addon-manager-611abfb13b0f3d5146a6df7745cb4351ecba97da76298063c76f898615935b0a.log
Dumping kube-apiserver-k8s-master-95854893-0_kube-system_kube-apiserver-cf0d263c10f53a7b507c61aab61064125818769d44aa636c3276e0c9b7c43225.log
Dumping kube-controller-manager-k8s-master-95854893-0_kube-system_kube-controller-manager-2b33328ead21463e5abf30dcb1b85f2c99e908c80941d9eaf0520656634141c9.log
Dumping kube-proxy-rmhcn_kube-system_kube-proxy-61cbbd77fe757685578e991449ae1a0cbe6c640e11821e358dad686f48afd95a.log
Dumping kube-scheduler-k8s-master-95854893-0_kube-system_kube-scheduler-11fec0a73bb4e9974d1ce8a554ae1d39e84c19588604979f554a1c85bf593e5e.log
Dumping kubelet.log
Dumping docker.log
Dumping etcd.log
Finished dumping logs for k8s-master-95854893-0
Logs successfully dumped to /Users/ernestwong/Desktop
Uninstalling log-dump-daemonset.yaml via kubectl
daemonset.apps "log-dump-node" deleted

Which issue(s) this PR fixes:

#227
Special notes for your reviewer:
Release note:
Add log dumping functionality to upload control-plane and node component logs to artifacts folder

/ok-to-test

/assign @feiskyer

/lgtm
/approve


MudDataTable, width of ToolBarContent

Bug type
Component
Component name
MudDataGrid
What happened?
Hello,
I´m using a MudDataTable with ToolBarContent.
    Produkte
On the right side of the toolbar are a few MudIconButtons which should be aligned at the end of the toolbar.

Problem is, that when I start to scroll to the right, the MudIconButtons are being moved to the left, out of sight:

Any idea how to adjust the width of the toolbar to the visible width of the MudDataGrid?
Thanks in advance,
Expected behavior
I assume the toolbar should be the width of the visible MudDataGrid component and thus, the items should be visible all the time.
In the reprodution link I just added the table with the toolbar ... sample does not work.
Reproduction link
https://try.mudblazor.com/snippet/GkwROlPWwuJIFRaM
Reproduction steps

Create a MudDatagrid with multiple columns so that after rendering a horizontal scrollbar will be displayed.
Add a  with a  and buttons after the spacer.
Scroll to the right, the buttons will move to the left until out of sight.

Relevant log output
No response
Version (bug)
6.1.7
Version (working)
No response
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Move the contents of  to a , which you place on top of your ,
to see if that makes a difference. At least you'll have access to the  properties such as Dense, Style etc.


BaseFolder is not found when running codecept from outside test root folder

Relative path is not resolved in the helper.
Fixed it so that it will be resolved if baseFolder and diffFolder are not absolute paths.
Fixed in PR: https://github.com/Percona-Lab/codeceptjs-resemblehelper/pull/51
Looking forward to your review!

Fixed


feat: added ability to send replies and threads through twitter toolkit

Description
This extends the Twitter Toolkit to allow for the creation of threads and general replies on Twitter.

Related Issues
None
Solution and Design
Added params "is_reply" and "in_reply_to_tweet_id" for send_tweet execute.
Test Plan

Type of change

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

Checklist

[x] My pull request is atomic and focuses on a single change.
[x] I have read the contributing guide and my code conforms to the guidelines.
[x] I have documented my changes clearly and comprehensively.
[x] I have added the required tests.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @xpluscal, appreciate the PR. We would love to merge this once you add the capability to identify/read the tweet and then reply to that tweet.


Add MarkPresentAll command and UnmarkPresentAll command

Closes #117

LGTM. One small change I made in markattendance is that it displays the date using  DateTimeFormatter.ofPattern("MMM d yyyy").parse(). So you can consider changing that as well in the (un)markpresentall MESSAGE_SUCCESS fields


[MSTeams][iOS][Accessibility]No decimal point selector for Input.Number

Platform

iOS
MSTeams
.Net Core
Botframework v4

Details
Decimal point selector for Input.Number is not available on MSTeams iOS App. This is available on Android.

@nesalang could you help investigate this one please? Thanks!

fixed via #3879


refactor: remove removed-prefixed keys

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
This PR removed the removed prefixed keys; these keys weren't in use.
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Thank you!
May you cross-refer where we remove they valid usage? Or why we don't need it now.

Actually, we don't require data to be restored from these key values.


object-ownership doc link broken.

https://docs.sui.io/build/objects#object-ownership is referenced in https://docs.sui.io/build/programming-with-objects/ch3-immutable-objects

Thanks for flagging this. We rebuild our docs website, so the new correct link for object ownership should be: https://docs.sui.io/concepts/object-ownership


refactor(lint): split files for no_side_effects rule

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2684  👈
#2683
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mysteryven and the rest of your teammates on  Graphite

Merge activity

Mar 11, 11:04 PM EDT: @Boshen started a stack merge that includes this pull request via Graphite.


Set missing controller name in test manifest

Signed-off-by: AliceProxy<EMAIL_ADDRESS>Adds the controllerName: gateway.envoyproxy.io/gatewayclass-controller in the expected output from one of the test manifests.

Codecov Report

Merging #408 (3778540) into main (3c6dc8d) will decrease coverage by 0.13%.
The diff coverage is 83.05%.

@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
- Coverage   61.17%   61.03%   -0.14%
==========================================
  Files          35       35
  Lines        3809     3824      +15
==========================================
+ Hits         2330     2334       +4
- Misses       1355     1362       +7
- Partials      124      128       +4

Impacted Files
Coverage Δ

internal/xds/translator/route.go
40.12% <0.00%> (ø)

internal/gatewayapi/translator.go
85.93% <83.52%> (-0.23%)
:arrow_down:

internal/provider/kubernetes/gateway.go
66.81% <0.00%> (-3.02%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Review visualization model from cartodb and add missing fields

As we're kinda building a visualization model ourselves, it would be good to mimick visualization model from cartodb so that we have as little properties mismatching as possible.
One of these can be a tags property that we definitely need to include.

Please @csubira add a list here of the properties, the job you did for kuviz will be useful here too!

@jesusbotella @csubira I think we have almost every relevant property on the kepler.gl side, but I would like to know about the tags thing.
With visualizations (including kuviz) Is it a many to many table, shared among datasets and maps?. I'm wondering how it could be managed  from the CustomStorage side (if it is not a big 'melón')

Here are visualization field from public visualization presenter:
id:               @visualization.id,
name:             @visualization.name,
display_name:     @visualization.display_name,
attributions:     @visualization.attributions,
source:           @visualization.source,
license:          @visualization.license,
type:             @visualization.type,
tags:             @visualization.tags,
description:      @visualization.description,
created_at:       @visualization.created_at,
updated_at:       @visualization.updated_at,
title:            @visualization.title,
kind:             @visualization.kind,
privacy:          @visualization.privacy.upcase,

I guess that fields like attributions, license, or source are not relevant for Kepler's visualizations, so tags field is the only that I guess we should add to our Kepler visualizations' table.
I thought of adding a JSON field to our visualizations' table, but we wouldn't be able to search visualizations with a certain tag from a SQL query with that approach.
Creating a Tags table to link tags to visualizations would be the best option, but we'll be mimicking cartodb functionality in our custom storage as we've done with more things already. I'm starting to think it'd have been better to use kuviz api to create our visualizations, as we already use COPY to upload our datasets.

Ok, I see. I agree that tags is probably the most relevant, because it would allow a better Search in the Dashboard (in the future...)
For tags, I think that a JSON field could still provide some searching (like in the example here: https://www.postgresql.org/docs/9.4/datatype-json.html), but a tags table would probably be better, yes.
Regarding to using kuviz API, doesn't it imply that we would just manage HTML? If that's the case, it would be less powerful than our current CustomStorage approach, including the full config of the kepler viz, with 'editable' styles.
So my proposal would be going after the Tag table + a Vis to Tags n to n relation table, how do you see it @jesusbotella  @neokore ?

The easiest way to do this is JSON, for sure, but I guess that the table approach is more flexible as I said :) So that would be my preferred approach.
What I meant with the kuviz API is that given that we have a custom visualizations controller to create cartodb visualizations for kuviz (kinda similar to ours apart from the parameters you talked about which could be JSON parameters within the table) we could've had something similar to save Kepler's visualizations, making the process easier and more compatible with our current dashboard.

OK.
After your comments I think we should not add tags now. Let's go with what we currently have for CustomStorage.
And later on we can try to make a bit more of research and that other approach; maybe a new package to manage kuvizs, properly integrated with the app, would be eventually the best option.

Do we close this issue then? :)


libGL error: No matching fbConfigs or visuals found

Ran into this issue when running ./run.sh host -w:
...
player 1 of 2 (2 nodes)
Use changemap instead. Map is for single-player only.
VIZ_Init: instance id: foDC642ZOg, async: 1, input: 1
VIZ_MQInit: Init message queues.
VIZ_SMInit: Init shared memory.
VIZ_ScreenInit: width: 1024, height: 576, pitch: 1024, format: CRCGCB
libGL error: No matching fbConfigs or visuals found
libGL error: failed to load driver: swrast
It stalls at the last line with no visual. The line that's been executed to run the docker image is:
nvidia-docker run --net=host -ti --rm --name cig2017_host --env=DISPLAY --privileged cig2017_host -w
Local system environment:
Ubuntu 16.04LTS, Nvidia Graphics Driver 375.26

Does it also stall without -w option?

It does not. But it will be great if we can watch the game also.


Update static.validate.md

validate function have to return undefined

http://jsbin.com/zosewigafo/edit?js,console,output

i will merge that if no one have doubt
cc @justinbmeyer

merge it.


Possible goblin units

https://www.hiveworkshop.com/threads/shredder-flamethrower.325110/

https://www.hiveworkshop.com/threads/goblin-drumfire-tank.264413/

https://www.hiveworkshop.com/threads/goblinminedrone.241572/

https://www.hiveworkshop.com/threads/goblin-super-heavy-tank-revilgaz.162913/

https://www.hiveworkshop.com/threads/herocryptlord.49836/


normalize file operate API.

What have you changed? (mandatory)
According to official implement, file access API normally retreive a AsRef<Path> as argument, instead of PathBuf or other something. This PR changes some functions as that style.
The PR is splited from #3311 in order to make the PR won't be too huge.
What are the type of the changes? (mandatory)
Improve.
How has this PR been tested? (mandatory)
make dev.
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.

test storage_cases::test_raftkv::test_batch_snapshot ... thread 'storage_cases::test_raftkv::test_batch_snapshot' panicked at 'assertion failed: s.is_some()', tests/storage_cases/test_raftkv.rs:99:9
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::print
             at libstd/sys_common/backtrace.rs:71
             at libstd/sys_common/backtrace.rs:59
   2: std::panicking::default_hook::{{closure}}
             at libstd/panicking.rs:211
   3: std::panicking::default_hook
             at libstd/panicking.rs:227
   4: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:475
   5: std::panicking::begin_panic
             at /checkout/src/libstd/panicking.rs:409
   6: integrations::storage_cases::test_raftkv::test_batch_snapshot
             at tests/storage_cases/test_raftkv.rs:99
   7: integrations::__test::TESTS::{{closure}}
             at tests/storage_cases/test_raftkv.rs:76
   8: core::ops::function::FnOnce::call_once
             at /checkout/src/libcore/ops/function.rs:223
   9: <F as alloc::boxed::FnBox<A>>::call_box
             at libtest/lib.rs:1454
             at /checkout/src/libcore/ops/function.rs:223
             at /checkout/src/liballoc/boxed.rs:640
  10: __rust_maybe_catch_panic
             at libpanic_unwind/lib.rs:106

PTAL @overvenus


git: commits with duplicate tags

There are two commits with duplicate tags, where the second tag looks like it should have been for the following commit:
1ad022a9b81d7829d5dc98cf5b8d0ee410558040 tag: v8.0.1367, tag: v8.0.1366
5d69da462f584a3aefb3427b127334bf9af3a4b0 tag: v8.0.1739, tag: v8.0.1738
Not sure what to do about this, but tags are not really used for releases, so could be moved?
There is also the following, but that's fine probably:
ed39e1d53cc4916c2390aea635cd8419497c1460 tag: v7.2.000, tag: v7.2
(found via git log --reverse --format='%H %D' | grep ', tag')

So I think to correct it, we would need this:
git tag -f v8.0.1739<PHONE_NUMBER>b0f168f4 && git tag -f v8.0.1367 461fe50fea245b2b1

that would move tag v8.0.1739 to commit<PHONE_NUMBER>b0f168f4 and tag v8.0.1367 to commit 461fe50fea245b2b1

Christian wrote:

So I think to correct it, we would need this:
git tag -f v8.0.1739<PHONE_NUMBER>b0f168f4 && git tag -f v8.0.1367 461fe50fea245b2b1

that would move tag v8.0.1739 to commit<PHONE_NUMBER>b0f168f4 and tag v8.0.1367 to commit 461fe50fea245b2b1

That works locally, but I can't push this:
% git push origin  v8.0.1367
To github.com:vim/vim.git
! [rejected]            v8.0.1367 -> v8.0.1367 (already exists)
error: failed to push some refs to 'git@github.com:vim/vim.git'
hint: Updates were rejected because the tag already exists in the remote.
Perhaps the remote tags need to be deleted first?
--
hundred-and-one symptoms of being an internet addict:
149. You find your computer sexier than your girlfriend
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

You can push them forcibly:
git push origin -f v8.0.1367 v8.0.1739

Other people can fetch the updated tags after that by:
git fetch --tags

Ken Takata wrote:

You can push them forcibly:
git push origin -f v8.0.1367 v8.0.1739

Thanks, that appeared to have worked.

Other people can fetch the updated tags after that by:
git fetch --tags

I always wonder why git makes things so complicated.  A "git pull"
should be sufficient to get all updates.
--
hundred-and-one symptoms of being an internet addict:
150. You find yourself counting emoticons to get to sleep.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

git fetch --tags

This should have been:
git fetch --tags -f

-f is also needed for fetch.


H3 layer prop forwarding

Match the coding style of other composite layers.

Coverage increased (+0.06%) to 59.273% when pulling 9707ee232048a299cac98581bfadbff8ec60a439 on h3-layer-prop-forwarding into cb0adbeb4d435d25ebd0c845442eaf6b4e3ff476 on master.


dulwich: support interactive SSH key passphrase prompt in asyncssh vendor

Closes https://github.com/iterative/dvc/issues/9737

Makes scmrepo behave the same as git when prompting for a passphrase

$ dvc import git@github.com:pmrowla/test-dvc-private.git data/data.xml
Importing 'data/data.xml (git@github.com:pmrowla/test-dvc-private.git)' -> 'data.xml'
Cloning test-dvc-private.git|                             |0.00/? [00:00,      ?obj/s]
Enter passphrase for key '~/git/scratch/test-ssh-passphrase/id_ed25519':

To track the changes with git, run:

        git add data.xml.dvc

To enable auto staging, run:

        dvc config core.autostage true

We should also probably consider using a similar InteractiveSSHClient implementation in dvc-ssh instead of using the current ask_password/ask_passphrase options (which require the user to use those remote flags and prompt before we try to make any SSH connection or try any specific SSH keys)
related: https://github.com/iterative/dvc-ssh/pull/3

Codecov Report
Patch coverage: 30.90% and project coverage change: -1.98 :warning:

Comparison is base (d8438d4) 81.26% compared to head (39578bd) 79.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   81.26%   79.29%   -1.98%
==========================================
  Files          27       27
  Lines        4020     4114      +94
  Branches      709      727      +18
==========================================
- Hits         3267     3262       -5
- Misses        630      711      +81
- Partials      123      141      +18

Impacted Files
Coverage Δ

src/scmrepo/git/backend/dulwich/asyncssh_vendor.py
61.32% <30.90%> (-13.28%)
:arrow_down:

... and 11 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

On the scmrepo side, some of this could also be generalized a bit more and re-used in git.credentials to implement full support for SSH auth in pygit2/libgit2 (which will require loading keys via asyncssh and then passing them into the pygit2 credential callbacks), but that can be done in a follow-up PR if/when needed


add tests for Welcome component

I'd say you've got nice tests, but that might sound like innuendo


Lost on upgrade

I don't have any critical data yet, but I wanted to go through an upgrade to make notes for the future.  I believe I initially installed 14.0.0, or at least that's what the admin console told me.  I made www and mysql backups and then ran git pull on the web folder.  When I went to http://[hostname]asset/file/Upgrade, it lists the following information for each version down to 0.1.6, all with the same Wrong "version name/number or version build" on the button.
Version Name: pKb, Version Number: 14.8.0, Version Build: 1
Wrong "version name/number or version build"
It will be terribly easy to start over from scratch right now, I just wanted to try to get an upgrade down so that I could have documentation in case I ever get a subscriber base and need to do it with real data in the future.
I was thinking about just running the MySQL scripts starting with https://github.com/pH7Software/pH7-Social-Dating-CMS/tree/master/_repository/upgrade/14.0.0-14.3.0 and each one up to the last.
14.0.0 - 14.3.0
14.3.0 - 14.3.4
14.3.4 - 14.3.6
14.3.6 - 14.7.0
14.7.0 - 14.8.0
Is there something I did wrong to break my asset/file/Upgrade page?

Hi @oedo808!
Please do not overwrite the _protected/framework/Security/Version.class.php when you are updating the files to the latest version.
So, if your website was on 14.0.0, please update Version.class.php lile 37 https://github.com/pH7Software/pH7-Social-Dating-CMS/blob/master/_protected/framework/Security/Version.class.php#L37
from
const KERNEL_VERSION = '14.8.0';

to
const KERNEL_VERSION = '14.0.0';

then refresh the upgrade wizard and you will see the available upgrade patches.
Don't forget to run from the oldest version to the newest one (all of the available upgrades have to be executed).

Hope this helps 🙂

Thank you!  I restore /_protected/framework/Security/Version.class.php and ran the upgrade scripts.  I manually updated KERNEL_VERSION to 14.8.0, but I guess I could have just overwritten the file with the newest Version.class.php, but they're the same now.  Thank you for what you do!

You're very welcome @oedo808
Normally, the upgrade script will automatically update KERNEL_VERSION constant to the correct version for each upgrade patch.

And if you have time to review the project on Sourceforge https://sourceforge.net/projects/ph7socialdating/
You will need an account to do so, but it really helps me and motivates me a lot to continue this project :)

You got it.  I wanted to ask, is there any way to have a free trial period on paid membership while still collecting payment details?  Also is there an avenue for paying for additional features or feature development?  I saw your Patreon campaign, but it looked like maybe it was related to a retired Pro version.


Fix Popover disappearing when hover off

Pull Request Test Coverage Report for Build 2384

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

Totals

Change from base Build 2378:
0.0%

Covered Lines:
9016

Relevant Lines:
13208

💛  - Coveralls


Segmentation Demo of DIGITS 5.1-dev did not work

I fully followed the operation steps of semantic_segmentation, which was a demonstration stored in examples folder of DIGITS 5.1-dev by using (Caffe ver 0.15.14) based on  CUDA 8.0 (Nvidia Driver 367.57), and  it did not work.  Can anyone tell me why ?
 Do your LMDBs look OK? Can you post a screenshot of the dataset job page?

https://github.com/BVLC/caffe/issues/2251
https://github.com/BVLC/caffe/tree/master/examples/imagenet

@toki2015
did you build nvcaffe from source? If yes, you might want to checkout to a specific tag before building.
https://github.com/NVIDIA/DIGITS/issues/1257
https://github.com/kertansul/dl-docker/commit/4d1f2d7b6fb7bb4d15ca7620bd8bec7ca09bd73f

The LMDB looks good (attach screen-shots)

Yes, I build caffe from the  source ( the newest version of NVCaffe)
user@DeepBox:~/caffe$ git status
On branch caffe-0.15
Your branch is up-to-date with 'origin/caffe-0.15'.
nothing to commit, working directory clean
user@DeepBox:~/caffe$ git checkout
Your branch is up-to-date with 'origin/caffe-0.15'.

Sorry to close this issue by a typo.

@toki2015
I guess there's something wrong with the latest commit on caffe-0.15 that have yet been fixed
git checkout v0.15.14 and rebuild

You are right.
This issue was solved by  git checkout -b v0.15.14 and rebuild nvcaffe again.
Thank you very much.

@toki2015 could you help identify which commit on 0.15 branch is causing this regression? Thanks!

Note: the regression on caffe-0.15 branch was fixed with https://github.com/NVIDIA/caffe/pull/268

Oh... let me take a look about the difference. I am not a developer and wish to find some helpful information ! By the way, nvcaffe still fault to pass through the make runtest based on  CPU (Intel(R) Xeon(R) CPU E5-2695 v2 @ 2.40GHz). I guess it comes from ATLAS (default) package, do you have any idea to solve this issue ?

Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                48
On-line CPU(s) list:   0-47
Thread(s) per core:    2
Core(s) per socket:    12
Socket(s):             2
NUMA node(s):          2
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 62
Model name:            Intel(R) Xeon(R) CPU E5-2695 v2 @ 2.40GHz
Stepping:              4
CPU MHz:               2393.906
CPU max MHz:           3200.0000
CPU min MHz:           1200.0000
BogoMIPS:              4801.45
Virtualization:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              30720K
NUMA node0 CPU(s):     0-11,24-35
NUMA node1 CPU(s):     12-23,36-47
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm epb tpr_shadow vnmi flexpriority ept vpid fsgsbase smep erms xsaveopt dtherm ida arat pln pts

If you don't mind I will close the issue since it is now resolved on caffe 0.15 branch.


is it STEREO ?

hey, i had an idea to make such data backups some time ago and thanks a lot for doing this project,
but i had an idea that if modem has just a mono input, then making it stereo channel can double the data size
so is it stereo? i think it is pretty much easy to make it 2 threads and with some cross error checks going to some Reverb modulation this can widen maybe broadband too?
well, yes i see - num_of_channels parameter, but they got 128-512k through same channel with Adsl, so some modulations or data archiving maybe ? basically cassette sound mostly is better than i got from the phone, that is about quality of my phone as well, but look Reverb means different power of sound and among STEREO there was JOINT-STEREO for mp3, so measuring also power of signal can add more bits per band and changing some technology\algorithms can boost that.
The easy example - your stereo mode is a 2 separate mono channels - it is stereo async mode, but using it stereo-sync mode can add more bits for the same, because you can remove some checks and use some more sophisticated methods to encode data, because you can measure the power of signal. so basically it is 8-16-24 bits, so you can go from 2-3 bits per 20khz = ~60kb to 12-20 easy. So you will need just pick the good data read -bits size and pack all data using it. Actually using delay\reverb you can modulate mono channel too and get same data, but let's say data correction is important and better with stereo.
If we got some stable bands to decode with high resolution bits - we should take it to 32bits and for those, that aren't good and got some errors - there should be some less bits, less important data or more data checks. i mean this can be more complicated, than just 2 threads to encode\decode. i was thinking for about 500-700mb data per cassette, but it depends on many things and just writing analog music sometimes is easier than pack digital data to the same cassette. if that was 1,5-2gb - that would be a cool stuff, i think 320kb mp3 is an equal and max data for the cassette, so packing with same technique can be best way, but it takes 44khz\16bit and there we got 19-22-32khz maybe 12-32-64 bits, 128 maybe? maybe flac \ ogg is better.
look on it as a string on equalizer or you can imagine it not as a string, but as an objects built with few strings - tetris 3d - data map.
i'm not sure on how to use it in windows \ linux, but i know that audacity and lmms has plugins support, uses sdl and can produce sounds in few formats. vst-plugin can be used by almost any sound program.
i am much more musician and thinker, than a programmer, sorry. all musicians needs to pack data, but all programmers too.

Well, let's imagine more.
let's say we got that system counting about 20-30k high resolution waves or more, we can make it as a spectral anaylsis (zx-spectrum). each wave we can make as a source of light, because light is the same wave but having more broadband for it, so we get a 20k led image, we can look on it as a matrix of any-color-points or light sources - so the obstacles of each source will also have info encoded between light sources, this way we can encode more info, but it has to have more rules to read and code it.


Could not load file or assembly 'System.IO.FileSystem, Version=<IP_ADDRESS>

I installed Microsoft.Extensions.Configuration.Json 2.0.0-preview1-final and got this when running xunit tests.
Running VS 2017 and targeting net461

Apparently related to this issue https://github.com/nuget/home/issues/4467

Looks like this is being tracked by the linked issue. Closing as external

@hulvei3 @shirhatti
Is this really the same issue? Seems the linked issue only happen at build time, but this one only happen when run.
Tested with VS2017 15.3.0 Preview 2, dotnet 2.0.0-preview1-005977, Win10 16199.
Test code
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net461</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="2.0.0-preview1-final" />
  </ItemGroup>

</Project>

using Microsoft.Extensions.Configuration;
using System;

namespace ConsoleApp5
{
    class Program
    {

        static void Main(string[] args)
        {
            new ConfigurationBuilder()
                .AddJsonFile("appsettings.json", true)
                .Build();

            Console.WriteLine("success...<press any key to continue>");
            Console.ReadKey(true);
        }
    }
}

And dotnet run

Found an workaround is to install NETStandard.Library.NETFramework 2.0.0-preview1-25305-02, but can't found any doc/relnote for this.

Sounds to me like an issue with the assembly redirect, which is usually handled by nuget automatically, but I guess since the introduction of all these new project types it's not expect to be handled for you automatically. I came across this issue when I referenced a std lib project from a (test) project having .net framework v4.7
this is a starting point of how to fix it (inside of configuration\runtime), and depending on your scenario you probably have to add more redirects.
 <assemblyBinding xmlns="urn:schemas-microsoft-com:asm.v1">
   <dependentAssembly>
     <assemblyIdentity name="System.IO.FileSystem" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
     <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
   </dependentAssembly>
 </assemblyBinding>

@ibrahimbensalah
Just install NETStandard.Library.NETFramework, see dotnet/core#686


Align docs with the official OCP ones

Docs for OCP 4.10 are finished, this PR aims at aligning with them.

/lgtm

/lgtm


review caveats for Casks to move to other repos

For example, widgets have never functioned, but we didn't want to throw out the work-thus-far.  These can be moved to the boneyard.

Checked every caveat; there’s nothing more there that would justify a cask being removed. Widgets have been taken care of.


add target app restrictions for remap keys

Description of the new feature / enhancement
add target app restrictions for remap keys
Scenario when this would be used?
this can shield the windows key during the game to avoid pop-up due to accidental press
Supporting information
No response

/dup #6756


Adds BMW sponsoredContent to all events in city view

Adds BMW sponsoredContent to all events in city view

Fails

:no_entry_sign:

[danger-plugin-jest] Could not read test results. Danger cannot pass or fail the build.

  Generated by :no_entry_sign: dangerJS

@orta I dont see rules corresponding to this style, Serif with color("black60") -- maybe need to update the component? Or should we just not use the Markdown component for now?

Fails

:no_entry_sign:

[danger-plugin-jest] Could not read test results. Danger cannot pass or fail the build.

  Generated by :no_entry_sign: dangerJS

Cool, sure, makes sense 👍


⚠ Bofrost.de has degraded performance

In 47c63c6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1601 ms

Resolved: Bofrost.de performance has improved in de5b508 after 6 minutes.


make stacked bar chart top panel the same size as bottom?

Any way the two panels on the land use figure could be the same size? On a smaller screen, the top label and bottom label barely fit on the page. It'd be nice if the whole thing completely fit, to avoid a cramped/awkward feel.

I showed the viz to John Hoffman, who gave some great feedback on the CRB drought viz, and he was using a smaller screen device, and I watched him struggle to get this figure to show up completely on his screen, by zooming out to 75% so he could see the whole thing....

@eread-usgs I have made adjustments due to the text rearrangements, can we relook at this before any changes are made? Each time new text is added, the graph is manipulated...etc, I have to go and adjustment the scroll magic triggers over again. I'm on around the 5th time doing so and would appreciate a relook before more changes were made.

I also resized the graph to be a bit smaller for screen adaptability as well in the last PR

Again on this, until we are actually using the -mobile figs in the html, we are kind of handcuffed. we can easily make some mods which would be best for each device, but only -desktop is being used...

Would it make sense to scale this fig based on % of vertical screen extent?

I always like the % scaling, but idk how that would effect the triggers on each screen. It's not the easiest thing to have the triggers match up on all devices.

I know nothing about mobile...I'm closing this issue, but feel free to reopen


Unsecure random Path

Hence, there is generated the random path via random, it might be better to use from secrets import token_hex for generating safer random path and files.
https://github.com/keras-team/keras-io/blob/7adc426c4cfa4871bcf380f8a48446f4459768c1/scripts/tutobooks.py#L223

Hi,
Thanks for opening the issue.
As reviewed in the PR, we don't have any plan to change the path format, feel free to close the PR. Thanks!