-
Notifications
You must be signed in to change notification settings - Fork 148
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.
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
chore: make refresh notebook still commit successful notebooks on fail #313
Conversation
ff1ba98
to
b2d4927
Compare
I am refreshing the notebooks here with the new workflow and will cherry-pick the commit in #304. Also need to revert the on: push. |
2a2b3cf
to
44e0e0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure these changes makes successful notebooks be committed even if another fails ? or at least I can't see what changes would make this possible here !
Yes they have been committed in #304 while Deployment notebook failed. |
echo "${NOTEBOOK}" >> "${SUCCESSFUL_NOTEBOOKS}" | ||
else | ||
echo "${NOTEBOOK}" >> "${FAILED_NOTEBOOKS}" | ||
fi | ||
END=$(date +%s) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what makes the refresh notebook not crash on failure @RomanBredehoft .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find it weird that this makes it possible but I trust you ! thanks a lot for this !!
closes https://github.com/zama-ai/concrete-ml-internal/issues/3764