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

docs : can't build server from documentation instruction #27

Open
frankfarsi opened this issue Sep 19, 2024 · 0 comments
Open

docs : can't build server from documentation instruction #27

frankfarsi opened this issue Sep 19, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation

Comments

@frankfarsi
Copy link

Hi - I think your server setup doc at https://github.com/grey-box/Project-Symmetry-Semantic-comparison-Alpha/blob/main/CONTRIBUTING.md is not complete: "main.py" is not found in the directory you're last in, './fastapi'. I found it in './fastapi/app', and from there I was able to start the server and also run pyinstaller.

And btw: I don't know what this means, but I did get the following when running pyinstaller:

313013 WARNING: Failed to upx strip on 'C:\\Users\\nathan\\AppData\\Local\\pyinstaller\\bincache01py31064bit\\charset.dll'!
Traceback (most recent call last):
  File "C:\Users\nathan\anaconda3\lib\site-packages\PyInstaller\building\utils.py", line 283, in process_collected_binary
    p = subprocess.run(
  File "C:\Users\nathan\anaconda3\lib\subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['upx', '--compress-icons=0', '--lzma', '-q', '--strip-loadconf', 'C:\\Users\\nathan\\AppData\\Local\\pyinstaller\\bincache01py31064bit\\charset.dll']' returned non-zero exit status 1.
313014 WARNING: Output from upx command:
                       Ultimate Packer for eXecutables
                          Copyright (C) 1996 - 2020
UPX 3.96        Markus Oberhumer, Laszlo Molnar & John Reiser   Jan 23rd 2020

        File size         Ratio      Format      Name
   --------------------   ------   -----------   -----------
upx: C:\Users\nathan\AppData\Local\pyinstaller\bincache01py31064bit\charset.dll: NotCompressibleException

Note: The exe built - so this could be a red herring?
CONTRIBUTING.md

After running pyinstaller, running app/dist/main gives:

(base) C:\projects\grey-box\symmetry\Project-Symmetry-Semantic-comparison-Alpha\fastapi\app\dist>main
←[32mINFO←[0m:     Started server process [←[36m14452←[0m]
←[32mINFO←[0m:     Waiting for application startup.
←[32mINFO←[0m:     Application startup complete.
←[31mERROR←[0m:    [Errno 10048] error while attempting to bind on address ('127.0.0.1', 8000): only one usage of each socket address (protocol/network address/port) is normally permitted
←[32mINFO←[0m:     Waiting for application shutdown.
←[32mINFO←[0m:     Application shutdown complete.

Not sure what I'm looking at...

One of the high priority tasks is to get our baseline of the product, where the UI is runnable along with the back-end being spun up. I just tried and got the UI by itself to run thanks to 'npm run package' - but the idea is for the team to be able to address areas in both the front and back end with building and running what we have. I don't know what we want to do about your swagger part - that's your call. My only nudge there is "keep the API middleware layer approach" - which I bet you already know.

@frankfarsi frankfarsi added the documentation Improvements or additions to documentation label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant