Skip to content

Fix slow performance on second and subsequent application startups#194

Merged
Neutree merged 1 commit intoNeutree:masterfrom
wanstarge:master
Apr 24, 2026
Merged

Fix slow performance on second and subsequent application startups#194
Neutree merged 1 commit intoNeutree:masterfrom
wanstarge:master

Conversation

@wanstarge
Copy link
Copy Markdown

软件针对Windows 将 config.json 和 run.log 放到当前工作目录。这会触发如下问题#192 ,导致二次启动显著变慢,我修改了软件的配置以及日志保存到AppData目录下,这也是Windows所推荐的,后续未再遇见此问题

  1. 目录不可写或受控时的权限/虚拟化(Program Files/UAC);
  2. 杀软/Defender 对可执行目录下频繁写入文件的深度扫描;
  3. 关闭时写入→立即再次启动时,上一轮写入文件的扫描/锁释放叠加,

Copilot AI review requested due to automatic review settings April 23, 2026 07:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses the Windows “slow second startup” issue (#192) by moving configuration and runtime log files out of the application’s current working directory (which can be under Defender/AV scanning and UAC virtualization) and into the user’s AppData directory, aligning with Windows-recommended storage locations.

Changes:

  • On Windows, switch config.json storage from the current working directory to %APPDATA%/COMTool.
  • As a result, run.log is also written under the same AppData directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread COMTool/parameters.py Outdated
Comment thread COMTool/parameters.py
Comment thread COMTool/parameters.py Outdated
@wanstarge wanstarge force-pushed the master branch 2 times, most recently from 8f8c2e9 to 9b7465e Compare April 23, 2026 08:14
Copy link
Copy Markdown
Author

@wanstarge wanstarge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the current code has achieved maximum compatibility

Comment thread COMTool/parameters.py
@Neutree Neutree merged commit 4b1e39e into Neutree:master Apr 24, 2026
@Neutree
Copy link
Copy Markdown
Owner

Neutree commented Apr 24, 2026

真棒!

@wanstarge
Copy link
Copy Markdown
Author

真棒!

没想到困扰很久的问题,在这发现原因了,哈哈哈,我之前用sscom的时候,怀疑过amd还是intel芯片的问题,也怀疑过是一些缓存,但是看不到源码只能将就着用,没想到在这里看到解决办法了,你也是,太强啦,共筑开源生态,祝你有美好的一天

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants