Shadowserver NTP Version Report parsing issue about intelmq

[elsif2]

The sys_ prefix was added inadvertently by an upstream update to our scanning tool. The prefix will not appear going forward from 2022-08-02.

[ClevenL]

Another solution if value type preservation is preferred:
Include an additional definition that extracts sys_:

def convert_jitter_float(value: str) -> Optional[float]:
    """ 
    Returns an float or None for empty strings. 
    Used to preserve float value type for jitter
    field in ntp_version reports.
    """
    if re.match(r'^sys_', value):
        return float(value[4:]) if value else None

    return float(value) if value else None

and use it:

('extra.', 'jitter', convert_jitter_float),

[sebix]

I don't know the intent of Shadowserver changing the type/content here, but removing the prefix sys_ looks cleaner to me. Except if there will be other prefixes in the future.
@elsif2 do you know more on this?