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 of Project Bench, Auto Crafting Bench and Plan Recipe #37

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

Elkatra2
Copy link

@Elkatra2 Elkatra2 commented Aug 31, 2024

Related issues :

1. Project Bench

2. Auto Crafting Bench

3. Plan Recipe

Related PR :

Changes in Project Bench :

1. Crafting logic/rework

1.1 Item with container now will return container during crafting.

  • If it's unstack-able item like GT tool or from Pam's HarvestCraft it will return to the same slot
  • If it's stack-able like graphite cell for arc furnace it will return to the storage or 2.

1.2 If the storage is full and the item is stack-able, the container from this item will be dropped to ground

1.3 If the crafting recipe is not the same as the plan recipe, required items will no more take resources from storage

1.4 No more milk duping machine, fixed OreDictionary

1.5 Empty buckets can now stacking

1.6 When shift + click items are send to Project Bench storage

1.7 When the inventory is full and you try to shift + click craft it will send the next message chat :

image
Yes it will send this message twice, due to weird logic behind me

  • TODO remove in future because I'm lazy to fix this now...

Changes in Auto Crafting Bench :

1. Crafting logic fixes

1.1 No more milk duping machine, fixed OreDictionary

Changes in Plan Recipe :

1. Animation fix/rework

Before :

Screencast.from.2024-08-29.23-47-52.webm

After :

Screencast.from.2024-08-31.15-20-25.webm
Added :
  • When you click shift at any time you can see what is the recipe output, without entering to one of the benches
Removed :
  • Glowing animation from benches GUI. Reason : when you do the same thing with AE2 pattern you can see also the output at any time, without glowing (this is what people from ru-general telled me), so I decided to do the similar way.

@Dream-Master Dream-Master requested a review from a team August 31, 2024 09:39
…hen you click shift the Icon of the Plan Recipe will be changed to the Icon of the output recipe if it has a recipe inside
@Elkatra2 Elkatra2 changed the title Fix* of ProjectBench Fix* of ProjectBench and Plan Recipe animation rework Aug 31, 2024
@Elkatra2 Elkatra2 changed the title Fix* of ProjectBench and Plan Recipe animation rework Fix* of Project Bench and Plan Recipe animation rework Aug 31, 2024
@Elkatra2 Elkatra2 marked this pull request as draft September 3, 2024 13:48
@Elkatra2
Copy link
Author

Elkatra2 commented Sep 3, 2024

I will see if I can fix several issues for Auto Crafting Bench and Project Bench and maybe others things from ProjectRed

@Elkatra2 Elkatra2 changed the title Fix* of Project Bench and Plan Recipe animation rework Fix* of Project Bench, Auto Crafting Bench and Plan Recipe Sep 3, 2024
@Elkatra2
Copy link
Author

Elkatra2 commented Sep 5, 2024

Closing due to working with prototypes of new primitives (like RailCraft) GT Coke Ovens for Stone/Steam ages

@Elkatra2 Elkatra2 closed this Sep 5, 2024
@Dream-Master
Copy link
Member

@Elkatra2 is this not fix able or you fix it iside other mods ?

@Elkatra2
Copy link
Author

Elkatra2 commented Sep 5, 2024

@Elkatra2 is this not fix able or you fix it iside other mods ?

this is about Project Bench ? It's maybe fixable, but I'm bad at Scala/Java, or I can try to do like in forestry wortable, when all the output is sended to inventory of the bench and then to player if the inv is full, also I tring to fix Overflow Valve from GT

@Dream-Master
Copy link
Member

better a bad fix that is working as no fix ? Sure if it is fixed elesewhere in another mod its good to.

@Elkatra2
Copy link
Author

Elkatra2 commented Sep 5, 2024

better a bad fix that is working as no fix ? Sure if it is fixed elesewhere in another mod its good to.

I will try to check again what's happen again

@Elkatra2 Elkatra2 reopened this Sep 10, 2024
@Elkatra2
Copy link
Author

It's time to do some works

…ity for each bench, and solving warning in `TileAutoCrafter`
@Elkatra2 Elkatra2 marked this pull request as draft September 11, 2024 00:08
@Elkatra2 Elkatra2 marked this pull request as ready for review September 14, 2024 12:52
@Elkatra2
Copy link
Author

Elkatra2 commented Sep 14, 2024

I think it's ready, but ... maybe 1.6 and 2 from Project Bench section should be done ...

@brandyyn
Copy link

GTNewHorizons/GT-New-Horizons-Modpack#16546

was apparently mentioned here but I can’t find it?

@Elkatra2
Copy link
Author

Elkatra2 commented Sep 16, 2024

GTNewHorizons/GT-New-Horizons-Modpack#16546

was apparently mentioned here but I can’t find it?

I haved plans to see, but then I made a decision: "maybe someday next time" and removed from pull.

Didn't know that it's not remove from the issue if I removed from PR.

@Elkatra2 Elkatra2 marked this pull request as draft September 16, 2024 14:10
@Elkatra2 Elkatra2 changed the title Fix* of Project Bench, Auto Crafting Bench and Plan Recipe Fix of Project Bench, Auto Crafting Bench and Plan Recipe Sep 16, 2024
@Elkatra2 Elkatra2 marked this pull request as ready for review September 16, 2024 22:02
with TSlot3 {

private val previous_indices = Array[Int](
Copy link
Author

Choose a reason for hiding this comment

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

This part can be weird, but it will be removed in future, still bad at scala syntax

@Elkatra2 Elkatra2 marked this pull request as draft September 17, 2024 20:12
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.

4 participants