Update evaluation of sampling efficiency in SampleContainer

[kamronald]

Normalize the sampling efficiency by number of samples saved, instead of the total number of MC iterations.

Summary

Major changes:

Changed the definition of sampling efficiency to be "# acceptances" / "# samples considered". For MC runs with sample thinning > 1, this becomes only an estimate for the true sampling efficiency, as we do not know the sampling efficiency of the discarded samples. However this estimate should be more accurate than the previous definition of sampling efficiency: "# acceptances" / "# total number of MC iterations"

Todos

If this is work in progress, what else needs to be done?

• feature 2: ...
• fix 2:

Checklist

• All existing tests pass.
• Tests have been added for any new features/fixes.
• Docstrings have been added in the Google docstring format.

Tip: Install pre-commit hooks to auto-check types and linting before every commit:

pip install -U pre-commit
pre-commit install

[lbluque]

thanks @kamronald !