more stuff

SapphicOverload · SapphicOverload · commit 6bb9fe75b6e2 · 2025-01-11T00:12:44.000-05:00
diff --git a/code/__DEFINES/skills.dm b/code/__DEFINES/skills.dm
@@ -25,3 +25,6 @@
 
 /// Experience required to increase your skills by one level. Increases exponentially the higher your level already is.
 #define EXPERIENCE_PER_LEVEL 500
+
+/// Calculates how much experience is required to reach a given level.
+#define EXP_REQ_CALC(level) (EXPERIENCE_PER_LEVEL * (((2**(level + 1)) / 2) - 1))
diff --git a/code/_onclick/hud/screen_objects.dm b/code/_onclick/hud/screen_objects.dm
@@ -155,7 +155,7 @@
 	data["skills"] = skill_data
 	data["skill_points"] = user.mind.skill_points
 	data["allocated_points"] = allocated_points
-	data["exceptional_skill"] = HAS_MIND_TRAIT(user, TRAIT_EXCEPTIONAL_SKILL)
+	data["skill_cap"] = EXP_MASTER + HAS_MIND_TRAIT(user, TRAIT_EXCEPTIONAL_SKILL)
 	return data
 
 /atom/movable/screen/skill_menu/ui_static_data(mob/user)
@@ -199,7 +199,7 @@
 				return TRUE
 			if(allocated_points + params["amount"] < 0)
 				return TRUE
-			if(allocated_skills[params["skill"]] + params["amount"] + user.get_skill(params["skill"]) > (4 + HAS_MIND_TRAIT(user, TRAIT_EXCEPTIONAL_SKILL)))
+			if(allocated_skills[params["skill"]] + params["amount"] + user.get_skill(params["skill"]) > (EXP_MASTER + HAS_MIND_TRAIT(user, TRAIT_EXCEPTIONAL_SKILL)))
 				return TRUE
 			if(allocated_skills[params["skill"]] + params["amount"] < 0)
 				return TRUE
diff --git a/code/datums/mind.dm b/code/datums/mind.dm
@@ -59,7 +59,7 @@
 		SKILL_FITNESS = EXP_NONE,
 	)
 
-	/// Progress towards increasing their skill level
+	/// Progress towards increasing their skill level.
 	var/list/exp_progress = list(
 		SKILL_PHYSIOLOGY = 0,
 		SKILL_MECHANICAL = 0,
@@ -68,8 +68,8 @@
 		SKILL_FITNESS = 0,
 	)
 
-	/// One-time experience gains that have already been acquired
-	var/list/unique_exp = list()
+	/// One-time experience gains that have already been acquired.
+	var/list/exp_sources = list()
 
 	/// Free skill points to allocate
 	var/skill_points = 0
@@ -121,6 +121,7 @@
 	key = _key
 	soulOwner = src
 	martial_art = default_martial_art
+	RegisterSignal(src, SIGNAL_ADDTRAIT(TRAIT_EXCEPTIONAL_SKILL), PROC_REF(update_skills))
 
 /datum/mind/Destroy()
 	SSticker.minds -= src
@@ -813,28 +814,41 @@
 		return FALSE
 	if(!amount)
 		return FALSE
-	if(source && (source in mind.unique_exp))
+	if(source && (source in mind.exp_sources))
 		return FALSE
-	mind.unique_exp.Add(source)
-	var/exp_required = EXPERIENCE_PER_LEVEL * (2**mind.skills[skill]) // exp required scales exponentially
-	if(mind.exp_progress[skill] + amount >= exp_required)
-		var/levels_gained = round(log(2, 1 + (mind.exp_progress[skill] + amount) / exp_required)) // in case you gained so much you go up more than one level
-		var/levels_allocated = hud_used?.skill_menu ? hud_used.skill_menu.allocated_skills[skill] : 0
-		if(levels_allocated > 0) // adjust any already allocated skills to prevent shenanigans (you know who you are)
-			hud_used.skill_menu.allocated_points -= min(levels_gained, levels_allocated)
-			hud_used.skill_menu.allocated_skills[skill] -= min(levels_gained, levels_allocated)
-		mind.exp_progress[skill] += amount - exp_required * (2**(levels_gained - 1))
-		mind.skill_points = max(mind.skill_points - levels_gained, 0) // remove an equal amount of unallocated skill points to prevent exploits
-		adjust_skill(skill, levels_gained)
-		to_chat(src, span_boldnotice("Your [skill] skill is now level [get_skill(skill)]!"))
+	mind.exp_sources.Add(source)
 	mind.exp_progress[skill] += amount
+	var/levels_gained = check_exp(skill)
+	if(levels_gained) // remove an equal amount of unallocated skill points to prevent exploits
+		mind.skill_points = max(mind.skill_points - levels_gained, 0)
 	return TRUE
 
