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

整合三个页面:推理、模型配置、生成记录 #1

Open
wants to merge 4 commits into
base: stable
Choose a base branch
from

Conversation

sunieee
Copy link

@sunieee sunieee commented May 15, 2024

尊敬的维护者,

我在使用 GPT-SoVITS-Inference 项目时,发现了一些可以提升用户体验和使用效率的改进点,因此进行了以下改动。这些改动基于实际使用需求,目的是提供更加便利和高效的功能,希望能为项目带来积极的影响。

  1. 将“推理”和“模型配置”放在同一个app内部的不同tab处,可以同时启动:可以使用户在使用过程中更加便捷地在推理和模型配置之间切换,提高了操作效率。

image

  1. 添加一个“生成记录”栏,能够列出在推理页面的所有推理记录
  • 每个推理记录需要包括:推理用户(IP地址),推理时间,推理结果音频,推理的配置(主要是模型文件夹地址,人物角色,情感,输入文本,其他配置细节在点开该选项卡之后才展示)。并且支持通过推理用户、推理时间、模型文件夹、人物角色、情感、文本中的字符串来搜索和过滤推理记录。
  • 通过此功能,用户可以查看历史推理记录,包括推理用户、推理时间、推理结果音频、推理配置等详细信息,并支持多条件搜索和过滤。这有助于用户跟踪和管理生成的推理结果。

image

  1. 模型配置界面修改了读取逻辑:结合了从json文件中读取和自动生成json的功能,避免了json文件缺失导致的错误,并添加了一个自动生成所有json的功能,使模型配置过程更加顺畅。

image

  1. 推理界面、模型配置支持不同模型文件夹路径:自动从文件夹下寻找以trained开头的文件夹,更方便用户将模型进行归类放置和管理,提升了文件组织和管理的灵活性。

image

@X-T-E-R
Copy link
Owner

X-T-E-R commented Jun 16, 2024

抱歉这两天在准备现实世界的事情没看到您的pr;我这两天看一下,合并一下

@X-T-E-R
Copy link
Owner

X-T-E-R commented Jun 16, 2024

您好,您写的功能非常好!

但是有一些建议

  1. 不同的gradio功能还是建议分文件,从其它文件导入,例如 from webuis.model_tab import run_model_tab , 它可以设计成传入app参数,然后with app: ,这样可以使文件更加简洁,并且能够手动关闭功能

  2. 端口建议使用这个app_v2.py下面的写法,这样可以将gradio挂载到fastapi上,这样可以实现更多功能,例如:跨域,权限控制等

我合并了其它部分的代码,app_v2暂时没合并上去

我将它贴到这里app_v2.txt,您可以同步一下stable分支,稍改一下重提PR;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants