-
Notifications
You must be signed in to change notification settings - Fork 82
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
base: main
Are you sure you want to change the base?
Fix Davis Bouldin Index Formula #182
Conversation
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
This means we would need to use four different colors and rows with their respective descriptions. |
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 |
@ParagEkbote you can use any of the ones defined in main.tex \definecolor{nmlpurple}{HTML}{3B0280} |
I've updated the colors. Could you please review the changes? cc: @santiviquez |
Gentle ping cc: @santiviquez |
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. |
@santiviquez Thank you for describing the formula with a sketch, will update the colors. |
Could you please review the changes? cc: @santiviquez |
Gentle ping |
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