Beta.9 Home Azimuth units issue

[George Silvis]

In the setup panel the tool tip calls for an azimuth in degrees. I put in 305 and the dome reset to 2 after finding the home sensor…. Tried 153 and got 1 degree. It is treating the input as Sed units.

Server and firmware all at Beta.9

minor- Tool tip for Park Azimuth is the same as Home Azimuth, mentioning Home by name.

wish list:

• After a home find operation, when the home azimuth gets reset, report in the log the event and what the dome azimuth was at the moment before reset. This will give us a handle on the issue of gear skipping.
• Terminal window to the rotator from the setup server? Just a thought, but probably too difficult.

[George Silvis]

I believe the problem lies here:

In RxControllerActions:
public void SetHomeSensorAzimuth(decimal azimuth) {
var command = string.Format(Constants.CmdSetHomeSensorAzimuthTemplate, (int)azimuth);
SendCommand(command);
}

But: in Constants.cs:
public const string CmdSetHomeSensorAzimuthTemplate = "@HWR,{0:000}";

And per the pdf:
HW R ddddd :HWR# @HWR,1000
Home position Write (seps clockwise from true
north)

[Tim Long]

@George Silvis , I believe you may be spot on in your analysis. I will attend to that now, and thank you for digging into the code!

Regarding your wish list, I don’t think I can do either of those because:

1. This is done within the firmware, not the driver. There’s not much code space left and I want to avoid messing with the firmware as much as possible at this stage.
2. Yes, too difficult

If you want to propose wish list items, it would be preferable to create a separate issue for each one, setting the type to “Proposal” or “Enhancement”.

[George Silvis]

Is the repo for the source for the firmwares available? My background is embedded software.
Thx,
George

[Tim Long]

Fix issue #40 by converting azimuth to steps before sending to hardware.

→ <<cset 2b38ce2e1fe0>>

[Tim Long]

Yes it is. See https://bitbucket.org/account/user/tigra-astronomy/projects/NEX

Best regards, Tim Long

[Tim Long]

Merged in bugfix/iss#40-home-azimuth-units (pull request #8)

Fix issue #40 home azimuth units

Approved-by: George Silvis [e-mail]

→ <<cset e35826031831>>