CesiumGS / CesiumGS/cesium

Clean up KTX/CRN support

Open
#8,070 0 comments 0 reactions 0 assignees View on GitHub
type - cleanup
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

There are at least 3 places where we check if a file is KTX, CRN, or a standard web image. This could probably be unified.

We are also missing tests in these places, so once we unify them we could also add a test for it. For example, ImagereyProvider.loadImage does not test KTX/CRN loading.

Contributor guide

Open the contributing guide

Research direction

Start by locating the three file-type checks mentioned in the issue and read ImagereyProvider.loadImage, which is specifically noted as lacking KTX/CRN loading tests. Unify the checks and add coverage for the supported image types, including KTX and CRN, then run the affected test suite to confirm the behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
web-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.