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

F3DEX3 implementation [WIP, Help needed] #280

Draft
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

eros71-dev
Copy link
Contributor

Currently trying to add F3DEX3 for the extra features and performance boosts, unfortunately the new rom extraction system doesn't allow it because some constants for the models use ROM_ASSET_LOAD_TEXTURE and the like, which the gbi.h doesn't like.

I need to figure out how to handle the gbi issue of textures due to the new rom extraction system
@eros71-dev eros71-dev marked this pull request as draft August 17, 2024 19:41
@eros71-dev
Copy link
Contributor Author

Here's an example of the errors being thrown:

include/PR/gbi.h:3743:5: error: initializer element is not constant
3743 | (unsigned int)(i)
| ^

@SonicTheHedgeHog64
Copy link

Why'd you remove all the light defines?

@eros71-dev
Copy link
Contributor Author

The original repo had a script called fixlights.py that apparently is needed to be ran to remove some things that f3dex3 doesn't use/need(?), running it did that, to be fair I'm also looking into it because the script doesn't even run properly without erroring out sometimes. I didn't have time to revert the actors and levels folder yet before I look into it.

@SonicTheHedgeHog64
Copy link

re-add the light's just change the direction?(idk what the menu means by direction) from 0x28 to 0x49 to fix lighting

@SonicTheHedgeHog64
Copy link

you forgot to change 0x28 to 0x49

@SonicTheHedgeHog64
Copy link

on all lights

@eros71-dev
Copy link
Contributor Author

To be fair, I'm fairly certain f3dex3 does not even need those anyways, and the HackerSM64 repo seems to confirm that as well, I'm just gonna see if I figure out why the fixlights script failed sometimes, so I made that commit in case I need to revert to it once again because I broke something

It breaks NUM_SAVE_FILES stuff, maybe shouldn't have done that and should have looked into that instead but it doesn't make much of a difference
@eros71-dev
Copy link
Contributor Author

Some updates, got it to "run"

imagen

@eros71-dev
Copy link
Contributor Author

Also, important, more or less, if we want these new features, we'll have to work on what this says (the occlusion plane may not be a must to be honest.)
imagen

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