Skip to content

Add tests to verify that sampler stats match the advertised stats_dtypes #5883

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

Open
michaelosthege opened this issue Jun 12, 2022 · 0 comments
Labels
tests trace-backend Traces and ArviZ stuff

Comments

@michaelosthege
Copy link
Member

Every step method has a stats_dtypes dictionary that advertises the dtypes to be expected.

This information is relevant for storage backends so they can create arrays of the correct dtypes.

In #5882 a fix was needed because previous changes to the pm.Metropolis stepper did not update the stats_dtypes dictionary.

We already have tests that are parametrized by step methods, so instead of creating a new test for this issue, we should try to add the asserts in there.

@ricardoV94 ricardoV94 assigned ricardoV94 and unassigned ricardoV94 Jun 13, 2022
ricardoV94 added a commit to ricardoV94/pymc that referenced this issue Jun 13, 2022
ricardoV94 added a commit that referenced this issue Jun 13, 2022
@michaelosthege michaelosthege added the trace-backend Traces and ArviZ stuff label Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests trace-backend Traces and ArviZ stuff
Projects
None yet
Development

No branches or pull requests

2 participants