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

IOS-net: Add 2 stations, group stations in file, standardize #277

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

bilbaoba
Copy link
Contributor

Hi Adam,

We've recently added 2 new stations on the French island of Mayotte.

This PR includes:

  • Addition of the 2 new stations
  • Grouping of all IOS-net stations in the solarstations.csv file
  • Standardization of information across stations (URL, comments with DOI)
  • Correction in the Instrumentation column (UV data not available for all stations)

I hope this PR meets your requirements; please feel free to let me know if any adjustments are needed.

Best regards, and thank you for your work on solarstations.

@bilbaoba
Copy link
Contributor Author

I read your discussion in PR #246 about how Reunion should be listed.

Mayotte and Reunion are both French overseas departments. Therefore, you could consider setting the country as France for accuracy. The only complication is that their continent is Africa.

I'll let you decide what you prefer

@AdamRJensen
Copy link
Member

@bilbaoba What are the names of the two new stations? Is it Dembeni and Ouangani (it's a bit hard to detect given the large change in the diff)?

A huge thanks for these contributions!

@bilbaoba
Copy link
Contributor Author

bilbaoba commented Nov 14, 2024

Yes, in Dembeni and Ouanagni. We installed it this past weekend. It was one of the last populated areas where we were not yet present in the Southwest Indian Ocean (Mayotte).

Data are already available on our THREDDS server and tools: https://galilee.univ-reunion.fr/thredds/catalog/dataStations/Mayotte/catalog.html

DOIs will be generated later when the dataset becomes more extensive.

Oh also, the stations are planned to last several years !

@AdamRJensen AdamRJensen merged commit 4bc8970 into AssessingSolar:main Nov 14, 2024
1 of 2 checks passed
@AdamRJensen
Copy link
Member

@bilbaoba thank you very much for this PR!

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.

2 participants