-
Notifications
You must be signed in to change notification settings - Fork 8
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 Updates #36
base: main
Are you sure you want to change the base?
Conversation
annacasavant
commented
Feb 6, 2025
- Updated to Documenter 1.0
- Re-structured to follow Diataxias guidelines
@annacasavant you probably want to rebase this on main |
…erAnalytics.jl into ac_docs_updates
…erAnalytics.jl into ac_docs_updates
I'll review again after documentation is passing and then this should be good to merge. |
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.
Few minor changes then we're good to go
@@ -11,7 +11,7 @@ values are loaded results datasets. | |||
- `scenarios::Union{Vector{AbstractString}, Nothing} = nothing`: a list of scenario | |||
subdirectories to load, or `nothing` to load all the subdirectories | |||
- `kwargs...`: keyword arguments to pass through to | |||
[`PSI.get_decision_problem_results`](@ref) | |||
`get_decision_problem_results` |
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.
It looks like the decision is to not try to make the PSI interlink work for now? That's fine, but if we hope to do it in the future can you add an issue for it? In general I expect the ability to nicely link to PSI reference will be useful more broadly than just this occurrence
Co-authored-by: Gabriel Konar-Steenberg <[email protected]>
…erAnalytics.jl into ac_docs_updates