Changed dendrogram representation, some format tests passed

[alexdepremia]

Proposed changes

I changed the dendrogram in order to better reflect the cluster populations

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

• Bugfix (non-breaking change which fixes an issue)
• [x ] New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

Put an x in the boxes that apply.

• [x ] Lint and unit tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works

[codecov-commenter]

Codecov Report

Merging #73 (c8fcb00) into main (f290feb) will decrease coverage by 2.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   80.61%   78.16%   -2.45%     
==========================================
  Files          10       10              
  Lines        1145     1177      +32     
==========================================
- Hits          923      920       -3     
- Misses        222      257      +35     

Impacted Files	Coverage Δ
dadapy/_utils/density_estimation.py	54.83% <0.00%> (-34.96%)	⬇️
dadapy/clustering.py	86.68% <0.00%> (-2.86%)	⬇️
dadapy/density_estimation.py	76.29% <0.00%> (-0.40%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f290feb...c8fcb00. Read the comment docs.

[AldoGl]

I @alexdepremia, I see that you changed many more modules than necessary, 7 instead of 3. If you are ok with it I will remove the unnecessary changes from this PR

[alexdepremia]

Yes, no problem...

[AldoGl]

Hi @alexdepremia I have generated the graphs before and after your change.

before

after

I see some changes in the saddle point locations. Could you explain once again what's the change in essence and how it improves visualisation?

[alexdepremia]

In both (old and new) dendrograms, each cluster is in the middle of a region proportional to its population. In the new one, this region is delimited by the links in which these clusters are involved and, in the case of the first and last clusters, by the beginning and end of the graph.

.

[AldoGl]

Ok it seems that we can close this PR