Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.
This repository was archived by the owner on May 22, 2025. It is now read-only.

Incorrect lumen calculation in some cases #17519

Closed

Description

@ThatLing

Severity

Inconvenience

Short Description

Incorrect lumen calculation in some cases

Reproduction

Supposedly podpeople struggle on lavaland because they do not get enough light, even though they should have enough

Round ID

Any

Test Merges

No response

BYOND Username

No response

Was this done on a locally hosted or non-Yogstation hosted server?

None

Additional information

No response

Activity

  1. GraveHat commented on Jan 16, 2023

    @GraveHat
    Contributor

    Note that moving to pre-existing light sources stops them from dying, but turning your light off and on again after getting the message does not. In-game this feels more like the system is lagging behind the player constantly moving but standing still after getting the notification does not seem to stop the suffocation as it still counts as dark.

    The forewarning notification Pod people get that warns them that there's not much light in the area will be seen frequently while simply walking around before getting immersed in total "Darkness."

  2. ThatLing commented on Feb 12, 2023

    @ThatLing
    ContributorAuthor

    After some investigating, it seems like some turfs don't have their lighting corners set (e.g. lighting_corner_NE). This happens in /datum/light_source/proc/update_corners which is called from SSlighting fire.

  3. ThatLing commented on Feb 12, 2023

    @ThatLing
    ContributorAuthor

    After some investigating, it seems like some turfs don't have their lighting corners set (e.g. lighting_corner_NE). This happens in /datum/light_source/proc/update_corners which is called from SSlighting fire.

    Disregard this, the corners are for static lighting. turf.dynamic_lumcount is used for dynamic lighting, maybe it's just not high enough?

  4. Oblisk234 commented on Nov 13, 2024

    @Oblisk234
    Member

    Was fixed by #21221.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    BugSomething isn't workingCoding IssueThis is an issue to be handled by codersIssue - ConfirmedThis issue has been confirmed and should be fixed

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions