Skip to content
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

[WIP] Experimental adjacency graph method + perimeter #157

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

asinghvi17
Copy link
Member

@asinghvi17 asinghvi17 commented Jun 13, 2024

Currently in docs/src/experiments/adjacency_graph.jl.

This PR also adds a perimeter method that can have several algorithms - linear, geodesic, and thumb. That is still WIP.

The adjacency matrix calculation "works" - it's unidirectional from source to target.

Cool pictures

download-7

graph_tightening.mp4
lg_touches_lg_valid.mp4

@asinghvi17
Copy link
Member Author

cc @00krishna

@asinghvi17
Copy link
Member Author

A visualization like the NYT Olympic medal visualization would be pretty easy to implement using this method....

@asinghvi17
Copy link
Member Author

@00krishna would you want this as like a 2-stage thing, where you can opt to either get the whole row for source and target or just the geometries?

@00krishna
Copy link

Hey @asinghvi17 sorry for not getting back to you soon, just been kinda swamped.
I have been playing around with this code to get a feel for it. Hmm interesting idea
about the 1 or 2 stage thing. Yeah the 2 stage things sounds better, in the sense that the
user can get either the source-target edgelist, and/or the geometries. Sometimes the user just wants the edgelist to work with in a different program, while in other cases the user wants the geometries to plot. So having that option is nice.

As a caveat, of course there are ways to get the edgelist from the geometries, but users who are less experience will probably just end up posting lots of questions on how to do that. So easier to save ourselves some trouble :).

@asinghvi17
Copy link
Member Author

True, I guess there could be multiple entry points to the function then. Will take a look at how that could be done potentially.

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