block_structure: pick_gf_struct_sumk with transformation

plus little bug fixes

python/block_structure.py

@@ -217,10 +217,10 @@ class BlockStructure(object):
 
     @property
     def sumk_to_solver_block(self):
-        if self.corr_to_inequiv is None:
+        if self.inequiv_to_corr is None:
             return None
         ret = []
-        for icrsh, ish in enumerate(self.corr_to_inequiv):
+        for ish, icrsh in enumerate(self.inequiv_to_corr):
             d = defaultdict(list)
             for block_solver, block_sumk in self.solver_to_sumk_block[ish].iteritems():
                 d[block_sumk].append(block_solver)
@@ -252,6 +252,7 @@ class BlockStructure(object):
             "give one transformation per correlated shell"
 
         for icrsh in range(len(trans)):
+            ish = self.corr_to_inequiv[icrsh]
             if trans[icrsh] is None:
                 trans[icrsh] = {block: np.eye(len(indices))
                                 for block, indices in self.gf_struct_sumk[icrsh]}
@@ -273,7 +274,7 @@ class BlockStructure(object):
                 # zero out all the lines of the transformation that are
                 # not included in gf_struct_solver
                 for iorb, norb in enumerate(self.gf_struct_sumk_dict[icrsh][block]):
-                    if self.sumk_to_solver[icrsh][(block, norb)][0] is None:
+                    if self.sumk_to_solver[ish][(block, norb)][0] is None:
                         trans[icrsh][block][iorb, :] = 0.0
         return trans
 
@@ -475,24 +476,46 @@ class BlockStructure(object):
             However, the indices are not according to the solver Gf
             but the sumk Gf.
         """
+        eff_trans_sumk = self.effective_transformation_sumk
 
-        # TODO: when there is a transformation matrix, this should
-        #       first zero out the corresponding rows of (a copy of) T and then
-        #       pick_gf_struct_solver all lines of (the copy of) T that have at least
+        assert len(eff_trans_sumk) == len(new_gf_struct),\
+            "new_gf_struct has the wrong length"
+
+        new_gf_struct_transformed = copy.deepcopy(new_gf_struct)
+
+        # when there is a transformation matrix, this first zeroes out
+        # the corresponding rows of (a copy of) T and then applies
+        # pick_gf_struct_solver for all lines of  T that have at least
         # one non-zero entry
 
+        for icrsh in range(len(new_gf_struct)):
+            for block, indices in self.gf_struct_sumk[icrsh]:
+                if not block in new_gf_struct[icrsh]:
+                    del new_gf_struct_transformed[block]
+                    continue
+                T = eff_trans_sumk[icrsh][block]
+                for index in indices:
+                    if not index in new_gf_struct[icrsh][block]:
+                        T[:, index] = 0.0
+                new_indices = []
+                for index in indices:
+                    if np.any(np.abs(T[index, :]) > 1.e-15):
+                        new_indices.append(index)
+                new_gf_struct_transformed[icrsh][block] = new_indices
+
         gfs = []
         # construct gfs, which is the equivalent of new_gf_struct
         # but according to the solver Gf, by using the sumk_to_solver
         # mapping
-        for ish in range(len(new_gf_struct)):
+        for icrsh in range(len(new_gf_struct_transformed)):
+            ish = self.corr_to_inequiv[icrsh]
             gfs.append({})
-            for block in new_gf_struct[ish].keys():
-                for ind in new_gf_struct[ish][block]:
+            for block in new_gf_struct_transformed[icrsh].keys():
+                for ind in new_gf_struct_transformed[icrsh][block]:
                     ind_sol = self.sumk_to_solver[ish][(block, ind)]
-                    if not ind_sol[0] in gfs[ish]:
-                        gfs[ish][ind_sol[0]]=[]
-                    gfs[ish][ind_sol[0]].append(ind_sol[1])
+                    if not ind_sol[0] in gfs[icrsh]:
+                        gfs[icrsh][ind_sol[0]] = []
+                    gfs[icrsh][ind_sol[0]].append(ind_sol[1])
         self.pick_gf_struct_solver(gfs)
 
     def map_gf_struct_solver(self, mapping):
@@ -730,6 +753,8 @@ class BlockStructure(object):
 
         # create a Green's function to hold the result
         if G_out is None:
+            if not 'mesh' in kwargs and not 'beta' in kwargs:
+                kwargs['mesh'] = G.mesh
             G_out = self.create_gf(ish=ish_to, space=space_to, **kwargs)
         else:
             self.check_gf(G_out, ish=ish_to, space=space_to)

python/sumk_dft.py

@@ -1080,7 +1080,7 @@ class SumkDFT(object):
         full_structure = BlockStructure.full_structure(
             [{sp:range(self.corr_shells[self.inequiv_to_corr[ish]]['dim'])
                 for sp in self.spin_block_names[self.corr_shells[self.inequiv_to_corr[ish]]['SO']]}
-                for ish in range(self.n_inequiv_shells)],None)
+                for ish in range(self.n_inequiv_shells)],self.corr_to_inequiv)
         G_transformed = [
             self.block_structure.convert_gf(G[ish],
                 full_structure, ish, mesh=G[ish].mesh.copy(), show_warnings=threshold,

test/blockstructure.py

@@ -106,6 +106,22 @@ cmp(pick3.effective_transformation_solver,
       'down_0': np.array([[0, 1, 2],
                           [3, 4, 5]])}])
 
+pick4 = original_bs.copy()
+pick4.transformation = [np.array([[0, 1, 0], [1, 0, 0], [0, 0, 1]])]
+pick4.pick_gf_struct_sumk([{'up': [1, 2], 'down': [0, 1]}])
+cmp(pick2.gf_struct_sumk, pick4.gf_struct_sumk)
+cmp(pick2.gf_struct_solver, pick4.gf_struct_solver)
+assert pick4.sumk_to_solver == [{('up', 0): ('up_0', 0),
+                                 ('up', 1): (None, None),
+                                 ('up', 2): ('up_1', 0),
+                                 ('down', 2): (None, None),
+                                 ('down', 1): ('down_0', 1),
+                                 ('down', 0): ('down_0', 0)}]
+assert pick4.solver_to_sumk == [{('up_1', 0): ('up', 2),
+                                 ('up_0', 0): ('up', 0),
+                                 ('down_0', 0): ('down', 0),
+                                 ('down_0', 1): ('down', 1)}]
+
 # check map_gf_struct_solver
 mapping = [{('down_0', 0): ('down', 0),
             ('down_0', 1): ('down', 2),