Vectorize mpc.mpc_orbit, mpc.comet_orbit#532
Vectorize mpc.mpc_orbit, mpc.comet_orbit#532JoshPaterson wants to merge 3 commits intoskyfielders:masterfrom
mpc.mpc_orbit, mpc.comet_orbit#532Conversation
|
I see that this and #526 got stuck - is there a way I could help them go through? |
|
@brandon-rhodes I've created PR to @JoshPaterson fork with changes allowing for observe.at to work. One weird issue is that it seems like rows with "E" in uncertainty column screw up propagation for all orbits when passed to Filtering out done using: Other than changes in PR not looking pretty it seems that it should work. Although some testing and refactoring would definetely not hurt. |
|
Thanks for the update! I have some time this week for open source work, so I'll expect to prioritize this parallel-processing Skkyfield issue. Thanks for reviving it. |
|
Awesome, there's somewhat related issue for using data form mpcorb.dat files #650 - found out about while trying get precise correlation with telescope observations and comparing differences with orbits pulled from JPL Horizons. That one fortunately would only require adding a paragraph or two to chapter on Kepler orbits as a warning. |
|
Now that it's getting cold outside I'll have time to devote to this as well! @xmichaelx It'll probably take me a little bit to get back up to speed, then I'll get your PR merged, thanks for making that! |
|
Hi @brandon-rhodes, @JoshPaterson Is there any progress on this topic of vectorizing the comet objects to enable vectorized calculations on multiple comet objects simultaneously? Best regards |
This commit is only changes that are necessary to vectorize the creation of
_KelperOrbitobjects. This allows theat()method to work but not the.observemethod. This PR needs to be merged before #526.In order for the tests to pass I need to change them so that they don't use
VectorSumobjects or theobservemethod.