getting development versions of ADMB from GitHub Actions? #225
-
In the discussion about a proposed revision to the admb console output (#219), @Cole-Monnahan-NOAA asked "Would you be willing to test the fork on your own models before I do a PR?". A separate email discussion today with @Rick-Methot-NOAA, @kellijohnson-NOAA, and @k-doering-NOAA about the I just confirmed that I can download the "artifact" at the bottom of the page for a github action, unzip the file, and click on the "ADMB Command Prompt" shortcut and successfully use ADMB as represented on the branch in question (in this case it was the However, the changes that @Cole-Monnahan-NOAA has contributed are in a fork and it appears that they all fail with this message: "This request was automatically failed because there were no enabled runners online to process the request for more than 1 days." (see https://github.com/Cole-Monnahan-NOAA/admb/actions/). Once a Pull Request is created, the "CodeQL" action is triggered, but the actions that compile the software are not. I know that we could all just compile ADMB locally, but downloading it already compiled would definitely make it easier to test things. First, thanks @johnoel for setting up the GitHub Actions in the first place. I think all the automatic checks associated with compiling on these platforms are really valuable and the transparency of having it all up there on github is great. My questions are,
|
Beta Was this translation helpful? Give feedback.
Replies: 4 comments 1 reply
-
I'll let @johnoel answer the big question. Note that my fork is totally broken wrt RE and autodiff models but should work for SS. We're working on resolving it. If you want to test please revert back to an earlier commit. Sorry about that. |
Beta Was this translation helpful? Give feedback.
-
For what it's worth, as a developer on Windows my workflow is to clone/pull whatever repo/branch you need. Then go to the ADMB 12.3 command prompt, migrate to the "admb" repo. Then just do |
Beta Was this translation helpful? Give feedback.
-
For 2., I think the issue on Cole's fork is that the However, I think when a pull request is made to admb, the jobs could run on the self-hosted runner. |
Beta Was this translation helpful? Give feedback.
-
For 1 and 2, GitHub Actions can be configured to run pull requests which will produce the artifacts. It's a nice feature and I have sent the artifact links to users for testing bug fixes. I will configure it later today to work with pull requests. For 3, the single admb windows installer has support for multiple compilers such as Visual Studio, rtools (versions 4 and 3) and msys2. For local machines without a compiler installed, Mingw-w64 compiler from Msys2 is included with the admb installer. |
Beta Was this translation helpful? Give feedback.
For 1 and 2, GitHub Actions can be configured to run pull requests which will produce the artifacts. It's a nice feature and I have sent the artifact links to users for testing bug fixes. I will configure it later today to work with pull requests.
For 3, the single admb windows installer has support for multiple compilers such as Visual Studio, rtools (versions 4 and 3) and msys2. For local machines without a compiler installed, Mingw-w64 compiler from Msys2 is included with the admb installer.