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.

Sign up for GitHub

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

Jump to bottom

Adding support for future features including: explanation object, ensembles, new metrics and pipelines #31

Merged
merged 71 commits into from
Jul 8, 2024
Merged

Adding support for future features including: explanation object, ensembles, new metrics and pipelines #31

merged 71 commits into from
Jul 8, 2024

Conversation

MarioTheOne
Copy link
Contributor

No description provided.

MarioTheOne and others added 30 commits April 23, 2024 15:58
@MarioTheOne
added new configs
ff4d85a
@MarioTheOne
Merge branch 'main' of https://github.com/aiim-research/GRETEL into s…
acbcc4d 
…ilver
@bardhprenkaj
launchers
c10aa8c
@MarioTheOne
Added Aggregators and Updated Evaluator
29e656c
@MarioTheOne
Merge branch 'silver' of https://github.com/aiim-research/GRETEL into…
172c089 
… silver
@MarioTheOne
added bidirectional aggregator
b815658
@MarioTheOne
logging added to bidirectional aggregator
654c873
@bardhprenkaj
fixed issue on Random Explanation Aggregator
22b97a4
@bardhprenkaj
The explainers return an Explanation object
a4899e6
@bardhprenkaj
Explanation classes updated
8a9fa74
@bardhprenkaj
notebook lab updated
4a37453
@bardhprenkaj
updated ensemble base to work with explanation class
18fa164
@bardhprenkaj
Aggregators TopSelect and Union now use the Explanation object
51250c9
@bardhprenkaj
Intersection Aggregator Updated
746b1f4
@bardhprenkaj
Frequency and Random Explanation Aggregators Updated
05a3268
@bardhprenkaj
Updated Bidirectional Explanation Aggregator
9da9e93
@bardhprenkaj
Updated ExplanationStochasticAggregator
f6993db
@bardhprenkaj
Added the ExplanationFilter class for the ExplanationAggregator
02e8df1
@bardhprenkaj
Started Updating Metrics
8bebb31
@bardhprenkaj
First batch of configurable metrics created
a752ea1
@bardhprenkaj
Metrics Updated
e879e92
@bardhprenkaj
Added New Evaluator Manager
f354bd6
@bardhprenkaj
New Evaluator and Evaluator Manager for the new metrics
f2359a7
@bardhprenkaj
New DataAnalyzer Updated to work with the new metric implementation
f9a4909
@bardhprenkaj
Instability metric added
f9281dd
@bardhprenkaj
Lab example updated and Stages classes added
60510b8
@leynier
feat: add empty explanation multi criteria aggregator class
d8e82e0
@leynier
feat: implement base criteria class
7fae031
@leynier
feat: implement the base distance class
57d7ca6
@leynier
feat: implement the ged criteria
63aee27
bardhprenkaj and others added 20 commits June 27, 2024 19:27
@bardhprenkaj
Added Pipeline and PipelineManager
356b618
@leynier
fix: make the ged logic in utils a function with only necessary argum…
0833018 
…ents
@leynier
fix: use graph_edit_distance_metric in old GraphEditDistanceMetric class
5c545c6
@leynier
fix: implement a sparsity function and use in all places to avoid dry
a84ab4a
@leynier
feat: implement sparsity criteria for multi criteria emsemble
dc60b3d
@leynier
feat: add class versions for metrics to allow load in configuration
a112ce2
@leynier
chore: add sparcity criteria to mutli criteria test config
4541936
@leynier
feat: add oracle, explainer and dataset to criteria evaluate method
3e1ed49
@leynier
feat: add fidelity criteria and refactoring fidelity metrics implemen…
2411ec7 
…tations
@bardhprenkaj
pipeline
b443a3a
@bardhprenkaj
base_explainers snippet
b62ef91
@leynier
Merge branch 'silver' into multi-criteria-emsembler
2ba47c1
@leynier
chore: save config files
0dc3d37
@bardhprenkaj
Improved Experiments Organization
26adbc2
@leynier
Merge branch 'silver' into multi-criteria-emsembler
d7489a3
@leynier
chore: revert changes
ce323e4
@leynier
chore: revert changes
149084c
@MarioTheOne
Merge pull request #28 from leynier/multi-criteria-emsembler
66d420c 
multi criteria emsembler
@MarioTheOne
Merge branch 'silver' into merge-main-silver
6b3c68a
@MarioTheOne
Merge pull request #30 from aiim-research/merge-main-silver
de07631 
Merging Main into Silver
@MarioTheOne MarioTheOne self-assigned this Jul 8, 2024
leynier
leynier reviewed Jul 8, 2024
View reviewed changes
main.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/dataset/manipulators/rank.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/evaluation/evaluation_metric_base.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/evaluation/evaluation_metric_correctness.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/evaluation/evaluation_metric_fidelity.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/explainer/heuristic/obs.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/explainer/search/dces.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/explainer/search/i_rand.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/explainer/search/maccs.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

src/explainer/search/p_rand.py Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes

@leynier leynier marked this pull request as ready for review July 8, 2024 16:13
@MarioTheOne MarioTheOne changed the title Fake Silver Adding support for future features including: explanation object, ensembles, new metrics and pipelines Jul 8, 2024
@MarioTheOne MarioTheOne merged commit 275e622 into main Jul 8, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leynier leynier leynier left review comments

Assignees

@MarioTheOne MarioTheOne

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MarioTheOne @leynier @bardhprenkaj