-
Notifications
You must be signed in to change notification settings - Fork 0
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
Standardize plotting colors with flexibility to change #30
Comments
reminder: use color-blindness-friendly palettes. |
It might be worth starting with JasonPoh's color and linestyle choices. |
Right now theses are the defaults as far as colors and linestyles go.
Which I believe were taken from JasonPoh's defaults, and I know viridis is supposed to be friendly. |
@voetberg are these set in the main input config yaml file? |
@bnord - sure are. There are a set of defaults ( |
sorry, I'm silly. I couldn't find the config file. |
where is it? |
There's no general config file - it's always supplied by the user when running. DeepDiagnostics/tests/conftest.py Line 60 in 2a9817f
Which is used here: DeepDiagnostics/src/client/client.py Line 62 in 2a9817f
Descriptions and documentation for each one of those fields is still in progress until there's a defined list of plots and metrics |
I was thinking that we'd do a config file input so that users didn't have to edit code AND so that they weren't relegated to long CLI strings. |
Sorry I think I'm explaining this poorly. They can use a config file (using the --config flag), and they don't need to edit code. There just isn't a supplied config template yet because not all the metrics/plots we want to use are in place. Once that's completely done, we can brainstorm what fields we want them to modify and what plots/metrics to show off in the template. |
oh, sorry. I get it. thanks. |
This might include other **kwargs
The text was updated successfully, but these errors were encountered: