-
Notifications
You must be signed in to change notification settings - Fork 7
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
Documentation (Geospatial, Meteorological Data, Materials Access) #136
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #136 +/- ##
===============================================
- Coverage 65.65% 65.42% -0.23%
===============================================
Files 36 36
Lines 3549 3575 +26
===============================================
+ Hits 2330 2339 +9
- Misses 1219 1236 +17 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobin-ford - This is looking great! There are a few figures that haven't been created by us. Do we cite them properly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobin-ford - Did we cite where this figure is from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are linked directly to the webpage from NREL. Let's wait to merge this until we get the new PVDeg logo today. Then we can add the logo back to the docs.
Describe your changes
Significant
User Guide
improvements. Added geospatial information with visual aids, added meteorological data page and materials access page.Issue ticket number and link
Fixes #128
Type of change
Please delete options that are not relevant.
Checklist before requesting a review