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

[1.20.1] Forge - Issue with textures #210

Open
MrPYF opened this issue Jul 5, 2024 · 14 comments
Open

[1.20.1] Forge - Issue with textures #210

MrPYF opened this issue Jul 5, 2024 · 14 comments
Labels
bug Something isn't working

Comments

@MrPYF
Copy link

MrPYF commented Jul 5, 2024

Issues with Inferium/Prosperity (Mystical Agriculture) and Brilliant Stone (Malum) in Andesite/Granite/Diorite/Tuff not rendering properly in world or as an item.
Drops are good, just a texture issue.

Blocks in world
Items in inventory

@lukebemish lukebemish added the bug Something isn't working label Jul 5, 2024
@MrPYF
Copy link
Author

MrPYF commented Jul 27, 2024

Hello,

Small update as I was working around my modpack again, it appears that Inferium/prosperity (Mystical Agriculture) and Brilliant Stone (Malum), actually have a texture issue with all those blocks :

  • Diorite (vanilla)

  • Andesite (vanilla)

  • Granite (vanilla)

  • Tuff (vanilla)

  • Smooth Basalt (vanilla)

  • Dripstone Block (vanilla)

  • Sandstone (vanilla)

  • Red Sandstone (vanilla)

  • Shale (Quark)

  • Jasper (Quark)

  • Limestone (Quark)

  • Scorchia (Create)

  • Scoria (Create)

  • Limestone (Create)

  • Crimsite (Create)

  • Calcite (Create)

  • Asurine (Create)

  • Ochrum (Create)

  • Veridium (Create)

  • White Sandstone (Biomes O' Plenty)

  • Orange Sandstone (Biomes O' Plenty)

Haven't noticed any other ore than those 3 having an issue.
Hope it will help figuring out the issue.

Regard.

2024-07-27_15 38 41

@lukebemish
Copy link
Member

Yep, okay. I think I know what's going on here -- I could swear I'd fixed this already but I guess not. I'm planning to release a new release for 1.20.1 in a week or so; it'll hopefully have a fix for this too.

@VictoriaSilver
Copy link

A quick workaround that I've found for the meantime is

  1. Disable all mods except Dynamic Asset Generator, Excavated Variants, and any of your mods which EV supports
  2. Open config/dynamic_asset_generator.json and change both cache_assets and cache_data to true
  3. Start the game, let it get to the title screen, close the game
  4. Re-enable all of your mods

You'll need to repeat this if you add any other mods supported by EV, but that's all.

@TorchTheDragon
Copy link

TorchTheDragon commented Sep 18, 2024

Hate to bring this back up, but this also exists in NeoForge 1.21.1
image

@lukebemish
Copy link
Member

Hmm. Just to confirm, the only mods needed to see this issue are Excavated Variants, Dynamic Asset Generator, and Malum or Mystical Agriculture?

@TorchTheDragon
Copy link

I can test that real quick with just those three mods (DAG, EV, and MA)

@TorchTheDragon
Copy link

Here are the mods installed in the test build

image

Still Occurs

image

@lukebemish
Copy link
Member

Thanks, I'll give it a poke; pretty sure I know what the issue is, just have to figure out why it broke again.

@jadynbrammer
Copy link

jadynbrammer commented Oct 9, 2024

Running in to the same issue, did some testing.

Configuration 1: EV + DAG + Malum + Mystical Agriculture Results:

  • error texture (magenta/black) blocks for all EV versions of Malum ores.
  • stone texture only (missing ore overlay) for MA ores.

Configuration 2: EV + DAG + Mystical Agriculture Results:

  • Everything working normally
  • I must have been mistaken about this, I think I saw one of the stock MA ores perhaps? No, MA is still broken.

Configuration 3: EV + DAG + Malum Results:

  • error texture (magenta/black) blocks for all EV versions of Malum ores.

The "trigger" seems to be the Malum ores specifically, which I'm guessing might have something to do with the glow and/or animation frames they have. It's the only thing that stands out as different about them.

There's also this funny little anomaly:
image

Somehow EV/DAG amalgamated 'mystical agriculture', 'soulstone' (from malum), and 'blazing quartz ore' (also from malum), into a single block? Very strange! :P

And, just in case it helps, here's one of the exceptions DAG spits out in relation to the broken textures:

[23:17:23] [Worker-Main-15/ERROR] [dynamic_asset_generator/]: Could not write image to stream for source {"sources":{"stoneNew0":{"source":{"path":"minecraft:block/sandstone_top","type":"dynamic_asset_generator:texture"}},"ore0":{"source":{"path":"minecraft:blank","type":"dynamic_asset_generator:texture"}},"ore1":{"source":{"path":"minecraft:blank","type":"dynamic_asset_generator:texture"}},"stoneOld0":{"source":{"path":"minecraft:block/stone","type":"dynamic_asset_generator:texture"}}},"generator":{"background":{"sources":[{"capture":"stoneOld0","type":"dynamic_asset_generator:frame_capture"}],"type":"dynamic_asset_generator:overlay"},"full":{"sources":[{"capture":"ore0","type":"dynamic_asset_generator:frame_capture"},{"capture":"ore1","type":"dynamic_asset_generator:frame_capture"}],"type":"dynamic_asset_generator:overlay"},"new_background":{"sources":[{"capture":"stoneNew0","type":"dynamic_asset_generator:frame_capture"}],"type":"dynamic_asset_generator:overlay"},"type":"dynamic_asset_generator:foreground_transfer"},"type":"dynamic_asset_generator:animation_splitter"}: excavated_variants:textures/block/sandstone_brilliant_stone_0_2.png
java.io.IOException: Issue loading texture: minecraft:blank
	at dev.lukebemish.dynamicassetgenerator.api.client.generators.texsources.TextureReaderSource.lambda$getSupplier$1(TextureReaderSource.java:56) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.client.TexSourceCache.lambda$fromCache$1(TexSourceCache.java:47) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.CacheReference.calcSync(CacheReference.java:16) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.client.TexSourceCache.fromCache(TexSourceCache.java:44) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.client.TexSourceCachingWrapper.lambda$getSupplier$2(TexSourceCachingWrapper.java:38) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.api.client.generators.texsources.AnimationSplittingSource.lambda$getSupplier$4(AnimationSplittingSource.java:69) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.client.TexSourceCache.fromCache(TexSourceCache.java:37) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.impl.client.TexSourceCachingWrapper.lambda$getSupplier$2(TexSourceCachingWrapper.java:38) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.api.client.generators.TextureGenerator.lambda$get$3(TextureGenerator.java:52) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at dev.lukebemish.dynamicassetgenerator.api.ResourceCache.lambda$wrapSafeData$2(ResourceCache.java:127) ~[0_Library_DynamicAssetGenerator.jar%23162!/:4.2.2] {re:classloading}
	at net.minecraft.server.packs.resources.Resource.m_215507_(Resource.java:46) ~[client-1.20.1-20230612.114412-srg.jar%23168!/:?] {re:classloading}
	at net.minecraft.client.renderer.texture.SpriteLoader.m_245083_(SpriteLoader.java:138) ~[client-1.20.1-20230612.114412-srg.jar%23168!/:?] {re:classloading,pl:runtimedistcleaner:A}
	at net.minecraft.client.renderer.texture.atlas.SpriteSource$Output.m_261059_(SpriteSource.java:22) ~[client-1.20.1-20230612.114412-srg.jar%23168!/:?] {re:classloading,re:mixin}
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1768) ~[?:?] {}
	at java.util.concurrent.CompletableFuture$AsyncSupply.exec(CompletableFuture.java:1760) ~[?:?] {}
	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373) ~[?:?] {}
	at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182) ~[?:?] {}
	at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655) ~[?:?] {}
	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1622) ~[?:?] {}
	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:165) ~[?:?] {}

I'll keep Malum out of my pack until this is resolved. Hope this helps! Thank you!

@lukebemish
Copy link
Member

Hmm... That is a weird error message. I bet one of the mods in question has changed how they handle models since I last had to poke this. I'll give it a poke. Thanks for the info!

@jadynbrammer
Copy link

Aaaand after turning on all of my other mods, Mystical Agriculture EV ores broke again. (Same way, base stone texture with no ore spots)

Gotta go to bed now but tomorrow I'll bisect my mod list and try to track down what's causing it. No exceptions in the logs for any of the MA ores, so no smoking guns to make it easy, unfortunately.

@jadynbrammer
Copy link

Ok, so, completely isolated MA+EV breaks in a brand new Forge47.3.0+mc1.20.1 instance. I must have been mistaken earlier about them working. It was late :P

@jadynbrammer
Copy link

jadynbrammer commented Oct 9, 2024

One last thing, I went ahead and generated clean instance logs for you:

EV + MA:
https://mclo.gs/FWsFPl5

EV + Malum:
https://mclo.gs/J0EEcI1

@lukebemish
Copy link
Member

Thanks! I'll poke it when I have the chance -- I have some ideas what's going on here I'll need to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants