multinet-app / multinet-api

Apache License 2.0
2 stars 2 forks source link

Require that Sessions.state is a python dict, not string #161

Closed JackWilb closed 1 year ago

JackWilb commented 1 year ago

Does this PR close any open issues?

No

Give a longer description of what this PR addresses and why it's needed

With our prior implementation, the state could be saved as any type, including string. Since the trrack library was exporting as string, this meant that saved state were of type string. We added a couple new functions to the trrack library to allow for importing and exporting the actual object, and they work seamlessly with these changes. There will be PRs in all vis apps to align with this change and get to a more consistent database state.

Provide pictures/videos of the behavior before and after these changes (optional)

N/A

Are there any additional TODOs before this PR is ready to go?

No