Nabil Freij

Results 806 comments of Nabil Freij

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.