Skip to content

2292: Adding Maximum AtB Contract Generation Attempts Option #2987

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.

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

Closed
wants to merge 5 commits into from

Conversation

Windchild292
Copy link
Contributor

This fixes #2292.

@Windchild292 Windchild292 added the (RFE) Enhancement Requests for Enhancement, new features or implementations label Dec 2, 2021
@Windchild292 Windchild292 self-assigned this Dec 2, 2021
@NickAragua
Copy link
Member

What possible value could this have to a user? There's basically no situation in which a contract generation will fail on one try but succeed on the next.

@Windchild292
Copy link
Contributor Author

I've experienced multiple failures for valid reasons (mostly relating to being unable to find a valid planet) since my original ask for this option, so it definitely isn't a no situation thing. I added it because I want it as an option, one that I will be immediately increasing to 10 retries.

I'd prefer to add a config file like EU4's defines.lua files, but this works until I have the time to create that setup.

@NickAragua
Copy link
Member

NickAragua commented Dec 3, 2021

Looking at the code, it's theoretically possible that the subset of systems being searched changes between location generation attempts. However, it's still as likely that this will fail on the 10th retry as it does on the 2nd or 3rd - sometimes, you're just not going to be able to generate a contract location no matter what.

A better solution would be to keep a list of factions for which contract location generation has failed and exclude those from the "random faction picker" logic. When you run out of factions, just go to "MERC" or "PIRATE". That way, you're guaranteed to generate a contract.

@SJuliez
Copy link
Member

SJuliez commented Apr 6, 2024

I feel that with the merge conflicts, the time gone and the general disagreement if this is useful at all, this isn't going to make it in at any point. Therefore closing it.

@SJuliez SJuliez closed this Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(RFE) Enhancement Requests for Enhancement, new features or implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maximum AtB Contract generation attempts should be a campaign option
3 participants