-
Notifications
You must be signed in to change notification settings - Fork 56
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
Update and include features in multibody #126
Conversation
Together with the previous commits fixes #112
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work, as always. Many thanks @ArturoMS13 !!
# if (self.data.structure.num_bodies > 1): | ||
# self.data.structure.ini_info.whole_structure_to_local_AFoR(self.data.structure) | ||
# self.data.structure.timestep_info[0].whole_structure_to_local_AFoR(self.data.structure) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Want to kill 👎 or leave 👍 this?
def initialise(self, data, custom_settings=None): | ||
|
||
self.data = data | ||
if custom_settings is None: | ||
self.settings = data.settings[self.solver_id] | ||
else: | ||
self.settings = custom_settings | ||
settings.to_custom_types(self.settings, self.settings_types, self.settings_default) | ||
settings.to_custom_types(self.settings, self.settings_types, self.settings_default, no_ctype=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
settings.to_custom_types(self.settings, self.settings_types, self.settings_default, no_ctype=True) | |
settings.to_custom_types(self.settings, self.settings_types, self.settings_default) |
I guess that thanks to your addition in #123 the no ctype is no longer needed
@@ -73,9 +92,29 @@ def initialise(self, data, custom_settings=None): | |||
self.Lambda_dot = np.zeros((self.num_LM_eq,), dtype=ct.c_double, order='F') | |||
self.Lambda_ddot = np.zeros((self.num_LM_eq,), dtype=ct.c_double, order='F') | |||
|
|||
if self.settings['write_lm']: | |||
dire = './output/' + self.data.settings['SHARPy']['case'] + '/NonLinearDynamicMultibody/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once we merge #130, the idea would be to change this to dire = self.data.output_folder + '/NonLinearDynamicMultibody/
@@ -142,8 +142,7 @@ def initialise(self, MBdict_entry, ieq): | |||
@abstractmethod | |||
# def staticmat(self, **kwargs): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to get rid of the commented function definitions?
This pull request modifies some features of the multibody computations which do not influence the user interface:
And it also includes new features:
Requirements before merging: