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

Tidy up readme #4

Closed
Tracked by #1
peterdudfield opened this issue Nov 25, 2023 · 3 comments · Fixed by #8
Closed
Tracked by #1

Tidy up readme #4

peterdudfield opened this issue Nov 25, 2023 · 3 comments · Fixed by #8
Assignees
@chloepilonv

Comments

@peterdudfield
Copy link
Contributor

peterdudfield commented Nov 25, 2023 •
edited

  • Add what NWP variables are used. Would be good to give the units of these too
@peterdudfield peterdudfield mentioned this issue Nov 25, 2023
Open
21 tasks
@chloepilonv
Copy link
Contributor

Hey! Could I be assign to this if that’s ok with you? I’m getting familiar with different NWP model and I think that would be a good exercise. Thanks :)

@peterdudfield
Copy link
Contributor Author

Yes please @chloepilonv , thank you for getting in touch

@peterdudfield
Copy link
Contributor Author

Would be great to fix the example.

The ts=datetime... needs to be removed

Screenshot 2023-11-28 at 10 12 34

@peterdudfield peterdudfield mentioned this issue Dec 4, 2023
Merged
peterdudfield added a commit that referenced this issue Dec 4, 2023
@peterdudfield
Merge pull request #8 from chloepilonv/main
e8a23b9 
Tidy up README.md - Fixes #4
@peterdudfield peterdudfield mentioned this issue Jan 22, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chloepilonv chloepilonv

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Tidy up README.md - Fixes #4 chloepilonv/Open-Quartz-Solar-Forecast
2 participants
@peterdudfield @chloepilonv