Closed lgrcia closed 7 months ago
This PR should be tested and merged only after #151. It includes:
(1+num_bodies, num_times)
SurfaceMapSystem
Map
1.0
None
light_curve_test.py
The tests against starry for multiple bodies pass! And the rest is a good excuse to think about some API choices (#148)
This PR should be tested and merged only after #151. It includes:
(1+num_bodies, num_times)
(the shape management is a bit shady, I'd appreciate a review in there)SurfaceMapSystem
has a defaultMap
of amplitude1.0
here. Do we want to considerSurfaceMapSystem
with aNone
map?light_curve_test.py
that now includes:None
maps for both central and/or bodyThe tests against starry for multiple bodies pass! And the rest is a good excuse to think about some API choices (#148)