Skip to content

Fix error in PR#654#661

Merged
mandy-li merged 6 commits into
mainfrom
dev/schoi/1.14.0
Jan 24, 2024
Merged

Fix error in PR#654#661
mandy-li merged 6 commits into
mainfrom
dev/schoi/1.14.0

Conversation

@schoi-habana
Copy link
Copy Markdown
Collaborator

ensure backward compatibility for models other than Falcon

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?

1) fix for invalid input shape with DS 0.12.4
2) revert --skip_hash_with_views changes
3) add falcon to model_on_meta()
@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.

Copy link
Copy Markdown
Collaborator

@mandy-li mandy-li left a comment

Choose a reason for hiding this comment

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

@schoi-habana , can you combine the two conditions to one so that only one else will be needed?

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!

@mandy-li mandy-li merged commit 9841f1e into main Jan 24, 2024
@mandy-li mandy-li deleted the dev/schoi/1.14.0 branch January 24, 2024 19:02
jychen21 pushed a commit to jychen21/optimum-habana that referenced this pull request Feb 27, 2024
* Falcon changes for 1.14.0

1) fix for invalid input shape with DS 0.12.4
2) revert --skip_hash_with_views changes
3) add falcon to model_on_meta()

* address comments

* README change

* make PR#654 backward compatible for other models

* address comments
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.

4 participants