openclimatefix / openclimatefix/data-platform

improve error message for >=1.1 forecast

Open
#197 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
3
Forks
1
Avg merge
44m
Merged PRs (30d)
2

Description

Verification
  • I have searched the existing issues to ensure this feature has not already been requested.
Provide a detailed description of the proposed feature.

If we try to add a forecast that is over 1.1 of the capacity, then we get a funny error that says smallint is too big.

It would be good if we get this error beforehand

What is the motivation for the feature?

we have got this error in live, and it was a bit confusing to debug

Outline a potential approach or solution (optional).

No response

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No file, test, or entry point is named. Trace the forecast-creation path and the capacity value handling first; done means forecasts at or above the reported threshold receive a clear validation error before the confusing smallint failure occurs.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.