Conversation
|
thanks @Smit-create this looks great. Would you mind to resolve the minor conflicts and then I will merge. |
@ kp992 would you mind to look into this? |
|
Thanks @mmcky. I have rebased it and made the changes locally but seems like a permission issue to push it on this branch. |
|
thanks @kp992 you are now a member of the |
|
I have merged the main branch into this branch, but when I try to push, I get the above error. |
|
Oh I see -- try now? |
|
Done @mmcky, thank you so much. |
|
thanks @Smit-create I will do a final review and merge once #361 and #355 are finalised |
mmcky
left a comment
There was a problem hiding this comment.
thanks for this tidy up @Smit-create
|
@Smit-create I am just running a full run without the build cache to test execution given there are some imports removed. |
|
Seems like everything is looking good. Thanks @mmcky |
mmcky
left a comment
There was a problem hiding this comment.
thanks @Smit-create -- this is great.
Please remember to fill in the top comment box with a description of the changes. I have added this to this PR. Thank you.
|
Thanks @mmcky |
This PR
%matplotlib inlineas no longer required+++markers introduced byjupytextconversioncc @mmcky