Skip to content

Performace improve #93

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
Merged
mkh-user merged 16 commits into from
Aug 26, 2025
Merged

Performace improve #93

mkh-user merged 16 commits into from
Aug 26, 2025

Conversation

@mkh-user
Copy link
Member

@mkh-user mkh-user commented Aug 23, 2025 •
edited
Loading

Type of Change

  • New feature
  • Refactoring

Description

Adds some performance tests and improvements for better performance.

Testing

Nothing

Impact

Environment

Hardware Information Software information
CPU Intel Core i5-102101J CPU @ 1.60GHz OS Windows 11 24H2
RAM DDR4 32GB 2667 MT/s Power Mode Best Performance
Disk SSD (NVMe) Godot v4.4.1.stable.official [49a5bc7b6]
GPU NVIDIA GeForce MX350 2GB Build editor

Before

Param Unit Min Max Avg Repeats
Startup Time msec 2127 2178 2158 10
Time to Open File* (First: 26) msec 24 28 26.8 10
Time to Open File (cached html mode) msec 8 21 9.3 10
Type Delay msec 1.231 8.776 5.040 1000
Search Delay msec 2 23 17.4 20
Param Unit Min Max Avg
CPU Usage (Stand by) % 5.4 15.6 9.1
CPU Usage (Heavy load) % 14.2 19.3 15.4
RAM Usage (Stand by) MB 400.7 514 446.8
RAM Usage (Heavy load) MB 451.6 800.3 473.3
Memory leak MB ----- ----- 0

After

Param Unit Min Max Avg Repeats
Startup Time msec 278 284 281.4 10
Action Scripts Loading Time (background) msec 1035 1095 1065.8 10
Total Startup Time (with background) msec 1315 1374 1347.2 10
Time to Open File* (First: 26) msec 21 26 23.8 10
Time to Open File (cached html mode) msec 6 17 9.4 10
Type Delay msec 0.609 5.534 1.174 1000
Param Unit Min Max Avg
CPU Usage (Stand by) % 2.9 7.6 4.1
CPU Usage (Heavy load) % 13.7 16.4 14.6
RAM Usage (Stand by) MB 374.2 425.8 404.5
RAM Usage (Heavy load) MB 400.0 532.8 408.9
Memory leak MB ----- ----- 0
  • Switch between .ini & .md files.

Additional Information

Nothing yet

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings
  • Changes were added to CHANGELOG

@mkh-user mkh-user self-assigned this Aug 23, 2025
@mkh-user
Copy link
Member Author

mkh-user commented Aug 24, 2025 •
edited
Loading

Todo:

  • Make a load_threaded in Global
  • Make a load_files_threaded in Global to load action scripts

@mkh-user mkh-user marked this pull request as ready for review August 24, 2025 09:47
@mkh-user

This comment was marked as resolved.

Sign in to view
@mkh-user
Copy link
Member Author

mkh-user commented Aug 26, 2025

After caching the translation, the startup time always remains less than 250 ms.

@mkh-user mkh-user merged commit c3ce6b9 into Main Aug 26, 2025
1 check passed
@mkh-user mkh-user deleted the performace-improve branch August 26, 2025 08:12
@mkh-user mkh-user added this to the Text Forge 0.1 milestone Aug 26, 2025
@mkh-user mkh-user mentioned this pull request Nov 30, 2025
Merged
6 tasks
@mkh-user mkh-user mentioned this pull request Dec 30, 2025
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mkh-user mkh-user

Labels

None yet

Projects

Release: Text Forge 1.0
Status: Completed

Milestone

Text Forge 0.1

Development

Successfully merging this pull request may close these issues.

2 participants

@mkh-user