-
Notifications
You must be signed in to change notification settings - Fork 15
test(aggregation): Improve interactive plotter #641
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
Merged
ValerianRey
merged 25 commits into
SimplexLab:main
from
rkhosrowshahi:feature/interactive-plotting-ui
Apr 14, 2026
+116
−35
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
5eebcf7
feat(aggregation): add GradVac aggregator
rkhosrowshahi a588c93
chore: Remove outdated doctesting stuff (#639)
ValerianRey 9d65f63
chore: Add governance documentation (#637)
PierreQuinton 3ab336c
refactor(gradvac): literal group types, eps/beta rules, and plotter UX
rkhosrowshahi e53849e
refactor(gradvac): base on GramianWeightedAggregator with GradVacWeig…
rkhosrowshahi 4909964
fix: update type hint for update_gradient_coordinate function
rkhosrowshahi a39f343
test(gradvac): cover beta setter success path for codecov
rkhosrowshahi 0359e60
Rename some variables in test_gradvac.py
ValerianRey 1da5f6e
Add comment about why we move to cpu
ValerianRey 21d55f9
Add GradVac to the aggregator table in README
ValerianRey 17b1dd5
Add changelog entry
ValerianRey 02a826b
Merge branch 'main' into feature/gradvac
ValerianRey f4e8e60
Remove seed setting in test_aggregator_output
ValerianRey 75c89c1
fix(aggregation): Add fallback in NashMTL (#640)
ValerianRey b100c8b
Merge branch 'main' into feature/gradvac
ValerianRey 193ffa6
Merge branch 'main' of https://github.com/TorchJD/torchjd into featur…
rkhosrowshahi 9ffdd13
Revert plot test refactors; keep GradVac in interactive plotter
rkhosrowshahi 50525a1
Merge branch 'main' into feature/gradvac (21f6b74)
rkhosrowshahi e626475
docs(aggregation): add grouping usage example and fix GradVac note
rkhosrowshahi a244d2b
docs(changelog): split Unreleased into Added and Fixed for GradVac an…
rkhosrowshahi 6a78932
feat(plots): restore enhanced interactive plotter UI
rkhosrowshahi 787f486
Merge branch 'main' into feature/interactive-plotting-ui
ValerianRey cd04362
Remove grouping example
ValerianRey ea8b5d5
Improve display of degrees
ValerianRey 0641812
Improve display of length
ValerianRey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like it would be nice to change the
__str__method ofAlignedMTLto also include thescale_mode. For example,str(AlignedMTL(scale_mode="min"))would becomeAlignedMTL-min.But this is quite independent from this PR, and should come in a different PR if you care about that! @rkhosrowshahi