-
Notifications
You must be signed in to change notification settings - Fork 192
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
Computer setup: proposed configure command does not include profile #4679
Comments
can i work on this |
Hello @ltalirz , I would like to work, but while setting up the environment I'm getting this error, will you please help me out.
|
Hey @lkbhitesh07 Maybe try pypa/setuptools#1694 (comment) I haven't heard of this in a while but we might want to have a look back at changing the pyproject.toml setup. Also, for next time please consider opening a new issue in case you encounter problems like this (since it is not related to this particular thread). Cheers! |
Hello @ltalirz, As far as I understand that we need to check the setup command, is it running on the default aiida profile or another profile but I'm confused on how to approach this particular issue, where should we look at to filter if we need default aiida profile or other. |
hey @lkbhitesh07 , I believe this is available from the "context", which is also passed to the "computer setup" command.
|
I'm trying to figure it out for past one hour that what's happening and how to fix it but unfortunately I'm not being able to do it, sorry for that. Incase @ltalirz you get some time will you please explain me what should I do as this codebase is totally new. |
Fixes aiidateam#4679 When setup computer for not the current profile by `-p <profile>`, the proposed message for computer configure command does not contain this profile name, which lead to the copy paste of the proposed message for configure commad not apply to the correct profile. We read the profile name from ctx.obj and pass it to the proposed configere message. - Some of unittests in test_computer.py are moved to pytest.
Is your feature request related to a problem? Please describe
When configuring a new computer, the command proposes to configure the computer next
However, this command is only correct, if the computer is to be configured on the default AiiDA profile.
Describe the solution you'd like
In order to be able to just copy-paste this command, there should be a check on whether the
setup
command is being run on the default AiiDA profile, and the proposed command should provide the AiiDA profile parameter (if needed).The text was updated successfully, but these errors were encountered: