Skip to content
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

Brachytherapy #521

Closed
wants to merge 31 commits into from
Closed

Brachytherapy #521

wants to merge 31 commits into from

Conversation

Copani
Copy link

@Copani Copani commented Aug 19, 2021

Including basic brachytherapy workflow.
view thesis fpr detailed describtion
https://nc.u4n.de/s/tmzd5sjbdYSRxQm

An attempt to develop a brachytherapy planing tool using the matRad framework.

For now, everything is stored in an extra folder called "brachytherapy"
First minimal prototype of stf generatin seed positions based on needle seeds and template position
also some changes and playarounds
almost done... except for some fixes it all works
What remains to do?
- write some better tests
- get GUI to reoptimize (-> show radiation mode)
The code is in a form that is considered closed and pull request - ready.
@wahln
Copy link
Contributor

wahln commented Aug 19, 2021

Thanks a lot for the pull request and the work you put in your thesis!

Before doing a code review I will start with two obvious things

  • .MatlabDriveTag -> All of these files should be removed, since they have no meaning to the matRad-code
  • You can add your name to the AUTHORS.txt

@wahln
Copy link
Contributor

wahln commented Aug 19, 2021

I resolved the merge conflict in the AUTHORS.txt file

@wahln wahln self-requested a review August 19, 2021 08:58
@wahln wahln self-assigned this Aug 19, 2021
@wahln
Copy link
Contributor

wahln commented Dec 2, 2021

Drive-Tags and conflicts have been resolved.

@ferdymercury
Copy link
Contributor

Thanks a lot @Copani
Could you provide an updated link to the thesis? the Nexcloud file no longer exists.

@Copani
Copy link
Author

Copani commented Nov 29, 2023

Sure, @ferdymercury - this link should work!

@ferdymercury
Copy link
Contributor

ferdymercury commented Nov 29, 2023

Sure, @ferdymercury - this link should work!

Sorry, I get:
Das hat nicht funktioniert.
Leider wurde "xx...@ y...." im Verzeichnis "bwedu-my.sharepoint.com" nicht gefunden.

@Copani
Copy link
Author

Copani commented Nov 29, 2023

Okay, I made a new link

Cheers
Claus

@ferdymercury
Copy link
Contributor

Works now, thanks a lot!

@wahln, are there plans to merge this? Thanks again :)

@wahln
Copy link
Contributor

wahln commented Nov 29, 2023

There are plans to merge this, we will probably start working on this early 2024. So keep following the branch and if you want to help, let us know.

@wahln
Copy link
Contributor

wahln commented May 23, 2024

I will close this PR in favor of #718, which targets a newer branch dev_varRBErobOpt and will receive the necessary update from #717. Follow the progress there.

@wahln wahln closed this May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants