Flake fixes

molSimplify/Informatics/MOF/MOF_descriptors.py

@@ -246,7 +246,7 @@ def make_MOF_SBU_RACs(SBUlist, SBU_subgraph, molcif, depth, name, cell_v, anchor
                 """""""""
                 functional_atoms = []
                 for jj in range(len(temp_mol.graph)):
-                    if not jj in link_list: # linker atom is not bonded to a metal
+                    if jj not in link_list: # linker atom is not bonded to a metal
                         if not set({temp_mol.atoms[jj].sym}) & set({"C","H"}): # not a carbon nor a hydrogen. syms get symbols.
                             functional_atoms.append(jj)
                 """""""""

molSimplify/Informatics/MOF/MOF_descriptors_alternate_functional.py

@@ -140,7 +140,7 @@ def make_MOF_SBU_RACs(SBUlist, SBU_subgraph, molcif, depth, name,cell,anchoring_
                 for jj in range(len(temp_mol.graph)):
                     #print(link_list)
                     #print(main)
-                    if not jj in main:
+                    if jj not in main:
                         if not set({temp_mol.atoms[jj].sym}) & set({"H"}):
                             functional_atoms.append(jj)
                 print(functional_atoms)

molSimplify/Informatics/MOF/MOF_descriptors_alternate_functional_2.py

@@ -139,7 +139,7 @@ def make_MOF_SBU_RACs(SBUlist, SBU_subgraph, molcif, depth, name,cell,anchoring_
                     #print(main)
                     #print(link_list)
                     if jj in main:
-                        if not jj in link_list:
+                        if jj not in link_list:
                             if not set({temp_mol.atoms[jj].sym}) & set({"C","H"}):
                                 functional_atoms.append(jj)
                     else:

molSimplify/Informatics/MOF/PBC_functions.py

@@ -392,7 +392,7 @@ def returnXYZandGraph(filename, atoms, cell, fcoords, molgraph):
         cart_coord=np.dot(fcoord,cell)
         coord_list.append([cart_coord[0],cart_coord[1],cart_coord[2]])
     tmpstr=",".join([at for at in atoms])
-    if filename != None:
+    if filename is not None:
         np.savetxt(filename[:-4]+".net",molgraph,fmt="%i",delimiter=",",header=tmpstr)
     return coord_list, molgraph
 
@@ -936,7 +936,7 @@ def get_closed_subgraph(linkers, SBUlist, adj_matrix):
             current_linker_list = current_linker_list-SBUlist
             checked_list.add(start_idx)
             for val in loop_over:
-                if (not val in SBUlist):
+                if val not in SBUlist:
                     current_linker_list.update(np.nonzero(adj_matrix[val])[1]) # np.nonzero(adj_matrix[val])[1] are the indices of atoms with bonds to the atom with index val
             left_to_check = current_linker_list-checked_list-SBUlist # Linker atoms whose connecting atoms still need to be checked.
             if len(left_to_check) == 0:
@@ -1042,7 +1042,7 @@ def disorder_detector(name):
         disordered_atom_types = []
         disordered_atom_occupancies = []
 
-        if occupancy_index != False: # This means that occupancy information is available
+        if occupancy_index: # This means that occupancy information is available
             for idx, at in enumerate(atomlines): # Go through the lines of the cif with atom specific information. Atom by atom.
                 ln=at.strip().split()
 

molSimplify/Informatics/MOF/cluster_extraction.py

@@ -83,7 +83,7 @@ def make_MOF_SBU_RACs(SBUlist, SBU_subgraph, molcif, depth, name,cell,anchoring_
                 """""""""
                 functional_atoms = []
                 for jj in range(len(temp_mol.graph)):
-                    if not jj in link_list:
+                    if jj not in link_list:
                         if not set({temp_mol.atoms[jj].sym}) & set({"C","H"}):
                             functional_atoms.append(jj)
         # At this point, we look at the cycles for the graph, then add atoms if they are part of a cycle

molSimplify/Informatics/MOF/fragment_MOFs_for_pormake.py

@@ -682,15 +682,13 @@ def identify_short_linkers(molcif, initial_SBU_list, initial_SBU_subgraphlist, r
             else:
                 # check number of times we cross PBC :
                 # TODO: we still can fail in multidentate ligands!
-                linker_cart_coords = np.array([at.coords() \
-                        for at in [molcif.getAtom(val) for val in atoms_list]])
+                linker_cart_coords = np.array([at.coords() for at in [molcif.getAtom(val) for val in atoms_list]])
                 linker_adjmat = np.array(linker_subgraphlist[ii])
                 pr_image_organic = ligand_detect(cell_v, linker_cart_coords, linker_adjmat, linkeranchors_list)
                 sbu_temp = linkeranchors_atoms.copy()
                 sbu_temp.update({val for val in initial_SBU_list[list(sbu_connect_list)[0]]})
                 sbu_temp = list(sbu_temp)
-                sbu_cart_coords = np.array([at.coords() \
-                       for at in [molcif.getAtom(val) for val in sbu_temp]])
+                sbu_cart_coords = np.array([at.coords() for at in [molcif.getAtom(val) for val in sbu_temp]])
                 sbu_adjmat = slice_mat(adj_matrix.todense(), sbu_temp)
                 pr_image_sbu = ligand_detect(cell_v, sbu_cart_coords, sbu_adjmat,set(range(len(linkeranchors_list))))
                 if not (len(np.unique(pr_image_sbu, axis=0))==1 and len(np.unique(pr_image_organic, axis=0))==1): # linker