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

the main_args is undefined in huixiangdou/server.py #364

Closed
leagend opened this issue Aug 21, 2024 · 5 comments
Closed

the main_args is undefined in huixiangdou/server.py #364

leagend opened this issue Aug 21, 2024 · 5 comments

Comments

@leagend
Copy link
Contributor

leagend commented Aug 21, 2024

error log | 日志或报错信息 | ログ

python3 -m huixiangdou.server
Traceback (most recent call last):
File "/root/.conda/envs/huixiangdou/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/root/.conda/envs/huixiangdou/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/group_share/jing/backup/huixiangdou/huixiangdou/server.py", line 135, in
start_llm_server(config_path=main_args.config_path)
NameError: name 'main_args' is not defined

context | 编译/运行环境 | バックグラウンド

python 3.10

how to reproduce | 复现步骤 | 再現方法

  1. go to source code https://github.com/InternLM/HuixiangDou/blob/main/huixiangdou/server.py#L130-L135
if __name__ == '__main__':
    args = parse_args()
    # start service
    if args.standalone is True:
        # hybrid llm serve
        start_llm_server(config_path=main_args.config_path)   <-- the main_args here is undefined

more | 其他 | その他

It failed the command line"python3 -m huixiangdou.server "

@tpoisonooo
Copy link
Collaborator

Thanks and the bug fixed, please feel free to send a Pull Request.

@tpoisonooo
Copy link
Collaborator

@leagend

@leagend
Copy link
Contributor Author

leagend commented Aug 21, 2024

Sure, #365 is the corresponding Pull Request.

@tpoisonooo
Copy link
Collaborator

merged.

@leagend
Copy link
Contributor Author

leagend commented Aug 21, 2024

close the issue as fixed.

@leagend leagend closed this as completed Aug 21, 2024
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

No branches or pull requests

2 participants