Offer part-in/synaptic queries on all neural regions; page NeuronsSyn…#74
Merged
Merged
Conversation
…aptic Term-info menu gates were tied to the old XMI 'Class + Synaptic_neuropil' takes, so ganglion-level neural regions (e.g. adult cerebrum FBbt_00007050) missed queries that do return results there: - ImagesNeurons (Images of neurons with some part in): now mirrors NeuronsPartHere (Class + not Neuron + (Synaptic_neuropil OR Anatomy)); returns 154,993 for adult cerebrum. - NeuronsSynaptic / NeuronsPresynapticHere / NeuronsPostsynapticHere: now Class + not Neuron + Nervous_system (superset of the old Synaptic_neuropil / Visual_system / Synaptic_neuropil_domain gate, adds ganglia). The OWL query (RO_0002130 some <region>) returns 8,716 neuron classes for adult cerebrum. Paging: - get_neurons_with_synapses_in_cached lacked an offset param while the query is in PAGED_QUERY_FUNCS and the underlying get_neurons_with_synapses_in takes offset -> 500 'unexpected keyword argument offset'. Add offset and forward it (mirrors the pre/post-synaptic cached wrappers). - ha_api _call now defensively drops offset/limit and retries once if a target does not declare them, so no query can 500 on paging kwargs even if a wrapper is missed. Bump 1.22.26 -> 1.22.27.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…aptic
Term-info menu gates were tied to the old XMI 'Class + Synaptic_neuropil' takes, so ganglion-level neural regions (e.g. adult cerebrum FBbt_00007050) missed queries that do return results there:
Paging:
Bump 1.22.26 -> 1.22.27.