Open DaanWielens opened 5 years ago
@DaanWielens Hi Daan: Thank you for reporting this issue. Your CIF file crashed on my computer as well, raising the same error. Actually there's no problem with your CIF file. It turns out that I've failed to consider the situation where the atom species could be charged ions since I am mostly dealing with covalent compounds. I've fixed the bug regarding this issue. Please let me know if you experience any other issue.
Today I have had the time to check your new code. However, I did not bring my laptop so I have created a virtual machine with Win10, downloaded Python 3.7.3 and installed the packages as listed in your Readme requirements section.
Today, I received an error
C:\Users\Daan Wielens\git\PyRHEED\Source>python Main.py
Traceback (most recent call last):
File "Main.py", line 5, in <module>
import broadening
ModuleNotFoundError: No module named 'broadening'
It looks like the re-styled code in commit aa9faa27925168673d4516f9930753799843d454 causes the problem, as I am able to import Broadening
but not broadening
. Same holds for Process
which is now called from Main.py
as process
. Probably all files need to be renamed to match the 'import' lines of code. I am not sure if all other re-styling options lead to any problems.
@DaanWielens Yes, that's because I've renamed all my filenames. I did not set my git to be case sensitive therefore it did not pick up some of the filename changes. I've fixed it in my last commit. Thanks.
I would like to try to use the software to simulate RHEED patterns of InP(111)A. To start, I have simply downloaded a InP .cif file (http://crystallography.net/cod/1010146.cif). From the main window, I selected 'Simulation', then 'Diffraction Pattern'. When loading the .cif file, the log shows:
Initialized! CIF opened! Adding sample 1 Sample 1 loaded! Constructing sample 1 Finished construction of sample 1
Then, Windows notifies me that Python has stopped working.
The console shows:
It looks like the program has trouble in reading the .cif file, but since it is downloaded directly from the crystallographic database and is working fine in VESTA, I assume the file itself is fine...