-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
**Feat: Integrate Loss Graphs into Gradio Interface** #2241
base: main
Are you sure you want to change the base?
Conversation
**Body:** * Implements helper functions to retrieve and display loss graphs for different projects. * Integrates with Gradio to provide interactive tabs with loss graph images. * Handles project selection and updates loss graphs accordingly. * Includes helper functions for generating and saving loss graphs from TensorBoard logs
- Updated `extract_scalar_data` to handle NaN values in TensorBoard logs. - If a scalar value is NaN, the method now falls back to the previous valid value. - If no previous value is available, a default of 0.0 is used. - This ensures continuity and robustness in the extracted scalar data.
…ault project name, also moved default_project_name logic into the get_projects function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot start the gradio because the path is None and the error appears. I've tried to expend your List comprehension from projects = [name for name in os.listdir(index_root) if os.path.isdir(os.path.join(index_root, name)) and name != 'mute' and os.path.isdir(os.path.join(index_root, name, 'loss_graphs'))] to projects = []
for name in os.listdir(index_root):
if os.path.isdir(os.path.join(index_root, name)) and name != 'mute':
workdir = os.path.join(index_root, name, 'loss_graphs')
if not os.path.isdir(workdir):
os.mkdir(workdir)
projects.append(name) in Line 823 of infer-web.py, and solve it. |
|
This has nothing to do with this pull request. That said, outside_index_root should be set here: https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/blob/main/.env |
I do not think this is needed, if there is nothing there it will be an empty list. |
It can start, thank you. QWQ |
Pull Request Checklist
PR Type
Description
This pull request introduces a new feature to enhance the Gradio interface by adding the ability to visualize loss graphs:
Screenshot