sblunt/orbitize

Standard to XYZ documentation needs a minor fix

Closed this issue · 1 comments

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.

Addressed in #363.