-
Notifications
You must be signed in to change notification settings - Fork 9
Initial checks cosmetic edits #6
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
Open
liranc6
wants to merge
14
commits into
OPTML-Group:main
Choose a base branch
from
liranc6:initial-checks-cosmetic-edits
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+26,529
−188
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… fast eval. files changed: eval.py, metrics/knowmem.py, metrics/privleak.py, metrics/verbmem.py
The primary purpose is to improve evaluation robustness and flexibility when managing model outputs and debug workflows. The primary changes are: - Updated `eval_model` to ensure `forget_data`, `retain_data`, and `holdout_data` are initialized consistently before use. - Replaced hardcoded paths with `os.path.join` using `MUSE_DIR` in `eval_model` for improved path handling. - Added a `kwargs` parameter to both `eval_model` and `load_then_eval_models` to support dynamic control over file creation and loading. - Implemented conditional logic in `eval_model` for managing `privleak` file generation based on `kwargs['create_new_files']`. - Removed unused imports and dynamic import logic from `eval.py`, replacing `importlib` with `sys.path.append` to streamline module loading. - Improved debug visibility in `eval_model` with additional `print` statements for key file paths and parameter values. - Increased `debug_subset_len` from 2 to 50 in `eval_model` for broader test coverage during debug mode. - Updated `exp.ipynb` to align with changes in model handling and evaluation behavior in `eval_model`.
8f76d7d
to
aba8ff4
Compare
Purpose: Improve the clarity and depth of ILL evaluation, and introduce new tools for classifier-based analysis. Changes: - Updated to clean outputs, improve ROC curves, and set . - Improved structure and markdown clarity in , with added analysis on loss distributions and unlearning. - Added and for classifier-based ILL feature exploration. - Added script for reproducible, scriptable Random Forest analysis. These updates improve reproducibility, interpretability, and support deeper ILL feature analysis.
…ion in notebooks The primary purpose is to fix broken imports and implement functional Input Loss Landscape feature computation for machine learning interpretability analysis. The primary changes are: - Enhanced import structure in with additional sklearn modules and SHAP availability check. - Replaced broken function calls with working ILL feature computation pipeline. - Added comprehensive logistic regression analysis with performance metrics, confusion matrix, and feature importance analysis. - Integrated permutation importance computation and visualization for feature interpretability. - Fixed execution flow by removing error-prone cells and replacing with successful feature extraction results. - Updated notebook outputs to show successful ILL feature computation for forget/retain/holdout datasets. - Added baseline logistic regression performance evaluation with 74% accuracy and detailed classification report. - Modified to align with the working implementation in .
The primary purpose is to evaluate the loss landscape of first neighbor sentences to understand the impact of unlearning. The primary changes are: - Created a new notebook `MUSE/notebooks/1st_neighbor_classification.ipynb` to analyze the loss landscape of first neighbor sentences. - Modified `loss_landscape.py` to extract logic into `new_ILL_eval`, `get_features`, and `normalize_features`. - Replaced dynamic imports with `sys.path` appends in `utils.py`. - Added `transformers` to `requirements.txt`. - Increased UMAP dimensionality from 2 to 10 in `embedding.py`. - Added AUC heatmap and bar chart of top features in `visualization.py`. - Modified `plotting.py` to return `matplotlib` figure objects instead of file paths. - Updated `plotting.py` to align with changes made in `visualization.py`.
The purpose of this change is to prevent errors when saving the statistical distances heatmap. The changes include: - Added `os.makedirs(plots_base_dir, exist_ok=True)` before saving the heatmap in `eval_with_ILL.py` to ensure the directory exists.
The primary purpose is to provide a reproducible workflow for evaluating Input Loss Landscape (ILL) features on the TOFU dataset using a Llama-2-7b model. The primary changes are: - Added `TOFU/notebooks/eval_with_ILL.ipynb` containing a step-by-step pipeline for: - Loading and preprocessing the TOFU dataset from Hugging Face. - Loading model and tokenizer with correct prompt formatting. - Running ILL evaluation using project utilities and saving results. - Extracting and normalizing ILL feature tensors for analysis. - Visualizing loss landscape features with matplotlib plots. - The notebook demonstrates integration between the TOFU, MUSE, and project source directories. - Example code for prompt formatting, model inference, and loss calculation is included for clarity. - Notebook serves as a reference for future ILL experiments and analysis on TOFU.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.