Skip to content

EfficientNet mypy fix#2126

Merged
wyli merged 2 commits intoProject-MONAI:devfrom
masadcv:efficientnet-mypy-fix
May 1, 2021
Merged

EfficientNet mypy fix#2126
wyli merged 2 commits intoProject-MONAI:devfrom
masadcv:efficientnet-mypy-fix

Conversation

@masadcv
Copy link
Copy Markdown
Contributor

@masadcv masadcv commented May 1, 2021

Fixes #2125 .

Description

Fixes mypy error on some platforms occuring in EfficientNet BlockArgs definition.

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.

@masadcv masadcv changed the title EfficientNet mypy fix [WIP] EfficientNet mypy fix May 1, 2021
@masadcv masadcv force-pushed the efficientnet-mypy-fix branch 3 times, most recently from b0ab4f1 to 8e41426 Compare May 1, 2021 01:33
@masadcv
Copy link
Copy Markdown
Contributor Author

masadcv commented May 1, 2021

@wyli Apolpgies for messing this up, it was supposed to be an easy fix. I cannot seem to figure out how to correct DCO issue here.

@wyli
Copy link
Copy Markdown
Contributor

wyli commented May 1, 2021

@wyli Apolpgies for messing this up, it was supposed to be an easy fix. I cannot seem to figure out how to correct DCO issue here.

no worries perhaps do a "git reset @~2" and commit again with -s, then push with -f

@masadcv masadcv force-pushed the efficientnet-mypy-fix branch from 8e41426 to 13c875b Compare May 1, 2021 17:58
@masadcv masadcv changed the title [WIP] EfficientNet mypy fix EfficientNet mypy fix May 1, 2021
@masadcv
Copy link
Copy Markdown
Contributor Author

masadcv commented May 1, 2021

@wyli many thanks for help with git instructions, I fixed the commit and added a BlockArgs class with from_string and to_string functions. I also added a new unittest to test the new functions. This is ready for review, all unittests pass locally.

Copy link
Copy Markdown
Contributor

@wyli wyli left a comment

Choose a reason for hiding this comment

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

thanks!

@wyli wyli merged commit 2db93d8 into Project-MONAI:dev May 1, 2021
@masadcv masadcv deleted the efficientnet-mypy-fix branch May 1, 2021 22:15
wyli pushed a commit that referenced this pull request May 26, 2021
wyli pushed a commit that referenced this pull request May 26, 2021
wyli pushed a commit that referenced this pull request May 26, 2021
wyli pushed a commit that referenced this pull request May 26, 2021
wyli pushed a commit that referenced this pull request May 27, 2021
wyli pushed a commit that referenced this pull request May 27, 2021
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.

mypy error in efficientnet BlockArgs

2 participants