Perses mapper

[jthorton]

Replaces #929 to enable openeye tests

Checklist

• Added a news entry

Developers certificate of origin

• I certify that this contribution is covered by the MIT License here and the Developer Certificate of Origin at https://developercertificate.org/.

[IAlibay]

Not sure why codecov isn't picking up the extra test, let's see if bumping the PR helps.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.85%. Comparing base (733908d) to head (78d67c9).
Report is 9 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #930      +/-   ##
==========================================
- Coverage   94.32%   92.85%   -1.48%     
==========================================
  Files         134      134              
  Lines        9917     9931      +14     
==========================================
- Hits         9354     9221     -133     
- Misses        563      710     +147     

Flag	Coverage Δ
fast-tests	92.85% <100.00%> (?)
slow-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[IAlibay]

@jthorton could you add a news entry please? (since it's technically changing behaviour from omm to off units). Otherwise it looks good to me!

P.S. macos failure is something else, issue has been raised upstream.

[jthorton]

thanks @IAlibay I have added it should it show in the docs build as part of the change log?

[IAlibay]

thanks @IAlibay I have added it should it show in the docs build as part of the change log?

I believe it will only show up when we call rever (we probably should fix that at some point?), so it should be good for now!

[IAlibay]

lgtm!