Steve Morley
Steve Morley
I know it's a losing argument, but I do not support the 80 (_actually 79 per PEP8_) character limit. I'll suck it up and follow it when I get outvoted,...
As a side note, the text boxes in the issue tracker here seem to wrap at 140.
Final comment for now, which is just a "current state of play" on how many lines are in each file and what the maximum line length in each file is...
Thanks! I've approved the CI run for this, I'll close the old PR (#575) in a minute, and I've edited the PR text to include the checklist from our PR...
That was merged already (#271)... but yes. Although I believe that PR also added some testing (not much, but some).
Two quick comments: 1. On the issue of loading the CDF library once in each thread to have independent states... This used to be extremely problematic, due to CDF's use...
@balarsen See my note about updates of the CDF library to include the process ID. I believe that separate processes should be fine. Technically there's a small chance of clashes...
This is something we're looking towards, although it's not on our projects boards for a near-term goal yet. Making your option 2 work out of the box is most likely...
Thanks for the update @seanhaley: As you've noticed, this is only an issue on older dependency versions. Because the exception is a `SystemError` it's either a Python bug or a...
The "OMNI2" is taken directly from NASA's OMNI2 data set (the 2 distinguishes it from the ~70s era OMNI product). The "OMNI" data set used by SpacePy is a derived...