p-zombie / freqtrade

Automatically deploy freqtrade to a remote Docker host and auto update strategies.
152 stars 56 forks source link

Is a typo in documentation? #10

Open AliM1988 opened 8 months ago

AliM1988 commented 8 months ago

In https://www.freqtrade.io/en/2019.8/data-analysis/#change-directory-to-root I think there is a typo.

import os
from pathlib import Path

# Change directory
# Modify this cell to insure that the output shows the correct path.
# Define all paths relative to the project root shown in the cell output
project_root = "somedir/freqtrade"
i=0
try:
    os.chdirdir(project_root)
    assert Path('LICENSE').is_file()
except:
    while i<4 and (not Path('LICENSE').is_file()):
        os.chdir(Path(Path.cwd(), '../'))
        i+=1
    project_root = Path.cwd()
print(Path.cwd())

There's a typo in the line (os.chdirdir should be os.chdir). It's likely meant to change the directory using os.chdir().

fniewijk commented 5 months ago

I would suggest you create a Pull Request with the changes and assign it to the author of the library.