Skip to content

Conversation

@nhz2
Copy link
Member

@nhz2 nhz2 commented Mar 6, 2025

This should help prevent over 1000 files from being in a directory, which can be an issue on some file systems.

Instead of storing snapshots in "traj/snap<i>.zarr.zip" they are now in "traj/<i÷1000>/<i%1000 zero padded to 3 digits>.zip" so for example, "traj/0/000.zip", "traj/0/001.zip" ... "traj/42/999.zip".

There is a function step_path(step::Int)::String to convert, for example, from 1 to "0/001.zip" and a function steps_traj_dir(traj::String)::Vector{Int} that
returns the snapshot steps from low to high found in the traj directory.

This PR also removes the deprecated run_sim function.

Lastly, when the simulation is run, the step variable is now updated before the calling loop instead of after.
This means the first call to loop gets step = 1 instead of step = 0

@nhz2 nhz2 requested a review from Leibniz1697 March 6, 2025 16:13
@nhz2 nhz2 merged commit 7cc30b5 into main Mar 7, 2025
6 checks passed
@nhz2 nhz2 deleted the change-output-format branch March 7, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants