VirtualPhotonics / VTS

Virtual Tissue Simulator
https://virtualphotonics.org
Other
34 stars 9 forks source link

Feature/code cleanup #49

Closed lmalenfant closed 2 years ago

hayakawa16 commented 2 years ago

Thanks for cleaning this all up!! I just pulled master and see now that there is no vts/tools folder. And UpdateBackupLocation is gone from Vts.csproj and Vts.Test.csproj. Where did you see the Vts.Desktop and Vts.Desktop.Test? In src?

I made some headway today on the zemax branch. I removed MCCL from the dependencies and rewrote unit tests. You'll see next time your on SourceTree. And I realized that I do give the user the option to designate which folder the database will reside, it is the same option that the user defines for all detector results. I wasn't very good switching topics this week. Put me on another task for a few days and the prior one has been erased!

Thanks for all you help this week! Carole

On 2022-09-16 14:17, Lisa Malenfant wrote:

Merged #49 [1] into master.

-- Reply to this email directly, view it on GitHub [2], or unsubscribe [3]. You are receiving this because your review was requested.Message ID: @.***>

Links:

[1] https://urldefense.com/v3/__https://github.com/VirtualPhotonics/VTS/pull/49__;!!CzAuKJ42GuquVTTmVmPViYEvSg!NxZWBdiUt1EujMioeblYz6avz4F1nm_X57iddocStiZAPVvg0Wvc1A4fWZRv1HOrAfn1Ypy7fndx_3tB13YvwYS_$ [2] https://urldefense.com/v3/__https://github.com/VirtualPhotonics/VTS/pull/49*event-7406176568__;Iw!!CzAuKJ42GuquVTTmVmPViYEvSg!NxZWBdiUt1EujMioeblYz6avz4F1nm_X57iddocStiZAPVvg0Wvc1A4fWZRv1HOrAfn1Ypy7fndx_3tB1wFejwav$ [3] https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AGNQ7U4GKFIDAHKIAXI45IDV6TPVBANCNFSM6AAAAAAQOVHXD4__;!!CzAuKJ42GuquVTTmVmPViYEvSg!NxZWBdiUt1EujMioeblYz6avz4F1nm_X57iddocStiZAPVvg0Wvc1A4fWZRv1HOrAfn1Ypy7fndx_3tB13gdlhDC$