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

Add a second baseline group and a migration to port existing result sets #1139

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

arteymix
Copy link
Member

@arteymix arteymix commented Jun 6, 2024

Add a secondBaselineGroup to our DEA result set model and use it when serializing VO instead of populating it on-the-fly.

TODO:

  • migrate existing result sets (script is ready, but that will affect baseline filling in production, so we have to hold it off)
  •  fill the second baseline group in LinearModelAnalyzer
  • remove the logic for populating baselines in ExpressionAnalysisResultSetDaoImpl

@arteymix
Copy link
Member Author

arteymix commented Jun 7, 2024

@ppavlidis maybe you can have a look at the populating interaction baseline part? This is a bit challenging because factors are held in an unsorted collection and the final picked order has to match what is used in the contrast results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant