Fixes to bring this back into working order #52
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Two real changes here: - Change package, as the old breseq package now has build bugs. - Change to use /tmp/ for the install target, so it'll work on pretty much any machine.
Tweaked the logged paths to reflect what we use on Aire, and adjusted ARC4 notes to talk of Aire
- Remove references to ARC4 and local customisations. - Switch to a newer version of Ubuntu to avoid fakeroot issues
- Removed references to ARC - Adjusted the builds to different versions and fixed dependency issues
murphyqm
reviewed
May 23, 2025
| - Conda also provides access to widely used machine learning and data science libraries such as TensorFlow, SciPy, NumPy that are available as pre-configured, hardware specific packages (such as GPU-enabled TensorFlow) allowing for code to be as performant as possible | ||
|
|
||
| (installing-conda)= | ||
| ## Installing Conda |
Contributor
There was a problem hiding this comment.
Will keep rewriting conda section for another pr, but adding it to the to-do list
murphyqm
approved these changes
May 23, 2025
Contributor
murphyqm
left a comment
murphyqm
left a comment
There was a problem hiding this comment.
Have read through all prose, and it looks good.
- Have also tested the steps for some sections, (specifically spack use) and it aligns with the output I got. Build works locally.
Happy to approve, but equally, if someone more HPC-focused than me wants to double check other sections - I might have missed something obvious!
Also agree re conda rewrite needed - but also agree that hould be a separate PR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There's a fair few tweaks in here to make the course function.
Could also do with the conda section being redone to remove references to Anaconda and trim it back to something usable in teaching and research.