Skip to content
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

AutoModel: let models check if the configuration is supported #352

Merged

Conversation

danieldk
Copy link
Contributor

@danieldk danieldk commented Oct 5, 2023

Description

This will allow us to split up more complex models like Falcon into multiple classes and corresponding entry points.

Checklist

  • I confirm that I have the right to submit this contribution under the project's MIT license.

This will allow us to split up more complex models like Falcon into
multiple classes and corresponding entry points.
@danieldk danieldk added type/feature Type: Feature feat/model Feature: models labels Oct 5, 2023
Copy link
Collaborator

@shadeMe shadeMe left a comment

Choose a reason for hiding this comment

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

Just one minor change.

curated_transformers/models/hf_hub/mixin.py Outdated Show resolved Hide resolved
Co-authored-by: Madeesh Kannan <[email protected]>
@danieldk danieldk merged commit dfe6d96 into explosion:main Oct 5, 2023
1 of 2 checks passed
@danieldk danieldk deleted the feature/better-automodel-detection branch October 5, 2023 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat/model Feature: models type/feature Type: Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants