Skip to content
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

draft/ folder feels like a misnomer — it's just a failed/ folder? #157

Open
jeffeaton opened this issue Jul 4, 2024 · 2 comments
Open

Comments

@jeffeaton
Copy link

Now with orderly_run() automatically committing completed runs to archive/, the draft/ folder feels misnamed. What is left in there are reports that have failed.

Personally, I sort of miss the old [draft] -> commit to [archive] workflow. Is there scope to bring that back as an option? (Maybe an arg orderly_run(..., archive = FALSE)?)

Otherwise, perhaps rename the folder to something reflecting what lives in there?

@richfitz
Copy link
Member

richfitz commented Jul 5, 2024

This is where things are run though, so they literally are drafts until successful. Unfortunately drafts is one of the things that users of orderly1 will have to unlearn: see https://mrc-ide.github.io/orderly2/reference/orderly_run.html#equivalence-to-the-old-use-draft-option for some docs on this.

The big issue we get is if you have done a run into draft, people inevitably ended up wanting to use it for things, so things in drafts were sort of both draft and also archive and there was lots of code duplicated doing sort of the same thing, with no clear workflow.

Now, you should be able to delete whatever you want, so the cost of running something is much lower

@jeffeaton
Copy link
Author

Thanks, sounds good. I think more directive instructions on 'how to correctly delete' will also help lower the bar on this — just added an issue suggesting on that: #161.

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

No branches or pull requests

2 participants