Oberlus wrote:Is this intended?
I don't think so; I am pretty sure it is an accidental glitch from the move of meter regen into Effects. The basic Research and Industry meter regen is currently part of BASIC_RESEARCH and BASIC_INDUSTRY, and so gets left out of NO_RESEARCH and NO_INDUSTRY.
I don't like it.
I agree. I find myself rather surprised I had not noticed this before. Certainly a tech like Nascent AI tech is pretty explicitly supposed to be independent of the racial research abilities: "Each planet can contain such structures regardless of the population's occupations, giving a bonus to research on all worlds." And regardless of whether other possible fixed sources would necessarily be to explicitly applicable, if the effect sets the target meter as shown in the screenshot, then the target meter should be allowed to grow. (The lone currently expected exception to that is for planets with a 0 target population, and even the current implementation of that seems a bit buggy to me since if it is going to hold the primary meter to zero it should also hold the target meter to zero, plus the comment description is "// removes residual production from a dead planet" whereas the implementation applies to any dying
planet. But that would be a different Issue than the one of your post.)
I think the most expeditious fix is simply to extract the respective meter regen clauses out of the BASIC_X macros, into a separate X_REGEN macro which could then be referenced by both the BASIC_X macro and the NO_X macro. Or probably better yet, just move all the regen clauses into a single METER_REGEN macro which could then be incorporated into BASIC_POPULATION.
Could you please open an Issue on this, and quote the above paragraph re suggested approach for fix?
(And feel free to open one about the dead vs dying planet issue, and about the related problem of zeroing the primary meter without zeroing the target meter too, if those make sense to you.)