Cleanup Duplicate Code

[j-c-c]

Resolves #1233

Removes duplicate code found in both src/aspire/utils/coor_trans.py and src/aspire/utils/rotation.py.

Also, I refactored mean_aligned_angular_distance to use the alignment code which now resides in the Rotation class in a way to not change the API. It might make sense to move this function into the Rotation class as well, but it will change the API.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.61%. Comparing base (9fa3ccc) to head (9f45cb4).
Report is 6 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1280      +/-   ##
===========================================
+ Coverage    90.58%   90.61%   +0.03%     
===========================================
  Files          132      132              
  Lines        14162    14105      -57     
===========================================
- Hits         12828    12781      -47     
+ Misses        1334     1324      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[garrettwrong]

Looks good! Minor string update and quick questions, thanks.

[janden]

Looks great!