Skip to content

[Fix] Remove deprecated argument from kernels call#47100

Open
remi-or wants to merge 1 commit into
mainfrom
depr-old-kernel-kwarg
Open

[Fix] Remove deprecated argument from kernels call#47100
remi-or wants to merge 1 commit into
mainfrom
depr-old-kernel-kwarg

Conversation

@remi-or

@remi-or remi-or commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

CI

In version 0.14 the kernels package removed the package_name argument from LocalLayerRepository.__init__ . Since we only support kernels>=0.15.2,<0.16 it means we should remove it as well when creating this class.

No deprecation warning because package_name is inferred internally.

Also added some stubs so that uv is happy.

@remi-or remi-or requested a review from vasqu July 6, 2026 08:03
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

CI recap

Dashboard: View test results in Grafana
Latest run: 28776923967
Result: failure | Grafana metrics are not available yet.

Code quality check failed: test jobs were skipped. Fix the code quality issues and push again to run tests.

@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.

@vasqu vasqu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ah crap I missed it during the refactor 😢 can we add a test to kernels so we detect this --> no deprecation message

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.

3 participants