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.

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

Link AIMS sample #123

Merged
merged 14 commits into from
Nov 2, 2023
Merged

Link AIMS sample #123

merged 14 commits into from
Nov 2, 2023

Conversation

moeyensj
Copy link
Owner

@moeyensj moeyensj commented Oct 27, 2023

Changes to link the AIMS sample include:

  • parallelizing TestOrbit.range_observations
  • parallelizing range_and_transform
  • parallelizing TestOrbitRadiusObservationFilter
  • set time linking precision to ms in GnomonicCoordinates.from_cartesian
  • add a new Config dataclass

@moeyensj moeyensj force-pushed the v2.0-link-aims-sample branch 3 times, most recently from 7cd2c6b to 7cbb913 Compare October 27, 2023 19:27
@moeyensj moeyensj marked this pull request as ready for review October 31, 2023 14:47
@moeyensj moeyensj force-pushed the v2.0-link-aims-sample branch 3 times, most recently from b287675 to 3d74782 Compare October 31, 2023 21:53
@moeyensj
Copy link
Owner Author

moeyensj commented Nov 2, 2023

@akoumjian I've requested a review from you but don't feel like you need to through everything. I'm happy with this branch as a starting point from which we can later make edits in other PRs. I'd like to get this into main so we don't have too many branches all pointing to this one.

@moeyensj moeyensj merged commit bdda0a0 into main Nov 2, 2023
5 checks passed
@moeyensj moeyensj deleted the v2.0-link-aims-sample branch November 2, 2023 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants