Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tfjs2keras integ test: switch on the --dev --tfkeras path #1511

Closed
caisq opened this issue Apr 11, 2019 · 0 comments
Closed

tfjs2keras integ test: switch on the --dev --tfkeras path #1511

caisq opened this issue Apr 11, 2019 · 0 comments
Assignees

Comments

@caisq
Copy link
Contributor

caisq commented Apr 11, 2019

Context: tensorflow/tfjs-layers#523
After the PR is merged, the following test still fails.

cd integration_tests/tfjs2keras
./run-test.sh --dev --tfkeras

This is due to the following error:
File "/tmp/tmp.Ax8E6v9mDu_venv/local/lib/python2.7/site-packages/tensorflow/python/ops/init_ops_v2.py", line 394, in init
raise ValueError("Invalid distribution argument:", distribution)
ValueError: ('Invalid distribution argument:', u'normal')

which has to do with b/129177721

@caisq caisq self-assigned this Apr 11, 2019
caisq added a commit to caisq/tfjs-layers-1 that referenced this issue Apr 11, 2019
- Previously only keras-team/keras was tested.
- The --tfkeras flag enables testing against tensorflow.keras

Related to tensorflow/tfjs#1511

DEV
caisq added a commit to tensorflow/tfjs-layers that referenced this issue Apr 11, 2019
Note that this is currently testing against tf-nightly, which is built
from tensorflow's master HEAD.

DEV

Fixes tensorflow/tfjs#1511
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant