Skip to content
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

ewm7891: Improve mantid-related Documentation Links #7

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

ktactac-ornl
Copy link

@ktactac-ornl ktactac-ornl commented Nov 11, 2024

Description of work

  • Improves dev-docs/source/AlgorithmProfiler.rst documentation by connecting relevant documentation for mantid and mantid-profiler
  • Updates the tool example image in dev-docs/source/images/MantidProfiler.png

Summary of work

This PR adds links to material that the reader is likely interested in, in several locations.

  • mantid.api.AlgoTimeRegister documentation linked to reference how timing is measured

  • mantid-profiler README is linked to point to the profiler's primary documentation

  • PropertiesFile#algorithm-profiling-settings is linked to point the reader to important user properties

  • The tool example image is updated as features have been added to the dashboard.

Relevant documents were previously disconnected and difficult to find.

Fixes #xxxx.

EWM 7891: Link mantid-profiler documentation

Further detail of work

To test:

Related documentation builds and displays correctly.

@ktactac-ornl ktactac-ornl marked this pull request as ready for review November 12, 2024 16:10
Copy link
Member

@peterfpeterson peterfpeterson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the URLs to the rendered pages rather than the raw .rst files

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@ktactac-ornl ktactac-ornl changed the title Add links ewm7891: Improve mantid-related Documentation Links Nov 12, 2024
@ktactac-ornl ktactac-ornl added the documentation Improvements or additions to documentation label Nov 12, 2024
ktactac-ornl and others added 3 commits November 12, 2024 13:11
Co-authored-by: Pete Peterson <[email protected]>
Co-authored-by: Pete Peterson <[email protected]>
Co-authored-by: Pete Peterson <[email protected]>
@peterfpeterson peterfpeterson merged commit b1aec2b into main Nov 12, 2024
2 checks passed
@peterfpeterson peterfpeterson deleted the ewm7891-linkDocs branch November 12, 2024 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants