DaymareOn/hdtSMP64

Remove hardcoded exterior world space ID list from WeatherManager

Open

#239 ouverte le 26 mars 2026

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (34 forks)github user discovery
enhancementhelp wantedrefactor

Métriques du dépôt

Stars
 (67 stars)
Métriques de merge PR
 (Merge moyen 15h 45m) (28 PRs mergées en 30 j)

Description

Summary

The NOT_EXTERIOR_WORLDS array in src/WeatherManager.h (used in WeatherManager::runWeatherTickImpl) contains a static, hardcoded list of Skyrim world space Form IDs that should be treated as non-exterior (i.e., no wind). This approach is fragile and difficult to maintain as it:

  • Requires recompiling the DLL to add/remove world spaces.
  • Does not account for modded world spaces that might also be interiors.
  • Was originally added in commit 34743ff.

Goal

Remove the hardcoded list by doing one or both of the following:

  1. Find a programmatic method — e.g., checking whether the world space has a valid sky/climate attached (similar to how the Creation Kit detects outdoor environments), or inspecting flags on the TESWorldSpace record. The RE server may have insight here.
  2. Move to a data/config file — allow users/maintainers to configure the excluded world space IDs externally (e.g., in the existing SMP config file) rather than baking them into the DLL.

References

Guide contributeur

Direction de recherche
Examinez la structure de l'enregistrement TESWorldSpace pour trouver des indicateurs ou des champs indiquant le statut externe/interieur, ou concevez un format de fichier de configuration pour l'exclusion des espaces mondiaux externes.
Stack technique
cpp
Domaine
game dev
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++GitSkyrim modding
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30