Details
-
Type:
Task
-
Status: Closed (View Workflow)
-
Priority:
Major
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:None
-
Story Points:0.5
-
Sprint:Fall 4 2023 Oct 16, Fall 5
Description
To deploy Shiny Apps, we need to provide code for making plots. However, our current shared code in Common.R loads edgeR and DESeq2 libraries. As a result, any time a Shiny app loads the code in Common.R, it also goes through a long process of loading up those two libraries, which it doesn't actually need, since the R Shiny apps are only providing visualization, not statistical processing or analysis. So, let's move code that requires edgeR and/or DESeq2 out of Common.R into a different file.
To do this, we'll need to modify a lot of the .Rmd files that use Common.R. They may need to now import both Common.R and the new statistical functions .R file instead of just Common.R.
Suggestion for implementation strategy:
- Loraine and/or Davis migrate the code out of Common.R?
- Loraine and/or Davis update the rest of the code base?
Attachments
Activity
| Field | Original Value | New Value |
|---|---|---|
| Epic Link | IGBF-3446 [ 22548 ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
| Assignee | Ann Loraine [ aloraine ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
| Assignee | Molly Davis [ molly ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
| Assignee | Molly Davis [ molly ] |
| Status | Post-merge Testing In Progress [ 10003 ] | To-Do [ 10305 ] |
| Assignee | Ann Loraine [ aloraine ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
| Status | Post-merge Testing In Progress [ 10003 ] | To-Do [ 10305 ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Comment | [ Pushed branch with changes to my fork. Commits are named for IGBF-3459 reflecting how the repository "common" code was refactored. Submitting PR to fix the errors with the repository. ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
| Assignee | Ann Loraine [ aloraine ] |
| Sprint | Fall 4 2023 Oct 16 [ 180 ] | Fall 4 2023 Oct 16, Fall 5 [ 180, 181 ] |
| Rank | Ranked higher |
| Assignee | Molly Davis [ molly ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
| Resolution | Done [ 10000 ] | |
| Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
| Assignee | Molly Davis [ molly ] | Ann Loraine [ aloraine ] |