search

marksgraham / OCT-Converter

Tools for extracting the raw optical coherence tomography (OCT) and fundus data from proprietary file formats.
https://pypi.org/project/oct-converter/
MIT License
193 stars 70 forks source link

.fda error #99

Closed marksgraham closed 1 year ago

marksgraham commented 1 year ago

Discussed in https://github.com/marksgraham/OCT-Converter/discussions/97

Originally posted by **antoniohupa** March 22, 2023 Hi Mark, hope you're doing well. Recently I've installed the package with the newest updates. When I tried to execute the demo example for .fda files it throws an error (I attach a file to see it). On the other hand, this errors does not replicate with .fds files. Could you give me some orientations about? Thanks a lot? A [fda_error.pdf](https://github.com/marksgraham/OCT-Converter/files/11039304/fda_error.pdf)
marksgraham commented 1 year ago

Hi @antoniohupa

Converted to an issue to help me track it. I'm unable to replicate this issue on my end, could you do a few things to help me out?

1) Confirm it is happening with this attached file? eg_oct_fda.fda.zip

2) share what python version you are using

3) share the printing of oct_volume.volume (sorry I asked you to print the wrong thing before)

4) pinpoint which version this bug begins? if you run pip install oct-converter==0.5.6 i guess you get it? could you then downgrade incrementally using pip install --upgrade oct-converter==0.5.5 and test until you stop getting the error? You need to include --upgrade to succesfully downgrade, go figure :P

marksgraham commented 1 year ago

Closing as no updates, feel free to reopen if needed