Skip to content

Fix flipped outputs from resample#1046

Merged
rafaqz merged 4 commits intorafaqz:mainfrom
tiemvanderdeure:reorder_resample
Nov 23, 2025
Merged

Fix flipped outputs from resample#1046
rafaqz merged 4 commits intorafaqz:mainfrom
tiemvanderdeure:reorder_resample

Conversation

@tiemvanderdeure
Copy link
Collaborator

closes #1045

I figured out this occured when resampling to dimensions that have different order than the dimensions of the raster that was being resampled. A call to reorder should fix that.

@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.72%. Comparing base (f1c4b75) to head (0e372d5).
⚠️ Report is 51 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1046      +/-   ##
==========================================
+ Coverage   82.89%   83.72%   +0.83%     
==========================================
  Files          58       68      +10     
  Lines        5425     5481      +56     
==========================================
+ Hits         4497     4589      +92     
+ Misses        928      892      -36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tiemvanderdeure
Copy link
Collaborator Author

Not sure what is going on with all those docstrings... It reports there are a bunch of downloading messages in them that aren't supposed to be there

@asinghvi17
Copy link
Collaborator

I think that might be because I disabled something. Maybe some redirect_stdout or so. But not sure.

@lazarusA
Copy link
Collaborator

yes, please always set these ones back before merge into main:

# Logging.disable_logging(Logging.Warn)

# Logging.disable_logging(Logging.BelowMinLevel)

@rafaqz rafaqz merged commit 5abf1eb into rafaqz:main Nov 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

resample flips the data upside down

4 participants