Skip to content

Upgrade pollination to 1.70.19 - #365

Open
Tom-Kingstone wants to merge 6 commits into
developfrom
LadybugTools_Toolkit-#349-UpgradePollination
Open

Tom-Kingstone wants to merge 6 commits into
developfrom
LadybugTools_Toolkit-#349-UpgradePollination

Conversation

@Tom-Kingstone

@Tom-Kingstone Tom-Kingstone commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

NOTE: Depends on

Issues addressed by this PR

Closes #349
Closes #364

This updates the pollination version to 1.70.19, and fixes problems that arise from this.

Also fixes an issue with plotting evaporative cooling potential from an EPW object.

Test files

Test with the milestone test procedure, and run run_tests.bat.

The batch tests currently throw 3 warnings: a CUDA warning about missing PATH environment variable(won't fix), a deprecation warning about python_toolkit's bson_unix_ticks using deprecated datetime.utcnow() method (to raise issue in python_toolkit and fix there), and a user warning about mismatched handles/labels (won't fix)

Changelog

  • Upgraded supported Pollination version to 1.70.19

Additional comments

@Tom-Kingstone
Tom-Kingstone requested review from a team, jamesramsden-bh and tg359 October 27, 2026 14:36
@Tom-Kingstone Tom-Kingstone self-assigned this Oct 27, 2026
@Tom-Kingstone Tom-Kingstone added type:external-api-changes Imposed changes, including from dependency across other BHoM repos type:feature New capability or enhancement labels Oct 27, 2026
@bhombot-ci

bhombot-ci Bot commented Nov 4, 2026

Copy link
Copy Markdown

Please be advised that the check with reference 126369183420 has more than 50 annotations of notes. API limitations restrict annotations to 50. You may need to rerun this check to obtain the next set when you make changes. At the time of reporting this check, there are 56 additional annotations waiting, made up of 56 errors and 0 warnings.

@SimosManiatis SimosManiatis left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it locally, all 240 tests pass.
3 warnings : CUDA, utcnow, legend handles/labels

Small thing: the GH test component (C#) still checks for "=== 237 passed in", so it returns false even when everything passes. Changing that line to this fixed it for me:

List<string> passingLines = lines.Where(line => line.Contains("===") && line.Contains(" passed") && !line.Contains("failed") && !line.Contains("error")).ToList();

Also saw the core check is failing, not sure if that's expected?

@bhombot-ci

bhombot-ci Bot commented Nov 5, 2026

Copy link
Copy Markdown

Please be advised that the check with reference 127018530292 has more than 50 annotations of notes. API limitations restrict annotations to 50. You may need to rerun this check to obtain the next set when you make changes. At the time of reporting this check, there are 56 additional annotations waiting, made up of 56 errors and 0 warnings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:external-api-changes Imposed changes, including from dependency across other BHoM repos type:feature New capability or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

evaporative_cooling_potential_epw missing comma when calling the series version of the method Upgrade pollination version

2 participants