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

impr(result): rework burst heatmap speed brackets (keenemeck) #4741

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

keenemeck
Copy link
Contributor

@keenemeck keenemeck commented Oct 25, 2023

fix for #4678

Indexes values to create uniform distribution, and prevents negative values from existing in burstmap.

Ranges can be modified by changing the decimal values; current distribution is below and appears similar to what is currently used.

<15%, 15%-35%, 35%-65%, 65%-85%, 85%+
where the percentages refer to the amount of words in a given range
i.e. the lowest ~15% of words are in the lowest range, and the next 20% are the in next, etc.

I believe it's less computationally expensive as well, so there's that.

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Oct 25, 2023
@Miodec Miodec changed the title fix: #4678, remove negative values from burstmap impr(result): rework burst heatmap speed brackets (keenemeck) Oct 27, 2023
@Miodec Miodec merged commit a30b367 into monkeytypegame:master Oct 27, 2023
9 checks passed
@Miodec
Copy link
Member

Miodec commented Oct 27, 2023

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend User interface or web stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants