Skip to content
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

rename all 'slice' functions/methods to 'patch/patchify' #84

Closed
wants to merge 4 commits into from

Conversation

rwood-97
Copy link
Collaborator

Summary

MapReaders is able to create 'patches' from maps/images but the functions/methods used to create these and were called '_slice', 'sliceAll' and 'sliceByPixel'. To streamline vocab, these are renamed to '_patchify', 'patchifyAll' and 'patchifyByPixel'.

Fixes #70

Describe your changes

'_slice', 'sliceAll' and 'sliceByPixel' are renamed to '_patchify', 'patchifyAll' and 'patchifyByPixel'.

  • arguments using the word slice are updated to 'patch/patches/patchify'
  • docstrings updated in all files changed

Checklist before assigning a reviewer (update as needed)

  • [ x] Self-review code
  • [ x] Ensure submission passes current tests
  • Add tests

Reviewer checklist

Please add anything you want reviewers to specifically focus/comment on.

  • Everything looks ok?

@rwood-97 rwood-97 self-assigned this Feb 20, 2023
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rwood-97 rwood-97 closed this Mar 6, 2023
@rwood-97
Copy link
Collaborator Author

rwood-97 commented Mar 6, 2023

changes merged into #82

@rwood-97 rwood-97 deleted the 70_slice2patch branch March 6, 2023 11:50
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.

Rename all 'slice' functions/methods to 'patchify' to align with documentation/streamline vocab
1 participant