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

setBundledOutput not working with EnderIO Redstone Conduit #2958

Closed
kman3107 opened this issue Oct 10, 2018 · 6 comments
Closed

setBundledOutput not working with EnderIO Redstone Conduit #2958

kman3107 opened this issue Oct 10, 2018 · 6 comments
Assignees
Labels
make-a-pr-if-you-want-this Feature request expired. PRs generally welcome

Comments

@kman3107
Copy link

kman3107 commented Oct 10, 2018

Forge: 14.23.4.2760
Mods:
EnderIO-1.12.2-5.0.36 (Current Relase)
EnderCore-1.12.2-0.5.41 (Current Release)
OpenComputersMC1.12.2-1.7.2.113 (Current Dev Build, also checked with current release 1.7.2.67)

I've been trying to use setBundledOutput with EnderIO conduits with no success.
I am using the same OC setup, and the exact same script as I used in MC1.10.
Which back then worked just fine.
2018-10-10_21 08 53
2018-10-10_21 09 04
2018-10-10_21 09 17

This is it working just fine in MC 1.10
Forge: 12.18.3.2316
Mods:
EnderIO-.1.10.2-3.1.183
EnderCore-1.10.2-0.4.1.66
OpenComputersMC1.10.2-1.6.2.7

2018-10-10_21 31 20

@payonel
Copy link
Member

payonel commented Oct 10, 2018

thank you very much for testing with our dev builds!!! seriously, thank you

@payonel payonel added this to the v1.7.3 milestone Oct 10, 2018
@payonel
Copy link
Member

payonel commented Oct 11, 2018

this comes down to enderio integration
you'll notice that when you open the enderio redstone conduit on the face against the OC machine, the 1.12 GUI shows you a color selector but in 1.10 there is no color selector

The EnderIO code is checking to see if our block implements their crazypants.enderio.conduit.redstone.IRedstoneConduit interface. And if it does, that means we can speak their bundled redstone language.

I'll review what it takes to bring this back in for 1.12, but it might fall out of the 1.7.3 milestone if it isn't trivial

@kman3107
Copy link
Author

I thought the new EnderIO Conduit setup might have something to do with it.
Thanks for looking into to it. I hope it's fairly easy to work around, if not then it's not ment to be :)

@kman3107
Copy link
Author

I just checked Thermal Dynamics Redstone Relay and it has the same problem as EnderIO Redstone Conduits.

@payonel payonel removed this from the v1.7.3 milestone Oct 11, 2018
@payonel
Copy link
Member

payonel commented Oct 11, 2018

btw, enderio 1.12 doesn't yet have a redstone api that we can hook into

@selli69
Copy link

selli69 commented May 26, 2019

Any news on this?

@payonel payonel self-assigned this Jan 28, 2020
@payonel payonel added make-a-pr-if-you-want-this Feature request expired. PRs generally welcome and removed bug-minor labels May 11, 2021
@payonel payonel closed this as completed May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
make-a-pr-if-you-want-this Feature request expired. PRs generally welcome
Projects
None yet
Development

No branches or pull requests

3 participants