Update default opt flags and retrain tests
Nobody has claimed this yet.
- Dominant language
- Fortran
- Stars
- 332
- Forks
- 124
- PR merge metrics
- No merged PRs in 30d
Description
Description
We should remove the -ffast-math flag from the default gfortran config and retrain all the tests without this flag. Users can still choose to add this flag and get the same behavior as before. We may also want to retrain the pyAerostructure tests.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the default gfortran configuration and the existing test suite or baselines, then determine which tests need retraining and whether the pyAerostructure tests are affected. Done means the default configuration no longer uses -ffast-math, users can still opt into it, and the relevant retrained tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- build-system, testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100