BirdNET-Analyzer
BirdNET-Analyzer copied to clipboard
fix: README.md version when using a Dockerfile with combination of ENTRYPOINT and CMD section
Since using the combination of ENTRYPOINT and CMD section in Dockerfile we have to explicitly call the analyze.py script on the command line, because the first argument on the command line overwrites the default command analyze.py and will be taken as command instead as an option to analyze.py like i.e. --i input and therefore python throws an error instead of calling the analyze.py script with option --i input.
BTW: great work all this!
AFAIK the combination of ENTRYPOINT and CMD section in Dockerfile only makes sense if one needs the feature to overwrite the default command, specified by CMD, with a different one i.e. like analyze_test.py specified on the command line.
My impression is, that this is never the case here, right?
If I'm right, it may be better to to go back in Dockerfile to ENTRYPOINT [ "python3" , "./analyze.py" ] and leave README.md as it is?!
I've added PR #52 for convenience
The idea was that you could also start the server.py or gui.py using the same Dockerfile, so this was the best solution right now. We still should add some documentation for running server.py and gui.py with Docker.