Skip to content

Conversation

vercte
Copy link
Collaborator

@vercte vercte commented Jun 11, 2025

This PR creates the HeatCondition interface, which defines how a basin should check if a heating requirement is satisfied.

It also provides methods for displaying the recipe in JEI: A method for rendering an IDrawable in the Category, a method for providing "hint items" (blaze burner, blaze cake), and a method to get the color that the Heat Condition's name will be displayed as.

This PR also creates the HEAT_CONDITION registry, which is used to register these HeatConditions and serialize them into recipes.

Basin Recipes have changed, with the heatRequirement field being now a namespaced value (create:heated, example:cooled) with create as the default (this is to maintain backwards compatiblity; don't want to break literally every datapack 😅)

An example usage of this is linked in this gist: https://gist.github.com/vercte/fe67c1bc6649c287f84905eb9cbfb2fb

@vercte vercte added area: api Issue or PR is related to API pr type: feature PR adds a new feature or changes an existing feature labels Jun 11, 2025
@vercte vercte closed this Jul 20, 2025
@vercte vercte force-pushed the mc1.20.1/heat-type branch from 9e91172 to a575d69 Compare July 20, 2025 15:00
…1/heat-type

# Conflicts:
#	src/generated/resources/.cache/5971ca79ecd127e7d1e65be1cd906ac939e6a245
#	src/generated/resources/.cache/b256105d8411632b0d585496ea8944a751a08034
@vercte vercte reopened this Jul 20, 2025
@vercte
Copy link
Collaborator Author

vercte commented Aug 21, 2025

This apparently makes Create require JEI. I'll have to fix that

@Syndaryl
Copy link

Could you consider also support for EMI? REI will probably also be necessary once Create 6 Fabric is ready. That should cover the current Big Three recipe viewers.

@IThundxr
Copy link
Member

Could you consider also support for EMI? REI will probably also be necessary once Create 6 Fabric is ready. That should cover the current Big Three recipe viewers.

Create forge does not natively have support for EMI so only supporting a few things with EMI is not good choice

@IThundxr IThundxr added the pr status: conflicts PR has merge conflicts label Sep 16, 2025
Copy link
Contributor

@vercte, this pull request has merge conflicts with the target branch. Please merge the latest changes and leave a message here so we can continue with the process of reviewing and merging this pull request. Thanks!

…at-type

# Conflicts:
#	src/generated/resources/.cache/5971ca79ecd127e7d1e65be1cd906ac939e6a245
#	src/generated/resources/.cache/b256105d8411632b0d585496ea8944a751a08034
#	src/main/java/com/simibubi/create/compat/jei/CreateJEI.java
#	src/main/java/com/simibubi/create/content/processing/basin/BasinRecipe.java
#	src/main/java/com/simibubi/create/content/processing/recipe/ProcessingRecipeBuilder.java
#	src/main/java/com/simibubi/create/foundation/data/recipe/CreateMixingRecipeGen.java
@vercte
Copy link
Collaborator Author

vercte commented Sep 17, 2025

merged with the current state of the Create repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: api Issue or PR is related to API pr status: conflicts PR has merge conflicts pr type: feature PR adds a new feature or changes an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants