Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for topologies without transformers and slack grids with multiple nodes #1104

Merged

Conversation

staudtMarius
Copy link
Member

Resolves #1099

@staudtMarius staudtMarius added the enhancement New feature or request label Jan 13, 2025
@staudtMarius staudtMarius self-assigned this Jan 13, 2025
Copy link
Member

@danielfeismann danielfeismann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor thing. And please remove the changes from #1098

@danielfeismann
Copy link
Member

Fine for me so far, but please rollback the changes from #1098 since I guess they're obsolte now, or?

@staudtMarius
Copy link
Member Author

staudtMarius commented Jan 23, 2025

Fine for me so far, but please rollback the changes from #1098 since I guess they're obsolte now, or?

Correct me, if I am wrong, but I think the changes are rolled back completely.

@danielfeismann
Copy link
Member

Fine for me so far, but please rollback the changes from #1098 since I guess they're obsolte now, or?

Correct me, if I am wrong, but I think the changes are rolled back completely.

Sorry, I overlooked that

Copy link
Member

@danielfeismann danielfeismann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I hope not to be too pedandic :) But I would like to avoid future misunderstandings here

Copy link
Member

@danielfeismann danielfeismann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for taking care of my points.

@danielfeismann danielfeismann merged commit 4036f4b into dev Jan 28, 2025
4 checks passed
@danielfeismann danielfeismann deleted the ms/#1099-allow-for-topologies-without-transformers branch January 28, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow for topologie where the slack node is not directly connected to a transformer
2 participants