Skip to content

Dataclass refactor #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
cmutel merged 35 commits into from
Nov 21, 2025
Merged

Dataclass refactor #12

cmutel merged 35 commits into from
Nov 21, 2025

Conversation

@cmutel
Copy link
Collaborator

@cmutel cmutel commented Nov 17, 2025 •
edited
Loading

  • Make Flow, NormalizedFlow, and Match domain object dataclasses
  • Move from "first match wins" to "ensure only one matching" paradigm for each strategy
  • Allow temporary transformations for matching functions
  • All pre-baked transformations are randonneur files, and easily support randonneur functions in flowmapper
  • Add well-tested functions to extract and isolate oxidation state and location parameters from names

cmutel added 30 commits November 7, 2025 10:34
@cmutel
Add more control for logging
4ae830a
@cmutel
Note that synonyms shouldn't overlap in README
186b14c
@cmutel
Move to src layout
4e782ba
@cmutel
Complete move to absolute imports
f7f5aa1
@cmutel
Remove loguru
82a5af0
@cmutel
Add additional tests for missing code paths
f4a5fa1
@cmutel
Bump minimum Python version
9f17313
@cmutel
Allow match functions to see all source and target flows
ac7b5d1
@cmutel
Add fixes for required randonneur fields
01ee4cd
@cmutel
Improve test reporting and edge cases
860ee1b
@cmutel
Additional test fixes
00a3b0a
@cmutel
Allow profiling of matching
1c7d383
@cmutel
Patch broken tests for now
f22e61d
@cmutel
Avoid running tests twice if they error out
14d27e7
@cmutel
Reduce profiler interval for smaller result file sizes
0a13345
@cmutel
Start refactoring to version flows
1f514cc
@cmutel
Add new field for oxidation state
b4950ba
@cmutel
Redo unit field
d6a6348
@cmutel
Change to cas_number
c891725
@cmutel
Redo CAS field
91d2d20
@cmutel
Remove StringList
aa767ca
@cmutel
Add remove_unit_slash
1abd0e2
@cmutel
Test split_location_suffix
c101918
@cmutel
Update StringField
8f691aa
@cmutel
Update ContextField
e213341
@cmutel
Big progress on refactoring
9c7a818
@cmutel
Add equality operator for CASField
86ac29f
@cmutel
Add reset_current and update_current to NormalizedFlow
380344d
@cmutel
Add is_resource to ContextField
ad9e05b
@cmutel
Fix exporting of results
eaae8c1
@cmutel cmutel marked this pull request as draft November 17, 2025 08:20
@cmutel cmutel marked this pull request as ready for review November 21, 2025 13:27
@cmutel cmutel merged commit 807927e into main Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cmutel