Conversation
✅ Deploy Preview for monarch-app ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1225 +/- ##
=======================================
Coverage 71.84% 71.84%
=======================================
Files 92 92
Lines 3232 3232
=======================================
Hits 2322 2322
Misses 910 910 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
This ticket may be unnecessary.
I think this issue is actually the medgen g2d data issue that is being worked on (#1226) because diseases shouldn't show more than 1 casual gene. so I think this change is unnecessary. Once the data is fixed, it will fix any weird formatting issues. @kevinschaper can you confirm this please? The max causal genes that should display at the top would be 2 but pretty much all will have 1 if they have a causal gene.
That being said, If Kevin confirms this is true, we can make the formatting change, I just don't think it's necessary.
Related issues
Summary
Checks