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

device for turning #1029

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

device for turning #1029

wants to merge 1 commit into from

Conversation

kostmo
Copy link
Member

@kostmo kostmo commented Jan 23, 2023

@kostmo kostmo marked this pull request as ready for review January 23, 2023 02:57
@kostmo kostmo requested review from xsebek and byorgey January 23, 2023 02:57
Copy link
Member

@xsebek xsebek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the concept (and shorter documentation), but I am not sure this should go in default entities.

threads still have turn, so how is this useful besides an academic “one-capability-one-device”?

This might as well be in a scenario and players would not have to waste resources on an extra device.

@kostmo
Copy link
Member Author

kostmo commented Jan 23, 2023

how is this useful besides an academic “one-capability-one-device”?

It is only useful for custom scenarios, where limited mobility (and restricted access to treads) would be part of the challenge mechanic.

This might as well be in a scenario and players would not have to waste resources on an extra device.

I would be fine with defining this per-scenario instead, but I don't quite understand this statement. Why would making this a member of the default entities obligate players to waste resources on it?

@xsebek
Copy link
Member

xsebek commented Jan 23, 2023

Because they will discover it through recipes. The game does not say “this entity is only for scenarios dont worry about it” in any way.

Also if we add an achievement like “all devices installed” then it will literally be required.

@byorgey
Copy link
Member

byorgey commented Jan 24, 2023

Yeah, once you have the ingredients to make a particular entity, its name shows up in your Recipes. But then the only way to find out what it is and what it does is to actually make one. At that point you can finally read its description. I think I agree with @xsebek. As a player, I assume that when the UI flashes "new recipe!" at me, it feels like it is giving me a hint that I might find it useful to construct this new thing. Given that, I would find it annoying if after going to the trouble of making that thing, I find out that it actually gives me no new capabilities whatsoever. It would feel sort of like the UI lied to me.

I think we should either just define this in whatever custom scenarios need it, or add this and also remove the turn capability from treads. But that would just add more complexity to the early game and tutorials.

(Maybe we should change how entity discovery works, but that is a bigger conversation.)

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

Successfully merging this pull request may close these issues.

3 participants