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

Add role info (core/support) #1844

Open
Tsury opened this issue Feb 5, 2019 · 9 comments
Open

Add role info (core/support) #1844

Tsury opened this issue Feb 5, 2019 · 9 comments

Comments

@Tsury
Copy link

Tsury commented Feb 5, 2019

Is it possible to add role info (core/support)? It might be obtained from GPM/wards purchased.
I'm not interested in specific position (1-5), but just whether a given player was a core (1-3) or a support (4-5).

@howardchung
Copy link
Member

currently for the analysis page we just base it off wards placed (>2) and less than 20 last hits at 10 minutes:
https://github.com/odota/web/blob/5436c56f491b1d9b759147407f9a48eb7d022570/src/utility/index.jsx#L257

You can compute the same value statically from a match data blob.

@Tsury
Copy link
Author

Tsury commented Feb 6, 2019

Sorry, I wasn't being clear. There's no way to retrieve this information without getting the match data blob, which means that if needed to obtain for multiple matches, one has to fetch and calculate for all of them separately. I feel like role is parallel to lane, in category/importance and should be /players/matches and /players/recentMatches. Is that something you can consider adding?

@howardchung
Copy link
Member

It could be added for future matches, but currently isn't saved to the cassandra player_caches table so can't be returned for each player-match

@Tsury
Copy link
Author

Tsury commented Feb 6, 2019

Sounds good to me, future matches is fine. Thanks!

@ahi-el
Copy link
Contributor

ahi-el commented Nov 9, 2019

I'm guessing this was before ranked roles were introduced. Do we get the roles now?

@howardchung
Copy link
Member

I have not played ranked roles. Is it available in the post-match stats in client?

@ahi-el
Copy link
Contributor

ahi-el commented Nov 14, 2019

I didn't find anything in 'ranked' matches, but there is a field that describes it well

image

This is from the data that is used in Table (in web)

Note - while this was accurate for 1 team, for the other team it displayed both offlane players as core, even though one of them was clearly support since he took no last hits, so it might be somewhat innaccurate

link to match with inaccurate desc: https://www.opendota.com/matches/5109807012/overview

@howardchung
Copy link
Member

We calculate support-ness just by the last hit at 10 minutes count and ward placement, see https://github.com/odota/web/blob/master/src/actions/transformMatch.js#L151 and the isSupport function in utility.

@howardchung
Copy link
Member

The reason that it's in the client side code and not returned in the API/server is that we were aware that it would probably be somewhat inaccurate to guess using these metrics and wanted the flexibility to change it

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

No branches or pull requests

3 participants