Deep-Learning-Playground icon indicating copy to clipboard operation
Deep-Learning-Playground copied to clipboard

[BUG]: Fix dlp-cli so that it has better regex for dlp env

Open dwu359 opened this issue 2 years ago • 3 comments

Operating system

Noah's machine

Bug description

The dlp-cli will detect a false positive for a mamba env if the mamba env contains the string "dlp" in it, even if the env is named differently.

Steps to reproduce

Create a mamba env with a name containing "dlp", install backend via dlp-cli

dwu359 avatar Sep 24 '23 19:09 dwu359

Hello @dwu359! Thank you for submitting the Bug Request Form. We appreciate your contribution. :wave:

We will look into it and provide a response as soon as possible.

To work on this bug request, you can follow these branch setup instructions:

  1. Checkout the main branch:
```
 git checkout nextjs
```
  1. Pull the latest changes from the remote main branch:
```
 git pull origin nextjs
```
  1. Create a new branch specific to this bug request using the issue number: git checkout -b bug-988

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

github-actions[bot] avatar Sep 24 '23 19:09 github-actions[bot]

@dwu359 this bug isn't just on Noah's machine. It seems to be across diff OS

karkir0003 avatar Oct 07 '23 22:10 karkir0003

Yep I know, that's just the OS where we discovered the bug, and it definitely isn't an OS dependent bug

dwu359 avatar Oct 07 '23 22:10 dwu359