Skip to content

seeds: update ONGEKI to June 4th, 2026#1658

Open
AlexC233 wants to merge 7 commits into
zkldi:mainfrom
AlexC233:main
Open

seeds: update ONGEKI to June 4th, 2026#1658
AlexC233 wants to merge 7 commits into
zkldi:mainfrom
AlexC233:main

Conversation

@AlexC233

Copy link
Copy Markdown

@tachi-repo-bot

tachi-repo-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

This pull request changes files under db/seeds/. View the seed diff in the Seeds web UI.

@nyairobi

Copy link
Copy Markdown
Contributor

Who are you
wikiwiki is not a reliable source

@AlexC233

Copy link
Copy Markdown
Author

Who are you wikiwiki is not a reliable source

Oh I'm sorry, I didn't know. I just assumed so since the silentblue wiki said they took it from there. What source should I be using?

@nyairobi

Copy link
Copy Markdown
Contributor

Thanks for the contribution but you could have communicated it with me ahead of time

Seeding instructions are at https://github.com/zkldi/Tachi/blob/main/typescript/seeds-scripts/rerunners/ongeki/README.md

wikiwiki is known to lie about notecounts or omit them (especially for BAS and ADV) and incorrect notecount data (maxPlatinumScore) will mess with p-stars calculation. And this is on top of internal levels also being semi-madeup. If you can vouch for maxPlatinumScore being correct, this is possibly mergable (that's ultimately up to zk); I'll take a closer look when I get home

@AlexC233

Copy link
Copy Markdown
Author

I see. Thanks for letting me know about all this. I will verify the maxPlatinumScore with the values on ongeki-net then.

@AlexC233

Copy link
Copy Markdown
Author

All the maxPlatScore values now match what is on ongeki-net.

@zkldi zkldi left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Can't vouch for the accuracy of this data (please avoid using wiki's, they are frequently wrong) but the empty str stuff here is definitely wrong and needs fix.

Comment thread db/seeds/charts-ongeki.json Outdated
},
{
"data": {
"chartViewURL": "",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

the empty strings here are wrong, it should be unset if not known

@AlexC233

Copy link
Copy Markdown
Author

Ok I’ll fix it on Tuesday when I get back to my computer.
I’ll get the internal levels for these charts from https://x.com/ongeki_level and the maxPlatScore from ongeki-net.

Comment thread db/seeds/charts-ongeki.json Outdated
{
"data": {
"chartViewURL": "https://sdvx.in/ongeki/luna/01038luna2.htm",
"chartViewURL": "https://sdvx.in/ongeki/luna/01038luna.htm",

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 does this change come from? This is the LUN 0 chart.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yeah that’s odd, it shouldn’t have changed. scrape-sdvx-in.ts swapped it and I did not notice.

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.

right, I haven't adjusted scrape-sdvx-in for the perfect shining merger; my bad

@AlexC233

Copy link
Copy Markdown
Author

I've verified the internal levels for these charts from https://x.com/ongeki_level and the maxPlatScore from ongeki-net.

@AlexC233 AlexC233 requested review from nyairobi and zkldi June 16, 2026 12:54
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.

3 participants