Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Splitting apart DropSizeDistribution #64

Open
josephhardinee opened this issue Jan 3, 2018 · 1 comment
Open

Splitting apart DropSizeDistribution #64

josephhardinee opened this issue Jan 3, 2018 · 1 comment

Comments

@josephhardinee
Copy link
Owner

@nguy I've been tossing around the idea for a while of pulling out a lot of the operations from DropSizeDistribution object and moving them into library functions. This would simplify the DropSizeDistribution object quite a bit, and make testing easier. It would also make it easier to do things like have an operation on a dsd object (like a select) return a new dsd object.
The functionality I had in mind was moving scattering to it's own submodule, and similarly for the rain fits that have been in the object just as a part of it's legacy.

@nguy
Copy link
Collaborator

nguy commented Mar 1, 2018

Not sure how I missed this. My bad.

I'll think on this a little, but on the surface I agree. The more modular the code, the easier the testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants