Team-EnderIO/EnderIO
1
0
Fork 0
Code Issues 114 Pull requests 13 Projects Releases 79 Packages Wiki Activity Actions

Conduit Cleaning #296

Merged
Rover656 merged 24 commits from feature/conduit-cleaning into dev/1.20.1 2023-07-24 19:14:47 +00:00
Conversation 1 Commits 24 Files changed 39 +502 -290
Rover656 commented 2023-07-12 11:49:44 +00:00 (Migrated from github.com)
Copy link

Description

I am opening this to track the changes made in our common conduit cleaning branch. Please add TODO's below as you both see fit [agnor/ferri].

Todo

  • Improve API for sending updates for extended conduit data client -> server.
  • Add insert/extract labels to the GUI
  • Change max conduits down to 4.
  • Track and fix any breakages of the graph (maybe add checks to disable the fixer during dev)
  • Add config to disable extended logging (and move it to debug.log)
  • Ensure (as much as reasonably possible) that old worlds can upgrade with few problems, if any.
  • General code cleaning
  • Add support for different default connection configurations #309

Checklist:

  • My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
  • I have performed a self-review of my own code.
  • I have commented my code in areas it may be challenging to understand.
  • I have made corresponding changes to the documentation.
  • My changes are ready for review from a contributor.
# Description I am opening this to track the changes made in our common conduit cleaning branch. Please add TODO's below as you both see fit [agnor/ferri]. <!-- Remove this section if you're submitting an already-complete PR --> # Todo - [x] Improve API for sending updates for extended conduit data client -> server. - [x] Add insert/extract labels to the GUI - [x] Change max conduits down to 4. - [x] Track and fix any breakages of the graph (maybe add checks to disable the fixer during dev) - [x] Add config to disable extended logging (and move it to debug.log) - [x] Ensure (as much as reasonably possible) that old worlds can upgrade with few problems, if any. - [x] General code cleaning - [x] Add support for different default connection configurations #309 <!-- For drafts, fill this in as you go; if you are leaving draft, make sure these are all done --> # Checklist: - [ ] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you). - [ ] I have performed a self-review of my own code. - [ ] I have commented my code in areas it may be challenging to understand. <!-- (Although we prefer code that is readable instead of over-commented) --> - [ ] I have made corresponding changes to the documentation. - [ ] My changes are ready for review from a contributor. <!-- Thanks to: https://embeddedartistry.com/blog/2017/08/04/a-github-pull-request-template-for-your-projects/ for the building blocks of this template -->
Rover656 (Migrated from github.com) reviewed 2023-07-24 18:53:20 +00:00
Rover656 (Migrated from github.com) left a comment
Copy link

I'm really happy with the look of this; thanks agnor for tackling all of this!!

I'm really happy with the look of this; thanks agnor for tackling all of this!!
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Area-Assets

   
Area-Backend

Backend work not usually visible to players.

  
Area-Conduits

Everything involving conduits

  
Area-Datapacks

   
Area-Lang

   
Area-Mod Compat

   
Area-Parity

1.12.2 missing features

  
Area-Rendering

Rendering code, might be shaders specific or general.

  
Good first issue

   
MC-1.19.2

   
MC-1.20.1

   
MC-1.20.4

   
MC-1.20.6

   
MC-1.21

   
MC-1.21.1

   
Modtoberfest

   
P-0-High

High priority, crash/breaking.

  
P-1-Medium

Medium priority. Potentially deleting items or breaking functionality.

  
P-2-Low

Low priority. Not hurting anyone, just annoying.

  
Status-Awaiting Response

Waiting for OP to respond

  
Status-Behind-Flag

This change is available but is placed behind a feature flag.

  
Status-Blocked

Blocked by a specific issue or protocol.

  
Status-Cannot Reproduce

   
Status-Duplicate

This issue or pull request already exists

  
Status-Help Wanted

We need some help to try and track this thing down..

  
Status-Incomplete Report

   
Status-Invalid

This doesn't seem right

  
Status-Needs LTS Backport

Fixed in latest but requires LTS backport.

  
Status-Needs Updating

Needs updating to a newer Minecraft version.

  
Status-Stale

This may not be updated by the OP any further.

  
Status-To Implement

   
Status-Triage

A Triage Member should take a look at this

  
Status-Wontfix

This will not be worked on

  
Status-Wontmerge

This PR won't be merged, but is left now for reference of ongoing or future works.

  
Type-Backport

   
Type-Bug

There is a problem.

  
Type-Documentation

Improvements or additions to documentation

  
Type-Enhancement

New feature or enhancement to existing feature.

  
Type-Question

Further information is requested

  
Type-RFC

Request for comments.

  
Type-Suggestion
No labels
Area-Assets
Area-Backend
Area-Conduits
Area-Datapacks
Area-Lang
Area-Mod Compat
Area-Parity
Area-Rendering
Good first issue
MC-1.19.2
MC-1.20.1
MC-1.20.4
MC-1.20.6
MC-1.21
MC-1.21.1
Modtoberfest
P-0-High
P-1-Medium
P-2-Low
Status-Awaiting Response
Status-Behind-Flag
Status-Blocked
Status-Cannot Reproduce
Status-Duplicate
Status-Help Wanted
Status-Incomplete Report
Status-Invalid
Status-Needs LTS Backport
Status-Needs Updating
Status-Stale
Status-To Implement
Status-Triage
Status-Wontfix
Status-Wontmerge
Type-Backport
Type-Bug
Type-Documentation
Type-Enhancement
Type-Question
Type-RFC
Type-Suggestion
Milestone
Clear milestone
No items
No milestone
The Great Squashening
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Team-EnderIO/EnderIO#296
No description provided.
Delete branch "feature/conduit-cleaning"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?