Closed TomSlater closed 3 years ago
Merging #72 (58feb90) into master (0ac5f09) will not change coverage. The diff coverage is
94.16%
.
@@ Coverage Diff @@
## master #72 +/- ##
=======================================
Coverage 88.49% 88.49%
=======================================
Files 15 15
Lines 1156 1156
=======================================
Hits 1023 1023
Misses 133 133
Impacted Files | Coverage Δ | |
---|---|---|
particlespy/custom_kernels.py | 100.00% <ø> (ø) |
|
particlespy/particle_save.py | 20.00% <0.00%> (ø) |
|
particlespy/segptcls.py | 94.56% <ø> (ø) |
|
particlespy/tests/generate_test_data.py | 100.00% <ø> (ø) |
|
particlespy/radial_profile.py | 9.09% <33.33%> (ø) |
|
particlespy/segimgs.py | 91.57% <83.33%> (ø) |
|
particlespy/particle_analysis.py | 94.22% <92.85%> (ø) |
|
particlespy/__init__.py | 100.00% <100.00%> (ø) |
|
particlespy/api.py | 91.17% <100.00%> (ø) |
|
particlespy/particle_load.py | 100.00% <100.00%> (ø) |
|
... and 6 more |
@CameronGBell, this PR is passing tests and I've run a new version of the example notebook with seg_ui and all works for me. Would you mind just checking a local install with you to see if this is all running okay? I'm just cautious that a lot has changed!
This PR should resolve #69 once merged.
I have updated all directories and functions to use solely lower case to avoid confusion / errors. All functions should now be in the same style, e.g. this_function().
All docs and tests have been updated.
I will remove the example notebook and update and put this somewhere else.