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

Avoid the import error `MessagePassing not defined` #351

Merged

WenjieDu merged 2 commits into main from dev

Apr 17, 2024

Owner

WenjieDu commented Apr 17, 2024 •

edited

Loading

What does this PR do?

fixing Need to test building pypots package during CI #352;
in previous commits, we expose ObservationPropagation in the below line, but this will cause the exceptionname 'MessagePassing' is not defined if torch_geometric is not installed. We remove this line in this PR to fix this issue.

PyPOTS/pypots/nn/modules/raindrop/__init__.py

Line 16 in aecd14b

from .layers import ObservationPropagation


          fix: avoid the import error MessagePassing not defined when torch_geo…

fe4005d

…metric not installed;

Collaborator

coveralls commented Apr 17, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 8724705047

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 80.701%

Totals
Change from base Build 8724484843:	-0.002%
Covered Lines:	7025
Relevant Lines:	8705

💛 - Coveralls


          Test building package during CI (#353)

ec8bee3

* test: test building without torch-geometric during CI;

WenjieDu merged commit cb1ae37 into main

7 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet