Uploaded image for project: 'Terrier Core'
  1. Terrier Core
  2. TR-231

Deprecate the 5-param score method in favor of the 2-param method

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.6
    • Component/s: .matching
    • Labels:
      None

      Description

      This is the 3.6 part of TR-222

        Attachments

          Issue Links

            Activity

            Hide
            craigm Craig Macdonald added a comment -

            Credit to Francois Rousseau

            Show
            craigm Craig Macdonald added a comment - Credit to Francois Rousseau
            Hide
            richardm Richard McCreadie added a comment -

            Committed at revision 3733. All classes implementing WeightingModel have had the 5-parameter score method deprecated. Classes in the queryexpansion and basicmodel have been left unchanged.

            Show
            richardm Richard McCreadie added a comment - Committed at revision 3733. All classes implementing WeightingModel have had the 5-parameter score method deprecated. Classes in the queryexpansion and basicmodel have been left unchanged.
            Hide
            richardm Richard McCreadie added a comment -

            Fixed. However, we may wish to revisit the organisation of the models package since their is currently no root interface for weighting models. WeightingModel, QueryExpansionModel and BasicModel should share an interface.

            Show
            richardm Richard McCreadie added a comment - Fixed. However, we may wish to revisit the organisation of the models package since their is currently no root interface for weighting models. WeightingModel, QueryExpansionModel and BasicModel should share an interface.
            Hide
            craigm Craig Macdonald added a comment -

            +1. Please create an issue for the future.

            Show
            craigm Craig Macdonald added a comment - +1. Please create an issue for the future.

              People

              • Assignee:
                craigm Craig Macdonald
                Reporter:
                craigm Craig Macdonald
              • Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: