cyllab / CalcUS

Quantum Chemistry Web Platform
https://calcus.readthedocs.io
GNU General Public License v3.0
66 stars 20 forks source link

Error on first startup of web_1 #1

Closed neflu closed 2 years ago

neflu commented 2 years ago

Describe the bug When starting CalcUS, I receive a loop of errors when starting web_1. This list of errors persists over time and is actually growing.

To Reproduce

  1. Install Prerequisites (python packages, C++ requirements)
  2. Generate environment through generate_env.bat
  3. Start via start.bat
  4. Error occurs when web_1 attempts to initalize. Error loops over and over for 5+ min.

Screenshots CalcUS_Error

Configuration

Additional context I've already tried re-running start.bat, but this does not fix the issue. All the necessary prerequisites (python and c++ redistributable) are already installed.

RaphaelRobidas commented 2 years ago

Thank you for your issue report. This is caused by the different end of line (EOL) standards between DOS-based systems (such as Windows) and Unix-based systems (such as the Linux distribution which CalcUS is running). The files in the repository should have the Unix EOL character, but modifying them on Windows can change the EOL. You can verify this for example by opening a file in Notepad and look at the bottom right corner; in one of the tiles, it should indicate either Unix (LF) or Windows (CRLF). This crash due to the EOL is of course problematic and I am working on a robust fix.

In the meantime, you can change the EOL character to the Unix one (LF) instead of the DOS ones (CRLF). Notepad does not appear to have this feature, but other text editors should (for example Notepad++). I have done some tests and I get the same error if I change the EOL of precisely scripts/run_web.sh. This is most likely the file that you will need to edit.

Let me know if the issue persists.

neflu commented 2 years ago

Thank you for the quick response! I modified run_web.sh in Notepad++ as per the following

  1. Click on Search > Replace (or Ctrl + H)
  2. Find what: \r\n
  3. Replace with: \n
  4. Search Mode: select Extended
  5. Replace All

Unfortunately, I get the following error when I attempt to access CalcUS at localhost:8080 https://pastebin.com/VWaafsTr

RaphaelRobidas commented 2 years ago

Yes, this is my bad, I forgot to include a file into the repository earlier. In any case, the version that I just pushed (commit f3f6d22) should fix both the original issue and this one. I tested it on Windows 10 as well and it appears to work.

You might notice that the footer says "Version unknown" - that's fine for now, I'm still working through this issue. It will not change anything in the functionalities.

neflu commented 2 years ago

Got it working now - thank you! Will be trying out some local calculations later.