Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature] Cotton harvesting in rain #2801

Closed
WeAreV3N0M opened this issue Sep 24, 2023 · 3 comments
Closed

[Feature] Cotton harvesting in rain #2801

WeAreV3N0M opened this issue Sep 24, 2023 · 3 comments
Assignees
@schwiti6190
Labels
enhancement New feature or request low priority more a nice to have then rly needed
Milestone
7.3.1.3

Comments

@WeAreV3N0M
Copy link
Contributor

What is the problem this feature/enhancement solves?
I just ran into a problem while a cotton harvesting contract was running and it started to rain. When I switched back to the cotton harvester (John Deere CP690 - base game machine) sometime later, I saw a warning about harvesting in rain might result reduced yield. That is exactly what happened and it made the contract fail.

Describe the solution you'd like
My suggestion is to modify the behavior of cotton harvesters controlled by CP, to make them stop during rainy weather, similar to regular harvesters.

Game mode: Single player
CP version: 7.3.1.0
Map: Elm Creek Edit 4x V1.0.0.9 by Stevie
@WeAreV3N0M WeAreV3N0M added the enhancement New feature or request label Sep 24, 2023
@Tensuko Tensuko added the low priority more a nice to have then rly needed label Sep 24, 2023
@Tensuko
Copy link
Contributor

Tensuko commented Sep 25, 2023

The Problem is that the cotton harvester is not completly a harvester, but also a baler.
Thats could be a bit tricky...

@WeAreV3N0M
Copy link
Contributor Author

WeAreV3N0M commented Sep 25, 2023 •
edited
Loading

What if the change is based on the machine's output? - (cotton bale)

@Tensuko
Copy link
Contributor

Tensuko commented Sep 25, 2023

That doesnt work like that, its about the specs and how we have to handle them so they work correctly, wich was a struggle with the cotton harvesters.

schwiti6190 added a commit that referenced this issue Oct 9, 2023
@schwiti6190
Fix for #2801 stops cotton harvesters during rain
5a879bd 
Cotton harvester have the combine spec but don't use the combine strategy.
To use the combine controller, it has to be initialized in the fieldwork strategy.
@Tensuko Tensuko added this to the 7.3.1.3 milestone Oct 10, 2023
Tensuko added a commit that referenced this issue Oct 10, 2023
@Tensuko
Merge pull request #2840 from Courseplay/Issue-2801
3aff6aa 
Fix for #2801 stops cotton harvesters during rain
@Tensuko Tensuko closed this as completed Oct 14, 2023
Tensuko added a commit that referenced this issue Oct 18, 2023
@Tensuko
Push Release 7.3.1.3
9c06246 
- Changed description for combine unload on 1st headland to be more clear, also set the default setting to false.
- fix for #2592
- fix for reversing turn.
- Pipe won't open for a CP combine unloader that is in idle state.
- Removed Giants collision that prevented our unloader to react to our combine.
- fix for #2812
- fix for #2801
- fuel info now is shown when AI driver is stopped because of low fuel for all setting states by @tn4799
- fix for the info messages not always clearing correctly on MP
- Adjusted silo end detection with shovel loading. #2783
- Added columns to silo map, for selection of the optimal silo part to load.
- Fixes bug for bunker silo driver, where driving to park position failed. #2783
- Added leveler height offset to the ground setting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schwiti6190 schwiti6190

Labels
enhancement New feature or request low priority more a nice to have then rly needed
Projects
None yet
Milestone
7.3.1.3
Development

No branches or pull requests
3 participants
@Tensuko @WeAreV3N0M @schwiti6190