Unable to start LiuAlgoTrader in Firefox

[ksilo]

Describe the bug
Unable to start LiuAlgoTrader in Firefox on Linux Debian

To Reproduce
Installation tutorial was followed exactly and all prerequisites were met

Screenshots

Desktop (please complete the following information):

• Linux Debian 10
• Firefox ESR
• 78.14.0esr (64-bit)

[github-actions]

Thank you for joining LiuAlgoTrader community and submitting your first issue. Feel free to check out Liu community: https://gitter.im/LiuAlgoTrader/community

[@amor71]

@ksilo It looks like DB connection failed:

1. Is your Docker engine working, and running the PostgreSQL?
2. Are you able to connect to the DB?

[ksilo]

@amor71 everything seems up and running

g

[@amor71]

Are you able to connect to local DB, using the port in your DSN environment variable? PG has a command-line tool called ‘psql’ : can u pls confirm you can use it to access the DB? If so i will replicate your setup and test. Lmk.

…

On Sep 24, 2021, at 7:10 AM, ksilo ***@***.***> wrote:  @amor71 everything seems up and running g — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[ksilo]

Yes, I can connect using \psql -h localhost -p 5400 -U liu

[@amor71]

Pls see the error - it is trying to connect to a different port - i think issue is in your DSN (see the .sh)

…

On Sep 24, 2021, at 1:39 PM, ksilo ***@***.***> wrote:  Yes, I can connect using \psql -h localhost -p 5400 -U liu — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[@amor71]

@ksilo The error shows:

The installation wizard should have created a file env_vars.sh and you're supposed to run it before running streamlit. The file should be something like this:

export DSN=postgresql://liu:liu@localhost:5400/liu

That should direct the DB session creation to port 5400 and not 5432 -> Are you 100% sure you executed the script prior? Does it look like the above?

Can you take a screenshot of the DSN prior to running stteamlit

[ksilo]

@amor71 I think what's happening is: installation wizard can't access
https://raw.githubusercontent.com/amor71/LiuAlgoTrader/master/examples/quickstart/

probablly that's why env_vars.sh scrip isn't created

running it manually did the job: export DSN=postgresql://liu:liu@localhost:5400/liu

[@amor71]

@ksilo The liu application creates the file, I'm not sure what you mean by can't access it?

            with open(f"{samples_location}env_vars.sh", "w") as f:
                f.write(
                    f"export DSN=postgresql://{user}:{passwd}@localhost:5400/{db}\n"
                )

Was the file not created for you, in the subfolder? Can you please confirm that and I'll look further?

[ksilo]

sample files were not created in the sub folder, files = ["tradeplan.toml", "vwap_short.py", "momentum_long_simplified.py"], env_vars.sh scrip was not created also

I've checked liu.py source code and found out that link is not working in def setup_samples (400: Invalid request) for downloading sample files, could it be that due to this reason code for creating env_vars.sh was not executed? <-- I already figured out that this is wrong

but nor the samples nor the .sh script were created in the sub folder after installation wizard was completed

[@amor71]

@ksilo actually, the files *are* accessible (e.g. https://raw.githubusercontent.com/amor71/LiuAlgoTrader/master/examples/quickstart/tradeplan.toml) ). I just tested locally and it seems to work, can you please share what is in your `liu_samples/` folder?

…

On Sep 25, 2021, at 8:44 AM, ksilo ***@***.***> wrote:  sample files were not created in the sub folder, files = ["tradeplan.toml", "vwap_short.py", "momentum_long_simplified.py"], env_vars.sh scrip was not created also i check the liu.py source code and found the link is not working in def setup_samples (400: Invalid request) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[ksilo]

I don't have liu_samples folder (that's the problem), just liu_data

[@amor71]

@ksilo I just restarted from scratch, and I do see the files. Can u please try to re-run the liu quickstart and confirm you selected to create the samples. Pls see the below screenshots, LMK if you had any errors in that process, maybe something prevented the download or file creation for some reason. Thank you for checking.

After DB installation you should see this:

[ksilo]

nope, setting up samples step not start

[@amor71]

So the entire step did you take place, even when you entered Y ? So strange

…

On Sep 25, 2021, at 12:33 PM, ksilo ***@***.***> wrote:  nope, setting up samples step not start — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[ksilo]

yep, as you see, I don't get prompt for selecting location for sample files

[@amor71]

@ksilo Thanks for sharing the full details. I found the issue and pushed a fix. Version 0.3.27 has just been published.

The issue was that you entered "Y" instead of pressing the enter key...

This line failed:
samples = not len(i) or i.lower in ("y", "yes")
it should be:
samples = not len(i) or i.lower() in ("y", "yes")

Can you please pull the latest (0.3.27) and confirm the fix and close the issue?

Thanks much for noticing, well done!

[ksilo]

yes! everything is working now!