-/// Returns whether experience has been gained from a given source
+/// Levels up a skill if it has enough experience to do so.
+/mob/proc/check_exp(skill)
+	if(!mind)
+		return FALSE
+	var/current_level = get_skill(skill)
+	var/exp_required = EXPERIENCE_PER_LEVEL * (2**current_level) // exp required scales exponentially
+	if(mind.exp_progress[skill] < exp_required)
+		return FALSE
+	var/skill_cap = EXP_MASTER + HAS_MIND_TRAIT(src, TRAIT_EXCEPTIONAL_SKILL)
+	var/levels_gained = min(round(log(2, 1 + (mind.exp_progress[skill] / exp_required))), max(skill_cap - current_level)) // in case you gained so much you go up more than one level
+	if(levels_gained < 1)
+		return FALSE
+	var/levels_allocated = hud_used?.skill_menu ? hud_used.skill_menu.allocated_skills[skill] : 0
+	if(levels_allocated > 0) // adjust any already allocated skills to prevent shenanigans (you know who you are)
+		hud_used.skill_menu.allocated_points -= min(levels_gained, levels_allocated)
+		hud_used.skill_menu.allocated_skills[skill] -= min(levels_gained, levels_allocated)
+	mind.exp_progress[skill] -= exp_required * (((2**round(levels_gained + 1)) / 2) - 1)
+	adjust_skill(skill, levels_gained, max_skill = skill_cap)
+	to_chat(src, span_boldnotice("Your [skill] skill is now level [get_skill(skill)]!"))
+	return levels_gained
+
+/// Returns whether experience has been gained from a given source.
 /mob/proc/has_exp(source)
 	if(!mind)
 		return FALSE
-	return (source in mind.unique_exp) ? TRUE : FALSE
+	return (source in mind.exp_sources) ? TRUE : FALSE
 
 /// Adds skill points to be allocated at will.
 /mob/proc/add_skill_points(amount)
@@ -843,6 +857,14 @@
 	mind.skill_points += amount
 	throw_alert("skill points", /atom/movable/screen/alert/skill_up)
 
+/// Called when [TRAIT_EXCEPTIONAL_SKILL] is added to the mob.
+/datum/mind/proc/update_skills(datum/source)
+	SIGNAL_HANDLER
+	if(!current)
+		return
+	for(var/skill in skills)
+		current.check_exp(skill)
+
 /datum/mind/proc/has_martialart(string)
 	if(martial_art && martial_art.id == string)
 		return martial_art
diff --git a/code/game/mecha/mecha.dm b/code/game/mecha/mecha.dm
@@ -1380,6 +1380,6 @@ GLOBAL_VAR_INIT(year_integer, text2num(year)) // = 2013???
 	if(!pilot)
 		pilot = occupant
 	var/effective_skill = pilot.get_skill(SKILL_TECHNICAL)
-	if(effective_skill < EXP_MASTER && HAS_TRAIT(pilot, TRAIT_SKILLED_PILOT))
-		effective_skill += EXP_LOW // mech pilot suit adds extra pilot skill, up to EXP_MASTER
-	return (12 - effective_skill) / 10
+	if(HAS_TRAIT(pilot, TRAIT_SKILLED_PILOT) || HAS_MIND_TRAIT(pilot, TRAIT_SKILLED_PILOT))
+		effective_skill += EXP_LOW
+	return 1 + (2 - effective_skill) * 0.075
diff --git a/code/game/objects/items/granters/_granters.dm b/code/game/objects/items/granters/_granters.dm
@@ -8,6 +8,8 @@
 	unique = 1
 	/// Flavor messages displayed to mobs reading the granter
 	var/list/remarks = list()
+	/// Whether to display the remarks in order.
+	var/ordered_remarks = FALSE
 	/// Controls how long a mob must keep the book in his hand to actually successfully learn
 	var/pages_to_mastery = 3
 	/// Sanity, whether it's currently being read
@@ -40,7 +42,7 @@
 	on_reading_start(user)
 	reading = TRUE
 	for(var/i in 1 to pages_to_mastery)
-		if(!turn_page(user))
+		if(!turn_page(user, i))
 			on_reading_stopped()
 			reading = FALSE
 			return
@@ -64,13 +66,13 @@
 	to_chat(user, span_notice("You finish reading [name]!"))
 
 /// The actual "turning over of the page" flavor bit that happens while someone is reading the granter.
-/obj/item/book/granter/proc/turn_page(mob/living/user)
+/obj/item/book/granter/proc/turn_page(mob/living/user, current_page = 1)
 	playsound(user, pick(book_sounds), 30, TRUE)
 
 	if(!do_after(user, 5 SECONDS, src))
 		return FALSE
 
-	to_chat(user, span_notice("[length(remarks) ? pick_n_take(remarks) : "You keep reading..."]"))
+	to_chat(user, span_notice("[ordered_remarks ? "[remarks[current_page]]" : (length(remarks) ? pick_n_take(remarks) : "You keep reading...")]"))
 	return TRUE
 
 /// Effects that occur whenever the book is read when it has no uses left.
diff --git a/code/game/objects/items/granters/martial_arts/racial.dm b/code/game/objects/items/granters/martial_arts/racial.dm
@@ -101,6 +101,7 @@
 	icon = 'icons/obj/module.dmi'
 	icon_state = "cyborg_upgrade"
 	remarks = list("MANKIND IS DEAD.", "BLOOD IS FUEL.", "HELL IS FULL.")
+	ordered_remarks = TRUE
 
 /obj/item/book/granter/martial/ultra_violence/on_reading_start(mob/user)
 	to_chat(user, span_notice("You plug \the [src] in and begin loading PRG$[martial_name]."))
diff --git a/code/game/objects/items/granters/mech_piloting.dm b/code/game/objects/items/granters/mech_piloting.dm
@@ -14,4 +14,4 @@
 
 /obj/item/book/granter/mechpiloting/on_reading_finished(mob/user)
 	. = ..()
-	user.adjust_skill(SKILL_TECHNICAL, EXP_MID, max_skill = EXP_GENIUS)
+	ADD_TRAIT(user.mind, TRAIT_SKILLED_PILOT, type)
diff --git a/code/game/objects/items/granters/skills.dm b/code/game/objects/items/granters/skills.dm
@@ -0,0 +1,77 @@
+/// Grants experience to the reader.
+/obj/item/book/granter/skill
+	name = "skill guide"
+	desc = "A guide to getting good, whatever that means."
+	remarks = list(
+		"Skill issue...?",
+	)
+	/// Experience gains from reading this book.
+	var/list/exp_gain = list(
+		SKILL_PHYSIOLOGY = EXPERIENCE_PER_LEVEL,
+		SKILL_MECHANICAL = EXPERIENCE_PER_LEVEL,
+		SKILL_TECHNICAL = EXPERIENCE_PER_LEVEL,
+		SKILL_SCIENCE = EXPERIENCE_PER_LEVEL,
+		SKILL_PHYSIOLOGY = EXPERIENCE_PER_LEVEL,
+	)
+
+/obj/item/book/granter/skill/can_learn(mob/living/user)
+	return !user.has_exp("[type]_[exp_gain[1]]")
+
+/obj/item/book/granter/skill/on_reading_finished(mob/living/user)
+	. = ..()
+	if(!user.mind)
+		CRASH("[user.type] somehow read [type] without a mind!")
+	for(var/skill in exp_gain)
+		user.add_exp(skill, exp_gain[skill], "[type]_[skill]")
+
+/obj/item/book/granter/skill/physiology
+	name = "\improper Guide to First Aid"
+	desc = "This book teaches basic first aid information."
+	remarks = list(
+		"Dying is bad..?",
+		"Suture or cauterize open wounds to prevent bleeding out...",
+		"Apply ointment or regenerative mesh to sterilize infected burns...",
+		"Move critical patients on rolling beds or over your shoulder..."
+	)
+	exp_gain = list(
+		SKILL_PHYSIOLOGY = EXP_REQ_CALC(EXP_MASTER),
+	)
+
+/obj/item/book/granter/skill/mechanics
+	name = "Nuclear Engineering for Dummies"
+	desc = "A step-by-step guide to operating a nuclear reactor."
+	remarks = list(
+		"Wear radiation protection during maintenance...",
+		"Adjust control rods to moderate the temperature...",
+		"High temperatures generate more power...",
+		"Don't press AZ-5..?",
+	)
+	exp_gain = list(
+		SKILL_MECHANICAL = EXP_REQ_CALC(EXP_MASTER),
+	)
+
+/obj/item/book/granter/skill/technical
+	name = "Hacking 101"
+	desc = "Contains detailed information on airlock maintenance."
+	remarks = list(
+		"Wear insulated gloves for protection...",
+		"Pulse wires twice to avoid changing settings...",
+		"Pry open unpowered doors with a crowbar...",
+		"Bolt an open door to prevent it closing behind you...",
+	)
+	exp_gain = list(
+		SKILL_TECHNICAL = EXP_REQ_CALC(EXP_MASTER),
+	)
+
+/obj/item/book/granter/skill/science
+	name = "Statistical Mechanics and Thermodynamics"
+	desc = "Perhaps it will be wise to approach this subject cautiously."
+	remarks = list(
+		"Ludwig Boltzmann, who spent much of his life studying statistical mechanics, died in 1906, by his own hand...",
+		"Paul Ehrenfest, carrying on the work, died similarly in 1933...",
+		"Now it is our turn to study statistical mechanics...",
+	)
+	ordered_remarks = TRUE
+	exp_gain = list(
+		SKILL_SCIENCE = EXP_REQ_CALC(EXP_MASTER),
+	)
diff --git a/code/modules/library/lib_items.dm b/code/modules/library/lib_items.dm
@@ -199,7 +199,6 @@
 	var/unique = 0		//0 - Normal book, 1 - Should not be treated as normal book, unable to be copied, unable to be modified
 	var/title			//The real name of the book.
 	var/window_size = null // Specific window size for the book, i.e: "1920x1080", Size x Width
