Closed tdrwenski closed 1 year ago
Update the findCoordAxis method used when subsetting coverages to first check axis name case sensitive, then case insensitive. Resolves https://github.com/Unidata/netcdf-java/issues/1219.
findCoordAxis
The unit test I made is a bit convoluted, I was doubting if it was the best way to test. The only alternative I see is testing in the TDS (which I did manually but could make an automated test as well).
Jenkins tests are passing with this PR: https://jenkins-aws.unidata.ucar.edu/view/Users/job/tara-netcdf-java/24/
Description of Changes
Update the
findCoordAxismethod used when subsetting coverages to first check axis name case sensitive, then case insensitive. Resolves https://github.com/Unidata/netcdf-java/issues/1219.The unit test I made is a bit convoluted, I was doubting if it was the best way to test. The only alternative I see is testing in the TDS (which I did manually but could make an automated test as well).
Jenkins tests are passing with this PR: https://jenkins-aws.unidata.ucar.edu/view/Users/job/tara-netcdf-java/24/
PR Checklist