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

Add GitHub Action for Nightly #1309

Merged
merged 5 commits into from
Nov 10, 2023
Merged

Conversation

sampathweb
Copy link
Collaborator

No description provided.

pip_build.py Outdated
@@ -45,14 +46,36 @@
]


def build():
def export_version_string(version, is_nightly=False, rc_index=None):
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copied from keras repo pip_build.py

Copy link
Member

@mattdangerw mattdangerw left a comment

Choose a reason for hiding this comment

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

Thanks! A few comments.

Copy link
Member

@mattdangerw mattdangerw left a comment

Choose a reason for hiding this comment

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

Just one last comment! Looks good to me after it is resolved.

@mattdangerw
Copy link
Member

Oh also I think the rc removal wasn't done right, check out the keras 2 tests.

Traceback (most recent call last):
  File "/home/runner/work/keras-nlp/keras-nlp/pip_build.py", line 143, in <module>
    whl_path = build(root_path, args.nightly, args.rc)
AttributeError: 'Namespace' object has no attribute 'rc'

@sampathweb
Copy link
Collaborator Author

Oh also I think the rc removal wasn't done right, check out the keras 2 tests.

Traceback (most recent call last):
  File "/home/runner/work/keras-nlp/keras-nlp/pip_build.py", line 143, in <module>
    whl_path = build(root_path, args.nightly, args.rc)
AttributeError: 'Namespace' object has no attribute 'rc'

Fixed it and also tested locally.

Copy link
Member

@mattdangerw mattdangerw left a comment

Choose a reason for hiding this comment

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

Looks good just minor nits!

@sampathweb sampathweb merged commit 239f6fd into keras-team:master Nov 10, 2023
@sampathweb sampathweb deleted the nightly-upload branch November 10, 2023 02:05
mattdangerw added a commit to mattdangerw/keras-hub that referenced this pull request Nov 10, 2023
It has been failing since keras-team#1309, see
https://github.com/keras-team/keras-nlp/actions/runs/6819858177/job/18547841938

I believe what is happening is that newly pinned nightly only had linux
wheels for 3.9. And our publish to pypi flow did not specify the python
version.
https://pypi.org/project/tensorflow-text-nightly/2.16.0.dev20231107/#files

Doing a few things here:
- Updating the pinned package to a version where tensorflow-text-nightly
  has support for python 3.9, 3.10, and 3.11 on linux.
- Fixing out publish to pypi workflow to set the python version like
  other workflow versions.
- Some other cleanups for consistency with our github actions.
mattdangerw added a commit that referenced this pull request Nov 10, 2023
It has been failing since #1309, see
https://github.com/keras-team/keras-nlp/actions/runs/6819858177/job/18547841938

I believe what is happening is that newly pinned nightly only had linux
wheels for 3.9. And our publish to pypi flow did not specify the python
version.
https://pypi.org/project/tensorflow-text-nightly/2.16.0.dev20231107/#files

Doing a few things here:
- Updating the pinned package to a version where tensorflow-text-nightly
  has support for python 3.9, 3.10, and 3.11 on linux.
- Fixing out publish to pypi workflow to set the python version like
  other workflow versions.
- Some other cleanups for consistency with our github actions.
mattdangerw pushed a commit to mattdangerw/keras-hub that referenced this pull request Dec 7, 2023
* Add GitHub Action for Nightly

* Add GitHub Action for Nightly

* Add GitHub Action for Nightly

* Move tf-nightly pin to 11/4

* Move tf-nightly pin to 11/7
mattdangerw added a commit to mattdangerw/keras-hub that referenced this pull request Dec 7, 2023
It has been failing since keras-team#1309, see
https://github.com/keras-team/keras-nlp/actions/runs/6819858177/job/18547841938

I believe what is happening is that newly pinned nightly only had linux
wheels for 3.9. And our publish to pypi flow did not specify the python
version.
https://pypi.org/project/tensorflow-text-nightly/2.16.0.dev20231107/#files

Doing a few things here:
- Updating the pinned package to a version where tensorflow-text-nightly
  has support for python 3.9, 3.10, and 3.11 on linux.
- Fixing out publish to pypi workflow to set the python version like
  other workflow versions.
- Some other cleanups for consistency with our github actions.
mattdangerw pushed a commit to mattdangerw/keras-hub that referenced this pull request Dec 7, 2023
* Add GitHub Action for Nightly

* Add GitHub Action for Nightly

* Add GitHub Action for Nightly

* Move tf-nightly pin to 11/4

* Move tf-nightly pin to 11/7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants