This PR fixes an error in the reading and writing of patch files for use with the low_mem=True option, reported by @chto in Issue #144. The patch files were intended to be written in radians to avoid an unnecessary multiply and divide on the round trip through a file. But there were some inconsistent applications of the units, so it ended up wrong. I cleaned that up in general, and added unit tests of this use case to make sure it works.
Will release this fix as a bug-fix release v4.2.9 shortly.
This PR fixes an error in the reading and writing of patch files for use with the low_mem=True option, reported by @chto in Issue #144. The patch files were intended to be written in radians to avoid an unnecessary multiply and divide on the round trip through a file. But there were some inconsistent applications of the units, so it ended up wrong. I cleaned that up in general, and added unit tests of this use case to make sure it works.
Will release this fix as a bug-fix release v4.2.9 shortly.