Skip to content

Conversation

@MuhammedHasan
Copy link
Collaborator

No description provided.

@jkanche jkanche changed the base branch from main to refactor May 14, 2025 21:40
Copy link
Collaborator

@jkanche jkanche left a comment

Choose a reason for hiding this comment

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

Let's also start adding tests as we go so we are not overwhelmed at the end

Otherwise looks great!

@MuhammedHasan MuhammedHasan merged commit 44d730b into Genentech:refactor May 28, 2025
5 checks passed
avantikalal pushed a commit that referenced this pull request Jun 18, 2025
* setting up the folder structure

* move scripts to cli and setting up package meta for cli access

* restructure the rest of the files

* Refactor attributions (#8)

* attribution refactoring low level api design

* updates of api of attirbutions and cli

* attribution cli

* wandb login, resources -> hub, and depreated tools.interpret

* update for multiple genes

* cli update to support multiple genes and attribution methods

* testcase fix

* test ci fix

* genome

---------

Co-authored-by: Muhammed Hasan Celik <muhammedhasancelik@gmail.com>

* vep added

* vep implemented

* vep logs and api update

* decima vep bug fix

* bug fix

* tangermeme version

* ci

* attr window bug

* fix of typo and depreceated

* mutate function deleted and fix of string literal

* Refactor vep (#9)

* vep added

* vep implemented

* vep logs and api update

* decima vep bug fix

* bug fix

* tangermeme version

* ci

* attr window bug

* fix of typo and depreceated

* mutate function deleted and fix of string literal

---------

Co-authored-by: Muhammed Hasan Celik <celik.muhammed_hasan@gene.com>

* pr code review

---------

Co-authored-by: Muhammed Hasan Celik <muhammedhasancelik@gmail.com>
Co-authored-by: Muhammed Hasan Celik <celik.muhammed_hasan@gene.com>
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