Skip to content

Moment of inertia #225

Closed
Closed

Description

@skfegan

Currently the code is using the diagonal elements of the moment of inertia. I think perhaps it should be using the eigenvalues of the moment of inertia tensor.

Activity

  1. self-assigned this
    on Dec 2, 2025
  2. ioanaapapa commented on Dec 2, 2025

    @ioanaapapa
    Collaborator

    I think we still want to use the diagonal elements for the torque averaging but the eigenvalues are used to sort the corresponding eigenvectors so that the direction of the axis is always consistent. I changed the moment of inertia calculation in #228 , using the same rotational coordinates as for the torques to calculate the eigenvalues, and the moment of inertia axes are now consistent with the torque axes, and results are a much closer match to previous code results. I think that might be fixing this?

  3. skfegan commented on Jan 15, 2026

    @skfegan
    MemberAuthor

    This is closed by merge #246. Using the sorted eigenvalues of the moment of inertia to weight the torques while using the principal axes for the rotational axes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions