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

Little birbs - distinct info quality dependent on player usage #239

Open
AlteOgre opened this issue Aug 20, 2018 · 2 comments
Open

Little birbs - distinct info quality dependent on player usage #239

AlteOgre opened this issue Aug 20, 2018 · 2 comments
Labels
improvement For improvements regarding existing features medium priority For something with medium priority needs testing Someone has to test this new feature For new features
Milestone

Comments

@AlteOgre
Copy link
Collaborator

AlteOgre commented Aug 20, 2018

The way '/pvp spy' currently works is suboptimal and unrealistic. I propose to improve the feature in the following manner:

When a player has PvP Mode ON, has no actve little birbs, and uses '/pvplist', (s)he should get rough direction info on players who have active little birbs.
When a player has PvP Mode ON and active little birbs, and uses '/pvplist', (s)he should get rough proximity and direction info on players who have no active little birbs, and (s)he should get detailed proximity and direction info on players who have active little birbs.

The distcintion between 'rough' and 'detailed' proximity and direction info is to be fixed:

  • Rough proximity info implies distances rounded in units of 1000 blocks.
  • Detailed proximity info implies distances rounded in units of 100 blocks.
  • Rough direction info implies either one of 4 wind directions.
  • Detailed direction info implies either one of 8 wind directions.

For this to be meaningful, the usage of '/pvp spy' should get a cooldown timer with a default value of 300 seconds (equal to the recommended default minimum cooldown on for LOTRmod FT).

I think it will be safe to remove the configurability of the direction info from the mod.

With this feature improvement I think the mod will also become way more useful to non LOTRmodded servers, and expansion of advertisement activities may be justifiable after this has been implemented. Think CurseForge, PMC and Minecraftforum.net.

This issue could be allocated to v2.0.0 or one of the subsequent sub-versions like v2.1.0.

@AlteOgre AlteOgre added improvement For improvements regarding existing features new feature For new features needs testing Someone has to test this medium priority For something with medium priority labels Aug 20, 2018
@CraftedMods
Copy link
Member

I would make this completely configurable, so the system can be used like it currently is and like you proposed that it will be. That will be a lot configuration options, but I think it's worth the work.

@AlteOgre
Copy link
Collaborator Author

For some it will be too hard to comprehend and they will not like to have to think through al possible setting combos. So I prefer we offer two basic set-ups: the present way and the system I proposed above.

@AlteOgre AlteOgre added this to the 2.0.0-BETA.2 milestone Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement For improvements regarding existing features medium priority For something with medium priority needs testing Someone has to test this new feature For new features
Projects
None yet
Development

No branches or pull requests

2 participants