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

Damping term not required for RelaxDriver #82

Merged
merged 1 commit into from
Mar 1, 2024

Conversation

kzqureshi
Copy link
Contributor

Mumax documentation is not clear about setting alpha, it is not needed or harmful :

"Relax() tries to evolve the magnetization as closely as possible to the minimum energy state. This function assumes all excitations have been turned off (temperature, electrical current, time-dependent magnetic fields). During relax precession is disabled and the time t does not increase. There is no need to set high damping."

Mumax documentation is not clear about setting alpha, it is not needed
or harmful :

"Relax() tries to evolve the magnetization as closely as possible to the minimum energy state. This function assumes all excitations have been turned off (temperature, electrical current, time-dependent magnetic fields). During relax precession is disabled and the time t does not increase. There is no need to set high damping."
@kzqureshi
Copy link
Contributor Author

Tests will only pass when #79 is merged.

@samjrholt samjrholt changed the base branch from master to check_system March 1, 2024 09:44
@samjrholt samjrholt merged commit faa8337 into check_system Mar 1, 2024
1 of 2 checks passed
@samjrholt samjrholt deleted the fix_relax_driver branch March 1, 2024 09:45
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.

2 participants