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.

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

Added scaling option to save disk space #297

Merged
merged 3 commits into from
Dec 23, 2020

Conversation

HamOP
Copy link
Contributor

@HamOP HamOP commented Dec 14, 2020

Hi Thomas,

Files of the Pi HQ cam are quite large (about 5+ MB). First, I tried the 2x2 binning option so save some space but that generated quite strange artifacts and especially a hot pixel that is not there when not using binning. You can get rid of it with dark frame subtraction, but only for some time - it seems to move and then leaves a black spot with a new hot pixel next to it in the binned image.

As this effect doesn't occur when using the full resolution (1x1 binning) and I even don't have hot pixels at all in that mode (on-chip dead-pixel correction is switched on), I added an option to scale down the picture before saving it in the daily folder. Reducing the image size by a factor of 2 leaves me with individual images of around 1,3 MB each - and with no artifacts or hot pixels, and no urgent need for darks. Additionally, switching on binning seems to dramatically reduce sensitivity of my HQ cam - not that I understand that ;-)

Maybe you want to add it to your version.

Stefan

This will help save space, especially with the Pi HQ cam. Binning generated moving hot pixels on my cam which were not really removed by dark frame subtraction.
@thomasjacquin thomasjacquin merged commit d554f63 into AllskyTeam:master Dec 23, 2020
@thomasjacquin
Copy link
Collaborator

Thanks Stefan, I just merged your code. It's a nice addition to the software. I have set the feature to false by default.

@HamOP
Copy link
Contributor Author

HamOP commented Dec 26, 2020

You're welcome. It's a really great project and I appreciate all your work. My cam is currently running in test mode and will be installed in it's final place above all street lights and buildings soon - check out https://sternwarte-gartenstadt.de/allsky

A default of false makes sense - I think it's mainly interesting for those PiHQ users among us...

@HamOP HamOP deleted the patch-1 branch December 26, 2020 11:19
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.

None yet

2 participants