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.

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

Methodology: Excerpt Prioritization in Operational Learning #2367

Conversation

susilnem
Copy link
Member

@susilnem susilnem commented Jan 3, 2025

Addresses

Changes

  • Modified the primary_prioritize_excerpts method to sort by appeal_name and appeal_year and interleave grouped rows.
  • Updated the seconday_prioritize_excerpts method to include appeal_name in the deduplication and sorting criteria, and to group by both component and appeal_name.

Checklist

Things that should succeed before merging.

  • Updated/ran unit tests
  • Updated CHANGELOG.md

Release

If there is a version update, make sure to tag the repository with the latest version.

@thenav56 thenav56 marked this pull request as ready for review January 3, 2025 05:50
@thenav56 thenav56 merged commit 2a4414b into project/operational-learning2.0 Jan 3, 2025
3 checks passed
@thenav56 thenav56 deleted the feature/methodology-excerpt-prioritization branch January 3, 2025 05:51
sudip-khanal pushed a commit that referenced this pull request Jan 6, 2025
susilnem added a commit that referenced this pull request Jan 7, 2025
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.

2 participants