-
Notifications
You must be signed in to change notification settings - Fork 46
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
PROOF Documentation Update #1078
Conversation
Updating the structure of this How-To-Doc based on discussions at Lakeside Chat
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.
Address style issue re: author list prior to approval.
updated text for server, validate, and submit sections
Co-authored-by: Taylor Firman <[email protected]>
Proof update srm
new workflow image
Job_failures
Job Failures screenshot
Update by Sitapriya Moorthi for the following sections: Background (removed) Using PROOF/PROOF overview Log into PROOF Workflow Details Quick Links (Resources and Help)
…-v1.2 Update proof-how-to.md
Alright PROOF team, I think these documentation updates are ready. Thank you all so much for your efforts throughout this! Can someone take a final look and approve so we can merge to main? One note: @sitapriyamoorthi, the tables you added explaining the columns for each tab look super helpful and I want to incorporate them somewhere, but they were making the article a bit lengthy/dense, especially for an intro/how-to article. I commented them out for now, but I'm wondering if we can split them off into a "PROOF Glossary" type SciWiki article. Does that sound okay to you? |
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.
Minor fixes recommeneded
@tefirman yes thats fine! I just put it in there because there are a lot of terms that folks may not be familiar with. It could potentially be helpful or a quick reminder of what those different fields mean |
@sitapriyamoorthi @vortexing -- thanks for the reviews! Addressed all comments, mind taking another look? |
Holding off on these updates for now while we address issues with the v1.2 release. May reopen later, but closing for now to avoid confusion. |
Description
Testing