Skip to content

Conversation

hartmark
Copy link
Contributor

@hartmark hartmark commented Sep 7, 2025

  • Updated lora.md to clarify that not all quants are supported
  • Fixed so --clip-on-cpu hint is not shown when using it
  • Creates parent directory if not exists

* Updated lora.md to clarify that not all quants are supported
* Fixed so --clip-on-cpu hint is not shown when using it
* Creates parent directory if not exists
* Added .idea folder used for CLion
@hartmark
Copy link
Contributor Author

hartmark commented Sep 7, 2025

This includes #785

@leejet
Copy link
Owner

leejet commented Sep 7, 2025

Thank you for your contribution. However, please do not include IDE-related configurations in the project. This is not a recommended practice.

@distlibs
Copy link

distlibs commented Sep 7, 2025

@leejet I added IDE-related files to gitignore #795

@hartmark
Copy link
Contributor Author

hartmark commented Sep 7, 2025

The .idea folder has some files that makes sense to share among the team.

https://rider-support.jetbrains.com/hc/en-us/articles/207097529-What-is-the-idea-folder

I can change my PR to just Ignite .idea of you like anyway

@leejet
Copy link
Owner

leejet commented Sep 7, 2025

The metadata related to the IDE is not recommended to be placed in the project. Please remove the relevant files.

@hartmark
Copy link
Contributor Author

hartmark commented Sep 7, 2025

I've updated to ignore .idea now

@hartmark
Copy link
Contributor Author

hartmark commented Sep 7, 2025

@leejet I added IDE-related files to gitignore #795

You can close that PR as I have already added the ignore here in this PR

@leejet leejet merged commit abb115c into leejet:master Sep 8, 2025
8 checks passed
@leejet
Copy link
Owner

leejet commented Sep 8, 2025

Thank you for your contribution.

@hartmark hartmark deleted the user/hartmark-doc-misc branch September 8, 2025 17:04
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.

3 participants