search

lobe / lobe-python

Python toolset for working with Lobe models
MIT License
185 stars 38 forks source link

fix signature bugs #10

Closed adiazulay closed 4 years ago

adiazulay commented 4 years ago

There were a couple of bugs caused by the names of tensors being changed from the version this was developed on.

There was also a bug in the loading from signature for tensor flow since it was expecting a string for the model path not an ospath

averypfeiffer commented 4 years ago

@adiazulay Ran into this issue today. Lobe is very user friendly and welcoming, but upon exporting I hit a brick wall of errors related to this issue. I stepped through and found the culprit, then saw that you had already applied a fix and initiated a PR.

In anycase, after manually applying the changes in your merge request all works as expected. I would suggest adding an issue related to this to the issue page so other users will see it is a known issue

adiazulay commented 4 years ago

Thanks for trying out the fix! I'll be merging this par later today.