forked from brettviren/wire-cell-toolkit
-
Notifications
You must be signed in to change notification settings - Fork 26
General cleanups #448
Copy link
Copy link
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.
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.
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).
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.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.