search

Azurency / CQUI_Community-Edition

Civilization 6 mod - UI enhancements, reduce clicks and manage your empire faster!
MIT License
399 stars 98 forks source link

[Integration] Builder Lens doesn't support Hotseat Multiplayer #390

Open ubereffect opened 5 years ago

ubereffect commented 5 years ago

How to test: Start a hotseat multiplayer game, setting the first 3 players as human (first 2 humans in a team, 3rd one solo), and the last player(s) occupied by computer(s). A duel sized pangaea probably works best for this.

What happens: The Builder Lens only works as expected for player 1. As for players 2 & 3, the Builder Lens darkens all their owned tiles. Interestingly, when player 2 uses the Builder lens, it highlights improvable tiles owned by player 1 (as they have shared visibility), but does not highlight it's own improvable tiles.

If you start a fresh game, it only works for player 1. But if you then save it when it's another players turn, and reload the game from that point, it will only work for that player going forward (and won't work for player 1).

I should also note that I played a number of hotseat multiplayer games with CQUI enabled prior to Gathering Storm, and this issue was not present until this recent update.

Azurency commented 5 years ago

Maybe you should also open an issue in https://github.com/astog/MoreLenses as he is the creator of the More Lenses mod that add this builder lens behaviour.

ubereffect commented 5 years ago

Thanks

He (astog/decoiff) acknowledged the request for hotseat support in the More Lenses Steam Workshop comment section. He mentioned being busy with grad school, but plans to add both hotseat support and better integration with map tacks.

You guys are awesome. I love your mods. They really make for a more enjoyable game.

cyrilh commented 4 years ago

Hi, MoreLenses has fixed this bug, could you upgrade as well?

Note: I tried subscribing to MoreLenses in parallel to CQUI but it seems it has completly broken at least the builder automatic lense... I could not even find bulder in the list of lenses on the UI.

Enjoy, Cyril

cyrilh commented 4 years ago

Hi,

I manually applied the commit below to the file below (manually with text editor, did not try git-apply), and it solved my problem! MoreLenses commit ML\Lenses\ModLens_Builder_Config_Default.lua

Following the discussion on #405, I propose as a first and trivial thing to only apply that commit above... while #405 could wait for someone with skills and time to handle it.

Enjoy, Cyril