Skip to content
This repository has been archived by the owner on May 26, 2024. It is now read-only.

Add precious metals alloy recipe for ABS #781

Merged
merged 5 commits into from
Nov 25, 2023
Merged

Add precious metals alloy recipe for ABS #781

merged 5 commits into from
Nov 25, 2023

Conversation

lordIcocain
Copy link
Contributor

@lordIcocain lordIcocain commented Nov 13, 2023 •
edited
Loading

No description provided.

@chochem chochem added the Affects Balance Requires admin approval before merging label Nov 13, 2023
@chochem
Copy link
Member

chochem commented Nov 13, 2023

which part of the spreadsheet is used? 0.8, 1, or 1.2?

@lordIcocain
Copy link
Contributor Author

0.8

@Dream-Master Dream-Master requested a review from a team November 13, 2023 17:21
@lordIcocain
Copy link
Contributor Author

Any comments about balance? I can manually make recipes for better balance/exclude some materials

@Dream-Master
Copy link
Member

@lordIcocain add a recipe is one thing but remove temp limit from auto generation is a NO for me

@Pxx500
Copy link
Member

Pxx500 commented Nov 20, 2023

I remove temp limit from auto generation, now more mid/high tier materials can be turned in molten state without extra steps.

I made spreadsheet, so you can evaluate changes in balance. https://docs.google.com/spreadsheets/d/1UcUOfZ-YLdZry5sUNt2flKcjokm9x58p9p0uWdW9S_w/edit?usp=sharing

Also added one hand made recipe for molten precious metals alloy, same reason, get molten without extra steps

please add more information to the spreadsheet, are the numbers seconds? what is the before/after?
this excel looks like it's some kind of backend, no explanation whatsoever about what anything means

@lordIcocain lordIcocain changed the title More recipes for abs Add precious metals alloy recipe for ABS Nov 20, 2023
@lordIcocain
Copy link
Contributor Author

Bump

@Dream-Master
Copy link
Member

need to check ingame

chochem
chochem approved these changes Nov 25, 2023
View reviewed changes
Copy link
Member

@chochem chochem left a comment

Choose a reason for hiding this comment

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

that one recipe is fine.

@Dream-Master Dream-Master merged commit 23c80dd into GTNewHorizons:master Nov 25, 2023
1 check passed
@lordIcocain lordIcocain deleted the more_recipes_for_ABS branch November 25, 2023 14:40
@chochem chochem mentioned this pull request Dec 1, 2023
Closed
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@chochem chochem chochem approved these changes

@Dream-Master Dream-Master Dream-Master left review comments

Assignees
No one assigned
Labels
Affects Balance Requires admin approval before merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lordIcocain @chochem @Dream-Master @Pxx500