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 raids death indicator #7229

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add raids death indicator #7229

wants to merge 11 commits into from

Conversation

vikke1234
Copy link
Contributor

@vikke1234 vikke1234 commented Jan 11, 2025

Similarly to nylo death indicators and tzhaar hp, this plugin is a death indicator for ToA. The idea is to make it able to reliably confirm whether an enemy dies or not. It highlights akkha as a range (e.g. shadow) hit procs shadows.

It accomplishes this by computing the internal fraction for hitpoints.

Currently this only supports Ampken and Akkha. The enemy data for other rooms is a placeholder for v1.x, this is an initial PoC.

@runelite-github-app
Copy link

runelite-github-app bot commented Jan 11, 2025

@vikke1234
Copy link
Contributor Author

re-running workflows

@vikke1234 vikke1234 closed this Jan 11, 2025
@vikke1234 vikke1234 reopened this Jan 11, 2025
@iProdigy
Copy link
Member

not a full review but: on plugin shutdown you should do overlayManager.remove(overlay); and wsClient.unregisterMessage(EntityDamaged.class);

@vikke1234
Copy link
Contributor Author

alright, I'll fix it tomorrow 👍

@vikke1234
Copy link
Contributor Author

re-running workflows

@vikke1234 vikke1234 closed this Jan 23, 2025
@vikke1234 vikke1234 reopened this Jan 23, 2025
@LlemonDuck
Copy link
Contributor

Pushing a commit will already rerun workflows, you don't need to close/open the PR unless it's very stale

@LlemonDuck
Copy link
Contributor

Downgrade to log.debug https://github.com/vikke1234/raids-death-indicator/blob/42e6b82f1ccdeaf8fce7f5bc4b89b3d0e51ec9a9/src/main/java/com/example/utils/PredictionTree.java#L129

You may also be interested in the scaling work we did in the wiki dps calculator since the raids HP values are rounded in an interesting way: https://github.com/weirdgloop/osrs-dps-calc/blob/main/src/lib/scaling/TombsOfAmascut.ts#L36-L40

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Jan 25, 2025
@vikke1234
Copy link
Contributor Author

Hi,

Sorry for the delay, I'm in the middle of a move.

Is the rounding done for all monsters (baba monkeys etc.)? I added that for akkhas shadow, as I found the same thing in the ToA plugin.

@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Jan 29, 2025
@vikke1234
Copy link
Contributor Author

Pushing a commit will already rerun workflows, you don't need to close/open the PR unless it's very stale

I did this because I had forgot to push to the plugin repo, so the commit didn't exist and I needed to re-run the workflows.

@LlemonDuck
Copy link
Contributor

Path of Apmeken monkeys aren't hp-rounded (they aren't scaled at all anymore). I'm not aware of any other relevant exceptions off the top of my head.

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin added waiting for author waiting for the pr author to make changes or respond to questions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants