Project

General

Profile

Actions

Feature #563

closed

Improvement flag "Infra"

Added by Marko Lindqvist about 1 month ago. Updated 16 days ago.

Status:
Closed
Priority:
Normal
Category:
General
Target version:
Start date:
05/07/2024
Due date:
% Done:

0%

Estimated time:

Description

Add conversion improvement flag "Infra" to make an improvement to produce infrapoints.


Files

Actions #1

Updated by Marko Lindqvist about 1 month ago

Actions #2

Updated by Alina Lenk about 1 month ago

This should probably document the new flag in the rulesets / comments.txt.

I'm not sure about making it a ruleset error to have a Convert improvement that does nothing; in a ruleset where no other Convert production is available in the beginning, we might still want a way to not produce anything else. Plus there could (theoretically) be scripts checking what's being produced.

Actions #3

Updated by Marko Lindqvist 22 days ago

Alina Lenk wrote in #note-2:

I'm not sure about making it a ruleset error to have a Convert improvement that does nothing; in a ruleset where no other Convert production is available in the beginning, we might still want a way to not produce anything else. Plus there could (theoretically) be scripts checking what's being produced.

I don't think the codebase is ready for convert genus building that's not actually producing anything. It's the targets "Gold" (and with this patch "Infra") that eat up the production so it does not accumulate towards a building, not the Convert genus. I don't know what would happen if you would let production to accumulate towards nothing.

Actions #4

Updated by Alina Lenk 22 days ago

Marko Lindqvist wrote in #note-3:

I don't think the codebase is ready for convert genus building that's not actually producing anything. It's the targets "Gold" (and with this patch "Infra") that eat up the production so it does not accumulate towards a building, not the Convert genus. I don't know what would happen if you would let production to accumulate towards nothing.

It does look like the current code resets the shield stock purely based on whether it's Convert; but it's a fair point, we might want to hold off until we've properly formalized stuff around it.

Actions #6

Updated by Marko Lindqvist 16 days ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF