fix(deps): update github-releases dependency kubernetes/kubernetes to v1.21.12

This PR contains the following updates:

Package
Update
Change

kubernetes/kubernetes
patch
1.21.6 -> 1.21.12

Release Notes

kubernetes/kubernetes
v1.21.12
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
v1.21.11
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
v1.21.10
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
v1.21.9
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
Release Assets

Kubernetes Source Code:  kubernetes.tar.gz
SHA2560fc6eab832e655b149686c4a64d264d9eff0be6f0f8d35ee9fdde349f950faab
SHA512525d981f5fa61abfd52df07e49a3bbe87a11e1e654d63e601124e35fb5b82d23390601562300cb0c1f62bdacb3bf9bb73c91067ab2cb0cb47b9b9cb34aadd289

v1.21.8
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
Release Assets

Kubernetes Source Code:  kubernetes.tar.gz
SHA25618ee476b7a2be6c40cac8f45477e1e165dc38c5f99ec2c708ae9295a347a20bb
SHA51284e6742f2341a7288b23dae90111a377621babf1bdd9e167a57c31b577b95594fd451e9acf21663e6f365f1a7cc6c983028819036facdb35066576b341af8d62

v1.21.7
Compare Source
See<EMAIL_ADDRESS>Additional binary downloads are linked in the CHANGELOG.
See the CHANGELOG for more details.
Release Assets

Kubernetes Source Code:  kubernetes.tar.gz
SHA25613e8b2c579e2122ee9b58700377e4050f1278fa718b90dea0e059f2101c077b2
SHA512272b73d76a8d90cea95940bf5b67eb04b9b5d118734398072955b556a9c05f72b5a200b1d546503a2ba106283d7e60c0c2634c2886bcc9fc3d6c0e045c3290de

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

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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

GitHub release
v1.0.0

Your semantic-release bot :package::rocket:


AutoSuggestBox validation

Is there any way to require the value entered into an AutoSuggestBox to be a valid AutoSuggestBoxItem and trigger the onSelected event with the Enter key?

You have control over the content of the auto suggest box content by providing a text controller. You can also pass the noResultsFoundBuilder parameter to build the dialog when no content is matched.
The onSelect event is already triggered on the enter key.


V1.8.1 maglev

Hi, we have implemented maglev hash based on facebook katran project.
Now I know that cilium will support maglev hash. I think our patch could make some help. Because now only external traffic will be processed by maglev hash by cilium.
The code have been test and run in our environment for a long time.
Thanks.

I've changed the base branch to 1.8 to make it easier to review.

@lrouter @AirbornWdd Hey, are you still planning to rebase against the Maglev implementation in the master branch? One feature which is missing is backend weights which you have implemented. It'd be great to get it.

@lrouter @AirbornWdd Hey, are you still planning to rebase against the Maglev implementation in the master branch? One feature which is missing is backend weights which you have implemented. It'd be great to get it.

OK. I will continue to work for it. It affects a lot of code.

This can be closed in favor of #13943.


Document issue with Gulp

This change is for GDS users with a managed device who cannot run Gulp.
fixes #1048

looks good thanks! Couple of questions:

do we need August in the heading?
should it use numbered list instead of bullets, as the instructions in the section above do?


Add FixPermissionsCommand. Closes #397

I can't remember why were we chmod./public, so it's removed. This will fix the storage directory too and only run on the database directory if needs be.

:+1:

Will squash.

Darn it.

Ok, now we can merge.

 Thanks so much @joecohens!

All done.

StyleCI = :heart_eyes:

Rebased.

Just need to remove the unused facade import now :)

We're almost there!

I can rebase and merge.

:+1:

Learnt a lot with this, thanks @GrahamCampbell

Awesome!

Thanks to both :smile: :smile: :smile:  !!


Added style or hover effect to social icon

Description
I would like to enhance the social icons by implementing a 3D hover effect and adding animations to make them more engaging and attention-grabbing. This update aims to improve user interaction and visual appeal, making the website feel more dynamic and modern.
Changes Made

Implemented a 3D hover effect on all social media icons.
Added smooth animations to make the hover transitions more fluid and visually attractive.
Adjusted icon styling to ensure consistent behavior across different screen sizes and browsers.

Screenshot

Please assign this issue to me @vishanurag


Cannot start copilotforxcode automatically when Xcode Launched

Before Reporting

[X] I have checked FAQ, and there is no solution to my issue
[X] I have searched the existing issues, and there is no existing issue for my issue

What happened?
I reinstalled my macOS and  xcodeforcopilot failed to launch itself when i open xcode（it worked perfectly before i reinstall my macOS），i dont know if it is a bug, or a feature which has removed recently. Everything works well, but every time I have to launch copilotfoexocde manually.
How to reproduce the bug.
It just happens!
Relevant log output
No response
macOS version
13.6.4
Xcode version
15.2
Copilot for Xcode version
0.30.2
Node version
21.6.1

When Copilot for Xcode.app launches for the first time, it will install a launch agent to the system. Please check if Copilot for Xcode is on the list, and make sure the toggle is on.
If it's not on the list, click remove launch agent and then setup launch agents in the app.
If it's still not working, you can find an old-school solution in Wiki/FAQ to setup the launch agent by hand.

bro it seems has been in the list and I reinstalled the agent in your way, but it seems not working...

Hmm, I think it's not about the launch agent. If the launch agent is not working, I believe the issue would become "the app is complaining that it can't connect to the XPC service".
Did you enable the Xcode source editor extension? The source editor extension is responsible for starting the app when Xcode launches.

dude it worked after a reboot! Thanks for your hard working'😄


txn: persist fair lock type in lock information and handle stale fair lock resolve

What is changed and how it works?

Issue Number: Ref #13298  https://github.com/pingcap/tidb/issues/43540
What's Changed:

Persist the lock with conflict information in the pessimistic lock type.
Check the persisted transaction information first resolving the primary pessimistic lock, if the lock is stale return
the actual transaction status and unlock the stale lock.

Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects

Performance regression

Consumes more CPU if there are lots of conflict pessimistic lock to resolve.

Release note
Fix the incorrect transaction status returned processing stale pessimistic lock with conflict.

/test

@you06
Checking if the transctiob status is ok theoratically, the only problem is the extra cost to get the committed key with certain "start_ts". Actually, for prewrite secondary index key even the write conflict check is skipped because of the cost and it brings us many corner cases.

Another thought is can we accept the conflict pessimistic lock from resumed requests only? So that we can reject the stale pessimistic lock because it's not in TiKV's waiting queue.

Another thought is can we accept the conflict pessimistic lock from resumed requests only? So that we can reject the stale pessimistic lock because it's not in TiKV's waiting queue.

@you06 I dont't quite get it, do you mean we check if it is a force lock by checking its existency in the waitting queue? Could you explain more about it in details?

@cfzjywxk If the a request acquired a lock successfully without awaking(from the fair lock queue), it's for_update_ts should be greater than the exist commit_ts, on the other word, if for_update_ts < commit_ts and the lock is granted, the request is awaken by AcquirePessimisticLockResumed. The stale pessimistic is called by AcquirePessimisticLock, which should not be able to acquire pessimistic locks because for_update_ts < commit_ts.

/test

/test

/merge

How will this affect TiFlash, is it safe to be ignored?


Nearby pin bubble at wrong place in right-to-left languages

Steps to reproduce:

Set your phone to Hebrew (probably also works with Arabic, Persian and Urdu)
Go to the nearby map
Tap on a pin
The bubble often appears at the wrong place

In this screenshot I actually tapped the pin at the left of the screen:

hi @nicolas-raoul ,
I would like to fix this.

@nicolas-raoul if the bug still exists, I would like to solve it :)

Yes it still exists, I checked yesterday. Thanks, it is yours! :-)

@nicolas-raoul Can I try to work on this issue?

@Claud-C Yes thanks! 🙂

I wonder if this issue still exists?  After I used a right-to-left language, my device made corresponding changes, but the problem mentioned in the issue did not appear in the software.

Thanks for testing! It might have been fixed, thanks for letting us know!
I will thus close for now, if anyone observes the issue again please do not hesitate to reopen 🙂

Since I couldn't reproduce the bug, I thought about it and came up with a few ideas.
First, what are the devices that reproduce this bug? Perhaps their common trait is the cause of the bug.
Second, is this bug only in Japan? The specific text content they provide may be the cause of this bug.
I'm sorry that I can't solve this issue, because I can't reproduce this bug and I don't have more ideas to solve this bug.

@Claud-C No need to be sorry, identifying what bug reports are no longer applicable is an important part of software maintenance. I guess another modification had the side effect of eliminating this bug. Thanks again and do not hesitate to check other issues if you want!


Fully convert monorepo to bzlmod

From what I understand about the design, this should allow us to depend on the monorepo from internal.git, while maintaining MVS semantics.
We should also set appropriate visibility rules to make sure we don't unintentionally expose APIs which we later regret.

This does add some workarounds for e.g. https://github.com/bazelbuild/bazel/issues/19301

And to completely remove WORKSPACE we need to wait for https://github.com/rmohr/bazeldnf/issues/60

This does add some workarounds for e.g. bazelbuild/bazel#19301

Are we blocked on this?

No we aren't. It's just not pretty :)

Technically not correct but we will leave it closed as we are currently only waiting for bazeldnf


Add number of test/train examples to report

This was some feedback mentioned in the peer review.

Done!


trace: consider batching in ExportSpan

It's great to see this exporter for OpenCensus from Honeycomb, congratulations!
I was just studying your exporter and noticed that in ExportSpan, per SpanData that's added it is converted immediately and then uploaded before ExportSpan returns.
ExportSpan is on the hot path and for example if we had 10K QPS all traces sampled say during a high latency spike, this would slow down the entire program since the number of SpanData being created will also be >=10K but also the network bandwidth from that upload of each span/event would spike too.
This is just a kind suggestion to perhaps perform some sort of batching. We talk about doing heavy work in a separate goroutine as per https://godoc.org/go.opencensus.io/trace#Exporter
and some exporters like Stackdriver use this bundler https://godoc.org/google.golang.org/api/support/bundler can allow you to set a "buffer time period" as well as a "batch max size".
Thank you.

Hi and thanks for the comment! I completely understand and agree that ExportSpan must be fast to return in order to avoid blocking the main execution of the app. Though it's insufficiently clear from the documentation, the Send() function invoked in ExportSpan is actually asynchronous. It adds the event to a sending queue that then gets batched up (with batch size being based on either time or number of events, as you mention happens in bundler) and sent eventually.
ev.Send() eventually calls tx.Add(), the function to add an event to the transmission queue. Its implementation is here: https://github.com/honeycombio/libhoney-go/blob/master/transmission.go#L72. We are using facebookgo/muster instead of bundler, and find it does an excellent job.
libhoney does offer control over batch size and timing.  Would you suggest also exposing that configuration at the level of the Exporter?


emq_sn_gateway crash after ACL deny

Environment

OS: Alpine 3.5
Erlang/OTP: 19
EMQ: 2.3.1

Description
After an ACL deny of a client that connects via the emq_sn plugin, something crashes:
12:28:24.177 [error] Client(123@<IP_ADDRESS>:58306): Cannot SUBSCRIBE [{<<"R/123">>,[{qos,1}]}] for ACL Deny
12:28:24.178 [error] gen_fsm <0.1563.0> in state connected terminated with reason: false in emq_sn_gateway:transform/2 line 591
12:28:24.178 [error] CRASH REPORT Process <0.1563.0> with 0 neighbours exited with reason: false in emq_sn_gateway:transform/2 line 591 in gen_fsm:terminate/7 line 559
12:28:24.179 [error] Supervisor emq_sn_gateway_sup had child sn_gateway started with {emq_sn_gateway,start_link,undefined} at <0.1563.0> exit with reason false in emq_sn_gateway:transform/2 line 591 in context child_terminated

This crash will be fixed in next official release.


[BUG][BACKPORT][v1.2.4] Instance manager crashes with invalid memory address or nil pointer dereference

https://github.com/longhorn/longhorn/issues/3466

Validated on v1.2.x-head 01/11/2021
Validation - Fail
I could see the nil pointer error in IM logs :
container (28).log
longhorn-support-bundle_afea219a-9014-4cdb-b09a-797243f377fe_2022-01-11T21-43-40Z.zip
I used kubectl command to install Longhorn from https://github.com/longhorn/longhorn-manager/tree/v1.2.x/deploy/install and replaced the image from master-head to v1.2.x-head.

@khushboo-rancher Which engine version are you using for the test? Please make sure the engine image is the latest one for v1.2.x.

I used engine image as longhornio/longhorn-engine:v1.2.x-head, IM - longhornio/longhorn-instance-manager:v1_20211210 and still see the problem.
@shuo-wu If they are not latest could you please update images in  https://github.com/longhorn/longhorn-manager/tree/v1.2.x/deploy/install "1.2.x' branch

The Drone build for the v1.2.4 backported PR merging failed hence the engine image was not updated. I just restarted it so that image longhornio/longhorn-engine:v1.2.x-head could contain the fix. Now everything should work fine.

Validation - Pass
Validated on v1.2.x-head 01/21/2022
time="2022-01-21T22:44:01Z" level=error msg="open failed, err: open: Failed to open/create local source file, path: volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta, err: open volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta: is a directory"
--
Fri, Jan 21 2022 2:44:01 pm | time="2022-01-21T22:44:01Z" level=info msg="Closing ssync server"
Fri, Jan 21 2022 2:44:01 pm | time="2022-01-21T22:44:01Z" level=info msg="Done running ssync server for file volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta at port 10005"
Fri, Jan 21 2022 2:44:01 pm | [volume-test-1-r-fe4359e5] time="2022-01-21T22:44:01Z" level=error msg="Sync agent gRPC server failed to rebuild replica/sync files: replica tcp://<IP_ADDRESS>:10000 failed to send file volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta to <IP_ADDRESS>:10005: failed to send file volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta to <IP_ADDRESS>:10005: rpc error: code = Unknown desc = failed to SyncContent for source: volume-snap-04dda9b4-3fa6-41b8-af98-55727b67bbde.img.meta: openServer failed, err: resp.StatusCode(500) != http.StatusOK"
Fri, Jan 21 2022 2:44:01 pm | ERROR: 2022/01/21 22:44:01 grpc: server failed to encode response: rpc error: code = Internal desc = grpc: error while marshaling: proto: Marshal called with nil
Fri, Jan 21 2022 2:44:01 pm | [volume-test-1-r-fe4359e5] time="2022-01-21T22:44:01Z" level=info msg="Lost connection from: <IP_ADDRESS>:43524"
Fri, Jan 21 2022 2:44:01 pm | time="2022-01-21T22:44:01Z" level=info msg="Closing volume"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: prepare to delete process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: deleted process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: wait for process volume-test-1-r-fe4359e5 to shutdown before unregistering process"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: trying to stop process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=info msg="wait for process volume-test-1-r-fe4359e5 to shutdown"
Fri, Jan 21 2022 2:44:01 pm | [volume-test-1-r-fe4359e5] time="2022-01-21T22:44:01Z" level=warning msg="Received signal interrupt to shutdown"
Fri, Jan 21 2022 2:44:01 pm | time="2022-01-21T22:44:01Z" level=warning msg="Starting to execute registered shutdown func github.com/longhorn/longhorn-engine/app/cmd.startReplica.func4"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=info msg="Process Manager: process volume-test-1-r-fe4359e5 stopped"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: prepare to delete process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: deleted process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=info msg="Process Manager: successfully unregistered process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=debug msg="Process Manager: prepare to delete process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:01 pm | [longhorn-instance-manager] time="2022-01-21T22:44:01Z" level=info msg="Process Manager: successfully unregistered process volume-test-1-r-fe4359e5"
Fri, Jan 21 2022 2:44:02 pm | [longhorn-instance-manager] time="2022-01-21T22:44:02Z" level=info msg="Process Manager: prepare to create process volume-test-1-r-4136924e"


scrcpy works on wireless adb too ... some output felt i'd share ...

@rom1v ... really like that your software works even on wireless adb ...
just some error i thought i'd share just in case it would help improve ...
$ scrcpy -b1M -m600
INFO: scrcpy 1.10 <https://github.com/Genymobile/scrcpy>
/usr/local/share/scrcpy/scrcpy-server.jar: 1 file pushed. 0.5 MB/s (22546 bytes in 0.039s)
adb: error: more than one device/emulator
ERROR: "adb reverse" returned with value 1
WARN: 'adb reverse' failed, fallback to 'adb forward'
27183
INFO: Initial texture: 336x600

See #381


Invalid Link to *.pgm files

The link to the zip file containing the *.pgm files is outdated.
To allow replication of the results, please update the link.
https://github.com/boschresearch/mrp_bench/blob/c39c865aace1275f870ed9b9cfa78ad531269502/docker/Dockerfile#L96C102-L96C148

Hi, I'll probably use git lfs for this soon. However, I can't test the fixed script on the machine I'm using right now, so until this is solved properly, here's the missing file as an attachment:
maps.zip

Thank you for the quick reply and the provided maps!


Vertical-Align for a cell

Hello !
I have a question , i was trying to get the elements vertically aligned in the center of each cell .
I'm trying to accomplish this by using  the  style: {'verticalAlign':'middle'} property on the columns definition, but i'm afraid the values are not getting centered as i need.
Is there any special way to perform this align?
Thanks!

I think this question is better asked on the #react-table Slack channel (there is a badge at the top of the readme). It is going to be related to the fact that react-table uses flexbox to layout all the cells (so you may have to provide some additional wrapping around your cell content to get verticalAlign to work).  It will be better answered by a wider group of people on the #react-table channel. Thanks.

Ok! thanks!

@hbulgarini did you get an answer on slack? I'm curious what the solution here was too

ok, I found a solution.  use this CSS to override the default styles
div.rt-tr {
  align-items: center;
}

for td can use
div.rt-td {   align-self: center; }

Add style: {'display': 'flex', 'flexDirection': 'column', 'justifyContent': 'center'} for each column

for me, it works
.ReactTable .rt-td {text-align: center; }


feat: jaas integration tests

Description
This PR adds an integration test workflow to test the provider against a JIMM controller.
The full list of changes include:

Add workflow to start JAAS and run tests.
Add logic to skip tests that are not appropriate for JAAS.
Fix offer tests that used a fixed "admin" user in assertions.

Fixes: CSS-6347
Type of change

Change in tests (one or several tests have been changed)

Additional notes
The Github action at canonical/jimm/.github/action/test-server is ready but because the canonical/jimm repo is not public, it is not accessible. The repo will become public very early next week. Marking this PR as a draft until then.

@hmlanigan This PR and #552 are ready for review, the only blocker on the JAAS integration tests is that we are waiting on the JIMM repo to be made public which should be done on Monday.

@hmlanigan tests are now passing. Two notes,

The issue pulling from the Github container registry was solved by using the GITHUB_TOKEN and adding the packages: read permission AND in the JIMM repo, the image was set to "internal", that has now been changed to "public".
I noticed once the test started working, in the first run, all tests passed except for one with an error "model not found", seems something went wrong in a flakey way. Tough to say if it was Juju or JAAS that caused the issue but I'll keep an eye on these scenarios.

/merge

/merge


Audio Issues

Video seems to work fine, but I get audio crackling most of the time. Would you be able to add the option of the end user changing the audio bit rate, mine needs to be higher bit rate on steam deck for some reason, other will prob be able to use whatever you have currently set, video is spot on.
Not related but I had the same issue when trying to stream the ps5 as well (audio wise).
Otherwise Amazing Work :)

what's your wifi like? I did too until I switched to ethernet

Xbox is connected to the router via ethernet, but steam deck obviously using wifi. i set the reset sound to back l4 button so over minute i press it to fix it, hopefully can be fixed in the future.

yeah..i noticed in wifi on my laptop it desynced after a a while..lag was bad.


Fix text in README.md

What happened?

Pull Request -> Create a pull request
compare and pull requests -> compare and pull request
How can we reproduce this bug?
Open README.md and scroll down to Step 7
I would like to work on this as a part of GSSoC 23.

hey...I want to solve this issue...
please assign me this task @metabiswadeep

Sure @metabiswadeep


Added constructors for DataLakePathScheduleDeletionOptions

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Storage.Files.DataLake


Standardized command verb

Proposing:
Webhook

add (sounds a little funny, original is create)
remove

this is in line with particle device

particle device add
particle device remove

I'm happy with add/remove, but would also love to support 'del' as well.

Sound like a one line addition: https://github.com/spark/particle-cli/blob/master/commands/WebhookCommands.js#L94


sq.pl.spatial_scatter produces blank plot

Description
sq.pl.spatial_scatter is giving me a blank plot for my MERFISH data although it has the leiden cluster colours on the side as a legend. It has worked for previous MERFISH data. It does not throw an error and traceback. I just wanted to know what can cause this.
Minimal reproducible example

    adata,
    color="leiden",
    library_id = 'library',
    shape=None,
    size=10,
    img=False,
    dpi=100,
    figsize=[8,8],

)
...

Version
Squidpy version: 1.2.0
I need to use this version as I'm using other packages which require an older version.
...

I'm not sure if this is your case, but I had the same issue and it turned out that the barcode identifiers in the *h5 file that I used to create the adata, and the spatial/tissue_positions.csv were different. The barcodes in the adata had a sample prefix which was missing in the tissue_positions.csv. Maybe check that.

Still have not fixed it. Odd behaviour. This is how I read it in:
adata = sc.read('061023_annotated_target_data.h5ad')
I'm not sure where the problem is since I also don't get an error message and traceback.

yeah that unfortunately doesn't say much cause it depends on how the anndata was built, without access to the original data it's difficult to give any suggestion. WIll close this for now, as it seems a data related issue and maybe it's better to be asked in https://discourse.scverse.org/


Differences between amp-by-example and /amphtml/examples/

Should the examples directory in the main amphtml repo be migrated here, or linked from here? Any reason they're split across two repos?
I was looking into <amp-form> and noticed that the documentation points to their own "full example" several times.

The examples in the main repo are written by the core eng team and serve as the initial rough verification that a feature works. The examples here are meant to show a wide range of developers how to take advantage of AMP for more complex scenarios that often combine multiple AMP components.
I prefer the solution proposed in #394 where we link to the official docs and/or the core eng team's examples in these examples.


order by rand()?

how order by rand?

Can you provide an example how to use this? Also, is there a way of using "RANDOM()" in case of Postgres database?

Can you provide an example how to use this? Also, is there a way of using "RANDOM()" in case of Postgres database?

err = unusedWordsQuery.
     Order(field.Func.Rand()).
     Limit(num).
     Scan(&wordIds)

maybe this method can help
 	err = query.Question.WithContext(c.Request().Context()).
			UnderlyingDB().
			Clauses(clause.OrderBy{Expression: clause.Expr{SQL: "RANDOM()"}}).
			Where(q.KnowledgeBaseID.Eq(req.KbID)).Limit(10).Find(&questions).Error


Link takes me to MSDN

In the first section there is a link: How to: Generate XML from CSV Files, which takes me to MSDN, and after I choose C#, back to docs: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/concepts/linq/how-to-generate-xml-from-csv-files.
The issue is similar to: https://github.com/dotnet/docs/issues/6955

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

