-
Notifications
You must be signed in to change notification settings - Fork 186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Infinite state, unless k/t parameters are specified. #85
Comments
|
I'll explain in more detail. We'd like to be 100% certain about the vehicle position. Unfortunately barefoot doesn't allow that at the moment. S - state vector S(0) C1(5%) .....after N points, S(N)...S(0) S(0) C2(40%) - the only one left. Now you know for sure this is the correct position. To get this behaviour we implement our oun logic. We have a wrapper around your KState, that alows us to peek into your private data. But with the new version we get C3 is still wrong. There are no transitions from this one to the next position, but it's still here because it was the best estimate at S(0) So now our custom logic doesn't work, and we are left with an infinite state. I hope what I said makes sence. |
The best solutions is actually to have the essential feature in the mainline. The reason is: (1) Only this way, I can efficiently help on that because everything else is more or less groping in the dark. (2) Only this way, the feature will also be supported in future releases and won't break again.
Technically I see two possibilities to address the problem: |
Thinking about a solution, me and my colleagues were thinking about option a) but both solutions will work. |
Okay, i think I've found a very simple solution. However, I have to check that it doesn't break commit cd1b275. |
…pler fix for #83. Change-Id: I409c4e2a3bc151f6cabc19224871f3935ccb486d
This issue was fixed with commit 25fb150 and is now pushed with version 0.1.4. |
We use version 0.1.1 in production and it's working fine. However we have issues with the follow up versions. I think we have determined the problem and I'll try to explain.
It works with version 0.1.1. In version 0.1.2 however the sequence was modified to hold the current estimate.
Triple<Set, S, C>> sequence
And there is an aditioanal if statement in the remove() function of KState.
It doesn't allow the estimate of the predecessor to be removed. That leads to an infinite state unless there is a time or count limit, which is not our case. In our situation we never get to the point where there is exactly one candidate, so we can't clear the state. Eventually we receive a Stackoverflow exception.
The text was updated successfully, but these errors were encountered: