Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
make sysinfo more robust
change
--dump-sysinfo
some info are missingModuleNotFoundError: No module named 'psutil'
and so--dump-sysinfo
failedshared
andextensions
into function level inputshared_cmd_options
overshared.cmd_opts
forcmd_opts
git -C <webui_rooot> status
pip freeze --all
withimportlib.metadata
as fallback for listing pacages"triton @ https://github.com/aaa/bbb/releases/download/v1/xxx.whl#sha256=abcde",
importlib.metadata
becausepkg_resources
is being deprecated/sysinfo-????-??-??-??-??.json
--dump-sysinfo
idea
including
ui-config.json
might be valuable as welldiscussion
I think the term
sysinfo
is confusing in boath the UI and in theissue post template
I belive this increases the chance of those who don't read carefully to just onput something like
as opposed to a generating and uploading the sysinfo we want
I think something like
diagnosis info
debug info
might work betterChecklist: