[performance] Сократить аллокации строк в model/particle API, сохранив совместимость #30

New Issue

2026-10-29T12:24:57Z

chexit commented

2026-10-29 12:24:57 +00:00

Проверено на опубликованном 90379f6 при проверке дополнительного ревью. Незакоммиченные изменения из чужой рабочей копии недоступны.

emit() каждый тик строит новый Vec/ParticleSpawn и делает m.name.to_string()/p.kind.to_string() на каждый объект. Повторяющиеся имена моделей/частиц кодируются в owned Strings и передаются через WIT снова и снова.

Важно: HashMap<&str,String> с последующим String::clone() НЕ устраняет аллокацию на объект. Предложение из чужого ревью «просто кэшировать String по static str» в таком виде недостаточно.

Сначала измерить allocation/boundary cost в #9. Возможные этапы: переиспользование буферов и строк с capacity на стороне мода; для существенного устранения повторной передачи — версионированный API регистрации имён/моделей один раз и обновлений экземпляров с числовыми handles. Старый string API оставить совместимым. Handles инвалидировать при reload ресурсов/смене registry; интернирование должно быть ограничено реестром, а не накапливать произвольные динамические строки.

Критерий: счётчики аллокаций/переданных байт падают при массовом бое без потери моделей, света, частиц и hot reload. Native время не выдавать за фактический выигрыш FPS Mac.

Код:

		airdef/airdef-mod/src/lib.rs
		Line 387 in 90379f6
	
				    fn emit(&mut self, out: Out) {

.

Проверено на опубликованном `90379f6` при проверке дополнительного ревью. Незакоммиченные изменения из чужой рабочей копии недоступны. emit() каждый тик строит новый Vec<ModelInstance>/ParticleSpawn и делает m.name.to_string()/p.kind.to_string() на каждый объект. Повторяющиеся имена моделей/частиц кодируются в owned Strings и передаются через WIT снова и снова. Важно: HashMap<&str,String> с последующим String::clone() НЕ устраняет аллокацию на объект. Предложение из чужого ревью «просто кэшировать String по static str» в таком виде недостаточно. Сначала измерить allocation/boundary cost в #9. Возможные этапы: переиспользование буферов и строк с capacity на стороне мода; для существенного устранения повторной передачи — версионированный API регистрации имён/моделей один раз и обновлений экземпляров с числовыми handles. Старый string API оставить совместимым. Handles инвалидировать при reload ресурсов/смене registry; интернирование должно быть ограничено реестром, а не накапливать произвольные динамические строки. Критерий: счётчики аллокаций/переданных байт падают при массовом бое без потери моделей, света, частиц и hot reload. Native время не выдавать за фактический выигрыш FPS Mac. Код: https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/airdef-mod/src/lib.rs#L387.

chexit added the area:rendering needs:profiling priority:P3 type:performance labels 2026-10-29 13:11:15 +00:00

chexit referenced this issue from a commit

2026-10-29 14:02:16 +00:00

TAA and cheaper shadows in HD (patch 16), model handles in the mod API so the mod sends no model names each tick (patch 17, #30); doc updated

chexit commented

2026-10-29 14:06:03 +00:00

🤖 Agent: сделано в https://gitea.maoniao.team/chexit/airdef/commit/ab4a789 (API уровня 11: model-id, model-generation, set-models-by-id; частицы пока по строкам). CI зелёный.

chexit closed this issue

2026-10-29 14:06:03 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: chexit/airdef#30