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

Requirement: Combine and drop unused / underspecified bit flags #23

Open
krischer opened this issue Jan 3, 2018 · 13 comments
Open

Requirement: Combine and drop unused / underspecified bit flags #23

krischer opened this issue Jan 3, 2018 · 13 comments

Comments

@krischer
Copy link
Contributor

krischer commented Jan 3, 2018

Simplification of MiniSEED2: Combine and drop unused / underspecified bit flags.

@crotwell
Copy link

crotwell commented Jan 8, 2018

Provide standard mapping for any that are deemed important to the addition/optional header structure.

@krischer
Copy link
Contributor Author

Summary

(Please let me know if I missed a point or misunderstood something)

Please vote on:

  1. Should unused/underspecified bit-flags be dropped? (Yes/No)

@kaestli
Copy link

kaestli commented Jan 29, 2018

do we need to agree on what is "widely unused" or "underspecified"?
imho, i have seen little recent usage of everything except the a) time correction flag, b) the leap second flags and c) the the clock lock flag (however for a and b better solutions are available),
and i think everything is underspecified except the time correction and the leap second flags (thus, the clock lock flag is actually underspecified too)

@krischer
Copy link
Contributor Author

I'd say no as I think most of us agree on which fields are actually used. Furthermore it is a detail that can be figured out in the final stage with a fully specified format.

@crotwell
Copy link

yes

@chad-earthscope
Copy link
Member

  1. Yes.

The ones I would like to drop are:

  • I/O flags, Bit 0: station volume parity error possibly present
  • I/O flags, Bit 1: long record read
  • I/O flags, Bit 2: short record read
  • I/O flags, Bit 3: start of time series
  • I/O flags, Bit 4: end of time series

But to be honest all of them can be retained if we have optional header capability. With some marked as deprecated if we wish to discourage their use. The ones above are just the most worthless of the flags and we could drop them to ease the coding to the new standard a bit.

@kaestli
Copy link

kaestli commented Jan 30, 2018

yes

@ozym
Copy link

ozym commented Jan 30, 2018

  1. Yes

@ketchum-usgs
Copy link

Yes, Chad's list is a good starting list. I suspect that "clock locked" is also useless and that this should be communicated with a better value.

@claudiodsf
Copy link

Yes

@ihenson-bsl
Copy link

  1. Yes

2 similar comments
@ValleeMartin
Copy link

  1. Yes

@JoseAntonioJara
Copy link

  1. Yes

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

No branches or pull requests

10 participants