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

chili: make basis truncation parameters a required user input? #158

Open
stestoll opened this issue Nov 3, 2019 · 0 comments
Open

chili: make basis truncation parameters a required user input? #158

stestoll opened this issue Nov 3, 2019 · 0 comments
Labels
proposal A new idea

Comments

@stestoll
Copy link
Member

stestoll commented Nov 3, 2019

chili uses a default basis set truncation independent of the nature of the spin system and independent of the presence or absence of an orientational potential (Sys.Potential) or director ordering (Exp.Ordering).

This choice is fine for some of the common simple X-band nitroxide situations, but can be severely overtruncated in other cases.

This suggests that chili could require the user to always provide some basis set truncation information.

Pros:

  • Raises user awareness about importance of basis set choices.
  • Prevents chili from simulating unconverged spectra from default basis.

Cons:

  • Likely a big hassle for users.
  • Users might default to using the same basis in different scripts (copy-paste...), without investigating convergence properly.

The extent of this requirement is unclear. Which of the basis set parameters (orientational basis: LLMK, deltaK, jKmin; spin basis: pSmin, pImax) should be required and which can have reasonably general default values?

This appears to resemble Opt.nKnots for rigid-limit powder simulations. However, there, the choice is much less crucial due to the interpolative spectrum construction.

@stestoll stestoll added this to the 6.0 milestone Nov 3, 2019
@stestoll stestoll added the proposal A new idea label Nov 5, 2019
@stestoll stestoll removed this from the 6.0 milestone Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal A new idea
Development

No branches or pull requests

1 participant