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

Lammps output #270

Merged
merged 16 commits into from Jul 27, 2019
Merged

Lammps output #270

merged 16 commits into from Jul 27, 2019

Conversation

samwaseda
Copy link
Member

No description provided.

@jan-janssen
Copy link
Member

Be careful before merging, as this depends on pymatgen

@samwaseda samwaseda requested a review from liamhuber July 16, 2019 15:17
@jan-janssen
Copy link
Member

I implemented pymatgen in conda-forge conda-forge/staged-recipes#8888 so we could merge it now, still as pymatgen is restricted to Python > 3.5 and windows is not supported this would also restrict the availability of pyiron.

Copy link
Member

@jan-janssen jan-janssen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tests fail with:

======================================================================
FAIL: test_lammps_water (lammps.test_lammps.TestLammps)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/pyiron/pyiron/tests/lammps/test_lammps.py", line 132, in test_lammps_water
    self.assertTrue(all([node in hdf_nodes for node in nodes]))
AssertionError: False is not true
----------------------------------------------------------------------

@coveralls
Copy link

coveralls commented Jul 26, 2019

Pull Request Test Coverage Report for Build 2024

  • 28 of 31 (90.32%) changed or added relevant lines in 2 files are covered.
  • 52 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.3%) to 69.158%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pyiron/lammps/base.py 23 26 88.46%
Files with Coverage Reduction New Missed Lines %
pyiron/base/job/generic.py 1 60.94%
pyiron/lammps/control.py 1 58.52%
pyiron/base/pyio/parser.py 50 64.86%
Totals Coverage Status
Change from base Build 1999: -0.3%
Covered Lines: 12250
Relevant Lines: 17713

💛 - Coveralls

@jan-janssen
Copy link
Member

While I was able to fix the bug above I now get the following error only for Python 2:

======================================================================
ERROR: test_lammps_water (lammps.test_lammps.TestLammps)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/pyiron/pyiron/tests/lammps/test_lammps.py", line 127, in test_lammps_water
    self.job_water.run()
  File "pyiron/base/job/generic.py", line 593, in run
    self._run_if_collect()
  File "pyiron/base/job/generic.py", line 1133, in _run_if_collect
    self.collect_output()
  File "pyiron/lammps/interactive.py", line 371, in collect_output
    super(LammpsInteractive, self).collect_output()
  File "pyiron/lammps/base.py", line 284, in collect_output
    self.collect_output_log(file_name="log.lammps", cwd=self.working_directory)
  File "pyiron/lammps/base.py", line 370, in collect_output_log
    delim_whitespace=True) for llst, llen in zip(l_start, l_end)]
TypeError: initial_value must be unicode or None, not str
----------------------------------------------------------------------

@jan-janssen
Copy link
Member

The test on windows fail with the following error message:

700======================================================================
701ERROR: test_lammps_water (lammps.test_lammps.TestLammps)
702----------------------------------------------------------------------
703Traceback (most recent call last):
704  File "C:\projects\pyiron\tests\lammps\test_lammps.py", line 125, in test_lammps_water
705    self.job_water.run()
706  File "pyiron\base\job\generic.py", line 593, in run
707    self._run_if_collect()
708  File "pyiron\base\job\generic.py", line 1133, in _run_if_collect
709    self.collect_output()
710  File "pyiron\lammps\interactive.py", line 371, in collect_output
711    super(LammpsInteractive, self).collect_output()
712  File "pyiron\lammps\base.py", line 287, in collect_output
713    self.collect_output_log(file_name="log.lammps", cwd=self.working_directory)
714  File "pyiron\lammps\base.py", line 365, in collect_output_log
715    self.collect_errors(file_name)
716  File "pyiron\lammps\base.py", line 345, in collect_errors
717    error = extract_data_from_file(file_name, tag="ERROR", num_args=1000)
718  File "pyiron\base\pyio\parser.py", line 71, in extract_data_from_file
719    with open(file_name) as infile:
720IOError: [Errno 22] invalid mode ('r') or filename: u'C:/projects/pyiron/tests/lammps/lammps/lammps_water_hdf5/lammps_water/C:/projects/pyiron/tests/lammps/lammps/lammps_water_hdf5/lammps_water/log.lammps'
721
722----------------------------------------------------------------------
723Ran 347 tests in 74.929s

@sudarsan-surendralal Can you take a look at this?

@jan-janssen jan-janssen mentioned this pull request Jul 27, 2019
@jan-janssen jan-janssen merged commit 41ab6c8 into master Jul 27, 2019
@jan-janssen jan-janssen deleted the lammps_output branch July 27, 2019 11:34
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.

None yet

3 participants