FSharp.Stats icon indicating copy to clipboard operation
FSharp.Stats copied to clipboard

Update Normalization module

Open bvenn opened this issue 3 years ago • 4 comments

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

bvenn avatar Jan 17 '22 09:01 bvenn

Agreed with everything here, just note that it is not absolutely necessary to update blog posts as the respective packages still work.

kMutagene avatar Jan 17 '22 17:01 kMutagene

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.

bvenn avatar Jan 17 '22 17:01 bvenn

yup, so i think blog posts should always pin dependencies then

kMutagene avatar Jan 18 '22 08:01 kMutagene

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:

  1. Disconnect from folder structure and order all modules by their dependency structure. FSharp.Stats grew quite a bit during the last years, circular module dependencies cannot be circumvented anymore.
  2. Move RMT.fs out of folder structure, move it to the very bottom, and add it to FSharp.Stats.Signal namespace. The folder structure would be broken, and the search for the correct fs would be tedious if this happens more often.
  3. Leave RMT in Testing folder but update namespace from FSharp.Stats.Testing to FSharp.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.

bvenn avatar Jan 27 '22 15:01 bvenn