kelcyno

@freemansw1 Nothing was going right on my end so I just started fresh. Let me know if there's anything wonky.

Sean, I still have some edits from Julia to work on. It can wait until after your defense.

@freemansw1 @JuliaKukulies I updated the merge_split.py to comply with documentation and Julia's previous comments. Looking forward to more input/comments.

@freemansw1 @JuliaKukulies I've added documentation and recent changes we discussed in the last tobac meeting. Ready for another round of reviews!

@snilsn I did not update the assertion at the end when I updated how I was numbering tracks. I've tried it with your cells and your test dataset made it...

@freemansw1 I'm not a huge fan of only making the distance threshold mandatory. I think it's important to have a default option, as the other functions all have default values...

@JuliaKukulies I can review the PR. Real quick, to make sure our labeling is consistent - the Feature numbering begins at 1, the cell numbering begins at 0 or 1?...

Oh this looks really interesting @w-k-jones I'll get started on it!

I'm not sure if this fully addresses was @deeplycloudy was needing, but the combined xarray dataset of feature/segmentation/tracking is available now in the utils (standardize_track_dataset) as of merge #136.

I also like this plan and I agree with will’s note below. For the larger scale tests and more real data, I have several gridded radar cases that are anywhere...