Skip to content

Commit ec587de

Browse files
KhusPatel4450claude
andcommitted
refactor: move webapp files to tests/plots to avoid code duplication
Move app.py, README.md, and requirements.txt from visualization/ into tests/plots/ so they share _utils.py with the existing plotters. Update the deploy workflow to point at tests/plots/. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 118fddc commit ec587de

5 files changed

Lines changed: 3 additions & 212 deletions

File tree

.github/workflows/deploy-visualization.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
push:
55
branches: [main]
66
paths:
7-
- visualization/**
7+
- tests/plots/**
88

99
concurrency:
1010
group: ${{ github.workflow }}-${{ github.ref }}
@@ -28,7 +28,7 @@ jobs:
2828
from huggingface_hub import HfApi
2929
api = HfApi(token=os.environ['HF_TOKEN'])
3030
api.upload_folder(
31-
folder_path='visualization',
31+
folder_path='tests/plots',
3232
repo_id='torchjd/interactive-plotter',
3333
repo_type='space',
3434
)
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ license: mit
1515
Interactive visualization of gradient aggregation methods from [TorchJD](https://torchjd.org).
1616

1717
Adjust the angle and length of each gradient vector and select aggregators to see how they combine
18-
the gradients. The green region shows the dual cone — any descent direction must lie inside it.
18+
the gradients. The green region shows the dual cone: the set of vectors with a non-negative inner product with each gradient.
1919

2020
## URL parameters
2121

File renamed without changes.

visualization/_utils.py

Lines changed: 0 additions & 209 deletions
This file was deleted.

0 commit comments

Comments
 (0)