Skip to content

F1sort plothit - #17

Merged
tensulin merged 3 commits into
add_percentlabelfrom
f1sort_plothit
Mar 31, 2026
Merged

F1sort plothit#17
tensulin merged 3 commits into
add_percentlabelfrom
f1sort_plothit

Conversation

@sjanssen2

Copy link
Copy Markdown
Member

generate Figures like:
image

and sort DE by F1-score, a combination of True Positives and False Positives - highlights more differences to recovery

@github-actions

Copy link
Copy Markdown

Python linting (black) is failing

To keep the code consistent with lots of contributors, we run automated code consistency checks.
To fix this CI test, please run:

  • Install black: pip install black
  • Fix formatting errors in your pipeline: black .

Once you push these changes the test should pass, and you can hide this comment 👍

We highly recommend setting up Black in your code editor so that this formatting is done automatically on save. Ask about it on Slack for help!

Thanks again for your contribution!

annotations = pd.concat(annotations).set_index([0, 'locus_tag'])

# add genomic positions for genes
summary = summary.merge(annotations[[3,4,6, 'organism']].reset_index(), left_on='gene_name', right_on='locus_tag', how='left').rename(columns={0: 'assembly_accession', 3: 'gene_genomic_start', 4: 'gene_genomic_stop', 6: 'strand'})

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

start and end are switched if strand is -
i swapped start and end for the Marbel file so there is a non issue
gff is not entirely accurate, the edgar gffs is in marbel/src/marbel/data/cds_genomic_locations.parquet
for lengths: /vol/jlab/tlin/marbel/reference/lenghts_JLAB2.tsv
assembly_accession currently is the Chromosome ID
Regarding species selection: the species name from the summary is correct, currently cds_genomic_locations.parquet lists chromosome name in species col

@tensulin
tensulin merged commit 8152fe9 into add_percentlabelMar 31, 2026
1 of 6 checks passed
Sign up for freeto 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.

2 participants

@sjanssen2@tensulin