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

Include python interface in the CI testing #53

Open
JaeseungYeom opened this issue Oct 27, 2023 · 1 comment
Open

Include python interface in the CI testing #53

JaeseungYeom opened this issue Oct 27, 2023 · 1 comment

Comments

@JaeseungYeom
Copy link
Contributor

JaeseungYeom commented Oct 27, 2023

Currently, the CI relies on the tutorial codes written for old ECP tutorial material.
I believe we labeled it with the date of demonstration such that when it becomes obsolete we are aware of that.
We should maintain the tutorial such that it is up to date, which seems to be already happening fortunately. That means converting the tutorial directory name to a generic one without date tag.
Finally, we should create python interface tests.

@ilumsden
Copy link
Collaborator

I absolutely agree with this. One this I wanted to add is that #27 adds some initial testing of the Python API. It's not true unit testing (i.e., testing every function to ensure it does what is expected), but it will allow us to make sure that the Python API continues to work for a simple workflow.

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

No branches or pull requests

2 participants