-
Notifications
You must be signed in to change notification settings - Fork 31
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
New distribution: ParticleDistribution #106
New distribution: ParticleDistribution #106
Conversation
3 similar comments
Thanks, this looks like a very good start on #105! I'll look over and see if I have any concrete suggestions, but after a cursory look, I think it's great. Before merging, I'd like to fix the Travis build error, though. It looks like it's unrelated to this PR, but is happening due to changes in QuTiP upstream; now that 4.0 is out, we should be able to make the doctest build more stable, so I'll go on and take care of that in a separate PR soon. Thanks! |
1 similar comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks good, merging now. Thanks!
This PR simply adds the new distribution
ParticleDistribution
and corresponding tests. All methods and properties ofSMCUpdater
which were (in my opinion) relevant to a bare particle distribution were copied over (but left intact). This is the first step of #105. The next step, in my view, is to implement the hierarchyParticleDistribution
>AbstractUpdater
>SMCUpdater
.It would be worthwhile for someone to compare the properties and methods of
ParticleDistribution
andSMCUpdater
to check if my choices are appropriate and comprehensive.[Sorry for all the whitespace changes my text editor did automatically]