#263 Update todo

Author: Lucas Rebscher

netpyne_ui/netpyne_geppetto.py

@@ -103,9 +103,6 @@ def simulateNetPyNEModelInGeppetto(self, args):
                 else:
                     logging.info("Starting simulation")
 
-                    # TODO: (#263) we should instantiate the network here per default
-                    #   or we need a mechanism to detect if the currently instantiated network matches with
-                    #   the current net params and simConfig, otherwise this can cause errors
                     if not args.get('usePrevInst', False):
                         logging.debug('Instantiating single thread simulation')
                         netpyne_model = self.instantiateNetPyNEModel()

webapp/components/topbar/menuConfiguration.js

@@ -325,6 +325,8 @@ export const getModelMenu = (props) => (
       label: TOPBAR_CONSTANTS.SIMULATE,
       action: {
         handlerAction: 'redux',
+        // TODO: (#263) this logic causes issues by potentially simulating
+        //  old instance with modified netParams and simConfig
         parameters: [props.modelState === MODEL_STATE.NOT_INSTANTIATED ? createAndSimulateNetwork : simulateNetwork],
       },
     },