Skip to content

#518 fix: Fix "Save to json" ignored options#613

Merged
ddelpiano merged 1 commit intodevelopmentfrom
feature/518
Feb 1, 2023
Merged

#518 fix: Fix "Save to json" ignored options#613
ddelpiano merged 1 commit intodevelopmentfrom
feature/518

Conversation

@aranega
Copy link
Copy Markdown
Member

@aranega aranega commented Nov 25, 2022

Fixes #518

This PR fixes the options that are passed to

When I check/uncheck options, they options are well passed/removed from the include variable in Python that is passed to sim.saveData(include) (line 606 of this PR in netpyne_geppetto.py)
However, whether I uncheck all options or I check all of them, the produced json always includes this:

"saveDataInclude": [
      "netParams",
      "netCells",
      "netPops",
      "simConfig",
      "simData"
    ],

Is it a normal behavior of the saveData(...) method in NetPyNE?

@ddelpiano
Copy link
Copy Markdown
Member

Fixes #518

This PR fixes the options that are passed to

When I check/uncheck options, they options are well passed/removed from the include variable in Python that is passed to sim.saveData(include) (line 606 of this PR in netpyne_geppetto.py) However, whether I uncheck all options or I check all of them, the produced json always includes this:

"saveDataInclude": [
      "netParams",
      "netCells",
      "netPops",
      "simConfig",
      "simData"
    ],

Is it a normal behavior of the saveData(...) method in NetPyNE?

ccing @salvadord since he could give us more info on the above.

Thanks!

@aranega
Copy link
Copy Markdown
Member Author

aranega commented Dec 13, 2022

@ddelpiano @salvadord Here are the options that are passed to the saveData function depending on what you check in the save modal:

Save with "netParams, simConfig"
param passed to the saveData function of NetPyNE ['netParams', 'simConfig']

Save with None options:
param passed to the saveData function of NetPyNE []

Save with "netParams"
param passed to the saveData function of NetPyNE ['netParams']

Save with "netParams, simConfig, netCells" ('netPops' is automatically added because of 'netCells')
param passed to the saveData function of NetPyNE ['netParams', 'netCells', 'simConfig', 'netPops']

Save with "netParams, simConfig, netCells, simData" ('netPops' is automatically added because of 'netCells')
param passed to the saveData function of NetPyNE ['netParams', 'netCells', 'simConfig', 'simData', 'netPops']

@aranega aranega changed the title #518 fix: Fix "Save to json" isngored options #518 fix: Fix "Save to json" ignored options Dec 13, 2022
@ddelpiano ddelpiano requested review from salvadord and removed request for enicolasgomez December 14, 2022 16:54
@vvbragin
Copy link
Copy Markdown
Contributor

Fixes #518

This PR fixes the options that are passed to

When I check/uncheck options, they options are well passed/removed from the include variable in Python that is passed to sim.saveData(include) (line 606 of this PR in netpyne_geppetto.py) However, whether I uncheck all options or I check all of them, the produced json always includes this:

"saveDataInclude": [
      "netParams",
      "netCells",
      "netPops",
      "simConfig",
      "simData"
    ],

Is it a normal behavior of the saveData(...) method in NetPyNE?

Yes, it is expected. This saveDataInclude is part of simConfig, and the items in this list doesn't get affected by whta os passed as include into sim.saveData() call. Instead, it serves as default saving options in case no includeis passed to that method call.

@ddelpiano ddelpiano merged commit 5dea291 into development Feb 1, 2023
@ddelpiano ddelpiano deleted the feature/518 branch April 14, 2023 22:00
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