Skip to content

implement CrossTextBinaryCorefTaskModuleByNli#120

Open
ArneBinder wants to merge 3 commits into
mainfrom
taskmodules/cross_text_binary_coref_nli
Open

implement CrossTextBinaryCorefTaskModuleByNli#120
ArneBinder wants to merge 3 commits into
mainfrom
taskmodules/cross_text_binary_coref_nli

Conversation

@ArneBinder
Copy link
Copy Markdown
Owner

@ArneBinder ArneBinder commented Sep 19, 2024

without target handling and metrics so far

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 19, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.74%. Comparing base (4c4e0a7) to head (2d34b03).
⚠️ Report is 147 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
+ Coverage   95.68%   95.74%   +0.06%     
==========================================
  Files          61       62       +1     
  Lines        5071     5148      +77     
==========================================
+ Hits         4852     4929      +77     
  Misses        219      219              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@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.

2 participants