FermiLib icon indicating copy to clipboard operation
FermiLib copied to clipboard

Tests for ipython notebooks

Open babbush opened this issue 8 years ago • 7 comments

We've had a few problems with people submitting pull requests that break the demo file. The demo is quite important because it is likely the first thing that new users will read. However, we currently have no method to test whether it runs without errors. Such a test would save us a lot of trouble!

babbush avatar Jun 23 '17 18:06 babbush

To illustrate the importance of this: I think @idk3 recent changes which moved the UCC code into the circuits folder (which I merged) may have broken the demo. @idk3 can you please fix the demo? I think getting tests for the demo in place should really be a priority.

babbush avatar Aug 09 '17 17:08 babbush

Yep, I'm on it now.

idk3 avatar Aug 09 '17 18:08 idk3

Fixing the imports was not enough, there's a deeper problem somewhere in the call to energy_objective. I'll continue to look at this.

idk3 avatar Aug 09 '17 18:08 idk3

👍

babbush avatar Aug 09 '17 19:08 babbush

OK, I have a fix. The problem was pretty subtle and is not related to my changes, though in retrospect I should've thought of it earlier. I'll send a PR soon.

idk3 avatar Aug 09 '17 19:08 idk3

https://github.com/ProjectQ-Framework/FermiLib/pull/138

idk3 avatar Aug 09 '17 19:08 idk3

Thanks. @damiansteiger are these tests for the demo ipython notebook (just something to make sure the notebook can run without errors) something you can help us with in the next couple weeks? It is really bad that we keep breaking the demo and I frankly don't really know how to write such tests myself. You had once mentioned to me that it could be done.

babbush avatar Aug 10 '17 07:08 babbush