Exawind / amr-wind

AMReX-based structured wind solver
https://exawind.github.io/amr-wind
Other
103 stars 78 forks source link

Modifying abl_sampling reg test to avoid NetCDF error #1051

Closed mbkuhn closed 1 month ago

mbkuhn commented 1 month ago

Summary

Specifying velocity as a sampling field and velocity components as derived fields doesn't produce issues in the mechanics of AMR-Wind, but it creates redundant variable names that produce a conflict in NetCDF. "NetCDF: String match to name in use". This avoids that error

Pull request type

Checklist

The following is included:

This PR was tested by running:

mbkuhn commented 1 month ago

@marchdf feel free to correct me if you can't reproduce this error, but this was popping up when running the reg tests locally

marchdf commented 1 month ago

I wasn't seeing this before... let me try locally.

mbkuhn commented 1 month ago

gonna put checks in the code instead