-
Notifications
You must be signed in to change notification settings - Fork 18
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
[WIP] WLS_sparse2 solver #121
base: main
Are you sure you want to change the base?
Conversation
Rebasing the fork
Signed-off-by: Bas des Tombe <[email protected]>
…ntial calibration in chunks. Signed-off-by: Bas des Tombe <[email protected]>
Signed-off-by: Bas des Tombe <[email protected]>
Signed-off-by: Bas des Tombe <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #121 +/- ##
==========================================
- Coverage 75.25% 74.86% -0.39%
==========================================
Files 8 8
Lines 3180 3258 +78
Branches 687 713 +26
==========================================
+ Hits 2393 2439 +46
- Misses 585 610 +25
- Partials 202 209 +7
Continue to review full report at Codecov.
|
Signed-off-by: Bas des Tombe <[email protected]>
Signed-off-by: Bas des Tombe <[email protected]>
Signed-off-by: Bas des Tombe <[email protected]>
Nice work! One question though; is |
I would think the latter. There would be no need for two sparse solvers so It is going to be a pain to fully implement this though in |
But I'm open to using other names :) we could drop the |
wls_sparse2 is fine as a temporary name as it's going to replace the old one. For continuity sake I think we have to keep the name 'wls_sparse' though, to avoid breaking people's code (which is something we have to worry about now we're at version 1.0). Fixing parameters is a mess yes, but extremely useful! Would there be a cleaner way to handle it in the back-end? Also wrap it in one or multiple functions perhaps? |
For now, I think the path forward would be to:
Thus quite some manual labor, but full control.. And If you'd like to fix certain parameters, you could use Maybe there should be two example notebooks. 1. Without fixing any parameters and 2. With fixing gamma. |
Signed-off-by: Bas des Tombe <[email protected]>
Introduces a new solver that accepts prior knowledge of the parameter set. And should generally be faster than
wls_spare
.Todo:
ds.calibration_single_ended()
andds.calibration_double_ended()
functions.