Incorrect highlighting of keywords and brackets

If keywords stand alone, everything is displayed correctly.
However, if a bracket follows, the keyword is interpreted as a function and is displayed incorrectly.
Beispiel
if a < b then ... // is displayed correctly
if (a<b) then ... // is displayed incorrectly.

Fix in 0.8.6


[FLINK-18536][kinesis] Adding enhanced fan-out related configurations.

Note
This is a redo of the closed PR(https://github.com/apache/flink/pull/12920).
What is the purpose of the change
This is the fifth milestone of FLIP-128 to add EFO support to the FlinkKinesisConsumer. The EFO related configurations has been introduced.
This change should not introduce any functional differences.
Brief change log

ConsumerConfigConstants

Add several efo related configuration constants

KinesisConfigUtil

Add the validation of efo related configuration constants.

KinesisProxy

Add fields about operations like registerStream, deregisterStream, listStream
Give initial value to those fields.

FanOutProperties

Add a new configuration class for the future FanOutRecordPublisher
Validate fields and give them initial values

FlinkKinesisConsumer

Since the change of the validation interface, modify the call of validation method.

Verifying this change
This change added tests and can be verified as follows:

KinesisConfigUtilTest
FanOutPropertiesTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@xiaolong-sn I am using this branch for my EFO testing and in order to configure the connector I am doing this:

consumerConfig.setProperty(EFO_CONSUMER_ARN_PREFIX + "." + streamName, "<my-consumer-arn>");

It would be much nicer if there was a method (similar to the AWSConfigConstants.accessKeyId(...) etc):

consumerConfig.setProperty(ConsumerConfigConstants.efoConsumerArn(streamName), "<my-consumer-arn>");

@xiaolong-sn I am using this branch for my EFO testing and in order to configure the connector I am doing this:

consumerConfig.setProperty(EFO_CONSUMER_ARN_PREFIX + "." + streamName, "<my-consumer-arn>");

It would be much nicer if there was a method (similar to the AWSConfigConstants.accessKeyId(...) etc):

consumerConfig.setProperty(ConsumerConfigConstants.efoConsumerArn(streamName), "<my-consumer-arn>");

Sure, I'll add the method and do some refactor job right now.


Text extractor auto detect language

Description of the new feature / enhancement
I am pretty sure previously you could just switch keyboard language and I can extract text based on the language I am currently using. Now I need to go to the actual app and manually change it to the language I would like to extract text from. Pretty annoying for someone who uses it for more than one language and worst of all it was a feature before, perhaps its now a bug? Hope it gets fixed ready for the next update/
Scenario when this would be used?
I use multiple languages and extract from both so its much more convenient.
Supporting information
No response

I would like to +1 this as an enhancement.  Having to open it every time I need to switch languages is not ideal.

Just to be clear what is being asked for here. Are you looking to have Text Extractor identify what language is being OCR'd? Or are you asking to have Text Extractor remember the last used language?
I am looking to improve the language selector to be more like Text Grab with hot keys to switch languages.

Text Extractor will not be able to auto detect the language unless there is a change to the underlying Windows OCR API. Changing that API is beyond the scope of this project, so I am going to close this issue.


Placeholder text for custom attributes

Would be super neat to have custom attributes support placeholder text to show the expected format of the data entered. For example, we could have the following attribute show 2023-01-01 in the text box. Note, I suppose this value would be obscured whenever a default value is set but I think that's perfectly fine/natural.
attribute :click_date, :string, placeholder: "2023-01-01"
validates :click_date, format: { with: /\A(\d{4}-\d{2}-\d{2}){0,1}\z/ }

I am personally interested in giving a go at implementing such functionality.

Ah I see. While UI wise we render textareas which support placeholders the underlying attribute system does not support such a key/value pair. Indeed using date sounds like the best option. 👍
Will be closing this as while an interesting idea, it's probably not worth the investment right now.


Improve auto-upgrade enablement UX

There are some current UX gaps when it comes to setting auto-upgrade. We should improve them such that each of the below items are addressed:

Keep the structure API as-is
the server side/backend/daemon should only look at isPattern() or autoUpgrade == true. those are the only scenarios that puts an app into auto-upgrade mode. Implications of this:

If an app just has notifyUpgrade set, it is NOT autoUpgrade
If an app just has interval set, it is NOT autoUpgrade
If an app just has a pattern for its tag, it IS autoUpgrade
If app just as autoUpgrade set, it IS autoUpgrade

CLI should compensate to turn on autoUpgrade if related fields are set. it should set autoUpgrade to true if:

the interval is set
if notifyUpgrade is set
the image is a pattern (maybe this one is not necessary and it could actually possibly complicate things)

we'll have to consider update scenarios that turn off auto-upgrade. Seems like one cannot currently unset interval from the cli. if thats true, we should fix that.

Related https://github.com/acorn-io/manager/issues/801

we'll have to consider update scenarios that turn off auto-upgrade. Seems like one cannot currently unset interval from the cli. if thats true, we should fix that.
Everything except this is adjusted now. Going to be looking at that and the other linked issue.

Also realted https://github.com/acorn-io/runtime/issues/1996

Tested with acorn version v0.8.0-alpha7-210-g2e8339a2+2e8339a2
Following tests are done from CLI:

Deploy apps with images in autoupgrade format (no --auto-upgrade  or --interval set or --notify-upgrade)
Deploy apps with only  --notify-upgrade set
Deploy apps with only --interval set
Deploy apps with only --auto-upgrade set

Autoupgrade gets triggered as expected in all the above cases.

Following tests were done for nested Acorns:
Deploy nested apps using the following Acornfile:
acorns: {
    "my-acorn1": {
        image: "docker.io/sangeetha/myfirstacorn:v#.#.#"
    }
   "my-acorn2": {
       image: "docker.io/sangeetha/myfirstacorn:latest"
        notifyUpgrade: true
   }
    "my-acorn3": {
        image: "docker.io/sangeetha/myfirstacorn:latest"
        autoUpgradeInterval: "15s"
    }
   "my-acorn4": {
       image: "docker.io/sangeetha/myfirstacorn:latest"
       autoUpgrade: true
        notifyUpgrade: true
   }

}

Only my-acorn1 and my-acorn4 will get deployed in auto upgrade mode as expected.
acorn apps -j acorn
NAME                     IMAGE                                             COMMIT    HEALTHY   UP-TO-DATE   CREATED    ENDPOINTS                                                                              MESSAGE
mytestnested             e4294669211c                                                0         0            110m ago                                                                                OK
mytestnested.my-acorn1   index.docker.io/sangeetha/myfirstacorn:v46.0.0*             1         1            110m ago   http://myweb-mytestnested-my-acorn1-b3a46aa5-8985526f.local.oss-acorn.io => myweb:80   OK
mytestnested.my-acorn2   docker.io/sangeetha/myfirstacorn:latest                     1         1            110m ago                                                                                OK
mytestnested.my-acorn3   docker.io/sangeetha/myfirstacorn:latest                     1         1            110m ago                                                                                OK
mytestnested.my-acorn4   docker.io/sangeetha/myfirstacorn:latest*                    1         1            110m ago   http://myweb-mytestnested-my-acorn4-7dbfce55-5ccfcc15.local.oss-acorn.io => myweb:80   OK


Installation steps mentioned in readme.md is not working

In readme file  npm install -D express-routes-catelogue, there is a typo
it should be  npm install -D express-routes-catalogue
Try to publish the latest version, it will solve this issue.

@NaveenDA , Thanks for notifying this mistake, updated in<EMAIL_ADDRESS>


Feature: Location

I am loving your package.
Any chance you worked with location.
I used regular libraries but it always timed out.
Is there specific api from sunmi for location ?

what ？
---- Replied Message ----
FromMahir @.>Date08/23/2024 15:29 @.> @.>Subject[Surile/react-native-sunmi-printer] Feature: Location (Issue #55)
I am loving your package.
Any chance you worked with location.
I used regular libraries but it always timed out.
Is there specific api from sunmi for location ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.>

Did you have to get location from Sunmi devices, how do you do it ?

I'm not asked to get my location


Serial monitor not responding

I am using the OV5640, and I followed the instructions in the video https://www.youtube.com/watch?v=j9Z0KBuwEGI&t=1321s to upload the code. I waited for about an hour, but the serial monitor is not responding at all.
I also tried uploading a simpler piece of code, but the serial monitor still doesn't display anything.
I am not using the PCIE-SIM module, so I'm not sure if that has any effect.

Enable CDC
https://github.com/Xinyuan-LilyGO/LilyGo-Camera-Series/blob/984166b0426e73df8551c0a9abc5e97c93a29649/platformio.ini#L52

I have enabled CDC, but the serial monitor still doesn't return anything

After starting up, wait a while and test again. As long as you can upload the sketch, it means there is no problem with the communication.


Add PHP type hints for constructor and optional args

Add type hints for array arguments in constructor and methods. cc @dwsupplee @jdpedrie

Codecov Report

Merging #2055 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master    #2055      +/-   ##
============================================
- Coverage     86.82%   86.82%   -0.01%
- Complexity     4987     4989       +2
============================================
  Files           449      449
  Lines         19731    19733       +2
  Branches       2081     2081
============================================
+ Hits          17132    17133       +1
  Misses         1891     1891
- Partials        708      709       +1

Impacted Files
Coverage Δ
Complexity Δ

...gen/viewmodel/DynamicLangDefaultableParamView.java
75% <0%> (-25%)
4 <2> (+2)

.../transformer/php/PhpApiMethodParamTransformer.java
91.83% <100%> (+0.08%)
20 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


⚠ Bofrost.at has degraded performance

In 1a6dfd4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1240 ms

Resolved: Bofrost.at performance has improved in 5074272.


MB-16107: Use AWS ECS metadata v4

Jira ticket
Summary
The PR has links to the relevant documentation.
This has been tested in the loadtest environment.
Verification Steps for the Author
These are to be checked by the author.

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

Messages

:book:
:link: MB-16107

  Generated by :no_entry_sign: dangerJS against ebd92b90192d000c533c2af3d3aef20c7d9c91a2


Record unused schema ledgers to ZK for partitioned topics

Motivation
When creating schema for a partitioned topic, e.g. my-topic, only one z-node /schemas/public/default/my-topic would be created to record the schema's metadata. However, multiple ledgers would be created because each single partition would try to create the schema: first append a ledger, then write the same z-node.
#2959 and #6683 solved the race condition of accessing the same z-node by calling putSchema() again. However, the unused ledgers of other partitions were created and were not recorded in the z-node. So, there's no way to locate these ledgers from ZK, which means there's no way to delete a topic's all associated schema ledgers.
Modifications

Record these unused ledgers' id under the topic's z-node, e.g. /schemas/public/default/my-topic/unusedLedgers/0 for topic my-topic and ledger 0.
Add a unit test to check if unusedLedgers z-node has these ledgers' id and these ledgers exist, after a partitioned topic's schema was created.

Verifying this change

[ ] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:
Run test PartitionedTopicsSchemaTest.testUnusedLedgers.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

Close this PR first because it's a temporary solution for current schema design. After discussing with @codelipenghui , the schema storage design should change, then this PR may not be needed.


How to set RUSTFLAGS in release.yml?

I need to set
rustflags = ["--cfg", "tokio_unstable"]
or export RUSTFLAGS="--cfg tokio_unstable"
as part of my build, but can't figure out how to get release.yml to pick it up. When I modify the release.yml generated by cargo-dist to set RUSTFLAGS inline whenever it runs build I get an error that the release.yml is dirty as part of plan.
What's the correct way to set build flags for cargo-dist as part of CI?

Okay got it woorking this way, thanks!


Fixes formatting.

Docs Build status updates of commit 3870509:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3870509:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/introduction-sharepoint-online-management-shell.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3fe1f4e:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3fe1f4e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/introduction-sharepoint-online-management-shell.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 10d86cb:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 10d86cb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/introduction-sharepoint-online-management-shell.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


MeshSync icon aligned with heading

Signed off by : Ambareen Azam<EMAIL_ADDRESS>Notes for Reviewers
This PR fixes #7286
Signed commits

[X] Yes, I signed my commits.

🚀 Preview for commit 57bfc5febed681d211bfb3579c39764a8acfd4c2 at: https://6419e2483e5b1c036bda1d33--meshery-docs-preview.netlify.app

@Ambareen09  Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)


Store issuer chains

Codecov Report
Attention: Patch coverage is 11.68831% with 68 lines in your changes missing coverage. Please review.

Project coverage is 34.46%. Comparing base (46ec9c2) to head (42aaa9c).
Report is 49 commits behind head on main.

Files
Patch %
Lines

personalities/sctfe/storage/gcp/map.go
0.00%
43 Missing :warning:

personalities/sctfe/storage.go
0.00%
16 Missing :warning:

personalities/sctfe/ct_server_gcp/main.go
0.00%
7 Missing :warning:

personalities/sctfe/handlers.go
33.33%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
- Coverage   35.80%   34.46%   -1.34%
==========================================
  Files          16       34      +18
  Lines        1363     2849    +1486
==========================================
+ Hits          488      982     +494
- Misses        801     1758     +957
- Partials       74      109      +35

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


Data string whitespaces

Hi,
I got this one reported to wavedrompy: https://github.com/wallento/wavedrompy/issues/12
Isn't it actually something that should work differently in wavedrom?
Best,
Stefan

Yes, the behavior is different. I think you are doing it better. Will fix WaveDrom.


Refresh search person after update

fixes #352

 Updating a Customer does not refresh the person details in SearchCustomerView #352

and fixes #437


esp32 azure direct method sample

Hi,
can you please help with a sample code to connect device(esp32) with azure iothub using direct method in espidf.
thanks

@naznaaa Thank you for your query . We will investigate and update this thread further.

@naznaaa Please have a look at this repo: https://github.com/espressif/esp-azure and let us know if you need further help.

Hi,
I am referring to direct method, here no response is send to iothub after receiving messages from iothub.
can you please check and let me know.
thanks

Hello @naznaaa as this is not related with a doc under docs.microsoft.com we kindly ask you to file an issue on the repository mentioned by @AshokPeddakotla-MSFT - https://github.com/espressif/esp-azure/issues
Thank you!

We will now proceed to close this thread. If there are further questions regarding this matter, please tag @AshokPeddakotla-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.

hello @AshokPeddakotla-MSFT we have to connect esp32 and azure using x.509 certificate authentication by direct method communication in espidf . can you please provide a sample or document to refer

@naznaaa please follow-up on the https://github.com/espressif/esp-azure/issues/25 since @mahavirj is already helping you there. As mentioned here: "Customers can extend our platform support by porting the Azure IoT C SDK, specifically, creating the platform abstraction layer (PAL) of the SDK. Microsoft works with partners to provide extended support. A list of partners has ported the C SDK on to more platforms and maintaining the PAL."
Thanks!


Add Okta converters Faros Destination

Description
Add Okta converters Faros Destination
Type of change

[ ] Bug fix
[x] New feature
[ ] Breaking change

Related issues
https://github.com/faros-ai/airbyte-connectors/issues/224

Resolve conflicts


Does this project support Chinese data sets?

I will complete a Chinese emotion analysis task. Can this project complete the task of Chinese emotion analysis?

Yes, you've to train the embedding weights along with the model (in case you don't have embeddings for Chinese words already)

Yes, you've to train the embedding weights along with the model (in case you don't have embeddings for Chinese words already)


Revert workaround. Scala.js now supports Character.isLetter

Could you make two commits out of this. First the Scala.js upgrade, then simply git revert 37cc28891403c9807fd9c9e17ecace3d69eab702.

The Travis builds crash the JVMs, for some reason :s

I fixed the JVM crashes through #4. You can now rebase on top of scalajs.

LGTM


Fix TextEdit.get_rect_at_line_column returning negative pos even though cursor is in viewable area of the control

Sister PR of https://github.com/godotengine/godot/pull/81352 (for Godot 3.x)
When both are merged, https://github.com/godotengine/godot/issues/81349 can be closed
Production edit: Fixes https://github.com/godotengine/godot/issues/81349
Before fix:
textedit-bug_godot-4.webm
After fix:
after.webm

@AThousandShips: done, ready for review

I would do it, but this does not make any sense in my head: if you drag the mouse from column 1 to 7 you should get his is. But apparently you even land in line 1, not 0... Similar confusion with other uses of get_pos_at_line_column in test_text_edit.h
text_edit->set_text("this is some text\nfor selection");
text_edit->grab_focus();
MessageQueue::get_singleton()->flush();

SEND_GUI_MOUSE_BUTTON_EVENT(text_edit->get_pos_at_line_column(0, 1), MouseButton::LEFT, MouseButtonMask::LEFT, Key::NONE);
SEND_GUI_MOUSE_MOTION_EVENT(text_edit->get_pos_at_line_column(0, 7), MouseButtonMask::LEFT, Key::NONE);
CHECK(text_edit->has_selection());
CHECK(text_edit->get_selected_text() == "for s");

Thanks!


Do Yi-34B-Chat-Int4  support function call

The chat version has been released.  Does it support function calls?  How should I build a prompt for function call.

No, this version doesn't support function calls yet.

@findmyway Do you have any plan to add function call functionality? if yes, when will it be available?

Hi @gunpal5 ,
Yes, we have the plan. But we do not have a clear timeline yet.

如果基于ms-agent数据集微调之后，能支持function call吗


Ubuntu snap created for rclone

Hi ncw
i wanted to make it easier for people to install rclone on different types of Linux distros, so i went ahead and setup an Ubuntu snap for rclone. you can install rclone by doing the following command listed below. luckily snaps can be deployed on other distro's as well such as Redhat, CentOS, Debian etc. currently i have my launchpad account pulling from my fork automatically using git and building an new snap each time an change is  detected. so you could add my snapcraft.yaml file to the main repo for rclone, then you could  update the version in the file when an new version is pushed out.  then i can have launchpad pointed at the main repo for rclone if you would like.
https://github.com/Dedsec1/rclone/
https://github.com/Dedsec1/rclone/blob/master/snapcraft.yaml
sudo snap install rclone --classic
Regards
Dedsec1

That looks great!
Do you think you could send a pull request with

the snapcraft.yml
some updates to the docs (you only need to edit docs in docs/content the others are auto generated) describing how to configure snap and install rclone

Then I can integrate the changes :-)
Thanks
Nick

docs has been created https://github.com/Dedsec1/rclone/blob/master/docs/content/rclone snap.md

I've merged your PR now thank you!
I've got some questions about it though...
Are you willing to maintain the snap going forwards?  If so I'd like to put some details for you in the install.md (or you could send a PR).
Can you point your builder towards the official rclone source now?
Is the snap builder checking out a release or is it just using the latest commit?
And do I need to update the version number in the snapcraft file when I make new releases? I presume so...

sure thing, i dont mind maintain the snap going forward, the only thing i have to do is change the version number in the yaml file each time. i already pointed the auto builder for your git repo so Ubuntus servers will do the rest when it comes to building.

Great.  Can you give me a URL to point to the builder for the docs?

here is the repo i created on my launchpad account, https://launchpad.net/~deathero291/+snap/rclone
as of right now i have it setup to pull any changes from your repo every few hours.

I can't merge this as it stands.  What you need to do is reset your master branch back to origin/master, then make a feature branch with this change it.
I'd really like it to pull the current stable version, so it should use the git tag of that version number.
And we should make a separate rclone-dev to install the development version.
Is that possible?

This is a great idea! I tried using it and seems to be working fine. Though I think it needs more permissions. I am using it on a raspberry pi, so obviously my storage is external. To access it, rclone needs to access the removable-media plug.
Also, you might want to try the fuse-support to see if you can get mount working.
Thanks!

@Dedsec1 any chance you could look at @joewashear007 s comments?  I think fixing them would probably fix #1188

i will have to look into this since fuse might have different deps that it needs to run. but it should be possible

I'm new to snapd, so correct me if I am wrong here.
In the documentation it says to install rclone in classic mode, which I believe means rclone will not be sandboxed to its snap directories on the filesystem.  However, in snapcraft.yaml the confinement mode is strict.  Thus, when I try to access files outside of the snap sandbox, or my home directory, access is denied.
I think one of two things should change:

The documentation should be updated to reflect the filesystem limitations of snaps.
snapcraft.yaml should be updated to classic confinement

Which is the best choice I am not sure.

@Dedsec1 do you have an opinion on this?

Regarding the sandbox file system, you can use mount --rbind  /home//directory. I maintain few snap myself but setting classic confinement are very risky and most of the time it won't get approve.

I'm going to close this for the moment as a snap is created.
However I would much appreciate PR to update the snap or the docs.

sure thing nick, i will update it today

Is the snap dead? It seems it's at version 1.36 and latest is 1.40 :(

We've stopped releasing rclone with snap as it became to hard to make it work especially with rclone mount.  If you look at the rclone.org/downloads page you'll see there are lots of convenient way of installing rclone nowadays.

Darn I really wanted a way I could install it safely and have it confined in a snap.


[WIP] Improve code

Still working on it. Next step is adding more test case to verify the correctness and illuminate corn case. Kill such case via test cases.

This change is

No google c++style, too ugly.

Added some fun game, check my comments.

Still need this constructor:
Chan(size_t quota_, size_t capacity_) : quota(quota_), capacity(capacity_)


Getting unsupported operand type(s) for |: 'PydanticRecursiveRef' and 'NoneType' with PEP 604 formatting and recursive classes

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
We recently started using pyupgrade for formatting in a repo with complex pydantic models to switch from Optional[type] to type | None but one of our models now throws an error at runtime. The strange thing about this is, in the example code, if the typing in Foo for bar, bar: Bar[Baz] | None = None, is changed to use Union like bar: Union[Bar[Baz], None] = None the code no longer throws an error. So I think there is likely a bug in how Union vs UnionType is handled in recursive references.
Example Code
from __future__ import annotations

from pydantic import BaseModel
from typing import TypeVar, Generic

class Foo(BaseModel):
    id: int
    bar: Bar[Baz] | None = None

_T = TypeVar("_T")

class Bar(BaseModel, Generic[_T]):
    name: str
    parent: Foo | None = None

class Baz(BaseModel):
    name: str | None

Foo.model_rebuild()
Bar.model_rebuild()

Python, Pydantic & OS Version
pydantic version: 2.4.2
        pydantic-core version: 2.10.1
          pydantic-core build: profile=release pgo=false
                 install path: /fn/lib/venv/lib/python3.10/site-packages/pydantic
               python version: 3.10.13 (main, Oct 16 2023, 01:57:50) [GCC 10.2.1 20210110]
                     platform: Linux-5.15.49-linuxkit-aarch64-with-glibc2.31
             related packages: typing_extensions-4.8.0 mypy-1.6.0

Here's a fix that you can patch in right away:
from pydantic._internal._forward_ref import PydanticRecursiveRef
from typing import Union

def __or__(self, other):
    return Union[self, other]

PydanticRecursiveRef.__or__ = __or__

@pengelsman7,
Thanks for reporting this. This does seem like a bug, and might be related somewhat to #7873.

Thank you for the quick response! I'll give the patch a try while we wait for the new version.


az webapp up for Python doesn't trigger the kudu build

Looks like az webapp up for Python doesn't trigger the kudu build based on the requirements.txt file.

this is fixed


Song plays when exiting spectator mode

When exiting spectator mode (and still in the pause menu), the level song starts playing. May be related to #19.

should be fixed next release


Info entities doesn't reflect the new sensor display precision setting

Is your feature request related to a problem? Please describe.
For example I set a temperature value to be displayed without decimals, then the info_entities part still uses the original precision for display.
Have you tried if it is already a feature? Have you checked the wiki?
The display precision for sensors was introduced just now, in release 2023.3
Describe the solution you'd like
It would be great if the display of sensors would use this new feature.
Describe alternatives you've considered
none
Additional context
none

Sorry for the late response, but this isn't something HA passes to the card. You can set the precision like shown here
https://github.com/marcokreeft87/room-card/wiki/Configuration#formatting:
precision<0-9> Set decimal precision of number value (precision3 -> 18.123)

Thanks for the help, I just missed this part in the documentation.
I thought that this is a generally available function which could be used in any custom card. The new feature announcement states that:
`The examples below show the output of a temperature sensor with state 20.001, unit °C and user configured presentation rounding set to 1 decimal.
The following example results in the string "20.0 °C": `
{{ states('sensor.temperature', with_unit=True) }}

Unfortunately, thats now how it works. HA has its own parsing and not everything is available in the javascript libraries.

It looks like this may no longer be the case, as the third party mushroom custom card is showing numbers with the precision set in home assistant where room-card is not. I've not looked into mushroom's code to see what it's doing differently, but I have tested and seen mushroom updating the precision of numbers where room-card is not.
It would be awesome if this precision handling could be backported into room-card now <3

It sounds like I need to have another look at this

Shouldn’t we reopen this issue?


How I can create a virtualbox machine offline?

I tested with absolute file and file:/// scheme, but with no success.
Many thanks
Peter
$ docker-machine create -d virtualbox tomcat-2 --virtualbox-boot2docker-url ///Users/xxx/.docker/machines/tomcat/boot2docker.iso
ERRO[0000] Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: stat /etc/resolv.conf: no such file or directory
WARN[0000] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0000] Error creating machine
$ docker-machine rm tomcat-2
INFO[0000] machine does not exist, assuming it has been removed already
$ docker-machine create -d virtualbox tomcat-2 --virtualbox-boot2docker-url /Users/xxx/.docker/machines/tomcat/boot2docker.iso
ERRO[0000] Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: stat /etc/resolv.conf: no such file or directory
WARN[0000] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0000] Error creating machine
$ docker-machine rm tomcat-2
INFO[0000] machine does not exist, assuming it has been removed already
$ docker-machine create -d virtualbox tomcat-2 --virtualbox-boot2docker-url file:///Users/xxx/.docker/machines/tomcat/boot2docker.iso
ERRO[0000] Error creating machine: Get https://api.github.com/repos/boot2docker/boot2docker/releases: dial tcp: stat /etc/resolv.conf: no such file or directory
WARN[0000] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0000] Error creating machine

We should skip the release check if a file is specified.  Thanks for reporting.

Great:) Many thanks!

This is very relevant in a hackathon setup.

