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.

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

Bug fix in SV validator #6786

Merged
merged 1 commit into from
Nov 7, 2019
Merged

Bug fix in SV validator #6786

merged 1 commit into from
Nov 7, 2019

Conversation

ao508
Copy link
Contributor

@ao508 ao508 commented Nov 6, 2019

Describe changes proposed in this pull request:
Fixes NameError exception thrown at

cbioportal/core/src/main/scripts/importer/validateData.py", line 2952, in checkFusionValues

NameError: free variable 'ncbi_build' referenced before assignment in enclosing scope

Signed-off-by: Angelica Ochoa 15623749+ao508@users.noreply.github.com

@ao508
Bug fix in SV validator
26cd3cc 
Signed-off-by: Angelica Ochoa <15623749+ao508@users.noreply.github.com>
@ao508 ao508 requested review from n1zea144 and mandawilson November 6, 2019 21:32
@ao508 ao508 merged commit 99408ca into cBioPortal:master Nov 7, 2019
@ao508 ao508 mentioned this pull request Nov 7, 2019
Merged
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mandawilson mandawilson mandawilson approved these changes

@n1zea144 n1zea144 Awaiting requested review from n1zea144

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ao508 @mandawilson