Skip to content

Upcoming breaking change: target should be an ID, not a frame #754

Description

@ChristopherRabotin

High level description

While each kernel defines the segments with their associated frame definitions, requiring ANISE users to specify the full frame definition in the target parameter for functions like transform, translate, StateSpec, etc. is confusing to newcomers.

Users shouldn't care about the segment definition in their query. Instead, they should only provide it for the observer, similarly to SIPCE's spkezr.

Requirements

All functions that require a target frame and an observer frame should switch to requiring a target NaifId and an observer frame. The Almanac should use center_frame_uid to initialize the initial frame in translate_parts_to_parent, instead of specifying a source frame.

Test plans

The current test suite should not change the test cases: the code should be updated to the new signature.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions