Skip to content

[Bot] Update tasks specs #1030

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 1 commit into from
Nov 22, 2024
Merged

[Bot] Update tasks specs #1030

merged 1 commit into from
Nov 22, 2024

Conversation

HuggingFaceInfra
Copy link
Contributor

This PR updates the @huggingface/tasks specs. It has been generated by running:

pnpm run inference-tgi-import
pnpm run inference-tei-import
pnpm run inference-codegen

This PR was automatically created by the Tasks - Update specs workflow.

Make sure the changes are correct before merging.

@Wauplin
Copy link
Contributor

Wauplin commented Nov 15, 2024

This PR has been created as part of #964. Let's not merge for now.

@Wauplin Wauplin marked this pull request as draft November 15, 2024 10:15
@HuggingFaceInfra HuggingFaceInfra force-pushed the update-tasks-specs-automated-pr branch from 0553d63 to 3ba1213 Compare November 15, 2024 10:49
Wauplin added a commit that referenced this pull request Nov 18, 2024
This PR adds a Github Action to automatically run the
`@huggingface/tasks-gen` scripts and open a PR if anything has changed.
The workflow is triggered once a day (3am) and can be triggered manually
if needed.

Here is an example of auto-generated PR:
#1030.

---

Note: lint issues seems unrelated. But it doesn't seem to happen on main
(see
[here](https://github.com/huggingface/huggingface.js/actions/workflows/lint.yml?query=branch%3Amain))
so I don't know what's causing it 😕

---------

Co-authored-by: Pauline Bailly-Masson <[email protected]>
Base automatically changed from add-workflow-update-specs to main November 18, 2024 15:40
@HuggingFaceInfra HuggingFaceInfra force-pushed the update-tasks-specs-automated-pr branch from 3ba1213 to 65c9415 Compare November 19, 2024 03:17
Comment on lines 21 to 24
/**
* A guideline to be used in the chat_template
*/
guideline?: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a fan of this argument as it's not used anymore. Best solution would be to remove it from TGI specs (see slack thread). Let's wait without merging this PR for now.

Copy link
Contributor

Choose a reason for hiding this comment

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

opened huggingface/text-generation-inference#2762 to remove guideline from TGI API

@HuggingFaceInfra HuggingFaceInfra force-pushed the update-tasks-specs-automated-pr branch from 81d33ba to fb18fe3 Compare November 20, 2024 03:16
@HuggingFaceInfra HuggingFaceInfra force-pushed the update-tasks-specs-automated-pr branch from fb18fe3 to 3f9a70d Compare November 21, 2024 03:16
@HuggingFaceInfra HuggingFaceInfra force-pushed the update-tasks-specs-automated-pr branch from 3f9a70d to 592029f Compare November 22, 2024 03:17
Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Looks good now :)

@Wauplin Wauplin marked this pull request as ready for review November 22, 2024 08:36
@Wauplin Wauplin merged commit afdfb0b into main Nov 22, 2024
5 checks passed
@Wauplin Wauplin deleted the update-tasks-specs-automated-pr branch November 22, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
specs tasks @huggingface/tasks related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants