BUG: `test_load_multiple_days` failures

[jklenzing]

Description

pysat 3.2.0 has a new universal test: test_load_multiple_days. Several instruments fail this test

To Reproduce this bug:

run pytest with pysat develop

Test configuration

• OS: Ventura
• Version: Python 3.10.9
• pysat develop

Additional context

• [x] ace_epam_l2, inst_id='12sec', tag='base'
• [x] ace_mag_l2, inst_id='16sec', tag='base'
• [x] ace_mag_l2, inst_id='16sec', tag='key'
• [x] ace_sis_l2, inst_id='256sec', tag='base'
• [x] ace_swepam_l2, inst_id='164sec', tag='base'
• [x] cnofs_vefi, tag='dc_b'
• [x] de2_fpi
• [x] de2_lang
• [x] de2_nacs
• [x] de2_rpa
• [x] de2_vefi, tag=''
• [ ] icon_euv
• [x] icon_fuv, tag='day'
• [x] icon_fuv, tag='night'
• [x] icon_mighti, inst_id='a', tag='los_wind_green'
• [x] icon_mighti, inst_id='a', tag='los_wind_red'
• [x] icon_mighti, inst_id='a', tag='temperature'
• [x] icon_mighti, inst_id='b', tag='los_wind_green'
• [x] icon_mighti, inst_id='b', tag='los_wind_red'
• [x] icon_mighti, inst_id='b', tag='temperature'
• [ ] maven_insitu_kp
• [x] ses14_gold, tag='tdisk'
• [x] ses14_gold, tag='o2den'
• [x] timed_guvi, inst_id='high_res', tag='sdr_spectrograph'
• [x] timed_guvi, inst_id='low_res', tag='sdr_spectrograph'

[jklenzing]

Some of these are cause by an issue with CDAS download routines, where the localize function adds a timestamp of '00:00:00' to a date, meaning only the first day will be returned rather than two days. BUGFIX downstream of #198

https://github.com/pysat/pysatNASA/blob/4f0df4667aa8112f9cee39a4d61f8cd0c72b42b5/pysatNASA/instruments/methods/cdaweb.py#L965-L967

Using pysat.utils.time.filter_datetime_inputinstead fixes this problem for some of the data products. Fixes 15 of the above errors

[jklenzing]

There's also an issue if data files do not begin exactly at zero seconds. CDAS WS interprets the end date as a beginning of day, so if data starts at 00:00:01 it will ignore the file. Adding 86399 seconds to the end date fixes this for another 5 instruments.

[jklenzing]

DE2 VEFI fixed by #217. New tests run when those changes are pulled in

[jklenzing]

TODO:

• [ ] fix icon tests
• [ ] version bump for other insts

[jklenzing]

The remaining issues (icon_euv, maven_insitu_kp) are basically the same => the datasets have covariance matrices with duplicate dimension names, ie dims = (time, compno_3, compno_3). Will write up a new issue.