Update pynspect query semantic from "return bool" to "return bool deciding data"
Expression "ConnCount or FlowCount" returns simple True or False, where for greater pynspect versatility it would make sense to return value, which decided boolean expression, like for example python expressions.
There might be pitfalls though. Pynspect within Mentat often works on lists of lists, and list full of empty lists in this context makes sense as False. This semantics must stay, so other places, depending on this behaviour, must be identified and fixed.
Lib is very well covered by unit tests, so this will be good pointer (however, it should be verified that these corner cases are covered).
Updated by Jan Žerdík about 3 years ago
- Assignee changed from Jan Žerdík to Jan Mach
Waiting for accepting pull request
Updated by Jan Mach about 3 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 90
I have just merged the pull request to the base code, built WHL packages and updated dependencies in Mentat system. I will now build Mentat package, deploy it to the development server and watch for any possible problems.