Skip to content

Mutate Patch Action#214

Open
allison-li-1016 wants to merge 5 commits intomainfrom
allison/mutate-action
Open

Mutate Patch Action#214
allison-li-1016 wants to merge 5 commits intomainfrom
allison/mutate-action

Conversation

@allison-li-1016
Copy link
Copy Markdown
Contributor

Estimated review time: medium

Description of functionality: The PR adds a new action into Patch that converts all healthy tissue cells within a given radius into cancer cells.

Summary of changes: new Mutate action file

Discussion points:

  1. I created this action to simulate a very specific use case for my T cells. Is it worth merging this into the main model? Do we see a functionality for this in the future?
  2. Given the above point, should we add more functionality for this action? (e.g. "RATIO" parameter to convert a fraction of cells instead of all the cells, etc)

@allison-li-1016 allison-li-1016 requested review from a team, Jannetty, cainja, daniellevahdat, jacob-evarts, jessicasyu, kristaphommatha and pohaoc2 and removed request for a team March 23, 2026 21:59
Copy link
Copy Markdown
Contributor

@kristaphommatha kristaphommatha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I think it just needs some documentation and name changes for consistency.

@allison-li-1016
Copy link
Copy Markdown
Contributor Author

@allison-li-1016 add some tests if we are going to merge it in

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.

2 participants