Skip to content

EfficientNet mypy fix #2126

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

Merged
merged 2 commits into from
May 1, 2021
Merged

Conversation

masadcv
Copy link
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
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
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
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
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