Added README file and image files it references #541
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Lreg.png and LSeg.png are those image files. README is a first draft and under review.
The previous method made strikethrough text.
…magnetic resonance–derived left ventricular mass to be a link.
Attempting to have the images render correctly.
…Variant. Additionally removed unnecessary linebreak.
I wasn't sure if it made sense in the README.
"Machine learning enables new insights into clinical significance of and genetic contributions to liver fat accumulation" to be exact. I believe I have found the specific paper about this model.
…c Magnetic Resonance-Derived Left Ventricular Mass and Hypertrophy from 12-Lead Electrocardiograms" The paper links directly back the ML4H Github.
…mages" a link to the article describing the model.
…aper on this model. The heading of this README was not close to the title of the paper. So I decided to put the link in the citation instead.
I may have had it link to a very similar article from the same authors.
…article. I can't tell if the names of the same article are different, or they're two different but very similar articles.
…student_and_mri_teacher. Moved description of each model to their own line for easier reading. Also changed some commas to periods. Also made the sentence "Three pre-trained models are included here" end in a colon.
…entricular_mass Put some sentences on the same line, changed the tense to present tense in the 4th line, and removed the line from the original paper pointing the reader back to this Github.
It was not working right before.
…the image in the README link directly to the added local file.
…from_ecg_student_and_mri_teacher/README.md
… down the alt text of the images. (cardiac_mri_derived_left_ventricular_mass/README.md) Please correct me if "the models were trained with the InlineVF contours" and "we trained with the InlineVF" are not equivalent statements.
The README of cardiac_mri_derived_left_ventricular_mass to be exact.
architecture_graph_sax_diastole_segment_no_flat.png is not returning.
All images which matches it's description "An overview of the models layers" were already in the results section.
Deleting test image with git rm looks very different from using git rm on an image tracked with git lfs.
This appears to be the version on Github despite not being on my local machine...
The capital S one shouldn't be there. But doing this removes Lseg with a lower case L.
git rm was not run.
lucidtronix
approved these changes
Oct 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Lreg.png and LSeg.png are those image files. README is a first draft and under review.