Skip to content

Merge pull request #1 from kolonuk/feat/string-art-generator - #3

Merged
kolonuk merged 7 commits into
mainfrom
dev
Oct 14, 2025
Merged

Merge pull request #1 from kolonuk/feat/string-art-generator#3
kolonuk merged 7 commits into
mainfrom
dev

Conversation

@kolonuk

@kolonuk kolonuk commented Oct 13, 2025

Copy link
Copy Markdown
Owner

Create String Art Generator Website

kolonuk and others added 3 commits October 13, 2025 17:24
Create String Art Generator Website
This commit resolves a workflow failure caused by a deprecated action dependency. The `actions/upload-pages-artifact` action has been updated from `v2` to `v3` to comply with the latest requirements and ensure the deployment workflow runs successfully.
fix(deploy): Update upload-pages-artifact to v3
This commit resolves a workflow failure where the `configure-pages` action could not find a configured GitHub Pages site.

- The `enablement: true` parameter has been added to the `actions/configure-pages@v4` step.
- This allows the workflow to automatically enable GitHub Pages for the repository if it's not already active, preventing the 'Not Found' error.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: John Wood <john@kolon.co.uk>
This commit significantly enhances the string art generator based on user feedback.

Algorithm Improvements:
- The core generation algorithm has been updated to produce higher-quality, more detailed images.
- It now prevents the same line from being drawn multiple times, forcing the generator to find new paths and create a more distributed and accurate pattern.
- The process of "bleaching" the image data after drawing a line has been refined to better handle light and dark areas.

UI/UX Enhancements:
- A progress bar overlay is now displayed during the generation process, providing clear visual feedback for this long-running operation.
- The "Generate" button is disabled during processing to prevent multiple clicks.
- The default number of threads has been increased from 1000 to 3000 to produce a better result out-of-the-box.

Technical Changes:
- The generation function is now asynchronous, allowing the UI to remain responsive and update the progress bar.
- CSS and HTML have been updated to support the new progress bar elements.
- Fixed a CSS specificity issue that was causing the progress overlay to block clicks in tests.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: John Wood <john@kolon.co.uk>
This commit delivers a robust, high-quality string art generator by addressing all user feedback and passing a full suite of pre-commit checks.

Key Improvements:
- **Algorithm Enhancement:** The core generation algorithm has been significantly improved. It now uses a scoring system that factors in both line darkness and line length, preventing the repetitive short-path issue and producing much more accurate and detailed images.
- **UI Transparency:** A "Processed Image" canvas has been added to the UI. This shows the user the exact low-resolution, grayscale image that the algorithm is using as a source, making the process much more transparent.
- **Workflow Fix:** The GitHub Actions deployment workflow has been corrected to use the non-deprecated `actions/upload-pages-artifact@v4`, ensuring reliable deployments.
- **Improved Testing:** A new unit test has been added for the core `generateStringArt` function to catch future regressions. Frontend verification scripts were also refined to be more specific and reliable.

This version of the application is now feature-complete, robust, and produces high-quality results as per the user's requirements.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: John Wood <john@kolon.co.uk>
This commit adds explanatory comments to the `.github/workflows/deploy.yml` file.

The comments clarify that when a deployment is run for the `dev` branch, a unique preview URL is generated. It explicitly states that this URL can be found in the summary of the workflow run on the 'Actions' tab of the GitHub repository. This is intended to make it easier for users to find the link to their test server/preview environment.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: John Wood <john@kolon.co.uk>
@kolonuk
kolonuk merged commit 9e60aaa into main Oct 14, 2025
1 check passed
@kolonuk
kolonuk deleted the dev branch October 14, 2025 21:27
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.

1 participant