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

Explaining graph visualization, partially resolves #275 #277

Merged
merged 2 commits into from Mar 27, 2019

Conversation

Projects
None yet
3 participants
@ianmilligan1
Copy link
Member

ianmilligan1 commented Mar 27, 2019

GitHub issue(s):

If you are responding to an issue, please mention their numbers below.

What does this Pull Request do?

This pull request partially resolves #275 by explaining the graph visualization, both the in-browser sigma one as well as the Gephi download.

  • Adds tooltip to collection page
  • Refines documentation

How should this be tested?

It should build locally, pass style checks (I've tried my best), and the tooltip should pop up.

@ianmilligan1 ianmilligan1 requested a review from ruebot Mar 27, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Mar 27, 2019

Codecov Report

Merging #277 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #277   +/-   ##
=======================================
  Coverage   96.22%   96.22%           
=======================================
  Files          35       35           
  Lines         557      557           
=======================================
  Hits          536      536           
  Misses         21       21

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e55812...25b0252. Read the comment docs.

@ruebot

ruebot approved these changes Mar 27, 2019

@ruebot ruebot merged commit 9a80b86 into master Mar 27, 2019

3 checks passed

codecov/patch Coverage not affected when comparing 3e55812...25b0252
Details
codecov/project 96.22% remains the same compared to 3e55812
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ruebot ruebot deleted the issue-275 branch Mar 27, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.