Skip to content

Dataclass refactor #77

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
Closed
cmutel wants to merge 132 commits into from
Closed

Dataclass refactor #77

cmutel wants to merge 132 commits into from

Conversation

@cmutel
Copy link
Collaborator

@cmutel cmutel commented Nov 17, 2025

  • 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

cmutel added 30 commits February 2, 2024 08:50
@cmutel
Add gas volume units
2e941b0
@cmutel
Fix inconsistent comma in suffix
be0a4d3
@cmutel
Merge branch 'main' of https://github.com/cmutel/flowmapper
3858ac4
@cmutel
Add groundwater category mapping
1951a2c
@cmutel
Add radiation unit conversion
5e2de3b
@cmutel
Fix country code extraction to only use existing countries
5900476
@cmutel
Better understand pint unit errors
0bbd464
@cmutel
Match natural resources with a different subcategory
3ff3079
@cmutel
Black reformatting
905758b
@cmutel
Add more category matching
38ba9b8 
Undefined is supposed match automatically but it seems like it doesn't
@cmutel
Add some manual matches
67de996
@cmutel
Add more non-country codes
edaa6ae
@cmutel
Match upper and lower case country codes
8f054c1
@cmutel
Add tests for upper and lowercase country codes
7f0d2f4
@cmutel
isort and black linting
f521e05
@cmutel
Rewrite Context away from a dataclass
66c71cf
@cmutel
Add description of concept
043c044
@cmutel
Some match simplifications
fbc2f1d
@cmutel
Refactor units
2abf326
@cmutel
Update README.md
a2c4c61
@cmutel
Add CAS to main library imports
63160a3
@cmutel
Correctly mark optional transformations for Typer
a51b297
@cmutel
Add generic class for strings
2997a68
@cmutel
Normalized string for None is ''
38f75a8
@cmutel
Export CAS as string
bc6e01d
@cmutel
Move constants to randonneur files
809d764
@cmutel
Remove unused code
e454e1b
@cmutel
Add StringList class for context
6919c1c
@cmutel
Remove default unit harmonization from Unit class
7273600
@cmutel
Linting and cleanup
8e30e56
cmutel added 28 commits November 7, 2025 11:15
@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
Add context manager for temporarily changing normalized flows
301429e
@cmutel
Fix matching for more functions
6ed9096
@cmutel cmutel marked this pull request as draft November 17, 2025 08:18
@cmutel cmutel closed this Nov 17, 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.

1 participant

@cmutel