CleverRaven/Cataclysm-DDA

Light do not respect 3D FoV

Open

#65.514 aberto em 6 de mai. de 2023

Ver no GitHub
 (3 comments) (2 reactions) (0 assignees)C++ (3.991 forks)batch import
(S2 - Confirmed)Help WantedZ-levels[C++]

Métricas do repositório

Stars
 (9.264 stars)
Métricas de merge de PR
 (Mesclagem média 6d 5h) (292 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine o sistema de iluminação, provavelmente em arquivos como src/lightmap.cpp ou src/fov.cpp. O bug indica que a luz de itens como lanternas só ilumina o nível z atual; a correção requer modificar a propagação da luz para incluir deslocamentos verticais. Revise os algoritmos de campo de visão 3D existentes na base de código para entender o sistema de coordenadas e implementar a propagação vertical da luz.
Pilha de tecnologia
None
Domain
desktop
Tipo Issue
Bug
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65