Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add weighted blended stacking to MultiScene (fixes multi-band handling) #2394

Merged
merged 25 commits into from Apr 17, 2023
+489 −303

Consolidate some multiscene blend tests

0210ae4
Select commit
Failed to load commit list.
Merged

Add weighted blended stacking to MultiScene (fixes multi-band handling) #2394

Consolidate some multiscene blend tests
0210ae4
Select commit
Failed to load commit list.
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main) failed Apr 6, 2023 in 42s

CodeScene PR Check

  • View detailed results in CodeScene
  • Quality Gates: FAILED
  • Recommended Review Level: Detailed -- Inspect the code that degrades in code health.

Details

Warnings

  • Degrades in Code Health
    • satpy/satpy/multiscene.py degrades from a Code Health of 9.23 -> 8.64

Annotations

Check warning on line 1 in satpy/multiscene.py

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Lines of Code in a Single File 

This module has 634 lines of code, improve code health by reducing it to 600

Check warning on line 192 in satpy/tests/multiscene_tests/test_blend.py

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Excess Number of Function Arguments 

TestBlendFuncs.test_blend_two_scenes_using_stack_weighted has 5 arguments, threshold = 4
View more details on CodeScene Delta Analysis