Skip to content
This repository has been archived by the owner on Oct 17, 2021. It is now read-only.

[tfjs2keras test]: Add test against tf.keras #523

Merged
merged 3 commits into from
Apr 11, 2019

Conversation

caisq
Copy link
Contributor

@caisq caisq commented Apr 11, 2019

  • Previously only keras-team/keras was tested.
  • The --tfkeras flag enables testing against tensorflow.keras

Related to tensorflow/tfjs#1511

DEV


This change is Reviewable

- Previously only keras-team/keras was tested.
- The --tfkeras flag enables testing against tensorflow.keras

DEV
@caisq caisq requested a review from davidsoergel April 11, 2019 16:19
Copy link
Member

@davidsoergel davidsoergel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @caisq and @davidsoergel)


integration_tests/tfjs2keras/run-test.sh, line 48 at r1 (raw file):

source "${VENV_DIR}/bin/activate"

# If in Travis, use the `--user` flag when performing `pip install` of

Comment is obsolete (and I guess the --user flag is obsolete too because we're not using Travis anymore?)

Copy link
Contributor Author

@caisq caisq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @davidsoergel)


integration_tests/tfjs2keras/run-test.sh, line 48 at r1 (raw file):

Previously, davidsoergel (David Soergel) wrote…

Comment is obsolete (and I guess the --user flag is obsolete too because we're not using Travis anymore?)

Good catch. Removed the obsolete comment.

Copy link
Contributor Author

@caisq caisq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review!

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @davidsoergel)

Copy link
Contributor Author

@caisq caisq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review!

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @davidsoergel)

Copy link
Contributor Author

@caisq caisq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review!

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @davidsoergel)

@caisq caisq merged commit a49f520 into tensorflow:master Apr 11, 2019
@caisq caisq deleted the integ2tf2 branch April 11, 2019 16:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants