Skip to content
This repository has been archived by the owner on Mar 20, 2023. It is now read-only.

Add --verbose flag for CLI to control output messages #338

Closed
pramodk opened this issue Jul 15, 2020 · 4 comments · Fixed by #341
Closed

Add --verbose flag for CLI to control output messages #338

pramodk opened this issue Jul 15, 2020 · 4 comments · Fixed by #341
Assignees
Labels
improvement Improvement over existing implementation neuron

Comments

@pramodk
Copy link
Collaborator

pramodk commented Jul 15, 2020

Currently when we run CoreNEURON, we get output like:

 Duke, Yale, and the BlueBrain Project -- Copyright 1984-2019
 version id unimplemented

 Additional mechanisms from files
 Ca.mod CaDynamics_E2.mod Ca_LVAst.mod Ih.mod Im.mod K_Pst.mod K_Tst.mod KdShu2007.mod NaTa_t.mod NaTs2_t.mod Nap_Et2.mod ProbAMPANMDA_EMS.mod ProbGABAAB_EMS.mod SK_E2.mod SKv3_1.mod

 Memory (MBs) :             After mk_mech : Max 40.2578, Min 40.2578, Avg 40.2578
 Memory (MBs) :            After MPI_Init : Max 40.2617, Min 40.2617, Avg 40.2617
 Memory (MBs) :          Before nrn_setup : Max 40.2891, Min 40.2891, Avg 40.2891
 Setup Done   : 0.00 seconds
 Memory (MBs) :          After nrn_setup  : Max 40.6055, Min 40.6055, Avg 40.6055
GENERAL PARAMETERS
--mpi=false
--gpu=false
--dt=0.025
--tstop=30

GPU
--nwarp=0
--cell-permute=0

INPUT PARAMETERS
--voltage=-65
--seed=-1
--datpath=.
--filesdat=files.dat
--pattern=
--report-conf=
--restore=

PARALLEL COMPUTATION PARAMETERS
--threading=false
--skip_mpi_finalize=true

SPIKE EXCHANGE
--ms_phases=2
--ms_subintervals=2
--multisend=false
--spk_compress=0
--binqueue=false

CONFIGURATION
--spikebuf=100000
--prcellgid=-1
--forwardskip=0
--celsius=34
--extracon=0
--multiple=1
--mindelay=10
--report-buffer-size=4

OUTPUT PARAMETERS
--dt_io=0.1
--outpath=.
--checkpoint=

 Start time (t) = 0

 Memory (MBs) :  After mk_spikevec_buffer : Max 40.6094, Min 40.6094, Avg 40.6094

 WARNING: nrn_nrn_wrote_conc support on GPU need to validate!
 Memory (MBs) :     After nrn_finitialize : Max 40.6367, Min 40.6367, Avg 40.6367

 psolve |========================================================| t: 30.00  ETA: 0h00m00s

Solver Time : 0.168263


 Simulation Statistics
 Number of cells: 1
 Number of compartments: 273
 Number of presyns: 1
 Number of input presyns: 0
 Number of synapses: 0
 Number of point processes: 2
 Number of transfer (gap) targets: 0
 Number of spikes: 0
 Number of spikes with non negative gid-s: 0

When we run CoreNEURON via NEURON, such long output is not desirable. We should add --verbose option in CLI that will allow to enable/disable all above output messages printed in CoreNEURON.

@pramodk pramodk added neuron improvement Improvement over existing implementation labels Jul 15, 2020
alexsavulescu added a commit that referenced this issue Jul 16, 2020
@alexsavulescu alexsavulescu linked a pull request Jul 16, 2020 that will close this issue
@alexsavulescu
Copy link
Contributor

alexsavulescu commented Jul 16, 2020

@pramodk As far as I understand, we should have a --quiet option here. That can be passed on from NEURON and be defaulted to false in CORENEURON. See PR.

@nrnhines
Copy link
Collaborator

nrnhines commented Jul 16, 2020

That option is already available as neuron.coreneuron.verbose as an integer with 0 meaning off. Just need to either activate it so that it appears on the command line as --quiet or change its name to quiet.

@pramodk
Copy link
Collaborator Author

pramodk commented Jul 16, 2020

@alexsavulescu : your suggestion completely make sense given that the current default behaviour is to show messages (and we found those useful).

Me & Michael were thinking of having something like verbosity level option so that user can select the level of log messages. For example --verbose 0 or --verbose-level 0 means turn off completely. This will also fit nicely with what will be done in #300.

Sorry, I should have thought / mentioned about this earlier !

@alexsavulescu
Copy link
Contributor

Ok that changes 'a bit' the perspective :)
I will go with --verbose then

@pramodk pramodk added this to the NEURON release 8.0 milestone Jul 24, 2020
pramodk pushed a commit that referenced this issue Aug 8, 2020
* Add verbose option
* Disable brew update to speedup builds
* Use Xcode 11.3 for travis OSX CI
pramodk pushed a commit to neuronsimulator/nrn that referenced this issue Nov 2, 2022
…euron#341)

* Add verbose option
* Disable brew update to speedup builds
* Use Xcode 11.3 for travis OSX CI

CoreNEURON Repo SHA: BlueBrain/CoreNeuron@7ff03a4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvement Improvement over existing implementation neuron
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants