Skip to content

First forecast test version#359

Merged
killer0071234 merged 5 commits into
mainfrom
forecast-test
Jul 16, 2026
Merged

First forecast test version#359
killer0071234 merged 5 commits into
mainfrom
forecast-test

Conversation

@killer0071234

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 15, 2026 21:03
@killer0071234 killer0071234 linked an issue Jul 15, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces an initial implementation of forecast retrieval/parsing for GeoSphere Austria data in ZamgData, along with tests and related developer-environment/tooling updates.

Changes:

  • Add forecast endpoints plus helpers to extract “current” forecast values and trim forecast payloads to “now onward”.
  • Add initial forecast tests and adjust existing async-failure testing to work with aresponses v3.
  • Update dev tooling/config (Poetry dependency groups, pre-commit hooks, devcontainer/Dockerfile, VS Code launch config).

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
src/zamg/zamg.py Adds forecast URLs, forecast caching, parsing helpers, and forecast parameter/metadata handling.
tests/test_zamg.py Adds forecast-related tests and adjusts a connection-failure test to avoid old aresponses behavior.
src/zamg/__init__.py Import formatting cleanup for exception exports.
requirements.txt Bumps aresponses to 3.0.0.
requirements_test.txt Bumps aresponses to 3.0.0.
pyproject.toml Moves dev dependencies to Poetry’s dependency group format.
forecast_test.py Adds a new example script to manually exercise forecast calls (currently named like a pytest test module).
examples/read_data.py Minor formatting (blank line).
Dockerfile.dev Simplifies apt setup and consolidates pip installs with constraints.
control.py Minor formatting (blank line).
.vscode/launch.json Switches debugger type to debugpy and adds a launch config for forecast_test.py.
.pre-commit-config.yaml Adds isort, removes reorder-python-imports, and tweaks bandit deps.
.flake8 Adds E203 ignore (Black compatibility).
.devcontainer/devcontainer.json Removes Intellicode extension from the devcontainer extensions list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/zamg/zamg.py
Comment thread src/zamg/zamg.py Outdated
Comment thread src/zamg/zamg.py
Comment thread src/zamg/zamg.py Outdated
Comment thread src/zamg/zamg.py Outdated
Comment thread src/zamg/zamg.py Outdated
Comment thread src/zamg/zamg.py Outdated
Comment thread src/zamg/zamg.py Outdated
Comment thread tests/test_zamg.py
Comment thread forecast.py
killer0071234 and others added 4 commits July 15, 2026 23:10
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@killer0071234
killer0071234 merged commit 683d41f into main Jul 16, 2026
@killer0071234
killer0071234 deleted the forecast-test branch July 16, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Available Forecast Data Integration

2 participants