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

SVBONY: Fixed some bugs and added workaround code for SVBONY Camera SDK #849

Merged
merged 8 commits into from
Oct 19, 2023

Conversation

jctk
Copy link
Contributor

@jctk jctk commented Oct 15, 2023

Fixed some bugs and added workaround code for SVBONY Camera SDK

  • Save default value to prevent crash(Merge with d1622b9)
  • Changed the SDK to set the ROI only if it is different from the current ROI when setting the ROI.
  • Added code to call SVBSetCameraMode function in SVBONYBase::Connect(). This is a workaround code for a bug in the SDK.
  • Fixed parameter for setting target cooling temperature in SVB_TARGET_TEMPERATURE.
  • Modified to set Bayer pattern to INDI control panel in SVBONYBase::Connect changes

SVBONY re-implementing monochrome camera

  • Related modifications have been made but not yet tested with a monochrome camera (testing will be conducted after the actual equipment is obtained).

jctk added 5 commits October 1, 2023 21:46
- SVBONYBase::grabImage disassembled and embedded in SVBONYBase::workerExposure
- Remove unused SVBONYBase::grabImage function

svbony_helpers.h
- Added processing in preparation for monochrome camera support (monochrome support not yet completed)
- Changed Luma to Luma 8 bit and Luma 16 bit
- Added DEBUGLOG output for camera attributes.
- Modified to set Bayer pattern to INDI control panel.

Modification of SVBONYBase::setupParam
- modified to consider the number of channels when allocating buffers

Improvement of SVBONYBase::sendImage
- Modified to use Helper function to determine whether Bayer is present or not.
- Changed the SDK to set the ROI only if it is different from the current ROI when setting the ROI
- Added code to call SVBSetCameraMode function in SVBONYBase::Connect(). This is a workaround code for a bug in the SDK.
- Fixed parameter for setting target cooling temperature in SVB_TARGET_TEMPERATURE
@jctk
Copy link
Contributor Author

jctk commented Oct 15, 2023

Hi @knro
Please adopt my PR for all conflicts.
I do not have the permission to "Resolve conflicts".

@knro
Copy link
Collaborator

knro commented Oct 15, 2023 via email

@jctk
Copy link
Contributor Author

jctk commented Oct 15, 2023

Okay. I'll try to rebase.

jctk added 3 commits October 15, 2023 22:54
- Changed the SDK to set the ROI only if it is different from the current ROI when setting the ROI
- Added code to call SVBSetCameraMode function in SVBONYBase::Connect(). This is a workaround code for a bug in the SDK.
- Fixed parameter for setting target cooling temperature in SVB_TARGET_TEMPERATURE
@jctk
Copy link
Contributor Author

jctk commented Oct 15, 2023

I've rebased this branch.

@knro knro merged commit 350e5a5 into indilib:master Oct 19, 2023
4 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.

2 participants