Skip to content

Initial mood roll-out - #1921

Open
drtheodor wants to merge 8 commits into
mainfrom
feat/mood
Open

Initial mood roll-out#1921
drtheodor wants to merge 8 commits into
mainfrom
feat/mood

Conversation

@drtheodor

Copy link
Copy Markdown
Member

About the PR

Technical details

All emotions have a value from 0 to 1. Emotions are grouped together either in "groups" or "ranges". Emotions in "groups" increase when one of them increases. Emotions in "ranges" decrease.

Requirements

Changelog

🆑

  • add: initial TARDIS mood roll-out! Suggestions, input, bug reports and mood event ideas are appreciated!

drtheodor and others added 6 commits October 1, 2025 22:30
Signed-off-by: Theo <git@theo.is-a.dev>
Signed-off-by: Theo <git@theo.is-a.dev>
Signed-off-by: Theo <git@theo.is-a.dev>
Signed-off-by: Theo <git@theo.is-a.dev>
…al roll-out

Signed-off-by: DrTheodor <DrTheodor@users.noreply.github.com>
Signed-off-by: DrTheodor <DrTheodor@users.noreply.github.com>
@drtheodor
drtheodor requested review from a team as code owners October 6, 2025 17:45
@drtheodor drtheodor added the T: New Feature Type: New feature or content, or extending existing content. label Oct 6, 2025
@github-actions github-actions Bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. S: Needs Review Status: Requires additional reviews before being fully accepted. size/M Denotes a PR that changes 100-999 lines. A: Tardis Components Area: Tardis components & manager. A: Admin Tooling Area: Admin tooling and moderation. A: Console Area: Tardis console. and removed size/M Denotes a PR that changes 100-999 lines. S: Needs Review Status: Requires additional reviews before being fully accepted. labels Oct 6, 2025
@GSMPBot
GSMPBot requested a review from duzos October 6, 2025 17:45
@github-actions github-actions Bot added the S: Needs Review Status: Requires additional reviews before being fully accepted. label Oct 6, 2025
Signed-off-by: DrTheodor <DrTheodor@users.noreply.github.com>
@github-actions github-actions Bot added the size/M Denotes a PR that changes 100-999 lines. label Oct 6, 2025
@drtheodor drtheodor removed the S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. label Oct 6, 2025
serverWorld.getPlayers().forEach(player -> tardis.loyalty().subLevel(player, 2));
serverWorld.getPlayers().forEach(player -> {
tardis.loyalty().subLevel(player, 2);
tardis.<MoodHandler2>handler(TardisComponent.Id.MOOD).add(Emotion.Type.UPSET, 0.001f);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

wont this cause it to lose 0.001 based on the amount of players in the interior

Comment thread src/main/java/dev/amble/ait/core/likes/Opinion.java

static {
if (FabricLoader.getInstance().isDevelopmentEnvironment()) {
HudRenderCallback.EVENT.register((drawContext, v) -> {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this'll cause a serverside crash if we run it in dev env

@github-actions github-actions Bot added S: Awaiting Changes Status: Changes are required before another review can happen. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. labels Oct 6, 2025
@github-actions

Copy link
Copy Markdown

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions Bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted. label Oct 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: Admin Tooling Area: Admin tooling and moderation. A: Console Area: Tardis console. A: Tardis Components Area: Tardis components & manager. S: Awaiting Changes Status: Changes are required before another review can happen. S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted. size/M Denotes a PR that changes 100-999 lines. T: New Feature Type: New feature or content, or extending existing content.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants