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

capture_ZWO changes to gather more info #2667

Merged
merged 4 commits into from
May 22, 2023

Conversation

EricClaeys
Copy link
Collaborator

This fixes a likely bug that can cause "ping pong" messages.
It also provides more log information to help debug a problem Jim has where at night, the MEAN brightness goes near 255 and stays there.

* Output lastMeanFull as FULLMEAN to aid in determining if this mean is the same as what removeBadImages.sh produces.  This code will 
* Allow length_in_units() to be called up to 3 times on one line.
* Return immediately in length_in_units() if "us == 0" for readability.  This caused a lot of indentation changes.
* (Potential) bug fix: don't use "debug_text" in a Log() statement since it also uses "debug_text".  I think doing that may have been overwritting memory and causing "ping pong" messages.
* Use length_in_units() multiple times in one Log() call to simplify code (needs to be done elsewhere).
* Change debug level of some Log() statements so level 3 displays exposure-related messages.
* Change computeHistogram() to optionally produce a MEAN of the whole image, not just the histogram box.
* Some minor indentation changes.
@EricClaeys EricClaeys merged commit 7a4adea into dev May 22, 2023
1 check passed
@EricClaeys EricClaeys deleted the capture_ZWO-changes-to-gather-more-info branch May 22, 2023 12:02
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

1 participant