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

2024.12 ForeverFPS #3304

Open
wants to merge 63 commits into
base: main
Choose a base branch
from
Open

2024.12 ForeverFPS #3304

wants to merge 63 commits into from

Conversation

brad-linden
Copy link
Collaborator

@brad-linden brad-linden commented Dec 18, 2024

as of = 2025-01-28 for RC deploy
build = https://github.com/secondlife/viewer/releases/tag/Second_Life_Release%2352a9316-ForeverFPS
cohort = ForeverFPS
deployed = https://github.com/secondlife/viewer/releases/tag/Second_Life_Release%2352a9316-ForeverFPS
desired = 3000

relnotes:

Second Life Viewer - ForeverFPS Beta

With this release we are continuing to refine performance and fix crashes. We are monitoring performance metrics broadly and for discrete GPUs with the goal of improving FPS "forever".

A new graphics setting has also been included: Max LOD Resolution. This allows users to limit texture sizes to the selected resolution in order to reduce VRAM load and data consumption.

Some users were experiencing a crash on exit with previous ForeverFPS builds. This should now be resolved, please let us know if you are still crashing while exiting the viewer.

For all Second Life viewers you may file a bug report directly at https://feedback.secondlife.com

@brad-linden
Copy link
Collaborator Author

static void setHandler(const Handler&);
static void setOutOfMemoryStrings(const std::string& title, const std::string& message);

// When viewer encounters bad alloc or can't access files try warning user about reasons
static void showOutOfMemory();
static void showMissingFiles();
// Genering error
static void show(const std::string&);
static void show(const std::string&, S32 error_code = -1);
Copy link
Contributor

Choose a reason for hiding this comment

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

If ERROR_OTHER is the "something else went wrong" code, wouldn't we want ERROR_OTHER to be the default value for show(error_code)?

Copy link
Contributor

@akleshchev akleshchev Jan 28, 2025

Choose a reason for hiding this comment

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

At the moment -1 means 'don't create a marker' in case you only need a message. The idea was that LL_ERRS will be creating llerror marker or more likely writing LAST_EXEC_LLERROR_CRASH flag into the marker (but time will tell how it turns out).

I intended to fix llerror marker files separately from new exit states, before the 'review priorities' thing happened.

Copy link
Contributor

Choose a reason for hiding this comment

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

May be I should have created a separate createMarkerFile in LLError instead of reusing show().

ERROR_OTHER = 0,
ERROR_BAD_ALLOC = 1,
ERROR_MISSING_FILES = 2,
} eLastExecEvent;
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be good to add a comment something like: "These are used to select LAST_EXEC_OTHER_CRASH, LAST_EXEC_BAD_ALLOC and LAST_EXEC_MISSING_FILES, respectively, which aren't visible from here."

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree, will keep it in mind if I get to continue.

Copy link
Contributor

Choose a reason for hiding this comment

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

I probably should have done somthing like ERROR_BAD_ALLOC = 6 instead of using switch.

@AtlasLinden
Copy link
Contributor

ForeverFPS Beta 7.1.12.12999043440 Result Comment
secondlife/iqa#436 Acceptance, Integration & Additional tests Passed Passed
Overall Result Passed OK to ship

NOTES
No blockers.

Anti-virus false positive report:

File Security vendor analysis
slplugin.exe SecureAge - Malicious
SLVersionChecker.exe SecureAge - Malicious
uninst.exe Bkav Pro - W32.AIDetectMalware

Copy link

@kylelinden kylelinden left a comment

Choose a reason for hiding this comment

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

ForeverFPS Beta approved
viewer = 7.1.12.12999043440
build = https://github.com/secondlife/viewer/releases/tag/Second_Life_Release%2352a9316-ForeverFPS
cohort = ForeverFPS
desired = 2000

@kylelinden kylelinden added team:viewer approved Thumbs up from Product labels Jan 28, 2025
@vir-linden
Copy link
Collaborator

@vir-linden vir-linden removed the approved Thumbs up from Product label Jan 28, 2025
@kylelinden
Copy link

Increase cohort size to 3000

Geenz and others added 7 commits January 30, 2025 01:59
* A lot of tweaking for water.

* Get water much closer to 6.6.17.

This won't produce 1:1 results, but it should be much closer to where we once were.
Fix media failing to function on PBR materials (#3445)
* #3507 Add distance haze to water to get us closer to 6.6.17's appearance.
* #3455 Add support for water exclusion surfaces
@github-actions github-actions bot added the cmake label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants