Skip to content

Conversation

@rchristie
Copy link
Member

Built from ellipsoid octants, with 1 row of nodes on inner hilum common to all lobes on each lung.

Copy link
Contributor

@mlin865 mlin865 left a comment

Choose a reason for hiding this comment

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

Scaffold looks great and passed all tests. I've made couple of minor suggestions and also pending your fixes to the two issues discussed today:

  1. issues with edge related annotation groups when using refinement
  2. broken mesh when "lower lobe extension" = 0.0

Are you planning to add markers that are in older versions of lung to lung 4?

@rchristie
Copy link
Member Author

Scaffold looks great and passed all tests. I've made couple of minor suggestions and also pending your fixes to the two issues discussed today:

  1. issues with edge related annotation groups when using refinement
  2. broken mesh when "lower lobe extension" = 0.0

Are you planning to add markers that are in older versions of lung to lung 4?

We're not fixing any marker points -- the apex location is a bit amorphous and everything else is tied down by the surfaces and edges already.

@rchristie
Copy link
Member Author

I did add markers which are not used for fitting but in subsequence PCA steps.
Ready for final review.

@rchristie rchristie requested a review from mlin865 December 10, 2025 22:36
Copy link
Contributor

@mlin865 mlin865 left a comment

Choose a reason for hiding this comment

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

All good and passed all tests. Just one typo to fix and good to go!

tmp_line_ids.append(line.getIdentifier())
face_line_ids.append(tmp_line_ids)
new_line_ids = set()
# if there is a single line between 2 faces can do less work later, but not if there are collpased faces
Copy link
Contributor

Choose a reason for hiding this comment

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

correct typo - collapsed

@mlin865
Copy link
Contributor

mlin865 commented Dec 11, 2025

@rchristie - Sorry i just noticed that the markers disappear when I am making the right lung only. Can you please resolve that issue?

Put marker nodes in left/right lung groups too.
Copy link
Contributor

@mlin865 mlin865 left a comment

Choose a reason for hiding this comment

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

Works fine now. Thanks for making the changes

@rchristie rchristie merged commit bb99598 into ABI-Software:main Dec 11, 2025
9 checks passed
@rchristie rchristie deleted the lung4 branch December 11, 2025 03:51
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