-
Notifications
You must be signed in to change notification settings - Fork 64
feat: finish_with_warning() #570
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
base: master
Are you sure you want to change the base?
Conversation
|
What do you think about instead of a I think I could rearrange this and add that pretty easily. Otherwise this looks fine. Added the feature to the download_videos processor and tested it out. Looks good. All the downloaders could make use of it, but that one in particular is great since new downloaders will just copy already downloaded videos. |
|
I think that makes a lot of sense, also with an eye on the future. |
|
(we should through the processors and add this where relevant before merging) |
|
I replaced the I also added a One oddity I found via the migrate script. I thought this would be sufficient for the remaining statues: |
Adds a dataset.finish_with_warning() function.
warningbool column to the datasets table to indicate if this dataset's last status update is a warning.Can be tested by batching prompts with Gemini 2.5 Flash (it often fails).
Any processors that may use it?