Skip to content

Conversation

Copy link
Contributor

Copilot AI commented May 19, 2025

  • Understand the issue and create a plan
  • Implement the fix in Model::Generate() to clear previous results
  • Remove accidentally committed libpict.so binary file
  • Revert .gitignore changes
  • Keep only the essential changes in model.cpp

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits May 19, 2025 18:04
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
Copilot AI changed the title [WIP] PictGenerate does not clean previous results before generating the new ones Fix PictGenerate to clear previous results before generating new ones May 19, 2025
Copilot AI requested a review from jaccz May 19, 2025 18:08
Copilot finished work on behalf of jaccz May 19, 2025 18:08
@jaccz
Copy link
Member

jaccz commented Jan 9, 2026

@copilot just keep changes needed in model.cpp and discard the rest

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 9, 2026

@copilot just keep changes needed in model.cpp and discard the rest

Done. Removed libpict.so binary and reverted .gitignore changes. The PR now only contains the fix in model.cpp (commit ea621f2).

@jaccz jaccz marked this pull request as ready for review January 9, 2026 21:48
@jaccz jaccz merged commit 166f30c into main Jan 9, 2026
10 checks passed
@jaccz jaccz deleted the copilot/fix-123 branch January 9, 2026 21:48
takeyaqa added a commit to takeyaqa/pict-wasm that referenced this pull request Jan 12, 2026
…microsoft#134) (#39)

* Initial plan for issue

* Fix PictGenerate to clear previous results before generating new ones



* Fix PictGenerate to clear previous results and update .gitignore



* Remove libpict.so binary and revert .gitignore changes



---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
takeyaqa added a commit to takeyaqa/pict-wasm that referenced this pull request Jan 25, 2026
* Fix PictGenerate to clear previous results before generating new ones (microsoft#134)

* Initial plan for issue

* Fix PictGenerate to clear previous results before generating new ones

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix PictGenerate to clear previous results and update .gitignore

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Remove libpict.so binary and revert .gitignore changes

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Adding handing for DBCS characters (microsoft#140)

* Initial plan for issue

* Fix PictGenerate to clear previous results before generating new ones

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix PictGenerate to clear previous results and update .gitignore

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Remove libpict.so binary and revert .gitignore changes

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix Unicode handling and add test for Japanese Kanji characters

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Add comprehensive development guidelines for the PICT repository

* chore: resolve conflict

* chore: resolve conflict

* Revert "chore: resolve conflict"

This reverts commit 3087955.

* Revert "chore: resolve conflict"

This reverts commit 3870bb6.

* chore: remove copilot instructions document

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
Co-authored-by: Jacek Czerwonka <jacekcz@microsoft.com>
takeyaqa added a commit to takeyaqa/pict-wasm that referenced this pull request Jan 25, 2026
* Fix PictGenerate to clear previous results before generating new ones (microsoft#134)

* Initial plan for issue

* Fix PictGenerate to clear previous results before generating new ones

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix PictGenerate to clear previous results and update .gitignore

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Remove libpict.so binary and revert .gitignore changes

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Adding handing for DBCS characters (microsoft#140)

* Initial plan for issue

* Fix PictGenerate to clear previous results before generating new ones

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix PictGenerate to clear previous results and update .gitignore

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Remove libpict.so binary and revert .gitignore changes

Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>

* Fix Unicode handling and add test for Japanese Kanji characters

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Add comprehensive development guidelines for the PICT repository

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jaccz <9115223+jaccz@users.noreply.github.com>
Co-authored-by: Jacek Czerwonka <jacekcz@microsoft.com>
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.

2 participants