DOC: Update two reference links in X-ray image processing #289
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Seems fine to me. It might be nice to link to the OpenCV edge detectors canny and sobel/structured forests. I am a little surprised that this cell is rendered as code not text in the binder. |
rossbar
reviewed
Nov 24, 2025
Co-authored-by: Ross Barnowski <rossbar@caltech.edu>
Collaborator
Yeah the cloud (or better yet, client-side) rendering of the notebooks still needs work. On binder, if you right-click and select "Open with... Jupytext Notebook" then it should render correctly. This info (and related "howtos" for cloud) need to be added to the contributor guide! |
rossbar
approved these changes
Nov 24, 2025
Collaborator
rossbar
left a comment
There was a problem hiding this comment.
Thanks @star1327p and @mattip !
github-actions bot
pushed a commit
that referenced
this pull request
Nov 24, 2025
* DOC: Update two reference links in X-ray image processing Co-authored-by: Ross Barnowski <rossbar@caltech.edu> --------- Co-authored-by: Ross Barnowski <rossbar@caltech.edu> 128e602
github-actions bot
pushed a commit
to rossbar/numpy-tutorials
that referenced
this pull request
Nov 24, 2025
* DOC: Update two reference links in X-ray image processing Co-authored-by: Ross Barnowski <rossbar@caltech.edu> --------- Co-authored-by: Ross Barnowski <rossbar@caltech.edu> 128e602
github-actions bot
pushed a commit
to dumpmemory/numpy-tutorials
that referenced
this pull request
Nov 25, 2025
* DOC: Update two reference links in X-ray image processing Co-authored-by: Ross Barnowski <rossbar@caltech.edu> --------- Co-authored-by: Ross Barnowski <rossbar@caltech.edu> 128e602
Member
|
Binder works without issue for the other repos, I'll have a look at this tomorrow |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Edge detection in Python and NumPy (Towards Data Science)
Original link does not work: https://towardsdatascience.com/edge-detection-in-python-a3c263a13e03
Available link: https://medium.com/data-science/edge-detection-in-python-a3c263a13e03
The Medium link is not a public article, but it is the closest one I can find.
Edge detection with Scikit-Image (Data Carpentry)
Original link does not work: https://datacarpentry.org/image-processing/08-edge-detection/
Correct link: https://datacarpentry.github.io/image-processing/edge-detection.html