Is this still an issue in HEAD? the code in virtualbox.go seems to imply that the release check only occurs if no b2d ISO URL was specified.

Oh! This might just be a flag issue. I seem to recall running into this a while back.
Try putting a = between the flag and the argument:
$ docker-machine create -d virtualbox tomcat-2 --virtualbox-boot2docker-url=file:///Users/xxx/.docker/machines/tomcat/boot2docker.iso

Hmm looks like a regression.  I'm getting unsupported protocol scheme when trying to use the file.  Checking.

virtualbox driver use a http.client module. It doesn't support file scheme!

It doesn't out of the box, but it can: http://golang.org/pkg/net/http/#Transport.RegisterProtocol

This looks relevant too: https://golang.org/src/net/http/filetransport.go

@hairyhenderson @arun-gupta #805 should fix this.  Would you mind testing?  Thanks!

Sure, I can. Which repository should I checkout?

@arun-gupta https://github.com/docker/machine/pull/805

@ehazlett This is likely a git noob question :)
How do I clone that repo?

@arun-gupta ah np :)  i can give you a test build.  What platform are you on (OS/arch)?

@ehazlett OSX, thanks!

@arun-gupta https://public.evanhazlett.com/docker-machine/gh-539/

Hi Evan,
works
$ cp ~/.docker/images/boot2docker.iso .
$ chmod +x docker-machine_darwin-amd64
$ ./docker-machine_darwin-amd64 create -d virtualbox tomcat-2 --virtualbox-boot2docker-url=file:///$(pwd)/boot2docker.iso
INFO[0001] Creating SSH key...
INFO[0001] Creating VirtualBox VM...
INFO[0009] Starting VirtualBox VM...
INFO[0010] Waiting for VM to start...
INFO[0044] "tomcat-2" has been created and is now the active machine.
INFO[0044] To point your Docker client at it, run this in your shell: eval "$(docker-machine_darwin-amd64 env tomcat-2)"

But I got a strange ls error with the binary with ls
$ ./docker-machine_darwin-amd64 ls
panic: runtime error: index out of range

goroutine 16 [running]:
runtime.panic(0x6538e0, 0xa5935c)
	/usr/src/go/src/pkg/runtime/panic.c:279 +0xf5
github.com/skarademir/naturalsort.NaturalSort.Less(0x208fbc380, 0x8, 0x8, 0x0, 0x6, 0x0)
	/go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/skarademir/naturalsort/naturalsort.go:23 +0x4f8
github.com/skarademir/naturalsort.(*NaturalSort).Less(0x208f17660, 0x0, 0x6, 0x0)
	<autogenerated>:3 +0xa6
sort.doPivot(0x2208b60c08, 0x208f17660, 0x0, 0x8, 0x200000000, 0x39992)
	/usr/src/go/src/pkg/sort/sort.go:129 +0x4ac
sort.quickSort(0x2208b60c08, 0x208f17660, 0x0, 0x8, 0x7)
	/usr/src/go/src/pkg/sort/sort.go:173 +0xa8
sort.Sort(0x2208b60c08, 0x208f17660)
	/usr/src/go/src/pkg/sort/sort.go:200 +0x7b
main.sortHostListItemsByName(0x208f958c0, 0x8, 0x8)
	/go/src/github.com/docker/machine/commands.go:69 +0x268
main.cmdLs(0x208b82840)
	/go/src/github.com/docker/machine/commands.go:508 +0x7a1
