search

BrownDwarf / gollum

A microservice for programmatic access to precomputed synthetic spectral model grids in astronomy
https://gollum-astro.readthedocs.io/
MIT License
21 stars 5 forks source link

Make sure we are using deep copies instead of self? #43

Closed gully closed 1 year ago

gully commented 2 years ago

We learned a few lessons from muler that we should incorporate into gollum. We had to make deep copies instead of self in order to avoid in-place operations. We should do the same here. It has not appeared to cause a problem yet, but it might eventually.

Sujay-Shankar commented 1 year ago

After parsing through the gollum code for a bit, it looks like the coast is clear in terms of us doing any accidental in-place operations. We should be good to close this.