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

Multiblocked compatibility (incomplete) #287

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Wizzerinus
Copy link
Contributor

Here we go.

Things that I have implemented already:

  • Loading recipe maps
  • Recipe conditions and ingredient mappers for every capability in the base mod except for ParticleStack (because QMD depends on NCO which crashes with GrS)
  • API for other MBD addons to depend on (i.e. I plan to do it for MBDNBT)

Things that I know how to implement and will do so, but did not get there yet:

  • Creating multiblocks (preinit) - probably will have a builder that lets you define structure, add traits, link the recipe map, etc.

Things that I do not know how to do at all/to some extent and need help with:

  • Documentation, wiki generation, example generation
  • Potentially better syntax than doing .input(mbd.item(item( and such for cases where per-tick/chanced inputs/outputs are desired

Opening this PR to track my progress.
(I am aware debug_multiblocked = true rn)

}

// Particle Stack - TODO
// Can't be done right now because QMD depends on NCO, which crashes with Groovyscript
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just load an older version of nuclearcraft from before groovyscript compat was added

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we wont have a good way to create particle stacks anyway, because QMD does not have a compat. I think this could be looked at when we do decide to add QMD compat.

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