:fix: correct plot scaling calculation and leaf parameters

[qh681248]

PR Type

• Bugfix
• Feature

Description

The algorithm's execution was slow, not because of image size but due to the leafsize being too close to the coreset_size and there was a bug when we set downsampling factor = 8 which has been fixed here:

• Increase leafsize was from 10,000 to either 16,000, which speeds up the algorithm, even when downsampling = 1.
• Fix scale calculation to use downsampling factor squared
• Update leaf count to 16,000

How Has This Been Tested?

Test A: (Write your answer here.)
Test B: (Write your answer here.)
Test C: (Write your answer here.)

Does this PR introduce a breaking change?

No

Screenshots

(Write your answer here.)

Checklist before requesting a review

• I have made sure that my PR is not a duplicate.
• My code follows the style guidelines of this project.
• I have commented my code, particularly in hard-to-understand areas.
• I have performed a self-review of my code.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.
• Any dependent changes have been merged and published in downstream modules.

[github-actions]

Performance review

Commit 7119d65 - Merge a4cbc80 into bc5acf0

No significant changes to performance.

[github-actions]

Performance review

Commit eded51e - Merge 07dbc36 into bc5acf0

No significant changes to performance.

[gw265981]

From my local testing, I get RecursionError before the fix and it runs after the fix (takes around 200s).