Skip to content
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.

Sign up for GitHub

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

Jump to bottom

pylint error after update #4016

Closed
wyli opened this issue Mar 28, 2022 · 0 comments · Fixed by #4021
Closed

pylint error after update #4016

wyli opened this issue Mar 28, 2022 · 0 comments · Fixed by #4021
Assignees
@wyli

Comments

@wyli
Copy link
Contributor

wyli commented Mar 28, 2022

Describe the bug

pylint
pylint 2.13.2
astroid 2.11.2
Python 3.8.12 (default, Oct 18 2021, 14:07:50) 
[GCC 9.3.0]
************* Module monai.networks.nets.dints
monai/networks/nets/dints.py:820:22: E0203: Access to member 'node2in' before its definition line 836 (access-member-before-definition)
monai/networks/nets/dints.py:821:23: E0203: Access to member 'node2out' before its definition line 837 (access-member-before-definition)
Check failed!
@wyli wyli self-assigned this Mar 28, 2022
wyli added a commit that referenced this issue Mar 28, 2022
@wyli
fixes #4016
df2756e 
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli mentioned this issue Mar 29, 2022
Merged
7 tasks
wyli added a commit that referenced this issue Mar 30, 2022
@wyli
4010 4016 fixes mmar integration tests (#4021)
df19315 
* fixes unit test

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* temp tests

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* adds docker version tests

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* fixes #4016

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* local tests

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* remove mmar cache daily

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* remove cache

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* Revert "temp tests"

This reverts commit 238d883.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* fixes typo

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wyli wyli

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

4010 4016 fixes mmar integration tests Project-MONAI/MONAI
1 participant
@wyli