Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conserved quantities are transferred by default during phagocytosis #308

Merged
merged 8 commits into from
Oct 15, 2024

Conversation

MathCancer
Copy link
Owner

In response to Issue #307, cells by default ingest 100% of internalized substrates and conserved variables when eating another cell.

In response to Issue #307, cells by default ingest 100% of internalized substrates when eating another cell.
In response to Issue #307, cells by default ingest 100% of conserved custom variables when eating another cell.
update version number to 1.14.1-development
In response to Issue #307, cells by default ingest 100% of internalized substrates when eating another cell.

(Note that the values for std::vector<bool> fraction_tranferred_when_ingested in Basic_Agent is *overwritten* by constructors in the cell Phenotype, so we have ot do it there.)
// so that each daughter cell gets half of the original ;
for( int nn = 0 ; nn < custom_data.variables.size() ; nn++ )
{
custom_data.variables[nn].conserved_quantity = true;
Copy link
Collaborator

@drbergman drbergman Oct 14, 2024

Choose a reason for hiding this comment

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

I think we want to delete this line. we don't want to change the boolean value here.

Copy link
Owner Author

Choose a reason for hiding this comment

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

oof, right. I think I was forcing it during testing since it wasn't being parsed.

Copy link
Owner Author

Choose a reason for hiding this comment

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

did the change, thank you!

MathCancer and others added 3 commits October 15, 2024 09:08
remove testing line that set all custom variables to `conserved = true` that was mistakenly left in.

Thanks, Daniel!
fixed the comment
@heberlr
Copy link
Collaborator

heberlr commented Oct 15, 2024

I changed the comment about the conservation for phagocytosis. Sorry about that second commit because I accidentally changed two lines in the first one.

@MathCancer MathCancer merged commit f15cefe into development Oct 15, 2024
208 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.

3 participants