Acc. card and export bus. #348

Closed
hisorange opened this Issue Oct 25, 2014 · 13 comments

Comments

Projects
None yet
4 participants
@hisorange

Aye the acceleration card's tooltip says 4 to import / export bus but the export only takes 3, with 4 it does nothing. ////// just a minor detail ///////

@Cisien

This comment has been minimized.

Show comment
Hide comment
@Cisien

Cisien Oct 26, 2014

Contributor

The tool tip displays the amount that is allowed in the upgrade slots, not necessarily what is functional

Contributor

Cisien commented Oct 26, 2014

The tool tip displays the amount that is allowed in the upgrade slots, not necessarily what is functional

@Cisien Cisien closed this Oct 26, 2014

@hisorange

This comment has been minimized.

Show comment
Hide comment
@hisorange

hisorange Oct 26, 2014

I see, but it can be easly missleading since the site does not document how many you can insert, and what I meant under 'it does nothing' was that it's stopping the export, so when one insert 4 card the export bus will not work anymore and since it's not mentioned anywhere just on the tooltip, either a line of explanation on the site or the tooltip could fix it.

I see, but it can be easly missleading since the site does not document how many you can insert, and what I meant under 'it does nothing' was that it's stopping the export, so when one insert 4 card the export bus will not work anymore and since it's not mentioned anywhere just on the tooltip, either a line of explanation on the site or the tooltip could fix it.

@thatsIch thatsIch reopened this Oct 31, 2014

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Oct 31, 2014

Member

That it stops working is definitly a bug, cause it CAN use all 4 upgrade speeds (

    case 0:
        itemToSend = 1;
        break;
    case 1:
        itemToSend = 8;
        break;
    case 2:
        itemToSend = 32;
        break;
    case 3:
        itemToSend = 64;
        break;
    case 4:
        itemToSend = 96;
        break;
    }
Member

thatsIch commented Oct 31, 2014

That it stops working is definitly a bug, cause it CAN use all 4 upgrade speeds (

    case 0:
        itemToSend = 1;
        break;
    case 1:
        itemToSend = 8;
        break;
    case 2:
        itemToSend = 32;
        break;
    case 3:
        itemToSend = 64;
        break;
    case 4:
        itemToSend = 96;
        break;
    }
@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Oct 31, 2014

Member

There is a piece which sets it back to 64 if > 64. I have no idea why. Maybe some inventories have issues if extracting more than a single stack?

Member

yueh commented Oct 31, 2014

There is a piece which sets it back to 64 if > 64. I have no idea why. Maybe some inventories have issues if extracting more than a single stack?

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Oct 31, 2014

Member

@yueh which part are you referring to?

Member

thatsIch commented Oct 31, 2014

@yueh which part are you referring to?

@Cisien

This comment has been minimized.

Show comment
Hide comment
@Cisien

Cisien Oct 31, 2014

Contributor

Is it failing because of ae or because of the target inventory?

Say we set it to max stack size for the item? The issue is still there, except there is no difference between 2 and 4 when moving say ender pearls into a dsu

Contributor

Cisien commented Oct 31, 2014

Is it failing because of ae or because of the target inventory?

Say we set it to max stack size for the item? The issue is still there, except there is no difference between 2 and 4 when moving say ender pearls into a dsu

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Nov 1, 2014

Member

Maybe it was another bus, but I am fairly sure that I remember seeing some code which sets the itemsToSend to min(64, itemToSend). It was also not directly adjacent to the switch itself.

Member

yueh commented Nov 1, 2014

Maybe it was another bus, but I am fairly sure that I remember seeing some code which sets the itemsToSend to min(64, itemToSend). It was also not directly adjacent to the switch itself.

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Nov 12, 2014

Member

Tested it in a creative world and an export bus with 3 upgrades will export 64 items each tick and 96 with 4.

The issue will be closed, until more information is provided on how to reproduce it.

Member

yueh commented Nov 12, 2014

Tested it in a creative world and an export bus with 3 upgrades will export 64 items each tick and 96 with 4.

The issue will be closed, until more information is provided on how to reproduce it.

@yueh yueh closed this Nov 12, 2014

@hisorange

This comment has been minimized.

Show comment
Hide comment
@hisorange

hisorange Nov 12, 2014

If you need a featured presentation how to reproduce the issue, just ask for it. Since I opened the issue there was not even a single comment like 'I can't reproduce it' so here is the setup.

http://i.imgur.com/OApDDeZ.png

Since I got a bit free time here is a deeper view, any other inventory can accept 96 pcs / export but DSU, maybe this is in fault, tested with barrels / chests / other intentories, seems to working.

If you need a featured presentation how to reproduce the issue, just ask for it. Since I opened the issue there was not even a single comment like 'I can't reproduce it' so here is the setup.

http://i.imgur.com/OApDDeZ.png

Since I got a bit free time here is a deeper view, any other inventory can accept 96 pcs / export but DSU, maybe this is in fault, tested with barrels / chests / other intentories, seems to working.

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Nov 12, 2014

Member

Maybe but I am faintly remembering it is split up into 64 items per operation or something like that. So 96 would be a stack of 64 and a stack of 32.

@hisorange I do not really bother to ask for it explicitly, because we already do it when someone is opening an issue. It is mostly too tedious writing a first comment to the majority of issues about "read the guidelines".

Member

yueh commented Nov 12, 2014

Maybe but I am faintly remembering it is split up into 64 items per operation or something like that. So 96 would be a stack of 64 and a stack of 32.

@hisorange I do not really bother to ask for it explicitly, because we already do it when someone is opening an issue. It is mostly too tedious writing a first comment to the majority of issues about "read the guidelines".

@yueh yueh reopened this Nov 12, 2014

@yueh yueh added the type-bug label Nov 12, 2014

@Cisien

This comment has been minimized.

Show comment
Hide comment
@Cisien

Cisien Nov 12, 2014

Contributor

I remember reproducing this with ender pearls into a dsu

Contributor

Cisien commented Nov 12, 2014

I remember reproducing this with ender pearls into a dsu

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Nov 14, 2014

Member

mh with the DSU having 2 input slots, there should be no issue regarding that, probably needs further investigation

Member

thatsIch commented Nov 14, 2014

mh with the DSU having 2 input slots, there should be no issue regarding that, probably needs further investigation

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Nov 14, 2014

Member

I have already investigated it.
The problem is that we want to push 96 items into the DSU, which it just does not accept and the export completely fails.

There is already some code in place which tries to split the export into smaller parts, but this is not really working.
I do not want to add more complexity to it, because this is already handled. We bascially just have to correctly use the max stack size supported by any inventory and return the leftover amount back for a next try. This should automatically split every push if 96 items into one with 64 and one with 32, of course in the same tick.

Member

yueh commented Nov 14, 2014

I have already investigated it.
The problem is that we want to push 96 items into the DSU, which it just does not accept and the export completely fails.

There is already some code in place which tries to split the export into smaller parts, but this is not really working.
I do not want to add more complexity to it, because this is already handled. We bascially just have to correctly use the max stack size supported by any inventory and return the leftover amount back for a next try. This should automatically split every push if 96 items into one with 64 and one with 32, of course in the same tick.

@yueh yueh closed this in cf89873 Dec 3, 2014

yueh added a commit that referenced this issue Dec 3, 2014

Merge pull request #519 from yueh/fix-348
Fixes #348 by respecting the inventory stack limit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment