forked from brettviren/wire-cell-toolkit
-
Notifications
You must be signed in to change notification settings - Fork 24
Open
Labels
cleanupSomething not strictly needed but which makes the code base more better.Something not strictly needed but which makes the code base more better.
Description
Some amount of "stale" code exists in WCT. It is not actively harmful but adds conceptual noise to the signal. This issue attempts to list things to be cleaned up when anyone might have some spare time (haha).
- The
BinnedDiffusion.hand code that uses it ingen/is an early, inferior convolution algorithm. It has never been used in production (to my knowledge) and instead the superiorBinnedDiffusion_transform.his the established algorithm. - The entire
sig/subpackage. I believe @HaiwangYu had some initial intention to do something in there but that did not fully materialize (please correct me if that is wrong). Note, the "spng" branch holds a SP redo that is active.
Please add anything else to this list in comments.
Metadata
Metadata
Assignees
Labels
cleanupSomething not strictly needed but which makes the code base more better.Something not strictly needed but which makes the code base more better.