Remove internal uses of deprecated Orbit.from_body_ephem
See original GitHub issueDoing a quick search, the only place where we are using Orbit.from_body_ephem is this one:
Instead, we should instead be doing this:
ephem = Ephem.from_body(body, J2000_TDB)
orbit = Orbit.from_ephem(Sun, ephem, J2000_TDB)
And, after that, tests covering Orbit.from_body_ephem can be removed.
Issue Analytics
- State:
- Created 3 years ago
- Comments:8 (8 by maintainers)
Top Results From Across the Web
Remove deprecated Orbit.from_body_ephem and its tests
Usage Workflow file. Triggered via pull request 2 years ago. @ishanSrt ishanSrt. synchronize #1110 · ishanSrt:remove-deprecated-Orbit.from_body_ephem.
Read more >1 Enhanced Deprecation - Java
A newer, better API can be used instead. The deprecated API is going to be removed. In prior releases, APIs were deprecated but...
Read more >When are API methods marked "deprecated" actually going ...
It is recommended that programs be modified to eliminate the use of deprecated APIs, though there are no current plans to remove such...
Read more >Tempo for Server Code Deprecation and Removal
Deprecated API Path / Location Public? Use Case
POST worklog attributes /rest/tempo‑rest/2.0/worklogs//attributes No Update work attrib...
GET worklog attributes /rest/tempo‑core/1/work‑attribute/value Yes Get all work attrib...
POST...
Read more >How long to wait before deleting a deprecated method?
Major version releases are a good time to remove deprecated methods. ... I worked for a company developing software for internal use.
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

Good catch @Yash-10 !
@ishanSrt The function has already been deprecated, there are no more changes needed to its function definition and docs.