-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add new fun facts to Fact list #3905
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Removed duplicate facts and added new interesting facts to the list.
|
Looks great to me, thanks! Just checking - this is tested and works? |
|
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) |
|
That would have been handy to know in the PR! You need to run this code and use the BangleApps/apps/textfacts/lib.js Lines 6 to 21 in 4bb0e47
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? |
|
@RKBoss6 any chance of fixing this? If you run the code again with the new data and update the library file it should be fine |
|
Oh yeah, I had put this aside for a while, I'll get this working :) |
|
I think it's fixed now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR aims to expand the textfacts module from 162 to 316 fun facts while removing redundant entries. The version is bumped from 0.02 to 0.03, and the description is simplified to remove specific references to Google Gemini and the previous fact count.
Key changes:
- Updated fact count from 162 to 316 in lib.js
- Refreshed the base64-encoded index data for fact retrieval
- Replaced and expanded the facts list with new content
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 37 comments.
| File | Description |
|---|---|
| apps/textfacts/metadata.json | Version bumped to 0.03 and description simplified |
| apps/textfacts/lib.js | Updated getCount() to return 316 and refreshed base64-encoded fact indices |
| apps/textfacts/facts.txt | Replaced 162 facts with 316 new facts, though extensive duplicates remain |
| apps/textfacts/ChangeLog | Added entry documenting version 0.03 changes |
Critical Issue: Despite the PR description stating that "redundant facts" were removed, the facts.txt file contains extensive duplication. Many facts appear 2-3 times throughout the file (e.g., lines 32-143 are largely duplicated in later sections). Additionally, line 214 contains a meta-comment that shouldn't be displayed as a fact, and several facts have missing punctuation (commas). These issues significantly undermine the stated goal of removing redundancy.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/textfacts/facts.txt
Outdated
| Your tongue is the only muscle in your body that is not attached at both ends | ||
| The total number of stars in the universe is more than all the grains of sand on Earth | ||
| A bolt of lightning is five times hotter than the surface of the sun | ||
| A group of jellyfish is called a smack |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears multiple times in the file (lines 41, 129). The PR aims to remove redundant facts, but this duplicate was not removed.
| A bolt of lightning is five times hotter than the surface of the sun | ||
| A group of jellyfish is called a smack | ||
| There's a planet made largely of diamonds, 55 Cancri e | ||
| Hawaii is moving towards Japan at the rate of a few inches per year |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears multiple times in the file (lines 43, 131). The PR aims to remove redundant facts, but this duplicate was not removed.
| The total weight of all the plastic in the ocean is estimated to be over 150 million tons | ||
| There are more chickens than people in the world | ||
| The average person walks 7,500 steps a day | ||
| The national animal of North Korea is the Chollima a mythical winged horse |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing comma before "a mythical winged horse" for proper punctuation.
| The national animal of North Korea is the Chollima a mythical winged horse | |
| The national animal of North Korea is the Chollima, a mythical winged horse |
| A group of owls is called a parliament | ||
| Cows moo with regional accents | ||
| There are more possible chess game permutations than atoms in the observable universe | ||
| A "butt" was a medieval unit of measurement for wine |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears three times in the file (lines 84, 172, 228). The PR aims to remove redundant facts, but these duplicates were not removed.
| A "butt" was a medieval unit of measurement for wine |
| Some fungi glow in the dark | ||
| The inventor of the stop sign was a woman | ||
| The Earth's atmosphere is about 100 kilometers thick | ||
| A group of armadillos is called a roll |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears three times in the file (lines 116, 204, 308). The PR aims to remove redundant facts, but these duplicates were not removed.
| The smallest bone in the human body is the stapes bone in the ear | ||
| The world's largest snowflake on record was 15 inches wide and 8 inches thick | ||
| There are more unique ways to shuffle a deck of cards than there are atoms on Earth | ||
| A group of hedgehogs is called an array |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears three times in the file (lines 106, 194, 311). The PR aims to remove redundant facts, but these duplicates were not removed.
| The Great Pyramid of Giza was originally covered in highly polished white limestone | ||
| The total mass of all the bacteria on Earth is greater than that of all animals combined | ||
| The smell of books is called bibliosmia | ||
| The highest point on Earth is Mount Everest but the highest point from the Earth's center is Mount Chimborazo |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing comma before "but" for proper punctuation.
| The highest point on Earth is Mount Everest but the highest point from the Earth's center is Mount Chimborazo | |
| The highest point on Earth is Mount Everest, but the highest point from the Earth's center is Mount Chimborazo |
| The Earth's core is as hot as the surface of the sun | ||
| A group of sloths is called a bed | ||
| The average person will produce enough saliva in their lifetime to fill two swimming pools | ||
| The smallest bone in the human body is the stapes bone in the ear |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears three times in the file (lines 103, 191, 230). The PR aims to remove redundant facts, but these duplicates were not removed.
| The only food that doesn't spoil is honey | ||
| There are more trees on Earth than stars in the Milky Way galaxy | ||
| The sound of a whip cracking is actually a tiny sonic boom | ||
| There are more than 2,000 thunderstorms happening on Earth at any given moment |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fact appears multiple times in the file (lines 50, 137). The PR aims to remove redundant facts, but this duplicate was not removed.
Co-authored-by: Copilot <[email protected]>
Removed the fact about jellyfish being called a smack.
Co-authored-by: Copilot <[email protected]>
Added many more fun facts, removed redundant ones. eg. (Koalas are native to australia, Koalas are only found in australia)