Use `Model` for `.vctr` and `.geom` contents
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 15.8k
- Forks
- 3.9k
- Avg merge
- 4d 6h
- Merged PRs (30d)
- 34
Description
Right now, vctr and geom contents still use custom classes (Vector3DTileContent and Geometry3DTileContent).
This would involve:
- Making
VctrLoaderto parse.vctrfiles and transcode to aModelComponentsinstance, much likeB3dmLoaderdoes for.b3dmfile - Similarly, make a
GeomLoaderfor.geomfiles. - Add pipeline stages to
Modelas needed to implement any unique functionality - Check that picking/styling works with
ModelFeatureTableinstead of the oldCesium3DTileBatchTable
Ideally, at the end of this, nothing will use Cesium3DTileBatchTable (vctr and geom were the main stragglers), and we can consolidate that class with ModelFeatureTable to make things simpler.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by comparing the planned VctrLoader and GeomLoader with B3dmLoader, then trace how ModelComponents and Model pipeline stages handle the existing vctr and geom paths. Inspect picking and styling through ModelFeatureTable versus Cesium3DTileBatchTable. Done means both formats use Model, their unique behavior works, and the old batch table is no longer needed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- computer-graphics
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100