-
Notifications
You must be signed in to change notification settings - Fork 19.4k
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
API Generator Code for Keras #19523
API Generator Code for Keras #19523
Conversation
shell/format.sh
Outdated
@@ -3,6 +3,10 @@ set -Eeuo pipefail | |||
|
|||
base_dir=$(dirname $(dirname $0)) | |||
|
|||
echo "Generating public APIs..." | |||
python3 "${base_dir}"/api_gen.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upon thinking more about it -- an issue with generating the API in this script is that it takes quite a bit of time, but most of the time it won't be needed (it's only needed when we add new APIs). So it's actually better to have it in a separate script after all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the feedback. Added shell/api_gen.sh
to generate APIs and kept shell/format.sh
for Code formatting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good!
0bfc2f3
to
1e79af8
Compare
Closing this PR since #19530 is now merged. |
API Generator Code.