search

zoccoler / napari-flim-phasor-plotter

BSD 3-Clause "New" or "Revised" License
10 stars 1 forks source link

Adding widget to init for workflow access #42

Closed sviaro closed 1 year ago

sviaro commented 1 year ago

Hey, @zoccoler! I changed the init file so that imports widget.py.

You can import it but there is one issue. When I tried to call the function flim_plot._widget.make_flim_phasor_plot and put the parameters in it didn't execute it and showed an error. You can see the code in test.ipynb

codecov[bot] commented 1 year ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (f07d4f7) 64.72% compared to head (c382f01) 64.72%.

:exclamation: Current head c382f01 differs from pull request most recent head d2547a8. Consider uploading reports for the commit d2547a8 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #42 +/- ## ======================================= Coverage 64.72% 64.72% ======================================= Files 15 15 Lines 859 859 ======================================= Hits 556 556 Misses 303 303 ``` | [Files](https://app.codecov.io/gh/zoccoler/napari-flim-phasor-plotter/pull/42?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marcelo+Zoccoler) | Coverage Δ | | |---|---|---| | [src/napari\_flim\_phasor\_plotter/\_\_init\_\_.py](https://app.codecov.io/gh/zoccoler/napari-flim-phasor-plotter/pull/42?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marcelo+Zoccoler#diff-c3JjL25hcGFyaV9mbGltX3BoYXNvcl9wbG90dGVyL19faW5pdF9fLnB5) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

zoccoler commented 1 year ago

Awesome, it works! I am just removing the test file then and merging. Thanks again :)