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.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
➕ Add FastFlow Model #336
➕ Add FastFlow Model #336
Changes from 24 commits
838494c
09f5d9e
89e599c
0723896
52fcb8b
80ffcde
ca34609
4546bdf
c3a21ce
545e29b
77eda6d
d51516f
e43d160
98b1af7
57df26b
1322150
65a2c43
1e891a0
1f9677b
dbd5f26
652c592
b684657
8ad1d27
c53adeb
5725366
4ad4886
fc0ffde
920bb4a
a30f4e6
729e043
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the average score? Also, a minor issue but this format is a bit different from our other models (rows <-> columns). Maybe for consistency we should decide on a single format. Maybe this can be addressed in a different PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some categories that I need to run again to exactly produce the numbers, which is the reason why I left it blank. As agreed, I could add these later.
Regarding the table format, I would prefer this because 15 MVTec categories doesn't fit to the screen when they are placed into columns.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could perhaps modify all the tables once we have all the benchmark results merged