-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
Blossoming Pots #1070
base: master
Are you sure you want to change the base?
Blossoming Pots #1070
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would probably want to limit the number of display entities per pot to something like 5. Anything above that I feel goes beyond of what "feels vanilla". Especially the coral decorations, where it's using 41 display entities for a single pot. That's not only a performance issue, but I also don't think it looks good visually, since there are too many small details and you just don't see that with vanilla. This already has the coral fans, I think those are a better fit.
I was a bit unsure on how this module worked tech wise, with the storage. From my point of view coming from beet, I would probably generate a mcfunction for each supported item, which just had the item display summon commands, that would probably allow you more flexibility without confirming to a strict storage schema.
I think it is probably fine to keep using the storage, unless you come across a major hurdle.
I didn't do a full technical review yet, but it seems fairly well put together after a first look.
gm4_blossoming_pots/data/gm4_blossoming_pots/function/main.mcfunction
Outdated
Show resolved
Hide resolved
Totally get it. I feel a little bad removing the coral blocks because I worked hard on them when I made them, but I totally get it and am down to axe them. I'm curious on your thoughts for a limit of display entities. There's only 5 other things that have more than 5 displays. The next highest count after Coral Blocks is the 3rd stage of Chorus Fruit with 18, which im much more hesitant to remove. After that is Kelp and Twisting Vines which go up to 16 and I'm willing to chop those down a good bit. Bamboo and Glow Berries have 8 and Weeping Vines has 9, again willing to chop those down a little bit. |
…into gm4-blossoming-pots-intial
Blossoming Pots initial commit.
Ready for code review and further discussion, I think...