Recommender.py for FastAPI

[jpjon]

Describe your changes

Created the .py file for the recommender that FastAPI will reference to.

Issue number

#99
Closes #99

Checklist before requesting a review

• Performed a self-review of my code
• Formatted my code with pkgmt format

---

📚 Documentation preview 📚: https://ploomber-sql--100.org.readthedocs.build/en/100/

[jpjon]

Hi @lfunderburk

Can you take a look at what I have so far? My main concern regards my comment on how we would connect to the S3 Bucket using Lambda.

Besides that, I think the main get_recommendation function is of viable use for FastAPI.

[jpjon]

Please navigate to the /recommendations folder and run uvicorn app:app --reload to launch the API.

Go to http://127.0.0.1:8000/docs#/default in your browser to test out the API with a movie. I used the movie "Barbie" and received the expected output.

[lfunderburk]

Hi @jpjon

Did a bit of code clean up, type checking, pydantic modelling and added some testing

To execute tests

cd sql/mini-projects/movie-rec-system
pytest

From the Docker container we don't want to specify the path, we should be running things from the project's root directory (in this case sql/mini-projects/movie-rec-system)

Please execute the app as follows

 uvicorn movie_rec_system.app.app:app --reload

Visit

http://127.0.0.1:8000/docs

[lfunderburk]

To do:

1. Run the tests, there are one or two warnings. See if you can fix them
2. Add a test under test/test_app.py to check that a given movie is returning the desired result
3. Think of other ways the user might interact with the api - does it make sense to add another entry point?

[jpjon]

Changed made to fix warnings:

Replaced @validator with @field_validator: https://events.bizzabo.com/458007/settings/registration/1eaNwY9AETInoqcCd_pBC3mdpyVozaTeGSNDrqw3yMsJPmiZu8MIS4Fbc0Qmsa1f

Added additional test for validating response:
https://events.bizzabo.com/458007/settings/registration/1eaNwY9AETInoqcCd_pBC3mdpyVozaTeGSNDrqw3yMsJPmiZu8MIS4Fbc0Qmsa1f

[lfunderburk]

Test coverage passes with new upgrades to pyproject.toml and app.py

Awesome!