Skip to content

EntityStatePDU: EntityID vs EntityIdentifier #88

Closed

Description

@joeclarkia

In November 2025 a change was merged to open-dis-python which changed the "entityID" in EntityStatePDU from an "EntityID" type to an "EntityIdentifier" type. Doesn't this fundamentally break compatibility with the DIS standards? Also, it breaks compatibility with the other open-dis implementations (I'm seeing it on Python->Javascript). Am I missing something?

Activity

  1. ngjunsiang commented on Oct 18, 2026

    @ngjunsiang
    Contributor

    I'll note here that the packet format was never affected — both spellings serialize the same 6-byte Entity Identifier record (§6.2.28: site u16 + application u16 + entity number u16), so network interoperability with the other open-dis implementations is unchanged.

    You're right, though, that the in-memory object shape changed, which breaks application-level code like your Python→JS glue that used the flat siteID/applicationID/entityID names.

    Filing a PR for backward compatibility: #91 restores EntityID as an alias of EntityIdentifier, with property setters.

  2. added a commit that references this issue on Oct 19, 2026
    727fcb8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions