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

Update Pilot3 programs for .zip build+run with pilot3utils namespace and library reference #40

Closed
robertdevine opened this issue Mar 15, 2024 · 8 comments · Fixed by #41
Assignees
Labels
enhancement New feature or request fda pilots

Comments

@robertdevine
Copy link
Collaborator

robertdevine commented Mar 15, 2024

With the .zip package installation instructions, the programs need updated reference to the package namespace and library for build and run. See here for details. thx.

@robertdevine robertdevine self-assigned this Mar 15, 2024
@robertdevine robertdevine added enhancement New feature or request fda pilots labels Mar 15, 2024
@laxamanaj
Copy link
Collaborator

Hi, @robertdevine

I just saw this. Do you suggest we do all of the testing from this branch before we move to main? I was about to run the copy script in main to bring in all of the current programs and documents from the project repo into here.

anyone have a preference?

@bms63 @AlexandraP-21 @kaz462

@laxamanaj
Copy link
Collaborator

Hi, @robertdevine

I just saw this. Do you suggest we do all of the testing from this branch before we move to main? I was about to run the copy script in main to bring in all of the current programs and documents from the project repo into here.

anyone have a preference?

@bms63 @AlexandraP-21 @kaz462

Just to save time, I went ahead and just copied everything to this branch. When doing testing and downloading the eCTD pilot 3 submission structure, please make sure to switch to this 40 branch first, then download the .zip.

image

The adrg.pdf in the m5 should also be the latest and updated. Fixed the layouts and manually fixed the wrapping. Test away!!!

@bms63
Copy link
Collaborator

bms63 commented Mar 22, 2024

Roger roger!

@robertdevine
Copy link
Collaborator Author

Hi @laxamanaj - yes, that branch selection should be appropriate. So far, the testing is going very well. During the testing if you start with the renv-lock.txt from the Installation Instructions issue (closed earlier today) then snapshot to install pilot3utils your test sandbox actually matches what we have been doing here - so, in summary, using the branch you identified here to complete the tests prior to merge onto main should be fine. I have run into no issues -including fast forwarding all code changes into the latest source changes. thx

@robertdevine
Copy link
Collaborator Author

robertdevine commented Mar 22, 2024

@laxamanaj , @bms63 - just to be certain pilot3 team is aware the difference between the .txt lock provided with the Installation Instructions - the slight difference is due to the snapshot when the {pilot3utils} package gets appended to the lock during the installation process. A nuance of difference to be aware of since the actual {pilot3utils} package dependency appears in the proprietary pilot3 lock file for folks cloning, forking, sub-modeling, etc. thx.

image

@bms63
Copy link
Collaborator

bms63 commented Mar 22, 2024

I think pilto3utils in renv.lock file could use a refresh - pilto3utils should be Version 0.0.2

@robertdevine
Copy link
Collaborator Author

Thx. @bms63
image

All programs/outputs run successfully. I'll issue the PR. thx. again.

@bms63
Copy link
Collaborator

bms63 commented Mar 22, 2024

Thanks Robert!!

laxamanaj added a commit that referenced this issue Apr 12, 2024
…-zip-build+run-with-pilot3utils-namespace-and-library-reference

Closes #40 update pilot3 programs for zip build+run with pilot3utils namespace and library reference
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fda pilots
Projects
None yet
5 participants