Skip to content

Conversation

liamhuber
Copy link
Member

In pyiron/semantikon#266, inheritance of uri= is removed when derived_from=, and the bug of inheriting restrictions= on connections is resolved. Here I make the small adjustments necessary to stay up-to-date with that.

Pending the next semantikon release.

URIs are not inherited, so add them. And the undesirable side effect of inheriting restrictions has been resolved upstream, so adapt/remove those tests.

Signed-off-by: liamhuber <[email protected]>
To explicitly re-state URIs since they are no longer inherited with `derived_from`. And to update expectations around how requirements (don't!) propagate

Signed-off-by: liamhuber <[email protected]>
Signed-off-by: liamhuber <[email protected]>
Signed-off-by: liamhuber <[email protected]>
Signed-off-by: liamhuber <[email protected]>
This release doesn't exist yet, but this commit is compatible with the HEAD of semantikon right now, so let's look forward.

Signed-off-by: liamhuber <[email protected]>
Signed-off-by: liamhuber <[email protected]>
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

Binder 👈 Launch a binder notebook on branch pyiron/pyiron_workflow/update_semantikon

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.

1 participant