search

LSSTDESC / rail_base

Base classes for RAIL
MIT License
0 stars 1 forks source link

PointEstimateMixin overwrites existing ancil data #82

Closed drewoldag closed 6 months ago

drewoldag commented 6 months ago

Bug report Calling PointEstimateMixin.calculate_point_estimates will overwrite existing ancil data. This is because we're using .set_ancil instead of .add_to_ancil.

The solution here would be something like:

if qp_dist.ancil is None:
    qp_dist.set_ancil(ancil_dict)
else:
    qp_dist.add_to_ancil(ancil_dict)