-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
search nuclei via id and seg id + caching #374
Conversation
df.to_pickle(settings.STATIC_ROOT+'/'+supported_tables.get(table_type)+'.pkl') | ||
return df | ||
else: | ||
return float('NaN') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think returning False
or raising an exception would be better here. When this function is called when can then make sure its returned something by just pass an if statement to it (which would work with this code right now too but a direct False
would be more readable IMO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated it to log the error of querying a resource table that does not exist -- happy to adjust it, though, if there are cases you can think of where this would be an undesirable response to a table not being queryable !
Issue #367 Users can now enter any combination of nuclei and/or seg-ids to search nuclei and neuron types.
New column added to nuclei viewer table to cross-search between seg-id and nuclei-id. Input ids that are not found in the neuron table will be listed below for awareness.
Nuclei and neuron type tables are now also cached in static files for the nuclei states.