eth-easl / modyn

Modyn is a research-platform for training ML models on growing datasets.
MIT License
25 stars 3 forks source link

feat: Add tool for patching old logfiles #568

Closed robinholzi closed 1 month ago

robinholzi commented 1 month ago

Motivation

We want to use old logfiles in our new logfile format

robinholzi commented 1 month ago

fyi. @XianzheMa

XianzheMa commented 1 month ago

Thank you so much! Let me check out your branch to test first

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.00%. Comparing base (4666461) to head (28b6aab).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #568 +/- ## ======================================= Coverage 83.00% 83.00% ======================================= Files 226 226 Lines 10528 10528 ======================================= Hits 8739 8739 Misses 1789 1789 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 1 month ago

Line Coverage: -% ( % to main) Branch Coverage: -% ( % to main)

XianzheMa commented 1 month ago

I think we can merge this. The dataset id discrepancy and the eval request thingy are irrelevant to this PR

robinholzi commented 1 month ago

@XianzheMa @MaxiBoether can one of you approve this PR?

XianzheMa commented 1 month ago

sorry I forgot this