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

Add safepower node to data libraries #1689

Merged

Conversation

ld-kerley
Copy link
Contributor

Add node definition and nodegraph for safepower which is already in the v1.39 specification, but currently missing in the source code.

As this is an additive change we can safely add this to main for inclusion in any v.138 release.

ld-kerley and others added 4 commits February 2, 2024 12:03
Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
Copy link
Member

@jstone-lucasfilm jstone-lucasfilm left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks @ld-kerley!

@jstone-lucasfilm jstone-lucasfilm changed the title Add safepower node Add safepower node to data libraries Feb 4, 2024
@jstone-lucasfilm jstone-lucasfilm merged commit d8f4af3 into AcademySoftwareFoundation:main Feb 4, 2024
31 checks passed
@ld-kerley ld-kerley deleted the add_safepower_node branch May 21, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants