The International Simutrans Forum

 

Author Topic: New debug sums to monitor car/pedestrian randomness  (Read 740 times)

0 Members and 1 Guest are viewing this topic.

Offline freddyhayward

  • Devotee
  • *
  • Posts: 693
  • Languages: EN
New debug sums to monitor car/pedestrian randomness
« on: September 18, 2021, 10:40:29 AM »
The purpose of this pull request is to help track down frequent desyncs occurring at some point during the sync_step. My hypothesis is that there is a discrepancy in the random behaviour of cars or pedestrians. If this is correct, then we would observe differences in these new sums which would help to further isolate the problem. If not, another approach will have to be found.

These sums replace the existing sums at [8] & [9] counting the number of sync objects, which didn't seem to correspond to any recent desyncs.

The pull request can be found here: https://github.com/jamespetts/simutrans-extended/pull/445

Offline jamespetts

  • Simutrans-Extended project coordinator
  • Administrator
  • *
  • Posts: 21016
  • Cake baker
    • Bridgewater-Brunel
  • Languages: EN
Re: New debug sums to monitor car/pedestrian randomness
« Reply #1 on: September 18, 2021, 10:45:24 AM »
Excellent, now incorporated. Thank you for your work on this.

Offline freddyhayward

  • Devotee
  • *
  • Posts: 693
  • Languages: EN
Re: New debug sums to monitor car/pedestrian randomness
« Reply #2 on: September 18, 2021, 10:59:01 AM »
Excellent, now incorporated. Thank you for your work on this.
It doesn't seem to have been incorporated yet. Was this intentional?

Offline jamespetts

  • Simutrans-Extended project coordinator
  • Administrator
  • *
  • Posts: 21016
  • Cake baker
    • Bridgewater-Brunel
  • Languages: EN
Re: New debug sums to monitor car/pedestrian randomness
« Reply #3 on: September 18, 2021, 11:00:36 AM »
It doesn't seem to have been incorporated yet. Was this intentional?

Oops - forgot to push. Now done.

Online prissi

  • Developer
  • Administrator
  • *
  • Posts: 11056
  • Languages: De,EN,JP
Re: New debug sums to monitor car/pedestrian randomness
« Reply #4 on: September 18, 2021, 12:48:04 PM »
If it is as standard, the eye-candy like clouds and pedestrians should not affect the random state.

Offline freddyhayward

  • Devotee
  • *
  • Posts: 693
  • Languages: EN
Re: New debug sums to monitor car/pedestrian randomness
« Reply #5 on: September 19, 2021, 07:41:45 AM »
If it is as standard, the eye-candy like clouds and pedestrians should not affect the random state.
I don't know why, but extended pedestrians do affect the random state.