-	var/list/skill_gain
 
 
 /obj/item/book/attack_self(mob/user)
diff --git a/code/modules/uplink/uplink_items.dm b/code/modules/uplink/uplink_items.dm
@@ -2103,6 +2103,29 @@ GLOBAL_LIST_INIT(uplink_items, subtypesof(/datum/uplink_item))
 	item = /obj/item/book/granter/mechpiloting
 	cost = 5	//this is genuinely a REALLY strong effect, don't sleep on it
 
+/datum/uplink_item/device_tools/physiology_guide
+	name = "Guide to First Aid"
+	desc = "A book that improves the reader's physiological knowledge."
+	item = /obj/item/book/granter/skill/physiology
+	cost = 3
+
+/datum/uplink_item/device_tools/mechanics_guide
+	name = "Nuclear Engineering for Dummies"
+	desc = "A book that improves the reader's mechanical skills."
+	item = /obj/item/book/granter/skill/mechanics
+	cost = 3
+
+/datum/uplink_item/device_tools/technical_guide
+	name = "Hacking 101"
+	desc = "A book that improves the reader's technical abilities."
+	item = /obj/item/book/granter/skill/technical
+	cost = 3
+
+/datum/uplink_item/device_tools/science_guide
+	name = "Statistical Mechanics and Thermodynamics"
+	desc = "A book that improves the reader's scientific proficiency."
+	item = /obj/item/book/granter/skill/science
+	cost = 3
 
 /datum/uplink_item/device_tools/mech_drop
 	name = "Orbital Mech Drop Fulton"
diff --git a/tgui/packages/tgui/interfaces/SkillMenu.js b/tgui/packages/tgui/interfaces/SkillMenu.js
@@ -73,7 +73,7 @@ export const SkillMenu = (props, context) => {
 const AdjustSkill = (props, context) => {
   const { act, data } = useBackend(context);
   const { skill, name, index, tooltip, color } = props;
-  const { skills, skill_points, allocated_points, exceptional_skill, exp_per_level } = data;
+  const { skills, skill_points, allocated_points, skill_cap, exp_per_level } = data;
   const { base, allocated, exp_progress } = skills[index];
 
   const exp_required = exp_per_level * Math.pow(2, base);
@@ -100,25 +100,25 @@ const AdjustSkill = (props, context) => {
               amount: -1,
             })}
           />
-          {base + allocated} / {exceptional_skill ? 5 : 4}
+          {base + allocated} / {skill_cap}
           <Button
             ml="2px"
             icon="plus"
-            disabled={(base + allocated) >= (exceptional_skill ? 5 : 4) || allocated_points >= skill_points}
+            disabled={(base + allocated) >= skill_cap || allocated_points >= skill_points}
             onClick={() => act('allocate', {
               skill: skill,
               amount: 1,
             })}
           />
           <ProgressBar
-            value={exp_progress}
+            value={(base >= skill_cap) ? exp_required : exp_progress}
             maxValue={exp_required}
             color={color}
             width={7.5}
             ml={1}
             textAlign="left"
           >
-            {Math.floor(exp_progress)} / {Math.floor(exp_required)}
+            {(base >= skill_cap) ? "Max" : (Math.floor(exp_progress) + " / " + Math.floor(exp_required))}
           </ProgressBar>
         </Box>
       </LabeledList.Item>
diff --git a/yogstation.dme b/yogstation.dme
@@ -1387,6 +1387,7 @@
 #include "code\game\objects\items\granters\_granters.dm"
 #include "code\game\objects\items\granters\mech_piloting.dm"
 #include "code\game\objects\items\granters\origami.dm"
+#include "code\game\objects\items\granters\skills.dm"
 #include "code\game\objects\items\granters\crafting\_crafting_granter.dm"
 #include "code\game\objects\items\granters\crafting\ashwalkers.dm"
 #include "code\game\objects\items\granters\crafting\bone_notes.dm"
