compas-dev / compas-dev/compas_viewer

Simplify Configs

Open
#108 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
12
Forks
12
PR merge metrics
No merged PRs in 30d

Description

Currently there exist a series of Config related classes. For instance for`Camera` there is a `CameraConfig`, then even a `CameraConfigType`. For me these layers feels very redundant. Why not inherent all these objects from `Data` class, then data needed for setting up the camera can be handled by `__data__` and `__from_data__`, for type checking we use `DATASCHEMA`. @tomvanmele What do you think?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.