Skip to content

Centralise the binary file saving metadata #3442

Open

Description

@JoeZiminski

Following on from a discussion at #3245. At present there is a binary.json metadata file written during the saving of binary data. This duplicates the si_folder.json metadata which has caused some problems with failure to propagate metadata and confusing behaviour during debugging #3245. The logic flow for the data-saving is discussed here. It would be nice to completely deprecate binary.json and use only si_folder.json to centralise the metadata.

Activity

  1. jmamodeo commented on Nov 6, 2026

    @jmamodeo

    I’d be happy to help with this. Since #4671 moved properties, annotations, and probe metadata into separate files, is deprecating binary.json still the intended direction?

    Would you prefer keeping it as the binary-storage configuration, with si.load(folder) documented as the way to restore the complete recording, or migrating away from it while retaining support for older folders?

    I’d suggest a small first PR for the agreed direction, with regression tests covering properties and annotations on save/load, plus legacy-folder compatibility. I’d leave new metadata fields and fingerprints out of scope. Happy to put together the implementation once the direction is clear.

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