-
Notifications
You must be signed in to change notification settings - Fork 40
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
Debug arg parsing error while launching the app & Contribute Initial Documentation #307
Conversation
Thanks @sora-ryu! When you are ready to merge this, can you assign a reviewer? |
Documentation is finally ready to review! @dzalkind and @mayankchetan , please let me know if any further improvements would be needed. |
How does this documentation work with the WEIS readthedocs that already exists? |
I thought we are having separate documentation for Viz tool. Do you want me to include it into existing WEIS docs? |
Added this WEIS Viz documentation to the existing WEIS documentation, not keeping separate. |
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.
Looks good to me. Thank you for moving it to the main docs
Purpose
Type of change
What types of change is it?
Select the appropriate type(s) that describe this PR
Testing
Explain the steps needed to test the new code to verify that it does indeed address the issue and produce the expected behavior.
Checklist
Put an
x
in the boxes that apply.