ICOHP vs bond length plotter

[naik-aakash]

Changes

1. Added an IcohpPlotter class to generate icohp vs bond-length plots from icohplist object
2. Added tests to verify labels of generated plots

• Integrate to cli and add corresponding tests

Closes #133

[naik-aakash]

Hi, here are the example plots obtained from the IcohpPlotter class

ICOBI vs bond lengths

ICOOP vs bond lengths

ICOHP vs bond lengths

The style sheet used can also be changed (At present, the default is lobsterpy style sheet), and marker size and styles can also be modified as per need.

[naik-aakash]

Hi @JaGeo, I could not figure out how to retrieve and test data in matplotlib scatter plots. But all other things are tested.

[naik-aakash]

Hi @ajjackson , is there any way to test the data in matplotlib scatter plot? I could not figure it out. It would be great if you can help me with this 😄

[JaGeo]

Does this maybe help? https://stackoverflow.com/questions/33336343/recover-data-from-matplotlib-scatter-plot

[naik-aakash]

Does this maybe help? https://stackoverflow.com/questions/33336343/recover-data-from-matplotlib-scatter-plot

Thanks @JaGeo! I will try this out

[ajjackson]

Beat me to it! The data has to be in there somewhere, but it's not always easy to figure out the object heirarchy in a given Matplotlib plot...

[naik-aakash]

Hi @JaGeo and @ajjackson , I managed to get the plot data from figure object and have updated tests now. So this PR is also ready for review 😄

[JaGeo]

@naik-aakash I have made the changes myself and will merge now!

[naik-aakash]

@naik-aakash I have made the changes myself and will merge now!

Thanks @JaGeo ! 😄