Skip to content

West Midlands | 26 March SDC | Iswat Bello | Sprint 1 | Number systems#370

Open
Iswanna wants to merge 3 commits intoCodeYourFuture:mainfrom
Iswanna:tools/sprint-1-number-systems
Open

West Midlands | 26 March SDC | Iswat Bello | Sprint 1 | Number systems#370
Iswanna wants to merge 3 commits intoCodeYourFuture:mainfrom
Iswanna:tools/sprint-1-number-systems

Conversation

@Iswanna
Copy link

@Iswanna Iswanna commented Mar 13, 2026

  • Filled in answers for binary, decimal, and hex conversions
  • Provided answers for bit storage, binary addition, and comparison questions
  • Added responses for interpreting bytes as numbers, ASCII, greyscale, and RGB colors

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

This pull request completes the answers for all exercises in the README.md file related to binary, hexadecimal, and colour representation. All previously blank "Answer:" fields have been filled in with the correct values.

Questions

Hi, could you please review my PR? I would really appreciate your feedback.

- Filled in answers for binary, decimal, and hex conversions
- Provided answers for bit storage, binary addition, and comparison questions
- Added responses for interpreting bytes as numbers, ASCII, greyscale, and RGB colors
@Iswanna Iswanna added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Tools The name of the module. labels Mar 13, 2026
@Iswanna Iswanna changed the title West Midlands | 2 Feb Piscine | Iswat Bello | Sprint 1 | Number systems West Midlands | 2 Feb SDC | Iswat Bello | Sprint 1 | Number systems Mar 14, 2026
@Iswanna Iswanna changed the title West Midlands | 2 Feb SDC | Iswat Bello | Sprint 1 | Number systems West Midlands | 2026 March SDC | Iswat Bello | Sprint 1 | Number systems Mar 14, 2026
@Iswanna Iswanna changed the title West Midlands | 2026 March SDC | Iswat Bello | Sprint 1 | Number systems West Midlands | 26 March SDC | Iswat Bello | Sprint 1 | Number systems Mar 14, 2026
- Updated README.md to provide a clearer explanation for testing powers of two in binary.
- Replaced the previous answer with a more descriptive method.
@nedssoft nedssoft added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Mar 21, 2026
Copy link

@cjyuan cjyuan left a comment

Choose a reason for hiding this comment

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

Can you do a self-check against the answers on https://github.com/CodeYourFuture/Module-Tools/blob/solutions/number-systems/README.md?plain=1

Should you have any question regarding number systems, feel free to raise them in the comment of this PR.

@cjyuan cjyuan added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Mar 24, 2026
@Iswanna
Copy link
Author

Iswanna commented Mar 26, 2026

Can you do a self-check against the answers on https://github.com/CodeYourFuture/Module-Tools/blob/solutions/number-systems/README.md?plain=1

Should you have any question regarding number systems, feel free to raise them in the comment of this PR.

Hi @cjyuan,

Thank you for the review. I will carry out a self-check against my answers and leave a comment on this PR as advised.

Apologies for the delayed response. I didn’t receive a notification as I usually do when my PR is reviewed. I only noticed your review when I manually checked my PRs.

@Iswanna
Copy link
Author

Iswanna commented Mar 27, 2026

Can you do a self-check against the answers on https://github.com/CodeYourFuture/Module-Tools/blob/solutions/number-systems/README.md?plain=1

Should you have any question regarding number systems, feel free to raise them in the comment of this PR.

@Iswanna Iswanna closed this Mar 27, 2026

How many bits would you need in order to store the numbers between 0 and 1000 inclusive?
Answer:
Answer: 10 bits
Copy link
Author

Choose a reason for hiding this comment

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

I have compared my answer with the answer in the reference material, and I can see that the provided answer is more detailed. It correctly notes that 10 bits provide 1,024 combinations, leaving 23 spare values after storing the range 0–1000 (which contains 1,001 total values). I will update my answer to include this context.


How can you test if a binary number is a power of two (e.g. 1, 2, 4, 8, 16, ...)?
Answer:
Answer: It can be tested if one of the bits in the value is one e.g 0100, 10, 100
Copy link
Author

Choose a reason for hiding this comment

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

I also see here that my answer was not precise enough. Stating that ‘one of the bits in the value is one' could incorrectly include numbers like 3 (0011) or 6 (0110), which are not powers of two. I have updated the answer to specify that there must be exactly one bit set to 1.


If reading the byte 0x21 as a greyscale colour, as described in "Approaches for Representing Colors and Images", what colour would it mean?
Answer:
Answer: very dark gray
Copy link
Author

Choose a reason for hiding this comment

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

I used 'very dark grey' because 0x21 (decimal 33) is quite close to 0 (black). However, I will update my answer to 'Dark grey' to maintain consistency with the reference material.

Copy link

Choose a reason for hiding this comment

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

They mean the same thing. There is no standard for "dark" or "very dark".

@Iswanna Iswanna reopened this Mar 27, 2026
@cjyuan
Copy link

cjyuan commented Mar 27, 2026

Your proposed changes look good.

Why close the PR when you were making changes? I think you did it several times in ITP too.

@cjyuan cjyuan added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Mar 27, 2026
@Iswanna
Copy link
Author

Iswanna commented Mar 27, 2026

Your proposed changes look good.

Why close the PR when you were making changes? I think you did it several times in ITP too.

Hi @cjyuan, I just noticed that the PR was closed 5 hours ago; this was unintentional. If this happened previously in ITP, those were also accidents. I'll be more careful to avoid clicking the 'Close with comment' button in the future. Thank you for approving my PR!

- Specify that 10 bits leave 23 spare values (1001–1023) for the 0–1000 range
- Clarify power-of-two test: exactly one '1' in binary representation
- Adjust greyscale wording/capitalization to "Dark grey"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed. Module-Tools The name of the module. 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants