Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Orekit Orekit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 90
    • Issues 90
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 16
    • Merge requests 16
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Orekit
  • OrekitOrekit
  • Merge requests
  • !8
The source project of this merge request has been removed.

Issue 518

Merged Petrus Hyvönen requested to merge (removed):issue-518 into develop May 17, 2019
  • Overview 3
  • Changes 1

Change of AbstractGNSSAttitudeProvider from public to non-public.

Petrus Hyvönen petrus.hyvonen@gmail.com a écrit :

Hi Luc,

Hi Petrus,

I'm mailing you directly, let me know if you prefer to take such discussions through the forum.

I'm working on creating python extension classes more systematic for all the interfaces and Abstract classes (where it is not stated that they are for internal use, or otherwise low sense).

I ran into some issue with the AbstractGNSSAttitudeProvider. The correctedYaw(GNSSAttitudeContext context); uses the GNSSAttitudeContext which is non-public and only visible within the attitude package. (python extension classes go to org.orekit.python) Is the idea that this abstract class is not intended for common use, or should the GNSSAttitudeContext be public maybe?

Good catch! This abstract class is not intended for public use, only the
GNSSAttitudeProvider interface should be public.

Could you open an issue on the gitlab forge to ask for changing visibility to package private? We cannot fix it for 9.3 since it was already
published in 9.2, so we have to wait until 10.0. I doubt anybody implemented another
implementation, but prefer to be safe.

best regards, Luc

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: issue-518