Skip to content

PR: Vectorise *Robertson (1968)* Correlated Colour Temperature Computation#1367

Merged
KelSolaar merged 1 commit intodevelopfrom
feature/robertson1968
Oct 18, 2025
Merged

PR: Vectorise *Robertson (1968)* Correlated Colour Temperature Computation#1367
KelSolaar merged 1 commit intodevelopfrom
feature/robertson1968

Conversation

@KelSolaar
Copy link
Copy Markdown
Member

@KelSolaar KelSolaar commented Oct 18, 2025

Summary

This PR vectorisescolour.temperature.uv_to_CCT_Robertson1968 and colour.temperature.CCT_to_uv_Robertson1968 definitions for 32.78x and 93.61x performance improvement.

The implementation has been verified to reproduce the same values than the non-vectorised version with samples as follows:

  • 50 CCT values: 2,000K to 25,000K
  • 50 D_uv values: -0.100 to +0.100
  • Total: 2,500 samples

Preflight

Code Style and Quality

  • Unit tests have been implemented and passed.
  • Pyright static checking has been run and passed.
  • Pre-commit hooks have been run and passed.
  • [N/A] New transformations have been added to the Automatic Colour Conversion Graph.
  • [N/A] New transformations have been exported to the relevant namespaces, e.g. colour, colour.models.

Documentation

  • [N/A] New features are documented along with examples if relevant.
  • The documentation is Sphinx and numpydoc compliant.

…mperature.CCT_to_uv_Robertson1968` definitions for 32.78x and 93.61x performance improvement.
@KelSolaar KelSolaar added this to the v0.4.7 milestone Oct 18, 2025
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 99.25%. remained the same
when pulling 66e5c4d on feature/robertson1968
into 0135c3a on develop.

@KelSolaar KelSolaar merged commit acdc0d0 into develop Oct 18, 2025
32 checks passed
@KelSolaar KelSolaar changed the title PR: Vectorise Robertson (1968) Correlated Colour Temperature Computation PR: Vectorise *Robertson (1968)* Correlated Colour Temperature Computation Oct 18, 2025
@KelSolaar KelSolaar deleted the feature/robertson1968 branch October 19, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants