Wrong Cthun Planner

Wrong Cthun Planner

6.9k Downloads

set_marks() doesn't reference current_layout

rainecheck opened this issue ยท 1 comments

commented

Last night we manually moved positions in WCP so we could retain group buffs for melee groups during p2, and found that /wcp marks was marking based on our raid groups, not our manually configured WCP groups. I'm not exactly a great coder so please take with a grain of salt, but I suspect it's that WCP.LIB.Grid:set_marks() is directly referencing their position in the raid group, rather than their position in the current_layout.

We ended up manually marking and getting people to turn off the DBM auto-mark feature, but having /WCP marks follow the WCP map rather than the raid groups would be ideal.

commented

Hey, thanks for your feedback!
Your observation is totally correct.

Unfortunately, marks are (right now) not considering the repositioning or swapping of positions.
I totally agree that /wcp marks might not work as expected when moving positions.

We somehow have to think of a way of how to "assign" a mark to a specific position.
I'll try to think about this problem next weekend and will come back to you!