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
d actual edits to first para #41
d actual edits to first para #41
Conversation
I've restored the order to 'maps' -> 'images' so we get a clearer narative as in the current existing repo; and shortened / combined a sentence, as it was repeating 'non-maps' and 'maps', so I used 'any images' instead to make it more intuitive to read. I was also going to add a few sentences giving the nice positive spin about interdisciplinary cross-pollination of image analysis, but not sure where this should go: I don't want to break the flow to the instructions, so perhaps it can go after the bullet points?
@dcsw2 Thanks, re:
Sounds good. |
Ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok!
@@ -3,7 +3,7 @@ | |||
<p align="center"> | |||
<h1>MapReader</h1> | |||
</p> | |||
<h2>A computer vision pipeline for analyzing and exploring images and maps at scale</h2> | |||
<h2>A computer vision pipeline for analyzing and exploring maps and images at scale</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kasra-hosseini this change doesn't seem to have gone through yet, but the two below on lines 33/35 have
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @dcsw2 It did, but today, I changed it to: A computer vision pipeline for exploring and analyzing images at scale
to emphasise its generalizability. However, to emphasize maps and images as two input types, the first line reads:
MapReader is an end-to-end computer vision (CV) pipeline for analyzing and exploring large collections of maps and images.
Does it make sense?
I've restored the order to 'maps' -> 'images' so we get a clearer narative as in the current existing repo; and shortened / combined a sentence, as it was repeating 'non-maps' and 'maps', so I used 'any images' instead to make it more intuitive to read.
I was also going to add a few sentences giving the nice positive spin about interdisciplinary cross-pollination of image analysis, but not sure where this should go: I don't want to break the flow to the instructions, so perhaps it can go after the bullet points?