kbss-cvut / bpmn2stamp

1 stars 0 forks source link

Make dozor-nad-provozovateli-letist transformation idempotent #29

Closed blcham closed 2 years ago

blcham commented 2 years ago

Try to make the transformation of BPMN diagrams dozor-nad-provozovateli-letist as idempotent as possible, if some parts are not possible to transform, let's discuss it here.

Related issue: #8

To test the transformation do the following:

git clone https://github.com/kbss-cvut/bpmn2stamp
git clone https://github.com/kbss-cvut/bpmn-convertor-examples
cd bpmn-convertor-examples/processes/ucl
./bin/extract-bos.sh bos/dozor-nad-provozovateli-letist.bos
../../bin/build-bpmn2stamp.sh
./bin/process-bpmn.sh bpmn/dozor-nad-provozovateli-letist.bpmn
git diff dozor-nad-provozovateli-letist-bpmn.ttl

With these steps, you should get the following output of git diff dozor-nad-provozovateli-letist-bpmn.ttl: image

A/C:

grigobog commented 2 years ago

Now ids of such parts are generated from pool Id, action id (related to the boundary event) and one of the event property:

This will make the convertion idempotent for such parts, described in #8, #23 and this issues.

blcham commented 2 years ago

After merging, I will test with my scripts (or you can do it in the pull request).

grigobog commented 2 years ago

Need to change delimiter from "_" to "-", see comment in the pull request.