Skip to content
New issue

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

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

Already on GitHub? # to your account

Speed and scoring improvements #21

Merged
merged 6 commits into from
Aug 30, 2021
Merged

Speed and scoring improvements #21

merged 6 commits into from
Aug 30, 2021

Conversation

djdembeck
Copy link
Collaborator

@djdembeck djdembeck commented Aug 30, 2021

Enhancements:

  • Reduce limits, greatly increasing speed
  • Remove duplicate parse function, reducing time to search.
  • Check author name validity (don't use author name in search if it's bad).
  • Reduce score by 2 if language isn't library language:
    • Only reducing by 2 will help picking a book from match dialog, without compromising auto-match score threshold of 98, if you do add a foreign language book.

Fixes:

  • Series not found/added to album sort.
  • Fix series title regex.

Documentation:

  • Add changelog

Closes #19

@djdembeck djdembeck added the enhancement New feature or request label Aug 30, 2021
@djdembeck djdembeck merged commit 13e1374 into master Aug 30, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refine score when multiple of same book
1 participant