Simon Biggs

Results 652 comments of Simon Biggs

So, I needed to use [WiXInstaller](https://pyoxidizer.readthedocs.io/en/latest/tugger_starlark_type_wix_installer.html?highlight=WiXInstaller#wixinstaller) to be able use that parameter `install_files_root_directory_id`. It has an `add_simple_installer` which I made use of... but I couldn't work out how to include...

Having a look at the following: https://github.com/indygreg/PyOxidizer/blob/4d9007221087cd99e748fef9b6d5361fd2690cd2/tugger/src/starlark/wix_installer.rs#L544-L560 It appears that potentially [the docs](https://pyoxidizer.readthedocs.io/en/latest/tugger_starlark_type_wix_installer.html?highlight=WiXInstaller#starlark_tugger.WiXInstaller.add_simple_installer) might be missing the id parameter. So tried using an id parameter. See https://github.com/pymedphys/pymedphys/pull/1557/commits/a52eb773d76334ae938b768f2e424c40ff648cbb

So, that worked. New error is: ``` error LGHT0094 : Unresolved reference to symbol 'Directory:LocalAppDataFolder' in section 'Fragment:'. ``` Tried changing the name to `'LOCALAPPDATAFOLDER'` but no luck... hmmm... I...

> Thanks for the call-out! Thanks for all the work building PyOxidizer :) It'd be pretty awesome to be able to make a standalone Streamlit app installer, I'm sure I'll...

So, this project did get so close to crossing the finish line and, it would be quite helpful to have an "easy distribution" option for PyMedPhys which can be downloaded...

I use `asdf` locally to manage my various programming languages. And even though pyoxidizer can be installed through pip, [its preferred approach](https://pyoxidizer.readthedocs.io/en/v0.4.0/getting_started.html#installing-pyoxidizer) is installed as a rust create. So I...

Hi @crcrewso, I'd be happy to help you if you were willing to set up a Streamlit app for this?

> Is it easy to turn a streamlit app to a native executable? A while ago I toyed with making a PyMedPhys .exe that booted the Streamlit GUI: https://github.com/pymedphys/pymedphys/releases/tag/v0.36.0-dev2 That...

Actually, just making another python file something like the following is what I was thinking: https://github.com/pymedphys/pymedphys/blob/37f456768824d569e78503a5cbadcb832b0845f0/lib/pymedphys/_experimental/streamlit/apps/dicom_explorer.py#L15-L41 If you copy the above file call it something else, change the TITLE to...

We now have a means to have these helper functions be tested within. The PyMedPhys testing suite. See the following: https://github.com/pymedphys/pymedphys/blob/e75029b51eef564caf25fbfc0001025fcc03adbd/lib/pymedphys/tests/mosaiq/test_patient_name_fields.py#L59 Would you be able to include a test with...