Skip to content

96 cswrapp 85422 torch non parametric#97

Merged
danibcorr merged 12 commits intomainfrom
96-cswrapp-85422-torch-non-parametric
Mar 16, 2026
Merged

96 cswrapp 85422 torch non parametric#97
danibcorr merged 12 commits intomainfrom
96-cswrapp-85422-torch-non-parametric

Conversation

@lucia-ferrer
Copy link
Copy Markdown
Contributor

No description provided.

@lucia-ferrer lucia-ferrer requested a review from danibcorr March 6, 2026 08:58
@lucia-ferrer lucia-ferrer linked an issue Mar 6, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to adjust the image resolution so that it is at the edge of the content?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to adjust the image resolution so that it is at the edge of the content?

@lucia-ferrer lucia-ferrer requested a review from danibcorr March 9, 2026 11:10
illia/layers.py Outdated


# Non-parametric layers by category
_POOLING_LAYERS: Final[frozenset[str]] = frozenset(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is something that maybe we can discuss, but we could use Final[tuple[str, ...] as typing instead of frozenset. Also, maybe we need to find a way of creating this layers configuration dynamically and not hardcoded.

@danibcorr danibcorr merged commit 47a4c21 into main Mar 16, 2026
12 checks passed
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.

CSWRAPP-85422 [Torch]

2 participants