-
Notifications
You must be signed in to change notification settings - Fork 43
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
EIA V2 documentation #277
base: main
Are you sure you want to change the base?
EIA V2 documentation #277
Conversation
i think showing how to use |
Hey Max! How is this looking for a first pass? |
hi - sorry, i havent had time to look at this. might not have time for a bit, but ill comment here once i do. really sorry |
one comment - i see the docs aren't actually building. i think you need to merge in main with this fix #285. i recommend building docs locally and confirm the interactivity is actually working. ![]() |
Ah sorry I didn't mean to close this. My local instance of grid status became a mess for some reason and I had corrupted files and ended up just deleting the whole thing and re-cloning. I have the EIA example file..just need to get this PR all sorted.. |
6th times the charm... |
the git diff is unreadable since it looks like you committed a bunch of your local dev env. i cant review this until that's cleaned up |
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.
Hey, @jleonard8812 we can try and get this merged if you're still up for it. You'll want to remove the .jupyter_cache/
, jupyter_execute/
, and gridenv
directories from the PR.
Is this PR closed by #327? |
No, that PR was just fixing existing documentation. It would be great if we could get some more new EIA documentation notebooks. |
Are there any particular methods you are hoping to be included in this? And how many examples would you like included?