Skip to content

[ref:seggan/feature/elektrikity] ELEKTRIKITY - #971

Open
Seggan wants to merge 105 commits into
idra/boilersfrom
seggan/feature/elektrikity
Open

[ref:seggan/feature/elektrikity] ELEKTRIKITY#971
Seggan wants to merge 105 commits into
idra/boilersfrom
seggan/feature/elektrikity

Conversation

@Seggan

@Seggan Seggan commented May 5, 2026

Copy link
Copy Markdown
Member

See #987 section "Foundational Pylon PR"

Seggan and others added 5 commits July 3, 2026 14:26
Co-authored-by: JustAHuman-xD <65748158+JustAHuman-xD@users.noreply.github.com>
…e/elektrikity

# Conflicts:
#	src/main/java/io/github/pylonmc/pylon/PylonFluids.java
#	src/main/java/io/github/pylonmc/pylon/content/components/FluidHatch.java
#	src/main/java/io/github/pylonmc/pylon/content/components/FluidInputHatch.java
#	src/main/java/io/github/pylonmc/pylon/content/components/FluidOutputHatch.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/diesel/machines/DieselCoreDrill.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/diesel/machines/PalladiumCondenser.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/diesel/production/Biorefinery.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/experience/FluidExperienceBottler.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/hydraulics/BurnerHydraulicPurifier.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/hydraulics/ConvectionHydraulicPurifier.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/hydraulics/HydraulicCoreDrill.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/hydraulics/LiseletteHydraulicPurifier.java
#	src/main/java/io/github/pylonmc/pylon/content/machines/smelting/Kiln.java
#	src/main/resources/lang/en.yml
#	src/main/resources/recipes/minecraft/crafting_shaped.yml
#	src/main/resources/recipes/pylon/assembling.yml
@Seggan
Seggan changed the base branch from master to idra/boilers July 3, 2026 18:55
@Seggan
Seggan requested a review from JustAHuman-xD July 7, 2026 14:57
@Seggan

Seggan commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

Quarries appear to now be trying to mine air? Not 100% sure if that was introduced by this PR

This is fine cause getBreakTicks returns null for air anyway. But I'll exclude it explicity.

Seggan added 3 commits July 7, 2026 16:35
…e/elektrikity

# Conflicts:
#	src/main/java/io/github/pylonmc/pylon/PylonFluids.java
#	src/main/resources/lang/en.yml
#	src/main/resources/recipes/minecraft/crafting_shaped.yml
@Seggan
Seggan requested a review from LordIdra August 25, 2026 16:47
@Seggan

Seggan commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

Added displays/animations to every iron block machine except brick molder and pipe bender

@LordIdra LordIdra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing fluid keys

Image

Capacitors do not appear to work

Comment on lines +2658 to +2660
machines_electricity_production: "<yellow>Electricity Production"
machines_electric_machines: "<yellow>Electric Machines"
machines_electricity_utilities: "<yellow>Electricity Utilties"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How come these are yellow?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚡⚡⚡

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be good to display power production as its own item, since it's a bit hidden down at the bottom of the turbine lore?

Also:

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having the turbine go from 0 to 60 is a bit jarring, and it stutters if it doesn't have enough input. How much bother would it be to make the speed scale? It doesn't have to be complicated, I'm just thinking constant acceleration/deceleration until stopped / at max speed

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'Bus' is the wrong word for this 'hub' maybe?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the use case of this when any number of wires can connect to a node?

I can see a connector being useful but having 6 ports feels strange given the above

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wires can't go through blocks/bypassing the wire length limit

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I learnt in Quaptics, updating text display text can be really quite expensive, so I'd suggest maybe we limit the text display to only one side and/or make it toggleable? (Since I can imagine if players build a big capacitor bank they might want to turn this off)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to show contents in WAILA too

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.

4 participants