fix sanitize print #313

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

xieofxie merged 9 commits into dev from fangyangci/fixSanitize

Jun 17, 2025

fangyangci commented Jun 12, 2025 •

edited

Loading

fangyangci added 5 commits

June 12, 2025 18:36


          fix error

c56137f


          fix sanitize.py print

a3b725d


          change name

99a650b


          revert

4ce19fb


          fix mistake

3f32cfc

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Outdated Show resolved Hide resolved

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Outdated Show resolved Hide resolved

fangyangci added 3 commits

June 13, 2025 11:11


          do not exit when error occurs

35c1197


          fix naming

483ad0e


          revert miss merge

cc53735

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Outdated Show resolved Hide resolved

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Outdated Show resolved Hide resolved

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py

@@ @@ -963,9 +954,7 @@ def checkOliveFile(self, oliveJson: Any): @@
                           diff['values_changed'] = newChangeds
                       if diff:
-                          # Check out branch hualxie/example_align for alignments

xieofxie Jun 13, 2025

add this comment in error mesg?

xieofxie Jun 16, 2025

i don't see it. what's your opinion?

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Outdated Show resolved Hide resolved

xieofxie reviewed

View reviewed changes

model_lab_configs/scripts/sanitize.py Show resolved Hide resolved

fix

85b1703

xieofxie approved these changes

View reviewed changes

xieofxie merged commit 4e2213f into dev

1 check passed

xieofxie deleted the fangyangci/fixSanitize branch

June 17, 2025 02:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet