-
Notifications
You must be signed in to change notification settings - Fork 468
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
Implement molecular-profile-sample-counts endpoint #10798
base: demo-rfc80-poc
Are you sure you want to change the base?
Conversation
fa6c559
to
2f71790
Compare
import org.cbioportal.model.ClinicalData; | ||
import org.cbioportal.model.ClinicalDataCount; | ||
import org.cbioportal.model.Sample; | ||
import org.cbioportal.model.*; |
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.
i think IDE did this. what's right?
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.
Do you need help with your ide settings?
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.
Looks good (Now you are a backend dev.) Just small changes
@Override | ||
public List<GenomicDataCount> getGenomicDataCounts(StudyViewFilter studyViewFilter, CategorizedClinicalDataCountFilter categorizedClinicalDataCountFilter) { | ||
|
||
//return new ArrayList<GenomicDataCount>(); |
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.
Remove
|
||
<!-- for /molecular-profile-sample-counts/fetch (returns GenomicDataCount) --> | ||
<select id="getGenomicDataCounts" resultType="org.cbioportal.model.GenomicDataCount"> | ||
SELECT genetic_profile.stable_id AS value, genetic_profile.stable_id AS value2, genetic_profile.name AS label, count(sample_profile.genetic_profile_id) AS count FROM sample_profile |
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.
Can we have each column on a new line? (just easier to read)
Ex.
Select
stable_id,
name
@@ -13,6 +13,26 @@ | |||
#{studyId} | |||
</foreach> | |||
</if> | |||
|
|||
|
|||
<if test="studyViewFilter.genomicProfiles != null and !studyViewFilter.genomicProfiles.isEmpty()"> |
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.
@gblaih here is the filter. i found a small problem with foreach at the bottom but this is the basic idea.
f744d2e
to
c6b9031
Compare
No description provided.