-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add online demo in gh-pages #14
Comments
@jywarren I've rendered the above factors and made some changes to the projection idea on the gh-pages deployment. I've actually tried to show each matched point in one image wrt the other but overlapping them would have made it seem less discernable and while core actually "matches" the images, overlapping or "stitching" the images would result in images overlapped but shifted over each other (as you've seen in the sticher module) so I was unsure if that's what we needed here. However, if you want to show the stitching part in this as an example, we can probably drop a link of the LDI gh pages after merging the core into it? |
This looks awesome! Wouldn't it be better to use two images of actually the
same thing, not just 2 sunflowers? Like, two images that overlap from say
https://mapknitter.org/maps/ceres--2/
https://s3.amazonaws.com/grassrootsmapping/warpables/306187/DJI_1207.JPG
and
https://s3.amazonaws.com/grassrootsmapping/warpables/306188/DJI_1205.JPG
(maybe scaled down?)
This is a fantastic demo! Great work!
I think the stitcher/projector isn't urgent to show in the demo here. We
can, but there are plenty of other things to prioritize as well. Thanks!
…On Sun, Aug 18, 2019 at 8:33 AM Pranshu Srivastava ***@***.***> wrote:
@jywarren <https://github.com/jywarren> I've rendered the above factors
and made some changes to the projection idea on the gh-pages deployment
<https://publiclab.github.io/matcher-core/>.
I've actually tried to show each matched point in one image wrt the other
but overlapping them would have made it seem less discernable and while
core actually "matches" the images, overlapping or "stitching" the images
would result in images overlapped but shifted over each other (as you've
seen in the sticher module) so I was unsure if that's what we needed here.
However, if you want to show the stitching part in this as an example, we
can probably drop a link of the LDI gh pages after merging the core into it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14?email_source=notifications&email_token=AAAF6J2BODMZBGEJPZVY5UDQFE6QHA5CNFSM4IJCVFPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4Q7DAA#issuecomment-522318208>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAF6JZ6Y6Y5PHOUKBYDXZDQFE6QHANCNFSM4IJCVFPA>
.
|
@jywarren No need to downscale, core's inner mechanisms will take care of that. For the images above, I've redesigned the demo, so let me know what you think! Thanks! |
it's amazing. i love it!
Let's meet up in
publiclab/Leaflet.DistortableImage#312 and
publiclab/image-sequencer#1198 for next steps, then?
…On Mon, Aug 19, 2019 at 7:20 PM Pranshu Srivastava ***@***.***> wrote:
@jywarren <https://github.com/jywarren> No need to downscale, core's
inner mechanisms will take care of that. For the images above, I've
redesigned the demo, so let me know what you think! Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14?email_source=notifications&email_token=AAAF6J66F4KHM55J47DVHZDQFMTCXA5CNFSM4IJCVFPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4USLZQ#issuecomment-522790374>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAF6JYYBNO5TJKFZJGPKWLQFMTCXANCNFSM4IJCVFPA>
.
|
Sure thing! Closing this one now. |
This can be something really simple, which:
This will provide a substantial guide to how to implement #5 and #11
Thanks!
The text was updated successfully, but these errors were encountered: