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

Researcher can view Results Pane for each cluster #107

Open
eethann opened this issue Sep 30, 2021 · 4 comments
Open

Researcher can view Results Pane for each cluster #107

eethann opened this issue Sep 30, 2021 · 4 comments

Comments

@eethann
Copy link
Collaborator

eethann commented Sep 30, 2021

Time Estimate: 4 hours

Researchers should see the integration results for each cluster on the cluster detail page in the population section. See this frame in Figma mockup: https://www.figma.com/file/QImAyi1guE4AICGMN7uODK/Drawdown-Portfolio-UI-[Figma-Chakra-UI]?node-id=3207%3A132653

Acceptance Criteria:

Note that the current design is a rough placeholder using the existing spreadsheet, should be implemented using standard Chakra UI components, same as the assumptions and TAM table for clusters.

@eethann
Copy link
Collaborator Author

eethann commented Oct 25, 2021

I think the page needs some UI tweaks. Currently on a standard hi res laptop screen, only the graph is visible, with the other data hidden below the line.

I think the quickest fix might be to put the graph below the data charts.

Also, let's separate the assumptions from the TAM and put them in their own card, placed to the right of the TAM card. This way we'll have 3 cards each side by side, then the chart below them.

A few other notes:

  • The chart looks a bit off (only one kind of data displayed).
  • The formatting of the results is hard to read. Perhaps use empty rows for the LLDC, MDC, Total labels, so the chart can expand. Also the background should be lighter and should extend to the Solution / Conventional split.

If helpful, let's connect about that table and how to tweak the UI.

@shrutimehta13
Copy link

shrutimehta13 commented Nov 25, 2021

Results table in excel sheet is blank for many cluster. @colleenskemp Please confirm if values need to be confirmed?
verified for Space Heating.

notes-

  1. Heading & subheading of results table is missing
  2. Graph has been moved down below data charts as per above comment. see a typo error in conventional

Screenshot 2021-11-25 at 2 42 21 PM

Screenshot 2021-11-25 at 2 44 29 PM

@colleenskemp
Copy link
Collaborator

@sutjin - can you advise about the missing data in the excel for clusters such as electricity?

Please correct the spelling error beneath the chart.

@colleenskemp colleenskemp assigned sutjin and unassigned colleenskemp Dec 15, 2021
@sutjin
Copy link
Collaborator

sutjin commented Dec 17, 2021

@colleenskemp I verified the data with the sheet Mamta has sent on an email to us (CORE_PopulationChange_Oct13_PostIntegration) and the data is now updated

@sutjin sutjin assigned colleenskemp and shrutimehta13 and unassigned sutjin Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants