Skip to content

Conversation

@pgleeson
Copy link
Member

@sanjayankur31 This contains a change in the ci:

   pytest --cov=pyneuroml -m "not localonly" .

Is this required any more?

* feat/add-neuron-cell-info-utils: (53 commits)
feat(neuron-util): allow skipping mechs that aren't present on any
sections
  feat(neuroml-utils): replace `_NML2` suffix in names
  ci: skip tests that segfault on GH
  chore(test-plots): make test non-interactive
  fix(test-neuron-utils): use setup to compile required mod files once
  test(neuroml-utils): add basic tests for `getinfo`
  test(neuron-utils): remove unneeded comments
  test(neuron-utils): check in olm cell as test data
  test(neuron-utils): complete `morphinfo` test
  chore(neuron-utils): correct `morphinfo` return value doc
  chore: remove hayet al tests data files
  chore: format with black
  fix: correctly set logging level
  fix(neuron-utils): correctly output param values for each segment
  feat(neuroml-utils): replace sq brackets
  fix(neuroml-utils): correctly divide by number of segments
  chore(neuron-utils): improve logging
  chore: set logger level to info
  feat(neuron-utils): provide information segment wise
  refactor(neuron-utils): `seclistinfo` -> `getinfo`
  ...
@sanjayankur31
Copy link
Member

Yeh, the few tests that are marked "localonly" segfault on GH actions. We can test them again to see if they pass I guess.

@pgleeson pgleeson merged commit 5c57be3 into development Oct 17, 2022
@pgleeson
Copy link
Member Author

Ok, merging for now...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants