-
Notifications
You must be signed in to change notification settings - Fork 26
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
Create CODEOWNERS #1
Merged
Merged
Conversation
This file contains 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
Add @lava-nc/lava-optimization-committers to CODEOWNERS ensuring only committers can merge patches for lava-nc/lava-optimization
awintel
approved these changes
Oct 25, 2021
GaboFGuerra
approved these changes
Oct 25, 2021
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.
Approved!
Thanks @GaboFGuerra @awintel |
srrisbud
added a commit
to srrisbud/lava-optimization
that referenced
this pull request
May 11, 2023
…ax cut-off Signed-off-by: Risbud, Sumedh <[email protected]>
srrisbud
added a commit
to srrisbud/lava-optimization
that referenced
this pull request
Jul 31, 2023
…ax cut-off Signed-off-by: Risbud, Sumedh <[email protected]>
srrisbud
added a commit
that referenced
this pull request
Nov 9, 2023
* Prelim commit Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: first pass Signed-off-by: Risbud, Sumedh <[email protected]> * Prelim commit Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: second pass Signed-off-by: Risbud, Sumedh <[email protected]> * Post code-review: removed `debug` from NEBM, added docstrings, reinstated `best_solution` in read_gate Signed-off-by: Risbud, Sumedh <[email protected]> * First VRP app commit; incomplete, needs tests Signed-off-by: Risbud, Sumedh <[email protected]> * Adding helper functions to generate Q matrices for a) clustering b) tsp. TODO: Add Typing code * Second VRP app commit; functional with VRPy. Includes tests. Signed-off-by: Risbud, Sumedh <[email protected]> * Second VRP app commit; solver complete, needs correct Q matrices Signed-off-by: Risbud, Sumedh <[email protected]> * Changes to clustering matrix complete. Made changes to TSP API only. Further work needed on TSP logic * New tsp matrix generator with tests * Changed formulation of distance in TSP. Encoding is now accurate * Added proper clustering Q matrix generator (with test) * VRP Solver: Almost there Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: first pass Signed-off-by: Risbud, Sumedh <[email protected]> * Prelim commit Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: second pass Signed-off-by: Risbud, Sumedh <[email protected]> * Post code-review: removed `debug` from NEBM, added docstrings, reinstated `best_solution` in read_gate Signed-off-by: Risbud, Sumedh <[email protected]> * Adding helper functions to generate Q matrices for a) clustering b) tsp. TODO: Add Typing code * Resolved conflicts * Prelim commit Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: first pass Signed-off-by: Risbud, Sumedh <[email protected]> * Prelim commit Signed-off-by: Risbud, Sumedh <[email protected]> * NEBM+SCIF merger: second pass Signed-off-by: Risbud, Sumedh <[email protected]> * Post code-review: removed `debug` from NEBM, added docstrings, reinstated `best_solution` in read_gate Signed-off-by: Risbud, Sumedh <[email protected]> * Changes to clustering matrix complete. Made changes to TSP API only. Further work needed on TSP logic * New tsp matrix generator with tests * Changed formulation of distance in TSP. Encoding is now accurate * VRPSolver first milestone: successfully solves VRPs Signed-off-by: Risbud, Sumedh <[email protected]> * SCIF CPU backend model minor change to remove `state_hist` and pass tests Signed-off-by: Risbud, Sumedh <[email protected]> * Delinting working VRPSolver Signed-off-by: Risbud, Sumedh <[email protected]> * Commented out a piece of code dependent on a draft PR Signed-off-by: Risbud, Sumedh <[email protected]> * Remove lint and add VRPy to PyProject.TOML Signed-off-by: Risbud, Sumedh <[email protected]> * Sparsification attempt #1: DistProxy with sign inversion and max cut-off Signed-off-by: Risbud, Sumedh <[email protected]> * Intermediate check point commit for scenario sweep Signed-off-by: Risbud, Sumedh <[email protected]> * Profiling and sparsification related improvements to VRPSolver and VRPConfig Signed-off-by: Risbud, Sumedh <[email protected]> * Script to sweep various scenarios for performance modelling of VRPSolver Signed-off-by: Risbud, Sumedh <[email protected]> * Profiling and sparsification related improvements to VRPSolver and VRPConfig Signed-off-by: Risbud, Sumedh <[email protected]> * Fixed the way to check if VRPy is installed Signed-off-by: Risbud, Sumedh <[email protected]> * Code clean-up refactoring in LCA module Signed-off-by: Risbud, Sumedh <[email protected]> * Corrected TSP Q matrix name Signed-off-by: Risbud, Sumedh <[email protected]> * Added edge-pruning based sparsification Signed-off-by: Risbud, Sumedh <[email protected]> * Tests and scripts for quantification of the effect of dist-mat sparsity on solution quality Signed-off-by: Risbud, Sumedh <[email protected]> * Sparsification attempt #1: DistProxy with sign inversion and max cut-off Signed-off-by: Risbud, Sumedh <[email protected]> * Delint VRP solver.py Signed-off-by: Risbud, Sumedh <[email protected]> * First commit of clustering and TSP. Clustering is almost complete. Signed-off-by: Risbud, Sumedh <[email protected]> * Cleaner unittest for solver Signed-off-by: Risbud, Sumedh <[email protected]> * Functioning Clustering and TSP apps Signed-off-by: Risbud, Sumedh <[email protected]> * Delinting Signed-off-by: Risbud, Sumedh <[email protected]> * Removed VRP from this branch Signed-off-by: Risbud, Sumedh <[email protected]> * Removed VRP unittests Signed-off-by: Risbud, Sumedh <[email protected]> * Clustering demo jupyter notebook added Signed-off-by: Risbud, Sumedh <[email protected]> * TSP demo jupyter notebook added Signed-off-by: Risbud, Sumedh <[email protected]> --------- Signed-off-by: Risbud, Sumedh <[email protected]> Co-authored-by: Ashish Rao Mangalore <[email protected]>
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.
Add @lava-nc/lava-optimization-committers to CODEOWNERS ensuring
only committers can merge patches for lava-nc/lava-optimization