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

Totally random thoughts after spending a full evening using exp pipes. #3

Closed
4 tasks done
BrewingCoder opened this issue Apr 26, 2019 · 5 comments
Closed
4 tasks done
Labels
feature New feature

Comments

@BrewingCoder
Copy link
Collaborator

BrewingCoder commented Apr 26, 2019

Feel free to just close this out of hand, let me know if you want any of these items opened into separate issues, or if you're open to pull requests:

  1. what I found most troublesome when attempting to create a pipe network was the lack of polymorphic-sink. The provider pipe only provides 9 filter locations for sink items. This makes the pipe useless on simple storage multi-block networks like Storage Drawers, Yabba, etc.
  • Done

*this also applies to mass sink machines like say ore crushing/washing where you want to sink a large number of items. This used to be slapping on a LP MK-V Chassis with many sink modules in it. Unfortunately a poly-sink wouldn't solve that issue. In my test world I made multiple chests with many sinks, then a separate network to extract all those into the machine, but it wasn't ideal.

  1. Even with a polymorphic sink of some type a provider needs an option to leave at least 1, or one stack, of an item in it's connected inventory. This prevents those connected storage devices from potentially unlocking it's storage slot for that item.
  • Done
  1. I still have issues with routing. Not as bad as before your last update (which was amazing, more on that in a moment). The network routing appears great but it doesn't always pick the default route. On my test system I only have one default route, which is a chest. It's purpose is to catch everything that doesn't have a specific route. In places I have Yabba barrel networks and Storage drawer networks with provider pipes. These pipes DO NOT have default route set, and have specific item sink entries.
  • Fixed

If I put an empty barrel on the Yabba network, which is closer to the extraction point than the Storage Drawers network, the pipes will route something into the empty barrel. It shouldn't do this right? the provider pipe doesn't have an entry for it. There is a provider pipe about 10 blocks away on the Storage drawers network, but it ignored that in favor of the empty barrel.

  1. You totally conquered the issue I and others were having with large # of items in the network with your routing update. Previously if I inserted mass amounts of items they would flail around inside the pipes back and forth before eventually finding a home. I imagine that when you try to drop 20K items into a small network that poor routing was probably the issue.

  2. Stack size, Stack size, Stack size. 1 per 4 ticks is slow. Your network can handle it now, I know because I popped it up to extract full stacks at a time every 2 ticks, threw an RF tools builder at full speed piping crap into a chest, and it kept up with only two extractor pipes and very very very little routing issues.

  • Done

Like I said. feel free to delete. I just loved LP so much and am so disheartened at the speed at which they are working. I get that personal time is sometimes hard but they do a commit once every two to three months. by the time they clean it up we'll be on MC 1.15 and they'll bitch about all the work they have to do to upgrade :/

You are getting there so fast!

@BrewingCoder
Copy link
Collaborator Author

you can take a peek @ my fork for some other progress besides my pull-req. just a beginning. too tired and haven't fleshed out keeping the filter up to date yet.

@Ihamfp
Copy link
Owner

Ihamfp commented Apr 26, 2019

  1. I looked at your code, if you manage to make something stable I will merge it here :p
  2. I may add this as an upgrade... adds it in the list of ideas
  3. require investigation, if you could provide the map or a screenshot it would help a lot (my test setup for this is an array of chests with providers configured as sinks, one for each type of wool except the last two, and a routing pipe as default sink to another chest)
  4. Actually the issue was complete stupidity... Some pipes would "connect" on the opposite side of where they should have, and items only continued flowing when the network gave up.
  5. I planned upgrades for this (see list of ideas) to extract 16 and 64 items per extraction, will probably be there quickly (but I may use your code in the meantime if you open a pull request...).

Also the development will be hugely slowed down in the next 2-3 weeks, sorry for this... I'll try to fix/add as much as possible before this.

EDIT: I edited your comment to help tracking what's fixed

@Ihamfp Ihamfp added the feature New feature label Apr 26, 2019
@BrewingCoder
Copy link
Collaborator Author

#3 is a UI Issue.. See here once it's done processing: https://www.youtube.com/watch?v=_-2nnpjxOC8&feature=youtu.be

@Ihamfp
Copy link
Owner

Ihamfp commented Apr 26, 2019

The issue at 2:30 is fixed as of 0.2.0, as the config GUI now uses ghost slots (and doesn't handle the shift-click, could add it later tho, it's a minor feature).

@Ihamfp
Copy link
Owner

Ihamfp commented Apr 26, 2019

Marked 3. as fixed as the UI is fixed.

@Ihamfp Ihamfp closed this as completed Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

No branches or pull requests

2 participants