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.

Sign up for GitHub

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

Jump to bottom

[WIP] Remove resources required for Learning to Rank #1831

Open

sihugh wants to merge 1 commit into main from remove-search-sagemaker

Contributor

sihugh commented May 15, 2024 •

edited

Loading

This is no longer used in search, so we don't need to train or host models any more.

This won't remove any of the actual Sagemaker resources, as this was done in the deploy pipeline. We'll remove these separately.

sihugh force-pushed the remove-search-sagemaker branch from 5ccef70 to d542507 Compare

May 15, 2024 16:19

sihugh marked this pull request as ready for review

May 15, 2024 16:23


          Remove resources required for Learning to Rank

d854f7a

Learning to Rank is no longer used in search, so we don't need to train or host models any more.

This won't remove any of the actual Sagemaker resources, as this was done in [the deploy pipeline](https://github.com/alphagov/search-api/blob/f037f11c6f0299b8bd4d065a1d947b8f827bbc0d/ltr/sagemaker/deploy.py). We'll remove these separately.

sihugh force-pushed the remove-search-sagemaker branch from d542507 to d854f7a Compare

May 16, 2024 09:13

sengi approved these changes

View reviewed changes

sihugh changed the title ~~Remove resources required for Learning to Rank~~ [WIP] Remove resources required for Learning to Rank

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

Labels

None yet