Skip to content

[Fix] plot_indices#221

Draft
Schiano-NOAA wants to merge 4 commits intodevfrom
fix-indices-plt
Draft

[Fix] plot_indices#221
Schiano-NOAA wants to merge 4 commits intodevfrom
fix-indices-plt

Conversation

@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

Make changes per issues #212 & #214

A new function gets created so that we can do the same plotting for other observed and predicted plot comparisons.

I also made an adjusted to process data so that the resulting df indexing variables are all character format.

@Schiano-NOAA
Copy link
Copy Markdown
Collaborator Author

@sbreitbart-NOAA for some reason, exporting the rda for plot_indices is not working now. Could you ID the issue and potentially fix it? It might be due to the facetting, but I'm not sure.

Here is an example:

plot_indices(example_data, make_rda = TRUE)
Error in if (file.exists(fs::path(dir, "captions_alt_text.csv"))) { : 
  argument is of length zero

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator

sbreitbart-NOAA commented Apr 3, 2026

@sbreitbart-NOAA for some reason, exporting the rda for plot_indices is not working now. Could you ID the issue and potentially fix it? It might be due to the facetting, but I'm not sure.

Here is an example:

plot_indices(example_data, make_rda = TRUE)
Error in if (file.exists(fs::path(dir, "captions_alt_text.csv"))) { : 
  argument is of length zero

The plot_indices.R file isn't up-to-date with that on the main branch, which contains the new key quantities extraction/insertion pipeline. Try a rebase?

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.

2 participants