great-lounge Saturday 8 May 2021

80 messages5 people

kalucky0Team07:09
wtf
I'm not a beta tester but ok
meme_man_not_ava18:49
Finally got the aether log_top textures to work, except....(edited)
now other crops are fucked
Jack PapelTeam18:53
That one on the left there looks like bamboo, is that correct? Maybe it uses a different naming scheme than the rest
Like it has bamboo_side but not bamboo_top
Just a thought, I haven't looked at the code for bamboo
meme_man_not_ava18:57
Replying toJack PapelThat one on the left there looks like bamboo, is that correct? Maybe it uses a different…
it's not bamboo, it's a Palo Verde Log from BYG
Jack PapelTeam18:58
Oh gotcha
meme_man_not_ava18:58
I get why you'd think bamboo though
Jack PapelTeam18:58
Honestly I think your life would be much easier if you used the block model json instead of all this context-specific code(edited)
meme_man_not_ava19:00
Replying toJack PapelHonestly I think your life would be much easier if you used the block model json instead…
Your both right AND wrong
Let me show you why
Here's a text file with all my crops, total crop count: 412
SweepSwap19:03
wow
412 lines?
meme_man_not_ava19:05
6 JSON files per crop: - 1. Crop Blockstate - 2. Crop Model - 3. Crop Item Model(aka Seeds) - 4. Essence Item Model - 5. Recipe for the Seeds - 6. Recipe for <something> made with the essence from this crop
I am NOT making 2.472 JSONs manually
I've already done hundreds manually in the past
I'm switching to automatic JSON generation cause I'm tired AF from doing that
Jack PapelTeam19:08
I don't think you should do it manually either
meme_man_not_ava19:12
anyhow what actually happens is this. The red area is supposed to detect that the specified resource texture ends with "side", which SHOULD mean it "catches" the aether crops and registers them differently by removing "side" from the specified resource texture and adds the word "top" to it. However after some testing I realized that for some reason they aren't "caught" by the if statement and move straight into the else, but why? But that only adds "_top" to the specified resource texture which is why the aether crops end up having missing top textures.
meme_man_not_ava19:13
Replying tomeme_man_not_avaanyhow what actually happens is this. The red area is supposed to detect that the…
But that only adds "_top" to the specified resource texture which is why the aether crops end up having missing top textures.
*Note: It doesn't do this right now because I forgot to add that back(edited)
kalucky0Team19:15
Lack of spaces before and after == hurts me
meme_man_not_ava19:16
Replying tokalucky0Lack of spaces before and after == hurts me
thank you for telling me this
I completely agree
hadn't noticed it yet
fixed
meme_man_not_ava19:33
hopefully this will do the trick
Izuny19:44
this might sound dumb, but anyone knows good fabric ambient mod that add also sword swing sound?
Jack PapelTeam20:26
Amely if that doesn't work, try using .equals() when comparing strings. I'm pretty sure that's why the if statement isn't working
Jack PapelTeam22:30
Replying tomeme_man_not_avahopefully this will do the trick
Did you get things working?
M
meme_man_not_ava22:31
still didn't work
22:35
but since I KNOW that this way of naming log side textures only occurs for The Aether Reborn and BetterEnd(out of the currently supported mods that is), I decided to say fuck it and just made this the if statement:
```
if(modName == Mods.Aether || modName == Mods.AetherShort || modName == Mods.BetterEnd) {/*DO STUFF*/}
```
(edited)
Jack PapelTeam22:36
If it works 🤷‍♂️
Maybe in the future we'll change the naming to be like other mods
meme_man_not_ava22:36
should probably only do so if there's an advantage to it for your own mod
Btw Mods.Aether is the aether reborn's mod ID "the_aether". Before I automated almost everything, I had to create JSONs manually and I refused to continuously type "the_" before "aether", so that's what Mods.AetherShort is.(edited)
Jack PapelTeam22:40
Gotcha
meme_man_not_ava22:41
I'd check if a mod with the id "the_aether" was loaded but I registered my crops as resourcecrops:crop_aether/<material_name>(edited)
so happy I figured out that you can use / when registering a block
it's the only bloody way to sort the blockstates folder into sub-folders
Jack PapelTeam22:42
Pertaining to what you said before, about why you're finding the textures instead of using the block model .jsons, you said that it'd be a lot of manual work, but I don't think any manual work would be required at all for getting the textures from the block models. Of course, I don't want to try to influence you into doing something you don't want. I just don't quite understand(edited)
meme_man_not_ava22:43
I have a very simple reason(edited)
I did not know that you can pull textures from block models
Jack PapelTeam22:43
Oh
meme_man_not_ava22:45
besides what would be the point, if some mod decides to name their log_side material side_log whilst everyone else names it log_side then I'll end up in the same situation where I have to make certain exceptions for a specific mod
Jack PapelTeam22:45
I don't think so
meme_man_not_ava22:45
Jack PapelTeam22:45
As long as they call the blockstate [something]_log
meme_man_not_ava22:46
but it's the BlockModel I need to pull from?
Jack PapelTeam22:46
Or sorry block model not block state
meme_man_not_ava22:47
Is there a function to pull this from block models or would I need to make a block model JSON Serializer+De-Serializer?
Jack PapelTeam22:48
No clue tbh
meme_man_not_ava22:48
Replying tomeme_man_not_avaIs there a function to pull this from block models or would I need to make a block model…
cause that shit is way beyond me
Jack PapelTeam22:49
I'm sure you can import something that does that for you
meme_man_not_ava22:49
true
I'm going to keep it as is for now though, I haven't released a new update in a month due to how long I've been working on this update
Jack PapelTeam22:50
Alright
meme_man_not_ava22:51
Since it's stable-ish now it's more important to push out a new release so I can give people something new to play with whilst I work on the next update
Jack PapelTeam22:51
Makes sense
meme_man_not_ava22:52
Also need to work on updating the growth stages as they're still based on the old crop model
I also added some joke crops as well as crops for all 17 types of wool
Jack PapelTeam22:53
Haha bbq chicken seeds
Seems like an interesting mod
meme_man_not_ava22:53
could've called it cooked chicken but that's no fun
Also when I said 17 types of wool, I meant it. It's not a typo
the thing that minecraft itself lacks
Jack PapelTeam22:55
Oh you even implemented jeb wool blocks
meme_man_not_ava22:56
Not sure if the speed is the same but at least the order of colors is correct
checked the code for that
Jack PapelTeam22:57
Oh I have a meeting in 4 minutes. Your mod seems pretty interesting though
meme_man_not_ava23:00
It's just a resource automation mod, but I try to add as many jokes or neat recipes and references as possible without making the entire mod itself a complete joke
Like I have sand seeds but do we really need glass seeds?
no we don't
but I added it anyway just to make the sand seeds smelt-able in a furnace
which gives you glass seeds
SweepSwap23:10
Replying tomeme_man_not_avaI also added some joke crops as well as crops for all 17 types of wool
add a crippling depression crop 🤣