Skip to content

Conversation

@Zanieon
Copy link
Contributor

@Zanieon Zanieon commented Dec 13, 2025

From discord:

Laser Core was supposed to have extended sustain like Sword Core as long as kills are being made to stack up the time, however Laser Core ends early if nothing is killed in a 5 second window, despite the core meter saying it is active, it just ends prematurely, and with the core meter being stuck in the active state when it is no longer active entirely prevents Ion from building up core again

@Zanieon Zanieon added the commits vanilla file For PRs that commit vanilla files from VPKs label Dec 13, 2025
@Zanieon Zanieon moved this to Zanieon Doing in Personal TODOs Dec 13, 2025
@github-actions github-actions bot added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Dec 13, 2025
@Zanieon Zanieon requested a review from Alystrasz December 13, 2025 01:18
@github-actions github-actions bot added the merge conflicts Blocked by merge conflicts, waiting on the author to resolve label Dec 21, 2025
Copy link
Member

@Alystrasz Alystrasz left a comment

Choose a reason for hiding this comment

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

Have you tested this change?

float coreFrac = min( 1.0, remainingTime / duration )
//Defensive fix for this sometimes resulting in a negative value.
if ( coreFrac > 0.0 )
if ( coreFrac > 0.01 )
Copy link
Member

Choose a reason for hiding this comment

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

Why changing this value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ends the state of Core active early and is actually what prevents Ion from getting stuck in there after the laser finishes.

@github-actions github-actions bot removed the merge conflicts Blocked by merge conflicts, waiting on the author to resolve label Dec 21, 2025
@catornot catornot requested review from a team and catornot December 24, 2025 03:25
@EM4Volts
Copy link
Contributor

This sounds like a bug in how vanilla implements it, I thought we don't fix those unless it's some heavy issue like the splitter rifle dps

@Zanieon
Copy link
Contributor Author

Zanieon commented Dec 26, 2025

Ion not being able to gain core back if the laser ends prematurely in comparison to the core meter is quite important in FD, mind that this is an Aegis Laser Core fix, it does not affect Ion in normal core even with Grand Cannon kit.

@catornot
Copy link
Member

how to test this?

@ASpoonPlaysGames
Copy link
Contributor

how to test this?

Max aegis Ion -> laser core -> kill a bunch of stuff with the core -> core should end at the right time (should match UI in the bottom left)

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

Labels

commits vanilla file For PRs that commit vanilla files from VPKs needs code review Changes from PR still need to be reviewed in code needs testing Changes from the PR still need to be tested

Projects

Status: No status
Status: Zanieon Doing

Development

Successfully merging this pull request may close these issues.

6 participants