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

Small fixes #1578

Open
wants to merge 12 commits into
base: unstable
Choose a base branch
from
Open

Small fixes #1578

wants to merge 12 commits into from

Conversation

MauveCloud
Copy link
Collaborator

@MauveCloud MauveCloud commented Oct 28, 2020

This is a pull request for fixes that seemed too small for separate pull requests. These can be evaluated and approved separately and manually added to the base if desired.

Details will mostly be in the commit messages, though for one I forgot to mention issue #1324

See Blood-Asp#1555
These gems could be from small ores.
See Blood-Asp#1556 - most (or maybe all) GT-specific circuits are already blacklisted, and I don't know of any reason to insist on unifying circuits, but this had a weird side effect described in that bug related to looking up recipes for HV-tier circuits.
@MauveCloud
Copy link
Collaborator Author

Apparently mentioning issues in the commit messages will only link them back to those commits, not to this PR, so here are the other affected issues:
#1555
#1556
#1573

@MauveCloud
Copy link
Collaborator Author

Implemented #1553

Possible by using a GregTech sensor kit on a muffler hatch
@MauveCloud
Copy link
Collaborator Author

Implemented #1554

@draknyte1
Copy link
Collaborator

For 1553, just take the existing code from the GTNH repo. It’s much cleaner.

at draknyte1's suggestion - apparently already had a cleaner version of the auto-output functionality.
@draknyte1
Copy link
Collaborator

Let me know when this is done and I’ll approve.

@MauveCloud
Copy link
Collaborator Author

MauveCloud commented Oct 31, 2020

Chemical formula subscripts inspired by GTNewHorizons/GT5-Unofficial#38 , but hopefully my version is cleaner than that attempt. There might already be something similar in GT++ (in code from before it went private), but I didn't know where to look. Edit: actually, I found the similar function in GT++ before the commit below, it helped inspire me.

@MauveCloud
Copy link
Collaborator Author

Added a copy of GTNewHorizons/GT5-Unofficial#331 for inclusion in main fork.

@MauveCloud
Copy link
Collaborator Author

Should help with #1568 which as far as I can tell was caused by f207774

@MauveCloud MauveCloud changed the title [WIP] Small fixes Small fixes Dec 4, 2020
@MauveCloud
Copy link
Collaborator Author

@draknyte1 I think this is basically done. If I think of more "small fixes" or you want to list some changes from GTNH to carry over, I can make a new PR for those.

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