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

Automate arbitrary noise parameters. #205

Merged
merged 3 commits into from
Sep 9, 2024
Merged

Conversation

drjbarker
Copy link
Member

For the arbitrary noise thermostats, based on Fourier transforming the noise power spectrum, we have two parameters, the number of frequencies and the number of truncated point which have default values or can be user specified. However, it is very difficult for the user to choose the correct values without doing several simulations to check the number of frequencies is large enough for the discrete Fourier transform to be a good approximation of the integral, or the truncation number to be large enough not to cut off the tail of the memory kernel. Even if everything looks ok, the number of frequencies needed actually needs to be very high in many cases.

I've changed the code so that these parameters are now determined automatically. This is done by systematically increasing their values until the DFT or memory kernel are identical within a (quite tight) tolerance.

@drjbarker drjbarker merged commit bf16b01 into master Sep 9, 2024
2 checks passed
@drjbarker drjbarker deleted the fix-arbitrary-noise branch September 9, 2024 07:41
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.

1 participant