Merge pull request #946 from pyiron/revert-945-remove_job_reconsituti…

…on_on_pyiron_table_structure_get

Revert "Don't reconstitute job object in pyiron table func"

pyiron_atomistics/table/funct.py

@@ -238,8 +238,7 @@ def get_equilibrium_parameters(job):
 
 
 def get_structure(job):
-    atoms = Atoms().from_hdf(job["output"])
-    # I don't understand why any of this is needed?
+    atoms = pyiron_to_ase(job.to_object().get_structure())
     atoms_dict = {
         "symbols": atoms.get_chemical_symbols(),
         "positions": atoms.get_positions().tolist(),
@@ -253,10 +252,8 @@ def get_structure(job):
         atoms_dict["masses"] = atoms.get_masses().tolist()
     if atoms.has("momenta"):
         atoms_dict["momenta"] = atoms.get_momenta().tolist()
-    # is this wrong? why are we setting magmoms to be initial magmoms?
     if atoms.has("initial_magmoms"):
         atoms_dict["magmoms"] = atoms.get_initial_magnetic_moments().tolist()
-    # is this wrong? why are we setting charge to be initial charge?
     if atoms.has("initial_charges"):
         atoms_dict["charges"] = atoms.get_initial_charges().tolist()
     if not atoms.__dict__["_calc"] == None: