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

New Feature - deselect compounds, coloured and highlighted class boxes. #63

Merged
merged 3 commits into from
Sep 11, 2018

Conversation

ricgillams
Copy link
Contributor

  • Add colours to class name inputs
  • Highlight input for current class
  • Introduced use of zero key to remove compound from toBuyList
  • Removed unnecessary imports and clean code based on LGTM suggestions

ricgillams and others added 3 commits September 11, 2018 16:06
- Fixed unused imports
- Refactored dead code
- Add var instantinations
- Add LGTM badge
- Add colours to class name inputs
- Highlight input for current class
- Introduced use of zero key to remove compound from toBuyList
- Removed more unnecessary imports and clean code based on LGTM suggestions
@abradle
Copy link
Contributor

abradle commented Sep 11, 2018

This pull request introduces 2 alerts and fixes 69 when merging 1ec399f into b686c66 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

fixed alerts:

  • 66 for Unused variable, import, function or class
  • 3 for Expression has no effect

Comment posted by LGTM.com

@abradle abradle merged commit a95db79 into xchem:master Sep 11, 2018
reskyner pushed a commit that referenced this pull request Jun 14, 2019
* In cases where the structures have not been handled through the XChem pipeline, no bound structures are available.
* A readme is added for information in place of empty pdb files.
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