dstl / Stone-Soup

A software project to provide the target tracking community with a framework for the development and testing of tracking algorithms.
https://stonesoup.rtfd.io
MIT License
385 stars 127 forks source link

Bug introduced in #832 #838

Closed sglvladi closed 10 months ago

sglvladi commented 10 months ago

In trying to fix one bug in #832, I inadvertently introduced another bug that somehow evaded all the tests.

This is caused by incorrect use of the walrus operator on line 244 of base.py, meaning that the variable excess always takes a Boolean value (i.e. 0 or 1), therefore leading to inappropriate execution of the while loop for each character in val, which in turn led to scripts hanging.