Skip to content

Conversation

2513502304
Copy link

@2513502304 2513502304 commented May 5, 2025

What does this PR do?

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (if yes, ignore all other checks!).
  • Did you read the contributor guideline?
  • Was this discussed/approved via a Github issue? Please add a link to it if that's the case.
  • Did you make sure to update the documentation with your changes (if applicable)?
  • Did you write any new necessary tests?

@MaartenGr
Copy link
Owner

Thanks for the PR and your work on this.

Did you check the Before submitting? I'm missing the related issue where this was discussed.

@2513502304
Copy link
Author

This is a fix for spaCy version compatibility. I don't seem to have found any discussion about the related issue in this repository, but in the spaCy repository, you can find the discussion about this issue here. You can find the relevant pipeline design here.

@MaartenGr
Copy link
Owner

I meant an issue within BERTopic. Typically, it is discussed first over there and validated before moving over to a PR. Now, it is not entirely clear to me when this issue appears and how to reproduce it.

That said, could you share some code to reproduce this issue? Also, I noticed some commented-out code. Are you planning to do something with that?

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.

2 participants