Skip to content

[Gear] Trinket - Font of Venomous Rage#11531

Open
criller7070 wants to merge 3 commits into
simulationcraft:midnightfrom
Lucasmingus:font-of-venomous-rage_implementation
Open

[Gear] Trinket - Font of Venomous Rage#11531
criller7070 wants to merge 3 commits into
simulationcraft:midnightfrom
Lucasmingus:font-of-venomous-rage_implementation

Conversation

@criller7070

Copy link
Copy Markdown
Contributor

No description provided.

@nyterage nyterage left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neither of these are really vital. but wanted to point out their existence as they can make things a bit easier.

Comment thread engine/player/unique_gear_midnight.cpp Outdated
generic_aoe_proc_t( e, "venom_splatter", e.player->find_spell( 1307222 ) )
{}

size_t available_targets( std::vector<player_t*>& tl ) const override

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be able to use target_filter_callback for this rather than manually adjusting available_targets.

specifically target_filter_callback = secondary_targets_only();.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat! i just found an earlier implementation of a way to remove main target

player->reset_auto_attacks( composite_dot_duration( execute_state ) );
}

void tick( dot_t* d ) override

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can just set venom_splatter to be the channels tick action to achieve this as well

tick_action = venom_splatter in the channel constructor.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't this overwrite the tick dmg of the actual main target dmg of the channel?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, the actual main action tick damage is handled in the base_td. has nothing to do with tick_action. all tick_action setting does is automatically execute that on the target on tick, while also handling the automatic state updates, direct_tick flags, etc. all this means is tick_action should ideally be used whenever you have an action being executed on a tick.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm sorry, but when i use the tick action, this is the log

Player schedules execute for Action 'use_item_font_of_venomous_rage' (0)
Player performs Action 'font_of_venemous_rage' (1297908) (216117.5)
Player Action 'font_of_venemous_rage' (1297908) hits Enemy 'Pull_1_9tar' for 0.000000 nature damage (crit)
Player schedules execute for Action 'venom_splatter' (1307222)
Player Action 'font_of_venemous_rage' (1297908) ticks (0 of 4) Enemy 'Pull_1_9tar'
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy1' for 73636.785750 nature damage (crit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy2' for 32571.874132 nature damage (hit)

and on subsequent ticks

Player schedules execute for Action 'venom_splatter' (1307222)
Player Action 'font_of_venemous_rage' (1297908) ticks (1 of 4) Enemy 'Pull_1_9tar'
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy1' for 32571.874132 nature damage (hit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy2' for 32571.874132 nature damage (hit)

instead of

Player schedules execute for Action 'use_item_font_of_venomous_rage' (0)
Player performs Action 'font_of_venemous_rage' (1297908) (215752.5)
Player Action 'font_of_venemous_rage' (1297908) hits Enemy 'Pull_1_9tar' for 0.000000 nature damage (hit)
Player Action 'font_of_venemous_rage' (1297908) ticks (0 of 4) on Enemy 'Pull_1_9tar' for 592263.906001 nature damage (hit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy1' for 32571.874132 nature damage (hit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy2' for 32571.874132 nature damage (hit)

and

Player Action 'font_of_venemous_rage' (1297908) ticks (1 of 4) on Enemy 'Pull_1_9tar' for 592263.906001 nature damage (hit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy1' for 32571.874132 nature damage (hit)
Player Action 'venom_splatter' (1307222) hits Enemy 'Pull_1_9tar_copy2' for 73636.785750 nature damage (crit)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants