fix: pass run_main/vacant/hedonic/ensemble flags through finalize_models

[drussellmrichie]

Problem

finalize_models() accepted no model-type flags and hardcoded run_main=True, run_vacant=True, run_hedonic=True, run_ensemble=True in its internal call to benchmark.run_models().

This means callers that disable certain model types (e.g. run_vacant=False, run_hedonic=False) would still trigger those model types during finalize_models, causing crashes when no sales data exists for those types -- specifically a ValueError: Input data must be 2 dimensional and non empty from LightGBM when the validation set is empty.

Note that try_models() already accepts and passes through these flags correctly; finalize_models() was inconsistent.

Fix

Add run_main, run_vacant, run_hedonic, run_ensemble keyword arguments to finalize_models with the same defaults as before (all True), and pass them through to benchmark.run_models. The docstring is updated accordingly.

[github-actions]

Thank you for your contribution.
Please sign our CLA at the following link:
Click here to sign the CLA.
A maintainer will verify your signature and confirm it here by commenting with the following sentence:

---

I affirm that this contributor has signed the CLA

---

_{You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.}