Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

hrv_satellite_fake should return the requested image_size_pixels, not 3 x image_size_pixels #693

Closed
Tracked by #124
JackKelly opened this issue Jun 28, 2022 · 2 comments · Fixed by #694
Closed
Tracked by #124
Assignees
Labels
bug Something isn't working

Comments

@JackKelly
Copy link
Member

Describe the bug
If you ask hrv_satellite_fake for an image 256 pixels wide by 128 pixels high then it'll return an image of shape 768 wide x 384 high.

Which, presumably, is because of the * 3 on lines 376 & 377 of batch.py:

image

@peterdudfield is it safe to simply remove the * 3 from those lines? 🙂

Yes, the HRV images are higher resolution than the other satellite channels. But the configuration requests the image size in pixels, so, if I've understood correctly, the HRV images should always be exactly the pixel sizes that were requested in the config?

@JackKelly JackKelly added the bug Something isn't working label Jun 28, 2022
@JackKelly JackKelly changed the title hrv_satellite_fake should return the requested image_size_pixels, not 3x image_size_pixels hrv_satellite_fake should return the requested image_size_pixels, not 3 x image_size_pixels Jun 28, 2022
@peterdudfield
Copy link
Contributor

Yea, you can remove the *3. Sorry about that

@JackKelly
Copy link
Member Author

No worries at all! I'll submit a PR in a few mins 🙂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
No open projects
Archived in project
2 participants