Skip to content

Added option to specify output file name with -o flag. Currently only…#35

Merged
jhs507 merged 3 commits intomasterfrom
file_name_output_option
May 7, 2021
Merged

Added option to specify output file name with -o flag. Currently only…#35
jhs507 merged 3 commits intomasterfrom
file_name_output_option

Conversation

@jhs507
Copy link
Collaborator

@jhs507 jhs507 commented May 5, 2021

… takes realative paths.

@jhs507 jhs507 marked this pull request as ready for review May 6, 2021 22:51
@jhs507
Copy link
Collaborator Author

jhs507 commented May 6, 2021

This feature is ready for review. You should be able to specify a relative or absolute path for the output file location with the -o option flag.

@jhs507 jhs507 linked an issue May 6, 2021 that may be closed by this pull request
@KevinShook
Copy link
Collaborator

Worked for me -once I actually pulled the pull request.
One minor thing, which had me slightly confused, was that
the --help hasn't been updated yet:

$ ./crhm --help

crhm [options] PROJECT_FILE

	-h - Display this message.

	-t TIME_FORMAT - Select the format for date time outputs.
		Valid formats are:
		MS - Microsoft Excel floating point.
		ISO - ISO 8601 extended format YYYY-MM-DDThh:mm
		YYYYMMDD - YYYY MM DD hh mm

	-f OUTPUT_FORMAT - Select the file format for the output.
		Valid formats are:
		STD - Standard output format. Sutable for a spreadsheet.
		OBS - Observation file .obs format. Sutable for reading with CRHM GUI.


@jhs507
Copy link
Collaborator Author

jhs507 commented May 7, 2021

Thanks Kevin. I will make sure the --help message is updated.

@jhs507 jhs507 merged commit 0ee46e6 into master May 7, 2021
@jhs507 jhs507 deleted the file_name_output_option branch May 7, 2021 17:16
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

Successfully merging this pull request may close these issues.

Command line option to override the output file name

2 participants