Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syncing lights #15

Open
dcsan opened this issue Nov 10, 2020 · 16 comments
Open

syncing lights #15

dcsan opened this issue Nov 10, 2020 · 16 comments
Assignees
Labels

Comments

@dcsan
Copy link
Owner

dcsan commented Nov 10, 2020

lights

The textual descriptions, icons, and images of lights are not in sync. At the start of the game, 0/4 lights are on. With each task completed, one should come one until all four are completed. 4 lights are not directly related to 6 tasks-- do you think this is confusing? (I don't necessarily think so)

right now

  • task1 = 1 lights
  • task2 = 2 lights etc.

But it may get out of sync. it's a bit complicated/messy to track really (one thing i want to clean up with a proper hints/task system). did you notice where it goes out of sync?

or should it be

  • task1 = 0 lights
  • task2 = 1 lights

maybe thats why i'm confused
ach task3 is out of sync.
let me know what it should be here and i'll see if we can make a system to fix it.
ideally this would be part of a task list and we'd assign everything in a centralised place, but thats a lot more code.
its just a mess cleaning up scatted across the script.
also i did get confused originally on the counting.

I could make something like progress: 3 a function that just renders the little progress bar. but then it would be more "fixed"

task2

image

task3

yeah looks like this is out of sync
image

task4

image

it looks like maybe lights were edited?

image

@dcsan
Copy link
Owner Author

dcsan commented Nov 10, 2020

lets' discuss this one here, big topic

@dcsan dcsan mentioned this issue Nov 10, 2020
11 tasks
dcsan added a commit that referenced this issue Nov 10, 2020
@intellikat
Copy link
Collaborator

intellikat commented Nov 10, 2020

I think it's prob best to playthrough and test all the lights to see where it isn't syncing up. 

First light should come on when clothes are donned, then proceed from there. There should only be 3 lights on in the dark, but now there is still the old 4 lights image.

Sometimes the image doesn't sync with the icons or the textual description as seen in this one, at the beginning when no lights are on:

Screen Shot 2020-11-10 at 10 44 21 PM

Another time is here:

Screen Shot 2020-11-10 at 10 48 36 PM

@dcsan
Copy link
Owner Author

dcsan commented Nov 10, 2020

you mean

tasks 3/6
img: 2 lights
emoji: 2 lights?

it should be 3 lights?

@dcsan dcsan added the p1 label Nov 12, 2020
@intellikat
Copy link
Collaborator

intellikat commented Nov 13, 2020

The "task x/x" is something you coded so I don't want to comment on that. I think your task number is +1 of the lights, but you can check that part to see if it's synced. There's also a difference between "doing task 3" vs. "task 3 completed" since the completion of a puzzle task results in a light coming on.

What I mean is that at any point there are 3 different ways that the number of lights currently on is indicated to the player.

#1 command "look room" (jpeg image) (also automatically triggered when entering the room and when the room goes dark)

The proper illustration should always match the state of the lights in the room. The progression is: no lights, 1 light, 2 lights, 3 lights/dark room, 4 lights.

Screen Shot 2020-11-13 at 8 55 22 AM

#2 command "look lights" (icon) (also automatically triggered when completing a puzzle task)

You can see in the image here how this is not synced properly as only one light should be on.

Screen Shot 2020-11-13 at 9 00 37 AM

#3 text descriptions. Any text that describes the lights at any point should describe the proper state.

Easiest way is to run through the game and test each of these by "look room" and "look lights" during each state to check.

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

this lights thing is very confusing to track. I just need a clear list of what lights go with what and I can fix the story.

enter room - no lights
you get assigned task1 - to get dresssed
on completion 1 light comes on?

so basically at start of task1 = 0 light
at end of task1 => task2 => 1 light

so basically task[num] - 1

task1: get dressed => 0 light => complete => 1 light
task2: 1 light exercise =>
task3: 2 lightss wash =>
task4: 3 lights / (dark room) green =>

seems to match:

[Task 4/6] Three of the green lights are on, and a yellow light seeps through the window.

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

if that's correct one of us could clean up the script

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

task cuts


      - name: task1
        imageUrl: bureau/ch1/500/bh-ch1-no-lights.jpg
        short: "`[Task 1/6]` You are in what appears to be a mental patient’s bedroom. \n
          You see a desk, a sink, a window, a door, lights, and a wall covered with images. \n"
        navbar: full

      # repeat room description
      - name: task2
        imageUrl: bureau/ch1/500/bh-ch1-1-light.jpg
        short: "💡 … … …\n
          `[Task 2/6]` You are in what appears to be a mental patient’s bedroom. \n
          You see a desk, a sink, a window, a door, lights, and a wall covered with images.
          \n
          \nOne of the green lights has come on.
          "
        navbar: full

      - name: task3
        imageUrl: bureau/ch1/500/bh-ch1-2-lights.jpg
        short: "💡💡 … …\n
        `[Task 3/6]`
          Two of the green lights are on."
        navbar: full

      - name: task4
        # imageUrl: asylum/rooms/cell-dark.jpg
        # imageUrl: bureau/ch1/500/bh-ch1-3-lights.jpg
        imageUrl: bureau/ch1/500/bh-ch1-4-lights-dark.jpg
        short: "💡💡💡 …\n
          `[Task 4/6]`
          Three of the green lights are on, and a yellow light seeps through the window."
        navbar: full

      - name: task5
        imageUrl: bureau/ch1/500/bh-ch1-4-lights.jpg
        short: "💡💡💡💡\n
          All four of the green lights are on."

lights

      - name: lights
        called: wall lights|lightbulbs|lamps
        states:
          - name: default
            short: A vertical row of small green lightbulbs is built into the wall above the sink.
          - name: task1
            short: … … … …
              None of the lights are lit up.
          - name: task2
            short: 💡 … … …
              One of the green lights has come on.
          - name: task3
            short: 💡💡 … …
              Two of the green lights are lit up.
          - name: task4
            short: 💡💡💡 …
              Three of the green lights are lit.
          - name: task5
            short: 💡💡💡💡
              More green lights are lighting up; it seems you're making progress.

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

There's also a difference between "doing task 3" vs. "task 3 completed" since the completion of a puzzle task results in a light coming on.

has code

        imageUrl: bureau/ch1/500/bh-ch1-4-lights-dark.jpg
        short: "💡💡💡 …\n

is this still wrong?

easy enough to change it back to:

    # imageUrl: bureau/ch1/500/bh-ch1-3-lights.jpg

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

image

synced at least

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

before

image

OK there is no '3 lights dark' so ..

@dcsan
Copy link
Owner Author

dcsan commented Nov 14, 2020

there's only 3 lights - normal
so ... i'm not sure if we're missing an image or how to deal with this

image

dcsan added a commit that referenced this issue Nov 14, 2020
@dcsan dcsan removed their assignment Nov 14, 2020
@intellikat
Copy link
Collaborator

intellikat commented Nov 14, 2020 via email

@dcsan
Copy link
Owner Author

dcsan commented Nov 15, 2020

ok found the image.

dcsan added a commit that referenced this issue Nov 15, 2020
@dcsan
Copy link
Owner Author

dcsan commented Nov 15, 2020

think done now

@dcsan dcsan closed this as completed Nov 15, 2020
@dcsan dcsan reopened this Nov 17, 2020
@dcsan
Copy link
Owner Author

dcsan commented Nov 17, 2020

seems like it goes backwards

image

image

image

@dcsan
Copy link
Owner Author

dcsan commented Nov 17, 2020

another issue

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants