generated from pharmaverse/admiraltemplate
-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
5 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
a1127a3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @vinhn23, I have some small additional comments below:
I see update in code but SUPPRS dataset is not refreshed and I still see "01-701-1028" before changes
line 39
"01-701-1034" has "NON-CR/NON-PD" as RECIST1.1 overall response, please update
line 118-120
"01-701-1028" has PD, is flagged as not evaluable with baseline level not elevated , but as a reason of PD we have CNOTNORM = Y instead of CANORM2X = Y (category C of PD)
line 136
"01-701-1015" took mouse antibodies but there is no SUPPRS record for combined response. Should we add such a record? I exclude records after mouse antibodies for RSCAT="CA-125", but I assume this should be the case for RSCAT="RECIST 1.1 - CA125" records
a1127a3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@starosto items 1,2,3 should be resolved. I added a record for the mouse antibodies; however, I am unsure how that would fit into the derivation of the combined response since we do not have a combined crf page or guidance. @amitjaingsk do you have any insight on item 4?