Topic: Permanent Resource Indicators
tothxa
Topic Opener
Joined: 2021-03-24, 12:44
Posts: 460 OS: antix / Debian Version: some new PR I'm testing  Tribe Member |
Posted at: 2021-11-25, 00:38
@littlekatana re https://www.widelands.org/forum/topic/5458 and https://www.widelands.org/forum/topic/5467 I created an addon to make the resource indicators (geologist sign posts) stay around until a matching mine removes them. Unfortunately a bug prevents it from working. A fix is available in the #5141 pull request. I've only tested the posted version with the Barbarians so far. I will only upload it to the addons server after the bug is fixed in master. I may also try first to add some more functionality for addons to use, to allow only changing what is actually necessary instead of duplicating so much of the official unit definitions. 
Top
Quote
|
|
tothxa
Topic Opener
Joined: 2021-03-24, 12:44
Posts: 460 OS: antix / Debian Version: some new PR I'm testing Tribe Member |
Posted at: 2021-11-27, 23:01
The PR was merged and I uploaded the addon. Unfortunately Nordfriese found a fundamental problem that I missed: When no resource indicators are present in the workarea of a mine, then the mine will falsely report out of resources and the worker will get no experience, even though the mine will produce materials normally. This is because I implemented the removal by abusing a feature in a way it wasn't meant to be used in, so it will be hard to fix it. It will require either a complete rewrite of the add-on in a much more complicated way, or some bigger change to the main program. I aplogise for the rushed announcement.
Top
Quote
|
|
tothxa
Topic Opener
Joined: 2021-03-24, 12:44
Posts: 460 OS: antix / Debian Version: some new PR I'm testing Tribe Member |
Posted at: 2021-11-28, 20:10
Big thank you to Nordfriese who already made the necessary changes in the game code that will make this add-on work. I attach the updated version of the add-on. I'll upload it to the server when Nordfriese's changes are merged in master.
Top
Quote
|
|
tothxa
Topic Opener
Joined: 2021-03-24, 12:44
Posts: 460 OS: antix / Debian Version: some new PR I'm testing Tribe Member |
Posted at: 2021-12-01, 01:05
The required changes are now merged in the development version with commit 2c4e78b so I uploaded version 0.4 of this add-on. It should work now as intended.
Top
Quote
|
|
tothxa
Topic Opener
Joined: 2021-03-24, 12:44
Posts: 460 OS: antix / Debian Version: some new PR I'm testing Tribe Member |
Posted at: 2021-12-05, 22:31
With today's update, the development version on flathub-beta also supports this add-on.
Top
Quote
|
|
simplypeachy
Joined: 2009-04-23, 12:42
Posts: 155  At home in WL-forums Location: Hampshire, UK |
Posted at: 2024-10-13, 16:41
I hadn't noticed this add-on exists, what a great idea! I had a quick multiplayer test and couldn't get it to desync. Do you know if desyncing is still a known problem? WARNING: New-style view packet not found. There may be strange effects regarding unseen areas.
Top
Quote
|