Skip to content

Ripples server integration with namespace changes#429

Merged
yatisht merged 30 commits into
yatisht:masterfrom
pgangwar-ucsd:ripples-server-tbb-update
Apr 29, 2026
Merged

Ripples server integration with namespace changes#429
yatisht merged 30 commits into
yatisht:masterfrom
pgangwar-ucsd:ripples-server-tbb-update

Conversation

@pgangwar-ucsd

Copy link
Copy Markdown
Contributor

This PR separates the namespaces for matOptimize::Mutation_Annotated_Tree and Mutation_Annotated_Tree. This allows us to integrate the usher-server codebase (which uses the matOptimize version) with Ripples.

Key Changes:

  1. src/mutation_annotated_tree.hpp: Made all_nodes public and added preallocated_nodes to enable faster tree copy from one namespace to another.
  2. src/ripples/util/copy/mat_copy_helper.cpp: Contains code for copying trees from matOptimize::Mutation_Annotated_Tree to Mutation_Annotated_Tree
  3. src/usher-sampled/driver/socket.cpp: Added the Ripples server code (around line 500).

Note:
Currently, the Ripples code runs after place_sample_sequential and final_output every time a new sample is added. We might need to find a better place for its integration.

mrkylesmith and others added 30 commits November 17, 2025 13:00
… and link against TBB::tbbmalloc in previous commit for usher-server binaries using scalable allocator C interface
@AngieHinrichs

Copy link
Copy Markdown
Contributor

Great work, it's looking good here. While I develop the front end for this, I will probably have some additional requests (or might just submit some follow-up minor PRs myself) but this is enough for me to start developing with.

@yatisht yatisht merged commit 9d7ecf7 into yatisht:master Apr 29, 2026
2 checks passed
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.

4 participants