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

Current working package versions and changes to README.md #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

RaoefTaki
Copy link

  • Currently-working package versions are provided in requirements.txt, obtained via the pip freeze > requirements.txt command in a conda environment which is able to run MNTDP.
    I am not sure whether any redundant packages are included in the changed requirements.txt file or not.
    Also, it would probably be good to keep the current packages in the requirements.txt file somewhere in the repository.
  • The commands conda install graphviz and conda install pydot were required to be run by me, in order to run configs/streams/s_in.yaml. This information has been added to README.md.
  • Added the information that configs/streams/s_test.yaml, instead of the default configs/streams/s_long.yaml, is a debug stream, useful for testing whether MNTDP works correctly.

The code should work fine with the latest package versions, at this point in time, however to be sure with these versions the code worked correctly for me.
Indicate that configs/streams/s_test.yaml is a debug stream, in README.md, which can easily be used to test the workings of your setup or the code.
Graphviz and pydot were required to be installed for me when running the `configs/streams/s_in.yaml` stream. Installing it via `conda install [package]` worked for me.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant