Skip to content

feat: Add volume range metadata for reporters - #246

Open
Luis-manzur wants to merge 2 commits into
mainfrom
245-add-volume-range-metadata-for-reporters
Open

feat: Add volume range metadata for reporters#246
Luis-manzur wants to merge 2 commits into
mainfrom
245-add-volume-range-metadata-for-reporters

Conversation

@Luis-manzur

Copy link
Copy Markdown
Contributor

Fixes #245

@Luis-manzur
Luis-manzur requested review from flooie and grossir January 26, 2026 16:37
@Luis-manzur Luis-manzur linked an issue Jan 26, 2026 that may be closed by this pull request
@Luis-manzur Luis-manzur moved this to PRs to Review in Sprint (Case Law) Jan 26, 2026
freelawbot added a commit that referenced this pull request Jan 26, 2026
freelawbot added a commit that referenced this pull request Jan 26, 2026
@Luis-manzur
Luis-manzur force-pushed the 245-add-volume-range-metadata-for-reporters branch from ad7fbb4 to 7b06ca1 Compare January 26, 2026 16:50
Comment thread reporters_db/data/reporters.json Outdated
Comment on lines +25992 to +25994
"last_updated": "2025-01-01",
"max": 606,
"min": 1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

where'd you find this data

@Luis-manzur Luis-manzur Jan 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I asked gemini about volume 606, and told me that Part 1 started around January 25

freelawbot added a commit that referenced this pull request Jan 26, 2026
@freelawbot

Copy link
Copy Markdown

The Eyecite Report 👁️

Gains and Losses

There were 0 gains and 0 losses.

Click here to see details.
id Gain Loss

Time Chart

image

Generated Files

Branch 1 Output
Branch 2 Output
Full Output CSV

@flooie

flooie commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Does this handle non integer volumes? @Luis-manzur

@Luis-manzur

Copy link
Copy Markdown
Contributor Author

Does this handle non integer volumes? @Luis-manzur

no, only works for integer volumes

@Luis-manzur
Luis-manzur requested a review from flooie February 9, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: PRs to Review

Development

Successfully merging this pull request may close these issues.

Add volume range metadata for reporters

3 participants