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

Merge main into develop #13282

Conversation

danieldk
Copy link
Contributor

Description

Merge main into develop.

Types of change

Maintenance

Checklist

  • I confirm that I have the right to submit this contribution under the project's MIT license.
  • I ran the tests, and all new and existing tests passed.
  • My changes don't require a change to the documentation, or if they do, I've added all required information.

svlandeg and others added 3 commits January 26, 2024 10:58
* add line to ensure that apple is in fact in the vocab

* add that the vocab may be empty
* attempt to clarify additional annotations on .spacy file

* suggestion by Daniël

* pipeline instead of pipe
* add custom code support to CLI speed benchmark

* sort imports

* better copying for warmup docs
@danieldk danieldk added the meta Meta topics, e.g. repo organisation and issue management label Jan 29, 2024
The doc/token extension serialization tests add extensions that are not
serializable with pickle. This didn't cause issues before due to the
implicit run order of tests. However, test ordering has changed with
pytest 8.0.0, leading to failed tests in test_language.

Update the fixtures in the extension serialization tests to do proper
teardown and remove the extensions.
@danieldk danieldk force-pushed the maintenance/merge-main-into-develop-20240129 branch from 7d83e77 to a364746 Compare January 29, 2024 13:31
@svlandeg svlandeg merged commit c32c128 into explosion:develop Jan 30, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Meta topics, e.g. repo organisation and issue management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants