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

GitHub-AmlodipineExample - Added details for manufacturing operation and site for Pfizer #645

Merged
merged 30 commits into from
Jan 14, 2025

Conversation

ElisaKendall
Copy link
Contributor

This update is needed to answer the latest batch / manufacturing operation related use cases.

ElisaKendall and others added 3 commits December 18, 2024 11:12
PREFIX rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX owl: <http://www.w3.org/2002/07/owl#>

Copy link
Contributor

Choose a reason for hiding this comment

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

Something seems to be missing here.

Copy link
Contributor

Choose a reason for hiding this comment

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

I had not wanted to commit cmc_cq3, because I could not create the sparql query, but I had it already prepared to be filled in and overlooked that during the commit

ElisaKendall and others added 7 commits December 23, 2024 10:07
… added material name and identifier to material

Signed-off-by: Elisa Kendall <[email protected]>
…uthorizations and added restrictions accordingly

Signed-off-by: Elisa Kendall <[email protected]>
Signed-off-by: mereolog <[email protected]>
Signed-off-by: mereolog <[email protected]>
@mereolog
Copy link
Contributor

@ElisaKendall note the difference in the parameters for uc_cmc_cq1 and uc_cmc_cq3:

uc_cmc_cq1_parameter_1: "<https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascPackagedMedicinalProduct>"

uc_cmc_cq3_parameter_1: "<https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascMedicinalProduct>"

The reason for this difference is that, as of now, https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascPackagedMedicinalProduct does not seem to linked to any site.

@ElisaKendall
Copy link
Contributor Author

@ElisaKendall note the difference in the parameters for uc_cmc_cq1 and uc_cmc_cq3:

uc_cmc_cq1_parameter_1: "<https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascPackagedMedicinalProduct>"

uc_cmc_cq3_parameter_1: "<https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascMedicinalProduct>"

The reason for this difference is that, as of now, https://spec.pistoiaalliance.org/idmp/ontology/EXT/Examples/AmlodipineExample/NorvascPackagedMedicinalProduct does not seem to linked to any site

@mereolog Working on that now - needed some additional details in the ontology to include site names, identifiers, etc. which were just added

…spect to all revisions in Commons 1.2 and initial revisions to Commons 1.3, including addresses and sites and facilities ontologies (initial versions)

Signed-off-by: Elisa Kendall <[email protected]>
… with the latest version of sites and facilities submitted to Commons 1.3 (and 1.2 now published versions); eliminated unnecessary references to products and services,

Signed-off-by: Elisa Kendall <[email protected]>
…rial (not sure why it wasn't); added Amlodipine physical substance and material and substance internal identifiers for it

Signed-off-by: Elisa Kendall <[email protected]>
…jurisdiction of a country rather than the countrry itself per the model

Signed-off-by: Elisa Kendall <[email protected]>
@mereolog mereolog requested a review from tw-osthus January 10, 2025 13:50
@mereolog mereolog force-pushed the GitHub-AmlodipineExample branch from e8a28d5 to b01e1f1 Compare January 13, 2025 16:20
@mereolog mereolog merged commit ec5bca2 into master Jan 14, 2025
5 of 7 checks passed
@mereolog mereolog deleted the GitHub-AmlodipineExample branch January 14, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants