rsgain
rsgain copied to clipboard
Change scan log file extension to tsv
This extension accurately represents the format of the file, which is TSV (tab-separated values) as opposed to CSV (comma-separated values).
It seems likely that this would address #56 without the need for 6402b2b.
This would be a breaking change. Currently, I'm only allowing changes to the program that are backwards compatible with previous versions.
One way to implement this while maintaining backwards compatibility would be to add another mode for the -O option.
This would be a breaking change. Currently, I'm only allowing changes to the program that are backwards compatible with previous versions.
I agree that this is something to tag or otherwise consider for a major release.
One way to implement this while maintaining backwards compatibility would be to add another mode for the
-Ooption.
That could be helpful. As evidenced by #56, the issue the program is currently facing is that it applies an erroneous extension to the log files it outputs, leading to misinterpretation both by users and applications. Maybe a strategy like the following would make sense:
- Non-breaking changes:
- Add one of the following modes—choose the one that is intended to be later replaced by the amended
-O:-Oc: output a CSV file with extension.csv- To account for mode chaining,
-Ocswould add redundant separator headersep=,to the CSV file.
- To account for mode chaining,
-Ot: output a TSV file with extension.tsv- To account for mode chaining,
-Otswould add redundant separator headersep=\tto the TSV file.
- To account for mode chaining,
- Add a warning in the readme that
-Ocurrently outputs a TSV file with extension.csv, noting that it will be amended in the next major release.
- Add one of the following modes—choose the one that is intended to be later replaced by the amended
- Breaking changes (e.g., for v4.0):
- Amend
-Osuch that its behavior conforms to the new mode (-Ocor-Ot). Note that this PR currently assumes the-Otbehavior for simplicity's sake. - Deprecate/remove the no-longer-needed new mode (
-Ocor-Ot). - Optionally deprecate/remove the
-Osmode as correcting the file extension renders the separator header redundant.
- Amend
Edit: simplified strategy—the previous revision detailed adding both of the new modes, which would be practical only if the intention is to expose output format specification as an option.