[WIP] [WIP] Address feedback to apply coefficient rotation change in computeAccel() calls

[Copilot]

Thanks for the feedback on #220. I've created this new PR, which merges into #220, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #220
Triggering review: #220 (comment)

@copilot This suggestion is good; please applying the same coefficient rotation change to all computeAccel() call sites in expui/BiorthBasis.cc as a new PR against devel.

[The9Cat]

Looks fine to me: Copilot seems to have changed all computeAccel implementations.

[michael-petersen]

@copilot resolve the merge conflicts in this pull request

[michael-petersen]

@copilot can you take the failure from the CI run and suggest an edit?

[Copilot]

@copilot can you take the failure from the CI run and suggest an edit?

I addressed the CI failure in commit a935940 by removing the duplicate Eigen::Vector3d pos declaration in Cube::computeAccel() (expui/BiorthBasis.cc), which resolved the redeclaration compile error from the failing Build and Test run.

[The9Cat]

Huh, that's odd. I pulled devel earlier, built successfully, and ran the full cmake suite.

[michael-petersen]

All computeAccel calls have been updated!

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.