[Raidevents] Overlapping Adds event causes problems

Open
#4,343 1 comment 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
20/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
cpp
Domain
game-dev

Research direction

Compare the two linked Raidbots reports and their Pastebin inputs, focusing on the difference in add-event naming and overlap. Then trace how the project handles those events; the issue does not name files or tests, so first locate the relevant simulation code and a way to reproduce the DPS change. Done means the named overlapping adds no longer cause the reported drastic DPS drop.

Written by the indexing model from the issue text.

Description

bug
Dominant language
C++
Stars
1.6k
Forks
789
Avg merge
13h 25m
Merged PRs (30d)
52

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from simulationcraft/simc

All issues in simulationcraft/simc

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.