Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update documentation for ParticleSet.from_particlefile #1198

Closed
claudiofgcardoso opened this issue Jul 15, 2022 · 3 comments
Closed

Update documentation for ParticleSet.from_particlefile #1198

claudiofgcardoso opened this issue Jul 15, 2022 · 3 comments

Comments

@claudiofgcardoso
Copy link

Hello all,

The addition of zarr as an output format (#1165) is a great step forward, as it significantly improves the storage efficiency of my large particle files! It is also important to note that the initialisation of a ParticleSet from a .zarr particle file is also possible. So I suggest to update the parcels documentation for ParticleSet.from_particlefile() so that other users know that this functionality is also possible with zarr.

Cláudio

@erikvansebille
Copy link
Member

Hi @claudiofgcardoso, thanks for the comment. Yes, it would be a good addition to also support zarr in ParticleSet.from_particlefile(). But I'll do that as part of the larger refactoring project this summer, where I'm implementing dumping directly to zarr during a pset.execute() (instead of dumping to temporary npy files).

There are at least three advantages of using zarr directly:

  1. It doesn't require a conversion step at the end of the run anymore (and thus fixes issues like Adding particles to pset in MPI run #1193 (comment)).
  2. It will be much easier to inspect data when the execute is still running.
  3. It will hopefully also reduce file size of the output

See the branch at https://github.com/OceanParcels/parcels/tree/dump_to_zarr to check progress. It's still a bit too early for a Pull Request, but hopefully this is a complete feature by the end of August!

@claudiofgcardoso
Copy link
Author

Hi @erikvansebille, glad to read that Parcels is migrating to zarr and I'm looking forward for this merge! There's no need to keep this issue open so I'll close it :)

@erikvansebille
Copy link
Member

Thanks @claudiofgcardoso! FYI, the branch is now also a PR, see #1199

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

No branches or pull requests

2 participants