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

Fix GT tools with duplicates in TFC cannot set on the tool racks, and fix GT spare parts melting recipes #334

Merged
merged 7 commits into from
Apr 27, 2024

Conversation

chemlzh
Copy link
Contributor

@chemlzh chemlzh commented Apr 27, 2024

What for

Fix GT tools with duplicates in TFC cannot set on the tool racks in upcoming TFG 0.7.3. Since original program will throw ReferenceError from KubeJS.

Fix inconsistency in molten metal production from GT spared parts. Now 1 bolt equals to 18mB molten metal, 1 screw equals to 16mB molten metal, and 1 ring equals to 36mB molten metal.

Running platform

Tested on TFG 0.7.2 with TFC & GTM manually updated to newest version.

Some unfinished tasks

However, tools appear only in GTM still cannot put on the tool racks. Need further investigation.

Fix GT tools with duplicates in TFC cannot set on the tool racks. Since original program will throw ReferenceError.
Tested on TFG 0.7.2 with TFC & GTM manually updated to newest version. However, tools appear only in GTM still cannot put on the tool racks. Need further investigation.
Fix spared parts molten product inconsistency between GT and TFC.

Now 1 bolt can get 18mB molten metal, 1 screw can get 16mB molten metal, and 1 ring can get 36mB molten metal, suited with GT.

Signed-off-by: EliAyase <chemlzh@pku.edu.cn>
@chemlzh chemlzh changed the title Fix GT tools with duplicates in TFC cannot set on the tool racks Fix GT tools with duplicates in TFC cannot set on the tool racks, and fix GT spare parts melting recipes Apr 27, 2024
Change EV coil description

Signed-off-by: EliAyase <chemlzh@pku.edu.cn>
Signed-off-by: EliAyase <chemlzh@pku.edu.cn>
@Exzept1on
Copy link
Contributor

What you fixed in the tools doesn't really fix anything, because even now there is no error that you wrote about. ToolHelper should be used without $, because the binding for it is in TerraFirmaGregCore. You may be using scripts 0.7.3 on version 0.7.2 with the old version of TerraFirmaGregCore.
image

Only the correction in the file turned out to be useful constants.js , where mining_hammer was replaced with a regular hard_hammer.

@Exzept1on
Copy link
Contributor

And about second part of PR.
Ring should have 36mb.
Screw - 18mb.
Bolt - 18mb.

Fix that and then I merge this

Copy link
Contributor

@Exzept1on Exzept1on left a comment

Choose a reason for hiding this comment

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

Fix some stuff pls

chemlzh and others added 3 commits April 27, 2024 19:34
Delete "$" before ToolHelper. This rollback is paired with TFG-Core 0.7.3

Signed-off-by: EliAyase <chemlzh@pku.edu.cn>
Signed-off-by: Dmitry <52341158+Exzept1on@users.noreply.github.com>
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.

None yet

2 participants