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

HighPerf pusher no longer pulls stacks from the harvester #22

Closed
fluxionary opened this issue May 6, 2019 · 3 comments
Closed

HighPerf pusher no longer pulls stacks from the harvester #22

fluxionary opened this issue May 6, 2019 · 3 comments

Comments

@fluxionary
Copy link
Contributor

It used to pull full stacks, and is still documented as doing so, but it currently doesn't. Looks like this is because the harvester doesn't implement 'on_pull_stack'. I'm not sure if this is intentional or not. If it's an oversight, it's a trivial fix.

@joe7575
Copy link
Owner

joe7575 commented May 7, 2019

Right, code and documentation does not match. It could easily be added but the harvester can't pick such amount of items that the HighPerf pusher would be needed, or?

@fluxionary
Copy link
Contributor Author

fluxionary commented May 7, 2019

The harvester can pull up to 18 nodes every 4 seconds when farming tall trees, and positioned at the same level as the base of the tree. While 5 fast pushers can keep the machine from clogging in such a setup, that's much more expensive and space-intensive. 4 fash pushers have been enough in any practical tree farm I've built, which, while slightly more reasonable, isn't ideal - I usually have them all piped into a single HP chest, and an HP pusher moving things from there to an HP distributor to separate the wood, the leaves, and anything else.

I leave it to you to decide whether this is a reasonable solution, though.

@joe7575
Copy link
Owner

joe7575 commented May 9, 2019

fixed with v2.03.02

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

No branches or pull requests

2 participants