Skip to content

Fix table round-tripping. - #486

Merged
luciansmith merged 4 commits into
developmentfrom
fix-table-html-md
Aug 29, 2026
Merged

Fix table round-tripping.#486
luciansmith merged 4 commits into
developmentfrom
fix-table-html-md

Conversation

@luciansmith

Copy link
Copy Markdown
Member

The table from Biomodel 197 was being mangled both going and coming; this fixes both. We should also push them upstream to the source libraries, too.

The table from Biomodel 197 was being mangled both going and coming; this fixes both.  We should also push them upstream to the source libraries, too.
Pulled in the latest changes from maddy, and added two of my PRs that have not yet been accepted, but are needed for libsbml.
@fbergmann

Copy link
Copy Markdown
Member

@luciansmith that looks great, if anything it is unfortunate, that the changes from: 92a3f5c are undone with the commit. Those were needed last nov to compile.

@luciansmith

Copy link
Copy Markdown
Member Author

I can certainly add them back in, but it's weird that everything seems to be passing/compiling without the changes? Did the failing compiler change, or something?

@fbergmann

Copy link
Copy Markdown
Member

I can certainly add them back in, but it's weird that everything seems to be passing/compiling without the changes? Did the failing compiler change, or something?

it failed on another system, not the one we were testing under. Anyways it would be great to have it in, thanks

@luciansmith

Copy link
Copy Markdown
Member Author

Got it. Do you remember which one? I ask just because I'd like to include that information when I post a PR to the original library.

@luciansmith

Copy link
Copy Markdown
Member Author

Changes added!

@fbergmann fbergmann left a comment

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.

@luciansmith thank you so much

@luciansmith
luciansmith merged commit 6d26cef into development Aug 29, 2026
35 checks passed
@luciansmith
luciansmith deleted the fix-table-html-md branch August 29, 2026 15:50
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