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

Feature/pass reads #361

Merged
merged 9 commits into from
Aug 26, 2024
Merged

Feature/pass reads #361

merged 9 commits into from
Aug 26, 2024

Conversation

Adoni5
Copy link
Contributor

@Adoni5 Adoni5 commented Jun 19, 2024

  1. Uses the pass_reads function on the caller, rather than individual calls to pass_read, should be faster and has removed sleeping.
  2. Tidies up the example reads in the explainer about sub_tag and makes them comments.
  3. Added a log statement about generating the aligner description, as it felt to me that after logging "starting main loop" it was hanging, when in actuality a large reference takes a while to generate a description for.

N.B Committed as @satriobio from the remote Vs code server, is this identity theft?

@Adoni5 Adoni5 mentioned this pull request Jun 19, 2024
6 tasks
mattloose
mattloose previously approved these changes Jul 9, 2024
Copy link
Contributor

@mattloose mattloose 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 to me.

Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Aug 24, 2024
* Made a proper SVG thought i'd just include it here

* Remove setting the read_number on the result
Add in the correct strand classifications for minknow 6

* Remove Read number from Result and all references to read number

* Update target versions for MinKNOW to 6.0.0 - 6.0.0
Explicitly exit if running on MInKNOW 5.X.X

* Update vendored read until API to v3.5.2

* Update test for conencted base caller version to work

* Pin ont-pybasecall-client-lib at 7.4.12

* Add downgrade to 2024.2.0 to error message

* Fix bug in reads not unblocking as expected #372

* change list() to []

* Chunk tracker uses read_id not read_number, update trackers internal variable names

* bump version to 2024.3.0

* Fix tests for the new MinKNOW version compatibility
Fix tests for the _statistics too not use `read_number`

* Bump summarise version to 0.2.7

* Drop incorrect gotcha

* Feature/deprecate guppy (#374)

* This commit addresses the removal of guppy.py, partial removal of guppy tests, edits to documentation to largely replace guppy with dorado, removes the ont-pyguppy-client-lib and adds a warning to the readme. It also adds some extra classes to the read_until init in targets.

* Remove builtin module for guppy

* Update FAQ questions to focus dorado as well as Guppy

* Change validate caller warning to mention dorado not guppy

* Change readme warnings to Focus dorado and minknow breaking changes more
Add link/requirement for dorado_basecall_server not guppy_basecall_server

* Rename tests to dorado validation tests

* Add dorado to mappy-test install target
alexomics
alexomics previously approved these changes Aug 26, 2024
@Adoni5
Copy link
Contributor Author

Adoni5 commented Aug 26, 2024

Fixes #372 in 9ee6b8b, caused by extra clipping shenanigans of the base-called Results read-id. Was losing the first 3 UUID chars, so no read existed to unblock.

@Adoni5 Adoni5 merged commit 8c013a4 into main Aug 26, 2024
9 checks passed
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.

4 participants