Open codingwithsurya opened 8 months ago
Hello @codingwithsurya! Thank you for submitting the Feature Request Form. We appreciate your contribution. :wave:
We will look into it and provide a response as soon as possible.
To work on this feature request, you can follow these branch setup instructions:
Checkout the main branch:
git checkout nextjs
Pull the latest changes from the remote main branch:
git pull origin nextjs
Create a new branch specific to this feature request using the issue number:
git checkout -b feature-1149
Feel free to make the necessary changes in this branch and submit a pull request when you're ready.
Best regards, Deep Learning Playground (DLP) Team
@codingwithsurya update on this?
Feature Name
Add pre-installation check logging to dlp-cli to advise on removing the existing miniforge3/envs/dlp directory to prevent conflicts.
Your Name
Surya Subramanian
Description
When attempting to reinstall or update the backend using
dlp-cli backend install
, if theminiforge3/envs/dlp
directory already exists, it can lead to issues that are not immediately apparent to the user. This scenario can cause confusion and slow down the debugging process.It would be beneficial for the
dlp-cli
tool to include a logging statement before attempting the installation or update process that checks for the existence of theminiforge3/envs/dlp
directory. If found, the tool should log a clear message advising the user to remove or rename the existing directory to prevent potential conflicts. A suggested message could be:Adding this feature would improve user experience by making the debugging process more straightforward and reducing the time spent identifying common installation issues.