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

Adding Requirements and QA to Docs #2076

Open
wants to merge 53 commits into
base: main
Choose a base branch
from
Open

Adding Requirements and QA to Docs #2076

wants to merge 53 commits into from

Conversation

john-science
Copy link
Member

@john-science john-science commented Feb 5, 2025

What is the change?

Here I add a "QA Documentation" section to the ARMI docs. This section will have:

  • Open-Sourced ARMI Requirements
  • Software Requirements Specification Document (SRSD)
  • Software Design and Implementation Document (SDID)
  • Software Test Report (STR)

Why is the change being made?

close #2064

This is part of on-going work to improve the documentation of ARMI's high-level design requirements, and to open-source that information.


Checklist

  • The release notes have been updated if necessary.
  • The documentation is still up-to-date in the doc folder.
  • The dependencies are still up-to-date in pyproject.toml.

@john-science john-science added the documentation Improvements or additions to documentation label Feb 5, 2025
@john-science
Copy link
Member Author

Okay, this appears to work fine. Here are some examples. First, the TOC at the top now includes this:

image

The sphinx-needs tooling is working:

image

image

The SDID looks fine:

image

The STR matrix is working:

image

image

@john-science
Copy link
Member Author

Okay, this is working great.

But I am now getting a TON of Sphinx warnings:

WARNING: Ignored `filter:progid:DXImageTransform.Microsoft.gradient(GradientType=0,StartColorStr="#f9f9f9", EndColorStr="#e0e0e0")` at 16:4443, unknown property.
WARNING: Ignored `text-shadow:0 1px 0 #c4def1` at 16:4687, unknown property.
WARNING: Ignored `background-image:-webkit-linear-gradient(top, #bddef4 0%, #79ace9 100%)` at 16:4753, invalid value.
WARNING: Ignored `background-image:-o-linear-gradient(top, #f0f0f0 0%, #dadada 100%)` at 16:6528, invalid value.
WARNING: Ignored `filter:progid:DXImageTransform.Microsoft.gradient(GradientType=0,StartColorStr="#f0f0f0", 
WARNING: Ignored `box-shadow:0 0 3px #444` at 28:2080, unknown property.
WARNING: Anchor defined twice: 'setting-up-the-blueprints'
WARNING: Anchor defined twice: 'nuclide-flags'
WARNING: Anchor defined twice: 'defining-fuel-management'
WARNING: Anchor defined twice: 'id13'
WARNING: Anchor defined twice: 'new-features'
WARNING: Anchor defined twice: 'api-changes'
WARNING: Anchor defined twice: 'bug-fixes'
WARNING: Anchor defined twice: 'quality-work'
WARNING: Anchor defined twice: 'bug-fixes'
WARNING: Anchor defined twice: 'quality-work'
WARNING: Anchor defined twice: 'subpackages'
WARNING: Anchor defined twice: 'submodules'
WARNING: Anchor defined twice: 'T_ARMI_MAT_PROPERTIES1'
WARNING: Anchor defined twice: 'T_ARMI_MAT_FLUID0'
WARNING: Anchor defined twice: 'submodules'
WARNING: Anchor defined twice: 'id1'
WARNING: Anchor defined twice: 'id2'
WARNING: Anchor defined twice: 'I_ARMI_MAT_FLUID'

The docs look fine though.

@john-science john-science requested a review from ntouran February 7, 2025 18:39
@john-science john-science marked this pull request as ready for review February 27, 2025 18:41
@opotowsky
Copy link
Member

Okay, this is working great.

But I am now getting a TON of Sphinx warnings:

WARNING: Ignored `filter:progid:DXImageTransform.Microsoft.gradient(GradientType=0,StartColorStr="#f9f9f9", EndColorStr="#e0e0e0")` at 16:4443, unknown property.
WARNING: Ignored `text-shadow:0 1px 0 #c4def1` at 16:4687, unknown property.
WARNING: Ignored `background-image:-webkit-linear-gradient(top, #bddef4 0%, #79ace9 100%)` at 16:4753, invalid value.
WARNING: Ignored `background-image:-o-linear-gradient(top, #f0f0f0 0%, #dadada 100%)` at 16:6528, invalid value.
WARNING: Ignored `filter:progid:DXImageTransform.Microsoft.gradient(GradientType=0,StartColorStr="#f0f0f0", 
WARNING: Ignored `box-shadow:0 0 3px #444` at 28:2080, unknown property.
WARNING: Anchor defined twice: 'setting-up-the-blueprints'
WARNING: Anchor defined twice: 'nuclide-flags'
WARNING: Anchor defined twice: 'defining-fuel-management'
WARNING: Anchor defined twice: 'id13'
WARNING: Anchor defined twice: 'new-features'
WARNING: Anchor defined twice: 'api-changes'
WARNING: Anchor defined twice: 'bug-fixes'
WARNING: Anchor defined twice: 'quality-work'
WARNING: Anchor defined twice: 'bug-fixes'
WARNING: Anchor defined twice: 'quality-work'
WARNING: Anchor defined twice: 'subpackages'
WARNING: Anchor defined twice: 'submodules'
WARNING: Anchor defined twice: 'T_ARMI_MAT_PROPERTIES1'
WARNING: Anchor defined twice: 'T_ARMI_MAT_FLUID0'
WARNING: Anchor defined twice: 'submodules'
WARNING: Anchor defined twice: 'id1'
WARNING: Anchor defined twice: 'id2'
WARNING: Anchor defined twice: 'I_ARMI_MAT_FLUID'

The docs look fine though.

This is probably because you have a file getting processed by sphinx 2x somewhere via an include. If you add that file to exclude_patterns in your conf.py you should be able to get rid of these. I think. I've spent 0 time confirming this.

Copy link
Member

@opotowsky opotowsky left a comment

Choose a reason for hiding this comment

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

I'll review this at the high level (not commenting on content since that is being reviewed by @ntouran)

There are minor formatting issues with the PDF (some tables and text includes run off the page, and some math isn't rendering in the PDF that renders fine in HTML. I sent screencaps to you offline.

In the QA section (HTML), it appears I can click from an impl or test back to a requirement, but I can't click the impl/test link from the requirement (I get a "page does not exist" error).

Looks good though! Let me know if some of these things I commented on are out of scope.

@john-science
Copy link
Member Author

@opotowsky I cannot seem to change all these warnings from simplepdf. So I opened a ticket; useblocks/sphinx-simplepdf#109

Hopefully, this is an easy fix.

@john-science john-science removed the request for review from ntouran March 5, 2025 02:21
Copy link
Member

@opotowsky opotowsky left a comment

Choose a reason for hiding this comment

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

ok after all the troubleshooting, I re-approve!

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.

Open-Source ARMI's Requirements
2 participants