Skip to content

New procedurally generated animations for the blocks #737

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

Open
wants to merge 4 commits into
base: dev/1.21.1
Choose a base branch
from

Conversation

sjeulu
Copy link

@sjeulu sjeulu commented May 18, 2025

Makes the parts of the blocks that were previously left static move, glow, shift, whatever you want to call it. The animations are generated by a tool that can be found here. It uses the old textures as the assets that it then modifies in order to build the animations, which is why the old textures are left untouched in the sources directory. This ensures that the original textures can be easily found and accessed by anyone who wishes to further improve or modify them, because applying changes to their post-processed versions can be somewhat tedious.

@@ -1,2 +1,7 @@
<<<<<<< HEAD
Copy link
Collaborator

@zyxkad zyxkad May 18, 2025

Choose a reason for hiding this comment

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

guh conflicts :/

can you remove the .cache folder and do runData again?

Copy link
Author

Choose a reason for hiding this comment

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

done

@zyxkad
Copy link
Collaborator

zyxkad commented May 18, 2025

I noticed the highlight area changed, are they randomly generated? I personally think randomly generate stuff on this is not good, so maybe you can set a fixed seed?
Or idk what is @SirEndii 's opinion here.

Copy link
Contributor

github-actions bot commented May 18, 2025

Build Preview

badge

You can find files attached to the below linked Workflow Run URL (Logs).

Name Link
Commit 5329ebf
Logs https://github.com/IntelligenceModding/AdvancedPeripherals/actions/runs/15100306407
Jar Files AdvancedPeripherals PR 737
Expires At 2025-08-16T21:54:14Z

@sjeulu
Copy link
Author

sjeulu commented May 18, 2025

sure, i'll add an option to the script

@sjeulu
Copy link
Author

sjeulu commented May 18, 2025

done

@SirEndii
Copy link
Member

Can you document the procedure exactly step by step here, so I and other contributors know exactly how things will work, what we would need to do when we have new textures, trying to run the data gens etc

@sjeulu
Copy link
Author

sjeulu commented May 18, 2025

sure, done

@SirEndii SirEndii self-requested a review June 5, 2025 14:36
@SirEndii SirEndii added the 1.21.1 label Jun 5, 2025
@SirEndii SirEndii moved this to Needs testing/review in Advanced Peripherals Jun 5, 2025
@SirEndii SirEndii added this to the 0.7.x milestone Jun 5, 2025
# Conflicts:
#	src/generated/resources/.cache/b8526e444ae7356037f3a813274f6835d1f3dd16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Needs testing/review
Development

Successfully merging this pull request may close these issues.

3 participants