This is a good catch - thanks! I did have a reason for changing these values back when I implemented it, but I can't track down why. It does make sense for the class called FardalStreamDF to be true to the Fardal et al. 2015 publication, as you've fixed, but I'm worried that this will cause some backwards compatibility issues for users who were using it previously. One option would be to add an argument to the class (with a default False value) to let the user specify whether they want to use the original prescription, or the Gala augmented values? Is that something you'd be interested in implementing here?

This is a good catch - thanks! I did have a reason for changing these values back when I implemented it, but I can't track down why. It does make sense for the class called FardalStreamDF to be true to the Fardal et al. 2015 publication, as you've fixed, but I'm worried that this will cause some backwards compatibility issues for users who were using it previously. One option would be to add an argument to the class (with a default False value) to let the user specify whether they want to use the original prescription, or the Gala augmented values? Is that something you'd be interested in implementing here?

Thanks for reviewing the change! It's a great idea to protect the change by a flag for backwards compatibility. But I guess you could be the better person implementing it in terms of naming, adding description and consistent code style. Just let me know. Thanks!

Sure -- I can take it from here. I'll just contribute to this PR, so your contribution is still tracked. Thanks again for raising this!

Sure -- I can take it from here. I'll just contribute to this PR, so your contribution is still tracked. Thanks again for raising this!

Thanks for the thorough fixes!