Error when loading `.amp` files trained with Tensor Flow

[Muammar El Khatib]

Take the test located at amp/tests/test_gaussian_tflow.py. Run it and try to load calc.amp:

from amp import Amp
calc = Amp.load('train_test/calc.amp')

You get the following error:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/users/melkhati/git/amp/amp/__init__.py", line 176, in load
    Model = importhelper(p['model'].pop('importname'))
KeyError: 'importname'

[Zachary Ulissi]

Should be easy to fix but not quite sure what the mechanism for loading things is. Currently I use Amp.load('calc.amp',Model=tflow.NeuralNetwork) to avoid importhelper and specify the model class directly.

[Muammar El Khatib]

Thanks for your reply, Zachary!. I will load for the moments the .amp file as you suggested.

[andrew_peterson]

We should be able to fix this with a line or two.

[Muammar El Khatib]

This pull request should fix issue #111.

→ <<cset 0162f31ba533>>

[Muammar El Khatib]

I changed the status to open, because I will submit a pull request to be reviewed first.

[andrew_peterson]

This pull request should fix issue #111.

→ <<cset 0162f31ba533>>

[andrew_peterson]

Merged in tflowfix (pull request #10)

This pull request should fix issue #111.

→ <<cset 3e8b882563c3>>

[Muammar El Khatib]

Merge branch 'master' into symmetryfunctions

• master: More instances of "get" to "calculate". Fix spelling typo. Simplify example script. Change "get"s to "calculate"s. Can now turn off logging. commiting for merge Some cleanup to tensorflow module. Re-label plot. update the retraining doc Add SGE to assign_cores. Removed extra comma in init function inside tflow. This pull request should fix issue #111.

→ <<cset b63022f931d1>>