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

chore!: remove localizer #379

Merged
merged 5 commits into from
Apr 4, 2024
Merged

chore!: remove localizer #379

merged 5 commits into from
Apr 4, 2024

Conversation

korikuzma
Copy link
Contributor

@korikuzma korikuzma commented Apr 2, 2024

close #268 . Do we even want to keep this? I can clean it up more. Just wanted to get something out

* rather than having Chromosome Locations, allow for cytoband parameters to be passed

Update: We have decided to remove the localizer for now

* rather than having Chromosome Locations, allow for cytoband parameters
  to be passed
@korikuzma korikuzma added enhancement New feature or request priority:low Low priority labels Apr 2, 2024
@korikuzma korikuzma self-assigned this Apr 2, 2024
@korikuzma korikuzma requested review from a team as code owners April 2, 2024 01:53
@korikuzma korikuzma changed the title feat!: update localizer chore!: remove localizer Apr 3, 2024
@korikuzma korikuzma added chore Changes that do not relate to a fix or feature and don't modify src or test files and removed enhancement New feature or request labels Apr 3, 2024
Copy link
Contributor

@larrybabb larrybabb left a comment

Choose a reason for hiding this comment

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

+1

@larrybabb larrybabb merged commit e057610 into main Apr 4, 2024
8 checks passed
@larrybabb larrybabb deleted the issue-268 branch April 4, 2024 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not relate to a fix or feature and don't modify src or test files priority:low Low priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Localizer
2 participants