CesiumGS / CesiumGS/cesium-native

Use `SharedAssetSystem` for schemas in `MAXAR_content_geojson`

Open
#1,406 0 comments 0 reactions 0 assignees View on GitHub
quality
Dominant language
C++
Stars
623
Forks
277
PR merge metrics
No merged PRs in 30d

Description

In #1402 we had to do a workaround to pass the schema from `MAXAR_content_geojson` to external tileset loaders, due to incorrect URLs in target test data. Since they're all supposed to reference the same schema object, we should change this to use the `SharedAssetSystem` for the schema in the future.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing MAXAR_content_geojson and the workaround described in #1402, then trace how its schema is passed to external tileset loaders. Compare that flow with SharedAssetSystem usage. The work is done when the schema uses SharedAssetSystem consistently and the workaround for the target test data is no longer needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.