Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Reference API #225

Closed
3 tasks done
5A11 opened this issue Aug 2, 2022 · 3 comments
Closed
3 tasks done

Add Reference API #225

5A11 opened this issue Aug 2, 2022 · 3 comments
Assignees
@VijaySharma815

Comments

@5A11
Copy link
Member

5A11 commented Aug 2, 2022

Prerequisites

  • I checked the documentation and made sure this feature does not already exist.
  • I checked the existing issues to make sure this feature has not already been requested.
  • I have read the code of conduct before creating this issue.

Problem

There is currently no reference API in ComsPy's documentation.

Feature / Solution

Add reference API in CosmPy's documentation, maybe starting with what's in the Aerial package and then doing something clever on the low-level APIs as well to avoid having a useless protobuf functions.

Alternatives

No response

Additional Context

No response
@5A11 5A11 mentioned this issue Aug 2, 2022
Closed
6 tasks
@5A11
Copy link
Member Author

5A11 commented Aug 2, 2022

Can be started after #126 for Aerial package is completed.

@VijaySharma815
Copy link
Contributor

PR: #247

@5A11
Copy link
Member Author

5A11 commented Aug 24, 2022

The PR is now merged.

@5A11 5A11 closed this as completed Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VijaySharma815 VijaySharma815

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@5A11 @VijaySharma815