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

Use ABC for abstract base classes #27

Closed
gefux opened this issue Nov 16, 2021 · 1 comment
Closed

Use ABC for abstract base classes #27

gefux opened this issue Nov 16, 2021 · 1 comment
Assignees
Labels
tidy up Something needs to be tidied up
Milestone

Comments

@gefux
Copy link
Member

gefux commented Nov 16, 2021

The BaseAPIClass, BaseProcessTensor, BaseSystem, BaseCorrelations, should probably be inherited from ABC.

@gefux gefux added tidy up Something needs to be tidied up someday Maybe something to work on someday labels Nov 16, 2021
@gefux gefux removed the someday Maybe something to work on someday label Jan 27, 2024
@gefux gefux self-assigned this Jan 27, 2024
@gefux gefux added this to the version 0.5 milestone Jan 27, 2024
@gefux
Copy link
Member Author

gefux commented Jun 19, 2024

This seems currently only really necessary for BaseProcessTensor and has been implemented with commit 73e8259.

@gefux gefux closed this as completed Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tidy up Something needs to be tidied up
Projects
None yet
Development

No branches or pull requests

1 participant