Skip to content

Conversation

@PatrikLundell
Copy link
Contributor

@PatrikLundell PatrikLundell commented Oct 3, 2025

Summary

None

Purpose of change

Partiall address #83161 by allowing harvesters to remove overgrown crops, while still not dealing with harvestable and overgrown crops not being traversible by vehicles.

Describe the solution

Modify the harvesting code to process overgrown crops as well, to clear the tile but producing no yields.

Describe alternatives you've considered

Deal with the steel plants as well. The harvester vehicle in the game is not compatible with the fields in the game when steel plants are around:

  • Fields are planted with double rows, while harvesters process 3 single rows at a time.
  • Fields are planted in a real world fashion with curving rows. Game vehicles can't move like that.
    This means game harvesters will crash into game crops on fields and gradually get destroyed unless they're rebuilt by the player before use.

Harvest straw or withered plants as some kind of generic result instead of nothing.

Testing

Teleported to overgrown field, debug spawned a reaper tractor, engaged the reaper, and tried to reap the overgrown crops at the edge of the field (so the vehicle wouldn't smash into any crops). Verified nothing happens to the crops.
Modified the code.
Repeated the above and saw the overgrown crops disappeared, with nothing harvested.

Additional context

@github-actions github-actions bot added Vehicles Vehicles, parts, mechanics & interactions [C++] Changes (can be) made in C++. Previously named `Code` astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Oct 3, 2025
@PatrikLundell PatrikLundell marked this pull request as draft October 4, 2025 08:23
@PatrikLundell PatrikLundell marked this pull request as ready for review October 4, 2025 08:23
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Oct 4, 2025
@Maleclypse Maleclypse merged commit 6a0ba10 into CleverRaven:master Oct 4, 2025
40 of 51 checks passed
@PatrikLundell PatrikLundell deleted the remove_overgrowth branch October 5, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions Vehicles Vehicles, parts, mechanics & interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants