UCL / TLOmodel-profiling

Profiling outputs for developer use
0 stars 0 forks source link

Change landing page to be `profiling-results` #16

Closed willGraham01 closed 4 months ago

willGraham01 commented 4 months ago

For https://github.com/UCL/TLOmodel/issues/1355

Note that the current index.rst contents have been moved to contents.rst so that the sphinx-build still works. This change just means that the index.html page immediately sends a person to the profiling results.

Flag any other aesthetic changes that we need to make here in PR discussion.

tamuri commented 4 months ago

Yes, the redirect feels clunky.

willGraham01 commented 4 months ago

My reasoning for the redirect was that it's the least noisy solution for what we initially wanted to achieve (since it saved going through the rst files and updating links), but yeah now I think about it I could've just moved the profiling results to that page :sweat_smile:

Have also changed the profiling.rst (and run-statistics.rst) files to have the lookup table / plots .. included:ed from files with underscores like in the main TLO repo, rather than using the placeholders.