Skip to content
This repository was archived by the owner on Oct 15, 2021. It is now read-only.

[WIP] Add a network diagram #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Add a network diagram #5

wants to merge 1 commit into from

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Jul 30, 2018 •
edited
Loading

screen shot 2018-07-30 at 3 22 37 pm

@jcoyne jcoyne added the review label Jul 30, 2018
@mjgiarlo
Copy link
Member

This was rescued from the rialto-frontend repo which will be deleted momentarily. Not sure if use cases require this visualization, so leaving this WIP PR open for now. Feel free to disregard for the time being.

@mjgiarlo mjgiarlo changed the title Add a network diagram [WIP] Add a network diagram Jul 30, 2018
@mjgiarlo mjgiarlo removed the review label Jul 30, 2018
@jcoyne
Copy link
Contributor Author

jcoyne commented Aug 2, 2018

@mjgiarlo this is still specified here: https://github.com/sul-dlss/rialto/wiki/Web-App-Specs#visualizations

@mjgiarlo
Copy link
Member

mjgiarlo commented Aug 2, 2018

Thanks, @jcoyne. Then I wonder:

  • How high of a priority is this visualization at this point, @peetucket?
  • How much effort do you think this PR requires, @jcoyne?

@peetucket
Copy link
Contributor

This network visualization is part of use case RIALTO-17, which is not one of the top 3 identified use cases, so priority is below the heatmap visualization and associated outputs (RIALTO-8).

@jcoyne
Copy link
Contributor Author

jcoyne commented Aug 3, 2018

@mjgiarlo I think this is pretty low effort, but I agree that this shouldn't be a high priority.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jcoyne @mjgiarlo @peetucket