Skip to content

Fixed typo in function#38

Open
bjarketol wants to merge 1 commit intoEUFLOW:mainfrom
bjarketol:fix-bug-in-weighted-quantile
Open

Fixed typo in function#38
bjarketol wants to merge 1 commit intoEUFLOW:mainfrom
bjarketol:fix-bug-in-weighted-quantile

Conversation

@bjarketol
Copy link
Contributor

@bjarketol bjarketol commented Jan 19, 2026

Fixed typo bug

  • File: wifa/pywake_api.py:96
  • Changed np.suyamlFilem(sample_weight) → np.sum(sample_weight)
  • This was a critical bug that would cause a runtime error if weighted_quantile() was ever called

Removed redundant deflection model check

  • File: wifa/pywake_api.py:766-767
  • Removed the redundant elif deflection_model_data["name"] == "None" block
  • This was already covered by the .lower() == "none" check above it

Fixes #37

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.

fix bug in weighted_quantile

1 participant