Skip to content

Lopster integration + bug fixes#27

Merged
sfluegel05 merged 3 commits intodevfrom
feature/lopster
Feb 16, 2026
Merged

Lopster integration + bug fixes#27
sfluegel05 merged 3 commits intodevfrom
feature/lopster

Conversation

@sfluegel05
Copy link
Contributor

@sfluegel05 sfluegel05 commented Jan 19, 2026

Adds a wrapper for the ChemLog Lopster classifier (in both algorithmic and Clingo ASP version)
Also:

  • PPV/NPV are now calculated from the confusion matrix
  • I removed some parameters that got added to checkpoints automatically and were not expected (causing the instantiation to fail) - I don't know why this only fails now and has not been failing previously
  • Added dummy labels to reader.to_data calls

@sfluegel05 sfluegel05 marked this pull request as ready for review February 16, 2026 14:10
@sfluegel05 sfluegel05 merged commit acda786 into dev Feb 16, 2026
2 checks passed
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.

1 participant