-
Notifications
You must be signed in to change notification settings - Fork 45
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
TaxVAMB #204
TaxVAMB #204
Conversation
aaaa2a7
to
4eda306
Compare
058fc88
to
fc26518
Compare
1236af3
to
6ab6d0f
Compare
""" | ||
The following code is a modification of a k-means based reclustering algorithm first published at https://github.com/BigDataBiology/SemiBin | ||
The original code is distributed under MIT License. | ||
""" | ||
|
||
import os | ||
import subprocess | ||
import tempfile | ||
import sys | ||
import contextlib | ||
from typing import IO | ||
from sklearn.cluster import KMeans | ||
import numpy as np | ||
from collections import defaultdict | ||
import pandas as pd | ||
from sklearn.cluster import DBSCAN | ||
from sklearn.metrics import pairwise_distances | ||
import gzip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re this file: Let's discuss the plan for removing it when I'm back in the office. I.e. if we manage to implement SCG in the latent space, how can we excise this module, and all the related functionality in a clean way?
#!/usr/bin/bash | ||
module load tools computerome_utils/2.0 | ||
module load anaconda3/2023.03 | ||
module unload gcc | ||
module load gcc/11.1.0 | ||
module load minimap2/2.17r941 samtools/1.10 | ||
|
||
source ~/.bashrc | ||
conda init bash | ||
conda activate /home/projects/cpr_10006/people/svekut/.conda/vamb | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should all these Computerome scripts be part of the repo? They're very dev-specific for our group. Perhaps have them in their own folder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nah, they will be deleted
Implementation of the predictor, VAEVAE and hierarchical loss, all in one commit
The squashed version of PR#184. Keeping the old PR to not lose the comments.