Skip to content

Conversation

@EngineersBox
Copy link

@EngineersBox EngineersBox commented Jul 7, 2021

Changes

  • Added offset to slot index when creating backpack slots

Edit: Was made aware that @Mi1ax had commented this fix on Ep8 so adding them as the original author of the fix.

@Mi1ax
Copy link

Mi1ax commented Jul 8, 2021

U just took my youtube comment and turned it into pull requests -.-

@EngineersBox
Copy link
Author

EngineersBox commented Jul 10, 2021

Didn't realise you had left a comment. It's not exactly a unique fix though. I imagine it will be quickly outdated with further abstraction to a set of base machine classes.

Happy to add you as the original author of the fix though.


On a side note, I would have included a more math oriented parallel version of inventory slot initialisation. I fear that it would be too out of context for the tutorial (people who are doing this for the very first time). It would also be pre-optimisation which can be quite dangerous.

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

Successfully merging this pull request may close these issues.

2 participants