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

Demo monolayer growth #278

Merged
merged 22 commits into from
Sep 30, 2022
Merged

Demo monolayer growth #278

merged 22 commits into from
Sep 30, 2022

Conversation

nicogno
Copy link
Contributor

@nicogno nicogno commented Sep 27, 2022

Added monolayer growth demo that simulates the growth of a 2D monolayer as the cells proliferate. Cells go through a whole cycle (G0/G1 -> S -> G2 -> M) and duplicate when their volume has doubled. The growth of the monolayer replicates the experimental data shown in Figure 1 from Drasdo and Hoehme (2005).

@nicogno nicogno added WIP Work in progress examples Add / remove / change examples labels Sep 27, 2022
Copy link
Contributor

@TobiasDuswald TobiasDuswald left a comment

Choose a reason for hiding this comment

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

Great contribution @nicogno! I marked a few, mostly cosmetic things that would be good to address before merging into the master. I've also added a few commits earlier to remove results and slightly simplify the IO with time series.

Other than that it looks really, really good, let's fix these small things and off to master it goes :)

demo/monolayer_growth/src/cycling_cell.h Show resolved Hide resolved
demo/monolayer_growth/src/cycling_cell.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/cycling_cell.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/monolayer_growth.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/sim_param.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/behaviours.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/behaviours.h Show resolved Hide resolved
demo/monolayer_growth/src/behaviours.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/behaviours.h Outdated Show resolved Hide resolved
demo/monolayer_growth/src/behaviours.h Outdated Show resolved Hide resolved
nicogno and others added 7 commits September 28, 2022 16:00
Adding minor comments.
Added brief comments for what each Setter and Getter is for. Additionally, removed the Vrel setter and getter along with its data members as I believe this is an artefact from the cell cycle specific unit test.
Removed Vrel as it is now redundant.
@BJackal
Copy link
Contributor

BJackal commented Sep 28, 2022

Should we also include a .toml file so that a user of the demo can directly see the simulation outputs ? I dont currently see one and think its a bit of a shame as the simulation itself looks very nice.

@TobiasDuswald
Copy link
Contributor

Should we also include a .toml file so that a user of the demo can directly see the simulation outputs ? I dont currently see one and think its a bit of a shame as the simulation itself looks very nice.

Are you thinking of turning on the ParaView export? Is that what you're saying? I think that would be nice. But we planned on deprecating toml in favour of the already present json format so let's use the json file or directly turn it on via the source code in set_param [...] param->export_visualization=true.

@BJackal
Copy link
Contributor

BJackal commented Sep 29, 2022

Thanks for all the changes Tobias, after passing is this ready to go through or do we still need a few changes ?

@BJackal
Copy link
Contributor

BJackal commented Sep 29, 2022

Should we also include a .toml file so that a user of the demo can directly see the simulation outputs ? I dont currently see one and think its a bit of a shame as the simulation itself looks very nice.

Are you thinking of turning on the ParaView export? Is that what you're saying? I think that would be nice. But we planned on deprecating toml in favour of the already present json format so let's use the json file or directly turn it on via the source code in set_param [...] param->export_visualization=true.

If we are depreciasting toml then yes json would be best.

change delta t to iterate by one each time step.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@TobiasDuswald TobiasDuswald merged commit 17257e4 into master Sep 30, 2022
@TobiasDuswald TobiasDuswald deleted the demo_monolayer_growth branch September 30, 2022 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Add / remove / change examples WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants