SQFT Finetuning #947
Open
SQFT Finetuning #947
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jpablomch
force-pushed
the
sqft_finetuning
branch
from
0491748 to
e12b468
Compare
|
Please fix the code scan issue and e2e test issue in CI test. |
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
for more information, see https://pre-commit.ci Signed-off-by: J. Pablo Muñoz <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
Signed-off-by: Yuan0320 <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
Signed-off-by: Yuan0320 <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
…validation Signed-off-by: Yuan0320 <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
Signed-off-by: ZePan110 <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
Co-authored-by: Yuan0320 <[email protected]> Signed-off-by: J. Pablo Muñoz <[email protected]>
jpablomch
force-pushed
the
sqft_finetuning
branch
from
522fff4 to
98bc3f8
Compare
|
Hi @jpablomch, could you please help check the failures in the test? |
…ion) Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Yuan0320
force-pushed
the
sqft_finetuning
branch
from
c1cd906 to
c83f86a
Compare
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
…atch Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
Co-authored-by: J. Pablo Muñoz <[email protected]> Signed-off-by: Yuan0320 <[email protected]>
lkk12014402
approved these changes
Dec 4, 2024
|
pls hold on this PR. we are syncing on how to deal with this one. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR enables SQFT's NLS fine-tuning as a microservice in OPEA.
SQFT details can be found in the paper: SQFT: Low-cost Model Adaptation in Low-precision Sparse Foundation
Models (https://aclanthology.org/2024.findings-emnlp.749/)
Issues
N/A
Type of change
New feature (non-breaking change which adds new functionality) as detailed in our paper https://aclanthology.org/2024.findings-emnlp.749.pdf
Dependencies
Transformers, PEFT, and NNCF.
Tests
We include a test to validate the fine-tuning, similar to other existing fine-tuning approaches in OPEA.