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

feat: network transmitter #450

Merged
merged 5 commits into from
Nov 3, 2023
Merged

Conversation

raoulvdberge
Copy link
Contributor

The network transmitter now has an indicator
on the block state whether the connection with the receiver is active.
With not enough energy, it's inactive.
Without network card or without connection, it's in errored state. Otherwise, it's in active state.

The network transmitter will actively retry to reconnect if the connection is lost (for example due to chunk unloading).

Fixes #97

The network transmitter now has an indicator
 on the block state whether the connection with the receiver is active.
With not enough energy, it's inactive.
Without network card or without connection, it's in errored state.
Otherwise, it's in active state.

The network transmitter will actively retry to reconnect
if the connection is lost (for example due to chunk unloading).
This bug made it so that every time a network device is placed, it would trigger an extra update
after the initial update to connect
to an network.

This extra update was caused by the fact that the device was
going from inactive to active after the connection was made.

We now only update the network if the direction has changed, since that is the
only component that can influence graph scanning for now.

This bug also caused some flickering with the network transmitter
in combination with redstone mode.
@raoulvdberge raoulvdberge force-pushed the feat/GH-97/network-transmitter branch from 0690b89 to 1c2323d Compare November 3, 2023 18:29
@raoulvdberge raoulvdberge marked this pull request as ready for review November 3, 2023 19:50
@raoulvdberge raoulvdberge force-pushed the feat/GH-97/network-transmitter branch from f447ecd to f7de1e9 Compare November 3, 2023 20:30
Copy link

sonarqubecloud bot commented Nov 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
1.6% 1.6% Duplication

@raoulvdberge raoulvdberge merged commit c782f9e into develop Nov 3, 2023
7 checks passed
@raoulvdberge raoulvdberge deleted the feat/GH-97/network-transmitter branch November 3, 2023 20:40
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Network Transmitter
1 participant