github.com/codegangsta/cli.Command.Run(0x6ef5b0, 0x2, 0x0, 0x0, 0x703790, 0xd, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/command.go:113 +0xeb2
github.com/codegangsta/cli.(*App).Run(0x208b3e000, 0x208b4c000, 0x2, 0x2, 0x0, 0x0)
	/go/src/github.com/docker/machine/Godeps/_workspace/src/github.com/codegangsta/cli/app.go:156 +0xaea
main.main()
	/go/src/github.com/docker/machine/main.go:67 +0x7cd

goroutine 19 [finalizer wait]:
runtime.park(0x2b4a0, 0xa5d8f8, 0xa5ba09)
	/usr/src/go/src/pkg/runtime/proc.c:1369 +0x89
runtime.parkunlock(0xa5d8f8, 0xa5ba09)
	/usr/src/go/src/pkg/runtime/proc.c:1385 +0x3b
runfinq()
	/usr/src/go/src/pkg/runtime/mgc0.c:2644 +0xcf
runtime.goexit()
	/usr/src/go/src/pkg/runtime/proc.c:1445

goroutine 20 [syscall]:
os/signal.loop()
	/usr/src/go/src/pkg/os/signal/signal_unix.go:21 +0x1e
created by os/signal.init·1
	/usr/src/go/src/pkg/os/signal/signal_unix.go:27 +0x32

goroutine 69 [select]:
net/http.(*persistConn).writeLoop(0x208b7e370)
	/usr/src/go/src/pkg/net/http/transport.go:885 +0x38f
created by net/http.(*Transport).dialConn
	/usr/src/go/src/pkg/net/http/transport.go:601 +0x957

goroutine 67 [select]:
net/http.(*persistConn).writeLoop(0x208b7e160)
	/usr/src/go/src/pkg/net/http/transport.go:885 +0x38f
created by net/http.(*Transport).dialConn
	/usr/src/go/src/pkg/net/http/transport.go:601 +0x957

goroutine 68 [IO wait]:
net.runtime_pollWait(0x2208b52410, 0x72, 0x0)
	/usr/src/go/src/pkg/runtime/netpoll.goc:146 +0x66
net.(*pollDesc).Wait(0x208b648b0, 0x72, 0x0, 0x0)
	/usr/src/go/src/pkg/net/fd_poll_runtime.go:84 +0x46
net.(*pollDesc).WaitRead(0x208b648b0, 0x0, 0x0)
	/usr/src/go/src/pkg/net/fd_poll_runtime.go:89 +0x42
net.(*netFD).Read(0x208b64850, 0x208bce000, 0x2000, 0x2000, 0x0, 0x2208b503f0, 0x23)
	/usr/src/go/src/pkg/net/fd_unix.go:242 +0x34c
net.(*conn).Read(0x208b7c3a8, 0x208bce000, 0x2000, 0x2000, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/net/net.go:122 +0xe7
crypto/tls.(*block).readFromUntil(0x208c1fe60, 0x2208b52838, 0x208b7c3a8, 0x5, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:451 +0xd9
crypto/tls.(*Conn).readRecord(0x208c14000, 0x17, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:536 +0x1ff
crypto/tls.(*Conn).Read(0x208c14000, 0x208c8d000, 0x1000, 0x1000, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:901 +0x16a
net/http.noteEOFReader.Read(0x2208b60560, 0x208c14000, 0x208b7e3c8, 0x208c8d000, 0x1000, 0x1000, 0xa71900, 0x0, 0x0)
	/usr/src/go/src/pkg/net/http/transport.go:1203 +0x72
net/http.(*noteEOFReader).Read(0x208d29180, 0x208c8d000, 0x1000, 0x1000, 0x208b6b9b0, 0x0, 0x0)
	<autogenerated>:124 +0xca
bufio.(*Reader).fill(0x208f6d080)
	/usr/src/go/src/pkg/bufio/bufio.go:97 +0x1b3
bufio.(*Reader).Peek(0x208f6d080, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/bufio/bufio.go:132 +0x101
net/http.(*persistConn).readLoop(0x208b7e370)
	/usr/src/go/src/pkg/net/http/transport.go:782 +0x95
created by net/http.(*Transport).dialConn
	/usr/src/go/src/pkg/net/http/transport.go:600 +0x93f

goroutine 66 [IO wait]:
net.runtime_pollWait(0x2208b522b0, 0x72, 0x0)
	/usr/src/go/src/pkg/runtime/netpoll.goc:146 +0x66
net.(*pollDesc).Wait(0x208b64990, 0x72, 0x0, 0x0)
	/usr/src/go/src/pkg/net/fd_poll_runtime.go:84 +0x46
net.(*pollDesc).WaitRead(0x208b64990, 0x0, 0x0)
	/usr/src/go/src/pkg/net/fd_poll_runtime.go:89 +0x42
net.(*netFD).Read(0x208b64930, 0x208bc8000, 0x2000, 0x2000, 0x0, 0x2208b503f0, 0x23)
	/usr/src/go/src/pkg/net/fd_unix.go:242 +0x34c
net.(*conn).Read(0x208b7c368, 0x208bc8000, 0x2000, 0x2000, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/net/net.go:122 +0xe7
crypto/tls.(*block).readFromUntil(0x208c1fb30, 0x2208b52838, 0x208b7c368, 0x5, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:451 +0xd9
crypto/tls.(*Conn).readRecord(0x208b4f8c0, 0x17, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:536 +0x1ff
crypto/tls.(*Conn).Read(0x208b4f8c0, 0x208c83000, 0x1000, 0x1000, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/crypto/tls/conn.go:901 +0x16a
net/http.noteEOFReader.Read(0x2208b60560, 0x208b4f8c0, 0x208b7e1b8, 0x208c83000, 0x1000, 0x1000, 0xa71900, 0x0, 0x0)
	/usr/src/go/src/pkg/net/http/transport.go:1203 +0x72
net/http.(*noteEOFReader).Read(0x208d29080, 0x208c83000, 0x1000, 0x1000, 0x208b6b9a8, 0x0, 0x0)
	<autogenerated>:124 +0xca
bufio.(*Reader).fill(0x208f6cde0)
	/usr/src/go/src/pkg/bufio/bufio.go:97 +0x1b3
bufio.(*Reader).Peek(0x208f6cde0, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
	/usr/src/go/src/pkg/bufio/bufio.go:132 +0x101
net/http.(*persistConn).readLoop(0x208b7e160)
	/usr/src/go/src/pkg/net/http/transport.go:782 +0x95
created by net/http.(*Transport).dialConn
	/usr/src/go/src/pkg/net/http/transport.go:600 +0x93f

goroutine 46 [sleep]:
time.Sleep(0x12a05f200)
	/usr/src/go/src/pkg/runtime/time.goc:39 +0x31
net.func·019()
	/usr/src/go/src/pkg/net/dnsclient_unix.go:183 +0x56
created by net.loadConfig
	/usr/src/go/src/pkg/net/dnsclient_unix.go:212 +0x153
$
$ docker-machine ls
NAME               ACTIVE   DRIVER       STATE     URL                         SWARM
amazonec2-03                amazonec2    Stopped
dev                         virtualbox   Running   tcp://<IP_ADDRESS>:2376
dev2                        virtualbox   Running   tcp://<IP_ADDRESS>:2376
ec2-swarm-01                amazonec2    Stopped                               ec2-swarm-master
ec2-swarm-02                amazonec2    Stopped                               ec2-swarm-master
ec2-swarm-03                amazonec2    Stopped                               ec2-swarm-master
ec2-swarm-master            amazonec2    Stopped                               ec2-swarm-master (master)
tomcat-2           *        virtualbox   Running   tcp://<IP_ADDRESS>:2376

@rossbachp that looks like it might be related to the new sorting.  @ahmetalpbalkan any idea here?

@ehazlett ack. this comes from a corner case bug in the naturalsort library we hacked the other night. fix on the way.

Fix is in. Pls ingest at your earliest convenience.

@ehazlett unfortunately godep is quite buggy for me every time I try. Can you please vendor the latest commit from the naturalsort repo?

@ehazlett got a different error:
~/Downloads/docker-machine_darwin-amd64 create mydocker --virtualbox-boot2docker-url=file:///Users/arungupta/Downloads/boot2docker.iso --driver=virtualbox
INFO[0000] Creating CA: /Users/arungupta/.docker/machine/certs/ca.pem
INFO[0000] Creating client certificate: /Users/arungupta/.docker/machine/certs/cert.pem
INFO[0001] Downloading boot2docker.iso from file:///Users/arungupta/Downloads/boot2docker.iso...
INFO[0001] Creating SSH key...
INFO[0002] Creating VirtualBox VM...
ERRO[0007] Error creating machine: exit status 1
WARN[0007] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0007] Error creating machine

@arun-gupta can you add --debug flag right after machine command? @ehazlett in the exit code <> 0 cases drivers must print the stderr pipe without --debug, right? I am not sure what's the case but something similar is happening at the other issue regarding azure driver as well (exit status: 1, no message printed, we don't even know which cmd it was)
Maybe we should put the cmd's details next to the "exit status" error.

@arun-gupta If you are sure that machine doesn't exists? If, remove it.

@ahmetalpbalkan yeah unfortunately we do stuff with the output to not show debug -- maybe we can get something to change that on error.

@ehazlett Removed ~/.docker, tried the command. Here is more detail with -D:
INFO[0001] Creating VirtualBox VM...
DEBU[0001] Creating 20000 MB hard disk image...
Converting from raw image file="stdin" to file="/Users/arungupta/.docker/machine/machines/mydocker/disk.vmdk"...
Creating dynamic image with size<PHONE_NUMBER>0 bytes (20000MB)...
DEBU[0007] executing: /usr/bin/VBoxManage createvm --basefolder /Users/arungupta/.docker/machine/machines/mydocker --name mydocker --register
DEBU[0007] STDOUT: Virtual machine 'mydocker' is created and registered.
UUID: 318397cd-2d84-4754-b92c-6f075acd52e3
Settings file: '/Users/arungupta/.docker/machine/machines/mydocker/mydocker/mydocker.vbox'

DEBU[0007] STDERR:
DEBU[0007] executing: /usr/bin/VBoxManage modifyvm mydocker --firmware bios --bioslogofadein off --bioslogofadeout off --natdnshostresolver1 on --bioslogodisplaytime 0 --biosbootmenu disabled --ostype Linux26_64 --cpus 8 --memory 1024 --acpi on --ioapic on --rtcuseutc on --cpuhotplug off --pae on --synthcpu off --hpet on --hwvirtex on --nestedpaging on --largepages on --vtxvpid on --accelerate3d off --boot1 dvd
DEBU[0007] STDOUT:
DEBU[0007] STDERR: VBoxManage: error: The machine 'mydocker' is already locked for a session (or being unlocked)
VBoxManage: error: Details: code VBOX_E_INVALID_OBJECT_STATE (0x80bb0007), component Machine, interface IMachine, callee nsISupports
VBoxManage: error: Context: "LockMachine(a->session, LockType_Write)" at line 471 of file VBoxManageModifyVM.cpp

ERRO[0007] Error creating machine: exit status 1
WARN[0007] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0007] Error creating machine

Yea vbox weirdness. Use a different machine name.

@ahmetalpbalkan Cool, that worked like a charm.
@ehazlett When is this getting integrated in the trunk?

@arun-gupta i'm working on the godeps now -- you said that fixed it?

@ehazlett can we make sure all cmd.Exec's print the output in case of exit status != in all drivers

@ehazlett Specifying a different machine name did fix it, yes!

ah ok thanks!
@ahmetalpbalkan see https://github.com/docker/machine/pull/821 for the godep update / fix


Report package name, script name, cwd during script errors

I found it very difficult to debug lifecycle scripts without informative output. node-gyp scripts are especially opaque when they fail, with no indication of what package they originated from.
This was especially useful when tracking #100 and #101!
Hopefully this is ok, but please let me know if changes would be preferred!

Makes sense. What's currently on master will be deprecated pretty soon. Do you think it would be possible to submit this PR against rebirth?

Ugh, GH doesn't let me change the base of my commits. Closed, will reopen.

:+1:


Revert to older tf

Reverts terraform-ibm-modules/terraform-ibm-client-to-site-vpn#320

/run pipeline

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

GitHub release
v1.7.21

Your semantic-release bot :package::rocket:


feat(clickhouse): include probes for ClickHouse operator and CHI pods

Features

include probes for ClickHouse operator and CHI pods

Related Issue

https://github.com/SigNoz/charts/issues/511