sync branch by Jana-Elst · Pull Request #38 · Jana-Elst/passionProject

Jana-Elst · 2026-02-07T10:43:00Z

No description provided.

sync branches?

did some arduino tests

Blog

did some arduino tests

Dev

changed planning

Blog

Dev

changed some typos

Dev

added warning/problem

Removed completed tasks and added new tasks for weeks 3 to 5.

Blog

Dev

Removed completed tasks from the README for clarity.

Co-authored-by: Jana-Elst <187283975+Jana-Elst@users.noreply.github.com>

Optimize media assets to fix GitHub Pages deployment failures

Copilot

Pull request overview

This PR syncs content and prototype code updates by migrating blog media references to more web-friendly formats, adding/adjusting Arduino test sketches, and updating the project planning notes in the README.

Changes:

Update blog post heroImage references from .png to .webp and embedded video imports from .MOV to .mp4.
Add multiple Arduino test sketches for dial reading and SD-card audio playback; tweak relay test default states.
Refresh the README planning section with progress tracking and additional tasks.

Reviewed changes

Copilot reviewed 24 out of 105 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
blog/src/content/blog/20260202-making-the-housing.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260201-soldering-2.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260129-ring-the-bell-2.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260128-ring-the-bell.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260127-debugging-and-soundcards.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260123-first-python-script.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260122-code-for-the-arduinos.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260121-a-professional-circuit.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260120-recap.md	Add a highlighted callout block in the post.
blog/src/content/blog/20260118-buffering-and-bytes.mdx	Switch hero image to `.webp`; update video imports to `.mp4`.
blog/src/content/blog/20260117-raspberry-pi-getting-started.mdx	Switch hero image to `.webp`; update video import to `.mp4`.
blog/src/content/blog/20260115-my-first-pi-steps.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260114-reading-dialed-number.mdx	Switch hero image to `.webp`; update video imports to `.mp4`.
blog/src/content/blog/20260113-the-wrong-relays.mdx	Switch hero image to `.webp`; update video imports to `.mp4`.
blog/src/content/blog/20260110-music.mdx	Update video imports to `.mp4`.
blog/src/content/blog/20260109-a-broken-phone.md	Switch hero image reference to `.webp`.
blog/src/content/blog/20260108-componets.md	Switch hero image and inline image to `.webp`.
blog/src/content/blog/20260106-intercom.mdx	Switch hero image + images to `.webp`; update video imports to `.mp4`.
blog/src/content/blog/20260105-hardware.md	Switch hero image + gallery images to `.webp`.
blog/src/assets/pictures/20260113-header.webp	Add a new `.webp` asset to support updated blog references.
arduinoTests/20260114-1-reading-dial/20260114-1-reading-dial.ino	Add dial pulse counting sketch with debounce + timeout logic.
arduinoTests/20260113_1_relaisPlay2Songs/20260113_1_relaisPlay2Songs.ino	Add SD/WAV playback sketch alternating between two outputs.
arduinoTests/20260113-1-relaisPlay1Song/20260113-1-relaisPlay1Song.ino	Add SD/WAV playback sketch for a single output.
arduinoTests/20260112-1-testRelays/20260112-1-testRelays.ino	Change initial relay pin writes (LOW → HIGH).
README.md	Update planning checklist/progress notes and add tasks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-07T10:48:53Z

+    // If the pulse is real and the state has actually changed
+    static int stableState = LOW;
+    if (reading != stableState) {
+      stableState = reading;


stableState is initialized to LOW while dialPin uses INPUT_PULLUP (idle HIGH). On first stable read this will flip to HIGH and increment pulseCount, creating a phantom pulse on startup. Initialize stableState from the first reading (or set it to HIGH in setup) to avoid counting an initial pulse.

Suggested change

// If the pulse is real and the state has actually changed

static int stableState = LOW;

if (reading != stableState) {

stableState = reading;

// Initialize the stable state from the first stable reading to avoid a phantom pulse

static int stableState;

static bool stableStateInitialized = false;

if (!stableStateInitialized) {

stableState = reading;

stableStateInitialized = true;

} else if (reading != stableState) {

// If the pulse is real and the state has actually changed

stableState = reading;

Copilot · 2026-02-07T10:48:54Z

  // Start with the relay OFF (NC connected to C)
-  digitalWrite(RELAY_PIN1, LOW);
-    digitalWrite(RELAY_PIN2, LOW);
+  digitalWrite(RELAY_PIN1, HIGH);
+    digitalWrite(RELAY_PIN2, HIGH);


The comment says the relay starts OFF with “NC connected to C”, but the code now sets both relay pins HIGH. If these relay modules are active-low, HIGH would actually be OFF; if they’re active-high, this would be ON. Please align the comment (and indentation) with the actual relay logic so the setup state is unambiguous.

Copilot · 2026-02-07T10:48:54Z

+  - ~~send instructions to the arduino to open & close the relais at the right moments~~
+- ~~add relais for the bell to the circuit~~
+    - ~~write extra arduino code~~
+-~~change sound system to the usbcards (instead of the arduino) (Wednesday)~~


Markdown list item is missing a space after -, so it may not render as a list entry (and the strikethrough may not apply consistently). Add a space so it’s - ~~change ...~~.

Suggested change

-~~change sound system to the usbcards (instead of the arduino) (Wednesday)~~

- ~~change sound system to the usbcards (instead of the arduino) (Wednesday)~~

Copilot · 2026-02-07T10:48:55Z

- The dail -> Thursday
-    - How does it work?
-    - Detect which number is dialed
+- ~~The dail -> Thursday~~


Typo in task name: “dail” should be “dial”.

Suggested change

- ~~The dail -> Thursday~~

- ~~The dial -> Thursday~~

Copilot · 2026-02-07T10:48:55Z

 pubDate: 'Feb 01 2026'
-heroImage: ../../assets/pictures/20260201-header.png
+heroImage: ../../assets/pictures/20260201-header.webp


After switching the hero image reference to the .webp, the old .png version appears to be unreferenced in the repo. Consider removing the unused .png asset (or moving source assets to a separate location) to avoid repo/bundle bloat.

Copilot · 2026-02-07T10:48:55Z

+<div style="background-color: #FFD700; padding: 25px; margin-top: 50px;">
+
+> Vanaf hier heb ik eigenlijk geen idee meer hoe ik verder moet...
+> Geen idee hoe ik de verschillende circuits aan elkaar moet hangen, noch hoe ik geluidsignalen moet opnemen...
+
+</div>


The callout uses inline CSS styles inside markdown. This is the only place in the blog posts using style=, which makes theming harder and can be blocked by stricter CSP/sanitizers. Consider replacing this with a semantic wrapper + CSS class defined in the site styles (and keep the markdown content outside the HTML wrapper if possible).

Copilot · 2026-02-07T10:48:55Z

+const int bufferSize = 64; 
+uint8_t buf[bufferSize];
+volatile int head = 0, tail = 0;
+
+File file;
+volatile bool playing = false;
+


head/tail are volatile int and are read/written from both loop() and the ISR. On AVR, int is 16-bit and updates are not atomic, so the ISR can observe torn values and corrupt the ring buffer. Use an 8-bit type for indices (bufferSize=64 fits), or protect multi-byte reads/writes with an atomic section.

Copilot · 2026-02-07T10:48:55Z

+// Use 'volatile' for variables shared between loop and ISR
+const int bufferSize = 64; // Reduced to 64 to save RAM
+uint8_t buf1[bufferSize];
+volatile int head1 = 0, tail1 = 0;


head1/tail1 are volatile int and accessed from both loop() and ISR(TIMER3_COMPA_vect). On AVR, int is 16-bit and not atomically updated, which can corrupt the ring buffer under interrupt. Prefer volatile uint8_t indices (bufferSize=64) or wrap accesses in an atomic block.

Suggested change

volatile int head1 = 0, tail1 = 0;

volatile uint8_t head1 = 0, tail1 = 0;

Copilot · 2026-02-07T10:48:56Z

 ### Week 4 (26 - 1 feb)
 **Scenografie + test the installation**
- Solder everything together
+- ~~wirte the whole python logic (Monday)~~


Typo: “wirte” should be “write”.

Suggested change

- ~~wirte the whole python logic (Monday)~~

- ~~write the whole python logic (Monday)~~

Copilot · 2026-02-07T10:48:56Z

 ---
-import piGettingStarted from '../../assets/videos/20260117-pi-getting-started.MOV?url';
+import piGettingStarted from '../../assets/videos/20260117-pi-getting-started.mp4?url';


Now that the post imports the .mp4 version, the old .MOV asset appears to be unreferenced. Consider removing the unused .MOV to reduce repo size (or keep originals in a separate source-assets location).

Jana-Elst and others added 20 commits January 13, 2026 10:02

Merge pull request #20 from Jana-Elst/main

a403eaa

sync branches?

did some arduino tests

017a15d

Merge pull request #21 from Jana-Elst/arduino

76cabe7

did some arduino tests

Merge pull request #22 from Jana-Elst/blog

0991d9a

Blog

Merge pull request #23 from Jana-Elst/arduino

866434e

did some arduino tests

Merge pull request #24 from Jana-Elst/dev

4995b5e

Dev

add planning

ef07acb

Merge pull request #25 from Jana-Elst/dev

e416ce4

changed planning

Merge pull request #26 from Jana-Elst/blog

985c0e2

Blog

Merge pull request #27 from Jana-Elst/dev

8ca042d

Dev

Merge pull request #28 from Jana-Elst/blog

1e1c7a3

changed some typos

Merge pull request #29 from Jana-Elst/dev

74863d5

Dev

Merge pull request #30 from Jana-Elst/blog

0148732

added warning/problem

Update README with completed and new tasks

cd2fd4d

Removed completed tasks and added new tasks for weeks 3 to 5.

Merge pull request #33 from Jana-Elst/blog

21607b8

Blog

Merge pull request #34 from Jana-Elst/dev

de93b40

Dev

Update README to reflect completed tasks

5583635

Removed completed tasks from the README for clarity.

Initial plan

e753e07

Optimize media files: convert videos to MP4 and images to WebP

fba1873

Co-authored-by: Jana-Elst <187283975+Jana-Elst@users.noreply.github.com>

Merge pull request #35 from Jana-Elst/copilot/fix-website-update-issue

f280bd3

Optimize media assets to fix GitHub Pages deployment failures

Copilot AI review requested due to automatic review settings February 7, 2026 10:43

Copilot started reviewing on behalf of Jana-Elst February 7, 2026 10:43 View session

Jana-Elst merged commit 5821a0d into blog Feb 7, 2026
6 checks passed

Copilot AI reviewed Feb 7, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

sync branch#38

sync branch#38
Jana-Elst merged 20 commits into
blogfrom
main

Jana-Elst commented Feb 7, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Copilot AI Feb 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	-~~change sound system to the usbcards (instead of the arduino) (Wednesday)~~
	- ~~change sound system to the usbcards (instead of the arduino) (Wednesday)~~

	volatile int head1 = 0, tail1 = 0;
	volatile uint8_t head1 = 0, tail1 = 0;

	- ~~wirte the whole python logic (Monday)~~
	- ~~write the whole python logic (Monday)~~

Conversation

Jana-Elst commented Feb 7, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants