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

Reseting a project does not eliminate imported images and generated CSV files #204

Open
Miguel-Hermanns opened this issue Feb 5, 2020 · 4 comments
Milestone

Comments

@Miguel-Hermanns
Copy link

When I reset a project, I see that the imported questionnaires (tif file) as well as the generated csv file remain in the project directory, so that importing a new set of questionnaires and generating a new csv leads to two tif files and two csv files (with an increasing counter in their names).

Shouldn't the reseting of a project also clear these files?

@benzea
Copy link
Member

benzea commented Mar 22, 2020

Hmm, I wonder how to handle files outside/inside the project directory though (i.e. the --no-copy option for add).

Of course, it is trivial to just delete all .tif files yourself. But I do see how it would be convenient and that it is kind of unexpected.

@benzea
Copy link
Member

benzea commented Apr 20, 2023

So, these days reset will delete all .tif files that are inside the project (not, however, if they are referencing a file outside of the project, which can be done using add --no-copy.

Other files, such as generated CSV files are not yet deleted. So, maybe that is still worth changing?

@Miguel-Hermanns
Copy link
Author

I would probably delete de CSV files as well, for consistency. Or?

@benzea
Copy link
Member

benzea commented Apr 21, 2023

Yeah ... deleting all of the file types we would generate likely makes sense.

There is a slight risk of us deleting files that belong to the LaTeX questionnaire document, as we do not record what has been copied in using sdaps setup tex --add X.

Maybe the ideal thing would be to either:

  1. Make reset ask for confirmation (and suggest making a copy first)
  2. Add a setup existing or so command. So, conceptually create a new empty project instead of resetting an existing one.

@benzea benzea added this to the 1.9.14 milestone Apr 21, 2023
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