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

Fixes #2281 PK-Sim DB: unused tables #2288

Merged

Conversation

Yuri05
Copy link
Member

@Yuri05 Yuri05 commented Aug 3, 2022

Fixes #2282 PK-Sim DB: unused table in the query
Fixes #2283 PK-Sim DB: missing primary key

Fixes Open-Systems-Pharmacology#2282 PK-Sim DB: unused table in the query
Fixes Open-Systems-Pharmacology#2283 PK-Sim DB: missing primary key
@Yuri05 Yuri05 requested a review from msevestre August 3, 2022 14:00
hidden bigint NOT NULL DEFAULT 0,
editable bigint NOT NULL DEFAULT 1,
PRIMARY KEY(table_name, column_name)
);
Copy link
Member Author

Choose a reason for hiding this comment

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

removed unused table

text_value text NOT NULL,
description text NOT NULL,
PRIMARY KEY(property)
);
Copy link
Member Author

Choose a reason for hiding this comment

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

remove unused table

[sequence] bigint NOT NULL,
[transport_type] text,
PRIMARY KEY([gene], [species], [container_type], [container_name]),
FOREIGN KEY([container_type], [container_name]) REFERENCES [tab_container_names]([container_type], [container_name]) ON UPDATE CASCADE,
Copy link
Member Author

@Yuri05 Yuri05 Aug 3, 2022

Choose a reason for hiding this comment

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

Removed duplicated entries and added the primary key

[x_min] double NOT NULL,
[x_max] double NOT NULL,
[frequency] double NOT NULL,
PRIMARY KEY([distribution_type], [x_min], [x_max]));
Copy link
Member Author

Choose a reason for hiding this comment

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

removed unused table

[TAB_REPRESENTATION_INFO].[ICON_NAME] AS [IconName],
'' AS [SubType]
FROM [TAB_REPRESENTATION_INFO]
ORDER BY [TAB_REPRESENTATION_INFO].[DISPLAY_NAME];
Copy link
Member Author

Choose a reason for hiding this comment

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

removed unused table in the query

@msevestre msevestre merged commit 3e31130 into Open-Systems-Pharmacology:develop Aug 3, 2022
@Yuri05 Yuri05 deleted the 2281_unused_tables branch April 5, 2023 08:30
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.

PK-Sim DB: missing primary key PK-Sim DB: unused table in the query
2 participants