Download and locally install Swig 4.0.2 on linux if required #351
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
Member
ptheywood
commented
Jul 7, 2020
ptheywood
commented
- Also adds SWIGWORDSIZE64 definition if not using MSVC
- Enable swig compilation on github actions
- Remove externals directory from python module include path
+ Also adds SWIGWORDSIZE64 definition if not using MSVC + Enable swig compilation on github actions + Remove externals directory from python module include path
The specific configuration is having issues with THRUST which we are not planning on fixing, and appveyor is going to be removed so just simplify it for now
Robadob
pushed a commit
that referenced
this pull request
Jul 7, 2020
* Download and locally install Swig 4.0.2 on linux if required + Also adds SWIGWORDSIZE64 definition if not using MSVC + Enable swig compilation on github actions + Remove externals directory from python module include path * Minor fix w.r.t. SWIGWORDSIZE64 * Fix cmake 3.18 CUDA_ARCHITECTURES warning for swig build * Stop appveyor from building anything other than flamegpu2.lib The specific configuration is having issues with THRUST which we are not planning on fixing, and appveyor is going to be removed so just simplify it for now
Robadob
pushed a commit
that referenced
this pull request
Sep 9, 2020
basic swig build swig almost working need to sort build locations trying to get the library to import and failing added missing swig input file swig working needs some tidying up tidied up python packaging cleaned up python packaging and virtual env stop deleting my input file Adds autodownload of SWIG on Windows to CMake Appveyor changes to build SWIG. Suppress warning in compilation of SWIG gen'd file ..._wrap.cxx Switch to Python3 and venv module. Enable swig on travis (#283) This should result in a CI failure added swig ingore to fix linux added fpic option added forced venv install to travis Prototype swig messaging working (bruteforce). This will break lots of things! python test working (but not commited yet) tests passing massive message refactor swig input file minor fix. ignore current warnings added swig doxygen translation Swig cpplint 1.5.0 (#298) * Fix header order issues newly detected by cpplint 1.5.0 * Minor swig fix Co-authored-by: mondus <p.richmond@sheffield.ac.uk> starting pytest port improved first two tests completed argparse tests added first proper simulation test some tests and need help with stl very basic python callback working for step passing step test commit added host init,step and exit funcs. Agent Death test failing. added simple fail case in test.py added possible fix for building venv Fixes swig venv install on linux (#315) trying better exceptions Improved swig exception handling and assertions added new pytest class instanciate types io test passing lots of warnings tidied up some warnings added another test set for models agent function tests added model env tests added more tests fixed minor rtc bug with function name Revert "Fix header order issues newly detected by cpplint 1.5.0" This reverts commit 51be709. Other minor swig lint fixes agent instance tests test population validation added bug fix on 64 bit type for rtc on win and python tests for env added char types and agent random tests minor bug fix and LOTS of device agent creation tests Fix for all the problems created by the swig rebase (mostly bad conflict handling). Update swig to version 4.0.2, this fixes access violation at swig compile. Download and locally install Swig 4.0.2 on linux if required (#351) * Download and locally install Swig 4.0.2 on linux if required + Also adds SWIGWORDSIZE64 definition if not using MSVC + Enable swig compilation on github actions + Remove externals directory from python module include path * Minor fix w.r.t. SWIGWORDSIZE64 * Fix cmake 3.18 CUDA_ARCHITECTURES warning for swig build * Stop appveyor from building anything other than flamegpu2.lib The specific configuration is having issues with THRUST which we are not planning on fixing, and appveyor is going to be removed so just simplify it for now Minor appveyor fix Remove Appveyor and Travis as CI providers env test manager and removed temp test files test agent function conditions added exit function condition callbacks working on tests completed host agent creation tests sort failing added host agent sort test host api tests massive host env test set half the host random tests completed host rand tests host reduction tests complete append truncate test completed updated rtc example with failing test temp save of messages tests array messaging tests and fixes to message device functions for rtc location message tests
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.