search

carriontrooper / Alpha-Frontier

An Unciv total conversion mod, taking inspiration from Alpha Centauri and Beyond Earth. Uses FantasyHex tileset.
85 stars 16 forks source link

Buildings not applying bonus to resources #28

Closed Zoogzin closed 2 years ago

Zoogzin commented 2 years ago

The civilopedia lists chitin as a bonus resource, but the greenhouse does not add 1 food to it upon construction. Screenshot_20220605-104946_UnCiv Screenshot_20220605-104957_UnCiv

Zoogzin commented 2 years ago

Same for exotic fruits Screenshot_20220605-115547_UnCiv

Zoogzin commented 2 years ago

Ore refinery not applying bonus production to the titanium and hexanite. Been thinking for a while, but a little tricky to know what are "metallic" resources. Screenshot_20220605-122730_UnCiv I think the mining tunnels work but not 100% sure, havent explicitly checked it. Screenshot_20220605-123110_UnCiv

carriontrooper commented 2 years ago

Hmm, this might be something to take up to vanilla unciv, seems like using custom uniques to filter tiles doesn't work in applying them.

carriontrooper commented 2 years ago

4.1.9 supposedly fixes this, test again plz

Zoogzin commented 2 years ago

May take a couple of days to test this particular one, will get back to you soon

Zoogzin commented 2 years ago

Buildings appear to work now (greenhouse, mining tunnels, synthetic recycling station, resource market, ore refinery). Havent tested on all applicable resources, but definitely works on some (confirmed for gold ore, titanium, promethine, vitamass, exotic fruit). In theory, could test all buildings that specify artifacts (field lab, research laboratory, museum of the frontier), cloud seeder station, and soma biolab and other unique buildings. Cant promise i'll test that extensively, especially since the bugs i ran into have all been fixed. Feel free to close if you like.

Noticed that my gold ore hill mine got a +2 prod from mining tunnels, for some reason i thought mining tunnels only gave max +1 for mines on hills, or hills, or mines on non-hill terrain.

Excited to see all the new icon assets you've added!

carriontrooper commented 2 years ago

Alrighty, seems like this one is solved.

Zoogzin commented 2 years ago

Sorry, but i just noticed greenhouse didnt apply food bonus to chitin. Out of time to keep testing other outcomes at the moment, will keep in touch a little over the next week.

carriontrooper commented 2 years ago

Greenhouse is flora bonus resources only, Chitin isn't flora.

Zoogzin commented 2 years ago

Oh, okay. The description just says "bonus resources", doesn't mention "flora," may help others to clarify, though i'll remember either way, of course. Thanks!