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
scene.available_composite_names() return a composite even the dependency is not fullfilled #853
Comments
Can you double check that this is still an issue @yufeizhu600 with current master? I forgot about this and this is an important issue. |
I upgraded to satpy==0.16.2.dev340+gd09edbc and done the same test as above. I still got ['airmass'] in the output of available composite names. [DEBUG: 2019-09-30 20:42:56 : satpy.scene] Setting 'PPP_CONFIG_DIR' to '/local/home/goesdev1/project/goesr/abi_l2_reader/config/' |
Describe the bug
It seems the scene.available_composite_names() function has bug to check the dependencies of the ABI airmass composite, it returns airmass as one of the available composite even one of the dependency is missing.
To Reproduce
The ABI airmass composite depend on 4 channels data files, however even one of the file is missing, the scene.available_composite_names() still return airmass in the list as as a available composite.
To reproduce the issue, one can copy a set of 16 GOES-16 FullDisk passes to the test input directory then manually delete one of the dependency of airmass composite.
%cp /goes-16-grb/*s20191922000356* ./test/
%rm ./test/OR_ABI-L1b-RadF-M6C08_G16_s20191922000356_e20191922010064_c20191922010119.nc
Expected behavior
Not expecting airmass in the print out list and loading 'airmass' composite should failed.
Actual results
'airmass' was in the available composite list, loading 'airmass' failed as expected.
['airmass', 'ash', 'cloudtop', 'convection', 'day_microphysics', 'dust', 'fog', 'green', 'green_crefl', 'green_raw', 'green_snow', 'ir108_3d', 'ir_cloud_day', 'natural', 'natural_color', 'natural_color_raw', 'natural_color_sun', 'night_fog', 'night_ir', 'night_microphysics', 'overview', 'overview_raw', 'pseudo_color_nomod', 'true_color', 'true_color_crefl', 'true_color_raw']
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Reading in get_dataset C12.
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Calibrating to brightness temperatures
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Reading in get_dataset C14.
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Calibrating to brightness temperatures
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Reading in get_dataset C10.
[DEBUG: 2019-07-11 20:30:26 : satpy.readers.abi_l1b] Calibrating to brightness temperatures
[WARNING: 2019-07-11 20:30:27 : satpy.readers.yaml_reader] Required file type 'c08' not found or loaded for 'C08'
[DEBUG: 2019-07-11 20:30:27 : satpy.scene] Missing prerequisite for 'DatasetID(name='airmass', wavelength=None, resolution=None, polarization=None, calibration=None, level=None, modifiers=None)': 'DatasetID(name='C08', wavelength=(5.77, 6.185, 6.6), resolution=2000, polarization=None, calibration='brightness_temperature', level=None, modifiers=())'
[WARNING: 2019-07-11 20:30:27 : satpy.scene] The following datasets were not created and may require resampling to be generated: DatasetID(name='airmass', wavelength=None, resolution=None, polarization=None, calibration=None, level=None, modifiers=None)
[DEBUG: 2019-07-11 20:30:27 : satpy.scene] Unloading dataset: DatasetID(name='C12', wavelength=(9.42, 9.61, 9.8), resolution=2000, polarization=None, calibration='brightness_temperature', level=None, modifiers=())
[DEBUG: 2019-07-11 20:30:27 : satpy.scene] Unloading dataset: DatasetID(name='C14', wavelength=(10.8, 11.2, 11.6), resolution=2000, polarization=None, calibration='brightness_temperature', level=None, modifiers=())
[DEBUG: 2019-07-11 20:30:27 : satpy.scene] Unloading dataset: DatasetID(name='C10', wavelength=(7.24, 7.34, 7.44), resolution=2000, polarization=None, calibration='brightness_temperature', level=None, modifiers=())
Environment Info:
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: