Skip to content

perf: use concurrent instead of coroutine#146

Merged
ktro2828 merged 3 commits into
mainfrom
perf/render/speedup
May 14, 2025
Merged

perf: use concurrent instead of coroutine#146
ktro2828 merged 3 commits into
mainfrom
perf/render/speedup

Conversation

@ktro2828

@ktro2828 ktro2828 commented May 14, 2025

Copy link
Copy Markdown
Collaborator

What

This PR contributes to the speed up of the rendering processes by replacing asynchronous operations to multiprocessing.

Issue Link: #18

scene pointcloud
Before 0m3.797s 1m2.000s
After 0m2.789s 0m49.222s

@github-actions github-actions Bot added the refactor Refactoring code or increasing performance label May 14, 2025
@github-actions

github-actions Bot commented May 14, 2025

Copy link
Copy Markdown
Contributor

☂️ Python Coverage

current status: ❌

Overall Coverage

Lines Covered Coverage Threshold Status
2260 1543 68% 50% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
t4_devkit/helper/rendering.py 13% 🔴
t4_devkit/tier4.py 13% 🔴
TOTAL 13% 🔴

updated for commit: 7b69562 by action🐍

@ktro2828 ktro2828 force-pushed the perf/render/speedup branch from d33383f to f1584fa Compare May 14, 2025 08:23
ktro2828 added 3 commits May 14, 2025 20:08
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
@ktro2828 ktro2828 force-pushed the perf/render/speedup branch from f1584fa to 7b69562 Compare May 14, 2025 11:18
@ktro2828 ktro2828 marked this pull request as ready for review May 14, 2025 11:19
@ktro2828 ktro2828 merged commit ef4541e into main May 14, 2025
5 checks passed
@ktro2828 ktro2828 deleted the perf/render/speedup branch May 14, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactoring code or increasing performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant