-
Notifications
You must be signed in to change notification settings - Fork 11
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
Low code coverage #10
Comments
The code coverage is up to 72% if we include the tutorial. Of course this is not an automated unit test and there is room for improvement. For the purpose of this review (openjournals/joss-reviews#3418) this is satisfactory. You could pursue this further in the future @JohnVolk. |
Note: Bokeh causes outputs to change, so I used the
|
Thanks for the info @ashwinvis I wasn't aware of this plugin and the options, it is really useful. I was also having issues with the --cov option on the notebooks. Yes, I plan on flushing out the automated tests. |
I used the plugins |
If I run
I notice that the tests mostly target only the
data
module. If the tutorials cover the rest then it should be good enough and one could check the coverage as follows.However, in order to do that, #9 should be fixed first.
The text was updated successfully, but these errors were encountered: