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

add pilot3 notes from FDA feedback #95

Open
laxamanaj opened this issue Nov 3, 2023 · 5 comments
Open

add pilot3 notes from FDA feedback #95

laxamanaj opened this issue Nov 3, 2023 · 5 comments
Assignees

Comments

@laxamanaj
Copy link
Collaborator

From the R Consortium R Submissions WG meeting on 03Nov2023, some feedback was given to Pilot 3 submission :

  1. Currently, FDA may test on different R environments (i.e. one reviewer using Rv4.2.1 and other using Rv4.2.3). ADRG states to use Rv4.2.3.
  2. Pilot 3 package didn't install using renv::install() when using Rv4.2.1, but did install when reviewer was using Rv4.2.3.
  3. With the submission of specifications adam-pilot-3.xlsx, this wasn't updated in our ADaM programs to match the file name appropriately. Reviewers were able to update the file name convention appropriately and adsl.r ran fine.
  4. When noting to change source data path in the ADRG, do not include paths to change to in the steps, instead keep note open just to mention to reviewer that "they should change the source data path to the location of where the data was downloaded to from the EDR".

Action : to include these notes in the minutes.

@laxamanaj laxamanaj self-assigned this Nov 3, 2023
@saghirb
Copy link

saghirb commented Nov 6, 2023

As a follow-up to the discussions in the WG meeting, would be it useful to include a file with check sums (e.g. MD5 sums) for all the files in this package? It could be useful in identifying any differences post downloaded.

@laxamanaj
Copy link
Collaborator Author

As a follow-up to the discussions in the WG meeting, would be it useful to include a file with check sums (e.g. MD5 sums) for all the files in this package? It could be useful in identifying any differences post downloaded.

Thanks, @saghirb . Are you talking about running this : https://rdrr.io/r/tools/md5sum.html

@saghirb
Copy link

saghirb commented Nov 8, 2023

As a follow-up to the discussions in the WG meeting, would be it useful to include a file with check sums (e.g. MD5 sums) for all the files in this package? It could be useful in identifying any differences post downloaded.

Thanks, @saghirb . Are you talking about running this : https://rdrr.io/r/tools/md5sum.html

@laxamanaj yes I am. I use it in conjunction with list.files() to write (via a data.frame) a .md5 file that contains hashes and respective filenames. On Unix based OSes you can verify the integrity of files (data) using md5sum -c filename.md5.

If it helps, I am happy to provide example code.

@laxamanaj
Copy link
Collaborator Author

Thanks for your patience in my response @saghirb . Sure , would be interested in a sample code you may have to share. Thanks!

@saghirb
Copy link

saghirb commented Dec 28, 2023

Apologies for the delay in my response @laxamanaj. I have created a GitHub repo with some Demo code for tools:md5sum at:

Let me know if this helps or if it is not clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants