ecc: introduces the vascular proliferation characteristic#9
ecc: introduces the vascular proliferation characteristic#9
Conversation
1a6e690 to
a7d1316
Compare
5fadca1 to
3915ae1
Compare
|
state: proposed Few things:
|
fc80b55 to
ffb8ac0
Compare
5506c1a to
cac277d
Compare
|
@Ssandor13 @mcrusch @claymcleod. I added a binary true/false value to the characteristic based on the question: "Is vascular proliferation (VP) observed, and what features indicate this?" That said, based on Stephanie’s second comment (#9 (comment)), should the different types of VP go under the values section instead, which would make the values section the sub-characteristics of VP or should they be treated as separate ECCs? |
ecc/morph/ECC-MORPH-000001.yml
Outdated
There was a problem hiding this comment.
Vascular proliferation is observed in the ???.
ecc/morph/ECC-MORPH-000001.yml
Outdated
ecc/morph/ECC-MORPH-000001.yml
Outdated
There was a problem hiding this comment.
Is there any concrete measurement that we can reference or grade here? Or is it just totally up to the pathologist's experience and opinion?
There was a problem hiding this comment.
There are some concrete metrics like microvessel density (MVD) and Ki-67 proliferation index that can be used, especially in research or tumor grading. But in routine pathology, it's usually a semi-quantitative call—based on pattern recognition and experience, with numbers used when more precision is needed.
ecc/morph/ECC-MORPH-000001.yml
Outdated
There was a problem hiding this comment.
Perhaps there should be mention of tissue-specific definitions of "within normal limits"? That would be something I would assume, but could not be true.
claymcleod
left a comment
There was a problem hiding this comment.
We should also address the failing CI issues below.
30f7257 to
8fb7a5c
Compare
This pull request adds the Vascular Proliferation ECC
Close #7
Before submitting this PR, please make sure:
CHANGELOG.md(see "keep a changelog" for more information).