Kuanchou/681 relative coordinates

[loveluthien]

Description

This PR closes #681 adding an offset coordinates. The offset coordinates consider the projection. We can turn on the offset coordinates using overlay coordinates in the toolbar or the image view setting. The default offset center is the image center and can be customized by setting it to the view center or a manual input. The unit of delta angles changes with the field of view size.

Due to the offset mapping using AST which changes the projection center, the distance measurement went wrong when applying the offset coordinates. Therefore, we replace the distance measurement with the ruler annotation which is not affected by the overlay coordinates.

Checklist

• coordinates show delta angles.
• UI: toolbar and image view setting panel.
• distance measurement is replaced with ruler annotation.

For linked issues (if there are):

• assignee and label added
• GitHub Project estimate added

For the pull request:

• reviewers and assignee added
• GitHub Project estimate added
• changelog updated / no changelog update needed
• unit test added (for functions with no dependenies)
• API documentation added (for public variables and methods in stores)

For dependencies:

• e2e test passing / corresponding fix added / new e2e test created
• protobuf version bumped / no protobuf version bumped needed
• protobuf updated to the latest dev commit / no protobuf update needed
• corresponding ICD test fix added (BackendService changed) / no ICD test fix needed (BackendService unchanged)
• user manual prepared (for large new features)

[loveluthien]

@kswang1029 @crocka There is a discussion item. The distance measurement fails when using a relative coordinate, more precisely after applying an AST shifted map. The easiest way is to replace the distance measurement with the ruler annotation. @YuHsuan-Hwang has some concerns about doing that. What's your opinion?

[kswang1029]

Do you mean we deprecate the distance measurement tool and ask users to use the rule annotation instead? If so, I am fine with the idea which actually enables multiple measurements and users can see all the results at the same time.

[loveluthien]

Yes. I feel the ruler is more useful than the distance measurement.

[YuHsuan-Hwang]

I'm ok with deprecating the distance measuring tool.

[kswang1029]

looks very nice! A corresponding fix is applied to an e2e test branch. New tests will be added asap.

[YuHsuan-Hwang]

The implementation looks nice. The removal of distance measuring is also good.
Minor comments only:

[YuHsuan-Hwang]

Looks like function plotDistText could be removed.

[loveluthien]

This might old version. I had removed entire if (showCurve) section.

[YuHsuan-Hwang]

Sorry I meant the function plotDistText itself in this file could be removed.

[loveluthien]

okay, I got it.

[kswang1029]

UPDATE: Added a corresponding e2e test which checks AST grid rendering including the offset cases.