pm4py / pm4py-core

Public repository for the PM4Py (Process Mining for Python) project.
https://pm4py.fit.fraunhofer.de
GNU Affero General Public License v3.0
741 stars 287 forks source link

Stochastic playout fix #477

Closed pastra98 closed 8 months ago

pastra98 commented 8 months ago

Hello, thank you for making this wonderful library! I was trying to do the stochastic playout based off a smap, but ran into two issues:

  1. pm4py/algo/simulation/playout/petri_net/variants/stochastic_playout.py the STOCHASTIC_MAP param unrolls to "stochastic_map", but the parameter expected by sim.play_out() is 'smap' and thus any stochastic map passed to play_out gets lost. This should be fixed by the first commit
  2. pm4py/objects/petri_net/utils/performance_map.py expects a legacy log, but does not enforce or check this anywhere, so I added a conversion with the second commit

Thank you for considering these changes - I wasn't able to select a branch other than release, let me know if I should do anything else.