Add aliases to gui arguments, fix logs variable #2381
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull Request Review by G4F Copilot
Thank you, H Lohaus, for your contribution to the project!
Changes Made:
- Added aliases to GUI arguments
- Fixed the
logsvariable in the debug module
File Changes:
- Modified
g4f/debug.pyto add thelogslist variable - Modified
gui/client/static/css/style.cssto update the CSS style - Modified
g4f/gui/gui_parser.pyto add aliases for host, port, and debug arguments - Modified
g4f/gui/server/api.pyto adjust thelogsvariable and log handling
The changes look good overall. However, please ensure all changes are tested thoroughly before merging.
Thank you again for your contribution!
Best regards,
G4F Copilot
| @@ -6,6 +6,7 @@ | |||
| last_model: str = None | |||
| version: str = None | |||
| log_handler: callable = print | |||
| logs: list = [] | |||
There was a problem hiding this comment.
Consider using a more descriptive name for the variable logs to clarify its purpose.
| @@ -1060,7 +1060,6 @@ a:-webkit-any-link { | |||
|
|
|||
| .log { | |||
| white-space: pre-wrap; | |||
There was a problem hiding this comment.
Consider consolidating the font-family property values for better maintainability.
| parser.add_argument("-port", type=int, default=8080, help="port") | ||
| parser.add_argument("-debug", action="store_true", help="debug mode") | ||
| parser.add_argument("--host", type=str, default="0.0.0.0", help="hostname") | ||
| parser.add_argument_alias('-h', '--host') |
There was a problem hiding this comment.
The add_argument_alias method is not a standard method of argparse.ArgumentParser. Ensure that this method is properly defined or imported before use.
| parser.add_argument("--host", type=str, default="0.0.0.0", help="hostname") | ||
| parser.add_argument_alias('-h', '--host') | ||
| parser.add_argument("--port", type=int, default=8080, help="port") | ||
| parser.add_argument_alias('-p', '--port') |
There was a problem hiding this comment.
The add_argument_alias method is not a standard method of argparse.ArgumentParser. Ensure that this method is properly defined or imported before use.
| parser.add_argument("--port", type=int, default=8080, help="port") | ||
| parser.add_argument_alias('-p', '--port') | ||
| parser.add_argument("--debug", action="store_true", help="debug mode") | ||
| parser.add_argument_alias('-d', '--debug') |
There was a problem hiding this comment.
The add_argument_alias method is not a standard method of argparse.ArgumentParser. Ensure that this method is properly defined or imported before use.
| parser.add_argument_alias('-p', '--port') | ||
| parser.add_argument("--debug", action="store_true", help="debug mode") | ||
| parser.add_argument_alias('-d', '--debug') | ||
| parser.add_argument_alias('-debug', '--debug') |
There was a problem hiding this comment.
The alias -debug seems redundant since --debug already exists. Consider removing this alias to avoid confusion.
No description provided.