Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Davis Bouldin Index Formula #182

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ParagEkbote
Copy link
Contributor

Fixes #180

As described in the issue, I've kept the same colors (purple and cyan) for the text and arrows. Please let me know if any further corrections are needed and I will make the necessary changes.

cc: @santiviquez

@santiviquez
Copy link
Collaborator

Hi @ParagEkbote,

Sorry, but this wasn’t exactly what I had in mind. I think I did a poor job explaining the issue.

As you know, the current formula only colors the $i$ and $j$, but that’s not what we want. What I would prefer instead is to color and add arrows to the following parameters:

$k$: total number of clusters
$R_{ij}$: similarity between clusters $i$ and $j$
$s_i$: within-cluster scatter
$d_{ij}$: distance between the centroids

This means we would need to use four different colors and rows with their respective descriptions.

@ParagEkbote
Copy link
Contributor Author

ParagEkbote commented Jan 25, 2025

No worries! I'll modify the colors as required.Could you please let me know what 4 colors you'd prefer to use for the given parameters?

cc: @santiviquez

@santiviquez
Copy link
Collaborator

@ParagEkbote you can use any of the ones defined in main.tex

\definecolor{nmlpurple}{HTML}{3B0280}
\definecolor{nmlcyan}{HTML}{00C8E5}
\definecolor{nmlred}{HTML}{DD4040}
\definecolor{nmlgreen}{HTML}{4EB046}
\definecolor{nmlyellow}{HTML}{E1BC29}

@ParagEkbote
Copy link
Contributor Author

I've updated the colors. Could you please review the changes?

cc: @santiviquez

@ParagEkbote
Copy link
Contributor Author

Gentle ping

cc: @santiviquez

@santiviquez
Copy link
Collaborator

Hi @ParagEkbote, unfortunately the latest changes still don't align with what it is described in the issue.

Here's a sketch of what I had in mind.

IMG_4219

@ParagEkbote
Copy link
Contributor Author

@santiviquez Thank you for describing the formula with a sketch, will update the colors.

@ParagEkbote
Copy link
Contributor Author

Could you please review the changes?

cc: @santiviquez

@ParagEkbote
Copy link
Contributor Author

Gentle ping

@santiviquez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Davis Bouldin Index Formula
2 participants