search

uDALES / u-dales

uDALES: large-eddy-simulation software for urban flow, dispersion and microclimate modelling
https://udales.github.io/u-dales
GNU General Public License v3.0
47 stars 17 forks source link

Re-define prerequisites #117

Closed bss116 closed 3 years ago

bss116 commented 3 years ago

I noticed MATLAB is missing in the Prerequisites (https://github.com/uDALES/u-dales/blob/master/docs/udales-getting-started.md#prerequisites), and I was wondering if we should be more specific about them.

  1. add MATLAB to list, since it is an essential part of the pre-processing.
  2. a github account is not strictly needed, you can just press return while using the cookiecutter, then it just stays a local repository

Should we make a distinction for what these things are needed? NCO -> post processing only MATLAB -> pre processing only Python, Git -> repository setup and testing

EDIT: Optional deps like Graphviz

bss116 commented 3 years ago

@samoliverowens @dmey How about these new prerequisites: c498f5b? @dmey what about the prerequisites for developers? everything for uDALES + python + graphviz (optional)?

samoliverowens commented 3 years ago

@samoliverowens @dmey How about these new prerequisites: c498f5b? @dmey what about the prerequisites for developers? everything for uDALES + python + graphviz (optional)?

That looks good to me!

dmey commented 3 years ago

@bss116 this looks great 👍. Re the ones for developers, how about we just refer to DEVELOP.md. That already contains those (although I say conda for consistency with the deps)

bss116 commented 3 years ago

@bss116 this looks great 👍. Re the ones for developers, how about we just refer to DEVELOP.md. That already contains those (although I say conda for consistency with the deps)

Okay, I just specified that only the uDALES section is required (e.g. no MATLAB), which is in line with what the code snippet does.

bss116 commented 3 years ago

closed with #122