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

refactor: search boost and pitch trim results #983

Draft
wants to merge 3 commits into
base: primary
Choose a base branch
from

Conversation

Valastiri
Copy link
Member

Summary

Includes fixes to boost finding "Pitch Trim" in search results on the beginners guide.

Future Boosting Control: After this PR we should opt to establish a boosting criteria for relevant pages if we need.

Currently it is set as the following to tweak results:

  • boost: 2 (this is to bump desired result to the top)
  • boost: 1 (demote other results)
  • boost: 0.5 (further demote - see note below)

Header classification plays a a factor in natural boosting of results i.e. if an h1 contains trim it will most likely appear ahead of other searchs that are plain body or as a sub-heading.

TO DO

  • Add additional information to Pitch Trim (request from Dev's)
  • Check if applicable to other pitch trim related pages that are not beginners guide

Location

  • docs/pilots-corner/beginner-guide/engine-start-taxi.md
  • docs/pilots-corner/a32nx-briefing/flight-deck/pedestal/rudder-trim.md
  • docs/pilots-corner/a32nx-briefing/flight-deck/pedestal/thrust-pitch-trim.md

Discord username (if different from GitHub):
valastiri

Copy link

vercel bot commented May 31, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 31, 2024 6:45am

@Sleinmaster
Copy link
Collaborator

Sleinmaster commented May 31, 2024

As far as I understand, you only boost the whole page, no matter of the search term.
In my opinion, the Beginners Guide could be boosted in general, but otherwise it very much depends on the search term and the users' intent/question, so boosting entire pages might not be ideal.

Does a boost value of 1 make sense at all? That's just the default.

@Valastiri
Copy link
Member Author

As far as I understand, you only boost the whole page, no matter of the search term. In my opinion, the Beginners Guide could be boosted in general, but otherwise it very much depends on the search term and the users' intent/question, so boosting entire pages might not be ideal.

Does a boost value of 1 make sense at all? That's just the default.

As far as I understand, you only boost the whole page, no matter of the search term. In my opinion, the Beginners Guide could be boosted in general, but otherwise it very much depends on the search term and the users' intent/question, so boosting entire pages might not be ideal.

Does a boost value of 1 make sense at all? That's just the default.

As far as I understand, you only boost the whole page, no matter of the search term. In my opinion, the Beginners Guide could be boosted in general, but otherwise it very much depends on the search term and the users' intent/question, so boosting entire pages might not be ideal.

Does a boost value of 1 make sense at all? That's just the default.

Sorry I've been a bit busy, let's actually boost all of beginner guide to 2. Since it's primarly where the bulk of our content is. I think that should be fine overall.

I'll work on that this week. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants