Skip to content

Comments

Add cometkim codec - 5,797,365 bytes#21

Merged
agavra merged 7 commits intoagavra:mainfrom
cometkim:cometkim
Feb 4, 2026
Merged

Add cometkim codec - 5,797,365 bytes#21
agavra merged 7 commits intoagavra:mainfrom
cometkim:cometkim

Conversation

@cometkim
Copy link
Contributor

@cometkim cometkim commented Feb 4, 2026

Thanks for the fun golf contest!

Core idea: time-series encoding and out-of-order error correction. This uses events as input, not sorted_events, based on the original data observations.

Perhaps operating without sorting in advance is a bonus? :P

@agavra
Copy link
Owner

agavra commented Feb 4, 2026

Confirmed in CI! Nice submission, curious why you chose not to sort? there's no time penalty in the golf challenge πŸ˜‰

β”Œβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”¬β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”¬β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”
β”‚ Codec                  β”‚           Size β”‚ vs Naive   β”‚
β”œβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”Όβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”Όβ”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€
β”‚ Naive                  β”‚    210,727,389 β”‚   baseline β”‚
β”‚ cometkim               β”‚      5,797,365 β”‚     -97.2% β”‚
β””β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”΄β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”΄β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”€β”˜

@agavra agavra merged commit 77195fe into agavra:main Feb 4, 2026
@cometkim
Copy link
Contributor Author

cometkim commented Feb 4, 2026

I assumed the data was already mostly sorted chronologically. Therefore, I used the data's characteristics as they are. I didn't want to store timestamp values at all.

If we allow comparisons with pre-sorted data, I could remove the errors block, but it wouldn't be able to restore the original event order.

@agavra
Copy link
Owner

agavra commented Feb 4, 2026

oh i missed that you weren't storing the timestamps at all! I'll have to take a deeper look into your implementations.

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