Comments (8)
You are perfectly right, we need more tests.
However a tests should ideally test one thing and one thing only
in the spirit of unit tests. Later we may introduce combination tests.
The reason why there are only few tests at this moment is
that I was not sure (and still am not sure) about all aspects
of the API and first wanted to stabilize the API before introducing
many tests which use the API.
from autocmake.
IMO, as long as is autocmake employed in "bigger", more complex projects, we should consequently include more testing.
For example, when autocmake-for-DIRAC is using math libs detection, we are supposed to have one (simple) test for math detection.
from autocmake.
Absolutely.
from autocmake.
Hi,
as the next simple test, I am suggesting a fc_openblas test , where simple a Fortran code would call some BLAS routine located in the famous OpenBLAS library.
The reason for OpenBLAS is that it's parallel and is availalble also for MS Windows (we have it in DIRAC, see https://testboard.org/cdash/buildSummary.php?buildid=16874). Unfortunately, there is no GNU /usr/lib/libblas equivalent on MS Windows.
from autocmake.
A good idea but the math module will go through at least one more iteration so expect API changes.
from autocmake.
I am closing this issue because it is a bit too diffuse. But it is of course a good point
and will be pursued.
from autocmake.
Hi,
could I please get green light for implementing the #16 (comment) ?
from autocmake.
More tests are always welcome!
from autocmake.
Related Issues (20)
- Make it simpler to place modules to the "standard library".
- [Discussion] Versioning strategy HOT 1
- [Discussion] move this repo to another org HOT 4
- Discover math libraries: show MKL and ScaLAPACK
- omp module logic troubled by newer cmake HOT 26
- Offer option to specify project version in YAML file
- DeprecationWarning: URLopener style of invoking requests is deprecated HOT 1
- Drop Python 2 support in the update machinery HOT 1
- CMAKE_BINARY_DIR vs PROJECT_BINARY_DIR HOT 2
- Restore CI
- GNU blas/lapack integer64 detection HOT 2
- Remove boost module and tests
- Improve the way we figure out whether `cmake` command is available
- Make setup to print out wrong keyword(s)
- How to launch autocmake tests ? HOT 1
- tests failed... HOT 1
- missing "pure" python HOT 2
- python script error after running update.py HOT 1
- setup not showing the default python interpeter HOT 1
- [question] LAPACK and BLAS linking using `modules/math_libs.cmake` HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from autocmake.