Should we remove "Old ALS Code" in the `ch09/1.personalization.ipynb` ?

treygrainger / ai-powered-search

The codebase for the book "AI-Powered Search" (Manning Publications, 2024)

https://aipoweredsearch.com

152 stars 37 forks source link

Should we remove "Old ALS Code" in the `ch09/1.personalization.ipynb` ? #119

Closed alexott closed 4 months ago

alexott commented 6 months ago

It doesn't work anyway...

treygrainger commented 6 months ago

Yep, I'll take it out. The "old" code is cleaner (python vs. SQL to map back from numbers to string ids), but doesn't currently run. Kept it there to see if I could use it to refactor later, but will just remove it and check the history if ever needed.

treygrainger commented 4 months ago

This was committed on Feb 12: https://github.com/treygrainger/ai-powered-search/commit/0a74eebb0c09c152fbbbbaf2492f91df0a13e131.