Skip to content

Community counts attribute query - #207

Open
bgajria wants to merge 3 commits into
masterfrom
community-counts-attribute-query
Open

Community counts attribute query#207
bgajria wants to merge 3 commits into
masterfrom
community-counts-attribute-query

Conversation

@bgajria

@bgajria bgajria commented Jul 30, 2026

Copy link
Copy Markdown
Member

Moves communitycount and community_flag off OrganismAttributes.organismAttrs into a new OrganismAttributes.CommunityCounts query that
counts visible gene comments directly, rather than reading apidbtuning.organismAttributes.communitycount.

Also removes an orphaned <column name="community_flag"/> from organismAttrs whose SELECT expression was already gone, and a duplicate communitycount columnAttribute.

Not yet validated: wb model aborts earlier at presenterInjectTemplates on an unrelated ClonedInsertEnds/sourceIdField error, so wdkXml never ran. XML is well-formed per xmllint; WDK ref resolution and the SQL are unverified.

Last commit removes the dead CommunityCount tuning code from apiTuningManager.xml.

Also note that once approved this branch has to be merged to dnaseq-merge-experiments branch (not master).

bgajria and others added 2 commits July 27, 2026 11:35
…nityCount in OrgAbbrev table -

this commit is ONLY for the dnaseq-merge-experiments branch, and is being tuning for tuning.
Fix the CommunityCounts join, use @REMOTE_COMMENT_SCHEMA@, and scope
comment counts per project.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@bgajria
bgajria requested a review from jbrestel July 30, 2026 01:30
Superseded by the CommunityCounts attribute query.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@jbrestel jbrestel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good

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