tensorflow-deep-learning icon indicating copy to clipboard operation
tensorflow-deep-learning copied to clipboard

Update Magic Function TensorBoard

Open sc0v0ne opened this issue 2 years ago • 4 comments

Update issue-610

sc0v0ne avatar Dec 09 '23 14:12 sc0v0ne

Hi, @sc0v0ne ,

Thank you for this!

Will review shortly and get back to you (sorry for late reply I was on holidays over December).

mrdbourke avatar Jan 11 '24 04:01 mrdbourke

I finished the next module. I found another point that needs the same change. Just wanted to ask before you approve, wait until the weekend. Now I can push changes to the next module.

sc0v0ne avatar Jan 11 '24 10:01 sc0v0ne

@mrdbourke Good morning, I fixed the other notebooks and the helper_functions.py file that had the function that was updated. I ran the notebooks to check for any errors, it went through all the cell steps and the result of the tensorboard that appears in the notebook on the board.

sc0v0ne avatar Jan 14 '24 14:01 sc0v0ne

I'm a little late to the party, but I noticed you all aren't using a notebook review tool and wanted to invite you to review this pull request with GitNotebooks: https://gitnotebooks.com/mrdbourke/tensorflow-deep-learning/pull/611

It lets you do things like comment on rendered markdown and code cells, so might be an easy win for your PR reviews.

smith-kyle avatar Feb 05 '24 21:02 smith-kyle