Interactive Visualization

[heatherspacek]

Proposed changes

Add plotly interactive graphs to the Validator page.

Notes

I have a good idea of the roadmap to fully integrate these, but I don't have the resources right now to test the changes properly myself.

• the routine generate_visualizations() (from generate_visualizations.py) should be called in controller.py's validator() (around line 405, maybe?)
• generate_visualizations() is equipped to dump working HTML to a file (specified by an argument) - it will be wrapped in a etc but those are easy to trim out. It's approximately 3MB of javascript.
• Either render_template() (the call on line 406 of controller.py currently) can be modified to take another argument similar to the 'result' string, or you can staple the figures to 'result' itself for a convenient way to test without changing any architecture?

Types of changes

• [ ] Bugfix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionalitiy)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] Other (if none of the other choices apply)

Checklist

• [ ] Changes have been tested to ensure that fix is effective or that a feature works.
• [ ] Changes pass the unit tests
• [x] I have included necessary documentation or comments (as necessary)
• [ ] Any dependent changes have been merged and published

_PR template was adopted from appium_

[tkkuehn]

Alrighty, I think this is in shape for a review now.

[tkkuehn]

Notes from today's meeting:

• Put table back
• Label colorbar
• Remove tiny text on histogram

[tkkuehn]

I think this is ready for another review!

[tkkuehn]

Wondering do we want to merge it into master now, or release it as part of 0.2.x?

Either way seems fine to me. We could even merge this, make it 0.2.0, then make the AFIDs class stuff 0.3.0.

[kaitj]

Given that I somehow did some weird thing with creating the AFIDs class and this doesn't actually break functionality with 0.1.x (though it is a major enhancement), I'd be happy with merging this in as 0.1.x and still call the AFIDs class 0.2.x

[tkkuehn]

Great, feel free to merge and tag it as a new release in that case!