Skip to content

Remove deprecated /indicators/base API aliases #624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Copilot wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Remove deprecated /indicators/base API aliases #624

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ba88b65
Initial plan
Copilot Aug 26, 2025
9b46b52
Remove /indicators/base aliases from API
Copilot Aug 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions luts.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,13 @@
"/beetles/point/",
"/elevation/point/",
"/taspr/point/",
"/indicators/base/point/",
"/ncr/permafrost/point/",
"/eds/hydrology/point/",
"/alfresco/flammability/area/",
"/alfresco/veg_type/area/",
"/beetles/area/",
"/elevation/area/",
"/taspr/area/",
"/indicators/base/area/",
]

# Used for generating output for vector_data for /places/all
Expand Down
12 changes: 3 additions & 9 deletions routes/indicators.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""Endpoints for climate indicators

These endpoint(s) query a coverage containing summarized versions of the indicators dataset created from the 12km NCAR dataset.
The thresholds and eras are preconfigured in the coverage. Calling this the "base" indicators for now (i.e., url suffix: /indicators/base).
The thresholds and eras are preconfigured in the coverage.
"""

import json
Expand Down Expand Up @@ -498,12 +498,9 @@ def run_fetch_cmip6_indicators_point_data(lat, lon):
return render_template("500/server_error.html"), 500


@routes.route(
"/indicators/base/point/<lat>/<lon>"
) # original route, kept for backwards compatibility
@routes.route(
"/indicators/cmip5/point/<lat>/<lon>/"
) # new route, matches API documentation
) # route matches API documentation
def run_fetch_cmip5_indicators_point_data(lat, lon):
"""Query the NCAR 12km indicators_climatologies rasdaman coverage which contains indicators summarized over NCR time eras

Expand Down Expand Up @@ -558,12 +555,9 @@ def run_fetch_cmip5_indicators_point_data(lat, lon):
return render_template("404/no_data.html"), 404


@routes.route(
"/indicators/base/area/<var_id>"
) # original route, kept for backwards compatibility
@routes.route(
"/indicators/cmip5/area/<var_id>/"
) # new route, matches API documentation
) # route matches API documentation
def get_cmip5_indicators_area_data(var_id):
"""Area aggregation data endpoint. Fetch data within polygon area for specified variable and return JSON-like dict.

Expand Down