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.

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

Feature request: Add data export option for Flexibility Index time plot #123

Open
rosamaryo opened this issue Apr 3, 2021 · 2 comments
Open
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@rosamaryo
Copy link

rosamaryo commented Apr 3, 2021

Currently, the output options of Flexibility index are: time plot, common plot, summary table (1 row per org attribute) or full data. What I think is missing and would be really helpful is output of the data used in the time trend chart (1 row per week with the average FI) to be able to recreate that chart in ppt and keep adding weeks of data - customer will do this to track this on a monthly basis as a KPI.

Describe the solution you'd like

return = "table_time"

Describe alternatives you've considered
The solution now is to just save the full data (return = "data"), and do a simple summary:

group_by(Date) %>%
summarise(FlexibilityIndex = mean(FlexibilityIndex)) %>%

which maybe is the right approach if we don't want to start creating very specific data outputs. If that's the case, I suggest to just save this approach as an FAQ. Thank you!

@martinctc
Copy link
Member

Thanks @rosamaryo for this issue. I lean towards adding this as an extra example in the section under https://microsoft.github.io/wpa/reference/flex_index.html, because it is only a one-line code to run. Maybe good to get some thoughts from current users to see what is preferred, because keeping the return options simple would also be great

@martinctc martinctc added the documentation Improvements or additions to documentation label Jul 6, 2021
@martinctc martinctc self-assigned this Jul 6, 2021
@martinctc
Copy link
Member

I will add the following example to the documentation, instead of adding another return option:

To return the underlying data for the time trend view, you may run the following code: 
em_data %>%
  flex_index(return = "data")  %>%
 group_by(Date) %>%
 summarise(FlexibilityIndex = mean(FlexibilityIndex))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants