ASFHyP3 / hyp3-docs

Documentation for HyP3 products and services
https://hyp3-docs.asf.alaska.edu/
BSD 3-Clause "New" or "Revised" License
10 stars 10 forks source link

Jackie's feedback on merge_bursts notebook #415

Closed jacquelynsmale closed 5 months ago

jacquelynsmale commented 5 months ago

Feedback for the notebook with the new merge_tops_burst workflow. To be more concise, I rearranged some things for the new notebook created in the add_merge_burst_to_notebook branch. I also was going over the story I thought was associated with this notebook, and I noticed that the acceptance criteria said :

The [burst insar time-series notebook](https://github.com/ASFHyP3/hyp3-docs/blob/main/docs/tutorials/hyp3_isce2_burst_stack_for_ts_analysis.ipynb) contains a section demonstrating how to merge bursts.

Would we prefer a new notebook with the merge_tops_bursts example, this new addition to be in the burst stack notebook, or both?

review-notebook-app[bot] commented 5 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

jacquelynsmale commented 5 months ago

I just wanted to clarify some of the changes I made more specifically

General questions and feedback: