-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v0.6-rc #1814
v0.6-rc #1814
Conversation
…example/single_cell_nml.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of minor comments, but otherwise looks good.
@@ -92,11 +92,15 @@ jobs: | |||
# See https://github.com/open-mpi/ompi/issues/6518 | |||
OMPI_MCA_btl: "self,tcp" | |||
steps: | |||
- name: Get clang/gcc 8 | |||
- name: Get clang/gcc 8, libxml2, Linux |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we rename to "Linux: get clang/gcc 8, libxml2" (and "MacOS: get libxml2" below)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure!
with: | ||
submodules: recursive | ||
- name: Build and install Arbor using pip | ||
run: python3 -m pip install --verbose --install-option="-DARB_VECTORIZE=ON" --install-option="-DARB_ARCH=native" . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we verify that vectorization is actually enabled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do now :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool :)
* bump version to v0.6-rc * Update `MANIFEST.in` and `scripts/run_python_examples.sh` * Update Release instructions `docs/contrib/release.rst` * Fix missing quotes in `python/example/single_cell_nml.py` * Add testing to sdist creation in `ciwheel.yml` * Rename `basic.yml` to `test-everything.yml` * Remove Python examples step from matrix (mostly because plotting takes much CPU on macos) * change to build /w nml * add test of pip builds (`pip install` in addition to CMake builds) * Python examples are tested here, once is enough.
MANIFEST.in
andscripts/run_python_examples.sh
docs/contrib/release.rst
python/example/single_cell_nml.py
ciwheel.yml
basic.yml
totest-everything.yml
pip install
in addition to CMake builds)