Skip to content

add chebiFromList dataset#164

Merged
sfluegel05 merged 4 commits intodevfrom
feature/chebi-from-list
May 5, 2026
Merged

add chebiFromList dataset#164
sfluegel05 merged 4 commits intodevfrom
feature/chebi-from-list

Conversation

@sfluegel05
Copy link
Copy Markdown
Collaborator

@sfluegel05 sfluegel05 commented Mar 18, 2026

ChEBI dataset which takes a list of classes as input. Useful if you want to define the labels yourself and not use a fixed threshold.

@sfluegel05 sfluegel05 marked this pull request as ready for review May 5, 2026 12:59
@sfluegel05
Copy link
Copy Markdown
Collaborator Author

sfluegel05 commented May 5, 2026

@copilot fix failing unittests

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

@copilot fix failing unittests

Fixed in eeccaa7. The tests were failing due to a NameError: name 'List' is not defined in chebi.py — List was used as a return type annotation in ChEBIFromList.select_classes but wasn't imported from typing. Added it to the import line.

@sfluegel05 sfluegel05 merged commit 76fe93b into dev May 5, 2026
4 checks passed
@sfluegel05 sfluegel05 deleted the feature/chebi-from-list branch May 5, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sfluegel05