Lab 4 - #4
Open
tutoringjedi wants to merge 4 commits into
Open
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Completing Lab 4 (Convolutions): exploring how Conv2D kernel size and padding affect output shape, implementing an edge-detection kernel (Laplacian) on a grayscale image, applying max/average pooling, and classifying images (a sample cat photo and a webcam snapshot) using a pre-trained ResNet50 model.
What did you learn from the changes you have made?
padding="valid"shrinks the output's spatial dimensions bykernel_size - 1in each direction, whilepadding="same"zero-pads the input so the output stays the same size regardless of kernel size. A larger kernel also means each output pixel mixes a wider neighborhood of input pixels, giving a more smoothed-out result. A Laplacian kernel highlights edges in every direction at once (unlike Sobel, which needs separate horizontal/vertical kernels), by responding strongly to local intensity changes and staying near zero over flat, uniform regions.Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Considered using a Sobel kernel pair (separate horizontal and vertical kernels, combined via magnitude) for edge detection instead of a single Laplacian kernel, which would give directional edge information rather than an omnidirectional response.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Hit a
ModuleNotFoundErrorforcv2when trying to use the webcam capture section; resolved by installingopencv-python(the pip package name doesn't match thecv2import name, which tripped me up initially).How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist