ilumsden
commented
10 months ago 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.
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.