climatology 1.6.36.1#1365
Conversation
|
Please don't publish this in the master catalogue until it has been tested in all platforms. It has only been tested on Windows, as far as I'm aware. It could be published to the alpha or beta plugins catalog to allow further testing before being added to the master catalog please 🙏 |
|
@rgleason I tested the packages in cloudsmith on Debian 13, MacOS 26.3.1 and Windows 11 as you requested here. The plugin does not appear to work correctly on any of them, specifically:
More generally I'm not sure that the changes introduced in this proposed release (v1.6.36.1) are needed. The plugin works fine in the existing release (v1.6.35.1) in the master catalog. Most of the proposed changes introduced in v1.6.36.1 in rgleason/climatology_pi#75 appear to be in response to rgleason/climatology_pi#67 but most if not all of the changes appear not to be needed. |
|
@rgleason All of the data in the climatology plugin is read using the zu_read() function, which decompresses on the fly. So there is no need to add all of the decompression code that you added to the main body of the plugin above. That is why the plugin worked just fine before you added all the new decompression code. I think we should probably back out all of the recent code changes and start over again with clear goals and a plan. We can move this discussion to the climatology_pi repo, and sort it all out there. |
No description provided.