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

Encoding "no physical parameters" in Trajectory files #22

Open
MarkIggy opened this issue Apr 15, 2023 · 3 comments
Open

Encoding "no physical parameters" in Trajectory files #22

MarkIggy opened this issue Apr 15, 2023 · 3 comments
Assignees
Labels
TrajFile Related to trajectory files

Comments

@MarkIggy
Copy link
Collaborator

How should we encode "no physical parameters" (no variables) in the Trajectory files? There will be cases where there is no "physical parameter" data to record in a Trajectory file but there is still other valuable information.

We have two options:

  1. N_PARAM = 1; TRAJECTORY_PARAMETERS= [" "]; no variables
  2. No N_PARAM, TRAJECTORY_PARAMETERS, or variables in the file

Which would be more instructive to the user?

(They may not be mutually exclusive. Could/should we allow both?)

@MarkIggy MarkIggy self-assigned this Apr 15, 2023
@MarkIggy MarkIggy added the TrajFile Related to trajectory files label Apr 16, 2023
@jegilson
Copy link

PRES is a fundamental dimension of the trajectory file, 2nd only to TIME. I would suggest PRES must be included (N_PARAM=1 minimum), and TRAJECTORY_PARAMETERS must include at least PRES, matching the N_PARAM definition. The PRES variables can be empty. Although it is hard for me to think that this is possible, even back in the earliest floats there was a few PRES data. In any case, the lack of any PRES information supplies the user information on the validity of whatever remains from a sparse traj netCDF as is described here.

@ClaudiaAOML
Copy link

we had a handful of protoptype floats long ago that did not report any pressure data during drift. Because that makes it hard to use the trajectory for deriving currents at a known depth we did not make trajectory files for these floats.

In addition to that, they aslo had only one position per cycle, which limits the usefullness of trajectory files even more.

@mscanderbeg
Copy link

I agree with John's comment and that we should have PRES, even if it is empty. This seems like a modified Option 1:
N_PARAM = 1; TRAJECTORY_PARAMETERS= ["PRES"]; PRES may be fillvalue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TrajFile Related to trajectory files
Projects
None yet
Development

No branches or pull requests

4 participants