Can't open some Perso-arabic script EPUB

Package ver: cosmos_epub: ^0.0.1+3
Flutter doctor:
[✓] Flutter (Channel stable, 3.13.1, on macOS 14.2.1 23C71 darwin-arm64, locale en-UZ)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.3)
[✓] VS Code (version 1.88.1)

E/flutter (20393): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Invalid argument(s): Illegal percent encoding in URI
E/flutter (20393): #0      _Uri._uriDecode (dart:core/uri.dart:3064:11)
E/flutter (20393): #1      Uri.decodeFull (dart:core/uri.dart:1302:17)
E/flutter (20393): #2      ChapterReader.getChaptersImpl (package:epubx/src/readers/chapter_reader.dart:34:29)
E/flutter (20393): #3      ChapterReader.getChapters (package:epubx/src/readers/chapter_reader.dart:11:12)
E/flutter (20393): #4      EpubBookRef.getChapters (package:epubx/src/ref_entities/epub_book_ref.dart:56:26)
E/flutter (20393): #5      EpubReader.readBook (package:epubx/src/epub_reader.dart:92:41)
E/flutter (20393): <asynchronous suspension>
E/flutter (20393): #6      CosmosEpub.openAssetBook (package:cosmos_epub/cosmos_epub.dart:64:25)
E/flutter (20393): <asynchronous suspension>
E/flutter (20393): #7      _MyHomePageState._openEpubReader (package:cosmos_epub_example/main.dart:51:5)
E/flutter (20393): <asynchronous suspension>

Error is coming from EPUBX package.
The file:
book.epub.zip

I can't open attached book even in Books (Mac native application)
But I can open book_RTL in mac with 5-6 sec delay. It strrugles.
I opened zak_LTR without any problem in both viewers. I guess some books are broken or malformatted.
Maybe for they are formatted RTL (Right to LEFT)
Books:
books-LTR-and-RTL.zip

Hello, the problem was with the original structure of the epub file

Hello, the problem was with the original structure of the epub file

Oh, I see, Anyways I should handle these exceptions.

软件包版本：cosmos_epub：^0.0.1+3
颤振医生：
[✓] Flutter (Channel stable, 3.13.1, on macOS 14.2.1 23C71 darwin-arm64, locale en-UZ)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.3)
[✓] VS Code (version 1.88.1)

E/flutter (20393): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Invalid argument(s): Illegal percent encoding in URI
E/flutter (20393): #0      _Uri._uriDecode (dart:core/uri.dart:3064:11)
E/flutter (20393): #1      Uri.decodeFull (dart:core/uri.dart:1302:17)
E/flutter (20393): #2      ChapterReader.getChaptersImpl (package:epubx/src/readers/chapter_reader.dart:34:29)
E/flutter (20393): #3      ChapterReader.getChapters (package:epubx/src/readers/chapter_reader.dart:11:12)
E/flutter (20393): #4      EpubBookRef.getChapters (package:epubx/src/ref_entities/epub_book_ref.dart:56:26)
E/flutter (20393): #5      EpubReader.readBook (package:epubx/src/epub_reader.dart:92:41)
E/flutter (20393): <asynchronous suspension>
E/flutter (20393): #6      CosmosEpub.openAssetBook (package:cosmos_epub/cosmos_epub.dart:64:25)
E/flutter (20393): <asynchronous suspension>
E/flutter (20393): #7      _MyHomePageState._openEpubReader (package:cosmos_epub_example/main.dart:51:5)
E/flutter (20393): <asynchronous suspension>

错误来自 EPUBX 包。
文件： book.epub.zip

I also encountered the same problem. Have you solved it


[9.x] Add previousPath (standalone) to UrlGenerator

This is the "standalone" version of the idea outlined in https://github.com/laravel/framework/discussions/41612 ("standalone" because previousPath doesn't depend on previousWithoutQuery as in PR https://github.com/laravel/framework/pull/41662)
In short this would enable:
Submit form when URL is e.g. https://website.dev/foo?bar=1 then in the controller:
return redirect(url()->previousPath()); // /foo

This will redirect to URL: https://website.dev/foo (without the query string)

PR for the "dependent" version: https://github.com/laravel/framework/pull/41662

This is a breaking change and it can not be merged into 9.x branch.

@seriquynh Thanks for noticing! I removed the method from the contract. Should be good now.

Could you add some tests for this?

@taylorotwell Test added.


[Annotation] Take the border into account when computing the font size (bug 1794403)

/botio test

The "regressions" in reftests are expected.

/botio makeref


Configure GitHub organization permissions

If possible to add permission for GitHub organization members. I'd like to upload the logo we chose as the organization icon but I am unable to do so due to lack of permissions.
@arcanistzed

This is kind of complicated to configure with the branch protections. I updated the icon.


Add timestamp to VeDelegation

Add timestamp to VeDelegation

veDelegation.timestamp = event.block.timestamp.toI32()


sorting order

what is the default sorting order.  the whole lambda stuff is very unclear.  Given what I've seen, it isn't alphabetical based on the text, so what is it.  You might have a base selection for sort order (description, tags, start, due, scheduled, ).  And give a real example of using the lambda to do a few of these.

The default order is based on the status of the task, which follows the following order:
["overdue", "due", "scheduled", "start", "process", "unplanned", "done", "cancelled"].
In fact I am considering canceling this option cause:

It is dangerous to evaluate a user provided lambda.
Ordering by status or date time is already reasonable.

Allow sorting by text/description.  I start all my items with:

[ ] HH:MM-HH:MM rest of text

and this lets me have a time based ordering with unplanned at the end.  A second option to sort by dataview formatted start time would be good for many.  So, having a drop down that sorts by:

text
start time (dataview format, ascending)
tags

and then a second sort (if things equal) by

text
start time (dataview format, ascending)
end time (data view format, ascending)
tags

Thanks for the idea, mark as a new feature to be implemented.

Hi, with the latest release the sorting options are now provided by dropdown selection, includes status, text, start time, due time and tags. New suggestions are still welcomed.

than ks.  saw it and great and simpler.


0.7.9 BOOTx64.efi

Hi Yao,
Firstly do appreciate such excellent work from you!
I tried to use the BOOTx64 file either from your prior update (0.6.8) or from other system (Dell Optiplex w/ OC 0.7.9), the result turns the same: After I select "Install Monterey" The computer froze at the Apple Logo for hours.
Any suggestions whether this problem is caused by the version conflict in BOOTx64.efi? May I ask for an applicable BOOTx64.efi file for OC 0.7.9 Monterey?
BTW my mirror is Monterey 12.0.1, which is lower than yours

Hi Yao, Firstly do appreciate such excellent work from you! I tried to use the BOOTx64 file either from your prior update (0.6.8) or from other system (Dell Optiplex w/ OC 0.7.9), the result turns the same: After I select "Install Monterey" The computer froze at the Apple Logo for hours. Any suggestions whether this problem is caused by the version conflict in BOOTx64.efi? May I ask for an applicable BOOTx64.efi file for OC 0.7.9 Monterey? BTW my mirror is Monterey 12.0.1, which is lower than yours

please check your PCI LIST whether your hardware is same as mine, otherwise, you have to fix some driver issue.

can you please provide the BOOTx64 i have same configuration that yours

also OC


Metrics gathering blocks if connection is timing out

Hi,
I have multiple ESXi Hosts im am scraping. Some of them are switched off most of the time.
It seems that if the vmware_exporter is in the process trying to connect to a currently switched off system it cannot process a second connection.
This can easily be reproduced using a web browser by connecting to a currenlty "off" target and opening a second tab trying to connect to an existent machine. Both fail.
Can the exporter process a single request only or is the problem mybe in the python vware library?
Andreas

Hi,
Yes for know it handles only 1 request at a time.
I do not have availability yet to make it handle many connections, help is welcome ;)

Hi @ipsecguy,
coud you try the threads branch to check concurrent connections?
Thanks

Hi, just did. Seems to work in general (I am able to get metrics from the running ESXi machines while the exporter is trying to connect for one of the switched off machines).
What I have observed (Might or might not be related to the change) if I do a scrape on a running ESXI (in a browser) and at almost the same time to another (also running) I get most of the time a websever such as (not always the same object = '' part):
pyVmomi.VmomiSupport.vim.fault.NotAuthenticated: (vim.fault.NotAuthenticated) { dynamicType = , dynamicProperty = (vmodl.DynamicProperty) [], msg = '', faultCause = , faultMessage = (vmodl.LocalizableMessage) [], object = 'vim.VirtualMachine:16', privilegeId = 'System.View' }
Retrying the scrape then works. Might or might not be a race condition.
Cool stuff with the threading, I was looking a little but I soon realized my python/twisted skills are too poor.
Andreas

Hi,
thanks for your feedback. I have the same issue with the NotAuthenticated error, I guess it's a thread issue, but I'm not confortable enough - yet - with twisted and threads. I'll keep you in touch.

I ran into the same issue with the lack of concurrent requests;  are you committed to Twisted?
Moving it over to BaseHTTPServer was straight-forward and seems to work well:
https://github.com/britcey/vmware_exporter/blob/basic_webserver/vmware_exporter/vmware_exporter.py
I'll open a PR if you're willing to make this change.

@britcey great if BaseHTTPServer does the correct job.
I'll take a look when I'll have some free time!

This issue was moved to pryorda/vmware_exporter#10


feat(tmp): Add openshift-redhat-marketplace in nss CR.

We can remove it after we can input the namespace from OperandRegistry into the nss CR.

thanks. do we want also to add kube-system same way?

@pgodowski
We can add kube-system in 1Q and remove them after we can get namespaces from the OperandRegistry in the fixpack.


Can't run chronicles

Steps to reproduce:

Run pnpm install - Ok
Run pnpm build:all - Ok
Run pnpm run:chronicles - Fails, 21 errors
When trying to run through Xcode, the errors from above show up. Apparently from the native code in React-native-skia

Xcode version: 14.3.1
Node version: 16.20.0
Npm version: 8.19.4
Pnpm version: 8.6.1

This issue has been bothering us (and the pipeline) since the latest xcode version and may be fixed by one or many of the following:

Bump the Expo recommended version of react native skia to ..185
Downgrade pnpm to the latest major 7 release. We have not tested 8, and have faced some issues with it. Should probably make an issue to upgrade.

