Skip to content

fix: correct filter_overlapping for dtype mismatch#158

Closed
oriolfornes wants to merge 1 commit intoGenentech:mainfrom
oriolfornes:bugfix/filter_overlapping
Closed

fix: correct filter_overlapping for dtype mismatch#158
oriolfornes wants to merge 1 commit intoGenentech:mainfrom
oriolfornes:bugfix/filter_overlapping

Conversation

@oriolfornes
Copy link

Fixed a bug in filter_overlapping when interval.index and overlap["index"] had different dtype. This could happen when the intervals.index was numeric (e.g., 1, 2, 3, 4, ...) but was stored as strings (e.g., "1", "2", "3", "4", ...).

I attach an example following the finetune tutorial:
debug-filter_blacklist.ipynb.zip

@oriolfornes oriolfornes reopened this Jul 24, 2025
@avantikalal
Copy link
Collaborator

I have fixed this issue without adding a new dependency in #160

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