Nabil Freij
Nabil Freij
Thanks for the PR @ViciousEagle03 and @ayshih
I decided not to backport this and add a whatsnew as I don't want to break previous behaviour for filenames for the released versions.
That PR was waiting for unit tests to be added. IF you want to help @samish55 finish off the PR, you should contact them first.
~~I pushed a unit test thinking I understood what the problem is but turns out I did not.~~ ~~The test passed before and after the code change.~~ ~~@samaloney, what was...
I wasn't able to replicate the original issue, so I will close this for now.
I think this is now fixed upstream and we do not suggest anaconda install. So I will close this. If you think that's not a fix, we can reopen this.
> This is great, though the amount of code we need to repeat both here and in `GenericMap.plot` is worrying. I don't know of a good way to avoid that...
Will need to verify it works on the JP2 files but hopefully.
I think some unit tests to ensure the files are read correctly would be good.
That base deps failed but the check passed. Will need to fix that.