Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix issue with custom organism cytogenetic band files #308

Merged
merged 5 commits into from
Jul 23, 2022

Conversation

NicoNekoru
Copy link
Contributor

No breaking changes and doesn't affect anything unless user specifies organismMetadata and taxid in config.

Not a true fix for #307 but resolves the issue if the parameters organismMetadata and taxid are specified

@coveralls
Copy link

coveralls commented Jul 22, 2022

Coverage Status

Coverage increased (+0.3%) to 87.592% when pulling decd25d on NicoNekoru:master into e805cf7 on eweitz:master.

@NicoNekoru
Copy link
Contributor Author

I do not remember what I changed but apparently more things than just the commits were changed

Going through the minified files to see what changed

Copy link
Owner

@eweitz eweitz left a comment

Choose a reason for hiding this comment

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

Thanks, @NicoNekoru! This looks good. I appreciate your detailed description of the issue with chickpea (Cicer arietinum) data over in #307.

I'll add you to AUTHORS. What name and email address should I use?

Also, if/when your genomics application is public, I'd be interested to see it!

@eweitz eweitz merged commit 2347493 into eweitz:master Jul 23, 2022
@NicoNekoru
Copy link
Contributor Author

Sounds great,

Name: Kai Hidajat
Email: kai.hidajat@gmail.com

@eweitz eweitz changed the title Fix #307 with an optional config parameter organismMetadata Fix issue with custom organism cytogenetic band files Aug 16, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants