-
Notifications
You must be signed in to change notification settings - Fork 2
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
Is CoverCalibrator check correct? #12
Comments
Hi Carlos,
Are you using the browser based Conform Universal or the original WinForms Conform, it’s difficult to tell from the log fragment you posted? Assuming you are using ConformU, please can you:
* Enable Display method calls and Debug in ConformU settings as shown below.
* Reproduce the issue and post the whole log here. The log will be in your Documents\ASCOM\Logs 2024-04-XX folder where xx is the day number on which you run the test.
With the full log I should be able to determine where the issue originates.
Many thanks, Peter
|
Hi Carlos, OK, I can see you are using the original Conform program. I've found the bug, which is that when Conform detects an issue, it gives an incorrect error message. Your device returned a CalibratorState value other than CalibratorStatus.Ready in response to the CalibratorOn() call with a brightness of 0, so I think there is an issue with your device. However, instead of telling you what CalibratorState value it received, Conform is incorrectly reporting the CoverState value. This issue has been fixed in Conform Universal. Best wishes, Peter |
Hi Peter,
Thank you for your help. Turns out I missed the scenarios when the calibrator in On with 0 intensity.
Works perfect now.
Kind regards
Carlos Gil
On Sunday, 28 April 2024 at 22:13:52 BST, Peter Simpson ***@***.***> wrote:
Hi Carlos,
OK, I can see you are using the original Conform program. I've found the bug, which is that when Conform detects an issue, it gives an incorrect error message.
Your device returned a CalibratorState value other than CalibratorStatus.Ready in response to the CalibratorOn() call with a brightness of 0, so I think there is an issue with your device.
However, instead of telling you what CalibratorState value it received, Conform is incorrectly reporting the CoverState value.
This issue has been fixed in Conform Universal.
Best wishes, Peter
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Hi Carlos, I'm glad you found the issue. I'll fix the Conform error in the next release, please can you leave the issue open and I'll close it when the fix is developed. Best wishes, Peter |
Hi,
I'm working on my own CoverCalibrator, and so far so good: everything moves, driver seems to works nice in N.I.N.A., etc.
But when I run the Conform Check tool i get this:
The offending line does not make a lot of sense to me:
18:15:38.044 CalibratorOn ERROR CalibratorOn with brightness 0 was unsuccessful - the returned CoverState was 'Unknown' instead of 'Ready'. The asynchronous operation took 0.1 seconds
I can imagine that the cover being closed might be a requisite to switch the calibrator on (with 0 or non 0 values) but, as per the standard,
Ready
is not in the enum forCoverState
value. TheUnknown
comes from the previousHaltCover
check that leaves the cover neither open nor closed.Am I missing something, or is there a bug here?
Kind regards,
Carlos
The text was updated successfully, but these errors were encountered: