Skip to content

GitHub Issue #875: LuminexUpgradeCode.checkForMissingSummaryRows#989

Draft
cnathe wants to merge 1 commit intorelease25.11-SNAPSHOTfrom
25.11_fb_luminexSummaryRowUpgradeScript
Draft

GitHub Issue #875: LuminexUpgradeCode.checkForMissingSummaryRows#989
cnathe wants to merge 1 commit intorelease25.11-SNAPSHOTfrom
25.11_fb_luminexSummaryRowUpgradeScript

Conversation

@cnathe
Copy link
Contributor

@cnathe cnathe commented Mar 2, 2026

Rationale

Related to #875 Luminex Multi-File Transform Import Skips First Data Row

For one of the Luminex clients, a set of the assay runs that are missing data rows for the multi-file input case includes rows where the run included both summary and raw data rows and just the summary data rows are missing. In the case of background type rows, these summary rows can be recalculated and inserted into the DB attached to the assay run (dataid, analyteid, type). This PR adds a Java upgrade script which will not be triggered via the regular SQL upgrade path but will instead of available for a site admin to manually run on the client's server.

Related Pull Requests

Changes

  • java upgrade script for the Luminex case of missing summary background type rows for runs that have both summary and raw data

- java upgrade script for the Luminex case of missing summary background type rows for runs that have both summary and raw data
@cnathe cnathe self-assigned this Mar 2, 2026
@cnathe cnathe requested a review from labkey-klum March 2, 2026 20:25
@cnathe
Copy link
Contributor Author

cnathe commented Mar 2, 2026

Step 1: from the admin console > sql scripts > upgrade code, select Luminex checkForMissingSummaryRows and invoke:
Screenshot 2026-03-02 at 1 45 31 PM

Step 2: review log info / warnings to see list of affected / updated runs:
Screenshot 2026-03-02 at 1 45 47 PM

Step 3: review audit logs inserted for rows affected:
Screenshot 2026-03-02 at 1 45 52 PM

@cnathe cnathe marked this pull request as draft March 2, 2026 21:31
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.

1 participant