Skip to content

Conversation

@plakrisenko
Copy link
Collaborator

add datasetId column and visualization specification

@plakrisenko
Copy link
Collaborator Author

I think there was a mix up in the simulations file.

Before:
plot2

After:
plot7_backup

@plakrisenko plakrisenko requested a review from stephanmg October 30, 2025 10:24
@plakrisenko plakrisenko marked this pull request as ready for review October 30, 2025 10:25
@stephanmg
Copy link
Collaborator

Drop markers for simulation

@plakrisenko
Copy link
Collaborator Author

Drop markers for simulation

At the moment, this is not part of visualization specification, but can be set in PEtab:
https://github.com/PEtab-dev/libpetab-python/blob/main/petab/v1/visualize/plotter.py#L19-L32

@stephanmg
Copy link
Collaborator

Okay, perfect then.

Copy link
Collaborator

@m-philipps m-philipps left a comment

Choose a reason for hiding this comment

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

Thanks!

@m-philipps m-philipps merged commit c811f6d into master Nov 5, 2025
6 checks passed
@plakrisenko plakrisenko linked an issue Nov 12, 2025 that may be closed by this pull request
14 tasks
@plakrisenko plakrisenko deleted the sneyd_model_updates branch November 21, 2025 15:17
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.

Add missing visualization files

4 participants