Skip to content

Conversation

hugokallander
Copy link

No description provided.

Copy link

@oeway oeway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for the PR, could you move the artifact-recovery scripts to utils folder instead of a separate one, there we already have the backup_database.py

Is there a command for dry-run the script without causing any damage, but check if the script working?

@hugokallander
Copy link
Author

Will do!

@hugokallander
Copy link
Author

hugokallander commented Jun 3, 2025

@oeway I moved the files to util/ and using only Claude I added the option --dry-run, but I haven't been able to test it fully because I lack the necessary environment variable values. Upon inspection, however, I think it looks pretty good

@oeway
Copy link

oeway commented Jun 3, 2025

OK, thanks! I think you can test it in the juptyer notebook we used to perform recovery, that has already env there.
Otherwise we put the PR on hold a bit until we tested it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants