Open mogres opened 1 year ago
@mogres is there still work to do on this issue or was it closed by https://github.com/simularium/simulariumio/pull/155
@mogres is there still work to do on this issue or was it closed by #155
Yes, we can close this issue now.
System and Software
Description
While appending new
AgentData
obtained from theCytosimConverter
to an existingTrajectoryData
object, I ran into the following issues:used_uids
,new_agent_index
andraw_uid
are being saved as floats instead of ints, which caused issues during indexingresult.display_data
dictionary was not updated during the call toappend_agents
, which resulted in the added agents defaulting to a sphere geometryget_copy_with_increased_buffer_size
was too large (1000) for my use case. This resulted in 1000 elements being added every step even when they were not necessary.Expected Behavior
New
AgentData
should be added to the existingTrajectoryData
with itsDisplayData
carried over.Reproduction
CytosimData
object (fiber_data
) from a cytosim output fileCytosimConverter
object (cytosim_converter
) from theCytosimData
object.CytosimConverter
objectnew_cytosim_converter
I can share the data and a notebook with the reproduction example if needed.
More Information
Local changes I made to fix these issues are available at https://github.com/mogres/simulariumio/tree/multiple_cytosim_repeats