Closed jenna-tomkinson closed 4 months ago
@yfukai
I would appreciate it if you could give this pull request a review! I can not add anyone, so I will mention you in this comment. Thank you!
Hi @yfukai, @tying84, and @Nicholas-Schaub,
I know you all are probably super busy but wanted to give a small poke about this PR.
I am really excited to see what comments you all have 😄.
Thank you in advance!
Hi @jenna-tomkinson, thank you for this pull request, and I'm sorry for the late reply; I myself have been completely involved with the benchmarking for the publication (and also other projects). Right, as you point out, it is more useful if the link is directed to the notebook, including the results.
Actually, those notebooks are automatically executed when building the document ( https://basicpy.readthedocs.io/ ), and I believe that may be better than including the executed results since
About the image saving, we actually don't recommend
since it can lead to information loss (by discretization) or new artifacts. The images can be saved as npy
files or TIFF images but in a float format.
With this, I was wondering what you think about
Thanks!
@yfukai
Thank you for the review! I did not realize that is why the notebooks were not run, so I am glad to learn something new!
I agree with all of the comments and I can easily implement this!
I will note, I do have competing priorities that I will be directing my attention to, so I hope to get to finishing this PR in the near future. I will be keeping this on my list of to-dos, but I probably won't get back to this for a few weeks. If it seems like it has been a very long time, please feel free to poke me.
Thank you again!
@jenna-tomkinson I also apologize for being slow to respond. Do you still intend on completing this?
Hello @Nicholas-Schaub!
I am still interested in completing this PR, but my bandwidth this month is stretched thin with other priorities.
I will likely come back to finish this up in November.
Hello all,
My apologies for not getting back to this PR sooner. After considerable thought, I have decided to close this PR until further notice due to other competing priorities.
Thank you for the opportunity, and I wish you all the best of luck with this great tool!
Hello!
In this PR, I included the following changes:
skimage.io.imsave
from issue #91)