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.

Sign up for GitHub

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

Jump to bottom

[严重][Bug] Cores 页面性能极差 CPU高占用以及内存泄漏 #49

Closed
Tracked by #145
natsurainko opened this issue Feb 25, 2023 · 7 comments
Closed
Tracked by #145

[严重][Bug] Cores 页面性能极差 CPU高占用以及内存泄漏 #49

natsurainko opened this issue Feb 25, 2023 · 7 comments
Labels
bug Something isn't working help wanted Extra attention is needed need to fix 需要修复

Comments

@natsurainko
Copy link
Member

natsurainko commented Feb 25, 2023 •
edited
Loading

FluentLauncher错误报告：

FluentLauncher版本：2.1.x.x - 最新
主要大意：
Cores 页面性能差
是游戏崩溃的问题吗：否

主要内容：

在多核心情况下会出现CPU高占用以及内存泄漏的情况
同时每次加载核心的时间都存在几秒的延迟

部分附件：

image

这个问题严重嘛：7
@natsurainko natsurainko changed the title [Bug] [Bug] Cores 页面性能极差 CPU高占用以及内存泄漏 Feb 25, 2023
@natsurainko natsurainko changed the title [Bug] Cores 页面性能极差 CPU高占用以及内存泄漏 [严重][Bug] Cores 页面性能极差 CPU高占用以及内存泄漏 Feb 25, 2023
@natsurainko natsurainko added bug Something isn't working help wanted Extra attention is needed need to fix 需要修复 labels Feb 25, 2023
@natsurainko
Copy link
Member Author

我认为问题主要出在

ListView 子项较多时 设置ItemsSource会卡顿
游戏核心的读取和解析上
以及这东西ui的动态更新

image

@gaviny82
Copy link
Collaborator

这里是不是用ItemsRepeater就可以了，说真的不看源码我是真没想到这里ListView选的游戏核心是和主界面同步的...

@natsurainko
Copy link
Member Author

这里是不是用ItemsRepeater就可以了，说真的不看源码我是真没想到这里ListView选的游戏核心是和主界面同步的...

这个页面的逻辑写的稀碎 8PZ)9J1I~XC0DW`62990T2 这个右侧的功能栏是随着宽度的改变而改变的

@gaviny82
Copy link
Collaborator

这里是不是用ItemsRepeater就可以了，说真的不看源码我是真没想到这里ListView选的游戏核心是和主界面同步的...

这个页面的逻辑写的稀碎 8PZ)9J1I~XC0DW`62990T2 这个右侧的功能栏是随着宽度的改变而改变的

其实我觉得每个版本做成一个可以点进去的子页面比较好，这样详细信息也可以看清楚一点，然后重命名和选项两个按钮就可以省掉了，生成启动脚本也可考虑放进去，剩下的直接做成固定宽度

不过实现这个导航又是一个大坑，现在settings ui还没研究明白...

@natsurainko
Copy link
Member Author

这里是不是用ItemsRepeater就可以了，说真的不看源码我是真没想到这里ListView选的游戏核心是和主界面同步的...

这个页面的逻辑写的稀碎 8PZ)9J1I~XC0DW`62990T2 这个右侧的功能栏是随着宽度的改变而改变的

其实我觉得每个版本做成一个可以点进去的子页面比较好，这样详细信息也可以看清楚一点，然后重命名和选项两个按钮就可以省掉了，生成启动脚本也可考虑放进去，剩下的直接做成固定宽度

不过实现这个导航又是一个大坑，现在settings ui还没研究明白...

以前其实有过这样的实现
在2.0.0.8的uwp版本中用的就是单独页面
我也不知道到底是设计成Dialog还是Page比较好
两边都有人支持

@xingxing2008 xingxing2008 pinned this issue Feb 26, 2023
@gaviny82
Copy link
Collaborator

可以考虑一下这种设计，主要是 Dialog 太小了，而且不能在 Dialog 里面放下一级子页面。现在有 BreadcrumbBar 控件，做导航应该挺方便的。
image

@natsurainko natsurainko mentioned this issue Jun 11, 2023
Closed
24 tasks
@natsurainko
Copy link
Member Author

最新分支中已解决文件 等待后续合并

@natsurainko natsurainko unpinned this issue Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed need to fix 需要修复
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

145-refactor-UI-feature-PART-2 Xcube-Studio/Natsurainko.FluentLauncher
2 participants
@gaviny82 @natsurainko