Skip to content

Split recon-count into multiple included files#13

Open
kshakir wants to merge 1 commit into
MacoskoLab:mainfrom
kshakir:ks_recon_count_split_only
Open

Split recon-count into multiple included files#13
kshakir wants to merge 1 commit into
MacoskoLab:mainfrom
kshakir:ks_recon_count_split_only

Conversation

@kshakir

@kshakir kshakir commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@kshakir

kshakir commented Apr 14, 2026

Copy link
Copy Markdown
Contributor Author

This is part 1. After splitting recon-count.jl in this PR, part 2 wraps the split parts in new CLIs.

@kshakir kshakir force-pushed the ks_recon_count_split_only branch from 4c3a6d2 to 86bc4ec Compare May 1, 2026 23:53
Comment thread reconstruction/remove-chimeras.jl Outdated
function remove_chimeras(
df::DataFrame,
metadata::Dict{String,Int64},
):Nothing

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

to be changed to ::Nothing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done, thanks for the catch!

@mukundraj mukundraj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

minor fix request in remove-chimeras.jl

@kshakir kshakir force-pushed the ks_recon_count_split_only branch from 86bc4ec to 794d7ae Compare May 29, 2026 17:34
@kshakir kshakir requested a review from mukundraj May 29, 2026 20:10
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.

2 participants