Open-Systems-Pharmacology / Fluvoxamine-Model

Whole-body PBPK model of fluvoxamine as CYP1A2 and CYP3A4 DDI perpetrator drug
0 stars 3 forks source link

Cyp1a2 cyp2 c19 ddi #17

Closed PavelBal closed 2 years ago

PavelBal commented 2 years ago

Add PK-Sim project and report files. This is required as it is an alternative version of the fluvoxamine model and cannot be published as part of the OSP mode data base.

PavelBal commented 2 years ago

@Yuri05

Yuri05 commented 2 years ago

Fluvoxamine.pksim5 I thought we decided NOT to add project files to a model repo, but only the snapshots

Yuri05 commented 2 years ago

Add PK-Sim project and report files. This is required as it is an alternative version of the fluvoxamine model and cannot be published as part of the OSP mode data base.

Read it just now, lol. On the one side it explains, why you added the project and the report here. On the other side - "alternative" model means it is different from the Fluvoxamin project snapshot in this repository? Then we should either put it into a different repository (which I would prefer) or at least clearly separate it from the existing fluvoxamine snapshot - e.g. by creating a subfolder "" with its own Readme, where the project, the report etc. is placed.

PavelBal commented 2 years ago

On the other side - "alternative" model means it is different from the Fluvoxamin project snapshot in this repository?

I remember we had a long discussion on how to deal with the two fluvoxamine models. If I remember correctly, we agreed to keep the "alternative" version in a separate branch - so both PK-Sim projects and evaluation report in this repository correspond to the Fluvoxamine-snapshot in the cyp1a2_cyp2C19_DDI branch (this PR does not push into the main branch)

Yuri05 commented 2 years ago

I remember we had a long discussion on how to deal with the two fluvoxamine models. If I remember correctly, we agreed to keep the "alternative" version in a separate branch - so both PK-Sim projects and evaluation report in this repository correspond to the Fluvoxamine-snapshot in the cyp1a2_cyp2C19_DDI branch (this PR does not push into the main branch)

Ah, ok. Should be fine in this case. I did not notice that. Then the link in the report (where the latest version of the model can be found etc.) should point either to this branch or (if you create a Tag from this branch) to this tag