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

issue 116: Generate test posterior samples #117

Closed
wants to merge 11 commits into from
Closed

Conversation

SamuelBrand1
Copy link
Collaborator

@SamuelBrand1 SamuelBrand1 commented Nov 12, 2024

This PR closes #116.

This is a simple PR that tidies up the local fit loop utility, and creates a test directory which can be aimed at with loop_fit.sh. .pickle files are ignored so a test run with unit tests should start with this step.

Run fake posterior generation

% bash ./iteration_helpers/loop_fit.sh ./tests/run_test_inference

from /pipelines.

This will be part of planned test runs.

@damonbayer
Copy link
Collaborator

damonbayer commented Nov 12, 2024

Can we nudge this PR toward #57 ? I think we could start by putting tests not in the nssp_demo directory?

@SamuelBrand1
Copy link
Collaborator Author

Can we nudge this PR toward #57 ? I think we could start by putting tests not in the nssp_demo directory?

I'd rather not guess what structure we want. Happy to hold this until we resolve #57 .

@SamuelBrand1
Copy link
Collaborator Author

This works with the new directory structure now.

@SamuelBrand1
Copy link
Collaborator Author

Reacting to changes in the code base; I've added a priors.py file to the test model dir

@SamuelBrand1
Copy link
Collaborator Author

updated test folder priors.py to track changes in code base.

@SamuelBrand1
Copy link
Collaborator Author

Closed as superseded by #148

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.

Generate test posterior samples
2 participants