Skip to content

Wechall: Update max score for HTS#137

Open
livinskull wants to merge 1 commit intomasterfrom
hts-update-max-score
Open

Wechall: Update max score for HTS#137
livinskull wants to merge 1 commit intomasterfrom
hts-update-max-score

Conversation

@livinskull
Copy link
Copy Markdown
Collaborator

A lot of users have > 100% reported.

It seems that the "onsitescore" variable, which was returned as "maxscore" was way too low. Calculated a current potential max score from https://www.hackthissite.org/user/rankings/explanation and used this.

Updated the chall count, too.

Thanks Munto for pointing to the ranking explanation page.

PS: This is yet another workaround. Maybe we can get things working correctly art some point with #32

A lot of users have > 100% reported.

It seems that the "onsitescore" variable, which was returned as
"maxscore" was way too low. Calculated a current potential max score
from https://www.hackthissite.org/user/rankings/explanation and used
this.

Updated the chall count, too.

Thanks Munto for pointing to the ranking explanation page.
@spaceone
Copy link
Copy Markdown
Collaborator

@gizmore Approve this?

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.

2 participants