Skip to content

Conversation

@jochym
Copy link
Owner

@jochym jochym commented Jan 2, 2026

This PR is for work on the interactive book for computational solid state physics.

@jochym jochym requested a review from Copilot January 2, 2026 15:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


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

@jochym
Copy link
Owner Author

jochym commented Jan 2, 2026

@copilot translate the first chapter of the book (book/main.tex) to english. Let me review critical passages and formulations (mark them in the text).

Copy link
Contributor

Copilot AI commented Jan 2, 2026

@jochym I've opened a new pull request, #26, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 14 commits January 2, 2026 16:32
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
Complete translation of Introduction and Chapter 1 from Polish to English with review markers
* Initial plan

* Add GitHub Copilot instructions for the repository



* Clarify academic tasks can be modified if directly requested



---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
…goal in copilot-instructions.md (#29)

* Initial plan

* Update copilot-instructions.md with comprehensive book branch documentation

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Clarify LaTeX guideline: allow chapter translation by user request

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Add Quarto migration goal to copilot instructions

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
@jochym
Copy link
Owner Author

jochym commented Jan 2, 2026

@copilot start a new PR based on branch book for translation of chapter 2

Copy link
Contributor

Copilot AI commented Jan 2, 2026

@jochym I've opened a new pull request, #30, to work on those changes. Once the pull request is ready, I'll request review from you.

…glish (#30)

* Initial plan

* Translate Chapter 2 (Electronic states in crystals) from Polish to English

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Address review comments: remove review markers and simplify phrasing

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

@copilot start a new PR against book branch for translation of chapter 3, and make the translation

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #31, to work on those changes. Once the pull request is ready, I'll request review from you.

@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

As before work in the dedicated PR against book branch. The same way you did for previous chapters @copilot

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #37, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits January 3, 2026 11:56
* Initial plan

* Translate Chapter 7 (Electric polarization) from Polish to English

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Fix missing summation index m in equation (line 3268)

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Change "extreme cases" to "limiting cases" per author feedback

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Remove review marker for "structural phase transition" terminology

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

@copilot as with previous chapters: crete PR for translation of ch8 in the book branch and translate the chapter.

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #38, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* Translate Chapter 8 (Van der Waals interactions) to English

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

Nice work on ch8 @copilot . Now create new PR in the book branch for the translation of ch. 9 and run the translation there.

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #39, to work on those changes. Once the pull request is ready, I'll request review from you.

…#39)

* Initial plan

* Translate Chapter 9 to English

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Refine Chapter 9 translation based on code review feedback

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

* Remove review markers after user confirmation of terminology

Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jochym <5993422+jochym@users.noreply.github.com>
@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

@copilot How many chapters is in this text?

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #40, to work on those changes. Once the pull request is ready, I'll request review from you.

@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

@copilot Please make a new PR against book branch, for the translation of chapter 10 and all remaining text in the document (main.tex).

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #41, to work on those changes. Once the pull request is ready, I'll request review from you.

@jochym
Copy link
Owner Author

jochym commented Jan 3, 2026

@copilot We are missing an action for building the pdf output of the book. Can you create such action? As a separate PR against book branch, or maybe this needs to be in the main branch? I am not expert in github actions. Please describe how this should work in our particular case.

Copy link
Contributor

Copilot AI commented Jan 3, 2026

@jochym I've opened a new pull request, #42, to work on those changes. Once the pull request is ready, I'll request review from you.

@jochym
Copy link
Owner Author

jochym commented Jan 4, 2026

@copilot the PDF creation action is nice but where we can access the log files from the run? Right now there is no easy way to debug army problems?

Copy link
Contributor

Copilot AI commented Jan 4, 2026

@jochym I've opened a new pull request, #43, to work on those changes. Once the pull request is ready, I'll request review from you.

@jochym
Copy link
Owner Author

jochym commented Jan 4, 2026

@copilot I had to close this PR. It was based on wrong branch. Please, create a new one based on main with the same modifications to the action ( PDF creation debugging)

Copy link
Contributor

Copilot AI commented Jan 4, 2026

@jochym I've opened a new pull request, #44, to work on those changes. Once the pull request is ready, I'll request review from you.

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