sblunt / orbitize

Orbit-fitting for directly imaged objects
https://orbitize.info
Other
74 stars 43 forks source link

Standard to XYZ documentation needs a minor fix #347

Closed michaelkmpoon closed 6 months ago

michaelkmpoon commented 11 months ago

Describe the bug I think orbitize.basis.XYZ.standard_to_xyz has some incorrect documentation. To convert from standard orbital elements to XYZ for more than 1 set of parameters, the documentation says that the first dimension must be the number of orbital parameter sets.

image

To Reproduce Here is a simple example that shows the error, as well as the how instead I got it to work (by transposing the array). Instead, it should be the second dimension that is the number of orbital parameter sets.

image

This is with orbitize! Version 2.2.2.

sblunt commented 6 months ago

Addressed in #363.