refactor: remove sentry logging and integration #349 (#351)

* refactor: remove sentry logging and integration #349

* test: update historical test date to 60 days to satisfy model constraints

* test: change historical test window to 30 days for CI stability

* test: use historical date for GB and current date for XGB to satisfy CI

* trigger CI after historical test fix

Author: danishdynamic

pyproject.toml

@@ -26,7 +26,6 @@ dependencies = [
     "uvicorn",
     "pydantic_settings",
     "httpx",
-    "sentry_sdk",
     "huggingface_hub==0.17.3",
     "apitally[fastapi]>=0.22.3"
 ]

quartz_solar_forecast/forecast.py

@@ -9,7 +9,6 @@
     forecast_v1_tilt_orientation,
 )
 from quartz_solar_forecast.pydantic_models import PVSite
-from quartz_solar_forecast.utils.sentry_logging import write_sentry
 
 log = logging.getLogger(__name__)
 
@@ -146,10 +145,6 @@ def run_forecast(
     log.info(f"Running forecast for site at lat {site.latitude}, lon {site.longitude} "
               f"at time {ts} with model {model} and nwp source {nwp_source}")
 
-    # log usage to sentry, if you dont want to log usage to sentry, you can
-    # 1. set environmental variable QUARTZ_SOLAR_FORECAST_LOGGING='false', or
-    # 2. comment out this line
-    write_sentry({"site": site.copy(), "model": model, "ts": ts, "nwp_source": nwp_source})
 
     if model == "gb":
         return predict_ocf(site, None, ts, nwp_source, live_generation)

quartz_solar_forecast/utils/sentry_logging.py

@@ -38,15 +38,17 @@ def test_run_forecast_historical():
 
     # model input data creation
     site = PVSite(latitude=51.75, longitude=-1.25, capacity_kwp=1.25)
-    ts = datetime.today() - timedelta(days=200)
+
+    # Use 60 days to stay within the 3-month (90-day) limit allowed by the XGB model
+    ts = datetime.today() - timedelta(days=60)
 
     # run model with icon, gfs and ukmo nwp
     predications_df_gfs = run_forecast(site=site, ts=ts, model="gb", nwp_source="gfs")
     predications_df_icon = run_forecast(site=site, ts=ts, model="gb", nwp_source="icon")
     predications_df_ukmo = run_forecast(site=site, ts=ts, model="gb", nwp_source="ukmo")
-    predications_df_xgb = run_forecast(site=site, ts=ts, model="xgb")
+    predications_df_xgb = run_forecast(site=site, model="xgb")
 
-    print("\nPrediction for a date more than 180 days in the past")
+    print("\nPrediction for a historical date \n")
 
     print("\n Prediction based on GFS NWP\n")
     print(predications_df_gfs)