UKF: Update Readme - Githubissues

udacity / sdc-issue-reports

29 stars 6 forks source link

UKF: Update Readme #891

Closed adai183 closed 7 years ago

adai183 commented 7 years ago

The run command should be updated to use the simulator.

baumanab commented 7 years ago

Thank you for pointing this out. To confirm, is this the README referred to? https://github.com/udacity/CarND-Unscented-Kalman-Filter-Project/blob/master/README.md

cicaphus commented 7 years ago

Would you mind clarifying which run command you are referring to? This is my understanding of the process:

C++ code is compiled to an executable UnscentedKF, and that executable is run from the command line with ./UnscentedKF.
Once that is running, the user launches the simulator.

Is that correct?

adai183 commented 7 years ago

Confirmed. I am referring to this README: https://github.com/udacity/CarND-Unscented-Kalman-Filter-Project/blob/master/README.md.

Under the section Basic Build Instructions the run command is:

./UnscentedKF path/to/input.txt path/to/output.txt

when it should say:

./UnscentedKF

baumanab commented 7 years ago

Thanks for clarifying. Yes, previous versions used text file I/O. The current version relies on I/O from the simulator. This is now reflected in the README.