Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added BARTScore #89

Merged
merged 66 commits into from
Feb 22, 2022
Merged

Added BARTScore #89

merged 66 commits into from
Feb 22, 2022

Conversation

Sophylax
Copy link
Member

Closes #71.

devrimcavusoglu
devrimcavusoglu approved these changes Feb 21, 2022
View reviewed changes
Copy link
Member

@devrimcavusoglu devrimcavusoglu left a comment

Choose a reason for hiding this comment

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

Let's set the timeout as 400.

devrimcavusoglu and others added 28 commits February 22, 2022 16:12
@devrimcavusoglu
fixture refactor, trial.
3a8ff78
@devrimcavusoglu
Code formatting.
4af5099
@devrimcavusoglu
Test cases refactored.
1a9903b 
- Bertscore removed from regular tests.
@devrimcavusoglu
Merge branch 'main' into bart-score
428461c 
# Conflicts:
#	tests/jury/metrics/test_bartscore.py
@devrimcavusoglu
Test cases refactored.
391b4cd 
- Bertscore removed from regular tests.
@devrimcavusoglu
Merge remote-tracking branch 'origin/bart-score' into bart-score
90f94a1
@Sophylax @devrimcavusoglu
Start Implementing BARTScore Metric
3a17857
@Sophylax @devrimcavusoglu
Complete BARTScore Implementation.
e914507
@Sophylax @devrimcavusoglu
Cast Non-Segmented Scores into Regular Float
a369c3f
@Sophylax @devrimcavusoglu
Add BARTScore Tests
93b7439
@Sophylax @devrimcavusoglu
Add Newline at the End of BARTScore Test JSON
285c621
@Sophylax @devrimcavusoglu
Update BARTScore Checkpoint and Weights Definition
4042005
@Sophylax @devrimcavusoglu
Update Example Docs to Actually Contain BARTScore
90fdccb
@Sophylax @devrimcavusoglu
Clean Up BARTScore
0b2b02b
@Sophylax @devrimcavusoglu
Change Leftover Prism name to BARTScore
bd35c9e
@Sophylax @devrimcavusoglu
Add BARTScore License Information
34e97a6
@Sophylax @devrimcavusoglu
Add Description to BARTScore
40e21ec
@Sophylax @devrimcavusoglu
Update Test JSON Checkpoints
b987d5d
@Sophylax @devrimcavusoglu
Collapse BARTScore Source Dowload Code
f50df47
@Sophylax @devrimcavusoglu
Add BARTScore Test for Segment Scores
70598c3
@Sophylax @devrimcavusoglu
Increase Pytest Timeout
47204af
@devrimcavusoglu
Test cases refactored.
603b3c6 
- Bertscore removed from regular tests.
@devrimcavusoglu
Commit reverted.
d1b81ee
@devrimcavusoglu
Pytest, pytest-cov & pytest-timeout updated.
c3e6bd8
@devrimcavusoglu
Revert "Pytest, pytest-cov & pytest-timeout updated."
40119a2 
This reverts commit 31a60ed.
@devrimcavusoglu
Version upgrades for test suite packages .
1eee973
@devrimcavusoglu
Merge remote-tracking branch 'origin/bart-score' into bart-score
ea6af38
@devrimcavusoglu
Adding cer for reverting deletion.
c6e1fd3
@devrimcavusoglu devrimcavusoglu merged commit 2f85931 into main Feb 22, 2022
@devrimcavusoglu devrimcavusoglu deleted the bart-score branch February 22, 2022 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devrimcavusoglu devrimcavusoglu devrimcavusoglu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

New Metric: Add BARTScore
2 participants
@Sophylax @devrimcavusoglu