Topic: Enhancement / Upgrade buildings via lua-code.
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 20:01
How far is the program code to force a building upgrade in Lua using a command? I submitted the proposal to Github a few weeks ago. -> https://github.com/widelands/widelands/issues/4931 I need the code to fix a bug in the Europeans. The AI does not upgrade the training sites on its own, so I will force it to do so using a timer in the starting conditions. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |
Nordfriese |
Posted at: 2021-07-07, 20:13
If nothing's changed in the issue yet then nothing has happened so far. We are very short of developers at the moment. However I had labelled it Good First Issue because this is an issue that can be worked on with minimal knowledge of C++ and the codebase required. Most of the implementation would be duplicating the function
Won't this also affect human players? Top Quote |
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 20:19
If there's a way to evaluate if a player is an AI, it don't. Then the script can simply switch off this timer in the starting condition. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 20:21
I have rudimentary knowledge of C ++, it is enough to understand your code to some extent. Maybe I should get down to it and do something about it. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |
hessenfarmer |
Posted at: 2021-07-07, 20:28
Well, the expectation that every filed issue is solved within weeks is simply not right. there are much mor even older issues that aren't solved yet, due to the lack of somebody wanting to work on this. Shouting at the developers won't help the motivation to do so.
Exactly. You need it. But the rest of the game or the community does not necessarily need it. Top Quote |
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 20:30
I dont shouting at the developers, its only a question or reminder. I've create a piece of code and will make an request. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |
hessenfarmer |
Posted at: 2021-07-07, 20:36
Ok, Sorry for the misunderstanding then. Top Quote |
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 20:59
I've posted the code snippets on Github. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |
hessenfarmer |
Posted at: 2021-07-07, 21:05
that is suboptimal in terms of workload. Please clone the repository, create a branch with your changes, upload it ither to the master repository or your forked repository and create a PR to merge the changes into the main trunk. Top Quote |
MarkMcWire Topic Opener |
Posted at: 2021-07-07, 21:18
I will test it und if it works, I will make it so. My widelands addons: https://drive.google.com/file/d/1jopANlODo41T2reHJ0zaCOMYxq_rxXP-/view?usp=sharing Top Quote |