Tokenizable atom mappers

[richardjgowers]

Following this change, OpenFreeEnergy/gufe#266 updates to openfe

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/.

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (f95e75d) 91.37% compared to head (443fcf2) 91.36%.
Report is 1 commits behind head on main.

Files	Patch %	Lines
openfe/tests/setup/atom_mapping/test_atommapper.py	87.50%	1 Missing ⚠️
openfe/tests/setup/test_network_planning.py	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #690      +/-   ##
==========================================
- Coverage   91.37%   91.36%   -0.01%     
==========================================
  Files         132      132              
  Lines        9158     9174      +16     
==========================================
+ Hits         8368     8382      +14     
- Misses        790      792       +2     

Flag	Coverage Δ
fast-tests	91.36% <87.50%> (-0.01%)	⬇️

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]

I assume it's ok not to test the from_dict extra bit?

[richardjgowers]

@IAlibay yeah those are toy mappers that aren't ever used in anger