Closed Aariq closed 1 year ago
Should I add a NEWS.md
to the settings package and bump the version?
Looks like test coverage for these functions is 0%, but I did install this PR and test with an existing run and the behavior is correct. Not sure how to write tests for database stuff, but I can try if you'd like.
Description
Adds a
write
argument toget.trait.data()
andget.trait.data.pft()
and changesrunModule.get.trait.data()
andget.trait.data()
to correctly pass thewrite
argument. Minor fixes to documentation, code formatting, and fixed one unrelated check warning insite.pft.link.settings()
Motivation and Context
Closes #2968
Review Time Estimate
Types of changes
Checklist: