Skip to content

Conversation

@MaartenGr
Copy link
Owner

What does this PR do?

Fixes #1749

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?

@rhys-thompson-deel
Copy link

Is this still being looked into?

@MaartenGr
Copy link
Owner Author

@rhys-thompson-deel I haven't gotten a response yet from those that initially had this issue. I want to make sure this works for those facing this issue before merging. If this something you need and have found to work?

@rhys-thompson-deel
Copy link

@rhys-thompson-deel I haven't gotten a response yet from those that initially had this issue. I want to make sure this works for those facing this issue before merging. If this something you need and have found to work?

yes I have been using the fix for a few weeks and haven't ran into any issues so far!

@MaartenGr MaartenGr merged commit 04c52a8 into master Jul 4, 2025
5 checks passed
@MaartenGr
Copy link
Owner Author

@rhys-thompson-deel Sorry for the delay and thank you for testing this! I merged it and will soon release 0.17.1 of BERTopic to have all these new changes.

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.

IndexError: index -2 is out of bounds for axis 0 with size 1 for the zero shot code.

3 participants