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

Smooth warming for ASI CCD #700

Closed
wants to merge 11 commits into from
Closed

Smooth warming for ASI CCD #700

wants to merge 11 commits into from

Conversation

pmneo
Copy link
Contributor

@pmneo pmneo commented Jan 8, 2023

Because of the silicon oil issue of the asi 2600 cameras, I have implemented a smooth warming procedure, if cooler is switched off.

In this case the temperature is stepped in 2 degree up (has to settle for at least aprox. 5 seconds) until the temp does not reach the next 2 deg step within 5 minutes. This indicates the end of the warming procedure.

@knro
Copy link
Collaborator

knro commented Jan 9, 2023

I believe this should be in INDI::CCD layer and not just limited to ZWO. There is already a gradual cooling functionality in the CCD (degrees per minute). Maybe on setting a high temperature, it should gradually warm until reaching the ambient temperature?

@pmneo
Copy link
Contributor Author

pmneo commented Jan 9, 2023

Yes, I have seen that function of ramping too. But to implement it clean like this, the indicdd should define the enableCooler switches and delegation to enableCooler like SetTemperature ...

@knro knro added this to the 2.0.1 milestone Jan 31, 2023
@knro
Copy link
Collaborator

knro commented Mar 29, 2023

Any chance you can re-work this for INDI::CCD? We can't just have it for this camera.

@luzpaz
Copy link

luzpaz commented Dec 17, 2023

bumping

@pmneo pmneo deleted the branch indilib:master February 24, 2024 18:17
@pmneo pmneo closed this Feb 24, 2024
@pmneo pmneo deleted the master branch February 24, 2024 18:17
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.

4 participants