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.

Sign up for GitHub

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

Jump to bottom

Feat/add chart zoom to quartz energy #502

Open
wants to merge 92 commits into
base: india-mvp
Choose a base branch
from
Open

Feat/add chart zoom to quartz energy #502

wants to merge 92 commits into from

Conversation

rachel-labri-tipton
Copy link
Contributor

@rachel-labri-tipton rachel-labri-tipton commented Apr 16, 2024 •
edited
Loading

Pull Request

Description

This PR adds a zoom feature to the india-mvp app.
I've been able to select the data that needs to be zoomed in on, but I'm having trouble adjusting the x-axis and the data reloads every time the chart is clicked.

Screenshot 2024-04-16 at 11 37 44

How Has This Been Tested?

Tested by running the code locally.

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

braddf and others added 30 commits February 8, 2024 16:07
@braddf
new nextjs app initial commit
0acc778
@braddf
add logo; fix build type errors
fa1cc72
@braddf
add OCF greys from design files; fix config
7badd4c
@braddf
add initial apiClient, types, data fetching, chart
2a2ce9a
@braddf
update title
55dcdd0
@braddf
fix linting errors
c9ca9d9
@braddf
update api to proxied quartz route
1ffb239
@braddf
Switch wind/solar on graphs, solar on bottom
c3ad49c
@rachel-labri-tipton
update icons and add wide sidebar
d60c3b4
@rachel-labri-tipton
add narrow sidebar styling
4813c9f
@rachel-labri-tipton
adjust container width
8c4953f
@rachel-labri-tipton
adding sidebar components
6d01ff5
@rachel-labri-tipton
add mini-cards for collapsed sidebar
bb7a37e
@rachel-labri-tipton
move left arrow to the right of sidebar
28c59a8
@rachel-labri-tipton
fix text color
4ff90b4
@rachel-labri-tipton
sidebar with card and minicard components
5698d10
@rachel-labri-tipton
fix linting errors
6b38c6e
@rachel-labri-tipton
fix linting for time label:
f5c9bf1
@rachel-labri-tipton
fix type error in card component
23f10ae
@braddf
Amend corrected forecast to display MW
66549ae
@braddf
Round generation values down to minute
a5b20cf
@braddf
Flip and sort formatted Chart data
e10c81a
@peterdudfield
resample generation to 15 minutes
448dec1
@peterdudfield
run prettier
fd3a319
@peterdudfield
role back
af07e58
@peterdudfield
adjust schema
d8febf1
@peterdudfield
use query object
6217b0f
@peterdudfield
Merge branch 'issue/generation-15-mins' into india-mvp
58af4b5
@braddf
Add now reference line in Indian Standard Time
437ccd6
@braddf
Merge remote-tracking branch 'origin/india-mvp' into india-mvp
6decc48
braddf and others added 24 commits March 13, 2024 12:26
@braddf
Make combinedData global; add user menu
6ba2eda
@rachel-labri-tipton
small fix
d7c5dc7
@rachel-labri-tipton
Merge pull request #484 from openclimatefix/issue/483-verify-generati…
1b9bf15 
…on-numbers

update solar generation to reflect correct value
@braddf
Add downloadCsv function first draft
934c56d
@braddf
Show/Hide toggled forecasts in tooltip
6403256
@braddf
Sidebar layout/styling
ad8444d
@braddf
Fix gap in area chart
0adbdf1
@braddf
Fix CSV & amend format/filename
98aa6b0
@braddf
Fix overflow horizontal scroll bug
b35c002
@braddf
Add day name x axis
47b70bc
@braddf
Solid now line
bc17f74
@braddf
Sidebar: GW -> MW
b16815a
@braddf
mini sidebar number format
30291b5
@braddf
amend csv: KW -> MW
cd8c25f
@braddf
fix chart data bug(s) and now gap; add test
4737c56
@braddf
fix build error(s)
d162743
@braddf
fix nowcasting build error(s)
c5aca1b
@braddf
No data label in tooltip
ed11954
@braddf
Add refetch param
9fa5268
@peterdudfield @braddf
move wind below solar in sidebar
c8070e0
@braddf
Dynamic ordering in tooltip values
915e423
@braddf
Merge pull request #495 from openclimatefix/swap-solar-and-wind-sidebar
fb16b16 
Swap solar and wind sidebar
@rachel-labri-tipton
filter chart data
e0c3622
@rachel-labri-tipton
select data to zoom in on
e6d6c25
@vercel Vercel
Copy link

vercel bot commented Apr 16, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
quartz-app ✅ Ready (Inspect) Visit Preview 💬 1 unresolved Apr 19, 2024 0:37am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
nowcasting-app ⬜️ Ignored (Inspect) Visit Preview 💬 Add feedback Apr 19, 2024 0:37am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@braddf braddf Awaiting requested review from braddf

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rachel-labri-tipton @braddf @peterdudfield