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

AE2 BC Builder Integration #942

Closed
Bestadd opened this Issue Feb 25, 2015 · 8 comments

Comments

Projects
None yet
4 participants
@Bestadd

Bestadd commented Feb 25, 2015

So last night I was working on a funky locomotion miner using annihilation planes (inspired by DW20) The base of this build (bottom two layers, top layer of frame blocks and the bottom being AE2 cables, interfaces, storage bus and annihilation planes). Just getting two rows of the 18 rows done took me a while so I thought lets create a BC blue print and then use the path markers to let the builder build the remaining rows. Sadly the builder wont place AE2 cables or parts. As a low priority future enhancement would it be possible to get AE2 to work with the BC Builder.

I'm just a user, so this may be a BC problem. In which case let me know and I'll ask the BC devs.

This would also be amazing for building SSD or extending them.

Just a thought I wanted to through out as I know there is already some integration with AE2 and BC when it comes to BC pipes.

However, just in case this actually should have worked and is a config option or mod interaction problem I was in single player FTB Infinity 1.0.2

@AlgorithmX2

This comment has been minimized.

Show comment
Hide comment
@AlgorithmX2

AlgorithmX2 Feb 26, 2015

Contributor

As far as I know AE2 already fully supports the Builder, unless something has changed with its API, or a bug was introduced.

Contributor

AlgorithmX2 commented Feb 26, 2015

As far as I know AE2 already fully supports the Builder, unless something has changed with its API, or a bug was introduced.

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Feb 26, 2015

Member

There was a major change in the API with 6.3 in general, so I would expect that the Builder has probably changed too. But I am not sure, if we can migrate everything. If I remember correctly, there was no way through the API to support BC Facades as currently with <6.3.

Member

yueh commented Feb 26, 2015

There was a major change in the API with 6.3 in general, so I would expect that the Builder has probably changed too. But I am not sure, if we can migrate everything. If I remember correctly, there was no way through the API to support BC Facades as currently with <6.3.

@thatsIch thatsIch self-assigned this Mar 9, 2015

@thatsIch thatsIch added the state-fixed label Mar 9, 2015

@thatsIch thatsIch added this to the rv2 milestone Mar 9, 2015

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Mar 9, 2015

Member

I think it is fixed, but I have no clue how the builder works, so somebody needs to test this.

Member

thatsIch commented Mar 9, 2015

I think it is fixed, but I have no clue how the builder works, so somebody needs to test this.

@thatsIch thatsIch added type-bug and removed type-enhancement labels Mar 9, 2015

@Bestadd

This comment has been minimized.

Show comment
Hide comment
@Bestadd

Bestadd Mar 10, 2015

@thatslch, since I'm the one who requested it it makes since I should test
it. I'm not setup at all for Java development (all of my work currently is
PHP and C#) and have not been for quite a few years, would it be possible
for you to produce a build?

On Mon, Mar 9, 2015 at 5:14 AM, thatsIch notifications@github.com wrote:

I think it is fixed, but I have no clue how the builder works, so somebody
needs to test this.


Reply to this email directly or view it on GitHub
#942 (comment)
.

Ross Lanes

Bestadd commented Mar 10, 2015

@thatslch, since I'm the one who requested it it makes since I should test
it. I'm not setup at all for Java development (all of my work currently is
PHP and C#) and have not been for quite a few years, would it be possible
for you to produce a build?

On Mon, Mar 9, 2015 at 5:14 AM, thatsIch notifications@github.com wrote:

I think it is fixed, but I have no clue how the builder works, so somebody
needs to test this.


Reply to this email directly or view it on GitHub
#942 (comment)
.

Ross Lanes

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Mar 10, 2015

Member

The build is produced by TeamCity, just download it from there.
The build server is always linked to each build

go to the PR #992
and click the green hook next to the commit http://puu.sh/gubHU/d63696fe80.png

Member

thatsIch commented Mar 10, 2015

The build is produced by TeamCity, just download it from there.
The build server is always linked to each build

go to the PR #992
and click the green hook next to the commit http://puu.sh/gubHU/d63696fe80.png

@Bestadd

This comment has been minimized.

Show comment
Hide comment
@Bestadd

Bestadd Mar 10, 2015

@thatsIch may take me several days to test. However, I'll report back on what I find as I find it by updating this comment. My plan is to test this build with the specified version of forge and BC in an isolated MC instance both SP and Server. If that all goes well I'll try updating the FRB infinity and DW20 packs and see if there are any problems there.

Bestadd commented Mar 10, 2015

@thatsIch may take me several days to test. However, I'll report back on what I find as I find it by updating this comment. My plan is to test this build with the specified version of forge and BC in an isolated MC instance both SP and Server. If that all goes well I'll try updating the FRB infinity and DW20 packs and see if there are any problems there.

@thatsIch thatsIch closed this in e300bf9 Mar 10, 2015

thatsIch added a commit that referenced this issue Mar 10, 2015

Merge pull request #992 from thatsIch/e-319-new-bc
Fixes #743 crash with plugs, fixes #942 builder integration, closes #319 BC 6 prep
@Bestadd

This comment has been minimized.

Show comment
Hide comment
@Bestadd

Bestadd Mar 11, 2015

@thatsIch it works in both single player and server with BC and AE2 only. When I created this issue I did not expect it to be fixed for a few months, the AE2 team is truly amazing, thank you.

Bestadd commented Mar 11, 2015

@thatsIch it works in both single player and server with BC and AE2 only. When I created this issue I did not expect it to be fixed for a few months, the AE2 team is truly amazing, thank you.

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Mar 11, 2015

Member

tbh the fix was already there a few month ago, but I kinda wanted to improve the usage of the new BC API, but sadly its still restricting us on its usage, so we pushed for the fix first before working on enhancements.

Member

thatsIch commented Mar 11, 2015

tbh the fix was already there a few month ago, but I kinda wanted to improve the usage of the new BC API, but sadly its still restricting us on its usage, so we pushed for the fix first before working on enhancements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment