Skip to content

From main branch to dev. Delete main after merge. #2

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.

Sign up for GitHub

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

Jump to bottom
Merged
sfluegel05 merged 37 commits into from
May 19, 2025
Merged

From main branch to dev. Delete main after merge. #2

sfluegel05 merged 37 commits into from
May 19, 2025

Conversation

@aditya0by0
Copy link
Member

@aditya0by0 aditya0by0 commented May 15, 2025

No description provided.

aditya0by0 and others added 30 commits April 3, 2025 16:27
@aditya0by0
add protein-related code from https://github.com/ChEB-AI/python-chebai
2519852
@aditya0by0
remove chebi imports in init.py
7221a9e
@aditya0by0
changes for fix
2422518
@aditya0by0
change loss module for protein data
64d7623
@aditya0by0
update trainer for protein reader
beaf74e
@aditya0by0
remove chebi imports and libraries
9fd19a9
@aditya0by0
remove chebi version param from base data class
7048cd0
@aditya0by0
electra config: update vocab size & max pos for protein seq
9c8521f
@sfluegel05
add changes from out_dim PR (#74 in python-chebai)
b45b266
@aditya0by0
remove not required files
a293931
@aditya0by0
Update .gitignore
9120538
@aditya0by0
update readers for proteins
6d7e6bd
@aditya0by0
import offset constants from chebai + remove its worflow
83e3342
@aditya0by0
rename base folder to chebai_proteins
22815fb
@aditya0by0
update notebook for chebai_proteins root
68d4040
@aditya0by0
add chebai repo to to setup.py
78d79da
@aditya0by0
Update setup.py
8dce9cb
@aditya0by0
update unit test
71e361e
@aditya0by0
fix imports from chebai_proteins
3819fd3
@aditya0by0
BCELoss config for deepgo2
ab9bd1c
@aditya0by0
scope esm2 config
dcbd578
@aditya0by0
MultilabelAUROC for deepgo MLP
1b2856d
@aditya0by0
update migration script
31b6f45
@aditya0by0
update configs
6c2506d
@aditya0by0
make python dir
19ab4a7
@aditya0by0
deepgo: raise error if no classes are selected
add85e3
@aditya0by0
rectify consistent naming of scope
c89f26d
@aditya0by0
reader: add collator to esm reader
196d662
@aditya0by0
set weight_only=False for esm reader
5af20c8
@aditya0by0
use TokenIndexerReader for ProteinDataReader
d653f52
@aditya0by0 aditya0by0 requested a review from sfluegel05 May 15, 2025 12:49
@aditya0by0 aditya0by0 self-assigned this May 15, 2025
@sfluegel05 sfluegel05 merged commit 919064d into dev May 19, 2025
6 checks passed
@sfluegel05
Copy link
Contributor

sfluegel05 commented May 19, 2025

I don't see the need the delete the main branch. In general, the idea is that one does development on the dev branch and for releases, merges that into the main branch. Tbh in #1 I hadn't seen that it was merging into the main branch, I assumed it was merging into dev.

My suggestion is to get dev on the same level as main (as you have done with this PR) and then continue on the dev branch (without deleting main).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sfluegel05 sfluegel05 Awaiting requested review from sfluegel05

Assignees

@aditya0by0 aditya0by0

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aditya0by0 @sfluegel05