ID: b2730d0d-f5e5-9948-5566-008fb42b8a2d
Version Independent ID: a9b9f250-0107-cf1b-c182-469c7fbe5fb5
Content: LINQ and Strings (C#)
Content Source: docs/csharp/programming-guide/concepts/linq/linq-and-strings.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

We have a whole issue to review the MSDN links (#2037). You can see we're making progress towards that.
But for this one in particular, I ended up already fixing that when I was addressing the other issue you reported:
https://github.com/dotnet/docs/pull/7374

@mairaw Thanks for letting me know. I assume that I don't need to report MSDN routing related issues separately.


Add PRs to WorkflowRunPayload fixes #140

I am not certain if this completely captures all the fields, but it captures all the fields GitHub is sending me. Tests updated and passing.

Coverage remained the same at 88.208% when pulling 28a310346d680651310367f2821c950a97f246f6 on craftyc0der:add-pull-requests-to-workflow-run into 9c954e23a1b901399c0f9d71645f84d3287641ca on go-playground:master.


Add Dockerfile

This PR does a few things, namely:

Adds a Dockerfile based on openresty/openresty:trusty, which makes it easier to get lua-resty-openidc up and running with all dependencies.
Removes the need to have a hard coded version number in the Lua .rockspec file; the version information is retrieved from git describe instead.
When a tag is being built, the packed .rock file will be uploaded to the repository's release for the given tag.
With the release tracking in place, uploading of the .rock to Lua Rocks has been moved to after_deploy, which means it won't be attempted for all builds; only for tagged ones.

A note to point 3 above: This requires a personal access token to be created and added in the Travis settings for the repository with the environment variable name GITHUB_OAUTH_TOKEN.
What's blatantly missing from this PR are tests to exercise the Docker container's NGINX server with OAuth and OpenID Connect test scenarios. I've added #14 to track this.
I'd also like to add uploading the Docker image to Docker Hub with docker push, but that requires quite a bit of setup on Docker Hub first. I've added #15 to track this.

I think this deserves to be in a separate repository on its own title, not even a fork of lua-resty-openidc. IMHO this repo should be core dev, any tooling around it could go elsewhere. That makes it easier to maintain as well since focus would be specific. Let me know what you think.

@zandbelt That's a good idea, but it would make continuous integration more difficult, since we then need to (or at least should) trigger a build of the Docker image repository every time we change something in lua-resty-openidc, to ensure that the Docker image doesn't break unintentionally. I'm not sure how to do that in Travis, but can take a look if you still think that's the best way forward.


fp8 marlin kernel

Adopted from: https://github.com/vllm-project/vllm/commit/47f0954af0a5aefd0db19875f6bdcbe933d055a9
Pinging @danieldk for interest since he added the marlin kernels

So this would enable FP8 support pre-H100 when using quantize="fp8" when wired up?

Yes, exactly

@danieldk could you please take a look for the packing ?
I am not sure how to repack the normal weights into marlin format with your functions.
Please feel free to overtake this PR if it's a fast shot :)

@danieldk could you please take a look for the packing ? I am not sure how to repack the normal weights into marlin format with your functions.

I was hoping/thinking that this could just hook into
https://github.com/huggingface/text-generation-inference/blob/5c7c9f13903f09636aaf99210710bf07002cdb87/server/text_generation_server/layers/fp8.py#L21
for compute capabilities that don't support FP8 directly, but that are supported by this Marlin kernel? I don't think it needs any special packing?

Ok, so I think we need to:

Convert to int32, though we may want to check what torch's representation is and whether it's possible to cast instead.
Run gptq_marlin_repack (with empty perm/g_idx).
Permute scales (see gptq repack).
The result the repack can be used as B in the marlin FP8 gemm.

Just pushed changes, do you mean that style ? @danieldk

Just pushed changes, do you mean that style ? @danieldk

Yeah, that looks in the direction that I was thinking. The scales need to be repeated in the output features dimension before permuting.
Let me know if you want to continue working on this or whether I should pick it up. I can probably do it this week between other things.

It would be great if you could take this, since I got some emergencies over night it have to prioritize, so it would be faster if you would finish it I think.

It would be great if you could take this, since I got some emergencies over night it have to prioritize, so it would be faster if you would finish it I think.

Ok, I'll work on it. Best of luck with the emergencies!

Closing this in favor of #2213 (to run CI).


Example doesn't make sense

Was trying to see connecting to Azure AD using automation account. But the example referencing a vm action not anything with AD. Can this be corrected?

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

ID: 747b9d5c-5785-2c35-11b1-7d4ea65cfa71
Version Independent ID: 087270c1-9c2f-80d0-38c0-1fa5926d9683
Content: Use Azure AD in Azure Automation to authenticate to Azure
Content Source: articles/automation/automation-use-azure-ad.md
Service: automation
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@cconrado Thank you for your feedback, our team will look into it and get back to you at the earliest.

Hello @cconrado - to clarify, the intent of this doc is to provide guidance on how to use Azure AD as the authentication provider within Azure Automation Runbook  in order to authenticate to an Azure resource - An Azure VM is used in the example to illustrate an Azure resource.
In the example, his runbook first uses Get-AutomationPSCredential to retrieve the credential to use to authenticate to Azure. It then calls the Connect-AzAccount cmdlet to connect to Azure using the credential.

Please share your specific use-case that requires connecting to Azure AD using automation account and we can try to provide appropriate references.
@MGoedtel  - FYI only.


Wsl 2 docker localhost bindings

Windows build number: Microsoft Windows [Version 10.0.19018.1]

I have docker installed in WSL Ubuntu-18.04 and Mysql container in it.
Mysql docker ports look like this <IP_ADDRESS>:3306->3306/tcp, 33060/tcp
Netstat inside WSL looks like:
tcp6       0      0 :::3306                 :::*                    LISTEN      23443/docker-proxy
Netstat on Windows host looks like:
  TCP    [::1]:3306             [::]:0                 LISTENING       12532

What's wrong / what should be happening instead:
Connecting to mysql using localhost:3306 on Windows Machine works as expected, but doesn't work using <IP_ADDRESS>:3306
Having similar issues with a couple of other containers as well.
Is there any way of fixing this issue?

I had a same problem too.
see the netstat on windows host

TCP [::1]:3306 [::]:0 LISTENING 12532

I think you should use ::1
not <IP_ADDRESS> 😀

I had the same problem. I run nginx and redis by docker on wsl2.
I can use <IP_ADDRESS>:80 to access to nginx .
but I can not use <IP_ADDRESS>@6379 to access to redis by redis client,
only can use ::1@6379 to access.

I had a same problem too.
see the netstat on windows host

TCP [::1]:3306 [::]:0 LISTENING 12532

I think you should use ::1
not <IP_ADDRESS> 😀

you can use localhost instead ::1. I use localhost work fine.


Confusion regarding the "sstype" variable in propagation module

There is a feature for defining the "sstype" when propagating uncertainty, but the value of that variable is closely tied to the type of sum-of-squares function used to compute the likelihood.  Adding additional documentation/tutorials would be beneficial.

Documentation has been added to the "propagation.py" module, which will be included in the v1.9.2 release.


Node MaxListenersExceededWarning: Possible EventEmitter memory leak detected

What issue do you have? Please be as thorough and explicit as possible.
Upon initializing the plugin, I get this error. Everything works as it should; however, I'm obviously concerned about the possible memory leak.
(node:2024) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 devices loaded listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit (Use node --trace-warnings ... to show where the warning was created) (node:2024) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
Details of your setup.
Homebridge–UIX

Which version of Homebridge/HOOBS do you have?

HomeBridge 1.6.1

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.

macOSX | Node.js [v18.16.1]

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?

HomeBridge-govee 9.1.0

Which Govee devices do you have that are causing issues? Please include product models if applicable.

All devices work as they should. This error displays upon the plugin's startup.
Please paste any relevant logs below.
(node:2024) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 devices loaded listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit (Use node --trace-warnings ... to show where the warning was created) (node:2024) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit

hi @mrlightison
thanks for this, although i believe this is simply due to the amount of govee devices you have -
out of interest how many do you have?


ASP.NET Core 2.1 high-level planning for docs - master

Last Updated: 10/29/2018
Copy from aspnet/Announcements#288 - order rearranged by priority.
ASP.NET Core 2.1 roadmap blog
2.1 Project Page
The following high-level features are planned for the ASP.NET Core 2.1 release:

[x] .NET Core meta doc, add ASP.NET Core features
[x] SignalR #5389 @rachelappel - Real-time web framework on ASP.NET Core (aspnet/SignalR#394)
[x] HTTPS #5393, @rick-anderson - On by default and easy to configure (aspnet/Home#2308)
[x] Add HttpsRedirection and HSTS middleware documentation #4568 @Rick-Anderson
[ ] HTTPS  #3310 @scottaddie - How to create cert xplat, config Kestresl to use that cert in prod/dev. etc
[x] IHttpClientFactory #5402- @scottaddie HttpClient as a service, handle cross-cutting concerns like caching, retry
[x] UI as a library #5409 - @Rick-Anderson -  Package Razor pages and views as reusable libraries (aspnet/Razor#1809) Check stability before starting.
[x] Identity UI package and scaffolder #5410 -@Rick-Anderson - Add identity to any application (aspnet/Home#2311)
[x] GDPR compliance cookies #5400 - Templates updated with new privacy features (aspnet/Security#1561, aspnet/Identity#1341) @rick-anderson
[x] GDPR delete custom Identity user data. #6226 @rick-anderson
[x] Web API conventions #5401 - @scottaddie - Rich Swagger support without attributes (aspnet/Mvc#6870, aspnet/Mvc#6784)
logic, timeouts and circuit breakers (aspnet/HttpClientFactory#42)
[x] Build-time Razor #5408 - @scottaddie -Compile Razor pages and views as part of your build, improved startup performance (aspnet/Razor#1809)
[x] Razor pages

[x] Support for areas #5407 - @Rick-Anderson

[x]  ~/Pages/Shared (aspnet/Mvc#6926, aspnet/Mvc#6604) also described by @DamianEdwards on the Standup :point_right: https://youtu.be/-EgE79M3tuc?t=36m20s See this

[x]  CUT from V2.1 Bind all props on a page or controller :point_right: https://youtu.be/-EgE79M3tuc?t=41m10s

[x] Decorate simple properties as [Required] #5834 - @Rick-Anderson (will be in preview2) :point_right: https://youtu.be/-EgE79M3tuc?t=45m25s

---------------- Issues not in aspnet/Announcements#288 -------
@guardrex

[x] Update Kestrel host configuration guidance to include new HostFiltering middleware
[x] Update note regarding using ASP.NET Core Previews on Azure App Service #5568
[x] Sample Update Tracking Issue #5495 (The initial assignment is complete; work continues on the issue.)
[x] 2.1 Update config for 2.x Azure KV config provider topic samples #4211
[x] Document improved support for integration testing #4652
[x] Hosting non-server apps with Generic Host  #5798
[x] Update doc for IHostedService #3352
[x] New kestrel configuration & related APIs #5056
[x] Add HTTPS setup instructions for IIS Dev scenario #5899
[x] Add AllowedHosts to proxy config topic #5793
[x] Update libuv references in Kestrel docs #5695
[x] RP conventions features update #5520
[x] Add KeyVault encryption to DataProtection #5439
[x] Change ANCM requestTimeout property ... #5264
[x] MVC: Document ViewData-backed properties #6123
[x] Document how Razor Pages handles HEAD requests #6124
[x] Document updates to launchSettings.json #6127
[x] Document Kestrel support for SNI #6129
[x] Enhance DP cert protect/unprotect keys coverage #7358
[x] Razor Pages Tutorial - Seed Data as Migration #7342
[x] Update 'dev-certs' tools link in Kestrel server topic #6097
[ ] Update BuildWebHost to CreateWebHostBuilder (MD topics only) #6091 :bell: IN REVIEW
[ ] Sample Update Tracking Issue #5495 🏃 IN PROGRESS
[x] Replace M.A.All meta-package with Microsoft.AspNetCore.App #5314
.....
[x] Add Microsoft.AspNetCore.App to dotnet docs repo #5937 @rick-anderson
[x]  Partial tag helper analyzer warnings #5938 @rick-anderson
[x]  Document SetCompatibilityVersion #5939 @rick-anderson
[x] Document IPageFilter and filter methods on PageModel  #5754-- contact @pranavkm Related remarks in the Standup :point_right: :point_right: https://youtu.be/-EgE79M3tuc?t=43m16s    * Code that runs before or after every handler (OnPageHandlerExecuting/OnPageHandlerExecuted) :point_right: https://youtu.be/-EgE79M3tuc?t=43m16s
[x] Document Microsoft.AspNetCore.app #5913 @rick-anderson
[x] Partial Tag Helper #5420 -- https://github.com/aspnet/Mvc/issues/5916
[ ] Update the filters documentation to include info about the IAlwaysRunFilter and IAsyncAlwaysRunFilter filters #5419 - @Rick-Anderson
[x] Update security/authentication/identity-enable-qrcodes for 2.1 changes #5487
[ ] Doc new IOptionsBuilder and related overloads #5924

@Rick-Anderson we're missing docs for IPageFilter and filter methods on PageModel. Would this be an appropriate issue to track these docs?

@pranavkm Thanks! Added to the list and put you down as the contact. Do we have any information to get us jump started?

I wrote some in the same vein as IActionFilter: https://github.com/aspnet/Home/issues/2981#issuecomment-374706661

Closing as this doesn't have much useful info.


[Bug-FE][UI Next][V1.0.0-Alpha]SUB_PROCESS component execution completion display problem

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
SUB_PROCESS component execution completion display problem

What you expected to happen
SUB_PROCESS component will not have host execution host, should be straight line logo, will not generate logs should hide the log display button, or enter the log details page prompt this component will not generate logs
How to reproduce
SUB_PROCESS component execution
Anything else
No response
Version
3.0.0-alpha
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I will fix it.


| WARNING  | nndet.core.boxes.nms::26 - nnDetection was not build with GPU support!

:question: Question

Validation sanity check:   0%|                                                                                | 0/10 [00:00<?, ?it/s]using pin_memory on device 0
Traceback (most recent call last):
File "train.py", line 497, in
train()
File "/var/data/wly/nnDetection-main/scripts/nndet/utils/check.py", line 62, in wrapper
return func(*args, **kwargs)
File "train.py", line 70, in train
_train(
File "train.py", line 290, in _train
trainer.fit(module, datamodule=datamodule)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 553, in fit
self._run(model)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 918, in _run
self._dispatch()
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 986, in _dispatch
self.accelerator.start_training(self)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/accelerators/accelerator.py", line 92, in start_training
self.training_type_plugin.start_training(trainer)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/training_type_plugin.py", line 161, in start_training
self._results = trainer.run_stage()
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 996, in run_stage
return self._run_train()
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1031, in _run_train
self._run_sanity_check(self.lightning_module)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1115, in _run_sanity_check
self._evaluation_loop.run()
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 111, in run
self.advance(*args, **kwargs)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/loops/dataloader/evaluation_loop.py", line 110, in advance
dl_outputs = self.epoch_loop.run(
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 111, in run
self.advance(*args, **kwargs)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/evaluation_epoch_loop.py", line 110, in advance
output = self.evaluation_step(batch, batch_idx, dataloader_idx)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/evaluation_epoch_loop.py", line 154, in evaluation_step
output = self.trainer.accelerator.validation_step(step_kwargs)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/accelerators/accelerator.py", line 211, in validation_step
return self.training_type_plugin.validation_step(*step_kwargs.values())
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/training_type_plugin.py", line 178, in validation_step
return self.model.validation_step(*args, **kwargs)
File "/var/data/wly/nnDetection-main/scripts/nndet/ptmodule/retinaunet/base.py", line 172, in validation_step
losses, prediction = self.model.train_step(
File "/var/data/wly/nnDetection-main/scripts/nndet/core/retina.py", line 146, in train_step
prediction = self.postprocess_for_inference(
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/var/data/wly/nnDetection-main/scripts/nndet/core/retina.py", line 187, in postprocess_for_inference
boxes, probs, labels = self.postprocess_detections(
File "/var/data/wly/nnDetection-main/scripts/nndet/core/retina.py", line 326, in postprocess_detections
boxes, probs, labels = self.postprocess_detections_single_image(boxes, probs, image_shape)
File "/var/data/wly/nnDetection-main/scripts/nndet/core/retina.py", line 375, in postprocess_detections_single_image
keep = box_utils.batched_nms(boxes, probs, labels, self.nms_thresh)
File "/var/data/wly/nnDetection-main/scripts/nndet/core/boxes/nms.py", line 106, in batched_nms
return nms(boxes_for_nms, scores, iou_threshold)
File "/home/wly/miniconda3/envs/lung_medicle/lib/python3.8/site-packages/torch/amp/autocast_mode.py", line 14, in decorate_autocast
return func(*args, **kwargs)
File "/var/data/wly/nnDetection-main/scripts/nndet/core/boxes/nms.py", line 78, in nms
return nms_fn(boxes.float(), scores.float(), iou_threshold)
TypeError: 'NoneType' object is not callable
The error during training indicates that the GPU is not functioning properly, despite having four GPUs available

Something went wrong during the installation, please check the FAQ and other Issues for guidance.

Something went wrong during the installation, please check the FAQ and other Issues for guidance.

Thank you, but during the final step of the model inference process, it shows:
INFO Running preprocessing of test cases INFO Found 0 cases for preprocessing in /var/data/wly/nnDetection-main/projects/Task016_Luna/raw_splitted and 0 already processed cases. INFO Running inference INFO Found 2 models to ensemble INFO Architecture overwrites: {} Anchor overwrites: {} INFO Building architecture according to the plan of RetinaUNetV001 INFO Start channels: 32; head channels: 128; fpn channels: 128 INFO Discarding anchor generator kwargs {'stride': 1} INFO Building:: encoder Encoder: {} INFO Building:: decoder UFPNModular: {'min_out_channels': 8, 'upsampling_mode': 'transpose', 'num_lateral': 1, 'norm_lateral': False, 'activation_lateral': False, 'num_out': 1, 'norm_out': False, 'activation_out': False} INFO Running ATSS Matching with num_candidates=4 and center_in_gt False. INFO Building:: classifier BCECLassifier: {'num_convs': 1, 'norm_channels_per_group': 16, 'norm_affine': True, 'reduction': 'mean', 'loss_weight': 1.0, 'prior_prob': 0.01} INFO Init classifier weights: prior prob 0.01 INFO Building:: regressor GIoURegressor: {'num_convs': 1, 'norm_channels_per_group': 16, 'norm_affine': True, 'reduction': 'sum', 'loss_weight': 1.0, 'learn_scale': True} INFO Learning level-specific scalar in regressor INFO Overwriting regressor conv weight init INFO Building:: head DetectionHeadHNMNative: {} sampler HardNegativeSamplerBatched: {'batch_size_per_image': 32, 'positive_fraction': 0.33, 'pool_size': 20, 'min_neg': 1} INFO Sampling hard negatives on a per-batch basis INFO Building:: segmenter DiCESegmenterFgBg {'dice_kwargs': {'batch_dice': True}} INFO Running batch dice True and do bg False in dice loss. INFO Model Inference Summary: detections_per_img: 100 score_thresh: 0 topk_candidates: 10000 remove_small_boxes: 0.01 nms_thresh: 0.6 INFO Loaded /var/data/wly/nnDetection-main/projects/Task016_Luna/models/Task016_Luna/RetinaUNetV001_D3V001_3d/fold0/model_last.ckpt with  INFO Architecture overwrites: {} Anchor overwrites: {} INFO Building architecture according to the plan of RetinaUNetV001 INFO Start channels: 32; head channels: 128; fpn channels: 128 INFO Discarding anchor generator kwargs {'stride': 1} INFO Building:: encoder Encoder: {} INFO Building:: decoder UFPNModular: {'min_out_channels': 8, 'upsampling_mode': 'transpose', 'num_lateral': 1, 'norm_lateral': False, 'activation_lateral': False, 'num_out': 1, 'norm_out': False, 'activation_out': False} INFO Running ATSS Matching with num_candidates=4 and center_in_gt False. INFO Building:: classifier BCECLassifier: {'num_convs': 1, 'norm_channels_per_group': 16, 'norm_affine': True, 'reduction': 'mean', 'loss_weight': 1.0, 'prior_prob': 0.01} INFO Init classifier weights: prior prob 0.01 INFO Building:: regressor GIoURegressor: {'num_convs': 1, 'norm_channels_per_group': 16, 'norm_affine': True, 'reduction': 'sum', 'loss_weight': 1.0, 'learn_scale': True} INFO Learning level-specific scalar in regressor INFO Overwriting regressor conv weight init INFO Building:: head DetectionHeadHNMNative: {} sampler HardNegativeSamplerBatched: {'batch_size_per_image': 32, 'positive_fraction': 0.33, 'pool_size': 20, 'min_neg': 1} INFO Sampling hard negatives on a per-batch basis INFO Building:: segmenter DiCESegmenterFgBg {'dice_kwargs': {'batch_dice': True, 'smooth_nom': 1e-05, 'smooth_denom': 1e-05, 'do_bg': False}} INFO Running batch dice True and do bg False in dice loss. INFO Model Inference Summary: detections_per_img: 100 score_thresh: 0 topk_candidates: 10000 remove_small_boxes: 0.01 nms_thresh: 0.6 INFO Loaded /var/data/wly/nnDetection-main/projects/Task016_Luna/models/Task016_Luna/RetinaUNetV001_D3V001_3d/fold0/model_best.ckpt with  INFO Found inference plan: {'model_iou': 1e-05, 'model_nms_fn': <function batched_weighted_nms_model at 0x7f795d071700>, 'model_score_thresh': 0.0, 'model_topk': 1000, 'model_detections_per_image': 100, 'ensemble_iou': 0.2, 'ensemble_nms_fn': <function batched_wbc_ensemble at 0x7f795d071940>, 'ensemble_topk': 1000, 'remove_small_boxes': 2.0, 'ensemble_score_thresh': 0.0} for prediction INFO Adding 2D Mirror TTA for prediction. INFO Adding 3D Mirror TTA for prediction. INFO Using 8 TTA transformations for prediction (one dummy trafo). INFO Found 0 files for inference.
I placed my custom test set in 'raw_splitted/imagesTs', but the prediction tells me that 0 data files were found. How should I address this issue?

The path where nnDetection looks for the data is mentioned in the first line of your posted message, pleas make sure that the imagesTs folder exists there. The files need to follow the same structure as the imagesTr i.e. end with '_0000' and needs to be nifties (ending with '.nii.gz').
Please double check that all paths and names are correct.

The path where nnDetection looks for the data is mentioned in the first line of your posted message, pleas make sure that the imagesTs folder exists there. The files need to follow the same structure as the imagesTr i.e. end with '_0000' and needs to be nifties (ending with '.nii.gz'). Please double check that all paths and names are correct.
thanks so much，but I'm trying to reproduce the results of luna16 using NNdetection, but I found that the predicted results only contain the coordinates of the boxes, without diameters. Also, the official prepare_eval_cpm.py script provided in Task016 for converting results to luna16 format files only includes the coordinates of the centers, without diameters. I want to know how to solve this issue because without diameters, I can't use the evaluation program provided by luna16 for assessment.

LUNA16 only requires the prediction of a center point of the lesion and not the diameter. They check if the predicted point is inside the radius of the ground truth lesion.
You can see an example results file here where they also only use points: https://luna16.grand-challenge.org/Evaluation/

Note however: LUNA16 was run as a 10Fold Cross-Validation as was done in DeepLung; so to reproduce the results you don't need imagesTs which was mentioned in your previous message.


Converted FieldEntry to record

Task from #13207
Avoid the usage of compact constructors because the compact constructor can only access the components of record, and most of FieldEntry relies on  IndexInput object which is not a part of record class. Hence, The purposed code is using the static factory method to create the FieldEntry.
However, there is another way where we can make IndexInput a component of the record class and then access it in compact constructor. While it moves the code inside compact constructor, but the code accessing IndexInput has to be surrounded with try-catch as compact constructors cannot throw checked exception.

The change looks good to me, I'd like @uschindler to have a look since he knows more about records than I do.

They are not too complicated, you just have to think simpler and use the special syntax. Something to keep in mind when you add records:

The default ctor cannot be hidden, so when you add public records, make sure all consistency checks are done in the default ctor (the one in the source file having no parameters: public Classname {). This ensures that all checks are at one place. My recommendation is to override the default ctor. You can add additional ctors for easy usage for common special cases (see IOContext class for examples), calling the default one with this().
Be sure to remove equals, hashcode, toString: it does not warn you when you define your own

Thanks, will merge soon.


IMERG Precipitation Reader cut-off

Bug Description
The IMERG reader is failing due to a file truncation error. Output from LOG File:
[INFO] LIS cycle time: 02/01/2019 03:30:00
[WARN] Missing IMERG precipitation data::
/discover/nobackup/projects/lis/MET_FORCING/IMERG/Final_V06B//201902/3B-HHR.MS.
MRG.3IMERG.20190201-
Steps to Replicate
Relevant lis.config file settings:
IMERG forcing directory:  /discover/nobackup/projects/lis/MET_FORCING/IMERG/Final_V06B/
IMERG product:           'final'
IMERG version:           V06B

Hi Tim, could be you’ve hit the 120-character limit for the path & file name.
v/r
Jerry
Jerry Wegiel, Ph.D.
Principal Investigator, DoD-/IC- components of LIS: https://lis.gsfc.nasa.govhttps://lis.gsfc.nasa.gov/
NASA/GSFC Hydrological Sciences Laboratory: http://science.gsfc.nasa.gov/earth/hydrology/
SETA Research Scientist Senior Principal (w/ Gov’t NDAs)
101 Nelson Drive
Offutt AFB, NE 68113-1023
240-261-8457 (Mobile)
402-232-7690 (Office)
SIPRNET<EMAIL_ADDRESS><EMAIL_ADDRESS>https://www.linkedin.com/in/jerrywegielhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.linkedin.com%2Fin%2Fjerrywegiel&data=04|01|jerry.w.wegiel%40nasa.gov|fbc39fd973a34d74367f08d8ba4c46a0|7005d45845be48ae8140d43da96dd17b|0|0|637464184057167215|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0iwOcs9qqnofB%2Bu5%2FPSDC%2Fy91QzwvGA4IceUV45QyRM%3D&reserved=0
From: TimLahmers @.>
Reply-To: NASA-LIS/LISF @.>
Date: Monday, March 22, 2021 at 6:39 PM
To: NASA-LIS/LISF @.>
Cc: Subscribed @.>
Subject: [EXTERNAL] [NASA-LIS/LISF] IMERG Precipitation Reader cut-off (#749)
Bug Description
The IMERG reader is failing due to a file truncation error. Output from LOG File:
[INFO] LIS cycle time: 02/01/2019 03:30:00
[WARN] Missing IMERG precipitation data::
/discover/nobackup/projects/lis/MET_FORCING/IMERG/Final_V06B//201902/3B-HHR.MS.
MRG.3IMERG.20190201-
Steps to Replicate
Relevant lis.config file settings:
IMERG forcing directory: /discover/nobackup/projects/lis/MET_FORCING/IMERG/Final_V06B/
IMERG product: 'final'
IMERG version: V06B
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNASA-LIS%2FLISF%2Fissues%2F749&data=04|01|jerry.w.wegiel%40nasa.gov|b927e69fc2f14363b3fc08d8ed8bac59|7005d45845be48ae8140d43da96dd17b|0|0|637520531444883143|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=jrVEcoifvG31kDxvq1o84eEqWlu1sXlTfHIROVqWsIQ%3D&reserved=0, or unsubscribehttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJY534JPLPDQFO7MGDUBU73TE7IJBANCNFSM4ZUCRELA&data=04|01|jerry.w.wegiel%40nasa.gov|b927e69fc2f14363b3fc08d8ed8bac59|7005d45845be48ae8140d43da96dd17b|0|0|637520531444893097|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RE4tSRBApoZhSaLNmnl%2FfeL4wPAlQbyGyWtQxTKuvh4%3D&reserved=0.

This is related to #371 and #533 and will be resolved when #536 is merged.
Workaround for Discover: soft-link to the LIS project directory in your running directory and use that instead of the absolute path.


Allow for a version readout of each drivers native maven dependencies

A user would like to be able to do something like --list-driver-versions, which would introspect the maven artifact for each driver module's native driver, if there is one, and provide a report of the maven coordinates on the console.

We should look into SBOM generation.


Do I understand the layers of my model right?

Hi, I'm new to neural Keras, and I'm confused about the number of layers in my model.I want to have a model of three layers.
First layer: 4 neurons
Second layer:5 neurons
Third layer: 3 neurons
My code is as follows:
model = Sequential()
model.add(Dense(5, input_dim=4))
model.add(Activation('relu'))
model.add(Dense(3))
model.add(Activation('softmax'))
Am I right?
Actually, I added two dense layers in the model, but it seems that the model is a three layer neural network. I'm so confused. Can you help me?Thank you very much!

Right, In papers we have input layer(or many input layers) + many hidden layers + output layer(or many output layers). But in Keras, the input_dim or input_shape is likely to be the input layer.

@meanmee Thank you for your help!


New Language

Language
Catalan

Thanks for volunteering, I've added the language and you can now contribute here
https://crowdin.com/project/battery-notes

Done


When a type error suffices, this is thrown: error FS0002: This function takes too many arguments, or is used in a context where a function is not expected

When a function is used as an argument where a value type is expected, FS0002 is thrown. It seems to make more sense to use a type error in such case, as raising FS0002 is rather confusing.

Repro steps
Consider this, or any similar snippet where an argument of a non-function type is expected:
let x = Seq.init (fun x -> Result.Ok x)

Expected behavior
It should raise something similar to:

error FS0001: This expression was expected to have type 'int' but here has type ''a -> Result<'a>'

Or if FS0002 is by design, perhaps it can be improved if the compiler can detect the case where it is in fact a type error and give a more descriptive error detailing what would otherwise be shown with FS0001.
Actual behavior
Instead it raises:

error FS0002: This function takes too many arguments, or is used in a context where a function is not expected

This is confusing as it is besides the point (even though the last part says "used in a context where a function is not expected", this is hard to grok, even more so since many programmers are puzzled by the concept of 'context').
Known workarounds
N/A
Related information
Any current F# version, including latest.
As an aside, type inference in the screenshot shows that x is of type ((int -> obj) -> seq<obj>), I have no clue why it thinks that, it is clearly confused...

@abelbraaksma I agree the message is confusing and your proposed expected behaviour would make sense AFAICS.
Tracking down the error message, it stems from throwing FunctionExpected in UnifyFunctionType https://github.com/Microsoft/visualfsharp/blob/ad77930e44276dd28a34372145f61be8197150a9/src/fsharp/TypeChecker.fs#L736-L742
while the other (FS0001) is in OutputPhasedErrorR https://github.com/Microsoft/visualfsharp/blob/ad77930e44276dd28a34372145f61be8197150a9/src/fsharp/CompileOps.fs#L695-L701 where there seems to be couple of special cases (tuples and bool ref apparently).
The challenge on this one is to find the higher level place in the stacktrace where the branch occurs on the expression type, it doesn't seem as easy as some other error message adjustments I've encountered so far.
If you want to give it a look, you can debug FSI and use your sample code, having break point  in UnifyFunctionType and walking back the stack would be the first step to investigate further.
Maybe someone more familiar with that kind of error adjustment can complement those first elements.

Going to label this as a bug rather than an improvement, as this does not look like the correct error message to emit.


fix: wrong use of apivalication.ValidateImmutableField

What type of PR is this?
/kind bug
/kind flake
What this PR does / why we need it:
wrong use of apivalication.ValidateImmutableField, order of parameters is (newObj,oldObj,fieldPath) not  is (oldObj,newObj,fieldPath)
add two test cases:

immutable queue name not suspend
queue name can changes when it is  suspend

Does this PR introduce a user-facing change?
NONE

/assign mimowo tenzen-y

/assign @mimowo @tenzen-y

/ok-to-test

Thanks!. Also update it here

I think this actually have a user-facing impact, though minimal:
/release-note-edit
Fix the validation messages, to report the new value rather than old, for the following immutable labels: `kueue.x-k8s.io/queue-name`, `kueue.x-k8s.io/prebuilt-workload-name`, and `kueue.x-k8s.io/priority-class`.


bug: big b start of line

This addresses #50.
This took me quite a while to follow through. In short, it's hitting the while loop one additional time because of the utils.char_class falling through to the default value of other which is the same class as the initial match. So the resulting offset is one less than it should be, resulting in a column of zero instead of the correct 1.
The docs for vim.fn.strcharpart say it returns empty string on error, which is what is happening here when the index is outside the line. It made the most sense to add a guard for this edge case so I introduced an additional char_classes enum member.
At least, this is how it was working until the simultaneous change to use the FFI in #49. The tests still pass, things still look to work, my issue remains fixed, but this might want a slightly more detailed review due to the FFI changes.
Edit: I also added a missed test for the emoji char_classes member from #49, in commit bf4df53

Awesome, thanks! Sorry, I maybe should've waited to merge the FFI branch.

No worries, I didn't post the bug until after I had a solution. I should have posted it sooner and mentioned I was looking into it.
I did test the FFI changes on their own without the fix here, and they didn't solve this issue alone.

I'd have to look at the other code
But can we fix at the other side with a <= or something?
The char class function is basically just the cls function from Nvim/Vim and id like to keep it as close to that as possible
My preference would be to fix the prev_word implementation to exit early and add an asset here that the string isnt empty
If this is possible

I'll take a look, should be able to move the empty string check I think. I wasn't aware of the Vim cls stuff.

The above changes appear to work for the issue I logged, and all tests pass.
If I add assert(char ~= "") to the char_class function, a number of tests involving whitespace and padding fail. I'm not sure if this is because they were originally passing incorrectly and this change has revealed a source of issues, or if this assertion has legitimately broken things.
Currently this assertion has not been committed.

Upon initial investigation, it looks like there are a number of additional places that would need to handle the empty string case since it is no longer part of the char_class function; every call to this function would need to be guarded against empty string. It looks like there 13 or so calls to this function (outside of tests), which really adds a lot of extra logic everywhere.
I wonder if it's best to disregard adding an empty string assertion in char_class since things look to be working ok without it. It might be more effort to support it than is worthwhile.

I don't know if we need the guard and it might be that the offset ranges need to be modified to be no inclusive of 0

I think we need to make emojis return 3 for this to be complete,

as in this example the b is under the p not the cloud

this turned into a bigger refactor of how we class chars but this should have much better support for different langs and locales
ffi was short lived, as there is a vim function for it directly
@willothy any thoughts?
@josh-nz thanks for raising the right questions for this!


Is websocket working correctly?

i tried to add some debugs, but none are hitting. Is there a requirement that we are missing in the setup?
     from websocket import create_connection

        @self.sock.route("/audiostream", websocket=True)
        def on_media_stream(ws):
            try:
                logging.debug("WebSocket connection initiated.")

                # Log the initial details about the WebSocket.
                logging.debug("WebSocket object: %s", ws)

                # Instantiate a new TwilioCallSession with the WebSocket and other details.
                session = TwilioCallSession(
                    ws,
                    self.client,
                    remote_host=self.remote_host,
                    static_dir=self.static_dir,
                )

                logging.debug("WebSocket: %s, Client: %s, Remote Host: %s, Static Dir: %s",
                              ws, self.client, self.remote_host, self.static_dir)

                # If an on_session callback is defined, run it in a new thread.
                if self.on_session is not None:
                    logging.debug("Starting on_session callback in a new thread.")
                    thread = threading.Thread(target=self.on_session, args=(session,))
                    thread.start()

                # Start the session.
                session.start_session()

            except Exception as e:
                # Catch and log any exceptions that might occur during WebSocket handling.
                logging.error("Error during WebSocket handling: %s", str(e))

            finally:
                # Log when the function exits or the WebSocket connection is closed.
                logging.debug("WebSocket connection closed or function exited.")

I have also added

        def incoming_voice():
            print(XML_MEDIA_STREAM.format(host=self.remote_host))
            print(self.client,self.remote_host,self.static_dir)
            return XML_MEDIA_STREAM.format(host=self.remote_host)

And i see the XML is returning correctly.
<Response>
    <Start>
        <Stream name="Audio Stream" url="wss://https://SOME_LINK-93ff-fe7c-5cb7.ngrok-free.app/audiostream" />
    </Start>
    <Pause length="60"/>
</Response>

What can be the issue here?

Hey! Actually I'm not sure if that XML is correct, looks like it's wss https but I believe it should just be wss

Hey! Actually I'm not sure if that XML is correct, looks like it's wss https but I believe it should just be wss

Yeah, I realized that during the test, and removed the https, But it still did not get it to work, Also as for default. I replaced that link with my link.
I will show more logs soon.

Hey! Actually I'm not sure if that XML is correct, looks like it's wss https but I believe it should just be wss

Yeah, I realized that during the test, and removed the https, But it still did not get it to work, Also as for default. I replaced that link with my link.
I will show more logs soon.

I'm also facing a similar issue like this
Have you got any fixes?

Hm interesting, at some point I was able run as is but wouldn't be crazy if something broke. Currently travelling but I'll try to debug/clean up repo in the next week.

Added some example scripts to the README that seem to work fine for me.

<IP_ADDRESS> - - [2023-09-18 17:14:16] "POST /incoming-voice HTTP/1.1" 200 289 0.000662
INFO:pyngrok.process.ngrok:t=2023-09-18T17:14:16+0530 lvl=info msg="join connections" obj=join id=e38116dff61e l=<IP_ADDRESS>:8080 r=<IP_ADDRESS>:50054
INFO:pyngrok.process.ngrok:t=2023-09-18T17:14:18+0530 lvl=info msg="join connections" obj=join id=88bb1700f2cd l=<IP_ADDRESS>:8080 r=<IP_ADDRESS>:49818
INFO:root:Call connected, {'accountSid': '', 'streamSid': '', 'called': '', 'tracks': ['inbound'], 'mediaFormat': {'encoding': 'audio/x-mulaw', 'sampleRate': 8000, 'channels': 1}}
-> Hello! Welcome to the Machine Learning hotline, how can I help? ['Hello! Welcome to the Machine Learning hotline, how can I help?']
Exception in thread Thread-8 (run_chat):
Traceback (most recent call last):
File<EMAIL_ADDRESS>line 1038, in _bootstrap_inner
self.run()
File<EMAIL_ADDRESS>line 975, in run
self._target(*self._args, **self._kwargs)
File "/Users/saravanarajapp/development/ai/llm_convo/twilio_ngrok_ml_rhyme_hotline.py", line 39, in run_chat
run_conversation(agent_a, agent_b)
File "/Users/saravanarajapp/development/ai/llm_convo/llm_convo/conversation.py", line 10, in run_conversation
text_b = agent_b.get_response(transcript)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/saravanarajapp/development/ai/llm_convo/llm_convo/agents.py", line 64, in get_response
self._say(transcript[-1])
File "/Users/saravanarajapp/development/ai/llm_convo/llm_convo/agents.py", line 59, in _say
duration = self.speaker.get_duration(tts_fn)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/saravanarajapp/development/ai/llm_convo/llm_convo/audio_output.py", line 46, in get_duration
duration = float(output.split("=")[1].split("\r")[0])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: could not convert string to float: '4.944000\n[/FORMAT]\n'
INFO:root:Call media stream ended.

Thanks for the example scripts. I've tried that keyboard chat with GPT that works fine but the other examples with twilio integration throw me the above errors.

Hm whats your ffmpeg -version seems like its very close to working but its unable to do some audio conversions

my ffmpeg version is 6.0

I just reinstalled ffmpeg+ffprobe (realized it's ffprobe thats behind the issue):
ffprobe version 6.0-essentials_build-www.gyan.dev Copyright (c) 2007-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-bzlib --enable-lzma --enable-zlib --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-sdl2 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-libaom --enable-libopenjpeg --enable-libvpx --enable-mediafoundation --enable-libass --enable-libfreetype --enable-libfribidi --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-d3d11va --enable-dxva2 --enable-libvpl --enable-libgme --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libtheora --enable-libvo-amrwbenc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-librubberband
libavutil      58.  2.100 / 58.  2.100
libavcodec     60.  3.100 / 60.  3.100
libavformat    60.  3.100 / 60.  3.100
libavdevice    60.  1.100 / 60.  1.100
libavfilter     9.  3.100 /  9.  3.100
libswscale      7.  1.100 /  7.  1.100
libswresample   4. 10.100 /  4. 10.100
libpostproc    57.  1.100 / 57.  1.100

python examples\twilio_ngrok_ml_rhyme_hotline.py --preload_whisper --start_ngrok
And it seems to work 🤔

Ah think I got it, was windows vs linux issue with \r: https://github.com/sshh12/llm_convo/commit/ba19dad208583fae181a41f9aedb76781222db78
Try updating the repo and trying it.

For me this rewrite seems to fix the above error. Looks like an issue with ffprobe on some Linux environments.
def get_duration(self, audio_fn: str) -> float:
    try:
        popen = subprocess.Popen(
            ["ffprobe", "-i", audio_fn],
            stdout=subprocess.PIPE,
            stderr=subprocess.STDOUT  # Redirect stderr to stdout
        )
        output, _ = popen.communicate()
        output = output.decode("utf-8")

        # Extract the duration
        for line in output.split('\n'):
            if line.startswith('  Duration:'):
                duration_str = line.split('Duration: ')[1].split(',')[0].strip()
                hours, minutes, seconds = map(float, duration_str.split(':'))
                duration = hours * 3600 + minutes * 60 + seconds
                return duration

        print("Duration not found in ffprobe output.")
        return 0.0
    except Exception as e:
        print(f"Error in get_duration: {e}")
        return 0.0


UIWebView deprecated

I'm using the WKWebView component version of the library, but when I deploy the app to the Apple Store I get the following error: ITMS-90809: Deprecated API Usage - New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability. Learn more (https://developer.apple.com/documentation/uikit/uiwebview).
How can I remove all UIWebView references from the library?
Thank you

There are version available if you search without UIWebView.
But using these , I can't connect as I get negotiation issue. Not sure what needs to change in the server side.
Any ideas?

I managed to get it working by changing the library code by removing all references to the UIWebView object

I managed to get it working by changing the library code by removing all references to the UIWebView object
Hi @NeedNap I am going through similar problem, and i can't seem to remove references to the UIWebView.
Can you please share your code?
Thank you

This is my fixed code
SwiftR.zip
;)

Thanks A lot.
For some reason the code was not working, maybe commenting UIWebView messes up with some features.
I had to convert UIWebView to WKWebview everywhere.

It's very strange: the code I posted works very well for me but I don't use the useWKWebView flag to true because my SignalR is not the .NET Core version.

config.setValue(true, forKey: "allowUniversalAccessFromFileURLs")

has anyone else had trouble with WKWebView not connecting properly/timing out after 40 seconds or so? UIWebView works great, WKWebView seems to have connection issues

I have discovered that using @NeedNap solution (downloaded the .zip you shared, thanks a million btw) works. I have a .Net self-hosted server running in windows 10 coded in VB.net. At first, the connection wasn't being established, I thought it was the client at the fault - which is an iOS app using swift and UIKit, and after hours of researching turns out if your server is on your local network then use the local IP address of the machine running the server instead of the global IP. Also, make sure all your ports are accessible. My issue was the device being on the same wifi as the server had to communicate on a local IP, and when I turned off the wifi on the test device then I had to switch back to the global IP. I hope this helps ✊🏾


The information on the resource details page does not update after upgrade and rollback

Describe the bug

After upgrading or rolling back resources, the details page information doesn't update automatically.
Steps to reproduce
Issue 1: The latest run status doesn't update after update the resource.
https://github.com/seal-io/walrus/assets/26084756/bed6c766-3109-47ee-9ca0-d75fdd2e9edc
Issue 2: The resource configuration doesn't update after rollback.
https://github.com/seal-io/walrus/assets/26084756/40f46b28-07dc-4918-a7c3-599c26ec68de
Expected behavior

Environment

Walrus version: main 7bb79389ab0436969f4c4214e38918ee6c6e9070
UI version: dev-7885278

Additional context
Add any other context about the problem here.

Environment

Walrus version: main aba5070fb2d7f86be3c14bd419ee7be9a8b3d302
UI version: dev-8f22705

Test Result:
Issue 1: failed
The latest run information doesn't update after clicking Deploy Stop and Start as shown in the following screenshot:

Issue 2: pass

Environment

Walrus version: main aba5070fb2d7f86be3c14bd419ee7be9a8b3d302
UI version: dev-0c8c37a

Test Result: pass


Better handling of invalid Date headers

I spotted in some emails a date in the following format:
Date: 06-MAY-2016 19:00:22

I think due to the uppercase month, k-9 mail is unable to handle them, and displays them as they were sent at the time they were received. Can you work around this issue?

Oh, I forgot: I am receiving said emails with IMAP

We won't add support for decoding another non-standard date format. What we could do is mark these messages in the database and display the value of the date header when viewing the message. For sorting purposes we'd still have to use the date of retrieval, though.

Looks perfectly reasonable to me. You should still evaluate if you could be confusing your users showing a date which is different from the sort order; I would do something like a red date, and if the user taps it shows a popup of some sort explainig what happened with that date.

There's one problem with just ignoring invalid dates. Many (too many to ignore) people use the gmail's function of retrieving messages from different e-mail accounts via POP3 and gmail doesn't sanitize the dates here. So even if the original server fixes the date headers to be RFC-compliant, the mails already fetched are irrecoverably broken.
It'll just end up with non-technical people ranting that gmail just works

I have (probably) similar problem (k9mail shows date of sync instead of correct value). IMHO the raw value should be displayed instead. Regarding sorting order I would prefer to have the order of messages as on (IMAP dovecot) server which is order of arrival (or folder move).
Alpine shows this date as correct:  Thu, 21 Nov 2013 21:28:42 GMT-07:00

Hi,
I'll comment on here instead of opening a new issue, since my problem appears to be quite similar.
I've got some email from a specific sender that are displayed with the time of sync.
I don't believe that the Date field is an improper format... since I can't find any Date field in the headers.
The fields I have are:
Return-Path<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: from localhost (HELO queue) (<IP_ADDRESS>)
    by localhost with SMTP; 20 Feb 2019 10:26:15 +0200
Received: from unknown (HELO mail.ovh.net) (123.456.789.123)
    by ovh.net with AES256-GCM-SHA384 encrypted SMTP; 20 Feb 2019 10:26:15 +0200
Received: from mail.ovh.net (unknown [<IP_ADDRESS>])
    by mail.ovh.net (Postfix) with ESMTP id 4449fv4q1jz25pXfp
    for<EMAIL_ADDRESS>Wed, 20 Feb 2019 08:26:15 +0000 (UTC)
Received: from mail.ovh.net (unknown [<IP_ADDRESS>])
    by mail.ovh.net (Postfix) with ESMTP id 4449fr1xWQzd0Ys5
    for<EMAIL_ADDRESS>Wed, 20 Feb 2019 08:26:12 +0000 (UTC)
Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=123.456.789.123; helo=subdomain.senderdomain.com<EMAIL_ADDRESS>receiver=<UNKNOWN>
Authentication-Results: mail.ovh.net;
    dkim=pass (2048-bit key; unprotected) header.d=senderdomain.com<EMAIL_ADDRESS>header.b="QKvEtMWj";
    dkim-atps=neutral
Received: from subdomain.senderdomain.com (subdomain.senderdomain.com [123.456.789.123])
    by mail.ovh.net (Postfix) with ESMTPS id 4449fq6mXczD02cW
    for<EMAIL_ADDRESS>Wed, 20 Feb 2019 08:26:11 +0000 (UTC)
Received: from Alerts.senderdomain.com (subdomain.senderdomain.com [123.456.789.123])
    by subdomain.senderdomain.com (Postfix) with SMTP id 708AC17954C
    for<EMAIL_ADDRESS>Wed, 20 Feb 2019 03:26:03 -0500 (EST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=senderdomain.com;
   ......
To<EMAIL_ADDRESS>From<EMAIL_ADDRESS>Reply-To<EMAIL_ADDRESS>Subject: yyyyyyyyyyy
MIME-Version: 1.0
Content-Type: text/html
Content-Disposition: inline
X-Ovh-Remote: 123.456.789.123 (subdomain.senderdomain.com)
X-Ovh-Tracer-Id:<PHONE_NUMBER>543130857
X-VR-SPAMSTATE: OK
X-VR-SPAMSCORE: 49
X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgedtledruddtledgjeelucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuqfggjfdpvefjgfevmfevgfenuceurghilhhouhhtmecuhedttdenucfpohcuuggrthgvuchfihgvlhguucdlgeelmd
X-Ovh-Spam-Status: OK
X-Ovh-Spam-Reason: vr: OK; dkim: disabled; spf: disabled
X-Ovh-Message-Type: OK

Roundcube displays the message with the "right" date.
Would it be possible for K9 to use a value from one of the Received field, when no date is present, instead of the value from the last sync?

I have changed my mind on this. I'm no longer opposed to performing all kinds of magic to derive a sensible display date from a message. However, in those cases I want K-9 Mail to display an error banner, letting users know that the email is broken. Ideally users will contact the sender of the message and get them to fix their system so future messages won't show up in K-9 Mail with an annoying error banner.

Sounds fair to me.
Do let me know if you need more details or other example than the one I posted above.

I think many mails with broken/missing date headers are created by poor mail scripts (newsletter / spam mailers) and sometimes the date value is wrong (unrealistic old date / future date), too. IMHO it's totally fine to have some "ugly" (less clean/polished) display style in details view for those  (like banner / warning sign ⚠ / additional line etc).
Some thoughts:
a) date is correct => internally use and display this parsed value
b) date is not correct but can be parsed (fallback logic) => internally use parsed value, but display raw value with warning sign.
c) date is not parseable or ambiguous or missing: find some fallback date in "Received:" header (IIRC Outlook does something like this when ordering by arrival) and internally use that value. Display replacement value + warning + raw value (or "Date header missing") + maybe some hint/icon that raw value is a guess/replacement
Just my 2 cent.

One reason I think that the current approach should be changed (trying to use any date from the headers, rather than the received date of the email) is because ideally there is no state in an IMAP client. So, when I see a mail displayed in K-9 mail with the wrong date, I immediately start to fear that it has corrupted my emails. This isn’t the case but still frightening.
I also don’t think that displaying an error banner  should be a requirement for such a change: anyway displaying the retrieval date is already a broken situation. Maybe a banner should be shown, or maybe not, but there is no reason that a banner is more required if the app is showing a date extracted from another header, rather than the retrieval date.

I want to show an error banner because silently fixing the situation will take away the little bit of pressure there is to fix the bug causing the problem. Showing a banner is more annoying than showing a wrong date. So ideally users will a) blame the correct party and b) create enough pressure to get the software fixed that generated the broken message.
Adding the "date fixing" first and the error banner later, is a great way to never show an error banner because nobody will ever get around to adding it. And now K-9 Mail is part of the problem (silently fixing the errors of others).

Ok, that’s fair. I just hope that it does not mean that the more complex fix is not implemented ("the best is the enemy of the good" as we say in French).
I would also recommend that the banner is more an "info" than an error, to not scare users who might not understand what this is about.

The origination date aka "Date:" field is a (one of two) standards-required header field. I don't think that a client should try to reverse engineer a "correct" date. Rather I wouldn't display any date and would let the message fall where it may on a "date sent" sort (with a "no date provided" error message in place of the missing date).
There are clear standards (IETF RFCs) for these things. I don't favour mail client developers spending their time trying to patch things up for mail server administrators who can't be bothered with running standards-compliant servers. As cketti noted, if you silently fix these types of errors you become part of the problem (and waste a lot of time in the process).

There are new buggy softwares written everyday, computers with mis-configured date, etc.. So even if they are regularly fixed, a user of K9 mail will continue to receive faulty emails forever. While we can try to improve the situation by displaying a warning and hoping that users will reach out to the sender, providing a voluntarily broken experience would ultimately be punishing K9 mail users for something that is out of their hands (but I understand that this is not what @cketti was proposing).
Also, the Date: header is required in an email, but there is no provision that this is the header that must be displayed by an email client. A simpler approach than the proposed parsing of other header could be to use email the internal date from the server. I believe that this is reasonably common (some clients offer the option to use that one by default).

Could I ask for the (very simple IMHO) option to just leave emails with missing Date: header alone (that is, actually leave blank the spot where the date appears), and sort them as being the oldest in the list?

This issue has been open for a long time. When is the better handling of invalid date headers going to be implemented please?

IMHO, emails with a missing or invalid Date: header should use the date from the first Received: header. I would have no objection to a warning banner being displayed on them though.


Fix generation of failures.csv when there are only non-outcome failures

Fix a corner case from #144 that prevents the upload of outcomes.csv when there are failed jobs but no failed test cases (e.g. all tests that compile are passing, but there are configurations where the compilation fails).
Test run:

development, release job (with Windows-2015 failing and causing the problem): OpenCI
2.28, release job (all passing): OpenCI

I checked the CI results and they look good: the development run has an empty failures.csv and a non-empty outcomes.csv.xz as expected.


SW-4368 Track changes to batch details

Add history tables to track changes to user-editable batch details including
sub-locations. Like the quantity history tale, each row in the details history
table is a snapshot of the values of a particular version.
Since some of the details are references to things that can be deleted
(projects, sub-locations), we also record the names of those things as of the
time the edit happened, so that we can show a sensible name when someone looks
at the batch's history afterwards.

Current dependencies on/for this PR:

main

PR #1447   👈

PR #1448

This stack of pull requests is managed by Graphite.

Merge activity

Nov 2, 3:33 PM: @sgrimm started a stack merge that includes this pull request via Graphite.


[BUG] Custom webhook fails if Slack subject previously configured for trigger action

What is the bug?
Configuring a custom webhook after previously configuring a Slack webhook fails to parse JSON. This is due to OpenSearch sending the previously-defined Slack subject along with the custom webhook message contents. This will cause a parse error in, for example, OpsGenie.
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to a Alerting -> Monitors -> any monitor that uses custom webhook
Click on Edit -> Trigger -> Actions
Select a Slack notification, a "Message Subject" field appears
Set a subject for the Slack notification
Select the custom webhook, which hides the "Message Subject" field
Click "Send a test message"
OpsGenie will report a "Invalid JSON" error
Re-select a Slack notification channel
Clear "Message Subject"
Re-select custom webhook
Click "Send a test message"
Test message will be sent succesfully

What is the expected behavior?
When a custom webhook is selected and "Message Subject" field is hidden, OpenSearch should not send the message subject data with the webhook.
What is your host/environment?

OS: Fedora Linux
Version: OpenSearch 2.4.1

Transferring to alerting dashboards plugin repo, as early investigation suggests this is a bug in the frontend code.

@antti-aiven I'm having some trouble reproducing this issue on my end. Would you mind sharing the configurations for your slack and custom webhook channels (scrubbed of any personal info)? The configurations can be obtained using the list all  or get channel API.

Hi, thank you for the request, I'll get back to you as soon as possible. Currently this seems like an UI bug to me - Dashboards hides the UI element when switching from Slack to OpsGenie, but sends the contents anyway with the OpsGenie payload if the slack message subject is not empty.