Skip to content

docs: fix dialect.py description — AAAK is lossy, not "30x lossless" - #894

Closed
moltate wants to merge 1 commit into
MemPalace:developfrom
moltate:fix/dialect-lossy-description
Closed

docs: fix dialect.py description — AAAK is lossy, not "30x lossless"#894
moltate wants to merge 1 commit into
MemPalace:developfrom
moltate:fix/dialect-lossy-description

Conversation

@moltate

@moltate moltate commented Apr 14, 2026

Copy link
Copy Markdown

What this fixes

The File Reference table describes dialect.py as:

AAAK compression — entity codes, emotion markers, 30x lossless ratio

This contradicts the maintainers' own April 7 correction, which states:

"30x lossless compression was overstated. AAAK is a lossy abbreviation
system (entity codes, sentence truncation)."

Change made

Updated the description to:

AAAK compression — entity codes, emotion markers, lossy abbreviation (experimental)

This aligns with the honest status already documented in the AAAK Dialect
section of the README.

No code changes

README documentation only.

release: sync develop → main (v3.3.0 manifest, SECURITY.md, version guard, Pages CNAME)
@igorls igorls added the documentation Improvements or additions to documentation label Apr 14, 2026
@milla-jovovich

Copy link
Copy Markdown
Collaborator

Thanks for flagging this — we actually caught it ourselves and fixed it in PR #835 last week. The file
reference table now says 'AAAK index format for closet pointers' on both main and develop. We also added 42 TDD tests that verify every README claim against the actual code, including this one, so it can't drift again. Closing as already resolved. Thank you from Milla and Lu✨v

@moltate

moltate commented Apr 15, 2026

Copy link
Copy Markdown
Author

Thanks for the quick update and the detailed explanation! Glad to hear it was already fixed in #835 and that you added those TDD tests — that's a solid improvement. Appreciate the review from you and Lu. I'll check out the changes.

Best,
Ankit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants