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

5.24 support #574

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

teunvdingenen
Copy link

I got most the changes down to generate 5.24 content. If possible it would be nice to get some people to use this code so I can get more feedback on things that are working and are not. There isn't much more I can do at the moment.

There is one remaining problem right now and that is that the cli is generating identical pages for summoned creatures. For example "undead spirit" is getting 7 entires that are the same. I'm not sure if this is a bug or not but either way I can't figure out how to resolve.

Also there seems to have been a change in how features or monsters are being categorized. The effect is that variants aren't correctly being found. I think it's best to wait for future releases of dmg and mm before making big changes. For now it does seem to generate all content you would want.

@ebullient
Copy link
Owner

Those 7 entries should not be identical. I'll double check. There should be one per spell level with most of the math already done. (I prefer this when adding creatures to encounters, for example).

I have been pulling in some of your changes, but I have more coming. So .. this will be taken as (very welcome!) advice. ;)

@ebullient
Copy link
Owner

ebullient commented Nov 14, 2024

As for the bit about PHB being the default: PHB is still the default for link/reference resolution. XPHB is present and used explicitly in cross-references. I have other changes pending that remove all empty segments to make this explicit and avoid conflicts.

You end up relying on aliasing as a result of reprints to get the right one, which does work, now that I've fixed aliasing. I'll add some tests for XPHB/XDMG only, of course..

@teunvdingenen
Copy link
Author

Let me know if there is anything else I can do, I hope these commits are useful.

@ebullient
Copy link
Owner

They are!!

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