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

[Rando] Grassanity #4889

Draft
wants to merge 22 commits into
base: develop
Choose a base branch
from
Draft

[Rando] Grassanity #4889

wants to merge 22 commits into from

Conversation

Varuuna
Copy link
Contributor

@Varuuna Varuuna commented Jan 16, 2025

General

Setting up this draft PR to help me keep track of things a bit better.
Temp. location names. I can make them a bit easier to locate by using better names (and not use "X Grass 1").

Overworld (excl. grottos) is pretty much all done with one exceptions:

2 bushes behind the sign near the crawlspace in KF, due to their identical coords currently counts as 2 "Child/Adult" locations instead or 2 Child and 2 Adult. I believe other randomizers separate them.
I would prefer to separate them, but unsure of how I would go about it. Looked at Pots in the Guard House, but seems like no 2 pots share coordinates.

Remaining

  • Deku Tree + MQ
  • Dodongo's Cavern + MQ
  • Jabu Jabu MQ
  • Bottom of the Well + MQ
  • Grottos
  • Hints

Known Issues

  • Market locations currently can only be done during the day, due to day and night being different scenes.
  • Not collecting an item from grass that can regrow, letting it regrow and cutting it again drops another (duplicate) item, however, collecting the other item after collecting one (any order) does not do anything.
  • The 2 grass above Big Octo in JJB MQ are currently not working.

Build Artifacts

* Known issue with Market Night + 2 bushes in KF
* Known issue with Adult bush in ZR
* changed grassnity to be a combobox option
* added first 5 deku tree locations
* added ZR 14 (adult only bush on the platform, not the same?)
* updated draw method to fix regrowing grass
@Varuuna
Copy link
Contributor Author

Varuuna commented Jan 16, 2025

Noticed some old locacc files in there that are not supposed to still be there, will make sure to clean those out

@Malkierian
Copy link
Contributor

Yeah, I don't know where those locacc files came from, but they aren't in develop, so you were restoring that state rather than removing them.

@Varuuna
Copy link
Contributor Author

Varuuna commented Jan 16, 2025

Yeah, I don't know where those locacc files came from, but they aren't in develop, so you were restoring that state rather than removing them.

Yeah I thought I had gotten rid of them (ran into an issue with git after I accidentally kept them during merging). It looks correct now based on the files changed, but if you want me to go back and get it sorted so it doesn't even show up, I'd understand.

@Malkierian
Copy link
Contributor

Nah, we squash merge, so those files will basically disappear to develop when that happens.

@Varuuna
Copy link
Contributor Author

Varuuna commented Jan 18, 2025

Currently using this sickly green color to show that there is a check, because I have no design sense. Open for suggestions.
image

@Malkierian
Copy link
Contributor

If you're going to go color code, you might as well add a color for each CSMC tier, but perhaps custom textures would be better.

@Varuuna
Copy link
Contributor Author

Varuuna commented Jan 18, 2025

I could probably make it a less jarring green color, but I might leave the idea of custom textures to someone else. It's really not my domain to look into, haha.

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