Cardioid

[kfahn22]

Adding 133-cardioid

[netlify]

✅ Deploy Preview for codingtrain ready!

Name	Link
🔨 Latest commit	19d7be7
🔍 Latest deploy log	https://app.netlify.com/sites/codingtrain/deploys/62a9008b21a66f00082d769d
😎 Deploy Preview	https://deploy-preview-256--codingtrain.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[kfahn22]

I checked the deploy log and there errors are related to Image.png files. Do you know what is going on?

[NwE0kmCE]

@kfahn22 It looks like you haven't uploaded an image named pointcloud.png. Once you upload that it should work, unless I'm missing something!

[dipamsen]

I have no idea, this PR only changes 133-time-tables-cardioid-visualisation, and there is an error in a different directory, 168-the-mandelbulb (and more).
Also, the required image(s) is/are present in this branch. (there are multiple no such file or directory error messages)

Maybe something wrong with the deploy settings? (my PR is also failing)

[NwE0kmCE]

Oh, didn't notice, my bad! @fdoflorenzano do you have any idea what might be causing the issue?

[runemadsen]

I'll take a look at this first thing in the morning!

[shiffman]

Hmmmm, looks like something did go wrong somewhere! The same tests are failing with my recent pull request #258 too! I'm investigating now but leaving shortly to have dinner with @runemadsen maybe we can sort it out before our entree comes. 😂

[shiffman]

I see the errors, working on them in a new branch, fixes coming soon!

[shiffman]

#261 should hopefully resolve this 🤞, this (and other recent pull requests) will need to be updated/merged with main once I merge those fixes!

[kfahn22]

I have reduced the file size for four of the contribution images.