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

[BUG] nam GUI fails to start: ImportError: cannot import name 'nam_gui' from 'nam.cli' #528

Closed
ibodog opened this issue Jan 20, 2025 · 2 comments
Assignees
Labels
bug Something isn't working priority:high Prioritize

Comments

@ibodog
Copy link

ibodog commented Jan 20, 2025

Thanks for taking the time to write a bug report! Use the following prompts to help you describe what's going on. The more info you provide, the easier it'll be to understand how to fix it for you without having to come back and ask you questions.

Describe the bug
(nam) PS C:\Users\jkruse\projects\NAM\neural-amp-modeler> nam
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\jkruse\miniconda3\envs\nam\Scripts\nam.exe_main
.py", line 4, in
ImportError: cannot import name 'nam_gui' from 'nam.cli' (C:\Users\jkruse\projects\NAM\neural-amp-modeler\nam\cli.py). Did you mean: '_nam_gui'?

To Reproduce
Steps to reproduce the behavior:

  1. From conda powershell prompt:
  2. git clone https://github.com/sdatkinson/neural-amp-modeler.git
  3. cd neural-amp-modeler
  4. conda env update -f .\environments\environment_gpu.yml
  5. conda activate nam
  6. nam

(nam) PS C:\Users\myuser\projects\NAM\neural-amp-modeler> nam
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\myuser\miniconda3\envs\nam\Scripts\nam.exe_main
.py", line 4, in
ImportError: cannot import name 'nam_gui' from 'nam.cli' (C:\Users\jkruse\projects\NAM\neural-amp-modeler\nam\cli.py). Did you mean: '_nam_gui'?

Expected behavior
Expected nam GUI to launch and be ready to configure training

Desktop (please complete the following information):

  • Windows 11
  • Running locally
  • Package version: commit af1c4d5 (HEAD -> main, origin/main, origin/HEAD)

commit af1c4d5 (HEAD -> main, origin/main, origin/HEAD)
Author: Steven Atkinson steven@atkinson.mn
Date: Mon Jan 20 11:49:29 2025 -0800

Fix environment name

Additional context
Add any other context about the problem here.

@ibodog ibodog added bug Something isn't working priority:low Low-priority issues unread This issue is new and hasn't been seen by the maintainers yet labels Jan 20, 2025
@sdatkinson
Copy link
Owner

Yep, thanks for catching this.

@sdatkinson sdatkinson added priority:high Prioritize and removed unread This issue is new and hasn't been seen by the maintainers yet priority:low Low-priority issues labels Jan 22, 2025
@sdatkinson sdatkinson self-assigned this Jan 22, 2025
@sdatkinson
Copy link
Owner

This should be fixed with #533

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high Prioritize
Projects
None yet
Development

No branches or pull requests

2 participants