Skip to content

Infinite Energy Emitter (from CEU)#21

Merged
serenibyss merged 4 commits intoGregTechCEu:masterfrom
bruberu:creativeEnergy
Jul 6, 2021
Merged

Infinite Energy Emitter (from CEU)#21
serenibyss merged 4 commits intoGregTechCEu:masterfrom
bruberu:creativeEnergy

Conversation

@bruberu
Copy link
Copy Markdown
Member

@bruberu bruberu commented Jul 5, 2021

What:
This particular PR addresses an issue in testing where it is often difficult to test certain machines and recipes without a singleblock energy source, which becomes an especially pronounced problem for the higher tiers.

How solved:
To address this, I have imported the Infinite GTEU Energy Emitter from CEU. For the most part, this implementation was copied entirely, although I did decide to do a little refactoring with the GUI builder, since I could not find an easy way to work around it. Later on, the code there should probably be cleaned up, although I do not have the skills to pull off such a thing. Furthermore, a rather large excess of extraneous classes, used for the rest of CEU's infinite energy handling tile entities, have also been imported, to serve both the purpose of leaving the door open for importing the rest in, and to serve the purpose of making this less of a headache for me to handle.

Outcome:
Adds a new MTE for infinite GTEU generation.

Possible compatibility issue:
None are expected.

@serenibyss
Copy link
Copy Markdown
Member

If you are using Intellij, can you please run the Reformat Code function on these files, excluding en_us.lang. You can find it by right-clicking on the files themselves on the left

@bruberu
Copy link
Copy Markdown
Member Author

bruberu commented Jul 5, 2021

Cleaned up code in the latest commit, excluding en_us.lang.

@bruberu
Copy link
Copy Markdown
Member Author

bruberu commented Jul 6, 2021

Fixed all merge conflicts.

@serenibyss serenibyss merged commit be14a10 into GregTechCEu:master Jul 6, 2021
@bruberu bruberu deleted the creativeEnergy branch July 6, 2021 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants