Removing analyses, targets, experiments from AnalysisSerializer for performance

[jsmedmar]

Are there any negative side effects to removing analyses, targets, experiments from AnalysisSerializer in isabl_api/serializers.py?

We would like to do that to speed up the Isabl API. I thought this would have an impact on filtering via isabl cli, but filtering seems to be working just fine.

[jsmedmar]

where are you having performance issues? you can remove them from your query using fields!=analyses,targets,references

If you remove them altogether, then the Isabl CLI logic to determine unique analyses will stop working

[jsmedmar]

"...then the Isabl CLI logic to determine unique analyses will stop working"

Can you point me to code where removing them could influence the logic to determine unique analyses?

[jsmedmar]

https://github.com/isabl-io/cli/blob/21e77dc5380b0d773a8794702fb2401d34206f15/isabl_cli/app.py#L1344-L1419