Skip to content

code cleanup: remove unneeded subframe check for single_exposure#1405

Merged
agalasso merged 1 commit intomasterfrom
andy/remove-unneed-subframe-check
Dec 31, 2025
Merged

code cleanup: remove unneeded subframe check for single_exposure#1405
agalasso merged 1 commit intomasterfrom
andy/remove-unneed-subframe-check

Conversation

@agalasso
Copy link
Copy Markdown
Contributor

@agalasso agalasso commented Dec 22, 2025

code cleanup: remove unneeded subframe check for single_exposure

Remove an unneeded subframe check in the server get_single_exposure
code path. The subframe is already validated in Capture().

@agalasso agalasso force-pushed the andy/GuideCamera-HasBayer-member-variable branch from b352397 to 20bfb95 Compare December 24, 2025 17:58
@agalasso agalasso force-pushed the andy/remove-unneed-subframe-check branch from c84dd7c to ea0c80b Compare December 24, 2025 17:58
@agalasso agalasso force-pushed the andy/GuideCamera-HasBayer-member-variable branch from 20bfb95 to 6cd49c7 Compare December 25, 2025 02:30
@agalasso agalasso force-pushed the andy/remove-unneed-subframe-check branch from ea0c80b to 8c49751 Compare December 25, 2025 02:30
@agalasso agalasso force-pushed the andy/GuideCamera-HasBayer-member-variable branch from 6cd49c7 to 7c0825d Compare December 31, 2025 22:08
Base automatically changed from andy/GuideCamera-HasBayer-member-variable to master December 31, 2025 22:08
Remove an unneeded subframe check in the server get_single_exposure
code path.  The subframe is already validated in Capture().
@agalasso agalasso force-pushed the andy/remove-unneed-subframe-check branch from 8c49751 to fcfb7d7 Compare December 31, 2025 22:08
@agalasso agalasso merged commit 72d2788 into master Dec 31, 2025
4 checks passed
@agalasso agalasso deleted the andy/remove-unneed-subframe-check branch December 31, 2025 22:08
Eyeke2 pushed a commit to Eyeke2/phd2.planetary that referenced this pull request Mar 6, 2026
…nPHDGuiding#1405)

Remove an unneeded subframe check in the server get_single_exposure
code path.  The subframe is already validated in Capture().
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.

2 participants