Skip to content

Data Analysis of TZ-SAM Dataset#272

Merged
zakwatts merged 11 commits intoopenclimatefix:mainfrom
hrsvrn:main
Apr 2, 2025
Merged

Data Analysis of TZ-SAM Dataset#272
zakwatts merged 11 commits intoopenclimatefix:mainfrom
hrsvrn:main

Conversation

@hrsvrn
Copy link
Copy Markdown
Contributor

@hrsvrn hrsvrn commented Mar 22, 2025

Pull Request

Description

The Jupyter notebook in the quartz_solar_forecast/dataset/TZ-SAM presents a comprehensive data analysis of the TZ-SAM dataset, which contains information about solar facilities worldwide. The notebook is structured to explore key statistics, patterns, and relationships within the dataset.

The notebook begins by importing essential libraries for data analysis, including pandas for data manipulation, matplotlib for visualization, and cartopy for geographical plotting.

I have not included the csv at the moment because it is too large (95,000 records)

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have checked my code and corrected any misspellings

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 23, 2025

@peterdudfield @zakwatts i have done some data analysis on the tz sam dataset. Could you please review and give some feedback ?

@peterdudfield
Copy link
Copy Markdown
Contributor

@zakwatts are you ok to look at this?

@zakwatts
Copy link
Copy Markdown
Contributor

Hi @hrsvrn, would you be able to repush your notebook but with the output cells cleared ts-sam-analysis.ipynb

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 24, 2025

yes sure! @zakwatts give me a few mins

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 24, 2025

@zakwatts i have cleared the output cells.. I noticed that i have mistakenly got the name of the jupyter notebook to be wrong. Can i rename it after you have reviewed the changes?

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 25, 2025

any updates on this @zakwatts ?

@zakwatts
Copy link
Copy Markdown
Contributor

Hi @hrsvrn, please go ahead and rename the file. Then I'll run it locally and be able to give it a review.

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 25, 2025

@zakwatts I have renamed the file... requesting you ti kindly check and give valuable feedback

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 26, 2025

@zakwatts i have also added a script that downloads the dataset in the rightplace reccomended by @peterdudfield
Can you review that also please?

@zakwatts zakwatts self-requested a review March 26, 2025 11:37
Copy link
Copy Markdown
Contributor

@zakwatts zakwatts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its looking really good. Thanks for doing this analysis. Could you move the current tz-sam-analysis.ipynb notebook into the current data_analysis folder located here /Users/zakwatts/Coding/OCF/Open-Source-Quartz-Solar-Forecast/quartz_solar_forecast/dataset/dataset_analysis. And update corresponding paths in the notebook if needed.

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 30, 2025

@peterdudfield any updates on this one?

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Mar 30, 2025

@peterdudfield if i am pushing my notebook should i do with the output as well?

@hrsvrn
Copy link
Copy Markdown
Contributor Author

hrsvrn commented Apr 1, 2025

@zakwatts @peterdudfield any updates on this one?

@zakwatts
Copy link
Copy Markdown
Contributor

zakwatts commented Apr 2, 2025

It looks great! I've ran it locally again and the analysis is good. I like the map of the locations and the look it capacity distributions. Thanks!

@zakwatts zakwatts merged commit 479f602 into openclimatefix:main Apr 2, 2025
4 checks passed
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.

3 participants