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

Improvements in the cell division demo #401

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

vasvav
Copy link
Contributor

@vasvav vasvav commented Feb 16, 2025

This PR concerns fixing and improving the demo that demonstrates the division of agents (representing biological cells in this example).

  • highlighted the use of the pertinent model parameters for cell division, e.g., growth rate and maximum cell diameter to signal division;
  • added the corresponding input parameter file that is usually represented by a "bdm.toml" file;
  • provided a new demo, called as cell_division_enhanced, which builds on the existing cell_division one while it adds a bit more complexity to it by adding a user-defined "cell" type that has a dedicated way of multiplying (i.e., cell division).
  • added ReadMe files that outline each demo - they provide important guidance how to run the demos and what to expect in the end.

@TobiasDuswald
Copy link
Contributor

I'd rename ReadMe.md to README.md for consistency across the code base. Very much like the idea of having a README. Are they copied with the bdm demo cell-division command?

@TobiasDuswald
Copy link
Contributor

@UmarAbubacar @sportokalidis 🔔 can you review this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants