[PR #12] [MERGED] startup performance improvements #14

New Issue

OMGeeky · 2025-10-27T18:43:46+01:00

OMGeeky commented

2025-10-27 18:43:46 +01:00

📋 Pull Request Information

Original PR: https://github.com/AndorsTrailRelease/ATCS/pull/12
Author: @OMGeeky
Created: 8/31/2025
Status: ✅ Merged
Merged: 9/3/2025
Merged by: @NutAndor

Base: master ← Head: performance

📝 Commits (3)

703b723 improve GameDataCategory lookup time
e2b0b0e remove not double linking
364bf8e cleanup some duplicate code

📊 Changes

9 files changed (+154 additions, -132 deletions)

View changed files

📝 src/com/gpl/rpg/atcontentstudio/model/Project.java (+16 -38)
📝 src/com/gpl/rpg/atcontentstudio/model/gamedata/GameDataCategory.java (+109 -24)
📝 src/com/gpl/rpg/atcontentstudio/model/gamedata/GameDataSet.java (+9 -49)
📝 src/com/gpl/rpg/atcontentstudio/model/gamedata/Quest.java (+0 -1)
📝 src/com/gpl/rpg/atcontentstudio/model/tools/i18n/PotGenerator.java (+6 -6)
📝 src/com/gpl/rpg/atcontentstudio/model/tools/resoptimizer/ResourcesCompactor.java (+10 -10)
📝 src/com/gpl/rpg/atcontentstudio/ui/SaveItemsWizard.java (+1 -1)
📝 src/com/gpl/rpg/atcontentstudio/ui/WorkspaceActions.java (+2 -2)
📝 src/com/gpl/rpg/atcontentstudio/ui/gamedataeditors/JSONElementEditor.java (+1 -1)

📄 Description

This reduced the startup time from ~87 seconds to about ~22 seconds in multiple tests

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AndorsTrailRelease/ATCS/pull/12 **Author:** [@OMGeeky](https://github.com/OMGeeky) **Created:** 8/31/2025 **Status:** ✅ Merged **Merged:** 9/3/2025 **Merged by:** [@NutAndor](https://github.com/NutAndor) **Base:** `master` ← **Head:** `performance` --- ### 📝 Commits (3) - [`703b723`](https://github.com/AndorsTrailRelease/ATCS/commit/703b72332235903018277a034075cbbfab935c7a) improve GameDataCategory lookup time - [`e2b0b0e`](https://github.com/AndorsTrailRelease/ATCS/commit/e2b0b0e81e363ff603af0edfab78b2d6104355d2) remove not double linking - [`364bf8e`](https://github.com/AndorsTrailRelease/ATCS/commit/364bf8ee113a927b7f1d0a7fa523c6025076e919) cleanup some duplicate code ### 📊 Changes **9 files changed** (+154 additions, -132 deletions) <details> <summary>View changed files</summary> 📝 `src/com/gpl/rpg/atcontentstudio/model/Project.java` (+16 -38) 📝 `src/com/gpl/rpg/atcontentstudio/model/gamedata/GameDataCategory.java` (+109 -24) 📝 `src/com/gpl/rpg/atcontentstudio/model/gamedata/GameDataSet.java` (+9 -49) 📝 `src/com/gpl/rpg/atcontentstudio/model/gamedata/Quest.java` (+0 -1) 📝 `src/com/gpl/rpg/atcontentstudio/model/tools/i18n/PotGenerator.java` (+6 -6) 📝 `src/com/gpl/rpg/atcontentstudio/model/tools/resoptimizer/ResourcesCompactor.java` (+10 -10) 📝 `src/com/gpl/rpg/atcontentstudio/ui/SaveItemsWizard.java` (+1 -1) 📝 `src/com/gpl/rpg/atcontentstudio/ui/WorkspaceActions.java` (+2 -2) 📝 `src/com/gpl/rpg/atcontentstudio/ui/gamedataeditors/JSONElementEditor.java` (+1 -1) </details> ### 📄 Description This reduced the startup time from ~87 seconds to about ~22 seconds in multiple tests --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OMGeeky added the

pull-request

label 2025-10-27 18:43:46 +01:00

OMGeeky closed this issue

2025-10-27 18:43:46 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: AndorsTrailRelease/archived-ATCS#14