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

Add FreeLayerThickness parameter #266

Merged
merged 1 commit into from
Aug 5, 2020
Merged

Conversation

JeroenMulkers
Copy link
Collaborator

The Slonczewski torque depends on the thickness of the free layer.
Before this commit, this value was deduced from the thickness of the
mesh. There could be cases in which this is not what the user wants.

The changes made in this commit allow the user to set the free layer
thickness by hand through the "FreeLayerThickness" regionwise parameter.

If FreeLayerThickness is set to zero (the default), then the thickness
will be deduced from the thickness of the mesh, ensuring backwards
compatibility.

The Slonczewski torque depends on the thickness of the free layer.
Before this commit, this value was deduced from the thickness of the
mesh. There could be cases in which this is not what the user wants.

The changes made in this commit allow the user to set the free layer
thickness by hand through the "FreeLayerThickness" regionwise parameter.

If FreeLayerThickness is set to zero (the default), then the thickness
will be deduced from the thickness of the mesh, ensuring backwards
compatibility.
@godsic godsic merged commit 6000d64 into master Aug 5, 2020
Copy link
Contributor

@JLeliaert JLeliaert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as intended in a testcase where the dynamics are simulated with and without a spacer layer. Previously the spacer layer gave rise to erroneous results by also being included in the freelayerthickness. This is now mitigated

@JeroenMulkers JeroenMulkers deleted the feature/slonczewskithickness branch September 18, 2020 16:36
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.

3 participants