Skip to content

Update library docs: adapter-transformers -> adapters #1184

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

Merged
merged 5 commits into from
Feb 9, 2024

Conversation

calpt
Copy link
Contributor

@calpt calpt commented Jan 10, 2024

Updated docs page of adapter-transformers library to our newly released Adapters library. Tried to keep the general structure of the doc as is and only update code snippets, some explanation texts and necessary bits.

Todo:

  • Update code & text in "Use in Adapter Transformers" popup window on the hub
  • Update screenshots of hub page

cc @lenglaender @hSterz @TimoImhof

@HuggingFaceDocBuilderDev

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
Contributor

@osanseviero osanseviero left a comment

Choose a reason for hiding this comment

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

Very cool! 🔥

osanseviero pushed a commit to huggingface/huggingface.js that referenced this pull request Feb 1, 2024
Updated from `adapter-transformers` library to the newly released
_Adapters_ library. Related to
huggingface/hub-docs#1184.
@osanseviero
Copy link
Contributor

Let me know whenever this PR is ready for review and I can take another look

@calpt
Copy link
Contributor Author

calpt commented Feb 8, 2024

@osanseviero Thanks! The only missing thing is to update the screenshots, otherwise it's ready for review. Should we update these ourselves and contribute to the HF dataset the old ones were located?

@calpt calpt marked this pull request as ready for review February 8, 2024 18:29
@osanseviero
Copy link
Contributor

Yes, please, if you could open a PR to update the images that would be great!

Copy link
Contributor

@osanseviero osanseviero left a comment

Choose a reason for hiding this comment

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

Very cool! I'll merge ahead and as soon as the images are updated in the docs, it will work 🔥

@osanseviero osanseviero merged commit de75f90 into huggingface:main Feb 9, 2024
@@ -6,7 +6,7 @@ The table below summarizes the supported libraries and their level of integratio

| Library | Description | Inference API | Widgets | Download from Hub | Push to Hub |
|-----------------------------------------------------------------------------|--------------------------------------------------------------------------------------|---|---:|---|---|
| [Adapter Transformers](./adapter-transformers) | Extends 🤗Transformers with Adapters. | ❌ | ❌ | ✅ | ✅ |
| [Adapters](https://github.com/adapter-hub/adapters) | A unified Transformers add-on for parameter-efficient and modular fine-tuning. | ❌ | ❌ | ✅ | ✅ |
Copy link
Member

Choose a reason for hiding this comment

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

tiny nit but we link to the internal doc page from this table now

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