Out of tree user API auto documenation support part of Zephyr's offering

I have discussed this feature in slack and people confirmed this is not part of the zephyr ecosystem. I think this is a useful feature that would enhance the development experience. The idea is to document all the user introduced API and possibly source codes in multiple directory levels.
I am an intermediate embedded C programmer to set up doxygen, breathe and sphinx along with CMake wasn't easy and straightforward. In fact, I lost motivation while trying to bring this feature up.
My tree contains only user-level source code and separated from Zephyr tree to have a dedicated docs folder on a top directory.

@manoj153 I'm not sure what all you have in mind.  Consider:

Are you wanting to reuse/extend the structure/organization of the Zephyr documentation system?
Are you wanting to specify your own logo/versioning/platform name (e.g. "Breeze Platform" instead of "Zephyr Project")
Are you wanting to include Zephyr documentation as a distinct subset?
Are you wanting documentation from each discovered module to be a distinct subset in the documentation tree?
Are you wanting this to be available and maintained in an example template/boilerplate?

@carlescufi Does this issue touch on features similar to those needed for Zephyr to integrate stand-alone west documentation?

Hi @gregshue , my goal is always to document the user introduced code. Zephyr docs are available perfectly on their website and so accessible by anyone. Since user-introduced code is also written in C and similar, I think re-using the doc framework makes sense. What do you think?
Specifically doc on own project-specific tree. I hope I conveyed the message right and clear.
This can be useful when releasing the project-specific code as a library with an API doc.

@manoj153 After playing around with the documentation in my oot_example_module and in a couple private manifest+module repos, I am considering the partitioning model below. The focus being what is needed by new owners/contributors for (re)verification, integration, and support.

Manifest repo documentation contains at least the following:

The manifest file specifies the source in the "workspace" (integrated collection of modules & applications).
The "workspace" is what is versioned and released, no matter how many projects are built out of it.
The versioned workspace needs release notes.
It is useful to have comprehensive documentation for some(?) content in the workspace, e.g.:

Kconfig
APIs
boards
drivers
subsystems
projects
samples
A listing of all modules included and at which SHAs (or release tags)
A collection of module-specific documentation

Module repo documentation contains at least the following:

overview
maintenance organization
drivers
subsystems
boards
guides for in-module subsystems/stacks, samples

Getting there will take some restructuring of the zephyr repository documentation and identifying what module documentation looks like and how it gets aggregated into manifest-repo generated documentation.
Thoughts?

My request was to support out-of-tree doc,
My application code does not influence zephyr tree (they can be separate)
I start my project with a new directory and git tree with the following structure.
/PROJECTA
    /src
        main.c
        main.h
CMakeLists.txt

The idea is to document API in main.h

The code tree will fit the Zephyr documentation process more easily if:

All project-specific logic has been refactored into subsystems, leaving an empty PROJECTA/src/main.c::main().
Project documentation follows the location and pattern of sample documentation.  (After all, samples and tests are simply special purpose projects!)
The project repository is recognized as a Zephyr module (i.e. zephyr/module.yml exists), where

Paths to your project tree(s) are part of the module samples configuration
Your CMakeLists.txt adds the subsystem interface include folder to the zephyr include path.  :-)

@carlescufi Does this issue touch on features similar to those needed for Zephyr to integrate stand-alone west documentation?

Not really, those are quite different issues. Zephyr doesn't really integrated standalone west doc, but rather contains the whole of the west documentation in its main tree (doc/guides/west).


Release/2.3.0

Coverage decreased (-2.4%) to 97.062% when pulling 91087a2d0244f6a098b223467071ced067d5808e on release/2.3.0 into 245cf62db426eae72ca3aaa82e13fbb9b0d9e482 on master.

Oh, maybe add a test to cover some of the logic in BlockingFlush()?

Fixed the coverage - going to merge it in!


fix saving objects as array in embedded documents

If embeded document contains non plain attribute(MongoId, MongoDate), Mapping convert it to array:
object(MongoId)#37432 (1) {
          ["$id"]=>
          string(24) "555b08002d447401008b70a2"
        }

to
array(1) {
          ["$id"]=>
          string(24) "555b08002d447401008b70a2"
        }

then mongodb throws error "Found $id field without a $ref before it, which is invalid"

Applied: https://github.com/klimov-paul/yii2-mongodb/commit/3fc5d67c83cb60f6b72998a6031d91de121092c0


Incorrect destination port setting in example task.json

Incorrect port setting in example task.json file in the extension details.
"destinationPort": 11160,
should  be
"destinationPort": 11169,
 {             "label": "BrAsTransfer: ArSim with default install settings",             "type": "BrAsTransfer",             "pviConnectionSettings": {                 "deviceInterface": "tcpip",                 "sourceNode": 42,                 "destinationAddress": "<IP_ADDRESS>",                 "destinationPort": 11160,                 "communicationTimeout": 1000,                 "connectionEstablishedTimeout": 60             },

Hi Johan
This PVI configuration template is the one for the ArSim connection. If I'm not mistaken, the default ANSL port in ArSim is 11160 and the default port for real PLCs is 11169.
Just to be sure I will check it again and close it / fix it depending on the result.

Yes your absolutely right, and I am wrong, were running in simulation of a real PLC.
I have checked verified that the ARSIM port is 11160. Close the case

I reopened the issue. because it seems the port 11160 in the ArSim is only used for INA.
The template which is generated from the extension does not work.
The extension uses the /PT=<port> parameter for the port in the connection settings.
Automation Studio uses /REPO=11160 ... /PT=11169 for the connection settings with a default ArSim configuration.

According to the AS help /REPO only exist for backwards compatibility with INA connections, and if both /PT=<port> and /REPO=<port> are set, /PT will win.

Therefore Automation Studio also uses port 11169 for ANSL in ArSim and the template in the extension needs to be adjusted.

Fixed, will be in next release


Display a help message when running the jar directly and add support for headless self-install

This PR adds functionality to have the Sponge mod jar generate a help message if the user tries to run it directly, and also provides a self-install mechanism for headless servers designed to make the transition from other "just run the jar" platforms more straightforward.
Generated Help Message
Running the sponge jar in the console now produces the following message:
--------------------------------------------------------------------------------

  Oops! You attempted to run the Sponge Forge Mod directly!

  This jar file is a mod for Minecraft Forge. You can run this file as a Forge
  mod by installing Minecraft Forge and simply dropping the jar file into the
  Forge "mods" directory. You can get Minecraft Forge from:

    http://files.minecraftforge.net/

  Once you have downloaded and installed Forge, you can load Sponge as a mod.

  Alternatively, for headless server installations you can use this Sponge jar
  as a launcher for Minecraft Forge. To use the jar as a launcher, you can:

    1) Install Minecraft Forge and the dependencies by running:

        java -jar sponge-1.8-1371-2.1DEV-0.jar install

    2) Run Minecraft Forge with Sponge by running

        java -jar sponge-1.8-1371-2.1DEV-0.jar run {args}

  If you need more help, you should check out our forums:

    http://forums.spongepowered.org/

--------------------------------------------------------------------------------

This is intended to really drive home the fact that this jar is a Minecraft Forge Mod and not a stand-alone server. It also tells users where to go to download Forge, and where to go to get help.
Headless Self-Installation
For users used to bukkit, having to download and install Forge seperately is inconvenient, especially if the server owner only wishes to run Sponge. Whilst it would be perfectly possible to have the jar automatically self-install Forge if run directly, I feel it's important that the user knows they are effectively using the jar as a launcher for Forge, and thus encourage them to run an install step followed by a run step. I think it would also be useful at some point to add a setup step which grabs the file and also writes out a bootstrap script which can then be customised and copies the sponge mod jar into the mods folder, effectively putting the user back in the state they would have been if they'd installed Forge separately.
The self-installation process uses Apache Ivy and is thus quite robust and can be easily maintained going forward.
The self-installation also creates the same libraries folder structure as the Minecraft launcher and the Minecraft Forge installer for consistency.
Benefits
These features present the following benefits:

Catches users who don't read the instructions and just try to run the jar, explains what the jar is and also points them to resources they will need.

Doesn't obfuscate the fact that Forge is in the mix, in fact it rubs it right in the user's face so that they are aware that this is a Forge mod and they're using Forge.

Self-Install feature uses Apache Ivy and thus updating will use the local Ivy cache where appropriate, this means that even if the user deletes the whole libraries folder it can simply be re-deployed from the Ivy cache without having to re-download everything. It also makes maintenance of dependencies and transitive dependencies a breeze because Ivy is awesome.

Because Forge is downloaded by the implementation, it is guaranteed to be the correct version.

If the user downloads a new Sponge and runs the same process, the new Forge will automatically be downloaded but existing dependencies will be used straight from the Ivy cache

META-INF Additions
I have also updated the manifest with the target Forge version and Forge build. The intention is that these data can be used to provide more helpful error messages in other crash messages such as the mixin failure messages.
One-command deployment for platforms
The self-installer also includes a command go which executes the same tasks as install followed by run. This means that for server platforms and other shared hosting solutions there is no "set up" step, the jar can simply be executed and trusted to set up and run Sponge in a single pass.

Pinging @Zidane and @bloodmc

loving this. such a cool idea.

This is only if it is run from commandline, right? It won't catch users who try and run the jar in a GUI environment?

Now it displays this if you start the jar by double-clicking on it in windows or macos:

So pretty.

This is a cool feature, maybe we should move it to the API (jar), then we could just overwrite the help text file in the Sponge. That way we have consistent help/startup-error messages/windows/bevahiour along all sponge jars. (SpongeAPI, SpongeCommon, SpongeVanilla, SpongeForge....)

You know... it would be great if you could provide the same Panel for SpongeAPI's Main...


Migrate build system to the main apalache repo?

We have discussed moving the build system (but not the specs) to the main apalache repo.
Benefits:

Possibility of integrating CI easier
All code in one place
Would make it slightly easier to build from source
Essentially moves us towards a monorepo development practice

Draw backs:

Introduces tech debt (if we don't do a re-write first)
Could encourage tighter coupling between systems that don't really need to be tightly coupled
Takes some labor

Requirements

Would need to rework the build system to pull in the the specs from a remote repo.

Suggested here: https://github.com/informalsystems/apalache-tests/pull/15#pullrequestreview-581837178


Backport memory leak fix to release-0.11 (0.11.3)

Description
Backport memory leak fix to release-0.11 (0.11.3)
Issue reference
#2093
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

End-to-end tests failed on linux. Please check the build logs

This is the false negative. E2E test is passed. this failure is because of current change of e2e test workflow in master.


experimental: patch content of AIMessage generated by OllamaFunctions

OllamaFunctions generates an AIMessage with function call info stored in additional_kwargs field, which will be dropped by the underlying ChatOllama if we send the message back (to get further response like OpenAI though more tweaking is needed).
So a temporary and minimal patch would be to fill the raw response into content field since downstream using hardly depends on it, i.e. typically we use if "function_call" in msg.additional_kwargs to tell if there is a fucntion call involved.

Hey @xleven, I think the better thing to do here would be to automatically extract the function call from the additional_kwargs field and add it to the content here: https://github.com/langchain-ai/langchain/pull/16993/files#diff-f6bebcd7238bb10fc7b6916e17c7e2a4bed4e3a19cf52625dcd8f331815ea970L90
The idea with this class is to mimic OpenAI's behavior as much as possible. So I think it's better to keep the outputted messages the same but pass the inner ChatOllama instance some transformed message instead.

Hey @xleven, I think the better thing to do here would be to automatically extract the function call from the additional_kwargs field and add it to the content here: #16993 (files)

Yeah I dived into it and found under the hood it calls ChatOllama which belongs to langchain-community then not sure if a cross lib update is a good idea.

The idea with this class is to mimic OpenAI's behavior as much as possible. So I think it's better to keep the outputted messages the same but pass the inner ChatOllama instance some transformed message instead.

Totally got the idea and will be happy to see FunctionMessage support in ChatOllama.
Few extra notes about mimicking:

Ollama recently provides OpenAI compatible API
JSON mode in Ollama, which OllamaFunctions is built on, has an equivalent OpenAI API para while functions/tools support usually requires model fine-tuning.

I'll leave it open for further updates but feel free to close.


⚠ Bofrost.nl has degraded performance

In 5702aef, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2791 ms

Resolved: Bofrost.nl performance has improved in a046aad.


Warn on passing non-reference goat.Context

A context is only passed to a goat function if it takes a *goat.Context.
If it is not a pointer/reference, it is not passed and codegen gets messed up.
We need to properly report an error here instead of failing.

Won't do as we're currently dropping the context altogether.


Add searchfox links for files which are in crash stat reports

Crash stat reports do not have the full path to the file, thus we would need some kind of reverse index, or to rely on searchfox to find the matching files.
Also, we should be able to target a specific version given that the repository hash are provided.
[task 2024-01-02T16:14:10.705Z] 16:14:10     INFO -  2  libxul.so!JS::Zone::clearKeptObjects() [Zone.cpp:c44f0d09630a8cf36dac42c4eb31bf2a9395bf90 : 966 + 0x17]
[task 2024-01-02T16:14:10.705Z] 16:14:10     INFO -      rbx = 0x00007f1aa40cb000    rbp = 0x00007ffd075fc300
[task 2024-01-02T16:14:10.705Z] 16:14:10     INFO -      rsp = 0x00007ffd075fc2d0    r12 = 0x0000053b9c8fb488
[task 2024-01-02T16:14:10.705Z] 16:14:10     INFO -      r13 = 0xaaaaaaaaaaaaaaaa    r14 = 0x0000000000000000
[task 2024-01-02T16:14:10.706Z] 16:14:10     INFO -      r15 = 0x0000000000000000    rip = 0x00007f1ab352afd9
[task 2024-01-02T16:14:10.706Z] 16:14:10     INFO -     Found by: call frame info
[task 2024-01-02T16:14:10.706Z] 16:14:10     INFO -  3  libxul.so!JS::ClearKeptObjects(JSContext*) [jsapi.cpp:c44f0d09630a8cf36dac42c4eb31bf2a9395bf90 : 1365 + 0x4]
[task 2024-01-02T16:14:10.706Z] 16:14:10     INFO -      rbx = 0x00007f1aa3f3c000    rbp = 0x00007ffd075fc330
[task 2024-01-02T16:14:10.706Z] 16:14:10     INFO -      rsp = 0x00007ffd075fc310    r12 = 0x0000053b9c8fb488
[task 2024-01-02T16:14:10.707Z] 16:14:10     INFO -      r13 = 0xaaaaaaaaaaaaaaaa    r14 = 0x0000000000000000
[task 2024-01-02T16:14:10.707Z] 16:14:10     INFO -      r15 = 0x0000000000000000    rip = 0x00007f1ab30ea1d8

This can potentially be achieved by running the following search query:
https://searchfox.org/mozilla-central/search?q=&path=<file>&case=true&regexp=false
And collecting any a tag which has the exact same text content as the searched file.
However, the hash cannot be used with searchfox as it corresponds to a git commit, but it does exists in hg:
https://hg.mozilla.org/mozilla-central/file/c44f0d09630a8cf36dac42c4eb31bf2a9395bf90/js/src/gc/Zone.cpp#l966

Apparently this is possible with searchfox using the following url format:
https://searchfox.org/mozilla-central/hgrev/44f0d09630a8cf36dac42c4eb31bf2a9395bf90/js/src/gc/Zone.cpp#966


Improvements to sloped terrain UI feature

For future improvements:

Consider enabling slope inputs for combinations that have no effect. For example, disable when self shading is enabled but backtracking is disabled.
Consider moving to "Shading and Layout" page.

Originally posted by @cpaulgilman in https://github.com/NREL/SAM/issues/742#issuecomment-965967070

@cpaulgilman Let's discuss these changes when looking at system design / land area pieces of the user interface

@mjprilliman I think we decided to 1) enable slope inputs based on their applicability, and 2) leave the slope inputs on the System Design page because they are linked to inputs on both the System Design and Shading and Layout pages, so there is not a perfect place to put them.


Log version of client and server

We should log the version of the client (.NET/JS/whatever) and the version of the server to help diagnose issues when devs create issues with attached logs.

Can pull version in .NET from assembly version attribute. Is the version embedded anywhere in JS?

How do we embed the version number in the TS file? It is not something I've done before.
Lodash does it but I couldn't track down their build scripts.
Thoughts, ideas?

We'd basically do it the same way we do it in any other file. We'd have to just do a simple find-replace for a string in one of our build scripts, such as build/repo.targets

Updated the title to reflect how we approach this: https://github.com/aspnet/SignalR/pull/2229

https://github.com/aspnet/SignalR/commit/f005e80b14055a4ca629a17413c4c8647ccf70a5


