Skip to content

Conversation

@arrjon
Copy link
Member

@arrjon arrjon commented Apr 14, 2025

The change corrects a conditional statement to ensure proper validation of the length of variable_names against the shape of x. The previous statement only worked for integer smaller than 255 as is not checks for object identity not the values.

@arrjon arrjon requested a review from stefanradev93 April 14, 2025 13:12
@arrjon arrjon self-assigned this Apr 14, 2025
@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
bayesflow/utils/dict_utils.py 62.12% <100.00%> (ø)

@stefanradev93 stefanradev93 merged commit 12921dc into dev Apr 14, 2025
15 checks passed
@stefanradev93 stefanradev93 deleted the fix_dict_utils branch April 14, 2025 22:44
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.

3 participants