pytest satpy/tests does not work

[sahar-github]

Describe the bug/error
Import error while loading conftest
could you please guide me how should i resolve this issue?
Thanks

Actual results
Import error while loading conftest
satpy/--init--.py:21: in module
from satpy.version import version as --version--
module not found: no module named satpy.version

[mraspaud]

Hi, could you tell us how you installed satpy?

[sahar-github]

Hi thanks for your prompt response I just followed the steps from the developer's guide: Developer’s Guide — Satpy 0.30.1.dev0+ga6f26f3a.d20210917 documentation <https://satpy.readthedocs.io/en/stable/dev_guide/index.html>

…

On Tue, Sep 21, 2021 at 12:16 PM Martin Raspaud ***@***.***> wrote: Hi, could you tell us how you installed satpy? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1829 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASRG2E6JDYCHLOHP4LQBO3LUDBLO3ANCNFSM5EOAQZRQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[djhoese]

Can you try the pip install -e . part again and make sure it succeeds? The version.py module is created when Satpy is first installed. If it isn't properly installed that module won't be created and you get this error.

[sahar-github]

thanks for your suggestion! It is successfully installed
It seems the first issue was that I had two satpy folders instead of one, so I had to run the test via satpy/satpy/tests
but now I have several other errors raising, please see the attached screenshot

[djhoese]

Some of these are optional dependencies. Unless you plan on doing development on the things being tested in these failing tests I personally wouldn't worry about it. You can find the full list of dependencies needed to run all tests here:

satpy/setup.py

Lines 38 to 40 in a6f26f3

	test_requires = ['behave', 'h5py', 'netCDF4', 'pyhdf', 'imageio', 'libtiff',
	'rasterio', 'geoviews', 'trollimage', 'fsspec', 'bottleneck',
	'rioxarray', 'pytest', 'pytest-lazy-fixture']

I'm realizing though that there isn't an easy way to install these dependencies except to do it manually. I'll make a pull request for that. Regardless, what are you hoping to work on? I wouldn't worry about running all the Satpy tests unless you plan on doing large changes to the library. Each reader and writer has their own dependencies so installing all of them can get annoying (especially the C-based compile ones).

Hope this clears some stuff up. I'll work on making this easier in a new PR this morning and documenting it better.

[sahar-github]

Thanks for the declaration. I would like to add a new reader to satpy, and for that I do not require dependencies of MODIS. Then I can just ignore the errors right?

[djhoese]

Yes, I think they can be ignored. Or you could install the extra dependencies yourself and then the tests would pass. It is up to you.

I'm going to consider this issue closed as I think we've addressed the primary issue (missing dependencies).

[sahar-github]

Thank you

…

On Wed, Sep 22, 2021 at 3:29 PM David Hoese ***@***.***> wrote: Yes, I think they can be ignored. Or you could install the extra dependencies yourself and then the tests would pass. It is up to you. I'm going to consider this issue closed as I think we've addressed the primary issue (missing dependencies). — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1829 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASRG2E6JE4UOYGKDRNBA7PDUDHK4RANCNFSM5EOAQZRQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.