Skip to content

Conversation

RKBoss6
Copy link
Contributor

@RKBoss6 RKBoss6 commented Jun 26, 2025

Added many more fun facts, removed redundant ones. eg. (Koalas are native to australia, Koalas are only found in australia)

@RKBoss6 RKBoss6 marked this pull request as draft June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as draft June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as ready for review June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as draft June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as draft June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as ready for review June 26, 2025 01:25
@RKBoss6 RKBoss6 marked this pull request as draft June 27, 2025 23:59
@gfwilliams
Copy link
Member

Looks great to me, thanks!

Just checking - this is tested and works?

@RKBoss6
Copy link
Contributor Author

RKBoss6 commented Sep 2, 2025

I'm actually not sure what the issue is, but each fact when pulled is a mashup of other facts (ippos. A group of ravens i)

@gfwilliams
Copy link
Member

That would have been handy to know in the PR!

You need to run this code and use the idx and count values it prints:

/*
var s = require("Storage").read("textsource.txt");
var idx = s.indexOf("\n");
var lengths = [idx];
while (idx>=0) {
var next = s.indexOf("\n",idx+1);
if (next>=0) {
var len = next-idx;
if (len>255) throw new Error("Line too long!");
lengths.push(len);
}
idx = next;
}
print(`Count = ${lengths.length}`);
print(`var idxs = E.toUint8Array(atob("${btoa(lengths)}"));`);*/

I guess you did run them before, but count is 237 and yet there are 316 lines so I guess maybe it was run on out of date data?

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