Skip to content

Conversation

@wsmoses
Copy link
Member

@wsmoses wsmoses commented Apr 17, 2022

@wsmoses wsmoses requested review from tgymnich and vchuravy April 17, 2022 05:53
Copy link
Member

@vchuravy vchuravy left a comment

Choose a reason for hiding this comment

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

Modulo removal of debug output and test passing

@wsmoses wsmoses force-pushed the vwidth branch 4 times, most recently from c013641 to b21609d Compare April 19, 2022 05:10
@vchuravy vchuravy added this to the release-0.10 milestone Apr 19, 2022
@wsmoses wsmoses force-pushed the vwidth branch 3 times, most recently from 2e611a9 to 6af233c Compare April 25, 2022 03:57
@wsmoses wsmoses force-pushed the vwidth branch 2 times, most recently from a7c144a to d1020a9 Compare May 1, 2022 18:53
Copy link
Member

@vchuravy vchuravy left a comment

Choose a reason for hiding this comment

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

Needs docs for new API functions e.g. gradient and jacobian.

Also can we split out the bug fixes into separate PRs? Or at least one PR with the bug fixes that is backportable to the 0.9 release?

Copy link
Member

@vchuravy vchuravy left a comment

Choose a reason for hiding this comment

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

Will need to add a documenter preview

@wsmoses wsmoses merged commit d0e66f9 into main May 3, 2022
@wsmoses wsmoses deleted the vwidth branch May 3, 2022 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants