Cleaned up assignment of self.coeffs to discard wasteful initilization.

lsst / rubin_sim

Scheduler, survey strategy analysis, and other simulation tools for Rubin Observatory.

https://rubin-sim.lsst.io

GNU General Public License v3.0

41 stars 37 forks source link

Cleaned up assignment of self.coeffs to discard wasteful initilization. #412

Closed bwSimpkins closed 3 months ago

bwSimpkins commented 4 months ago

I am new to the LSST code community. Thank you all for the interesting work you all are doing here. I saw an extremely small change that would discard the neew to initialize self.coeffs in the cheby_values class. It doesn't changed the functionality of the set_coefficients but makes it more concise in my opinion.

rhiannonlynne commented 3 months ago

Closing PR as the change causes tests to fail. Please feel free to open a new PR with other changes!