Skip to content
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

Issue #103: Update doubleFilter.py #104

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

GonzoBS
Copy link

@GonzoBS GonzoBS commented Nov 6, 2022

Probleme mit dem double Filter.
Immer wenn die Liste nach Zeit gelöscht wird, wird das nächste doppelte Paket als ok durchgelassen. Die Liste ist dann bis auf einen Eintrag leer, der erste Eintrag wird aber nicht auf Duplikat geprüft. Einstellung ist RIC, SUBRIC, Message. Die Zeit habe ich zum testen auf 2 sekunden gestellt.

type: module
res: filter.doubleFilter
config:
ignoreTime: 2
maxEntry: 40
pocsagFields:
ric
subric
message

#103

Probleme mit dem double Filter.
Immer wenn die Liste nach Zeit gelöscht wird, wird das nächste doppelte Paket als ok durchgelassen. Die Liste ist dann bis auf einen Eintrag leer, der erste Eintrag wird aber nicht auf Duplikat geprüft.
Einstellung ist RIC, SUBRIC, Message. Die Zeit habe ich zum testen auf 2 sekunden gestellt.

type: module
res: filter.doubleFilter
config:
ignoreTime: 2
maxEntry: 40
pocsagFields:
ric
subric
message
@GonzoBS GonzoBS changed the title Update doubleFilter.py Issue #103: Update doubleFilter.py Nov 6, 2022
if all(listPacket.get(x) == bwPacket.get(x) for x in filterFields):
logging.debug("found duplicate: %s", bwPacket.get("mode"))
return False

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier muss der Whitespace noch raus, damit die Tests durchlaufen

Suggested change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants