-
Notifications
You must be signed in to change notification settings - Fork 69
feat: 271 feature implement hps hpsv2 VQA #272
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
base: main
Are you sure you want to change the base?
Changes from all commits
a6d6cb8
1b98ffb
0b07425
e0bd484
5a40613
806d2ed
c18e279
6ac22c6
36b4e35
81e2120
b587517
f3fbce7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -60,6 +60,8 @@ gptqmodel = [ | |
| { index = "pruna_internal", marker = "sys_platform != 'darwin' or platform_machine != 'arm64'"}, | ||
| { index = "pypi", marker = "sys_platform == 'darwin' and platform_machine == 'arm64'"}, | ||
| ] | ||
| image-reward = { git = "https://github.com/PrunaAI/ImageReward" } | ||
| hpsv2 = { git = "https://github.com/PrunaAI/HPSv2" } | ||
|
|
||
| [project] | ||
| name = "pruna" | ||
|
|
@@ -70,7 +72,7 @@ authors = [ | |
| ] | ||
| license = {file = "LICENSE"} | ||
| readme = "README.md" | ||
| requires-python = ">=3.9,<3.13" | ||
| requires-python = ">=3.10,<3.13" | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| keywords = ["AI", "machine learning", "model optimization", "pruning"] | ||
| classifiers = [ | ||
| "Development Status :: 4 - Beta", | ||
|
|
@@ -106,7 +108,6 @@ dependencies = [ | |
| "codecarbon", | ||
| "pynvml", | ||
| "thop", | ||
| "timm", | ||
| "bitsandbytes; sys_platform != 'darwin' or platform_machine != 'arm64'", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did we remove timm from the dependencies? |
||
| "optimum-quanto>=0.2.5", | ||
| "ctranslate2==4.6.0", | ||
|
|
@@ -115,11 +116,14 @@ dependencies = [ | |
| "torchao", | ||
| "llmcompressor", | ||
| "gliner; python_version >= '3.10'", | ||
| "piq", | ||
| "opencv-python", | ||
| "kernels", | ||
| "aenum" | ||
|
|
||
| "aenum", | ||
| "image-reward", | ||
| "piq", | ||
| "clip", | ||
| "hpsv2", | ||
| "t2v-metrics", | ||
| ] | ||
|
|
||
| [project.optional-dependencies] | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -20,6 +20,8 @@ | |
| from pruna.evaluation.metrics.metric_memory import DiskMemoryMetric, InferenceMemoryMetric, TrainingMemoryMetric | ||
| from pruna.evaluation.metrics.metric_model_architecture import TotalMACsMetric, TotalParamsMetric | ||
| from pruna.evaluation.metrics.metric_pairwise_clip import PairwiseClipScore | ||
| from pruna.evaluation.metrics.metric_reward import HPSMetric, HPSv2Metric, ImageRewardMetric | ||
| from pruna.evaluation.metrics.metric_sharpness import SharpnessMetric | ||
| from pruna.evaluation.metrics.metric_torch import TorchMetricWrapper | ||
|
|
||
| __all__ = [ | ||
|
|
@@ -37,4 +39,8 @@ | |
| "TotalMACsMetric", | ||
| "PairwiseClipScore", | ||
| "CMMD", | ||
| "ImageRewardMetric", | ||
| "HPSMetric", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If HPS does not work, I would prefer to leave it out for now, or fix it. |
||
| "HPSv2Metric", | ||
| "SharpnessMetric", | ||
| ] | ||
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.
There were some issues with both repositories.
ImageReward, had pushed their fixes to
mainHPSv2 had very strict constraints for protobuf where they set it to lower than 6
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.
@johnrachwan123 perhaps we should publish these on our own index?
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.
also we could add t2v-metrics with a more relaxed Python constraint