Skip to content

Conversation

@detkov
Copy link

@detkov detkov commented May 6, 2025

The latest diffusers version uses peft>=0.15.0, which is shown in this commit. However, CogKit uses peft~=0.14.0.
Because of this, when trying to install CogKit, I have encountered the following error:

    Updated https://github.com/huggingface/diffusers (8c661ea586bf11cb2440da740dd3c4cf84679b85)
  × No solution found when resolving dependencies:
  ╰─▶ Because only cogkit==0.0.1.dev167 is available and cogkit==0.0.1.dev167 depends on peft>=0.14.0,<0.15.dev0, we can conclude that all versions 
  of cogkit depend on peft>=0.14.0,<0.15.dev0.
      And because your project depends on cogkit and peft>=0.15.2, we can conclude that your project's requirements are unsatisfiable.

Thus, we have to allow to use a different peft version.
I updated pyproject.toml with a less restricted version of peft>=0.14.0.

@detkov detkov mentioned this pull request May 6, 2025
@detkov
Copy link
Author

detkov commented May 6, 2025

This PR resolves the following issue: #31

@lxxie298
Copy link

lxxie298 commented Jun 8, 2025

It raises up "ImportError: cannot import name 'PEFT_TYPE_TO_PREFIX_MAPPING' from 'peft.utils.constants'" after updating peft to 0.15.0. But it is normal when I use peft==0.14.0.

The "PEFT_TYPE_TO_PREFIX_MAPPING" which is located in the line 69 of "src/cogkit/finetune/utils/ckpt.py" was removed after 0.15.0.

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