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

Right and accessory lungs #118

Merged
merged 2 commits into from
Feb 11, 2021

Conversation

arti-sukasem
Copy link
Contributor

the node and element functions (i.e. getLungNodes, getLungElements, getDiaphragmaticLungNodes, getDiaphragmaticLungElements) for both right, left and accessory lungs including its parameters were added into the human and mouse sections (excluding the left mouse lung)

the node and elements function for the accessory lung including its parameters were added into the mouse section
Copy link
Member

@rchristie rchristie left a comment

Choose a reason for hiding this comment

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

Refactoring into functions is really improving this code... well done.
Some fixes needed, especially the inconsistency between 2 pairs of elements in the diaphragmatic lobe which I sent a graphic of on Slack; I think I've marked the offending lines in the code review.

src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
src/scaffoldmaker/meshtypes/meshtype_3d_lung1.py Outdated Show resolved Hide resolved
…90-91), changed the mouse parameters to 3.5f
@rchristie rchristie merged commit f112653 into ABI-Software:master Feb 11, 2021
@arti-sukasem arti-sukasem deleted the accessoryLobeLung branch April 13, 2022 22:22
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