One cannot merge the nodes immediately below the root

ecmwf / ecpoint-calibrate

Interactive GUI (developed in Python) for calibration and conditional verification of numerical weather prediction model outputs.

GNU General Public License v3.0

21 stars 8 forks source link

One cannot merge the nodes immediately below the root #155

Closed FatimaPillosu closed 3 years ago

FatimaPillosu commented 3 years ago

It is not possible to merge the nodes immediately below the root. This is a bug. Please, allow that.

onyb commented 3 years ago

To reproduce: add a breakpoint on the Root node, and then add another one at the same level.

onyb commented 3 years ago

While fixing this bug, I found out that there was another bug while introducing breakpoints manually. We were not making sure that the breakpoint is introduced in the right place (relative to other breakpoints). Note that this does not apply to breakpoints introduced by running the KS Test.

Both these bugs have been fixed and will be available in the next release.

onyb commented 3 years ago

Fixed in v0.28.0.