Skip to content

[AIMOD-1206] Add flight normalization + Graduate finance / sports query normalization to prod#1558

Draft
vazish wants to merge 4 commits into
mainfrom
aimod-1206-add-flight-query-normalization
Draft

[AIMOD-1206] Add flight normalization + Graduate finance / sports query normalization to prod#1558
vazish wants to merge 4 commits into
mainfrom
aimod-1206-add-flight-query-normalization

Conversation

@vazish

@vazish vazish commented May 27, 2026

Copy link
Copy Markdown
Contributor

References

JIRA: AIMOD-1206

Description

Making this a draft PR only at this time until our Friday meeting to discuss the experiment results for Sports & Finance.

This PR does 2 things, which we can separate out but are kinda tied together

  1. Adds flight query normalization handling
  2. Enables query normalization for finance, sports and flights in production

For flights,

  1. Handling is slightly different since we have dynamic numbers at the end, so the regular query normalization with the canonical set cannot work the same
  2. We handled by checking different iterations of the query, e.g. ("ua 123", "123 ua", "united airlines 123", "123 united airlines", "united airlines flight 123"). LMK if we want to add some other combinations!

Best place to start review is
FlightAwareNormalizePipeline.normalize()

PR Review Checklist

Put an x in the boxes that apply

  • This PR conforms to the Contribution Guidelines
  • The PR title starts with the JIRA issue reference, format example [DISCO-####], and has the same title (if applicable)
  • [load test: (abort|skip|warn)] keywords are applied to the last commit message (if applicable)
  • Documentation has been updated (if applicable)
  • Metric documentation has been updated (if applicable)
  • Functional and performance test coverage has been expanded and maintained (if applicable)

┆Issue is synchronized with this Jira Task

@vazish vazish requested review from ncloudioj and tiftran May 27, 2026 15:19
@misaniwere misaniwere self-requested a review June 2, 2026 19:07
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.

1 participant