Skip to content

Chnage the TMB number in the blue circle of the report from in-house TMB to Illumina TMB #104

Merged
xiaoliz0 merged 4 commits intomainfrom
develop_issue100
Apr 29, 2026
Merged

Chnage the TMB number in the blue circle of the report from in-house TMB to Illumina TMB #104
xiaoliz0 merged 4 commits intomainfrom
develop_issue100

Conversation

@xiaoliz0
Copy link
Copy Markdown
Contributor

Request from the national molecular biology group:
Change the TMB number in the blue circle of report from in-house TMB to Illumina TMB. And the number will be rounded when reading into the report.

…TMB to Illumina TMB. And the number needs to be rounded. This is the request from national molecular biology group. GitHub issue 100.
Copy link
Copy Markdown
Collaborator

@marrip marrip left a comment

Choose a reason for hiding this comment

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

Hey, I think your changes are quite straight forward but I would consider to make this part in the code DRYer. Please have a look at my suggestion and let me know what you think ☺️

Comment thread Script/PRONTO.py Outdated
xiaoliz0 and others added 3 commits April 28, 2026 12:29
Co-authored-by: Martin Rippin <74295098+marrip@users.noreply.github.com>
…rrect order numbers for the textbox in report.
Copy link
Copy Markdown
Collaborator

@marrip marrip left a comment

Choose a reason for hiding this comment

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

Looks good, nice work, @xiaoliz0 ! 👏

@xiaoliz0 xiaoliz0 merged commit 4bd6112 into main Apr 29, 2026
1 check passed
@marrip marrip deleted the develop_issue100 branch April 29, 2026 07:16
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.

Change the TMB number in the blue circle of report from in-house TMB to Illumina TMB

3 participants