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

fix package name bug for nightly #155

Merged
merged 1 commit into from
Sep 10, 2024
Merged

fix package name bug for nightly #155

merged 1 commit into from
Sep 10, 2024

Conversation

dhuangnm
Copy link
Collaborator

SUMMARY:
fix package name bug for nightly where nightly wheel still uses "llmcompressor" instead of "llmcompressor-nightly".

TEST PLAN:
manually tested for nightly, release and dev builds

Copy link

👋 Hi! Thank you for contributing to llm-compressor. Please add the ready label when the PR is ready for review.

@dsikka
Copy link
Collaborator

dsikka commented Sep 10, 2024

Will the name be updated by a bash script from another action?

@dhuangnm
Copy link
Collaborator Author

Will the name be updated by a bash script from another action?

Correct, the workflow will change the setting build_type in src/llmcompressor/version.py to nightly/release/dev based on run type.

@dsikka dsikka merged commit 051650d into main Sep 10, 2024
7 of 8 checks passed
@dsikka dsikka deleted the fixname branch September 10, 2024 14:31
markmc pushed a commit to markmc/llm-compressor that referenced this pull request Nov 13, 2024
* add workflow to build and test

* fix indent

* clean up

* add project id

* minor update

* fix a bug

* test release

* add test

* fix test reporting issue

* add upload and use compressed-tensors bucket

* use secret for bucket

* not install

* still uses k8s-util

* clean up

* try GCP_GHA_SA

* fix path

* use NM_PYPI_SA again

* authenticate with GCP_GHA_SA and NM_PYPI_SA

* fix id

* seperate install and test steps

* test more configs

* fix error

* fix a bug

* fix again

* add step-status

* fix failure

* test release

* clean up

* clean up

* update

* test nightly

* test release

* fix a bug

* fix a bug

* change default back to NIGHTLY

* clean up

---------

Co-authored-by: dhuangnm <dhuang@MacBook-Pro-2.local>
Co-authored-by: dhuangnm <dhuang@Rogers-iPhone.localdomain>
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