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

TASK-5512 - Fix the resource URL configuration for OpenCGA analysis #2390

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

jtarraga
Copy link
Member

@jtarraga jtarraga commented Feb 6, 2024

Add resource URL into the configuration file

@jtarraga
Copy link
Member Author

jtarraga commented Feb 6, 2024

@jtarraga jtarraga requested a review from j-coll February 7, 2024 07:29
@jtarraga jtarraga changed the title TASK-5512 TASK-5512 Fix the resource URL configuration for OpenCGA analysis Feb 7, 2024
j-coll
j-coll previously approved these changes Feb 7, 2024
@@ -65,4 +65,16 @@ default AlignmentStorageManager getAlignmentStorageManager() throws ToolExecutor
}
}

// default AnalysisResourceUtils getAnalysisResourceUtils() throws ToolExecutorException {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting brand new code!

Resolve Conflicts:
	opencga-analysis/src/main/java/org/opencb/opencga/analysis/ResourceUtils.java
	opencga-analysis/src/main/java/org/opencb/opencga/analysis/family/qc/IBDComputation.java
	opencga-analysis/src/main/java/org/opencb/opencga/analysis/variant/mutationalSignature/MutationalSignatureAnalysis.java
	opencga-core/src/main/java/org/opencb/opencga/core/config/Analysis.java
	opencga-server/src/main/java/org/opencb/opencga/server/rest/analysis/VariantWebService.java
j-coll
j-coll previously approved these changes Apr 29, 2024
…ml, #TASK-5512

ConfigurationUtils is moved from the opencga/analysis to opencga/core/config, therefore imports are updated for some files.
…or's parameter name 'opencgaHome' to locate the configuration file, #TASK-5512
@j-coll j-coll changed the title TASK-5512 Fix the resource URL configuration for OpenCGA analysis TASK-5512 - Fix the resource URL configuration for OpenCGA analysis May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants