Merge pull request #44 from andrewm4894/test-pre-commit-pr

make streamlit use wide mode

.github/workflows/pre-commit.yaml

@@ -32,4 +32,4 @@ jobs:
           body: |
             :x: **pre-commit** failed.
             Please run `pre-commit run --all-files` locally and commit the changes.
-            Find more information in the repository's CONTRIBUTING.md
+            Find more information in the repository's CONTRIBUTING.md

.pre-commit-config.yaml

@@ -19,9 +19,3 @@ repos:
     hooks:
       - id: isort
         args: ["--profile", "black"]
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.5
-    hooks:
-      - id: ruff
-        args: ["--ignore", "E501, E711, E712"] # ignore line length and comparison to None, refer full list of codes here: https://pycodestyle.pycqa.org/en/latest/intro.html#error-codes
-      - id: ruff-format

Makefile

@@ -3,6 +3,7 @@ SHELL=/bin/bash
 .PHONY: dashboard
 .PHONY: local
 .PHONY: docker
+.PHONY: pre-commit
 
 ## start streamlit dashboard
 dashboard:
@@ -16,11 +17,14 @@ local:
 docker:
 	docker compose up -d --build
 
+## pre-commit
+pre-commit:
+	pre-commit run --all-files
+
 ## setup local development environment and install dependencies
 dev:
 	pre-commit install
 
-
 # Self-documenting Makefiles:
 # https://gist.github.com/klmr/575726c7e05d8780505a
 .DEFAULT_GOAL := show-help

dashboard.py

@@ -17,6 +17,8 @@
 
 load_dotenv()
 
+st.set_page_config(layout="wide")
+
 
 def plot_time_series(df, metric_name) -> go.Figure:
     """
@@ -61,6 +63,7 @@ def plot_time_series(df, metric_name) -> go.Figure:
         title_text=f"{metric_name} (n={len(df)})",
         hovermode="x",
         legend=dict(orientation="h", yanchor="bottom", y=1.02, xanchor="right", x=1),
+        autosize=True,
     )
 
     return fig

metrics/examples/hackernews/hn_top_stories_scores.yaml

@@ -12,4 +12,3 @@ alert_always: False
 alert_methods: "email"
 ingest_fn: >
   {% include "./examples/hackernews/hn_top_stories_scores.py" %}
-

metrics/examples/netdata/netdata.yaml

@@ -12,4 +12,3 @@ alert_always: False
 alert_methods: "email"
 ingest_fn: >
   {% include "./examples/netdata/netdata.py" %}
-

metrics/examples/python/python_ingest_simple/python_ingest_simple.yaml

@@ -5,4 +5,3 @@ train_cron_schedule: "*/4 * * * *"
 score_cron_schedule: "*/3 * * * *"
 ingest_fn: >
   {% include "./examples/python/python_ingest_simple/ingest.py" %}
-

metrics/examples/weather/weather.yaml

@@ -12,4 +12,3 @@ alert_always: False
 alert_methods: "email"
 ingest_fn: >
   {% include "./examples/weather/ingest_weather.py" %}
-

metrics/examples/weather_forecast/weather_forecast.yaml

@@ -11,4 +11,3 @@ alert_always: False
 alert_methods: "email"
 ingest_sql: >
   {% include "./examples/weather_forecast/weather_forecast.sql" %}
-

metrics/examples/yfinance/yfinance.yaml

@@ -12,4 +12,3 @@ alert_always: False
 alert_methods: "email"
 ingest_fn: >
   {% include "./examples/yfinance/yfinance.py" %}
-