Skip to content

RoCELib Refactoring#2

Open
shlokcshah55 wants to merge 192 commits intoRobustCounterfactualX:mainfrom
JStadl:main
Open

RoCELib Refactoring#2
shlokcshah55 wants to merge 192 commits intoRobustCounterfactualX:mainfrom
JStadl:main

Conversation

@shlokcshah55
Copy link

@shlokcshah55 shlokcshah55 commented Jul 8, 2025

Main contributions of:

  • Visualisation + Exportation of evaluation metrics
  • Refactoring of evaluation metrics into extensible format with a tree like structure
  • Refactoring of models to support both trained and trainable models
  • Introducing concept of a Task and builder counter part to easily instantiate a model dataset pair
  • CI/CD setup
  • Documentation
  • Packaging (allowing pip install RoCELib)

See attached presentation and report for more details:
slides.pdf
segp-report.pdf

SethMack and others added 30 commits January 17, 2025 15:58
@SethMack
Fix: removed tensorflow-intel from requirements
ac9a09b
@SethMack
Feat: added skeleton files for RoCourseNet
ca9b656
@JStadl
Create python-build-test.yml
b31cda9
@SpyroL7
[le522, jws22] fix: changed directory pytest runs in
7744a7f
@SpyroL7
[le522, jws22] Feat: added multicore pytesting
2b3a8a9
@SpyroL7
[le522, jws22] Fix: stops pipeline running twice after PR commit
9b504d7
@SethMack
(svm122 scs22) Feat: implementation of RoCourseNet simple test and ba…
676f612 
…sic implementation of generation_method algorithm
@JStadl
[jws22, le522] Feat: Added coverage output and caching of dependencies
23518c7
@SpyroL7
Merge pull request #1 from JStadl/CICD
62cfdfb 
Create a build and test and deploy documentation pipeline
@SethMack
(svm122) Fix: RoCourseNet code runs but does not pass test
733d615
@SethMack
(svm122) Fix: fixed RoCourseNet test to test validity rather than rob…
f2ad926 
…ustness
@SethMack
(svm122) Fix: fixed implemenation of rocoursenet test
d019a8e
@SethMack
Merge pull request #2 from JStadl/RoCourseNet
c392310 
Ro course net
@ayushpatel4
feat: created TrainedModel abstract class and added functionality to …
6596239 
…import a pytorch model
@SethMack
(svm122) Feat: initial names refactoring
12b506e
@SethMack
(svm122) Fix: ensured a trained model is passed into a task. Can no l…
ea418ce 
…onger train a task
@SethMack
(svm122) svm122: train() methods returns a trained method
63452a2
@SpyroL7
[le522] Fix: made PytorchModel extend TrainedModel
2f3e081
@SpyroL7
[le522] Feat: initial implementation of KerasModel
701e172
@SpyroL7
[le522] Fix: updated return types for evaluate method
35a50f5
@SpyroL7
[le522] Fix: consolidated predict_proba into multimethods so that Pyt…
a9b780f 
…orchModel matches TrainedModel template
@artvanya
[ia1022] Feat: implemented SKLearnModel
623e0ae
@artvanya
[ia1022]: changes to the SKLearn and tests for it
97189be
@ayushpatel4
fix: implemented predict_proba for tensors and dataframes in PytorchM…
b071520 
…odel
@SethMack
(svm122) Feat: Tasks now hold a trained model and train() functions r…
41cf2c0 
…eturn a trained model. Currently broken
@ayushpatel4
fix: ensure predict_proba returns the probabilities
17920a4
@SpyroL7
[le522] Feat: added tests for KerasModel which all pass
d7ba12a
@SethMack
Merge with import-models. Not using new proba methods or TrainedModel
5a00538
@SethMack
(svm122) Fix: added fixes for testing
d558fc6
@SpyroL7
[le522] Feat: added descriptive exceptions when incorrectly importing…
96dd67b 
… to KerasModel
JStadl and others added 30 commits March 2, 2025 17:06
@JStadl
jws22 Fix: fix bad imports failing tests
edb3bcf
@JStadl
jws22 fix: fix imports for test fixture
c028210
@SethMack
(svm122) Fix: fixed delta robustness evaluator. Instance was being ev…
335c15f 
…aluated rather than the actual counterfactual
@ayushpatel4
(ap2322) fix: ensure MMMILP counterfactuals have correct column names
dade773
@ayushpatel4
(ap2322) fix: ensure MMMILP counterfactuals have correct column names
c87c430
@ayushpatel4
(ap2322) fix: BinaryLinearSearch working and proper handling of evalu…
68b239b 
…ating mm metrics when you have not generated for mm
@JStadl
jws22 fix: Moved test helpers to own directory to fix import errors
6615091
@JStadl
jws22 fix: change pyproject.toml to reflect package structure
da0d98e
@JStadl
jws22 fix: change yml to fix relative import issue
13e70ac
@ayushpatel4
(ap2322) feat: added testing for APAS
14084fd
@JStadl
jws22 Fix: package installation in pipeline
4c7c7f6
@JStadl
jws22 fix: csv files for tests
9bc38ee
@JStadl
merge packaging with main
01fa663
@JStadl
Merge pull request #8 from JStadl/packaging
5adf37e 
Packaging
@ayushpatel4
(ap2322) feat: creation of output package
f720443
@ayushpatel4
(ap2322) feat: creation of output package
0216d50
@JStadl
jws22 fix: changed versioning to be automatic on tag push
e1fbac3
@JStadl
jws22 fix: add version dynamic tag to pyproject to allow build
82395bf
@JStadl
jws22 fix: test importing structure
ce24fe6
@ayushpatel4
(ap2322) fix: make generate_mm use pass a ClassificationTask to the r…
9b107c6 
…ecourse method rather than Task
@ayushpatel4
(ap2322) fix: add mm_models to generate_mm
a4ab632
@JStadl
Merge pull request #9 from JStadl/packaging
d26121f 
Packaging
@SethMack
(svm122) Fix: removed redundant files being pushed to main. Fixed pyt…
00ae352 
…hon version bug.
@JStadl
jws22 feat: updated readme with insructions to install
2dd975e
@JStadl
Merge branch 'main' of https://github.com/JStadl/RoCELib
bbf088c
@ayushpatel4
(ap2322) fix: mmmilp now works for keras and sklearn models
f4536e9
@ayushpatel4
(ap2322) fix: mmmilp now works for keras and sklearn models
2e2d573
@JStadl
jws22 Feat: Updated example notebooks
51b8f2d
@JStadl
jws22 Feat: Updated readme to point to notebooks
f9c5b59
@JStadl
jws22 Feat: added command to install package in example notebook
4dc9f74
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@shlokcshah55 @SethMack @JStadl @SpyroL7 @ayushpatel4 @artvanya