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

Add seed argument to EnsembleSampler #376

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Add seed argument to EnsembleSampler #376

wants to merge 20 commits into from

Conversation

dfm
Copy link
Owner

@dfm dfm commented Jan 16, 2021

This was previously opened as #332, but because of branch renaming issues we had to reopen it. This is @OriolAbril's work, but @carlosgmartin might also be interested. I'll try to summarize where we stand from the previous thread shortly, but I think we were just about there. Anything in particular that was missing?

@dfm
Copy link
Owner Author

dfm commented Jan 16, 2021

The tests are failing because we need to change

import packaging

to

import packaging.version

@dfm
Copy link
Owner Author

dfm commented Jan 16, 2021

@OriolAbril: If you check the "allow edits from maintainers" checkbox, then I can take over from here.

@OriolAbril
Copy link
Contributor

OriolAbril commented Jan 16, 2021

I don't have the option to either, probably because I did not open the PR, but I can give both you and @carlosgmartin access to my fork.

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

2 participants