Skip to content

Add new OPAL-RT Asynchronous Process node-type#924

Merged
stv0g merged 2 commits intomasterfrom
node-opal-async
Nov 6, 2025
Merged

Add new OPAL-RT Asynchronous Process node-type#924
stv0g merged 2 commits intomasterfrom
node-opal-async

Conversation

@stv0g
Copy link
Contributor

@stv0g stv0g commented Jun 12, 2025

This PR adds a new node-type for interfacing VILLASnode as an Asynchronous Process with OPAL-RT's RT-LAB software platform.

See:

@stv0g stv0g requested review from al3xa23 and fwege June 12, 2025 11:12
@stv0g stv0g requested a review from n-eiling as a code owner June 12, 2025 11:12
@stv0g stv0g self-assigned this Jun 12, 2025
@stv0g stv0g added enhancement New feature or request node::opal labels Jun 12, 2025
@stv0g stv0g force-pushed the node-opal-async branch from 01586b2 to e331b4e Compare June 12, 2025 11:52
@stv0g stv0g force-pushed the node-opal-async branch 2 times, most recently from 92c2295 to cc0f7d3 Compare August 18, 2025 13:20
@stv0g
Copy link
Contributor Author

stv0g commented Aug 18, 2025

Ping @al3xa23 @n-eiling

@n-eiling
Copy link
Member

I didn't review, because the CI is failing.

n-eiling
n-eiling previously approved these changes Aug 20, 2025
Copy link
Member

@n-eiling n-eiling left a comment

Choose a reason for hiding this comment

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

LGTM, some minor comments

@stv0g
Copy link
Contributor Author

stv0g commented Aug 21, 2025

Blocked by #952

al3xa23
al3xa23 previously approved these changes Sep 2, 2025
stv0g added 2 commits November 6, 2025 09:32
… Asynchronous process in OPAL-RT's RT-LAB

Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
@stv0g stv0g requested a review from windrad6 as a code owner November 6, 2025 08:33
Copy link
Member

@n-eiling n-eiling left a comment

Choose a reason for hiding this comment

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

looks good.
Nitpick: I think ending all comment with a "." is a bit unnecessary, espc. since most of your comments aren't full sentences.

@stv0g stv0g merged commit e7c6153 into master Nov 6, 2025
3 checks passed
@stv0g stv0g deleted the node-opal-async branch November 6, 2025 08:51
@stv0g
Copy link
Contributor Author

stv0g commented Nov 6, 2025

@n-eiling Thanks for the feedback :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request node::opal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants