Closed NoraLoose closed 1 year ago
Resolving this issue will also solve issue #8.
Question for @jerrylin96 and others: Are the demo notebooks meant to be included on the documentation website?
The table of contents suggests so:
Note that the 3 demo notebooks do not show up because they live outside of the website
directory. I can make them show up, if that is the intention.
If the demo notebooks are meant to appear on the documentation website, maybe they should be cleaned up a little bit?
I'll defer to @jerrylin96 of course, but to me it seems like including the demo notebooks on the website is a nice value-add for the website. As discussed in the meeting, otherwise the website is "just" the README. So I'd be š on including them.
Apologies for the delayed response. Iām on board with this.
On Thu, Aug 17, 2023 at 21:36 Charles Stern @.***> wrote:
I'll defer to @jerrylin96 https://github.com/jerrylin96 of course, but to me it seems like including the demo notebooks on the website is a nice value-add for the website. As discussed in the meeting, otherwise the website is "just" the README. So I'd be š on including them.
ā Reply to this email directly, view it on GitHub https://github.com/leap-stc/ClimSim/issues/15#issuecomment-1683342178, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFNO6XHMYNBKBN5A3TMJ4M3XV3WOXANCNFSM6AAAAAA3UUA74Q . You are receiving this because you were mentioned.Message ID: @.***>
In response to the NeurIPS reviewers, we will improve the website as follows:
I should be able to get this done by tomorrow.