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

Load all important application hparams from .env file on root dir #8

Closed
2 tasks
Reckless-Satoshi opened this issue Jan 2, 2022 · 0 comments
Closed
2 tasks
Labels
good first issue Good for newcomers
Milestone

Comments

@Reckless-Satoshi
Copy link
Collaborator

There are many hyperparameters that could be centralized into an .env in the root directory to ease the application managing
For example, limits for trade size could be imported from .env
https://github.com/Reckless-Satoshi/robosats/blob/9335e6abbe797e4fb281d7280da3a7114c7ef03b/api/models.py#L9-L10

  • Create an .env-sample file on root directory (e.g. MIN_TRADE_LIMIT, LND_CERT, LND_MACAROON, MAX_CONCURRENT_TRADES, INVOICE_EXPIRATION_WINDOW, ORDER_PUBLISHED_EXPIRATION_WINDOW, FIAT_RATE_API_NAME, FIAT_RATE_API_URL)
  • Substitute most important variables for those moved .env in the whole project
@Reckless-Satoshi Reckless-Satoshi added the good first issue Good for newcomers label Jan 2, 2022
@Reckless-Satoshi Reckless-Satoshi added this to the v0.1.0 MVP milestone Jan 2, 2022
@Reckless-Satoshi Reckless-Satoshi changed the title Read app hparams from .env file Load all important application hparams from .env file on root dir Jan 2, 2022
@turizspace turizspace mentioned this issue Aug 26, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant