Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

⚡️ Rework warnings in mgm-hurry #136

Open
Rikj000 opened this issue Sep 7, 2021 · 1 comment
Open

⚡️ Rework warnings in mgm-hurry #136

Rikj000 opened this issue Sep 7, 2021 · 1 comment
Labels
Feature - Cleanup Remove unneeded files Feature - Enhancement Update or improvement to existing feature Planned Planned feature, improvement or bugfix (not being worked on yet)

Comments

@Rikj000
Copy link
Owner

Rikj000 commented Sep 7, 2021

Some of the warnings thrown in mgm-hurry are unneeded to be thrown.

  • 🔳 When installing Freqtrade initially & no installation is present yet:
    • WARNING - Could not find .hurry config file at /mnt/Windows/Users/Rik/Desktop/Rikj-Home/Projects/temp/freqtrade-mgm/.hurry
      INFO - 🍺 Configuration data written to ".hurry" file
      WARNING - FreqtradeCli - installation_exists() failed. Freqtrade binary not found in /mnt/Windows/Users/Rik/Desktop/Rikj-Home/Projects/temp/freqtrade-mgm/.env/bin/freqtrade.
      WARNING - 🤷 No Freqtrade installation found. Please run "mgm-hurry install_freqtrade" before attempting to go further!
      INFO - 💨 💨 💨
             👉 The all in one command. Hurry up, turn it up!
             💨 💨 💨
      ? 💨 Do you want to install Freqtrade? (y/N)
  • ☑️ File not always present, should only be thrown once (in-code comment points where in the code):
    • WARNING - 🤷 No "{installdir}/user_data/mgm-config-hyperopt.json" file found in the "user_data" directory. Please run: mgm-hurry setup
    • in-code: mgm-hurry - sys.exit(1) # Warning should already be thrown when loading the config files
@Rikj000 Rikj000 added Feature - Enhancement Update or improvement to existing feature Feature - Cleanup Remove unneeded files labels Sep 7, 2021
@Rikj000 Rikj000 added the In Progress This is being worked on label Sep 8, 2021
@Rikj000 Rikj000 added this to the v0.13.0 milestone Sep 8, 2021
@Rikj000 Rikj000 added Planned Planned feature, improvement or bugfix (not being worked on yet) and removed In Progress This is being worked on labels Sep 18, 2021
@Rikj000 Rikj000 removed this from the v0.13.0 milestone Sep 27, 2021
@Rikj000 Rikj000 changed the title ⚡️ Remove unneeded warnings from mgm-hurry ⚡️ Rework warnings in mgm-hurry Oct 30, 2021
@Rikj000
Copy link
Owner Author

Rikj000 commented Oct 30, 2021

🔳 Rewrite following warning:

WARNING - No history data for pair: "XXX/XXX", timeframe XX. Use `freqtrade download-data` to download the data

To:

WARNING - No history data for pair: "XXX/XXX", timeframe XX. Use `mgm-hurry download_candle_data` to download the data

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature - Cleanup Remove unneeded files Feature - Enhancement Update or improvement to existing feature Planned Planned feature, improvement or bugfix (not being worked on yet)
Projects
None yet
Development

No branches or pull requests

1 participant