SlimeDog
commented
10 months ago Since it is possible -- even reasonable -- to run FancyHolograms in parallel with DecentHolograms or HolographicDisplays, I suppose we need to add a selector to the configuration, first-up after the header comments. Something like:
# ========== HOLOGRAM PROVIDER =====================================================================
# One of
# DecentHolograms
# FancyHolograms
# HolographicDisplays
# If the configured hologram provider fails to load or enable,
# pHD will disable and post an ERROR message to the console/log.
hologram-provider: DecentHologramsSuch a specification should be required if more than one supported hologram provider is enabled.
Alternatively, we could support more than one hologram provider, if enabled. But we cannot guarantee that the hologram names would be unique. That could be resolved with a name header decentholograms:lobby-blackdog for example. And pHD could require resolution at the command invocation. But I'm not convinced that introducing that complexity is a good thing, given that most users will employ only one hologram provider.
Given current usage, DecentHolograms (60+%) should be the default, if the specification is missing (legacy configuration file, for example) and in the canonical configuration file. If usage statistics suggest otherwise in future, it can be changed in a subsequent release.
Is your feature request related to a problem? Please describe. Updates for DecentHolograms and HolographicDisplays are increasingly delayed. In order to hedge our bets, we should expand our offerings.
Describe the solution you'd like Add support for FancyHolograms. The API looks to be sufficient.
Describe alternatives you've considered None.
Additional context Add any other context or screenshots about the feature request here.