X Tutup
Skip to content

Balancing the Scout mission ads#6095

Merged
zonkmachine merged 1 commit intopioneerspacesim:masterfrom
zonkmachine:scoutads
Apr 4, 2025
Merged

Balancing the Scout mission ads#6095
zonkmachine merged 1 commit intopioneerspacesim:masterfrom
zonkmachine:scoutads

Conversation

@zonkmachine
Copy link
Member

@zonkmachine zonkmachine commented Mar 22, 2025

In response to #5760 where the BBS over time lose balance as some modules take over. Especially the Scout module keeps adding missions in excess of the demand. This is a quick fix to soften the issue and with this PR the missions balance out at around ~30 missions per station if none is take by the player.

The fix works by creating fewer adverts for Scout missions. One sixth of previous rate.

I haven't tested this other than sitting at Cydonia for 9 months and watching the mission list build up.
Here is what it looks like on master when the missions build up while not doing anything.

Number of ads per module after...
                     ~2 months            ~4 months
------------------------------------------------------
Assassination               10                  5
Cargo run                   21                 32
Scoop                       ~1                  1
Scout                      119 !!!            174 ???
Combat                       6                  4
Deliver Package             24                 24
Search and rescue            3                  5
Second hand                  2                  1
Taxi                        34                 38

Copy link
Member

@impaktor impaktor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still like my way of doing it: #3243
Perhaps over-complicated.
To each his own.

@zonkmachine
Copy link
Member Author

I still like my way of doing it: #3243 Perhaps over-complicated. To each his own.

I love your way of doing it. This is just me getting stressed up when it's time for release and I see a random bug. It looked easy and safe enough to make it into the next release. It's quite an annoying bug. As for the method, I think your way of doing it is super nice and I think this should be revisited for a proper makeover.

@zonkmachine zonkmachine marked this pull request as draft March 24, 2025 12:47
@zonkmachine
Copy link
Member Author

Marked as a draft as I found some issues. Will look into this in the evening.

@zonkmachine
Copy link
Member Author

zonkmachine commented Mar 26, 2025

Marked as a draft as I found some issues. Will look into this in the evening.

I misread the code for ad removal in onUpdateBB and have reverted that part of the PR but instead made some changes for improved clarity.

I also softened the change in adverts created just slightly. This is now just a one-liner which cuts the number of adverts created by the Scout module from about 2 per day to 1 every 3rd day.

@zonkmachine zonkmachine marked this pull request as ready for review March 26, 2025 02:32
@zonkmachine zonkmachine force-pushed the scoutads branch 3 times, most recently from f266117 to 926ee94 Compare March 29, 2025 17:28
@zonkmachine
Copy link
Member Author

Did a last push to make the code more clear. Squashed and done.
Merge?

@zonkmachine
Copy link
Member Author

Can I merge this?

The only change I would do personally is do an even more drastic cut in Scout missions but I think maybe it's best to gradually dial in the right level. There is also the prospect of having a more advanced algorithm that takes population and other system data into consideration.

@impaktor
Copy link
Member

impaktor commented Apr 4, 2025

Can I merge this?

Fine by me

The Scout module creates on average two new ads every day. If the
player sits on or around the same station for a month or more this
will eventually build up to over a hundred ads. This PR slows down
the creation of new ads by one sixth to about one new ad every three
days.

In response to pioneerspacesim#5760
@zonkmachine zonkmachine merged commit dd7d40d into pioneerspacesim:master Apr 4, 2025
@zonkmachine zonkmachine deleted the scoutads branch April 4, 2025 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup