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

Implement experience and experiene/xp commands #521

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

Conversation

drakeerv
Copy link
Contributor

Description

I implemented experience and the commands to control it (currently the only way to get experience).

Testing

Tested it in game and ran cargo fmt & clippy

@drakeerv drakeerv changed the title Implement experience and experienc/xp commands Implement experience and experiene/xp commands Jan 31, 2025
Comment on lines 140 to 145
/// The player's experience level
pub experience_level: AtomicI32,
/// The player's experience progress (0.0 to 1.0)
pub experience_progress: AtomicCell<f32>,
/// The player's total experience points
pub total_experience: AtomicI32,
Copy link
Member

Choose a reason for hiding this comment

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

  1. I wonder if experience_level and total_experience can become unsigned since i'm not sure if you can get negative XP?
  2. I don't really get the difference between experience_level and total_experience, I would suggest to improve the doc on these

@drakeerv drakeerv changed the title Implement experience and experiene/xp commands (In progress) Implement experience and experiene/xp commands Jan 31, 2025
@drakeerv drakeerv marked this pull request as draft January 31, 2025 22:17
@drakeerv drakeerv changed the title (In progress) Implement experience and experiene/xp commands Implement experience and experiene/xp commands Jan 31, 2025
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