Related:
https://github.com/Shopify/react-native-skia/issues/1374

Tried with xcode 14.2, node 16.14.2, and pnpm 7.28.0, and it looks like it works. So I guess we have to somehow make this repo compatible with the newest versions, or specify in the readme which versions you should use.

The former would be appreciated 👍

@tormodAase I think this was resolved by #131. Should the issue be closed?

duplicate of #130


Add perl5 and .cpanm rule

There is an open issue for .cpan andk/cpanpm#149. This only applies to perl5 and .cpanm.

Unfortunately appending a dir to PATH might no work on all shells. Can you remove it and we will need to think of a way to support this if need arises

I don't really understand the issue. The substitution syntax should work on any shell (except maybe fish but I've never used it so I don't know). These variables are shown during setup to append to your .bashrc or .zshrc. But I'll remove it anyway if it does cause some other issues I'm unaware of. It works fine for me on zsh.


Docker V2 needs docker registry connection(Security issue)

Hi Team,
There is a design on docker v2 task to push images to ACR you ll need to have the Docker Registry Service Connection but this Service Connection is not using an existing app registration rather it needs an access to create a new app registration which needs a global admin access. Can we please use the existing one because giving rights to the service principal to create an app registration on fly is not secure.

Any update about this ?

@Jyoti492 Wouldn't this mitigate your problem right now?
https://github.com/microsoft/azure-pipelines-tasks/issues/11084#issuecomment-553265363

I am able to connect the ACR via Service Connection only on hosted agents and on our private agents it fails with the error as below although the ACR is open for all networks.
The push refers to repository [mcpsharedreg.azurecr.io/coesd/aab-base-deploy-helm]
Get ***v2/: read tcp <IP_ADDRESS>:43262-><IP_ADDRESS>:443: read: connection reset by peer
##[debug]Exit code 1 received from tool '/bin/docker'
##[debug]STDIO streams have closed for tool '/bin/docker'
##[error]Get ***v2/: read tcp <IP_ADDRESS>:43262-><IP_ADDRESS>:443: read: connection reset by peer

@Jyoti492 where are you seeing this error?

@Jyoti492 , closing the issue. please reopen issue if you are still facing the issue.


"bad rdlength" error in dynamic update request

I'm porting some dynamic DNS code from python to this dns library and I'm getting an error. I have a small server that listens for DNS updates. The code works fine when requesting with nsupdate, but errors when I try with a FortiGate firewall.
The requests are not identical between the two clients but I am not sure why all other systems I've tested against seem to work fine with the FortiGate. The only difference that I can tell is that the FortiGate sends two updates (summary from wireshark):
testhost.dns.com: type A, class ANY
testhost.dns.com: type A, class IN, addr <IP_ADDRESS>

and nsupdate just sends one:
testhost.dns.com: type A, class IN, addr <IP_ADDRESS>

The part of the library that throws the error is in the UnpackRR function in msg.go where it throws "bad rdlength". The first update from the FortiGate has data length of 0 but I've tried deletes from nsupdate as well that have length 0 with no problem.
I dug into the RFC but didn't find anything that would make me think what the FortiGate is trying to do is illegal.

I've tried to provoke this behaviour again and been unable to—call it fixed?


Google failures

Context
Silent failures
Changes proposed in this pull request
Added new exception type to be thrown when there google does not returns OK
Handle thrown exception and log it via telemetry client, this will now show up in Azure.
Guidance to review
null out the api key to see exceptions in action

 Log Google Places API failures


added some comments to the README

added some comments to the CSS so people can see the context of the posts if they cared to look for it

Superb work, thanks 🎊


PROOF OF CONCEPT: onboarding operators to FBC via a script and use of composite catalog template

To reduce complex processing flows in the pipeline for operator authors wishing to onboard to FBC, we prefer advertised interfaces and valid FBC instead of relying on specific tooling and knowledge of FBC "precursors".
This high-level catalog publishing flow illustration breaks the FBC-to-catalog into three logical phases:

build the update graph FBC (possibly as a result of a new bundle version published)
validate and aggregate (non-FBC & FBC) contributions
publish the aggregate in an accessible registry

Before those steps can execute, operator authors need to be able to onboard to FBC.
This POC demonstrates the use of a script to fulfill an FBC onboarding process for one/more authors with clear messaging and interfaces, and provide capabilities to fulfill (1) above, external to any catalog construction pipeline. It will:

cache the (controlled by script vars) appropriate version of the catalog as FBC
extract the operator's catalog contribution from the catalog and convert to a basic catalog template
scaffold a composite template to generate FBC for all supported OCP versions
template a Makefile in the operator's subdirectory which will be configured by default to use the composite template to generate contributions via catalog make target.
mark the operator's ci.yaml file as FBC-onboarded by setting updateGraph property to "FBC".

Conventions:

uses yq/yaml
requires being executed from local repo root (and enforces it)
presumes the directory structure

operators/$operator_name
├── catalog-data
│   └── basic-catalog-template.yaml
└── ci.yaml

Moving proof of concept PR from k8s-operatorhub/community-operators since that may be deprecated.

@grokspawn Based on your PoC we created an official version that will be available to both community and isv users.
https://github.com/redhat-openshift-ecosystem/operator-pipelines/pull/653
I'll close this PR but if you think it is still relevant please let me know.


[ME-828] Always override client with new config

https://fieldwire.atlassian.net/browse/ME-828
Previously, we only initialized the Client object (used to upload files for Filestack) when it's null and reuse the same object if it's not (even though there might have been a different config param passed in, i.e. config with a different region).
Additionally, Filestack is currently being set on a different process as compared to our application (separate android:process is set in the AndroidManifest, this was to avoid crashes from Filestack to impact our application since it's not being maintained anymore), thus even though the Filestack Activity might be dismissed, it still retains all the references to the previous usage.
TLDR; Filestack reuses the same storage config once it sets even though we create a new FilestackPicker with new config every time, so removing the client == null logic lets the initializeClient() call to set the client every time so we don't reuse the old config.

@davidnguyen2302 to master?

@davidnguyen2302 to master?

@mbalajee I'm assuming that's the main branch for this repo, once it's merged I can get the new commit hash code to update our imported version in our FW app, should I be committing to a different branch?


How do i prevent the bot from stopping from running when it hasn't detected any new messages?

Can you clarify what you mean here?
I've been using this for ~6 months, but what you've written doesn't mean anything to me.
What is it doing now? Are you using the latest version? Is your settings.json set up correctly? Does "stopping" mean it closes, or it just stops?
Some more details might help.

The bot doesn't stop running. It will stay alive until you interrupt it or it crashes. I'll need more info to be able to help resolve.


RunDemo() better error handling

Running demo I have "FireFox Software updater" PopUp message.
This violates the script's runtime.
I was hoping to catch it somehow and close all sessions.
So I provide this changes to wd_demo.au3 which partially solved the problem, but not completely, FireFox still remains running.
> _WD_Startup
_WD_IsLatestRelease: True
_WD_IsLatestRelease ==> Success
_WD_Startup: OS:	WIN_10 WIN32_NT 22000
_WD_Startup: AutoIt:	<IP_ADDRESS>
_WD_Startup: WD.au3:	<IP_ADDRESS> (Up to date)
_WD_Startup: WinHTTP:	<IP_ADDRESS>
_WD_Startup: Driver:	geckodriver.exe
_WD_Startup: Params:	--log trace
_WD_Startup: Port:	4444
> _WD_CreateSession
__WD_Post: URL=HTTP://<IP_ADDRESS>:4444/session; $sData={
    "capabilities":{
        "alwaysMatch":{
            "browserName":"firefox",
            "acceptInsecureCerts":true,
            "moz:firefoxOptions":{
                "prefs":{
                    "app.update.auto":false,
                    "app.update.enabled":false
                }
            }
        }
    }
}
__WD_Post: StatusCode=500; ResponseText={"value":{"error":"unknown error","message":"Process unexpectedly closed with status 0","stacktrace"...
__WD_Post ==> Webdriver Exception: {"value":{"error":"unknown error","message":"Process unexpectedly closed with status 0","stacktrace":""}}
_WD_CreateSession: {"value":{"error":"unknown error","message":"Process unexpectedly closed with status 0","stacktrace":""}}
_WD_CreateSession ==> Webdriver Exception: HTTP status = 500
__WD_Delete: URL=HTTP://<IP_ADDRESS>:4444/session/
__WD_Delete: StatusCode=405; ResponseText=HTTP method not allowed...
__WD_Delete ==> Webdriver Exception: HTTP method not allowed
_WD_DeleteSession: HTTP method not allowed
_WD_DeleteSession ==> Webdriver Exception: HTTP status = 405
! $iError = 10
! $_WD_HTTPRESULT = 405
! $_WD_SESSION_DETAILS =

As far as I understand/remember..... UDF is designed to have single return point, not multiple and this is intentional, design choice of @Danp2
I know there are exceptions.
So there is no need to use _dispose($sSession) as this will be used once.

@mlipok I know this is out of scope regarding your changes, but you start to extract code into separate functions (I ♥ it, 😁), so maybe it's a good time to do the same thing for the region code #Region - Web**e**Driver update and #Region - Headless (in this RunDemo function)?

and:

@mlipok I still recommend to get rid of unnecessary nesting by ContinueLoop 😉 .

@Danp2  do you agree for such kind of changes ?
If so I create new ISSUE and in next step new PR, as this is out of scope for this PR RunDemo() better error handling

@Danp2 do you agree for such kind of changes ?

If not, I am totally fine with your and @mlipok decisions, because we don't have an coding guideline or other commited coding style agreements or documents (as far as I know). I am just totally aware of such things, because of my professional living regarding bigger teams of developers.
These are all just recommendations and suggestions, not more 😀 .

ContinueLoop

I'm fine with this if one of you wants to implement it.

I hope this is finally all in this PR

RE: c77d3e3
I originally wrote the code the way you are suggesting, but moved it inside the If statement due to excessive updates that made the button "blink" when you run the mouse over the buttons without clicking.

I understand.
I think this $idButton_Run, $GUI_DISABLE, $GUI_ENABLE will be the best solution.

I think this $idButton_Run, $GUI_DISABLE, $GUI_ENABLE will be the best solution.

Ok. However, I still believe this code would be better placed inside here --
				For $i = 0 To $iCount - 1
					If $aCheckboxes[$i] = $nMsg Then
						$aDemoSuite[$i][1] = Not $aDemoSuite[$i][1]

						; Move your code here
					EndIf
				Next

With the current placement, your code is constantly being run for all events that hit the Case Else section. Alternatively, we could change the code so that it doesn't rely on Case Else.

I just merged to resolve confilcts.
Now I think is ready to merge, please do the last review (I hope it will be last time in this PR).

RE: c77d3e3
I originally wrote the code the way you are suggesting, but moved it inside the If statement due to excessive updates that made the button "blink" when you run the mouse over the buttons without clicking.

This one is already reverted.


API 8: Tweaks to permissions API

SpongeAPI | SpongeCommon
These are some QOL tweaks, plus a few methods to make the API more defined.
I have a few more things I'd like to add -- the idea of a fallback permission for subjects, and a context refactor based on work I've been testing out in PEX.
The changes so far fall into a few categories:
Permissions descriptions / role template changes
Permissions descriptions are currently quite limited in what they can do. These changes make permissions descriptions more powerful by more thoroughly linking some relatively spread out portions of the API. New features include:

Permissions descriptions can supply a default value
Role templates are now explicitly namespaced by plugin -- so a staff role for Nucleus would be stored internally as nucleus:staff. This allows more fine-grained control over templates, or, for example, only copying the permissions provided by a role from one plugin.
There are methods on PermissionService that allow querying role template information without having to know the details of how the implementation stores that data.

Bulk data editing methods
Having these operations allows plugins to more efficiently work with permissions subjects. Having copy/move operations explicitly as part of the API will hopefully encourage permissions plugins to provide that functionality.

Allow setting all permissions, options, or parents on a SubjectData
Allow copying/moving all data from another subject data entry
Added a new TransferMethod to customize the behaviour when overwriting existing subject data -- either replacing everything or merging with existing values

QOL tweaks

Make most generic parameters wildcards to allow implementations to interoperate more easily with native objects

@lucko @zml2008 Is this PR going to go anywhere or set this aside? Need something done on this soon else it won't be in API 8.0.0

I'd almost forgotten I had this -- i'll take some time this week to get everything I can buttoned down. The improvements already in are still useful -- and i have a couple more design changes I'd like to explore.

Alright, I've pared this down a bit and taken out the context changes -- everything that's left should be good to go. Impl PR coming soon.

for example, only copying the permissions provided by a role from one plugin.
Can it not only be copied, but applied transparently? So when plugin u[date introduces new permission, it would be applied to user / group / etc automatically.

Can it not only be copied, but applied transparently? So when plugin u[date introduces new permission, it would be applied to user / group / etc automatically.

Yes, it would be possible to add the role template subject as a parent of the user.

Heya, SubjectData::getPermissions gives me all directly assigned permissions of a player. Is there a way to get all inherited permissions as well?

From #dev on 2020-08-03 00:06
It would be nice to have some way of getting all permissions for a given Subject, regardless of the source. Right now, the only way to do it would be to check through each parent individually

Why do you want to get all permissions?


Create a nightly release workflow

So that we have a fresh build all the time to test, let's have a nightly build by publishing a new build to npm with tag: 0.0.0-nightly-${SHORT_SHA}-${DATE}. ${DATE} should be $(date '+%Y%m%d') every day at 12AM PST.
For reference if needed, check out Remix's nightly workflow.

Reopen because the workflow failed


[WIP] SPATEM/MAPEM Support

Current WIP State

coding packages successfully generated by asn1c
download of external asn1 definitions still needs to be automated (currently executed manually)
generation of msgs and conversion packages still failing

Hey @v0-e,
would you mind taking a look at this branch? I can successfully generate the C-structs for SPATEM/MAPEM, but msgs/conversion generation fails with an error like the following.
thread 'main' panicked at src/msgs/bin.rs:23:35:
called `Result::unwrap()` on an `Err` value: Error { kind: InvalidData, message: "stream did not contain valid UTF-8" }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

In order to test this yourself, make sure to initialize the submodules recursively and to execute ./asn1/external/download.sh for downloading external ASN.1 dependencies for the is_ts103301 submodule.
Also tagging @jpbusch, FYI.

@lreiher I have taken a look at your code and I can help to make it work a little bit more.
First thing is to delete the offending characters in the source .asn file. This will show the lines:
grep -axv '.*' asn1/raw/is_ts103301/build/asn1/ISO14816_AVIAEINumberingAndDataStructures.asn

Then you can run utils/codegen/codegen-rust/asn1ToRosMsg.py ....
This will throw an error because the Rust version of the tool doesn't recognise BITSTRING types.
The Python version of the script seems to work: utils/codegen/codegen-py/asn1ToRosMsg.py ....
HTH - this isn't a full fix, but might help get this PR further. This is a really great project and we'd love to use it too!

@james-ward Thank you very much for the hint! I have pushed an automated patch of the offending characters.
I have then tried to implement handling of bit string in the rust codegen and successfully generated messages and conversion headers. However, we will have to look into the bit string handling again to verify. I'm a little confused right now why this problem hasn't yet surfaced with the other message types.

Also mention limitations in README?


[ENH] - Improve package version selector

Feature description
Proposal: allow to add package name and version without scrolling.
Example: I want to add "python". The expectation is that I'll scroll and find it, but there're too many matches. The only way to add it is to go to raw YAML, but it'll erase all your changes if those are not saved. Not practical.
Ideal workflow: I type "python", it shows the selector, but I can still add it (by keeping it as is and moving focus to the version field) and specify its version. When I switch to the version field, it should pull actual "python" versions, what I specified in the first field. The same for version: in case there are too many versions, I should be able to just type it in the new UI, but also be able to select from the list.
Also: when I mistype something, it should be visually clear. For instance, text highlighted in red immediately. Not sure if red is the best color for this. Colors need to be chosen with a11y in mind.
Value and/or benefit
Improved UX.
Anything else?
No response

This is hugely overlapped with #126 and already in line with other workflow and UI related tasks in flight. Closing as duplicate but will move some of the details to the other workflow issues.


Strip Excludes should be fuzzy like Excludes

Fixes #254 --strip-exclude Doesn't work for relative paths

@garlicnation PTAL

LGTM


Breaking changes in intel_tex_2 0.3.1

This will probably require releasing a patch release that fixes the version of the dependency if the issue isn't resolved on intel_tex_2 itself.

Resolved. The problematic version has been yanked.


[#2062][FOLLOWUP]Improvement[dashboard] Display detailed info for application in dashboard ui

What changes were proposed in this pull request?

Display detailed info for application in dashboard ui
Persist the application info in coordinator.
Display application history information in dashboard ui.

Why are the changes needed?
(Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.)

Fix: https://github.com/apache/incubator-uniffle/issues/2062
Does this PR introduce any user-facing change?
(Please list the user-facing changes introduced by your change, including

Change in user-facing APIs.
Addition or removal of property keys.)

Yes.
How was this patch tested?
(Please test your changes, and provide instructions on how to test it:

If you add a feature or fix a bug, add a test to cover your changes.
If you fix a flaky test, repeat it for many times to prove it works.)

This looks a huge change, could you help check this? @maobaolong If this has been applied in your internal cluster, please let me know, this will strength my approval confidence.

@zuston yeah, this has been  applied to our internal codebase and deployed in our RSS cluster already.

Could we use rocksDB to store the history data? If we load the history data, maybe we can't start coordinator quickly.

Could we use rocksDB to store the history data? If we load the history data, maybe we can't start coordinator quickly.

I am not good at rocksDB. Is it possible to open a new PR to optimize this using rocksDB?

Could we use rocksDB to store the history data? If we load the history data, maybe we can't start coordinator quickly.

I am not good at rocksDB. Is it possible to open a new PR to optimize this using rocksDB?

You should have a common abstraction both for file and Rocksdb first. What's more, you should reply my second question, f we load the history data, maybe we can't start coordinator quickly. How to handle this case.

Could we use rocksDB to store the history data? If we load the history data, maybe we can't start coordinator quickly.

I am not good at rocksDB. Is it possible to open a new PR to optimize this using rocksDB?

You should have a common abstraction both for file and Rocksdb first. What's more, you should reply my second question, f we load the history data, maybe we can't start coordinator quickly. How to handle this case.

Next, I will split this PR into more independent PRs and deal with the abstraction suggestion.
About you second question, coordinator only loads a portion of the latest history data, not the whole history data.

Could we use rocksDB to store the history data? If we load the history data, maybe we can't start coordinator quickly.

I am not good at rocksDB. Is it possible to open a new PR to optimize this using rocksDB?

You should have a common abstraction both for file and Rocksdb first. What's more, you should reply my second question, f we load the history data, maybe we can't start coordinator quickly. How to handle this case.

Next, I will split this PR into more independent PRs and deal with the abstraction suggestion. About you second question, coordinator only loads a portion of the latest history data, not the whole history data.

I can't see the logic to read part of history data. And although we only read part of data, the start time will still be long.


[Task]: Track performance of each individual response in consolidated API flow

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
https://theappsmith.slack.com/archives/C024GUDM0LT/p1702625681545779

Hi, I would like to work on this issue, can you please assign it to me?


Create Ebook kotlin

This Kotlin roadmap offers a structured journey from basic syntax and fundamentals to advanced features like coroutines, Kotlin Multiplatform, and Android development. It highlights crucial concepts such as null safety, scoping functions, higher-order functions, and Domain-Specific Language (DSL) design. Mastering these steps equips developers to build efficient, modern applications across platforms.
What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

This Kotlin roadmap guides developers through essential concepts, building foundational to advanced skills for creating versatile, high-quality applications.


Use an event handle to check for pending SimConnect messages

It just seems cleaner this way to me.
Could probably do away with the thread altogether, but would need to refactor the startup routine a bit since start() needs to return something.
My local git had a minor conflict in the .h when merging this over top of the first PR (and vice versa), but GitHub is sometimes smarter it seems, so maybe it'll be OK.
Thanks,
-Max

Ok, thanks Max. Will take a look and let you know.
John

Rebased to fix conflict and force pushed.


Customizing parsing options for the test case generator

For a new test case, currently the baseline (.tree.json) is always created with the following parsing options: {loc: true, range: true}. This is not ideal because we want to generate the test case for e.g. tolerant parsing.

One proposal than @jasonLaster and I were discussing is an out-of-band information for the parsing options. Thus, each test case is arranged as in this example:

foo.js is the source to be parsed
foo.options.json is the parsing options used to parse it
foo.tree.json is the expected syntax tree (baseline)

If the corresponding options file does not exist, it is assumed to be the above default.

Side note: this is probably necessary so that we can complete #1270.

https://gist.github.com/jasonLaster/10e46802c08de1d99e9d

Looks like we are not pursuing this at all. Will reopen if the idea is revived.


Documentation for the behaviour of composite nodes is misleading

The current documentation for the behaviour of composite nodes contains conflicting information and inaccuracies.
Firstly, both Sequence and Selector documentation pages state the following:

Every tick, the Selector/Sequence node processes all its children, even if one of them is currently RUNNING.

This is wrong, since only the Reactive variants process all child nodes, regardless of the return status.
Secondly, the Types of Sequence/Selector Nodes tables in the Sequence and Selector documentation pages correctly illustrate the tick behaviour of each composite node sub-type, depending on the child node's return status. However, it uses the term restart to mean that the composite starts ticking from the first child, as opposed to tick again which means that it ticks from where it left off the last time. This is confusing, because the term 'restart' is explained in Composites page as follows:

Restarting a composite
If a parent node restarts a child node, it means that the parent node will start the child node from scratch the next time it is ticked. This means that any progress made by the child node will be reset, and it will start its execution from the beginning.

This suggests it's the child node that gets restarted - i.e. it receives the before_run callback - and not the execution of the sequencer node itself gets restarted. This is contrary to how the term is used in the tables on Sequence and Selector pages. It is also contrary to the heading, i.e. Restarting a composite* suggests it's the composite itself that gets restarted, not the child node.
Relevant documentation pages:
https://bitbra.in/beehave/#/manual/composites
https://bitbra.in/beehave/#/manual/selector
https://bitbra.in/beehave/#/manual/sequence

Thank you for the detailed report.

This is wrong, since only the Reactive variants process all child nodes, regardless of the return status.

This is untrue: as you can see in this example RUNNING children will always be executed. The difference to a reactive composite is that a reactive composite will always restart from the beginning, while the normal version re-ticks a running node.

[...] This suggests it's the child node that gets restarted

I agree on this, the documentation can be improved here.

@rxlecky please feel free to give me your feedback on https://github.com/bitbrain/beehave/pull/249 which hopefully addresses this issue.

Oh, that was quick! Thanks for taking a look, I believe improving the docs will help the new users.

This is wrong, since only the Reactive variants process all child nodes, regardless of the return status.

This is untrue: as you can see in this example RUNNING children will always be executed. The difference to a reactive composite is that a reactive composite will always restart from the beginning, while the normal version re-ticks a running node.

Oh, I'm sorry, that was poor wording on my end. I meant exactly what you said, you just worded it better. 🙂
Anyway, in my opinion, my point is still valid. The documentation says that every tick the composite processes all its children. Maybe I'm misunderstanding what the term processing the children is used for, but in my opinion, this makes it sound like the composite is restarted and all child nodes are ticked on every tick - which is the behaviour of only the reactive composites and that's what I meant with my original comment. Well, at least that's how I understood this section of the documentation the first time I read it and what I found confusing. And even now, I'm not sure what its actual intended meaning is, to be honest.

Tick Again = only the latest running node will be ticked again
Restart = even with a currently running action, all nodes until that running action will be reticked.

Sorry if I'm not making myself clear. I fully understand the terms Tick Again and Restart in the context of the composite nodes and I understand how the different types of composite nodes behave, depending on the return status of their children.
All I'm saying is that this section of the documentation:

Every tick, the Selector/Sequence node processes all its children, even if one of them is currently RUNNING.

to me sounds like selector and sequence nodes are restarted every tick, because it says that all child nodes are processed, i.e. ticked, on every frame.

My suggestion would be to reword this paragraph to something like this:

Every tick, the Selector/Sequence node either restarts and starts ticking its children from the beginning, or it continues where it left off last frame and starts ticking from the child that was ticked last on the last frame. The exact behaviour depends on the type of Selector/Sequence node and on the return status of the child node that was ticked last on the last frame.

And then this is followed by the section which has a neat table, exactly explaining the behaviours depending on the type.

Restart means that during a single frame (the moment the next tick happens), all children will be ticked until the running child is reached (within a single frame). So the sequence node loops through the children and ticks them within a single tick.

Hahah, this seems like it's either me misunderstanding what you're saying or the other way around 😀
So, again. The documentation says that on every frame, the sequence node ticks ALL of its children. However, this is not quite true for the normal sequence node. On first frame, the sequence node starts ticking all its children in order, starting from the first one, until either all of them return SUCCESS or one of them returns RUNNING. In the latter case, on the next frame, it will only tick the running child node. Not all of them, just the running one - and possibly the child nodes following it, if it has completed and returned `SUCCESS

`


Support parallel running

In the early stage, we actually ran multiple workers at the same time to achieve "parallel running". But it will hit on some confusing errors that the browser might not support running too many web workers at the same time. So it was removed.
We should bring this back with better control over the amount of workers spawned. This should be applied to both the web playground and the CLI.
Concern: the concurrent memory usage will increase and might cause OOM, maybe a CLI option can be a good idea?

Is it possible to properly collect exception error messages and create issues in a semi-automated way?

@StringKe Sure, can you help me create an issue for this?

@StringKe Sure, can you help me create an issue for this?

create by #72


Add Cashew

Cashew - https://github.com/joakimskoog/Cashew
A .NET library for caching responses easily with an HttpClient through an API that is simple and elegant yet powerful.

Although it's small and quite new, it fills the caching gap for HttpClient quite nicely, the quality of documentation and code is good so I'm going to include it


Bug: API query fails when the supportPhoneNumber attribute is not returned

When querying an EV charger, your code fails because supportPhoneNumber is a mandatory attribute for the Location class.
IMO it should not be mandatory, and I have created a PR to fix that.

@bushvin should be fixed now in latest version

Issue closed, stale.


ByronBlock / MultiEraBlock explicit era tag

In Byron the era tag wrapping struct is explicitly a part of the CDDL:
byron_block = [
     ; @name ebb_wrapper
     tag: 0, block: byron_eb_block //
     ; @name main_block_wrapper
     tag: 1, block: byron_main_block
]

For all the other eras there is no mention at all of this outer wrapping struct e.g. Conway (all other eras are similar):
block =
  [ header
  , transaction_bodies         : [* transaction_body]
  , transaction_witness_sets   : [* transaction_witness_set]
  , auxiliary_data_set         : {* transaction_index => auxiliary_data }
  , invalid_transactions       : [* transaction_index ]
  ]

When we retrieve blocks from oura/cardano-net-fetcher we get even these blocks with explicit era tags as if it were:
conway_block = [7, block]

One approach in order to respect the CDDL it could make sense to keep the tag info in the byron blocks but not touch them in the other eras, and then in MultiEraBlock do the check as if it were defined like:
multi_era_block =
    byron_block            ; @name byron
  / [2, shelley_block]     ; @name Shelley
  / [3, allegra_block]     ; @name Allegra
  / [4, mary_block]        ; @name Mary
  / [5, alonzo_block]      ; @name Alonzo
  / [6, babbage_block]     ; @name Babbage
  / [7, block]             ; @name Conway

or similar.
We currently have MultiEraBlock::from_explicit_network_cbor_bytes() to parse using this era tag struct.

When we retrieve blocks from oura/cardano-net-fetcher we get even these blocks with explicit era tags as if it were:

As a reminder, the Cardano network specification has a separate CDDL definition from the ledger specification and CML currently doesn't contain any codegen from the networking spec. I'm not sure the networking cddl is a particularly good state (I've never heard good things about it)


ExtProc/ExtAuthz: support header mutation after data processing for streaming requests

Description:
By https://github.com/envoyproxy/envoy/pull/15935, header mutation after data processing is supported for unary requests and we need the support on the streaming case. In our case, we want to inject some header from the first gRPC streaming message.
For ExtProc filter
Currently, you can configure BUFFERED/BUFFERED_PARTIAL mode for streaming requests, just treating it like unary requests by buffering all the messages until buffer_limit(256M by default). Apparently it copies too much data,  introducing 1) CPU overhead 2) memory overhead 3) latency by waiting for a data of buffer_limit size 4) maybe not work if the first message is larger than the buffer_limit.
Ideally for our case in streaming, the extProc filter can

stopIteration on decodeHeaders after sending HeaderProcessingRequest
stopIterationAndBuffer on decodeData until seeing the first gRPC  delimiter and only send the buffered data before that to the extProc server(I am afraid that logic isn't generic enough to directly add into extProc filter)
receive the DataProcessingResponse, mutate headers and continueProcessing

For ExtAuthz filter
The ExtProc filter almost support the same functionality of forwarding the request body(but with static size) and injecting request headers

@gbrail @ggreenway

cc @snowp

Hmmm -- this is interesting but I'm not sure how to do this generically. BUFFERED_PARTIAL today kind of does what you want -- it waits until it has enough data to fill the buffer, and holds up header processing until that first chunk is processed -- but it won't work with a gRPC stream because it'll wait until it has "enough" data.
I could see changing ext_proc to hold the headers until the first chunk of the body has been processed by the processor. The problem is that I don't know how to generically know whether the first chunk contains a complete gRPC message. Certainly in the current interface Envoy just works on a stream of bytes and those chunks could be grown or shrunk by a previous filter or something on the network.
I think you're asking for the chunking implementation in Envoy or ext_proc to take into consideration the HTTP/2 message framing. I only know enough about HTTP/2 to be dangerous, but is that something that someone could imagine doing?

One idea might be for someone to write a "grpcFraming" filter that would make sure that the (en|de)codeData callbacks would only be called with a single, complete gRPC message. Then ext_proc could hold the hedesr after the first chunk.
Frameworks in other languages use this technique to build up filter chains.

Thanks Greg!
Yes, the problem is how the filter intellectually know the gRPC message framing?Could you please clarify the codepath of this gRPCFraming filter and the extProc filter? How gRPCFraming filter notify the extProc filter when the first gRPC message is there?

Hi @gbrail,
Not sure if I understand your idea correctly. Are you saying

in front of the extProc filter we have a gRPCFraming filter. During deocdeData time, it will return StopIterationAndNoBuffer
and keep buffering the data. When it sees the end of the first gRPC message, it calls Continue and inject the data back to the filter chain.
The extProc filter's STREAMED mode will be changed to support header buffering and header mutation for the first dataProcessing. In this case, the filter buffers the headers first and then receives the first gRPC message in the first decodeData call. It will forward the data of the first gRPR message to the extProc server and the server will respond the header mutations. The filter finishes the header mutation, propagates headers and data.

Are we allowed to make ext_proc server stateful and buffer data?


Crashing due to getMacAddress is not a function

The plugin has been running for months but suddenly today it keeps crashing due to the the error below:
[6/17/2024, 10:27:36 AM] [Daikin Local Platform] Restoring accessory 'ワークスペース' (5026EFAE30CC) from cache.
[6/17/2024, 10:27:36 AM] TypeError: cachedAccessory.context.device.getMacAddress is not a function
    at DaikinPlatform.checkDevices (/usr/local/lib/node_modules/homebridge-daikin-local-platform/src/platform.ts:131:53)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at DaikinPlatform.configurePlugin (/usr/local/lib/node_modules/homebridge-daikin-local-platform/src/platform.ts:72:5)
[6/17/2024, 10:31:07 AM] Got SIGTERM, shutting down Homebridge...

Tried reverting to 1.0.10 but the crash still continues.
System: macOS 12.7.5 on Intel Mac mini
Nodejs: v20.14.0

Resolved by removing all plugin cache.


WSL Instance-1 <---> Instance-2 Communication

Is your feature request related to a problem? Please describe.
I use ansible a lot for my development workflows. During testing I run the playbook on the control node which will trigger the plays on multiple nodes. Since all the WSL2 instances regardless of the distro, they are running share the same network-space have same IP address. Because of this it's difficult to use such kind of development workflows
Describe the solution you'd like
A VNIC would be best which can assign different ip address to different instances. The same VNIC can also be used for port forwarding(done preferably using cmdline or GUI of VNIC accessed from windows) host port to instance port (similar to virtualbox)
Describe alternatives you've considered
The solution described above is possible in virtualbox where we can create NAT network(not same as NAT ❌) and assign multiple VM's in the same subnet. This can be used for both VM-to-VM communication(different ip addresses) as well as VM-to-hostORInternet(port forwarding)
Additional context
Add any other context or screenshots about the feature request here.

AF_LOCAL or AF_UNIX can be used to communicate between two WSL2 instances.

@Biswa96 Unfortunately AF_LOCAL cannot be used with ansible. We have inventory file in ansible which has the IP address of all the hosts where it will run the plays.
Also AF_LOCAL can only be used for process-to-pricess communication running in the same machine

In fact, multiple distros are running in the same VM, sharing the same kernel, so also having the same IP address.
The file system is isolated, except the /mnt/wsl directory, so you can create AF_UNIX sockets there from one distro and connect from another.


[MB-16227] Set a nil destination address for an HHG to new duty station location address

Jira ticket
Summary
Coming soon...
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Messages

:book:
:link: MB-16227

  Generated by :no_entry_sign: dangerJS against c99e9e8edad9610b61511931f346788642f332bf


imshow RGB plot overlaps map borders when using projection

xarry version: '0.14.0'
matplotlib version: '3.1.1'
When I create an RGB plot using ds.plot.imshow with a procection, the plot overlaps the map borders:
import cartopy.crs as ccrs
import xarray
import matplotlib.pyplot as plt

ds = xr.load_dataset(...)

ax = plt.subplot(projection=ccrs.Robinson())
(ds.isel(time=[0, 4, 8])).plot.imshow(robust=True, ax=ax, transform=ccrs.PlateCarree())

When using ds.plot.imshow with a single layer, the plot looks as expected:
import cartopy.crs as ccrs
import xarray
import matplotlib.pyplot as plt

ds = xr.load_dataset(...)

ax = plt.subplot(projection=ccrs.Robinson())
(ds.isel(time=0)).plot.imshow(robust=True, ax=ax, transform=ccrs.PlateCarree(), add_colorbar=False)

Does this work if you use matplotlib directly? i.e. plt.plot(ds.isel(time=[0, 4, 8]).values, ...)

Does this work if you use matplotlib directly? i.e. plt.plot(ds.isel(time=[0, 4, 8]).values, ...)

@dcherian Thanks, yes, this works as expected:
ax = plt.subplot(projection=ccrs.Robinson())
ax.imshow(ds.isel(time=[0, 4, 8]).transpose('lat', 'lon', 'time').values[-1::-1, :, :], transform=ccrs.PlateCarree())

Thanks, I'm marking this as a bug.
If you have the time, it would be nice if you could look into this and send in a PR.
We do some RGB stuff here:
https://github.com/pydata/xarray/blob/8a650a11d1f859a88cc91b8815c16597203892aa/xarray/plot/plot.py#L632-L642
and imshow is here:
https://github.com/pydata/xarray/blob/8a650a11d1f859a88cc91b8815c16597203892aa/xarray/plot/plot.py#L843-L915

Thanks, I'll give it a try


checkov: 3.1.9 -> 3.1.10

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.1.9...3.1.10
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.1.10
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

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

2 packages built:

checkov
checkov.dist


Pep8

just initial pep8 didn't clean all the stuff

Thanks a lot @julianhille! The code was very messy in some parts from a coding style perspective and others still urgently need a refactoring to reduce complexity, but this is definitely a good start to get things going in this aspect.
I made some comments while looking over your changes, but as you mostly adapted the formatting it mainly wasn't you that introduced those things I might do differently ;-)


Package proposal: pafish.vm

Package Name
pafish
Tool Name
pafish
Package type
ZIP_EXE
Tool's version number
0.6
Category
Utilities
Tool's authors
Alberto Ortega, Others
Tool's description
Pafish is a testing tool that uses different techniques to detect virtual machines and malware analysis environments in the same way that malware families do
Download URL
https://github.com/a0rtega/pafish/releases/download/v0.6/pafish64.exe
Download SHA256 Hash
ff24b9da6cddd77f8c19169134eb054130567825eee1008b5a32244e1028e76f
Why is this tool a good addition?
Pafish is a testing tool that uses different techniques to detect virtual machines and malware analysis environments in the same way that malware families do.

Note: Windows Defender and some other AVs falsely flag this EXE as malware, because it does many of the same VM/sandbox checks that malware does.

pafish is ultra dead, al-khaser is way much better for VM detection https://github.com/LordNoteworthy/al-khaser

al-khaser does not provide compiled binaries, otherwise, I'd vote to add it instead of pafish.

It seems there is a build workflow, but the result is only uploaded as artifact (which means it is only kept for a short period of time). It should be easy though to convert it into a release workflow. @mandiant/flare-vm do you think this is a useful tool that should be added to FLARE-VM?

I do not think al-khaser or pafish make sense in flare-vm. They are useful if you are writing anti-anti-vm or anti-anti-dbg tooling but there is no situation I can think of where you'd run one of these tools to better understand a malware sample


inference

You guys are doing a great job, and I have a few questions for you:
1 What is the effect of outpaint without lora training?
2 Have you tested the maximum times of outpaint original?
3 There is no direct inference script

Full finetuning the model like DreamBooth will make the model completely overfit to the reference image, LoRA and some additionally small tricks should be used to avoid that.

I dont understand this question

Just slab the original inference script of diffusers is fine, but I will add it in this repo

hello, another question to bother you. you set the validation in the code, but the output validation images seems not to be saved.

The images are already saved in tensorboard where you can view both the loss, images and some other hyper-param. I find it more convenient to visualize this way.

thank you!  can see the 4 validation images in the tensorboard.


Generate r2 score figures and sc crops

Generate r2 score figures and sc crops
In this PR, R2 scores are are calculated and figures are generated. As well, min/max single-cell crops are generated but not saved to the repo.

@MikeLippincott Thank you for the review! I am merging now!


Lf partner preview

This PR adds configurable customisation options for Open Targets partners, as defined in the epic https://github.com/opentargets/platform/issues/1714
Since it requires some local configuration for development, a number of issues were addressed in the same branch/PR: I understand this is not ideal, so the following description will be as thorough as possible.
Intro notes
Config
In production, the configuration will be handled and set by @mbdebian in back end team
The preview (Netlify) works thanks to some extra config being set just for preview purposes (see below “before merging into main”)
API
Currently uses a separate partner-platform API, which returns “private” data such as specific facets and widgets.
This API is not compatible with the new target profile endpoint/widget.
Before merging into main

Recent commits in main will be merged after review (since target profile page breaks with new production API), but before merging back into main
Remove Netlify specific config from both netlify.toml and config.js, essentially all those process.env.REACT_APP_ ….
The API will need to be updated with the latest changes to target endpoint.

Addressed in this PR
Set relevant configuration for partner preview #1729
In the config we set:

(API)
flag for partner (private) instance
primary colour
colour range for heath maps
list of private widgets for each page (profile and evidence)
list of widgets to hide for each page
list of private data types
lift of data types to hide

Page header #1718
Add a “lock” icon with tooltip to evidence and profile pages. This lock icon component is reused also in facets and widgets.
Widgets
Add lock icon component to the widget header/title based on config (i.e. if the widget is “private” and this is a partner instance).
Widgets have an optional isPrivate field defined in index.js for each widget (currently set only for relevant widgets, others will be updated later). Its value is overwritten via config.js.
OTAR Projects widget #1720
Update to load in data from the API
OTAR Crispr widget #1721
New widget on evidence page
Associations page #1730
Datatypes and datasources also have isPrivate field. Currently this is hardcoded for datasources, while datatypes can be configured via config.js. We'll decide if this is the best approach or whether everything should be configurable or hardcoded.
Heatmap

Update colours based on custom colour range
Hide columns based on list of datatypes to hide

Facets

Show private datatypes if it’s partner instance
Add lock icon component next to private facet
Hide datatypes facets based on config (list of data types to hide)

Data download #1717
Add simple info note to the downloads page

@mirandaio and @carcruz, can I ask you to please review the code in this PR? I have checked the UI while on the EBI VPN and everything looks good. I'm keen to get this PR merged so that we can test out main with all of the changes made by @LucaFumis.
Once you have approved the PR, @LucaFumis can adjust the Netlify config before merging into main.


Use .netrc for downloading with youtube-dl

Please make sure these boxes are checked (put an 'x' inside them) before creating the pull request.

[x] I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
[x] I have tested my changes on Python 3.5/3.6

Description
Use .netrc for downloading with youtube-dl so that we can play some videos which require logging in (such as niconico).
This youtube-dl option is documented: Authentication with .netrc file and YoutubeDL.py#L146.
Related issues (if applicable)
#457, #1232

I see no issues with this, thanks.


HTML improvements

add interaction radius as a per-particle parameter
update the position after the forces are stored.
This produces less 'missiles' but more pulsar.
Can be reverted later.
add a simplistic UI in HTML. I'm a noob in web UI, so
this is probably non-sensical code! Oh, and CSS is a
nightmare!

By changing 'Colors[]', one can use more or less particles
types easily now.

@skal65535 Wow this is amazing! I think people who have less JavaScript background will love it. However, GitHub tells me this branch has conflicts with the current file. There was another pull request today which made performance improvements.  It Improved the frame rate by 2-3 times. I guess your changes might conflict with that one. I don't know how to resolve these conflicts. If it is easy for you, it will be great if you could add your HTML improvements on top of our latest branch, then when merge the pull request without conflicts. Thanks a lot in advance.

@skal65535
also I noticed that after the last update when I click "R" on keyboard,
the seed, hash, and rules changes but not updated on the canvas.

@skal65535
@hunar4321
also I noticed that after the last update when I click "R" on keyboard,
the seed, hash, and rules changes but not updated on the canvas.

also the hash was removed.

@skal65535
@hunar4321
also I noticed that after the last update when I click "R" on keyboard,
the seed, hash, and rules changes but not updated on the canvas.

@skal65535
@hunar4321
also I noticed that after the last update when I click "R" on keyboard,
the seed, hash, and rules changes but not updated on the canvas.

Interesting! I didn't play much with the JavaScript version as I was focused on the C++ one but obviously the previous pull request made some changes to the JS file that makes it harder for anyone to follow the script because it uses matrices, but the frame-rate increased dramatically which is also a nice thing!

hmm... that's strange, my local version (with the speed-improvement patch) seems to work ok with the 'r' key, at least on Chrome + Mac M1.
I made a capture here: https://www.youtube.com/shorts/PsQEhX636-M
I'll certainly try to sync to HEAD and re-test!

sorry, i meant: "...my local version (without the speed-improvement patch)..."

hmm... that's strange, my local version (with the speed-improvement patch) seems to work ok with the 'r' key, at least on Chrome + Mac M1. I made a capture here: https://www.youtube.com/shorts/PsQEhX636-M
I'll certainly try to sync to HEAD and re-test!

The GUI addition is really a cool one and the patterns you generate in your video are amazing! It will be amazing if this is integrated with the speed improvements. That way more particles can be played with.

'r' key only stopped for me after the speed update,
I'm not sure why, and weather its only me,
and the fps went x3.5
you people are really making some interesting scripts

Hi, sorry to intervene here too. I made the performance improvements, I'm also commiting some code that uses a gui also (uses lil-gui, a favourite of three js users. I think it might be easier to hehe may next PR first and then add these further improvements from @skal65535 . Apologies if this is an imposition. In fact, I'll do it now...

I just added my PR, I'm happy to help @skal65535 with incorporating these changes tomorrow, I do think I prefer my attempt at the GUI, but it's up to you as always. My GUI parameterises anything you want.

That is awesome guys. I never thought JavaScript is that cool. I may give up C++ and learn more JS 😅

By the way, I see that the symmetric rules also produce some nice interesting patterns! I haven't systematically investigated the difference between symmetric and asymmetric rules but there was a discussion online that we only see these interesting patterns because we are breaking Newton's third law.

let's drop this one, the new UI has definitely more potential and is nicer!


⚠ Garbage Collector has degraded performance

In ec0eae2, Garbage Collector (https://gc.fef.moe/login) experienced degraded performance:

HTTP code: 200
Response time: 14536 ms

Resolved: Garbage Collector performance has improved in fe995e9.


Add oneOf type validator

This pull request closes #30 by adding a simple oneOf validator:
  class Foo extends EmberObject {
    @type(oneOf('red', 'blue', 'green'))
    @argument
    bar;
  }

Tests closely mirror shapeOf.

Thanks so much for doing this, awesome work 😄!

Publish in v0.8.15

Thank you, @allthesignals! I really appreciate that 😍
One more thing: do you think it'd make sense to allow any primitive value instead of just strings?

One more thing: do you think it'd make sense to allow any primitive value instead of just strings?

@buschtoens yes! Happy to work on that as well.


npm audit fix

Just ran npm audit fix to fix vulnerabilities

Pull Request Test Coverage Report for Build 173

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

Totals

Change from base Build 170:
0.0%

Covered Lines:
136

Relevant Lines:
136

💛  - Coveralls

The build error is weird. I can reproduce it with node v13.13.0 on my local machine, but it also fails on master with that node version. With v13.12.0 both master and my PR are building without problem.

Thanks. It's likely a 13-specific error that will resolve itself over time.
Shall we maybe for simplicity close this one? I always update the packages anyway before a version bump, and package-lock.json only affects dev, not production.

Hmm, I usually do it the other way arround, to not start developing new stuff based on outdated versions. If something breaks like here I would have suspected my changes to be the cause. But it's fine for me if you prefer otherwise

Is these changes indeed, but indirectly; some other package broke in an in-range update. I assume they will fix, given that it’s Babel.

The package in question was fixed; did a full lockfile update in c8e8362f3b797e794336c631a0c1937d91206918.


Be API compatible for set_packet_send_interval

See https://github.com/sdwilsh/brultech-serial2mqtt/issues/214 for context on how this was found.

Seems totally reasonable to me. My library just ignores the return value: https://github.com/jkeljo/greeneye-monitor/blob/master/greeneye/monitor.py#L440


Bumping go version to 1.20.7

What this PR does / why we need it:
Bumping go version to 1.20.7 to get rid of CVE's in previous versions.
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?

Additional Notes for your reviewer:
Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

/cherry-pick v0.55.x

/cherry-pick v0.54.x

/cherry-pick v0.53.x

/cherry-pick v0.57.x


Category (topic tags) pages don't work.

Category (topic tags) pages don't work.
Need to figure out what Justin broke? They were working at one point but fixing all the link stuff in #2  appear to have broken them?

fixed but there may still be a few broken links on it? Open another issue if found


Failed after 30seconds

Tried using this and the producer works and creates messages and sends to Rabbit MQ. However, RMQ fails everytime after 30seconds.
[PhpAmqpLib\Exception\AMQPRuntimeException]
fwrite(): send of 21 bytes failed with errno=104 Connection reset by peer
And in the RMQ logs:
2019-02-07 14:31:31.018 [info] <0.1020.0> accepting AMQP connection <0.1020.0> (<IP_ADDRESS>:40438 -> <IP_ADDRESS>:5672)
2019-02-07 14:31:31.023 [info] <0.1020.0> connection <0.1020.0> (<IP_ADDRESS>:40438 -> <IP_ADDRESS>:5672): user 'corecog' authenticated and granted access to vhost '/'
2019-02-07 14:32:01.379 [error] <0.1020.0> closing AMQP connection <0.1020.0> (<IP_ADDRESS>:40438 -> <IP_ADDRESS>:5672):
{writer,send_failed,{error,timeout}}
Consumer works for 30 seconds (records updated in ES instance) but then, as the error describes, timesout.
Have you experienced this?

Am trying to reproduce your issue now but without any luck.
Queue is consumed perfectly.
Still have the same issue?

Fixed the issue but I don't remember what it was. Having used it for a while now it might be useful to add common problems somewhere. I can add PR with documentation for common issues if you think it will be useful?

That sounds great
Waiting for your pr


File not uploading in the "file upload question text section"

Is your feature request related to a problem? Please describe.
While trying to upload file in the shiny new program, its gets uploaded but when you hit next, it takes you back to page 1. And if you hit review and check the "file upload question text section", the uploaded file cannot be seen.
Describe the solution you'd like
To be able to successfully upload the file and be seen in the review section post uploading.
Additional context

While uploading File

After hitting next

After hitting review

Done when
What conditions need to be met before this issue can be closed?

Thanks! A few questions:

How large was this PDF file?
Does it work if you upload smaller PDFs?
Does it work if you upload large PNGs or other image files?
What happens if you upload Word docs or some random file type?

I tried with three PDF sizes: 25 kb, 56 kb & 1491kb
It didn't work for either of the above PDF sizes
It didn't work with image files as well
Word and Excel files didn't work either.

Thanks! cc'ing @hmhsu in case he can figure out what's going on here.

Hey @ialv44 thanks for reporting the issue! Have you tried clicking -- FILE UPLOADED -- (click to download) text and verified it is not downloading? Currently the file name is not retained after upload so you wouldn't be able to see the original file name.

@ialv44 we've made quite a few improvements to file upload question in the last week. Now the uploaded file name is shown in both the file upload question screen and the review page. Closing this as I think the issue is sufficiently resolved. Feel free to open a new issue if you find additional things missing.


Compiler does not reset free vars after partially failed overload match

Given
def foo(x : T, y : String) forall T
  1
end

def foo(x : Char, y : T) forall T
  2
end

foo('a', 1) # Error: no overload matches 'f' with types Char, Int32

Neither overload is more specialized than the other, so overload resolution proceeds in the order they are defined. The first one fails because y isn't a String, but at this point T = Char is a successful match, so the second overload becomes f(x : Char, y : Char) forall T, and matching fails again because y is an Int32, not a Char.
If the two defs are swapped, or if one of them uses a different name for the free var, the snippet properly prints 2:
def foo(x : Char, y : T) forall T
  2
end

def foo(x : T, y : String) forall T
  1
end

foo('a', 1) # => 2

def foo(x : T, y : String) forall T
  1
end

def foo(x : Char, y : U) forall U
  2
end

foo('a', 1) # => 2

c.f. https://github.com/crystal-lang/crystal/pull/10185#issuecomment-762546171

Thank you for taking the time to not only explain these issues in detail, but also fix them ❤

Thank you for taking the time to not only explain these issues in detail, but also fix them ❤


Update definition of reference result and benchmark result

According to the rules:

A reference result is the median of five run results provided by the MLPerf organization for each reference implementation
A benchmark result is the median of five run results normalized to the reference result for that benchmark. Normalization is of the form (reference result / benchmark result) such that a better benchmark result produces a higher number.

I believe we have changed the definition of a result, so we should update these accordingly. I think we are using means and the # of run results varies by benchmark.

Yes, will do.


Run Project doesn't automatically focus the game window

Tested versions

Tested in v4.3.rc2
Not present in v4.2

System information
Godot v4.3.rc2.mono - Ubuntu 24.04 LTS 24.04 - Wayland - Vulkan (Forward+) - integrated Intel(R) HD Graphics 4400 (HSW GT2) - Intel(R) Core(TM) i7-4600U CPU @ 2.10GHz (4 Threads)
Issue description
With v4.3 (rc2), running a project with F5 or with the Run Project button, focus is not on the newly-created window, but on the Godot editor instead.
NOTE: Cannot reproduce consistently.
Steps to reproduce
CANNOT REPRODUCE CONSISTENTLY.
The display mode has to be set to Exclusive Fullscreen or Fullscreen according to my current tests
Here's how I have gotten the bug to occasionally happen:

By upgrading a 4.2 project to 4.3, then immediately following the following steps:

Open a script file (to easily test focus).
Run project (in any way).
Press Enter a few times
Go back to the editor view, and see if you typed any line breaks.

Open a 4.3 project, then fullscreen it (Shift + F11).

Follow the above steps

Minimal reproduction project (MRP)
N/A

I tested with 4.3.rc2 on Fedora 40 KDE Wayland, with both the X11 and Wayland DisplayServers, and I can't seem to reproduce the issue.

I can confirm this happens on Linux Mint 21.3 - X11 - Forward+ Vulkan - 4.3.stable (standard version), I have also been experiencing inconsistent focus issues running my project that were not present in 4.2, though I am experiencing it with my game in windowed mode (have not tested fullscreen).

My issue must be unrelated as I bisected this to #93682 being the cause on my system, which is specifically for X11.


Add: Global Composer Dependencies to README

Adds documentation about how to handle global composer dependencies.
Not certain if you need to reshim after you change php versions?
See #54 and #55

Looks good. Thanks @ctsstc.
@vic By the way, is there a smartest solution to avoid this? I think asdf-golang needed a similar process; it's certainly not very smart.

as far as I can remember, there's no other way around installing new package binaries than regenerating via reshim (that is without using something like direnv).
plugins can having a bin/bin_paths (can't remember if that's the actual hook name ) can provide a list of directories containing executables, but again that's for asdf to be able to generate shims for them.
there's a plug-in file bin/env (again can't remember the name right now) where you can export environment variables but they are intended to setup a correct environment immediately before shims execution (eg GOPATH).

It might be best to recommend adding some paths to the user at README, but that conflicts with the previous PR, so I will merge this PR for now.

Thanks again @ctsstc.


✨ [Feature Request]:  Implement "Scroll Down Progress Bar"

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Hey @HimanshuNarware and @Shubh942
I would like to suggest adding a scroll down progress bar to the website to enhance user experience.
Use Case

A scroll progress bar visually indicates how much of the page has been scrolled and how much is left, providing users with a better sense of their position on the page.
This feature can be especially helpful for long pages blogs, docs, and articles.

Benefits

Enhances user experience by providing visual feedback.
Useful for long pages or articles.
Improves overall navigation and user satisfaction
Looks more engaging.

Add ScreenShots
like this

Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@HimanshuNarware and @Shubh942 please assign me this issue under gssoc.

Looks good @MastanSayyad


Migration On Enum

DBFlow Version: 3.1.1
Issue Kind: Question
Description:
Hello Good People of DBFlow I am A Big fan of your work. Love and recommend It basically.
So Here is a Question
Can any one explain How to do Migration On enum will work i want to add onther type actually
`@Table(database = ClinicDatabase.class)
public class Ticket extends BaseModel implements Parcelable {
@PrimaryKey String id;
@Column TicketStatus status;

public enum TicketStatus {
    visited,
    visiting,
    waiting,
    pending_checkin,
    no_show,
    cancelled
}

}
so I want to add missing to TicketStatus
public enum TicketStatus {
visited,
visiting,
waiting,
pending_checkin,
no_show,
cancelled,
missing
}
`

Enums just convert to SQLiteType.TEXT. So you can add as necessary. If you remove one, itll convert to null.


Add utility function for annotating queries with the full sector name

Description of change
Adds a helper function that can be used to generate a subquery that can be passed to QuerySet.annotate() to annotate a query set with the full sector names.
This will be used when defining query sets for reports and data exports.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #1066 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1066      +/-   ##
===========================================
+ Coverage    96.18%   96.18%   +<.01%
===========================================
  Files          216      217       +1
  Lines         6180     6189       +9
  Branches       594      594
===========================================
+ Hits          5944     5953       +9
  Misses         166      166
  Partials        70       70

Impacted Files
Coverage Δ

datahub/metadata/query_utils.py
100% <100%> (ø)

Continue to review full report at Codecov.

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


Wrong error message when setting object in an RDF Literal table

I issued the following query:
kuzu> MATCH (a)-[p:TurtleTest_lt]->(o)
WHERE p.iri = "http://kuzu.io/rdf-ex#population"
SET o.val=cast(true, "BOOL")
RETURN p.iri, o.val;
Error: Binder exception: Cannot set property of o because it refers to table TurtleTest_r under rdf graph TurtleTest.

o must be in TurtleTest_l not TurtleTest_r.
So is it the case now that we cannot modify anything in RDFGraphs? First, I think we can allow them to set literals. I think that's fine. But if this is much easier, so we force them to delete and insert triples, then we should just give a better error message. Something like: Error: Binder exception: Cannot set properties of RDFGraph tables. You are trying to set o.val but o is part of table TurtleTest_l under RDFGraph TurtleTest.
Also Let's use the term RDFGraph instead of "rdf graph", which is what you have used in the error message.

Fixed in PR #2800


Composer installation

I have a question, is this proper package for composer https://packagist.org/packages/smith197/steamauthentication ?
As there are no information what-so-ever in README of the project. When I'm trying to install it with composer I receive error.

[InvalidArgumentException]
Could not find package smith197/steamauthentication.
Did you mean this?
smith197/steamauthentication

Can you shoot us a PR so we can get this to work properly?

Sure - https://github.com/SmItH197/SteamAuthentication/pull/206
After it will be merged if package was properly configured on packagist.org it will automatically identify changes. If not @MartinBecker need to do it manually or give maintainer to someone here who can do it manually or configure for it to happen automatically on every git push request.


build test error: undefined reference to `printf_s'

there is an undefined function in the test file:
/usr/bin/ld: build/sutest.o: in function `TEST':
/coding/lock-free-ring-buffer/test/sutest.c:56: undefined reference to `printf_s'
/usr/bin/ld: /coding/lock-free-ring-buffer/test/sutest.c:54: undefined reference to `printf_s'

You probably tried to build it on Linux or macOS (not Windows)...
But anyway, the file sutest.c has been changed to use only standard printf(), not the safe version.
Please don't complain that the "unsafe" printf() is used...
--MMS

thanks for the update.
btw, is the library suitable to pass a memory address (64 bit) ?

The LFRB (Lock-Free Ring Buffer) should work with 64-bit elements, such as pointers on a 64-bit CPU. A section in REDME talks about it.
--MMS

This original issue with printf_s() missing in sutest has been resolved by moving to the Embedded-Test framework. With this comment, the issue is closed.


Minimal Genera support

Genera doesn't yet implement PPRINT-LOGICAL-BLOCK.
So, don't use '~@<…~@:>' in restart descriptions on Genera.

Thanks, merged.


CSV generated by the code in the "Human Pose Classification with MoveNet and TensorFlow Lite" example inconsistent

Click to expand!
Issue Type
Documentation Bug
Source
source
Tensorflow Version
2.9.1
Custom Code
No
OS Platform and Distribution
Windows
Mobile device
No response
Python version
3.8.0
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Running part 1 here (after running the preparation code) generates CSVs different from the CSVs in the section here.
The CSVs should be the same, but the CSVs generated by running the code has discrete keypoint coordinates while the CSVs linked in the above section have decimal keypoint coordinates. I believe that the code in the example is missing some scaling code. In the def process(self, per_pose_class_limit=None, detection_threshold=0.1): function, under the # Get landmarks and scale it to the same size as the input image comment, I don't think the landmarks are actually scaled:
pose_landmarks = np.array(
    [[keypoint.coordinate.x, keypoint.coordinate.y, keypoint.score]
        for keypoint in person.keypoints],
    type=np.float32)

Yoga CSVs generated by running the code:
yoga_train_data.csv
yoga_test_data.csv
Yoga CSVs linked in the code for skipping part 1 which claims to be identical:
yoga_train_data.csv
yoga_test_data.csv
Standalone code to reproduce the issue
https://www.tensorflow.org/lite/tutorials/pose_classification
Relevant log output
No response

Hi @rjfc
The issue seems to be resolved with TF 2.12. I have observed with the test_data.csv which has the  has discrete keypoint coordinates.
Please find the gist here and the head data below:

index
file_name
NOSE_x
NOSE_y
NOSE_score
LEFT_EYE_x
LEFT_EYE_y
LEFT_EYE_score
RIGHT_EYE_x
RIGHT_EYE_y
RIGHT_EYE_score
LEFT_EAR_x
LEFT_EAR_y
LEFT_EAR_score
RIGHT_EAR_x
RIGHT_EAR_y
RIGHT_EAR_score
LEFT_SHOULDER_x
LEFT_SHOULDER_y
LEFT_SHOULDER_score
RIGHT_SHOULDER_x

0
chair/guy3_chair070.jpg
150.0
101.0
0.69633734
155.0
96.0
0.81174135
145.0
96.0
0.7756405
163.0
97.0
0.7792505
140.0
97.0
0.5823091
172.0
112.0
0.80950165
133.0

1
chair/guy3_chair071.jpg
151.0
103.0
0.7204095
155.0
99.0
0.86422026
146.0
98.0
0.80267566
163.0
99.0
0.7085024
141.0
99.0
0.54238826
173.0
113.0
0.79421246
132.0

2
chair/guy3_chair072.jpg
150.0
105.0
0.724821
155.0
100.0
0.84365886
146.0
100.0
0.79857314
164.0
101.0
0.74888885
142.0
101.0
0.6859938
174.0
114.0
0.75105125
134.0

3
chair/guy3_chair073.jpg
150.0
107.0
0.7161201
155.0
102.0
0.63974524
146.0
101.0
0.5465567
163.0
103.0
0.7929039
142.0
102.0
0.725726
174.0
116.0
0.85323805
134.0

4
chair/guy3_chair074.jpg
149.0
109.0
0.49887785
154.0
104.0
0.7880822
145.0
103.0
0.65956235
163.0
104.0
0.57862025
142.0
105.0
0.62644887
174.0
117.0
0.7471118
134.0

Thanks.


Question about the results

I followed your README, modifying nothing of your code. Here is the final results, not that promising. What's more, I find that sample-100, sample-200 and sample-300 did not change much.  Can u share how to improve it?

please refer to https://github.com/showlab/Tune-A-Video/issues/15

Thank you, it seems like the xformers installation problem


Add an ability to replay and loop FIFO data

This is a re-submission of a PR created by @akumanatt. See #11. The changes Natt made were simply re-applied to the (now refactored due to the FX merge) main branch. In fact only the changes to top.v had to be adjusted. This was done by hand as was therefore prone to human error.
IMPORTANT NOTE: the submitter (JeffreyH) did NOT create this change and does (as of yet) NOT know whether it would work in real HW!
Orignal text from @akumanatt (Jun 10, 2023):

This update adds 2 write bits to VERA registers: AUDIO_CTRL bit 6 and AUDIO_RATE bit 7.
Writing 1 to AUDIO_CTRL bit 6 will only reset FIFO's read position to 0. This allows a sample data in the buffer to be replayed any time without refilling it again. AUDIO_RATE bit 7 set will enable looped playback where the read position is automatically reset to 0 if the buffer becomes empty in the next sample. Although an AUDIO_RATE value of 128 will still play at the maximum rate without looping.
Note that a position of 0 here means an internal buffer's address. Which means a FIFO reset (write 1 to AUDIO_CTRL bit 7) is required in order to reset the write position to 0 too before uploading a sample data and make these methods above play correctly.

Comment/review by JeffreyH (Jun 16, 2023)

I have tried to include these changes into the fx-branch. After doing so I found out a few things:
The change raised the LUT count by 46, which is more than you would expect.
Attempts to reduce the LUT-count were partially successful (down to 16 LUTs) but resulted in a different way of interfacing with the looping/reset settings, which may be undesired.
The change resets rdidx_r (in audio_fifo.v) when a restart of the read position is triggered. However, the state_r (in pcm.v) is not reset. This means that the fifo-reading might be in an intermediate state (say: 8-bit stereo, and only the left byte has been read, but not the right one) when the fifo read position is reset. This may cause strange/unwanted effects.
It is probably required to wait for the state to be DONE or IDLE before executing a read-reset. This means some logic needs to be added and/or adjusted.
It is probably prudent to re-think how the logic should work and how the interface should work for this feature. Then implement that. Then LUT-optimization (and review) can be performed.
JeffreyH

Comment by @akumanatt (12 Aug, 2023):

I updated the loop method to be a combination of AUDIO_CTRL bits 6-7 now. Also, I made it reset only in IDLE state.

After (trying to) synthesize:

Place and Routing failed.

ERROR - All 1 iterations failed with design error(s).
It is recommended to correct all design errors before running multiple iterations.
Please check the Place and Route reports (.par) for the individual iterations under the directory
"...\vera-module-fifo\fpga\impl\vera_module_impl_par.dir\5_1.par".
Done: error code 1

Cost: +26 LUTs (5015 -> 5041 LUT usage)

No analysis/review done.

After removing the syn_hier="hard" attribute (committed and pushed) in pcm.v and in psg.v the issue is resolved. This attribute was added (in the FX update) for a more efficient LUT-usage, but this can sometimes causes Place and Route issues. This does however come at a small LUT cost.
After synthesizing:

No timing errors during Place and Routing
Cost: +31 LUTs (5015 -> 5046 LUT usage)

No HW test or analysis/review done.


Make example branches

For example:

chunk, for a system without flakes
unwrapped for a basic flaked system that does not use a lot of wrapper functions, but shows how to use flakes to build the OS and use home-manager (and agenix)

iso for a flake to build live systems, as described by hoverbear (and then fold it into the unwrapped branch with the slight changes in variable names and conventions)

This doesn't seem worth the effort anymore -- I want to use flakes moving forward.


Update options.mdx

Minor typo mistake
-signature-verfication-failure
+signature-verification-failure

Can you update the same content currently in the website/ directory? We're in the process of migrating our site and the current site at flow.org has content from website/, so both need to be updated. After you update I can import and merge

Okay, sure 👍


Default region to .aws/config

With reference to:

https://github.com/sst/sst/issues/2237

Is it possible to have this implemented? Would be a good convenience while setting up SST projects.

Can you clarify this?
You want the region to be taken from the AWS config when a new app is created through sst init?

Can you clarify this?
You want the region to be taken from the AWS config when a new app is created through sst init?

Yes. Correct.

Don't we do that right now?
Or are you saying you want this added to the sst config?


[stable/spinnaker] Use gcs.secretName when it exists for install-using-hal job

If using an external secret for the GCS json key the install-using-hal job will fail to start because it does not honor this value.
Signed off by: Adam Setters<EMAIL_ADDRESS>
[x] DCO signed
[x] Chart Version bumped

/lgtm

/assign
/ok-to-test

/retest

/lgtm


[libaaplus] Use versioned download link

A download link with the version in the filename is provided now for
releases of aaplus.
E.g. http://www.naughter.com/download/aaplus_v2.12.zip instead of
previously http://www.naughter.com/download/aaplus.zip, which was
overwritten, when a new version was released.
Describe the pull request

What does your PR fix? Fixes issue #
The PR implements a stable download link for a released version of aaplus.
This avoids changing SHA512 checksums of http://www.naughter.com/download/aaplus.zip upon each release.
From now on, PJ Naughter kindly provides zip files of releases with the version in the file name.
E.g. http://www.naughter.com/download/aaplus_v2.12.zip
Which triplets are supported/not supported? Have you updated the CI baseline?
All current triplets are supported
CI baseline: OK, no need to update
Does your PR follow the maintainer guide?
Yes

LGTM, thanks for the PR!


Code-Text Styling Consistency

Context
I am opening this ticket for internally discussing the styling inconsistency in code-text, based on the issues raised in https://github.com/pydata/pydata-sphinx-theme/issues/1852. The goal is to gather thoughts, suggestions, and feedback before posting final recommendations to the original issue.
Documentation
I have created a document outlining the current issues and proposed changes. It includes detailed examples and comparisons with other industry standards. Link to Document: Code-Text Styling Consistency
This document is intended for easy reference and detailed explanations, because large tables and detailed descriptions can be difficult to read and write in GitHub comments. I will summarize the discussion points and action items from this document and post them here once finalized.
Figma Designs
I've also created some designs to illustrate proposed changes: Figma Design

Based on today's discussion. here are the final recommendations:

S.no
Category
Description
Example Pages / Links on PyData Sphinx Theme
Current Text Formatting
Recommendation / Discussion

A.
Navigation Pane
This refers to the headings in the Section Navigation on the left side of the webpage.
Section Navigation
Proportional
We should switch to Monospace. This is a recurring design pattern in other documentation themes and will help in clear identification of code-text.

B.
Table of Contents
Function signatures in ToC
Functions ToC
Monospace, Proportional
All text in the function signature should be monospaced. This includes function name, parameters, kwargs, type annotation, default values etc.

C.
Title Heading
Main headings on documentation pages which contain code-text.
Title Heading
Proportional
Code-text in headings should be monospaced, but normal text should be proportional. Also, code-text should not wrap or break. Example: 1. Heading with normal and code-text: This is a page about math.h 2. Heading with just code-text: math.h

D.
Function Signature
Representation of function names and parameters.
Function Signature
Monospace, bold, italic
This code text is very busy visually so we should simplify the text formatting. Here’s the redesigned function signature: Figma Design

E.
Parameter names in descriptions
Descriptions of function parameters.
Parameter Descriptions
Proportional, bold
Description text will remain proportional.

F.
Parameter Type
Description of parameter data types
Parameter Type
Proportional, italics
Description text will remain proportional.

G.
Function Source Link
Source link to the original function headers
Function Source Link
Monospace, bold
Source link does not refer to code text, and should be treated as a normal link and should be proportional text.

The most significant changes are in the function signature and definition formatting:

Class name is no longer bold.
Parameters now use a muted text color.
Default values are not bold and are displayed in green.
Type annotations are no longer bold and are shown in blue.
The source link text is proportional and no longer bold.
The heading text for parameters, returns, and return types now has a muted color.
Padding between paragraphs has been increased for better readability.

I've also added the summarised findings to the original issue: https://github.com/pydata/pydata-sphinx-theme/issues/1852#issuecomment-2312184516


DBZ-2240 Use 1.1 compatible API in tests

https://issues.redhat.com/browse/DBZ-2240

Tes failure unrelated, already reported


Homogenize algorithm headers

Right now some algorithms report "Linear Minimization Oracle: xxx" and others don't (in particular BCFW @JannisHal).
Moreover, we probably want to replace "Linear Minimization Oracle:" by simply "LMO: "

The reason for Linear Minimization Oracle was to have something explicit for users not familiar with FW-specific lingo

it is a little long though, especially for the block-coordinate one.


Move command history into WebUI

This way the WebUI users do not have to deal with command window

folks want the history file in the directory with the generated images

might i post a usecase. feel free to disregard or scrap post.
im tinking with different promts and CFGs to make beautifull city landscapes. i take a break from it for a day and return. its hard to know where i left off when i cant link the image to the promt used. i would want to know the model, steps, prompt, negative prompt, seed, CFG scale and which scheduler. right know i get all that in the excel file but having created 50ish images its impossible to sort through trying to remember which image came from which prompt. since i might be using the same seed but different promts.

I think this already added an png metadata. We can open new issues to track anything more we need.


Refactor Authenticator GetToken function to exchange token manually

The Authenticator GetToken function is failing with {"message": "Too Many Requests", "error": 429}. When I run manually I'm able to retrieve the token, so refactor this to exchange the token without using the oauth2 Exchange function.

Fixed - had an extra space in my client secret that was causing the issue.