Skip to content

Bitbyte model. Bitbyte tree fix.#602

Merged
508312 merged 10 commits intocompserv:masterfrom
508312:master
Mar 6, 2026
Merged

Bitbyte model. Bitbyte tree fix.#602
508312 merged 10 commits intocompserv:masterfrom
508312:master

Conversation

@508312
Copy link
Contributor

@508312 508312 commented Jan 12, 2026

Welp who would have known that following bad practices is gonna lead to stuff breaking. Removed hardcoded csv. Added a model for bitbyte groups. Should be relatively general so if ppl want to code candidate portal bitbyte group stuff (idk points? leaderboards? might be fun) it will be compatible.

@508312
Copy link
Contributor Author

508312 commented Jan 12, 2026

Ask me for a CSV on slack(@ artem) to test correctness

@508312 508312 requested a review from colangdon January 15, 2026 18:40
@hkatzdev
Copy link

hkatzdev commented Mar 5, 2026

Changes on slack were done, mostly LGTM now - just need to capitalize the T in Bitbyte tree to match the rest of the website and I think its good to push. Tested with weird edges and non existent users.

Copy link

@colangdon colangdon left a comment

Choose a reason for hiding this comment

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

lgtm!

@508312
Copy link
Contributor Author

508312 commented Mar 6, 2026

uhh sorry for merging and not squashing, i need that github activity in my profile frfr

@508312 508312 merged commit 5932fe1 into compserv:master Mar 6, 2026
2 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.

3 participants