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

Incompatible with Connected Textures mod #61

Closed
5HT2 opened this issue Jul 6, 2020 · 12 comments
Closed

Incompatible with Connected Textures mod #61

5HT2 opened this issue Jul 6, 2020 · 12 comments
Labels
E-duplicate Closed: Duplicate issue T-bug Type: Bug

Comments

@5HT2
Copy link

5HT2 commented Jul 6, 2020

Expected Behavior

Connected textures mod functions normally.

Actual Behavior

Textures do not connect.

Reproduction Steps

Built sodium on 20w27a via this

Using latest release of CBT from here

  1. Place glass
  2. Observe

System Information

  • Operating System: 5.7.4-arch1-1
  • Java Version: OpenJDK 8 latest AUR build
  • CPU: Intel i7-5600U
  • GPU: Integrated

Additional Context:

I filed an issue in their repo here as I'm not sure which mod is at fault

@jaskarth
Copy link

jaskarth commented Jul 6, 2020

Sodium does not support the Fabric Rendering API yet, so Connected Block Textures won't work.

@5HT2
Copy link
Author

5HT2 commented Jul 6, 2020

I see. So this issue would belong here then?

@broken1arrow
Copy link

Is it build in, soo you can use something similar too clearglass (only borders show, no gitters on the glass)?

When i has try 1.16, is a pain to see so ugly vanilla glass is, so i do not want to use 1.16 before it get fixed.

And yes i use Fabric (vanilla launcher not work properly).

@5HT2
Copy link
Author

5HT2 commented Jul 7, 2020

I'm not sure what clearglass is, and no, the mod isn't built in, as I stated in the reproduction instructions

@broken1arrow
Copy link

https://www.curseforge.com/minecraft/texture-packs/clear-glass-pack

You need optifine for this. So if some or you add something similar, it is assume :) Yes i know this is a mod (some mods work thru vanilla launcher or forge) :)

@5HT2
Copy link
Author

5HT2 commented Jul 7, 2020

That's not a mod, that's a resource pack for glass, and you still need an external mod for it to work, such as CBT, which Sodium breaks.

Meaning, installing that resourcepack is just as useless as not installing it.

@broken1arrow
Copy link

That's not a mod, that's a resource pack for glass, and you still need an external mod for it to work, such as CBT, which Sodium breaks.

Meaning, installing that resourcepack is just as useless as not installing it.

You most have misunderstand or I use wrong worlds.
Yes it is resourcepack, but need opfine, that I ask if you have any similar in Sodium, build in or can add in.

@5HT2
Copy link
Author

5HT2 commented Jul 7, 2020

I'm saying that Sodium breaks connected texture support that other mods provide. That's why I opened a bug report.

@broken1arrow
Copy link

I'm saying that Sodium breaks connected texture support that other mods provide. That's why I opened a bug report.

I´m slow to day :) yes hope it get fixed soon.

@jellysquid3 jellysquid3 added the S-nyi Status: Not yet implemented label Jul 8, 2020
@jellysquid3 jellysquid3 added this to the Release 0.2 milestone Jul 8, 2020
@leo60228
Copy link

Sodium isn't intended to add new features like Optifine does. However, as time goes on, it should become compatible with more and more separate mods that provide those features. The resource pack you linked should work with this mod, but that mod is currently incompatible with Sodium.

@smonnnn
Copy link

smonnnn commented Jul 16, 2020

@leo60228 Support for the fabric rendering api (thus also for CBT) is planned for the next release (0.2) of Sodium.

@amnotbananaama
Copy link
Contributor

After discussing this on discord, this issue is being merged with #5, as it has the same cause and this incompatibility will be fixed when that issue is fixed.

Thank you for the report!

@amnotbananaama amnotbananaama added the E-duplicate Closed: Duplicate issue label Aug 27, 2020
@amnotbananaama amnotbananaama removed this from the Release 0.3 milestone Aug 27, 2020
@amnotbananaama amnotbananaama removed the S-nyi Status: Not yet implemented label Jan 2, 2021
embeddedt referenced this issue in FiniteReality/embeddium Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-duplicate Closed: Duplicate issue T-bug Type: Bug
Projects
None yet
Development

No branches or pull requests

7 participants