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

Overwrite analysis inputs file #163

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GuiMacielPereira
Copy link
Collaborator

Description of work:
This change improves the quality of life of scientists because in order to update their analysis inputs file following a version update they no longer need to delete their entire config folder.

Running vesuvio config will now update the analysis_inputs.py file, even if the config folder ./mvesuvio already exists.

To test:
Activate an environment with mantidworkbench installed:

  • Checkout this branch and install with pip install -e .
  • Run mvesuvio config
  • Open file ~/.mvesuvio/analysis_inputs.py and write anything to it (changes will be overwritten)
  • Run mvesuvio config
  • Open file ~/.mvesuvio/analysis_inputs.py and check that your changes were overwritten
  • Delete ~/.mvesuvio/
  • Run mvesuvio config
  • Check that a new folder ~/.mvesuvio was created

Fixes #xxxx.

This change quality of life of scientists because in order to update their analysis inputs file
following a version update they no longer need to delete their entire config folder.

The file will update in the existing config folder.
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.

Command for vesuvio config should override .mvesuvio/ except IP files
1 participant