Skip to content

Make adding summary data optional in draw_ldc #9

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

Open
2 tasks
mps9506 opened this issue Jan 21, 2022 · 0 comments
Open
2 tasks

Make adding summary data optional in draw_ldc #9

mps9506 opened this issue Jan 21, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@mps9506
Copy link
Contributor

mps9506 commented Jan 21, 2022

Currently draw_ldc requires the summary data argument point to the output of summ_ldc in order to add the median measured loading points to the graph. This should be optional for two reasons. (1) Users have the option to draw the ldc easily without additional summary data. (2) If there is a desire to overlay output from other daily loading estimate methods (LOADEST, WRTDS, GAMs etc.), that can be manually added to the output of draw_ldc (I think).

To do:

  • make the .tbl_summ argument optional
  • add examples of incorporating GAM or LOADEST daily load estimates to the output of draw_ldc (this probably should be a pkgdown article)
@mps9506 mps9506 self-assigned this Jan 21, 2022
@mps9506 mps9506 added the enhancement New feature or request label Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant