NEW: Add micronet
#293
Merged
NEW: Add micronet
#293
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
- Fix HoVerNet plus documentation
- Add models/architecture/micronet
Codecov Report
@@ Coverage Diff @@
## develop #293 +/- ##
=========================================
Coverage 99.82% 99.82%
=========================================
Files 53 54 +1
Lines 5004 5146 +142
Branches 823 825 +2
=========================================
+ Hits 4995 5137 +142
Misses 2 2
Partials 7 7
Continue to review full report at Codecov.
|
- Fix deepsource errors
- Add docstring for weights initialization
- Update docstrings for various Micronet functions.
- Upload micronet consep model.
Update models/architecture/micronet.py to work with tiatoolbox
- Add tests for micronet.
- Fix spelling errors in tests/models/test_hovernet - Fix linter issues.
- Fix deepsource bugs
- Fix deepsource errors
- Update the forward function to accept args and kwargs
- Update abc to include input_images in modelabc
- Add map_location to tests
- Update forward input - Update MicroNet docstring.
- Update docs to include micronet
- update docstring for micronet
- Skip potential bug error
- update docstring
- Update docstring
- Add performance table to micronet
- Add tests to improve coverage.
- Update __init__ to include fetch_pretrained_weights
|
|
||
| """ | ||
|
|
||
| def group1_branch(layer, in_tensor, resized_feat): |
Contributor
There was a problem hiding this comment.
Would the below group functions be better defined in init under a slight different name such as group_link? Makes forward quite clunky otherwise.
- Rearrange group functions to clean the code.
vqdang
suggested changes
Feb 23, 2022
Contributor
There was a problem hiding this comment.
@shaneahmed Can you add a functional test for the precision of the output? You will use the conditional decorator to flag that test for local run only?
Otherwise, I am fine with this.
- Update MicroNet to add local test and post process nuclei
- Fix micronet test to take two outputs from postproc.
vqdang
approved these changes
Feb 24, 2022
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.
micronetarchitecture.