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

[FEATURE] Disable pforma for RT130 and Geode/Stratavisor data #530

Open
hrotman-pic opened this issue Apr 25, 2024 · 0 comments
Open

[FEATURE] Disable pforma for RT130 and Geode/Stratavisor data #530

hrotman-pic opened this issue Apr 25, 2024 · 0 comments

Comments

@hrotman-pic
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Loading RT130 data or Geode/Stratavisor (multichannel) data with pforma is highly likely to result in an invalid archive.

Describe the solution you'd like
Only allow ingestion of these data via command line. These formats are rarely used in PH5 and data sizes are very likely to be relatively small.

Describe alternatives you've considered
The RT130 problem is related to response table. The multichannel data is related to data from the same DAS S/N not all being in the same mini. It would be possible to open issues requesting relevant modifications, but given the possible number of affected datasets (low) I think removing the ability to ingest these formats with pforma is the most efficient/viable solution.

Additional context
Ingestion error messages for RT130 data when using pforma are somewhat different than error messages when using command line. It may be useful to permit a 'debug only mode' for RT130 data, or pforma and terminal output that the PH5 should not be used/1302ph5 log is for debugging purposes only. But again, the relevant number of datasets are likely very small and it may not be worth trying to do this.

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

1 participant