-
Notifications
You must be signed in to change notification settings - Fork 19
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
Updated and corrected the wf3cte.rst and wf3cte.py files #69
Conversation
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.
Nice edits. Comments mostly include grammar and hyperlinks.
Removed background documentation on the CTE algorithm which does not belong in this source file. Replaced with URLs for handy reference, particularly so the documentation is not duplicated in multiple locations, and the latest information is available for viewing. Removed support for incorrect items from the source: ability to set the output filename and ability for the code to accept an association table as input. Removal of the output filename modifies the API.
to the documentatio to clarify understanding and correct typos. Question with regard to removal of parameters from the Python interface where addressed. The parameters were effectively no-ops.
9a3a5a9
to
6101480
Compare
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.
All comments in review were addressed. Just one hyperlink issue and checking on a previous comment.
@cshanahan1 @FDauphin I am starting to prep for a new build release candidate which can include as many WFC3 changes that can fit. This PR needs a re-review. Thanks. |
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.
All of my comments have been addressed. Approving for merge; thanks again 👍🏾
Significant updates to the wf3cte.rst file.
Modified the wf3cte.py file as follows:
Removal of the output filename modifies the API.