DaymareOn/hdtSMP64

Remove hardcoded exterior world space ID list from WeatherManager

Open

#239 aberto em 26 de mar. de 2026

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (34 forks)github user discovery
enhancementhelp wantedrefactor

Métricas do repositório

Stars
 (67 stars)
Métricas de merge de PR
 (Mesclagem média 15h 45m) (28 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue a estrutura do registro TESWorldSpace para encontrar flags ou campos que indiquem o status externo/interno, ou projete um formato de arquivo de configuração para exclusão de espaços mundiais externos.
Pilha de tecnologia
cpp
Domain
game dev
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++GitSkyrim modding
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30