Fix 1246: var_list not being applied appropriately to obs_vars from networks

[lewisblake]

Change Summary

• colocator.py::_filter_var_matches_var_names() checks on string equality not subset inclusion
• pyaerocom/aeroval/experiment_processor.py: code has been added such that if a var_list is given, only run against the obs networks which contain that variable
• A test has been added which checks the above functionality

Related issue number

Fix #1246

Checklist

• Start with a draft-PR
• The PR title is a good summary of the changes
• PR is set to AeroTools and a tentative milestone
• Documentation reflects the changes where applicable
• Tests for the changes exist where applicable
• Tests pass locally
• Tests pass on CI
• At least 1 reviewer is selected
• Make PR ready to review

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.33%. Comparing base (b1fa238) to head (77da745).
Report is 447 commits behind head on main-dev.

Additional details and impacted files

@@             Coverage Diff              @@
##           main-dev    #1249      +/-   ##
============================================
- Coverage     79.33%   79.33%   -0.01%     
============================================
  Files           133      133              
  Lines         20289    20308      +19     
============================================
+ Hits          16096    16111      +15     
- Misses         4193     4197       +4     

Flag	Coverage Δ
unittests	79.33% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[heikoklein]

Nice, thanks.

[lewisblake]

After some subsequent testing I worry that this fix may have broken something with _sync_heatmaps_with_menu_and_regions, so will need to keep an eye out for that.

[heikoklein]

I had problems with _sync_heatmaps_with_menu_and_regions before, i.e. I could not add EEA to a run which previously only had EBAS observations without deleting all data. (key error in the heatmaps-data)

[heikoklein]

I'm not sure why the config runs ana.update_interface() before ana.run()? (ana.run() runs update_interface internally, if I understand correctly.)

[lewisblake]

I had problems with _sync_heatmaps_with_menu_and_regions before, i.e. I could not add EEA to a run which previously only had EBAS observations without deleting all data. (key error in the heatmaps-data)

Yes this key error is the same one I encountered.

I'm not sure why the config runs ana.update_interface() before ana.run()? (ana.run() runs update_interface internally, if I understand correctly.)

Not quite sure either, I'd have to debug, but I expect it's to avoid key errors when writing the json files.