CDR Tickets

Issue Number 4366
Summary [Glossary] Display VideoLink elements on GTC Full QC Report
Created 2018-01-05 13:55:06
Issue Type Improvement
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2018-04-10 13:26:01
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.219388
Description

Please display VideoLink elements on the Glossary Term Concept Full QC Report.

Example of a concept with a VideoLink: Metastasis GTC (CDR622340)

Comment entered 2018-01-05 13:55:57 by Juthe, Robin (NIH/NCI) [E]

Adding Margaret and William as watchers.

Comment entered 2018-03-28 14:39:04 by Englisch, Volker (NIH/NCI) [C]

We don't have EmbeddedVideo elements included in any of our QC reports at this point.
How would we want to display this element? For a MediaLink we're displaying the media document MediaTitle element and the image. What do we want to display for a video?
Should we display a placeholder, the link to YouTube, or nothing along with the MediaTitle?

Comment entered 2018-03-28 17:01:16 by Englisch, Volker (NIH/NCI) [C]

After discussing the report with Robin we decided to represent the element EmbeddedVideo with an image placeholder for the video and a link to YouTube allowing the user to view the image.

The following program has been modified:

Comment entered 2018-04-09 09:52:21 by Osei-Poku, William (NIH/NCI) [C]

The full QC report returns a python script error. Here is a sample GTC 622340

Comment entered 2018-04-09 11:19:50 by Englisch, Volker (NIH/NCI) [C]

The report works for me. What is the error you're getting?

Comment entered 2018-04-09 11:27:41 by Osei-Poku, William (NIH/NCI) [C]

This is the error message:

-> -> -->
A problem occurred in a Python script.

D:\cdr\Log\tmprc4pma.html contains the description of this error.

Comment entered 2018-04-09 11:33:58 by Englisch, Volker (NIH/NCI) [C]

Arghh! Today is not my day. I was looking on DEV because I tested the New Published Glossary Terms report.
I'll check it out on QA now.

Comment entered 2018-04-09 14:49:09 by Englisch, Volker (NIH/NCI) [C]

You had included Spanish SpecificMediaTitle elements that my code didn't handle properly.

Please try again on QA.

Comment entered 2018-04-09 15:36:23 by Osei-Poku, William (NIH/NCI) [C]

It is showing now. Thanks!

Comment entered 2018-04-10 12:03:35 by Englisch, Volker (NIH/NCI) [C]

The program did not display the SpecificMediaTitle instead of the VideoID title if it exists. This has been fixed in

Comment entered 2018-04-10 15:25:05 by Juthe, Robin (NIH/NCI) [E]

Volker, would it be possible to make the video screenshot a bit smaller on this QC report? I don't know if the illustration size is standardized on the QC report, but it could be the same size as that if it is. Thanks.

Comment entered 2018-04-10 15:57:59 by Englisch, Volker (NIH/NCI) [C]

I'm guessing the size is the same as the size of the video would be.
I don't think we have any "standard" for the size of the images. We can start to create a standard now. :-)
The image above is 300px wide. Should we try the same width for the screenshot?

Comment entered 2018-04-10 16:41:17 by Englisch, Volker (NIH/NCI) [C]

Please have a look on DEV for the modified video display size.

Comment entered 2018-04-10 16:54:17 by Englisch, Volker (NIH/NCI) [C]

Updated the following program to adjust the image size:

Comment entered 2018-04-11 14:08:37 by Juthe, Robin (NIH/NCI) [E]

This looks good to me on DEV. Please promote to QA. Thanks!

Comment entered 2018-04-11 15:02:56 by Englisch, Volker (NIH/NCI) [C]

The changes have been copied to QA.

Comment entered 2018-04-11 15:09:05 by Juthe, Robin (NIH/NCI) [E]

Verified on QA. Thanks!

Comment entered 2018-05-31 12:57:35 by Osei-Poku, William (NIH/NCI) [C]

Verified on PROD. Thanks!

Elapsed: 0:00:00.001206