Improve default HDRI

[Ni-g-3l]

Describe your changes

This PR update the default HDRI according to what we decide on discord.

Issue ticket number and link if any

• Improve default HDRI #1063

Checklist for finalizing the PR

• I have performed a self-review of my code
• I have added tests for new features and bugfixes
• I have added documentation for new features
• If it is a modifying the libf3d API, I have updated bindings
• If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

[Ni-g-3l]

\ci fast

[mwestphal]

@Ni-g-3l Im not sure we want to make it default right now, but we want to provide it and let users opt in with CL option/libf3d options.

Wdyt @Meakk ?

[Meakk]

I think it's fine to change it to be honest. Worst case is users complaining but we can give the old HDRI file and associated config entries to revert to the old behavior.

[mwestphal]

I think it's fine to change it to be honest. Worst case is users complaining but we can give the old HDRI file and associated config entries to revert to the old behavior.

Do you mean as a proper CLI option or just as a file to use with --hdri-file ?

[Meakk]

yes

[mwestphal]

a file to use with --hdri-file ?

Ok. thanks this MR can proceed.

[Ni-g-3l]

\ci main

[mwestphal]

The default color or mesh without texture need to be changed to something more gray.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.04%. Comparing base (4102575) to head (5e30b2a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2969      +/-   ##
==========================================
- Coverage   97.05%   97.04%   -0.02%     
==========================================
  Files         213      213              
  Lines       17546    17547       +1     
==========================================
- Hits        17029    17028       -1     
- Misses        517      519       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mwestphal]

LGTM :)

Now just needs new baselines.

[mwestphal]

Hi @Ni-g-3l , need any help moving forward ? Merging this in june is important to avoid bad surprises before 4.0 :)

[Ni-g-3l]

\ci full

[mwestphal]

I'm afraid you need a rebase on master and recover baselines again. sorry about that :/

[Ni-g-3l]

Done ! I'm waiting for the CI to ensure that all baseline are correctly generated

[mwestphal]

Done ! I'm waiting for the CI to ensure that all baseline are correctly generated

Thanks! Ill make sure this get merged before we change all baselines, again ^^