NCAS-CMS / cf-python

A CF-compliant Earth Science data analysis library
http://ncas-cms.github.io/cf-python
MIT License
125 stars 19 forks source link

Allow DSG tractories with identical `trajectory_id` values to be aggregated #724

Closed davidhassell closed 8 months ago

davidhassell commented 8 months ago

Fixes #723

Needs https://github.com/NCAS-CMS/cfdm/pull/290 for the new test to work.

Full fix also needs #730

sadielbartholomew commented 8 months ago

Hi David, I am in the middle of a meeting but afterwards I will complete my review. I noticed before that I messed up the one trivial conflict resolution for the change log, sorry (one entry was appearing at the top and bottom of the list and I didn't notice) - all that is needed is to remove one duplicated entry from another PR, so I will push up another commit to do that after the meeting.