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

Add one more check to ensure minimum count is met #1903

Merged
merged 1 commit into from Oct 2, 2023

Conversation

knro
Copy link
Contributor

@knro knro commented May 30, 2023

When sending commands, sometimes we receive response from another command in the buffer despite flushing the buffer. So in case the response does not meet the minimum count then we retry.

@knro knro added this to the 2.0.3 milestone Jun 1, 2023
@JulesArcher
Copy link

Hi @knro I am also having the same error message when using Wifi connection on my CEM26 as reported earlier:

[2023-09-25T18:01:01.845 PDT INFO ][ org.kde.kstars.indi] - iOptron CEM26 : "[ERROR] Error setting RA/DEC. "
[2023-09-25T18:01:46.918 PDT INFO ][ org.kde.kstars.indi] - iOptron CEM26 : "[ERROR] bool IOPv3::Driver::getCoords(double*, double*, IOPv3::IOP_PIER_STATE*, IOPv3::IOP_CW_STATE*): Expected 20 bytes but received 23. "

Is there a modified driver with the fix that I could test on my system?

Thanks!

@knro knro marked this pull request as ready for review October 2, 2023 09:17
@knro
Copy link
Contributor Author

knro commented Oct 2, 2023

Can you please check both wired and wireless connections?

@JulesArcher
Copy link

JulesArcher commented Oct 2, 2023 via email

@knro knro merged commit 769623a into indilib:master Oct 2, 2023
11 checks passed
@knro knro deleted the ioptron_fix branch October 2, 2023 15:55
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.

None yet

2 participants