Skip to content

Conversation

@carlosgjs
Copy link
Collaborator

  • Implementation of SemScore metric: https://arxiv.org/pdf/2401.17072.pdf
  • Renaming the train optional dependency group to pipelines since those dependencies are used across all the pipelines (training, evaluation).

Part of #30

Copy link
Collaborator

@cdcore09 cdcore09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2a26151) 97.87% compared to head (1e2ce17) 98.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
+ Coverage   97.87%   98.00%   +0.12%     
==========================================
  Files           6        6              
  Lines         235      250      +15     
==========================================
+ Hits          230      245      +15     
  Misses          5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@carlosgjs carlosgjs merged commit fb36435 into main Feb 6, 2024
@carlosgjs carlosgjs deleted the carlosg/semscore branch February 6, 2024 22:31
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.

4 participants