Skip to content

Conversation

ChristopherKotthoff
Copy link
Collaborator

@ChristopherKotthoff ChristopherKotthoff commented Jan 28, 2025

This is a stacked pull request on Overlapping Tiles - More Training Data.

This pull request includes the following updates to improve the ResNet-stem functionality and maintain code quality:

Weight Initialization Fix

Issue Addressed: The weight initialization of the ResNet-stem was not functioning correctly.
Changes Made:
Modified the resume_or_load method to correctly handle weight loading.
When loading RGB model weights for multispectral models, the weights are now repeated appropriately to ensure proper initialization. They are also divided by the times they are repeated to allow for similar output values.

The tutorial file docs/source/tutorial.rst has also been change to reflect the new simplified usage.

@ChristopherKotthoff ChristopherKotthoff marked this pull request as ready for review January 28, 2025 15:36
@ChristopherKotthoff ChristopherKotthoff force-pushed the chris/ms-weights-initialization branch from 8440321 to 206dbf9 Compare January 28, 2025 17:25
@ChristopherKotthoff ChristopherKotthoff marked this pull request as draft January 28, 2025 18:29
@ChristopherKotthoff ChristopherKotthoff force-pushed the chris/ms-weights-initialization branch 2 times, most recently from 29b904c to 5b1497f Compare January 29, 2025 12:37
@ChristopherKotthoff ChristopherKotthoff force-pushed the chris/ms-weights-initialization branch 4 times, most recently from 005ab85 to 5a17563 Compare February 27, 2025 21:03
@ChristopherKotthoff ChristopherKotthoff force-pushed the chris/ms-weights-initialization branch from 5a17563 to a6bfa2d Compare February 27, 2025 21:16
@ChristopherKotthoff ChristopherKotthoff marked this pull request as ready for review February 27, 2025 21:25
@ChristopherKotthoff ChristopherKotthoff merged commit fd150b7 into PatBall1:master Feb 27, 2025
2 checks passed
@ChristopherKotthoff ChristopherKotthoff deleted the chris/ms-weights-initialization branch February 27, 2025 21:26
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.

1 participant