Fabricators-of-Create / Create

[Fabric Mod] Building Tools and Aesthetic Technology
MIT License
834 stars 188 forks source link

KubeJS compat and better tags #275

Closed StockiesLad closed 1 year ago

StockiesLad commented 2 years ago

Describe the Suggestion

I was trying to get create to work with other mods' ores, ingots etc. Things like getting the crushing wheel to properly process other ores. All the trial and error to no avail. My concern is that since create (fabric) uses forge tags, it doesn't work at all with anything else besides vanilla. I already assume you guys are aware of this and I'm curious about the plans of mitigation.

Screenshots and Videos

No response

Additional Context

No response

Arathain commented 2 years ago

if you want it to work with other mods, I'd think that editing the base tags yourself would be viable, seeing as none of this is hardcoded

JieningYu commented 2 years ago

try CreatePlus. https://modrinth.com/mod/createplus it can fix issue about tags.

JieningYu commented 2 years ago

try CreatePlus. https://modrinth.com/mod/createplus it can fix issue about tags.

https://github.com/Fabricators-of-Create/Create/pull/245 Pull Request about this

StockiesLad commented 2 years ago

try CreatePlus. https://modrinth.com/mod/createplus it can fix issue about tags.

trying it now.

StockiesLad commented 2 years ago

Oh wait, its for quilt exclusively... Does it really have to be exclusive for quilt?

JieningYu commented 2 years ago

It has a datapack format. https://cdn.modrinth.com/data/svgZx0UN/versions/1.0-beta2/CreatePlus%20Datapack.zip

StockiesLad commented 2 years ago

oh ok thanks!

StockiesLad commented 2 years ago

Any plans to have fabric support in your mod addon.

JieningYu commented 2 years ago

Any plans to have fabric support in your mod addon.

It's easy to port(you can port yourself or use Quilt. Most of Fabric mods can play on Quilt.

StockiesLad commented 2 years ago

Any plans to have fabric support in your mod addon.

It's easy to port(you can port yourself or use Quilt. Most of Fabric mods can play on Quilt.

quilt is very buggy. I'm building from source anyways so it's all good.

JieningYu commented 2 years ago

Any plans to have fabric support in your mod addon.

It's easy to port(you can port yourself or use Quilt. Most of Fabric mods can play on Quilt.

quilt is very buggy. I'm building from source anyways so it's all good.

note that the source doesn't include createplus-core(datapack) by default.

StockiesLad commented 2 years ago

[B#744] Caused by: org.spongepowered.asm.mixin.injection.throwables.InvalidInjectionException: Critical injection failure: @Inject annotation on port_lib$clientStartTickEvent could not find any targets matching 'Lnet/minecraft/class_1657;tick()V' in net.minecraft.class_1657. Using refmap porting_lib-refmap.json [PREINJECT Applicator Phase -> porting_lib.mixins.common.json:PlayerMixin from mod porting_lib -> Prepare Injections -> -> handler$zeg000$port_lib$clientStartTickEvent(Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V -> Parse]

It spits out this error when I use the compiled version.

JieningYu commented 2 years ago

[B#744] Caused by: org.spongepowered.asm.mixin.injection.throwables.InvalidInjectionException: Critical injection failure: @Inject annotation on port_lib$clientStartTickEvent could not find any targets matching 'Lnet/minecraft/class_1657;tick()V' in net.minecraft.class_1657. Using refmap porting_lib-refmap.json [PREINJECT Applicator Phase -> porting_lib.mixins.common.json:PlayerMixin from mod porting_lib -> Prepare Injections -> -> handler$zeg000$port_lib$clientStartTickEvent(Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V -> Parse]

It spits out this error when I use the compiled version.

how you compile?

StockiesLad commented 2 years ago

I changed the port lib hash to the new one.

JieningYu commented 2 years ago

createplus don't need port lib and you can use the latest version of create fabric directly.

StockiesLad commented 2 years ago

I was talking about ur PR of create

JieningYu commented 2 years ago

fetched upstream

StockiesLad commented 2 years ago

fetched upstream

Is it getting merged now? Or is it already merged.

StockiesLad commented 2 years ago

oh i saw...

StockiesLad commented 2 years ago

latest.log It causes a porting lib issue tho...

StockiesLad commented 2 years ago

Player Mixin is failing

JieningYu commented 2 years ago

?

JieningYu commented 2 years ago

if it's a mixin issue, report it to TropheusJ

StockiesLad commented 2 years ago

if it's a mixin issue, report it to TropheusJ

ok

TropheusJ commented 1 year ago

KubeJS support: https://www.curseforge.com/minecraft/mc-mods/kubejs-create we will be handling tags with the 1.19 update.

JieningYu commented 1 year ago

pog

TropheusJ commented 1 year ago

tag change happened