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

Can Competitive be added to the DestinyActivityModeType? #712

Closed
ghost opened this issue Sep 29, 2018 · 10 comments
Closed

Can Competitive be added to the DestinyActivityModeType? #712

ghost opened this issue Sep 29, 2018 · 10 comments
Labels
bug filed A bug has been filed in BNet's internal bug tracking system for this request/report. enhancement ready for release

Comments

@ghost
Copy link

ghost commented Sep 29, 2018

For reference: DestinyActivityModeType in the docs

For private matches, you have each game type and an umbrella over all of them.
For Iron Banner, you have clash/control/supremacy specific to iron banner, then an umbrella over them.
For trials, you have each game type, then an umbrella over them.
For doubles, there is crimson doubles, doubles, and an umbrella over them.
For competitive, there is no hierarchy.

Why is this important you ask? Now that clash/control are in both the comp playlist and the quick play playlist, you can't use the data from the activity history endpoint to determine which playlist they were in. I'd have to get the PGCRs for every game to see how many people were on each team, but I'm pretty sure that would have edge cases (if 4 people finish the game on each team then I'd detect that as a comp match)

So far I have not needed to get the PGCRs for my clan's automation, but now that I know about this issue I think I'm going to. I have roles for competitive PVP KD/EFF to showcase people's skill, but right now it's only using countdown/survival for the calculation because I don't have easy access to which clash/control matches were done in the comp playlist. Is there an easier way to get a comp KD now?

Proposal: Add CompClash and CompControl and/or a Competitive type to the mode types.

@vthornheart-bng
Copy link
Contributor

I happen to be discussing that with folks as we speak! What I'd like to see is a larger overhaul of our activity modes to more consistently group them together, and this would be part of that effort.

I can't promise when we will do this, as there's a lot on our plate at the moment: but if we add a Competitive filter I'd like that to come with these overall improvement and standardization of how we return activity mode info.

@vthornheart-bng vthornheart-bng added enhancement bug filed A bug has been filed in BNet's internal bug tracking system for this request/report. labels Oct 2, 2018
@vthornheart-bng
Copy link
Contributor

TFS 701113

@vthornheart-bng
Copy link
Contributor

After some discussion on Reddit, I realized that waiting for the larger refactoring at some unknown point in the future is causing harm to the data people want now.

For the winter wishlist, I'm going to see about adding these now instead of making it part of the large refactoring attempt as a result. Better to do some good now than to wait for greater good that may never come.

@vthornheart-bng
Copy link
Contributor

I should note as well to set expectations, that it's too late for me to add any winter wishlist changes in time for the December 11th release, which is our last scheduled release of the year. I'll be attempting to work on these as I can, but I won't be able to deploy any winter wishlist fixes until the new year as a result.

@bladefist
Copy link

@vthornheart-bng sounds good, I think this is a great move

@ghost
Copy link
Author

ghost commented Dec 19, 2018

Just saw this. When the new mode(s) are added, will previously played games get their modes updated? This would be ideal, but I don't know about the feasibility of it.

@vthornheart-bng
Copy link
Contributor

vthornheart-bng commented Dec 26, 2018

Oy, I need to fix my spam filters: I didn't see this (or many other) notifications that someone commented. Bleh.

Sorry for the delay on that!

Yes, previously played games should automatically flow into the new mode(s) thankfully!

Here's the current gameplan:

  • The existing Clash and Control modes will remain as the "umbrella" for all clash/control, to maximize backcompat.
  • The following modes will be added:
    • pvpCompetitive (aggregate mode for all competitive playlist matches, left separate from the activity mode hierarchy... for now)
    • pvpQuickplay (aggregate mode for all quickplay playlist matches, left separate from the activity mode hierarchy... for now)
    • clashQuickplay (specifically clash matches played in quickplay, parented by "clash" mode)
    • clashCompetitive (specifically clash matches played in competitive, parented by "clash" mode)
    • controlQuickplay (specifically control matches played in quickplay, parented by "control" mode)
    • controlCompetitive (specifically control matches played in competitive, parented by "control" mode)

This approach should maximize backcompat while providing you with the data you've been requesting for so long!

Let me know if you have any questions or concerns!

This should be coming in the January deployment!

@TorKallon
Copy link

@vthornheart-bng would it be possible to let us know all of those mode numbers before the release? It would be great to get them added to our code before you guys make the change so nothing breaks unexpected.

@vthornheart-bng
Copy link
Contributor

Good call! They will be:

PvPCompetitive = 69,
PvPQuickplay = 70,
ClashQuickplay = 71,
ClashCompetitive = 72,
ControlQuickplay = 73,
ControlCompetitive = 74

@vthornheart-bng
Copy link
Contributor

This should be fixed as of today's deployment (1/29)!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug filed A bug has been filed in BNet's internal bug tracking system for this request/report. enhancement ready for release
Projects
None yet
Development

No branches or pull requests

3 participants