Fix for transposed angles in safe-msi reader

[mraspaud]

This PRs fixed the broken angles interpolation in the safe-msi (sentinel 2) reader

• Closes Part of Sentinel-2 images missing when atmospheric corrected #1625 closes 'NoData' area is not black(clean) in the Sentinel-2 MSI output #1727
• Tests added

[codecov]

Codecov Report

Merging #1628 (d3519c3) into main (b415e9b) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1628      +/-   ##
==========================================
+ Coverage   92.58%   92.84%   +0.26%     
==========================================
  Files         258      259       +1     
  Lines       38345    38370      +25     
==========================================
+ Hits        35501    35625     +124     
+ Misses       2844     2745      -99     

Flag	Coverage Δ
behaviourtests	4.83% <0.00%> (-0.01%)	⬇️
unittests	93.39% <100.00%> (+0.26%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
satpy/readers/msi_safe.py	76.33% <100.00%> (+74.08%)	⬆️
satpy/tests/reader_tests/test_msi_safe.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b415e9b...d3519c3. Read the comment docs.

[ghost]

Congratulations 🎉. DeepCode analyzed your code in 3.822 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

👉 The DeepCode service and API will be deprecated in August, 2021. Here is the information how to migrate. Thank you for using DeepCode 🙏 ❤️ !

If you are using our plugins, you might be interested in their successors: Snyk's JetBrains plugin and Snyk's VS Code plugin.

[coveralls]

Coverage increased (+0.3%) to 93.335% when pulling d3519c3 on mraspaud:fix-msi-angles into b415e9b on pytroll:main.

[mraspaud]

@djhoese this is ready for review

[djhoese]

One small comment and just wondering if it is worth it to replace some of the long XML in the tests with python string formatting since it seems a lot of the lines/nans are repeated? Not a big deal. Merge when you feel ready.

[djhoese]

Since you're always telling me...this should probably be made into an issue right?

[mraspaud]

yep thanks, creating issue now

[mraspaud]

As for the xml, I'd rather keep it as close as possible to the original data, for readability.