Skip to content

tools: move repo infrastructure to scripts/, ANN test source to tests/runANN - #259

Open
antonio-leblanc wants to merge 1 commit into
forefireAPI:devfrom
antonio-leblanc:chore/tools-reorg
Open

antonio-leblanc wants to merge 1 commit into
forefireAPI:devfrom
antonio-leblanc:chore/tools-reorg

Conversation

@antonio-leblanc

Copy link
Copy Markdown
Collaborator

Filippi, trying to help organize tools/ (you said in #239 it needs a reform). This only moves files and fixes the paths:

  • LFS checks and the manylinux NetCDF install go to scripts/. CMake, CI and cibuildwheel call them, nobody runs them by hand.
  • ANNTest.cpp goes to tests/runANN/, where it runs.

tools/ keeps what users run. htdocs stays, the HTTP server looks for FOREFIREHOME/tools/htdocs.

…o tests/runANN

tools/ is left with what users run (pre/post-processing, the MesoNH case,
htdocs, wildfire_ROS_models). The LFS checks and the manylinux NetCDF
install are called by CMake, CI and cibuildwheel, never by hand.

The Docker builder no longer copies tools/: it now passes
-DFOREFIRE_BUILD_TOOLS=OFF, so ANN_test's source is not needed there.

This branch has not been deployed

No deployments
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