-
Notifications
You must be signed in to change notification settings - Fork 49
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
Code quality #154
Code quality #154
Conversation
|
||
|
||
rect = read_dictionary('scaling.csv') | ||
rect = read_dictionary('scaling.csv') |
Check notice
Code scanning / CodeQL
Unused global variable Note
del self.hetmols[k] | ||
try: # RM 2023/04/22: I don't think this is necessary as stripAtoms takes care of deleting the hetmol | ||
del self.hetmols[k] | ||
except KeyError: |
Check notice
Code scanning / CodeQL
Empty except Note
smicat_zero = temp_ligand['coord_atoms_zero_index'].copy() | ||
if 'frag_func_smiles' in list(ligand.keys()): | ||
smiles = temp_ligand['frag_func_smiles'] | ||
original = temp_ligand['frag_func_smiles'] |
Check warning
Code scanning / CodeQL
Variable defined multiple times Warning
redefined
original = temp_ligand['frag_func_smiles'] | ||
else: | ||
smiles = temp_ligand['macrocycle_smiles'] | ||
original = temp_ligand['macrocycle_smiles'] |
Check warning
Code scanning / CodeQL
Variable defined multiple times Warning
redefined
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #154 +/- ##
==========================================
+ Coverage 25.66% 25.75% +0.08%
==========================================
Files 102 102
Lines 33600 33652 +52
==========================================
+ Hits 8624 8667 +43
- Misses 24976 24985 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Latest batch of type hints and new test cases