Skip to content

Synapse parameterization task#490

Open
chr-pok wants to merge 25 commits intomainfrom
syn_param_task
Open

Synapse parameterization task#490
chr-pok wants to merge 25 commits intomainfrom
syn_param_task

Conversation

@chr-pok
Copy link
Collaborator

@chr-pok chr-pok commented Nov 28, 2025

Implement of an obi-one task which runs a physiological parameterization of an anatomical synaptome. It uses a synaptome (with or without synaptic physiology) from entitycore as an input and generates a (re-)parameterized synaptome as an output which will be registered as a new entity in entitycore.

Implementation details:

  • Added a random seed for reproducibility
  • Added an explicit overwrite option for re-parameterization
  • Added explicit node property by which pathways are defined
  • Added entity cache option: If the staged synaptome already exists in the cache, it will be copied to the output folder. Otherwise, it will be directly staged in the output folder and modified in-place.
  • Refactored code into commonly used helper functions, to be shared between EM synapse mapping and parameterization tasks
  • Changed compression to tarfile for consistency with what we use with all other circuits
  • Added a derivation link of type “circuit_rewiring” from the input to the parameterized output synaptome; perhaps we would need a new derivation type for that?

Related issue: #437

@chr-pok chr-pok marked this pull request as ready for review November 28, 2025 08:08
@chr-pok
Copy link
Collaborator Author

chr-pok commented Nov 28, 2025

Hi @james-isbister and @MWolfR! This is my initial implementation proposal, as a basis for further iterations/discussions.

@chr-pok
Copy link
Collaborator Author

chr-pok commented Nov 28, 2025

@MWolfR, I noticed that the input synaptomes (produced by the EM synapse mapping task) are missing a node_sets.json file but which is referenced in the circuit_config.json. It would be good to add that file (even if empty) since the missing file breaks certain bluepysnap operations.

description="Neuron property (e.g., 'synapse_class') by which to group neurons into"
" pathways between source and target neuron populations.",
)
pathway_param_dict: dict = Field(
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think there might be some different ways we can specify these different dictionaries using Blocks. But maybe easier if we discuss over a call or in person

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.

2 participants