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

Generalize continuum integration #260

Merged
merged 39 commits into from
Oct 4, 2022
Merged

Generalize continuum integration #260

merged 39 commits into from
Oct 4, 2022

Conversation

TobiasDuswald
Copy link
Contributor

@TobiasDuswald TobiasDuswald commented Apr 5, 2022

Add a high-level class interface to BioDynaMo that allows attaching time-dependent continuum models of various types, e.g analytical, third-party packages, or our FD scheme DiffusionGrid. By default, the interface uses the same time steps as the agent simulation via param->simulation_time_step but it's possible to set and set arbitrary time steps for the continuum model. Thus it's possible to realize continuum models with explicit (typically small dt), implicit (typically large dt), and agent-based models with arbitrary time steps. Continuum time steps can both be larger and smaller than the agent updates. Explicit time integration schemes often come with stability conditions (e.g. EulerGrid) requiring a certain time resolution. Setting the time step directly rather than some frequency of the diffusion operation gives additional freedom and makes it easier to satisfy the stability conditions (e.g. if you vary the agent-time scale param->simulation_time_step the stability remains unaffected if we have fixed dt)

@TobiasDuswald TobiasDuswald changed the title WIP: generalize continuum integration Generalize continuum integration Apr 14, 2022
@TobiasDuswald
Copy link
Contributor Author

TobiasDuswald commented Apr 28, 2022

  • ! Before merging, discuss 30113e0

@sonarcloud
Copy link

sonarcloud bot commented Apr 28, 2022

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 6 Code Smells

86.0% 86.0% Coverage
0.0% 0.0% Duplication

@TobiasDuswald TobiasDuswald added api-change Changes affecting the API WIP Work in progress examples Add / remove / change examples bdm-core Changes in BDM core labels Aug 29, 2022
@TobiasDuswald TobiasDuswald added ready Ready for merge / review and removed WIP Work in progress labels Oct 3, 2022
@sonarcloud
Copy link

sonarcloud bot commented Oct 3, 2022

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 12 Code Smells

86.8% 86.8% Coverage
0.0% 0.0% Duplication

@TobiasDuswald
Copy link
Contributor Author

[x] ! Before merging, discuss 30113e0

Fixed in 11caf04

@TobiasDuswald TobiasDuswald merged commit 4d366a9 into master Oct 4, 2022
@TobiasDuswald TobiasDuswald deleted the continuum-api branch October 4, 2022 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-change Changes affecting the API bdm-core Changes in BDM core examples Add / remove / change examples ready Ready for merge / review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant