Open
Conversation
added 3 commits
May 16, 2025 16:48
…, g(r) = -g_0 r e_r). Add an option in the definition of the problem and modify the matrix
commit 5cbbdcedca625852772a7e31fa0092b8466fb0ab
Author: Line Colin <line.colin@ens-lyon.fr>
Date: Fri May 16 17:05:14 2025 +0200
Add the option for uniform or radius dependant gravity acceleration
This update introduces the option of choosing between uniform and radius-dependent gravity acceleration in the SphStability class.
The gravity acceleration that depends of the radius is defined as:
g(r) = - g_0 * r * e_r
The 'uniform_gravity' boolean flag has been added to the dataclass to control this option. When set to 'False', gravity acceleration varies linearly with radius.
This change required modifications in 'lmat'.
Owner
|
@LineColin Thanks for your contribution. In order to keep the code modular, I've implemented the same idea in a slightly different fashion in the Could you confirm that the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add an option to take into account gravity acceleration that varies linearly with radius.