Suggestion: Add Agricraft compatability for the new Harvesting Drone

Question

Suggestion: Add Agricraft compatability for the new Harvesting Drone

wrincewind opened this issue 6 years ago · 9 comments

wrincewind commented 6 years ago

Currently the drone just sits there, floating next to my fully grown crops like some big fat idiot. >:I

[TBF I have a nice little program for doing this with proper drones, but... it'd be nice, right?

desht · Answer 1 · 2018-07-10T09:40:08.000Z

Yeah, I think this shouldn't be too hard. Looks like Agricraft crops are just a tile entity with some handy methods for checking maturity and doing harvesting. @MineMaarten wrote a nice extensible system for crop harvesting so plugging in Agricraft should be fairly straightforward.

wrincewind · Answer 2 · 2018-07-10T09:44:50.000Z

Brilliant, thanks! Now I just need to pass this onto EnderIO, thaumcraft, and Industrial foregoing, 'cause none of them seem to know how to do it either :p

desht · Answer 3 · 2018-07-10T10:01:09.000Z

Actually, no - I was wrong. The example I was looking at (Botania Tweaks) uses ASM to patch Agricraft to make the crops harvestable. I'm not touching ASM with a bargepole...

wrincewind · Answer 4 · 2018-07-10T10:41:16.000Z

Blast, I had a feeling it couldn't be that easy... I'll go and pester the Agricraft devs to see if they can add something.

desht · Answer 5 · 2018-07-10T11:04:55.000Z

I'll continue to research. I'm pretty sure there's a way...

wrincewind · Answer 6 · 2018-07-10T15:44:56.000Z

Well, fingers crossed...!

Sewef · Answer 7 · 2018-07-10T12:35:34.000Z

Back to 2017, with Agricraft 1.10.2 (alpha build), I went into compatibility issues : machines from others mods couldn't harvest crops without using a FakePlayer (Mechanical User from Ex2, Drone from PNC, Turtle from CC, Robot from OC)... I checked the Agricraft Github and they just removed all mod compat with 1.10 update and they were going to re-add them.
As Agricraft 1.12.2 is still in alpha, it has the same non-compat problem, I guess. That's one the the reasons that made me uninstall this mod :l

wrincewind · Answer 8 · 2018-07-10T12:36:57.000Z

Buggery. Maybe I should swap it for magical crops or something similar after all, then... Its that or I start handing out custom-made farmer drones like frigging candy :I

desht · Answer 9 · 2018-07-10T12:43:13.000Z

I'm looking at https://github.com/AgriCraft/AgriCraft/tree/master/src/main/java/com/infinityraider/agricraft/api/v1 now, and it does appear to have the necessary support for checking if crops are mature and/or harvestable. It looks workable, but I won't know until I've tried some integration...

Share to