Skip to content

Conversation

@thongnt99
Copy link
Collaborator

Two changes:

  • Refactoring code: add comments, and split long methods to smaller methods
  • Add diff_html api to generate web interface from code

Changes was tested on Trec-Robust04.

@lgtm-com
Copy link

lgtm-com bot commented Dec 5, 2021

This pull request introduces 1 alert when merging 7622902 into 90906ce - view on LGTM.com

new alerts:

  • 1 for Testing equality to None

@lgtm-com
Copy link

lgtm-com bot commented Dec 16, 2021

This pull request introduces 1 alert when merging bb9392f into 90906ce - view on LGTM.com

new alerts:

  • 1 for Testing equality to None

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2022

This pull request introduces 1 alert when merging d878820 into 90906ce - view on LGTM.com

new alerts:

  • 1 for Testing equality to None

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.

3 participants