Documentation says:
|
- `lowest` (float) — Lower bound of this segment (inclusive). Slightly below 0.0 (e.g. `-0.01`) at the start to handle precision. |
|
- `highest` (float) — Upper bound of this segment (inclusive). Slightly above 1.0 (e.g. `1.01`) at the end. |
But in the code it looks like they're not inclusive:
|
if value > lowest and value < highest: |
I think the documentation is incorrect because based on values in macro.json it makes sense to treat ranges as exclusive.
Documentation says:
mpfb2/docs/fileformats/target_metadata.md
Lines 91 to 92 in 437dd51
But in the code it looks like they're not inclusive:
mpfb2/src/mpfb/services/targetservice.py
Line 915 in 437dd51
I think the documentation is incorrect because based on values in
macro.jsonit makes sense to treat ranges as exclusive.