analysis.py redundancies

[Efrem Braun]

In analysis.py, plot_parity and plot_error both use the same energy information, but if one wants to do both of them, they need to be called separately, and the energies need to be calculated twice. I think it makes more sense to have a single function, plot_parity_and_error, which makes both plots.

I don't think a user would ever want just one or the other of these plots, so I'd suggest getting rid of the individual plotting functions. I think it's a bit early in the project to worry about backwards-compatibility issues that might arise.

I'm happy to do the simple coding work and to put in a pull request. I just want to make sure I'm not overstepping any boundaries as a new user.

[andrew_peterson]

Go for it! (Agreed there shouldn't be any worries about backwards compatibility for things in analysis.py.)

You might want to consider writing it as a class so that you can store the info and pass it around.

[Efrem Braun]

Pull request sent!

[andrew_peterson]

Fixed around commit fcc5d6d