Skip to content

Strip down to core simulation functionality - #56

Merged
gyanezfeliu merged 13 commits into
masterfrom
feature/tim/repo-split
Oct 9, 2026
Merged

gyanezfeliu merged 13 commits into
masterfrom
feature/tim/repo-split

Conversation

@timrudge

@timrudge timrudge commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Splitting CellModeller into a base system CellModeller and plug in components here. The cmcomponents package will be pinned to a corresponding CellModeller version here.

NB: for existing models the import paths to components such as biophysics, integration etc. should point to the cmcomponents package. E.g.

from CellModeller.Biophysics.BacterialModels import CLBacterium

becomes

from cmcomponents.Biophysics.BacterialModels import CLBacterium

All example models have been updated and should work.

@timrudge
timrudge requested a review from gyanezfeliu October 7, 2026 08:40
@timrudge timrudge self-assigned this Oct 7, 2026
gyanezfeliu
gyanezfeliu previously approved these changes Oct 8, 2026

@gyanezfeliu gyanezfeliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One fix in the description, the migration note should read:

from cmcomponents.biophysics.BacterialModels.CLBacterium import CLBacterium

Tested the change on Linux.

@timrudge
timrudge requested a review from gyanezfeliu October 8, 2026 22:58

@gyanezfeliu gyanezfeliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All working, checked in two Linux machines.

@gyanezfeliu
gyanezfeliu merged commit e31b949 into master Oct 9, 2026
1 check 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.

2 participants