Skip to content

fix #347 - #350

Merged
antoineatstariongroup merged 2 commits into
developmentfrom
347-tech-debt-assemblersynchronize-is-on-89-s-to-assemble-a-50-000-element-model
Aug 25, 2026
Merged

fix #347#350
antoineatstariongroup merged 2 commits into
developmentfrom
347-tech-debt-assemblersynchronize-is-on-89-s-to-assemble-a-50-000-element-model

Conversation

@antoineatstariongroup

Copy link
Copy Markdown
Collaborator

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the SysML2.NET code style guidelines
  • I have provided test coverage for my change (where applicable)

Description

Fix #347

Provide Assembler performance improvement

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
48.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@antoineatstariongroup
antoineatstariongroup merged commit e0348c9 into development Aug 25, 2026
7 of 8 checks passed
@antoineatstariongroup
antoineatstariongroup deleted the 347-tech-debt-assemblersynchronize-is-on-89-s-to-assemble-a-50-000-element-model branch August 25, 2026 10:59
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.

[Tech Debt]: Assembler.Synchronize is O(n²) — 89 s to assemble a 50 000-element model

2 participants