Skip to content
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

The module explainer #60

Open
wants to merge 67 commits into
base: main
Choose a base branch
from
Open

Conversation

MarcusRostSAP
Copy link
Contributor

This is the code from my thesis topic, where I introduce an Explainer module into bpm2constraints.

There are also some changes to the xml parser, made previously from my fork

MarcusRost and others added 30 commits October 8, 2023 13:52
* added xml counterparts for every json file

* Changed test constants to include xmlpath

* Add tests for xml code

* added some test utils, but xml seems broken

* some parser configs

* Reformat tests

* Removed unecessary print statement

* Included more variants of XOR gateways

* Removed file

* reformatting

* Reformatting according to linter

* Linting

* Linting

---------

Co-authored-by: MarcusRostSAP <[email protected]>
* Initial commit explainer module

* Some experimentation

* Tried a new heurstic

* Some code clean up

* Slight change in logic

* added activation feature

* Made some changes to the activation and logic

* Updated notebook

* some small logic changes

* Notebook compile

* added shapley_values

* Progress on shapely for traces

* Added comments

* Linting issues

* Linting

* Linting

* more linting

* More linting

* Linting

* Linting

* linter

* Linter

* Linting
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