Skip to content

Rename TrimNodeMutation to TrimGeneMutation#781

Closed
chrxh wants to merge 1 commit into
developfrom
claude/rename-trim-gene-mutation
Closed

Rename TrimNodeMutation to TrimGeneMutation#781
chrxh wants to merge 1 commit into
developfrom
claude/rename-trim-gene-mutation

Conversation

@chrxh

@chrxh chrxh commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • Renames the TrimNodeMutation struct/field/function family to TrimGeneMutation, matching the existing gene-level mutation naming convention (the mutation trims a random node from each gene, keyed by geneProbability, same pattern as DuplicateGeneMutation/DeleteGeneMutation).
  • Covers descriptor structs (TrimNodeMutationDesc), TO structs, GPU kernel structs, serialization ids, the applyMutations_trimNode kernel function, the trimNodeMetaMutationsSigma parameter, GUI labels/settings keys, and test class/case names.
  • Leaves the unrelated trimNodes(...) helper in GenomeDescEditService.cpp (genome-preview cell-count trimming) untouched — different feature, coincidental name overlap.

Test plan

  • Build (blocked locally: external/vcpkg submodule not initialized in this environment)
  • EngineInterfaceTests.exe, NetworkTests.exe, PersisterTests.exe
  • EngineTests.exe --gtest_filter=TrimGeneMutationTests.*:MetaMutationTests.*trimGene*:AccumulatedMutationTests*

🤖 Generated with Claude Code

Struct, field, function, and test names updated for consistency
(mutation operates per gene, not per node).
@chrxh

chrxh commented Jul 3, 2026

Copy link
Copy Markdown
Owner Author

Folded into #782 — combining both mutation renames into a single PR as requested.

@chrxh chrxh closed this Jul 3, 2026
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.

1 participant