Skip to content

Release v0.8.8: pre-publication citation filter#340

Merged
neuromechanist merged 4 commits into
mainfrom
develop
Jun 10, 2026
Merged

Release v0.8.8: pre-publication citation filter#340
neuromechanist merged 4 commits into
mainfrom
develop

Conversation

@neuromechanist

Copy link
Copy Markdown
Member

Release summary

Promotes the citation pre-publication fix to production. Version automation strips .dev and tags on merge (do not bump manually).

What's included since v0.8.7

Deploy follow-up

After prod deploys, re-run the citation sync per enabled community to refresh the floored histograms:
docker exec osa python -m src.cli.main sync papers --community {eeglab,bids} --citations

Validation

  • Reviewed (no critical); CI green across Python 3.11/3.12/3.13.
  • Verified on dev: BIDS 2016 series starts at 2016 (2013 bucket gone); LSL floored at 2024.

Merge with a merge commit (release convention).

…p-after-v0.8.7

chore: sync develop with main after v0.8.7
* fix(citations): drop impossible pre-publication citation buckets

OpenAlex citing works sometimes carry bad publication years, producing
citation counts in years before the paper existed (e.g. the BIDS 2016 paper
showed a 2013 bucket). Floor each canonical paper's histogram at its earliest
version publication year.

- resolve_work returns the work id AND publication_year (ResolvedWork).
- sync_citing_papers takes the minimum publication year across the version
  group (the preprint, when present) and drops any count in earlier years.

Tests: pre-publication bucket dropped; floor uses the earliest version year.

* fix(citations): address review on pre-publication floor

- Use 'is not None' for publication_year filter (don't drop a year-0 edge).
- Test that an over-high floor (bogus future year) leaves existing counts
  intact via the empty-counts guard rather than wiping them.
@neuromechanist neuromechanist merged commit e670694 into main Jun 10, 2026
18 checks passed
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.

1 participant