Skip to content

Conversation

@ninjanomnom
Copy link

A few things I did while going through the setup process on my own computer. If you don't want one of them I can take it out easily enough:

  • I have multiple python installations so I had a bit of trouble at first when it was using the wrong version, I've changed the batch file to instead specify using python 3.6
  • When I was first setting up and saw usage information in the readme, I thought there were no setup instructions and so missed the dependencies information at the bottom, only finding it later after I had figured it out from reading the source files. I modified the readme to bring that up earlier.
  • I added a setup batch file to create configs that aren't stored in git and download python modules through pip. gitignore mentions a settings.json but I didn't see what used that so I don't know what the default contents for it should be.

Just creating the default config and installing pip modules for now
I missed it myself and went digging around to figure out dependencies before realizing too late it's at the bottom of the readme.
- [gnuplot 5+](http://www.gnuplot.info/)

A brief note on parsing, we compile each round into one data-point, attached to some other info about it.
Make sure both are in your path, then restart your computer. You should then run `Setup.bat`, this will setup the default header config in [root/config](root/config) and download any python modules needed through pip.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you need to restart for this?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally I had to restart to get the path changes to propagate, and this is recommended for path changes in general.

Copy link
Owner

@LemonInTheDark LemonInTheDark Oct 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was under the impression that you just needed to open a new terminal to refresh environmentals

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was running these not through a terminal but instead by double clicking the batch files themselves, doing it through terminal might have worked. Want me to replace the restart mention with a recommendation to use a terminal?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's odd that it what you describe wouldn't just work then. I thought the only restriction was needing to refresh a terminal's copy of environment vars after modifying them. are you sure a restart is needed in the situation you describe?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No I'm not but even the windows dialogue itself recommends restarting when you edit the path variable, which is why I repeated it here after it had fixed my own issue.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh. ok I guess this is fine then

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.

2 participants