wetransform-os / ProMCDA

A tool to estimate ranks of alternatives and their uncertainties based on the Multi Criteria Decision Analysis approach.
Eclipse Public License 2.0
5 stars 1 forks source link

Pipeline for no variability #3

Closed Flaminietta closed 1 year ago

Flaminietta commented 1 year ago

I'd like to open an intermediary PR.

In this PR all the normalization functions (the ones present in the R-code) are implemented and tested. For now, I am considering only an MCDA run with no variability analysis (no Monte Carlo sampling).

The next steps will then be:

Please, have a look at the logic in particular, by testing I've seen that it is simple to make distraction errors in the implementation of the functions.

Flaminietta commented 1 year ago

@mspada What I still did not do for all the norm Python functions is to test them against the R-functions, something I can easily do next. I'll report on these tests soon.

Flaminietta commented 1 year ago

All normalization functions have been tested against the ones implemented in R. minmax reverse is now correct. All norm functions produce the same results in both implementation.

mspada commented 1 year ago

Thanks Flaminia, all fine to me

Flaminietta commented 1 year ago

@mspada @kapil-agnihotri

The PR is now ready for your final review. It adds:

I'll squash the commits during the merge.