Skip to content

WIP DLC skills - #2

Merged
benny-dreamly merged 12 commits into
Simsipelago:sims4-devfrom
silasary:sims4-dev
Aug 11, 2025
Merged

WIP DLC skills#2
benny-dreamly merged 12 commits into
Simsipelago:sims4-devfrom
silasary:sims4-dev

Conversation

@silasary

Copy link
Copy Markdown

This should be everything needed on the apworld side to support DLC skills.

I have not touched the visibility yet, as it definitely shouldn't be turned on without corresponding mod changes.

hsy_entrepreneur_skill = "Entrepreneur Skill"
hr_horse_riding_skill = "Horse Riding Skill"
hr_nectar_making_skill = "Nectar Making Skill"
cc_gemology_skill = "Gemology Skill"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can't remember for the life of me what cc means lol (the rest of them I think are fairly easy to understand, nk is nifty knitting, sy is snowy escape (maybe sy could be sc instead? not sure) pa is... oh no I don't remember either... cgl is cottage living, hsy is high school years, hr is horse ranch, lv is love struck, and is life and death, bnh is businesses and hobbies, ebn is enchanted by nature)

@silasary silasary Jul 30, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Cc is Crystal Creations Stuff. It's an entirely forgettable pack that for some reason added a skill.

Pa is Paranormal.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ahhhh right yeah that makes a lot of sense LOL

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ahhhh paranormal stuff right... yep very forgettable

Comment thread worlds/sims4/__init__.py
for i in range(item["count"]):
sims4_item = self.create_item(item["name"])
pool.append(sims4_item)
for item_data in item_table.values():

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

the refactor here looks great, and I'm hoping it just works fine with my multi-career PR branch I had been working on

Comment thread worlds/sims4/Locations.py
'expansion': "base"},
skill_locations_table: dict[int, Sims4LocationDict] = {}

def add_skill_location(skill_name: str, expansion: str, max_level: int) -> int:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this, and the items are MUCH better methods of generating the location/item table

@benny-dreamly
benny-dreamly merged commit 173edc4 into Simsipelago:sims4-dev Aug 11, 2025
4 of 10 checks passed
@benny-dreamly

Copy link
Copy Markdown
Member

Merged. Thank you :D

benny-dreamly pushed a commit that referenced this pull request Aug 16, 2025
Resync the repository with Main AP
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.

3 participants