feat(ci): add GitHub Action for Python tests, fix EarlyStopping logic, and add unit tests #27
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces several key changes:
EarlyStopping
class logic, changingbest_score
tobest_val_loss
to improve clarity and correctness.EarlyStopping
class to ensure its correct behavior after the logic fix.Changes:
Added Python application tests:
3.9
,3.10
,3.11
,3.12
).main
branch.Fixed EarlyStopping logic:
best_score
was renamed tobest_val_loss
for clarity, improving the code's readability and matching the purpose of the variable.Added Unit Tests for
EarlyStopping
:test_early_stopping.py
contains a thorough set of tests covering:Why this change is important:
EarlyStopping
class, making it more robust and clear.EarlyStopping
class functions as expected through the added unit tests.