ImperialCollegeLondon / ImperialCollegeLondon/SWMManywhere

`calculate_weights` uses freshly initialised `weights` rather than from `config`

Open
#355 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
51
Forks
11
PR merge metrics
No merged PRs in 30d

Description

So you have to include all weights..

Workaround for now: set all the weights with arbitrary values, they will be ignored in weight calculations by any custom weights values in `config`

Contributor guide

Open the contributing guide

Research direction

Start by locating the calculate_weights entry point and tracing how weights are initialized and read from config. Reproduce the issue with only some weights supplied, then verify that configured custom weights are used without requiring every weight to be present.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.