-
Notifications
You must be signed in to change notification settings - Fork 7
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
Migrate unit tests to GitHub #175
Open
WeinaJi
wants to merge
62
commits into
main
Choose a base branch
from
weji/actions_improve
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+257,238
−1,315
Open
Changes from 1 commit
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
3a01907
download neocortex mods from github repo
WeinaJi de28dde
Add integration unit tests
WeinaJi 46c30e9
install neurodamus with full dependency
WeinaJi f52b592
set environment vars for all steps
WeinaJi a553e45
Add neurodamus-models branch
jorblancoa 0df5eba
LIBSONATA_REPORT variable set in github env
jorblancoa 4ddeeca
Get latest tags instead of hardcode them
jorblancoa fc17666
fixup! Get latest tags instead of hardcode them
jorblancoa 6ce5a57
Add quotes to pip install
jorblancoa 4eae330
Build models
jorblancoa 9956dfb
fixup! Build models
jorblancoa f563736
Add NEURODAMUS_PYTHON to github env
jorblancoa 5c5a1b3
Comment integration-e2e since uses bb5 circuit
jorblancoa d6c7735
add local sub circuit fof v5_sonata for Mini5
WeinaJi d55493e
adapt unit tests using v5_sonata mini5
WeinaJi 4bfb9e0
add missing file
WeinaJi 64f6bad
Add checks for 3.9.* python version
jorblancoa 4f7fe3f
improve subcircuit: rename pop name from external to external_default
WeinaJi 25628bb
v5_gap subcircuit
WeinaJi 0518807
add missing files
WeinaJi cfd9346
Adapt v5_sonata unit tests to v5_mini circuit
jorblancoa ea40ce8
Remove 1k_v5_balance and use Mini5 instead
jorblancoa 48c941f
local subcircuit for sscx-v7-plasticity
WeinaJi 2a7ce64
Change output dir
jorblancoa e2d638b
Add RDMAV_FORK_SAFE=1 for all python versions
jorblancoa 1c79745
Replace bluepysnap with libsonata in the unit test
WeinaJi 6916878
fix flake8
WeinaJi 8ecd69f
Enable debug session
jorblancoa b71c422
Add 60 min timeout
jorblancoa f799821
Compare currents from compartments to avoid summation differences dep…
jorblancoa f0db30d
extract subcircuit for ngv
WeinaJi 11856f0
Update morphologies dir for v5_gapjunctions
jorblancoa f76c54b
fix test_connection.py for github runners
WeinaJi 27774c8
fix v5_gapjunctions
WeinaJi 6a507f9
roll back neuron version for fast testing
WeinaJi cb43ccf
Revert "roll back neuron version for fast testing"
WeinaJi c0c724e
comment out some tests
WeinaJi 26ad2d2
Uncomment the tests
jorblancoa 96cb668
build ngv model for ngv test
WeinaJi 5f505db
correct ngv build
WeinaJi c96ce57
fix ngv build
WeinaJi 90d6416
enable debug session
WeinaJi cc97046
Use only cmake build
WeinaJi df15055
fix example run
WeinaJi bbe7a51
write PATH to GITHUB_ENV and remove repeated pkg install
WeinaJi 60d8fa8
NEURODAMUS_NCX_V5=ON for neocortex model
WeinaJi e52fcb3
Fix crash test
jorblancoa 90aaec9
Activate virtual env only once
jorblancoa 0a71f18
fixup! Activate virtual env only once
jorblancoa 8e081f7
Adapt crashmode test
jorblancoa 9d14e52
Separate docs_build
jorblancoa 95f4c92
fixup! Separate docs_build
jorblancoa 058330a
fixup! Separate docs_build
jorblancoa b92349e
Fix docs warning with latest sphinx
jorblancoa 9497c44
fixup! Fix docs warning with latest sphinx
jorblancoa a9dce11
fixup! Fix docs warning with latest sphinx
jorblancoa 2ed1be2
fixup! Fix docs warning with latest sphinx
jorblancoa c4889b5
Fix an issue reading extra properties for cells where there is no syn…
jorblancoa 9998205
remove build docs workflow, enable PR builds on ReadTheDocs side
WeinaJi dea058f
don't copy circuit data iin test_cli_opts.py
WeinaJi f85b27a
Use tmp_path fixture instead of creating new tmp files
jorblancoa 533687c
Add flake8 in github actions and fix issues
jorblancoa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
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 you run flake independently on the sources only with no installation? As in separate job with separate status?
Also,
ruff
may be a better all-in-one linter.