Skip to content

FirePheonix GSOC'25 week-02 report #224

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

Merged
merged 6 commits into from
Jun 23, 2025

Conversation

FirePheonix
Copy link
Contributor

weekly GSOC Report

@FirePheonix FirePheonix changed the title FirePheonixGSOC'25week02 FirePheonix GSOC'25 week-02 report Jun 16, 2025
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@sa-fw-an
Copy link
Member

@FirePheonix Please Add an Authors file and change the titles of the blog. For documentation refer this

Update all your Blogs to look like this and raise a seperate PR for it

Have a look at my blog for new authors page and make a page somewhat like this


2. **Graphics for Lego Bricks**
- The following feature helps to use a real time video(webcam) onto the Lego Notation detection interface. Also, you may freely edit and move it around the canvas.
![screenshot-description](https://i.ibb.co/cXL4Hpxq/2nd.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

These images are not rendering for me.

Screenshot from 2025-06-16 13-02-19

Perhaps add them to the repo's assets?

Also, please add descriptions to the images.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed.


3. **Identified methods to exporting blocks' output as an action block**
- I read and went through a lot of documentation, saw a lot of code in the Phrase Maker widget that would help in exporting the code as: midi and as action block.
![screenshot-description](https://i.ibb.co/bVD8Z54/image.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Some comment as above.

date: "2025-06-14"
slug: "2025-06-14-gsoc-25-FirePheonix-week02"
description: "GSoC'25 Contributor at SugarLabs (Color Sensor for music blocks)"
author: "Shubham Singh"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update this to our new author system.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

@pikurasa
Copy link
Contributor

@FirePheonix Please address comments, so that it may be merged.

Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@@ -0,0 +1,35 @@
---
name: "Shubham Singh"
slug: "shubham singh"
Copy link
Contributor

Choose a reason for hiding this comment

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

Because you don't have a hyphen here, the page is not the same as what is linked from your blog.

This is resulting in a blank page, when navigating from the blog.

Screenshot from 2025-06-22 20-35-54

Please add the hyphen, test the results, and push your changes upon success.

Copy link
Member

Choose a reason for hiding this comment

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

I tried that in his PR as well, somehow it does not seem to work. Ive tried pasting my blog and changing the slugs, does not work :( May be @FirePheonix Try raising a new PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

It works for me, so I recommend trying the change in this PR.

hyphen added:
Screenshot from 2025-06-22 21-23-48

result (page works):
Screenshot from 2025-06-22 21-22-26

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i've been trying to do this for so long(since wednesday). thanks devin.
And yeah, I found the ACTUAL issue.

I put FirePheonix in .md file's slug.
I should've been firepheonix.
Now everything is working fine.

Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@pikurasa pikurasa merged commit 9f8035f into sugarlabs:main Jun 23, 2025
3 checks passed
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.

3 participants