Skip to content

Add documentation for applying a transform after finding it#81

Draft
jamesyan-git wants to merge 5 commits intojni:mainfrom
jamesyan-git:fix-transform
Draft

Add documentation for applying a transform after finding it#81
jamesyan-git wants to merge 5 commits intojni:mainfrom
jamesyan-git:fix-transform

Conversation

@jamesyan-git
Copy link
Copy Markdown
Contributor

@jamesyan-git jamesyan-git commented Nov 1, 2023

Widget and documentation for correctly handling matrices based on transformation method.
Addresses #61

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 1, 2023

Codecov Report

Merging #81 (860fb02) into main (af68ac6) will increase coverage by 3.46%.
Report is 4 commits behind head on main.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   85.96%   89.42%   +3.46%     
==========================================
  Files           4        6       +2     
  Lines         171      227      +56     
==========================================
+ Hits          147      203      +56     
  Misses         24       24              

see 4 files with indirect coverage changes

@jamesyan-git
Copy link
Copy Markdown
Contributor Author

jamesyan-git commented Nov 1, 2023

Hi @jni, documentation has been updated to be more narrative. Also I'm not super clear on WHY some transformation methods go from reference to moving, so any additional explanation here would be appreciated.

@jni jni changed the title Transform Widget Add documentation for applying a transform after finding it Nov 1, 2023
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.

2 participants