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 Cistus crispus #164

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

ldemirdj
Copy link
Collaborator

Assembly review request

  • ToLID: ddCisCris1
  • Species: Cistus crispus
  • Project: ERGA-BGE
  • Affiliation: Genoscope

Copy link
Contributor

erga-ear-bot bot commented Feb 24, 2025

Hi @ldemirdj, thanks for sending the EAR of Cistus crispus.
I added the corresponding tag to the PR and will contact a supervisor and a reviewer ASAP.

Copy link
Contributor

erga-ear-bot bot commented Feb 24, 2025

Hi @diegomics, do you agree to supervise this assembly?
Please reply to this message only with OK to give acknowledge.

@diegomics
Copy link
Collaborator

Ok

Copy link
Contributor

erga-ear-bot bot commented Feb 24, 2025

*****
EAR Reviewer Selection Process
Date: 2025-02-24 10:38

All Eligible Candidates:

Github ID  | Full Name      | Institution | Total Reviews | Last Review | Active | Busy | Calling Score | Adjusted Score
------------------------------------------------------------------------------------------------------------------------
DomAbsolon | Dom Absolon    | Sanger      | 3             | 2024-11-15  | Y      | N    | 1008          | 1058          
diegomics  | Diego De Panis | IZW         | 8             | 2024-10-22  | Y      | N    | 991           | 986           

Selected reviewer: Dom Absolon (DomAbsolon)
The decision was based on:
- different institution ('Sanger')
- active ('Y')
- not busy ('N')
- highest adjusted calling score in this particular selection (1058)

Copy link
Contributor

erga-ear-bot bot commented Feb 24, 2025

Hi @DomAbsolon, do you agree to review this assembly?
Please reply to this message only with Yes or No by 28-Feb-2025 at 15:38 CET

@diegomics
Copy link
Collaborator

diegomics commented Feb 24, 2025

Quicky went through the assembly, looks nice.
I think ddCisCris1_3 @ 3,2-3,5kb should be placed in the gap at ddCisCris1_4 @ 182,35kb, right?
Let's wait for the review...

@DomAbsolon
Copy link
Collaborator

DomAbsolon commented Feb 25, 2025 via email

@erga-ear-bot erga-ear-bot bot requested a review from DomAbsolon February 25, 2025 08:13
Copy link
Contributor

erga-ear-bot bot commented Feb 25, 2025

Thanks for agreeing!
I appointed you as the EAR reviewer.
I will keep your status as Busy until you finish this review.
Please check the Wiki if you need to refresh something. (and remember that you must download the EAR PDF to be able to click on the link to the contact map file!)
Contact the PR assignee for any issues.

@DomAbsolon
Copy link
Collaborator

ddCisCris1_2 is telo joined. From the start to the first gam needs to break and move to the opposite end. The join does not look great but you can tell by both the tell signal and the centromere signal. The small piece at the start of _3 also needs to move to _4 as suggested by @diegomics

@ldemirdj
Copy link
Collaborator Author

Hi @diegomics and @DomAbsolon,

Thanks for your feedback. I will make the changes and regenerate the EAR report.

Best,

Lola

Copy link
Contributor

erga-ear-bot bot commented Mar 3, 2025

Attention @DomAbsolon, the EAR PDF was updated.

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

Successfully merging this pull request may close these issues.

3 participants