Skip to content

MCT FX Tracer#1385

Merged
yarden-yagil-sony merged 24 commits intomainfrom
featue/pre_mct_custom_layer_steaching
Apr 9, 2025
Merged

MCT FX Tracer#1385
yarden-yagil-sony merged 24 commits intomainfrom
featue/pre_mct_custom_layer_steaching

Conversation

@yarden-yagil-sony
Copy link
Copy Markdown
Contributor

@yarden-yagil-sony yarden-yagil-sony commented Mar 13, 2025 •
edited
Loading

Pull Request Description:

Support models with custom layers

Checklist before requesting a review:

  • I set the appropriate labels on the pull request.
  • I have added/updated the release note draft (if necessary).
  • I have updated the documentation to reflect my changes (if necessary).
  • All function and files are well documented.
  • All function and classes have type hints.
  • There is a licenses in all file.
  • The function and variable names are informative.
  • I have checked for code duplications.
  • I have added new unittest (if necessary).

haihabi
haihabi reviewed Mar 27, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@haihabi haihabi left a comment

Choose a reason for hiding this comment

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

Few comment and questions

ofirgo
ofirgo suggested changes Mar 27, 2025
View reviewed changes
ofirgo
ofirgo reviewed Mar 31, 2025
View reviewed changes
ofirgo
ofirgo suggested changes Mar 31, 2025
View reviewed changes
yarden-sony added 4 commits April 3, 2025 14:21
remove box decode
2b758a9
Merge branch 'refs/heads/main' into featue/pre_mct_custom_layer_steac…
dd00aed 
…hing

# Conflicts:
#	model_compression_toolkit/target_platform_capabilities/targetplatform2framework/attach2pytorch.py
#	requirements.txt
empty commit
78b0214
change dev custom layers requirement to ==
fec4e57
@yarden-yagil-sony yarden-yagil-sony merged commit 23af7f0 into main Apr 9, 2025
36 checks passed
@ofirgo ofirgo deleted the featue/pre_mct_custom_layer_steaching branch July 2, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@haihabi haihabi haihabi left review comments

@Idan-BenAmi Idan-BenAmi Idan-BenAmi left review comments

@ofirgo ofirgo ofirgo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

auto:core

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yarden-yagil-sony @haihabi @ofirgo @Idan-BenAmi