Grass seeds#2558
Conversation
|
I still really do want the in-world interaction of placing grass seeds (described in #929, ) but not being able get grass blocks in survival is so irritating that I would rather just have this now and the placing can be added later.
This is also neat. If you live in a biome with mixed grass types, you could convert one into the other depending on which you want. |
| .inputs = .{"cubyz:{type}_grass_vegetation"}, | ||
| .output = "3 cubyz:grass_seeds", | ||
| .reversible = true, | ||
| }, |
There was a problem hiding this comment.
This doesn't allow converting regular/temperate grass vegetation into seeds (since it's just cubyz:grass_vegetation)
Also, this should be 1 grass : 4 seed to be a little more generous
There was a problem hiding this comment.
did both of these htings
There was a problem hiding this comment.
mayhaps cubyz:grass_vegetation should be cubyz:regular_grass_vegetation or something of the like in a future pr, for consistency
There was a problem hiding this comment.
all the grass items should be migrated to cubyz:grass/{type} and cubyz:grass/vegetation/{type} and default grass should be "temperate grass"
| .inputs = .{"cubyz:mud", "cubyz:grass_seeds"}, | ||
| .output = "cubyz:lush_grass", | ||
| .reversible = true, | ||
| }, |
There was a problem hiding this comment.
Recipes with multiple inputs can't be reversible (you get an error for this ingame)
I don't really think this should be reversible anyways. If you want the soil block back, just place and break the grass block. (grass seeds are so cheap you don't really need to get them back)
There was a problem hiding this comment.
forgot this when copy and pasting
fixed
New grass seeds texture thanks
|
alright changes have been made |
| }, | ||
| .{ | ||
| .inputs = .{"cubyz:grass_vegetation"}, | ||
| .output = "5 cubyz:grass_seeds", |
There was a problem hiding this comment.
It should be 1 grass into 4 grass seeds.
There was a problem hiding this comment.
I think 5 is fine, 120 is divisible by 5, not everything need to be a power of 2.
IntegratedQuantum
left a comment
There was a problem hiding this comment.
I think since there are so many soil types and so many special cases in the recipes, they should be in their own file.
| .{ | ||
| .inputs = .{"cubyz:{type}_grass_vegetation"}, | ||
| .output = "5 cubyz:grass_seeds", | ||
| .reversible = true, |
There was a problem hiding this comment.
I don't think these should by reversible.
There was a problem hiding this comment.
its not really a balance issue if players can convert between grass types
There was a problem hiding this comment.
Surely it doesn't make sense to craft seeds into the fully grown plant?
There was a problem hiding this comment.
| .reversible = true, |
Please apply @Crepestrom so we can proceed with this change.
We also can soon figure out interaction with AAA series of PRs
|
lol, i and i was playing with the thought of making grass types just spread over time over onto neighbouring dirt blocks. nevermind i guess |
| .{ | ||
| .inputs = .{"cubyz:grass_vegetation"}, | ||
| .output = "5 cubyz:grass_seeds", | ||
| .reversible = true, |
| }, | ||
| .{ | ||
| .inputs = .{"5 cubyz:grass_seeds", "cubyz:{type}_grass_vegetation"}, | ||
| .output = "2 cubyz:{type}_grass_vegetation", |
There was a problem hiding this comment.
It's better than just crafting it directly, but I'm still not happy with the fact that you can craft them back.
I would prefer to keep these recipes out. (I am not opposed to being able to get more grass vegetation, it would make a lot of sense to be able to grow it as a crop, but having a recipe for it just feels wrong).
Maybe someone else could back me up on this, or tell me that it's just irrelevant and we should allow this.
There was a problem hiding this comment.
(The grassy addon also doesn't have recipes that transmute grass to other grass types)
There was a problem hiding this comment.
Hm growing it as a plant sounds cool.
We could make it so when you right click specific grass block it plants corresponding vegetation.
There was a problem hiding this comment.
Yeah, that's how I would imagine farming would go, you place the seeds on the corresponding block, and then wait a while for it to grow.
There was a problem hiding this comment.
i mostly did this to allow players to get rid of exess seeds
exess seeds would be more inventory clutter
and letting players convert them in a way where they still needed to go to the biome in the first place helps keep players stocked up on a nice decorative block
There was a problem hiding this comment.
You'd still have clutter if the number of seeds left is not divisible by 5.
|
anyways the grass conversion recipies have been removed |
| }, | ||
| .{ | ||
| .inputs = .{"cubyz:grass/{type}"}, | ||
| .output = "5 cubyz:grass_seeds", |
There was a problem hiding this comment.
This looks wrong, you shouldn't be able to turn the soil-containing block back.
Also this creates duplicate recipes, since it also matches grass/vegetation/*.
Adds grass seeds Obtaining: you can convert grass vegitation into grass seeds Use: turning the dirt/soil types into their grassy version converting grass vegitation types from one type to another progress towards PixelGuys#929 --------- Co-authored-by: Carrie <carriecapp9@gmail.com>
Adds grass seeds
Obtaining:
you can convert grass vegitation into grass seeds
Use:
turning the dirt/soil types into their grassy version
converting grass vegitation types from one type to another
progress towards #929