Skip to content

Conversation

@YBCS
Copy link
Contributor

@YBCS YBCS commented Apr 2, 2024

Currently figures are not saved if pynml-modchananalysis is used on a .mod file
Closes #201

@YBCS YBCS changed the base branch from master to development April 2, 2024 08:51
@YBCS
Copy link
Contributor Author

YBCS commented Apr 2, 2024

@sanjayankur31 PTAL
let me know if any change is required

@YBCS
Copy link
Contributor Author

YBCS commented Apr 8, 2024

@sanjayankur31 who should I ask for review ?

@sanjayankur31 sanjayankur31 self-requested a review April 8, 2024 11:41
@sanjayankur31
Copy link
Member

I've just returned from leave, so I'll take a look this week.

YBCS added 4 commits April 9, 2024 22:55
* development:
  chore(vispy): add note about env var
  fix(tests): only parse sims in parallel
  fix(tests): compile mod files before testing
  test(runners): add new test
  feat(runners): add general multi command runner wrapper
  fix: handle cases where args/kwargs are not provided
  chore: do not re-compile mods in examples folder
  chore(runners): correct rst in docstring
  chore(runners): correct documentation formatting
  chore(runners): correct documentation
  deps: make ppft a default dep
  feat(runners): add parallel runner and tests
* development:
  run generate man in right dir, check cell length
  update logger output
  adress comments
  add -swc in pynml
@sanjayankur31 sanjayankur31 self-requested a review April 16, 2024 15:58
Copy link
Member

@sanjayankur31 sanjayankur31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One or two small cosmetic fixes and should be ready to merge.

@YBCS
Copy link
Contributor Author

YBCS commented Apr 16, 2024

@sanjayankur31 I have addressed the comments
Please Take a look

@sanjayankur31
Copy link
Member

sanjayankur31 commented Apr 16, 2024

Could you quickly regenerate the man page too?

@sanjayankur31
Copy link
Member

OK, looks good now. We'll merge once the tests all pass. 👏

* development:
  feat(LEMSSimulation): allow using float/str for duration/dt
  docs(lems): document `generate_lems_file_for_neuroml`
@sanjayankur31 sanjayankur31 merged commit 4df6c54 into NeuroML:development Apr 16, 2024
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.

pynml-modchananalysis: allow saving of plots without user intervention (for batch processing)

2 participants