VEuPathDB / VEuPathDB/lib-eda-subsetting

Implement consolidated counts API

Open
#26 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Java
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Currently, in order for the client to fetch all of the counts required for the entity diagram, two calls to /count need to be created per entity. This can lead to slowness in populating the counts in the entity diagram.

In theory, we could consolidate these calls, and even produce counts for multiple entities in a single traversal of the tree if the filters span multiple entities.

This is particularly noticeable on a study with a lot of entities, such as the Vectorbase megastudy.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the existing /count calls used to populate the entity diagram and identify how filters and entity traversals are handled. Done means a consolidated API can provide the required counts for multiple entities, including filters spanning entities, while reducing the repeated traversals described in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend, performance
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.