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

Review and merge with odf(?) #20

Open
mtennekes opened this issue Oct 13, 2020 · 7 comments
Open

Review and merge with odf(?) #20

mtennekes opened this issue Oct 13, 2020 · 7 comments

Comments

@mtennekes
Copy link

Hi Robin (cc @agila5 and @rogerbeecham),

Great to see your progress on the od package!

Finally, I have made some progress on 'my' od package, which I called odf. As you will see below, odf doen't have as much features as od. I have put the emphasis on the data structure, for which I used a graph structure. However, I have a different approach than I had in mind during my visit in Leeds: now I'm following a much more light-weighted approach.

Could you take a look and see if it's useful for your package? I've rendered a pdf from the vigette and also uploaded our draft paper. See also the latest version of the 'halfline' doughnut visualization, for which I've partly used odf.

I have the feeling that we can merge our approaches quite well. Let me know what you think.

@Robinlovelace
Copy link
Member

Hi @mtennekes, many thanks for raising this and the offer to join forces. I will take a look and let you know my thoughts. First impression: as stated in the README the od package has no class system but is something I would like to at least consider. The fact that you have developed a class system is very attractive and, as you say, I think the code bases are not only compatible but complimentary. Suggest that a 'minihack' one afternoon or early evening could be the best way forward on this. Suggest inviting some others - will send an email now to help make it happen!

Thanks again, this will provide a strong basis for od work in R 🚀

@mtennekes
Copy link
Author

Awesome!

@mtennekes
Copy link
Author

For time-series and/or multi-categorical OD data, an interesting approach is to use vector cubes from the stars package. See https://keen-swartz-3146c4.netlify.app/raster.html#datacubes (section 4.3.2.).

@Robinlovelace
Copy link
Member

That is a very interesting approach. I've still not fully understood the potential of that way of doing things but I imagine it opens doors, thanks for flagging it! Another approach, more on the side of visualisation, is this: https://github.com/rogerbeecham/od-flowvis-ggplot2

I recall @rogerbeecham was happy to 'donate' the relevant curve-producing code to a good cause but wonder if vis and class systems / processing should be kept separate?

@rogerbeecham
Copy link
Collaborator

rogerbeecham commented Oct 20, 2020 via email

@Robinlovelace
Copy link
Member

Next step: include functions to convert to 'edge-node' format consisting of a list of 2.

@Robinlovelace
Copy link
Member

Could also convert to sfnetwork objects, heads-up @luukvdmeer.

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

No branches or pull requests

3 participants