CleverRaven/Cataclysm-DDA

Light do not respect 3D FoV

Open

#65 514 ouverte le 6 mai 2023

Voir sur GitHub
 (3 commentaires) (2 réactions) (0 assignés)C++ (3 991 forks)batch import
(S2 - Confirmed)Help WantedZ-levels[C++]

Métriques du dépôt

Stars
 (9 264 stars)
Métriques de merge PR
 (Merge moyen 6j 5h) (292 PRs mergées en 30 j)

Description

Describe the bug

If you try to use lighter at night, only your z-level would be illuminated

Attach save file

N/A

Steps to reproduce

  1. Set time to 23:00
  2. Teleport to any roof or z-level 1
  3. Use flashlight
  4. See nothing above and under your character is illuminated

Expected behavior

Being able to see stuff below

Screenshots

GIF 06-05-2023 20-00-17

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19044.2846 (21H2)
  • Game Version: bbfa357 [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [ Dark Days Ahead [dda], Disable NPC Needs [no_npc_food], No Fungal Growth [no_fungal_growth], Bionic Professions [package_bionic_professions], Magiclysm [magiclysm], Xedra Evolved [xedra_evolved], Mythical Martial Arts [MMA] ]

Additional context

No response

Guide contributeur

Direction de recherche
Examinez le système d'éclairage, probablement dans des fichiers comme src/lightmap.cpp ou src/fov.cpp. Le bug indique que la lumière provenant d'objets comme les lampes torches n'éclaire que le niveau z actuel ; la correction nécessite de modifier la propagation de la lumière pour inclure les décalages verticaux. Passez en revue les algorithmes de champ de vision 3D existants dans la base de code pour comprendre le système de coordonnées et implémenter la diffusion verticale de la lumière.
Stack technique
Aucun
Domaine
desktop
Type d'issue
Bug
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65