Skip to content

Reproducibility and TCAV Improvements#2

Open
h1yung wants to merge 3 commits into
unitn-sml:mainfrom
h1yung:main
Open

Reproducibility and TCAV Improvements#2
h1yung wants to merge 3 commits into
unitn-sml:mainfrom
h1yung:main

Conversation

@h1yung
Copy link
Copy Markdown

@h1yung h1yung commented Mar 4, 2026

  • TCAV extraction fix (batch + shuffle safe)
  • MNIST concept generator script
  • README clarification on NN/CLIP evaluation pipeline

h1yung added 3 commits March 4, 2026 06:42
- assertion errors when true_concepts don’t match tmp_concepts (due to dataloader shuffling)
- batch sizes > 1 cause shape mismatches
The script generates concept folders for the SHORTMNIST dataset by extracting individual digits from 2-digit images and saving them in corresponding directories.
Added instructions for evaluating NN/CLIP models using TCAV and analysis notebooks.
@samuelebortolotti
Copy link
Copy Markdown
Collaborator

Thank you for your interest in and contribution to rsbench!

Before we can merge this pull request, could you please:

  • Remove the hardcoded local path (e.g., /home/...) and replace it with a more portable solution (e.g., relative paths or config-based handling) in tcav/main.py
  • Clean up the large commented configuration block to keep the codebase tidy

Once these are addressed, we will be happy to merge your contribution.

We are now planning to add more models and dataset to this repository in the future.

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