Skip to content

Conversation

@michael-0brien
Copy link
Owner

@michael-0brien michael-0brien commented Oct 20, 2025

C.f. #330 @geoffwoollard. Where we left off was:

  • Port over tests + uniform phase sampling from @shuyuvankerkwijk's fork (thanks @shuyuvankerkwijk!)
  • Calibrate model in a reasonable way so that thickness_in_angstroms can be used to increase noise level. Where we left off was we were trying to calibrate noise levels using physical parameters, but the code is currently incomplete.
  • Add solvent masking (not sure what the best way to do this is)

Note that #461 must be merged first.

@michael-0brien michael-0brien added the enhancement New feature or request label Oct 20, 2025
@michael-0brien michael-0brien marked this pull request as draft October 20, 2025 04:43
Base automatically changed from api-updates-for-453454 to dev October 28, 2025 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implementing a more physical solvent model for ice phase noise

3 participants