Skip to content

Fix version check when console output is disabled#688

Merged
regisss merged 2 commits into
huggingface:mainfrom
vidyasiv:fix_version_check
Feb 7, 2024
Merged

Fix version check when console output is disabled#688
regisss merged 2 commits into
huggingface:mainfrom
vidyasiv:fix_version_check

Conversation

@vidyasiv
Copy link
Copy Markdown
Contributor

@vidyasiv vidyasiv commented Feb 5, 2024

What does this PR do?

Fixes #684

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@vidyasiv vidyasiv marked this pull request as ready for review February 5, 2024 19:32
@vidyasiv vidyasiv requested a review from regisss as a code owner February 5, 2024 19:32
@vidyasiv
Copy link
Copy Markdown
Contributor Author

vidyasiv commented Feb 5, 2024

@regisss , I went with a different approach than what we discussed so we don't have to skip the check. This should capture the case if version is not set: https://github.com/huggingface/optimum-habana/blob/main/optimum/habana/utils.py#L201 . Let me know what you think.

Copy link
Copy Markdown
Collaborator

@regisss regisss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@regisss
Copy link
Copy Markdown
Collaborator

regisss commented Feb 6, 2024

@vidyasiv Can you also run the following from the root of the repo please?

pip install -U ruff
make style

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@vidyasiv
Copy link
Copy Markdown
Contributor Author

vidyasiv commented Feb 6, 2024

@vidyasiv Can you also run the following from the root of the repo please?

pip install -U ruff
make style

Apologies, I ran it and updated code.

@regisss regisss added the run-test Run CI for PRs from external contributors label Feb 7, 2024
Copy link
Copy Markdown
Collaborator

@regisss regisss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@regisss regisss merged commit 44be555 into huggingface:main Feb 7, 2024
jychen21 pushed a commit to jychen21/optimum-habana that referenced this pull request Feb 27, 2024
dudilester pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Feb 28, 2024
HolyFalafel pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-test Run CI for PRs from external contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ENABLE_CONSOLE disrupts version checks in examples

3 participants