Skip to content

Conversation

@ArneBinder
Copy link
Owner

@ArneBinder ArneBinder commented Sep 19, 2024

without target handling and metrics so far

@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.74%. Comparing base (69ff92d) to head (2d34b03).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
+ Coverage   95.73%   95.74%   +0.01%     
==========================================
  Files          61       62       +1     
  Lines        5061     5148      +87     
==========================================
+ Hits         4845     4929      +84     
- Misses        216      219       +3     

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

@ArneBinder ArneBinder force-pushed the taskmodules/cross_text_binary_coref_nli branch from d290b29 to 2d34b03 Compare September 20, 2024 16:35
ArneBinder added a commit to ArneBinder/argumentation-structure-identification that referenced this pull request Sep 24, 2024
ArneBinder added a commit to ArneBinder/argumentation-structure-identification that referenced this pull request Oct 5, 2024
ArneBinder added a commit to ArneBinder/argumentation-structure-identification that referenced this pull request Oct 5, 2024
* add sciarg_semantically_same_nli experiment config

* add cross_text_binary_coref_nli.py from ArneBinder/pie-modules#120, but with slight changes

* use src.taskmodules.CrossTextBinaryCorefTaskModuleByNli in sciarg_semantically_same_nli experiment

* revert: add probability_threshold and create all annotations

* upgrade to pie-modules>=0.13.1,<0.14.0

* outsource taskmodule config to cross_text_coref_by_nli.yaml

* add documentation to CrossTextBinaryCorefTaskModuleByNli

* rename cross_text_coref.yaml to cross_text_coref_with_context.yaml

* add note about base model alternative
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants