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

Fix error when trying to copy a natively written Edward random variable #658

Merged
merged 2 commits into from May 29, 2017

Conversation

dustinvtran
Copy link
Member

@dustinvtran dustinvtran commented May 29, 2017

An error is raised when trying to call ed.copy on a DirichletProcess, Empirical, ParamMixture, or PointMass random variable. This is noted in

The problem is that args/kwargs are not recorded correctly during logic in the class inheritance. If they aren't, then ed.copy has trouble recreating the random variable when copying its arguments.

@dustinvtran dustinvtran merged commit dfcadc5 into master May 29, 2017
@dustinvtran dustinvtran deleted the bugfix/copy-edward-rvs branch May 29, 2017 05:15
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

Successfully merging this pull request may close these issues.

None yet

1 participant