nexdome / Firmware

NexDome Dome and Shutter Motor Kit Firmware
https://www.nexdome.com
Other
3 stars 6 forks source link

Incorrect values returned for `@RRS` #31

Closed NameOfTheDragon closed 3 years ago

NameOfTheDragon commented 4 years ago

Testing of pull request #30 has revealed a puzzling issue as follows:

@FRS
FRS3.4.0-auto-close.5
@ZDS
ZDS
@RRS
RRS46000
@RWS,20000
RWS
@RRS
RRS200009

The final reading of @RRS returns the wrong value, always some extra data appended to the end.

Some thoughts on this:

Conclusion: something is happening to the value between receiving it as a command and the value being written into the configuration structure.

This needs to be traced in detail to determine what's going wrong.

NameOfTheDragon commented 4 years ago

After reverting to factory defaults and resetting the target system, the problem can no longer be reproduced :(

NameOfTheDragon commented 3 years ago

Fixed by PR#30