CDR Tickets

Issue Number 3380
Summary [Terminology] Python script error during import
Created 2011-06-22 14:43:29
Issue Type Bug
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2011-07-13 15:20:43
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.107708
Description

BZISSUE::5073
BZDATETIME::2011-06-22 14:43:29
BZCREATOR::William Osei-Poku
BZASSIGNEE::Bob Kline
BZQACONTACT::William Osei-Poku

Users are getting a python script error when importing terms from the NCI Thesaurus. This error is different from the usual error when a term is not available for import due to the lag time.

Example of the error message:
d:\cdr\Log\tmpvj_f4h.html contains the description of this error. D:\Python\lib\cgitb.py:173: DeprecationWarning: BaseException.message has been deprecated as of Python 2.6 value = pydoc.html.repr(getattr(evalue, name))

Examples of terms that produced errors on Bach"

C95894/696436
C95894/696436
C96036/696523

Comment entered 2011-06-22 15:38:15 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2011-06-22 15:38:15
BZCOMMENTOR::Bob Kline
BZCOMMENT::1

Looks more like a bug than an enhancement to me, don't you think?

Comment entered 2011-06-22 15:48:45 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2011-06-22 15:48:45
BZCOMMENTOR::Bob Kline
BZCOMMENT::2

(In reply to comment #12 for issue #5004)
> (In reply to comment #11 in the same issue)
> > The rewritten module has been installed on Franck, ready for user testing.
>
> Verified on Franck and Mahler. Please promote to Bach.

Looks like #5004 didn't get tested after all.

I have fixed the bug on Franck and Mahler. Please test all the paths and then I'll promote the fix to Bach.

Comment entered 2011-06-22 16:24:17 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-22 16:24:17
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::3

(In reply to comment #2)
> I have fixed the bug on Franck and Mahler. Please test all the paths and then
> I'll promote the fix to Bach.

Could you elaborate on testing "all the paths" ?

Tested On Mahler and Franck and it seems to be working fine.

Comment entered 2011-06-22 17:12:47 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2011-06-22 17:12:47
BZCOMMENTOR::Bob Kline
BZCOMMENT::4

(In reply to comment #3)

> Could you elaborate on testing "all the paths" ?

Well, there are two check boxes on the import form, so all the valid combinations of those options need to be tested, and with and without specifying a CDR ID. The nature of the bug I just found indicated that #5004 wasn't tested with a definition update.

Comment entered 2011-06-22 17:21:38 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-22 17:21:38
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::5

(In reply to comment #4)
> (In reply to comment #3)
>
> > Could you elaborate on testing "all the paths" ?
>
> Well, there are two check boxes on the import form, so all the valid
> combinations of those options need to be tested, and with and without
> specifying a CDR ID. The nature of the bug I just found indicated that #5004
> wasn't tested with a definition update.

OK. Thanks! They are all tested now. Please promote to Bach.

Comment entered 2011-06-22 17:53:36 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2011-06-22 17:53:36
BZCOMMENTOR::Bob Kline
BZCOMMENT::6

(In reply to comment #5)

> They are all tested now. Please promote to Bach.

Done.

Comment entered 2011-07-13 15:20:43 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-07-13 15:20:43
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::7

(In reply to comment #6)
> (In reply to comment #5)
>
> > They are all tested now. Please promote to Bach.
>
> Done.

So far, we haven't encountered any problems so I am closing this issue and re-open if needed.

Elapsed: 0:00:00.001571