search

pytroll / pytroll-pps-runner

Pytroll runner for PPS
GNU General Public License v3.0
1 stars 8 forks source link

Add runner for seviri #11

Closed smhi-erik closed 4 years ago

smhi-erik commented 4 years ago

For seviri the orbit number is not enough for finding files since seviri do not have any orbit number. Therefore added the start time as well

mraspaud commented 4 years ago

could you run flake8 with the docstring plugin on your PR ? I see for example some trailing spaces that should be removed

mraspaud commented 4 years ago

Also: could a test be added to check the new feature?

mraspaud commented 4 years ago

@smhi-erik don't close the PR, just push to your branch for updates!

adybbroe commented 4 years ago

I still don't manage to run pytest:

========================================================================= ERRORS ==========================================================================
__________________________________________________ ERROR collecting nwcsafpps_runner/tests/test_utils.py __________________________________________________
ImportError while importing test module '/home/a000680/usr/src/forks/pytroll-pps-runner/nwcsafpps_runner/tests/test_utils.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
nwcsafpps_runner/tests/test_utils.py:24: in <module>
    from nwcsafpps_runner.utils import get_outputfiles
E   ModuleNotFoundError: No module named 'nwcsafpps_runner'
================================================================= short test summary info =================================================================
ERROR nwcsafpps_runner/tests/test_utils.py