Move bot/* commands to consume the boltdb over the REST API

The one downside of boltdb is that it can only be accessed from one process at a time due to how the file is opened with exclusive file locks.
To be able to expose the database to more then just the bot interface, a proxy layer is needed. This will be done via the REST API (already implemented).
The remaining work is to convert the bot commands from using the direct query interface to using the RESTful API.

Complete.


invoke: add --event and --event-file flags

What
Add new event-related flags to the invoke subcommand:

--events: enable printing host/debug events
--events-file: write events to a file instead of stdout

Why

Close #128
Enable saving events to a file/pipe which in turn will be used by soroban-cli serve to subscribe to local events (a new matching --events-file flag will be added to to sorobab-cli for that purpose). This is part of https://github.com/stellar/soroban-cli/issues/39

Known limitations
N/A

2. Enable saving events to a file/pipe which in turn will be used by soroban-cli serve to subscribe to local events (a new matching --events-file flag will be added to to sorobab-cli for that purpose)

The dependency of serve on invoke via a file/pipe seems unnecessary, and unintuitive, and breaks the "these are separate runtimes" model.
I don't think someone running soroban-cli serve should see events from soroban-cli invoke commands. Invoke and serve should be independent of each other, other than sharing the same ledger.json file for data storage if they so choose.
If a transaction is submitted to soroban-cli serve via a HTTP call then it's events should be accessible via the HTTP interface, because the events were generated within that runtime.
But if a contract is invoked via soroban-cli invoke the only place those events should go are to the command line (or to file) for the use of the invoker, not for the use of the HTTP service, because they are both different runtimes and events are ephemeral.
cc @paulbellamy

@2opremio if a developer doesn't supply an --events-file then events won't get plumbed to the serve http server?

@paulbellamy @2opremio The intent with the invoke command was that it could also invoke contracts on remote hosts by building a tx and submitting it to it. Originally that remote host was just going to be a Horizon. But if you'd like the invoke command to invoke a transaction in an RPC server I think that makes sense too. That's how I think it would make the most sense to integrate invoke with a running RPC server (or the serve command which is just an RPC server emulated). In this situation the runtime would be inside the serve, so the events would be generated inside the serve.
Does that achieve what you're wanting to do?

breaks the "these are separate runtimes" model.

I think that's the fundamental disagreement here. I think they should be the same "runtime". If you have an rpc server running against a network, and submit a new txn there via "invoke". You'd expect it to be the same runtime. I don't see why the sandbox should be different. ledger.json is equivalent to a "network".

Got it, interesting.
I think a developer will see the events file like a log file. For example, debug events (called logs in the SDK) are logged as a non-roundtrippable string because the intent is for the developer to write this code:
log!("uh oh: {}, value: {}", symbol!("hi"), 2);

and to see a human readable formatted string, like this:
uh oh: Symbol(hi), value: I32(2)

To make the output roundtrippable we'd have to output the debug event as a JSON object which would make logs not human readable as intended.
I think this means at the very least we need to separate debug events from contract and system events when configuring output destinations, so that we can continue to output human readable logs without needing them to be roundtrippable.
The second thing is that when I look at the CLI interface and see an events file it isn't intuitive to me that it is a communication mechanism.  From the perspective of the invoke I would describe it as just an input and output. Having two applications read and write from the same file also seems like it will have it's challenges with corruption, etc.
I think it is very reasonable to expect that the invoke command can affect the serve runtime and to generate events there, but I think the way it should do that is by submitting a transaction to the serve http endpoint, and then streaming back the results and events from the RPC server in the same way it will do so when invoke a contract with a real remote RPC server.
We could maybe even make this seamless to the developer by writing the serve URL to a file in .soroban/ and having invoke default to a http request to serve when it sees that file there.

re: having soroban-cli invoke do a jsonrpc request to serve if it is running. Seems... complicated to get reliable, and like it could break in strange ways. Doing an internal jsonrpc request seems like a hacky way to re-use code. But, if we need to in order to prevent data corruption, then fine. Maybe a better solution would be to share the implementation code, and have both processes use something like sqlite that would ensure consistency?
As it is now, I don't expect things in the .soroban to be user-friendly/readable, so I'm not worried about the format of things there (though we could change that in future if we want).

having soroban-cli invoke do a jsonrpc request to serve if it is running. Seems... complicated to get reliable

This is the way that the dapp will communicate with the RPC server and the serve emulator, so we need a reliable http server in any case?

a pipe would avoid file consistency issues, at the cost of historical and user readability

Regarding pipes in general: Keep in mind that soroban-cli will be used on five targets, three operating systems across two architectures and pipes come with their own set of challenges such a deadlock, and may not behave the same on all operating systems.
Would the pipe be where the soroban-cli sends the JSON-RPC request? That sounds fine if you prefer that, although I think http will present less issues, and I think this creates additional work unnecessarily.
If the pipe will be a side channel for events only, this seems like a not ideal architecture, and again creates additional work unnecessarily.
The serve already exposes a transaction endpoint, and we're already adding logic to invoke to send a transaction to FutureNet in #42, so in minimal case where a developer points invoke at their local serve it's a zero additional work on top of other work we already have planned.

If the pipe will be a side channel for events only, this seems like a not ideal architecture, and again creates additional work unnecessarily.

That was the idea, using the pipe as an events one-way communication channel from soroban-cli invoke to  soroban-cli serve
If you feel strongly about using an http endpoint instead, that's fine, but please let's make a choice and move forward. We have already spent too much time on this.
@leighmcculloch as I understand it, what you mean is

The user starts soroban-cli serve (listening on, say localhost:8000) , which incorporates a new endpoint (say eventsSend) which can be used to send events
The user subscribes to events subscribes to events by calling the eventsSubscribe endpoint on the running on soroban-cli serve from (1)
The user runs soroban-cli invoke --events-server localhost:8000 ....
The user recieves events notifications from soroban-cli serve (which have been sent from soroban-cli invoke using the new eventsSend endpoint).

Please confirm.
Having to deal with --events-server reduces usability (the pipe location can be implicit) and adding extra http machinery for this seems like a waste of time. However, we have spent way more time discussing this, so I am willing to cave in.

@2opremio That's not what I'm suggesting.
I'm suggesting we build no side-channel for events. I'm suggesting we build what we plan to build already for invoking contracts on remote RPC servers, which serve is a local emulator for, i.e. https://github.com/stellar/soroban-cli/issues/42.
By invoking a contract on the remote RPC server (i.e. soroban-cli serve), events should naturally be generated within the soroban-cli serve runtime, and the invoke command should be able to subscribe to events like it would any remote RPC server or dapp.
To break it down:

Use runs:soroban-cli invoke --rpc-url http://localhost:8000 --account <G...> --id <CONTRACTID> --fn <FN> --arg ...

Contract is invoked within soroban-cli serve, in the same way the dapp will invoke a contract within serve.
soroban-cli serve will produce events in the same way it will when the dapp invokes contracts on serve.
soroban-cli invoke collects events from serve like a dapp would.

i.e. I'm suggesting we build nothing unique / bespoke for sandbox invoke with sandbox serve, and instead rely on features we already have to build for the dapp to serve integration.

I don’t fully understand what needs to be done from your description. Do you mean that #39 shouldn’t be implemented at all?
Please be clear on the mechanics of —rpc-url. Does this mean serve will also implement the sandbox emulation?

Does this mean serve will also implement the sandbox emulation?

serve is an implementation of the soroban-rpc. It exposes the same API and is backed by the ledger.json, or what we have been calling the sandbox.


Tx are not showing up in Polygon prod or being executed at all

Issue:
New tx are not showing up in the Queue tab for safes with 2 owners needed NOR being executed when only 1 owner is needed
Safe:
https://polygon.gnosis-safe.io/app/#/safes/0x39A4D23652d40AF13812E1d01C4aF65559Dd6850/transactions
is a safe with "2 owners" policy
The safe should have 3 tx's in the queue tab currently
Safe2:
https://polygon.gnosis-safe.io/app/#/safes/0x97702cf24fE1AEc59db9D617eE1Ac21D42c8BC51/settings
This safe has a "1 owner" policy
I created and executed a change in policy. That tx never showed up in the History tab and the policies remain the same.

The tx are now showing up, but it took over 40 minutes to do so

I'm unable to reproduce this. That second Safe has 2 owners for me.  -> Closing @francovenica pls reopen in case it persists.


Cannot read property '_t' of undefined with modals and vue-i18n 7.3.3

Version & browser
uiv 0.15.0, vue-i18n 7.3.3, Chrome 63.0.3239.108
Which component
Modals and MessageBox
Steps to reproduce the problem
Include a Modal or MessageBox component on the page
Expected behavior
No errors
Actual behavior
"Cannot read property '_t' of undefined" is thrown with vue-i18n 7.3.3. It works fine with vue-i18n 7.3.2.

I just created an issue to vue-i18n: https://github.com/kazupon/vue-i18n/issues/264
pls use 7.3.2 for now, before I found any solution from there.

0.15.1 fixed this.

Getting Cannot read property '_t' of undefined in vue-i18n v7.4.2.
Version
Vue - 2.9.3
Nuxt - 1.3.0

TypeError: undefined is not an object (evaluating 'i18n._t')
uiv 0.26.2
vue-i18n 8.1.0


Job control flow chart

Not quite done yet. I tried to figure out the syntax tro add images and looks like this is broken on th rendred version atm?

I was just updating the simulator tutorial and noticed the same problem. The mechanism for linking is pretty straightforward, however, I had originally put all image assets in a folder called icons/ which is git ignored.
I can't seem to find it ... I wonder if it was on the grigoriefflab google drive? I (obvi) don't have access to that acct anymore. I'll keep poking around.


Cannot handle non-ASCII characters

Here's two more or less identical python scripts, one of which contains UTF-8 encoded characters:
$ cat test1.py
# -*- coding: utf-8 -*-
"""abc"""
$ cat test2.py
# -*- coding: utf-8 -*-
"""æøå"""
$ python test1.py && echo ok
ok
$ python test2.py && echo ok
ok
$ isort test1.py && echo ok
ok
$ isort test2.py && echo ok
Traceback (most recent call last):
  File "/usr/local/bin/isort", line 43, in <module>
    SortImports(file_name, **arguments)
  File "/usr/local/lib/python2.7/dist-packages/isort/isort.py", line 73, in __init__
    self.in_lines = file_contents.split("\n")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 27: ordinal not in range(128)
$

Boom!

Hi!
Sorry to re-open such an old thread.
I just ran  into this problem when running isort on OSX yosemite, with python 3.4.2 and isort 3.9.4
The problems seems to be that OSX doesn't have LANG set to anything by default, so when isort opens the file to check (line 94), it doesn't specify an encoding, which then default to "ASCII" on OSX.
If LANG is set to something like en_US.UTF-8 the problem goes away, but will break in systems where for any reason changing the locale is not an option.
The second link I put here talks about a “Unicode sandwich” approach. That might solve the problem.
http://robjwells.com/post/61198832297/get-your-us-ascii-out-of-my-face
http://nedbatchelder.com/text/unipain.html
Traceback (most recent call last):
  File "/Users/fjm/Development/acora_dj/venv/bin/isort", line 11, in <module>
    sys.exit(main())
  File "/Users/fjm/Development/acora_dj/venv/lib/python3.4/site-packages/isort/main.py", line 119, in main
    incorrectly_sorted = SortImports(file_name, **arguments).incorrectly_sorted
  File "/Users/fjm/Development/acora_dj/venv/lib/python3.4/site-packages/isort/isort.py", line 95, in __init__
    file_contents = file_to_import_sort.read()
  File "/usr/local/Cellar/python3/3.4.2_1/Frameworks/Python.framework/Versions/3.4/lib/python3.4/encodings/ascii.py", line 26, in decode
    return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 2709: ordinal not in range(128)


docs(action): 修复文档修改流程

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Suggest: '(& f)' -> 'f'

Thanks for the pull request!

If you care, my PR preferences are at https://github.com/ndmitchell/neil#contributions, but they're all guidelines, and I'm not too fussy - you don't have to read them.

Mind you, I am totally new to hlint internals.  But indeed, the existing
rule for Data.Function.& (which I missed) does not fire for my test
case:
***@***.***:~/Projects/Haskell/hlint$ cat test.hs
module Main where

import Control.Lens

main :: IO ()
main = do
  print $ (& (+ 1)) 0

***@***.***:~/Projects/Haskell/hlint$ git checkout master
Switched to branch 'master'
Your branch is up to date with 'upstream/master'.
***@***.***:~/Projects/Haskell/hlint$ hlint --hint data/hlint.yaml test.hs
No hints
***@***.***:~/Projects/Haskell/hlint$ git checkout redunant
Switched to branch 'redunant'
Your branch is up to date with 'origin/redunant'.
***@***.***:~/Projects/Haskell/hlint$ hlint --hint data/hlint.yaml test.hs
test.hs:7:11-19: Warning: Redundant &
Found:
  (& (+ 1))
  Perhaps:
    (+ 1)

1 hint

--
CYa,
⡍⠁⠗⠊⠕

I've removed the Data.Function qualification. At the beginning & was a bit of a rare function, so I wanted to be careful about when it triggered. Now it's much more common, so I think relaxing it makes a lot more sense. Thanks for the suggestion!


Use oopsy-data for warnings

oopsy-data will make the warnings API much richer and easier to use for the console client or the try site.

This was fixed in the last update.


v3 - LocalFile, StructuredText, Slices data returning null

Hey @angeloashmore
Seems like we are getting issues when we upgraded from 2.3.0-alpha.6 to 3.0.0-beta.18.
Here are our relevant dependencies:
    "gatsby": "^2.5.0",
    "gatsby-image": "^2.0.29",
    "gatsby-plugin-sharp": "^2.0.20",
    "gatsby-source-filesystem": "^2.0.20",
    "gatsby-source-prismic": "^3.0.0-beta.18",
    "gatsby-transformer-sharp": "^2.1.13",
    "react": "^16.8.0",
    "react-dom": "^16.8.0"

and the gatsby-config.json
  {
      resolve: 'gatsby-source-prismic',
      options: {
        repositoryName: '<REPO-NAME>',
        accessToken: `<ACCESS-TOKEN>`,
        linkResolver: () => (post) => `/${post.slug}`,
        htmlSerializer: () => (type, element, content, children) => {},
        schemas: {
          examplePage: require('./src/schemas/example-page.json')
          ...
        },
        lang: '*',
        shouldDownloadImage: () => true
      }
    },

When we try the following
{
  ...
  imageFieldName {
     localFile {
      childImageSharp {
        fluid(maxWidth: 750, quality: 80) {
          ...GatsbyImageSharpFluid_withWebp
         }
       }
     }
   }
}

We see this type error TypeError: Cannot read property 'childImageSharp' of null as localFile is null. Also, not only is localFile, our other fields return as null. However, when I check the raw data from GraphQL, we can see that the fields exist (but we still do not see localFile).
We have tried reverting the gatsby-source-prismic plugin to various v3 versions and we see the issue. We have also updated the gatsby plugin versions, but we still see the same issue. Finally we've tried gatsby clean and deleting and reinstalling node_modules

related to #161
#135

Hi @TatsumiSuenaga, thank you for reporting this issue. I'm going to close this one in favor of #161, but please feel free to add to that issue if you have more to share.
If you're able to share a JSON schema of the custom type that is failing, I can test it out.


python web.py reports ImportError

Hello,
I'm quite new to this. Don't understand python very well. Just followed the instructions here and everything worked well until i tried opening the web using 'pyton web.py' ,which reported the below error [attached png]. Can you let me know how to fix this ?
I run Windows 7, with python 2.7.13.

You guys doing a great job. Very useful stuff for beginners like me.
Thanks
Ashwin

Did you install flask module in your virtual environment?

I just followed the installation instructions on the page. Didn't know it
wouldn't install these modules by default. Could you let me know any
particulars of it.
Thank you
Ashwin
On Sun, Sep 2, 2018, 8:13 PM Daxeel Soni<EMAIL_ADDRESS>wrote:

Did you install flask module in your virtual environment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/daxeel/blockshell/issues/20#issuecomment-417935671,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ao7EICTIx92dz1152jWS8dvPPF3QFrqwks5uW-6fgaJpZM4WWmhg
.

Install flask in your virtual environment and then try again

@ashwinanand2526 is this resolved? If yes, close this issue. Thanks.


Command `\Zbar' already defined.

When compiling with Mac , faced with problem
Command [{ 	"resource": "/usr/local/texlive/2022/texmf-dist/tex/latex/newtx/newtxmath.sty", 	"owner": "LaTeX", 	"severity": 8, 	"message": "Command \Zbar' already defined.",
"source": "LaTeX",
"startLineNumber": 928,
"startColumn": 1,
"endLineNumber": 928,
"endColumn": 65536
}]
I solved this by simply delete the Line
\DeclareMathSymbol{\Zbar}{\mathord}{lettersA}{94}

Thanks for the help. Hope the graduation period goes well!


Cant mount component with required: true props

If I try to mount a component with:
    const id = 'hello';
    const url = 'bad thing';
    const wrapper = mount(Dropzone, {
      propsData: {
        id,
        url,
      },
    });

It fails to create it, because the component requires props to have a value. If I mount it using:
const getComponentWithProps = (component, propsData) => {
  const Ctor = Vue.extend(component);
  return new Ctor({
    propsData,
  });
};
    const id = 'hello';
    const url = 'bad thang';
    const wrapper = getComponentWithProps(Dropzone, { id, url });

The wrapper has the instance of my component created. Any help? =X

Hi @pablohpsilva, I'm busy most of today but will take a look tomorrow and try to get a fix up asap

No worries. Thanks for this amazing project!

Hi @pablohpsilva. The example you provided creates an unmounted Vue component, which doesn't contain a vNode. Without a vNode most avoriaz methods won't work. If you try to call $mount on the Vue component, you will have the same problem as mounting it with avoriaz (an Error will be thrown).
What's the reason you want to mount a Component with required props without specifying the props when mounting?
As far as I understand - you can't mount a component that has a required prop if the prop isn't passed in propsData? Why can you not pass the prop?
Can you post the component you're trying to mount?

Hello @eddyerburgh :D
Let me give you a real example:
export const getComponentWithProps = (Vue, component, propsData) => {
  const Ctor = Vue.extend(component);
  return new Ctor({
    propsData,
  });
};

const drop = getComponentWithProps(Vue, Dropzone, {
  id: 'some1231id',
  url: 'https://github.com',
}).$mount();

Like I shown before, I am passing the props the component requires, but its not working :(
Is there a way to make it work with avoriaz?

Does this work:
avoriaz.mount(Dropzope, {
  propsData: {
    id: 'some1231id',
    url: 'https://github.com',
  }
})

That's really weird... I can swear I tried that before and now everything worked out.
Wow... OK! That's awesome! Thank you for your time and amazing tool @eddyerburgh ! :D


Small Article Img Update

1. What type of PR is this? (check all applicable)

[X] ♻ Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🔖 Release
[ ] 🚩 Other

2. Description

3. Relevant Documents, Websites, & Issue Numbers

4. Mobile & Desktop Screenshots/Recordings

5. Add tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

6. Remarks, notes, or any future suggestions?
7. Add to the documentation(s)?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 docs
[ ] 📕 storybook
[ ] 🙅 no documentation needed

8. Are there any post-deployment tasks we need to perform?
9. What gif best describes this PR or how it makes you feel?

Yup!


Add support for specifying ETHTOOL_OPTS

Description
Allow specifying ETHTOOL_OPTS via ethtool_opts on Red Hat
Issues Resolved
None - improvement
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

@chef/client-core needs another review and a merge

I'll merge once the tests go green.

Cheers guys – I think I have fixed the unit tests now. Thanks for all the help – please let me know if there’s anything additional I need to do to get these changes in!
From: Adam Leff<EMAIL_ADDRESS>Sent: 07 September 2017 21:29
To: chef/chef<EMAIL_ADDRESS>Cc: Tom Doherty<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [chef/chef] Add support for specifying ETHTOOL_OPTS (#6384)
@adamleff requested changes on this pull request.
Looks like you added some more stuff here (which still looks good) but you have some failing unit tests that need to get squared away first. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chef/chef/pull/6384#pullrequestreview-61342652, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeRZJtN7loja1YFwXzWewBKvy6D2NAAkks5sgFH6gaJpZM4PQMKp.

Merging this change isn't why the proxy tests are failing

@tomdoherty Congrats on your first contribution

Thank you! Can't wait to roll it out accross our fleet :-)
Thank you to all they guys on Slack for their constant support and patience!


Between and DateTime filters

fix JS
now you use
protected $js = [
    **'moment/min/moment.min.js',**
    'eonasdan-bootstrap-datetimepicker/build/js/bootstrap-datetimepicker.min.js'
];

change to
protected $js = [
    'moment/min/moment-with-locales.min.js',
    'eonasdan-bootstrap-datetimepicker/build/js/bootstrap-datetimepicker.min.js'
];

and remove
$js[] = "moment/locale/{$options['locale']}.js";

ok  I'll fix this later.

issue fixed  thx!


New installer (1.1.7) displays 1.1.5 as current version

My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[ ] Windows 8.1

[x] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[x] My Windows installation is non-English.

I'm using NVM4W version:

[x] 1.1.7
[ ] 1.1.6
[ ] 1.1.5
[ ] 1.1.4
[ ] 1.1.3
[ ] 1.1.2
[ ] 1.1.1
[ ] Older
[ ] OTHER (Please Specify)

Expected Behavior
nvm version should print 1.1.7. in the latest release.
Actual Behavior
It prints 1.1.5, which was the former installed version on my system. The installer itself shows 1.1.7.
I tried to completely  uninstall the previous version, with no success.

It could be that nvm uninstall 10.X.X stills not function in this version ?

yes, I have the same problem.
and use nvm download nodejs 10.8.0,  threr is a error, it display can not install npm.

Reproduce this.

version shown as 1.1.5 instead of 1.1.7 and can't install npm

Yep. Same here. So I just uninstalled 1.1.6 and all my pre-compiled stuff for nothing.

Duplicate of #374


Accelerating video on mpv does not lead to accelerated playback for other participants.

Describe the bug
When I accelerate or slowdown video with mpv, I expect syncplay to duplicate those actions across other participants. However, it resyncs and resets speed back to 1x instead.
To Reproduce

Start two instances of syncplay, join the same private room. mpv starts along with syncplay's window.
Add a file from media folder to playlist and playit.
Start playing the file.
In one of mpv's window, press } key to speed up video 2x.

Expected behavior
Other mpv also switches playback speed to 2x without explicit keyboard input.
Actual behavior
Other mpv still plays video at 1x. syncplay detects it and rolls back video speed to 1x.
Version and platform:

OS: Linux Debian Bullseye
Syncplay version and build type: 1.6.7 (Yoitsu)
Media player and version: mpv 0.32.0

Additional context
Add any other context about the problem here.

I'm afraid nothing has really changed since this was discussed in #339
Syncplay isn't designed to sync playback speeds across and adding this feature would need very significant amount of effort. Possibly up to including rethinking entire syncing algorithm.

@daniel-123 has summarised it well and linked to relevant discussions so I'll not go into details. In summary, while I can see why this would be a neat feature it is a big development headache due to how Syncplay and its protocol have been designed.


Update Schema.php

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
?

Tests pass?
?

Fixed issues
fix #18507

Looks good. Please add a line for CHANGELOG. Also, is it possible to cover it with a test?

Perhaps after weekends. I'm not so well familiar with your testing suite and rules

Thank you! Merged.


contributing docs: Update "Version Control" > "Commit discipline" links.

Updates .html links as a follow-up to #23750.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Links.

@timabbott quick follow-up ready for your review!

@timabbott  I have not reviewed these changes.

Merged, thanks @drrosa!


How do I input an open-vocabulary text

Hello, thank you for developing a good work, i am really interested in it. But I want to know how do I input an open-vocabulary text description like the example you gave?

Thank you for your interest in our work. We have not yet released the single detection testing code and plan to do so around early next month with the latest version of the paper. We will inform you in due course.
Best,
Zhening

Thank you for your response and the update. I look forward to seeing the single detection testing code and the latest version of the paper.

We have just released a newer version of the code, which showcases how this is done under
zero_shot_single_voc.py.
If you look into this code, you will see how the results in the teaser images are detected. Essentially, we designed a CLIP ranking and filtering system to effectively reduce false positive detections for single-vocabulary open-world inference.
Best,
Zhening


Introduce EmbeddedDocument

Can be merged after

#85

what do you think about like a tupel of document classes:
TextDocument and VectorDocument - both have an UUID and Metadata
/**
 * @param TextDocument|TextDocument[] $documents
 */
DocumentEmbedder::create(TextDocument|array $documents, ...): void

// only converting than

$vectorDocuments[] = new VectorDocument($vectors[$i], $document->id, $document->metadata);

// and drop text in store context completely

/**
 * @param VectorDocument|VectorDocument[] $documents
 **/
StoreInterface::add(VectorDocument|array $documents): void;

/**
 * @return VectorDocument[]
 */
VectorStoreInterface::query(Vector $vector, array $options = []): array;

I like it, I thought sth like this without inheritance too in the beginning

No I don't think we need that abstraction upstream

you want to tackle that with your PRs or shall I?

If you have time, go ahead. Either use my PR or open a new one where you do the transformation completely

Superseded by

#110


Service initiated in main does not display Ballerina connector exception

Description:
When a service is initiated in the main like this:
import ballerina/log;
import ballerina/http;

@final string REMOTE_BACKEND = "ws://echo.websocket.org";

function main(string... args) {
    endpoint http:WebSocketClient wsClientEp {
        url: REMOTE_BACKEND,
        callbackService: ClientService
    };
    wsClientEp->pushText("{'id':123,", final = false) but {
        error e => log:printError("Error occurred when sending text message", err = e)
    };

    wsClientEp->pushText("'name':'Riyafa'}", final = true) but {
        error e => log:printError("Error occurred when sending text message", err = e)
    };
}
service<http:WebSocketService> ClientService {

    onText(endpoint caller, string text, boolean finalFrame) {
        log:printInfo(text);
    }
    onClose(endpoint caller, int status, string reason){
        log:printInfo(status+": "+ reason);
    }
}

A BallerinaConnectorException is thrown by the client endpoint because of incorrect service type. But this fails silently and no error is logged in the console.
But if it is initiated in a service resource the thrown error is logged in the console:
import ballerina/log;
import ballerina/http;

@final string REMOTE_BACKEND = "ws://echo.websocket.org";

service<http:Service> httpservice bind { port: 9090 } {
    hello(endpoint caller, http:Request request) {
        endpoint http:WebSocketClient wsClientEp {
            url: REMOTE_BACKEND,
            callbackService: ClientService
        };
        wsClientEp->pushText("{'id':123,", final = false) but {
            error e => log:printError("Error occurred when sending text message", err = e)
        };

        wsClientEp->pushText("'name':'Riyafa'}", final = true) but {
            error e => log:printError("Error occurred when sending text message", err = e)
        };
        _ = caller->respond("hey");
    }
}
service<http:WebSocketService> ClientService {

    onText(endpoint caller, string text, boolean finalFrame) {
        log:printInfo(text);
    }
    onClose(endpoint caller, int status, string reason) {
        log:
        printInfo(status + ": " + reason);
    }
}

Error logged is:
error: ballerina/runtime:CallFailedException, message: call failed
        at httpservice.hello(server_client.bal:8)
caused by error, message: incorrect endpoint: ballerina/http:WebSocketListener
        at ballerina/http:init(websocket_client.bal:45)

In both cases error should be logged and should not fail silently

This was due to exceptions thrown during main function execution being ignored if services exist, at https://github.com/ballerina-platform/ballerina-lang/blob/v0.981.1/bvm/ballerina-core/src/main/java/org/ballerinalang/BLangProgramRunner.java#L73.
This is now fixed on master with #10060.


[Web] Have a fallback website for checking

This should apply to devices that don't support the app

This links to #309

This links to #309


Remove --bootstrap-version command line option

description
once v2 has been fully removed the bootstrap-version no longer makes any sense (see https://www.envoyproxy.io/docs/envoy/latest/operations/cli#cmdoption-bootstrap-version)
apart from removing the cli option and related docs i think we also need to ensure that the logic for loading as v2 and upgrading is also removed
there is (also) some stuff in api/envoy/config/core/v3/config_source.proto that may need to change

cc @htuch

Yeah, we should probably do this at some point, let's have the dust settle first. https://github.com/envoyproxy/envoy/pull/16274 stopped users doing anything dangerous here, so I think maybe clean this one up after next release.

Still available?

Sure

Yeah, we should probably do this at some point, let's have the dust settle first. #16274 stopped users doing anything dangerous here, so I think maybe clean this one up after next release.

Is time to clean it? If so, I would like to take.

If you want to work on the PR, we can merge it after 1.19 ships in early July.

got, would do it after 1.19.
/assign

Will this option make sense again as the v4 API matures?

@jpeach postmortem pending, but I think based on  v2 -> v3 experiences, v4 is very far away (at least multiple years) if it happens at all.

Fixed in #17724


Licence terms and forks

This is lovely, @afex! I was planning to write something like it for the Elastic stack. Would it be OK if I started with a fork of your work? I didn't see an explicit licence, so wanted to check with you.
Cheers.


protect_content Parameter

In all send method in bot we can set protect_content but i don't find in package
please add it

@fabio-ivona

->protected()

Thank You very much


Class extends value [object Object] is not a constructor or null

Just had issues adding<EMAIL_ADDRESS>to a new ember app (4.6.0), on start throws this error Uncaught TypeError: Class extends value [object Object] is not a constructor or null
This was being thrown from the LinkToExternal component.
I realized that LinkComponent was removed from @ember/routing package starting at ember v4, and that the package @ember/legacy-built-in-components is required for ember-engines to work.
After adding @ember/legacy-built-in-components my new ember app loaded.
This isn't stated anywhere in the ember-engine docs, shouldn't this be noted in the installation section or somewhere in the docs?
Note: I installed ember-engines using yarn yarn add ember-engines and not ember install ember-engines ~ unsure if this matters?

@roncodes thanks for logging the issue! This is indeed the miss in the documentation.
@roncodes btw, were you using https://ember-engines.com/docs/installation during your endeavor? I think that's the page where such info should be provided.

@SergeAstapov Yes I was using the docs installation page. I can create if a PR if it's needed or should indeed be included in the installation instruction for v4+ users.

Same issue as https://github.com/ember-engines/ember-engines/issues/795

fixed as of 0.11.0
fixed by #855


Slides get stuck at the end when dragging mouse

This is a:

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 5.3.0
You can see this on Swiper's own demo.
https://swiperjs.com/demos/110-slides-per-view.html

What you did
Use the mouse to drag slides, when you get to the end the slides get stuck and the next navigation arrow isn't in the disabled state.
Expected Behavior
To spring back like it does when you try and go back from the first slide.
Actual Behavior
The slides get stuck and the next navigation arrow isn't in the disabled state.

Same problem here.
Thanks @francesco-bizen, adding slidesPerGroup with same value of slidesPerView solved, for me.
Anyway, waiting for a proper bugfix from Swiper in the next release.

Adding slidesPerGroup with same value as slidesPerView prevents carousel from moving in any direction when the value is 'auto'.

I can also confirm that this bug is occuring. In my instance I am using slidesPerView: "auto" so the slidesPerGroup fix won't work for me.
I had a quick check of @codesignist's PR #3414 and it seems to fix the issue.

@codesignist's PR #3414 fixed the issue for me

Fixed by #3414

Looks like the bug just reappeared again : unable to reach the last 3-5 slides 🤔

@nolimits4web

I have the same problem.

setting touchReleaseOnEdges to true solves my problem

I solved it by adding touchStartForcePreventDefault

setting touchReleaseOnEdges to true solves my problem

This fixed mine thanks for this

You can reproduce the bug if you set a margin on the x axis on each swiper-slide.
I just make sure to use spaceBetween instead, that was it for me.

Setting freeMode to true solves my problem


freerdp should store by ip/port the key and not only by ip

Our company has several server  with one public ip address so we have
IP:portA -> sever A
IP:portB ->  server B
and freerdp will refuse to connect to the second , as he think server B is not legit

I ran into this issue with Remmina. The error message does not say why. The workaround is to create different FQDNs for each connection, which is really stupid.

Already implemented.


Ensure test graphdriver plugin runs on test host

Sets a kernel requirement for for TestGraphdriverPlugin since the
graphdriver being used is overlay2.. and also makes sure to skip the
kernel check in the actual graphdriver since we may be able to detect
kernels with backported support for overlay2 style mounts a bit more
freely in the test code.
Fixes #29505

LGTM


Add pipeline to remove out of date docker images in ACR.

@liushilongbuaa , please add the description, thanks.


poll_event() never returns Pending state, causing the examples to fail

Since the introduction of the runtime agnostic code (I guess!) the poll_event() function no longer returns Poll::Pending, only Poll::Ready, possibly with a Err(Error::EAGAIN) value. This makes the examples fail with
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Unexpected(Resource temporarily unavailable)', examples/reply.rs:12:19

Yep, it shouldn't do that.

Sorry for the late update. I have update how event is polled. Now it will return Poll::Pending if socket doesn't contain the event we want.
Thanks for reporting the issue!


Two pages spread misaligns next pages when reading on landscape mode.

When reading on Kindle with two-page spreads, rotating or no rotating, causes the pages after to become misaligned since the spread counts as 1 location(page) on Kindle instead of the 2 it would count on a physical copy. Can we have the option to duplicate the spread or add a blank page after it so this doesn't happen?

@e1ker are you using the latest v5.6.5?

@e1ker are you using the latest v5.6.5?

When this specific manga was converted it was with v5.6.4, but I didn't see anything on the changelog for v5.6.5 that would fix it

Can you upload pictures of the issue? I can't understand what is being misaligned after a spread.

I use the rotate and split option with a full checkmark, which I think should do both? When using portrait it is a single rotated page as I expected but when in landscape it is just a single rotated page with another page next to it. Also, if I rotated AND split, would that make the picture not work properly in portrait and show as 2 pages instead? I'm a bit confused about how the option works

@e1ker try the option only partially checked and see if it does what you want.

@e1ker Finally got to my PC, do it like this:

A problem exists with rotate and split in which some comics in landscape show one singular page after a splitted spread, misaligning again the pages

Can you show an example picture of a comic in landscape only showing a singular page after a split spread using the partially checked option?

This is the page shown as a single. Following it are these pages:

Note that the white margins are looking to the outside, while the proper way should be looking to the inside

What's the page right before the single page?

What's the page right before the single page?

This spread:

Here it is rotated.

After that is a split spread (these are 2 pages but shown together since it's in landscape)

This is how the last page of the spread appears in portrait. That's expected behaviour since it's a split.

Thank you for the information, I've replicated it. Can you send a screenshot of your KCC settings? I want to confirm it's a bug with the manga mode checkbox being unchecked This bug was never caught since most people read right to left content.

@e1ker This is now fixed in master. A release will come out eventually if you can't run the code directly.


Get notification

Hi @Fleker
I am not sure if this is a bug:
https://stackoverflow.com/questions/72880033/sensorstate-smoke-sensor-notification

Please see stackoverflow for the response to this issue. Thanks. Closing out.


Add support to Rails 7 and Ruby 3

set graphql dependency to rails 7
update ruby version to version 3

Hi @davidsilveira,
Do you have an estimate on when this PR will be merged?

@maartenesser I'm using this code in a project already and haven't had any problems yet.
Gemfile:
gem 'graphql-auth', git: 'https://github.com/davidsilveira/graphql-auth.git', branch: 'rails-7'


chore: Replace deprecated command with environment file

Signed-off-by: jongwooo<EMAIL_ADDRESS>Description
Resolve  #3920
Update workflows to use environment file instead of deprecated set-output command.
For more information, see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
I found the workflow files that use set-output command through the following command:
$ find .github/workflows -name '*.yml' | xargs egrep '\bset-output\b'

AS-IS
run: echo "::set-output name=hashes::$(cat zip.sha256 | base64 -w0)"

run: echo "::set-output name=date::$(date +'%Y-%m-%d' --utc)"

TO-BE
run: echo "hashes=$(cat zip.sha256 | base64 -w0)" >> $GITHUB_OUTPUT

run: echo "date=$(date +'%Y-%m-%d' --utc)" >> $GITHUB_OUTPUT

@emmileaf Can you also review this change whenever you get a chance? Changes look good to me.

LGTM - Thank you @jongwooo for the contribution!


Not checking for and downloading/creating directories/files that might be referenced later

Hi Andrew,
Love your work, keep it up!
At the moment I'm having an issue with the script, I'm using for the first time and getting the below errors:

17/02/2022 5:05:53 PM | Syncing EvtxECmd with GitHub for the latest Maps
&: J:\Triage\KAPE\KAPE-EZToolsAncillaryUpdater.ps1:62
Line |
  62 |  & "$currentDirectory\ZimmermanTools\EvtxExplorer\EvtxECmd.exe" --sync
     |    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'J:\Triage\KAPE\ZimmermanTools\EvtxExplorer\EvtxECmd.exe' is not recognized as a name of a
     | cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was
     | included, verify that the path is correct and try again.

17/02/2022 5:05:53 PM | Syncing RECmd with GitHub for the latest Batch files
&: J:\Triage\KAPE\KAPE-EZToolsAncillaryUpdater.ps1:68
Line |
  68 |  & "$currentDirectory\ZimmermanTools\RegistryExplorer\RECmd.exe" --syn …
     |    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'J:\Triage\KAPE\ZimmermanTools\RegistryExplorer\RECmd.exe' is not recognized as a name of a
     | cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was
     | included, verify that the path is correct and try again.

17/02/2022 5:05:53 PM | Deleting .\KAPE\ZimmermanTools\SQLECmd\Maps for a fresh start prior to syncing SQLECmd with GitHub

Which also are related to the errors below:

17/02/2022 5:05:56 PM | Copying EvtxECmd, RECmd, and SQLECmd and all associated ancillary files to .\KAPE\Modules\bin
Copy-Item: J:\Triage\KAPE\KAPE-EZToolsAncillaryUpdater.ps1:88
Line |
  88 |  & Copy-Item -Path $PSScriptRoot\ZimmermanTools\EvtxExplorer -Destinat …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path 'J:\Triage\KAPE\ZimmermanTools\EvtxExplorer' because it does not exist.

17/02/2022 5:05:58 PM | Copied EvtxECmd, RECmd, and SQLECmd and all associated ancillary files to .\KAPE\Modules\bin successfully
17/02/2022 5:05:58 PM | Copying remaining EZ Tools binaries to .\KAPE\Modules\bin
Copy-Item: J:\Triage\KAPE\KAPE-EZToolsAncillaryUpdater.ps1:107
Line |
 107 |  & Copy-Item -Path $PSScriptRoot\ZimmermanTools\ShellBagsExplorer\SBEC …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path 'J:\Triage\KAPE\ZimmermanTools\ShellBagsExplorer\SBECmd.exe' because it does not
     | exist.

17/02/2022 5:06:00 PM | Copied remaining EZ Tools binaries to .\KAPE\Modules\bin successfully

A few things I noticed:

line 58 to 84 We are not checking that the executables exist in the first place, just assuming they are. This can be fixed with an if statement and download of the needed/missing files
On some of the lines like 88 you have the & in front of the Cmdlet. I don't believe you need this. Is there a reason for that? It's pretty similar to saying Invoke-Expression "Copy-Item -Path $PSScriptRoot\ZimmermanTools\EvtxExplorer -Destination $binPath\EvtxECmd -Recurse -Force" but Copy-Item will work fine on its own.
There isn't much error handling, but that's ok! We can work to fix that
For a first script, awesome work!! Much better than my one liner I saved as a .ps1 file
I think there might be other stuff, but I'm happy to put it all in a pull request if that helps?

P.S I'm neither programmer nor expert, but if there is something I think I can improve I will let you know.

@Donovoi sorry for the delay, I must've accidentally deleted the email notification for this Issue being created. Truly apologize!
I'm working on version 3 of the script right now. Hope to have it up shortly. I will hold off on addressing any of this until that's out as it may solve some of those issues you're seeing!

Sounds good thanks @AndrewRathbun!

Sounds good thanks @AndrewRathbun!

https://github.com/AndrewRathbun/KAPE-EZToolsAncillaryUpdater/releases/tag/3.0
Please provide feedback when able!

3. Also we are downloading the maps every time. Is there a way to check that we are up to date and not download them? Maybe a checksum of a zip? It might be an issue I need to raise with whoever makes EvtxECmd.exe

that would be me. the maps are small enough it doesnt warrant the amount of work to checksum the zip, upload the new one, etc.

Awesome thank you @AndrewRathbun & @EricZimmerman I understand why those choices were made, and they make sense!
I still think we need to check if the executables are in the folder before trying to move them, to prevent the error as per:

Running the script the first time worked well with no errors. Running it the second time produced errors. Have a look at my Output below:

My thoughts here are we don't need to run the Move-EZToolsNET6 function if we have already got the latest tools in the correct folder. But you can just add a check in Move-EZToolsNET6 on line 348 to see if the tools are in the $kapeModulesBin\ZimmermanTools\net6\ folder.

And we should be logging the error to the log file you've set up.
If you agree to the two things above, I'm happy to do a pull request if it will help.
Just lastly - If I'm at a remote location doing triage on a live machine - and for some reason one version of the binaries don't work, I'd like to be able to try the other version and have it already available to me (especially if I'm in an offline environment).
This won't be an issue I guess if .NET 6 binaries are backwards compatible, or we publish all exe's as Single File Apps as per https://dotnetcoretutorials.com/2021/11/10/single-file-apps-in-net-6/
I'll post this to Eric's repo as something to consider (if he hasn't already).

no way i am building 60 Mb EXEs to download if net 6 is not installed. people can do that if they want, but thats crazy to me.
net 6 stuff is not backward compatible. you need .net 6 runtime for them to work. the net 4 stuff should work just about everywhere tho

Awesome that makes sense thank you Eric!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Eric @.>
Sent: Thursday, March 17, 2022 6:40:17 AM
To: AndrewRathbun/KAPE-EZToolsAncillaryUpdater @.>
Cc: Michael Moran @.>; Mention @.>
Subject: Re: [AndrewRathbun/KAPE-EZToolsAncillaryUpdater] Not checking for and downloading/creating directories/files that might be referenced later (Issue #2)
no way i am building 60 Mb EXEs to download if net 6 is not installed. people can do that if they want, but thats crazy to me.
net 6 stuff is not backward compatible. you need .net 6 runtime for them to work. the net 4 stuff should work just about everywhere tho
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAndrewRathbun%2FKAPE-EZToolsAncillaryUpdater%2Fissues%2F2%23issuecomment-1069544614&data=04|01||698fe2bae38b45a1225108da0784cd45|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637830564205100390|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=7BYxIDSUC0N%2BB2DhfL252Off2jcp2n0r3ITSC0LKktc%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAF5EQS6KU3VFZ2OHO2PR6MLVAI2KDANCNFSM5OT5SWHA&data=04|01||698fe2bae38b45a1225108da0784cd45|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637830564205100390|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cYyF01wXNPtciqHYg%2BQrOE%2Fjqos0HwbwXOEPqsSViII%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Any error handling you want to add, I ask you simply use the Log function that's already being used script-wide to keep things consistent.
In regards to checking for if the executables are there before we move them, if you wanna do a PR with Test-Path and all that other fun stuff, by all means! When merged, I will resign and republish.

Awesome thanks Andrew! I'll get that done
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Andrew Rathbun @.>
Sent: Thursday, March 17, 2022 7:02:21 AM
To: AndrewRathbun/KAPE-EZToolsAncillaryUpdater @.>
Cc: Michael Moran @.>; Mention @.>
Subject: Re: [AndrewRathbun/KAPE-EZToolsAncillaryUpdater] Not checking for and downloading/creating directories/files that might be referenced later (Issue #2)
Any error handling you want to add, I ask you simply use the Log function that's already being used script-wide to keep things consistent.
In regards to checking for if the executables are there before we move them, if you wanna do a PR with Test-Path and all that other fun stuff, by all means! When merged, I will resign and republish.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAndrewRathbun%2FKAPE-EZToolsAncillaryUpdater%2Fissues%2F2%23issuecomment-1069562450&data=04|01||9f0ed2ce5c78443b706308da0787e1f8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637830577437292565|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=FavdGZ4FKcZOwLuCeBI5HuYTnvSHlMPhtgcn8jsK2wY%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAF5EQS24F5GN3VZEPWCAEATVAI443ANCNFSM5OT5SWHA&data=04|01||9f0ed2ce5c78443b706308da0787e1f8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637830577437292565|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=wwRkJnd0X8VYNrIHMyFV3U1YqBn1Hx31upHjBNgkSJk%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Add support for cerebro

I'd keep it more or less synced up with https://github.com/pires/kubernetes-elasticsearch-cluster. Can you create similar PR there as well?
@clockworksoul your thoughts?

@andrey-moor Yeah, I guess that makes sense

I agree. I don't think the goal needs to be 1:1 exact, but we should keep in sync where we can.

Cerebro does not follow Elastic release cycles, and is designed as a isolated client, so one cerebro can access every ES cluster.
i suggest to keep cerebro as a separated helm

Sounds reasonable

I agree!


Allow homekit script execution on tv remote keypress

Description:
Add support for script execution for Homekit TVs keypress
https://github.com/home-assistant/home-assistant/pull/22968 recently added a way to expose media_player entities with device_class: tv. Currently only play/pause button is connected to media_player.media_play_pause:
https://github.com/home-assistant/home-assistant/blob/db48d5effdfc40d1ade8e54ea9fc52801e1e300a/homeassistant/components/homekit/type_media_players.py#L70-L84
Since there is no existing service on media_player for generic button navigation/presses (and I think that is a whole can of worms), this PR adds a way for users to provide a map of scripts to run on key presses

Pull request with documentation for home-assistant.io (if applicable): https://github.com/home-assistant/home-assistant.io/pull/10393
Example entry for configuration.yaml (if applicable):
homeassistant:
  customize:
    media_player.my_smart_tv:
      device_class: tv

homekit:
  entity_config:
    media_player.my_smart_tv:
      key_actions:
        arrow_right:
          service: logbook.log
          data:
            name: homekit
            message: 'right key pressed'

The available action keys are defined in the following mapping:
https://github.com/home-assistant/home-assistant/blob/bcd54f7fb771fdd9cda9aeb7ee64750088bbd74d/homeassistant/components/homekit/const.py#L179-L193
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io https://github.com/home-assistant/home-assistant.io/pull/10393

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
~[ ] Untested files have been added to .coveragerc.~

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hey @cdce8p would you be able to take a look at this PR?

@nickw444 I'm no longer responsible for the HomeKit integration. For details https://github.com/home-assistant/home-assistant/pull/25043

Hey @MartinHjelmare I noticed you adjusted the title, so just pinging you. Since @cdce8p is no longer available to review this PR, do you think you would be able to review this code? Cheers

Sorry, I don't know the homekit integration.
Script options for media_player platforms has been suggested before:
https://github.com/home-assistant/home-assistant/pull/24731#issuecomment-505657948
So generally it doesn't seem like a bad idea. But someone with homekit integration knowledge should review this.

So generally it doesn't seem like a bad idea. But someone with homekit integration knowledge should review this.

Looks like there are very few people in this project who have much knowledge around homekit. There are a handful of open PRs with comments from only yourself and @balloob
I'm not all too familiar with how ownership works around here, is this a case where this is escalated to Paulus? As a side note, I think it's important this lack of ownership is resolved in the short term as in my opinion homekit integration is a core feature that many come to Home Assistant for.

You can escalate all you want to me, but I can't do anything about it. That's not volunteer based open source works.

I also think that this should just be events ?

Superseded by https://github.com/home-assistant/home-assistant/pull/29588


Use tokio::io::copy_buf for spawn_copy

Helps to avoid blocking, reduces system calls, can stop more cleanly between read/write cycles, better backpressure.

I don't know if tokio:: io::copy uses zero copy and if copy_buff will pass through the user state buffer once?


Fix the solar system radius calculation

Bug fixes

Fixed returning to solar system radius not being calculated correctly

To explain this. 30000 is the edge, while 24000 is the interloper.
If you did multiplication if your farthest body was at 30000, it would make the edge 24000.
Obviously it should be the opposite and so I changed it to divide.
30000 * (24000 / 30000) => 30000 * 0.8 => 24000
30000 / (24000 / 30000) => 30000 / 0.8 => 37500


5.3-4

Compared to CLRS2e, the procedure additionally accepts the array length n and explicitly creates the array B.


Frame sampling method

Hey, thank you so much for your work! I think it's very good and I seriously enjoyed reading the paper so I want to do some experimentations with it and try to experiment with ways to improve or modify it.
I have read about the sampling method in the paper, that you sample globally (by default 16 frames for training) and I wish to change it such that it only samples f frames from behind the key/target frame with a stride of 2, may I know where the sampling logic/code is?
Is the current sampling method the TrainSampler or VIDBatchSampler in yolox/data/datasets/vid.py?

Yep, please custome this func and pass kwargs to control the sampling method:
https://github.com/YuHengsss/YOLOV/blob/5f069b29e201c4e099c7c3827cc6c63823ce3141/yolox/data/datasets/vid.py#L73

Hey, thanks for your quick reply. I've just looked into it, it seems that function passes out a list of lists that has the same length as training iterations and each list contains randomized global frames and the intervals depend on whether the mode is random or uniform.
My goal is to get the last few frames with n intervals in between with respect to key/target frame. An example of a list inside the list given by photo_to_sequence is ['video1_006306.jpg', 'video1_006315.jpg', 'video1_006324.jpg', 'video1_006333.jpg', 'video1_006342.jpg', 'video1_006351.jpg', 'video1_006360.jpg', 'video1_006369.jpg', 'video1_006378.jpg', 'video1_006387.jpg', 'video1_006396.jpg', 'video1_006405.jpg', 'video1_006414.jpg', 'video1_006423.jpg', 'video1_006432.jpg', 'video1_006441.jpg'] and my question is are these 16 elements all for reference frames or is the last element the key/target frame?
If all elements are used for reference frames, is it possible to get information of the key/target frame in the photo_to_sequence function so that I can get t-16 to t frames where t-th frame is the key/target frame? (basically get past 16 frames from the key/target frame)

Btw, I got these twp visualisation for the attention scores on the cls score and cls score masked, just curious, how might you interpret these? I'm a bit confused,  please advise 😄

Also regarding the MSA_yolov_visual, how do we use it to generate the visuals? I notice I cannot just replace MSA_yolov with it since the inputs for forward method is different

def visual_sim(attn,imgs,simN,predictions,cos_sim):
    sort_res = torch.sort(attn,descending=True)
    #cos_sim_res = torch.sort(cos_sim,descending=True)
    img = imgs[0,:,:,:].permute(1,2,0)
    support_idx = sort_res.indices[:simN, :]
    for bidx in range(int(5)):
        box = predictions[0][bidx,:4]
        visual_pred(img,box,title='key proposal'+str(bidx))
        for sidx in range(6):
            frame_idx = support_idx[bidx,sidx]
            fth = int(frame_idx/simN)
            bth = int(frame_idx%simN)
            simg = imgs[fth,:,:,:].permute(1,2,0)
            box = predictions[fth][bth,:4]
            visual_pred(simg,box,'key-'+str(bidx)+'-support'\
                        +str(sidx)+"-"+str(float(sort_res.values[bidx,sidx]))[:4]\
                        + '-'+str(float(cos_sim[bidx,frame_idx]))[:4])
        if bidx>=5:
            break
     ```
     According to the visual_sim function it shows that the key/target frame is the first element of the images while the other images in the list are all support frames, is this correct?

     And I have one other question, where is the code that you used to produce the reference and keyframe images from the paper? I found the MSA_yolov_visual function and subsequent functions but there are no files that uses it and we cannot simply swap out MSA_yolov for MSA_yolov_visual as well

Hello, the training and inference actually could be implemented in batch, which meaning when you take one of the frame as the key frame, the others will act as the reference frames. In this case, every frame is the key frame while they act as the reference frame for others.

these 16 elements in my case all for reference frames or is the last element the key/target frame?
(I see that during training, the inputs are of (gframe, 3, h, w) so I assumed that the key/target frame should be part of the 16 elements please let me know if I'm right or wrong)

The "l" frame denotes for the local frames, and in current sampling func, they just sampled like uniform with a stride parameter. However, in the VID dataset, the local frame sampling strategy is not work as well as the global one.

Hello, the training and inference actually could be implemented in batch, which meaning when you take one of the frame as the key frame, the others will act as the reference frames. In this case, every frame is the key frame while they act as the reference frame for others.

Oh I see, 请问 code 里面的这个 logic 是在哪里？

The "l" frame denotes for the local frames, and in current sampling func, they just sampled like uniform with a stride parameter. However, in the VID dataset, the local frame sampling strategy is not work as well as the global one.

Thanks for this reply, I understand now

Hello, the training and inference actually could be implemented in batch, which meaning when you take one of the frame as the key frame, the others will act as the reference frames. In this case, every frame is the key frame while they act as the reference frame for others.

Oh I see, 请问 code 里面的这个 logic 是在哪里？

Due to there are not limit (such as mask or only query from key freme while key and value from the references) to the reference proposal in the attention, each proposal could receive support from all other proposals in the same batch.  So, the design is inherently "all key frames".
Btw, thank you very much for replying recent issues in this repo !

Due to there are not limit (such as mask or only query from key freme while key and value from the references) to the reference proposal in the attention, each proposal could receive support from all other proposals in the same batch. So, the design is inherently "all key frames".
Btw, thank you very much for replying recent issues in this repo !

Ohhhh okay I see what you mean. 没问题兄弟，我正在帮 research organization 写一个 paper 刚好看到其他的 issue 还有你看起来也很忙所以就帮帮一下。


find() timed out during startup

I receive the following error during homebridge startup:
StarProjector: An error occured during connecting to Tuya API: find() timed out. Is the device powered on and the ID or IP correct? - Retry in 60s
Credentials from tuya-cli:
[
  {
    name: 'Panamalar-Smart Star Projector',
    id: '***',
    key: '***'
  }
]

I configured the plugin as following:
{
            "name": "StarProjector",
            "projectors": [
                {
                    "name": "StarProjector",
                    "tuyaId": "ID_FROM_ABOVE",
                    "tuyaKey": "KEY_FROM_ABOVE",
                    "color": true,
                    "laser": true,
                    "starRotation": true,
                    "scenes": [
                        {
                            "active": false
                        },
                        {
                            "active": false
                        }
                    ]
                }
            ],
            "platform": "StarProjector"
        }

Is there any way to debug this? Looking at the source code it seems my homebridge instance can't access the tuya server at all..?!

Probably a problem in the upstream library or in my setup. Closing this now.

@tfohlmeister could you solve the issue? having the same problem with two setups and can't find a way to get it working

@ScharV Unfortunately not, I returned the projector by now (also because of this).


Table: Avoid checkbox overflow

Description

Resolves #9839 by using the Dense padding for checkboxes in a table, removing the overflowing hover effect that caused a scrollbar to appear. This includes the MudDataGrid.
See it here: https://try.mudblazor.com/snippet/mOcIvkbybOVthUow
How Has This Been Tested?

visually
Type of Changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Vertical height of both tables are identical and width was manually set to 400px to keep consistency, making the dimensions of both in the screenshots below equal to 400px by 186px.
Number 4 has mouse over in both screenshots
Before

After

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

From my POV the hover/focus styling with the default colour becomes practically invisible, I have to focus on it to be able to see it :(
Had a peek at other libraries:

MUI hides overflow
others don't have a hover effect on the checkbox to begin with
fluent ui has focus styling on the checkbox cell rather than the checkbox

overflow: hidden seems to work and keeps the hover/focus effects readable, what do you think? https://try.mudblazor.com/snippet/GOQSPOYTBVqnDEwI
Could even consider making the checkboxes Size.Small for an even denser look as a bonus

@igotinfected Yeah I'm not super satisfied with it but likely the simplest solution.

Hiding overflow could drastically change user layouts (and should be applied on all in that case, not only ones with checkboxes)
Clipping the content avoids fixing the root issue
Making the checkboxes small would have to be done on the user's end so would break a lot of tables
A small checkbox still has the same negative margin issue

But if MUI clips it then it's worth considering. You can find a similar long-standing issue with dialogs here: https://try.mudblazor.com/snippet/QEmIbkYpLtxRseHO

@danielchalmers could you fix this for the DataGrid too? 🙏

I think this is one of those decisions @Garderoben can help us with.

@danielchalmers could you fix this for the DataGrid too? 🙏

Could you send me a repro link? I'm not super well versed with the DataGrid

@danielchalmers here you go: https://try.mudblazor.com/snippet/mYQybkkAJCedBTIs Thanks for taking a look.

@dennisrahmen It's actually using the same styles so it works out of the box with this PR. Thanks for helping me find this side effect!

Is it possible to solve the scrollbar problem without making the focus circle smaller?

@henon as it is right now the focus circles of each row are overlapping. So you are clicking in row number two but the checkbox in row number three would be selected.
You could keep the circle size for the normal view and only make it smaler when the DataGrid/Table is set to Dense="true"

OK, so I see two possible solutions. Either configure the circle to be so small that it doesn't overflow the row or make it so that the row cuts off the circle to avoid overflow onto other rows.

OK, so I see two possible solutions. Either configure the circle to be so small that it doesn't overflow the row or make it so that the row cuts off the circle to avoid overflow onto other rows.

In general I don't think controls should exceed their space, even just the hover effects. Looks weird

In gmail they made the circle fit the table row exactly

In gmail they made the circle fit the table row exactly

Would work great but the rows are dynamically sized based on the icon button

Can't we assume a reasonable default here?
Like setting padding to 5px instead of 0?
I would say for nearly all DataGrids the smallest it can be is just text in all columns without any Typography applied and the DataGrid/Table in Dense mode.

@dennisrahmen 5px causes an increase in the height of the original example https://try.mudblazor.com/snippet/QaQRuBcfWQyOUycL (unless you're talking about changing the style in another place)

There is already a precedent for this with the Dense property

https://mudblazor.com/components/checkbox#dense

@dennisrahmen @henon See description for new proposal.
Now uses the dense padding. Still uses a negative margin but this should be suitable for almost all cases and a definite improvement over the current behavior.

Awesome. Can we get a final screenshot for the historical record please?

Awesome. Can we get a final screenshot for the historical record please? Light, if you will, because in dark mode the hover circle ist almost invisible.

I think you are looking for the second screenshot in the updated description @henon

Thanks @danielchalmers !


cmd/compile: xdcl_list / testdclstack slow when there are lots of decls

$ go get github.com/davecheney/genpkg
$ genpkg -n 50000 > x.go
$ go tool compile -cpuprofile=cpu.prof -d=ssa/check/off x.go  # slow!
$ go tool pprof -pdf `go tool -n compile` cpu.prof > cpu.pdf

testdclstack shows up as ~20% of the cpu time, with xdcl_list one frame up.
I quick look suggests that there may be an O(n^2) interaction between xdcl_list and testdclstack--it looks like xdcl_list adds decls one at a time and calls testdclstack for the entire stack each time. (Could testdclstack be called after the loop instead?)
I think that @mdempsky or @griesemer may be looking into this (and I am not); just filing an issue so we don't forget about it.
cc @davecheney -- thanks for the test case :)

testdclstack is a consistency check only - fine to disable (and only run in a special debug mode in the compiler)
In the long run we should replace this sequential list and use a proper block structure. I've started working on this yesterday but hit some roadblocks.

I'm curious if the problem is that declare is adding PFUNC declarations (i.e., global function declarations) to dclstack, rather than adding them to externdecl like it already does for PEXTERN declarations (i.e., global variable, const, type declarations).  There shouldn't be any need for us to track package scoped symbols in dclstack.


kie-issues#46: Disable/Enable Constraint modal OK button properly

Previous changes [1] caused the constraint modal ok button was always disabled. The issue the 'disabled' html attribte can not be present at all.
[1] kiegroup/kie-tools#1352

@jomarko Thank you, https://github.com/kiegroup/kie-tools/pull/1352 is included in the upcoming 0.26.0 version, I would check if this can back-ported in 0.26.0 version too @paulovmr

@paulovmr well it is blocking for the DMN DataTypes Constraints feature users.

@jomarko Please cherry-pick it to the 0.26.0-prerelease branch.
@paulovmr Please wait for this to be merged on main and cherry-picked to 0.26.0-prerelease. There's no need to run the sanity checks again after the cherry-pick.

Cherry-pick created https://github.com/kiegroup/kie-tools/pull/1382


Merging of fire detection model to the project

Problem description
Transfering the Human Activity Recognition (HAR) model for facial detection to fire detection within a household.
Possible solution
Upload Python modules under /indoor_surveillance/fire_detection.
Additional context
Hash out passwords and emails before committing to a public repository.

Issues resolved with reference to PR #10


terminal typing cursor may be hidden behind other windows

See
https://youtu.be/ZqbsI4WTxMQ

Environment in the recording
workshop-dashboard
-terminal region
--lower-split-terminal
---tmux
----half right pane.

Does it occur when you aren't using tmux? Need to determine if this isn't an issue related to tmux.

I have seen it happen outside of tmux. Most likely after multiple window resizing.

Have removed border padding within iframes. Seems to improve things a bit. Going to close this until see more evidence of it.


[#52] Remind students to update mainClass after Launcher is made entry point

Fixes #52
Description:
Modified javaFxPart1.md to remind students to change the mainClass attribute in build.gradle after the creation of the Launcher class.

@se-edu/summer-2024-pr-reviewers for your review ...

Would it make more sense to have the reminder after the part that talks about defining the no-argument constructor?
Since currently if that is missing and the reader tries to run the application they would encounter an error anyways which can be confusing.
If the order still seems weird then perhaps the whole part that talks about defining the no-argument constructor can be moved to before the creation of the Launcher class. The new reminder section can be come after that.
So something like:
...

Note how we have created a Label to contain the text that we want to show. We then create the Scene and set its content. Finally, we set the stage and show it.

[MOVED HERE]:

JavaFX creates the Application instance by calling the no-argument constructor. So, you need to define a no-argument constructor in the Duke class (i.e., the Application subclass you use) if you don't have one already e.g.,

...

Next, we create another Java class, Launcher, as an entry point to our application. The Launcher class is reproduced below in its entirety.

...
[NEW REMINDER ADDED]

Now that we have changed the entry point to our application, we need to update the mainClassattribute in build.gradle accordingly to point at the Launcher class.

@mfjkri thanks for the suggestion! Yes, it does make sense to do that, actually. But a concern I had was that I wanted to emphasise that the entry point changed from whatever they named their application previously to Launcher, and I think an appropriate change would be to put how to run the application after the portion talking about the no-argument constructor.


tests: Return errorcode on Windows if layer tests fail

The call to validate docs was stomping on the layer validation test result.

cherry-picked to master


Multi backend merkle

@mambisi are this PR and #388 still relevant? if no, why?

No its not, its already merged with fb_storage_extended_actions_backtracking_support

@mambisi I see, so, how was this merged? directly without going through Github? We should avoid doing that.

Btw please close this and the other PRs that are not longer relevant.


chore: bump the MSRV to 1.58

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

We saw the mistake here: #3189

closing in preference of #3579


save custom ordered list start index in render JSON

Bug/issue #, if applicable: rdar://73847907
Summary
While cmark-gfm saves the starting index for an ordered list, swift-markdown currently discards this information. This PR is part of a set to preserve this starting index when rendering documentation.
Dependencies
https://github.com/apple/swift-markdown/pull/22
Testing
Add the following markdown to Sources/SwiftDocC/SwiftDocC.docc/SwiftDocC.md:
2. this is a test
3. this is a test

Steps:

Build the associated swift-docc-render branch (https://github.com/apple/swift-docc-render/pull/30) and set $DOCC_HTML_DIR to the resulting dist directory.
bin/preview-docs
Ensure that the added list correctly starts with the index 2.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci Please test

Rebased to resolve merge conflict and to bring up to date with the swift-markdown PR.
@swift-ci Please test

@swift-ci Please test


cyclops-monad-api : Investigate creating Comprehenders for Monad Transformers

This would allow them to be used inside for-comprehensions

Completed in cyclops-react


use regular doc comments

So I just learned from the previous PR that you actually want to eventually drop the bitflags dependency, so in light of that I split the previously-single commit into two commits in case you'd like to at least merge one of them. However, feel free to close and disregard this if neither one is desired!
The first switches bitflags documentation over to regular doc comments ///, because the previous #[doc="something"] doc trick for macros is thankfully no longer necessary. However, I wonder if you kept them despite knowing this on purpose, perhaps to support older rustc versions or something. If so, disregard!
The second commit bumps the bitflags requirement to "0.7". Newer bitflags requires an explicit pub specifier to expose the constants, so that work is also done.

Hmmm... I don't think the doc format commit 0855fc9 came out correctly, I'll fix it.

I'm a little wary to upgrade bitflags just yet as it's likely to be a breaking change (as it tweaks the API here and there), but once a new major release of git2-rs is made I can do the bump. Could the docs be fixed up with 0.1 though?

Ah okay no problem!
Yeah I believe it'll work with 0.1, as far as I know it's a rustc change that removed the need for #[doc], so all macros benefit.
However, I'm in the process of fixing up the docs change commit, since I didn't split it up correctly. I'll let you know when it's ready.

Alright the branch (now a misnomer) only includes the change to regular doc comments.
I did generate the docs and made sure that they're working fine. Feel free to close this though or postpone it if you like, it's no problem!

Just now rebased over master, hadn't noticed that there were new commits.

Thanks!