astro icon indicating copy to clipboard operation
astro copied to clipboard

Add TensorFlow to lambda-packages

Open jbencook opened this issue 7 years ago • 5 comments

This adds a TensorFlow tarball and build script as mentioned in #31. I've got a simple smoke test running here. I plan to write a more useful TensorFlow example in Zappa as soon as I have a little time.

jbencook avatar Apr 22 '17 22:04 jbencook

Excellent!

But, can I ask - did the manylinux version of Tensorflow not work out of the box? It seems there are manylinux wheels available on PyPI: https://pypi.python.org/pypi/tensorflow

So, it should just automatically use those. Is this package optimized further?

Miserlou avatar May 12 '17 16:05 Miserlou

Yeah it could be user error, but I couldn't get the manylinux version to work.

jbencook avatar May 12 '17 18:05 jbencook

Can you be more specific about how it didn't work?

Miserlou avatar May 12 '17 18:05 Miserlou

Hm... I can try to repro later, but I'm pretty sure the import tensorflow as tf line threw an error.

jbencook avatar May 12 '17 18:05 jbencook

@Miserlou I checked that, and it's because tensorflow has runtime dependency, and throws that error. But I think that this PR it's to heavy for lambda-packages.

Maybe we should think something about #36

brunoalano avatar Aug 16 '17 14:08 brunoalano

Going to update this to use import.meta.env.astro instead of static for right now.

matthewp avatar Mar 31 '21 18:03 matthewp