Skip to content

Conversation

@jguarato
Copy link
Collaborator

@jguarato jguarato commented Oct 4, 2024

Pull Request created to make corrections related to the plotting of modes in the Results class:

  • Removal of duplicate information related to whirl direction, which was inconsistent because the colors were not in accordance with the whirl direction shown in figures.

    • Only information from the Shape class was kept.
    • Unused methods were then removed.
  • Some adjustments were made to the _eigen method of the Rotor class.

    • To avoid constant changes in the plotting of the modes in each simulation, due to the random nature of the solver initialization, the parameter v0 of the las.eigs function was set with a fixed array of values.
    • A limitation related to the amount of eigenvalues ​​to be found was applied when sparse = True.
  • As tests were carried out, the appearance of bugs and warnings were also corrected.

@jguarato jguarato marked this pull request as ready for review October 11, 2024 18:40
@raphaeltimbo raphaeltimbo merged commit d939735 into petrobras:main Oct 17, 2024
9 checks passed
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