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.

Sign up for GitHub

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

Jump to bottom

Rename -extra_* and -ts_* to -spatial_* and -scalar_*, respectively #509

Open
ckhroulev opened this issue Jun 6, 2022 · 1 comment
Open

Rename -extra_* and -ts_* to -spatial_* and -scalar_*, respectively #509

ckhroulev opened this issue Jun 6, 2022 · 1 comment
Assignees
@ckhroulev
Labels
feature_request

Comments

@ckhroulev
Copy link
Member

This change in names of command line options and configuration parameters would clarify their meanings: right now -extra_vars are named this way because these are additional (extra) outputs (compared to contents of an output file written at the end of a run) and -ts_vars are "time-series". But -ts_vars are also "extra" and -extra_vars are also "time-series" of spatially-variable quantities...

New names clearly identify that one corresponds to reporting spatially-variable quantifies and the other one to scalars.
@ckhroulev ckhroulev self-assigned this Jun 6, 2022
@bueler
Copy link
Member

bueler commented Jun 7, 2022

👍

ckhroulev added a commit that referenced this issue Oct 14, 2022
@ckhroulev
Rename -extra_{file,times,vars} to -spatial_{file,times,vars}; see #509
b020380
ckhroulev added a commit that referenced this issue Oct 14, 2022
@ckhroulev
Rename -ts_{file,times,vars} to -scalar_{file,times,vars}; see #509
2f4114b
ckhroulev added a commit that referenced this issue Oct 17, 2022
@ckhroulev
Rename -extra_{file,times,vars} to -spatial_{file,times,vars}; see #509
2ea8d1f
ckhroulev added a commit that referenced this issue Oct 17, 2022
@ckhroulev
Rename -ts_{file,times,vars} to -scalar_{file,times,vars}; see #509
bae3c22
ckhroulev added a commit that referenced this issue Jan 27, 2023
@ckhroulev
Rename -extra_{file,times,vars} to -spatial_{file,times,vars}; see #509
699077a
ckhroulev added a commit that referenced this issue Jan 27, 2023
@ckhroulev
Rename -ts_{file,times,vars} to -scalar_{file,times,vars}; see #509
45422b9
ckhroulev added a commit that referenced this issue Oct 30, 2023
@ckhroulev
Rename -extra_{file,times,vars} to -spatial_{file,times,vars}; see #509
773f331
ckhroulev added a commit that referenced this issue Oct 30, 2023
@ckhroulev
Rename -ts_{file,times,vars} to -scalar_{file,times,vars}; see #509
166ac48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ckhroulev ckhroulev

Labels
feature_request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ckhroulev @bueler