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

Fix broken humidity reading after reconnect of device #933

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

peter-englmaier
Copy link
Contributor

I noticed, that my cloudwatcher was showing mutliple humidity values in the parameters tab. All are zero and kstars observatory tab was not showing humidity value. This happens, when kstars disconnects and reconnects the driver. Each time the handshake method is called, adding another parameter.

This fix simply checks if the parameter already exists before adding it.

@knro
Copy link
Collaborator

knro commented Jun 16, 2024

Thanks, it appears there is also another issue with latest firmware and the driver. Which firmware are you running? also do you have SQM?

@peter-englmaier
Copy link
Contributor Author

Yes, it has the "SQM" feature, but I have not yet tried everything. What is the other issue?

The device was just delivered with firmware 5.89 and I have not changed it. I am having the version with RH/Pressure sensors but without the wind sensor.

@knro
Copy link
Collaborator

knro commented Jun 17, 2024

is the SQM pluggable? if you unplug it and run the driver what happens?

@peter-englmaier
Copy link
Contributor Author

No, the cloudwatcher is a small white box with only two water proof connectors:

https://lunaticoastro.com/aag-cloud-watcher/
https://lunaticoastro.com/wp-content/uploads/cw-RAIN-NEW-min.png

One is for attaching the serial port, power supply, and two wires to signal the safe/unsafe switch (all with one cable). The other is for the optional wind sensor. The SQM is buildin, and I suppose it is the small round window seen at the bottom of the image. I had no time to calibrate it so far, so no idea if it works as expected.

There is another model, the "CW Hydreon version", with a different kind of rain sensor, which has 3 connectors, and the additional connector is only for that rain sensor. And, there is a "mobile" version. But I have the normal type.

My PR is for the humidity sensor, which is also buildin. It sticks out from the bottom as a black knob (maybe this is also for ambient temperature). Here you can see it:

https://lunaticoastro.com/wp-content/uploads/CloudWatcher-Relative-humidity-and-Atmospheric-pressure-sensor-v-2-02-min.jpg

If there is another bug, I can probably fix it, if I can reproduce it, but I am not aware of other problems (have the device only one week).

@knro knro merged commit 440a6bf into indilib:master Jun 18, 2024
2 of 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