update readmes #845
Merged
update readmes #845
Conversation
This file contains 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
paskino
requested changes
Oct 31, 2023
|
@paskino @casperdcl what's the status of this one? |
|
going to split this out into separate PRs |
|
@KrisThielemans ready for review :) |
KrisThielemans
requested changes
Mar 8, 2024
README.md
Outdated
| cmake --build . --config Release | ||
|
|
||
| ```sh | ||
| cmake --build ./build --config Release | ||
| ``` | ||
|
|
||
| Note that there is no separate install step. | ||
|
|
||
| ### MATLAB and Python conflicts |
There was a problem hiding this comment.
delete this whole section. I've moved it to
https://github.com/SyneRBI/SIRF/wiki/SIRF-and-MATLAB
README.md
Outdated
|
|
||
| This `USE_SYSTEM_*` function can be used for as many or as few packages as desired. Advantages to building against your own version of certain packages are decreased compilation times and the potential to use newer versions of these packages. | ||
|
|
||
| However, we have only tested SIRF with the versions of the required dependencies that are built by default in the Superbuild. If you decide to compile SIRF using system versions of the dependencies, you run a greater risk of something going wrong. | ||
| However, we have only tested SIRF with the versions of the required dependencies that are built by default in the SuperBuild. If you decide to compile SIRF using system versions of the dependencies, you run a greater risk of something going wrong. | ||
|
|
||
| For this reason, we advise new SIRF users to compile with all the `USE_SYSTEM_*` options disabled. If you decide to use system versions of certain packages, we would be interested to hear about it any compatibility issues that you may run into. | ||
|
|
||
| ### Python and MATLAB installation locations |
mostly related to MATLAB
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.
re-enable CCPi-Regularisationmoved to CCPi regularisation fails test #832