-
Notifications
You must be signed in to change notification settings - Fork 10
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
FY24 documentation improvements (part 1) #142
Conversation
Python Binding Test Results (ubuntu-latest)63 tests ±0 39 ✅ ±0 2s ⏱️ ±0s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 63 and adds 63 tests. Note that renamed tests count towards both.
This pull request removes 24 skipped tests and adds 24 skipped tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Test Results (macos-12-Debug-ON) 3 files ±0 3 suites ±0 1m 33s ⏱️ +5s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Python Binding Test Results (macos-12)63 tests ±0 39 ✅ ±0 2s ⏱️ -48s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 63 and adds 63 tests. Note that renamed tests count towards both.
This pull request removes 24 skipped tests and adds 24 skipped tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Test Results (ubuntu-latest-Debug-ON) 3 files ±0 3 suites ±0 1m 23s ⏱️ -13s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (ubuntu-latest-Release-ON) 3 files ±0 3 suites ±0 34s ⏱️ -1s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (macos-12-Release-ON) 3 files ±0 3 suites ±0 50s ⏱️ -37s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (macos-12-Debug-OFF) 3 files ±0 3 suites ±0 1m 27s ⏱️ - 1m 4s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (ubuntu-latest-Release-OFF) 3 files ±0 3 suites ±0 35s ⏱️ -1s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (ubuntu-latest-Debug-OFF) 3 files ±0 3 suites ±0 1m 22s ⏱️ -14s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Test Results (macos-12-Release-OFF) 3 files ±0 3 suites ±0 49s ⏱️ +3s Results for commit 27dd988. ± Comparison against base commit 2c12c8c. This pull request removes 1 test.
♻️ This comment has been updated with latest results. |
Note that this PR gets rid of all submodules. Now this relies on CMake's FetchContent to pull dependencies on-demand. I am now a huge fan of CMake's FetchContent idea, and I think its development has become mature especially now that we require a recent CMake. |
|
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.
For you
- Add some additional documentation on the use of the python bindings.
- Make some moderate effort to expand the object docstring and input documentation. Doesn't have to be fully comprehensive, but if we don't work on that now it's never going to happen.
For me
- Crystal plasticity docs.
After this
- Issue a new major release (b/c of python changes)
- Update copyright year when we do that.
Missing syntaxNothing, good job! 💜 |
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.
LGTM. Unfortunately, docstring of individual options cannot render latex. I'll found two occurrences which I'll fix by either removing the formula or relocate them into the object summary doc.
src/neml2/models/crystallography/user_tensors/FillMillerIndex.cxx
Outdated
Show resolved
Hide resolved
The option docstring formula rendering issue is documented in issue #146. I'll merge when tests pass. |
Coverage after merging doc into main will be
Coverage Report
|
- Remove the page tag in README.md - Detect formula in option description ref #146
Annual documentation season...
This is the first PR to document changes made in FY24
A follow on PR will add the user guide.