Skip to content

data: enrich data set with minimum illumination level (based on data from datasheet, specified in in sources) - #171

Merged
ch-bas merged 4 commits into
ch-bas:mainfrom
fvdpol:enrich-lux
Jul 28, 2026
Merged

data: enrich data set with minimum illumination level (based on data from datasheet, specified in in sources)#171
ch-bas merged 4 commits into
ch-bas:mainfrom
fvdpol:enrich-lux

Conversation

@fvdpol

@fvdpol fvdpol commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Enrich data set with minimum illumination level (based on data from datasheet, specified in in sources)

Checklist

For corrections

  • Include a source URL confirming the correct value
  • npm run build passes locally with no errors

Source(s)

sources used are the datasheets listed in the camera.json sources


Notes

Where a minimum illumination for color image has been specified this has been stored in the min_lux_color attribute; otherwise, if no mention of color image, or a black & white image has been specified, this has been stored in the generic "min_lux" attribute.

Total cameras: 2540
Entries with 'night_vision': 2482
Entries with 'min_lux': 31 1.25%
Entries with 'min_lux_color': 300 12.09%

@fvdpol

fvdpol commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

this PR gives some progress on the enrichment discussed in issue #161

@ch-bas
ch-bas merged commit 4b19cc5 into ch-bas:main Jul 28, 2026
2 checks passed
ch-bas added a commit that referenced this pull request Jul 28, 2026
…ux_color 300, min_lux 31)

Co-authored-by: fvdpol <fvdpol@users.noreply.github.com>
@ch-bas

ch-bas commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Thank you, @fvdpol! 🙌
This doubles the low-light-lux coverage (min_lux_color 136 → 300, min_lux 6 → 31) and is exactly the datasheet sourced back-fill that lane was created for. It's merged now!

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