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

new laser engraver multi accept non-borosilicate glass #17959

Closed
3 tasks done
brnaaaa opened this issue Nov 13, 2024 · 2 comments · Fixed by GTNewHorizons/GT5-Unofficial#3584
Closed
3 tasks done

new laser engraver multi accept non-borosilicate glass #17959

brnaaaa opened this issue Nov 13, 2024 · 2 comments · Fixed by GTNewHorizons/GT5-Unofficial#3584
Labels
Bug: Minor Mod: GT GregTech and its (former) addons Status: Ready for Developer Issue ready for a developer to pick up and implement Suggestion Type: Tooltip

Comments

@brnaaaa
Copy link

brnaaaa commented Nov 13, 2024

Your GTNH Discord Username

.barnac

Your Pack Version

nightly 733

Your Server

SP

Java Version

Java 21

Type of Server

Single Player

Your Expectation

изображение
hyper-intensity laser engraver should accept only borosilicate glass

The Reality

изображение
изображение
with managlass and thorium yttrium glass block laser engraver running fine

Your Proposal

fix this

Final Checklist

  • I have searched this issue tracker and there is nothing similar already. Posting on a closed issue saying the bug still exists will prompt us to investigate and reopen it once we confirm your report.
  • I can reproduce this problem consistently by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
  • I have asked other people and they confirm they also have this problem by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
@brnaaaa brnaaaa added Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed labels Nov 13, 2024
@RAFAEL-SOSA-UH RAFAEL-SOSA-UH added Suggestion Status: Ready for Developer Issue ready for a developer to pick up and implement Type: Tooltip Type: Not a Bug Is not actually a bug. Mod: GT GregTech and its (former) addons Status: Needs Discussion Issue requires developer/staff/admin discussion and removed Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed Status: Ready for Developer Issue ready for a developer to pick up and implement labels Nov 14, 2024
@AbdielKavash
Copy link
Member

AbdielKavash commented Nov 17, 2024

There is as of right now no method to accept some types of glass and reject others of the same tier (despite what some comments in Bartworks' code suggest). We have had a discussion about whether this is something we want to support in the future, but the results were inconclusive.

Tooltip could be fixed easily enough to describe actual current behavior.

@FourIsTheNumber
Copy link
Contributor

I approve removing the tooltip and probably just switching to the standard glass method for this. If someone wants to clear it up before I get around to doing it myself, feel free.

@FourIsTheNumber FourIsTheNumber added Bug: Minor Status: Ready for Developer Issue ready for a developer to pick up and implement and removed Status: Needs Discussion Issue requires developer/staff/admin discussion Type: Not a Bug Is not actually a bug. labels Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Minor Mod: GT GregTech and its (former) addons Status: Ready for Developer Issue ready for a developer to pick up and implement Suggestion Type: Tooltip
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants