Update Normalization module
Description
The normalization module is located within the Signal folder but is within the FSharp.Stats namespace. This should be updated to FSharp.Stats.Signal.
@HLWeil
Furthermore the random matrix theory is in Testing.RMT. Although statistical tests are key elements of this method, it is more associated to Signal. Consequently the references must be updated in fslab blogpost.
- [x] update Signal.fs namespace
- [ ] move RMT to Signal
- [x] either update fslab blogpost or lock FSharp.Stats version
Agreed with everything here, just note that it is not absolutely necessary to update blog posts as the respective packages still work.
Agreed with everything here, just note that it is not absolutely necessary to update blog posts as the respective packages still work.
It does not work if there is no version defined in the #r nuget reference and a new package is released with the changes, right? At least for FSharp.Stats it seems that no one pinned the version.
yup, so i think blog posts should always pin dependencies then
RMT migration to Signal is more difficult than expected:
- Signal
-
- Distribution
- KernelDensity <- dependence of Signal.FFT
- Testing <- dependence of Distribution
- RMT <- dependence of Testing.chiSquared, move to Signal
There is no ordering, that does not provoke circular module dependencies. I can think of three options:
- Disconnect from folder structure and order all modules by their dependency structure.
FSharp.Statsgrew quite a bit during the last years, circular module dependencies cannot be circumvented anymore. - Move
RMT.fsout of folder structure, move it to the very bottom, and add it toFSharp.Stats.Signalnamespace. The folder structure would be broken, and the search for the correct fs would be tedious if this happens more often. - Leave RMT in Testing folder but update namespace from
FSharp.Stats.TestingtoFSharp.Satts.Signal.
Are there any other options I missed, except of leaving everything as it is? RMT uses Testing but actually it would make sense to move it to Signal.